Modify ↓

Opened 6 years ago

Closed 5 years ago

Last modified 5 years ago

#15008 closed defect (fixed)

DataIntegrityProblemException: Primitive must be part of the dataset

Reported by: geri-oc Owned by: team
Priority: critical Milestone: 17.12
Component: Core Version:
Keywords: template_report selection event edt Cc: bastiK, stoecker, Klumbumbus, michael2402

Description (last modified by Don-vip)

translation from comment:1:

The error occurred after (accidentaly) unglue(?) all nodes of a way (instead of only one node) and pressing Undo.

What steps will reproduce the problem?

  1. Error after Edit -> Undo

What is the expected result?

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: 2017-07-03 00:44:07 +0200 (Mon, 03 Jul 2017)
Build-Date:2017-07-02 23:26:36
Revision:12450
Relative:URL: ^/trunk

Identification: JOSM/1.5 (12450 de) Windows Vista 32-Bit
OS Build number: Windows Vista (TM) Home Premium (6002)
Memory Usage: 398 MB / 989 MB (184 MB allocated, but free)
Java version: 1.8.0_131-b11, Oracle Corporation, Java HotSpot(TM) Client VM
Screen: \Display0 1440x900
Maximum Screen Size: 1440x900
Dataset consistency test: No problems found

Plugins:
+ FastDraw (33182)
+ PicLayer (33385)
+ SeaMapEditor (33054)
+ buildings_tools (33004)
+ measurement (33088)
+ public_transport (33166)
+ reverter (33088)
+ terracer (33088)
+ turnlanes (33294)
+ turnrestrictions (33088)
+ utilsplugin2 (33328)
+ wikipedia (33288)

Tagging presets:
+ https://josm.openstreetmap.de/josmfile?page=Presets/NewTags&zip=1
+ https://josm.openstreetmap.de/josmfile?page=Presets/Crafts&preset&zip=1
+ https://josm.openstreetmap.de/josmfile?page=Presets/OneClick&zip=1
+ https://raw.github.com/rurseekatze/OpenRailwayMap/master/josm-presets/de.xml
+ https://josm.openstreetmap.de/josmfile?page=Presets/Diving&zip=1
+ https://raw.githubusercontent.com/species/josm-preset-wheelchair/master/sidewalks_kerbs.xml
+ https://josm.openstreetmap.de/josmfile?page=Presets/Historical_Objects&zip=1
+ https://josm.openstreetmap.de/josmfile?page=Presets/Historic_Stone&zip=1
+ https://josm.openstreetmap.de/josmfile?page=Presets/PublicTransportOneClick&preset&zip=1
+ https://josm.openstreetmap.de/josmfile?page=Presets/hiking_routes_with_trail_marking&preset&zip=1
+ https://raw.github.com/Flacus/Windrad/master/windrad.xml
+ https://josm.openstreetmap.de/josmfile?page=Presets/LaneAttributes&zip=1
+ https://josm.openstreetmap.de/josmfile?page=Presets/ParkingLanes&zip=1
+ https://raw.githubusercontent.com/OpenNauticalChart/josm/master/Presets_Hafen.xml
+ https://raw.githubusercontent.com/yopaseopor/traffic_signs_preset_JOSM/master/DE.zip
+ https://raw.githubusercontent.com/OpenSeaMap/josm/master/Presets_Sport.xml
+ https://josm.openstreetmap.de/josmfile?page=Presets/hiking_routes_with_trail_marking&zip=1
+ https://josm.openstreetmap.de/josmfile?page=Presets/Golf_Course&zip=1
+ https://josm.openstreetmap.de/josmfile?page=Presets/Playground_Equipment&zip=1

Map paint styles:
- https://github.com/bastik/mapcss-tools/raw/osm/mapnik2mapcss/osm-results/mapnik.zip
- https://josm.openstreetmap.de/josmfile?page=Styles/Cycleways&zip=1
- https://josm.openstreetmap.de/josmfile?page=Styles/Lane_and_Road_Attributes&zip=1

Validator rules:
+ http://www.openrailwaymap.org/validator/de-openrailwaymap.validator.mapcss
+ https://josm.openstreetmap.de/josmfile?page=Rules/CzechRepublicAddressSystem&zip=1

Last errors/warnings:
- E: Fehler beim Laden des Bildes 'crossover-28.png'
- W:  Crossover: Could not get presets icon crossover-28.png
- E: Fehler beim Laden des Bildes 'milestone.png'
- W:  Milestone: Could not get presets icon milestone.png
- E: Fehler beim Laden des Bildes 'phone-32.png'
- W:  Phone (Railway): Could not get presets icon phone-32.png
- E: Fehler beim Laden des Bildes 'owner-change.svg'
- W:  Change of Operator: Could not get presets icon owner-change.svg
- E: Handled by bug report queue: org.openstreetmap.josm.data.osm.DataIntegrityProblemException: Primitive must be part of the dataset: {Node id=-40407 version=0 MV lat=51.1002497,lon=14.4010348}
- E: Handled by bug report queue: org.openstreetmap.josm.data.osm.DataIntegrityProblemException: Primitive must be part of the dataset: {Node id=-40407 version=0 MV lat=51.1002497,lon=14.4010348}


=== REPORTED CRASH DATA ===
BugReportExceptionHandler#handleException:
No data collected.

