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

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"

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"

ApiBlock: Improve username validation

The current username validation lets any invalid username through, on
the assumption that it's an IP address. We can do better: call the
backend to get the actual type and target, and reject anything with
TYPE_USER where the actual input name is invalid (regardless of
underlying mangling for stuff like T31797).

Bug: T183211
Change-Id: I676642eee1222447df22a1c32b24f55e6273bcec

[MCR] Fix SqlBlobStore using DB_REPLICA for writes

Bug: T183242
Bug: T183245
Bug: T183252
Change-Id: Ic0b9910b5de5ce6817d6a4a3e5e32103e2113926

Merge "Revert "[MCR] Turn Revision into a proxy to new code.""

Merge "Add HEBREW POINT METEG to the Hebrew special characters"

Revert "[MCR] Turn Revision into a proxy to new code."

This reverts commit 9dcc56b3c9e7a3212d0900ee23808b9402cad952.

With this patch applied, newly created revisions are sometimes not found
just after submitting an edit, until replicas have caught up.

Our best theory is that it somehow interfere with ChronologyProtector,
but we don't have a good idea how.

Also, as legoktm mentioned, the commit message is terrible and needs fixing.

Change-Id: Idf3404f3fa8f8d08a7fb2ab8268726e2c1edecfe

Merge "RCLFilters: make target and to/from parameters sticky again"

Merge "Require indentation of CASE statements in PHP code"

RCLFilters: make target and to/from parameters sticky again

Regression from Id0bf89ce397ff8a54ce0a30d1ba7d8f9bbdd14a3

Change-Id: Ied7a73b662266eb1ea0c085e5b9c88df0de7e2f6

Add HEBREW POINT METEG to the Hebrew special characters

It's very common in the Bible, and in many other books for indicating stress.

Change-Id: I4a564df82398b15ea4ec9c0a67f119c7ef1bdf95

RevisionTest: Make @covers tags absolute

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

Change-Id: I92fadc51cc924477d884536b860272b5a230cd36

RevisionTest: Revision::constructFromRowArray is not a function

I'm guessing that RevisionStore::newMutableRevisionFromArray() is
what is supposed to be tested here, since Revision::__construct()
calls that if it gets an array.

Change-Id: Ia6c2e7eb667aa746a12ce74aad62e58d2468defd

Localisation updates from https://translatewiki.net.

Change-Id: I7b73fc1564496c5e6988fda30b4fccddea9d0e2a

Merge "Fix how "Live updates" behave when user logs out"

Merge "RCLFilters: change working of 'to-and-from' selector"

Merge "ApiMain: Fix call to Linker::makeHeadline()"

Merge "Fix heading selecting over-select"

ApiMain: Fix call to Linker::makeHeadline()

It's required that the $attribs parameter also contain the closing '>'.

Bug: T183125
Change-Id: Ic3015067a6b1e95e3487a8580e6a9aa8a6c280ed

[MCR] Turn Revision into a proxy to new code.

Change-Id: I4f24e7fbb683cb51f3fd8b250732bae9c7541ba2

Fix heading selecting over-select

In Chrome, multiple clicking selecting a heading cause a bug that the
first character in the next paragrpah is also selected.

Change-Id: Ib52662c39720dcd36ea0b60c9884530895dd9e6a

Localisation updates from https://translatewiki.net.

Change-Id: If167af95a237f67c19013cd5c1daf9c1cbdad492

Merge "Use another pageid in WikiPageDbTestBase::testInsertOn_idSpecified"

Merge "Fix tooltip accessibility for screen readers"

Fix tooltip accessibility for screen readers

By adding an additional aria-label attribute to the tooltip,
we can ensure that every tooltip will be accessible
for accessibility tools like screen readers.

Works with Echo extension, but I am not sure if there
are extensions who are using the original-title attribute.

Tested with ChromeVox on Chrome 62.0.3202.94.

Bug: T54711
Change-Id: I19500c4e8ccbdcb8288b9c2299a29b3f8a31639d

MergeHistory: Duplicate watchers on history merge

Bug: T132130
Change-Id: Ibb2be4f6f975f8f3c542689530fc120f8a464914

Use another pageid in WikiPageDbTestBase::testInsertOn_idSpecified

integer bigger than 2147483648 is bad on 32-bit php

1) WikiPageContentHandlerDbTest::testInsertOn_idSpecified
Failed asserting that -816015107 is identical to 3478952189.

> echo intval( 3478952189 );
-816015107

Change-Id: Ia2ab870dfe53bef873def3905ae93ea52d4ab2ad

Normalize path in AutoloadGenerator::setExcludePaths

Fix is for windows machine

Change-Id: Iadedeb6a04df439c0e940b8eb62fa46757c571c3

Merge "SqlBlobStoreTest: Use utf-8 as name in iconv"

Localisation updates from https://translatewiki.net.

Change-Id: I81ca4030d9c7e37c26828049921737fd0395fa52

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

Change-Id: I4beb1441790f5371d9fe55ca05cd17dc9fcdcf80

Merge "Fix additional usage of incorrect case"

SqlBlobStoreTest: Use utf-8 as name in iconv

utf8 is an alias that doesn't work with all PHP installations.

