CommentStore: Try to improve description of insert()

And the similar insertWithTempTable() too.

Change-Id: Ibf0f96d3b9b0f7fe53eb9e11b94fabadc4580632

Merge "Handle comment truncation in CommentStore"

Convert INDEX into PRIMARY KEY in l10n_cache

Bug: T146591
Change-Id: I85ab18bfa30e562e76253e3ef71653dbf46db041

Merge "EditPage: Hard deprecate submit()"

Merge "EditPage: Don't use RequestContext::getMain()"

Merge "EditPage: Don't use $wgOut"

Merge "Enable save button when removing item from UsersMultiselectWidget list"

resourceloader: Reduce severity for non-fatal JS errors to warn()

Follows-up 37df74151 and 12cca1c6, which made the internal exceptions
from sortDependencies() for unknown modules, or circular dependencies,
get logged to the console instead of silently caught.

There were logged using mw.track('resourceloader.exception') which
get logged to the console using console.error() by default.

Change this to using console.warn() instead.

Until 37df74151 and 12cca1c6, messages from `resourceloader.exception` where
quite rare. However both the rare cases, as well as the cases we added in
these commits, are non-fatal. Given various false reports in recent weeks
on Phabricator from users thinking they are witnessing fatal errors, let's
change the severity level of these in the browser console to "warn" instead
of "error", which makes them easier to distinguish from fatal errors.

Change-Id: I8d7aff6c72e74836c034b18a5ff58428d27628ff

Merge "Check for Wikidiff2 version 0.3 NOT for 0.3.0"

Merge "Fix RestbaseVirtualRESTService URL standardization"

EditPage: Hard deprecate submit()

Change-Id: I456139a8bd968dd3199225d6526624e778ed3381

EditPage: Don't use RequestContext::getMain()

We already have a context object.

Change-Id: I09816dd485413d858e5ee422ae07a843cb543da5

EditPage: Don't use $wgOut

Use $this->context->getOutput() instead everywhere, except in one static
function, that will be dealt with later.

For the most part OutputPage is a singleton, so this should affect very
little.

Bug: T144366
Change-Id: I59699a47e55f281284c5e96676cfe7bc0c499e28

Merge "EditPage: Deprecate getCheckboxes() and getCheckboxesOOUI()"

Enable save button when removing item from UsersMultiselectWidget list

UsersMultiselectWidget uses a hidden input to store values. Hidden inputs do not trigger onChange events. The onChange event must be trigger manually.

Bug: T173973
Change-Id: Ib9ec31791f1fe7bdde8d91ae16a2111f4320c615

Localisation updates from https://translatewiki.net.

Change-Id: I896047415adbedd7d474845a1dac4531d9c8d196

Always put type information before variable name for @var tags

Like other tags, @var should have the type information before the
variable name.

<https://docs.phpdoc.org/references/phpdoc/tags/var.html>

Change-Id: I9eca6957b1990fa8cc687103dc02ee38af5f9086

Merge "SpecialUpload: Stop loading 'mediawiki.action.edit'"

Fix you -> your in 'pagedata-text' message

Bug: T174961
Change-Id: I7cfcfca4decac3e2afc09e58959a001ac13d3d49

Merge "Turn Special:NewFiles list of media types into list of checkboxes"

SpecialUpload: Stop loading 'mediawiki.action.edit'

The comment says it's for <charinsert> support, but that's not true
anymore, as the CharInsert extension itself takes care of loading its
script on Special:Upload (see CharInsert::onBeforePageDisplay()).

Back in the day, this module included a lot more stuff related to the
edit toolbar and such, so this might have been warranted, but there is
nothing useful for uploading left in it anymore.

It's very unintuitive that it would be loaded there; folks did not
realize that it is the case and recent changes to the module started
causing issues on Special:Upload.

Bug: T174935
Change-Id: I11e886801a27f796e5a5935ec9c93b1329cc5b87

Merge "Drop compatibility with PHP 5.3"

Merge "Fix typo in ILBFactory::waitForReplication phpdoc"

Merge "resourceloader: Preload base modules request from startup module"

Check for Wikidiff2 version 0.3 NOT for 0.3.0

version_compare( '0.3', '0.3.0', '>=' ) gives false, so we need to check
for 0.3 since the Wikidiff2 version is defined as 0.3 in
https://gerrit.wikimedia.org/r/#/c/319866/60/Wikidiff2.h

Bug: T166571
Change-Id: Id6e9f402af2fab784a3db2444f201022e2d0de3e

Localisation updates from https://translatewiki.net.

Change-Id: I7b033dfc8380deaed29661b51cacc29f08c5b7f6

Localisation updates from https://translatewiki.net.

Change-Id: I5ae624cf1b2ed597abb2ffb96dc6f13267e93967

Merge "EditPage: Remove unused variable in getCheckboxesOOUI()"

EditPage: Deprecate getCheckboxes() and getCheckboxesOOUI()

Follow-up to 478caa076f75fde935c66eb9334410d868c30818, which asked
for someone else to disentangle that. After this patch, we're left
with separate blocks of code that can just be deleted when we remove
the EditPageBeforeEditChecks hook or the getCheckboxes() function.

Change-Id: I88d2dda3a6e030f73e209e0d389d4c0aa6c45139

Drop compatibility with PHP 5.3

Since 5.4, $this works inside closures.

Change-Id: I6987f76a1e33d6d7ab020a57186e5477186bbf2e

TableSorter: Sort unrecognized dates as -Infinity

Follow-up to: I664b4cc
Bug: T174814
Change-Id: I028681172be7a30e51cab42847c4fc35e9358aca

Localisation updates from https://translatewiki.net.

Change-Id: Ibf8a472053f8dafc581fbe3bd8d9be188048995d

Merge "WLFilters: reword watchlist-details slightly"

Merge "mediawiki.special.changeslist.legend.js: Remove wait for document ready"

mediawiki.special.changeslist.legend.js: Remove wait for document ready

mw.hook( 'wikipage.content' ) gets fired after document ready.

Change-Id: Ia1536e734f8ae08cc7cc07c3b1854bdaafb54866

Merge "Remove reference to deprecated IDatabase->nextSequenceValue()"

WLFilters: reword watchlist-details slightly

It won't show up like this on en.wp due to
site customization but other wikis will see it.

Bug: T172030
Change-Id: I2dbaad3c5f8200d95431fda6f246decdb1633c62

Merge "EditPage: Restore getSummaryInput(); deprecate it and getSummaryInputOOUI()"

EditPage: Restore getSummaryInput(); deprecate it and getSummaryInputOOUI()

Follow-up to 478caa076f75fde935c66eb9334410d868c30818.
We can't just remove it without a deprecation.

Change-Id: I3286e8de12bacb50a6d080477adbecf09b3be71c

Handle comment truncation in CommentStore

Since the caller doesn't (and shouldn't) know whether CommentStore is
using the old or the new schema, it should leave truncation of comments
to CommentStore.

Change-Id: I92954c922514271d774518d6a6c28a01f33c88c2

Merge "WLFilters: fix msg keys to comply with standard"

Merge "RCFilters: Minimize vertical spacing; get rid of legend and <br>s"

WLFilters: fix msg keys to comply with standard

Change-Id: Iea19ddb38b362fb01535cde595d82cc7fd1ad479

RCFilters: Minimize vertical spacing; get rid of legend and <br>s

Bug: T174121
Change-Id: I2249bf3d313232fe2f0dff6e34c5140885a13735

Remove patch-log_search-rename-index.sql from MW 1.16 updaters

Otherwise, the PK on log_search keeps getting dropped, ls_field_val readded and
this is then repeated on every run of update.php

Change-Id: I6b00933c29b5e24ec2833001fcf45053292d190c
Follows-up: Id635297838938c7c5dfe65d45285a4d16d65152d

Remove reference to deprecated IDatabase->nextSequenceValue()

The method was deprecated and made unnecessary in Ib308190c.

Change-Id: I1729ac0b3a88270a4c2f064187a2472112aaeb1e

Merge "Re add wpScrolltop id in EditPage"

Merge "Deprecate IDatabase::nextSequenceValue()"

