eslint-plugin-jsx-a11y - npm Package Compare versions

Comparing version 1.5.2 to 1.5.3

CHANGELOG.md

@@ -0,1 +1,6 @@
+1.5.3 / 2016-06-16
+==================
+- [fix] Fix crash when ``<ELEMENT role />`` for `role-supports-aria-props`.
+
+
 1.5.2 / 2016-06-16
@@ -2,0 +7,0 @@ ==================

lib/rules/role-supports-aria-props.js

@@ -49,3 +49,3 @@ 'use strict';
       // This actually isn't true - should fix in future release.
-      if (!roleValue || _role2.default[roleValue.toUpperCase()] === undefined) {
+      if (typeof roleValue !== 'string' || _role2.default[roleValue.toUpperCase()] === undefined) {
         return;
@@ -52,0 +52,0 @@ }

package.json

 {
   "name": "eslint-plugin-jsx-a11y",
-  "version": "1.5.2",
+  "version": "1.5.3",
   "description": "A static analysis linter of jsx and their accessibility with screen readers.",
@@ -5,0 +5,0 @@ "keywords": [

src/rules/role-supports-aria-props.js

@@ -36,3 +36,3 @@ /**
     // This actually isn't true - should fix in future release.
-    if (!roleValue || ROLES[roleValue.toUpperCase()] === undefined) {
+    if (typeof roleValue !== 'string' || ROLES[roleValue.toUpperCase()] === undefined) {
       return;
@@ -39,0 +39,0 @@ }

tests/src/rules/role-supports-aria-props.js

@@ -73,2 +73,3 @@ /**
     { code: '<div id="main" />', parserOptions },
+    { code: '<div role />', parserOptions },
     { code: '<div role="presentation" {...props} />', parserOptions },
@@ -75,0 +76,0 @@ { code: '<Foo.Bar baz={true} />', parserOptions },

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

369461

increased by0.05%

Lines of code

12079

increased by0.01%

No dependency changes