Change-Id: I6ce9c5b4ece1a8c263afb1d4895dacd3c790a51a

Fix typo in ImportHandleUnknownUser hook documentation

Change-Id: Ib37facf546e3088cc64373882bb276d2703575b4

RCLFilters: change working of 'to-and-from' selector

Bug: T172161
Change-Id: I749d7d4ba383daa79685da389d27bde832a508c2

Fix additional usage of incorrect case

Courtesy of the PhpStorm inspection Case mismatch in method call or
class usage.

Bug: T166759
Change-Id: I27c53658b99048fa0dd8f9d6ef1398620386e1cc

Merge "RevisionTest: Use utf-8 as name in iconv"

RevisionTest: Use utf-8 as name in iconv

utf8 is an alias that doesn't work with all PHP installations.

Change-Id: Ief6e5d8b73c9b4a6614430b699f3399b8137689a

Merge "Split limit report out of Parser::parse()"

Remove WatchedItem constants and methods deprecated in REL1_27

Remove associated tests

Bug: T182381
Change-Id: I0214250a7cb81518074a14e35590370b13d4ba64

Merge "ContentHandler: simplify tag tests"

ContentHandler: simplify tag tests

Follows up 62713be990.

Change-Id: Id1f7e6ae470b461e26ef8b7346f715873ccea4c9

Merge "Localise some special pages into Korean (ko)"

Localise some special pages into Korean (ko)

* Synced from MessageEn.php
* Change Special:ListGroupRights so it's less ambigious

Change-Id: I1dfac96825850cdcda135b2e1ce3e19cf71e99e9
Signed-off-by: Yongmin Hong <revi@pobox.com>

Fix @covers tags to use absolute class names in Storage tests

PHPUnit doesn't support use statements when evaluating @covers tags.

Change-Id: I7f649afd3e417a6b71b2f61c6ff59be5f404be8f

Merge "XMPReader: Log filename if available"

Merge "mw.rcfilters.ui.MenuSelectWidget: Always open this menu downwards"

OOjs UI: Backport Iad4a2fd1bd985b4924e5ef1f822e1ea81ce0a988

Bug: T182359
Bug: T182395
Change-Id: I764a96780e418830e5c92ec57c50f549951ff47f

Merge "[MCR] Tests for RevisionStore & Related classes"

Merge "[MCR] Break Revision into RevisionRecord and RevisionStore"

Merge "mediawiki.loader: Fix logError() not to print exceptions twice"

Merge "Add dropSequence to postgres"

Localisation updates from https://translatewiki.net.

Change-Id: I0e9482e99525f1fa6fd791f09597abd91485342b

mediawiki.loader: Fix logError() not to print exceptions twice

Current behaviour results in:

Chrome 65 (initial)
> ▶ [WARN] Error: foo Error: foo
>  at bar
>  at main

Chrome 65 (expanded)
> ▼ [WARN] Error: foo Error: foo
>            at bar
>            at main
>  at logError
>  at main

Safari 11 (initial)
> ▶ [WARN] Error: foo, Error: foo

Safari 11 (expanded)
> ▼ [WARN] Error: foo is undefined
>     * Error: foo is undefined
>         at bar
>         at main

Firefox 57
> [WARN] Error: foo | Error: foo
>                   | Stacktrace:
>                   |   bar
>                   |   main

I don't recall why I did it this way, but it seems redundant.
This commit effectively removes the first "Error: foo"

Change-Id: Idc5dee34ce2b5068e3a2d8800e2b5f6b879525d1

Avoid using array_map in ExtensionRegistry and ExtensionProcessor for simple operations

Bug: T182645
Change-Id: Ia98b1d798b1f93ca30686c009d479668616229ae

[MCR] Tests for RevisionStore & Related classes

Code introduced in:
I140f43a6fb443b38483f41f268c906b9cea64cf7

Change-Id: Iefad870baf2d16f12e9901b303246c64d6431ca6

Merge "Change EnqueueJob docs to discourage obsolete use-cases"

Change EnqueueJob docs to discourage obsolete use-cases

Bug: T181216
Change-Id: I9332d26ae9a74c3721cff9c497d5356f05efb428

RCFilters: Don't load JS or redirect when transcluding

We were changing the URL on every page that contained
{{Special:Recentchanges}}.

Bug: T181032
Change-Id: Ib4a0d3c1733da0bd0076aa43b91fc06939c51b75

Fix manual IN queries in checkStorage.php

Bug: T182689
Change-Id: Ifca7bd6d0385dd470cddc483f4641eeb2832580c

Merge "Track which namespaces edit conflicts are resolved in"

Merge "RCFilters: Change findHighlightedItem due to deprecation"

Merge "Convert includes/Feed.php to Mustache templates"

RCFilters: Change findHighlightedItem due to deprecation

SelectWidget's getHighlightedItem was deprecated and switched with
findHighlightedItem, see T76630

Change-Id: I90535c9f72d9a63e8e8d4184cc85aaf77e7b454c

[MCR] Break Revision into RevisionRecord and RevisionStore

Bug: T174025
Change-Id: I140f43a6fb443b38483f41f268c906b9cea64cf7