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"

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"

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

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

Merge "maintenance: Deprecate Maintenance::hasArg/getArg with no param"

Move MediaTransformOutput classes to own files

Change-Id: I1c676f18101250ffc28ea412213d568c5a4f527d

Merge "Move class SearchEngineDummy to own file"

Merge "resourceloader: Remove deprecated MessageBlobStore::getBlob"

jobqueue: change MWException throw statements to using JobQueueError

Also fix some minor phpstorm warnings in JobQueue

Change-Id: Ie4970f3e93b504fae509ec755b0e0afad71a7749

Merge "Do not return invalid JSON in runJobs.php"

Do not return invalid JSON in runJobs.php

Change-Id: Ib94c23bfc5e599a628a5fd65bb14baf750874995

jobqueue: remove entire unused JobQueueAggregator class hierarchy

Change-Id: I3d56d5db54fcb4278c7cf2a8d08933c27773c3f7

resourceloader: Remove deprecated MessageBlobStore::getBlob

This class is basically `@internal`, but not yet marked as such.
The only non-core use case is a WikimediaMaintenance script,
so I suppose we can keep it public.

It's purpose is for batching, the single blob method is not
used anywhere in Wikimedia Git.

Change-Id: If2a0497b1412efbfd55b34b512d2b9e5346359fb

Merge "auth: Make 'LocalUserCreated' hook non-abortable"

Merge "mediawiki.user: Use then() instead of done()"

Merge "mediawiki.user: Improve test suite"

Merge "Avoid use of deprecated wikiId field in JobQueue"

Merge "Avoid use of deprecated "wikiId" parameter for FileBackend"

Merge "Fix function/class case"

Merge "Replace several uses of wfWikiId() with WikiMap methods"

Merge "Make RefreshLinksJob run DataUpdates such that they have outer scope"

Merge "Clean up some database domain ID related code and fix some comments"

Fix unused vars/pointless assignments

Change-Id: If475c738b4af7208024c866594d4c0048af053dd

Make RefreshLinksJob run DataUpdates such that they have outer scope

This is how DeferrableUpdates are supposed to run as documented.

Change-Id: Ie4fb8c9a64b94bc2422d7e51391998315f4eeddb

Don't return a result of a void function

Change-Id: Ie22b0a44a489bb2f1115608c8dd782656b45537e

Merge "Fix partially blocked admin undelete permissions"

Fix function/class case

Change-Id: I18c205736be9d76d2c09a6dc4d7377740a742ccc

Merge "Make Job::hasExecutionFlag() actually work"