alberto-art3ch commented on code in PR #6514:
URL: https://github.com/apache/fineract/pull/6514#discussion_r4149493288
##########
fineract-working-capital-loan/src/main/java/org/apache/fineract/portfolio/workingcapitalloan/domain/WorkingCapitalLoanPeriodPaymentRateChange.java:
##########
@@ -64,9 +64,10 @@ public class WorkingCapitalLoanPeriodPaymentRateChange
extends AbstractAuditable
/**
* Snapshot taken when the change was booked, never restated; null for
changes booked before it existed. The annual
- * EIR is a percentage, like the rates.
+ * EIR is a percentage, like the rates. Wider than the other rate columns
because it is solved rather than entered:
+ * it matches the decimal(27,8) field the same figure is published in, and
the cap the solver enforces on it.
*/
- @Column(name = "calculated_annual_eir", scale = 6, precision = 19)
+ @Column(name = "calculated_annual_eir", scale = 8, precision = 27)
Review Comment:
Reverted: column back to precision = 19, scale = 6, changeset removed. It
was only widened to fit the old 1E+19 cap, which is gone now.
--
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]
