Modify

Opened 4 years ago

Closed 4 years ago

#3575 closed defect (fixed)

Nodes not merge in way downloader

Reported by: Nakor Owned by: Nakor
Priority: critical 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 Changed 4 years ago by Harry Wood

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 Changed 4 years ago by stoecker

  • Resolution set to fixed
  • Status changed from new to closed

Update the plugin.

comment:3 Changed 4 years ago by Nakor

  • Resolution fixed deleted
  • Status changed from closed to reopened

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

comment:4 Changed 4 years ago by Gubaer

  • Resolution set to worksforme
  • Status changed from reopened to closed

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

comment:5 Changed 4 years ago by Nakor

  • Resolution worksforme deleted
  • Status changed from closed to reopened

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 Changed 4 years ago by Gubaer

  • Owner changed from team to Nakor
  • Status changed from reopened to needinfo

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

Please upgrade agrain, I hope this helps.

comment:7 Changed 4 years ago by anonymous

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

comment:8 Changed 4 years ago by Gubaer

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

Does it work with [o17885]?

comment:9 Changed 4 years ago by Nakor

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

comment:10 Changed 4 years ago by stoecker

  • Owner changed from Nakor to Gubaer
  • Status changed from needinfo to new

comment:11 Changed 4 years ago by stoecker

  • Priority changed from normal to critical

comment:12 Changed 4 years ago by stoecker

  • Component changed from unspecified to Plugin

comment:13 Changed 4 years ago by Gubaer

  • Owner changed from Gubaer to Nakor
  • Status changed from new to needinfo

Is this still an issue?

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

comment:14 Changed 4 years ago by Gubaer

  • Resolution set to fixed
  • Status changed from needinfo to closed

Should be fixed in [o18613]

Add Comment

Modify Ticket

Change Properties
<Author field>
Action
as closed .
as The resolution will be set. Next status will be 'closed'.
The resolution will be deleted. Next status will be 'reopened'.
Author


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

 
Note: See TracTickets for help on using tickets.