Hide edit toolbar Signature button in non-discussion namespaces

Most wikis only use user signatures on pages set aside from discussion,
(Talk namespaces and sometimes project/main namespaces depending on
the wiki), so having the button available everywhere is confusing.
The few wikis that need the button (especially non-content,
internal/corporate/planning wikis), can add relevant namespaces to the
$wgExtraSignatureNamespaces array in LocalSettings.

This would make it possible to solve bugs like T59727 or T53154.

Since this is a change to default behavior, a release note is added.

Bug: T7645
Change-Id: I7ccf1093b888c7b33721234349ca0ac054c3cd3f

Add base interface for setters in RequestContext and DerivativeContext

Change-Id: I819633ca5344f73a196623569bb58fd8372d6779

Merge "Improve messages related to $wgAllowCategorizedRecentChanges"

Merge "OutputPage: Add \n between </style> and <script>"

registration: Actually set the merge strategy for $wgExtensionCredits

$wgExtensionCredits is special and needs to be set in the
ExtensionRegistry. Also change the cache key so any bad cache entries
will be ignored.

Follows up 1ebb0f5659667f.

Change-Id: Iec08ab8d9ef7fe7cccde979530839ef553779658

Merge "Update the comment in MessagesAzb.php to be consistent with Names.php"

registration: Allow setting $wgCapitalLinkOverrides

Change-Id: Iaf8b44d02adf94ac7f7cb6a8a090b4069b98a88c

registration: Allow setting $wgNamespaceProtection

Change-Id: Id071eeeb3e734f7833d36610a6b4e786f2d26fa6

Merge "registration: Overhaul merging of globals"

Localisation updates from https://translatewiki.net.

Change-Id: Ib39a676086a94d604ccac932170a992ad67b4a41

registration: Overhaul merging of globals

Instead of hardcoding specific global settings in ExtensionRegistry,
create specific "merge strategies" that are used to merge globals.

Merge strategies are set for core properties in the ExtensionProcessor,
and extensions can set them for their own configuration settings using
the magic "_merge_strategy" key.

The following merge strategies are included:
* array_merge_recursive - call `array_merge_recursive` on the two arrays
* array_plus - use the "+" operator to combine arrays, preserving
               integer keys
* array_plus_2d - A version of array_plus that works on 2d arrays, used
                  for merging arrays like $wgGroupPermissions
* array_merge - call `array_merge` (default)

This changes the merging of various namespaces related settings to use
array_plus so they actually work.

Bug: T107646
Change-Id: I64cb0553864e3b78b0f203333f58bb73b86a6434

Update the comment in MessagesAzb.php to be consistent with Names.php

Change-Id: I86ae4c514eebaaea6072c9b67bd35915c7b4c3e4

ApiQueryInfo: Remove unused variable

Change-Id: I071ba28ef8ba9f8b137f5e0ea4c7e1fca35cdede

Remove classmap-authoratative: true from composer.json

The use of classmap-authoratative: true provides a measurable
performance improvement for MediaWiki on the WMF production cluster, but
that environment really uses the composer.json in the
mediawiki/vendor.git repo and not the composer.json in
mediawiki/core.git. Since this setting is merely performance enhancing
and can be worked around for WMF and other large scale users, it seems
reasonable to change the default behavior to be more friendly to
development use where there may be legitimate reasons for PSR0/4
autoloaders (eg bootstrapped tests in extensions).

Bug: T107682
Change-Id: I805c1b82eb6bc93bf3a74a8a9d07e34f2f8c645f

Use notification on successful preferences save

"Now 98.6% leaner!"

* Use Xml::tags() for cleaner (old) successbox generation.
* Use #mw-preferences-success for the old successbox
  to make its removal more specific.
* Use the mw.notify box for saved prefs message.
* Remove box on keydown or mousedown in #preftoc or
  .prefsection.
* Remove success=1 querystring on load to prevent
  unnecessary reappearance of notification on reload.

Bug: T19496
Change-Id: Ibecbe21aa52ddc061d4bb27815f6fa5161a96735

Use XmlSelect in SpecialListusers

Bug: T93234
Change-Id: I25cdd8f7eb1bbfd893d201f6c68f5ccfabe70bf3

Localisation updates from https://translatewiki.net.

Change-Id: Ibdd18216129fb8ab0180d07083521e4a0520c30c

resourceloader: Remove ESI support (previously disabled)

These were never enabled or used in production and are not
compatible with the upcoming async changes (T107399). To avoid
having to maintain compatibility with this, remove it for now.

The current on-going request to operations for ESI support is unrelated
to this code.

Considered making makeResourceLoaderLink() protected as it's not
used anywhere in @wikimedia Git outside mediawiki-core. And the unit
test actually treated it as protected already. However it's called
in SpecialJavaScriptTest so leaving that as-is for now.

In Icba6d7a87b239 the signature will change again with the removal
of the $loadCall parameter, which is obsolete in an async world
due to document.write being forbidden.

Change-Id: I9f557cc794638ffd15329934865e21e1027f7cfa

OutputPage: Add \n between </style> and <script>

The ResourceLoader does not add a newline anymore.

Change-Id: Idfdbd0a8836ab2ec7abef1530ca883fb6d618eb4

registration: Fix merging of $wgRevokePermissions

It's the same as $wgGroupPermissions.

Change-Id: I9c289219c53314970e7af5998c00d6a372bb00cb

registration: Fix "namespaces" schema to match code

The "namespaces" property is supposed to be an array of objects, not an
object. Document the properties of the individual objects, and mark the
actually required ones as required.

Change-Id: Ie3cee3e720b962073a1ff098659faa9f11d44fe3

Merge "Revision::newNullRevision: Make sure the Title is set in the constructor"

Revision::newNullRevision: Make sure the Title is set in the constructor

Revision::__construct() calls Revision::getContentModel() to initialize
the 'content_model' field. If a Title object is not set, it will try to
look up the page in the slave database, which might be behind. At that
point it will default to wikitext, which is problematic during imports.

By passing the Title object in the constructor instead of afterwards,
the content model should be intialized properly.

Bug: T91170
Change-Id: I7fea996ea5b723967272af8cb344150aea10bb54

Merge "mediawiki.filewarning: Use 'visibility: hidden' instead of 'display: none'"

Merge "mw.widgets.DateInputWidget: Allow not having a date selected"

Revert I4afaecd8: Avoiding writing sessions for no reason

I4afaecd8 reduced the frequency at which the TTL of sessions keys is reset, on
the assumption that we were doing it a whole lot more often than necessary.
We can now assume that this assumption was false, because an uncommitted revert
of this change which I pushed to production earlier caused the rate of session
loss errors in the logs to plummet.

Bug: T102199
Change-Id: Ie67f4ca000afcf3d4f44155c13f91cd4c286866d

Fix spelling mistake in ResourceLoader inline documentation

Change-Id: Ie099eed12ba45aeb1d3b8b08c911222af25e42fa

mediawiki.filewarning: Use 'visibility: hidden' instead of 'display: none'

Setting 'display: none' causes OOjs UI to get confused when it's
trying to show the popup and calculating its position.

Bug: T107554
Change-Id: Iaf8d728c58efd67cdb965b4cd529164aaabc02be

Merge "Monolog: Add Formatter that uses MWExceptionHandler::getRedactedTraceAsString"

Localisation updates from https://translatewiki.net.

Change-Id: Ibfb6ba5ceb9cc7cc8365891a76936c7754afabea

Monolog: Add Formatter that uses MWExceptionHandler::getRedactedTraceAsString

Add a Monolog Formatter class that uses
MWExceptionHandler::getRedactedTraceAsString when outputting stack
traces.

Bug: T107440
Change-Id: Ic580c137e27aac95435f7b073a18cf61820b172f

mw.widgets.DateInputWidget: Allow not having a date selected

* Allow empty value, that is, no date being selected. Display
  an appropriate label when that is the case.
* Start with no date selected, rather than today's date.
* When the field is empty and gets focus, set the date to today, on
  the assumption that this is likely what the user wants. Permit
  emptying the field, though.
* Highlight today's date on the calendar.

Bonus cleanup:

* Update the UI from #setValue, immediately when needed.
* Improve validation of incoming values.
* Correct some documentation.
* Correct some bad copy-paste in styles which caused disabled widget
  to not display correctly.

Change-Id: I7a1f7ff20eb6fc21ea59ecfe48deb9305c8e29e8

resourceloader: Use WrappedString library to merge RLQ inline scripts

Update unit test to account for the internal 'html' prop now
being an array instead of string. And update expected values to
no longer have a trailing line break.

Bug: T27202
Change-Id: I105b6ef2e64ab8b891562e16940edb88592bd415

XmlSelect: Tweak documentation

Correct some variable names and types, also some words.

Change-Id: I7ff0664f7ac4f26be8427a2140e2280aa7620657

Use Perso-Arabic numberic system for South Azeri (azb)

Change-Id: Ifb0e8edc1ca4d0720ceb048a46589158b34eec52

Merge "Added sanitizeHdrs() tests for Swift"

Merge "array_shift returns null on empty array, not false"

array_shift returns null on empty array, not false

Change-Id: I690f9dd0de11ff60f9cba9f401c6e456e7ba8bd6

Add wikimedia/wrappedstring library

Depends on I9a468fd65f1 in mediawiki/vendor.

Change-Id: I3f952aa059d930130a8d712e39798cc1031e0a25

Merge "Actually run the Less compilation tests"

Make Special:MIMESearch expensive again

Anything slow enough to get reaped by query timeouts is expensive, right?
This reverts change I88826ae7bf.

Bug: T107265
Change-Id: Ic9ed00f164d8fa076bb53649f77a9e5ac1e33a6d

Merge "Switch search-suggest message to use Message::parse()"

Merge "Throw exception on falsy server in RedisConnectionPool."

Merge "Use XmlSelect in ProtectionForm"

Throw exception on falsy server in RedisConnectionPool.

Surprisingly, IP::splitHostAndPort does not return false for these.

Also, tweak documentation of splitHostAndPort.

Bug: T107498
Change-Id: I4230b1e7ba0fa2d28be8de3e0e7b064e58f0579c

resourceloader: Restore minification for 'user' and 'site' module

Follows-up 19a40cd3ad, Id599b6be4.

TODO: Change minification to happen per-module instead of per-request so that
these hacks aren't needed (T107377).

Change-Id: Iaa281ee117f2ae7a51884d256dfbb1807224fc52

Merge "Add getImageInfo to mw.Upload"

Merge "Use XmlSelect in WebInstallerPage"

Use XmlSelect in ProtectionForm

Bug: T93234
Change-Id: I47e08fb364dda6e4f59cd84c4c135e267e2c7bd9

Use XmlSelect in WebInstallerPage

Bug: T93234
Change-Id: Iba0067b4fac381e715e92d87c682261eef8d6669

Merge "Revert "Use OOUI HTMLForm for Special:Watchlist""

Add getImageInfo to mw.Upload

Just a little shortcut and so you don't need to rely on the promises.

Bug: T107460
Change-Id: I1e36f3cd8cb6b50ec5db6d77512ce03cd98a2caf

Localisation updates from https://translatewiki.net.

Change-Id: I9b25e1abaaf2e6e4ba21e4c540a79bdb30521cea

Revert "Use OOUI HTMLForm for Special:Watchlist"

Issues with spacing (T107311), probably shouldn't have
been merged right before the branch cut.

This reverts commit 9508c5bd576ee4206eb8e89b7d2ba82985f4e151.

Change-Id: Ibf2ca5a33b8ab0f7381c720c6c92fbfd7a7c819d

Merge "Add pageid to API parse output"

resourceloader: Convert inline statements to queued functions

Instead of having inline statements be plain statements wrapped
in an if-conditional block, convert them to inline functions
pushed into a queue.

The queue is kept in-memory until the startup module is loaded
at which point it transforms into a function that is immediately
invoked.

This is a prerequisite to making the top queue asynchronous.
Until then these functions will just run immediately as they
occur in the HTML after the startup module.

This is based on the previously reverted commit e86e5f8460.

Bug: T107399
Change-Id: Ifb38efca219c10ab973ad4c4ebb21c6a4239b005

Add pageid to API parse output

Bug: T27134
Change-Id: I4ef765e18020f93d7861e62d20649c89fca0f828

Use STRAIGHT_JOIN in SpecialWhatlinkshere query

* Also added a type hint to avoid IDE errors in this area

Bug: T106682
Change-Id: I825eedaceb0c0323de85466e48582d72c57b201a

ObjectFactory: add support for setter injection

Extend ObjectFactory::getObjectFromSpec() to support setter injection on
created objects when the specification includes a 'calls' member.

Bug: T107440
Change-Id: Ie2ece2e9658dd2d895d3935da4dc2da8a0a316e2

Merge "logger: Fix undefined variable $data"

Added sanitizeHdrs() tests for Swift

Change-Id: I2e3c3225c729e5220ca16f6ef4518da49e7f721c

logger: Fix undefined variable $data

Follows-up 77a397125f. Also add unit test that would've caught
this "PHP Notice: Undefined variable: data" error.

Change-Id: I8a3bd9c8b685c2aa7a466e3d3c61ffa027be02fa

resourceloader: Ensure 'user' loads after 'site' (asynchronously)

Regression from 19a40cd3ad which made the 'site' module load asynchronously,
but the 'user' module was still loaded synchronously which meant it ran before
the site module finished.

Full test script at <https://gist.github.com/Krinkle/5db1d237da241b243485>.

Also:
* This changes the 'user' module to load asynchronously.
* Similar to 19a40cd3ad for site module, this makes the styles for the user
  module load twice. Harmless but doesn't look pretty internally.
* Remove the obsolete XXX-comment from 0b5389d98d (r56770).
* Add comment documenting the fact that the 'excludepages' feature can cause
  User/common.js and User/vector.js to be mis-ordered when the user previews
  common.js edits. This has always been the case (since 2009) and is merely
  being documented here.

Bug: T32358
Bug: T106736
Bug: T102077
Change-Id: Id599b6be42613529fb7f4dd3273f36ccadb3a09e

Merge "Decolonize 'viewsourcetext' and 'viewyourtext' messages"

Merge "HTMLForm: Correct documentation"

Have RedisConnectionPool explicitly implement Psr\Log\LoggerAwareInterface

Follows up 93f360a01b49

Change-Id: I0266be9771b7bf58de9f573249f28c6f28adf059

RedisConnectionPool: convert to PSR3 logging

Convert from wfDebug* logging to Psr\Log\LoggerInterface. Use structured
logging data to tag log messages the associated redis server.

Bug: T88649
Change-Id: I5fc4c68e52b13a688bdcc93d9defc9f973323241

resourceloader: Implement support for 'site' into mw.loader

* No longer a dedicated <script> with only=scripts.
  This means it creates no extra script request and becomes a versioned
  request using data from the startup module.

* No longer in group=site.
  This means it collapses into the existing bottom queue.
  Not even one dedicated script request, but zero.

* No longer exclude from module storage. This can be cached like any other module.
  It was previously excluded because it was already loaded separately.

* Change mw.loader#execute to special-case the 'site' module with $.globalEval.

* Add hack to ensure the styles of the 'site' module still load without
  JavaScript, in the top, and after the ResourceLoaderDynamicStyles marker.
  This unfortunately stays its own request. Not sure how to avoid that.

Bug: T32358
Bug: T106736
Bug: T102077
Change-Id: I291a8c3aae1a71760bec58161891c1bd77c9b724

Localisation updates from https://translatewiki.net.

Change-Id: I5de4b112125b9be8f2aa2ae23a0defaaec323407

Merge "Populate rev_content_model when a move causes default content model to change"

Merge "Add Content-Length header to thumb.php redirects"

Add Content-Length header to thumb.php redirects

Without the Content-Length header, the response is sent with
Transfer-Encoding: Chunked, which is somehow mangled by
mod_fastcgi. Varnish then claims the response is malformed and declines
to process it, sending the client a 503 instead.

This is a followup of 0ee9e717f49b9e15bd63fd66f3d0967939c35b05

Bug: T84842
Change-Id: Ia610a43789e6ff14cfc0964f285bbec39c890152

Merge "mw.widgets.CalendarWidget: Silence jshint for one line, not entire file"

mw.widgets.CalendarWidget: Silence jshint for one line, not entire file

Follows up I6f760f7c32e2e6ed2008e897af72fb9e17dd663b

Suppress warning at the specific line that uses the static keyword
rather than the entire file.

> Expected an identifier and instead saw 'static' (a reserved word). (W024)

Change-Id: I0e4f9062b01b82893205da9e72c41acae3d6967f

Switch search-suggest message to use Message::parse()

This was recently made more explicit and use escape() which matches
the english definition, but frwiki (and maybe others) has overridden
these messages on wiki via MediaWiki:search-suggest to include HTML
code.  To get the correct output from that HTML we need to use ::parse()
instead of ::text()

Change-Id: I5251dead7f1262728b4cab5bcfc5b0d1b9acc761

Special:WhatLinksHere Don't show edit links for non-direct-editing pages

It's possible, that pages links to a page, which aren't editable directly
through action=edit. Don't show an edit link for such pages.

Bug: T106680
Change-Id: I01ff6dbd5b4e9fff84795f7c3d8ada23c09c7ae8

Avoid revision lookup post-save in InfoAction::invalidateCache()

We just saved the page, so we know the revision id. Pass it on to
InfoAction::invalidateCache() so it doesn't have to be looked up again.

Follows-up 0452855044d2d.

Change-Id: I990c0da09fae94f403f3550069036d3f208090a6

Defer the InfoAction purge in onArticleEdit()

Change-Id: I73fba15c26c12bea1abad219611e660fb3901a86

Merge "RedisBagOStuff: if no alternatives, skip master link status check"

RedisBagOStuff: if no alternatives, skip master link status check

If RedisBagOStuff::getConnection() is able to establish a connection, only
check the master link status if automatic failover is enabled and if there are
other viable servers left to consider. If there are no servers left to
consider, or if automatic failover is not configured, just return the
connection handle without subjecting it to further tests.

This will have the side-effect of making RedisBagOStuff compatible with
Nutcracker, which does not implement the INFO command. This is because when
MediaWiki is configured to use Nutcracker, the server pool will consist of a
single server (namely, Nutcracker itself), and thus there will be no other
server to consider, so INFO will never be executed.

Change-Id: I3812ec5a0b22df122bdf44350bc0496574c02ce8

Update OOjs UI to v0.12.2

Release notes:
 https://git.wikimedia.org/blob/oojs%2Fui.git/v0.12.2/History.md

Change-Id: Ic7f4be9eee8c17e00270de8b731ec6206a35f9ad

Merge "Track key authentication metrics"

Merge "Update liuggio/statsd-php-client: v1.0.12 -> v1.0.16"

HTMLForm: Correct documentation

Change-Id: I84f4d886907b2ae988956563fda48e78afb3cfa6

Track key authentication metrics

Logs a 'login' event for logins via Special:UserLogin
and API action=login. Does not log for implicit login after
account creation and for autologin (e.g. based on an active
CentralAuth global login).

Logs an 'accountcreation' event for account creation via
Special:UserLogin/signup and API action=createaccount. Does not
log for autocreation.

Both successful and unsuccessful attempts are logged, except for
failures that throw exceptions (internal errors + some permission
errors).

Bug: T91701
Change-Id: I101b11d05400b073065da10f1e537412309d9102

Update liuggio/statsd-php-client: v1.0.12 -> v1.0.16

vendor patch: I307b58f4abe4d615ac9b2f86f50d0c5d6b906d83

Bug: T106457
Change-Id: I6e013485175c63a984d2de205dc45704483d149b

Log errors in Http::request()

Instead of silently discarding errors in server-side HTTP requests,
log them to a 'http' channel.

Make ForeignAPIFile::httpGet() (which sort of reimplements Http::get())
log to the same channel, for consistency.

Bug: T103043
Change-Id: Ibf552e22adc7fde4a751f92e92dad6ceba2f335c

Localisation updates from https://translatewiki.net.

Change-Id: I550fd6329d15eff1f06d70dd6fde7dfb519f9adb

Fix Southern Luri autonym

Received as feedback from Southern Luri community:
"Dumeni" means "Under" (Southern)

Also removing Northern Luri from the fallback list.
Southern Luri speekers can't understand Northern Luri,
but everyone in the region can understand Persian
(official language in the region).

Follow-up to I68154c19880ee59da05493fb72e735e61c3183aa
Follow-up to I1099cb846f029df850126e59306849b0eec88e7b
Follow-up to I3cc17d61c6f99b42f6befb39a4c1d6544558cd7e

Change-Id: Iaa1d3da94261f23a2e7b6f488ebfcce5cf457413

Merge "Don't retry invalid thumbnail requests due to impossible width"

Merge "Implement redirects in CssContent"

Merge "Gather more information about pre rendering 500s"

Merge "Put "userjs-" in <code> in apihelp-options-description"

Implement redirects in CssContent

Just like ad9f14d662f959 which was for JavaScript. The redirect will be
of the form "/* #REDIRECT */@import url(...);".

Bug: T73201
Bug: T35973
Change-Id: I10bae44af4b4923f8797172702974cd45dc25ab4

Fix typo: requiered -> required

Thanks to Ricordisamoa

Follow-Up: I095c545f77aa50d6be4cd48588bd1ae1c82cf343
Change-Id: I3c40b02865170ba0391f3637a5bebb6058c053fd

Don't retry invalid thumbnail requests due to impossible width

At the moment this isn't going to work in production, because varnish
turns 400s into 500s. But I'll try to fix that separately.

Bug: T106740
Change-Id: Id156ee4ac986ad2a6d7e49dfe8aa7577764eca11

Gather more information about pre rendering 500s

Bug: T106740
Change-Id: I4a1436f1724fcc74d4c1076b21fcdb3b5d58b1de

Merge "Fixed various FileBackendDBRepoWrapper errors found in IDE"

Merge "Clear the stat cache in addMissingMetadata() to avoid more POSTs"

Merge "Improved addMissingMetadata() on POST failure"