Merge "Fix MWDebugTest::testAppendDebugInfoToApiResultXmlFormat"

Fix MWDebugTest::testAppendDebugInfoToApiResultXmlFormat

When this test was added in Ifb2e392d3, it did not pass the correct
number of arguments to ApiFormatXml::recXmlPrint().

Bug: T182368
Change-Id: I63aee08307054137c6b50b6fbd271e03f6d4e45f

Add quotes to comment based strip markers

Bug: T180159
Change-Id: Ic9dbb8ef3948fe751d16c3963769b616b5db2fc7

Add overlay to list of changes

- Prevent users from accidentally clicking links on filtered results,
when RCFilters dropdown menu is opened, by adding overlay which closes
dropdown menu.
- Introduce MainWrapperWidget, to factor out some of the logic from
the init file.

Bug: T177626
Change-Id: Id55702ecbe6b96ee57453d4f86f20bd94a401d7c

Merge "Fix maintenance script failure when run as a child of a FastCGI worker"

Fix maintenance script failure when run as a child of a FastCGI worker

Use PHP_SAPI==cli instead of checking $_SERVER['REQUEST_METHOD'],
since $_SERVER is populated from the environment when running HHVM in
CLI mode. Environment variables set by a FastCGI worker thus leak
through to child processes run via the shell, and cause this check to
fail.

When I wrote this check in March 2004 (r2803), I didn't know about
PHP_SAPI. Checking PHP_SAPI is quite sufficient to prevent web execution,
we use it in other places.

Bug: T111441
Change-Id: Iad8469ee25df4b0e0c2371e7975a300b1695dd8d

Merge "Preferences: Display the default skin first in the list"

Merge "Preferences: Sort list of skins by internal name"

Merge "Make travis test against PHP 7.2"

Merge "Fix missing if from $wgDBssl comment"

Fix missing if from $wgDBssl comment

Change-Id: I46d1a87c990198090fd322da3193014487e871e6

Merge "Add custom collation for Northern Sami"

Make travis test against PHP 7.2

Change-Id: I3e77c383486550e116b4630b56d255ec6c043e2d

Merge "SpecialPages: Add ul { margin-top: 0; margin-bottom: 0 } for multicolumn"

SpecialPages: Add ul { margin-top: 0; margin-bottom: 0 } for multicolumn

The two-column layout has on skin Timeless currently a different top
margin on the first and on the second column. This change fixes this.

Bug: T181947
Change-Id: If995845c6cab58fe24b1d9b7859c0614db75ceb7

Localisation updates from https://translatewiki.net.

Change-Id: I42ccf64af59b022731649b95edd0f6eff33146e6

Add custom collation for Northern Sami

This commit adds a custom collation order for
Northern Sami ('se'). Northern Sami exists in ICU,
but the version of ICU that Wikimedia uses is a
few years old, and does *not* include Northern
Sami. It could be years before Wikimedia's production
servers use the one that includes Northern Sami (see
bug), so this is a temporary workaround to amend this
issue.

Bug: T181503
Change-Id: Ib8a48b8db99bef8ec4b05144aace5dbdcacfeded

Merge "Changed Instances of rawParams() to plaintextParams()"

Changed Instances of rawParams() to plaintextParams()

Bug: T182213
Change-Id: Ibf24dee5ea19092e8a38e57f34c98f513d4c9b21

Preferences: Display the default skin first in the list

Bug: T181112
Change-Id: If3de084b2a5f214f7d6d05b86daed151a23678a4

Preferences: Sort list of skins by internal name

Sort by the internal name, so that the ordering is the same for each
display language, especially if some skin names are translated to use
a different alphabet and some are not.

Bug: T181112
Change-Id: I763150cc578e9aa70990a53895c032f17d96e97a

Merge "Fix "Save current filter settings" menu and legend overlap"

Merge "GitInfo: Fix shell restrictions for submodules"

Merge "WLFilters: Correctly check if RCFilters should be enabled on WL"

ImageListPager: Don't stomp on $join_conds

When adding a join to the oldimage table to get counts, don't stomp on
any existing values in $join_conds. Just add the condition.

Bug: T182245
Change-Id: I4cd3332355f9974bdf35dc083be5bba537fcfb5f

Merge "RCFilters: Set up conditional views for RCLinked"

WLFilters: Correctly check if RCFilters should be enabled on WL

Side-effect of Ib933de3a3f9e876924386e80f315506f60f8af54

