Per wikitech-l, remove premature wfDeprecated()

This way of deprecation is way too rushed, we generally don't mark as
@deprecated and put wfDeprecated() in the same release.

Change-Id: I5dc07d9819b6fb77fcf3194ed16676eaacb834a8

Fix spacing after @param and friends in comments

Searched for:
\@(param|return|throws|since|deprecated|access|todo|var)[ \t]{2,}

Change-Id: Icce22ba9fe0635455691ca58d9872d618151f346

Merge "Localisation updates from https://translatewiki.net."

Localisation updates from https://translatewiki.net.

Change-Id: I44b9547b386f7c169a7add27096e2708d5a3cce9

fix typo in comment in Preferences.php

The name of the preference was spelled wrong.

Change-Id: I443d0a8fc67c89da36b5c725dd3f3513d4170357

Merge "Remove trailing line break check from MemcachedBagOStuff::debugLog()"

Correctly order AutoLoader class definitions

- Group per directory
- Sort by class name for each directory

Change-Id: If4771f2868c2183f60a6321533142a1ad5cdc7cd

Remove trailing line break check from MemcachedBagOStuff::debugLog()

wfDebugLog() already does the same, so it's unnecessary here.

Change-Id: Ia7bbbd2806fcd638861d010835c33969322d262a

Fix path after file got moved

Change-Id: Ic67f590fb53f2b96cfd8a0192968cb4953bb0d2b

Fix documentation typo

Change-Id: I535dd23eeccd6686c64a885f06e030b756e67fec

Revert "Localisation updates from https://translatewiki.net."

wtf?

This reverts commit ff744475d7d37d4887ebaba3d69884317a736c34.

Change-Id: Ie7ea50de88f87aa527205a816837dbfa5950bb0b

Localisation updates from https://translatewiki.net.

Change-Id: Idfe1499c79cd459377b31ca022fc45a9e0b3e833

Merge "mw.hook: Make hook.fire actually chainable"

Merge "Update jquery.fullscreen to newer version (fix IE11 support)"

mw.hook: Make hook.fire actually chainable

It was being called without being bound to the mw.hook instance,
as such the (detachable) callbackList.fireWith method (which
returns its given 'this') returned the callbackList instance
instead of the mw.hook object literal.

The tests weren't catching it because it was just checking one
can call .fire() without it crashing, but it was in fact calling
list.fire instead of hook.fire (which unlike #add and #remove are
not compatible, and we might add new methods later on).

Change-Id: If3d4dfbed494e7ff9f32514e539cfa089aea18ac

Merge "Add a dot at the end of the deprecate messages"

resourceloader: Refactor module links output

Changes:

* Removed hardcoded logic in OutputPage regarding modules
  being "enabled".
  Previously we would always output state=loading and use
  $wgAllowUserJs (and others) to decide whether to output
  state=ready or makeResourceLoaderLink.

  Now, we no longer unconditionally output state=loading and
  simply always call makeResourceLoaderLink. That method takes
  care of checking whether modules are enabled and non-empty
  and returns state=ready when that is the case.

  This cleans up cases where the duplicated and incomplete logic
  in OutputPage thought the module was non-empty but turned out
  to be empty and thus would output both state=loading and later
  state=ready for the same module.

* Clean up documentation for makeResourceLoaderLink (inconsistent
  ordering of type hint and $var, and @return was missing the fact
  that the returned html can also contain <link>).

* makeResourceLoaderLink now returns an array of html and module
  states. This allows the consumer of this method to combine the
  states in 1 larger script tag on top of multiple
  makeResourceLoaderLink calls (e.g. one state script followed
  by multiple <script src=load.php>).
  This isn't to reduce html output, but to make sure we inform
  mw.loader about modules before the <script src=load.php>.
  If we were to mix/alternate the state script and load.php
  requests (which are blocking in html), it is possible for those
  scripts to request other modules. We need to prevent duplicate
  loading of modules we already know are going to be requested
  by the HTML output futher down.

