Clean up EntityDiff and ItemDiff
Closed this issue · 5 comments
JeroenDeDauw commented
EntityDiff
makes assumptions that all entities have aliases, labels, descriptions and claims. It is generally not well designed. claim
is used as key for statements.
Creating a well designed alternative from ground up might be the way to go.
Benestar commented
We might also discuss if the diffing and patching code should stay in the DataModel or be moved into another package like it is already done for the serialization code.
Benestar commented
No longer in this component
JeroenDeDauw commented
Well better create a new one for it then ;p wmde/WikibaseDataModelServices#108
Benestar commented
It is already tracked on phabricator so I assumed that would be enough.