Page MenuHomePhabricator

🐼️ Improve PatchTestOperationFailedException error reports
Closed, ResolvedPublic

Description

  • Exception exists, but needs data of:
    • the failed operation
    • the actual value that failed the test operation check
  • covers everything from Exception to RouteHandler and HttpReponse

Event Timeline

Change 845374 had a related patch set uploaded (by Silvan Heintze; author: Silvan Heintze):

[mediawiki/extensions/Wikibase@master] REST: Add context to PatchItemStatement's error

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

Change 845375 had a related patch set uploaded (by Silvan Heintze; author: Silvan Heintze):

[mediawiki/extensions/Wikibase@master] REST: Improve PatchItemStatement's error reports

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

Change 845375 abandoned by Silvan Heintze:

[mediawiki/extensions/Wikibase@master] REST: Improve PatchItemStatement's error reports

Reason:

Covered in parent changes

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

Change 845374 merged by jenkins-bot:

[mediawiki/extensions/Wikibase@master] REST: Add context to PatchItemStatement error

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

Change 849481 had a related patch set uploaded (by Jakob; author: Jakob):

[mediawiki/extensions/Wikibase@master] REST: Don't use JSON_FORCE_OBJECT in error message

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

Change 849481 merged by jenkins-bot:

[mediawiki/extensions/Wikibase@master] REST: Don't use JSON_FORCE_OBJECT in error message

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