Make list=logevents display log entries by anonymous users

Also fix warnings being shown and "anon" property being given to non-anon entries
when "userid" leprop was used but no "user".

Bug: 50315
Change-Id: I4df1c8c1040fb39d75ead5851d42b02d9de87a5a

Merge "docs: Fix member index for mw.html"

Merge "Fix return types in documentation"

Fix return types in documentation

Change-Id: I726e6201a737ab406dd0e46754d2bbbd95e769b4

Merge "New function Language::getParentLanguage()."

Localisation updates from translatewiki.net.

Change-Id: I90f14b88ea9fd4cf27e0c77817f15aa1d735a773

Merge "API: Add prop=sha1 to list=recentchanges"

Sync up with Parsoid tests

Change-Id: Id1d80a581564d0365c04953da4f41b086a2bc0d8

Merge "Localisation updates from translatewiki.net."

Localisation updates from translatewiki.net.

Change-Id: I943e34620f78a0af8ee078c918b14e3521957a12

Preserve returnto when upload requires logging in

Changes link text to loginreqlink message since
OutputPage->showPermissionsErrorPage passes that in.

Change-Id: Iebde4398cdb74eb2fcc441601f074c93a7ad6073

Merge "Allow uca-sv@collation=standard to be a collation name."

Allow uca-sv@collation=standard to be a collation name.

The "standard" collation for Swedish sorts V and W
only as secondary differences. Compare this to
the "reformed" collation which sorts them
as separate letters. Which collation is default
for sv seems to vary on icu version, but for icu 4.8
(which wmf uses) reformed is default. svwikisource
wants to use the "standard" collation.

Change-Id: I051590cf687ddea2e2cd84203d6e8eed3a6efd99

Merge "Add fa to collation list."

Merge "Further improve load order of RC and watchlist styling"

Further improve load order of RC and watchlist styling

This mainly addresses a regression in
I639a5ccbc3b7b2794e7b12ecb64e5ee6b10a640a, causing the bolding of
unread titles to be applied trough JS loading of the CSS.

Bug: 49610
Change-Id: I5cc20a342ebb566262e3aa1e9fe2cdf866a662c3

docs: Fix member index for mw.html

Method mw.html#element was indexed as being a member of
class mw.html.Cdata due to the lack of an explicit @member
tag, so it defaulted to linear association to the previous
@class block.

Quick fix can be done inline with @member but might as well
do it propert and separate the linear sequences.

Change-Id: I22580a6257a681a30e1ecd7c1283b57d7cb730f2

database: Improve DatabaseMysql::masterPosWait() performance by caching the position

* This is useful for things like loads of refreshLinks jobs that call wfGetLB()->waitFor().

Change-Id: I05a564a3b73999517fec2476ed946e092c16c3d4

Merge "Add user rights 'viewmywatchlist', 'editmywatchlist'"

Merge "Fix getPageDimensions failure handling"

Merge "benchmarks: README file having run recommendations"

Merge "Change autocomment-prefix to be blank. Looks weird being '- '"

Merge "DifferenceEngine: Don't display empty header row"

Fix getPageDimensions failure handling

getPageDimensions returning false for failure wasn't being handled
properly, causing ugly output. The doc comment on that method was
wrong as well.

