Changeset 33558 in osm for applications/editors/josm

Timestamp:

2017-08-27T19:12:12+02:00 (7 years ago)

Author:

donvip

Message:

fix warnings

Location:

applications/editors/josm/plugins/canvec_helper/src/org/openstreetmap/josm/plugins/canvec_helper

Files:

• CanVecTile.java (modified) (5 diffs)
• CanvecHelper.java (modified) (2 diffs)
• CanvecHelperAction.java (modified) (2 diffs)
• CanvecLayer.java (modified) (4 diffs)

applications/editors/josm/plugins/canvec_helper/src/org/openstreetmap/josm/plugins/canvec_helper/CanVecTile.java

@@ -16 +16 @@
 import java.util.zip.ZipFile;
 
-import org.openstreetmap.josm.Main;
 import org.openstreetmap.josm.data.Bounds;
 import org.openstreetmap.josm.data.coor.LatLon;
+import org.openstreetmap.josm.data.osm.DataSet.UploadPolicy;
+import org.openstreetmap.josm.gui.MainApplication;
 import org.openstreetmap.josm.gui.MapView;
 import org.openstreetmap.josm.gui.io.importexport.OsmImporter;
@@ -24 +25 @@
 import org.openstreetmap.josm.io.CachedFile;
 import org.openstreetmap.josm.io.IllegalDataException;
+import org.openstreetmap.josm.tools.Logging;
 
 class CanVecTile {
@@ -190 +192 @@
     private ZipFile openZip() throws IOException {
         File downloadPath = new File(layer.plugin.getPluginDir() + File.separator);
-        if (!downloadPath.mkdir() && Main.isDebugEnabled()) {
-            Main.debug("Unable to create directory: "+downloadPath);
+        if (!downloadPath.mkdir() && Logging.isDebugEnabled()) {
+            Logging.debug("Unable to create directory: "+downloadPath);
         }
         CachedFile tileZip = new CachedFile(getDownloadUrl()).setDestDir(downloadPath.toString());
@@ -203 +205 @@
             zipFile = openZip();
         } catch (IOException e) {
-            Main.error(e);
+            Logging.error(e);
             return;
         }
@@ -230 +232 @@
                     OsmImporter importer = new OsmImporter();
                     OsmImporterData temp = importer.loadLayer(rawtile, null, entry.getName(), null);
-                    Main.worker.submit(temp.getPostLayerTask());
-                    Main.getLayerManager().addLayer(temp.getLayer());
-                    temp.getLayer().data.setUploadDiscouraged(false);
+                    MainApplication.worker.submit(temp.getPostLayerTask());
+                    MainApplication.getLayerManager().addLayer(temp.getLayer());
+                    temp.getLayer().data.setUploadPolicy(UploadPolicy.NORMAL);
                 }
             }
         } catch (IOException | IllegalDataException e) {
-            Main.error(e);
+            Logging.error(e);
             return;
         }

applications/editors/josm/plugins/canvec_helper/src/org/openstreetmap/josm/plugins/canvec_helper/CanvecHelper.java

@@ -2 +2 @@
 package org.openstreetmap.josm.plugins.canvec_helper;
 
-import org.openstreetmap.josm.Main;
+import org.openstreetmap.josm.gui.MainApplication;
 import org.openstreetmap.josm.plugins.Plugin;
 import org.openstreetmap.josm.plugins.PluginInformation;
@@ -17 +17 @@
     public CanvecHelper(PluginInformation info) {
         super(info);
-        Main.main.menu.imagerySubMenu.add(new CanvecHelperAction(this));
+        MainApplication.getMenu().imagerySubMenu.add(new CanvecHelperAction(this));
     }
 }

applications/editors/josm/plugins/canvec_helper/src/org/openstreetmap/josm/plugins/canvec_helper/CanvecHelperAction.java

@@ -2 +2 @@
 package org.openstreetmap.josm.plugins.canvec_helper;
 
-import org.openstreetmap.josm.Main;
 import org.openstreetmap.josm.actions.JosmAction;
+import org.openstreetmap.josm.gui.MainApplication;
 
 class CanvecHelperAction extends JosmAction {
@@ -14 +14 @@
     @Override
     public void actionPerformed(java.awt.event.ActionEvent action) {
-        Main.getLayerManager().addLayer(new CanvecLayer("canvec tile helper", parentTemp));
+        MainApplication.getLayerManager().addLayer(new CanvecLayer("canvec tile helper", parentTemp));
     }
 }

applications/editors/josm/plugins/canvec_helper/src/org/openstreetmap/josm/plugins/canvec_helper/CanvecLayer.java

@@ -26 +26 @@
 import javax.swing.JMenuItem;
 
-import org.openstreetmap.josm.Main;
 import org.openstreetmap.josm.data.Bounds;
 import org.openstreetmap.josm.data.osm.visitor.BoundingXYVisitor;
@@ -34 +33 @@
 import org.openstreetmap.josm.gui.layer.Layer;
 import org.openstreetmap.josm.io.CachedFile;
+import org.openstreetmap.josm.tools.Logging;
 
 // most of the layout was copied from the openstreetbugs plugin to get things started
@@ -72 +72 @@
                     lastCell = cell;
                 } else if (!line.contains("Metadata.txt")) {
-                    Main.warn("bad line '" + line + "'\n");
+                    Logging.warn("bad line '" + line + "'\n");
                 }
             }
@@ -78 +78 @@
             if (tile.isValid()) tiles.add(tile);
 
-            if (Main.isDebugEnabled()) {
+            if (Logging.isDebugEnabled()) {
                 long end = System.currentTimeMillis();
-                Main.debug((end-start)+"ms spent");
+                Logging.debug((end-start)+"ms spent");
             }
         } catch (IOException e) {
-            Main.error("exception getting index");
-            Main.error(e);
+            Logging.error("exception getting index");
+            Logging.error(e);
         }
     }