Merge "FormOptions: Document getValueReal(), consistency fixes"

Merge "Split changes list classes into separate files"

Split changes list classes into separate files

Having them separate improves readability of the code
and makes it easier to work with this code.

Change-Id: Ic6aaf1ace3640f66ad41c2d85b4f7d740b259e0c

Merge "Tweak bidi on Special:Contributions/newbies"

mediawiki.notification: Expose close method to hide a Notification

Bug: 51338
Change-Id: Icd7fd1ac328094ddc2560d4804dd52fc85879867

Merge "Added more Message parameter functions"

Added more Message parameter functions

Similar to numParams(), added functions for other parameters
that can be formatted by the Language class. Adds functions for
expiry, size, timePeriod, duration, and bitrate parameters so
that the formatting doesn't have to be done at the caller.

Change-Id: I7b435fcc11824ead55e4c0f5512418187eae9a6f

Merge "Rename Toolbox to Tools"

Merge "Correct release notes for I9dd9f0fd499c37f2"

Correct release notes for I9dd9f0fd499c37f2

Change-Id: I8f6e84b958d1bdadb578de655c4306f6b9d1c873

Merge "deleteEqualMessages: Exclude messages that are empty by default"

filebackend: try to combine SH and EX lock acquisition

* getPathsToLockForOpsInternal() now returns an array in the format LockManager can use
* Also fixed getScopedLocksForOps() for multiwrite backend (it should accept proxy backend paths)
* Updated getScopedFileLocks() docs

Change-Id: Id0dc50c159c5fbc8fca10a9e46c5be23afcb3e9a

Small JobQueueFederated performance tweaks

* Made "find non-empty queues" and isEmpty() checks start with the most
  weighted partitions first
* Cleaned up a few exception catches to use JobQueueError
* Added short-circuit for doGetSiblingQueuesWithJobs()

Change-Id: I4d96a8c61c6b1c41dbb44000ccda0fa05a418a09

Fixed a bunch of annoying deprecation notices

Change-Id: Ic2eb0ea94d8e27574f7dc76f1b448ccd6c01bb98

Rename Toolbox to Tools

The name "Toolbox" is strangely anachronistic. Other functions in the
sidebar such as languages and navigation use very simple, functional
names.

Most translations already use a single word with meaning equivalent to
simply "Tools"; several use literal translations of "Box of tools",
which tends to look rather awkward.

Bug: 54910
Change-Id: I9ba287ec6426632847fe7d3a0d71948838496317

Reduce excess lock contention and chance for failure on rename

* DB locks and transactions were getting held up by sometimes slow purges (10+ seconds)

Change-Id: I0eead60e8faf5aee6b2ddf59cfdd928f9c8083ff

Localisation updates from translatewiki.net.

Change-Id: I239f39d2a75aabffa1586776bcb8fb3c447c5df4

Merge "Split change_tag sql patch"

Merge "Add @covers tags for /content files"

Merge "Add @covers for GenderCache + MessageCache tests"

Merge "Add @covers for main Database test types"

Merge "importImages.php: Correctly generate automatic summary"

Merge "Removed unused const QoS_Atomic constant (odd casing)"

importImages.php: Correctly generate automatic summary

If the summary parameter is not set, then use the page text of the
given page, NOT that of the first image uploaded.

Bug: 55885
Change-Id: I00ced9d2ce62fc1a00f26ff513866359cf96d535

Don't blindly overwrite $options in importImages.php

Bug: 55883
Change-Id: I695e080000f6abec3ded966f12af3e25c585c3ba

Add @covers for GenderCache + MessageCache tests

Change-Id: I379f2788c1b308354a3dbb73c889aa4a90b2cd20

Add @covers tags for /content files

Change-Id: Ied40e5afb1027f7d8ff46ffbf722ff6558eed880

Add @covers for main Database test types

Change-Id: I010ff5ce7afd85c7f9d528a98f2448547874b8b3

Tweak bidi on Special:Contributions/newbies

Special:Contributions/newbies when page title is not RTL will mixed
username and page title.

Change-Id: Icccf953cd2b6784f83fdd6a5775b6af8b1216ed5

Removed unused const QoS_Atomic constant (odd casing)

Change-Id: I2b03ad1700e3a1184f7209394096cde61d23c6a5

jobqueue: various cleanups to JobQueueFederated

