User rights API: Abstract out some stuff about core's form into separate methods

To we can make a CentralAuth API module for global user rights.

Bug: 71495
Change-Id: I139c43e5eaa1fb63b75baddbafb7caa3c964d6b5

Merge "Move editoptions styling to mediawiki.skinning/interface.css"

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

ResourceLoaderFileModule: Support fallback in 'languageScripts'

Bug: 58139
Change-Id: Idd7657aa48a9eb2b075a85f77e066b425f330890

Merge "Make mediawiki.action.view.dblClickEdit recheck preference"

Make mediawiki.action.view.dblClickEdit recheck preference

There's a preference which makes the browser go to action=edit when double
clicking on the content. In VE we want to disable this while the editor is
open. With this check in place, we can turn it off by falsifying the value
of the preference (and then restoring the original value when VE closes).

Bug: 53058
Change-Id: I0f6225b25ad279755548361f82b855727758da2b

Merge "Fix string callbacks for special page registration"

Fix string callbacks for special page registration

Currently registering a special page with a factory by passing the
callback string does not work because the is_string check comes
before the is_callback check. If we change the order of this if-
clauses we can also support callbacks like 'Factory::createPage'.

Change-Id: I751f97026ed8c580732c74d66b2804dd180ed16f

Merge "mediawiki.action.history.diff: Remove white background from table.diff"

Merge "Add ability to pre-render thumbnails at upload time"

mediawiki.action.history.diff: Remove white background from table.diff

Skins where a white background works generally already have a white background
behind this. Skins where it doesn't work generally just wind up overriding this
back to the default (transparent).

Simply removing the background should simplify things.

Change-Id: I1a906f533f7ae2697ee18c998f316a0957ac8bda

Couple of type hints for Block

Change-Id: I9d16fbc2415fc4e751edc76be4dc3c7a62bd62c0

More sensible behavior when special page aliases conflict

Right now, SpecialPageFactory::getAliasListObject() just chooses the
last-seen alias and allows any alias to completely override the page's
"canonical" name (from SpecialPageFactory::$list or $wgSpecialPages).
Although the latter doesn't come up often since (almost?) all special pages
have their canonical name as one of their English-language aliases.

More sensible behavior is to always prefer the "canonical" name over any
conflicting aliases, and to prefer an alias that's the first alias for a
special page over one that is a fallback.

Also, when a special page's first alias winds up not actually referring
to that special page, we MUST NOT go redirecting other names for that
special page to that wrong alias.

Bug: 70686
Change-Id: I4b17ec0fdc87b4b0d7ae9d9eea7ffacb54dd6891

Minor code cleanup and a doc type cleanup

Change-Id: Ib89226a83738d1205405eac3926bfaa578966133

resourceloader: Clean up $localBasePath variable assignment

No need to set a variable to value of itself. Also consistent
with the way $remoteBasePath is set, already.

Change-Id: I9b71322592a5ca2d85927fcf6732096f758847e9

Merge "Fixed header in file history"

Merge "Limit active DB connections spawned in waitForAll() to one"

Merge "Use a less confusing column alias in findMissingFiles.php"

Limit active DB connections spawned in waitForAll() to one

* If new connections are created, they will be closed before
  making the next ones now.

Change-Id: I289d81ec00d3e1e313624e2a4c28a67bfb317feb

Fixed doc typo

Change-Id: I80aa85337357d25b1973b4a266a3c961843b08b6

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

Fixed header in file history

Change-Id: I622ddac6ea5bd1a3f8c14253e70c1a5a4c56b478

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

Use a less confusing column alias in findMissingFiles.php

Change-Id: I9c0a948e6a0dc2ed604fb8c2828fc6e5a94fec9f

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"