id	summary	reporter	owner	description	type	status	priority	milestone	component	version	resolution	keywords	cc
5162	Very Rare Relation Editing Display Code Bug	ezekielf@…	team	"What steps will reproduce the problem?
 1. download a relation (Ive experienced this with multipolygons)
 2. open edit relation window
 3. attempt to re-order relation members

What is the expected result?
I should be able to re-order relation members

What happens instead?
I get an error saying something like ""an unexpected error has occured, this is always a coding error, please file a bug report""


I'm marking this ""Major"" because I've created a bunch of fairly complex multipolygon relations using Potlach.  They render fine in Mapnik, but are completely screwed up in Osmarender.  I believe the problem is that Potlach doesn't allow you to specify an order for the relation members and Osmarender needs that order to render them properly even though Mapnik doesn't.  I'm attempting to make them render properly in osmarender by re-ordering the members in JOSM, but this bug is blocking me from doing that.  Clearly the order doesn't matter to Mapnik.  If this were true of all renderers then this would not be an issue, but that's beyond the scope of this bug report….



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

{{{
Repository Root: http://josm.openstreetmap.de/svn
Build-Date: 2010-06-11 01:31:34
Last Changed Author: stoecker
Revision: 3329
Repository UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
URL: http://josm.openstreetmap.de/svn/trunk
Last Changed Date: 2010-06-10 09:53:42 +0200 (Thu, 10 Jun 2010)
Last Changed Rev: 3329

Identification: JOSM/1.5 (3329 en)
Memory Usage: 102 MB / 123 MB (16 MB allocated, but free)
Java version: 1.6.0_20, Apple Inc., Java HotSpot(TM) 64-Bit Server VM
Operating system: Mac OS X
Dataset consistency test: No problems found


java.lang.NullPointerException
	at org.openstreetmap.josm.gui.dialogs.relation.MemberTableMemberCellRenderer.renderPrimitive(MemberTableMemberCellRenderer.java:19)
	at org.openstreetmap.josm.gui.dialogs.relation.MemberTableMemberCellRenderer.getTableCellRendererComponent(MemberTableMemberCellRenderer.java:33)
	at javax.swing.JTable$AccessibleJTable.getAccessibleChild(JTable.java:7023)
	at javax.swing.JTable$AccessibleJTable.getAccessibleAt(JTable.java:7410)
	at javax.swing.JTable$AccessibleJTable.valueChanged(JTable.java:6923)
	at javax.swing.DefaultListSelectionModel.fireValueChanged(DefaultListSelectionModel.java:167)
	at javax.swing.DefaultListSelectionModel.fireValueChanged(DefaultListSelectionModel.java:147)
	at javax.swing.DefaultListSelectionModel.fireValueChanged(DefaultListSelectionModel.java:194)
	at javax.swing.DefaultListSelectionModel.changeSelection(DefaultListSelectionModel.java:388)
	at javax.swing.DefaultListSelectionModel.changeSelection(DefaultListSelectionModel.java:398)
	at javax.swing.DefaultListSelectionModel.removeSelectionIntervalImpl(DefaultListSelectionModel.java:559)
	at javax.swing.DefaultListSelectionModel.clearSelection(DefaultListSelectionModel.java:403)
	at javax.swing.JTable.clearSelection(JTable.java:2080)
	at javax.swing.JTable.clearSelectionAndLeadAnchor(JTable.java:2088)
	at javax.swing.JTable.tableChanged(JTable.java:4433)
	at javax.swing.table.AbstractTableModel.fireTableChanged(AbstractTableModel.java:280)
	at javax.swing.table.AbstractTableModel.fireTableDataChanged(AbstractTableModel.java:182)
	at org.openstreetmap.josm.gui.dialogs.relation.MemberTableModel.moveUp(MemberTableModel.java:212)
	at org.openstreetmap.josm.gui.dialogs.relation.GenericRelationEditor$MoveUpAction.actionPerformed(GenericRelationEditor.java:936)
	at javax.swing.AbstractButton.fireActionPerformed(AbstractButton.java:2028)
	at javax.swing.AbstractButton$Handler.actionPerformed(AbstractButton.java:2351)
	at javax.swing.DefaultButtonModel.fireActionPerformed(DefaultButtonModel.java:387)
	at javax.swing.DefaultButtonModel.setPressed(DefaultButtonModel.java:242)
	at javax.swing.plaf.basic.BasicButtonListener.mouseReleased(BasicButtonListener.java:236)
	at java.awt.AWTEventMulticaster.mouseReleased(AWTEventMulticaster.java:272)
	at java.awt.Component.processMouseEvent(Component.java:6348)
	at javax.swing.JComponent.processMouseEvent(JComponent.java:3267)
	at java.awt.Component.processEvent(Component.java:6113)
	at java.awt.Container.processEvent(Container.java:2085)
	at java.awt.Component.dispatchEventImpl(Component.java:4714)
	at java.awt.Container.dispatchEventImpl(Container.java:2143)
	at java.awt.Component.dispatchEvent(Component.java:4544)
	at java.awt.LightweightDispatcher.retargetMouseEvent(Container.java:4618)
	at java.awt.LightweightDispatcher.processMouseEvent(Container.java:4282)
	at java.awt.LightweightDispatcher.dispatchEvent(Container.java:4212)
	at java.awt.Container.dispatchEventImpl(Container.java:2129)
	at java.awt.Window.dispatchEventImpl(Window.java:2478)
	at java.awt.Component.dispatchEvent(Component.java:4544)
	at java.awt.EventQueue.dispatchEvent(EventQueue.java:635)
	at java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:296)
	at java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:211)
	at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:201)
	at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:196)
	at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:188)
	at java.awt.EventDispatchThread.run(EventDispatchThread.java:122)

}}}
"	defect	closed	minor		Core		othersoftware	template_report