Split ApiImport.php to have one class in one file

Change-Id: I3fbf5d271585722e95b7966ba62dd5e16fa3623f

Remove ApiMessage from Generic.Files.OneObjectStructurePerFile.MultipleFound

Change-Id: Ib1a08c53f509bdae96ff8d3a5dadc9a55eb6b6af

Merge "Fix a couple of Generic.Files.OneObjectStructurePerFile.MultipleFound in api"

Merge "changes: Split Feed.php into a class per file"

Fix a couple of Generic.Files.OneObjectStructurePerFile.MultipleFound in api

Change-Id: Ibb6b324b286f62153ce5d08a66454e0b05a0ef78

Merge "HistoryAction: Consider 'feed=' and 'feed=0' as invalid feed actions"

changes: Split Feed.php into a class per file

Change-Id: I1f11a52871dcb249b3ba790e484dd10fe8bb049f

HistoryAction: Consider 'feed=' and 'feed=0' as invalid feed actions

Previously it did a PHP truthy check, which is unrelated to the
check in question and outside its responsibility to do.

The validation of the value happens inside HistoryAction::feed
and more specifically in FeedUtils::checkFeedOutput().

Also remove needless use of unicode normalization as this only
accepts exact pre-configured presets. Those presets are linked
by the sidebar and thus never considered user or otherwise
in need of any normalization.

Change-Id: Id68a753bb5a5f03f983f5b813a42e784cf91e8c9

Merge "registration: Allow to require environment abilities"

Merge "installer: Remove TODO per discussion in Ia9f0cd7d0117f67d2017e"

resourceloader: Add newline after 'mw.loader.implement()' in debug mode

The next mw.loader.implement() starts on its own line.

Change-Id: Iae0a5eab58f1065c354018ed23c6a9834973b19d

installer: Remove TODO per discussion in Ia9f0cd7d0117f67d2017e

So other developers think that the class property name makes more
sense than the proposed name in the TODO section. In an attempt to
rename it in this patc: Ia9f0cd7d0117f67d2017e, it was rather adviced
to remove the todo and leave the class property name as-is.

Change-Id: I3336b5a6112f098e290f102caac6b04c0b50b076

Merge "Reporting API and Feature Policy reporting support"

registration: Allow to require environment abilities

This patch adds the possibility for extensions and skins to require
certain environment abiltites that are not necessarily PHP extensions.

For now, the only ability introduced is the ability to shell out, but
the processing and testing is written in a more general way to allow
users to add more abilties later on by just changing getAbilities().

In theory, this allows using VersionChecker to check for random
abilities if they are specified in the constructor, as it is comletely
environment agnostic and not really bound to just be used for checking
extension compatibility.

Furthermore, it is possible to specify custom error messages for each of
these abilities in the constructor. Other parts of MediaWiki may use
these features to check for requirements while working with totally
different abilities.

Bug: T212472
Change-Id: Iff8512530b08ef509e7ac0b6ed8fe9578ef3e2a1

Merge "linting: Start enforcing a basic CSS class naming rule (with lots of opt-outs)"

Merge "Third argument to unpack() requires PHP 7.1"

Reporting API and Feature Policy reporting support

These are experimental APIs that lets one report
policy violations:

https://www.w3.org/TR/reporting/
https://github.com/w3c/webappsec-feature-policy/blob/master/reporting.md

Bug: T209572
Change-Id: I002e7802000ec37b3320d8ae761cc1888d4f6edb

Improve comments of outputVariesOnRevisionMetaData()

Change-Id: Ie54e068515ac09af27fdc38eef7863c3a4d252f8

Merge "Add vary-revision-exist flag to handle {{REVISIONID}} and parser cache"

Add vary-revision-exist flag to handle {{REVISIONID}} and parser cache

Follow-up to c537eb186862b3

Bug: T220854
Change-Id: Idc19cc29764a38e3671ca1dea158bd5fb46eaf4d

Merge "Remove bogus field references in DerivedPageDataUpdater"

Merge "parser: Fix return type for methods and match phpdoc comments"

Merge "logging: Replace deprecated use of CommentStore::getStore()"

Merge "Category: Remove "todo" comment about moving code from CategoryPage"

parser: Fix return type for methods and match phpdoc comments

Change-Id: I867d7eb6fc56cc52eb8e129977b7a62607a11268

Remove bogus field references in DerivedPageDataUpdater

