IllegalStateException in the reverter plugin when reverting a changeset

[rickmastfan67]

This is the result of the changes to the core in [5914] that fixed a NPE that was triggered in #8635.

Steps to reproduce:

1. Have the "reverter" plugin installed (if you don't, install and restart and then continue to Step 2).
2. History > Revert Changeset
3. Revert the following changeset: 9728028

What happens:
During the revering process for said changeset, you get a IllegalStateException because of a missing merge target for a node.

What should happen:
The revert of the changeset should be finished and the "conflict" window should show up with any conflicts that should be fixed.

Repository Root: http://josm.openstreetmap.de/svn
Build-Date: 2013-04-29 01:35:36
Last Changed Author: Don-vip
Revision: 5914
Repository UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
URL: http://josm.openstreetmap.de/svn/trunk
Last Changed Date: 2013-04-29 02:34:17 +0200 (Mon, 29 Apr 2013)
Last Changed Rev: 5914

Identification: JOSM/1.5 (5914 en) Windows 7 64-Bit
Memory Usage: 218 MB / 2730 MB (111 MB allocated, but free)
Java version: 1.7.0_21, Oracle Corporation, Java HotSpot(TM) 64-Bit Server VM
VM arguments: [-Xmx3072M]
Dataset consistency test: No problems found

Plugin: ImageryCache (29484)
Plugin: OpeningHoursEditor (29435)
Plugin: buildings_tools (29435)
Plugin: mapdust (29525)
Plugin: measurement (29534)
Plugin: mirrored_download (29505)
Plugin: openstreetbugs (29435)
Plugin: osmarender (29435)
Plugin: reverter (29503)
Plugin: turnrestrictions (29435)
Plugin: undelete (29538)
Plugin: utilsplugin2 (29435)

java.lang.IllegalStateException: Missing merge target for node with id 1,490,137,701
	at reverter.ChangesetReverter.getCommands(ChangesetReverter.java:341)
	at reverter.RevertChangesetTask.realRun(RevertChangesetTask.java:87)
	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)

[Don-vip]

Hopefully resolving this ticket would restore this plugin to a functional state, and would eliminate the first source of nodes without coordinates.

[Don-vip]

Fixed in [o29548].
However, it does not yet fixes all the issues of this plugin...