Modify

Opened 6 years ago

Closed 6 years ago

#9162 closed defect (fixed)

IllegalArgumentException while reverting a changeset

Reported by: naoliv Owned by: Upliner
Priority: normal Milestone:
Component: Plugin reverter Version:
Keywords: template_report Cc:

Description

What steps will reproduce the problem?

  1. With the revert plugin start reverting the following changesets: 17487949, 17487832, 17487816
  2. Then, revert 17479071 and it crashes
Repository Root: http://josm.openstreetmap.de/svn
Build-Date: 2013-10-07 01:35:18
Last Changed Author: Don-vip
Revision: 6311
Repository UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
URL: http://josm.openstreetmap.de/svn/trunk
Last Changed Date: 2013-10-07 01:49:19 +0200 (Mon, 07 Oct 2013)
Last Changed Rev: 6311

Identification: JOSM/1.5 (6311 pt_BR) Linux Debian GNU/Linux unstable (sid)
Memory Usage: 343 MB / 1756 MB (200 MB allocated, but free)
Java version: 1.7.0_25, Oracle Corporation, OpenJDK 64-Bit Server VM
Java package: openjdk-7-jre:amd64-7u25-2.3.12-4
VM arguments: [-Djava.net.useSystemProxies=true, -Dawt.useSystemAAFontSettings=on]
Dataset consistency test: No problems found

Plugin: AddrInterpolation (29854)
Plugin: Create_grid_of_ways (29854)
Plugin: ImageryCache (29946)
Plugin: PicLayer (29854)
Plugin: SimplifyArea (29854)
Plugin: buildings_tools (29854)
Plugin: editgpx (29854)
Plugin: geotools (29767)
Plugin: jts (29854)
Plugin: merge-overlap (29854)
Plugin: opendata (29938)
Plugin: pdfimport (29913)
Plugin: reverter (30000)
Plugin: tagging-preset-tester (29864)
Plugin: turnrestrictions (29854)
Plugin: undelete (29854)
Plugin: utilsplugin2 (30002)

java.lang.IllegalArgumentException: New way {Way id=234848858 version=3 MT> nodes=[]} has 0 nodes
	at org.openstreetmap.josm.command.ChangeCommand.sanityChecks(ChangeCommand.java:50)
	at org.openstreetmap.josm.command.ChangeCommand.<init>(ChangeCommand.java:34)
	at reverter.DataSetCommandMerger.addChangeCommandIfNotEquals(DataSetCommandMerger.java:47)
	at reverter.DataSetCommandMerger.mergeWay(DataSetCommandMerger.java:111)
	at reverter.DataSetCommandMerger.merge(DataSetCommandMerger.java:154)
	at reverter.DataSetCommandMerger.<init>(DataSetCommandMerger.java:42)
	at reverter.ChangesetReverter.getCommands(ChangesetReverter.java:335)
	at reverter.RevertChangesetTask.realRun(RevertChangesetTask.java:88)
	at org.openstreetmap.josm.gui.PleaseWaitRunnable.doRealRun(PleaseWaitRunnable.java:82)
	at org.openstreetmap.josm.gui.PleaseWaitRunnable.run(PleaseWaitRunnable.java:150)
	at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:471)
	at java.util.concurrent.FutureTask$Sync.innerRun(FutureTask.java:334)
	at java.util.concurrent.FutureTask.run(FutureTask.java:166)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1145)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:615)
	at java.lang.Thread.run(Thread.java:724)

Attachments (0)

Change History (2)

comment:1 Changed 6 years ago by Don-vip

Step 2 (reverting 17479071) is enough to reproduce, thanks.

comment:2 Changed 6 years ago by Don-vip

Resolution: fixed
Status: newclosed

Fixed in [o30008].

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Upliner.
as The resolution will be set.
The resolution will be deleted.

Add Comment


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

 
Note: See TracTickets for help on using tickets.