oleksii-novikov-onix commented on code in PR #5931:
URL: https://github.com/apache/fineract/pull/5931#discussion_r3381617011
##########
fineract-core/src/main/java/org/apache/fineract/commands/service/CommandWrapperBuilder.java:
##########
@@ -863,6 +864,14 @@ public CommandWrapperBuilder
discountFeeAdjustmentWorkingCapitalLoanTransaction(
return this;
}
+ public CommandWrapperBuilder
undoDiscountFeeAdjustmentWorkingCapitalLoanTransaction(final Long loanId) {
+ this.actionName = ACTION_UNDODISCOUNTFEEADJUSTMENT;
+ this.entityName = ENTITY_WORKINGCAPITALLOAN;
Review Comment:
Kept ENTITY_WORKINGCAPITALLOAN on purpose, because all other WC transaction
commands (repayment, discountFee, etc.) use it, and the core Loan
implementation does the same for repaymentTransaction, adjustTransaction, and
chargebackTransaction.
I thought this was the correct approach, but please correct me again :)
--
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]
