Make (Title|User)Array implements Countable

count( $results ) always returned 1, now expected number

Bug: 58377
Change-Id: I7d6b3bc1b09b46fe60cb5a74c25f9b28fffa5b73

Release notes for Iabf4873f

Change-Id: Ia20d0b9eb7e671b9b93bf459ddb42033f8ab9a46

Break long lines and remove some spaces in QueryPage.php

Change-Id: Ia4840580ada4b42bb70d38965f7955d175db9247

filebackend: exception handling cleanups

* Split out FileBackendException class and reduced direct use of MWException

Change-Id: I325c1798b6d90972c12a5dccc37989af34d857f3

Made redis lock manager get EX/SH locks in one go

* Also made the TTLs properly per-lock as they should be
* Also properly extract type/session from the keys in LUA

Change-Id: I4608b7d551ac7aa4b3f7e2f5ce92b50662b1d4e4

Include error code in error message when convert fails

This info is logged, but it makes it much easier to debug
things if the info is right there.

Change-Id: I3a331a19a13a932c6b7aefdd9885ad351f3f48e3

Release notes for I0ef9674e

Change-Id: Id25fe69c8b48c9a681d165699876736be2e319c3

Merge "Remove unused message 'confirmemail_error'"

Set 'watchcreations' and 'watchdefault' options to true

Also modify unit tests to be more resilient to various
other tests automatically having their pages being
watched.

Co-authored-by: Bartosz Dziewoński <matma.rex@gmail.com>
Co-authored-by: Brian Wolff <bawolff+wn@gmail.com>
Bug: 45020
Change-Id: I458783f02813ff023b2d142b57b44e5ec3ba30b9

Merge "Add PLURAL support to youhavenewmessages*"

Remove unused message 'confirmemail_error'

Follows-up r33333 (41d48826812b).

Change-Id: I1bbb828202ea26d0a031270ba78608d4fb238ae0

Merge "Remove $dbw->close() call from convertLinks.php"

Merge "Remove 5 unused block messages"

Merge "Set up local date formats for Sorani Kurdish (ckb)"

Merge "Break long lines and remove some spaces in Revision.php"

Break long lines and remove some spaces in Revision.php

Change-Id: If62a01096319dddaa898daaabdd6e881316a0ad7

Break long lines and remove some spaces in Defines.php

Change-Id: I3963cb55b8bac6abe02ef5ebc8327fe1c1793d0d

Remove $dbw->close() call from convertLinks.php

This confuses LoadBalancer; after this script is run from the
updater, the closed DB connection is (unsuccessfully) reused.

Change-Id: Ie9b25b73873fcddb062037e34877afeb75c5d081

Set up local date formats for Sorani Kurdish (ckb)

Including Iranian (Persian) and Islamic (Hijri) calendars.

Please note that the inclusion of the anglophone "dmy" and "ymd"
formats was by specific request of a Sorani Kurdish speaker. ckb
is allied to fa, which is one of many other languages to include
the anglophone formats alongside native ones. Please read the
discussions at the bug for more information.

Bug: 52009
Change-Id: Ib20e9185842a15fab6bbebd6e6e881654f276a0a

Merge "Remove unused message 'rcnote'"

Merge "Remove unused message 'limitall'"

Merge "Remove unused message 'undeletereset'"

Merge "Remove unused message 'compare-selector'"

Localisation updates from https://translatewiki.net.

Change-Id: Id32d5c87533081016e6a00312703f450782a1d6d

Remove unused message 'undeletereset'

Follows-up r88885 (36b95353bba3).

Change-Id: Ie5f8c6710b93309c54b34cf573d3f802b551ff07

Remove unused message 'limitall'

Follows-up r64181 (744342366b19).

Change-Id: I6411585f270bb1de8d3bd1a2e71bc01f8bc25695

Remove unused message 'compare-selector'

Follows-up r79752 (128f33aa5edc), which moved the text boxes
(from inside the fieldset to which this label applied) into two
separate fieldsets.

Change-Id: I6bad9167fe39cec28d14795ef0b87713e7ac2346

Remove unused message 'rcnote'

Follows-up r102284 (703dd35a7b2f).

Change-Id: If5795652e4276112dee7167add433deb30d31da0

