Merge "Add an interface for getting "standard" file metadata."

Add an interface for getting "standard" file metadata.

Currently file metadata is handler dependant. However they usually
end up extracting the same type of data (author, date, etc) plus
one or two handler specific things. This adds a handler independent
interface for getting metadata that is likely to be common for all
types of file (At the moment, this is the exif/iptc/xmp information)

This commit used to also contain stuff adding parser functions,
which is now split to its own commit. This commit is needed
by a bunch of other commits, in particular I0d957891e0.

Change-Id: I43d9252f69dc5b8ba0b848cf40aa1b97329c85ae

UploadStash::removeFileNoAuth shouldn't need auth

UploadStash::getFile has a "noAuth" parameter, which should be set true
when this is called from UploadStash::removeFileNoAuth.

Callers should already be doing their own auth checking, or calling
UploadStash::removeFile.

Bug: 56298
Change-Id: Ic70c4e2bc6df5cb9b139a3631766f8ff0ec4ef02

Merge "mediawiki.js: Fix docucumentation breakage"

Merge "Backport information boxes' styles from vforms to shared CSS"

Merge "Revert .htaccess permissions back to 0644"

Remove poorly-phrased doc comment and the FIXME which went with it

Change-Id: I82994ba8f92322b420b4a3fed40d5d579b06a6d2

Fix double space in trigger_error() call

Change-Id: I31b0aab630bf4083f4fa15d3fbce8a0d0759c94e

mediawiki.js: Fix docucumentation breakage

* Unformatted code and unclosed <media> tag.
  Depending on which versions of other libs are installed, this
  error can occur. Current version (v4.x) in production doesn't
  require this yet. But local environment with the same version
  does. And the next version of jsduck (v5.x) will consistently
  work this way for compliance with Markdown.

* Simplified @link while at it to use the cleaner implied link
  instead like we do elsewhere ({@link} should rarely be
  used directly).

* Add /** @class */ because half the file was no longer indexed
  as part of mw (instead became part of mw.Message after 4f23099)

* Index the private mw#log as log_ instead because it clashes
  with the public mw#log. This should've clashed in 4f23099, but
  didn't because the other one was seen as part of mw.Message
  by the test.

Change-Id: Ic5b8c5c743ecdb1509652c3655d83e501f6fa9ee

Merge "Cache ResourceLoader modules in localStorage"

