Merge "(bug 56184) Allow 3-way merge from arbitrary revisions"

Merge "Remove commented defines"

Merge "Make it a bit safer to paste input to eval.php"

Various fixes to job running code in Wiki.php

* Make it async on Windows
* Fixed possible "invalid filename" errors on Windows
* Redirect output to dev/null to avoid hanging PHP

bug: 60231
bug: 58719
Change-Id: If4e52b86fb3a1bff283dfba3dfac213367f47543

Merge "Renamed "removeDuplicates" field to be slightly clearer"

Merge "Special:PrefixIndex omits stripprefix=1 for "Next page" link"

Special:PrefixIndex omits stripprefix=1 for "Next page" link

Bug: 60543
Change-Id: I05c26db132d0f7edadb82a798fdba37277a3eddb

Merge "Add an SVG version of watch icon"

Documentation: Tweak documentation of MediaWiki UI buttons

Document big and block

Change-Id: Idebda1bca28f93d077ba77cbcbd1b1aee9e5f2ef

Merge "Update mediawiki.ui button styles"

Merge "SECURITY: Sanitize shell command args"

SECURITY: Sanitize shell command args

Add validation and sanitization to several code paths.

Bug: 60339
Change-Id: Id124281d21ec730a2e0bbace843dd97194a712b4

Merge "Add missing uploadstash.us_props for PostgreSQL"

Use master db for showing log extract on RevDel success page

On revision delete success, we show an excerpt from the log,
it is very confusing when that excerpt doesn't include the
action that you just did.

Bug: 57033
Change-Id: Ica79bf76243f7ab7a2a0fb40689c34bd2ffc0297

Renamed "removeDuplicates" field to be slightly clearer

Change-Id: I1dfeca54b928f20f4ea28724095d7dc6b907acf1

Merge "Save generated parser output to cache in RefreshLinks"

Merge "Remove $wgUseTeX for checking if editing tools should be shown"

Merge "Added a simple JobSpecification class for pushing jobs"

Merge "OutputPage::showErrorPage raises a notice if arguments are incoherent"

OutputPage::showErrorPage raises a notice if arguments are incoherent

The method has the following signature:
OutputPage::showErrorPage( $title, $msg, $params = array() )

$msg can be a string or a Message object.

If it's a string, a Message object is built, with $params as parameters.
If it's a Message object, $params is ignored.

The core now triggers a notice in the case a call is made with $msg an instance
of Message object, and a (non-empty array) $params argument is given.

Change-Id: I227a416f088fc1acd6a04345ed0e24d06f967ecc

Merge "Improve OutputPage::showErrorPage method documentation"

Improve OutputPage::showErrorPage method documentation

- Added the most usual case of call
- Fixed the incoherence between -Object and -Obj

Change-Id: I81207f4ef46a43529974e3ada194f2948a1e9017

Merge "ChangesListSpecialPage: Implement buildMainQueryConds()"

Merge "Removed "Justify paragraphs" from MW user Preference."

Merge "Made Special:Contributions includable"

Made Special:Contributions includable

This commit make Special:Contributions transcludable by changing
extended class SpecialPage to IncludableSpecialPage also it adds if
condition in execute function to avoid the inclusion of form when
Special:Contributions is transcluded into other appropriate MediaWiki pages.

Bug: 4365
Change-Id: I25c5cc3775b91fdc17c1209953d425c640e5bc77

Merge "Add oldRev parameter to DiffRevisionTools hook"

Add oldRev parameter to DiffRevisionTools hook

Add a third parameter to the DiffRevisionTools hook, containing the old
revision.

Bug: 57721
Change-Id: I6ee956c5b9901033f618d7e4d09f8f416f3d72cb

Merge "Propagate the favicon information to getInfo() for local repos"

Update mediawiki.ui button styles

Based on:
* https://www.mediawiki.org/wiki/File:Agora_specs_Buttons_annotated.png
* https://www.mediawiki.org/wiki/File:Guide_Color.png
* http://33cc77.com/wikipedia/ by Daniel Zarick

This has quiet buttons that all start gray, and only reveal
constructive/progressive/destructive color on hover and active.

