Followup 1cf5a6e: use $wgResourceBasePath to refer to resources/assets, not $wgScriptPath

Bonus: actually make $wgResourceBasePath default to $wgScriptPath, rather than
special-casing it in ResourceLoaderFileModule.

Change-Id: I608435cef00d3e77a5bbdb0a0122d3e7e1a4eb78

Move assets/ to resources/assets/

While it's "semantically" incorrect (these files are not
ResourceLoader resources), putting them in that subdirectory is a lot
less hassle than introducing a new toplevel directory.

Follow-up to 2b4b9a3f. Discussion that resulted in the toplevel
assets/ took place on I6268d663 (now abandoned).

Change-Id: Iedbfd802457fe35803899e3479540177760ec30b

Merge "Start moving page move logic out of Title"

Make generating Parser test class names more robust

This includes the extension name, and it also does much
more stringent validation. In the (now rather unlikely)
event of a duplicate name, it will append a number.

This is important, as it is very confusing when this bug strikes.
There exists extensions like CharRangeSpan which will trigger this bug.

Bug: 42174
Change-Id: Idf14b4cbdb8ec103340d48855e0361acf707b101

Make GetLocalFileCopy pool counter call from tiff be generic

I was to make PagedTiffHandler subclass TransformationalImageHandler.
The biggest thing that's different is the pool counter call.
However, given that a file is just as expensive in terms of
network bandwidth regardless of its type, seems to make sense
to make this check generic.

Change-Id: I7e4a4769edaa4742803df407b9c719c54fb77de3

Merge "Use 0x for hexadecimal prefix"

Use 0x for hexadecimal prefix

0X is a legal hex prefix, but tools like pfff and my syntax highlighter
prefer the more common 0x.

Change-Id: I38bba068f9a97566b0df9fb0a2fe72d25519a21b

Merge "Save pages content in the default format of their content type"

Merge "When cloning TextContent, include model"

Localisation updates from https://translatewiki.net.

Change-Id: I4885e74d38770b40d2b3eebfb66cb4a5a3eaf8d4

ResourceLoaderEditToolbarModule: Fix PHP Strict Standards warning

PHP Strict Standards:  Declaration of ResourceLoaderEditToolbarModule::getLessCompiler()
should be compatible with ResourceLoaderFileModule::getLessCompiler(ResourceLoaderContext $context = NULL)
in …/includes/resourceloader/ResourceLoaderEditToolbarModule.php on line 0

Change-Id: I6b2de82028a81db7c65eb22f2535fbe4fa70cc91

Merge "Using ULS in Special:PageLanguage"

Merge "i18n: improve "tog-hide*" messages"

Merge "Revamp classic edit toolbar not to hardcode paths in HTML"

Merge "Split BitmapHandler into two classes."

When cloning TextContent, include model

The model id should be preserved when cloning values
on save.

Change-Id: I397d36af30cf1401023b761e8c41f0dc83e76c12

Updated mediawiki dict

Change-Id: Ia8eb4213fbc83be27de88a62d1c9c332f17b6579

Merge "Improve hidden field validation"

Update OOjs UI to v0.1.0-pre (49b64bdba7)

New changes:
c82b429 Work around position: fixed; bug in Safari
9e1432e Split out isPending, pushPending, popPending, etc. into PendingElement

Change-Id: Ifd213123adfc9c96de2f16379c8ed3d706c210f8

Localisation updates from https://translatewiki.net.

Change-Id: Iaa9c20fe36b527c5ede84daa50abdda48a71d957

Merge "Readd word-separator between edithelp link and newwindow hint"

Merge "Better false positive detection for Special:Wantedfiles"

Merge "Allow WantedQueryPage subclasses to override the existence check"

Declare visibility of functions explicitly

Be consistent in entire class.

Change-Id: I7266bf9d232f1dc1be32105ad566f13be2a2bee4

Readd word-separator between edithelp link and newwindow hint

Was removed on I47db5eab4569514d039261d11b6dedb0eeae17b5