Fix populateRevisionLength.php so it's not trying to select revision
fields from the archive table (they won't exist!)

Follows up https://gerrit.wikimedia.org/r/#/c/54111/

Change-Id: Ie161a08097a2cd3393d69574263cfe78e4329bb4

Minor optimization to the AutoLoader

When MediaWiki autoloading fails, we should gracefully return false.
Instead, we have been calling strtolower roughly 1,000 times in the hope
of finding a case-insensitive match.

This patch preserves the legacy case-insensitivity, but improves its
performance by approximately 100x, by storing the case-insensitive class
lookups as a static variable.

There is a new global $wgAutoloadAttemptLowercase which will switch the
behavior if desired.  The default is to support case-insensitive loading.

Change-Id: Ifb12e05614a48390b730167e9d4ddcd8545db764

Merge "installer: Run the LESS compiler for *.less files"

Revert .htaccess permissions back to 0644

Change Id6916fe999c0faa38de878b7b5687e7ea00901bd accidently
changed the permissions on this file, changing it back.

Change-Id: I14b68b066ed16b2221f4be52172ad7b32b86a61f

Cache ResourceLoader modules in localStorage

To minimize the number of discrete requests that the browser has to make in
order to render the page, ResourceLoader tries to condense as many modules as
possible into each request. To ensure that the response is not stale,
ResourceLoader tacks the modification time of the most recently modified module
to the request. This behavior makes poor use of locality: an update to a single
module will change the URL that is used to retrieve it and a number of
unrelated modules, causing those modules to be re-retrieved even though they
have not changed since they were last retrieved. This is because the browser
cache is not aware that the response from load.php is a composite of modules
that should be versioned separately.

This patch adds mw.loader.store. On browsers that implement the localStorage
API, the module store serves as a smart complement to the browser cache. Unlike
the browser cache, the module store can slice a concatenated response from
ResourceLoader into its constituent modules and cache each of them separately,
using each module's versioning scheme to determine when the cache should be
invalidated.

Because the localStorage API is synchronous and slower than memory access,
modules are cached as a single JSON blob. At the beginning of the page load
process, the entire blob is loaded from storage into memory. During the load
process, any required modules that are not in the store are fetched from the
server and set in the in-memory store. When the DOM is complete, the store is
synced back to localStorage in a single operation.

* NOTE: The module store is enabled only if $wgResourceLoaderStorageEnabled is
  set to true; it is false by default. We can change the default if / when we
  establish conclusively that the feature is beneficial and stable.

Change-Id: If2ad2d80db2336fb447817f5c56599667141ec66

Localisation updates from translatewiki.net.

Change-Id: If1d555c052dbfacd3f5af202ebb9234346005c09

Add correct font color override to beta variables

Note that originally the font color was specified incorrectly in
variables.less (fixed in I118215e7578e8af94a6909988620dcf04805d14a)

This restores the color to how it should be in beta
Also rename variable as the name is very misleading - it only applies
to the content

Change-Id: Iff397075ef4d11b6f815d8a596ed249d2708398a

Update populateRevisionLength maintenance script to populate archive.ar_len

Bug: 24538
Change-Id: I0f685aa5ea48468279475b487a627ff73c70d5cd

Merge "vector: Restore @body-font-color to black"

Merge "Revert "Revert "Add new recentchanges field rc_source to replace rc_type"""

Merge "Remove unneeded methods in tests"

Remove unneeded methods in tests

Change-Id: Ia2da4027f2cc4a3aaff02695bab35de32a33ed60

Merge "Fix typo in usage of mwjsduck-gen"

Fix typo in doc comment for Message.prototype#toString

"it's string form" => "its string form".

Change-Id: I867283c5333aa17dd146c19684c1959e0de2a1ca

Fix typo in usage of mwjsduck-gen

Change-Id: Ib249c42d77059b18f8d6172599c631d5e1483c5d

Merge "mediawiki.Title: Generalise filename-matching in newFromImg"

mediawiki.Title: Generalise filename-matching in newFromImg

Both core and extensions can (and do) modify the thumbnail string
e.g. to add page numbers, languages, etc.

As bawolff notes in the commentary to bug 55963, the assumption
that the filename itself appears twice is likely to be violated
from time to time, but for the moment it suffices to fix the
apparent regression described in that bug report.

Also includes some additional test cases.

Bug: 55963
Change-Id: I9bb29a24f9bf50924699b913e31f44689906c525

Merge "Improve wording of eauthentsent message for email confirmation"

Improve wording of eauthentsent message for email confirmation

eauthentsent message is used on Special:ChangeEmail confirmation page.

Change-Id: I34455b1452b670f5d1dcf58982d52326731c519d

Merge "Localisation updates from translatewiki.net."

Merge "Set relevant User on Special:Unblock"

Merge "API: Remove leading/trailing spaces from error and description text"

Localisation updates from translatewiki.net.

Change-Id: Ibf8e46c56d36a83f4c0bc225e34839a8c6e832d6

Correctly update wl_notificationtimestamp when viewing old revisions

== Prelude ==
wl_notificationtimestamp controls sending the user e-mail
notifications about changes to pages, as well as showing the "updated
since last visit" markers on history pages, recent changes and
watchlist.

== The bug ==
Previously, on every view of a page, the notification timestamp was
cleared, regardless of whether the user as actually viewing the latest
revision. When viewing a diff, however, the timestamp was cleared only
if one of the revisions being compared was the latest one of its page.

The same behavior applied to talk page message indicators (which are
actually stored sepately to cater to anonymous users).

This was inconsistent and surprising when one was attempting to, say,
go through the 50 new posts to a discussion page in a peacemeal
fashion.

== The fix ==
If the revision being viewed is the latest (or can't be determined),
the timestamp is cleared as previously, as this is necessary to
reenable e-mail notifications for given user and page.

If the revision isn't the latest, the timestamp is updated to
revision's timestamp plus one second. This uses up to two simple
(selectField) indexed queries per page view, only fired when we
do not already know we're looking at the latest version.

Talk page indicator is updated to point at the next revision after the
one being viewed, or cleared if viewing the latest revision. The
UserClearNewTalkNotification hook gained $oldid as the second argument
(a backwards-compatible change). In Skin, we no longer ignore the
indicator being present if we're viewing the talk page, as it might
still be valid.

== The bonus ==
Comments and formatting was updated in a few places, including
tables.sql and Wiki.php.

The following functions gained a second, optional $oldid parameter
(holy indirection, Batman!):
* WikiPage#doViewUpdates()
* User#clearNotification()
* WatchedItem#resetNotificationTimestamp()

DifferenceEngine gained a public method mapDiffPrevNext() used
to parse the ids from URL parameters like oldid=12345&diff=prev,
factored out of loadRevisionIds(). A bug where the NewDifferenceEngine
hook would not be called in some cases, dating back to its
introduction in r45518, was fixed in the process.

Bug: 41759
Change-Id: I4144ba1987b8d7a7e8b24f4f067eedac2ae44459

API: Remove leading/trailing spaces from error and description text

Change-Id: Id866c7258a297fe965a64d52e3458d53e140aa4c

Localisation updates from translatewiki.net.

Change-Id: I5b8bcbcdd7538ba9856dff1c60502f6fa93a0116

vector: Restore @body-font-color to black

Changed in Ica0b69ad, I presume accidentally.

https://en.wikipedia.org/w/?oldid=578807552#Font_color_of_article_text

Change-Id: I118215e7578e8af94a6909988620dcf04805d14a

Test for Status::hasMessage

Change-Id: I080e006fa6a7d8ad68bdf827c4bab7d0ea160230

Merge "FormatJson: Remove whitespace from empty arrays and objects"

Merge "Add HTML ID to tag filter input"

Add HTML ID to tag filter input

Makes it so if you click "Tag filter" the input box is
now focused.

Bug: 56175
Change-Id: I502f86f227bc0d1cc5ea936d9047bc51d169323a

Merge "Distinguish redactions from the string "REDACTED" in formatRedactedTrace"

Revert "Revert "Add new recentchanges field rc_source to replace rc_type""

This reverts commit 113f49bb21487f813548fa4e75ecb568c433ecbf after the
schema change has been completed.

Change-Id: I5412466444526986e1209d0982dd62cd5b387ad1

Merge "Fix Tidy quietly breaking TOC disabling"

Merge changes I61f10917,Id5b9872e

* changes:
  mw.loader: Always log exceptions caught from userland callbacks
  mediawiki.js: Handle error better in log() and move out of mw.loader

mw.loader: Always log exceptions caught from userland callbacks

We usually don't catch these at all (not supposed to, as they're
foreign). But in case of handlePending() we catch them because
we need to ensure the state is kept track of properly (an error
is never final in javascript since execution always continues).

However we should at least log it in that case. We already did this
in case of the error() callback, it now does this for the ready()
callback as well.

Bug: 55989
Change-Id: I61f109178833fed60fbba7725fd47b4ff3d2edf4

Fix Tidy quietly breaking TOC disabling

The functionality was introduced in I2889bcb9
but doesn't work in presence if Tidy.

Change-Id: Ibf96cc3bc94fac75fd92ec5b9205011fcb68f0c2

mediawiki.js: Handle error better in log() and move out of mw.loader

console.error (in Chrome dev tools) takes an Error object. It
renders the stack trace nicely (incl. dynamic references to files)
but doesn't stringify the exception itself. It renders it as an
object (eg. no "{}" instead of "TypeError: Foo bar").

Some of the callers were embedding e.message manually. That was
better though, that still omitted the class (eg. "TypeError").
Error#toString composes the full message incl. class name prefix.

Change-Id: Id5b9872e2050326ab83889e3a806d015ec2ff3eb

Localisation updates from translatewiki.net.

Change-Id: I7d0ca25613dd2eb49dde2fb1cba56e24458ff442

Distinguish redactions from the string "REDACTED" in formatRedactedTrace

In the output of MWExceptionHandler::formatRedactedTrace, it is not
possible to determine (without checking the configuration) whether arg 0
in "foo('REDACTED')" was redacted or was merely passed the string
"REDACTED".

This patch changes redaction to instead output "foo(REDACTED)" in the
case of redaction. This parallels the situation with arrays and objects,
where for example "foo(Array)" was passed an array while "foo('Array')"
was passed the string "Array".

Change-Id: Ia2a761687c69b630afa3ccd8668b06b28e3ecdd3

Replace php_sapi_name() with PHP_SAPI

Change-Id: I222789cacaa2653c4dac62708ce228674c90af4c

Merge "Fixed use of wrong callback array element"

Merge "Use i18n instead of hardcoded brackets and spaces"

Fixed use of wrong callback array element

bug: 56124
Change-Id: Ia9f058f42616f5c7c5936733747d0c3a8d56dc19

Add some missing :'s

Change-Id: I4f48fdd3e5d81b8a0a34505795a7ab318c931a47

Merge "@covers tags for unit tests"

Merge "@covers tags for more test classes"

Merge "@covers tags for the rest of test files.."

Merge "add basic Status unit test"

Merge "Triple default $wgMaxShellMemory (raise to 300 MB)"

Pass language to File::getDescriptionText

Follow-Up: Ifcae821a51f4207e7816e710d3b3857c7ed438b6
Change-Id: I8e13f672fd37319add1c090a5b2f38c45d18fda5

Fixed broken quickDescribe() method

Change-Id: Ic68f483c4008ff6ae8c4accef2f9d1f180a5d377

Triple default $wgMaxShellMemory (raise to 300 MB)

100 MB is no longer sufficient.

That value was chosen back in 2006 (r14795); the world has largely
moved on to 64-bit programs since then, and applications such as
ImageMagick seem to use 16-bit precision for each image channel.
It's not enough to scale regular large-ish images on default system
settings anymore.

Bug: 34241
Change-Id: I66bbe12925689f9273f3b5fb29adc8c15c828c70

add basic Status unit test

Adds unit tests for some of the more basic
parts of the Status class

has todos at the bottom of the file for all other
methods that need to be tested

Change-Id: Ic2f2abafb44ef86af207c7595e440672dfcf4f1e

Merge "mediawiki.inspect: Support IE8's style.sheet.rules property"

Merge "Set $parentLen in contributions"

Merge "Localisation updates from translatewiki.net."

Localisation updates from translatewiki.net.

Change-Id: I7d7dfd1a1e1a395c639b30489bacceedfbfc6330

Merge "Fix undefined variable in Status::getMessage()"

Set $parentLen in contributions

Follow-up to I77dd45eb.

Bug: 56115
Change-Id: I17bd160577e6b287cc5a61e2a3b983b035c87f81

@covers tags for the rest of test files..

Change-Id: I0fafe80531325a412472ab7c9fc6d81c861b3751

mediawiki.inspect: Support IE8's style.sheet.rules property

Standards-compliant browsers use 'cssRules', IE8 uses 'rules',
some browsers (e.g. Chrome) support both.

Bug: 56107
Change-Id: Ief0b5e38c8ff29efea510bf3725127c2bbfccd59

@covers tags for more test classes

Change-Id: I19d49c279646a4b4c595700e53b790ba4eb9521e

Fix undefined variable in Status::getMessage()

If $shortContext and $longContext are false, and there
are errors, then $s is undefined.

Change-Id: Ie07f80b43a48a6fc4ed28b2c519f51fd32690bc8

Merge "Make TOC hideable"

Merge SimpleAntiSpam extension into core

Adds an empty, hidden input field to the edit interface, and will
reject any edit if that field is filled in. This will protect
against the simplest form of spambots.

Localized messages will be imported separately.

Bug: 52063
Change-Id: I185b6a1e232299f496244a23f4bf0dc6ea22b6c9

Backport information boxes' styles from vforms to shared CSS

They're obviously prettier and non-intrusive enough to fit old
interfaces. The colors are changed to more pastel ones and the general
size of the boxes is reduced.

Also remove unnecessary bolds on the informations on
Special:Preferences and Special:ChangePassword.

Change-Id: Ieae62db1a124261ae7f5bf67aced8b84cfbadd3d

Merge "Cleanup Api phpunit Tests"

Merge "Add more ApiFormat format tests"

Merge "@covers tags for maintenance tests"

Merge "@covers tags for the rest of /media/* tests"

Cleanup Api phpunit Tests

- Splits multiple classes into individual files
- Adds @covers tags
- Fixes scope

Change-Id: I7d2816d3574fa53a2aaa8e2a84b7a7ecdd245252

@covers tags for maintenance tests

Change-Id: Ica5f351f71ee5fccae97560f2de6d75fc5da721b

Add more ApiFormat format tests

Adds tests for Json and wddx formats
Add @covers tags
Fixes some comments
Fixes scope

Change-Id: I5f22097ddcaeb917f856ba7aa0cb793caa17925e

@covers tags for the rest of /media/* tests

Change-Id: I12879baf7810648d3ec0789feac5af364d5696f7

Merge "@covers for ApiQuery stuff"

Merge "@covers tags for inclues/*Test"

@covers tags for unit tests

Change-Id: I2b486b38326bf23bd379efba2142034e24c0745f

Initial bump to 1.23

Change-Id: I1c2d7bdd1d861ace3f7e7bd8f8f0047c31194446

@covers tags for inclues/*Test

Change-Id: Id28acdd8fe0028bf1e46344cfed131076c8f4c95

Fix double-parsing of account creation messages.

Account creation messages don't need to be parsed. This is
a temporary fix to follow up when double-parsing was accidentally
added in 69ea4400037 (I402c6bebcfe).

Bug: 44718
Bug: 52191
Change-Id: I333d5468820994625348316ebf6c57d4df025284

Cleanup skins/SiteBarTest and @todo/covers

Change-Id: I501d04ddbeee23539337b68142ee6c1371a63d98

@covers for ApiQuery stuff

Change-Id: Id8a89abd545d8e2a95b1d246776049b9c888e53c

@covers for all GlobalFunc tests

Also split 2 tests off into their correct test classes,
this methods are clearly no longer global functions

Change-Id: I482433f3099e72507a766e85d9576ff36e58b9ad

Improve linkprefix regular expressions

The regular expression in the linkprefix message is run against the
entire page up to each wikilink, and is expected to capture one group
having everything except the prefix and another having only the prefix.
For long pages this winds up being a lot of text, so inefficient regular
expressions are going to cause problems.

The current regex is this:

  /^(.*?)([a-zA-Z\\x80-\\xff]+)$/sD

This is not efficient: it will scan through the string trying to match
against every run of one or more letters/non-ASCII characters,
backtracking at every one except possibly the last. The only reason this
hasn't been a huge problem everywhere is because only a few languages
have this feature enabled.

This change replaces this with this regex:

  /^((?>.*(?<![a-zA-Z\\x80-\\xff])))(.+)$/sD

This is rather more efficient: it will grab the whole string (which is
actually fast even for huge strings), then back off character by
character until it finds one that isn't a letter/non-ASCII.

Note that the above could be simplified somewhat:

  /^((?>.*[^a-zA-Z\\x80-\\xff]|))(.+)$/sD

The performance improvement here is minor, and Gujarati, Church Slavic,
Udmurt, and Ukrainian would still need the other style for their current
implementations.

For Gujarati, we also use another regex trick: a look-behind assertion
in PCRE must be fixed length, so something like (?<!a|bb) won't work.
But that regex fragment is equivalent to (?<!a)(?<!bb) which is allowed,
so we use that instead.

Bug: 52865
Change-Id: Iaa7eaa446b3f045a9ce970affcb2a889f44bdefd

Fix scope on all /phpunit test methods

Change-Id: I3ce92463d485a0fb23e464e9a8059330f32d79af

Cleanup tests/includes/media

- Adds @covers tags
- fixes comments
- fixes scope
- adds @todos
- split files per class

Change-Id: Id5aaa20769725a96f22fb838d376697c250874a5

installer: Run the LESS compiler for *.less files

Includes the following improvements:

* Use the ResourceLoader module definitions (with a hardcoded list of
  module names) instead of a hardcoded list of filenames.
* Make the errors more discoverable like we do in load.php, prepending
  them instead of burying them in the middle somewhere.

Bug: 55589
Change-Id: Iab78a60209ab46a10d4c492c75527e717f36f803

Merge "mediawiki.inspect: use $.toJSON & add workaround for FF oddity"

mediawiki.inspect: use $.toJSON & add workaround for FF oddity

* Use $.toJSON rather than JSON.stringify. Make mediawiki.inspect depend on
  jquery.json.
* Firefox doesn't define a console.table, but doesn't throw an exception either
  if you attempt to invoke it. Use Function.prototype#call to call Firefox's
  bluff.

Change-Id: Ib45487d10488f08a98d9635ab9a2a9212e9ef69f