Provide RequestContext from fixDoubleRedirects

Lots of code depends on RC having the title.  I think setting it to
titleA is correct here, but maybe it should be titleB.

Bug: 62966
Change-Id: I875e1bf8f6d3468ea78c5a557a07cb6384965514

Merge "Sync up with Parsoid parserTests."

Avoid header notice log spam from RunJobs API

* Moved ApiRunJobs to a special page instead of going through
  ApiMain and having to fight the logic there. As a separate
  internal API, this does not show up on the API help page and
  is no longer effected by $wgEnableAPI.

bug: 62233
Change-Id: I1db6f526d02e130a66ee03289858a734d89e6c00

Sync up with Parsoid parserTests.

This now aligns with Parsoid commit 52b4e410acae520c1ecaed76efe062945b123187

Change-Id: I36ad89667817a94720fc0674cb39a5802d2c026c

Merge "Added a __sleep() method to ParserOutput to exclude cruft"

Localisation updates from https://translatewiki.net.

Change-Id: Id6221ab7d0d906278a9163513ca63b74c9d41645

Added a __sleep() method to ParserOutput to exclude cruft

Change-Id: I1a3c45288bf8b29c27277a3e70362ddc36713538

Merge "Call ignore_user_abort() right before committing to the DB"

Merge "Fix Status::getMessage accidentially returning string instead of Message"

Parser tests: Make Foobar.svg consistent.

The parserTest.inc file created metadata saying that Foobar.svg was
240x180px, but created an empty SVG file; the DEFAULT_WIDTH and
DEFAULT_HEIGHT in includes/media/SVGMetadataExtractor.php would have
caused this to be treated as a 512x512px image.

In NewParserTest.php, a 200x200px image was created for Foobar.svg.

That caused inconsistent and confusing results for SVG-related parser
tests, depending on which of the testing frameworks you used.

Fixed both of these to use a consistent 240x180px image, since
non-square images are better for checking correct scaling.
(Parsoid has always used a 240x180px size for Foobar.svg).
The non-square image has caused three parser test results to
slightly change.

Change-Id: Ib60a7412d9be808a0995e94d3aa373f2c5ca9bad

Merge "Moved ProcessCacheLRU to /libs"

Merge "Put SiteList navigation ids into cache"

Merge "Moved $wgQueryPages stuff out of the global scope and into a function"

Moved $wgQueryPages stuff out of the global scope and into a function

Change-Id: Ica034bdad89133bfce0b4238d62ed00865936644

Inprove the exception message in JobSpecification::validateParams

Change-Id: Ifcaa5e89e6b5dd4d89ca26203400082bff6fd85a

Put SiteList navigation ids into cache

These are ids used by extension(s) such as Wikibase
to add language links in the sidebar.

Bug: 62754
Change-Id: Ie21fc3ea500bd7de3c01a9d8ec05373f016f339a

Add user friendlier display text to links

Follow-up to 782afa6ef01940.

Change-Id: Ib6236fc453e8fbc68ae2f3dab2ddba4727a6051c

Merge "Method to identify if a ChangesList is watchlist or normal"

Merge "Send the full title to the 'nogomatch' debug log group"

Merge "Don't use isset() to check whether an existing variable is null"

Merge "Only put returnto parameter if needed on the from-http redirect in Special:Userlogin"

Fix profiling error in LocalisationCache::readJSONFile()

Follow-up I8d137e15e1 (6380e81).

Also added more blank lines for better readability.

Change-Id: Iae7921b017f81d5512e71384d7999502154c034c

Merge "Change date and decimal separators for Uzbek (uz)"

Change date and decimal separators for Uzbek (uz)

Change the date format dd mm to dd-mm and adding space instead
of commas for 3-digit grouping, decimal comma instead of
decimal point.

Bug: 46453
Change-Id: I1318b18f10a1b048c3574013a52775d5309f8abc

Merge "Refactor password validity checking"

Localisation updates from https://translatewiki.net.

Change-Id: I91e14444039b2b1345602a8c3291794ab3beafd1

Fix Status::getMessage accidentially returning string instead of Message

This mistake was introduced in commit
92e284d3fa62f45e20fed34c4359c575481d583c and the reason for the two
disabled tests. I did not enabled the second test because of an
unrelated problem. The first enabled test already covers the fix.

