Skip to content

Commit 53d688e

Browse files
committed
AC-391: Static test to cover "deprecated" jQuery methods
- Fix styling issues
1 parent 88f0e6d commit 53d688e

File tree

1 file changed

+11
-8
lines changed

1 file changed

+11
-8
lines changed

eslint/rules/utils.js

Lines changed: 11 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@
66
function define(node) {
77
'use strict';
88
var defineStmt, args;
9+
910
defineStmt = node.body.find(function (stmt) {
1011
return (
1112
stmt.type === 'ExpressionStatement' &&
@@ -35,7 +36,10 @@ function define(node) {
3536
* @returns {null|*}
3637
*/
3738
function getJqueryName(defineObject) {
39+
'use strict';
40+
3841
var jQueryPathIndex;
42+
3943
if (!defineObject.modulePaths || !defineObject.moduleNames) {
4044
return null;
4145
}
@@ -60,8 +64,6 @@ function getProgramNode(node) {
6064
return getProgramNode(node.parent);
6165
}
6266

63-
64-
6567
/**
6668
* Traverses the node to identify its id
6769
*
@@ -101,16 +103,17 @@ function getExpressionId(node) {
101103

102104
function isjQuery(node) {
103105
'use strict';
104-
var parentNode = getProgramNode(node);
105-
var defineNode = define(parentNode);
106+
var parentNode, defineNode, jQueryId, id;
107+
108+
parentNode = getProgramNode(node);
109+
defineNode = define(parentNode);
106110
if (!defineNode) {
107111
return false;
108112
}
109-
var jQueryID = getJqueryName(defineNode);
110-
111-
var id = getExpressionId(node);
113+
jQueryId = getJqueryName(defineNode);
114+
id = getExpressionId(node);
112115

113-
return id && jQueryID && id.name === jQueryID.name;
116+
return id && jQueryId && id.name === jQueryId.name;
114117
}
115118

116119
module.exports = {

0 commit comments

Comments
 (0)