Prefix comment table when adding indexes

Change-Id: Ie87d2ef68050a402d6c81ca98cc7402f7c5f14aa
Follows-Up: Ic3a434c061ed6e443ea072bc62dda09acbeeed7f

Merge "RCFilters: Include invert param when reading default saved query"

Merge "SpecialRecentchanges: Restore ability to use language links in 'recentchangestext' message"

Localisation updates from https://translatewiki.net.

Change-Id: Ia9564e3135fe8cc5ba61588cd921180af7489167

parserTests: Improve class name guessing for PHPUnit

Since we know the extension name, actually use it instead of trying to
guess it based on the directory name.

The new directory layout for autodiscovery encourages "tests/parser" so
the directory one level up is unlikely to be the extension name.

Change-Id: I352944a84197acba6fe425b6044b17c520e263d7

Add `comment` table and code to start using it

A subsequent patch will remove the old columns.

Bug: T166732
Change-Id: Ic3a434c061ed6e443ea072bc62dda09acbeeed7f

RCFilters: Include invert param when reading default saved query

Bug: T174506
Change-Id: Ibca0e0c80d8069ac3e73533f1f342c2946cce6ca

Merge "WLFilters: Use displaysOnUnstructuredUi instead of isVisible"

Merge "Add @since tag to Title::canHaveTalkPage()"

Avoid using deprecated Title::canTalk()

Change-Id: Ibd224f9de595435524e683262882c9ebf2761abf

Add @since tag to Title::canHaveTalkPage()

It was added in f8ac44cca56.

Change-Id: Iee3e074f5108a905b66c52305978c99b2604f2cd

Merge "Rename IP::isValidBlock to isValidRange, deprecating the former"

Merge "Get better error message for duplicate names in MagicWordArray"

Rename IP::isValidBlock to isValidRange, deprecating the former

This is to remove confusion with the MediaWiki Block class.

All instances of isValidBlock within MediaWiki core have been updated.

Usage of this function will be more widespread with this patch:
https://gerrit.wikimedia.org/r/#/c/349457/

Change-Id: Ice1bdae3d16cf365da14c6df0e8d91d2b914e067

Convert UNIQUE keys into PRIMARY KEY

WMF DBAs have been doing a massive effort to convert UNIQUE KEYS into
PRIMARY KEY.

Having a PK is essential to do maintenance, specially on large tasks.
By not having a PK it is impossible to add it in a safe way if not done
directly on the master.

Having a PK means that we can easily change the PK into another one if
needed in the future. The ones we chose might not be the best ones, but
will allow us to get them changed.

Bug: T172514
Change-Id: Id635297838938c7c5dfe65d45285a4d16d65152d

Merge "RCFilters: Target pushPin icon for size, not all icons"

WLFilters: Use displaysOnUnstructuredUi instead of isVisible

Follow up I3e48a9f2d9b70f0b9f6d7c6329db9c8e8001ee49

Bug: T171134
Change-Id: Ife0b10265c34d2dfd9a4d234f4a5409382530cc9

Merge "Run Selenium tests for skins"

Localisation updates from https://translatewiki.net.

Change-Id: I31af5076f228c31072f4fca54e049eb4cf54feb0

Merge "WLFilters: set default values"

RCFilters: Target pushPin icon for size, not all icons

Whoops.

Bug: T173264
Change-Id: Icb92c55a5b067d77138bae93f4db96309c72a223

Enable mediawiki.hlist on mobile

Bug: T42062
Change-Id: Iee059a6605cf12a073954378f49ddc721a7e016a

RCFilters: Minimize saved query before comparison

Bug: T174193
Change-Id: I8e9345e8899fd27565930896e62c4d3be235ad58

Merge "EditPage: Remove temporary OOUI page class name"

Merge "Making sister search sidebar links blue"

Enable mediawiki.special.userlogin.common.styles on mobile and allow skins to style it

Depends-On: I0fe9efcf98bbd8e188d91a29d4cfda897d47808d
Bug: T173597
Change-Id: I36e596e4d6060ef168464732028cd0dcbf21d0f8

WLFilters: set default values

* Respect different default values for 'limit' and 'day'
  in RC and WL.

* Make 'latestrevision' respect 'watchlistextended'

Introducing 2 properties to ChangesListBooleanFilter
* activeValue: The value that defines when a filter is active.
  Most filters are active when they are set to 'true' but
  'extended' has no effect when it is 'true' and applies
  filtering when it is set to false.

* isVisible: Whether this filter is visible anywhere.
  'extended' is not visible in the legacy form but
  it is activated from preference or URL. When
  understanding form submission, it should not be assume
  to be 'false' when not present in the request.

