Exit populateIpChanges maintenance script if there are no revisions

Bug: T188225
Change-Id: Ice1bdae3d16cf365da14c6df0e8d91d2b914f064

Merge "Chinese Conversion Table Update 2018-1"

Merge "SpecialPageAliasTest: Fix arguments of Language::fetchLanguageNames"

Localisation updates from https://translatewiki.net.

Change-Id: I1eed5fd6944bbd829ed1fbb5950e89c8b85c87f6

Chinese Conversion Table Update 2018-1

1. Update the Chinese conversion table routinely to fix bugs reported from https://zh.wikipedia.org/wiki/Wikipedia:%E5%AD%97%E8%AF%8D%E8%BD%AC%E6%8D%A2/%E4%BF%AE%E5%A4%8D%E8%AF%B7%E6%B1%82 .

2. Fully support the conversion of Chinese characters included in "Table of General Standard Chinese Characters" (通用规范汉字表).

It is only data changes and only works for Chinese WikiProjects.

Change-Id: I7f94eb01b3b50427cbb96084152e574b066d5fbf

SpecialPageAliasTest: Fix arguments of Language::fetchLanguageNames

Language::fetchLanguageNames( 'mwfile' ) means all languages with the
default filter 'mw' and names in the language 'mwfile'.

Language::fetchLanguageNames( null, 'mwfile' ) means language all
languages with the filter 'mwfile' and names in the default language.

This change removes the test for the language codes:
* aa
* als
* bat-smg
* be-x-old
* cho
* fiu-vro
* ho
* hz
* kj
* kr
* mh
* mus
* ng
* no
* rn
* roa-rup
* shi-latn
* shi-tfng
* simple
* tum
* uz-cyrl
* uz-latn
* zh-classical
* zh-min-nan
* zh-yue

Change-Id: I7266a67e37862daf863d1565d84cfeebaf5cb680

content: Fix @return doc for AbstractContent::getDeletionUpdates()

If a subclass re-implements this function, then there's no guarantee
that it will be returning instances of LinksDelet

Change-Id: I09a0c0a786b687997e61344847f8af08b20cebf2

Fix phpdoc for Maintenance::getDB

It has the same parameter as wfGetDB including $groups as string

Change-Id: Id531488e1c2bddce71946423010620323e08bc0a

Don't suggest using extract()

Especially for things as trivial as this.

Bug: T28496
Change-Id: Id4bc7f0e6411e943dff45e45964c08d44e6ccb96

Remove redundant item loading code in User::createNew

These lines were removed in 27c61fb1 but left in when I rebased
61b0c193 due to the conflict. The load() call handles all these
fields, so there is no need to set mName and the load state flag.

Change-Id: Id751241abf8d5592309744e96493122c6bc65f2b

Force READ_LATEST for User::newFromId() if writes had been done

The User::newFromName() case already does this, there seems to be no
reason not to do it for User::newFromId() too.

Bug: T188014
Change-Id: Ic7fdef0cc1f5750cb5e6b2a7f48f1549862b41cb

Have User::createNew() load the object from master

When the new User is created, it's leaving it to be lazy-loaded from a
replica. That seems to be causing attempts to add groups immediately
after creation to fail because the load-from-replica doesn't find the
just-created master row.

Bug: T188014
Change-Id: I841c434086bfaaca1cf1ce23673f32dc5a77915d

Merge "wgGroupPermissions: Don't grant editmyusercss/editmyuserjs to anons"

Merge "Add `actor` table and code to start using it"

Allow limiting comment length by characters rather than bytes in JS

For unfortunate historical reasons, browsers' native maxlength counts
the number of UTF-16 code units rather than Unicode codepoints [1],
which means that codepoints outside the Basic Multilingual Plane
(e.g. many emojis) count as 2 characters each. That could be good
enough, but we want our software to be excellent rather than merely
good enough.

