Merge "Skin: Remove redundant use of QuickTemplate::setRef()"

Localisation updates from https://translatewiki.net.

Change-Id: I7319928e9a7f6a9ec00cfcc27dba4e9a5fbf2f61

Merge "rdbms: Support secondary autocommit connections in LoadBalancer"

rdbms: Support secondary autocommit connections in LoadBalancer

This is useful for things like SqlBagOStuff and JobQueueDB
being able to write in auto-commit mode while the main
transaction round still goes on.

SqlBagOStuff already had this sort of logic.

JobQueueDB now also takes this approach so it does not have to
defer insertion except for sqlite. This makes callers more likely
to know when insertion failed or not.

Make sure LoadBalancer sets "master"/"replica" LB info itself;
this fixes a bug found in the new tests.

Bug: T140338
Bug: T42451
Change-Id: I4199a9598d7afbf976a6efa8ed84b85b56da02bd

Skin: Remove redundant use of QuickTemplate::setRef()

As part of long-term moving towards having skins consume prepared
data and not making additional run-time database queries or additional
PHP-specific semantics and global state, try to phase out use
of QuickTemplate::setRef(), which is incompatible with the idea of
data only flowing in one direction (given it allows the value to
change at any time).

* jsmimetype: Deprecated/unused, has a constant value for compat.
* mimetype: Obsolete, Content-Type header comes from OutputPage.
* skin: Objects are references by default, '$this' can't be
  modified anyway.

Bug: T140664
Change-Id: Ia505cc3e1b774a380fb36b4f0e2522550aef5bbd

Merge "rdbms: Fix grammar in DBTransactionSizeError log message"

Merge "Make mysql getHeartbeatData() method not count query/RTT in estimate"

Merge "Restore the newFromId() approach in SpecialNewpages::feedItemDesc"

rdbms: Fix grammar in DBTransactionSizeError log message

Change-Id: Icc09d674359012f60494b95a36127c14c1a8d630

Restore the newFromId() approach in SpecialNewpages::feedItemDesc

Trying to use the row data is problematic since the text related
fields are missing. This fixing a regression from 4475e1c8c.

Also make SpecialNewpages::feedItemDesc() handle null content.

Bug: T173541
Change-Id: I2213675d3392c6e9761bdc7acde35fd1caee4517

Merge "RCFilters: Fix validation for single_option groups"

Merge "rdbms: Rename "memCache" to "memStash" in LBFactory"

rdbms: Rename "memCache" to "memStash" in LBFactory

This needs to be cross DC, so fix the name and comments.
Also update the one MWLBFactory reference to it.

Change-Id: I5ef10000276c300299fd1ece35ada0d78388bef7

Merge "Support AJAX watch in skins that use a different class structure"

Merge "Remove "memCache" field from LoadBalancer"

Make mysql getHeartbeatData() method not count query/RTT in estimate

Optimize for the common case where lag does not strictly increase
during trip times instead of being pessimistic.

Change-Id: Ibb0e3faad23862d397e1cc2a9f7e2a8293fa5f2b

RCFilters: Fix validation for single_option groups

The single_option groups have a base definition that means they
always must have a value, and must never have more than one value.
The previous way we attempted to do that was confusing and missed
a case where after resetting filters, two values were selected before
one could be unselected, which then broke the behavior in the
entire group.

This fix reorganizes the validation when an item in the group is
selected or unselected to make sure the group retains its promised
behavior.

Bug: T173303
Change-Id: I5758ec324a26c0e5e6f5c473d206e818a1d22523

RCFilters UI: Correct full coverage message

Not all groups have this behavior, so it's not accurate to say
that "Selecting all filters in *A* group" behaves this way.
Instead say "all filters in *THIS* group".

Change-Id: Iaa92db76b27a2b0175e1ba1c71d1ef087857e87e

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

Remove "memCache" field from LoadBalancer

The only user was LoadMonitor, which is converted to only use
the WAN and server caches. Previously, the lock() calls there
were useless since LBFactory never injected "memcCache" to
LoadBalancer, ergo making it EmptyBagOStuff in LoadMonitor.

Change-Id: I0c7793d47b93b763dee478d16fb87ec637dc6cab

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

Conflicts with Ia9e669376390a56e52422352840a5d9e2576b1a8

This reverts commit 26b883a2f8407847e673a23357035794b1c3ddc3.

Change-Id: I532b7d2782e9087cb6a5f114af5aac9ee14e3f2f

Merge "Avoid lock acquisition errors for multi-title refreshlinks jobs"

Avoid lock acquisition errors for multi-title refreshlinks jobs

Bug: T173462
Change-Id: I9dab9b4e5c4cae7306dc29bad9e62287d54b2281

Localisation updates from https://translatewiki.net.