* Removed spurious new line.

Example of change in HTML output:

* The output has been reduced from:
  - loader.state( site: loading, user: loading, user.groups: loading )
  - loader.load( .. )
  - <script src="load.php?modules=site ..">
  - loader.state( user: ready, user.groups: ready )

  to:
  - loader.state( site: loading, user: ready, user.groups: ready )
  - loader.load( .. )
  - <script src="load.php?modules=site ..">

Change-Id: I91754ce5fae3d05b4bfa7372372eba81ee2fc579

Merge "Callers of ResourceLoader::getModule should check for null return"

Callers of ResourceLoader::getModule should check for null return

ResourceLoader::getModule returns null if cannot find a ResourceLoaderModule
instance or a module configuration array matching the requested name. Existing
callers of that method chain ResourceLoaderModule method calls to the return
value of ResourceLoader::getModule without bothering to check for a null
result. This patch makes the documentation for the method more explicit, and it
adds checks for the return value to existing calls of the method. In some
cases, it is possible to reason to that the call will succeed, on the
assumption that missing modules would have been filtered out by earlier
iterations through the modules array. But in the interest of robustness, it is
always good to check.

Bug: 63310
Change-Id: Ic32f1e61ffee0383f7a8761423099041e3b6b8cc

Update OOjs UI to v0.1.0-pre (eaa1b7f06d)

New changes:
468b1c8 Localisation updates from https://translatewiki.net.
3e2a5e1 Update OOjs to v1.0.9
eaa1b7f Add pressed state to option widgets and use instead of initializing items

Change-Id: I6e93d9ab0e4a926c09c5e08e3db045cff99d2098

Merge "New API module, list=prefixsearch"

Added per-wiki queue stats information

Change-Id: I25f42408c9a7a368aa99f4ceb04214f188a74be6

Use rgba() for gallery caption background instead of opacity

Opacity applies to child elements, and can't be unset.

Change-Id: Id4b58ccf82cf3b6d23f67408c5311da0eb92fbce

Merge "mediawiki.api: Refactor getToken and postWithToken methods"

Merge "ApiFeedRecentChanges: Validate param target"

Add a dot at the end of the deprecate messages

Change-Id: I73b4bc296051bf506683434de9bc7b56f3ff825f

ApiFeedRecentChanges: Validate param target

Change-Id: I6933177d47c25d48b9b55d479afa4dedcc64299d

Merge "mediawiki.html: Add missing @return documentation"

mediawiki.html: Add missing @return documentation

Change-Id: I63f12ee44aace6cf4f7762dd67244648b95d7301

Merge "Update OOjs UI to v0.1.0-pre (8986c46d35)"

Adapt signup form numbers to new Vector typography

The .mw-number-text CSS was less specific than Vector's typography
styling of 'div#content h3'.  Match Vector's serif font stack.

Bug: 63450
Change-Id: I23041c356ad4356dcf085479721cb950a89b33d0

Update OOjs to v1.0.9

Code:
 https://oojs.github.io/releases/oojs-1.0.9.js

Release notes:
 https://git.wikimedia.org/blob/oojs%2Fcore.git/v1.0.9/History.md

Change-Id: Ic1fea7dd3a07af32a29511191c45da90bc306b17

mediawiki.api: Refactor getToken and postWithToken methods

* getToken now keeps promise objects around so that it can:
  1) Re-use already retrieved tokens instead of always making
     a new request (this was previously done locally inside
     one of the callers, namely #postWithToken).
  2) By storing promises instead of the tokens, we can re-use
     it asynchronously. While it is unlikely that one module
     calls getToken twice in a short period amount of time,
     it is quite likely that two separate modules (both using
     mediawiki.api) request similar tokens from the API at the
     same time (e.g. at page load).

  This cache has to be additionally keyed by API endpoint since
  mw.Api is a generic class that can be (and in practice, is)
  instantied for foreign wiki as well.

