In API module wbremoveclaims a line is lost and the params is not pulled in and used during load, which would later create problems during save as conflicts will be handled in some weird way.
Version: unspecified
Severity: major
Description
Description
Details
Details
- Reference
- bz45761
Event Timeline
Comment Actions
The above fix has serious logic problems (not caused by the fix itself, but preventing the fix from working in some cases). We may need to revert it. Reopening until this has been re-examined.
Comment Actions
The current fix will cause the request to Fail with an Error if a base revision ID is supplied and the mentioned Statements belong to two different items (because the base revision will not be correct for at least one of them).
If we allow a list of statements from different items, we need to accept a list of base revisions. Better to restrict this module to statements on a single item.
Comment Actions
A request removing statements from two different items would be erroneous call in my opinion, and then the correct behavior would be to fail. I think the module should have a restriction on the ids it can accept.
This is although another bug.
Comment Actions
(In reply to comment #4)
A request removing statements from two different items would be erroneous
call
in my opinion, and then the correct behavior would be to fail. I think the
module should have a restriction on the ids it can accept.
Yes, that would also greatly simply the code.
This is although another bug.
Yes, but this bug should perhaps depend on that other bug.