Merge branch 'release/5.7.4'

5.7.3

.github/dependabot.yml

@@ -19,3 +19,10 @@ updates:
     schedule:
       interval: "weekly"
 
+  - package-ecosystem: "github-actions"
+    directory: "/" 
+    target-branch: develop
+    labels: ["bug"]
+    versioning-strategy: increase
+    schedule:
+      interval: "weekly"

.github/workflows/lock.yml

@@ -5,8 +5,14 @@ on:
   schedule:
     - cron: '0 0 * * *'
 
+permissions:
+  contents: read
+
 jobs:
   lock:
+    permissions:
+      issues: write  # for dessant/lock-threads to lock issues
+      pull-requests: write  # for dessant/lock-threads to lock PRs
     runs-on: ubuntu-latest
     steps:
       - uses: dessant/lock-threads@v2

.github/workflows/stale.yml

@@ -3,8 +3,14 @@ on:
   schedule:
     - cron: "30 1 * * *"
 
+permissions:
+  contents: read
+
 jobs:
   stale:
+    permissions:
+      issues: write  # for actions/stale to close stale issues
+      pull-requests: write  # for actions/stale to close stale PRs
     runs-on: ubuntu-latest
     steps:
       - uses: actions/stale@v3

app/Api/V1/Controllers/Controller.php

@@ -73,7 +73,7 @@ abstract class Controller extends BaseController
     }
 
     /**
-     * Method to grab all parameters from the URI.
+     * Method to grab all parameters from the URL.
      *
      * @return ParameterBag
      * @throws ContainerExceptionInterface
@@ -148,7 +148,7 @@ abstract class Controller extends BaseController
 
 
     /**
-     * Method to help build URI's.
+     * Method to help build URL's.
      *
      * @return string
      */

app/Api/V1/Controllers/Models/Transaction/StoreController.php

@@ -125,7 +125,7 @@ class StoreController extends Controller
 
         $selectedGroup = $collector->getGroups()->first();
         if (null === $selectedGroup) {
-            throw new FireflyException('Cannot find transaction. Possibly, a rule deleted this transaction after its creation.');
+            throw new FireflyException('200032: Cannot find transaction. Possibly, a rule deleted this transaction after its creation.');
         }
         /** @var TransactionGroupTransformer $transformer */
         $transformer = app(TransactionGroupTransformer::class);

app/Api/V1/Controllers/Models/TransactionCurrency/ListController.php

@@ -103,7 +103,7 @@ class ListController extends Controller
     {
         $manager = $this->getManager();
 
-        // read type from URI
+        // read type from URL
         $type = $request->get('type') ?? 'all';
         $this->parameters->set('type', $type);
 

app/Api/V1/Controllers/Models/TransactionLink/ShowController.php

@@ -79,7 +79,7 @@ class ShowController extends Controller
     {
         // create some objects:
         $manager = $this->getManager();
-        // read type from URI
+        // read type from URL
         $name = $request->get('name');
 
         // types to get, page size:

app/Api/V1/Requests/Autocomplete/AutocompleteRequest.php

@@ -40,7 +40,7 @@ class AutocompleteRequest extends FormRequest
      */
     public function getData(): array
     {
-        $types = $this->string('types');
+        $types = $this->convertString('types');
         $array = [];
         if ('' !== $types) {
             $array = explode(',', $types);
@@ -53,7 +53,7 @@ class AutocompleteRequest extends FormRequest
 
         return [
             'types' => $array,
-            'query' => $this->string('query'),
+            'query' => $this->convertString('query'),
             'date'  => $this->getCarbonDate('date'),
             'limit' => $limit,
         ];

app/Api/V1/Requests/Data/Export/ExportRequest.php

@@ -43,9 +43,9 @@ class ExportRequest extends FormRequest
         $result     = [
             'start' => $this->getCarbonDate('start') ?? Carbon::now()->subYear(),
             'end'   => $this->getCarbonDate('end') ?? Carbon::now(),
-            'type'  => $this->string('type'),
+            'type'  => $this->convertString('type'),
         ];
-        $parts      = explode(',', $this->string('accounts'));
+        $parts      = explode(',', $this->convertString('accounts'));
         $repository = app(AccountRepositoryInterface::class);
         $repository->setUser(auth()->user());
 

app/Api/V1/Requests/Models/Account/StoreRequest.php

@@ -56,35 +56,35 @@ class StoreRequest extends FormRequest
             $includeNetWorth = $this->boolean('include_net_worth');
         }
         $data = [
-            'name'                    => $this->string('name'),
+            'name'                    => $this->convertString('name'),
             'active'                  => $active,
             'include_net_worth'       => $includeNetWorth,
-            'account_type_name'       => $this->string('type'),
+            'account_type_name'       => $this->convertString('type'),
             'account_type_id'         => null,
             'currency_id'             => $this->integer('currency_id'),
             'order'                   => $this->integer('order'),
-            'currency_code'           => $this->string('currency_code'),
-            'virtual_balance'         => $this->string('virtual_balance'),
-            'iban'                    => $this->string('iban'),
-            'BIC'                     => $this->string('bic'),
-            'account_number'          => $this->string('account_number'),
-            'account_role'            => $this->string('account_role'),
-            'opening_balance'         => $this->string('opening_balance'),
+            'currency_code'           => $this->convertString('currency_code'),
+            'virtual_balance'         => $this->convertString('virtual_balance'),
+            'iban'                    => $this->convertString('iban'),
+            'BIC'                     => $this->convertString('bic'),
+            'account_number'          => $this->convertString('account_number'),
+            'account_role'            => $this->convertString('account_role'),
+            'opening_balance'         => $this->convertString('opening_balance'),
             'opening_balance_date'    => $this->getCarbonDate('opening_balance_date'),
-            'cc_type'                 => $this->string('credit_card_type'),
-            'cc_monthly_payment_date' => $this->string('monthly_payment_date'),
+            'cc_type'                 => $this->convertString('credit_card_type'),
+            'cc_monthly_payment_date' => $this->convertString('monthly_payment_date'),
             'notes'                   => $this->stringWithNewlines('notes'),
-            'interest'                => $this->string('interest'),
-            'interest_period'         => $this->string('interest_period'),
+            'interest'                => $this->convertString('interest'),
+            'interest_period'         => $this->convertString('interest_period'),
         ];
         // append location information.
         $data = $this->appendLocationData($data, null);
 
         if ('liability' === $data['account_type_name'] || 'liabilities' === $data['account_type_name']) {
-            $data['opening_balance']      = app('steam')->negative($this->string('liability_amount'));
+            $data['opening_balance']      = app('steam')->negative($this->convertString('liability_amount'));
             $data['opening_balance_date'] = $this->getCarbonDate('liability_start_date');
-            $data['account_type_name']    = $this->string('liability_type');
-            $data['liability_direction']  = $this->string('liability_direction');
+            $data['account_type_name']    = $this->convertString('liability_type');
+            $data['liability_direction']  = $this->convertString('liability_direction');
             $data['account_type_id']      = null;
         }
 
@@ -101,7 +101,7 @@ class StoreRequest extends FormRequest
         $accountRoles   = implode(',', config('firefly.accountRoles'));
         $types          = implode(',', array_keys(config('firefly.subTitlesByIdentifier')));
         $ccPaymentTypes = implode(',', array_keys(config('firefly.ccTypes')));
-        $type           = $this->string('type');
+        $type           = $this->convertString('type');
         $rules          = [
             'name'                 => 'required|min:1|uniqueAccountForUser',
             'type'                 => 'required|' . sprintf('in:%s', $types),

app/Api/V1/Requests/Models/Account/UpdateRequest.php

@@ -107,9 +107,9 @@ class UpdateRequest extends FormRequest
         $rules = [
             'name'                 => sprintf('min:1|uniqueAccountForUser:%d', $account->id),
             'type'                 => sprintf('in:%s', $types),
-            'iban'                 => ['iban', 'nullable', new UniqueIban($account, $this->string('type'))],
+            'iban'                 => ['iban', 'nullable', new UniqueIban($account, $this->convertString('type'))],
             'bic'                  => 'bic|nullable',
-            'account_number'       => ['between:1,255', 'nullable', new UniqueAccountNumber($account, $this->string('type'))],
+            'account_number'       => ['between:1,255', 'nullable', new UniqueAccountNumber($account, $this->convertString('type'))],
             'opening_balance'      => 'numeric|required_with:opening_balance_date|nullable',
             'opening_balance_date' => 'date|required_with:opening_balance|nullable',
             'virtual_balance'      => 'numeric|nullable',

app/Api/V1/Requests/Models/Attachment/StoreRequest.php

@@ -45,10 +45,10 @@ class StoreRequest extends FormRequest
     public function getAll(): array
     {
         return [
-            'filename'        => $this->string('filename'),
-            'title'           => $this->string('title'),
+            'filename'        => $this->convertString('filename'),
+            'title'           => $this->convertString('title'),
             'notes'           => $this->stringWithNewlines('notes'),
-            'attachable_type' => $this->string('attachable_type'),
+            'attachable_type' => $this->convertString('attachable_type'),
             'attachable_id'   => $this->integer('attachable_id'),
         ];
     }
@@ -68,7 +68,7 @@ class StoreRequest extends FormRequest
             }, $models
         );
         $models = implode(',', $models);
-        $model  = $this->string('attachable_type');
+        $model  = $this->convertString('attachable_type');
 
         return [
             'filename'        => 'required|between:1,255',

app/Api/V1/Requests/Models/Attachment/UpdateRequest.php

@@ -70,7 +70,7 @@ class UpdateRequest extends FormRequest
             }, $models
         );
         $models = implode(',', $models);
-        $model  = $this->string('attachable_type');
+        $model  = $this->convertString('attachable_type');
 
         return [
             'filename'        => 'between:1,255',

app/Api/V1/Requests/Models/BudgetLimit/StoreRequest.php

@@ -46,9 +46,9 @@ class StoreRequest extends FormRequest
         return [
             'start'         => $this->getCarbonDate('start'),
             'end'           => $this->getCarbonDate('end'),
-            'amount'        => $this->string('amount'),
+            'amount'        => $this->convertString('amount'),
             'currency_id'   => $this->integer('currency_id'),
-            'currency_code' => $this->string('currency_code'),
+            'currency_code' => $this->convertString('currency_code'),
         ];
     }
 

app/Api/V1/Requests/Models/Category/StoreRequest.php

@@ -45,7 +45,7 @@ class StoreRequest extends FormRequest
     public function getAll(): array
     {
         return [
-            'name'  => $this->string('name'),
+            'name'  => $this->convertString('name'),
             'notes' => $this->stringWithNewlines('notes'),
         ];
     }

app/Api/V1/Requests/Models/PiggyBank/StoreRequest.php

@@ -47,15 +47,15 @@ class StoreRequest extends FormRequest
             'order' => ['order', 'integer'],
         ];
         $data                       = $this->getAllData($fields);
-        $data['name']               = $this->string('name');
+        $data['name']               = $this->convertString('name');
         $data['account_id']         = $this->integer('account_id');
-        $data['targetamount']       = $this->string('target_amount');
-        $data['current_amount']     = $this->string('current_amount');
+        $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_title'] = $this->string('object_group_title');
+        $data['object_group_title'] = $this->convertString('object_group_title');
 
         return $data;
 

app/Api/V1/Requests/Models/RuleGroup/StoreRequest.php

@@ -53,8 +53,8 @@ class StoreRequest extends FormRequest
         }
 
         return [
-            'title'       => $this->string('title'),
-            'description' => $this->string('description'),
+            'title'       => $this->convertString('title'),
+            'description' => $this->convertString('description'),
             'active'      => $active,
             'order'       => $order,
         ];

