MDL-41417 course: fixed idnumbertaken string
This commit is contained in:
@@ -343,7 +343,7 @@ class course_edit_form extends moodleform {
|
||||
if (!empty($data['idnumber']) && (empty($data['id']) || $this->course->idnumber != $data['idnumber'])) {
|
||||
if ($course = $DB->get_record('course', array('idnumber' => $data['idnumber']), '*', IGNORE_MULTIPLE)) {
|
||||
if (empty($data['id']) || $course->id != $data['id']) {
|
||||
$errors['idnumber'] = get_string('courseidnumbertaken', 'error', $course->fullname);
|
||||
$errors['idnumber'] = get_string('idnumbertaken', 'error');
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
+1
-1
@@ -3972,7 +3972,7 @@ function update_course($data, $editoroptions = NULL) {
|
||||
// Check we don't have a duplicate idnumber.
|
||||
if (!empty($data->idnumber) && $oldcourse->idnumber != $data->idnumber) {
|
||||
if ($DB->record_exists('course', array('idnumber' => $data->idnumber))) {
|
||||
throw new moodle_exception('courseidnumbertaken', '', '', $data->idnumber);
|
||||
throw new moodle_exception('idnumbertaken', 'error');
|
||||
}
|
||||
}
|
||||
|
||||
|
||||
@@ -130,7 +130,7 @@ class courselib_testcase extends advanced_testcase {
|
||||
update_course($created2);
|
||||
$this->fail('Expected exception when trying to update a course with duplicate idnumber');
|
||||
} catch (moodle_exception $e) {
|
||||
$this->assertEquals(get_string('courseidnumbertaken', 'error', $created2->idnumber), $e->getMessage());
|
||||
$this->assertEquals(get_string('idnumbertaken', 'error'), $e->getMessage());
|
||||
}
|
||||
|
||||
// Test duplicate shortname.
|
||||
|
||||
Reference in New Issue
Block a user