Merge "AuthManager: Break AuthPlugin::addUser more explicitly"

Localisation updates from https://translatewiki.net.

Change-Id: Ib93bbea5f21ed351de36fdf88f864d6f0a678da0

Merge "resourceloader: Track state of page-style modules"

resourceloader: Track state of page-style modules

This allows dynamically loaded modules to depend on page-style modules
without it causing the page-style module to be loaded a second time.

* New method Module::getType() indicates whether a module is
  a page-style module or supposed to be dynamically loaded.

* Emit warning from addModuleStyles() when given a module that is
  not a page-style module (to be enforced later)

Bug: T92459
Bug: T87871
Change-Id: I8b6c6a10d965e73965f877c42e995d04202524f3

Revert "mediawiki.toolbar: temporary workaround for T113868"

No longer needed after 280d29215a89ecc68cf860a25f28f7091707a107.

This reverts commit 08cb9076d4903578591c61fff93905d9bddd7237.

Bug: T113868
Change-Id: I480f70fdc223db9c8083e7ee5b5be8d91a8c60c0

Merge "DumpStringOutput: Rename getOutput() to __toString()"

Merge "registration: Convert "config" into an object with metadata"

DumpStringOutput: Rename getOutput() to __toString()

Though getOutput() is what first came to mind, I do not particularly
like the name, partly because it is used in many, many places as a
method that returns an OutputPage object.

* Renamed the method to __toString(). This is appropriate because
  each instance, at any given time, corresponds to a single string
  value (and exceptions cannot occur during this conversion).
* Removed unnecessary variables from ApiQuery and ExportTest. In
  these and most other cases, it should no longer be necessary to
  call getOutput() explicitly.

Change-Id: Icf202743d1f332f8981338f42eb6e3e5a04abdf1

Add Shan (shn)

Bug: T135867
Change-Id: I6cd17284b82f6cab0b1fe12557005f0ac541f1e9

SpecialLog: Add help link

Bug: T140013
Change-Id: Ief7b596e9972911106a55a4d2353a63a1ca0a436

registration: Convert "config" into an object with metadata

To add extra metadata for "config" options without constantly adding
hacky underscore prefixed keys, convert "config" items into an object,
where the value is under a "value" key.

"_merge_strategy" is now just "merge_strategy", but still has the
underscore prefix for the cache structure, to avoid changing it.

Since this is a fully breaking change, it only applies to files with
manifest_version: 2 set. A conversion script has been added to assist
with automated conversion.

Bug: T133626
Change-Id: Id1071fc0647892438e5cd0e3ee621fbdaaa64014

Localisation updates from https://translatewiki.net.

Change-Id: I9aa72a1de8d331e8f2b2b832a50effc8c97a8763

Merge "mw.widgets.CategorySelector: Avoid JS exceptions for invalid user input"

Merge "API: Remove index forcing in ApiQueryRecentChanges"

Merge "Don't use $wgContentNamespaces directly"

Don't use $wgContentNamespaces directly

Change-Id: If8ed1bf7e7c6429c405c956413af07a191ec9c40

MWExceptionHandler: Fix PHP7 compatibility

The `Exception` type-hint will cause issues if a PHP7 `Error` is thrown.

Change-Id: Iab0af90235badd57304131ae2274a1cfa0be822a

Rewrite TidySupport and add option --use-tidy-config

* Have TidySupport provide $wgTidyConfig instead of the legacy globals
* Add --use-tidy-config option to parserTests.php. This tells
  TidySupport to use the tidy configuration from LocalSettings.php
  instead of the traditional safe defaults.
* Add a way for TidySupport to disable tidy via $wgTidyConfig, using
  driver=>disabled

Change-Id: Ie76e68e2d5238d0a1aef49a1a815c0d1cd8bfdae

Update OOjs UI to v0.17.6

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

Change-Id: I82b26aa68c7402346c449cafb3f6ec9b772bc5c6

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

Merge "Balancer: Inline BalancerStack::length()"

Merge "Balancer: remove all Assert::parameterType() calls"

Localisation updates from https://translatewiki.net.

