Merge "build: karma now reports with mocha formatter"

Merge "Ensure we use mw.now() instead of 'new Date' for relative measurement"

Ensure we use mw.now() instead of 'new Date' for relative measurement

'new Date' is subject to clock drift etc.

mw.now() uses performance.now() when available, which will always
increase at a constant rate independent of the system clock.

Change-Id: Ib653103bf6116544f35c930fb33421f1bb362c7d

Tweak CSS for Parsoid <figure>s

* These tweaks give us zero pixel diffs for images (when the
  magnify icon is hidden) outside galleries.

Change-Id: Icd5a280a13162401f0bced93a91f9dfa45a3a481

Merge "Decode '0'-valued user options to integer 0"

Merge "Add @since tags to WikiRevision"

Merge "Add missing @group Database tags in tests"

Merge "Check for string initialization in lcfirst() for HHVM 3.18"

Add @since tags to WikiRevision

Change-Id: I02005184d2cb3fa50aa0a51221b40e7c621219d2

qunit: Remove experimental CompletenessTest feature

The CompletenessTest was my attempt at measuring a basic code coverage
using run-time inspection instead of static instrumentation.

Originally added in 540419a82e5a5 (2010; MediaWiki 1.17).

It was never finished, remained fairly buggy and disabled by default.
It is also no longer used anywhere.

Bug: T155194
Change-Id: I26e7466426dddb43596f402e31005a89060c1b96

Localisation updates from https://translatewiki.net.

Change-Id: I52ddf38b7a669842ed8157f5929cf4519ff9a57a

Check for string initialization in lcfirst() for HHVM 3.18

HHVM 3.18 emits a notice when attempting to access the first offset of
an empty string.  We had that fixed for ucfirst() in 3605066c96. This is
the same for lcfirst().

Bug: T161095
Change-Id: I1456611222c24290f259298e883ca89dd830c74b

RC Filters docs: Directly mention group classes

Directly mention ChangesListBooleanFilterGroup and
ChangesListStringOptionsFilterGroup so people know what to construct.

Change-Id: I3c105d072889c1da26f94402326ba48635081833

Merge "SpecialAllMessages: Remove unused variable langcode"

Merge "Tweak figure styles based on visual diff testing"

Localisation updates from https://translatewiki.net.

Change-Id: Id631b382683ce4ac3ef648bdb546821c74a9985c

Merge "Follow-up Improve sorting on SpecialWanted*-Pages"

Merge "ResourceLoaderImage: Point to the right skin when generating URL"

Merge "ResourceLoaderOOUIImageModule: Actually load non-default themes' images"

Follow-up Improve sorting on SpecialWanted*-Pages

Change the SpecialWanted*-Pages so that they do sort
1. by the number of links to a site (as is now) and
2. alphabetically for entries which have the same number of links (new)

This is a follow-up for commit 5b15728.
The previous patch was reverted because using getOrderFields and
prefixing each order field with qc_ in fetchFromCache threw DB errors in
production. This follow-up differs in introducing a new function
getCacheOrderFields that defines order fields only for fetching from the
cache and doesn't reuse getOrderFields which is for the raw query
(without cache). It returns ['value'] by default and thus keeps the
known behaviour.

Bug: T4335
Change-Id: Ic01a3c28892c78ba9a9f0a7d665937efe87b44bd

Merge "RCFilters: Only show filter capsule as conflicted if it's selected"

RCFilters: Only show filter capsule as conflicted if it's selected

Bug: T161172
Change-Id: Ib15ffd58290c616de216178cee6ae9b86178d871

Decode '0'-valued user options to integer 0

I13d0e402f fixed a MySQL strict-mode bug by having boolean false be
sent to the database as 0 rather than "", since so many of our
logically-boolean fields are typed as tinyints. That happened to also
cause logically-false user preferences to be stored in the
user_properties table as "0" rather than "", which works fine in PHP but
confuses JavaScript since it considers string-0 as truthy rather than
falsey.

To avoid this situation, convert "0" to 0 when loading the user
options. Completely solving T54542 is left for another time, since
identifying which type to normalize each option to seems nontrivial.

Change-Id: Ia3280b7ce923641eac077141b47cba10d3fb88db

