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

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 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

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

Localisation updates from translatewiki.net.

Change-Id: Ib5e4b598df854da0bf279190047508a5f276eaab

Remove spaces in function signature

Change-Id: I45aea7a7af88cd913b2f485913620a8af0ab2fed

Merge "Fix type of list=tags&tgcontinue"

Merge "(Bug 45775) Adjusted the margin of "userloginForm""

Merge "Use gettype only for debugging text"

(Bug 45775) Adjusted the margin of "userloginForm"

Change-Id: Ibede75a3f4e998d1972cacf1f5840882d39550ee

Merge "Added ScopedCallback functions for early execution or cancellation."

Merge "[FileBackend] Replaced redundant "disposition" param with "headers"."

Merge "Cleanup for ApiImageRotate"

acessor for MediaWikiTestCase::$useTemporaryTables

useTemporaryTables is marked private to prevent childs from overriding
the value which would cause a lot of madness whenever it forgot to reset
its status.  But that in turns prevents a child class from figuring out
whether we use temporary tables or not. The ActiveAbstract extension
test suite has such a requirement and hence need a proper accessor.

MediaWikiTestCase::usesTemporaryTables() is a public accessor to for the
private property MediaWikiTestCase::$useTemporaryTables.

Change-Id: I8016c49116a187d7523cc0d6190514f8a513eafb

Add array with watchlist options to SpecialWatchlistQuery

This data is very useful with the hook as it holds the
criteria to build the watchlist query.

Change-Id: I688ab0b35b301f180bbfde3ad3043b1ee602de10

Merge "Move plural rules of Samogitian(sgs) to plurals-mediawiki.xml"

Remove error suppression from Parser::formatHeadings

The array index looks always initialized

Change-Id: I63a1c264ded0c4a744ee990de9bef4f8f9d20d1d

Move plural rules of Samogitian(sgs) to plurals-mediawiki.xml

* CLDR does not define plural rules for sgs.
* Port the plural rules present in LanguageSgs.php to CLDR plural
  definition syntax
* Remove LanguageSgs.php
* Update the tests, reorder/rename the plural form names

Change-Id: I44658402d69a6805cdfd189fe780eadee94056c7

Remove error suppression in Exif::isRational

preg_match throws warning, when getting a array as input, but there is a
is_array check, so no need for error suppression

Change-Id: Ie7f180f45c04e8e396186cd6a5a9e40e2c94a80a

Merge "Revert "Refactored use of $wgMemc in JobQueueDB to use a field.""

Revert "Refactored use of $wgMemc in JobQueueDB to use a field."

This reverts commit 7293699561f8751c66d55697e63b13a18653ac6b

Unit tests fail.

Change-Id: Iac8025844b37c584e451a4120e54898ae4f2524e

[LockManager] Refactor lock TTL code to base class.

* Also made the TTL configurable.

Change-Id: I713979267fe7bd92ac6cd5fe4d4168d20e539c75

Refactored use of $wgMemc in JobQueueDB to use a field.

Change-Id: Ic2a0a16142abdc5a971729fe550c10c1b8fa42b9

a43f751 removed the usage of $wgMiserMode

Change-Id: I5528dba582d218721324431015bd930b9b6ab57e

jquery.tablesorter: Fix explodeRowspans

$.tablesorter's explodeRowspans works ok for simple tables, but row
headers or unusual spanning structures will confuse it.

This rewrite makes it more robust.

Bug: 41889
Change-Id: Icb674f7eece053435ca9525d45709579df14cc74

jquery.tablesorter: Fix broken getElementSortKey()

Change I3e9898f3 introduced changes to
getElementText()/getElementSortKey() that broke certain behavior with
respect to tables having colspanned cells. Restore the old behavior for
these cases.

Change-Id: I18be2e6c1f0ab926ff4c795530600b52bc6488c0

(Bug 42461) Remove call to "new Database()"

 * Refactored LoadBalancer::reportConnectionError()
 * Removed PHP4-style reference from DatabaseError constructors
 * Allowed to not pass a database to the DBError family of exceptions.

Change-Id: If9570b45ea7285de2b8b2391e704bc01f76be48a

(bug 45054) Fix rsvg converter command line when thumbnailing.

Change-Id: Ia6c1d4e5d61629e9651495ee92fe1c7298a50ef1

Merge "Remove spaces between parentheses in function calls"

Merge "Set initial language and variant user preferences to user's preferred variant on user registration"

Localisation updates from translatewiki.net.

Change-Id: I36833fa0e34d515906005009fcfe7ffb7f59a3e9

Fix type of list=tags&tgcontinue

Was already interpretted as string before, but with a default of null,
this is easy to read and looks like all other 'continue' param

Change-Id: Ica972e927d4949bb0e8925f10471b9273c052011

Remove spaces between parentheses in function calls

Change-Id: If8ec665a01b566fa5189dfcc810b2d9a17f371e1

(bug 46239) jquery.makeCollapsible: don't fail if no options given

Also a typo elsewhere. That's what you get when coding at 2 AM.