Warning issued by: BugReportExceptionHandler#handleException

=== STACK TRACE ===
Thread: AWT-EventQueue-0 (16) of main
org.openstreetmap.josm.data.osm.DataIntegrityProblemException: Primitive must be part of the dataset: {Node id=-40407 version=0 MV lat=51.1002497,lon=14.4010348}
	at org.openstreetmap.josm.data.osm.OsmPrimitive.checkDataset(OsmPrimitive.java:252)
	at org.openstreetmap.josm.data.osm.OsmPrimitive.getReferrers(OsmPrimitive.java:996)
	at org.openstreetmap.josm.data.osm.OsmPrimitive.getReferrers(OsmPrimitive.java:1021)
	at org.openstreetmap.josm.plugins.turnrestrictions.list.TurnRestrictionsInSelectionListModel.initFromSelection(TurnRestrictionsInSelectionListModel.java:40)
	at org.openstreetmap.josm.plugins.turnrestrictions.list.TurnRestrictionsInSelectionListModel.selectionChanged(TurnRestrictionsInSelectionListModel.java:66)
	at org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo.fire(SelectionEventManager.java:57)
	at org.openstreetmap.josm.data.osm.event.SelectionEventManager.fireEvent(SelectionEventManager.java:203)
	at org.openstreetmap.josm.data.osm.event.SelectionEventManager.lambda$selectionChanged$0(SelectionEventManager.java:198)
	at java.awt.event.InvocationEvent.dispatch(Unknown Source)
	at java.awt.EventQueue.dispatchEventImpl(Unknown Source)
	at java.awt.EventQueue.access$500(Unknown Source)
	at java.awt.EventQueue$3.run(Unknown Source)
	at java.awt.EventQueue$3.run(Unknown Source)
	at java.security.AccessController.doPrivileged(Native Method)
	at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(Unknown Source)
	at java.awt.EventQueue.dispatchEvent(Unknown Source)
	at java.awt.EventDispatchThread.pumpOneEventForFilters(Unknown Source)
	at java.awt.EventDispatchThread.pumpEventsForFilter(Unknown Source)
	at java.awt.EventDispatchThread.pumpEventsForHierarchy(Unknown Source)
	at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
	at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
	at java.awt.EventDispatchThread.run(Unknown Source)

=== RUNNING THREADS ===
Thread: RemoteControl HTTPS Server (39) of main
java.net.DualStackPlainSocketImpl.accept0(Native Method)
java.net.DualStackPlainSocketImpl.socketAccept(Unknown Source)
java.net.AbstractPlainSocketImpl.accept(Unknown Source)
java.net.PlainSocketImpl.accept(Unknown Source)
java.net.ServerSocket.implAccept(Unknown Source)
sun.security.ssl.SSLServerSocketImpl.accept(Unknown Source)
org.openstreetmap.josm.io.remotecontrol.RemoteControlHttpsServer.run(RemoteControlHttpsServer.java:407)

Thread: File Watcher (10) of main
sun.misc.Unsafe.park(Native Method)
java.util.concurrent.locks.LockSupport.park(Unknown Source)
java.util.concurrent.locks.AbstractQueuedSynchronizer$ConditionObject.await(Unknown Source)
java.util.concurrent.LinkedBlockingDeque.takeFirst(Unknown Source)
java.util.concurrent.LinkedBlockingDeque.take(Unknown Source)
sun.nio.fs.AbstractWatchService.take(Unknown Source)
org.openstreetmap.josm.io.FileWatcher.processEvents(FileWatcher.java:114)
org.openstreetmap.josm.io.FileWatcher$$Lambda$4/6800315.run(Unknown Source)
java.lang.Thread.run(Unknown Source)

Thread: Attach Listener (5) of system

Thread: Timer-0 (38) of main
java.lang.Object.wait(Native Method)
java.util.TimerThread.mainLoop(Unknown Source)
java.util.TimerThread.run(Unknown Source)

Thread: D3D Screen Updater (20) of system
java.lang.Object.wait(Native Method)
sun.java2d.d3d.D3DScreenUpdateManager.run(Unknown Source)
java.lang.Thread.run(Unknown Source)

Thread: Finalizer (3) of system
java.lang.Object.wait(Native Method)
java.lang.ref.ReferenceQueue.remove(Unknown Source)
java.lang.ref.ReferenceQueue.remove(Unknown Source)
java.lang.ref.Finalizer$FinalizerThread.run(Unknown Source)

Thread: RemoteControl HTTP Server (35) of main
java.net.DualStackPlainSocketImpl.accept0(Native Method)
java.net.DualStackPlainSocketImpl.socketAccept(Unknown Source)
java.net.AbstractPlainSocketImpl.accept(Unknown Source)
java.net.PlainSocketImpl.accept(Unknown Source)
java.net.ServerSocket.implAccept(Unknown Source)
java.net.ServerSocket.accept(Unknown Source)
org.openstreetmap.josm.io.remotecontrol.RemoteControlHttpServer.run(RemoteControlHttpServer.java:103)

Thread: DestroyJavaVM (42) of main

Thread: Java2D Disposer (11) of system
java.lang.Object.wait(Native Method)
java.lang.ref.ReferenceQueue.remove(Unknown Source)
java.lang.ref.ReferenceQueue.remove(Unknown Source)
sun.java2d.Disposer.run(Unknown Source)
java.lang.Thread.run(Unknown Source)

