From 42ff4c1b1ca01ecfbff2c4688f971de6578574a4 Mon Sep 17 00:00:00 2001
From: Mohammed Nafees <nafees.technocool@gmail.com>
Date: Fri, 5 Dec 2014 01:22:44 +0530
Subject: [PATCH] MIFOSX-1550 Check for null principal amount when creating
 journal entry for loan write-off

---
 .../service/CashBasedAccountingProcessorForLoan.java  | 11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)

diff --git a/mifosng-provider/src/main/java/org/mifosplatform/accounting/journalentry/service/CashBasedAccountingProcessorForLoan.java b/mifosng-provider/src/main/java/org/mifosplatform/accounting/journalentry/service/CashBasedAccountingProcessorForLoan.java
index ea0bbd18f..8bc5f26cc 100755
--- a/mifosng-provider/src/main/java/org/mifosplatform/accounting/journalentry/service/CashBasedAccountingProcessorForLoan.java
+++ b/mifosng-provider/src/main/java/org/mifosplatform/accounting/journalentry/service/CashBasedAccountingProcessorForLoan.java
@@ -73,10 +73,13 @@ public class CashBasedAccountingProcessorForLoan implements AccountingProcessorF
              **/
             else if (loanTransactionDTO.getTransactionType().isWriteOff()) {
                 final BigDecimal principalAmount = loanTransactionDTO.getPrincipal();
-                this.helper.createCashBasedJournalEntriesAndReversalsForLoan(office, currencyCode,
-                        CASH_ACCOUNTS_FOR_LOAN.LOSSES_WRITTEN_OFF.getValue(), CASH_ACCOUNTS_FOR_LOAN.LOAN_PORTFOLIO.getValue(),
-                        loanProductId, paymentTypeId, loanId, transactionId, transactionDate, principalAmount,
-                        loanTransactionDTO.isReversed());
+                if (principalAmount != null && !(principalAmount.compareTo(BigDecimal.ZERO) == 0)) {
+                    this.helper.createCashBasedJournalEntriesAndReversalsForLoan(office, currencyCode,
+                            CASH_ACCOUNTS_FOR_LOAN.LOSSES_WRITTEN_OFF.getValue(), CASH_ACCOUNTS_FOR_LOAN.LOAN_PORTFOLIO.getValue(),
+                            loanProductId, paymentTypeId, loanId, transactionId, transactionDate, principalAmount,
+                            loanTransactionDTO.isReversed());
+
+                }
             } else if (loanTransactionDTO.getTransactionType().isInitiateTransfer()
                     || loanTransactionDTO.getTransactionType().isApproveTransfer()
                     || loanTransactionDTO.getTransactionType().isWithdrawTransfer()) {