Merge "Expand the protocol for proto-relative links when printing"

Localisation updates from https://translatewiki.net.

Change-Id: I8a1715baff4ddbc098e202232db3b1c87d01ccef

Expand the protocol for proto-relative links when printing

Things like "User's Guide (//meta.wikimedia.org/wiki/Help:Contents)"
(on the default main page) don't look like the expected behavior to
me, and I think regular users expect links to start with either "www"
or "http".

Let's assume HTTPS for these and prepend the protocol to the URL.

Change-Id: If48f61de26a37eddbb134b824186706c08dabb72

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

New changes:
351db20 Localisation updates from https://translatewiki.net.
1a21253 Set up jsduck
d492a0f Revert "Add aggregate method"

Change-Id: Ida003c21a1f748f78ac0b36511d86e9dd3f8da58

Merge "Lowered limit for wfThumbIsAttemptThrottled() to 4"

Lowered limit for wfThumbIsAttemptThrottled() to 4

Change-Id: I2d91647abdaff820eb87b0bd45505ffa1fbc25fe

Merge "More Profiler class refactoring"

Merge "Update formatting for languages/utils/"

Merge "Remove underscore from classes CLDRPluralRule*"

Merge "PostgreSQL: Improve speed for page edit in imports"

More Profiler class refactoring

* Factored a ProfilerStandard class out of Profiler
* Profiler is now an abstract base class
* For b/c, using Profiler as the profiler switches to ProfilerStub
  to avoid trying to instantiate an abstract class
* Also factored out a small TransactionProfiler class
* Fixed fatal in "new ProfilerStub()" line in getInstance();
  Profiler has long since type hinted $params as an array
* Removed bogus debug var usage left in ProfilerMwprof::getRawData()
* Removed useless mTimeMetric stuff

Change-Id: I854dabba6c8b1fb7d92baca59ebd4af25878d11d

Merge "thumb.php and img_auth.php cleanups"

Update formatting for languages/utils/

Change-Id: I54a19cff97067b71cb6531bc37802adb2d6c52ba

Remove underscore from classes CLDRPluralRule*

Part of program to remove underscores from class names. Checked core and 675
extensions for occurrences. All uses are in core in core are updated in this
patch. There are no uses in Gerrit extensions.

Change-Id: Id3ff16710a11ce1909a83b58505b250736a073e8

Merge "Make room for preloadFileStat() call in FileBackend::doOperationsInternal"

Merge "Bail out on FileBackend operations if the initial stat calls failed"

Merge "Update formatting of includes/title/"

Merge "Avoid image table updates on file upload failure"

Merge "Call wfThumbIncrAttemptFailures() in thumb.php on fatals"

Call wfThumbIncrAttemptFailures() in thumb.php on fatals

Change-Id: I3dbe07d12abdc265d377cd05bb87c837df156995

Merge "Support tighter rate limiting for "non-standard" thumbnails"

Update formatting of includes/title/

Change-Id: I862dde47b569ca99f098d795e13b1999a33d790b

Make includes/title pass phpcs-strict

Change-Id: Iac03d1f3d4ee622496b3cb0bf8086d2f362b52af

PostgreSQL: Improve speed for page edit in imports

Whenever a new revision is added, a deferred update gets enqueued.
When it is fired, it clears the searchable text from all earlier
revisions for the article.  This becomes very slow for articles
with long revision histories, as it re-clears the textvector even
when it has already been cleared by earlier actions.  This leads to
very high load in the database for runs of importDump.php

This patch improves this situation by adding a condition to the WHERE
clause such that it does not update rows in which the textvector
is already NULL.

PostgreSQL cannot automatically remove such degenerate updates
in general because the updated rows must be locked and have their
transaction markers increased.  However, in this particular case
those things are unimportant.

This change improves the performance of importDump.php on a
wiki with long revision histories by 7 fold, and moves the
major bottleneck from the database to PHP.  It might also
improve the performance of ordinary page edits, but that
was not tested.

There are more improvements that could be made here.  For example,
a partial index or expression index could make it so that already
cleared rows do not have to be visited at all.  Or the deferred
update mechanism could have a notion of "idempotency" so that many
indentical updates enqueued during bulk loading would be condensed
to only a single execution.  However, this change is very much
simpler and is sufficient to shift the bottleneck elsewhere.

Change-Id: I458603767c6a86425010d02ffc1f8079c4b3c9a0

thumb.php and img_auth.php cleanups

* Specifically check isDeleted() instead of relying on the file
  being moved to the deleted zone alone.
* Made img_auth.php skip some unnecessary checks for public wikis.
  Also made it set the private cache control headers only if needed.

Change-Id: If82589acd53c02943d5f3bbafd29381fd63f8f68

Merge "Deprecate ResourceLoaderGetStartupModules"

Deprecate ResourceLoaderGetStartupModules

In bug 63240, Roan calls it a 'misfeature', which I think is exactly right. It
has been used as a lazy workaround for having to think about load-order,
dependency management, and race conditions.

Bug: 63240
Change-Id: Ic48ad39c6d3d166d6bb8f60dfdd5f95aa7ed16a6

mediawiki.mixins.less: Correctly embed images in .background-image-svg

Using the /* @embed */ annotation twice for the same property
(background-image) confuses the LESS compiler, resulting in one of the
images not being embedded.

Let's use the embeddable() and embed() MediaWiki LESS functions,
like in the .background-image mixin.

And let's extend and clean up the documentation while we're here.

Bug: 61941
Change-Id: I39ea2004b9db34d096eed08b3ffdfcaf8d5fcd7d

Localisation updates from https://translatewiki.net.

Change-Id: I7816dbccd7d162fc934cf36ff7c23d3b12f0fa9b

Merge "Allow additional interwiki prefixes on local interwiki links"

Merge "Fixed some @params documentation (includes/api)"

Merge "Fixed some @params documentation (includes/[db|installer])"

Merge "Fixed some @params documentation (includes/[file...|upload])"

Merge "Clarifying double-redirect messages"

Clarifying double-redirect messages

Change-Id: Ifbe13b4ed98d34ecbb6dc5b583422c2dc4ff307e

Merge "jquery.suggestions: Handle CSS ellipsis when calculating suggestions' widths"

Merge "Fixed some @params documentation (includes/*)"