Re add wpScrolltop id in EditPage

Regression from:
I9bd66b8444c069b4055fc5c3396256b688b78561

Bug: T174723
Change-Id: I2c174eaab822b5990663503ca9258a5811425054

Localisation updates from https://translatewiki.net.

Change-Id: Ib2b10d3b2e691568eafcdf2342cd703487013dca

Merge "EtcdConfig: allow slashes in config key names"

EtcdConfig: allow slashes in config key names

Allowing slashes in config key names allows us to trivially support the
proposed hierarchical structure with a single EtcdConfig object, by
fetching values with the relevant prefixes on startup.

Bug: T156924
Change-Id: Ica0914e61baba9c0462481925be15d79b66dc342

resourceloader: Preload base modules request from startup module

This allows browsers that support W3C Preload <https://www.w3.org/TR/preload/>,
to start downloading and parsing the second load.php JavaScript request
as soon as the headers of the startup module arrive.

Before:
- HTML request, response start.
|\- Discover <script src async> for startup.js, request, response start.
| \ - Startup response end, JS parsing.
|  |
|__|- HTML parsing and rendering yields for async JS execution.
   \- JavaScript inserts <script>, Base-modules request starts. <<<

After:
- HTML request, response start.
|\- Discover <script src async> for startup.js, request, response start
  \ - Base-modules request starts. <<<
|  |- Startup response end, JS parsing.
|  |
|__|- HTML parsing and rendering yields for async JS execution.
   \- JavaScript inserts <script>. (Base-modules req on-going, or done)

On local testing with "Slow 3G" network throttling in Chrome,
this reduced page load time (window.onload, time to interaction for JS)
by about 1 second (from 14s to 13s).
See also https://phabricator.wikimedia.org/T164299#3572231

Bug: T164299
Change-Id: I7047f4ab881947cf3392256087cc5a0cb177dd3a

Merge "Revert "Fix link prefix/suffixes around Category and Language links.""

Revert "Fix link prefix/suffixes around Category and Language links."

This reverts commit c66c9aa5352743b2f87bc4627a1be529c5d18535.

Bug: T174639
Change-Id: Ibf6d3780f384ba8edc80bf28c893f1aee8ce28a8

Merge "Move section ID fallbacks into headers themselves"

Merge "WLFilters: fix server-side tag filtering"

Merge "WLFilters: wrap Watchlist-details in a div instead of a p"

Merge "WLFilters: adapt to customized watchlist-details"

Merge "EditPage: Make getCheckboxesDefinition() public"

WLFilters: wrap Watchlist-details in a div instead of a p

Bug: T172030
Change-Id: I65c50d75b21b3e5d7b69fdecb79c032f5d1d8853

EditPage: Make getCheckboxesDefinition() public

Bug: T174613
Change-Id: I2dc4104580ac36551ec1ca0deed38ce50e5473ac

Deprecate IDatabase::nextSequenceValue()

It's often forgotten because MySQL and Sqlite don't use it, the only
users are PostgreSQL and Oracle. And when used, if inserts to multiple
tables are being done it's easy to get the ordering wrong.

This patch reimplements DatabasePostgres::insertId() in terms of PG's
lastval() function, and adds triggers to the Oracle schema to make it
work the same as the other databases.

Bug: T164900
Change-Id: Ib308190c52673a9266c8495a589ae644f9fbefce

EditPage: Remove unused variable in getCheckboxesOOUI()

This was unnecessarily copypasted from getCheckboxes().
The label's 'id' attribute is correctly being set below.

Change-Id: I1557461bfd16c22bb949e26d419e154f80c963d7

Merge "RCFilters: Make top links in RecentChanges expand to full-width"

WLFilters: adapt to customized watchlist-details

The watchlist-details message is sometimes customized
to add community notices, which makes it much taller.

This fixes the position of the 'edit watchlist' button
so it's always right and bottom aligned regardless of
the size of the watchlist-details section.

Bug: T172030
Change-Id: I8b6b9d799f7929a1d78c8f0e30020a4222fc3bb6

UploadFromUrl: Fix typo

Change-Id: I4df0db5b4a0df0191e1eaa305fc49f1959c4a354

