This issue was identified while examining source code. So as of the time of
this filing we don't have a UI based repro to report.
The code to remove fees was found to still be invoking old application logic for
rounding. The code needs to be changed to use the new rounding rules introduced
with the refactoring of financial calculation code.
Platform: All, OS: Windows XP
Target v1.1 per triage meeting.
Van/Beth-- sorry to do this when both of you were out-- but I think this is an
issue we can triage for v1.1. I think the scenario of removing fees is rare
and isn't a primary use case we need to support. Can be an immediate fix after
If we do have spare cycles-- we might want to re-evaluate getting this into
John can you see if this is still true?
I talked to Van about this.
So, Van went and checked through the loan code. He determined that the current code was now using the correct 'V2' rounding method / algorithm.
The only change made was to remove the method 'applyRounding' (which was delegating to the correct rounding method 'applyRounding_v2' and call the '_v2' method directly. This was just for clarity. The '_v2' naming can go at sometime in the future but not urgent.