Update cached user ID after user is added to the database

Bug: T119021
Change-Id: I5e0599d1d045b0389a7825fddc2b346e4cfd001d

Use User::getId instead of trying to fix the ID in BlockTest

User::setId() has no effect on User::addToDatabase whatsoever,
and directly messing with the database is fragile due to
internal ID caching. Just use the insert ID instead.

Change-Id: Ib92f2b6d73deacaec90dc06634d8b3ad195d53e3

Merge "Update OOjs to v1.1.10"

Update OOjs to v1.1.10

Release notes:
 https://git.wikimedia.org/blob/oojs%2Fcore.git/v1.1.10/History.md

Change-Id: Id19682f59690aafc70fa05c6febf32b1206090c2

Merge "Make TitleInputWidget configurable, if term should be highlighted or not"

Merge "mw.loader: Use requestIdleCallback to update module store"

Merge "Add flexbox mixins to mediawiki.mixins"

Merge "Language: Convert names cache from MapCacheLRU to HashBagOStuff"

Merge "Title: Convert cache from MapCacheLRU to HashBagOStuff"

mediawiki.Uri: Remove redundant jshint overrides for latedef

These aren't being violated. They were either left-over from
refactoring or added to work around bugs in JSHint that have
been fixed since.

Change-Id: I3ac4a6e97b6294f392ae9ec2474eab448b044a13

Merge "build: Enable all remaining jsduck jsDoc rules except two"

Merge "TitleSearchWidget: Only update results if the query is current"

Merge "UserMailer: Use PHP_EOL instead of DIY"

TitleSearchWidget: Only update results if the query is current

With near-simultaneous inputs (e.g. holding down the backspace key
while deleting text) you can cause the suggestion promise to resolve
out-of-order. This causes apparently incorrect suggestions to be
displayed for the current input.

To fix this, abort the existing promise if it exists.

Bug: T114178
Change-Id: I9332452fd914b54e7c564284da2a8a00865ae806

UserMailer: Use PHP_EOL instead of DIY

Also fixes arrayToHeaderString() to be correct when called without
the $endl parameter

Change-Id: I8b9f72500f9a055f9f2e1cfcbb276d565674f1b8

Add flexbox mixins to mediawiki.mixins

Includes some fallback support for 2009 (old firefox and some mobile
browsers) and 2012 (IE10) specs.

Change-Id: I0c6fca82b778a28930bf83fd43a3059858fe9243

Merge "Avoid using !important and hide the nolabel instead"

Merge "Update OOjs UI to v0.13.2"

Merge "Make runOnTransactionIdleCallbacks() handle dangling transactions"

Update OOjs UI to v0.13.2

Release notes:
 https://git.wikimedia.org/blob/oojs%2Fui.git/v0.13.2/History.md

Change-Id: I0626d91d4b93e1d680579f1e6a02555c6050127f

Merge "Validates wgArticlePath does start with slash (/)."

Validates wgArticlePath does start with slash (/).

When relative URL used in $wgArticlePath, and $wgArticlePath does not
start with slash (/), raise FatalError.

Bug: T48998
Change-Id: Ic7cd6f774cff97081f4f35af351161170b4b26eb

Merge "Avoid doNotifyQueueEmpty() race conditions for Redis"

Avoid doNotifyQueueEmpty() race conditions for Redis

Previously, races could cause queues to be seen as empty
and ignored when they actually have jobs in them.

Change-Id: I6cdf5c1fbd33c9ed7982e5d2d494974b1d7e09fd

mw.loader: Use requestIdleCallback to update module store

.. instead of setTimeout.

Also adapt the code accordingly since it is no longer needed to
cancel and re-schedule an update every time update() is called.
This is already handled by requestIdleCallback.

We can let the already-scheduled callback survive. The data
that will be saved is serialised during the flush anyway,
not when scheduling.

Change-Id: Iea48919b4baba7647040b6ed9ff4b6d0d387b962

Make TitleInputWidget configurable, if term should be highlighted or not

Depends on: I36502da43c383373

Bug: T106273
Change-Id: I393eef9cd20bdf80e4db72891f23de08bcdfd376

Localisation updates from https://translatewiki.net.

Change-Id: Ic027cac2ca0839cbb6083078c5a90eabee280510

Merge "Migrate feeds from $messageMemc to the WAN cache"

Merge "Various JobQueue test tweaks"

Avoid using !important and hide the nolabel instead

Instead of playing catchup with width settings, let's simply hide the
table cells with nolabel. This seems closer to the original intent,
and it works with at least IE8 as well, so why not.

