Skip to content
This repository was archived by the owner on Apr 12, 2024. It is now read-only.

Commit a7e69f7

Browse files
committed
refactor(ngClass): move helper functions outside the closure
1 parent 2871a11 commit a7e69f7

File tree

1 file changed

+31
-30
lines changed

1 file changed

+31
-30
lines changed

src/ng/directive/ngClass.js

+31-30
Original file line numberDiff line numberDiff line change
@@ -97,41 +97,42 @@ function classDirective(name, selector) {
9797
}
9898
}
9999
};
100+
}];
100101

101-
function arrayDifference(tokens1, tokens2) {
102-
var values = [];
102+
// Helpers
103+
function arrayDifference(tokens1, tokens2) {
104+
var values = [];
103105

104-
outer:
105-
for (var i = 0; i < tokens1.length; i++) {
106-
var token = tokens1[i];
107-
for (var j = 0; j < tokens2.length; j++) {
108-
if (token === tokens2[j]) continue outer;
109-
}
110-
values.push(token);
106+
outer:
107+
for (var i = 0; i < tokens1.length; i++) {
108+
var token = tokens1[i];
109+
for (var j = 0; j < tokens2.length; j++) {
110+
if (token === tokens2[j]) continue outer;
111111
}
112-
return values;
112+
values.push(token);
113113
}
114-
115-
function arrayClasses(classVal) {
116-
var classes = [];
117-
if (isArray(classVal)) {
118-
forEach(classVal, function(v) {
119-
classes = classes.concat(arrayClasses(v));
120-
});
121-
return classes;
122-
} else if (isString(classVal)) {
123-
return classVal.split(' ');
124-
} else if (isObject(classVal)) {
125-
forEach(classVal, function(v, k) {
126-
if (v) {
127-
classes = classes.concat(k.split(' '));
128-
}
129-
});
130-
return classes;
131-
}
132-
return classVal;
114+
return values;
115+
}
116+
117+
function arrayClasses(classVal) {
118+
var classes = [];
119+
if (isArray(classVal)) {
120+
forEach(classVal, function(v) {
121+
classes = classes.concat(arrayClasses(v));
122+
});
123+
return classes;
124+
} else if (isString(classVal)) {
125+
return classVal.split(' ');
126+
} else if (isObject(classVal)) {
127+
forEach(classVal, function(v, k) {
128+
if (v) {
129+
classes = classes.concat(k.split(' '));
130+
}
131+
});
132+
return classes;
133133
}
134-
}];
134+
return classVal;
135+
}
135136
}
136137

137138
/**

0 commit comments

Comments
 (0)