Remove unneeded "eslint-disable no-restricted-properties" from several files
authorBartosz Dziewoński <matma.rex@gmail.com>
Fri, 23 Nov 2018 19:39:13 +0000 (20:39 +0100)
committerBartosz Dziewoński <matma.rex@gmail.com>
Fri, 23 Nov 2018 19:47:05 +0000 (20:47 +0100)
This was added in 160fc196fe43300843cc6cb68ad912051b7fb1d9 to suppress
warnings about usage of jQuery utility methods.

In eslint-config-wikimedia v0.9.0, the jQuery utility methods got its
own warning code, and change 3e4e0a5774cb7531c8be7516e57550477489753d
updated all files that use them to suppress each individual warning,
but did not remove this now-unnecessary global suppression.

Currently (since 6ad73362f376a0399bd1568fe7bbf40425737311), these
comments are instead incorrectly suppressing warnings about usage of
JavaScript methods introduced in ES6 and later.

Change-Id: Ie70aeb325946c8e6b62333e1ca1515cd0f5f326b

12 files changed:
resources/src/mediawiki.jqueryMsg/mediawiki.jqueryMsg.js
resources/src/mediawiki.legacy/protect.js
resources/src/mediawiki.rcfilters/dm/mw.rcfilters.dm.FilterGroup.js
resources/src/mediawiki.rcfilters/dm/mw.rcfilters.dm.FiltersViewModel.js
resources/src/mediawiki.rcfilters/dm/mw.rcfilters.dm.SavedQueriesModel.js
resources/src/mediawiki.rcfilters/mw.rcfilters.Controller.js
resources/src/mediawiki.searchSuggest/searchSuggest.js
resources/src/mediawiki.special.apisandbox/apisandbox.js
resources/src/mediawiki.widgets.datetime/DateTimeFormatter.js
resources/src/mediawiki.widgets.datetime/DiscordianDateTimeFormatter.js
resources/src/mediawiki.widgets.datetime/ProlepticGregorianDateTimeFormatter.js
resources/src/mediawiki.widgets/mw.widgets.NamespaceInputWidget.js

index 4044f95..ebb469f 100644 (file)
@@ -1,4 +1,3 @@
-/* eslint-disable no-restricted-properties */
 /*!
 * Experimental advanced wikitext parser-emitter.
 * See: https://www.mediawiki.org/wiki/Extension:UploadWizard/MessageParser for docs
index 3258609..2da5122 100644 (file)
@@ -1,4 +1,3 @@
-/* eslint-disable no-restricted-properties */
 ( function () {
        var ProtectionForm,
                reasonCodePointLimit = mw.config.get( 'wgCommentCodePointLimit' ),
index 20e37a9..9264a39 100644 (file)
@@ -1,4 +1,3 @@
-/* eslint-disable no-restricted-properties */
 ( function () {
        /**
         * View model for a filter group
index 5ba0b03..36575f7 100644 (file)
@@ -1,4 +1,3 @@
-/* eslint-disable no-restricted-properties */
 ( function () {
        /**
         * View model for the filters selection and display
index e84e396..7565a0d 100644 (file)
@@ -1,4 +1,3 @@
-/* eslint-disable no-restricted-properties */
 ( function () {
 
        var byteLength = require( 'mediawiki.String' ).byteLength;
index 554aadb..0069cf9 100644 (file)
@@ -1,4 +1,3 @@
-/* eslint-disable no-restricted-properties */
 /*!
  * Add search suggestions to the search form.
  */
index b38fa87..9b00ea2 100644 (file)
@@ -1,4 +1,3 @@
-/* eslint-disable no-restricted-properties */
 ( function () {
        'use strict';
        var ApiSandbox, Util, WidgetMethods, Validators,
                        if ( done ) {
                                container = Util.apiBool( p.info.deprecated ) ? that.deprecatedItemsFieldset : that.itemsFieldset;
                                // FIXME: ES6-ism
-                               // eslint-disable-next-line jquery/no-each-util
+                               // eslint-disable-next-line no-restricted-properties
                                index = container.getItems().findIndex( function ( el ) {
                                        return el.apiParamIndex !== undefined && el.apiParamIndex > p.info.index;
                                } );
index 87e536d..e8c1b9b 100644 (file)
@@ -1,4 +1,3 @@
-/* eslint-disable no-restricted-properties */
 ( function () {
 
        /**
                        }
                }
 
-               // eslint-disable-next-line jquery/no-each-util
+               // eslint-disable-next-line no-restricted-properties
                if ( v.normalize ) {
-                       // eslint-disable-next-line jquery/no-each-util
+                       // eslint-disable-next-line no-restricted-properties
                        v = v.normalize();
                }
                re = new RegExp( '^\\s*' + v.replace( /([\\{}()|.?*+\-^$\[\]])/g, '\\$1' ), 'i' ); // eslint-disable-line no-useless-escape