Changeset 13654 in josm for trunk/src

Timestamp:

2018-04-21T02:42:26+02:00 (7 years ago)

Author:

Don-vip

Message:

More uses of OsmDataLayer.getDataSet()

Location:

trunk/src/org/openstreetmap/josm/gui

Files:

• MainApplication.java (modified) (1 diff)
• datatransfer/importers/PrimitiveDataPaster.java (modified) (1 diff)
• dialogs/layer/DuplicateAction.java (modified) (1 diff)
• dialogs/layer/LayerListTransferHandler.java (modified) (1 diff)
• dialogs/relation/ChildRelationBrowser.java (modified) (3 diffs)
• dialogs/relation/ParentRelationLoadingTask.java (modified) (1 diff)
• dialogs/relation/actions/SavingAction.java (modified) (2 diffs)
• history/HistoryBrowserModel.java (modified) (2 diffs)
• io/AsynchronousUploadPrimitivesTask.java (modified) (1 diff)
• io/UploadLayerTask.java (modified) (1 diff)
• layer/AutosaveTask.java (modified) (2 diffs)
• layer/MainLayerManager.java (modified) (3 diffs)

trunk/src/org/openstreetmap/josm/gui/MainApplication.java

@@ -239 +239 @@
             Layer layer = e.getRemovedLayer();
             if (layer instanceof OsmDataLayer) {
-                undoRedo.clean(((OsmDataLayer) layer).data);
+                undoRedo.clean(((OsmDataLayer) layer).getDataSet());
             }
         }

trunk/src/org/openstreetmap/josm/gui/datatransfer/importers/PrimitiveDataPaster.java

@@ -83 +83 @@
             }
         }
-        return new AddPrimitivesCommand(bufferCopy, toSelect, layer.data);
+        return new AddPrimitivesCommand(bufferCopy, toSelect, layer.getDataSet());
     }
 

trunk/src/org/openstreetmap/josm/gui/dialogs/layer/DuplicateAction.java

@@ -55 +55 @@
             String newName = LayerListTransferHandler.suggestNewLayerName(layer.getName(), MainApplication.getLayerManager().getLayers());
             OsmDataLayer oldLayer = (OsmDataLayer) layer;
-            MainApplication.getLayerManager().addLayer(new OsmDataLayer(new DataSet(oldLayer.data), newName, null));
+            MainApplication.getLayerManager().addLayer(new OsmDataLayer(new DataSet(oldLayer.getDataSet()), newName, null));
         }
     }

trunk/src/org/openstreetmap/josm/gui/dialogs/layer/LayerListTransferHandler.java

