Opened 14 years ago

Closed 14 years ago

#4612 closed defect (fixed)

delete members while relation dialog is open leads to data integrity problem

Reported by: bastiK Owned by: team
Priority: critical Milestone:
Component: Core Version:
Keywords: Cc:


What steps will reproduce the problem?

  1. create a relation
  2. remove some members while the dialog is open
  3. make changes in the relation dialog
  4. confirm the relation dialog -> conflicts
  5. try to resolve these conflicts -> error

Please provide any additional information below. Attach a screenshot if

Build-Date: 2010-02-24 19:08:30		
Revision: 2965
Is-Local-Build: true

Memory Usage: 42 MB / 297 MB (10 MB allocated, but free)
Java version: 1.6.0_15, Sun Microsystems Inc., Linux

Dataset consistency test:
No problems found

Plugins: buildings_tools,ghost,public_transport,validator
Plugin public_transport Version: 20072
Plugin buildings_tools Version: 32
Plugin validator Version: 19688
Plugin ghost Version: 1 Primitive  must be part of the dataset: {Relation id=-23777 version=0 MV [way -23772, node -23770, node -23771, node -23773, node -23769]}
	at org.openstreetmap.josm.gui.conflict.pair.ConflictResolver.populate(
	at org.openstreetmap.josm.gui.dialogs.ConflictDialog.resolve(
	at org.openstreetmap.josm.gui.dialogs.ConflictDialog.access$000(
	at org.openstreetmap.josm.gui.dialogs.ConflictDialog$ResolveAction.actionPerformed(
	at javax.swing.AbstractButton.fireActionPerformed(
	at javax.swing.AbstractButton$Handler.actionPerformed(
	at javax.swing.DefaultButtonModel.fireActionPerformed(
	at javax.swing.DefaultButtonModel.setPressed(
	at javax.swing.plaf.basic.BasicButtonListener.mouseReleased(
	at java.awt.AWTEventMulticaster.mouseReleased(
	at java.awt.Component.processMouseEvent(
	at javax.swing.JComponent.processMouseEvent(
	at java.awt.Component.processEvent(
	at java.awt.Container.processEvent(
	at java.awt.Component.dispatchEventImpl(
	at java.awt.Container.dispatchEventImpl(
	at java.awt.Component.dispatchEvent(
	at java.awt.LightweightDispatcher.retargetMouseEvent(
	at java.awt.LightweightDispatcher.processMouseEvent(
	at java.awt.LightweightDispatcher.dispatchEvent(
	at java.awt.Container.dispatchEventImpl(
	at java.awt.Window.dispatchEventImpl(
	at java.awt.Component.dispatchEvent(
	at java.awt.EventQueue.dispatchEvent(
	at java.awt.EventDispatchThread.pumpOneEventForFilters(
	at java.awt.EventDispatchThread.pumpEventsForFilter(
	at java.awt.EventDispatchThread.pumpEventsForHierarchy(
	at java.awt.EventDispatchThread.pumpEvents(
	at java.awt.EventDispatchThread.pumpEvents(

Attachments (0)

Change History (6)

comment:1 by mjulius, 14 years ago

Resolution: fixed
Status: newclosed

(In [3042]) fixes #4612 - delete members while relation dialog is open leads to data integrity problem

don't require primitive to be part of a dataset in OsmPrimitive.getReferrers()

comment:2 by jttt, 14 years ago

I've reverted the fix, the check for dataset is there for reason. getReferrers() on primitive without dataset returns random data because every clone of Way/Relation will add new item to referrers list.

comment:3 by anonymous, 14 years ago

Resolution: fixed
Status: closedreopened

comment:4 by jttt, 14 years ago

How should I fix this? Is it good idea to remove the deleted member also from relation dialog, so that no conflict will be created at all?

comment:5 by bastiK, 14 years ago

I think it would be best if any change in the dataset triggers and update of the relation dialog. This would make the "save/apply" button obsolete.

comment:6 by jttt, 14 years ago

Resolution: fixed
Status: reopenedclosed

(In [3061]) Fix #4612 delete members while relation dialog is open leads to data integrity problem

Modify Ticket

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