fix for bug that value of restrictmodules field wasn't available whic…

…h breaks disabling of allowedmods multiple select
moodle · Oct 9, 2006 · 0d26ea7 · 0d26ea7
1 parent a44091b
commit 0d26ea7
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/course/edit2.php b/course/edit2.php
@@ -164,9 +164,9 @@
     if(!isset($default->defaultrole)) {
         $default->defaultrole = 0;  // Use site default
     }
-    include("edit_form.php");
     $mform->setDefaults((array)$default);
-
+    include("edit_form.php");
+
 /*    //setconstant overrides data coming from the form as well as default
     //we use this to pass data into the form which we've already processed.
     $mform->setConstants(array('id'=>$id,

diff --git a/course/edit_form.php b/course/edit_form.php
@@ -204,7 +204,9 @@
                             "((this.selectedIndex==0)?true:false);"));
         $mods = array(0=>get_string('allownone'));
         $mods += get_records_menu("modules", '','','','id, name');
-        $disabled=($mform->exportValue('restrictmodules')===0) ? array('disabled' => 'disabled') : array();
+        $disabled=($mform->exportValue('restrictmodules')==1) ? array() :array('disabled' => 'disabled') ;
+
+
         $mform->addElement('select', 'allowedmods', get_string("to"),$mods,
                         array('multiple'=>"multiple", 'size'=>"10", 'id'=>"allowedmods")
                                 +$disabled);