Merge "RCFilters UI: Pass in userId to ChangesListHighlights schema"

Merge "RemexCompatMunger: fix a couple of memory leaks"

Merge "Update RemexHtml to 1.0.1"

Add MusikAnimal to CREDITS

Ran maintenance/updateCredits.php which determines new contributors
using git log. Found 1.

Change-Id: I0d3a2626051ca058ea391c39550d4c1d19f4f133

RemexCompatMunger: fix a couple of memory leaks

Change-Id: I47578b3f73320e84a157417c288de97b5d26e18f

Update RemexHtml to 1.0.1

For a bug fix

Depends-On: I4421afbfa52d7711d9369398891e91ae09db50c9
Change-Id: I5285c0b1c6f42891c002637cd3e4bd2f54615940

RCFilters UI: Pass in userId to ChangesListHighlights schema

Bug: T160928
Depends-On: Ia2bc679e37deb34b215ad16dfc3b8cfefdc8365c
Change-Id: I9d3f9281c2681ab1c89676234ddd544f76f3dc7f

Merge "Allow extensions to add jQueryMsg magic words"

Localisation updates from https://translatewiki.net.

Change-Id: I83074d7d6edd5cf2e3b69d0bb2382012a5b4e438

Merge "RCFilters UI: Make the 'no results' message bold"

Merge "RCFilters: Normalize overlay font-size for Monobook"

Allow extensions to add jQueryMsg magic words

Change-Id: Ie82a147ff32ccda3f757108474f5cbab71d45ace

Merge "Stop declaring ImageHistoryPseudoPager props dynamically"

Stop declaring ImageHistoryPseudoPager props dynamically

The 3 properties declared here were introduced in:
0ca92d0bf5dd4e71b678d373a1beb3089d16a866

Change-Id: Ibbd0a86f02ab010720446db1aa2dc9ea0eb05aef

Merge "Fix log msg param in UploadFromUrl::saveTempFileChunk"

Merge "Mark public methods as public in ImageHistoryPseudoPager"

Merge "Bugzilla -> Phab task number in $wgUrlProtocols phpdoc"

Merge "Fix undefined ExternalStore::decompressRevisionText"

Check for string initialization in ucfirst() to make HHVM 3.18 happy

Bug: T161095
Change-Id: I45b5d9e819061f443d4342c004bad80bd87c2a17

Mark public methods as public in ImageHistoryPseudoPager

Change-Id: Iad4442d87df683daca978fa95068f9ca5221d8a5

Fix undefined ExternalStore::decompressRevisionText

make ExternalStore::decompressRevisionText self::decompressRevisionText

Change-Id: I3dc7317550f47e7e94d1403d3861d1059c7af6c9

Fix log msg param in UploadFromUrl::saveTempFileChunk

$this->nbytes does not exist and is never written to.
It is probably intended that this uses the local $nbytes.

Change-Id: I8e923a27625d04c81b2e272a597d40d59397851f

Bugzilla -> Phab task number in $wgUrlProtocols phpdoc

Change-Id: I1478ece1691d0b86aeae272dd041425eac4ed960

Merge "Make Revision::getRevisionText() cache the converted text"

Make Revision::getRevisionText() cache the converted text

Follows-up c2bbb51c2082.

This makes it consistent with Revision::loadText() to avoid polluting
the text cache values.

Bug: T161037
Change-Id: If2bd114870c7f7db0e56d4e8092a5dd722d3bde5

Merge "resourceloader: Follow redirects for JavaScript/CSS in WikiModule"

Merge "Cast to bool in Database::update"

Merge "RCFilters: Add conflicts for minor edits"

RCFilters: Add conflicts for minor edits

Bug: T160803
Change-Id: I91d736d772e36f54144330c30092fac85dc70d20

Tweak figure styles based on visual diff testing

* These bring figure rendering with whitespace much closer to
  the PHP parser rendering. Someone with better CSS chops should
  be able to verify / tweak this more.

Change-Id: Ia5cc191a57a34c04189ee57f6bde0387e9e683aa

Deprecate WikiRevision::$user

This was introduced in 436a028086fb3f01c4605c5ad2964d56f9306aca
but was not used there.
It also currently looks unused.

