Merge "jsduck: Load OOjs and OOjs UI in eg-iframe.html"

Merge "Fix hook documentation for UploadFormSourceDescriptors"

Add hook to allow lazy loading of import sources

Currently import sources have to be set into $wgImportSources as part of
wiki startup. This is not practical for the WMF cluster, where we need some
reasonably complex logic to set up the import source structure.

This change allows the import source list to be populated from a new
"ImportSources" hook. This hook is only called when the list of import
sources is actually needed (namely, when a user with relevant permissions
loads Special:Import).

Bug: T17583
Change-Id: Ice9a19cb6dfe53ae72aa71353d0553ee9338f233

Fix hook documentation for PageRenderingHash

The last parameter was not in documentation.

Follows Ieecb27216c39b7e6e354b4a1904fbff06506307d

Change-Id: I1eaf0e98a97f804f6b402517e45fcad54ee65ac0

jsduck: Load OOjs and OOjs UI in eg-iframe.html

This makes live preview work for OOUI classes.

Also:
* Remove duplicate definition of window.onerror (the latter which
  fails to include the 'error' variable in the logged message).
* Set mwPerformance to match startup.js as it was throwing
  a fatal exception on every page.

Bug: T112116
Change-Id: I00e789a2853e82e9a89d07752ce92fa6bb9615f5

Merge "Translate password reset e-mail subject into proper language"

Merge "build: Enable jscs jsDoc rule 'checkRedundantReturns' and make pass"

build: Enable jscs jsDoc rule 'checkRedundantReturns' and make pass

Change-Id: I5006ea3df6798201edfacbf8ad4624266554f064

Add my second gerrit e-mail to .mailmap

Change-Id: I6f57e871a54ae73a375023e6ab033c74dd67c8fc

Merge "Update mailmap from USERINFO"

Merge "objectcache: Fix some doc bullet points in WANObjectCache"

Merge "objectcache: Add comments about DB isolation to WANObjectCache"

objectcache: Fix some doc bullet points in WANObjectCache

Change-Id: Id3e65b0768d45087aaf83ed3f432923c82acc713

objectcache: Add comments about DB isolation to WANObjectCache

Change-Id: Ic3454d0ebfdb05488dd650f77e09264842caea1c

Merge "Update OOjs UI to v0.12.9"

mw.storage: Fix broken test (incompatible with Chrome 45)

The localStorage global is read-only which Chrome 45 enforces.

Simplify things by dropping mediaWiki.storage.isStorageEnabled, and
by wrapping each operation with a try / catch.

Bug: T113413
Change-Id: Iecbdeb050b9a69febd3388898d98293a84588a94

Update OOjs UI to v0.12.9

Release notes:
 https://git.wikimedia.org/blob/oojs%2Fui.git/v0.12.9/History.md

Change-Id: I8780a6d31a3c9d11e42f0858bb66a39f678bf28f

Localisation updates from https://translatewiki.net.

Change-Id: I22fc6e0c1af4d2db3c1a7e053a1597764dce8339

Mark UserRights hook as deprecated

Bug: T113358
Change-Id: I0b4a319f703c549da42d579063e1eadb2260fa21

Merge "Moved LinksDeletionUpdate to a separate file"

Merge "Add the performer to the UserRightsChanged hook"

Moved LinksDeletionUpdate to a separate file

* Also removed unused updateCategoryCounts() method

Change-Id: I25f64d3771a23d18e7993433dab449adec375d9b

Merge "mediawiki.Upload: Return same data for upload & stashed upload"

Add the performer to the UserRightsChanged hook

This allows extensions (e.g. Echo) to detect who made the change without
relying upon $wgUser. It also allows for differentiation between
autopromotion entries which will pass in `false` as the performer.

Change-Id: Idebd78b54dcea1bdc84c83f402e87b240ab4ade1

mediawiki.Upload: Return same data for upload & stashed upload

The stashed upload was omitting the imageinfo and raw result in the promise.

Change-Id: I79462c1121c879d90eed3d245948335cd08bba30

Merge "Set visibility of monolog tests setUp method to protected"

Merge "Also check Kafka class exists in KafkaHandlerTest"

Set visibility of monolog tests setUp method to protected

In phpunit and MediaWikiTestCase, the setUp method is protected,
and it's nice if the subclasses are consistent.

Change-Id: I4b8ac612951168913bb2dc9cf16d3dc18762a211

Also check Kafka class exists in KafkaHandlerTest

I have monolog installed but not Kafka, so still
the test was run and broken for me.