1. Introduce a few new functions, added to the existing modules:
   * mediawiki.String:
     * codePointLength() counts the length of a string in Unicode
       codepoints (characters).
     * trimCodePointLength() trims a string to the desired length in
       Unicode codepoints (characters).
   * jquery.lengthLimit:
     * $.fn.codePointLimit() enforces the specified maximum length in
       codepoints of an input field.
   * mediawiki.widgets.visibleLengthLimit:
     * mw.widgets.visibleCodePointLimit() enforces the maximum length
       in codepoints of an OOUI widget and displays the remaining
       length in an inline label.

2. Add client-side mw.config variables:
   * wgCommentByteLimit for the old byte limit, equal to 255.
   * wgCommentCodePointLimit for the new codepoint limit, equal to 1000.

   Only one of them may be set, depending on which limit should be applied.

3. Make use of both of these new features. For the sake of an example,
   I updated the forms shown on action=edit (using visibleCodePointLimit)
   and on action=protect (using codePointLimit). Many remain to be updated.

[1] https://www.w3.org/TR/html5/sec-forms.html#limiting-user-input-length-the-maxlength-attribute

Bug: T185948
Change-Id: Ia1269fd898dabbcf1582618eab46cef97e10a3b1

Merge "mediawiki.mixins: Extend `hyphens()` mixin for `none` case"

Merge "Localisation updates from https://translatewiki.net."

Localisation updates from https://translatewiki.net.

Change-Id: I1eaeb45405e151b5c25afe598a70d041453fc9d0

Merge "Adjust tests for JPEG segmentation fix"

Merge "rdbms: make DBMasterPos implement Serializable"

rdbms: make DBMasterPos implement Serializable

ChronologyProtector uses these classes to briefly store positions
and everytime the fields change then errors can happen when old
values are unserialized and used. Use a simple two-element map
format for serialized positions. The fields are recomputed back
from the data map.

Values from before this change will issue the warning
"Erroneous data format for unserializing". To avoid that, bump
the ChronologyProtector key version. Future field changes will
not require this.

This change should be deployed on all wikis at once.

Bug: T187942
Change-Id: I71bbbc9b9d4c7e02ac02f1d8750b70bda08d4db1

mediawiki.mixins: Extend `hyphens()` mixin for `none` case

Slightly extending `hyphens()` mixin with Less mixin guard for
`none` value “reset” case.

Change-Id: Iebb3f5190d31173ea6d843e0434f794f3b3c8e8a

Rename jquery.byteLimit→lengthLimit, mediawiki.widgets.visibleByteLimit→visibleLengthLimit

In change Ia1269fd898dabbcf1582618eab46cef97e10a3b1 I want to add
functions that deal with codepoints instead of bytes to these modules,
after which the names wouldn't make sense. Doing this in a separate
commit to make the diffs clearer.

Change-Id: Ia554eb2265248e72b04fce69a662a9db1a5f1275

Add `actor` table and code to start using it

Storing the user name or IP in every row in large tables like revision
and logging takes up space and makes operations on these tables slower.
This patch begins the process of moving those into one "actor" table
which other tables can reference with a single integer field.

A subsequent patch will remove the old columns.

Bug: T167246
Depends-On: I9293fd6e0f958d87e52965de925046f1bb8f8a50
Change-Id: I8d825eb02c69cc66d90bd41325133fd3f99f0226

Adjust tests for JPEG segmentation fix

Added a second test case truncated in a different place, and
added some clarifying comments.

Change-Id: I7b8e5296a203264b7e7e428f82c8c948242a1272

Merge "Revert "JobQueueGroup: Improve failed-to-insert error""

Revert "JobQueueGroup: Improve failed-to-insert error"

I made a useless commit.

This reverts commit a82cb5aa18e2d5dc886d51f71b224dbd9de83cae.

Change-Id: I70328bcbcf9af5480c7a0b8e297ed41a463fd457

Merge "JobQueueGroup: Improve failed-to-insert error"

RCLFilters specific message for invalid target page

* always hide the big red .errorbox when rcfilters
  is enabled

* always identify the changes-list section with
  .mw-changeslist or .mw-changeslist-empty

* conditionally add .mw-changeslist-empty-<reason>
  to the changeslist section when the reason for it
  being empty is known

