Last modified: 2014-07-16 10:32:54 UTC
ChangeOps should perform validation against a target entity separately from the apply() method, in validate() method. The rationale is: 1) when applying a batch of ChangeOps, we want to be able to validate all before applying any. 2) for some batch operations like merging, we don't want to apply validation, since we are dealing with "status quo" data, and we shouldn't fail if that status quo does not conform to soft constraints (hard constraints should and will of course still be enforced).
Change 129244 had a related patch set uploaded by Daniel Kinzler: (bug 64309) Introducing ChangeOp::validate https://gerrit.wikimedia.org/r/129244
Does this resolve bug 65215?
Could you take a look at bug 65215?