Merge "Simplify ApiQueryRecentChangesIntegrationTest"

Merge "Integration tests for ApiQueryRecentChanges"

Simplify ApiQueryRecentChangesIntegrationTest

This patch does two major things to the newly introduced
ApiQueryRecentChangesIntegrationTest:

* It rips the confusingly complicated assertArraySubsetsEqual appart.

* It removes all wikitext content. Note that no assertion is done on this
content (not even in the test that is about adding a category). The tests
run just fine with automatically generated content. I had to introduce the
$i counter to make sure consecutive edits are done with different content.
Otherwise the later edit is not stored.

Change-Id: Iae585be86f3fd9bff7d2b4b7b218d5b0b4258603

resources: Upgrade jquery.chosen from 0.9.14 to 1.8.2

Change-Id: If9c720ebca8ca663869d0f73b0cd3ea415262cff

Fix inconsistent capitalization of different method calls

Change-Id: I9f5b9e59e8cdadf65e80077fe2d3a9822b4592fe

Merge "Special:ProtectedPages: Use HTMLForm"

Special:ProtectedPages: Use HTMLForm

We could easily get rid of functions like getExpiryCheck
and getCascadeCheck if required.

Bug: T117722
Change-Id: Ia964192b50464ebec00e0812e14b804fe91132ca

Localisation updates from https://translatewiki.net.

Change-Id: I9f3aca4cca70f4e0d747eeec0020948298cc14fc

Localisation updates from https://translatewiki.net.

Change-Id: I495ab349530db87303fc33f06b2610681ba3efc5

tests: Use checkPHPExtension() instead of re-implementing it

Change-Id: I7f5e8684d556befc0aefa302187c573e7a3cff62

Add @covers tags for Collation tests

Change-Id: I8b0623a6b716acdc9d369349fd4e306dbdc91d18

Integration tests for ApiQueryRecentChanges

It looks big but it's mostly copy from ApiQueryWatchlistIntegrationTest

Change-Id: I96ccd3e2813e77f47a4432ca434c1c6a212d90ea

Localisation updates from https://translatewiki.net.

Change-Id: I69b50503f8a9db7b3883f664f2703ab16f9132dc

Merge "Allow fallbacks for $wgReferrerPolicy"

Fix @covers in XmlTest

Follows up 389fa13ff875.

Change-Id: I32ddf37b24a6b5162b383260233d616cb49f7ddb

Add tests for several methods of Xml

Added tests for methods: monthSelector, span, buildTable, buildTableRow

Change-Id: I5ca6bb231b5ed95f57f649a92d659245a4852c85

Merge "[MCR] Don't require $title to be passed to Revision::newFromId"

Merge "[MCR] Introduce BlobStoreFactory"

Merge "Add `margin-bottom` for editButtons (EditPage)"

Add @covers tags to CategoriesRdfTest

Change-Id: I6e2eae3fc2ac722dc94bee1a7aa6632e08a4d097

[MCR] Introduce BlobStoreFactory

This allows Revision::getRevisionText to get
a different BlobStore instance when $wiki is passed in
restoring the behaviour for $wiki before the MCR Revision
overhaul patch was merged.
Ia4c20a91e98df0b9b14b138eb4825c55e5200384

Bug: T183634
Bug: T183631
bug: T183583
Change-Id: Ib0949454e9a003c2965adc1aab38e31fcf121afe

Allow fallbacks for $wgReferrerPolicy

For browsers that support older versions of the Referrer Policy
specification (Edge and Safari), using a value from a newer version of
the specification will result in those browsers falling back to a
default of "default".

So allow $wgReferrerPolicy to have fallbacks if the browser does not
recognize the currently set value. It will emit <meta> tags for each
value in the array, but in reverse order, as browsers will use the last
one that they recognize.

Bug: T180921
Change-Id: Ie0f523fc6937c9ecffc8a6fc791c6b54d5a1cb06

Merge "Improve documentation of $wgReferrerPolicy"

Merge "JavaScriptMinifier: Improve docs around parsing of regexp literals"

Merge "JavaScriptMinifier: Remove support for unused $statementsOnOwnLine flag"

JavaScriptMinifier: Improve docs around parsing of regexp literals

Bug: T75556
Change-Id: Ifcb6bc21418dfc2e1d3e44dbd2497a0f5f691bf3

Add `margin-bottom` for editButtons (EditPage)

Bug: T179040
Change-Id: I9e6c04556e4ff7f0cc567877041449f50abf4b4c

Merge "Remove duplicate ServiceWiring definitions"

Localisation updates from https://translatewiki.net.

Change-Id: I26ca1588d1f8063a673809759e5b05728e4dc550

Remove duplicate ServiceWiring definitions

Change-Id: I70b377c417ce6937e5d52d1790cf0b6f2ed33c15

Merge "Set wgCommentTableSchemaMigrationStage in RevisionStoreTest"

[MCR] Don't require $title to be passed to Revision::newFromId

If the title is not passed in as a param (already known) then select
it in Revision::newFromId instead of waiting for it to be selected
further down the tree.
This means that we can use the same Title object to pass into the
RevisionRecord as well as our legacy Revision object.

The selection chooses either a slave or master depending on
recent writes.

This logic used to be in Revision::getTitle and also in
Revision::newFromConds which was called by newFromId

Bug: T183505
Change-Id: I9cf4ce2c3c86d6bf979a3c88eb423b942b9a1ba4

Improve documentation of $wgReferrerPolicy

Reference current version of the spec.

Change-Id: I1cca89b9e46a2b5ae8242715eddfdbd23ccb2d83

Update reference to DatabaseBase in hooks.txt

Change-Id: I65f84c0deadb611380e17a5b1b7593876894c099

Localisation updates from https://translatewiki.net.

Change-Id: I1edcc9b4af56fb6b66961a14e38d878ff97843e9

Merge "RCLFilters: support longer page names"

Merge "resourceloader: Improve JavaScriptMinifier code coverage"

JavaScriptMinifier: Remove support for unused $statementsOnOwnLine flag

The $wgResourceLoaderMinifierStatementsOnOwnLine config var was deprecated
in MediaWiki 1.27. The parameter of minify() was not used by other code, and
no new usage has been introduced since then, either.

Remove the feature from JavaScriptMinifier, and add a release note for that.
The same 1.31 release notes also contain a note already about the removal
of the configuration variable.

The feature was not covered by unit tests.

The following private variables have been removed, that are no longer used
due to this change: $newlineBefore, $newlineAfter, $newlineAdded.

Change-Id: I2cbf271156c1954abb982531d0125b4b9573b12c

RCLFilters: support longer page names

* Make target page selector at least 400px wide
* Set the page name as title (tooltip)

Bug: T183484
Change-Id: Ife806c0b5e5c2ab6e9be32194804e89daaf8b71c

RCFilters: Move product-specific rules into RCFilters styles

Afterwards, the fix in Vector skin should be merged:
Ie36eeddc46c8ea3d154a006fed2fe578be1cd05c

Change-Id: I8357ef2ff0fe4f46b9ee665ef068624de0afd2e5

resourceloader: Improve JavaScriptMinifier code coverage

Change-Id: I234b996cfec8ef48ce3dc48aabbdf88bc9439c15

[MCR] Revision::newFromArchiveRow convert overrides for rows

This method used to overwrite attributes, then passed to
Revision::__construct
RevisionStore::newRevisionFromArchiveRow instead overrides row field
names

This patch adds a conversion for the one field that we need to care
about which is 'page' -> 'page_id'.

After looking through the usages in core and extensions it looks
like this will also fix a bug in the following classes which also
passes in 'page'.
 - RevDelArchivedRevisionItem
 - RevDelArchiveItem

Bug: T183564
Change-Id: I6a472b93663a0599abb55453c6939463ff56275d

Merge "[MCR] Add and use $title param to RevisionStore getPrevious/Next"

[MCR] Add and use $title param to RevisionStore getPrevious/Next

When the title is already known use it.

This is similar to the issue fixed in
I714ee391caac9bc56ce4c037967e424b44d9c2fe.

As of this patch all methods within RevisionStore
that call getTitle internally also have an option
to pass in an already known Title object.

Bug: T183548
Change-Id: Ieabca1cf157fb667c75fc907b9da2917f71c61b3

Merge "Remove ESLint inline comments in WebdriverIO configuration files"

[MCR] Add optional $title param to Revision byId methods

Bug: T183505
Change-Id: I714ee391caac9bc56ce4c037967e424b44d9c2fe

Merge "Improve coverage of PageArchive"

Improve coverage of PageArchive

Some minor tests

Change-Id: I5c455d6665febc20304e278828ddf8fb3cb5e709

Merge "RCFilters: Create one single source of truth for item display"

Merge "[WikiImporter::notice] use wfDebug instead of echo in notice()"

jquery.tablesorter.less: Simplify by using less syntax

Change-Id: I4f7ee23c02c6ef0d39cb0135e6669f23b6977c59

RevisionTest: Fix @covers tags

Namespaced classes need to be absolutely named.

Change-Id: I48a4b356835e9edf3151a0c8d3bed08ea8cfb452

RevisionTest: Make @covers tags absolute (re-apply)

PHPUnit requires @covers tags to be absolute, they cannot depend
upon `use ...` statements.

Change-Id: I06efa94be3a814acbcb116162db162dcbf36432e

[WikiImporter::notice] use wfDebug instead of echo in notice()

Make the default notice implementation of notice() function to
send the notice to wfDebug() instead of simply echo to prevent XSS

Bug: T177997
Change-Id: I9ffb597e0642afc1603cdd15e2d09c6f2584b448

Merge "objectcache: fix getMultiWithUnionSetCallback() doc"

Merge "shell: Add NO_LOCALSETTINGS restriction"

Merge "shell: Add debug logging to find binaries that aren't being restricted"

Merge "shell: Run firejail inside limit.sh, make NO_EXECVE work"

RCFilters: Create one single source of truth for item display

Synchronize the search input, filter view and all view selection into
a single source of truth through the model.
- Stop using OO.ui.MenuSelectWidget's internal filtering logic, and
  use our own filtering logic in the model instead.
- Add a concept of 'search' to the model, that is synchronized with
  the current search term of the systme. The search dictates the view
  and the visible items in the filters.
  -- Whenever the search changes for whatever reason,
     the model automatically updates the visibility of all items and
     groups in the filter list.
  -- This means that after changing view or search, the widgets can
     always trust that they have the most updated visible item list
     to scroll to or select.
- Make the view model's findMatches
  as the ultimate source of truth as to what should be visible.
  The items are now **always** all included in the menu, but they
  change their visibility according to the logic in the findMatches.
  If there is a search term, the function will account for that and
  the selected view, so we can be sure that search results also
  change between changing views.

This allows us to simplify the operation of selecting a tag, where
we know that the order of operation is bound to switching the views,
that will also show us the relevant results. We can bind to the display
event and properly select -- and scroll to -- the item.

Bug: T182720
Bug: T178540
Change-Id: If575f348705d863b9d68d19761ded55a4d453550

[MCR] Use RevisionStore::getArchiveQueryInfo in PageArchive

This is needed to allow RevisionStore to be switched to an
alternative database schema for MCR.

Bug: T174026
Bug: T183509
Change-Id: I51318cb102f6fb8962e608196bee96bde023bcff

[MCR] Add @covers tag for PageArchive::undeleteRevisions

Change-Id: I8dd19244e29f05e22aa5014ec6d6182b5917cc3c

Merge "[MCR] Add test for PageArchive::listRevisions"

[MCR] Fix RevisionInsertComplete Revision pass by reference

This was broken in:
6af796f3e0cf3e66cd7d7e59af8445f5712d68fe
Ia4c20a91e98df0b9b14b138eb4825c55e5200384
https://gerrit.wikimedia.org/r/#/c/399174/7

Bug: T183503
Change-Id: I674493507ca4aa6677a3bab7a01d6dc3c2cdbc5b

Localisation updates from https://translatewiki.net.

Change-Id: Iaad62f37560b8d965c81d72158dbcf5335babdd8

Merge "ApiBlock: Improve username validation"

Merge "MCR: Deprecate and gut Revision class"

Merge "RecentChangesLinked: Include title without subpage in form"

MCR: Deprecate and gut Revision class

This is a re-submission of I4f24e7fbb68.

As a first major step towards Multi-Content-Revisions (MCR),
this patch turns the Revision class into a legacy proxy for
the new RevisionRecord and RevisionStore classes.

Backwards compatibility is maintained for all but some
rare edge cases, like constructing a completely empty
Revision object.

For more information on MCR, see
<https://www.mediawiki.org/wiki/Requests_for_comment/Multi-Content_Revisions>.

NOTE: once this is merged, verify create/delete/restore cycle on beta,
      ideally with emulated replication lag.

Bug: T174025
Change-Id: Ia4c20a91e98df0b9b14b138eb4825c55e5200384

[MCR] Add test for PageArchive::listRevisions

Change-Id: If4ab58cd889e7166142a9f9a28fc81d776a7db3f

Remove ESLint inline comments in WebdriverIO configuration files

Comments are not needed. ESLint has configuration file(s).

Bug: T175179
Change-Id: Ieafd7b5666faaccb73b8ef7daa7aef3ed4114662

RecentChangesLinked: Include title without subpage in form

When the 'title' hidden field contains the subpage, the 'target'
input field is never taken into account since the subpage always
takes precedence in ChangesListSpecialPage->setup()

I75cfb2b56a4da6357e6117b3f34f3178bfb2c90c introduced
SpecialRecentChangesLinked->getPageTitle which overrides
SpecialPage->getPageTitle to include the subpage by default.

