Merge "Make runOnTransactionIdleCallbacks() handle dangling transactions"

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

Localisation updates from https://translatewiki.net.

Change-Id: Ic027cac2ca0839cbb6083078c5a90eabee280510

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

Merge "Various JobQueue test tweaks"

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

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

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

Enforce stricter slave lag limits for bot API requests

* All users with the bot right sending requests to
  write APIs will get a read-only error when slave
  lag is high. The lag tolerance is configured via
  $wgAPIMaxLagThreshold.
* Making this lower than 'max lag' in the LBFactory
  config means bots get read-only errors before other
  users, which is sensible.
* The default LoadBalancer 'max lag' is now 10 instead
  of 30, which now matches the timeout used for waiting
  on MASTER_POS_WAIT() for ChronologyProtector. This is
  better for assuring that users see their own changes.

Bug: T95501
Change-Id: I56a7f3538278378c67fa3ef5ff0ba4aa6a9a8c64

Merge "Add triggeringRevisionId to LinksUpdate JobSpec"

Always defer the log publish() in recordUpload2

This is more performant and simplifies the code a bit.

Change-Id: I7a30e000ae4eff47519b83eb6bdde57eeb8a18f6

Make Swift iterators throw errors on failure

This is in line with FSFileBackend and is provides the calling
loop of a way to know the listing failed, rather than just
giving incomplete information.

Change-Id: I34c66d9dbf84e5f33982d17e61364c1595d3e9f7

Merge "Allow AUTHOR|CREDITS with .{media,}wiki extension"

Merge "Bump composer-merge-plugin to v1.3.0"

Bump composer-merge-plugin to v1.3.0

Update composer-merge-plugin and enable the new feature of omitting
"require-dev" sections from merged files. This will help prevent
Composer package conflicts when used with extensions that have dev
requirements which are not reconcilable with those of MediaWiki itself.

Requires: I7453f68e671b9d519ec3f7dfaa22de6ec8f252e9
Change-Id: I768cf21c6601496e34f2b63199df04cbc0090145

Add header to flag API POST requests with no write intentions

This performs sanity check that request *is* for a non-write module.
By handling the validation, the CDN layer can simply use the presence
of this header to route POST requests to the local datacenter.
Without validation, users could cause strange traffic patterns and slow
cross-DC database writes (which can involve many RTTs).

This is useful for AJAX widgets that need to post a payload to get a
response, but that don't actually change anything in the process. They
should be able to use the local datacenter.

Bug: T91820
Change-Id: I34248ddee33033e3d1d86c3391236259d917d4a7

Localisation updates from https://translatewiki.net.

Change-Id: Iaf10c5bedfc505c5891bf32c95479cc52bb189dc

Hotlink InstantCommons images by default to speed up parsing

The configuration, tested on wiki.wikimedia.it, generally improves
performance for all parties involved.

Bug: T114098
Change-Id: I76a34e8782908a28523531b2a928ea4ef7710b19

Log errors in thumb.php

Add new streamFileWithStatus() methods to FileRepo and
MediaTransformOutput that can be used to get more detailed error
information on failure. The historic streamFile() methods become sinple
wrappers to the new methods. Thumb.php is changed to use the
streamFileWithStatus() methods so that failure reasons can be logged.

Change-Id: I3088cde2044a7ff00841e53ca252d0b222c8b518

mediawiki.jqueryMsg: Update tests to mock magic.SITENAME

Change-Id: I75f92986dc0e44c85e592ab0bb969431993259fd

Merge "Remove 'bot' check before trying the edit stash"

Merge "Convert JobQueueGroup::getCachedConfigVar to using WAN cache"

Merge "Make edit link conditional on Special:BrokenRedirects/DoubleRedirects"

Make edit link conditional on Special:BrokenRedirects/DoubleRedirects

When an user has no permission, do not show an edit link.
Also do not show an edit link when content model does not allow direct
editing, like WikiData items.

For performance reason the special pages now run a LinkBatch to collect
the content model information.

Bug: T117900
Change-Id: Ieb56a6f314059356b95bc045f3d255930d54f213

Merge "Implement section support for OOUIHTMLForm"

Remove 'bot' check before trying the edit stash

Some user groups, like the 'flood' one on wmf sites might
get caught up as false positives.

Change-Id: I31be62b2239477572bc063f1af0329f248bbcaf6

LocalisationCache: try harder to use LCStoreCDB

When the LocalisationCache class is set to 'detect', we prefer LCStoreCDB if
$wgCacheDirectory is set, but we default to LCStoreDB otherwise. Rather than
give up, we should try wfTempDir(), since any directory that meets its criteria
is also suitable for LCStoreCDB.

Change-Id: Id3e2d2b18ddb423647bf2e893bcf942722c0e097

Improve mismatched atomic section errors

Change-Id: I9b1561b6426ac00f72867452bee396b649ad8bfd

Allow AUTHOR|CREDITS with .{media,}wiki extension

With this fix, Special:Version shows "and others" for MobileFrontend
authors, and Special:Version/Credits/MobileFrontend works.

Bug: T117932
Change-Id: I80bc334aba80d3621416cfec1a13db4872ac53cf

Merge "Title: Add warning if newFromText is given non-string/non-null value"

Localisation updates from https://translatewiki.net.

Change-Id: I14a339b32cc0ce21e1d8ed571a64f01fdcfb726e

Title: Add warning if newFromText is given non-string/non-null value

Specify null as allowed value to make this method more easy to use.
The method is often used like that already and this makes it official.
Previously we were on a months-long trajectory to try and remove all
instances of passing null, but it was a never-ending quest (with new
ones introduced all the time). And in all cases the added handling
in the caller is the same as the handling for when this method
returns null. Requiring the caller to bounce null makes code more
complex, duplicates logic and serves no reason. Let this method be
a non-throwing validation method that returns null if given null.

Bug: T76305
Change-Id: I47517021471ac6fd6daca6deee039c8047224a33

Merge "mw.ForeignStructuredUpload: Link author's username to their user page"