mediawiki.searchSuggest: Code clean up
authorTimo Tijhof <krinklemail@gmail.com>
Wed, 21 May 2014 01:07:08 +0000 (03:07 +0200)
committerKrinkle <krinklemail@gmail.com>
Wed, 21 May 2014 01:20:34 +0000 (01:20 +0000)
commit73404332e83a7bf7bb6f85d1099bae41d5f5e8e2
tree5fccaf6795531e2d449b93f27b40efc37d137941
parentee5b96fe5df3bfdbc811259230d2f3da7e4ff1b6
mediawiki.searchSuggest: Code clean up

* Cache mw.Api instance.
* Remove redundant "str.length" check in fetch(),
  jquery.suggestions already takes care of this (it never calls
  fetch if the string is empty, and if it did, then this code would've
  left it unhandled causing bugs (e.g. suggestions of the previous
  fetch remain visible).
* Remove redundant promise.abort check.
* Use $.data() and $.removeData() to bypass unneeded convenience
  logic for collections and HTML data attributes.

Change-Id: I59ae21c4b481ef52e4db14cb1f3f2fe026c23b3f
resources/src/mediawiki/mediawiki.searchSuggest.js