Merge "Return null from EditPage::importContentFormData"

Merge "Actually assign suppression-related rights to 'suppress' group"

Merge "Unsuppress other phan issues with low count"

Merge "Globally unsuppress phan issues with low count"

Also set $wgContLang in MWITC::setContentLang()

This assignment got lost in change Ie7a89f6ed7 (commit 47464abb4f) – the
resetServices() now implicit in setMwGlobals() resets the
ContentLanguage service, but does not update the $wgContLang global.
setService( 'ContentLanguage' ) does update it, but that method is
deliberately not used in this branch of setContentLang(), for reasons
that I assume are still relevant even though the comment explaining them
was removed – see change I8c60e37c17 (commit c70c2e4714).

Bug: T231630
Change-Id: I0170cc6bf40a5524523c45e8ffff8f72adc2fc47

Merge "Add some LocalRepo integration tests"

Unsuppress other phan issues with low count

And also update approximated counts, which for the most part are lower
than reported (hooray!)

Bug: T231636
Depends-On: Ica50297ec7c71a81ba2204f9763499da925067bd
Change-Id: I78354bf5f0c831108c8f606e50c87cf6bc00d8bd

Introduce BlobStore::getBlobBatch method

Bug: T230834
Change-Id: I56306c50a6617dc91e4eb362ef010703ac25d951

Globally unsuppress phan issues with low count

All of these suppression prevent the detection of many common mistakes,
and could easily prevent things like T231488. Especially if there are
few issues of a given type, it's way better to suppress them inline,
instead of disabling them for the whole core.
This patch only touches the one with a lower count (although those
counts may be out of date).

Bug: T231636
Change-Id: Ica50297ec7c71a81ba2204f9763499da925067bd

Add some LocalRepo integration tests

Coverage is still far from 100%, but better than nothing.

Change-Id: Ifb22491414653a5730a83434c4ea31a3540f7d26

Merge "resourceloader: Remove Grade C sniffs for WebOS <=1.4 and SymbianOS"

resourceloader: Remove Grade C sniffs for WebOS <=1.4 and SymbianOS

Added originally in c46fc745a9de7201e based on the support document for
jQuery Mobile 1.3, via <https://jquerymobile.com/browser-support/1.3/>.

These have been obsolete since T102318 because startup.js is now a feature
test, with UA-snifs only for browsers that both *do* have all Grade A
capabilities for which we still want to disable the JS pipeline.

* Palm WebOS: Upstream jQuery 1.x allegedly worked fine on WebOS 1.4+.
  Not sure why we excluded WebOS <= 1.4, looks like that should have
  been <= 1.3 instead.
  If it doesn't have the neccecary HTML5/ES5/CSS3 features,
  it will remain in Grade C without a sniff. Otherwise, it's probably
  fine to enable. Either way, not officially supported (Grade X).

  Usage on WMF wikis is presumed less than 0.1% as neiter Palm nor
  WebOS show up on https://analytics.wikimedia.org/.

  The example UAs mention WebKit/525 and WebKit/532 which
  correlate with Safari 3.x and Safari 4.x, which we know
  doesn't pass the feature test.

* Symbian:
  The example UAs mention WebKit/413 which corelates with Safari 2,
  which we know doesn't pass the feature test.

Change-Id: I62a83e3b4967f309739095d9c73650f9859ee856

Merge "logging: Deprecate LogEventsList::$mDefaultQuery member"

Merge "Improve formatting of constructor documentation for block classes"

Merge "Preserve grapheme clusters in upper corner completion suggester highlighting"

Preserve grapheme clusters in upper corner completion suggester highlighting

Grapheme clusters in Indic scripts can be split by highlighting,
resulting in really ugly text in the suggestions. For example,
म +  े should equal मे, but doesn't if highlighting markup comes between
the two characters. Affected scripts include Devanagari, Tamil, Telegu,
Thai, Myanmar, Khmer, and Javanese, and probably others. Latin and
Cyrillic script also have the problem, but the presentation is not as
bad (at least on some browser/OS combinations).

