Changeset 33809 in osm for applications/editors/josm/plugins/piclayer/src

Timestamp:

2017-11-14T20:02:52+01:00 (7 years ago)

Author:

donvip

Message:

fix #josm15551 - update to JOSM 13050

Location:

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

Files:

• PicLayerPlugin.java (modified) (4 diffs)
• actions/GenericPicTransformAction.java (modified) (7 diffs)
• actions/newlayer/NewLayerFromClipboardAction.java (modified) (2 diffs)
• actions/newlayer/NewLayerFromFileAction.java (modified) (3 diffs)
• actions/transform/MovePictureAction.java (modified) (2 diffs)
• actions/transform/RotatePictureAction.java (modified) (2 diffs)
• actions/transform/ScaleXPictureAction.java (modified) (2 diffs)
• actions/transform/ScaleXYPictureAction.java (modified) (2 diffs)
• actions/transform/ScaleYPictureAction.java (modified) (2 diffs)
• actions/transform/ShearPictureAction.java (modified) (2 diffs)
• actions/transform/affine/MovePointAction.java (modified) (1 diff)
• actions/transform/affine/RemovePointAction.java (modified) (1 diff)
• actions/transform/affine/TransformPointAction.java (modified) (1 diff)
• command/TransformCommand.java (modified) (3 diffs)
• layer/PicLayerAbstract.java (modified) (10 diffs)
• layer/PicLayerFromFile.java (modified) (4 diffs)

applications/editors/josm/plugins/piclayer/src/org/openstreetmap/josm/plugins/piclayer/PicLayerPlugin.java

@@ -32 +32 @@
 import org.openstreetmap.josm.actions.mapmode.MapMode;
 import org.openstreetmap.josm.gui.IconToggleButton;
+import org.openstreetmap.josm.gui.MainApplication;
 import org.openstreetmap.josm.gui.MainMenu;
 import org.openstreetmap.josm.gui.MapFrame;
@@ -76 +77 @@
 
         // Add menu items
-        MainMenu.add(Main.main.menu.imagerySubMenu, newLayerFromFileAction);
-        MainMenu.add(Main.main.menu.imagerySubMenu, newLayerFromClipboardAction);
+        MainMenu.add(MainApplication.getMenu().imagerySubMenu, newLayerFromFileAction);
+        MainMenu.add(MainApplication.getMenu().imagerySubMenu, newLayerFromClipboardAction);
         updateEnabledState();
         // Listen to layers
-        Main.getLayerManager().addLayerChangeListener(this);
-        Main.getLayerManager().addActiveLayerChangeListener(this);
+        MainApplication.getLayerManager().addLayerChangeListener(this);
+        MainApplication.getLayerManager().addActiveLayerChangeListener(this);
     }
 
