Merge "Add GENDER support to protection null revision edit summary"

Add GENDER support to protection null revision edit summary

The edit summary for protection null revisions are still messy
"Lego messages", but now with GENDER support.

The message documentation (qqq) for "protectedarticle",
"modifiedarticleprotection", and "unprotectedarticle" was incorrect, but
now matches the fact that these are IRC-only messages.

Bug: T121091
Change-Id: Ia1151fb5aab8334d3351588531fcdac499fcc486

Merge "Disable magic links by default"

Merge "Don't cache replacement patterns in wfEscapeWikitext() during tests"

Merge "Add tracking categories when magic links are used"

Disable magic links by default

And add release notes for all the magic links changes in 1.28.

Bug: T147536
Change-Id: I66b2c4424b17cc86bbd7339de588f1c00cd2f18d

Don't cache replacement patterns in wfEscapeWikitext() during tests

Parser tests depend upon being able to change the value of
$wgEnableMagicLinks at runtime, so skip caching the replacement patterns
while parser tests or phpunit tests are being run.

Change-Id: Ic9fe4fe898aa353bdefa2bf8207b77d2ac14887f

Add tracking categories when magic links are used

This adds 3 tracking categories, one for each type of magic link (ISBN,
RFC, PMID). This will allow wikis to gauge usage and identify pages that
need migrating.

These will only show up if the respective magic links are enabled via
$wgEnableMagicLinks.

Change-Id: Ic483f0c493112bf6373e1b37961e1241c20c3582

installer: Make help link in sidebar same as default Main Page one

Point to www.mediawiki.org instead of meta.wikimedia.org.

Bug: T148004
Change-Id: I1bc8a84773e69529aed719ce36f1196f12b56226

Make LBFactorySingle external LB methods throw exceptions

There are no external clusters with this LB factory class,
so calling these methods makes no sense.

Change-Id: I63a9c6c76ba036980cfaab3ef03574295e46c6bf

Merge "Break long lines in OutputPage::sendCacheControl()"

Localisation updates from https://translatewiki.net.

Change-Id: Ia3c53700406b070077e3f2381c7735d54e545c9b

Break long lines in OutputPage::sendCacheControl()

Change-Id: I72d494f4c97aef9d500bb5d7b36dad30a44f501a

Merge "htmlform: Tighten CSS rules for .mw-htmlform-flatlist"

Merge "Don't fatal on invalid timestamps"

Merge "Improve getUrlDomainDistance() return values"

Merge "DatabaseMysqlBase visibility cleanups"

Don't fatal on invalid timestamps

Database::timestamp() intentionally lets errors bubble up, so catch
invalid user-provided timestamps in
ReverseChronologicalPage::getDateCond(), and avoid using an offset in
that case.

Bug: T149257
Change-Id: Ida85eb44b66e8a0166e7f68a101ff094e04b1c8e

htmlform: Tighten CSS rules for .mw-htmlform-flatlist

They accidentally applied to a MenuOptionWidget in autocompletion dropdowns, too.

Change-Id: I54d0988e78b65e0fb922ef353626453cce56e474

Add hooks for WatchedItemQueryService / ApiQueryWatchlist

In order for an extension to add data to ApiQueryWatchlist, we need to
provide a way to allow it to manipulate the database query made by
WatchedItemQueryService. We also need some hooks in ApiQueryWatchlist to
handle the marshalling of data to and from WatchedItemQueryService.

To better handle hooking, this also moves some of the continuation logic
from ApiQueryWatchlist to WatchedItemQueryService.

Bug: T147939
Change-Id: Ie45376980f92da964a579887b28175c00fd8f57e

DatabaseMysqlBase visibility cleanups

Change-Id: I015fc37a5525a31fb779ba841a1966675ebc494d

Improve getUrlDomainDistance() return values

This change is just cosmetic given the caller it has.

Change-Id: I591f92e56d9f5574d032dda380bc03b491b10738

Merge "resourceloader: Optimise startup by merging regexes"

Merge "Use Database::addQuotes in Special:WantedPages/UnwatchedPages"

Use Database::addQuotes in Special:WantedPages/UnwatchedPages

Database::addQuotes uses the correct quotes/apos, hopefully together
with numeric values

Change-Id: Ia3aba8cabf3ae877f56185e5724dca204a19f73a

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

