Stop calling FileRepo->streamFile()

Change-Id: Ifdf379dc7f2501e561dfbc06f9afa7a9089306f2

Merge "rcfeed: Replace usage of deprecated CommentStore::getStore()"

Merge "editstash: segregate stats by content type for more useful graphing"

editstash: segregate stats by content type for more useful graphing

Change-Id: Ice590a163ab7790993e44995220ca093f90c7c4f

Merge "rdbms: add getChronologyProtectorClientId() to ILBFactory"

rdbms: add getChronologyProtectorClientId() to ILBFactory

Returns the client ID used by ChronologyProtector.

Bug: T212550
Change-Id: Ia7d24d6591d6a8696103c560ea4b2c9718fd0ec2

Merge "Merge last-seen stash data at more points in SpecialWatchlist"

Merge "RCFilters: display timestamp of new changes in refresh link"

Merge "Gallery slideshow: Code cleanup"

Merge "Gallery slideshow: Fix height calculation"

Merge "Linker: Use parentheses by default in userToolLinksRedContribs()"

Linker: Use parentheses by default in userToolLinksRedContribs()

This function had been changed to unconditionally disable parentheses,
but none of its callers load the CSS needed to make that work.

Bug: T220767
Change-Id: I566d65e155258d3bd1a1a06bca9aa2b3a2d417ee
Follows-Up: I6eeeaa3b58d37adb7fefb4cc6915022229b3b324

Translate several special pages into Czech

Change-Id: I5e1e597af8ec62d536f0b041b568e961b96981c3

Gallery slideshow: Code cleanup

* Use $-prefix jQuery vars
* Use #connect for OOUI events
* Some jQuery cleanups
* Remove unused return valu of setImageSize
* Only resolve one value in loadImage
* Doc fixes

Change-Id: I0ff4252300aa02c228577961a7d1ede3b9628d90

Gallery slideshow: Fix height calculation

Remove $container check. This check was broken due to
recent changes in skin output (adding mw-parser-output),
and was not necessary, as the computation without it is
based on $imgContainer, which is also bound by a container.

This avoids the height being set to "NaNpx", or the full
height of the page.

Bug: T196723
Change-Id: I9b4fda9c71502bf749271c55a6945d9f4a4f913e

Merge "Change "e-mail" to "email" in Log-action-filter-newusers-byemail/en"

Localisation updates from https://translatewiki.net.

Change-Id: I03b16a02855278de5c424151cda244bbb2e97b09

rcfeed: Replace usage of deprecated CommentStore::getStore()

Change-Id: Icaa475a86becb7ef95304f5df31d3fdcf93168e9

Merge "search: Remove unused deprecated methods replaced by MW Services"

Merge "SearchEngine: Hard deprecate unused static methods deprecated in 1.27"

Merge "resourceloader: Omit non-existent messages in MessageBlobStore"

resourceloader: Omit non-existent messages in MessageBlobStore

Previously, it was exporting the result of $msg->plain() regardless
of $msg->exists(). This meant that, client-side, via mw.message
the message would always make mw.Message#exists return true, even
if in fact it did not exist.

Bug: T221294
Change-Id: I77122777ddaaa2d43f8385df9292540a8d21b328

search: Remove unused deprecated methods replaced by MW Services

The following methods deprecated in .27 and no longer used have
been removed;

- SearchEngine::userNamespaces()
  https://codesearch.wmflabs.org/search/?q=%3A%3AuserNamespaces&i=nope&files=&repos=

- SearchEngine::namespacesAsText()
  https://codesearch.wmflabs.org/search/?q=%3A%3AnamespacesAsText&i=nope&files=&repos=

- SearchEngine::create()
  https://codesearch.wmflabs.org/search/?q=SearchEngine%3A%3Acreate&i=nope&files=&repos=

- SearchEngine::getSearchTypes()
  https://codesearch.wmflabs.org/search/?q=%3A%3AgetSearchTypes&i=nope&files=&repos=

- SearchEngine::getNearMatch()
  https://codesearch.wmflabs.org/search/?q=SearchEngine%3A%3AgetNearMatch%5C(&i=nope&files=&repos=

Also, replace deprecated use of searchableNamespaces() in SearchEngine.

Bug: T220656
Change-Id: I330fbfea158f70b2328da7c850fdb833e39b6c7d

SearchEngine: Hard deprecate unused static methods deprecated in 1.27

Change-Id: I96c4454a2f89df0be5de20ca0506c3ce2c75cf7c

Merge "Fix the scripts to generate ucfirst overriddes"

resourceloader: Sync order of startup test cases with actual code

* Re-order the blacklist test cases to match the order of the
  regex for easier review.

* Give the variables better names in startup.js.

Change-Id: Id0498329d3e331d1b0babe9b1d79410ca1e4e7dc

Fix the scripts to generate ucfirst overriddes

* Fix a couple mistakes in generateUcfirstOverrides.php
* Skip surrogate codepoints as they can't be converted to json.

Change-Id: I5eddd90f7b348806d502dfbac08c367701d17f25

Merge "Add block type filter to Special:BlockList"

Merge "rdbms: add missing return value to LoadBalancer::waitForReplication()"

Merge "Simplify and unify the {{REVISIONID}} handling code in Parser"

Merge "Refactor edit stashing into a PageEditStash service"

Merge "resourceloader: Remove unused logic for WikiModule::getDB() as null"

Merge "build: Set Fresnel scenario run count to 7"

Localisation updates from https://translatewiki.net.

Change-Id: I76304ba54bedaae2d5f38c44bb9a01c250433dca

Refactor edit stashing into a PageEditStash service

Additional code cleanup:
* Call setCacheTime() in parseAndStash instead of relying
  on the one in DerivedPageDataUpdater.
* Improve the SPI logging by adding more extra fields.
* Treat requests in CLI/job mode (aside from tests) like
  those from bots with regard to checking the stash.
  This should avoid stats/logging pollution.

Change-Id: I8c6be919e399378e401a60502add0ecec7764d2d

resourceloader: Remove unused logic for WikiModule::getDB() as null

This was intended at some point for GlobalCssJs but ended up
being implemented differently once we realised that not having
any information about the remote wiki was unacceptable.

The local placeholder that points to metawiki (in case of GlobalCssJs)
must still at the very least be able to query version information
so that it gets a new version hash after changes etc. to ensure
cache rolls over.

This is being removed in preparation for refactoring where
compatibility with this broken model is no longer kept.

Change-Id: I4f1a06ace881daf1a5f202bdba0075460804de57

resourceloader: Move remaining module registrations to ServiceWiring

Also restore the order of registrations as it was before
last week with 47422fabe2813f. (That is, core modules are registered
before extension modules, in case of conflicts with a warning, the
core one wins).

Bug: T32956
Change-Id: I3a50508178159dfc8e5db1e218a5e6d10e2d4b2a

Merge "Use LinkTarget in Linker instead of Title"

Merge "Add ability to override mb_strtoupper in Language::ucfirst"

Add ability to override mb_strtoupper in Language::ucfirst

Different PHP versions treat unicode differently, and specifically some
wiki resources become unreachable if mb_strtoupper's behavior has changed.
This patch allows to introduce an override table that allows to smooth
the transition period.

It also provides maintenance scripts to generate such an override table.

Bug: T219279
Change-Id: I0503ff4207fded4648c58c7b50e67c55422a4849

Remove all $wgParser use from core

Bug: T160811

Change-Id: I0556c04d33386d0339e02e2bf7a1ee74d97c2abd

Add block type filter to Special:BlockList

Now that Partial Blocks feature is being enabled on multiple wikis
a new filter (Block Type) is added to Special:BlockList to facilitate
viewing the different blocks in place.

Bug: T191549
Change-Id: I0bc8c6eaff22930c2e3c12ae5e77e03287910db9

RCFilters: display timestamp of new changes in refresh link

The timestamp displayed in the link is not from the last refresh
but from when new changes are available.
I don't think this distinction is consequential. The fundamental
meaning is that clicking the link will bring changes newer than
the timestamp.

I considered using a relative timestamp (2h ago) like is
proposed in the task but I didn't think it was worth it
for the following reasons:
1. Client-side formatting with momentjs is hacky
   at best and I couldn't find a way to have it
   respect user's preferences.
2. I don't think it makes the meaning of the timestamp
   in the message any clearer. I'll admit the meaning
   of the current timestamp is not explicit but I think
   it works when you don't think too much about it.
3. A relative timestamp requires a setInterval with
   decreasing interval to keep it up to date.
   ("A few seconds ago" -> "2 minutes ago", "1h ago", etc)

Bug: T200353
Change-Id: I9058d47645914783227d1520adb78e75399b5504

Update OOUI to v0.31.4

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

Bug: T163142
Bug: T208547
Bug: T220041
Bug: T220204
Bug: T220227
Bug: T220248
Bug: T220840
Change-Id: Ic94a2f6790334baad7bc9c3055e990f7d3917e17
Depends-On: I0a2e152e61700e723a9ad7cd061682efaa9faf4d

Merge "api: Remove 'recenteditcount' set for BC to be removed in 1.25"

Merge "Remove usages of 'text' flag in revision-related getQueryInfo() calls"

api: Remove 'recenteditcount' set for BC to be removed in 1.25

This was a TODO that needed cleanup in mediawiki version 1.25 but
code was still hanging around. Removing it as we're in .34

Change-Id: I0d996362612495b3c523aed5af3a0e30ee4ed79c

Remove usages of 'text' flag in revision-related getQueryInfo() calls

Field rev_text_id will no longer be populated once the legacy
schema is disabled, so joins against it will not work.
Remove all usages of the 'text' flag in calls to both
Revision::getQueryInfo() and RevisionStore::getQueryInfo()
so that these joins are no longer attempted.

Bug: T198342
Change-Id: I9be6a544c6f68555d4ea856f949f0040d05eac0f

Merge "changetags: $param type should be string and not array"

Localisation updates from https://translatewiki.net.

Change-Id: Ief398fcf2575c4ff49e6db20f7f4c3a64b7209d8

changetags: $param type should be string and not array

Per the call to updateTagsWithChecks(), $params should be a string
and not an array as soft type hinted. The previous calls updateTags()
which uses $param as a string or null.

Change-Id: I87fa39682d91b7cb3b0844d0aabbfaea0cb6f206

Merge "context: Replace deprecated use of SkinFactory::getDefaultInstance()"

Merge "diff: Wrap condition around brackets for more readability"

Simplify and unify the {{REVISIONID}} handling code in Parser

Improve documentation for Parser::getRevisionId().

Change-Id: I3cb8721e3bc2e3a06c3158cd60742bc10a458f20

Merge "resourceloader: Remove 'wgMainPageTitle' mw.config key"

rdbms: add missing return value to LoadBalancer::waitForReplication()

Change-Id: Idcb5ee63279746160c0ee4ac1e8a242876e66906

Merge "clientpool: Remove unused parameter from constructor header"

Merge "resources: Rename a few local 'filterFn' vars to 'filterFunction'"

resources: Rename a few local 'filterFn' vars to 'filterFunction'

Abbreviations are discouraged, so let’s use the full word.

Change-Id: I31fb4cca32d766559005d1291d6d22da3d74b5e6

Merge "Allow fragment-only TitleValues"

Merge "Normalize input to TitleParser::parseTitle()"

Merge "New Title::castFromLinkTarget/TitleValue"

Merge "Update MediaWikiTitleCodec to use NamespaceInfo"

Merge "Rename un-prefixed rcfilters selectors"

Merge "Tweak margins for collapsible OOUI HTMLForms some more"

