Remove "Generated by PHPUnit" comments in 3 files

Change-Id: I8624f8cd0f658a37384a948076e8f6d68f484ce1

Set $wgUserEmailUseReplyTo = true by default

Mainly to work around restrictive DMARC policies (e.g., Yahoo!).

Bug: T66795
Change-Id: I3b34f6820915cc68d46ff6fd38feacd06efd2b69

Merge "Mark pages on watchlist as visited using JS without reload"

Merge "Move Special:Watchlist auto-reload check from PHP to JS"

Localisation updates from https://translatewiki.net.

Change-Id: I1b174ddfd90298e39e68a7495439e8c3b08feb1c

Mark pages on watchlist as visited using JS without reload

When the "mark pages as visited" is clicked, a dialog appears,
asking for confirmation. On confirmation, an API request is sent
to mark all pages as visited, and all unvisited watchlist entries
are changed to appear visited.

Based on a userscript by User:NQ (from English Wikipedia)
https://en.wikipedia.org/wiki/User:NQ/WatchlistResetConfirm.js

Bug: T150045
Change-Id: I45fb02a1edc1b0331925e9a244a2455f86ad3886

Article::fetchContent() was removed

Bug: T145728
Change-Id: Ie8bc12e391f2983452bf472333ea3cfaa72e4918

Merge "WikiPage::getText() was removed"

WikiPage::getText() was removed

Bug: T145728
Change-Id: I9e5ac149ad1780719b5c8f4e27048168cf33c0b9

Merge "ImageHistoryPseudoPager: Don't ignore limit from URL"

Merge "WikiPage::prepareTextForEdit() was removed"

WikiPage::prepareTextForEdit() was removed

Bug: T145728
Change-Id: If15777c04a52bd398092884fab592cfb0930255b

Merge "installer: Restore PHP 5.3 compatibility in index.php"

Localisation updates from https://translatewiki.net.

Change-Id: I204f2ce7bacf1bcf382dc51b2a574e405220d5d3

installer: Restore PHP 5.3 compatibility in index.php

This file should remain compatible with PHP 5.3, so that we can
actually display the error message about old PHP version, rather than
a syntax error.

Change-Id: If12f79f5d375e925291728aac7c9db18ca226d26

ImageHistoryPseudoPager: Don't ignore limit from URL

I thought the parent class would handle this for us, but no: our setLimit()
call has been overriding the limit from URL set in parent constructor.

Follow-up to f94d5239b528c406c85e090152805a1306e13ded.

Bug: T152813
Change-Id: I93762021f70613bfde72da1eb737e9fa32d7d97c

Move Special:Watchlist auto-reload check from PHP to JS

This makes the watchlist js load all the time, instead of only
if the auto-reload preference is set, so we can put other things
in the same js module.

Bug: T150045
Change-Id: Ib8acca39593fe3d2369dc3187a9a55413553843d

Localisation updates from https://translatewiki.net.

Change-Id: I73bab410a14742c1c93b79bceb7de73feed5d752

Merge "Fix links problem on Special:Search"

Include numerical namespace in slow-parse.log

This makes it easier to sort by namespace rather than trying to parse it
out of the title.

Change-Id: I946cb00548bcb69bd2be98c15a9f1e02e546fa24

Merge "Special:RC 'hidemajor' filter"

Merge "Make refreshImageMetadata not fail completely if it doesn't like a single file"

Localisation updates from https://translatewiki.net.

Change-Id: Ic25484ba66bfe1291c39d7e12ed08fa7d8b633c6

Fix links problem on Special:Search

Fix bug that most links on Special:Search did not have text rendered
Also restore some changes from commit f736c3709a54be33f808e8d12f85707b90763165
where Linker::link() usage was replaced by LinkRenderer

This reverts commit 791d5319bff0be23c3ea474a84d42ed788355688

Change-Id: Id25d1b1acd0dcafc47511b2db3782e5d441f3731

Follow-up e8dd1f43fb169: fix JS error when $wgTranslateNumerals is false

Bug: T152800
Change-Id: I99c341168dada063876a66689d68e0b9adf7f09a

Make refreshImageMetadata not fail completely if it doesn't like a single file

Change-Id: I1170cf91ba6dea08d35457c9859b8c18482d4aa6

Merge "ApiQueryUsers: Add ability to search by user ID"

Merge "Revert "Replace Linker::link() usage with LinkRenderer""

Revert "Replace Linker::link() usage with LinkRenderer"

Most links on Special:Search no longer have links with text rendered.
Please revert this revert and fix the issue before re-merging.

This reverts commit f736c3709a54be33f808e8d12f85707b90763165.

Change-Id: Id92c484adc1d3339f9412bfb60ffe213cc1315ee

Merge "Remove empty lines from PHP and JavaScript comment blocks"

Remove empty lines from PHP and JavaScript comment blocks

This is a pure documentation change. It mostly removes empty lines from
comments (and entirely empty comments), as well as adds a few missing
documentation blocks and fixes a minor mistake. I hope it's ok to have
this in one patch. I can split it, please tell me.

Change-Id: I9668338602ac77b903ab6b02ff56bd52743c37c4

Merge "Use the phan exit code directly"

Merge "Use 1 processes instead of 4 for phan"

Use the phan exit code directly

Change-Id: Ic99e334576e350d11c57ac4b74085c99313401c4

Use 1 processes instead of 4 for phan

Running against mediawiki core phan needs around 1GB of memory
per process. >4GB is too excessive for a default setting. Instead
set the default to 1 process and add the ability for users to
select more processes on the command line.

Bug: T132636
Change-Id: I3d9fa462916fba519fe6d7f1916f881eefb6a942

Merge "API: More i18n cleanup"

Merge "HtmlArmor: Document that null is an acceptable value"

API: More i18n cleanup

* Use errorformat for action=login Failed responses in non-BC mode.
* We removed 'messageHtml' from action=rollback's response on error, but
  left it for success. Remove it there too, it's even less useful.
* We changed action=watch's reporting of errors, but left the
  mostly-pointless reporting of "success" UI messages. These should be
  handled on the client side.

Change-Id: Ia6c402a4254fbacf4c2c3f125ce8bf0bcc71e509

Localisation updates from https://translatewiki.net.

Change-Id: Iba22019e88d103bf2c7b98309e57e24be3328dff

HtmlArmor: Document that null is an acceptable value

And that you'll get null back.

Change-Id: Ibd74e6af16b51796b46c2e23b7442740f5f69a0f

Merge "Let mw.Api recognize errorformat!=bc"

ApiQueryUsers: Add ability to search by user ID

Added new 'userids' parameter to ApiQueryUsers.php. It accepts user ID numbers.

Bug: T34494
Change-Id: I8021bc8660f19e90f0f7d43ea0e82d5258c586ab

Let mw.Api recognize errorformat!=bc

Iae0e2ce3bd42dd4776a9779664086119ac188412 introduced a new
API error format where, in case of error, there is no
'error' property in the response, but an 'errors'.
mw.Api doesn't recognize those as errors and just resolves
the deferred, instead of rejecting it.

Change-Id: I4c05ef7d7574e2ca538170609fab85d8463b7b68

Merge "Replace Linker::link() usage with LinkRenderer"

Merge "Special:Userrights should set isself on page view, not just on submit"

Provide context to ChangeTags::buildTagFilterSelector

Bug: T105649
Change-Id: I5f44f4c054010c865ddb51e7d69107bdf2e88c00

Merge "Add configuration for running etsy/phan against core"

Merge "Chinese Conversion Table Update 2016-7"

Add configuration for running etsy/phan against core