Localisation updates from https://translatewiki.net.

Change-Id: Ic0c342a71a911ed1c68e48212769d6c462ed946f

Merge "Follow-up Id0021594: Remove extra code for redlink suggestions"

Merge "Revert "Parser functions now format numbers according to page language""

Merge "Restore default $defaultDateFormat in sk (Slovak)"

Merge "mw.loader.store: Count modules which fail to execute, too"

Revert "Parser functions now format numbers according to page language"

This reverts commit 4290f686c07265d40718fc3358f196de41bbde57.

Bug: T149840
Change-Id: I7e2dfe14813b510a1a974ae0007390bb2879f804

Restore default $defaultDateFormat in sk (Slovak)

Follow-up to b321f48d80db27ecff3d9d88c6881c63f27b86c0
As spotted in https://sk.wikipedia.org/?diff=6358764 , cs has
$defaultDateFormat = 'ČSN basic dt' which is not defined in sk.
Restore the "dmy" default which sk used to inherit from en and
which is defined below.

Change-Id: I1c7e7f71587c95090f3bb0bfe58eb013485970bd

mw.loader.store: Count modules which fail to execute, too

"Failed" will probably always be either zero or one, because it is
unlikely the asyncEval error callback can ever be called more than once.
But the stats will let us verify this, and will give us an indication of
how often errors happen in general.

Change-Id: I6dea28d84bbca0f02ad8c5bf9a8e9d976622d815

Merge "CSSMin: Correctly avoid fallbacks when embedding SVG files"

Merge "SpecialActiveUsers: Change checkboxes to dropdown"

SpecialActiveUsers: Change checkboxes to dropdown

Moved form to new function and
fixed text about cached version.

Bug: T116354
Change-Id: I29ae63472536f99f7f9546f30d86e7bd324d3094

Revert "EditPage: Show EditFilterMergedContent hook errors in an errorbox"

This reverts commit a02a7ff8eab9d589d77c03d4b4c58fc4a05e65c8.

Bug: T149473
Change-Id: I35a650c6ec478542351824e8ae8a8d8354ffa50f

Merge "Revert "Gallery: Use intrinsic width for gallery to center caption""

Revert "Gallery: Use intrinsic width for gallery to center caption"

Resulted in unforeseen changes to galleries using mode=packed
(and other modes other than traditional).

This reverts commit e6dd624baeca16efe462e3938969a72c9ef4ed1e.

Bug: T149412
Change-Id: I57dabbd3b5c0e52f3fad1b2144764bcc2fdecda0

Merge "Check MW version requires before manifest_version"

Merge "Add PHP DateInputWidget"

CSSMin: Correctly avoid fallbacks when embedding SVG files

The check for file existence before checking its MIME type was
building the path incorrectly, causing this piece of code to always
think the file does not exist. (The code to actually embed the file
was correct.)

This was not caught by the tests before of a bug in the test code,
which cancelled out this one.

Bug: T100958
Change-Id: I4469079f41d48052d0a02f98bb1ed02407abd54c

mediawiki.util: Fix replacement of $ signs in mw.util.getUrl

Bug: T149767
Change-Id: I91c1afa1f93966d1f01793d124fe8f2ae3dab0cf

WatchedItemStore: Update process cache when adding items

This way, when submitting POST to action=watch, the response will
not show "Unwatched" in the navigation 'content action' tabs.

This was previously the case on any wiki installation that has
more than 1 database as the skin would call User::isWatched(),
which would query stale data from a slave.

This doesn't yet account for the reverse use case (removing from
watchlist). That's to be addressed in a later commit.

Bug: T28292
Change-Id: Ie4b69c985815a77b70692db0c4dbf52e1a6a018d

Check MW version requires before manifest_version

Bug: T149751
Change-Id: I914be4da431dcd613bdf2d1fd5ba9f28cf5b936b

Add PHP DateInputWidget

Bug: T120733
Change-Id: I8f80e27304a916bba16ed255c8e78073f9da3813

Merge "Don't parse <nowiki><span class="error"></nowiki>"

Localisation updates from https://translatewiki.net.

Change-Id: I21d3b38d7ec6af6e2a8e6baa6092a6810eab1820

Merge "DBError expects IDatabase|null as its first argument"

Merge "Parser test file editor"

DBError expects IDatabase|null as its first argument

