diff --git a/app/controllers/PiggybankController.php b/app/controllers/PiggybankController.php index 3fb864bfc2..6f08e87c1f 100644 --- a/app/controllers/PiggybankController.php +++ b/app/controllers/PiggybankController.php @@ -94,7 +94,7 @@ class PiggybankController extends BaseController View::share('subTitle', 'Delete "' . $piggyBank->name . '"'); if ($piggyBank->repeats == 1) { View::share('title', 'Repeated expenses'); - View::share('mainTitleIcon', 'fa-rotate-right') + View::share('mainTitleIcon', 'fa-rotate-right'); } else { View::share('title', 'Piggy banks'); View::share('mainTitleIcon', 'fa-sort-amount-asc'); @@ -196,8 +196,13 @@ class PiggybankController extends BaseController } break; } + if($piggyBank->repeats == 1) { + $route = 'piggybanks.index.repeated'; - return Redirect::route('piggybanks.index'); + } else { + $route = 'piggybanks.index.piggybanks'; + } + return Redirect::route($route); } /** @@ -219,7 +224,8 @@ class PiggybankController extends BaseController $account = $piggybank->account; $id = $account->id; if (!isset($accounts[$id])) { - $accounts[$id] = ['account' => $account, 'left' => $this->_repository->leftOnAccount($account)]; + $account->leftOnAccount = $this->_repository->leftOnAccount($account); + $accounts[$id] = ['account' => $account, 'left' => $this->_repository->leftOnAccount($account)]; } } @@ -268,7 +274,8 @@ class PiggybankController extends BaseController $account = $piggybank->account; $id = $account->id; if (!isset($accounts[$id])) { - $accounts[$id] = ['account' => $account, 'left' => $this->_repository->leftOnAccount($account)]; + $account->leftOnAccount = $this->_repository->leftOnAccount($account); + $accounts[$id] = ['account' => $account, 'left' => $this->_repository->leftOnAccount($account)]; } } @@ -373,10 +380,19 @@ class PiggybankController extends BaseController { $piggyBank = $this->_repository->update($piggyBank, Input::all()); if ($piggyBank->validate()) { - Session::flash('success', 'Piggy bank "' . $piggyBank->name . '" updated.'); + if ($piggyBank->repeats == 1) { + $route = 'piggybanks.index.repeated'; + $message = 'Repeated expense'; + } else { + $route = 'piggybanks.index.piggybanks'; + $message = 'Piggy bank'; + } + + + Session::flash('success', $message . ' "' . $piggyBank->name . '" updated.'); Event::fire('piggybanks.update', [$piggyBank]); - return Redirect::route('piggybanks.index'); + return Redirect::route($route); } else { Session::flash('error', 'Could not update piggy bank: ' . $piggyBank->errors()->first()); diff --git a/app/views/piggybanks/index.blade.php b/app/views/piggybanks/index.blade.php index 88c45e551b..218044670e 100644 --- a/app/views/piggybanks/index.blade.php +++ b/app/views/piggybanks/index.blade.php @@ -1,28 +1,12 @@ @extends('layouts.default') @section('content') -
-
-
-
-

Save money for large expenses

-

- Saving money is hard. Firefly's piggy banks can help you to save money. Simply set the amount - of money you want to save, set an optional target date and whether or not - Firefly should remind you to add money - to the piggy bank. -

Create new piggy bank

-

Save money for repeated expenses

-

- Taxes are due every year. Or maybe you want to save up for your yearly fireworks-binge. Buy a new smart - phone every three years. Firefly can help you organize these repeated expenses. -

Create new repeated expense

@@ -59,7 +43,7 @@
- @if($piggyBank->leftInAccount > 0) + @if($accounts[$piggyBank->account_id]['account']->leftOnAccount > 0) Add money @endif @if($piggyBank->currentRelevantRep()->currentamount > 0) @@ -122,7 +106,7 @@
- @if($repeated->leftInAccount > 0) + @if($accounts[$repeated->account_id]['account']->leftOnAccount > 0) Add money @endif @if($repeated->currentRelevantRep()->currentamount > 0)