Merge "InstantCommons: do not cache thumbs"

InstantCommons: do not cache thumbs

There is no point in local thumb caching when we set apibase
and thumbUrl to the remote wiki. It will only confuse MediaWiki
into creating local thumbnails for certain LinksUpdate tasks
even though the rendered page will reference thumbnails hosted
on Commons.

Bug: T145496
Change-Id: I0bd4b539f575036dd865bdd946660f10a9ee8801

Merge "Update html/php clauses for subpage parserTests."

Merge "Sync up with Parsoid parserTests."

Sync up with Parsoid parserTests.

This now aligns with Parsoid commit 643d5392bcf4dfebf906102627c51e8a608125bf

Change-Id: I4d7dc7378ca7cfdb3919f33959f58eb5c4d88ca8

Prevent unexpected }- in converter output

Previously for input -{<span title="-{X}-">X</span>}-, the converter
sees -{<span title="-&#123;X}-">A</span>}-, so <span title="-&#123;X
becomes the content in the first block, and a stray }- is left to output.

Now, the converter sees -{<span title="-&#123;X&#125;-">A</span>}- with
this change. In further processing, the span tag may be parsed and have
its title attrib converted. For cases where the content is not processed
further (eg. "R" = raw flag), "-{X}-" is left as is in the attrib, which
is not so ideal, but at least it's better than the original extra }-
outside the whole tag.

Change-Id: Idbaaf53f914f362e5b8cc9fad02a524f8d591bb7

Merge "Don't remove border for floatright and floatleft"

Update html/php clauses for subpage parserTests.

The NS_USER namespace needs to have subpages enabled before these tests
work on the PHP parser.

Change-Id: I8e5e3bbd0dea6fc12f3b9ff9feeb58812fc51af1

Localisation updates from https://translatewiki.net.

Change-Id: I9a3d688828f6d3085eca91ba1653835b7854da21

Merge "Remove unused protected method SpecialUserrights::splitGroups"

Merge "Fix functionality and usages of SpecialUserrights::userCanChangeRights"

Merge "RCFilters UI: Add 'remove' and 'restore defaults' to filter list"

Merge "RCFilters UI: Read default states of filters"

RCFilters UI: Add 'remove' and 'restore defaults' to filter list

Bug: T144448
Bug: T149391
Change-Id: I418ad6b34ae8a7456a7e66bc703d4dccf36764a5

RCFilters UI: Read default states of filters

Add the functionality to read the default state of filters
and preserve them, so that they are considered when either
base state is requested or when filters are explicitly told
to revert to default.

Bug: T149391
Bug: T144448
Change-Id: I9e8e3430ca2f80d3f67422681e8fb69a43ee4bef

Merge "rcfilters: Remove implemented filters from list of links"

Merge "Follow-up e3da2e23: Fix typo in hideanons filter name"

Merge "Add tests for OutputPage::addMeta and set{Index|Follow}Policy"

Merge "Installer: don't treat the existance of main page as an error"

Add tests for OutputPage::addMeta and set{Index|Follow}Policy

This is a very basic test setup for the core functionality only. All
the special cases in getHeadLinksArray are currently not tested. But
this is a start.

This is relevant for the changes made in Ie8fd697.

Bug: T51859
Change-Id: I61e2da68ae0daea15fab2065a5dd63605d0b837d

Don't remove border for floatright and floatleft

This is a leftover from T2691, from a time when the skin still used
borders to fake a margin that didn't overlap with H2 elements. We fixed
this a long time ago in T28449 and it's follow ups, but this border: 0;
remained.

I discovered this now no longer desired side effect when trying to
apply these classes to tables.

Change-Id: I52b18c7c5b3dfea39c4653a8add00a607e7eab73

Create active/inactive behavior for complementary filters

Filters that are complementary or that contain one another should
indicate that they are inactive (or ineffective/disabled/excluded)
from the search.

Bug: T149452
Bug: T149391
Change-Id: Ie58493ef940698dddb04362473664c404f392b2b

Update Balancer to latest HTML5 spec

This corresponds to the 1.0.27 release of domino, and matches the
latest HTML5 spec as of 2016-10-18.

Changes include:
* <menuitem> is no longer an empty element.
* <isindex> has been removed.
* Updated html5lib-tests (copied from domino 1.0.27).
* Round-trip-safe serialization of <pre>/<listing>/<textarea> is only
  used when "tidy compatibility" mode is enabled; the behavior in
  the HTML5 spec no longer cleanly round trips.

Change-Id: I656944b0d7bb6c3c0e4fe44fc6ebd1a4c36412ad

rcfilters: Remove implemented filters from list of links

This is a bit hacky because the filter name needs to be inferred
from the class on each span, and because the separators aren't
wrapped.

