Drop support for XHTML 1.0

* $wgHtml5 = false; is now ignored completely.
* $wgDocType and $wgDTD have been removed.
* $wgXhtmlDefaultNamespace is now ignored.
* XHTML5 will be output if $wgMimeType is set to an XML mime type (according to HTML5's rules).
* For backwards compatibility with extensions $wgHtml5 and $wgXhtmlDefaultNamespace are set
  in Setup.php but depending on them is deprecated.

Change-Id: Iad9634e2ee420b5a3bbffe550421fde4fa1819b0

Revert "(bug 46615) updateCollation.php: sanity check the collation before proceeding"

Sorry, forgot that method was not in the base class, and I had only tested with uca based collations. This breaks on uppercase type collations.

This reverts commit 6eb84144df605feb717efc91c84c00eaec62a5f6

Change-Id: Ib7b9597ff842a76185ba5c153922834ffb741237

Merge "Localisation updates from translatewiki.net."

Merge "Fixed total breakage of HashRing in 9d1cd9a."

Fixed total breakage of HashRing in 9d1cd9a.

* This had PHP warnings but no errors, but was broken.
* Added some unit tests for this class.

Change-Id: I7c7ea2c30ca747aae7a3c3951fc33c69052de085

Fixed bug where FORCE INDEX did not make it to the SQL.

Change-Id: Id19d7b2e9453faf9031a7409329376b98a2fbb17

Localisation updates from translatewiki.net.

Change-Id: I9428a80e060d8c544ed0dc342fa198a7cd3dca4d

[JobQueue] Added a function to delete all jobs from a queue.

Change-Id: Ibad122148bdd2f7baf528929e15bae803fccfeea

Merge "Localisation updates from translatewiki.net."

Merge "[JobQueue] In addition to flushing any transaction, be sure to avoid new ones"

Localisation updates from translatewiki.net.

Updates for magic words translations.

Change-Id: I964a572cb36c9eca7c23f98c46148bdf5b82b2cb

Merge "Fixed bug in LocalFile::isCacheable()."

Localisation updates from translatewiki.net.

Updates for special page aliases.

Change-Id: If62552c379f34c6298df0e5144c85c08e41a9a7e

Do not bold all entries of an unvisited page on the Watchlist

When a page is unvisited (wl_notificationtimestamp IS NOT NULL) all
entries of that page on the watchlist gets bold (class=mw-changeslist-line-watched),
because wl_notificationtimestamp is only checked for null.

wl_notificationtimestamp should be compared against rc_timestamp to
find only the really unvisited pages since the last visit on that page.

It is confusing, when all entries of a page on the watchlist gets bold,
but only some of the newer are unvisited.

Support for old and enhanced watchlist.

Change-Id: I7265141a6c84bcbdd7b651bd2983a08b73c6ddfe

Merge "Clean up sublime ignores"

Merge "Made DatabaseBase::getSoftwareLink() dynamic."

(bug 41337) Fix fatal error in Special:Preferences

The error is Fatal error: Call to a member function msg() on a
non-object at includes/Preferences.php on line 1207.

The problem is that fields created in Preferences::getPreferences() for
validation don't have their parent set and thus an error occurs when the
validation fails since they want to use their parent to get a Message
object.

This commit adds a dummy parent object to these fields to fix the error.

bug: 41337
Change-Id: I5826d6e3f1262c8d26af0cfe7074a939f80bcaca

Merge "Fix inconsistent docs"

docs: Remove odd colons after @todo

Most were this way already:
https://doc.wikimedia.org/mediawiki-core/master/php/html/todo.html

Ran a find/replace on the odd ones. Also made them all
lower case.

Change-Id: I70c6a69344ddebc603e9a1c1d87e3cc4f4f4c560

Made DatabaseBase::getSoftwareLink() dynamic.

* All callers are calling it this way and it breaks
  hhvm when declared statically and called dynamically.

Change-Id: I894e615fd828615384aa8457a16a759c8aa416ef

[JobQueue] In addition to flushing any transaction, be sure to avoid new ones

* This probably only matters when jobs are not run via CLI scripts.

Change-Id: Ie04c8baac218f5df770388e17096dcb4f2eb9cba

docs: Consistently use "since" phrasing in @deprecated

Most have it already:
https://doc.wikimedia.org/mediawiki-core/master/php/html/deprecated.html

Ran a find/replace on "@deprecated \d" to fix the odd ones.

Change-Id: I5217aaf75264a0036b5426cd8536116b4bb8f535

cleanup: variable referenced without initialization

declared so that IDE knows how to use it and initialized

Change-Id: I4841fd3f03220f837c981951f07c1c21ddbd76af

Merge "Remove the space from between headline and its section edit link"

OutputPage: Guard jQuery.ready() against absence of jQuery

We already wrap usage of global "mediaWiki" in a condition
for window.mw (see method
ResourceLoader::makeLoaderConditionalScript) because:
1) The startup module blacklists certain
   browsers in which we won't load jquery+mediawiki.
