Use IGNORE INDEX(ls_log_id) instead of FORCE INDEX(ls_field_val)

ls_field_val was renamed to PRIMARY, but this is only partially
complete in WMF production. There is only one other index on the
log_search table, so ignoring that one is equivalent to forcing
the other one.

Bug: T17441
Change-Id: I63182a9f94eabb4cc47414d86b02c82bde1e58b0

Merge "Add mw-content-ltr|rtl to file redirect pages"

Merge "Fix small typo in documentation of IDatabase"

Make "eval.php -d 2" work again

Instead of iterating through load balancer info, just set the flag in
the master and replica connections, since that seems more robust.

Deprecate LoadBalancer::setServerInfo() and
LoadBalancer::getServerInfo(), no remaining callers in core or
in-tree extensions, I think I added the function just for this feature.

Do a service reset, since Logger instances are injected into
LoadBalancer by ServiceWiring (via MWLBFactory::applyDefaultConfig()).
Similarly CryptRand, MimeAnalyzer.

Fix the usage text for shell.php

Change-Id: I3e85a6e8cfa1243a0371cfb3ce1c18665e8c711d

Fix small typo in documentation of IDatabase

Change-Id: Ic9ac07700ded26e8a14f457428a7fd52a693923f

Add mw-content-ltr|rtl to file redirect pages

28ea73bc3a07a4588f1b5fde4f5c41522eaa9518 added NS_FILE to the list,
because most of the content is user language. That does not
really apply for #REDIRECT, however.

As a side effect, this also makes the redirect arrows visible on File
redirect pages, because their CSS depends on these classes.

Another side effect is that now "additional" content will show for the
redirect as reported broken in T29857.

Bug: T60955
Change-Id: I6b14c17a4b74ff5964ea61554aa3afb197d206bf

Merge "Convert mwdoc-filter.php to Maintenance class so --wiki works"

Convert mwdoc-filter.php to Maintenance class so --wiki works

Change-Id: Ie2fc4109bc2e7a23a6549d9705be44cf2eb858b9

Merge "maintenance/resources/update-oojs-ui: Copy across map files for debug mode"

Merge "Move RELEASE-NOTES-1.29 to HISTORY"

Merge "Follow-up 14cffee7: Correct deprecation version comment"

Merge "EditPage: Vary 'save button' label for appropriate i18n"

RCFilters: Override highlighted background

Muted state has overridden the 'highlighted' state which meant that
arrow movement weren't visible on items that were muted. This fix
corrects that.

Bug: T159768
Change-Id: I9d853eb3de1e482bd01e27635e92af505176eef5

Move RELEASE-NOTES-1.29 to HISTORY

Changes to master that are going to be immediately backported to
REL1_29 before release should write changes to HISTORY directly.

Change-Id: Ia3e02293b13406e7a4741ef14897dd83bca98d46

RCFilter UI: allow getParametersFromFilters to accept filter list

Up until now we assumed that this method should output the parameters
from the state of the system, which meant that even if it accepted an
external list, it required that it be the format of this.groups.

Instead, allow the method to accept a "flat" list of filter name/values
and translate that to a parameter state without changing the model state.

Change-Id: Ib41da620a16d874326fe73220ae7a0114c555639

Merge "RC Filters: watch filter highlight"

Merge "Document "What's This" group UI feature"

Merge "Toggle logo preloading via the skin"

RC Filters: watch filter highlight

When checking if lines are 'watchednew' for highlight purposes,
make sure 'rc_timestamp' and 'wl_notificationtimestamp' are truthy.

Bug: T163964
Change-Id: I3cd6cc9835db01229b8d1cc6ccfc371a36709ef2

Document "What's This" group UI feature

Bug: T163069
Change-Id: I1100dbea1549ee208dc0a1e9bcd56d8c93775302

Merge "RC Filters: Support multiple tag filters"

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

Localisation updates from https://translatewiki.net.

Change-Id: I036a25f9df1810a9a484e05389bd82a0a70e5457

Merge "RC Filters: support multiple namespaces"

Merge "userExpLevel test: use $tables from function under tests"

RC Filters: Support multiple tag filters

Bug: T164133
Change-Id: Id9404940644197bac235202a45324b201613303d

Remove unused var assign in Parser::getTemplateDom

Change-Id: If11d7a2568d4235df6888e4001500bdf45f58eae

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

RC Filters: support multiple namespaces

Make the namespace filter support multiple values separated by ','.

Bug: T164132
Change-Id: I6a83eafef69ea416dd9ba8cecc524efbd85db438

Merge "RC Filters: watchlist"

Merge "RC Filters: group.allSelected consider subsets"

mw.special.apisandbox: Don't pass $label to OO.ui.FieldLayout

They removed that functionality. Instead, access the FieldLayout's
$label property to attach the click event handler we need. It's not
documented (I think), but seems stable.

Bug: T164149
Change-Id: I30e7ad51dc1a484203138919c19a55f77552f330

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

mw.special.apisandbox: Specify auto width on PopupButtonWidgets' popups

Bug: T133710
Change-Id: Id6b05aa3d589040efeb7baf4ac9d4176182f538b

ApiSandbox: Specify a $overlay for menu-using widgets

This makes the menu work more sensibly when the widget is inside a
scrolling box of some sort.

Bug: T129308
Change-Id: Ic3b26883984030bb0ab37fcca8acd8b8c4535c2d

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

Toggle logo preloading via the skin

Bug: T100999
Change-Id: I0e18dc7fdbf133f3e4002f48f3040604687571dc

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"

EditPage: Vary 'save button' label for appropriate i18n

Change-Id: I0fd2995aa3178d383aa099cb0852336a1c67893d

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

Follow-up 14cffee7: Correct deprecation version comment

We merged into 1.29, not 1.30.

Change-Id: Ie2ebe0a44ea84a18c9ad3c21a6dd43ccbe3f0036

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"