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

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

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

Pass Context to User::resetOptions

Change-Id: I0444777697ed9a8b8afdd0e5a96841848b162a7a

Merge "Add bot to createAndPromote.php groups"

Localisation updates from translatewiki.net.

Change-Id: I018c32bec9b2b04590657afd96606ed473cdfed6

Merge "Revert visibility change to HTMLForm $typeMappings"

Update formatting

Change-Id: If3dd6a1ac4b16479244a847420f0488ba8c51e2b

FormOptions: More documentation!

Change-Id: I088b07c1672426c45d995fdf96ac7fc7bda21307

Revert visibility change to HTMLForm $typeMappings

Change Id5668141835e22e3e1b043109e50d73ccd6b1229 changed the visibility
of static $typeMappings to private.  It turns out this breaks
http://www.blue-spice.org/ who want to add their own HTMLForm types and
have no way to extend or override this except by changing it.  So set it
to public static.

As https://www.mediawiki.org/wiki/Manual:Coding_conventions/PHP says,
"It's generally a good idea to avoid visibility changes unless you're
making changes to the function which would break old uses of it anyway."
That trumps PHP CodeSniffer warnings.

Change-Id: Ife82596e96a4650b977422aae6b32f24a392d5fb

Add APIQuerySiteInfoStatisticsInfo hook

Allows extensions to modify the output of
action=query&meta=siteinfo&siprop=statstics

Accompanies the SpecialStatsAddExtra hook for Special:Statistics

Change-Id: I712748077a0ac2fb729bfe3b6616ac4981cb754e

Modernize SquidUpdate php code

- Update to PHP5 object coding style
- Fix @param comments using wrong argument order
- Add some phpdoc comments

Change-Id: Ia3190a6276e807ebef4f4aaeedc0e39819dc992a

Merge "Blacklist Nokia's Ovi Browser from ResourceLoader"

Blacklist Nokia's Ovi Browser from ResourceLoader

It runs scripts proxy-side much like Opera Mini does, resulting in nothing
resembling a real JS engine. This causes problems and the browser is popular
enough to warrant a block.

Bug: 55600

Change-Id: I2c1ee78828c2b9cead4f82ef795972a4c28140b4

Localisation updates from translatewiki.net.

Change-Id: I8f4d44a3917bc7264d7dacc4b1e75641dab845c0

Merge "exception: Account for $call['file'] and $call['line'] being unset"

exception: Account for $call['file'] and $call['line'] being unset

Bug: 55634
Change-Id: I1173216cade73216848816f6bb51e54096abdfde

Merge "Add script to purge changed files from cache"

Merge "mw.inspect: decline to report module sizes when in debug mode"

Merge "FormatJson: minor cleanup"

Merge "Add an explicit targets declaration for mediawiki.inspect & $.byteLength"

Add an explicit targets declaration for mediawiki.inspect & $.byteLength

Jon Robson said[1] he'd be interested in making mediawiki.inspect usable on
the mobile site by specifying both mobile & desktop targets explicitly in the
resource declaration. That's what this patch does. The cost to this is very
minimal because the module is never loaded by default. mw.loader.inspect()
wraps the call to the inspector with an mw.loader.using() call that causes the
module to be retrieved. It is not retrieved otherwise.

[1]: http://lists.wikimedia.org/pipermail/wikitech-l/2013-October/072381.html

Change-Id: I7cf292d3b8fb554bd1fc2143ca45399efccb70aa

mw.inspect: decline to report module sizes when in debug mode

- In debug mode, 1 resource = 1 request, so module size is easier to determine
  via the network view of the developer tools in Firefox & WebKit browsers.
- Determining the module size in debug mode in JavaScript would require
  modifying core ResourceLoader code or re-fetching each resource with a
  separate XHR to measure its size.
- It's not worth it.

Bug: 55553
Change-Id: I73d1bb4141c1b6ed64a72fdc416048937a2e1146