Remove 5 unused block messages

Follows-up r83786 (78059976086f):
* blockip-title
* ipbreasonotherlist
* ipbotheroption
* ipbotherreason
* cant-block-while-blocked

Change-Id: Ic2bb1261dbc9a1fa46ed551159e8ce8ceff774bf

Merge "mediawiki.page.ready: Fix whitespace in IIFE"

Merge "Normalize newlines in DjVu text-layer metadata."

Merge "Make a single colon an invalid title in php and js"

Merge "Add UI to discover translated SVG files."

Add UI to discover translated SVG files.

Currently we support rendering SVGs in multiple languages,
but there is no mechanism for users to discover what languages
a file is available in. Show this information on the image page.

At the moment, if unspecified we always default the language to
english (I believe to avoid mass cache splitting, especially
if most languages wouldn't have a translation of the file in
their language). This code was written in such a way so that
this assumption should be changable in the future if we so
desire.

Long term, Jarry has a super cool svg translation extension
which would take over some of this. However I still believe
we should have an interface for this in core, since we do
support the different language renderings in core.

Change-Id: I84506436514e09d71200aa2db3932aa001b55c71

Extract list of languages svg is translated into.

SVG files support having different renderings based on language.
We support changing the rendering language, but we offer no
discoverability of what languages are available.

Long term Jarry's TranslateSVG will hopefully be used and have
all the associated awesomeness. In the mean time, we should
probably have some indication on the image page what languages
are available for the SVG. This is step 1 (extract the language).
I expect to follow this up with a commit adding some sort of
simple selector on the image page.

As an aside, it should be noted, this only detects languages
if its in the first $wgSVGMetadataCutoff (256 kb). This is a
compromise to avoid OOM on huge SVG files.

Change-Id: I2a462270fe63eb3e3023419eddc8b06f5a617ab5

Merge "Restore Signupstart and Signupend messages for account creation"

Merge "Fix SvgHandler::getScriptParams to not give warnings."

Fix SvgHandler::getScriptParams to not give warnings.

Change-Id: I664e4667bdcff89c1c5ee01a11b723a36212a57f

Normalize newlines in DjVu text-layer metadata.

Currently, newlines in DjVu text layer are stored as the literal
string '\n'. Its up to the consumer to unescape that into
a real newline. Other formats like pdfs return newlines
as an actual \n character when getPageText() is called.
I think getPageText() should not require callers to do this.

Change-Id: Ie1a438bbce5444c53ff6b7b3aaf2b5267ba3c8b4

Update documentation for media related classes

Change-Id: I7a9c8d59f88c68dc3835cb8a18f22a77cd7890fc

Merge "Declare visibility on class properties of Exif"

Merge "Add no-dimensions option to MediaTransformOutput::toHtml"

Add no-dimensions option to MediaTransformOutput::toHtml

Skips width and height attributes (useful if set in CSS).

Change-Id: Ib52182f1a56b71e381b1faa767ebad19ba92889c

Merge "Unblacklist group-specific JS/CSS for the user group"

i18n: Remove ckb from link underline removal

Just like I8168baff Central Kurdish Wikipedia
do not like removal of underline from links.

Change-Id: If55b6ea75fd2bd9e5c080d0b595a407e58d388de

Merge "Fix text extraction on DjVu so that it works beyond first call."

Fix text extraction on DjVu so that it works beyond first call.

This was caching the metadata tree in-process, but it was
returning this cache even for calls for things other then this
item.

As a result, if you did $file->getHandler()->getPageText( $file, 2 )
twice in a single request, the second time it would return false.

Change-Id: I6ee4dce3177c0898e98210d7471cfadc1108ca52

mediawiki.page.ready: Fix whitespace in IIFE

Change-Id: I4ad0bc4250452f029255da83f63bcde2dd6c819a

Make a single colon an invalid title in php and js

When secureAndSplit gets a single colon as input, the leading colon
is stripped and produces a title with a empty string. This change makes
that impossible by moving the empty string check to after the substring
removal of the colon.

Bug: 54044
Change-Id: I574168c9ed281c535901c36dea7c179a4e794d20

media: limit size of stderr being logged