It does effectively move every element without a label 1em (padding) +
1px to the left in your preferences. This will likely be noticeable by
endusers.

Follow-up to: I24d9b16ed672

Change-Id: I92083687ef8a9a6e5fc9c822f9449a1aa7ed7ea1

Merge "Make getLaggedSlaveMode() use reuseConnection() as needed"

Merge "resourceloader: Log if MessageBlobStore failed to fetch a message"

Make getLaggedSlaveMode() use reuseConnection() as needed

Bug: T118162
Change-Id: I03e1386b952a9e9243518c12fb175727f81c9976

Merge "Make FindOrphanedFiles handle bad file titles"

Merge "Make FindOrphanedFiles handle files with ! properly"

Migrate feeds from $messageMemc to the WAN cache

This makes the delete() calls work properly for all DCs.
Also, using the message cache was fairly bizzare.

Change-Id: Idec7fa47811e982ba89bb8fbbd9565a26585e77f

Merge "Move Math specific CSS from core to extension"

Merge "SpecialAllMessages: Use cached getHtmlCode() instead of wfBCP47()"

Implement mw.requestIdleCallback for deferred background tasks

We often use the idiom "window.onload" or "$(window).on('load')".
Since code loads asynchronous, this is problematic because the event won't
always be observed as it may fire before the event handler is attached.

Most tasks also don't really want to wait until the page is loaded (in which
case it would run immediately if the page is already loaded). Rather their intent
is just to defer it to a later point in time – to avoid disrupting user events.

Bug: T111456
Change-Id: Ieba0440c6d83086762c777dfbbc167f1c314a751

SpecialAllMessages: Use cached getHtmlCode() instead of wfBCP47()

Follows-up 5efbb10f

Change-Id: Icf00a4a20559d666b2ac551b933873e695735c9c

Move Math specific CSS from core to extension

Depends on I3775980916de12aef23bac85c38a6f92253aa479
in extension Math.

Bug: T110492
Change-Id: If6a4ca34a54c451cdb10d011ffd806a34cbccd2c

Mention main WAN and stash instances in cache debug statement

Change-Id: Ic98ce5aecc093400f23d3595be75df3bf54268bf

Various JobQueue test tweaks

* Remove some getAcquiredCount() assertions when claimTTL=0
  as this is not well defined enough (queues may take a few
  minutes to garbage collect the failed jobs).
* Added some tests to make sure push() only de-duplicates
  among unclaimed jobs.

Change-Id: Ie0a5e539095c245dfcc8c160417e12824eb7ab83

Improve LoadBalancer::doWait() performance via APC

* This avoids bothering with MASTER_POS_WAIT() if an equal/higher
  position was already successfully waited on by another process.
* Add DBMasterPos toString() and hasReached() methods, which
  MySQLMasterPos already implemented and used.
* Moved more wfDebug() statements to the 'replication' log.

Change-Id: I423b5fe2da8d97889a6d204a635e351342de7649

Merge "Break long lines and cleanup some RefreshLinksJob checks"

resourceloader: Log if MessageBlobStore failed to fetch a message

Bug: T93800
Change-Id: I6f174251a6b9a6b31921037f672dbc197f3abb7e

objectcache: Reduce MediaWiki-specificness from Memcached classes

Follows-up f36b73e96c, which moved these classes to libs/objectcache.

* Fix wrong @throws in MemcachedPeclBagOStuff.
* Fix wrong @returns in MemcachedBagOStuff::getClient().
* Rename MWMemcached to MemcachedClient.
* Remove mention of 'memcached.php', which doesn't exist anymore.

Change-Id: I34dbc859be4778cea489fd2344f233f30452605c

Merge "Sync up with Parsoid parserTests."

Merge "Removed useless field in SqlBagOStuff"

Merge "Upgrade JobRunner "COMMIT ENQUEUED" log entries to WARNING"

Sync up with Parsoid parserTests.

This now aligns with Parsoid commit b869b084be641d0e2150648ea926ca974590e6e3

Change-Id: I3e8ebd578db8b6630d23ebbdedb5036fa5a092e5

Removed useless field in SqlBagOStuff

Change-Id: I27c0b35769667ce48ba761ff4bbc828033c6849d

Merge "MessageCache: Remove redundant Language::ucfirst() optimisation"

Merge "languages: Avoid getPreferredVariant() in ucfirst/lcfirst unless needed"

Upgrade JobRunner "COMMIT ENQUEUED" log entries to WARNING

Change-Id: I3f3bbb5c3a59975e152b099481dc07617c845b8e

languages: Avoid getPreferredVariant() in ucfirst/lcfirst unless needed

