Avoid exceptions by first checking language code validity

Bug: 49423
Change-Id: I3fd98ba08393856311a48fa40769027460c72ef9

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

Remove all uses of obsolete .editsection class

Cleanup after I6a6c12a9. To be merged after appropriate caches are purged.

Change-Id: I4950496e584735a3ceeb0e1c4123dfe6597f2ae1

Remove Wikimedia-specific hack in ParserOutput

Cleanup after I6a6c12a9. To be merged after appropriate caches are purged.

Change-Id: I849c4bd1fe59e582f9f16789e8b511719e2a5d06

Support installing PHPUnit using composer.

"phpunit/phpunit" already exists inside our composer.json's "require-dev" however this has been
entirely useless as we don't include the autoloader which would load composer's PHPUnit.

This change begins including composer's autoloader when present and also tweaks phpunit.php
to ensure PHPUnit isn't double loaded. As a result besides supporting PHPUnit via composer this
also means that we're ready to make use of any library we add to our composer.json in the future.

Change-Id: I891740e8fd3d237c5f473862027205d951f564b9

Merge ""Your domain" appearance fixes for new forms"

Fixed long memcached keys for infoaction.

* One place was already using SHA1, so invalidation was broken.

Change-Id: Ie0d3b9274ed2e1c5a7b1616cd237a5e25fd99d12

edit.php maintenance script not setting up context, just globals

Was breaking one of Echo's hooks

Bug: 49271
Change-Id: Ie1b5f5bc4a672bcfff3c99b9a76b37c502c09985

action=parse now can parse in preview mode

Bug: 49239
Change-Id: Iacfea0219ef21dc91d669f51140ec129c2f41f76

CSSJanus: don't mangle 5+ consecutive numeric values

