Element.update() and loadScripts...

[Animal]

Turned out the regex still wasn't there. It was greedy in matching the ([\'\"]) pattern, and in the string <script src="foo" type="text/javascript"></script> if would extract the source as 'foo" type="text/javascript"'

Also, on the else side of the srcMatch, match[1] is always true for valid script tags - it's a string, but zero length, so I added a length check:

        YAHOO.util.Event.onAvailable(id, function(){
            var hd = document.getElementsByTagName("head")[0];
            var re = /(?:<script.*?>)((\n|\r|.)*?)(?:<\/script>)/img; 
            var srcRe = /\ssrc=[\'\"]([^\'\"]*)[\'\"]/i; // <------ this changed
            var match;
            while(match = re.exec(html)){
                var srcMatch = match[0].match(srcRe);
                if(srcMatch && srcMatch[1]){
                   var s0 = document.createElement("script");
                   s0.src = srcMatch[1];
                   hd.appendChild(s0);
                }else if(match[1] && (match[1].length != 0)){ //<----- this changed too
                   eval(match[1]);
                }                     
            }
            var el = document.getElementById(id);
            if(el){el.parentNode.removeChild(el);}
            if(typeof callback == 'function'){
                callback();
            }
        });

[jack.slocum]

Damn, I wish I would have found these before the release. That sucks.

[Animal]

Well, I guess I should have tested with the src and type attributes the other way round when I did my initial testing too!

[Animal]

In fact, the regex to extract the src should be

/\ssrc=([\'\"])(.*?)\1/i

The "?" after the .* makes it reluctant rather than greedy, so it stops at the first occurrence of capture group 1 (' or ").

Obviously then, the src is in match[2]

[jack.slocum]

I'm ahead of ya. I didn't use the back reference but using it makes sense.

[Animal]

In fact using the backreference method to delimit the src="" string, I'm sure you could get the script search for both src URLs and source code content back into one regex! Hours of fun for the geekly minded........ I'll have a play! :lol:

[young_matthewd]

ran into the same problem. wondering if the fix will make it's way into the rc4 or final 33 release? currently using rc3. thanks / matthew

[Animal]

I'm not seeing the problem in RC3

[young_matthewd]

hej,

saw that the regular expression for srcRe was replaced between rc2 and rc3 but the src property gets set to blank if the srcMatch[1] item is pulled. had to pull srcMatch[2] instead (regardless of the order of script tag attributes).

[jack.slocum]

It should be 2, the 1st match is the quote. I will be doing a bug fix release to correct this and other rc3 bugs this weekend.