Also:
* Update KSS docs to reflect disabled state.
* Remove unnecessary text-decoration: none rules (the one in
  a.mw-ui-button should be enough).

Bug: 58297
Bug: 54360
Change-Id: I1dd9bfac65092ad80bb8cd410031c22da0a18305

Merge "Split Title and User ArrayFromResult classes to own files"

Localisation updates from https://translatewiki.net.

Change-Id: Icb726976f02aaed49f1b1e83a429c6a05c8ae103

Use PHPUnit filter/whitelist to select the files that need testing.

This has the effect of excluding the tests themselves from the code coverage
and including files that no test loads.

Change-Id: I65674ca3569d1ae6484b6769366213d71c0bdbc4

Merge "(bug 47070) check content model namespace on import."

Merge "Remove math specific code from ParserOptions"

Merge "Removed version check for mysql 4.1.0 from SqlBahOStuff::createTables"

Merge "Removed unused postConnectionBackoff() from LoadMonitor"

Removed unused postConnectionBackoff() from LoadMonitor

* This setting proved to be not terrible useful

Change-Id: I58b18947ff0edb8937229d0f130d7d10e846125e

Merge "Remove setting referrer from HttpFunctions"

Added a simple JobSpecification class for pushing jobs

* Both this and the full Job class can be used to push jobs

bug: 60403
Change-Id: I7e78321b5919e48fd8228580ddde7c90a6e4024e

Merge "Avoid variable naming conflicts in DatabasePostgres::selectSQLText"

Merge "Correct sequence name for fresh Postgres installation"

Avoid variable naming conflicts in DatabasePostgres::selectSQLText

Spotted by gebhkla on bug 60531

Bug: 60531
Change-Id: Iaf100ea0db05b142549bd98da108e48eae8b9101

Ignore certain tidy warnings in assertValidHtmlDocument

Those aren't really useful, especially as tidy often
cries about parameters missing which have been
deprecated since HTML 4, so that these warnings have
no value for us.

Change-Id: Ic27c597aa988079ed08e152861bf1dee9581b829

Removed version check for mysql 4.1.0 from SqlBahOStuff::createTables

Presently mysql v5.0.2 or later is supported.

Bug: 59125
Change-Id: I9043a6f792c6479a85411addbe9fd448ed5e02a6

Merge "skin: Remove the id "anonlogin" and merge pt-anonlogin with pt-login"

Merge "Config: Add Config and GlobalConfig classes"

Config: Add Config and GlobalConfig classes

Allows configuration options to be fetched from context.

Only one implementation, GlobalConfig, is provided, which
simply returns $GLOBALS[$name]. There can be more classes
in the future, possibly a database-based one. For convinience
the "wg" prefix is automatically added.

Ironically, this adds the $wgConfigClass global variable
which is used to determine which implementation of Config
to use by default.

The ContextSource getConfig and setConfig methods were introduced
in I23194d1ba (1.23), but have no uses in Gerrit, so they can safely
be re-purposed.

Change-Id: I13baec0b6d4ea7badf20b9c5f9b40846348838e4

Don't include empty categories in Special:Mostlinkedcategories

We never delete things from the category table, so there are
entries in it with 0 members, which really shouldn't show up
on the special page.

Bug: 58967
Change-Id: I6bdf5a5196724f7970fb79497ef912cf8268ec3e

Save generated parser output to cache in RefreshLinks

* This only applies to non-fast parses to avoid cache churn

Change-Id: If3763811a3aa6360799a10ff64a725f17d8cee6d

skin: Remove the id "anonlogin" and merge pt-anonlogin with pt-login

This was used as an alternative id for the login link in the
personal tools at the top right for ShowIPinHeader context.

