fix for bug that value of restrictmodules field wasn't available which breaks disabli...

diff --git a/course/edit2.php b/course/edit2.php
index 794ebbdbfb9e13851f4dc9064750832325da3185..da9c2c0c4fd0e055137f2e6bdfe91088af5a73ea 100644 (file)
--- 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
index 7509bf4caf3da4ec5bc786e310ad322973e4b8bd..963a9afb54c238369ca41fef40013ce06c573a14 100644 (file)
--- 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);