Change-Id: I4f781d9136e01c4a7bbc316fee622c0933329a0e

logging: Replace deprecated use of CommentStore::getStore()

Change-Id: I5c34eda43d0d312bc53fc5e9ad618cb80284e34b

Category: Remove "todo" comment about moving code from CategoryPage

All stuff was already moved from CategoryPage.php to Category.php

Change-Id: I19820ec573c39d7368e3c6644e7c54ca9d8104c5

Merge "Improve documentation of constants throughout the codebase"

Merge "parser: Convert wikitext Preprocessor class to using WAN cache"

Merge "HistoryPager: Unset 'date-range-to' parameter when paging"

parser: Convert wikitext Preprocessor class to using WAN cache

Change-Id: Ida96a976109f7ceee0003b9d1d487c8cf9b80114

Merge "Get rid of unnecessary func_get_args() and friends"

Merge "Set MW_INSTALL_PATH in Maintenance.php"

Merge "HTMLFormFieldWithButton: Allow passing 'inputtype' and pass through 'buttonid'"

Merge "dao: Correct return type doc for getLoadBalancer()"

Get rid of unnecessary func_get_args() and friends

HHVM does not support variadic arguments with type hints.  This is
mostly not a big problem, because we can just drop the type hint, but
for some reason PHPUnit adds a type hint of "array" when it creates
mocks, so a class with a variadic method can't be mocked (at least in
some cases).  As such, I left alone all the classes that seem like
someone might like to mock them, like Title and User.  If anyone wants
to mock them in the future, they'll have to switch back to
func_get_args().  Some of the changes are definitely safe, like
functions and test classes.

In most cases, func_get_args() (and/or func_get_arg(), func_num_args() )
were only present because the code was written before we required PHP
5.6, and writing them as variadic functions is strictly superior. In
some cases I left them alone, aside from HHVM compatibility:

* Forwarding all arguments to another function. It's useful to keep
  func_get_args() here where we want to keep the list of expected
  arguments and their meanings in the function signature line for
  documentation purposes, but don't want to copy-paste a long line of
  argument names.
* Handling deprecated calling conventions.
* One or two miscellaneous cases where we're basically using the
  arguments individually but want to use them as an array as well for
  some reason.

Change-Id: I066ec95a7beb7c0665146195a08e7cce1222c788

Add test to check action- messages exist

Bug: T220779
Change-Id: Ide681e1f03957a8024f538ea6cac4e4396fef23e

linting: Start enforcing a basic CSS class naming rule (with lots of opt-outs)

Wow, so many opt-outs, and a lot of probably-not-going-to-change ones.

Actually wanted pattern:
* "^mw-…"

Content-related classes we're probably not going to scrap:
* "^wikitable$"

Hard-to-change classes used in lots of skins:
* "^toc$"
* "^toctoggle$"
* "^tochidden$"

Necessary for over-rides/specificity battles:
* "^oo-ui-…"

Probably worth considering changing soon:
* "^client-js$"
* "^client-nojs$"

Some of the exemptions have been applied to deprecated styles (where they're being
renamed); some are for code in entirely deprecated areas themselves; some have the
"wrong" name but sit alongside classes with an appropriate one. And some are, very
clearly, in the wrong repo. Such sigh.

Change-Id: I9b8423d53b6750b5ca79d2f69e111f75c93c5bdc

Add missing action- messages to MW Core

Bug: T220810
Change-Id: Ic43435bc42c23f055bde0c6cf7648d9f219cfb34

resourceloader: Avoid global window.mwTestFooJsCounter in unit tests

This change avoids a failure when running Special:JavaScriptTest with
the option noglobals.

Change-Id: Id61fca5ccedbd793f7786ad95681eaebb9feafb6

Merge "Fix order of @var parameter in PHP"

Merge "Use https://php.net/ instead of https://secure.php.net/"

rdbms: drop inappropriate and poorly documented "DatabaseOraclePostInit" hook

Change-Id: I3152e80d99b040b7918927e6c2a215cab59a9b10

Localisation updates from https://translatewiki.net.

Change-Id: Id524db06f2e39860f8455a22ee5805cfe06e4f8f

Use https://php.net/ instead of https://secure.php.net/

Change-Id: I0acca592c6909e91b28b904da49dcbd6a43cd2a5

search: make legalSearchChars() non-static

No outside callers from code in gerrit are using this statically

