Modify

Opened 15 years ago

Closed 15 years ago

#3575 closed defect (fixed)

Nodes not merge in way downloader

Reported by: Nakor Owned by: Nakor
Priority: critical Milestone:
Component: Plugin Version: latest
Keywords: Cc:

Description

When th eplugin way downloader detects a duplicate nodes, a dialog pops up to merge them. If one click "Yes" the nodes are not merged.

Attachments (0)

Change History (14)

comment:1 by Harry Wood, 15 years ago

hmmm yes. somebody's renamed/removed a mergeNodes method

java.lang.NoSuchMethodError: org.openstreetmap.josm.actions.MergeNodesAction.mergeNodes(Ljava/util/LinkedList;Lorg/openstreetmap/josm/data/osm/Node;)Lorg/openstreetmap/josm/data/osm/Node;

at WayDownloaderPlugin$WayDownloadAction.run(WayDownloaderPlugin.java:140)
at java.util.concurrent.ThreadPoolExecutor$Worker.runTask(Unknown Source)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(Unknown Source)
at java.lang.Thread.run(Unknown Source)

comment:2 by stoecker, 15 years ago

Resolution: fixed
Status: newclosed

Update the plugin.

comment:3 by Nakor, 15 years ago

Resolution: fixed
Status: closedreopened

I can still reproduce the issue in josm-latest after updating the plugin

comment:4 by Gubaer, 15 years ago

Resolution: worksforme
Status: reopenedclosed

There's a plugin compatible with JOSM > r2098. Please upgrade both. This should resolve the issue.

comment:5 by Nakor, 15 years ago

Resolution: worksforme
Status: closedreopened

I downloaded nightly build 2217, went to Edit, Preferences, Plugins and then hit the update button.

I can still reproduce the bug. Should I do something else?

comment:6 by Gubaer, 15 years ago

Owner: changed from team to Nakor
Status: reopenedneedinfo

I did another build for waydownloader plugin, see [o17886]. It should be compatible with r2217.

Please upgrade agrain, I hope this helps.

comment:7 by anonymous, 15 years ago

I am still getting 17885 when I update from the settings. Should I do something to get 17886?

comment:8 by Gubaer, 15 years ago

[o17885] is fine, it's the jars current revision ID.

Does it work with [o17885]?

comment:9 by Nakor, 15 years ago

Unfortunately not. The error message in the console is gone but the merge does not happen.

comment:10 by stoecker, 15 years ago

Owner: changed from Nakor to Gubaer
Status: needinfonew

comment:11 by stoecker, 15 years ago

Priority: normalcritical

comment:12 by stoecker, 15 years ago

Component: unspecifiedPlugin

comment:13 by Gubaer, 15 years ago

Owner: changed from Gubaer to Nakor
Status: newneedinfo

Is this still an issue?

I tried a couple of way downloads but I didn't run into the problem.

comment:14 by Gubaer, 15 years ago

Resolution: fixed
Status: needinfoclosed

Should be fixed in [o18613]

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Nakor.
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.