Change-Id: Icab7d562d8131e2c59b5f3d7c51d2a94f0ea3945

Merge "Balancer: Introduce BalanceElement::isHtmlNamed()"

Merge "Balancer: cache BalanceStack::currentNode()"

OutputPage: Make use of WrappedStringList in headElement()

Right now, getInlineHeadScripts(), buildCssLinks() and getExternalHeadScripts()
all return WrappedString::join(). But because they don't know about each other
and because they need to return strings (not arrays), headElement() has no way
of merging them.

WrappedStringList allows this array to be kept, whilst still being backward-compatible
with code that calls these methods and assumig a string (since it will lazy-join
the array if the object is treated like a string).

To be used by I8b6c6a10d965e7396. Output is not changed in this commit.
Merely refactoring.

Change-Id: Iae08345473bd93cc0948d51b62c48aeb1ea460a3

Merge "Add any prior block to BlockIpComplete hook"

AuthManager: Commit transaction after auto-creating a user

As things stand now, DBO_TRX or DBO_DEFAULT will cause a transaction to
be started, and then anything in the entire request failing will
probably cause the newly-added user row to be lost. But updates to
external databases (e.g. CentralAuth) likely won't be since those DB
connections were probably shut down after the update was completed.

So let's explicitly commit changes af the end of auto-creation so that
problems with the request itself don't undo it.

Bug: T119736
Change-Id: I6c13c8feb86d8b9a01df894733c38445d048fea0

API: Remove index forcing in ApiQueryRecentChanges

It's preventing better indexes from being chosen; hopefully removing it
doesn't cause worse indexes to be chosen anymore in other cases.

Bug: T140108
Change-Id: Iee480e221dc26f89e8583be5f26c6571b240e390

Rename 'slider' gallery mode to 'slideshow'

Bug: T140093
Change-Id: I6f01344b9ac61e3d2a8e7a9d920ba37786537ff8

Balancer: Inline BalancerStack::length()

Provides 1% reduction in benchmark time

Change-Id: Ie8ff66a836cd137234828effcce9547e2cb3cd58

Balancer: remove all Assert::parameterType() calls

Profiling shows they are especially expensive. 29% reduction in
benchmark time.

Change-Id: I5206b05007c7e1d6552974bcd7c57aa03eea231d

Balancer: Introduce BalanceElement::isHtmlNamed()

An optimised special case of BalanceElement::isA, reducing benchmark
time by 4%

Change-Id: I1204de9454eb7b8f9f3a5ed137218c3293f9ab27

Balancer: cache BalanceStack::currentNode()

Make currentNode into a public property instead of a function call, for
a performance improvement of about 4%.

Change-Id: I89861557531c55a63abef52c0acabbfb5c155bda

Some Balancer improvements for performance and compatibility

* Use a doubly-linked list for the AFE list, instead of an array,
  allowing efficient insertion and removal from the middle, and trivial
  O(1) lookup of existing elements.
* Use a hashtable of singly-linked lists for storing Noah's Ark buckets,
  instead of iterating through the entire AFE list on every push.
* Store attributes in an array instead of serializing them in the
  tokenizer. This allows us to avoid sorting them in the output. For the
  Noah's Ark clause, the array is copied and then sorted on demand.
* XHTML-style serialization with self-closing tags.
* Clear the AFE list in stopParsing(), otherwise all the BalanceElement
  objects are kept alive until after serialization, thus using O(N^2)
  memory (in stack depth N) since the full serialization is stored at
  each stack level.

Change-Id: I517129c0658f03eb2ddee61fdf33ffe6fbd48509

Hook up Balancer as a Tidy implementation.

This is an HTML5-compliant parse/serialize tidy implementation, with
well-delineated hacks to support the <p>-wrapping done by legacy tidy.

Change-Id: I4fd433fd6f1847061b0bf4b3e249c918720d4fae

HTML5 Balancer

This adds an implementation of the HTML5 Tree Builder algorithm to PHP,
along with test cases from the tree builder derived from the
html5lib-tests package on github.  The test cases were preprocessed
into JSON for the `domino` HTML5 parser, and we're using the JSON
form of the tests.

