Merge "mediawiki.searchSuggest: Show full article title as a tooltip for each suggestion"

Merge "Adding header font change to release notes for 1.24"

Localisation updates from https://translatewiki.net.

Change-Id: I4d2d181fa0ccda0dd781008d4ed0e68103de451a

Merge "Correctly parse 'redirect' XML tag during Special:Import."

vector: Add tabindex back to search box

Another regression, this time in Iaea60834. This brings back the
implementation as it was originally before I220057c799bef7e removed it as
part of the collapsibleNav module.

Bug: 29199
Change-Id: I3f29c3d80e43b9df16b397fff3dc983dc60652a7

Added BeforeHttpsRedirect hook allowing extensions to cancel forceHTTPS

For ZERO, we need to be able to avoid forceHTTPS from taking place in
case user logged in while on wifi, and later switched to
zero.wikipedia.org for the carrier who doesn't support HTTPS.
In that case, the extension will need to be able to cancel redirect
and force regular HTTP handling.

Open Question: should this code log off user in case hook returns false?

Bug 65567

Change-Id: If04c83066c5d47b3c04ad7674e3c4e95a4cd464b

Add release notes for Ie3f7273c

Change-Id: Id39c4d779d138493a2a7a1db2cc4388b125e5bb5

Allow filtering log entries by namespace (API)

Add parameter lenamespace to the API, allowing filtering of log entries by
namespace.

Change-Id: I53c4c6411e0b9e6383969afced0e4c193f1b64a1

Correctly parse 'redirect' XML tag during Special:Import.

Bug: 65481
Change-Id: Id9b3b7878b2e7b6fc7a06b163e5bac60e700490e

Tweaked HTMLCacheUpdate immediate run treshold

Change-Id: I123c27862821e01fe232f55d984905fcda2a2498

Reduce Title::invalidateCache contention a bit

* Lots of deadlocks seems to happen on page deletion,
  where the ID is likely 0 (this also has the worst locks)

bug: 37519
Change-Id: I028783a998946615b93e1425c6f69e00afd7b1bb

Add Composer managed libraries

This is the initial step towards implementing the Structured logging RFC
[0].

The Psr\Log library is declared as a requirement in the composer.json
file and Monolog is suggested as a package that can enhance MediaWiki.

To use this in a local checkout:
* Install composer [1]
* `composer update`
* `composer require monolog/monolog 1.9.*`
* `composer dump-autoload --optimize`
* `git update-index --assume-unchanged composer.json`

Additional work will be needed to provide both the Psr\Log and Monolog
libraries available for deployment to the Wikimedia Foundation servers.
The Psr\Log library will also need to be packaged in the release
tarballs.

[0]: https://www.mediawiki.org/wiki/Requests_for_comment/Structured_logging
[1]: https://getcomposer.org/doc/00-intro.md#installation-nix

See also: I1431b24 (Monolithic implementation)

Change-Id: Ie667944416187cfd2ae6016c9e2fa28f4204bcd7

Merge "Add missing possible errors to ApiRollback.php"

Update OOjs UI to v0.1.0-pre (7b283a9dcc)

New changes:
71d9671 Add more Agora dialogs styling
457a0ef Localisation updates from https://translatewiki.net.
c92ba96 Compute directionality for frame correctly
f3414b8 Localisation updates from https://translatewiki.net.
90fbb6f Localisation updates from https://translatewiki.net.
1d4d52a Localisation updates from https://translatewiki.net.
08d1c7b Add ability to put a header at the top of a toolGroup pop-up

Change-Id: I6f4adbca80abeec0931380233e0e45a9f45af330

Localisation updates from https://translatewiki.net.

Change-Id: I1ac7593bbc66a78e74a62ea03bab4fd105728a28

Merge "Remove Vector from the default LESS import path"

Remove Vector from the default LESS import path

When Ia193571a added Vector to core for VectorBeta's benefit it introduced
a regression that broke all skins that used imports like 'variables.less'
which happen to match a filename used by Vector.

