Page MenuHomePhabricator

Fix error codes returned by the API.
Closed, ResolvedPublic

Description

ApiBase::dieUsage takes an error code. In the WIkibase API modules, we often include the module name in that code. This is redundant: ApiBase::dieUsage already attaches the module name to the code. So we should not do this again, but use the same code to indicate the same problem in different modules. This also eases code reuse between modules.


Version: unspecified
Severity: normal

Details

Reference
bz45276

Event Timeline

bzimport raised the priority of this task from to Needs Triage.Nov 22 2014, 1:17 AM
bzimport set Reference to bz45276.
bzimport added a subscriber: Unknown Object (MLST).

Change 72528 had a related patch set uploaded by Addshore:
Standardising many errors throught the api

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

Change 72528 had a related patch set (by Addshore) published:
Standardising the api errors and methods

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

Change 72528 merged by Addshore:
Standardising the api errors and methods

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

Change 72958 had a related patch set uploaded by Addshore:
Revert Revert + Standardising the api errors

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

Change 72958 merged by Addshore:
Revert Revert + Standardising the api errors

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

Verified in Wikidata demo July 31th.

Restricted Application added a subscriber: StudiesWorld. · View Herald Transcript