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

Commit 5df7e6f

Browse files
committed
style(jqLite): clean up the code
1 parent fff31d8 commit 5df7e6f

File tree

1 file changed

+12
-4
lines changed

1 file changed

+12
-4
lines changed

src/jqLite.js

+12-4
Original file line numberDiff line numberDiff line change
@@ -84,7 +84,7 @@ var jqCache = {},
8484
? function(element, type, fn) {element.removeEventListener(type, fn, false); }
8585
: function(element, type, fn) {element.detachEvent('on' + type, fn); });
8686

87-
function jqNextId() { return (jqId++); }
87+
function jqNextId() { return ++jqId; }
8888

8989

9090
var SPECIAL_CHARS_REGEXP = /([\:\-\_]+(.))/g;
@@ -187,7 +187,8 @@ function JQLiteDealoc(element){
187187

188188
function JQLiteRemoveData(element) {
189189
var cacheId = element[jqName],
190-
cache = jqCache[cacheId];
190+
cache = jqCache[cacheId];
191+
191192
if (cache) {
192193
if (cache.bind) {
193194
forEach(cache.bind, function(fn, type){
@@ -206,6 +207,7 @@ function JQLiteRemoveData(element) {
206207
function JQLiteData(element, key, value) {
207208
var cacheId = element[jqName],
208209
cache = jqCache[cacheId || -1];
210+
209211
if (isDefined(value)) {
210212
if (!cache) {
211213
element[jqName] = cacheId = jqNextId();
@@ -452,10 +454,16 @@ forEach({
452454
// in a way that survives minification.
453455
if (((fn.length == 2 && (fn !== JQLiteHasClass && fn !== JQLiteController)) ? arg1 : arg2) === undefined) {
454456
if (isObject(arg1)) {
457+
455458
// we are a write, but the object properties are the key/values
456459
for(i=0; i < this.length; i++) {
457-
for (key in arg1) {
458-
fn(this[i], key, arg1[key]);
460+
if (fn === JQLiteData) {
461+
// data() takes the whole object in jQuery
462+
fn(this[i], arg1);
463+
} else {
464+
for (key in arg1) {
465+
fn(this[i], key, arg1[key]);
466+
}
459467
}
460468
}
461469
// return self for chaining

0 commit comments

Comments
 (0)