Move editoptions styling to mediawiki.skinning/interface.css

Adding the editoptions box styling to mediawiki.skinning.interface instead
of the skin-agnostic mediawiki.action.* means it will only be added to skins
where it's likely to be appropriate, as this is also where the similar styles
for toc, categories, etc are defined.

This way skins in which the editoptions box would be inappropriate need never
worry about it anyway because they won't be using mediawiki.skinning.interface
for the same reason.

Bug: 58254
Change-Id: Ibd1c0617c426e34e7213ca0ce89a126340b012a7

Merge "Made JobQueueDB::doWaitForBackups wait for the right slaves"

Made JobQueueDB::doWaitForBackups wait for the right slaves

Change-Id: I6022fea7add5c317da89d5361d166a047bcc95e4

Localisation updates from https://translatewiki.net.

Change-Id: I66241c255b5cf101a32f239df9b4276ca3c37946

Merge "Fix function case: WebRequest::getIntorNull -> getIntOrNull"

SECURITY: Enhance CSS filtering in SVG files

* Filter <style> elements
* Normalize style elements and attributes before filtering
* Add checks for attributes that contain css
* Add unit tests for html5sec and reported bugs

Bug:69008
Change-Id: I732eece710f1bfaaeea1e5de541fcd4cfb375de7

Fix function case: WebRequest::getIntorNull -> getIntOrNull

Change-Id: I3cdbb17ae00b9565f522a80e1455edf375043763

HtmlForm: Add @since annotation for methods added to UI standardization

Change-Id: I74f58df9de79d2ad055eaf9760763005996dc0a0

Merge "HtmlForm: Use mw-ui-checkbox only if wgUseMediaWikiUIEverywhere is enabled"

Merge "Remove unused system messages "loginstart", "loginend" and "loginend-https""

Merge "Per 5a4a33a, remove support for magic quotes gpc"

$wgTitle is not needed for running parser tests

Change-Id: Iaf14b1bc15763207a30cf624d5876120b0d5ce01

Add a process cache for fetchLanguageNames

Since it is slow and is called many times when articles with lots of
interlanguage links are viewed.

Change-Id: Ib826ea6f1c105ffd43e456fb1c0c3b8aa6d8e391

Profiling for Language::fetchLanguageNames

It is 16% of an enwiki Main Page parser cache hit, under HHVM.

Change-Id: Ifb2ce7592fa5b5cb9c7fb5af5949c43fecf18ca3

Merge "Split SkinTemplate.php per-class"

Merge "Fix API-related 1.25 release notes"

Fix API-related 1.25 release notes

Use the section headers from 1.24, and move an entry that was placed in
the wrong section in Iff8d444c.

Change-Id: I460551491f23352f08e64aa3f216e2e5f39b5a6f

API: ApiBase::PROP_* constants are deprecated

They should have been marked so along with Ie11a401d.

Change-Id: I029f81dabf89c1765fc514579f44a6a0c940b572

Merge "Add cURL constants test"

Merge "Add API output skin"

Add API output skin

The API output for help and 'fm' formats will soon have need of
including ResourceLoader modules on an otherwise-bare page. The easiest
way to do this is to use OutputPage, but that requires a skin. So let's
add a skin that outputs a basic page without any navigation elements or
other chrome (that may be added later, but that can wait for Design to
decide they want to design it).

Change-Id: Ifa95fae5acaa3cfbf2ca58a15f8d0c51d84b455a

Merge "Fixes to prevent duplicate rows in ActiveUser cache"

Split SkinTemplate.php per-class

Change-Id: Id964cdc3bd362d696628d62cd34615aa1fa6b014

HtmlForm: Use mw-ui-checkbox only if wgUseMediaWikiUIEverywhere is enabled

Change-Id: I5d8f1bbcea72253d71df8c9e13253ff5612be363

Merge "MediaWikiTestCase: Centralise insertPage() logic from SearchEngineTest"

