Merge "Transaction coverage fix for doDeleteArticleReal"

Merge "Also set the queue types map when rebuilding the ready queue map"

Merge "CSSMin: Add tests for handling existing data: URIs"

CSSMin: Add tests for handling existing data: URIs

Just a sanity check inspired by bug 66112.

Change-Id: Iac60a1eb15b3ff60822a3f24f2511c4bc64c092a

Also set the queue types map when rebuilding the ready queue map

Change-Id: I0686859541661a22131c387d8fac6dc7b074c18c

Merge "Cleanup message file documentation"

Merge "Soften test for incomplete edit POST"

Merge "Don't die on empty keys in MWMemcached"

Cleanup message file documentation

* Interface strings are now elsewhere
* MessagesQQQ no longer exists
* Prefer https for translatewiki.net

Change-Id: I76652ea94cca80441cd5d978029e4707ee41c4fd

Merge "Add ParserOutput::unsetProperty, add tests"

Don't die on empty keys in MWMemcached

Bug: 64287
Change-Id: Iedfec96286212577a181b83957cdc4545be206d2

Soften test for incomplete edit POST

Do not reject POSTs from forms that were rendered before
wpUltimateParam was added.

There is also an open question about whether we should respect
kludgey scripts which have hardcoded form params.

See Umherirrender's comment on
change I78f1bbcc9bb092568fbfbec19c08d697c9ae75f8

Change-Id: I28bd6d45442bc3ba352f0de7a50709adb98b1c50

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

New changes:
47b147e [BREAKING CHANGE] Use promises for confirmation dialog instead of done event
bcbf56b Localisation updates from https://translatewiki.net.
997292b Localisation updates from https://translatewiki.net.

Change-Id: Iafee3075f79f064d2eb9f5528e7d0bf7c1c52c11

Merge "Made executeReadyPeriodicTasks avoid using merge() when not needed"

Made executeReadyPeriodicTasks avoid using merge() when not needed

Change-Id: I9bb3a5af6ebd12ac5a31578e17142fb0a2dba029

Track the types of queues in the redis queue aggregator

* This tracks all queue types in one simple location that
  can be inspected without using MediaWiki.

Change-Id: I197664b9aa620b29f1400e5a60c55a301348120d

SpecialPage: Remove wfSpecial* call syntax

Should've done this years ago. Anything still using this syntax is long
since broken in half a dozen other ways anyway.

Change-Id: I0e78453544abb5de7935b046c838ae5b3c4da802

Merge "Installer: Simplify css request"

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

Localisation updates from https://translatewiki.net.

Change-Id: I7c4ffc96032192391b4999840bc5b0044f360ee0

Merge "editintro given a css class of its own"

Merge "Installer: Disable broken CACHE_DB after backend is disabled"

Installer: Simplify css request

Instead of passing around the direction in the request, just set
'css=1'. We already have the language code and directionality
available and even rely on it between requests to set other
localisation-related things.

Change-Id: I717cb299b3639024e79880039e0d1f2ff273ab44

Installer: Disable broken CACHE_DB after backend is disabled

We're currently not hitting any code that calls wfGetCache(), but
this seems like a good thing to do and would allow us to re-use
more code instead of duplicating a large method just because it
has a call to wfGetCache() somewhere.

This is already done for DB access and LocalisationCache in
general by the installer, but via SqlBagOStuff it can still
cause an exception from LBFactoryFake.

Change-Id: I69c1e34365c1d7503aea8ef47942d20c34318e85

Merge "Vector: Label the more actions menu "More", not "Actions""

Merge "ParserTests: fix `bits` column in uploaded images for test cases."

Add ParserOutput::unsetProperty, add tests

Change-Id: I29af31918c48a1225b6487c74fd638de26f07b28

Merge "Add global limit to PoolCounter"

Merge "Documentation for page_props getter in ParserOutput"

Merge "Add jMY to Arab date formats ($datePreferences)"

Add wl_user_notificationtimestamp index

Bug: 65246
Change-Id: If3e9e855c8a21874ca69c4976af207833dcc53f2

RCFeed: Use named types instead of exposing internal MW constant

Like we do in ApiQueryRecentChanges as well, expose the named
rc.type property, not the internal integer constants.

