* Revert r55555-55557 for now, breaks tabs in special preferences, don't know why
authorNiklas Laxström <nikerabbit@users.mediawiki.org>
Tue, 25 Aug 2009 13:56:42 +0000 (13:56 +0000)
committerNiklas Laxström <nikerabbit@users.mediawiki.org>
Tue, 25 Aug 2009 13:56:42 +0000 (13:56 +0000)
Also, the following looks wrong:
-       for (var id = 0; id < ta.length; id++) {
+       for (var id in ta) {
                var n = document.getElementById(id);

skins/common/mwsuggest.js
skins/common/protect.js
skins/common/wikibits.js

index 8e37a48..77792b0 100644 (file)
@@ -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]);
index 874f92e..d9650c8 100644 (file)
@@ -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-/)) {
index d85cfec..ed58877 100644 (file)
@@ -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);