Merge "Randomize the JobRunner slave lags checks a bit"

Randomize the JobRunner slave lags checks a bit

Change-Id: Iee777426776c12051761d29c90da80cea27619b1

Slave lag check tweaks to JobRunner

* Do not block forever, but wait up to 10 seconds. Likewise,
  check the lag times in memcached on startup. This at least
  lets runners avoid lagged wikis but still work on others.
* Made a few small related documentation and code cleanups.

Change-Id: Ic1339bab54cba6b6cbea7d97a80ff87c7c5c87af

Merge "Special:BookSources: Use getConfig instead of Global"

Merge "Fixed bug in BloomCacheRedis::getConnection"

Special:BookSources: Use getConfig instead of Global

follow up to I00c4a00d4a2aeb7f6b39ede6fca1725991b68d7c

Change-Id: Ice8ec212e5a5fbb0e34ad02f4fffdcc4ce1424fc

Fixed bug in BloomCacheRedis::getConnection

* The array keys need to be re-indexed to avoid picking a deleted one

Change-Id: I3c22674a0194597598d66179bb5e3625da1dd38c

Merge "Special:BookSources Add global"

Special:BookSources Add global

Follow up to I00c4a00d4a2aeb7f6b39ede6fca1725991b68d7c

Change-Id: I7d09d6b648ccd710c4e3147c0c2b33d6ecaca15a

Merge "Tweak checkbox sizes to be relative depending on where used."

Merge "Special:BlockList: Use mw-ui-progressive for search button"

Tweak checkbox sizes to be relative depending on where used.

Use ems rather than pixels.
Add margins

Bug: 70133
Change-Id: Ia6adc4bda000508f14acf4214e778118efa35be0

Merge "Special:EditWatchlist: Use destructive button and style checkboxes"

Merge "Special:BookSources: Use appropriate mediawiki.ui classes and update strings"

Merge "Make "/*@noflip*/ /*@embed*/" annotation work without CSSJanus hacks"

Merge "Move non-user specific things from Title::isValidMoveOperation() to MovePage"

Make "/*@noflip*/ /*@embed*/" annotation work without CSSJanus hacks

This reverts most of commit 2d842f14250646475b5c2ffa2fe4f5a131f94236,
leaving only the test added in it, and reimplements the same
functionality better.

Instead of stripping /*@noflip*/ annotations in CSSJanus, which is
incompatible with other implementations that preserve it, extend
CSSMin to allow other CSS comments to be present before the
rule-global @embed annotation. (This required making the regex logic
in it even worse than it was, but it's actually slightly less terrible
than I expected it would be. Good thing we have tests!)

Bug: 69698
Change-Id: I58603ef64f7d7cdc6461b34721a4d6b15f15ad79

Update OOjs UI to v0.1.0-pre (880100c45e)

New changes:
e43584b Re-clip popups after resizing them

Change-Id: Ic4fa16be8e7bb0af9b05c9ad46ddbb45880b9ccc

Special:BlockList: Use mw-ui-progressive for search button

Bug: 71144
Change-Id: I56969edb6651a26ff2579e1ed9f9da34a9536e46

Fixes to prevent duplicate rows in ActiveUser cache

* Handle repeatable-read snapshots better for web requests
* In CLI mode, handle automatic reconnects better (rolling
  back). It has to abort since it lost the lock in that case.

bug: 71086
Change-Id: I5228889cf05857c87a06f7d073b5aa46f0c240be

Special:EditWatchlist: Use destructive button and style checkboxes

Bug: 71147
Change-Id: Icddec7a3b66ba1cc86459c2776a471391e92c8c7

Merge "Add tests for Job::toString"

ResourceLoaderWikiModule: Only check content format instead of namespace

Follows up I64e86c741.

Bug: 70835
Change-Id: Ieb9e5d045ed9b59cc83749234a9157689244c597

Special:BookSources: Use appropriate mediawiki.ui classes and update strings

