Modify

Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#12262 closed defect (fixed)

NPE when trying to open specific turn restriction in relation editor

Reported by: hjart Owned by: Don-vip
Priority: critical Milestone: 15.12
Component: Core Version: latest
Keywords: regression Cc:

Description (last modified by hjart)

When trying to open the broken turn restriction http://www.openstreetmap.org/relation/5814388 with the "to" way selected (http://www.openstreetmap.org/way/388841919) I get a NPE:

{{{
URL:http://josm.openstreetmap.de/svn/trunk
Repository:UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
Last:Changed Date: 2015-12-31 03:24:56 +0100 (Thu, 31 Dec 2015)
Build-Date:2015-12-31 02:33:56
Revision:9227
Relative:URL: ^/trunk

Identification: JOSM/1.5 (9227 da) Linux openSUSE 13.2 (Harlequin) (x86_64)
Memory Usage: 874 MB / 1328 MB (425 MB allocated, but free)
Java version: 1.8.0_65, Oracle Corporation, OpenJDK 64-Bit Server VM
Dataset consistency test: No problems found

Plugins:
- DirectUpload (31871)
- ImproveOsm (18)
- InfoMode (31772)
- Mapillary (31884)
- SimplifyArea (31772)
- apache-commons (31772)
- apache-http (31875)
- buildings_tools (31772)
- rasterfilters (31772)
- reverter (31772)
- scoutsigns (71)
- todo (29154)
- undelete (31772)
- utilsplugin2 (31890)
- wikipedia (31866)

Last errors/warnings:
- E: java.lang.NullPointerException
- E: java.lang.NullPointerException
- E: java.lang.NullPointerException
- E: java.lang.NullPointerException
- E: java.lang.NullPointerException

java.lang.NullPointerException
	at org.openstreetmap.josm.tools.ColorHelper.getForegroundColor(ColorHelper.java:84)
	at org.openstreetmap.josm.gui.util.GuiHelper.setBackgroundReadable(GuiHelper.java:380)
	at org.openstreetmap.josm.gui.dialogs.relation.MemberTableCellRenderer.renderBackgroundForeground(MemberTableCellRenderer.java:55)
	at org.openstreetmap.josm.gui.dialogs.relation.MemberTableRoleCellRenderer.getTableCellRendererComponent(MemberTableRoleCellRenderer.java:18)
	at javax.swing.JTable.prepareRenderer(JTable.java:5723)
	at javax.swing.plaf.synth.SynthTableUI.paintCell(SynthTableUI.java:683)
	at javax.swing.plaf.synth.SynthTableUI.paintCells(SynthTableUI.java:580)
	at javax.swing.plaf.synth.SynthTableUI.paint(SynthTableUI.java:364)
	at javax.swing.plaf.synth.SynthTableUI.update(SynthTableUI.java:275)
	at javax.swing.JComponent.paintComponent(JComponent.java:780)
	at javax.swing.JComponent.paint(JComponent.java:1056)
	at javax.swing.JComponent.paintChildren(JComponent.java:889)
	at javax.swing.JComponent.paint(JComponent.java:1065)
	at javax.swing.JViewport.paint(JViewport.java:728)
	at javax.swing.JComponent.paintChildren(JComponent.java:889)
	at javax.swing.JComponent.paint(JComponent.java:1065)
	at javax.swing.JComponent.paintChildren(JComponent.java:889)
	at javax.swing.JComponent.paint(JComponent.java:1065)
	at javax.swing.JComponent.paintChildren(JComponent.java:889)
	at javax.swing.JSplitPane.paintChildren(JSplitPane.java:1047)
	at javax.swing.JComponent.paint(JComponent.java:1065)
	at javax.swing.JComponent.paintChildren(JComponent.java:889)
	at javax.swing.JComponent.paint(JComponent.java:1065)
	at javax.swing.JComponent.paintChildren(JComponent.java:889)
	at javax.swing.JSplitPane.paintChildren(JSplitPane.java:1047)
	at javax.swing.JComponent.paint(JComponent.java:1065)
	at javax.swing.JComponent.paintChildren(JComponent.java:889)
	at javax.swing.JComponent.paint(JComponent.java:1065)
	at javax.swing.JComponent.paintChildren(JComponent.java:889)
	at javax.swing.JComponent.paint(JComponent.java:1065)
	at javax.swing.JComponent.paintChildren(JComponent.java:889)
	at javax.swing.JComponent.paint(JComponent.java:1065)
	at javax.swing.JComponent.paintToOffscreen(JComponent.java:5210)
	at javax.swing.BufferStrategyPaintManager.paint(BufferStrategyPaintManager.java:290)
	at javax.swing.RepaintManager.paint(RepaintManager.java:1272)
	at javax.swing.JComponent._paintImmediately(JComponent.java:5158)
	at javax.swing.JComponent.paintImmediately(JComponent.java:4969)
	at javax.swing.RepaintManager$4.run(RepaintManager.java:831)
	at javax.swing.RepaintManager$4.run(RepaintManager.java:814)
	at java.security.AccessController.doPrivileged(Native Method)
	at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(ProtectionDomain.java:76)
	at javax.swing.RepaintManager.paintDirtyRegions(RepaintManager.java:814)
	at javax.swing.RepaintManager.paintDirtyRegions(RepaintManager.java:789)
	at javax.swing.RepaintManager.prePaintDirtyRegions(RepaintManager.java:738)
	at javax.swing.RepaintManager.access$1200(RepaintManager.java:64)
	at javax.swing.RepaintManager$ProcessingRunnable.run(RepaintManager.java:1732)
	at java.awt.event.InvocationEvent.dispatch(InvocationEvent.java:311)
	at java.awt.EventQueue.dispatchEventImpl(EventQueue.java:756)
	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.awt.EventQueue.dispatchEvent(EventQueue.java:726)
	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 (6)

comment:1 Changed 4 years ago by hjart

Description: modified (diff)

comment:2 Changed 4 years ago by Don-vip

Keywords: regression added
Owner: changed from team to Don-vip
Status: newassigned

comment:3 Changed 4 years ago by Don-vip

Milestone: 15.12

comment:4 Changed 4 years ago by Don-vip

Resolution: fixed
Status: assignedclosed

In 9229/josm:

fix #12262 - NPE

comment:5 in reply to:  3 Changed 4 years ago by Klumbumbus

Should be milestone 16.01? Or do you plan a hotfix?

Last edited 4 years ago by Klumbumbus (previous) (diff)

comment:6 Changed 4 years ago by Don-vip

hotfix released, the impact could have been huge.

Modify Ticket

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