ChangesListSpecialPage#isStructuredFilterUiEnabled needs
to call static::checkStructuredFilterUiEnabled() to reach
the implementation from SpecialWatchlist.

Bug: T182318
Change-Id: I1e90d7e29f9e7fc5acece6466ed818210058dd2c

Merge "placeholder-message for html form should be ->text() not ->parse()"

LocalSettingsGenerator: Fix typo in exception message

Change-Id: I61fe67468eec659f312fb9aa4c20ec9bf083e78b

Merge "Always return a string from Language::formatNum()"

Always return a string from Language::formatNum()

It says it returns a string, and so it should.

Bug: T182277
Change-Id: Ic68c65c634c2557a1d07281623cd6c971b000323

RCFilters: Set up conditional views for RCLinked

The views that are only meant for specific pages should be set up
conditionally only when we are in that specific page, to prevent
issues with saved queries or param overload.

Change-Id: Id0bf89ce397ff8a54ce0a30d1ba7d8f9bbdd14a3

WikiPageTest: Fix typo in @covers

Change-Id: I3f3a3b2a1f5e5aed61cd71db17ae6a6a01fe2ecb

Fix "Save current filter settings" menu and legend overlap

Pass overlay to SaveFiltersPopupButtonWidget, so it gets onto same
stacking context as the rest of popup widgets.

Bug: T182269
Change-Id: I2c45f82840f651c39a0e572e003c82cbbd8c1c2e

placeholder-message for html form should be ->text() not ->parse()

placeholder is an html attribute, it does not support arbitrary
html, so ->text() is most appropriate, not full parse.

Follow up d3cd609e

Change-Id: Ia2aa5a001e19ee90e99936ef4f0dc879f182999f

Merge "RCLFilters: Make 'target' and 'to/from' sticky"

media: Ensure there ie enough data to extract software version

The Software EXIF / other metadata field was expected to contain
the software name followed by the version number.

There are occurences in Wikimedia production logs of errors showing
that's not always the case.

Bug: T178130
Change-Id: I4187a41b5fd8d7b5574ab50523668d8feb11bccc

Localisation updates from https://translatewiki.net.

Change-Id: I6a56314c866bd36b92a85362bae533992a04df79

Localisation updates from https://translatewiki.net.

Change-Id: Ic9f1eafc6ea82af6eabedff7dea438f773a420cf

RCLFilters: Make 'target' and 'to/from' sticky

Make these parameters sticky to exclude them
from Saved Queries and related features
(empty, restoreDefaults).

Bug: T172161
Change-Id: Ib8b3ecd34c6af9e0983573586485a9a9f503c060

Merge "RCLFilters: UI tweaks"

Merge "Prevent results from wiggling"

RCFilters: Reserve space between filters and results

- Add bottom margin to RCFilters head. There is no more reserved space
for "View newest changes" link, so we need to make sure we have enough
white space separator between head and results.

Bug: T182149
Change-Id: I592ab7bb1f12785d5bd802bb16f11b4e74fb26b7

RCLFilters: UI tweaks

Follow-up to I96af7ba583d03e6ff9833ac3b5f4b80cfd0ee626

* Reword 'Recentchangeslinked-summary'
* Bold ToOrFrom selector label
* Bold 'FROM' in 'rcfilters-filter-showlinkedfrom-option-label'
* Bold 'TO' in 'rcfilters-filter-showlinkedto-option-label'
* Reword target page selector placeholder
* Prevent title suggestions from showing duplicate entries
* Show a specific message when there is no results
  because no target page is specified.

Bug: T172161
Change-Id: I1b1ee793b3eff4183039acc27fe6cca042ee57d0

Added file icon for mp3 files

Bug: T182139
Change-Id: Ib834ef5b0bb15b3981288b2e711107830dcb5af8

Merge "Allow wikilinks in "show/hide <type of change>" labels on Watchlist and RecentChanges"

Merge "Add missing @throws in Importers"

Add missing @throws in Importers

Change-Id: I02760e59d73f4278cca87483ab4578b6e76b2324

Get conflict results from diff3

This patch extends the global wfMerge function to also return the
result of the first merge attempt that detects merge conflicts.

The additional output explicitly names the conflicting lines and
could help when solving edit conflicts.

Bug: T151320
Change-Id: I97acebdc87b31779200c7fde4dd4449cd1ee8ead

Revert "Support changing icon variants on hover"

