Merge pull request #80 from JC5/scrutinizer-patch-3

Scrutinizer Auto-Fixes
2025-02-25 18:45:27 -06:00 · 2015-05-26 20:16:39 +02:00 · 2015-05-26 20:16:39 +02:00 · ffc6139e21
commit ffc6139e21
parent 2738ac5a5c 571cac6644
16 changed files with 128 additions and 137 deletions
--- a/app/Handlers/Events/UpdateJournalConnection.php
+++ b/app/Handlers/Events/UpdateJournalConnection.php
@ -50,7 +50,7 @@ class UpdateJournalConnection
            return;
        }
        $amount = $journal->amount;
-        $diff   = $amount - $event->amount;// update current repetition
+        $diff   = $amount - $event->amount; // update current repetition

        $repetition->currentamount += $diff;
        $repetition->save();
--- a/app/Helpers/Collection/Bill.php
+++ b/app/Helpers/Collection/Bill.php
@ -41,7 +41,7 @@ class Bill
    public function getBills()
    {
        $this->bills->sortBy(
-            function (BillLine $bill) {
+            function(BillLine $bill) {
                $active = intval($bill->getBill()->active) == 0 ? 1 : 0;
                $name   = $bill->getBill()->name;

--- a/app/Helpers/Collection/Category.php
+++ b/app/Helpers/Collection/Category.php
@ -55,7 +55,7 @@ class Category
    public function getCategories()
    {
        $this->categories->sortByDesc(
-            function (CategoryModel $category) {
+            function(CategoryModel $category) {
                return $category->spent;
            }
        );
--- a/app/Helpers/Collection/Expense.php
+++ b/app/Helpers/Collection/Expense.php
@ -67,7 +67,7 @@ class Expense
    public function getExpenses()
    {
        $this->expenses->sortByDesc(
-            function (stdClass $object) {
+            function(stdClass $object) {
                return $object->amount;
            }
        );
--- a/app/Helpers/Collection/Income.php
+++ b/app/Helpers/Collection/Income.php
@ -68,7 +68,7 @@ class Income
    public function getIncomes()
    {
        $this->incomes->sortByDesc(
-            function (stdClass $object) {
+            function(stdClass $object) {
                return $object->amount;
            }
        );
@ -77,7 +77,7 @@ class Income
    }

    /**
-     * @return float
+     * @return string
     */
    public function getTotal()
    {
--- a/app/Helpers/Report/ReportHelper.php
+++ b/app/Helpers/Report/ReportHelper.php
@ -66,7 +66,7 @@ class ReportHelper implements ReportHelperInterface

        // remove cash account, if any:
        $accounts = $accounts->filter(
-            function (Account $account) {
+            function(Account $account) {
                if ($account->accountType->type != 'Cash account') {
                    return $account;
                }
--- a/app/Helpers/Report/ReportQuery.php
+++ b/app/Helpers/Report/ReportQuery.php
@ -35,15 +35,15 @@ class ReportQuery implements ReportQueryInterface
        $query = $this->queryJournalsWithTransactions($start, $end);
        if ($includeShared === false) {
            $query->where(
-                function (Builder $query) {
+                function(Builder $query) {
                    $query->where(
-                        function (Builder $q) { // only get withdrawals not from a shared account
+                        function(Builder $q) { // only get withdrawals not from a shared account
                            $q->where('transaction_types.type', 'Withdrawal');
                            $q->where('acm_from.data', '!=', '"sharedAsset"');
                        }
                    );
                    $query->orWhere(
-                        function (Builder $q) { // and transfers from a shared account.
+                        function(Builder $q) { // and transfers from a shared account.
                            $q->where('transaction_types.type', 'Transfer');
                            $q->where('acm_to.data', '=', '"sharedAsset"');
                        }
@ -61,14 +61,14 @@ class ReportQuery implements ReportQueryInterface
        );

        $data->each(
-            function (TransactionJournal $journal) {
+            function(TransactionJournal $journal) {
                if (intval($journal->account_encrypted) == 1) {
                    $journal->name = Crypt::decrypt($journal->name);
                }
            }
        );
        $data = $data->filter(
-            function (TransactionJournal $journal) {
+            function(TransactionJournal $journal) {
                if ($journal->amount != 0) {
                    return $journal;
                }
@ -91,26 +91,26 @@ class ReportQuery implements ReportQueryInterface
    public function getAllAccounts(Carbon $start, Carbon $end, $includeShared = false)
    {
        $query = Auth::user()->accounts()->orderBy('accounts.name', 'ASC')
-                     ->accountTypeIn(['Default account', 'Asset account', 'Cash account']);
+                        ->accountTypeIn(['Default account', 'Asset account', 'Cash account']);
        if ($includeShared === false) {
            $query->leftJoin(
                'account_meta', function (JoinClause $join) {
                $join->on('account_meta.account_id', '=', 'accounts.id')->where('account_meta.name', '=', 'accountRole');
            }
            )
-                  ->orderBy('accounts.name', 'ASC')
-                  ->where(
-                      function (Builder $query) {
+                    ->orderBy('accounts.name', 'ASC')
+                    ->where(
+                        function (Builder $query) {

-                          $query->where('account_meta.data', '!=', '"sharedAsset"');
-                          $query->orWhereNull('account_meta.data');
+                            $query->where('account_meta.data', '!=', '"sharedAsset"');
+                            $query->orWhereNull('account_meta.data');

-                      }
-                  );
+                        }
+                    );
        }
        $set = $query->get(['accounts.*']);
        $set->each(
-            function (Account $account) use ($start, $end) {
+            function(Account $account) use ($start, $end) {
                /**
                 * The balance for today always incorporates transactions
                 * made on today. So to get todays "start" balance, we sub one
@ -151,15 +151,15 @@ class ReportQuery implements ReportQueryInterface
            // only get deposits not to a shared account
            // and transfers to a shared account.
            $query->where(
-                function (Builder $query) {
+                function(Builder $query) {
                    $query->where(
-                        function (Builder $q) {
+                        function(Builder $q) {
                            $q->where('transaction_types.type', 'Deposit');
                            $q->where('acm_to.data', '!=', '"sharedAsset"');
                        }
                    );
                    $query->orWhere(
-                        function (Builder $q) {
+                        function(Builder $q) {
                            $q->where('transaction_types.type', 'Transfer');
                            $q->where('acm_from.data', '=', '"sharedAsset"');
                        }
@ -178,14 +178,14 @@ class ReportQuery implements ReportQueryInterface
        );

        $data->each(
-            function (TransactionJournal $journal) {
+            function(TransactionJournal $journal) {
                if (intval($journal->account_encrypted) == 1) {
                    $journal->name = Crypt::decrypt($journal->name);
                }
            }
        );
        $data = $data->filter(
-            function (TransactionJournal $journal) {
+            function(TransactionJournal $journal) {
                if ($journal->amount != 0) {
                    return $journal;
                }
@ -210,16 +210,16 @@ class ReportQuery implements ReportQueryInterface
    {

        return floatval(
-                   Auth::user()->transactionjournals()
-                       ->leftJoin('transactions', 'transactions.transaction_journal_id', '=', 'transaction_journals.id')
-                       ->leftJoin('budget_transaction_journal', 'budget_transaction_journal.transaction_journal_id', '=', 'transaction_journals.id')
-                       ->transactionTypes(['Withdrawal'])
-                       ->where('transactions.account_id', $account->id)
-                       ->before($end)
-                       ->after($start)
-                       ->where('budget_transaction_journal.budget_id', $budget->id)
-                       ->get(['transaction_journals.*'])->sum('amount')
-               ) * -1;
+                    Auth::user()->transactionjournals()
+                        ->leftJoin('transactions', 'transactions.transaction_journal_id', '=', 'transaction_journals.id')
+                        ->leftJoin('budget_transaction_journal', 'budget_transaction_journal.transaction_journal_id', '=', 'transaction_journals.id')
+                        ->transactionTypes(['Withdrawal'])
+                        ->where('transactions.account_id', $account->id)
+                        ->before($end)
+                        ->after($start)
+                        ->where('budget_transaction_journal.budget_id', $budget->id)
+                        ->get(['transaction_journals.*'])->sum('amount')
+                ) * -1;
    }

    /**
@ -258,24 +258,24 @@ class ReportQuery implements ReportQueryInterface
            $join->on('t_from.transaction_journal_id', '=', 'transaction_journals.id')->where('t_from.amount', '<', 0);
        }
        )
-                                   ->leftJoin('accounts as ac_from', 't_from.account_id', '=', 'ac_from.id')
-                                   ->leftJoin(
-                                       'account_meta as acm_from', function (JoinClause $join) {
-                                       $join->on('ac_from.id', '=', 'acm_from.account_id')->where('acm_from.name', '=', 'accountRole');
-                                   }
-                                   )
-                                   ->leftJoin(
-                                       'transactions as t_to', function (JoinClause $join) {
-                                       $join->on('t_to.transaction_journal_id', '=', 'transaction_journals.id')->where('t_to.amount', '>', 0);
-                                   }
-                                   )
-                                   ->leftJoin('accounts as ac_to', 't_to.account_id', '=', 'ac_to.id')
-                                   ->leftJoin(
-                                       'account_meta as acm_to', function (JoinClause $join) {
-                                       $join->on('ac_to.id', '=', 'acm_to.account_id')->where('acm_to.name', '=', 'accountRole');
-                                   }
-                                   )
-                                   ->leftJoin('transaction_types', 'transaction_types.id', '=', 'transaction_journals.transaction_type_id');
+                                    ->leftJoin('accounts as ac_from', 't_from.account_id', '=', 'ac_from.id')
+                                    ->leftJoin(
+                                        'account_meta as acm_from', function (JoinClause $join) {
+                                        $join->on('ac_from.id', '=', 'acm_from.account_id')->where('acm_from.name', '=', 'accountRole');
+                                    }
+                                    )
+                                    ->leftJoin(
+                                        'transactions as t_to', function (JoinClause $join) {
+                                        $join->on('t_to.transaction_journal_id', '=', 'transaction_journals.id')->where('t_to.amount', '>', 0);
+                                    }
+                                    )
+                                    ->leftJoin('accounts as ac_to', 't_to.account_id', '=', 'ac_to.id')
+                                    ->leftJoin(
+                                        'account_meta as acm_to', function (JoinClause $join) {
+                                        $join->on('ac_to.id', '=', 'acm_to.account_id')->where('acm_to.name', '=', 'accountRole');
+                                    }
+                                    )
+                                    ->leftJoin('transaction_types', 'transaction_types.id', '=', 'transaction_journals.transaction_type_id');
        $query->before($end)->after($start)->where('transaction_journals.user_id', Auth::user()->id);

        return $query;
--- a/app/Http/Controllers/AccountController.php
+++ b/app/Http/Controllers/AccountController.php
@ -3,7 +3,6 @@
 use Auth;
 use Carbon\Carbon;
 use Config;
-use FireflyIII\Http\Requests;
 use FireflyIII\Http\Requests\AccountFormRequest;
 use FireflyIII\Models\Account;
 use FireflyIII\Repositories\Account\AccountRepositoryInterface;
@ -95,7 +94,7 @@ class AccountController extends Controller
     * @param AccountRepositoryInterface $repository
     * @param Account                    $account
     *
-     * @return View
+     * @return \Illuminate\View\View
     */
    public function edit(AccountRepositoryInterface $repository, Account $account)
    {
@ -140,7 +139,7 @@ class AccountController extends Controller
     * @param AccountRepositoryInterface $repository
     * @param                            $what
     *
-     * @return View
+     * @return \Illuminate\View\View
     */
    public function index(AccountRepositoryInterface $repository, $what)
    {
@ -155,7 +154,7 @@ class AccountController extends Controller
        $start = clone Session::get('start', Carbon::now()->startOfMonth());
        $start->subDay();
        $accounts->each(
-            function (Account $account) use ($start, $repository) {
+            function(Account $account) use ($start, $repository) {
                $account->lastActivityDate = $repository->getLastActivity($account);
                $account->startBalance     = Steam::balance($account, $start);
                $account->endBalance       = Steam::balance($account, clone Session::get('end', Carbon::now()->endOfMonth()));
@ -169,7 +168,7 @@ class AccountController extends Controller
     * @param AccountRepositoryInterface $repository
     * @param Account                    $account
     *
-     * @return View
+     * @return \Illuminate\View\View
     */
    public function show(AccountRepositoryInterface $repository, Account $account)
    {
@ -204,7 +203,7 @@ class AccountController extends Controller
            'openingBalanceCurrency' => intval($request->input('balance_currency_id')),

        ];
-        $account     = $repository->store($accountData);
+        $account = $repository->store($accountData);

        Session::flash('success', 'New account "' . $account->name . '" stored!');

@ -226,7 +225,7 @@ class AccountController extends Controller
     * @param AccountRepositoryInterface $repository
     * @param Account                    $account
     *
-     * @return $this|\Illuminate\Http\RedirectResponse
+     * @return \Illuminate\Http\RedirectResponse
     */
    public function update(AccountFormRequest $request, AccountRepositoryInterface $repository, Account $account)
    {
--- a/app/Http/Controllers/Auth/AuthController.php
+++ b/app/Http/Controllers/Auth/AuthController.php
@ -67,7 +67,7 @@ class AuthController extends Controller
     *
     * @param  Request $request
     *
-     * @return \Illuminate\Http\Response
+     * @return \Illuminate\Http\RedirectResponse
     */
    public function postRegister(Request $request)
    {
@ -91,7 +91,7 @@ class AuthController extends Controller

        // send email.
        Mail::send(
-            'emails.registered', [], function (Message $message) use ($email) {
+            'emails.registered', [], function(Message $message) use ($email) {
            $message->to($email, $email)->subject('Welcome to Firefly III!');
        }
        );
--- a/app/Http/Controllers/BillController.php
+++ b/app/Http/Controllers/BillController.php
@ -1,10 +1,8 @@
 <?php namespace FireflyIII\Http\Controllers;

 use Config;
-use FireflyIII\Http\Requests;
 use FireflyIII\Http\Requests\BillFormRequest;
 use FireflyIII\Models\Bill;
-use FireflyIII\Models\Transaction;
 use FireflyIII\Models\TransactionJournal;
 use FireflyIII\Repositories\Bill\BillRepositoryInterface;
 use Input;
@ -32,7 +30,7 @@ class BillController extends Controller
    }

    /**
-     * @return $this
+     * @return \Illuminate\View\View
     */
    public function create()
    {
@ -53,7 +51,7 @@ class BillController extends Controller
    /**
     * @param Bill $bill
     *
-     * @return $this
+     * @return \Illuminate\View\View
     */
    public function delete(Bill $bill)
    {
@ -85,7 +83,7 @@ class BillController extends Controller
    /**
     * @param Bill $bill
     *
-     * @return $this
+     * @return \Illuminate\View\View
     */
    public function edit(Bill $bill)
    {
@ -112,7 +110,7 @@ class BillController extends Controller
    {
        $bills = $repository->getBills();
        $bills->each(
-            function (Bill $bill) use ($repository) {
+            function(Bill $bill) use ($repository) {
                $bill->nextExpectedMatch = $repository->nextExpectedMatch($bill);
                $bill->lastFoundMatch    = $repository->lastFoundMatch($bill);
            }
@ -151,7 +149,7 @@ class BillController extends Controller
     * @param BillRepositoryInterface $repository
     * @param Bill                    $bill
     *
-     * @return mixed
+     * @return \Illuminate\View\View
     */
    public function show(BillRepositoryInterface $repository, Bill $bill)
    {
@ -167,7 +165,7 @@ class BillController extends Controller
     * @param BillFormRequest         $request
     * @param BillRepositoryInterface $repository
     *
-     * @return $this|\Illuminate\Http\RedirectResponse
+     * @return \Illuminate\Http\RedirectResponse
     */
    public function store(BillFormRequest $request, BillRepositoryInterface $repository)
    {
@ -192,7 +190,7 @@ class BillController extends Controller
     * @param BillRepositoryInterface $repository
     * @param Bill                    $bill
     *
-     * @return $this|\Illuminate\Http\RedirectResponse
+     * @return \Illuminate\Http\RedirectResponse
     */
    public function update(BillFormRequest $request, BillRepositoryInterface $repository, Bill $bill)
    {
--- a/app/Http/Controllers/BudgetController.php
+++ b/app/Http/Controllers/BudgetController.php
@ -3,7 +3,6 @@
 use Amount;
 use Auth;
 use Carbon\Carbon;
-use FireflyIII\Http\Requests;
 use FireflyIII\Http\Requests\BudgetFormRequest;
 use FireflyIII\Models\Budget;
 use FireflyIII\Models\LimitRepetition;
@ -56,7 +55,7 @@ class BudgetController extends Controller
    }

    /**
-     * @return $this
+     * @return \Illuminate\View\View
     */
    public function create()
    {
@ -111,7 +110,7 @@ class BudgetController extends Controller
    /**
     * @param Budget $budget
     *
-     * @return $this
+     * @return \Illuminate\View\View
     */
    public function edit(Budget $budget)
    {
@ -132,7 +131,7 @@ class BudgetController extends Controller
    /**
     * @param BudgetRepositoryInterface $repository
     *
-     * @return View
+     * @return \Illuminate\View\View
     */
    public function index(BudgetRepositoryInterface $repository)
    {
@ -174,7 +173,7 @@ class BudgetController extends Controller
    /**
     * @param BudgetRepositoryInterface $repository
     *
-     * @return View
+     * @return \Illuminate\View\View
     */
    public function noBudget(BudgetRepositoryInterface $repository)
    {
@ -206,7 +205,7 @@ class BudgetController extends Controller
     * @param Budget                    $budget
     * @param LimitRepetition           $repetition
     *
-     * @return View
+     * @return \Illuminate\View\View
     */
    public function show(BudgetRepositoryInterface $repository, Budget $budget, LimitRepetition $repetition = null)
    {
@ -240,7 +239,7 @@ class BudgetController extends Controller
            'name' => $request->input('name'),
            'user' => Auth::user()->id,
        ];
-        $budget     = $repository->store($budgetData);
+        $budget = $repository->store($budgetData);

        Session::flash('success', 'New budget "' . $budget->name . '" stored!');

@ -261,7 +260,7 @@ class BudgetController extends Controller
     * @param BudgetRepositoryInterface $repository
     * @param Budget                    $budget
     *
-     * @return $this|\Illuminate\Http\RedirectResponse
+     * @return \Illuminate\Http\RedirectResponse
     */
    public function update(BudgetFormRequest $request, BudgetRepositoryInterface $repository, Budget $budget)
    {
@ -287,7 +286,7 @@ class BudgetController extends Controller
    }

    /**
-     * @return View
+     * @return \Illuminate\View\View
     */
    public function updateIncome()
    {
--- a/app/Http/Controllers/CurrencyController.php
+++ b/app/Http/Controllers/CurrencyController.php
@ -1,9 +1,7 @@
 <?php namespace FireflyIII\Http\Controllers;

 use Cache;
-use FireflyIII\Http\Requests;
 use FireflyIII\Http\Requests\CurrencyFormRequest;
-use FireflyIII\Models\Transaction;
 use FireflyIII\Models\TransactionCurrency;
 use FireflyIII\Repositories\Currency\CurrencyRepositoryInterface;
 use Input;
--- a/app/Http/Controllers/PiggyBankController.php
+++ b/app/Http/Controllers/PiggyBankController.php
@ -4,7 +4,6 @@ use Amount;
 use Carbon\Carbon;
 use Config;
 use ExpandedForm;
-use FireflyIII\Http\Requests;
 use FireflyIII\Http\Requests\PiggyBankFormRequest;
 use FireflyIII\Models\PiggyBank;
 use FireflyIII\Repositories\Account\AccountRepositoryInterface;
--- a/app/Http/Controllers/TransactionController.php
+++ b/app/Http/Controllers/TransactionController.php
@ -5,7 +5,6 @@ use Carbon\Carbon;
 use ExpandedForm;
 use FireflyIII\Events\JournalCreated;
 use FireflyIII\Events\JournalSaved;
-use FireflyIII\Http\Requests;
 use FireflyIII\Http\Requests\JournalFormRequest;
 use FireflyIII\Models\Transaction;
 use FireflyIII\Models\TransactionJournal;
--- a/app/Http/Middleware/Cleanup.php
+++ b/app/Http/Middleware/Cleanup.php
@ -8,7 +8,6 @@ use FireflyIII\Models\Category;
 use FireflyIII\Models\PiggyBank;
 use FireflyIII\Models\Preference;
 use FireflyIII\Models\Reminder;
-use FireflyIII\Models\Transaction;
 use FireflyIII\Models\TransactionJournal;
 use Illuminate\Contracts\Auth\Guard;
 use Illuminate\Http\Request;
--- a/app/Http/breadcrumbs.php
+++ b/app/Http/breadcrumbs.php
@ -17,7 +17,7 @@ use FireflyIII\Models\TransactionJournal;
 */
 Breadcrumbs::register(
    'home',
-    function (Generator $breadcrumbs) {
+    function(Generator $breadcrumbs) {

        $breadcrumbs->push(trans('breadcrumbs.home'), route('index'));
    }
@ -25,7 +25,7 @@ Breadcrumbs::register(

 Breadcrumbs::register(
    'index',
-    function (Generator $breadcrumbs) {
+    function(Generator $breadcrumbs) {

        $breadcrumbs->push(trans('breadcrumbs.home'), route('index'));
    }
@ -34,21 +34,21 @@ Breadcrumbs::register(

 // accounts
 Breadcrumbs::register(
-    'accounts.index', function (Generator $breadcrumbs, $what) {
+    'accounts.index', function(Generator $breadcrumbs, $what) {
    $breadcrumbs->parent('home');
    $breadcrumbs->push(trans('breadcrumbs.' . strtolower(e($what)) . '_accounts'), route('accounts.index', $what));
 }
 );

 Breadcrumbs::register(
-    'accounts.create', function (Generator $breadcrumbs, $what) {
+    'accounts.create', function(Generator $breadcrumbs, $what) {
    $breadcrumbs->parent('accounts.index', $what);
    $breadcrumbs->push(trans('breadcrumbs.new_' . strtolower(e($what)) . '_account'), route('accounts.create', $what));
 }
 );

 Breadcrumbs::register(
-    'accounts.show', function (Generator $breadcrumbs, Account $account) {
+    'accounts.show', function(Generator $breadcrumbs, Account $account) {

    $what = Config::get('firefly.shortNamesByFullName.' . $account->accountType->type);

@ -58,7 +58,7 @@ Breadcrumbs::register(
 }
 );
 Breadcrumbs::register(
-    'accounts.delete', function (Generator $breadcrumbs, Account $account) {
+    'accounts.delete', function(Generator $breadcrumbs, Account $account) {
    $breadcrumbs->parent('accounts.show', $account);
    $breadcrumbs->push(trans('breadcrumbs.delete_account', ['name' => e($account->name)]), route('accounts.delete', $account->id));
 }
@ -66,7 +66,7 @@ Breadcrumbs::register(


 Breadcrumbs::register(
-    'accounts.edit', function (Generator $breadcrumbs, Account $account) {
+    'accounts.edit', function(Generator $breadcrumbs, Account $account) {
    $breadcrumbs->parent('accounts.show', $account);
    $what = Config::get('firefly.shortNamesByFullName.' . $account->accountType->type);

@ -76,40 +76,40 @@ Breadcrumbs::register(

 // budgets.
 Breadcrumbs::register(
-    'budgets.index', function (Generator $breadcrumbs) {
+    'budgets.index', function(Generator $breadcrumbs) {
    $breadcrumbs->parent('home');
    $breadcrumbs->push(trans('breadcrumbs.budgets'), route('budgets.index'));
 }
 );
 Breadcrumbs::register(
-    'budgets.create', function (Generator $breadcrumbs) {
+    'budgets.create', function(Generator $breadcrumbs) {
    $breadcrumbs->parent('budgets.index');
    $breadcrumbs->push(trans('breadcrumbs.newBudget'), route('budgets.create'));
 }
 );

 Breadcrumbs::register(
-    'budgets.edit', function (Generator $breadcrumbs, Budget $budget) {
+    'budgets.edit', function(Generator $breadcrumbs, Budget $budget) {
    $breadcrumbs->parent('budgets.show', $budget);
    $breadcrumbs->push(trans('breadcrumbs.edit_budget', ['name' => e($budget->name)]), route('budgets.edit', $budget->id));
 }
 );
 Breadcrumbs::register(
-    'budgets.delete', function (Generator $breadcrumbs, Budget $budget) {
+    'budgets.delete', function(Generator $breadcrumbs, Budget $budget) {
    $breadcrumbs->parent('budgets.show', $budget);
    $breadcrumbs->push(trans('breadcrumbs.delete_budget', ['name' => e($budget->name)]), route('budgets.delete', $budget->id));
 }
 );

 Breadcrumbs::register(
-    'budgets.noBudget', function (Generator $breadcrumbs, $subTitle) {
+    'budgets.noBudget', function(Generator $breadcrumbs, $subTitle) {
    $breadcrumbs->parent('budgets.index');
    $breadcrumbs->push($subTitle, route('budgets.noBudget'));
 }
 );

 Breadcrumbs::register(
-    'budgets.show', function (Generator $breadcrumbs, Budget $budget, LimitRepetition $repetition = null) {
+    'budgets.show', function(Generator $breadcrumbs, Budget $budget, LimitRepetition $repetition = null) {
    $breadcrumbs->parent('budgets.index');
    $breadcrumbs->push(e($budget->name), route('budgets.show', $budget->id));
    if (!is_null($repetition) && !is_null($repetition->id)) {
@ -122,33 +122,33 @@ Breadcrumbs::register(

 // categories
 Breadcrumbs::register(
-    'categories.index', function (Generator $breadcrumbs) {
+    'categories.index', function(Generator $breadcrumbs) {
    $breadcrumbs->parent('home');
    $breadcrumbs->push(trans('breadcrumbs.categories'), route('categories.index'));
 }
 );
 Breadcrumbs::register(
-    'categories.create', function (Generator $breadcrumbs) {
+    'categories.create', function(Generator $breadcrumbs) {
    $breadcrumbs->parent('categories.index');
    $breadcrumbs->push(trans('breadcrumbs.newCategory'), route('categories.create'));
 }
 );

 Breadcrumbs::register(
-    'categories.edit', function (Generator $breadcrumbs, Category $category) {
+    'categories.edit', function(Generator $breadcrumbs, Category $category) {
    $breadcrumbs->parent('categories.show', $category);
    $breadcrumbs->push(trans('breadcrumbs.edit_category', ['name' => e($category->name)]), route('categories.edit', $category->id));
 }
 );
 Breadcrumbs::register(
-    'categories.delete', function (Generator $breadcrumbs, Category $category) {
+    'categories.delete', function(Generator $breadcrumbs, Category $category) {
    $breadcrumbs->parent('categories.show', $category);
    $breadcrumbs->push(trans('breadcrumbs.delete_category', ['name' => e($category->name)]), route('categories.delete', $category->id));
 }
 );

 Breadcrumbs::register(
-    'categories.show', function (Generator $breadcrumbs, Category $category) {
+    'categories.show', function(Generator $breadcrumbs, Category $category) {
    $breadcrumbs->parent('categories.index');
    $breadcrumbs->push(e($category->name), route('categories.show', $category->id));

@ -156,7 +156,7 @@ Breadcrumbs::register(
 );

 Breadcrumbs::register(
-    'categories.noCategory', function (Generator $breadcrumbs, $subTitle) {
+    'categories.noCategory', function(Generator $breadcrumbs, $subTitle) {
    $breadcrumbs->parent('categories.index');
    $breadcrumbs->push($subTitle, route('categories.noCategory'));
 }
@ -164,20 +164,20 @@ Breadcrumbs::register(

 // currencies.
 Breadcrumbs::register(
-    'currency.index', function (Generator $breadcrumbs) {
+    'currency.index', function(Generator $breadcrumbs) {
    $breadcrumbs->parent('home');
    $breadcrumbs->push(trans('breadcrumbs.currencies'), route('currency.index'));
 }
 );

 Breadcrumbs::register(
-    'currency.edit', function (Generator $breadcrumbs, TransactionCurrency $currency) {
+    'currency.edit', function(Generator $breadcrumbs, TransactionCurrency $currency) {
    $breadcrumbs->parent('currency.index');
    $breadcrumbs->push(trans('breadcrumbs.edit_currency', ['name' => e($currency->name)]), route('currency.edit', $currency->id));
 }
 );
 Breadcrumbs::register(
-    'currency.delete', function (Generator $breadcrumbs, TransactionCurrency $currency) {
+    'currency.delete', function(Generator $breadcrumbs, TransactionCurrency $currency) {
    $breadcrumbs->parent('currency.index');
    $breadcrumbs->push(trans('breadcrumbs.delete_currency', ['name' => e($currency->name)]), route('currency.delete', $currency->id));
 }
@ -186,33 +186,33 @@ Breadcrumbs::register(

 // piggy banks
 Breadcrumbs::register(
-    'piggy-banks.index', function (Generator $breadcrumbs) {
+    'piggy-banks.index', function(Generator $breadcrumbs) {
    $breadcrumbs->parent('home');
    $breadcrumbs->push(trans('breadcrumbs.piggyBanks'), route('piggy-banks.index'));
 }
 );
 Breadcrumbs::register(
-    'piggy-banks.create', function (Generator $breadcrumbs) {
+    'piggy-banks.create', function(Generator $breadcrumbs) {
    $breadcrumbs->parent('piggy-banks.index');
    $breadcrumbs->push(trans('breadcrumbs.newPiggyBank'), route('piggy-banks.create'));
 }
 );

 Breadcrumbs::register(
-    'piggy-banks.edit', function (Generator $breadcrumbs, PiggyBank $piggyBank) {
+    'piggy-banks.edit', function(Generator $breadcrumbs, PiggyBank $piggyBank) {
    $breadcrumbs->parent('piggy-banks.show', $piggyBank);
    $breadcrumbs->push(trans('breadcrumbs.edit_piggyBank', ['name' => e($piggyBank->name)]), route('piggy-banks.edit', $piggyBank->id));
 }
 );
 Breadcrumbs::register(
-    'piggy-banks.delete', function (Generator $breadcrumbs, PiggyBank $piggyBank) {
+    'piggy-banks.delete', function(Generator $breadcrumbs, PiggyBank $piggyBank) {
    $breadcrumbs->parent('piggy-banks.show', $piggyBank);
    $breadcrumbs->push(trans('breadcrumbs.delete_piggyBank', ['name' => e($piggyBank->name)]), route('piggy-banks.delete', $piggyBank->id));
 }
 );

 Breadcrumbs::register(
-    'piggy-banks.show', function (Generator $breadcrumbs, PiggyBank $piggyBank) {
+    'piggy-banks.show', function(Generator $breadcrumbs, PiggyBank $piggyBank) {
    $breadcrumbs->parent('piggy-banks.index');
    $breadcrumbs->push(e($piggyBank->name), route('piggy-banks.show', $piggyBank->id));

@ -221,7 +221,7 @@ Breadcrumbs::register(

 // preferences
 Breadcrumbs::register(
-    'preferences', function (Generator $breadcrumbs) {
+    'preferences', function(Generator $breadcrumbs) {
    $breadcrumbs->parent('home');
    $breadcrumbs->push(trans('breadcrumbs.preferences'), route('preferences'));

@ -230,14 +230,14 @@ Breadcrumbs::register(

 // profile
 Breadcrumbs::register(
-    'profile', function (Generator $breadcrumbs) {
+    'profile', function(Generator $breadcrumbs) {
    $breadcrumbs->parent('home');
    $breadcrumbs->push(trans('breadcrumbs.profile'), route('profile'));

 }
 );
 Breadcrumbs::register(
-    'change-password', function (Generator $breadcrumbs) {
+    'change-password', function(Generator $breadcrumbs) {
    $breadcrumbs->parent('profile');
    $breadcrumbs->push(trans('breadcrumbs.changePassword'), route('change-password'));

@ -246,33 +246,33 @@ Breadcrumbs::register(

 // bills
 Breadcrumbs::register(
-    'bills.index', function (Generator $breadcrumbs) {
+    'bills.index', function(Generator $breadcrumbs) {
    $breadcrumbs->parent('home');
    $breadcrumbs->push(trans('breadcrumbs.bills'), route('bills.index'));
 }
 );
 Breadcrumbs::register(
-    'bills.create', function (Generator $breadcrumbs) {
+    'bills.create', function(Generator $breadcrumbs) {
    $breadcrumbs->parent('bills.index');
    $breadcrumbs->push(trans('breadcrumbs.newBill'), route('bills.create'));
 }
 );

 Breadcrumbs::register(
-    'bills.edit', function (Generator $breadcrumbs, Bill $bill) {
+    'bills.edit', function(Generator $breadcrumbs, Bill $bill) {
    $breadcrumbs->parent('bills.show', $bill);
    $breadcrumbs->push(trans('breadcrumbs.edit_bill', ['name' => e($bill->name)]), route('bills.edit', $bill->id));
 }
 );
 Breadcrumbs::register(
-    'bills.delete', function (Generator $breadcrumbs, Bill $bill) {
+    'bills.delete', function(Generator $breadcrumbs, Bill $bill) {
    $breadcrumbs->parent('bills.show', $bill);
    $breadcrumbs->push(trans('breadcrumbs.delete_bill', ['name' => e($bill->name)]), route('bills.delete', $bill->id));
 }
 );

 Breadcrumbs::register(
-    'bills.show', function (Generator $breadcrumbs, Bill $bill) {
+    'bills.show', function(Generator $breadcrumbs, Bill $bill) {
    $breadcrumbs->parent('bills.index');
    $breadcrumbs->push(e($bill->name), route('bills.show', $bill->id));

@ -281,7 +281,7 @@ Breadcrumbs::register(

 // reminders
 Breadcrumbs::register(
-    'reminders.index', function (Generator $breadcrumbs) {
+    'reminders.index', function(Generator $breadcrumbs) {
    $breadcrumbs->parent('home');
    $breadcrumbs->push(trans('breadcrumbs.reminders'), route('reminders.index'));

@ -290,7 +290,7 @@ Breadcrumbs::register(

 // reminders
 Breadcrumbs::register(
-    'reminders.show', function (Generator $breadcrumbs, Reminder $reminder) {
+    'reminders.show', function(Generator $breadcrumbs, Reminder $reminder) {
    $breadcrumbs->parent('reminders.index');
    $breadcrumbs->push(trans('breadcrumbs.reminder', ['id' => e($reminder->id)]), route('reminders.show', $reminder->id));

@ -300,14 +300,14 @@ Breadcrumbs::register(

 // reports
 Breadcrumbs::register(
-    'reports.index', function (Generator $breadcrumbs) {
+    'reports.index', function(Generator $breadcrumbs) {
    $breadcrumbs->parent('home');
    $breadcrumbs->push(trans('breadcrumbs.reports'), route('reports.index'));
 }
 );

 Breadcrumbs::register(
-    'reports.year', function (Generator $breadcrumbs, Carbon $date, $shared) {
+    'reports.year', function(Generator $breadcrumbs, Carbon $date, $shared) {
    $breadcrumbs->parent('reports.index');
    if ($shared) {
        $title = trans('breadcrumbs.yearly_report_shared', ['date' => $date->year]);
@ -319,7 +319,7 @@ Breadcrumbs::register(
 );

 Breadcrumbs::register(
-    'reports.month', function (Generator $breadcrumbs, Carbon $date, $shared) {
+    'reports.month', function(Generator $breadcrumbs, Carbon $date, $shared) {
    $breadcrumbs->parent('reports.year', $date, $shared);

    if ($shared) {
@ -334,7 +334,7 @@ Breadcrumbs::register(

 // search
 Breadcrumbs::register(
-    'search', function (Generator $breadcrumbs, $query) {
+    'search', function(Generator $breadcrumbs, $query) {
    $breadcrumbs->parent('home');
    $breadcrumbs->push(trans('breadcrumbs.searchResult', ['query' => e($query)]), route('search'));
 }
@ -342,33 +342,33 @@ Breadcrumbs::register(

 // transactions
 Breadcrumbs::register(
-    'transactions.index', function (Generator $breadcrumbs, $what) {
+    'transactions.index', function(Generator $breadcrumbs, $what) {
    $breadcrumbs->parent('home');
    $breadcrumbs->push(trans('breadcrumbs.' . $what . '_list'), route('transactions.index', $what));
 }
 );
 Breadcrumbs::register(
-    'transactions.create', function (Generator $breadcrumbs, $what) {
+    'transactions.create', function(Generator $breadcrumbs, $what) {
    $breadcrumbs->parent('transactions.index', $what);
    $breadcrumbs->push(trans('breadcrumbs.create_' . e($what)), route('transactions.create', $what));
 }
 );

 Breadcrumbs::register(
-    'transactions.edit', function (Generator $breadcrumbs, TransactionJournal $journal) {
+    'transactions.edit', function(Generator $breadcrumbs, TransactionJournal $journal) {
    $breadcrumbs->parent('transactions.show', $journal);
    $breadcrumbs->push(trans('breadcrumbs.edit_journal', ['description' => $journal->description]), route('transactions.edit', $journal->id));
 }
 );
 Breadcrumbs::register(
-    'transactions.delete', function (Generator $breadcrumbs, TransactionJournal $journal) {
+    'transactions.delete', function(Generator $breadcrumbs, TransactionJournal $journal) {
    $breadcrumbs->parent('transactions.show', $journal);
    $breadcrumbs->push(trans('breadcrumbs.delete_journal', ['description' => e($journal->description)]), route('transactions.delete', $journal->id));
 }
 );

 Breadcrumbs::register(
-    'transactions.show', function (Generator $breadcrumbs, TransactionJournal $journal) {
+    'transactions.show', function(Generator $breadcrumbs, TransactionJournal $journal) {

    $breadcrumbs->parent('transactions.index', strtolower($journal->transactionType->type));
    $breadcrumbs->push($journal->description, route('transactions.show', $journal->id));
@ -378,28 +378,28 @@ Breadcrumbs::register(

 // tags
 Breadcrumbs::register(
-    'tags.index', function (Generator $breadcrumbs) {
+    'tags.index', function(Generator $breadcrumbs) {
    $breadcrumbs->parent('home');
    $breadcrumbs->push(trans('breadcrumbs.tags'), route('tags.index'));
 }
 );

 Breadcrumbs::register(
-    'tags.create', function (Generator $breadcrumbs) {
+    'tags.create', function(Generator $breadcrumbs) {
    $breadcrumbs->parent('tags.index');
    $breadcrumbs->push(trans('breadcrumbs.createTag'), route('tags.create'));
 }
 );

 Breadcrumbs::register(
-    'tags.edit', function (Generator $breadcrumbs, Tag $tag) {
+    'tags.edit', function(Generator $breadcrumbs, Tag $tag) {
    $breadcrumbs->parent('tags.show', $tag);
    $breadcrumbs->push(trans('breadcrumbs.edit_tag', ['tag' => e($tag->tag)]), route('tags.edit', $tag->id));
 }
 );

 Breadcrumbs::register(
-    'tags.delete', function (Generator $breadcrumbs, Tag $tag) {
+    'tags.delete', function(Generator $breadcrumbs, Tag $tag) {
    $breadcrumbs->parent('tags.show', $tag);
    $breadcrumbs->push(trans('breadcrumbs.delete_tag', ['tag' => e($tag->tag)]), route('tags.delete', $tag->id));
 }
@ -407,7 +407,7 @@ Breadcrumbs::register(


 Breadcrumbs::register(
-    'tags.show', function (Generator $breadcrumbs, Tag $tag) {
+    'tags.show', function(Generator $breadcrumbs, Tag $tag) {
    $breadcrumbs->parent('tags.index');
    $breadcrumbs->push(e($tag->tag), route('tags.show', $tag->id));
 }