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

Made performance settings for testing triggers configurable

Browse Source
This commit is contained in:
Robert Horlings 2016-02-17 16:39:26 +01:00
parent 7840a5ea49
commit 919187f7fd
2 changed files with 11 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
app/Http/Controllers/RuleController.php
View File

@ -18,6 +18,7 @@ use FireflyIII\Models\RuleGroup;
use FireflyIII\Models\RuleTrigger;
use FireflyIII\Repositories\Rule\RuleRepositoryInterface;
use FireflyIII\Repositories\RuleGroup\RuleGroupRepositoryInterface;
use FireflyIII\Rules\TransactionMatcher;
use Illuminate\Database\Eloquent\Relations\HasMany;
use Input;
use Preferences;
@ -25,7 +26,7 @@ use Response;
use Session;
use URL;
use View;
use FireflyIII\Rules\TransactionMatcher;
use Config;
/**
* Class RuleController
@ -348,9 +349,8 @@ class RuleController extends Controller
// We start searching for transactions. For performance reasons, there are limits
// to the search: a maximum number of results and a maximum number of transactions
// to search in
// TODO: Make these values configurable
$maxResults = 50;
$maxTransactionsToSearchIn = 1000;
$maxResults = Config::get('firefly.test-triggers.limit');
$maxTransactionsToSearchIn = Config::get('firefly.test-triggers.max_transactions_to_analyse');
// Dispatch the actual work to a matched object
$matchingTransactions =

8
config/firefly.php
View File

@ -219,5 +219,11 @@ return [
'append_description',
'prepend_description',
],
'test-triggers' => [
// The maximum number of transactions shown when testing a list of triggers
'limit' => 50,
// The maximum number of transactions to analyse, when testing a list of triggers
'max_transactions_to_analyse' => 1000
]
];