Remove schema changes to linkComponents

Author: gbakernet

docs/rules/jsx-no-target-blank.md

@@ -14,13 +14,13 @@ This rule aims to prevent user generated links from creating security vulnerabil
 ## Rule Options
 ```json
 ...
-"react/jsx-no-target-blank": [<enabled>, { "enforceDynamicLinks": <enforce>, "components": [<string>|<object>] }]
+"react/jsx-no-target-blank": [<enabled>, { "enforceDynamicLinks": <enforce> }]
 ...
 ```
 
 * enabled: for enabling the rule. 0=off, 1=warn, 2=error. Defaults to 0.
 * enforce: optional string, 'always' or 'never'
-* components: optional array
+* Link components can be something other than an `<a>`, see [shared settings](https://github.com/yannickcr/eslint-plugin-react/blob/master/README.md#configuration) for `linkComponents` configuration)
 
 ### `enforceDynamicLinks`
 
@@ -55,19 +55,9 @@ When {"enforceDynamicLinks": "never"} is set, the following patterns are **not**
 var Hello = <a target='_blank' href={ dynamicLink }></a>
 ```
 
-### `components`
+### Link components
 
-An array specifying the names of additional components that should also be validated.
-Each array element can either be a string with the component name or object specifying the name and url attribute:
-
-When using an object like so:
-
-```js
-{
-  "name": "Link",
-  "linkAttribute": "to"
-}
-```
+Link components can be something other than an `<a>`, see [shared settings](https://github.com/yannickcr/eslint-plugin-react/blob/master/README.md#configuration) for `linkComponents` configuration)
 
 The following patterns are considered errors:
 
@@ -85,15 +75,6 @@ var Hello = <Link target='_blank' to="/absolute/path/in/the/host"></Link>
 var Hello = <Link />
 ```
 
-When you use a string such as `'Link'`, it's equivalent to:
-
-```js
-{
-  "name": "Link",
-  "linkAttribute": "href"
-}
-```
-
 ## When Not To Use It
 
 If you do not have any external links, you can disable this rule

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

@@ -56,27 +56,6 @@ module.exports = {
       properties: {
         enforceDynamicLinks: {
           enum: ['always', 'never']
-        },
-        components: {
-          type: 'array',
-          items: {
-            oneOf: [{
-              type: 'string'
-            }, {
-              type: 'object',
-              properties: {
-                name: {
-                  type: 'string',
-                  minLength: 1
-                },
-                linkAttribute: {
-                  type: 'string',
-                  minLength: 1
-                }
-              }
-            }]
-          },
-          uniqueItems: true
         }
       },
       additionalProperties: false
@@ -86,9 +65,8 @@ module.exports = {
   create: function(context) {
     const configuration = context.options[0] || {};
     const enforceDynamicLinks = configuration.enforceDynamicLinks || 'always';
-    const linkComponents = DEFAULTS.concat(context.settings.linkComponents || []);
 
-    const elements = new Map((linkComponents.concat(configuration.components || [])).map(value => {
+    const elements = new Map((DEFAULTS.concat(context.settings.linkComponents || [])).map(value => {
       const name = typeof value === 'string' ? value : value.name;
       const linkAttribute = typeof value === 'string' ? 'href' : value.linkAttribute;
       return [name, linkAttribute];

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

@@ -48,14 +48,6 @@ ruleTester.run('jsx-no-target-blank', rule, {
       code: '<a target="_blank" href={ dynamicLink }></a>',
       options: [{enforceDynamicLinks: 'never'}]
     },
-    {
-      code: '<Link target="_blank" href={ dynamicLink }></Link>',
-      options: [{enforceDynamicLinks: 'never', components: ['Link']}]
-    },
-    {
-      code: '<Link target="_blank" to={ dynamicLink }></Link>',
-      options: [{enforceDynamicLinks: 'never', components: [{name: 'Link', linkAttribute: 'to'}]}]
-    },
     {
       code: '<Link target="_blank" href={ dynamicLink }></Link>',
       options: [{enforceDynamicLinks: 'never'}],
@@ -102,22 +94,11 @@ ruleTester.run('jsx-no-target-blank', rule, {
     options: [{enforceDynamicLinks: 'always'}],
     errors: defaultErrors
   }, {
-    code: '<Link target="_blank" href={ dynamicLink }></Link>',
-    options: [{enforceDynamicLinks: 'always', components: ['Link']}],
-    errors: defaultErrors
-  },
-  {
-    code: '<Link target="_blank" to={ dynamicLink }></Link>',
-    options: [{enforceDynamicLinks: 'always', components: [{name: 'Link', linkAttribute: 'to'}]}],
-    errors: defaultErrors
-  },
-  {
     code: '<Link target="_blank" href={ dynamicLink }></Link>',
     options: [{enforceDynamicLinks: 'always'}],
     settings: {linkComponents: ['Link']},
     errors: defaultErrors
-  },
-  {
+  }, {
     code: '<Link target="_blank" to={ dynamicLink }></Link>',
     options: [{enforceDynamicLinks: 'always'}],
     settings: {linkComponents: {name: 'Link', linkAttribute: 'to'}},