The implementation follows both the language of the HTML5 specification
and the implementation in `domino` very closely, easing updates if the
specification changes.

This code is used in follow-on commits to support an HTML5-based
"tidy" for mediawiki and the `{{#balance}}` parser function, which
ensures that a template expands to properly-balanced HTML, with all
tags closed and nothing left on the HTML active formatting elements
list.

See: https://github.com/fgnass/domino
Change-Id: I6f4d20a43510dd819776bb333b639315b19d150d

Add tracking category when editors use the deprecated self-closed tag hack.

Some pages use constructs like `<b/>` or `<span/>` to protect spaces or
special characters at the beginning/end of templates.  This syntax is
incompatible with HTML5 parsing rules, which dictate that these should
be treated as open tags, and instead rely on an unusual quirk of the
`tidy` program that removes invalid constructs.

This syntax is deprecated as part of the process of reconciling `tidy`
with modern HTML5 parsing semantics.  Authors can use `&#32;` or `<nowiki/>`
as valid replacements.

In order to provide time to transition existing content, pages using
self-closing tags in violation of the HTML5 parsing specification
will have their templates/pages added to a new tracking category.
After these uses are fixed, we will change the sanitizer to treat these
as normal open tags, to be consistent with the HTML5 parsing spec.

Note that this construct is already disallowed if tidy is disabled; it
is rendered as `&lt;b/>`.  We add a tracking category in the no-tidy
case as well, in preparation for eventually making the no-tidy and
with-tidy behaviors consistent.

Bug: T134423
Change-Id: Ie1cf3aa40d5483bf395ece539f0240b694ff04ab

Merge "Improve comments on fields and fix opening_text - needs no highlights."

Localisation updates from https://translatewiki.net.

Change-Id: Ic1d591a09a31a027c0799e6833a302bb4de9477e

Merge "mw.Upload.BookletLayout: Better handle error messages from AbuseFilter and TitleBlacklist"

Merge "ApiUpload: Better handle ApiMessage errors from UploadVerifyFile hook"

Merge "Move span.comment back to shared.css as its more widely used"

Move span.comment back to shared.css as its more widely used

...than the other changeslist CSS classes. This commit partially
reverts 9e458504c7b4.

Bug: T139722
Bug: T139832
Bug: T139833
Bug: T139835
Bug: T139836
Bug: T139837
Bug: T139838
Bug: T139839
Bug: T139840
Bug: T139841
Change-Id: Ia4a7b6d0d19f0f62cd2402cd81f944ab31fb84c0

Merge "build: Bump grunt-karma and related tools to 1.0.x"

Update documentation for I30e359fb

Change-Id: I3a13c02ecaf76ac9affcae1a895a6194466c7658

build: Bump grunt-karma and related tools to 1.0.x

Also bump karma to 1.1.0 see changelog at
https://github.com/karma-runner/karma/releases please

Change-Id: Ic982dfc7ccebeecd6082b5476372cbb87fec5f1b

Merge "Make classes actually work in lines in EnhancedChangesList"

Merge "API: Allow anonymous CORS from anywhere, when specifically requested"

Merge "ApiQuery: Don't mess with PHP output buffering"

Merge "Use READ_LATEST for the WikiPage in RefreshLinksJob"

Merge "Fix grammar in transaction-duration-limit-exceeded"

Merge "Use more short array syntax in comments (/tests/)"

Merge "Parse sidebar message for main page when cache is enabled"

Merge "Revert "If rev_content_model is null, use page_content_model.""

Revert "If rev_content_model is null, use page_content_model."

This patch effectively reverted a5bc9f49cd01e49c, which
was the correct behavior. If a page were to have its content
model changed, the NULL revision rows would now be interpreted
as the page's new content model instead of the default content
model (which they should be).

This reverts commit 9a6ff0bd82faafbf9746a6fa3ed33f42092587c3.

Change-Id: I6801d09bd847f477222e503b912005b02a9df045

Merge "Remove old WebResponse::setCookie() calling method"

composer/semver 1.4.0 -> 1.4.1

