MDL-24321 switching to stdClass in /c*/
[moodle.git] / cohort / edit_form.php
index c81c0c5..fe3f357 100644 (file)
 /**
  * Cohort related management functions, this file needs to be included manually.
  *
- * @package    moodlecore
+ * @package    core
  * @subpackage cohort
- * @copyright  2010 Petr Skoda  (info@skodak.org)
+ * @copyright  2010 Petr Skoda  {@link http://skodak.org}
  * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
 
+if (!defined('MOODLE_INTERNAL')) {
+    die('Direct access to this script is forbidden.');    ///  It must be included from a Moodle page
+}
+
 require_once($CFG->dirroot . '/lib/formslib.php');
 
 class cohort_edit_form extends moodleform {
@@ -64,13 +68,17 @@ class cohort_edit_form extends moodleform {
         $errors = parent::validation($data, $files);
 
         $idnumber = trim($data['idnumber']);
-        if ($data['id']) {
+        if ($idnumber === '') {
+            // fine, empty is ok
+
+        } else if ($data['id']) {
             $current = $DB->get_record('cohort', array('id'=>$data['id']), '*', MUST_EXIST);
             if ($current->idnumber !== $idnumber) {
                 if ($DB->record_exists('cohort', array('idnumber'=>$idnumber))) {
                     $errors['idnumber'] = get_string('duplicateidnumber', 'cohort');
                 }
             }
+
         } else {
             if ($DB->record_exists('cohort', array('idnumber'=>$idnumber))) {
                 $errors['idnumber'] = get_string('duplicateidnumber', 'cohort');
@@ -93,7 +101,7 @@ class cohort_edit_form extends moodleform {
             $context = get_context_instance(CONTEXT_COURSECAT, $cid, MUST_EXIST);
             $options[$context->id] = $name;
         }
-        // always add current - this is not likely, but if the logic get's changed it might be a problem
+        // always add current - this is not likely, but if the logic gets changed it might be a problem
         if (!isset($options[$currentcontextid])) {
             $context = get_context_instance_by_id($currentcontextid, MUST_EXIST);
             $options[$context->id] = print_context_name($syscontext);
@@ -101,4 +109,4 @@ class cohort_edit_form extends moodleform {
         return $options;
     }
 }
-    
\ No newline at end of file
+