* Initialise the cache with pre-resolved promises for values
  from user.tokens from the local wiki (embedded in the page
  output, no point in requesting these again from the API).

* postWithToken no longer has its own token cache, it now
  unconditionally uses getToken.

* postWithToken already took care of clearing the token cache when
  the server responds with a 'badtoken' error. However, it then
  changed the request to delete the 'token' query and try again
  without any token. That seems rather silly. Change this to
  instead have getToken fetch a fresh one and re-submit with that.

* Add unit tests.

Bug: 34733
Change-Id: I5c25ae5ea4bf3336899bc60fd94ec3b0948050e1

Better memory reporting when running in HHVM

Change-Id: If06b7b3ff4dc7ba8b873361f7878f8abb718f571

Visualize profile output in debug toolbar

This adds a new pane for the debug toolbar labeled Profiler.  It groups
together events sent to wfProfileIn/Out and draws a timeline giving a
visual representation of what the page was doing at a particular point
in time. Hovering over a timeline brings up a flyout about the contained
events.

Depends on I08854d80f57286129f19cb14566f84ed3c7fa21c for proper positioning
of the flyout.

Change-Id: I23a0ce4137abcd74ad87967fb263214c08cb9685

Merge "Made BagOStuff fail fast in cas/lock on certain errors"

Merge "mediawiki.ui: Do not depend on divs being present inside .mw-ui-vform"

Merge "Fix language statistics after messages have been moved to JSON"

Respect sizing from user-supplied class

A user-supplied class provided to tipsy that affects the
height or width of the flyout breaks tipsys positioning.
Adjust tipsy to apply the user-supplied class earlier so
its css rules are taken into account during positioning.

Also clean up trailing whitespace.

Change-Id: I08854d80f57286129f19cb14566f84ed3c7fa21c

Merge "Cleaning up function definitions in Language.php"

Cleaning up function definitions in Language.php

Some of these were confusing or incorrect.

Change-Id: I2dd02669a7cf9d8b92e2d5c1b86ab8c5e0cbf6bd

Update jquery.fullscreen to newer version (fix IE11 support)

jquery.fullscreen wouldn't work properly in IE11. This commit fixes the
issues that were preventing it from working in that browser.

Source
* https://github.com/theopolisme/jquery-fullscreen/blob/9f8f97d127/jquery.fullscreen.js

Change log
* https://github.com/theopolisme/jquery-fullscreen/compare/v2.0.0...9f8f97d127

Change-Id: I6cb853f05ef9a464990a785827074c78ac61ac7f

Re-organized PoolCounter files

* No actual changes to the classes

Change-Id: I4e35a2b097e4bc72315688d0390d3f5c4c1ca4ed

Fix language statistics after messages have been moved to JSON

Statistics are now showing always no messages since languages.inc
does not manage to load the JSON messages.

Had to make LocalisationCache::readJSONFile() public so that it can
be used in  languages.inc; since all methods from LocalisationCache
return localisations merged with fallbacks, which is not what we
want here.

Fix for I918cfdc46c (0dd91d5).

Change-Id: Ib52287db618b9d072e847130070d165a3e7ae44b

Merge "Adding debugging for bug 62241"

Rename $p to $parts in Special:Search

Change-Id: I794fca2d24fcebb59067d5e619b68d86cbaa2a8f

Fix variable naming in SpecialSearch, $t => $title

Change-Id: I70ec1e147f99a73bc0b12511669ab5a799018695

New API module, list=prefixsearch

It's pretty much like action=opensearch but can be used as a generator
which can be handy when you need to retrieve both the list of pages and
some information about them.

Change-Id: Iaffe30a0f7402e1316c4885a805692a34bbe1a6a

Rename $t to $title in SpecialSearch

$title is a bit more informative than $t

