Merge "Make chunkedUpload match upload behavior"

Merge "RCFilters: Live update: no data returns 204"

Merge "mediawiki.Title: Remove obsolete Object.create ES3 fallback"

mediawiki.Title: Remove obsolete Object.create ES3 fallback

We no longer support non-ES5 browsers.

Change-Id: If1e18dc3111899f3b0a5ae53909624186101a8ba

Merge "mw.loader: Remove Deferred overhead from execute() hot code path"

Merge "mw.loader: Add unit test to confirm implicit 'site' dependency for 'user'"

Merge "Fix saved query names cutoff"

Merge "De-duplicate HTMLCacheUpdate jobs with a page array of size 1"

mw.loader: Remove Deferred overhead from execute() hot code path

This was introduced by 0ac4f99804c for the concept of globally
implied legacy dependencies.

Commit ba257035b07 then re-purposed this for the site/user module
dependency.

The legacy wait was removed in 0ac6076b4c0, thus leaving only the
site/user code.

This commit now simplifies the code back to how it was before 0ac4f99804c
(but keeping the improved error handling from ba257035b07).

Change-Id: I2eae3b78fbe4f03a7a098d8a6233bdc2b79171b8

mw.loader: Add unit test to confirm implicit 'site' dependency for 'user'

The underlying logic is in mw.loader#execute and mw.loader#runScript,
but currently untested.

Adding tests in prep for refactoring this code.

Change-Id: I77ffb5f23443e24ca1d86f36ddf434673e9c1538

De-duplicate HTMLCacheUpdate jobs with a page array of size 1

BacklinkJobUtils consistently uses the "pages" field in leaf
jobs, even when there is only one page per leaf job.

RefreshLinksJob already has this logic for de-duplication.

Change-Id: Ia189bbc9df44f2161cfed4192c23b2ac3cfa65ce

Fix saved query names cutoff

Fix saved filters name misalignment and cutoff for
letters that have descenders on RC page.

Bug: T174111
Change-Id: I32e61fad98f38b48e0234467056a939d634eb15c

Prefer that enqueueable post-send updates are run immediately

If the user is not waiting at this point, so there is not much
reason to enqueue a job over just doing the work now. Running
the update now gives more immediate results however.

This has the effect of  making LinksUpdate run post-send for
forward link updates, since the addUpdate() call in WikiPage uses
the default POSTSEND mode. These updates used to be synchronous
in the past, before proper post-send update support. With post-send
updates, there is not much benefit to using the job queue here.

If post-send updates are not supported, this will continue to
use the job queue.

If a caller needs such updates to enqueue post-send to avoid DB
updates on HTTP GET or if the update is too big to run outside of
JobRunner, it can always just use JobQueueGroup::lazyPush() with
a direct job object or JobSpecification.

Change-Id: Ibc4b1e17538cc8b1fba7d13759e1ebb83abed869

Merge "Improve flaky SiteStatsTest"

Merge "JobQueueMemory: Actually return values in MappedIterator callback"

Merge "Allow users to restrict who can send them direct emails via Special:EmailUser"

RCFilters: Output namespace class on log actions as well

Bug: T174732
Change-Id: Icaf7117d9566bb2bb31c69cf86078491d6c217cc

populateIpChanges.php: Insert rows in batches

Change-Id: I0233b5a17fc64751514ec6d0701f155a73906873

Merge "Center items in 'Save current filter settings'"

Merge "RCFilters: make live update part of the beta feature"

Merge "Fix submit button id on Special:Preferences"

RCFilters: make live update part of the beta feature

Cleanup wgStructuredChangeFiltersEnableLiveUpdate

Bug: T175766
Change-Id: I03db105bef6c3730221c2d00ce56afb2c78bf57c

Merge "Sync up with Parsoid parserTests.txt"

Sync up with Parsoid parserTests.txt

This now aligns with Parsoid commit dcf2706cff795b84fbbfb6cd22749a86655b04f2

Change-Id: I24700184204000465922930dbac2b019259b5a01

Fix submit button id on Special:Preferences

For reasons beyond human fathoming, the button had the id set
to 'prefsubmit' in PHP code and then changed to 'prefcontrol'
in JS code. This functionality has been carefully preserved
across multiple rewrites of this code since 2004, when it was
added in 30d0ccd0 (rSVN3618). Let's just set the id in PHP.

Change-Id: Ib23bd0e481e73a51ff0a16731f47a2df11b2c1b8

Localisation updates from https://translatewiki.net.