* handle RCL being empty because the specified title
  is invalid or inter-wiki

Bug: T184952
Change-Id: I5dd974f5f769503e89301dd22bdfd3d49b0dd11f

JobQueueGroup: Improve failed-to-insert error

This could possibly use LoggerFactory & friends to use them as
parameters, but either way let's sort the entries so they're
at least remotely capable of self-grouping.

Change-Id: Iaf435093d70add02e2f82e3037c64fec11870979

CSSMin: Trim whitespace from attribute selectors and url tokens

* Trim whitespaces after opening and before closing parentheses and
  brackets.
* Ensure by test case that the invalid "url (" will not become the
  valid "url(" by minification.

This change also prevents the parsing problem in Firefox 57.0
https://bugzilla.mozilla.org/1418152 which is fixed in Firefox 57.0.1.

Change-Id: I804477ba7c6363f0e964fc8c7c0bc74d2d4c1a0d

Merge "Shell: Don't hang on empty stdin"

Merge "Drop pointer-events fallback code now IE10 is Grade C"

Shell: Don't hang on empty stdin

If the write buffer for a file descriptor is empty, don't try to write
to it. Just close it and continue on.

Bug: T188019
Change-Id: Ie5b5ac1ef1aec4ae763cf4d0d58d3a28e42b7d2a

Make LocalFile check early if the revision store is available

This reduces the odds of having files without corresponding
wiki pages, given that the later is done in a deferred update.

Also made some documentation cleanups.

Bug: T187942
Change-Id: Iff516669f535713d37e0011e2d7ed285c667f1c5

Merge "Add ExternalStoreMedium::isReadOnly() method"

Drop pointer-events fallback code now IE10 is Grade C

IE10 was the last supported browser without pointer-events.

Change-Id: I8e1637f527dd058fe2a5491896b0cdd7ac65b0ba

Localisation updates from https://translatewiki.net.

Change-Id: I143b63f429ecea18b60f7b30aeb7c42b65d45bd6

Merge "EditPage: Remove unused $is*Subpage variables, deprecated in 1.30"

wgGroupPermissions: Don't grant editmyusercss/editmyuserjs to anons

Change-Id: I4b3216a06c3b214e9f0be52cfbefa9f489c910b4

Merge "mediawiki.mixins: Add `hyphens` mixin"

Merge "objectcache: improve TTL reduction warning log entries"

EditPage: Remove unused $is*Subpage variables, deprecated in 1.30

Change-Id: I9a9a9b2652f63d18634a60205a6e8a4d76380e71

mediawiki.mixins: Add `hyphens` mixin

As `hyphens` values are used in various places now, with most
recent addition I1637ce3f5bddb we should add it in a central place.

Change-Id: I660f3ecd2a4669e8d5c8415990fa2e8a25aea32a

Merge "Use PHP's implode() with the suggested order of arguments"

Use PHP's implode() with the suggested order of arguments

https://secure.php.net/manual/en/function.implode.php defines the order
of arguments as

 string implode ( string $glue , array $pieces )
 string implode ( array $pieces )

Note:
  implode() can, for historical reasons, accept its parameters in
  either order. For consistency with explode(), however, it may be less
  confusing to use the documented order of arguments.

Change-Id: I03bf5712204e283f52d3ede54af9b9ec117d4280

Merge "Title: Refactor JS/CSS page handling to be more sane"

Fix Profiler::getOutputs() type documentation

Change-Id: I4ff2ccc16363bfc3093b75263da2b09c3af40627

objectcache: improve TTL reduction warning log entries

Add "lag" and "age" parameters for more context

Change-Id: Iea029e78d4c0bbec0e4334de4fbe7780b1bea524

Add ExternalStoreMedium::isReadOnly() method

Use this to abort out of store() calls early

Bug: T187942
Change-Id: I9334d36e8bc3e4589775471eee03be4f4a3119a3

RevisionStore: Remove reference to unimplemented 'ar' blob address schema

It was thought that an 'ar:' schema for SqlBlobStore addresses would be
needed to support archive rows from before MediaWiki 1.5 that still
stored the text in ar_text and ar_flags instead of using the text table
and ar_text_id. But this schema was never actually implemented.

