From b999a8f0fbd5ed18533ae8c6b86e1c39255fcb4c Mon Sep 17 00:00:00 2001 From: James Cole Date: Mon, 15 Dec 2014 19:39:01 +0100 Subject: [PATCH] Some cleaning up. About to release 3.2.1 --- app/controllers/HomeController.php | 2 +- app/database/seeds/TestContentSeeder.php | 5 +-- .../FireflyIII/Collection/PiggybankPart.php | 40 +++++++++---------- .../FireflyIII/Database/Account/Account.php | 40 +++++++++---------- .../Database/PiggyBank/PiggyBank.php | 6 +++ .../Database/PiggyBank/RepeatedExpense.php | 6 +++ .../RecurringTransaction.php | 5 +++ .../TransactionJournal/TransactionJournal.php | 9 ++++- app/lib/FireflyIII/Shared/Toolkit/Form.php | 1 - bootstrap/start.php | 18 +++++---- 10 files changed, 74 insertions(+), 58 deletions(-) diff --git a/app/controllers/HomeController.php b/app/controllers/HomeController.php index 1dd8d28644..5e944f0386 100644 --- a/app/controllers/HomeController.php +++ b/app/controllers/HomeController.php @@ -48,7 +48,7 @@ class HomeController extends BaseController $transactions = []; foreach ($accounts as $account) { - $set = $jrnls->getInDateRangeAccount($account, 10, $start, $end); + $set = $jrnls->getInDateRangeAccount($account, $start, $end, 10); if (count($set) > 0) { $transactions[] = [$set, $account]; } diff --git a/app/database/seeds/TestContentSeeder.php b/app/database/seeds/TestContentSeeder.php index 11f0bdf8dc..e1b40ee435 100644 --- a/app/database/seeds/TestContentSeeder.php +++ b/app/database/seeds/TestContentSeeder.php @@ -138,8 +138,7 @@ class TestContentSeeder extends Seeder */ public function createTransaction( Account $from, Account $to, $amount, TransactionType $type, $description, $date, Budget $budget = null, Category $category = null - ) - { + ) { $user = User::whereEmail('thegrumpydictator@gmail.com')->first(); $euro = TransactionCurrency::whereCode('EUR')->first(); @@ -161,7 +160,6 @@ class TestContentSeeder extends Seeder 'transaction_journal_id' => $journal->id, 'amount' => $amount * -1 ] - ); Transaction::create( [ @@ -169,7 +167,6 @@ class TestContentSeeder extends Seeder 'transaction_journal_id' => $journal->id, 'amount' => $amount ] - ); if (!is_null($budget)) { $journal->budgets()->save($budget); diff --git a/app/lib/FireflyIII/Collection/PiggybankPart.php b/app/lib/FireflyIII/Collection/PiggybankPart.php index f4e9e43b96..a0919c2e96 100644 --- a/app/lib/FireflyIII/Collection/PiggybankPart.php +++ b/app/lib/FireflyIII/Collection/PiggybankPart.php @@ -14,13 +14,10 @@ class PiggybankPart { /** @var float */ public $amountPerBar; - - /** @var float */ - public $currentamount; - /** @var float */ public $cumulativeAmount; - + /** @var float */ + public $currentamount; /** @var \Reminder */ public $reminder; @@ -38,11 +35,12 @@ class PiggybankPart */ public function getReminder() { - if(is_null($this->reminder)) { + if (is_null($this->reminder)) { $this->reminder = $this->repetition->piggybank->reminders()->where('startdate', $bar->getStartdate()->format('Y-m-d'))->where( 'enddate', $bar->getTargetdate()->format('Y-m-d') )->first(); } + return $this->reminder; } @@ -145,22 +143,6 @@ class PiggybankPart $this->currentamount = $currentamount; } - /** - * @return float - */ - public function getAmountPerBar() - { - return $this->amountPerBar; - } - - /** - * @param float $amountPerBar - */ - public function setAmountPerBar($amountPerBar) - { - $this->amountPerBar = $amountPerBar; - } - /** * @return float */ @@ -177,7 +159,21 @@ class PiggybankPart $this->cumulativeAmount = $cumulativeAmount; } + /** + * @return float + */ + public function getAmountPerBar() + { + return $this->amountPerBar; + } + /** + * @param float $amountPerBar + */ + public function setAmountPerBar($amountPerBar) + { + $this->amountPerBar = $amountPerBar; + } } \ No newline at end of file diff --git a/app/lib/FireflyIII/Database/Account/Account.php b/app/lib/FireflyIII/Database/Account/Account.php index abd0dc2eae..a7f5c71683 100644 --- a/app/lib/FireflyIII/Database/Account/Account.php +++ b/app/lib/FireflyIII/Database/Account/Account.php @@ -223,28 +223,28 @@ class Account implements CUD, CommonDatabaseCalls, AccountInterface // delete journals: \TransactionJournal::whereIn( 'id', function ($query) use ($model) { - $query->select('transaction_journal_id') - ->from('transactions')->whereIn( - 'account_id', function ($query) use ($model) { - $query - ->select('id') - ->from('accounts') - ->where( + $query->select('transaction_journal_id') + ->from('transactions')->whereIn( + 'account_id', function ($query) use ($model) { + $query + ->select('id') + ->from('accounts') + ->where( + function ($q) use ($model) { + $q->where('id', $model->id); + $q->orWhere( function ($q) use ($model) { - $q->where('id', $model->id); - $q->orWhere( - function ($q) use ($model) { - $q->where('accounts.name', 'LIKE', '%' . $model->name . '%'); - // TODO magic number! - $q->where('accounts.account_type_id', 3); - $q->where('accounts.active', 0); - } - ); + $q->where('accounts.name', 'LIKE', '%' . $model->name . '%'); + // TODO magic number! + $q->where('accounts.account_type_id', 3); + $q->where('accounts.active', 0); } - )->where('accounts.user_id', $this->getUser()->id); - } - )->get(); - } + ); + } + )->where('accounts.user_id', $this->getUser()->id); + } + )->get(); + } )->delete(); /* diff --git a/app/lib/FireflyIII/Database/PiggyBank/PiggyBank.php b/app/lib/FireflyIII/Database/PiggyBank/PiggyBank.php index 4dc53d136c..114b7646d3 100644 --- a/app/lib/FireflyIII/Database/PiggyBank/PiggyBank.php +++ b/app/lib/FireflyIII/Database/PiggyBank/PiggyBank.php @@ -87,6 +87,12 @@ class PiggyBank implements CUD, CommonDatabaseCalls, PiggyBankInterface * Validates an array. Returns an array containing MessageBags * errors/warnings/successes. * + * Ignore PHPMD rules because Laravel 5.0 will make this method superfluous anyway. + * + * @SuppressWarnings("Cyclomatic") + * @SuppressWarnings("NPath") + * @SuppressWarnings("MethodLength") + * * @param array $model * * @return array diff --git a/app/lib/FireflyIII/Database/PiggyBank/RepeatedExpense.php b/app/lib/FireflyIII/Database/PiggyBank/RepeatedExpense.php index 1792b5fdba..b6886deed2 100644 --- a/app/lib/FireflyIII/Database/PiggyBank/RepeatedExpense.php +++ b/app/lib/FireflyIII/Database/PiggyBank/RepeatedExpense.php @@ -158,6 +158,12 @@ class RepeatedExpense implements CUD, CommonDatabaseCalls, PiggyBankInterface * Validates an array. Returns an array containing MessageBags * errors/warnings/successes. * + * + * ignored because this method will be gone soon. + * @SuppressWarnings("Cyclomatic") + * @SuppressWarnings("NPath") + * @SuppressWarnings("MethodLength") + * * @param array $model * * @return array diff --git a/app/lib/FireflyIII/Database/RecurringTransaction/RecurringTransaction.php b/app/lib/FireflyIII/Database/RecurringTransaction/RecurringTransaction.php index 80068ac6fd..446a0926f4 100644 --- a/app/lib/FireflyIII/Database/RecurringTransaction/RecurringTransaction.php +++ b/app/lib/FireflyIII/Database/RecurringTransaction/RecurringTransaction.php @@ -102,6 +102,11 @@ class RecurringTransaction implements CUD, CommonDatabaseCalls, RecurringTransac * Validates an array. Returns an array containing MessageBags * errors/warnings/successes. * + * ignored because this method will be gone soon. + * @SuppressWarnings("Cyclomatic") + * @SuppressWarnings("NPath") + * @SuppressWarnings("MethodLength") + * * @param array $model * * @return array diff --git a/app/lib/FireflyIII/Database/TransactionJournal/TransactionJournal.php b/app/lib/FireflyIII/Database/TransactionJournal/TransactionJournal.php index 2e2a140e32..e32d1280db 100644 --- a/app/lib/FireflyIII/Database/TransactionJournal/TransactionJournal.php +++ b/app/lib/FireflyIII/Database/TransactionJournal/TransactionJournal.php @@ -194,6 +194,11 @@ class TransactionJournal implements TransactionJournalInterface, CUD, CommonData * Validates an array. Returns an array containing MessageBags * errors/warnings/successes. * + * ignored because this method will be gone soon. + * @SuppressWarnings("Cyclomatic") + * @SuppressWarnings("NPath") + * @SuppressWarnings("MethodLength") + * * @param array $model * * @return array @@ -614,13 +619,13 @@ class TransactionJournal implements TransactionJournalInterface, CUD, CommonData /** * @param \Account $account - * @param int $count * @param Carbon $start * @param Carbon $end + * @param int $count * * @return Collection */ - public function getInDateRangeAccount(\Account $account, $count = 20, Carbon $start, Carbon $end) + public function getInDateRangeAccount(\Account $account, Carbon $start, Carbon $end, $count = 20) { $accountID = $account->id; diff --git a/app/lib/FireflyIII/Shared/Toolkit/Form.php b/app/lib/FireflyIII/Shared/Toolkit/Form.php index 226ff34b2e..a3a7d58376 100644 --- a/app/lib/FireflyIII/Shared/Toolkit/Form.php +++ b/app/lib/FireflyIII/Shared/Toolkit/Form.php @@ -15,7 +15,6 @@ class Form * Takes any collection and tries to make a sensible select list compatible array of it. * * @param Collection $set - * @param null $titleField * @param bool $addEmpty * @SuppressWarnings("CyclomaticComplexity") * diff --git a/bootstrap/start.php b/bootstrap/start.php index 1d6083d86d..107bc62ed7 100644 --- a/bootstrap/start.php +++ b/bootstrap/start.php @@ -14,14 +14,16 @@ if (!function_exists('mf')) { $amount = round($amount, 2); $string = number_format($amount, 2, ',', '.'); - if ($coloured === true && $amount === 0.0) { - return '€ ' . $string . ''; - } - if ($coloured === true && $amount > 0) { - return '€ ' . $string . ''; - } - if ($coloured === true && $amount < 0) { - return '€ ' . $string . ''; + if ($coloured === true) { + if ($amount === 0.0) { + return '€ ' . $string . ''; + } + if ($amount > 0) { + return '€ ' . $string . ''; + } + if ($amount < 0) { + return '€ ' . $string . ''; + } } return '€ ' . $string;