Codemod arrow-functions

index.js

@@ -90,14 +90,10 @@ function configureAsError(rules) {
   return result;
 }
 
-const activeRules = filterRules(allRules, function(rule) {
-  return !rule.meta.deprecated;
-});
+const activeRules = filterRules(allRules, rule => !rule.meta.deprecated);
 const activeRulesConfig = configureAsError(activeRules);
 
-const deprecatedRules = filterRules(allRules, function(rule) {
-  return rule.meta.deprecated;
-});
+const deprecatedRules = filterRules(allRules, rule => rule.meta.deprecated);
 
 module.exports = {
   deprecatedRules: deprecatedRules,

lib/rules/boolean-prop-naming.js

@@ -40,7 +40,7 @@ module.exports = {
     }]
   },
 
-  create: Components.detect(function(context, components, utils) {
+  create: Components.detect((context, components, utils) => {
     const sourceCode = context.getSourceCode();
     const config = context.options[0] || {};
     const rule = config.rule ? new RegExp(config.rule) : null;
@@ -118,7 +118,7 @@ module.exports = {
       const component = components.get(node) || node;
       const invalidProps = component.invalidProps || [];
 
-      proptypes.forEach(function (prop) {
+      proptypes.forEach(prop => {
         const propKey = getPropKey(prop);
         const flowCheck = (
           prop.type === 'ObjectTypeProperty' &&
@@ -146,7 +146,7 @@ module.exports = {
      * @param {Object} component The component to process
      */
     function reportInvalidNaming(component) {
-      component.invalidProps.forEach(function (propNode) {
+      component.invalidProps.forEach(propNode => {
         const propName = getPropName(propNode);
         context.report({
           node: propNode,
@@ -192,7 +192,7 @@ module.exports = {
         }
 
         // Search for the proptypes declaration
-        node.properties.forEach(function(property) {
+        node.properties.forEach(property => {
           if (!isPropTypesDeclaration(property.key)) {
             return;
           }
@@ -213,7 +213,7 @@ module.exports = {
         }
 
         const list = components.list();
-        Object.keys(list).forEach(function (component) {
+        Object.keys(list).forEach(component => {
           // If this is a functional component that uses a global type, check it
           if (
             list[component].node.type === 'FunctionDeclaration' &&

lib/rules/default-props-match-prop-types.js

@@ -33,7 +33,7 @@ module.exports = {
     }]
   },
 
-  create: Components.detect(function(context, components, utils) {
+  create: Components.detect((context, components, utils) => {
     const configuration = context.options[0] || {};
     const allowRequiredDefaults = configuration.allowRequiredDefaults || false;
     const propWrapperFunctions = new Set(context.settings.propWrapperFunctions || []);
@@ -139,7 +139,7 @@ module.exports = {
 
     function resolveUnionTypeAnnotation(node) {
       // Go through all the union and resolve any generic types.
-      return node.types.map(function(annotation) {
+      return node.types.map(annotation => {
         if (annotation.type === 'GenericTypeAnnotation') {
           return resolveGenericTypeAnnotation(annotation);
         }
@@ -154,11 +154,9 @@ module.exports = {
      * @returns {Object[]}        Array of PropType object representations, to be consumed by `addPropTypesToComponent`.
      */
     function getPropTypesFromObjectExpression(objectExpression) {
-      const props = objectExpression.properties.filter(function(property) {
-        return property.type !== 'ExperimentalSpreadProperty';
-      });
+      const props = objectExpression.properties.filter(property => property.type !== 'ExperimentalSpreadProperty');
 
-      return props.map(function(property) {
+      return props.map(property => {
         return {
           name: property.key.name,
           isRequired: isRequiredPropType(property.value),
@@ -188,7 +186,7 @@ module.exports = {
 
         case 'UnionTypeAnnotation':
           const union = resolveUnionTypeAnnotation(node.typeAnnotation);
-          properties = union.reduce(function(acc, curr) {
+          properties = union.reduce((acc, curr) => {
             if (!curr) {
               return acc;
             }
@@ -206,11 +204,9 @@ module.exports = {
           break;
       }
 
-      const props = properties.filter(function(property) {
-        return property.type === 'ObjectTypeProperty';
-      });
+      const props = properties.filter(property => property.type === 'ObjectTypeProperty');
 
-      return props.map(function(property) {
+      return props.map(property => {
         // the `key` property is not present in ObjectTypeProperty nodes, so we need to get the key name manually.
         const tokens = context.getFirstTokens(property, 1);
         const name = tokens[0].value;
@@ -237,7 +233,7 @@ module.exports = {
         return 'unresolved';
       }
 
-      return objectExpression.properties.map(function(defaultProp) {
+      return objectExpression.properties.map(defaultProp => {
         return {
           name: defaultProp.key.name,
           node: defaultProp
@@ -348,7 +344,7 @@ module.exports = {
         return;
       }
 
-      defaultProps.forEach(function(defaultProp) {
+      defaultProps.forEach(defaultProp => {
         const prop = propFromName(propTypes, defaultProp.name);
 
         if (prop && (allowRequiredDefaults || !prop.isRequired)) {
@@ -567,7 +563,7 @@ module.exports = {
         }
 
         // Search for the proptypes declaration
-        node.properties.forEach(function(property) {
+        node.properties.forEach(property => {
           if (property.type === 'ExperimentalSpreadProperty') {
             return;
           }

lib/rules/display-name.js

@@ -30,7 +30,7 @@ module.exports = {
     }]
   },
 
-  create: Components.detect(function(context, components, utils) {
+  create: Components.detect((context, components, utils) => {
     const sourceCode = context.getSourceCode();
     const config = context.options[0] || {};
     const ignoreTranspilerName = config.ignoreTranspilerName || false;
@@ -209,7 +209,7 @@ module.exports = {
       ObjectExpression: function(node) {
         if (ignoreTranspilerName || !hasTranspilerName(node)) {
           // Search for the displayName declaration
-          node.properties.forEach(function(property) {
+          node.properties.forEach(property => {
             if (!property.key || !isDisplayNameDeclaration(property.key)) {
               return;
             }

lib/rules/forbid-elements.js

@@ -50,7 +50,7 @@ module.exports = {
 
     const indexedForbidConfigs = {};
 
-    forbidConfiguration.forEach(function(item) {
+    forbidConfiguration.forEach(item => {
       if (typeof item === 'string') {
         indexedForbidConfigs[item] = {element: item};
       } else {

lib/rules/forbid-foreign-prop-types.js

@@ -45,7 +45,7 @@ module.exports = {
       },
 
       ObjectPattern: function(node) {
-        const propTypesNode = node.properties.find(function(property) {
+        const propTypesNode = node.properties.find(property => {
           return property.type === 'Property' && property.key.name === 'propTypes';
         });
 

lib/rules/forbid-prop-types.js

@@ -74,7 +74,7 @@ module.exports = {
      * @returns {void}
      */
     function checkForbidden(declarations) {
-      declarations.forEach(function(declaration) {
+      declarations.forEach(declaration => {
         if (declaration.type !== 'Property') {
           return;
         }
@@ -154,7 +154,7 @@ module.exports = {
       },
 
       ObjectExpression: function(node) {
-        node.properties.forEach(function(property) {
+        node.properties.forEach(property => {
           if (!property.key) {
             return;
           }

lib/rules/jsx-equals-spacing.js

@@ -41,7 +41,7 @@ module.exports = {
 
     return {
       JSXOpeningElement: function(node) {
-        node.attributes.forEach(function(attrNode) {
+        node.attributes.forEach(attrNode => {
           if (!hasEqual(attrNode)) {
             return;
           }

lib/rules/jsx-filename-extension.js

@@ -64,7 +64,7 @@ module.exports = {
         }
 
         const allowedExtensions = getExtensionsConfig();
-        const isAllowedExtension = allowedExtensions.some(function (extension) {
+        const isAllowedExtension = allowedExtensions.some(extension => {
           return filename.slice(-extension.length) === extension;
         });
 

lib/rules/jsx-first-prop-new-line.js

@@ -36,7 +36,7 @@ module.exports = {
           (configuration === 'multiline-multiprop' && isMultilineJSX(node) && node.attributes.length > 1) ||
           (configuration === 'always')
         ) {
-          node.attributes.some(function(decl) {
+          node.attributes.some(decl => {
             if (decl.loc.start.line === node.loc.start.line) {
               context.report({
                 node: decl,

lib/rules/jsx-indent-props.js

@@ -157,7 +157,7 @@ module.exports = {
      * @param {Boolean} excludeCommas skip comma on start of line
      */
     function checkNodesIndent(nodes, indent, excludeCommas) {
-      nodes.forEach(function(node) {
+      nodes.forEach(node => {
         const nodeIndent = getNodeIndent(node, false, excludeCommas);
         if (
           node.type !== 'ArrayExpression' && node.type !== 'ObjectExpression' &&

lib/rules/jsx-key.js

@@ -33,9 +33,7 @@ module.exports = {
     }
 
     function getReturnStatement(body) {
-      return body.filter(function(item) {
-        return item.type === 'ReturnStatement';
-      })[0];
+      return body.filter(item => item.type === 'ReturnStatement')[0];
     }
 
     return {

lib/rules/jsx-max-props-per-line.js

@@ -51,7 +51,7 @@ module.exports = {
       const back = line[line.length - 1].end;
       for (let i = 0; i < line.length; i += max) {
         const nodes = line.slice(i, i + max);
-        output.push(nodes.reduce(function(prev, curr) {
+        output.push(nodes.reduce((prev, curr) => {
           if (prev === '') {
             return sourceCode.getText(curr);
           }
@@ -77,7 +77,7 @@ module.exports = {
         const firstProp = node.attributes[0];
         const linePartitionedProps = [[firstProp]];
 
-        node.attributes.reduce(function (last, decl) {
+        node.attributes.reduce((last, decl) => {
           if (last.loc.end.line === decl.loc.start.line) {
             linePartitionedProps[linePartitionedProps.length - 1].push(decl);
           } else {
@@ -86,7 +86,7 @@ module.exports = {
           return decl;
         });
 
-        linePartitionedProps.forEach(function (propsInLine) {
+        linePartitionedProps.forEach(propsInLine => {
           if (propsInLine.length > maximum) {
             const name = getPropName(propsInLine[maximum]);
             context.report({

lib/rules/jsx-no-bind.js

@@ -40,7 +40,7 @@ module.exports = {
     }]
   },
 
-  create: Components.detect(function(context, components, utils) {
+  create: Components.detect((context, components, utils) => {
     const configuration = context.options[0] || {};
 
     return {

lib/rules/jsx-no-duplicate-props.js

@@ -38,7 +38,7 @@ module.exports = {
       JSXOpeningElement: function (node) {
         const props = {};
 
-        node.attributes.forEach(function(decl) {
+        node.attributes.forEach(decl => {
           if (decl.type === 'JSXSpreadAttribute') {
             return;
           }

lib/rules/jsx-no-target-blank.js

@@ -15,7 +15,7 @@ function isTargetBlank(attr) {
 }
 
 function hasExternalLink(element) {
-  return element.attributes.some(function (attr) {
+  return element.attributes.some(attr => {
     return attr.name &&
       attr.name.name === 'href' &&
       attr.value.type === 'Literal' &&
@@ -24,7 +24,7 @@ function hasExternalLink(element) {
 }
 
 function hasSecureRel(element) {
-  return element.attributes.find(function (attr) {
+  return element.attributes.find(attr => {
     if (attr.type === 'JSXAttribute' && attr.name.name === 'rel') {
       const tags = attr.value && attr.value.type === 'Literal' && attr.value.value.toLowerCase().split(' ');
       return tags && (tags.indexOf('noopener') >= 0 && tags.indexOf('noreferrer') >= 0);

lib/rules/jsx-sort-props.js

@@ -52,7 +52,7 @@ function validateReservedFirstConfig(context, reservedFirst) {
     if (Array.isArray(reservedFirst)) {
       // Only allow a subset of reserved words in customized lists
       // eslint-disable-next-line consistent-return
-      const nonReservedWords = reservedFirst.filter(function(word) {
+      const nonReservedWords = reservedFirst.filter(word => {
         if (!isReservedPropName(word, RESERVED_PROPS_LIST)) {
           return true;
         }
@@ -135,12 +135,10 @@ module.exports = {
       JSXOpeningElement: function(node) {
         // `dangerouslySetInnerHTML` is only "reserved" on DOM components
         if (reservedFirst && !isDOMComponent(node)) {
-          reservedList = reservedList.filter(function(prop) {
-            return prop !== 'dangerouslySetInnerHTML';
-          });
+          reservedList = reservedList.filter(prop => prop !== 'dangerouslySetInnerHTML');
         }
 
-        node.attributes.reduce(function(memo, decl, idx, attrs) {
+        node.attributes.reduce((memo, decl, idx, attrs) => {
           if (decl.type === 'JSXSpreadAttribute') {
             return attrs[idx + 1];
           }

lib/rules/no-array-index-key.js

@@ -106,7 +106,7 @@ module.exports = {
 
       if (node.type === 'TemplateLiteral') {
         // key={`foo-${bar}`}
-        node.expressions.filter(isArrayIndex).forEach(function() {
+        node.expressions.filter(isArrayIndex).forEach(() => {
           context.report({node: node, message: ERROR_MESSAGE});
         });
 
@@ -117,7 +117,7 @@ module.exports = {
         // key={'foo' + bar}
         const identifiers = getIdentifiersFromBinaryExpression(node);
 
-        identifiers.filter(isArrayIndex).forEach(function() {
+        identifiers.filter(isArrayIndex).forEach(() => {
           context.report({node: node, message: ERROR_MESSAGE});
         });
 
@@ -144,7 +144,7 @@ module.exports = {
             return;
           }
 
-          props.properties.forEach(function (prop) {
+          props.properties.forEach(prop => {
             if (!prop.key || prop.key.name !== 'key') {
               // { ...foo }
               // { foo: bar }

lib/rules/no-children-prop.js

@@ -53,9 +53,7 @@ module.exports = {
         }
 
         const props = node.arguments[1].properties;
-        const childrenProp = props.find(function(prop) {
-          return prop.key && prop.key.name === 'children';
-        });
+        const childrenProp = props.find(prop => prop.key && prop.key.name === 'children');
 
         if (childrenProp) {
           context.report({