Make linker warning useful

Follow-up to I4c397de289ae51bc463b5ae

Change-Id: I4695172b13988fc333707e09fc9d909f20adb7c1

inputs.less: Add disabled state

Bug: 71149
Change-Id: Ia97dcb26bdae65a681f3d33b2e32d59831a55191

Merge "Begin logging RequestContexts relying on global title"

Merge "Special:ActiveUsers: Use mw-ui-input-inline"

Merge "Break accidental references in Parser::__clone"

Begin logging RequestContexts relying on global title

Change-Id: I72c5c3fb75c9ceff02390c91b3735f2da90845de

Merge "Remove $wgTitle from WebStart forever"

Merge "Set site logo url in ResourceLoaderSiteModule instead of inline styles"

Remove $wgTitle from WebStart forever

Change-Id: Icdd091f81547b76a403fc78f2e832ef3e85a67a2

Special:ActiveUsers: Use mw-ui-input-inline

Bug: 71142
Change-Id: Ie737b8bd74179ce916747c5e9cd23304e2ad8b16

Localisation updates from https://translatewiki.net.

Change-Id: Icba92059caac149e239ee1a90d962dbddd7e5605

Merge "mw.Title: Add method to get title relative to an arbitrary namespace"

Set site logo url in ResourceLoaderSiteModule instead of inline styles

Requires skins to use the 'mw-wiki-logo' class on the logo element
for the styles to apply. This is backwards compatible with skins
not using that class and using inline styles instead.

Bug: 56257
Change-Id: I89e32fcd27c9f07e889aa5b67d7d97d06b412aea

Merge "Delete maintenance/purgeOldText.inc"

Merge "Add maxlength of 6 for time correction in preferences"

Break accidental references in Parser::__clone

If you have a reference *to* an object field (anywhere in the call
stack) when you clone the object, the field will be cloned as a
reference rather than as a value.

So we have to break those unexpected references in the cloned object
manually, which is easy enough by making a non-reference copy and then
rebinding the cloned object's reference to this copy.

Bug: 56226
Change-Id: I9c600e9c0845b4fde0366126ce3809d74e2240b4

Delete maintenance/purgeOldText.inc

The only function it contains (PurgeRedundantText()) was superseded by
Maintenance::purgeRedundantText() in r52006, yet the file had not been
deleted. I grepped core and extensions in Gerrit for the names of both
the function and the file and found no usage.

The purgeOldText.php maintenance script uses the "new" (since 1.16)
method and has been retained.

Change-Id: I40afbd264242eab7fadc8352380779074d61d7f8

mw.Title: Add method to get title relative to an arbitrary namespace

I'm not sure where this kind of thing could be used outside of
dealing with templates/transclusion. Making it generic anyway.

Bug: 67448
Change-Id: Ie554adefec43997d362b5d7b45c30403912743b5

Merge "Remove 'move' from LogPage::getTitleLink"

Merge "Improve mod_security warning"

Add maxlength of 6 for time correction in preferences

* Supported formats (controlled by filterTimezoneInput) are:
** -12:34/01:23.  It says, "Max is +14:00 and min is -12:00", both
   of which fit in 6 (if someone uses -24:00/+24:00, it's the same width
   anyway).
** If it's not in that format, it's treated as a simple hour, which
   means it should be from -12 to +14 (max 3).

Bug: 70405
Change-Id: I47ec2c07929069cb5243c306a1c502751e57a31b

Arrays in PHP are a bit special...

Change-Id: Ic1c16e297f68417ba4a12c109e3a6e83974a82d5

Merge "Update CLDRPluralRuleParser library"

Merge "Add @covers tags to SpecialPageFactoryTest"

Merge "Allow logged-in users to view and use the login form again"

Allow logged-in users to view and use the login form again

The fix for bug 15484 (d0439af8) has introduced the behavior of
automatically redirecting users who view the login form when already
logged in to the requested 'returnto' location. If it was not given,
users were redirected to main page instead.

However, that has annoyed people who often switch between several
accounts and have grown accustomed to the old behavior of being able
to log in while logged in. Given that there are no conflicts between
these two features, let's just restore the old behavior when
Special:UserLogin is visited directly (no 'returnto' location given).

This reverts 5dfc57eb which removed then-dead code for showing login
form to logged-in users and tweaks an if() condition.

Bug: 70855
Change-Id: I7e40c13a6ca566b4d66d943c006af9edb6941ee9

Merge "Fix special case at Special:MediaStatistics"

Update CLDRPluralRuleParser library

