[Cdk-devel] [ cdk-Patches-2883249 ] Delete of atoms

[SourceForge.net <no...@so...>]

Patches item #2883249, was opened at 2009-10-21 16:13
Message generated for change (Settings changed) made by shk3
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=320024&aid=2883249&group_id=20024

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
>Status: Closed
>Resolution: Out of Date
Priority: 5
Private: No
Submitted By: Stefan Kuhn (shk3)
Assigned to: Nobody/Anonymous (nobody)
Summary: Delete of atoms

Initial Comment:
commit bc86cd9547b511781e62ebae7522c9dfdda200de
Author: Stefan Kuhn <Ste...@eb...>
Date:   Wed Oct 21 15:09:10 2009 +0100

    Fixes the deleting of atoms, the Edit was not exectud. Also fixed the undo/redo of remove
    atom. Note the undo added newly created bonds, this is wrong. It needs to readd the old ones.
    Other undo/redos rely on identity (the old code produced errors if you had chains of actions,
    e. g. draw a ring, delete one atom, do undo, do undo - the second undo tries to remove the
    ring, but two bonds of the rings have been replaced, which the undo of draw ring has no idea
    about)


----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=320024&aid=2883249&group_id=20024