Modify

Opened 15 years ago

Closed 15 years ago

#2195 closed enhancement (fixed)

[PATCH] Werkzeug "Überlappende Flächen verbinden" erzeugt Fehlermeldung

Reported by: michael@… Owned by: xeen
Priority: minor Milestone:
Component: unspecified Version:
Keywords: Cc:

Description

Erstellen zweier Flächen (landuse=commercial; name=y und landuse=residential; name=x)
Verbunden sind beide Flächen nur zwischen zwei Nodes
Überprüfung mit Validator
Fenster "Ergebnisse der Datenprüfung" zeigt an:
-> Andere-> Überlappende Flächen (1)-> 2 Wege: y,...
"2 Wege: y,..." wird im Fenster "Ergebnisse der Datenprüfung" ausgewählt
Beide Flächen sind markiert
Umschalt+j, Warnhinweis (experimentell) erscheint, fortfahren
Fenster "Werte für alle Konflikte festlegen" erscheint,
Auswahlvorgaben für "landuse" und "name" werden von "commercial" und "x" auf "residential" und "y" geändert
"Konflikte auflösen" wird betätigt
Folgende Fehlermeldung wird ausgegeben:

Path: trunk
URL: http://josm.openstreetmap.de/svn/trunk
Repository Root: http://josm.openstreetmap.de/svn
Repository UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
Revision: 1418
Node Kind: directory
Last Changed Author: stoecker
Last Changed Rev: 1418
Last Changed Date: 2009-02-16 15:24:57 +0100 (Mon, 16 Feb 2009)

Java version: 1.6.0_0
Plugins: AgPifoJ;openstreetbugs;remotecontrol;usertools;utilsplugin;validator;wmsplugin
Plugin AgPifoJ Version: 13409
Plugin openstreetbugs Version: 13477
Plugin remotecontrol Version: 13409
Plugin usertools Version: 13455
Plugin utilsplugin Version: 13723
Plugin validator Version: 13598
Plugin wmsplugin Version: 13741

java.lang.NullPointerException
	at org.openstreetmap.josm.data.UndoRedoHandler.add(UndoRedoHandler.java:38)
	at UtilsPlugin.JoinAreasAction.commitCommands(JoinAreasAction.java:444)
	at UtilsPlugin.JoinAreasAction.joinAreas(JoinAreasAction.java:239)
	at UtilsPlugin.JoinAreasAction.actionPerformed(JoinAreasAction.java:190)
	at javax.swing.AbstractButton.fireActionPerformed(AbstractButton.java:2012)
	at javax.swing.AbstractButton$Handler.actionPerformed(AbstractButton.java:2335)
	at javax.swing.DefaultButtonModel.fireActionPerformed(DefaultButtonModel.java:404)
	at javax.swing.DefaultButtonModel.setPressed(DefaultButtonModel.java:259)
	at javax.swing.AbstractButton.doClick(AbstractButton.java:374)
	at javax.swing.plaf.basic.BasicMenuItemUI.doClick(BasicMenuItemUI.java:1688)
	at javax.swing.plaf.basic.BasicMenuItemUI$Handler.mouseReleased(BasicMenuItemUI.java:1732)
	at java.awt.AWTEventMulticaster.mouseReleased(AWTEventMulticaster.java:289)
	at java.awt.Component.processMouseEvent(Component.java:6106)
	at javax.swing.JComponent.processMouseEvent(JComponent.java:3276)
	at java.awt.Component.processEvent(Component.java:5871)
	at java.awt.Container.processEvent(Container.java:2105)
	at java.awt.Component.dispatchEventImpl(Component.java:4467)
	at java.awt.Container.dispatchEventImpl(Container.java:2163)
	at java.awt.Component.dispatchEvent(Component.java:4293)
	at java.awt.LightweightDispatcher.retargetMouseEvent(Container.java:4461)
	at java.awt.LightweightDispatcher.processMouseEvent(Container.java:4125)
	at java.awt.LightweightDispatcher.dispatchEvent(Container.java:4055)
	at java.awt.Container.dispatchEventImpl(Container.java:2149)
	at java.awt.Window.dispatchEventImpl(Window.java:2478)
	at java.awt.Component.dispatchEvent(Component.java:4293)
	at java.awt.EventQueue.dispatchEvent(EventQueue.java:604)
	at java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:275)
	at java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:200)
	at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:190)
	at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:185)
	at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:177)
	at java.awt.EventDispatchThread.run(EventDispatchThread.java:138)


