Merge "deferred: Improve DeferredUpdates test coverage"

Merge "shell: Deduplicate code in Command.php by combining else paths"

shell: Deduplicate code in Command.php by combining else paths

Also reduce indenting.

Change-Id: I33f83786c38bba0919372df0d5cdfa806d4361fc

deferred: Improve DeferredUpdates test coverage

From 1% of lines to 12% in deferred/.
From 6% of lines to 68% in DeferredUpdates.php.

* Adding relevant @covers tags to existing tests.
* Add coverage for MWCallableUpdate.
* Add coverage for MergeableUpdate.

Also:
* Make MergeableUpdate extend DeferrableUpdate.
  1. Because PHPUnit doesn't support implementing multiple interfaces
     in a mock, and would make the mock fail the typehint at run-time.
  2. It DeferredUpdates doesn't support having a MergeableUpdate that isn't
     itself a DeferrableUpdate given the only way to reach that code is past
     methods that are type-hinted with DeferrableUpdate.
  3. Making the interface extend DeferrableUpdate helps produce better and
     earlier errors. Instead of run-time error:
     > Argument 1 passed to addUpdate() must implement interface DeferrableUpdate
     > instance of MergeableUpdate given
     We get:
     > Fatal error: Class .. contains 1 abstract method and must therefore be
     > declared abstract or implement the remaining methods (doUpdate)

Change-Id: Ie384bf849a96bb37dc3e4a4154da2b02889e9fc8

Follow-up d25a944635: fix PHP notice

Change-Id: Id4d58f4eb2a26cb2f9cf6d36194f3e0ae9c84135

Merge "SpecialRunJobs: Remove unused JSON response"

Merge "Remove useless commit calls in JobRunner"

Remove useless commit calls in JobRunner

These were meant as sanity checks, but would fail in those
unusual cases anyway with exceptions. Instead, have an
early check to make sure no explicit transaction rounds
are active when JobRunner:run is called.

Change-Id: I723c77c8d3ef7ec4dcf09ce6d549b4fd57bdf1c2

Merge "Drop call of deprecated IE8-support method"

SpecialRunJobs: Remove unused JSON response