Change-Id: I9513a5384bca71f419a9af09908678bcc86c0263

HistoryPager: Unset 'date-range-to' parameter when paging

Follow-up to 1b27cee8979b2fefa65dafee237485787693eb98. This fixes
paging on Special:History, and matches the existing behavior for the
previously supported 'year'/'month' parameters.

It is possible that this should be unset in IndexPager. However, in a
quick test, other pagers that have filtering by date seem to function
correctly already (tested Special:Contributions and Special:Log).
I'm making this change in HistoryPager to avoid accidentally breaking
anything, but perhaps this should be reevaluated.

Maybe the real bug is that 'date-range-to' is not ignored when
'offset' is specified?

Bug: T220781
Change-Id: I01c309178e6d5d9c919f410c64e2b833a5750141

Merge "Fix docs for MWNamespace::clearCaches() removal"

Merge "Follow-up 4ee606f: Add note to RELEASE-NOTES-1.34 now we know where it landed"

Fix docs for MWNamespace::clearCaches() removal

Originally I had intended setMwGlobals() to magically reset the
namespace-related services if namespace-related settings were changed,
but Tim told me to remove it during review and I forgot to update the
release notes. Moving forward, everyone will need to reset services
after every config change to ensure correctness, and there's no point in
trying to reset specific services automatically as special cases.

I also moved the note to the 1.34 notes, since this missed the cutoff
and should not be backported.

Change-Id: Ib7cbdaef22a15ddfc7aaf99d0972b99d3cddc011

Merge "rdbms: inject reserved word table name rewrite logic into DatabaseOracle"

Merge "rdbms: optimize DBConnRef::getType() to avoid connections when possible"

Merge "resourceloader: Move expandModuleNames() to ResourceLoader.php"

rdbms: inject reserved word table name rewrite logic into DatabaseOracle

This works similar to that of DatabasePostgres, which uses similar rules.

Fix bogus field access in duplicateTableStructure() as well.
Also, remove a pointless wfDebug() call in DatabaseOracle.

Change-Id: Ia06ff78011dfabc17c525d7a6d2ffad98fe297de

dao: Correct return type doc for getLoadBalancer()

This method creates a load balancer object using services and calls
getMainLB() which returns an instance of ILoadBalancer and not an
instance of LoadBalancer, hence, getLoadBalancer() should return an
instance of ILoadBalancer instead.

Change-Id: I8859f293985d256d02ddc6f90e2aea813ebfc955

rdbms: optimize DBConnRef::getType() to avoid connections when possible

Change-Id: Ibe7f2ab173ba1457b6b11c4fe37e599962ae51b0

Follow-up 4ee606f: Add note to RELEASE-NOTES-1.34 now we know where it landed

Change-Id: I12dbff9dc9d7572d5f7caa057775044dffc92842

Merge "rdbms: mark DatabaseSqlite::checkForEnabledSearch() as public"

Merge "rdbms: add limitResults() to IDatabase"

Merge "OutputPage: Remove unused variable left out after a refactor"

Merge "HistoryAction: Amend `margin` in collapsed form state"

Merge "RCFilters: hygiene"

Merge "TemplateParser: Use operator short cut where necessary"

resourceloader: Move expandModuleNames() to ResourceLoader.php

This has always been an odd case, as indicicated by the cross-class
comment references, and the fact that its test cases are already
in ResourceLoaderTest.php, for convenience, as that's also where
the creation of 'module name strings' is done and tested.

Actually move it there instead of pretending it is there.

Change-Id: Ied9569436cc78704a5c1b75eeebb73f8631350f6

TemplateParser: Use operator short cut where necessary

Change-Id: I7f9dbe54cec876863920155ccd7cd735a3101481

OutputPage: Remove unused variable left out after a refactor

This variable was added in I0e18dc7fdbf133f3e400 (2017) and later
removed in I11b390f2e4f5e7db8b4 (2018) but the $sk was left which
now makes it used. Hence, cleaning this up.

Change-Id: I277b9de23409da3429a3b74d42e8ab9e9d5f5399

HistoryAction: Amend `margin` in collapsed form state

Also update comment at top.

Bug: T220555
Change-Id: I20b0f42d36a6909625bc9f481d0e016ae8cd1d96

Remove deprecated Language::setCode

Language::setCode was hard-deprecated in 76f5584377 included in
MediaWiki release 1.33.

