Merge "Tests for WikiPage::insertRedirectEntry"

Merge "RCLFilters: Show images and descriptions with page suggestions"

Merge "ImagePage: Make metadata table's initial collapse CSS-only"

Merge "Absolutely position "legend" on grouped results"

Merge "Warn for uploads with new name but same content as local file"

Merge "ImagePage: Clean up metadata.js collapsing"

RCLFilters: Show images and descriptions with page suggestions

Bug: T172161
Change-Id: I1a5501eefdb721367436db50be0787c8f6a78d47

Merge "RCFilters: Update invert URL param when toggled"

Merge "RCLFilters: convert related changes tool to new UX"

Merge "RCFilters: Clarify 'hidden' and 'sticky' filters"

Merge "RCFilters: Make 'days' and 'limit' sticky"

Tests for WikiPage::insertRedirectEntry

Bug: T180989
Change-Id: I677202b22594c7fc8fcf97b805ae69726945f96d

ImagePage: Make metadata table's initial collapse CSS-only

Make the initial collapse of the metadata table not depend on
JavaScript. This eliminates the FOUC, and will also reduce JS
footprint once compat can be removed.

* Move the 'display: none;' styles from metadata.css to
  filepage.css so that they are part of the base styles for
  file pages (metadata.css is only for supporting metadata.js,
  which is loaded dynamically).

* Apply "collapsed" class from the PHP side immediately,
  instead of calling "addClass" in JavaScript.

* Restrict hide-styles to only apply under .client-js to make
  sure the rows remain visible in no-js mode.

* Declare dependency between metadata.js and filepage.css.
  This is just for documentation purposes, as they are both
  already separately loaded on all file pages.

Change-Id: If22bf7acb47b59151dc3b0843a62507c100e548b

ImagePage: Clean up metadata.js collapsing

In preparation for trying out a CSS-only approach for the initial
collapse that doesn't cause the expanded view to flash first.

Bug: T182047
Change-Id: I42941ccb05d5525599a0209f0f33db9fe836580b

RCFilters: Update invert URL param when toggled

Bug: T182045
Change-Id: Ia9a8afb336305969ac8b5c2e7ffab146844ad9ee

Merge "API: Account for PHP 7.2 change"

Localisation updates from https://translatewiki.net.

Change-Id: Ib8affd4b492c3be56458b54d28e4c02cb7ecfc9c

RCLFilters: convert related changes tool to new UX

Bug: T172161
Change-Id: I96af7ba583d03e6ff9833ac3b5f4b80cfd0ee626

Merge "A few doc comment fixups"

Merge "Test for WikiPage::updateRedirectOn"

Merge "Add test for WikiPage::updateCategoryCounts"

Merge "Refactor test for WikiPage::doRollback"

Merge "WikiPage tests @covers for doDeleteArticleReal & commitRollback"

Merge "Split WikiPageTest into ContentHandler & NoContentHandler tests"

API: Account for PHP 7.2 change

PHP 7.2 broke existing functionality in making count( null ) raise a
warning. So add tests for null all over the place, or change tests where
we know the value is null or an array (but not false, empty-string, or
0) to just cast to boolean.

Bug: T182004
Change-Id: Idfe23a07daa9f60eee72f2daf04304be87057a29

Merge "objectcache: various cleanups to WANObjectCache"

Merge "Replace some Special:Undelete button with OOUI ones"

Replace some Special:Undelete button with OOUI ones

Bug: T180207
Change-Id: Icb550a6ad8f90d7718441ec17a1026a2cfb9d754

RCFilters: Clarify 'hidden' and 'sticky' filters

Clarify the meaning and behavior of sticky and hidden filters
according to the following definitions:

When 'sticky' is false, the filter is saved in Saved Queries. Its
state is considered and altered by the 'Clear all filters' and
'Restore default filters' buttons.

When 'sticky' is true, the filter state is only changed using its
associated control (i.e. Number of days selector).

