Merge "Fix "UTPage" creation in tests"

Merge "Explain make kss environment variable in README"

Merge "Drop -{webkit,moz}-box-shadow"

Fix "UTPage" creation in tests

Change-Id: If414c633a3abb6019f9d677b70e6bc790c0c241d

Merge "API: Don't rollback changes if the exception was a UsageException"

API: Don't rollback changes if the exception was a UsageException

In I1b7396ceb, we were trying to log failures to Special:Log, but
since $this->dieUsage throws an exception, the log entry insertion
was being rolled back.

This is similar to I36cd645d which was for index.php calls.

Change-Id: I136ba66d2f939fcdd0c9326e04d686a30449bf4c
Follows-Up: I8f1da51187b281fe4afc0d5a0c49f5caf3612e92

Merge "Made TempFSFile try to purge files on fatals too"

Merge "Make languages/utils pass phpcs-strict"

Merge "Remove unused message 'livepreview-loading'"

Merge "Make languages/classes pass phpcs-strict"

Merge "Include pool type in pool counter log entries"

Merge "Avoid uncommitted transaction notices in thumb.php and img_auth.php"

Avoid uncommitted transaction notices in thumb.php and img_auth.php

bug: 56269
Change-Id: I8cf5c070899d281c7efbac853f04c2fea9374e4d

Make languages/classes pass phpcs-strict

Change-Id: I0985f3c7e4b36338c68a4a63cfba4eaa4af567c0

Make languages/utils pass phpcs-strict

Change-Id: I5e6521e30384f46575573647b24217d4301fcd59

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

Merge "Put class FakeConverter in its own file"

Merge "JSON i18n shim: Only register LocalisationCacheRecache handler once"

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"

Put class FakeConverter in its own file

Change-Id: I84b256d668579a76034ad08c3bb763d33e7ffe31

Make languages/ pass phpcs-strict

Change-Id: I0c4a68d140fae27857cbc3684fe51d7880d92118

Remove unused message 'livepreview-loading'

Follow-up to 53bd3da58fc5c3f. The message has been added to
SemanticMediaWiki in change I14448aecbbef2cfcfb5f47bd882046f0cc3c3ad1

Change-Id: I76d10534c64d3308423611a8288f7f7335cff1d3

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"

Explain make kss environment variable in README

Change-Id: I9e83187050ed360c57a1c3d4aeba913d69211349

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

Made TempFSFile try to purge files on fatals too

Change-Id: I3e80bea0d869be989c951dfb35a218bb66c9de17

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

Include pool type in pool counter log entries

Change-Id: I1b0213b5af755208843cb2cd02d3446c480f6882

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

Drop -{webkit,moz}-box-shadow

* The last version of Firefox to require the 'moz' prefix was 3.6, 25(!)
  versions ago.
* Most recent versions of WebKit that require the vendor prefix: Chrome 9,
  Safari 5.0, iOS Safari 4.3, Android 3.0.

Change-Id: If0c484444ef9db2f5435193cfe7a2e445862e956

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