From 8ab94a36fe4956fba5d487732834339318d1d07c Mon Sep 17 00:00:00 2001
From: samhemelryk <samhemelryk>
Date: Fri, 26 Jun 2009 05:55:55 +0000
Subject: [PATCH] mod-chat MDL-16706 Removed inline JS and replaced with new
 PAGE methods

---
 mod/chat/gui_header_js/chat_gui_header.js | 24 +++++++++++
 mod/chat/gui_header_js/chatinput.php      |  2 +-
 mod/chat/gui_header_js/users.php          | 52 ++++-------------------
 3 files changed, 33 insertions(+), 45 deletions(-)

diff --git a/mod/chat/gui_header_js/chat_gui_header.js b/mod/chat/gui_header_js/chat_gui_header.js
index 7c7540054d..ddfe2614a3 100644
--- a/mod/chat/gui_header_js/chat_gui_header.js
+++ b/mod/chat/gui_header_js/chat_gui_header.js
@@ -18,4 +18,28 @@ function enableForm() {
     waitFlag = false;
     input_chat_message.className = '';
     input_chat_message.focus();
+}
+
+var timer = null
+var f = 1; //seconds
+function stop() {
+    clearTimeout(timer)
+}
+
+function start() {
+    timer = setTimeout("update()", f*1000);
+}
+
+function update() {
+    for(i=0; i<uidles.length; i++) {
+        el = document.getElementById(uidles[i]);
+        if (el != null) {
+            parts = el.innerHTML.split(":");
+            time = f + (parseInt(parts[0], 10)*60) + parseInt(parts[1], 10);
+            min = Math.floor(time/60);
+            sec = time % 60;
+            el.innerHTML = ((min < 10) ? "0" : "") + min + ":" + ((sec < 10) ? "0" : "") + sec;
+        }
+    }
+    timer = setTimeout("update()", f*1000);
 }
\ No newline at end of file
diff --git a/mod/chat/gui_header_js/chatinput.php b/mod/chat/gui_header_js/chatinput.php
index 2729a6e18e..9e3a648e51 100644
--- a/mod/chat/gui_header_js/chatinput.php
+++ b/mod/chat/gui_header_js/chatinput.php
@@ -31,7 +31,7 @@
 
     //Setup course, lang and theme
     $PAGE->set_course($course);
-    $PAGE->requires->js('mod/chat/gui_header_js/chat_gui_header.js');
+    $PAGE->requires->js('mod/chat/gui_header_js/chat_gui_header.js')->in_head();
 
     print_header('', '', '', 'input_chat_message', '', false);
 
diff --git a/mod/chat/gui_header_js/users.php b/mod/chat/gui_header_js/users.php
index a8f976b695..18f2beb762 100644
--- a/mod/chat/gui_header_js/users.php
+++ b/mod/chat/gui_header_js/users.php
@@ -57,54 +57,18 @@
         print_error('errornousers', 'chat');
     }
 
-    ob_start();
-    ?>
-    <script type="text/javascript">
-    //<![CDATA[
-    var timer = null
-    var f = 1; //seconds
-    var uidles = new Array(<?php echo count($chatusers) ?>);
-    <?php
-        $i = 0;
-        foreach ($chatusers as $chatuser) {
-            echo "uidles[$i] = 'uidle{$chatuser->id}';\n";
-            $i++;
-        }
-    ?>
-
-    function stop() {
-        clearTimeout(timer)
-    }
-
-    function start() {
-        timer = setTimeout("update()", f*1000);
-    }
-
-    function update() {
-        for(i=0; i<uidles.length; i++) {
-            el = document.getElementById(uidles[i]);
-            if (el != null) {
-                parts = el.innerHTML.split(":");
-                time = f + (parseInt(parts[0], 10)*60) + parseInt(parts[1], 10);
-                min = Math.floor(time/60);
-                sec = time % 60;
-                el.innerHTML = ((min < 10) ? "0" : "") + min + ":" + ((sec < 10) ? "0" : "") + sec;
-            }
-        }
-        timer = setTimeout("update()", f*1000);
+    $uidles = Array();
+    $i = 0;
+    foreach ($chatusers as $chatuser) {
+        $uidles[$i] = 'uidle{$chatuser->id}';
+        $i++;
     }
-    //]]>
-    </script>
-    <?php
-
-
-    /// Print headers
-    $meta = ob_get_clean();
-
+    $PAGE->requires->data_for_js('uidles', $uidles)->in_head();
+    $PAGE->requires->js('mod/chat/gui_header_js/chat_gui_header.js')->in_head();
 
     // Use ob to support Keep-Alive
     ob_start();
-    print_header('', '', '', '', $meta, false, '', '', false, 'onload="start()" onunload="stop()"');
+    print_header('', '', '', '', '', false, '', '', false, 'onload="start()" onunload="stop()"');
 
 
     /// Print user panel body
-- 
2.39.5