From bfb6c23266db263158c6f37b7210bedc033e1890 Mon Sep 17 00:00:00 2001 From: Sam Reed Date: Mon, 6 Dec 2010 22:34:15 +0000 Subject: [PATCH] (bug 26265) Many semicolons are missing from JavaScript files And some more unterminated statements.. --- resources/jquery/jquery.collapsibleTabs.js | 2 +- resources/jquery/jquery.textSelection.js | 2 +- resources/mediawiki.language/languages/bs.js | 2 +- resources/mediawiki.language/languages/mt.js | 2 +- resources/mediawiki/mediawiki.js | 4 +-- skins/common/wikibits.js | 32 ++++++++++---------- 6 files changed, 22 insertions(+), 22 deletions(-) diff --git a/resources/jquery/jquery.collapsibleTabs.js b/resources/jquery/jquery.collapsibleTabs.js index 195ca86389..47811b02f4 100644 --- a/resources/jquery/jquery.collapsibleTabs.js +++ b/resources/jquery/jquery.collapsibleTabs.js @@ -112,4 +112,4 @@ $.collapsibleTabs.handleResize(); } }; -} )( jQuery ) \ No newline at end of file +} )( jQuery ); \ No newline at end of file diff --git a/resources/jquery/jquery.textSelection.js b/resources/jquery/jquery.textSelection.js index ebbfebc7e1..e21768e221 100644 --- a/resources/jquery/jquery.textSelection.js +++ b/resources/jquery/jquery.textSelection.js @@ -173,7 +173,7 @@ encapsulateSelection: function( options ) { if ( preRange.compareEndPoints( "StartToEnd", preRange ) == 0 ) { preFinished = true; } else { - preRange.moveEnd( "character", -1 ) + preRange.moveEnd( "character", -1 ); if ( preRange.text == preText ) { rawPreText += "\r\n"; } else { diff --git a/resources/mediawiki.language/languages/bs.js b/resources/mediawiki.language/languages/bs.js index 87d756e60c..9f22426ae4 100644 --- a/resources/mediawiki.language/languages/bs.js +++ b/resources/mediawiki.language/languages/bs.js @@ -17,4 +17,4 @@ mediaWiki.language.convertPlural = function( count, forms ) { default: return forms[2]; } -} +}; diff --git a/resources/mediawiki.language/languages/mt.js b/resources/mediawiki.language/languages/mt.js index 1e27383e88..44c7bfffc1 100644 --- a/resources/mediawiki.language/languages/mt.js +++ b/resources/mediawiki.language/languages/mt.js @@ -14,4 +14,4 @@ mediaWiki.language.convertPlural = function( count, forms ) { return forms[2]; } return forms[3]; -} +}; diff --git a/resources/mediawiki/mediawiki.js b/resources/mediawiki/mediawiki.js index e886e70ad2..66fbedc170 100644 --- a/resources/mediawiki/mediawiki.js +++ b/resources/mediawiki/mediawiki.js @@ -910,7 +910,7 @@ window.mediaWiki = new ( function( $ ) { return formatVersionNumber( registry[module].version ); } return null; - } + }; /* Cache document ready status */ @@ -954,7 +954,7 @@ window.mediaWiki = new ( function( $ ) { */ this.Cdata = function( value ) { this.value = value; - } + }; /** * Create an HTML element string, with safe escaping. diff --git a/skins/common/wikibits.js b/skins/common/wikibits.js index 0a949ee62d..7b97830afd 100644 --- a/skins/common/wikibits.js +++ b/skins/common/wikibits.js @@ -763,7 +763,7 @@ window.ts_resortTable = function( lnk ) { if ( ts_alternate_row_colors ) { ts_alternate( table ); } -} +}; window.ts_initTransformTable = function() { if ( typeof wgSeparatorTransformTable == 'undefined' @@ -816,11 +816,11 @@ window.ts_initTransformTable = function() { "[-+\u2212]?" + digitClass + "+%?" + // Generic localised ")$", "i" ); -} +}; window.ts_toLowerCase = function( s ) { return s.toLowerCase(); -} +}; window.ts_dateToSortKey = function( date ) { // y2k notes: two digit years less than 50 are treated as 20XX, greater than 50 are treated as 19XX @@ -885,7 +885,7 @@ window.ts_dateToSortKey = function( date ) { } } return '00000000'; -} +}; window.ts_parseFloat = function( s ) { if ( !s ) { @@ -906,15 +906,15 @@ window.ts_parseFloat = function( s ) { } var num = parseFloat( s.replace(/[, ]/g, '').replace("\u2212", '-') ); return ( isNaN( num ) ? -Infinity : num ); -} +}; window.ts_currencyToSortKey = function( s ) { return ts_parseFloat(s.replace(/[^-\u22120-9.,]/g,'')); -} +}; window.ts_sort_generic = function( a, b ) { return a[1] < b[1] ? -1 : a[1] > b[1] ? 1 : a[2] - b[2]; -} +}; window.ts_alternate = function( table ) { // Take object table and get all it's tbodies. @@ -937,7 +937,7 @@ window.ts_alternate = function( table ) { tableRows[j].className = newClassName + ( j % 2 == 0 ? 'even' : 'odd' ); } } -} +}; /* * End of table sorting code @@ -997,7 +997,7 @@ window.jsMsg = function( message, className ) { messageDiv.innerHTML = message; } return true; -} +}; /** * Inject a cute little progress spinner after the specified element @@ -1015,7 +1015,7 @@ window.injectSpinner = function( element, id ) { } else { element.parentNode.appendChild( spinner ); } -} +}; /** * Remove a progress spinner added with injectSpinner() @@ -1027,7 +1027,7 @@ window.removeSpinner = function( id ) { if( spinner ) { spinner.parentNode.removeChild( spinner ); } -} +}; window.runOnloadHook = function() { // don't run anything below this for non-dom browsers @@ -1047,7 +1047,7 @@ window.runOnloadHook = function() { for ( var i = 0; i < onloadFuncts.length; i++ ) { onloadFuncts[i](); } -} +}; /** * Add an event handler to an element @@ -1062,11 +1062,11 @@ window.addHandler = function( element, attach, handler ) { } else if( element.attachEvent ) { element.attachEvent( 'on' + attach, handler ); } -} +}; window.hookEvent = function( hookName, hookFunct ) { addHandler( window, hookName, hookFunct ); -} +}; /** * Add a click event handler to an element @@ -1076,7 +1076,7 @@ window.hookEvent = function( hookName, hookFunct ) { */ window.addClickHandler = function( element, handler ) { addHandler( element, 'click', handler ); -} +}; /** * Removes an event handler from an element @@ -1091,7 +1091,7 @@ window.removeHandler = function( element, remove, handler ) { } else if( window.detachEvent ) { element.detachEvent( 'on' + remove, handler ); } -} +}; // note: all skins should call runOnloadHook() at the end of html output, // so the below should be redundant. It's there just in case. hookEvent( 'load', runOnloadHook ); -- 2.20.1