Bug Tracker

Changes between Initial Version and Version 1 of Ticket #155


Ignore:
Timestamp:
Nov 17, 2006, 4:48:29 PM (16 years ago)
Author:
john
Comment:

Legend:

Unmodified
Added
Removed
Modified

  • Ticket #155

    • Property Priority changed from major to minor

  • Ticket #155 – Description

    initial v1  
    88treats prefixes as part of the tag name. Using rev 234, lines 1228-1233
    99could be replaced with this:
     10{{{
     11for ( var i = 0; i < ret.length; i++ ) {
     12  // create an array of tagnames to search for
     13  var tags = {};
    1014
    11 for ( var i = 0; i < ret.length; i++ ) {
    12 // create an array of tagnames to search for
    13 var tags = {};
    14 // fix for IE leaving in node prefixes
    15 if (jQuery.browser.msie && ret[i].xml !== undefined && m[2] != '*')
    16 for (var regex = new RegExp('</?((?:[^:]+:)?' + m[2] + ')\b', 'g'), tag;
    17 (tag = regex.exec(ret[i].xml)) != null; tags[tag[1]] = true);
    18 else tags[m[2]] = true;
     15  // fix for IE leaving in node prefixes
     16  if (jQuery.browser.msie && ret[i].xml !== undefined && m[2] != '*')
     17    for (var regex = new RegExp('</?((?:[^:]+:)?' + m[2] + ')\b', 'g'), tag;
     18      (tag = regex.exec(ret[i].xml)) != null; tags[tag[1]] = true);
     19  else tags[m[2]] = true;
    1920
    20 for ( var tag in tags )
    21 r = jQuery.merge( r,
    22 tag == "*" ?
    23 jQuery.getAll(ret[i]) :
    24 ret[i].getElementsByTagName(tag)
    25 );
     21  for ( var tag in tags )
     22    r = jQuery.merge( r, tag == "*" ?
     23      jQuery.getAll(ret[i]) :
     24       ret[i].getElementsByTagName(tag));
    2625}
     26}}}
    2727
    2828If it detects that we're searching an XML document and IE is being used,
    … …  
    3636preserved in the filter function, specifically because of line 1014:
    3737
     38{{{
    3839"": "m[2]== '*'||a.nodeName.toUpperCase()==m[2].toUpperCase()",
     40}}}
    3941
    4042Which could be replaced with:
    4143
     44{{{
    4245"": "m[2]== '*'||a.nodeName.replace(/[^:]+:/,
    4346'').toUpperCase()==m[2].toUpperCase()";
     47}}}
    4448
    4549That'll make sure only the local name is being compared instead of the