Modify

Opened 14 years ago

Closed 13 years ago

Last modified 13 years ago

#5566 closed defect (duplicate)

Validator allows multiple deleting of objects and so causes exceptions

Reported by: malenki Owned by: team
Priority: normal Milestone:
Component: Core validator Version: latest
Keywords: Cc: malenki

Description (last modified by skyper)

How to reproduce:

Run the validator
Delete an object which the validator hilights in josm
Select the deleted object again by double clicking it in the validator
Hit Del

-> Exception

Repository Root: http://josm.openstreetmap.de/svn
Build-Date: 2010-10-18 01:31:07
Last Changed Author: ulfl
Revision: 3616
Repository UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
URL: http://josm.openstreetmap.de/svn/trunk
Last Changed Date: 2010-10-17 17:46:55 +0200 (Sun, 17 Oct 2010)
Last Changed Rev: 3616

Identification: JOSM/1.5 (3616 de)
Memory Usage: 729 MB / 910 MB (160 MB allocated, but free)
Java version: 1.6.0_22, Sun Microsystems Inc., Java HotSpot(TM) 64-Bit Server VM
Operating system: Linux
Dataset consistency test: No problems found

Plugin: DirectUpload (22017)
Plugin: PicLayer (22549)
Plugin: fuzzer (15)
Plugin: lakewalker (21706)
Plugin: measurement (22547)
Plugin: multipoly (22171)
Plugin: openstreetbugs (22723)
Plugin: osmarender (22765)
Plugin: remotecontrol (22734)
Plugin: reverter (23278)
Plugin: slippymap (23575)
Plugin: tageditor (21026)
Plugin: terracer (22169)
Plugin: turnrestrictions (23593)
Plugin: undelete (22365)
Plugin: validator (23192)
Plugin: wmsplugin (23454)

java.lang.IllegalArgumentException: {Node id=954722680 version=1 MVD lat=12.8793,lon=34.00455} is already deleted
	at org.openstreetmap.josm.command.DeleteCommand.executeCommand(DeleteCommand.java:117)
	at org.openstreetmap.josm.command.SequenceCommand.executeCommand(SequenceCommand.java:53)
	at org.openstreetmap.josm.data.UndoRedoHandler.addNoRedraw(UndoRedoHandler.java:36)
	at org.openstreetmap.josm.data.UndoRedoHandler.add(UndoRedoHandler.java:58)
	at org.openstreetmap.josm.actions.mapmode.DeleteAction.doActionPerformed(DeleteAction.java:153)
	at org.openstreetmap.josm.actions.DeleteAction.actionPerformed(DeleteAction.java:28)
	at javax.swing.SwingUtilities.notifyAction(SwingUtilities.java:1633)
	at javax.swing.JComponent.processKeyBinding(JComponent.java:2851)
	at javax.swing.KeyboardManager.fireBinding(KeyboardManager.java:267)
	at javax.swing.KeyboardManager.fireKeyboardAction(KeyboardManager.java:216)
	at javax.swing.JComponent.processKeyBindingsForAllComponents(JComponent.java:2928)
	at javax.swing.JComponent.processKeyBindings(JComponent.java:2920)
	at javax.swing.JComponent.processKeyEvent(JComponent.java:2814)
	at java.awt.Component.processEvent(Component.java:6044)
	at java.awt.Container.processEvent(Container.java:2041)
	at java.awt.Component.dispatchEventImpl(Component.java:4630)
	at java.awt.Container.dispatchEventImpl(Container.java:2099)
	at java.awt.Component.dispatchEvent(Component.java:4460)
	at java.awt.KeyboardFocusManager.redispatchEvent(KeyboardFocusManager.java:1850)
	at java.awt.DefaultKeyboardFocusManager.dispatchKeyEvent(DefaultKeyboardFocusManager.java:712)
	at java.awt.DefaultKeyboardFocusManager.preDispatchKeyEvent(DefaultKeyboardFocusManager.java:990)
	at java.awt.DefaultKeyboardFocusManager.typeAheadAssertions(DefaultKeyboardFocusManager.java:855)
	at java.awt.DefaultKeyboardFocusManager.dispatchEvent(DefaultKeyboardFocusManager.java:676)
	at java.awt.Component.dispatchEventImpl(Component.java:4502)
	at java.awt.Container.dispatchEventImpl(Container.java:2099)
	at java.awt.Window.dispatchEventImpl(Window.java:2478)
	at java.awt.Component.dispatchEvent(Component.java:4460)
	at java.awt.EventQueue.dispatchEvent(EventQueue.java:599)
	at java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:269)
	at java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:184)
	at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:174)
	at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:169)
	at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:161)
	at java.awt.EventDispatchThread.run(EventDispatchThread.java:122)

Attachments (0)

Change History (5)

comment:1 by malenki, 14 years ago

Cc: malenki added

comment:2 by skyper, 13 years ago

Ticket #7205 has been marked as a duplicate of this ticket.

comment:3 by skyper, 13 years ago

Description: modified (diff)
Resolution: duplicate
Status: newclosed

clossing this ticket in favour of #6640 which has more useful information

comment:4 by skyper, 13 years ago

Closed as duplicate of #6640.

comment:5 by akks, 13 years ago

Exactly this one fixed in r5037, see also #6640.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain team.
as The resolution will be set.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.