This method calls out to LanguageConverter which involve the User,
Request, and additional validation.

Change-Id: I3edae1244073767a8d8888708024bb5498c70dc9

MessageCache: Remove redundant Language::ucfirst() optimisation

Language::ucfirst() already has this optimisation. The expensive
method is Language::uc(), which is only called when needed.

The ord() < 128 is one of the conditionals in Language::ucfirst().

Change-Id: Idd648b7b84eba1f92dda904438122fe38d3cf22b

Merge "BacklinkCache: Remove use of ProcessCacheLRU"

Merge "Don't apply CSS columns if less than 3 results were found on AllPages & PrefixIndex"

BacklinkCache: Remove use of ProcessCacheLRU

Follows-up af89b09a06 and e40a90f0bc, and general phasing out
of MapCacheLRU and ProcessCacheLRU.

Use of ProcessCacheLRU seems redundant in this case as it is
essentially just a keyed singleton.

Change-Id: I7cc84cf6c76ecc422ea337dba2d216c3d7ebf281

Language: Convert names cache from MapCacheLRU to HashBagOStuff

Change-Id: I3237c8ed97d23e82caa171d936ecb5745a2fe3d5

Title: Convert cache from MapCacheLRU to HashBagOStuff

Change-Id: I0d4c21be1356216970d9ad424ffc848f82778e50

LinkCache: Convert from MapCacheLRU to HashBagOStuff.

Change-Id: I28a9509ad4c6d724a12dc72bc50d3cf58642c651

LinkCache: Minor clean up of documentation and variable names

In preparation for conversion from MapCacheLRU to HashBagOStuff.

* Use early-return pattern for error and rejection cases.
* Add 'int' to @return for getGoodLinkFieldObj() per the
  values set in addGoodLinkObj().
* Add type hints for Title where possible.
* Avoid single-letter variables like $f and $s.
* Add visibility for public static methods.

Change-Id: Iab90539922f648c21cb147f749943a64f598e8de

Localisation updates from https://translatewiki.net.

Change-Id: Ibb56238c1948221039ad18c36cba14af79881847

Merge "Convert LCStoreDB to using startAtomic/endAtomic"

Merge "Revert "Disable Preferences save button before setting change""

Merge "Always defer the log publish() in recordUpload2"

Merge "Make Swift iterators throw errors on failure"

Convert LCStoreDB to using startAtomic/endAtomic

* This avoids throwing atomic section errors when recache()
  is triggered in the middle of transactions.
* Also made the read-only checks handle $wgReadOnly mode.

Change-Id: Ie00cf8454656a6dc3b1862475a959b2af3b472f0

Revert "Disable Preferences save button before setting change"

This solution became broken after all JS started loading async. If desired we
should find a new solution for bug T89457.

This reverts commit b5d46d22b5e49b76fb51fdd7e666c69efb4e1afe.

Bug: T112379
Change-Id: Ia2b527aade8128d1a58af3d42325171f518a68d3

resourceloader: Remove use of msg_resource_links table

This table is not needed because module names and their messages
array are available to the runtime environment at very little cost.

The only purpose it was serving is reverse lookup from message
key to module name (e.g. when MessageCache receives update that
need to propagate to MessageBlobStore). However that is better
achieved by simply looping through modules in PHP. The overhead
of a database is not worth this minor convenience.

MessageBlobStore
* insertMessageBlob: Doesn't need to update msg_resource_links.
* updateModule: Doesn't need to update msg_resource_links.
* getUpdatesForMessage: Reimplement with list from memory
  instead of msg_resource_links.

The database table will be removed in If009e2620e59002e1.

Bug: T113092
Change-Id: Ia9131f570001f00c9800b260ac4b3469d54d2784

Don't apply CSS columns if less than 3 results were found on AllPages & PrefixIndex

If there are less than 3 entries, the browser still tries to render as it would
appear as if there are 3 columns, so the final rendering is broken. So don't apply
CSS columns and just show a normal list if there are less than 3 results. This also
matches the behavior of CategoryViewer.

Bug: T117887
Change-Id: Ie6ac0e1174ff8cc14008f39a91c95bcd6f616353

Merge "Use HTMLForm in Special:AllPages"

Use HTMLForm in Special:AllPages

Renamed namespaceForm to outputHTMLForm and make it protected
Also made sure that this is the only place where it's used
https://github.com/search?q=namespaceForm+%40wikimedia&ref=reposearch&type=Code

Also make HTMLSelectNamespace's work when 'all' is set to null.

Change-Id: Ia559b52464bceaf1202e3a6696728781bb62cdbc

