Merge "EditPage: Mark public functions"

Merge "PHAN: lock phpstorm-stubs at a specific hash in composer.json"

PHAN: lock phpstorm-stubs at a specific hash in composer.json

This will stop updates to the upstream package from
breaking our CI with no warning.

Change-Id: I1cafe663c25d46134025f7c6987e48676615b7b0

PHAN: Add tideways stub

Used in Xhprof

Change-Id: I9f208bbaa0799d194dbef61cad851995020a5d17

PHAN: load stubs from phpstorm-stubs library

Also add a single small stub for the Memcached class
constructor that is slightly wrong in the phpstorm
stubs library.

jetbrains/phpstorm-stubs has been added to packagist via:
https://github.com/JetBrains/phpstorm-stubs/pull/149

Change-Id: I1bca7390d1dc24c38d65c1c8968f356326ff4636

Merge "Fixes for more robust dealing with content handlers."

Merge "Make user_groups queries honor $wgDisableUserGroupExpiry"

Make user_groups queries honor $wgDisableUserGroupExpiry

Follow-up to Icf78cce3f3e362677d10897b8d1103b3df91fa08.

Change-Id: I94102ceb00398940965280d8742a2747631dcc87

Merge "Fix positioning error with draggable, revert and grid"

Fix positioning error with draggable, revert and grid

This minor change fixes an error when using draggable revert and grid.

Since upgrading of jQuery UI probably won't happen in the near future [1]
I looked for another way to fix the issue.

See [2]. Unlike stated in the second comment adding the lines below
won't fix the bug. Moving the lines up fixed the issue.

See also the description of the bug in [3].

[1] https://phabricator.wikimedia.org/T71386
[2] https://bugs.jqueryui.com/ticket/4696
[3] https://phabricator.wikimedia.org/T140965#2944610

Bug: T140965
Change-Id: Ib0f386c6a2447740f44591be70c34cf4a83b88c4

Merge "Remove unused variables in RCCacheEntryFactoryTest"

parserTests: Avoid using <big> for adoption agency algorithm test

The behavior of <big> may change in the future, c.f. T154067.

Change-Id: I817894c25cab96a491028fe2a9443140ea1d6e97

Merge "registration: Don't allow extra properties in "requires""

registration: Don't allow extra properties in "requires"

VersionChecker is using a switch with a exception in the default path,
which makes it very unhappy to see additional properties.

Change-Id: Ief84497de6b2fa2d2715fc713088bee66c21fdc0

Merge "API: Add reference to the mailing list in errors and deprecation warnings"

Merge "Fix tags not being set in Special:Block"

Localisation updates from https://translatewiki.net.

Change-Id: I1963ac8ae1166037a2550fe628fe06812bf2b843

Fix tags not being set in Special:Block

Currently, a PHP error is thrown when tags are not set
in Special:Block on line 832. This patch fixes this
by adding an extra isset() check to see if the tags
variable is set.

Bug: T156486
Change-Id: Ib8722bffbcac5953263ded41eceb3d389d0932f0

Localisation updates from https://translatewiki.net.

Change-Id: I274c0404681ea7e97d3c100384a58bbde8f80cdc

Remove unused variables in RCCacheEntryFactoryTest

Change-Id: I41478dc2086577aacf444d0620c8b950e94e61ff

Remove HTMLFileCache::newFromTitle()

Change-Id: I0076801aa7b9af5083c5a7a66f2d1c9884b70ee9

Remove ArchivedFile::getUserText()

Deprecated since 1.23
No usages found on gerrit

Change-Id: I037d56d6e57288753ae281388270d7e216707192

Merge "Add tests for SkinTemplate::setupSkinUserCss"

Remove User::crypt() & User::comparePasswords()

These have both been deprecated since 1.24
Hard deprecation happened back in 2014

Both methods are still used by the SecurePasswords
extension, but this extension is documented on mw.org
as not working with MW1.24+.

I can find no other uses.
Lets finally get rid of these!

Change-Id: I94a7b65d2216bbc505e190af3182de2317976ed1

Merge "Allow OpenGraph RFDa meta tags"

Merge "translateBlockExpiry: Duration is block expiry minus current time"

translateBlockExpiry: Duration is block expiry minus current time

For relative timestamps in $str, strtotime( $str, $now ) returns an
absolute Unix timestamp $str since $now, and this timestamp is given
to $time. However, Language::formatDuration expects a time duration,
not an absolute timestamp. We obtain this duration from the difference
between $time, the absolute timestamp of block expiry, and $now, the
absolute timestamp of the time in which the block action happened.

Tests have been added to test both this patch and 01936fa, the patch
that caused this regression.

Bug: T156453
Change-Id: I6fd8c02dc3c6456067fe25cb9f33f5b4c78332aa

Add Related Changes Link for all categories

Add Related Changes link into the toolbox for the redlinked
categories as well since users need to see changes made in these
categories as well.

Bug: T147762
Change-Id: Ic5dd6f531b5ff47ce68fd2893fd1550d10f74e4e

Disregard expired user_group rows in special page and API DB queries

An essential follow-up to I93c955dc7a970f78e32aa503c01c67da30971d1a.

Bug: T12493
Change-Id: Icf78cce3f3e362677d10897b8d1103b3df91fa08

Merge "Allow SearchEngine users to access features data"

Merge "User group memberships that expire"

Allow SearchEngine users to access features data

Useful in case the client wants to re-evaluate what was set
here, or if the SearchEngine implementation wants to expose
some of its states.
In our case it allows CirrusSearch to inform SpecialSearch
that we prefer to display search results with a new experimental
layout.

Bug: T156299
Change-Id: I7f661c852ef70ea7bc9ae2959f7d6e48776a9877

Localisation updates from https://translatewiki.net.

Change-Id: I8ccae21da1014b794a87ddbd4ade8be9b2c9edd5

Merge "Add content model indexing"

User group memberships that expire

This patch adds an ug_expiry column to the user_groups table, a timestamp
giving a date when the user group expires. A new UserGroupMembership class,
based on the Block class, manages entries in this table.

When the expiry date passes, the row in user_groups is ignored, and will
eventually be purged from the DB when UserGroupMembership::insert is next
called. Old, expired user group memberships are not kept; instead, the log
entries are available to find the history of these memberships, similar
to the way it has always worked for blocks and protections.

Anyone getting user group info through the User object will get correct
information. However, code that reads the user_groups table directly will
now need to skip over rows with ug_expiry < wfTimestampNow(). See
UsersPager for an example of how to do this.

NULL is used to represent infinite (no) expiry, rather than a string
'infinity' or similar (except in the API). This allows existing user group
assignments and log entries, which are all infinite in duration, to be
treated the same as new, infinite-length memberships, without special
casing everything.

The whole thing is behind the temporary feature flag
$wgDisableUserGroupExpiry, in accordance with the WMF schema change policy.

The opportunity has been taken to refactor some static user-group-related
functions out of User into UserGroupMembership, and also to add a primary
key (ug_user, ug_group) to the user_groups table.

There are a few breaking changes:
- UserRightsProxy-like objects are now required to have a
  getGroupMemberships() function.
- $user->mGroups (on a User object) is no longer present.
- Some protected functions in UsersPager are altered or removed.
- The UsersPagerDoBatchLookups hook (unused in any Wikimedia Git-hosted
  extension) has a change of parameter.

Bug: T12493
Depends-On: Ia9616e1e35184fed9058d2d39afbe1038f56d7fa
Depends-On: I86eb1d5619347ce54a5f33a591417742ebe5d6f8
Change-Id: I93c955dc7a970f78e32aa503c01c67da30971d1a

Further updates to .mailmap for the generation of CREDITS

Mostly adding current and former WMF/WMDE employees, and a couple of
others, under the correct version of their full names.

Change-Id: I36ca297967125cc211705509466358d9d762e5c8

objectcache: add WANObjectCacheReaper for assuring purges

* This fixes keys based on some sort of change log.
  Updates are wrapped in a mutex and keep track of the
  last known good position.
* Make WANObjectReapUpdate class that cleans up title
  related keys using the recentchanges table. This triggers
  as a deferred updates on RC view.

Change-Id: I7f14b9ca2533032147e62b1a3cc004a23da86579

Merge "rcfeed: Add basic PHPUnit integration test"

Add content model indexing

Bug: T156371
Change-Id: I1514f9db70870ec05b30153cc56558af0a28b8c9

Fixes for more robust dealing with content handlers.

Change-Id: I12a02da005f4b2bceaa850bd1f41a90ac4e1754a

Merge "ApiUpload: Raise an error on attempts to async-upload a POSTed file"

EditPage: Mark public functions

Change-Id: I269db9dcbf9cc6ef75a323eb395d12faa09ea20e

Merge "Avoid master queries when old filter checks are viewed"

Merge "ApiSandbox: Allow displaying query parameters in various formats"

ApiUpload: Raise an error on attempts to async-upload a POSTed file

"Upload from stash" and "Upload from POST" use the same code path via
ApiUpload::performUpload(). It looks like when a31d0f8edd implemented
"async upload from stash", it was overlooked that someone might attempt
to do an unsupported "async upload from POST".

Bug: T156365
Change-Id: I6d207270af490cfc59d994a22a6406df45a9f6a2

Clarify emailuserfooter mw-message

Recently we modified this message to add a privacy disclaimer which is
automatically added to outbound email sent using the Special:EmailUser
function on MediaWiki.

The disclaimer, notwithstanding, it only shown for the receiver of the
message so we should clarify that their email address will be disclosed to
the original sender only if they decide to reply to the email.

This patch therefore adds a "If you reply"-like wording to the
emailuserfooter message clearly stating that only if the user decides to
reply the message, their email address will be indeed disclosed to that
original sender.

Change-Id: Ie2ccf26b078e768f615eea8c8a83d92ff83abce0

Merge "Fixing grammar, capitalization and spelling in comments"

Merge "refreshLinks.php: allow refreshing by categories, tracking or not"

Merge "Inject remaining cache objects into MessageCache"

Merge "Avoid races in MessageCache::replace()"

rcfeed: Add basic PHPUnit integration test

Change-Id: Ifb5ddd4bda6cd1be050da7a5419ebe90f5ecf60f

Merge "UploadBase: Allow RDF Schema namespace in SVG files"

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."

Allow OpenGraph RFDa meta tags

See: http://stackoverflow.com/questions/22350105/whats-the-difference-between-meta-name-and-meta-property
And https://developers.facebook.com/docs/sharing/webmasters#markup

Bug: T51859
Change-Id: Ie8fd697e8588435a69f02e779038ae5a11d67be6

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

Avoid races in MessageCache::replace()

Do the process cache update immediately (as before) but push
the shared cache updates to a deferred update. This update
will thus start with a clear transaction snapshot, so it can
acquire the lock before the first SELECT as is proper.

Also added some missing method visibilities.

Bug: T144952
Change-Id: I462554b300d4688b09ab80cd1bb8a4340ffaa786

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"

refreshLinks.php: allow refreshing by categories, tracking or not

Needed for selective updates of pages using a particular feature.
Intended to be run in production, so needs to scale.

Bug: T149723
Change-Id: If20fb1f91de8d4227def5b07d6d52b91161ed3fd

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

ApiSandbox: Allow displaying query parameters in various formats

I9c24a230 added a box with a JSON-formatted version of the request
parmeters, saying "In the future we can provide more formats via a
dropdown". That future should have been then. It is now.

Beyond simply making it possible to select the format via a dropdown,
this change adds a JS hook, apisandbox.formatRequest, to allow adding
new formats from extensions, gadgets, and site scripts instead of
requiring them all to be added to core.

Change-Id: I20b093e0891f36cb3f1faa3eb3d6aed0f17b6b7d

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

Localisation updates from https://translatewiki.net.

Change-Id: If2cf309a446a47958a10a6fc6d16498e1b52eb80