Remove unused var assign in Parser::getTemplateDom

Change-Id: If11d7a2568d4235df6888e4001500bdf45f58eae

Merge "userExpLevel test: use a single time()"

Merge "RC Filters: watchlist"

Merge "RC Filters: group.allSelected consider subsets"

Merge "RC Filters: don't bind onMenuToggle twice"

RC Filters: don't bind onMenuToggle twice

Having 2 event handlers for onMenuToggle means
double-counting popup openings, which breaks
the highlight guided tour prompt schedule.

Bug: T164228
Change-Id: Ia6855e6b07e1364ff76c2bccd82c33b21b82619b

RC Filters: group.allSelected consider subsets

Consider a group to be all selected if its unselected
items are subsets of selected items.

This allows the watchlist group to show as no-effect
when 'watched' and 'notwatched' are selected but
'watchednow' is not selected.

Bug: T163964
Change-Id: I40d2a02ab074bc87f8a6f2498834b89fbbe55771

RC Filters: watchlist

Introducing filtering on watchlist status
  watchlist=watched|watchednew|notwatched

'watchednew' is a subset of 'watched

Bug: T163964
Change-Id: I04df40c8399e15a03a400b4a24afedf1df242a93

Merge "Check supportsDirectEditing when suggesting page creation"

mediawiki.user: Remove deprecated mw.user.bucket

The function mw.user.bucket is already deprecated since MediaWiki 1.23.

Change-Id: I2589dc7ce94004504a7aad080374877391f20057

resourceloader: Bump severity of style queue violation to Warning

Previously it was only in debug logs (which are enabled in Jenkins,
MediaWiki-Vagrant, Beta, and for mwdebug hosts in wmf-production).

Turning it into a warning() will log it for regular requests as well
which is the last step before we can consider hard enforcement.

Bug: T92459
Change-Id: I87c7794c5cfe35521bf76cc42f94907001e9d24b

Merge "Adding explantation for why to use User::incEditCount()"

Merge "RC Filters: Last revision filter group"

Merge "RC Filters: Make filters header sticky"

Merge "config: Use less generic cache key, and not fragmented by wiki"

Merge "config: Add unit tests for EtcdConfig"

Merge "Avoid master queries in loadAndLazyInit() for miser mode"

Merge "Use isSpecialPage() where possible"

Merge "api: Use NS_SPECIAL instead of -1 in ApiPageSet"

api: Use NS_SPECIAL instead of -1 in ApiPageSet

Follows-up e2055fe0a55c.

Change-Id: Ifd33f339a3c5ec475cc3c6b2fba57c35386bab8b

Use IPSet for checking of wgProxyList

Instead of using array_* functions, use the IPSet for checking, if a
specific IP address matches a set of addresses.

This also deprecates a backward-compatibility functionality, that
the wgProxyList array could also be an associative array, where the blocked
IP address is set as a key of the array insted of a value. All IP address
keys will be mved to values on-the-fly, however a deprecation warning will
be emitted. A notice in the Release notes was added, too.

Bug: T161580
Change-Id: I69d9534942c415ab044177969ecd54160079b593

config: Use less generic cache key, and not fragmented by wiki

* Use makeGlobalKey instead of makeKey to avoid fragmenting
  the cache by the wiki-specific key space.

* Add "EtcdConfig" and $this->host to the cache key to make
  it less generic and less likely to conflict with unrelated values
  in case multiple instances exist somewhere in the MediaWiki run
  time (or during testing/debugging).

Adding $this->host should be fine given we were already including
the directory within that host. Although I do recognise that we
may want to encourage slow cache rollover instead of instance
cache rollover in case the host name needs to changed. Perhaps
we should have some kind of canonical name for the instance for
the purpose of caching so that directory or host can be changed
without changing the cache key, and thus have cache roll over
normally over the course of 10 seconds, instead of immediately.

On the other hand, deployment will likely already spread things
out a little bit.

In addition, due to the cache being kept indefinitely this means
the old key will stay indefinitely. Perhaps not an issue given
it's just one key, which will clear on restart eventually?

Bug: T156924
Change-Id: I622e6618f2d4171626e4b272bcb11a97a85cb770

config: Add unit tests for EtcdConfig

Change-Id: I6e647985b98724b5448831a6fae1b2d2521fd363

Merge "Avoid implicit globals"

Avoid implicit globals

Wrap in IIFE if variables is not intended to be global, otherwise
explicitly assign to window for better readability, and avoiding
problems if code were to get wrapped in IIFE later.

Change-Id: I2313daeaf00f27b6ea9eb6b907c066844c5d3836

Localisation updates from https://translatewiki.net.

Change-Id: If2d4e2b342645f88f166ff54ce9102439599e24c

RC Filters: Last revision filter group

New filter group to show changes related to the last
revision of a page or show changes on previous revisions.

Bug: T163561
Change-Id: Ifc376a34bbe564714285894f93ffb80aa82f5fdf

RC Filters: Make filters header sticky

Bug: T163431
Change-Id: Id6d3b1be6d9655cd65ff924e95aa9a8d464771b7

Remove unnecessary Function.binds

Also fix ,->; typo

Change-Id: Icf87ae6e2aa111f3cb578c7355c92ffc6019df55

Spread and indent string-concatenations

Avoids single lint string concatenations and multi-line strings.

Change-Id: Iada91c1019cda87d48b13bdcff963d1950b54874

Add GenderAliases for namespaces

Change-Id: I16b54e35c4212ccc26e7c9d23ee898e29c04a69f

Localisation updates from https://translatewiki.net.

Change-Id: Id132ebc6e31dd5ef9c4a5e774c39c393aa67e3d5

Use isSpecialPage() where possible

Change-Id: Ie4d0838acf96a7ed4a1fe4cfdc901c77d3312174

Localisation updates from https://translatewiki.net.

Change-Id: Id5d126cf626391b5f7d562758a8a2e1a603fed21

Avoid master queries in loadAndLazyInit() for miser mode

Bug: T92357
Change-Id: Idaef633d9039cdbfa3bd123018f2339c6d489fe4

Merge "SpecialJavaScriptTest: Make sure we don't catch our own exception"

Improve EtcdConfig fallback logic

If the cache is stale and the lock keeps being acquired, but the re-fetch
fails each time, the method would end up failing after the timeout was
reached. Instead, use the stale cache if available.

Change-Id: Ieafc9de17e6c60d8eea7b937923b4ad548e99be8

Localisation updates from https://translatewiki.net.

Change-Id: Id3c6419b44d67846ad7aaf882a635a3b41a98cf3

Adding explantation for why to use User::incEditCount()

... rather than User::incEditCountImmediate() for most cases.

Change-Id: Ia16baabaeaca5c726e1dd1fc2506dc0b85500b6e

Merge "Simplify Image/Video/Audio rules using LESS nesting"

Simplify Image/Video/Audio rules using LESS nesting

Also allow Video/Audio to contain img instead of video
for VE support.

Change-Id: Ic54662bbcd50de70133c6581cd7d8025c872a7f4

Merge "EtcdConfig: use array_key_exists(), the value can be null"

Merge "Add space version of img_upright and img_page for language de"

EtcdConfig: use array_key_exists(), the value can be null

Change-Id: Iae175401da8d49f5475d8c88cc5390b16e7cba32

Merge "Document rationale or primary use case for many indexes"

Fix help text for --d on shell.php

1 is stderr, not 0

Change-Id: Iaee4ffd5767f8ae693e8ef19371ce38fe6bd1817

Add ConsoleLogger, use it for eval.php -d

eval.php previously set $wgDebugLogFile to /dev/stdout. This had the
following problems:

* It doesn't work if the maintenance script is executed via sudo, since
  /dev/stdout is typically owned by the original user, so MW can't open
  it. Using php://stdout worked on HHVM but not PHP.
* Setting $wgDebugLogFile has no effect if the wiki uses MonologSpi.
* Setting $wgDebugLogFile has no effect on channels configured with
  $wgDebugLogGroups.
* stderr is a more appropriate place to send logging output.
* Writing to configuration variables is discouraged.

So, add ConsoleSpi, which is a very simple logging service provider
which sends all messages to stderr. This should be suitable for
debugging with eval.php or shell.php in WMF production or beta.

Change-Id: Ib0d6ce45e0cbecd58263fc4e360c63d4149acb3a

Cleanup postEdit and match styles to mw.notification

* Remove HTML template
* Add mw-notification class and dependency
* Fix variable scoping issues so firing two notifications
  doesn't cause one to become stuck

Bug: T58313
Change-Id: I9a702d23a8a9b5cc098cbb413213d73813bec097

Merge "Expire the cache of a User object when a group membership is set to expire"

Localisation updates from https://translatewiki.net.

Change-Id: Ia2686ac474c045e91fd29daef68ec0925b73823e

Add space version of img_upright and img_page for language de

On de.wikipedia.org the localized version of upright is used with
spaces, not underscores.
Add the version as alias to fix the pages and make linter happy
(T162961)

Change-Id: Id4726aecb51887fc82583e4d6b9284706b6b5b69