Thread: AWT-Windows (14) of system
sun.awt.windows.WToolkit.eventLoop(Native Method)
sun.awt.windows.WToolkit.run(Unknown Source)
java.lang.Thread.run(Unknown Source)

Thread: Reference Handler (2) of system
java.lang.Object.wait(Native Method)
java.lang.Object.wait(Unknown Source)
java.lang.ref.Reference.tryHandlePending(Unknown Source)
java.lang.ref.Reference$ReferenceHandler.run(Unknown Source)

Thread: Timer-1 (46) of main
java.lang.Object.wait(Native Method)
java.lang.Object.wait(Unknown Source)
java.util.TimerThread.mainLoop(Unknown Source)
java.util.TimerThread.run(Unknown Source)

Thread: Thread-1 (9) of main
sun.nio.fs.WindowsNativeDispatcher.GetQueuedCompletionStatus0(Native Method)
sun.nio.fs.WindowsNativeDispatcher.GetQueuedCompletionStatus(Unknown Source)
sun.nio.fs.WindowsWatchService$Poller.run(Unknown Source)
java.lang.Thread.run(Unknown Source)

Thread: message-notifier-0 (41) of main
sun.misc.Unsafe.park(Native Method)
java.util.concurrent.locks.LockSupport.parkNanos(Unknown Source)
java.util.concurrent.locks.AbstractQueuedSynchronizer$ConditionObject.awaitNanos(Unknown Source)
java.util.concurrent.ScheduledThreadPoolExecutor$DelayedWorkQueue.take(Unknown Source)
java.util.concurrent.ScheduledThreadPoolExecutor$DelayedWorkQueue.take(Unknown Source)
java.util.concurrent.ThreadPoolExecutor.getTask(Unknown Source)
java.util.concurrent.ThreadPoolExecutor.runWorker(Unknown Source)
java.util.concurrent.ThreadPoolExecutor$Worker.run(Unknown Source)
java.lang.Thread.run(Unknown Source)

Thread: RemoteControl HTTP Server (36) of main
java.net.DualStackPlainSocketImpl.accept0(Native Method)
java.net.DualStackPlainSocketImpl.socketAccept(Unknown Source)
java.net.AbstractPlainSocketImpl.accept(Unknown Source)
java.net.PlainSocketImpl.accept(Unknown Source)
java.net.ServerSocket.implAccept(Unknown Source)
java.net.ServerSocket.accept(Unknown Source)
org.openstreetmap.josm.io.remotecontrol.RemoteControlHttpServer.run(RemoteControlHttpServer.java:103)

Thread: main-worker-0 (44) of main
sun.misc.Unsafe.park(Native Method)
java.util.concurrent.locks.LockSupport.park(Unknown Source)
java.util.concurrent.locks.AbstractQueuedSynchronizer$ConditionObject.await(Unknown Source)
java.util.concurrent.LinkedBlockingQueue.take(Unknown Source)
java.util.concurrent.ThreadPoolExecutor.getTask(Unknown Source)
java.util.concurrent.ThreadPoolExecutor.runWorker(Unknown Source)
java.util.concurrent.ThreadPoolExecutor$Worker.run(Unknown Source)
java.lang.Thread.run(Unknown Source)

Thread: Map Status Collector (49) of main
sun.misc.Unsafe.park(Native Method)
java.util.concurrent.locks.LockSupport.park(Unknown Source)
java.util.concurrent.locks.AbstractQueuedSynchronizer$ConditionObject.await(Unknown Source)
java.util.concurrent.LinkedBlockingQueue.take(Unknown Source)
org.openstreetmap.josm.gui.MapStatus$Collector.run(MapStatus.java:406)
java.lang.Thread.run(Unknown Source)

Thread: TimerQueue (18) of system
sun.misc.Unsafe.park(Native Method)
java.util.concurrent.locks.LockSupport.parkNanos(Unknown Source)
java.util.concurrent.locks.AbstractQueuedSynchronizer$ConditionObject.awaitNanos(Unknown Source)
java.util.concurrent.DelayQueue.take(Unknown Source)
javax.swing.TimerQueue.run(Unknown Source)
java.lang.Thread.run(Unknown Source)

Thread: image-fetcher-0 (27) of main
sun.misc.Unsafe.park(Native Method)
java.util.concurrent.locks.LockSupport.park(Unknown Source)
java.util.concurrent.locks.AbstractQueuedSynchronizer$ConditionObject.await(Unknown Source)
java.util.concurrent.LinkedBlockingQueue.take(Unknown Source)
java.util.concurrent.ThreadPoolExecutor.getTask(Unknown Source)
java.util.concurrent.ThreadPoolExecutor.runWorker(Unknown Source)
java.util.concurrent.ThreadPoolExecutor$Worker.run(Unknown Source)
java.lang.Thread.run(Unknown Source)

Thread: styled-map-renderer-240 (563)
sun.misc.Unsafe.park(Native Method)
java.util.concurrent.ForkJoinPool.awaitWork(Unknown Source)
java.util.concurrent.ForkJoinPool.runWorker(Unknown Source)
java.util.concurrent.ForkJoinWorkerThread.run(Unknown Source)