Change-Id: I89a948eee999032bb2e42cc23345affa879afb42

Vector: Label the more actions menu "More", not "Actions"

Vector has a consistent way of showing most actions – as tabs, or, if
there are too many or the action is unimportant, shunting them into an
additional actions menu. This menu was problematically labelled "Actions"
which though correct is misleading as it is merely /some/ actions,
alongside the generally-promoted "Read", "Edit" and "View history". The
recent change I09c8031c makes the label of this menu much more prominent
(as it is no longer hidden), which makes the labelling issue more acute.

This menu could have many labels, and this is merely a suggestion. Some
alternatives:

 – "Tools" — However, this conflicts with the page tools shown in the
   left-hand nav, which although arguably should be moved into this menu
   is an argument for another day, as it would notably alter the skin for
   readers who otherwise don't get this menu at all.

 – "Tasks" — This feels more like a term that would be used for an
   individual, or perhaps for the article itself, to denote a worklist –
   a list of outstanding issues to be ticked off ("Orphaned article";
   "Needs wikifying"; "Copyedit"; etc.) rather than a set of task actions
   you can do as atomic steps to the page.

 – "More actions" — This is rather wordy and ideally should be
   unnecessary in the minds of editors who should (through the consistent
   use of simple verbs as action labels) already recognise each of the
   tab links along the top-right of the navigation to be page-
   specific actions.

Bug: 44591
Change-Id: I8697beab5695198c0acddcf9ad8eeca189660c94

Grunt: Clean up outdated information

* Use jshintrc:true which lets grunt defer autodiscovery of the file
  to jshint instead of forcing it to '.jshintrc'. This also us to
  override certain settings from a subdirectory, to match behaviour
  of standalone jshint.

* Remove comments about "Known issues" in node-jscs, those have
  been fixed a while back and were applied when we updated jscs
  in df8c455.

* Match load order.

* Use plain string instead of single-value array.

Change-Id: I7fd8db63ba164de705239c4b419de446ca7d6809

Add SpecialPage::getConfig

This is necessary since SpecialPage doesn't extend ContextSource yet.

Change-Id: I94c28dc88e9eb569eec5c5ab6e2ce97acc6a25e3

ParserTests: fix `bits` column in uploaded images for test cases.

When running parser tests on a sqlite3 database, the insertion of the
djvu image before running the test suite will fail because `NULL` is not
a valid value for the `bits` column of the `image` table.  This will
cause the test suite to eventually fail, since {{NUMBEROFFILES}} differs.

Test uploads show that `bits` is usually set to 0 for both SVG and
DJVU uploads, so fix this (in both the standalone test runner and the
phpunit test runner).

Change-Id: I8689a547d34035534723e87c4c2680c4e67245f2

Merge "Deprecate Language::getLangObj()"

Merge "(bug 56849) Deprecate dangerous edittime-based content update functions"

Merge "Make "mini" the default alias for "thumb" in German"

Deprecate Language::getLangObj()

Since it is unused in core and extensions, I'm marking it both
with @deprecated and wfDeprecated().

Change-Id: I3a1bf96b805339e1438fcdd6e0aeb5922e53c4e8

Merge "Copy various random files used by tests to tests/phpunit/data"

Merge "Renamed $ipSet static variable in IP to something better"

Copy various random files used by tests to tests/phpunit/data

The tests currently depend on them never being renamed, which is bad.
(Actual file data in git is de-duplicated automatically AFAIK.)

Change-Id: Id2440326981218f9e7d51541a168db59183fdadf

Merge "Disable jQuery Migrate by default (enable via $wgIncludejQueryMigrate)"

Disable jQuery Migrate by default (enable via $wgIncludejQueryMigrate)

Follows-up e5502d1f and 63c36506.

Per the plan[1], after a few weeks of migration, the jQuery Migrate plugin
will be disabled in MediaWiki core.

The added configuration variable allows sysadmins to continue to use code
that is not yet compatible with jQuery 1.9 and higher (e.g. extensions
and/or gadgets that still use legacy jQuery functionality).

[1] http://www.mail-archive.com/wikitech-l@lists.wikimedia.org/msg75735.html