Merge "JsonContent: Remove never-used caption styles"

Merge "resourceloader: Use 'fallback' as default for ResourceLoaderTestCase"

Merge "resourceloader: Declare ResourceLoaderImage fields explicitly"

Merge "Remove obsolete `color` property from horizontal rule"

Merge "Set global config for test to avoid failure"

clientpool: Remove unused parameter from constructor header

Per https://codesearch.wmflabs.org/search/?q=SquidPurgeClient&i=nope&files=&repos=,
constructing an SquidPargeClient requires only the $server argument and
doesn't make use of $options yet. Maybe this was a WIP but for now, it's
never used, so cleaning it up.

Change-Id: I2d86ac6b17ac8dce85c05887de8eee4bb3dd28a5

Localisation updates from https://translatewiki.net.

Change-Id: Ida17cce17aa1b580a88aa9c767362e739364d1e3

Tweak margins for collapsible OOUI HTMLForms some more

* Move the code from mediawiki.action.history.styles, as it's
  theoretically applicable to other similar forms
* Fix it so that it actually only applies when JS is available
* Make top and bottom margins/paddings even

Follow-up to 32bca54dc94b1befcf21483d9f08017a91ad73eb.

Change-Id: Ifde2303f78f971f1e82418c9bd5feddbd6f1f25d

Merge "specials: Add $subpage param to RedirectSpecialPage::getRedirectQuery"

Merge "db: Use $fname instead of __METHOD__ magic constant"

context: Replace deprecated use of SkinFactory::getDefaultInstance()

Replace with MediaWikiServices::getInstance()->getSkinFactory()

Change-Id: I2cf7afad249e52e13a56fbe92b1665dfebec793f

resourceloader: Use 'fallback' as default for ResourceLoaderTestCase

Follows-up fa05976f5f. This broke the Travis CI tests because
the Vector skin is not installed there (unlike in Wikimedia CI).

Make the test no longer reliant on that detail.

