Page MenuHomePhabricator

๐Ÿˆ๏ธ [Generalised error] missing-field in request body
Closed, ResolvedPublic5 Estimated Story Points

Description

Acceptance criteria:

  • Current errors it will replace:
    • sitelink-data-missing-title
    • statement-data-missing-field
    • missing-json-patch-field
    • top level fields that currently throw a mw framework exception
  • HTTP status code: 400
  • Error code: missing-field
  • Error message: Required field missing
  • Context: { "path": "{json_pointer_to_parent}", "field": "{missing_field}" }

Event Timeline

Restricted Application added a subscriber: Aklapper. ยท View Herald TranscriptMay 29 2024, 1:43 PM
Ollie.Shotton_WMDE renamed this task from [Generalised error] missing-field / context to [Generalised error] missing-field.Jul 4 2024, 2:46 PM
Ollie.Shotton_WMDE updated the task description. (Show Details)

Task breakdown notes:

  • one subtask for each of:
    • sitelink-data-missing-title
    • statement-data-missing-field
    • missing-json-patch-field
    • top level fields that currently throw a mw framework exception

Check if e2e tests for top-level fields are there (all edit requests minus DELETE) and add where missing.

Silvan_WMDE renamed this task from [Generalised error] missing-field to ๐Ÿˆ๏ธ [Generalised error] missing-field.Jul 10 2024, 10:37 AM
Ollie.Shotton_WMDE renamed this task from ๐Ÿˆ๏ธ [Generalised error] missing-field to ๐Ÿˆ๏ธ [Generalised error] missing-field in request body.Jul 24 2024, 10:30 AM
Ollie.Shotton_WMDE updated the task description. (Show Details)