Change-Id: Ibd0b5f8fac26cf59df39172d2297597b776a4aec

Merge "RCFilters: tweak old changes indicator"

Merge "RCFilters: Add marker between old and new changes in enhanced mode"

Merge "database: make close() sanity check mTrxPreCommitCallbacks"

database: make close() sanity check mTrxPreCommitCallbacks

The other callback types are already checked too.

Change-Id: Ibdc82110c0a0e74bb07ed92ee667bdf0d9f69f90

Hard-deprecate ProfileSection

No usages anywhere, to be removed in 1.31

Change-Id: I3ed4c2aa07d97e18c026e9ee6dae66d3b023896f

Merge "Restore non-mangled anchors in API help in all cases"

Merge ".gitignore: Ignore /composer.phar file"

.gitignore: Ignore /composer.phar file

The file is needed if using composer via a phar file.

Change-Id: I910bc8929d0cf26d9d110eeb1051c4d41c25aa3c

Restore non-mangled anchors in API help in all cases

Prior to Id304010a, the API help included non-mangled anchors even when
$wgExperimentalHtmlIds was false. This was lost in Id304010a when the
new $wgFragmentMode is [ 'legacy' ], as is the default, despite the loss
being pointed out multiple times in code review.

Change-Id: Ibcb86deb4e3ea2131211f184f46960e5ad390212

Merge "mediawiki.ui: Bring checkbox and radio on par with WikimediaUI design"

Support AJAX watch in skins that use a different class structure

Skipping links that are in #bodyContent or #content doesn't work for
skins like Timeless. Instead of trying to filter based on those classes,
have SkinTemplate set a data-mw=interface attribute on the <a> element,
and filter based on that. The Sanitizer prevents any data-mw attributes
being set by users, so it must have been created by MediaWiki, and
therefore safe to use.

If no link meets that criteria, then it will fallback to trying to find
a link using the old criteria of not being in #bodyContent or #content.

Bug: T173279
Change-Id: I1688a499dda40428dd514230b78ccded0d228ca3

mediawiki.ui: Bring checkbox and radio on par with WikimediaUI design

Bringing checkbox and radio on par with WikimediaUI design templates,
also slightly enhancing accessibility by using colors of new palette.
Width & height is taken from current OOUI size, based on 14px Vector base
font-size.
Adding several similar variables to variables.less, which will be replaced
by WikimediaUI Base vars in future.
Also:
- removing necessity for images for radio checked states
- replacing `margin` with `padding` between widget and label in order to
  have `cursor: pointer` on full element

Bug: T148265
Change-Id: I68848d48ea45c67124ac75be58748b4e9fb3085d

Fix spacing for @param and indent of function comments

In phpcs.xml rename renamed sniffs and add the failing sniffs,
because now the whole sniff is no longer excluded.

Change-Id: If5b0bd16028761abc2c47ace9e97d37ad14bb36f

Merge "RCFilters: preserve invert parameter in saved queries"

Localisation updates from https://translatewiki.net.

Change-Id: I95727cea51152a116d75a2843d837cedbff53566

Merge "Move RawMessage out of Message.php to its own file"

Merge "Make Special:NewPages show "originally created as" correctly"

Move RawMessage out of Message.php to its own file

Change-Id: Idae9617dafa3c314085eb097f78f1c8d38672f31

Merge "Perform a permission check on the title when changing the page language"

Merge "Special:Newpages feed now shows first revision instead of latest revision"

Change WikiImporter to now requires the second parameter to be an instance of the Config class.

Prior to this change, the second parameter to WikiImporter was optional.  If the parameter didn't exist, it would get filled in with a MediaWikiServices::getInstance()->getMainConfig().  It would also emit a hard deprecation warning. This behavior has existed since 1.25.

Bug: T61113
Change-Id: Ic15cc103c7d2b7fb499f1632c0001b8a3dbdf778

Merge "Ignore errors in RedisConnectionPool destructor"

Ignore errors in RedisConnectionPool destructor

The destructor can be called on shutdown so any operation which
relies on another object or resource can fail. E.g. running
anything involving Redis (such as >>new Message('').''<<)
from shell.php and then exiting will result in a RedisException
since the PHP engine closes the Redis connection before destroying
the connection pool. Such errors can be safely ignored.

Change-Id: I38474a9dda89c82edbcb878facb4a97740e9189a

Localisation updates from https://translatewiki.net.

Change-Id: If08fcdfd7268397691d725d198d222f9fce9cfbb

Remove deprecated MagicWord::replaceMultiple()

wfDeprecated since 1.25, not used anywhere.

Change-Id: I09fd362a3d795328ffc12572025702ef9b951378

Merge "BaseTemplate: Handle `$this->data['language_urls'] === false` specially"