Now I5608c6b6 has migrated that content to the text table and
ar_text_id, and I18f1c740b will remove the database fields entirely. So
there's no reason to ever implement this schema.

Change-Id: Ic449478c244bb13a8c5139e20488e876c0a793bb

Merge "Translate some magic words to Czech"

Merge "rdbms: add type hint to reuseConnection()"

Merge "rdbms: minor improvements to LoadBalancer logging"

Merge "rdbms: use DEBUG level for more ChronologyProtector log entries"

rdbms: add type hint to reuseConnection()

This avoids IDEA warnings.

Change-Id: Ic772d47785319f111c267461e4e42e9183d2815b

Merge "Move $.byteLength and $.trimByteLength to new module 'mediawiki.String'"

Move $.byteLength and $.trimByteLength to new module 'mediawiki.String'

These methods do not belong on the jQuery object. And to resolve
T185948, we need to also add codePointLength and trimCodePointLength,
and this new module seems like a good place to put them.

There is no `mw.String` global, this module has to be used via `require()`.

Deprecations:
* Function `$.byteLength` (from module 'jquery.byteLength') is
  deprecated, use `require( 'mediawiki.String' ).byteLength` instead.
* Function `$.trimByteLength` (from module 'jquery.byteLimit') is
  deprecated, use `require( 'mediawiki.String' ).trimByteLength` instead.
* Module 'jquery.byteLength' is deprecated, use 'mediawiki.String' instead.

Note that `$.fn.byteLimit` and the 'jquery.byteLimit' module are not
deprecated.

Change-Id: I2501a79efee644e5f4a9f5c977fe49c8c05c6eb3

rdbms: minor improvements to LoadBalancer logging

Change-Id: I03f54a65b1932b99ac4e129000547a6c2640c489

tests: Remove IE9 mobile from Grade A

Change-Id: I0e1da9d155c778edaa85d19a3e556fe14f83fdd2

Fix infinite loop in JpegMetadataExtractor

One of the skip-over loops was missing an feof() check and could
cause infinite loops.

Includes test file created by truncating a tiny tiny .jpeg at
the right place...

With the fix, it doesn't loop but dies on an exception, which
is good!

Bug: T184048
Change-Id: Ica13d6b68c3c12f7ce414edd081bf0886714e465

rdbms: use DEBUG level for more ChronologyProtector log entries

Change-Id: Ie1795abe52d9162af556a6534a94cc3e4a565d43

Add TimedMediaHandler's images/transcoded to .gitignore

Avoids git looking in your transcoded audio/video files
for changes...

Change-Id: I5610354a1575bb97f4523d58d3e9713512416305

Merge "sql.php: Provide --json output mode"

Merge "Drop Grade A support for IE10"

Drop Grade A support for IE10

Bug: T187869
Change-Id: I68c882e64fba95d5e1bf41f64658fcef7524dcf5

Merge "Contributions: Only add form JS when the form is added"

Merge "Update README file for Selenium tests"

Merge "Truncate tag filter descriptions"

Merge "Fix English name for lb"

Update README file for Selenium tests

Bug: T187862
Change-Id: Iadd41307a48a1b50f64a495e7efcee3b88d0ff68

Fix English name for lb

This version has been in MediaWiki longer than my email history,
since 2005 at least. This spelling is not present in
https://en.wikipedia.org/wiki/Luxembourgish

Change-Id: Ibda7e6428a2c79b9f7d88892ef1c16e9921ae934

Merge "Factor OldRevisionImporter & ImportableOldRevision out of WikiRevision"

Merge "Factor UploadRevisionImporter & ImportableUploadRevision out of WikiRevision"

Contributions: Only add form JS when the form is added

Special:Contributions can be transcluded ('including') on to another
page. In this mode, the form is not present, but the OOUI JS infusing,
payload and styling, was added regardless. This caused errors on pages
transcluding such lists, as the OOUI HTML was missing, causing the JS to
throw errors.