Bug: T171134
Change-Id: I3e48a9f2d9b70f0b9f6d7c6329db9c8e8001ee49

Merge "Consistent behavior of read-only reason"

SpecialRecentchanges: Restore ability to use language links in 'recentchangestext' message

Bug: T172461
Change-Id: I7b660c78f410f09815a82810824d56cb7a477280

Merge "RCFilters: Adjust highlight for seen/unseen states in Watchlist"

Consistent behavior of read-only reason

Before this commit, the reason set in the global $wgReadOnly was differently handled
on different special pages. While on most of them, like Special:Upload, the reason
is allowed to have HTML, which can be used in Wikitext, too, Special:CreateAccount
always outputted an escaped version of this reason.

Most special pages uses the ReadOnlyError exception to print a read-only error,
however, AuthManager uses Status objects to communicate between the backend and the
frontend. Therefore the same message and parameters were wrapped in a Status object
and, in the frontend, directly passed to the constructor of ErrorPageError. Unfortunately,
Status::getMessage() escapes the parameters of a message, which is the reason, why the
wiki is read-only. To bypass this restriction, AuthManager now creates a Message object
directly, does not escape the reason, and uses the resulting object to create a Status
object from.

Now the reason is not escaped on Special:CreateAccount anymore, like on most other
special pages.

The read-only message on the protection form is, also before this commit, not escaped and
already displayed correctly, as the read-only is checked in the constructor of the
protection form already and, if the Wiki is read only, handled as a permission error and
already displayed correctly. This commit fixes the behavior of WikiPage in case of it's used
somewhere else, subclassed or if the check in the frontend will be removed and the Status of
WikiPage will be used.

Bug: T157036
Change-Id: Idbfe556fcb90f8bda8fae9d728ca9dee5ea02f67

RCFilters: Adjust highlight for seen/unseen states in Watchlist

Bug: T171235
Change-Id: I33d059d2be83e4730e6dc82f689f49a514286874

Merge "Add Atayal (tay) language"

Merge "Produce RDF dump of all categories and subcategories in a wiki."

Merge "postEdit: Generate close button with CSS instead of HTML"

Merge "postEdit: Simplify close button"

Merge "widgets: Replace outdated colors with WikimediaUI palette ones"

Localisation updates from https://translatewiki.net.

Change-Id: If6db589c4d0fdbff5c23ab467424a1f7e9cdaa6f

Produce RDF dump of all categories and subcategories in a wiki.

Example:
http://en.wiki.local.wmftest.net:8080/wiki/Category:Ducks> a mediawiki:Category ;
        rdfs:label "Ducks" ;
        mediawiki:isInCategory <http://en.wiki.local.wmftest.net:8080/wiki/Category:Birds> .

Bug: T157676
Change-Id: I59b9603581b37af59d17dd6c38247c85aee44911

Add 1.27.3 to RELEASE-NOTES

Change-Id: I56fddd375352ad42a315bc078e33edde50bdae66

postEdit: Generate close button with CSS instead of HTML

Also remove :hover for cursor:pointer because cursor:pointer is always
shown on hovering.

Change-Id: Ib44f927a1764f703a8c31af363ebd5a8fca27375

postEdit: Simplify close button

Use <span> instead of <a href="#"> for the close button. This makes
resetting the CSS style and the preventDefault() superfluous and
prevents open a new tab on middle click and does not show an
unnecessary status bar on hover.

Change-Id: I8c60aae454d64803686088f1dbe0228b1ecc70be

Merge "importDump.php: Declare uploadCount property"

importDump.php: Declare uploadCount property

Bug: T174255
Change-Id: Ic5014821a8f10ab29b729104cd7d4e434085e2f2

Installer: Fix "Array to string conversion" notice

PHP Notice: Array to string conversion in .../includes/installer/Installer.php on line 1392

Follows-up fd8d75c1fd26e.

Bug: T174258
Change-Id: I6f8497a74a83aec183684bcb441191c32c67ccde

Localisation updates from https://translatewiki.net.

Change-Id: Ia6cbcd04a82a4dc499bed0f9e94502631d4820db

Merge "Fix old regression in HTMLCacheUpdate de-duplication"

Merge "Avoid preemptive DB replication waits for farm cross-wiki redirects"

Fix old regression in HTMLCacheUpdate de-duplication

* The condition timestamp should be the root job timestamp,
  as the comments imply, though that was not the case. Only
  the SET timestamp should be the current timestamp. As it
  was, jobs effectively did not "see" each other since each
  one would be expecting newer than the ones that ran prior.
  The page_touched condition for the UPDATE query was mostly
  useless, since it was always the current timestamp.
