diff --git a/app/Console/Commands/UseEncryption.php b/app/Console/Commands/UseEncryption.php index 8cf70208b9..89e6bf012d 100644 --- a/app/Console/Commands/UseEncryption.php +++ b/app/Console/Commands/UseEncryption.php @@ -1,4 +1,6 @@ $journal) { + foreach ($journals as $journal) { $sources = $journal->sourceAccountList(); $destinations = $journal->destinationAccountList(); if ($sources->count() > 1) { diff --git a/app/Http/Controllers/Transaction/SplitController.php b/app/Http/Controllers/Transaction/SplitController.php index 32f0807c3b..23225eae8f 100644 --- a/app/Http/Controllers/Transaction/SplitController.php +++ b/app/Http/Controllers/Transaction/SplitController.php @@ -114,7 +114,7 @@ class SplitController extends Controller 'transactions.split.edit', compact( 'subTitleIcon', 'currencies', 'optionalFields', - 'preFilled', 'subTitle', 'amount', 'sourceAccounts', 'uploadSize', 'destinationAccounts', 'assetAccounts', + 'preFilled', 'subTitle', 'uploadSize', 'assetAccounts', 'budgets', 'journal' ) ); diff --git a/app/Import/FileProcessor/CsvProcessor.php b/app/Import/FileProcessor/CsvProcessor.php index b10549c703..10657c35bb 100644 --- a/app/Import/FileProcessor/CsvProcessor.php +++ b/app/Import/FileProcessor/CsvProcessor.php @@ -260,8 +260,8 @@ class CsvProcessor implements FileProcessorInterface private function specifics(array $row): array { $config = $this->job->configuration; - // - foreach ($config['specifics'] as $name => $enabled) { + $names = array_keys($config['specifics']); + foreach ($names as $name) { if (!in_array($name, $this->validSpecifics)) { throw new FireflyException(sprintf('"%s" is not a valid class name', $name)); diff --git a/app/Import/Storage/ImportStorage.php b/app/Import/Storage/ImportStorage.php index eea594fdbf..8e417efc61 100644 --- a/app/Import/Storage/ImportStorage.php +++ b/app/Import/Storage/ImportStorage.php @@ -252,12 +252,11 @@ class ImportStorage /** * @param ImportJournal $importJournal - * @param Account $account * @param TransactionCurrency $localCurrency * * @return int|null */ - private function getForeignCurrencyId(ImportJournal $importJournal, Account $account, TransactionCurrency $localCurrency): ?int + private function getForeignCurrencyId(ImportJournal $importJournal, TransactionCurrency $localCurrency): ?int { // get journal currency, if any: $currency = $importJournal->getCurrency()->getTransactionCurrency(); @@ -394,7 +393,7 @@ class ImportStorage $asset = $importJournal->asset->getAccount(); $amount = $importJournal->getAmount(); $currency = $this->getCurrency($importJournal, $asset); - $foreignCurrencyId = $this->getForeignCurrencyId($importJournal, $asset, $currency); + $foreignCurrencyId = $this->getForeignCurrencyId($importJournal, $currency); $date = $importJournal->getDate($this->dateFormat); $transactionType = $this->getTransactionType($amount); $opposing = $this->getOpposingAccount($importJournal->opposing, $amount); diff --git a/app/Mail/RegisteredUser.php b/app/Mail/RegisteredUser.php index eba52d7dfa..01d7f675cd 100644 --- a/app/Mail/RegisteredUser.php +++ b/app/Mail/RegisteredUser.php @@ -1,4 +1,6 @@ $entry) { + $keys = array_keys($expenses); + foreach ($keys as $key) { if ($expenses[$key]['count'] > 1) { $expenses[$key]['average'] = bcdiv($expenses[$key]['sum'], strval($expenses[$key]['count'])); } diff --git a/app/Support/Import/Configuration/Csv/Initial.php b/app/Support/Import/Configuration/Csv/Initial.php index 1c3363eac1..fec13649e9 100644 --- a/app/Support/Import/Configuration/Csv/Initial.php +++ b/app/Support/Import/Configuration/Csv/Initial.php @@ -134,7 +134,8 @@ class Initial implements ConfigurationInterface { // loop specifics. if (isset($data['specifics']) && is_array($data['specifics'])) { - foreach ($data['specifics'] as $name => $enabled) { + $names = array_keys($data['specifics']); + foreach ($names as $name) { // verify their content. $className = sprintf('FireflyIII\Import\Specifics\%s', $name); if (class_exists($className)) { diff --git a/app/Support/Import/Configuration/Csv/Map.php b/app/Support/Import/Configuration/Csv/Map.php index cad59d1dd2..fd038857bd 100644 --- a/app/Support/Import/Configuration/Csv/Map.php +++ b/app/Support/Import/Configuration/Csv/Map.php @@ -90,10 +90,12 @@ class Map implements ConfigurationInterface } } } - foreach ($this->data as $index => $entry) { + $setIndexes = array_keys($this->data); + foreach ($setIndexes as $index) { $this->data[$index]['values'] = array_unique($this->data[$index]['values']); asort($this->data[$index]['values']); } + unset($setIndexes); // save number of rows, thus number of steps, in job: $steps = $rowIndex * 5; @@ -233,8 +235,8 @@ class Map implements ConfigurationInterface { // run specifics here: // and this is the point where the specifix go to work. - foreach ($this->configuration['specifics'] as $name => $enabled) { - + $names = array_keys($this->configuration['specifics']); + foreach ($names as $name) { if (!in_array($name, $this->validSpecifics)) { throw new FireflyException(sprintf('"%s" is not a valid class name', $name)); } diff --git a/app/Support/Import/Configuration/Csv/Roles.php b/app/Support/Import/Configuration/Csv/Roles.php index eb1d49ea3e..f75c82f4bc 100644 --- a/app/Support/Import/Configuration/Csv/Roles.php +++ b/app/Support/Import/Configuration/Csv/Roles.php @@ -228,7 +228,8 @@ class Roles implements ConfigurationInterface */ private function processSpecifics(array $row): array { - foreach ($this->job->configuration['specifics'] as $name => $enabled) { + $names = array_keys($this->configuration['specifics']); + foreach ($names as $name) { /** @var SpecificInterface $specific */ $specific = app('FireflyIII\Import\Specifics\\' . $name); $row = $specific->run($row); diff --git a/app/Support/Search/Search.php b/app/Support/Search/Search.php index ce7b271973..5da8c674cb 100644 --- a/app/Support/Search/Search.php +++ b/app/Support/Search/Search.php @@ -119,7 +119,7 @@ class Search implements SearchInterface // Filter transactions that match the given triggers. $filtered = $set->filter( - function (Transaction $transaction) use ($words) { + function (Transaction $transaction) { if ($this->matchModifiers($transaction)) { return $transaction; @@ -237,7 +237,7 @@ class Search implements SearchInterface return false; } foreach ($needle as $what) { - if (($pos = strpos($haystack, $what)) !== false) { + if (strpos($haystack, $what) !== false) { return true; } } diff --git a/config/auth.php b/config/auth.php index 9145ce8641..ddf966efc8 100644 --- a/config/auth.php +++ b/config/auth.php @@ -1,4 +1,6 @@ 'This code should be ISO compliant (Google it for your new currency).', -]; \ No newline at end of file +]; diff --git a/resources/lang/en_US/list.php b/resources/lang/en_US/list.php index 91254e401d..c44721b8d5 100644 --- a/resources/lang/en_US/list.php +++ b/resources/lang/en_US/list.php @@ -1,4 +1,6 @@ - \ No newline at end of file + diff --git a/tests/CreatesApplication.php b/tests/CreatesApplication.php index 0574e65161..b4cc8b15b2 100644 --- a/tests/CreatesApplication.php +++ b/tests/CreatesApplication.php @@ -1,4 +1,6 @@