From f4ba7e1a54f4798ddc27fa4e5f14d11ac64cc4a0 Mon Sep 17 00:00:00 2001
From: moodler <moodler>
Date: Wed, 3 Jan 2007 14:44:17 +0000
Subject: [PATCH] Bend over for XHTML 1.0 Strict ...  MDL-8049

Removed Javascript language attribute tags and added CDATA
---
 admin/xmldb/javascript.php                    |   4 +-
 backup/backup_form.html                       |   2 +-
 backup/restore_form.html                      |   2 +-
 blocks/loancalc/block_loancalc.php            |   6 +-
 blog/tags.php                                 |   4 +-
 calendar/export_basic.html                    |   2 +-
 files/index.php                               |  12 +-
 grade/exceptions.html                         |   8 +-
 lib/editor/htmlarea/htmlarea.class.php        |  16 +-
 lib/editor/htmlarea/popups/createanchor.php   |  10 +-
 lib/editor/htmlarea/popups/dlg_ins_char.php   |   6 +-
 lib/editor/htmlarea/popups/dlg_ins_smile.php  |  10 +-
 lib/editor/htmlarea/popups/insert_image.php   |   4 +-
 .../htmlarea/popups/insert_image_std.php      |   6 +-
 lib/editor/htmlarea/popups/insert_table.php   |   7 +-
 lib/editor/htmlarea/popups/link.php           |   8 +-
 lib/editor/htmlarea/popups/link_std.php       |   6 +-
 .../htmlarea/popups/searchandreplace.php      |   8 +-
 lib/editor/tinymce/tinymce.class.php          |   8 +-
 lib/formslib.php                              |   2 +-
 lib/moodlelib.php                             | 342 ++----------------
 lib/soap/nusoap.php                           |   6 +-
 lib/speller/server-scripts/spellchecker.php   |  10 +-
 lib/weblib.php                                |  10 +-
 message/index.php                             |   2 +-
 mod/assignment/mod.html                       |   4 +-
 mod/chat/chatd.php                            |   6 +-
 mod/chat/gui_header_js/chatinput.php          |   2 +-
 mod/chat/gui_header_js/insert.php             |   4 +-
 mod/chat/gui_header_js/jsupdated.php          |   4 +-
 mod/chat/gui_header_js/users.php              |   2 +-
 mod/data/templates.php                        |   2 +-
 mod/hotpot/lib.php                            |   8 +-
 mod/hotpot/mod.html                           |   6 +-
 mod/hotpot/report/fullstat/report.php         |  12 +-
 mod/hotpot/show.php                           |   6 +-
 mod/hotpot/view.php                           |   4 +-
 mod/lams/mod.html                             |   6 +-
 mod/quiz/attempt.php                          |  12 +-
 mod/quiz/attempt_close_js.php                 |   6 +-
 mod/quiz/config.html                          |   6 +-
 mod/quiz/jsclock.php                          |   6 +-
 mod/quiz/jstimer.php                          |  14 +-
 mod/quiz/protect_js.php                       |   2 +-
 mod/quiz/view.php                             |   8 +-
 mod/resource/type/file/file.html              |  14 +-
 mod/resource/type/file/localfile.php          |   6 +-
 mod/resource/type/file/localpath.php          |  12 +-
 mod/resource/type/html/html.html              |  14 +-
 mod/resource/type/ims/ims.html                |  20 +-
 .../type/repository/hive/makelink.php         |   6 +-
 .../type/repository/hive/openhive.php         |   4 +-
 mod/resource/type/repository/repository.html  |  18 +-
 mod/resource/type/text/text.html              |  14 +-
 mod/scorm/datamodels/aicclib.php              |   8 +-
 mod/scorm/datamodels/scorm_12lib.php          |   8 +-
 mod/scorm/datamodels/scorm_13lib.php          |   8 +-
 mod/scorm/loadSCO.php                         |   8 +-
 mod/scorm/mod.html                            |   6 +-
 mod/scorm/player.php                          |  17 +-
 mod/wiki/ewiki/ewiki.php                      |   2 +-
 mod/wiki/ewiki/fragments/homepage.src         |   4 +-
 question/type/calculated/editquestion.html    |   4 +-
 theme/standardwhite/footer.html               |  16 +-
 64 files changed, 308 insertions(+), 502 deletions(-)

diff --git a/admin/xmldb/javascript.php b/admin/xmldb/javascript.php
index abf86cd597..4513c97d38 100644
--- a/admin/xmldb/javascript.php
+++ b/admin/xmldb/javascript.php
@@ -42,14 +42,14 @@
         $file    = $CFG->dirroot . '/'.$CFG->admin.'/xmldb/actions/' . $action . '/' . $action . '.js';
         $wwwfile = $CFG->wwwroot . '/'.$CFG->admin.'/xmldb/actions/' . $action . '/' . $action . '.js';
         if (file_exists($file) && is_readable($file)) {
-            echo '<script language="JavaScript" type="text/javascript" src="' . $wwwfile . '"></script>' . "\n";
+            echo '<script type="text/javascript" src="' . $wwwfile . '"></script>' . "\n";
         } else {
         /// Try to load the postaction javascript if exists
             if ($postaction) {
                 $file    = $CFG->dirroot . '/'.$CFG->admin.'/xmldb/actions/' . $postaction . '/' . $postaction . '.js';
                 $wwwfile = $CFG->wwwroot . '/'.$CFG->admin.'/xmldb/actions/' . $postaction . '/' . $postaction . '.js';
                 if (file_exists($file) && is_readable($file)) {
-                    echo '<script language="JavaScript" type="text/javascript" src="' . $wwwfile . '"></script>' . "\n";
+                    echo '<script type="text/javascript" src="' . $wwwfile . '"></script>' . "\n";
                 }
             }
         }
diff --git a/backup/backup_form.html b/backup/backup_form.html
index 9d6bfd9aa8..534eac496e 100644
--- a/backup/backup_form.html
+++ b/backup/backup_form.html
@@ -72,7 +72,7 @@
 
 ?>
 
