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"

Make Database::rollback() also suppress callback errors

This is consistent with query error suppression, let the
listener callbacks have a chance to run. This also makes
LoadBalancer::rollbackMasterChanges move on to the other
connections.

Change-Id: Ic8b9dd4b868bfe69b04fb20f7be5fce11d864fc4

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

I registered the additional shutdown function for CLI only
as it shouldn't have effect otherwise.

Bug: T177414
Change-Id: I440d294eef5e307743cfc7f5ab3b531e8c973873

Merge "Update OOjs UI to v0.23.3"

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

Bug: T176236
Change-Id: I0c68ea0e090cfb065d2e3e8acad7025c61c344ca

Remove some old mysql 4 messages from installer i18n files

Bug: T34217
Change-Id: Id9859769235a03229f962545cef6786f5e700c71

Inject dependencies into Shell\Command

This slightly changes how execution time limits fall back on each other.

Change-Id: I7754a9e6be9638eebe90cb953adb8e2a6ee97cef

Update OOjs UI to v0.23.3

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

Depends-on: Ica85dce164ab0ff269ca26ac078ea245219ad4c1
Change-Id: Ied8d6d6dfa9965912c17e0573fec33d2b67a5027

Merge "Support uploads with UTF-8 names on Windows"

Support uploads with UTF-8 names on Windows

On PHP 7.1 and later, filesystem functions on Windows
use the Unicode system functions, which makes our file
handling work for non-ASCII file names.

Previously this was blacklisted for Windows on all PHP
versions. Versions before 7.1 will still reject Unicode
filenames with non-ASCII chars.

Bug: T3780
Change-Id: I94377faa5185f133be2dfb7b9b6aeacbd582834f

Remove some remaining mentions of unstubbing from core

Change-Id: Ife34335db7afd9bd4f0936cb90be227d7c4c9651

Merge "RCFilters: Init highlight button based on model state"

Stop stubbing StubUserLang

Stub objects are confusing as heck and are a performance optimization
that really aren't fit for the modern era. They were designed to avoid
loading the actual code from the disk back in the days when bytecode
caching wasn't always gonna be there.

It's 2017. If you're using HHVM, you've got a bytecode cache. If you're
using any reasonably recent version of PHP then you've got the opcode
caching enabled by default in basically every distro-related build.

