Fix noMethodSetState docsUrl's

Fixes #2078
jsx-eslint · Dec 18, 2018 · 7da9e0d · 7da9e0d
1 parent 9358489
commit 7da9e0d
Showing 1 changed file with 14 additions and 1 deletion.
diff --git a/lib/util/makeNoMethodSetStateRule.js b/lib/util/makeNoMethodSetStateRule.js
@@ -10,14 +10,27 @@ const docsUrl = require('./docsUrl');
 // Rule Definition
 // ------------------------------------------------------------------------------
 
+function mapTitle(methodName) {
+  const map = {
+    componentDidMount: 'did-mount',
+    componentDidUpdate: 'did-update',
+    componentWillUpdate: 'will-update'
+  };
+  const title = map[methodName];
+  if (!title) {
+    throw Error(`No docsUrl for '${methodName}'`);
+  }
+  return `no-${title}-set-state`;
+}
+
 function makeNoMethodSetStateRule(methodName, shouldCheckUnsafeCb) {
   return {
     meta: {
       docs: {
         description: `Prevent usage of setState in ${methodName}`,
         category: 'Best Practices',
         recommended: false,
-        url: docsUrl(methodName)
+        url: docsUrl(mapTitle(methodName))
       },
 
       schema: [{