Merge "Release prior row locks beforehand in LinksUpdate::updateCategoryCounts"

Merge "SpecialJavaScriptTest: Do not accidentally pass parameters to QUnit.start()"

Merge "SpecialJavaScriptTest: Format long JavaScript code fragments"

Merge "Prepare for 1.30 branch"

SpecialJavaScriptTest: Make sure we don't catch our own exception

If the jQuery promise returned by mw.loader.using() is rejected
synchronously (e.g. because one of the modules has a dependency on
non-existent module), the function passed to fail() also executes
synchronously. Note that this is true also with jQuery 3.

The exception it throws was caught by the catch() below, which
resulted in start() being called twice, which resulted in QUnit
throwing 'Uncaught Error: Called start() outside of a test
context too many times' rather than actually starting.

Change-Id: I5c6b50647c0af0fdec6547aaa59165f6b4a42642

SpecialJavaScriptTest: Do not accidentally pass parameters to QUnit.start()

If the mw.loader.using() call's promise is rejected, we end up
passing an Error object as the 'count' parameter to QUnit.start().
This seems to be harmless, but is very confusing when debugging.

Change-Id: I44caca5285dbced5a5876d9d7ff6236dbd3efc35

SpecialJavaScriptTest: Format long JavaScript code fragments

Change-Id: Iaa6369467e9fa846a8d5eb8f43cf0b523ef1a64e

Merge "Block: Remove old IP addresses from default autoblock_whitelist message"

Merge "mw.widgets.DateInputWidget: Highlight widget and open calendar on focus"

Merge "Correct the "Help", "Media" and talk namespace names for Dinka"

Merge "Do not add limit to ApiQueryPagePropNames when database type is mysql"

Correct the "Help", "Media" and talk namespace names for Dinka

The corrections come from users Dinkawiki and Mawanmuortat,
who did most of the translation into this language.

No aliases are needed, because the file was only created recently,
and not yet used anywhere.

Change-Id: Ic1ddfaca403e05e5971c516d6e7a454311794f3c

Release prior row locks beforehand in LinksUpdate::updateCategoryCounts

Bug: T163801
Change-Id: I6a026ee9a249da5a8fd9ceb216416340a9188d3f

Document rationale or primary use case for many indexes

Change-Id: I1f0a13adbc6f618f38c3b05a1d356d76e2f5bd23

Merge "Add a space between "Go to page" dropdown and button for multipage images"

Merge "Revert, Follow-up: -1 recent contributors"

Revert, Follow-up: -1 recent contributors

This reverts commit b67f40e5b01af569794eba26da79497329590d36.

There was an issue that the web installer, when inserting initial
values into the db table site_stats, would not insert any value for
ss_active_users. That lead to some places, most notably
Special:CreateAccount, to show off -1 recent contributors.

The commit to revert solved this by setting a default value of 1 for
the ss_active_users field in the database schema. However, while the
default schema has been changed, there hasn't been any schema update
procedure implemented. Also, the default of 1 is inconsistent with
other defaults in that table.

Revert that schema change completely. Instead, make the web installer
insert 0 to the field ss_active_users (just like it does with all other
fields in that table).

Bug: T56888
Change-Id: I4298754c2238f6e5447a5df77a4ada722ba302b2

Merge "mediawiki.toolbar: Emit deprecation warnings"

mediawiki.toolbar: Emit deprecation warnings

Callers to the EditPageBeforeEditToolbar hook can now return false to
signal that the toolbar is over-written, in which case this old code
won't be called (so no deprecation warnings for users of WikiEditor,
CodeEditor, etc.).

Bug: T30856
Depends-On: I5e755ef5dffb843368563511044d3123f14dc4bc
Change-Id: I3bed70a20e4b94fe3d04a00925b8012312202574

Merge "exception: Add newline at the end of a debugging line for CLI users"

exception: Add newline at the end of a debugging line for CLI users

Bug: T162315
Change-Id: Ia2963fa298ab0e4782ef823e7f53322cd6a9c99b

Localisation updates from https://translatewiki.net.

Change-Id: Ice5f1ec383e94de1b6e43c581a821e415edfe767

Merge "Add percent and permille to special character list"

Extend grammar rules for Ukraininan language

Change-Id: I2a18acee6a1c4a7462953477da50d57239994f37

Add percent and permille to special character list

Bug: T163894
Change-Id: I82a926412fb0f7c886e1e17ed4cf05cf2d4f00d9

Prepare for 1.30 branch

Change-Id: If9842b19ec995d03f93f96bfad4d61ca96c37f3c

