From 8f36c245ed278378ba379181a2c8e4ebd2252647 Mon Sep 17 00:00:00 2001
From: samhemelryk <samhemelryk>
Date: Thu, 2 Jul 2009 06:37:30 +0000
Subject: [PATCH] mod-assignment MDL-16706 Eliminated inline scripts to use
 PAGE methods

---
 mod/assignment/assignment.js | 11 +++++++++++
 mod/assignment/lib.php       | 19 +++----------------
 2 files changed, 14 insertions(+), 16 deletions(-)
 create mode 100644 mod/assignment/assignment.js

diff --git a/mod/assignment/assignment.js b/mod/assignment/assignment.js
new file mode 100644
index 0000000000..0a028faf80
--- /dev/null
+++ b/mod/assignment/assignment.js
@@ -0,0 +1,11 @@
+function setNext(){
+    document.getElementById('submitform').mode.value = 'next';
+    document.getElementById('submitform').userid.value = assignment.nextid;
+}
+
+function saveNext(){
+    document.getElementById('submitform').mode.value = 'saveandnext';
+    document.getElementById('submitform').userid.value = assignment.nextid;
+    document.getElementById('submitform').saveuserid.value = assignment.userid;
+    document.getElementById('submitform').menuindex.value = document.getElementById('submitform').grade.selectedIndex;
+}
\ No newline at end of file
diff --git a/mod/assignment/lib.php b/mod/assignment/lib.php
index b6f4609273..b569cbdabe 100644
--- a/mod/assignment/lib.php
+++ b/mod/assignment/lib.php
@@ -847,7 +847,7 @@ class assignment_base {
      * @param string $extra_javascript
      */
     function display_submission($extra_javascript = '') {
-        global $CFG, $DB;
+        global $CFG, $DB, $PAGE;
         require_once($CFG->libdir.'/gradelib.php');
         require_once($CFG->libdir.'/tablelib.php');
 
@@ -919,22 +919,9 @@ class assignment_base {
         /// Print any extra javascript needed for saveandnext
         echo $extra_javascript;
 
-        ///SOme javascript to help with setting up >.>
-
-        echo '<script type="text/javascript">'."\n";
-        echo 'function setNext(){'."\n";
-        echo 'document.getElementById(\'submitform\').mode.value=\'next\';'."\n";
-        echo 'document.getElementById(\'submitform\').userid.value="'.$nextid.'";'."\n";
-        echo '}'."\n";
-
-        echo 'function saveNext(){'."\n";
-        echo 'document.getElementById(\'submitform\').mode.value=\'saveandnext\';'."\n";
-        echo 'document.getElementById(\'submitform\').userid.value="'.$nextid.'";'."\n";
-        echo 'document.getElementById(\'submitform\').saveuserid.value="'.$userid.'";'."\n";
-        echo 'document.getElementById(\'submitform\').menuindex.value = document.getElementById(\'submitform\').grade.selectedIndex;'."\n";
-        echo '}'."\n";
+        echo $PAGE->requires->data_for_js('assignment', Array('nextid'=>$nextid, 'userid'=>$userid))->asap();
+        echo $PAGE->requires->js('mod/assignment/assignment.js')->asap();
 
-        echo '</script>'."\n";
         echo '<table cellspacing="0" class="feedback '.$subtype.'" >';
 
         ///Start of teacher info row
-- 
2.39.5