The method should return Message objects only but did return a
string in that special case (multiple warnings set but no context
message key).

Unfortunatelly Status::getMessage does have many, many more
problems but I understand it's not a good idea to address them all
in a single confusing patch.

Change-Id: I0dc37e248f407019d5921aaaca3eabba338b0fd3

Merge "Cleanup a bunch of tests and add todos"

Moved ProcessCacheLRU to /libs

Change-Id: I7052d04d9847f0310c1e62bd66365c813fddeab5

Merge "mediawiki.api: Use Promise.then instead of manual Deferred wrap"

mediawiki.api: Use Promise.then instead of manual Deferred wrap

This is exactly what Promise.then is built for.

As a nice bonus, this way the fail/reject handler is transferred
automatically, and we don't need to call resolve() manually
since the return value from then(Function done) is the resolved
value (or another promise).

Minor clean up:

* category: Fix type documentation (primitive string instead of String object).
* category: Use simple boolean check to assert value is no longer false.
* Use String() cast instead of explicitly calling .toString.
* parse: Fix bug where in theory the deferred would never be resolved if
   the condition evaluated to false.
* watch: Add missing @static.
* watch: Simplify params/$.extend logic.

Change-Id: Iedbc70b2573c4f6b0d9e133c6f31e8f0b19c6f5e

Merge "New 'profileerror' log group for profiling errors"

Merge "Move variable definitions near to where they are used"

Merge "Renamed /job to /jobqueue"

Only put returnto parameter if needed on the from-http redirect in Special:Userlogin

The default value of the $mReturnTo and $mReturnToQuery member variables is '',
even if they were not passed. This means that they would always be present
in the redirected URL when accessing Special:Userlogin from HTTP and HTTPS
is available.

Now the special case '' is changed to null so that they don't appear anymore
if not needed.

Change-Id: I49a085feaa49d4b2954ff280db241234d2d46258

Rename some local vars to start with a lowercase letter

Change-Id: I6e5975ed7351c1439eda19afaba5120c6afa50f1

Fix spacing between two functions

Added and removed some new lines to have one new line between two
functions

Change-Id: I1ccfbd575dd26b160396ef3d3e2e079f5cdbe196

Fix formatting of some php docs

php docs of functions should have newline in it and not in one line.

Change-Id: If4a29aefb46a7f73d75593814e597fdb792d5470

New 'profileerror' log group for profiling errors

Easier to catch them than if they are in the default debug log.

Added Profiler::debugGroup() as wrapper to wfDebugLog(), as
there already is Profiler::debug() for wfDebug(), so that
there won't be a fatal error if the error happens before the
inclusion of GlobalFunctions.php and converted other calls
to wfDebugLog() to use it.

Change-Id: Ie8481a2e13a94efa0248dd5a36b6b1a22811817e

Localisation updates from https://translatewiki.net.

Change-Id: If66ddc1478b986417b08bd6d84cf441ea7f4526c

Merge "Reapplying r45820 to support the NewSignupPage extension."

Merge "Use "elseif" instead of "else if" in PHP"

Merge "Put the "else" (or "elseif") on the same line as the previous closing brace"

Move variable definitions near to where they are used

In LoadBalancer::reuseConnection(), if the condition
"$serverIndex === null || $refCount === null" is met,
then those variables will not be used. So only define
them when they will readlly be used.

Change-Id: Ifbd4131f40d3babe733b8723d0d29d39890bb309

Mention possibility of $revision being null in PageContentSaveComplete

Lead to a fatal in WMF production, see I40c1412d

Change-Id: I39e05210e77f3dc0a1fcd995201a9c1ec9bc99f3

Fix variable name of param doc in RCCacheEntryFactory

Change-Id: I8039ab9ea117ba351061da71a92409e320d8eb84

Fixed typo: available

Change-Id: If4e28cf04699084ed325835d5ebe1c5900002040

Use "elseif" instead of "else if" in PHP

Per https://www.mediawiki.org/wiki/Manual:Coding_conventions/PHP#C_borrowings

Change-Id: I5e3a301c2f3c5ec6b965500c43d16022b8b82e59

Put the "else" (or "elseif") on the same line as the previous closing brace

Per https://www.mediawiki.org/wiki/Manual:Coding_conventions#Indenting_and_alignment

