From a39fe289b0c07e92a493ccf07f68060b726f15cc Mon Sep 17 00:00:00 2001
From: samhemelryk <samhemelryk>
Date: Fri, 26 Jun 2009 07:32:52 +0000
Subject: [PATCH] mod-feedback MDL-16706 Removed inline JS and replaced with
 new PAGE methods

---
 mod/feedback/analysis_course.php | 12 +-----------
 mod/feedback/feedback.js         |  6 ++++++
 2 files changed, 7 insertions(+), 11 deletions(-)

diff --git a/mod/feedback/analysis_course.php b/mod/feedback/analysis_course.php
index 1d3d90a3ca..3f7a5b7c90 100644
--- a/mod/feedback/analysis_course.php
+++ b/mod/feedback/analysis_course.php
@@ -133,17 +133,7 @@
         echo '<input type="hidden" name="courseitemfilter" value="'.$courseitemfilter.'" />';
         echo '<input type="hidden" name="courseitemfiltertyp" value="'.$courseitemfiltertyp.'" />';
         echo '<input type="hidden" name="courseid" value="'.$courseid.'" />';
-        echo '<script language="javascript" type="text/javascript">
-                <!--
-                function setcourseitemfilter(item, item_typ) {
-                    document.report.courseitemfilter.value = item;
-                    document.report.courseitemfiltertyp.value = item_typ;
-                    document.report.submit();
-                }
-                -->
-                </script>';
-
-
+        $PAGE->requires->js('mod/feedback/feedback.js')->asap();
         $sql = 'select c.id, c.shortname from {course} c, '.
                                               '{feedback_value} fv, {feedback_item} fi '.
                                               'where c.id = fv.course_id and fv.item = fi.id '.
diff --git a/mod/feedback/feedback.js b/mod/feedback/feedback.js
index 2bb1b6d3f0..40b348fa8a 100644
--- a/mod/feedback/feedback.js
+++ b/mod/feedback/feedback.js
@@ -8,4 +8,10 @@ function set_item_focus(itemid) {
 function feedbackGo2delete(form) {
     form.action = moodle_cfg.wwwroot+'/mod/feedback/delete_completed.php';
     form.submit();
+}
+
+function setcourseitemfilter(item, item_typ) {
+    document.report.courseitemfilter.value = item;
+    document.report.courseitemfiltertyp.value = item_typ;
+    document.report.submit();
 }
\ No newline at end of file
-- 
2.39.5