Replace use of &$this

Use of &$this doesn't work in PHP 7.1. For callbacks to methods like
array_map() it's completely unnecessary, while for hooks we still need
to pass a reference and so we need to copy $this into a local variable.

Bug: T153505
Change-Id: I8bbb26e248cd6f213fd0e7460d6d6935a3f9e468

Merge "Add TestLogger::setCollectContext and fluent interface"

Add TestLogger::setCollectContext and fluent interface

TestLogger has a setCollect() but lacked a similar method for the
$collectContext introduced with:
    7b85b6b09e - Optionally collect context in TestLogger

Document $collectContext on __construct()
Add setCollectContext()
Make both setCollect() and setCollectContext() to return $this for
fluent interface:

  $logger = new \TestLogger();
  $logger
      ->setCollect( true ),
      ->setCollectContext( true );

Change-Id: I806b07cfddbc7b428d41e3d619ef4d1db16f6747

Localisation updates from https://translatewiki.net.

Change-Id: If6140720728447ed5b4a80b4df8cb5f6b9269f96

Merge "Option for DateInputWidget to display full month and day names"

Merge "Don't return expired user groups in UserRightsProxy"

Merge "Merge duplicate SpecialSearchTest files"

Merge "Add release notes for recent language fallback changes"

Merge "Add explicit GENDER support to userrights-*usergroup"

Option for DateInputWidget to display full month and day names

Add a "longDisplayFormat" config option to DateInputWidget to show
full month and day names when using the default locale-specific
display format.

Bug: T120733
Change-Id: I2db6892720abf86dfc9655291b1070aa7f7bf77b

Merge duplicate SpecialSearchTest files

Change-Id: I1d6be3f6c32102ee8c7f4ee3dc6f0c752ef05534

Don't return expired user groups in UserRightsProxy

Also add a missing type hint in UserGroupMembership.

Bug: T156779
Change-Id: Ica31541f59115db3db92de82592e6a9422082bba

Merge "registration: Allow properties in "requires" from v2"

Merge "parser test editor: Fix emitting of !! hooks"

Merge "Move ChronologyProtector/TransactionProfiler to Rdbms namespace"

Merge "objectcache: Avoid using deprecated ScopedCallback alias"

parser test editor: Fix emitting of !! hooks

The first newline was missing so a block like:
 !! hooks
 source
 !! endhooks

would turn into:
 !! hookssource
 !! endhooks

Change-Id: I2a4c5e52050d55fb0c9b4f5d0494eb00e34b233c

objectcache: Avoid using deprecated ScopedCallback alias

Change-Id: Ica8a066c3f28adc710ee11919c07dd188144beb5

Localisation updates from https://translatewiki.net.

Change-Id: Id9bbcde94cbbd77e360022504b3114629525527e

Merge "EditPage: switch from NS_CATEGORY check to instanceof CategoryPage"

Merge "EditPage: Mark public functions"

registration: Allow properties in "requires" from v2

v2 of the schema allows extensions and skins to be definied under
"requires". This is also used by some extensions in extension.json for
v1.
It works, so allow the same properties in v1 as in v2.

Change-Id: I9b5f1986cadbd714d6f3460ac3e1db3bf7aae65d

Merge "PHAN: lock phpstorm-stubs at a specific hash in composer.json"

PHAN: lock phpstorm-stubs at a specific hash in composer.json

This will stop updates to the upstream package from
breaking our CI with no warning.

Change-Id: I1cafe663c25d46134025f7c6987e48676615b7b0

PHAN: Add tideways stub

Used in Xhprof

Change-Id: I9f208bbaa0799d194dbef61cad851995020a5d17

PHAN: load stubs from phpstorm-stubs library

Also add a single small stub for the Memcached class
constructor that is slightly wrong in the phpstorm
stubs library.

jetbrains/phpstorm-stubs has been added to packagist via:
https://github.com/JetBrains/phpstorm-stubs/pull/149

Change-Id: I1bca7390d1dc24c38d65c1c8968f356326ff4636

Merge "Fixes for more robust dealing with content handlers."

Merge "Make user_groups queries honor $wgDisableUserGroupExpiry"

Make user_groups queries honor $wgDisableUserGroupExpiry

Follow-up to Icf78cce3f3e362677d10897b8d1103b3df91fa08.

Change-Id: I94102ceb00398940965280d8742a2747631dcc87

Merge "Fix positioning error with draggable, revert and grid"

Fix positioning error with draggable, revert and grid

This minor change fixes an error when using draggable revert and grid.

Since upgrading of jQuery UI probably won't happen in the near future [1]
I looked for another way to fix the issue.

See [2]. Unlike stated in the second comment adding the lines below
won't fix the bug. Moving the lines up fixed the issue.