Upstream release:
https://github.com/santhoshtr/CLDRPluralRuleParser/releases/tag/v1.1.3

Does not have any feature changes.
License changed to MIT

Change-Id: Icb4c00cca86083d77028a4774122acd8c595152e

Merge "Fix duplicate messages key in 'mediawiki.special.preferences' definition"

Improve coverage of MailAddress::toString()

Change-Id: I8d5a1e8f293afa03d94167b722d3078f467cf59e

Add @covers tags to SpecialPageFactoryTest

Change-Id: I997cd4296a60eb9950f7fee0caed7a2349e07bd2

Fix duplicate messages key in 'mediawiki.special.preferences' definition

Follows up: Idb00f50a

Change-Id: I852e5965de8e0bee7e976637df95daded691e11c

Localisation updates from https://translatewiki.net.

Change-Id: I2a459df4c43343ca7988e8c702e68b912aa01112

mediawiki.api.edit: Remove "format: 'json'"

"format: 'json'" is in the defaultOptions in mediawiki.api.

Change-Id: Ifdf61205d94dbf918be5427cef89a037ce9f08a2

Merge "Fix typo in release notes"

Merge "Copy translations of Special:MyLanguage alias from Translate"

Fix typo in release notes

Change-Id: I2651bc9c8b6ace473a503719e2d2930cfb50bc04

Copy translations of Special:MyLanguage alias from Translate

Tuvan (tyv) didn't have a special page section, so now it's added.

See also I6f84eaf414b6d4239522bad14e90b1cbfc82590f.

Bug: 69461
Change-Id: Ie6a4d878b87ee030f9c042b43f7b1fb424b54099

Merge "Unset redirect param in MoveLogFormatter::getMessageParameters"

Merge "Move default logo to resources/assets/ directory"

Merge "jquery.confirmable: Add a space between buttons"

findHooks.php, importSiteScripts.php: Use format=json

* Also added query continuation support to findHooks.php.
* Also fixed query continuation support in importSiteScripts.php
  (broken by 2b3f4d821cd4).

Change-Id: I7ef62d370f5e2f598ac4c5857ac0dbf3ee4c8fa2

CSSJanus: Add Apache license header

Follows-up f9e8d61 which removed the Python library and its copy
of the Apache license file.

Change-Id: Ic3ed4a2fb935e713916cb8eb6ba1673839f40532

Fix special case at Special:MediaStatistics

When a percentage was equal to 100 %, makePercentPretty
determined it should have zero decimal places, and the
resulting string "100" had "trailing 0's" removed, resulting
in a completely wrong "1".

We might either check if the number contains the decimal dot
prior to trimming, or spare the hassle for the special case
completely.

Change-Id: I15ac5caa275d72909adba27b6b88824a830bd574

Merge "UserTest: Fix edit count test which incorrectly added user to db"

UserTest: Fix edit count test which incorrectly added user to db

Although 'added' to the database, the user name was cleared
in User::loadDefaults() and not added correctly to the database.

Then if one has BetaFeatures extension, then the test fails.
Via a hook, BetaFeatures does "User::newFromName( $user->getName() );",
that produces a null object as name is '127.0.0.1' and then
$user->getOption() fails.

loadDefaults() is automatically called by addToDatabase() with correct
parameters, so we can just remove the call.

Bug: 68626
Change-Id: Ibb5c60192eb83b2608f72b59d7705854549a5dac

Merge "maintenance: Remove cssjanus.py"

Merge "DatabaseMysqlBase: Use VERSION() rather than server_info"

maintenance: Remove cssjanus.py

Follows-up r55510 (aa4d684a82) which added it for use with the
Vector stylesheets at times where we didn't have CSSJanus.php.

This has been unused since we introduced ResourceLoader and the
PHP-port of CSSJanus.

Change-Id: I2c9a022354617060d99bf1c44f0726b495ec070d

Move default logo to resources/assets/ directory

The problem here is that the path to 'wiki.png' is saved in users'
LocalSettings.

We likely should not remap the path, like we did for footer license
icons in Ic7c32e56:

* It's likely that users changed their logo image by overwriting the
  file in skins/common/.
* If the normal upgrade process is followed (overwrite-uploading new
  files), the old file will still be there with the skins/common/
  directory.
* If it does cause problems, they'll be rather easy to notice and fix.

On the other hand, maybe we should?

* This is going to be annoying for git users.
* It will bite anyone who deletes all MediaWiki files when upgrading
  via tarball, which is more likely with the recent skin system
  changes encouraging users to remove old cruft from skins/ directory.

Bug: 69277
Change-Id: I175fe57048ebf9d348fb2fe67bf62cf5df389003

