Last modified: 2013-06-27 12:37:12 UTC
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.
https://gerrit.wikimedia.org/r/#/c/52346/
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.
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.
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.
(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.
seems to be ok now