We could go ahead and just remove this now, as anything that wants
to set the $user when the property is removed would still succeed.
But lets adhere to the deprecation policy here.

Change-Id: Id47323bb1ff480a16b46107825ed5f2f861ad976

Deprecate WikiRevision::$importer

This was introduced in 9b3128eb2b654761f21fd4ca1d5a1a4b796dc912
but was not used there.
It also currently looks unused, even when grepping for possible
magic usages of 'importer' and "importer".

We could go ahead and just remove this now, as anything that wants
to set the $importer when the property is removed would still succeed.
But lets adhere to the deprecation policy here.

Change-Id: I5ac16dd87a95be6a3cdd4b16988d37d1d6104749

Deprecate WikiRevision::$fileIsTemp and fix $isTemp

Cleanup from 43d5d3b682cc1733ad01a837d11af4a402d57e6a

This commit introduces both $fileIsTemp $isTemp
It would appear that the intent was to have one property here
and it looks like both of these are meant to do the same thing.

It also looks as if both public properties are actually unused
(only used by the getter and setters)

As isTemp will now sometimes return true line 766 $autoDeleteSource
will be true in new cases.
As a result $tmpFile->autocollect(); will be called in more cases

Change-Id: I8014172fb9fd7a0e4a89d0872bafdf6c50f92646

RCFilters: Normalize overlay font-size for Monobook

Bug: T161033
Change-Id: Ia2215046f6d3fd026ecd1b27625cb94e806ababf

Cast to bool in Database::update

The interface is documented as returning a bool, so stick to that
interface.

Bug: T160956
Change-Id: I725651e1d66fdd6c52ef82e93fd9e3c837edb022

RCFilters UI: Make the 'no results' message bold

Bug: T160935
Change-Id: I237c81abfbd50e910e428cc495e38f3d364500e0

Localisation updates from https://translatewiki.net.

Change-Id: Iecc7b545f9858bef1789765e5abf010802638fd1

Merge "Update OOjs UI to v0.20.0"

Merge "maintenance: Fix broken url in comment for createCommonPasswordCdb.php"

Merge "Remove erroneous space in unset() call"

Merge "SpecialRecentchanges: Unbreak $wgAllowCategorizedRecentChanges"

Merge "User::getRights phpdoc from return array to string[]"

Remove erroneous space in unset() call

Change-Id: I67cac037aa1db46ef60ae0c321c1394d9a3bd410

RCFilters UI: Don't set reset button title in "restore" mode

In "clear" mode, the button only has an icon, so it needs a title.
In "restore" mode, the button has text, so it doesn't need a title,
much less a wrong one (the one from the "clear" mode).

Bonus: Remove properties from constructor that are already
set in reevaluateResetRestoreState().

Change-Id: Idab5731f51b42455801a505475002adf8437c66e

Merge "Remove the localStorage replication of the block cookie"

maintenance: Fix broken url in comment for createCommonPasswordCdb.php

Follows-up 2d15dcfc3f4b.

Use a permalink instead. The files have changed since.

Change-Id: I52882f29bb1d0fc791e8a43452c329344188fa53

Xml: Add test for listDropDown and remove unused getArrayFromWikiTextList

Follows-up 4b49705613 and 8c7095be85 (T34950).

Change-Id: I95b7f7ce03b2cbe68215a7ff17afc1997a54f9b1

phpunit: Fix RecentChangeTest failure when coverage is enabled

From <https://integration.wikimedia.org/ci/job/mediawiki-core-code-coverage/>

2) RecentChangeTest::testIsInRCLifespan with data set #0 (6000, 1490019080, 0, true)
   Failed asserting that false matches expected true.

3) RecentChangeTest::testIsInRCLifespan with data set #3 (3000, 1490016080, 6000, true)
   Failed asserting that false matches expected true.

Change-Id: I309bb229542f3af62e3e83025d88e77ecb55cd31

Merge "Revert "RCFilters: Change reset button title when it changes state""

Merge "Track usage of highlight feature"

Revert "RCFilters: Change reset button title when it changes state"

It makes no sense to add a tooltip to a button that already
has text, especially if the tooltip text is (almost)
the same as the button text.

This reverts commit 31047fb1bf03944158fe4f7da99686e7955a3b5c.