See also the description of the bug in [3].

[1] https://phabricator.wikimedia.org/T71386
[2] https://bugs.jqueryui.com/ticket/4696
[3] https://phabricator.wikimedia.org/T140965#2944610

Bug: T140965
Change-Id: Ib0f386c6a2447740f44591be70c34cf4a83b88c4

Merge "Remove unused variables in RCCacheEntryFactoryTest"

parserTests: Avoid using <big> for adoption agency algorithm test

The behavior of <big> may change in the future, c.f. T154067.

Change-Id: I817894c25cab96a491028fe2a9443140ea1d6e97

Merge "registration: Don't allow extra properties in "requires""

registration: Don't allow extra properties in "requires"

VersionChecker is using a switch with a exception in the default path,
which makes it very unhappy to see additional properties.

Change-Id: Ief84497de6b2fa2d2715fc713088bee66c21fdc0

Merge "API: Add reference to the mailing list in errors and deprecation warnings"

Merge "Fix tags not being set in Special:Block"

Localisation updates from https://translatewiki.net.

Change-Id: I1963ac8ae1166037a2550fe628fe06812bf2b843

Add release notes for recent language fallback changes

Change-Id: I5dfba8eeca45a77c7c67091615c742a8b96bd202

Fix tags not being set in Special:Block

Currently, a PHP error is thrown when tags are not set
in Special:Block on line 832. This patch fixes this
by adding an extra isset() check to see if the tags
variable is set.

Bug: T156486
Change-Id: Ib8722bffbcac5953263ded41eceb3d389d0932f0

Localisation updates from https://translatewiki.net.

Change-Id: I274c0404681ea7e97d3c100384a58bbde8f80cdc

Remove unused variables in RCCacheEntryFactoryTest

Change-Id: I41478dc2086577aacf444d0620c8b950e94e61ff

Remove HTMLFileCache::newFromTitle()

Change-Id: I0076801aa7b9af5083c5a7a66f2d1c9884b70ee9

Remove ArchivedFile::getUserText()

Deprecated since 1.23
No usages found on gerrit

Change-Id: I037d56d6e57288753ae281388270d7e216707192

Merge "Add tests for SkinTemplate::setupSkinUserCss"

Remove User::crypt() & User::comparePasswords()

These have both been deprecated since 1.24
Hard deprecation happened back in 2014

Both methods are still used by the SecurePasswords
extension, but this extension is documented on mw.org
as not working with MW1.24+.

I can find no other uses.
Lets finally get rid of these!

Change-Id: I94a7b65d2216bbc505e190af3182de2317976ed1

Merge "Allow OpenGraph RFDa meta tags"

Add explicit GENDER support to userrights-*usergroup

Best practice for documentation and for message checkers.
https://translatewiki.net/wiki/Thread:Support/Gender_in_MediaWiki:Userrights-viewusergroup

Change-Id: Ib08617d576b3285d4c40e0edccc2e99931e55298

Merge "translateBlockExpiry: Duration is block expiry minus current time"

translateBlockExpiry: Duration is block expiry minus current time

For relative timestamps in $str, strtotime( $str, $now ) returns an
absolute Unix timestamp $str since $now, and this timestamp is given
to $time. However, Language::formatDuration expects a time duration,
not an absolute timestamp. We obtain this duration from the difference
between $time, the absolute timestamp of block expiry, and $now, the
absolute timestamp of the time in which the block action happened.

Tests have been added to test both this patch and 01936fa, the patch
that caused this regression.

Bug: T156453
Change-Id: I6fd8c02dc3c6456067fe25cb9f33f5b4c78332aa

Add Related Changes Link for all categories

Add Related Changes link into the toolbox for the redlinked
categories as well since users need to see changes made in these
categories as well.

Bug: T147762
Change-Id: Ic5dd6f531b5ff47ce68fd2893fd1550d10f74e4e

Disregard expired user_group rows in special page and API DB queries

An essential follow-up to I93c955dc7a970f78e32aa503c01c67da30971d1a.

Bug: T12493
Change-Id: Icf78cce3f3e362677d10897b8d1103b3df91fa08

Merge "Allow SearchEngine users to access features data"

Merge "User group memberships that expire"

Allow SearchEngine users to access features data

Useful in case the client wants to re-evaluate what was set
here, or if the SearchEngine implementation wants to expose
some of its states.
In our case it allows CirrusSearch to inform SpecialSearch
that we prefer to display search results with a new experimental
layout.

Bug: T156299
Change-Id: I7f661c852ef70ea7bc9ae2959f7d6e48776a9877

Localisation updates from https://translatewiki.net.