Change-Id: Ib39ad435d3b48fa38533926e4ab49942c3bd5d6f

Follow-up e3da2e23: Fix typo in hideanons filter name

Change-Id: I5d3d72147f1c0ce7994e253801128dbe145f72e8

Merge "Pass the right username to 'tool-link-userrights' / 'tool-link-userrights-readonly'"

Installer: don't treat the existance of main page as an error

It results in CLI installer returning status code 1, messing various
automation such as our own Vagrant.

Change-Id: I7c14596db4ddd91339b2d2af463d851123a00ae7

Merge "mediawiki.api.options: Use sequential API requests in the remaining edge cases"

mediawiki.api.options: Use sequential API requests in the remaining edge cases

Bug: T100908
Change-Id: I927c098fd750bd765ffd746d40c3f7408f99f2de

Localisation updates from https://translatewiki.net.

Change-Id: Ied68f6be3ea1dfec0f05d6e9ca36004ade979fb3

Merge "Reciprocal fallback between ca (Catalan) and oc (Occitan)"

Reciprocal fallback between ca (Catalan) and oc (Occitan)

The two languages are extremely close, to the point linguists have
considered one a variant of the other till not too long ago. The
two linguistic communities have strong ties according to Donneky,
2011: http://hdl.handle.net/10443/1302 . Speakers should have no
difficulty understanding each other.

A translator has compared a sample of translations in the two
languages and found them to be largely similar and compatible.

Use European date formats; no $namespaceGenderAliases for now in oc.

Discussed at:
https://translatewiki.net/wiki/Thread:Portal_talk:Ca/Joining_forces_with_Occitan

The proposal has been notified on all Catalan Wikimedia projects, see
* https://ca.wikipedia.org/wiki/Tema:Tjc8xr0z769mskl5
* https://ca.wiktionary.org/w/index.php?diff=903923
etc. has met a very positive response.

This change is not incompatible with the oc→fr fallback (I02fc9af984).

Change-Id: Ibbad7a830bc010654db8ad5679d6a0f45357d382

Remove useless title parameter in CategoryPager

The title is already added as the action of the form tag,
so adding it again as a hidden input field is pretty useless
and more confusing in the url. Therefore: Removing it.

Change-Id: I146b562357105e337f86a6ccfeb55477262f67cd

Language fallback from sh (Serbo-Croatian) to bs, sr-el, hr

Bosnian, Serbian and Croatian are closely related languages and have
respectively about 2700, 1400 and 1200 translations identical to that
of Serbo-Croatian, which has about 5400 in total. This demonstrates
that we can profitably reuse some translators' work to reduce the
usage of English in Serbo-Croatian despite the low level of translation
and that there is a logical order in the fallback sequence.

The information has been collected and discussed with translators at
https://translatewiki.net/wiki/Thread:Portal_talk:Sh/Language_fallback
and Wikipedia editors have been notified of the discussion with
https://sh.wikipedia.org/w/index.php?diff=40677113&oldid=40676567
and messages in the talk page of each individual editor.

Inherit linktrail and digit separators, but not $defaultDateFormat,
$datePreferences, $datePreferenceMigrationMap or
$namespaceGenderAliases to reduce the impact and variance.

Change-Id: I04310ee32370ae20c00ea970b3163dcb8f167eba

Fix mode of CategoryPager.php to 644

Follow up: I4811771210384a32407b7378c7428361e46da33e

Change-Id: I0c87e04ca82f9b3f18f276cb41adbe6890cb870a

Merge "PHAN: conditionally load stubs"

Merge "Soft deprecated EditPage::submit to avoid new use"

Soft deprecated EditPage::submit to avoid new use

Currently this has no usages.
::edit can be called directly

Change-Id: I892e3de61c0513dc186d0f35e05107ed7136447d

Remove unused protected method SpecialUserrights::splitGroups

Change-Id: I7e06086aab6c939ec4c92887fd8d75ae7c09b860

Fix functionality and usages of SpecialUserrights::userCanChangeRights

SpecialUserrights::userCanChangeRights is a funky method that is
called by things outside of SpecialUserrights to see if they should
display a link to it. It had some bugs and missing documentation.

* SpecialUserrights::userCanChangeRights relied on $this->isself to
  see if the acting user and target user are the same, but it gets set
  in execute() which never runs when used like this.
* It wasn't clear whether the $user parameter represented the acting
  user or the target user, resulting in incorrect usage in
  SkinTemplate.

The net effect of these bugs is that skin sidebar would display
"Change user groups" instead of "View user groups" for all users when
the current user was only allowed to change some of their own groups.

Change-Id: Ie47b9c7463b373fe17006567239aa09e824b015d

Pass the right username to 'tool-link-userrights' / 'tool-link-userrights-readonly'

