From f8f5006acbb0b5f3301286f4362dd1e223180b9b Mon Sep 17 00:00:00 2001 From: Igor Minar Date: Mon, 28 Nov 2011 22:07:33 -0500 Subject: [PATCH] fix(jqLite): JQLiteHasClass should work even when minified closure compiler is smarter than we expected and drops the unused fn argument - this breaks the meta-programing logic of jqLite. The fix special cases JQLiteHasClass since its the only fn that needs this treatment in a way that is minification-proof. --- src/jqLite.js | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/jqLite.js b/src/jqLite.js index ec0d327ed37f..510735bb7bae 100644 --- a/src/jqLite.js +++ b/src/jqLite.js @@ -219,9 +219,7 @@ function JQLiteData(element, key, value) { } } -function JQLiteHasClass(element, selector, _) { - // the argument '_' is important, since it makes the function have 3 arguments, which - // is needed for delegate function to realize the this is a getter. +function JQLiteHasClass(element, selector) { return ((" " + element.className + " ").replace(/[\n\t]/g, " "). indexOf( " " + selector + " " ) > -1); } @@ -427,7 +425,9 @@ forEach({ JQLite.prototype[name] = function(arg1, arg2) { var i, key; - if ((fn.length == 2 ? arg1 : arg2) === undefined) { + // JQLiteHasClass has only two arguments, but is a getter-only fn, so we need to special case it + // in a way that is minification-proof. + if (((fn.length == 2 && fn !== JQLiteHasClass) ? arg1 : arg2) === undefined) { if (isObject(arg1)) { // we are a write, but the object properties are the key/values for(i=0; i < this.length; i++) { pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy