Merge "Add meta=userinfo&uiprop=latestcontrib"

Update OOUI to v0.31.3

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

Bug: T193731
Bug: T194523
Bug: T217381
Bug: T218408
Bug: T219033
Bug: T219826
Depends-On: I94d1fd4398ec92c2ec63ebf1e57b3b4fc72bd76a
Change-Id: Ibb276860815fc29beaea131401bb5dfeca5159d6

Merge "Use HTTP method as hint for the locking mode of sqlite"

foreign-resources.yaml: Add TODOs for the four remaining unverified libraries

Change-Id: I493415f89745eddf4a64ae23aced586028cc16aa

Avoid hitting DB in Category getters when they have the required info

Bug: T63045
Change-Id: Ib881ce263fa7a4a8256f627a56b774aaa815d3df

Merge "Collapse some nested if statements"

Collapse some nested if statements

Change-Id: I9a97325d738d09370d29d35d5254bc0dadc57ff4

Merge "resourceloader: Fix typo in comment to make more sense"

Merge "resourceloader: Fix return type doc for getSelectors() method"

Merge "jobqueue: dependency inject more objects into JobQueue"

resourceloader: Fix typo in comment to make more sense

Change-Id: I5d519fcded529d022461a040efe0fe17e7c1c589

Merge "Add &$tags argument to MarkPatrolled hook"

Merge "Fix wpCreateAccount bug when set on Special:Block url params"

resourceloader: Fix return type doc for getSelectors() method

So I've checked and this method should return an array and this array
specifically is an array of strings (string[]). In addition, remove an
irrelevant semicolon after foreach().

Change-Id: Id7dffe882583a7555d87a5b9c409584478c30a8c

Use HTTP method as hint for the locking mode of sqlite

Bug: T93097
Change-Id: I4aa44345e2f9e252c1a351f252bdcb9e15f180a5

jobqueue: dependency inject more objects into JobQueue

Also moved some WikiMap/$wgJobClasses checks to JobQueueGroup::pop
which is the method callers are supposed to use.

Change-Id: I2ab82d8adc4ae1f54697d2935afa2053539cf2db

Merge "tests: enable MariaDB/MySQL strict mode"

Localisation updates from https://translatewiki.net.

Change-Id: I41a457d73c94bab3dd7a1104d513aa9598aba4e3

Merge "phpunit: Call 'teardownTestDB' from shutdown instead of destruct."

Merge "Define a default Monolog logger if inexistant"

Define a default Monolog logger if inexistant

In the array 'loggers' the key '@default' is assumed to be present,
hence be sure it is defined: this is the system default for MediaWiki,
it can be overwritten if desired.

This default php://stderr with level ERROR is inspired from Monolog’s
own default, which is php://stderr with level DEBUG: this is similar
but less violent for the average MediaWiki sysadmin.

Bug: T196906
Change-Id: Id74083fc20ddf906c40e6d149095e8ade4c68572

Remove `background-size()` LESS mixin.

It is not useful any more, as it only applies to Android 2 and
got deprecated in 1.27.

Bug: T219956
Change-Id: If5174704c2b50ce45b6a02e1b53691df380c6f3c

Merge "Check line length of HISTORY"

Merge "ChangesList: Comment out newlines in EnhancedChangesListGroup.mustache"

Merge "protect.js: Use event 'input' instead of 'keyup'"

Merge "protect.js: Simplify by using jQuery"

Merge "protect.js: Reorder function declararion to avoid forward reference"

Merge "protect.js: Simplify by removing object ProtectionForm"

Merge "protect.js: Remove JavaScript global variable window.ProtectionForm"

Merge "Language: Use type hint instead of manual type check"

protect.js: Use event 'input' instead of 'keyup'

The event 'input' is already triggered when a character gets inserted.
The event 'keyup' gets only triggered when the key gets released.

* https://developer.mozilla.org/en-US/docs/Web/Events/input
* https://developer.mozilla.org/en-US/docs/Web/Events/keyup

Change-Id: If1990af8304e4941565367ab558e1f870480cea3

Merge "LogFormatter: ignore unrecoverable data"

Merge "Remove `.background-size()` mixin usage"

Fix wpCreateAccount bug when set on Special:Block url params

wpCreateAccount default is dynamically changed if the user
has not posted the form or interacted with the field in any way

Bug: T218554
Change-Id: Ied68dc60d57ed35ed3285c5cf89b772c8c4e41d1

Set synchronous = NORMAL for cache tables in Sqlite installer

In local testing, this makes SqlBagOStuff::set 17x faster.

Change-Id: I54d0d4f009ca1289485b89a1dcab46ffb31d8f11

Merge "ServiceWiring: Use RequestContext instead of $wgUser global"

Merge "installer: make sqlite installer move the job queue to another DB"

Merge "Add Job::getMetadata() and Job::setMetadata() accessors"

Merge "jobqueue: remove entire unused JobQueueAggregator class hierarchy"

Language: Use type hint instead of manual type check

Change-Id: I9c68d0838d5865c29bff85b9a6bf50e02e83f556

Add Job::getMetadata() and Job::setMetadata() accessors

Change-Id: I3a97008d324f600a1c9f6005673073277ee564fa

Fix comment typo in fresnel config file and remove an unnecessary comma

Change-Id: I43123d9c001ea59f38f904cd232cb5f8a11e15d8

LogFormatter: ignore unrecoverable data

It is possible for the log_params column of the logging
table to contain serialized data that cannot be deserialized
anymore because the types it references are missing.

It is currently the case with old Flow log entries on enwiki.
The extension is uninstalled and the UUID class is not found.

This patch proposes to simply skip the params and log a
warning in that case.

Bug: T212742
Change-Id: I3226b8fb338dd2b81e087af5d798d8f35368282d

Merge "MigrateActors: Improve query for log_search rows"

ChangesList: Comment out newlines in EnhancedChangesListGroup.mustache

This is a follow-up to 9c5a4773952f2e914eda668b71dedb7c78a9a258.

This change removes all spaces and newlines inside of the <td> elements
by a Mustache comment. This prevents that the newline get converted to a
<p> element when the recent changes get included by

  {{Special:RecentChanges|enhanced=1}}

Bug: T215737
Change-Id: I179d0b36b122efc3e1b7c193354ad03c3c9cefcf

protect.js: Simplify by using jQuery

Also remove useless boolean return.

Change-Id: I5a1ee977c2e7548bab98f8a7b6731262d175db51

Remove `.background-size()` mixin usage

Removing deprecated LESS `.background-size()` mixin calls.

Bug: T219956
Change-Id: I48a939eae125802778d0a8822961bef826970f46

Merge "RecentChanges updated to use pseudo elements for presentation"

Merge "Fully utilize LinkTarget passed to getRevisionByTitle"

RecentChanges updated to use pseudo elements for presentation

Bug: T219348
Change-Id: I6eeeaa3b58d37adb7fefb4cc6915022229b3b324

Merge "jobqueue: change MWException throw statements to using JobQueueError"

Fully utilize LinkTarget passed to getRevisionByTitle

Failure of getRevisionByTitle to pass its LinkTarget argument
to newRevisionFromRow resulted in a needless second instantiation
of the Title (an extra query).

Because newRevisionFromRow needs a Title, not just a LinkTarget,
it is unfortunately necessary to call Title::newFromLinkTarget
on it for now -- however this does not involve a DB lookup and
is on track to be fixed with revisions to the Title class.

Bug: T206498
Change-Id: Ic6f98d8fbf66d85121668571c17e148efc5ec2be

Merge "Style change tags correctly on log extracts"

Merge "Disable expensive {{REVISIONID}} magic word in miser mode"

Check line length of HISTORY

Slight modifications, almost all for URLs.

Change-Id: I612852e9a677a6a14fd48e2ccd7fcb998da9152a

Style change tags correctly on log extracts

You can see them when you view a previously deleted page
and some of those events have been tagged.

Bug: T212613
Change-Id: I88e637a0ee688dd23c0ec90790e8189ecf1a8789

Merge "Add test for Special:page::buildPrevNextNavigation"

Merge "maintenance: Rewrite parts of the purgeChangedPages script"

Merge "Sunsetting viewPrevNext"

Merge "Fix typo in MigrateActors.php"

MigrateActors: Improve query for log_search rows

The query in question is, generically,

 SELECT ls_value, ls_log_id, actor_id
  FROM log_search LEFT JOIN actor ON (ls_value = actor_user)
  WHERE ls_field = 'target_author_id'
  ORDER BY ls_value, ls_log_id
  LIMIT 100;

