Fix for #1169
[jquery.git] / src / selector / selector.js
index 3bd6b34..6ad8e5c 100644 (file)
@@ -27,8 +27,8 @@ jQuery.extend({
                        contains: "jQuery.fn.text.apply([a]).indexOf(m[3])>=0",
 
                        // Visibility
-                       visible: 'a.type!="hidden"&&jQuery.css(a,"display")!="none"&&jQuery.css(a,"visibility")!="hidden"',
-                       hidden: 'a.type=="hidden"||jQuery.css(a,"display")=="none"||jQuery.css(a,"visibility")=="hidden"',
+                       visible: '"hidden"!=a.type&&jQuery.css(a,"display")!="none"&&jQuery.css(a,"visibility")!="hidden"',
+                       hidden: '"hidden"==a.type||jQuery.css(a,"display")=="none"||jQuery.css(a,"visibility")=="hidden"',
 
                        // Form attributes
                        enabled: "!a.disabled",
@@ -37,15 +37,15 @@ jQuery.extend({
                        selected: "a.selected||jQuery.attr(a,'selected')",
 
                        // Form elements
-                       text: "a.type=='text'",
-                       radio: "a.type=='radio'",
-                       checkbox: "a.type=='checkbox'",
-                       file: "a.type=='file'",
-                       password: "a.type=='password'",
-                       submit: "a.type=='submit'",
-                       image: "a.type=='image'",
-                       reset: "a.type=='reset'",
-                       button: 'a.type=="button"||jQuery.nodeName(a,"button")',
+                       text: "'text'==a.type",
+                       radio: "'radio'==a.type",
+                       checkbox: "'checkbox'==a.type",
+                       file: "'file'==a.type",
+                       password: "'password'==a.type",
+                       submit: "'submit'==a.type",
+                       image: "'image'==a.type",
+                       reset: "'reset'==a.type",
+                       button: '"button"==a.type||jQuery.nodeName(a,"button")',
                        input: "/input|select|textarea|button/i.test(a.nodeName)"
                },
                ".": "jQuery.className.has(a,m[2])",
@@ -59,7 +59,7 @@ jQuery.extend({
                        _resort: function(m){
                                return ["", m[1], m[3], m[2], m[5]];
                        },
-                       _prefix: "z=a[m[3]];if(!z||/href|src/.test(m[3]))z=jQuery.attr(a,m[3]);"
+                       _prefix: "var z=a[m[3]];if(!z||/href|src/.test(m[3]))z=jQuery.attr(a,m[3]);"
                },
                "[": "jQuery.find(m[2],a).length"
        },
@@ -67,16 +67,17 @@ jQuery.extend({
        // The regular expressions that power the parsing engine
        parse: [
                // Match: [@value='test'], [@foo]
-               /^\[ *(@)([a-z0-9_-]*) *([!*$^=]*) *('?"?)(.*?)\4 *\]/i,
+               /^\[ *(@)([\w-]+) *([!*$^=]*) *('?"?)(.*?)\4 *\]/,
 
                // Match: [div], [div p]
                /^(\[)\s*(.*?(\[.*?\])?[^[]*?)\s*\]/,
 
                // Match: :contains('foo')
-               /^(:)([a-z0-9_-]*)\("?'?(.*?(\(.*?\))?[^(]*?)"?'?\)/i,
+               /^(:)([\w-]+)\("?'?(.*?(\(.*?\))?[^(]*?)"?'?\)/,
 
-               // Match: :even, :last-chlid
-               /^([:.#]*)([a-z0-9_*-]*)/i
+               // Match: :even, :last-chlid, #id, .class
+               new RegExp("^([:.#]*)(" + 
+                       ( jQuery.chars = "(?:[\\w\u0128-\uFFFF*_-]|\\\\.)" ) + "+)")
        ],
 
        token: [
@@ -134,7 +135,7 @@ jQuery.extend({
                }
 
                // Initialize the search
-               var ret = [context], done = [], last = null;
+               var ret = [context], done = [], last;
 
                // Continue while a selector expression exists, and while
                // we're no longer looping upon ourselves
@@ -142,22 +143,21 @@ jQuery.extend({
                        var r = [];
                        last = t;
 
-                       t = jQuery.trim(t).replace( /^\/\//i, "" );
+                       t = jQuery.trim(t).replace( /^\/\//, "" );
 
                        var foundToken = false;
 
                        // An attempt at speeding up child selectors that
                        // point to a specific element tag
-                       var re = /^[\/>]\s*([a-z0-9*-]+)/i;
+                       var re = new RegExp("^[/>]\\s*(" + jQuery.chars + "+)");
                        var m = re.exec(t);
 
                        if ( m ) {
                                // Perform our own iteration and filter
-                               jQuery.each( ret, function(){
-                                       for ( var c = this.firstChild; c; c = c.nextSibling )
-                                               if ( c.nodeType == 1 && ( jQuery.nodeName(c, m[1]) || m[1] == "*" ) )
+                               for ( var i = 0; ret[i]; i++ )
+                                       for ( var c = ret[i].firstChild; c; c = c.nextSibling )
+                                               if ( c.nodeType == 1 && ( m[1] == "*" || jQuery.nodeName(c, m[1]) ) )
                                                        r.push( c );
-                               });
 
                                ret = r;
                                t = t.replace( re, "" );
@@ -165,18 +165,17 @@ jQuery.extend({
                                foundToken = true;
                        } else {
                                // Look for pre-defined expression tokens
-                               for ( var i = 0; i < jQuery.token.length; i += 2 ) {
+                               for ( var i = 0, tl = jQuery.token.length; i < tl; i += 2 ) {
                                        // Attempt to match each, individual, token in
                                        // the specified order
-                                       var re = jQuery.token[i];
+                                       var re = jQuery.token[i], fn = jQuery.token[i+1];
                                        var m = re.exec(t);
 
                                        // If the token match was found
                                        if ( m ) {
                                                // Map it against the token's handler
-                                               r = ret = jQuery.map( ret, jQuery.isFunction( jQuery.token[i+1] ) ?
-                                                       jQuery.token[i+1] :
-                                                       function(a){ return eval(jQuery.token[i+1]); });
+                                               r = ret = jQuery.map( ret, jQuery.isFunction( fn ) ?
+                                                       fn : new Function( "a", "return " + fn ) );
 
                                                // And remove the token
                                                t = jQuery.trim( t.replace( re, "" ) );
@@ -192,7 +191,7 @@ jQuery.extend({
                                // Handle multiple expressions
                                if ( !t.indexOf(",") ) {
                                        // Clean the result set
-                                       if ( ret[0] == context ) ret.shift();
+                                       if ( context == ret[0] ) ret.shift();
 
                                        // Merge the result sets
                                        done = jQuery.merge( done, ret );
@@ -205,7 +204,7 @@ jQuery.extend({
 
                                } else {
                                        // Optomize for the case nodeName#idName
-                                       var re2 = /^([a-z0-9_-]+)(#)([a-z0-9\\*_-]*)/i;
+                                       var re2 = new RegExp("^(" + jQuery.chars + "+)(#)(" + jQuery.chars + "+)");
                                        var m = re2.exec(t);
                                        
                                        // Re-organize the results, so that they're consistent
@@ -215,35 +214,31 @@ jQuery.extend({
                                        } else {
                                                // Otherwise, do a traditional filter check for
                                                // ID, class, and element selectors
-                                               re2 = /^([#.]?)([a-z0-9\\*_-]*)/i;
+                                               re2 = new RegExp("^([#.]?)(" + jQuery.chars + "*)");
                                                m = re2.exec(t);
                                        }
 
-                                       var last = ret[ret.length-1];
+                                       m[2] = m[2].replace(/\\/g, "");
+
+                                       var elem = ret[ret.length-1];
 
                                        // Try to do a global search by ID, where we can
-                                       if ( m[1] == "#" && last && last.getElementById ) {
+                                       if ( m[1] == "#" && elem && elem.getElementById ) {
                                                // Optimization for HTML document case
-                                               var oid = last.getElementById(m[2]);
+                                               var oid = elem.getElementById(m[2]);
                                                
                                                // Do a quick check for the existence of the actual ID attribute
                                                // to avoid selecting by the name attribute in IE
-                                               if ( jQuery.browser.msie && oid && oid.id != m[2] )
-                                                       oid = jQuery('[@id="'+m[2]+'"]', last)[0];
+                                               // also check to insure id is a string to avoid selecting an element with the name of 'id' inside a form
+                                               if ( (jQuery.browser.msie||jQuery.browser.opera) && oid && typeof oid.id == "string" && oid.id != m[2] )
+                                                       oid = jQuery('[@id="'+m[2]+'"]', elem)[0];
 
                                                // Do a quick check for node name (where applicable) so
                                                // that div#foo searches will be really fast
                                                ret = r = oid && (!m[3] || jQuery.nodeName(oid, m[3])) ? [oid] : [];
-
                                        } else {
-                                               // Pre-compile a regular expression to handle class searches
-                                               if ( m[1] == "." )
-                                                       var rec = new RegExp("(^|\\s)" + m[2] + "(\\s|$)");
-
-                                               // We need to find all descendant elements, it is more
-                                               // efficient to use getAll() when we are already further down
-                                               // the tree - we try to recognize that here
-                                               for ( var i = 0, rl = ret.length; i < rl; i++ ) {
+                                               // We need to find all descendant elements
+                                               for ( var i = 0; ret[i]; i++ ) {
                                                        // Grab the tag name being searched for
                                                        var tag = m[1] != "" || m[0] == "" ? "*" : m[2];
 
@@ -256,23 +251,20 @@ jQuery.extend({
 
                                                // It's faster to filter by class and be done with it
                                                if ( m[1] == "." )
-                                                       r = jQuery.grep( r, function(e) {
-                                                               return rec.test(e.className);
-                                                       });
+                                                       r = jQuery.classFilter( r, m[2] );
 
                                                // Same with ID filtering
                                                if ( m[1] == "#" ) {
-                                                       // Remember, then wipe out, the result set
-                                                       var tmp = r;
-                                                       r = [];
-
-                                                       // Then try to find the element with the ID
-                                                       jQuery.each( tmp, function(){
-                                                               if ( this.getAttribute("id") == m[2] ) {
-                                                                       r = [ this ];
-                                                                       return false;
+                                                       var tmp = [];
+
+                                                       // Try to find the element with the ID
+                                                       for ( var i = 0; r[i]; i++ )
+                                                               if ( r[i].getAttribute("id") == m[2] ) {
+                                                                       tmp = [ r[i] ];
+                                                                       break;
                                                                }
-                                                       });
+
+                                                       r = tmp;
                                                }
 
                                                ret = r;
@@ -292,8 +284,14 @@ jQuery.extend({
                        }
                }
 
+               // An error occurred with the selector;
+               // just return an empty set instead
+               if ( t )
+                       ret = [];
+
                // Remove the root context
-               if ( ret && ret[0] == context ) ret.shift();
+               if ( ret && context == ret[0] )
+                       ret.shift();
 
                // And combine the results
                done = jQuery.merge( done, ret );
@@ -301,17 +299,28 @@ jQuery.extend({
                return done;
        },
 
+       classFilter: function(r,m,not){
+               m = " " + m + " ";
+               var tmp = [];
+               for ( var i = 0; r[i]; i++ ) {
+                       var pass = (" " + r[i].className + " ").indexOf( m ) >= 0;
+                       if ( !not && pass || not && !pass )
+                               tmp.push( r[i] );
+               }
+               return tmp;
+       },
+
        filter: function(t,r,not) {
+               var last;
+
                // Look for common filter expressions
-               while ( t && /^[a-z[({<*:.#]/i.test(t) ) {
+               while ( t  && t != last ) {
+                       last = t;
 
                        var p = jQuery.parse, m;
 
-                       jQuery.each( p, function(i,re){
-               
-                               // Look for, and replace, string-like sequences
-                               // and finally build a regexp out of it
-                               m = re.exec( t );
+                       for ( var i = 0; p[i]; i++ ) {
+                               m = p[i].exec( t );
 
                                if ( m ) {
                                        // Remove what we just matched
@@ -321,26 +330,26 @@ jQuery.extend({
                                        if ( jQuery.expr[ m[1] ]._resort )
                                                m = jQuery.expr[ m[1] ]._resort( m );
 
-                                       return false;
+                                       m[2] = m[2].replace(/\\/g, "");
+
+                                       break;
                                }
-                       });
+                       }
+
+                       if ( !m )
+                               break;
 
                        // :not() is a special case that can be optimized by
                        // keeping it out of the expression list
                        if ( m[1] == ":" && m[2] == "not" )
                                r = jQuery.filter(m[3], r, true).r;
 
-                       // Handle classes as a special case (this will help to
-                       // improve the speed, as the regexp will only be compiled once)
-                       else if ( m[1] == "." ) {
-
-                               var re = new RegExp("(^|\\s)" + m[2] + "(\\s|$)");
-                               r = jQuery.grep( r, function(e){
-                                       return re.test(e.className || "");
-                               }, not);
+                       // We can get a big speed boost by filtering by class here
+                       else if ( m[1] == "." )
+                               r = jQuery.classFilter(r, m[2], not);
 
                        // Otherwise, find the expression to execute
-                       } else {
+                       else {
                                var f = jQuery.expr[m[1]];
                                if ( typeof f != "string" )
                                        f = jQuery.expr[m[1]][m[2]];
@@ -397,8 +406,9 @@ jQuery.extend({
                for ( ; cur; cur = cur[dir] ) {
                        if ( cur.nodeType == 1 ) num++;
                        if ( num == result || result == "even" && num % 2 == 0 && num > 1 && cur == elem ||
-                               result == "odd" && num % 2 == 1 && cur == elem ) return cur;
+                               result == "odd" && num % 2 == 1 && cur == elem ) break;
                }
+               return cur;
        },
        
        /**