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

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

Merge "Remove $wgTitle from WebStart forever"

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

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"

Merge "Allow logged-in users to view and use the login form again"

Allow logged-in users to view and use the login form again

The fix for bug 15484 (d0439af8) has introduced the behavior of
automatically redirecting users who view the login form when already
logged in to the requested 'returnto' location. If it was not given,
users were redirected to main page instead.

However, that has annoyed people who often switch between several
accounts and have grown accustomed to the old behavior of being able
to log in while logged in. Given that there are no conflicts between
these two features, let's just restore the old behavior when
Special:UserLogin is visited directly (no 'returnto' location given).

This reverts 5dfc57eb which removed then-dead code for showing login
form to logged-in users and tweaks an if() condition.

Bug: 70855
Change-Id: I7e40c13a6ca566b4d66d943c006af9edb6941ee9

Merge "Fix special case at Special:MediaStatistics"

Update CLDRPluralRuleParser library

Upstream release:
https://github.com/santhoshtr/CLDRPluralRuleParser/releases/tag/v1.1.3

Does not have any feature changes.
License changed to MIT

Change-Id: Icb4c00cca86083d77028a4774122acd8c595152e

Merge "Fix duplicate messages key in 'mediawiki.special.preferences' definition"

Improve coverage of MailAddress::toString()

Change-Id: I8d5a1e8f293afa03d94167b722d3078f467cf59e

Add @covers tags to SpecialPageFactoryTest

Change-Id: I997cd4296a60eb9950f7fee0caed7a2349e07bd2

Fix duplicate messages key in 'mediawiki.special.preferences' definition

Follows up: Idb00f50a

Change-Id: I852e5965de8e0bee7e976637df95daded691e11c

Localisation updates from https://translatewiki.net.

Change-Id: I2a459df4c43343ca7988e8c702e68b912aa01112

mediawiki.api.edit: Remove "format: 'json'"

"format: 'json'" is in the defaultOptions in mediawiki.api.

Change-Id: Ifdf61205d94dbf918be5427cef89a037ce9f08a2

Merge "Fix typo in release notes"

Merge "Copy translations of Special:MyLanguage alias from Translate"

Fix typo in release notes

Change-Id: I2651bc9c8b6ace473a503719e2d2930cfb50bc04

Copy translations of Special:MyLanguage alias from Translate

Tuvan (tyv) didn't have a special page section, so now it's added.

See also I6f84eaf414b6d4239522bad14e90b1cbfc82590f.

Bug: 69461
Change-Id: Ie6a4d878b87ee030f9c042b43f7b1fb424b54099

Merge "Unset redirect param in MoveLogFormatter::getMessageParameters"

Merge "Move default logo to resources/assets/ directory"

Merge "jquery.confirmable: Add a space between buttons"

findHooks.php, importSiteScripts.php: Use format=json

* Also added query continuation support to findHooks.php.
* Also fixed query continuation support in importSiteScripts.php
  (broken by 2b3f4d821cd4).

Change-Id: I7ef62d370f5e2f598ac4c5857ac0dbf3ee4c8fa2

CSSJanus: Add Apache license header

Follows-up f9e8d61 which removed the Python library and its copy
of the Apache license file.

Change-Id: Ic3ed4a2fb935e713916cb8eb6ba1673839f40532

Fix special case at Special:MediaStatistics

When a percentage was equal to 100 %, makePercentPretty
determined it should have zero decimal places, and the
resulting string "100" had "trailing 0's" removed, resulting
in a completely wrong "1".

We might either check if the number contains the decimal dot
prior to trimming, or spare the hassle for the special case
completely.

Change-Id: I15ac5caa275d72909adba27b6b88824a830bd574

Merge "UserTest: Fix edit count test which incorrectly added user to db"

UserTest: Fix edit count test which incorrectly added user to db

Although 'added' to the database, the user name was cleared
in User::loadDefaults() and not added correctly to the database.

