Changeset 10398 in josm for trunk

Timestamp:

2016-06-16T15:10:50+02:00 (8 years ago)

Author:

Don-vip

Message:

fix #12975 - areas in the mapview are not filled anymore (patch by michael2402) - gsoc-core

Location:

trunk

Files:

• src/org/openstreetmap/josm/gui/MapViewState.java (modified) (1 diff)
• test/unit/org/openstreetmap/josm/gui/MapViewStateTest.java (modified) (3 diffs)

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

@@ -172 +172 @@
      */
     public AffineTransform getAffineTransform() {
-        return new AffineTransform(1.0 / scale, 0.0, 0.0, -1.0 / scale, topLeft.east() / scale,
+        return new AffineTransform(1.0 / scale, 0.0, 0.0, -1.0 / scale, -topLeft.east() / scale,
                 topLeft.north() / scale);
     }

trunk/test/unit/org/openstreetmap/josm/gui/MapViewStateTest.java

@@ -3 +3 @@
 
 import static org.junit.Assert.assertEquals;
+
+import java.awt.geom.AffineTransform;
+import java.awt.geom.Point2D;
+import java.util.Arrays;
 
 import org.junit.Before;
@@ -100 +104 @@
         assertHasViewCoords(WIDTH / 2, HEIGHT / 2, p);
 
-
         EastNorth eastnorth = p.getEastNorth();
         LatLon shouldLatLon = Main.getProjection().getWorldBoundsLatLon().getCenter();
@@ -117 +120 @@
         assertEquals("north", 3, p2.getEastNorth().north(), 0.01);
     }
+
+    /**
+     * Test {@link MapViewState#getAffineTransform()}
+     */
+    @Test
+    public void testGetAffineTransform() {
+        for (EastNorth en : Arrays.asList(new EastNorth(100, 100), new EastNorth(0, 0), new EastNorth(300, 200),
+                new EastNorth(-1, -2.5))) {
+            MapViewPoint should = state.getPointFor(en);
+            AffineTransform transform = state.getAffineTransform();
+            Point2D result = transform.transform(new Point2D.Double(en.getX(), en.getY()), null);
+
+            assertEquals("x", should.getInViewX(), result.getX(), 0.01);
+            assertEquals("y", should.getInViewY(), result.getY(), 0.01);
+        }
+    }
 }