Changeset 30828 in osm for applications/editors/josm/plugins/OsmInspectorPlugin/src

Timestamp:

2014-11-29T20:04:57+01:00 (10 years ago)

Author:

donvip

Message:

[josm_osminspector] use geotools plugin, update to geotools 12.1, fix deprecation and Java 7 warnings

Location:

applications/editors/josm/plugins/OsmInspectorPlugin/src/org/openstreetmap/josm/plugins/osminspector

Files:

• OSMIFeatureTracker.java (modified) (2 diffs)
• OsmInspectorLayer.java (modified) (10 diffs)
• gui/OsmInspectorDialog.java (modified) (3 diffs)

applications/editors/josm/plugins/OsmInspectorPlugin/src/org/openstreetmap/josm/plugins/osminspector/OSMIFeatureTracker.java

@@ -6 +6 @@
 import org.geotools.data.memory.MemoryFeatureCollection;
 import org.geotools.feature.FeatureCollection;
+import org.geotools.feature.FeatureIterator;
 import org.opengis.feature.simple.SimpleFeature;
 import org.opengis.feature.simple.SimpleFeatureType;
@@ -34 +35 @@
                 }
 
-                features.addAll(featuresIn);
+                try (FeatureIterator<SimpleFeature> it = featuresIn.features()) {
+                    while (it.hasNext()) {
+                        features.add(it.next());
+                    }
+                }
         }
 
         public boolean mergeFeatures(
                         FeatureCollection<SimpleFeatureType, SimpleFeature> newFeatures) {
-                for (Iterator<SimpleFeature> it = newFeatures.iterator(); it.hasNext();) {
-                        SimpleFeature element = (SimpleFeature) it.next();
-                        try {
-                                Long ID = (Long.parseLong((String) element
-                                                .getAttribute("problem_id")));
-
-                                if (!hashFeatures.containsKey(ID)) {
-                                        hashFeatures.put(ID, element);
-                                        features.add(element);
-                                }
-                        } catch (NumberFormatException e) {
-                                continue;
-                        }
-                }
+            try (FeatureIterator<SimpleFeature> it = newFeatures.features()) {
+                while (it.hasNext()) {
+                        SimpleFeature element = (SimpleFeature) it.next();
+                        try {
+                                Long ID = (Long.parseLong((String) element
+                                                .getAttribute("problem_id")));
+    
+                                if (!hashFeatures.containsKey(ID)) {
+                                        hashFeatures.put(ID, element);
+                                        features.add(element);
+                                }
+                        } catch (NumberFormatException e) {
+                                continue;
+                        }
+                }
+            }
 
                 return true;

applications/editors/josm/plugins/OsmInspectorPlugin/src/org/openstreetmap/josm/plugins/osminspector/OsmInspectorLayer.java

@@ -25 +25 @@
 import org.geotools.feature.FeatureCollection;
 import org.geotools.feature.FeatureIterator;
-import org.geotools.filter.Filter;
 import org.geotools.geometry.jts.ReferencedEnvelope;
-import org.geotools.map.DefaultMapContext;
-import org.geotools.map.MapContext;
+import org.geotools.map.FeatureLayer;
+import org.geotools.map.MapContent;
 import org.geotools.referencing.CRS;
 import org.geotools.renderer.lite.StreamingRenderer;
@@ -46 +45 @@
 import org.opengis.filter.FilterFactory2;
 import org.opengis.filter.identity.FeatureId;
+import org.opengis.filter.spatial.Intersects;
 import org.opengis.referencing.FactoryException;
 import org.opengis.referencing.NoSuchAuthorityCodeException;
@@ -73 +73 @@
 
         private SimpleFeatureSource featureSource;
-        private MapContext context;
+        private MapContent content;
         private boolean bIsChanged;
 
@@ -355 +355 @@
                 CRS.decode(Main.getProjection().toCode());
                 crsOSMI = CRS.decode("EPSG:4326");
-                context = new DefaultMapContext(crsOSMI);
+                content = new MapContent(crsOSMI);
 
                 selectGeomType.add(GeomType.POINT);
@@ -371 +371 @@
                         OSMIFeatureTracker tracker = new OSMIFeatureTracker(features);
                         arrFeatures.add(tracker);
-                        FeatureIterator<SimpleFeature> it = tracker.getFeatures()
-                                        .features();
+                        FeatureIterator<SimpleFeature> it = tracker.getFeatures().features();
 
                         while (it.hasNext()) {
@@ -379 +378 @@
                         }
 
-                        context.addLayer(tracker.getFeatures(), style);
+                        content.addLayer(new FeatureLayer(tracker.getFeatures(), style));
                 }
 
                 osmiIndex = new BugIndex(osmiBugInfo);
-                context.setTitle("Osm Inspector Errors");
-                renderer.setContext(context);
+                content.setTitle("Osm Inspector Errors");
+                renderer.setMapContent(content);
                 bIsChanged = true;
 