* The relevant partition information is now stored in
  the fields instead of the general information for all wikis.
* Also improved hash ring caching and made getCoalesceLocationInternal()
  more robust.
* Improved availability of sibling queue methods.

Change-Id: I7ac268bb1db4615e0f735576c506593d5688e17e

Try to make sure jobs will not get pushed twice on failures

* Some queues like the DB one, might split up batches into chunks
  which we do not want when used by federated queues (it already
  manages chunking anyway). If only X of Y got inserted, then they
  would all get inserted into another partition, causing duplicates

Change-Id: Id73b34f3500f8aeaf456e6bcac862a1e4832c870

Merge "Fix SQLite exception when $wgDBuser is set"

Fix SQLite exception when $wgDBuser is set

Fixed from several different directions:
* The error reporting was screwed up, so I wrapped the PDOException
  in a DBUnexpectedError.
* Make $db->open() trigger a close/open sequence if called a second
  time, like MySQL, instead of breaking mTrxLevel.
* Don't call $this->open() twice from the constructor, even if $user is
  set.

The bug dates to r84485. The nasty hack allowing you to construct a
do-nothing database object with "new Database()" dates back to 2004 and
has probably outlived its usefulness -- noted this. It was formerly used
by LoadBalancer::reportConnectionError().

Bug: 49254
Change-Id: I571f9209bec8e8ed5058b6327e1738eb4315d5a0

Add $wgEnableParserLimitReporting to control whether the NewPP limit
report is included as a HTML comment.

Bug: 26792
Change-Id: I2b2b3c9d76487168243b733c1154c4c26c7de3d6

Merge "mediawiki.api: add postWithToken and getToken"

mediawiki.api: add postWithToken and getToken

Similar to postWithEditToken in mw.api.edit, but more generic.

Change-Id: Ic2af66b91a996bc9b46dc3e999b4099541573b08

Merge "If $wgShowHostnames is true, add parsed by to NewPP limit report"

Localisation updates from translatewiki.net.

Change-Id: I3560103a9de1e613c5878422172e070878ec7d43

Add archive, externallinks PK

* New fields: ar_id, el_id. el_id is presently not used for
  anything, but will help with online schema changes.

Bug: 15441
Bug: 39675
Change-Id: Ib6b0fc3736d173fa4ba7b786ecfc710b2f4711bb

Fix mysqli support

Exception was thrown if $dbType=mysql, regardless of whether a driver
was available. Also, the installer support was broken if the mysql
driver was missing.

Change-Id: I29df3fc9b3bf5743c1365f757e0cf899d5b96ae9

Resubmit "Add support for mysqli extension"

This reverts commit 1d7e12179df0db332f9ac139e8b7d831e04a6a10.

Change-Id: Id1a3fc34e94006b880f58de83e5e7c70b21d26ed

Merge "Revert "Add new recentchanges field rc_source to replace rc_type""

Revert "Add new recentchanges field rc_source to replace rc_type"

No feature flag, and also not deployed on WMF wikis. Asking for schema changes by the time we branch tomorrow is just unfair

This reverts commit e478c230f6d3f1b7638d4443682eff0df7650def.

Change-Id: I125c30e76fa04679cc04ae7b8728ae3a61f35194

Vector: Fix font size regression

As Krinkle noted the font size of the body content ballooned with
Ica0b69ad95051e15cd8ed9888927e034c13fbf92 - this rectifies it.

Change-Id: I3a5a159cbb04a1a24277596ca304805a84d3b77f

Merge "Revert "Add support for mysqli extension""

Revert "Add support for mysqli extension"

Throws exception unconditionally, installer support is totally broken.

This reverts commit b2d64432ee02661ea29cb727686e10e7a4980dfb.

Change-Id: I2edb78440806754c42d48c5da2ff74b4cb40fa2d

Merge "Added a "syncviadelete" param to copyFileBackend script"

Merge "Add new recentchanges field rc_source to replace rc_type"

Added a "syncviadelete" param to copyFileBackend script

* Added a bit of tabbing to clarify the output

Change-Id: I4b0095595d742452b151e6ed6765a9dc805a55ae

Add support for mysqli extension

DatabaseBase::factory()'s parameter array now takes an optional
'driver' variable to allow specifying a specific driver for the
database class if we support more than one.