2) It might have failed to load (for whatever reason).

Adding guard for window.jQuery for the same reasons.

Follows-up Ic3d0c937268d09, which caused a TypeError
'ready of undefined not a function' in browsers where
jquery+mediawiki isn't loaded by the startup module.

Change-Id: I9dcd8d347c6b00efe207d031a480e5b85bf78936

Merge "add basic print styles to CologneBlue"

Update messages.inc and rebuild MessagesEn.php.

Change-Id: Id8fdb78452da385f364baf8b886c6e1276fd8a9c

Localisation updates from translatewiki.net.

Change-Id: Id1c3618e6ce426dd5ecb73253b56badef06b366c

Fixed bug in LocalFile::isCacheable().

* The metadata field is actually defaulting to "" not null.

Change-Id: I745441c7061494db26de7f64dda30b80171c9443

Fixed LocalFile::isCacheable() to account for object type metadata.

Change-Id: I8104201c84ea008446df587650a098f6732b8d39

Fix typo in RELEASE-NOTES

Follow-up a2c6734 (I42be68c6a0): a "is false" was missing.

Change-Id: Ia81fbcded13c2da4354f1dce383bf5b854b99e49

(bug 41545) Allow kbd, samp, and var to be nested.

HTML5 has various semantics that allow -- or rather require --
<kbd> and <samp> and even <var> to be nested.

eg: <kbd><kbd>Shift</kbd>+<kbd>F3</kbd></kbd>
eg: <var>x_<var>i</var></var>, <var>y_<var>i</var></var>

This fixes the sanitizer to permit their nesting and adds test cases to ensure that some
of HTML5's special semantics are permitted by our sanitizer and not broken.

Change-Id: I6ad64e6eb4c9b5bdc15be513f55c58f6717c3939

Clean up sublime ignores

Change-Id: Ia7e22c2219040c7137a0f4ddc1dbb8ec1861a2d3

Merge "Clear stat cache before determining local copy error message."

Fix Qqq link for userlogin-resetpassword-link

* ResetPassword redirects to ChangePassword, which doesn't work logged out
* PasswordReset is the actual "forgot my password".

Change-Id: I4b9a7c3fe80feb8d2a8e4fe3be1238d2cc55d348

Clear stat cache before determining local copy error message.

* This makes it clearer if the listings are just wonky.

Change-Id: I78078fb9fac88b17e010b0eb990dbbeb83ea48f1

Merge "Localisation updates from translatewiki.net."

Merge "Strengthened validation for HashRing a bit."

Merge "Use Html::element() to build the links used in 'alphaindexline' message"

Merge "Remove "will be removed in ..." comments"

Merge "jquery.tablesorter.test: Reset transform tables in mw.config"

Localisation updates from translatewiki.net.

Change-Id: Ic2a29f252b16e103c55841e39935c74aabcc4532

jquery.tablesorter.test: Reset transform tables in mw.config

Having $wgLanguageCode set to "de" will pre-populate the transform tables
in mw.config for German, but the English transform tables are expected
(which are empty) for the jquery.tablesorter tests to pass.

Change-Id: Ief16ff9d0767c6c9332bc4e56c96be3d7d4f6ac8

Merge "Disambiguate buildForm docs, add TODO"

Merge "Whitelist a bunch of url protocols."

Strengthened validation for HashRing a bit.

Change-Id: Ib0331662c785d806063ca730905dd88b6545d8f0

Added wiki ID to backend copy script errors.

Change-Id: Ibe1fc6037d20f6669578295b6db62a4e414ee6b7

Disambiguate buildForm docs, add TODO

Xml::buildForm needs some love. I've given it a better documentation
string, but left a TODO for accessibility: We need to use labels for
input labelling, not table elements.

Change-Id: I4a9ff174e83c8ca3242c82bbe7cb0da0b8be7785

(Bug 5798) Prevent overflow in the footer when there are no links

Simple pure CSS solution to a minor overflow bug.

Change-Id: I337bab53600efb4cb27b65b01d62f59968a4fba9

Whitelist a bunch of url protocols.

Our url whitelisting is just to reject unsafe protocols like javascript:.
We have no reason to reject a bunch of urls to open standardized schemes.

Whitelist a bunch of them:
ftps, ssh, sftp, xmpp, sip, sips, tel, sms, bitcoin, magnet, urn, and geo.

Change-Id: I941190203ee1442d912d46144584bf2e7733f32c

Localisation updates from translatewiki.net.

Change-Id: I7805de9f5046bf94f0800d07564548711373e2e0

Added missing continue to avoid fatal error.

