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

Merge "Update plural rules for Hebrew"

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

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

Merge "Remove custom plurals for Nso and Sl in favour of CLDR"

Remove custom plurals for Nso and Sl in favour of CLDR

Nso - Northern Sotho
Sl - Slovenian

Plural rules were not changed. They are same in CLDR and MW

Change-Id: I0e0c84352de2de8f58af5a9147ba18b0fe1fb39a

Added a PoolCounterWorkViaCallback convenience class.

Change-Id: I1cbb2cc8a43eea2d28f38cfd2d3a4908dc7d0da9

Merge "Add test for table with empty line between table and row tag"

Merge "Extend paragraph / line with block tag interaction tests"

[JobQueue] Factored "root job" de-duplication code into base class.

* Moved the root job handling up to the base class and provided
  some default function implementations.
* Also bumped ROOTJOB_TTL from 14 to 28 days.

Change-Id: I70bc043bfc039c5d0b009e0b5d39fd2887f46093

Add parser method to call parser functions

There is currently no straightforward way for anything to call a parser
function and get the result. This abstracts out that portion of
braceSubstitution() to allow this.

The immediate motivation for this patch is to close bug 41769 against
Scribunto, see I0138836654b0e34c5c23daaedcdf5d4f9d1c7ab2.

Bug: 41769
Change-Id: I339b882010dedd714e7965e25ad650ed8b8cd48f

[JobQueue] Optimized redis queue to use Lua scripts.

* Cleaned up some data structures into hashes, which get better
  compression and play well with the KEYS parameter in Lua scripts.
  The claim list is now a sorted set with O(logN) removal in ack()
  and O(log(N)+M) searching in recycleAndDeleteStaleJobs().
* Made the class itself control object serialization, so that lua
  scripts have an easy time. Only the job data itself needs to be
  serialized, where as other things just get bloated.
* Used Lua scripts to get push(), pop() and ack() down to 1 RTT.
* Likewise rewrote recycleAndDeleteStaleJobs() to use a script.
* Fixed bug where claimed duplicate jobs removed the data on ack(),
  which meant that claimed duplicated jobs could no-op newer ones.
  De-duplication should only apply to unclaimed jobs like for the
  JobQueueDB class, so that unfinished jobs don't no-op new ones.
* Removed locking in recycleAndDeleteStaleJobs(), which would not do
  much since the exclusive set request would serialize on the lua
  script anyway. The lua script will finish quickly the next times
  if done more than once in a row due to sorted set usage.
  Also made recycleAndDeleteStaleJobs() run randomly to reduce the
  chance of a single calling tying up the server.
* Removed useless hDel() call in getJobFromUidInternal().
* Changed unit tests to handle the different supported orders better.
  Added tests for the 'timestamp' ordering.

Change-Id: Ib2d7aff18753195248ab856afd4a46e18b301db9

Add tests for case-(in)sensitive magic words

Parsoid needs these to make sure it doesn't regress along these lines, and for
some reason they aren't tested anywhere else.

Change-Id: Id8fd91b0d36d31f9bb209aa4088f1e28cadbd7ca

Add test for table with empty line between table and row tag

Test case:

{|

|-
| foo
|}

Change-Id: I71c0be624aef3b9cf40b58cab8c7a448dda44033

Extend paragraph / line with block tag interaction tests

Tests this case too:

<div>a</div>b

<div>b</div>d

<div>e</div>

Change-Id: Ic0cc36ad8f43c3147737d38b6d9acd1100062230

Localisation updates from translatewiki.net.

Change-Id: Ided02055632d04dc2223fbcdd308132ea3b1cd81

Revert "Change login/createaccount forms to new appearance"

The code did not go through a proper review process, and - quite
simply - it is unacceptable by core standards (or by any standards, to
be honest).

* It "[u]ses JavaScript to munge the create account CAPTCHA", to quote
  the commit message
* It is littered with TODOs and FIXMEs, as well as messages from one
  coder to the other and commented-out code
* It modifies some HTTPS-related logic for no obvious reason (it's not
  documented or even mentioned *at all* in a otherwise front-end
  changeset)
