Merge "Use getFileContents() instead of incorrect file system call to fetch file contents."

Remove unused class, SearchUpdate is backend-agnostic

Change-Id: I8c91c28ff2bde361490691b4abc107329f0d8194

Merge "filebackend: throw exceptions during file iteration."

Merge "UserCache::getUserName: Docs said @since 1.21, but not in REL1_21"

UserCache::getUserName: Docs said @since 1.21, but not in REL1_21

Change-Id: I4ad85ee36ed348e8b082b5a4dba11d7cb92d8ad0
Follow-up: ef915e37e3d872

Merge "vector: Add SVG versions of collapsible menu icons"

vector: Add SVG versions of collapsible menu icons

SVG with a PNG fallback is added for the collapsible tab icon.
It allows SVG-capable browsers to view images at full resolution
while keeping backwards compatibility with browsers that don't support
SVG.

Implementation similar to I914da06494.

Change-Id: I0448e34bd10aa5cf83f1530a44b1555caee271b1

Merge "Switched FileBackendStore to use ProfileSection when possible."

Merge "Avoid exceptions by first checking language code validity"

Improve documentation of action=parse (followup Ie4f7324c)

The documentation for action=parse needs clarification, particularly in
light of the changes made in Ie4f7324c which were too hastily merged.

Also, add the RELEASE-NOTES entry missing from Ie4f7324c.

Change-Id: Ia8847f7e99620a9f7af24956d972c882f78aae26

Restore API action=parse&text=foo functionality on wikidata.org

Bug: 48201
Change-Id: Ie4f7324c2a10700e52dadc4facd6b4fe6c76cf48

Merge "Only new vertical format login and signup forms"

Vector: Add the collapsibleTabs script from the Vector extension

* Move jquery.collapsibleTabs from the Vector extension
  to the Vector skin (see 097f387362)
* Register the module and its dependencies
* Merge the init script into the existing vector.js

Bug: 46513
Change-Id: I7bd77e6e4c384c9e2033f9d05ffcae86b0bb7a69

Switched FileBackendStore to use ProfileSection when possible.

* Also removed outer profiling layer.

Change-Id: Ice3b921fe0c9981ac16f005d66a78bf4f4a5c601

Merge "Support updating search index when page is deleted"

Support updating search index when page is deleted

Right now, if you delete a page then you end up with a stale
entry in the search index, this affects all core SQL-based
searches.

For extensions, this means they no longer have to implement
something like ArticleDelete, they can just add the delete()
method to their SearchEngine subclass.

Change-Id: I9d8a9878aeebc53f453ab1cbacc03fe73fcca949

Localisation updates from translatewiki.net.

Change-Id: I4e865f40a8c69fdc7520bd3a89ec61ca9af57337

Merge "Removed redundant doc comments."

Merge "Make rc_cur_id have proper value for upload log entries."

Merge "Link to mediawiki.org page in 'edithelppage' message"

Merge "Pass along tagFilter to ContribsPager, it already expects it"

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

Only new vertical format login and signup forms

* The "VForm" templates replace existing templates.
* Remove useNew switch logic and $wgUseVForm{UserLogin,CreateAccount}.
* The CSS and JS files for forms lose the .vform qualifier, except for
  the CSS for styling vform elements.
* Merge tiny mediawiki.special.userLogin.signup.js into createAccount.js.
* Remove replaced and obsolete messages (see
  https://www.mediawiki.org/wiki/Account_creation_user_experience/Strings
  for details), and remove mention of "new" forms and useNew from
  MessagesQqq.php.

Bug: 46333
Change-Id: I50f25583c3aa9a4a263fb40f50256f92c18ddb4d

Merge "mediawiki.api: Implement login method"

Make rc_cur_id have proper value for upload log entries.

Primary motivation here is to make Special:Recentchangelinked work
correctly for uploads where the description page doesn't exist yet.
(Previously it was mostly only showing reupload log entries)

Mild downside to this approach is adding the recentchanges row is
now in a different commit...

Change-Id: I74c68b972c47c03066d3065e3748769a27a654fb

Merge "Localisation updates from translatewiki.net."

test: rework interwiki lookup in parser tests

Some of our parser tests lookup interwikis.  This was originally done
(parser/parserTest.inc) by inserting a set of interwikis in the database
and was later lamely copy pasted in the PHPUnit wrapped test suite
(phpunit/includes/parser/NewParserTest.php).

Since that time, we had duplicate code and had the test hitting the
database to fetch interwiki.  Nowadays, we can trick the Interwiki
lookup class by using the InterwikiLoadPrefix hook, that let us skip
database lookup entirely (by having the hook returning false) and get
rid of the duplicate code.

The good old parserTests.php still pass the tests :-]

Change-Id: I36865e3890e08a05b8a6aaafa309a87556e235b9

Localisation updates from translatewiki.net.

Change-Id: I32973f48e15c917221cefe5d6d36a122e30cf706

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

filebackend: throw exceptions during file iteration.

* This lets calling code be far more robust rather than just
  silently ignoring entries due to some temporary problem.

Change-Id: I3ce2ae34f6cff5e40a80b8da5688503a387ce2a6

Removed redundant doc comments.

Change-Id: Iced16c6d32ef0e4d58376ed3c092a500ec917446

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"

Avoid exceptions by first checking language code validity

Bug: 49423
Change-Id: I3fd98ba08393856311a48fa40769027460c72ef9

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

mediawiki.api: Implement login method

Change-Id: I1113a076ff66e20ece1db9380969e7a7b5a68f1a

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

Use getFileContents() instead of incorrect file system call to fetch file contents.

Change-Id: I2b9be4165ae85c52ff9a9b68b9d71ec011a6c5b4