Change-Id: I66deeb6672dc45161e0fcc30988bcc1efcb93f56

i18n: Fix the parser function name

- Fix the parser function name "#ifexist"
- Put the parser function in code tags

Spotted by Siebrand
https://translatewiki.net/wiki/Thread:Support/About_MediaWiki:Expensive-parserfunction-category-desc/en

Change-Id: If29585ceda7f409c668804ff13c821b4eece95fd

Add punctuation to "pool-servererror"

Follow-up on eb03cf3f7dda4.

Change-Id: Iee1fd4ca9f4baad0d03998dbc1ea1e24f2210ba7

Merge "Put LESS test back into extension suite in XML"

Update OOjs UI to v0.1.0-pre (8986c46d35)

New changes:
735cebc Filter calls to selectItem and highlightItem on SelectWidget
897b30f Return null in Element#getDocument
b5ede88 Add Element#isElementAttached
68df19f Disable tool groups when all its tools are disabled
1aaa507 Grunt-watch less files
e4dd7b2 build: Update devDependencies
64d6c32 Localisation updates from https://translatewiki.net.
b5c6db5 Localisation updates from https://translatewiki.net.
b9460cb apex/button: Reset border-color of disabled button when not hovered
32c4799 Localisation updates from https://translatewiki.net.
674165e Enable jscs for OOjs UI repo
d98b338 grunt: Add messageDocumentation task to ensure each 'en' message has a 'qqq'
d421dd6 demos/tools: Indentation fix

Change-Id: Ic9b7ff01fe066eb5cba6a0fd0fc515ff3c62b66a

Added a Redis pool counter class

* This should be easier to set up for typical installs

Change-Id: Icb4a7481b944fa0818c4635e3edbe12d08af9924

vector: Remove redundant margin-top reset

There is nothing setting this to anything else. The only other
preceeding relevant rule inherited before this is "h1, h2, h3 .."
which sets "margin: 0;" already.

Change-Id: If6531221891ae3c7eb792c5bdc930ddaaf613f52

Put LESS test back into extension suite in XML

Followup to 13f3e21e9c7ddcef0d59982aeb3bb4b9e240e5f4

Change-Id: I1a15a0d8aba05b774f072673787b712f6ba97fcd

Migrate translations for core to JSON format

Largest patch set in the history of MediaWiki!

Change-Id: I918cfdc46c099017ca3622d1bcbbdd30c4fd00f6

Preparations for migrating core to use JSON based i18n

LocalisationCache and Language have to take the JSON files into account
in deciding if a language is present or not.

Standardizing language validity checking with isSupportedLanguage
and isValidBuiltInCode.

Co-Authored-By: Niklas Laxström <niklas.laxstrom@gmail.com>
Co-Authored-By: Siebrand Mazeland <siebrand@kitano.nl>
Change-Id: I35bbb3a7a145fc48d14fff620407dff5ecfdd4fc

Made BagOStuff fail fast in cas/lock on certain errors

* This avoids wasting time waiting on down servers

bug: 63058
Change-Id: Ia78a2036ffa7a20102ec9e8bf941cacb7fa63435

Send profiled items under the correct name

Fix for Ie8481a2e13 (811a084). The "-total" item
was send under the name "close" since it's ended
by close().

Bug: 63347
Change-Id: Iad91f1d12303e5cfb92c111f79c0ce5a8187f53b

Merge "Include parsed revision ID in parser cache"

New Hook rc/watchlist hook ChangesListInitRows

An extension hooking into recent changes may need to load additional
data that did not fit into the recentchanges table.  This hook gives
extensions an opportunity to see the full result prior to rendering
and batch load where approprite rather than loading piecemeal during
the render process.

This is implemented as an optional method called by the ChangesList
consumer, since the ChangesList never sees the full result set. Hook
implementers must be able to work regardless of the hook being called,
they just have the oportunity to be more efficient when it is called.