Change-Id: I34e10fb4d43384b9170d238629128ab46976f6b2

Merge "Make sure processResponsiveImages checks for valid thumb object"

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

Save pages content in the default format of their content type

Useful in order to be sure that only the default format is saved in the database (allow to implement optional formats useful for APIs but not used in the storage)

Change-Id: Ia703319aefc8d56c377cd7766dc5985c5c3c27c1

Supress email validation on cancellation in SpecialChangeEmail

Bug: 70676
Change-Id: Ic5246ea0ebfdff633092e2f99a179f31bd3eaa71

Merge "ApiQuerySiteinfo: Use proper localised skin display names if available"

Merge "API: Add timestamp of block to blockinfo"

Update OOjs UI to v0.1.0-pre (073f37e258)

New changes:
073f37e Improve FieldLayout help popup appearance

Change-Id: I6f201a99f40a8504a4c42d8f2220a3a7319a6025

ApiQuerySiteinfo: Use proper localised skin display names if available

Based on the code in Preferences::generateSkinOptions(). This should
probably be turned into a helper method one day…

Bug: 48953
Change-Id: Ib85a783bd19de8a59e7b81a4203a05b786e67772

CSS/JSON/JavaScript ContentHandler refactoring

* All content handlers that deal with code/data tend to have
English as their page language & pageview language, so moved common
code to the abstract CodeContentHandler class.

* Renamed JSONContent & JSONContentHandler into JsonContent*

Change-Id: I46819a0572ef5becc211d0d82471ff7102edaa3c

Merge "Prevent "Notice: Undefined index" in ApiQuerySiteinfo"

Merge "Don't list nonexistent groups as addable or removable"

Don't list nonexistent groups as addable or removable

You can't add or remove a group that doesn't exist, so don't claim that
it's possible on Special:ListGroupRights or the API.

Change-Id: I02d3f00142ca1cb0cdcbf30e79fecb3c96e96405

Prevent "Notice: Undefined index" in ApiQuerySiteinfo

$row['iw_trans'] wont be set if $wgInterwikiCache is used.

Change-Id: Iea1041b9c699667c251442ed7f4c95e3ba550e7c

Start moving page move logic out of Title

This moves most of the Title::moveTo() function into a new MovePage
class, and the entirety of the Title::moveToInternal() function.

No actual code was changed except for requiring a User object in
function arguments instead of relying up on $wgUser, and changes
to "$this" as necessary.

Change-Id: I5479fa8f3920a51ddf789d55edae7dd0d9b24382

Prevent "Notice: Undefined index" in wfDiff

Change-Id: I4cc7d10021a114a2035dd2d9274c2340242800cb

Merge "Mark ParserOutput of MessageContent as varying by userlanguage"

Merge "Make image/vnd.microsoft.icon be an alias for image/x-icon mime type."

Mark ParserOutput of MessageContent as varying by userlanguage

Hopefully this will stop cache pollution when a MessageContent
ends up in parser cache. MessageContent objects just do
wfMessage()->parse(), which uses user language.

Bug: 41065
Change-Id: I75e8256a61b828cb9fc93227eb517a34ac6b9d15

Update categorylinks.cl_type when moving a page

cl_type might need to be changed if the page was moved between
special-cased namespaces (Category ↔ File ↔ (other)).

Also update cl_collation, which also might have changed in the
meantime.

Bug: 69740
Change-Id: I46032a8ba04d77005731c02680f77a8808974087

Merge "Fix double output of mimesearch-summary"

Fix double output of mimesearch-summary

The execute method was calling a outputHeader(), and then calling
it's parent implementation which was also calling outputHeader()
causing double output.

Refactor class to use $this->getPageHeader() to output menus
before result list, as that appears to be what that method is
for.

Change-Id: Ic0a79cb73136e636f96e33859dadb384eca38255

Localisation updates from https://translatewiki.net.

Change-Id: I0d9118c2c9329361474b41519114484344cae73d

API: Add timestamp of block to blockinfo

