mirror of
https://github.com/firefly-iii/firefly-iii.git
synced 2025-02-25 18:45:27 -06:00
Merge branch 'develop' into 5.8-dev
This commit is contained in:
commit
749ce2ca9b
app
Api/V1/Requests
Data/Bulk
Models
Account
Attachment
AvailableBudget
Bill
Budget
BudgetLimit
ObjectGroup
PiggyBank
Recurrence
Rule
RuleGroup
TransactionCurrency
TransactionLink
System
Http/Requests
AccountFormRequest.phpBillStoreRequest.phpBillUpdateRequest.phpBudgetFormStoreRequest.phpBudgetFormUpdateRequest.phpCurrencyFormRequest.phpJournalLinkRequest.phpPiggyBankStoreRequest.phpPiggyBankUpdateRequest.phpRecurrenceFormRequest.phpRuleFormRequest.phpUserFormRequest.php
Support/Request
@ -43,8 +43,8 @@ class MoveTransactionsRequest extends FormRequest
|
||||
public function getAll(): array
|
||||
{
|
||||
return [
|
||||
'original_account' => $this->integer('original_account'),
|
||||
'destination_account' => $this->integer('destination_account'),
|
||||
'original_account' => $this->convertInteger('original_account'),
|
||||
'destination_account' => $this->convertInteger('destination_account'),
|
||||
];
|
||||
}
|
||||
|
||||
|
@ -61,8 +61,8 @@ class StoreRequest extends FormRequest
|
||||
'include_net_worth' => $includeNetWorth,
|
||||
'account_type_name' => $this->convertString('type'),
|
||||
'account_type_id' => null,
|
||||
'currency_id' => $this->integer('currency_id'),
|
||||
'order' => $this->integer('order'),
|
||||
'currency_id' => $this->convertInteger('currency_id'),
|
||||
'order' => $this->convertInteger('order'),
|
||||
'currency_code' => $this->convertString('currency_code'),
|
||||
'virtual_balance' => $this->convertString('virtual_balance'),
|
||||
'iban' => $this->convertString('iban'),
|
||||
|
@ -67,8 +67,8 @@ class UpdateRequest extends FormRequest
|
||||
'notes' => ['notes', 'stringWithNewlines'],
|
||||
'interest' => ['interest', 'convertString'],
|
||||
'interest_period' => ['interest_period', 'convertString'],
|
||||
'order' => ['order', 'integer'],
|
||||
'currency_id' => ['currency_id', 'integer'],
|
||||
'order' => ['order', 'convertInteger'],
|
||||
'currency_id' => ['currency_id', 'convertInteger'],
|
||||
'currency_code' => ['currency_code', 'convertString'],
|
||||
'liability_direction' => ['liability_direction', 'convertString'],
|
||||
'liability_amount' => ['liability_amount', 'convertString'],
|
||||
|
@ -49,7 +49,7 @@ class StoreRequest extends FormRequest
|
||||
'title' => $this->convertString('title'),
|
||||
'notes' => $this->stringWithNewlines('notes'),
|
||||
'attachable_type' => $this->convertString('attachable_type'),
|
||||
'attachable_id' => $this->integer('attachable_id'),
|
||||
'attachable_id' => $this->convertInteger('attachable_id'),
|
||||
];
|
||||
}
|
||||
|
||||
|
@ -49,7 +49,7 @@ class UpdateRequest extends FormRequest
|
||||
'title' => ['title', 'convertString'],
|
||||
'notes' => ['notes', 'stringWithNewlines'],
|
||||
'attachable_type' => ['attachable_type', 'convertString'],
|
||||
'attachable_id' => ['attachable_id', 'integer'],
|
||||
'attachable_id' => ['attachable_id', 'convertInteger'],
|
||||
];
|
||||
|
||||
return $this->getAllData($fields);
|
||||
|
@ -47,7 +47,7 @@ class Request extends FormRequest
|
||||
{
|
||||
// this is the way:
|
||||
$fields = [
|
||||
'currency_id' => ['currency_id', 'integer'],
|
||||
'currency_id' => ['currency_id', 'convertInteger'],
|
||||
'currency_code' => ['currency_code', 'convertString'],
|
||||
'amount' => ['amount', 'convertString'],
|
||||
'start' => ['start', 'date'],
|
||||
|
@ -52,17 +52,17 @@ class StoreRequest extends FormRequest
|
||||
'name' => ['name', 'convertString'],
|
||||
'amount_min' => ['amount_min', 'convertString'],
|
||||
'amount_max' => ['amount_max', 'convertString'],
|
||||
'currency_id' => ['currency_id', 'integer'],
|
||||
'currency_id' => ['currency_id', 'convertInteger'],
|
||||
'currency_code' => ['currency_code', 'convertString'],
|
||||
'date' => ['date', 'date'],
|
||||
'end_date' => ['end_date', 'date'],
|
||||
'extension_date' => ['extension_date', 'date'],
|
||||
'repeat_freq' => ['repeat_freq', 'convertString'],
|
||||
'skip' => ['skip', 'integer'],
|
||||
'skip' => ['skip', 'convertInteger'],
|
||||
'active' => ['active', 'boolean'],
|
||||
'order' => ['order', 'integer'],
|
||||
'order' => ['order', 'convertInteger'],
|
||||
'notes' => ['notes', 'stringWithNewlines'],
|
||||
'object_group_id' => ['object_group_id', 'integer'],
|
||||
'object_group_id' => ['object_group_id', 'convertInteger'],
|
||||
'object_group_title' => ['object_group_title', 'convertString'],
|
||||
];
|
||||
|
||||
|
@ -50,17 +50,17 @@ class UpdateRequest extends FormRequest
|
||||
'name' => ['name', 'convertString'],
|
||||
'amount_min' => ['amount_min', 'convertString'],
|
||||
'amount_max' => ['amount_max', 'convertString'],
|
||||
'currency_id' => ['currency_id', 'integer'],
|
||||
'currency_id' => ['currency_id', 'convertInteger'],
|
||||
'currency_code' => ['currency_code', 'convertString'],
|
||||
'date' => ['date', 'date'],
|
||||
'end_date' => ['end_date', 'date'],
|
||||
'extension_date' => ['extension_date', 'date'],
|
||||
'repeat_freq' => ['repeat_freq', 'convertString'],
|
||||
'skip' => ['skip', 'integer'],
|
||||
'skip' => ['skip', 'convertInteger'],
|
||||
'active' => ['active', 'boolean'],
|
||||
'order' => ['order', 'integer'],
|
||||
'order' => ['order', 'convertInteger'],
|
||||
'notes' => ['notes', 'stringWithNewlines'],
|
||||
'object_group_id' => ['object_group_id', 'integer'],
|
||||
'object_group_id' => ['object_group_id', 'convertInteger'],
|
||||
'object_group_title' => ['object_group_title', 'convertString'],
|
||||
];
|
||||
|
||||
|
@ -49,11 +49,11 @@ class StoreRequest extends FormRequest
|
||||
$fields = [
|
||||
'name' => ['name', 'convertString'],
|
||||
'active' => ['active', 'boolean'],
|
||||
'order' => ['active', 'integer'],
|
||||
'order' => ['active', 'convertInteger'],
|
||||
'notes' => ['notes', 'convertString'],
|
||||
|
||||
// auto budget currency:
|
||||
'currency_id' => ['auto_budget_currency_id', 'integer'],
|
||||
'currency_id' => ['auto_budget_currency_id', 'convertInteger'],
|
||||
'currency_code' => ['auto_budget_currency_code', 'convertString'],
|
||||
'auto_budget_type' => ['auto_budget_type', 'convertString'],
|
||||
'auto_budget_amount' => ['auto_budget_amount', 'convertString'],
|
||||
|
@ -50,9 +50,9 @@ class UpdateRequest extends FormRequest
|
||||
$fields = [
|
||||
'name' => ['name', 'convertString'],
|
||||
'active' => ['active', 'boolean'],
|
||||
'order' => ['order', 'integer'],
|
||||
'order' => ['order', 'convertInteger'],
|
||||
'notes' => ['notes', 'convertString'],
|
||||
'currency_id' => ['auto_budget_currency_id', 'integer'],
|
||||
'currency_id' => ['auto_budget_currency_id', 'convertInteger'],
|
||||
'currency_code' => ['auto_budget_currency_code', 'convertString'],
|
||||
'auto_budget_type' => ['auto_budget_type', 'convertString'],
|
||||
'auto_budget_amount' => ['auto_budget_amount', 'convertString'],
|
||||
|
@ -47,7 +47,7 @@ class StoreRequest extends FormRequest
|
||||
'start' => $this->getCarbonDate('start'),
|
||||
'end' => $this->getCarbonDate('end'),
|
||||
'amount' => $this->convertString('amount'),
|
||||
'currency_id' => $this->integer('currency_id'),
|
||||
'currency_id' => $this->convertInteger('currency_id'),
|
||||
'currency_code' => $this->convertString('currency_code'),
|
||||
];
|
||||
}
|
||||
|
@ -49,7 +49,7 @@ class UpdateRequest extends FormRequest
|
||||
'start' => ['start', 'date'],
|
||||
'end' => ['end', 'date'],
|
||||
'amount' => ['amount', 'convertString'],
|
||||
'currency_id' => ['currency_id', 'integer'],
|
||||
'currency_id' => ['currency_id', 'convertInteger'],
|
||||
'currency_code' => ['currency_code', 'convertString'],
|
||||
];
|
||||
|
||||
|
@ -44,7 +44,7 @@ class UpdateRequest extends FormRequest
|
||||
{
|
||||
$fields = [
|
||||
'title' => ['title', 'convertString'],
|
||||
'order' => ['order', 'integer'],
|
||||
'order' => ['order', 'convertInteger'],
|
||||
];
|
||||
|
||||
return $this->getAllData($fields);
|
||||
|
@ -44,17 +44,17 @@ class StoreRequest extends FormRequest
|
||||
public function getAll(): array
|
||||
{
|
||||
$fields = [
|
||||
'order' => ['order', 'integer'],
|
||||
'order' => ['order', 'convertInteger'],
|
||||
];
|
||||
$data = $this->getAllData($fields);
|
||||
$data['name'] = $this->convertString('name');
|
||||
$data['account_id'] = $this->integer('account_id');
|
||||
$data['account_id'] = $this->convertInteger('account_id');
|
||||
$data['targetamount'] = $this->convertString('target_amount');
|
||||
$data['current_amount'] = $this->convertString('current_amount');
|
||||
$data['startdate'] = $this->getCarbonDate('start_date');
|
||||
$data['targetdate'] = $this->getCarbonDate('target_date');
|
||||
$data['notes'] = $this->stringWithNewlines('notes');
|
||||
$data['object_group_id'] = $this->integer('object_group_id');
|
||||
$data['object_group_id'] = $this->convertInteger('object_group_id');
|
||||
$data['object_group_title'] = $this->convertString('object_group_title');
|
||||
|
||||
return $data;
|
||||
|
@ -47,15 +47,15 @@ class UpdateRequest extends FormRequest
|
||||
{
|
||||
$fields = [
|
||||
'name' => ['name', 'convertString'],
|
||||
'account_id' => ['account_id', 'integer'],
|
||||
'account_id' => ['account_id', 'convertInteger'],
|
||||
'targetamount' => ['target_amount', 'convertString'],
|
||||
'current_amount' => ['current_amount', 'convertString'],
|
||||
'startdate' => ['start_date', 'date'],
|
||||
'targetdate' => ['target_date', 'convertString'],
|
||||
'notes' => ['notes', 'stringWithNewlines'],
|
||||
'order' => ['order', 'integer'],
|
||||
'order' => ['order', 'convertInteger'],
|
||||
'object_group_title' => ['object_group_title', 'convertString'],
|
||||
'object_group_id' => ['object_group_id', 'integer'],
|
||||
'object_group_id' => ['object_group_id', 'convertInteger'],
|
||||
];
|
||||
|
||||
return $this->getAllData($fields);
|
||||
|
@ -54,7 +54,7 @@ class StoreRequest extends FormRequest
|
||||
'description' => ['description', 'convertString'],
|
||||
'first_date' => ['first_date', 'date'],
|
||||
'repeat_until' => ['repeat_until', 'date'],
|
||||
'nr_of_repetitions' => ['nr_of_repetitions', 'integer'],
|
||||
'nr_of_repetitions' => ['nr_of_repetitions', 'convertInteger'],
|
||||
'apply_rules' => ['apply_rules', 'boolean'],
|
||||
'active' => ['active', 'boolean'],
|
||||
'notes' => ['notes', 'stringWithNewlines'],
|
||||
|
@ -55,7 +55,7 @@ class UpdateRequest extends FormRequest
|
||||
'description' => ['description', 'convertString'],
|
||||
'first_date' => ['first_date', 'date'],
|
||||
'repeat_until' => ['repeat_until', 'date'],
|
||||
'nr_of_repetitions' => ['nr_of_repetitions', 'integer'],
|
||||
'nr_of_repetitions' => ['nr_of_repetitions', 'convertInteger'],
|
||||
'apply_rules' => ['apply_rules', 'boolean'],
|
||||
'active' => ['active', 'boolean'],
|
||||
'notes' => ['notes', 'convertString'],
|
||||
|
@ -48,8 +48,8 @@ class StoreRequest extends FormRequest
|
||||
$fields = [
|
||||
'title' => ['title', 'convertString'],
|
||||
'description' => ['description', 'convertString'],
|
||||
'rule_group_id' => ['rule_group_id', 'integer'],
|
||||
'order' => ['order', 'integer'],
|
||||
'rule_group_id' => ['rule_group_id', 'convertInteger'],
|
||||
'order' => ['order', 'convertInteger'],
|
||||
'rule_group_title' => ['rule_group_title', 'convertString'],
|
||||
'trigger' => ['trigger', 'convertString'],
|
||||
'strict' => ['strict', 'boolean'],
|
||||
|
@ -48,12 +48,12 @@ class UpdateRequest extends FormRequest
|
||||
$fields = [
|
||||
'title' => ['title', 'convertString'],
|
||||
'description' => ['description', 'stringWithNewlines'],
|
||||
'rule_group_id' => ['rule_group_id', 'integer'],
|
||||
'rule_group_id' => ['rule_group_id', 'convertInteger'],
|
||||
'trigger' => ['trigger', 'convertString'],
|
||||
'strict' => ['strict', 'boolean'],
|
||||
'stop_processing' => ['stop_processing', 'boolean'],
|
||||
'active' => ['active', 'boolean'],
|
||||
'order' => ['order', 'integer'],
|
||||
'order' => ['order', 'convertInteger'],
|
||||
];
|
||||
|
||||
$return = $this->getAllData($fields);
|
||||
|
@ -49,7 +49,7 @@ class StoreRequest extends FormRequest
|
||||
$active = $this->boolean('active');
|
||||
}
|
||||
if (null !== $this->get('order')) {
|
||||
$order = $this->integer('order');
|
||||
$order = $this->convertInteger('order');
|
||||
}
|
||||
|
||||
return [
|
||||
|
@ -48,7 +48,7 @@ class UpdateRequest extends FormRequest
|
||||
'title' => ['title', 'convertString'],
|
||||
'description' => ['description', 'stringWithNewlines'],
|
||||
'active' => ['active', 'boolean'],
|
||||
'order' => ['order', 'integer'],
|
||||
'order' => ['order', 'convertInteger'],
|
||||
];
|
||||
|
||||
return $this->getAllData($fields);
|
||||
|
@ -57,7 +57,7 @@ class StoreRequest extends FormRequest
|
||||
'name' => $this->convertString('name'),
|
||||
'code' => $this->convertString('code'),
|
||||
'symbol' => $this->convertString('symbol'),
|
||||
'decimal_places' => $this->integer('decimal_places'),
|
||||
'decimal_places' => $this->convertInteger('decimal_places'),
|
||||
'default' => $default,
|
||||
'enabled' => $enabled,
|
||||
];
|
||||
|
@ -49,7 +49,7 @@ class UpdateRequest extends FormRequest
|
||||
'name' => ['name', 'convertString'],
|
||||
'code' => ['code', 'convertString'],
|
||||
'symbol' => ['symbol', 'convertString'],
|
||||
'decimal_places' => ['decimal_places', 'integer'],
|
||||
'decimal_places' => ['decimal_places', 'convertInteger'],
|
||||
'default' => ['default', 'boolean'],
|
||||
'enabled' => ['enabled', 'boolean'],
|
||||
];
|
||||
|
@ -46,10 +46,10 @@ class StoreRequest extends FormRequest
|
||||
public function getAll(): array
|
||||
{
|
||||
return [
|
||||
'link_type_id' => $this->integer('link_type_id'),
|
||||
'link_type_id' => $this->convertInteger('link_type_id'),
|
||||
'link_type_name' => $this->convertString('link_type_name'),
|
||||
'inward_id' => $this->integer('inward_id'),
|
||||
'outward_id' => $this->integer('outward_id'),
|
||||
'inward_id' => $this->convertInteger('inward_id'),
|
||||
'outward_id' => $this->convertInteger('outward_id'),
|
||||
'notes' => $this->stringWithNewlines('notes'),
|
||||
];
|
||||
}
|
||||
|
@ -46,10 +46,10 @@ class UpdateRequest extends FormRequest
|
||||
public function getAll(): array
|
||||
{
|
||||
return [
|
||||
'link_type_id' => $this->integer('link_type_id'),
|
||||
'link_type_id' => $this->convertInteger('link_type_id'),
|
||||
'link_type_name' => $this->convertString('link_type_name'),
|
||||
'inward_id' => $this->integer('inward_id'),
|
||||
'outward_id' => $this->integer('outward_id'),
|
||||
'inward_id' => $this->convertInteger('inward_id'),
|
||||
'outward_id' => $this->convertInteger('outward_id'),
|
||||
'notes' => $this->stringWithNewlines('notes'),
|
||||
];
|
||||
}
|
||||
|
@ -54,7 +54,7 @@ class UpdateRequest extends FormRequest
|
||||
return ['value' => $this->boolean('value')];
|
||||
case 'configuration.permission_update_check':
|
||||
case 'configuration.last_update_check':
|
||||
return ['value' => $this->integer('value')];
|
||||
return ['value' => $this->convertInteger('value')];
|
||||
}
|
||||
|
||||
return ['value' => $this->convertString('value')];
|
||||
|
@ -48,7 +48,7 @@ class AccountFormRequest extends FormRequest
|
||||
'name' => $this->convertString('name'),
|
||||
'active' => $this->boolean('active'),
|
||||
'account_type_name' => $this->convertString('objectType'),
|
||||
'currency_id' => $this->integer('currency_id'),
|
||||
'currency_id' => $this->convertInteger('currency_id'),
|
||||
'virtual_balance' => $this->convertString('virtual_balance'),
|
||||
'iban' => $this->convertString('iban'),
|
||||
'BIC' => $this->convertString('BIC'),
|
||||
@ -77,7 +77,7 @@ class AccountFormRequest extends FormRequest
|
||||
// that could have been selected.
|
||||
if ('liabilities' === $data['account_type_name']) {
|
||||
$data['account_type_name'] = null;
|
||||
$data['account_type_id'] = $this->integer('liability_type_id');
|
||||
$data['account_type_id'] = $this->convertInteger('liability_type_id');
|
||||
if ('' !== $data['opening_balance']) {
|
||||
$data['opening_balance'] = app('steam')->negative($data['opening_balance']);
|
||||
}
|
||||
|
@ -43,14 +43,14 @@ class BillStoreRequest extends FormRequest
|
||||
return [
|
||||
'name' => $this->convertString('name'),
|
||||
'amount_min' => $this->convertString('amount_min'),
|
||||
'currency_id' => $this->integer('transaction_currency_id'),
|
||||
'currency_id' => $this->convertInteger('transaction_currency_id'),
|
||||
'currency_code' => '',
|
||||
'amount_max' => $this->convertString('amount_max'),
|
||||
'date' => $this->getCarbonDate('date'),
|
||||
'end_date' => $this->getCarbonDate('bill_end_date'),
|
||||
'extension_date' => $this->getCarbonDate('extension_date'),
|
||||
'repeat_freq' => $this->convertString('repeat_freq'),
|
||||
'skip' => $this->integer('skip'),
|
||||
'skip' => $this->convertInteger('skip'),
|
||||
'notes' => $this->stringWithNewlines('notes'),
|
||||
'active' => $this->boolean('active'),
|
||||
'object_group_title' => $this->convertString('object_group'),
|
||||
|
@ -44,14 +44,14 @@ class BillUpdateRequest extends FormRequest
|
||||
return [
|
||||
'name' => $this->convertString('name'),
|
||||
'amount_min' => $this->convertString('amount_min'),
|
||||
'currency_id' => $this->integer('transaction_currency_id'),
|
||||
'currency_id' => $this->convertInteger('transaction_currency_id'),
|
||||
'currency_code' => '',
|
||||
'amount_max' => $this->convertString('amount_max'),
|
||||
'date' => $this->getCarbonDate('date'),
|
||||
'end_date' => $this->getCarbonDate('bill_end_date'),
|
||||
'extension_date' => $this->getCarbonDate('extension_date'),
|
||||
'repeat_freq' => $this->convertString('repeat_freq'),
|
||||
'skip' => $this->integer('skip'),
|
||||
'skip' => $this->convertInteger('skip'),
|
||||
'notes' => $this->stringWithNewlines('notes'),
|
||||
'active' => $this->boolean('active'),
|
||||
'object_group_title' => $this->convertString('object_group'),
|
||||
|
@ -46,8 +46,8 @@ class BudgetFormStoreRequest extends FormRequest
|
||||
return [
|
||||
'name' => $this->convertString('name'),
|
||||
'active' => $this->boolean('active'),
|
||||
'auto_budget_type' => $this->integer('auto_budget_type'),
|
||||
'currency_id' => $this->integer('auto_budget_currency_id'),
|
||||
'auto_budget_type' => $this->convertInteger('auto_budget_type'),
|
||||
'currency_id' => $this->convertInteger('auto_budget_currency_id'),
|
||||
'auto_budget_amount' => $this->convertString('auto_budget_amount'),
|
||||
'auto_budget_period' => $this->convertString('auto_budget_period'),
|
||||
];
|
||||
|
@ -47,8 +47,8 @@ class BudgetFormUpdateRequest extends FormRequest
|
||||
return [
|
||||
'name' => $this->convertString('name'),
|
||||
'active' => $this->boolean('active'),
|
||||
'auto_budget_type' => $this->integer('auto_budget_type'),
|
||||
'currency_id' => $this->integer('auto_budget_currency_id'),
|
||||
'auto_budget_type' => $this->convertInteger('auto_budget_type'),
|
||||
'currency_id' => $this->convertInteger('auto_budget_currency_id'),
|
||||
'auto_budget_amount' => $this->convertString('auto_budget_amount'),
|
||||
'auto_budget_period' => $this->convertString('auto_budget_period'),
|
||||
];
|
||||
|
@ -45,7 +45,7 @@ class CurrencyFormRequest extends FormRequest
|
||||
'name' => $this->convertString('name'),
|
||||
'code' => $this->convertString('code'),
|
||||
'symbol' => $this->convertString('symbol'),
|
||||
'decimal_places' => $this->integer('decimal_places'),
|
||||
'decimal_places' => $this->convertInteger('decimal_places'),
|
||||
'enabled' => $this->boolean('enabled'),
|
||||
];
|
||||
}
|
||||
|
@ -45,7 +45,7 @@ class JournalLinkRequest extends FormRequest
|
||||
$linkType = $this->get('link_type');
|
||||
$parts = explode('_', $linkType);
|
||||
$return['link_type_id'] = (int) $parts[0];
|
||||
$return['transaction_journal_id'] = $this->integer('opposing');
|
||||
$return['transaction_journal_id'] = $this->convertInteger('opposing');
|
||||
$return['notes'] = $this->convertString('notes');
|
||||
$return['direction'] = $parts[1];
|
||||
|
||||
|
@ -43,7 +43,7 @@ class PiggyBankStoreRequest extends FormRequest
|
||||
return [
|
||||
'name' => $this->convertString('name'),
|
||||
'startdate' => $this->getCarbonDate('startdate'),
|
||||
'account_id' => $this->integer('account_id'),
|
||||
'account_id' => $this->convertInteger('account_id'),
|
||||
'targetamount' => $this->convertString('targetamount'),
|
||||
'targetdate' => $this->getCarbonDate('targetdate'),
|
||||
'notes' => $this->stringWithNewlines('notes'),
|
||||
|
@ -44,7 +44,7 @@ class PiggyBankUpdateRequest extends FormRequest
|
||||
return [
|
||||
'name' => $this->convertString('name'),
|
||||
'startdate' => $this->getCarbonDate('startdate'),
|
||||
'account_id' => $this->integer('account_id'),
|
||||
'account_id' => $this->convertInteger('account_id'),
|
||||
'targetamount' => $this->convertString('targetamount'),
|
||||
'targetdate' => $this->getCarbonDate('targetdate'),
|
||||
'notes' => $this->stringWithNewlines('notes'),
|
||||
|
@ -61,14 +61,14 @@ class RecurrenceFormRequest extends FormRequest
|
||||
'description' => $this->convertString('recurring_description'),
|
||||
'first_date' => $this->getCarbonDate('first_date'),
|
||||
'repeat_until' => $this->getCarbonDate('repeat_until'),
|
||||
'nr_of_repetitions' => $this->integer('repetitions'),
|
||||
'nr_of_repetitions' => $this->convertInteger('repetitions'),
|
||||
'apply_rules' => $this->boolean('apply_rules'),
|
||||
'active' => $this->boolean('active'),
|
||||
'repetition_end' => $this->convertString('repetition_end'),
|
||||
],
|
||||
'transactions' => [
|
||||
[
|
||||
'currency_id' => $this->integer('transaction_currency_id'),
|
||||
'currency_id' => $this->convertInteger('transaction_currency_id'),
|
||||
'currency_code' => null,
|
||||
'type' => $this->convertString('transaction_type'),
|
||||
'description' => $this->convertString('transaction_description'),
|
||||
@ -76,14 +76,14 @@ class RecurrenceFormRequest extends FormRequest
|
||||
'foreign_amount' => null,
|
||||
'foreign_currency_id' => null,
|
||||
'foreign_currency_code' => null,
|
||||
'budget_id' => $this->integer('budget_id'),
|
||||
'budget_id' => $this->convertInteger('budget_id'),
|
||||
'budget_name' => null,
|
||||
'bill_id' => $this->integer('bill_id'),
|
||||
'bill_id' => $this->convertInteger('bill_id'),
|
||||
'bill_name' => null,
|
||||
'category_id' => null,
|
||||
'category_name' => $this->convertString('category'),
|
||||
'tags' => '' !== $this->convertString('tags') ? explode(',', $this->convertString('tags')) : [],
|
||||
'piggy_bank_id' => $this->integer('piggy_bank_id'),
|
||||
'piggy_bank_id' => $this->convertInteger('piggy_bank_id'),
|
||||
'piggy_bank_name' => null,
|
||||
],
|
||||
],
|
||||
@ -91,8 +91,8 @@ class RecurrenceFormRequest extends FormRequest
|
||||
[
|
||||
'type' => $repetitionData['type'],
|
||||
'moment' => $repetitionData['moment'],
|
||||
'skip' => $this->integer('skip'),
|
||||
'weekend' => $this->integer('weekend'),
|
||||
'skip' => $this->convertInteger('skip'),
|
||||
'weekend' => $this->convertInteger('weekend'),
|
||||
],
|
||||
],
|
||||
|
||||
@ -101,7 +101,7 @@ class RecurrenceFormRequest extends FormRequest
|
||||
// fill in foreign currency data
|
||||
if (null !== $this->float('foreign_amount')) {
|
||||
$return['transactions'][0]['foreign_amount'] = $this->convertString('foreign_amount');
|
||||
$return['transactions'][0]['foreign_currency_id'] = $this->integer('foreign_currency_id');
|
||||
$return['transactions'][0]['foreign_currency_id'] = $this->convertInteger('foreign_currency_id');
|
||||
}
|
||||
// default values:
|
||||
$return['transactions'][0]['source_id'] = null;
|
||||
@ -113,16 +113,16 @@ class RecurrenceFormRequest extends FormRequest
|
||||
default:
|
||||
throw new FireflyException(sprintf('Cannot handle transaction type "%s"', $this->convertString('transaction_type')));
|
||||
case 'withdrawal':
|
||||
$return['transactions'][0]['source_id'] = $this->integer('source_id');
|
||||
$return['transactions'][0]['destination_id'] = $this->integer('withdrawal_destination_id');
|
||||
$return['transactions'][0]['source_id'] = $this->convertInteger('source_id');
|
||||
$return['transactions'][0]['destination_id'] = $this->convertInteger('withdrawal_destination_id');
|
||||
break;
|
||||
case 'deposit':
|
||||
$return['transactions'][0]['source_id'] = $this->integer('deposit_source_id');
|
||||
$return['transactions'][0]['destination_id'] = $this->integer('destination_id');
|
||||
$return['transactions'][0]['source_id'] = $this->convertInteger('deposit_source_id');
|
||||
$return['transactions'][0]['destination_id'] = $this->convertInteger('destination_id');
|
||||
break;
|
||||
case 'transfer':
|
||||
$return['transactions'][0]['source_id'] = $this->integer('source_id');
|
||||
$return['transactions'][0]['destination_id'] = $this->integer('destination_id');
|
||||
$return['transactions'][0]['source_id'] = $this->convertInteger('source_id');
|
||||
$return['transactions'][0]['destination_id'] = $this->convertInteger('destination_id');
|
||||
break;
|
||||
}
|
||||
|
||||
@ -219,7 +219,7 @@ class RecurrenceFormRequest extends FormRequest
|
||||
'category' => 'between:1,255|nullable',
|
||||
'tags' => 'between:1,255|nullable',
|
||||
];
|
||||
if ($this->integer('foreign_currency_id') > 0) {
|
||||
if ($this->convertInteger('foreign_currency_id') > 0) {
|
||||
$rules['foreign_currency_id'] = 'exists:transaction_currencies,id';
|
||||
}
|
||||
|
||||
|
@ -45,7 +45,7 @@ class RuleFormRequest extends FormRequest
|
||||
{
|
||||
return [
|
||||
'title' => $this->convertString('title'),
|
||||
'rule_group_id' => $this->integer('rule_group_id'),
|
||||
'rule_group_id' => $this->convertInteger('rule_group_id'),
|
||||
'active' => $this->boolean('active'),
|
||||
'trigger' => $this->convertString('trigger'),
|
||||
'description' => $this->stringWithNewlines('description'),
|
||||
|
@ -44,10 +44,10 @@ class UserFormRequest extends FormRequest
|
||||
{
|
||||
return [
|
||||
'email' => $this->convertString('email'),
|
||||
'blocked' => 1 === $this->integer('blocked'),
|
||||
'blocked' => 1 === $this->convertInteger('blocked'),
|
||||
'blocked_code' => $this->convertString('blocked_code'),
|
||||
'password' => $this->convertString('password'),
|
||||
'is_owner' => 1 === $this->integer('is_owner'),
|
||||
'is_owner' => 1 === $this->convertInteger('is_owner'),
|
||||
];
|
||||
}
|
||||
|
||||
|
@ -39,7 +39,7 @@ trait ConvertsDataTypes
|
||||
*
|
||||
* @return int
|
||||
*/
|
||||
public function integer(string $field): int
|
||||
public function convertInteger(string $field): int
|
||||
{
|
||||
return (int) $this->get($field);
|
||||
}
|
||||
|
Loading…
Reference in New Issue
Block a user