* This problem was a regression from f598ca81e3f40166.

Change-Id: I1398366e87c959be1c98e4db959c41309b0ac1b8

Merge "WLFilters: Fix seen/unseen filters"

Merge "Improve some timeout docs"

Add Atayal (tay) language

Bug: T173383
Change-Id: Id0607ca4a748f88476fefd081864b1282e9bcb8d

Merge "Ignore npm-debug.log"

Merge "SpecialPageLanguage: Show current page language in the dropdown"

Ignore npm-debug.log

Change-Id: Ia984f7baa2aa94e00b7be712d5dc4846d29ede85

Merge "WLFilters: avoid querying the DB when filters are in conflict"

Run Selenium tests for skins

So far, only extension had Selenium tests, but now MinervaNeue skin also has them.

Bug: T174018
Change-Id: I6bac14f44e677cb0b1fdefc6474d08fb17a0b911

WLFilters: Fix seen/unseen filters

Handle the fact that wl_notificationtimestamp
can be null.

Always join with 'page' table to support highlight.

Bug: T171127
Change-Id: I150a41b131d64163e06bfa56bddbd762bb636655

Improve some timeout docs

Change-Id: I799ea1e647d4d67a3c7d0e8dfddff893aef32736

Localisation updates from https://translatewiki.net.

Change-Id: I9e46d4a2a740433b768ef2d07139829de8bd587f

Merge "widgets.TitleWidget: Use mediawiki.mixins `.box-sizing()`"

Merge "widgets.StashedFileWidget: Simplify and reduce LESS code"

widgets.TitleWidget: Use mediawiki.mixins `.box-sizing()`

Using general `box-sizing()` mixin from mediawiki.mixins instead of
decentral vendor prefixes.

Change-Id: I844dc28a627dc984540e9a013dae188187c4f151

widgets.StashedFileWidget: Simplify and reduce LESS code

Simplifying and reducing LESS code where applicable.

Change-Id: Ic0f138d33064b4c8f2bb6253a20012a3b577372b

Add sslCAFile option to DatabaseMysqli

This makes all arguments to the mysqli::set_ssl() call be controllable.

Change-Id: I67ed742add633a77e97d08b812e420a73cd83a52

widgets: Replace outdated colors with WikimediaUI palette ones

Replacing outdated colors in core widgets with WikimediaUI color
palette ones. Also increasing contrast in one case ensuring
WCAG 2.0 level AA conformance.

Bug: T172850
Change-Id: Ic7cf0842275529372cdfbfddad4eb63fec4847ce

Merge "Disable rebound CDN purges for backlinks in HTMLCacheUpdateJob"

Merge "RCFilters: Align remaining outdated greys to WikimediaUI palette ones"

Merge "RCFilters: Enable 'View newest'"

Disable rebound CDN purges for backlinks in HTMLCacheUpdateJob

These are there to handle regeneration race conditions due to
replication lag. If there happens to already be a huge amount
of purges, the cost of this can be too high. Also, if the queue
is backlogged, then it is already delayed, so extra delayed jobs
do not help and just add to the backlog in that scenario.

Bug: T173710
Change-Id: Ida0bf44233072e1356a9fe63866d522e4bb2d0ca

RCFilters: Align remaining outdated greys to WikimediaUI palette ones

Replacing abandoned grey colors with appropriate colors from WikimediaUI
color palette.

Additional changes:
* Also simplified selector specificity in one case as OOUI code has already
changed.

Bug: T172850
Change-Id: I01e84d9a4d03ba894cbb67114821036b0ffe1ddb

RCFilters: Enable 'View newest'

Keep the feature flag for 'live update'
but enable 'view newest' by default on RC.

Bug: T163426
Change-Id: I733cf99bbeb6d90d3aa21e87507c513354b81791

Merge "Prevent editing textarea from jumping when toolbar is loaded"

Merge "IP: Remove unused static member $proxyIpSet"

Merge "RCFilters: Truncate long saved query titles"

Merge "Skin: Remove more use of QuickTemplate::setRef()"

Merge "Make API multivalue limits configurable"

Localisation updates from https://translatewiki.net.

Change-Id: I4336d8b1d77bbdc7b169ec5fa0fdfe2518392b41

Make API multivalue limits configurable

Adds two new parameter settings, ApiBase::PARAM_ISMULTI_LIMIT1
and PARAM_ISMULTI_LIMIT2 for configuring the maximum number of values
that can be contained in a multivalue field (for unprivileged and
apihighlimits users, respectively). When present, these replace the
default 50/500.

