Changeset 1629 in josm for trunk/src/org/openstreetmap/josm/data/osm/visitor

Timestamp:

2009-05-30T17:24:29+02:00 (16 years ago)

Author:

stoecker

Message:

close #2426 - now mappaint handles area types which require closed ways (e.g. cliff) correctly

File:

• trunk/src/org/openstreetmap/josm/data/osm/visitor/MapPaintVisitor.java (modified) (4 diffs)

trunk/src/org/openstreetmap/josm/data/osm/visitor/MapPaintVisitor.java

@@ -223 +223 @@
         else if (wayStyle instanceof AreaElemStyle)
         {
+            AreaElemStyle areaStyle = (AreaElemStyle) wayStyle;
             /* way with area style */
             //if(!profilerOmitDraw)
             //{
-            if (fillAreas > dist)
+            if (fillAreas > dist && (!areaStyle.closed || w.isClosed()))
             {
             //    profilerVisibleAreas++;
-                drawArea(w, w.selected ? selectedColor : ((AreaElemStyle)wayStyle).color);
+                drawArea(w, w.selected ? selectedColor : areaStyle.color);
                 if(!w.isClosed())
                     w.putError(tr("Area style way is not closed."), true);
             }
-            drawWay(w, ((AreaElemStyle)wayStyle).line, ((AreaElemStyle)wayStyle).color, w.selected);
+            drawWay(w, areaStyle.line, areaStyle.color, w.selected);
             //}
         }
@@ -456 +457 @@
             if(style instanceof AreaElemStyle)
             {
-                drawWay((Way)osm, ((AreaElemStyle)style).line, selectedColor, true);
-                if(area)
-                    drawArea((Way)osm, areaselected ? selectedColor
-                    : ((AreaElemStyle)style).color);
+                Way way = (Way)osm;
+                AreaElemStyle areaStyle = (AreaElemStyle)style;
+                drawWay(way, areaStyle.line, selectedColor, true);
+                if(area && (!areaStyle.closed || way.isClosed()))
+                    drawArea(way, areaselected ? selectedColor : areaStyle.color);
             }
             else
@@ -907 +909 @@
                         inner.add(p);
                     }
+                    public boolean isClosed()
+                    {
+                        return (poly.npoints >= 3
+                        && poly.xpoints[0] == poly.xpoints[poly.npoints-1]
+                        && poly.ypoints[0] == poly.ypoints[poly.npoints-1]);
+                    }
                     public Polygon get()
                     {
@@ -967 +975 @@
                     o.addInner(polygon);
                 }
+                AreaElemStyle areaStyle = (AreaElemStyle)wayStyle;
                 for (PolyData pd : poly)
                 {
-                    if(isPolygonVisible(pd.get()))
-                    {
-                        drawAreaPolygon(pd.get(), (pd.way.selected || r.selected) ? selectedColor
-                        : ((AreaElemStyle)wayStyle).color);
+                    Polygon p = pd.get();
+                    if(isPolygonVisible(p) && (!areaStyle.closed || pd.isClosed()))
+                    {
+                        drawAreaPolygon(p, (pd.way.selected || r.selected) ? selectedColor
+                        : areaStyle.color);
                         visible = true;
                     }