Adding comma's at end of arrays where this hasn't been done already for consistency

diff --git a/resources/Resources.php b/resources/Resources.php
index 5760562..e157e48 100644 (file)
--- a/resources/Resources.php
+++ b/resources/Resources.php
@@ -12,14 +12,14 @@ return array(
        /* Skins */
 
        'skins.vector' => array(
-               'styles' => array( 'skins/vector/screen.css' => array( 'media' => 'screen' ) )
+               'styles' => array( 'skins/vector/screen.css' => array( 'media' => 'screen' ) ),
        ),
        'skins.monobook' => array(
                'styles' => array(
                        'skins/monobook/main.css' => array( 'media' => 'screen' ),
                        // Honor $wgHandheldStyle. This is kind of evil
-                       //$GLOBALS['wgHandheldStyle'] => array( 'media' => 'handheld' )
-               )
+                       //$GLOBALS['wgHandheldStyle'] => array( 'media' => 'handheld' ),
+               ),
        ),
 
        /* jQuery */
@@ -32,48 +32,48 @@ return array(
        /* jQuery Plugins */
 
        'jquery.async' => array(
-               'scripts' => 'resources/jquery/jquery.async.js'
+               'scripts' => 'resources/jquery/jquery.async.js',
        ),
        'jquery.autoEllipsis' => array(
                'scripts' => 'resources/jquery/jquery.autoEllipsis.js',
                'dependencies' => 'jquery.highlightText',
        ),
        'jquery.checkboxShiftClick' => array(
-               'scripts' => 'resources/jquery/jquery.checkboxShiftClick.js'
+               'scripts' => 'resources/jquery/jquery.checkboxShiftClick.js',
        ),
        'jquery.client' => array(
                'scripts' => 'resources/jquery/jquery.client.js',
        ),
        'jquery.collapsibleTabs' => array(
-               'scripts' => 'resources/jquery/jquery.collapsibleTabs.js'
+               'scripts' => 'resources/jquery/jquery.collapsibleTabs.js',
        ),
        'jquery.colorUtil' => array(
-               'scripts' => 'resources/jquery/jquery.colorUtil.js'
+               'scripts' => 'resources/jquery/jquery.colorUtil.js',
        ),
        'jquery.color' => array(
                'scripts' => 'resources/jquery/jquery.color.js',
-               'dependencies' => 'jquery.colorUtil'
+               'dependencies' => 'jquery.colorUtil',
        ),
        'jquery.cookie' => array(
-               'scripts' => 'resources/jquery/jquery.cookie.js'
+               'scripts' => 'resources/jquery/jquery.cookie.js',
        ),
        'jquery.delayedBind' => array(
-               'scripts' => 'resources/jquery/jquery.delayedBind.js'
+               'scripts' => 'resources/jquery/jquery.delayedBind.js',
        ),
        'jquery.expandableField' => array(
-               'scripts' => 'resources/jquery/jquery.expandableField.js'
+               'scripts' => 'resources/jquery/jquery.expandableField.js',
        ),
        'jquery.highlightText' => array(
-               'scripts' => 'resources/jquery/jquery.highlightText.js'
+               'scripts' => 'resources/jquery/jquery.highlightText.js',
        ),
        'jquery.hoverIntent' => array(
-               'scripts' => 'resources/jquery/jquery.hoverIntent.js'
+               'scripts' => 'resources/jquery/jquery.hoverIntent.js',
        ),
        'jquery.placeholder' => array(
-               'scripts' => 'resources/jquery/jquery.placeholder.js'
+               'scripts' => 'resources/jquery/jquery.placeholder.js',
        ),
        'jquery.localize' => array(
-               'scripts' => 'resources/jquery/jquery.localize.js'
+               'scripts' => 'resources/jquery/jquery.localize.js',
        ),
        'jquery.makeCollapsible' => array(
                'scripts' => 'resources/jquery/jquery.makeCollapsible.js',
@@ -85,10 +85,10 @@ return array(
                'styles' => 'resources/jquery/jquery.suggestions.css',
        ),
        'jquery.tabIndex' => array(
-               'scripts' => 'resources/jquery/jquery.tabIndex.js'
+               'scripts' => 'resources/jquery/jquery.tabIndex.js',
        ),
        'jquery.textSelection' => array(
-               'scripts' => 'resources/jquery/jquery.textSelection.js'
+               'scripts' => 'resources/jquery/jquery.textSelection.js',
        ),
        'jquery.tipsy' => array(
                'scripts' => 'resources/jquery.tipsy/jquery.tipsy.js',
@@ -130,7 +130,7 @@ return array(
        'jquery.ui.droppable' => array(
                'scripts' => 'resources/jquery.ui/jquery.ui.droppable.js',
                'dependencies' => array(
-                       'jquery.ui.core', 'jquery.ui.mouse', 'jquery.ui.widget', 'jquery.ui.draggable'
+                       'jquery.ui.core', 'jquery.ui.mouse', 'jquery.ui.widget', 'jquery.ui.draggable',
                ),
        ),
        'jquery.ui.resizable' => array(
@@ -237,7 +237,7 @@ return array(
                        'vi' => 'resources/jquery.ui/i18n/jquery.ui.datepicker-vi.js',
                        'zh-cn' => 'resources/jquery.ui/i18n/jquery.ui.datepicker-zh-CN.js',
                        'zh-hk' => 'resources/jquery.ui/i18n/jquery.ui.datepicker-zh-HK.js',
-                       'zh-tw' => 'resources/jquery.ui/i18n/jquery.ui.datepicker-zh-TW.js'
+                       'zh-tw' => 'resources/jquery.ui/i18n/jquery.ui.datepicker-zh-TW.js',
                ),
        ),
        'jquery.ui.dialog' => array(
@@ -430,8 +430,7 @@ return array(
        'mediawiki.legacy.ajax' => array(
                'scripts' => 'skins/common/ajax.js',
                'messages' => array(
-                       'watch', 'unwatch', 'watching', 'unwatching', 'tooltip-ca-watch',
-                       'tooltip-ca-unwatch'
+                       'watch', 'unwatch', 'watching', 'unwatching', 'tooltip-ca-watch', 'tooltip-ca-unwatch',
                ),
                'dependencies' => 'mediawiki.legacy.wikibits',
        ),

diff --git a/resources/mediawiki.util/mediawiki.util.js b/resources/mediawiki.util/mediawiki.util.js
index b11847c..34ad85b 100644 (file)
--- a/resources/mediawiki.util/mediawiki.util.js
+++ b/resources/mediawiki.util/mediawiki.util.js
@@ -1,19 +1,19 @@
-/*
+/**
  * Utilities
  */
 
-(function ($, mw) {
+(function($, mw) {
 
        mw.util = {
 
                /* Initialisation */
                'initialised' : false,
-               'init' : function () {
+               'init' : function() {
                        if ( this.initialised === false ) {
                                this.initialised = true;
 
                                // Any initialisation after the DOM is ready
-                               $(function () {
+                               $(function() {
 
                                        // Shortcut to client profile return
                                        var profile = $.client.profile();
@@ -374,12 +374,12 @@
                /**
                 * Add a little box at the top of the screen to inform the user of
                 * something, replacing any previous message.
+                * Calling with no arguments, with an empty string or null will hide the message
                 *
-                * @param message       mixed   The DOM-element or HTML-string to be put inside the message box]
-                *                                                      Calling with no arguments, with an empty string or null will hide the message
-                * @param className     string  Used in adding a class; should be different for each
-                * call to allow CSS/JS to hide different boxes. null = no class used.
-                * @return Boolean         True on success, false on failure
+                * @param message mixed The DOM-element or HTML-string to be put inside the message box.
+                * @param className     string Used in adding a class; should be different for each call
+                *                         to allow CSS/JS to hide different boxes. null = no class used.
+                * @return boolean True on success, false on failure
                 */
                'jsMessage' : function( message, className ) {
                
@@ -423,7 +423,7 @@
                 * according to HTML5 specification. Please note the specification
                 * does not validate a domain with one character.
                 *
-                * FIXME: should be moved to a JavaScript validation module.
+                * FIXME: should be moved to or replaced by a JavaScript validation module.
                 */
                'validateEmail' : function( mailtxt ) {
                        if( mailtxt === '' ) {
@@ -475,7 +475,7 @@
                                        // User part which is liberal :p
                                        '[' + rfc5322_atext + '\\.' + ']' + '+'
                                        +
-                                       // "at"
+                                       // 'at'
                                        '@'
                                        +
                                        // Domain first part