Opened 9 years ago

Last modified 7 years ago

#3860 reopened defect

Potlatch 2.2 does not discard a way's first point when Undo-ing it

Reported by: Peter Owned by: potlatch-dev@…
Priority: major Milestone:
Component: potlatch2 Version: 2.0
Keywords: Cc: malenki


Click two or more points (=n) on map - a way connecting clicked points is being created. After (n-1) Undo's the way is gone, but the initial point still remains visible (a green dot) and the Undo button is already grayed out - one step got forgot.

It is possible to delete the point manually (red (x) button), but it should be also deleted by the n-th Undo.

Change History (6)

comment:1 Changed 9 years ago by Nop

Priority: minormajor

I observe this behaviour regularly. All that's needed is to create a two-node way and press undo once. This happens quite frequently when I try to drag the visible area but somehow P2 recognizes clicks instead of the drag. I think this is a major nuisance as it tends to create nonsensical empty nodes if you don't spot the problem and delete them manually. Does it break the undo system if such ghost objects are left over?

comment:2 Changed 8 years ago by stevage

Resolution: fixed
Status: newclosed

Fixed. Ow. That was a lot harder than I expected.

comment:3 Changed 8 years ago by Peter

Thanks, it was indeed a frequent nuisance.

comment:4 Changed 7 years ago by malenki

Cc: malenki added
Resolution: fixed
Status: closedreopened

The misbehaviour is still/again there. Node cannot be deleted using either undo-button or ctrl-z

comment:5 Changed 7 years ago by stevage

Yes. Although I submitted a pull request with the bug fix, it has not been accepted by the maintainer of Potlatch 2, Richard Fairhurst. I suggest you ask him politely about it.

comment:6 Changed 7 years ago by malenki

#4406 seems a duplicate of this bug.

Last edited 7 years ago by malenki (previous) (diff)
Note: See TracTickets for help on using tickets.