Change-Id: I8dd4f0d06cc5dbf7fe67f2b584c9afe8f5d1b36a

Merge "Readd GENDER to english logentry messages"

Localisation updates from translatewiki.net.

Change-Id: Ie28bd2d1bce13b5a78eeb8573d8b5b9527b1afeb

Readd GENDER to english logentry messages

Was removed due to performance reason, but some fixed are applied in
meantime, so there should not be a problem on adding the GENDER back.

This is a (manually) partial revert of r110909

bug: 24156
Change-Id: I8139c51bb8a1ea4fa69a1327c46d347ea043cc9f

Special:RevisionDelete: DatabaseBase::makeList: empty input

bug:45211
Change-Id: I3777c1332aa2a0dd9e9cf4b643b53f31db3f5e91

Merge "Default $wgContentHandlerUseDB to true"

Check for rev_len not rev_sha1

Change-Id: Ida29357d1277ec1ddeada2aaca2d1b889ba742a8

(bug 44486) Rephrase password reset messages to remove 'reminder' language

Change-Id: Ie2a18ddb92b8c992bc72ea4263a43e6928681041

Merge "Cache Revision::selectFields() call result"

Cache Revision::selectFields() call result

Change-Id: Ia41208658b601f68e77a0e1fa36c13e28ea8b9be

Merge "jquery.makeCollapsible: add plainMode support"

Merge "jquery.makeCollapsible: add a few explicit options to the jQuery plugin"

Merge "jquery.makeCollapsible: move helper functions to the outer scope"

Merge "jquery.makeCollapsible: clean up the handler toggling logic"

jquery.makeCollapsible: add plainMode support

When .makeCollapsible() is used on a <ol>/<ul> list, the toggler is
inserted as the zeroth list item and clicking is causes all the other
list items to be hidden separately, one by one.

This hiding behavior is applied even when using custom togglers placed
outside of the list. This has some pretty serious performance
considerations when applied to large lists or tables (more than
~25 items/rows).

Enter plainMode. In combination with custom togglers, it makes all
types of elements to be treated the same and hidden/shown as a whole.

Change-Id: Idd9429b20d70741af72657feef26e1c95c76d51f

jquery.makeCollapsible: add a few explicit options to the jQuery plugin

Add a few options to the jQuery plugin to allow for explicit passing
of various options in addition to the conventions used now (via data-
attributes or magic id/class combinations).

Change-Id: Ic498e7a3b2d2e964bda8d31a70f71cda08560cec

jquery.makeCollapsible: move helper functions to the outer scope

They do not directly use any variables defined inside
$.fn.makeCollapsible (although they used to), so let's move them to
the appropriate place.

Also indent the body of the immediately-invoked wrapper function.

Change-Id: I3d5f33b4a3ed8bd72c5f55c4d9fa545e4f3bd8c4

jquery.makeCollapsible: clean up the handler toggling logic

Merged three functions which did almost the same thing into one,
togglingHandler(), which takes a few options instead.

Left the old functions as wrappers for now, I'll clean them up in
another commit.

Change-Id: Id3f457a84064382f2d962e9ac4d926827de93126

Add id for domain list on user create form

The user login form already has this id for the domain list, so
I'm adding it for the create form as well.

Change-Id: I1eec31c461a5c76565acf1e0bbfa07f367591678

Use gettype only for debugging text

Changed some gettype == 'array', 'object' and similar to is_array,
is_object or similar

Output of gettype must not stable across versions and it is slow

Change-Id: I07bfc063b03be1200989dd6facee66b35ab51d77

Merge "Make update.php and install.php use wfPHPVersionError() and reorganise it"

Merge "Fixed importScopedSession() and moved exportUserSession() to RequestContext."

Fixed importScopedSession() and moved exportUserSession() to RequestContext.

* Renamed WebRequest::exportUserSession -> RequestContext::exportSession.
  Updated the only callers of this new function.
* Init the user with User::newFromId() instead of relying on the session
  (which breaks when things like CentralAuth are enabled).
* Made RequestContext::exportSession() include the user ID.
* Removed now-redundant user ID checks in upload jobs.
* Added unit tests for the session import function.

Change-Id: I543e6766f7a8a828ea5d270328c3bc7738c6fe94

Localisation updates from translatewiki.net.

Change-Id: I66bc1bec2e64fef9dee6562c9e2a7268c8cdb1cc

(bug 45436) Update autonym of the Pali (pi) language

Change-Id: I078eb210eacb0f4c89a8858636bd5b84da68e79d

Merge "API: Remove useless PARAM_REQUIRED = false"

$.tablesorter: remove and inline getTextFromRowAndCellIndex()

It introduced unnecessary indirection and was only called once.

Change-Id: I519cc1f04a581d41aa23229182c55466ed7757c8

Cleanup for ApiImageRotate

Follows up Id15a92d19cda8256917e7e1e5ee4241012214102

Minor tweak to filerevert too

Change-Id: I742c854ebb97a5a1f954388be4b15f992fa93bd8