@@ -407 +406 @@
                 String typeNames[] = wfsClient.getTypeNames();
 
-                context.clearLayerList();
+                content.layers().clear();
                 selectGeomType.clear();
                 selectGeomType.add(GeomType.POINT);
@@ -438 +437 @@
 
                         Style style = createDefaultStyle(idx, selectedFeatures);
-                        context.addLayer(tracker.getFeatures(), style);
+                        content.addLayer(new FeatureLayer(tracker.getFeatures(), style));
                 }
 
@@ -596 +595 @@
                 ReferencedEnvelope mapArea = new ReferencedEnvelope(envelope, crsOSMI);
 
-                Filter filter = (Filter) ff.intersects(ff.property("msGeometry"),
+                Intersects filter = ff.intersects(ff.property("msGeometry"),
                                 ff.literal(mapArea));
                 //
                 // Select features in all layers
                 //
-                context.clearLayerList();
+                content.layers().clear();
 
                 // Iterate through features and build a list that intersects the above
@@ -615 +614 @@
                         try {
                                 selectedFeatures = tempfs.getFeatures(filter);
-
-                                FeatureIterator<SimpleFeature> iter = selectedFeatures.features();
-                                Set<FeatureId> IDs = new HashSet<>();
-
-                                Main.info("Selected features " + selectedFeatures.size());
-
-                                while (iter.hasNext()) {
-                                        SimpleFeature feature = iter.next();
-                                        IDs.add(feature.getIdentifier());
+                Set<FeatureId> IDs = new HashSet<>();
+
+                                try (FeatureIterator<SimpleFeature> iter = selectedFeatures.features()) {
+    
+                                Main.info("Selected features " + selectedFeatures.size());
+    
+                                while (iter.hasNext()) {
+                                        SimpleFeature feature = iter.next();
+                                        IDs.add(feature.getIdentifier());
+                                }
                                 }
-
-                                iter.close();
 
                                 geometryType = selectGeomType.get(idx + layerOffset);
                                 Style style = createDefaultStyle(idx + layerOffset, IDs);
-                                context.addLayer(features, style);
+                                content.addLayer(new FeatureLayer(features, style));
                         } catch (IOException e) {
-                                // TODO Auto-generated catch block
-                                e.printStackTrace();
+                                Main.error(e);
                         }
                 }

applications/editors/josm/plugins/OsmInspectorPlugin/src/org/openstreetmap/josm/plugins/osminspector/gui/OsmInspectorDialog.java

@@ -44 +44 @@
         
         public void updateNextPrevAction(OsmInspectorLayer l) {
-                this.actNext.layer = l;
-                this.actPrev.layer = l;
+                this.actNext.inspectlayer = l;
+                this.actPrev.inspectlayer = l;
         }
         
@@ -164 +164 @@
                         ListSelectionListener {
 
-                private OsmInspectorLayer layer;
+                private OsmInspectorLayer inspectlayer;
 
                 public OsmInspectorNextAction(Layer inspector) {
                         super("next");
-                        layer = (OsmInspectorLayer) inspector;
+                        inspectlayer = (OsmInspectorLayer) inspector;
                 }
 
                 @Override
                 public void actionPerformed(ActionEvent e) {
-                        layer.getOsmiIndex().next();
-                        BugInfo next = layer.getOsmiIndex().getItemPointedByNext();
+                        inspectlayer.getOsmiIndex().next();
+                        BugInfo next = inspectlayer.getOsmiIndex().getItemPointedByNext();
                         Geometry geom = next.getGeom();
                         Point centroid = geom.getCentroid();
                         LatLon center = new LatLon(centroid.getY(), centroid.getX());
                         Main.map.mapView.zoomTo(center);
-                        layer.selectFeatures(center);
+                        inspectlayer.selectFeatures(center);
                         bugInfoDialog.setBugDescription(next);
                         updateSelection(next);
@@ -199 +199 @@
                         ListSelectionListener {
 
-                private OsmInspectorLayer layer;
+                private OsmInspectorLayer inspectlayer;
 
                 public OsmInspectorPrevAction(Layer inspector) {
                         super("prev");
-                        layer = (OsmInspectorLayer) inspector;
+                        inspectlayer = (OsmInspectorLayer) inspector;
                 }
 
                 @Override
                 public void actionPerformed(ActionEvent e) {
-                        layer.getOsmiIndex().prev();
-                        BugInfo prev = layer.getOsmiIndex().getItemPointedByPrev();
+                        inspectlayer.getOsmiIndex().prev();
+                        BugInfo prev = inspectlayer.getOsmiIndex().getItemPointedByPrev();
                         Geometry geom = prev.getGeom();
                         Point centroid = geom.getCentroid();
                         LatLon center = new LatLon(centroid.getY(), centroid.getX());
                         Main.map.mapView.zoomTo(center);
-                        layer.selectFeatures(center);
+                        inspectlayer.selectFeatures(center);
                         bugInfoDialog.setBugDescription(prev);
                         updateSelection(prev);