Thread: JCS-ElementEventQueue-Thread-7 (32) of main
sun.misc.Unsafe.park(Native Method)
java.util.concurrent.locks.LockSupport.park(Unknown Source)
java.util.concurrent.locks.AbstractQueuedSynchronizer$ConditionObject.await(Unknown Source)
java.util.concurrent.LinkedBlockingQueue.take(Unknown Source)
java.util.concurrent.ThreadPoolExecutor.getTask(Unknown Source)
java.util.concurrent.ThreadPoolExecutor.runWorker(Unknown Source)
java.util.concurrent.ThreadPoolExecutor$Worker.run(Unknown Source)
java.lang.Thread.run(Unknown Source)

Thread: AWT-Shutdown (13) of system
java.lang.Object.wait(Native Method)
java.lang.Object.wait(Unknown Source)
sun.awt.AWTAutoShutdown.run(Unknown Source)
java.lang.Thread.run(Unknown Source)

Thread: AWT-EventQueue-0 (16) of main
Stacktrace see above.

Thread: Weak reference cleaner (21) of main
java.lang.Object.wait(Native Method)
java.lang.ref.ReferenceQueue.remove(Unknown Source)
java.lang.ref.ReferenceQueue.remove(Unknown Source)
org.openstreetmap.josm.tools.ListenableWeakReference.clean(ListenableWeakReference.java:61)
org.openstreetmap.josm.tools.ListenableWeakReference$$Lambda$76/25607935.run(Unknown Source)
java.lang.Thread.run(Unknown Source)

Thread: Signal Dispatcher (4) of system

Thread: RemoteControl HTTPS Server (37) of main
java.net.DualStackPlainSocketImpl.accept0(Native Method)
java.net.DualStackPlainSocketImpl.socketAccept(Unknown Source)
java.net.AbstractPlainSocketImpl.accept(Unknown Source)
java.net.PlainSocketImpl.accept(Unknown Source)
java.net.ServerSocket.implAccept(Unknown Source)
sun.security.ssl.SSLServerSocketImpl.accept(Unknown Source)
org.openstreetmap.josm.io.remotecontrol.RemoteControlHttpsServer.run(RemoteControlHttpsServer.java:407)

Attachments (1)

ticket15008.osm​ (645.9 KB) - added by rickmastfan67 5 years ago.

Download all attachments as: .zip

Change History (34)

comment:1 Changed 6 years ago by geri-oc

Es wurde nach dem (versehentlichen) Trennen aller nodes eines Weges (statt nur eines node)und dem "Rückgängig" gemeldet.

comment:2 Changed 6 years ago by Don-vip

Cc: bastiK stoecker Klumbumbus added
Owner: changed from team to geri-oc
Status: new → needinfo

can anyone please translate this bug report? :)

comment:3 Changed 6 years ago by Klumbumbus

Summary: Fehlermeldung nach Bearbeiten -> zurück → Error after Edit -> Undo

comment:4 Changed 6 years ago by Klumbumbus

Description: modified (diff)

comment:5 Changed 6 years ago by Klumbumbus

geri-oc: meintest du Linien trennen (shortcut G)?

comment:6 Changed 6 years ago by Don-vip

Component: Core → Plugin turnrestrictions

comment:7 Changed 6 years ago by anonymous

Linie aus mehreren Linien auswählen (mittleres Mausrad)
mit STRG+Klick zwei Punkte zum trennen auswählen
jetzt normalerweise mit p trennnen - statt dessen STRG weiter gedrückt -> also STRG+P (alle Punkte)

Bedienungs-Fehler gemerkt

Letzte Aktion zurücknehmen (Rückgängig)geklickt:

"Fehlermeldung"

English Google-Translator:
Select line from multiple lines (middle mouse wheel)
With CTRL + click two points to separate
Now normally separated by p - instead of pressing CTRL -> CTRL + P (all points)

Operation error noted

Undo last action (Undo):

"Error message"

comment:8 Changed 6 years ago by geri-oc

vorigen Kommentar leider ohne Anmeldung gepostet:
Ist von mir: geri-oc

Previous posting unfortunately unregistered posted:
Is from me: geri-oc

comment:9 Changed 6 years ago by Klumbumbus

Strg+P ist standardmäßig nicht belegt (außer für Drucken reserviert). Meintest du vlt. Strg+Shift+P (Bereinigen) oder Strg+Alt+Shift+P (Benachbarte Linien trennen)?

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

comment:10 Changed 6 years ago by geri-oc

Ich konnte das Beispiel von heute früh eben nicht reprodizieren. Aber wahrscheinlich ist es Umschalt gewesen:

gerade mehrmals probiert
way auswählen - way markiert
Umschalt+P zwei nodes klicken - way und nodes markiert
G - trennen - Fehler bemerkt
auf Rückgängig klicken(passiert auch bei STRG+Z) - Fehlermeldung

I could not reproach the example from today. But probably it was change:
Just tried several times
Way - way
Shift + P two nodes click - way and tag marked
G - disconnect errors
To Undo (also happens with CTRL + Z) - Error message

comment:11 Changed 6 years ago by Klumbumbus

I can reproduce:

  1. display the toggle dialog from the turnrestrictions plugin or the reltoolbox plugin (maybe other plugins are affected too?)
  2. select two ways or two nodes
  3. purge
  4. confirm the purge dialog
  5. org.openstreetmap.josm.data.osm.DataIntegrityProblemException: Primitive must be part of the dataset...

comment:12 Changed 6 years ago by Klumbumbus

another way to reproduce:

  1. display the toggle dialog from the turnrestrictions plugin or the reltoolbox plugin
  2. select a way which has waynodes which belong to another way too (e.g. ​https://www.openstreetmap.org/way/108574602)
  3. press G (un-glue)
  4. press ctrl+Z (undo)

comment:13 Changed 6 years ago by Don-vip

Description: modified (diff)
Summary: Error after Edit -> Undo → DataIntegrityProblemException: Primitive must be part of the dataset

comment:14 Changed 6 years ago by Don-vip

Ticket #15018 has been marked as a duplicate of this ticket.

comment:15 Changed 6 years ago by Don-vip

Ticket #15129 has been marked as a duplicate of this ticket.

comment:16 Changed 6 years ago by Don-vip

Ticket #15137 has been marked as a duplicate of this ticket.

comment:17 Changed 6 years ago by Don-vip

Cc: michael2402 added
Owner: changed from geri-oc to team
Priority: normal → major
Status: needinfo → new

comment:18 Changed 6 years ago by Klumbumbus

Ticket #15140 has been marked as a duplicate of this ticket.

comment:19 Changed 6 years ago by rickmastfan67

This might have to be bumped up to critical if not blocker. Why you ask? Well, I just encountered it while doing some of my first OSM work in awhile, and I got a never ending crash with one of my files that I was working on. The crash message NEVER showed up in JOSM, I only noticed it in the Command Prompt window. It was so bad that the text was 'flickering' because so many error messages were getting triggered!! The only way I could pin it to this bug, was because thankfully, the PrintScreen button is faster and caught the error report for me.

So, I went looking for a quick, and easy way to duplicate this on command, and I've found it. And it doesn't require any data to be edited what so ever. And I was able to do this on a completely fresh profile.

Steps to reproduce:

  1. Start with a fresh JOSM profile.
  2. Download and install the turnrestrictions & utilsplugin2 plugins.
  3. Restart JOSM so they are activated.
  4. Download the following area: ​https://www.openstreetmap.org/#map=20/42.548799462597714/-80.0439110494081
  5. Activate the 'Turn Restrictions' window for the right side of the screen.
  6. Ctrl+A to select all of the data that was download.
  7. Ctrl+Shift+P to activate the 'Purge' command from utilsplugin2.
  8. Hit the 'OK' button to purge the data.

What happens:
JOSM freaks out big time. But will eventually calm down and give an error report (unlike with my file that I was working with).

URL:http://josm.openstreetmap.de/svn/trunk
Repository:UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
Last:Changed Date: 2017-08-13 19:54:54 +0200 (Sun, 13 Aug 2017)
Build-Date:2017-08-14 01:35:37
Revision:12598
Relative:URL: ^/trunk

Identification: JOSM/1.5 (12598 en) Windows 7 64-Bit
OS Build number: Windows 7 Professional (7601)
Memory Usage: 845 MB / 1820 MB (195 MB allocated, but free)
Java version: 1.8.0_131-b11, Oracle Corporation, Java HotSpot(TM) 64-Bit Server VM
Screen: \Display0 1920x1080
Maximum Screen Size: 1920x1080
Dataset consistency test: No problems found

Plugins:
+ turnrestrictions (33088)
+ utilsplugin2 (33328)

Last errors/warnings:
- W: Too many errors. Dropping ReportedException [thread=Thread[AWT-EventQueue-0,6,main], exception=org.openstreetmap.josm.data.osm.DataIntegrityProblemException: Primitive must be part of the dataset: {Node id=808616109 version=1 V lat=42.5543671,lon=-80.2456531}, methodWarningFrom=BugReportExceptionHandler#handleException]
- E: Handled by bug report queue: org.openstreetmap.josm.data.osm.DataIntegrityProblemException: Primitive must be part of the dataset: {Node id=808616109 version=1 V lat=42.5543671,lon=-80.2456531}
- W: Too many errors. Dropping ReportedException [thread=Thread[AWT-EventQueue-0,6,main], exception=org.openstreetmap.josm.data.osm.DataIntegrityProblemException: Primitive must be part of the dataset: {Node id=808616109 version=1 V lat=42.5543671,lon=-80.2456531}, methodWarningFrom=BugReportExceptionHandler#handleException]
- E: Handled by bug report queue: org.openstreetmap.josm.data.osm.DataIntegrityProblemException: Primitive must be part of the dataset: {Node id=808616109 version=1 V lat=42.5543671,lon=-80.2456531}
- W: Too many errors. Dropping ReportedException [thread=Thread[AWT-EventQueue-0,6,main], exception=org.openstreetmap.josm.data.osm.DataIntegrityProblemException: Primitive must be part of the dataset: {Node id=808616109 version=1 V lat=42.5543671,lon=-80.2456531}, methodWarningFrom=BugReportExceptionHandler#handleException]
- E: Handled by bug report queue: org.openstreetmap.josm.data.osm.DataIntegrityProblemException: Primitive must be part of the dataset: {Node id=808661965 version=1 V lat=42.5545802,lon=-80.247079}
- E: Handled by bug report queue: org.openstreetmap.josm.data.osm.DataIntegrityProblemException: Primitive must be part of the dataset: {Node id=808603402 version=1 V lat=42.5546621,lon=-80.248474}
- E: Handled by bug report queue: org.openstreetmap.josm.data.osm.DataIntegrityProblemException: Primitive must be part of the dataset: {Node id=808603402 version=1 V lat=42.5546621,lon=-80.248474}
- E: Handled by bug report queue: org.openstreetmap.josm.data.osm.DataIntegrityProblemException: Primitive must be part of the dataset: {Node id=808617092 version=2 V lat=42.5548728,lon=-80.250063}
- E: Handled by bug report queue: org.openstreetmap.josm.data.osm.DataIntegrityProblemException: Primitive must be part of the dataset: {Node id=808617092 version=2 V lat=42.5548728,lon=-80.250063}


=== REPORTED CRASH DATA ===
BugReportExceptionHandler#handleException:
No data collected.

Warning issued by: BugReportExceptionHandler#handleException

=== STACK TRACE ===
Thread: AWT-EventQueue-0 (19) of main
org.openstreetmap.josm.data.osm.DataIntegrityProblemException: Primitive must be part of the dataset: {Node id=1402755071 version=1 V lat=42.5481376,lon=-80.067343}
	at org.openstreetmap.josm.data.osm.OsmPrimitive.checkDataset(OsmPrimitive.java:252)
	at org.openstreetmap.josm.data.osm.OsmPrimitive.getReferrers(OsmPrimitive.java:996)
	at org.openstreetmap.josm.data.osm.OsmPrimitive.getReferrers(OsmPrimitive.java:1021)
	at org.openstreetmap.josm.plugins.turnrestrictions.list.TurnRestrictionsInSelectionListModel.initFromSelection(TurnRestrictionsInSelectionListModel.java:40)
	at org.openstreetmap.josm.plugins.turnrestrictions.list.TurnRestrictionsInSelectionListModel.selectionChanged(TurnRestrictionsInSelectionListModel.java:66)
	at org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo.fire(SelectionEventManager.java:57)
	at org.openstreetmap.josm.data.osm.event.SelectionEventManager.fireEvent(SelectionEventManager.java:203)
	at org.openstreetmap.josm.data.osm.event.SelectionEventManager.lambda$selectionChanged$0(SelectionEventManager.java:198)
	at java.awt.event.InvocationEvent.dispatch(Unknown Source)
	at java.awt.EventQueue.dispatchEventImpl(Unknown Source)
	at java.awt.EventQueue.access$500(Unknown Source)
	at java.awt.EventQueue$3.run(Unknown Source)
	at java.awt.EventQueue$3.run(Unknown Source)
	at java.security.AccessController.doPrivileged(Native Method)
	at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(Unknown Source)
	at java.awt.EventQueue.dispatchEvent(Unknown Source)
	at java.awt.EventDispatchThread.pumpOneEventForFilters(Unknown Source)
	at java.awt.EventDispatchThread.pumpEventsForFilter(Unknown Source)
	at java.awt.EventDispatchThread.pumpEventsForHierarchy(Unknown Source)
	at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
	at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
	at java.awt.EventDispatchThread.run(Unknown Source)

I also had to go thru about 50 or so error message windows after JOSM stopped freaking out. That alone is a major problem if you triggered this error another way and you wanted to save the file that you were working on.

Last edited 6 years ago by rickmastfan67 (previous) (diff)

comment:20 Changed 6 years ago by rickmastfan67

Ok, take the utilsplugin2 out of the equation. Didn't know that the 'Purge' command had been moved into the 'Core'.

comment:21 Changed 6 years ago by Don-vip

Priority: major → critical

comment:22 Changed 6 years ago by rickmastfan67

Ok, this is nuts, but I can no longer trigger this error on command anymore with r12606. So, something between r12598 & r12606 has caused my example above not to be triggered anymore.

I'm guessing the changes in [12605] for the purge performance issues, happened to eliminate the way I could trigger this bug.

I couldn't even trigger it anymore on the file that I was having major problems with last night in JOSM.

comment:23 Changed 6 years ago by Klumbumbus

with r10606 I can't reproduce with comment:11 anymore (which seems to be the same way like comment:19) but I still can reproduce with comment:12

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

comment:24 Changed 6 years ago by Don-vip

In 12970/josm:

see #15008, see #15421 - add toString() methods to allow to report potentially incorrect selection events

comment:25 Changed 6 years ago by Don-vip

In 12973/josm:

see #15008, see #15421 - collect debug data when a DataIntegrityProblemException occurs when firing a selection event

comment:26 Changed 6 years ago by Don-vip

If someone manages to reproduce it with r12973+, can you please post the new status report? It will contain precious information to help to fix the underlying bug (the crash in turnrestrictions plugin is only a symptom, not the root cause)

comment:27 Changed 5 years ago by rickmastfan67

Just got it to crash randomly tonight. Attached is a copy of the area where I got it to happen. Full steps below after you open the file (was working in the area).

  1. Find way '15651880' (labeled "County Route 26")
  2. Hit 'g' to unglue the way.
  3. Hit 'Crtl+Z' to undo.

What happens:
JOSM crashes saying error might have come from the turnrestrictions plugin.

What should happen:
The undo should have worked without any problems.

URL:http://josm.openstreetmap.de/svn/trunk
Repository:UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
Last:Changed Date: 2017-12-12 01:11:39 +0100 (Tue, 12 Dec 2017)
Build-Date:2017-12-12 02:41:40
Revision:13199
Relative:URL: ^/trunk

Identification: JOSM/1.5 (13199 en) Windows 7 64-Bit
OS Build number: Windows 7 Professional (7601)
Memory Usage: 721 MB / 1820 MB (337 MB allocated, but free)
Java version: 1.8.0_144-b01, Oracle Corporation, Java HotSpot(TM) 64-Bit Server VM
Screen: \Display0 1920x1080
Maximum Screen Size: 1920x1080
Dataset consistency test: No problems found

Plugins:
+ OpeningHoursEditor (33876)
+ buildings_tools (33735)
+ measurement (33760)
+ reverter (33865)
+ tageditor (33806)
+ turnlanes-tagging (260)
+ turnrestrictions (33780)
+ undelete (33835)
+ utilsplugin2 (33742)

Map paint styles:
- https://josm.openstreetmap.de/josmfile?page=Styles/Maxspeed&zip=1
- https://josm.openstreetmap.de/josmfile?page=Styles/Lane_and_Road_Attributes&zip=1

Last errors/warnings:
- W: No configuration settings found.  Using hardcoded default values for all pools.
- E: Handled by bug report queue: org.openstreetmap.josm.data.osm.DataIntegrityProblemException: Primitive must be part of the dataset: {Node id=-42073 version=0 MV lat=37.673044,lon=-80.8972585}


=== REPORTED CRASH DATA ===
SelectionEventManager#fireEvent:
 - event: SelectionRemoveEvent [remove=[{Node id=-42074 version=0 MV lat=37.6732015,lon=-80.8963913}], current=[{Node id=-42073 version=0 MV lat=37.673044,lon=-80.8972585}]]
 - listeners: 
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@7b92e280
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@350342b2
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@3a109d21
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@abae961
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@66d783bd
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@49ef522c
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@77a93ede
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@2a843a10
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@7015946b
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@43d9cf98
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@7dbb5d9a
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@767ba8ab
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@2a90f61a
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@27bf45f6
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@4894fd
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@2b5c73b2
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@64e744d
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@5b0208ec
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@1f288936
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@57a0b1a9
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@22ba4073
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@69b560cd
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@320e3cdd
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@60f0e7b3
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@10b8b35d
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@52ca01b6
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@11e22cdb
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@76c9b572
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@23bdda16
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@2179a2cd
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@217eee10
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@1a26b7bf
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@5799b791
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@3b19041c
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@6d5e6029
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@426a51d5
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@58d25ccd
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@2ba7aac1
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@35e7d20b
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@2510c126
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@6f6dce46
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@79099bf1
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@72ea4dcd
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@51d1b22c
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@504c4305
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@65ae9455
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@4589222f
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@1994a195
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@2bfd9bef
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@3ae3a5dc
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@36f68972
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@a550209
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@4d31388a
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@6a58b25c
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@68660253
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@4a9e04a2
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@ea88ffa
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@214e1d88
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@7961ccf5
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@43e16584
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@97decb4
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@58891da5
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@2ff96d5c
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@677bf009
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@184bef3
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@11f1e66e
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@7585e1e8
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@4d39544b
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@69ef2160
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@773271cc
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@2de35d4a
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@2a7cf4ae
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@3a893dc7
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@5ed29c6c
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@69ef88d8
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@24fac1a6
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@57eddcec
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@1d9712a8
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@8a8fbe1
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@7aace5c8
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@439e55a4
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@66380444
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@298185ea
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@341dad1e
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@59ee5a3
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@23a1c05d
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@62ba67b
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@5f889851
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@2d48e4a1
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@457144
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@5b873e56
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@40d5749c
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@2ad3e430
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@1b47456
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@63a094cf
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@83d6244
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@3d7bfc7e
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@1f4c322a
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@78ae3065
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@1b61595b
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@799ad72a
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@7b87e78
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@e9d5307
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@1fa442dd
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@30a9fced
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@74271653
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@10c90069
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@2dc5117e
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@5275d7c4
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@1eafed88
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@46bc65c5
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@43f6cbe6
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@3f42f502
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@5e6792e6
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@7b534e17
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@e0e19c5
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@26c30187
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@44d04efc
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@250df751
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@5b30093b
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@65468b72
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@2326979f
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@695840d0
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@32fbcc8e
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@6336c58f
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@74bbd420
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@5fcd600e
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@33b968ec
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@66f136ae
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@4cc0fdab
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@7947e1fb
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@1400158
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@6eea0aef
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@258da63b
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@6302fa8f
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@624a76ff
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@5c484d12
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@143dac3f
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@609805cc
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@51548247
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@4fe57344
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@36409a74
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@1655447c
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@63e5d19c
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@77dfa9e
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@35dd4b16
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@99687dc
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@44b5c1ea
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@2c73753f
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@73a618ca
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@4e30c22c
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@24e1f9b8
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@130c426f
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@50ac73f9
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@18c4f387
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@3342ecf7
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@42ad2641
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@71acbf2d
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@4ecc5dff
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@6a4c5cec
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@41fbebaf
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@55bb688d
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@79b40c83
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@71968dca
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@6f2f4f81
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@61d9a310
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@385755a1
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@7d697d15
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@68286885
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@444a91d7
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@46332815
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@700b7d29
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@5bcb4054
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$DataListenerInfo@4b1ccdc4
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@144f4fc6
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@3f879328
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@3ecca643
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@139035dc
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@7d4c723e
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@7ed0a983
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@2576dd3b
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@37d939f1
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@299b79db
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@52fb6e81
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@41f6d73b
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@3f9f6178
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@4d3ba9ae
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@44ba4653
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@330327a7
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@77521d37
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@6735a5cb
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@71a8d4b7
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@5bbd9261
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@3c60ef0f
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@76322434
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@4189227e
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@769187cd
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@4d3df101
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@2a4877fb
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@22db232f
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@588d6c31
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@7a153d20
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@130a5327
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@68b221f9
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@5b6c2d41
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@266ff791
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@5f696550
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@842fe9a
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@49117e87
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@30d42a80
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@a6b0483
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@15c59293
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@57654cb9
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@46f3e354
    - org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo@14e987ca

BugReportExceptionHandler#handleException:
No data collected.

Warning issued by: BugReportExceptionHandler#handleException

=== STACK TRACE ===
Thread: AWT-EventQueue-0 (20) of main
org.openstreetmap.josm.data.osm.DataIntegrityProblemException: Primitive must be part of the dataset: {Node id=-42073 version=0 MV lat=37.673044,lon=-80.8972585}
	at org.openstreetmap.josm.data.osm.OsmPrimitive.checkDataset(OsmPrimitive.java:253)
	at org.openstreetmap.josm.data.osm.OsmPrimitive.getReferrers(OsmPrimitive.java:997)
	at org.openstreetmap.josm.data.osm.OsmPrimitive.getReferrers(OsmPrimitive.java:1022)
	at org.openstreetmap.josm.plugins.turnrestrictions.list.TurnRestrictionsInSelectionListModel.initFromSelection(TurnRestrictionsInSelectionListModel.java:40)
	at org.openstreetmap.josm.plugins.turnrestrictions.list.TurnRestrictionsInSelectionListModel.selectionChanged(TurnRestrictionsInSelectionListModel.java:66)
	at org.openstreetmap.josm.data.osm.event.SelectionEventManager$OldListenerInfo.fire(SelectionEventManager.java:59)
	at org.openstreetmap.josm.data.osm.event.SelectionEventManager.fireEvent(SelectionEventManager.java:204)
	at org.openstreetmap.josm.data.osm.event.SelectionEventManager.lambda$selectionChanged$0(SelectionEventManager.java:198)
	at java.awt.event.InvocationEvent.dispatch(Unknown Source)
	at java.awt.EventQueue.dispatchEventImpl(Unknown Source)
	at java.awt.EventQueue.access$500(Unknown Source)
	at java.awt.EventQueue$3.run(Unknown Source)
	at java.awt.EventQueue$3.run(Unknown Source)
	at java.security.AccessController.doPrivileged(Native Method)
	at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(Unknown Source)
	at java.awt.EventQueue.dispatchEvent(Unknown Source)
	at java.awt.EventDispatchThread.pumpOneEventForFilters(Unknown Source)
	at java.awt.EventDispatchThread.pumpEventsForFilter(Unknown Source)
	at java.awt.EventDispatchThread.pumpEventsForHierarchy(Unknown Source)
	at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
	at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
	at java.awt.EventDispatchThread.run(Unknown Source)

Changed 5 years ago by rickmastfan67

Attachment: ticket15008.osm​ added

comment:28 Changed 5 years ago by Don-vip

OK reproduced (the turn restrictions dialog has to be displayed).
When undoing the glue operation we get:

  • AddCommand.undoCommand
  • DataSet.removePrimitive
  • DataSet.removePrimitiveImpl
  • DataSet.clearSelection
  • DataSet.doSelectionChange(new SelectionRemoveEvent)
  • DataSet.selectionListeners.fireEvent
  • SelectionEventManager.selectionChanged <- At this stage all primitives still refer the dataset
  • SwingUtilities.invokeLater is called
  • (EDT) SelectionEventManager.fireEvent <- At this stage all primitives have lost their parent dataset

We should probably replace the call invokeLater by a call to invokeAndWait to make sure EDT listeners receive a correct event.

comment:29 Changed 5 years ago by Don-vip

Component: Plugin turnrestrictions → Core
Keywords: selection event edt added
Milestone: → 17.12

comment:30 Changed 5 years ago by Don-vip

Resolution: → fixed
Status: new → closed

In 13223/josm:

fix #15008 - make sure EDT listeners receive correct selection events

comment:31 Changed 5 years ago by Don-vip

Ticket #15421 has been marked as a duplicate of this ticket.

comment:32 Changed 5 years ago by Don-vip

Ticket #15021 has been marked as a duplicate of this ticket.

comment:33 Changed 5 years ago by Don-vip

In 13225/josm:

see #15008 - avoid deadlock in selection event processing

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain team.
as The resolution will be set.
The resolution will be deleted.

Add Comment

E-mail address and name can be saved in the Preferences.

Attachments ↑ Description ↑
 
Note: See TracTickets for help on using tickets.