Merge "Migrate Special:Unblock to OOUI"

Merge "Update cleanupSpam.php to hide its activity from RecentChanges"

Merge "Improve namespace handling in tests"

Merge "Fix patchfile used to add newtalk table on mysql"

Update cleanupSpam.php to hide its activity from RecentChanges

Spam cleanup activities, mostly when it comes from automated processes
(the so called 'spambots') can involve a fair ammount of edits and log
actions to accomplish.

As such, to avoid flooding Special:RecentChanges, this script will now
set a bot flag on the script run so its edits and actions won't show
up in that special page.

To prepare this patch I had the help of Bartosz Dziewoński which I'd
also like to thank and I'll credit as co-author as well.

Bug: T176206
Co-Authored-By: Bartosz Dziewoński <matma.rex@gmail.com>
Change-Id: Ifde08056a7481b877b4b82699cc0a5ea3f962dd9

Merge "Do not include RC_EXTERNAL in number active users on Special:Statistics"

Improve namespace handling in tests

MWNamespace has three internal caches, only one of which can be cleared
(and that somewhat oddly by passing a boolean to
MWNamespace::getCanonicalNamespaces()).

This change introduces a MWNamespace::clearCaches() method to clear all
three caches. It also adds some resetting in tests that seemed to be
missing it.

Change-Id: I1dcfcd8713888b3ff8fc75e95329ba72bd95d0c9

Merge "Revert "DumpFilter is autoloaded. No need to require in maintenance script""

Merge "Consistently preserve anchors in the title specified in 'mainpage' message"

Merge "Fix CreditsAction when user has no real name set but real names enabled"

Merge "objectcache: Improve WANObjectCache test coverage"

Do not include RC_EXTERNAL in number active users on Special:Statistics

We already exclude external (mostly Wikidata) actions from counting on
Special:ActiveUsers as of 56524c05. Should be consistent.

Change-Id: Ib69a614a65246a0f4f47365751f9bef130f47475

Follow-up 67f04373d fix stupid typo

Fot should be For

Change-Id: I3cb87a94feea32f1c98345ceaa19f9a75fde266d

Merge "Make the RC/Watchlist legend not jump when initially collapsed"

Localisation updates from https://translatewiki.net.

Change-Id: I02b4624b999123b0841af96adf9f6c3467fa9b3b

Make the RC/Watchlist legend not jump when initially collapsed

Look at the collapsed/expanded cookie and add class="mw-collapsed"
if it's going to be collapsed.

This is kind of a prelude to T42812.

Change-Id: Icaca762a9d7b000c973d9275f395623129f67979

Merge "Distinguish GET from POST for PostSend expectations"

Merge "Add more ProcessCacheLRU comments"

objectcache: Improve WANObjectCache test coverage

class WANObjectCache (methods: 40% -> 67%, lines: 65% -> 88%)

* Allow indirect coverage of protected/private utility methods from
  existing tests.
* Add basic test for setLogger().
* Add basic test for newEmpty().
* Add basic test for getQoS().

Change-Id: Ifb79ed2ff1febbd2f5477b8ed6319992ce88eb29

Merge "HTMLFormField: Treat weird '&#160;' labels as empty in OOUI mode"

Merge "LogFormatter: Don't return a non-array from formatParameterValueForApi"

Unregister hooks that interfere with unit testing

Id7403f57cc9d751ada85b611193c1d8f3503e713 did not properly
reference the hook

Bug: T176102
Change-Id: Ibff72c0efe82ba6fd4458e3c8a96179f50ca21b0

LogFormatter: Don't return a non-array from formatParameterValueForApi

If the title for a 'title' or 'title-link' type is invalid, the code
still must not return a non-array from the method.

I'm not sure this is what was behind T176938, but it's the only thing I
can find that might cause the errors I saw logged at about the same
time.

Change-Id: Iae77eb6ad9a64d8b67074164ff0c0fea36826f3c

HTMLFormField: Treat weird '&#160;' labels as empty in OOUI mode

I have no idea where this convention came from, but we had them in
core until 265ff105aaa814c828a35e599ed35d0b0bb00e53 and they still
appear widely in extensions. The non-OOUI code also has special
handling for it (a label equalling '&#160;' is treated as raw HTML
even when not marked as such). In OOUI-style "vertical" forms these
fake labels cause a lot of unnecessary white space to appear, so let's
just not display them.

Change-Id: I45559fa69dc1ae4b9d048445e27a24815fe93b6d

Merge "Migrate and convert WL settings to the new UX"

Merge "Fix insertSelect() with IGNORE in PostgreSQL"

Merge "Reopen db connection after doing SpecialPageCacheUpdates"

Distinguish GET from POST for PostSend expectations

This avoids needless log entries for POST requests.

Bug: T166199
Change-Id: I515fde1f619f7f016d08b7c270a1c155ee171bf9

Fix CreditsAction when user has no real name set but real names enabled

Change-Id: Ief347739a1231fa8bd814694bdb5b4f44ba41a77

Merge "Revert Message dependency from DB exception classes"

Merge "user: Unregister hooks that interfere with unit testing"

user: Unregister hooks that interfere with unit testing

testExecute_email() should unregister the hooks
mailPasswordInternal and SpecialPasswordResetOnSubmit
for proper unit testing.

Bug: T176102
Change-Id: Id7403f57cc9d751ada85b611193c1d8f3503e713

Reopen db connection after doing SpecialPageCacheUpdates

Special page updates on commons have been broken recently due
to Mysql "going away" after doing the ActiveUsers special page
cache update. Presumably this is because we were only repinging
the server after QueryPage updates, and not the callback updates
(Combined with a spike in wikidata recentchanges entries making
the ActiveUsers callback now take about an hour to complete)

Bug: T176055
Change-Id: Ic344940460c353544d7c08bbe2d8c900f6cc73da

Consistently preserve anchors in the title specified in 'mainpage' message

When 'mainpage' was used in the 'sidebar' message, the anchor would be
preserved (code in Skin::addToSidebarPlain() uses Title::getLinkURL()).

When 'mainpage' was used on its own in Skin::makeMainPageUrl() (which
is used in SkinTemplate::buildNavUrls(), and many skins use that data
to generate the logo link), it would not be preserved (the code used
Title::getLocalURL()).

Bug: T176583
Change-Id: Icc4895c08089caaa06e37ce765819829b90de0ff

Localisation updates from https://translatewiki.net.

Change-Id: I7ea7ab422e07d18a233e2c9e8df9e8d0c3d158c0

Migrate and convert WL settings to the new UX

Convert Watchlist settings to new UX, with following rules:
- If "Hide edits by anonymous users from the watchlist" is checked,
add "Registered" filter to defaults.
- If "Hide edits by logged in users from the watchlist" is checked,
add "Unregistered" filter to defaults.
- Change test for Special:Watchlist page, so new structured filters
are supported.

Bug: T172757
Change-Id: I27bdaa1401323fa3143f79a57dc5b9773e48fd1d

Revert "DumpFilter is autoloaded. No need to require in maintenance script"

Broke dumps, turns out autoloader isn't infallible

This reverts commit f127125ec6b76f843fc73a1faba31d973e33bfa5.

Change-Id: I7711094351cf3f186730857ae59975e183daed1d

Fix watchlist "in the last X hours" display

If6280ad6 changed the interpretation of the URL parameter "days=0" (or
any negative value) from meaning "maximum number of days" to actually
meaning "0 days". That's a somewhat annoying breaking change, but easily
worked around by updating bookmarks.

But it didn't update the display on the page, so despite querying 0 days
it would still incorrectly say something like "in the last 720 hours"
rather than "in the last 0 hours".

Bug: T176857
Change-Id: I90c79ef9c4819a34060515b863277fd185828ed9

Revert Message dependency from DB exception classes

This was added in a90bbf1a.

These classes are in /libs and should not depend on all of MediaWiki.

Change-Id: I67bc818b0c674f1aa3e0dde27a38a9f385fcab75

Merge "objectcache: Improve WANObjectCache test coverage"

Merge "Update jQuery Migrate to v3.0.1"

Merge "Update OOjs UI to v0.23.2"

Fix patchfile used to add newtalk table on mysql

Remove orghaned patch-drop-user_newtalk.sql

Change-Id: I27ad5a628deedbab2e659200f003cdeb18ea3620

Update jQuery Migrate to v3.0.1

* Fixes IE9 issues with console.log.
* Add warning for $.fn.hover().
* Add warning for $.holdReady().
* Improve handling of jQuery.easing methods.

Change-Id: I38084ac6367b6d2ba47d6ecc6c9934a8bb7d8ae0

Update OOjs UI to v0.23.2

Release notes:
 https://phabricator.wikimedia.org/diffusion/GOJU/browse/master/History.md;v0.23.2

Depends-On: I9d1264ebf456cadc7e350c2558a9997a2a0633c4
Change-Id: I9bbd4b8ff6333816e7ba73b3c033623753bb6df5

Correcting check icon placement in RTL

Bug: T176796
Change-Id: Ib0c6faa77f393a9f0f78b9d9d492c4c25fb0d3b5

Localisation updates from https://translatewiki.net.

Change-Id: Idf6479521c3219958d6e50941b5876e735e9cf9b

Merge "Make mw.widget.TitleWidget more flexible"

Merge "RCFilters: Log performance data"

Make mw.widget.TitleWidget more flexible

* Factor out #createOptionWidget so different option
  widget subclasses can be constructed
* Factor out #getApiParams so extra data can be fetched
  from the API
* Pass through raw API data to pageData, so extra API
  data can be used in option widgets

Change-Id: I150c513e4144ad5b57643e98dd48866ce2d37850

Merge "RCFilters: Also set toplinks-collapsed class if no cookie"

Merge "RCFilters: Cache ChangesListSpecialPage::buildChangeTagList()"

Fix insertSelect() with IGNORE in PostgreSQL

I0e6a9e6d overlooked the special handling PG needs (prior to 9.5 anyway)
to properly emulate MySQL's IGNORE option when delegating to the parent
implementation.

For now, then, don't use the native implementation in PG when IGNORE is
specified. Instead, fall back to the non-native implementation that does
a select() then an insert() where PG can handle the IGNORE properly.

In the future we might use the ON CONFLICT DO NOTHING clause added in PG
9.5 to be able to do native insertSelect() with IGNORE (and to better
handle multi-row insert() with IGNORE, and we could use the related ON
CONFLICT DO UPDATE to implement upsert()). All that is left for a future
patch.

Change-Id: I7987d59580543de03d5c6a5ed7fa3ce551ac12f3

Merge "SpecialPages: Reduce selector specificity"

objectcache: Improve WANObjectCache test coverage

class WANObjectCache (methods: 30% -> 40%, lines: 62% -> 65%)

* doGetWithSetCallback: 98% -> 100%
 - Add case: Throw for invalid callback.
* getWithSetCallback: 86% -> 100%
 - Cover case: Else branch for `$cur[self::VFLD_VERSION] === $version`.
* getMulti: 91% -> 98%
 - Cover case: If branch for `$this->warmupCache`.
* set: 93% -> 100%
 - Cover case: If branch for `lockTSE >= 0`.
* reap: 84% -> 100%
 - Add case: Error when changeTTL() fails.

Change-Id: I4844ab0ebdd4d3ec9acc9cd4500721b04cfda317

Merge "Allow two-queue style insertion in MapCacheLRU"

RCFilters: Log performance data

Just UI ready time and wgBackendResponseTime for now.
Sanitizing responseStart and especially firstPaint cross-browser
is too messy. NavigationTiming has code for it but that would
need to be exposed somewhere.

Bug: T176652
Change-Id: I6caf52fe8bc77fac0426d73549553301c5951c32

RCFilters: Also set toplinks-collapsed class if no cookie

If the cookie was not set, the PHP code treated that as
"expanded", but the JS code treated it as "collapsed".
Fix the PHP code.

Bug: T176380
Change-Id: Iff9b865bd20a571ade2ae619cbb5b3a70629003c

Give thumbor swift user r/w access to containers

readUsers and writeUsers are new optional values
of the swift backend configuration. They allow
giving read and/or write rights to additional
users than the default swift user.

Bug: T144479
Change-Id: I0f81a013ec994eee3f156a89f29f4fcfc37c42b7

Allow two-queue style insertion in MapCacheLRU

Change-Id: I1cd98ea8965b51e43371efd990fb9302bb507928

Merge "WLFilters: Live update and View newest"

Merge "HTMLSelectAndOtherField/HTMLSelectOrOtherField: Change how ID is handled in OOUI mode"

Localisation updates from https://translatewiki.net.

Change-Id: Ic032757aa36350153e2df304b7636a4b66916f64

Merge "Add print styles for gallery mode packed-hover"

RCFilters: Cache ChangesListSpecialPage::buildChangeTagList()

Calling Message::parse() on 2 messages per tag (for 80+ tags)
is fairly expensive. It takes about 400ms in production, but
adding that to requests that normally take 150-400ms is a pretty
big hit.

Bug: T176652
Change-Id: I9114f69de8b18007735de3438809f5695e380738

Merge "Block same-file reuploads"

Merge "mw.special.preferences.timezone: Remove some very old unnecessary code"

HTMLSelectAndOtherField/HTMLSelectOrOtherField: Change how ID is handled in OOUI mode

Set the ID on the main widget (mw.widgets.SelectWithInputWidget), so
that it can be infused by ID.

The dropdown/textbox by themselves can't be infused individually,
therefore IDs on them are pretty useless; remove those where they're
not necessary.

Follow-up to 8bd6605736c47259bd5f901284cbd2e639cef30c.

Change-Id: If54dd48f1000e3e0f5a978428a5b622797b4f765

Merge "resourceloader: Remove deprecated MessageBlobStore::insertMessageBlob"

mw.special.preferences.timezone: Remove some very old unnecessary code

The first two removed blocks date to the pre-HTMLForm versions of the
preferences form (rSVN44915 / c672ce603272d5206f0967f70f2474b4c09ce43a),
where the dual dropdown/textbox field was implemented specially for
this form. Its behavior currently is handled by HTMLSelectOrOtherField
and selectorother.js, and this code is entirely unnecessary.

The third removed block was added as a workaround for an issue caused
by the second block, in e8d94f433c43982003fb0d008e1fc8ee04ed7744.
Looks like we didn't realize at the time that we can just delete the
buggy line.

This fixes a minor issue where no warning would be displayed when
exiting the preferences page without saving the changes after the user
changed the "Time zone" field from "Other" to "Fill in from the
browser" and the guessed value was different from the saved one.

Change-Id: I9b5317111aaf7f6039d476b263ee9e54c4b4cf52

Merge "Unify monospace font family stack"

Merge "Update documentation for IApiMessage interface"

resourceloader: Remove deprecated MessageBlobStore::insertMessageBlob

Deprecated since MediaWiki 1.27. No uses in Wikimedia Git.

Follows-up 5d5b269e0e63. (T113092)

Change-Id: I98c4db4fb0a3909d25d0fcd587e0f34391612423

Add print styles for gallery mode packed-hover

Bug: T140346
Change-Id: Ifd94706c427c6bb7577040bbfecfdc6a303e0d8b

Merge "getConfiguration: Don't bail when a valid variable is set null"

Update documentation for IApiMessage interface

It was created for a specific purpose, and apparently this is confusing
some people who're inexplicably trying to implement it outside of that
purpose. So, this patch more clearly documents that purpose.

Change-Id: I638482327370522bedea1ec514ecbb9d98207382

Merge "Make `ApiUploadTest` up to date so it passes"