The intention is that it'll pull out 100 rows from log_search using its
primary key, and for each of those 100 rows find the actor_id for the
referenced user_id (using the actor_user index) if an actor ID exists.

The twist comes in the fact that ls_value is a string-type column while
actor_user is an integer-type. MySQL doesn't usually care, but other DBs
do so we have to cast one into the other.

Currently the code is casting actor_user to a string. But that means the
DB can't use the index on actor_user to find the one matching row,
instead it needs to scan the whole table.

The fix is simple enough: instead of casting actor_user to a string,
cast ls_value to an integer. That allows the actor_user index to be used
as expected.

Bug: T215525
Change-Id: I2f7a6ba9fd537336594088a0281a62ea5601cd59

Merge "Set default for Block::appliesToPasswordReset to true"

Fix typo in MigrateActors.php

Change-Id: Ic08210a8cd394f6ad49673a2d2e4800e6bcf2989

Set default for Block::appliesToPasswordReset to true

Follow-up to Ia3f46aba. Before that patch, the default case for
whether a block blocks password reset was true.

Bug: T218905
Change-Id: I86cb381762890abddcf32f4f7199799391cc6dfb

protect.js: Reorder function declararion to avoid forward reference

The ESLint rule exception no-use-before-define can removed.

Change-Id: Ib730102ae33f37ff95b001d689666f729885cd2d

Add test for Special:page::buildPrevNextNavigation

Change-Id: Ie17221305ec979acebebb62798b83f5d91df1f60

protect.js: Simplify by removing object ProtectionForm

Change-Id: If7060c41c9b904e174a34dfc7ea2c053ecc1c3fc

protect.js: Remove JavaScript global variable window.ProtectionForm

The JavaScript global variable window.ProtectionForm is not needed since
c805e14eec7b.

Bug: T35871
Change-Id: I9089640d6c0058da14ba179fba9656fc7a4f5c68

Merge "maintenance/dev: Clean up router.php"

monolog: add MwlogHandler

Introduce a new handler, similar to SyslogHandler, that will be used to
replace udp2log. The main feature of the handler is being able to vary
the 'application name' with each record's channel. This way the channel
can be reconstructed later, e.g. when writing to plaintext files.

See also an overview of the plan here:
https://phabricator.wikimedia.org/T205856#4957430

Bug: T126989
Change-Id: I0c040825b31cd70f197dc8c1d406a103dc0ed1d1

maintenance: Rewrite parts of the purgeChangedPages script

My motivation to touch this code was a few count() I found confusing.
I ended rearranging it a lot to improve readability and clarity.

Change-Id: I9a0ea0812d4cc0c6e8ed6134e462dfdf325fd5d9

Merge "rdbms: make $wgSQLMode work as documented when it is NULL"

Merge "Fix unused vars/pointless assignments"

maintenance/dev: Clean up router.php

* Remove code paths that were unreachable for me.

* Remove display_errors/error_reporting overrides.
  These can be set as-needed from LocalSettings.php if desired,
  and similar (but slightly different) settings are already set
  by DevelopmentSettings.php.

* Default to text/plain instead of letting PHP handle it.
  This makes makes the result more predictable, instead of varying
  by PHP version and underlying OS behaviour. Anything we want to
  support is already in our 'mime.types' file, and ours is a
  superset of PHP's anyway.

Change-Id: I46608ebd8e225642cfeb804ec32739c2fcbd4d25

Merge "languages: Replace loop with array_pad()"

PreferencesFormOOUI: Restore ID attributes for each fieldset

Follow-up to 255a58bedd29491c5550eebe4b8d0a5968f0d56c.

Change-Id: I00f00576ffc7f9d13218b21649822e0f826ecdf9

jobqueue: fix DBO_TRX logic in JobQueueDB for avoiding transactions

Various methods were missing the flag setting logic and tests could
fail or have "outer scope" warnings in the logs for sqlite.

Change-Id: Ia0607d189a307667297f06109a34363c92e37d92

Merge "Update help icon from OOUI"

Merge "Change job table params from blob to mediumblob"

tests: enable MariaDB/MySQL strict mode

Pass sql_mode=TRADITIONAL to the SQL connection which would raise an
error instead of warning, for example when a field is truncated.

This change will most probably cause some extensions test to start
magically failling.

Bug: T119371
Change-Id: I2051952090e48876f629690981e25603eff79f67