Merge "SpecialMediaStatistics: Remove WMF-specific hack, no longer needed"

SpecialMediaStatistics: Remove WMF-specific hack, no longer needed

We deleted the broken row from the databases per T155769.
This reverts 6e02a95227c4ef194ff2299a823aaa51aa9a223c.

Bug: T74535
Bug: T155769
Change-Id: Iac0873eb7112502dd7fc6b34ca956d9a2dffcb44

Merge "EditPage: Factor out getSaveButtonLabel method"

Merge "Interwiki: Don't override interwiki map order"

Update OOjs UI to v0.21.2

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

Change-Id: Id7cae80a4f3d84ea5d50003f2409537b307c0c58

Merge "RCFilters UI: Create base classes for shared objects"

RCFilters UI: Create base classes for shared objects

Preparing for adding other types of items than filters (namespaces,
users, tags, etc) this commit creates base classes for the model
and relevant widgets and extends Filter* from them.

Bug: T159942
Bug: T163521
Change-Id: I61c88a1f14a3ca9d91aa831187eda156468a6591

Merge "Special:Block: Fix FOUC of bold 'mw-block-hideuser'"

Interwiki: Don't override interwiki map order

The ksort() here was causing the order to be enforced as
alphabetical instead of preserving the original order.

The order usually doesn't matter, except with regards to handling
of duplicates. Due to Parsoid normalising external links to interwiki
links, it has to do a reverse lookup. In doing so it has to decide
which one to prefer. It currently picks the first match from the
API request for meta=siteinfo&siprop=interwikimap, which didn't
match the defined order in the actual Interwiki map due to ksort()
being called in getAllPrefixes().

Sort in this function was originally introduced in 2010 with
commit 844e7c83e4 (2011; r92528; T21838), which is otherwise unrelated
and left no rationale.

The existing unit tests needed to be adjusted slightly as they
assumed alphabetical order. While it appeared they were also defined
in alphabetical order, this was merely the order of the variable
creation. The effective order is preserved within locals and globals,
but overall globals come before locals.

Also removed the duplicate test for Hash and CDB in InterwikiTest
that belongs in ClassicInterwikiLookupTest instead.

Bug: T145337
Change-Id: I7348748801cbdf16c6ceea5b0654fc174b79707e

Special:Block: Fix FOUC of bold 'mw-block-hideuser'

Loading Special:Block/<your own name> shows "Confirm block", but
this then becomes bold a second later. This is because the styles
are part of 'mediawiki.special.block' which also contains JavaScript
and is loaded with addModules().

Move the styles to mediawiki.special.css instead.

Change-Id: I1c47256227603a6cde36238fdd7e11205d9494ec

Localisation updates from https://translatewiki.net.

Change-Id: I53453d3129920681a8eb8ece0c9c2cba400cab77

Add a space between "Go to page" dropdown and button for multipage images

Change-Id: I174da5597563be36cc6e67a485f8556b9bcd84f6

Expire the cache of a User object when a group membership is set to expire

To avoid user permissions persisting for up to 1 hour beyond their expiry
time.

Bug: T163691
Change-Id: I85b578e6b7816639c3d0101d66efb1efedb17af6

Merge "Keep headers from jumping when expire interface is shown"

Merge "Localisation updates from https://translatewiki.net."

Localisation updates from https://translatewiki.net.

Change-Id: I4ccfc62901bfa044c19b63fa30b450af8e9aa0a5

Merge "Remove usages of RequestContext::getStats()"

Remove usages of RequestContext::getStats()

Bug: T156810
Change-Id: Ic07896a68389e20ce0a35a06049b8c51b6c3f699

Keep headers from jumping when expire interface is shown

Change-Id: If2cb7c8e2e30c5e40f32113353f571a1fba63d29

Add Dinka (din) to Names.php and languages/messages/

Bug: T163600
Change-Id: I2bf03c97680c17cddf48d05adee88581640268ec

Merge "New maintenance script to clean up rows with invalid DB keys"

Do not add limit to ApiQueryPagePropNames when database type is mysql

It's a known bug in some versions of mysql (including WMF dbs) that it
won't do a loose index scan when a limit is applied. Given that number
of possible types is still very low, this shouldn't be a problem.
More info: https://bugs.mysql.com/bug.php?id=61517

Bug: T115825
Change-Id: I4c3b885ac05b793088a92e054a38a36b9d07c0d4

userExpLevel test: use a single time()

Capture `$now = time()` only once and propagate
the value in the test and production code.

Bug: T161041
Change-Id: Ib2887a81a4e8015e3c80ef89d48181c7e5e1d406