mediawiki.page.gallery: Clean up documentation for hook

Follows-up 14b3f87.

Change-Id: Ia917516d985a6e6f6e4bbca9ea8b5437175a6aa8

Vector: New beta module with new typography styles

To enable install Extension:BetaFeatures and Extension:VectorBeta
in LocalSettings.php

Changes to core:
* Refactor screen.less to make use of variables

Introduce dormant skins.vector.beta module
* Activated by https://gerrit.wikimedia.org/r/#/admin/projects/mediawiki/extensions/VectorBeta
* Load different environment variables depending on whether in a beta
  experiment or not
* Switch global fonts for text from browser default sans-serif to a
  specific sans-serif stack.
* Switch global fonts for headings to Georgia/serif
* Cleanup over-specific specifiers in process
* Adjust styling of navigation menus to focus on content
* Adjust styling of collapsible navigation menu to be less prominent

See https://www.mediawiki.org/wiki/Wikimedia_Foundation_Design/Typography

Change-Id: Ica0b69ad95051e15cd8ed9888927e034c13fbf92

Merge "Do not set wgServer in cli install unless explicitly passed"

Merge "Made JobQueueDB use getConnectionRef()"

Localisation updates from translatewiki.net.

Change-Id: I6ed3b4bc205bb8b6a09aea841a73aa0d8e317554

Merge "Handle raw <h#> when calculating $rawtoc"

Do not set wgServer in cli install unless explicitly passed

Currently, if we do not pass a wgServer value (via --server),
the CLI installer sets the value to the default value
specified in DefaultSettings.php - which is to 'guess' by
calling into WebRequest::detectServer. This yields terrible
results, since WebRequest::detectServer expects to be working
in the context of a Web Request - and hence with HTTP Host
header information. Since calling from the CLI does not give
it host header information, it falls back to 'localhost',
which is not the value you usually want.

If we just do not set wgServer when it is not specified,
it is automatically calculated on every request by
WebRequest::detectServer, which does a splendid job.

Bug: 55376
Change-Id: I5436dd8c340604cbb59406a507188e11c8f86e86

Remove silly test cases that are likely to fail

Nobody should be giving objects or arrays to addIdentifierQuotes() anyway

Change-Id: Ibebf5d32196ac5b80cd477aedfef982fd2c9a0e6

Merge "Fixed trx isolation bug that could cause text not to be found"

Made JobQueueDB use getConnectionRef()

* Also moved insertFields() function up a bit about DB connection methods

Change-Id: I7a9d9c9bc71038fe280542b13e0c0e94bfc4de1e

Merge "Made root job de-duplication work without cache setup"

Merge "Escape backticks when quoting MySQL identifiers"

Merge "Add mediawiki.inspect module"

Add mediawiki.inspect module

This patch adds a mediawiki.inspect ResourceLoader module. The module contains
tools for inspecting page composition and performance. It is currently
restricted to a set of static methods for querying the ResourceLoader module
registry, the most useful of which is #inspectLoadedModules, which prints a
list of loaded ResourceLoader modules and their size in kilobytes, ordered from
largest to smallest. This method has a lazy-loading alias: mw.loader.inspect.

Change-Id: I5a007fdf11196664f425cd7800eec7fc3fdb629f

Escape backticks when quoting MySQL identifiers

Previously mysql_real_escape_string() was improperly used which escapes
for inclusion in a plain string rather than an identifier.

Also adds basic test support for the DatabaseMysqlBase class.

Co-Authored by: Antoine Musso <hashar@free.fr>
Bug: 55427
Change-Id: Ic045e195c89d8d5d0f0edbda0cd1df781de7025c

Localisation updates from translatewiki.net.

Change-Id: I91e6b1a32aaca18175c97130685a26e9a5704f8d

Merge "Fix ForeignAPIRepo::fileExistsBatch()"

Merge "Tests for API section editing"

Tests for API section editing