Then if one has BetaFeatures extension, then the test fails.
Via a hook, BetaFeatures does "User::newFromName( $user->getName() );",
that produces a null object as name is '127.0.0.1' and then
$user->getOption() fails.

loadDefaults() is automatically called by addToDatabase() with correct
parameters, so we can just remove the call.

Bug: 68626
Change-Id: Ibb5c60192eb83b2608f72b59d7705854549a5dac

Merge "maintenance: Remove cssjanus.py"

Merge "DatabaseMysqlBase: Use VERSION() rather than server_info"

maintenance: Remove cssjanus.py

Follows-up r55510 (aa4d684a82) which added it for use with the
Vector stylesheets at times where we didn't have CSSJanus.php.

This has been unused since we introduced ResourceLoader and the
PHP-port of CSSJanus.

Change-Id: I2c9a022354617060d99bf1c44f0726b495ec070d

Move default logo to resources/assets/ directory

The problem here is that the path to 'wiki.png' is saved in users'
LocalSettings.

We likely should not remap the path, like we did for footer license
icons in Ic7c32e56:

* It's likely that users changed their logo image by overwriting the
  file in skins/common/.
* If the normal upgrade process is followed (overwrite-uploading new
  files), the old file will still be there with the skins/common/
  directory.
* If it does cause problems, they'll be rather easy to notice and fix.

On the other hand, maybe we should?

* This is going to be annoying for git users.
* It will bite anyone who deletes all MediaWiki files when upgrading
  via tarball, which is more likely with the recent skin system
  changes encouraging users to remove old cruft from skins/ directory.

Bug: 69277
Change-Id: I175fe57048ebf9d348fb2fe67bf62cf5df389003

Fix CSSJanus flipping in LESS mixins and remove broken custom LESS functions

Custom LESS functions are problematic for us for a number of reasons,
as outlined by Timo on bug 67368. We should get rid of them.

The only use case was implementing CSSMin data: URI embedding in LESS,
which used to be impossible due to lessc not preserving comments (bug
54673). However, thanks to new syntax added in f3779e06 we can insert
the annotations in such a way that the compiler won't mess with them.
The same technique is used in OOjs UI since 584ed144.

The LESS-function-based embedding implementation also meant that we
were unable to flip images for RTL (bug 66091 and friends: bug 66773,
bug 68326). The annotation one doesn't have this limitation.

Bug: 67368
Bug: 66091
Bug: 66773
Bug: 68326
Change-Id: I3062346ed63272a1c22b5df27b4cc1de2a699d9a

Set mw.config wgFileCanRotate only on Special:Upload instead of
site-wide

UploadWizard also uses wgFileCanRotate and set the variable by
I3c50b56e6a11742cfc1c02d3d8dbeb0c151b0e91 itself.

Change-Id: Iff6a952c8f4f21c0e27bd6b4191f3b0b453eec03

Merge "Automatic direction for the license in Special:Version"

Localisation updates from https://translatewiki.net.

Change-Id: I38d6fce89373f5ec424d44c757d3011ae9ea7eb4

Merge "Strict standards: Static function PasswordTestCase::providePasswordTests() should not be abstract"

Strict standards: Static function PasswordTestCase::providePasswordTests() should not be abstract

Caused by 96771e3a655bad2d3a9440fababaf2dcf5a62bc5. No idea why that
change merged successfully.

Change-Id: I946f179699a5f249b032c4c8f90a9e6d2b033f55

Merge "UserTest: Don't rely on the behavior of unknown user options"

direction for the license in Special:Version

Change-Id: I357d92eca18de9a7a79ddfb215337ec231250ca3

UserTest: Don't rely on the behavior of unknown user options

They are not actually expected to be preserved, although they might be.

Change-Id: I34e15f3d1e5b9b825e6fc14814c8e9a6ac151ce9

Improve mod_security warning

Make it detect mod_security2 as well, and make the warning message
scarier.

