Merge "mw.Upload.BookletLayout: Pass image info when emitting the fileSaved event"

Merge "docs: mw.Upload.Dialog: Add another usage example in introduction"

Merge "jquery.tablesorter: Minor coding style cleanup and DRY"

mw.Upload.BookletLayout: Pass image info when emitting the fileSaved event

Don't want to reach into the layout's upload object to get this. Anything
listening to this event would want the image details.

Change-Id: Idd391382292f31d0570a6548c3b89d5e3795e832

docs: mw.Upload.Dialog: Add another usage example in introduction

Change-Id: I8ed1470b712643b23e86a167f06d2fd1116e8627

Merge "Database::selectRowCount(): support $join_conds"

Database::selectRowCount(): support $join_conds

Add a $join_conds parameter for Database::selectRowCount(), to allow the caller
to specify join conditions for the subquery that it generates.

Change-Id: I8c0a93713c121bc5b691ae65d6b6d8f8c08c9e4c

Merge "resourceloader: Disable flaky test ResourceLoaderModule::getVersionHash"

Merge "build: Use my gmail address for attribution"

resourceloader: Disable flaky test ResourceLoaderModule::getVersionHash

Bug: T109394
Change-Id: I8e62d9e8326046a895588d9cc63f47e6c0d12eda

build: Use my gmail address for attribution

Per the .mailmap file, my gmail address is the canonical address
for this purpose.

Change-Id: I85de941ef5f8cf9a80cc35647500c5baaa594ac3

jquery.tablesorter: Minor coding style cleanup and DRY

* Consistently name 'length' cached variables 'len' (e.g. not 'l').

* Fix invalid documentation syntax for convertSortList().
  (Build is passing because this file is not yet indexed.)

* Remove redundant 'len' variable from pure JavaScript arrays where
  iterations can't change the length. Caching the length is a dated
  practice meant to micro-optimise execution time. For the moment,
  kept any loops with complex iteration bodies as-is, as well as loops
  over structures that are not pure arrays (e.g. NodeList objects).

* Use getParserById() in addParser() instead of duplicating its logic.

Change-Id: I566eff6efd97b5d9672277bacb4cb2b501f4625f

Merge "Update ResourceLoader for Ib7fc2f939b"

Merge "Rename getMainClusterInstance() -> getLocalClusterInstance()"

Update ResourceLoader for Ib7fc2f939b

Use BagOStuff::makeKey() and BagOStuff::makeGlobalKey() instead of
wfMemcKey() and wfGlobalCacheKey().

Change-Id: Id4e58c0e616b74de220faf13ba7c9ea606fef8c1

Rename getMainClusterInstance() -> getLocalClusterInstance()

Also corrected some ObjectCache docs

Change-Id: I322f4cbd72fbd5d4c6887d90ee75d0baddb6ac25

Remove ObjectCache dependency from MultiWriteBagOStuff

This brings it closer to being able to move to /libs.

Change-Id: Ia733f9023e56d4a25ffcb99ca0cc8b29cbb2ad45

Merge "Add make(Global)Key() method to WANObjectCache"

Merge "objectcache: Remove getWithSetCallback() signature backwards-compatability"

objectcache: Remove getWithSetCallback() signature backwards-compatability

All callers have been migrated

Change-Id: I7b6b87594dd724434ba24d8206fe07d66c1d5d25

Fix bad call to Database::getCacheSetOptions()

Change-Id: I8ac756851b4cc7d36129fbd4e01282a151dc2694

Add make(Global)Key() method to WANObjectCache

Change-Id: I8e739fbe8614c9bd1c3595bbf16c8cd423aff7cc

Localisation updates from https://translatewiki.net.

Change-Id: Ia1685d4f8347c81ef6eb1fe95ebbe66bbe95398e

Merge "Convert some users to WANObjectCache for consistency"

Merge "Deprecate redundant SquidUpdate::newSimplePurge()"

More SquidUpdate cleanups

* Remove $wgMaxSquidPurgeTitles; silently discarding URLs is
  the wrong way to limit things. The purge methods already batch,
  which HTCP pushing one at a time. Jobs already batch and have
  $wgJobBackoffThrottling. If code does purge 500 URLs at once
  with no limiting (flooding UDP), then it is already broken.
* Make the HTCP method protected.
* Rename $urlArr field.

Change-Id: I17cced187fe7e93f5a5188022f12202a7746bdb7

Convert some users to WANObjectCache for consistency

These callers don't need to do purges, but can still perfectly
take advantage of this instance over a plain BagOStuff. Namely:
* Replication and snapshot lag awareness
* Preemptive regeneration
* Easy process cache support

The idea is for there to only be one caching class/factory
to use, instead of having rules for picking which one to use.

Change-Id: I8e362df451c0c28731fc853c044c4c4b8e097f01

Merge "Remove "activeusers-updated" cache key usage"

Merge "SquidUpdate cleanups"

Deprecate redundant SquidUpdate::newSimplePurge()

Change-Id: Id6d92fca2a2b87e23930946f054cecd1f6d433be

Merge "Use responsive CSS columns on Special:PrefixIndex and Special:AllPages"

Make attributes explicit

Two attributes were referenced, but not declared.

Change-Id: Icb3ce046c15c5709e2f3738ba03e44beaaafd46b

ApiDelete: Don't pass $user to ChangeTags::addTags()

It doesn't take that as a parameter...

Change-Id: I9f78ba4151d131ecf4ca476ecead0ed60408becb

SquidUpdate cleanups

* Actually implement DeferrableUpdate
* Remove unused parameter to __construct()
* Remove redundant check in newFromTitles()
* Short-circuit purge() if  is empty

Change-Id: I23b1f712001d72305dfa1c49063c3384ea67fb2f

Fix Maintenance typo

Change-Id: I3e0ae4446f74fac25e1cc79a86ff6e0738ada52d

Localisation updates from https://translatewiki.net.

Change-Id: Ib609589d2354e36ef89fb77e97932c823087110c

Use responsive CSS columns on Special:PrefixIndex and Special:AllPages

Stop using table on Special:PrefixIndex and use CSS columns instead and
removed 'column' parameter from this page.
This will also change the weird ordering (side to side) to the standard
top to bottom instead.

The pagination links hide at the corner on small screens. This can be fixed
later.

Bug: T32965
Change-Id: I720d34e21950c18de3c6e0b1d6d8a4461db495cb

Merge "mediawiki.special.preferences: Use standard IIFE"

Merge "Mention WANObjectCache::TTL_INDEFINITE in set() method"

Merge "zhtable makefile: Remove unused variables, fix almost all pep8 errors"

mediawiki.js: Show line number of caller of mw.log.warn and .error

Currently the JavaScript console shows the line number from mediawiki.js
where the function mw.log.warn is defined.
With this change the JavaScript console shows the file and the line number
where the function mw.log.warn is called.

Inspirited by https://matthewspencer.github.io/console-log/

Change-Id: I2345333fc0158a66ebcb3abf0e94e6e622b3bdc0

Avoid creating lots and lots of cat_id gaps

Currently, INSERT...ON DUPLICATE KEY UPDATE is used to update the page
counts in the category table. However, MySQL 5.1.22 and newer, by default,
increment the counter for cat_id before checking for duplicate key errors.
This creates many gaps in the cat_id sequence.

To avoid this, check for existing category rows, and instead UPDATE any
that were found. It is hoped that the extra queries will not significantly
harm performance.

Change-Id: Ic2ab9ff14f04a0c7ea90a5b6756cade0c78e2885

Category::refreshCounts(): Save new counts using upsert()

This doesn't completely address the TODO comment. However, I intend
to finish doing so in a separate change.

Change-Id: I71c6dafd4223d79a0153fa8ce7cb0ee1354c0ec6

Merge "Add `makeKey` and `makeGlobalKey` to BagOStuff"

Avoid exposure of local path in PNG thumbnails

Bug: T108616
Change-Id: I952068d2d175d71f86dec0dbb92af5a122c05a49

Merge "SECURITY: RevDel: Check all revisions for suppression, not just the first"

Merge "SECURITY: API: Improve validation in chunked uploading"

SECURITY: RevDel: Check all revisions for suppression, not just the first

If any of the revisions in the list are deleted, do not allow changing
the permissions without suppressrevision.

Bug: T95589
Change-Id: Ia8a3ef13b73925d14a79775a50d46b939c01664f

SECURITY: API: Improve validation in chunked uploading

This fixes a few shortcomings in the chunked uploader:
* Raises an error if offset + chunksize > filesize.
* Enforces a minimum chunk size for non-final chunks.
* Refuses additional chunks after seeing a final chunk.
* Status of a chunked upload in progress is now available with
  'checkstatus'.

Bug: T91203
Bug: T91205
Change-Id: I2262db1bc8460616b069c564475d2e4148001768

Merge "SECURITY: Throttle uploads"

Localisation updates from https://translatewiki.net.

Change-Id: Iec6e48de8a855f6f21b1cded93cb03bd9e5f7d56

Merge "Clarify that ParserOutput::getDisplayTitle() returns HTML."

SECURITY: Throttle uploads

Add throttle check in ApiUpload and SpecialUpload.

Bug: T91850
Change-Id: If33cc99f304aab2486507c7500b4abb06b6b5d70

Clarify that ParserOutput::getDisplayTitle() returns HTML.

This change only improves documentation, it does not touch code.

For reference:
* CoreParserFunctions::displaytitle generates HTML, with &, < and so on
property escaped. OutputPage::setPageTitle() treats the input as wikitext,
which works for both plain text and HTML.

* Wikibase calls ParserOutput::setDisplayTitle() with HTML including <span>
structures.

In the API action=query&prop=pageprops returns HTML. There doesn't seem to
be a good place to document that fact, though.

Change-Id: I043b26b82f066abe2830a81d3bc073543b3748d8

Merge "mw.ForeignStructuredUpload.BookletLayout: Switch order of widgets"

mw.ForeignStructuredUpload.BookletLayout: Switch order of widgets

As per Pau's observation and mw.Upload.BookletLayout, the description
input should come after the name input.

Change-Id: I6492f305734268de15d721476bf751e5bc9c805b

Typo in comment of Html::radio()

Incorrectly copied documentation from Html::check().

Change-Id: I508e4beedb01b62989a7f37466c47ddd37091b49

Merge "More specific @return doc in WikiPage::getDeletionUpdates"

zhtable makefile: Remove unused variables, fix almost all pep8 errors

Change-Id: I7ab1ffeeb2a5592fbe143f27801c11248c167093

Merge "LoadBalancer::getLaggedSlaveMode needs to know about the wiki"

Merge "Fix mw.notify not being visible if first loaded when scrolled down"

Merge "ApiMessage: Improve documentation"

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

ApiMessage: Improve documentation

Based on an excerpt from Anomie's IRC conversation with me.

Change-Id: I02661548a30fd40eaa72d77c98a6801dbbaff456

Localisation updates from https://translatewiki.net.

Change-Id: Ia14ad3a350ff7e552c0001c5ccf4cda36e192294

Merge "Move some styles of convenience links to mediawiki.special"

Merge "Move styles for Special:FileDuplicateSearch to mediawiki.special"

Merge "Improve the API help for format=xmlfm"

Remove unused ApiQueryORM

Bug: T114538
Change-Id: I4673977cef336ab3b4e375a3deab4d4fc67830eb

Make Special:Redirect work for blocked users and in read-only mode

Bug: T115570
Change-Id: Id26af6efe9dc7eddb2066a2f61c86116305fff7c

Mention WANObjectCache::TTL_INDEFINITE in set() method

Change-Id: I6e5cb2e7ba34ceaf4ca128e4b32da99d08917156

Merge "objectcache: Rename WANObjectCache TTL_NONE to TTL_INDEFINITE"

Use specified pageId for LinksDeletionUpdate→DeleteLinksJob

Not page->getId() which may now be 0 due to the page having been
deleted.

Bug: T115586
Change-Id: I5a3482bbe1df23dda0f33e7d675b06d616395bb6

More specific @return doc in WikiPage::getDeletionUpdates

Change-Id: I8d126944411c91f87cb3bec3269e60d23f5dceac

Added a parenthetical to fileextensions description

Changed "fileextensions for more clarity of the description.

Bug: T62121
Change-Id: Ida361a370069e6d5565a5ef5b03f0976dfd3515b

Improve the API help for format=xmlfm

