Merge "HISTORY: Add MediaWiki 1.3 post-release change notes"

Merge "Make ApiStashEdit::pruneExcessStashedEntries use user name"

Merge "objectcache: remove some unneeded code from BagOStuff::mergeViaCas"

Merge "objectcache: rename SET_DELAY_HIGH_SEC => SET_DELAY_HIGH_MS and lower it"

Merge "objectcache: make WANObjectCache prefer ADD over GET/CAS for misses"

Merge "specials: Fix incomplete documentation of execute() param"

Merge "Replace strlen() calls with strict string comparisons"

HISTORY: Add MediaWiki 1.3 post-release change notes

Add MediaWiki 1.3 post-release change notes, sourced from
https://www.mediawiki.org/wiki/Release_notes/1.3

Bug: T213714
Change-Id: I197d52a7b7a685b6fcb9e751983a5c11d9ed96ad

specials: Fix incomplete documentation of execute() param

The parameter passed to the execute() method of each special page can
be null, and usually is. In some of these files this fact was already
mentioned in the comment, but not listed as a type.

In this patch I also remove comments that do not explain much. Saying
that the execute() method of a special page "executes a special page" or
is the "main entry point to a special page" is not super helpful. That's
usually what the documentation in the parent class shoudl explain.
We can add @inheritDoc tags in all these cases if you prefer. Please
tell me.

Change-Id: I1d811ab0c6d5c956e36f6a74120a425abc4332e6

resources: Allow mediawiki.action.edit.editWarning on mobile target

Bug: T219476
Change-Id: Ifa745d937b954647dfde2b185064cd86a3cc8320

Merge "profileinfo: Remove unused constructor call parameter"

Merge "title: Rewrite Title::getBaseText() for performance/readability"

Replace strlen() calls with strict string comparisons

Note there is an important difference between the two ways to express
this: strlen() does a string cast, but the `=== ''` and `!== ''`
comparisons will only detect empty strings, but not null, false, or any
other falsy value that becomes an empty string when cast to be one.

I am only touching code where I'm sure the variable is guaranteed to be
a string.

This change is done because I find the strict comparisons much more
readable. The code does exactly one thing now, and no magic casts any
more.

Change-Id: I3e908a0c7c7b6c29b0e5a1414f2ba9062a215b93

objectcache: rename SET_DELAY_HIGH_SEC => SET_DELAY_HIGH_MS and lower it

Change-Id: I92a1e9a713f60288eb02bdea4881d104bdf7d0f5

Merge "Move HistoryBlob classes to own files"

objectcache: make WANObjectCache prefer ADD over GET/CAS for misses

This avoids an extra cache query and also avoids I/O if another thread
already saved the value in the meantime, bloating the GET response.

Bug: T203786
Change-Id: I05539873f55d3254e2b9ecad0df158db1e6a1a1a

Merge "Remove temporary rollback confirmation toggle"

Merge "mw.widgets.DateInputWidget: Deactivate after blurring if focus was on calendar"

Merge "Only show config-install-success on command line installer where it makes sense"

Merge "Deprecate Block::isValid method"

Merge "resourceloader: Improve test cases for DerivativeResourceLoaderContext"

Merge "Change name of main page in Sardinian (sc)"

Merge "Revert "Provide direct deprecation message for jquery.ui.spinner""

Merge "Add namespace translation for N'Ko"

Revert "Provide direct deprecation message for jquery.ui.spinner"

This reverts commit 5b812305da96b4acfade7feab5885580886bb0d3.

Reason for revert:

Everything about jQuery UI is terrible, including the naming of modules.

Change-Id: Ide2bb589a4af3b166c46bcd163e6269a80b6d5c0

objectcache: remove some unneeded code from BagOStuff::mergeViaCas

Change-Id: Ib92090fe4e1b1eb2c4f9e031f8476b24189392c8

Make ApiStashEdit::pruneExcessStashedEntries use user name

Follow up to c334883309778

Change-Id: I35213d83a00b50bb469aab5bf8fbeb52239cff5e

Merge "Annotate different parts of the contributions UI with classes"

