Mercurial > dive4elements > river
changeset 4054:102050c4fc00
fix issue909: strange validation code. be fearles.
author | Felix Wolfsteller <felix.wolfsteller@intevation.de> |
---|---|
date | Mon, 08 Oct 2012 10:03:51 +0200 |
parents | f85f230c5c6c |
children | bbce9e1ef807 |
files | flys-artifacts/ChangeLog flys-artifacts/src/main/java/de/intevation/flys/artifacts/states/WDifferencesState.java flys-artifacts/src/main/java/de/intevation/flys/utils/FLYSUtils.java |
diffstat | 3 files changed, 8 insertions(+), 2 deletions(-) [+] |
line wrap: on
line diff
--- a/flys-artifacts/ChangeLog Mon Oct 08 09:52:21 2012 +0200 +++ b/flys-artifacts/ChangeLog Mon Oct 08 10:03:51 2012 +0200 @@ -1,3 +1,9 @@ +2012-10-08 Felix Wolfsteller <felix.wolfsteller@intevation.de> + + * flys-artifacts/src/main/java/de/intevation/flys/artifacts/states/WDifferencesState.java, + flys-artifacts/src/main/java/de/intevation/flys/utils/FLYSUtils.java: + Validate partial strings. + 2012-10-08 Felix Wolfsteller <felix.wolfsteller@intevation.de> * flys-artifacts/src/main/java/de/intevation/flys/artifacts/access/ExtremeAccess.java,
--- a/flys-artifacts/src/main/java/de/intevation/flys/artifacts/states/WDifferencesState.java Mon Oct 08 09:52:21 2012 +0200 +++ b/flys-artifacts/src/main/java/de/intevation/flys/artifacts/states/WDifferencesState.java Mon Oct 08 10:03:51 2012 +0200 @@ -137,7 +137,7 @@ // Validate the Data-Strings. for (String s: datas) { - if (!WaterlevelSelectState.isValueValid(winfo.getDataAsString("diffids"))) { + if (!WaterlevelSelectState.isValueValid(s)) { // TODO: escalate. } }
--- a/flys-artifacts/src/main/java/de/intevation/flys/utils/FLYSUtils.java Mon Oct 08 09:52:21 2012 +0200 +++ b/flys-artifacts/src/main/java/de/intevation/flys/utils/FLYSUtils.java Mon Oct 08 10:03:51 2012 +0200 @@ -738,7 +738,7 @@ // Validate the Data-Strings. for (String s: datas) { - if (!WaterlevelSelectState.isValueValid(winfo.getDataAsString("diffids"))) { + if (!WaterlevelSelectState.isValueValid(s)) { return ""; } }