Skip to content

Commit

Permalink
MDL-40308: Fix javascript for lti mod_form
Browse files Browse the repository at this point in the history
It was toggling the state of the hidden input - not the checkbox.
  • Loading branch information
Damyon Wiese authored and danpoltawski committed Jul 4, 2013
1 parent 1619a06 commit 76e6fbd
Showing 1 changed file with 3 additions and 5 deletions.
8 changes: 3 additions & 5 deletions mod/lti/mod_form.js
Expand Up @@ -190,12 +190,10 @@

var setting, control;

// Can't look these up by ID as they seem to get random IDs.
// Setting an id manually from mod_form made them turn into text boxes.
var privacyControls = {
sendname: Y.one('input[name=instructorchoicesendname]'),
sendemailaddr: Y.one('input[name=instructorchoicesendemailaddr]'),
acceptgrades: Y.one('input[name=instructorchoiceacceptgrades]')
sendname: Y.one('#id_instructorchoicesendname'),
sendemailaddr: Y.one('#id_instructorchoicesendemailaddr'),
acceptgrades: Y.one('#id_instructorchoiceacceptgrades')
};

// Store a copy of user entered privacy settings as we may overwrite them
Expand Down

0 comments on commit 76e6fbd

Please sign in to comment.