Change-Id: Id145914494c1acfac06ec68cef3805341cbdcc88

RCFilters: Live update: no data returns 204

Return status 204 when peek=1 and there is no data

Also refactor _fetchChangesList to allow checking for new changes
and pulling them different handling.

Bug: T173613
Change-Id: I8fe2556156bac3d0cfa4f557ae82a163b6eb4d37

Merge "Move call of Sanitizer::decodeCharReferences from Skin to Parser"

Merge "Do not double decode HTML entities for IDs"

Merge "RCFilters: Add an initialization hook"

Merge "RCFilters: Actually apply proper classes to grouped pages"

Allow users to restrict who can send them direct emails via Special:EmailUser

Users can now specify a blacklist of users who are prevented from sending them a direct email.

Bug: T138166
Change-Id: Ifa26153f593b0ca3a9121e1e29961911c616c9e4

Merge "mediawiki.htmlform.checker: Add missing dependency on jquery.throttle-debounce"

RCFilters: Actually apply proper classes to grouped pages

Bug: T174734
Change-Id: Ie0a8f5587103c3caed31d29585db78017ac4ab4c

mediawiki.htmlform.checker: Add missing dependency on jquery.throttle-debounce

Change-Id: I56345cac3d52f2862d4b3b5df3ba3b6d0ed29c34

Merge "WikiPage::doCreate() should provide 12th parameter to PageContentSaveComplete hook"

Merge "Add CommentStoreComment::newUnsavedComment()"

Merge "Replace more problematic uses of "SELECT *""

WikiPage::doCreate() should provide 12th parameter to PageContentSaveComplete hook

In case the hook user has created the 12th param without a default.

Bug: T175596
Change-Id: I63142daaaeea069364dc70b45ee62680ac955f1e

Center items in 'Save current filter settings'

- Center cancel button (X) in 'Save current filter settings' menu.
- Center head label in 'Save current filter settings' menu.

Bug: T174454
Change-Id: I44ce0b40aa6acf5797f70fda731e5e4bde9f9748

Merge "RCFilters: Preference length is in bytes, so use $.byteLength"

Merge "Skin.php: HTML escape message pipe-separator"

Replace more problematic uses of "SELECT *"

With the introduction of CommentStore, selects from various table
require certain joins or column aliases for proper operation.

Change-Id: I2ecb6030bf253664c3227d69b5ed2277596a83f0

Move call of Sanitizer::decodeCharReferences from Skin to Parser

Supporting HTML entities in Wikitext is part of the parser not of the
skin.

Change-Id: Ib30ff08f4f4ce585f31211aa24b719783f06456e

Skin.php: HTML escape message pipe-separator

This prevents HTML and JavaScript injections.

Change-Id: I6241dadf29e96aa1aa15324a3edc51538214345d

Merge "Linker: Accept LinkTargets in makeCommentLink()"

Merge "Linker: Accept LinkTarget objects in link()"

Merge "Linker: Don't use Title::setFragment()"

Merge "ImageHistoryPseudoPager: Don't use Title::setFragment()"

RCFilters: Preference length is in bytes, so use $.byteLength

Also, a minor fix to $.byteLength docs

Change-Id: I0f93edb0cb61a057da5ce0876b74cd17179c6587

Merge "Fix coverage annotations"

Merge "Use db query builder not raw sql"

Merge "Quote $default in PostgresUpdater::setDefault"

Merge "Remove @covers from private Sanitizer method"

Merge "Chinese Conversion Table Update 2017-4"

Merge "Split off remaining helper classes for special pages to separate files"

Chinese Conversion Table Update 2017-4

Update the Chinese conversion table routinely to fix bugs reported from https://zh.wikipedia.org/wiki/Wikipedia:%E5%AD%97%E8%AF%8D%E8%BD%AC%E6%8D%A2/%E4%BF%AE%E5%A4%8D%E8%AF%B7%E6%B1%82 .

It is only data changes and only works for Chinese WikiProjects.

Change-Id: Ic66546d3a47e4dd946ae67c636a638ca181b0ca1

Use db query builder not raw sql

Change-Id: I8e26932e28fdda9a52e0486c5d2eb269eb9ecae5
Follows-up: Ice1bdae3d16cf365da14c6df0e8d91d2b914e064

Merge "postEdit: Use standard close icon"

Split off remaining helper classes for special pages to separate files

includes/specials/formfields/
* EditWatchlistCheckboxSeriesField (for SpecialEditWatchlist)
* UploadSourceField                (for SpecialUpload)
* Licenses                         (for SpecialUpload)