Don't mangle 5+ consecutive numeric values in the
'four_notation_quantity' and 'four_notation_color' rules. This
prevents box-shadow rules from being utterly broken in RTL (but still
doesn't flip them properly).

Bug: 45677
Change-Id: I16cb9e171a79c6f299b40fa02908b0c045e3c474

"Your domain" appearance fixes for new forms

* Restore lost id=mw-user-domain-section (so wikitech can hide it).
* Remove obsolete pos-above class.
* Fix message key.

Bug: 49225
Change-Id: I73bcd44eff10a4fd2d1a392e82427e42e92fc3bf

Refactored WikiPage::updateCategoryCounts() to use upsert().

* Also made it avoid inserting aggregate rows on membership
  decrement if there was no prior row aggregate row.

Change-Id: I7e6d07e42cc06cedbd9d10ed9f276f20636dc222

Merge "Doc fix for MediaHandler::getThumbType"

The language used to render SVGs should be definable

See bug for context.

The implementation is slightly untidy because I've written it so
as to avoid invalidating the existing SVG thumbs -- there will be
no immediate difference (visual/performance/other) as a result of
this.

Tested by me in both...
* [[File:Example.svg|thumb|lang=fr]] AND
* http://example.org/w/index.php?title=File:Example.svg&lang=fr
...modes. Example file on
https://commons.wikimedia.org/wiki/File:Gerrit_patchset_25838_test.svg

Added parser tests.

Bug: 32987
Change-Id: I4cadf96ecd5e169a88ad468a0478d355db980103

Doc fix for MediaHandler::getThumbType

Change-Id: I892937b3bcaf976537c89cb4f52fab20a2fba5b0

Follow up I6e64a missing global deceleration for $wgHTTPConnectTimeout

Bug: 47027
Change-Id: I3e2a84ee8a759dd2bdb2fd0837e0cd5605541816

Merge "RefreshLinks on imagelinks on new uploads to kill tracking cat."

Shorten our in-comment urls to the whatwg HTML spec.

whatwg.org has a redirect to /specs/web-apps/current-work/multipage/ from /html/.

Change-Id: If21705c214ca8f14db5a0c6dda3c43c22f9ca811

RefreshLinks on imagelinks on new uploads to kill tracking cat.

When someone uploads a new file, we should refresh all the pages
that use that image, so that [[Category:Pages with broken file links]]
gets removed.

Future fixme would to run this again on file deletes, but that's
less critical as they will eventually get added on next reparse
and people don't expect that to happen instantly like when fixing
the issue. Ditto for someone creating a new file redirect.

Bug: 46749
Change-Id: I7889598eee54a1c07c64f750b8499e922ba59704

Merge "jquery.client: Fix profile of some UAs and add exactMatchOnly param"

jquery.client: Fix profile of some UAs and add exactMatchOnly param

Fix detection of Iceweasel (add to wildUserAgents), Safari (make
comma after KHTML optional) and Android (add to versionPrefixes
and names).

Add 'exactMatchOnly' parameter to test function which triggers
a return value of false if the browser is not found (previously
was always true). Also making ltr/rtl splitting in support map
optional.

Change-Id: I541a6c134e9668f6bf5af49d4508a82d4f546bb6

Merge "jquery.tablesorter: Reset unaffected columns' sort counts when sorting"

[LockManager] Made it easier to get both SH and EX locks at once.

* This also makes it possible for subclasses to optimize this case.
* Added a timeout parameter to ScopedLock::factory().
* Cleaned up a few bits of documentation.

Change-Id: Id3e9cf01f25ab498ea00d87ffb6d00aa8b05052b

Allow setting of connection timeouts for HTTP requests using cURL

Bug: 47027
Change-Id: I6e64a8bfc58e899149463d305eac672c1e8ad2ba

Remove unused global

Follow-up 22eb569

Change-Id: I2c34b1f02c7304bd6864a7402c8be0ab313c7162

Follow-up 05077dc3740

* s/___METHOD__/__METHOD__/
* Remove the mention to cur table as deprecation reason,
since it is obviously not trying to use cur (removed in 1.5),
this code is clearly newer (expecting data from a mixture of
tables), although using an old interface.

Change-Id: I0b4fdf7de4f56c1100ef3cdd7705e8f6701bc7cf

Merge "Remove the $wgHandheldForIPhone config variable entirely"

Remove the $wgHandheldForIPhone config variable entirely

It only really made sense in pair with $wgHandheldStyle, and that has
been removed in Ia8d79b4a.

Remove irrelevant tests, adjust still relevant ones.

Change-Id: I7c24128f7b148d0244538ad95bb60bf09ec4b5cb

Merge "More cleanup when removing accounts with removeUnusedAccounts.php"

Merge "mediawiki.icon: Move arrow icons to a separate module"

More cleanup when removing accounts with removeUnusedAccounts.php

Delete entries from user_groups, user_former_groups and user_properties tables too

Change-Id: I581e5e6b749b9718fee3187513adc02e78c1a71b

Added tests for new DatabaseBase::upsert

Follow up Id7fc6652
Pass function name to begin/rollback/commit
Use __METHOD__ as default param (see I86cbdeab)

Change-Id: I9eb326c035d4a604db5b3492f090d8dd9d21c920

Merge "User: Clean up coding style and documentation"

Introducing ContentGetParserOutput hook.

This hooks allows extensions to override the normal model-specific rendering
of page content. A typical use would be to provide syntax highlighting for
pages that contain scripts. In that sense, ContentGetParserOutput is a
generalization of the old ShowRawCssJs hook.

Change-Id: Ibfb2cbefea44eeee9f2a027f47e7721bf177ba0f

mediawiki.icon: Move arrow icons to a separate module

The arrow icons for the enhanced watchlist/recent changes could be
useful elsewhere. But to do that, we need to be able to load them
separately; this does that by adding a new module just for the CSS
needed for the arrows.

Also, this fixes a use of OutputPage::addModuleStyles that really should
have been OutputPage::addModules so dependencies are resolved.

Change-Id: I639a5ccbc3b7b2794e7b12ecb64e5ee6b10a640a

User: Clean up coding style and documentation

* Parameter type hint should be a variable type or a class
  name. Types Null, Bool, Boolean, Int, Integer, String do
  not exist and are invalid values. Changed to their PHP
  type names (basically just lowercase).
* Made inline comments consistently use "//" instead of "#".
* Removed various odd spurious spaces sequences in @param
  lines. They are sometimes used to pretty-align multi-line
  descriptions. Though I think that is silly, I've left those
  alone. I've only removed the spaces where it seems arbitrary
  and not aligning with anything. Last I checked the common
  convention for multi-line descriptions is to simply continue
  on the next line with one extra space as indention.
* Swapped "$var Type" in cases where the surrounding @param's
  already use "Type $var".

Change-Id: I6b315e308271194815879c509d3ac6aeb409851b

Merge "RecentChange: deprecate newFromCurRow and loadFromCurRow"

Merge "Cologne Blue: add missing 'undeletethispage' message"

Merge "Allow patrolling page creations from Special:RecentChanges"

Localisation updates from translatewiki.net.

Change-Id: Ibaaab99ded40c3a8badde9abfcc9710d01555dd9

[Database] Added an upsert() function to perform/emulate ON DUPLICATE KEY UPDATE.

Change-Id: Id7fc6652268a439af0457309a678677351867f37

Add a script to remove deleted files from storage

Bug: 47990
Change-Id: I2fcc2b2c3e92cd851c38f947de6756a982cf194d

remove invalid comment from SkinVector::setupSkinUserCss

It seems to refer to pre-ResourceLoader situation (or at least
pre-skins.vector one). Originally added in 6df94d49, back in 2010.

Change-Id: I759fa99041db465ec6baff94b6b96a79b47438dd

Merge "Fix edit patrolling in multi Diffs"

invalidparammix/missingparam get the module prefix twice

dieUsage will call encodeParamName, which will already prefix the error code with the module prefix

E.g. for ApiViewFeedbackArticleFeedbackv5 (prefix afvf), this error code was
(incorrectly): afvfafvfinvalidparammix. Should be: afvfinvalidparammix

Change-Id: Ia351678b49e96ef58dce773331ebe9a1b3ebf824

mediawiki.legacy.ajax: Wrap Sajax methods in mw.log.deprecate

Bug: 40785
Change-Id: I52f01e4e7d8a0ee595b4775068c8a3d630140092

Merge "Make headers in FauxRequest case insensitive"

Merge "Remove occurence of $wgOut in HTMLForm::displayForm()'s documentation"

Make headers in FauxRequest case insensitive

HTTP headers are case insensitive per spec, and WebRequest
treats them like that, so FauxRequest should too.

Change-Id: I4257af7a8de2792ac556c670dcc7f28e4af4cb44

Merge "mediawiki.legacy.ajax: Pass JSHint"