Modify

Opened 8 years ago

Closed 8 years ago

#13262 closed defect (fixed)

PlugIn photoadjust zum Fotos verschieben bringt Fehlermeldung, wenn man auch Kartendaten bearbeiten möchte

Reported by: volkmarkreiss@… Owned by: holgermappt
Priority: normal Milestone:
Component: Plugin photoadjust Version:
Keywords: template_report Cc:

Description

What steps will reproduce the problem?

  1. Fotos an GPX korrellieren und einzelne verschieben
  2. Korrekturen an der Karte vornehmen
  3. Programm beenden

What is the expected result?

Ich werde aufgefordert das PlugIn zu deaktivieren oder upzudaten

What happens instead?

Please provide any additional information below. Attach a screenshot if possible.

URL:http://josm.openstreetmap.de/svn/trunk
Repository:UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
Last:Changed Date: 2016-07-31 23:02:22 +0200 (Sun, 31 Jul 2016)
Build-Date:2016-08-01 02:16:18
Revision:10693
Relative:URL: ^/trunk

Identification: JOSM/1.5 (10693 de) Linux Ubuntu 14.04.4 LTS
Memory Usage: 585 MB / 3563 MB (207 MB allocated, but free)
Java version: 1.8.0_91-8u91-b14-0ubuntu4~14.04-b14, Oracle Corporation, OpenJDK 64-Bit Server VM
Java package: openjdk-8-jre:amd64-8u91-b14-0ubuntu4~14.04
VM arguments: [-Djosm.restart=true, -Djosm.home=<josm.pref>, -Djava.net.useSystemProxies=true]

Plugins:
- FixAddresses (32445)
- HouseNumberTaggingTool (32584)
- Mapillary (32639)
- OpeningHoursEditor (32583)
- PicLayer (32495)
- apache-commons (32584)
- apache-http (32584)
- buildings_tools (32728)
- editgpx (32485)
- measurement (32732)
- osmarender (32501)
- photo_geotagging (32392)
- photoadjust (32713)
- print (32502)
- terracer (32426)
- wikipedia (32749)

Last errors/warnings:
- W: java.io.IOException: Attribution is not loaded yet
- W: java.io.IOException: Attribution is not loaded yet
- W: java.io.IOException: Attribution is not loaded yet
- W: java.io.IOException: Attribution is not loaded yet
- W: java.io.IOException: Attribution is not loaded yet
- W: java.io.IOException: Attribution is not loaded yet
- W: java.io.IOException: Attribution is not loaded yet
- W: java.io.IOException: Attribution is not loaded yet
- W: java.io.IOException: Attribution is not loaded yet
- E: java.lang.IllegalArgumentException: Attempted to remove listener that was not in list: org.openstreetmap.josm.plugins.photoadjust.PhotoAdjustMapMode@1c0013e1


=== REPORTED CRASH DATA ===
BugReportExceptionHandler$BugReporterThread#askForBugReport:
No data collected.

=== STACK TRACE ===
Thread: AWT-EventQueue-0 (18) of main
java.lang.IllegalArgumentException: Attempted to remove listener that was not in list: org.openstreetmap.josm.plugins.photoadjust.PhotoAdjustMapMode@1c0013e1
	at org.openstreetmap.josm.gui.layer.MainLayerManager.removeActiveLayerChangeListener(MainLayerManager.java:196)
	at org.openstreetmap.josm.plugins.photoadjust.PhotoAdjustMapMode.exitMode(PhotoAdjustMapMode.java:147)
	at org.openstreetmap.josm.gui.MapFrame.activeOrEditLayerChanged(MapFrame.java:768)
	at org.openstreetmap.josm.gui.layer.MainLayerManager.fireActiveLayerChange(MainLayerManager.java:254)
	at org.openstreetmap.josm.gui.layer.MainLayerManager.setActiveLayer(MainLayerManager.java:247)
	at org.openstreetmap.josm.gui.layer.MainLayerManager.realRemoveSingleLayer(MainLayerManager.java:279)
	at org.openstreetmap.josm.gui.layer.LayerManager.realRemoveLayer(LayerManager.java:219)
	at org.openstreetmap.josm.gui.layer.LayerManager.lambda$removeLayer$1(LayerManager.java:205)
	at org.openstreetmap.josm.gui.util.GuiHelper.runInEDTAndWaitWithException(GuiHelper.java:138)
	at org.openstreetmap.josm.gui.layer.LayerManager.removeLayer(LayerManager.java:205)
	at org.openstreetmap.josm.gui.layer.LayerManager.resetState(LayerManager.java:423)
	at org.openstreetmap.josm.gui.layer.MainLayerManager.resetState(MainLayerManager.java:392)
	at org.openstreetmap.josm.Main.shutdown(Main.java:1151)
	at org.openstreetmap.josm.gui.MainApplication.shutdown(MainApplication.java:116)
	at org.openstreetmap.josm.Main.exitJosm(Main.java:1132)
	at org.openstreetmap.josm.gui.MainFrame$1.windowClosing(MainFrame.java:94)
	at java.awt.AWTEventMulticaster.windowClosing(AWTEventMulticaster.java:349)
	at java.awt.AWTEventMulticaster.windowClosing(AWTEventMulticaster.java:349)
	at java.awt.Window.processWindowEvent(Window.java:2058)
	at javax.swing.JFrame.processWindowEvent(JFrame.java:305)
	at java.awt.Window.processEvent(Window.java:2017)
	at java.awt.Component.dispatchEventImpl(Component.java:4891)
	at java.awt.Container.dispatchEventImpl(Container.java:2294)
	at java.awt.Window.dispatchEventImpl(Window.java:2750)
	at java.awt.Component.dispatchEvent(Component.java:4713)
	at java.awt.EventQueue.dispatchEventImpl(EventQueue.java:758)
	at java.awt.EventQueue.access$500(EventQueue.java:97)
	at java.awt.EventQueue$3.run(EventQueue.java:709)
	at java.awt.EventQueue$3.run(EventQueue.java:703)
	at java.security.AccessController.doPrivileged(Native Method)
	at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(ProtectionDomain.java:76)
	at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(ProtectionDomain.java:86)
	at java.awt.EventQueue$4.run(EventQueue.java:731)
	at java.awt.EventQueue$4.run(EventQueue.java:729)
	at java.security.AccessController.doPrivileged(Native Method)
	at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(ProtectionDomain.java:76)
	at java.awt.EventQueue.dispatchEvent(EventQueue.java:728)
	at java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:201)
	at java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:116)
	at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:105)
	at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:101)
	at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:93)
	at java.awt.EventDispatchThread.run(EventDispatchThread.java:82)

Attachments (0)

Change History (8)

comment:1 by Klumbumbus, 8 years ago

Component: CorePlugin photoadjust
Owner: changed from team to holgermappt

comment:2 by holgermappt, 8 years ago

Owner: changed from holgermappt to volkmarkreiss@…
Status: newneedinfo

Tritt der Fehler mit der letzten Version 32796 des Plugins noch auf? Ich kann es nicht reproduzieren. Welche Ebene ist aktiv, während Fotos verschoben werden?

comment:3 by volkmarkreiss@…, 8 years ago

Die Fehlermeldung kommt nach wie vor zuverlässig, wenn ich josm beende (sowohl unter Win7 als auch unter Ubuntu 14.04.
Ich hatte mehrere Bilderebenen offen, einen GPX-Track und Bing Luftbild.
Der Fehler kommt, bevor die Bing Luftbildebene beendet werden soll.
PS: Ist auch noch geplant, dass ich mehrere Bilder mit einem Schwung bearbeiten kann? Machmal ist mein gpx-Track so schlecht, dass die Fotos durch die Häuserzeilen gehen oder ich möchte gerne von allen Fotos den Aufnahmewinkel um 45Grad zum GPX-Track nach links verschieben. So als Idee - wenn mal Zeit ist :-). Gerne per Mail volkmarkreiss@…

comment:4 by michael2402, 8 years ago

Owner: changed from volkmarkreiss@… to holgermappt
Status: needinfonew

Fehler gefunden ;-)

You need to delete this line to the enterMode() method:

        MapView.removeLayerChangeListener(this);

It does not seem to be intended. While doing this, can you switch to the new map interface using Main.getLayerManager()... - you will find all methods there.

in reply to:  4 comment:5 by holgermappt, 8 years ago

Replying to michael2402:

You need to delete this line to the enterMode() method:

        MapView.removeLayerChangeListener(this);

It does not seem to be intended. While doing this, can you switch to the new map interface using Main.getLayerManager()... - you will find all methods there.

We talk about the photoadjust plugin, file PhotoAdjustMapMode.java, right? There is no MapView.removeLayerChangeListener(this); in that file, just Main.getLayerManager().removeLayerChangeListener(this); in method exitMode(). Are you looking at an older version of that file?

in reply to:  3 comment:6 by holgermappt, 8 years ago

Replying to volkmarkreiss@…:

PS: Ist auch noch geplant, dass ich mehrere Bilder mit einem Schwung bearbeiten kann? Machmal ist mein gpx-Track so schlecht, dass die Fotos durch die Häuserzeilen gehen oder ich möchte gerne von allen Fotos den Aufnahmewinkel um 45Grad zum GPX-Track nach links verschieben. So als Idee - wenn mal Zeit ist :-).

Siehe auch #11905. Problem ist, dass JOSM nur ein Foto selektieren kann. Denkbar ist ein Dialog um all Fotos um x Meter nach Norden, y Meter nach Osten zu verschieben und den Winkel um z Grad zu drehen. GPX-Track auswählen, Fotos korrelieren, Richtung berechnen und Winkel anpassen klingt kompliziert. Zeit ist leider gerade keine :-(

comment:7 by holgermappt, 8 years ago

I'm able to reproduce:

  • download data
  • open photo with GPS data
    • the photo adjust map mode button is not active, why? It should be active if there are images with coordinates
  • activate geo image layer
    • photo adjust map mode button is activated
  • click on photo adjust map mode button
  • exit JOSM

comment:8 by holgermappt, 8 years ago

Resolution: fixed
Status: newclosed

This is fixed in photoadjust plugin version 32863. [o32863]

Bitte nochmal mit Plugin-Version 32863 probieren.

Modify Ticket

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