Merge "Rewrite ORMTable::unprefixFieldNames"

Merge "Remove unused variable in ProfilerMwprof"

Remove unused variable in ProfilerMwprof

Change-Id: I7643717c3c899999f8393a6af726397b28d441fc

Merge "SqlBagOStuff: fix percentage in deleteObjectsExpiringBefore()"

Always supported delayed jobs in JobQueueRedis for simplicity

* Previously, each job type had to be configured to allow delayed jobs

Change-Id: I5f3cf04deca76d6684aa6b14eea97a8673a7f5af

Merge "Added another size limit check to Job::toString"

Merge "Add Content-Length header for job queue requests"

Merge "Fix exception while bubbling titles"

Merge "Remove deprecated GlobalVarConfig::set()"

Removed deprecated Job methods

Change-Id: Ib7a06ab19b31472c99cdccd418011beb6f075ca4

Revert "Use a fixed regex for StripState"

Breaks extensions, doesn't entirely fix the problem it was meant to fix.

This reverts commit 6da3f169ac55ae87837a4ba3cf3e30f83fbf9d7d.

Change-Id: Ic193abcff8c72b0c8b434fcac514f88603a45beb

Merge "API: Internationalize all remaining core API modules"

Add Content-Length header for job queue requests

Include the Content-Length header in job queue POST requests to meet the
requirement by certain servers, avoiding HTTP 411 responses.

Bug: 72274
Change-Id: Icf34bca58c792225d735d576213d6887015459ca

Merge "Remove hitcounters and associated code"

Removed extra "tries" increment in the redis job queue

* This could result in jobs getting one less extra tries than they should

Change-Id: Ia2e82ca0965f6ce9e1354820eb03f40f047a949c

API: Internationalize all remaining core API modules

This also adds some new ApiBase::PARAM_* constants to generate more
helpful help, and a method to override the default description message
for the use of ApiDisabled and ApiQueryDisabled.

Bug: 71638
Change-Id: Ic0c3d232e0498d58a043037e2e0c6f0b1c3edad3

Avoid calling recyclePruneAndUndelayJobs() in daemonized mode

* This could cause problems if claimTTL is 0 in MW config but not
  in the runner JSON config.

Change-Id: I86418fd4426b58be1c17c97e026d14239f5e36de

Remove hitcounters and associated code

The hitcounter implementation in MediaWiki is flawed
and needs removal. For proper metrics, it is suggested to use
something like Piwik or Google Analytics.

RFC: https://www.mediawiki.org/wiki/Requests_for_comment/Removing_hit_counters_from_MediaWiki_core
Change-Id: I0e5006a7e8a09c800f8fa4effa9399e8afdd7a57

Localisation updates from https://translatewiki.net.

Change-Id: I071ca41395b65765a6506f7c89b63cfd613af93a

Merge "Revert "Add maxlength of 6 for time correction in preferences""

Revert "Add maxlength of 6 for time correction in preferences"

As noted by Bartosz, System|60 is also valid (apparently also
ZoneInfo|something, but not sure what can go after ZoneInfo), and the
maxlength interferes with this.