Change-Id: If396815d77b0ae25fbd5c16b628f217478062b43

Merge "Fixes for RedisBagOStuff when using twemproxy"

Merge "Fix typo in KafkaHandlerTest"

Merge "Fix missing dependencies needed for tests."

Fix typo in KafkaHandlerTest

Change-Id: Ib73152a269e01bc64a4b1e7a9157de2088d152d2

Fix exception in Import, when import of a revision fails

A 'notice' is thrown when an import fails, for some reason,
such as the user does not have permission, and the reason
is reported to the user.

In this case, $title is false and not a Title object,
as needed by the beforeImportPage callback (which calls
WikiPage::factory).  As well, $pageInfo['_title'] is undefined,
in pageOutCallback, which also calls WikiPage::factory via
finishImportPage.

Bug: T108544
Change-Id: I55042fdf305cd1198d3a4b28a0ebb5ce31b76a1f

Fix missing dependencies needed for tests.

Added to require-dev: wikimedia/avro, nmred/kafka-php,
monolog/monolog. These were added to vendor, but are
only suggested in core.

Change-Id: Ic760077df11bfac461ad85c5d6d395bed55d6c92

Follow-up 5cd0a43e5: Add SelectFileWidget's i18n to ResourcesOOUI

Change-Id: Ibd8ff7e66ac48bdfb24a3d2a2673e1586c4bb5dc

Merge "Use CSS columns instead of tables in Special:SpecialPages (2)"

Fix typo in AvroFormatterTest

Also removes trailing whitespace in that file.

Change-Id: Icf30a68ea7356735f416baae8a435e0f10d5caaa

Merge "jquery.byteLimit: Trigger 'change' event after limitting"

Merge "Add new authentication-related hooks (and remove one) replacing some AuthPlugin methods"

Merge "Set getFeatures() for MemoryFileBackend"

Merge "Update UploadWizard-specific messages"

Merge "By default have Main page text on Main page tab"

By default have Main page text on Main page tab

Bug: T111645
Change-Id: I8e3ca7e6b636466c145d90bd22a2d9098708af73

Merge "registration: Allow extensions to specify which MW core versions they require"

jobqueue: A few small code cleanups to RefreshLinksJob

Change-Id: Ia331e9dbf9d2be137c34a8c93ef2d6da8aad6c56

Merge "Removed DeferredUpdates::addHTMLCacheUpdate()"

Removed DeferredUpdates::addHTMLCacheUpdate()

* That method does not belong in DeferredUpdates
  as this class should agnostic to the work done

Change-Id: I8098ffb82bf9c33ccb48a31aa9dbad514e8724a8

Merge "Reduced the DOS potential of 404 page floods"

Merge "Removed deprecated $wgDeferredUpdateList"

Reduced the DOS potential of 404 page floods

* showMissingArticle() will now only show the log excerpt
  if the page was deleted very recently or if the viewing
  user is logged in. Crawlers and poorly written bots tend
  to be logged out, and logged in users are better tracked.
  If many hits to the same recently deleted page are made,
  then the innoDB buffer pool should actually have the
  relevant index pages in cache (especially with query grouping).
* There have been several outages or DB performance problems
  caused by crawlers or sloppy mirrors hitting many bogus pages
  (due to bad URL generation, ect...). Previously a redis bloom
  filter was used to handle this, but was removed due to high
  complexity (especially if it was to be supported in multi-DC
  setup). This is a simpler solution to keep 404 pages cheap,
  as they cannot really be cached (there are innumerable possible
  titles that never existed).

Change-Id: If948602a32deb16dba21d232d0c6128568a980d6

Removed deprecated $wgDeferredUpdateList

Change-Id: I7072ad3b34dd9a0726df6d1c044616c3b4bcd79f

Merge "Produce monolog messages through kafka+avro"

Set getFeatures() for MemoryFileBackend

Change-Id: I565e2e2c8a4a89c6706349a4ef9826f87445d1d2

Merge "Added read affinity tests for FileBackendMultiWrite"

Produce monolog messages through kafka+avro

This allows a logging channel to be configured to write
directly to kafka. Logs can be serialized either to json
blobs or the more compact apache avro format.

The Kafka handler for monolog needs a list of one of more
kafka servers to query cluster metadata from. This should be
able to use any monolog formatter, although some like
JsonFormatter require you to disable formatBatch as Kafka
protocol would prefer to encode each record independently in
the protocol.  This requires the nmred/kafka-php library,
version >= 1.3.0.