Fix CSSJanus flipping in LESS mixins and remove broken custom LESS functions

Custom LESS functions are problematic for us for a number of reasons,
as outlined by Timo on bug 67368. We should get rid of them.

The only use case was implementing CSSMin data: URI embedding in LESS,
which used to be impossible due to lessc not preserving comments (bug
54673). However, thanks to new syntax added in f3779e06 we can insert
the annotations in such a way that the compiler won't mess with them.
The same technique is used in OOjs UI since 584ed144.

The LESS-function-based embedding implementation also meant that we
were unable to flip images for RTL (bug 66091 and friends: bug 66773,
bug 68326). The annotation one doesn't have this limitation.

Bug: 67368
Bug: 66091
Bug: 66773
Bug: 68326
Change-Id: I3062346ed63272a1c22b5df27b4cc1de2a699d9a

Set mw.config wgFileCanRotate only on Special:Upload instead of
site-wide

UploadWizard also uses wgFileCanRotate and set the variable by
I3c50b56e6a11742cfc1c02d3d8dbeb0c151b0e91 itself.

Change-Id: Iff6a952c8f4f21c0e27bd6b4191f3b0b453eec03

Merge "Automatic direction for the license in Special:Version"

Localisation updates from https://translatewiki.net.

Change-Id: I38d6fce89373f5ec424d44c757d3011ae9ea7eb4

Merge "Strict standards: Static function PasswordTestCase::providePasswordTests() should not be abstract"

Strict standards: Static function PasswordTestCase::providePasswordTests() should not be abstract

Caused by 96771e3a655bad2d3a9440fababaf2dcf5a62bc5. No idea why that
change merged successfully.

Change-Id: I946f179699a5f249b032c4c8f90a9e6d2b033f55

Merge "UserTest: Don't rely on the behavior of unknown user options"

direction for the license in Special:Version

Change-Id: I357d92eca18de9a7a79ddfb215337ec231250ca3

UserTest: Don't rely on the behavior of unknown user options

They are not actually expected to be preserved, although they might be.

Change-Id: I34e15f3d1e5b9b825e6fc14814c8e9a6ac151ce9

Improve mod_security warning

Make it detect mod_security2 as well, and make the warning message
scarier.

Bug: 39463
Change-Id: Ic8dbd8425299f60e9fc0b54ff1b9a9a991d24190

Unset redirect param in MoveLogFormatter::getMessageParameters

The redirect param is used to get a different message key, so there is
no need to pass it to the message, because the value is stable.
'logentry-move-move' always get a '0' and
'logentry-move-move-noredirect' always a '1'
The value is only visible with uselang=qqx

Change-Id: Ib6a3303e7401e29561165355b3611c3c123bfcd0

DatabaseMysqlBase: Use VERSION() rather than server_info

I noticed enwiki's Special:Version page was reporting the version
of MariaDB 10 as "5.5.5-10.0.11-MariaDB-log" rather than just
"10.0.11-MariaDB-log". This change should fix that.

Change-Id: I6bf7e27e88014f70594b33d089636b09b6c97527

Merge "Completely remove skin autodiscovery"

Completely remove skin autodiscovery

Also update documentation.

Bug: 65748
Change-Id: I393896281e901de2000c09c0abd16ee2a0f41e2f

Remove WMF deployment hack for 'Enlarge' image

Follow-up to I849a2222. To be merged 31 days after that cut lands
(i.e.,  after 19 September).

Bug: 69673
Change-Id: I7a920105cadf3a4dea7d3bffee7ae6ff08e40887

Merge "Fixed --maxtime handling by JobRunner"

Fixed --maxtime handling by JobRunner

Bug: 71073
Change-Id: I4ddebd5aad27d0882dd2e4614df91ac565a71d2d

Reduce new button min-width from 80px to 70px

80px is too large for mobile and causes some buttons to wrap to
other lines.

Change-Id: I00b44508e7783957a8c24c4890cc1ba4dfe265da

Bump 1.25

Change-Id: I361206d6c0522e89f685a1b76fec56ec593032c5

Remove redundant $db->resultObject() calls

DatabaseBase::query() already passes the value returned by the
database driver (usually an object or resource for SELECT queries)
through $db->resultObject(). There is no need to pass it through
the method a second time to get a ResultWrapper.

Change-Id: I5d34ab2bffb7f236d2f824b4ef0e8670f89d4599

Merge "Re-apply Ic917c7d, don't cast SpecialPageFactory::$list to an object"

Remove 'move' from LogPage::getTitleLink

