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

Skin: Make skins aware of their registered skin name

Remove the need for skin classes to have a hardcoded string as
skinname property value. This previously created the possibility
for the value to not match the skinname in the SkinFactory registry,
which creates confusing situations where message keys and load.php
urls are crafted with the internal skinname, but all other
handling (useskin, preferences, hooks, SkinFactory, ResourceLoader,
etc.) operate on the names in the registry.

We could enforce the matching by requiring a 1:1 relationship between
skinnames and Skin sub classes, but that is not backwards-compatible
with the 1:many map that wgValidSkinNames provides, and not compatible
SkinFactory either, which supports a factory function to return an
object. This makes a lot of sense and allows Skin-classees to be
re-used and composed with injection. If we do want to enforce 1:1,
we could validate it with a structure PHPUnit test, but instead this
change just uses the injected name from the constructor (passed by
ServiceWiring, previously unused).

The added unit test shows the new behaviour. Before this change,
getSkinName() on SkinFallback would always return 'fallback',
whereas now each instance of the class adheres to the registered
name (if it differs from the default).

Update the two direct uses of protected $skin->skinname to use
$skin->getSkinName() instead to enable sub-classes to optionally
implement an alternate source for the self-name (or to hardcode
it there as before).

Bug: T173546
Change-Id: I4383dcc3094da6e3c9ac12dc6c9311128db9db6e

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

SearchInputWidget: Don't show images

But we should also have a different class name for this, as this isn't
upstream's SearchInputWidget.

Bug: T177659
Change-Id: Ie62e8678e89b2403d769694adb23fe21a047019c

Localisation updates from https://translatewiki.net.

Change-Id: Id60644489cb7332c77becff126ef97b4643f83e2

Clarify purgeExpiredUserrights maintenance script

This moves an @since tag where it belongs (class-level instead of
file-level comment) and removes a wrong message (purges are run directly
after execute() ended and will have immediate effect).

Follows-up on fdb4557.

Change-Id: I26fdb1c147b9b44690a3dd0f7c836aca85430c5e

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

Use .data( 'highlightedFilters' ) instead of .attr( 'data-highlightedFilters' ),
and make the value an array rather than a string that we have to convert
back and forth every time. We have to use .attr() for data-color because it's
used by CSS, but data-highlightedFilters isn't used by anything else.

