Merge "Tests for Revision::userCanBitfield"

Merge "Test for Revision::newKnownCurrent"

Provide message/warning/error box abstraction

This will help us consolidate the various uses into one single
method which will help us drive standardisation of these defacto
widgets.

Hopefully, by being a method of the Html class, which has a very
low barrier for use will drive down the inconsistent display of
warning/error boxes across MediaWiki's products

Various usages of warningbox and errorbox have been ported over.
I've retained some more complicated usages which make use of the
parser (wrapWikiMsg) and any where id and class are medled with
- we'll probably want to consider whether we want to encourage
those going forward as they encourage adjusting the styling.

Bug: T166915
Change-Id: I2757e1f4ff2599e93a7257fc644cab69063896d2

Merge "RCFilters: fix call to changesListModel.update()"

Localisation updates from https://translatewiki.net.

Change-Id: I964eda712b69d542ca230f44ba125f0f29a328a2

Merge "Change disabled saved filter tooltip"

Merge "Use mw-parser-output for content styles, instead of mw-body-content"

Merge "Change the rfcurl from protocol relative to HTTPS"

Localisation updates from https://translatewiki.net.

Change-Id: Ic11c64813ee04e8520771bfa156f8e51404273e6

Document language changes for Mirandese

Bug: T180052
Change-Id: I60d809145678a2cd523776ba8e981da21f6e3ef9

Merge "Move watcheditem classes to watcheditem directory"

Merge "Remove htmlspecialchars for Html::hidden"

Localisation updates from https://translatewiki.net.

Change-Id: Ie6054daa2b17649b21f5d1b167d3ce517bc604cc

Gendered NS_USER namespaces for Mirandese (mwl)

* Adds gendered NS_USER namespaces for when the user has chosen to
  reveal his or her gender via Special:Preferences.
* Changes the NS_USER namespace to use the gender-neutral
  "Outelizador(a)" in an identical fashion as the Portuguese projects
  do with "Usuário(a)".
* General fixes.

See community consensus in the linked Phabricator Task.

Bug: T180052
Change-Id: I9bac9e6a8cb5b6feee6490cf251342389237bd5c

RELEASE-NOTES-1.30: Add upgraded, added libraries based on git history

Change-Id: I4d433f8f75cfe04638adbd60d94346f629cc2f18

Suggest running composer (install|update) with --no-dev

Change-Id: I1a931b6ac2767aedfc4c71a0ddac460bd3322484

Localisation updates from https://translatewiki.net.

Change-Id: I244eb49276f46f4b722d381188b527d93295e9d3

Merge "Convert @var Array => array"

RCFilters: fix call to changesListModel.update()

Followup to Ie2bc4fe4bd19103b513e4753e59f4adb295fbafa

3rd argument is
  {string} noResultsDetails Type of no result error
... not a boolean (isTimeout) like it used to.

Change-Id: I7766ed082eb2f4e43a89e3b5c0fb2e61bf5f1169

Specify an explicit text colour on some components that change the background colour

These components separate themselves from the default colouring of the content
area by specifying a dedicated – typically light grey – background colour.

However they leave text colour unspecified and as a result if the skin should be
white on black instead of black on white this results in these components becoming
white on white or white on light grey and becoming unreadable.

Fix this by explicitly specifying a text colour to use where the background colour is set.

Bug: T66732
Change-Id: I12584de40a18929b610d46dceaea73f06fdc6541

Updated dev dependancy phpunit/phpunit from v4.8.35 to v4.8.36

Change-Id: Ib800b243541557b3aaab03c10a796dd564f6f3ef

Tests for Revision::userCanBitfield

Bug: T180210
Change-Id: Idf7e9d0f05b967b2fc083e6a70e9411b53390ed7

Use mw-parser-output for content styles, instead of mw-body-content

Change-Id: I8bddeda294b8f52fd71754b63b5afb6d30ff2b7d

Test for Revision::newKnownCurrent

Bug: T180210
Change-Id: I2cc83cbc91583b4631f4798dd14612c49024eeb4

Tests for Revision::decompressRevisionText

Bug: T180210
Change-Id: I21f6bb02757bcf59ce483e7d61685d372f134048

Run RevisionIntegrationTest for both ContentHandlerUseDB settings

core used to have a test case that ran various Revision tests
with and without ContentHandlerUseDB set, however it did not
run due to the class / file having a bad name.

This patch reintroduces the running on Revision tests for both
ContentHandlerUseDB settings.

The global has to be set in setup / before the page used for
testing is created, else this page will be created with the
wrong values in the DB for use in the tests.

Change-Id: Ied1023e22eeac413fa86ea6946aa31619fd66319

Make Revision::testIsUnpatrolled more reliable

Bug: T180210
Change-Id: Ib88c39527fafefdaafdb05c944716b5a1449cf1b

Convert @var Array => array

Change-Id: Ie5c2d7b8e73cce74af982a5c54b0d487a7f6981d

Merge "Rewrite userOptions.php"

Merge "Setup: Include StartProfiler before others"

Merge "Merge ProfilerFunctions into GlobalFunctions"

Merge "Sync up with Parsoid parserTests.txt"

Setup: Include StartProfiler before others

Bug: T180183
Change-Id: Ibcf78d094cf4dcf09bc919a5f8168f45ae225ebc

Merge ProfilerFunctions into GlobalFunctions

Even if people use these (deprecated) functions in the earliest hooks or in
LocalSettings.php, it will keep working because GlobalFunctions is loaded
between DefaultSettings.php and LocalSettings.php.

The only places affected would be files in core: AutoLoader.php, Defines.php,
and DefaultSettings.php, which don't use these functions.

Change-Id: If4c0e8cbe1ea918283df22d72f792a3806569216

Use the main stash for LBFactory "memStash" parameter

This store is used for ChronologyProtector positions.
It should be cross-DC since the sticky DC cookie may not work
for rapid cross-wiki farm activity, causing some request go to
the non-primary DC.

NOTE: this change should be deployed on all farm wikis at once

Change-Id: Ife126592aacace696e43912b9461164a9ea98bc1

Sync up with Parsoid parserTests.txt

This now aligns with Parsoid commit 1d6c39d8f6f5972e72974f8d64e7a0a5c2288bf2

Change-Id: I38d9d47c9cd74257b9bedc892baad90146885ef4

Merge "Add action/user tracking to html cache purge jobs"

PopulateRecentChangesSource: remove unused variable

Change-Id: I0c1fb623ad508d604c0e7760c2133920ec9a81f8

Merge "widgets.DateInputWidget: Enhance desktop UX cursor handling"

Merge "widgets.DateInputWidget Align design with WikimediaUI"

Localisation updates from https://translatewiki.net.

Change-Id: Ic3b7173db986c1984b910f00fd95c6fcb761a296

widgets.DateInputWidget: Enhance desktop UX cursor handling

Enhance desktop experience with better cursor handling.

Bug: T169034
Depends-on: I4e0ca6c472e2d6ddbe64eb783acf8c38c5beacc4
Change-Id: Ie4c847caf727051ed5bbcf9937863b007e62d3c7

Merge "Escape single quotes in attributes"

Move watcheditem classes to watcheditem directory

Change-Id: If915c875380b4ecd74fad64df7833de87ea6d6f7

Remove htmlspecialchars for Html::hidden

Html::hidden already makes an HTML escaping of the values.

This is a followup to ff2f868a9.

Change-Id: I6f63494038ffbdde0df34ad5566dfa03707dde46

Add tests for various simple Revision get and is methods

These tests dont cover all code paths for these methods
in some cases but it does ensure that tests actually call
them / test basical functionality.

Some of these methods are touched in
I4f24e7fbb683cb51f3fd8b250732bae9c7541ba2
hence adding some test coverage now.

Change-Id: I7eb67cb5876f89034238591c0a98f4ec6d890a40

Add tests for Revision::getTitle

This method is touched in I4f24e7fbb683cb51f3fd8b250732bae9c7541ba2
hence adding some test coverage now.

Change-Id: Ie39a294546d84413bfb716efb27fd2c0321b349b

Add tests for Revision::getParentLengths

This method name is quite missleading as it doesnt
get the parent lengths at all, it gets the lengths of
the rev ids that you ask for......

This method is touched in I4f24e7fbb683cb51f3fd8b250732bae9c7541ba2
hence adding some test coverage now.

Change-Id: Ic837bd60a34341860c4d2287a1b999f40d4f95f1

widgets.DateInputWidget Align design with WikimediaUI

Aligning DateInputWidget with WikimediaUI theme widgets by:
- using “Red50” color `#d33` from color palette,
- using standard dialog `box-shadow` value,
- ensure handle appearance is similar to standard TextInputWidget,
- align variable name,
- introduce similar variables from WikimediaUI theme, and also
- removing unnecessary properties.
Adding a `max-height` to address vendor specific UI elements for
`type=date` in Chrome.

Bug: T180094
Change-Id: I4e0ca6c472e2d6ddbe64eb783acf8c38c5beacc4

RCFilters: some more highlight cleanup

Change-Id: I81eb3bf86f786c79a94e99d51602d904504f918f

Merge "RCFilters: Don't reload the list if the change was highlights-only"

Escape single quotes in attributes

Replaces a long excuse in Html class not to call
Sanitizer::encodeAttribute() with a call to it.
In anything security related, excuses are a sign that you're doing
something wrong:)

Change-Id: Icf7f60d8cd6ea757d8f1999d638b82733001f68a

Change disabled saved filter tooltip

Bug: T179743
Change-Id: I2359bec7d89c6184dafbb5a1f90ea0f47cccd962

RCFilters: Don't reload the list if the change was highlights-only

Bug: T164131
Change-Id: I9952467d0acef84a6445d970977c1265a3ebff95

Localisation updates from https://translatewiki.net.

Change-Id: I4f1007bfb2661ea9f18dc00e6795f102ed29c918

Merge "Group messages in WANObjectCache by key"

Group messages in WANObjectCache by key

Change-Id: I31c570a095735c9157332660e9f1f542e679410f

Remove trailing newline and period from message

Change-Id: I3247edd22d12ea390cd969d8d89f84fa7b1dfdba

Merge "Fix wrong variable in ShowSearchHit call."

Finish migration to Maintenance::getBatchSize()

Change-Id: I02d89f71d820e4d00a39e86a30397b614bbdb432

RCFilters: Remove redundant replaceUpdatedUrl method

The method should call updateURL anyways. If it's initializing, the
comparison to the original URL will still be false and the system
will apply replaceState on the URL.

Change-Id: Ibfa104ed2aba94a5c17ed8ce3580a089d60e4d97

Fix wrong variable in ShowSearchHit call.

The code uses $size, which does not exist and never used otherwise.
The actual size HTML is stored in $desc.

Change-Id: Ida5e69c81acea6bdec75810cf7b192f9dc7cf327

Merge "Timeout autoHide notifications based on visible time"

Merge "Update OOjs UI to v0.24.2"

Timeout autoHide notifications based on visible time

On supported browsers handle the auto hide timeout with a
count of cumulative time the page has been visible to the
user. Old functionality can still be accessed, if desired,
by setting the visibleTimeout notification option to false.
On browsers without support for this visibilitychange event
wall clock time (the old behaviour) is used.

Adds a library function functionally similar to setTimeout that
only considers time when the page is visible. This is useful
both for analytics purposes, and when you want to temporarily
put something on screen and be reasonably certain it doesn't
go away until a user has seen it.

Bug: T42322
Change-Id: I7d8ea85602cae9cfc72e0155bc3092049ecafd43

Merge "mw.special.apisandbox: Use a real button for "Auto-fill the token""

Update OOjs UI to v0.24.2

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

Depends-On: I0e9d3f690349f9f01fb6424f9cb8468ebb21455a
Change-Id: I0e9d3f690349f9f01fb6424f9cb8468ebb21455b

Update for Phan 0.10.1

Change-Id: Id292944b80eb0c81956b203ce95bb0def3b4b0a3

mw.special.apisandbox: Use a real button for "Auto-fill the token"

I've never liked this interface. It is not obvious what the little
triangle does. Let's just use words.

Change-Id: Ica477713adf67a6e1909629eae6b6ee8b35a48f6

Merge "upload: Simplify UploadStashTest by using getNewTempFile()"

Rewrite userOptions.php

* Convert to use Maintenance
* Clean up
* I want to use the class name UserOptions for something else
  so rename it.

Change-Id: Ic441087702376b1ca0e70554c71cdf7ecad908af

Localisation updates from https://translatewiki.net.

Change-Id: Ic0bf3eb6356c270bfa90bd0ade9a08af77de0c94

Merge "RCFilters: Remove excluded params from URL"

RCFilters: Remove excluded params from URL

This was gone unnoticed in the previous upgrade of the UriProcessor
but should be reimplemented: excluded parameters (sticky) should be
excluded from the URL. They should be merged into the uri values so
that their **values** are passing through, but the eventual result
of the URI query itself should not have them.