Bug: T187710
Change-Id: I7d45dd3ddd3ba75d44f7d24e03cef5e95301f4ed

rdbms: make sure non-native replace() uses one transaction

This is similar to what upsert() already does

Change-Id: Ide83eefe0d937fb2cdc20aa3c7dc9654c4d34beb

Merge "mw.Title: Use $.trimByteLength from jquery.byteLimit module"

Merge "tests: Remove dead code in jquery.byteLimit tests"

Merge "Switch the sidebar cache to using checkKeys"

mw.Title: Use $.trimByteLength from jquery.byteLimit module

Change-Id: I39ebd559a72ce72743cbe3bb86abe679d108d94a

tests: Remove dead code in jquery.byteLimit tests

These options do nothing.

Change-Id: I15cc91b9208e8cfd3f3f42bb0a9c91514d6532e3

Merge "parser: Deprecate string type for $lang arg in DateFormatter::getInstance"

Merge "jquery.byteLimit: Handle characters outside BMP (surrogate pairs) when trimming"

parser: Deprecate string type for $lang arg in DateFormatter::getInstance

The parameter $lang is not used with type string in the Git repository:
https://codesearch.wmflabs.org/search/?q=DateFormatter%3A%3AgetInstance
This function is only used with type Language and without parameter.

The support for the language as string will be removed with Change-ID
I677ab561d67b63f0c86f65fadf9319e41444a22a.

The removal of the support for the type string simplifies the code.

Change-Id: I0c0ab46ffc0fab6b68672940e979ad7e5754a7d1

Merge "Localisation updates from https://translatewiki.net."

Localisation updates from https://translatewiki.net.

Change-Id: Iea6405478e926c0bce200213a211bb66da7efbf3

Merge "mediawiki.cldr: Use require() for pluralruleparser"

Fix for Ia97b750b

Change-Id: I2eb5ba754334864f65dede92b138f9a81a210e70

Merge "Add support for 'es-formal'"

Merge "mw.loader: Document mw.loader.require() as private"

Factor OldRevisionImporter & ImportableOldRevision out of WikiRevision

This is to be used within the FileImporter extension to allow
adding custom loggers to this import process.

Change-Id: Ib094d4829764ccc8e5bd2619fb827d701ae06d43

Factor UploadRevisionImporter & ImportableUploadRevision out of WikiRevision

This is to be used within the FileImporter extension to allow
adding custom loggers to this import process.

Change-Id: I4a6c573fc0a69b06d696cd2afca9226fb492a9bc

mediawiki.cldr: Use require() for pluralruleparser

Use require( 'mediawiki.libs.pluralruleparser' ) instead of
mw.libs.pluralRuleParser. (Added in 1.27, with bc4e07b6f63b0)

Change-Id: I7388cf631068458596250b8da82edec3f7cbd98b

mw.loader: Document mw.loader.require() as private

Only public for debugging purposes.

Change-Id: Icc44e11ec6874d7f74ed527e87a80dd06b4edb3a

Merge "Cleanup IDEA warnings in MessageCache"

GD zero filesize error

Checking if the filesize is zero before passing to GD. This can result
in a fatal error.

Bug: T182387
Change-Id: I26a24286f3035f233de682caed21a8e8c2d5aaad

Merge "Pass change tags to NewRevisionFromEditComplete hook"

Merge "rdbms: tweak var names in LoadMonitor::scaleLoads()/getLagTimes()"

Merge "tests: Remove verbose logging settings from DevelopmentSettings.php"

tests: Remove verbose logging settings from DevelopmentSettings.php

Follows-up fdb7d36903. It seems these make unit tests on Travis CI
fail, per T187581. And when we briefly enabled DevelopmentSettings.php
in our Jenkins setup, the PHPUnit tests also failed.

Remove these from DevelopmentSettings.php to confirm that the
tests pass that way. Then, once our own Jenkins setup has switched
back to using DevelopmentSettings.php, we can re-consider adding
these in a way that is verified by Gerrit/Jenkins before merging.

Bug: T187581
Change-Id: Ib81c5909849598bdad02955f6414d68c038742c5