This is so we can gracefully support mysql/mysqli as the former is
deprecated, but the concept could be extended to other databases
as well.

Bug: 45288
Co-Author: Chad Horohoe <chadh@wikimedia.org>
Change-Id: I6733fe21c4aa7443e409c5dfa7c789552b2b62b7

Localisation updates from translatewiki.net.

Change-Id: Id8e5920d1a5f58fc904a812369df35fe2f8f448e

Merge "Keep returnto, returntoquery on Special:CreateAccount redirect"

Add new recentchanges field rc_source to replace rc_type

The existing field to differentiate between kinds of recentchanges rows is
the rc_type field. We want to allow extensions to insert their own custom
data into recentchanges, but we have learned via the NS_* series of constants
that requiring extensions to "register" a specific number is very error prone.

The solution, which this commit implements the first phase of, is to utilize
a new 16 byte string field rc_source.  Within that field change types will be
prefixed strings such as 'mw.edit' and 'mw.new'.

This commit adds the new field and begins populating it with data.  At some
point in the future the rc_type field will be dropped.  While WMF wiki's will
simply wait out the 30 day recentchanges history, other wiki's have the option
of letting update.php populate rc_source, or manually applying the db change and
utilizing the PopulateRecentChangeSource maintenance script.

Change-Id: Iaddd6c446373a68d31586ed54346db7d04e13b2c

Merge "Use max() for job count for sanity"

Merge "Support rate-limiting thumbnail generation"

Merge "Move noflip annotations into rules themselves"

Move noflip annotations into rules themselves

In LESS these seem to be honoured better inside the rule.

Bug: 55779
Change-Id: I412575f37dd647d018d46b96f5642f5d68ebeb74

Merge "Include miscellaneous query parameters in $wgSecureLogin redirect"

Merge "Remove unnecessary defined( 'MEDIAWIKI' ) check"

Merge "Reindent UsercreateTemplate and UserloginTemplate"

Merge "Add a line of explanation to Html::expandAttributes"

Remove unnecessary defined( 'MEDIAWIKI' ) check

This file doesn't run any code, it just defines a class.

Change-Id: I92664e2c1a5617a28c0fb22c1d1fd99a46224a04

Reindent UsercreateTemplate and UserloginTemplate

A perceptive reviewer will note several issues with this code,
including at least one outright bug. I intend to fix them in
subsequent commit; I tried to avoid any non-whitespace changes
here to make it at all possible to review this.

This file has several lines longer than the phpcs limit of
100 characters, and several cases of the <?php } ?> pattern
phpcs complains about. I don't think it makes sense to enforce
these rules here; the code would IMO become less readable.

Change-Id: I95ab0f644fd5963878018f483f22e650867b5135

Add a line of explanation to Html::expandAttributes

Change-Id: I7ad453638075a5e875c6c97df10690747c2488f0

Include miscellaneous query parameters in $wgSecureLogin redirect

The redirect from http to https should include any extra query
parameters, such as the ones for "campaigns".

But we still want to override returnto and returntoquery (in case they
were changed in ::load(), e.g. lines 139–140), and we omit any 'title'
parameter that may have been passed.

Bug: 55761
Change-Id: Icc8b4132e8682d782b0c049631328b94745eb355

Fix double-space in messages

Change-Id: I5dbc0512b6447dfeb7c287e212245e86d105928b

Keep returnto, returntoquery on Special:CreateAccount redirect

Bug: 55765
Change-Id: I1d05a1b91b2ac4097e09d11510d53fdfce1e0565

Escape backslashes in image metadata

Uploading a file with backslashes in the metadata
causes database exception on PostgreSQL, because
backslashes are sent to the 'bytea' datatype unencoded.

Bug: 52017
Change-Id: Ife56cffe3df83ec1a7bd39cdc4a0489f23354494

Merge "In maintenance/postgres/tables.sql, the profiling table should be defined like this:"

In maintenance/postgres/tables.sql, the profiling table should be defined like
this:

CREATE TABLE profiling (
  pf_count   INTEGER         NOT NULL DEFAULT 0,
  pf_time    FLOAT           NOT NULL DEFAULT 0,
  pf_memory  FLOAT           NOT NULL DEFAULT 0,
  pf_name    TEXT            NOT NULL,
  pf_server  TEXT            NULL
);

