From 0d23acbadc4d3b3e2ea0d521956fc3b4d29ab72f Mon Sep 17 00:00:00 2001 From: Krinkle Date: Mon, 10 Jan 2011 05:02:05 +0000 Subject: [PATCH] Adding comma's at end of arrays where this hasn't been done already for consistency --- resources/Resources.php | 41 +++++++++++----------- resources/mediawiki.util/mediawiki.util.js | 22 ++++++------ 2 files changed, 31 insertions(+), 32 deletions(-) diff --git a/resources/Resources.php b/resources/Resources.php index 576056238e..e157e48e4a 100644 --- 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 b11847c8ef..34ad85b2a1 100644 --- 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 -- 2.20.1