Revert throwing exceptions on preg_* failures

This reverts I3840a56adc0a6e50963b930051892491f8e90245 which
threw exceptions on broken UTF-8 in $text.

Bug: T115514
Bug: T117066
Change-Id: Ie665056a13f9e3678a49790d787b0a41dbba6362

Fix "Notice: Undefined property: Category::$mId" errors

This should also improve performance while at it.

Change-Id: Ib62ce801f597c653d0a8d5fc19b81c1ce8e0f5f2

Merge "Include preg_last_error() in error message when preg_* fails"

Include preg_last_error() in error message when preg_* fails

Bug: T115514
Bug: T117066
Change-Id: I1d13f8542a4cf11c8e87cd4f6a983fa852dfdf1e

Merge "Styling tweaks for inline interwiki search"

Merge "[search] Fix method call on null value"

Localisation updates from https://translatewiki.net.

Change-Id: Ia42d861907521a354efd99d97703c09035927fd2

API: Be consistent in siteinfo showhooks output

This was originally made 'BCarray' to avoid breaking BC, but in this
case the nonstandard output is so rare it's probably best to just be
internally consistent.

Bug: T117022
Change-Id: I501c415f6170383c163110b00e54ac88c3dfec89

Merge "mediawiki.Upload.BookletLayout: Allow for asynchronous initialization"

Merge "TitleWidget: Sort results correctly"

TitleWidget: Sort results correctly

Bug: T117039
Change-Id: Ie86c1751dacb5c32afe708b354809b914a54ea51

Remove ORM code from core

Bug: T114538
Change-Id: Ic4196ca9da927fc5c85b01cfff65f0636e3202ae

Merge "Add tests for MemcachedBagOStuff::validateKeyEncoding"

Merge "Convert buildSidebar() to using getWithSetCallback()"

[search] Fix method call on null value

Seeing these pop up in the production logs. This can be null under
certain circumstances, so expect and handle it.

Change-Id: I25a355174e0ad93280eccffd4ec711a641d14bf3

Language: Remove logic for class file inclusion

* Remove strcspn() check in newFromCode already in isValidCode().
* Leverage the autoloader instead via class_exists instead of
  including files based on user input.
* Create fallback instance directly instead of recursing back
  into newFromCode().

* Remove method preloadLanguageClass (unused).
* Remove method getClassFileName (unused)

Change-Id: I90035ca4b07facae051b1a584e92df72b42c4a49

Convert buildSidebar() to using getWithSetCallback()

Change-Id: Id9a27ba2bbd3aceee26bf35844d1c970dbb32d47

Merge "Fix slow callbacks in getWithSetCallback() using lockTSE"

Client-side migration for empty li preservation

It is desirable in terms of user-friendly syntax to display an empty
list item if the user adds one to the source. However, we suspect that
this change will break the rendering of existing templates. So, preserve
the empty <li> element, but style it with display:none so that there is
no user-visible change. Changes can then be observed with a user script,
then eventually the CSS can be removed so that the desired behaviour will
be user visible.

This is imagined as a staged deployment of T89331, i.e. it is better to
resolve differences with Html5Depurate one at a time instead of
deploying it all at once.

The CSS module is specified in parser/MWTidy.php since the tidy driver
hierarchy is not meant to be so closely tied to the MW environment.

Bug: T49673
Change-Id: Ifb44b782c617240e3de73dcdf76c8737c7307d94

Fix slow callbacks in getWithSetCallback() using lockTSE