The current use of NUMERIC(18,10) very rapidly overflows the pf_memory column,
generating errors.  Also, the NUMERIC is very much slower than float, and in
this case it has no advantages.

Bug: 55722
Change-Id: I48b00d55aaed821a4ceb9365033817a3b477d71a

mediawiki.Title: Fix doc reference error

Follows-up 165373d which introduced a reference to a browser
global not in our index (causing a jsduck error).

Unless we're using an element that has special DOM methods I
don't think it's worth adding element subclasses to the doc index.
We generally just use HTMLElement so changed it to that.

Also made a minor optimisation in merging the img/src assignments
by using .prop( 'src' ) and .src directly. And while at it also
reduced it to direct access with [0] instead of .get().

Change-Id: I3a521eda0aeeb7847a96d474316d1d5e6e8917d1

Localisation updates from translatewiki.net.

Change-Id: Ie45e695bf4dc5fc8177ed33e400f7ca2000519a0

Merge "Remove comment which is plainly wrong"

Remove comment which is plainly wrong

Change-Id: I89bb9243a19e0355260e4e1c217e3868014c4886

Merge "Expose changed links in LinksUpdate"

Expose changed links in LinksUpdate

These links are only really available during the LinksUpdateComplete
hook.

To make sure they are always available this also removes dump links
updates which would prevent this data from being built.

Change-Id: I115f8cf9d3b0a1dba979ceb58b5f14dd0b76ec49

Add vform format to HTMLForm, use in PasswordReset

This adds a new 'vform' display format to HTMLForm which styles forms
using it in the new compact stacked vertical format:
* Applies class mw-ui-vform to the form.
* Gives Submit button 'mw-ui-{button,big,block,primary}' classes and
  spaced block styling.
* Styles the error class.
* HTMLForm divs are too nested to get styling "for free", so add
  .mw-ui-vform-div selector to Agora CSS and apply this class to form
  divs (maybe not ideal fix).
* HTMLCheckField nests the checkbox inside label.
* Add method to setShowEmptyLabel(); vform format sets this false as it
  doesn't want generated HTML for empty labels adding vertical space.

In FormSpecialPage, don't set the wrapper legend with its old-school
line around the form if in vform style.

Build new mediawiki.ui CSS with the .mw-ui-vform-div styling

Special:PasswordReset requests the new styling with setDisplayFormat( 'vform' )
and turns off the wrapper legend.

(This version of the patch doesn't switch the default format of HTMLForm,
individual forms have to choose it.)

Notes are at
https://www.mediawiki.org/wiki/Agora/Engineering#HTMLForm_Issues

Change-Id: Id03d185bbee990595bfc469a61163cc598fc3441

Merge "Vector: New beta module with new typography styles"

Add configurable delay between purgeChangedPages batches

UDP flooding is possible when purging large numbers of files. This
change adds a command line switch to the purgeChangedPages.php
maintenance script to insert an artificial delay between purge batches.
The intent is to allow the network time to process the batch that was
just sent before flooding with a second batch.

`--sleep-per-batch` can be used in combination with `--batch-size` to
manage network and to some extent query load. A suggested starting point
would be to use `--sleep-per-batch=500` in concert with the default
batch size of 100 to target a maximum rate of 200 packets/second.

Bug: 55632
Change-Id: Ibfc54b1767f145098465404a2b23cd92852e41fd

If $wgShowHostnames is true, add parsed by to NewPP limit report

Change-Id: I89d6588d77864598ddc5f23e7befc5d46e9035d2

Simplified IPv6 regex per comment

* We already require PHP 5.3.2, which includes PCRE 7.2+.
  PHP 5.2.4 includes PCRE 7.2 per http://php.net/ChangeLog-5.php.

Change-Id: Ie9c1a0a93bef30a3a67e0140e621814351f8cea1

resourceloader: Don't catch LESS error in ResourceLoaderFileModule

Instead let it throw up to ResourceLoader::makeModuleResponse.. By using
the central logic for exception handling the parse error becomes more
discoverable and obvious.

* The module is marked as in "error" state (currently it would still serve
  the result of file A and C, even if file B has a parse error).
  This matches the behaviour of the javascript checker we have.
* The exception comment is produced on top of the load.php response
  (where users expect it) instead of buried somewhere between the
  different modules and the different files within the modules.
* The exception comment will not be stripped out by CSSMin.
* DRY :)