I came across LBFactory.php:357 while testing my
master/replica db setup. It was not configured correctly
and I got "Argument 1 passed to DBExpectedError::__construct()
must implement interface IDatabase, string given" instead
of the actual error.

Change-Id: I949e1803f84459f8bb83e957a88d1e927cad16f7

Merge "Document diff/DifferenceEngine.php function"

Don't parse <nowiki><span class="error"></nowiki>

Which means we can't check if a parser limit was exceeded while trying
to expand the content of a tag, but that's probably not a huge loss.
It'll just result in potentially strange output rather than an exception.

Bug: T149622
Change-Id: I7910dfa0f61b1cc9168c7ed1498b2bda27c47f0e

Follow-up Id0021594: Remove extra code for redlink suggestions

We do a second query now instead which handles this by itself.

Move the description stuff to be generic for all redlinks

Bug: T149130
Change-Id: Iddc542d2aa02fd14a88014c78bd5a18e1f39f37e

Localisation updates from https://translatewiki.net.

Change-Id: I3f98e468088f53f688b357ed1ebf7a36c0df0f0b

Merge "Rename JobQueueDB::getSlaveDB() to getReplicaDB()"

API: Error if sensitive AuthManager parameters are in the query string

There was only 1 hit in the feature usage log in the past 30 days for
this code path, so this should be good to go.

ApiLogin, on the other hand, received 27048 hits over the same time
period. So let's not do that one just yet.

Change-Id: I1ae3b928fda9ddc94c8182155637920713dd404d

Merge "Allow empty href in SVGs"

Remove spaces after cast operators

This fixes the outstanding mis-spaced cast operators to bring them
into line with the coding standards on mediawiki.org (and with the
more common usage within this codebase).

Bug: T149545
Change-Id: Ib7bcf95bbee83d20c05f6d621ce7b4e1fb58a347

Allow empty href in SVGs

bawolff is right, this shouldn't harm anything.

Bug: T149549
Change-Id: I377efdee7478940154ef5ec921bd0c8f7ec0c110

Merge "FormatMetadata: Do not format 'UserComment' as a number"

FormatMetadata: Do not format 'UserComment' as a number

It's a freeform text field.

Change-Id: Ic3150f48c2e38fa26728bc22017b992d8ddfd152

Merge "Don't show warning header for non-standard workflows on login-continue"

Merge "Add firstLetter data for ~50 additional languages"

Document diff/DifferenceEngine.php function

Change-Id: I999824d8ce569de93fe4c9a889edea582d07611c

Localisation updates from https://translatewiki.net.

Change-Id: I695ddf72027c55d160192f0ee4e88d6738a13fa6

Merge "Make determineCoreSchema a public function again in DatabasePostgres"

Make determineCoreSchema a public function again in DatabasePostgres

It was changed to a private function today, but it is called in
PostgresInstaller so it is causing this error

PHP Fatal error:  Call to private method
DatabasePostgres::determineCoreSchema() from context 'PostgresInstaller'
in /home/travis/build/paladox/mw/includes/installer/PostgresInstaller.php
on line 515

Caused by I2a067ca89a03f9ebf3f70a4f36ddae92e5b1e468

Change-Id: Ifac5a5c7dfeceeb8c7827d6d70e757633df77ca5

Merge "Move LinkBatch operations for QueryPage to a helper function"

Move LinkBatch operations for QueryPage to a helper function

This is easier to maintain as duplicated code everywhere.

This change also adds LinkBatch operations to specialpages, that does
not had it so far, but where it's useful.

Change-Id: I4936ecfb2be3a7709d99570c0561ab0cc9cd754f

Add missing Database::timestamp() calls

This was broken for Postgres installations

Change-Id: I0c1d442e3cf1d9a4d5359e7a8f8fd02a31d1ac65

Merge "Fix missing commit() flag in postgres savepoint class"

Merge "Add LinkBatch to Special:UnwatchedPages"

Localisation updates from https://translatewiki.net.

Change-Id: Ia735ef197567ad5151b508bc3b2851610f9b19a4

Add LinkBatch to Special:UnwatchedPages

Get all the information for blue links in one query.

Change-Id: I18cbaf69c5f75e00e48e27d4b5e7220f51c31808

Merge "Make LBFactoryTest.php pass for sqlite"

Fix missing commit() flag in postgres savepoint class

