Merge "Fix escaping for MSSQL LIKE queries."

Fix escaping for MSSQL LIKE queries.

MSSQL allows for more operators than standard LIKE queries. In addition,
an ESCAPE clause must be specified in order to backslash-escapes to work.

Bug: T73207
Change-Id: Idadf9d56cadc48cf47d000598d8a3214c684f9d5

Merge "Add X-Robots-Tag header to Special:Export dumps"

Localisation updates from https://translatewiki.net.

Change-Id: Ib552061b1052fae52ba0a3c3cba6dfa6a9f20ea0

Merge "Align to CSS coding guidelines and fix CSSLint errors & warnings"

Add X-Robots-Tag header to Special:Export dumps

Adds "X-Robots-Tag: noindex,nofollow" header to Special:Export
dumps so that dumps are not indexed by search engines.

Bug: T91363
Change-Id: Ia96ec40ff185fdfd024312ee3cfe3c736ba8d806

Merge "Namespace LinkTarget under MediaWiki\Linker"

chmod 664 SpecialSearch.php

Change-Id: I860a4439e34280f19dcd2c7b2a0ce7951503c262

Localisation updates from https://translatewiki.net.

Change-Id: I08afd3e2ef249cd2392012537592f33703cbfe4d

Merge "Make HTMLCheckField::loadDataFromRequest always return a boolean"

Make HTMLCheckField::loadDataFromRequest always return a boolean

Bug: T133163
Change-Id: I4592d9a44c6ca4d3f680a53ecfd628f385a65c09

Merge "Format count of files on Special:ListFiles as number"

registration: Explain skipFunction more succinctly

Don't use the RL documentation for skipFunction which is wordy and
not helpful, to be in keeping with the rest of this file.

Change-Id: Ic72672be94537176a9aa0a138052137c9715f7cf

Format count of files on Special:ListFiles as number

Change-Id: I712009a0f58e9ebfea6bbfbf401f5e2284714e31

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

Localisation updates from https://translatewiki.net.

Change-Id: Idcbc72b595213f1c7637525cfe72eaa88053dd80

Merge "Fix EventRelayerKafka IDEA errors"

Fix EventRelayerKafka IDEA errors

Change-Id: I2808cb3cefb4d97f5ae2650554076c92b0d877d9

Merge "HTMLForm: Don't render 'mw-htmlform-submit-buttons' if there aren't any buttons"

Names.php: Replace remaining tabs and double spaces

Follow-up to a3552004
Change-Id: I6a76de73fcfc21516fab7415057260e27e924696

Fix GENDER support in logentry-rights-rights

Introduced in 70f0b0e588e40a18ce53c56a59efcc740f2766d2.
Got to core by my +1, sorry

Bug: T52583

Change-Id: I328fadacf7e5e3b7b250b76556ed585eeba424ca

Merge "BagOStuff: Optionally log duplicate key lookups"

HTMLForm: Don't render 'mw-htmlform-submit-buttons' if there aren't any buttons

Visually, this only affects OOUIHTMLForm, in other kinds of forms
the element has no height anyway.

Change-Id: I90e73c49fbefec23532368848bb30b2e7b69075c

Merge "Have Linker::linkUrl() accept LinkTarget"

Merge "Add LinkTarget::createFragmentTarget() function"

Merge "RELNOTES for Skin::__call unroll & removal"

Merge "Remove unused Skin -> Linker deprecated methods"

Set an alias for TrackingCategories page in Czech

Change-Id: I0c97cf6ce6aa342a50b667b06dc2e78175d9fbc1

Unify the spaces before the comment in Names.php

Change everything from a varying number of tabs
to a single space.

The tabs don't really help make it more readable.

Change-Id: Ie84ee675ce4a50f78ec67d3840edc1a19f2c7884

BagOStuff: Optionally log duplicate key lookups

Add new 'reportDupes' parameter to BagOStuff. This parameter enables scheduling
of a callback after the current web request ends. This callback will emit warning
log messages for all keys that were requested more than once.

The default ObjectCache factory for MediaWiki enables this option by default.
Not by default for plain BagOStuff instances, however. E.g. `new HashBagOStuff()`.

It also set 'asyncHandler' for all classes now (not just MultiWriteBagOStuff).