Change-Id: Iba2e41942352b834acb4ecd0781313ae522c25c4

PHAN: conditionally load stubs

Change-Id: I6f78c573468a393d82927d614df5bceda86a9e9c

Merge "Standardize html form by using OOUI in Special:Categories"

Localisation updates from https://translatewiki.net.

Change-Id: If2cf309a446a47958a10a6fc6d16498e1b52eb80

Merge "Revert "ApiSandbox: Display params as JSON on request page""

Standardize html form by using OOUI in Special:Categories

Bug: T117728
Change-Id: I4811771210384a32407b7378c7428361e46da33e

Merge "mw.notification: Re-implement autoHideSeconds as string preset option"

mw.notification: Re-implement autoHideSeconds as string preset option

Instead of allowing freeform number values, make it a preset-based
string where pre-defined values are encouraged.

A numerical override is still possible by creating a new preset,
but this undocumented/private for now.

Follows-up 4b9992a0, which added this option. No usage found
as of yet in Wikimedia Git, thus no deprecation/back-compat.

Also:
* Unbreaks `#autoHideSeconds` reference in #autoHideLimit doc.
* Improve wording of #autoHideLimit doc.

Bug: T155228
Change-Id: Id84d8b616fe26b9976263f4bd6bdc70cc8a539ba

Parser: Trim leading whitespace from links before checking for leading :

The leading spaces on the link only cause us problems, such as for the
$noforce check 20 lines later.

Bug: T129218
Change-Id: I93a8da1f73b38fa3da362f8f27479b3039ed3f13

Localisation updates from https://translatewiki.net.

Change-Id: I846c7cf640fe159540ba25c6abfdce30b9dc5351

Merge "Revert "Added reason suggestion in block/delete/protect forms""

Merge "SpecialMIMEsearch: Add a dropdown with input suggestions"

Merge "Replaced all deprecated Linker methods with proper ones in core(1)"

Revert "Added reason suggestion in block/delete/protect forms"

See the task, this was probably entirely my fault not having
looked at this more carefully. Technically the change is ok,
however, it seems to doesn't make much sense in combination
with the Reason dropdown box.

This reverts commit faab2411c29a2ad0851c596f1cadfec1d7e06892.

Task: T34950
Change-Id: I1eeb9c68ff0db20d29e7d5f0fb18f0bfa3224416

Merge "BlockLogFormatter: Durations are relative to block's timestamp, not Unix epoch"

Fix typo in README

Change-Id: If32667e5e41d924f14ad803c470a21cd54f3f17b

Merge "ApiErrorFormatter_BackCompat: Use first error, not last"

Merge "Remove unused fields in SpecialUnusedimages query"

Merge "Special:Upload should not crash on failing previews"

Special:Upload should not crash on failing previews

A thumbnail transformation can fail, but the preview on Special:Upload
was not accounting for failed previews and caused a stacktrace on
accessing getUrl on 'false'.

Bug: T155771
Change-Id: Iff3a2ae3512a34a2d2efb981b7ea85da71aaf637

Merge "Added release notes for 'ContentHandler::runLegacyHooks' removal"

Merge "Add small script for common job queue admin tasks"

Added release notes for 'ContentHandler::runLegacyHooks' removal

Bug: T154498
Change-Id: I37402478beaac1848feff2ed95e97bebbec5cee0

Merge "Update docs/hooks.txt for ShowSearchHitTitle"

Merge "Fix Selenium tests"

Fix Selenium tests

The tests were broken because the feedback meesage is no longer
displayed as part of the page, but using HTML5 `required` attribute.

Bug: T155640
Change-Id: I9c46db374ec7f50262f84d721a21110aa6e81699

Replaced all deprecated Linker methods with proper ones in core(1)

Change-Id: Ie3a718dc1eae1507f8829fcf419c64c6846d2cb6

Update docs/hooks.txt for ShowSearchHitTitle

Reasons:
* $text is vague. It's the label of the link. There is a function named
  getTitleSnippet, which is what I'm reusing for the variable name.
* $page is vague. It's not the page of the search result, but the search
  special page.
* $queryString is not a string but an array of strings.

I also updated https://www.mediawiki.org/wiki/Manual:Hooks/ShowSearchHitTitle
accordingly.

Change-Id: Ia18dbf5ccf7f9fcc8890ad01641d0b7ff3e6eee8

Localisation updates from https://translatewiki.net.

Change-Id: Id4e35d1705d767dfd1130533acfb0d721cc5829e

Merge "Make most of DateFormatter private"

Merge "SpecialContributions: Username input is not really required"

Make most of DateFormatter private