Bug: 39463
Change-Id: Ic8dbd8425299f60e9fc0b54ff1b9a9a991d24190

Unset redirect param in MoveLogFormatter::getMessageParameters

The redirect param is used to get a different message key, so there is
no need to pass it to the message, because the value is stable.
'logentry-move-move' always get a '0' and
'logentry-move-move-noredirect' always a '1'
The value is only visible with uselang=qqx

Change-Id: Ib6a3303e7401e29561165355b3611c3c123bfcd0

DatabaseMysqlBase: Use VERSION() rather than server_info

I noticed enwiki's Special:Version page was reporting the version
of MariaDB 10 as "5.5.5-10.0.11-MariaDB-log" rather than just
"10.0.11-MariaDB-log". This change should fix that.

Change-Id: I6bf7e27e88014f70594b33d089636b09b6c97527

Merge "Completely remove skin autodiscovery"

Completely remove skin autodiscovery

Also update documentation.

Bug: 65748
Change-Id: I393896281e901de2000c09c0abd16ee2a0f41e2f

Remove WMF deployment hack for 'Enlarge' image

Follow-up to I849a2222. To be merged 31 days after that cut lands
(i.e.,  after 19 September).

Bug: 69673
Change-Id: I7a920105cadf3a4dea7d3bffee7ae6ff08e40887

Merge "Fixed --maxtime handling by JobRunner"

Fixed --maxtime handling by JobRunner

Bug: 71073
Change-Id: I4ddebd5aad27d0882dd2e4614df91ac565a71d2d

Reduce new button min-width from 80px to 70px

80px is too large for mobile and causes some buttons to wrap to
other lines.

Change-Id: I00b44508e7783957a8c24c4890cc1ba4dfe265da

Bump 1.25

Change-Id: I361206d6c0522e89f685a1b76fec56ec593032c5

Remove redundant $db->resultObject() calls

DatabaseBase::query() already passes the value returned by the
database driver (usually an object or resource for SELECT queries)
through $db->resultObject(). There is no need to pass it through
the method a second time to get a ResultWrapper.

Change-Id: I5d34ab2bffb7f236d2f824b4ef0e8670f89d4599

Merge "Re-apply Ic917c7d, don't cast SpecialPageFactory::$list to an object"

Remove 'move' from LogPage::getTitleLink

LogPage::getTitleLink only called for log types which are in
$wgLogActions, but move is not there.

Change-Id: I230b25988f9a420970d807437263798e5ad1db69

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

i18n: Native digits on 'mergehistory-success'

Change-Id: If3319c20362785a4b0f875ad8b5c1526e3953b6c

Improve test coverage of Title::isValidMoveOperation

Change-Id: Ifd81b0a894714ca94ab3e627ba83c881d96c3377

Localisation updates from https://translatewiki.net.

Change-Id: Ibdc7c61789400b1ab18f05da42ecd62c54cc7bb3

Added missing NULL value to sql insert statement.

In tables.sql the page table is defined with 14 columns.  Following
this, the dummy table insert statement included 13 values causing
failure of the installation process.  This change adds the required 14th
value.

Bug: 71022
Change-Id: I8f8ee95fb4c0147ac09232ef253ecf7784b7fb52
Signed-off-by: Bill Traynor <btraynor@gmail.com>

Merge "Fix some issues with Microsoft SQL Server Driver"

Fix some issues with Microsoft SQL Server Driver

* INSERT IGNORE now works properly
* Only return an inserted id if the INSERT was actually successful
* Fix syntax error when doing LIMIT queries

Bug: 71024
Change-Id: I4dddc5c9c234b17040a28937987406511ec6ea9f

Merge "Remove unused message"

Re-apply Ic917c7d, don't cast SpecialPageFactory::$list to an object

applied on top of Ia2107dc

Change-Id: I7420b9ec9df2ebd8e983fddfc7ee17655788cfde