# HG changeset patch # User Felix Wolfsteller # Date 1330690540 0 # Node ID 3c26e16ae6c22cca210a18464031fa363bd1a4ff # Parent 67f362883fe42bca49a13223c4bbb944635c70de Fix or workaround flys/issue632, avoid NPE. flys-backend/trunk@4112 c6561f87-3c4e-4783-a992-168aeb5c3f6f diff -r 67f362883fe4 -r 3c26e16ae6c2 flys-backend/ChangeLog --- a/flys-backend/ChangeLog Tue Feb 14 16:37:19 2012 +0000 +++ b/flys-backend/ChangeLog Fri Mar 02 12:15:40 2012 +0000 @@ -1,3 +1,10 @@ +2012-03-02 Felix Wolfsteller + + Fix or workaround flys/issue632 . + + * src/main/java/de/intevation/flys/model/Wst.java: Avoid NPE when + trying to get min/max q values. + 2012-02-14 Sascha L. Teichmann * src/main/java/de/intevation/flys/model/DischargeTable.java: diff -r 67f362883fe4 -r 3c26e16ae6c2 flys-backend/src/main/java/de/intevation/flys/model/Wst.java --- a/flys-backend/src/main/java/de/intevation/flys/model/Wst.java Tue Feb 14 16:37:19 2012 +0000 +++ b/flys-backend/src/main/java/de/intevation/flys/model/Wst.java Fri Mar 02 12:15:40 2012 +0000 @@ -42,7 +42,7 @@ public static final String SQL_SELECT_MINMAX = "select min(q) as minQ, max(q) as maxQ from wst_q_values " + - "where wst_id = :wst and not (a > :km or b < :km)"; + "where wst_id = :wst and not (a > :km or b < :km)"; public Wst() { } @@ -197,12 +197,18 @@ List minmaxQ = sqlQuery.list(); + if (minmaxQ.isEmpty()) { return null; } Object[] mm = minmaxQ.get(0); + if (mm[0] == null || mm[1] == null) { + logger.warning ("No min/max Q for km " + km + " found."); + return null; + } + return new double[] { (Double) mm[0], (Double) mm[1] }; } }