@@ -138 +138 @@
             if (layer instanceof OsmDataLayer) {
                 String newName = suggestNewLayerName(layer.getName(), layerNames);
-                OsmDataLayer newLayer = new OsmDataLayer(new DataSet(((OsmDataLayer) layer).data), newName, null);
+                OsmDataLayer newLayer = new OsmDataLayer(new DataSet(((OsmDataLayer) layer).getDataSet()), newName, null);
                 layers.add(newLayer);
                 layerNames.add(newName);

trunk/src/org/openstreetmap/josm/gui/dialogs/relation/ChildRelationBrowser.java

@@ -399 +399 @@
 
         /**
-         * Merges the primitives in <code>ds</code> to the dataset of the
-         * edit layer
+         * Merges the primitives in <code>ds</code> to the dataset of the edit layer
          *
          * @param ds the data set
@@ -406 +405 @@
         protected void mergeDataSet(DataSet ds) {
             if (ds != null) {
-                final DataSetMerger visitor = new DataSetMerger(getLayer().data, ds);
+                final DataSetMerger visitor = new DataSetMerger(getLayer().getDataSet(), ds);
                 visitor.merge();
                 if (!visitor.getConflicts().isEmpty()) {
@@ -465 +464 @@
         protected void mergeDataSet(DataSet dataSet) {
             if (dataSet != null) {
-                final DataSetMerger visitor = new DataSetMerger(getLayer().data, dataSet);
+                final DataSetMerger visitor = new DataSetMerger(getLayer().getDataSet(), dataSet);
                 visitor.merge();
                 if (!visitor.getConflicts().isEmpty()) {

trunk/src/org/openstreetmap/josm/gui/dialogs/relation/ParentRelationLoadingTask.java

@@ -161 +161 @@
             referrers = reader.parseOsm(progressMonitor.createSubTaskMonitor(1, false));
             if (referrers != null) {
-                final DataSetMerger visitor = new DataSetMerger(getLayer().data, referrers);
+                final DataSetMerger visitor = new DataSetMerger(getLayer().getDataSet(), referrers);
                 visitor.merge();
 
                 // copy the merged layer's data source info
-                getLayer().data.addDataSources(referrers.getDataSources());
+                getLayer().getDataSet().addDataSources(referrers.getDataSources());
                 // FIXME: this is necessary because there are dialogs listening
                 // for DataChangeEvents which manipulate Swing components on this thread.

trunk/src/org/openstreetmap/josm/gui/dialogs/relation/actions/SavingAction.java

@@ -73 +73 @@
         if (newRelation.getMembersCount() == 0 && !newRelation.hasKeys())
             return;
-        MainApplication.undoRedo.add(new AddCommand(layer.data, newRelation));
+        MainApplication.undoRedo.add(new AddCommand(layer.getDataSet(), newRelation));
 
         // make sure everybody is notified about the changes
@@ -95 +95 @@
         memberTableModel.applyToRelation(editedRelation);
         Conflict<Relation> conflict = new Conflict<>(editor.getRelation(), editedRelation);
-        MainApplication.undoRedo.add(new ConflictAddCommand(layer.data, conflict));
+        MainApplication.undoRedo.add(new ConflictAddCommand(layer.getDataSet(), conflict));
     }
 

trunk/src/org/openstreetmap/josm/gui/history/HistoryBrowserModel.java

@@ -619 +619 @@
         if (oldLayer instanceof OsmDataLayer) {
             OsmDataLayer l = (OsmDataLayer) oldLayer;
-            l.data.removeDataSetListener(this);
+            l.getDataSet().removeDataSetListener(this);
         }
         Layer newLayer = e.getSource().getActiveLayer();
@@ -628 +628 @@
         }
         OsmDataLayer l = (OsmDataLayer) newLayer;
-        l.data.addDataSetListener(this);
+        l.getDataSet().addDataSetListener(this);
         OsmPrimitive primitive = history != null ? l.data.getPrimitiveById(history.getId(), history.getType()) : null;
         HistoryOsmPrimitive newLatest;

trunk/src/org/openstreetmap/josm/gui/io/AsynchronousUploadPrimitivesTask.java

@@ -123 +123 @@
         GuiHelper.runInEDTAndWait(() -> {
             // Remove the commands from the undo stack
-            MainApplication.undoRedo.clean(uploadDataLayer.data);
+            MainApplication.undoRedo.clean(uploadDataLayer.getDataSet());
             MainApplication.getLayerManager().prepareLayerForUpload(uploadDataLayer);
 

trunk/src/org/openstreetmap/josm/gui/io/UploadLayerTask.java

@@ -107 +107 @@
     public void run() {
         monitor.indeterminateSubTask(tr("Preparing objects to upload ..."));
-        APIDataSet ds = new APIDataSet(layer.data);
+        APIDataSet ds = new APIDataSet(layer.getDataSet());
         try {
             ds.adjustRelationUploadOrder();

trunk/src/org/openstreetmap/josm/gui/layer/AutosaveTask.java

@@ -298 +298 @@
     private void registerNewlayer(OsmDataLayer layer) {
         synchronized (layersLock) {
-            layer.data.addDataSetListener(datasetAdapter);
+            layer.getDataSet().addDataSetListener(datasetAdapter);
             layersInfo.add(new AutosaveLayerInfo<>(layer));
         }
@@ -324 +324 @@
             synchronized (layersLock) {
                 OsmDataLayer osmLayer = (OsmDataLayer) e.getRemovedLayer();
-                osmLayer.data.removeDataSetListener(datasetAdapter);
+                osmLayer.getDataSet().removeDataSetListener(datasetAdapter);
                 cleanupLayer(osmLayer);
             }

trunk/src/org/openstreetmap/josm/gui/layer/MainLayerManager.java

@@ -114 +114 @@
         public DataSet getPreviousDataSet() {
             if (previousDataLayer != null) {
-                return previousDataLayer.data;
+                return previousDataLayer.getDataSet();
             } else {
                 return null;
@@ -403 +403 @@
     public synchronized DataSet getEditDataSet() {
         if (dataLayer != null && !dataLayer.isLocked()) {
-            return dataLayer.data;
+            return dataLayer.getDataSet();
         } else {
             return null;
@@ -417 +417 @@
     public synchronized DataSet getActiveDataSet() {
         if (dataLayer != null) {
-            return dataLayer.data;
+            return dataLayer.getDataSet();
         } else {
             return null;