Without this, debugging less parse errors is harder than it should be.

Bug: 55442
Change-Id: I6510b43f02cf82568b7fd2421209a8e636d5e115

Merge "Refresh Deletereason-dropdown"

Merge "Add APIQuerySiteInfoStatisticsInfo hook"

Document class property and break long lines

Change-Id: Id391dc8911de8b6df0fb95cba58cffee972ca507

Refresh Deletereason-dropdown

I went around to a few MW installations on the web, and all that had
modified this message added "Spam". (Also per the bug.) And "Broken
redirect" seems a sensible, universal addition as well.

Also putting in order of "commonness" and fixing the inconsistent spacing
of the bulleted list - it annoys me every time.

Bug: 47573
Change-Id: If50f30df95f969f0b56a9058fc57e7c542b7215a

Localisation updates from translatewiki.net.

Change-Id: I2e39d39032e0a7fb32b9f5385f95f3047bbdb264

exception: Move logging logic to static method of MWExceptionHandler

Follows-up c28251a9fd, which unconditionally called logException
on objects that aren't always instances of MWException.

This makes it possible to, for example, log PHP errors or errors
from Less in ResourceLoader to be logged as well (which naturally
don't have a logException method).

As a result of the handling now being generic, non-MWException
errors will now be loggable, and also have their ID fixed between
HTML output and the debug log entry.

* Not yet removing MWException::getLogMessage (introduced in bcb9f9e1c)
  as it has been around since 2006 (released in 1.8.0).

* Not yet removing MWException::getLogId (introduced in 70841c5867)
  as it was part of the 1.20.0 release.

* Removed MWException::logException (introduced in c28251a9fd)
  as it was only added a few weeks ago within this release cycle.

Change-Id: Iab98e3a7a9b78d8602e69e0571b35cf107a96b72

vector: Apply coding conventions to screen.less and minor cleanup

Follows-up 08cf0b1, 7e953bbe.

* Removed obsolete comment about @embed, since this file no longer
  uses that.

* Removed ".less" per LESS convention.

* Made whitespace consistent with coding conventions.
  No spaces inside .transition(), just like there are no spaces
  in .background-image() or url().

* Line breaks between different rules.

Change-Id: I5956d179eb46f6ee4e0c313eafe008c434e61e31

Remove calls to deprecated HTMLForm#setTitle

Use DerivativeContext with modified title instead.

Most of those should be using FormSpecialPage…

Change-Id: Ia66f3b4b22866cc4ac4ac7b1f2fc2a4b1a9e73f6

Rename SpecialRecentChanges::addRecentChangesJS() to addModules()

…and make it protected. It makes no sense to include class name in the
function name, especially since subclasses might want to customize it.

Change-Id: Ic0d23f4400b88596fdb11f6c0d8aefc9aea589ba

SpecialRecentChanges: Don't call getUser() repeatedly in getDefaultOptions()

Change-Id: If0ec42e89887c98968cd7878e45b1944e9fdf7ac

SpecialWatchlist: Remove old irrelevant comment

The code it was commenting has been removed in r10652 in 2005…

Change-Id: I33846196c1f62800db2a37b6b03bdb47280af7d4

SpecialWatchlist: Remove dead cutoff code

Added back in r1532, it got broken in one of the numerous refactors
since then and now only triggers when 'days' request parameter is
passed explicitly and is malformed.

Just use floatval() instead.

Change-Id: I20c1e816a11fca5037af93bf81b46839f2a5efb0

Localisation updates from translatewiki.net.

Change-Id: I3b0c0763a0f2a7b40c4b25821cffac6b136ea6af

Ensure that $call['args'] is set before using it

Fixes PHP Notice:  Undefined index: args in /home/ki/Projects/mediawiki/
core/includes/Exception.php on line 755 when running code from
maintenance/eval.php .

Change-Id: Ic4a806232aadae0a60f5bc06de1604d9c5996e6f

FormOptions: Document getValueReal(), consistency fixes

I missed it in my previous doco commit, good opportunity
to try out the fancy new tool.

Change-Id: I52005b243042c3d66250d1cfecfceabbd370ea5c

Pass Context to User::resetOptions

Change-Id: I0444777697ed9a8b8afdd0e5a96841848b162a7a

Merge "Add bot to createAndPromote.php groups"