Change-Id: I208981db0a866524156bad18cb687f010afeac2c

Use Tab instead of Space as per PHP codesniffer

Change-Id: Iea84018c3dee45d5f062b41b1337af40e3de2fcf

Refactor password validity checking

Refactor the password checks to return a status object, so the function
can handle the entire error message, or return multiple error messages.

This patchset aims to keep the functionality identical. A followup
patchset can further improve the functionality. E.g., although
getPasswordValidity stated it could return an array of messages, it
never did so except from the hook, so most callers expect and handle a
single string.

Change-Id: I87644486f5572dc067ebdbacd01fb39c67e5612a

Merge "Split revdelete-text on Special:RevisionDelete for each type"

Merge "Execute LocalFilePurgeThumbnails hook before purging thumbnails"

Merge "Make MySQLi work with non-standard port"

Merge "Correct the destination of checkLastModified debug messages"

Correct the destination of checkLastModified debug messages

- OutputPage: if $wgCachePages is set to false, then it can be shown
  back to the user
- AjaxResponder: don't send back to the user (for consistency with the
  other calls to wfDebug(), and this can't be displayed to the user)

Change-Id: I17794016cfaef65ee3df3b82ceb8cb3a32ac7c67

Merge "Use preloadFileStat() to speed up syncFileBackend"

Merge "Add two new debug log groups"

Renamed /job to /jobqueue

Change-Id: I4c8a2b42140630838867c77a70d45ba14b5d95e2

Use preloadFileStat() to speed up syncFileBackend

Change-Id: Idd8e67090d7d5e7b6952fb46012981309475b902

Remove assignment in conditon in phpunit.php

Change-Id: I817af3cf5344d9ffe2a66048f6d32835712ba11c

Sync up with Parsoid parserTests.

This now aligns with Parsoid commit d6ebbd78101c6191e213e46838582d00f37c3bbb

Change-Id: I4fd04ac2bf4cd7a4fc195ad615e9940f4ddb29a5

Remove phpunit compatability function calls

phpunit.php now requires we run tests with
3.7.0+ so we no longer need these safety
calls

Change-Id: I15d215630fe4d8ab71fc97883dde2eed13d03678

Merge "Prevent "Log in" being used in two different contexts"

Emit $wgSearchType as JavaScript config variable

Allows for the search interface to be customzied for the particular
search-engine used, and allows collecting client-side performance measurements
that specify which search engine was used.

Change-Id: Ibeda834e9d5dbaf1d7e40c2dacbc60feb2cc4bba

Merge "Fix minor typo in parser tests error message."

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

New changes:
5c41645 Localisation updates from https://translatewiki.net.
c4651d5 readme: Use <br/> instead of <br> to satisfy gitblit's Markdown parser
5345806 Make it possible to configure LabeledElement to not use autoEllipsis
ac0cc69 Use boolean instead of bool in LabeledElement docs

Change-Id: I0e8bf836035402bc893c4dc58cb9d5441b7bd330

Merge "Update OOjs UI to v0.1.0-pre (46ccd5b3a7)"

Merge "Add prebodyhtml template variable"

Add prebodyhtml template variable

Allows the VectorBeta BetaFeature to add HTML underneath the heading

Change-Id: I6d9326d9e84a92b01dee2a29028208ef3f875767

Merge "Changing URLs of mediawiki.org in scripts to the SSL-based website"

Changing URLs of mediawiki.org in scripts to the SSL-based website

http://www.mediawiki.org --> https://www.mediawiki.org

Part 3

Change-Id: Ica633881b1744fa2854f4b012b79dbf5a7e5e7e2

Merge "Adjust default interwiki result text"

Fix minor typo in parser tests error message.

This was introduced in Ie4e68960ca7c352af495ebb59ba83488935a44c4.

Change-Id: I45352dd3dfd26acdcda147d8aa07be6f0d60698f

Merge "Allow more-descriptive section names for parserTests"

Merge "Vector: Make animation LESS component"

Merge "Adjust magic bolding in api help to only match lines inside *"

Adjust default interwiki result text

'Results from $1' sounds nicer than '$1 results' for sites that
haven't set the custom text.

Change-Id: I810c8e0503dde238ea8813b7cecb1a1dec927d3e

Merge "Title and redirect snippets don't have a parameter anymore"

