eslint-plugin-no-jquery 2.4.0 → 2.7.0

package/src/index.js

@@ -2,12 +2,12 @@
 
 module.exports = {
 	rules: {
-		'variable-pattern': require( './rules/variable-pattern' ),
 		'no-ajax': require( './rules/no-ajax' ),
 		'no-ajax-events': require( './rules/no-ajax-events' ),
 		'no-and-self': require( './rules/no-and-self' ),
 		'no-animate': require( './rules/no-animate' ),
 		'no-animate-toggle': require( './rules/no-animate-toggle' ),
+		'no-append-html': require( './rules/no-append-html' ),
 		'no-attr': require( './rules/no-attr' ),
 		'no-bind': require( './rules/no-bind' ),
 		'no-box-model': require( './rules/no-box-model' ),
@@ -30,6 +30,7 @@ module.exports = {
 		'no-each-util': require( './rules/no-each-util' ),
 		'no-error': require( './rules/no-error' ),
 		'no-error-shorthand': require( './rules/no-error-shorthand' ),
+		'no-escape-selector': require( './rules/no-escape-selector' ),
 		'no-event-shorthand': require( './rules/no-event-shorthand' ),
 		'no-extend': require( './rules/no-extend' ),
 		'no-fade': require( './rules/no-fade' ),
@@ -39,19 +40,21 @@ module.exports = {
 		'no-find-util': require( './rules/no-find-util' ),
 		'no-fx-interval': require( './rules/no-fx-interval' ),
 		'no-global-eval': require( './rules/no-global-eval' ),
+		'no-global-selector': require( './rules/no-global-selector' ),
 		'no-grep': require( './rules/no-grep' ),
 		'no-has': require( './rules/no-has' ),
 		'no-hide': require( './rules/no-hide' ),
 		'no-hold-ready': require( './rules/no-hold-ready' ),
 		'no-html': require( './rules/no-html' ),
 		'no-in-array': require( './rules/no-in-array' ),
+		'no-is': require( './rules/no-is' ),
 		'no-is-array': require( './rules/no-is-array' ),
 		'no-is-empty-object': require( './rules/no-is-empty-object' ),
-		'no-is-plain-object': require( './rules/no-is-plain-object' ),
 		'no-is-function': require( './rules/no-is-function' ),
 		'no-is-numeric': require( './rules/no-is-numeric' ),
+		'no-is-plain-object': require( './rules/no-is-plain-object' ),
 		'no-is-window': require( './rules/no-is-window' ),
-		'no-is': require( './rules/no-is' ),
+		'no-jquery-constructor': require( './rules/no-jquery-constructor' ),
 		'no-live': require( './rules/no-live' ),
 		'no-load': require( './rules/no-load' ),
 		'no-load-shorthand': require( './rules/no-load-shorthand' ),
@@ -63,6 +66,8 @@ module.exports = {
 		'no-noop': require( './rules/no-noop' ),
 		'no-now': require( './rules/no-now' ),
 		'no-on-ready': require( './rules/no-on-ready' ),
+		'no-other-methods': require( './rules/no-other-methods' ),
+		'no-other-utils': require( './rules/no-other-utils' ),
 		'no-param': require( './rules/no-param' ),
 		'no-parent': require( './rules/no-parent' ),
 		'no-parents': require( './rules/no-parents' ),
@@ -76,7 +81,6 @@ module.exports = {
 		'no-ready-shorthand': require( './rules/no-ready-shorthand' ),
 		'no-selector-prop': require( './rules/no-selector-prop' ),
 		'no-serialize': require( './rules/no-serialize' ),
-		'no-global-selector': require( './rules/no-global-selector' ),
 		'no-show': require( './rules/no-show' ),
 		'no-size': require( './rules/no-size' ),
 		'no-sizzle': require( './rules/no-sizzle' ),
@@ -96,10 +100,16 @@ module.exports = {
 		'no-val': require( './rules/no-val' ),
 		'no-visibility': require( './rules/no-visibility' ),
 		'no-when': require( './rules/no-when' ),
-		'no-wrap': require( './rules/no-wrap' )
+		'no-wrap': require( './rules/no-wrap' ),
+		'variable-pattern': require( './rules/variable-pattern' )
 	},
 	configs: {
-		// Use this profile if you are using jQuery slim build
+		recommended: {
+			rules: {
+				'no-jquery/variable-pattern': 'error'
+			}
+		},
+		// Use this config if you are using jQuery slim build
 		slim: {
 			rules: {
 				// Effects
@@ -114,103 +124,214 @@ module.exports = {
 				'no-jquery/no-parse-xml': 'error'
 			}
 		},
-		// Use this profile if you want to avoid all known deprecated jQuery tools.
+		// Use this config if you want to avoid all known deprecated jQuery tools.
 		deprecated: {
+			extends: 'plugin:no-jquery/deprecated-3.6'
+		},
+		// Use this config if you're writing code targetting jQuery 3.6.x environments.
+		'deprecated-3.6': {
 			extends: 'plugin:no-jquery/deprecated-3.5'
 		},
-		// Use this profile if you're writing code targetting jQuery 3.5.x environments.
 		'deprecated-3.5': {
 			extends: 'plugin:no-jquery/deprecated-3.4',
 			rules: {
-				'no-jquery/no-event-shorthand': 'error',
-				'no-jquery/no-trim': 'error'
+				// Pass empty options here to override options set in deprecated-3.3
+				'no-jquery/no-event-shorthand': [ 'warn', {} ],
+				'no-jquery/no-trim': 'warn'
 			}
 		},
-		// Use this profile if you're writing code targetting jQuery 3.4.x environments.
 		'deprecated-3.4': {
 			extends: 'plugin:no-jquery/deprecated-3.3',
 			rules: {
-				'no-jquery/no-sizzle': [ 'error', { allowPositional: false, allowOther: true } ]
+				'no-jquery/no-sizzle': [ 'warn', { allowPositional: false, allowOther: true } ]
 			}
 		},
-		// Use this profile if you're writing code targetting jQuery 3.3.x environments.
 		'deprecated-3.3': {
 			extends: 'plugin:no-jquery/deprecated-3.2',
 			rules: {
-				'no-jquery/no-camel-case': 'error',
-				'no-jquery/no-event-shorthand': [ 'error', { allowAjaxEvents: true } ],
-				'no-jquery/no-is-function': 'error',
-				'no-jquery/no-is-numeric': 'error',
-				'no-jquery/no-is-window': 'error',
-				'no-jquery/no-now': 'error',
-				'no-jquery/no-proxy': 'error',
-				'no-jquery/no-type': 'error'
+				'no-jquery/no-camel-case': 'warn',
+				'no-jquery/no-event-shorthand': [ 'warn', { allowAjaxEvents: true } ],
+				'no-jquery/no-is-function': 'warn',
+				'no-jquery/no-is-numeric': 'warn',
+				'no-jquery/no-is-window': 'warn',
+				'no-jquery/no-now': 'warn',
+				'no-jquery/no-proxy': 'warn',
+				'no-jquery/no-type': 'warn'
 			}
 		},
-		// Use this profile if you're writing code targetting jQuery 3.3.x environments.
 		'deprecated-3.2': {
-			extends: 'plugin:no-jquery/deprecated-3.0',
+			extends: 'plugin:no-jquery/deprecated-3.1',
 			rules: {
-				'no-jquery/no-hold-ready': 'error',
-				'no-jquery/no-is-array': 'error',
-				'no-jquery/no-node-name': 'error'
+				'no-jquery/no-hold-ready': 'warn',
+				'no-jquery/no-is-array': 'warn',
+				'no-jquery/no-node-name': 'warn'
 			}
 		},
-		// Use this profile if you're writing code targetting jQuery 3.0.x environments.
+		'deprecated-3.1': {
+			extends: 'plugin:no-jquery/deprecated-3.0'
+		},
 		'deprecated-3.0': {
-			extends: 'plugin:no-jquery/deprecated-1.10',
+			extends: 'plugin:no-jquery/deprecated-2.2',
 			rules: {
-				'no-jquery/no-bind': 'error',
-				'no-jquery/no-delegate': 'error',
-				'no-jquery/no-fx-interval': 'error',
-				'no-jquery/no-parse-json': 'error',
-				'no-jquery/no-ready-shorthand': 'error',
-				'no-jquery/no-unique': 'error'
+				'no-jquery/no-bind': 'warn',
+				'no-jquery/no-delegate': 'warn',
+				'no-jquery/no-fx-interval': 'warn',
+				'no-jquery/no-parse-json': 'warn',
+				'no-jquery/no-ready-shorthand': 'warn',
+				'no-jquery/no-unique': 'warn'
 			}
 		},
-		// Use this profile if you're writing code targetting jQuery 1.10.x environments.
+		'deprecated-2.2': {
+			extends: 'plugin:no-jquery/deprecated-2.1'
+		},
+		'deprecated-2.1': {
+			extends: 'plugin:no-jquery/deprecated-2.0'
+		},
+		'deprecated-2.0': {
+			extends: 'plugin:no-jquery/deprecated-1.12'
+		},
+		'deprecated-1.12': {
+			extends: 'plugin:no-jquery/deprecated-1.11'
+		},
+		'deprecated-1.11': {
+			extends: 'plugin:no-jquery/deprecated-1.10'
+		},
 		'deprecated-1.10': {
 			extends: 'plugin:no-jquery/deprecated-1.9',
 			rules: {
-				'no-jquery/no-context-prop': 'error'
+				'no-jquery/no-context-prop': 'warn'
 			}
 		},
-		// Use this profile if you're writing code targetting jQuery 1.9.x environments.
 		'deprecated-1.9': {
 			extends: 'plugin:no-jquery/deprecated-1.8',
 			rules: {
-				'no-jquery/no-support': 'error'
+				'no-jquery/no-support': 'warn'
 			}
 		},
-		// Use this profile if you're writing code targetting jQuery 1.8.x environments.
 		'deprecated-1.8': {
 			extends: 'plugin:no-jquery/deprecated-1.7',
 			rules: {
-				'no-jquery/no-and-self': 'error',
+				'no-jquery/no-and-self': 'warn',
 				// FIXME: `deferred.pipe()`
-				'no-jquery/no-error-shorthand': 'error',
-				'no-jquery/no-load-shorthand': 'error',
-				'no-jquery/no-on-ready': 'error',
-				'no-jquery/no-size': 'error',
+				'no-jquery/no-error-shorthand': 'warn',
+				'no-jquery/no-load-shorthand': 'warn',
+				'no-jquery/no-on-ready': 'warn',
+				'no-jquery/no-size': 'warn',
 				// FIXME: `$(...).toggle(fn,fn) (excluding https://api.jquery.com/toggle/)
-				'no-jquery/no-unload-shorthand': 'error'
+				'no-jquery/no-unload-shorthand': 'warn'
 			}
 		},
-		// Use this profile if you're writing code targetting jQuery 1.7.x environments.
 		'deprecated-1.7': {
-			extends: 'plugin:no-jquery/deprecated-1.3',
+			extends: 'plugin:no-jquery/deprecated-1.6',
 			rules: {
 				// FIXME: `deferred.isRejected()/isResolved()`
-				'no-jquery/no-live': 'error',
-				'no-jquery/no-sub': 'error',
-				'no-jquery/no-selector-prop': 'error'
+				'no-jquery/no-live': 'warn',
+				'no-jquery/no-sub': 'warn',
+				'no-jquery/no-selector-prop': 'warn'
 			}
 		},
-		// Use this profile if you're writing code targetting jQuery 1.3.x environments.
+		'deprecated-1.6': {
+			extends: 'plugin:no-jquery/deprecated-1.5'
+		},
+		'deprecated-1.5': {
+			extends: 'plugin:no-jquery/deprecated-1.4'
+		},
+		'deprecated-1.4': {
+			extends: 'plugin:no-jquery/deprecated-1.3'
+		},
 		'deprecated-1.3': {
+			extends: 'plugin:no-jquery/deprecated-1.2',
+			rules: {
+				'no-jquery/no-box-model': 'warn',
+				'no-jquery/no-browser': 'warn'
+			}
+		},
+		'deprecated-1.2': {
+			extends: 'plugin:no-jquery/deprecated-1.1'
+		},
+		'deprecated-1.1': {
+			extends: 'plugin:no-jquery/deprecated-1.0'
+		},
+		'deprecated-1.0': {},
+		// Use this config to disallow all usage of jQuery
+		all: {
 			rules: {
-				'no-jquery/no-box-model': 'error',
-				'no-jquery/no-browser': 'error'
+				'no-jquery/no-other-methods': 'warn',
+				'no-jquery/no-other-utils': 'warn',
+				'no-jquery/no-jquery-constructor': 'warn',
+
+				// methods
+				'no-jquery/no-animate': 'warn',
+				'no-jquery/no-bind': 'warn',
+				'no-jquery/no-class': 'warn',
+				'no-jquery/no-closest': 'warn',
+				'no-jquery/no-delegate': 'warn',
+				'no-jquery/no-each-collection': 'warn',
+				'no-jquery/no-event-shorthand': 'warn',
+				'no-jquery/no-fade': 'warn',
+				'no-jquery/no-find-collection': 'warn',
+				'no-jquery/no-has': 'warn',
+				'no-jquery/no-html': 'warn',
+				'no-jquery/no-is': 'warn',
+				'no-jquery/no-live': 'warn',
+				'no-jquery/no-load': 'warn',
+				'no-jquery/no-map-collection': 'warn',
+				'no-jquery/no-parent': 'warn',
+				'no-jquery/no-parents': 'warn',
+				'no-jquery/no-ready-shorthand': 'warn',
+				'no-jquery/no-serialize': 'warn',
+				'no-jquery/no-size': 'warn',
+				'no-jquery/no-slide': 'warn',
+				'no-jquery/no-trigger': 'warn',
+				'no-jquery/no-val': 'warn',
+				'no-jquery/no-visibility': 'warn',
+				'no-jquery/no-wrap': 'warn',
+
+				// Utils
+				'no-jquery/no-ajax': 'warn',
+				'no-jquery/no-camel-case': 'warn',
+				'no-jquery/no-contains': 'warn',
+				'no-jquery/no-deferred': 'warn',
+				'no-jquery/no-each-util': 'warn',
+				'no-jquery/no-error': 'warn',
+				'no-jquery/no-escape-selector': 'warn',
+				'no-jquery/no-extend': 'warn',
+				'no-jquery/no-find-util': 'warn',
+				'no-jquery/no-global-eval': 'warn',
+				'no-jquery/no-grep': 'warn',
+				'no-jquery/no-hold-ready': 'warn',
+				'no-jquery/no-in-array': 'warn',
+				'no-jquery/no-is-array': 'warn',
+				'no-jquery/no-is-empty-object': 'warn',
+				'no-jquery/no-is-function': 'warn',
+				'no-jquery/no-is-numeric': 'warn',
+				'no-jquery/no-is-plain-object': 'warn',
+				'no-jquery/no-is-window': 'warn',
+				'no-jquery/no-map-util': 'warn',
+				'no-jquery/no-merge': 'warn',
+				'no-jquery/no-node-name': 'warn',
+				'no-jquery/no-noop': 'warn',
+				'no-jquery/no-now': 'warn',
+				'no-jquery/no-param': 'warn',
+				'no-jquery/no-parse-html': 'warn',
+				'no-jquery/no-parse-json': 'warn',
+				'no-jquery/no-parse-xml': 'warn',
+				'no-jquery/no-proxy': 'warn',
+				'no-jquery/no-trim': 'warn',
+				'no-jquery/no-type': 'warn',
+				'no-jquery/no-unique': 'warn',
+				'no-jquery/no-when': 'warn',
+
+				// Method+utils
+				'no-jquery/no-attr': 'warn',
+				'no-jquery/no-clone': 'warn',
+				'no-jquery/no-css': 'warn',
+				'no-jquery/no-data': 'warn',
+				'no-jquery/no-filter': 'warn',
+				'no-jquery/no-prop': 'warn',
+				'no-jquery/no-sub': 'warn',
+				'no-jquery/no-text': 'warn'
+
 			}
 		}
 	}

package/src/rules/no-ajax-events.js

@@ -3,17 +3,14 @@
 const utils = require( '../utils.js' );
 
 const disallowedEvents = [
-	'ajaxStart',
-	'ajaxSend',
-	'ajaxSuccess',
-	'ajaxError',
 	'ajaxComplete',
-	'ajaxStop'
+	'ajaxError',
+	'ajaxSend',
+	'ajaxStart',
+	'ajaxStop',
+	'ajaxSuccess'
 ];
 
-const MemberExpression = 'MemberExpression';
-const Literal = 'Literal';
-
 module.exports = {
 	meta: {
 		type: 'suggestion',
@@ -25,8 +22,8 @@ module.exports = {
 
 	create: function ( context ) {
 		return {
-			CallExpression: function ( node ) {
-				if ( node.callee.type !== MemberExpression ) {
+			'CallExpression:exit': function ( node ) {
+				if ( node.callee.type !== 'MemberExpression' ) {
 					return;
 				}
 				let usedMethod;
@@ -36,7 +33,7 @@ module.exports = {
 				) {
 					const arg = node.arguments[ 0 ];
 					if (
-						arg.type === Literal &&
+						arg.type === 'Literal' &&
 						disallowedEvents.includes( arg.value )
 					) {
 						usedMethod = arg.value;
@@ -48,7 +45,8 @@ module.exports = {
 				if ( usedMethod && utils.isjQuery( context, node ) ) {
 					context.report( {
 						node: node,
-						message: `Prefer local event to ${usedMethod}`
+						message: 'Prefer local event to {{method}}',
+						data: { method: usedMethod }
 					} );
 				}
 			}

package/src/rules/no-and-self.js

@@ -7,7 +7,7 @@ module.exports = utils.createCollectionMethodRule(
 	'Prefer `.addBack` to `.andSelf`',
 	{
 		fixable: 'code',
-		fix: function ( node, fixer ) {
+		fix: function ( node, context, fixer ) {
 			return fixer.replaceText( node.callee.property, 'addBack' );
 		}
 	}

package/src/rules/no-animate-toggle.js

@@ -18,10 +18,10 @@ module.exports = {
 		const forbidden = [ 'show', 'hide', 'toggle' ];
 
 		return {
-			CallExpression: function ( node ) {
+			'CallExpression:exit': function ( node ) {
 				if (
 					node.callee.type !== 'MemberExpression' ||
-					forbidden.indexOf( node.callee.property.name ) === -1 ||
+					!forbidden.includes( node.callee.property.name ) ||
 					node.arguments.length === 0
 				) {
 					return;
@@ -46,7 +46,8 @@ module.exports = {
 				if ( utils.isjQuery( context, node ) ) {
 					context.report( {
 						node: node,
-						message: 'Prefer CSS transitions to .' + node.callee.property.name
+						message: 'Prefer CSS transitions to .{{method}}',
+						data: { method: node.callee.property.name }
 					} );
 				}
 			}

package/src/rules/no-animate.js

@@ -25,7 +25,7 @@ module.exports = {
 
 	create: function ( context ) {
 		return {
-			CallExpression: function ( node ) {
+			'CallExpression:exit': function ( node ) {
 				if (
 					node.callee.type !== 'MemberExpression' ||
 					node.callee.property.name !== 'animate'

package/src/rules/no-append-html.js

@@ -0,0 +1,53 @@
+'use strict';
+
+const utils = require( '../utils.js' );
+const methods = [ 'append', 'prepend', 'before', 'after', 'replaceWith' ];
+
+function alljQueryOrEmpty( context, node ) {
+	if ( node.type === 'ConditionalExpression' ) {
+		return alljQueryOrEmpty( context, node.consequent ) &&
+			alljQueryOrEmpty( context, node.alternate );
+	} else if ( node.type === 'Literal' ) {
+		return node.value === null || (
+			typeof node.value === 'string' && node.value.match( /^\s*$/ )
+		);
+	} else if ( node.type === 'Identifier' && node.name === 'undefined' ) {
+		return true;
+	} else {
+		return utils.isjQuery( context, node );
+	}
+}
+
+module.exports = {
+	meta: {
+		type: 'suggestion',
+		docs: {
+			description: 'Disallows using ' + methods.map( utils.jQueryCollectionLink ).join( '/' ) +
+			' to inject HTML, in order to prevent possible XSS bugs.'
+		},
+		schema: []
+	},
+
+	create: function ( context ) {
+		return {
+			'CallExpression:exit': function ( node ) {
+				if ( !(
+					node.callee.type === 'MemberExpression' &&
+					methods.includes( node.callee.property.name )
+				) ) {
+					return;
+				}
+				if ( node.arguments.every( ( arg ) => alljQueryOrEmpty( context, arg ) ) ) {
+					return;
+				}
+
+				if ( utils.isjQuery( context, node.callee ) ) {
+					context.report( {
+						node: node,
+						message: 'Avoid injection of possibly unescaped HTML. Create DOM elements instead, or use .text.'
+					} );
+				}
+			}
+		};
+	}
+};

package/src/rules/no-class-state.js

@@ -17,7 +17,7 @@ module.exports = {
 
 	create: function ( context ) {
 		return {
-			CallExpression: function ( node ) {
+			'CallExpression:exit': function ( node ) {
 				if ( !(
 					node.callee.type === 'MemberExpression' && (
 						node.callee.property.name === 'hasClass' ||

package/src/rules/no-constructor-attributes.js

@@ -13,7 +13,7 @@ module.exports = {
 
 	create: function ( context ) {
 		return {
-			CallExpression: function ( node ) {
+			'CallExpression:exit': function ( node ) {
 				if ( node.callee.type === 'MemberExpression' ) {
 					if ( !(
 						node.callee.property.name === 'add' &&

package/src/rules/no-deferred.js

@@ -28,7 +28,7 @@ module.exports = {
 		}
 
 		return {
-			CallExpression: enforce,
+			'CallExpression:exit': enforce,
 			NewExpression: enforce
 		};
 	}

package/src/rules/no-error.js

@@ -4,5 +4,11 @@ const utils = require( '../utils.js' );
 
 module.exports = utils.createUtilMethodRule(
 	'error',
-	'Prefer `throw` to `$.error`'
+	'Prefer `throw` to `$.error`',
+	{
+		fixable: 'code',
+		fix: function ( node, context, fixer ) {
+			return fixer.replaceText( node.callee, 'throw new Error' );
+		}
+	}
 );

package/src/rules/no-escape-selector.js

@@ -0,0 +1,14 @@
+'use strict';
+
+const utils = require( '../utils.js' );
+
+module.exports = utils.createUtilMethodRule(
+	'escapeSelector',
+	'Prefer `CSS.escape` to `$.escapeSelector`',
+	{
+		fixable: 'code',
+		fix: function ( node, context, fixer ) {
+			return fixer.replaceText( node.callee, 'CSS.escape' );
+		}
+	}
+);

package/src/rules/no-event-shorthand.js

@@ -3,12 +3,12 @@
 const utils = require( '../utils.js' );
 
 const ajaxEvents = [
-	'ajaxStart',
-	'ajaxStop',
 	'ajaxComplete',
 	'ajaxError',
-	'ajaxSuccess',
-	'ajaxSend'
+	'ajaxSend',
+	'ajaxStart',
+	'ajaxStop',
+	'ajaxSuccess'
 ];
 
 const rule = utils.createCollectionMethodRule(
@@ -72,17 +72,17 @@ const parentCreate = rule.create;
 rule.create = function ( context ) {
 	const rules = parentCreate( context );
 	return {
-		CallExpression: function ( node ) {
+		'CallExpression:exit': function ( node ) {
 			if (
 				node.callee.type === 'MemberExpression' &&
 				context.options[ 0 ] && context.options[ 0 ].allowAjaxEvents
 			) {
 				const name = node.callee.property.name;
-				if ( ajaxEvents.indexOf( name ) !== -1 ) {
+				if ( ajaxEvents.includes( name ) ) {
 					return;
 				}
 			}
-			return rules.CallExpression( node );
+			return rules[ 'CallExpression:exit' ]( node );
 		}
 	};
 };

package/src/rules/no-extend.js

@@ -23,7 +23,7 @@ module.exports = {
 
 	create: function ( context ) {
 		return {
-			CallExpression: function ( node ) {
+			'CallExpression:exit': function ( node ) {
 				if ( node.callee.type !== 'MemberExpression' ) {
 					return;
 				}

package/src/rules/no-global-selector.js

@@ -30,7 +30,7 @@ module.exports = {
 
 	create: function ( context ) {
 		return {
-			CallExpression: function ( node ) {
+			'CallExpression:exit': function ( node ) {
 				if (
 					node.callee.type !== 'Identifier' ||
 					!utils.isjQueryConstructor( context, node.callee.name ) ||

package/src/rules/no-html.js

@@ -4,5 +4,6 @@ const utils = require( '../utils.js' );
 
 module.exports = utils.createCollectionMethodRule(
 	'html',
-	'Prefer `Element#innerHTML` to `.html`'
+	'Prefer `Element#innerHTML` to `.html`',
+	{ getAndSetOptions: true }
 );

package/src/rules/no-is-array.js

@@ -7,7 +7,7 @@ module.exports = utils.createUtilMethodRule(
 	'Prefer `Array.isArray` to `$.isArray`',
 	{
 		fixable: 'code',
-		fix: function ( node, fixer ) {
+		fix: function ( node, context, fixer ) {
 			return fixer.replaceText( node.callee, 'Array.isArray' );
 		}
 	}

package/src/rules/no-is-function.js

@@ -7,7 +7,7 @@ module.exports = utils.createUtilMethodRule(
 	'Prefer `typeof` to `$.isFunction`',
 	{
 		fixable: 'code',
-		fix: function ( node, fixer ) {
+		fix: function ( node, context, fixer ) {
 			const calleeRange = node.callee.range;
 			return [
 				fixer.replaceTextRange( [ calleeRange[ 0 ], calleeRange[ 1 ] + 1 ], 'typeof ' ),