Opened 14 years ago
Last modified 3 years ago
#5846 new defect
AssertionError when undoing PurgeCommand
| Reported by: | Upliner | Owned by: | team |
|---|---|---|---|
| Priority: | normal | Milestone: | |
| Component: | Core | Version: | |
| Keywords: | purge undo command stack | Cc: | skyper |
Description (last modified by )
Steps to reproduce:
- Download some data
- Purge it without clearing undoredo
- Download the data again
- Undo the PurgeCommand
java.lang.AssertionError: Primitive {Node id=338367679 version=4 V lat=51.4074839,lon=31.2550692} was removed when purging, but is still there on undo
at org.openstreetmap.josm.command.PurgeCommand.undoCommand(PurgeCommand.java:137)
Attachments (0)
Change History (12)
comment:1 by , 14 years ago
| Description: | modified (diff) |
|---|---|
| Summary: | AssertionException when undoing PurgeCommand → AssertionError when undoing PurgeCommand |
comment:2 by , 14 years ago
comment:4 by , 12 years ago
| Cc: | added |
|---|---|
| Description: | modified (diff) |
follow-up: 11 comment:5 by , 12 years ago
It's also possible to get the data again by merging layers (see #7706). Why not simply remove this assertion ?
comment:6 by , 12 years ago
You can edit, purge and redownload an object. If you want to restore the modified object on undo, it gets slightly more complicated.
comment:9 by , 7 years ago
| Keywords: | purge added |
|---|
comment:10 by , 5 years ago
| Keywords: | undo added |
|---|
comment:11 by , 3 years ago
Replying to bastiK:
I'd say the problem is not with the purge command, but the fact that "download data" does not show up in the command list.
See #7489 for this approach but I am not sure if it is really needed or useful.
Replying to Don-vip:
It's also possible to get the data again by merging layers (see #7706). Why not simply remove this assertion ?
Is creating an object version conflict the way to go?
comment:12 by , 3 years ago
| Keywords: | command stack added |
|---|
I'd say the problem is not with the purge command, but the fact that "download data" does not show up in the command list.