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

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

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

Merge "Remove useless comments from search output"

Make languages/ pass phpcs-strict

Change-Id: I0c4a68d140fae27857cbc3684fe51d7880d92118

Remove useless comments from search output

They show up when results are broken, but it's better to skip
the broken result entirely and not put in a pointless comment.

Change-Id: I691b04f31e2e439f9481c954f177e21b7f80ca2c

Merge "Remove 4 unused live preview messages"

Merge "generateJsonI18n.php: Use tabs for indentation"

Merge "Remove 3 unused upload related error messages"

Merge "Remove unused message 'dberr-header'"

Merge "Remove unused message 'notacceptable'"

Fixed slow query in LocalFileDeleteBatch::getHashes()

* All the old files will have the oi_name of the $file

bug: 62360
Change-Id: Ie3aeee44a7ec1ef4cad0c0fcde9b7d0dba003da2

Remove unnecessary h2 element on create account

We added these headings during the last account creation and login form
redesign but on recollection I'm not sure they're needed.

The messages are pretty much redundant with the h1 heading,
and now h2 elements have a margin-top of 1em it adds too much whitespace.

Change-Id: Ice5e8bf23cc00bf00835e7dc26acc33182654ec5

Merge "Remove 2 unused search related messages"

Remove 3 unused upload related error messages

Follows-up r106752 (5275f9b097cd):
* fileappenderrorread
* fileappenderror
* fileexistserror

Change-Id: I0b953827ca617ca123e1018d3e38c4df7b22a117

Remove unused message 'notacceptable'

Follows-up 4ebd4debf407.

Change-Id: I887041bad0be4f0fe6167f35c6c55e72610e2aa8

Remove 2 unused search related messages

Follows-up 7e8c48ac3d11 by removing 'powersearch-redir'.
Follows-up 11c04f9ecbaf by removing 'prefs-displaysearchoptions'.

Change-Id: I13ce4eb1f9431e23f983ec737a684213d1dcd235

Remove unused message 'dberr-header'

Follows-up 2bcffe8be420.

Change-Id: I55178414ebfec9cb6e7c0f97e3889637074556d7

Fixed some @params documentation (includes/parser)

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: I49f8f48b521878de7abd9cc40efdeff6cf9a37e0

Remove 4 unused live preview messages

Most of these are obsolete because there is no longer a separate
"Live preview" button on the edit form.

* showlivepreview
* livepreview-ready
* livepreview-failed
* livepreview-error

SemanticMediaWiki uses 'livepreview-loading', so I left that message
in for now.

Change-Id: I51d014eb2c3dcf5ff1b91a00d5dc3e5b0c580459

generateJsonI18n.php: Use tabs for indentation

Bug: 63444
Change-Id: Iaaf6b1a08c316f29c18fd5c9494decee401a43f1

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

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.

Change-Id: Ifbb1da2a6278b0bde2a6f6ce2e7bd383ee3fb28a

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

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: I7f605aa9e117b5fd80d9b1440864fe526d2b14a5

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()"