# HG changeset patch # User Tim Englich # Date 1259073576 0 # Node ID f0f106c7b9064059b0acaa5f0a9327cf7833c4a8 # Parent e7f1c79bf9cdb666502d758911e416e7d2b191c8 Fixed some broken Method-Extentions. Added some null-value checks. gnv-artifacts/trunk@365 c6561f87-3c4e-4783-a992-168aeb5c3f6f diff -r e7f1c79bf9cd -r f0f106c7b906 gnv-artifacts/ChangeLog --- a/gnv-artifacts/ChangeLog Tue Nov 24 13:29:36 2009 +0000 +++ b/gnv-artifacts/ChangeLog Tue Nov 24 14:39:36 2009 +0000 @@ -1,5 +1,18 @@ 2009-11-24 Tim Englich + * src/test/ressources/timeseries/timeseries_step_06_out_chart.xml: + Added mime-type to get a propper output-result and not an NPE + * src/main/java/de/intevation/gnv/chart/AbstractXYLineChart.java (findValueTitle): + Added null-value check to prevent NPEs. + * src/main/java/de/intevation/gnv/transition/profile/verticalcrosssection/VerticalCrossSectionOutputTransition.java (getChart), + src/main/java/de/intevation/gnv/transition/profile/vertical/VerticalProfileOutputTransition.java (getChart), + src/main/java/de/intevation/gnv/transition/profile/horizontalcrosssection/HorizontalCrossSectionMeshOutputTransition.java (getChart), + src/main/java/de/intevation/gnv/transition/profile/horizontal/HorizontalProfileOutputTransition.java (getChart): + Changed Method-Signature to this one which the base method provide. + Now the Methods will be used again. + +2009-11-24 Tim Englich + * src/test/java/de/intevation/gnv/artifacts/TestCallContext.java (TestCallContext): Removed Compilationerrors that were triggert by API-Changes in Module Artifact-Database. diff -r e7f1c79bf9cd -r f0f106c7b906 gnv-artifacts/src/main/java/de/intevation/gnv/chart/AbstractXYLineChart.java --- a/gnv-artifacts/src/main/java/de/intevation/gnv/chart/AbstractXYLineChart.java Tue Nov 24 13:29:36 2009 +0000 +++ b/gnv-artifacts/src/main/java/de/intevation/gnv/chart/AbstractXYLineChart.java Tue Nov 24 14:39:36 2009 +0000 @@ -100,12 +100,14 @@ protected String findValueTitle(Collection values, String id) { log.debug("find description of dataset"); - Iterator it = values.iterator(); - while (it.hasNext()) { - KeyValueDescibeData data = (KeyValueDescibeData) it.next(); - - if (id.equals(data.getKey())) - return data.getValue(); + if (values != null){ + Iterator it = values.iterator(); + while (it.hasNext()) { + KeyValueDescibeData data = (KeyValueDescibeData) it.next(); + + if (id.equals(data.getKey())) + return data.getValue(); + } } return ""; } diff -r e7f1c79bf9cd -r f0f106c7b906 gnv-artifacts/src/main/java/de/intevation/gnv/transition/profile/horizontal/HorizontalProfileOutputTransition.java --- a/gnv-artifacts/src/main/java/de/intevation/gnv/transition/profile/horizontal/HorizontalProfileOutputTransition.java Tue Nov 24 13:29:36 2009 +0000 +++ b/gnv-artifacts/src/main/java/de/intevation/gnv/transition/profile/horizontal/HorizontalProfileOutputTransition.java Tue Nov 24 14:39:36 2009 +0000 @@ -65,10 +65,9 @@ super(); super.domainLable = "Distance [km]"; } - + @Override protected Chart getChart( ChartLabels chartLables, - ChartTheme chartTheme, Collection parameters, Collection measurements, Collection result, @@ -89,7 +88,7 @@ log.info("Chart not in cache yet."); chart = new HorizontalProfileChart( chartLables, - chartTheme, + createStyle(), parameters, measurements, result, diff -r e7f1c79bf9cd -r f0f106c7b906 gnv-artifacts/src/main/java/de/intevation/gnv/transition/profile/horizontalcrosssection/HorizontalCrossSectionMeshOutputTransition.java --- a/gnv-artifacts/src/main/java/de/intevation/gnv/transition/profile/horizontalcrosssection/HorizontalCrossSectionMeshOutputTransition.java Tue Nov 24 13:29:36 2009 +0000 +++ b/gnv-artifacts/src/main/java/de/intevation/gnv/transition/profile/horizontalcrosssection/HorizontalCrossSectionMeshOutputTransition.java Tue Nov 24 14:39:36 2009 +0000 @@ -46,10 +46,9 @@ super.domainLable = "Latitude [m]"; } - + @Override protected Chart getChart( ChartLabels chartLables, - ChartTheme chartTheme, Collection parameters, Collection measurements, Collection result, diff -r e7f1c79bf9cd -r f0f106c7b906 gnv-artifacts/src/main/java/de/intevation/gnv/transition/profile/vertical/VerticalProfileOutputTransition.java --- a/gnv-artifacts/src/main/java/de/intevation/gnv/transition/profile/vertical/VerticalProfileOutputTransition.java Tue Nov 24 13:29:36 2009 +0000 +++ b/gnv-artifacts/src/main/java/de/intevation/gnv/transition/profile/vertical/VerticalProfileOutputTransition.java Tue Nov 24 14:39:36 2009 +0000 @@ -38,16 +38,12 @@ super.domainLable = "Depth [m]"; } - protected Chart getChart( - ChartLabels chartLables, - ChartTheme chartTheme, - Collection parameters, - Collection measurements, - Collection result, - Collection dates, - Locale locale, - String uuid - ) { + + + @Override + protected Chart getChart(ChartLabels chartLables, Collection parameters, + Collection measurements, Collection result, + Collection dates, Locale locale, String uuid) { Chart chart = null; if (CACHE_CHART) { @@ -61,7 +57,7 @@ log.info("Chart not in cache yet."); chart = new VerticalProfileChart( chartLables, - chartTheme, + this.createStyle(), parameters, measurements, result, @@ -78,6 +74,8 @@ return chart; } + + /** * @see de.intevation.gnv.transition.timeseries.TimeSeriesOutputTransition#getStatisticsGenerator() */ diff -r e7f1c79bf9cd -r f0f106c7b906 gnv-artifacts/src/main/java/de/intevation/gnv/transition/profile/verticalcrosssection/VerticalCrossSectionOutputTransition.java --- a/gnv-artifacts/src/main/java/de/intevation/gnv/transition/profile/verticalcrosssection/VerticalCrossSectionOutputTransition.java Tue Nov 24 13:29:36 2009 +0000 +++ b/gnv-artifacts/src/main/java/de/intevation/gnv/transition/profile/verticalcrosssection/VerticalCrossSectionOutputTransition.java Tue Nov 24 14:39:36 2009 +0000 @@ -55,10 +55,9 @@ super.domainLable = "Depth [m]"; } - + @Override protected Chart getChart( ChartLabels chartLables, - ChartTheme chartTheme, Collection parameters, Collection measurements, Collection result, diff -r e7f1c79bf9cd -r f0f106c7b906 gnv-artifacts/src/test/ressources/timeseries/timeseries_step_06_out_chart.xml --- a/gnv-artifacts/src/test/ressources/timeseries/timeseries_step_06_out_chart.xml Tue Nov 24 13:29:36 2009 +0000 +++ b/gnv-artifacts/src/test/ressources/timeseries/timeseries_step_06_out_chart.xml Tue Nov 24 14:39:36 2009 +0000 @@ -4,6 +4,7 @@ +