Adjust magic bolding in api help to only match lines inside *

list=logevents&leaction has param values with *, which gets bold, when
there was two of them in the same line.

Bug: 61834
Change-Id: Idace9afd4f3a2dce9be539b209a02fa318df8f45

Method to identify if a ChangesList is watchlist or normal

There are differences, like grouping, between watchlist and
normal rendering that hook recipients need to know about.
This exposes if the setWatchlistDivs method has been called
which currently happens immediatly after instantiation in
SpecialWatchlist

Change-Id: Ibc06c6d9b878cad3f5da92cfbe3f650ad3f63efa

Title and redirect snippets don't have a parameter anymore

Change-Id: I5aae8caf14a5dd569042069fe64bc30736734cd8

Localisation updates from https://translatewiki.net.

Change-Id: I603eeeed9980a1190075cb54092edefdaf3c2ed5

Remove request_with_session/request_without_session from showCacheStats.php

Fix for I0ed1e87574 (42fcd43).

Since request_with_session/request_without_session were removed in the
mentionned commit, the maintenance/showCacheStats.php refuses to show
the other statistics.

Change-Id: I955cc537f8bc9116f759af5a5c3005aa7d60c83b

Execute LocalFilePurgeThumbnails hook before purging thumbnails

bug: 62273
Change-Id: I08ae902debb36039d2c0be829eec07a109e70a24

Remove $wgTitle from all maintenance scripts

Shouldn't be needed and aren't for any core operations. If any
extension relies on these $wgTitles being set in maintenance
environments those extensions are broken and should be fixed.

Change-Id: Ie02a5042ab96e155d783d56d5340dd0da8e3d55c

Update OOjs UI to v0.1.0-pre (46ccd5b3a7)

New changes:
9247e3a Localisation updates from https://translatewiki.net.
0a078dc Localisation updates from https://translatewiki.net.
3027b13 readme: Fix broken Markdown list syntax
d7ab327 Update oojs to v1.0.8
4dd7320 Drop file "OO.ui" file prefix
7c607c2 Localisation updates from https://translatewiki.net.
46ccd5b Make dialogs a little taller

Change-Id: I51caa22711fd10ff464b895e90a69292cbf003f8

Changing URLs of mediawiki.org in scripts to the SSL-based website

http://www.mediawiki.org --> https://www.mediawiki.org

Part 2

Change-Id: I3be61fe3dfb502cc20180486eb1a8016eac151df

Merge "Add --regenerate-all option"

Merge "Set a title for the context during import on the cli"

Call ignore_user_abort() right before committing to the DB

Change-Id: I48b9e90418c38c7c9970cf0739a877b5be3a2b64

Merge "HTMLForm: Support 'url' type"

Merge "API: Handle exceptions from ApiBeforeMain hook in a user-friendly manner"

Merge "HTMLForm: Regularize punctuation in messages"

API: Handle exceptions from ApiBeforeMain hook in a user-friendly manner

The immediate impetus behind this change is this series of events:

1. CirrusSearch hooks ApiBeforeMain to handle some setup that requires
   the User object.
2. So User is loaded from the session.
3. OAuth checks the headers as part of loading User.
4. OAuth sees that the headers are invalid, and since it was called from
   the API it throws a UsageException, expecting the API to catch it and
   return an appropriate response to the client.
5. But nothing does so, leading to an unhelpful "Internal Error" page
   being returned to the client.

We can do better than that.

Bug: 62312
Change-Id: Ib5735661eec6ebe57eaa69c67b399e703cc90fc4

Split revdelete-text on Special:RevisionDelete for each type

This allows specific text for each type.
Types here are: text, file and log

Change-Id: Icb24dfb483f940e90215ecc01f5765185963c7ed

HTMLForm: Support 'url' type

SecurePoll will have a field that's a url, so we may as well use the
correct HTML5 input type for it.

Change-Id: Id3ddd8f2efdff08c8a188089d321d143ce5ef9c9

HTMLForm: Regularize punctuation in messages

To make Siebrand happy.

Change-Id: I0f622e4cdd2e6d8b33eb1ff38f0d78f8eec5bce3

Merge "Fix spacing in SpecialRevisiondelete.php"

Fix spacing in SpecialRevisiondelete.php

Change-Id: I5ae28e7e7b937ad4b60b919c87858e373ce85757