- Define a comboMarks regex (equivalent to \p{Mark} in other programming
languages) and use it in a new highlighting function,
prefixPlusComboHighlight, to include trailing combining characters in
highlighted text.

- Change the calling function to use prefixPlusComboHighlight instead of
prefixHighlight.

Note: This only fixes problems with **grapheme clusters** in
suggestions in the **search box in the upper corner** (upper right in
LTR languages, upper left in RTL languages). The main search box on the
Special:Search page uses different code. Ligatures, such as
Arabic ي + ا becoming يا, are not covered by this simple regex-based
solution.

Bug: T35242
Change-Id: I915c50180bc2196e0302f27835241624b7837f16

logging: Deprecate LogEventsList::$mDefaultQuery member

This is a follow up on follow-up-I91003a9a3f2a56114de5b

Change-Id: If670ed959bcbc3149b0979541c641ada63377b7f

Merge "Move User::getAllRights to PermissionManager."

Merge "Remove usages of Title::quickUserCan"

Merge "ContribsPage: bring back getContribs() method"

ContribsPage: bring back getContribs() method

Follows-up 73664393f8da2ec.

Apparently the method still has callers. Hot fix (UBN).

Bug: T231540
Change-Id: I09ba81fc7ac4afe4c5cc54c3a589a54e31e9c419

Remove usages of Title::quickUserCan

Change-Id: Ifa53e0ec800e23dc4184d133a100fb9378dfee9e

Merge "mime,logging: Remove unused private methods"

Merge "Integration tests for Language fallback methods"

Merge "createNoOpMock() method for PHPUnit tests"

Merge "filerepo: rename resolveToStoragePath() and tweak file operation arrays"

Merge "HashRing optimizations and PhpStorm warning work-arounds"

Merge "Remove $wgMemCachedDebug"

Merge "filebackend: fix @throws comment in FileBackend::makeContentDisposition()"

Merge "filebackend: add idiom constant to FileBackend for null results"

mime,logging: Remove unused private methods

These methods are defined but not used.

Change-Id: I91003a9a3f2a56114de5baede6a1034ee4ed8d80

filerepo: rename resolveToStoragePath() and tweak file operation arrays

This makes it clearer what the method actually does

Change-Id: Ifd69484560eb2949242927bea820c1c7786c43d2

HashRing optimizations and PhpStorm warning work-arounds

Change-Id: I640636575f06a419e0f78504d8e7ba61edfad8e8

Remove $wgMemCachedDebug

Change-Id: I98380ac1cb66e33719b4836410311aea3513115c

Make ObjectCache check the value of apc.enable_cli in CLI mode

Add HashBagOStuff fallback for APC in MWLBFactory::injectObjectCaches.

Also fix APC-cache variable typo in MWLBFactory.

Bug: T227838
Change-Id: I71cb2ca58972ea09ab2f64f7e47bda7a5096c19b

Integration tests for Language fallback methods

100% coverage.  Next: reimplementation as a service, and tests for the
new service.

Change-Id: Icdd5b84eb91fc83905209ce08d499011f7288e33

Merge "build: Upgrade eslint-utils dependency from 1.3.1 to 1.4.2 for security issues"

createNoOpMock() method for PHPUnit tests

This is just a shortcut for a commonly-used pattern, when you want to
create a mock that's never supposed to have methods called.

Change-Id: Ia7267e3d3108c1ff94485f7e44bf409808a762be

Merge "Fix FileBackendGroup test for some configurations"

Move User::getAllRights to PermissionManager.

Bug: T220191
Change-Id: I7f4bf7f6a85b01ffd7f9ea3991597f1bd40ab1f6

Fix FileBackendGroup test for some configurations

The expected value of FileBackendGroup::config()['srvCache'] was being
obtained from MediaWikiServices::getLocalServerObjectCache(), but the
class actually used ObjectCache::getLocalServerInstance( 'hash' ). It
happens these are often the same, so it passed the gate-and-submit jobs,
but it's causing failures on Travis CI.

Follow-up to bd2a4395025, which added the failing test. Thanks to
Krinkle for pointing out the failures on CI.

Change-Id: I17651766f4ee2752dfcae9574d2538269dec4ebe