Change-Id: If74ae600ffba949364dd381dd3d466cbbaa27286

Include parsed revision ID in parser cache

One theory for what's behind bug 46014 is that the vandal submits the
edit, then someone (maybe the vandal) gets into the branch of
Article::view that uses PoolWorkArticleView, then ClueBot comes along
and reverts before the PoolWorkArticleView actually executes. Once that
PoolWorkArticleView actually does execute, it overwrites the parser
cache entry from ClueBot's revert with the one from the old edit.

To detect this sort of thing, let's include the revision id in the
parser cache entry and consider it expired if that doesn't match. Which
makes sense to do anyway.

And for good measure, let's have PoolWorkArticleView not save to the
parser cache if !$isCurrent.

Bug: 46014
Change-Id: Ifcc4d2f67f3b77f990eb2fa45417a25bd6c7b790

Merge "i18n: add "Special:MyLanguage/""

Merge "Fix order of expected, actual in some tests"

Fix order of expected, actual in some tests

Change-Id: Iabd431517586c06d411a0894e5d3621a4c4ddf05

i18n: add "Special:MyLanguage/"

Change-Id: I791399e1a9bdff0bc672fc6ae69f4c2374690636

Add newline at the end of converted json files

1) Some text editors add missing newline at the end of the file
automatically.
2) It's generally a nice thing to do to avoid messing up display
for example when cat'ing a file

Change-Id: Icf636721c4edcf4ae706c9a346fe36233928cae8

Speed up LocalFile locking behavior

* Lowered the lock() timeout
* Wrap the destination DB write with memcached locks for move
  ops since it is easier to control that lock timeout than the
  DB lock timeouts.
* Cleanup locks in the destructor (e.g. on exceptions). Other
  process can pile up more if locks have to expire.

bug: 63058
Change-Id: I38b28d81ec96daa80ece2354db284a614289ba0b

Adjust debug log for caches being used

I always found out the debug message listing the cache being in use to
be slightly confusing:

    MemcachedPeclBagOStuff[main] MemcachedPeclBagOStuff[message]
    MemcachedPhpBagOStuff[parser]

Swap the class name and identifier to output:

    main: MemcachedPeclBagOStuff, message: MemcachedPeclBagOStuff,
    parser: MemcachedPhpBagOStuff

Change-Id: If069dec70eb31ed2dafe0e88ec4d4653005ff7e3

Merge "Add a LESS test suite"

Add a LESS test suite

Add the LessFileCompilationTest test case class, which represents the
validation of a LESS file by compilation.

Add the LessTestSuite test suite, which tests all LESS files registered
with the ResourceLoader, and use this to rewrite the checkLess.php
maintenance script.

Bug: 54665
Change-Id: Iedb8dc31e4817d8b4e40b655cf9b8fb092979e90

Merge "mediawiki.log: Fix unknown method "apply" error in IE9 and below"

mediawiki.log: Fix unknown method "apply" error in IE9 and below

In IE < 10, the console.log function is implemented as a non-standard
host function (not inheriting the regular Function prototype) which
doesn't provide an "apply" method.

Change-Id: I6ecf933589ec430ff4d1748a5f3c511b692e3b9a

Adding debugging for bug 62241

In the event of a bad filename warning record the relavent variables
so that we can figure out what is causing the failure.

See also change I532f8ee7c7455e43862b0204a7fff23d6792412e

Bug: 62241
Change-Id: If5d8b9e7f34c0f9bd09684d7dada8fafd7f5f7c2

Merge "MWException: Fix sitename appearing twice in <title>"

Localisation updates from https://translatewiki.net.

Change-Id: I54a8e52634c1e36fadd7a84f236747765fc98684

MWException: Fix sitename appearing twice in <title>

The MWException::getPageTitle method is wrapping the page title
in the 'pagetitle' ($1 - sitename) message, but OutputPage does
this as well when outputing <title>.