Localisation updates from https://translatewiki.net.

Change-Id: I324f599c4c46011cf7348ca75fc1d08e1beafe3f

Merge "jquery.tablesorter: Improve detection and handling of isoDate"

jquery.tablesorter: Improve detection and handling of isoDate

* Detect years 0...99 correct.
* Short forms possible: JJJJ, JJJJ-MM, JJJJMM, JJJJMMTT
  QUnit Test sorts now with parser 'isoDate' (because of 2009 former test sorts with parser 'text')
* Prefix and postfix allowed.
* Between date and time a 'T' or 'any white space' is allowed (Bug: T126886)

Bug: T126886
Change-Id: I664b4cc9d5fb472ea0bc0e36a3c209f04048e769

Remove test code that depends on extension

And allow extensions to add their own media handlers.

I'm not too happy with the introduction of another global,
but didn't like the alternatives either:
* Add some hook to MockMediaHandlerFactory that would allow
  extensions to add their own stuff in.
* Use another hook (like ParserTestTables or ParserTestGlobals)
  and then override the service with a new instance - seemed
  too hacky
The good thing about this is that it lets us kill off a class.
I'm other to other suggestions in case I missed something.

Bug: T169258
Depends-On: I5875621c58597426ad5242bf3d07714555c439b5
Change-Id: I1c2e903fb235395a8de8e0f7bf65ce07739d2930

Unwatch link for pages in Special:Watchlist

When the 'watchlistunwatchlinks' preference option is enabled, this
adds a '×' link to each entry of the watchlist that unwatches the page
of that entry. When clicked, it changes into a '+' which can be used to
re-watch the page (effectively undoing the earlier unwatch).
When a page is unwatched, its entries and the entries of its associated
talk page (or vice versa) become translucent and are struck through.

Without JS, '×'/'+' link to action=(un)watch for the relevant page.

In addition, ChangesList classes have been modified to allow a prefixer
that adds a prefix to each line (used in this case to put the unwatch
link) and to add HTML data attributes to reliably determine the target
page of each entry. Unit tests have been updated accordingly.

Bug: T2424
Change-Id: I450b2901413d7e75c11de2a446829fdbb22d31e1

Merge "Warn users when installing on a 32-bit system"

Warn users when installing on a 32-bit system

Don't fail to install, because we don't explicitly need to break
32-bit support. But in this day and age it's becoming increasingly
likely to break things on a 32-bit system. So let's at least let
a system admin know that YMMV.

Change-Id: Ifb3bfa049532d82f67fe4375944af1516400513c

Localisation updates from https://translatewiki.net.

Change-Id: Ic427ce81276365b5e4ce288bd1b330850592ac81

Merge "Disallow job pushes from JobQueueGroup to bogus wikis"

Merge "Fix 'logName' parameter (5th) in many uses of mw.log.deprecate"

Merge "mw.util: Add a deprecation message to escapeId()"

Do not use broken bootstrap in phpcs.xml

The bootstrap in mediawiki-codesniffer is broken, because it was not
adjust when PHP CodeSniffer Version 3.0.0 was used. Remove the broken
bootstap.

Bug: T172933
Change-Id: I29e776662055077f6de3a9bc6d0869120b387ea7

Disallow job pushes from JobQueueGroup to bogus wikis

Bug: T171371
Change-Id: I03a5dbd18cf6b5bcacb3ec07cef9e0b051bc147c

Merge "Add missing @param and @return documentation"

Merge "Add missing type to @param documentation"

Merge "update mediawiki-codesniffer to 0.11.0 and fix issues"

update mediawiki-codesniffer to 0.11.0 and fix issues

- mostly auto fixes
- some too long lines fixed
- ignore amp space in one case  passing by reference

Change-Id: I6472f83bc3cbf4bd629d83050cc3319b19ec465c

Add missing @param and @return documentation

Change-Id: I1d1098eec3933df6561cceef646576013ddc08c8

Localisation updates from https://translatewiki.net.

Change-Id: I4e8ad1e97a08441dad773c8435309df93ab347d9

Add missing type to @param documentation

Change-Id: I6b2c9c7af9a281fe457099cc3a336a60a25e74aa

Merge "Use correct variable name in @param documentation"

Merge "Add missing & to @param documentation to match functon call"

Merge "Fixing styling of interwiki results header"

Use correct variable name in @param documentation

For some varargs a variable name is added with suffix ,... as seen for
many other varargs

Some @param are swapped, because there are in the wrong order

Enable Sniff MediaWiki.Commenting.FunctionComment.ParamNameNoMatch

Change-Id: I60fec6025bce824d5c67563ab7b65ad6cd628ad8

Add missing & to @param documentation to match functon call

