Merge pull request jsx-eslint#1723 from gwenaellarmet/handle-ternary

[fix] handle ternary operation for jsx-no-bind

Author: ljharb

lib/rules/jsx-no-bind.js

@@ -81,6 +81,12 @@ module.exports = {
         node.callee.property.name === 'bind'
       ) {
         return 'bindCall';
+      } else if (
+        nodeType === 'ConditionalExpression'
+      ) {
+        return getNodeViolationType(node.test) ||
+               getNodeViolationType(node.consequent) ||
+               getNodeViolationType(node.alternate);
       } else if (
         !configuration.allowArrowFunctions &&
         nodeType === 'ArrowFunctionExpression'
@@ -154,7 +160,7 @@ module.exports = {
         }
       },
 
-      JSXAttribute: function(node) {
+      JSXAttribute: function (node) {
         const isRef = configuration.ignoreRefs && propName(node) === 'ref';
         if (isRef || !node.value || !node.value.expression) {
           return;

tests/lib/rules/jsx-no-bind.js

@@ -397,6 +397,47 @@ ruleTester.run('jsx-no-bind', rule, {
       ],
       parser: 'babel-eslint'
     },
+    {
+      code: `
+        const foo = {
+          render: ({onClick}) => (
+            <div onClick={(returningBoolean()) ? onClick.bind(this) : onClick.bind(this)}>Hello</div>
+          )
+        };
+      `,
+      errors: [{message: 'JSX props should not use .bind()'}]
+    },
+    {
+      code: `
+        const foo = {
+          render: ({onClick}) => (
+            <div onClick={(returningBoolean()) ? onClick.bind(this) : handleClick()}>Hello</div>
+          )
+        };
+      `,
+      errors: [{message: 'JSX props should not use .bind()'}]
+    },
+    {
+      code: `
+        const foo = {
+          render: ({onClick}) => (
+            <div onClick={(returningBoolean()) ? handleClick() : this.onClick.bind(this)}>Hello</div>
+          )
+        };
+      `,
+      errors: [{message: 'JSX props should not use .bind()'}]
+    },
+    {
+      code: `
+        const foo = {
+          render: ({onClick}) => (
+            <div onClick={returningBoolean.bind(this) ? handleClick() : onClick()}>Hello</div>
+          )
+        };
+      `,
+      errors: [{message: 'JSX props should not use .bind()'}]
+    },
+
 
     // Arrow functions
     {