includes/specials/forms/
* EditWatchlistNormalHTMLForm      (for SpecialEditWatchlist)
* PreferencesForm                  (for SpecialPreferences)
* UploadForm                       (for SpecialUpload)

includes/specials/helpers/
* ImportReporter                   (for SpecialImport)
* License                          (for SpecialUpload)

Change-Id: I58abcbb44dbf9bf1762b4252555f7552bfa7c253

Merge "Enable recursive partials in TemplateParser"

Localisation updates from https://translatewiki.net.

Change-Id: I23445b9466fb16491dc794c730e743a035cc5b17

Merge "Reduce code duplication for parsing messages into dropdown menus"

Merge "EditPage: Stop using $wgLang"

Reduce code duplication for parsing messages into dropdown menus

The same behavior was implemented by Xml::listDropDown(),
Article::confirmDelete() and HTMLFormField::getOptions().
A new function Xml::listDropDownOptions() is added and
all three are changed to use it.

Additionally:
* Xml::listDropDown() now uses XmlSelect internally to generate the
  dropdown HTML.
* Xml::listDropDownOptionsOoui() is introduced to handle converting
  the existing format to the OOUI format, which was previously
  duplicated in Article::confirmDelete() and HTMLFormField::getOptionsOOUI().
* This change allows HTMLForm 'select' fields (HTMLSelectField) to
  support nested options (optgroups) in OOUI mode. This is a
  prerequisite for T117781.

Bug: T117781
Change-Id: I0a088f61eb32ec59677113583c7ecdcbc3fd2af0

EditPage: Stop using $wgLang

Change-Id: Ifca2cdcff45b4e06aef6e06dc5a3b852752a7a1a

EditPage: Rename and make getSubmitButtonLabel() protected

Calling this method "SaveButton" is misleading since this function
controls whether the button is called "save" or "publish". The
corresponding config option (EditSubmitButtonLabelPublish) already uses
the terminology "SubmitButton" so re-use it here.

And make it protected so subclasses (e.g. TwoColConflict) can use it
instead of having to re-implement it.

Change-Id: Iae811f0487f661bb501a9893037c352efa4e1e73

Sync up with Parsoid parserTests.txt

This now aligns with Parsoid commit c695317294087a691bad649237f9d86fbfce64c5

Change-Id: I6cb1eafcaf23846aba25714e6177c0e510c0b174

Remove @covers from private Sanitizer method

Breaks mediawiki-core-code-coverage

https://integration.wikimedia.org/ci/job/mediawiki-core-code-coverage/3003/console

Change-Id: I3204289c1d45e5ea0bee5c4bb6869434152e6017

JobQueueMemory: Actually return values in MappedIterator callback

Change-Id: Ie489d43ba3e9f4a10580716f38ec0f123d9ab5bb

Improve flaky SiteStatsTest

SiteStatsTest::testJobsCountGetCached() is somewhat flaky in that if it
runs after a test that adds a page (thereby producing htmlCacheUpdate
and recentChangesUpdate jobs) but doesn't have the CI framework reset
the `page` tables (which has the side effect of clearing all such jobs),
it will fail.

This change manually clears those jobs so it doesn't depend on test
ordering.

Change-Id: I1277e633c81b29bda7564fa12d23f13ded7298c7

Merge "Update padding of mw-ui-input to match OOUI"

Do not double decode HTML entities for IDs

* in links (T103714)
* in indicators (T104196)

This change removes the automatic Sanitizer::decodeCharReferences from
Sanitizer::escapeId and Sanitizer::escapeIdInternal. Where decoding of
HTML entities are wanted an explicit call to
Sanitizer::decodeCharReferences is added.

Explicit decode HTML entities in non local autocomments. (T104311)

Bug: T103714
Bug: T104196
Bug: T104311
Change-Id: I88e8e2077e6f5eec2b232391f7818370894a62dc

Update padding of mw-ui-input to match OOUI

Change-Id: I85c7044666be41a9baa52e8f7ba019a96dac4a6b

Merge "Port BCP47 formatter from PHP to JavaScript"

Port BCP47 formatter from PHP to JavaScript

It can be accessed via mw.language.bcp47.
To be used in ContentTranslation, see T157212

Change-Id: I37d32ab8a55c7101b903b03869899f00a39afd11

Merge "Replace autocomment color with WCAG conformant WikimediaUI palette one"

Merge "Sanity check "stashedtexthash" param before checking memcached"

