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 the last issues. Now onto code complexity.

Browse Source
This commit is contained in:
James Cole 2015-07-06 18:48:17 +02:00
parent cd08484a13
commit 52df2edc8f
2 changed files with 12 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
app/Repositories/Account/AccountRepository.php
View File

@ -389,7 +389,9 @@ class AccountRepository implements AccountRepositoryInterface
public function store(array $data) public function store(array $data)
{ {
$newAccount = $this->storeAccount($data); $newAccount = $this->storeAccount($data);
$this->storeMetadata($newAccount, $data); if (!is_null($newAccount)) {
$this->storeMetadata($newAccount, $data);
}
// continue with the opposing account: // continue with the opposing account:
@ -457,7 +459,9 @@ class AccountRepository implements AccountRepositoryInterface
'virtualBalance' => 0, 'virtualBalance' => 0,
]; ];
$opposing = $this->storeAccount($opposingData); $opposing = $this->storeAccount($opposingData);
$this->storeInitialBalance($account, $opposing, $data); if (!is_null($opposing)) {
$this->storeInitialBalance($account, $opposing, $data);
}
} }
} else { } else {

10
app/Repositories/Journal/JournalRepository.php
View File

@ -143,7 +143,9 @@ class JournalRepository implements JournalRepositoryInterface
foreach ($array as $name) { foreach ($array as $name) {
if (strlen(trim($name)) > 0) { if (strlen(trim($name)) > 0) {
$tag = Tag::firstOrCreateEncrypted(['tag' => $name, 'user_id' => $journal->user_id]); $tag = Tag::firstOrCreateEncrypted(['tag' => $name, 'user_id' => $journal->user_id]);
$tagRepository->connect($journal, $tag); if (!is_null($tag)) {
$tagRepository->connect($journal, $tag);
}
} }
} }
} }
@ -186,19 +188,19 @@ class JournalRepository implements JournalRepositoryInterface
} }
// store accounts (depends on type) // store accounts (depends on type)
list($from, $to) = $this->storeAccounts($transactionType, $data); list($fromAccount, $toAccount) = $this->storeAccounts($transactionType, $data);
// store accompanying transactions. // store accompanying transactions.
Transaction::create( // first transaction. Transaction::create( // first transaction.
[ [
'account_id' => $from->id, 'account_id' => $fromAccount->id,
'transaction_journal_id' => $journal->id, 'transaction_journal_id' => $journal->id,
'amount' => $data['amount'] * -1 'amount' => $data['amount'] * -1
] ]
); );
Transaction::create( // second transaction. Transaction::create( // second transaction.
[ [
'account_id' => $to->id, 'account_id' => $toAccount->id,
'transaction_journal_id' => $journal->id, 'transaction_journal_id' => $journal->id,
'amount' => $data['amount'] 'amount' => $data['amount']
] ]