Context Navigation

Modify ↓

#8096 closed defect (fixed)

error removing duplicate relations

Reported by:	mcheck	Owned by:	team
Priority:	normal	Milestone:
Component:	Core validator	Version:
Keywords:	fix	Cc:	geri-oc, naoliv

Description (last modified by Don-vip)

Repository Root: http://josm.openstreetmap.de/svn
Build-Date: 2012-08-31 01:31:11
Last Changed Author: Don-vip
Revision: 5485
Repository UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
URL: http://josm.openstreetmap.de/svn/trunk
Last Changed Date: 2012-08-31 00:21:56 +0200 (Fri, 31 Aug 2012)
Last Changed Rev: 5485

Identification: JOSM/1.5 (5485 it)
Memory Usage: 281 MB / 455 MB (145 MB allocated, but free)
Java version: 1.6.0_26, Sun Microsystems Inc., Java HotSpot(TM) Server VM
Operating system: Linux
Dataset consistency test: No problems found

Plugin: AddrInterpolation (28541)
Plugin: FastDraw (28656)
Plugin: FixAddresses (28656)
Plugin: HouseNumberTaggingTool (28656)
Plugin: ImageWayPoint (28656)
Plugin: ImportImagePlugin (28656)
Plugin: OpeningHoursEditor (28656)
Plugin: PicLayer (28660)
Plugin: RoadSigns (28656)
Plugin: SimplifyArea (28656)
Plugin: buildings_tools (28656)
Plugin: continuosDownload (28565)
Plugin: contourmerge (1003)
Plugin: dataimport (28656)
Plugin: download_along (28656)
Plugin: editgpx (28689)
Plugin: ext_tools (28656)
Plugin: graphview (28656)
Plugin: imageryadjust (28656)
Plugin: importvec (28656)
Plugin: irsrectify (28656)
Plugin: lakewalker (28656)
Plugin: mirrored_download (28656)
Plugin: multipoly-convert (28656)
Plugin: namemanager (28541)
Plugin: openstreetbugs (28656)
Plugin: osmarender (28541)
Plugin: pbf (28656)
Plugin: pdfimport (28656)
Plugin: photo_geotagging (28656)
Plugin: plastic_laf (26605)
Plugin: print (28656)
Plugin: proj4j (28495)
Plugin: public_transport (28656)
Plugin: reltoolbox (28720)
Plugin: reverter (28656)
Plugin: scripting (30100)
Plugin: tageditor (28656)
Plugin: tagging-preset-tester (28566)
Plugin: terracer (28656)
Plugin: utilsplugin2 (28656)
Plugin: walkingpapers (28541)
Plugin: wms-turbo-challenge2 (28541)

java.lang.RuntimeException: java.lang.reflect.InvocationTargetException
	at org.openstreetmap.josm.gui.dialogs.ValidatorDialog$FixTask.realRun(ValidatorDialog.java:613)
	at org.openstreetmap.josm.gui.PleaseWaitRunnable.doRealRun(PleaseWaitRunnable.java:82)
	at org.openstreetmap.josm.gui.PleaseWaitRunnable.run(PleaseWaitRunnable.java:145)
	at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:441)
	at java.util.concurrent.FutureTask$Sync.innerRun(FutureTask.java:303)
	at java.util.concurrent.FutureTask.run(FutureTask.java:138)
	at java.util.concurrent.ThreadPoolExecutor$Worker.runTask(ThreadPoolExecutor.java:886)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:908)
	at java.lang.Thread.run(Thread.java:662)
Caused by: java.lang.reflect.InvocationTargetException
	at java.awt.EventQueue.invokeAndWait(EventQueue.java:1042)
	at javax.swing.SwingUtilities.invokeAndWait(SwingUtilities.java:1326)
	at org.openstreetmap.josm.gui.dialogs.ValidatorDialog$FixTask.realRun(ValidatorDialog.java:585)
	... 8 more
