Consistent TODO spelling in JavaScript
authorAmir E. Aharoni <amir.aharoni@mail.huji.ac.il>
Tue, 6 Oct 2015 08:32:34 +0000 (11:32 +0300)
committerAmir E. Aharoni <amir.aharoni@mail.huji.ac.il>
Tue, 6 Oct 2015 08:32:34 +0000 (11:32 +0300)
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

resources/src/jquery/jquery.hidpi.js
resources/src/mediawiki.special/mediawiki.special.upload.js
resources/src/mediawiki.special/mediawiki.special.userlogin.signup.js
resources/src/mediawiki.widgets/mw.widgets.TitleWidget.js
resources/src/mediawiki/page/image-pagination.js
tests/qunit/suites/resources/mediawiki/mediawiki.test.js

index aa6590b..7d308f8 100644 (file)
@@ -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();
 
index 7693340..9fcec02 100644 (file)
                        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 );
 
index a0c6ee2..24f54d0 100644 (file)
                                                .empty()
                                                .append(
                                                        // Ugh…
-                                                       // @todo Change the HTML structure in includes/templates/Usercreate.php
+                                                       // TODO Change the HTML structure in includes/templates/Usercreate.php
                                                        $( '<strong>' ).text( mw.message( 'createacct-error' ).text() ),
                                                        $( '<br>' ),
                                                        document.createTextNode( message )
index 5732aa5..0ef5095 100644 (file)
                                                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 );
index 49a51df..d858b62 100644 (file)
@@ -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();
index cec0552..ac4f16c 100644 (file)
                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';