OutputPage::prepareErrorPage takes the first parameter as page
title (unwrapped, appears in <h1>,) and the second optional one
is for <title>, which (if omitted, like before) results in
pagetitle being applied by OutputPage.

Now that we specify them separately, this also removes the
sitename from the <h1> on the page, which always looked weird.

It also fixes DBQueryError (extends MWException) where previously
no sitename was added, it just returns 'databaseerror' from
its DBQueryError::getPageTitle() method.

Change-Id: I85d65d89e72b4497a050f440524ff6f959518b88

Merge "Adjust margin between h2 and special page table element"

Adjust margin between h2 and special page table element

With the recent typography update, this margin is much too small.

.mw-specialpages-table applies "margin-top: -1em;" style which
doesn't fit with the vector styles. So this provides an override
for vector.

Bug: 63312
Change-Id: I18a711802a18b5976cf749d76138a1a6f59e0a4e

Merge "API: Allow for format modules that cannot handle errors"

Merge "Expose thumbnail file to extensions"

Merge "Improve handling of uncommitted DB txns with "uncaught" exceptions"

Merge "Provide RequestContext from fixDoubleRedirects"

Merge "Send the profiler output to the 'profileoutput' log group"

Merge "Add missing line breaks to wfDebug() calls"

Merge "Update IRCLineURL hook to include RecentChange object"

Follow-up I7d4bb90: Message tweak + add code comment

When I was saying it in my head, "No description available."
sounded better to me than "Description not available", although
perhaps that's just me.

Add a code comment to Parser::addTrackingCategory telling people
to register their tracking categories.

Change-Id: I17bb0cf4b3118dda8647e0607588685c2b5cdb86

Merge "Implement listing for tracking categories"

Localisation updates from https://translatewiki.net.

Change-Id: Ifc65d7c3844bfaabd97c803b2c72dc8a49cdcc65

Merge "vector: Style .mw-editsection-like in addition to .mw-editsection"

Localisation updates from https://translatewiki.net.

Change-Id: I6ac7ce50b6204e9e9d63695c9532d33796d4cc7d

Merge "commonElements: Move sup, sub styling from Vector's CSS to common styles"

Send the localisation store debug message to the "caches" log group

Rationale is the same as for Ifb6dc2666f (96b04ce): ability to
separate it form the main log group, as it always has a predicitible
value.

Change-Id: I0ac5baf5ab8fbe4d2025ccf6439b883dc82e9d5b

commonElements: Move sup, sub styling from Vector's CSS to common styles

This really should be a basic style, not limited to one skin. All
Wikimedia wikis which used this hack seem to have it on their
common.css, not vector.css.

Follow-up to Ic5ba8363.

Bug: 49965
Change-Id: Ie98f8b57a71b703fbba19c201c13edef3f6adc55

vector: Style .mw-editsection-like in addition to .mw-editsection

Follow-up to Ic5ba8363.

Change-Id: Ia60281595b5903474b660b776bbbe8f2c3819d3f

mediawiki.ui: Do not depend on divs being present inside .mw-ui-vform

This breaks forms that try to use semantic elements like <section>,
e.g. the user creation form here in core.

Bug: 63233
Change-Id: Icb4a6d1d74490808b2fd1777d7dbe812e66bd206

Add missing line breaks to wfDebug() calls

Also removed true as second parameter to it from CloneDatabase.php
since it is the default value of that parameter.

Change-Id: I727ebae2bd4df0e26019985ce8c7ce73381c5642

Implement listing for tracking categories

Special page to implement list of Tracking Categories. Global
$wgTrackingCategories added containing list of tracking categories

Bug: 60333
Change-Id: I7d4bb90622a6bae60845942ef93cfe64f229d2d2

i18n: Give grep a chance to find the usages

Change-Id: Iac0d7cba8633fedd420c6b8e21999e94b4d438a2