Caused by: java.lang.IllegalArgumentException: {Relation id=2100704 version=1 MVDT]} is already deleted
	at org.openstreetmap.josm.command.DeleteCommand.executeCommand(DeleteCommand.java:119)
	at org.openstreetmap.josm.command.SequenceCommand.executeCommand(SequenceCommand.java:54)
	at org.openstreetmap.josm.data.UndoRedoHandler.addNoRedraw(UndoRedoHandler.java:36)
	at org.openstreetmap.josm.gui.dialogs.ValidatorDialog$FixTask$1.run(ValidatorDialog.java:588)
	at java.awt.event.InvocationEvent.dispatch(InvocationEvent.java:199)
	at java.awt.EventQueue.dispatchEventImpl(EventQueue.java:641)
	at java.awt.EventQueue.access$000(EventQueue.java:84)
	at java.awt.EventQueue$1.run(EventQueue.java:602)
	at java.awt.EventQueue$1.run(EventQueue.java:600)
	at java.security.AccessController.doPrivileged(Native Method)
	at java.security.AccessControlContext$1.doIntersectionPrivilege(AccessControlContext.java:87)
	at java.awt.EventQueue.dispatchEvent(EventQueue.java:611)
	at java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:269)
	at java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:184)
	at java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:178)
	at java.awt.Dialog$1.run(Dialog.java:1046)
	at java.awt.Dialog$3.run(Dialog.java:1098)
	at java.security.AccessController.doPrivileged(Native Method)
	at java.awt.Dialog.show(Dialog.java:1096)
	at java.awt.Component.show(Component.java:1584)
	at java.awt.Component.setVisible(Component.java:1536)
	at java.awt.Window.setVisible(Window.java:842)
	at java.awt.Dialog.setVisible(Dialog.java:986)
	at org.openstreetmap.josm.gui.progress.PleaseWaitProgressMonitor$4.run(PleaseWaitProgressMonitor.java:170)
	at java.awt.event.InvocationEvent.dispatch(InvocationEvent.java:209)
	at java.awt.EventQueue.dispatchEventImpl(EventQueue.java:641)
	at java.awt.EventQueue.access$000(EventQueue.java:84)
	at java.awt.EventQueue$1.run(EventQueue.java:602)
	at java.awt.EventQueue$1.run(EventQueue.java:600)
	at java.security.AccessController.doPrivileged(Native Method)
	at java.security.AccessControlContext$1.doIntersectionPrivilege(AccessControlContext.java:87)
	at java.awt.EventQueue.dispatchEvent(EventQueue.java:611)
	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 (1)

validator-error.osm.bz2 (72.4 KB ) - added by naoliv 13 years ago.: The attached file causes this issue. Run validator, select the error category and click fix.

Download all attachments as: .zip

Change History (14)

comment:1 by Don-vip, 13 years ago

Description:	modified (diff)

comment:2 by skyper, 13 years ago

Component:	Core → Core validator

comment:3 by anonymous, 13 years ago

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

comment:4 by skyper, 13 years ago

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

comment:5 by skyper, 13 years ago

Cc:	geri-oc added
Keywords:	fix added

From #8357 (translated):

After validation error was shown (overlapping ways). Automatical fix was offered and lead to following error message.

Repository Root: http://josm.openstreetmap.de/svn
Build-Date: 2012-11-29 02:31:07
Last Changed Author: stoecker
Revision: 5608
Repository UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
URL: http://josm.openstreetmap.de/svn/trunk
Last Changed Date: 2012-11-28 23:47:46 +0100 (Wed, 28 Nov 2012)
Last Changed Rev: 5608

Identification: JOSM/1.5 (5608 de)
Memory Usage: 80 MB / 494 MB (18 MB allocated, but free)
Java version: 1.7.0_11, Oracle Corporation, Java HotSpot(TM) Client VM
Operating system: Windows Vista
Dataset consistency test: No problems found

Plugin: OpeningHoursEditor (28656)
Plugin: PicLayer (28807)
Plugin: buildings_tools (28891)
Plugin: graphview (28807)
Plugin: measurement (28898)
Plugin: reltoolbox (28857)
Plugin: reverter (28820)
Plugin: terracer (29037)
Plugin: turnrestrictions (28807)
Plugin: undelete (28807)

java.lang.RuntimeException: java.lang.reflect.InvocationTargetException
	at org.openstreetmap.josm.gui.dialogs.ValidatorDialog$FixTask.realRun(ValidatorDialog.java:612)
	at org.openstreetmap.josm.gui.PleaseWaitRunnable.doRealRun(PleaseWaitRunnable.java:82)
	at org.openstreetmap.josm.gui.PleaseWaitRunnable.run(PleaseWaitRunnable.java:145)
	at java.util.concurrent.Executors$RunnableAdapter.call(Unknown Source)
	at java.util.concurrent.FutureTask$Sync.innerRun(Unknown Source)
	at java.util.concurrent.FutureTask.run(Unknown Source)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(Unknown Source)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(Unknown Source)
	at java.lang.Thread.run(Unknown Source)
Caused by: java.lang.reflect.InvocationTargetException
	at java.awt.EventQueue.invokeAndWait(Unknown Source)
	at javax.swing.SwingUtilities.invokeAndWait(Unknown Source)
	at org.openstreetmap.josm.gui.dialogs.ValidatorDialog$FixTask.realRun(ValidatorDialog.java:584)
	... 8 more