I thought it dumped raw text with attached JSON "debugging elements",
but it's just jsonfm with a little extra.

Change-Id: I55f2ae2c9aa295bcb972b4e02e7352532d64341c

Remove "activeusers-updated" cache key usage

RecentChangesUpdateJob already takes care of this.

Change-Id: I166b37c60fbb05046150a63592fa26fc47027181

objectcache: Rename WANObjectCache TTL_NONE to TTL_INDEFINITE

"None" has a somewhat unobvious meaning, also ambiguous with "uncachable".

Change-Id: I799de385427aeb8f581d51800606552bdc998252

Localisation updates from https://translatewiki.net.

Change-Id: I821a5724c2f14c6d1697d60454cd17618998e2bd

Re-enable last phpcs rule

Bug: T102609
Change-Id: I49eb778ccb5f0a3662fe4573872d1235c52fdd81

Merge "Move comment in backup_PageTest.php"

Merge "Fix array index spacing in ResourceLoaderEditToolbarModule"

Merge "Change some comment style in userOptions.inc"

Merge "Add curly braces to while"

Merge "Remove empty comment line from GlobalTest.php"

Add $suppress to ArticleDelete hook

We should be telling extensions whether a deletion is a suppression or not, so
they can behave appropriately.

Change-Id: I2cb6ffd61dd12766fe0266514c9360ff0c90b788

Use the frame 'type' in wfFormatStackFrame, like MWExceptionHandler::prettyPrintTrace

Instead of assuming '::'

Change-Id: I2452bc5ebd3716eb61f53da41a0050f4833fb5a4

mediawiki.special.preferences: Use standard IIFE

Directly using 'mediaWiki' variable rather than 'mw' makes it harder
to grep for things.

Change-Id: I2652ecd9a52b5ba05989dcfe8b106e3b4dd504d7

Mark Title::setFragment() as @private

It is deprecated for public use, so indicate that it should only be used
privately.

Change-Id: I1b32f52f427a36ddb81bd7befadc9099ebc4f191

adding 7zip archives to the MIMEs

Change-Id: Ib3392b7fcec475c153099a1b92fe195c5eb4b195

Handle $frame['function'] not being set in stack frames

Bug: T115107
Change-Id: I9fbfcd53fbcba1800a924acb1a1702c3c9037cc5

Merge "Add recentChangesLine to ChangesList"

Use correct @codingStandardsIgnore in DairikiDiff.php

@codingStandardsIgnoreFile is for whole files, use
@codingStandardsIgnoreStart instead

Also correct a MediaWiki.WhiteSpace.SpaceBeforeSingleLineComment.
SingleSpaceBeforeSingleLineComment sniff in the same file

Follows I06cdab4616b5bff47c85152df28f18c861730a23

Change-Id: I75e6e936e2fb4453bd56848ff39cabd92ae171fc

Remove empty comment line from GlobalTest.php

Remove empty line comments as found by the
MediaWiki.WhiteSpace.SpaceBeforeSingleLineComment.EmptyComment sniff

Change-Id: Ibd60165ae8ddef2a2fd443856e7476f32a0c6392

Fix array index spacing in ResourceLoaderEditToolbarModule

Code style is not using spaces in array index

Follows Ic3eab71691e502bfe19bdf4eb6f82cc679a7782f

Change-Id: I0e20787c41d2edbc2899308647092ed6831d2489

Add curly braces to while

Follows Iaa1a479b5eefcae0fc8f82b1d20e9575d609c0e9

Change-Id: I7f2b9b12e3282f5b8ba54a315fa12f1a97985572

Revert "Add // @codingStandardsIgnoreFile to FormatMetadata"

For testing.

This reverts commit df706d2457e675ce3896b6b2aace27dac3fb9ff1.

Change-Id: If63199293ac3539030f3503fbcbd5e0f4aa2eece

Merge "Show a confirmation message on Special:UserRights"

Change some comment style in userOptions.inc

Change-Id: Ida429683ee5b6194c96d782cf5fb4e0527c89103

Move comment in backup_PageTest.php

Move the comment at the begin of the function out of it to a doc
comment.

Change-Id: I6febf51a74de88204d54b1e3251df187197c1819