Merge "objectcache: fix changeTTL() in RedisBagOStuff for edge cases"

Annotate different parts of the contributions UI with classes

Distinguish the user tools (talk, block, block log etc)
from the pager navigation navigation bar so that skins can style
them.

Change-Id: I32950aa452347aabb8a37253bafc5f20d518c110

Merge "objectcache: improve "version" documentation for WANObjectCache::getWithSetCallback()"

objectcache: fix changeTTL() in RedisBagOStuff for edge cases

Change-Id: I2bb4bf843c8de8cefaf4188a8866ae51a28bbf71

Merge "Make LinksDeletionUpdate a subclass of LinksUpdate"

Merge "SpecialPreferences: Use real OOUI PHP layouts"

Deprecate Block::isValid method

Block::isValid checks whether the block target equals null. It seems to
have been unused since commit 40a926ab2c566052eefd08c36f246d6b5321afc5.
It was called from Block::newFromDB, which was removed in favour of
Block::newFromTarget, which has its own logic for checking the target.

Since the method is public, it should be deprecated before removal.

Change-Id: Ibcfb377a382f55e80933836960be7e3589908390

Make LinksDeletionUpdate a subclass of LinksUpdate

This is proposed as an alternative to I0b636dc144f34bb.

The idea is to ensure that page deletion causes the exact same
database updates to be performed, and the exact same hooks to
be fired, as a page edit.

Bug: T216249
Change-Id: I665320e27da8edc2867b47d181cc0f324e75d102

SpecialPreferences: Use real OOUI PHP layouts

Change-Id: Id8c798c0d5be07afd1c4d6584af22c54c3465621

Merge "HISTORY: Add MediaWiki 1.5 post-release change notes"

HISTORY: Add MediaWiki 1.5 post-release change notes

Add MediaWiki 1.5 post-release change notes, sourced from
https://www.mediawiki.org/wiki/Release_notes/1.5

Bug: T213714
Change-Id: I6c29cca7903df3f9f718aa13512c500c5fced63c

mw.widgets.DateInputWidget: Deactivate after blurring if focus was on calendar

Bug: T219189
Change-Id: I5f03dbd12afac31f1d72fc1aeb78125647595b5d

Use DateInputWidget for date inputs which do not need time

For those that still need the precision of time, the
DateTimeInputWidget can be used.

Bug: T107069
Change-Id: I2e0b0168b53594760c2630dc4ecd4e6af7404a7c

objectcache: improve "version" documentation for WANObjectCache::getWithSetCallback()

Change-Id: Ia48f21945e6da7f2d7892677d559718f824b62c1

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

Localisation updates from https://translatewiki.net.

Change-Id: I54b566c6b6f484b068911e49dabe15213ffdf9b6

Merge "objectcache: merge getWithToken() into doGet() for simplicity"

Merge "HISTORY: Add MediaWiki 1.8 post-release change notes"

objectcache: merge getWithToken() into doGet() for simplicity

Change-Id: I581f866521e1086ca350973d9cdeff6656f48fe8

Merge "Fix reference to removed truncate() function"

resourceloader: Improve test cases for DerivativeResourceLoaderContext

Make the methods more descriptive of the stories and use cases
they cover, and less method-oriented given they don't only
just test a getter or setter anyway (and doing so is arguably
not useful).

Also fold the "testGetInherited" case into the specific ones,
add additional assertions for inheritence that was previously
not tested (specifically, for getVersion and getRaw).

Change-Id: I01c72112e517e66cea5ecf9e2ab53b9039fb99bb

Merge "HISTORY: Add MediaWiki 1.4 post-release change notes"

Merge "HISTORY: Add MediaWiki 1.6 post-release change notes"

HISTORY: Add MediaWiki 1.8 post-release change notes

Add MediaWiki 1.8 post-release change notes, sourced from https://www.mediawiki.org/wiki/Release_notes/1.8

Bug:T213714
Change-Id: Ib821066ad1a4940be02aa1127ddec80065f688db

Merge "rdbms: add ATTR_SCHEMAS_AS_TABLE_GROUPS attribute"

HISTORY: Add MediaWiki 1.4 post-release change notes