* Keys that take a long time to generate would run into
  the MAX_SNAPSHOT_LAG check and have set() no-op. This
  would be fine except that leaves no key there to figure
  out the time since expiry and therefore whether to use
  the mutex, so it didn't. This now saves the keys but with
  a FLG_STALE bit set, making the next caller that sees it
  perform a regeneration (unless it can't get the mutex).
* Bumped LOCK_TTL so that keys can stay locked much longer.
* This is easy to test via sleep(5) in tagUsageStatistics()
  and two Special:Tags browser tabs.

Bug: T91535
Change-Id: I549e70ace3d2e7da40d3c4346ebacc36024cd522

wikimedia/running-stat 1.0.0 → 1.1.0

* 9d7170d3ee: Rename RunningStat::push() to RunningStat::addObservation()
* 09b6e53be9: Add PSquare class for online percentile estimation

mediawiki/vendor change: I48dd51c9068

Change-Id: Ie6b19833c282e0f493c4d306807366a6671455a8

mediawiki.Upload.BookletLayout: Allow for asynchronous initialization

It happens to be convenient, since it turned out that we need to fire
a few API requests to correctly render some bits of the UI.

Follow-up to 86dedeea7f7cae5452acf5162672f34c32b593fd. This makes the
code less hacky and more reliable.

Change-Id: I65a92e3915a6844056fb7b3a2758a168eb12e6b8

Fix diff/history links not showing up for ungrouped enhanced RC

Looks like a typo or accident from d40cd42b9f960c.

Bug: T116899
Change-Id: I33b65afa78480612b9312282922686a64346fef2

Localisation updates from https://translatewiki.net.

Change-Id: I29f42fc05b323dacc3dd33ebbd26bf4e50c25bfa

HTMLRadioField: Use another variable for the radio value to avoid conflicts

$value is used in a foreach loop to build the radioselect fields array for OOUI,
but $value is also the variable, which holds the default value for the element
(or a value, which ws submitted by the user), to pre-select the correct radio input
field. That results in the problem, that $value will always contain the last data,
which passed the foreach loop, and always the last radio input field will be pre-
selected, no matter, what the developer defined as the default or what the user
submitted in a prior form instance.

Change the variable used in the foreach loop to avoid this conflict.

Change-Id: I36c760be8ccd86e75249c966f697b499576c83d4

Add tests for MemcachedBagOStuff::validateKeyEncoding

If3e20c6 and the following patches introduced a breaking change and
cause a regression in Wikibase because we are using the version number
constant as part of a cache key prefix. Currently the Wikibase version
is set to "0.5 alpha".

Space characters were allowed before and encoded as "%20". This does
not happen any more.

Change-Id: Ia2fd4ed6738a10e02050bced947ef5d4e8b98980

Merge "Make WAN cache HOLDOFF_TTL smaller by combining db/snapshot lag"

Merge "resourceloader: Consistently refer to the framework as ResourceLoader"

Merge "resourceloader: Remove 'loaderScripts' option from FileModule"

Merge "Add help link to Special:Search"

Merge "Convert DjVU cache to WANObjectCache"

Merge "Improve WAN cache delete() docs a bit"

Merge "Bump development Monolog version"

Improve WAN cache delete() docs a bit

Also use more TTL constants in getWithSetCallback() examples

Change-Id: I866f0464c07fa0c8977c6e908e92186b2ef9de43

Convert DjVU cache to WANObjectCache

Also unified the total and per page dimensions cache

Change-Id: Ib6ad7c3cdfb4fe3a88d25240c3e362664a9b41fd

Convert wfGenerateThumbnail() away from $wgMemc

Change-Id: I06c2d90cb3e75142f781afa80f917d4155513a22

Merge "objectcache: Introduce IExpiringStore for convenient TTL constants"

objectcache: Introduce IExpiringStore for convenient TTL constants

Also consistently use self:: instead of BagOStuff:: for constants
referenced within the BagOStuff class.

Change-Id: I20fde9fa5cddcc9e92fa6a02b05dc7effa846742

resourceloader: Consistently refer to the framework as ResourceLoader

Change-Id: Ia59e4eac9662723e80d62f7cfcb9e4292e3ee4de

resourceloader: Remove 'loaderScripts' option from FileModule

Not used in any of our public repositories.

Bug: T65240
Change-Id: I1e9f741c3ef0f922129ecd10039228b58565bf62

Merge "set serialize_precision to 17, current PHP default"

LinksUpdate: Keep track of the triggering User

So extensions like Echo are able to attribute post-edit link updates to
specific the users who triggered them.

Bug: T116485
Change-Id: I083736a174b6bc15e3ce60b2b107c697d0ac13da

Bump development Monolog version

Related mediawiki/vendor change: I6d6db4c

Bug: T116550
Change-Id: I71b2051c4c6193a435183ed00da52e9dc6c0673c

Localisation updates from https://translatewiki.net.

Change-Id: Ib468dabe3f7d642b58087711945f6026f6d143c8

Replace local RunningStat with wikimedia/running-stat library

* mediawiki/vendor change: I66b419c8168df.
* Add back-compat shim in includes/compat/RunningStatCompat.php.

Change-Id: Ibad4dc0ce5aa8697791865ed8ea61a25bae7b3e0

Merge "Update OOjs UI to v0.13.0"

Make WAN cache HOLDOFF_TTL smaller by combining db/snapshot lag

* In the common ~0 second lag case, transactions up to ~7 seconds
  long will have normal set() behavior (instead of just ~5 seconds).
* Like wise with ~0 second transactions tolerating ~7 seconds of
  lag (instead of just ~5).
* The lower hold-off time lets caching resume 3 seconds sooner.

Change-Id: I21e2a0a6915492cec422b6a6af5541937745c15b

Restore TitleMoveComplete by-reference flags to avoid errors

They should be removed from extensions, but they don't
really hurt much for now.

Change-Id: I64b570e376062155afdf879c3ce85560127c6792

Update OOjs UI to v0.13.0

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

Change-Id: Ie4f66b15f4f5ec9932d5fd72e12b4453724e3742

set serialize_precision to 17, current PHP default

Follow up to:
https://gerrit.wikimedia.org/r/#/c/249018/

Bug: T116683
Change-Id: I6e8dfab473ae2c18e68c8e4360159c9cc0e09758

Merge "Get rid of CSS transitions on form elements in mediawiki.ui"

Merge "Use ImportStringSource for simple import sources"

Merge "Revert "Enforce stricter slave lag limits for bot API requests""

Revert "Enforce stricter slave lag limits for bot API requests"

This broke most Wikidata API PHPUnit tests.

This reverts commit dc10216e77b06b0fc7bb5e0824eb0967129c2e1f.

Change-Id: I19285ff0eddffb38cdbf45bbd05ba2d8a1816b41

Merge "Convert ForeignApiRepo::httpGetCached() to the WAN cache"

Merge "Completely defer EditPage::updateWatchlist"

Merge "Make Swift backend respect Content-Type in create/store"

Merge "filebackend: Remove some old Content-Disposition b/c code"

Merge "Enforce stricter slave lag limits for bot API requests"

Merge "Measure commitMasterChanges() run time"

Merge "Protect WAN cache sets() against uncommitted data"

Merge "Fixes related to WikiPage::triggerOpportunisticLinksUpdate()"

Merge "Follow-up to Ie3499d"

Merge "resoureloader: Consolidate styles-only queue at the top"

Follow-up to Ie3499d

My first pass at what became Ie3499d9f2 was to keep the exception for duplicate
registrations with conflicting parameters but downgrade to a warning if the
registration attempts are identically parametrized. I then removed the
exception entirely, but left the if-block, which does not make much sense, as
it means we pass over silently module registrations that set different
parameters, and throw a warning only when they are the same.

Change-Id: I13da33faf2f058ac75ef4b1615c4fafc18f14505

Merge "mediawiki.jqueryMsg: Lazy-initialize parsers with getFailableParserFn()"

Styling tweaks for inline interwiki search

To make this look reasonable had to adjust the other styling around
it, I think this now makes more sense too

* moved p tag top margin to .searchresults, the only p tag is at the top
  of search results anyways,
* moved the left margins on every element inside .searchresults
  to .searchresults for consistency
* removed bottom padding from .searchresults p completely, this is
  unnecessary. The top margin of .mw-search-results (.3em) along
  with the bottom margin of generic p tags (.5em) is plenty on its own
* strip the top margin on mw-search-interwiki-header when it comes
  immediatly after .mw-search-nonefound for better spacing to match
  mockup

Bug: T112349
Change-Id: I3dec110e1ac0ab9c3dab1a3f7e6c2b747f2e28c8

Merge "mediawiki.jqueryMsg: Remove 'astCache'"

mediawiki.jqueryMsg: Lazy-initialize parsers with getFailableParserFn()

Because they rely on data set by setParserDefaults(), and that is only
called later (from ResourceLoaderJqueryMsgModule).

Bug: T115708
Change-Id: I085d27b1b00873bc5b22573ef7eb9d17b8e5e1fd

mediawiki.jqueryMsg: Remove 'astCache'

All it does is cause problems.

Bug: T54042
Change-Id: I955ee61f6c0040203959ab8f45efb56a32431768

registration: Support custom classes in "ResourceModules" schema

Instead of expecting everything to be a ResourceLoaderFileModule, use
the "anyOf" property to validate against multiple schemas. The following
schemas are now allowed:
* The ResourceLoaderFileModule schema, now with the "targets" property
* A schema for ResourceLoaderImageModule
* A generic schema that requires the "class" property is set, but not to
"ResourceLoaderFileModule" or "ResourceLoaderImageModule".

The last schema will allow for any custom ResourceLoaderModule class to
be set in extension.json with arbitrary parameters.

One downside of this is that the error messages shown when a file does
not validate get a little more confusing, as it shows the error messages
for each schema it does not match.

Bug: T105236
Change-Id: I5e4bfa69c733187c7b27294c159cac05b3b92e81

Merge "Improve DBReadOnlyError/DBExpectedError page title"

Completely defer EditPage::updateWatchlist

This should reduce their profile time shown on xenon

Change-Id: I8e92de22755b592cc7b2d3fab36cff2761ab2bb7

Merge "Reset serialize_precision to stabilize tests"

resoureloader: Consolidate styles-only queue at the top

This effectively reverts d6b4d3c537 and declines T97420.
This was previously attempted in b7c0e537eb.

Drop support for position "bottom" for addModuleStyles().
This feature was only recently introduced with the intent
to optimise page load performance, but had an adverse effect.

It increases chances of FOUC due to late discovery of these styles.
It also caused minor problems for some gadgets and extensions
that did not or were unable to set these flags. Some mobile
code was introduced around the same time as this feature and
was never given position=top.

Stylesheets that don't affect initial render or are only needed
on interaction should be loaded via addModules() instead; which
is handled by the asynchronous loader in JavaScript.

Change-Id: Ib9821b7b87cfc8a59062bb6ca358974fdb01ced1

Merge "Language::truncate(): don't chop up multibyte characters when input contains newlines"

Get rid of CSS transitions on form elements in mediawiki.ui

They seem to serve no function whatsoever other than to irritate and delay the
loading of the page.

Change-Id: I54c195541713a5bfbc8bd7078499ac49671697c6

Language::truncate(): don't chop up multibyte characters when input contains newlines

To detect whether the truncation had chopped up a multibyte
character after the first byte, a regex was used. But in this
regex, the dot (.) didn't match newlines, so it failed to
detect chopped multibyte characters (after the first byte)
if there was a newline preceding the chopped character.

Bug: T116693
Change-Id: I66e4fd451acac0a1019da7060d5a37d70963a15a

Merge "Migrate more callers away from $wgMemc"

Merge "Remove begin/commit transaction calls in refreshLinks.php"

Remove begin/commit transaction calls in refreshLinks.php

these sometimes lead to issues with database deadlock loops
and probably are not needed here.

Bug: T75456
Change-Id: Iad03f6076d8e319ff02cb26259ec9705105e92a7

Merge "Displaying search results for multiple wikis"

Reset serialize_precision to stabilize tests

With serialize_precision = 100 the following
failures were generated:

1) ApiFormatPhpTest::testGeneralEncoding with data set #7
(array(1.0E+42), 'a:1:{i:0;d:1.0E+42;}', array(1))
Failed asserting that two strings are identical.
--- Expected
+++ Actual
@@ @@
-a:1:{i:0;d:1.0E+42;}
+a:1:{i:0;d:1000000000000000044885712678075916785549312;}

