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

replacing "images" by "files" in listfiles-show-all for consistency

Change-Id: I2adabce83df0b429809f8256f3ae333fd8a36902

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

tests: ensure parserTestRunner restores $wgParser

ParserTestRunner::addArticle() set a dummy $wgParser to ease test
running. However if doEditContent() fails, it would abort and never
restore $wgParser.

Make sure the restoration is done even on exception.

I caught that one while running parser tests with Scribunto and Cite. My
setup lacked a luastandalone which causes Scribunto to throw an
exception. When the Cite tests are run, the MessageCache parser ended up
using $wgParser = ParserTestMockParser;

Change-Id: I33a0a5c1f40f197405d40b7925cc043342e6757e

Merge "build: Updating mediawiki/mediawiki-codesniffer to 0.12.0"

Merge "Improve some parameter docs"

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

It's deprecated for public usage.

Change-Id: I3edde2826abc8e3e87259ae602eda679f7cb7ed4

Linker: Accept LinkTargets in makeCommentLink()

Change-Id: I47b421b38ec60aaf8b4b9f762084f42b44ef8026

Linker: Accept LinkTarget objects in link()

Even though this function is deprecated, it'll make the migration to
TitleValue easier.

Change-Id: Idb16dd2dd50a5273bff5e0537825dc5e91f00d86

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

It's deprecated for public usage.

Change-Id: I916841f28351567dfa71f5a65c5cbc273ddbabd4

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

Change-Id: I1591c517ad8325e7c249197e6601fcc3d3efcc99

Localisation updates from https://translatewiki.net.

Change-Id: Ic79b226d568bfe5c9c6c4e016ea273d74dc055e3

build: Updating mediawiki/mediawiki-codesniffer to 0.12.0

Change-Id: Iefaae5043fa77d5d556c31079549dab8f61bd3ef

Improve some parameter docs

Add missing @return and @param to function docs and fixed some @param

Change-Id: I810727961057cfdcc274428b239af5975c57468d

Merge "FancyCaptcha: Align to WikimediaUI Style Guide"

Merge "HTMLForm: Implement OOUI version of HTMLTagFilter"

Merge "Replace wfShellExec() with a class"

Enable recursive partials in TemplateParser

Recursive partials are the only way to handle tree-like structures
such as nested lists. Allow setting FLAG_RUNTIMEPARTIAL in LightnCandy
so they can be used.

Since this has a slight performance impact (makes partial invocations
evaluation-time functions calls instead of compilation-time transclusions)
make it optional.

Change-Id: Ie37105a9f1ff92e1a79bfcd9f8578965e3d347f0

Merge "Minor fixes to ParsoidVirtualRESTService"

Merge "Avoid using the deprecated ParserCache::singleton()"

Fix return in HttpStatus::header

No caller expected a bool as return, which is not given in any case,
so remove it

Change-Id: Iaf994342e9f783108c767bb2de13af7345247805

Fix return of Interwiki::invalidateCache

InterwikiLookup::invalidateCache does not expected a return value

Change-Id: I5b2ba21ccb42ba9925205eb1645f2152fd51c0a0

Avoid using the deprecated ParserCache::singleton()

Change-Id: I0da6d9cbfad26c89bf5dab564071ef97acaf44f9

Fix return in Preprocessor::cacheSetTree

Function would not return always a bool,
caller does not expected a value, so remove it.

Change-Id: I506355fbd1bd304a6f3a161ac4c9d0bc765fda4a

Fix return in ParserTestTopLevelSuite::debug

wfDebugLog does not return anything

Change-Id: I85274a85342036b76a72ba82bb7de1d05382cf14

Localisation updates from https://translatewiki.net.

Change-Id: I6486b0d6441be32dad0c2404691ce2cd3383f3d2

LocalFile: Fix setting $this->description after CommentStore changes

Bug: T175444
Bug: T175443
Change-Id: I3447a4129300c2c7dc0d585404e7747b6a5e2b77

FeedbackDialog: Improve alignment

Improving alignment of FeedbackDialog items and increasing
`line-height` slightly for better readability.

Change-Id: Ic1d569efbdd1340fadb696e76c7a871f29794077

Quote $default in PostgresUpdater::setDefault

Bug: T175439
Change-Id: Id5f5f4f34c7b9ff9af4b706e3e126ba8fda6a0d5

Add some devs with significant contributions to core

Change-Id: I2f904c3c6748a4ad42d38dd89166cc5090365f67

FancyCaptcha: Align to WikimediaUI Style Guide

Aligning FancyCaptcha appearance to WikimediaUI color palette and
guidelines, adding `border-radius`, reducing `padding` slightly to
`8px` base and bring captcha and input closer together.

Bug: T152025
Change-Id: I26d9aadb7456ea6b7ba68a40cb4c624e995d9341

Replace outdated greys with WCAG conformant WikimediaUI palette ones

Replacing outdated greys with WCAG level AA conformant ones from
WikimediaUI color palette.

Bug: T152025
Change-Id: Ide230784be654def9f120d5242d036f7423e65b2

postEdit: Use standard close icon

Making use of standard close icon and also prevent screenreaders
from reading 'x' if they ever come across the message.
Also slightly bumping `font-size` to better harmonize
with close icon and clean-up CSS.

Bug: T50067
Change-Id: I82cdad23462f62d5e8ab2eef8632a2b3ac35e93f

Replace wfShellExec() with a class

This function has gotten so unwieldy that a helper was
introduced. Instead, here's this class that makes
shelling out easier and more readable.

Example usage:
  $result = Shell::command( 'shell command' )
       ->environment( [ 'ENVIRONMENT_VARIABLE' => 'VALUE' ] )
       ->limits( [ 'time' => 300 ] )
       ->execute();

  $exitCode = $result->getExitCode();
  $output = $result->getStdout();

This is a minimal change, so lots of stuff remains
unrefactored - I'd rather limit the scope of this commit.
A future improvement could be an ability to get stderr
separately from stdout.

Caveat: execution errors (proc_open is disabled/returned error) now
throw errors instead of returning a status code. wfShellExec() still
emulates this behavior though.

Competing commit: I7dccb2b67a4173a8a89b035e444fbda9102e4d0f
<legoktm> MaxSem: so you should continue working on your patch and I'll
          probably refactor on top of it later after its merged :P

Change-Id: I8ac9858b80d7908cf7e7981d7e19d0fc9c2265c0

Remove deprecated function wfFixSessionID()

Deprecated since 1.27, no callers anywhere.

Change-Id: I2d1019fdb21fe4a9d0aa767a71eb18739f841dfb

Deprecate wfUsePHP()

No callers anywhere.

Change-Id: I47256448d313983e3cebd713f7c149e3c3a9ab47

Merge "registration: Improve error message for invalid "callback" in extension.json"

registration: Improve error message for invalid "callback" in extension.json

Getting the following error for an invalid callback is not helpful:
Warning: Invalid argument: function: class not found in
/includes/registration/ExtensionRegistry.php on line 335

Change-Id: I4dfc011cb0a5cd06f1836d73e58f407d468e4546

Add Title::isValid method

Make the notion of Title objects representing invalid titles explicit.

Bug: T165149
Change-Id: I89aaabdff9614fe63bd1244784a1d677dbc26f9e

Merge "RCFilters: Remove getValue and use FormOptions"

RCFilters: Remove getValue and use FormOptions

Ensure the default is boolean, then FormOptions will
do the rest for us.

Bug: T174725
Change-Id: I92f2f77e1225f536a38f592578012855b948050c

Localisation updates from https://translatewiki.net.

Change-Id: I80104ab312390512e35c648847ec49f179e6a046