Merge "Remove unneeded overrideMwServices/resetServices"

Merge "Call resetServices() when setting globals in tests"

Improve formatting of constructor documentation for block classes

Change-Id: Idced6ce907f63d2c041d1bb926b8224ece54c3de

Merge "Add UnknownContentHandler."

Remove unneeded overrideMwServices/resetServices

Change-Id: If6cbdec05b8f310ef3a0b4649aaa16d9fb80a047

Call resetServices() when setting globals in tests

Now that resetServices() will preserve (but reset) customized services,
it should be reasonably safe to call it every time globals are changed,
and much more effective than relying on tests to call it every time
themselves.

Depends-On: Iab8ea3a61bbc6803805d855ef23c071067646f71
Depends-On: I00e35ecea6a27468674b2a6e7d9d9eb6518e3bd5
Change-Id: Ie7a89f6ed7d52a0bc01672019ff92e7ee105a1f3

Avoid fatal errors when reporting exceptions.

When reporting exceptions that occur during initialization, wgUser may
be null. Don't die when that happens.

Change-Id: I65d5a17d80f9021e28a218c7a5a17e399bc7ce98

Add UnknownContentHandler.

UnknownContentHandler can be configued to handle models that
belong to extensions that have been undeployed:

  $wgContentHandlers['xyzzy'] = 'UnknownContentHandler';

This way, no errors will be thrown when trying to access
pages with the unsupported model. Instead, an error message is
shown, and editing is prevented.

This patch also improves handling of non-editable content in
EditPage and in DifferenceEngine.

Bug: T220608
Change-Id: Ia94521b786c0a5225a674e4dc3cb6761a723d75b

filebackend: add idiom constant to FileBackend for null results

Change-Id: I65f043e87d82192c13d3627fb45ef222f0290bf8

Merge "Fix categories detele SPARQL clause"

filebackend: fix @throws comment in FileBackend::makeContentDisposition()

Change-Id: I653b0c9b28939a0976328ed1de6444dd38b7fa55

Merge "profiler: Centralise output responsibility from ProfilerOutputText to Profiler"

Merge "mediawiki.Uri: Add another test for arrayParams"

mediawiki.Uri: Add another test for arrayParams

Verify that the added handling is correctly conditional.

Bug: T231382
Change-Id: I71b834c1c0a7bdaf28d990eafd4ce5d70a55e141

Merge "Special:NewFiles: Use a proper user widget instead"

Merge "resourceloader: Purge localStorage blob if last written 30+ days ago"

Merge "Pagers: Drop 'newbie' feature"

Merge "Fix call to non-existing TempFSFileFactory::getTempFSFile()"

Merge "Special:DeletedContributions: Drop 'newbies' feature"

Fix call to non-existing TempFSFileFactory::getTempFSFile()

Should be TempFSFileFactory::newTempFSFile()

Bug: T231488
Change-Id: I9fbf7d993773f55965268ac10b347110148671c9

Merge "Fix typo in comment"

Merge "Make WantedQueryPage::makeWlhLink protected"

Merge "PermissionManager should not cache anonymous rights under ID 0"

Special:NewFiles: Use a proper user widget instead

Change-Id: I276f55e9d4dd2d539dae232c7a4c1e71745a6168

Pagers: Drop 'newbie' feature

Change-Id: I14c4da7b107aa9c712bd81522021ae7ecaf2b790

Special:DeletedContributions: Drop 'newbies' feature

Change-Id: I32c67fb19f50f3949ffe981488c009c82a244999

Merge "Sync up with Parsoid parserTests.txt"

Merge "Remove unused localisation message 'wlshowlast'"

Merge "Remove unused localisation messages 'ascending_abbrev', 'descending_abbrev'"

Merge "Upstream "pulsating dot" widget from VE"

Make WantedQueryPage::makeWlhLink protected

Bug: T231481
Change-Id: I6ace1bb99879ec8256b29f86220e8fc2e785f593

Merge "Integration tests for FileBackendGroup"

Merge "Grunt: Expand js{,on} for readability"

PermissionManager should not cache anonymous rights under ID 0

