Clean up and code review

Author: alexzherdev

lib/rules/no-unused-prop-types.js

@@ -466,7 +466,7 @@ module.exports = {
           return;
         }
 
-        if (prop.node && !isPropUsed(component, prop, key)) {
+        if (prop.node && !isPropUsed(component, prop)) {
           context.report(
             prop.node,
             UNUSED_MESSAGE, {
@@ -523,8 +523,8 @@ module.exports = {
         return;
       }
 
-      Object.values(propTypes).forEach(propType => {
-        const node = propType.node;
+      Object.keys(propTypes).forEach(key => {
+        const node = propTypes[key].node;
 
         if (node && (
           node.type === 'ArrowFunctionExpression'

lib/rules/prop-types.js

@@ -187,7 +187,6 @@ module.exports = {
           // If it's a computed property, we can't make any further analysis, but is valid
           return key === '__COMPUTED_PROP__';
         }
-
         if (typeof propType === 'object' && !propType.type) {
           return true;
         }

lib/util/Components.js

@@ -695,7 +695,7 @@ function componentRule(rule, context) {
   // Update the provided rule instructions to add the component detection
   const ruleInstructions = rule(context, components, utils);
   const updatedRuleInstructions = util._extend({}, ruleInstructions);
-  const propTypesInstructions = propTypes.instructions(context, components, utils);
+  const propTypesInstructions = propTypes(context, components, utils);
   const allKeys = new Set(Object.keys(detectionInstructions).concat(Object.keys(propTypesInstructions)));
   allKeys.forEach(instruction => {
     updatedRuleInstructions[instruction] = function(node) {

lib/util/ast.js

@@ -94,7 +94,6 @@ function isNodeFirstInLine(context, node) {
   return startLine !== endLine;
 }
 
-
 module.exports = {
   findReturnStatement: findReturnStatement,
   getPropertyName: getPropertyName,