Fixed some @params documentation (includes/*)

Swapped some "$var type" to "type $var" or added missing types
before the $var. Changed some other types to match the more common
spelling. Makes beginning of some text in captial.
Also added some missing @param.

Change-Id: Iced714bca004756b461b66067a49a925a7e3b877

Localisation updates from https://translatewiki.net.

Change-Id: I58e3db9db609509bd6c4de229c13524bc262cc88

Fixed typo of encyclopedic in qqq.json

Change-Id: I78861c584d224e915c015172dbc1430749583753

Fixed some @params documentation (includes/[file...|upload])

Swapped some "$var type" to "type $var" or added missing types
before the $var. Changed some other types to match the more common
spelling. Makes beginning of some text in captial.
Also added some missing @param.

Change-Id: I8804ebe0922d3a414863b162a2110e0b9e49b80f

Fixed some @params documentation (includes/[db|installer])

Swapped some "$var type" to "type $var" or added missing types
before the $var. Changed some other types to match the more common
spelling. Makes beginning of some text in captial.
Also added some missing @param.

Change-Id: I3595e9aac69ac42fbb74b0783fae0ad9bcc831aa

Merge "Fixed some @params documentation (includes/[specialpage|specials])"

Fixed some @params documentation (includes/[specialpage|specials])

Swapped some "$var type" to "type $var" or added missing types
before the $var. Changed some other types to match the more common
spelling. Makes beginning of some text in captial.
Also added some missing @param.

Change-Id: Icf6f36bb53322b39cd5c89523dbd0e4ab10b9ec9

docs: closure -> Closure; callback -> callable

Changed closure to capital word Closure in doc and type hint,
also changed callback in docs to callable

Change-Id: I52c8e8f13d38a837052101c38b9986be780ca057

Merge "Purge removed OOjs UI files"

update-oojs-ui.sh: Replace target dir instead of adding file copies

* A command like "cp dist/* $TARGET_DIR" does not remove files
  that were deleted in the release dist. Using rsync instead.

* Use bash -n or -z instead of "" and "x$VAR" == "x" tricks.

* The "git commit <dir>" doesn't properly take care of staging
  deletion of files removed by the update script, nor does it
  reliably (?) stage new files.
  Instead using a combination of "git add -u <dir>" (The -u
  stages deletion and modifcation of tracked files), and
  "git add <dir>" (stages modification and creation of files in
  that directory).

* Use "grunt test" instead of "grunt <default>" to make more
  explicit what is going on (no change in behaviour, default=test).
  And document why we don't use plain grunt-build without tests.

Change-Id: Id97a66df64d9e43e3c05388486b9f959108363c1

Purge removed OOjs UI files

The old update script didn't remove files in git if there were absent in
a newer version and left these behind.

Change-Id: I087657183c29ff7d9e2b636171961db62687b32f

oojs-update.sh: Replace target dir instead of adding file copies

* A command like "cp dist/* $TARGET_DIR" does not remove files
  that were deleted in the release dist. Using rsync instead.

  Though addition can happen, deletion should never happen as we
  refer to these files in Resources.php. But we also shouldn't
  leave behind old files mixed with new files, so delete them.
  Our structure unit tests for resources take care of catching
  references to files that don't (or no longer) exist.

* Use bash -n instead of != "".

* Refactor slightly to make it re-usable in other projects without
  too much modification (only REPO_DIR and TARGET_DIR need updating,
  the rest is generic now).

* The "git commit <dir>" doesn't properly take care of staging
  deletion of files removed by the update script, nor does it
  reliably (?) stage new files.
  Instead using a combination of "git add -u <dir>" (The -u
  stages deletion and modifcation of tracked files), and
  "git add <dir>" (stages modification and creation of files in
  that directory).

Change-Id: Ieb20978c887c5a141f31cab704b8d239f1572af0

Make room for preloadFileStat() call in FileBackend::doOperationsInternal

Change-Id: I60f71155abcf7f69423639b10ec301ac192df728

Merge "Fixed some @params documentation (includes/[cache|objectcache])"

Merge "vector: Do styles the ResourceLoader way"

Fixed some @params documentation (includes/[cache|objectcache])

Swapped some "$var type" to "type $var" or added missing types
before the $var. Changed some other types to match the more common
spelling. Makes beginning of some text in captial.
Also added some missing @param.

Change-Id: I86fd10e3f2d4bb80e7432533038d124693acfb3c

Improved HashRing usage in JobQueueFederated

* Added proper ejection and caching in HashRing to avoid rebuilding
  the hash all the time (or doing manual caching).
* Made JobQueueFederated blacklist failing servers for a few seconds.
* Also made the JobQueueFederated root job methods properly respect
  the weights when they fail over.

Change-Id: Ifa4c03272c1777cfff2523ab21f780074ddcf359

Add forgotten OOjs UI i18n files

Change-Id: I8387150a9caf09c16de2ad816ba5dcaa8b56af23

jquery.suggestions: Handle CSS ellipsis when calculating suggestions' widths

This is so nasty :(

Bug: 64093
Change-Id: Ia6b1376dddf69a0faa214d5c9a756bdcd9658367

Update OOjs UI to v0.1.0-pre (989950a4db)

New changes:
ff327f2 Localisation updates from https://translatewiki.net.
21bbbdb Add getCurrentItem method
ca0e453 Remove animation of grid layout inside booklets
b1c75fa Add aggregate method
df59f32 Add push/pop pending to Dialog
21c6d92 Make dialog overlays... overlaid?
989950a Localisation updates from https://translatewiki.net.

Change-Id: Iff681390616a2fa9ce5becf846365e7421f28433

Merge "Remove # from dataProvider"

Merge "Remove superfluous # at end of line from languages.inc"

Bail out on FileBackend operations if the initial stat calls failed

* The preloadFileStat() call acts as a good canary for whether all
  affected servers are up. If anything failed in it, then it's not
  worth bothering to send the actual write requests.
* Also made FileBackend::preloadFileStat abstract while at it since
  the two subclasses implemented it and so should any others.
* Fixed a silly comment typo.

Change-Id: I5bd427f654aa4a9d6bfe4ed7566276e8ac520b30

Avoid image table updates on file upload failure

* If the publish() call did not really do anything, then there is
  no use in saving the page and file DB records. Previously the
  user would get an error but the description page would be made.

bug: 64071
Change-Id: I1959f8c1ef33dbd1b238f6020117d2862d4b0fd3

Merge "Don't cache $wgResourceLoaderLESSVars in a static"

Merge "WebInstallerOutput: Fix path to jquery.js"

vector: Do styles the ResourceLoader way

It used to be done like this a few releases ago, but went
through some perturbation when the LESS rewrite happened
and afterwards during beta-testing the typography refresh.

The current way causes enough issues that I think the gain
from getting rid of it is worth losing some brevity.

Bug: 56592
Change-Id: Ibeff450fad789fbc67d95a1a9befd315f7405421

Localisation updates from https://translatewiki.net.

Change-Id: I6945aedd9791855d4ed1cc62738d1b206d7b950e

Don't cache $wgResourceLoaderLESSVars in a static

Caching the value of $wgResourceLoaderLESSVars in a static variable that is
enclosed in a closure makes it harder to reset global state for test runs, and
it does so in the name of a performance benefit that is miniscule.

Change-Id: I0958b03818f56ab73c8c9124daa9e54cc59f2428

Merge "Make HtmlFormatter return removed elements"

Merge "Made runParallelBatches() fully set the status array on abort"

Merge "mwjsduck-gen: Don't fail when running from different directory"

Made runParallelBatches() fully set the status array on abort

Change-Id: Ie35b28b85e734db6910a42ed384374ba3d594a8e

Remove # from dataProvider

Change-Id: Ie5414173b95e846d735827bffa34c73698e48c17

Remove superfluous # at end of line from languages.inc

Change-Id: Ia625e8de5e4d04dd35b537fd905af53b66f4026b

Merge "Remove 2 unused Special:Search hooks"

Cleanup SpecialAllmessages.php according to conventions

Change-Id: I21a6d4359faf9d86ce3b01e423e8c3ab4c02658f

WebInstallerOutput: Fix path to jquery.js

Follow-up to Id5ead145 and I335e4c08.

Bug: 64091
Change-Id: If40f479df7d68071d6ce5d2f3b8251e6bf3eb80e

Remove 2 unused Special:Search hooks

Nothing is listed on mw.org as using these and nothing in Git uses
them either. Less hooks is a good thing, we've still got 11 here :(

Change-Id: I2dd1dbb269820b192f8d9e9a496a161f4ef851bf

Merge "Simplify SpecialVersion:listToText()"

Fixed some @params documentation (includes/api)

Swapped some "$var type" to "type $var" or added missing types
before the $var. Changed some other types to match the more common
spelling. Makes beginning of some text in captial.
Also added some missing @param.

Change-Id: I758fa4ad80ac95e2ddd3770bcb9b7d2e57ec34ea

Merge "Make API action=rsd work on private wikis"

Make API action=rsd work on private wikis

Tweak ApiRsd's settings so that it doesn't require read permissions to function.

Bug: 64085
Change-Id: I15288458ec3b0b583eece930d323f80314bb6ee3

Make HtmlFormatter return removed elements

This shouldn't cause much overhead and is useful for getting the text of
the removed tags.

Change-Id: I97cf66014719244b8bb2b0509b419c82202bdb01

Merge "Fixed some @params documentation (maintenance)"

Merge "Update documentation for $wgExtensionMessagesFiles"

Merge "Remove #bodyContent { width: 100%; }"

mwjsduck-gen: Don't fail when running from different directory

* Do not directly use $0 in help output, use basename instead
* Do not use paths relative to the currect directory

(Full disclosure: I can't write bash.)

Change-Id: I4c73155cd691a2ddf134724e9df94e6594b78a02

mediawiki.Title: Correct documentation structure for Title.exist

Apparently the way we were using the documentation, with sub-properties
of properties, only worked due to the way that JSDuck v4.x happened to
be implemented, and was unsupported; as of JSDuck v5.x this no longer
works, so instead, change the form used so as to document the property
as a single item, as suggested in response to our complaint:

  https://github.com/senchalabs/jsduck/issues/551

Change-Id: If0c66e6e8a904769ee857c1ee94377dc13000d41

Apply coding conventions for whitespaces

Change-Id: Ibd32177ac1f608fbb480d74a573c20b6487d28c4

Fixed some @params documentation (maintenance)

Swapped some "$var type" to "type $var" or added missing types
before the $var. Changed some other types to match the more common
spelling. Makes beginning of some text in captial.
Also added some missing @param.

Change-Id: I727deec35a712de0f0c676cc87dfa661f1ee965b

Update documentation for $wgExtensionMessagesFiles

Follows-up e0043257c02e.

Change-Id: Id8212fca83fde09a89891ead1e0a107ff0d66398

Merge "Add 'mw-anonuserlink' class for anonymous users"

Merge "includes/Linker.php: Added hook for "Media:" links"

Simplify SpecialVersion:listToText()

Don't special case $list only having one item

Unindent else block

Bug: 64069
Change-Id: I171a64147fe00f00ac0523ccf58fea11e800e2fd

Merge "Fixed some @params documentation (tests)"

Fixed some @params documentation (tests)

Swapped some "$var type" to "type $var" or added missing types
before the $var. Changed some other types to match the more common
spelling. Makes beginning of some text in captial.
Also added some missing @param.

Change-Id: Ic8aaf0a93796b97d0fa4617c1f86ff59f4b36131

Localisation updates from https://translatewiki.net.

Change-Id: If20be037a2feffec2a926bc51a66b4f11235bd4f

Support tighter rate limiting for "non-standard" thumbnails

* Also avoid passing some junk params to File::thumbName()

Change-Id: I7e89f72ade47be7fcc7bf33f0afb970183aac936

Merge "mediawiki.toc.test: Cleanup"

Merge "Removed 'Remember my login' preference"

Merge "Remove deprecated function mw.util.toggleToc"

mediawiki.toc.test: Cleanup

Used consistent wording, fixed some typos ("visible" and "hidden" were
swapped in one place, etc.) and simplified the flow.

Change-Id: I7745aa48484fb1a36f8b2654f2a3df326b43c9c4