Merge "resourceloader: Implement isKnownEmpty for user.options module"

Merge "resourceloader: Support isKnownEmpty for general modules"

Merge "registration: Fix caching of load_composer_autoloader"

Merge "EditPage: Disable preview on open if $wgRawHtml is enabled"

Unify monospace font family stack

Bug: T176636
Change-Id: I4bab7f13b5bd641df9a2a9638e60e14951313d52

Merge "WikiPage: Hard deprecate second arg of prepareContentForEdit() as an int"

WLFilters: Live update and View newest

Moved handling for 'from', 'days' and 'limit'
to base class (ChangesListSpecialPage)

I moved 'days' because its implementation
is related to 'from'.

I moved 'limit' because it was getting lonely
and it's identical in all cases.

Bug: T176348
Change-Id: If6280ad6fbad65909e1d0b2a48344e24d485aca2

Merge "EditPage: Hard-deprecate isOouiEnabled(), getCheckboxes(), getCheckboxesOOUI()"

Add more ProcessCacheLRU comments

Change-Id: I5b59dd065dc387a6c7f9cae42ea5244247448650

Localisation updates from https://translatewiki.net.

Change-Id: I690f31abfbbfaff56761bde38a0813d12c949f39

Merge "RCFilters: Don't load all of OOUI"

EditPage: Disable preview on open if $wgRawHtml is enabled

Otherwise the user will see a session failure message since their
request wasn't posted with a token, which doesn't make sense since they
just clicked on an edit link.

Bug: T172251
Change-Id: Id4a20818c737886f5a7ec72dba52c531f1ba78a0

Localisation updates from https://translatewiki.net.

Change-Id: I41d3d09b418676e2b1171c02ac72182455ba5e16

SpecialPages: Reduce selector specificity

Reducing unneeded selector specificity.

Change-Id: I96b46b4e8b267c239c9a2cb73f65848c9fff9be1

SpecialPages: Align subtle and disabled text color to WikimediaUI palette

Aligning ProtectedPages unknown and disabled user rights text colors
to WikimediaUI color palette and make them conforming to WCAG level AA.

Bug: T152025
Change-Id: Id10477f92dd8b43caa26659cb63352637b436bda

Merge "HTMLForm: Do not generate wrappers for empty sections"

registration: Fix caching of load_composer_autoloader

Move the file_exists() check out of the extension processor and into the
extension registry so that it is evaluated at run time instead of during
caching. The prior way is problematic since we don't invalidate the
cache if the existence of the file were to change.

Bug: T176534
Change-Id: I98e4ffdfac9f98397a103966824519afe1375356

Merge "Use native ES5 Array prototype methods instead of jQuery"

RCFilters: Don't load all of OOUI

Just the widgets module is enough, we don't need
the windows or toolbars modules.

Change-Id: Ie8ce45f0d46fbc759d4579d619ab8c99c13cb0d5

Merge "Improve "selfmove" message's wording"

Improve "selfmove" message's wording

The error shown when a page is to be moved (renamed) with the same
title is not immediately obvious to the user, so use simpler
and clearer language.

Bug: T164987
Change-Id: I639c4ae27866234fed9bcc5f2afc4684155418f8

Use native ES5 Array prototype methods instead of jQuery

Replace
* $.each( array, function ( index, value ) { ... } ) by
  array.forEach( function ( value ) { ... } )

* $.grep( array, function ( value ) { ... } ) by
  array.filter( function ( value ) { ... } )

* $.map( array, function ( value ) { ... } ) by
  array.map( function ( value ) { ... } )

Change-Id: I985ddf710e13c9ae788245349e2791571aeec97e

SpecialSearch: Fix unintended `margin` when zoom level is above 100%

With zoom level > 100% on Firefox, `#mw-searchoptions` which is a
`fieldset` element adds unintended `margin` with negative `margin-top`
applied. This is a workaround for wrong browser behaviour and
regression of I4bda42c03a5.

Bug: T176499
Change-Id: I329f83e6063460dc11ff45583e335280c9257ef7