Beide Flächen vereint.
Ich bin einige Male wie oben beschrieben vorgegangen. In den meisten Fällen war das Ergebnis so wie gewünscht und vorgesehen. Beide Flächen waren vereint unter den gewünschten Vorgaben. Es kam allerdings auch ein mal vor, dass keine Veränderung eintrat, also beide Flächen nach shift+j unverändert blieben. Einmal kam es vor, dass der Validator weiterhin "-> Andere-> Überlappende Flächen (1)-> 2 Wege: y,..." anmeckerte. Erst nach dem Upload, erneutem Starten von Josm und erneutem Download des betreffenden Gebietes war diese Fehlermeldung verschwunden. Beide Fehler waren nicht reproduzierbar.
hth

Attachments (1)

Fix NullPointer when there are no inner ways.patch (62.9 KB ) - added by xeen 15 years ago.
Patch looks huge but isn't: I've updated the test file and JOSM didn't consider re-writing it diff friendly

Download all attachments as: .zip

Change History (5)

comment:1 by anonymous, 15 years ago

Owner: changed from framm to xeen

comment:2 by xeen, 15 years ago

Deutsche Version unten

  • Error message with stack trace: Patch coming
  • If specific areas are not joined properly do the following:
    • undo joining (if it was applied, look in the recent commands dialog)
    • Save (File -> Save As) your current layer to an osm file
    • zip it and upload it to a new bug
    • If not obviously clear, explain which areas you wanted to join or provide coordinates
      • Zoom to "critical area" (where the actual joining will happen)
      • Open Jump To Dialog by clicking the lon/lat in the status bar
      • Copy OSM Url and paste here
    • explain how they should look like when join properly
  • Validator issue might be related to #1405. This is no Join Areas problem though. Try Re-Validating everything using the button in the Validator dialog.

(German Version)

  • Fehlermeldung: ist behoben, Patch kommt
  • Wenn bestimmte Gebiete sich nicht verbinden lassen, mach doch bitte das:
    • Das verbinden rückgängig machen, falls es angewendet wurde (im kürzliche Aktionen Dialog nachsehen)
    • Den aktuellen Layer speichern (Datei -> Speichern unter)
    • Gezippt an einen neuen Bug anhängen
    • Wenn nicht sofort klar ist, welche Gebiete gemeint sind bitte beschreiben ob die Koordinaten angeben. Das geht am einfachsten so:
      • auf kritisches Gebiet zoomen (also da wo sich beim Verbinden auch was ändert)
      • Springe Zu Dialog durch klicken auf die Lon/Lat Anzeige in der Statusleiste
      • OSM URL kopieren und an den Bug heften
    • Das Problem mit dem Validator könnte mit #1405 zusammenhängen. Auf jeden Fall ist das kein spezielles Problem von JoinAreas. Probier das nächste mal im Validator Dialog alles erneut zu überprüfen, evtl. geht die alte Fehlermeldung dann weg

by xeen, 15 years ago

Patch looks huge but isn't: I've updated the test file and JOSM didn't consider re-writing it diff friendly

comment:3 by xeen, 15 years ago

Summary: Werkzeug "Überlappende Flächen verbinden" erzeugt Fehlermeldung[PATCH] Werkzeug "Überlappende Flächen verbinden" erzeugt Fehlermeldung

Please close once patch applied. Other issues too imprecise, open a new bug instead when they occur again

(bitte einen neuen Bug öffnen, falls die anderen Fehler nochmal auftauchen)

comment:4 by anonymous, 15 years ago

Resolution: fixed
Status: newclosed

Fix in 13776.

Modify Ticket

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