This seems to have been inherited since the days of the
PHPTal-based skins (at least git-blame shows their presence back
in Brion's rewrite of the skin in november 2004).

I've asked several maintainers about this and nobody could figure
out why it would need a different id. After all, whenever the
user sees the login link it means they are using the wiki as an
anonymous user.

My best guess is that this was meant to allow the ability to
style the login differently when the (anonymous) user has
already made an edit (or if we init a session for other reasons).

This is also what toggles the anon userlinks are shown (i.e. the
links to the IP userpage and talkpage), but I see no evidence of
this, and in any case it would be better to change the class of
the whole personal tools div instead of changing the IDs.

Change-Id: Ib3445934853db5361c3c6fecfe75623b0b921400

Merge "Release notes for I9359682c"

Merge "Make preference "Email me when a page or file on my watchlist is changed" true by default"

Release notes for I9359682c

Forgotten earlier.

Change-Id: I8e2cad1bfb5f98916baaba104e3a34c8abd51dad

Remove setting referrer from HttpFunctions

It was introduced in 5d210b50 but no callers have ever been updated
so this always falls back on $wgTitle (which is usually wrong). It
only works on cURL, and is marginally useful at best.

Change-Id: Ic498108af9f466cefbd0072f94f0efa4055cd169

Ignore 'unknown' in XFF

Can come from Squid
http://www.squid-cache.org/Doc/config/forwarded_for/

Bug: 57467
Change-Id: I3fd3170d31a1aff2e748302c30dd6e364ed3bdcd

Make preference "Email me when a page or file on my watchlist is changed" true by default

Consistent with the recent work in making vanilla MediaWiki closer to what a standard site
would do, with more obvious defaults, and bug 45020.

WMF can sort out its special problems separately and is not affected by this, see Iad6d13509.

Bug: 45022
Change-Id: I9757cff9728f377363c540b1ab809679a1bfa01d

Remove commented defines

Change-Id: I0447646127e05d80c87db5b88942105d41d802bd

Remove IE80Fixes.css

Reverts e30edaf0 (r62191).

Bug: 60237
Change-Id: Iba648a8bd6e917cec2e9906a6e412aa4d538cb60

Merge "Use box-sizing:border-box for textarea"

Correct sequence name for fresh Postgres installation

Spotted by gebhkla on bug 60083

Bug: 60083
Change-Id: I34283c2539f4ce9d1b85756b3e1c16d255000ed5

Put expensive info action count behind $wgMiserMode

Change-Id: Ie0507b9ed62556f8caaaaf9499a4a40b3342b5af

Merge "Always return an array from SwiftFileBackend::addMissingMetadata"

Always return an array from SwiftFileBackend::addMissingMetadata

Correct the behavior of SwiftFileBackend::addMissingMetadata to match
the documentation and calling code expectations by always returning an
array.

Bug: 60477
Change-Id: I8d12888a6ebfa23502f8cf3a28866dcb00a18ec8

ChangesListSpecialPage: Implement buildMainQueryConds()

This also involved adding some default options to getDefaultOptions()
and a new method getDB().

Change-Id: I7389a72bfcf176480bfc36f9d6efc467e1a5e76a

Don't override limits and offsets in SearchEngine::create()

The class default is 10/0, it doesn't make sense to set that
as 0/0 here. Nor does it make sense to really override it
here at all, as class defaults should take precedent unless
explicitly overriden by callers.

Bug: 60475
Change-Id: I4a48fd3c55908b52c8b0ab081d856874d16b876e

Merge "Fix phpdoc in SwiftFileBackend"

Fix phpdoc in SwiftFileBackend

Docs for $status and $code were swapped.

Change-Id: I88bd81b671f0e9a4176483fb744e45f2477672b5

Create jquery.fullscreen

New plugin for making elements go fullscreen.
Incorporates some code from MultimediaViewer.

Bug: 56477
Change-Id: I58138afa05c43f5a24e74fa3a30a658428c39ce0

Merge "Localisation updates from https://translatewiki.net."

Merge "Initial Tests for TitleArrayFromResult"

Merge "Fix rebuildall.php fatal error with PostgreSQL"

Merge "Remove unused globals"

Merge "ChangesListSpecialPage: Implement execute()"

Merge "No longer display the number of rows shown on Special:Watchlist"

Remove unused globals

Change-Id: Id1c27653650f1d60892a9fddf4f6fa5dd534615f

ChangesListSpecialPage: Implement execute()

Merging common content from SpecialRecentChanges and SpecialWatchlist
plus some cleanup.

Change-Id: Ic4bbedf2015a9f20a6e63ec53a72df91f1e3a2fe

No longer display the number of rows shown on Special:Watchlist

My rationale for doing this is that passing this several functions
down is awkward and arguably bad design.

It's still a better rationale than the one used to add this in 2004,
which was replacing another value to fix HTML injection bug (r4728).

Change-Id: Iada3a93762dd25fdaf35fb707304abc40bd4a5b1

Fix typo: "oaram" --> "param"

Change-Id: Iff06f32c05cc83657770857b48d6201d1285eb07

Merge "Add initial tests for UserArrayFromResult"

Split Title and User ArrayFromResult classes to own files

Change-Id: I8744627330eb9c8f183a8d418d6eb168e6c5407b

Initial Tests for TitleArrayFromResult

Change-Id: I18635ed25a09e4646f68db4678179d437a700659

Add initial tests for UserArrayFromResult

Change-Id: I92498011be44cd765cf336191810f47b61a0fd7e

Fix rebuildall.php fatal error with PostgreSQL

The fix for 47055 introduced a fatal error when running rebuildall.php. This
is a workaround suggested by gebhkla on Bugzilla (idle for 10+ days). It just
checks to make sure $options is actually an array before calling array_search
on it.

Bug: 60094
Change-Id: Ib54420b5a749c60b82a4757a9f5fa511f48fdb72

Merge "Expand MWTimestamp tests"

Expand MWTimestamp tests

Change-Id: I00215ecd85b7403a05a75d5eaccbbd2819343624

Localisation updates from https://translatewiki.net.

Change-Id: I8187963fdb3b2cb88ee7afc1cc3b32cf8672e20b

Merge "SpecialWantedcategories: Actually strike the category if it was emptied"

Merge "Add Special:Diff as an internally-linkable redirect to diff pages"

Remove $wgUseTeX for checking if editing tools should be shown

The EditPage toolbar used the global $wgUseTeX to determine, if
the icon for inserting math should be shown. Now we simply check
if the class MathRenderer exits.

Change-Id: I0a0203f2d8c295bfc56694eb08c9986420c086cf

Merge "+Test for Status->CleanParams with a callback"

Merge "Add test that covers Status::fatal"

Merge "Add test for Status->merge with overrideValue true"

Merge "Cleanup RefreshLinksPartitionTest"

Remove math specific code from ParserOptions

There is some math specific code in ParserOptions.
To allow extensions like math to change the cache key in a reasonable matter,
they need to get access to the information which options were used. Therefore,
optionUsed is public now and the additional argument $forOptions was added to
the PageRenderingHash hook.

Bug: 60234
Change-Id: Ieecb27216c39b7e6e354b4a1904fbff06506307d

Merge "Add release notes for removed RdfMetaData class"

Add Special:Diff as an internally-linkable redirect to diff pages

This is similar to Special:PermanentLink added in r79036 and has been
asked for several times in different places, including:
- on the English Wikipedia (oldid=539308532)
- on mediawiki.org (lqt_oldid=31691)
- on the French Wikipedia (oldid=93029892)
- on the English Wikipedia again (oldid=588408888)

A notable use-case is linking to diffs in the edit summaries, where
external links are not yet allowed (bug 14892).

All of the following are valid usages:
- [[Special:Diff/12345]] (diff of a revision with the previous one)
- [[Special:Diff/12345/prev]] (diff of a revision with the previous one as well)
- [[Special:Diff/12345/next]] (diff of a revision with the next one)
- [[Special:Diff/12345/cur]] (diff of a revision with the latest one of that page)
- [[Special:Diff/12345/98765]] (diff between arbitrary two revisions)

Co-authored-by: Jérémie Roquet <jroquet@arkanosis.net>
Co-authored-by: Bartosz Dziewoński <matma.rex@gmail.com>
Change-Id: I77fdaf8e04375caa1d67ca4a3ec3bd93920c3309

Add release notes for removed RdfMetaData class

Follow-Up: I3454d605a741bc97a51a1ad471a17eb3053d662f
Change-Id: Id1a04ea330da90ed84dae692bde7c64a5ac06cd5

Split Timestamp.php into class and exception files

Change-Id: I438adfe5479a1017baee8f2b663f3fb2e49c685a

+Test for Status->CleanParams with a callback

Change-Id: I4e7570ea041d28303388c630b3ca07225faf8551