Replace autocomment color with WCAG conformant WikimediaUI palette one

Replacing outdated `.autocomment` color with WCAG level AA conformant
one from WikimediaUI color palette and also replace `.wikitable`'s
color with normal text color. Black is to be used solely on emphasized
UI elements and we're remaining high contrast level.
Follow-up to Ide230784be654def9f120d5242d036f7423e65b2

Bug: T152025
Change-Id: I8e860b53481caf03e11f9bcdd836a80c3a2fc39f

Fix coverage annotations

Change-Id: I1f9b421a4ce55f1e0d4a81ed4a886aae20100fab

Merge "Don't percent-encode HTML5 IDs"

Don't percent-encode HTML5 IDs

During the TechCom meeting, it was decided this is the better way.

Bug: T152540

Change-Id: I6c3ec1c407225b4e925b7373bf52208e2f6b6c4a

Fix bug in dl-dt list output generation

* An open <dt> (;) should be closed when we encounter a new <dd> (:)
  char even if it is on a new line that has other nested lists inside.

* Tidy was hiding this PHP parser bug by closing a <dt> and opening
  a <dd> when given this HTML: "<dl><dt>a<ul><li>b</li></ul></dt></dl>"
  It generates "<dl><dt>a</dt><dd><ul><li>b</li></ul></dd></dl>"

  However, a HTML5 parser like RemexHTML, domino (used by Parsoid),
  or browsers don't do this fixup.

* So, what I thought was a bug in RemexHTML turned out to be a bug
  in the PHP parser that was being hidden by the use of Tidy.

* Added a regression test.

Bug: T175099
Change-Id: I6d5b225b82cecf9a43f23837ed8ec359b31aadad

Merge "Provide an opt-out preference for New Filters for RecentChanges"

RCFilters: Add an initialization hook

Change-Id: I9adc529bee0d5e25566409cc98f7bbac8178c478

Provide an opt-out preference for New Filters for RecentChanges

Currently this is disabled by default. On wikis with the BetaFeatures and
WikimediaMessages extensions installed, this preference is set (if the
$wgEnableRcFiltersBetaFeature flag is set) via BetaFeatures. This change
lets users on normal wikis use these too, and lets BetaFeatures-capable
wikis "graduate" the feature to be provided to all users by default.

Bug: T168376
Change-Id: I3c75f9f2f6287414bf330f116d959d078250392d

Merge "Add some devs with significant contributions to core"

Merge "FeedbackDialog: Improve alignment"

Merge "Add <body> class marking whether RCFilters is enabled"

Merge "RCFilters: Live update: disable peek param"

Add <body> class marking whether RCFilters is enabled

Also added OutputPage->addBodyClasses to allow this

Bug: T175062
Change-Id: I2111d0762328f2e59038c92d63583a9bb267fc9f

Add CommentStoreComment::newUnsavedComment()

Revision (or more accurately, its upcoming MCR replacement) should allow
for the full flexibility of CommentStore in its interface, and the most
straightforward way to do that is to use CommentStoreComment. But for
that to work sanely, we need a non-@private way to create a new
CommentStoreComment that hasn't been saved to the database yet.

Change-Id: I76028e8ef3da4568a16e098cdfbc806a67b814e3

Merge "Show a warning in edit preview when a template loop is detected"

Merge "Use CommentStore::getCommentLegacy with CommentStore::getFields"

ApiQuery: Fix type docs for $groups in ApiQueryBase::selectNamedDB() call chain

Change-Id: Iaa2ed914347fc4a1b477406cc8b83d7f21adf1a7

Merge "ApiQueryWatchlist: Document type of $commentStore property"

Merge "ApiQueryWatchlist: Don't reinvent Title::newFromLinkTarget()"

Use CommentStore::getCommentLegacy with CommentStore::getFields

It doesn't matter when $wgCommentTableSchemaMigrationStage is
MIGRATION_OLD, but it'll fail when we start changing that to later
migration stages.

Follows up I3447a412.

Change-Id: I6c010cefedd99324080ec078b83159d12709c1b1

ApiQueryWatchlist: Document type of $commentStore property

Change-Id: I884bdd347aeddadfd4a7de89261e5ab1a0d8f6f2

RCFilters: Live update: disable peek param

Temporarily disable the peek parameter until
we figure out why it doesn't work in betalabs.

This makes the responses larger but still much
smaller than they were originally.

Bug: T173613
Change-Id: I15af48d520644e3da073f0cc3bc63a7f7b7b3850