Modify

Opened 5 years ago

Closed 5 years ago

#10258 closed defect (fixed)

reverter plugin doesn't revert

Reported by: malenki Owned by: Upliner
Priority: major Milestone:
Component: Plugin reverter Version: latest
Keywords: Cc: Don-vip

Description

As the subject reads.
When trying to revert a changeset the message just sais "loading changeset" – and that's it. No data shows up in the data layer.

Tested with
https://api.openstreetmap.org/api/0.6/changeset/24071517 and
https://api.openstreetmap.org/api/0.6/changeset/23109033
with both http and https API URL and a blank ~/.josm

Attachments (0)

Change History (8)

comment:1 Changed 5 years ago by malenki

Version: latest

comment:2 Changed 5 years ago by bastiK

Cannot reproduce. Please update plugins & add status report. What is the console output? For me it is

INFO: GET https://api.openstreetmap.org/api/0.6/changeset/24071517
INFO: GET https://api.openstreetmap.org/api/0.6/changeset/24071517/download
[...]
INFO: GET https://api.openstreetmap.org/api/0.6/way/179692051/1
INFO: GET https://api.openstreetmap.org/api/0.6/node/1901159958/1
[...]
INFO: Way 136,816,538 with 7 nodes has incomplete nodes because at least one node was missing in the loaded data.
INFO: Way 136,816,539 with 68 nodes has incomplete nodes because at least one node was missing in the loaded data.

comment:3 Changed 5 years ago by malenki

With a blank ~/.josm I need to load the plugin list and install the reverter plugin. I'd assume thus I should have recent versions. JOSM is r7303.

java version "1.7.0_55"
OpenJDK Runtime Environment (IcedTea 2.4.7) (7u55-2.4.7-2)
OpenJDK 64-Bit Server VM (build 24.51-b03, mixed mode)

console output:

 $ Lokale Revision ist 7303, neueste verfügbare Revision ist 7303 - benutze lokale Revision 7303
starte josm...
josm wurde mit mit Prozess-ID 22002 gestartet
INFO: Lade Plugin 'reverter' (Version 30436)
INFO: RemoteControl: adding command "revert_changeset" (handled by RevertChangesetHandler)
INFO: GET https://api.openstreetmap.org/api/0.6/changeset/24071517
INFO: GET https://api.openstreetmap.org/api/0.6/changeset/24071517/download

Last edited 5 years ago by malenki (previous) (diff)

comment:4 Changed 5 years ago by skyper

Can reproduce:

Repository Root: http://josm.openstreetmap.de/svn
Build-Date: 2014-07-11 01:36:22
Last Changed Author: Don-vip
Revision: 7303
Repository UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
URL: http://josm.openstreetmap.de/svn/trunk
Last Changed Date: 2014-07-11 02:59:09 +0200 (Fri, 11 Jul 2014)
Last Changed Rev: 7303

Identification: JOSM/1.5 (7303 en) Linux Debian GNU/Linux 7.6 (wheezy)
Memory Usage: 297 MB / 882 MB (100 MB allocated, but free)
Java version: 1.7.0_55, Oracle Corporation, OpenJDK 64-Bit Server VM
Java package: openjdk-7-jre:amd64-7u55-2.4.7-1~deb7u1
Dataset consistency test: No problems found

Plugin: OpeningHoursEditor (30519)
Plugin: conflation (0.1.7)
Plugin: imagery-xml-bounds (30495)
Plugin: jts (30416)
Plugin: mirrored_download (30495)
Plugin: notes (v0.9.2)
Plugin: photoadjust (30428)
Plugin: reverter (30436)
Plugin: terracer (30416)
Plugin: todo (29154)
Plugin: undelete (30416)
Plugin: utilsplugin2 (30460)
Plugin: wikipedia (30449)
INFO: GET https://api.openstreetmap.org/api/0.6/changeset/23109033
INFO: GET https://api.openstreetmap.org/api/0.6/changeset/23109033/download

comment:5 Changed 5 years ago by bastiK

Cc: Don-vip added

Regression from [7299].

comment:6 Changed 5 years ago by Don-vip

Woups, didn't see the plugin impact, sorry.

comment:7 Changed 5 years ago by Don-vip

Plugin recompiled in [o30521:30522] I don't know if it's enough to fix the issue.

comment:8 Changed 5 years ago by malenki

Resolution: fixed
Status: newclosed

Revert works now again.
Thanks for the quick fix!

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.