MDL-30344 lti - Hiding roster permission from the UI as it's not implemented in LTI...
authorChris Scribner <scriby@gmail.com>
Fri, 18 Nov 2011 00:07:36 +0000 (19:07 -0500)
committerEloy Lafuente (stronk7) <stronk7@moodle.org>
Mon, 21 Nov 2011 15:42:28 +0000 (16:42 +0100)
mod/lti/ajax.php
mod/lti/edit_form.php
mod/lti/mod_form.js
mod/lti/mod_form.php

index bf1ba74..f29eafb 100644 (file)
@@ -63,7 +63,7 @@ switch ($action) {
                 FROM {lti_types_config}
                 WHERE
                     typeid = :typeid
-                AND name IN (\'sendname\', \'sendemailaddr\', \'acceptgrades\', \'allowroster\')
+                AND name IN (\'sendname\', \'sendemailaddr\', \'acceptgrades\')
             ';
                         
             $privacyconfigs = $DB->get_records_sql($query, array('typeid' => $toolid));
index 8600597..3cf2ef8 100644 (file)
@@ -126,9 +126,9 @@ class mod_lti_edit_types_form extends moodleform{
         $mform->addHelpButton('lti_acceptgrades', 'accept_grades_admin', 'lti');
 
         // Add grading preferences fieldset where the tool is allowed to retrieve rosters
-        $mform->addElement('select', 'lti_allowroster', get_string('share_roster_admin', 'lti'), $options);
-        $mform->setDefault('lti_allowroster', '2');
-        $mform->addHelpButton('lti_allowroster', 'share_roster_admin', 'lti');
+        //$mform->addElement('select', 'lti_allowroster', get_string('share_roster_admin', 'lti'), $options);
+        //$mform->setDefault('lti_allowroster', '2');
+        //$mform->addHelpButton('lti_allowroster', 'share_roster_admin', 'lti');
 
         $mform->addElement('checkbox', 'lti_forcessl', '&nbsp;', ' ' . get_string('force_ssl', 'lti'), $options);
         $mform->setDefault('lti_forcessl', '0');
index 1e5cf23..e068eec 100644 (file)
                 toolInfo = {
                     sendname: M.mod_lti.LTI_SETTING_DELEGATE,
                     sendemailaddr: M.mod_lti.LTI_SETTING_DELEGATE,
-                    acceptgrades: M.mod_lti.LTI_SETTING_DELEGATE,
-                    allowroster: M.mod_lti.LTI_SETTING_DELEGATE
+                    acceptgrades: M.mod_lti.LTI_SETTING_DELEGATE
                 }
             }
             
             var privacyControls = {
                 sendname: Y.one('input[name=instructorchoicesendname]'),
                 sendemailaddr: Y.one('input[name=instructorchoicesendemailaddr]'),
-                acceptgrades: Y.one('input[name=instructorchoiceacceptgrades]'),
-                allowroster: Y.one('input[name=instructorchoiceallowroster]')
+                acceptgrades: Y.one('input[name=instructorchoiceacceptgrades]')
             };
             
             //Store a copy of user entered privacy settings as we may overwrite them
index 1d4d8ba..ec1b5bf 100644 (file)
@@ -164,9 +164,9 @@ class mod_lti_mod_form extends moodleform_mod {
         $mform->setDefault('instructorchoiceacceptgrades', '1');
         $mform->addHelpButton('instructorchoiceacceptgrades', 'accept_grades', 'lti');
 
-        $mform->addElement('checkbox', 'instructorchoiceallowroster', '&nbsp;', ' ' . get_string('share_roster', 'lti'));
-        $mform->setDefault('instructorchoiceallowroster', '1');
-        $mform->addHelpButton('instructorchoiceallowroster', 'share_roster', 'lti');
+        //$mform->addElement('checkbox', 'instructorchoiceallowroster', '&nbsp;', ' ' . get_string('share_roster', 'lti'));
+        //$mform->setDefault('instructorchoiceallowroster', '1');
+        //$mform->addHelpButton('instructorchoiceallowroster', 'share_roster', 'lti');
 
         //-------------------------------------------------------------------------------