Whenever a rendering commands emit a ton of errors (spotted a case
generating 125MB), the call to trim() makes PHP copy the stderr which
might exhaust the remaining available memory.

The new constant MediaHandler::MAX_ERR_LOG_SIZE = 65535 is used to
substr() the error log before it get trimmed and logged.

bug: 57985
Change-Id: Ibd4cf0331529d323a68273163ac230b07a6c6108

generalize BitmapHandler::logErrorForExternalProcess

BitmapHandler::logErrorForExternalProcess is a wrapper around
wfDebugLog( 'thumbnail' ). It got copy pasted from some other class at
one point.

This patch move the method up to general class MediaHandler and makes
other child class uses it.  The method will thus be available to
extensions such as TimedMediaHandler.

The reason I am doing that is that trim($err) generates a copy of $err
which causes a memory allocation fatal error whenever $err is larger
than the remaining memory allocatable.  The patch will let us fix the
bug by only altering one part of the code.

bug: 57985
Change-Id: I5657f07d6e2cca05d53f2a5c30ec23622c171343

Move init of $noFollow outside of if

Bug: 58044
Change-Id: I5655a0dc64055f3ec5d0a287aa8e0c85c11d3a1f

Merge "Remove mergeExpected and assertResultRecursive"

Merge "Remove 3 unused movepage messages"

Merge "Remove unused message 'resetpass'"

Unblacklist group-specific JS/CSS for the user group

Changes ResourceLoaderUserGroupsModule so that if the
user is in the 'user' group, i.e., the user is not
anonymous, then custom JS or CSS can be inserted using
system messages in the same way as is allowed for other
user groups.

Bug: 47812
Change-Id: I0ef9674e1a9e1e5db9537ba7b9578d2b7b14fd8e

Remove 3 unused allpages messages

Follows-up r10199 (1a9a48502fff):
* allnotinnamespace
* allpagesprev
* allpagesnext

Change-Id: I1f2d0d2e7fe0719b9c6e092eae0b49d1afc4fe4f

Remove unused message 'resetpass'

This was (unintentionally?) replaced by 'changepassword' in r86280
(1a80d4637002), which renamed the special page from Special:ResetPass
to Special:ChangePassword.

'changepassword' was previously used in Special:Preferences until
r49790 (155ddf6de4a2) for a heading directly above the link to
Special:ResetPass.

I moved messages 'changepassword' and 'changepassword-summary' to
the correct blocks in MessagesEn.php and messages.inc and also
updated message documentation that referred to Special:ResetPass.

Change-Id: Ie5cbda5538f53c8e8d84c5e9d4d818fd745ff46c

Remove 3 unused movepage messages

Follows-up r4134 (751a0a35f26b):
* movedto

Follows-up r24618 (877d2bac2f0b):
* movenologin

Follows-up r35213 (c75649cf5a10):
* talkexists

Change-Id: I1db90a202f3cb0d93cecd10f6e60874d5478e50a

Merge "Simplify ternary construction"

Simplify ternary construction

Change-Id: I17872a978a5a788f10811153ea2ac9a4ef0f1f6d

Remove function call from for loop test part in GIFMetadataExtractor::readGCT()

Change-Id: Ia19ddc32212786d7050287c6e07748ea2213ed40

Small doc comment fix

Change-Id: I236379f9bcd3821df15f25df849b3eabca013c4d

Merge "Converted html cache updates to use BacklinkJobUtils"

Converted html cache updates to use BacklinkJobUtils

* De-duplicate squid purges using page_touched
* Removed excess title validity check
* Also removed broken versioning in refreshLinks jobs
* Added TODO comment about $wgMaxBacklinksInvalidate
* Other documentation tweaks

Change-Id: If91e06549e5afa2d4e27595d0e131b4d78412ad2

Merge "Update documentation for log related classes"

Merge "Change line breaks in LocalFile::recordUpload2()"

Merge "Add more type hints in comments for static code analysis"

Merge changes Id5e7b3d2,I4338caad,I95785992

* changes:
  Declare visibility on class property of FormatExif
  Declare visibility on class properties of GIFMetadataExtractor
  Declare visibility on class property of MediaHandler

Merge "Declare visibility on class props of MediaTransformOutput and MediaTransformError"

