Permalink
Browse files

MDL-40308: Fix javascript for lti mod_form

It was toggling the state of the hidden input - not the checkbox.
  • Loading branch information...
damyon authored and danpoltawski committed Jul 4, 2013
1 parent 77182cd commit 392c581c45768d5d928ef47f1805d3aac649290c
Showing with 3 additions and 5 deletions.
  1. +3 −5 mod/lti/mod_form.js
View
@@ -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

0 comments on commit 392c581

Please sign in to comment.