Change-Id: I06bf2f4da54297e2ec6962dcadb7a4924351cc35

Merge "Fix mergeMessageFileList.php --extensions-dir extensions:skins"

Merge "Remove nbsp and similar characters from section IDs"

Merge "Add render moved paragraphs marker in diff view"

Fix mergeMessageFileList.php --extensions-dir extensions:skins

Only last directory of multiple was being used.

Follows up e74bc3b32effac7e0bbf9e314f4a6f15eedf7bff

Change-Id: I30190a30c387f1c34a41f9bbc033d421be631756

Merge "Make it possible to override SpecialPreferences::showResetForm()"

Merge "Capitalise URL in uploadstash-file-not-found-no-remote-thumb"

RCFilters:Make bookmark icon gray when unavailable

- Make bookmark icon gray out when unavailable instead of disappearing.
- Include a tooltip in the grayed out state only to say:
"These filters are already saved."

Bug: T179743
Change-Id: I3e4d711f9b8f2b0a2c1195c9d7632de417e6d0d5

Make it possible to override SpecialPreferences::showResetForm()

Change-Id: I390e9d46fd2b8d4d8a1f9fd250c964a696b48244

Merge "jquery3.js -> jquery.js"

upload: Simplify UploadStashTest by using getNewTempFile()

The parent class has a built-in list that tracks these files and
deletes them in a teardown.

I was unable to reproduce a case where $path + '.' is created by
MediaWiki. As far as I can tell, no current code exists in core
that would do this. If it does exist, we can keep the tearDown()
override with just that second case, but I'm removing it for now,
given it doesn't appear to be used.

Change-Id: I5847b7b266f1393d983aeb4b115bc0ae000e3547

jquery3.js -> jquery.js

Per Krinkle's comment.

Change-Id: Ib8c78b47b6a700dcf753a7a1fd90d05d649f52d2

Merge "Introduce Maintenance::getBatchSize()"

Move edit font styles to separate module

This allows pages that don't load mediawiki.legacy.shared
(e.g. MobileFrontend) to still have access to edit fonts.

Change-Id: Ifff40f756558e30f6551f29a945a31f0eef4cfed

Capitalise URL in uploadstash-file-not-found-no-remote-thumb

Bug: T179872
Change-Id: Ib32ffedd20fac4c79c333b9be8b96e6ace80ab65

Localisation updates from https://translatewiki.net.

Change-Id: I9153b70314bacb5fb0e6e979d9a3274d60f6bd29

Merge "Ensure WCAG level AA contrast"

Add missing @throws in doc

Change-Id: I978369a727d557db8650b32ec155a6b490c4c866

i18n UploadStash exception messages

In some cases, the error messages have changed slightly, mostly because
of capitalisation & punctuation.
In a few other cases (mainly UploadStashNotLoggedInException), the content
has also slightly changed (removed mention of the __METHOD__ it occurred in)

Bug: T178291
Change-Id: I184067f2d7fe0a0a2df1114d2525fd9ab95b6c86

Rework Upload*Exception classes to implement ILocalizedException

Bug: T154781
Change-Id: Ia64295d7ea502014586b8b8e3e3f34272b72443c

Merge "Use Maintenance::setBatchSize() everywhere"

Merge "Specify a blocker for all UserTest blocks"

Introduce Maintenance::getBatchSize()

Just to isolate the internals. Fix most of usages in the core.

Change-Id: I8b3e9ca1f42b7c49ee57f17b88ca2fc7b404f342

Localisation updates from https://translatewiki.net.

Change-Id: Ib78fbd243c028637110c05371ec0d8b935e760a1

Specify a blocker for all UserTest blocks

Also, un-register the PerofmrRetroactiveAutoblock hook from CheckUser
as it will assume an IP address for the user which is not specified in
UserTest. This will keep the unit tests focused on unit testing, as
opposed to integration.

Bug:T176103
Change-Id: I3a090f7bae5ba266c3cd4500ecf83338fb8503bb

Language: Update git.wikimedia.org URLs to point to Phabricator

Bug: T139089
Change-Id: I38492e0c8e90f3418bcb831f086fbe6ea2b319b5

Ensure WCAG level AA contrast

Increasing contrast on span.stats slightly to meet
WCAG level AA contrast minimum requirement on possible
background color `#f9f9f9`.
Also bringing selector in correct parent > child order.

Change-Id: If033cd61bd81dbd5773810b1a944f20a9bf2e58d