Adds a new formatter which serializes to the apache avro
format. This is a compact binary format which uses pre-
defined schemas. This initial implementation is very simple
and takes the plain schemas as a constructor argument.

Adds a new option to MonologSpi to wrap handlers in a
BufferHandler. This doesn't flush until the request shuts
down and prevents any network requests in the logger from
adding latency to web requests.

Related mediawiki/vendor update: Ibfe4bd2036ae8e998e2973f07bd9a6f057691578

The necessary config is something like:

array(
    'loggers' => array(
        'CirrusSearchRequests' => array(
            'handlers' => array( 'kafka' ),
        ),
    ),
    'handlers' => array(
        'kafka' => array(
            'factory' => '\\MediaWiki\\Logger\\Monolog\\KafkaHandler::factory',
            'args' => array( 'localhost:9092' ),
            'formatter' => 'avro',
            'buffer' => true,
        ),
    ),
    'formatters' => array(
        'avro' => array(
            'class' => '\\MediaWiki\\Logger\\Monolog\\AvroFormatter',
            'args' => array(
                array(
                    'CirrusSearchRequests' => array(
                        'type' => 'record',
                        'name' => 'CirrusSearchRequests'
                        'fields' => array( ... )
                    ),
                ),
            ),
        ),
    ),
)

Bug: T106256
Change-Id: I6ee744b3e5306af0bed70811b558a543eed22840

Localisation updates from https://translatewiki.net.

Change-Id: I63f5687fe78418403fdb3de398ece449efd3c72a

Merge "Refactor changeTypes in RecentChange"

Merge "Add tests for RecentChange::parse(To|From)RCType"

Merge "Cleaned up $asyncWrites member variable in MultiWriteBagOStuff"

Cleaned up $asyncWrites member variable in MultiWriteBagOStuff

Change-Id: I27e48aef77ac0f6453b7837f2b2686b7b5b726d4

MultiWriteBagOStuff: require nonempty 'caches' param

MultiWriteBagOStuff::__construct() barfs if the 'caches' parameter is unset,
but it permits it to be an empty array. This means that subsequent operations
that need to reference the first (primary) cache backend need to check that
it is there at all.

Since there is not much sense in having a MultiWriteBagOStuff object with zero
cache backends, make the constructor require a nonempty array for 'caches', and
remove the subsequent checks that are made redundant by this new restriction.

Change-Id: I30f3e0dcbfe67570a368e64b8233cc0ba7f90b2f

Added read affinity tests for FileBackendMultiWrite

Change-Id: I1407273175396d4d3631531ff0ca0afa3009913e

Merge "Add i18n messages for ApiStashEdit"

registration: Allow extensions to specify which MW core versions they require

This adds a "requires" property to extension.json, which extensions and
skins can use to indicate which versions of MediaWiki core they support.
The hacky wfUseMW() is now deprecated in favor of this.

Rather than writing our own version constraint and parser library, we
can re-use composer's, which was recently split out into a separate
library named "composer/semver" for this patch.

Any syntax accepted by composer[1] is available for usage here. Test
cases have been provided to demonstrate how versions are parsed. For now
it is recommended that people stick to expressing compatability with
stable versions (e.g. ">= 1.26").

This patch does not support requiring specific MediaWiki core WMF
branches, since those do not follow the standard semver format that
composer parses. If we are unable to parse $wgVersion, all checking will
be skipped and reported as compatible.

[1] https://getcomposer.org/doc/01-basic-usage.md#package-versions

Bug: T99084
Change-Id: I7785827216e16c596356d0ae42d6b30f3f179f10

Refactor changeTypes in RecentChange

This is split from:
I03516bb34144d95e5f25c46ae98ab70ce699b31b

Change-Id: I285cdbe3d1a62aa1071a5c174928136c3f59d557

Update mailmap from USERINFO

Create mailmap rules for any @users.mediawiki.org email that has a
matching entry under USERINFO.

The script used is proposed in mediawiki/tools/release.git
https://gerrit.wikimedia.org/r/239343

Manually added a lot more by digging in Gerrit commit Authors: fields
since that is publicly available as well.

Change-Id: I517b0593f9861d68aad3d93f2556a9cf135e28d8

Add i18n messages for ApiStashEdit

Bug: T113189
Change-Id: Iee7b2fbb7b9cd4e7255c87d78be9396073f41236

Add tests for RecentChange::parse(To|From)RCType

Change-Id: I60278aaeef475ed3ec1515b1f5f28d6afcc700c5

Add test for RecentChange::newFromRow

Change-Id: I1e21d296e15c1b6d18f19ae9cc7038387c8f4e2b

