Modify

Opened 10 years ago

Closed 10 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 by Don-vip, 10 years ago

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

comment:2 by Don-vip, 10 years ago

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. 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.