Bug: T128125
Co-Authored-By: Timo Tijhof <krinklemail@gmail.com>
Change-Id: I8a2b06cf54d2acf5950eed71756ecdf50e224be1

Merge "Add missing tests for some edge cases in WatchedItem::getNotificationTimestamp"

Merge "Remove unused JobQueueRedis vars"

Namespace LinkTarget under MediaWiki\Linker

And add a GPL file header while we're at it.

Change-Id: I15a6f240124c879b21fb655ade1baaebf4f34ffd

Have Linker::linkUrl() accept LinkTarget

This is a private function so it doesn't really make much of a
difference, but will allow further refactoring.

Change-Id: Idacad7b1bd4776c04659ed9af64540d6ce4b2f28

Add LinkTarget::createFragmentTarget() function

The createFragmentTarget function allows for switching the fragment on a
target in an easier way. TitleValue already had a now-renamed
createFragmentTitle function (no uses outside of tests), and an
implementation was added for Title.

This will also help with reducing the amount of public usage of
Title::setFragment(), which is deprecated.

Change-Id: I1e8ba2f85e748b1b4394fb2f2a1ccce69cf6e3c5

Align to CSS coding guidelines and fix CSSLint errors & warnings

Fixes included are:
- Coding guidelines: align colors
- Coding guidelines: put every selector of combined selectors on
one line
- Coding guidelines: use `border: 0` instead of `border: none`
and put border shortcut structure into same order across selectors
for better gzipping
- Coding guidelines: whitespace in parenthesis
- CSSLint: disable applying `margin` for inline elements
- General: clean-up comments
- General: join pagination and 'Generic HTML elements'' rules
- General: add `word-wrap: break-word` to `pre, .mw-code` selector
to aggressively wrap long words
- General: swap `1px` border values with `1pt`
- General: integrate `word-wrap: break-word` on `.external` links
with possible overlong URLs
- General: do the `list-style` shorthand property right.

Bug: T120154
Change-Id: I29bf055e2b4fc0e26ce84243f6b612b4a6214b02

Merge "TableDiffFormatter: Don't repeatedly call array_shift()"

Merge "Improve getErrorsByType() docs"

Localisation updates from https://translatewiki.net.

Change-Id: If37acfa27b8f6648209998996ebef2e8bbe83622

OOjs UI: Correct display of MenuToolGroup checkmarks

Pulling I6b2eeffd forward from next week's release.

Bug: T133227
Change-Id: Ic7adcb78c50ec89071f1efb4e35f42a3809f874b

Remove unused JobQueueRedis vars

Change-Id: I953df3daa2ec6b667971a8acd69ff59259d7ab55

Merge "mediawiki.api.upload: Unbreak finishing stash uploads"

Merge "Catch JobQueue errors in SiteStats::jobs"

mediawiki.api.upload: Unbreak finishing stash uploads

The upload would complete successfully, but the upload
dialog would just hang for the user.

Follow-up to 11a2791668260211f1b82ec4954082533ff3adee.

Bug: T133295
Change-Id: I92724c6356d7dbf5c4cb87c05366bb7ed1ec8587

Merge "Add getStatsdDataFactory to MediawikiServices"

Add missing tests for some edge cases in WatchedItem::getNotificationTimestamp

Change-Id: I18dd5976cc0f44f9dca8ee420ace226e49ab6288

Make MessageCache handle lock timeouts better

* Timeouts happen when memcached is cleared or on DC-switchover likewise
* Disable the sidebar cache when the MessageCache is disabled
* Also lower any output expiry for CDN and set a custom header
  when the MessageCache is disabled
* Log when this happens to a dedicated "MessageCacheError" channel

Bug: T133069
Bug: T7092
Change-Id: I1b80e250532033c52680246279e1e8f56350ae83

Merge "SessionManager: Ignore Session object destruction during global shutdown"

Merge "Add LinkTarget::hasFragment() helper function"

Merge "Remove unused wfGetDB() in SiteStats::jobs"

Add LinkTarget::hasFragment() helper function

LinkTarget::hasFragment() is a helper function which returns a boolean
of whether the target has a fragment. Title already had such a function,
and one was added to TitleValue.

Co-Authored-By: addshore <addshorewiki@gmail.com>
Change-Id: I49e607ae5a58c3aef96d0246297740e7d88ac816