Change-Id: Ibb87c9e59e424fa84d662bb42349085d186dcc99

Merge "(bug 34666) Escape URLs in XML files created by generateSitemap.php"

(bug 34666) Escape URLs in XML files created by generateSitemap.php

Bug: 34666
Change-Id: Ifb6ddb0bc6ca03c411f938837b2f89a5a30e4fc3

Change default importImages comment to "Importing file"

Change-Id: Id65de095d7d7b697092a369f429b9e3b171d3e38

Merge "Variable $wgRestrictionLevels unused since 1cbaa921"

Variable $wgRestrictionLevels unused since 1cbaa921

Change-Id: I251ecaf319b2b4379ef80417ba23022a56064a94

Merge "Fix unit tests when $wgEmailAuthentication is set to false"

Merge "Made copyFileBackend.php skip files that cannot be copied instead of stopping."

Made copyFileBackend.php skip files that cannot be copied instead of stopping.

Change-Id: Ib15efdd35f38adefe3db500b4b6e5a71ea1858c7

Add $wgCascadingRestrictionLevels

A page can only be protected with cascading protection if the
requested restriction level is included in this array.

This replaces previously hard-coded values of 'sysop' and 'protect'.

This is necessary, because if any protection could be
cascading, users could who cannot normally protect pages could
"protect" them by transcluding them on protected pages they are
allowed to edit.

Bug: 47617
Change-Id: I5f8bcc899b46d466161894606cd27bf3b8624bd0

Fix unit tests when $wgEmailAuthentication is set to false

Since d2a5cf3 (I0b906b23de), 'emailaddress' and 'emailauthentication'
fields don't have any more CSS classes when $wgEmailAuthentication is
set to false which is breaking the tests.

Force $wgEmailAuthentication to true, so that the tests work in all cases.

Change-Id: Idc156f88ff1bc8595009056166f13191cf5c5c25

Remove "will be removed in ..." comments

All of them are wrong, since 1.20 is out for a while now, and
these methods are still here.

Change-Id: I3b59b61490c9aea5252e25623b0b282295dcc3f6

Localisation updates from translatewiki.net.

Change-Id: I32706ffc47ccd85835a8a0ee1e716648b054854b

Remove the space from between headline and its section edit link

I kept it in I6a6c12a9, but really there's no reason for it to be
there, as the link is delineated with square brackets and visually
with a 1 em padding. It only causes funkiness when copying text.

Change-Id: I7bde0e4fd646108f9319d5d5d00e7ea49fee9252

Don't shown red box around unconfirmed email address in preferences if authentication is disabled

Susan and Base-w in #wikimedia discovered that a red box is always shown around
their email address on foundationwiki. This is because the box is shown by
default - even when email address authentication is disabled.

Change-Id: I0b906b23dec6018bc21a179e2cf950b705100c4c

Merge "Message changes for new login form"

Message changes for new login form

Shorter "Reset your password" (new key userlogin-resetpassword-link).
Shorter "Use secure connection" (existing key userlogin-signwithsecure).
Better explanations.

Change-Id: Ia8e4219b200c25a61cbbade36841bb2c45cc9704

Fix --file parameter description in MediaWikiPHPUnitCommand.php

Change-Id: Ie134c06943be38cd65f8dae54332d38278ea0f96

Merge "[FileBackend] Moved short-circuit check from 6b1194b9 up to FileBackend."

Add identifying info (img_timestamp and img_sha1) to upload log.

I came across people complaining that it was hard to associate
upload log events to actual images since img_timestamp could
be different from log_timestamp, and generally no unique id.

Well I was there I made uploads use the new logging system.

Change-Id: Icd8662ecb9eb0f6c0ff9841bdbd5736d6dd0d015

[FileBackend] Moved short-circuit check from 6b1194b9 up to FileBackend.

Change-Id: I57566a1779f353c18d96c59c47112c753aea6b90

[FileBackend] Optimize the case were no operations are provided.

Change-Id: Iaf858ce2e8b21e031d27b984029f0424518201af

Merge "[FileBackend] Cleaned up resyncFiles() and added conservative resync option to FileBackendMultiWrite."

Merge "Localisation updates from translatewiki.net."

Add value to add() call when locking

When BagOStuff::lock() was called, it called $this->add() with only
two parameters, the key and the timeout. But BagOStuff::add() takes
three parameters (key, value, timeout), so all locks got an infinite
timeout.

Change-Id: I82bed11b0b799f2cda13a8a0bd0cd94908b6ce8e

Localisation updates from translatewiki.net.

Change-Id: Iae8a69a2054039f19fe1ae5de1e06b4d7fe51bb5

tests: mock parser tests file access

I eventually got tired of our parser tests creating and deleting fixture
files over and over.  This patch mock the files in memory and just
expose the file metadata which is all we need for parser tests.