Using an array makes the code much simpler. Also add the class
mw-rcfilters-highlighted to every highlighted item, so we can find
them back easily and don't have to use $( '[data-highlightedFilters]' )
(attribute selectors are slow, and this wouldn't work with .data() anyway).

Bonus: use the comma-separator message

Change-Id: I04de7f8fb74d60fb23ef47bf50bacfeb176a55b1

Merge "HTMLMultiSelectField: Support sections in OOUI mode"

Merge "Add maintenance script: Clean up expired userrights"

jquery.ui: Fix various jQuery Migrate warnings

These are the most common three, triggered whenever a jQuery UI
dialog is created.

> JQMIGRATE: jQuery.fn.bind() is deprecated
> JQMIGRATE: jQuery.expr.filters is deprecated; use jQuery.expr.pseudos
> JQMIGRATE: jQuery.fn.andSelf() was removed, use jQuery.fn.addBack()

Bug: T169385
Change-Id: I717f2580e3aae443cb9fc734de8f4b0dd8825b8c

Localisation updates from https://translatewiki.net.

Change-Id: I62a90758714d96ddcd91f8959b2e019aa72cfb9b

jquery-migrate: Disable console.trace() call by default

They are slow and spammy, especially given in modern browsers
console.warn() already provides a trace (and one that is collapsed
by default in Chrome, which seems preferred).

Change-Id: I94d3c7bf6deb47c2f3f660dfd2298ee293e7d825

jquery-migrate: Remove redundant PATCH note for Deferred bug

This patch was merged upstream and is no longer a local patch
since 2e2ac756e7224 upgraded us to jQuery Migrate 3.0.1.
I mistakenly kept this comment when re-applying our patches.

Change-Id: Id5fa8c03bc5791971a6406376ca8abf5503268f6

Add maintenance script: Clean up expired userrights

This adds a maintenance script that removes all expired user group
entries from the user_groups table and adds them to the
user_former_groups table.

Bug: T176754
Change-Id: Ie0f4afd3dbecd3af42e89bb99cf03dbe03768d07

Merge "Refactor global function wfBCP47 to static function LanguageCode::bcp47"

Merge "Revert "Remove some remaining mentions of unstubbing from core""

Revert "Remove some remaining mentions of unstubbing from core"

This reverts commit 0f91e75c20e040f0e688fb05654ba7797afcff76.

Bug: T177478
Change-Id: I1937bd116d22479c38a9a34714c2090e2a4c0d7e

Merge "RCFilters: Load default saved query if it exists"

RCFilters: Load default saved query if it exists

Bug: T166908
Change-Id: Ic3931bc0d67b340db11ed100aad836e8d867fa56

Revert "Stop stubbing StubUserLang"

This reverts commit 7ab57ba290f670847f981d5fc2c79339f1d4844e.

Bug: T177478
Change-Id: I28ac95ebcb64231a12c178165a0cb174e70f4e18

Merge "Use Special:PageData as the canonical capitalization"

Merge "Fix inability of fetching message object"

Use Special:PageData as the canonical capitalization

We want Special:PageData (uppercase D) to be the proper page title and
Special:Pagedata (lowercase d) to redirect to it, not vice versa.
(Special page names are case insensitive, so it’s not necessary to add
the old name Special:Pagedata as an explicit alias.)

Note: if a localisation cache is configured, it needs to be rebuilt
after this change.

Bug: T177411
Change-Id: I27c47a16326540a74d0850467d430faaa25369df

Refactor global function wfBCP47 to static function LanguageCode::bcp47

Deprecate global function wfBCP47.

Change-Id: Ie6bb061b5d6ca67289bb18bc468a87421f38fc94

Merge "Allow duplicate articles in parser tests with same content"

Merge "CLI: Make sure we don't exit with 0 when an exception is encountered"

Merge "Hard deprecate wfInitShellLocale()"

Merge "Make Database::rollback() also suppress callback errors"

Merge "TitleWidget: Perform diacritic-insensitive highlighting"

Hard deprecate wfInitShellLocale()

Soft deprecated since 1.30, no callers anywhere.

Change-Id: Ifec11eb3c2c25ea9d7e5ecb72c53127536ab6c30

Merge "mediawiki.notification: Move offset() computation to next frame"

Merge "mediawiki.notification: Refactor $.ready handler and init()"

Merge "mediawiki.notification: Use css() instead of slow show()/hide()"

mediawiki.notification: Move offset() computation to next frame

Crrently on all page views in WMF production, the $.ready handler
is inserting the notif $area and subsequently doing a forced
style calculation due to getBoundingClientRect() from offset().

Move this to an animation frame instead and re-order the statements
so that DOM reads go before DOM writes.

Change-Id: I7c6201dc8d4e3227e01b75e853b6e4dc9a734031

mediawiki.notification: Refactor $.ready handler and init()

* Merge code from $.ready handler into init().
  This makes it more obvious in which order the code runs, and
  makes it easier to make perf fixes in the next commit.

* Move updateAreaMode() to the start of the init() function.

Change-Id: I088446e17c2dc8ed88742bfeae80352444126e88

Merge "Remove perf tracking code that was moved to WikimediaEvents in Ib300af5c"

Merge "RCFilters: Allows specifying default highlights from the server"

RCFilters: Allows specifying default highlights from the server

Bug: T172757
Change-Id: I0545b4e0222e14be99d567e7890ccf33722a5d1e

mediawiki.notification: Use css() instead of slow show()/hide()

There is a lot of complexity and overhead involved with show()
and hide() methods for back-compat and edge cases that we don't
want to involve here (such as storing the last known display,
figuring out the default display, dealing with elements hidden
by default via CSS rules etc.).

We know the stylesheet does not hide the element by default, so
we merely need to set or unset the 'display: none;' inline style.

Ref T159911.

Change-Id: Ia22a2adddddf8514368124d93e3a753440814d97

Localisation updates from https://translatewiki.net.

Change-Id: I348badd5ddcd70157dd07c87717b6f64cfcafb3e

Merge "RCFilters: Avoid adding raw filter and group objects to js vars"