Change-Id: I8ccae21da1014b794a87ddbd4ade8be9b2c9edd5

Merge "Add content model indexing"

User group memberships that expire

This patch adds an ug_expiry column to the user_groups table, a timestamp
giving a date when the user group expires. A new UserGroupMembership class,
based on the Block class, manages entries in this table.

When the expiry date passes, the row in user_groups is ignored, and will
eventually be purged from the DB when UserGroupMembership::insert is next
called. Old, expired user group memberships are not kept; instead, the log
entries are available to find the history of these memberships, similar
to the way it has always worked for blocks and protections.

Anyone getting user group info through the User object will get correct
information. However, code that reads the user_groups table directly will
now need to skip over rows with ug_expiry < wfTimestampNow(). See
UsersPager for an example of how to do this.

NULL is used to represent infinite (no) expiry, rather than a string
'infinity' or similar (except in the API). This allows existing user group
assignments and log entries, which are all infinite in duration, to be
treated the same as new, infinite-length memberships, without special
casing everything.

The whole thing is behind the temporary feature flag
$wgDisableUserGroupExpiry, in accordance with the WMF schema change policy.

The opportunity has been taken to refactor some static user-group-related
functions out of User into UserGroupMembership, and also to add a primary
key (ug_user, ug_group) to the user_groups table.

There are a few breaking changes:
- UserRightsProxy-like objects are now required to have a
  getGroupMemberships() function.
- $user->mGroups (on a User object) is no longer present.
- Some protected functions in UsersPager are altered or removed.
- The UsersPagerDoBatchLookups hook (unused in any Wikimedia Git-hosted
  extension) has a change of parameter.

Bug: T12493
Depends-On: Ia9616e1e35184fed9058d2d39afbe1038f56d7fa
Depends-On: I86eb1d5619347ce54a5f33a591417742ebe5d6f8
Change-Id: I93c955dc7a970f78e32aa503c01c67da30971d1a

Further updates to .mailmap for the generation of CREDITS

Mostly adding current and former WMF/WMDE employees, and a couple of
others, under the correct version of their full names.

Change-Id: I36ca297967125cc211705509466358d9d762e5c8

objectcache: add WANObjectCacheReaper for assuring purges

* This fixes keys based on some sort of change log.
  Updates are wrapped in a mutex and keep track of the
  last known good position.
* Make WANObjectReapUpdate class that cleans up title
  related keys using the recentchanges table. This triggers
  as a deferred updates on RC view.

Change-Id: I7f14b9ca2533032147e62b1a3cc004a23da86579

Merge "rcfeed: Add basic PHPUnit integration test"

Add content model indexing

Bug: T156371
Change-Id: I1514f9db70870ec05b30153cc56558af0a28b8c9

Fixes for more robust dealing with content handlers.

Change-Id: I12a02da005f4b2bceaa850bd1f41a90ac4e1754a

Merge "ApiUpload: Raise an error on attempts to async-upload a POSTed file"

EditPage: switch from NS_CATEGORY check to instanceof CategoryPage

Change-Id: I0ee3672a3d698eda25191bccbbc10c9e2f9f7546

EditPage: Mark public functions

Change-Id: I269db9dcbf9cc6ef75a323eb395d12faa09ea20e

Move ChronologyProtector/TransactionProfiler to Rdbms namespace

Change-Id: I37a655bd8bd267c9bc32028b55925b2dce527d33

Merge "Avoid master queries when old filter checks are viewed"

Merge "ApiSandbox: Allow displaying query parameters in various formats"

ApiUpload: Raise an error on attempts to async-upload a POSTed file

"Upload from stash" and "Upload from POST" use the same code path via
ApiUpload::performUpload(). It looks like when a31d0f8edd implemented
"async upload from stash", it was overlooked that someone might attempt
to do an unsupported "async upload from POST".

Bug: T156365
Change-Id: I6d207270af490cfc59d994a22a6406df45a9f6a2

Clarify emailuserfooter mw-message

Recently we modified this message to add a privacy disclaimer which is
automatically added to outbound email sent using the Special:EmailUser
function on MediaWiki.

The disclaimer, notwithstanding, it only shown for the receiver of the
message so we should clarify that their email address will be disclosed to
the original sender only if they decide to reply to the email.

This patch therefore adds a "If you reply"-like wording to the
emailuserfooter message clearly stating that only if the user decides to
reply the message, their email address will be indeed disclosed to that
original sender.

Change-Id: Ie2ccf26b078e768f615eea8c8a83d92ff83abce0

Merge "Fixing grammar, capitalization and spelling in comments"

Merge "refreshLinks.php: allow refreshing by categories, tracking or not"

Merge "Inject remaining cache objects into MessageCache"