Most notably causing random whitespace output:
https://commons.wikimedia.org/wiki/?curid=22151015
(screenshot: http://i.imgur.com/c21EpVx.png).

Bug: 41281
Change-Id: I1a49474309e15808928f877dfc29ae366d028928

Merge "Remove unnecessary clears"

Merge "Added SSL verification to PHPHttpRequest."

Merge "XML text dumps: skip rev length check for weird content models"

Remove unnecessary clears

This is a followup to Ibb9371ee.

Bug: 33438
Change-Id: I0b39138967f2d29d5e50db0a5990d175fcc81a8e

Merge "mwdocgen: support multiple --file values"

Merge "Removed some inappropriate global state from FSFile"

Merge "Include the backtrace in the log for job exceptions"

Merge "(bug 49840) Moving a page with subpages may lead in the page being moved twice"

Merge "CologneBlue: Use search input and button generators from SkinTemplate"

Merge "Doc fixes for MediaHandler. No behaviour changes."

Merge "Add broken-file-category for file links inside <gallery/>"

CologneBlue: Use search input and button generators from SkinTemplate

This also makes sure that the search inputs receive their placeholder,
which fixes bug 47475.

Bug: 47475

Change-Id: Ie1215078edb38f98ec57d2a3ec46b1d0ace0bc30

(bug 49840) Moving a page with subpages may lead in the page being moved twice

This happens if a page is moved to a subpage of itself and the option
"move subpages" is enabled, since the freshly moved page will be found
as being one of its subpages and moved a second time as such.

Bug: 49840
Change-Id: I318aab8c3c6e7ae718a3366ab00aba6e5420529b

Localisation updates from translatewiki.net.

Change-Id: I69f2c11c204026c01c582f5714e21cc89d8eb929

Add user rights 'viewmywatchlist', 'editmywatchlist'

These are needed for OAuth grants.

Note that, even if 'editmywatchlist' is not granted, various actions
will still allow for adding but not removing of pages.

Change-Id: Ie33446a228dd6ed0114730935c1bf65667f5ce01

Merge "Fix the error in html email when pear mail_mime is missing"

mwdocgen: support multiple --file values

It is sometimes useful to generate documentation for multiple files or
directories. Doxygen INPUT statement supports multiple space separated
paths. Example usage:

 php maintenance/mwdocgen.php --file docs,input

Change-Id: I66a805d80307c6502e9b440df63a37bb13093d84

benchmarks: README file having run recommendations

In a wikitech thread about benchmarking, Tim Starling has been running
benchmarks using CPU binding and raised priority. This patch copy paste
Tim recommendations so we do not forget about them.

Change-Id: Ic6f2803c005f70a2f0400f28fafc3c8d209e0667

test: coverage recording now needs to be explicit

To better control what we are really covering, this patch uses PHPUnit
forceCoversAnnotation option. That makes it to record covering if and
only if @covers is used.

Also rewrapped the options to uses tabulations.

Change-Id: I467a5077a65b93cce08072a8d088b0b9c652a426

Merge "Cologne Blue: Make sure the links don't bleed into the quickbar"

Merge "database: Simplify selectSQLText() and fix USE INDEX bug"

Merge "Fix history and add section tabs being collapsed on RTL wikis"

Fix history and add section tabs being collapsed on RTL wikis

The module is now loaded in the 'top' queue instead of the 'bottom'
one, so the <body> tag isn't loaded when the RTL detection is done.
Use the <html> tag's 'dir' attribute instead.

Bug: 50196
Change-Id: I6e277a9503a1e6003bc7bf9f9468ed7b35552e60

Include the backtrace in the log for job exceptions

Change-Id: Iff478f452142f1ccc9af70b4a64cfb66d04014f3

Stop spamming exception log with random upload stash failures

* Things like authorization and validation problems do not belong in the
  exception log.

Change-Id: I3c4c1bab65cfe9111dc2161bda6d023234112900

Avoid the "key conflict attempting to insert a user row" exception in some cases

bug: 41609
Change-Id: Iebfcf50e648a43f2970e6d0e4073cb6d84d260df

Merge "Localisation updates from translatewiki.net."

Localisation updates from translatewiki.net.

Change-Id: Ic90e20578bf9dfb77d2e0ebe648270b41c83f40a

Allow explicit & implicit null returns from hook handlers

Most hook handlers are written with the intent of complementing or augmenting
core functionality rather than vetoing it, making it quite natural for a
developer to forget that the caller is waiting for permission to proceed. The
potential for confusion is magnified by the fact that DOM event handlers and
jQuery event handlers are not required to return an explicit value for the
handled event to continue propagating.

This change tolerates null return values (both implicit and explicit -- that
is, both 'return null' and no return statement at all) from hook handlers. To
abort processing, a hook function must return an explicit false or an error
string.

This change should not break any existing hook functions, as returning null is
currently an error.

Bug: 50134
Change-Id: I11deb2117ff9233c77868470f50e0d8f74053545

Merge "(bug 50078) Allow a string other than '*' as condition for DatabaseBase::delete()"

Merge "(bug 50025) rebuildall.php script needs database admin rights"

API: Add prop=sha1 to list=recentchanges

People have asked for this in connection with patrolling recentchanges
for vandalism and such.

Bug: 48359
Change-Id: I5cab770c2dd740d586dd75d8795cbf4e3c1d05b7

(bug 50078) Allow a string other than '*' as condition for DatabaseBase::delete()

Currently, either '*' is given as condition meaning "everything" or an array must
be passed since DatabaseBase::makeList() requires an array. Now the parameter is
consistent with one of other similar methods, since a string will be handled
correctly.

Bug: 50078
Change-Id: Id5a8220d21245669f1091a3b5ed1def65b22d375

Move Search page from redirects to pages

Having it in "Redirecting special pages" seems wrong to me. It's better to
have it in the company of "Lists of pages" like Special:PrefixIndex,
Special:AllPages and friends.

Change-Id: I8c59275e64dce158cd7a5b027c7efbbdb941e4c4

(bug 50025) rebuildall.php script needs database admin rights

It calls rebuildtextindex.php, which use the ALTER command.

Bug: 50025
Change-Id: Ifbd1d1d1d9c0d6f53ce13b92d94824beee67963f

Adjust parserTests.txt for bug 43015

Change-Id: Ied95c87fda13dbea2b8c46ad1e96fde1c50c1517

Merge "Localisation updates from translatewiki.net."

Localisation updates from translatewiki.net.

Change-Id: I6088523647a98c8a79cb634bd531b25d85df60a9

Merge "Localisation updates from translatewiki.net."

Rebuild MessagesEn.php

Change-Id: I8c792c66f26020ad65efc38f1baf1a57950667f1

Add broken-file-category for file links inside <gallery/>

Bug: 50119
Change-Id: I7d06f48e53df43213bda52feec0d3f390e10f742

Merge "Added new + updated old tests for preventing regressions in Parsoid"

Added new + updated old tests for preventing regressions in Parsoid

Change-Id: I2963d8f2a91ec473eb7c9b063c4c5a4c2de8f82d

Merge "Improve help text for list=alllinks and alltransclusions"

Localisation updates from translatewiki.net.

Updates for namespace translations.

Change-Id: Ifc3452fa3617f27e0cbe5d05e37c358c3c3ceb81

Merge "API: Fix imageinfo iiurlheight on audio files"

API: Fix imageinfo iiurlheight on audio files

When iiurlheight is given without iiurlwidth, we want to limit the
thumbnail by just the height. MediaWiki's file classes don't really
support this, though, so we pass the image's full width as the limiting
width to achieve the same effect.

However, this fails for audio files where $file->getWidth() returns 0,
but we can still get a thumbnail of a placeholder image. In that
situation, we should just choose an arbitrary non-zero limiting width.

Change-Id: I4318a06a96265d39e39e90cc706d49a1c3b6e8e3

Merge "Capitalize URL in api help"

Fix the error in html email when pear mail_mime is missing

When pear package mail_mime is missing, attempt to send html email throws error:
mail() expects parameter 3 to be string, array given.  This is because the code
doesn't unset the html part of the body when falling back to text email

Change-Id: Ie8c73a481cb9003a0b43893cf4751475bdf5c690

Merge "Fix typo "relavent" in release notes"

Localisation updates from translatewiki.net.

Change-Id: I053c79ff1f095d49668e94a0e05a2b56e668a9ea

Fix typo "relavent" in release notes

Follows-up 8c328d030fc1f7f.

Change-Id: I7395d78b29ba4f754466ded6e0c3b14f35f92440

Localisation updates from translatewiki.net.

Change-Id: I6c67e5c688c85cb291255f67f8fe10c576474272

Capitalize URL in api help

As part of bug 34965

Change-Id: Ia1df9748b5de33f6ef5088103ccc6b886f9a17d5

Removed some inappropriate global state from FSFile

Change-Id: I48bc82eab486c119ed1ac7812d2da97490088cd4

Improve help text for list=alllinks and alltransclusions

This patch set changed some
- "link titles" to "linked titles"
- "transclusion titles" to "transcluded titles"

Change-Id: If536368e95ca5e160d8a69e832e67e3248b787fd

Localisation updates from translatewiki.net.

Change-Id: Ifa64f58104e4e0fdeafd137079676a33698bb229

Merge "Allow SearchUpdate hook to abort core update call"

Allow SearchUpdate hook to abort core update call

Going to use this in CirrusSearch since the default text
handling is insane for Solr.

While we're at it, further move content handling to SearchEngine
so children can override behavior here.

Change-Id: I09d11b81c224d53609c57d75d54021e697b56629

Merge "Docs and drop silly m prefix from private variables"

Allow modification of flags on Recent Changes

Moved the message map in ChangesList::flag to a new setting named
$wgRecentChangesFlags and used it keys to iterate over the flags
Also allow setting the class to deal with minor vs. minoredit

Also make ChangesList::recentChangesFlags public to be used from
extensions, when they will build its own line

Changed one list of placeholders to use recentChangesFlags to have the
correct number of spaces there

Change-Id: If69de05c4edec4e84c3e059090b9cb89845e3269

Merge "mediawiki.api.parse: Specify wikitext as the content model"

mediawiki.api.parse: Specify wikitext as the content model

The parse API defaults to use the content model of the title, which
defaults to "API" in the main namespace which can be configured to have
a different content model.

Bug: 48201
Change-Id: I51143873e3782df4df500194a7ac71c4b446746b

Change @since and @deprecated notes to 1.22

Using the following command line, I have found doc comments mentioning
"1.21" when they should mention "1.22" instead, which I have fixed
manually:

git diff REL1_21 | grep --color=always -C 10 -iE \
'^\+.*(since|deprecated).*1\.21(\D|$)' | aha > oldver.html

I also moved the release notes for I1987190f ("Combine JavaScript and
JSON encoding logic") from RELEASE-NOTES-1.21 to RELEASE-NOTES-1.22
because I had reverted the commit on REL1_21 only (see Id3b88102 and
bug 47431 for the rationale).

Change-Id: I11b917a371e07267dfa98b8449776d0c1cb29b15
Follows-Up: I25cf5a94f6e47f85a9d0b80cc1c9c9f957288478
Follows-Up: I3d72e4105f6244b0695116940e62a2ddef66eb66
Follows-Up: I3faa9c3e8107c6e46cdf21f8c18adda1f42890d7
Follows-Up: I6aab19c8d68bf47beddad42632b0360a7b12f251
Follows-Up: I86368821fc2cd0729df5342b8572eb470c0f77a0
Follows-Up: Id3b88102e768318e3605a19e9952121091a40915
Follows-Up: Ie667088010e24eb6cb569f9e8e8e2553005223eb

database: Simplify selectSQLText() and fix USE INDEX bug

* Previously, using a USE INDEX array with one table string resulted
  in the FORCE INDEX statement not appearing in the SQL.
* Renamed two variables in tableNamesWithUseIndexOrJOIN().

Change-Id: Id084db7df3b07d125ea73faca7e4506a58be5f35

mediawiki.js: Clean up and improve documentation coverage

Function addStyleTag was only documented as a private function,
since a while now it is also exposed as a public method.

Rename the private function so that they can both be included
in the JSDuck index (jsduck can't deal with name clashes between
public and private). Optimised by letting it inherit the
documentation instead of repeating it.

The following were missing documentation:
* mw
* mw.hook
* mw.legacy
* mw.loader#addEmbeddedCSS

Moved assignment of `done` in mw.loader#addScript to where it
was needed. The other if/else paths in that function don't use
it and don't need it to have a value.

With the introduction of mw.log#deprecate (fe46903) we no longer
need mw.legacy. mw.legacy was created for future use to move
legacy stuff into to make it easier to keep track of where they
are used (simple search for 'mw.legacy') however that never
happened and in retrospect was probably a bad idea as it would
require people to change code twice.

We now have a much cleaner solution now that we're able to
create little accessors on any host-object with mw.log#deprecate.
This makes mw.legacy obsolete and it should not be used for
anything. Just in case something is using it I won't remove it
just yet, instead I've deprecated the (what would have become
the) deprecator ;-). We should be able to remove this in 1.23.
Note: Can't use mw.log#deprecate here because that isn't defined
yet at this point, this is the core library which can't have any
dependencies.

