fix: jsx-no-target-blank support for conditionals

Author: akulsr0

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

@@ -65,7 +65,7 @@ function hasDynamicLink(node, linkAttribute) {
   }
 }
 
-function getStringFromValue(value) {
+function getStringFromValue(value, targetValue) {
   if (value) {
     if (value.type === 'Literal') {
       return value.value;
@@ -75,24 +75,32 @@ function getStringFromValue(value) {
         return value.expression.quasis[0].value.cooked;
       }
       const expr = value.expression;
-      return expr && (
-        expr.type === 'ConditionalExpression'
-          ? [expr.consequent.value, expr.alternate.value]
-          : expr.value
-      );
+      if (expr && (expr.type === 'ConditionalExpression')) {
+        const relValues = [expr.consequent.value, expr.alternate.value];
+        if (targetValue.type === 'JSXExpressionContainer' && targetValue.expression && targetValue.expression.type === 'ConditionalExpression') {
+          const targetTestCond = targetValue.expression.test.name;
+          const relTestCond = value.expression.test.name;
+          const targetBlankIndex = [targetValue.expression.consequent.value, targetValue.expression.alternate.value].indexOf('_blank');
+          if (targetTestCond === relTestCond) return relValues[targetBlankIndex];
+        }
+        return relValues;
+      }
+      return expr.value;
     }
   }
   return null;
 }
 
 function hasSecureRel(node, allowReferrer, warnOnSpreadAttributes, spreadAttributeIndex) {
   const relIndex = findLastIndex(node.attributes, (attr) => (attr.type === 'JSXAttribute' && attr.name.name === 'rel'));
+  const targetIndex = findLastIndex(node.attributes, (attr) => (attr.type === 'JSXAttribute' && attr.name.name === 'target'));
   if (relIndex === -1 || (warnOnSpreadAttributes && relIndex < spreadAttributeIndex)) {
     return false;
   }
 
   const relAttribute = node.attributes[relIndex];
-  const value = getStringFromValue(relAttribute.value);
+  const targetAttributeValue = node.attributes[targetIndex] && node.attributes[targetIndex].value;
+  const value = getStringFromValue(relAttribute.value, targetAttributeValue);
   return [].concat(value).every((item) => {
     const tags = typeof item === 'string' ? item.toLowerCase().split(' ') : false;
     const noreferrer = tags && tags.indexOf('noreferrer') >= 0;

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

@@ -155,6 +155,15 @@ ruleTester.run('jsx-no-target-blank', rule, {
       code: '<a href={href} target="_blank" rel={isExternal ? "noreferrer" : "noopener"} />',
       options: [{ allowReferrer: true }],
     },
+    {
+      code: '<a href={href} target={isExternal ? "_blank" : undefined} rel={isExternal ? "noreferrer" : undefined} />',
+    },
+    {
+      code: '<a href={href} target={isSelf ? "_self" : "_blank"} rel={isSelf ? undefined : "noreferrer"} />',
+    },
+    {
+      code: '<a href={href} target={isSelf ? "_self" : ""} rel={isSelf ? undefined : ""} />',
+    },
   ]),
   invalid: parsers.all([
     {