This change affects list=allusers, list=users
and meta=userinfo.

Note: This change also add block expiry to meta=userinfo.
Unlike this field in other modules, it formats the timestamp
properly, instead of just dumping db contents.

Resurrecting from abandoned change Ifdeac5c5f547

Bug: 63326
Change-Id: I4b3e55fe2d07271e1ded89d36d0b98de0e643177

Better false positive detection for Special:Wantedfiles

Previously the page crossed out entries based on Title::isKnown.
This could potentially catch pages which are valid file redirects,
which shouldn't be crossed out.

This shouldn't affect anything performance wise, as the previous
Title::isKnown check would call wfFindFile anyways.

Change-Id: Ia9eac8f5618181c631a5b2b1c461b12a90f2dde7

Allow WantedQueryPage subclasses to override the existence check

Some pages are still "wanted" even if the page exists. For
example, consider a page in the file namespace that exists but
doesn't have a corresponding file, in the context of
special:wantedfiles.

Change-Id: Ie40f2baea22b3564e7f06c4fa21bc1efdc0e4d54

Split BitmapHandler into two classes.

BitmapHandler has a lot of generic-ish functionality that could
be re-usable by extension classes (Such as how it organizes
$scalerParams array, or various image magick escaping methods).
However it's combined with a lot of very format specific things,
such as the shell-out call to image magick.

Try to separate out the more generic stuff into
TransformationalImageHandler. In order to do this, I also made
canRotate, autoRotateEnabled, and getScalerType non-static. No
extensions in our repo appeared to be using these methods, and they
don't really make sense to be static (imo).

In particular, I think code duplication can be reduced in
PagedTiffHandler by extending this new class. See comments
on I1b9a77a4a56eeb65.

Change-Id: Id3a8b25a598942572cb5791a95e86054d7784961

Make sure processResponsiveImages checks for valid thumb object

If the thumbnail is an error (or worse yet, if the MediaHandler
returns false, like TMH does in certain circumstances), we
should not add responsive thumbnail links.

Change-Id: I2aafbd07e8435ce2d1b4cb32e49e98cb5f6c00ab

Merge "MediaWikiTestCase: Enforce children call parent::tearDown"

MediaWikiTestCase: Enforce children call parent::tearDown

Can't be implemented as a test case because that's not being
run before MediaWikiTestCase::tearDown.
Also there's no parent destructor, so no need to call one.

Also fixed BagOStuffTest.

Change-Id: Ifd8659b6c8a748c6716099f8822e2c50ab51bda7

Add MailAddress::newFromUser()

And tests!

Change-Id: I5214c50855f6bc756f6d748e435ae2124b2264c1

Add tests for MailAddress

Change-Id: Ia192919322743854096aceb24be0fcf9b1aa32b4

Split UserMailer.php into includes/mail/

Change-Id: Ic435bbdbb690028a5d34e8176522fcf4aa44fa92

Fix Special:Listusers i18n issues

* Changed hard coded brackets to use message 'brackets'
* Changed hard coded space to use message 'word-separator'

Bug: 39845
Change-Id: If117e8f1b2436ed1d259add92fc8f6925403afc3

Merge "Move footer icons to assets/ directory"

Merge "API: Add rawcontinue parameter to action=query"

Merge "Use local var in Action::setHeaders"

Merge "Allow preloadparams for Special:MyPage"

ResourceLoaderWikiModule: Check content format instead of content model

For Gadgets 2.0, we have custom content models that represent
JavaScript and CSS pages. Comparing the exact content model here
is too strict, when we really want to know if the page supports
the JavaScript or CSS formats.

Also fix a debug log comment that didn't work due to improper quoting.

Change-Id: I64e86c74173b1c88cac71483327cacdd8b9967b9

Fix subclassing of CssContent and JavaScriptContent

We need to be able to set a different content model via the
constructor, so make it an optional parameter.

Change-Id: I8b31084e0cfbcca1b3785176eac492d837e0cb0f

Revamp classic edit toolbar not to hardcode paths in HTML