As discussed in https://gerrit.wikimedia.org/r/#/c/332702/ , these
methods and fields shouldn't have been marked public in the first place.
No outside users. Also, declare a couple of fields and remove unused ones.

Change-Id: I7775978c87d983784a484ee2ad901d25c42499b3

SpecialContributions: Username input is not really required

Bug: T155780
Change-Id: If5cd6867029c005cf56a1628efc239955cd49931

Merge "Remove unused $resourceLoader"

Remove unused $resourceLoader

Change-Id: I6b5aebe2c6221ee101ec3bc4c84eafd9aeb7b0e2

resources: Update OOjs UI with fixes on top of v0.18.4 (master)

Cherry-picked fixes:

* be46e1773cd0f6b04510c325054d49835714b5f3
* 629389a777b35d131289a12cecf986e52ff65534
* 92dcdcf5351f6d499624d247b2a3ebff66346aca
* f991c382ce3dd80ed3498bb09e81885b94f8795e

Bug: T155711
Bug: T155728
Change-Id: I741db3a17eec812b50b4befcc7eba603943b759c

SpecialMIMEsearch: Add a dropdown with input suggestions

I think people usually distinguish file types by their extensions.
We shouldn't expect them to know what the valid values are here.

Change-Id: I854d48494e7b8c5eb1f49e364c5300df5cd73fe9

Merge "Un-blacklist PhanUndeclaredVariable"

Merge "Use HTTPS instead of HTTP for Amazon"

HTMLTextField: Set the 'dir' attribute in OOUI mode

Bug: T153636
Change-Id: I7ddf6a358f8150ef252d8d8e76661fe1eb93a790

Use HTTPS instead of HTTP for Amazon

This change is performed using:

sed -i -e 's/http:\/\/www.amazon/https:\/\/www.amazon/g' languages/messages/Messages*.php

Change-Id: I22c5e7a8d33a80b8cbe814f9735e7c24f1b004f9

Merge "Use parsed HTML error responses instead of api-error-*"

Merge "Add deprecated annotation to Article::doEditContent()"

Merge "Add more $wgPoolCounterConf comments"

Merge "Update tests/phpunit/MediaWikiTestCase.php with support for apcu"

Add deprecated annotation to Article::doEditContent()

Change-Id: I5a15b22c8b12fe77105fbd31427e8a15cde327c6

Update tests/phpunit/MediaWikiTestCase.php with support for apcu

Change-Id: I22d54e96c2c002ea53dabfd2772d2ba4c7eb96b1

Mark Article::doEditContent() as deprecated

Bug: T155696
Change-Id: Ie91b428f3a5992ba85c6ca2b2909953d8fce139b

Remove deprecated Parser::replaceUnusualEscapes()

Deprecated since 1.24, no callers.

Change-Id: Ib780f1a7b77d3ce624112f59c8e57820fecb6bf2

Remove unused fields in SpecialUnusedimages query

None of img_user, img_user_text, or img_description are used
by UnusedimagesPage nor ImageQueryPage.

Change-Id: Id0fdac9ac37eef097e8ca9f7bf3d66ef10bb83c2

Remove deprecated function Article::doEdit() and WikiPage::doEdit()

Bug: T61113
Bug: T154498
Change-Id: Ie6c1bb4ee5ee45ac2db432877a7038659231c091

Revision::getText() was removed

Bug: T154498
Change-Id: I29dcf768a9c176ca27b94359ac99cdc2b5ea930b

Article::getContent() was removed

Bug: T154498
Change-Id: I27956d494cf19d1204988807130b43f77186db9b

mediawiki.rcfilters: Add the remaining MW core filters

Add JSON definitions and i18n messages.

Bug: T149452
Change-Id: I287e9e5db9ff625bde026056efc81ce4b2401607

Add more $wgPoolCounterConf comments

Change-Id: Ic7b376dcada2b58a2f2ddc13b79e392e9d4fae6f

Add small script for common job queue admin tasks

This can handle dropping queued and re-enqueueing abandoned jobs.
Also use the DB cache table to avoid re-enqueuing duplicate jobs
if this has to get used several times for a queue during a week.

Change-Id: Ib56cf57e56f03ce6f1c4b52e3e36f228413c1056

Merge "Unbreak extensions using SpecialSearchResults hook"

Merge "Unbreak extensions from ShowSearchHitTitle signature change"

Unbreak extensions using SpecialSearchResults hook

Changing hook signatures with no deprecation period is a no-no.

Fixes Ib527fc3a3c39eb2e56985e5d1e4905fc4562353c
Reverts I153a7d590faceb1e2c4fe7d95a5cb931cda0c301

Bug: T155625
Change-Id: I2b408691fc4435ef24fc62360bd7ad6114f385c7

Merge "Avoid fatal error with FlaggedRevs when running rebuildFileCache.php"