Localisation updates from https://translatewiki.net.

Change-Id: I910ce04ca2372c9b7ac534664464b4edca6f759f

Merge "Add missing visualClear on search result page"

Merge "Create Kafka event relayer"

Create Kafka event relayer

Bug: T125138
Change-Id: I9d7705cb164bc975c3a0ddf4a33ac54fe7de931c

Fix ExternalStoreDB IDEA errors

Change-Id: I6200173fd1cba4bdf03b61659a48016107672fad

Catch errors in pushLazyJobs() and log them

This happens post-send anyway, so throwing errors is not
very useful and the other post-send updates are independent.

Change-Id: Ife8f9825da9592be1a38cf4c757550213838463b

SessionManager: Ignore Session object destruction during global shutdown

We already save all open SessionBackends when shutdown handlers are run,
which *should* make the Session object destructors that run during
global shutdown not have anything to save. But it can get fooled if the
Session data contains other objects that have already gotten destroyed
during the global shutdown, leading to spurious warnings and errors as
it tries to access partically-destroyed objects.

The solution is to set a flag when we do the shutdown handlers and just
ignore the last gasps from Session::__destruct() that might come after.

Change-Id: Ic3eb0bac2d29a30488c84b6525ad796a7f1c9ce9

Avoid ack() race condition in JobQueueRedis

The race would cause jobs to be be unclaimed but have
no actual job data, causing pop() to return false for
them.

Bug: T133089
Change-Id: Icc60388d4431ab7fa2ca2e46bbb0d212b2b94201

Merge "Remove stray variable in processLogin()"

Catch JobQueue errors in SiteStats::jobs

Bug: T133045
Change-Id: I68a9334a63c04609227f4c4e9ae510633543c3d5

Merge "Deprecated 3 methods in DummyLinker"

Merge "Unroll DummyLinker::__call"

Deprecated 3 methods in DummyLinker

These methods are already deprecated in
Linker but they should also warn with the
method in DummyLinker

Change-Id: I41f0ee5b37f8a92711d1396b8b40d5235b643238

Merge "Move DummyLinker to own file"

Unroll DummyLinker::__call

Change-Id: Ie1325c2b8f06f9936e7b7ad10780c4b0cf702d18

Move DummyLinker to own file

Change-Id: I1808a6b77af2597b4dd2c9f1e0c8604c2bc8c801

RELNOTES for Skin::__call unroll & removal

Change-Id: I3d03338cbb48fe00f2beeb0dc7f5a1c2ff25d77d

Remove unused Skin -> Linker deprecated methods

This also creates full pass through methods for
the remaining methods that are used by extensions
and skins in gerrit still.

Release notes will be added in a followup

Change-Id: I31c3b754cd70b1c60310add0de9d29a76f713cad

Re add wfDeprecated calls after Skin::__call unroll

Change-Id: I0418d88e0fe355b7efff6e923d3144db3a294cea

Specially handle Title objects in Title::newFromLinkTarget()

Every single caller of Title::newFromLinkTarget() already special cases
when the LinkTarget object is already a Title, so move that logic into
one function.

Change-Id: Iba5432ae01c87850e5b34893092427c5b1629188

Add root title class to <body>

This adds a class to the body node of output pages in the form:

rootpage-X

Where X is the root title of the page.

This provides for the use cases in the linked tasks, and allows the
numerous existing cases of expensive css selectors using class*='page-X'
that are otherwise needed to apply a style to a page and or its subpages.

Bug: T123156
Bug: T17075
Change-Id: Ib16e380d97bd7bb55c5b5044a084fefbd039d599

Merge "RebuildRecentchanges improvements"

Update OOjs UI to v0.16.6

Release notes:
 https://git.wikimedia.org/blob/oojs%2Fui.git/v0.16.6/History.md

Change-Id: Ib9bb35d87633d06211dd75e237d7b105947f8df1

RebuildRecentchanges improvements

* Always delete entries in the time range
* Batch queries and wait for slaves
* Optimized size-fetching query (pass 2)
* Set with rc_type with the actual constants (pass 2)
* Avoid flagging non-bot accounts as 'bot edits
* Use more modern DB layer coding style

Bug: T133053
Change-Id: I55cc099bf40977fa9e0a365da104c208441eec00

