Fix code as per PR comments

Author: burtek

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

@@ -23,7 +23,7 @@ Examples of **correct** code for this rule:
 <a href={"javascript:"}></a>
 ```
 
-This rule takes into account `linkComponents` setting.
+This rule takes the `linkComponents` setting into account.
 
 ## Rule Options
 
@@ -48,9 +48,8 @@ This rule takes into account `linkComponents` setting.
 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 `linkComponents` setting and it should be used as primary source of link components.
-The rule still allows passing link components as rule option, but it is meant only as backwards-compatibility feature.
-New setups should only use `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

@@ -31,14 +31,12 @@ function shouldVerifyProp(node, config) {
   return name === config.get(parentName)
 }
 
-function parseLegacyOption(option) {
-  const config = linkComponentsUtil.getLinkComponents({}) // get defaults
-  option.forEach(function(opt) {
-    opt.props.forEach(function(prop) { // FIXME: only last prop will work at the moment
-      config.set(opt.name, prop)
-    })
+function parseLegacyOption(config, option) {
+  option.forEach((opt) => {
+    opt.props.forEach((prop) => {
+      config.set(opt.name, prop);
+    });
   })
-  return config
 }
 
 const messages = {
@@ -75,13 +73,16 @@ module.exports = {
         },
         required: ['name', 'props'],
         additionalProperties: false,
-        deprecated: true, // ?
+        deprecated: true,
       },
     }],
   },
 
   create(context) {
-    const linkComponents = context.options[0] ? parseLegacyOption(context.options[0]) : linkComponentsUtil.getLinkComponents(context);
+    const linkComponents = linkComponentsUtil.getLinkComponents(context);
+    if (context.options[0]) {
+      parseLegacyOption(linkComponents, context.options[0]);
+    }
 
     return {
       JSXAttribute(node) {