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

--- Comment #12 from Roan Kattouw <roan.katt...@gmail.com> 2011-06-18 20:23:47 
UTC ---
(In reply to comment #9)
> Marking as enhancement unless / until this patch is shown to fix an existing
> bug, or explained in some way.
Neil, this looks legit to me. Apparently some config var is assigned to another
var somewhere and manipulated later, but because of the by-reference nature of
array assignments in JS, this ends up manipulating the config itself, which is
kind of evil and apparently leads to pollution. The proposed fix (there's no
patch but see comment #6) is to clone the array with something like var bar =
$.extend( {}, foo ); instead of var bar = foo;

-- 
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
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l

Reply via email to