kenmirrors/firefly-iii
1
0
Fork 0
mirror of https://github.com/firefly-iii/firefly-iii.git synced 2025-12-17 20:08:52 +00:00
Code Issues Projects Releases Wiki Activity

Code cleanup.

Browse Source
This commit is contained in:
James Cole
2021-03-28 11:43:07 +02:00
parent 5abbb8a807
commit 2c575f3ca5
379 changed files with 304 additions and 1727 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
app/Services/Internal/Destroy/AccountDestroyService.php
View File

@@ -66,8 +66,6 @@ class AccountDestroyService
// delete account meta:
$account->accountMeta()->delete();
// delete account.
try {
$account->delete();

2
app/Services/Internal/Support/AccountServiceTrait.php
View File

@@ -66,8 +66,6 @@ trait AccountServiceTrait
return null;
}
return $iban;
}

2
app/Services/Internal/Support/BillServiceTrait.php
View File

@@ -61,8 +61,6 @@ trait BillServiceTrait
$ruleAction->save();
}
}
/**
* @param Bill $bill
* @param string $note

4
app/Services/Internal/Support/RecurringTransactionTrait.php
View File

@@ -41,8 +41,6 @@ use FireflyIII\Models\RecurrenceTransactionMeta;
use FireflyIII\Repositories\Account\AccountRepositoryInterface;
use FireflyIII\Validation\AccountValidator;
use Log;
/**
* Trait RecurringTransactionTrait
*
@@ -136,8 +134,6 @@ trait RecurringTransactionTrait
$validator = app(AccountValidator::class);
$validator->setUser($recurrence->user);
$validator->setTransactionType($recurrence->transactionType->type);
if (!$validator->validateSource($source->id, null, null)) {
throw new FireflyException(sprintf('Source invalid: %s', $validator->sourceError)); // @codeCoverageIgnore
}

2
app/Services/Internal/Update/BillUpdateService.php
View File

@@ -76,8 +76,6 @@ class BillUpdateService
'amount_max' => $bill->amount_max,
'transaction_currency_name' => $bill->transactionCurrency->name,
];
// update note:
if (array_key_exists('notes', $data)) {
$this->updateNote($bill, (string)$data['notes']);

4
app/Services/Internal/Update/CategoryUpdateService.php
View File

@@ -39,8 +39,6 @@ use Log;
class CategoryUpdateService
{
private $user;
/**
* Constructor.
*/
@@ -138,8 +136,6 @@ class CategoryUpdateService
->where('rt_meta.value', $oldName)
->update(['rt_meta.value' => $newName]);
}
/**
* @param Category $category
* @param array $data

4
app/Services/Internal/Update/GroupCloneService.php
View File

@@ -51,8 +51,6 @@ class GroupCloneService
foreach ($group->transactionJournals as $journal) {
$this->cloneJournal($journal, $newGroup, (int)$group->id);
}
return $newGroup;
}
@@ -147,6 +145,4 @@ class GroupCloneService
$newMeta->save();
}
}
}

14
app/Services/Internal/Update/JournalUpdateService.php
View File

@@ -145,8 +145,6 @@ class JournalUpdateService
$this->updateField('description');
$this->updateField('date');
$this->updateField('order');
$this->transactionJournal->save();
$this->transactionJournal->refresh();
@@ -294,8 +292,6 @@ class JournalUpdateService
$validator->setTransactionType($expectedType);
$validator->setUser($this->transactionJournal->user);
$validator->source = $this->getValidSourceAccount();
$result = $validator->validateDestination($destId, $destName, null);
Log::debug(sprintf('hasValidDestinationAccount(%d, "%s") will return %s', $destId, $destName, var_export($result, true)));
@@ -393,8 +389,6 @@ class JournalUpdateService
// refresh transactions.
$this->sourceTransaction->refresh();
$this->destinationTransaction->refresh();
Log::debug(sprintf('Will set source to #%d ("%s")', $source->id, $source->name));
Log::debug(sprintf('Will set dest to #%d ("%s")', $destination->id, $destination->name));
}
@@ -505,8 +499,6 @@ class JournalUpdateService
// do some parsing.
Log::debug(sprintf('Create date value from string "%s".', $value));
}
$this->transactionJournal->$fieldName = $value;
Log::debug(sprintf('Updated %s', $fieldName));
}
@@ -692,13 +684,9 @@ class JournalUpdateService
$origSourceTransaction = $this->getSourceTransaction();
$origSourceTransaction->amount = app('steam')->negative($amount);
$origSourceTransaction->save();
$destTransaction = $this->getDestinationTransaction();
$destTransaction->amount = app('steam')->positive($amount);
$destTransaction->save();
// refresh transactions.
$this->sourceTransaction->refresh();
$this->destinationTransaction->refresh();
@@ -738,8 +726,6 @@ class JournalUpdateService
$source->foreign_currency_id = $foreignCurrency->id;
$source->foreign_amount = app('steam')->negative($foreignAmount);
$source->save();
$dest->foreign_currency_id = $foreignCurrency->id;
$dest->foreign_amount = app('steam')->positive($foreignAmount);
$dest->save();

4
app/Services/Internal/Update/RecurrenceUpdateService.php
View File

@@ -103,8 +103,6 @@ class RecurrenceUpdateService
$this->updateRepetitions($recurrence, $data['repetitions'] ?? []);
}
// update all transactions:
// update all transactions (and associated meta-data)
if (array_key_exists('transactions', $data)) {
$this->updateTransactions($recurrence, $data['transactions'] ?? []);
@@ -240,8 +238,6 @@ class RecurrenceUpdateService
return;
}
$currencyFactory = app(TransactionCurrencyFactory::class);
// loop all and try to match them:
if ($originalCount === count($transactions)) {
Log::debug('Loop and find');