These paths aren't supposed to be exposed anyway, and VectorBeta has been
fixed to no longer rely on this.

Bug: 64595
Change-Id: Iee47bdc23630e02ccfcbd28496ec5268892eb629

Merge "Enable mediawiki.skinning.content.parsoid on Mobile target too"

Merge "Remove "Help/Project" profile from Special:Search"

Merge "Remove silly 'm' prefix from StubObject/DeprecatedGlobal members"

Merge "Move Modern and CologneBlue out of core"

Enable mediawiki.skinning.content.parsoid on Mobile target too

Bug: 65823
Change-Id: Iee2e3d00576556418c666d393e8935f75801326c

Merge "Introduce ContentHandler::exportTransform()"

Move Modern and CologneBlue out of core

Import in the new repositories:

- Modern: I5348fedddf14affcee2688833f4e759e5af6499c
- CologneBlue: I9cc051f48074a033c684e5efa56a7b4dde51d366

Change-Id: I65768b4c29faf46895a7005a3090f43cf6e86258

Merge "Introduce ContentHandler::importTransform."

Merge "Cleaned up database reconnection logic"

Merge "Installer: output css correctly when session errors occur"

Merge "Use language from context in Skin::subPageSubtitle"

Merge "Add ar_text to the list from Revision::selectArchiveFields(). It is checked later."

Merge "Fix docs on File::getImageSize() they were very misleading"

Merge "Revert more visibility changes in HistoryBlob.php"

Merge "Don't use isset() to check for null"

Add ar_text to the list from Revision::selectArchiveFields(). It is checked later.

See Revision::newFromArchiveRow().  This is called from
populateRevisionLength.php's upgradeRow() which ultimately populates
$row from the fields returned by Revision::selectArchiveFields().

Revision::newFromArchiveRow() has the following condition:

if ( isset( $row->ar_text ) && !$row->ar_text_id ) {

And from the bug report:

> for old archived revisions, the rows in the archive table do _not_
> contain a pointer to the text in the text table, but that the text
> is stored in the archive table directly. Obviously the above code
> cannot handle this situation properly.

Credit goes to Joerg who found this and figured out the problem.

Bug: 65765
Change-Id: Iadcaae5e531d439780f5b6095286df78926ccaa7

Merge "Change one Linker::formatSize call to Message::sizeParams"

Merge "DBConnectionError: Expand {{SITENAME}} in pagetitle with Message::text()"

Merge "Restructure Media related tests to avoid duplicated code"

Restructure Media related tests to avoid duplicated code

Change-Id: I4428d38e0199b12255f3746e79f8093259a03da1

Merge "Fix doc comments on get[General](Short|Long)Desc"

Merge "SpecialRevisiondelete: Simplify if/elseif logic"

SpecialRevisiondelete: Simplify if/elseif logic

This avoids checking $comment twice
by Krinkle on I3b21f66f84d68777a18e66e4e5f59886e55d7936

Change-Id: I39df27d3edb097024d136455145505429ccdbe39

Merge "DerivativeContext::setConfig should take a Config object"

Merge "Allow more flexibility in RC feeds"

DerivativeContext::setConfig should take a Config object

Change-Id: Icc16a9c206c57fac2d26547cb874e02ef09c0485
Follows-Up: I13baec0b6d4ea7badf20b9c5f9b40846348838e4

Allow more flexibility in RC feeds

The user can now specify which feeds to send to in
RecentChange::notifyFeeds if they don't want to use
$wgRCFeeds.

Additionally, the 'formatter' in $wgRCFeeds can now be an object
rather than just a class name.

Change-Id: Ibfdffc17a934e35223887c123331795563102752

Merge "Make abstract Config class truly implementation-agnostic"

Localisation updates from https://translatewiki.net.

Change-Id: Idab23e6c2f532b1aae67dc15e9138f6602c15a0f

Use language from context in Skin::subPageSubtitle

Change-Id: Ia6cff8aa9e0b12215839f8b9c5a64a09150d4002

Change one Linker::formatSize call to Message::sizeParams

Change-Id: I2a91f0ef1cf82360da2d17c7dc42633ae3e79100

DBConnectionError: Expand {{SITENAME}} in pagetitle with Message::text()

Follows-up 485ff9ead94b by making the same change in DBConnectionError,
which overrides MWException::msg() to not use database messages.

Bug: 58447
Change-Id: Iafa9dd741f1f099a4a6cfb8cf655b7d5f504011a

Make abstract Config class truly implementation-agnostic

Follow up to I13baec0b6 ("Config: Add Config and GlobalConfig classes"):

Config:
* Rather than returning Status objects, Config::set will now throw an exception
  if an error is encountered
* Config::factory was moved into it's own ConfigFactory class.
* Since there are no more functions in it, Config was turned into an interface.

GlobalConfig:
* Remove $prefix args from Config::set and ::get. The idea of having an
  abstract Config class is to abstract some notion of configuration data from
  the particular way in which it is currently implemented (global variables).
  So the abstract base class has no business dealing with variable name
  prefixes.
** Instead GlobalVarConfig's implementations of get and set call getWithPrefix
   and setWithPrefix internally, which are now protected
* Rename GlobalConfig to GlobalVarConfig, which makes it clearer that it isn't
  referring to the scope of the configuration value, but to the scope of the
  variable name which provides it.

ConfigFactory:
* ConfigFactory is where Config objects are registered, and later constructed.
* Config objects are registered with a given name, and a callback factory function.
  This allows for implementations to construct the object with the parameters they want,
  and avoids the overhead of needing an entire class.
** The name 'main' is the default object returned by RequestContext::getConfig(),
   and is intended to be used by core.
* This is a singleton class, the main instance can be obtained with:
  ConfigFactory::getDefaultInstance()

In addition to the above:
* $wgConfigClass was removed, and $wgConfigRegistry was introduced, which
  stores a name => callback. The name is to be what the Config instance is
  registered with, and the callback should return an implementation of Config.
* Tests were written for the new ConfigFactory, and GlobalVarConfig's tests
  were improved.

Co-Authored-By: Ori Livneh <ori@wikimedia.org>
Co-Authored-By: Chad Horohoe <chadh@wikimedia.org>
Co-Authored-By: Mattflaschen <mflaschen@wikimedia.org>
Co-Authored-By: Parent5446 <tylerromeo@gmail.com>
Co-Authored-By: Reedy <reedy@wikimedia.org>
Co-Authored-By: Daniel Kinzler <daniel.kinzler@wikimedia.de>
Change-Id: I5a5857fcfa07598ba4ce9ae5bbb4ce54a567d31e

Merge "Return value directly when using getter"

Merge "Revert "Pass phpcs-strict on includes/revisiondelete/""

Revert more visibility changes in HistoryBlob.php

This follows-up bebcf2c58381, which only fixed DiffHistoryBlob.

MediaWiki will fail to load some revisions if the following properties
do not unserialize correctly:

* HistoryBlobStub::$mOldId
* HistoryBlobStub::$mHash
* HistoryBlobCurStub::$mCurId

Visibility of HistoryBlobStub::$mRef is also restored; though the
property's value does not appear to be used after unserialization,
the property is nevertheless a serialized property.

Bug: 65665
Change-Id: I2724d15f9206d66d615d020117e60bb772048c19

Localisation updates from https://translatewiki.net.

Change-Id: I5fcddad181c862b8f825611aa2aa3a17864e1b28

HTMLForm: fix double escaping of checkbox labels in vform

mLabel has already been escaped if required. It isn't run through htmlspecialchars()
in other display formats, so doesn't need to be done in vform.

Change-Id: I0b0feec408687cdadf01da5f2078980015538bb7

Merge "Fix broken @return documentation"

Merge "Removed double hook explanation AfterFinalPageOutput"

Removed double hook explanation AfterFinalPageOutput

Bug: 65690
Change-Id: I0990568875db854579974f611e7378bca0b0eaed

Revert "Pass phpcs-strict on includes/revisiondelete/"

Causes fatal when using revdeleting an old image

This reverts commit 2e8ec52154ad33680f92c3bf49d3b35ed5a33153.

Bug: 65733
Change-Id: I948cbf0f9a60929020d8cd92f4264bee7a11104e

Merge "jquery.makeCollapsible: Use .prop() for value attribute of list item"

jquery.makeCollapsible: Use .prop() for value attribute of list item

The issue with "value" attributes is popularised through form
fields (e.g. change <input> value with prop, because it doesn't work
with attribute, attributes only change the DOM, not the live rendering).
However the attribute/property issue isn't limited to form fields and the
typical value/selected/disabled properties . It applies to all properties
and for any element that affects active state (if it changes the display,
one should use a property, not an attribute).

As far as I know, unlike for form fields, browsers do actually synchronise
attributes in the many of those cases (including list items), so there is
no immediate danger. But since it's caught by jQuery Migrate and a good
practice regardless, change it to use prop() as well.

To test:
 https://test.wikipedia.org/?oldid=199994#head-collapsible-ol

Change-Id: I246c26bd2958209feed73fc2ef31cf678b12fa3b

Fix broken @return documentation

I found one of these in Wikibase and run a global search.

Change-Id: I76c1519a47d7222f136272c59ea454b650808a86

Fix doc comments on get[General](Short|Long)Desc

Follow up f3cc3980bc78afdb where somehow I managed to do this.

Change-Id: I25e5a01ddbecd1a991f6d121a7ac3f3598004b25

Fix docs on File::getImageSize() they were very misleading

Change-Id: I2a3e1e8c10da7ade4ea713a561e05fd4ab9203c8

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

Merge "Revert "Restore search box tabindex""

Localisation updates from https://translatewiki.net.

Change-Id: I978097374e44627aeb91f6fe740899be0ae29e97

Revert "Restore search box tabindex"

This does not restore the same behavior as collapsibleNav.js had.  That
used $( document ).lastTabIndex() + 1 in JavaScript.  This means it
would dynamically put the search box's tabindex after other elements.

This commit specified an explicit value of 1.  This is normally used
for forms.  For example, 1 is the username field on Special:UserLogin.
Thus, this made tabbing from the username field take you to the search
input.  It also caused issues with at least MassMessage.

Another solution is necessary, whether it's dynamically adding it with
JavaScript as before, using a different PHP-generated tabindex, both
(PHP-generated tabindex overridden by JavaScript), or something else.

In the meantime, this reverts to avoid the mentioned regressions.

This reverts commit be76751e8c7a8c33b8a9f41faba78592a22ded09.

Bug: 29199
Bug: 65704
Change-Id: Iaea60834aa8f1564aca50c21ee62ae01f958cc65

Don't use isset() to check for null

Fixes in includes/specials

Change-Id: I6c382aeaa92372bc11220694c479006e3c658362

Remove not-working scripts in maintenance/language

Change-Id: Idd6b1d5dc93a5bd71f0ad7445ce2f6ddc4beeb33

Correct a comment about updateRevisionOn

to reflect its compare-and-swap semantics.

Change-Id: I6a68e673195e0b6688f6fbc9199a8355b20c0da0

Check category-move-redirect-override in content language

When testing whether category-redirect-move-override is disabled, do it in
the content language rather than the user language.

Change-Id: Id1f03240c203f32a12953f49a075cfd5c25f0f31

Merge "mediawiki.test.js: Suppress false postives for deprecated mw.* properties"

Followup dd1c7f7fd7: fix LESSification errors in Parsoid styles

.mw-halign-right should probably have float: right;, not float: left;

Also, use & because .mw-halign-* are applied to <figure>, not its descendants.

Bug: 65596
Change-Id: I7cd00f0049523f064703d7355b80457a4ab5d0d0

mediawiki.test.js: Suppress false postives for deprecated mw.* properties

[qunit]
  mediawiki.jscompat...OK
  mediawiki - Initial check...
   Use of "anonymous" is deprecated. Use mw.user.isAnon instead.
   Use of "name" is deprecated. Use mw.user.getName instead.
   Use of "jsMessage" is deprecated. Use mw.notify instead.
   Use of "wikiGetlink" is deprecated. Use mw.util.getUrl instead.
  OK
  mediawiki - mw.Map...OK
  mediawiki - mw.config...OK
  mediawiki - mw.msg...OK
  mediawiki - mw.loader...OK

Change-Id: I4a79bc64cc26c1237fefe1503504cde9d40a5087

jquery.autoEllipsis.test: Swap $.browser for $.client for compatibility

Change-Id: I50d57371edebe7f93f01c83284c0a94fc2e7dbac

Merge "Store page_id in logging table for deletions and make queryable"

Merge "Make HTTPS port configurable"

Merge "MagicWord: Fix an indent"

MagicWord: Fix an indent

Change-Id: I3640fb7a783a68c43a1249b87ef04e587dd0cfd5

Merge "Make sure DjVu files do not attempt metadata extraction repeatedly"

Merge "Do not include user column on Special:MyUploads"

Merge "Document ArticleUndelete change"

Merge "Prevent deleteBatch.php deleting files from redirects"

Merge "Revert visibility change on DiffHistoryBlob::$mCompressed"

Merge "Made getName() in ArchivedFile work when loaded via sha1"

Prevent deleteBatch.php deleting files from redirects

Previous behavior had the maintenance script deleteBatch.php
delete both the redirect in the file namespace and the file
from the target page the redirect is pointing to. This is
no longer the case, but it will still delete the redirect
page if it is still on the listfile.

Bug: 52587
Change-Id: I83374ddb98131f9f8f766bb2b444ab3e7cda7735

Revert visibility change on DiffHistoryBlob::$mCompressed

Changing the visibility from public to protected causes unserialization
to fail.

Bug: 65665
Change-Id: Id7ee2068749ff421fab4d1c9067763adf4b89563

Return value directly when using getter

It is not necessary to save the value in a variable

Change-Id: Ia58c94bb19e22476821451bddc2f66e556b773f6

Improved FileBackend failure logging a bit

* This partly follows up on 2cfce9f

Change-Id: I9d15f44d8000f52fd7b307d358c895452ee96a8d

Make sure DjVu files do not attempt metadata extraction repeatedly

If a file is broken, we don't want to spend time trying to find
its metadata over and over again.

Bug: 41090
Change-Id: Iad63b8942af99e1ec44530599a43ec1d6b2b8a62

MediaWiki::performAction, return 404 on nosuchaction error

Nosuchaction error page was sending 200 as the HTTP status code,
which was misleading. Instead return 404.

Change-Id: I9b432c9e53caf0a13e9a0c798f78a7bd8a78443b

Made getName() in ArchivedFile work when loaded via sha1

Change-Id: I44e9c129f6c5841f8cbc9792aa9ff0f353bce6c0

Merge "API: Enforce section=new constraint when using 'redirect' mode"

Merge "format the tests/phpunit/TODO document for readability"

API: Enforce section=new constraint when using 'redirect' mode

Finish implementation of the 'redirect' edit API parameter.  This flag
should only be used as a convenience when adding a new section, appending,
or prepending.  Any other usage must be done using the long-cut (fetch page,
determine if it is a redirect, follow redirects and edit the target page's
content).

This patch takes apart the "EditConflict_redirect" test, because it no
longer makes sense--unless you dear reader can figure out how to force an
edit conflict when only adding a section?

Bug: 24330
Change-Id: Ie3f7273c18e156da1e50e1a36aac2e5341710982

Localisation updates from https://translatewiki.net.

Change-Id: I589dbdf292abaf998d18fe553b7afb4880cb40de

Do not include user column on Special:MyUploads

If Special:ListFiles is being filtered to show only a specific
user, then there's not much point showing which user uploaded
the image, as they are all uploaded by the same user.

Bug: 30627
Change-Id: Ibe1be82cb8f3354889051ef00f3c0b9f9ea5180e

Merge "wfDebugMem: Fix binary prefix and improve usability"

Merge "Cached misses in getMulti should be omitted"