Adds the necessary configuration and stubs for phan static analysis to
run against mediawiki core. A variety of fixes have been applied to core
recently such with this current configuration we are completely passing,
except for one issue with a bug in Phan (https://github.com/etsy/phan/issues/286)

In it's current configuration Phan will detect fatal errors trying to
access non existant classes, undefined method calls, and other errors.
The analysis can be expanded as we cleanup more of the codebase.  This
is in preparation for working on getting the CI to run phan as part of
the review process. I have found phan to be usefull in CirrusSearch
(although with stricter config) for catching bugs and I think it could
help in core as well.

We arn't too far from being able to disable `null_casts_as_any_type`,
there are only a few classes that need adjusting. Lots more work needs
to be done to reduce `minimum_severity` from 10 to 5 (normal) or
0 (low).

Change-Id: Iafd55b1380f37d7b0d195eed081f8042166690e8

Localisation updates from https://translatewiki.net.

Change-Id: Ic36aaa135b8702f13e4dd4400878d8d9cc2146ce

Special:Userrights should set isself on page view, not just on submit

Partially reverts I57e9ca4f20fe557e4024c4f5a4865170f02ebb45

Bringing back lines that were incorrectly removed

Bug: T152600
Change-Id: I5e1171be4cb75a6d7cce5f99b9d31dc0cb5d8db3

Replace Linker::link() usage with LinkRenderer

Linker::link() replaced with LinkRenderer in some pages in includes/specials

Bug: T149346
Change-Id: I937bdc99d1342e73f20b8437a9b55dffb04161c5

Special:RC 'hidemajor' filter

'hidemajor', along with the existing 'hideminor'
RC filter, allow showing just the minor edits,
just the major edits, or both.

This will be used by the ERI project.

Bug: T149863
Change-Id: I936a4a1b13d8c4a15c745012cd0f82207d9e57ca

Merge "ApiRevisionDelete: Use standard formatting for errors and warnings"

Merge "Replace Linker::link() & Linker::linkKnown() with LinkRenderer"

Replace Linker::link() & Linker::linkKnown() with LinkRenderer

Change-Id: I1350819165095d9be79aea8d9189760a4dbddd40

Merge "Replace Linker::link() with LinkRenderer in all revisiondelete pages"

Special:RC filter: hideunpatrolled

Allows hiding edits that have not been patrolled.
In other words, showing only edits that have been patrolled.

Will be used by the new Special:RC filters (ERI project).

Bug: T152061
Change-Id: I3cd896812c5af87bc4be53d493055c37298c712f

Special:RC filter: hidehumans

Allows showing only bot edits

Will be used by the ERI project.

Bug: T149862
Change-Id: I748eb3c614abd7b8b228efe3da5e9cb569a8618f

Special:RC filters: hide page edits, new pages, log entries

New RC filters for ERI project:
  hidepageedits
  hidenewpages
  hidelog

Bug: T150060
Change-Id: I6fe852fb0f5386c3385ada5189ceaf0d9bbc979d

Merge "RC/Watchlist: Filter out parameters that cannot be displayed"

Merge "Cleanup static analysis errors"

Merge "Update OOjs UI to v0.18.2"

Cleanup static analysis errors

While prepping the code base for using etsy/phan i found a few problems.
A couple are real problems, while others are just places etsy couldn't
figure out the types.

* AuthPlugin referenced UserLoginTemplate which doesn't exist. Tracing
  back usages this accepts classes that extend from BaseTemplate.
* DatabaseSqlite had an invalid @var annotation.
* LoadBalancer had some missing or under-defined annotations. Make them
  more specific where possible.
* SqlBagOStuff didn't have the appropriate use statement for
  WaitConditionLoop and probably doesn't work. Also updated annotations
  that point to non-existent DatabaseBase to use IDatabase.

Change-Id: Iff2270b418ad2f8f97cfdb6df646c435d3283536

Merge "profiler: Support tideways for PHP7 profiling"

Update OOjs UI to v0.18.2

Release notes:
 https://phabricator.wikimedia.org/diffusion/GOJU/browse/master/History.md;v0.18.2

Change-Id: Id08147e9015fc68a882733268a2bf860dd1ccb27

profiler: Support tideways for PHP7 profiling

xhprof does not support PHP7, and it doesn't seem like upstream will be
working on that any time soon.

Tideways is a profiler that is basically a drop-in replacement for
xhprof with functions renamed. So Xhprof::enable() and Xhprof::disable()
will now try to use tideways if that is installed and xhprof is not.

Bug: T152186
Change-Id: I0d7d2de56ac638ca2851c662f527049bd620c0e9

Make remote content skin tabs non-File specific

This is one of the steps for implementing shadow namespaces -
generalizing foreign file specific code to be usable for any namespace.
This allows us to no-longer special case WikiFilePage in SkinTemplate
when building content_navigation, and will let GlobalUserPage to add
similar tabs just by overriding the same functions.

The new WikiPage::getWikiDisplayName() and WikiPage::getSourceURL()
functions will return expected values for both local and remote content.

Change-Id: Ib3d7dcbefe95da351872e63f306799eef83e00a7

Merge "Special:RC tests: ignore order when asserting conditions"

Merge "MapCacheLRU: Properly handle bogus cache keys"

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

includes/api/i18n/qqq.json looks insane, probably conflict with https://gerrit.wikimedia.org/r/#/c/321406/

This reverts commit 50135c64ca83c5903ddbc04a9af8638321dc25e0.

Change-Id: I63d88b6ef11eb796ff80e83763a2b0c99363b7f9

ApiRevisionDelete: Use standard formatting for errors and warnings

Change-Id: I8e88c356ac81572d866fbdd8f15e7a8dd4384c06

Localisation updates from https://translatewiki.net.

Change-Id: I57e6697462f86b9ef684d82e739781b04dbfa250

MapCacheLRU: Properly handle bogus cache keys

They can only be strings or integers. Anything else is wrong. We
should be throwing exceptions here so developers find this early
when running tests and manual testing.

Exceptions (instead of warnings) allow us to get a full useful
stacktrace.

Change-Id: I823ce33523283509c14a05f816f261d6b400e243

API: Fix i18n for apihelp-main-param-errorformat

Renamed the parameter at some point, but forgot to update this message.

Change-Id: I84fb44e7419bd7aee28f4dfd2bd37c51384330cc

Merge "API: i18n for warnings and errors"

Replace Linker::link() with LinkRenderer in all revisiondelete pages

* RevDelArchiveItem
* RevDelArchivedFileItem
* RevDelFileItem
* RevDelLogItem
* RevDelRevisionItem

Bug: T149346
Change-Id: I96f83d06b4d3ccf3f76ccfca843a6aaf76d89063

Merge "registration: Provide credits information to callbacks"

Merge "registration: Refactor validation logic to avoid duplication"

Merge "Convert Special:DeletedContributions to use OOUI."

Merge "registration: Don't let extensions load late"

objectcache: Remove broken apcu_set() calls

The method call was misnamed (should be apcu_store) and not needed anyway,
since the other BagOStuff classes fail if no key is present.

Change-Id: I726965e665c2d0c5843d8dd33504810d130b9e98

API: i18n for warnings and errors

API warnings and error messages are currently hard-coded English
strings. This patch changes that.

With a few exceptions, this patch should be compatible with non-updated
extensions:
* The change to ApiBase::$messageMap will blow up anything trying to
  mess with it.
* The changes to the 'ApiCheckCanExecute' hook will cause a wrong
  (probably unparsed) error message to be emitted for extensions not
  already using an ApiMessage. Unless they're currently broken like
  Wikibase.

Bug: T37074
Bug: T47843
Depends-On: Ia2b66b57cd4eaddc30b3ffdd7b97d6ca3e02d898
Depends-On: I2e1bb975bb0045476c03ebe6cdec00259bae22ec
Depends-On: I53987bf87c48f6c00deec17a8e957d24fcc3eaa6
Depends-On: Ibf93a459eb62d30f7c70d20e91ec9faeb80d10ed
Depends-On: I3cf889811f44a15935e454dd42f081164d4a098c
Depends-On: Ieae527de86735ddcba34724730e8730fb277b99b
Depends-On: I535344c29d51521147c2a26c341dae38cec3e931
Change-Id: Iae0e2ce3bd42dd4776a9779664086119ac188412

Localisation updates from https://translatewiki.net.

Change-Id: I74a8995410784d438e0168919c5216aacadfc723

Merge "phpunit: Make getResourceLoaderContext() more extendable"

Misleading messages on Special:Userrights

Corrects what is not addressed by I57e9ca4f20fe557e4024c4f5a4865170f02ebb45

Bug: T152428
Change-Id: I105be73acc5a2bc088b557ccafbdf9db726480da

Merge "OOjs UI: Backport I73f95965694ec7fb0fa9a474742286e1105e5c85"

Use single quotes

Follow-up to Icc9bc096

Change-Id: I4863e0e7946fd59022846198e86302f556fee617

OOjs UI: Backport I73f95965694ec7fb0fa9a474742286e1105e5c85

Bug: T151061
Change-Id: I715d2ac077c5a6f9de420d90f19ef4b631168f26

Localisation updates from https://translatewiki.net.

Change-Id: Idc4a0a2896c29601b3f099c3bb2a622e995cfc4e

Removed deprecated class RevisiondeleteAction

RevisiondeleteAction was deprecated in release 1.25

Bug: T61113
Change-Id: I9238d0558b21fccde70e7e23a169e114362f793e

registration: Don't let extensions load late

We want all extensions to be queued together and load at the same time
so in the future we can properly evaluate dependencies as a whole. If
extensions load late, they would bypass this, potentially causing
issues.

Bug: T117277
Change-Id: I09b306bd6f6ccf4210f36be0118e7f17f2c3d264

Merge "Add integration test for MWHttpRequest user/pass options"

Merge "Replaced Linker::link() usage with LinkRenderer in some special pages"

Replaced Linker::link() usage with LinkRenderer in some special pages

Bug: T149346
Change-Id: I73c2b7402012a49e4419b6214f1d03f2e819220b

registration: Provide credits information to callbacks

Registration callbacks now provide basic credits information (name,
path, type, authors, license-name, version, etc.) as the first argument.
The main use case right now for this is to support extension VERSION
constants for backwards-compatibility.

In addition, callbacks now run *after* attributes are exposed, so
callbacks could use data from them if they wanted to.

Bug: T151136
Change-Id: Ic5965dd4e259e1f46222ac92b8e78750e67b51d6

Merge "Add $magicWords translation for Bengali (bn)"

Add integration test for MWHttpRequest user/pass options

Change-Id: I5bcb263a725e8042ee51109080d2075744fe7001

Merge "Clarify in emailuserfooter that by responding one reveals their email address"

Clarify in emailuserfooter that by responding one reveals their email address

Bug: T152242
Change-Id: I686f7bb8dd1d225620266ac9cbecadcf06bd7705