IntenseFinance/firefly-iii
3
0
Fork 0
mirror of https://github.com/firefly-iii/firefly-iii.git synced 2025-02-25 18:45:27 -06:00
Code Issues Packages Projects Releases Wiki Activity

Fixed a bug where the associated total amount for a bar would be too high.

Browse Source
This commit is contained in:
James Cole 2014-11-24 18:08:20 +01:00
parent 743deb4227
commit fd6e7fc1ab
1 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
app/lib/FireflyIII/Database/RepeatedExpense.php
View File

@ -95,9 +95,12 @@ class RepeatedExpense implements CUD, CommonDatabaseCalls, PiggybankInterface
$amountPerBar = floatval($piggyBank->targetamount) / $parts; $amountPerBar = floatval($piggyBank->targetamount) / $parts;
$cumulative = $amountPerBar; $cumulative = $amountPerBar;
/** @var PiggybankPart $bar */ /** @var PiggybankPart $bar */
foreach ($bars as $bar) { foreach ($bars as $index => $bar) {
$bar->setAmountPerBar($amountPerBar); $bar->setAmountPerBar($amountPerBar);
$bar->setCumulativeAmount($cumulative); $bar->setCumulativeAmount($cumulative);
if($parts -1 == $index) {
$bar->setCumulativeAmount($piggyBank->targetamount);
}
$reminder = $piggyBank->reminders() $reminder = $piggyBank->reminders()
->where('startdate', $bar->getStartdate()->format('Y-m-d')) ->where('startdate', $bar->getStartdate()->format('Y-m-d'))