Changeset 13497 in josm for trunk/src

Timestamp:

2018-03-04T15:18:05+01:00 (6 years ago)

Author:

Don-vip

Message:

fix #16051, see #8039, see #10456 - more fixes for download/upload policies and locked status (merge of layers)

Location:

trunk/src/org/openstreetmap/josm

Files:

• actions/UpdateModifiedAction.java (modified) (2 diffs)
• actions/UploadSelectionAction.java (modified) (3 diffs)
• data/osm/DataSet.java (modified) (1 diff)
• data/osm/DataSetMerger.java (modified) (3 diffs)
• gui/layer/OsmDataLayer.java (modified) (1 diff)

trunk/src/org/openstreetmap/josm/actions/UpdateModifiedAction.java

@@ -11 +11 @@
 import org.openstreetmap.josm.Main;
 import org.openstreetmap.josm.data.osm.DataSet;
+import org.openstreetmap.josm.data.osm.DataSet.DownloadPolicy;
 import org.openstreetmap.josm.data.osm.OsmPrimitive;
 import org.openstreetmap.josm.io.OnlineResource;
@@ -46 +47 @@
     @Override
     protected void updateEnabledState() {
-        setEnabled(getLayerManager().getEditDataSet() != null && !Main.isOffline(OnlineResource.OSM_API));
+        DataSet ds = getLayerManager().getEditDataSet();
+        setEnabled(ds != null && !DownloadPolicy.BLOCKED.equals(ds.getDownloadPolicy())
+                && !Main.isOffline(OnlineResource.OSM_API));
     }
 

trunk/src/org/openstreetmap/josm/actions/UploadSelectionAction.java

@@ -64 +64 @@
     protected void updateEnabledState(Collection<? extends OsmPrimitive> selection) {
         updateEnabledStateOnModifiableSelection(selection);
+        OsmDataLayer editLayer = getLayerManager().getEditLayer();
+        if (editLayer != null && !editLayer.isUploadable()) {
+            setEnabled(false);
+        }
     }
 
@@ -89 +93 @@
     public void actionPerformed(ActionEvent e) {
         OsmDataLayer editLayer = getLayerManager().getEditLayer();
-        if (!isEnabled())
+        if (!isEnabled() || !editLayer.isUploadable())
             return;
         if (editLayer.isUploadDiscouraged() && UploadAction.warnUploadDiscouraged(editLayer)) {
@@ -203 +207 @@
                 hull.add(w);
                 for (Node n: w.getNodes()) {
-                    // we upload modified nodes even if they aren't in the current
-                    // selection.
+                    // we upload modified nodes even if they aren't in the current selection.
                     n.accept(this);
                 }

trunk/src/org/openstreetmap/josm/data/osm/DataSet.java

@@ -329 +329 @@
             version = copyFrom.version;
             uploadPolicy = copyFrom.uploadPolicy;
+            downloadPolicy = copyFrom.downloadPolicy;
             isReadOnly.set(copyFrom.isReadOnly.get());
         } finally {

trunk/src/org/openstreetmap/josm/data/osm/DataSetMerger.java

@@ -4 +4 @@
 import static org.openstreetmap.josm.tools.I18n.tr;
 
+import java.awt.geom.Area;
 import java.util.ArrayList;
 import java.util.Collection;
@@ -14 +15 @@
 import java.util.Set;
 
+import org.openstreetmap.josm.data.DataSource;
 import org.openstreetmap.josm.data.conflict.Conflict;
 import org.openstreetmap.josm.data.conflict.ConflictCollection;
@@ -432 +434 @@
             candidates.clear();
             fixReferences();
+
+            Area a = targetDataSet.getDataSourceArea();
+
+            // copy the merged layer's data source info.
+            // only add source rectangles if they are not contained in the layer already.
+            for (DataSource src : sourceDataSet.getDataSources()) {
+                if (a == null || !a.contains(src.bounds.asRect())) {
+                    targetDataSet.addDataSource(src);
+                }
+            }
+
+            // copy the merged layer's API version
+            if (targetDataSet.getVersion() == null) {
+                targetDataSet.setVersion(sourceDataSet.getVersion());
+            }
+
+            // copy the merged layer's policies and locked status
+            if (sourceDataSet.getUploadPolicy() != null && (targetDataSet.getUploadPolicy() == null
+                    || sourceDataSet.getUploadPolicy().compareTo(targetDataSet.getUploadPolicy()) > 0)) {
+                targetDataSet.setUploadPolicy(sourceDataSet.getUploadPolicy());
+            }
+            if (sourceDataSet.getDownloadPolicy() != null && (targetDataSet.getDownloadPolicy() == null
+                    || sourceDataSet.getDownloadPolicy().compareTo(targetDataSet.getDownloadPolicy()) > 0)) {
+                targetDataSet.setDownloadPolicy(sourceDataSet.getDownloadPolicy());
+            }
+            if (sourceDataSet.isLocked() && !targetDataSet.isLocked()) {
+                targetDataSet.lock();
+            }
         } finally {
             targetDataSet.endUpdate();

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

@@ -552 +552 @@
         }
 
-        Area a = data.getDataSourceArea();
-
-        // copy the merged layer's data source info.
-        // only add source rectangles if they are not contained in the layer already.
-        for (DataSource src : from.getDataSources()) {
-            if (a == null || !a.contains(src.bounds.asRect())) {
-                data.addDataSource(src);
-            }
-        }
-
-        // copy the merged layer's API version
-        if (data.getVersion() == null) {
-            data.setVersion(from.getVersion());
-        }
-
         int numNewConflicts = 0;
         for (Conflict<?> c : visitor.getConflicts()) {