Permalink
Cannot retrieve contributors at this time
| /** | |
| * @fileoverview A rule to suggest using of the spread operator instead of `.apply()`. | |
| * @author Toru Nagashima | |
| */ | |
| "use strict"; | |
| const astUtils = require("./utils/ast-utils"); | |
| //------------------------------------------------------------------------------ | |
| // Helpers | |
| //------------------------------------------------------------------------------ | |
| /** | |
| * Checks whether or not a node is a `.apply()` for variadic. | |
| * @param {ASTNode} node A CallExpression node to check. | |
| * @returns {boolean} Whether or not the node is a `.apply()` for variadic. | |
| */ | |
| function isVariadicApplyCalling(node) { | |
| return ( | |
| node.callee.type === "MemberExpression" && | |
| node.callee.property.type === "Identifier" && | |
| node.callee.property.name === "apply" && | |
| node.callee.computed === false && | |
| node.arguments.length === 2 && | |
| node.arguments[1].type !== "ArrayExpression" && | |
| node.arguments[1].type !== "SpreadElement" | |
| ); | |
| } | |
| /** | |
| * Checks whether or not `thisArg` is not changed by `.apply()`. | |
| * @param {ASTNode|null} expectedThis The node that is the owner of the applied function. | |
| * @param {ASTNode} thisArg The node that is given to the first argument of the `.apply()`. | |
| * @param {RuleContext} context The ESLint rule context object. | |
| * @returns {boolean} Whether or not `thisArg` is not changed by `.apply()`. | |
| */ | |
| function isValidThisArg(expectedThis, thisArg, context) { | |
| if (!expectedThis) { | |
| return astUtils.isNullOrUndefined(thisArg); | |
| } | |
| return astUtils.equalTokens(expectedThis, thisArg, context); | |
| } | |
| //------------------------------------------------------------------------------ | |
| // Rule Definition | |
| //------------------------------------------------------------------------------ | |
| module.exports = { | |
| meta: { | |
| type: "suggestion", | |
| docs: { | |
| description: "require spread operators instead of `.apply()`", | |
| category: "ECMAScript 6", | |
| recommended: false, | |
| url: "https://eslint.org/docs/rules/prefer-spread" | |
| }, | |
| schema: [], | |
| fixable: null | |
| }, | |
| create(context) { | |
| const sourceCode = context.getSourceCode(); | |
| return { | |
| CallExpression(node) { | |
| if (!isVariadicApplyCalling(node)) { | |
| return; | |
| } | |
| const applied = node.callee.object; | |
| const expectedThis = (applied.type === "MemberExpression") ? applied.object : null; | |
| const thisArg = node.arguments[0]; | |
| if (isValidThisArg(expectedThis, thisArg, sourceCode)) { | |
| context.report({ | |
| node, | |
| message: "Use the spread operator instead of '.apply()'." | |
| }); | |
| } | |
| } | |
| }; | |
| } | |
| }; |