Change-Id: I5d241e4284f07a6be271c04cec3a5833dbaeed91

Docs and drop silly m prefix from private variables

Change-Id: Ib2e5507e5735dd94423174f475e6a8e733f89fd7

Merge "displaytitle: Pass-by-reference fix for PHP 5.4"

displaytitle: Pass-by-reference fix for PHP 5.4

PHP 5.4 includes some pass-by-reference changes which apparently make
the '&' sigil necessary here.

Spotted by Mormegil <mormegil@centrum.cz>. Followup to Ie162535b.

Change-Id: I038fa7f8e1559c95dba7b9b498fc1d1f5eb76887

Return errors from WatchAction

Currently, WatchAction::doWatch and WatchAction::doUnwatch return true
always. Let's have them return a status object instead.

This also cleans up the handling of Status objects in some of the API
modules.

Change-Id: I9dd9f0fd499c37f29fa12bcdb6142238a1f11e4d

doc: speed up dot generation

dot supports generating multiple graphs in one run, that was not always
the case and Doxygen had a default of not using that feature.  Given it
got introduced in dot 1.8.10 and Wikimedia CI server has 2.26.x, I guess
it is safe to enable this.

End result, is less shell out overhead.

Change-Id: Ia07b430bd1af608bbe9c10eecbe2df3e0b9c74fc

doc: raise doxygen lookup cache

At the end of a run, doxygen emits a notice regarding the cache lookup
size being too small.  This cache is used to find symbols given their
name which would speed up processing a bit.

The notice before this patch:

lookup cache used 65536/65536 hits=2258467 misses=112595
Note: based on cache misses the ideal setting for LOOKUP_CACHE_SIZE is 1
at the cost of higher memory usage.

And after:

lookup cache used 104632/131072 hits=2131285 misses=105434

That is 7% less miss, not that much but still.  Note how the cache is
well filled (80%).

Change-Id: I0b97e8c5581a63c44a382891f8f1fc8d60975e80

Merge "Localisation updates from translatewiki.net."

Localisation updates from translatewiki.net.

Change-Id: I213cd86665ecdbd77e7504e9cd77eea9cbd15b07

doc: sync with doxygen 1.7.6.1

The Wikimedia CI server runs doxygen 1.7.6.1, this patch sync our
configuration with that version.

TCL_SUBST and LOOKUP_CACHE_SIZE have been added.
IDL_PROPERTY_SUPPORT default value changed, unused by us though

Change-Id: Ic43d99c4675eb8f1772a72a9b9bfaf16057e19f1

Merge "Fixed doc typo"