Bug: 44740
Change-Id: If1c9ab722c7ce025b550b457506dcdf90d3326c3

Restructure output of ApiExpandTemplates

Create a new output format of ApiExpandTemplates and deprecate the old
one.

Change-Id: Id12cbe4aeaa39bea382cf3b78810589bb1c9368f

Localisation updates from https://translatewiki.net.

Change-Id: I70279ffd1137ec4a7c5627889a5ae24b8f62c904

Add global limit to PoolCounter

Adds a 'slot' option to $wgPoolCounterConf. When this option is set,
there are a limited number of slots for the given worker type,
regardless of key. Workers with the same key are still limited by
the 'workers' option.

The global limit is implemented by simply using a deterministic
hash of the key instead of the real key. This avoids deadlocks,
but results in slot underallocation due to hash collisions - even
when there are significantly more jobs that slots, some of the slots
might remain empty.

Bug: 65691
Change-Id: Ibdf8ec222f9756d70de2bed7ff14913351dc394b

Merge "Add some SVG element namespaces"

Merge "Made TransactionProfiler handle interlaced transactions better"

Merge "Remove use of strencode() outside Database classes"

Renamed $ipSet static variable in IP to something better

Change-Id: Ie7e049c9b114d2a76e1e8e985c3500bda4bbc0ef

Fix documentation for Title::getCascadeProtectionSources and others

The documentation for Title::getCascadeProtectionSources was rendered
nonsensical in r79682, and somehow several following "documentation
cleanup" revisions failed to notice this while cleaning up this same
documentation string.

Fixing that led to fixing Title::getRestrictions and
Title::getAllRestrictions as well.

Change-Id: I48638e370c01fe5ef8ad7d3ebf7fb9aa4385bad7

Merge "Remove redundant filter for negative namespace ids in search preferences"

Remove redundant filter for negative namespace ids in search preferences

Follows-up 5dc4dc099. MWNamespace::getValidNamespaces() filters these
out already.

Change-Id: Icad951fb9a4c4cb368cd994d1861dc2eff1f4ae0

Don't double-parse edit summaries

Fix a bug where edit summaries on permalinks would be parsed twice,
resulting in breakage of links, etc. in them. Follow-up to Ie19adfa5.

Change-Id: I3d7e70bb6d055184d36df35ef2e747ba32ad4ea0

Update OOjs UI to v0.1.0-pre (6379e76bf5)

New changes:
f55ab55 Localisation updates from https://translatewiki.net.
6379e76 Followup 205fa98a, 8dd3460a: move away from isResolved()/isRejected()

Change-Id: I54fcf6fed97d95358c606a82a6e444f03f9247ad

Update OOjs UI to v0.1.0-pre (527ad0ee5a)

New changes:
50355d0 Localisation updates from https://translatewiki.net.
8dd3460 Resolve the opened promise after teardown
5153f85 Followup 205fa98a: fix incorrect use of .super

Change-Id: I81986ef0b5b768753ffb3b25ebe41bf648a796d1

Merge "Make convertUserOptions.php more self-contained"

Documentation for page_props getter in ParserOutput

I would love to make this getter return null instead of false,
similar to getExtensionData() in the same class. A property that's
not set should be null. Not a boolean. This makes it impossible to
have boolean properties. But I can't change this because it's used
in extensions with "!== false" and such.

Change-Id: I2bc0bed1c58435d791f7a055bac2763a3411bb6f

Make "mini" the default alias for "thumb" in German

It's already "mini" in other languages.

The German Wikipedia community (which is obviously not the only German
one but the biggest) always complained a bit about the previously
used "miniatur". It's German, yes, but error-prone. The solution
was to add "mini". Which made most users happy as far as I can
tell after all these years.

My patch doesn't change anything except for one little detail:
The JavaScript call
mw.config.get( 'wgWikiEditorMagicWords' ).img_thumbnail
which is created in WikiEditorHooks::getMagicWords and used by
the WikiEditor extension will return "mini" instead of "miniatur".

Change-Id: I4e9ad3f2e56e08eaf3006a1af4b1c9746cd55715

Localisation updates from https://translatewiki.net.

Change-Id: If59db33f98025822dcb2f886599fbc8dae7f2513

Add 'X-Robots-Tag: noindex' header in action=render pages