Change-Id: Ibc05c54cd2c9e1775f72dd969e7677b5238e12ad

Merge "Remove wfDl() and cleanup everything it touched"

Merge "Fix PHP CodeSniffer warnings and errors"

Remove wfDl() and cleanup everything it touched

wfDl() is a wrapper around dl(), which is an evil function and
basically only works from the command line of Zend. Luckily
no extension has ever used this thing, so let's just remove it
outright.

For comparison, here's a list of places it does not work:
- hhvm
- php as apache module
- php compiled with zts support
- safe_mode
- Basically any shared host that cares about security

Most callers are using it to check for extension support and are
actually failing gracefully when wfDl() returns false. In these
places we're just going to use extension_loaded().

While we're at it, clean up some of the test skip logic in the
media tests so we can bail as early as possible if we know we
can't complete the test.

This also immediately removes $wgLoadFileinfoExtension. It's been
enabled by default since 5.3 and falls back gracefully when the
support isn't available.

Change-Id: Ieb430dfc74483731dde51d6e20fa700d641ba1f4

Handle raw <h#> when calculating $rawtoc

When the parser is constructing $rawtoc, it needs the sectionIndex
number to be able to calculate the byteoffset. This number is only
available for wikitext headings ("== foo =="), HTML headings
("<h2>foo</h2>") do not have it and the lack makes byteoffset be wrong
for all subsequent headings in the page.

To fix this, we just omit output of byteoffset in this situation.

Bug: 25203
Change-Id: I39e5faa4ac22d915f06125aac36ced11607b94a3

Fix ForeignAPIRepo::fileExistsBatch()

ForeignAPIRepo::fileExistsBatch() is extremely broken:
* If passed an array with numeric keys (e.g. as called from
  FileRepo::fileExists()), it will cache the fact that the item at index
  0 from the first call was found or not found, no matter what the
  actual file being queried is!
* But it doesn't even use its cache correctly. Rather than checking the
  value cached, it just assumes anything that was cached exists.
* It checks whether the page in the remote repository has a description
  page, not whether a file actually exists for the title.
* But it doesn't do that right either, so in the vast majority of cases
  it will decide everything exists.

All of this is offset by the fact that I can't find any code path that
would actually call this function.

Change-Id: I421e8892b642f5267b8dd755183ebd711db83152

Fixed trx isolation bug that could cause text not to be found

* Also added some logging for when revision text is not found

bug: 53687
Change-Id: I0993e8f49d36771cc526fd4afef67455209b8e3a

Fix PHP CodeSniffer warnings and errors

Also apply formatting convention.

Change-Id: I792210d0d1c5f9089a71cecba95e17822f6da0e3

Merge "Remove incorrect span in translation"

Remove incorrect span in translation

Probably originates from a bugged machine translation API.

Change-Id: I39c7d154ffc085991f683de107deb05d730f065d

Remove unused local variable

Change-Id: I93ce25fe158f7a82ade102953d6a2453620b7e0e

Merge "tests: remove date from @author docs statements"

Merge "Make items in the StringUtilsTest data provider easier to identify by adding names."

Merge "Fix PHP CodeSniffer errors and warnings"

Make items in the StringUtilsTest data provider easier to identify by adding names.

Change-Id: I026eff69236187ce3d2ff2fc261a5e1d9cd88b24

Update formatting

With some manual corrections.

Change-Id: Icd947288c32fdb90a0f35388963ef5a17ffa58e7

FormatJson: minor cleanup

* Prefer feature detection over version comparison.
* Prefer pre- over post- increment and decrement operators.
* Remove the statement that FormatJson::decode() decodes `true`,
  `false`, and `null` case insensitively. Nobody should assume
  this is (or is not) the case, even though the PHP manual says so.
* Avoid using the ternary operator with long strings; prior to
  PHP 5.4, the operator prevented the copy-on-write optimization.
* Avoid placing comments on the same lines as code.

