Changeset 2920 in josm for trunk/src/org/openstreetmap

Timestamp:

2010-02-01T21:56:31+01:00 (14 years ago)

Author:

jttt

Message:

Fixed NPE from log in #4409

File:

• trunk/src/org/openstreetmap/josm/data/osm/event/DatasetEventManager.java (modified) (1 diff)

trunk/src/org/openstreetmap/josm/data/osm/event/DatasetEventManager.java

@@ -142 +142 @@
                     if (consolidatedEvent != null && dataSet != event.getDataset()) {
                         fireConsolidatedEvents(inEDTListeners, consolidatedEvent);
-                        dataSet = event.getDataset();
                         consolidatedEvent = null;
                     }
 
+                    dataSet = event.getDataset();
+
                     // Build consolidated event
-                    if (consolidatedEvent == null) {
-                        consolidatedEvent = event;
-                        dataSet = event.getDataset();
-                    } else if (consolidatedEvent instanceof DataChangedEvent) {
-                        ((DataChangedEvent)consolidatedEvent).getEvents().add(event);
+                    if (event instanceof DataChangedEvent) {
+                        // DataChangeEvent can contains other events, so it gets special handling
+                        DataChangedEvent dataEvent = (DataChangedEvent) event;
+                        if (dataEvent.getEvents() == null) {
+                            consolidatedEvent = dataEvent; // Dataset was completely changed, we can ignore older events
+                        } else {
+                            if (consolidatedEvent == null) {
+                                consolidatedEvent = new DataChangedEvent(dataSet, dataEvent.getEvents());
+                            } else if (consolidatedEvent instanceof DataChangedEvent) {
+                                List<AbstractDatasetChangedEvent> evts = ((DataChangedEvent) consolidatedEvent).getEvents();
+                                if (evts != null) {
+                                    evts.addAll(dataEvent.getEvents());
+                                }
+                            } else {
+                                AbstractDatasetChangedEvent oldConsolidateEvent = consolidatedEvent;
+                                consolidatedEvent = new DataChangedEvent(dataSet, dataEvent.getEvents());
+                                ((DataChangedEvent) consolidatedEvent).getEvents().add(oldConsolidateEvent);
+                            }
+                        }
                     } else {
-                        consolidatedEvent = new DataChangedEvent(dataSet,
-                                new ArrayList<AbstractDatasetChangedEvent>(Arrays.asList(consolidatedEvent)));
+                        // Normal events
+                        if (consolidatedEvent == null) {
+                            consolidatedEvent = event;
+                        } else if (consolidatedEvent instanceof DataChangedEvent) {
+                            List<AbstractDatasetChangedEvent> evs = ((DataChangedEvent) consolidatedEvent).getEvents();
+                            if (evs != null) {
+                                evs.add(event);
+                            }
+                        } else {
+                            consolidatedEvent = new DataChangedEvent(dataSet,
+                                    new ArrayList<AbstractDatasetChangedEvent>(Arrays.asList(consolidatedEvent)));
+                        }
+
                     }
                 }