eslint-plugin-react - npm Package Compare versions

Comparing version 7.20.1 to 7.20.2

lib/rules/forbid-prop-types.js

@@ -91,29 +91,31 @@ /**
     function checkProperties(declarations) {
-      declarations.forEach((declaration) => {
-        if (declaration.type !== 'Property') {
-          return;
-        }
-        let target;
-        let value = declaration.value;
-        if (
-          value.type === 'MemberExpression'
-          && value.property
-          && value.property.name
-          && value.property.name === 'isRequired'
-        ) {
-          value = value.object;
-        }
-        if (value.type === 'CallExpression') {
-          value.arguments.forEach((arg) => {
-            reportIfForbidden(arg.name, declaration, target);
-          });
-          value = value.callee;
-        }
-        if (value.property) {
-          target = value.property.name;
-        } else if (value.type === 'Identifier') {
-          target = value.name;
-        }
-        reportIfForbidden(target, declaration, target);
-      });
+      if (declarations) {
+        declarations.forEach((declaration) => {
+          if (declaration.type !== 'Property') {
+            return;
+          }
+          let target;
+          let value = declaration.value;
+          if (
+            value.type === 'MemberExpression'
+            && value.property
+            && value.property.name
+            && value.property.name === 'isRequired'
+          ) {
+            value = value.object;
+          }
+          if (value.type === 'CallExpression') {
+            value.arguments.forEach((arg) => {
+              reportIfForbidden(arg.name, declaration, target);
+            });
+            value = value.callee;
+          }
+          if (value.property) {
+            target = value.property.name;
+          } else if (value.type === 'Identifier') {
+            target = value.name;
+          }
+          reportIfForbidden(target, declaration, target);
+        });
+      }
     }
@@ -120,0 +122,0 @@

lib/rules/jsx-no-useless-fragment.js

@@ -175,4 +175,5 @@ /**
         const closer = node.type === 'JSXFragment' ? node.closingFragment : node.closingElement;
-        const childrenText = context.getSourceCode().getText().slice(opener.range[1], closer.range[0]);
 
+        const childrenText = opener.selfClosing ? '' : context.getSourceCode().getText().slice(opener.range[1], closer.range[0]);
+
         return fixer.replaceText(node, trimLikeReact(childrenText));
@@ -179,0 +180,0 @@ };

lib/rules/jsx-pascal-case.js

@@ -31,13 +31,13 @@ /**
   }
-  let atLeastOneLowerCase = false;
-  for (let i = 1; i < name.length; i += 1) {
-    const char = name.charAt(i);
-    if (!(char.toLowerCase() !== char.toUpperCase() || testDigit(char))) {
-      return false;
-    }
-    if (!atLeastOneLowerCase) {
-      atLeastOneLowerCase = testLowerCase(char);
-    }
+  const anyNonAlphaNumeric = Array.prototype.some.call(
+    name.slice(1),
+    (char) => char.toLowerCase() === char.toUpperCase() && !testDigit(char)
+  );
+  if (anyNonAlphaNumeric) {
+    return false;
   }
-  return atLeastOneLowerCase;
+  return Array.prototype.some.call(
+    name.slice(1),
+    (char) => testLowerCase(char) || testDigit(char)
+  );
 }
@@ -44,0 +44,0 @@

lib/util/usedPropTypes.js

@@ -294,2 +294,3 @@ /**
     switch (node.type) {
+      case 'OptionalMemberExpression':
       case 'MemberExpression':
@@ -525,3 +526,3 @@ name = getPropertyName(node);
 
-    MemberExpression(node) {
+    'MemberExpression, OptionalMemberExpression'(node) {
       if (isPropTypesUsageByMemberExpression(node, context, utils, checkAsyncSafeLifeCycles)) {
@@ -528,0 +529,0 @@ markPropTypesAsUsed(node);

package.json

 {
   "name": "eslint-plugin-react",
-  "version": "7.20.1",
+  "version": "7.20.2",
   "author": "Yannick Croissant <[e-mail]>",
@@ -5,0 +5,0 @@ "description": "React specific linting rules for ESLint",

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

706339

increased by0.13%

Lines of code

15202

increased by0.02%

No dependency changes