Change-Id: I81e68310abcbc59964b22e0e74842d509f6b1fb9

Merge "Make phpcs pass"

Change @inheritdoc to @inheritDoc

Only @inheritDoc works for the Sniff
MediaWiki.Commenting.FunctionComment

Change-Id: I91fc02cda6701d790e4334fc2bc47f230955545c

Make phpcs pass

Reenable MediaWiki.WhiteSpace.SpaceBeforeClassBrace.NoSpaceBeforeBrace,
because the mentioned bug is fixed

Bug: T172933
Change-Id: I1593bdba2295ebed401b921f2beabed69dba7638

Merge "SpecialDoubleRedirects: Fix undefined '$result->namespace' notice"

Merge "Create new table to be used for querying IP ranges."

Create new table to be used for querying IP ranges.

When revisions by logged out users are created, they will get copied
to this table. We can then JOIN it with revision when querying for
ranges at Special:Contributions.

DBA approval for this table can be found at T156318

Bug: T163562

Change-Id: I6593eb13701128faa782691a6b25ec01869c827d

Localisation updates from https://translatewiki.net.

Change-Id: Ib722424a58bbb3d15c56c70ba29732f2be2d6669

SpecialDoubleRedirects: Fix undefined '$result->namespace' notice

Follows-up 7be1a8c0bc4d29d.

$result is used for both the original and the on-demand deep
query (in case of cache). However this re-query can fail if the
page no longer exists.

This used to be accounted for by creating the Title object before
(possibly) trying to re-query, and also checked for with "if !$result",
which is only intended to happen if the re-query failed (the initial
value of $result comes from the main (cacheable) query, and always
exists.

Could be moved by moving the $titleA assignment back up, but that
doesn't address the confusion problem that led to this.

Fix it by using a separate variable names.

Bug: T173045
Change-Id: I0e9ae89a3772b33b916b506033bd334ade5f03fa

Merge "Add new static function LanguageCode::replaceDeprecatedCodes"

Merge "Make it possible for subclasses to provide a different form"

Add new static function LanguageCode::replaceDeprecatedCodes

Refactor the deprecatedLanguageCodeMapping to a private variable.

Change-Id: I5f8e601e53de183e6268c9ef601eef8390b725cd

Merge "Rename autonym for 'no' from 'norsk bokmål' to 'norsk'"

Merge "Document Special:Diff and Special:PermanentLink"

Merge "mediawiki.toc.print.css: Restore ID selector #toc"

Merge "installer: Link to skin screenshots"

Merge "installer: Disable <div> wrapping of parsed content"

Document Special:Diff and Special:PermanentLink

If no revision is given to Special:Diff or PermaLink show a form
(cf. Special:ComparePages)
List these on Special:SpecialPages under "redirects".

Bug: 45221
Change-Id: I77edd4a1bbd342d4d18c27d5bc10dac76b8ab6c9

Make it possible for subclasses to provide a different form

This makes it possible for subclasses of SpecialPreferences to
specify a different HtmlForm to use for the preferences' form.

Bug: T68869
Change-Id: I9d6bbc6383a3d5b5c6839394de49ce9ca81efec9

installer: Link to skin screenshots

Skins can now place *.png files in a "screenshots/" directory and the
installer will link to them. It assumes that the skin directory is in a
web accessible directory.

Bug: T70331
Co-Authored-By: Isarra <zhorishna@gmail.com>
Change-Id: I30a4374619ac2cb6afe6d8c0d9181dd6b14aae79

Merge "Add page id to the API representation of pages in search results"

Merge "PHPVersionCheck: Add 'fileinfo' extension"

installer: Disable <div> wrapping of parsed content

The mw-parser-output wrapper breaks the visual layout of the installer,
causing the radio button or checkbox to be one line above the text that
describes it.

Change-Id: I0a7ec34d80d6372927fd89b7752bc08ab0395308

sanitizer: Match Parsoid's attribute sanitation

Change-Id: I23323dced95c80eb24aa65df68dcbe5ecbad1c75

mediawiki.toc.print.css: Restore ID selector #toc

The ID selector #toc is necessary to overwrite

 #toc {
    display: table;
 }

with a higher specificity.

This change partly reverts fbf939cdda and restores the functionality
from T2482.

Change-Id: I7134132979a832828cf9618218d489727fc2dbc3

Fix 'logName' parameter (5th) in many uses of mw.log.deprecate

It's apparently supposed to contains the full name of the deprecated
property/function and apparently almost no one ever remembers it.

This fixes error messages like "Use of "wikiGetlink" is deprecated"
to instead read "Use of "mw.util.wikiGetlink" is deprecated", and
apparently improves some error logging detais.

Change-Id: I9b0d6761754e5844d51b71535e0547205eb20781