Merge "Fix regression in I24d9b16"

Replace Language::factory( $wgLanguageCode ) by $wgContLang

$wgContLang is filled with Language::factory( $wgLanguageCode ) in Setup.php

Change-Id: I2cb222ee793106140755cf109f9fcc5fcbaa4ddd

Fix regression in I24d9b16

I accidently lost this a CSS block in the transition, and the first
page was missing the prefsection class.

Bug: T115692
Change-Id: I712824cf33bf842e42ca9c6e6660a4cbbf9e6d8e

Localisation updates from https://translatewiki.net.

Change-Id: Ia89debf1a0b89ebd41c1515c0dbb0e27fd378021

Merge "Work around broken HHVM ini_get() for 'upload_max_filesize' and 'post_max_size'"

Work around broken HHVM ini_get() for 'upload_max_filesize' and 'post_max_size'

In HHVM, the settings 'upload_max_filesize' and 'post_max_size' are
not available via ini_get() due to some long-standing bug
(https://github.com/facebook/hhvm/issues/4993). Instead, one can use
'hhvm.server.upload.upload_max_file_size' and 'hhvm.server.max_post_size'
(in a typical PHP fashion, their names are subtly different than the
originals as to increase the potential for confusion).

Added a new method UploadBase::getMaxPhpUploadSize() to handle this.

Additionally:
* 'post_max_size' can be set to 0, which is equivalent to no limit.
  Handle this correctly.
* $wgMaxUploadSize can be an array structure, instead of just a number.
  Handle this correctly by using UploadBase::getMaxUploadSize().
* When no maximum is set, use PHP_INT_MAX rather than 1e100. It should
  be big enough, and the latter is a float, results in 0 when cast to
  int, and doesn't look as pretty when formatted in GB in the interface.

Bug: T116347
Change-Id: Idf707253eeae1b90792a7e26d2ab66d1317e67ae

Rework the Preferences to prevent FOUC

New styles modules that is always added, so that all JS specific styling
is guaranteed to load before first paint. Reworked the HTML to generate
the preftoc (hidden when user has no JS).

Set htmlform nolabel class to use !important, so that it doesn't get
overriden by the 20% width rule of labels.

Also requires changes to the skinstyles of Vector preferences, which
is an a separate patch (I59f0f45), and other skins.

Bug: T115692
Change-Id: I24d9b16ed6729fdf0d59adcc2f0ba16f4f621b44

Merge "mediawiki.jqueryMsg: Update tests to mock magic.SITENAME"

Merge "Add/update phpdoc"

Merge "Remove various unused variables"

Merge "Add class member definition for params"

Merge "Fix casing of function calls in API"

Pass wpExistingTags in the request when multiple revisions are selected

Not doing so breaks the option to remove all tags.

Bug: T102186
Change-Id: I961b29d78ec7dd7a48f3bb4633f9c057125cce5b

Localisation updates from https://translatewiki.net.

Change-Id: I859174f3033d89aac2121387b52c2d25b38b8810

Add/update phpdoc

Change-Id: I9cb6f041a0242c3257a24aac6538f7fbbca60cb0

Remove various unused variables

Change-Id: I4b1b20b4126735cb32a80e473fe48d523bcb24d1

Add class member definition for params

Change-Id: I56cc77bad61691c123ecfae8e004b69141959e57

Fix casing of function calls in API

Change-Id: I44b343568e5713118c41f29a73e112162dd3e635

ApiHelp: Fix undefined variable warning

Change-Id: I5b2790af9182dd76cd52be284f56e4833ac464c6

Merge "Use Arabic numerals in Punjabi (pa) locale"

Use Arabic numerals in Punjabi (pa) locale

A handful users requesting and no opposition.

Bug: T113688
Change-Id: Ie31f0b3a66f389706968860f871086d053c73d3e

TOC: Fix problems on RTL wikis

This is a follow-up to c7894deb.

Bug: T118029
Change-Id: I0cd614063daff20e226a2ba1f819cebd71a2244b

Break long lines and cleanup some RefreshLinksJob checks

Change-Id: I02c007a2c2032610551d71ce1b21e03db5c011db

build: Enable all remaining jsduck jsDoc rules except two

Enabling 'checkAnnotations' with a set of extra tags bespoke to this repo,
and 'checkRedudantAccess' means we're pretty close to compliant in core.
Leaving 'checkRedundantParams' and 'checkReturnTypes' off for now.

Change-Id: I89c4d5250fee30fdfd87a68e173cf222c08cab32

Merge "Hotlink InstantCommons images by default to speed up parsing"