Previously, the transaction was left dangling, causing massive
unit failure.

Change-Id: I8fb6e69b44ff0ceb0cf7c4fb45f204e140472406

Don't show warning header for non-standard workflows on login-continue

When a user needs/wants to login again (reauthentication or user account switch),
the Login special page shows a header to explain what happened ("already logged
in" or "need to re-authenticate"). For now, the form also displays this header
for a continued login workflow (e.g. when the user needs to enter secondary auth-
entication data, like a one time password). This is misleading, as the user already
knows what and why he has to login (again). Other error/warning messages are maybe
not as visible as they should be.

This change omits the header message for the login-continue auth action.

Bug: T146317
Change-Id: I2115bbe7965e8d0326f71872a4d33767dd0f956b

Add firstLetter data for ~50 additional languages

Based on CLDR 29 data files.

This did the relatively easy languages in CLDR 29 (Which is most
of them). I skipped languages with complicated tailoring files.

Change-Id: I8367604f7d3a1cdef9cb4e15813893c8cbfff1ff

Rename JobQueueDB::getSlaveDB() to getReplicaDB()

Change-Id: I7a88c4876e76836c92cc0b45b4e72124a087121c

Make LBFactoryTest.php pass for sqlite

Change-Id: I4fe929e82218231f6c8afa64da8c0ccb42d2c362

Various small cleanups to DatabasePostgres

* Add missing method visibilites
* Removed redundant doc blocks
* Use empty string for mSchema for consistency with
  the base class

Change-Id: I2a067ca89a03f9ebf3f70a4f36ddae92e5b1e468

Merge "Postgres updater fixes to make update.php able to run"

Postgres updater fixes to make update.php able to run

* Remove redundant schema prefix from relname=x query. The
  schema filtering is already done via the JOIN. The relname
  portion is just the table name not <schema>.<table name>.
* Avoid explicit table schema qualification and rely on the
  search path, as MW 1.27 did. Previously it only used the
  global $wgDBschema var to pass to determineCoreSchema()
  instead of keeping it in mSchema.
* Clean up some code duplication in Database::tableName() and
  make the code comments clearer.
* Make DatabasePostgres::tableName() use parent::tableName()
  instead of a method that just wraps this method. The intent
  seems clearer this way.
* Remove unused return value in
  PostgresUpdater::rebuildTextSearch().

Bug: T148628
Change-Id: Id11d9576b7c2fdad22ff7f90727c12997217a632

resourceloader: Optimise startup by merging regexes

Merge webOS, misc and Google Glass regexes.

* Add various test cases for PlayStation.
* Add tests for unrelated user agent matching "Glass".

Change-Id: Ifb0944d190f230bb36197b22cf3099c187dad091

Merge "Make updateCategoryCounts() have better lag checks"

Localisation updates from https://translatewiki.net.

Change-Id: Ic7e8cf96d2bd8ca7a583824eea1ea4a3ed6429ac

Merge "Add Parser to MediaWikiServices"

Merge "Introduce InterwikiLookupAdapter on top of SiteLookup"

Merge "Pass context to ChangeTags::tagDescription"

Add space between label and selector

Change-Id: Id5c8c857a2a1339b0ce626bac98f0869e27d5a75

Parser test file editor

Add editTests.php, which provides an interactive interface to the
parser tests, with semi-automated editing.

Change-Id: I1a20d007ba4627d562a16c03849bbad7aec0e516

Merge "PHPSessionHandler: Workaround PHP5 bug"

Merge "Change .tocnumber to use grey text for section numbers in Table of Contents boxes"

Pass context to ChangeTags::tagDescription

To avoid wfMessage calls.

Change-Id: Icaa984fa8c69ccb38fbfd3e30877a4c572501c28

Merge "Refactor rev_delete handling on deletion"

Localisation updates from https://translatewiki.net.

Change-Id: I83ef5fa88f65ec941b87888943273892857d5561

Remove tracking category stuff that accidentally slipped into 61adc1e14

Bug: T149310
Change-Id: I0a3725a72b1467c57280ae1880935dd5fa54ae9e

Refactor rev_delete handling on deletion

Refactoring handling of the rev_delete database field, which is not
needed to select, when the suppress option is set.

Also added a new const, to avoid a comment about that things should be
15 ...

Change-Id: Ia5bcb835172bf6c10e2e872555fc67e91806ebe1