'hidden' simply means it is not shown as a capsule in the active
filters area.

Change-Id: I7ccb7c202e21917eb288055c509c9f904a0d01bb

Merge "Only send statsd data for WAN cache in non-CLI mode"

Merge "Remove unuseful debug messages"

Absolutely position "legend" on grouped results

- When results are grouped by page, on pages where new RC filters
are enabled, apply position: absolute to legend, to avoid top
result entries being pushed too much.

Bug: T174728
Change-Id: Ifecaaff5727905b3c4e860555ba094f71aac6f80

Test for WikiPage::updateRedirectOn

Bug: T180989
Change-Id: I798ded72142503e52395b45fe9ec71a215d5ab92

Remove unuseful debug messages

I spotted one of these in my debug log. Without context it is not
useful nor informative. Given it can actually come from three places,
it is useless.

Change-Id: Ic293d8944e9a1a8152c5a5e0b695c5ddc07ee6aa

Add test for WikiPage::updateCategoryCounts

Bug: T180989
Change-Id: Iff4bb885e5d385a7b3ec6169ee9a3e1e82d7d993

Refactor test for WikiPage::doRollback

This test was not running and a comment said
it was broken.
This patch refactors the test.

Bug: T180989
Change-Id: I46b73371e5fd65860fce6cc101bad6abfa20e91d

WikiPage tests @covers for doDeleteArticleReal & commitRollback

Bug: T180989
Change-Id: I70dcad9d527be4d91a88a14fe9787c53a7cf7d2f

Split WikiPageTest into ContentHandler & NoContentHandler tests

Follow up to Iccdcd4a4f72c8b1329d36ff2e8699ba6b23c9ae7

Bug: T180989
Change-Id: Ib37a058586c7222ffe3ee5dc5f0e134ee18f0ff7

Fix phpdoc for WikiPage::updateRedirectOn $redirectTitle param

Change-Id: I8e666db2a7502bd1773252aa71db47819f63f08b

A few doc comment fixups

* Remove some creation dates, they are not protected by GPL
* Remove duplicate @defgroup API
* Remove @ingroup from some @file doc comments on class files. It is not
  useful to list class files alongside classes in the doxygen module menu.
  Add @ingroup to some more class files that had @ingroup on their file,
  that was probably the author's intent.
* In PackedOverlayImageGallery, use the file comment as a class comment
* Don't put @defgroup and @file in the same comment. @defgroup makes the
  whole doc comment describe the group.
* Instead of putting AnsiTermColorer in two groups, use hierarchical
  groups.

Change-Id: If54f6e0b2bc1ea6de42045885cf836ee67b8e961

Localisation updates from https://translatewiki.net.

Change-Id: Id5fe73a56a251d6feafca45edba583532180bd4c

Show protection log on every use of moveddeleted-notice

As a follow-up to T171338, the other uses of the message were ignored.
This change adds the protection log to these uses; that is, history and
image pages.

Bug: T174889
Change-Id: Ib9f22d6e06c372c4a32e77a08c846d768853bc21

.gitignore: Exclude Extension:Score generated files

Change-Id: Ic9cd0051cc8bd241b933037466a2af76d0f6a287

Merge "xmp: Lower severity of XMP parse failure log events"

Lower DatabaseMysqlBase::lock logging to INFO

Whether the lock() acquisition failing is a huge problem depends
on what the caller is doing. Let the caller do any logging it needs.

Bug: T180793
Change-Id: I3d05138d312d8b973df153bb511e69619d663c9d

xmp: Lower severity of XMP parse failure log events

Errors when parsing XMP information blocks broadly indicate that the
file is somehow malformed. This is generally a content issue rather than
an operational issue.

Bug: T118799
Change-Id: I3c7992343f28db75cd9ec0701c1eceebf6db2d08

Revert "Special:Preferences: Use OOjs UI" and follow-ups

The number of issues with the new interface is unacceptable and we
will not be able to fix them reasonably quickly. See subtasks of
T180538 for the list of issues, raised both by the Wikimedia community
and by WMF employees.

I should have pushed back harder against the merging of this half-baked
change with the promise that we'll fix it later. I convinced myself
that the regressions were not so noticeable and that the issues that
were pointed out will in fact be fixed by someone. Predictably,
however, regressions were bad and the only person fixing the issues
was me.

I am not going to work nights to make this page decent again within a
reasonable timeframe; I'm not sure if I'd even be able to since many
issues are problems with the design rather than the implementation. No
one else seems to be working on improving it, therefore I am reverting
the change.

On the bright side, this work has resulted in a number of improvements
to HTMLForm and Preferences code, which are not being reverted here:
<https://gerrit.wikimedia.org/r/#/q/topic:T117781>.

If anyone reattempts this, I recommend gating the new interface behind
a configuration variable and URL parameter, like we did with
$wgOOUIEditPage in the past, and testing thoroughly in production
before enabling it for everyone.

* Revert "Special:Preferences: Use OOjs UI"
  This reverts commit 486e566cfef612de6773df435a74d5fc37e27174.
* Revert "Preferences: Show preview of edit fonts in edit font selector"
  This reverts commit 6634ff729dc7f6d0f541639dbdf3c4d9f786ddf6.
* Revert "Follow-Up Iae63b6994: Add missing editfont dependency"
  This reverts commit ce42fdf151c39f91cf4077673219fa6228a54d7f.
* Revert "Preferences: Improve visual appearance by “unboxing” sections"
  This reverts commit c9415bb0059f4dae4abc5e39a8af844328333120.
* Revert "Remove box-shadow from preference panels for ooui-apex"
  This reverts commit a934b82ca27971e8f0553a7ab7c8ee30fccf3283.
* Revert "Preferences: Don't show the watchlist token; just link to ResetTokens"
  This reverts commit e8c9102fc7b66460c33f643b0dea7190cb89ac83.
* Revert "mw.special.preferences: Make the "Basic information" section more compact"
  This reverts commit d48b7260f30c1ec57046a1f8e4c82057aed45e5f.
* Revert "mw.special.preferences: Widen the dropdown of the "Time zone" field"
  This reverts commit afd5f1417efdd463d86186257c4ec77dd4a442ce.

Bug: T117781
Bug: T180538
Change-Id: I44b5daea1828f71881b5bd35218f5ecb7ab7f36e

Localisation updates from https://translatewiki.net.

Change-Id: I1995cac694d9093d8356fd122b92f44e86bb643e

Only send statsd data for WAN cache in non-CLI mode

Bug: T181385
Change-Id: Ic51a084206cea93621393359ded4ff77beef5f13

Merge "Try to avoid bogus JobTest failures for PHP 7"

Add GENDER where strings could differ

Bug: T129731
Change-Id: If1100a5c9ec4882f96237c70e5270a9aa40e6b24

Merge "Fix double escaping for 'userlogin-createanother', 'userlogin-joinproject'"

Fix double escaping for 'userlogin-createanother', 'userlogin-joinproject'

Change-Id: I6773420cf656e0bf47897198251e43917e1d57c6

Try to avoid bogus JobTest failures for PHP 7

Change-Id: I07b4bd796a668fef83ebb3e2824ccc608781a6e8

objectcache: various cleanups to WANObjectCache

Change-Id: Ibc3c7e59b47b4f9b982176f81dff9f81d99de460

Merge "objectcache: add example use case to WANObjectCache::getMultiCheckKeyTime()"

Merge "objectcache: Fix HashBagOStuffTest test in PHP 7.1"

objectcache: add example use case to WANObjectCache::getMultiCheckKeyTime()

Change-Id: I874070d2f770be7f9e06c381b2413f6e133418b8

