Pass along tagFilter to ContribsPager, it already expects it

ContribsPager already has this below line:

    $this->tagFilter = isset( $options['tagfilter'] ) ? $options['tagfilter'] : false;

The reason the code already works fine is because, in ChangeTags.php, there's this fallback:

    if ( $filter_tag === false ) {
     $filter_tag = $wgRequest->getVal( 'tagfilter' );
    }

Which essentially boils down to the same result as passing along tagfilter, which in
SpecialContributions is read from $wgRequest, passed to ContribsPager, and (well, currently
not) passed to ChangeTags

Change-Id: I12c0336ae31c5524e41fa0087afe2ba104dc7d37

Merge "Do not output numberofviews, if $wgDisableCounters = true"

Do not output numberofviews, if $wgDisableCounters = true

Follow up I9975dceedeffafadf560ed71238c51b94eb8c3ad
But now for the parser function

Change-Id: I18261b884b7f3be57a42c0b10594384fab6f9c9a

Merge "Remove Selenium tests from core"

Merge "jquery.makeCollapsible: cleanup"

Fixed mysql warning about expecting a resource.

Change-Id: I799d063fec7a8e96f49db34ce669453bd26c3fb3

Merge "Localisation updates from translatewiki.net."

Merge "Don't reinvent the wheel in SpecialRecentchangeslinked::getExtraOptions"

Localisation updates from translatewiki.net.

Change-Id: I352ed34fa912240395a72e84e60f19c42fe10fb3

ResourceLoader: Serve stylesheets with "Access-Control-Allow-Origin: *"

This allows Javascript to access our styling information (cssRules and
related cross domain protected dom information) even if it is hosted
on a different domain.

Bug: 25886
Change-Id: I8e05c13ae1a1589fd120d5c439b1a7128ce2b659

jquery.makeCollapsible: cleanup

* Ensure correct handling of togglers which hook to multiple elements
  and elements which have multiple togglers
* Move the `options === undefined` check outside the loop
  per CR on I8dd4f0d0
* Get rid of the old private toggling functions for various kinds of
  toggles (obsoleted, but left alone, in Id3f457a8)
* Don't use separate logic for initial toggling of custom togglers
* Consistently use collapseText/expandText (uppercase T) instead of
  collapsetext/expandtext (lowercase t) where possible
* Adjust comments and whitespace

Change-Id: Id8ecd8f577fb4cd3f640a7830a85436a301b3e2c

Localisation updates from translatewiki.net.

Change-Id: Ice334feb284bd21c83c26ea92705a1345d10b225

Merge "Cleaned up variable/type order in docs."

Fix typo ('associed' => 'associated').

Change-Id: I5ea2eec92ebe397f2d269562f1650e59cb969a19

Merge "Localisation updates from translatewiki.net."

Fixed doc typo

Change-Id: I1e4ab410cd02a74e667cc8611e58ad791558896a

Localisation updates from translatewiki.net.

Change-Id: If58755fdcfc3c12f16436c4a245a520cb037d994

Remove Selenium tests from core

This never really worked properly, doesn't conform to coding
conventions, and isn't actually used by anyone for testing.

Our selenium stuff these days are in the qa/browsertests repo

Change-Id: I8f1efaa118fe41821fb3f4a6099d75a33681f17b

(bug 47449) {{#special:}} should always return a usable title

{{#special:}} should always return a usable title in NS_SPECIAL,
even when given an unknown special page title. Currently, it returns
"No such special page" text for such unknown names, which is
surprising.

Bug: 47449
Change-Id: I7f184956ec9d56c849f201f1230736fcedd0fadf

Merge "Fix FOUC on preferences."

Merge "(Bug 48952) Show debug output in toolbar"

API: Fix action=parse without any page or title or text

action=parse was raising an error about missing 'text' with 'title' even
when 'title' wasn't actually passed, due to a fix for bug 33865. But
this broke using action=parse to parse an edit summary without also
giving it wikitext to parse.

Instead, let's give the client a warning (unless it seems clear they
know what they are doing) and assume the empty string for 'text'.

Also, while we're editing the file, add some more examples as also
requested in bug 48319.

Bug: 48319
Change-Id: I03c1fbcb0bd31dea8bd84e164104f7ced0ace449

Merge "Added parsoid-only tests for parsoid's native cite implementation"

Made purgeDeletedFiles.php delete files still in the public zone.

* This makes purges more useful since the file won't come back.
* This only applies for files that are confirmed to already be
  in the deleted zone too. This is for sanity.

Change-Id: I60316c1d9323347d09607e36a334fc5eb0b6a2e7

Added parsoid-only tests for parsoid's native cite implementation

Change-Id: If5cf091d7c6c864f6bd90165086a19ae38ec257e

Fix patrolling where NP patrol isn't explicitly enabled

Hot fix! Follow-up fc5bcf1147f9

Change-Id: Iba7e70e1a1ddd9826f6e5f4022b88ccafc5f571d

Localisation updates from translatewiki.net.

Change-Id: I4e12c5f98fb8da8d7b83e167363668b2504b26b2

Cleaned up variable/type order in docs.

* A few other minor doc cleanups and obvious type hints.

Change-Id: I454fad0c1942068b4f3632ebd1b605bfbb9dfe7c

API: Fix IPv6 handling in list=blocks

The current handling of the bkip parameter assumes IPv4, and breaks for
IPv6 CIDR ranges (it also isn't working right for IPv6 non-CIDR, but not
in an obvious way).

This rewrite handles IPv6 correctly. It also necessarily adds validation
for the bkip parameter, which would formerly return (not very sensible)
results when passed invalid values.

Bug: 48129
Change-Id: I02471bb32c3a217004d07a79d9f98b62133b31ef

Merge "Add section preview mode to API action=parse"

Merge "Ensure <h5> and <h6> have different computed font size"

(Bug 48952) Show debug output in toolbar

The debug output written with the function wfDebugLog should  be
printed to the debug toolbar in the same way as wfDebug does.

Change-Id: Ib979df080332e16c8e0e6a89eab8741986ab39c3

Prepare DatabaseMysql for mysqli

PHP extensions mysql and mysqli have lots in common.
In order to support mysqli, this diff separates common MySQL behavior to abstract class DatabaseMysqlBase and implements the mysql-extension specific in DatabaseMysql.
Outside behavior remains the same.

I've tried to minimize future code duplication by introducing mysql*() methods even if the parent method just unwraps the result.

Bug: 45288
Change-Id: I905d4a4550377bc849a860f0962dad710d9dc71f

Fixme comment for bug 49504

Change-Id: Ie229e84b299c357cfe9a2db0a5ba07713aac7597

Add section preview mode to API action=parse

In https://gerrit.wikimedia.org/r/#/c/67266/, it was enabled automatically
when a section was specified in preview mode. However, this only works for
previews of a saved page's content, failing for user input previews which
might contain several sections of various degrees of nestedness.

Change-Id: I987144e5e315ed9e0a67f599083461a78879c0df

Merge "Remove non UTF-8 chars from debug output"

Removed redundant documentation for some methods.

Change-Id: I72f132c0b6e9a300efff6c15957f163d84e7d9c2

Merge "Localisation updates from translatewiki.net."

Removed redundant documentation for some methods.

Change-Id: Iad422716f6796cc4b69405182f4a74daa0c93663

support setting Memcached::OPT_SERVER_FAILURE_LIMIT, Memcached::OPT_RETRY_TIMEOUT

Change-Id: Ib1fcae92b65e8fd23baee7af7ed64abd908b600e

Localisation updates from translatewiki.net.

Change-Id: I599770c4c8b53d0d46c74f1426168a5c3d963a81

Remove unused searchhelp-url and searchmenu-help

searchhelp-url was introduced around r43595 to improve
searchmenu-help (r43376). Both were part of a Special:Search redesign
that never really happened. Commented out in r50207, and removed
in r51228

See also Ic0469c6c51a4d54ee214347fc416615c2f6cb1f0

Change-Id: I82259bd836ba8c4a3042cb81c99165c312ca04cc

Fix return type of MessageCache::getMsgFromNamespace for existing

Functions expect the message cache to return a string if a message
exists, even empty, and false if it does not exist. This adds casting
to the substr() function, which would return false for existing
messages that were just blank.

Bug: 14176
Change-Id: Id91914a3701fe53f1e2e894824512489392c628b

Merge "Setting tooltip gravity to 's' so that the tooltip shows above"

Merge "Localisation updates from translatewiki.net."

Localisation updates from translatewiki.net.

Change-Id: I7458f257824c9b1a4f14d6cc7a9567ee454749c7

In 'uctop', replace 'top' with 'current' [revision]

We can continue calling the "top revision" such in English,
but no need to confuse everyone looking at Special:Contributions.
The closest "translation" to human English would be "last",
however that can be confused with "last by you" etc.
Most translations are already "current" or "last revision/for the page".
https://translatewiki.net/w/i.php?title=Special%3ATranslations&message=Uctop&namespace=8

Also changed on en.wiki, 24 February 2013, by WilliamH:
«"top" is vague and its meaning must be inferred».
<https://en.wikipedia.org/w/index.php?title=MediaWiki:Uctop&oldid=540122804>
Kudos to the first sysop caring 9 years after complaints on talk.

Change-Id: I75d28d344d2e4e468295b0d6bbb0dcf19b459427

Setting tooltip gravity to 's' so that the tooltip shows above

Right now the tooltip gravity is defaulting to north which causes
the tooltip to be partially overlapped by the cursor. Setting it
to south makes it easier to read since it appears above the icon
and isn't obscured by the cursor.

Change-Id: I9aa43c9ff93fbded1b51bf34c084ec0f8cad28b7

Merge "Only show notoken as possible API error once"

Merge "(bug 42253) link user in accountcreatedtext msg"

Merge "Remove unnecessary exception from Title::getLatestRevID()"

Remove unnecessary exception from Title::getLatestRevID()

Bug 37209: An exception was thrown where the data cached in the Title
object was inconsistent with the data in LinkCache. It shouldn't be
surprising that this happens, since there is no guarantee that the data
was derived from the same transaction or even the same DB server.

But I don't think it is a problem worth troubling the user over, since
with $flags=0, the slave DB server will be used, and no special guarantee
of consistency should be expected by callers. If callers do need
consistency, then they should make their own arrangements to get it,
such as clearing the LinkCache.

Since we have to pick a winner, and the choice is mostly arbitrary since
it's not possible to tell which is fresher, I think LinkCache is a better
choice since the lifetime of its cache entries can be controlled.

Change-Id: I0add48463341e56fe8c155b1007487278ad2705d

(bug 42253) link user in accountcreatedtext msg

Why not provide links to new user's page and talk page?  This is only
when a logged-in user creates a new account, so presumably he or she
knows what he's doing and appreciates the convenience links.

Also improve documentation of accountcreated in MessagesQqq.php.

Change-Id: I4e88829d8665c5c61a59877962a8a14b55c91aa0

Merge changes I4950496e,I849c4bd1

* changes:
  Remove all uses of obsolete .editsection class
  Remove Wikimedia-specific hack in ParserOutput

Allow patrolling pages by revision id

This became necessary as it's quite hard to
get the rc id of a change from the html after
I1e24733c.

Bug: 49259
Change-Id: Ia7d3960cf11bf8ae0fc06ae1a0f7fcfb3c080f21

[Database] Tweaked LoadMonitor::getLagTimes() to further reduce stampedes

Change-Id: I79e3232e3e87bb298ae1a3fe214fcd2506818884

Drop __autoload.

spl_autoload_register exists in >=5.1.2, we require >=5.3.2
We no longer support hphpc and it looks like hhvm supports spl_autoload_register

We've got no reason to keep using __autoload.
We'll also need to exclusively use spl_autoload_register when we start using composer.json stuff.

Change-Id: I694b7b96825e5d136fa76461511efc9a002149fa

Only show notoken as possible API error once

Currently notoken is being listed as possible
API error twice if it's explicitly set as
required parameter and needsToken() returns
true.

See:
https://www.wikidata.org/w/api.php?action=paraminfo&modules=edit

Change-Id: Ia17c5cfa634919b43affa146df0d1dc0ff06b758

Add new hook AbortTalkPageEmailNotification

This hook will allow extension to disable the regular talk page email notification,
Echo is doing this with a hackish solution of setting global variable
wgEnotifUserTalk to false on the fly, this approach is problematic and will
be replaced with this hook

Change-Id: Ie043d9a1a771d64e4b05b7ddeca75c7542a6c575

Merge "Add user rights 'editmyuserjs' and 'editmyusercss'"

Merge "Remove revision patrol links from normal page views"

Add user rights 'editmyuserjs' and 'editmyusercss'

These are needed for OAuth grants.

Change-Id: I52f8e4a5cb48573cb2dbc26fc508e61a95d748c3

Merge "Revert "Introducing ContentGetParserOutput hook.""

Revert "Introducing ContentGetParserOutput hook."

This change removed getHtml(), but not the call to it from fillParserOutput.
It has a few other issues that I'll flag in-line after deploying the revert.

This reverts commit fda090a7e7f85f146ec84798fda45232ab67613e

Bug: 49398
Change-Id: Idcef8d4cedc7c03f72bc1743e0f89cc5ed4ad7a7

Merge "Add vendor to mwdocgen's excludes list."

Merge "Fix bad copy-paste error in deprecated method"

Merge "mw.toolbar: Clean up code and jsduckify"

mw.toolbar: Clean up code and jsduckify

* Removed remnants of the 7th 'selectText' parameter. This
  was still tossed around in a few places, but in the function
  it ends up in (toolbar.insertTags) it is no longer used.
* Per code conventions, don't use jQuery's magic attrMap
  argument, instead call the (in this case attr) method directly.
* Optimise way we get the reference to the detached Array#slice,
  access 'slice' on local array instead of global variabile
  lookup for 'Array' and accessing prototype (learned from jQuery core).
* Link "mw.toolbar.addButton" is invalid in jsduck, members
  of modules are accessed by a hash sign. Should be
  "mw.toolbar#addButton" or even just #addButton in this case.
* Eliminated scrollEditBox function by inlining its contents into the
  $( document ).ready handler.

Change-Id: I867f2bcfd97e00a498a963e5d9730e5b0d36dcae

Merge "Kill <meta name="keywords" content="..."> in a blazing fire."

Merge "Followup mistake in I82711cee"

Merge "Remove unneeded parenthesis from warning text in prop=revisions"

Localisation updates from translatewiki.net.

Change-Id: I826587e7cb21c3bf1376701839b1a8c11c2e751e

Merge "Move post-edit font-family rule to Vector-specific selector"

Merge "mediawiki.user: Simplify implementation and jsduckify"

Remove unneeded parenthesis from warning text in prop=revisions

Change-Id: I8d2ca6a4b7199fa1f4fe3b3e63dbefb2cc273d7b

Merge "CSSJanus: don't mangle 5+ consecutive numeric values"

mediawiki.user: Simplify implementation and jsduckify

Similar to mediawiki.util:
* Return plain object instead of an instantiated function.
* Refer to local reference instead of 'this' which makes it
  possible to use all methods "detached" without needing
  a specific context bound.
* Fix documentation comments be valid in jsduck syntax.
  - @example was invalid
  - "foo String : desc" was invalid, changed to:
    "{string} foo Desc"
  - type {false} was invalid, changed to {boolean}
  - reference mw.user.foo was invalid, changed to
    mw.user#foo.
  - getUserInfo was missing @private.
* Include in JSDuck index.

Change-Id: I2732cd7422f444b5cad1c6737a2de5dafb12d541

Remove revision patrol links from normal page views

Users want and expect it this way, per bug 49123

Follow up to I1e24733c

Bug: 49123
Change-Id: Ib4d72179e4029f0c089c3147bdf4bd6daac0374e

Move post-edit font-family rule to Vector-specific selector

The post-edit confirmation message is rendered in the same font-family
across skins due to the font-family rule not being scoped to Vector. This
change moves the font-family rule to a Vector-specific selector, making
the font-family in other skins the inherited one.

Change-Id: I6dd0d0aa5fc6f4773529bec416a7ab35606a1bbc

Kill <meta name="keywords" content="..."> in a blazing fire.

meta-keywords is completely and totally useless. It has absolutely no effect
on search engine and no-one uses it anymore. Time to euthanize it.

There isn't a single user of OutputPage::addKeywords inside any extension
inside of Gerrit. So I'm just deleting this method without a deprecation.

Change-Id: I188755521dcde3a9e191975d1ae3cabf7a5d67cd

Followup If21705c2, Shorten remaining urls to the whatwg HTML spec.

Finish up with the /specs/web-apps/current-work/multipage/ urls that
haven't been updated to /html/.

Change-Id: I4dbee0477eea440b0e8f113b1d393c6e0c739c4c

Add vendor to mwdocgen's excludes list.

Now that we support composer libraries we should exclude vendor/
so that our Doxygen script doesn't go generating documentation
for 3rd party libraries we use.

Change-Id: I40502015820f0b9b4b1ef5810bcddcc5970134c5

Followup mistake in I82711cee

Can't believe the fact the @deprecated has the wrong version number
made it through our review process.

Change-Id: If9beea75bb909484b242c1c4cb787fef8f6501d3

Merge "Fix XmlTypeCheck docs"

Merge "Fix comment for mw.user.sessionId()"

Fix XmlTypeCheck docs

Change-Id: Ia6a394a0dd4fc74c2c2435e48104df367a6b714a

Fix comment for mw.user.sessionId()

Change-Id: I19ab8b6de9f7bfe4418a6cab863ee97b77905795

Fix Thai link trail regexp

Escaping the end anchor is unlikely to be useful. This was noticed as it
resulted in an invalid regexp after being transformed in Parsoid.

Change-Id: I955215c83ff29032f8276b0fa3e6b9701a1f2359

Merge "Made SwiftFileBackend::loadObjectListing() populate stat entries in reverse order."

Merge "Hide exceptions in MediaWiki::doJobs() as deferred updates do."

Gitweb -> Gitblit

Change-Id: I54387a6ca5ed020e66594d4bf410a042d4ae010f

Merge "Support installing PHPUnit using composer."

Merge "Enhancement: Allow XmlTypeCheck for strings"

Merge "Allow server selection when running sql.php"

Merge "Use getPrefixedDBkey() for template profile call."

Enhancement: Allow XmlTypeCheck for strings

This change allows that the class XmlTypeCheck can be used to
check strings as well as files. Therefore the constructor has
been extended to one further element that indicates if the
input to be checked is a file or a string.

The functionality is tested with php unit tests.

Change-Id: I7432a754a512e0324c171667c3eac9d8e7838e0e

Show rollback link on changes list only for RC_EDIT

Changing from !RC_NEW to RC_EDIT to have no rollback links, when this
method is called accidently for RC_LOG or RC_EXTERNAL (or for the old
RC_MOVE/RC_MOVE_OVER_REDIRECT). For that types there are 0 revisions to
rollback.

Change-Id: I1cbdc87a8b8e0cecf6fbfa3b3a64d5015057bccf

Use getPrefixedDBkey() for template profile call.

Change-Id: Iac5183635984aa9261c1e85e32f6bb173ec513ae

Merge "Fixed long memcached keys for infoaction."

Ensure <h5> and <h6> have different computed font size

Browsers tend to have a default font size of 16px and the Vector and Monobook
skins use various techniques to end up at 12.8px resp. 12.7px for the bodytext.

Now, in commonElements.css, we have 105% for <h5> and 100% for <h6>. This gives
rounded values of 13px for <h5> and <h6> in Vector and Monobook, making them
hard to distinguish.

This patch enlarges <h5> to 108% in Vector and Monobook.

Bug: 48671
Change-Id: I9d0fccc687ceba05a6af847efc374fdef04f9b38