User "Krinkle" posted a comment on MediaWiki.r92113. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/92113#c19605 Commit summary:
Unit tests for jquery.localize (Follow-up r92069) Also fixing a bug in jquery.localize. Previously it did mw.msg(key, [p1, p2, ...]); (passing an array as second argument), however mw.msg doesn't take an array as second argument, instead the second argument is the first in a list of optional variadic arguments. (like mw.msg(key, p1, p2, ..); In cases were only 1 variable is passed, this didn't brake the test as [p1].toString() === p1. Fixing by using .apply instead and creating an array of arguments, starting with the key (unshifted) and if available adding parameters. All these tests appear to be broken in Gecko-browsers because it uses a different attribute order (WebKit moslty in order of touch, Gecko mostly alphabetical, Trident/Presto engine different too). They are reported because the elements are compared as strings. I'll fix this in a better way when I think of one (soon!) Comment: Gecko (and opera) fixed in r92125 as seen [http://toolserver.org/~krinkle/testswarm/user/MediaWiki here] , between [http://toolserver.org/~krinkle/testswarm/job/213/ 213] and [http://toolserver.org/~krinkle/testswarm/job/214/ 214]. _______________________________________________ MediaWiki-CodeReview mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/mediawiki-codereview
