Changeset 27231 in osm for applications

Timestamp:

2011-12-14T11:35:33+01:00 (13 years ago)

Author:

larry0ua

Message:

'PicLayer - changed marker icon to make it more visible; #5451; #7124 - first stage'

Location:

applications/editors/josm/plugins/piclayer

Files:

• build.xml (modified) (1 diff)
• images/v6_64.png (added)
• src/org/openstreetmap/josm/plugins/piclayer/ActionVisibilityChangeMenu.java (deleted)
• src/org/openstreetmap/josm/plugins/piclayer/actions/newlayer/NewLayerFromFileAction.java (modified) (8 diffs)
• src/org/openstreetmap/josm/plugins/piclayer/layer/PicLayerAbstract.java (modified) (6 diffs)
• src/org/openstreetmap/josm/plugins/piclayer/layer/PicLayerFromKML.java (added)
• src/org/openstreetmap/josm/plugins/piclayer/layer/kml (added)
• src/org/openstreetmap/josm/plugins/piclayer/layer/kml/KMLGroundOverlay.java (added)
• src/org/openstreetmap/josm/plugins/piclayer/layer/kml/KMLReader.java (added)

applications/editors/josm/plugins/piclayer/build.xml

@@ -22 +22 @@
 -->
 <project name="PicLayer" default="dist" basedir=".">
-    <property name="commit.message" value="PicLayer - changed marker icon to make it more visible"/>
+    <property name="commit.message" value="PicLayer - changed marker icon to make it more visible; #5451; #7124 - first stage"/>
     <property name="plugin.main.version" value="4549"/>
     <!--

applications/editors/josm/plugins/piclayer/src/org/openstreetmap/josm/plugins/piclayer/actions/newlayer/NewLayerFromFileAction.java

@@ -26 +26 @@
 import java.io.File;
 import java.io.IOException;
+import java.util.ArrayList;
+import java.util.List;
 
 import javax.imageio.ImageIO;
@@ -36 +38 @@
 import org.openstreetmap.josm.data.osm.visitor.BoundingXYVisitor;
 import org.openstreetmap.josm.gui.layer.Layer;
+import org.openstreetmap.josm.plugins.piclayer.layer.PicLayerAbstract;
 import org.openstreetmap.josm.plugins.piclayer.layer.PicLayerFromFile;
