MDL-19756 Refactored JS code from SCORM and choice modules into the core function submit_form_by_id()

lib/javascript-static.js

@@ -205,6 +205,22 @@ function lockoptionsallsetup(formid) {
     return lockoptionsall(formid);
 }
 
+/**
+ * Helper function mainly for drop-down menus' onchange events,
+ * submits the form designated by args.id. If args.selectid is also
+ * given, it only submits the form if the selected <option> is not 
+ * the first one (usually the "Choose..." option)
+ * Example usage of the moodle_select_menu component with this function:
+ * <pre>
+ * $selectmenu = new moodle_select_menu();
+ * $selectmenu->options = array('delete' => get_string('delete'));
+ * $selectmenu->name = 'action';
+ * $selectmenu->button->label = get_string('withselected', 'quiz');
+ * $selectmenu->id = 'menuaction';
+ * $selectmenu->add_action('change', 'submit_form_by_id', array('id' => 'attemptsform', 'selectid' => 'menuaction'));
+ * echo $OUTPUT->select_menu($selectmenu);
+ * </pre>
+ */
 function submit_form_by_id(e, args) {
     var theform = document.getElementById(args.id);
     if (!theform) {
@@ -213,10 +229,14 @@ function submit_form_by_id(e, args) {
     if (theform.tagName.toLowerCase() != 'form') {
         return false;
     }
-    if(!theform.onsubmit || theform.onsubmit()) {
+    if (args.selectid) {
-        return theform.submit();
+        var select = document.getElementById(args.selectid);
+        if (select.selectedIndex == 0) {
+            return false;
         }
     }
+    return theform.submit();
+}
 
 /**
  * Either check, or uncheck, all checkboxes inside the element with id is

mod/choice/choice.js

@@ -1,5 +0,0 @@
-function submit_attempts_form(e, args) {
-    if (e.target.selectedIndex > 0) {
-        submit_form_by_id(null, {id: 'attemptsform'});
-    }
-}

mod/choice/lib.php

@@ -544,13 +544,12 @@ function choice_show_results($choice, $course, $cm, $allresponses, $forcepublish
                 echo '<a href="javascript:select_all_in(\'DIV\',null,\'tablecontainer\');">'.get_string('selectall', 'quiz').'</a> / ';
                 echo '<a href="javascript:deselect_all_in(\'DIV\',null,\'tablecontainer\');">'.get_string('selectnone', 'quiz').'</a> ';
                 echo '&nbsp;&nbsp;';
-                $PAGE->requires->js('mod/choice/choice.js');
                 $selectmenu = new moodle_select_menu();
                 $selectmenu->options = array('delete' => get_string('delete'));
                 $selectmenu->name = 'action';
                 $selectmenu->button->label = get_string('withselected', 'quiz');
                 $selectmenu->id = 'menuaction';
-                $selectmenu->add_action('change', 'submit_attempts_form');
+                $selectmenu->add_action('change', 'submit_form_by_id', array('id' => 'attemptsform', 'selectid' => 'menuaction'));
                 echo $OUTPUT->select_menu($selectmenu);
                 echo '<noscript id="noscriptmenuaction" style="display: inline;">';
                 echo '<div>';

mod/scorm/report.php

@@ -198,13 +198,12 @@
                     echo '<a href="javascript:select_all_in(\'DIV\',null,\'scormtablecontainer\');">'.get_string('selectall', 'quiz').'</a> / ';
                     echo '<a href="javascript:deselect_all_in(\'DIV\',null,\'scormtablecontainer\');">'.get_string('selectnone', 'quiz').'</a> ';
                     echo '&nbsp;&nbsp;';
-                    $PAGE->requires->js('mod/scorm/scorm.js');
                     $selectmenu = new moodle_select_menu();
                     $selectmenu->options = array('delete' => get_string('delete'));
                     $selectmenu->name = 'action';
                     $selectmenu->button->label = get_string('withselected', 'quiz');
                     $selectmenu->id = 'menuaction';
-                    $selectmenu->add_action('change', 'submit_attempts_form');
+                    $selectmenu->add_action('change', 'submit_form_by_id', array('id' => 'attemptsform', 'selectid' => 'menuaction'));
                     echo $OUTPUT->select_menu($selectmenu);
                     echo '<noscript id="noscriptmenuaction" style="display: inline;">';
                     echo '<div>';

mod/scorm/scorm.js

@@ -1,5 +0,0 @@
-function submit_attempts_form(e, args) {
-    if (e.target.selectedIndex > 0) {
-        submit_form_by_id(null, {id: 'attemptsform'});
-    }
-}