Caused by: java.lang.IllegalArgumentException: {Way id=199399759 version=1 MVDT nodes=[]} is already deleted
	at org.openstreetmap.josm.command.DeleteCommand.executeCommand(DeleteCommand.java:119)
	at org.openstreetmap.josm.command.SequenceCommand.executeCommand(SequenceCommand.java:54)
	at org.openstreetmap.josm.data.UndoRedoHandler.addNoRedraw(UndoRedoHandler.java:36)
	at org.openstreetmap.josm.gui.dialogs.ValidatorDialog$FixTask$1.run(ValidatorDialog.java:587)
	at java.awt.event.InvocationEvent.dispatch(Unknown Source)
	at java.awt.EventQueue.dispatchEventImpl(Unknown Source)
	at java.awt.EventQueue.access$200(Unknown Source)
	at java.awt.EventQueue$3.run(Unknown Source)
	at java.awt.EventQueue$3.run(Unknown Source)
	at java.security.AccessController.doPrivileged(Native Method)
	at java.security.ProtectionDomain$1.doIntersectionPrivilege(Unknown Source)
	at java.awt.EventQueue.dispatchEvent(Unknown Source)
	at java.awt.EventDispatchThread.pumpOneEventForFilters(Unknown Source)
	at java.awt.EventDispatchThread.pumpEventsForFilter(Unknown Source)
	at java.awt.EventDispatchThread.pumpEventsForFilter(Unknown Source)
	at java.awt.WaitDispatchSupport$2.run(Unknown Source)
	at java.awt.WaitDispatchSupport$4.run(Unknown Source)
	at java.security.AccessController.doPrivileged(Native Method)
	at java.awt.WaitDispatchSupport.enter(Unknown Source)
	at java.awt.Dialog.show(Unknown Source)
	at java.awt.Component.show(Unknown Source)
	at java.awt.Component.setVisible(Unknown Source)
	at java.awt.Window.setVisible(Unknown Source)
	at java.awt.Dialog.setVisible(Unknown Source)
	at org.openstreetmap.josm.gui.progress.PleaseWaitProgressMonitor$4.run(PleaseWaitProgressMonitor.java:170)
	at java.awt.event.InvocationEvent.dispatch(Unknown Source)
	at java.awt.EventQueue.dispatchEventImpl(Unknown Source)
	at java.awt.EventQueue.access$200(Unknown Source)
	at java.awt.EventQueue$3.run(Unknown Source)
	at java.awt.EventQueue$3.run(Unknown Source)
	at java.security.AccessController.doPrivileged(Native Method)
	at java.security.ProtectionDomain$1.doIntersectionPrivilege(Unknown Source)
	at java.awt.EventQueue.dispatchEvent(Unknown Source)
	at java.awt.EventDispatchThread.pumpOneEventForFilters(Unknown Source)
	at java.awt.EventDispatchThread.pumpEventsForFilter(Unknown Source)
	at java.awt.EventDispatchThread.pumpEventsForHierarchy(Unknown Source)
	at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
	at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
	at java.awt.EventDispatchThread.run(Unknown Source)

comment:6 by skyper, 13 years ago

@geri-oc:

Please attach your .osm file. Thanks.

comment:7 by geri-oc, 13 years ago

Ich kann die Datei oder den Bereich nicht wiederfinden. Werde es beim nächsten Mal berücksichtigen.

comment:8 by Don-vip, 13 years ago

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

comment:9 by Don-vip, 13 years ago

Cc:	naoliv added

by naoliv, 13 years ago

Attachment:	validator-error.osm.bz2 added

The attached file causes this issue. Run validator, select the error category and click fix.

comment:10 by skyper, 13 years ago

Your attached file contains two ways with only one node (http://www.openstreetmap.org/browse/way/201303565/history and http://www.openstreetmap.org/browse/way/201303565/history) created by Potlatch 2. Seems to be a bug in Potlatch 2.

JOSM might want to silent delete these ways.

Last edited 13 years ago by skyper (previous) (diff)

comment:11 by Don-vip, 13 years ago

These two single-node ways are erroneous in two categories: "one-node way" and "duplicated ways".
Fixing all errors at once make JOSM try to delete one of these ways twice.

comment:12 by Don-vip, 13 years ago

Resolution:	→ fixed
Status:	new → closed

In 5667/josm:

fix #8096 - Do not consider deleted primitives when fixing duplicate ones

comment:13 by anonymous, 13 years ago

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

Modify Ticket

Change Properties

Summary:
Type:		Priority:
Milestone:		Component:
Version:		Keywords:
Cc:	Set your email in Preferences

Action

leave as closed The owner will remain team.

change resolution as The resolution will be set.

reopen The resolution will be deleted. Next status will be 'reopened'.

Add Comment

Your email or username:

E-mail address and name can be saved in the Preferences .

You may use WikiFormatting here.

Attachments ↑ Description ↑

Note: See TracTickets for help on using tickets.

Download in other formats: