From 46b956d679148746293ef5913e08df7481bc5901 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Matija=20Marohni=C4=87?= <matija.marohnic@gmail.com>
Date: Thu, 7 Jan 2016 13:45:39 +0100
Subject: [PATCH] Add allowRefs to the jsx-no-bind rule

Fixes #330.
Refs #357.
---
 docs/rules/jsx-no-bind.md      | 35 ++++++++++++++++++++++++++++++++++
 lib/rules/jsx-no-bind.js       |  7 ++++++-
 tests/lib/rules/jsx-no-bind.js | 22 +++++++++++++++++++++
 3 files changed, 63 insertions(+), 1 deletion(-)

diff --git a/docs/rules/jsx-no-bind.md b/docs/rules/jsx-no-bind.md
index 9218335aba..cc2ca5eee7 100644
--- a/docs/rules/jsx-no-bind.md
+++ b/docs/rules/jsx-no-bind.md
@@ -18,6 +18,41 @@ The following patterns are not considered warnings:
 <div onClick={this._handleClick}></div>
 ```
 
+## Rule Options
+
+```js
+"jsx-no-bind": [<enabled>, {
+  "ignoreRefs": <boolean> || false,
+  "allowArrowFunctions": <boolean> || false,
+  "allowBind": <boolean> || false
+}]
+```
+
+### `ignoreRefs`
+
+When `true` the following are not considered warnings:
+
+```jsx
+<div ref={c => this._div = c} />
+<div ref={this._refCallback.bind(this)} />
+```
+
+### `allowArrowFunctions`
+
+When `true` the following is not considered a warning:
+
+```jsx
+<div onClick={() => alert("1337")} />
+```
+
+### `allowBind`
+
+When `true` the following is not considered a warning:
+
+```jsx
+<div onClick={this._handleClick.bind(this)} />
+```
+
 ## Protips
 
 ### Lists of Items
diff --git a/lib/rules/jsx-no-bind.js b/lib/rules/jsx-no-bind.js
index e366a005ce..349741c80e 100644
--- a/lib/rules/jsx-no-bind.js
+++ b/lib/rules/jsx-no-bind.js
@@ -14,7 +14,8 @@ module.exports = function(context) {
 
   return {
     JSXAttribute: function(node) {
-      if (!node.value || !node.value.expression) {
+      var isRef = configuration.ignoreRefs && node.name.name === 'ref';
+      if (isRef || !node.value || !node.value.expression) {
         return;
       }
       var valueNode = node.value.expression;
@@ -45,6 +46,10 @@ module.exports.schema = [{
     allowBind: {
       default: false,
       type: 'boolean'
+    },
+    ignoreRefs: {
+      default: false,
+      type: 'boolean'
     }
   },
   additionalProperties: false
diff --git a/tests/lib/rules/jsx-no-bind.js b/tests/lib/rules/jsx-no-bind.js
index 89e23ae9b1..e751ab32c1 100644
--- a/tests/lib/rules/jsx-no-bind.js
+++ b/tests/lib/rules/jsx-no-bind.js
@@ -34,6 +34,18 @@ ruleTester.run('jsx-no-bind', rule, {
       parser: 'babel-eslint'
     },
 
+    // bind() and arrow functions in refs explicitly ignored
+    {
+      code: '<div ref={c => this._input = c}></div>',
+      options: [{ignoreRefs: true}],
+      parser: 'babel-eslint'
+    },
+    {
+      code: '<div ref={this._refCallback.bind(this)}></div>',
+      options: [{ignoreRefs: true}],
+      parser: 'babel-eslint'
+    },
+
     // bind() explicitly allowed
     {
       code: '<div onClick={this._handleClick.bind(this)}></div>',
@@ -66,6 +78,11 @@ ruleTester.run('jsx-no-bind', rule, {
       errors: [{message: 'JSX props should not use .bind()'}],
       parser: 'babel-eslint'
     },
+    {
+      code: '<div ref={this._refCallback.bind(this)}></div>',
+      errors: [{message: 'JSX props should not use .bind()'}],
+      parser: 'babel-eslint'
+    },
 
     // Arrow functions
     {
@@ -82,6 +99,11 @@ ruleTester.run('jsx-no-bind', rule, {
       code: '<div onClick={param => { first(); second(); }}></div>',
       errors: [{message: 'JSX props should not use arrow functions'}],
       parser: 'babel-eslint'
+    },
+    {
+      code: '<div ref={c => this._input = c}></div>',
+      errors: [{message: 'JSX props should not use arrow functions'}],
+      parser: 'babel-eslint'
     }
   ]
 });