/usr/home/saper/public_html/ybabel/tests/phpunit/includes/api/format/ApiFormatTestBase.php:61
/usr/home/saper/public_html/ybabel/tests/phpunit/MediaWikiTestCase.php:137

2) ApiFormatPhpTest::testGeneralEncoding with data set #30
(array(1.0E+42), 'a:1:{i:0;d:1.0E+42;}', array(2))
Failed asserting that two strings are identical.
--- Expected
+++ Actual
@@ @@
-a:1:{i:0;d:1.0E+42;}
+a:1:{i:0;d:1000000000000000044885712678075916785549312;}

/usr/home/saper/public_html/ybabel/tests/phpunit/includes/api/format/ApiFormatTestBase.php:61
/usr/home/saper/public_html/ybabel/tests/phpunit/MediaWikiTestCase.php:137

Bug: T116683
Change-Id: I5e760b4632d2ab326954fe5769476e143c376d7c

JavaScriptContentTest: $wgScript != $wgScriptPath

Set $wgScript, $wgScriptPath and $wgResourceBasePath
to avoid failures when running on a wiki with
a non-standard $wgScriptPath.

Reported-on:

https://lists.wikimedia.org/pipermail/wikitech-l/2015-October/083675.html

Change-Id: I8acbcca5449060ff5604bf275f690b53343e706e

