Improve OutputPage::showErrorPage method documentation

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

Change-Id: I81207f4ef46a43529974e3ada194f2948a1e9017

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"

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"

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

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

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"

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

Add test that covers Status::fatal

Change-Id: I7cb1e19569f77df5230808d7b35200b0ec9a6698

Add test for Status->merge with overrideValue true

Change-Id: I049c67e80277f8dbbbadae6e7a8f30d66066e7bf

Cleanup RefreshLinksPartitionTest

Change-Id: I9c8e66b7370f8ea8d9b4f8843718b29e992b1180

Merge "Improve organization of TestsAutoLoader"

Update jStorage

Contains bugfixes, new license and smaller file size: mw.loader.inspect
went from 16.6 to 11.9. This is still bigger than what they claim:
"jStorage is really small, just about 7kB when minified (4kB when gzipped)!"

Change-Id: I0b6f083a72a27af1e3fd367f61e8f98f74fea5d8

Add .gitignore to the /skins directory

To make it easier to work on 3rd party skins.

Bug: 55486
Change-Id: I986b6fe71bba429d1274bd86822448838601af64

Added some constants to speed up Setup.php

Change-Id: I528b362d6ef3d07c6c301e7a1767a1be0fc93725

Add support for "tabindex" in HTMLFormField subclasses

It is already set for some fields in Special:Block, but are
discarded by HTMLForm and its fields.

Some notes:
- fields with multiple inputs (radio, select and other, select
  or other) will have the same tabindex set on all elements
- Some items such as multi-select and check matrix are not yet
  implemented

Change-Id: I3e1ba7f16f3a3183f231afcf60dd392ce6d6eb6b

Removed "Justify paragraphs" from MW user Preference.

Removed the option 'Justify paragraphs' from MW Preferences
as it is not a necessary option there.
Added RELEASE NOTES.

Bug: 52810
Change-Id: I1fe6a5857070828726077e6ba229b786c017c858

Add direct links to translation interface on Special:AllMessages

The local Special:AllMessages page is the first and most friendly place
where average users will look for the message and try to translate it:
when they find it, add a link to translatewiki.net so that they can start
translating without searching for it from scratch.

Special:SearchTranslations is the only target usable for this: the solr
search is smart enough to present the most relevant results first and the
translation interface is directly available in place.
On the bright side, similar messages popping up from other projects or areas
of the code will provide suggestions and help find other occurrences of
translations which can similarly be improved.

More specifically, we are using a combined language, key and text search:
* the English text is the only thing guaranteed to exist, if the message is
  available, though quoting it can give some false negatives and not quoting
  it would give it too much weight if it's long;
* the key can be used thanks to the feature revealed by Niklas Laxström in
  comments to this patch: it is simpler to handle but allows only exact matches
  and can be wrong if TWN adds a prefix for the extension, so it can't be used
  alone, but in the best case it's what we're looking for;
* the language makes sense in combination with the key match.
We boost key and language to get the current translation of the message where
possible; if it's not, the first result will usually be English text.
https://wiki.apache.org/solr/SolrQuerySyntax
https://lucene.apache.org/core/2_9_4/queryparsersyntax.html

From the initial search, the user can further refine its search and navigate
the interface to find the message(s) and editor where to add or edit the
translation.

Change-Id: I8de3faba829c50cdc0f2d09726f738c585792e00

i18n: Native digits on #wpLocalTime

Change-Id: Ief3c4700247809e59dda30b9b224e3f9dfaa5e62

Merge "Little change of readability in Title::secureAndSplit"

Merge "StatusTest is language dependant"

Merge "More use of Title::hasFragment"

Merge "Add hints to two function comments in Title.php"

Merge "Replace <i> and <b> by <em> and <strong> in i18n"

Add hints to two function comments in Title.php

Change-Id: I47627892c6450396cfb658ddef7617046fdf4209

Little change of readability in Title::secureAndSplit

Move init of mArticleId to bottom, where other fields are filled and
change ternary operator to a simple if for better readability.
Break a double assignment in two statements.

Change-Id: Ib61435e62e883142dd0c5dcb40e0e50e7ca18439

More use of Title::hasFragment

Missing two places for the new method.

Also change a remaining var access of Title::mFragment to accessor.

Follow-Up: I27d9c3e40e6de6800f4488de167cf06e83c88ce6
Change-Id: I5d26b9a7662673b902e3f7147cbc95ca0626d721

Merge "Ignore testRunLegacyHooks in findHooks.php"

Merge "Remove preference "Show table of contents (for pages with more ...)""

Remove preference "Show table of contents (for pages with more ...)"

The table of contents box is auto-inserted, can trivially be hidden or
exposed on a per-page basis with __MAGICWORDS__, includes a sticky
[show|hide] link, and can be easily hidden with site-wide CSS as
necessary. It needlessly adds complexity and user interface clutter.

Bug: 52813
Change-Id: If2139317dae4aa980b373c73d7b81dac627b5af8

Replace <i> and <b> by <em> and <strong> in i18n

Change-Id: If84e78e413fd042ef9451c68df2321fd52a35c79

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

Localisation updates from https://translatewiki.net.

Change-Id: Idf89d2bff84d9c4e74be06a2532222891b0fb30e

Merge "Change behavior of Special:Watchlist when user's watchlist is empty"

Merge "Fix comment on JobQueue::push"

Fix comment on JobQueue::push

Change-Id: I2a7e570ee29ca9dcab2e5c92616ecb69310ae7d5