Bug: 71145
Change-Id: I00c4a00d4a2aeb7f6b39ede6fca1725991b68d7c

Merge "Restore Composer hook providing MediaWiki version"

Message: Correct output of wfMessage( 'non-existent-msg' )->text()

The output of Message::text() should always be acceptable to pass into
external html escaping, such as when the response is returned over an
API request and escaped by the client side code. Calling ->text() on a
non-existent key was returning the entity encoded value which leads to
double encoding down the line, this patch fixes that oversight.

Bug: 66199
Change-Id: Ieec94d4e4c7e5c36e5e68bbf01792e96368e54e0

Add tests for Job::toString

Change-Id: I00f41808af42a198a1e45a93201dd7bb3e4d9c2c

Merge "Update OOjs UI to v0.1.0-pre (4ea8e2cba9)"

Merge "mediawiki.api: Support assert parameter in getToken and postWithToken"

Update OOjs UI to v0.1.0-pre (4ea8e2cba9)

New changes:
5d58e70 Documentation: ActionWidget is non-framed by default
7ecfa7c TextInputWidget: Determine validity on blur and change.
7c0834d Localisation updates from https://translatewiki.net.
948b303 build: Split test into a lint task for quickly running those tasks
bab8bef Localisation updates from https://translatewiki.net.

Change-Id: I5ccdeb9b5815ffef674dd17fe713a6b0e87ec2a4

Merge "API: Remove XML tag highlighting from non-XML formats"

Merge "Move code to load User data from cache out of loadFromId"

Merge "doxygen: Document problem with slashes and fix a few"

API: Remove XML tag highlighting from non-XML formats

Since 926afc65c316, the transformation assumes < and > occur in pairs.
This assumption is invalid for formats such as JSON.

Implementing proper JSON syntax highlighting falls outside the scope
of this workaround and is left for separate changes, likely including
the addition of a hook. That is part of the API roadmap RfC.

https://www.mediawiki.org/wiki/Requests_for_comment/API_roadmap#Changes_to_pretty-printed_HTML_formats

Bug: 65403
Change-Id: Iff8d444c82f7efd2bd1c9f703defc4f0984e8211

Localisation updates from https://translatewiki.net.

Change-Id: I3870b75bf183b176544fdd53f34b711c4ee1baaa

Move code to load User data from cache out of loadFromId

This way, outside code can load data from cache only if present. E.g. when data
for a lot of users is needed, it'd be preferable to batch-load all that is not
yet in cache.

Since there's currently no way to check for data in cache, one would have to
load all User objects separately (which may result in lots of DB queries if
nothing is in cache) or batch-load all of them (which may result in few DB
queries, but ones that may not be needed if all data is already in cache)

Change-Id: Ia22d04744760c3b6f3264786434e218b8aa88d53

doxygen: Document problem with slashes and fix a few

Change-Id: I39f8f394e7421fca71e7d26818f7118e85de7e4f

Restore Composer hook providing MediaWiki version

Restores the Composer hook integration first implemented in
I8df66a92971146ab79cd4fcbd181e559115ca240 to composer.json.

Change-Id: I708f619d2de8a760eaaeaf6be54d8ae0db1b18d6

Reduce duplication in some Special:ListGroupRights code

Change-Id: I05bf9f3eef5f71d2d3789c8a6a1aff09ac46c14e

Merge "doxygen: Fix leading space in class member descriptions"

Merge "doxygen: Fix trailing star in class member descriptions"

Use as strict comparison to check whether an user name was provided in Special:ListFiles

Otherwise Special:ListFiles/0 will not behave as expected.

Change-Id: I1fa503b4b514a471ded798b54e867435d30b80c5

Merge "Add missing space to log message"

Merge "Add mw-ui-icon"

Disable thumbnail chaining support for PNGs

Bug: 67525
Change-Id: Id2a57d6fc6b0f415b88b7185220c33c70fee33e7
Mingle: https://wikimedia.mingle.thoughtworks.com/projects/multimedia/cards/600

