showSiteStats now shows ss_active_users statistic

When we implemented tracking the number of active users, we forgot to
update the stat dumper.  Simply show up the value of ss_active_users.

Change-Id: I7324d1aadfaabcd023abb93360f1f7e6d809ff3d

Merge "Added UserCache::getUserName() convenience function."

Check return value of dir() in Skin.php

This avoids a fatal, when something is wrong with $wgStyleDirectory or
the directory referred there.

PHP Fatal error: Call to a member function read() on a non-object in
Skin.php

bug: 42167
Change-Id: I3d63cb984bc2aa0f85ade04607862c639ba51fdb

[JobQueue] Added a function to purge the aggregator cache.

Change-Id: If9eabf202c372d5c594f5faeb75265ff78104f7f

Merge "JSDuck: Cover more of external.js and implement @source"

Merge "(bug 46059) giving #p-personal a higher z-index than #content in monobook"

(bug 46059) giving #p-personal a higher z-index than #content in monobook

... so that its children (like Echo's flyout) will float above
the content area.

Change-Id: Iccb0cc1a96942682007d6570d15a30dd80d7ebd0

Merge "Localisation updates from translatewiki.net."

Localisation updates from translatewiki.net.

Change-Id: I6bf83c7b927a391e90515feb4eff40026c8a628e

Updated PostgreSQL insertId() docs to actually make sense.

Change-Id: Ia6c7240094fc431924b650a327fa19e93ed18261

Merge "(bug 46547) use content language for rotation comment"

Merge "JSHint: Ignore docs/html and docs/js"

JSHint: Ignore docs/html and docs/js

The MediaWiki doxygen documentation is generated by default under
/docs/html and includes some JavaScript files.  Since we do not maintain
that output, simply ask JSHint to kindly ignore them.

Same for the static site generated by JSDuck.

Change-Id: Ief4661128992b65aa85dfaa6f48d31745fe6ff19

(bug 46547) use content language for rotation comment

Bug: 46547
Change-Id: I93519557e15857176f50ea34cb59832c9575f20e

test: ignore code coverage for CleanUpTest

The tests under tests/phpunit/includes/normal/CleanUpTest.php do not use
data provider. When being run for code coverage (bug 31518) they take so
long that they ends up timing out.

The whole class need to be refactored to use data providers (bug 46561).

Change-Id: I78c270e71d23ffbab00d8812ee91d380aef342c6

Fix case of some Title methods

Change-Id: I1a37ffb0e8cde4deac027721b0e463585b58d634

Merge "Move subclasses of LogFormatter to their own file"

Merge "Fixes to Special:WithoutInterwiki form"

Merge "Fixes to Special:FileDuplicateSearch form"

[FileBackend] Cleanup behavior for coping/moving a file over itself.

* Previously, for doQuickOperations(), the default move function
  would just delete the file if it was moved over itself. In fact,
  the php-cloudfiles bindings have this same bug in its move function.
* For both copy and move, when the source and destination are the
  same path, make sure that the headers get updated as specified.
  This only applies for the 'overwrite' case (not 'overwriteSame').
* Fixed bad status for doQuickOperations() when copying a file over
  itself using FSFileBackend.
* Clarified the documentation for 'overwriteSame' option.
* Renamed destSameAsSource to overwriteSameCase in FileOp for clarity.

Change-Id: I3f609d9413795c654de27958b1e807b1fc785f31

Merge "(bug 1495) Include the site language in the fallback list"

(bug 1495) Include the site language in the fallback list

In Iaaf6ccebd8c40c9602748c58c3a5c73c29e7aa4d, the behavior of MessageCache
was changed so that it iterates over all possible on-wiki fallbacks including
the site language fallbacks. However, it omitted the site language itself.
Obviously, the problem appears only when the site language is not English,
because English is included always as the fallback language of last resort.
But in other cases, we need to add it explicitly.

Bug: 1495
Change-Id: Ib607a446d3499a3c042dce408db5cbaf12fa9e3d

Merge "Remove Special:ActiveUsers"

fix some spacing

* Removed spaces around array index
* Removed double spaces or added spaces to begin or end of function
  calls, method signature, conditions or foreachs
* Added braces to one-line ifs
* Changed multi line conditions to one line conditions
* Realigned some arrays

Change-Id: Ia04d2a99d663b07101013c2d53b3b2e872fd9cc3

Remove Special:ActiveUsers

This feature was poorly thought out from the start. It's a huge
performance drain when used, for a mimimally useful set of data

Change-Id: I138622e1c184f74e2a7ce13a9a544ab4e610d7a0

Localisation updates from translatewiki.net.

Change-Id: Ib8ac46b3ac8228020f8744978d141bd8c5a0301b

Merge "QUnit: Re-enable config.requireExpects and add missing numbers."

Merge "Update plural rules for Hebrew"

JSDuck: Cover more of external.js and implement @source

Taken from VisualEditor repo (authored by myself under MIT).
Hereby released under GPL as well.

Change-Id: I4dc1a0e5c618e0a20ae6f308b4ecf266d041e22c

Merge "TimeAdjustTest: Use dataProvider"

(bug 46529) Add gender aliases for Yiddish

Change-Id: I607016bddc6759998c36e73e44754663ae78be70

Merge "Add mobile target to jquery.getAttrs"

Add mobile target to jquery.getAttrs

mediawiki.tests.qunit.testrunner has a dependency on it, which we want
on mobile.

Change-Id: Iffa7166362fe568ce8ed95db427c432e111d65a1

Fixes to Special:FileDuplicateSearch form

- Use Html class instead of Xml where possible
- Add line breaks to the output for better readability

Change-Id: Ia58e0d017a2a54c2458da625bd27a775204908d9

Fixes to Special:WithoutInterwiki form

- Use Html class instead of Xml where possible
- Add line breaks to the output for better readability

Change-Id: I794da047f0664913cfd7ceb00247145f9b7d3a1b

Move RELEASE-NOTES entry to the correct section

Follow-up I719d84c5 (d240930d); that change is in the API,
so the RELEASE-NOTES entry belongs to the API section.

Change-Id: I48581bb59b3ac593443ff092c3fb7a42ac8958e1

Merge "release notes update due to Change-Id:  Idab489fc017dcf226ea39f2f1766e06064e4ec0a (bug 25325) fix hiding bot edits"

release notes update due to Change-Id:
 Idab489fc017dcf226ea39f2f1766e06064e4ec0a
(bug 25325) fix hiding bot edits

Change-Id: Ib07d4923dd6ac44520e459f0e9886bd1f8e0a4b7

Merge "(bug 25325) fix hiding bot edits"

(bug 25325) fix hiding bot edits

I added support for 'wlshow' parameter, which allows user
to decide if to include or exclude bot/minor/anon/patrolled
edits from feed watchlist.

Plus: I bound the params checkup and params' descriptions with
ApiQueryWatchlist (Yurik's suggestions).

Change-Id: Idab489fc017dcf226ea39f2f1766e06064e4ec0a

Merge "IcuCollation::$tailoringFirstLetters: letter removal rules for Finnish"

Merge "IcuCollation::$tailoringFirstLetters: 'sv', 'vi' verified"

Pass function name to insert() in replace() in Database class

Change-Id: I891370aa5d623ac9971c719da7996a93db641a2c

Add to docs that input will be HTML escaped

Change-Id: Ie90a6bc29f54361ec74cbf54551266b94ce6061f

TimeAdjustTest: Use dataProvider

Also removes setting of wgContLang/wgLanguageCode, because test extends
MediaWiki*Lang*TestCase

Change-Id: I8a24fbe161c8f2ecae7591baf99845ff4434f983

Merge "DefaultSettings: Remove some empty lines"

Merge "Add a full stop to the message "noemailcreate""

Merge "New paragraphs wrapping tests in the presence of comments & WS lines."

Add a full stop to the message "noemailcreate"

Change-Id: I10c8ef9fbccfb06e331368a0a36046a646272975

Merge "Remove (edit) from Special:Tags for non-editinterface users"

Merge "Installer: Check if /extensions is readable and a directory"

Revert "Misc follow-ups to I2fc3966e (a161c5e)"

That apparently caused bug 46397.

This reverts commit 388b14a15de6c531d876796dde02605f046fcf53

Bug : 46397
Change-Id: Ideaa86f0d535873a08e27d6f98f6bd4255b8c594

DefaultSettings: Remove some empty lines

Add a missing comment - text taken from
https://www.mediawiki.org/wiki/Manual:$wgProxyMemcExpiry

Change-Id: Ifccf920e1e6d1312d8adb39f83eef59f33cc148b

Merge "Tests: Fix case of some methods"

Merge "ContentHandlerTest: Use dataProvider"

Merge "Tests: Use more setMwGlobals"

New paragraphs wrapping tests in the presence of comments & WS lines.

Change-Id: I3654f28846cce321ea97f70389cabe11b2786082

Installer: Check if /extensions is readable and a directory

Also close the dirhandler and fix one return value

bug: 46447
Change-Id: I0abe540d66375e6833640bb234e62b6cc902beb0

Tests: Fix case of some methods

Change-Id: Iab247f805410f2e9157d325edd80880538a3e54c

Merge "Add tests for case-(in)sensitive magic words"

Merge "Remove <code></code> because it is not parsed"

ContentHandlerTest: Use dataProvider

Changing a global 3 times in a test is bad, using dataProvider for two
tests and split another in 3, because there are different.

Using mwglobals to set the globals, than restoring is done
automatically.

Change-Id: Ie0f9ce0be39e457d18752e905d6deeb1803cb4c3

Merge "Test to make sure numRows() calls don't show unrelated errors"

Test to make sure numRows() calls don't show unrelated errors

Bug: 42430
Change-Id: Ie939785c1f75aa99bdf59e013d5ebf65e866289c

Update plural rules for Hebrew

A recent CLDR update changed the plural rules for Hebrew
and added a "many" form. That rule has a mistake, however -
it is not supposed to include the number 10. This is reported as
http://unicode.org/cldr/trac/ticket/5828

This commit updates the plural overrides for Hebrew and makes
them closer to the latest CLDR, but with a fix for that rule.

It also updates the tests to include support for the new rule
and to make sure that the right fallbacks are used when
less than four rules are supplied, because usually that is
the case. It is thus a partial revert of the changes introduced
in I3d72e4105f6244b0695116940e62a2ddef66eb66 .

Mingle-Task: 2715

Change-Id: I1315e6900ef7a89bf246e748b786f7fc31a629c6

Remove <code></code> because it is not parsed

This reverts a part of Iaa2c89d6
bug: 46142

Change-Id: I33bf2a3ee46156cecc52ebbab79433a56817e2e1

Merge "Tests: Make phpunit providers "public static"."

Merge "MessageTest: Split test and use setMwGlobals"

A list of many words mediawiki uses as strings, comments, names.

This file can be attached to various IDEs to significantly reduce
the number of false positive spellchecking errors.

Change-Id: Ibfb0d89a1cbf58ea031e7873b5e6e01009f141bc

Tests: Make phpunit providers "public static".

Follows-up I9d2b148e57 (including phpunit/languages this time).

Bug: 46434
Change-Id: I30e5efcd88c516121c454676bd7a18f9b7c8fca6

Merge "(bug 23393) (bug 45803) Parser: Fix whitespace handling within headings"

(bug 23393) (bug 45803) Parser: Fix whitespace handling within headings

* HTML <hN> headings containing line breaks are now handled correctly
  (bug 23393).

* Whitespace within == Headline == syntax and within <hN> headings is
  now non-significant and not preserved in the HTML output (bug
  45803).

Change-Id: I0f2d81dd0b2f7742c5cdb6b7d2cc58a15d3f1029

Call loadFromRow() as needed in Title::moveToInternal.

* Follows up 61c0064de538c3b1ea61470eadd88fab87d3513e,
  which was missing a load call to fully fix the bug.
  Calling loadFromRow() forced the page to load as
  non-existing (which it should be).
* The redirect title was triggering a load from a slave in
  updateRevisionOn() and wrongly updating the destination page
  to point to the redirect revision and leaving the redirect
  page with page_latest=0;

Bug : 46397
Change-Id: Id6abe062a0266a790716df8c592911fd89b83223

Localisation updates from translatewiki.net.

Change-Id: I1ffbea309c530d5433a11d98a3844bda1a0d9584

Tests: Use more setMwGlobals

Change some tests to use setMwGlobals to have restoring of globals after
the test.
This also removes some save/restore code, which is not needed, due to
the automatically restoring on tearDown with setMwGlobals.

Change-Id: I8d2ac9f6cc14f0bd4ee8eb851c09f2e71babc6e0

MessageTest: Split test and use setMwGlobals

Change-Id: I9d7f16ed519622d5ac947da8538449f520eb0eae

Merge "API: Fix parameter validation in setnotificationtimestamp"

Merge "Call loadPageData() as needed in Title::moveToInternal."

Call loadPageData() as needed in Title::moveToInternal.

* This avoids use of a slave for loading the page ID to do
  the updates using $newpage. That bug prevented page moves
  by using the old 0 ID and throwing an exception.

Bug: 46397
Change-Id: Iea3259dce6840e3f2959d98a20177acd60433b64

API: Fix parameter validation in setnotificationtimestamp

This was broken in I7a3d7b6e, when the ApiPageSet parameters stopped
being returned by getAllowedParams() and so by extractRequestParams().
Although it would be broken differently if they had been.

Change-Id: I4b6ec21fd7b7c932856546df1ccad574d996db1f

Add SpecialSearchResultsPrepend/Append to release notes

Commit 1c3b2a1 added the SpecialSearchResultsPrepend and
SpecialSearchResultsAppend hooks, which were documented in
docs/hooks.txt, but were not added to the release notes.

Change-Id: I226349aa0199f5d72da0d1fab61e4facf92306b6

[JobQueue] Added support for delayed jobs with JobQueueRedis.

* The queue can handle delaying jobs until a given timestamp is reached.
* Added Job::getReleaseTimestamp() to let jobs specifiy delay amounts.
* Added a "checkDelay" option and a supportsDelayedJobs() function to JobQueue.
  There are also getDelayedCount() and getAllDelayedJobs() functions.
* Simplified a bit of code in doBatchPush() and pushBlobs().
* Improved the logic in redisEval().

Change-Id: I40b3e3438e659f6844bdbdd5e9d3ccc6c4dc82b2

Merge "Fix WDDX formatted output"

Merge "Add comments for grep (includes/installer/)"

Fix wfWaitForSlaves() for foreign DBs

We thought we'd fixed it previously, but the LB now connects to the
foreign slaves and tries to select $wgDBname. Fixing this by having
doWait() open connections that don't select any DB name, by passing
the empty string as the DB name. This is an undocumented feature
in the DatabaseMysql constructor, apparently.

Change-Id: Icc5594143a3eaa31170a7fc8c93d429c1a7dd575

IcuCollation::$tailoringFirstLetters: letter removal rules for Finnish

Four non-ASCII letters - Ǥ, Ŋ, Ŧ, Ʒ - are sorted the same as their
unaccented base ASCII versions - G, N, T, Z - causing unexpected
output on category pages.

Bug: 46330
Change-Id: I976dedfdc651fcc96a2291934924aa40b27f4c2f

Add comments for grep (includes/installer/)

Change-Id: Idb44b343267978ae214b2cfe9a05947b220629d7

Merge "Fix wfWaitForSlaves() so the $wiki parameter actually works correctly"

Fix wfWaitForSlaves() so the $wiki parameter actually works correctly

Change-Id: I1f3da5921f2d724e10c448700ea8b958ff159451

Merge "Localisation updates from translatewiki.net."

Merge "MagicVariableTest.php: Use dataProvider"

Localisation updates from translatewiki.net.

Change-Id: Id18fb2721534f365dbcab26c7531fca87f859b28

MagicVariableTest.php: Use dataProvider

Setting a global 3 times in one test method looks weird

Change-Id: I97f6a932ee2d5c9a5e427cf0f40ced730b7e12ec

SideBarTest.php: Use dataProvider/use setMwGlobals

One test method should cover one thing, so splitting this up.
Removing save/restore of globals, done automatically with setMwGlobals

Change-Id: Ideefaf23723b224fc14aff9fdc448f1ee6ec9ea9

Merge "Added a PoolCounterWorkViaCallback convenience class."

Merge "Allow the retrieval of the plural rule type for a given number"

Merge "$wgContLang global unneeded in braceSubstitution() since fc00763f0"

Merge "A few minor doc group tweaks."

$wgContLang global unneeded in braceSubstitution() since fc00763f0

Change-Id: I0fc7f1ca5989d4a7ca8b4a49fd1f5229f23f9be5

Allow the retrieval of the plural rule type for a given number

For example, find out which rule type should be applied for 5 items
in Arabic. The result would be 'few'.

This implementation should be non-disruptive and completely backwards
compatible (which is the main reason it isn't a lot simpler).

Change-Id: I3d72e4105f6244b0695116940e62a2ddef66eb66