mediawiki.legacy.ajax: Pass JSHint
authorTimo Tijhof <krinklemail@gmail.com>
Sat, 1 Jun 2013 11:53:42 +0000 (12:53 +0100)
committerTimo Tijhof <krinklemail@gmail.com>
Sat, 1 Jun 2013 12:05:39 +0000 (13:05 +0100)
commit2a14ddb3c9826af2c601942d9322ce4f87d0661a
tree6237cc17be11998f9d8ac5aade8b61f631270b75
parent41bd1fee921eeb802ed2cea9431be479eb51a0ba
mediawiki.legacy.ajax: Pass JSHint

* Added closure so that "mw.util.wikiScript" doesn't access
  global "mw" directory but uses local reference to global
  "mediaWiki".
* Keps names as-is, as such disabled camelcase rule locally
  for this file. This code will be wrap-deprecated and removed
  over the next 2 releases.
* Turned into local cross-references and exposed those, that
  way the individual methods don't all call implied globals but
  are ensured to be referring to the original methods as they
  were defined in the same scope.
* Fixed clashing 'e' variable name.
* Removed unexpected "delete x" statement. Does nothing and
  causes a lint warning.
* Replaced "delete request" with "request = undefined". The
  purpose of the statement is to explicitly dereference the
  XHR object in older IE browsers. However "delete" is not
  allowed for local varables, setting to undefined has the
  same effect.

* Moved debug_mode and request_type down to the other exposures.

Change-Id: Ib45ed9bd8c6f4ebdb060a527a63eaf59d2839493
.jshintignore
skins/common/ajax.js