From 65ce277a201099c788c2ffa83ed63d44fa313f69 Mon Sep 17 00:00:00 2001 From: James Cole Date: Fri, 2 Jan 2015 08:59:16 +0100 Subject: [PATCH] Updated models [skip ci] --- app/models/Account.php | 12 +++++++----- app/models/Piggybank.php | 14 ++++++-------- app/models/PiggybankRepetition.php | 10 +++++----- app/models/Transaction.php | 2 +- app/models/TransactionJournal.php | 14 +++++++------- 5 files changed, 26 insertions(+), 26 deletions(-) diff --git a/app/models/Account.php b/app/models/Account.php index a579ff862e..667aae069e 100644 --- a/app/models/Account.php +++ b/app/models/Account.php @@ -1,8 +1,10 @@ joinedAccountTypes)) { $query->leftJoin('account_types', 'account_types.id', '=', 'accounts.account_type_id'); @@ -78,9 +80,9 @@ class Account extends Eloquent /** * - * @param Builder $query + * @param EloquentBuilder $query */ - public function scopeWithMeta(Builder $query) + public function scopeWithMeta(EloquentBuilder $query) { $query->with(['accountmeta']); } diff --git a/app/models/Piggybank.php b/app/models/Piggybank.php index 55be709037..c3ba7e0054 100644 --- a/app/models/Piggybank.php +++ b/app/models/Piggybank.php @@ -3,7 +3,7 @@ use Carbon\Carbon; use Illuminate\Database\Eloquent\Model as Eloquent; use Illuminate\Database\Eloquent\SoftDeletingTrait; use Watson\Validating\ValidatingTrait; -use Illuminate\Database\Eloquent\Builder; +use Illuminate\Database\Eloquent\Builder as EloquentBuilder; /** * Class PiggyBank @@ -73,24 +73,23 @@ class PiggyBank extends Eloquent if ($this->repeats == 0) { $rep = $this->piggyBankRepetitions()->first(['piggy_bank_repetitions.*']); $this->currentRep = $rep; - \Log::debug('currentRelevantRep() reports $rep is null: ' . boolstr(is_null($rep))); return $rep; } else { $query = $this->piggyBankRepetitions()->where( - function (Builder $q) { + function (EloquentBuilder $q) { $q->where( - function (Builder $q) { + function (EloquentBuilder $q) { $q->where( - function (Builder $q) { + function (EloquentBuilder $q) { $today = new Carbon; $q->whereNull('startdate'); $q->orWhere('startdate', '<=', $today->format('Y-m-d 00:00:00')); } )->where( - function (Builder $q) { + function (EloquentBuilder $q) { $today = new Carbon; $q->whereNull('targetdate'); $q->orWhere('targetdate', '>=', $today->format('Y-m-d 00:00:00')); @@ -98,7 +97,7 @@ class PiggyBank extends Eloquent ); } )->orWhere( - function (Builder $q) { + function (EloquentBuilder $q) { $today = new Carbon; $q->where('startdate', '>=', $today->format('Y-m-d 00:00:00')); $q->where('targetdate', '>=', $today->format('Y-m-d 00:00:00')); @@ -108,7 +107,6 @@ class PiggyBank extends Eloquent } )->orderBy('startdate', 'ASC'); $result = $query->first(['piggy_bank_repetitions.*']); - \Log::debug('Result is null: ' . boolstr(is_null($result))); $this->currentRep = $result; \Log::debug('Found relevant rep in currentRelevantRep(): ' . $result->id); diff --git a/app/models/PiggybankRepetition.php b/app/models/PiggybankRepetition.php index a12a5b53a3..e38eabd4b4 100644 --- a/app/models/PiggybankRepetition.php +++ b/app/models/PiggybankRepetition.php @@ -1,6 +1,6 @@ where('startdate', $date->format('Y-m-d 00:00:00')); } /** - * @param Builder $query + * @param EloquentBuilder $query * @param Carbon $date */ - public function scopeTargets(Builder $query, Carbon $date) + public function scopeTargets(EloquentBuilder $query, Carbon $date) { $query->where('targetdate', $date->format('Y-m-d 00:00:00')); } diff --git a/app/models/Transaction.php b/app/models/Transaction.php index d641262004..c4a77ee917 100644 --- a/app/models/Transaction.php +++ b/app/models/Transaction.php @@ -1,7 +1,7 @@ joinedTransactions)) { $query->leftJoin('transactions', 'transactions.transaction_journal_id', '=', 'transaction_journals.id'); @@ -129,7 +129,7 @@ class TransactionJournal extends Eloquent * @param Builder $query * @param $amount */ - public function scopeLessThan(Builder $query, $amount) + public function scopeLessThan($query, $amount) { if (is_null($this->joinedTransactions)) { $query->leftJoin( @@ -145,7 +145,7 @@ class TransactionJournal extends Eloquent * @param Builder $query * @param $amount */ - public function scopeMoreThan(Builder $query, $amount) + public function scopeMoreThan($query, $amount) { if (is_null($this->joinedTransactions)) { $query->leftJoin( @@ -172,7 +172,7 @@ class TransactionJournal extends Eloquent * @param Builder $query * @param array $types */ - public function scopeTransactionTypes(Builder $query, array $types) + public function scopeTransactionTypes($query, array $types) { if (is_null($this->joinedTransactionTypes)) { $query->leftJoin( @@ -189,10 +189,10 @@ class TransactionJournal extends Eloquent * * @param $query */ - public function scopeWithRelevantData(Builder $query) + public function scopeWithRelevantData($query) { $query->with( - ['transactions' => function (Builder $q) { + ['transactions' => function (HasMany $q) { $q->orderBy('amount', 'ASC'); }, 'transactiontype', 'budgets','categories', 'transactions.account.accounttype', 'bill', 'budgets', 'categories'] );