Add MediaWiki 1.4 post-release change notes, sourced from https://www.mediawiki.org/wiki/Release_notes/1.4

Bug:T213714
Change-Id: Ibbfd4bfe604910a81f3d741f0f7c0d7178679071

Merge "objectcache: remove redundant merge() definitions"

Merge "objectcache: defined some missing methods in MultiWriteBagOStuff"

objectcache: remove redundant merge() definitions

Change-Id: Ide24dd395b300f8d747b2be307dbbad08f6b221d

Merge "objectcache: define some missing methods in ReplicatedBagOStuff"

Merge "Make EnqueueableDataUpdate use DB domain IDs"

Merge "Update OOUI to v0.31.2"

Merge "Remove unused rollback module"

Merge "objectcache: fix return value type in SqlBagOStuff::incr"

Merge "objectcache: fix race condition in SqlBagOStuff::changeTTL"

Merge "Replace misplaced strlen() checks with strict `false` comparisons"

Make EnqueueableDataUpdate use DB domain IDs

This makes it consistent with what JobQueueGroup actually expects.

Change-Id: Id6c6b3bf19fb1e85a2e3e231d216974d831a2f7c

Remove unused rollback module

The mediawiki.page.rollback module was previously introduced in this
patch:

https://gerrit.wikimedia.org/r/c/mediawiki/core/+/242050

However, the change was reverted soon after and this module alongside
its JavaScript were left in the code, even though they are not used and
have been replaced by the mediawiki.page.rollback.confirmation module.

Bug: T218574
Change-Id: I2c71adcfcf86e5032e82c237760a9d85a63de3f8

HISTORY: Add MediaWiki 1.6 post-release change notes

Add MediaWiki 1.6 post-release change notes, sourced from https://www.mediawiki.org/wiki/Release_notes/1.6

Bug:T213714
Change-Id: I283478823d62c0ee6844062f18e7018cec043705

objectcache: defined some missing methods in MultiWriteBagOStuff

Also, reorder some method definitions to match the base class.
This makes it easier to compare the two. In addition, refactor
the doWrite() method to make the calls clearer.

Change-Id: I795b395b9c54645f62962461a28067ae38b291ed

Merge "Move logic for checking block behaviour to Block class"

Merge "linkeddata: Remove unnecessary arguments and shrink if...else"

Merge "HISTORY: Add MediaWiki 1.7 post-release change notes"

linkeddata: Remove unnecessary arguments and shrink if...else

if...else that returns bool can be shrinked down to a single return
statement like in this case.

Change-Id: Ic50fac1825cbdbcc052dcd470e896a9344c8e418

Merge "user: Remove not needed array_values()"

Merge "Fix api error for apierror-upload-missingresult"

Move logic for checking block behaviour to Block class

User::trackBlockWithCookie and PasswordReset::isBlocked make decisions
about block behaviour based on the block parameters. This should be
done in the Block class.

Bug: T218905
Change-Id: Ia3f46abacdaf70e720b715b39dc60aed53be2d0a

user: Remove not needed array_values()

Change-Id: I0a2321d5f363b86d01ab7ad94df920008e607390

HISTORY: Add MediaWiki 1.7 post-release change notes

Add MediaWiki 1.7 post-release change notes, sourced from https://www.mediawiki.org/wiki/Release_notes/1.7

Bug:T213714
Change-Id: I94f81ff8f361decedbbbeed4885baf580c038d67

title: Rewrite Title::getBaseText() for performance/readability

This code does not really need to explode a path in all it's individual
segments. All it needs to know is *if* a title is a path (with at least
one slash), and then remove the last segment. That's pretty much exactly
what the code proposed in this patch now does.

Change-Id: I9cf714f7bb637ecd9e1f44340219c76ddd60ee29

Replace misplaced strlen() checks with strict `false` comparisons

As far as I can tell the intention of this code is *not* to detect
empty strings, but `false`.

Change-Id: I406d065a54a1437fdfd46827fec9493e5fad05f9

profileinfo: Remove unused constructor call parameter