In preparation of making SpecialRunJobs a normal special page
that doesn't override output handling, but instead uses normal
HTML-based responses for sync, and DeferredUpdates for async,
start by removing the text/plain JSON response it currently
has (also to verify that it really isn't used).

Bug: T175146
Change-Id: I8621c4e2ba94a7d312adaf04a46e1a214af909fe

API: Handle pltitles/tltemplates/clcategories/imimages with only invalid titles

If the parameter contains only invalid titles, it shouldn't be ignored.

Bug: T162816
Change-Id: I3ee6aeab421db5732b652fed21292d8509f8d757

Drop call of deprecated IE8-support method

Bug: T123219
Change-Id: Ie85dd4442e93f1f1745227f5f3f9d7ddefcd2d46

Merge "Return stderr from Shell\Command"

Return stderr from Shell\Command

Change-Id: I5551ae4bbe7b539b528a734aa82198b11f103871

Merge "Fix inconsistent i18n escaping in category paging links."

Merge "Delete RevisionStorageTestContentHandlerUseDB"

Merge "RevisionStorageTest: use MediaWikiTestCase methods for setting globals"

Delete RevisionStorageTestContentHandlerUseDB

Due to this test file not ending in Test.php it doesn't
get run with all of the other phpunit tests.

When changing the name so that the test are run they fail:
https://gerrit.wikimedia.org/r/#/c/383587/
I119e80c6674b22dc3eb02a023a346f842ef8cd21

It is likely these tests have actually never run.

Rather than try to fix these 5 year olds tests lets just scrap
them and start fresh.

Change-Id: I0318614127d6b4d9661b875ab53af3f6e286ac95

Merge "RevisionStorageTest: Add tests for Revision::newFromTitle()"

RevisionStorageTest: use MediaWikiTestCase methods for setting globals

Change-Id: I8712517b50c5b552fe2d0e248b5735945a98fba4

RevisionStorageTest: Add tests for Revision::newFromTitle()

Change-Id: I131b033d72cd4775618b972dbaf40b9f28dd3374

Merge "Fix PostgreSQL patch-add-3d.sql by replacing it"

Merge "Have CommentStore limit the maximum character length of comments"

Fix PostgreSQL patch-add-3d.sql by replacing it

Follows-up 6260545fee, a2f5d05ae8. (T157348)

If updates are run for the first time on an installation that already
has '3D' in the enum (e.g. because it's a fresh install), the update
fails.

Instead of blindly running a patch file, we instead add a method that
checks whether the enum type already contains the value before adding
it.

Bug: T177417
Change-Id: Iad10cb88cf1cb35cfb95ce98a556b33688158a88

Hard deprecate wfShellExecDisabled()

No users anywhere, only ever had 1 caller outside of core.

Change-Id: I1361d3b555b642b75ec58f071a02407a611b44f9

Merge "Revision split __construct method"

Merge "Revision: test and fix __construct exceptions"

Revision split __construct method

This makes the logic much easier to follow and each type of
construction easier to see.
This also emphisises the testing of construction being split
into row object vs array

Change-Id: Ie3aa6ec4c026f0249ccd438903fec27fcd266b67

Revision: test and fix __construct exceptions

This adds tests for each exception that can be thrown
in the Revision constructor.
It also fixes where the exception for the content part of a row
not containing a content object is thrown.
Prior to this ->getModel() could be called on the content row
element before the check had actually occoured.

Change-Id: Ia2d2cfdca01871fc6dbb96707d781db33d7d0a40

Merge "RevisionStorageTest: code style fixes"

Merge "RevisionTest: refactor test dataProviders"

Merge "phpcs.xml: Remove missing functions from PrefixedGlobalFunctions 'ignoreList'"

RevisionStorageTest: code style fixes

Change-Id: I0d8e6e58c74c3ba849f9cb4a3a159a62049604a3

RevisionTest: refactor test dataProviders

Change-Id: I238d0294de7112a78ef18ffba195c0c175d6d735

Deprecate $wgEnableAPI and $wgEnableWriteAPI

$wgEnableAPI and $wgEnableWriteAPI are now deprecated and will be removed in
a future version. The API is now considered to be stable, secure and
essential.

Bug: T115414
Change-Id: I8c98ce63afd82677ecf662f8159e58be1759bde0

Merge "RCFilters: Store invert as a standard filter/parameter"

Fix inconsistent i18n escaping in category paging links.

prev-page and next-page were being escaped when they were a
link, but not when they were plain text. They should be escaped
in both cases.

This issue was discovered with an experimental phan plugin I'm
working on.

Change-Id: I6caac76299b21a54cd7cb732ad93187e3d37c99e

Localisation updates from https://translatewiki.net.

Change-Id: I2cb99cda301b6a76bc6ccb76819e4774d539910c

phpcs.xml: Remove missing functions from PrefixedGlobalFunctions 'ignoreList'

Some of these functions appear to have been removed or moved into
proper classes, so remove the unnecessary exceptions.

Also include a list of where these functions are defined.

Change-Id: I7d525f17dd52321475707d021f489857ce98894d

Merge "Make $mText an explicit member variable in TextContent"

Merge "RevisionTest code style fixes & file split"

Make $mText an explicit member variable in TextContent

Change-Id: I1e1f652e4ef6d02bc76105407cb23975625560cd

Fix unblocking autoblocks

Bug: T177952
Change-Id: Ib9c245fe47ed0009a8db8437798574ad4603a8a5

RevisionTest code style fixes & file split

Change-Id: I054a6810e29225e4341c518631a6dba9f40a1531

Merge "Setting Urdu translations of some magic words and special pages"

Setting Urdu translations of some magic words and special pages

Change-Id: I4ac5ac7e13c473adc0026776ef7d63b853a42976

mw.action.edit.styles: Remove max-width on edit summary field's label too

Bug: T162849
Change-Id: I37d483dba63b9694d67a9c99eb70ca892d8d8662

Merge "Change Tagged edits help icon"

Change Tagged edits help icon

- Change the link that help icon leads to.
- Reduce the size of help icon.

Bug: T176661
Change-Id: Ifc3b3e9efce00ea601795bd67a2de17c4be3cf6f

Fix SVG href error message to be more accurate and useful

Change-Id: I2519e1802a62e4d7c1d666b4325982063ff9b40c

Merge "Add test cases for digit grouping (commafy) in Polish"

Have CommentStore limit the maximum character length of comments

I92954c9 accidentally allowed comments, when CommentStore is enabled, to
be the full 65535 bytes available in the database field. This was never
intended.

There is not yet any consensus in T6714 or T6715 on just how long the
longer comments should be, for now we'll set 1000 because Tim says so.
Note this doesn't change the UI to actually allow more characters and
will only take effect once $wgCommentTableSchemaMigrationStage is raised
above MIGRATION_OLD.

Note this also doesn't make the limit configurable. That too can be done
later if needed, again along with whatever changes are necessary to the
UI to account for a variable limit.

Change-Id: I7e0c55619210ebab467436f0bb915271c0a7ccdc

RCFilters: Store invert as a standard filter/parameter

Change-Id: Ie623dac923cc2feb30f406c48accdadfd8d80753

Merge "Remove outdated code comments in MessagesXx.php files"

Merge "Ensure only strings are passed to mysql_real_escape_string()"

Add test cases for digit grouping (commafy) in Polish

According to the typographical convention, a thousands separator
should not be inserted in numbers that are four digits long (between
1000 and 9999), unlike in English where it's usually acceptable.
This logic is currently implemented in LanguagePl::commafy().

Bug: T177846
Change-Id: I6dbd8febcf59000067cdd7d3c11111f2f77f4e66

Remove outdated code comments in MessagesXx.php files

MessagesKk_arab.php:
MessagesKk_cyrl.php:
MessagesKk_latn.php:
MessagesRu.php:
* Translations of a comment warning end-users against editing
  localisation messages in these files. That comment was removed
  from  MessagesEn.php in b38247894422fb34dbddaea23cfc08cd72f48d5b,
  as we no longer store localisation messages here.

MessagesKsh.php
* Warning for translators not to use encodings other than UTF-8.

Change-Id: I1ff2cf035b84dc53c368ad7182f294c7fb8f1a3d

Localisation updates from https://translatewiki.net.

Change-Id: I07c8db3e1ae9bbccac12ab3d1fff365091b84794

Merge "Suppress error in MediaWiki\Shell\Command"

Merge "Define MEDIAWIKI_JOB_RUNNER in runJobs.php"

Merge "RCFilters UI: In the highlight code, use .data() the way it was meant to be used"

Merge "Special:UserLogin: Remove unnecessarily specificity from selectors"

Merge "Special:UserLogin: Set easier readable `line-height` on form"

Localisation updates from https://translatewiki.net.

Change-Id: I088f6eec0c374cb1d4d6cb54ac1468a971b9d53d

Unwrap types in function docs from {}

{} is used in javascript, not in php

Used @(?:param|return|var|throws) \{[^\}]+\} to find&replace

Change-Id: I2e0c9c64b6528f3899d2809d585b5e4859838007

Define MEDIAWIKI_JOB_RUNNER in runJobs.php

This is the same value used by the WMF /rpc entrypoint.

Change-Id: I146a36c76ff2136e5fa9b768673987ebd611403c

Merge "SearchInputWidget: Actually connect to change event"

Merge "SearchInputWidget: Pass through description data"

Merge "Remove setting of SearchInputWidget configs to already-default values"

Merge "Align jQuery.suggestions colors to WikimediaUI palette"

SearchInputWidget: Actually connect to change event

This makes the clear indicator-button work, not sure if it
ever worked in the past.

Change-Id: Ia54add0fa5475c37871d82e7750ea4dedbf9dfe3

SearchInputWidget: Pass through description data

Also make result-parsing code clearer.

Change-Id: Ibf90ba3fc330f442606f243855721182701f7c02

Suppress error in MediaWiki\Shell\Command

Command uses a certain error message to detect and ignore
EINTR in stream_select, and uses trigger_error to clear
the message from get_last_error (clear_last_error is PHP7 only).
This works rather poorly with a system config that does not catch
or ignore most errors; specifically it breaks database tests
on Vagrant with the warnings_as_errors role on.

Change-Id: I9c8f922bc0a8f5ee6b8e7501b22223cce4f98ecb

Remove setting of SearchInputWidget configs to already-default values

Change-Id: I2710051386ba826895db9d9c675263847625e17f

Special:UserLogin: Remove unnecessarily specificity from selectors

Removing unnecessary tag selectors from compounded selectors.

Change-Id: Ib180ac6d5b63af587dd2cdacb66cd3889de086bd

Special:UserLogin: Set easier readable `line-height` on form

Change-Id: Ie29867a6b0ac0ba1558e1b302ffc6df8bff1dba9

Special:UserLogin: Align text to WikimediaUI color palette

Also removing unnecessary CSS `top` property.

Bug: T152025
Change-Id: I8945b7c610691de63eaadc465036833d851d1953

Ensure only strings are passed to mysql_real_escape_string()

Under some conditions (Semantic MediaWiki, Gadgets), an integer is
passed to DatabaseMysqli::mysqlRealEscapeString (). This integer is, in
turn, passed to mysqli::real_escape_string (), which needs a string.

Under HHVM 3.19.1 (at least) this type mismatch causes an exception.

A typecast should prevent it.

I repeated the patch in other DB drivers where I could find a function
that escaped strings for SQL.

Bug: T163646
Change-Id: I1b7820bc064dc79498cf9f17747f745990c526b7

Merge "Assert that blocks were inserted successfully in UserTest"

Merge "SearchInputWidget: Don't show images"

Merge "Get rid of $IP in Command"

Merge "JSONContent: Align to WikimediaUI color palette"

Localisation updates from https://translatewiki.net.

Change-Id: I4a05d91ddd309bf50c39fdea3e99dba312633afd

JSONContent: Align to WikimediaUI color palette

Aligning JSONContent colors to WikimediaUI palette. There's no
definition for a color like `.mw-json .value` yet, we'll probably
further amend in future.

Bug: T152025
Change-Id: I09ddabf52c8d41ade0eca805f73f2c7bc5ade989

Align jQuery.suggestions colors to WikimediaUI palette

For code sanity in core and consistency.

Change-Id: I8bcfd0f6d2feebe803a56d69d3423dc4d13714f7

Fix typo in 'language'

Change-Id: I3c4d090640892ae07d3da33dcfe3ace397a40808

Merge "HTMLForms: Reduce `padding` on top as heading `line-height provides"

Merge "HTMLForm: Mangle excessive whitespace"

HTMLForms: Reduce `padding` on top as heading `line-height provides

Reducing `padding` on top of `.mw-htmlform-ooui-wrapper`
as heading's `line-height provides similar distance.

Bug: T177668
Change-Id: Ieb4c0d1a08f9b7f3481a64ada1638c98f53bc90a

Get rid of $IP in Command

Change-Id: Iccfe1b79963462f9cad80ff327ccd574ee1122c5

installer: Reuse $db->makeList() rather than reinventing the wheel

Change-Id: Ib1f896e5bed17bc240fe585fd6d5f753995a12e1

HTMLForm: Mangle excessive whitespace

Addressing excessive whitespace in OOUIfied HTMLForms by
- setting form `line-height` to `1.4` in order to
  override core and
- explicitely setting header label `line-height`
  as well.

Bug: T177668
Depends-on: I6463bb204c0000d89fd12a39c4f3c571be0ffc37
Change-Id: I8f503dd5305aef856c6b842c0fdd95522871dca3

Merge "HTMLForm: Rename file to `ooui.styles.less` and use LESS notation"

Merge "installer: Add comments to purgeCache()"

Merge "docs/uidesign/monospace.html: Add textareas to the demo"

Merge "resourceloader: Clarify operator precedence"

docs/uidesign/monospace.html: Add textareas to the demo

It seems that textareas have their own font-size set in Chrome (but not
Firefox and other browsers), making them unaffected by this behavior.

Change-Id: I811245ea26b4d90ad252fe8f6d2e096e27cb9073

resourceloader: Clarify operator precedence

Change-Id: I478db046a1cc162c6767003ce45c9b56270f3372

HTMLForm: Rename file to `ooui.styles.less` and use LESS notation

Turning ooui.styles into a LESS files and take advantage of LESS
notation. This is in preparation for mangling whitespace of
OOUI Special:Pages forms.

Bug: T177668
Change-Id: Icd955358dbbf50712d8a66a0325aa14c8e7963d2

installer: Add comments to purgeCache()

Change-Id: I791f220b49e3e1c7d6df9ccab7d70ff8a0031ef1