https://bugzilla.wikimedia.org/show_bug.cgi?id=58274

Daniel Kinzler <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[email protected]

--- Comment #1 from Daniel Kinzler <[email protected]> ---
One problem with implementing this is that following the naive approach
(representing a change in order as a remove and an insert operation), both such
operations would have the same key in the (Map)Diff, so one would override the
other. Also, for the insert operation, it would need some context specifying
*where* to insert it.

An alternative approach is to represent the order explicitly as a list of keys,
and attach that list as an extra field to the MapDiff.

Note that the class tree based in the Diff class contains some overburdened and
some deprecated classes. Some refactoring may be in order. 

Also, care must be taken to allow for reliably application of Diffs as patches.

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
_______________________________________________
Wikibugs-l mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l

Reply via email to