Merge "Remove single-item HTML list for re-upload link"

rdbms: remove references to SQL_CACHE/SQL_NOCACHE

Change-Id: I4bace17f7f5171eea9bbf13e7f40fd7027a1ce35

rdbms: update LOCK IN SHARE MODE in IDatabase to reflect the level of support

Change-Id: I45eb27ffd105e9ca2c1bd9967c6db2719c835b27

rdbms: remove HIGH_PRIORITY/LOW_PRIORITY SQL references from IDatabase

These should not be used even with MySQL as they do not help concurrency

Change-Id: I5385fcbe65e676b184268286bedd5c2299b447d4

Merge "DevelopmentSettings: Clarify grouping of settings and purpose"

Remove single-item HTML list for re-upload link

The 'upload links box' only contains a single link (or message),
so there's no need for it to be a list. This also adds a top-level
class to this box, for easier CSS/JS selecting.

Change-Id: I3967eea82dcd63a3e161daa13e49204e4d6dde97

Merge "Create Special:NewSection special page"

Merge "ChangesListSpecialPage: skip associated for namespaces that don't have one."

Merge "resourceloader: Consider 'packagesFiles' in FileModule's canBeStylesOnly"

Merge "filebackend: deprecate FileBackend::getWikiId()"

Merge "objectcache: fix failing tests for non-HashBagOStuff backends"

Merge "Clean up use of IDatabase::affectedRows() in WikiPage::updateRedirectOn()"

Merge "ChangeTags: turn private getPrevTags() into public getTags()"

ChangesListSpecialPage: skip associated for namespaces that don't have one.

Bug: T229954
Change-Id: Ic69989c3ac60a86d58842ec34dc9e883988d785f

Merge "rdbms: allow automatic PDO creation of SQLite database files"

Merge "objectcache: avoid using deprecated phpredis::delete() alias"

resourceloader: Consider 'packagesFiles' in FileModule's canBeStylesOnly

A module with packageFiles should never be considered a styles-only
module.

Change-Id: I92f0d2f31656858d8dd00c73c4d9d7da329582a4

Merge "Add talk namespace option to Special:NewPages"

DevelopmentSettings: Clarify grouping of settings and purpose

Change-Id: Ia0fc063eb2679b466e19e6dabaf1f4bea7d6bb20

Merge "Revert "Add small HtmlCacheUpdater service class to normalize purging code""

filebackend: deprecate FileBackend::getWikiId()

Change-Id: Ib593178e95c572c28c873645128ef85513ef2de7

objectcache: fix failing tests for non-HashBagOStuff backends

Use real time for testing absolute expirations with changeTTL().
Otherwise, backends like memcached or redis will fail since
they do not use the mock time.

Also:
* Make SqlBagOStuff actually override changeTTLMulti() by
  using the right method name
* Check TTL_INDEFINITE more explicitly for clarity
* Rename TTL conversion methods for clarity
* Use isRelativeExpiration() in MemcachedBagOStuff

Change-Id: I9365ceb31d4e7bef65906363d42b8c3020a66346

Clean up use of IDatabase::affectedRows() in WikiPage::updateRedirectOn()

Bug: T229456
Change-Id: Ibce207904375a762943dff3352db7affd13c1aaa

Revert "Add small HtmlCacheUpdater service class to normalize purging code"

This reverts commit 35da1bbd7cb8b4414c4fbcf331473f1024bc638d.

Reason for revert: wrong tab, wrong patch. Ooops.

Change-Id: I5828fff6308d43460a3b2b10f60996409181f8b3

Merge "Add small HtmlCacheUpdater service class to normalize purging code"

ChangeTags: turn private getPrevTags() into public getTags()

This makes the most convenient way to get all the change tags for a
given recent change, revision, and/or log entry public. The database is
injected as an additional parameter so that callers can choose to read
from a replica, and the revision and log ID parameters are swapped for
consistency with other methods in the same class. Also, while we’re
already touching the method, let’s extract the change_tag_def store.

Change-Id: Id8f14044ba5d926447aedd60e8cc8eb7dc864899

objectcache: avoid using deprecated phpredis::delete() alias

Change-Id: I3ca8bd9160eefff6590228082f030a32d0edb511

Localisation updates from https://translatewiki.net.

Change-Id: I75e924d7b9d088cd8f705b56c90e038e6407cadd

rdbms: allow automatic PDO creation of SQLite database files

Define missing DatabaseSqlite::doSelectDomain() method to handle attempts
to change the database, prefix, and/or schema.

Also add sanity check to serverIsReadOnly() to make sure open() was called

Change-Id: I72c25bf4dab5e01def3fb9472217e7637aede1d4

Merge "Fix param doc of ChangeTagsList::updateChangeTagsOnAll"

Merge "Pass message arguments as array to ErrorPageError"

Merge "mediawiki.page.gallery.slideshow: Avoid manual Deferred wrapping"

Merge "Add type hints for class properties in SpecialVersion"

Add small HtmlCacheUpdater service class to normalize purging code

The purge() method handles purging of both file cache and CDN, using
a PRESEND deferred update. This avoids code duplication and missing
file cache purge calls.

Also:
* Migrate HTMLCacheUpdate callers to just directly using HTMLCacheUpdateJob
* Add HtmlFileCacheUpdate class and defer such updates just like with CDN
* Simplify HTMLCacheUpdate constructor parameters
* Remove BacklinkCache::clear() calls which do nothing since the backlink
  query does not actually happen until the job runs

Change-Id: Ic453b189a40109a73a9426538608eea87a76befa

Fix description for $wgGalleryOptions in DefaultSettings.php

Falsy change from Ib0d92e128e7b882bb5b838bd00c74fc16ef14303

Change-Id: Ia66be38c26f1bfa3da5c8100e59f986fe1f87c36

Merge "revisionstore: clean up redundant DB connection code in RevisionStore"

mediawiki.page.gallery.slideshow: Avoid manual Deferred wrapping

* Use then(), which automatically forwards errors, instead of manually
  forwarding fail() to d.reject(). This also ensures that any
  errors in the done() or fail() handler themselves are caught and reject
  the returned Deferred – instead of an causing the Deferred to indefinely
  be in the "pending" state.

* Use then(), which makes the code consistently asynchronous,
  instead of sometimes sync and sometimes async.

* Use then() for forward-compat with native ES6 Promise.

Change-Id: I58fb6ec23fb4056fe2bbb146ff4225dea9d9f379

Merge "watchlist: cleanup WatchedItemStore PhpStorm warnings"

watchlist: cleanup WatchedItemStore PhpStorm warnings

Change-Id: I21bcc41781744f0fd323b8c46f4a71b4fc040eff

Follow-up I3d56d5db5: Add release notes entry

Change-Id: Id836481ade41fdc0ea9951574677407f73f2611a
(cherry picked from commit cf21cda12cf03f5532b103bc7678fba6299afaed)

Merge "mediawiki.page.ready: Simplify logout code"

Merge "Clean up spacing of doc comments"

Merge "ResourceLoaderClientHtml: Simplify code"

Clean up spacing of doc comments

Align the doc stars and normalize start and end tokens

Change-Id: Ib0d92e128e7b882bb5b838bd00c74fc16ef14303

mediawiki.page.ready: Simplify logout code

* Use location.href instead of window.location.
  (Better static analysis, and theoretically quicker lookup)
* Initialize variable on declaration.
* Use .then() instead of .done().fail().

Change-Id: If007ae9bad37461cf1a1b51bd4d21281d7778254

Merge "resourceloader: Move mw.libs and mw.widgets from startup to mediawiki.base"

Merge "resourceloader: Merge mediawiki.legacy.wikibits into mediawiki.base"

Merge "resourceloader: Minor optims in startup JS"

Merge "Improve log entries display in change lists"

Localisation updates from https://translatewiki.net.