Bug: T228253
Change-Id: I8a54830842f220ff1ac4402a3380c2229a99b619

profiler: Centralise output responsibility from ProfilerOutputText to Profiler

Make it Profiler.php's responsibility to enforce this, based on the
existing signal from ProfilerOutput::logsToOutput().

The ProfilerOutputText class should not have to double-check this
a second time.

Long-term, I'd like even this check in Profiler::logDataPageOutputOnly
to be removed, because really the external caller of that should
know whether it is safe to output stuff or not rather than stashing
its own state inside Profiler::$allowOutput and then implicitly
reading it back out again later on. But, that's for another time.

Also:
* Remove use of deprecated Profiler::setTemplated while at it.
* Make 'visible' parameter explicit, as for other parameters.

Change-Id: Iaa3fc4ea25a059b90235d769db60c04b8f152f05

Sync up with Parsoid parserTests.txt

This now aligns with Parsoid commit 06a41a99d7811a361446b894da7c5c8224398ad1

Change-Id: I9d324b3e6e9167683c15b7fee2a53b598d79a67c

Docs: Standardise "messages used" comments

Change-Id: Ie99f8a62da734956863b1e1c41c8768f0e083fed

Grunt: Expand js{,on} for readability

Change-Id: Ib3bfb1a9d6547e1062ea3ad08de7ea721c111586

Merge "ApiFeedContributions: Drop 'newbies' feature"

ApiFeedContributions: Drop 'newbies' feature

It has been broken since 2011 (r100315), so no need for
deprecation.

Change-Id: I099f52211c8ce401d827405cb313e1d3932b92f8

Merge "skins: Skin::getSkinNameMessages() method is now deprecated"

Merge "skins: Remove use of deprecated MessageCache::singleton()"

Remove unused localisation message 'wlshowlast'

Not used since 34a3efd5f75ca25f1b8b3b3a2efd13bfb1f7c631.

Note that a similar message 'rclinks' is still used.

Bug: T224429
Change-Id: I471a6a88b838bd7f1fab34c531c556e8ccd2779a

Remove unused localisation messages 'ascending_abbrev', 'descending_abbrev'

Not used since 987a860df428e3c70fd81c404f1f7a300345dc48.

The messages 'sort-descending', 'sort-ascending' serve the same purpose.

Bug: T224429
Change-Id: Iabcf83c80aa3fc387ea908868838b73254765a82

Re-add missing messages in languages/i18n/exif/qqq.json

Added in 7c68604e4c5e3f61884d529a37a2878c3155d343, incorrectly removed
by translation updater in 7881d44f690cc1e7415bb6cbaedc02d586a64498?
Checks are failing because of this.

Change-Id: I264f0f1cf3dcd69bbf8d43ad057a81e3f39ed623

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

Merge "Move Title::isNamespaceProtected() to PermissionManager."

Merge "Fix entites typos"

Merge "LockManagerGroup tweaks"

Merge "profiler: Rename Profile::setTemplated to Profile::setAllowOutput"

Merge "Document that img_description_id is a comment id"

Integration tests for FileBackendGroup

100% coverage except for one bit of the code that I didn't understand.
Unit tests to come, together with rewrite as a service.

Change-Id: Ib01758d994a9e5587a4fcb5edc3d80010ef05615

LockManagerGroup tweaks

One small change that was supposed to be in 5a6c18a0863 but didn't make
it into the patch version that got merged, and removal of some long-dead
code.

phan objected to the use of new $class without being explicitly
reassured that $class is, in fact, a string. I don't know why.

Change-Id: Ifa00c59ab4464109414f21db37e3a6db21decdf3

Localisation updates from https://translatewiki.net.

Change-Id: I8380b27a31ae89200aa7e7ccf38424acb244e288

Fix typo in comment

Change-Id: Iebb57a49156f4efc6f45079d239eef2464c0f554

Merge "Let Html::element do the HTML encoding"

Merge "parserTests: Add parser test with filename containing single quotes"

Fix entites typos

Bug: T201491
Change-Id: I241fda87b0e53e804f6bdd48eb975a79e9f176d2

Merge "Revert actor changes to populateLogSearch.php"