You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@fineract.apache.org by "Michael Vorburger (Jira)" <ji...@apache.org> on 2020/09/23 21:19:00 UTC
[jira] [Commented] (FINERACT-987) Replace null values with zero for
bigDecimal values
[ https://issues.apache.org/jira/browse/FINERACT-987?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17201103#comment-17201103 ]
Michael Vorburger commented on FINERACT-987:
--------------------------------------------
https://github.com/apache/fineract/commit/5c3a89bb66c246cb89481148452bf4b16a5c6ab9
> Replace null values with zero for bigDecimal values
> ---------------------------------------------------
>
> Key: FINERACT-987
> URL: https://issues.apache.org/jira/browse/FINERACT-987
> Project: Apache Fineract
> Issue Type: Bug
> Components: Savings
> Reporter: Yemdjih Kaze Nasser
> Priority: Critical
>
> While trying to resolve https://github.com/apache/fineract/pull/915, I stumbled upon
> {code:java}
> org.apache.fineract.portfolio.savings.domain.SavingsAccountCharge since first historized release unspecified
> In class org.apache.fineract.portfolio.savings.domain.SavingsAccountCharge
> In method org.apache.fineract.portfolio.savings.domain.SavingsAccountCharge.percentageOf(BigDecimal, BigDecimal)
> Parameter value
> At SavingsAccountCharge.java:[lines 540-548]
> value must be non-null but is marked as nullable
> This parameter is always used in a way that requires it to be non-null, but the parameter is explicitly annotated as being Nullable. Either the use of the parameter or the annotation is wrong.
> {code}
> Seems to be related to this [line of code|https://github.com/apache/fineract/blob/beee2e5330b35f07ed16ba7019b57b8ab75e4e02/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/domain/SavingsAccountCharge.java#L478]
> which sets passes a null value to a method which does not handle null values. The reason for the use of null values is not clear to me but this is a glitch which be troublesome.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)