Also, try out a way to have per-module LESS variables defined in PHP.
This might come in handy in the future… Maybe for skin theme support?

(I recommend reviewing the file changes in the order below. :D)

includes/resourceloader/ResourceLoaderFileModule.php
  * Pass the context (ResourceLoaderContext) deeper down via
    readStyleFiles() and readStyleFile(). We need it to compile the
    .less files for the right language.
  * Extract LESS compiler creation to getLessCompiler().
  * Allow passing a LESS compiler instance to compileLessFile(), rather
    than getting one after the method is called.

  All of the changes are backwards-compatible.

includes/resourceloader/ResourceLoaderEditToolbarModule.php
  * New module to support getting the language data and passing it to
    LESS variables.

  It might be a good idea to factor out a reusable class for a LESS
  module with additional variables, but that would require more
  attention to design than I gave it.

resources/src/mediawiki.action/mediawiki.action.edit.toolbar/mediawiki.action.edit.toolbar.less
  * Glue code to use the language data defined by the module above and
    put it in final CSS.

includes/EditPage.php
  * Do not hardcode image URLs in output HTML, as they are provided in
    CSS now. This gets rid of some usage of globals.

  In fact, we should be able to finally move the inline JavaScript
  calls out of getEditToolbar(), but I'm already introducing too many
  changes for one patch. That can be done later.

languages/Language.php
  * Add getImageFiles() to complement existing getImageFile() method.
    Misleadingly named, it returns paths for images for the toolbar
    only (and no other ones at all).

skins/common/ → resources/src/mediawiki.action/mediawiki.action.edit.toolbar/
  * Moved all of the button images to new location.

  Also, boring cleanup that was harder before because we treated the
  paths as public API:
  * Placed default ones in en/ subdirectory.
  * Renamed cyrl/ to ru/.
  * Renamed ksh/button_S_italic.png → ksh/button_italic.png.

languages/messages/
  * Adjusting paths and filenames for the changes above.

resources/src/mediawiki.action/mediawiki.action.edit.css
resources/src/mediawiki.action/mediawiki.action.edit.js
  * Added styles and updated the script to make it possible to have
    non-<img> elements as toolbar buttons.
  * Consolidated styles that were already required, but defined
    somewhere else:
    * `cursor: pointer;` (from shared.css)
    * `vertical-align: middle;` (from commonElements.css)

Bug: 69277
Change-Id: I39d8ed4258c7da0fe4fe4c665cdb26c86420769c

API: Add rawcontinue parameter to action=query

Soon the simplified continuation will become the default for
action=query. This parameter will then serve select the return of the
raw continuation data. To allow for clients to transition without
"unrecognized parameter" warnings, I'd like to add the parameter now
before even adding deprecation warnings.

Change-Id: I5f6895b6a94c53a231a50707ee9463f91295da8f

Merge "Clean up mediawiki.legacy.protect a bit."

Merge "Move Config::set() to MutableConfig::set()"

Make image/vnd.microsoft.icon be an alias for image/x-icon mime type.

The mime type .ico files is detected as image/vnd.microsoft.icon on some PHP
installations. This fixes the issue by aliasing it to image/x-icon.

Bug: 69437
Change-Id: I1fdb4c7058546d6cdd47b1961706ef84a05dd360

Clean up mediawiki.legacy.protect a bit.

Also kills some inline JS stuff in ProtectionForm

Bug: 33871
Change-Id: Ie43c80bf5ebd6107458d5284cc68b7128f29359a

Merge "resourceloader: Condition-wrap the HTML tag instead of JS response"

Merge "Revert "Add support for svg in .list-style-image""

Revert "Add support for svg in .list-style-image"

The generated CSS is invalid, as you can't have multiple
list-style-images (unlike background-images). Browsers
either ignore the invalid properties with linear-gradient
and use non-embedded PNG fallback (Chrome, Firefox, modern
IE), or accept the properties but ignore everything after
the comma, using the fully transparent gradient (Opera).