* Fix ResourceLoaderTestCase to use a default that actually
  works in plain core (previously 'vector' would be rejected
  as being an unknown skin, but then be re-normalised back to
  'vector' thanks to wgDefaultSkin. This was fixed in fa05976f5f,
  and the unknown skin now produces 'fallback' which is better.

* Update ResourceLoaderOOUIImageModuleTest to match this
  new default and also fix the test message while at it
  to mention 'skin', not 'image'.

Change-Id: I5fb69a6a38a42b1a5f325c0134e01ad880f65087

Merge "specials: Remove invalid return from RedirectSpecialPage::execute"

Merge "Fix calls to Title::getUserPermissionsErrors on SpecialImport"

Merge "resourceloader: Remove needless MW-specific fallback"

Merge "parser: use "-" for revision ID for non-preview edit filter parse during save"

Remove obsolete `color` property from horizontal rule

Removing `color` property definition, which dates back to IE 5/Mac
support times.
`background-color` is sufficient in any of our basic supported
browsers.

Change-Id: I013b9c5bc614299c94bb733fc724148e5c7b1343

resourceloader: Remove 'wgMainPageTitle' mw.config key

This was added in 2009 with cf75bdf4ba (r54384) and moved to the
startup module in 2010 with a99f9ec28b9 (r72772).

It is no longer used by any extension in Wikimedia Git, nor
elsewhere indexed by MediaWiki Codesearch. It also no longer has
any uses in mwgrep from public WMF wikis.

Bug: T220926
Depends-On: Ifdedccb513f5dd4636a37badc329ac5f1c4dd943
Change-Id: Id2729653cd450d31c1d903358ccf5e4fac564228

resourceloader: Remove needless MW-specific fallback

This is never used by the mw.loader client code. For any
manually crafted requests or fringe usage where the load.php
might be used to fetch some raw JS code that doesn't vary by
language or skin, proceed with qqx/fallback.

In the future load.php might deny these requests and then we
could make this a constructor parameter to ResourceLoaderContext.

Bug: T32956
Change-Id: I4e4ee758cd22278cea9592d4745b4f7fc00e0add

Merge "selenium: Replace regex matching with includes()"

Merge "ContibsPager: Fix slow query with actor migration read-new mode"

selenium: Replace regex matching with includes()

Bug: T217544
Change-Id: Iedcbb1f7e21fc768069346ac346c2e499a689556

ContibsPager: Fix slow query with actor migration read-new mode

The change in I1678f7ec broke the workaround in I992aa50f by adding
the rev_id column to the ORDER BY clause. We need to extend the aliasing
to rev_id => revactor_rev to re-fix it.

Bug: T220991
Change-Id: I62dc3803621da982e526d7402ab8e7b08f8336bc

resourceloader: Declare ResourceLoaderImage fields explicitly

Change-Id: I4153cbf6080dabd29f6fbe1fb546dc10957d780e

Merge "resourceloader: Re-use internal addLink() for mw.loader.load()"

Rename un-prefixed rcfilters selectors

Only -head and -container have on-wiki usage, so
keep deprecated aliases for those.

Bug: T220852
Change-Id: I67b120d67fbbdf108da98ddfc7eae24e668bfe11

Merge "JsonContent: Add missing prefix to selector"

db: Use $fname instead of __METHOD__ magic constant

The insert() method calls insertOneRow() method on line 391. It then
passes $fname which defaults to __METHOD__ in its function header and
not using $fname instead of __METHOD__ in insertOneRow() method makes
this parameter unused which of course should be used.

Change-Id: Ic5f5ee878e4f919a96900ba88755641ac85929df

specials: Remove invalid return from RedirectSpecialPage::execute

Follows-up f739a8f368a, and 22c9aa5ec07.

The execute() method here should generally be void, with the
exception of includable special pages, where the return value
is a string of HTML.

This was likely left-over early iteration of commit f739a8f368a,
which ended up returning from getRedirect() instead.

Change-Id: I68715910114e19a0421625e814fd8881068f6406

specials: Add $subpage param to RedirectSpecialPage::getRedirectQuery

This will make it easier to create redirects where $subpage is the title,
e.g. "Special:Example/Foo?x=y" to "index.php?title=Foo&x=y".

To do that conveniently, getRedirectQuery() needs access to $subpage.
The alternative is to do Title-parsing inside getRedirect(), which then
complicates this significantly as one has to deal with absence of a title
(null) and invalid titles (illegal chars etc.).

By using it plainly as query parameter (defaulting to null/omitted), this
is all deferred to index.php, which seems like a better separation of
concerns.

Motivated by SpecialMobileHistory in MobileFrontend (Ic0aea7ee340a).

Change-Id: I9fe78f479053fb55952ba78850d2fc281a039fe3

Fix calls to Title::getUserPermissionsErrors on SpecialImport

Right now, it uses a deprecated way to set rigor causing it to read from
master on GET, giving out errors

Change-Id: I89bbe20080998bde2156680c631700b52f4e4712

Use LinkTarget in Linker instead of Title

Bug: T214318
Change-Id: I60b6208fa5b45a568e81f908a19cd0f244ef79be

Allow fragment-only TitleValues

This matches behavior of Title, and is meaningful for creating links.
However, in other contexts such a TitleValue doesn't make sense, so
perhaps we want to write special cases in linking code instead.

Change-Id: I812ad9090463766f38ef677de318c006912b4cca

Normalize input to TitleParser::parseTitle()

This matches behavior of Title::newFromText(), which assists porting old
code to TitleParser.

Change-Id: I50e9af09df843ea575250e276c7cfce660c00efd

New Title::castFromLinkTarget/TitleValue

These behave the same as newFromLinkTarget/TitleValue, but accept null
as well (and then just return null). This makes things much easier when
converting code from using Title to LinkTarget, because you can wrap in
castFromLinkTarget without adding null checks.

Change-Id: Id61f91d40b81ad226532917c43e51f0b69af712c