Change-Id: I8fc88e9b7b49aa0cbd4128216557836a3b2cd011

Add some siteinfo to the repo info array

Used for linking to user pages on remote repos in the
MultimediaViewer extension.

Change-Id: Ia703d63e8e02b1e8fbc90b7d7f90a748d9e177ec

Quick refactor of the http logic in ForeignAPIRepo

Automatic caching, will enable slightly smaller patches elsewhere.

Resetting after failing at a rebase.

Change-Id: I6849107a3af773687a31155b23a2d7b4d7a60645

Merge "Fix revision table cleanup on delete"

Add script to purge changed files from cache

This maintenance script enhances the prior purgeDeletedFiles script to
optionally also process create and modify events that may have left
stale information in front-end caches.

Changes from purgeDeletedFiles:
* `--starttime` & `--endtime` are now required
* `--type` allows selecting 'created' and/or 'modified' events in
    addition to 'deleted' events.
* `--dry-run` to list affected files but perform no purges
* `--htcp-dest` option to restrict HTCP broadcast messages

Change-Id: I7181ea461ba9758747bff4fd70530d00dab492cc

mediawiki.action.edit.preview: Fix for LiquidThreads hack

LiquidThreads is loading this core module when rendering an
inline edit form. Because the form doesn't have the same ID
as the one in core, this broke after becd284.

It previously worked because the click event on the button (of
which LQT did duplicate the ID from core) bubbles up to the
document body and thus trigger this private handler of the
core edit preview module.

Change-Id: I9e35d0c48b86e1c9073b68313a8a330c6ea85b86

Merge "Localisation updates from translatewiki.net."

Merge "Add file-link "parser" to mw.Title from commons"

Localisation updates from translatewiki.net.

Change-Id: I331db0fe25638bdab35fbf54d00dd77f93488a7a

Add file-link "parser" to mw.Title from commons

Rillke has been amazing and provided us with a first pass at a file URL
parser for MediaWiki:

https://commons.wikimedia.org/wiki/MediaWiki:Gadget-libUtil.js

Hoping to use this for Extension:MultimediaViewer and others in the near
future - potentially replacing patchset Ib69d1ffdfd3b58c2e8e3314940a2cad311b85533

Also tests.

Change-Id: Ie31e6df376bda087e92ec889d54df3374ba96b5e

Merge "Allow $context->setTitle( null )"

Add doc comment to Status explaining use

Tim posted a nice explanation of the recommended pattern for using the
Status object to wikitech-l. This change copies that explanation into
the phpdoc comment block of the Status class in an attempt to propagate
the meme.

Change-Id: Ia98543caaa829cad443abf0f0f5038b3de943ef8

Fix PHP CodeSniffer errors and warnings

Change-Id: I573d73b4cf1dcddfe14e9d565e4f170ad024ea31

Merge "Fixup parameter type hints"

Merge "Add method parameter type documentation"

Merge "Move forceHTTPS check until after wgTitle is setup"

Added $wgExtensionEntryPointListFiles for use with mergeMessageFileList.php

Going to be used to change target file for labs vs production

Change-Id: Id4b16083435ef7f4fce31861c72889e664d07236

Add list=allfileusages

This allows the client to enumerate through entries of the imagelinks
table.

This can be useful to find images from foreign repos used on the own
wiki.

Change-Id: I2ad0579e3521868d39f7d495ea9c83a2cd9731d6

Fix help string to avoid PHP notice

Escape a $ symbol in help message and rephrase slightly.  Without this,
`php maintenance/language/checkDupeMessages.php --help` prints
PHP Notice:  Undefined variable: dupeMessages in
        language/checkDupeMessages.php on line 56

Change-Id: Ie7c0f07e68cee1f7cd2758f71b09d34e339d4b7b

Made root job de-duplication work without cache setup

bug: 55454
Change-Id: I9fbc1e34b4b771a16e567a1a16e2720af4379188

Merge "Localisation updates from translatewiki.net."