Bug: 63891
Change-Id: I297a6269fcab999610bc01755462938c70eb64a1

Merge "Correct field used to detect incomplete POST"

Add some SVG element namespaces

Noticed that we have several old images on commons that use these
namespaces. They seem harmless, so I think we should add them.

Change-Id: Ib80298256c5d4717867eb857d5404a9487fed784

Remove use of strencode() outside Database classes

Change-Id: Idbc38d8089541e5154b2601297b429f7433fd59e

Merge "Sync up with Parsoid parserTests."

Merge "Include edit summary in permalink"

Sync up with Parsoid parserTests.

This now aligns with Parsoid commit 04a4bf2ba42c2ed568ef9e9c4679922f17b464d0

Change-Id: I330c1891efd47a4342474774c39dfc963b0ec13d

Correct doc of WikiImporter::__construct parameter

Change-Id: I0c61bb4f8d1e51f3b58ff99a9c632561dfd5134d

Merge "Change message escaping on ImagePage"

Revert "Use square bounding boxes for default-sized thumbnails"

Reverting for further discussion, due to community complaints at https://en.wikipedia.org/w/index.php?title=Wikipedia:Village_pump_%28technical%29&oldid=611154541#Infobox_Image

This reverts commit 7db55341791cb44ca599b84bea707b05de2f227d.

Change-Id: Ieea6224b2a295b5d65bbd486e30cf0573376c7e9

Localisation updates from https://translatewiki.net.

Change-Id: If43d75371246e2dc84d864cef6b2263a54196457

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

Vector: Fix conflicts of "Actions" dropdown with collapsibleTabs

Follow-up to I09c8031c.

* Don't override the width of the "Actions" dropdown when collapsing
  tabs. The width was hardcoded to 24px, using the actual width now.
  This only matters when the "Actions" dropdown disappears completely
  (it has no items inside), and then something gets moved inside it
  causing it to reappear.
* Use the same collapsing speed for the dropdown and the tabs.
* Account for the width of the "Actions" dropdown if the expansion
  would hide it. This issue was there before, but was not that
  noticeable because the width of the dropdown was negligible.
  (This required making the calculateTabDistance() function in
  collapsibleTabs part of the public API.)
* Do not collapse if the width of the dropdown would exceed the width
  of the tab we're collapsing. This issue was there before as well,
  but in practice the dropdown was always smaller; this is not
  necessarily the case now, for example in Lithuanian (lt).

Bug: 44591
Change-Id: Id4548ea388fd9f4341dd78bb132c716feb14b137

Make convertUserOptions.php more self-contained

* Moved the code from User::decodeOptions() to the maintenance script,
  the only place it is (indirectly) used.
* Changed the script to insert new rows itself rather than calling
  User::saveSettings(), avoiding bug 63677.
* Removed FOR UPDATE and COMMIT lacking a matching BEGIN.
* Made ORDER BY explicit.
* Removed pointless "Do each user sequentially [...]" comment dating
  back to r48732, in which the script would increment a user's ID
  to get the next one.

Bug: 63677
Change-Id: I86365a7363af3376cc4f55fa528b050b44378656

doc: Fix DefaultSettings's assertions about possible extension types

Change-Id: I79f0b6dab93272682157e2a0dcd727f14f7285e9

Remove 'SpecialVersionExtensionTypes' hook, deprecated in 1.17.

Change-Id: I98d15e2bbbec2ba754c4754c97a46f32e714398a

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

Release notes for I09c8031c

Change-Id: I6a3845544ae1b50989f82b5bbe7b55db59130f67

Localisation updates from https://translatewiki.net.

Change-Id: Ie45f563b849d231ff6173408dce004374ef18ccb

Vector: Show a label next to the actions dropdown menu

The little triangle arrow is not clear enough, as evidenced
by the number of people who don't discover the page moving
functionality hidden in that dropdown.

Before: http://i.imgur.com/g2N3tek.png
After: http://i.imgur.com/nUsqgqe.png

Co-Authored-By: Bartosz Dziewoński <matma.rex@gmail.com>
Co-Authored-By: Isarra <s@zaori.org>
Bug: 44591
Change-Id: I09c8031ccd076228a3c9d720e92e04d40a8d97e8

