From: Niklas Laxström <nikerabbit@users.mediawiki.org>
Date: Tue, 25 Aug 2009 13:56:42 +0000 (+0000)
Subject: * Revert r55555-55557 for now, breaks tabs in special preferences, don't know why
X-Git-Tag: 1.31.0-rc.0~40069
X-Git-Url: http://git.cyclocoop.org//%27%40script%40/%27?a=commitdiff_plain;h=26d55f5e34962db9cc10085af1277256f8479ce8;p=lhc%2Fweb%2Fwiklou.git

* Revert r55555-55557 for now, breaks tabs in special preferences, don't know why
Also, the following looks wrong:
-       for (var id = 0; id < ta.length; id++) {
+       for (var id in ta) {
                var n = document.getElementById(id);
---

diff --git a/skins/common/mwsuggest.js b/skins/common/mwsuggest.js
index 8e37a4859e..77792b0616 100644
--- a/skins/common/mwsuggest.js
+++ b/skins/common/mwsuggest.js
@@ -24,8 +24,8 @@ var os_mouse_moved = false;
 // delay between keypress and suggestion (in ms)
 var os_search_timeout = 250;
 // these pairs of inputs/forms will be autoloaded at startup
-var os_autoload_inputs = ['searchInput', 'searchInput2', 'powerSearchText', 'searchText'];
-var os_autoload_forms = ['searchform', 'searchform2', 'powersearch', 'search'];
+var os_autoload_inputs = new Array('searchInput', 'searchInput2', 'powerSearchText', 'searchText');
+var os_autoload_forms = new Array('searchform', 'searchform2', 'powersearch', 'search' );
 // if we stopped the service
 var os_is_stopped = false;
 // max lines to show in suggest table
@@ -364,7 +364,7 @@ function os_createResultTable(r, results){
 	var c = document.getElementById(r.container);
 	var width = c.offsetWidth - os_operaWidthFix(c.offsetWidth);
 	var html = "<table class=\"os-suggest-results\" id=\""+r.resultTable+"\" style=\"width: "+width+"px;\">";
-	r.results = [];
+	r.results = new Array();
 	r.resultCount = results.length;
 	for(i=0;i<results.length;i++){
 		var title = os_decodeValue(results[i]);
diff --git a/skins/common/protect.js b/skins/common/protect.js
index 874f92e8a8..d9650c829a 100644
--- a/skins/common/protect.js
+++ b/skins/common/protect.js
@@ -250,7 +250,7 @@ var ProtectionForm = {
 	 */
 	'getLevelSelectors': function() {
 		var all = document.getElementsByTagName("select");
-		var ours = [];
+		var ours = new Array();
 		for (var i = 0; i < all.length; i++) {
 			var element = all[i];
 			if (element.id.match(/^mwProtect-level-/)) {
@@ -279,7 +279,7 @@ var ProtectionForm = {
 	 */
 	'getExpiryInputs': function() {
 		var all = document.getElementsByTagName("input");
-		var ours = [];
+		var ours = new Array();
 		for (var i = 0; i < all.length; i++) {
 			var element = all[i];
 			if (element.name.match(/^mwProtect-expiry-/)) {
@@ -307,7 +307,7 @@ var ProtectionForm = {
 	 */
 	'getExpirySelectors': function() {
 		var all = document.getElementsByTagName("select");
-		var ours = [];
+		var ours = new Array();
 		for (var i = 0; i < all.length; i++) {
 			var element = all[i];
 			if (element.id.match(/^mwProtectExpirySelection-/)) {
diff --git a/skins/common/wikibits.js b/skins/common/wikibits.js
index d85cfec8f8..ed58877485 100644
--- a/skins/common/wikibits.js
+++ b/skins/common/wikibits.js
@@ -181,14 +181,23 @@ var mwEditButtons = [];
 var mwCustomEditButtons = []; // eg to add in MediaWiki:Common.js
 
 function escapeQuotes(text) {
-	return escapeQuotesHTML(text.replace(/'/g,"\\'").replace(/\n/g,"\\n"));
+	var re = new RegExp("'","g");
+	text = text.replace(re,"\\'");
+	re = new RegExp("\\n","g");
+	text = text.replace(re,"\\n");
+	return escapeQuotesHTML(text);
 }
 
 function escapeQuotesHTML(text) {
-	return text.replace(/&/g,"&amp;")
-		.replace(/"/g,"&quot;")
-		.replace(/</gre,"&lt;")
-		.replace(/>/g,"&gt;");
+	var re = new RegExp('&',"g");
+	text = text.replace(re,"&amp;");
+	re = new RegExp('"',"g");
+	text = text.replace(re,"&quot;");
+	re = new RegExp('<',"g");
+	text = text.replace(re,"&lt;");
+	re = new RegExp('>',"g");
+	text = text.replace(re,"&gt;");
+	return text;
 }
 
 
@@ -299,7 +308,7 @@ function addPortletLink(portlet, href, text, id, tooltip, accesskey, nextnode) {
 		link.setAttribute( "title", tooltip );
 	}
 	if ( accesskey && tooltip ) {
-		updateTooltipAccessKeys( [link] );
+		updateTooltipAccessKeys( new Array( link ) );
 	}
 
 	if ( nextnode && nextnode.parentNode == node )
@@ -345,7 +354,7 @@ function akeytt( doId ) {
 	// A lot of user scripts (and some of the code below) break if
 	// ta isn't defined, so we make sure it is.  Explictly using
 	// window.ta avoids a "ta is not defined" error.
-	if (!window.ta) window.ta = [];
+	if (!window.ta) window.ta = new Array;
 
 	// Make a local, possibly restricted, copy to avoid clobbering
 	// the original.
@@ -358,7 +367,7 @@ function akeytt( doId ) {
 
 	// Now deal with evil deprecated ta
 	var watchCheckboxExists = document.getElementById( 'wpWatchthis' ) ? true : false;
-	for (var id = 0; id < ta.length; id++) {
+	for (var id in ta) {
 		var n = document.getElementById(id);
 		if (n) {
 			var a = null;
@@ -489,7 +498,7 @@ function toggle_element_check(ida,idb) {
 	From http://www.robertnyman.com/2005/11/07/the-ultimate-getelementsbyclassname/
 */
 function getElementsByClassName(oElm, strTagName, oClassNames){
-	var arrReturnElements = [];
+	var arrReturnElements = new Array();
 	if ( typeof( oElm.getElementsByClassName ) == "function" ) {
 		/* Use a native implementation where possible FF3, Saf3.2, Opera 9.5 */
 		var arrNativeReturn = oElm.getElementsByClassName( oClassNames );
@@ -502,7 +511,7 @@ function getElementsByClassName(oElm, strTagName, oClassNames){
 		return arrReturnElements;
 	}
 	var arrElements = (strTagName == "*" && oElm.all)? oElm.all : oElm.getElementsByTagName(strTagName);
-	var arrRegExpClassNames = [];
+	var arrRegExpClassNames = new Array();
 	if(typeof oClassNames == "object"){
 		for(var i=0; i<oClassNames.length; i++){
 			arrRegExpClassNames[arrRegExpClassNames.length] =
@@ -671,8 +680,8 @@ function ts_resortTable(lnk) {
 
 	var reverse = (span.getAttribute("sortdir") == 'down');
 
-	var newRows = [];
-	var staticRows = [];
+	var newRows = new Array();
+	var staticRows = new Array();
 	for (var j = rowStart; j < table.rows.length; j++) {
 		var row = table.rows[j];
 		if((" "+row.className+" ").indexOf(" unsortable ") < 0) {
@@ -680,8 +689,8 @@ function ts_resortTable(lnk) {
 			var oldIndex = (reverse ? -j : j);
 			var preprocessed = preprocessor( keyText.replace(/^[\s\xa0]+/, "").replace(/[\s\xa0]+$/, "") );
 
-			newRows[newRows.length] = [row, preprocessed, oldIndex];
-		} else staticRows[staticRows.length] = [row, false, j-rowStart];
+			newRows[newRows.length] = new Array(row, preprocessed, oldIndex);
+		} else staticRows[staticRows.length] = new Array(row, false, j-rowStart);
 	}
 
 	newRows.sort(sortfn);