fineract-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
Subject [fineract] branch develop updated: remove bad 'default:' in SavingsEnumerations switch
Date Sun, 07 Jun 2020 16:40:18 GMT
This is an automated email from the ASF dual-hosted git repository.

vorburger pushed a commit to branch develop
in repository

The following commit(s) were added to refs/heads/develop by this push:
     new 4528231  remove bad 'default:' in SavingsEnumerations switch
4528231 is described below

commit 4528231fd106acb57d70141b53f01d426b0f042e
Author: Michael Vorburger <>
AuthorDate: Sun Jun 7 15:08:47 2020 +0200

    remove bad 'default:' in SavingsEnumerations switch
    This default: is "wrong", because it's not required - the switch,
    in this particular case, *IS* actually already handling all cases.
    But if in the future someone were to add an additional new
    SavingsAccountTransactionType, then we would want Error Prone to
    fail to signal that it needs to be handled here - but the "default:"
    would "hide" that.
    This is a follow-up to
    for FINERACT-822, and also related to the discussion started on
    There originally used to be a "default:" in the middle of the switch,
    that obviously WAS completely wrong and an old copy-paste bug.
 .../apache/fineract/portfolio/savings/service/   | 1 -
 1 file changed, 1 deletion(-)

diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/service/
index 071b2e4..ed604d4 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/service/
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/service/
@@ -199,7 +199,6 @@ public class SavingsEnumerations {
                 optionData = new SavingsAccountTransactionEnumData(SavingsAccountTransactionType.AMOUNT_RELEASE.getValue().longValue(),
                         SavingsAccountTransactionType.AMOUNT_RELEASE.getCode(), "Release
-            default:
         return optionData;

View raw message