-<script language="JavaScript" type="text/javascript">
+<script type="text/javascript">
 <!--
 function selectItemInMenuByName(formId, menuName, selectIndex ) {
     myForm = document.getElementById(formId)
diff --git a/backup/restore_form.html b/backup/restore_form.html
index 7c9f2c7eb2..50094b1a14 100644
--- a/backup/restore_form.html
+++ b/backup/restore_form.html
@@ -118,7 +118,7 @@
 
 ?>
 
-<script language="JavaScript" type="text/javascript">
+<script type="text/javascript">
 <!--
 function selectItemInMenuByName(formId, menuName, selectIndex ) {
     myForm = document.getElementById(formId)
diff --git a/blocks/loancalc/block_loancalc.php b/blocks/loancalc/block_loancalc.php
index 8b17aeca92..df7ce8c589 100644
--- a/blocks/loancalc/block_loancalc.php
+++ b/blocks/loancalc/block_loancalc.php
@@ -1,4 +1,4 @@
-<?php
+<?php // $Id$
 
 class block_loancalc extends block_base {
 
@@ -14,7 +14,7 @@ class block_loancalc extends block_base {
         $calc = $CFG->pixpath.'/i/calc.gif';
         
         $this->content->text = '
-        <script language="JavaScript">
+        <script type="text/javascript">
         <![CDATA[
     function Next()
     {
@@ -203,4 +203,4 @@ function comp(v) { // general entry point for all cases
 
     }
 }
-?>
\ No newline at end of file
+?>
diff --git a/blog/tags.php b/blog/tags.php
index 5169173e90..3345851b9f 100755
--- a/blog/tags.php
+++ b/blog/tags.php
@@ -1,4 +1,4 @@
-<?php
+<?php  // $Id$
 require_once('../config.php');
 
 /// main switch for form processing to perform, add/delete etc
@@ -56,7 +56,7 @@ switch ($action) {
             }
 
             /// Write newly added tags back into window opener.
-            echo '<script language="JavaScript" type="text/javascript">
+            echo '<script type="text/javascript">
             var o = opener.document.createElement("option");
             o.innerHTML = "<option>'.$tag->text.'</option>";
             o.value = '.$tagid.';
diff --git a/calendar/export_basic.html b/calendar/export_basic.html
index 09a2114106..56e1da3012 100644
--- a/calendar/export_basic.html
+++ b/calendar/export_basic.html
@@ -26,7 +26,7 @@
 <input type="hidden" name="cal_y" value="" />
 <input type="hidden" name="username" value="<?php echo $username; ?>" />
 <input type="hidden" name="authtoken" value="<?php echo $authtoken; ?>" />
-<script type="text/javascript" language="JavaScript">
+<script type="text/javascript">
 //<![CDATA[
 function generate_url() {
     if (document.getElementById("pw_course").checked) {
diff --git a/files/index.php b/files/index.php
index a3b5cb88e5..4f3a555fbf 100644
--- a/files/index.php
+++ b/files/index.php
@@ -80,25 +80,25 @@
             $chooseparts = explode('.', $choose);
             if (count($chooseparts)==2){
             ?>
-            <script language="javascript" type="text/javascript">
-            <!--
+            <script type="text/javascript">
+            //<![CDATA[
             function set_value(txt) {
                 opener.document.forms['<?php echo $chooseparts[0]."'].".$chooseparts[1] ?>.value = txt;
                 window.close();
             }
-            -->
+            //]]>
             </script>
 
             <?php
             } elseif (count($chooseparts)==1){
             ?>
-            <script language="javascript" type="text/javascript">
-            <!--
+            <script type="text/javascript">
+            //<![CDATA[
             function set_value(txt) {
                 opener.document.getElementById('<?php echo $chooseparts[0] ?>').value = txt;
                 window.close();
             }
-            -->
+            //]]>
             </script>
 
             <?php
diff --git a/grade/exceptions.html b/grade/exceptions.html
index 8820dbec85..30feacea75 100644
--- a/grade/exceptions.html
+++ b/grade/exceptions.html
@@ -1,5 +1,5 @@
-<script language="JavaScript" type="text/javascript">
-<!-- Begin 
+<script type="text/javascript">
+//<![CDATA[ 
 
 <?php
     // javascript objects for excluded students
@@ -169,10 +169,10 @@ function updateMembers(selectgrade_item) {
       </td>
     </tr>
   </table>
-<script language="JavaScript">
+<script type="text/javascript">
 <!-- Begin
 <?php
  echo 'updateMembers('.$selectedgrade_item.');';
  ?>
-// end hiding script -->
+//]]>
 </script>
diff --git a/lib/editor/htmlarea/htmlarea.class.php b/lib/editor/htmlarea/htmlarea.class.php
index 1f33620402..7d4af69b8e 100644
--- a/lib/editor/htmlarea/htmlarea.class.php
+++ b/lib/editor/htmlarea/htmlarea.class.php
@@ -234,8 +234,8 @@ class htmlarea extends editorObject {
         }
 
         echo "\n";
-        echo '<script language="javascript" type="text/javascript" defer="defer">'."\n";
-        echo '<!--'."\n";
+        echo '<script type="text/javascript" defer="defer">'."\n";
+        echo '//<![CDATA['."\n";
         echo '    var config = new HTMLArea.Config();'."\n";
 
         foreach ( $conf as $key => $value ) {
@@ -298,7 +298,7 @@ class htmlarea extends editorObject {
         }
 
         echo '    HTMLArea.replaceAll(config);'."\n";
-        echo '// done hiding -->'."\n";
+        echo '//]]>'."\n";
         echo '</script>'."\n";
 
     }
@@ -340,15 +340,15 @@ class htmlarea extends editorObject {
     */
     function print_speller_code ($usehtmleditor=false) {
 
-        if(!$usehtmleditor) {
-            echo "\n".'<script language="javascript" type="text/javascript">'."\n";
+        echo "\n".'<script type="text/javascript">'."\n";
+        echo '//<![CDATA['."\n";
+        if (!$usehtmleditor) {
             echo 'function openSpellChecker() {'."\n";
             echo "\tvar speller = new spellChecker();\n";
             echo "\tspeller.popUpUrl = \"" . $this->cfg->wwwroot ."/lib/speller/spellchecker.html\";\n";
             echo "\tspeller.spellCheckScript = \"". $this->cfg->wwwroot ."/lib/speller/server-scripts/spellchecker.php\";\n";
             echo "\tspeller.spellCheckAll();\n";
             echo '}'."\n";
-            echo '</script>'."\n";
         } else {
             echo "\n\tfunction spellClickHandler(editor, buttonId) {\n";
             echo "\t\teditor._textArea.value = editor.getHTML();\n";
@@ -360,7 +360,9 @@ class htmlarea extends editorObject {
             echo "\t\tspeller.openChecker();\n\t";
             echo '}'."\n";
         }
+        echo '//]]>'."\n";
+        echo '</script>'."\n";
     }
 
 }
-?>
\ No newline at end of file
+?>
diff --git a/lib/editor/htmlarea/popups/createanchor.php b/lib/editor/htmlarea/popups/createanchor.php
index 93b54a909b..034be6c6f8 100644
--- a/lib/editor/htmlarea/popups/createanchor.php
+++ b/lib/editor/htmlarea/popups/createanchor.php
@@ -7,9 +7,9 @@
 <head>
 <meta http-equiv="content-type" content="text/html; charset=utf-8" />
 <title>Create anchor</title>
-<script language="javascript" type="text/javascript" src="popup.js"></script>
-<script language="javascript" type="text/javascript">
-<!--
+<script type="text/javascript" src="popup.js"></script>
+<script type="text/javascript">
+//<![CDATA[
 function Init() {
     __dlg_init();
 }
@@ -35,7 +35,7 @@ function onCancel() {
   __dlg_close(null);
   return false;
 };
-// -->
+//]]>
 </script>
 <style type="text/css">
 <!--
@@ -46,7 +46,7 @@ button { width: 70px; }
 border-bottom: 1px solid black; letter-spacing: 2px;
 }
 .note { font-size: 8pt; }
-// -->
+-->
 </style>
 </head>
 <body>
diff --git a/lib/editor/htmlarea/popups/dlg_ins_char.php b/lib/editor/htmlarea/popups/dlg_ins_char.php
index c18eb7f9f9..ea5b0bda3d 100644
--- a/lib/editor/htmlarea/popups/dlg_ins_char.php
+++ b/lib/editor/htmlarea/popups/dlg_ins_char.php
@@ -67,8 +67,8 @@ form { padding: 0px; margin: 0px; }
 font-size: small; }
 // -->
 </style>
-<script language="javascript" type="text/javascript" src="popup.js"></script>
-<script language="javascript" type="text/javascript">
+<script type="text/javascript" src="popup.js"></script>
+<script type="text/javascript">
 //<![CDATA[
 function Init() {
   __dlg_init();
@@ -122,7 +122,7 @@ function cancel() {
       <tr valign="top">
         <td>
 
-       <script language="javascript" type="text/javascript">
+       <script type="text/javascript">
        //<![CDATA[
        document.write(tab(7,32))
        //]]>
diff --git a/lib/editor/htmlarea/popups/dlg_ins_smile.php b/lib/editor/htmlarea/popups/dlg_ins_smile.php
index 962ba71759..8ace906949 100644
--- a/lib/editor/htmlarea/popups/dlg_ins_smile.php
+++ b/lib/editor/htmlarea/popups/dlg_ins_smile.php
@@ -40,9 +40,9 @@
 <meta http-equiv="content-type" content="text/html; charset=utf-8" />
 <title><?php print_string('insertsmile', 'editor'); ?></title>
 <link rel="stylesheet" href="dialog.css" type="text/css" />
-<script language="javascript" type="text/javascript" src="popup.js"></script>
-<script language="javascript" type="text/javascript">
-<!--
+<script type="text/javascript" src="popup.js"></script>
+<script type="text/javascript">
+//<![CDATA[
 function Init() {
   __dlg_init();
 }
@@ -65,7 +65,7 @@ function cancel() {
   __dlg_close(null);
   return false;
 };
-// -->
+//]]>
 </script>
 </head>
 <body onload="Init()">
@@ -122,4 +122,4 @@ function cancel() {
     <button type="button" onclick="return cancel();"><?php print_string('close', 'editor'); ?></button></td></tr>
 </table>
 </body>
-</html>
\ No newline at end of file
+</html>
diff --git a/lib/editor/htmlarea/popups/insert_image.php b/lib/editor/htmlarea/popups/insert_image.php
index 9fcd8a76cd..8adeae2824 100644
--- a/lib/editor/htmlarea/popups/insert_image.php
+++ b/lib/editor/htmlarea/popups/insert_image.php
@@ -17,8 +17,8 @@
 <head>
 <meta http-equiv="content-type" content="text/html; charset=utf-8" />
 <title><?php print_string("insertimage","editor");?></title>
-<script language="javascript" type="text/javascript" src="popup.js"></script>
-<script language="javascript" type="text/javascript">
+<script type="text/javascript" src="popup.js"></script>
+<script type="text/javascript">
 //<![CDATA[
 var preview_window = null;
 
diff --git a/lib/editor/htmlarea/popups/insert_image_std.php b/lib/editor/htmlarea/popups/insert_image_std.php
index f5c7048cd1..cbdb26359f 100644
--- a/lib/editor/htmlarea/popups/insert_image_std.php
+++ b/lib/editor/htmlarea/popups/insert_image_std.php
@@ -16,9 +16,10 @@
 <meta http-equiv="content-type" content="text/html; charset=utf-8" />
   <title><?php print_string("insertimage","editor");?></title>
 
-<script language="javascript" type="text/javascript" src="popup.js"></script>
+<script type="text/javascript" src="popup.js"></script>
 
-<script language="javascript" type="text/javascript">
+<script type="text/javascript">
+//<![CDATA[
 var preview_window = null;
 
 function Init() {
@@ -116,6 +117,7 @@ function onPreview() {
   win.focus();
   return false;
 };
+//]]>
 </script>
 
 <style type="text/css">
diff --git a/lib/editor/htmlarea/popups/insert_table.php b/lib/editor/htmlarea/popups/insert_table.php
index 162abf61d2..1dfa446c47 100644
--- a/lib/editor/htmlarea/popups/insert_table.php
+++ b/lib/editor/htmlarea/popups/insert_table.php
@@ -7,8 +7,9 @@
 <head>
 <meta http-equiv="content-type" content="text/html; charset=utf-8" />
   <title><?php print_string("inserttable","editor");?></title>
-<script language="javascript" type="text/javascript" src="popup.js"></script>
-<script language="javascript" type="text/javascript">
+<script type="text/javascript" src="popup.js"></script>
+<script type="text/javascript">
+//<![CDATA[
 
 function Init() {
   __dlg_init();
@@ -44,7 +45,7 @@ function onCancel() {
   __dlg_close(null);
   return false;
 };
-
+//[[>
 </script>
 
 <style type="text/css">
diff --git a/lib/editor/htmlarea/popups/link.php b/lib/editor/htmlarea/popups/link.php
index dd64ae2dd2..8e9279e9f8 100644
--- a/lib/editor/htmlarea/popups/link.php
+++ b/lib/editor/htmlarea/popups/link.php
@@ -13,12 +13,14 @@
 <head>
 <title><?php print_string("insertlink","editor");?></title>
 <meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
-<script language="javascript" type="text/javascript">
+<script type="text/javascript">
+//<![CDATA[
 
 function onCancel() {
   window.close();
   return false;
-};
+}
+
 function checkvalue(elm,formname) {
     var el = document.getElementById(elm);
     if(!el.value) {
@@ -42,7 +44,7 @@ function submit_form(dothis) {
     window.fbrowser.document.dirform.submit();
     return false;
 }
-
+//]]>
 </script>
 <style type="text/css">
 html, body { background-color: rgb(212,208,200); }
diff --git a/lib/editor/htmlarea/popups/link_std.php b/lib/editor/htmlarea/popups/link_std.php
index c3eec4a429..310d5d7f9e 100644
--- a/lib/editor/htmlarea/popups/link_std.php
+++ b/lib/editor/htmlarea/popups/link_std.php
@@ -14,8 +14,9 @@
 <head>
 <meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
   <title><?php print_string("insertlink","editor");?></title>
-  <script language="javascript" type="text/javascript" src="popup.js"></script>
-  <script language="javascript" type="text/javascript">
+  <script type="text/javascript" src="popup.js"></script>
+  <script type="text/javascript">
+//<![CDATA[
 function onTargetChanged() {
   var f = document.getElementById("f_other_target");
   if (this.value == "_other") {
@@ -120,6 +121,7 @@ function seturl() {
         f_url.value = txt;
     }
 }
+//]]>
 </script>
 <style type="text/css">
 html, body {
diff --git a/lib/editor/htmlarea/popups/searchandreplace.php b/lib/editor/htmlarea/popups/searchandreplace.php
index 0efb6c2a93..05492b515e 100644
--- a/lib/editor/htmlarea/popups/searchandreplace.php
+++ b/lib/editor/htmlarea/popups/searchandreplace.php
@@ -7,9 +7,9 @@
 <head>
 <meta http-equiv="content-type" content="text/html; charset=utf-8" />
 <title><?php print_string("searchandreplace","editor");?></title>
-<script language="javascript" type="text/javascript" src="popup.js"></script>
-<script language="javascript" type="text/javascript">
-<!--
+<script type="text/javascript" src="popup.js"></script>
+<script type="text/javascript">
+//<![CDATA[
 function _CloseOnEsc(ev) {
     ev || (ev = window.event) || (ev = editor._iframe.contentWindow.event);
     if (ev.keyCode == 27) {
@@ -65,7 +65,7 @@ function onCancel() {
   return false;
 };
 
--->
+//]]>
 </script>
 
 <style type="text/css">
diff --git a/lib/editor/tinymce/tinymce.class.php b/lib/editor/tinymce/tinymce.class.php
index ab89d9b3c9..e710c36514 100644
--- a/lib/editor/tinymce/tinymce.class.php
+++ b/lib/editor/tinymce/tinymce.class.php
@@ -244,8 +244,8 @@ class tinymce extends editorObject {
         }
 
         echo "\n";
-        echo '<script language="javascript" type="text/javascript">'."\n";
-        echo '<!--'."\n";
+        echo '<script type="text/javascript">'."\n";
+        echo '//<![CDATA['."\n";
         echo '  tinyMCE.init({'."\n";
 
         if ( !empty($conf) ) {
@@ -278,7 +278,7 @@ class tinymce extends editorObject {
         if ( $this->printdialogs ) {
             $this->__dialogs();
         }
-        echo '// done hiding -->'."\n";
+        echo '//]]>'."\n";
         echo '</script>'."\n";
 
     }
@@ -378,4 +378,4 @@ class tinymce extends editorObject {
 
 }
 
-?>
\ No newline at end of file
+?>
diff --git a/lib/formslib.php b/lib/formslib.php
index 3741cf0d37..af320cd726 100644
--- a/lib/formslib.php
+++ b/lib/formslib.php
@@ -1126,7 +1126,7 @@ function validate_' . $this->_formName . '(frm) {
     }
 
     function getLockOptionEndScript(){
-        $js = '<script type="text/javascript" language="javascript">'."\n";
+        $js = '<script type="text/javascript">'."\n";
         $js .= '//<![CDATA['."\n";
         $js .= "var ".$this->_formName."items= {";
         foreach ($this->_dependencies as $dependentOn => $elements){
diff --git a/lib/moodlelib.php b/lib/moodlelib.php
index 7f52ce70ed..60e6dcf506 100644
--- a/lib/moodlelib.php
+++ b/lib/moodlelib.php
@@ -2611,11 +2611,11 @@ function get_complete_user_data($field, $value) {
         }
     }
 
-    if ($groups = get_records('groups_members', 'userid', $user->id)) {
-        foreach ($groups as $groupmember) {
-            $courseid = get_field('groups', 'courseid', 'id', $groupmember->groupid);
+    if ($groupids = groups_get_all_groups_for_user($user->id)) { //TODO:check.
+        foreach ($groupids as $groupid) {
+            $courseid = groups_get_course($groupid);
             //change this to 2D array so we can put multiple groups in a course
-            $user->groupmember[$courseid][] = $groupmember->groupid;
+            $user->groupmember[$courseid][] = $groupid;
         }
     }
 
@@ -2791,25 +2791,26 @@ function remove_course_contents($courseid, $showfeedback=true) {
         }
     }
 
-/// Delete any groups
-    if ($groups = get_records('groups', 'courseid', $course->id)) {
-        if (delete_records('groups', 'courseid', $course->id)) {
-            if ($showfeedback) {
-                notify($strdeleted .' groups');
+/// Delete any groups, removing members first. TODO: check.
+    if ($groupids = groups_get_groups($course->id)) {
+        foreach ($groupids as $groupid) {
+            if (groups_remove_all_group_members($groupid)) {
+                if ($showfeedback) {
+                    notify($strdeleted .' groups_members');
+                }
+            } else {
+                $result = false;
             }
-            foreach ($groups as $group) {
-                if (delete_records('groups_members', 'groupid', $group->id)) {
-                    if ($showfeedback) {
-                        notify($strdeleted .' groups_members');
-                    }
-                } else {
-                    $result = false;
+            /// Delete any associated context for this group ??
+            delete_context(CONTEXT_GROUP, $group->id);
+            
+            if (groups_delete_group($groupid)) {
+                if ($showfeedback) {
+                    notify($strdeleted .' groups');
                 }
-                /// Delete any associated context for this group
-                delete_context(CONTEXT_GROUP, $group->id);
+            } else {
+                $result = false;
             }
-        } else {
-            $result = false;
         }
     }
 
@@ -2926,10 +2927,10 @@ function reset_course_userdata($data, $showfeedback=true) {
                 notify($strdeleted .' '.get_string('students'), 'notifysuccess');
             }
 
-            /// Delete group members (but keep the groups)
-            if ($groups = get_records('groups', 'courseid', $data->courseid)) {
-                foreach ($groups as $group) {
-                    if (delete_records('groups_members', 'groupid', $group->id)) {
+            /// Delete group members (but keep the groups) TODO:check.
+            if ($groupids = groups_get_groups($data->courseid)) {
+                foreach ($groupids as $groupid) {
+                    if (groups_remove_all_group_members($groupid)) {
                         if ($showfeedback) {
                             notify($strdeleted .' groups_members', 'notifysuccess');
                         }
@@ -2951,9 +2952,9 @@ function reset_course_userdata($data, $showfeedback=true) {
     }
 
     if (!empty($data->reset_groups)) {
-        if ($groups = get_records('groups', 'courseid', $data->courseid)) {
-            foreach ($groups as $group) {
-                if (delete_records('groups', 'id', $group->id)) {
+        if ($groupids = groups_get_groups($data->courseid)) {
+            foreach ($groupids as $groupid) {
+                if (groups_delete_group($groupid)) {
                     if ($showfeedback) {
                         notify($strdeleted .' groups', 'notifysuccess');
                     }
@@ -2992,284 +2993,16 @@ function reset_course_userdata($data, $showfeedback=true) {
 }
 
 
-/// GROUPS /////////////////////////////////////////////////////////
-
-
-/**
- * Determines if the user a member of the given group
- *
- * @uses $USER
- * @param int $groupid The group to check the membership of
- * @param int $userid The user to check against the group
- * @return bool
- */
-function ismember($groupid, $userid=0) {
-    global $USER;
-
-    if (!$groupid) {   // No point doing further checks
-        return false;
-    }
-    //if groupid is supplied in array format
-    if (!$userid) {
-        if (empty($USER->groupmember)) {
-            return false;
-        }
-        //changed too for multiple groups
-        foreach ($USER->groupmember as $courseid => $mgroupid) {
-            //need to loop one more time...
-            if (is_array($mgroupid)) {
-                foreach ($mgroupid as $index => $mygroupid) {
-                    if ($mygroupid == $groupid) {
-                        return true;
-                    }
-                }
-            } else if ($mgroupid == $groupid) {
-                return true;
-            }
-        }
-        return false;
-    }
-
-    if (is_array($groupid)){
-        foreach ($groupid as $index => $val){
-            if (record_exists('groups_members', 'groupid', $val, 'userid', $userid)){
-                return true;
-            }
-        }
-    }
-    else {
-        return record_exists('groups_members', 'groupid', $groupid, 'userid', $userid);
-    }
-    return false;
-
-    //else group id is in single format
-
-    //return record_exists('groups_members', 'groupid', $groupid, 'userid', $userid);
-}
-
-/**
- * Add a user to a group, return true upon success or if user already a group member
- *
- * @param int $groupid  The group id to add user to
- * @param int $userid   The user id to add to the group
- * @return bool
- */
-function add_user_to_group ($groupid, $userid) {
-    if (ismember($groupid, $userid)) return true;
-    $record->groupid = $groupid;
-    $record->userid = $userid;
-    $record->timeadded = time();
-    return (insert_record('groups_members', $record) !== false);
-}
-
-
-/**
- * Get the group ID of the current user in the given course
- *
- * @uses $USER
- * @param int $courseid The course being examined - relates to id field in 'course' table.
- * @return int
- */
-function mygroupid($courseid) {
-    global $USER;
-    if (empty($USER->groupmember[$courseid])) {
-        return 0;
-    } else {
-        //this is an array of ids >.<
-        return $USER->groupmember[$courseid];
-    }
-}
-
-/**
- * For a given course, and possibly course module, determine
- * what the current default groupmode is:
- * NOGROUPS, SEPARATEGROUPS or VISIBLEGROUPS
- *
- * @param course $course A {@link $COURSE} object
- * @param object $cm A course module object
- * @return int A group mode (NOGROUPS, SEPARATEGROUPS or VISIBLEGROUPS)
- */
-function groupmode($course, $cm=null) {
-
-    if ($cm and !$course->groupmodeforce) {
-        return $cm->groupmode;
-    }
-    return $course->groupmode;
-}
+require_once($CFG->dirroot.'/group/lib.php');
+/*TODO: functions moved to /group/lib/legacylib.php
 
+ismember
+add_user_to_group
+mygroupid
+groupmode
+set_current_group
+... */
 
-/**
- * Sets the current group in the session variable
- *
- * @uses $SESSION
- * @param int $courseid The course being examined - relates to id field in 'course' table.
- * @param int $groupid The group being examined.
- * @return int Current group id which was set by this function
- */
-function set_current_group($courseid, $groupid) {
-    global $SESSION;
-
-    return $SESSION->currentgroup[$courseid] = $groupid;
-}
-
-
-/**
- * Gets the current group for the current user as an id or an object
- *
- * @uses $USER
- * @uses $SESSION
- * @param int $courseid The course being examined - relates to id field in 'course' table.
- * @param bool $full If true, the return value is a full record object. If false, just the id of the record.
- */
-function get_current_group($courseid, $full=false) {
-    global $SESSION, $USER;
-
-    if (!isset($SESSION->currentgroup[$courseid])) {
-        if (empty($USER->groupmember[$courseid]) or has_capability('moodle/site:accessallgroups', get_context_instance(CONTEXT_COURSE, $courseid))) {
-
-            return 0;
-        } else {
-            //trying to add a hack >.<, always first select the first one in list
-            $SESSION->currentgroup[$courseid] = $USER->groupmember[$courseid][0];
-        }
-    }
-
-    if ($full) {
-        return get_record('groups', 'id', $SESSION->currentgroup[$courseid]);
-    } else {
-        return $SESSION->currentgroup[$courseid];
-    }
-}
-
-/**
- * A combination function to make it easier for modules
- * to set up groups.
- *
- * It will use a given "groupid" parameter and try to use
- * that to reset the current group for the user.
- *
- * @uses VISIBLEGROUPS
- * @param course $course A {@link $COURSE} object
- * @param int $groupmode Either NOGROUPS, SEPARATEGROUPS or VISIBLEGROUPS
- * @param int $groupid Will try to use this optional parameter to
- *            reset the current group for the user
- * @return int|false Returns the current group id or false if error.
- */
-function get_and_set_current_group($course, $groupmode, $groupid=-1) {
-
-    if (!$groupmode) {   // Groups don't even apply
-        return false;
-    }
-
-    $currentgroupid = get_current_group($course->id);
-
-    if ($groupid < 0) {  // No change was specified
-        return $currentgroupid;
-    }
-
-    if ($groupid) {      // Try to change the current group to this groupid
-        if ($group = get_record('groups', 'id', $groupid, 'courseid', $course->id)) { // Exists
-            if (has_capability('moodle/site:accessallgroups', get_context_instance(CONTEXT_COURSE, $course->id))) {          // Sets current default group
-                $currentgroupid = set_current_group($course->id, $group->id);
-
-            } else if ($groupmode == VISIBLEGROUPS) {
-                  // All groups are visible
-                //if (ismember($group->id)){
-                    $currentgroupid = set_current_group($course->id, $group->id);//set this since he might post
-                /*)}else {
-                    $currentgroupid = $group->id;*/
-            } else if ($groupmode == SEPARATEGROUPS) { // student in separate groups switching
-                if (ismember($group->id)){//check if is a member
-                    $currentgroupid = set_current_group($course->id, $group->id); //might need to set_current_group?
-                }
-                else {
-                    echo ($group->id);
-                    notify('you do not belong to this group!',error);
-                }
-            }
-        }
-    } else {             // When groupid = 0 it means show ALL groups
-        //this is changed, non editting teacher needs access to group 0 as well, for viewing work in visible groups (need to set current group for multiple pages)
-        if (has_capability('moodle/site:accessallgroups', get_context_instance(CONTEXT_COURSE, $course->id)) AND ($groupmode == VISIBLEGROUPS)) {          // Sets current default group
-            $currentgroupid = set_current_group($course->id, 0);
-
-        } else if ($groupmode == VISIBLEGROUPS) {  // All groups are visible
-            $currentgroupid = 0;
-        }
-    }
-
-    return $currentgroupid;
-}
-
-
-/**
- * A big combination function to make it easier for modules
- * to set up groups.
- *
- * Terminates if the current user shouldn't be looking at this group
- * Otherwise returns the current group if there is one
- * Otherwise returns false if groups aren't relevant
- *
- * @uses SEPARATEGROUPS
- * @uses VISIBLEGROUPS
- * @param course $course A {@link $COURSE} object
- * @param int $groupmode Either NOGROUPS, SEPARATEGROUPS or VISIBLEGROUPS
- * @param string $urlroot ?
- * @return int|false
- */
-function setup_and_print_groups($course, $groupmode, $urlroot) {
-
-    global $USER, $SESSION; //needs his id, need to hack his groups in session
-
-    $changegroup = optional_param('group', -1, PARAM_INT);
-
-    $currentgroup = get_and_set_current_group($course, $groupmode, $changegroup);
-    if ($currentgroup === false) {
-        return false;
-    }
-
-    if ($groupmode == SEPARATEGROUPS and !has_capability('moodle/site:accessallgroups', get_context_instance(CONTEXT_COURSE, $course->id)) and !$currentgroup) {
-        //we are in separate groups and the current group is group 0, as last set.
-        //this can mean that either, this guy has no group
-        //or, this guy just came from a visible all forum, and he left when he set his current group to 0 (show all)
-
-        //for the second situation, we need to perform the trick and get him a group.
-        $courseid = $course->id;
-        if (!empty($USER->groupmember[$courseid])){
-            $currentgroup = get_and_set_current_group($course, $groupmode, $USER->groupmember[$courseid][0]);
-        }
-        else {//else he has no group in this course
-            print_heading(get_string('notingroup'));
-            print_footer($course);
-            exit;
-        }
-    }
-
-    if ($groupmode == VISIBLEGROUPS or ($groupmode and has_capability('moodle/site:accessallgroups', get_context_instance(CONTEXT_COURSE, $course->id)))) {
-        if ($groups = get_records_menu('groups', 'courseid', $course->id, 'name ASC', 'id,name')) {
-            echo '<div align="center">';
-            print_group_menu($groups, $groupmode, $currentgroup, $urlroot);
-            echo '</div>';
-        }
-    }//added code here to allow non-editting teacher to swap in-between his own groups
-    //added code for students in separategrous to swtich groups
-    else if ($groupmode == SEPARATEGROUPS and has_capability('moodle/course:view', get_context_instance(CONTEXT_COURSE, $course->id))) {
-        $validgroups = array();
-        //get all the groups this guy is in in this course
-        if ($p = user_group($course->id,$USER->id)){
-            //extract the name and id for the group
-            foreach ($p as $index => $object){
-                $validgroups[$object->id] = $object->name;
-            }
-            echo '<div align="center">';
-            //print them in the menu
-            print_group_menu($validgroups, $groupmode, $currentgroup, $urlroot,0);
-            echo '</div>';
-        }
-    }
-
-    return $currentgroup;
-}
 
 function generate_email_processing_address($modid,$modargs) {
     global $CFG;
@@ -6451,7 +6184,8 @@ function message_popup_window() {
                 if (get_user_preferences('message_showmessagewindow', 1) == 1) {
                     if (count_records_select('message', 'useridto = \''.$USER->id.'\' AND timecreated > \''.$USER->message_lastpopup.'\'')) {
                         $USER->message_lastpopup = time();
-                        return '<script language="JavaScript" type="text/javascript">'."\n openpopup('/message/index.php', 'message', 'menubar=0,location=0,scrollbars,status,resizable,width=400,height=500', 0);\n</script>";
+                        return '<script type="text/javascript">'."\n//<![CDATA[\n openpopup('/message/index.php', 'message',
+                        'menubar=0,location=0,scrollbars,status,resizable,width=400,height=500', 0);\n//]]>\n</script>";
                     }
                 }
             }
diff --git a/lib/soap/nusoap.php b/lib/soap/nusoap.php
index 78deafef54..d7ad5d7142 100644
--- a/lib/soap/nusoap.php
+++ b/lib/soap/nusoap.php
@@ -4779,8 +4779,8 @@ class wsdl extends nusoap_base {
 		    a:visited   { color: #666666; font-weight: bold; }
 		    a:hover     { color: cc3300; font-weight: bold; }
 		</style>
-		<script language="JavaScript" type="text/javascript">
-		<!--
+		<script type="text/javascript">
+		//<![CDATA[
 		// POP-UP CAPTIONS...
 		function lib_bwcheck(){ //Browsercheck (needed)
 		    this.ver=navigator.appVersion
@@ -4823,7 +4823,7 @@ class wsdl extends nusoap_base {
 		function popout(){ // Hides message
 		    if(oDesc) oDesc.css.visibility = "hidden"
 		}
-		//-->
+		//]]>
 		</script>
 		</head>
 		<body>
diff --git a/lib/speller/server-scripts/spellchecker.php b/lib/speller/server-scripts/spellchecker.php
index 9c2cb716c9..841da0346b 100644
--- a/lib/speller/server-scripts/spellchecker.php
+++ b/lib/speller/server-scripts/spellchecker.php
@@ -195,8 +195,9 @@ function print_checker_results() {
 <head>
 <meta http-equiv="Content-Type" content="text/html; charset=utf-8">
 <link rel="stylesheet" type="text/css" href="<?php echo $spellercss ?>" />
-<script language="javascript" src="<?php echo $word_win_src ?>"></script>
-<script language="javascript">
+<script type="text/javascript" src="<?php echo $word_win_src ?>"></script>
+<script type="text/javascript">
+//<![CDATA[
 var suggs = new Array();
 var words = new Array();
 var textinputs = new Array();
@@ -228,15 +229,16 @@ function init_spell() {
     }
 }
 
-
-
+//]]>
 </script>
 
 </head>
 <body onLoad="init_spell();">
 
 <script type="text/javascript">
+//<![CDATA[
 wordWindowObj.writeBody();
+//]]>
 </script>
 
 </body>
diff --git a/lib/weblib.php b/lib/weblib.php
index 450e845e2d..c6c614e4f4 100644
--- a/lib/weblib.php
+++ b/lib/weblib.php
@@ -3720,7 +3720,7 @@ function use_html_editor($name='', $editorhidebuttons='', $id='') {
     if ($id === '') {
         $id = 'edit-'.$name;
     }
-    echo "\n".'<script language="javascript" type="text/javascript" defer="defer">'."\n";
+    echo "\n".'<script type="text/javascript" defer="defer">'."\n";
     echo '//<![CDATA['."\n";
     echo "$editor = new HTMLArea('$id');\n";
     echo "var config = $editor.config;\n";
@@ -5186,17 +5186,17 @@ function print_speller_code ($usehtmleditor=false, $return=false) {
     global $CFG;
     $str = '';
     
+    $str .= "\n".'<script type="text/javascript">'."\n";
+    $str .= '//<![CDATA['."\n";
     if(!$usehtmleditor) {
-        $str .= "\n".'<script language="javascript" type="text/javascript">'."\n";
         $str .= 'function openSpellChecker() {'."\n";
         $str .= "\tvar speller = new spellChecker();\n";
         $str .= "\tspeller.popUpUrl = \"" . $CFG->wwwroot ."/lib/speller/spellchecker.html\";\n";
         $str .= "\tspeller.spellCheckScript = \"". $CFG->wwwroot ."/lib/speller/server-scripts/spellchecker.php\";\n";
         $str .= "\tspeller.spellCheckAll();\n";
         $str .= '}'."\n";
-        $str .= '</script>'."\n";
     } else {
-        $str .= "\nfunction spellClickHandler(editor, buttonId) {\n";
+        $str .= "function spellClickHandler(editor, buttonId) {\n";
         $str .= "\teditor._textArea.value = editor.getHTML();\n";
         $str .= "\tvar speller = new spellChecker( editor._textArea );\n";
         $str .= "\tspeller.popUpUrl = \"" . $CFG->wwwroot ."/lib/speller/spellchecker.html\";\n";
@@ -5206,6 +5206,8 @@ function print_speller_code ($usehtmleditor=false, $return=false) {
         $str .= "\tspeller.openChecker();\n";
         $str .= '}'."\n";
     }
+    $str .= '//]]>'."\n";
+    $str .= '</script>'."\n";
     if ($return) {
         return $str;
     }
diff --git a/message/index.php b/message/index.php
index 9bdf89cda9..7f12d4388f 100644
--- a/message/index.php
+++ b/message/index.php
@@ -25,7 +25,7 @@
 /// Popup a window if required and quit (usually from external links).
     if ($popup) {
         print_header();
-        echo '<script language="JavaScript" type="text/javascript">'."\n openpopup('/message/index.php', 'message', 'menubar=0,location=0,scrollbars,status,resizable,width=400,height=500', 0);\n</script>";
+        echo '<script type="text/javascript">'."\n//<![CDATA[\n openpopup('/message/index.php', 'message', 'menubar=0,location=0,scrollbars,status,resizable,width=400,height=500', 0);\n//]]>\n</script>";
         redirect("$CFG->wwwroot/");
         exit;
     }
diff --git a/mod/assignment/mod.html b/mod/assignment/mod.html
index bba84f6bf7..ac83366a5a 100644
--- a/mod/assignment/mod.html
+++ b/mod/assignment/mod.html
@@ -30,9 +30,11 @@
     }
 
 ?>
-<script type="text/javascript" language="javascript">
+<script type="text/javascript">
+//<![CDATA[
     var dueitems = ['dueday','duemonth','dueyear','duehour', 'dueminute', 'preventlate'];
     var availableitems = ['availableday','availablemonth','availableyear','availablehour', 'availableminute'];
+//]]>
 </script>
 
 <form name="form" method="post" action="../mod/assignment/details.php"> 
diff --git a/mod/chat/chatd.php b/mod/chat/chatd.php
index bb41bf9e8a..a6dd9d883d 100755
--- a/mod/chat/chatd.php
+++ b/mod/chat/chatd.php
@@ -247,8 +247,8 @@ class ChatDaemon {
 
         ob_start();
         echo '<html><head>';
-        echo '<script language="JavaScript">';
-        echo "<!-- //hide\n";
+        echo '<script text="text/javascript">';
+        echo "//<![CDATA[\n";
 
         echo 'function openpopup(url,name,options,fullscreen) {';
         echo 'fullurl = "'.$CFG->wwwroot.'" + url;';
@@ -259,7 +259,7 @@ class ChatDaemon {
         echo '}';
         echo 'windowobj.focus();';
         echo 'return false;';
-        echo "}\n-->\n";
+        echo "}\n//]]>\n";
         echo '</script></head><body style="font-face: serif;" bgcolor="#FFFFFF">';
 
         echo '<table style="width: 100%;"><tbody>';
diff --git a/mod/chat/gui_header_js/chatinput.php b/mod/chat/gui_header_js/chatinput.php
index 30f5190a34..5ef4bb8ddb 100644
--- a/mod/chat/gui_header_js/chatinput.php
+++ b/mod/chat/gui_header_js/chatinput.php
@@ -27,7 +27,7 @@
 
     ob_start();
     ?>
-    <script language="JavaScript" type="text/javascript">
+    <script type="text/javascript">
     //<![CDATA[
     var waitFlag = false;
     function empty_field_and_submit() {
diff --git a/mod/chat/gui_header_js/insert.php b/mod/chat/gui_header_js/insert.php
index 6305c819d6..384e410fd3 100644
--- a/mod/chat/gui_header_js/insert.php
+++ b/mod/chat/gui_header_js/insert.php
@@ -60,13 +60,13 @@
     if ($chatuser->version == 'header_js') {
         /// force msg referesh ASAP
         if ($CFG->chat_normal_updatemode == 'jsupdated') {  // See bug MDL-6791
-            echo '<script language="Javascript" type="text/javascript">'.
+            echo '<script type="text/javascript">'.
                  "//<![CDATA[ \n".
                  '  parent.input.enableForm();'.
                  "//]]> \n".
                  '</script>';
         } else {
-            echo '<script language="Javascript" type="text/javascript">'.
+            echo '<script type="text/javascript">'.
                  "//<![CDATA[ \n".
                  '  parent.jsupdate.location.href = parent.jsupdate.document.anchors[0].href;'.
                  '  parent.input.enableForm();'.
diff --git a/mod/chat/gui_header_js/jsupdated.php b/mod/chat/gui_header_js/jsupdated.php
index a9ce81c729..675e631206 100644
--- a/mod/chat/gui_header_js/jsupdated.php
+++ b/mod/chat/gui_header_js/jsupdated.php
@@ -153,7 +153,7 @@
             $num = 0;
         }
 
-        print '<script language="Javascript" type="text/javascript">' . "\n";
+        print '<script type="text/javascript">' . "\n";
         print "//<![CDATA[\n\n"; 
 
         $chat_newrow = ($chat_lastrow + $num) % 2;
@@ -211,7 +211,7 @@
 
     // here & should be written & :-D
     $refreshurl = "{$CFG->wwwroot}/mod/chat/gui_header_js/jsupdated.php?chat_sid=$chat_sid&chat_lasttime=$chat_lasttime&chat_lastrow=$chat_newrow&chat_lastid=$chat_lastid"; 
-    print '<script language="Javascript" type="text/javascript">' . "\n";
+    print '<script type="text/javascript">' . "\n";
     print "//<![CDATA[ \n\n"; 
     print "location.href = '$refreshurl';\n";
     print "//]]>\n";
diff --git a/mod/chat/gui_header_js/users.php b/mod/chat/gui_header_js/users.php
index 9023ff55be..eead4f30af 100644
--- a/mod/chat/gui_header_js/users.php
+++ b/mod/chat/gui_header_js/users.php
@@ -58,7 +58,7 @@
 
     ob_start();
     ?>
-    <script language="JavaScript" type="text/javascript">
+    <script type="text/javascript">
     //<![CDATA[
     var timer = null
     var f = 1; //seconds
diff --git a/mod/data/templates.php b/mod/data/templates.php
index 634e9e9063..c9471cf341 100755
--- a/mod/data/templates.php
+++ b/mod/data/templates.php
@@ -82,7 +82,7 @@
     $bodytag .= '" ';
 
     // Javascript to insert the field tags into the textarea.
-    $meta = '<script language="JavaScript" type="text/javascript">'."\n";
+    $meta = '<script type="text/javascript">'."\n";
     $meta .= '//<![CDATA['."\n";
     $meta .= 'function insert_field_tags(selectlist) {';
     $meta .= '  if (typeof(currEditor) != \'undefined\' && currEditor._editMode == \'wysiwyg\') {';
diff --git a/mod/hotpot/lib.php b/mod/hotpot/lib.php
index 6cdeaf602f..19f9a825dc 100644
--- a/mod/hotpot/lib.php
+++ b/mod/hotpot/lib.php
@@ -1674,7 +1674,7 @@ class hotpot_xml_quiz extends hotpot_xml_tree {
         $this->html = preg_replace($search, '', $this->html);
     }
     function insert_script($src=HOTPOT_JS) {
-        $script = '<script src="'.$src.'" type="text/javascript" language="javascript"></script>'."\n";
+        $script = '<script src="'.$src.'" type="text/javascript"></script>'."\n";
         $this->html = preg_replace('|</head>|i', $script.'</head>', $this->html, 1);
     }
     function insert_submission_form($attemptid, $startblock, $endblock, $keep_contents=false) {
@@ -2365,8 +2365,8 @@ function hotpot_print_show_links($course, $location, $reference, $actions='', $s
     }
     $strenterafilename = get_string('enterafilename', 'hotpot');
     $html = <<<END_OF_SCRIPT
-<script type="text/javascript" language="javascript">
-<!--
+<script type="text/javascript">
+//<![CDATA[
     function setLink(lnk) {
         var form = document.forms['form'];
         return setLinkAttribute(lnk, 'reference', form) && setLinkAttribute(lnk, 'location', form);
@@ -2424,7 +2424,7 @@ function hotpot_print_show_links($course, $location, $reference, $actions='', $s
         var i = s.lastIndexOf('/');
         return s.substring(0, i);
     }
-//-->
+//]]>
 </script>
 END_OF_SCRIPT;
 
diff --git a/mod/hotpot/mod.html b/mod/hotpot/mod.html
index 88a0ceacf6..7dddc45aa5 100644
--- a/mod/hotpot/mod.html
+++ b/mod/hotpot/mod.html
@@ -20,8 +20,8 @@ $text_source_options = array(
     HOTPOT_TEXTSOURCE_SPECIFIC => get_string("textsourcespecific", "hotpot")
 );
 ?>
-<script type="text/javascript" language="javascript">
-<!--
+<script type="text/javascript">
+//<![CDATA[
     var nameoffvalues = new Array(0,1,2);
     var namelockitems = new Array('name');
 
@@ -57,7 +57,7 @@ $text_source_options = array(
         }
         return(true);
     }
-//-->
+//]]>
 </script>
 <center>
 <form name="form" method="post" action="mod.php">
diff --git a/mod/hotpot/report/fullstat/report.php b/mod/hotpot/report/fullstat/report.php
index f038e9f4d8..42f387dc23 100644
--- a/mod/hotpot/report/fullstat/report.php
+++ b/mod/hotpot/report/fullstat/report.php
@@ -397,8 +397,8 @@ function hotpot_showhide_button($id) {
 	$pref = '1';
 	$text = ($pref=='1' ? $hide : $show);
 return <<<SHOWHIDE_BUTTON
-<script language="javascript">
-<!--
+<script type="text/javascript">
+//<![CDATA[
 	function showhide (id, toggle) {
 		var show = true;
 		obj = document.getElementById(id+'pref');
@@ -428,19 +428,19 @@ return <<<SHOWHIDE_BUTTON
 		html += '</form>';
 		document.writeln(html);
 	}
-//-->
+//]]>
 </script>
 SHOWHIDE_BUTTON
 ;
 }
 function hotpot_showhide_set($id) {
 return <<<SHOWHIDE_SET
-<script language="javascript">
-<!--
+<script type="text/javascript">
+//<![CDATA[
 	if (showhide_allowed) {
 		showhide('$id');
 	}
-//-->
+//]]>
 </script>
 SHOWHIDE_SET
 ;
diff --git a/mod/hotpot/show.php b/mod/hotpot/show.php
index 13122c3102..f55a7a7db1 100644
--- a/mod/hotpot/show.php
+++ b/mod/hotpot/show.php
@@ -22,8 +22,8 @@
     print_header($title, $title);
     hotpot_print_show_links($params->course, $params->location, $params->reference);
 ?>
-<script type="text/javascript" language="javascript">
-<!--
+<script type="text/javascript">
+//<![CDATA[
     // http://www.krikkit.net/howto_javascript_copy_clipboard.html
     function copy_contents(id) {
         if (id==null) {
@@ -39,7 +39,7 @@
         }
     }
     document.write('<span class="helplink"> &nbsp; <a href="javascript:copy_contents()"><?php print_string('copytoclipboard', 'hotpot') ?></A></span>');
--->
+//]]>
 </script>
 <?php
     print_simple_box_start("center", "96%");
diff --git a/mod/hotpot/view.php b/mod/hotpot/view.php
index 905f1b755b..b3beb2ed9d 100644
--- a/mod/hotpot/view.php
+++ b/mod/hotpot/view.php
@@ -255,7 +255,7 @@
         foreach ($feedback as $i=>$str) {
             $js .= 'FEEDBACK['.$i."] = $str;\n";
         }
-        $js = '<script type="text/javascript" language="javascript">'."\n//<![CDATA[\n"."FEEDBACK = new Array();\n".$js."//]]>\n</script>\n";
+        $js = '<script type="text/javascript">'."\n//<![CDATA[\n"."FEEDBACK = new Array();\n".$js."//]]>\n</script>\n";
         $hp->html = preg_replace('|</head>|i', "$js</head>", $hp->html, 1);
     }
     // insert hot-potatoes.js
@@ -411,7 +411,7 @@
                 default:
                     $iframe_id = 'hotpot_iframe';
                     $body_tags = " onload=\"set_iframe_height('$iframe_id')\"";
-                    $iframe_js = '<script src="iframe.js" type="text/javascript" language="javascript"></script>'."\n";
+                    $iframe_js = '<script src="iframe.js" type="text/javascript"></script>'."\n";
                     print_header(
                         $title, $heading, $navigation, 
                         "", $head.$styles.$scripts.$iframe_js, true, $button, 
diff --git a/mod/lams/mod.html b/mod/lams/mod.html
index e47a5c94a6..c18de9788c 100644
--- a/mod/lams/mod.html
+++ b/mod/lams/mod.html
@@ -106,8 +106,8 @@ print_textarea($usehtmleditor, 20, 50, 680, 400, "introduction", $form->introduc
 
 </form>
 
-<script language="JavaScript" type="text/javascript">
-<!--
+<script type="text/javascript">
+//<![CDATA[
 
 function validate(){
     workspaceObj = document.getElementById("workspace");
@@ -773,5 +773,5 @@ function DOL_selectChildOptions(obj,usePreselected) {
 }
 
 refreshLists();//refresh the lists when form is displayed.
--->
+//]]>
 </script>
diff --git a/mod/quiz/attempt.php b/mod/quiz/attempt.php
index 23c1e0d4b8..d9c2966fb6 100644
--- a/mod/quiz/attempt.php
+++ b/mod/quiz/attempt.php
@@ -470,13 +470,13 @@
     if($quiz->timelimit > 0) {
         // Make sure javascript is enabled for time limited quizzes
         ?>
-        <script language="javascript" type="text/javascript">
-        <!--
+        <script type="text/javascript">
+        //<![CDATA[
             document.write("<form name=\"responseform\" id=\"responseform\" method=\"post\" action=\"attempt.php\" autocomplete=\"off\">\n");
-        // -->
+        //]]>
         </script>
         <noscript>
-        <center><p><strong><?php print_string('noscript', 'quiz'); ?></strong></p></center>
+        <?php print_heading(get_string('noscript', 'quiz')); ?>
         </noscript>
         <?php
     } else {
@@ -490,7 +490,8 @@
     $numpages = quiz_number_of_pages($attempt->layout);
     if ($numpages > 1) {
         ?>
-        <script language="javascript" type="text/javascript">
+        <script type="text/javascript">
+        //<![CDATA[
         function navigate(page) {
             var ourForm = document.forms['responseform']; 
             ourForm.page.value=page;
@@ -499,6 +500,7 @@
             }
             ourForm.submit();
         }
+        //]]>
         </script>
         <?php
         echo '<input type="hidden" id="page" name="page" value="'.$page."\" />\n";
diff --git a/mod/quiz/attempt_close_js.php b/mod/quiz/attempt_close_js.php
index 7f62686aea..4932b57470 100644
--- a/mod/quiz/attempt_close_js.php
+++ b/mod/quiz/attempt_close_js.php
@@ -5,13 +5,13 @@
 if (!empty($popup)) {
 ?>
 
-<script language="javascript" type="text/javascript">
-<!--
+<script type="text/javascript">
+//<![CDATA[
 
 document.write('<input type="button" value="<?php print_string('closewindow') ?>" '+
                'onclick="javascript: window.opener.location.href=\'view.php?id=<?php echo $cm->id ?>\'; '+
                'window.close();" />');
-// -->
+//]]>
 </script>
 <noscript>
 <?php print_string('closewindow'); ?>
diff --git a/mod/quiz/config.html b/mod/quiz/config.html
index 0204b148a7..b5fdd2a994 100644
--- a/mod/quiz/config.html
+++ b/mod/quiz/config.html
@@ -1,4 +1,4 @@
-<?php
+<?php  // $Id$
     require_once($CFG->dirroot . '/mod/quiz/locallib.php');
 
     if (!isset($form->timeopen)) {
@@ -81,8 +81,10 @@
     }
 ?>
 
-<script type="text/javascript" language="javascript">
+<script type="text/javascript">
+//<![CDATA[
     var timelimititems = ['timelimit'];
+//]]>
 </script>
 
 <form method="post" action="module.php" name="form">
diff --git a/mod/quiz/jsclock.php b/mod/quiz/jsclock.php
index 44623b6eef..ca2ff1cca2 100644
--- a/mod/quiz/jsclock.php
+++ b/mod/quiz/jsclock.php
@@ -2,8 +2,8 @@
 defined('MOODLE_INTERNAL') or die('Direct access to this script is forbidden.');
 ?>
 
-<script language="javascript">
-<!--
+<script text="text/javascript">
+//<![CDATA[
 /// This Javascript clock provides a little countdown in the title bar
 
 var timerID = null;
@@ -63,5 +63,5 @@ function showtime() {
 document.onLoad = startclock();
 
 
-// -- End of JavaScript code -------------- -->
+//]]>
 </script>
diff --git a/mod/quiz/jstimer.php b/mod/quiz/jstimer.php
index bb5acc1e5b..ffb6e2b50e 100644
--- a/mod/quiz/jstimer.php
+++ b/mod/quiz/jstimer.php
@@ -6,8 +6,8 @@
 defined('MOODLE_INTERNAL') or die('Direct access to this script is forbidden.');
 ?>
 
-<script language="javascript" type="text/javascript">
-<!--
+<script type="text/javascript">
+//<![CDATA[
 var timesup = "<?php print_string("timesup","quiz");?>";
 var quizclose = <?php echo ($quiz->timeclose - time()) - $timerstartvalue; ?>; // in seconds
 var quizTimerValue = <?php echo $timerstartvalue; ?>; // in seconds
@@ -18,9 +18,9 @@ var ec_page_start = new Date().getTime();
 // @EC PF : client time when quiz should end
 var ec_quiz_finish = ec_page_start + <?php echo ($timerstartvalue * 1000); ?>;
 
-// -->
+//]]>
 </script>
-<script language="javascript" type="text/javascript" src="timer.js"></script>
+<script type="text/javascript" src="timer.js"></script>
 <div id="timer">
 <!--EDIT BELOW CODE TO YOUR OWN MENU-->
 <table class="generalbox" border="0" cellpadding="0" cellspacing="0" width="150">
@@ -43,8 +43,8 @@ var ec_quiz_finish = ec_page_start + <?php echo ($timerstartvalue * 1000); ?>;
 </table>
 <!--END OF EDIT-->
 </div>
-<script language="javascript" type="text/javascript">
-<!--
+<script type="text/javascript">
+//<![CDATA[
 
 var timerbox = xGetElementById('timer');
 var theTimer = xGetElementById('QuizTimer');
@@ -54,5 +54,5 @@ var old = theTop;
 movecounter(timerbox);
 
 document.onload = countdown_clock(theTimer);
-// -->
+//]]>
 </script>
diff --git a/mod/quiz/protect_js.php b/mod/quiz/protect_js.php
index cfe5f961bc..a6f9c47e05 100644
--- a/mod/quiz/protect_js.php
+++ b/mod/quiz/protect_js.php
@@ -20,7 +20,7 @@
 
 <style media="print">body {display:none}</style>
 
-<script language="JavaScript" type="text/javascript">
+<script type="text/javascript">
 //<![CDATA[
 
 var message = "<?php print_string('functiondisabled'); ?>";
diff --git a/mod/quiz/view.php b/mod/quiz/view.php
index 818697a9cf..300f5cb10b 100644
--- a/mod/quiz/view.php
+++ b/mod/quiz/view.php
@@ -379,17 +379,17 @@
 
                 // TODO eliminate this nasty JavaScript that prints the button.
 ?>
-<script language="javascript" type="text/javascript">
-<!--
+<script type="text/javascript">
+//<![CDATA[
 document.write('<input type="button" value="<?php echo $buttontext ?>" onclick="javascript: <?php
                 if ($strconfirmstartattempt) {
                     echo "if (confirm(\\'".addslashes_js($strconfirmstartattempt)."\\'))";
                 }
 ?> window.open(\'<?php echo $attempturl ?>\', \'<?php echo $window ?>\', \'<?php echo $windowoptions ?>\'); " />');
-// -->
+//]]>
 </script>
 <noscript>
-    <strong><?php print_string('noscript', 'quiz'); ?></strong>
+    <?php print_heading(get_string('noscript', 'quiz')); ?>
 </noscript>
 <?php
             }
diff --git a/mod/resource/type/file/file.html b/mod/resource/type/file/file.html
index 30e3a6b58b..828e496a19 100644
--- a/mod/resource/type/file/file.html
+++ b/mod/resource/type/file/file.html
@@ -3,7 +3,8 @@
     <td align="right" nowrap="nowrap">
 
 
-<script language="javascript" type="text/javascript">
+<script type="text/javascript">
+//<![CDATA[
     function showhide (id, set) {
         divobj = document.getElementById(id);
         butobj = document.getElementById(id+'button');
@@ -28,6 +29,7 @@
             }
         }
     }
+//]]>
 </script>
 
 
@@ -70,9 +72,11 @@
 
     <td colspan="2">
         <script type="text/javascript">
+        //<![CDATA[
             var popupitems = [<?php echo $popupoptions; ?>];
             var frameitem = [<?php echo $frameoption; ?>];
             var allitems = [<?php echo $alloptions; ?>];
+        //]]>
         </script>
         <input type="radio" name="windowpopup" value="0" alt="<?php print_string('pagewindow', 'resource') ?>" <?php echo ($windowtype != "popup") ? "checked=\"checked\"" : "" ?> 
         onclick="lockoptions('form', 'windowpopup[0]', frameitem); 
@@ -116,11 +120,15 @@
         <?php
             if ($windowtype == "page") {
                 echo "<script type=\"text/javascript\">";
+                echo "\n//<![CDATA[\n";
                 echo "lockoptions('form','windowpopup[1]', popupitems);";
+                echo "\n//]]>\n";
                 echo "</script>";
             } else {
                 echo "<script type=\"text/javascript\">";
+                echo "\n//<![CDATA[\n";
                 echo "lockoptions('form','windowpopup[0]', frameitem);";
+                echo "\n//]]>\n";
                 echo "</script>";
             }
         ?>
@@ -190,9 +198,11 @@ for ($i=0; $i < $this->maxparameters; $i++) {
     </div>
 
 
-<script language="javascript" type="text/javascript">
+<script type="text/javascript">
+//<![CDATA[
     showhide('parametersettings', true);
     showhide('windowsettings', true);
+//]]>
 </script>
 
 
diff --git a/mod/resource/type/file/localfile.php b/mod/resource/type/file/localfile.php
index 6dd63e2d8c..f953081081 100644
--- a/mod/resource/type/file/localfile.php
+++ b/mod/resource/type/file/localfile.php
@@ -18,8 +18,8 @@
     $chooseparts = explode('.', $choose);
 
     ?>
-    <script language="javascript" type="text/javascript">
-    <!--
+    <script type="text/javascript">
+    //<![CDATA[
     function set_value(txt) {
         if (txt.indexOf('/') > -1) {
             path = txt.substring(txt.indexOf('/'),txt.length);
@@ -31,7 +31,7 @@
         opener.document.forms['<?php echo $chooseparts[0]."'].".$chooseparts[1] ?>.value = '<?php p(RESOURCE_LOCALPATH) ?>'+path;
         window.close();
     }
-    -->
+    //]]>
     </script>
     
     <br />
diff --git a/mod/resource/type/file/localpath.php b/mod/resource/type/file/localpath.php
index ed177b38de..09bd9100fb 100644
--- a/mod/resource/type/file/localpath.php
+++ b/mod/resource/type/file/localpath.php
@@ -10,10 +10,10 @@
             set_user_preference('resource_localpath', $pathname);
         }
         ?>
-        <script language="javascript" type="text/javascript">
-        <!--
+        <script type="text/javascript">
+        //<![CDATA[
         window.close();
-        -->
+        //]]>
         </script>
         <?php
         exit;
@@ -24,8 +24,8 @@
     print_simple_box(get_string('localfilepath', 'resource', $CFG->wwwroot.'/user/edit.php?course='.SITEID), 'center');
 
     ?>
-    <script language="javascript" type="text/javascript">
-    <!--
+    <script type="text/javascript">
+    //<![CDATA[
     function set_value(txt) {
         if (txt.indexOf('/') > -1) {
             txt = txt.substring(0,txt.lastIndexOf('/'));
@@ -35,7 +35,7 @@
         document.myform.pathname.value = txt;
         document.myform.submit();
     }
-    -->
+    //]]>
     </script>
     
     <br />
diff --git a/mod/resource/type/html/html.html b/mod/resource/type/html/html.html
index fbc3c39753..e73c79859d 100644
--- a/mod/resource/type/html/html.html
+++ b/mod/resource/type/html/html.html
@@ -3,7 +3,8 @@
     <td align="right" nowrap="nowrap">
 
 
-<script language="javascript" type="text/javascript">
+<script type="text/javascript">
+//<![CDATA[
     function showhide (id, set) {
         divobj = document.getElementById(id);
         butobj = document.getElementById(id+'button');
@@ -28,6 +29,7 @@
             }
         }
     }
+//]]>
 </script>
 
 
@@ -63,9 +65,11 @@
 
     <td colspan="2">
         <script type="text/javascript">
+        //<![CDATA[
             var popupitems = [<?php echo $popupoptions; ?>];
             var blockitem = [<?php echo $blockoption; ?>];
             var allitems = [<?php echo $alloptions; ?>];
+        //]]>
         </script>
         <input type="radio" name="windowpopup" value="0" alt="<?php print_string('pagewindow', 'resource') ?>" <?php echo ($windowtype != 'popup') ? 'checked="checked"' : '' ?> 
         onclick="lockoptions('form', 'windowpopup[0]', blockitem);
@@ -109,9 +113,9 @@
         <?php p($strheight) ?><br />
         <?php
             if ($windowtype == "page") {
-                echo "<script type=\"text/javascript\">";
+                echo "<script type=\"text/javascript\">\n//<![CDATA[\n";
                 echo "lockoptions('form','windowpopup[1]', popupitems);";
-                echo "</script>";
+                echo "\n//]]>\n</script>";
             }
         ?>
         </blockquote>
@@ -123,8 +127,10 @@
     </div>
 
 
-<script language="javascript" type="text/javascript">
+<script type="text/javascript">
+//<![CDATA[
     showhide('windowsettings', true);
+//]]>
 </script>
 
 
diff --git a/mod/resource/type/ims/ims.html b/mod/resource/type/ims/ims.html
index 674d1ac62e..9969ec7911 100644
--- a/mod/resource/type/ims/ims.html
+++ b/mod/resource/type/ims/ims.html
@@ -29,7 +29,8 @@
     <td align="right" nowrap="nowrap">
 
 
-<script language="javascript" type="text/javascript">
+<script type="text/javascript">
+//<![CDATA[
     function showhide (id, set) {
         divobj = document.getElementById(id);
         butobj = document.getElementById(id+'button');
@@ -75,6 +76,7 @@
         }
 
     }
+//]]>
 </script>
 
 
@@ -113,8 +115,10 @@
 
     <td colspan="2">
         <script type="text/javascript">
+        //<![CDATA[
             var popupitems = [<?php echo $popupoptions; ?>];
             var allitems = [<?php echo $alloptions; ?>];
+        //]]>
         </script>
         <input type="radio" name="windowpopup" value="0" alt="<?php print_string('pagewindow', 'resource') ?>" <?php echo ($windowtype != "popup") ? "checked=\"checked\"" : "" ?> 
         onclick="lockoptions('form', 'windowpopup[1]', popupitems);" />
@@ -153,13 +157,13 @@
         <?php p($strheight) ?><br />
         <?php
             if ($windowtype == "page") {
-                echo "<script type=\"text/javascript\">";
+                echo "<script type=\"text/javascript\">\n//<![CDATA[\n";
                 echo "lockoptions('form','windowpopup[1]', popupitems);";
-                echo "</script>";
+                echo "\n//]]>\n</script>";
             } else {
-                echo "<script type=\"text/javascript\">";
+                echo "<script type=\"text/javascript\">\n//<![CDATA[\n";
                 echo "lockoptions('form','windowpopup[0]', frameitem);";
-                echo "</script>";
+                echo "\n//]]>\n</script>";
             }
         ?>
         </blockquote>
@@ -245,13 +249,13 @@
     
     </div>
 
-<script language="javascript" type="text/javascript"> 
+<script type="text/javascript"> 
+//<![CDATA[
     optiondeselector();
-</script>
 
-<script language="javascript" type="text/javascript">
     showhide('parametersettings', true);
     showhide('windowsettings', true);
+//]]>
 </script>
 
 
diff --git a/mod/resource/type/repository/hive/makelink.php b/mod/resource/type/repository/hive/makelink.php
index 84a37e9638..1889f774bd 100644
--- a/mod/resource/type/repository/hive/makelink.php
+++ b/mod/resource/type/repository/hive/makelink.php
@@ -30,13 +30,13 @@
     
 ?>
 
-<script language="javascript" type="text/javascript">
-<!--
+<script type="text/javascript">
+//<![CDATA[
     opener.document.forms['form'].reference.value = '<?php echo addslashes($resource) ?>';
     opener.document.forms['form'].name.value = '<?php echo addslashes(urldecode($title)) ?>';
     opener.focus();
     window.close();
--->
+//]]>
 </script>
 
 
diff --git a/mod/resource/type/repository/hive/openhive.php b/mod/resource/type/repository/hive/openhive.php
index f9111dea03..0df673eab9 100644
--- a/mod/resource/type/repository/hive/openhive.php
+++ b/mod/resource/type/repository/hive/openhive.php
@@ -60,8 +60,10 @@ $query .= '&HIVE_SESSION='.$SESSION->HIVE_SESSION;
     echo '<input type="hidden" name="HIVE_SEREF" value="'.$CFG->wwwroot.'/sso/hive/expired.php">';
     echo '<input type="hidden" name="HIVE_SESSION" value="'.$SESSION->HIVE_SESSION.'">';
     echo '</form>';
-    echo '<script language="javascript"/>';
+    echo '<script type="text/javascript"/>';
+    echo "\n//<![CDATA[\n";
     echo 'document.OPEN_HIVE_FORM.submit();';
+    echo "\n//]]>\n";
     echo '</script>';
 
     print_footer();
diff --git a/mod/resource/type/repository/repository.html b/mod/resource/type/repository/repository.html
index bb4ba8619a..d18b63b4b1 100644
--- a/mod/resource/type/repository/repository.html
+++ b/mod/resource/type/repository/repository.html
@@ -3,7 +3,8 @@
     <td align="right" nowrap="nowrap">
 
 
-<script language="javascript" type="text/javascript">
+<script type="text/javascript">
+//<![CDATA[
     function showhide (id, set) {
         divobj = document.getElementById(id);
         butobj = document.getElementById(id+'button');
@@ -28,6 +29,7 @@
             }
         }
     }
+//]]>
 </script>
 
 
@@ -84,9 +86,11 @@
 
     <td colspan="2">
         <script type="text/javascript">
+        //<![CDATA[
             var popupitems = [<?php echo $popupoptions; ?>];
             var frameitem = [<?php echo $frameoption; ?>];
             var allitems = [<?php echo $alloptions; ?>];
+        //]]>
         </script>
         <input type="radio" name="windowpopup" value="0" alt="<?php print_string('pagewindow', 'resource') ?>" <?php echo ($windowtype != "popup") ? "checked=\"checked\"" : "" ?> 
         onclick="lockoptions('form', 'windowpopup[0]', frameitem); 
@@ -129,13 +133,13 @@
         <?php p($strheight) ?><br />
         <?php
             if ($windowtype == "page") {
-                echo "<script type=\"text/javascript\">";
+                echo "<script type=\"text/javascript\">\n//<![CDATA[\n";
                 echo "lockoptions('form','windowpopup[1]', popupitems);";
-                echo "</script>";
+                echo "\n//]]>\n</script>";
             } else {
-                echo "<script type=\"text/javascript\">";
+                echo "<script type=\"text/javascript\">\n//<![CDATA[\n";
                 echo "lockoptions('form','windowpopup[0]', frameitem);";
-                echo "</script>";
+                echo "\n//]]>\n</script>";
             }
         ?>
         </blockquote>
@@ -204,9 +208,11 @@ for ($i=0; $i < $this->maxparameters; $i++) {
     </div>
 
 
-<script language="javascript" type="text/javascript">
+<script type="text/javascript">
+//<![CDATA[
     showhide('parametersettings', true);
     showhide('windowsettings', true);
+//]]>
 </script>
 
 
diff --git a/mod/resource/type/text/text.html b/mod/resource/type/text/text.html
index 2723292f65..1789778a2c 100644
--- a/mod/resource/type/text/text.html
+++ b/mod/resource/type/text/text.html
@@ -1,7 +1,8 @@
 
 <tr valign="top">
     <td colspan="2">
-    <script language="javascript" type="text/javascript">
+    <script type="text/javascript">
+    //<![CDATA[
         function showhide (id, set) {
             divobj = document.getElementById(id);
             butobj = document.getElementById(id+'button');
@@ -26,6 +27,7 @@
                 }
             }
         }
+    //]]>
     </script>
     </td>
 </tr>
@@ -76,9 +78,11 @@
 
     <td colspan="2">
         <script type="text/javascript">
+        //<![CDATA[
             var popupitems = [<?php echo $popupoptions; ?>];
             var blockitem = [<?php echo $blockoption; ?>];
             var allitems = [<?php echo $alloptions; ?>];
+        //]]>
         </script>
         <input type="radio" name="windowpopup" value="0" alt="<?php print_string("pagewindow", "resource") ?>" <?php echo ($windowtype != "popup") ? 'checked="checked"' : '' ?> 
         onclick="lockoptions('form', 'windowpopup[0]', blockitem);
@@ -124,9 +128,9 @@
         <?php p($strheight) ?><br />
         <?php
             if ($windowtype == "page") {
-                echo "<script type=\"text/javascript\">";
+                echo "<script type=\"text/javascript\">\n//<![CDATA[\n";
                 echo "lockoptions('form','windowpopup[1]', popupitems);";
-                echo "</script>";
+                echo "\n//]]>\n</script>";
             }
         ?>
         </blockquote>
@@ -138,8 +142,10 @@
     </div>
 
 
-<script language="javascript" type="text/javascript">
+<script type="text/javascript">
+//<![CDATA[
     showhide('windowsettings', true);
+//]]>
 </script>
 
 
diff --git a/mod/scorm/datamodels/aicclib.php b/mod/scorm/datamodels/aicclib.php
index 7881fb61a9..3fea46ca1b 100644
--- a/mod/scorm/datamodels/aicclib.php
+++ b/mod/scorm/datamodels/aicclib.php
@@ -1,4 +1,4 @@
-<?php
+<?php // $Id$
 function scorm_add_time($a, $b) {
     $aes = explode(':',$a);
     $bes = explode(':',$b);
@@ -467,8 +467,8 @@ function scorm_get_toc($user,$scorm,$liststyle,$currentorg='',$scoid='',$mode='n
     $result->toc .= "\t</ul>\n";
     if ($scorm->hidetoc == 0) {
         $result->toc .= '
-          <script language="javascript" type="text/javascript">
-          <!--
+          <script type="text/javascript">
+          //<![CDATA[
               function expandCollide(which,list,item) {
                   var nn=document.ids?true:false
                   var w3c=document.getElementById?true:false
@@ -485,7 +485,7 @@ function scorm_get_toc($user,$scorm,$liststyle,$currentorg='',$scoid='',$mode='n
                       new cookie("hide:SCORMitem" + item, 1, -1, "/").set();
                   }
               }
-          -->
+          //]]>
           </script>'."\n";
     }
     
diff --git a/mod/scorm/datamodels/scorm_12lib.php b/mod/scorm/datamodels/scorm_12lib.php
index c5caf14e5f..f4c9f929a9 100644
--- a/mod/scorm/datamodels/scorm_12lib.php
+++ b/mod/scorm/datamodels/scorm_12lib.php
@@ -1,4 +1,4 @@
-<?php
+<?php // $Id$
 
 function scorm_eval_prerequisites($prerequisites,$usertracks) {
     $element = '';
@@ -363,8 +363,8 @@ function scorm_get_toc($user,$scorm,$liststyle,$currentorg='',$scoid='',$mode='n
     $result->toc .= "\t</ul>\n";
     if ($scorm->hidetoc == 0) {
         $result->toc .= '
-          <script language="javascript" type="text/javascript">
-          <!--
+          <script type="text/javascript">
+          //<![CDATA[
               function expandCollide(which,list,item) {
                   var nn=document.ids?true:false
                   var w3c=document.getElementById?true:false
@@ -381,7 +381,7 @@ function scorm_get_toc($user,$scorm,$liststyle,$currentorg='',$scoid='',$mode='n
                       new cookie("hide:SCORMitem" + item, 1, -1, "/").set();
                   }
               }
-          -->
+          //]]>
           </script>'."\n";
     }
     
diff --git a/mod/scorm/datamodels/scorm_13lib.php b/mod/scorm/datamodels/scorm_13lib.php
index 9a25e752dd..852e37c5f1 100644
--- a/mod/scorm/datamodels/scorm_13lib.php
+++ b/mod/scorm/datamodels/scorm_13lib.php
@@ -1,4 +1,4 @@
-<?php
+<?php // $Id$
 /*                // Added by Pham Minh Duc
                 case 'IMSSS:SEQUENCING':
                     $parent = array_pop($parents);
@@ -534,8 +534,8 @@ function scorm_get_toc($user,$scorm,$liststyle,$currentorg='',$scoid='',$mode='n
     $result->toc .= "\t</ul>\n";
     if ($scorm->hidetoc == 0) {
         $result->toc .= '
-          <script language="javascript" type="text/javascript">
-          <!--
+          <script type="text/javascript">
+          //<![CDATA[
               function expandCollide(which,list,item) {
                   var nn=document.ids?true:false
                   var w3c=document.getElementById?true:false
@@ -552,7 +552,7 @@ function scorm_get_toc($user,$scorm,$liststyle,$currentorg='',$scoid='',$mode='n
                       new cookie("hide:SCORMitem" + item, 1, -1, "/").set();
                   }
               }
-          -->
+          //]]>
           </script>'."\n";
     }
     
diff --git a/mod/scorm/loadSCO.php b/mod/scorm/loadSCO.php
index 1d32290dc4..0a144ac012 100755
--- a/mod/scorm/loadSCO.php
+++ b/mod/scorm/loadSCO.php
@@ -1,4 +1,4 @@
-<?php
+<?php  // $Id$
     require_once('../../config.php');
     require_once('locallib.php');
 
@@ -109,10 +109,10 @@
 <html>
     <head>
         <title>LoadSCO</title>
-        <script language="javascript" type="text/javascript">
-        <!--
+        <script type="text/javascript">
+        //<![CDATA[
             setTimeout('document.location = "<?php echo $result ?>";',2000);
-        -->
+        //]]>
         </script>
         <noscript>
             <meta http-equiv="refresh" content="2;url=<?php echo $result ?>" />
diff --git a/mod/scorm/mod.html b/mod/scorm/mod.html
index 97a57ab2f2..e546cf5e4f 100755
--- a/mod/scorm/mod.html
+++ b/mod/scorm/mod.html
@@ -1,4 +1,4 @@
-<?php
+<?php  // $Id$
     require_once($CFG->dirroot.'/mod/scorm/locallib.php');
     if (!isset($form->name)) {
         $form->name = '';
@@ -458,9 +458,11 @@
                         </tr>
                     </table>
                 </div>
-                <script language="javascript" type="text/javascript">
+                <script type="text/javascript">
+                //<![CDATA[
                     showhide('advancedsettings', true);
                     showhide('windowsettings', true);
+                //]]>
                 </script>
             </td>
         </tr>
diff --git a/mod/scorm/player.php b/mod/scorm/player.php
index 51171eba43..2b8ab4e5ce 100755
--- a/mod/scorm/player.php
+++ b/mod/scorm/player.php
@@ -130,21 +130,21 @@
                  '', '', true, $exitlink.update_module_button($cm->id, $course->id, $strscorm), '', false, $bodyscript);
     if ($sco->scormtype == 'sco') {
 ?>
-    <script language="JavaScript" type="text/javascript" src="request.js"></script>
-    <script language="JavaScript" type="text/javascript" src="api.php?id=<?php echo $cm->id.$scoidstr.$modestr.$attemptstr ?>"></script>
+    <script type="text/javascript" src="request.js"></script>
+    <script type="text/javascript" src="api.php?id=<?php echo $cm->id.$scoidstr.$modestr.$attemptstr ?>"></script>
 <?php
     }
     if (($sco->previd != 0) && ((!isset($sco->previous)) || ($sco->previous == 0))) {
         $scostr = '&scoid='.$sco->previd;
-        echo '    <script language="javascript">var prev="'.$CFG->wwwroot.'/mod/scorm/player.php?id='.$cm->id.$orgstr.$modepop.$scostr."\";</script>\n";
+        echo '    <script type="text/javascript">'."\n//<![CDATA[\n".'var prev="'.$CFG->wwwroot.'/mod/scorm/player.php?id='.$cm->id.$orgstr.$modepop.$scostr."\";\n//]]>\n</script>\n";
     } else {
-        echo '    <script language="javascript">var prev="'.$CFG->wwwroot.'/mod/scorm/view.php?id='.$cm->id."\";</script>\n";
+        echo '    <script type="text/javascript">'."\n//<![CDATA[\n".'var prev="'.$CFG->wwwroot.'/mod/scorm/view.php?id='.$cm->id."\";\n//]]>\n</script>\n";
     }
     if (($sco->nextid != 0) && ((!isset($sco->next)) || ($sco->next == 0))) {
         $scostr = '&scoid='.$sco->nextid;
-        echo '    <script language="javascript">var next="'.$CFG->wwwroot.'/mod/scorm/player.php?id='.$cm->id.$orgstr.$modepop.$scostr."\";</script>\n";
+        echo '    <script type="text/javascript">'."\n//<![CDATA[\n".'var next="'.$CFG->wwwroot.'/mod/scorm/player.php?id='.$cm->id.$orgstr.$modepop.$scostr."\";\n//]]>\n</script>\n";
     } else {
-        echo '    <script language="javascript">var next="'.$CFG->wwwroot.'/mod/scorm/view.php?id='.$cm->id."\";</script>\n";
+        echo '    <script type="text/javascript">'."\n//<![CDATA[\n".'var next="'.$CFG->wwwroot.'/mod/scorm/view.php?id='.$cm->id."\";\n//]]>\n</script>\n";
     }
 ?>
     <div id="scormpage">
@@ -237,7 +237,8 @@
 <?php
         } else {
 ?>
-                    <script lanuguage="javascript">
+                    <script type="text/javascript">
+                    //<![CDATA[
                         function openpopup(url,name,options,width,height) {
                             fullurl = "<?php echo $CFG->wwwroot.'/mod/scorm/' ?>" + url;
                             windowobj = window.open(fullurl,name,options);
@@ -260,6 +261,7 @@
                         width = <?php p($scorm->width) ?>;
                         height = <?php p($scorm->height) ?>;
                         var main = openpopup(url, "scormpopup", "<?php p($scorm->options) ?>", width, height);
+                    //]]>
                     </script>
                     <noscript>
                     <iframe id="main"
@@ -282,4 +284,3 @@
     </div> <!-- Page -->
 </body>
 </html>
-
diff --git a/mod/wiki/ewiki/ewiki.php b/mod/wiki/ewiki/ewiki.php
index 6fbbcb5901..2958a70ad9 100644
--- a/mod/wiki/ewiki/ewiki.php
+++ b/mod/wiki/ewiki/ewiki.php
@@ -2575,7 +2575,7 @@ function ewiki_binary($break=0) {
 
       if ($id) {
          echo<<<EOF
-<html><head><title>File/Picture Upload</title><script language="JavaScript" type="text/javascript"><!--
+<html><head><title>File/Picture Upload</title><script type="text/javascript"><!--
  opener.document.forms["ewiki"].elements["content"].value += "\\nUPLOADED PICTURE: [$id$title]\\n";
  window.setTimeout("self.close()", 5000);
 //--></script></head><body bgcolor="#440707" text="#FFFFFF">Your uploaded file was saved as<br /><big><b>
diff --git a/mod/wiki/ewiki/fragments/homepage.src b/mod/wiki/ewiki/fragments/homepage.src
index 624c5ea2e8..eb5befba74 100644
--- a/mod/wiki/ewiki/fragments/homepage.src
+++ b/mod/wiki/ewiki/fragments/homepage.src
@@ -90,7 +90,7 @@
  //-->
  </style>
 
- <script language="JavaScript" type="text/javascript">
+ <script type="text/javascript">
  <!--
 
    function login()
@@ -169,4 +169,4 @@
 </CENTER>
 
 </BODY>
-</HTML>
\ No newline at end of file
+</HTML>
diff --git a/question/type/calculated/editquestion.html b/question/type/calculated/editquestion.html
index 22306d2df7..6bad383d97 100644
--- a/question/type/calculated/editquestion.html
+++ b/question/type/calculated/editquestion.html
@@ -84,7 +84,8 @@ $QTYPES[$question->qtype]->print_question_form_end($question,
         'onclick="return determineMinAndMax();"',
         '<input type="hidden" name="wizardpage" value="question" />');
 ?>
-<script language="javascript" type="text/javascript">
+<script type="text/javascript">
+//<![CDATA[
 function determineMinAndMax() {
     // This client-side script will determine the values for min and max
     // based on the input for answer and acceptederror.
@@ -111,4 +112,5 @@ function determineMinAndMax() {
         }
     }
 }
+//]]>
 </script>
diff --git a/theme/standardwhite/footer.html b/theme/standardwhite/footer.html
index d890f3fe7f..6d76503551 100644
--- a/theme/standardwhite/footer.html
+++ b/theme/standardwhite/footer.html
@@ -13,7 +13,21 @@
 <?php echo $loggedinas ?>
 <?php echo $homelink ?>
 
+
+<?php if (isadmin()) { 
+          echo $performanceinfo;
+?>
+          <br />
+          <hr noshade="noshade" />
+          <p align="center">
+          <a href="http://validator.w3.org/check?verbose=1&amp;ss=1&amp;uri=<?php echo urlencode(qualified_me()) ?>">Validate HTML</a> |
+          <a href="http://www.contentquality.com/mynewtester/cynthia.exe?rptmode=-1&amp;url1=<?php echo urlencode(qualified_me()) ?>">Section 508 Check</a> |
+          <a href="http://www.contentquality.com/mynewtester/cynthia.exe?rptmode=0&amp;warnp2n3e=1&amp;url1=<?php echo urlencode(qualified_me()) ?>">WCAG 1 (2,3) Check</a>
+          </p>
+<?php } ?>
+
+
 </div>
 </div>
 </body>
-</html>
\ No newline at end of file
+</html>
-- 
2.39.5