Change-Id: I32f6d1c9b7ad781182945222d40fd209bce23229

Remove CryptRand and related stuff

Deprecated in 1.31 and not used anywhere.

Change-Id: Idc2e9cec907e39cacc391fdd7e2718bd880081ae

Merge "Selenium: fix 'Page should be editable' test"

HTMLFormFieldWithButton: Allow passing 'inputtype' and pass through 'buttonid'

Change-Id: Ib1c94ac66caf6243631aa9d2b9f7fd2e6ca7a367

Selenium: fix 'Page should be editable' test

The test is broken only when and targeting beta cluster, because an
extension adds text to page. Instead of checking the entire body of
the page, test just checks if expected string is there.

Bug: T217544
Change-Id: Ibd67790c1df2cd8556b9c71d872d52cfc18f5ebd

Merge "Remove Preferences class"

Merge "Add use Wikimedia\StaticArrayWriter statements to maintenance scripts"

Add use Wikimedia\StaticArrayWriter statements to maintenance scripts

Follows-Up: I022c074e8a708fb5219bc4ff4b53e7e31f60dc4b
Change-Id: Ieea214b31a6ba9cc8eb2b6bae50529d70448a872

Merge "Update Parser to use NamespaceInfo"

Update Parser to use NamespaceInfo

Change-Id: I668a51487786e4ab05a153ca3995388e79c13b42

Merge "Update PermissionManager to use NamespaceInfo"

Merge "Enable/disable Special:Block widgets according to block parameters"

Merge "Simplify checking for widgets on special block page"

Set MW_INSTALL_PATH in Maintenance.php

Otherwise, if running update.php, there are require_once failures from
extensions (at least Echo and Flow) if they aren't inside extensions/ --
e.g., if they're just symlinked from there.

Change-Id: Iaf4231abae1621627f01171f955b5bb7a0fa77d8

Merge "Rehabilitate DateFormatter"

rdbms: inject more dependencies into DatabaseOracle and remove $wgContLang use

Instead of guessing/assuming input encodings for non-UTF8 fields,
just require that they be valid UTF-8 to begin with.

Also replace MWException use and break some long lines.

Change-Id: Ibb6f841d5d143776eca2a9d0d12cbb74f1e0d23e

rdbms: mark DatabaseSqlite::checkForEnabledSearch() as public

Change-Id: Ic059cc986b813bc4e667077a4a5de41b9714f123

rdbms: add limitResults() to IDatabase

Change-Id: Iec39db0b34a48cce0367fb91acb234a81715d145

Merge "rdbms: reduce code duplication and make LBFactoryMulti sanity checks work"

Remove Preferences class

Deprecated in 1.31, the only extension using it is broken on modern MW
anyway.

Change-Id: Ifa0ea5ad5727254e60511fbc1889923e4bcc189a

Merge "Prepare for REL1_33 cut, labelling master as 1.34-alpha"

Prepare for REL1_33 cut, labelling master as 1.34-alpha

Change-Id: I841ee1c898a6db2e4a10b5e8c68647d45f046dc7

RCFilters: hygiene

Remove unused code, unneeded initialization
and fix documentation.

Change-Id: I82d7239b9003a82bc30484b01cf29e01c988b0d6

HTMLDateTimeField: Load styles for PHP widget

Bug: T220601
Change-Id: Id507459fea434bd6eb1af2bb6d1163c138a3a9d1

HistoryAction: Clarify form legend for edit history page

Bug: T107069
Change-Id: Ie20408da1ebed8de442338e67703b898a38e0624

Merge "tests: Mock the time in NameTableStoreTest to avoid failures if they run slowly"

rdbms: reduce code duplication and make LBFactoryMulti sanity checks work

Change-Id: I2308719b3a8fe4745c86dc7e5af0950588732ebe

tests: Mock the time in NameTableStoreTest to avoid failures if they run slowly

Bug: T220626
Change-Id: I4f8e0321f540b9ff48f3297a1ca0c6196c99ecf8

Merge "Drop clearCache() from (Configured)ReadOnlyMode"

Drop clearCache() from (Configured)ReadOnlyMode

These are services, so tests should just override services to reset
them.

Change-Id: Icb1a041f8fac1ea0b7421e69b7b31b24171d868b

Merge "Bump Mustache from 1.0.0 to 3.0.1"

Merge "jobqueue: add GenericParameterJob and RunnableJob interface"