danielales2541 commented on code in PR #4554:
URL: https://github.com/apache/fineract/pull/4554#discussion_r2042825552


##########
fineract-provider/src/main/java/org/apache/fineract/infrastructure/bulkimport/populator/recurringdeposit/RecurringDepositTransactionWorkbookPopulator.java:
##########
@@ -204,7 +205,12 @@ private void populateSavingsTable(Sheet 
savingsTransactionSheet, String dateForm
             row = savingsTransactionSheet.createRow(rowIndex++);
             writeString(TransactionConstants.LOOKUP_CLIENT_NAME_COL, row,
                     savingsAccount.getClientName() + "(" + 
savingsAccount.getClientId() + ")");
-            writeString(TransactionConstants.LOOKUP_ACCOUNT_NO_COL, row, 
savingsAccount.getAccountNo());
+            try {
+                BigDecimal accountNoAsBigDecimal = new 
BigDecimal(savingsAccount.getAccountNo());

Review Comment:
   This cell is taken as a reference but since one is a number and the other a 
string, it does not find the related product when converting both into numbers, 
it takes the product that is



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to