rdbms: make $wgSQLMode work as documented when it is NULL

NULL means that the SET query should not be sent.
This was broken since 5bc9b990acd0e0302f.

Change-Id: I148b1153ef6ecca99c56e4f63247820a766ae779

ServiceWiring: Use RequestContext instead of $wgUser global

Bug: T159299
Change-Id: I2a52e9b002a93a869355f89247153b09527dc118

Merge "MediaResultWidget: Remove unused code depending on VisualEditor"

DateInputWidget: Fix focus reflow

Second attempt, follow-up to I1b5beee6553. This time with correct
specificity.

Change-Id: Ifeefe668c6d50b9452f6702dc632d67439a68135

Localisation updates from https://translatewiki.net.

Change-Id: I167dae8a2111bbad1f1656d6edfa41555e854a9f

Sunsetting viewPrevNext

Removing viewPrevNext from Language and moving it to SpecialPage.
Used soley in special pages, and we aim to remove/reduce the dependency
of messages and language

Bug:T207977
Change-Id: I49b41a89ba59cfc24982b321f02c5cca9939decd

Update help icon from OOUI

The normal 'help' icon no longer has a ring outline,
so use 'helpNotice' as this looks less confusing in
an unframed button.

Bug: T219719
Change-Id: I6af23236e9d3983cab6d2ce331dd67e3b42913ff

Merge "Fix flaky test BlockListPagerTest::testFormatValue"

Merge "i18n: Add line-break in email notificaton for minor edits"

Change job table params from blob to mediumblob

The default blob type holds 65kB which is often enough, but some
jobs push over this limit. When using mysql these are silently
truncated to 65kB which only issues errors when the data is retrieved.

Increase from blob to mediumblob which gives a max size of 16MB. There
may be some valid use cases for jobs > 64kB, but there should be no
use case for jobs > 16MB.

Bug: T124196
Change-Id: I6df6634bd3a5b5fa27ce6f8ff20d251a9b5eba32

Fix flaky test BlockListPagerTest::testFormatValue

The test relied on being executed within the same minute as when it starts
which is not always the case.

Bug: T219695
Change-Id: I99eb0d25138069ca08c2975ff2e60c7f1df0a99f

MediaResultWidget: Remove unused code depending on VisualEditor

It seems that this property is not actually used. The default thumb size
is also loaded in MediaResourceProvider by querying the meta=siteinfo API.

Bug: T219675
Change-Id: I42610a2ac1ba6b19f08ee9fc9a8fd4aa23e090a7

linkeddata: Simplity PageDataRequestHandler test setup

Most notably this gets rid of all regular expressions and replaces them
with straight string comparisons. This is mostly done for readability,
but also makes the test more independend from the setup of the server
that is running the test.

Change-Id: I8c34cf1756c8e6c126b95b3eda163561d53186b9

linkeddata: Fix broken check in PageDataRequestHandler

First I noticed a weird check that assumes that explode() would return
an integer. But it returns an array of strings. There is obviously a
count() missing.

Then I started simplifying the code:
* The count can only be 2 or 1, never anything else.
* If it is one, the first element in the array contains the original string.
* The empty string is already checked above and can never end down there.
* Finally I made the remaining `if ( … ) return true else return false` a
  straight `return …`.

Change-Id: I289144e64f449ee0875009aaa22e10a5c0eb2734

Add &$tags argument to MarkPatrolled hook

This will allow the OAuth extension to subscribe to this hook and add an
“OAuth CID: $consumerId” tag to patrols made via OAuth.

Bug: T219655
Change-Id: Ie5e6f820bbf399ec639e715afd908f78bf5c8e9a

Merge "Don't return a result of a void function"

Remove redundant parentheses from the "Historyempty" message

With pseudo-elements now being used, the parentheses in the "Historyempty"
message become redundant, and should be removed to avoid showing "((empty))"
on empty revisions in page histories.

Bug: T219714
Change-Id: I8aacd6ede4ab0cc03b00fb891088ef0c847a6b20

Merge "MediaResultWidget: Fix defaultThumbSzie config"

MediaResultWidget: Fix defaultThumbSzie config

Bug: T219675
Change-Id: I38bc34f6d44460ec59881b5e968dc8cf27d75fa3

Localisation updates from https://translatewiki.net.

Change-Id: I99e4d02a0dce78e9ac24a3a6040183bec5a56d60