* It happily disregards code conventions (trailing spaces, #-comments,
  CSS formatting)
* It includes different CSS font-family rules that the entire rest of
  software uses for display, creating design inconsistency
* It hardcodes links in the format "/wiki/XXX"
* It hardcodes English-language link hrefs

This reverts commit 92bb00d35688e72ebeea75ed55a6db981cfb71e5

Conflicts:

languages/messages/MessagesEn.php

Change-Id: I00d72fe157e697d5cf926e75bcea5db0bee153e5

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

bug: 46180
Change-Id: I5106428b78ebdeeda4afe6fd07fba4e1e264dd18

Merge "Rebuild MessagesEn.php"

Merge "Cleaned up PoolCounter docs, moved up some functions, and declared public functions."

Rebuild MessagesEn.php

Change-Id: I8fd6fbb455f57415c217c66e828dd27d6f2bc293

Merge "Change login/createaccount forms to new appearance"

Merge "wfProfile fixes:  0543c7b0 added a wfProfileIn() that should have been wfProfileOut()  Reorder Http::request() so that wfProfileOut() is at the same level as the above wfProfileIn() [checked by check-vars]"

wfProfile fixes:
 0543c7b0 added a wfProfileIn() that should have been wfProfileOut()
 Reorder Http::request() so that wfProfileOut() is at the same
level as the above wfProfileIn() [checked by check-vars]

Change-Id: Ia91445d792e524b9c570890397f1513af0d8cd73

Moved some RedisConnectionPool docs to the factory function.

Change-Id: I4183b96e123882de604de1484456abcd10091fb2

Merge "Check for rev_len not rev_sha1"

Change login/createaccount forms to new appearance

If a global variable is set or if you add ?useAgora=1 to the query
string, Special:Userlogin loads a different login or create account
template (Userlogin-/UsercreateAgora.php) with an Agora look and
different messaging.  Otherwise the current form is unchanged so
that wikis can cut over to the new look when desired.

These new templates apply mw-ui-formlist and mw-ui-button styles defined
in a new 'mediawiki.ui' CSS module in core (copied from Extension:Agora).

In useAgora mode, Special:Userlogin also:
* Adds new modules with some additional CSS for new form features
  ("Join wiki", benefits of creating an account).
* Defines new "userlogin/usercreate-xx" messages, many are the same as
  existing messages but without ':' on the end.
* Uses a distinct title for each mode instead of generic "Log in /
  Create account".
* Uses JavaScript to munge the create account CAPTCHA.
* Outputs checkboxes using UserloginTemplateAgora::labelledCheck()
* Displays a benefits column of wiki edits/users/contributor numbers.

TODO:
- Restyle/reposition language selector
- Munge CAPTCHA in PHP not JavaScript, i18n of new CAPTCHA messages.
- Identify the subset of Agora appropriate for non-Vector skins and
  create mediawiki.ui.default.css from that.

Patch set 18: Agora styles now in core.

Bug: 44628
Change-Id: I859edab4fc4fa9fe35fdef15fc429ae19a95305d

Merge "Add two table parser tests"

Add mobile target to QUnit and its dependencies

This allows us to run mobile JavaScript tests in the environment (skin)
they were written for.

Change-Id: I84e0512590de9ff2dbdf519d983a0c1c2d007194

Cleaned up PoolCounter docs, moved up some functions, and declared public functions.

Change-Id: I9b3a320a3ea12c3d7aea09236cd0ecbf37a19860

Add two table parser tests

One tests paragraph wrapping inside of table cells, the other mixed wikitext /
html syntax tables in Parsoid only.

Change-Id: I1e07e868d7f329ac1fa5d33b7fa07c7e701f41ba

A few minor doc group tweaks.

Change-Id: I18a8d733b3dbaa925a676c881225a63e90ff2f05

Added profiling to MWHttpRequest and Http.

Added profile calls to various functions in
MWHttpRequest as well as to the wrapper
function Http::request.

Change-Id: I7f2819a075891dbc565a7524d56e7bf077ab089a