Remove mediawiki.toc dependency on mediawiki.util

This reverts commit 1b658bf0729692027f96c022d131c044610dd658,
which was a temporary workaround for Wikimedia deployment
cache issues.

Change-Id: I72ee359e8ce9aef93e50448067edce104014f39f

Merge "Avoid using raw sql in SpecialUndelete.php"

Drop support for old skins that don't use head element

These skins have been obsolete since 1.16 and aren't supported well at this point.
This deprecates those skins and begins deprecation of the creation of <head> contents
with only chunks of OutputPage stuff. The entire head should be created by OutputPage.

This also deprecates some old methods responsible for returning raw chunks of html for the head:
* getScript
* getHeadItems

Output of HeadItems is also tweaked. Previously there was no newline added
after each item, now there is. Most of the callers of addHeadItem don't use
their own newline and as a result end up on one line.

Change-Id: I13e25cc8d8fc3aa682f23b019a2fda0e809a5f64

Change message escaping on ImagePage

by Nikerabbit on I2a91f0ef1cf82360da2d17c7dc42633ae3e79100

Change-Id: I96c42974a2bc435abd65e9b1e8daba6cc9669409

Avoid using raw sql in SpecialUndelete.php

The where part for the ar_timestamp was raw sql.
Refactored to avoid this. Now the IN-List is created by the database
abstraction layer. The addQuotes and implode with comma is done there.
Just the call to Database::timestamp is still needed.

This also avoids a '1 = 1' in sql.

Change-Id: I1deb055a1437ae84fc8a4e8cf2976e18c15bda39

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

New changes:
205fa98 [BREAKING CHANGE] Window refactor, introduction of Processes

Change-Id: Ia26243f74ecad4931cf455284a037216a8393348

Removed clumsy unused IP::toUnsigned() method

* Also avoid calling ip2long() on invalid addresses in toHex()
* Also use native base_convert() in toHex() for IPv4

Change-Id: I4fe4e1ac3c91edb9812e4c5bd173e4b54e315934

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

New changes:
e86dae9 [BREAKING CHANGE] Emit a single event from confirmation dialog

Change-Id: I305224cad8e7bcb7e48dfc31faafeab8ed8a376c

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

Merge "Add category output to ApiExpandTemplates"

Update OOjs UI to v0.1.0-pre (469d40c88f)

New changes:
9af9116 Ignore SelectWidget mouseup events that started outside the widget
28a7ca8 Localisation updates from https://translatewiki.net.
469d40c Localisation updates from https://translatewiki.net.

Change-Id: Ife58c324e39e2ee1b6297e5803fb8dd69445ffab

Merge "Don't use isset() to check for null"

Merge "Allow adding Deleted log entries"

Save advanced search namespace prefs on Special:Search itself

* Checkbox on own row below power search checkboxes per MatmaRex;
  avoiding a mw-search-ns* id leaves it untouched by All/None JS.
* The option searcheverything is removed: a "shortcut" which is no
  longer necessary now that options can be (un)selected at once
  with All/None buttons on search page itself.
* Require a token for saving: no accidental preferences changes.
* Keep the searchoptions/advancedsearchoptions prefs section in case
  something is using it (no known extension does though); options
  are converted to "api" type so it's empty and hidden by default.
* Add minimal documentation for saveSettings() and friends
  (@todo since 155ddf6de, 2009!).

Bug: 52817
Change-Id: I514cee835988600cc013658049e88a10b670e64a

Merge "Fixed fatal in SwiftFileBackend when a bogus HTTP status is given"

Transaction coverage fix for doDeleteArticleReal

* Do the FOR UPDATE select to get the ID *after* BEGIN, otherwise the
  locks will disappear right afterwards with the implicit commit.

Change-Id: Ia531126cb4a86ec7789ceb7327d1c235c7aa979c

Fixed fatal in SwiftFileBackend when a bogus HTTP status is given

* Error was "Fatal error: Call to a member function getSize() on a non-object"

Change-Id: Id1a1b1add9daee4402b2ed017b95c9a9f0e08170

Localisation updates from https://translatewiki.net.

Change-Id: I2ac98863fc1326e9eae37272a19a7174d80a8822