app/Api/V1/Requests/Models/Tag/StoreRequest.php

@@ -46,9 +46,9 @@ class StoreRequest extends FormRequest
     public function getAll(): array
     {
         $data = [
-            'tag'          => $this->string('tag'),
+            'tag'          => $this->convertString('tag'),
             'date'         => $this->getCarbonDate('date'),
-            'description'  => $this->string('description'),
+            'description'  => $this->convertString('description'),
             'has_location' => true,
         ];
 

app/Api/V1/Requests/Models/Transaction/StoreRequest.php

@@ -55,7 +55,7 @@ class StoreRequest extends FormRequest
         Log::debug('get all data in TransactionStoreRequest');
 
         return [
-            'group_title'             => $this->string('group_title'),
+            'group_title'             => $this->convertString('group_title'),
             'error_if_duplicate_hash' => $this->boolean('error_if_duplicate_hash'),
             'apply_rules'             => $this->boolean('apply_rules', true),
             'fire_webhooks'           => $this->boolean('fire_webhooks', true),

app/Api/V1/Requests/Models/Transaction/UpdateRequest.php

@@ -84,7 +84,7 @@ class UpdateRequest extends FormRequest
             'notes',
         ];
 
-        $this->stringFields  = [
+        $this->convertStringFields  = [
             'type',
             'currency_code',
             'foreign_currency_code',
@@ -133,7 +133,7 @@ class UpdateRequest extends FormRequest
             $data['fire_webhooks'] = $this->boolean('fire_webhooks', true);
         }
         if ($this->has('group_title')) {
-            $data['group_title'] = $this->string('group_title');
+            $data['group_title'] = $this->convertString('group_title');
         }
 
         return $data;
@@ -196,7 +196,7 @@ class UpdateRequest extends FormRequest
      */
     private function getStringData(array $current, array $transaction): array
     {
-        foreach ($this->stringFields as $fieldName) {
+        foreach ($this->convertStringFields as $fieldName) {
             if (array_key_exists($fieldName, $transaction)) {
                 $current[$fieldName] = $this->clearString((string) $transaction[$fieldName], false);
             }

app/Api/V1/Requests/Models/TransactionCurrency/StoreRequest.php

@@ -54,9 +54,9 @@ class StoreRequest extends FormRequest
         }
 
         return [
-            'name'           => $this->string('name'),
-            'code'           => $this->string('code'),
-            'symbol'         => $this->string('symbol'),
+            'name'           => $this->convertString('name'),
+            'code'           => $this->convertString('code'),
+            'symbol'         => $this->convertString('symbol'),
             'decimal_places' => $this->integer('decimal_places'),
             'default'        => $default,
             'enabled'        => $enabled,

app/Api/V1/Requests/Models/TransactionLink/StoreRequest.php

@@ -47,7 +47,7 @@ class StoreRequest extends FormRequest
     {
         return [
             'link_type_id'   => $this->integer('link_type_id'),
-            'link_type_name' => $this->string('link_type_name'),
+            'link_type_name' => $this->convertString('link_type_name'),
             'inward_id'      => $this->integer('inward_id'),
             'outward_id'     => $this->integer('outward_id'),
             'notes'          => $this->stringWithNewlines('notes'),

app/Api/V1/Requests/Models/TransactionLink/UpdateRequest.php

@@ -47,7 +47,7 @@ class UpdateRequest extends FormRequest
     {
         return [
             'link_type_id'   => $this->integer('link_type_id'),
-            'link_type_name' => $this->string('link_type_name'),
+            'link_type_name' => $this->convertString('link_type_name'),
             'inward_id'      => $this->integer('inward_id'),
             'outward_id'     => $this->integer('outward_id'),
             'notes'          => $this->stringWithNewlines('notes'),

app/Api/V1/Requests/Models/TransactionLinkType/StoreRequest.php

@@ -44,9 +44,9 @@ class StoreRequest extends FormRequest
     public function getAll(): array
     {
         return [
-            'name'    => $this->string('name'),
-            'outward' => $this->string('outward'),
-            'inward'  => $this->string('inward'),
+            'name'    => $this->convertString('name'),
+            'outward' => $this->convertString('outward'),
+            'inward'  => $this->convertString('inward'),
         ];
     }
 

app/Api/V1/Requests/Models/TransactionLinkType/UpdateRequest.php

@@ -45,9 +45,9 @@ class UpdateRequest extends FormRequest
     public function getAll(): array
     {
         return [
-            'name'    => $this->string('name'),
-            'outward' => $this->string('outward'),
-            'inward'  => $this->string('inward'),
+            'name'    => $this->convertString('name'),
+            'outward' => $this->convertString('outward'),
+            'inward'  => $this->convertString('inward'),
         ];
     }
 

app/Api/V1/Requests/Models/Webhook/UpdateRequest.php

@@ -66,7 +66,7 @@ class UpdateRequest extends FormRequest
         }
         $return['secret'] = null !== $this->get('secret');
         if (null !== $this->get('title')) {
-            $return['title'] = $this->string('title');
+            $return['title'] = $this->convertString('title');
         }
 
         return $return;

app/Api/V1/Requests/System/UpdateRequest.php

@@ -57,7 +57,7 @@ class UpdateRequest extends FormRequest
                 return ['value' => $this->integer('value')];
         }
 
-        return ['value' => $this->string('value')];
+        return ['value' => $this->convertString('value')];
     }
 
     /**

app/Api/V1/Requests/System/UserStoreRequest.php

@@ -59,10 +59,10 @@ class UserStoreRequest extends FormRequest
         }
 
         return [
-            'email'        => $this->string('email'),
+            'email'        => $this->convertString('email'),
             'blocked'      => $blocked,
-            'blocked_code' => $this->string('blocked_code'),
-            'role'         => $this->string('role'),
+            'blocked_code' => $this->convertString('blocked_code'),
+            'role'         => $this->convertString('role'),
         ];
     }
 

app/Api/V1/Requests/System/UserUpdateRequest.php

@@ -60,10 +60,10 @@ class UserUpdateRequest extends FormRequest
         }
 
         return [
-            'email'        => $this->string('email'),
+            'email'        => $this->convertString('email'),
             'blocked'      => $blocked,
-            'blocked_code' => $this->string('blocked_code'),
-            'role'         => $this->string('role'),
+            'blocked_code' => $this->convertString('blocked_code'),
+            'role'         => $this->convertString('role'),
         ];
     }
 

app/Api/V1/Requests/User/PreferenceStoreRequest.php

@@ -37,7 +37,7 @@ class PreferenceStoreRequest extends FormRequest
     public function getAll(): array
     {
         $array = [
-            'name' => $this->string('name'),
+            'name' => $this->convertString('name'),
             'data' => $this->get('data'),
         ];
         if ('true' === $array['data']) {

app/Api/V1/Requests/User/PreferenceUpdateRequest.php

@@ -38,7 +38,7 @@ class PreferenceUpdateRequest extends FormRequest
     public function getAll(): array
     {
         $array = [
-            'name' => $this->string('name'),
+            'name' => $this->convertString('name'),
             'data' => $this->get('data'),
         ];
         if ('true' === $array['data']) {

app/Handlers/Events/UserEventHandler.php

@@ -230,9 +230,9 @@ class UserEventHandler
         $oldEmail = $event->oldEmail;
         $user     = $event->user;
         $token    = app('preferences')->getForUser($user, 'email_change_confirm_token', 'invalid');
-        $uri      = route('profile.confirm-email-change', [$token->data]);
+        $url      = route('profile.confirm-email-change', [$token->data]);
         try {
-            Mail::to($newEmail)->send(new ConfirmEmailChangeMail($newEmail, $oldEmail, $uri));
+            Mail::to($newEmail)->send(new ConfirmEmailChangeMail($newEmail, $oldEmail, $url));
 
         } catch (Exception $e) { // @phpstan-ignore-line
             Log::error($e->getMessage());
@@ -256,9 +256,9 @@ class UserEventHandler
         $user     = $event->user;
         $token    = app('preferences')->getForUser($user, 'email_change_undo_token', 'invalid');
         $hashed   = hash('sha256', sprintf('%s%s', (string) config('app.key'), $oldEmail));
-        $uri      = route('profile.undo-email-change', [$token->data, $hashed]);
+        $url      = route('profile.undo-email-change', [$token->data, $hashed]);
         try {
-            Mail::to($oldEmail)->send(new UndoEmailChangeMail($newEmail, $oldEmail, $uri));
+            Mail::to($oldEmail)->send(new UndoEmailChangeMail($newEmail, $oldEmail, $url));
 
         } catch (Exception $e) { // @phpstan-ignore-line
             Log::error($e->getMessage());
@@ -308,7 +308,7 @@ class UserEventHandler
         if ($sendMail) {
             // get the email address
             $email     = $event->user->email;
-            $uri       = route('index');
+            $url       = route('index');
 
             // see if user has alternative email address:
             $pref = app('preferences')->getForUser($event->user, 'remote_guard_alt_email');
@@ -318,7 +318,7 @@ class UserEventHandler
 
             // send email.
             try {
-                Mail::to($email)->send(new RegisteredUserMail($uri));
+                Mail::to($email)->send(new RegisteredUserMail($url));
 
             } catch (Exception $e) { // @phpstan-ignore-line
                 Log::error($e->getMessage());

app/Helpers/Collector/Extensions/TimeCollection.php

@@ -555,7 +555,9 @@ trait TimeCollection
      */
     public function setObjectBefore(Carbon $date, string $field): GroupCollectorInterface
     {
-        die('a');
+        $afterStr = $date->format('Y-m-d 00:00:00');
+        $this->query->where(sprintf('transaction_journals.%s', $field), '<=', $afterStr);
+        return $this;
     }
 
     /**

app/Http/Controllers/Account/CreateController.php

@@ -119,7 +119,7 @@ class CreateController extends Controller
 
         // put previous url in session if not redirect from store (not "create another").
         if (true !== session('accounts.create.fromStore')) {
-            $this->rememberPreviousUri('accounts.create.uri');
+            $this->rememberPreviousUrl('accounts.create.url');
         }
         $request->session()->forget('accounts.create.fromStore');
         Log::channel('audit')->info('Creating new account.');
@@ -171,7 +171,7 @@ class CreateController extends Controller
         }
 
         // redirect to previous URL.
-        $redirect = redirect($this->getPreviousUri('accounts.create.uri'));
+        $redirect = redirect($this->getPreviousUrl('accounts.create.url'));
         if (1 === (int) $request->get('create_another')) {
             // set value so create routine will not overwrite URL:
             $request->session()->put('accounts.create.fromStore', true);

app/Http/Controllers/Account/DeleteController.php

@@ -82,7 +82,7 @@ class DeleteController extends Controller
         unset($accountList[$account->id]);
 
         // put previous url in session
-        $this->rememberPreviousUri('accounts.delete.uri');
+        $this->rememberPreviousUrl('accounts.delete.url');
 
         return view('accounts.delete', compact('account', 'subTitle', 'accountList', 'objectType'));
     }
@@ -111,7 +111,7 @@ class DeleteController extends Controller
         $request->session()->flash('success', (string) trans(sprintf('firefly.%s_deleted', $typeName), ['name' => $name]));
         app('preferences')->mark();
 
-        return redirect($this->getPreviousUri('accounts.delete.uri'));
+        return redirect($this->getPreviousUrl('accounts.delete.url'));
     }
 
 }

app/Http/Controllers/Account/EditController.php

@@ -118,7 +118,7 @@ class EditController extends Controller
 
         // put previous url in session if not redirect from store (not "return_to_edit").
         if (true !== session('accounts.edit.fromUpdate')) {
-            $this->rememberPreviousUri('accounts.edit.uri');
+            $this->rememberPreviousUrl('accounts.edit.url');
         }
         $request->session()->forget('accounts.edit.fromUpdate');
 
@@ -211,7 +211,7 @@ class EditController extends Controller
         }
 
         // redirect
-        $redirect = redirect($this->getPreviousUri('accounts.edit.uri'));
+        $redirect = redirect($this->getPreviousUrl('accounts.edit.url'));
         if (1 === (int) $request->get('return_to_edit')) {
             // set value so edit routine will not overwrite URL:
             $request->session()->put('accounts.edit.fromUpdate', true);

app/Http/Controllers/Account/ReconcileController.php

@@ -134,9 +134,9 @@ class ReconcileController extends Controller
         $subTitle     = (string) trans('firefly.reconcile_account', ['account' => $account->name]);
 
         // various links
-        $transactionsUri = route('accounts.reconcile.transactions', [$account->id, '%start%', '%end%']);
-        $overviewUri     = route('accounts.reconcile.overview', [$account->id, '%start%', '%end%']);
-        $indexUri        = route('accounts.reconcile', [$account->id, '%start%', '%end%']);
+        $transactionsUrl = route('accounts.reconcile.transactions', [$account->id, '%start%', '%end%']);
+        $overviewUrl     = route('accounts.reconcile.overview', [$account->id, '%start%', '%end%']);
+        $indexUrl        = route('accounts.reconcile', [$account->id, '%start%', '%end%']);
         $objectType      = 'asset';
 
         return view(
@@ -151,9 +151,9 @@ class ReconcileController extends Controller
                 'subTitle',
                 'startBalance',
                 'endBalance',
-                'transactionsUri',
-                'overviewUri',
-                'indexUri'
+                'transactionsUrl',
+                'overviewUrl',
+                'indexUrl'
             )
         );
     }

app/Http/Controllers/Account/ShowController.php

@@ -114,7 +114,7 @@ class ShowController extends Controller
         $fStart           = $start->isoFormat($this->monthAndDayFormat);
         $fEnd             = $end->isoFormat($this->monthAndDayFormat);
         $subTitle         = (string) trans('firefly.journals_in_period_for_account', ['name' => $account->name, 'start' => $fStart, 'end' => $fEnd]);
-        $chartUri         = route('chart.account.period', [$account->id, $start->format('Y-m-d'), $end->format('Y-m-d')]);
+        $chartUrl         = route('chart.account.period', [$account->id, $start->format('Y-m-d'), $end->format('Y-m-d')]);
         $firstTransaction = $this->repository->oldestJournalDate($account) ?? $start;
         $periods          = $this->getAccountPeriodOverview($account, $firstTransaction, $end);
 
@@ -152,7 +152,7 @@ class ShowController extends Controller
                 'subTitle',
                 'start',
                 'end',
-                'chartUri',
+                'chartUrl',
                 'location',
                 'balance'
             )
@@ -194,7 +194,7 @@ class ShowController extends Controller
         $collector->setAccounts(new Collection([$account]))->setLimit($pageSize)->setPage($page)->withAccountInformation()->withCategoryInformation();
         $groups = $collector->getPaginatedGroups();
         $groups->setPath(route('accounts.show.all', [$account->id]));
-        $chartUri = route('chart.account.period', [$account->id, $start->format('Y-m-d'), $end->format('Y-m-d')]);
+        $chartUrl = route('chart.account.period', [$account->id, $start->format('Y-m-d'), $end->format('Y-m-d')]);
         $showAll  = true;
         $balance  = app('steam')->balance($account, $end);
 
@@ -209,7 +209,7 @@ class ShowController extends Controller
                 'attachments',
                 'currency',
                 'today',
-                'chartUri',
+                'chartUrl',
                 'periods',
                 'subTitleIcon',
                 'groups',

app/Http/Controllers/Admin/LinkController.php

@@ -76,7 +76,7 @@ class LinkController extends Controller
 
         // put previous url in session if not redirect from store (not "create another").
         if (true !== session('link-types.create.fromStore')) {
-            $this->rememberPreviousUri('link-types.create.uri');
+            $this->rememberPreviousUrl('link-types.create.url');
         }
 
         return view('admin.link.create', compact('subTitle', 'subTitleIcon'));
@@ -113,7 +113,7 @@ class LinkController extends Controller
         }
 
         // put previous url in session
-        $this->rememberPreviousUri('link-types.delete.uri');
+        $this->rememberPreviousUrl('link-types.delete.url');
 
         return view('admin.link.delete', compact('linkType', 'subTitle', 'moveTo', 'count'));
     }
@@ -136,7 +136,7 @@ class LinkController extends Controller
         $request->session()->flash('success', (string) trans('firefly.deleted_link_type', ['name' => $name]));
         app('preferences')->mark();
 
-        return redirect($this->getPreviousUri('link-types.delete.uri'));
+        return redirect($this->getPreviousUrl('link-types.delete.url'));
     }
 
     /**
@@ -161,7 +161,7 @@ class LinkController extends Controller
 
         // put previous url in session if not redirect from store (not "return_to_edit").
         if (true !== session('link-types.edit.fromUpdate')) {
-            $this->rememberPreviousUri('link-types.edit.uri');
+            $this->rememberPreviousUrl('link-types.edit.url');
         }
         $request->session()->forget('link-types.edit.fromUpdate');
 
@@ -217,16 +217,16 @@ class LinkController extends Controller
     public function store(LinkTypeFormRequest $request)
     {
         $data     = [
-            'name'    => $request->string('name'),
-            'inward'  => $request->string('inward'),
-            'outward' => $request->string('outward'),
+            'name'    => $request->convertString('name'),
+            'inward'  => $request->convertString('inward'),
+            'outward' => $request->convertString('outward'),
         ];
         $linkType = $this->repository->store($data);
 
         Log::channel('audit')->info('User stored new link type.', $linkType->toArray());
 
         $request->session()->flash('success', (string) trans('firefly.stored_new_link_type', ['name' => $linkType->name]));
-        $redirect = redirect($this->getPreviousUri('link-types.create.uri'));
+        $redirect = redirect($this->getPreviousUrl('link-types.create.url'));
         if (1 === (int) $request->get('create_another')) {
             // set value so create routine will not overwrite URL:
             $request->session()->put('link-types.create.fromStore', true);
@@ -255,9 +255,9 @@ class LinkController extends Controller
         }
 
         $data = [
-            'name'    => $request->string('name'),
-            'inward'  => $request->string('inward'),
-            'outward' => $request->string('outward'),
+            'name'    => $request->convertString('name'),
+            'inward'  => $request->convertString('inward'),
+            'outward' => $request->convertString('outward'),
         ];
         $this->repository->update($linkType, $data);
 
@@ -265,7 +265,7 @@ class LinkController extends Controller
 
         $request->session()->flash('success', (string) trans('firefly.updated_link_type', ['name' => $linkType->name]));
         app('preferences')->mark();
-        $redirect = redirect($this->getPreviousUri('link-types.edit.uri'));
+        $redirect = redirect($this->getPreviousUrl('link-types.edit.url'));
         if (1 === (int) $request->get('return_to_edit')) {
             // set value so edit routine will not overwrite URL:
             $request->session()->put('link-types.edit.fromUpdate', true);

app/Http/Controllers/Admin/UserController.php

@@ -117,7 +117,7 @@ class UserController extends Controller
         }
         // put previous url in session if not redirect from store (not "return_to_edit").
         if (true !== session('users.edit.fromUpdate')) {
-            $this->rememberPreviousUri('users.edit.uri');
+            $this->rememberPreviousUrl('users.edit.url');
         }
         session()->forget('users.edit.fromUpdate');
 
@@ -217,7 +217,7 @@ class UserController extends Controller
 
         session()->flash('success', (string) trans('firefly.updated_user', ['email' => $user->email]));
         app('preferences')->mark();
-        $redirect = redirect($this->getPreviousUri('users.edit.uri'));
+        $redirect = redirect($this->getPreviousUrl('users.edit.url'));
         if (1 === (int) $request->get('return_to_edit')) {
 
             session()->put('users.edit.fromUpdate', true);

app/Http/Controllers/AttachmentController.php

@@ -77,7 +77,7 @@ class AttachmentController extends Controller
         $subTitle = (string) trans('firefly.delete_attachment', ['name' => $attachment->filename]);
 
         // put previous url in session
-        $this->rememberPreviousUri('attachments.delete.uri');
+        $this->rememberPreviousUrl('attachments.delete.url');
 
         return view('attachments.delete', compact('attachment', 'subTitle'));
     }
@@ -99,7 +99,7 @@ class AttachmentController extends Controller
         $request->session()->flash('success', (string) trans('firefly.attachment_deleted', ['name' => $name]));
         app('preferences')->mark();
 
-        return redirect($this->getPreviousUri('attachments.delete.uri'));
+        return redirect($this->getPreviousUrl('attachments.delete.url'));
     }
 
     /**
@@ -150,7 +150,7 @@ class AttachmentController extends Controller
 
         // put previous url in session if not redirect from store (not "return_to_edit").
         if (true !== session('attachments.edit.fromUpdate')) {
-            $this->rememberPreviousUri('attachments.edit.uri');
+            $this->rememberPreviousUrl('attachments.edit.url');
         }
         $request->session()->forget('attachments.edit.fromUpdate');
         $preFilled = [
@@ -196,7 +196,7 @@ class AttachmentController extends Controller
         $request->session()->flash('success', (string) trans('firefly.attachment_updated', ['name' => $attachment->filename]));
         app('preferences')->mark();
 
-        $redirect = redirect($this->getPreviousUri('attachments.edit.uri'));
+        $redirect = redirect($this->getPreviousUrl('attachments.edit.url'));
         if (1 === (int) $request->get('return_to_edit')) {
 
             $request->session()->put('attachments.edit.fromUpdate', true);

app/Http/Controllers/Auth/LoginController.php

@@ -169,11 +169,11 @@ class LoginController extends Controller
     public function logout(Request $request)
     {
         $authGuard = config('firefly.authentication_guard');
-        $logoutUri = config('firefly.custom_logout_url');
-        if ('remote_user_guard' === $authGuard && '' !== $logoutUri) {
-            return redirect($logoutUri);
+        $logoutUrl = config('firefly.custom_logout_url');
+        if ('remote_user_guard' === $authGuard && '' !== $logoutUrl) {
+            return redirect($logoutUrl);
         }
-        if ('remote_user_guard' === $authGuard && '' === $logoutUri) {
+        if ('remote_user_guard' === $authGuard && '' === $logoutUrl) {
             session()->flash('error', trans('firefly.cant_logout_guard'));
         }
 

app/Http/Controllers/Bill/CreateController.php

@@ -84,7 +84,7 @@ class CreateController extends Controller
 
         // put previous url in session if not redirect from store (not "create another").
         if (true !== session('bills.create.fromStore')) {
-            $this->rememberPreviousUri('bills.create.uri');
+            $this->rememberPreviousUrl('bills.create.url');
         }
         $request->session()->forget('bills.create.fromStore');
 

app/Http/Controllers/Bill/DeleteController.php

@@ -72,7 +72,7 @@ class DeleteController extends Controller
     public function delete(Bill $bill)
     {
         // put previous url in session
-        $this->rememberPreviousUri('bills.delete.uri');
+        $this->rememberPreviousUrl('bills.delete.url');
         $subTitle = (string) trans('firefly.delete_bill', ['name' => $bill->name]);
 
         return view('bills.delete', compact('bill', 'subTitle'));
@@ -94,6 +94,6 @@ class DeleteController extends Controller
         $request->session()->flash('success', (string) trans('firefly.deleted_bill', ['name' => $name]));
         app('preferences')->mark();
 
-        return redirect($this->getPreviousUri('bills.delete.uri'));
+        return redirect($this->getPreviousUrl('bills.delete.url'));
     }
 }

app/Http/Controllers/Bill/EditController.php

@@ -85,7 +85,7 @@ class EditController extends Controller
 
         // put previous url in session if not redirect from store (not "return_to_edit").
         if (true !== session('bills.edit.fromUpdate')) {
-            $this->rememberPreviousUri('bills.edit.uri');
+            $this->rememberPreviousUrl('bills.edit.url');
         }
 
         $currency         = app('amount')->getDefaultCurrency();
@@ -141,7 +141,7 @@ class EditController extends Controller
         if (count($this->attachments->getMessages()->get('attachments')) > 0) {
             $request->session()->flash('info', $this->attachments->getMessages()->get('attachments'));
         }
-        $redirect = redirect($this->getPreviousUri('bills.edit.uri'));
+        $redirect = redirect($this->getPreviousUrl('bills.edit.url'));
 
         if (1 === (int) $request->get('return_to_edit')) {
 

app/Http/Controllers/Budget/CreateController.php

@@ -98,7 +98,7 @@ class CreateController extends Controller
 
         // put previous url in session if not redirect from store (not "create another").
         if (true !== session('budgets.create.fromStore')) {
-            $this->rememberPreviousUri('budgets.create.uri');
+            $this->rememberPreviousUrl('budgets.create.url');
         }
         $request->session()->forget('budgets.create.fromStore');
         $subTitle = (string) trans('firefly.create_new_budget');
@@ -136,7 +136,7 @@ class CreateController extends Controller
             $request->session()->flash('info', $this->attachments->getMessages()->get('attachments'));
         }
 
-        $redirect = redirect($this->getPreviousUri('budgets.create.uri'));
+        $redirect = redirect($this->getPreviousUrl('budgets.create.url'));
 
         if (1 === (int) $request->get('create_another')) {
 

app/Http/Controllers/Budget/DeleteController.php

@@ -73,7 +73,7 @@ class DeleteController extends Controller
         $subTitle = (string) trans('firefly.delete_budget', ['name' => $budget->name]);
 
         // put previous url in session
-        $this->rememberPreviousUri('budgets.delete.uri');
+        $this->rememberPreviousUrl('budgets.delete.url');
 
         return view('budgets.delete', compact('budget', 'subTitle'));
     }
@@ -93,7 +93,7 @@ class DeleteController extends Controller
         $request->session()->flash('success', (string) trans('firefly.deleted_budget', ['name' => $name]));
         app('preferences')->mark();
 
-        return redirect($this->getPreviousUri('budgets.delete.uri'));
+        return redirect($this->getPreviousUrl('budgets.delete.url'));
     }
 
 }

app/Http/Controllers/Budget/EditController.php

@@ -108,7 +108,7 @@ class EditController extends Controller
 
         // put previous url in session if not redirect from store (not "return_to_edit").
         if (true !== session('budgets.edit.fromUpdate')) {
-            $this->rememberPreviousUri('budgets.edit.uri');
+            $this->rememberPreviousUrl('budgets.edit.url');
         }
         $request->session()->forget('budgets.edit.fromUpdate');
         $request->session()->flash('preFilled', $preFilled);
@@ -133,7 +133,7 @@ class EditController extends Controller
         $this->repository->cleanupBudgets();
         app('preferences')->mark();
 
-        $redirect = redirect($this->getPreviousUri('budgets.edit.uri'));
+        $redirect = redirect($this->getPreviousUrl('budgets.edit.url'));
 
         // store new attachment(s):
         $files = $request->hasFile('attachments') ? $request->file('attachments') : null;

app/Http/Controllers/Category/CreateController.php

@@ -73,7 +73,7 @@ class CreateController extends Controller
     public function create(Request $request)
     {
         if (true !== session('categories.create.fromStore')) {
-            $this->rememberPreviousUri('categories.create.uri');
+            $this->rememberPreviousUrl('categories.create.url');
         }
         $request->session()->forget('categories.create.fromStore');
         $subTitle = (string) trans('firefly.create_new_category');

app/Http/Controllers/Category/DeleteController.php

@@ -72,7 +72,7 @@ class DeleteController extends Controller
         $subTitle = (string) trans('firefly.delete_category', ['name' => $category->name]);
 
         // put previous url in session
-        $this->rememberPreviousUri('categories.delete.uri');
+        $this->rememberPreviousUrl('categories.delete.url');
 
         return view('categories.delete', compact('category', 'subTitle'));
     }
@@ -93,6 +93,6 @@ class DeleteController extends Controller
         $request->session()->flash('success', (string) trans('firefly.deleted_category', ['name' => $name]));
         app('preferences')->mark();
 
-        return redirect($this->getPreviousUri('categories.delete.uri'));
+        return redirect($this->getPreviousUrl('categories.delete.url'));
     }
 }

app/Http/Controllers/Category/EditController.php

@@ -78,7 +78,7 @@ class EditController extends Controller
 
         // put previous url in session if not redirect from store (not "return_to_edit").
         if (true !== session('categories.edit.fromUpdate')) {
-            $this->rememberPreviousUri('categories.edit.uri');
+            $this->rememberPreviousUrl('categories.edit.url');
         }
         $request->session()->forget('categories.edit.fromUpdate');
 
@@ -117,7 +117,7 @@ class EditController extends Controller
         if (count($this->attachments->getMessages()->get('attachments')) > 0) {
             $request->session()->flash('info', $this->attachments->getMessages()->get('attachments'));
         }
-        $redirect = redirect($this->getPreviousUri('categories.edit.uri'));
+        $redirect = redirect($this->getPreviousUrl('categories.edit.url'));
 
         if (1 === (int) $request->get('return_to_edit')) {
 

app/Http/Controllers/Controller.php

@@ -41,7 +41,7 @@ abstract class Controller extends BaseController
     protected string $dateTimeFormat;
     protected string $monthAndDayFormat;
     protected string $monthFormat;
-    protected string $redirectUri = '/';
+    protected string $redirectUrl = '/';
 
     /**
      * Controller constructor.
@@ -63,10 +63,10 @@ abstract class Controller extends BaseController
 
         // share custom auth guard info.
         $authGuard = config('firefly.authentication_guard');
-        $logoutUri = config('firefly.custom_logout_url');
+        $logoutUrl = config('firefly.custom_logout_url');
 
         app('view')->share('authGuard', $authGuard);
-        app('view')->share('logoutUri', $logoutUri);
+        app('view')->share('logoutUrl', $logoutUrl);
 
         // upload size
         $maxFileSize = app('steam')->phpBytes(ini_get('upload_max_filesize'));

app/Http/Controllers/CurrencyController.php

@@ -88,7 +88,7 @@ class CurrencyController extends Controller
 
         // put previous url in session if not redirect from store (not "create another").
         if (true !== session('currencies.create.fromStore')) {
-            $this->rememberPreviousUri('currencies.create.uri');
+            $this->rememberPreviousUrl('currencies.create.url');
         }
         $request->session()->forget('currencies.create.fromStore');
 
@@ -157,7 +157,7 @@ class CurrencyController extends Controller
         }
 
         // put previous url in session
-        $this->rememberPreviousUri('currencies.delete.uri');
+        $this->rememberPreviousUrl('currencies.delete.url');
         $subTitle = (string) trans('form.delete_currency', ['name' => $currency->name]);
         Log::channel('audit')->info(sprintf('Visit page to delete currency %s.', $currency->code));
 
@@ -204,7 +204,7 @@ class CurrencyController extends Controller
 
         $request->session()->flash('success', (string) trans('firefly.deleted_currency', ['name' => $currency->name]));
 
-        return redirect($this->getPreviousUri('currencies.delete.uri'));
+        return redirect($this->getPreviousUrl('currencies.delete.url'));
     }
 
     /**
@@ -297,7 +297,7 @@ class CurrencyController extends Controller
 
         // put previous url in session if not redirect from store (not "return_to_edit").
         if (true !== session('currencies.edit.fromUpdate')) {
-            $this->rememberPreviousUri('currencies.edit.uri');
+            $this->rememberPreviousUrl('currencies.edit.url');
         }
         $request->session()->forget('currencies.edit.fromUpdate');
 
@@ -375,7 +375,7 @@ class CurrencyController extends Controller
             Log::error('User ' . auth()->user()->id . ' is not admin, but tried to store a currency.');
             Log::channel('audit')->info('Tried to create (POST) currency without admin rights.', $data);
 
-            return redirect($this->getPreviousUri('currencies.create.uri'));
+            return redirect($this->getPreviousUrl('currencies.create.url'));
 
         }
 
@@ -388,7 +388,7 @@ class CurrencyController extends Controller
             $request->session()->flash('error', (string) trans('firefly.could_not_store_currency'));
             $currency = null;
         }
-        $redirect = redirect($this->getPreviousUri('currencies.create.uri'));
+        $redirect = redirect($this->getPreviousUrl('currencies.create.url'));
 
         if (null !== $currency) {
             $request->session()->flash('success', (string) trans('firefly.created_currency', ['name' => $currency->name]));
@@ -443,6 +443,6 @@ class CurrencyController extends Controller
 
         }
 
-        return redirect($this->getPreviousUri('currencies.edit.uri'));
+        return redirect($this->getPreviousUrl('currencies.edit.url'));
     }
 }

app/Http/Controllers/Json/ReconcileController.php

@@ -168,7 +168,7 @@ class ReconcileController extends Controller
         }
 
         $return = [
-            'post_uri' => $route,
+            'post_url' => $route,
             'html'     => $view,
         ];
 

app/Http/Controllers/NewUserController.php

@@ -92,7 +92,7 @@ class NewUserController extends Controller
      */
     public function submit(NewUserFormRequest $request, CurrencyRepositoryInterface $currencyRepository)
     {
-        $language = $request->string('language');
+        $language = $request->convertString('language');
         if (!array_key_exists($language, config('firefly.languages'))) {
             $language = 'en_US';
 

app/Http/Controllers/ObjectGroup/DeleteController.php

@@ -72,7 +72,7 @@ class DeleteController extends Controller
         $piggyBanks = $objectGroup->piggyBanks()->count();
 
         // put previous url in session
-        $this->rememberPreviousUri('object-groups.delete.uri');
+        $this->rememberPreviousUrl('object-groups.delete.url');
 
         return view('object-groups.delete', compact('objectGroup', 'subTitle', 'piggyBanks'));
     }
@@ -90,7 +90,7 @@ class DeleteController extends Controller
         app('preferences')->mark();
         $this->repository->destroy($objectGroup);
 
-        return redirect($this->getPreviousUri('object-groups.delete.uri'));
+        return redirect($this->getPreviousUrl('object-groups.delete.url'));
     }
 
 }

app/Http/Controllers/ObjectGroup/EditController.php

@@ -75,7 +75,7 @@ class EditController extends Controller
         $subTitleIcon = 'fa-pencil';
 
         if (true !== session('object-groups.edit.fromUpdate')) {
-            $this->rememberPreviousUri('object-groups.edit.uri');
+            $this->rememberPreviousUrl('object-groups.edit.url');
         }
         session()->forget('object-groups.edit.fromUpdate');
 
@@ -98,7 +98,7 @@ class EditController extends Controller
         session()->flash('success', (string) trans('firefly.updated_object_group', ['title' => $objectGroup->title]));
         app('preferences')->mark();
 
-        $redirect = redirect($this->getPreviousUri('object-groups.edit.uri'));
+        $redirect = redirect($this->getPreviousUrl('object-groups.edit.url'));
 
         if (1 === (int) $request->get('return_to_edit')) {
 

app/Http/Controllers/PiggyBank/CreateController.php

@@ -76,7 +76,7 @@ class CreateController extends Controller
 
         // put previous url in session if not redirect from store (not "create another").
         if (true !== session('piggy-banks.create.fromStore')) {
-            $this->rememberPreviousUri('piggy-banks.create.uri');
+            $this->rememberPreviousUrl('piggy-banks.create.url');
         }
         session()->forget('piggy-banks.create.fromStore');
 
@@ -115,7 +115,7 @@ class CreateController extends Controller
         if (count($this->attachments->getMessages()->get('attachments')) > 0) {
             $request->session()->flash('info', $this->attachments->getMessages()->get('attachments'));
         }
-        $redirect = redirect($this->getPreviousUri('piggy-banks.create.uri'));
+        $redirect = redirect($this->getPreviousUrl('piggy-banks.create.url'));
 
         if (1 === (int) $request->get('create_another')) {
 

app/Http/Controllers/PiggyBank/DeleteController.php

@@ -72,7 +72,7 @@ class DeleteController extends Controller
         $subTitle = (string) trans('firefly.delete_piggy_bank', ['name' => $piggyBank->name]);
 
         // put previous url in session
-        $this->rememberPreviousUri('piggy-banks.delete.uri');
+        $this->rememberPreviousUrl('piggy-banks.delete.url');
 
         return view('piggy-banks.delete', compact('piggyBank', 'subTitle'));
     }
@@ -90,6 +90,6 @@ class DeleteController extends Controller
         app('preferences')->mark();
         $this->piggyRepos->destroy($piggyBank);
 
-        return redirect($this->getPreviousUri('piggy-banks.delete.uri'));
+        return redirect($this->getPreviousUrl('piggy-banks.delete.url'));
     }
 }

app/Http/Controllers/PiggyBank/EditController.php

@@ -102,7 +102,7 @@ class EditController extends Controller
 
         // put previous url in session if not redirect from store (not "return_to_edit").
         if (true !== session('piggy-banks.edit.fromUpdate')) {
-            $this->rememberPreviousUri('piggy-banks.edit.uri');
+            $this->rememberPreviousUrl('piggy-banks.edit.url');
         }
         session()->forget('piggy-banks.edit.fromUpdate');
 
@@ -138,7 +138,7 @@ class EditController extends Controller
         if (count($this->attachments->getMessages()->get('attachments')) > 0) {
             $request->session()->flash('info', $this->attachments->getMessages()->get('attachments'));
         }
-        $redirect = redirect($this->getPreviousUri('piggy-banks.edit.uri'));
+        $redirect = redirect($this->getPreviousUrl('piggy-banks.edit.url'));
 
         if (1 === (int) $request->get('return_to_edit')) {
 

app/Http/Controllers/ProfileController.php

@@ -410,7 +410,7 @@ class ProfileController extends Controller
 
         /** @var User $user */
         $user     = auth()->user();
-        $newEmail = $request->string('email');
+        $newEmail = $request->convertString('email');
         $oldEmail = $user->email;
         if ($newEmail === $user->email) {
             session()->flash('error', (string) trans('firefly.email_not_changed'));

app/Http/Controllers/Recurring/CreateController.php

@@ -94,7 +94,7 @@ class CreateController extends Controller
 
         // put previous url in session if not redirect from store (not "create another").
         if (true !== session('recurring.create.fromStore')) {
-            $this->rememberPreviousUri('recurring.create.uri');
+            $this->rememberPreviousUrl('recurring.create.url');
         }
         $request->session()->forget('recurring.create.fromStore');
         $repetitionEnds   = [
@@ -139,7 +139,7 @@ class CreateController extends Controller
 
         // put previous url in session if not redirect from store (not "create another").
         if (true !== session('recurring.create.fromStore')) {
-            $this->rememberPreviousUri('recurring.create.uri');
+            $this->rememberPreviousUrl('recurring.create.url');
         }
         $request->session()->forget('recurring.create.fromStore');
         $repetitionEnds   = [
@@ -253,7 +253,7 @@ class CreateController extends Controller
             $request->session()->flash('info', $this->attachments->getMessages()->get('attachments'));
         }
 
-        $redirect = redirect($this->getPreviousUri('recurring.create.uri'));
+        $redirect = redirect($this->getPreviousUrl('recurring.create.url'));
         if (1 === (int) $request->get('create_another')) {
             // set value so create routine will not overwrite URL:
             $request->session()->put('recurring.create.fromStore', true);

app/Http/Controllers/Recurring/DeleteController.php

@@ -73,7 +73,7 @@ class DeleteController extends Controller
     {
         $subTitle = (string) trans('firefly.delete_recurring', ['title' => $recurrence->title]);
         // put previous url in session
-        $this->rememberPreviousUri('recurrences.delete.uri');
+        $this->rememberPreviousUrl('recurrences.delete.url');
 
         $journalsCreated = $this->recurring->getTransactions($recurrence)->count();
 
@@ -95,7 +95,7 @@ class DeleteController extends Controller
         $request->session()->flash('success', (string) trans('firefly.' . 'recurrence_deleted', ['title' => $recurrence->title]));
         app('preferences')->mark();
 
-        return redirect($this->getPreviousUri('recurrences.delete.uri'));
+        return redirect($this->getPreviousUrl('recurrences.delete.url'));
     }
 
 }

app/Http/Controllers/Recurring/EditController.php

@@ -112,7 +112,7 @@ class EditController extends Controller
 
         // put previous url in session if not redirect from store (not "return_to_edit").
         if (true !== session('recurrences.edit.fromUpdate')) {
-            $this->rememberPreviousUri('recurrences.edit.uri');
+            $this->rememberPreviousUrl('recurrences.edit.url');
         }
         $request->session()->forget('recurrences.edit.fromUpdate');
 
@@ -186,7 +186,7 @@ class EditController extends Controller
             $request->session()->flash('info', $this->attachments->getMessages()->get('attachments'));
         }
         app('preferences')->mark();
-        $redirect = redirect($this->getPreviousUri('recurrences.edit.uri'));
+        $redirect = redirect($this->getPreviousUrl('recurrences.edit.url'));
         if (1 === (int) $request->get('return_to_edit')) {
             // set value so edit routine will not overwrite URL:
             $request->session()->put('recurrences.edit.fromUpdate', true);

app/Http/Controllers/ReportController.php

@@ -377,7 +377,7 @@ class ReportController extends Controller
             return view('error')->with('message', (string) trans('firefly.end_after_start_date'));
         }
 
-        $uri = match ($reportType) {
+        $url = match ($reportType) {
             default => route('reports.report.default', [$accounts, $start, $end]),
             'category' => route('reports.report.category', [$accounts, $categories, $start, $end]),
             'audit' => route('reports.report.audit', [$accounts, $start, $end]),
@@ -386,7 +386,7 @@ class ReportController extends Controller
             'double' => route('reports.report.double', [$accounts, $double, $start, $end]),
         };
 
-        return redirect($uri);
+        return redirect($url);
     }
 
     /**

app/Http/Controllers/Rule/CreateController.php

@@ -122,7 +122,7 @@ class CreateController extends Controller
 
         // put previous url in session if not redirect from store (not "create another").
         if (true !== session('rules.create.fromStore')) {
-            $this->rememberPreviousUri('rules.create.uri');
+            $this->rememberPreviousUrl('rules.create.url');
         }
         session()->forget('rules.create.fromStore');
 
@@ -175,7 +175,7 @@ class CreateController extends Controller
 
         // put previous url in session if not redirect from store (not "create another").
         if (true !== session('rules.create.fromStore')) {
-            $this->rememberPreviousUri('rules.create.uri');
+            $this->rememberPreviousUrl('rules.create.url');
         }
         session()->forget('rules.create.fromStore');
 
@@ -225,7 +225,7 @@ class CreateController extends Controller
 
         // put previous url in session if not redirect from store (not "create another").
         if (true !== session('rules.create.fromStore')) {
-            $this->rememberPreviousUri('rules.create.uri');
+            $this->rememberPreviousUrl('rules.create.url');
         }
         session()->forget('rules.create.fromStore');
 
@@ -272,10 +272,10 @@ class CreateController extends Controller
 
         // redirect to new bill creation.
         if ((int) $request->get('bill_id') > 0) {
-            return redirect($this->getPreviousUri('bills.create.uri'));
+            return redirect($this->getPreviousUrl('bills.create.url'));
         }
 
-        $redirect = redirect($this->getPreviousUri('rules.create.uri'));
+        $redirect = redirect($this->getPreviousUrl('rules.create.url'));
 
         if (1 === (int) $request->get('create_another')) {
 

app/Http/Controllers/Rule/DeleteController.php

@@ -71,7 +71,7 @@ class DeleteController extends Controller
         $subTitle = (string) trans('firefly.delete_rule', ['title' => $rule->title]);
 
         // put previous url in session
-        $this->rememberPreviousUri('rules.delete.uri');
+        $this->rememberPreviousUrl('rules.delete.url');
 
         return view('rules.rule.delete', compact('rule', 'subTitle'));
     }
@@ -91,7 +91,7 @@ class DeleteController extends Controller
         session()->flash('success', (string) trans('firefly.deleted_rule', ['title' => $title]));
         app('preferences')->mark();
 
-        return redirect($this->getPreviousUri('rules.delete.uri'));
+        return redirect($this->getPreviousUrl('rules.delete.url'));
     }
 
 }

app/Http/Controllers/Rule/EditController.php

@@ -127,7 +127,7 @@ class EditController extends Controller
 
         // put previous url in session if not redirect from store (not "return_to_edit").
         if (true !== session('rules.edit.fromUpdate')) {
-            $this->rememberPreviousUri('rules.edit.uri');
+            $this->rememberPreviousUrl('rules.edit.url');
         }
         session()->forget('rules.edit.fromUpdate');
 
@@ -193,7 +193,7 @@ class EditController extends Controller
 
         session()->flash('success', (string) trans('firefly.updated_rule', ['title' => $rule->title]));
         app('preferences')->mark();
-        $redirect = redirect($this->getPreviousUri('rules.edit.uri'));
+        $redirect = redirect($this->getPreviousUrl('rules.edit.url'));
         if (1 === (int) $request->get('return_to_edit')) {
 
             session()->put('rules.edit.fromUpdate', true);

app/Http/Controllers/RuleGroup/CreateController.php

@@ -72,7 +72,7 @@ class CreateController extends Controller
 
         // put previous url in session if not redirect from store (not "create another").
         if (true !== session('rule-groups.create.fromStore')) {
-            $this->rememberPreviousUri('rule-groups.create.uri');
+            $this->rememberPreviousUrl('rule-groups.create.url');
         }
         session()->forget('rule-groups.create.fromStore');
 
@@ -94,7 +94,7 @@ class CreateController extends Controller
         session()->flash('success', (string) trans('firefly.created_new_rule_group', ['title' => $ruleGroup->title]));
         app('preferences')->mark();
 
-        $redirect = redirect($this->getPreviousUri('rule-groups.create.uri'));
+        $redirect = redirect($this->getPreviousUrl('rule-groups.create.url'));
         if (1 === (int) $request->get('create_another')) {
 
             session()->put('rule-groups.create.fromStore', true);

app/Http/Controllers/RuleGroup/DeleteController.php

@@ -73,7 +73,7 @@ class DeleteController extends Controller
         $subTitle = (string) trans('firefly.delete_rule_group', ['title' => $ruleGroup->title]);
 
         // put previous url in session
-        $this->rememberPreviousUri('rule-groups.delete.uri');
+        $this->rememberPreviousUrl('rule-groups.delete.url');
 
         return view('rules.rule-group.delete', compact('ruleGroup', 'subTitle'));
     }
@@ -97,7 +97,7 @@ class DeleteController extends Controller
         session()->flash('success', (string) trans('firefly.deleted_rule_group', ['title' => $title]));
         app('preferences')->mark();
 
-        return redirect($this->getPreviousUri('rule-groups.delete.uri'));
+        return redirect($this->getPreviousUrl('rule-groups.delete.url'));
     }
 
 }

app/Http/Controllers/RuleGroup/EditController.php

@@ -80,7 +80,7 @@ class EditController extends Controller
         ];
         // put previous url in session if not redirect from store (not "return_to_edit").
         if (true !== session('rule-groups.edit.fromUpdate')) {
-            $this->rememberPreviousUri('rule-groups.edit.uri');
+            $this->rememberPreviousUrl('rule-groups.edit.url');
         }
         session()->forget('rule-groups.edit.fromUpdate');
         session()->flash('preFilled', $preFilled);
@@ -131,7 +131,7 @@ class EditController extends Controller
     public function update(RuleGroupFormRequest $request, RuleGroup $ruleGroup)
     {
         $data = [
-            'title'       => $request->string('title'),
+            'title'       => $request->convertString('title'),
             'description' => $request->stringWithNewlines('description'),
             'active'      => 1 === (int) $request->input('active'),
         ];
@@ -140,7 +140,7 @@ class EditController extends Controller
 
         session()->flash('success', (string) trans('firefly.updated_rule_group', ['title' => $ruleGroup->title]));
         app('preferences')->mark();
-        $redirect = redirect($this->getPreviousUri('rule-groups.edit.uri'));
+        $redirect = redirect($this->getPreviousUrl('rule-groups.edit.url'));
         if (1 === (int) $request->get('return_to_edit')) {
 
             session()->put('rule-groups.edit.fromUpdate', true);

app/Http/Controllers/TagController.php

@@ -52,7 +52,7 @@ class TagController extends Controller
     public function __construct()
     {
         parent::__construct();
-        $this->redirectUri = route('tags.index');
+        $this->redirectUrl = route('tags.index');
 
         $this->middleware(
             function ($request, $next) {
@@ -90,7 +90,7 @@ class TagController extends Controller
 
         // put previous url in session if not redirect from store (not "create another").
         if (true !== session('tags.create.fromStore')) {
-            $this->rememberPreviousUri('tags.create.uri');
+            $this->rememberPreviousUrl('tags.create.url');
         }
         session()->forget('tags.create.fromStore');
 
@@ -109,7 +109,7 @@ class TagController extends Controller
         $subTitle = (string) trans('breadcrumbs.delete_tag', ['tag' => $tag->tag]);
 
         // put previous url in session
-        $this->rememberPreviousUri('tags.delete.uri');
+        $this->rememberPreviousUrl('tags.delete.url');
 
         return view('tags.delete', compact('tag', 'subTitle'));
     }
@@ -129,7 +129,7 @@ class TagController extends Controller
         session()->flash('success', (string) trans('firefly.deleted_tag', ['tag' => $tagName]));
         app('preferences')->mark();
 
-        return redirect($this->getPreviousUri('tags.delete.uri'));
+        return redirect($this->getPreviousUrl('tags.delete.url'));
     }
 
     /**
@@ -160,7 +160,7 @@ class TagController extends Controller
 
         // put previous url in session if not redirect from store (not "return_to_edit").
         if (true !== session('tags.edit.fromUpdate')) {
-            $this->rememberPreviousUri('tags.edit.uri');
+            $this->rememberPreviousUrl('tags.edit.url');
         }
         session()->forget('tags.edit.fromUpdate');
 
@@ -333,7 +333,7 @@ class TagController extends Controller
         if (count($this->attachmentsHelper->getMessages()->get('attachments')) > 0) {
             $request->session()->flash('info', $this->attachmentsHelper->getMessages()->get('attachments'));
         }
-        $redirect = redirect($this->getPreviousUri('tags.create.uri'));
+        $redirect = redirect($this->getPreviousUrl('tags.create.url'));
         if (1 === (int) $request->get('create_another')) {
 
             session()->put('tags.create.fromStore', true);
@@ -374,7 +374,7 @@ class TagController extends Controller
         if (count($this->attachmentsHelper->getMessages()->get('attachments')) > 0) {
             $request->session()->flash('info', $this->attachmentsHelper->getMessages()->get('attachments'));
         }
-        $redirect = redirect($this->getPreviousUri('tags.edit.uri'));
+        $redirect = redirect($this->getPreviousUrl('tags.edit.url'));
         if (1 === (int) $request->get('return_to_edit')) {
 
             session()->put('tags.edit.fromUpdate', true);

app/Http/Controllers/Transaction/BulkController.php

@@ -76,7 +76,7 @@ class BulkController extends Controller
     {
         $subTitle = (string) trans('firefly.mass_bulk_journals');
 
-        $this->rememberPreviousUri('transactions.bulk-edit.uri');
+        $this->rememberPreviousUrl('transactions.bulk-edit.url');
 
         // make amounts positive.
 
@@ -110,8 +110,8 @@ class BulkController extends Controller
             $journal   = $this->repository->find($journalId);
             if (null !== $journal) {
                 $resultA = $this->updateJournalBudget($journal, $ignoreBudget, $request->integer('budget_id'));
-                $resultB = $this->updateJournalTags($journal, $tagsAction, explode(',', $request->string('tags')));
-                $resultC = $this->updateJournalCategory($journal, $ignoreCategory, $request->string('category'));
+                $resultB = $this->updateJournalTags($journal, $tagsAction, explode(',', $request->convertString('tags')));
+                $resultC = $this->updateJournalCategory($journal, $ignoreCategory, $request->convertString('category'));
                 if ($resultA || $resultB || $resultC) {
                     $count++;
                 }
@@ -121,7 +121,7 @@ class BulkController extends Controller
         $request->session()->flash('success', (string) trans_choice('firefly.mass_edited_transactions_success', $count));
 
         // redirect to previous URL:
-        return redirect($this->getPreviousUri('transactions.bulk-edit.uri'));
+        return redirect($this->getPreviousUrl('transactions.bulk-edit.url'));
     }
 
     /**

app/Http/Controllers/Transaction/CreateController.php

@@ -121,7 +121,7 @@ class CreateController extends Controller
         $allowedOpposingTypes = config('firefly.allowed_opposing_types');
         $accountToTypes       = config('firefly.account_to_transaction');
         $defaultCurrency      = app('amount')->getDefaultCurrency();
-        $previousUrl          = $this->rememberPreviousUri('transactions.create.uri');
+        $previousUrl          = $this->rememberPreviousUrl('transactions.create.url');
         $parts                = parse_url($previousUrl);
         $search               = sprintf('?%s', $parts['query'] ?? '');
         $previousUrl          = str_replace($search, '', $previousUrl);

app/Http/Controllers/Transaction/DeleteController.php

@@ -85,8 +85,8 @@ class DeleteController extends Controller
         $subTitle   = (string) trans('firefly.delete_' . $objectType, ['description' => $group->title ?? $journal->description]);
         $previous   = app('steam')->getSafePreviousUrl(route('index'));
         // put previous url in session
-        Log::debug('Will try to remember previous URI');
-        $this->rememberPreviousUri('transactions.delete.uri');
+        Log::debug('Will try to remember previous URL');
+        $this->rememberPreviousUrl('transactions.delete.url');
 
         return view('transactions.delete', compact('group', 'journal', 'subTitle', 'objectType', 'previous'));
     }
@@ -115,6 +115,6 @@ class DeleteController extends Controller
 
         app('preferences')->mark();
 
-        return redirect($this->getPreviousUri('transactions.delete.uri'));
+        return redirect($this->getPreviousUrl('transactions.delete.url'));
     }
 }

app/Http/Controllers/Transaction/EditController.php

@@ -80,7 +80,7 @@ class EditController extends Controller
 
         $defaultCurrency = app('amount')->getDefaultCurrency();
         $cash            = $repository->getCashAccount();
-        $previousUrl     = $this->rememberPreviousUri('transactions.edit.uri');
+        $previousUrl     = $this->rememberPreviousUrl('transactions.edit.url');
         $parts           = parse_url($previousUrl);
         $search          = sprintf('?%s', $parts['query'] ?? '');
         $previousUrl     = str_replace($search, '', $previousUrl);

app/Http/Controllers/Transaction/LinkController.php

@@ -76,7 +76,7 @@ class LinkController extends Controller
     {
         $subTitleIcon = 'fa-link';
         $subTitle     = (string) trans('breadcrumbs.delete_journal_link');
-        $this->rememberPreviousUri('journal_links.delete.uri');
+        $this->rememberPreviousUrl('journal_links.delete.url');
 
         return view('transactions.links.delete', compact('link', 'subTitle', 'subTitleIcon'));
     }
@@ -95,7 +95,7 @@ class LinkController extends Controller
         session()->flash('success', (string) trans('firefly.deleted_link'));
         app('preferences')->mark();
 
-        return redirect((string) session('journal_links.delete.uri'));
+        return redirect((string) session('journal_links.delete.url'));
     }
 
     /**

app/Http/Controllers/Transaction/MassController.php

@@ -82,7 +82,7 @@ class MassController extends Controller
         $subTitle = (string) trans('firefly.mass_delete_journals');
 
         // put previous url in session
-        $this->rememberPreviousUri('transactions.mass-delete.uri');
+        $this->rememberPreviousUrl('transactions.mass-delete.url');
 
         return view('transactions.mass.delete', compact('journals', 'subTitle'));
     }
@@ -115,7 +115,7 @@ class MassController extends Controller
         session()->flash('success', (string) trans_choice('firefly.mass_deleted_transactions_success', $count));
 
         // redirect to previous URL:
-        return redirect($this->getPreviousUri('transactions.mass-delete.uri'));
+        return redirect($this->getPreviousUrl('transactions.mass-delete.url'));
     }
 
     /**
@@ -151,7 +151,7 @@ class MassController extends Controller
                 null : app('steam')->positive($journal['foreign_amount']);
         }
 
-        $this->rememberPreviousUri('transactions.mass-edit.uri');
+        $this->rememberPreviousUrl('transactions.mass-edit.url');
 
         return view('transactions.mass.edit', compact('journals', 'subTitle', 'withdrawalSources', 'depositDestinations', 'budgets'));
     }
@@ -187,7 +187,7 @@ class MassController extends Controller
         session()->flash('success', (string) trans_choice('firefly.mass_edited_transactions_success', $count));
 
         // redirect to previous URL:
-        return redirect($this->getPreviousUri('transactions.mass-edit.uri'));
+        return redirect($this->getPreviousUrl('transactions.mass-edit.url'));
     }
 
     /**

app/Http/Kernel.php

@@ -60,8 +60,6 @@ class Kernel extends HttpKernel
     /**
      * The application's global HTTP middleware stack.
      *
-     * These middleware are run during every request to your application.
-     *
      * @var array
      */
     protected $middleware

app/Http/Middleware/Installer.php

@@ -53,7 +53,7 @@ class Installer
      */
     public function handle($request, Closure $next)
     {
-        Log::debug(sprintf('Installer middleware for URI %s', $request->url()));
+        Log::debug(sprintf('Installer middleware for URL %s', $request->url()));
         // ignore installer in test environment.
         if ('testing' === config('app.env')) {
             return $next($request);

app/Http/Requests/AccountFormRequest.php

@@ -45,24 +45,24 @@ class AccountFormRequest extends FormRequest
     public function getAccountData(): array
     {
         $data = [
-            'name'                    => $this->string('name'),
+            'name'                    => $this->convertString('name'),
             'active'                  => $this->boolean('active'),
-            'account_type_name'       => $this->string('objectType'),
+            'account_type_name'       => $this->convertString('objectType'),
             'currency_id'             => $this->integer('currency_id'),
-            'virtual_balance'         => $this->string('virtual_balance'),
-            'iban'                    => $this->string('iban'),
-            'BIC'                     => $this->string('BIC'),
-            'account_number'          => $this->string('account_number'),
-            'account_role'            => $this->string('account_role'),
-            'opening_balance'         => $this->string('opening_balance'),
+            'virtual_balance'         => $this->convertString('virtual_balance'),
+            'iban'                    => $this->convertString('iban'),
+            'BIC'                     => $this->convertString('BIC'),
+            'account_number'          => $this->convertString('account_number'),
+            'account_role'            => $this->convertString('account_role'),
+            'opening_balance'         => $this->convertString('opening_balance'),
             'opening_balance_date'    => $this->getCarbonDate('opening_balance_date'),
-            'cc_type'                 => $this->string('cc_type'),
-            'cc_monthly_payment_date' => $this->string('cc_monthly_payment_date'),
+            'cc_type'                 => $this->convertString('cc_type'),
+            'cc_monthly_payment_date' => $this->convertString('cc_monthly_payment_date'),
             'notes'                   => $this->stringWithNewlines('notes'),
-            'interest'                => $this->string('interest'),
-            'interest_period'         => $this->string('interest_period'),
+            'interest'                => $this->convertString('interest'),
+            'interest_period'         => $this->convertString('interest_period'),
             'include_net_worth'       => '1',
-            'liability_direction'     => $this->string('liability_direction'),
+            'liability_direction'     => $this->convertString('liability_direction'),
         ];
 
         $data = $this->appendLocationData($data, 'location');
@@ -100,7 +100,7 @@ class AccountFormRequest extends FormRequest
             'name'                               => 'required|min:1|uniqueAccountForUser',
             'opening_balance'                    => 'numeric|nullable|max:1000000000',
             'opening_balance_date'               => 'date|required_with:opening_balance|nullable',
-            'iban'                               => ['iban', 'nullable', new UniqueIban(null, $this->string('objectType'))],
+            'iban'                               => ['iban', 'nullable', new UniqueIban(null, $this->convertString('objectType'))],
             'BIC'                                => 'bic|nullable',
             'virtual_balance'                    => 'numeric|nullable|max:1000000000',
             'currency_id'                        => 'exists:transaction_currencies,id',

app/Http/Requests/AttachmentFormRequest.php

@@ -43,8 +43,8 @@ class AttachmentFormRequest extends FormRequest
     public function getAttachmentData(): array
     {
         return [
-            'title' => $this->string('title'),
-            'notes' => $this->string('notes'),
+            'title' => $this->convertString('title'),
+            'notes' => $this->convertString('notes'),
         ];
     }
 

app/Http/Requests/BillStoreRequest.php

@@ -41,19 +41,19 @@ class BillStoreRequest extends FormRequest
     public function getBillData(): array
     {
         return [
-            'name'               => $this->string('name'),
-            'amount_min'         => $this->string('amount_min'),
+            'name'               => $this->convertString('name'),
+            'amount_min'         => $this->convertString('amount_min'),
             'currency_id'        => $this->integer('transaction_currency_id'),
             'currency_code'      => '',
-            'amount_max'         => $this->string('amount_max'),
+            '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->string('repeat_freq'),
+            'repeat_freq'        => $this->convertString('repeat_freq'),
             'skip'               => $this->integer('skip'),
             'notes'              => $this->stringWithNewlines('notes'),
             'active'             => $this->boolean('active'),
-            'object_group_title' => $this->string('object_group'),
+            'object_group_title' => $this->convertString('object_group'),
         ];
     }
 

app/Http/Requests/BillUpdateRequest.php

@@ -42,19 +42,19 @@ class BillUpdateRequest extends FormRequest
     public function getBillData(): array
     {
         return [
-            'name'               => $this->string('name'),
-            'amount_min'         => $this->string('amount_min'),
+            'name'               => $this->convertString('name'),
+            'amount_min'         => $this->convertString('amount_min'),
             'currency_id'        => $this->integer('transaction_currency_id'),
             'currency_code'      => '',
-            'amount_max'         => $this->string('amount_max'),
+            '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->string('repeat_freq'),
+            'repeat_freq'        => $this->convertString('repeat_freq'),
             'skip'               => $this->integer('skip'),
             'notes'              => $this->stringWithNewlines('notes'),
             'active'             => $this->boolean('active'),
-            'object_group_title' => $this->string('object_group'),
+            'object_group_title' => $this->convertString('object_group'),
         ];
     }
 

app/Http/Requests/BudgetFormStoreRequest.php

@@ -44,12 +44,12 @@ class BudgetFormStoreRequest extends FormRequest
     public function getBudgetData(): array
     {
         return [
-            'name'               => $this->string('name'),
+            '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_amount' => $this->string('auto_budget_amount'),
-            'auto_budget_period' => $this->string('auto_budget_period'),
+            'auto_budget_amount' => $this->convertString('auto_budget_amount'),
+            'auto_budget_period' => $this->convertString('auto_budget_period'),
         ];
     }
 

app/Http/Requests/BudgetFormUpdateRequest.php

@@ -45,12 +45,12 @@ class BudgetFormUpdateRequest extends FormRequest
     public function getBudgetData(): array
     {
         return [
-            'name'               => $this->string('name'),
+            '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_amount' => $this->string('auto_budget_amount'),
-            'auto_budget_period' => $this->string('auto_budget_period'),
+            'auto_budget_amount' => $this->convertString('auto_budget_amount'),
+            'auto_budget_period' => $this->convertString('auto_budget_period'),
         ];
     }
 

app/Http/Requests/CategoryFormRequest.php

@@ -42,7 +42,7 @@ class CategoryFormRequest extends FormRequest
     public function getCategoryData(): array
     {
         return [
-            'name'  => $this->string('name'),
+            'name'  => $this->convertString('name'),
             'notes' => $this->stringWithNewlines('notes'),
         ];
     }

app/Http/Requests/CurrencyFormRequest.php

@@ -42,9 +42,9 @@ class CurrencyFormRequest extends FormRequest
     public function getCurrencyData(): array
     {
         return [
-            'name'           => $this->string('name'),
-            'code'           => $this->string('code'),
-            'symbol'         => $this->string('symbol'),
+            'name'           => $this->convertString('name'),
+            'code'           => $this->convertString('code'),
+            'symbol'         => $this->convertString('symbol'),
             'decimal_places' => $this->integer('decimal_places'),
             'enabled'        => $this->boolean('enabled'),
         ];

app/Http/Requests/JournalLinkRequest.php

@@ -46,7 +46,7 @@ class JournalLinkRequest extends FormRequest
         $parts                            = explode('_', $linkType);
         $return['link_type_id']           = (int) $parts[0];
         $return['transaction_journal_id'] = $this->integer('opposing');
-        $return['notes']                  = $this->string('notes');
+        $return['notes']                  = $this->convertString('notes');
         $return['direction']              = $parts[1];
 
         return $return;

app/Http/Requests/ObjectGroupFormRequest.php

@@ -42,7 +42,7 @@ class ObjectGroupFormRequest extends FormRequest
     public function getObjectGroupData(): array
     {
         return [
-            'title' => $this->string('title'),
+            'title' => $this->convertString('title'),
         ];
     }
 

app/Http/Requests/PiggyBankStoreRequest.php

@@ -41,13 +41,13 @@ class PiggyBankStoreRequest extends FormRequest
     public function getPiggyBankData(): array
     {
         return [
-            'name'               => $this->string('name'),
+            'name'               => $this->convertString('name'),
             'startdate'          => $this->getCarbonDate('startdate'),
             'account_id'         => $this->integer('account_id'),
-            'targetamount'       => $this->string('targetamount'),
+            'targetamount'       => $this->convertString('targetamount'),
             'targetdate'         => $this->getCarbonDate('targetdate'),
             'notes'              => $this->stringWithNewlines('notes'),
-            'object_group_title' => $this->string('object_group'),
+            'object_group_title' => $this->convertString('object_group'),
         ];
     }
 

app/Http/Requests/PiggyBankUpdateRequest.php

@@ -42,13 +42,13 @@ class PiggyBankUpdateRequest extends FormRequest
     public function getPiggyBankData(): array
     {
         return [
-            'name'               => $this->string('name'),
+            'name'               => $this->convertString('name'),
             'startdate'          => $this->getCarbonDate('startdate'),
             'account_id'         => $this->integer('account_id'),
-            'targetamount'       => $this->string('targetamount'),
+            'targetamount'       => $this->convertString('targetamount'),
             'targetdate'         => $this->getCarbonDate('targetdate'),
             'notes'              => $this->stringWithNewlines('notes'),
-            'object_group_title' => $this->string('object_group'),
+            'object_group_title' => $this->convertString('object_group'),
         ];
     }
 

app/Http/Requests/ReconciliationStoreRequest.php

@@ -50,11 +50,11 @@ class ReconciliationStoreRequest extends FormRequest
         $data = [
             'start'         => $this->getCarbonDate('start'),
             'end'           => $this->getCarbonDate('end'),
-            'start_balance' => $this->string('startBalance'),
-            'end_balance'   => $this->string('endBalance'),
-            'difference'    => $this->string('difference'),
+            'start_balance' => $this->convertString('startBalance'),
+            'end_balance'   => $this->convertString('endBalance'),
+            'difference'    => $this->convertString('difference'),
             'journals'      => $transactions,
-            'reconcile'     => $this->string('reconcile'),
+            'reconcile'     => $this->convertString('reconcile'),
         ];
         Log::debug('In ReconciliationStoreRequest::getAll(). Will now return data.');
 

app/Http/Requests/RecurrenceFormRequest.php

@@ -56,23 +56,23 @@ class RecurrenceFormRequest extends FormRequest
         $repetitionData = $this->parseRepetitionData();
         $return         = [
             'recurrence'   => [
-                'type'              => $this->string('transaction_type'),
-                'title'             => $this->string('title'),
-                'description'       => $this->string('recurring_description'),
+                'type'              => $this->convertString('transaction_type'),
+                'title'             => $this->convertString('title'),
+                'description'       => $this->convertString('recurring_description'),
                 'first_date'        => $this->getCarbonDate('first_date'),
                 'repeat_until'      => $this->getCarbonDate('repeat_until'),
                 'nr_of_repetitions' => $this->integer('repetitions'),
                 'apply_rules'       => $this->boolean('apply_rules'),
                 'active'            => $this->boolean('active'),
-                'repetition_end'    => $this->string('repetition_end'),
+                'repetition_end'    => $this->convertString('repetition_end'),
             ],
             'transactions' => [
                 [
                     'currency_id'           => $this->integer('transaction_currency_id'),
                     'currency_code'         => null,
-                    'type'                  => $this->string('transaction_type'),
-                    'description'           => $this->string('transaction_description'),
-                    'amount'                => $this->string('amount'),
+                    'type'                  => $this->convertString('transaction_type'),
+                    'description'           => $this->convertString('transaction_description'),
+                    'amount'                => $this->convertString('amount'),
                     'foreign_amount'        => null,
                     'foreign_currency_id'   => null,
                     'foreign_currency_code' => null,
@@ -81,8 +81,8 @@ class RecurrenceFormRequest extends FormRequest
                     'bill_id'               => $this->integer('bill_id'),
                     'bill_name'             => null,
                     'category_id'           => null,
-                    'category_name'         => $this->string('category'),
-                    'tags'                  => '' !== $this->string('tags') ? explode(',', $this->string('tags')) : [],
+                    'category_name'         => $this->convertString('category'),
+                    'tags'                  => '' !== $this->convertString('tags') ? explode(',', $this->convertString('tags')) : [],
                     'piggy_bank_id'         => $this->integer('piggy_bank_id'),
                     'piggy_bank_name'       => null,
                 ],
@@ -100,7 +100,7 @@ class RecurrenceFormRequest extends FormRequest
 
         // fill in foreign currency data
         if (null !== $this->float('foreign_amount')) {
-            $return['transactions'][0]['foreign_amount']      = $this->string('foreign_amount');
+            $return['transactions'][0]['foreign_amount']      = $this->convertString('foreign_amount');
             $return['transactions'][0]['foreign_currency_id'] = $this->integer('foreign_currency_id');
         }
         // default values:
@@ -109,9 +109,9 @@ class RecurrenceFormRequest extends FormRequest
         $return['transactions'][0]['destination_id']   = null;
         $return['transactions'][0]['destination_name'] = null;
         // fill in source and destination account data
-        switch ($this->string('transaction_type')) {
+        switch ($this->convertString('transaction_type')) {
             default:
-                throw new FireflyException(sprintf('Cannot handle transaction type "%s"', $this->string('transaction_type')));
+                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');
@@ -149,7 +149,7 @@ class RecurrenceFormRequest extends FormRequest
      */
     private function parseRepetitionData(): array
     {
-        $value  = $this->string('repetition_type');
+        $value  = $this->convertString('repetition_type');
         $return = [
             'type'   => '',
             'moment' => '',
@@ -224,7 +224,7 @@ class RecurrenceFormRequest extends FormRequest
         }
 
         // if ends after X repetitions, set another rule
-        if ('times' === $this->string('repetition_end')) {
+        if ('times' === $this->convertString('repetition_end')) {
             $rules['repetitions'] = 'required|numeric|between:0,254';
         }
         // if foreign amount, currency must be  different.
@@ -233,12 +233,12 @@ class RecurrenceFormRequest extends FormRequest
         }
 
         // if ends at date X, set another rule.
-        if ('until_date' === $this->string('repetition_end')) {
+        if ('until_date' === $this->convertString('repetition_end')) {
             $rules['repeat_until'] = 'required|date|after:' . $tomorrow->format('Y-m-d');
         }
 
         // switchc on type to expand rules for source and destination accounts:
-        switch ($this->string('transaction_type')) {
+        switch ($this->convertString('transaction_type')) {
             case strtolower(TransactionType::WITHDRAWAL):
                 $rules['source_id']        = 'required|exists:accounts,id|belongsToUser:accounts';
                 $rules['destination_name'] = 'between:1,255|nullable';
@@ -254,7 +254,7 @@ class RecurrenceFormRequest extends FormRequest
 
                 break;
             default:
-                throw new FireflyException(sprintf('Cannot handle transaction type of type "%s"', $this->string('transaction_type')));
+                throw new FireflyException(sprintf('Cannot handle transaction type of type "%s"', $this->convertString('transaction_type')));
         }
 
         // update some rules in case the user is editing a post:
@@ -309,9 +309,9 @@ class RecurrenceFormRequest extends FormRequest
 
         // See reference nr. 45
 
-        switch ($this->string('transaction_type')) {
+        switch ($this->convertString('transaction_type')) {
             default:
-                throw new FireflyException(sprintf('Cannot handle transaction type "%s"', $this->string('transaction_type')));
+                throw new FireflyException(sprintf('Cannot handle transaction type "%s"', $this->convertString('transaction_type')));
             case 'withdrawal':
                 $sourceId      = (int) $data['source_id'];
                 $destinationId = (int) $data['withdrawal_destination_id'];

app/Http/Requests/RuleFormRequest.php

@@ -44,10 +44,10 @@ class RuleFormRequest extends FormRequest
     public function getRuleData(): array
     {
         return [
-            'title'           => $this->string('title'),
+            'title'           => $this->convertString('title'),
             'rule_group_id'   => $this->integer('rule_group_id'),
             'active'          => $this->boolean('active'),
-            'trigger'         => $this->string('trigger'),
+            'trigger'         => $this->convertString('trigger'),
             'description'     => $this->stringWithNewlines('description'),
             'stop_processing' => $this->boolean('stop_processing'),
             'strict'          => $this->boolean('strict'),

app/Http/Requests/RuleGroupFormRequest.php

@@ -48,7 +48,7 @@ class RuleGroupFormRequest extends FormRequest
         }
 
         return [
-            'title'       => $this->string('title'),
+            'title'       => $this->convertString('title'),
             'description' => $this->stringWithNewlines('description'),
             'active'      => $active,
         ];

app/Http/Requests/TagFormRequest.php

@@ -44,9 +44,9 @@ class TagFormRequest extends FormRequest
     public function collectTagData(): array
     {
         $data = [
-            'tag'         => $this->string('tag'),
+            'tag'         => $this->convertString('tag'),
             'date'        => $this->getCarbonDate('date'),
-            'description' => $this->string('description'),
+            'description' => $this->convertString('description'),
         ];
 
         return $this->appendLocationData($data, 'location');

app/Http/Requests/UserFormRequest.php

@@ -43,10 +43,10 @@ class UserFormRequest extends FormRequest
     public function getUserData(): array
     {
         return [
-            'email'        => $this->string('email'),
+            'email'        => $this->convertString('email'),
             'blocked'      => 1 === $this->integer('blocked'),
-            'blocked_code' => $this->string('blocked_code'),
-            'password'     => $this->string('password'),
+            'blocked_code' => $this->convertString('blocked_code'),
+            'password'     => $this->convertString('password'),
             'is_owner'     => 1 === $this->integer('is_owner'),
         ];
     }

app/Providers/AppServiceProvider.php

@@ -45,7 +45,6 @@ class AppServiceProvider extends ServiceProvider
         if ('heroku' === config('app.env')) {
             URL::forceScheme('https');
         }
-        Sanctum::ignoreMigrations();
     }
 
     /**
@@ -56,5 +55,6 @@ class AppServiceProvider extends ServiceProvider
     public function register(): void
     {
         Passport::ignoreMigrations();
+        Sanctum::ignoreMigrations();
     }
 }

app/Repositories/PiggyBank/ModifiesPiggyBanks.php

@@ -261,6 +261,12 @@ trait ModifiesPiggyBanks
         $piggyData     = $data;
         // unset fields just in case.
         unset($piggyData['object_group_title'], $piggyData['object_group_id'], $piggyData['notes'], $piggyData['current_amount']);
+
+        // validate amount:
+        if (array_key_exists('targetamount', $piggyData) && '' === (string) $piggyData['targetamount']) {
+            $piggyData['targetamount'] = '0';
+        }
+
         try {
             /** @var PiggyBank $piggyBank */
             $piggyBank = PiggyBank::create($piggyData);
@@ -403,7 +409,7 @@ trait ModifiesPiggyBanks
         // if the piggy bank is now smaller than the current relevant rep,
         // remove money from the rep.
         $repetition = $this->getRepetition($piggyBank);
-        if (null !== $repetition && $repetition->currentamount > $piggyBank->targetamount && 0.0 !== (float)$piggyBank->targetamount) {
+        if (null !== $repetition && $repetition->currentamount > $piggyBank->targetamount && 0.0 !== (float) $piggyBank->targetamount) {
             $diff = bcsub($piggyBank->targetamount, $repetition->currentamount);
             $this->createEvent($piggyBank, $diff);