https://bugzilla.wikimedia.org/show_bug.cgi?id=24268

[email protected] changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[email protected]

--- Comment #1 from [email protected] 2010-07-05 16:54:41 UTC ---
The problem is this:

var moduleRequestSet = [];
...
for(var moduleName in moduleRequestSet) { ... }

This works fine as long as no-one touches Array.prototype, but Twinkle adds
three new functions, which results in mwEmbed trying to load a nonexistent
module called "uniq".

The solution in this case should be as simple as changing the declaration to
var moduleRequestSet = {};, but that still doesn't protect against someone
messing with Object.prototype, and it won't allow you to use the Array type's
functions, e.g. push.

It might be a better idea to use something like Prototype's Hash.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

_______________________________________________
Wikibugs-l mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l

Reply via email to