canonical
Version:
Canonical code style linter and formatter for JavaScript, SCSS and CSS.
38 lines (31 loc) • 1.6 kB
JavaScript
/**
* @fileoverview Rule to check if a call to filter should be a call to compact
*/
;
//------------------------------------------------------------------------------
// Rule Definition
//------------------------------------------------------------------------------
module.exports = function (context) {
var lodashUtil = require('../util/lodashUtil');
var astUtil = require('../util/astUtil');
function isDoubleNegationOfParam(exp, paramName) {
return astUtil.isNegationExpression(exp) && astUtil.isNegationExpression(exp.argument) && astUtil.isIdentifierOfParam(exp.argument.argument, paramName);
}
function isCallToBooleanCastOfParam(exp, paramName) {
return exp && exp.type === 'CallExpression' && exp.callee.name === 'Boolean' && astUtil.isIdentifierOfParam(exp.arguments[0], paramName);
}
function isBooleanCastingFunction(func) {
var returnValue = astUtil.getValueReturnedInFirstLine(func);
var paramName = astUtil.getFirstParamName(func);
return func && func.type === 'Identifier' && func.name === 'Boolean' ||
(astUtil.isIdentifierOfParam(returnValue, paramName) ||
isDoubleNegationOfParam(returnValue, paramName) || isCallToBooleanCastOfParam(returnValue, paramName));
}
return {
CallExpression: function (node) {
if (lodashUtil.isCallToMethod(node, 'filter') && isBooleanCastingFunction(lodashUtil.getLodashIteratee(node))) {
context.report(node, 'Prefer _.compact over filtering of Boolean casting');
}
}
};
};