WLFilters: fix server-side tag filtering

Tag filtering was enabled in the UI but the
backend was not properly wired for it.

Change-Id: I1c50b67b0a4aaddb561e79fdacd9849ffe0ded8c

Fix typo in ILBFactory::waitForReplication phpdoc

Change-Id: Ifd79c80af49b03744190e4c5af56c904550fffcd

RCFilters: Make top links in RecentChanges expand to full-width

Bug: T174617
Change-Id: Iff4c563b894eaaf713d3818a97928d0342b1ca23

DB excepions: consistently declare constructors as public

Also DBTransactionSizeError::getKey() while we're here

Change-Id: Ic76abf3ca0bf74dd96018d67da466365957bf3b1

Fix various PostgreSQL failures

* Fix schema for image_comment_temp.
* Provide values in CommentStoreTest::provideInsertRoundTrip() for
  columns where the PG schema doesn't have a default value but the MySQL
  schema does.
* Call nextSequenceValue() from CommentStoreTest::testInsertRoundTrip().
* Correctly handle $options being the string 'FOR UPDATE' in
  DatabasePostgres::selectSQLText()
* Correctly handle the initial table in DatabasePostgres::selectSQLText() FOR
  UPDATE mangling.
* Correctly handle aliases in DatabasePostgres::selectSQLText() FOR
  UPDATE mangling.

Tests in PG are still going to be broken thanks to the fact that
nextSequenceValue() and insertId() can't be separated by another
nextSequenceValue()/insertId() pair. That should be taken care of by
T164898/T164900.

Change-Id: Ia770a003ca9170ab8bcc1436d8afe30700e00ada

WLFilters: describe how unseen changes should look

Makes the unseen changes description customizable
based on whether or not the beta feature is enabled.

Bug: T172030
Change-Id: I0cef7e5fb261e12f3ca069e7409f3aea3fc874cf

Merge "Only do one query for page ids in PageProps::getGoodIDs"

Merge "Fix highlight display for enhanced mode"

Merge "WLFilters: convert 'edit watchlist' button to new UX"

Merge "Batch request titles for cdn purge from db"

Only do one query for page ids in PageProps::getGoodIDs

Change-Id: I0d1cff3173d54de0b8fadb5bf9766789ffe030a4

Fix highlight display for enhanced mode

Bug: T170875
Change-Id: Ibd7d447aba1ae5576ea637a2d5ef06cb3f622b26

Batch request titles for cdn purge from db

Xenon shows a significant amount of time is spent individually fetching
the content model for titles being purged. It's probably not a huge
improvement but lets fetch them all in one go instead of lazily as
needed.

Change-Id: I71af6f13c84e740c1367617d7a51210501363b33

WLFilters: convert 'edit watchlist' button to new UX

Bug: T172030
Change-Id: Ifcde40c66fbb6543d4f2aebfc8499c03533b8b91

Merge "RCFilters: rephrase the feedback link text"

Merge "Fix index in PostgreSQL schema"

Merge "Prefix comment table when adding indexes"

Prefix comment table when adding indexes

Change-Id: Ie87d2ef68050a402d6c81ca98cc7402f7c5f14aa
Follows-Up: Ic3a434c061ed6e443ea072bc62dda09acbeeed7f

Merge "Use HTTPS protocol in resources/src/mediawiki.toolbar/images/ksh/LICENSE"

Fix index in PostgreSQL schema

Followup to Ic3a434c06

Change-Id: I68966726f91b1854e845b2a0757b599644749b4f

Turn Special:NewFiles list of media types into list of checkboxes

It's a pretty short, finite list of media types, so we don't need
it to be this dropdown thingy.
Just a list of checkboxes is more convenient because it'll be
easy to see which of the media types are missing/not selected.

Bug: T174518
Change-Id: Ice768f0808eb9d70f682697f48272795417bbe74

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"

RCFilters: rephrase the feedback link text

It now works fine whether the filters you are using
are in beta (WL) or not (RC, soon).

Bug: T174411
Change-Id: I1714d70f3f1016cba2d06ea0c11ca7c664735905

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"