diff flys-artifacts/src/main/java/de/intevation/flys/utils/FLYSUtils.java @ 2081:aa9cc91c8193

#444 Take care on NullPointerExceptions in getWQMode. Take 'false' as default value if no 'wq_free' item is existing. flys-artifacts/trunk@3597 c6561f87-3c4e-4783-a992-168aeb5c3f6f
author Ingo Weinzierl <ingo.weinzierl@intevation.de>
date Thu, 05 Jan 2012 14:15:07 +0000
parents 0cf60d64895c
children bf67eb014443
line wrap: on
line diff
--- a/flys-artifacts/src/main/java/de/intevation/flys/utils/FLYSUtils.java	Thu Jan 05 13:55:17 2012 +0000
+++ b/flys-artifacts/src/main/java/de/intevation/flys/utils/FLYSUtils.java	Thu Jan 05 14:15:07 2012 +0000
@@ -157,8 +157,14 @@
      * @return an enum WQ_MODE.
      */
     public static WQ_MODE getWQMode(FLYSArtifact flys) {
+        if (flys == null) {
+            return WQ_MODE.NONE;
+        }
+
         String  mode = flys.getDataAsString("wq_mode");
-        boolean free = flys.getDataAsBoolean("wq_free");
+        Boolean free = flys.getDataAsBoolean("wq_free");
+
+        free = free != null ? free : false;
 
         if (mode != null && mode.equals("Q")) {
             return free ? WQ_MODE.QFREE : WQ_MODE.QGAUGE;

http://dive4elements.wald.intevation.org