This reverts commit 502fffe3bf2cb6a8fccb8059436a038e21153596.

Change-Id: I87f8142b5e718f96ca8ee74578224b4db7dedce9

i18n: improve "tog-hide*" messages

Replace "Hide - in" by "Hide - from":
- tog-hideminot
- tog-hidepatrolled

Spotted by Lokal Profil
https://translatewiki.net/wiki/Thread:Support/About_MediaWiki:Tog-hideminor/sv

Change-Id: If9d7fbd48c2dcb3ac789c681c4ef3d44854706a8

Merge "Add margin below CAPTCHA box (and above submit) for signup and login"

Use local var in Action::setHeaders

Change-Id: I543d45835016fd6f4f2ec0e84f4e5453237bbf25

TablePager: Use wfScript() instead of $wgScript

Change-Id: Ib12bd3dfa102dc673789c27506becc2fd8138f56

Merge "SpecialEmailUser: Use Config instead of globals"

Merge "Typo: compatability -> compatibility"

SpecialEmailUser: Use Config instead of globals

Change-Id: I81f6fb942a9cb12332c4763ab4c26cfcc556ddcb

Typo: compatability -> compatibility

Change-Id: I533dfc9daff5e11c1732d5d81a54faa9d7b0c43a

Preferences: Use Config instead of globals

Change-Id: Iab8725ded4da532a3072fb68e3060bd8f8ddf30a

Merge "Make MySQLi work with non-standard socket"

Make MySQLi work with non-standard socket

MySQLi needs to specify the socket location as a parameter, and not as part
of the hostname.

Split the socket out of the hostname if it contains *one* colon (IPv6 safe)
after checking for the port number.

Bug: 65998
Change-Id: I66ee34e1029bc1630669bde4272d0eae9d3fd9f1

Use safe attribute accessor for RecentChange

The RecentChange does not guarantee that all attributes are populated.
Handily it provides a safe getter function in
RecentChange::getAttribute() that will return null with out triggering
an undefined index warning for missing attributes. Protects against log
spam like "Notice: Undefined index: rc_id".

Change-Id: Idee844f0d40a2a084e17f201b5e1501d59a0464d

Add margin below CAPTCHA box (and above submit) for signup and login

Only applies if the CAPTCHA box is shown, which is if ConfirmEdit is
installed.

Bug: 68788
Change-Id: I8fa5f9e0b0394594bb59b1c3f62077e5e7428525

Add support for svg in .list-style-image

* You can now use .list-style-image-svg for support for svg.

* Please see I5196952088e584d3121600e9e829647e8c195bbb for it being used on vector.

Note: This has been tested by me.

Change-Id: Iffff2bc52e1572e661930242ad15937bc83abace

Merge "Group E-mail settings stuff in Setup.php"

Localisation updates from https://translatewiki.net.

Change-Id: I4c78d0591a5cf4fff95dd076bf08741eec4ca050

Merge "RawAction: private variable removing m prefix + doc better"

Merge "Sort modules in alphabetical order and leave note about this"

Merge "SearchResult: clean up construction"

Merge "Fix for Ia9baaf0b: Make previously public variables public again"

SearchResult: clean up construction

- Remove old pre-1.17 constructor format. No core or extension callers
- Remove only usage of newFromRow(), needless abstraction

Change-Id: Ifad161aa36837bb4c8b5c9aecac1c46e2e6cc84c

Merge "Clear the DBO_TRX flag for sanity in ExternalStore"

Clear the DBO_TRX flag for sanity in ExternalStore

* This avoids an explicit commit of an implicit transaction when the site
  config for the ES cluster does not clear that flag by mistake.

Change-Id: I57d8b04cbf3249849db1643fc82673e9de8ea15b

Merge "Commented and re-ordered the LoadBalancer fields"

Merge "Remove patch-restructure.sql"

RawAction: private variable removing m prefix + doc better

Change-Id: I526ebcbb1429af42fa9e33cdf59d12c4854247f7

Merge "Fix SpecialPageFactory::getTitleForAlias"