Remove ObjectFactory

Change-Id: Ibfdb55101c893a6724e1f999eba85f2f8eabb79b

Split LocalFile.php to have one class in one file

Change-Id: Ic8e5220f2a1832dfc39f00720001235429ed2cab

Merge "resourceloader: Switch params for private addLink() function"

Merge "Move UploadStashFile to its own file"

Merge "Fix return type of UploadStashFile::remove()"

Merge "title: Allow passing MessageLocalizer to newMainPage()"

Move UploadStashFile to its own file

Change-Id: I557bcd8369014bf2ea43c5eb8336562cb6295890

Fix return type of UploadStashFile::remove()

Change-Id: Ib9da5827671a34798b58f7d736e4b663ac2c442d

Merge "Split logging classes to individual files"

title: Allow passing MessageLocalizer to newMainPage()

The method could not be used in session-less endpoints. This was
worked around once in the Startup module.

I plan to use this method in an extension module as well,
and would prefer not to duplicate core's logic for determining
the main page, outside this repository.

As general dependency-injection pattern, it seems desirable
to allow injecting a MessageLocalizer here.

Change-Id: I76cd02b2f489882e9404b93270f76aad9f0a4d9d

Merge "Remove PreferencesFormLegacy.php"

Merge "Avoid extra parse/save delay for users with non-canonical parser options"

Split logging classes to individual files

Change-Id: I3eaaf23612fe1aed65e49c06a9e5e565399af9ce

Remove PreferencesFormLegacy.php

Now unused, deprecated in 1.32

Change-Id: I4fc4bdf36b3832786c0266786cff57d512140d8e

Avoid extra parse/save delay for users with non-canonical parser options

If {{REVISIONID}} results in a re-parse, that re-parse will be post-send
unless the user has canonical parser options and will need the output for
page views anyway (e.g. the refresh after editing).

Also make getPreparedEdit() allow lazy-loading of the parser output via
a callback. A magic __get() method handles objects created the new way
but accessed by other code the old way.

Bug: T216306
Change-Id: I2012437c45dd605a6c0868dea47cf43dc67061d8

Remove unused constructor parameter and unused member variables

Make member variables private

Change-Id: Ie76f30a5c5731529256b2b160f6dde4d49646545

changes: Remove unused code from ChangesFeed.php

This class is only constructed by ApiFeedRecentchanges, which
only calls either '$feed->getFeedObject' or '::buildItems',
neither of which call into the remaining methods on this class.

Appears to be a left over from an old refactor, perhaps from
when it was split out into FeedItem/ChannelFeed etc.

Change-Id: I89e13af992fd091d714d6f74735f03a5bfe9d238

Merge "Replace PreferencesFormLegacy usages with PreferencesFormOOUI"

Merge "Remove LoginSignupSpecialPage.php from OneObjectStructurePerFile.MultipleFound"

Replace PreferencesFormLegacy usages with PreferencesFormOOUI

Change-Id: I4fc2a609ebb0f93b0767dddc343c3ff93ca8a39c

Remove LoginSignupSpecialPage.php from OneObjectStructurePerFile.MultipleFound

Change-Id: I145759b29d9e845d175cc7ac43ad72248b258df0

Split SVGReader to its own file

Change-Id: I9fc442225a37c14d0606508aed5ef496a5ad82ba

Merge "Split ApiImport.php to have one class in one file"

Merge "Remove PathRouter.php from Generic.Files.OneObjectStructurePerFile.MultipleFound"

Split ApiImport.php to have one class in one file

Change-Id: I3fbf5d271585722e95b7966ba62dd5e16fa3623f

resourceloader: Switch params for private addLink() function

I don't recall how this awkward parameter order came about,
but it easy to change given it is a private function.

Switching the order makes it cleaner to then internally re-use
for mw.loader.load() which currently contains a duplicate version
of this logic.

Change-Id: Id686389991315c3d05222f8fd0b69f93f65e9924

Remove PathRouter.php from Generic.Files.OneObjectStructurePerFile.MultipleFound

Change-Id: I9aa1eb7903dca3f63675c3be5a597829dc6d9ab3

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"