Merge "Unroll __call in Skin"

Localisation updates from https://translatewiki.net.

Change-Id: I945074db5ba068c8e6b49b0cef3990938e766f16

Merge "Improve HTMLSubmitField return value"

Merge "rebuildrecentchanges: Allow rebuilding specified time range only"

rebuildrecentchanges: Allow rebuilding specified time range only

Bug: T133053
Change-Id: I8eef6e72f1105352116248548ab077fef818a8f9

Add missing visualClear on search result page

I'm not sure, when this was removed or why it broke, but without
the visualClear (clear:both), the text match section is near the
title match section, instead of after it.

Bug: T133071
Change-Id: I68028a8658ba0aa526ce1a6ef6401f9fd79533f8

Remove stray variable in processLogin()

Change-Id: Ife23555755e69d1d40b0c37259f53f3df78468aa

Remove unused wfGetDB() in SiteStats::jobs

Change-Id: Id818a71a75e4d4cb8397df5b1ff3dfdd3b155ea7

Merge "Convert Special:WithoutInterwiki to HTMLForm"

Unroll __call in Skin

Change-Id: Ia07a6099fed42bddcf2d6135de9c06f744b2f309

Merge "Fix wrong @return type hints in Language::tsTo… methods"

Add getStatsdDataFactory to MediawikiServices

Change-Id: Iaf89d5d7887f766064266874ea7ba73362531ed2

Fix wrong @return type hints in Language::tsTo… methods

Change-Id: Ic4dec8e8612021f010734037b6c3527bef2fe04a

Merge "Hide hiddencat catwatch changes in special changelists"

Convert Special:WithoutInterwiki to HTMLForm

TODO: Should be eventually converted to OOUI

Bug: T117721
Change-Id: I56b6b78b53d2531ebdb9cd0f903a4ce475dbc56b

Merge "Segment stash edit cache stats by basis for hit/miss"

Segment stash edit cache stats by basis for hit/miss

Instead of just counting cache hits and misses, segment the counts by reason,
so we can differentiate (for example) timestamp-based cache hits from
staleness-check-survivor cache hits. I want this data so I can determine
whether increasing the cutoff for timestamp-based hits from 3 to 5 seconds has
a substantial enough impact to warrant the slightly weaker consistency.

Also changed 'cache-hit' to 'cache_hit'. MediaWiki normalizes the dash to an
underscore anyway, but the normalization is there for dynamically-constructed
key names (or name segments). In the case of hard-coded values, it is desirable
for the code to be as close as possible to the final form of the metric name,
to simplify metric lookup.

Change-Id: I0cd61da9746e3ca3695e23200f698b8b1371798c

registration: Support skipFunction module definitions

Change-Id: I9c5cea3e8df1fae001d4d98c0782a04dec803776

Merge "Reject usernames with namespace or interwiki prefixes"

Merge "Enforce $wgReadOnly for job queues"

Allow tag names for SqlBagOStuff consistent hashing

This is needed to avoid poor hit rates in multi-DC setups

Change-Id: I8830bd6bffa4619e31609676527282ef9af0c76c

Merge "DateTimeInputWidget: Set type before calling parent constructor"

Enforce $wgReadOnly for job queues

Bug: T130795
Change-Id: I9d8cf919de80dbe855086b9c590c0a0f20dc33b9

resourceloader: Use feature test instead of UA sniffing

Converting to a feature test reduces complexity of the startup module, and
makes the blacklist more meaningful - allowing us to safely remove some
unnecessary feature guards and polyfills.

It'll also provide unsupported browsers a much better experience. Previously,
unsupported browsers got the payload for Grade A runtime. Many of which would
likely fatal due to unimplemented DOM and JS methods. With the capability
filter in this commit, the client will only initialise the environment if the
browser is capable - falling back to Grade C otherwise.

The feature-test that replaces the user-agent filter in this commit is 99.96%
effectively the same - based on a 7 day analytics campaign on all Wikimedia
wikis (see T102318 for details).

Bug: T102318
Change-Id: I847b3f5f80f1eef3f57bbe7518768468e271082b

Localisation updates from https://translatewiki.net.

Change-Id: I3777ce409669883f0d7ac20ce834bf8526b6dfea