Add mw-ui-icon

Compatible with IE8+ (because of the use of pseudo-elements).
This includes support for icons on the left of the element and
icons which hide the text of the containing element.

I attempted to make an mw-ui-icon-element which didn't use pseudo
elements but the problem with this is how it plays with things such
as mw-ui-button and the gutter. I'd suggest exploring this later as this gets greater
adoption.

In addition to this I have dropped mw-ui-icon-after since I don't see any
clear use cases for this yet and I think it is overengineering the solution.

Bug: 55535
Change-Id: I68a1b207d8a4af57879361921a5f8c3aeda3fd9a

Add missing space to log message

Follow-up I72c5c3fb75 (6fc2e603).

Change-Id: I9375da8542b3c2fb33cf0b692ab38722445081bc

Make linker warning useful

Follow-up to I4c397de289ae51bc463b5ae

Change-Id: I4695172b13988fc333707e09fc9d909f20adb7c1

doxygen: Fix leading space in class member descriptions

Using ResourceLoader.php as example:
 $ php maintenance/mwdoc-filter.php includes/resourceloader/ResourceLoader.php

== Before ==

Filtered code:

    /**  Associative mapping ... */ protected array $moduleInfos = array();

    /**  $config */ private Config $config;

Note descriptions containing a leading slash which was matched after the type
hint in the original code.

== After ==

Filtered code:

    /** Associative mapping ... */ protected array $moduleInfos = array();

    /** $config */ private Config $config;

Change-Id: Idcdd487ad0f4fbabdd5665abfbb8492f5bac655a

doxygen: Fix trailing star in class member descriptions

Currently lots of member descriptions in generated Doxygen pages
have a trailing star or even just a star as their description.

This is due to the regex we use to *change* the code before Doxygen
is given the code. This filter script translates the code to be
invalid PHP that looks more like Java's strongly typed class members.

The regex has been broken up into pieces for better readabilty
but not changed in any way.

The replacement is where the fix was made. Here we now replace
with "${2}/" instead of "${2} */".

Using ResourceLoader.php as example:
 $ php maintenance/mwdoc-filter.php includes/resourceloader/ResourceLoader.php

== Before ==

Filtered code:

    /** @var array Module name/ResourceLoaderModule object pairs */
    protected $modules = array();

    /**  Associative mapping ... * */ protected array $moduleInfos = array();

    /**  $config * */ private Config $config;

    /**
     *  Associative array mapping framework ids
     *      like array( 'ext.foo.tests', .. )
     * */ protected array $testModuleNames = array();

    /** @var array E.g. array( 'http://.../load.php' ) */
    protected $sources = array();

    /**  * */ protected bool $hasErrors = false;