This helped with redirecting to the right page when parsing
a saved query on the server but changed the behavior everywhere
that function was used in the context of RCL. That includes
SpecialRecentChanges.php:545 where the option form is built.

Bug: T183437
Change-Id: Id39d1887719ecb6900b25fefbb133dff9c4d0456

RCFilters: Query using current (not default) sticky parameters values

For some sticky parameters, defaults are synchronized to current
using Item to Group event, but it only happens for some specific
group types.

That should be cleaned up, but regardless, there is no reason
to query using the default values. Current values are what matters.

Bug: T183438
Change-Id: I527b38ea7f91f70231ed74723212e493e9cce9e5

Merge "Fix typo"

Follow-up e213462f7c61c: fix typo ("constriant" -> "constraint")

Change-Id: I6e1fd74e387af5a913301bf296fe9f6da0dd1837

Fix typo

Change-Id: I56e762d0d565e36530a5dc7a7ad0aa7cb66531b8

Test for writes being prevented on replica connections.

Bug: T183265
Change-Id: I319efa418cf8a46985f6faa60096559efa15d267

Merge "Add tags for undo edits"

Merge "Simplify logic to prevent writes on replica DB connections"

objectcache: fix getMultiWithUnionSetCallback() doc

Change-Id: I5700ff8ccbe2ef3aa6730e20bdc17314bceade42

Localisation updates from https://translatewiki.net.

Change-Id: Ibf7138a1acb001da968e74c7cc24333188c675ae

Simplify logic to prevent writes on replica DB connections

This reverts most of 36f4daf32c591d6b7e2435629fc6e431398b641a.

Change-Id: Ie8205749b14be186e80296b168c32310c10ce875

Merge "Fix old style filters disappearing regression"

Merge "RCFilters: Don't load JS or redirect when transcluding"

Fix old style filters disappearing regression

Add FormWrapperWidget after controller is initialized, to avoid it
being removed when changes list view model is updated during
initialization.

Change-Id: I3bd4d1bc9efbcdbf99493498267ed1a690fd1ab2

Merge "Provide an appropriate comment"

Merge "objectcache: add another example case to WANObjectCache::getWithSetCallback()"

Follow-up 36f4daf32c: allow no-write database connections to use transactions

After 36f4daf32c was merged, Aaron pointed out on Gerrit that:

    Transaction can be used on replicas for REPEATABLE-READ mode, so
    queries in that transaction see the same version of the DB. Those
    do not involve writes and things would probably break sometimes
    without it (like reads following foreign keys and reference in
    separate queries).

So allow no-write connections to call begin() and commit(), and only
throw an exception when a write query is attempted.

Bug: T183265
Change-Id: I138677a714df13ead08061c3e9ef794c3befbe9d

Merge "Standardize editfont size across browsers/OSes"

Merge "Remove empty RevisionRecordTest"

Unify monospace font family stack

Bug: T176636
Change-Id: I28d490a6c1decb68fce20307cc58678c9fe64ecc

Standardize editfont size across browsers/OSes

Bug: T176636
Bug: T181324
Bug: T182320
Change-Id: I836a05674723ae0864372f8d0e32c7b430906a92

Localisation updates from https://translatewiki.net.

Change-Id: Ia4bc5f0583c9f8d5d3d1868d6e5f85d2e41afcbf

mw.Feedback: Remove weird unnecessary switch statements

They made sense prior to 7196fafcdcbe658caf9ffde508717714b9cf4510,
but no longer do since the success and error handling are in different
functions. I probably should have removed them in that commit.

Change-Id: I1167dc1b5a79eadfe607abfc73bd2f0e95194b78

Remove empty RevisionRecordTest

Bug: T183219
Change-Id: Ic58308da4a54186995a2d985b40e0bbf86b8cef3

Provide an appropriate comment

Let's stay with blue (`#36c`) for something called blue.

Change-Id: I1ebb0909d491f1ecf4d1e5733e7ac6589d28c731

Merge "Prevent write operations to database replicas."

Prevent write operations to database replicas.

Bug: T183265
Change-Id: I8e17644d1b447416adee18e42cf0122b52a80b22

Merge "Trigger name changes"

Trigger name changes

Modifid the names as described in the task/bug.

Additionally found and added two missing permissions in user.sql, and
fixed an unrelated error to make the schema function.

Bug: T182981
Change-Id: Ia213c0e6b6759a950cdee4181d9c5695e3c5da4a

Merge "[MCR] Fix SqlBlobStore using DB_REPLICA for writes"