LogPage::getTitleLink only called for log types which are in
$wgLogActions, but move is not there.

Change-Id: I230b25988f9a420970d807437263798e5ad1db69

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

i18n: Native digits on 'mergehistory-success'

Change-Id: If3319c20362785a4b0f875ad8b5c1526e3953b6c

Improve test coverage of Title::isValidMoveOperation

Change-Id: Ifd81b0a894714ca94ab3e627ba83c881d96c3377

Localisation updates from https://translatewiki.net.

Change-Id: Ibdc7c61789400b1ab18f05da42ecd62c54cc7bb3

Added missing NULL value to sql insert statement.

In tables.sql the page table is defined with 14 columns.  Following
this, the dummy table insert statement included 13 values causing
failure of the installation process.  This change adds the required 14th
value.

Bug: 71022
Change-Id: I8f8ee95fb4c0147ac09232ef253ecf7784b7fb52
Signed-off-by: Bill Traynor <btraynor@gmail.com>

Merge "Fix some issues with Microsoft SQL Server Driver"

Fix some issues with Microsoft SQL Server Driver

* INSERT IGNORE now works properly
* Only return an inserted id if the INSERT was actually successful
* Fix syntax error when doing LIMIT queries

Bug: 71024
Change-Id: I4dddc5c9c234b17040a28937987406511ec6ea9f

Merge "Remove unused message"

Re-apply Ic917c7d, don't cast SpecialPageFactory::$list to an object

applied on top of Ia2107dc

Change-Id: I7420b9ec9df2ebd8e983fddfc7ee17655788cfde

Remove unused message

Follow-up to Iafdf9303 and I50e2f60d

Change-Id: Ib1cecfe7bdf44e951d666417b98af0d04b9f3d81

Merge "Add Special:MediaStatistics page for file type stats"

Merge "Add missing </div> tag to packed-overlay galleries"

Add missing </div> tag to packed-overlay galleries

Bug: 71030
Change-Id: I7a1781797498b079137becee68f81bf478e6f10e

Merge "Add "unsaved changes" warning to Special:Preferences"

Add "unsaved changes" warning to Special:Preferences

Much like the similar warning on the edit page, it has been requested
that we display a warning if the user has made changes on
Special:Preferences and attempts to leave without saving.

This adapts the code from
resources/mediawiki.action/mediawiki.action.edit.editWarning.js to do
so.

Bug: 55966
Change-Id: Idb00f50ad8148cd80bd0af81b4cd06a0eb217d96

Merge "Deprecate HTMLFileCache::newFromTitle() in favor of constructor"

Merge "Add a hook to allow extensions to prevent HTML file caching"

Deprecate HTMLFileCache::newFromTitle() in favor of constructor

Change-Id: I903e68fc1f486501d790ca69146ecb835d90c9cc

Add a hook to allow extensions to prevent HTML file caching

Needed for MobileFrontend on third-party sites that use this feature.
Ideally, there should be a way to cache mobile requests too, but that's
a different issue requiring much more effort to do properly.

Bug: 68106
Change-Id: I01a76c571d9186b325f19a00cec136459707c791

Merge "SpecialUndelete: Check permissions on a per-page basis"

Merge "Update OOjs UI to v0.1.0-pre (f2c3f12959)"

SpecialUndelete: Check permissions on a per-page basis

This will allow for hooks related to Title::userCan to
grant or revoke permissions based on the specific page
or namespace.

Bug: 14801
Change-Id: I02d8eb91cc5daff6e70b0dea3d2f58ed5fbfcf81

Merge "Make ArchivedFile load title regardless of how constructed."

Update OOjs UI to v0.1.0-pre (f2c3f12959)

New changes:
f8e3599 Fix automatic sizing of windows
427b9a4 README: Add devDependencies badge
5018f7b build: Use csscomb to prettify the CSS
f2c3f12 Followup ec785c2: re-evaluate clipping state after positioning

Change-Id: I0e1d5d44a6e90a66cfa280c782b6e75ba44ce7c4

Merge "Made QueryPage use startAtomic() for trx handling"

API: Add prop modules for backlinks

With the success of prop=redirects, let's create prop=linkshere,
prop=transcludedin, and prop=fileusage.

Let's also add a miser-mode-sensitive 'namespace' parameter to
prop=redirects, since the other three have a 'namespace' parameter using
the new *_from_namespace fields.

Change-Id: Ib650db9988a553771582eed0c183d0cae4b71252

Merge "Style guide: Visually separate buttons"

Merge "checkbox.less: Set cursor on label instead of pseudo element"