QUnit testrunner: Fix extend call
authorMatthew Flaschen <mflaschen@wikimedia.org>
Tue, 12 Feb 2013 21:06:58 +0000 (16:06 -0500)
committerKrinkle <ttijhof@wikimedia.org>
Wed, 13 Feb 2013 05:45:58 +0000 (05:45 +0000)
* deep only has an effect as the first argument.

http://api.jquery.com/jQuery.extend/#jQuery-extend-deep-target-object1-objectN

Change-Id: I7d20f0b2f4ea63b7e8ce102d786df8d6ae008725

tests/qunit/data/testrunner.js

index 0c3d364..b751ca9 100644 (file)
                        // e.g. mw.config.set( 'wgFileExtensions', [] ) would not effect liveConfig,
                        // but mw.config.get( 'wgFileExtensions' ).push( 'png' ) would as the array
                        // was passed by reference in $.extend's loop.
-                       return $.extend( {}, liveConfig, custom, /*deep=*/true );
+                       return $.extend( /*deep=*/true, {}, liveConfig, custom );
                }
 
                function freshMessagesCopy( custom ) {
-                       return $.extend( {}, liveMessages, custom, /*deep=*/true );
+                       return $.extend(  /*deep=*/true, {}, liveMessages, custom );
                }
 
                log = QUnit.urlParams.mwlogenv ? mw.log : function () {};