Page MenuHomePhabricator

Change instances of MWException to be a plain Exception
Closed, ResolvedPublic

Description

We don't want to depend on MW so we should throw a built-in exception.

We checked and we don't need to implement some kind of "isLoggable()". the MWExceptionHandler already seems to do the right thing.

Event Timeline

Tarrow created this task.Jul 20 2020, 12:52 PM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptJul 20 2020, 12:52 PM
Restricted Application added a project: User-Ladsgroup. · View Herald TranscriptJul 21 2020, 9:52 AM

Change 615207 had a related patch set uploaded (by Ladsgroup; owner: Ladsgroup):
[mediawiki/extensions/Wikibase@master] Change instances of MWException to be a plain Exception in Lib\Changes

https://gerrit.wikimedia.org/r/615207

Change 615207 merged by jenkins-bot:
[mediawiki/extensions/Wikibase@master] Change instances of MWException to be a plain Exception in Lib\Changes

https://gerrit.wikimedia.org/r/615207

Maintenance_bot moved this task from Incoming to Done on the User-Ladsgroup board.Aug 5 2020, 9:15 AM