From 6944909a8c7e052d5070516adaf6f33eaa44568d Mon Sep 17 00:00:00 2001
From: Brion Vibber <brion@users.mediawiki.org>
Date: Thu, 14 Dec 2006 23:24:06 +0000
Subject: [PATCH] * (bug 8264) Fix JavaScript global vars for XHTML mode Don't
 use <!-- --> to wrap JavaScript code! This ACTUALLY MAKES THE CODE DISAPPEAR
 when using an XML parser. Instead, wrapped in /*<![CDATA[*/ /*]]>*/ to ensure
 XML compatibility for contents.

We've long since given up on super-ancient browsers that don't
know about <script>; they're so old you can't even use name-based
virtual hosts so why bother? ;)
---
 RELEASE-NOTES     | 1 +
 includes/Skin.php | 4 ++--
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/RELEASE-NOTES b/RELEASE-NOTES
index d83fb48ffe..83449e07d2 100644
--- a/RELEASE-NOTES
+++ b/RELEASE-NOTES
@@ -319,6 +319,7 @@ it from source control: http://www.mediawiki.org/wiki/Download_from_SVN
 * User::getOption now accept a default value to override default user values
   this makes it consistent with WebRequest::get* methods. Corrected code in
   various places accordingly.
+* (bug 8264) Fix JavaScript global vars for XHTML mode
 
 
 == Languages updated ==
diff --git a/includes/Skin.php b/includes/Skin.php
index 0e415505a8..8a297e4fd8 100644
--- a/includes/Skin.php
+++ b/includes/Skin.php
@@ -272,12 +272,12 @@ class Skin extends Linker {
 	static function makeVariablesScript( $data ) {
 		global $wgJsMimeType;
 
-		$r = "<script type= \"$wgJsMimeType\"><!--\n";
+		$r = "<script type= \"$wgJsMimeType\">/*<![CDATA[*/\n";
 		foreach ( $data as $name => $value ) {
 			$encValue = Xml::encodeJsVar( $value );
 			$r .= "var $name = $encValue;\n";
 		}
-		$r .= "--></script>\n";
+		$r .= "/*]]>*/</script>\n";
 
 		return $r;
 	}
-- 
2.20.1