Rendering currently at
https://doc.wikimedia.org/mediawiki-core/master/php/html/classResourceLoader.html

    bool   $hasErrors = false
           *
    array  $moduleInfos = array()
           Associative mapping ... *.
           $modules = array()
           $sources = array()
    array  $testModuleNames = array()
           Associative array mapping framework ids like array( 'ext.foo.tests', .
    Config $config
           $config *

Note the stray stars in hasErrors, moduleInfos and $config. $testModuleNames
doesn't have it because it has a multi-line block comment and presumably
Doxygen tolerates spaces in the final star sequence if it's on its own line.

== After ==

Filtered code:

    /** @var array Module name/ResourceLoaderModule object pairs */
    protected $modules = array();

    /**  Associative mapping ... */ protected array $moduleInfos = array();

    /**  $config */ private Config $config;

    /**
     *  Associative array mapping framework ids
     *      like array( 'qunit' => array( 'ext.foo.tests', .. ), .. )
     */ protected array $testModuleNames = array();

    /** @var array E.g. array( 'http://.../load.php' ) */
    protected $sources = array();

    /**  */ protected bool $hasErrors = false;

Change-Id: Id7c307dc2911ef4f1a6c2ca566c6b48735b763d7

inputs.less: Add disabled state

Bug: 71149
Change-Id: Ia97dcb26bdae65a681f3d33b2e32d59831a55191

Merge "Begin logging RequestContexts relying on global title"

Merge "Special:ActiveUsers: Use mw-ui-input-inline"

Merge "Break accidental references in Parser::__clone"

Begin logging RequestContexts relying on global title

Change-Id: I72c5c3fb75c9ceff02390c91b3735f2da90845de

Per 5a4a33a, remove support for magic quotes gpc

Change-Id: I6110b843e6c6ef3431206d25b8fd72ed4f113ee8

Merge "Remove $wgTitle from WebStart forever"

Merge "Set site logo url in ResourceLoaderSiteModule instead of inline styles"

MediaWikiTestCase: Centralise insertPage() logic from SearchEngineTest

Make it re-usable for other tests and update SearchEngineTest to
use the addDBData (which is called automatically) instead of
manually keeping track of the state.

Change-Id: I75a6951545d9824e71e00f0f96936c53b400dce6

Remove $wgTitle from WebStart forever

Change-Id: Icdd091f81547b76a403fc78f2e832ef3e85a67a2

Special:ActiveUsers: Use mw-ui-input-inline

Bug: 71142
Change-Id: Ie737b8bd74179ce916747c5e9cd23304e2ad8b16

Localisation updates from https://translatewiki.net.

Change-Id: Icba92059caac149e239ee1a90d962dbddd7e5605

Merge "mw.Title: Add method to get title relative to an arbitrary namespace"

Set site logo url in ResourceLoaderSiteModule instead of inline styles

Requires skins to use the 'mw-wiki-logo' class on the logo element
for the styles to apply. This is backwards compatible with skins
not using that class and using inline styles instead.

Bug: 56257
Change-Id: I89e32fcd27c9f07e889aa5b67d7d97d06b412aea

Merge "Delete maintenance/purgeOldText.inc"

Merge "Add maxlength of 6 for time correction in preferences"

Break accidental references in Parser::__clone

If you have a reference *to* an object field (anywhere in the call
stack) when you clone the object, the field will be cloned as a
reference rather than as a value.

So we have to break those unexpected references in the cloned object
manually, which is easy enough by making a non-reference copy and then
rebinding the cloned object's reference to this copy.

Bug: 56226
Change-Id: I9c600e9c0845b4fde0366126ce3809d74e2240b4

Delete maintenance/purgeOldText.inc

The only function it contains (PurgeRedundantText()) was superseded by
Maintenance::purgeRedundantText() in r52006, yet the file had not been
deleted. I grepped core and extensions in Gerrit for the names of both
the function and the file and found no usage.

The purgeOldText.php maintenance script uses the "new" (since 1.16)
method and has been retained.

Change-Id: I40afbd264242eab7fadc8352380779074d61d7f8

mw.Title: Add method to get title relative to an arbitrary namespace

I'm not sure where this kind of thing could be used outside of
dealing with templates/transclusion. Making it generic anyway.

Bug: 67448
Change-Id: Ie554adefec43997d362b5d7b45c30403912743b5

Merge "Remove 'move' from LogPage::getTitleLink"

Merge "Improve mod_security warning"

Add maxlength of 6 for time correction in preferences

* Supported formats (controlled by filterTimezoneInput) are:
** -12:34/01:23.  It says, "Max is +14:00 and min is -12:00", both
   of which fit in 6 (if someone uses -24:00/+24:00, it's the same width
   anyway).
** If it's not in that format, it's treated as a simple hour, which
   means it should be from -12 to +14 (max 3).

Bug: 70405
Change-Id: I47ec2c07929069cb5243c306a1c502751e57a31b

Arrays in PHP are a bit special...

Change-Id: Ic1c16e297f68417ba4a12c109e3a6e83974a82d5

Merge "Update CLDRPluralRuleParser library"

Merge "Add @covers tags to SpecialPageFactoryTest"