This reverts commit 46c62ff5f474c6fe7a409863058e81bb678229b2. After reconsidering menu item treatment under UI Standardization perspective, we don't need and shouldn't use such pattern.

Bug: T110051
Change-Id: I0e8b6035f516e6629010b4f18471445c8833156e

Prevent results from wiggling

The patch Ifecaaff572 introduces clearing to the right for the first
result list item, when grouping is disabled. The difference between
cleared vs non-cleared result becomes visible when legend is toggled,
causing really small (around a pixel), but annoying wiggle.

Change-Id: Ida6fb6f8ad0caf666258e3f1e770c6da9e24af58

Localisation updates from https://translatewiki.net.

Change-Id: I3432086a7a800e4c50609eb2a20cca9c314725af

Tests for WikiPage::insertOn

Change-Id: I1bf7d19c4baaf0f720fcd6a1d9c16bf7489b5392

Tests for WikiPage::updateIfNewerOn

This method is deprecated but due to refactoring
we should still test it.

Change-Id: I171ab731fff48ddd9fa54ef433881154377a2c7f

Tests for WikiPage::updateRevisionOn

Bug: T180989
Change-Id: I245d9d467e3113bf49e06ee5c725205d4225671a

Test for WikiPge::insertProtectNullRevision

Bug: T180989
Change-Id: Iffee0bbe9fb18bb6c7f7af10b6405007468ac0b9

Tests for WikiPage::newFromID

Bug: T180989
Change-Id: Idb4d7ce8ccc27226b0f00e2127acef6158dd32a9

Test for WikiPage::newFromRow

Bug: T180989
Change-Id: If630e3e3561ac2017a2b7c82ade842e86f8cb22e

Improve phpdoc of 'OldRestrictions' in Title

The previous version of the phpdoc simply says
that the field contains a comma seperated list of
page restrictions, which is wrong.
The field is actually a colon seperated list of comma
seperated lists.

The new phpdoc and example have been pulled from mw.org
and checked against what actually exists in a real db.

Change-Id: I6bfdfd87120150e4b53e1b4c49e95041f55d8c97

Merge "Tests for WikiPage::insertRedirectEntry"

Merge "RCLFilters: Show images and descriptions with page suggestions"

Merge "ImagePage: Make metadata table's initial collapse CSS-only"

Merge "Absolutely position "legend" on grouped results"

Merge "Warn for uploads with new name but same content as local file"

Merge "ImagePage: Clean up metadata.js collapsing"

RCLFilters: Show images and descriptions with page suggestions

Bug: T172161
Change-Id: I1a5501eefdb721367436db50be0787c8f6a78d47

Merge "RCFilters: Update invert URL param when toggled"

Merge "RCLFilters: convert related changes tool to new UX"

Merge "RCFilters: Clarify 'hidden' and 'sticky' filters"

Merge "RCFilters: Make 'days' and 'limit' sticky"

Tests for WikiPage::insertRedirectEntry

Bug: T180989
Change-Id: I677202b22594c7fc8fcf97b805ae69726945f96d

ImagePage: Make metadata table's initial collapse CSS-only

Make the initial collapse of the metadata table not depend on
JavaScript. This eliminates the FOUC, and will also reduce JS
footprint once compat can be removed.

* Move the 'display: none;' styles from metadata.css to
  filepage.css so that they are part of the base styles for
  file pages (metadata.css is only for supporting metadata.js,
  which is loaded dynamically).

* Apply "collapsed" class from the PHP side immediately,
  instead of calling "addClass" in JavaScript.

* Restrict hide-styles to only apply under .client-js to make
  sure the rows remain visible in no-js mode.

* Declare dependency between metadata.js and filepage.css.
  This is just for documentation purposes, as they are both
  already separately loaded on all file pages.

Change-Id: If22bf7acb47b59151dc3b0843a62507c100e548b

ImagePage: Clean up metadata.js collapsing

In preparation for trying out a CSS-only approach for the initial
collapse that doesn't cause the expanded view to flash first.

Bug: T182047
Change-Id: I42941ccb05d5525599a0209f0f33db9fe836580b

RCFilters: Update invert URL param when toggled

Bug: T182045
Change-Id: Ia9a8afb336305969ac8b5c2e7ffab146844ad9ee

Merge "API: Account for PHP 7.2 change"

Localisation updates from https://translatewiki.net.

Change-Id: Ib8affd4b492c3be56458b54d28e4c02cb7ecfc9c

RCLFilters: convert related changes tool to new UX

Bug: T172161
Change-Id: I96af7ba583d03e6ff9833ac3b5f4b80cfd0ee626

Merge "A few doc comment fixups"

Allow wikilinks in "show/hide <type of change>" labels on Watchlist and RecentChanges

This is done by treating the messages as wikitext rather than plain
text

Bug: T142406
Change-Id: I5342b9491df47bbb549e91ebba9d0a8b273a7d17

Merge "Test for WikiPage::updateRedirectOn"

Merge "Add test for WikiPage::updateCategoryCounts"

Merge "Refactor test for WikiPage::doRollback"

Merge "WikiPage tests @covers for doDeleteArticleReal & commitRollback"

Merge "Split WikiPageTest into ContentHandler & NoContentHandler tests"

API: Account for PHP 7.2 change

PHP 7.2 broke existing functionality in making count( null ) raise a
warning. So add tests for null all over the place, or change tests where
we know the value is null or an array (but not false, empty-string, or
0) to just cast to boolean.

Bug: T182004
Change-Id: Idfe23a07daa9f60eee72f2daf04304be87057a29

Merge "objectcache: various cleanups to WANObjectCache"

Merge "Replace some Special:Undelete button with OOUI ones"

Replace some Special:Undelete button with OOUI ones

Bug: T180207
Change-Id: Icb550a6ad8f90d7718441ec17a1026a2cfb9d754

RCFilters: Clarify 'hidden' and 'sticky' filters

Clarify the meaning and behavior of sticky and hidden filters
according to the following definitions:

When 'sticky' is false, the filter is saved in Saved Queries. Its
state is considered and altered by the 'Clear all filters' and
'Restore default filters' buttons.

When 'sticky' is true, the filter state is only changed using its
associated control (i.e. Number of days selector).

'hidden' simply means it is not shown as a capsule in the active
filters area.

Change-Id: I7ccb7c202e21917eb288055c509c9f904a0d01bb

Merge "Only send statsd data for WAN cache in non-CLI mode"

Merge "Remove unuseful debug messages"

Absolutely position "legend" on grouped results

- When results are grouped by page, on pages where new RC filters
are enabled, apply position: absolute to legend, to avoid top
result entries being pushed too much.

Bug: T174728
Change-Id: Ifecaaff5727905b3c4e860555ba094f71aac6f80

Test for WikiPage::updateRedirectOn

Bug: T180989
Change-Id: I798ded72142503e52395b45fe9ec71a215d5ab92

Remove unuseful debug messages

I spotted one of these in my debug log. Without context it is not
useful nor informative. Given it can actually come from three places,
it is useless.

Change-Id: Ic293d8944e9a1a8152c5a5e0b695c5ddc07ee6aa

Add test for WikiPage::updateCategoryCounts

Bug: T180989
Change-Id: Iff4bb885e5d385a7b3ec6169ee9a3e1e82d7d993

Refactor test for WikiPage::doRollback

This test was not running and a comment said
it was broken.
This patch refactors the test.

Bug: T180989
Change-Id: I46b73371e5fd65860fce6cc101bad6abfa20e91d

WikiPage tests @covers for doDeleteArticleReal & commitRollback

Bug: T180989
Change-Id: I70dcad9d527be4d91a88a14fe9787c53a7cf7d2f

Split WikiPageTest into ContentHandler & NoContentHandler tests

Follow up to Iccdcd4a4f72c8b1329d36ff2e8699ba6b23c9ae7

Bug: T180989
Change-Id: Ib37a058586c7222ffe3ee5dc5f0e134ee18f0ff7

Fix phpdoc for WikiPage::updateRedirectOn $redirectTitle param

Change-Id: I8e666db2a7502bd1773252aa71db47819f63f08b

A few doc comment fixups

* Remove some creation dates, they are not protected by GPL
* Remove duplicate @defgroup API
* Remove @ingroup from some @file doc comments on class files. It is not
  useful to list class files alongside classes in the doxygen module menu.
  Add @ingroup to some more class files that had @ingroup on their file,
  that was probably the author's intent.
* In PackedOverlayImageGallery, use the file comment as a class comment
* Don't put @defgroup and @file in the same comment. @defgroup makes the
  whole doc comment describe the group.
* Instead of putting AnsiTermColorer in two groups, use hierarchical
  groups.

Change-Id: If54f6e0b2bc1ea6de42045885cf836ee67b8e961