IntenseFinance/firefly-iii
3
0
Fork 0
mirror of https://github.com/firefly-iii/firefly-iii.git synced 2024-11-26 10:51:12 -06:00
Code Issues Packages Projects Releases Wiki Activity

Fix #2422

Browse Source
This commit is contained in:
James Cole 2019-08-14 20:23:11 +02:00
parent 1b97c4d58f
commit 12641f96b1
No known key found for this signature in database
GPG Key ID: C16961E655E74B5E
2 changed files with 7 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
app/Http/Controllers/Rule/SelectController.php
View File

@ -181,10 +181,10 @@ class SelectController extends Controller
// Warn the user if only a subset of transactions is returned
$warning = '';
if ($matchingTransactions->count() === $limit) {
if (count($matchingTransactions) === $limit) {
$warning = (string)trans('firefly.warning_transaction_subset', ['max_num_transactions' => $limit]); // @codeCoverageIgnore
}
if (0 === $matchingTransactions->count()) {
if (0 === count($matchingTransactions)) {
$warning = (string)trans('firefly.warning_no_matching_transactions', ['num_transactions' => $range]); // @codeCoverageIgnore
}

12
app/TransactionRules/TransactionMatcher.php
View File

@ -107,10 +107,10 @@ class TransactionMatcher
* transaction journals matching the given $triggers. This is accomplished by trying to fire these
* triggers onto each transaction journal until enough matches are found ($limit).
*
* @return Collection
* @return array
* @throws \FireflyIII\Exceptions\FireflyException
*/
public function findTransactionsByTriggers(): Collection
public function findTransactionsByTriggers(): array
{
if (0 === count($this->triggers)) {
return new Collection;
@ -125,9 +125,7 @@ class TransactionMatcher
// If the list of matchingTransactions is larger than the maximum number of results
// (e.g. if a large percentage of the transactions match), truncate the list
$result = $result->slice(0, $this->searchLimit);
return $result;
return array_slice($result, 0, $this->searchLimit);
}
/**
@ -251,7 +249,7 @@ class TransactionMatcher
*
* @param Processor $processor
*
* @return Collection
* @return array
*/
private function runProcessor(Processor $processor): array
{
@ -265,7 +263,7 @@ class TransactionMatcher
// - all transactions have been fetched from the database
// - the maximum number of transactions to return has been found
// - the maximum number of transactions to search in have been searched
$pageSize = min($this->searchLimit, min($this->triggeredLimit, 50));
$pageSize = $this->searchLimit;
$processed = 0;
$page = 1;
$totalResult = [];