Remove mw.util.inArray
authorKrinkle <krinkle@users.mediawiki.org>
Thu, 21 Jul 2011 16:41:21 +0000 (16:41 +0000)
committerKrinkle <krinkle@users.mediawiki.org>
Thu, 21 Jul 2011 16:41:21 +0000 (16:41 +0000)
- redundant and makes it even more confusing than jQuery's "inArray" already was.
- Reverts r92261
- Fixed usage in /trunk/*
-- http://toolserver.org/~krinkle/wikimedia-svn-search/view.php?id=245&hash=4ef50cd2f7017954e562155ccceba878

While at it, also took take of the r92297 fixme. Item argument must be before array argument.

resources/mediawiki.page/mediawiki.page.ajaxCategories.js
resources/mediawiki/mediawiki.util.js

index da73692..7c7ef90 100644 (file)
@@ -44,7 +44,7 @@ mw.ajaxCategories = function( options ) {
 
        var _saveAllButton;
        var _cancelAllButton;
-       var _multiEdit = mw.util.inArray( mw.config.get( 'wgUserGroups' ), 'user' );
+       var _multiEdit = $.inArray( 'user', mw.config.get( 'wgUserGroups' ) ) !== -1;
 
        /**
         * Helper function for $.fn.suggestion
index 758f915..ebd63ef 100644 (file)
 
                /* Main body */
 
-               /**
-                * Wether a value is in an array. Using jQuery's inArray cross-browser utility,
-                * (for browsers without Array indexOf support).
-                * @param val {Mixed} Needle
-                * @param arr {Array} Haystack
-                * @return Boolean
-                */
-               'inArray' : function( val, arr ) {
-                       return $.inArray( val, arr ) !== -1;
-               },
-
                /**
                 * Encode the string like PHP's rawurlencode
                 *