https://github.com/composer/semver/compare/1.4.0...1.4.1

Depends-On: I29afc0187f0fe0e920c27710ff06ff1d502896e6
Change-Id: Idd09dcee04c5781946079bcec087d15186fb1bea

Parse sidebar message for main page when cache is enabled

When the sidebar cache is enabled, parse the messages for the links and
link text of the sidebar with the main page as title, because variable
like FULLPAGENAME would vary on page, but cache does not.

Change-Id: I0bbb05f1abe4be1a61f520d647c8ca696d184ccf

Localisation updates from https://translatewiki.net.

Change-Id: I5b1be9bfed1741466b7364c6314f8cfe86c27128

Use more short array syntax in comments (/tests/)

Change-Id: I86c73cb9447ac562a73348b4030e24ebf49a90dc

Fix grammar in transaction-duration-limit-exceeded

Change-Id: I5e9db36abdb4c498c97640790490508452fdefd3

AuthManager: Break AuthPlugin::addUser more explicitly

AuthPlugin::addUser() is intended to only touch the external database
without creating a local user, which isn't possible under AuthManager
without reproducing too much of AuthManager's account creation methods
(and risking breaking things in even more obscure ways) to be
worthwhile.

As it is, either this will fail because the caller already called
User::addToDatabase() or the caller's subsequent User::addToDatabase()
call will fail because we're creating the local user.

So instead, let's just throw an exception unconditionally instead of
pretending it could work.

Bug: T137843
Change-Id: I8a439ea190c752a7fc49de5617e2c64c314c38f0

Use READ_LATEST for the WikiPage in RefreshLinksJob

Also sanity check that the revision belongs to that page.

Change-Id: I4e6897b52212d9787d74fb017861ec62f2927f0e

Merge "Add missing colons"

Add missing colons

Consistent with other log-action-filter-(action) messages.

Bug: T132889
Change-Id: I6b5e98aedd7bec37280684b362ccf936e45f6a30

Localisation updates from https://translatewiki.net.

Change-Id: I655debf814eca6ac00daaca122ed838da8c45a9b

Handle the default value of $wgExternalDiffEngine without warning

The default value (false) was not handled by the recent depreciation warnings
in 037f056, and was triggering the last warning. This removes this warning.

Change-Id: If7fe92d874eb10525a72f6d3a97f00117c79acc1

Run generateLocalAutoload.php

Follows-up ae0bae92afe0307d.

Change-Id: I5f7decd6a5279e941a7e1406dca77bbe37a1f3cb

ApiQuery: Don't mess with PHP output buffering

Specifically, it is not necessary to use output buffering functions
to capture XML generated by the export code because it is already
possible to set the "output sink" object to be used.

* Created a DumpStringOutput class, which appends all output to a
  string property rather than printing output immediately.
* Used that class, instead of ob_start() and ob_get_clean(), in
  ApiQuery and ExportTest.

Change-Id: I238f5d5ec7fd442c845b25cb59ef81ac3285099f

If rev_content_model is null, use page_content_model.

We currently leave rev_content_model = null  if it'S the default, to save space.
When loading a revision, we so far fell back to the per-namespace default content
model if rev_content_model was null, even if page_content_model was null.

This patch changes the fallback from rev_contentModel -> namespace-default-model
to rev_content_model -> page_content_model -> namespace-default-model.

This will prevent errors triggered when chaing a namespace'sdefault content model:
so far, revisions with the old content model would fail to deserialize becaue,
because they were being interpreted according to the changed namespace defrault,
instead of the correct model in page_content_model.

Bug: T128466
Change-Id: I75c60eb129428b0b433480443ab9153cc58cda8f

Localisation updates from https://translatewiki.net.

Change-Id: I7c5e1326e76de65612653cb8e8167c7d062fa7c2

Improve comments on fields and fix opening_text - needs no highlights.

Change-Id: Iad6876aae109ad84c5534619f47c72edc900d704

Merge "Clarify functionality of 'captionLength' in GalleryOptions"

TitleWidget: Add missing return documentation

Change-Id: Iea34d7e5dd4c54a13ac559bb2685246e0f096227

