Details
Description
On data Patch request processing (see PatchDataTransactionUtil.patchData(…))
- each operation is processed in a sequence, per-operation status is registered
- if there was no per-operation error then commit() is called, commit error is registered as a global error
For case when error occurs on commit the result displays global error together with per-operation status (all success) which is confusing and self contradicting.
Expected: I there is a global error then per-operation status should be omitted (dropped) bc it always displays all successes
Scope: JsonPatchBodyWriter + XmlPatchBodyWriter, ensure behavior is covered by unit tests.
Attachments
Issue Links
- split from
-
NETCONF-1103 Incorrect response for failed PATCH request
-
- Resolved
-
| # | Subject | Branch | Project | Status | CR | V |
|---|---|---|---|---|---|---|
| 107687,21 | Fix JSON and XML PatchBodyWriter errors output | master | netconf | Status: MERGED | +2 | +1 |
| 108882,2 | Fix JSON and XML PatchBodyWriter errors output | 6.0.x | netconf | Status: MERGED | +2 | +1 |
| 108883,2 | Fix JSON and XML PatchBodyWriter errors output | 5.0.x | netconf | Status: MERGED | +2 | +1 |
| 108884,2 | Fix JSON and XML PatchBodyWriter errors output | 4.0.x | netconf | Status: MERGED | +2 | +1 |