Improve DBReadOnlyError/DBExpectedError page title

Previously it was just "internal error".

Change-Id: I70d81dfc628b462ab525d83f4a20d899da8ca143

Merge "LoggerFactory: Only check for Psr\Log\LoggerInterface once"

Localisation updates from https://translatewiki.net.

Change-Id: I177066ee20e6c31810845cadf7ce6fef9d2ad1ce

Merge "Convert doDeleteArticleReal to startAtomic()/endAtomic()"

Merge "Convert MovePage to startAtomic()/endAtomic()"

Merge "PrefixSearch: avoid looking for titles that don't exist"

Convert doDeleteArticleReal to startAtomic()/endAtomic()

* They no longer commit the update, but just make sure
  it is part of a transaction. The BEGIN/COMMIT will
  happen at request start/end given DBO_TRX or in this
  method otherwise (like when in CLI mode). This avoids
  premature committing of other transactions.
* FileDeleteForm was the only caller using $commit=false
  for WikiPage::doDeleteArticleReal() and its proxies
  WikiPage::doDeleteArticle() and Article::doDeleteArticle().
  The ugly $commit flag is now removed.
* No caller was using $id for WikiPage::doDeleteArticleReal()
  and its proxies WikiPage::doDeleteArticle() and
  Article::doDeleteArticle(). It is now removed and we can
  be sure the lock() and CAS logic always hit in the method.
  The rollback() calls are not needed given the page row lock
  and having them there could break outer transactions.
