This project is archived and is in readonly mode.

#162 ✓invalid
dimor

BUG - Useless "()" in Browser.Plugins.Flash throws IE6

Reported by dimor | June 25th, 2008 @ 12:42 PM | in 2.0 (closed)

Browser.Plugins.Flash = (function(){

var version = ($try(function(){

return navigator.plugins['Shockwave Flash'].description;

}, function(){

return new ActiveXObject('ShockwaveFlash.ShockwaveFlash').GetVariable('$version');

}) || '0 r0').match(/\d+/g);

return {version: parseInt(version[0] || 0 + '.' + version[1] || 0), build: parseInt(version[2] || 0)};

})();

those "()" in the end seems pretty much useless, and actually makes IE6 (6.0.2800.1106.xpsp2_gdr.040517-1325) throw an exception and die.

after removing it, everything goes back to normal, i think...

Comments and changes to this ticket

  • Tom Occhino

    Tom Occhino June 25th, 2008 @ 02:25 PM

    • State changed from “new” to “invalid”

    those "()" are not useless... if you look at the code, a function is being created, and then called. This is a very common pattern in JavaScript, and is seen throughout the MooTools source.

    If there is a problem with IE6 and the Flash detection function, then yes, removing the call to the function will certainly fix the error, while also completely disabling the Flash detection... If there is a problem with Native (not standalone, or hacked) IE6 and Flash version detection, please give us some details.

Create your profile

Help contribute to this project by taking a few moments to create your personal profile. Create your profile »

Shared Ticket Bins

People watching this ticket

Pages