+import org.openstreetmap.josm.plugins.piclayer.layer.PicLayerFromKML;
+import org.openstreetmap.josm.plugins.piclayer.layer.kml.KMLGroundOverlay;
+import org.openstreetmap.josm.plugins.piclayer.layer.kml.KMLReader;
 
 /**
@@ -51 +57 @@
     private class ImageFileFilter extends FileFilter {
 
+        private String[] supportedExtensions;
+
+        public ImageFileFilter() {
+            List<String> extensions = new ArrayList<String>();
+            extensions.add("zip");
+            extensions.add("kml");
+            for (String ext : ImageIO.getReaderFormatNames())
+                extensions.add(ext);
+            supportedExtensions = extensions.toArray(new String[0]);
+        }
+
         @Override
         public boolean accept(File f) {
@@ -56 +73 @@
                 return true;
 
-            int dotIdx = f.getName().lastIndexOf('.');
-            if (dotIdx == -1) return false;
-            String fileExtension = f.getName().substring(dotIdx+1);
-            String[] supportedExtensions = ImageIO.getReaderFormatNames();
+            String fileExtension = PicLayerFromFile.getFileExtension(f);
 
-            if ("zip".equalsIgnoreCase(fileExtension)) return true;
             // Unfortunately, getReaderFormatNames does not always return ALL extensions in
             // both lower and upper case, so we can not do a search in the array
@@ -75 +88 @@
         @Override
         public String getDescription() {
-            return tr("Supported image files + *.zip");
+            return tr("Supported image files, *.zip, *.kml");
         }
 
-    }
-
-    private class AllFilesFilter extends FileFilter {
-        @Override
-        public String getDescription() {
-            return tr("All Files");
-        }
-
-        @Override
-        public boolean accept(File f) {
-            return true;
-        }
     }
 
@@ -119 +120 @@
             // The next layers we load will be placed one after the other after this first layer
             int newLayerPos = Main.map.mapView.getAllLayers().size();
-            for(Layer l : Main.map.mapView.getLayersOfType(PicLayerFromFile.class)) {
+            for(Layer l : Main.map.mapView.getLayersOfType(PicLayerAbstract.class)) {
                 int pos = Main.map.mapView.getLayerPos(l);
                 if (pos < newLayerPos) newLayerPos = pos;
@@ -127 +128 @@
                 // TODO: we need a progress bar here, it can take quite some time
 
-                // Create layer from file
-                PicLayerFromFile layer = new PicLayerFromFile( file );
-                // Add layer only if successfully initialized
-                try {
-                    layer.initialize();
-                }
-                catch (IOException e) {
-                    // Failed
-                    System.out.println( "NewLayerFromFileAction::actionPerformed - " + e.getMessage() );
-                    JOptionPane.showMessageDialog(null, e.getMessage() );
-                    return;
-                }
                 Main.pref.put(m_lastdirprefname, file.getParent());
 
-                Main.main.addLayer( layer );
-                Main.map.mapView.moveLayer(layer, newLayerPos++);
+                // Create layer from file
+                if ("kml".equalsIgnoreCase(PicLayerFromFile.getFileExtension(file))) {
+                    KMLReader kml = new KMLReader(file);
+                    kml.process();
+                    JOptionPane.showMessageDialog(null, tr("KML calibration is in beta stage and may produce incorrectly calibrated layers!\nPlease use http://josm.openstreetmap.de/ticket/5451 to upload your KMLs that were calibrated incorrectly."));
+                    for (KMLGroundOverlay overlay : kml.getGroundOverlays()) {
+                        //TODO: zoom to whole picture, not only the last
+                        addNewLayerFromKML(file, overlay, newLayerPos);
+                    }
 
-                if ( fc.getSelectedFiles().length == 1 && Main.pref.getInteger("piclayer.zoom-on-load", 1) != 0 ) {
-                    // if we are loading a single picture file, zoom on it, so that the user can see something
-                    BoundingXYVisitor v = new BoundingXYVisitor();
-                    layer.visitBoundingBox(v);
-                    Main.map.mapView.recalculateCenterScale(v);
-                }
+                } else {
+                addNewLayerFromFile(file, newLayerPos, fc.getSelectedFiles().length == 1);
 
             }
@@ -155 +147 @@
     }
 }
+
+    private void addNewLayerFromFile(File file, int newLayerPos, boolean isZoomToLayer) {
+        try {
+            PicLayerFromFile layer = new PicLayerFromFile( file );
+            layer.initialize();
+
+            placeLayer(layer, newLayerPos, isZoomToLayer);
+        }
+        catch (IOException e) {
+            // Failed
+            System.out.println( "NewLayerFromFileAction::actionPerformed - " + e.getMessage() );
+            JOptionPane.showMessageDialog(null, e.getMessage() );
+        }
+    }
+
+    private void placeLayer(PicLayerAbstract layer, int newLayerPos, boolean isZoomToLayer) throws IOException {
+        // Add layer only if successfully initialized
+
+        Main.main.addLayer( layer );
+        Main.map.mapView.moveLayer(layer, newLayerPos++);
+
+        if ( isZoomToLayer && Main.pref.getInteger("piclayer.zoom-on-load", 1) != 0 ) {
+            // if we are loading a single picture file, zoom on it, so that the user can see something
+            BoundingXYVisitor v = new BoundingXYVisitor();
+            layer.visitBoundingBox(v);
+            Main.map.mapView.recalculateCenterScale(v);
+        }
+    }
+    private void addNewLayerFromKML(File root, KMLGroundOverlay overlay, int newLayerPos) {
+        try {
+            PicLayerFromKML layer = new PicLayerFromKML(root, overlay);
+            layer.initialize();
+
+            placeLayer(layer, newLayerPos, true);
+        } catch (IOException e) {
+            // Failed
+            System.out.println( "NewLayerFromFileAction::actionPerformed - " + e.getMessage() );
+            JOptionPane.showMessageDialog(null, e.getMessage() );
+        }
+    }
+}

applications/editors/josm/plugins/piclayer/src/org/openstreetmap/josm/plugins/piclayer/layer/PicLayerAbstract.java

@@ -65 +65 @@
     // Counter - just for naming of layers
     private static int imageCounter = 0;
+
     // This is the main image to be displayed
-    private Image image = null;
-    private static Image pinImage;
+    protected Image image = null;
+    // Tiles of pin images
+    private static Image pinTiledImage;
+
     // Initial position of the image in the real world
-    private EastNorth initialImagePosition;
+    protected EastNorth initialImagePosition;
+
     // Position of the image in the real world
-    private EastNorth imagePosition;
+    protected EastNorth imagePosition;
+
     // The scale that was set on the map during image creation
-    private double initialImageScale = 1.0;
+    protected double initialImageScale = 1.0;
+
     // Layer icon
     private Icon layerIcon = null;
@@ -83 +89 @@
     }
 
-    private PictureTransform transformer;
+    protected PictureTransform transformer;
 
     public PictureTransform getTransformer() {
@@ -89 +95 @@
     }
 
-    // Keys for saving in Properties
-    private final String INITIAL_POS_X = "INITIAL_POS_X";
-    private final String INITIAL_POS_Y = "INITIAL_POS_y";
-    private final String POSITION_X = "POSITION_X";
-    private final String POSITION_Y = "POSITION_Y";
-    private final String ANGLE = "ANGLE";
-    private final String INITIAL_SCALE = "INITIAL_SCALE";
-    private final String SCALEX = "SCALEX";
-    private final String SCALEY = "SCALEY";
-    private final String SHEARX = "SHEARX";
-    private final String SHEARY = "SHEARY";
-
-    private final String MATRIXm00 = "M00";
-    private final String MATRIXm01 = "M01";
-    private final String MATRIXm10 = "M10";
-    private final String MATRIXm11 = "M11";
-    private final String MATRIXm02 = "M02";
-    private final String MATRIXm12 = "M12";
+    // Keys for loading from old/new Properties
+    private static final String INITIAL_POS_X = "INITIAL_POS_X";
+    private static final String INITIAL_POS_Y = "INITIAL_POS_y";
+    private static final String POSITION_X = "POSITION_X";
+    private static final String POSITION_Y = "POSITION_Y";
+    private static final String ANGLE = "ANGLE";
+    private static final String INITIAL_SCALE = "INITIAL_SCALE";
+    private static final String SCALEX = "SCALEX";
+    private static final String SCALEY = "SCALEY";
+    private static final String SHEARX = "SHEARX";
+    private static final String SHEARY = "SHEARY";
+    // new properties
+    private static final String MATRIXm00 = "M00";
+    private static final String MATRIXm01 = "M01";
+    private static final String MATRIXm10 = "M10";
+    private static final String MATRIXm11 = "M11";
+    private static final String MATRIXm02 = "M02";
+    private static final String MATRIXm12 = "M12";
+
+    // pin images properties - tile anchors, width and offset
+    // TODO: load these from properties file in images folder...
+    private static final int pinAnchorX = 31;
+    private static final int pinAnchorY = 31;
+    private static final int[] pinTileOffsetX = {74, 0, 74, 0};
+    private static final int[] pinTileOffsetY = {0, 74, 74, 0};
+    private static final int pinWidth = 64;
+    private static final int pinHeight = 64;
 
     /**
@@ -120 +135 @@
         layerIcon = new ImageIcon(Toolkit.getDefaultToolkit().createImage(getClass().getResource("/images/layericon.png")));
 
-        if (pinImage == null) {
+        if (pinTiledImage == null) {
             // allow system to load the image and use it in future
-            pinImage = new ImageIcon(Toolkit.getDefaultToolkit().createImage(getClass().getResource("/images/marker.png"))).getImage();
+            pinTiledImage = new ImageIcon(Toolkit.getDefaultToolkit().createImage(getClass().getResource("/images/v6_64.png"))).getImage();
         }
     }
@@ -256 +271 @@
                 tr.concatenate(transformer.getTransform());
 
-                for (Point2D p : transformer.getOriginPoints()) {
-                   Point2D trP = tr.transform(p, null);
+                for (int i = 0; i < transformer.getOriginPoints().size(); i++) {
+                   Point2D trP = tr.transform(transformer.getOriginPoints().get(i), null);
                    int x = (int)trP.getX(), y = (int)trP.getY();
-                   //gPoints.drawOval(x-2, y-2, 5, 5);
-                   gPoints.drawImage(pinImage, x-15, y-15, null);
+
+                   int dstx = x-pinAnchorX;
+                   int dsty = y-pinAnchorY;
+                   gPoints.drawImage(pinTiledImage, dstx, dsty, dstx+pinWidth, dsty+pinHeight, pinTileOffsetX[i], pinTileOffsetY[i], pinTileOffsetX[i]+pinWidth, pinTileOffsetY[i]+pinHeight, null);
                 }
             }
@@ -276 +293 @@
      * next (a couple of kilometers).
      */
-    private double getMetersPerEasting(EastNorth en) {
+    protected double getMetersPerEasting(EastNorth en) {
         /* Natural scale in east/north units per pixel.
          * This means, the projection should be able to handle