Change-Id: I2f9c819a85fc05f37c7c0c5acf2d72319bdb0d76

Merge "mediawiki.widgets: Remove autoEllipsis() from TitleOptionWidget"

Merge "Adjust message history-fieldset-title"

Merge "RCFilters: Scroll popup up to top when it is reopened"

Merge "Disable filter by redirect Special:AllPages and query=allpages in miser mode"

Merge "RecentChanges: Correct 'no matches found' message"

Merge "RCFilters: Change reset button title when it changes state"

Merge "Use executeOpHandlesInternal() in SwiftFileBackend"

Disable filter by redirect Special:AllPages and query=allpages in miser mode

Bug: T160916
Change-Id: Ib9562b404731e1f621b9f07c33821d04cd2aa6ae

build: karma now reports with mocha formatter

The progress reporter is quite nice to avoid spammy output to the
terminal, but it is not very human friendly when trying to find out what
is breaking.

Mocha has a very nice reporting organized as modules and list items for
each of the tests:

ext.cx.tools.template
  ✖ Equivalent template exists
  ✔ Adapting a template not existing in target language
  ✖ Adapting template having invalid title
  ✖ Template name and params adaptation using templatedata
  ✖ Template params adaptation using templateData - English to French
  ✖ Template params adaptation using templateData - English to Spanish

Add karma-mocha-reporter node module and make it the reporter.

Change-Id: Ic7d848b45e8e5ad3839d8a2bd822b63838319ad7

RCFilters: Scroll popup up to top when it is reopened

Change-Id: I0cad078f2d6b3062189ca3aca0085e346667d1bf

RecentChanges: Correct 'no matches found' message

Change-Id: Ic490fae5f3d7c474f368e981bfe9572108cf07e7

RCFilters: Change reset button title when it changes state

The button changes state between 'restore default filters' and
the trash icon which is to remove all selected filters.

The title for the button should also change accordingly.

Change-Id: I9070f0c4959f5c7c97d57d943103ae2baf89d6d2

Use executeOpHandlesInternal() in SwiftFileBackend

Instead calling doExecuteOpHandlesInternal() directly, use the
wrapper parent class method instead so that closeResources() is
called on all of the handles.

Change-Id: Iab0d4f7c08d155639a44f48147b78283059a6d24

mediawiki.widgets: Remove autoEllipsis() from TitleOptionWidget

Redundant with CSS 'text-overflow: ellipsis', which is already applied.

Aside from ellipsis overflow, autoEllipsis() was also serving as indirect
caller of `highlightText( option.matchText )` and `attr('title')`,
which we want to keep, so leave that in its place.

Bug: T160804
Change-Id: I550183750d66d769cc9c960150a2349d1b9181aa

User::getRights phpdoc from return array to string[]

Change-Id: I188ed3d427f53ff773235d35ef277de0a3221777

Merge "Selenium tests in Node.js using WebdriverIO"

Adjust message history-fieldset-title

"Browse" could assume to browse the old wikitext of this page. Changed
it to "Search" similar to message sp-contributions-search, which is used
on Special:Contributions for a similar fieldset

Change-Id: I87b17d352999536c29df898c5822f61016bd7873

Merge "RCFilters UI: Followup Ic97c7c6aae7: Only add state message if item is selected"

SpecialRecentchanges: Unbreak $wgAllowCategorizedRecentChanges

Bug: T85742
Change-Id: I8709af6031d1cb2d1b6d25eb250c1fce150514e7

Track usage of highlight feature

Bug: T158344
Change-Id: I9a0c5e40b813e075ec33eea882b625dc43a15df6

Merge "Replace deprecated Context::getStats() with MWServices::getStatsdDataFactory()"

Merge "Avoid database connection in SpecialSearchTest data provider"

Localisation updates from https://translatewiki.net.

Change-Id: I8909bf70bc12c12dd42e318e70987b0597760b67

Merge "Remove deprecated noop functions"

Add missing @group Database tags in tests

Change-Id: Id92aab0b1f8cde6657d6558d0d82605109daa588

Localisation updates from https://translatewiki.net.

Change-Id: I525052178b2be3c22e4016cea54d96ea6e687a87