Merge "Set saner default for parameters of LogEventsList::showOptions()"

Add more type hints in comments for static code analysis

This resolves a little over 100 "undefined method" warnings on
includes/filerepo/.

Change-Id: I87b6d875a4304d7beadfbc26a66e9a3f358707e6

Remove mergeExpected and assertResultRecursive

These ApiQueryBasicTest methods are no longer used
and we have a better alternatives in the parent ApiQueryTestBase.

Change-Id: I7a81ad3dd6e880be151c1901037bab4018493bcf

Update documentation for log related classes

Change-Id: I4e00cda46cc7bbcbd17ac2d3d025a76281ee84b6

Merge "Updates for Interwiki.php"

Merge "Update documentation for File::getMetadata()"

Change line breaks in LocalFile::recordUpload2()

Change-Id: Ib27e381106d048bb0a24286a877c85f0e62408cf

Merge "Remove 6 unused revdelete messages"

Merge "Do not access MediaTransformOutput->url directly in Linker"

Update documentation for File::getMetadata()

Change-Id: I24711e009f42303774b3c79ad65c38081c7c0982

Merge "Declare visibility for class properties of PNGMetadataExtractor"

Merge "Ignore CodeSniffer warning where unavoidable in media classes"

Merge "Remove function calls from for loop test part in media classes"

Merge "Break long lines in media related classes"

Merge "Define 'MW_UPDATER' when running update.php"

Merge "Removed "Disable search suggestions" from Mediawiki Preference"

Localisation updates from https://translatewiki.net.

Change-Id: I91af046a42d410457d6ccb6ce44657a29d184434

Merge changes Ic0eaac33,Iaa81af5b

* changes:
  Remove PHP4 style constructor
  Update formatting for media related classes

Merge "Remove unused messages 'resetprefs' and 'timezoneoffset'"

Merge "Remove unused messages 'protectedpagestext' and 'protectedtitlestext'"

Merge "Remove 9 unused search messages"

Add PLURAL support to youhavenewmessages*

Add PLURAL support on the number of talkpage messages received in
system messages youhavenewmessagesfromusers and youhavenewmessages.

Bug: 53897
Change-Id: Ic08c51f651fe4101ed0b6fcdaea4b4ae74331f70

Remove unused messages 'protectedpagestext' and 'protectedtitlestext'

Change-Id: Ia9c4d3b75f52b990af965d5b16a0b834e257a4f4

Remove unused messages 'resetprefs' and 'timezoneoffset'

Follows-up r49790 (155ddf6de4a2).

I would have also deleted 'searchresultshead' had it not been for the fact
that the SemanticDrilldown extension still uses it (albeit incorrectly).
So for now, I am leaving that message in, adding a note that the message
is deprecated.

Change-Id: I4485c6fbe4fc7640c9cb2d765738fbec6bee26ab

Remove 9 unused search messages

Follows-up r44773 (c4c7d7444948) and r51251 (bbc07fe77605):
* searchmenu-prefix

Follows-up r50207 (abd70a35546b):
* searchmenu-legend

Follows-up r54030 (bde3b1f926a8):
* searchresulttext
* searchsubtitle
* searchsubtitleinvalid
* notitlematches
* nonefound
* powersearch
* powersearch-field

Change-Id: Id5accd7bb66684c9fa3dacb280f7ce9b2920762e

Merge "Optimized LocalRepo::findFiles()"

Remove 6 unused revdelete messages

Follows-up r51279 (bc35951c93df):
* revdelete-nologtype-title
* revdelete-nologtype-text
* revdelete-nologid-title
* revdelete-nologid-text

Follows-up r91346 (1c71a6f02aec) and e93fa029e2d6:
* revdel-restore-deleted
* revdel-restore-visible

Change-Id: I4276c0244076e49a62e7542eef8c662fba01d439

Merge "Capitalization: 'jQuery', not 'jquery', and 'MediaWiki', not 'mediawiki'"

Capitalization: 'jQuery', not 'jquery', and 'MediaWiki', not 'mediawiki'

Change-Id: I7b53ac1340ef5fbc36d2ba8b6b2e68328597eaee

Merge "Remove unused local variable"