Merge "Avoid races in MessageCache::replace()"

rcfeed: Add basic PHPUnit integration test

Change-Id: Ifb5ddd4bda6cd1be050da7a5419ebe90f5ecf60f

Merge "UploadBase: Allow RDF Schema namespace in SVG files"

Merge "InstantCommons: do not cache thumbs"

InstantCommons: do not cache thumbs

There is no point in local thumb caching when we set apibase
and thumbUrl to the remote wiki. It will only confuse MediaWiki
into creating local thumbnails for certain LinksUpdate tasks
even though the rendered page will reference thumbnails hosted
on Commons.

Bug: T145496
Change-Id: I0bd4b539f575036dd865bdd946660f10a9ee8801

Merge "Update html/php clauses for subpage parserTests."

Merge "Sync up with Parsoid parserTests."

Allow OpenGraph RFDa meta tags

See: http://stackoverflow.com/questions/22350105/whats-the-difference-between-meta-name-and-meta-property
And https://developers.facebook.com/docs/sharing/webmasters#markup

Bug: T51859
Change-Id: Ie8fd697e8588435a69f02e779038ae5a11d67be6

Sync up with Parsoid parserTests.

This now aligns with Parsoid commit 643d5392bcf4dfebf906102627c51e8a608125bf

Change-Id: I4d7dc7378ca7cfdb3919f33959f58eb5c4d88ca8

Prevent unexpected }- in converter output

Previously for input -{<span title="-{X}-">X</span>}-, the converter
sees -{<span title="-&#123;X}-">A</span>}-, so <span title="-&#123;X
becomes the content in the first block, and a stray }- is left to output.

Now, the converter sees -{<span title="-&#123;X&#125;-">A</span>}- with
this change. In further processing, the span tag may be parsed and have
its title attrib converted. For cases where the content is not processed
further (eg. "R" = raw flag), "-{X}-" is left as is in the attrib, which
is not so ideal, but at least it's better than the original extra }-
outside the whole tag.

Change-Id: Idbaaf53f914f362e5b8cc9fad02a524f8d591bb7

Merge "Don't remove border for floatright and floatleft"

Update html/php clauses for subpage parserTests.

The NS_USER namespace needs to have subpages enabled before these tests
work on the PHP parser.

Change-Id: I8e5e3bbd0dea6fc12f3b9ff9feeb58812fc51af1

Localisation updates from https://translatewiki.net.

Change-Id: I9a3d688828f6d3085eca91ba1653835b7854da21

Merge "Remove unused protected method SpecialUserrights::splitGroups"

Merge "Fix functionality and usages of SpecialUserrights::userCanChangeRights"

Merge "RCFilters UI: Add 'remove' and 'restore defaults' to filter list"

Merge "RCFilters UI: Read default states of filters"

RCFilters UI: Add 'remove' and 'restore defaults' to filter list

Bug: T144448
Bug: T149391
Change-Id: I418ad6b34ae8a7456a7e66bc703d4dccf36764a5

RCFilters UI: Read default states of filters

Add the functionality to read the default state of filters
and preserve them, so that they are considered when either
base state is requested or when filters are explicitly told
to revert to default.

Bug: T149391
Bug: T144448
Change-Id: I9e8e3430ca2f80d3f67422681e8fb69a43ee4bef

Avoid races in MessageCache::replace()

Do the process cache update immediately (as before) but push
the shared cache updates to a deferred update. This update
will thus start with a clear transaction snapshot, so it can
acquire the lock before the first SELECT as is proper.

Also added some missing method visibilities.

Bug: T144952
Change-Id: I462554b300d4688b09ab80cd1bb8a4340ffaa786

Merge "rcfilters: Remove implemented filters from list of links"

Merge "Follow-up e3da2e23: Fix typo in hideanons filter name"

Merge "Add tests for OutputPage::addMeta and set{Index|Follow}Policy"

Merge "Installer: don't treat the existance of main page as an error"

Add tests for OutputPage::addMeta and set{Index|Follow}Policy

This is a very basic test setup for the core functionality only. All
the special cases in getHeadLinksArray are currently not tested. But
this is a start.

This is relevant for the changes made in Ie8fd697.

Bug: T51859
Change-Id: I61e2da68ae0daea15fab2065a5dd63605d0b837d

Don't remove border for floatright and floatleft

This is a leftover from T2691, from a time when the skin still used
borders to fake a margin that didn't overlap with H2 elements. We fixed
this a long time ago in T28449 and it's follow ups, but this border: 0;
remained.

I discovered this now no longer desired side effect when trying to
apply these classes to tables.

Change-Id: I52b18c7c5b3dfea39c4653a8add00a607e7eab73