diff flys-artifacts/src/main/java/de/intevation/flys/utils/GeometryUtils.java @ 3303:96daa8a4c48f

Fix and workaround NullPointerExceptions flys-artifacts/trunk@4988 c6561f87-3c4e-4783-a992-168aeb5c3f6f
author Christian Lins <christian.lins@intevation.de>
date Fri, 13 Jul 2012 10:54:56 +0000
parents c8f670ae96e8
children 1b41dc00b1f7
line wrap: on
line diff
--- a/flys-artifacts/src/main/java/de/intevation/flys/utils/GeometryUtils.java	Fri Jul 13 10:44:18 2012 +0000
+++ b/flys-artifacts/src/main/java/de/intevation/flys/utils/GeometryUtils.java	Fri Jul 13 10:54:56 2012 +0000
@@ -76,7 +76,7 @@
     public static String getRiverBounds(String rivername) {
         Envelope env = getRiverBoundary(rivername);
 
-        if (env == null) {
+        if (env != null) {
             return jtsBoundsToOLBounds(env);
         }
 
@@ -91,11 +91,12 @@
      * @return the OpenLayers boundary of <i>env</i>.
      */
     public static String jtsBoundsToOLBounds(Envelope env) {
-        return "" +
-            env.getMinX() + " " +
-            env.getMinY() + " " +
-            env.getMaxX() + " " +
-            env.getMaxY();
+        StringBuilder buf = new StringBuilder();
+        buf.append(env.getMinX()); buf.append(' ');
+        buf.append(env.getMinY()); buf.append(' ');
+        buf.append(env.getMaxX()); buf.append(' ');
+        buf.append(env.getMaxY());
+        return buf.toString();
     }
 
     public static String createOLBounds(Geometry a, Geometry b) {

http://dive4elements.wald.intevation.org