Merge "objectcache: add WANObjectCache::getMultiCheckKeyTime method"

Merge "build: Add PHP 7.1 to Travis CI test matrix"

objectcache: add WANObjectCache::getMultiCheckKeyTime method

Change-Id: I5675fabc5aa70f72659ce02d68caae88be20e06d

build: Add PHP 7.1 to Travis CI test matrix

Looks like we've got at least one test failure at the moment
(being fixed in Ibb1a59e373740772). Let's use Travis CI to also
check if there are other failures behind it.

Change-Id: Iec9857ee04b4811e912f2f43aed1e5a9fce0a2c4

Merge "Changes for Mirandese (mwl)"

objectcache: Fix HashBagOStuffTest test in PHP 7.1

> There was 1 error:
>
> 1) HashBagOStuffTest::testEvictionAdd
> A non-numeric value encountered
>
> tests/phpunit/includes/libs/objectcache/HashBagOStuffTest.php:106

This warning is new in PHP 7.1, however the behaviour is not new.
This code was already not behaving as it should have. Concatenation
preceeds addition/multiplication. As such, this was producing
(int)-10 each time instead of (str)"key0" through (str)"key10".

Change-Id: Ibb1a59e373740772f02dfec77ee7ebd9d181d852

Localisation updates from https://translatewiki.net.

Change-Id: I67c7200d7062e685c0d3eac069f6faf70f745f25

Changes for Mirandese (mwl)

* Renamed ns:-2 and added previous name as alias for backwards
  compatibility.
* Renamed NS_TEMPLATE_TALK.
* Added "utilizador/utilizadora" as alias for NS_USER for backwards
  compatibility.
* NS_MEDIAWIKI added to avoid issues.
* Renamed NS_PROJECT_TALK and added old naming as alias for backwards
  compatibility.

Author: <https://www.mediawiki.org/wiki/User:Athena_in_Wonderland>.

Bug: T180052
Change-Id: I85431339bf24520063077db49d5e9f6b2666bc23

RCFilters: Make 'days' and 'limit' sticky

* Each has a hidden preference to override the preferences value
* Each value is different between Watchlist and RecentChanges
* rcfilters-limit is updated when rclimit is changed
* Not conditionally hiding the rcdays, watchlistdays and wllimit yet
  because hide-if's behavior is annoying

Bonus:
* Add a static method to check whether RCFilters UI is enabled
  and enabled by default. Adjust the call for Watchlist which
  checks a slightly different configuration setup.

Bug: T174415
Change-Id: Ib933de3a3f9e876924386e80f315506f60f8af54

Merge "objectcache: Run preemptive WAN cache refreshes post-send"

Merge "RCFilters: Ignore 'invert' model if there are no namespaces in saved queries"

Merge "objectcache: fix a few documentation mistakes"

Merge "Cleanup WikiPageTest"

objectcache: Run preemptive WAN cache refreshes post-send

This keeps HTTP request time consistent in case of expensive keys

Change-Id: I0746fde29a6e2f27d1b92f1af599c741d5972f46

objectcache: fix a few documentation mistakes

* Fix useInterimHoldOffCaching() description
* Mention when staleTTL and graceTTL where added

Change-Id: Iea24b77f6816251e9dac814dc57f6a44382a3cea

Merge "objectcache: Add @ingroup to RedisBagOStuff"

Merge "Use ParserOutput stateless transforms"

Merge "ParserOutput: Add stateless transforms to getText()"

RCFilters: Ignore 'invert' model if there are no namespaces in saved queries

When comparing and when saving the query, normalize the 'invert'
state so that it represents the **effective** state of the filters.
If there are no namespaces, then 'invert' is irrelevant.

Make sure we ignore that state when it is irrelevant when we save
saved queries and when we compare the current state to a saved query.

Change-Id: If80b50c3d2b90b5116b6b0018a8d56bce2deb7c4

Merge "Title: Use a more proper way of detecting whether interwikis are local"

Merge "objectcache: Make WANObjectCache interim caching not interfere with ChronologyProtector"

objectcache: Make WANObjectCache interim caching not interfere with ChronologyProtector

Also removed useless line from testLockTSE(). That would have needed
to be using $this->internalCache and those locks are freed immediately.

Bug: T180035
Change-Id: Ida1a923f779aaf8410da76643457d2200da6cb20

Merge "Remove @defgroup from DefaultSettings.php"

objectcache: Add @ingroup to RedisBagOStuff

Was missing from "Cache" group, as well as from the cross-component
"Redis" group.

Change-Id: I52ebc93cf472a4acc383742e0ea5c15c5a6db537

Remove @defgroup from DefaultSettings.php

This renders in Doxygen as a blank page in the "Modules" sidebar,
doesn't seem to serve any purpose.

No mentions of "Globalsettings" or "Global settings" elsewhere
in the repository.

Change-Id: I2c03b4190c899f2acff400d63bbd3df09f8d49d1

Remove some dumb headers with my name in it

Change-Id: Ib34b623b47fb92f1383bc5c55c50a16dd6d49abd

Merge "resourceloader: Remove deprecated minifier config vars"

Title: Use a more proper way of detecting whether interwikis are local

Bug: T181575
Change-Id: Ifcfe609ce01b174120b81b8c0bf82c4ed6077641

Merge "Make ChronologyProtector actually use cpPosTime cookies"

resourceloader: Remove deprecated minifier config vars

Unused, their values have been hard-coded since 1.27.
No uses anywhere in Wikimedia Git, besides their definition
in this file.

Change-Id: I6689274d0ad028e2c468560816ab753b345048e9

Merge "Fix phpdoc for Category::newFromName"

Merge "WatchedItemStoreUnitTest return strings in db mocks"

Merge "Add missing @throw tags to IDatabase/Database"

Merge "Use parent function in mysql DB class in getWeightScale()"

Make ChronologyProtector actually use cpPosTime cookies

Setup.php was only injected the value from $_GET. LBFactory used
to check both before 0e5cd18b74d83c50, with the cookie check left
in LBFactoryMW. After d175b391ae2a7, LBFactoryMW class switched to
the default LBFactory method for making a ChronologyProtector
instance, so the cookie check was lost there too.

This means that an HTTP GET request due to a redirect from a POST
request will still make sure the DB position store is up-to-date.
This only matters is that store uses replication (e.g. cross DC).

Also refactor LBFactory handling of the cpPosTime URL parameter.

Change-Id: Ie5744577609f4c193cacd71334f92b30f02e3846

Add missing @throw tags to IDatabase/Database

Change-Id: I235c5beacce38e9121fb1db94ce410d010356fcf

Use parent function in mysql DB class in getWeightScale()

May as well keep this case in sync with the base class

Change-Id: I4c62c3247ec3372c6a274b889fac1b7a883f5cf5

Merge "jobs: Remove ClearUserWatchlistJob 'batchSize' option"

Localisation updates from https://translatewiki.net.

Change-Id: Idd9b2d5c55e485a3d30473323ee224b628c0e83b

jobs: Remove ClearUserWatchlistJob 'batchSize' option

Use $wgUpdateRowsPerQuery instead, and use its value at run-time
instead of inside the job parameters. Also helps with deduplication
if batchSize is changed.

Change-Id: Ifef25a3ae5ae2418359a41eba05778613fbb548f

Merge "RCFilters: Followup I75cfb2b56a: Fix function declaration"

Use ParserOutput stateless transforms

We still set the state in many cases for benefit of extensions, but all
calls within core should no longer be using non-default state.

Change-Id: I78b62ec33fcb8273acb9b3b4e9012215442be94c
Depends-On: I140ff32373430b61b92226689ef9b58cca317450