* Updated FileDeleteForm to use startAtomic()/endAtomic() so
  the article and file delete are still wrapped in a
  transaction. Note that LocalFile::delete() does reference
  counting and trxLevel() checks so it will not try to begin()
  and break FileDeleteForm's transaction via startAtomic().
* Moved less important 'page-recent-delete' key update down
  for sanity in case it blows up.

Change-Id: Idb98510506c0edd02236c30badaec97d86e07696

Migrate more callers away from $wgMemc

Callers should use more expliciy lazy-loaded
cache factory methods.

Change-Id: Ifa0bf389720a09a931ee6466b993f787d83a09a7

Merge "Update grunt-jsonlint to 1.0.5"

Merge "ApiBase::PARAM_DFLT => null is the default anyway"

LoggerFactory: Only check for Psr\Log\LoggerInterface once

LoggerFactory::getInstance() will be called many times during the course
of handling a typical MediaWiki request. The interface_exists() guard
condition it uses is an attempt to provide an informative error message
when Composer managed libraries are not installed. This check is only
needed on the first invocation of getInstance() to be effective. Using
an additional boolean to guard the interface_exists() call will allow
the PHP runtime to avoid a potentially expensive (at least compared to
a static boolean comparison) function call.

This is the sort of thing that smells of premature optimization, but its
addition is in fact informed by examination of performance reports from
the Wikimedia production environment.

Bug: T115729
Change-Id: I437bcb5326b06145081f2b86f6c4d0c8dc1a318c

PrefixSearch: avoid looking for titles that don't exist

WikiPage::getRedirectTarget() can very easily return null. Don't
blow up when that happens.

Properly document PrefixSearch::getRedirectTarget() while we're
here.

Bug: T116029
Change-Id: Ib509e8e3e6ec04a787de3857a4b5ee9b01560559

resourceloader: Deduplicate module registration conflicts

If an attempt is made to register a ResourceLoader module which has already
been registered, instead of bailing with an error, issue a warning instead, and
allow the lattermost set of module parameters to stick.

Because modules are registered from extensions, and because modules are often
migrated from one extension to another as extensions are refactored, and
because the developers of different extensions are not always aware of each
other's work, throwing an error runs a high risk of causing an outage.

Bug: T116628
Change-Id: Ie3499d9f2ff3bf071f1c2b0176e19440ff55211c

filebackend: Remove some old Content-Disposition b/c code

Change-Id: I6af4a84bd549e2bfb1b110fa790f499d144f236b

Make Swift backend respect Content-Type in create/store

Change-Id: I479d1fc4f261bdcdc380404db0cb2f2397877759

Merge "Make DateFormatter::getInstance use APC"