From dc580207941d170339666a55d3cd6fc49830e208 Mon Sep 17 00:00:00 2001
From: nicolasconnault <nicolasconnault>
Date: Tue, 28 Jul 2009 11:24:57 +0000
Subject: [PATCH] MDL-19756 Refactored JS code from SCORM and choice modules
 into the core function submit_form_by_id()

---
 lib/javascript-static.js | 30 +++++++++++++++++++++++++-----
 mod/choice/choice.js     |  5 -----
 mod/choice/lib.php       |  3 +--
 mod/scorm/report.php     |  3 +--
 mod/scorm/scorm.js       |  5 -----
 5 files changed, 27 insertions(+), 19 deletions(-)
 delete mode 100644 mod/choice/choice.js
 delete mode 100644 mod/scorm/scorm.js

diff --git a/lib/javascript-static.js b/lib/javascript-static.js
index 5e45a675f6..0f4de7a720 100644
--- a/lib/javascript-static.js
+++ b/lib/javascript-static.js
@@ -205,17 +205,37 @@ function lockoptionsallsetup(formid) {
     return lockoptionsall(formid);
 }
 
-function submit_form_by_id(e, args) { 
+/**
+ * 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) {
+    if (!theform) {
         return false;
     }
-    if(theform.tagName.toLowerCase() != 'form') {
+    if (theform.tagName.toLowerCase() != 'form') {
         return false;
     }
-    if(!theform.onsubmit || theform.onsubmit()) {
-        return theform.submit();
+    if (args.selectid) {
+        var select = document.getElementById(args.selectid);
+        if (select.selectedIndex == 0) {
+            return false;
+        }
     }
+    return theform.submit();
 }
 
 /**
diff --git a/mod/choice/choice.js b/mod/choice/choice.js
deleted file mode 100644
index 19897bd5d2..0000000000
--- a/mod/choice/choice.js
+++ /dev/null
@@ -1,5 +0,0 @@
-function submit_attempts_form(e, args) {
-    if (e.target.selectedIndex > 0) {
-        submit_form_by_id(null, {id: 'attemptsform'});
-    }
-}
diff --git a/mod/choice/lib.php b/mod/choice/lib.php
index b832cffd2a..5f1db1d42a 100644
--- a/mod/choice/lib.php
+++ b/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>';
diff --git a/mod/scorm/report.php b/mod/scorm/report.php
index cb474a2053..a03a715035 100755
--- a/mod/scorm/report.php
+++ b/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>';
diff --git a/mod/scorm/scorm.js b/mod/scorm/scorm.js
deleted file mode 100644
index 19897bd5d2..0000000000
--- a/mod/scorm/scorm.js
+++ /dev/null
@@ -1,5 +0,0 @@
-function submit_attempts_form(e, args) {
-    if (e.target.selectedIndex > 0) {
-        submit_form_by_id(null, {id: 'attemptsform'});
-    }
-}
-- 
2.39.5