Add test for RecentChange::isInRCLifespan

Change-Id: I0eac57d0cfabba3ed9b5b430e9972e7c5c0b9f8b

Merge "Use setUp instead of __construct in RecentChangeTest"

Merge "Cleanup RecentChangeTest, move things out that don't belong"

Use setUp instead of __construct in RecentChangeTest

Change-Id: Ia0d704a2832c54995fba0014ae6422765a743be0

Cleanup RecentChangeTest, move things out that don't belong

This moves out a bunch of stuff that doesnt
seem to test anything in RecentChange....

This also shows us how few tests RecentChange
actually has!

Change-Id: Ia5016c82c00dd7c35a6f1253d6e385b587ec485e

Translate password reset e-mail subject into proper language

The password reset e-mail should have the subject line localized into
the same language as the body, which is the preferred UI language
set by the user, not the content language of the wiki.

Bug: T113227
Change-Id: I3bb1e1035004a329b934afb2fbfbd407685c6e8c

Merge "ResourceLoaderModule: cache file content hash"

Merge "resourceloader: Log load.php exceptions to JavaScript console"

Merge "Move prefsection style to mediawiki.special.preferences.css"

Merge "Added readAffinity flag to FileBackendMultiWrite"

Merge "Fixed "last modified" date fudging on null edit"

Merge "resourceloader: Run ResourceLoaderRegisterModules after wgResourceModules"

Merge "Explicitly float local search results left so they come first"

Merge "Update documentation for EmailUser hooks"

Update UploadWizard-specific messages

Merge after I706d76ab4720a94e772d03196c91697d9047f5b9 in UploadWizard.

Two messages lost a silly link and $2 parameter:
* api-error-duplicate
* api-error-duplicate-archive

They were both confusingly phrased, as if the link led directly to the
file; instead, at least in UploadWizard, clicking on it opened a popup
dialog with a list. I don't expect the change to be problematic, the
few translations that I understand work equally well with the link
removed, and UploadWizard code can now handle both versions. I'm not
aware of any uses elsewhere, hopefully this makes the messages less
UploadWizard-specific and more reusable.

Two messages were removed because they are no longer used in Gerrit extensions::
* api-error-duplicate-archive-popup-title
* api-error-duplicate-popup-title

Bug: T34606
Change-Id: I9661f24323371f186976df1f5569aba002d4f34e

Merge "Removed old "noPush" config hacks from FileBackendMultiWrite"

Merge "Avoid master queries on image history view"

Merge "Prevent IDEA error in doOperationsInternal()"

Merge "Removed unused LockManager var"

Merge "Made some FileBackendMultiWrite docs more accurate"

Prevent IDEA error in doOperationsInternal()

Change-Id: I2f0a1885803fff9dc0eb8ca4a595a83736ce421b

Removed unused LockManager var

Change-Id: I02c1fef624e6f2fe04e3c6d525aaed881eacabd7

Localisation updates from https://translatewiki.net.

Change-Id: Icdbc2c339c73d7535bff79444065260c8a9a8065

Merge "Fix hook documentation for ArticleRevisionVisibilitySet"

Added readAffinity flag to FileBackendMultiWrite

* A backend with this set (normally 1) will be used for
  non-latest reads. This can be used to prefer a local,
  replicated, backend instead of one farther away (for
  multi-DC setups)
* Note that listings still come from the master always.

Bug: T112708
Change-Id: Ic4bf4ba5a2c9ef78abd11dbd0d4b48c73cad6923

Merge "Cleaned up getScopedFileLocks() return value"

Update documentation for EmailUser hooks

Document the missing $error parameter of the EmailUser hook.
Clarify the type of the address parameter
Also add an comment, why a variable is used twice

Follows 38c7c8f895f75ac04dbb6e0349779171f8a8e8a1 (r64903)

Change-Id: I1c5636dc378667ef2798c69659b43f70734f4144

Fix hook documentation for UploadFormSourceDescriptors

The second and third parameter were not in documentation.

Follows 91e4d80448dacc223d06a55a7fd7bf771c1af4e8 (r57868)

Change-Id: Ife98beab4e231d1a4236ec6bfb627afa4f818a6f

Fix hook documentation for ArticleRevisionVisibilitySet

The last parameter was not in documentation.

Follows I6a14c7b95041fdd44d7962af6dc139bcc29f7223

Change-Id: I5e5d8948d6a1f4dc2db09028c77cfb79fb4ed5d0

Merge "Strip comments before gathering url(...) refs in CSS"