Nothing actually relies on this object being a stub (that'd be silly),
so only references are basically things force unstubbing (also kind of
silly) the object. Once remaining code referencing this in extensions
are all cleaned up then we can remove the class itself.

Change-Id: I15df24aeeb729e8e764792daa933377f35042fab

Merge "RCFilters: Convert saved queries from filters to parameters"

Merge "Add SECURITY"

RCFilters: Init highlight button based on model state

Bug: T177028
Change-Id: Idea7e09152e3176084f0634d24615a79364d77bb

Localisation updates from https://translatewiki.net.

Change-Id: Id856ec22448bbe6d788185bbfbe020bb387d8a56

Add SECURITY

Intended for people who aren't familiar with our projects. To avoid
duplicating any information, just link to the wiki page that contains
information on how to report security issues.

Bug: T158119
Change-Id: I8eb05688cc028ba2b4b467fc464b590da0e60b0b

Merge "TitleWidget: Only apply padding fix to withImages mode"

TitleWidget: Perform diacritic-insensitive highlighting

Change-Id: Ib92877e4afa45f73bb086c04946f5c1d0c99ed6f

Merge "TitleOptionWidget: Don't show an icon when showImages=false"

Merge "mw.widgets.SearchInputWidget: Avoid overriding getOptionWidgetData"

Merge "Remove 'Browser default' editfont option"

Merge "mediawiki.searchSuggest: Add support for search namespace preferences"

mediawiki.searchSuggest: Add support for search namespace preferences

This way it allows namespaces other then NS_MAIN.

Bug: T26214
Change-Id: I6a545eb8563e81a66de39a8295ef2e40c3e73f95

OutputPage: Turn html5shiv into a raw ResourceLoader module

This lets us take advantage of ResourceLoader's on the fly minification
rather than needing to ship a minified copy, which is problematic for
other distributions like Debian.

Because the startup module excludes raw modules, it is not possible to
load or depend upon this module, and trying to use
`mw.loader.load('html5shiv')` for example will throw an exception.

This follows-up 3a30e03645f895b.

Change-Id: I635e224fb043c614c4918771c7c8f836575c683b

resourceloader: Add unit test for ClientHtml::makeLoad sync=>true

Used by Special:JavaScriptTest, and we may need to use this
in OutputPage as well (for html5shiv).

Change-Id: If7d7c12056dc3aab78486050f0798d42b158d9a9

phpunit: Include all raw modules in detection of illegal dependencies

* Use getStartupModules() instead of hardcoding these two module names.
* Also collect any other module that is registered with raw=>true.

Change-Id: I5f75cc8f99b90f58d76b7e87f068f50f9f608ed0

Merge "Add a help button to the Tagged Edits menu"

Merge "Change "Advanced filters" label color"

Merge "SpecialBlock: Tweaks for OOUI HTMLForm"

TitleWidget: Only apply padding fix to withImages mode

Change-Id: I60d0c3169cede2251595d68649c76823ea77afe9

TitleOptionWidget: Don't show an icon when showImages=false

Change-Id: Ib4262bc15a3845ec9e7af484dbdfa4277f2f4014

mw.widgets.SearchInputWidget: Avoid overriding getOptionWidgetData

This means we don't have to track upstream changes to the method.

Change-Id: I44739eefae4d47aeee9f506e0d99e605da4fb961

SpecialBlock: Tweaks for OOUI HTMLForm

* Expand both the dropdown and textfield of "Reason" field to the
  usual maximum width and display them on separate lines.
* Fix autohiding of form fields where applicable (checkboxes that only
  apply when blocking a user or an IP address).
* Fix bolding of the "Confirm block" label when blocking yourself.

Follow-up to d56758e134358fdac3f8dda16417bfd30f78bab9.

Bug: T107036
Change-Id: Ia5c54e518de99a79f5319c42ccf1a9a0e31ad471

Change "Advanced filters" label color

Change "Advanced filters" label color to Base20 (#54595D),
fixing monobook skin low contrast.

Bug: T172733
Change-Id: Idb1b9ad7120c387482f8df7fcad8508f2a95139e

Add a help button to the Tagged Edits menu

Bug: T176661
Change-Id: Ia295633dcf96a5466650620ef6e5cef766d19988

Selenium tests should log in before creating pages

MediaWiki Vagrant by default allows creating pages to anonymous users,
but other environments, like Beta Cluster, require user to log in
before creating pages.

Bug: T176315
Change-Id: I34eeaf3cc22a64be6580aa483b28885137ba60b6

Localisation updates from https://translatewiki.net.

Change-Id: I75b3048eb0491d6e67a2a3cdf34b47791bd6baed

Reapply "RCFilters: Don't apply/clear highlights 66 times"

Reverts 84ddcd5d23, reapplies 212d67c67.

The original version broke the highlight color circles, this fixes it
by using enable instead of this.highlightEnabled to pass down to the
items. The former was always false, so it didn't actually propagate
the state down correctly.

Bug: T177107
Change-Id: I341d9bcd39252eac07bfebfa6d46895bc62c372f

Merge "Revert "RCFilters: Don't apply/clear highlights 66 times""

Revert "RCFilters: Don't apply/clear highlights 66 times"

Breaks highlight display

This reverts commit 212d67c67e153f95fd2eb49a75b7ece27d794b53.

Change-Id: Ia0111bd35a40df56122a5b8c483de1be35609911

Merge "Updated type and function docs for MagicWord.php"

Merge "TitleWidget: Use OOUI query highlighting, instead of jQuery lib"

Updated type and function docs for MagicWord.php

Change-Id: I46ebd315a99b1677ff89ba392c0072b41bbeb36d

Merge "SpecialBlock: Switch to OOUI form"

Merge "Enable IP ranges in HTMLUserTextField"

Merge "Remove unneeded js module in SpecialUnblock"

Improve GitInfo for reading packed refs

GitInfo only reads branches in files /refs/heads/*; it should also
read those in /.git/packed-refs.

There are 4 tests in this commit to test this new "feature" and add some
code coverage to other methods. Given this class tests Git behaviour and
MediaWiki itself is versioned with Git, the test data are created during
the tests and removed at the end to avoid interference with Git as CVS.

Bug: T155747
Change-Id: Icccdf3ec797788582f836e2d8e267a1d714d50dc

Merge "mw.loader: Return consistent object shape from splitModuleKey()"

Merge "Avoid double escaping headers in InfoAction"

Merge "Clean up wiki ID and DB domain ID handling"

Localisation updates from https://translatewiki.net.

Change-Id: I9d12a53dfbb3eb90cae81cd670f94cb5305e3ff4

TitleWidget: Use OOUI query highlighting, instead of jQuery lib

Change-Id: I75ee7b3da74bdf9638c442322b699d22b4fee74c

Merge "Check requirements for parser test suite in phpunit tests"

Merge "build: Updating mediawiki/mediawiki-codesniffer to 13.0.0"

Localisation updates from https://translatewiki.net.

Change-Id: Ic025a6a40c6af463475b0e50ce14a20ea17243f6

mw.loader: Use Array#indexOf directly instead of jQuery.inArray

Unlike other jQuery methods (which are quite often significantly
faster or slower than native methods), jQuery.inArray is quite
literally a wrapper around Array#indexOf. Use it directly where
possible.

Also:
* Replace `$.inArray(val, [a, b]) !== -1` pattern
  with `val === a || val === b` which is upto 10x faster.
* mw.loader.store.set.: Remove redundant parenthesis in nested expression.

Change-Id: Ia2b922b3245bfb39b609b57e84938317e9c89cd6

mw.loader: Return consistent object shape from splitModuleKey()

This is used in mw.loader.implement() in a truthy check before
assigning as value of `registry[module].version`.

Both implicit undefined and empty string are falsey so the
behaviour is unchanged.

This reduces shape variability for JS engines, and also avoid
potential mistakes if the function is used elsewhere since
internally the registry, as maintained by mw.loader.register(),
uses empty string as default for version, not undefined.

Change-Id: I067137557a55d521fefa40986373e55b6bbb59ff

Clean up wiki ID and DB domain ID handling

Bug: T174017
Change-Id: I42299a393c6691d39817db1d83a176a17df2474b

Merge "RCFilters: Don't apply/clear highlights 66 times"

RCFilters: Don't apply/clear highlights 66 times

Applying highlights on init was very slow, because applyHighlights
was called 66 times and clearHighlights 64 times.

We first call toggleHighlight() on each filter item, which
leads to update events being emitted by each item, and onItemUpdate
calls clearHighlights() followed by applyHighlights().
There are 64 filter items in my setup, so that's 64 pairs of
clear+apply calls. Then we emit highlightChange, which causes
another apply call, and onModelUpdate calls apply as well.

This change makes sure that the model-wide ("global") highlight
flag is always set to false while we're toggling the highlight
flags on the individual items, so that onItemUpdate short-circuits
and doesn't call clear/apply. There are still other issues (too many
places call clear/apply, too many events emitted), but running the
highlight code twice instead of 130 times on init fixes most
of the performance problems.

Bug: T177107
Change-Id: I65c409ccfabd974ef0b5e2bddff70c9e78e8fd60

Sanitizer: hard deprecate passing $options to escapeIdReferenceList()

Change-Id: I22743102e75614bba848e09f032586923cef0f60

Merge "Remove "only newlines in trailer" special case for category/language links"

Avoid double escaping headers in InfoAction

Bug: T111763
Change-Id: I4f582ce5785448a4cb48f3a1d56371eddf38a232

SpecialBlock: Switch to OOUI form

* Remove additional wrapper, and the 'blockip-legend' message.
* Replace autocompletion using 'mediawiki.userSuggest' with OOUI,
  by using 'type' => 'user' for the username field

Bug: T107036
Change-Id: Ib8f7e03388073450ecff28ad2c0d3f9161f259d8

RCFilters: Convert saved queries from filters to parameters

This will allow us to load them in the backend, and to keep
consistency between RecentChanges and Watchlist if needed.

Added also a 'backup' preference to keep the previous version
before the conversion, in case of mangling of the queries.

Bug: T166908
Change-Id: I8e26b66e43bd16282b7bdb52abc152f92a9c877d

Enable IP ranges in HTMLUserTextField

Like an existing user and a single IP, allow to use an IP range, too.

Bug: T107039
Change-Id: I9137f71f6119c68d787f508a3f57b476404315d8

Remove unneeded js module in SpecialUnblock

Followup to I3c51183c0275dd2f9522ba60990852cc96409e2d

Bug: T107036
Change-Id: I6d1ca8b07e40950480344a106558a98a5bf5d341

Merge "Revert "CSSMin: Improve encoding of quotes in embedded SVGs""

Merge "Revert "CSSMin: Mangle whitespace in embedded SVGs""

Merge "TitleWidget: Upstream generic page icons from VE"

Tweak layout of TitleWidget results

Bug: T177085
Change-Id: I663e63e89b1be2d377de28ba74ad43519ff04741

TitleWidget: Upstream generic page icons from VE

Change-Id: I297648d14b4a2540a40f3a0b7d22a7905ef7c035

Revert "CSSMin: Improve encoding of quotes in embedded SVGs"

This reverts commit 5e3165f0080961e637e9ed600b696786aee7eda4.

Bug: T176884
Change-Id: I8bd9af67aeb11cb478ba402205ce834b1bec17a7

Revert "CSSMin: Mangle whitespace in embedded SVGs"

This reverts commit dfd42d2653ea9ff2ca05dab7cd6f2285abba9d08.

Bug: T176884
Change-Id: Iddc98332d0b5a31d3a1c56c14014a53a06393f21

Merge "Migrate Special:Unblock to OOUI"

Merge "Update cleanupSpam.php to hide its activity from RecentChanges"

Merge "Improve namespace handling in tests"

Merge "Fix patchfile used to add newtalk table on mysql"

Update cleanupSpam.php to hide its activity from RecentChanges

Spam cleanup activities, mostly when it comes from automated processes
(the so called 'spambots') can involve a fair ammount of edits and log
actions to accomplish.

As such, to avoid flooding Special:RecentChanges, this script will now
set a bot flag on the script run so its edits and actions won't show
up in that special page.

To prepare this patch I had the help of Bartosz Dziewoński which I'd
also like to thank and I'll credit as co-author as well.

Bug: T176206
Co-Authored-By: Bartosz Dziewoński <matma.rex@gmail.com>
Change-Id: Ifde08056a7481b877b4b82699cc0a5ea3f962dd9

Merge "Do not include RC_EXTERNAL in number active users on Special:Statistics"

Improve namespace handling in tests

MWNamespace has three internal caches, only one of which can be cleared
(and that somewhat oddly by passing a boolean to
MWNamespace::getCanonicalNamespaces()).

This change introduces a MWNamespace::clearCaches() method to clear all
three caches. It also adds some resetting in tests that seemed to be
missing it.

Change-Id: I1dcfcd8713888b3ff8fc75e95329ba72bd95d0c9

Merge "Revert "DumpFilter is autoloaded. No need to require in maintenance script""

Merge "Consistently preserve anchors in the title specified in 'mainpage' message"