Revision: 4768 http://sourceforge.net/p/jump-pilot/code/4768 Author: edso Date: 2016-01-10 18:47:43 +0000 (Sun, 10 Jan 2016) Log Message: ----------- adapt to new Logger
Modified Paths: -------------- core/trunk/src/de/latlon/deejump/wfs/client/WFSClientHelper.java core/trunk/src/org/deegree/datatypes/QualifiedName.java core/trunk/src/org/deegree/model/spatialschema/GMLGeometryAdapter.java core/trunk/src/org/deegree/model/spatialschema/JTSAdapter.java core/trunk/src/org/openide/awt/DropDownButton.java core/trunk/src/org/openide/awt/DropDownToggleButton.java core/trunk/src/org/openjump/core/rasterimage/RasterImageLayer.java core/trunk/src/org/openjump/core/rasterimage/styler/ui/CustomGradientColorsDialog.java core/trunk/src/org/openjump/core/rasterimage/styler/ui/GradientTablePanel.java core/trunk/src/org/openjump/core/rasterimage/styler/ui/IntervalPanel.java core/trunk/src/org/openjump/core/rasterimage/styler/ui/RasterStylesDialog.java core/trunk/src/org/openjump/core/rasterimage/styler/ui/SingleValuesPanel.java core/trunk/src/org/openjump/core/ui/plugin/customize/PythonToolsPlugIn.java core/trunk/src/org/openjump/core/ui/plugin/datastore/AddWritableDataStoreLayerWizard.java core/trunk/src/org/openjump/core/ui/plugin/datastore/WritableDataStoreDataSource.java core/trunk/src/org/openjump/core/ui/plugin/datastore/postgis2/PostGISDataStoreDataSource.java core/trunk/src/org/openjump/core/ui/plugin/datastore/transaction/DataStoreTransactionManager.java core/trunk/src/org/openjump/core/ui/plugin/datastore/transaction/DataStoreTransactionManagerPlugIn.java core/trunk/src/org/openjump/core/ui/plugin/datastore/transaction/Evolution.java core/trunk/src/org/openjump/core/ui/plugin/datastore/transaction/TransactionManagerPanel.java core/trunk/src/org/openjump/core/ui/plugin/mousemenu/category/SetCategoryVisibilityPlugIn.java core/trunk/src/org/openjump/core/ui/plugin/raster/RasterImageLayerPropertiesPlugIn.java core/trunk/src/org/openjump/core/ui/plugin/raster/nodata/ChangeRangeValuesToNoDataPlugIn.java core/trunk/src/org/openjump/core/ui/plugin/style/ImportSLDPlugIn.java core/trunk/src/org/openjump/core/ui/plugin/view/SuperZoomPanTool.java core/trunk/src/org/openjump/core/ui/plugin/wms/ZoomToWMSLayerPlugIn.java core/trunk/src/org/openjump/util/CustomTexturePaint.java core/trunk/src/org/openjump/util/SLDImporter.java core/trunk/src/org/openjump/util/XPathUtils.java core/trunk/src/org/saig/core/crypt/BlowFishManager.java core/trunk/src/org/saig/core/crypt/PBEManager.java core/trunk/src/org/saig/jump/lang/I18N.java core/trunk/src/org/saig/jump/widgets/config/HTTPProxySettings.java core/trunk/src/org/saig/jump/widgets/util/AbstractWaitDialog.java Modified: core/trunk/src/de/latlon/deejump/wfs/client/WFSClientHelper.java =================================================================== --- core/trunk/src/de/latlon/deejump/wfs/client/WFSClientHelper.java 2016-01-10 18:45:58 UTC (rev 4767) +++ core/trunk/src/de/latlon/deejump/wfs/client/WFSClientHelper.java 2016-01-10 18:47:43 UTC (rev 4768) @@ -58,6 +58,8 @@ import org.deegree.framework.log.LoggerFactory; import org.deegree.framework.util.CharsetUtils; +import com.vividsolutions.jump.workbench.Logger; + import de.latlon.deejump.wfs.DeeJUMPException; /** @@ -71,8 +73,6 @@ */ public class WFSClientHelper { - private static ILogger LOG = LoggerFactory.getLogger(WFSClientHelper.class); - /** * convenience method, big datasets tend to flood your memory * @@ -108,7 +108,7 @@ */ public static InputStream createResponseStreamfromWFS(String serverUrl, String postData) throws DeeJUMPException { - LOG.logDebug("WFS GetFeature: " + serverUrl + " -> " + postData); + Logger.debug("WFS GetFeature: " + serverUrl + " -> " + postData); //System.out.println(serverUrl); HttpClient httpclient = new WFSHttpClient(); Modified: core/trunk/src/org/deegree/datatypes/QualifiedName.java =================================================================== --- core/trunk/src/org/deegree/datatypes/QualifiedName.java 2016-01-10 18:45:58 UTC (rev 4767) +++ core/trunk/src/org/deegree/datatypes/QualifiedName.java 2016-01-10 18:47:43 UTC (rev 4768) @@ -48,6 +48,8 @@ import org.deegree.framework.log.LoggerFactory; import org.deegree.framework.util.StringTools; +import com.vividsolutions.jump.workbench.Logger; + /** * This class represent a qualified name for something. A name is thought to be built from an optional prefix and/or a * local name E.g.: <BR>- deegree - pre:deegree <BR> @@ -63,8 +65,6 @@ private static final long serialVersionUID = 5551137348397905772L; - private static final ILogger LOG = LoggerFactory.getLogger( QualifiedName.class ); - private String prefix = null; private String localName = null; @@ -93,7 +93,7 @@ this.namespace = new URI( name.getNamespaceURI() ); buildString(); } catch ( URISyntaxException e ) { - LOG.logError( "Invalid URI: " + e.getMessage(), e ); + Logger.error( "Invalid URI: " + e.getMessage(), e ); } } Modified: core/trunk/src/org/deegree/model/spatialschema/GMLGeometryAdapter.java =================================================================== --- core/trunk/src/org/deegree/model/spatialschema/GMLGeometryAdapter.java 2016-01-10 18:45:58 UTC (rev 4767) +++ core/trunk/src/org/deegree/model/spatialschema/GMLGeometryAdapter.java 2016-01-10 18:47:43 UTC (rev 4768) @@ -57,7 +57,6 @@ import org.apache.commons.lang.StringUtils; import org.deegree.framework.log.ILogger; -import org.deegree.framework.log.LoggerFactory; import org.deegree.framework.util.StringTools; import org.deegree.framework.xml.ElementList; import org.deegree.framework.xml.NamespaceContext; @@ -74,6 +73,8 @@ import org.w3c.dom.Element; import org.w3c.dom.Node; +import com.vividsolutions.jump.workbench.Logger; + /** * Adapter class for converting GML geometries to deegree geometries and vice versa. Some logical * problems results from the fact that an envelope isn't a geometry according to ISO 19107 (where @@ -94,8 +95,6 @@ */ public class GMLGeometryAdapter { - protected static final ILogger LOG = LoggerFactory.getLogger(GMLGeometryAdapter.class); - protected static final NamespaceContext nsContext = CommonNamespaces.getNamespaceContext(); protected static Map<String, CoordinateSystem> crsMap = new HashMap<String, CoordinateSystem>(); @@ -135,8 +134,8 @@ try { doc = XMLTools.parse(sr); } catch (Exception e) { - LOG.logError(I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.could-not-parse") + ": '" + gml + "' " + I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.as-gml-xml"), e); //$NON-NLS-1$ //$NON-NLS-2$ //$NON-NLS-3$ //$NON-NLS-4$ - throw new XMLParsingException(I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.could-not-parse") + ": '" + gml + "' " + I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.as-gml-xml") + e.getMessage()); //$NON-NLS-1$ //$NON-NLS-2$ //$NON-NLS-3$ //$NON-NLS-4$ + Logger.error(I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.could-not-parse") + ": '" + gml + "' " + I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.as-gml-xml"), e); + throw new XMLParsingException(I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.could-not-parse") + ": '" + gml + "' " + I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.as-gml-xml") + e.getMessage()); } return wrap(doc.getDocumentElement(), srsName); } @@ -172,35 +171,35 @@ Geometry geometry = null; try { String name = element.getLocalName(); - if ((name.equals("Point")) || (name.equals("Center"))) { //$NON-NLS-1$ //$NON-NLS-2$ + if ((name.equals("Point")) || (name.equals("Center"))) { geometry = wrapPoint(element, srsName); - } else if (name.equals("LineString")) { //$NON-NLS-1$ + } else if (name.equals("LineString")) { geometry = wrapLineString(element, srsName); - } else if (name.equals("Polygon")) { //$NON-NLS-1$ + } else if (name.equals("Polygon")) { geometry = wrapPolygon(element, srsName); - } else if (name.equals("MultiPoint")) { //$NON-NLS-1$ + } else if (name.equals("MultiPoint")) { geometry = wrapMultiPoint(element, srsName); - } else if (name.equals("MultiLineString")) { //$NON-NLS-1$ + } else if (name.equals("MultiLineString")) { geometry = wrapMultiLineString(element, srsName); - } else if (name.equals("MultiPolygon")) { //$NON-NLS-1$ + } else if (name.equals("MultiPolygon")) { geometry = wrapMultiPolygon(element, srsName); - } else if (name.equals("Box") || name.equals("Envelope")) { //$NON-NLS-1$ //$NON-NLS-2$ + } else if (name.equals("Box") || name.equals("Envelope")) { geometry = wrapBoxAsSurface(element, srsName); - } else if (name.equals("Curve")) { //$NON-NLS-1$ + } else if (name.equals("Curve")) { geometry = wrapCurveAsCurve(element, srsName); - } else if (name.equals("Surface")) { //$NON-NLS-1$ + } else if (name.equals("Surface")) { geometry = wrapSurfaceAsSurface(element, srsName); - } else if (name.equals("MultiCurve")) { //$NON-NLS-1$ + } else if (name.equals("MultiCurve")) { geometry = wrapMultiCurveAsMultiCurve(element, srsName); - } else if (name.equals("MultiSurface")) { //$NON-NLS-1$ + } else if (name.equals("MultiSurface")) { geometry = wrapMultiSurfaceAsMultiSurface(element, srsName); - } else if (name.equals("CompositeSurface")) { //$NON-NLS-1$ + } else if (name.equals("CompositeSurface")) { geometry = wrapCompositeSurface(element, srsName); } else { - new GeometryException(I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.not-a-supported-geometry-type") + ": " + name); //$NON-NLS-1$ //$NON-NLS-2$ + new GeometryException(I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.not-a-supported-geometry-type") + ": " + name); } } catch (Exception e) { - LOG.logError(e.getMessage(), e); + Logger.error(e.getMessage(), e); throw new GeometryException(StringTools.stackTraceToString(e)); } return geometry; @@ -297,12 +296,12 @@ CurveSegment[] segments = new CurveSegment[list.size()]; for( int i = 0; i < list.size(); i++ ) { - if (list.get(i).getLocalName().equals("LineStringSegment")) { //$NON-NLS-1$ + if (list.get(i).getLocalName().equals("LineStringSegment")) { segments[i] = parseLineStringSegment((Element) list.get(i), crs, srsDimension); - } else if (list.get(i).getLocalName().equals("Arc")) { //$NON-NLS-1$ + } else if (list.get(i).getLocalName().equals("Arc")) { segments[i] = parseArc((Element) list.get(i), crs, srsDimension); } else { - throw new GeometryException(I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.not-supported-type-for-a-curvesegment") + ": " + list.get(i).getLocalName()); //$NON-NLS-1$ //$NON-NLS-2$ + throw new GeometryException(I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.not-supported-type-for-a-curvesegment") + ": " + list.get(i).getLocalName()); } } return segments; @@ -324,7 +323,7 @@ Position[] pos = createPositions(element, null, srsDimension); segment = GeometryFactory.createCurveSegment(pos, crs); } catch (Exception e) { - throw new GeometryException(I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.error-creating-segments-for-the-element-arc") + "."); //$NON-NLS-1$ //$NON-NLS-2$ + throw new GeometryException(I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.error-creating-segments-for-the-element-arc") + "."); } return segment; } @@ -345,7 +344,7 @@ Position[] pos = createPositions(element, null, srsDimension); segment = GeometryFactory.createCurveSegment(pos, crs); } catch (Exception e) { - throw new GeometryException(I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.error-creating-segments-for-the-element-linestringsegment") + "."); //$NON-NLS-1$ //$NON-NLS-2$ + throw new GeometryException(I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.error-creating-segments-for-the-element-linestringsegment") + "."); } return segment; @@ -414,11 +413,11 @@ Curve[] curves = new Curve[curveList.size()]; multiCurve = GeometryFactory.createMultiCurve(curveList.toArray(curves), crs); } catch (XMLParsingException e) { - LOG.logError(e.getMessage(), e); - throw new XMLParsingException(I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.error-parsing") + "<gml:curveMember> " + I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.elements") + ". " + I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.please-check-the-xml-document") + "."); //$NON-NLS-1$ //$NON-NLS-2$ //$NON-NLS-3$ //$NON-NLS-4$ //$NON-NLS-5$ //$NON-NLS-6$ + Logger.error(e.getMessage(), e); + throw new XMLParsingException(I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.error-parsing") + "<gml:curveMember> " + I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.elements") + ". " + I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.please-check-the-xml-document") + "."); } catch (GeometryException e) { - LOG.logError(e.getMessage(), e); - throw new GeometryException(I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.error-creating-a-curve-from-the-curve-element") + ". " + I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.please-check-the-gml-specifications-for-correct-element-declaration") + "."); //$NON-NLS-1$ //$NON-NLS-2$ //$NON-NLS-3$ //$NON-NLS-4$ + Logger.error(e.getMessage(), e); + throw new GeometryException(I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.error-creating-a-curve-from-the-curve-element") + ". " + I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.please-check-the-gml-specifications-for-correct-element-declaration") + "."); } return multiCurve; } @@ -464,7 +463,7 @@ } List<Element> interiorList = - XMLTools.getElements(polygon, "gml:interior | gml:outerBounderyIs", //$NON-NLS-1$ + XMLTools.getElements(polygon, "gml:interior | gml:outerBounderyIs", nsContext); Curve[] interiorRings = null; if (interiorList != null && interiorList.size() > 0) { @@ -479,9 +478,9 @@ surfacePatches[i] = GeometryFactory.createSurfacePatch(exteriorRing, interiorRings, crs); } catch (InvalidGMLException e) { - LOG.logError(e.getMessage(), e); - throw new XMLParsingException(I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.error-parsing-the-polygon-element") + " '" + polygon.getNodeName() //$NON-NLS-1$ //$NON-NLS-2$ - + "' " + I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.to-create-a-surface-geometry") + "."); //$NON-NLS-1$ //$NON-NLS-2$ //$NON-NLS-3$ + Logger.error(e.getMessage(), e); + throw new XMLParsingException(I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.error-parsing-the-polygon-element") + " '" + polygon.getNodeName() + + "' " + I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.to-create-a-surface-geometry") + "."); } } @@ -489,7 +488,7 @@ try { surface = GeometryFactory.createSurface(surfacePatches, crs); } catch (GeometryException e) { - throw new GeometryException(I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.error-creating-a-surface-from") + " '" + surfacePatches.length + "' " + "polygons" + "."); //$NON-NLS-1$ //$NON-NLS-2$ //$NON-NLS-3$ //$NON-NLS-4$ //$NON-NLS-5$ + throw new GeometryException(I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.error-creating-a-surface-from") + " '" + surfacePatches.length + "' " + "polygons" + "."); } return surface; } @@ -615,12 +614,12 @@ surfaces = surfaceList.toArray(surfaces); multiSurface = GeometryFactory.createMultiSurface(surfaces, crs); } catch (XMLParsingException e) { - LOG.logError(e.getMessage(), e); + Logger.error(e.getMessage(), e); String msg = "Error parsing <gml:surfaceMember> elements. Please check the xml document."; throw new XMLParsingException(msg); } catch (GeometryException e) { - LOG.logError(e.getMessage(), e); + Logger.error(e.getMessage(), e); String msg = "Error creating a multi surface from the MultiSurface element. Please check the GML specifications for correct element declaration."; throw new GeometryException(msg); @@ -743,7 +742,7 @@ nl = XMLTools.getRequiredNodes(inns.get(i), CommonNamespaces.GML_PREFIX - + ":LinearRing", //$NON-NLS-1$ + + ":LinearRing", nsContext); ring = (Element) nl.get(0); @@ -881,7 +880,7 @@ * @throws GeometryException */ private static CompositeSurface wrapCompositeSurface( Element element, String srsName ) { - throw new UnsupportedOperationException("#wrapCompositeSurface(Element) " + I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.is-not-implemented-as-yet") + ". " + I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.work-in-progress") + "."); //$NON-NLS-1$ //$NON-NLS-2$ //$NON-NLS-3$ //$NON-NLS-4$ //$NON-NLS-5$ + throw new UnsupportedOperationException("#wrapCompositeSurface(Element) " + I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.is-not-implemented-as-yet") + ". " + I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.work-in-progress") + "."); } /** @@ -896,7 +895,7 @@ try { patches = (Element) XMLTools.getRequiredNode(surface, "gml:patches", nsContext); } catch (XMLParsingException e) { - throw new XMLParsingException(I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.error-retrieving-the-patches-element-from-the-surface-element") + "."); //$NON-NLS-1$ //$NON-NLS-2$ + throw new XMLParsingException(I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.error-retrieving-the-patches-element-from-the-surface-element") + "."); } return patches; } @@ -911,7 +910,7 @@ public static CoordinateSystem getCRS( String name ) throws UnknownCRSException { if ((name != null) && (name.length() > 2)) { - if (name.startsWith("http://www.opengis.net/gml/srs/")) { //$NON-NLS-1$ + if (name.startsWith("http://www.opengis.net/gml/srs/")) { // as declared in the GML 2.1.1 specification // http://www.opengis.net/gml/srs/epsg.xml#4326 int p = name.lastIndexOf("/"); @@ -976,17 +975,17 @@ // urn:ogc:def:${operationName}:EPSG::${epsgCode} // urn:ogc:def:${operationName}:EPSG:${epsgCode} result = EPSG_SINGLE + StringUtils.substringAfterLast(name, ":"); - } else if (StringUtils.containsIgnoreCase(name, "epsg.xml")) { //$NON-NLS-1$ + } else if (StringUtils.containsIgnoreCase(name, "epsg.xml")) { // http://www.opengis.net/gml/srs/epsg.xml#<EPSG code> result = EPSG_SINGLE + StringUtils.substringAfterLast(name, "#"); - } else if (StringUtils.containsIgnoreCase(name, "urn:epsg")) { //$NON-NLS-1$ + } else if (StringUtils.containsIgnoreCase(name, "urn:epsg")) { // urn:EPSG:geographicCRC:<epsg code> result = EPSG_SINGLE + StringUtils.substringAfterLast(name, ":"); } else { result = name; } - LOG.logDebug("Transformed from " + name + " to " + result); //$NON-NLS-1$ //$NON-NLS-2$ + Logger.debug("Transformed from " + name + " to " + result); return result; } @@ -1001,7 +1000,7 @@ double[] vals = StringTools.toArrayDouble(tmp, ", "); if (dim != 0) { if (vals.length != dim) { - throw new InvalidGMLException(I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.dimension-must-be-equal-to-the-number-of-coordinate-values-defined-in-pos-element") + "."); //$NON-NLS-1$ //$NON-NLS-2$ + throw new InvalidGMLException(I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.dimension-must-be-equal-to-the-number-of-coordinate-values-defined-in-pos-element") + "."); } } else { dim = vals.length; @@ -1055,11 +1054,11 @@ Position[] points = null; // fixing the failure coming from the usage of the xmltools.getAttrib method - String ts = XMLTools.getAttrValue(element, null, "ts", " "); //$NON-NLS-1$ //$NON-NLS-2$ + String ts = XMLTools.getAttrValue(element, null, "ts", " "); // not used because javas current decimal seperator will be used // String ds = XMLTools.getAttrValue( element, null, "decimal", "." ); - String cs = XMLTools.getAttrValue(element, null, "cs", ","); //$NON-NLS-1$ //$NON-NLS-2$ + String cs = XMLTools.getAttrValue(element, null, "cs", ","); String value = XMLTools.getStringValue(element).trim(); @@ -1104,19 +1103,19 @@ if (vals != null) { if (dim != 0) { if (vals.length != dim) { - throw new InvalidGMLException(I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.dimension-must-be-equal-to-the-number-of-coordinate-values-defined-in-pos-element") + "."); //$NON-NLS-1$ //$NON-NLS-2$ + throw new InvalidGMLException(I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.dimension-must-be-equal-to-the-number-of-coordinate-values-defined-in-pos-element") + "."); } } else { dim = vals.length; } } else { - throw new InvalidGMLException(I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.the-given-element") //$NON-NLS-1$ - + "{" //$NON-NLS-1$ + throw new InvalidGMLException(I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.the-given-element") + + "{" + element.getNamespaceURI() - + "}" //$NON-NLS-1$ + + "}" + element.getLocalName() - + I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.does-not-contain-any-coordinates") //$NON-NLS-1$ - + ". " //$NON-NLS-1$ + + I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.does-not-contain-any-coordinates") + + ". " + I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.this-may-not-be")); } @@ -1149,9 +1148,9 @@ srsDimension = findSrsDimension(element); } - if (LOG.getLevel() == ILogger.LOG_DEBUG) { + if (Logger.isDebugEnabled()) { XMLFragment doc = new XMLFragment(element); - System.out.println(doc.getAsPrettyString()); + Logger.debug(doc.getAsPrettyString()); } int dim = 0; if (srsDimension != null) { @@ -1171,32 +1170,32 @@ if (srsName == null) { if (srsDimension != null) { - throw new InvalidGMLException(I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.attribute-srsdimension-cannot-be-defined-unless-attribute-srsname-has-been-defined") + "."); //$NON-NLS-1$ //$NON-NLS-2$ + throw new InvalidGMLException(I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.attribute-srsdimension-cannot-be-defined-unless-attribute-srsname-has-been-defined") + "."); } if (axisLabels != null) { - throw new InvalidGMLException(I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.attribute-axislabels-cannot-be-defined-unless-attribute-srsname-has-been-defined") + "."); //$NON-NLS-1$ //$NON-NLS-2$ + throw new InvalidGMLException(I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.attribute-axislabels-cannot-be-defined-unless-attribute-srsname-has-been-defined") + "."); } } if (axisLabels == null) { if (uomLabels != null) { - throw new InvalidGMLException(I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.attribute-uomlabels-cannot-be-defined-unless-attribute-axisLabels-has-been-defined") + "."); //$NON-NLS-1$ //$NON-NLS-2$ + throw new InvalidGMLException(I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.attribute-uomlabels-cannot-be-defined-unless-attribute-axisLabels-has-been-defined") + "."); } } String tmp = XMLTools.getStringValue(element); double[] values = StringTools.toArrayDouble(tmp, "\t\n\r\f ,"); int size = values.length / dim; - LOG.logDebug(I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.number-of-points") + " = ", size); //$NON-NLS-1$ //$NON-NLS-2$ - LOG.logDebug(I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.size-of-the-original-array") + ": ", values.length); //$NON-NLS-1$ //$NON-NLS-2$ - LOG.logDebug(I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.dimension") + ": ", dim); //$NON-NLS-1$ //$NON-NLS-2$ + Logger.debug(I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.number-of-points") + " = "+ size); + Logger.debug(I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.size-of-the-original-array") + ": "+ values.length); + Logger.debug(I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.dimension") + ": "+ dim); if (values.length < 4) { - throw new InvalidGMLException(I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.a-point-list-must-have-minimum-two-coordinate-tuples") //$NON-NLS-1$ - + ". " //$NON-NLS-1$ - + I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.here-only") //$NON-NLS-1$ - + " '" //$NON-NLS-1$ - + size + "' " //$NON-NLS-1$ - + I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.are-defined") //$NON-NLS-1$ + throw new InvalidGMLException(I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.a-point-list-must-have-minimum-two-coordinate-tuples") + + ". " + + I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.here-only") + + " '" + + size + "' " + + I18N.getString("org.deegree.model.spatialschema.GMLGeometryAdapter.are-defined") + "."); } double positions[][] = new double[size][dim]; @@ -1378,7 +1377,7 @@ String srs = null; if (crs != null) { - srs = "<gml:Envelope srsName=\"" + crs + "\""; //$NON-NLS-1$ //$NON-NLS-2$ + srs = "<gml:Envelope srsName=\"" + crs + "\""; } else { srs = "<gml:Envelope"; } @@ -1452,14 +1451,14 @@ } String srs = null; if (crs != null) { - srs = "<gml:Point srsName=\"" + crs + "\">"; //$NON-NLS-1$ //$NON-NLS-2$ + srs = "<gml:Point srsName=\"" + crs + "\">"; } else { srs = "<gml:Point>"; } pw.println(srs); if (dim != 0) { - String dimension = "<gml:pos srsDimension=\"" + dim + "\">"; //$NON-NLS-1$ //$NON-NLS-2$ + String dimension = "<gml:pos srsDimension=\"" + dim + "\">"; pw.print(dimension); } else { pw.print("<gml:pos>"); @@ -1490,7 +1489,7 @@ } String srs = null; if (crs != null) { - srs = "<gml:Curve srsName=\"" + crs + "\">"; //$NON-NLS-1$ //$NON-NLS-2$ + srs = "<gml:Curve srsName=\"" + crs + "\">"; } else { srs = "<gml:Curve>"; } @@ -1554,7 +1553,7 @@ String srs = null; if (crs != null) { - srs = "<gml:Surface srsName='" + crs + "'>"; //$NON-NLS-1$ //$NON-NLS-2$ + srs = "<gml:Surface srsName='" + crs + "'>"; } else { srs = "<gml:Surface>"; } @@ -1618,7 +1617,7 @@ String formatType ) { // exterior ring if (formatType.contains(GML2_FORMAT_TYPE)) { - pw.print("<gml:outerBoundaryIs>"); //$NON-NLS-1$ + pw.print("<gml:outerBoundaryIs>"); } else { pw.print("<gml:exterior>"); } @@ -1634,7 +1633,7 @@ } pw.print("</gml:LinearRing>"); if (formatType.contains(GML2_FORMAT_TYPE)) { - pw.print("</gml:outerBoundaryIs>"); //$NON-NLS-1$ + pw.print("</gml:outerBoundaryIs>"); } else { pw.print("</gml:exterior>"); } @@ -1678,20 +1677,20 @@ */ private static void printPositions( PrintWriter pw, Position[] p, int coordinateDimension, String formatType ) { - String startTag = formatType.equals(GML2_FORMAT_TYPE) ? "<gml:coordinates" : "<gml:posList"; //$NON-NLS-1$ //$NON-NLS-2$ + String startTag = formatType.equals(GML2_FORMAT_TYPE) ? "<gml:coordinates" : "<gml:posList"; String endTag = - formatType.equals(GML2_FORMAT_TYPE) ? "</gml:coordinates>" : "</gml:posList>"; //$NON-NLS-1$ //$NON-NLS-2$ + formatType.equals(GML2_FORMAT_TYPE) ? "</gml:coordinates>" : "</gml:posList>"; StringBuilder posList = new StringBuilder(startTag); - String coordSeparator = formatType.equals(GML2_FORMAT_TYPE) ? "," : " "; //$NON-NLS-1$ //$NON-NLS-2$ + String coordSeparator = formatType.equals(GML2_FORMAT_TYPE) ? "," : " "; if (formatType.equals(GML2_FORMAT_TYPE)) { posList.append(" decimal=\".\" cs=\",\" ts=\" \""); } else { if (coordinateDimension > 0) { - posList.append(" srsDimension='").append(coordinateDimension).append("'"); //$NON-NLS-1$ //$NON-NLS-2$ + posList.append(" srsDimension='").append(coordinateDimension).append("'"); } - posList.append(" count='").append(p.length).append("'"); //$NON-NLS-1$ //$NON-NLS-2$ + posList.append(" count='").append(p.length).append("'"); } posList.append(">"); @@ -1700,7 +1699,7 @@ for( int j = 0; j < (p.length - 1); j++ ) { pw.print(p[j].getX() + coordSeparator + p[j].getY()); if (coordinateDimension == 3) { - pw.print(coordSeparator + p[j].getZ() + " "); //$NON-NLS-1$ //$NON-NLS-2$ + pw.print(coordSeparator + p[j].getZ() + " "); } else { pw.print(' '); } @@ -1725,7 +1724,7 @@ } String srs = null; if (crs != null) { - srs = "<gml:MultiPoint srsName=\"" + crs + "\">"; //$NON-NLS-1$ //$NON-NLS-2$ + srs = "<gml:MultiPoint srsName=\"" + crs + "\">"; } else { srs = "<gml:MultiPoint>"; } @@ -1762,7 +1761,7 @@ } String srs = null; if (crs != null) { - srs = "<gml:MultiCurve srsName=\"" + crs + "\">"; //$NON-NLS-1$ //$NON-NLS-2$ + srs = "<gml:MultiCurve srsName=\"" + crs + "\">"; } else { srs = "<gml:MultiCurve>"; } @@ -1783,7 +1782,7 @@ for( int k = 0; k < (p.length - 1); k++ ) { pw.print(p[k].getX() + " " + p[k].getY()); if (curve.getCoordinateDimension() == 3) { - pw.print(" " + p[k].getZ() + " "); //$NON-NLS-1$ //$NON-NLS-2$ + pw.print(" " + p[k].getZ() + " "); } else { pw.print(" "); } @@ -1818,7 +1817,7 @@ } String srs = null; if (crs != null) { - srs = "<gml:MultiSurface srsName=\"" + crs + "\">"; //$NON-NLS-1$ //$NON-NLS-2$ + srs = "<gml:MultiSurface srsName=\"" + crs + "\">"; } else { srs = "<gml:MultiSurface>"; } Modified: core/trunk/src/org/deegree/model/spatialschema/JTSAdapter.java =================================================================== --- core/trunk/src/org/deegree/model/spatialschema/JTSAdapter.java 2016-01-10 18:45:58 UTC (rev 4767) +++ core/trunk/src/org/deegree/model/spatialschema/JTSAdapter.java 2016-01-10 18:47:43 UTC (rev 4768) @@ -42,11 +42,10 @@ ---------------------------------------------------------------------------*/ package org.deegree.model.spatialschema; -import org.deegree.framework.log.ILogger; -import org.deegree.framework.log.LoggerFactory; import org.deegree.model.crs.CoordinateSystem; import com.vividsolutions.jts.geom.PrecisionModel; +import com.vividsolutions.jump.workbench.Logger; /** * Adapter between deegree-<tt>Geometry</tt>s and JTS-<tt>Geometry<tt> objects. @@ -62,8 +61,6 @@ */ public class JTSAdapter { - private static final ILogger LOG = LoggerFactory.getLogger(JTSAdapter.class); - // precision model that is used for all JTS-Geometries private static PrecisionModel pm = new PrecisionModel(); @@ -270,7 +267,7 @@ try { patch = surface.getSurfacePatchAt(0); } catch (GeometryException e) { - LOG.logError("", e); + Logger.error(e); } Position[] exteriorRing = patch.getExteriorRing(); Position[][] interiorRings = patch.getInteriorRings(); Modified: core/trunk/src/org/openide/awt/DropDownButton.java =================================================================== --- core/trunk/src/org/openide/awt/DropDownButton.java 2016-01-10 18:45:58 UTC (rev 4767) +++ core/trunk/src/org/openide/awt/DropDownButton.java 2016-01-10 18:47:43 UTC (rev 4768) @@ -52,8 +52,7 @@ import java.beans.PropertyChangeListener; import java.util.HashMap; import java.util.Map; -import java.util.logging.Level; -import java.util.logging.Logger; + import javax.swing.DefaultButtonModel; import javax.swing.Icon; import javax.swing.JButton; @@ -61,6 +60,8 @@ import javax.swing.event.PopupMenuEvent; import javax.swing.event.PopupMenuListener; +import com.vividsolutions.jump.workbench.Logger; + /** * JButton with a small arrow that displays popup menu when clicked. * @@ -345,7 +346,7 @@ @Override public void setText( String text ) { //does nothing - Logger.getLogger(DropDownToggleButton.class.getName()).log(Level.FINER, "DropDownButton cannot display text."); //NOI18N + Logger.debug("DropDownButton cannot display text."); } @Override Modified: core/trunk/src/org/openide/awt/DropDownToggleButton.java =================================================================== --- core/trunk/src/org/openide/awt/DropDownToggleButton.java 2016-01-10 18:45:58 UTC (rev 4767) +++ core/trunk/src/org/openide/awt/DropDownToggleButton.java 2016-01-10 18:47:43 UTC (rev 4768) @@ -52,14 +52,15 @@ import java.beans.PropertyChangeListener; import java.util.HashMap; import java.util.Map; -import java.util.logging.Level; -import java.util.logging.Logger; + import javax.swing.Icon; import javax.swing.JPopupMenu; import javax.swing.JToggleButton; import javax.swing.event.PopupMenuEvent; import javax.swing.event.PopupMenuListener; +import com.vividsolutions.jump.workbench.Logger; + /** * JToggleButton with a small arrow that displays popup menu when clicked. * @@ -341,7 +342,7 @@ @Override public void setText( String text ) { //does nothing - Logger.getLogger(DropDownToggleButton.class.getName()).log(Level.FINER, "DropDownToggleButton cannot display text."); //NOI18N + Logger.debug("DropDownToggleButton cannot display text."); } @Override Modified: core/trunk/src/org/openjump/core/rasterimage/RasterImageLayer.java =================================================================== --- core/trunk/src/org/openjump/core/rasterimage/RasterImageLayer.java 2016-01-10 18:45:58 UTC (rev 4767) +++ core/trunk/src/org/openjump/core/rasterimage/RasterImageLayer.java 2016-01-10 18:47:43 UTC (rev 4768) @@ -9,9 +9,34 @@ */ package org.openjump.core.rasterimage; -import com.vividsolutions.jts.geom.*; +import java.awt.Color; +import java.awt.Graphics2D; +import java.awt.Image; +import java.awt.Point; +import java.awt.Rectangle; +import java.awt.geom.NoninvertibleTransformException; +import java.awt.geom.Point2D; +import java.awt.image.BufferedImage; +import java.awt.image.ColorModel; +import java.awt.image.IndexColorModel; +import java.awt.image.Raster; +import java.awt.image.renderable.ParameterBlock; +import java.io.FileNotFoundException; +import java.io.IOException; +import java.util.UUID; + +import javax.media.jai.JAI; + +import org.openjump.util.metaData.MetaDataMap; +import org.openjump.util.metaData.ObjectContainingMetaInformation; + +import com.vividsolutions.jts.geom.Coordinate; +import com.vividsolutions.jts.geom.Envelope; +import com.vividsolutions.jts.geom.Geometry; +import com.vividsolutions.jts.geom.GeometryFactory; import com.vividsolutions.jts.geom.Polygon; import com.vividsolutions.jump.util.Blackboard; +import com.vividsolutions.jump.workbench.Logger; import com.vividsolutions.jump.workbench.WorkbenchContext; import com.vividsolutions.jump.workbench.model.AbstractLayerable; import com.vividsolutions.jump.workbench.model.LayerManager; @@ -20,22 +45,7 @@ import com.vividsolutions.jump.workbench.ui.LayerNameRenderer; import com.vividsolutions.jump.workbench.ui.LayerViewPanel; import com.vividsolutions.jump.workbench.ui.Viewport; -import org.openjump.util.metaData.MetaDataMap; -import org.openjump.util.metaData.ObjectContainingMetaInformation; -import javax.media.jai.JAI; -import java.awt.*; -import java.awt.Point; -import java.awt.geom.NoninvertibleTransformException; -import java.awt.geom.Point2D; -import java.awt.image.*; -import java.awt.image.renderable.ParameterBlock; -import java.io.FileNotFoundException; -import java.io.IOException; -import java.util.UUID; -import java.util.logging.Level; -import java.util.logging.Logger; - /** * Layer representing a georeferenced raster image (e.g. an areal photography) in OpenJump. * @@ -256,21 +266,21 @@ try { raster = new RasterImageLayer(getName(), getLayerManager(), getImageForDisplay(), getRasterData(null), new Envelope(getWholeImageEnvelope())); } catch (IOException ex) { - Logger.getLogger(RasterImageLayer.class.getName()).log(Level.SEVERE, null, ex); + Logger.error(ex); } catch (NoninvertibleTransformException ex) { - Logger.getLogger(RasterImageLayer.class.getName()).log(Level.SEVERE, null, ex); + Logger.error(ex); } catch (Exception ex) { - Logger.getLogger(RasterImageLayer.class.getName()).log(Level.SEVERE, null, ex); + Logger.error(ex); } } else { try { raster = new RasterImageLayer(getName(), getLayerManager(), getImageFileName(), getImageForDisplay(), new Envelope(getWholeImageEnvelope())); } catch (IOException ex) { - Logger.getLogger(RasterImageLayer.class.getName()).log(Level.SEVERE, null, ex); + Logger.error(ex); } catch (NoninvertibleTransformException ex) { - Logger.getLogger(RasterImageLayer.class.getName()).log(Level.SEVERE, null, ex); + Logger.error(ex); } catch (Exception ex) { - Logger.getLogger(RasterImageLayer.class.getName()).log(Level.SEVERE, null, ex); + Logger.error(ex); } } // clone must produce a layerable with the same name (as for Layer) not a unique name Modified: core/trunk/src/org/openjump/core/rasterimage/styler/ui/CustomGradientColorsDialog.java =================================================================== --- core/trunk/src/org/openjump/core/rasterimage/styler/ui/CustomGradientColorsDialog.java 2016-01-10 18:45:58 UTC (rev 4767) +++ core/trunk/src/org/openjump/core/rasterimage/styler/ui/CustomGradientColorsDialog.java 2016-01-10 18:47:43 UTC (rev 4768) @@ -1,11 +1,12 @@ package org.openjump.core.rasterimage.styler.ui; -import java.util.logging.Level; -import java.util.logging.Logger; import javax.swing.JOptionPane; + import org.openjump.core.rasterimage.styler.ColorMapEntry; import org.openjump.core.rasterimage.styler.RasterStylesExtension; +import com.vividsolutions.jump.workbench.Logger; + /** * * @author GeomaticaEAmbiente @@ -187,7 +188,7 @@ try { okButton(); } catch (Exception ex) { - Logger.getLogger(org.openjump.core.rasterimage.styler.ui.CustomGradientColorsDialog.class.getName()).log(Level.SEVERE, null, ex); + Logger.error(ex); } Modified: core/trunk/src/org/openjump/core/rasterimage/styler/ui/GradientTablePanel.java =================================================================== --- core/trunk/src/org/openjump/core/rasterimage/styler/ui/GradientTablePanel.java 2016-01-10 18:45:58 UTC (rev 4767) +++ core/trunk/src/org/openjump/core/rasterimage/styler/ui/GradientTablePanel.java 2016-01-10 18:47:43 UTC (rev 4768) @@ -4,16 +4,18 @@ import java.awt.Component; import java.util.ArrayList; import java.util.List; -import java.util.logging.Level; -import java.util.logging.Logger; + import javax.swing.JPanel; import javax.swing.JTable; import javax.swing.event.TableModelEvent; import javax.swing.event.TableModelListener; import javax.swing.table.DefaultTableModel; + import org.openjump.core.rasterimage.styler.ColorMapEntry; import org.openjump.core.rasterimage.styler.ColorUtils; +import com.vividsolutions.jump.workbench.Logger; + /** * * @author GeomaticaEAmbiente @@ -102,7 +104,7 @@ }catch(Exception ex){ updateGradient = true; - Logger.getLogger(org.openjump.core.rasterimage.styler.ui.GradientTablePanel.class.getName()).log(Level.SEVERE, null, ex); + Logger.error(ex); } updateGradient = true; Modified: core/trunk/src/org/openjump/core/rasterimage/styler/ui/IntervalPanel.java =================================================================== --- core/trunk/src/org/openjump/core/rasterimage/styler/ui/IntervalPanel.java 2016-01-10 18:45:58 UTC (rev 4767) +++ core/trunk/src/org/openjump/core/rasterimage/styler/ui/IntervalPanel.java 2016-01-10 18:47:43 UTC (rev 4768) @@ -1,8 +1,5 @@ package org.openjump.core.rasterimage.styler.ui; -import org.openjump.core.rasterimage.styler.ColorUtils; -import org.openjump.core.rasterimage.styler.RasterStylesExtension; -import com.vividsolutions.jump.util.Range; import java.awt.Color; import java.awt.Component; import java.awt.GridBagConstraints; @@ -14,17 +11,22 @@ import java.util.List; import java.util.Map; import java.util.Map.Entry; -import java.util.logging.Level; -import java.util.logging.Logger; + import javax.swing.JOptionPane; + import org.openjump.core.attributeoperations.Classifier1D; import org.openjump.core.rasterimage.RasterImageLayer; +import org.openjump.core.rasterimage.RasterSymbology; import org.openjump.core.rasterimage.styler.ColorMapEntry; -import org.openjump.core.rasterimage.RasterSymbology; -import org.openjump.core.rasterimage.styler.ui.ColorsTablePanel.TableType; +import org.openjump.core.rasterimage.styler.ColorUtils; import org.openjump.core.rasterimage.styler.RasterClassifier1D; +import org.openjump.core.rasterimage.styler.RasterStylesExtension; import org.openjump.core.rasterimage.styler.Utils; +import org.openjump.core.rasterimage.styler.ui.ColorsTablePanel.TableType; +import com.vividsolutions.jump.util.Range; +import com.vividsolutions.jump.workbench.Logger; + /** * * @author GeomaticaEAmbiente @@ -178,7 +180,7 @@ try { rampAll(); } catch (Exception ex) { - Logger.getLogger(IntervalPanel.class.getName()).log(Level.SEVERE, null, ex); + Logger.error(ex); } }//GEN-LAST:event_jButton_ValuesActionPerformed @@ -194,7 +196,7 @@ try { rampColors(); } catch (Exception ex) { - Logger.getLogger(IntervalPanel.class.getName()).log(Level.SEVERE, null, ex); + Logger.error(ex); } }//GEN-LAST:event_jButton_RampActionPerformed @@ -209,7 +211,7 @@ try { rampAll(); } catch (Exception ex) { - Logger.getLogger(IntervalPanel.class.getName()).log(Level.SEVERE, null, ex); + Logger.error(ex); } } Modified: core/trunk/src/org/openjump/core/rasterimage/styler/ui/RasterStylesDialog.java =================================================================== --- core/trunk/src/org/openjump/core/rasterimage/styler/ui/RasterStylesDialog.java 2016-01-10 18:45:58 UTC (rev 4767) +++ core/trunk/src/org/openjump/core/rasterimage/styler/ui/RasterStylesDialog.java 2016-01-10 18:47:43 UTC (rev 4768) @@ -1,23 +1,17 @@ package org.openjump.core.rasterimage.styler.ui; -import com.vividsolutions.jump.util.Range; -import com.vividsolutions.jump.util.StringUtil; -import com.vividsolutions.jump.workbench.WorkbenchContext; -import com.vividsolutions.jump.workbench.model.LayerEventType; -import com.vividsolutions.jump.workbench.ui.ErrorDialog; -import de.fho.jump.pirol.utilities.settings.PirolPlugInSettings; import java.awt.Color; import java.awt.event.ActionEvent; import java.awt.event.ActionListener; import java.awt.geom.NoninvertibleTransformException; import java.io.File; import java.io.IOException; -import java.util.logging.Level; -import java.util.logging.Logger; + import javax.swing.InputVerifier; import javax.swing.JColorChooser; import javax.swing.JComponent; import javax.swing.JFileChooser; + import org.openjump.core.rasterimage.RasterImageLayer; import org.openjump.core.rasterimage.RasterSymbology; import org.openjump.core.rasterimage.Stats; @@ -27,6 +21,15 @@ import org.openjump.core.ui.io.file.FileNameExtensionFilter; import org.xml.sax.SAXException; +import com.vividsolutions.jump.util.Range; +import com.vividsolutions.jump.util.StringUtil; +import com.vividsolutions.jump.workbench.Logger; +import com.vividsolutions.jump.workbench.WorkbenchContext; +import com.vividsolutions.jump.workbench.model.LayerEventType; +import com.vividsolutions.jump.workbench.ui.ErrorDialog; + +import de.fho.jump.pirol.utilities.settings.PirolPlugInSettings; + /** * * @author GeomaticaEAmbiente @@ -329,7 +332,7 @@ try { updateRasterImageLayer(); } catch (Exception ex) { - Logger.getLogger(RasterStylesDialog.class.getName()).log(Level.SEVERE, null, ex); + Logger.error(ex); } }//GEN-LAST:event_jButton_ApplyActionPerformed @@ -345,7 +348,7 @@ try { restoreToOriginal(); } catch (Exception ex) { - Logger.getLogger(RasterStylesDialog.class.getName()).log(Level.SEVERE, null, ex); + Logger.error(ex); } }//GEN-LAST:event_jButton_RestoreActionPerformed @@ -357,7 +360,7 @@ updateRasterImageLayer(); closeDialog(); } catch (Exception ex) { - Logger.getLogger(RasterStylesDialog.class.getName()).log(Level.SEVERE, null, ex); + Logger.error( ex); } }//GEN-LAST:event_jButton_OKActionPerformed Modified: core/trunk/src/org/openjump/core/rasterimage/styler/ui/SingleValuesPanel.java =================================================================== --- core/trunk/src/org/openjump/core/rasterimage/styler/ui/SingleValuesPanel.java 2016-01-10 18:45:58 UTC (rev 4767) +++ core/trunk/src/org/openjump/core/rasterimage/styler/ui/SingleValuesPanel.java 2016-01-10 18:47:43 UTC (rev 4768) @@ -7,15 +7,17 @@ import java.util.Iterator; import java.util.Set; import java.util.TreeSet; -import java.util.logging.Level; -import java.util.logging.Logger; + import javax.swing.JOptionPane; -import org.openjump.core.rasterimage.styler.ColorMapEntry; + import org.openjump.core.rasterimage.RasterImageLayer; import org.openjump.core.rasterimage.RasterSymbology; +import org.openjump.core.rasterimage.styler.ColorMapEntry; import org.openjump.core.rasterimage.styler.ColorUtils; import org.openjump.core.rasterimage.styler.RasterStylesExtension; +import com.vividsolutions.jump.workbench.Logger; + /** * * @author GeomaticaEAmbiente @@ -128,7 +130,7 @@ try { rampAll(false); } catch (Exception ex) { - Logger.getLogger(SingleValuesPanel.class.getName()).log(Level.SEVERE, null, ex); + Logger.error(ex); } }//GEN-LAST:event_jButton_RampActionPerformed @@ -136,7 +138,7 @@ try { rampAll(true); } catch (Exception ex) { - Logger.getLogger(SingleValuesPanel.class.getName()).log(Level.SEVERE, null, ex); + Logger.error(ex); } }//GEN-LAST:event_jButton_RandomActionPerformed @@ -148,7 +150,7 @@ findUniqueValues(); rampAll(true); } catch (Exception ex) { - Logger.getLogger(SingleValuesPanel.class.getName()).log(Level.SEVERE, null, ex); + Logger.error(ex); } } @@ -158,7 +160,7 @@ try { rampColors(); } catch (Exception ex) { - Logger.getLogger(SingleValuesPanel.class.getName()).log(Level.SEVERE, null, ex); + Logger.error(ex); } }//GEN-LAST:event_jButton1ActionPerformed Modified: core/trunk/src/org/openjump/core/ui/plugin/customize/PythonToolsPlugIn.java =================================================================== --- core/trunk/src/org/openjump/core/ui/plugin/customize/PythonToolsPlugIn.java 2016-01-10 18:45:58 UTC (rev 4767) +++ core/trunk/src/org/openjump/core/ui/plugin/customize/PythonToolsPlugIn.java 2016-01-10 18:47:43 UTC (rev 4768) @@ -44,7 +44,7 @@ import javax.swing.JMenu; import javax.swing.JMenuBar; -import org.apache.log4j.Logger; + import org.openjump.util.python.JUMP_GIS_Framework; import org.openjump.util.python.ModifyGeometry; import org.openjump.util.python.PythonInteractiveInterpreter; @@ -65,7 +65,7 @@ import com.vividsolutions.jump.workbench.ui.toolbox.ToolboxPlugIn; public class PythonToolsPlugIn extends ToolboxPlugIn { - private static final Logger LOG = Logger.getLogger(PythonToolsPlugIn.class); + private static String sName = "Python Console and Tools"; private static final ImageIcon icon = IconLoader .icon("famfam/application_python.png"); Modified: core/trunk/src/org/openjump/core/ui/plugin/datastore/AddWritableDataStoreLayerWizard.java =================================================================== --- core/trunk/src/org/openjump/core/ui/plugin/datastore/AddWritableDataStoreLayerWizard.java 2016-01-10 18:45:58 UTC (rev 4767) +++ core/trunk/src/org/openjump/core/ui/plugin/datastore/AddWritableDataStoreLayerWizard.java 2016-01-10 18:47:43 UTC (rev 4768) @@ -1,5 +1,15 @@ package org.openjump.core.ui.plugin.datastore; +import java.awt.Color; +import java.util.Collection; + +import javax.swing.SwingUtilities; + +import org.openjump.core.ccordsys.srid.SRIDStyle; +import org.openjump.core.ui.plugin.datastore.transaction.DataStoreTransactionManager; +import org.openjump.core.ui.plugin.file.open.ChooseProjectPanel; +import org.openjump.core.ui.swing.wizard.AbstractWizardGroup; + import com.vividsolutions.jump.I18N; import com.vividsolutions.jump.coordsys.CoordinateSystemRegistry; import com.vividsolutions.jump.feature.FeatureCollection; @@ -7,6 +17,7 @@ import com.vividsolutions.jump.io.datasource.DataSource; import com.vividsolutions.jump.io.datasource.DataSourceQuery; import com.vividsolutions.jump.task.TaskMonitor; +import com.vividsolutions.jump.workbench.Logger; import com.vividsolutions.jump.workbench.WorkbenchContext; import com.vividsolutions.jump.workbench.datastore.ConnectionDescriptor; import com.vividsolutions.jump.workbench.model.Category; @@ -16,23 +27,13 @@ import com.vividsolutions.jump.workbench.ui.images.IconLoader; import com.vividsolutions.jump.workbench.ui.plugin.AddNewLayerPlugIn; import com.vividsolutions.jump.workbench.ui.wizard.WizardDialog; -import org.apache.log4j.Logger; -import org.openjump.core.ccordsys.srid.SRIDStyle; -import org.openjump.core.ui.plugin.datastore.postgis2.PostGISDataStoreDataSource; -import org.openjump.core.ui.plugin.datastore.transaction.DataStoreTransactionManager; -import org.openjump.core.ui.plugin.file.open.ChooseProjectPanel; -import org.openjump.core.ui.swing.wizard.AbstractWizardGroup; -import javax.swing.*; -import java.awt.*; -import java.util.Collection; - /** * Add a wizard to the Open dialog to open database connections with read/write capability. */ public class AddWritableDataStoreLayerWizard extends AbstractWizardGroup { - final Logger LOG = Logger.getLogger(AddWritableDataStoreLayerWizard.class); + private static final String KEY = AddWritableDataStoreLayerWizard.class.getName(); private AddWritableDataStoreLayerWizardPanel dataStoreWizardPanel; @@ -140,13 +141,13 @@ sridStyle.setSRID(panel.getGeometryColumn().getSRID()); layer.addStyle(sridStyle); - LOG.info("Add layer '" + layer.getName() + "' to '" + layerManager.getTask().getName() + "' using WritableDataStoreDataSource with :"); - LOG.info(" geometry column = " + geometryAttributeName); - LOG.info(" srid = " + sridStyle.getSRID()); - LOG.info(" external PK column = " + identifierAttributeName); - LOG.info(" max features = " + limit); - LOG.info(" where clause = " + whereClause); - LOG.info(" limit to view = " + limitedToView); + Logger.info("Add layer '" + layer.getName() + "' to '" + layerManager.getTask().getName() + "' using WritableDataStoreDataSource with :"); + Logger.info(" geometry column = " + geometryAttributeName); + Logger.info(" srid = " + sridStyle.getSRID()); + Logger.info(" external PK column = " + identifierAttributeName); + Logger.info(" max features = " + limit); + Logger.info(" where clause = " + whereClause); + Logger.info(" limit to view = " + limitedToView); layerManager.setFiringEvents(true); // added by michaudm on 2009-04-05 } Modified: core/trunk/src/org/openjump/core/ui/plugin/datastore/WritableDataStoreDataSource.java =================================================================== --- core/trunk/src/org/openjump/core/ui/plugin/datastore/WritableDataStoreDataSource.java 2016-01-10 18:45:58 UTC (rev 4767) +++ core/trunk/src/org/openjump/core/ui/plugin/datastore/WritableDataStoreDataSource.java 2016-01-10 18:47:43 UTC (rev 4768) @@ -18,7 +18,6 @@ import javax.swing.JOptionPane; -import org.apache.log4j.Logger; import org.openjump.core.ui.plugin.datastore.postgis.PostGISConnectionUtil; import org.openjump.core.ui.plugin.datastore.postgis.PostGISQueryUtil; import org.openjump.core.ui.plugin.datastore.transaction.DataStoreTransactionManager; @@ -38,6 +37,7 @@ import com.vividsolutions.jump.task.TaskMonitor; import com.vividsolutions.jump.util.CollectionUtil; import com.vividsolutions.jump.workbench.JUMPWorkbench; +import com.vividsolutions.jump.workbench.Logger; import com.vividsolutions.jump.workbench.datastore.ConnectionDescriptor; import com.vividsolutions.jump.workbench.model.Layer; import com.vividsolutions.jump.workbench.ui.plugin.datastore.DataStoreDataSource; @@ -48,8 +48,8 @@ public abstract class WritableDataStoreDataSource extends DataStoreDataSource { private static final String KEY = WritableDataStoreDataSource.class.getName(); - Logger LOG = Logger.getLogger(WritableDataStoreDataSource.class); + // Inherited from DataStoreDataSource (query) // DATASET_NAME_KEY // GEOMETRY_ATTRIBUTE_NAME_KEY @@ -203,7 +203,7 @@ try { conn.setAutoCommit(false); if (!tableAlreadyCreated) { - LOG.debug("Update mode: create table"); + Logger.debug("Update mode: create table"); boolean exists = tableExists(conn); if (exists && !confirmOverwrite()) return; if (exists) { @@ -229,7 +229,7 @@ tableAlreadyCreated = true; } else { - LOG.debug("Update mode: update table"); + Logger.debug("Update mode: update table"); primaryKeyName = (String)getProperties().get(EXTERNAL_PK_KEY); FeatureSchema featureSchema = featureCollection.getFeatureSchema(); PostGISConnectionUtil connUtil = new PostGISConnectionUtil(conn); @@ -269,17 +269,17 @@ private void commit(java.sql.Connection conn, int srid, int dim, boolean normalizedColumnNames) throws Exception { - LOG.info("Evolutions to commit to " + schemaName + "." + tableName + " (PK=" + primaryKeyName +")"); + Logger.info("Evolutions to commit to " + schemaName + "." + tableName + " (PK=" + primaryKeyName +")"); for (Evolution evolution : evolutions.values()) { if (evolution.getType() == Evolution.Type.CREATION) { PreparedStatement pstmt = insertStatement(conn, evolution.getNewFeature().getSchema(), normalizedColumnNames); pstmt = setAttributeValues(pstmt, evolution.getNewFeature(), srid, dim, primaryKeyName); pstmt.execute(); - LOG.info(" create new feature " + evolution.getNewFeature().getID()+"/"); + Logger.info(" create new feature " + evolution.getNewFeature().getID()+"/"); } else if (evolution.getType() == Evolution.Type.SUPPRESSION) { deleteStatement(conn, evolution.getOldFeature()).executeUpdate(); - LOG.info(" delete " + evolution.getOldFeature().getID() + "/" + + Logger.info(" delete " + evolution.getOldFeature().getID() + "/" + evolution.getOldFeature().getAttribute(primaryKeyName)); } else if (evolution.getType() == Evolution.Type.MODIFICATION) { Feature oldFeature = evolution.getOldFeature(); @@ -294,7 +294,7 @@ updateOneAttributeStatement(conn, newFeature, i, srid, dim).executeUpdate(); } } - LOG.info(" modify " + evolution.getNewFeature().getID() + "/" + + Logger.info(" modify " + evolution.getNewFeature().getID() + "/" + evolution.getNewFeature().getAttribute(primaryKeyName)); } } @@ -353,14 +353,14 @@ else if (type == AttributeType.DATE) pstmt.setTimestamp(1, new Timestamp(((Date) feature.getAttribute(attribute)).getTime())); else if (type == AttributeType.OBJECT) pstmt.setObject(1, feature.getAttribute(attribute)); else throw new IllegalArgumentException(type + " is an unknown AttributeType !"); - LOG.debug(pstmt); + Logger.debug(pstmt.toString()); return pstmt; } private PreparedStatement deleteStatement(java.sql.Connection conn, Feature feature) throws SQLException { PreparedStatement pstmt = conn.prepareStatement("DELETE FROM " + compose(schemaName, tableName) + " WHERE \"" + primaryKeyName + "\" = ?"); pstmt.setObject(1,feature.getAttribute(primaryKeyName)); - LOG.debug(pstmt); + Logger.debug(pstmt.toString()); return pstmt; } Modified: core/trunk/src/org/openjump/core/ui/plugin/datastore/postgis2/PostGISDataStoreDataSource.java =================================================================== --- core/trunk/src/org/openjump/core/ui/plugin/datastore/postgis2/PostGISDataStoreDataSource.java 2016-01-10 18:45:58 UTC (rev 4767) +++ core/trunk/src/org/openjump/core/ui/plugin/datastore/postgis2/PostGISDataStoreDataSource.java 2016-01-10 18:47:43 UTC (rev 4768) @@ -1,37 +1,38 @@ package org.openjump.core.ui.plugin.datastore.postgis2; +import static org.openjump.core.ui.plugin.datastore.postgis.PostGISQueryUtil.compose; +import static org.openjump.core.ui.plugin.datastore.postgis.PostGISQueryUtil.normalize; +import static org.openjump.core.ui.plugin.datastore.postgis.PostGISQueryUtil.unquote; + +import java.sql.Connection; +import java.sql.PreparedStatement; +import java.sql.ResultSet; +import java.sql.SQLException; +import java.util.Iterator; + +import org.openjump.core.ui.plugin.datastore.WritableDataStoreDataSource; +import org.openjump.core.ui.plugin.datastore.postgis.PostGISQueryUtil; + import com.vividsolutions.jts.geom.Coordinate; import com.vividsolutions.jump.datastore.AdhocQuery; import com.vividsolutions.jump.datastore.postgis.PostgisDSConnection; +import com.vividsolutions.jump.datastore.postgis.PostgisDSDriver; import com.vividsolutions.jump.datastore.postgis.PostgisDSMetadata; -import com.vividsolutions.jump.datastore.postgis.PostgisDSDriver; import com.vividsolutions.jump.feature.Feature; import com.vividsolutions.jump.feature.FeatureCollection; import com.vividsolutions.jump.feature.FeatureDataset; import com.vividsolutions.jump.feature.FeatureSchema; import com.vividsolutions.jump.io.FeatureInputStream; +import com.vividsolutions.jump.workbench.Logger; import com.vividsolutions.jump.workbench.datastore.ConnectionDescriptor; -import org.apache.log4j.Logger; -import org.openjump.core.ui.plugin.datastore.WritableDataStoreDataSource; -import org.openjump.core.ui.plugin.datastore.postgis.PostGISQueryUtil; -import java.sql.Connection; -import java.sql.PreparedStatement; -import java.sql.ResultSet; -import java.sql.SQLException; -import java.util.Iterator; - -import static org.openjump.core.ui.plugin.datastore.postgis.PostGISQueryUtil.compose; -import static org.openjump.core.ui.plugin.datastore.postgis.PostGISQueryUtil.normalize; -import static org.openjump.core.ui.plugin.datastore.postgis.PostGISQueryUtil.unquote; - /** * A {@link WritableDataStoreDataSource} for PostGIS. */ public class PostGISDataStoreDataSource extends WritableDataStoreDataSource { - Logger LOG = Logger.getLogger(WritableDataStoreDataSource.class); + public PostGISDataStoreDataSource() { // Called by Java2XML [Jon Aquino 2005-03-16] } @@ -53,12 +54,12 @@ public void finalizeUpdate(Connection conn) throws Exception { conn.createStatement().execute("VACUUM ANALYZE " + PostGISQueryUtil.compose(schemaName, tableName)); - LOG.debug("VACUUM ANALYZE " + PostGISQueryUtil.compose(schemaName, tableName)); + Logger.debug("VACUUM ANALYZE " + PostGISQueryUtil.compose(schemaName, tableName)); } protected FeatureCollection createFeatureCollection() throws Exception { - LOG.debug("Create new FeatureCollection from " + getProperties().get(DATASET_NAME_KEY)); + Logger.debug("Create new FeatureCollection from " + getProperties().get(DATASET_NAME_KEY)); //String[] datasetName = PostGISQueryUtil.splitTableName((String)getProperties().get(DATASET_NAME_KEY)); //schemaName = datasetName[0]; //tableName = datasetName[1]; @@ -72,7 +73,7 @@ boolean hasPK = getProperties().get(EXTERNAL_PK_KEY) != null; String PK = (String)getProperties().get(EXTERNAL_PK_KEY); String query = buildQueryString(pgConnection); - LOG.debug(query); + Logger.debug(query); // Create the adhoc query corresponding to this datasource AdhocQuery adhocQuery = new AdhocQuery(query); @@ -253,7 +254,7 @@ String sql = schemaName == null ? "SELECT Find_SRID('public', '" + tableName + "', '" + unquote(column) + "');" : "SELECT Find_SRID('" + schemaName + "', '" + tableName + "', '" + unquote(column) + "');"; - LOG.debug(sql); + Logger.debug(sql); ResultSet rs = conn.prepareStatement(sql).executeQuery(); if (rs.next()) return rs.getInt(1); else return 0; Modified: core/trunk/src/org/openjump/core/ui/plugin/datastore/transaction/DataStoreTransactionManager.java =================================================================== --- core/trunk/src/org/openjump/core/ui/plugin/datastore/transaction/DataStoreTransactionManager.java 2016-01-10 18:45:58 UTC (rev 4767) +++ core/trunk/src/org/openjump/core/ui/plugin/datastore/transaction/DataStoreTransactionManager.java 2016-01-10 18:47:43 UTC (rev 4768) @@ -1,18 +1,31 @@ package org.openjump.core.ui.plugin.datastore.transaction; +import java.util.ArrayList; +import java.util.Arrays; +import java.util.Collection; +import java.util.Iterator; +import java.util.Map; +import java.util.WeakHashMap; + +import org.openjump.core.ui.plugin.datastore.WritableDataStoreDataSource; + import com.vividsolutions.jump.feature.Feature; import com.vividsolutions.jump.feature.FeatureCollection; import com.vividsolutions.jump.feature.FeatureDataset; import com.vividsolutions.jump.io.datasource.DataSource; import com.vividsolutions.jump.task.DummyTaskMonitor; import com.vividsolutions.jump.workbench.JUMPWorkbench; -import com.vividsolutions.jump.workbench.model.*; +import com.vividsolutions.jump.workbench.Logger; +import com.vividsolutions.jump.workbench.model.FeatureEvent; +import com.vividsolutions.jump.workbench.model.FeatureEventType; +import com.vividsolutions.jump.workbench.model.Layer; +import com.vividsolutions.jump.workbench.model.LayerAdapter; +import com.vividsolutions.jump.workbench.model.LayerEvent; +import com.vividsolutions.jump.workbench.model.LayerEventType; +import com.vividsolutions.jump.workbench.model.LayerListener; +import com.vividsolutions.jump.workbench.model.Task; import com.vividsolutions.jump.workbench.ui.TaskFrame; -import org.apache.log4j.Logger; -import org.openjump.core.ui.plugin.datastore.WritableDataStoreDataSource; -import java.util.*; - /** * Central class to manage datastore transactions from OpenJUMP. * There is only one DataStoreTransactionManager (singleton) for the whole Workbench. @@ -23,7 +36,7 @@ */ public class DataStoreTransactionManager { - final Logger LOG = Logger.getLogger(DataStoreTransactionManager.class); + final String KEY = DataStoreTransactionManager.class.getName(); private static DataStoreTransactionManager TXMANAGER = new DataStoreTransactionManager(); @@ -52,7 +65,7 @@ registeredListeners.put(task,listener); } registeredLayers.put(layer, task); - LOG.info("Register layer '" + layer.getName() + "' (" + task.getName() + ") in the DataStoreTransactionManager"); + Logger.info("Register layer '" + layer.getName() + "' (" + task.getName() + ") in the DataStoreTransactionManager"); } private boolean containsLayerManager(Task task) { @@ -70,20 +83,20 @@ if (e.getType() == FeatureEventType.ADDED) { for (Object feature : e.getFeatures()) { try { - LOG.debug("FeatureEventType: ADDED " + ((Feature)feature).getID()); + Logger.debug("FeatureEventType: ADDED " + ((Feature)feature).getID()); source.addCreation((Feature)feature); } catch (EvolutionOperationException ex) { - LOG.error("Error during creation of feature " + ((Feature)feature).getID(), ex); + Logger.error("Error during creation of feature " + ((Feature)feature).getID(), ex); } } } else if (e.getType() == FeatureEventType.DELETED) { for (Object feature : e.getFeatures()) { try { - LOG.debug("FeatureEventType: DELETED " + ((Feature)feature).getID()); + Logger.debug("FeatureEventType: DELETED " + ((Feature)feature).getID()); source.addSuppression((Feature)feature); } catch (EvolutionOperationException ex) { - LOG.error("Error during suppression of feature " + ((Feature)feature).getID(), ex); + Logger.error("Error during suppression of feature " + ((Feature)feature).getID(), ex); } } } @@ -93,10 +106,10 @@ Iterator oldFeatures = e.getOldFeatureAttClones().iterator(); for (Object feature : e.getFeatures()) { try { - LOG.debug("FeatureEventType: ATTRIBUTES_MODIFIED " + ((Feature)feature).getID()); + Logger.debug("FeatureEventType: ATTRIBUTES_MODIFIED " + ((Feature)feature).getID()); source.addModification((Feature)feature, (Feature)oldFeatures.next()); } catch (EvolutionOperationException ex) { - LOG.error("Error during modification of feature " + ((Feature)feature).getID(), ex); + Logger.error("Error during modification of feature " + ((Feature)feature).getID(), ex); } } } @@ -106,19 +119,19 @@ Iterator oldFeatures = e.getOldFeatureClones().iterator(); for (Object feature : e.getFeatures()) { try { - LOG.debug("FeatureEventType: GEOMETRY_MODIFIED " + ((Feature)feature).getID()); + Logger.debug("FeatureEventType: GEOMETRY_MODIFIED " + ((Feature)feature).getID()); source.addModification((Feature)feature, (Feature)oldFeatures.next()); } catch (EvolutionOperationException ex) { - LOG.error("Error during modification of feature " + ((Feature)feature).getID(), ex); + Logger.error("Error during modification of feature " + ((Feature)feature).getID(), ex); } } } else { - LOG.error(e.getType() + " is an unknown FeatureEventType"); + Logger.error(e.getType() + " is an unknown FeatureEventType"); } } else { - LOG.error("DataStoreTransactionManager should never contain a reference to a layer which does not use a WritableDataStoreDataSource"); + Logger.error("DataStoreTransactionManager should never contain a reference to a layer which does not use a WritableDataStoreDataSource"); } } } @@ -130,7 +143,7 @@ } } registeredLayers.remove(e.getLayerable()); - LOG.info("Unregister layer " + e.getLayerable().getName() + " from the DataStoreTransactionManager"); + Logger.info("Unregister layer " + e.getLayerable().getName() + " from the DataStoreTransactionManager"); } } }; @@ -169,10 +182,10 @@ // (see also WritableDataStoreDataSource#reloadDataFromDataStore) //source.getProperties().put(WritableDataStoreDataSource.CREATE_TABLE, false); try { - LOG.info("Commit layer \"" + layer.getName() + "\""); + Logger.info("Commit layer \"" + layer.getName() + "\""); writableSource.getConnection().executeUpdate(null,layer.getFeatureCollectionWrapper(), new DummyTaskMonitor()); } catch (Exception e) { - LOG.error("Error occurred while comitting layer \"" + layer.getName() + "\"", e); + Logger.error("Error occurred while comitting layer \"" + layer.getName() + "\"", e); throw e; } return true; @@ -190,16 +203,16 @@ WritableDataStoreDataSource writableSource = (WritableDataStoreDataSource)source; int conflicts = 0; try { - LOG.info("Update layer \"" + layer.getName() + "\""); + Logger.info("Update layer \"" + layer.getName() + "\""); FeatureCollection fc = writableSource.getConnection().executeQuery(null, new DummyTaskMonitor()); conflicts = manageConflicts(taskFrame, layer, fc); layer.getLayerManager().setFiringEvents(false); layer.setFeatureCollection(fc); - LOG.info("" + fc.size() + " features uploaded"); + Logger.info("" + fc.size() + " features uploaded"); layer.getLayerManager().setFiringEvents(true); - LOG.info("" + conflicts + " conflicts detected"); + Logger.info("" + conflicts + " conflicts detected"); } catch (Exception e) { - LOG.error("Error occurred while updating layer \"" + layer.getName() + "\"", e); + Logger.error("Error occurred while updating layer \"" + layer.getName() + "\"", e); return -1; } return conflicts; @@ -211,7 +224,7 @@ * Update all layers associated to a RWDataStoreDataSource. */ public void update(TaskFrame taskFrame) { - LOG.info("Update project \"" + taskFrame.getTask().getName() + "\""); + Logger.info("Update project \"" + taskFrame.getTask().getName() + "\""); int total_conflicts = 0; boolean no_error = true; for (Layer layer : registeredLayers.keySet()) { @@ -224,9 +237,9 @@ taskFrame.getLayerViewPanel().getSelectionManager().clear(); taskFrame.getLayerViewPanel().repaint(); if (no_error) { - LOG.info("Project update finished with 0 error and " + total_conflicts + " conflicts"); + Logger.info("Project update finished with 0 error and " + total_conflicts + " conflicts"); } - else LOG.info("Project update finished with errors"); + else Logger.info("Project update finished with errors"); } private void inspect(TaskFrame taskFrame, Layer layer) { @@ -262,7 +275,7 @@ else if (evolution.getType() == Evolution.Type.SUPPRESSION) { layer_s.getFeatureCollectionWrapper().add(evolution.getOldFeature()); } - else LOG.error("Tried to inspect an evolution which is neither a creation nor a modification or a suppression"); + else Logger.error("Tried to inspect an evolution which is neither a creation nor a modification or a suppression"); } } } @@ -281,15 +294,15 @@ public void commit() throws Exception { TaskFrame activeFrame = JUMPWorkbench.getInstance().getFrame().getActiveTaskFrame(); if (activeFrame == null) return; - LOG.info("Commit evolutions on project \"" + activeFrame.getTask().getName() + "\""); + Logger.info("Commit evolutions on project \"" + activeFrame.getTask().getName() + "\""); boolean no_error = true; for (Layer layer : registeredLayers.keySet()) { if (activeFrame.getTask().getLayerManager().getLayers().contains(layer)) { no_error = commit(layer) && no_error; } } - if (no_error) LOG.info("Commit finished without error"); - else LOG.info("Commit finished with error"); + if (no_error) Logger.info("Commit finished without error"); + else Logger.info("Commit finished with error"); update(activeFrame); } @@ -324,9 +337,9 @@ // We must check if the remote change is the same as the local change // If different, we have a conflict and must decide which one must be kept if (!Arrays.equals(((Feature) feature).getAttributes(), evo.getNewFeature().getAttributes())) { - LOG.warn("Conflict detected for feature " + evo.getNewFeature().getAttribute(pk)); - LOG.trace(" - Server: " + feature); - LOG.trace(" - Local : " + evo.getNewFeature()); + Logger.warn("Conflict detected for feature " + evo.getNewFeature().getAttribute(pk)); + Logger.trace(" - Server: " + feature); + Logger.trace(" - Local : " + evo.getNewFeature()); conflicts++; if (manageConflicts) { copyLocallyModifiedFeature(taskFrame, layer, evo.getNewFeature()); @@ -336,25 +349,25 @@ // we can remove the change from the evolution table else { dataSource.removeEvolution(evo.getNewFeature().getID()); - LOG.trace("Eliminate an evolution from evolution stack after detection of a false conflict: " + Logger.trace("Eliminate an evolution from evolution stack after detection of a false conflict: " + evo.getNewFeature().getAttribute(pk)); } } else { - LOG.trace("Database has not been changed since last transaction : Keep local changes"); + Logger.trace("Database has not been changed since last transaction : Keep local changes"); ((Feature)feature).setAttributes(evo.getNewFeature().getAttributes()); } } if (evo.getType()==Evolution.Type.SUPPRESSION) { if (!Arrays.equals(((Feature) feature).getAttributes(), evo.getOldFeature().getAttributes())) { - LOG.warn("Conflict detected for feature " + evo.getNewFeature().getAttribute(pk)); - LOG.trace(" - The feature has been locally deleted"); + Logger.warn("Conflict detected for feature " + evo.getNewFeature().getAttribute(pk)); + Logger.trace(" - The feature has been locally deleted"); conflicts++; copyLocallyDeletedFeature(taskFrame, layer, evo.getOldFeature()); } // Database feature has not been changed : keep the local change (delete it) else { - LOG.trace("Feature " + ((Feature) feature).getAttribute(pk) + " has been locally deleted, don't update it again !"); + Logger.trace("Feature " + ((Feature) feature).getAttribute(pk) + " has been locally deleted, don't update it again !"); toBeDeleted.add(evo.getOldFeature()); } } Modified: core/trunk/src/org/openjump/core/ui/plugin/datastore/transaction/DataStoreTransactionManagerPlugIn.java =================================================================== --- core/trunk/src/org/openjump/core/ui/plugin/datastore/transaction/DataStoreTransactionManagerPlugIn.java 2016-01-10 18:45:58 UTC (rev 4767) +++ core/trunk/src/org/openjump/core/ui/plugin/datastore/transaction/DataStoreTransactionManagerPlugIn.java 2016-01-10 18:47:43 UTC (rev 4768) @@ -1,31 +1,28 @@ package org.openjump.core.ui.plugin.datastore.transaction; -import com.vividsolutions.jump.I18N; +import java.awt.BorderLayout; + +import javax.swing.ImageIcon; +import javax.swing.event.InternalFrameAdapter; +import javax.swing.event.InternalFrameEvent; + +import org.openjump.core.model.TaskEvent; +import org.openjump.core.model.TaskListener; +import org.openjump.core.ui.plugin.datastore.WritableDataStoreDataSource; + import com.vividsolutions.jump.util.Blackboard; +import com.vividsolutions.jump.workbench.Logger; import com.vividsolutions.jump.workbench.WorkbenchContext; import com.vividsolutions.jump.workbench.model.Layer; import com.vividsolutions.jump.workbench.model.Task; -import com.vividsolutions.jump.workbench.plugin.EnableCheck; import com.vividsolutions.jump.workbench.plugin.PlugInContext; import com.vividsolutions.jump.workbench.ui.GUIUtil; -import com.vividsolutions.jump.workbench.ui.MenuNames; import com.vividsolutions.jump.workbench.ui.TaskFrame; import com.vividsolutions.jump.workbench.ui.images.IconLoader; -import com.vividsolutions.jump.workbench.ui.plugin.FeatureInstaller; import com.vividsolutions.jump.workbench.ui.plugin.WorkbenchContextReference; import com.vividsolutions.jump.workbench.ui.toolbox.ToolboxDialog; import com.vividsolutions.jump.workbench.ui.toolbox.ToolboxPlugIn; -import org.apache.log4j.Logger; -import org.openjump.core.model.TaskEvent; -import org.openjump.core.model.TaskListener; -import org.openjump.core.ui.plugin.datastore.WritableDataStoreDataSource; -import javax.swing.*; -import javax.swing.event.InternalFrameAdapter; -import javax.swing.event.InternalFrameEvent; -import javax.swing.event.InternalFrameListener; -import java.awt.*; - /** * DataStoreTransactionManagerPlugIn is a ToolBox containing: * <ul> @@ -37,8 +34,8 @@ */ public class DataStoreTransactionManagerPlugIn extends ToolboxPlugIn implements WorkbenchContextReference { - final Logger LOG = Logger.getLogger(DataStoreTransactionManagerPlugIn.class); + private static final String KEY = DataStoreTransactionManagerPlugIn.class.getName(); private static final String INSTANCE_KEY = KEY + " - INSTANCE"; @@ -71,7 +68,7 @@ context.getWorkbenchFrame().addTaskListener(new TaskListener() { public void taskAdded(TaskEvent taskEvent) { final Task task = taskEvent.getTask(); - LOG.info("Task added : " + task.getName()); + Logger.info("Task added : " + task.getName()); java.util.List<Layer> layers = task.getLayerManager().getLayers(); for (Layer layer : layers) { if (layer.getDataSourceQuery().getDataSource() instanceof WritableDataStoreDataSource) { @@ -82,7 +79,7 @@ } public void taskLoaded(TaskEvent taskEvent) { final Task task = taskEvent.getTask(); - LOG.info("Task loaded : " + task.getName()); + Logger.info("Task loaded : " + task.getName()); java.util.List<Layer> layers = taskEvent.getTask().getLayerManager().getLayers(); for (Layer layer : layers) { if (layer.getDataSourceQuery().getDataSource() instanceof WritableDataStoreDataSource) { @@ -110,7 +107,7 @@ getTaskFrame().addInternalFrameListener(new InternalFrameAdapter() { @Override public void internalFrameActivated(InternalFrameEvent e) { - LOG.trace("Task frame activated " + e.getInternalFrame().getTitle()); + Logger.trace("Task frame activated " + e.getInternalFrame().getTitle()); if (getTransactionManagerPanel() != null) { getTransactionManagerPanel().updateListener(task); } Modified: core/trunk/src/org/openjump/core/ui/plugin/datastore/transaction/Evolution.java =================================================================== --- core/trunk/src/org/openjump/core/ui/plugin/datastore/transaction/Evolution.java 2016-01-10 18:45:58 UTC (rev 4767) +++ core/trunk/src/org/openjump/core/ui/plugin/datastore/transaction/Evolution.java 2016-01-10 18:47:43 UTC (rev 4768) @@ -1,19 +1,19 @@ package org.openjump.core.ui.plugin.datastore.transaction; +import java.util.Arrays; +import java.util.Set; + import com.vividsolutions.jump.I18N; import com.vividsolutions.jump.feature.Feature; -import org.apache.log4j.Logger; +import com.vividsolutions.jump.workbench.Logger; -import java.util.Arrays; -import java.util.Set; - /** * A Feature Evolution. It keeps a clone of the previous state of the feature * ( null for a creation) and the new state of the feature (null for a suppression). */ public class Evolution { - final Logger LOG = Logger.getLogger(Evolution.class); + private final static String KEY = Evolution.class.getName(); public static enum Type {CREATION, MODIFICATION, SUPPRESSION} @@ -74,8 +74,8 @@ if (type1 != Type.SUPPRESSION && type2 != Type.CREATION && !Arrays.equals(evo1New.getAttributes(), evo2Old.getAttributes())) { - LOG.info("Try to merge : " + previous); - LOG.info(" with : " + this); + Logger.info("Try to merge : " + previous); + Logger.info(" with : " + this); throw new EvolutionOperationException(KEY + I18N.get(".cannot-merge-non-consecutive-evolutions")); } Modified: core/trunk/src/org/openjump/core/ui/plugin/datastore/transaction/TransactionManagerPanel.java =================================================================== --- core/trunk/src/org/openjump/core/ui/plugin/datastore/transaction/TransactionManagerPanel.java 2016-01-10 18:45:58 UTC (rev 4767) +++ core/trunk/src/org/openjump/core/ui/plugin/datastore/transaction/TransactionManagerPanel.java 2016-01-10 18:47:43 UTC (rev 4768) @@ -1,22 +1,40 @@ package org.openjump.core.ui.plugin.datastore.transaction; +import java.awt.GridBagConstraints; +import java.awt.GridBagLayout; +import java.awt.Insets; +import java.awt.event.ActionEvent; +import java.awt.event.ActionListener; +import java.util.Collection; + +import javax.swing.BoxLayout; +import javax.swing.JButton; +import javax.swing.JInternalFrame; +import javax.swing.JLabel; +import javax.swing.JPanel; +import javax.swing.JScrollPane; +import javax.swing.JTextArea; +import javax.swing.ScrollPaneConstants; + +import org.openjump.core.ui.plugin.datastore.WritableDataStoreDataSource; + import com.vividsolutions.jump.I18N; import com.vividsolutions.jump.io.datasource.DataSource; import com.vividsolutions.jump.workbench.JUMPWorkbench; +import com.vividsolutions.jump.workbench.Logger; import com.vividsolutions.jump.workbench.WorkbenchContext; -import com.vividsolutions.jump.workbench.model.*; +import com.vividsolutions.jump.workbench.model.FeatureEvent; +import com.vividsolutions.jump.workbench.model.Layer; +import com.vividsolutions.jump.workbench.model.LayerAdapter; +import com.vividsolutions.jump.workbench.model.LayerEvent; +import com.vividsolutions.jump.workbench.model.LayerEventType; +import com.vividsolutions.jump.workbench.model.LayerListener; +import com.vividsolutions.jump.workbench.model.LayerManager; +import com.vividsolutions.jump.workbench.model.Task; import com.vividsolutions.jump.workbench.ui.ErrorHandler; import com.vividsolutions.jump.workbench.ui.TaskFrame; import com.vividsolutions.jump.workbench.ui.plugin.WorkbenchContextReference; -import org.apache.log4j.Logger; -import org.openjump.core.ui.plugin.datastore.WritableDataStoreDataSource; -import javax.swing.*; -import java.awt.*; -import java.awt.event.ActionEvent; -import java.awt.event.ActionListener; -import java.util.Collection; - /** * Panel displaying current uncommitted edits and including the commit button. */ @@ -24,8 +42,8 @@ private static final String KEY = TransactionManagerPanel.class.getName(); - final Logger LOG = Logger.getLogger(TransactionManagerPanel.class); + final DataStoreTransactionManager transactionManager; final ErrorHandler errorHandler; final JTextArea textArea; @@ -125,7 +143,7 @@ updateTextArea(task); } else { - LOG.error(I18N.get(KEY + ".layer-with-irrelevant-datastore-datasource")); + Logger.error(I18N.get(KEY + ".layer-with-irrelevant-datastore-datasource")); } } } Modified: core/trunk/src/org/openjump/core/ui/plugin/mousemenu/category/SetCategoryVisibilityPlugIn.java =================================================================== --- core/trunk/src/org/openjump/core/ui/plugin/mousemenu/category/SetCategoryVisibilityPlugIn.java 2016-01-10 18:45:58 UTC (rev 4767) +++ core/trunk/src/org/openjump/core/ui/plugin/mousemenu/category/SetCategoryVisibilityPlugIn.java 2016-01-10 18:47:43 UTC (rev 4768) @@ -21,7 +21,7 @@ import javax.swing.JComponent; import javax.swing.JPopupMenu; -import org.apache.log4j.Logger; + import org.openjump.core.apitools.PlugInContextTools; import com.vividsolutions.jump.I18N; @@ -56,7 +56,7 @@ protected static SetCategoryVisibilityPlugIn instance= null; - private static final Logger LOG = Logger.getLogger(SetCategoryVisibilityPlugIn.class); + /** * Constructor needed to load PlugIn from classes, should NOT be used by any other Modified: core/trunk/src/org/openjump/core/ui/plugin/raster/RasterImageLayerPropertiesPlugIn.java =================================================================== --- core/trunk/src/org/openjump/core/ui/plugin/raster/RasterImageLayerPropertiesPlugIn.java 2016-01-10 18:45:58 UTC (rev 4767) +++ core/trunk/src/org/openjump/core/ui/plugin/raster/RasterImageLayerPropertiesPlugIn.java 2016-01-10 18:47:43 UTC (rev 4768) @@ -25,7 +25,7 @@ import javax.swing.JTabbedPane; import javax.swing.border.Border; -import org.apache.log4j.Logger; + import org.openjump.core.apitools.LayerTools; import org.openjump.core.rasterimage.RasterImageLayer; import org.openjump.core.rasterimage.TiffTags.TiffReadingException; @@ -169,7 +169,7 @@ .get("org.openjump.core.ui.plugin.raster.RasterImageLayerPropertiesPlugIn.ymax"); private final static String UNSPECIFIED = I18N .get("coordsys.CoordinateSystem.unspecified"); - private static final Logger LOGGER = Logger.getLogger(HTMLFrame.class); + private Envelope extent; /* Modified: core/trunk/src/org/openjump/core/ui/plugin/raster/nodata/ChangeRangeValuesToNoDataPlugIn.java =================================================================== --- core/trunk/src/org/openjump/core/ui/plugin/raster/nodata/ChangeRangeValuesToNoDataPlugIn.java 2016-01-10 18:45:58 UTC (rev 4767) +++ core/trunk/src/org/openjump/core/ui/plugin/raster/nodata/ChangeRangeValuesToNoDataPlugIn.java 2016-01-10 18:47:43 UTC (rev 4768) @@ -24,7 +24,7 @@ import javax.swing.JPanel; import javax.swing.JTextField; -import org.apache.log4j.Logger; + import org.openjump.core.apitools.LayerTools; import org.openjump.core.rasterimage.RasterImageIOUtils; import org.openjump.core.rasterimage.RasterImageLayer; @@ -59,8 +59,7 @@ * Substitute export to .flt file to .asc file * @date 2015_15_11 (Giuseppe Aruta) Improved GUI */ - private static final Logger LOGGER = Logger - .getLogger(ChangeNoDataValuePlugIn.class); + private Properties properties = null; private String byteOrder = "LSBFIRST"; private static String propertiesFile = LoadSextanteRasterImagePlugIn Modified: core/trunk/src/org/openjump/core/ui/plugin/style/ImportSLDPlugIn.java =================================================================== --- core/trunk/src/org/openjump/core/ui/plugin/style/ImportSLDPlugIn.java 2016-01-10 18:45:58 UTC (rev 4767) +++ core/trunk/src/org/openjump/core/ui/plugin/style/ImportSLDPlugIn.java 2016-01-10 18:47:43 UTC (rev 4768) @@ -70,7 +70,6 @@ import javax.swing.JPopupMenu; import javax.xml.parsers.DocumentBuilderFactory; -import org.apache.log4j.Logger; import org.openjump.core.ui.images.IconLoader; import org.openjump.core.ui.swing.SelectFromListPanel; import org.w3c.dom.Document; @@ -78,8 +77,9 @@ import com.vividsolutions.jump.feature.AttributeType; import com.vividsolutions.jump.feature.FeatureSchema; import com.vividsolutions.jump.util.Blackboard; +import com.vividsolutions.jump.util.Range; import com.vividsolutions.jump.util.Range.RangeTreeMap; -import com.vividsolutions.jump.util.Range; +import com.vividsolutions.jump.workbench.Logger; import com.vividsolutions.jump.workbench.WorkbenchContext; import com.vividsolutions.jump.workbench.model.Layer; import com.vividsolutions.jump.workbench.model.Layerable; @@ -88,14 +88,13 @@ import com.vividsolutions.jump.workbench.plugin.EnableCheckFactory; import com.vividsolutions.jump.workbench.plugin.MultiEnableCheck; import com.vividsolutions.jump.workbench.plugin.PlugInContext; +import com.vividsolutions.jump.workbench.ui.OKCancelDialog; import com.vividsolutions.jump.workbench.ui.OKCancelDialog.Validator; -import com.vividsolutions.jump.workbench.ui.OKCancelDialog; import com.vividsolutions.jump.workbench.ui.WorkbenchFrame; import com.vividsolutions.jump.workbench.ui.plugin.FeatureInstaller; import com.vividsolutions.jump.workbench.ui.renderer.style.BasicStyle; import com.vividsolutions.jump.workbench.ui.renderer.style.ColorThemingStyle; import com.vividsolutions.jump.workbench.ui.renderer.style.LabelStyle; -import com.vividsolutions.jump.workbench.ui.renderer.style.Style; import com.vividsolutions.jump.workbench.ui.renderer.style.VertexStyle; /** @@ -108,8 +107,6 @@ */ public class ImportSLDPlugIn extends AbstractPlugIn { - private static Logger LOG = getLogger(ImportSLDPlugIn.class); - public static final ImageIcon ICON = IconLoader.icon("sld_in_16.png"); @Override @@ -190,7 +187,7 @@ try { if (cts.getAttributeValueToLabelMap().keySet().iterator().next() instanceof Range) { - LOG.debug("Color theming values are ranges."); + Logger.debug("Color theming values are ranges."); //RangeTreeMap<Range, BasicStyle> map = new RangeTreeMap<Range, BasicStyle>(); //RangeTreeMap<Range, String> labelMap = new RangeTreeMap<Range, String>(); RangeTreeMap<Object, BasicStyle> map = new RangeTreeMap<Object, BasicStyle>(); @@ -202,7 +199,7 @@ Map<Object, String> oldLabelMap = cts.getAttributeValueToLabelMap(); if (c.equals(Integer.class)) { - LOG.debug("Color theming values are ranges of integers."); + Logger.debug("Color theming values are ranges of integers."); for (Object k : cts.getAttributeValueToBasicStyleMap().keySet()) { Range r = (Range) k; Range newRange = new Range(Integer.valueOf((String) r.getMin()), r.isIncludingMin(), @@ -213,7 +210,7 @@ } if (c.equals(Double.class)) { - LOG.debug("Color theming values are ranges of doubles."); + Logger.debug("Color theming values are ranges of doubles."); for (Object k : cts.getAttributeValueToBasicStyleMap().keySet()) { Range r = (Range) k; Range newRange = new Range(Double.valueOf((String) r.getMin()), r.isIncludingMin(), Double @@ -223,8 +220,8 @@ } } - if (LOG.isDebugEnabled()) { - LOG.debug("Color theming values are ranges of java class " + c + "."); + if (Logger.isDebugEnabled()) { + Logger.debug("Color theming values are ranges of java class " + c + "."); } cts.setAttributeValueToBasicStyleMap(map); @@ -235,13 +232,13 @@ } catch (NumberFormatException e) { throw e; } catch (Exception e) { - LOG.debug("Unknown error: ", e); + Logger.debug("Unknown error: ", e); // ignore, probably no elements in the map return; } if (c.equals(Integer.class)) { - LOG.debug("Color theming values are integers."); + Logger.debug("Color theming values are integers."); Map<Object, BasicStyle> map = new TreeMap<Object, BasicStyle>(); Map<?, ?> oldMap = cts.getAttributeValueToBasicStyleMap(); Map<Object, String> labelMap = new TreeMap<Object, String>(); @@ -255,7 +252,7 @@ } if (c.equals(Double.class)) { - LOG.debug("Color theming values are doubles."); + Logger.debug("Color theming values are doubles."); Map<Object, BasicStyle> map = new TreeMap<Object, BasicStyle>(); Map<?, ?> oldMap = cts.getAttributeValueToBasicStyleMap(); Map<Object, String> labelMap = new TreeMap<Object, String>(); @@ -268,8 +265,8 @@ cts.setAttributeValueToLabelMap(labelMap); } - if (LOG.isDebugEnabled()) { - LOG.debug("Color theming values are of java type " + c + "."); + if (Logger.isDebugEnabled()) { + Logger.debug("Color theming values are of java type " + c + "."); } return; @@ -279,7 +276,7 @@ showMessageDialog(frame, getMessage( "org.openjump.core.ui.plugin.style.ImportSLDPlugIn.Error-reading-styles", new Object[] { e .getLocalizedMessage() }), get("com.vividsolutions.wms.WMService.Error"), ERROR_MESSAGE); - LOG.debug("Probably unknown attribute name: ", e); + Logger.debug("Probably unknown attribute name: ", e); return; } } Modified: core/trunk/src/org/openjump/core/ui/plugin/view/SuperZoomPanTool.java =================================================================== --- core/trunk/src/org/openjump/core/ui/plugin/view/SuperZoomPanTool.java 2016-01-10 18:45:58 UTC (rev 4767) +++ core/trunk/src/org/openjump/core/ui/plugin/view/SuperZoomPanTool.java 2016-01-10 18:47:43 UTC (rev 4768) @@ -1,38 +1,39 @@ package org.openjump.core.ui.plugin.view; -import com.vividsolutions.jts.geom.Coordinate; +import java.awt.BasicStroke; import java.awt.Color; import java.awt.Cursor; +import java.awt.Dimension; +import java.awt.FontMetrics; +import java.awt.Graphics2D; +import java.awt.Image; import java.awt.Point; +import java.awt.Rectangle; import java.awt.Shape; +import java.awt.Transparency; import java.awt.event.ActionEvent; +import java.awt.event.ActionListener; import java.awt.event.MouseEvent; +import java.awt.event.MouseWheelEvent; +import java.awt.event.MouseWheelListener; import java.awt.geom.NoninvertibleTransformException; import java.awt.geom.Point2D; +import java.awt.image.BufferedImage; import javax.swing.Icon; +import javax.swing.SwingUtilities; +import javax.swing.Timer; +import org.openjump.core.ui.util.ScreenScale; + +import com.vividsolutions.jts.geom.Coordinate; import com.vividsolutions.jts.geom.Envelope; import com.vividsolutions.jump.I18N; +import com.vividsolutions.jump.workbench.Logger; import com.vividsolutions.jump.workbench.ui.LayerViewPanel; import com.vividsolutions.jump.workbench.ui.Viewport; import com.vividsolutions.jump.workbench.ui.cursortool.DragTool; import com.vividsolutions.jump.workbench.ui.images.IconLoader; -import java.awt.BasicStroke; -import java.awt.Dimension; -import java.awt.FontMetrics; -import java.awt.Graphics2D; -import java.awt.Image; -import java.awt.Rectangle; -import java.awt.Transparency; -import java.awt.event.ActionListener; -import java.awt.event.MouseWheelEvent; -import java.awt.event.MouseWheelListener; -import java.awt.image.BufferedImage; -import javax.swing.SwingUtilities; -import javax.swing.Timer; -import org.apache.log4j.Logger; -import org.openjump.core.ui.util.ScreenScale; /** * This tool have the following functions: @@ -476,10 +477,10 @@ redrawImage(); break; default: - Logger.getLogger(SuperZoomPanTool.class).warn("Unknown indicatorMode " + getIndicatorMode() + "!"); + Logger.warn("Unknown indicatorMode " + getIndicatorMode() + "!"); } } catch (Exception e) { - Logger.getLogger(SuperZoomPanTool.class).error("Unable to redraw the visual indicator!", e); + Logger.error("Unable to redraw the visual indicator!", e); } } Modified: core/trunk/src/org/openjump/core/ui/plugin/wms/ZoomToWMSLayerPlugIn.java =================================================================== --- core/trunk/src/org/openjump/core/ui/plugin/wms/ZoomToWMSLayerPlugIn.java 2016-01-10 18:45:58 UTC (rev 4767) +++ core/trunk/src/org/openjump/core/ui/plugin/wms/ZoomToWMSLayerPlugIn.java 2016-01-10 18:47:43 UTC (rev 4768) @@ -4,23 +4,25 @@ import javax.swing.ImageIcon; +import com.vividsolutions.jts.geom.Envelope; import com.vividsolutions.jump.I18N; +import com.vividsolutions.jump.geom.EnvelopeUtil; +import com.vividsolutions.jump.workbench.Logger; import com.vividsolutions.jump.workbench.WorkbenchContext; -import com.vividsolutions.jump.workbench.plugin.*; -import com.vividsolutions.jump.workbench.model.*; -import com.vividsolutions.wms.*; -import com.vividsolutions.jts.geom.*; +import com.vividsolutions.jump.workbench.model.WMSLayer; +import com.vividsolutions.jump.workbench.plugin.AbstractPlugIn; +import com.vividsolutions.jump.workbench.plugin.EnableCheck; +import com.vividsolutions.jump.workbench.plugin.EnableCheckFactory; +import com.vividsolutions.jump.workbench.plugin.MultiEnableCheck; +import com.vividsolutions.jump.workbench.plugin.PlugInContext; @@ Diff output truncated at 100000 characters. @@ ------------------------------------------------------------------------------ Site24x7 APM Insight: Get Deep Visibility into Application Performance APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month Monitor end-to-end web transactions and take corrective actions now Troubleshoot faster and improve end-user experience. Signup Now! http://pubads.g.doubleclick.net/gampad/clk?id=267308311&iu=/4140 _______________________________________________ Jump-pilot-devel mailing list Jump-pilot-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel