wkt-wickedcharts: NPE when invalid data is provided

[incodehq]

wkt-wickedcharts: NPE when invalid data is provided

[danhaywood]

martin-g:

I use the following fix locally to avoid the NPE. Not sure it is the correct one though

     private BigDecimal maxOf(BigDecimal max, final BigDecimal summaryMax) {
-        return max != null ? max.max(summaryMax) : summaryMax;
+        return max != null
+            ? summaryMax != null
+                ? max.max(summaryMax)
+                : max
+            : BigDecimal.ZERO;
     }

     private BigDecimal minOf(BigDecimal min, final BigDecimal summaryMin) {
-        return min != null ? min.min(summaryMin) : summaryMin;
+        return min != null
+            ? summaryMin != null
+                ? min.min(summaryMin)
+                : min
+            : BigDecimal.ZERO;
     }

[danhaywood]

danhaywood:

possibly not; what if every value is null? The result ought to be null,
but I think this code might give BigDecimal.ZERO.

On 9 October 2014 09:02, Martin Grigorov [email protected] wrote:

I use the following fix locally to avoid the NPE. Not sure it is the
correct one though

 private BigDecimal maxOf(BigDecimal max, final BigDecimal summaryMax) {-        return max != null ? max.max(summaryMax) : summaryMax;+        return max != null+            ? summaryMax != null+                ? max.max(summaryMax)+                : max+            : BigDecimal.ZERO;
 }

 private BigDecimal minOf(BigDecimal min, final BigDecimal summaryMin) {-        return min != null ? min.min(summaryMin) : summaryMin;+        return min != null+            ? summaryMin != null+                ? min.min(summaryMin)+                : min+            : BigDecimal.ZERO;
 }

—
Reply to this email directly or view it on GitHub
isisaddons-legacy/isis-wicket-wickedcharts#2 (comment)
.

[danhaywood]

martin-g:

returning null instead of BigDecimal.ZERO seems to do the job as well