Update no-attr.js

sharkykh · sharkykh · commit fc878a6ecdbe · 2018-08-08T19:32:15.000+03:00
Account for `.attr('attribute', value)`
diff --git a/rules/no-attr.js b/rules/no-attr.js
@@ -9,9 +9,10 @@ module.exports = function(context) {
       if (node.callee.property.name !== 'attr') return
 
       if (utils.isjQuery(node)) {
+        const getOrSet = node.arguments.length === 2 ? 'set' : 'get'
         context.report({
           node: node,
-          message: 'Prefer getAttribute to $.attr'
+          message: `Prefer ${getOrSet}Attribute to $.attr`
         })
       }
     }

Original file line number	Diff line number	Diff line change
`@@ -9,9 +9,10 @@ module.exports = function(context) {`
`9`	`9`	`if (node.callee.property.name !== 'attr') return`
`10`	`10`
`11`	`11`	`if (utils.isjQuery(node)) {`
	`12`	`+ const getOrSet = node.arguments.length === 2 ? 'set' : 'get'`
`12`	`13`	`context.report({`
`13`	`14`	`node: node,`
`14`		`- message: 'Prefer getAttribute to $.attr'`
	`15`	+ message: `Prefer ${getOrSet}Attribute to $.attr`
`15`	`16`	`})`
`16`	`17`	`}`
`17`	`18`	`}`