Maybe eventually we could revisit and use input#pattern
(http://www.w3.org/TR/html51/forms.html#the-pattern-attribute).  That
allows specifying a regex, which could have different length limits
depending whether it started with a number (AFAICT, the 6 length is valid
if it starts with a digit).

But for now, that is not urgent, and we need to fix the regression.

This reverts commit da470ce005652774049d346142b9509e89b31c6b.

Bug: 70405
Change-Id: Ib585c76688284dab78e85fb355785c06d0c3e353

Merge "Update OOjs UI to v0.1.0-pre (db065e5a9f)"

Merge "Get the value of microtime() directly as float in updateSpecialPages.php"

Get the value of microtime() directly as float in updateSpecialPages.php

It has a first parameter to directly get the value as float, so use it
instead of doing string manipulation.

Change-Id: Id2dff4486ea4f308ce03fc3d5546660c4e3c26b6

Merge "Mention translatewiki.net on edits only, when edit a default message"

Remove deprecated GlobalVarConfig::set()

Was deprecated in 1.24.

Change-Id: Iead2c8ba14c0524cba9585b5d4c29f2f806cfc15

Fix exception while bubbling titles

In prefix search if the search text is actually a title we make sure it is at
the top of the results.  But if it isn't a valid title we shouldn't blow up
because blowing up is bad.

Bug: 72260
Change-Id: I5bd795df447a33bb003a38ec2068dc184379a813

CoreParserFunctions::displaytitle(): Use Parser::killMarkers()

... instead of doing something similar to an old version of it.

Change-Id: I78db194291236208843a13b80d322d403774796c

Merge "Add name for bto language"

Merge "Add $tablesUsed for ApiTestCase"

Update OOjs UI to v0.1.0-pre (db065e5a9f)

New changes:
8e1a057 ListToolGroup: Implement collapsible/expandable tools
35b58b5 Introduce ButtonInputWidget
9ca3b79 FlaggedElement: Fix PHP notice
db065e5 Localisation updates from https://translatewiki.net.

Locally, add the two new messages to the Resources.php manifest.

Change-Id: I27a925d3f8799fe1f5635cb41de95f5d555d1f5a

Make PrefixSearchTest use MediaWikiLangTestCase

It uses namespaces which depends on the content language

Change-Id: I0a1ef4adea1e1ddff32e37cbbd8cf51a3398905b

Add $tablesUsed for ApiTestCase

This test creates 2 TestUser objects on setUp. Due to recent
changes in TestUser, this isn't even possible anymore if
MediaWikiTestCase hasn't setupTestDB() properly.

Change-Id: Idadfb94bd48270ad085cfece2bf17024ed9d97cd

Merge "Skip tests in PrefixSearchTest if not wikitext main namespace"

Fix incorrect replacement inside regular expression

Fix for Id3aa87cfa0 (2537ca2).

The "$" is for the end of the string, not for a variable.

Change-Id: I2d262582644b903992dc621079ea5a6a04d7af70

Use prefixed globals in eval.php main loop

Use globals with less-common names so that variables created by the
command line are less likely to conflict with the ones used by the
eval.php loop.

Change-Id: Id3aa87cfa039eb7555f44e17134c7cdbb5f25cc1

Merge "Make TestUser way faster"

Make TestUser way faster

By only updating the user row in the database, if needed
and by making use of md5 for passwords instead of slower
hashing.
This cut down run time of some Wikibase API tests to 20%
of the original value for me!
Also it reduces the run time of MediaWiki's test suite
by about 45s on jenkins.

Change-Id: I7024b287a71fe9b327dbcdc5427cd8edb5047606

Merge "Use a fixed regex for StripState"

Use a fixed regex for StripState

The JIT compiler in newer versions of PCRE experiences lock contention
when multithreaded applications perform a high rate of concurrent
compilations. We are seeing some performance impact on HHVM under normal
production traffic.

The random part of the strip marker is just there to protect against
deliberate insertion of strip markers into the source text, which is
very rare. So use a generic regex to find strip markers, and check in
the callback whether the random state ID is correct.

StripState::killMarkers() will be slower when it has to remove many
strip markers, but most calls to it will not match any strip markers, so
overall performance should be improved due to reduced JIT compilation.

Bug: 72205
Change-Id: I8d37ae929a8c669c9e39adc8096b89e5732b68d0

Localisation updates from https://translatewiki.net.

Change-Id: I289582368b87fb3f273d7fa1d65c95ca14b2942a

Do not include ilshowall as a hidden form field on Special:ListFiles

It is already a checkbox, including it in the hidden form inputs
means that it cannot be unchecked.

Bug: 72224
Change-Id: I430c219ac7a445a34f51a268c11c43e907bf9c10

Merge "Sanitize TestUser"

Set AdminEmail field in the installer to explicit ltr

Email addresses are usually left-to-right.

Change-Id: Iaf2320a17d62ad6c950cf83cc74b2cc020cbb9cb

Merge "API: Clean up skin handling"

Localisation updates from https://translatewiki.net.

Change-Id: I38583ba3417708c0f6992306785177cae82f552b

Sanitize TestUser

This also, as a side effect, makes sure we no longer
use hash_ functions here, which will reduce the impact of
https://github.com/facebook/hhvm/issues/3740 on us.

Change-Id: I6e0c776ef9aafe490b24b6869037bb5cb05cd151

Skip tests in PrefixSearchTest if not wikitext main namespace

Otherwise, running core tests with non-wikitext in the main
namespace fail with fatal error.

Change-Id: I895e7ee61ba80c0c345b6c05f05d05c368e52d59

commonPrint.css: Add @noflip to direction float classes to fix RTL

Same fix is already applied to oldshared.css

Change-Id: I170759de516da04a3ea3c5d57bea0b2f65faf45d

Add name for bto language

Translations added in I5e83ff26f39

Change-Id: I7ecdd557f56bc9f6027cfd16010f35eb722c2080

OOjs UI: Re-build pull-through from scratch to vet for unused files

Change-Id: Ic1625794af28f986c0f40d1289983f474602ec9e

Merge "Revert "Add RL template module with HTML markup language""

Revert "Add RL template module with HTML markup language"

Not ready for merging, and Roan says that the +2 was
most likely accidental and meant to be a -1.

This reverts commit d146934f94ba7028fb0ad0a180786bc5856990d6.

Change-Id: I3926c9ae9e3c8026fceb3aeedd3b1f1d9b91667b

Avoid needing config to be in sync between MW and the jobrunner

* This unifies the retry and no-retry code paths so that its easier to use
  the separate job runner service.
* All jobs will use the ack procedure. If no retries are enabled they will
  just get deleted. As a side-effect, abandoned jobs can be seen for a few
  days in the no-retry case, which used to not happen. This could actually
  be useful to know about anyway.

Change-Id: I185092e3696fb336b9edcf19280dcd9a561161d9

Implement page status indicators

Page status indicators are icons (or short text snippets) usually
displayed in the top-right corner of the page, outside of the main
content. Basically, <indicator name="foo">[[File:Foo.svg|20px]]</indicator>
may be used on a page to place the icon in the indicator area. They
are also known as top icons, page icons, heading icons or title icons.

I found the discussion on bug 23796 highly illuminating. I suggest
that everyone read it before suggesting different design choices.

I spent some time with a thesaurus pondering the name. "Emblems" and
"badges" were also considered, but the former has a much more limited
meaning and the latter is already taken by Wikidata, with a similar
but subtly different feature set. I am not aware of any naming
conflicts ;) besides new talk page message "indicator" (used by core
and Echo in some documents) and OOjs UI indicators (tiny icons like
the arrow on a dropdown form element), which shouldn't be confusing.

