Change rule and docs

Author: burtek

docs/rules/jsx-no-script-url.md

@@ -23,6 +23,8 @@ Examples of **correct** code for this rule:
 <a href={"javascript:"}></a>
 ```
 
+This rule takes the `linkComponents` setting into account.
+
 ## Rule Options
 
 ```json
@@ -44,6 +46,10 @@ Examples of **correct** code for this rule:
 ```
 
 Allows you to indicate a specific list of properties used by a custom component to be checked.
+This will override anything passed to `linkComponents` setting.
+
+NOTE: This rule now takes into account the `linkComponents` setting, which should be used as the sole source of truth for link components.
+The rule still allows passing link components as rule option for backwards compatibility, but this option is deprecated.
 
 ### name
 

lib/rules/jsx-no-script-url.js

@@ -6,6 +6,7 @@
 'use strict';
 
 const docsUrl = require('../util/docsUrl');
+const linkComponentsUtil = require('../util/linkComponents');
 const report = require('../util/report');
 
 // ------------------------------------------------------------------------------
@@ -21,26 +22,21 @@ function hasJavaScriptProtocol(attr) {
     && isJavaScriptProtocol.test(attr.value.value);
 }
 
-function shouldVerifyElement(node, config) {
-  const name = node.name && node.name.name;
-  return name === 'a' || config.find((i) => i.name === name);
-}
-
 function shouldVerifyProp(node, config) {
   const name = node.name && node.name.name;
   const parentName = node.parent.name && node.parent.name.name;
 
-  if (parentName === 'a' && name === 'href') {
-    return true;
-  }
+  if (!name || !parentName || !config.has(parentName)) return false;
 
-  const el = config.find((i) => i.name === parentName);
-  if (!el) {
-    return false;
-  }
+  return name === config.get(parentName);
+}
 
-  const props = el.props || [];
-  return node.name && props.indexOf(name) !== -1;
+function parseLegacyOption(config, option) {
+  option.forEach((opt) => {
+    opt.props.forEach((prop) => {
+      config.set(opt.name, prop);
+    });
+  });
 }
 
 const messages = {
@@ -82,11 +78,14 @@ module.exports = {
   },
 
   create(context) {
-    const config = context.options[0] || [];
+    const linkComponents = linkComponentsUtil.getLinkComponents(context);
+    if (context.options[0]) {
+      parseLegacyOption(linkComponents, context.options[0]);
+    }
+  
     return {
       JSXAttribute(node) {
-        const parent = node.parent;
-        if (shouldVerifyElement(parent, config) && shouldVerifyProp(node, config) && hasJavaScriptProtocol(node)) {
+        if (shouldVerifyProp(node, linkComponents) && hasJavaScriptProtocol(node)) {
           report(context, messages.noScriptURL, 'noScriptURL', {
             node,
           });