From: Amir E. Aharoni Date: Tue, 6 Oct 2015 08:32:34 +0000 (+0300) Subject: Consistent TODO spelling in JavaScript X-Git-Tag: 1.31.0-rc.0~9464^2 X-Git-Url: http://git.cyclocoop.org/%7B%24www_url%7Dadmin/compta/operations/?a=commitdiff_plain;h=42054295254dcba8ef03c1f9d95db333874329ce;p=lhc%2Fweb%2Fwiklou.git Consistent TODO spelling in JavaScript Almost all JavaScript files in core use "TODO", in all caps and without @. Onyl a few files used "todo", "@todo", or "Todo", and this is fixed here for consistency. Change-Id: I7478868c07f3a0bd16196ca1cfeb7f8f60fa2a9d --- diff --git a/resources/src/jquery/jquery.hidpi.js b/resources/src/jquery/jquery.hidpi.js index aa6590bf50..7d308f8e84 100644 --- a/resources/src/jquery/jquery.hidpi.js +++ b/resources/src/jquery/jquery.hidpi.js @@ -108,7 +108,7 @@ $.bracketedDevicePixelRatio = function () { */ $.fn.hidpi = function () { var $target = this, - // @todo add support for dpi media query checks on Firefox, IE + // TODO add support for dpi media query checks on Firefox, IE devicePixelRatio = $.devicePixelRatio(), testImage = new Image(); diff --git a/resources/src/mediawiki.special/mediawiki.special.upload.js b/resources/src/mediawiki.special/mediawiki.special.upload.js index 76933408e1..9fcec022e1 100644 --- a/resources/src/mediawiki.special/mediawiki.special.upload.js +++ b/resources/src/mediawiki.special/mediawiki.special.upload.js @@ -426,7 +426,7 @@ var reader = new FileReader(); if ( callbackBinary && 'readAsBinaryString' in reader ) { // To fetch JPEG metadata we need a binary string; start there. - // todo: + // TODO reader.onload = function () { callbackBinary( reader.result ); diff --git a/resources/src/mediawiki.special/mediawiki.special.userlogin.signup.js b/resources/src/mediawiki.special/mediawiki.special.userlogin.signup.js index a0c6ee2a63..24f54d0ada 100644 --- a/resources/src/mediawiki.special/mediawiki.special.userlogin.signup.js +++ b/resources/src/mediawiki.special/mediawiki.special.userlogin.signup.js @@ -123,7 +123,7 @@ .empty() .append( // Ugh… - // @todo Change the HTML structure in includes/templates/Usercreate.php + // TODO Change the HTML structure in includes/templates/Usercreate.php $( '' ).text( mw.message( 'createacct-error' ).text() ), $( '
' ), document.createTextNode( message ) diff --git a/resources/src/mediawiki.widgets/mw.widgets.TitleWidget.js b/resources/src/mediawiki.widgets/mw.widgets.TitleWidget.js index 5732aa5c12..0ef50950c0 100644 --- a/resources/src/mediawiki.widgets/mw.widgets.TitleWidget.js +++ b/resources/src/mediawiki.widgets/mw.widgets.TitleWidget.js @@ -140,7 +140,7 @@ params.wbptterms = 'description'; } req = new mw.Api().get( params ); - promiseAbortObject.abort = req.abort.bind( req ); // todo: ew + promiseAbortObject.abort = req.abort.bind( req ); // TODO ew return req; } } ).promise( promiseAbortObject ); diff --git a/resources/src/mediawiki/page/image-pagination.js b/resources/src/mediawiki/page/image-pagination.js index 49a51dfc23..d858b62b94 100644 --- a/resources/src/mediawiki/page/image-pagination.js +++ b/resources/src/mediawiki/page/image-pagination.js @@ -26,7 +26,7 @@ return $.Deferred().resolve( cache[ url ] ).promise(); } - // @todo Don't fetch the entire page. Ideally we'd only fetch the content portion or the data + // TODO Don't fetch the entire page. Ideally we'd only fetch the content portion or the data // (thumbnail urls) and update the interface manually. jqXhr = $.ajax( url ).then( function ( data ) { return $( data ).find( 'table.multipageimage' ).contents(); diff --git a/tests/qunit/suites/resources/mediawiki/mediawiki.test.js b/tests/qunit/suites/resources/mediawiki/mediawiki.test.js index cec0552071..ac4f16c46c 100644 --- a/tests/qunit/suites/resources/mediawiki/mediawiki.test.js +++ b/tests/qunit/suites/resources/mediawiki/mediawiki.test.js @@ -193,7 +193,7 @@ assert.equal( hello.key, 'hello', 'Message property "key" (currect key)' ); assert.deepEqual( hello.parameters, [], 'Message property "parameters" defaults to an empty array' ); - // Todo + // TODO assert.ok( hello.params, 'Message prototype "params"' ); hello.format = 'plain';