Potential use cases include:
* "Lock" indicators for page protection levels
* Featured/good article indicators
* Redirect shortcuts display ("WP:VPT")
* Links to help/manual for special pages
* Coordinates?… or globe icon for inline pop-up maps

Design features:
* Skin-customizable. Skins can fully control where and how indicators
  are shown, or may just do <?php echo $this->getIndicators(); ?> to
  output the default structure. By default they are not shown at all.
* Extension-customizable. Extensions can call ParserOutput::addIndicator()
  to insert an indicator from one of the numerous parser hooks.
* Wiki-customizable. In addition to just using the parser functions,
  on-wiki styles and scripts can use the provided classes and ids
  (.mw-indicator, #mw-indicator-<name>) to customize their display.

Design limitations:
* Every indicator must have a unique identifier (name). It's not
  possible to create arrays, or to have several indicators with the
  same name. In case of duplicates, the latest occurrence of the
  parser function wins.
* Indicators are displayed ordered by their names (and not occurrence
  order). This ensures consistency across pages and provides a simple
  means of ordering or grouping them.
* Indicators are not stored, tracked or accessible outside of
  ParserOutput (in particular they're not in the page_props table).
  They are intended to merely reflect the content or metadata that is
  already present on the page, and not be data themselves. If you ever
  think you need to list pages with a given status indicator, instead
  figure out what it means and use the appropriate tracking category,
  special page report, already existing page_prop, or other means.

Corresponding patch in Vector: I90a8ae15ac8275d084ea5f47b6b2684d5e6c7412.
I'll implement support in the other three skins included in the tarball
and document it on mediawiki.org after this is merged.

Bug: 23796
Change-Id: I2389ff9a5332a2b1d033eb75f0946e5241cfaaf4

Merge "Add RL template module with HTML markup language"

Merge "Make declaration of ApiFormatRaw compatible with ApiFormatBase"

Update OOjs UI to v0.1.0-pre (0d358b167a)

New changes:
630586d noflip left: 0; rule for overlays
03e5106 Rewrite OO.ui.Element.getRelativePosition() to be smarter
90d4eef Replace calls to OO.ui.bind( fn, ... ) with fn.bind( ... )
77f0cd3 [BREAKING CHANGE] Remove deprecated Minerva theme
3e415f0 Clean up unneeded use of bind()
21c5b18 PHP Implementation of OOjs UI
3eb2ee5 OOUI PHP: TitledElement
70b35b2 OOUI PHP: GroupElement, ButtonGroupWidget
2f0df0d OOUI PHP: Layouts: Field, Fieldset, Form, Panel, Grid
d856e81 OOUI PHP: Widgets: Icon, Indicator, Label, Input, TextInput, CheckboxInput
0d358b1 build: Fix PNG image generation

Change-Id: I2d02d29c267a1b0db0cd1e1c19b93f488d02a9c2

Merge "Update OOjs UI to v0.1.0-pre (99ec31d9b9)"

Make declaration of ApiFormatRaw compatible with ApiFormatBase

This was causing PHP warnings.

Change-Id: I96f66672756be3c09c20aea053f8a29241f56a19

Merge "mediawiki.hlist: Remove IE6 & 7 fallbacks"

API: Clean up skin handling

* Set a more sensible value for the "title" of the page (Special:ApiHelp
  rather than API)
* Avoid calling various skin-related hooks that aren't going to be used.
* Don't override the user in the DerivativeContext, MessageCache doesn't
  pay attention to it anyway for ->parse().

Bug: 72131
Bug: 72174
Change-Id: Idc42ab6526d640dc68fa93f6b4c5524b3d9e2e44

Localisation updates from https://translatewiki.net.

Change-Id: Ib16c880b53bda5528405b4efdc84e701222e356f

Also provide assertNotTag in MediaWikiTestCase

And mark that and assertTag as deprecated to avoid
new usages.

Bug: 69505
Change-Id: Ibfe78fd1914adfb08b29f61c59c43899bd8840d2

mediawiki.hlist: Remove IE6 & 7 fallbacks

Change-Id: I47e54f169f311b4f8211a6deef13704c97e43b06

Add RL template module with HTML markup language

Preparation work for templating in core.
RL should allow us to ship HTML / template markup from server to client.
Use in Special:Upload and mediawiki.feedback as a proof of concept.
Separation of concerns etc...

See Also:
Ia63d6b6868f23a773e4a41daa0036d4bf2cd6724

Change-Id: I6ff38c12897e3164969a1090449e626001926c3b

Merge "Add a messages file for Koyraboro Senni with French fallback"

Reduced LocalFileRestoreBatch::execute deadlocks when doing batch restores

* Often users restore a batch of many files with similar names.
  This fails due to gap locking on non-existing rows which tangles up
  unrelated actions. Avoid that locking for the common case.
* This changes the return value of LocalFile::lock() from a useless
  one (due to the exception) to one that conveys some useful information.
  The method is fairly internal in any case.

Change-Id: Idb86367cf45b731d4b2a67b1813a660bc0e7a84f

Merge "Add a PSR-3 based logging interface"

Merge "Localisation updates from https://translatewiki.net."

Localisation updates from https://translatewiki.net.

Change-Id: Id155f17da42d10128a582357d602fb4ff3d08c46

SqlBagOStuff: fix percentage in deleteObjectsExpiringBefore()

Partial deletion percentage progess was computed using
$remainingSeconds. This is incorrect since $remainingSeconds
is going to *decrease* ( instead of increase ) while the
process continues.
The correct implementation uses 'processed seconds' instead.

Change-Id: Ic745f6e8375a85da543de36703fad6d31f62ea90

Update OOjs UI to v0.1.0-pre (99ec31d9b9)

New changes:
6298415 ButtonWidget: Fix titled button
918ebed MediaWiki theme: Fix styling of SearchWidget
bd03644 Refine global event handling in non-isolated window managers
9a7633c WindowManager: Wait for window to open before trying to close it
5516490 Element: Correct method name reference in documentation
99ec31d Position popups and their anchors correctly in RTL

Change-Id: I4266c58ec1a2e6cee54ffc55891a5c2097f0222f

Add a messages file for Koyraboro Senni with French fallback

Change-Id: Idb53b501d11c295a7d1147724582456220720882

Add a tracking category for duplicate arguments

If a page accidentally duplicates an argument, such as
{{foo|bar=1|bar=2}} or {{foo|bar|1=baz}}, add it to a tracking category.

Bug: 69964
Change-Id: I3b6eeff8b51859bc7af0ea985f6f7528c2e9d220

Merge "DatabaseBase::tableName() fix limit while invoking explode()"

Merge "SqlBagOStuff: document public method deleteAll()"

Update OOjs UI to v0.1.0-pre (1829141230)

New changes:
393fd80 TextInputMenuWidget: Handle undefined value properly when toggling visibility

Change-Id: I4f6a7f62803d441de53271c645f07f76d23b3d06

Update OOjs UI to v0.1.0-pre (98cecf304d)

New changes:
98cecf3 Support MenuWidget's config.widget being in a different document

Change-Id: Ifc30790e67926bbdda0a8d5ec9464e7046f984ea

SpecialNewpages: Prepare for mw-ui

Use HTMLForm instead of self-built table structure, to be
prepared for use with MediaWiki UI.

Bug: 71446
Change-Id: I5c03dc543b910aab27a26a5a223341be50893cf3

DatabaseBase::tableName() fix limit while invoking explode()

Limit for explode() needs to be set to 3 to allow entering
branch at line 2350.

Change-Id: Idc2ba81a4bafa3e5511982db73a5c1549f1f9ef0

SqlBagOStuff: document public method deleteAll()

Change-Id: I23700714453b6243e564642f2bad4a2d1c2469c6

Fix copy/pasto in API help and a typo in message name

* The help for [[Special:ApiHelp/xml]] used the same copypasted
apihelp-json-param-callback message for both parameters instead of the
correct ones for those parameters.
* The message name in en.json and qqq.json was truncated, so that it did
not correspond to the parameter name. I presume this was unintentional
and not some strange limitation.

Change-Id: I666b2fd6511f08bccc9f05739bab68d7675ef5c6

Localisation updates from https://translatewiki.net.

Change-Id: I6834a3e482f32878082ebe86ac2a46e09e8f9cdb

Merge "Make prefs-help-realname more realistic"

Let BlockTest.php respect database constraints

This is an attempt to make BlockTest not to fail
with database referential constraints enabled
(from I326bb4a189bf881299b9fb678033a927b916efac)

Bug: 58191
Change-Id: I45c2d466f77d4d0aa25e880b5bb7d1120c39a561

Make prefs-help-realname more realistic

The message is shown whenever $wgMaxCredits > 0, but several actions
and extensions don't give attribution based on it anyway.
Also simplify the sentence structure a bit.

Bug: 72078
Change-Id: I4ae3abd9433c485e7066da922d2f47529b00d7d4

Remove broken parameter 'table' in wgObjectCaches for CACHE_DB

$wgObjectCaches in DefaultSettings.php had the useless
parameter 'table' for CACHE_DB.
SqlBagOStuff actually uses 'tableName' parameter, it
already defaults to 'objectcache' so there's no need
to write it again.

Change-Id: I5fe6b7c5f4d46cc8642c42861bc7078b795f5fb1

RequestContextTest: Distinguish initial test from restored test

Follows-up f31a046. In failure it's hard to see whether it's the
first or third group of assertions as they were carying the same
name.

Change-Id: I5891754342400155759e797337f8509d4eca39ec

Upgrade jStorage to v0.4.12

Project site

* http://www.jstorage.info/ and https://github.com/andris9/jStorage/tree/v0.4.12

jquery.jStorage.js file

* https://github.com/andris9/jStorage/blob/v0.4.12/jstorage.js

Author and contact

* Andris Reinman and email andris.reinman@gmail.com

README

* https://github.com/andris9/jStorage/blob/v0.4.12/README.md

Upgrading from 0.4.10 to 0.4.12.

Change-Id: I62899c8a2a7b9b2ae5b0f4b6ec06ee24cb966806

Update OOjs UI to v0.1.0-pre (d74a46ca6a)

New changes:
d74a46c Introduce oo-ui-window-inner-overlay

Change-Id: I3f325f3c514d334179de46d0ea092b48f13b9dfd

jquery.badge: Swap non-breaking space character for regular space

Follows-up d453f81f4e. PHP's native json_encode requires input
being a UTF-8 string. For some reason MediaWiki, Git or OS X is
not reading jquery.badge.css as UTF-8 for me.

https://bugs.php.net/bug.php?id=51947

Due to json_encode rejecting the entire value and returning null
instead, the jquery.badge stylesheet was absent. load.php:

 mw.loader.implement("jquery.badge",function, {"css":[null]} );

Change-Id: I84d2017a976249f55bdb269d2b9762db941292fe

Merge "Make enhanced recent changes and extended watchlist default"

Localisation updates from https://translatewiki.net.

Change-Id: I6ca4e358ecc0d008ae5f323b445d272f1f721a0b

Merge "Add LinkBatch to Special:TrackingCategories"

Break long lines

Change-Id: I8d4e883058c21023273df88439cd145888833115

Merge "Create tooltip for "Page information" link"

Merge "Deprecate ORMTable::getFieldPrefix"

Create tooltip for "Page information" link

Added an empty accesskey message for consistency

Change-Id: Ie57d63718990eb6974808550622c3899a2b18d8f

Deprecate ORMTable::getFieldPrefix

Internal calls to this method show up in my XDebug profiling as
the second most called method (after array_key_exists). Called
10000 times. Sure, it's very cheap and clearly not a bottleneck.
But just not necesarry to have a method for that. The benefit of
having a method is to have a dynamic prefix that can change
depending on other things in an object. But I think this is not
a good idea for a prefix.

Since this is protected it is part of the contract of this class.
I already cleaned known subclasses. Should be @deprecated for
some time and can be removed later.

Change-Id: I46a0d7e072d0a69e9aef5f77e92044b62e3d0ce7