Clarify functionality of 'captionLength' in GalleryOptions

Change-Id: Ieaacec06282785d789ed89acc984b4dff4ad4460

Make classes actually work in lines in EnhancedChangesList

Change-Id: I2a1dc712d735077dd3741aa0d066933a81c4e568

New parserTests.php features

Features to support T89331 analysis:

* Support dwdiff for word-level diffing
* Add --mark-ws feature which produces cleaner diffs when line breaks
  differ
* Add optional normalization of parser test output, allowing significant
  differences to be separated from insignificant differences.

Change-Id: I0e151caad1f8b2f97bf20b219f26f3101be82506

Merge "API: Generate head items in the context of the given title"

Update wikimedia/wrappedstring to v2.1.1

New method to be used in OutputPage.

Depends-On: I18c93dbb29c61c515de74b02d04aea077aadeb52
Change-Id: I70ab1b57cef5b988e215c7c4365b92f7eb9c5693

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

Localisation updates from https://translatewiki.net.

Change-Id: Idc43190454d0d375d7537815a400438c21101171

Permit use of User::getDefaultOptions() in-process cache in unit tests

MediaWiki uses an in-process cache to speed up repeat calls to
User::getDefaultOptions() -- except when the unit tests are running, in which
case the process cache is disabled, because otherwise it would be at risk of
becoming stale due to unit tests manipulating $wgContLang. Well, there's a less
aggressive option, which is to keep the cache enabled but use it only if
$wgContLang hasn't changed. Since MediaWiki's test setup code creates default
users for the unit tests, User::getDefaultOptions() ends up getting called
quite a lot, so enabling the process cache is worth the trouble.

Change-Id: I81f3ae42d356939c81e59ab12d7a9e7d1206cb40

Merge "Create API to allow content handlers to handle structured data definitions"

Merge "Add missing …|null $context documentation to Action class"

Gallery: Add new slider mode

Bug: T128429
Change-Id: I14cac38cf1c66d9ba9584772a1cd3f345731b2af

API: Generate head items in the context of the given title

$context->getOutput() returns an OutputPage tied to the main
RequestContext at the root of the chain, not to the modified context
we're actually using.

Bug: T139565
Change-Id: Ie086d7f2ad3f7b5f50e3a2f83b1680e760b85e5e

Merge "Modified Lojban interwiki"

Release notes for UploadVerifyUpload hook (8f2acfcd)

Change-Id: Ic8c79e7c56c747795b59a05c2cae1e5ca59e417f

Pass addModuleStyles an array instead of calling it multiple times

Change-Id: Ia359c441714c039c505acd4bd37dd1b52c60b8ea

Merge "Introduce new hook UploadVerifyUpload to allow preventing file uploads"

Merge "mediawiki.api: Don't abort token requests in postWithToken()"

Merge "Revert "Adding a bunch of hooks from wikiHow into DifferenceEngine""

Merge "Revert "Show parser output for diffs unless extension aborts""

Revert "Adding a bunch of hooks from wikiHow into DifferenceEngine"

Causes fatals.

This reverts commit b02bfac06b04fd0284faa785f932792ce9bb93ae.

Bug: T139526
Change-Id: I4c77b4d51408540a9238d38e8324028879b1bbf2

Merge "Log the session cache type along with other cache types"

Revert "Show parser output for diffs unless extension aborts"

Reverting the underlying change.

This reverts commit 3354e8d550e6d650a0b1681fcad427559aea25ce.

Bug: T139526
Change-Id: I2e0f2098981bda411f360eea89e5fac3b644f043

Log the session cache type along with other cache types

We already log the types of the main, WAN, stash, message, and parser
caches. For debugging, it would be useful to log the session cache too
instead of having to guess that it's probably the same as the main
cache.

Change-Id: I13e6c0f7f430f96be50d543a32c232f0c36dcd72

Merge "DifferenceEngine: Autodetect if wikidiff2 is installed"

Create API to allow content handlers to handle structured data definitions

Change-Id: Ia1738803c42f6114575587c1c838fec62b6f54aa
Bug: T89733