@@ -132 +133 @@
     public void activeOrEditLayerChanged(ActiveLayerChangeEvent e) {
         Layer oldLayer = e.getPreviousActiveLayer();
-        Layer newLayer = Main.getLayerManager().getActiveLayer();
+        Layer newLayer = MainApplication.getLayerManager().getActiveLayer();
         boolean oldPic = oldLayer instanceof PicLayerAbstract;
         boolean newPic = newLayer instanceof PicLayerAbstract;
@@ -177 +178 @@
 
     private void updateEnabledState() {
-        boolean enable = !Main.getLayerManager().getLayers().isEmpty();
+        boolean enable = !MainApplication.getLayerManager().getLayers().isEmpty();
         newLayerFromFileAction.setEnabled(enable);
         newLayerFromClipboardAction.setEnabled(enable);

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

@@ -8 +8 @@
 import java.awt.geom.Point2D;
 
-import org.openstreetmap.josm.Main;
 import org.openstreetmap.josm.actions.mapmode.MapMode;
 import org.openstreetmap.josm.data.coor.EastNorth;
+import org.openstreetmap.josm.gui.MainApplication;
 import org.openstreetmap.josm.gui.MapFrame;
 import org.openstreetmap.josm.gui.layer.Layer;
@@ -43 +43 @@
     public void enterMode() {
         super.enterMode();
-        Main.map.mapView.addMouseListener(this);
-        Main.map.mapView.addMouseMotionListener(this);
+        MainApplication.getMap().mapView.addMouseListener(this);
+        MainApplication.getMap().mapView.addMouseMotionListener(this);
     }
 
@@ -50 +50 @@
     public void exitMode() {
         super.exitMode();
-        Main.map.mapView.removeMouseListener(this);
-        Main.map.mapView.removeMouseMotionListener(this);
+        MainApplication.getMap().mapView.removeMouseListener(this);
+        MainApplication.getMap().mapView.removeMouseMotionListener(this);
     }
 
@@ -57 +57 @@
     public void mousePressed(MouseEvent e) {
         // Start action
-        if (Main.getLayerManager().getActiveLayer() instanceof PicLayerAbstract) {
-            currentLayer = (PicLayerAbstract) Main.getLayerManager().getActiveLayer();
+        if (MainApplication.getLayerManager().getActiveLayer() instanceof PicLayerAbstract) {
+            currentLayer = (PicLayerAbstract) MainApplication.getLayerManager().getActiveLayer();
 
             if (currentLayer != null && e.getButton() == MouseEvent.BUTTON1) {
@@ -64 +64 @@
                 isDragging = true;
                 prevMousePoint = new Point(e.getPoint());
-                prevEastNorth = Main.map.mapView.getEastNorth(e.getX(), e.getY());
+                prevEastNorth = MainApplication.getMap().mapView.getEastNorth(e.getX(), e.getY());
                 // try to find and fill selected point if possible
                 selectedPoint = currentLayer.findSelectedPoint(e.getPoint());
@@ -78 +78 @@
             doAction(e);
             prevMousePoint = new Point(e.getPoint());
-            prevEastNorth = Main.map.mapView.getEastNorth(e.getX(), e.getY());
+            prevEastNorth = MainApplication.getMap().mapView.getEastNorth(e.getX(), e.getY());
             currentLayer.invalidate();
         }
@@ -99 +99 @@
 
     protected void updateDrawPoints(boolean value) {
-        Layer active = Main.getLayerManager().getActiveLayer();
+        Layer active = MainApplication.getLayerManager().getActiveLayer();
         if (active instanceof PicLayerAbstract) {
             ((PicLayerAbstract) active).setDrawPoints(value);

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

@@ -28 +28 @@
 import javax.swing.JOptionPane;
 
-import org.openstreetmap.josm.Main;
 import org.openstreetmap.josm.actions.JosmAction;
+import org.openstreetmap.josm.gui.MainApplication;
 import org.openstreetmap.josm.plugins.piclayer.layer.PicLayerFromClipboard;
 
@@ -63 +63 @@
         }
         // Add layer
-        Main.getLayerManager().addLayer(layer);
+        MainApplication.getLayerManager().addLayer(layer);
     }
 }

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

@@ -37 +37 @@
 import org.openstreetmap.josm.actions.JosmAction;
 import org.openstreetmap.josm.data.osm.visitor.BoundingXYVisitor;
+import org.openstreetmap.josm.gui.MainApplication;
 import org.openstreetmap.josm.gui.layer.Layer;
 import org.openstreetmap.josm.plugins.piclayer.layer.PicLayerAbstract;
@@ -119 +120 @@
             // or at the bottom of the stack if there is no such layer yet
             // The next layers we load will be placed one after the other after this first layer
-            int newLayerPos = Main.getLayerManager().getLayers().size();
-            for (Layer l : Main.getLayerManager().getLayersOfType(PicLayerAbstract.class)) {
-                int pos = Main.getLayerManager().getLayers().indexOf(l);
+            int newLayerPos = MainApplication.getLayerManager().getLayers().size();
+            for (Layer l : MainApplication.getLayerManager().getLayersOfType(PicLayerAbstract.class)) {
+                int pos = MainApplication.getLayerManager().getLayers().indexOf(l);
                 if (pos < newLayerPos) newLayerPos = pos;
             }
@@ -164 +165 @@
         // Add layer only if successfully initialized
 
-        Main.getLayerManager().addLayer(layer);
-        Main.map.mapView.moveLayer(layer, newLayerPos++);
+        MainApplication.getLayerManager().addLayer(layer);
+        MainApplication.getMap().mapView.moveLayer(layer, newLayerPos++);
 
-        if (isZoomToLayer && Main.pref.getInteger("piclayer.zoom-on-load", 1) != 0) {
+        if (isZoomToLayer && Main.pref.getInt("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.zoomTo(v);
+            MainApplication.getMap().mapView.zoomTo(v);
         }
     }

applications/editors/josm/plugins/piclayer/src/org/openstreetmap/josm/plugins/piclayer/actions/transform/MovePictureAction.java

@@ -25 +25 @@
 import java.awt.event.MouseEvent;
 
-import org.openstreetmap.josm.Main;
 import org.openstreetmap.josm.data.coor.EastNorth;
+import org.openstreetmap.josm.gui.MainApplication;
 import org.openstreetmap.josm.gui.MapFrame;
 import org.openstreetmap.josm.plugins.piclayer.actions.GenericPicTransformAction;
@@ -41 +41 @@
      */
     public MovePictureAction(MapFrame frame) {
-        super(tr("PicLayer move"), tr("Moved"), "move", tr("Drag to move the picture"), frame, ImageProvider.getCursor("crosshair", null));
+        super(tr("PicLayer move"), tr("Moved"), "move", tr("Drag to move the picture"),
+                frame, ImageProvider.getCursor("crosshair", null));
     }
 
     @Override
     protected void doAction(MouseEvent e) {
-        EastNorth eastNorth = Main.map.mapView.getEastNorth(e.getX(), e.getY());
+        EastNorth eastNorth = MainApplication.getMap().mapView.getEastNorth(e.getX(), e.getY());
         currentLayer.movePictureBy(
             eastNorth.east() - prevEastNorth.east(),

applications/editors/josm/plugins/piclayer/src/org/openstreetmap/josm/plugins/piclayer/actions/transform/RotatePictureAction.java

@@ -27 +27 @@
 import java.awt.geom.Point2D;
 
-import org.openstreetmap.josm.Main;
+import org.openstreetmap.josm.gui.MainApplication;
 import org.openstreetmap.josm.gui.MapFrame;
 import org.openstreetmap.josm.plugins.piclayer.actions.GenericPicTransformAction;
@@ -48 +48 @@
     @Override
     protected void doAction(MouseEvent e) {
-        Point2D center = new Point(Main.map.mapView.getWidth()/2, Main.map.mapView.getHeight()/2);
+        Point2D center = new Point(MainApplication.getMap().mapView.getWidth()/2, MainApplication.getMap().mapView.getHeight()/2);
         double alpha1 = Math.atan2(e.getY() - center.getY(), e.getX() - center.getX());
         double alpha0 = Math.atan2(prevMousePoint.getY() - center.getY(), prevMousePoint.getX() - center.getX());

applications/editors/josm/plugins/piclayer/src/org/openstreetmap/josm/plugins/piclayer/actions/transform/ScaleXPictureAction.java

@@ -25 +25 @@
 import java.awt.event.MouseEvent;
 
-import org.openstreetmap.josm.Main;
+import org.openstreetmap.josm.gui.MainApplication;
 import org.openstreetmap.josm.gui.MapFrame;
 import org.openstreetmap.josm.plugins.piclayer.actions.GenericPicTransformAction;
@@ -40 +40 @@
      */
     public ScaleXPictureAction(MapFrame frame) {
-        super(tr("PicLayer scale X"), tr("Scaled by X"), "scale_x", tr("Drag to scale the picture in the X Axis"), frame, ImageProvider.getCursor("crosshair", null));
+        super(tr("PicLayer scale X"), tr("Scaled by X"), "scale_x", tr("Drag to scale the picture in the X Axis"),
+                frame, ImageProvider.getCursor("crosshair", null));
     }
 
     @Override
     protected void doAction(MouseEvent e) {
-        double centerX = Main.map.mapView.getWidth()/2;
+        double centerX = MainApplication.getMap().mapView.getWidth()/2;
         double dx0 = Math.max(Math.abs(prevMousePoint.getX() - centerX), 10);
         double dx = Math.abs(e.getX() - centerX);

applications/editors/josm/plugins/piclayer/src/org/openstreetmap/josm/plugins/piclayer/actions/transform/ScaleXYPictureAction.java

@@ -27 +27 @@
 import java.awt.geom.Point2D;
 
-import org.openstreetmap.josm.Main;
+import org.openstreetmap.josm.gui.MainApplication;
 import org.openstreetmap.josm.gui.MapFrame;
 import org.openstreetmap.josm.plugins.piclayer.actions.GenericPicTransformAction;
@@ -42 +42 @@
      */
     public ScaleXYPictureAction(MapFrame frame) {
-        super(tr("PicLayer scale"), tr("Scaled"), "scale", tr("Drag to scale the picture in the X and Y Axis"), frame, ImageProvider.getCursor("crosshair", null));
+        super(tr("PicLayer scale"), tr("Scaled"), "scale", tr("Drag to scale the picture in the X and Y Axis"),
+                frame, ImageProvider.getCursor("crosshair", null));
     }
 
     @Override
     protected void doAction(MouseEvent e) {
-        double centerX = Main.map.mapView.getWidth()/2;
-        double centerY = Main.map.mapView.getHeight()/2;
+        double centerX = MainApplication.getMap().mapView.getWidth()/2;
+        double centerY = MainApplication.getMap().mapView.getHeight()/2;
         double d0 = Math.max(prevMousePoint.distance(centerX, centerY), 10);
         Point2D mousePoint = new Point(e.getX(), e.getY());

applications/editors/josm/plugins/piclayer/src/org/openstreetmap/josm/plugins/piclayer/actions/transform/ScaleYPictureAction.java

@@ -25 +25 @@
 import java.awt.event.MouseEvent;
 
-import org.openstreetmap.josm.Main;
+import org.openstreetmap.josm.gui.MainApplication;
 import org.openstreetmap.josm.gui.MapFrame;
 import org.openstreetmap.josm.plugins.piclayer.actions.GenericPicTransformAction;
@@ -40 +40 @@
      */
     public ScaleYPictureAction(MapFrame frame) {
-        super(tr("PicLayer scale Y"), tr("Scaled by Y"), "scale_y", tr("Drag to scale the picture in the Y Axis"), frame, ImageProvider.getCursor("crosshair", null));
+        super(tr("PicLayer scale Y"), tr("Scaled by Y"), "scale_y", tr("Drag to scale the picture in the Y Axis"),
+                frame, ImageProvider.getCursor("crosshair", null));
     }
 
     @Override
     protected void doAction(MouseEvent e) {
-        double centerY = Main.map.mapView.getHeight()/2;
+        double centerY = MainApplication.getMap().mapView.getHeight()/2;
         double dy0 = Math.max(Math.abs(prevMousePoint.getY() - centerY), 10);
         double dy = Math.abs(e.getY() - centerY);

applications/editors/josm/plugins/piclayer/src/org/openstreetmap/josm/plugins/piclayer/actions/transform/ShearPictureAction.java

@@ -27 +27 @@
 import java.awt.event.MouseEvent;
 
-import org.openstreetmap.josm.Main;
 import org.openstreetmap.josm.data.coor.EastNorth;
+import org.openstreetmap.josm.gui.MainApplication;
 import org.openstreetmap.josm.gui.MapFrame;
 import org.openstreetmap.josm.plugins.piclayer.actions.GenericPicTransformAction;
@@ -48 +48 @@
     @Override
     protected void doAction(MouseEvent e) {
-        EastNorth eastNorth = Main.map.mapView.getEastNorth(e.getX(), e.getY());
+        EastNorth eastNorth = MainApplication.getMap().mapView.getEastNorth(e.getX(), e.getY());
         currentLayer.shearPictureBy(
             1000* (eastNorth.east() - prevEastNorth.east()),

applications/editors/josm/plugins/piclayer/src/org/openstreetmap/josm/plugins/piclayer/actions/transform/affine/MovePointAction.java

@@ -16 +16 @@
 
     public MovePointAction(MapFrame frame) {
-        super(tr("PicLayer Move point"), tr("Point added/moved"), "movepoint", tr("Drag or create point on the picture"), frame, ImageProvider.getCursor("crosshair", null));
+        super(tr("PicLayer Move point"), tr("Point added/moved"), "movepoint", tr("Drag or create point on the picture"),
+                frame, ImageProvider.getCursor("crosshair", null));
     }
 

applications/editors/josm/plugins/piclayer/src/org/openstreetmap/josm/plugins/piclayer/actions/transform/affine/RemovePointAction.java

@@ -13 +13 @@
 
     public RemovePointAction(MapFrame frame) {
-        super(tr("PicLayer Remove point"), tr("Point removed"), "removepoint", tr("Remove point on the picture"), frame, ImageProvider.getCursor("crosshair", null));
+        super(tr("PicLayer Remove point"), tr("Point removed"), "removepoint", tr("Remove point on the picture"),
+                frame, ImageProvider.getCursor("crosshair", null));
     }
 

applications/editors/josm/plugins/piclayer/src/org/openstreetmap/josm/plugins/piclayer/actions/transform/affine/TransformPointAction.java

@@ -15 +15 @@
 
     public TransformPointAction(MapFrame frame) {
-        super(tr("PicLayer Transform point"), tr("Point transformed"), "transformpoint", tr("Transform point on the picture"), frame, ImageProvider.getCursor("crosshair", null));
+        super(tr("PicLayer Transform point"), tr("Point transformed"), "transformpoint", tr("Transform point on the picture"),
+                frame, ImageProvider.getCursor("crosshair", null));
     }
 

applications/editors/josm/plugins/piclayer/src/org/openstreetmap/josm/plugins/piclayer/command/TransformCommand.java

@@ -10 +10 @@
 import org.openstreetmap.josm.command.Command;
 import org.openstreetmap.josm.data.osm.OsmPrimitive;
-import org.openstreetmap.josm.gui.layer.Layer;
+import org.openstreetmap.josm.gui.MainApplication;
 import org.openstreetmap.josm.plugins.piclayer.layer.PicLayerAbstract;
 import org.openstreetmap.josm.plugins.piclayer.transform.PictureTransform;
@@ -23 +23 @@
 
     public TransformCommand(PicLayerAbstract layer, String actionName) {
+        super(MainApplication.getLayerManager().getEditDataSet());
         this.layer = layer;
         this.actionName = actionName;
@@ -61 +62 @@
 
     @Override
-    public boolean invalidBecauselayerRemoved(Layer oldLayer) {
-        return oldLayer == layer;
-    }
-
-    @Override
     public Icon getDescriptionIcon() {
         return ImageProvider.get("layericon");

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

@@ -50 +50 @@
 import org.openstreetmap.josm.data.osm.visitor.BoundingXYVisitor;
 import org.openstreetmap.josm.data.projection.Projection;
+import org.openstreetmap.josm.gui.MainApplication;
 import org.openstreetmap.josm.gui.MapView;
 import org.openstreetmap.josm.gui.layer.Layer;
@@ -58 +59 @@
 import org.openstreetmap.josm.plugins.piclayer.actions.SavePictureCalibrationToWorldAction;
 import org.openstreetmap.josm.plugins.piclayer.transform.PictureTransform;
+import org.openstreetmap.josm.tools.JosmDecimalFormatSymbolsProvider;
+import org.openstreetmap.josm.tools.Logging;
 
 /**
@@ -156 +159 @@
 
         // If the map does not exist - we're screwed. We should not get into this situation in the first place!
-        if (Main.map != null && Main.map.mapView != null) {
-
-            EastNorth center = Main.map.mapView.getCenter();
+        if (MainApplication.getMap() != null && MainApplication.getMap().mapView != null) {
+
+            EastNorth center = MainApplication.getMap().mapView.getCenter();
 
 //            imagePosition = new EastNorth(center.east(), center.north());
@@ -164 +167 @@
 //            initialImagePosition = new EastNorth(imagePosition.east(), imagePosition.north());
             // Initial scale at which the image was loaded
-            initialImageScale = Main.map.mapView.getDist100Pixel();
+            initialImageScale = MainApplication.getMap().mapView.getDist100Pixel();
         } else {
             throw new IOException(tr("Could not find the map object."));
@@ -449 +452 @@
             // initialize matrix
             double[] matrix = new double[6];
-            matrix[0] = Double.parseDouble(props.getProperty(MATRIXm00, "1"));
-            matrix[1] = Double.parseDouble(props.getProperty(MATRIXm01, "0"));
-            matrix[2] = Double.parseDouble(props.getProperty(MATRIXm10, "0"));
-            matrix[3] = Double.parseDouble(props.getProperty(MATRIXm11, "1"));
-            matrix[4] = Double.parseDouble(props.getProperty(MATRIXm02, "0"));
-            matrix[5] = Double.parseDouble(props.getProperty(MATRIXm12, "0"));
+            matrix[0] = JosmDecimalFormatSymbolsProvider.parseDouble(props.getProperty(MATRIXm00, "1"));
+            matrix[1] = JosmDecimalFormatSymbolsProvider.parseDouble(props.getProperty(MATRIXm01, "0"));
+            matrix[2] = JosmDecimalFormatSymbolsProvider.parseDouble(props.getProperty(MATRIXm10, "0"));
+            matrix[3] = JosmDecimalFormatSymbolsProvider.parseDouble(props.getProperty(MATRIXm11, "1"));
+            matrix[4] = JosmDecimalFormatSymbolsProvider.parseDouble(props.getProperty(MATRIXm02, "0"));
+            matrix[5] = JosmDecimalFormatSymbolsProvider.parseDouble(props.getProperty(MATRIXm12, "0"));
 
             transform = new AffineTransform(matrix);
@@ -474 +477 @@
             for (int i = 0; i < 6; ++i) {
                 String line = br.readLine();
-                e[i] = Double.parseDouble(line);
+                e[i] = JosmDecimalFormatSymbolsProvider.parseDouble(line);
             }
             double sx = e[0], ry = e[1], rx = e[2], sy = e[3], dx = e[4], dy = e[5];
@@ -532 +535 @@
         // Position image at the right graphical place
 
-        EastNorth center = Main.map.mapView.getCenter();
-        EastNorth leftop = Main.map.mapView.getEastNorth(0, 0);
+        EastNorth center = MainApplication.getMap().mapView.getCenter();
+        EastNorth leftop = MainApplication.getMap().mapView.getEastNorth(0, 0);
         // Number of pixels for one unit in east north space.
         // This is the same in x- and y- direction.
-        double pixel_per_en = (Main.map.mapView.getWidth() / 2.0) / (center.east() - leftop.east());
+        double pixel_per_en = (MainApplication.getMap().mapView.getWidth() / 2.0) / (center.east() - leftop.east());
 
         EastNorth imageCenter = transformer.getImagePosition();
@@ -565 +568 @@
     public void rotatePictureBy(double angle) {
         try {
-            Point2D trans = transformPoint(new Point(Main.map.mapView.getWidth()/2, Main.map.mapView.getHeight()/2));
-
+            MapView mapView = MainApplication.getMap().mapView;
+            Point2D trans = transformPoint(new Point(mapView.getWidth()/2, mapView.getHeight()/2));
             transformer.concatenateTransformPoint(AffineTransform.getRotateInstance(angle), trans);
         } catch (NoninvertibleTransformException e) {
-            e.printStackTrace();
+            Logging.error(e);
         }
     }
@@ -575 +578 @@
     public void scalePictureBy(double scalex, double scaley) {
         try {
-            Point2D trans = transformPoint(new Point(Main.map.mapView.getWidth()/2, Main.map.mapView.getHeight()/2));
-
+            MapView mapView = MainApplication.getMap().mapView;
+            Point2D trans = transformPoint(new Point(mapView.getWidth()/2, mapView.getHeight()/2));
             transformer.concatenateTransformPoint(AffineTransform.getScaleInstance(scalex, scaley), trans);
         } catch (NoninvertibleTransformException e) {
-            e.printStackTrace();
+            Logging.error(e);
         }
     }
@@ -585 +588 @@
     public void shearPictureBy(double shx, double shy) {
         try {
-            Point2D trans = transformPoint(new Point(Main.map.mapView.getWidth()/2, Main.map.mapView.getHeight()/2));
-
+            MapView mapView = MainApplication.getMap().mapView;
+            Point2D trans = transformPoint(new Point(mapView.getWidth()/2, mapView.getHeight()/2));
             transformer.concatenateTransformPoint(AffineTransform.getShearInstance(shx, shy), trans);
         } catch (NoninvertibleTransformException e) {
-            e.printStackTrace();
+            Logging.error(e);
         }
     }

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

@@ -37 +37 @@
 
 import org.openstreetmap.josm.Main;
+import org.openstreetmap.josm.tools.Logging;
 /**
  * Layer displaying a picture loaded from a file.
@@ -71 +72 @@
     protected Image createImage() throws IOException {
         // Try to load file
-        Image image = null;
-
         if (isZip) {
             try (ZipFile zipFile = new ZipFile(m_file)) {
@@ -95 +94 @@
                     }
                 }
-                System.err.println("Warning: no image in zip file found");
+                Logging.warn("Warning: no image in zip file found");
                 return null;
             } catch (Exception e) {
-                System.err.println(tr("Warning: failed to handle zip file ''{0}''. Exception was: {1}", m_file.getName(), e.toString()));
+                Logging.warn(tr("Warning: failed to handle zip file ''{0}''. Exception was: {1}", m_file.getName(), e.toString()));
                 return null;
             }
         } else {
-            image = ImageIO.read(m_file);
-            return image;
+            return ImageIO.read(m_file);
         }
     }
@@ -170 +168 @@
                 }
             } catch (Exception e) {
-                Main.warn(tr("Warning: failed to handle zip file ''{0}''. Exception was: {1}", m_file.getName(), e.toString()));
+                Logging.warn(tr("Warning: failed to handle zip file ''{0}''. Exception was: {1}", m_file.getName(), e.toString()));
                 return;
             }