[PATCH] Fixed year extraction from date and message format for the years without group digit

Wald Commits scm-commit at wald.intevation.org
Fri Aug 17 09:22:29 CEST 2018


# HG changeset patch
# User mschaefer
# Date 1534490536 -7200
# Node ID a31cb674ddd1ecf35411ab923db029ac5a10abb4
# Parent  9fe26f525ae3b300c89e35310493c5d2484b170f
Fixed year extraction from date and message format for the years without group digit

diff -r 9fe26f525ae3 -r a31cb674ddd1 artifacts/src/main/java/org/dive4elements/river/artifacts/services/DynamicMainValuesTimeRangeDeterminationService.java
--- a/artifacts/src/main/java/org/dive4elements/river/artifacts/services/DynamicMainValuesTimeRangeDeterminationService.java	Thu Aug 16 18:15:54 2018 +0200
+++ b/artifacts/src/main/java/org/dive4elements/river/artifacts/services/DynamicMainValuesTimeRangeDeterminationService.java	Fri Aug 17 09:22:16 2018 +0200
@@ -18,6 +18,7 @@
 import org.dive4elements.artifacts.CallMeta;
 import org.dive4elements.artifacts.GlobalContext;
 import org.dive4elements.artifacts.common.ArtifactNamespaceContext;
+import org.dive4elements.artifacts.common.utils.DateUtils;
 import org.dive4elements.artifacts.common.utils.XMLUtils;
 import org.dive4elements.artifacts.common.utils.XMLUtils.ElementCreator;
 import org.dive4elements.river.artifacts.resources.Resources;
@@ -113,8 +114,8 @@
         // common Range and correct errorMsg
         final List<GaugeInfoResult.GaugeInfo> gaugeResultsSecondTurn = new ArrayList<>();
         for (final GaugeInfoResult.GaugeInfo gi : gaugeResults) {
-            gaugeResultsSecondTurn
-                    .add(new GaugeInfoResult.GaugeInfo(gi.errorMsg, gi.gauge, gi.startdate != null ? min : null, gi.enddate != null ? max : null));
+            gaugeResultsSecondTurn.add(new GaugeInfoResult.GaugeInfo(gi.errorMsg, gi.gauge, gi.startdate != null ? min : null,
+                    gi.enddate != null ? max : null));
         }
         if (globalErrorMsg.isEmpty() && (min.getTime() > max.getTime()))
             globalErrorMsg = getMsg(meta, "bundu.wst.gauge_timeranges_disjoint");
@@ -124,12 +125,8 @@
     }
 
     private String makeDoesNotCoverErrorMsg(final Date start, final Date end, final CallMeta meta) {
-        final Calendar cal = Calendar.getInstance();
-        return Resources.getMsg(meta, "bundu.wst.range_does_not_cover", new Object[] { getYear(start, cal), getYear(end, cal) });
-    }
-
-    private String getYear(final Date d, final Calendar cal) {
-        return String.valueOf(cal.get(Calendar.YEAR));
+        final String msgkey = "bundu.wst.range_does_not_cover";
+        return Resources.getMsg(meta, msgkey, msgkey, DateUtils.getYearFromDate(start), DateUtils.getYearFromDate(end));
     }
 
     @Override
diff -r 9fe26f525ae3 -r a31cb674ddd1 artifacts/src/main/resources/messages.properties
--- a/artifacts/src/main/resources/messages.properties	Thu Aug 16 18:15:54 2018 +0200
+++ b/artifacts/src/main/resources/messages.properties	Fri Aug 17 09:22:16 2018 +0200
@@ -1295,7 +1295,7 @@
 bundu.wst.export.csv.meta.header.bezugswst = Bezugswasserstand
 bundu.wst.export.pdf.meta.header.bezugswst = Bezugs-wasserstand
 bundu.wst_no_data_at_all = Es liegt keine Abflussganglinie vor ({0})
-bundu.wst.range_does_not_cover = Die Abflussganglinie ({0}-{1}) deckt nicht den Bezugszeitraum ab. 
+bundu.wst.range_does_not_cover = Die Abflussganglinie ({0,number,#}-{1,number,#}) deckt nicht den Bezugszeitraum ab. 
 bundu.wst.error_reading_gauges = Fehler beim Abruf des Pegels aus der Datenbank.
 bundu.wst.gauge_timeranges_disjoint = Der Bezugszeitraum konnte nicht auf die vorhandenen Abflusswerte eingeschr\u00e4nkt werden. 
 
diff -r 9fe26f525ae3 -r a31cb674ddd1 artifacts/src/main/resources/messages_de.properties
--- a/artifacts/src/main/resources/messages_de.properties	Thu Aug 16 18:15:54 2018 +0200
+++ b/artifacts/src/main/resources/messages_de.properties	Fri Aug 17 09:22:16 2018 +0200
@@ -1295,7 +1295,7 @@
 bundu.wst.export.csv.meta.header.bezugswst = Bezugswasserstand
 bundu.wst.export.pdf.meta.header.bezugswst = Bezugs-wasserstand
 bundu.wst_no_data_at_all = Es liegt keine Abflussganglinie vor ({0})
-bundu.wst.range_does_not_cover = Die Abflussganglinie ({0}-{1}) deckt nicht den Bezugszeitraum ab. 
+bundu.wst.range_does_not_cover = Die Abflussganglinie ({0,number,#}-{1,number,#}) deckt nicht den Bezugszeitraum ab. 
 bundu.wst.error_reading_gauges = Fehler beim Abruf des Pegels aus der Datenbank.
 bundu.wst.gauge_timeranges_disjoint = Der Bezugszeitraum konnte nicht auf die vorhandenen Abflusswerte eingeschr\u00e4nkt werden. 
 


More information about the Dive4Elements-commits mailing list