Change-Id: Ic1b1bcc7ff556b7762c8d2375d910cc4fcb43087

Merge "mediawiki.skinning: Vertical align inline media in Parsoid output"

mediawiki.skinning: Vertical align inline media in Parsoid output

Change-Id: I56df80a25b43f9084e8cb5e537836ac773732259

Merge "RCFilters: Adjust pushpin 'default' icon behavior and style"

Prevent editing textarea from jumping when toolbar is loaded

If you're using the "classic" editor (not the WikiEditor editor),
the editing textarea jumps down when the toolbar loads, which can
take a few seconds on a slow connection. This can cause editors to
accidently click the toolbar instead of the editing textarea and
accidently insert markup into the article that they didn't mean to.

This rule has been on English Wikipedia's Common.css since 2007.

Change-Id: Ie4ac15c95554608c838ef0e112924fc6381a6f66

RCFilters: Adjust pushpin 'default' icon behavior and style

Bug: T173264
Change-Id: I8d977bb97e91a19c24527c66120f142738135730

RCFilters: Hide saved queries from anonymous users

Bug: T173992
Change-Id: I5acb9ca4dd20de83da5db42c7c9053fccb091da0

Merge "Unit tests for CollationFa (0bfcbd724)"

Avoid preemptive DB replication waits for farm cross-wiki redirects

This previously only worked if $wgLocalVirtualHosts was set, which
was too specific to check and not used by WMF. Use the more generic
WikiMap class.

Two methods have been added there to do the work of enumerating
canonical wiki farm URLs and checking them against a given URL.

Bug: T172357
Change-Id: Id2415bab5d7f5a08b9f536858c32d329138384a2

SpecialPageLanguage: Show current page language in the dropdown

Also show canonical page title form while we're at it.

Bug: T74966
Change-Id: I52c19a0a018d76a4282c61c009b40b2f7e56b3d2

Merge "RCFilters: Minimize vertical space"

Merge "mw.widgets.SelectWithInputWidget: Invisible invalid fields should not block form submission"

Skin: Remove more use of QuickTemplate::setRef()

Follows-up 2c01147c13be.

* language_links: The local array variable is only used once to
  obtain the value from Skin::getLanguages(), and then never passed
  elsewhere or otherwise potentially changed, so this reference
  doesn't seem useful.
  The getLanguages() method itself also creates a new array every
  time it is called, so the reference from there can't change either.

* Same for content_navigation and content_actions. These are never
  modified, and the method that returns the array creates a new one
  on each call.
  Follows-up r3014 (f4755755daed1f8).

* Same for bodytext.

Presumably these were only set as references to save memory.

Bug: T140664
Change-Id: I1224a7a89c1d7ad05c73132ff73299fd78ade7fb

mw.widgets.SelectWithInputWidget: Invisible invalid fields should not block form submission

Same logic as in resources/src/mediawiki/htmlform/hide-if.js,
explanatory comment is copy-pasted from there.

Bug: T173839
Change-Id: I87603936d23165926d2ef4c64f3a61dca2062caf

RCFilters: Minimize vertical space

Bug: T173265
Change-Id: I38821571450c59b18c753c8b20ad9f19f01b6276

Make workItemCount() smarter for htmlCacheUpdate/refreshLinks

Do not count jobs that just make subdivide as having any
"work items". This makes $wgJobBackoffThrottling less
overzealous when used to limit these type of jobs.

The main reason to limit htmlCacheUpdate would be for
CDN purge rate limiting. For refreshLinks, it would
mostly be lag, though that is already handled for
leaf jobs and JobRunner itself.

Bug: T173710
Change-Id: Ide831b555e51e3111410929a598efb6c0afc0989

Merge "Set line height to 1 for sup and sub elements in print"

Set line height to 1 for sup and sub elements in print

It is already set in screen styles (shared.css).

Bug: T172501
Change-Id: I5a90e21f6eac300c57b6daddada0ae94fe579294

Merge "mediawiki.skinning: Vertical alignment for traditional galleries in Parsoid output"

Merge "MediawikiUI: Introduce mw-ui-icon-large"

mediawiki.skinning: Vertical alignment for traditional galleries in Parsoid output

 * Note that this just an extraction of inline styles that are
   already present in the output.

Change-Id: I718f16dfbb2c4122ac9fb4a714b607da3e62d5a0

MediawikiUI: Introduce mw-ui-icon-large

This will allow MobileFrontend and Minerva to share rules for icons

See Ib29736dc80ac2f84747e920c5b5c30918df12978

Bug: T143264
Change-Id: I456714086d2548b2e10cceb2a05f7a0fadf94a3b