The mocked classes are under /tests/phpunit/mocks/ and respect the
hierarchy of /includes/.

The wiki.png and headbg.jpg files are still copied on each test :/

Change-Id: Iccdff67222e66d48d01dd1596d09df2ea24b8c2a

Merge "Increasing z-index of personal toolbar from 1 to 100"

Merge "Removed unused join array from ActiveUser conds."

Merge "Change EXIF to Exif"

Change EXIF to Exif

Per https://en.wikipedia.org/wiki/Exchangeable_image_file_format. Spotted
by Shirayuki and documented on
https://translatewiki.net/wiki/Thread:Support/About_MediaWiki:Sesp-property-exif-data/en

Change-Id: I92ba67ec60ccfe7a173d950593357b86792b8ed3

CREDITS: Spell my name right

That's been bugging me for a while.  :-)

Change-Id: If543ac24ee36fdd913ac2e2839a282a677d90152

Removed unused join array from ActiveUser conds.

* The table was not in the table array so this did not affect
  the SQL, but was still unused cruft.

Change-Id: Ia5aa934fb6e6de9daeb0072d902df6e681575fd7

Increasing z-index of personal toolbar from 1 to 100

Since the personal toolbar is at the top of the page and may include
children that need a high z-index, such as dropdown menus or overlays
we should set it to at least be equal to the highest potential
competitor (the Visual Editor toolbar).

The other option would be to not set a z-index at all, but this would
regress bug 37158.

Bug: 48078
Change-Id: I61daea50209d040f7e0f71207a4e90998c15083f

Merge "Fixed spacing"

Merge "Localisation updates from translatewiki.net."

Merge "echo is not a function"

Merge "Add attributes to the Parsoid-only tests."

echo is not a function

Removed parenthesis after echo

Change-Id: Ia533aedf63b11d15dcc6a5cf75a56134a4b11d86

[FileBackend] Cleaned up resyncFiles() and added conservative resync option to FileBackendMultiWrite.

Change-Id: I4fd96fda71fc1aba44104241a8329e95ba5cdda7

Localisation updates from translatewiki.net.

Change-Id: I454062bca563af5a95bf57ada92a113532a7ed59

Add attributes to the Parsoid-only tests.

Needed to fix these up in order to get them running right when the Parsoid
test-runner begins to let through more attributes.

Several tests are make wt2html/wt2wt only because they involve templates
which we can't convert from html2wt without additional information.
The "SOL-sensitive wikitext tokens" test was previously wrong: the PHP
preprocessor generates lists from the template content.  Updated it to
the proper results while adding attributes, although it will currently
fail until we either fix the native Parsoid preprocessor or tweak our
parserTests to use the PHP preprocessor.

Co-Author: Mark Holmquist <mtraceur@member.fsf.org>
Co-Author: C. Scott Ananian <cscott@cscott.net>
Change-Id: I657e4869fb0c2ae66d6732d28c6fc6645ad8e534

print is not a function

Removed parenthesis after print

Change-Id: I1343872de7aa7c64952a3d86a63aaa091e46bda3

Fixed spacing

Added spaces around some parenthesis

Change-Id: If1e50e2a7b0046e91c0bbce8bf6641d8b2446a1d

[FileBackend] Added "adviseStat" option for the "listing followed by stat" case.

* Used this parameter to speed up copyFileBackend.php.
* Also added mtime checks to copyFileBackend.php and a few cleanups.
* Also fixed some incorrect getDirListPageInternal/getFileListPageInternal docs.

Change-Id: I424ef238f7adf4cf1f33b74e3a4e187dcb328a99

Merge "phpcs: Fix WhiteSpace.LanguageConstructSpacing warnings"

Merge "Refactored Special:ActiveUsers queries to remove filesort/temp table."

Fix inconsistent docs

Change-Id: I4a8d938b4e646ab4401fdf2dbd100401a415b32b

phpcs: Fix WhiteSpace.LanguageConstructSpacing warnings

Squiz.WhiteSpace.LanguageConstructSpacing:
   Language constructs must be followed by a single space;
   expected "require_once expression" but found
   "require_once(expression)"

It is a keyword (e.g. like `new`, `return` and `print`). As
such the parentheses don't make sense.

Per our code conventions, we use a space after keywords like
these. We appeared to have an unwritten exception for `require`
that doesn't make sense. About 60% of require/include usage
was missing the space and/or had superfluous parentheses.

It is as silly as print("foo") or return("foo"), it works
because keywords have no significance for whitespace between
it and the expression that follows, and since experessions can
be wrapped in parentheses for clarity (e.g. when doing string
concatenation or mathematical operations) the parenthesis
before and after basiclaly just ignored.

Change-Id: I2df2f80b8123714bea7e0771bf94b51ad5bb4b87