Also add a type hint that makes my IDE stop complaining about method
calls on an variable of unknown type.

Change-Id: I43e6a7de3313d84e81b94865eaa81ad47edd6d6f

objectcache: define some missing methods in ReplicatedBagOStuff

Also reorder methods to match the base class so it is easier
to compare the classes and see which methods should be defined.

Change-Id: I2b8c2cd63822ce49efd9b4b7e44bde92032b5a98

Merge "maintenance: Use the ?? feature together with explode()"

objectcache: fix return value type in SqlBagOStuff::incr

Change-Id: I14a70956f77f13346857236eda5c4b6fb80c6223

objectcache: fix race condition in SqlBagOStuff::changeTTL

Also fix --use-bagostuff parameter when given 0

Change-Id: I64971d4ccad2c856cc7688577ae564c8db068819

Merge "mediawiki.special.changeslist is a skinStyle module"

Merge "Replace spinner.gif with CSS solution"

Replace spinner.gif with CSS solution

Bug: T219163
Change-Id: Icca5c4e01fe4c0ae377245ae7afb456a17626208

Update OOUI to v0.31.2

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

Bug: T161854
Bug: T192131
Bug: T218882
Bug: T218977
Bug: T219004
Bug: T219240
Depends-On: I6179266e71901182614ec24e7b0510b7a44ff50a
Change-Id: I6179266e71901182614ec24e7b0510b7a44ff50b

Consistently use classes for deleted items in lists of diffs

Even if a revision has been deleted and the history-deleted class is
being used, the original class for its corresponding element when
not deleted should be applied. This is important as it allows skins
to skin consistently. "history-deleted" is a modifier class - it provides
further information on the original meaning.

This blocks styling the history page from core in Minerva skin

Bug: T216420
Change-Id: Ia659606838d7e38bc09054e36cd980b00f5e6da9

Merge "objectcache: minor refactoring to BagOStuff"

Merge "Allow skins full control of Contributions/History styles"

objectcache: minor refactoring to BagOStuff

Split up expiry functions and moved add() near other primitive writes.

Change-Id: I97bb91e648b797b1ab5a6934d212689b1e67f7c6

Merge "objectcache: improve comments for ATTR_/QOS_ constants"

Merge "rdbms: treat cloned temporary tables as "effective write" targets"

Merge "Prune old edit stash entries from users as they create more"

mediawiki.special.changeslist is a skinStyle module

Allow skins full control over styling changelists
Several styles relating to unicode-bidi and client-js are
made mandatory.

The mw-changeslist-legend rules I will review separately with
Volker to determine whether they should be mandatory too, once
I'm more informed about how Minerva might use them.

Change-Id: I92c5fdc33b50771bd45e4945185c27395c608eb4

Fix api error for apierror-upload-missingresult

The message key was wrong

Change-Id: I2cc4f9e1523bef6164eb9ff4b79655b007665fae

Allow skins full control of Contributions/History styles

This switches 2 special page modules to be defined via skinStyles.
This allows skins to skin these pages as they wish rather than
relying on the defaults.

This is very important for Minerva, which provides a distinctly
different look and feel.

Bug: T205581
Change-Id: I429ab712099ede0fb3cbd64ca3fd1840a1615652

rdbms: add ATTR_SCHEMAS_AS_TABLE_GROUPS attribute

Change-Id: Ia12a00c30039acd6c59726f35d03cc76aa957902

rdbms: treat cloned temporary tables as "effective write" targets

Make IDatabase::lastDoneWrites() reflect creation and changes to
the cloned temporary unit test tables but not other temporary tables.
This effects the LB method hasOrMadeRecentMasterChanges(). Other tables
are assumpted to really just be there for temporary calculations rather
acting as test-only ephemeral versions of permanent tables. Treating
writes to the "fake permanent" temp tables more like real permanent
tables means that the tests better align with production.

At the moment, temporary tables still have to use DB_MASTER, given
the assertIsWritableMaster() check in query(). This restriction
can be lifted at some point, when RDBMs compatibility is robust.

Bug: T218388
Change-Id: I4c0d629da254ac2aaf31aae35bd2efc7bc064ac6