Change-Id: Id08ec6c7c9efffd82de8cae27fdd9fa6e0e0f900

Merge "Add explanatory comment to unexpected line of code"

Improve log entries display in change lists

Log entries are wrapped with parenthesis - not hardcoded into HTML
causing stray brackets in Minerva where this element is floated.
Has no impact on other skins

In addition to this the entire log entry item is wrapped in a new
span mw-changeslist-log-entry to ensure the text is recognised
as a whole entity - on Minerva floats were leading to the action
and the action object being separated (e.g. uploaded and "File:Name
were separated

Bug: T229360
Change-Id: Idae997b15f68bb0dfcaa857f7458ba3497233d41

Merge "move rcfilters scroll logic to better support mobile mode"

ResourceLoaderClientHtml: Simplify code

* Use nowdoc instead of heredoc syntax where variable expansion is not
  wanted.
* Combine code paths for debug mode and normal mode.

Change-Id: Ic70d82aaec6396a31783e5a60419409f08c4ef73

Add explanatory comment to unexpected line of code

Bug: T229360
Change-Id: Ia3faab708925c4ea98ebc91cda79ba82ce8524e8

Merge "rcfilters mobile cleanup"

Merge "Code style cleanups to FileBackendGroup"

Merge "Make LinksUpdate no longer extend EnqueueableDataUpdate"

Make LinksUpdate no longer extend EnqueueableDataUpdate

LinksUpdate does not match RefreshLinksJob since the former is only a subset
of the later. Also, DeferredUpdates::doUpdates() only runs in "enqueue" mode
for cases in MediaWiki::restInPeace() if there is no post-send support.

In a future commit, the deferred callback in which LinksUpdate runs
currently, will be abstracted into its own deferred update, which
will then bring back EnqueueableDataUpdate for this update.

Bug: T206283
Change-Id: I0680be445e8b8e8d0dba85df135b84640f4fcb81

move rcfilters scroll logic to better support mobile mode

Moves the logic that scrolls the rcfilters search input into view
from the onInputFocus callback to the onMenuToggle callback.

This allows the scroll behaviour to be triggered on mobile when
you tap the active filters area, and prevents the scroll behaviour
from triggering when the menu closes on mobile.

Bug: T229360
Change-Id: I1ca5c59e16a5dc6c6f473bbf3825a3e7c1e0f28e

Merge "OutputPage: Remove 'html5shiv' module from RL registration"

Merge "Remove old comment and unused setting"

Merge "Avoid error in querypage API when there's no value field"

Merge "rdbms: update ILBFactory comments and simplify LoadBalancer "load" code a bit"

Merge "rdbms: In the query log, show the server name in the message"

Remove old comment and unused setting

Change-Id: I813c90197ba0fc76d493a47f83107ba4bb8dff2a

Merge "selenium: stop running extensions/skins specs"

rdbms: update ILBFactory comments and simplify LoadBalancer "load" code a bit

Change-Id: I391a2f873c623b3086703a9b078def40363d9520

rcfilters mobile cleanup

- ensure the advanced filters are hidden when clicking the "hide/show" button
- prevent horizontal scrolling of the "saved filters" and
  "other review tools" buttons on small screens.

Bug: T229360
Change-Id: I445df9124a6bb520dcf49b245f0a8392bf723c08

Localisation updates from https://translatewiki.net.

Change-Id: I8dfd153ca5be10f42e616ed11039f8e3fc8b7e46

revisionstore: clean up redundant DB connection code in RevisionStore

Change-Id: If9c2391046936862bee05cfbd8cdc9440347071c

Merge "changes: Avoid use of deprecated method Revision::userCanBitfield()"

changes: Avoid use of deprecated method Revision::userCanBitfield()

Revision::userCanBitfield() is deprecated and RevisionRecord should
be used instead.

Change-Id: I0d6ad6863be520d3264809be69532dfb2e3dab9a

resourceloader: Merge mediawiki.legacy.wikibits into mediawiki.base

Still executed under the same conditional, but no longer exposed
as its own public module bundle.

Change-Id: Ifba3a73b184ce02eeeeb2ccce6d4aece732dea13

OutputPage: Remove 'html5shiv' module from RL registration

Follows-up 66a011797d8e2fd, which changed the reference to this
JS file to be without the indirection of ResourceLoader.
It's been deployed well over the needed 7 days, so this can be
removed now.

Change-Id: I823c0b31c4478e5e34f4191d851b6a9c83a6019b

Merge "resourceloader: Remove unused 'getBaseModulesInternal' internal method"

Merge "Use `text` element for current content model on Special:ChangeContentModel"

Merge "Convert maintenance scripts to LoadBalancer::getMaintenanceConnectionRef()"

Convert maintenance scripts to LoadBalancer::getMaintenanceConnectionRef()

Change-Id: I8944a052f51a1941ad3b76a40fc9c46d1363c426

Use `text` element for current content model on Special:ChangeContentModel

Change-Id: I6881154fa5baf4d55634c691c45596f49e2cd4b0

Move WANObjectCache to wancache/ instead of WinCache

Follow up 6ca3de5fa0.

Change-Id: I7021dfe5c900f810ab3b6dc701a04e2c3e1c4c66

Merge "Show current content model on Special:ChangeContentModel"

Show current content model on Special:ChangeContentModel

Bug: T226914
Change-Id: I60c7075751cb180cb86cde0b9caea1a4bcf9b2a3

Merge "Move WAN cache classes under a new wancache/ directory"

resourceloader: Remove unused 'getBaseModulesInternal' internal method

I added this as wrapper for the private 'getBaseModules' method
in commit dec800968eb6 for "use by SpecialJavaScriptTest", but
I never did. And it doesn't need it so.. let's remove that.

Change-Id: Ia301bd1c1f3a15098b9e6cd2934dd8e80879dd50

Reorganize ResourceLoader fields

Change-Id: I9c9aece69b869165b8d9037336c8a2e7d7189c1e

resourceloader: reorganize ResourceLoaderModule fields

Also use AtEase for error suppression to avoid phpstorm warnings

Change-Id: Iaa8ca107f4e7c923e894416b40b4fadd2b3d162e

Move WAN cache classes under a new wancache/ directory

Change-Id: I6837761ebca7557e029e1f65beca738266e48efb

resourceloader: Minor optims in startup JS

Change-Id: I3bb6e77e3572c212ea32838f7668940b5abc99d8

resourceloader: Move mw.libs and mw.widgets from startup to mediawiki.base

These are not needed to help define mw.loader or to process RLQ,
as such, it does not need to be in the critical path. This can
instead download as part of mediawiki.base which loads concurrently
with regular modules (but will execute first).

Change-Id: I43f21706206aa7cb6fb779c68325623e07724125

Avoid error in querypage API when there's no value field

QueryPage does not require a value field, and post I8b67268e omitting
it might even be useful, so the API should not error out when it's
not present.

Change-Id: I83b49d4451ba71bb2161459bddf17b9ac00088d2

Create Special:NewSection special page

Bug: T207577
Change-Id: If69a8e13b524f55f78b2f61aadf1f9045fda5ff2

Merge "Make Special:MovePage note about redirects dependent on content model"

Merge "Deprecate SearchEngine:getTextFromContent() and SearchEngine::textAlreadyUpdatedForIndex()"

Merge "Simplify SearchUpdate constructor and hard deprecate some param types"

Merge "resourceloader: Merge $fileCache conditional blocks"

Merge "Document SearchDatabase::doSearchTextInDB to return null"

Merge "Deprecate SearchEngine::userHighlightPrefs()"

Merge "Fix bogus field use in RevisionRenderer::getSpeculativePageId"

Code style cleanups to FileBackendGroup

Set domainId in register() instead of on every config() call.
Use array_merge() to make the override precedence clearer.

Change-Id: I8792281cd9f1a4018255f9a9b87126c909095e68