Collapse some nested if statements

Change-Id: I9a97325d738d09370d29d35d5254bc0dadc57ff4

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"

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

Merge "MigrateActors: Improve query for log_search rows"

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"

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

Add test for Special:page::buildPrevNextNavigation

Change-Id: Ie17221305ec979acebebb62798b83f5d91df1f60

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"

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

Also update some of the other separate DBs/files to use IMMEDIATE

Bug: T93097
Change-Id: Ib250b74cb7ff743f155512d7410cb229441cc70a

Avoid use of deprecated wikiId field in JobQueue

Change-Id: I9e30519e7985c20525572b685234ad239d62c214

Avoid use of deprecated "wikiId" parameter for FileBackend

Also cleaned up the parameter/field naming for DBFileJournal

Change-Id: I70f2a870706f0813311a694cb8ea31ced58dcabb

Merge "Rename NameTableStore wiki ID field to a DB domain field"

Merge "Use the WebRequest::getCheck() shortcut where possible"

Merge "Fix flakey MCR test in RevisionDbTestBase by mocking the time"

Replace several uses of wfWikiId() with WikiMap methods

Also, reword some related comments.

Change-Id: I8aefa42f96b90e9ea871cf7ec1afcc7ed82a19b2

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

Change-Id: Id5c5640e05301161dbb7c2be363e32d7d1c9a511

Merge "jobqueue: simplify the signature of Job::factory() and Job::__construct()"

Fix flakey MCR test in RevisionDbTestBase by mocking the time

Change-Id: I1d089d7b155adb865b42f7e8a26466553508c9be

Fix partially blocked admin undelete permissions

If an admin is partially blocked they should be able to restore/undelete
any given page.

Bug: T219305
Change-Id: I404730273c33d72d7552cd69e93ccec78c5fad5a

Merge "Convert EnqueueJob to using DB domains"

Merge "Make WikiMap::isCurrentWikiDbDomain() more rigorous"

Merge "Fix invalid namespace restriction when js is disabled"

jobqueue: simplify the signature of Job::factory() and Job::__construct()

Remove the $title argument from these methods to simplify subclasses that
do not have a meaningful title to use. The Job::getTitle() method can be
overriden by subclasses to return something meaningful.

The old call signature is still supported for backwards compatibility.
This will automatically determine what getTitle() returns as before.

Use "Blankpage" as the "not applicable" title for jobs instead of one
that looks like some error occured.

Change-Id: I3d5bd012d9cef1e7daaccfb0d5d319552eb89fb6

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

Makes it compatible with standard ES6 Promise, and also
allows the browser to respond sooner to user input (if there was
any) rather than (synchronously) continuing within this same async
block.

Change-Id: I82aac439cecd7680f53a82649196c9168eda6755

mediawiki.user: Improve test suite

* Add missing test for getRights with Promise.

* Separate the 'callback' and 'Promise' test cases.

Change-Id: Ia332f05e117512f32f71ff2e22bd76f8f1ee101c

Move class SearchEngineDummy to own file

Each class should have its own file, even it is a no-op

Change-Id: I90124ed35e354c546fcd69d52dc1febb71b2c71f

Unify code comments across special page stylesheets

Change-Id: I6311b441552bfd05f737f106b534f213797d426a

Fix invalid namespace restriction when js is disabled

When JS is disabled on Special:Block you were able to specify
any namespace as part of the namespace restrictions field. This patch
rejects any virtual namespace submitted on that field

Bug: T216831
Change-Id: If09c43e5f836e1fdd9438b856d7f44f434c29fe1

Make WikiMap::isCurrentWikiDbDomain() more rigorous

This already requires a DB domain ID, so there is no reason to have
hacks for trying to handle a wiki ID being passed in instead. If the
provided domain has a schema, it should not simply be ignored in the
comparison.

Change-Id: I9ced7a46fa05f32843a9a7d17391c5d0576b099c