Revision: 5163 http://sourceforge.net/p/jump-pilot/code/5163 Author: michaudm Date: 2016-11-06 10:57:52 +0000 (Sun, 06 Nov 2016) Log Message: ----------- Small improvements, fix I18N typo
Modified Paths: -------------- core/trunk/src/org/openjump/core/ui/plugin/mousemenu/MeasureSelectedPlugIn.java Modified: core/trunk/src/org/openjump/core/ui/plugin/mousemenu/MeasureSelectedPlugIn.java =================================================================== --- core/trunk/src/org/openjump/core/ui/plugin/mousemenu/MeasureSelectedPlugIn.java 2016-11-06 10:55:04 UTC (rev 5162) +++ core/trunk/src/org/openjump/core/ui/plugin/mousemenu/MeasureSelectedPlugIn.java 2016-11-06 10:57:52 UTC (rev 5163) @@ -1,7 +1,6 @@ package org.openjump.core.ui.plugin.mousemenu; import java.util.Collection; -import java.util.Iterator; import javax.swing.Icon; import javax.swing.JComponent; @@ -9,12 +8,10 @@ import com.vividsolutions.jts.geom.Geometry; import com.vividsolutions.jts.geom.GeometryCollection; import com.vividsolutions.jts.geom.LineString; -import com.vividsolutions.jts.geom.MultiLineString; import com.vividsolutions.jts.geom.Point; import com.vividsolutions.jts.geom.Polygon; import com.vividsolutions.jump.I18N; import com.vividsolutions.jump.feature.Feature; -import com.vividsolutions.jump.feature.FeatureCollection; import com.vividsolutions.jump.workbench.WorkbenchContext; import com.vividsolutions.jump.workbench.model.Layer; import com.vividsolutions.jump.workbench.model.MeasureLayerFinder; @@ -47,7 +44,7 @@ LayerViewPanel layerViewPanel = context.getWorkbenchContext() .getLayerViewPanel(); WorkbenchContext wbc = context.getWorkbenchContext(); - Collection layers = ((SelectionManagerProxy) wbc.getWorkbench() + Collection<Layer> layers = ((SelectionManagerProxy) wbc.getWorkbench() .getFrame().getActiveInternalFrame()).getSelectionManager() .getFeatureSelection().getLayersWithSelectedItems(); // Giuseppe Aruta 2015-6-25 @@ -56,30 +53,21 @@ // multigeometries (multiPolygon, multiLinestring, multipoint and // geometry collections) // and for points (measure coordinates) - for (Iterator li = layers.iterator(); li.hasNext();) { - Layer layer = (Layer) li.next(); + for (Layer layer : layers) { - FeatureCollection featureCollection = layer - .getFeatureCollectionWrapper(); - SelectionManager manager = context.getLayerViewPanel() - .getSelectionManager(); - // Collection feats = getFeatures(layer,context); - Collection feats = manager.createFeaturesFromSelectedItems(layer); + SelectionManager manager = context.getLayerViewPanel().getSelectionManager(); - for (Iterator i = feats.iterator(); i.hasNext();) { + for (Feature feature : manager.createFeaturesFromSelectedItems(layer)) { try { - Feature feat = (Feature) i.next(); layerViewPanel.setViewportInitialized(true); - Geometry geom = feat.getGeometry(); + Geometry geom = feature.getGeometry(); if (geom instanceof Polygon || geom instanceof LineString) { measure(wbc, geom); } - else { // Giuseppe Aruta 2015-6-25 // Set here the code for multiplegeometries and point - } } catch (IllegalArgumentException e) { context.getWorkbenchFrame().warnUser(e.toString()); @@ -115,17 +103,14 @@ mec.add(new EnableCheck() { public String check(JComponent component) { - Feature feat = - - (Feature) context.getLayerViewPanel().getSelectionManager() + Feature feature = context.getLayerViewPanel().getSelectionManager() .getFeaturesWithSelectedItems().iterator().next(); - Geometry geom = feat.getGeometry(); + Geometry geom = feature.getGeometry(); return geom instanceof GeometryCollection - || geom instanceof MultiLineString || geom instanceof Point ? geom.getGeometryType() + " - " - + I18N.get("org.openjump.core.ui.plugin.mousemenu.MeasureSelectedFeaturePlugin.message1") + + I18N.get("org.openjump.core.ui.plugin.mousemenu.MeasureSelectedFeaturePlugIn.message1") : null; } }); ------------------------------------------------------------------------------ Developer Access Program for Intel Xeon Phi Processors Access to Intel Xeon Phi processor-based developer platforms. With one year of Intel Parallel Studio XE. Training and support from Colfax. Order your platform today. http://sdm.link/xeonphi _______________________________________________ Jump-pilot-devel mailing list Jump-pilot-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel