Omit 'window.' when accessing browsing location
authorTimo Tijhof <krinklemail@gmail.com>
Wed, 19 Nov 2014 00:11:04 +0000 (00:11 +0000)
committerKrinkle <krinklemail@gmail.com>
Wed, 19 Nov 2014 00:13:51 +0000 (00:13 +0000)
commit27dd806c6cd2cf86ed6bcf0e15f16443c5fe87df
treedca65cd665d1f4cd803a2f49df8dbe5e3b0af61d
parente42aa3befae9118fda981ec6b188f96d90f3dd5b
Omit 'window.' when accessing browsing location

The location object is a global, just like document.

Using it via 'window' needlessly adds complexity and, for example,
makes it harder to catch typos in static analysis.

Also standardise on location.href in place of the many different
variants, like:

 location =
 location.href =
 location.assign() =

And each with 'window', 'document' and without host object.

Change-Id: I77510294d8b5bd4b8a1b08e06817762a7839d43d
includes/installer/WebInstallerPage.php
resources/src/mediawiki.page/mediawiki.page.ready.js
resources/src/mediawiki.special/mediawiki.special.javaScriptTest.js
resources/src/mediawiki.special/mediawiki.special.preferences.js
resources/src/mediawiki/mediawiki.Uri.js
resources/src/mediawiki/mediawiki.util.js