DataValue objects should perform strict validation on the parameters they receive in their constructor, and throw well defined and informative exceptions when validation fails.
The API needs to catch these exceptions and turn them into an appropriate API error response.
Version: unspecified
Severity: normal
See Also:
https://bugzilla.wikimedia.org/show_bug.cgi?id=49433
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Open | None | T87764 Bugs related to time datatype (tracking) | |||
| Resolved | Lydia_Pintscher | T50962 localize display of time values | |||
| Resolved | None | T47046 Normalizing file names for display | |||
| Resolved | None | T50860 [ui] First version of geocoordinate datatype support | |||
| Resolved | None | T63254 Item values are not linked when Javascript is disabled | |||
| Resolved | None | T51014 Use value formatters for displaying values on Entity pages (non-JavaScript view) | |||
| Resolved | • adrianheine | T63738 Spaces in commons file names are replaced by plus signs instead of underscores | |||
| Resolved | None | T50888 Improve data value validation and error handling in api | |||
| Resolved | None | T50965 time data values are not validated when saved via api | |||
| Resolved | • adrianheine | T51186 Property value parsing, formatting and validation should be done (only?) in the backend | |||
| Resolved | None | T51425 Invalid Timevalues stored in database | |||
| Resolved | None | T51264 Apply strict validation when constructing DataValue objects |
https://gerrit.wikimedia.org/r/68150 (Gerrit Change I891606) | change APPROVED and MERGED [by jenkins-bot]
https://gerrit.wikimedia.org/r/67962 (Gerrit Change I699098) | change ABANDONED [by Daniel Kinzler]
Change 68952 abandoned by Daniel Kinzler:
(bug 49264) Handle corrupt data values gracefully.
Reason:
Use I992028 instead. It seems that handling bad values on the DataValue level is less awkward.
Change 70433 merged by Jeroen De Dauw:
(bug 49264) Handle bad values using UnDeserializableValue.
Change 70443 merged by jenkins-bot:
(bug 49264) Handle UnDeserializableValue gracefully.
Change 68002 merged by jenkins-bot:
(bug 49264) Handle UnDeserializableValue gracefully.
Change 69659 merged by jenkins-bot:
(bug 49264) Make SnakValidator fail on bad values.