Merge branch 'release/6.0.9'

Fix 7394

.github/workflows/lock.yml

@@ -5,26 +5,15 @@ on:
   schedule:
     - cron: '0 0 * * *'
 
-permissions:
-  contents: read
-
 jobs:
   lock:
     permissions:
-      issues: write  # for dessant/lock-threads to lock issues
+      issues: write
-      pull-requests: write  # for dessant/lock-threads to lock PRs
+      pull-requests: write
     runs-on: ubuntu-latest
     steps:
-      - uses: dessant/lock-threads@v4
+      - uses: JC5/lock-threads@main
         with:
           github-token: ${{ github.token }}
-          issue-lock-inactive-days: '90'
+          issue-inactive-days: 90
-          issue-comment: >
+          pr-inactive-days: 90
-            Hi there! This is an automatic reply. `Share and enjoy`
-
-            This issue is now `locked` :lock:. 
-
-            - If you feel there is more to be said about this specific issue, please feel free to open a new issue. Please refer to this issue for clarity. 
-            - Follow-up questions and comments can also be posted in a new [discussion](https://github.com/firefly-iii/firefly-iii/discussions/)
-
-            Thank you for your consideration.

.github/workflows/stale.yml

@@ -31,4 +31,4 @@ jobs:
             Thank you for your contributions.
           days-before-stale: 14
           days-before-close: 7
-          exempt-issue-labels: 'enhancement,feature,bug,announcement,layout-v3'
+          exempt-issue-labels: 'enhancement,feature,bug,announcement,epic'

app/Api/V1/Controllers/Controller.php

@@ -85,12 +85,11 @@ abstract class Controller extends BaseController
     {
         $bag  = new ParameterBag();
         $page = (int)request()->get('page');
-
         if ($page < 1) {
             $page = 1;
         }
-        if ($page > (2 ^ 16)) {
+        if ($page > pow(2, 16)) {
-            $page = (2 ^ 16);
+            $page = pow(2, 16);
         }
         $bag->set('page', $page);
 

app/Api/V1/Requests/Data/DestroyRequest.php

@@ -57,7 +57,7 @@ class DestroyRequest extends FormRequest
                  ',not_assets_liabilities';
 
         return [
-            'objects' => sprintf('required|min:1|string|in:%s', $valid),
+            'objects' => sprintf('required|max:255|min:1|string|in:%s', $valid),
             'unused'  => 'in:true,false',
         ];
     }

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

@@ -73,7 +73,7 @@ class ExportRequest extends FormRequest
     {
         return [
             'type'     => 'in:csv',
-            'accounts' => 'min:1',
+            'accounts' => 'min:1|max:65536',
             'start'    => 'date|before:end',
             'end'      => 'date|after:start',
         ];

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

@@ -103,8 +103,8 @@ class StoreRequest extends FormRequest
         $ccPaymentTypes = implode(',', array_keys(config('firefly.ccTypes')));
         $type           = $this->convertString('type');
         $rules          = [
-            'name'                 => 'required|min:1|uniqueAccountForUser',
+            'name'                 => 'required|max:1024|min:1|uniqueAccountForUser',
-            'type'                 => 'required|min:1|'.sprintf('in:%s', $types),
+            'type'                 => 'required|max:1024|min:1|'.sprintf('in:%s', $types),
             'iban'                 => ['iban', 'nullable', new UniqueIban(null, $type)],
             'bic'                  => 'bic|nullable',
             'account_number'       => ['between:1,255', 'nullable', new UniqueAccountNumber(null, $type)],
@@ -120,7 +120,7 @@ class StoreRequest extends FormRequest
             'credit_card_type'     => sprintf('nullable|in:%s|required_if:account_role,ccAsset', $ccPaymentTypes),
             'monthly_payment_date' => 'nullable|date|required_if:account_role,ccAsset|required_if:credit_card_type,monthlyFull',
             'liability_type'       => 'nullable|required_if:type,liability|required_if:type,liabilities|in:loan,debt,mortgage',
-            'liability_amount'     => 'required_with:liability_start_date|min:0|numeric',
+            'liability_amount'     => 'required_with:liability_start_date|min:0|numeric|max:1000000000',
             'liability_start_date' => 'required_with:liability_amount|date',
             'liability_direction'  => 'nullable|required_if:type,liability|required_if:type,liabilities|in:credit,debit',
             'interest'             => 'between:0,100|numeric',

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

@@ -94,7 +94,7 @@ class UpdateRequest extends FormRequest
         $ccPaymentTypes = implode(',', array_keys(config('firefly.ccTypes')));
 
         $rules = [
-            'name'                 => sprintf('min:1|uniqueAccountForUser:%d', $account->id),
+            'name'                 => sprintf('min:1|max:1024|uniqueAccountForUser:%d', $account->id),
             'type'                 => sprintf('in:%s', $types),
             'iban'                 => ['iban', 'nullable', new UniqueIban($account, $this->convertString('type'))],
             'bic'                  => 'bic|nullable',
@@ -104,7 +104,7 @@ class UpdateRequest extends FormRequest
             'virtual_balance'      => 'numeric|nullable',
             'order'                => 'numeric|nullable',
             'currency_id'          => 'numeric|exists:transaction_currencies,id',
-            'currency_code'        => 'min:3|max:3|exists:transaction_currencies,code',
+            'currency_code'        => 'min:3|max:51|exists:transaction_currencies,code',
             'active'               => [new IsBoolean()],
             'include_net_worth'    => [new IsBoolean()],
             'account_role'         => sprintf('in:%s|nullable|required_if:type,asset', $accountRoles),

app/Api/V1/Requests/Models/AvailableBudget/Request.php

@@ -67,7 +67,7 @@ class Request extends FormRequest
     {
         return [
             'currency_id'   => 'numeric|exists:transaction_currencies,id',
-            'currency_code' => 'min:3|max:3|exists:transaction_currencies,code',
+            'currency_code' => 'min:3|max:51|exists:transaction_currencies,code',
             'amount'        => 'numeric|gt:0',
             'start'         => 'date',
             'end'           => 'date',

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

@@ -82,7 +82,7 @@ class StoreRequest extends FormRequest
             'amount_min'     => 'numeric|gt:0|required',
             'amount_max'     => 'numeric|gt:0|required',
             'currency_id'    => 'numeric|exists:transaction_currencies,id',
-            'currency_code'  => 'min:3|max:3|exists:transaction_currencies,code',
+            'currency_code'  => 'min:3|max:51|exists:transaction_currencies,code',
             'date'           => 'date|required',
             'end_date'       => 'date|after:date',
             'extension_date' => 'date|after:date',

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

@@ -84,7 +84,7 @@ class UpdateRequest extends FormRequest
             'amount_min'     => 'numeric|gt:0',
             'amount_max'     => 'numeric|gt:0',
             'currency_id'    => 'numeric|exists:transaction_currencies,id',
-            'currency_code'  => 'min:3|max:3|exists:transaction_currencies,code',
+            'currency_code'  => 'min:3|max:51|exists:transaction_currencies,code',
             'date'           => 'date',
             'end_date'       => 'date|after:date',
             'extension_date' => 'date|after:date',

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

@@ -65,7 +65,7 @@ class StoreRequest extends FormRequest
             'end'           => 'required|after:start|date',
             'amount'        => 'required|gt:0',
             'currency_id'   => 'numeric|exists:transaction_currencies,id',
-            'currency_code' => 'min:3|max:3|exists:transaction_currencies,code',
+            'currency_code' => 'min:3|max:51|exists:transaction_currencies,code',
         ];
     }
 }

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

@@ -69,7 +69,7 @@ class UpdateRequest extends FormRequest
             'end'           => 'date',
             'amount'        => 'gt:0',
             'currency_id'   => 'numeric|exists:transaction_currencies,id',
-            'currency_code' => 'min:3|max:3|exists:transaction_currencies,code',
+            'currency_code' => 'min:3|max:51|exists:transaction_currencies,code',
         ];
     }
 

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

@@ -63,7 +63,7 @@ class UpdateRequest extends FormRequest
         $objectGroup = $this->route()->parameter('objectGroup');
 
         return [
-            'title' => sprintf('min:1|uniqueObjectGroup:%d', $objectGroup->id),
+            'title' => sprintf('max:1024|min:1|uniqueObjectGroup:%d', $objectGroup->id),
             'order' => 'numeric',
         ];
     }

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

@@ -158,9 +158,9 @@ class StoreRequest extends FormRequest
             'transactions.*.amount'                => 'required|numeric|gt:0',
             'transactions.*.foreign_amount'        => 'nullable|numeric|gt:0',
             'transactions.*.currency_id'           => 'nullable|numeric|exists:transaction_currencies,id',
-            'transactions.*.currency_code'         => 'nullable|min:3|max:3|exists:transaction_currencies,code',
+            'transactions.*.currency_code'         => 'nullable|min:3|max:51|exists:transaction_currencies,code',
             'transactions.*.foreign_currency_id'   => 'nullable|numeric|exists:transaction_currencies,id',
-            'transactions.*.foreign_currency_code' => 'nullable|min:3|max:3|exists:transaction_currencies,code',
+            'transactions.*.foreign_currency_code' => 'nullable|min:3|max:51|exists:transaction_currencies,code',
             'transactions.*.source_id'             => ['numeric', 'nullable', new BelongsUser()],
             'transactions.*.source_name'           => 'between:1,255|nullable',
             'transactions.*.destination_id'        => ['numeric', 'nullable', new BelongsUser()],

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

@@ -172,9 +172,9 @@ class UpdateRequest extends FormRequest
             'transactions.*.amount'                => 'numeric|gt:0',
             'transactions.*.foreign_amount'        => 'nullable|numeric|gt:0',
             'transactions.*.currency_id'           => 'nullable|numeric|exists:transaction_currencies,id',
-            'transactions.*.currency_code'         => 'nullable|min:3|max:3|exists:transaction_currencies,code',
+            'transactions.*.currency_code'         => 'nullable|min:3|max:51|exists:transaction_currencies,code',
             'transactions.*.foreign_currency_id'   => 'nullable|numeric|exists:transaction_currencies,id',
-            'transactions.*.foreign_currency_code' => 'nullable|min:3|max:3|exists:transaction_currencies,code',
+            'transactions.*.foreign_currency_code' => 'nullable|min:3|max:51|exists:transaction_currencies,code',
             'transactions.*.source_id'             => ['numeric', 'nullable', new BelongsUser()],
             'transactions.*.source_name'           => 'between:1,255|nullable',
             'transactions.*.destination_id'        => ['numeric', 'nullable', new BelongsUser()],

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

@@ -79,7 +79,7 @@ class StoreRequest extends FormRequest
                 $return[] = [
                     'type'            => $trigger['type'],
                     'value'           => $trigger['value'],
-                    'active'          => $this->convertBoolean((string)($trigger['active'] ?? 'false')),
+                    'active'          => $this->convertBoolean((string)($trigger['active'] ?? 'true')),
                     'stop_processing' => $this->convertBoolean((string)($trigger['stop_processing'] ?? 'false')),
                 ];
             }
@@ -100,7 +100,7 @@ class StoreRequest extends FormRequest
                 $return[] = [
                     'type'            => $action['type'],
                     'value'           => $action['value'],
-                    'active'          => $this->convertBoolean((string)($action['active'] ?? 'false')),
+                    'active'          => $this->convertBoolean((string)($action['active'] ?? 'true')),
                     'stop_processing' => $this->convertBoolean((string)($action['stop_processing'] ?? 'false')),
                 ];
             }
@@ -130,7 +130,7 @@ class StoreRequest extends FormRequest
             'rule_group_title'           => 'nullable|between:1,255|required_without:rule_group_id|belongsToUser:rule_groups,title',
             'trigger'                    => 'required|in:store-journal,update-journal',
             'triggers.*.type'            => 'required|in:'.implode(',', $validTriggers),
-            'triggers.*.value'           => 'required_if:actions.*.type,'.$contextTriggers.'|min:1|ruleTriggerValue',
+            'triggers.*.value'           => 'required_if:actions.*.type,'.$contextTriggers.'|min:1|ruleTriggerValue|max:1024',
             'triggers.*.stop_processing' => [new IsBoolean()],
             'triggers.*.active'          => [new IsBoolean()],
             'actions.*.type'             => 'required|in:'.implode(',', $validActions),

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

@@ -147,7 +147,7 @@ class UpdateRequest extends FormRequest
             'rule_group_title'           => 'nullable|between:1,255|belongsToUser:rule_groups,title',
             'trigger'                    => 'in:store-journal,update-journal',
             'triggers.*.type'            => 'required|in:'.implode(',', $validTriggers),
-            'triggers.*.value'           => 'required_if:actions.*.type,'.$contextTriggers.'|min:1|ruleTriggerValue',
+            'triggers.*.value'           => 'required_if:actions.*.type,'.$contextTriggers.'|min:1|ruleTriggerValue|max:1024',
             'triggers.*.stop_processing' => [new IsBoolean()],
             'triggers.*.active'          => [new IsBoolean()],
             'actions.*.type'             => 'required|in:'.implode(',', $validActions),

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

@@ -65,8 +65,8 @@ class StoreRequest extends FormRequest
     public function rules(): array
     {
         $rules = [
-            'tag'         => 'required|min:1|uniqueObjectForUser:tags,tag',
+            'tag'         => 'required|min:1|uniqueObjectForUser:tags,tag|max:1024',
-            'description' => 'min:1|nullable',
+            'description' => 'min:1|nullable|max:65536',
             'date'        => 'date|nullable',
         ];
 

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

@@ -71,8 +71,8 @@ class UpdateRequest extends FormRequest
         $tag = $this->route()->parameter('tagOrId');
         // TODO check if uniqueObjectForUser is obsolete
         $rules = [
-            'tag'         => 'min:1|uniqueObjectForUser:tags,tag,'.$tag->id,
+            'tag'         => 'min:1|max:1024|uniqueObjectForUser:tags,tag,'.$tag->id,
-            'description' => 'min:1|nullable',
+            'description' => 'min:1|nullable|max:65536',
             'date'        => 'date|nullable',
         ];
 

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

@@ -188,9 +188,9 @@ class StoreRequest extends FormRequest
 
             // currency info
             'transactions.*.currency_id'           => 'numeric|exists:transaction_currencies,id|nullable',
-            'transactions.*.currency_code'         => 'min:3|max:3|exists:transaction_currencies,code|nullable',
+            'transactions.*.currency_code'         => 'min:3|max:51|exists:transaction_currencies,code|nullable',
             'transactions.*.foreign_currency_id'   => 'numeric|exists:transaction_currencies,id|nullable',
-            'transactions.*.foreign_currency_code' => 'min:3|max:3|exists:transaction_currencies,code|nullable',
+            'transactions.*.foreign_currency_code' => 'min:3|max:51|exists:transaction_currencies,code|nullable',
 
             // amount
             'transactions.*.amount'                => 'required|numeric|gt:0',
@@ -225,25 +225,25 @@ class StoreRequest extends FormRequest
 
             // other interesting fields
             'transactions.*.reconciled'            => [new IsBoolean()],
-            'transactions.*.notes'                 => 'min:1,max:50000|nullable',
+            'transactions.*.notes'                 => 'min:1|max:50000|nullable',
             'transactions.*.tags'                  => 'between:0,255',
 
             // meta info fields
-            'transactions.*.internal_reference'    => 'min:1,max:255|nullable',
+            'transactions.*.internal_reference'    => 'min:1|max:255|nullable',
-            'transactions.*.external_id'           => 'min:1,max:255|nullable',
+            'transactions.*.external_id'           => 'min:1|max:255|nullable',
-            'transactions.*.recurrence_id'         => 'min:1,max:255|nullable',
+            'transactions.*.recurrence_id'         => 'min:1|max:255|nullable',
-            'transactions.*.bunq_payment_id'       => 'min:1,max:255|nullable',
+            'transactions.*.bunq_payment_id'       => 'min:1|max:255|nullable',
-            'transactions.*.external_url'          => 'min:1,max:255|nullable|url',
+            'transactions.*.external_url'          => 'min:1|max:255|nullable|url',
 
             // SEPA fields:
-            'transactions.*.sepa_cc'               => 'min:1,max:255|nullable',
+            'transactions.*.sepa_cc'               => 'min:1|max:255|nullable',
-            'transactions.*.sepa_ct_op'            => 'min:1,max:255|nullable',
+            'transactions.*.sepa_ct_op'            => 'min:1|max:255|nullable',
-            'transactions.*.sepa_ct_id'            => 'min:1,max:255|nullable',
+            'transactions.*.sepa_ct_id'            => 'min:1|max:255|nullable',
-            'transactions.*.sepa_db'               => 'min:1,max:255|nullable',
+            'transactions.*.sepa_db'               => 'min:1|max:255|nullable',
-            'transactions.*.sepa_country'          => 'min:1,max:255|nullable',
+            'transactions.*.sepa_country'          => 'min:1|max:255|nullable',
-            'transactions.*.sepa_ep'               => 'min:1,max:255|nullable',
+            'transactions.*.sepa_ep'               => 'min:1|max:255|nullable',
-            'transactions.*.sepa_ci'               => 'min:1,max:255|nullable',
+            'transactions.*.sepa_ci'               => 'min:1|max:255|nullable',
-            'transactions.*.sepa_batch_id'         => 'min:1,max:255|nullable',
+            'transactions.*.sepa_batch_id'         => 'min:1|max:255|nullable',
 
             // dates
             'transactions.*.interest_date'         => 'date|nullable',
@@ -270,7 +270,9 @@ class StoreRequest extends FormRequest
                 $this->validateTransactionArray($validator);
 
                 // must submit at least one transaction.
+                Log::debug('Now going to validateOneTransaction');
                 $this->validateOneTransaction($validator);
+                Log::debug('Now done with validateOneTransaction');
 
                 // all journals must have a description
                 $this->validateDescriptions($validator);

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

@@ -24,6 +24,7 @@ declare(strict_types=1);
 
 namespace FireflyIII\Api\V1\Requests\Models\Transaction;
 
+use FireflyIII\Exceptions\FireflyException;
 use FireflyIII\Models\TransactionGroup;
 use FireflyIII\Rules\BelongsUser;
 use FireflyIII\Rules\IsBoolean;
@@ -61,6 +62,7 @@ class UpdateRequest extends FormRequest
      */
     public function getAll(): array
     {
+
         Log::debug(sprintf('Now in %s', __METHOD__));
         $this->integerFields = [
             'order',
@@ -163,6 +165,9 @@ class UpdateRequest extends FormRequest
 
         /** @var array $transaction */
         foreach ($this->get('transactions') as $transaction) {
+            if(!is_array($transaction)) {
+                throw new FireflyException('Invalid data submitted: transaction is not array.');
+            }
             // default response is to update nothing in the transaction:
             $current  = [];
             $current  = $this->getIntegerData($current, $transaction);
@@ -330,9 +335,9 @@ class UpdateRequest extends FormRequest
 
             // currency info
             'transactions.*.currency_id'            => 'numeric|exists:transaction_currencies,id',
-            'transactions.*.currency_code'          => 'min:3|max:3|exists:transaction_currencies,code',
+            'transactions.*.currency_code'          => 'min:3|max:51|exists:transaction_currencies,code',
             'transactions.*.foreign_currency_id'    => 'nullable|numeric|exists:transaction_currencies,id',
-            'transactions.*.foreign_currency_code'  => 'nullable|min:3|max:3|exists:transaction_currencies,code',
+            'transactions.*.foreign_currency_code'  => 'nullable|min:3|max:51|exists:transaction_currencies,code',
 
             // amount
             'transactions.*.amount'                 => 'numeric|gt:0|max:100000000000',
@@ -359,25 +364,25 @@ class UpdateRequest extends FormRequest
 
             // other interesting fields
             'transactions.*.reconciled'             => [new IsBoolean()],
-            'transactions.*.notes'                  => 'min:1,max:50000|nullable',
+            'transactions.*.notes'                  => 'min:1|max:50000|nullable',
             'transactions.*.tags'                   => 'between:0,255',
 
             // meta info fields
-            'transactions.*.internal_reference'     => 'min:1,max:255|nullable',
+            'transactions.*.internal_reference'     => 'min:1|max:255|nullable',
-            'transactions.*.external_id'            => 'min:1,max:255|nullable',
+            'transactions.*.external_id'            => 'min:1|max:255|nullable',
-            'transactions.*.recurrence_id'          => 'min:1,max:255|nullable',
+            'transactions.*.recurrence_id'          => 'min:1|max:255|nullable',
-            'transactions.*.bunq_payment_id'        => 'min:1,max:255|nullable',
+            'transactions.*.bunq_payment_id'        => 'min:1|max:255|nullable',
-            'transactions.*.external_url'           => 'min:1,max:255|nullable|url',
+            'transactions.*.external_url'           => 'min:1|max:255|nullable|url',
 
             // SEPA fields:
-            'transactions.*.sepa_cc'                => 'min:1,max:255|nullable',
+            'transactions.*.sepa_cc'                => 'min:1|max:255|nullable',
-            'transactions.*.sepa_ct_op'             => 'min:1,max:255|nullable',
+            'transactions.*.sepa_ct_op'             => 'min:1|max:255|nullable',
-            'transactions.*.sepa_ct_id'             => 'min:1,max:255|nullable',
+            'transactions.*.sepa_ct_id'             => 'min:1|max:255|nullable',
-            'transactions.*.sepa_db'                => 'min:1,max:255|nullable',
+            'transactions.*.sepa_db'                => 'min:1|max:255|nullable',
-            'transactions.*.sepa_country'           => 'min:1,max:255|nullable',
+            'transactions.*.sepa_country'           => 'min:1|max:255|nullable',
-            'transactions.*.sepa_ep'                => 'min:1,max:255|nullable',
+            'transactions.*.sepa_ep'                => 'min:1|max:255|nullable',
-            'transactions.*.sepa_ci'                => 'min:1,max:255|nullable',
+            'transactions.*.sepa_ci'                => 'min:1|max:255|nullable',
-            'transactions.*.sepa_batch_id'          => 'min:1,max:255|nullable',
+            'transactions.*.sepa_batch_id'          => 'min:1|max:255|nullable',
 
             // dates
             'transactions.*.interest_date'          => 'date|nullable',
@@ -398,6 +403,7 @@ class UpdateRequest extends FormRequest
      */
     public function withValidator(Validator $validator): void
     {
+        Log::debug('Now in withValidator');
         /** @var TransactionGroup $transactionGroup */
         $transactionGroup = $this->route()->parameter('transactionGroup');
         $validator->after(

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

@@ -75,7 +75,7 @@ class StoreRequest extends FormRequest
             'name'           => 'required|between:1,255|unique:transaction_currencies,name',
             'code'           => 'required|between:3,51|unique:transaction_currencies,code',
             'symbol'         => 'required|between:1,51|unique:transaction_currencies,symbol',
-            'decimal_places' => 'between:0,20|numeric|min:0|max:20',
+            'decimal_places' => 'between:0,20|numeric|min:0|max:12',
             'enabled'        => [new IsBoolean()],
             'default'        => [new IsBoolean()],
 

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

@@ -74,7 +74,7 @@ class UpdateRequest extends FormRequest
             'name'           => sprintf('between:1,255|unique:transaction_currencies,name,%d', $currency->id),
             'code'           => sprintf('between:3,51|unique:transaction_currencies,code,%d', $currency->id),
             'symbol'         => sprintf('between:1,51|unique:transaction_currencies,symbol,%d', $currency->id),
-            'decimal_places' => 'between:0,20|numeric|min:0|max:20',
+            'decimal_places' => 'between:0,20|numeric|min:0|max:12',
             'enabled'        => [new IsBoolean()],
             'default'        => [new IsBoolean()],
         ];

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

@@ -59,9 +59,9 @@ class StoreRequest extends FormRequest
     public function rules(): array
     {
         return [
-            'name'    => 'required|unique:link_types,name|min:1',
+            'name'    => 'required|unique:link_types,name|min:1|max:1024',
-            'outward' => 'required|unique:link_types,outward|min:1|different:inward',
+            'outward' => 'required|unique:link_types,outward|min:1|different:inward|max:1024',
-            'inward'  => 'required|unique:link_types,inward|min:1|different:outward',
+            'inward'  => 'required|unique:link_types,inward|min:1|different:outward|max:1024',
         ];
     }
 }

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

@@ -64,9 +64,9 @@ class UpdateRequest extends FormRequest
         $linkType = $this->route()->parameter('linkType');
 
         return [
-            'name'    => [Rule::unique('link_types', 'name')->ignore($linkType->id), 'min:1'],
+            'name'    => [Rule::unique('link_types', 'name')->ignore($linkType->id), 'min:1','max:1024'],
-            'outward' => ['different:inward', Rule::unique('link_types', 'outward')->ignore($linkType->id), 'min:1'],
+            'outward' => ['different:inward', Rule::unique('link_types', 'outward')->ignore($linkType->id), 'min:1','max:1024'],
-            'inward'  => ['different:outward', Rule::unique('link_types', 'inward')->ignore($linkType->id), 'min:1'],
+            'inward'  => ['different:outward', Rule::unique('link_types', 'inward')->ignore($linkType->id), 'min:1','max:1024'],
         ];
     }
 }

app/Console/Commands/Correction/CorrectAmounts.php

@@ -0,0 +1,263 @@
+<?php
+/*
+ * CorrectAmounts.php
+ * Copyright (c) 2023 james@firefly-iii.org
+ *
+ * This file is part of Firefly III (https://github.com/firefly-iii).
+ *
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU Affero General Public License as
+ * published by the Free Software Foundation, either version 3 of the
+ * License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU Affero General Public License for more details.
+ *
+ * You should have received a copy of the GNU Affero General Public License
+ * along with this program.  If not, see <https://www.gnu.org/licenses/>.
+ */
+
+declare(strict_types=1);
+
+namespace FireflyIII\Console\Commands\Correction;
+
+use FireflyIII\Models\AutoBudget;
+use FireflyIII\Models\AvailableBudget;
+use FireflyIII\Models\Bill;
+use FireflyIII\Models\BudgetLimit;
+use FireflyIII\Models\CurrencyExchangeRate;
+use FireflyIII\Models\PiggyBank;
+use FireflyIII\Models\PiggyBankRepetition;
+use FireflyIII\Models\RecurrenceTransaction;
+use FireflyIII\Models\RuleTrigger;
+use Illuminate\Console\Command;
+
+/**
+ * Class ReportSkeleton
+ */
+class CorrectAmounts extends Command
+{
+    /**
+     * The console command description.
+     *
+     * @var string
+     */
+    protected $description = 'This command makes sure positive and negative amounts are recorded correctly.';
+    /**
+     * The name and signature of the console command.
+     *
+     * @var string
+     */
+    protected $signature = 'firefly-iii:fix-amount-pos-neg';
+
+    /**
+     * Execute the console command.
+     *
+     * @return int
+     */
+    public function handle(): int
+    {
+        // auto budgets must be positive
+        $this->fixAutoBudgets();
+        // available budgets must be positive
+        $this->fixAvailableBudgets();
+        // bills must be positive (both amounts)
+        $this->fixBills();
+        // budget limits must be positive
+        $this->fixBudgetLimits();
+        // currency_exchange_rates must be positive
+        $this->fixExchangeRates();
+        // piggy_bank_repetitions must be positive
+        $this->fixRepetitions();
+        // piggy_banks must be positive
+        $this->fixPiggyBanks();
+        // recurrences_transactions amount must be positive
+        $this->fixRecurrences();
+        // rule_triggers must be positive or zero (amount_less, amount_more, amount_is)
+        $this->fixRuleTriggers();
+
+
+        return 0;
+    }
+
+    /**
+     * @return void
+     */
+    private function fixAutoBudgets(): void
+    {
+        $set   = AutoBudget::where('amount', '<', 0)->get();
+        $count = $set->count();
+        if (0 === $count) {
+            $this->info('Correct: All auto budget amounts are positive.');
+            return;
+        }
+        /** @var AutoBudget $item */
+        foreach ($set as $item) {
+            $item->amount = app('steam')->positive((string)$item->amount);
+            $item->save();
+        }
+        $this->line(sprintf('Corrected %d auto budget amount(s).', $count));
+    }
+
+    /**
+     * @return void
+     */
+    private function fixAvailableBudgets(): void
+    {
+        $set   = AvailableBudget::where('amount', '<', 0)->get();
+        $count = $set->count();
+        if (0 === $count) {
+            $this->info('Correct: All available budget amounts are positive.');
+            return;
+        }
+        /** @var AvailableBudget $item */
+        foreach ($set as $item) {
+            $item->amount = app('steam')->positive((string)$item->amount);
+            $item->save();
+        }
+        $this->line(sprintf('Corrected %d available budget amount(s).', $count));
+    }
+
+    /**
+     * @return void
+     */
+    private function fixBills(): void
+    {
+        $set   = Bill::where('amount_min', '<', 0)->orWhere('amount_max', '<', 0)->get();
+        $count = $set->count();
+        if (0 === $count) {
+            $this->info('Correct: All bill amounts are positive.');
+            return;
+        }
+        /** @var Bill $item */
+        foreach ($set as $item) {
+            $item->amount_min = app('steam')->positive((string)$item->amount_min);
+            $item->amount_max = app('steam')->positive((string)$item->amount_max);
+            $item->save();
+        }
+    }
+
+    /**
+     * @return void
+     */
+    private function fixBudgetLimits(): void
+    {
+        $set   = BudgetLimit::where('amount', '<', 0)->get();
+        $count = $set->count();
+        if (0 === $count) {
+            $this->info('Correct: All budget limit amounts are positive.');
+            return;
+        }
+        /** @var BudgetLimit $item */
+        foreach ($set as $item) {
+            $item->amount = app('steam')->positive((string)$item->amount);
+            $item->save();
+        }
+        $this->line(sprintf('Corrected %d budget limit amount(s).', $count));
+    }
+
+    /**
+     * @return void
+     */
+    private function fixExchangeRates(): void
+    {
+        $set   = CurrencyExchangeRate::where('rate', '<', 0)->get();
+        $count = $set->count();
+        if (0 === $count) {
+            $this->info('Correct: All currency exchange rates are positive.');
+            return;
+        }
+        /** @var BudgetLimit $item */
+        foreach ($set as $item) {
+            $item->rate = app('steam')->positive((string)$item->rate);
+            $item->save();
+        }
+        $this->line(sprintf('Corrected %d currency exchange rate(s).', $count));
+    }
+
+    /**
+     * @return void
+     */
+    private function fixRepetitions(): void
+    {
+        $set   = PiggyBankRepetition::where('currentamount', '<', 0)->get();
+        $count = $set->count();
+        if (0 === $count) {
+            $this->info('Correct: All piggy bank repetition amounts are positive.');
+            return;
+        }
+        /** @var PiggyBankRepetition $item */
+        foreach ($set as $item) {
+            $item->currentamount = app('steam')->positive((string)$item->currentamount);
+            $item->save();
+        }
+        $this->line(sprintf('Corrected %d piggy bank repetition amount(s).', $count));
+    }
+
+    /**
+     * @return void
+     */
+    private function fixPiggyBanks(): void
+    {
+        $set   = PiggyBank::where('targetamount', '<', 0)->get();
+        $count = $set->count();
+        if (0 === $count) {
+            $this->info('Correct: All piggy bank amounts are positive.');
+            return;
+        }
+        /** @var PiggyBankRepetition $item */
+        foreach ($set as $item) {
+            $item->targetamount = app('steam')->positive((string)$item->targetamount);
+            $item->save();
+        }
+        $this->line(sprintf('Corrected %d piggy bank amount(s).', $count));
+    }
+
+    /**
+     * @return void
+     */
+    private function fixRecurrences(): void
+    {
+        $set   = RecurrenceTransaction::where('amount', '<', 0)
+                                      ->orWhere('foreign_amount', '<', 0)
+                                      ->get();
+        $count = $set->count();
+        if (0 === $count) {
+            $this->info('Correct: All recurring transaction amounts are positive.');
+            return;
+        }
+        /** @var PiggyBankRepetition $item */
+        foreach ($set as $item) {
+            $item->amount         = app('steam')->positive((string)$item->amount);
+            $item->foreign_amount = app('steam')->positive((string)$item->foreign_amount);
+            $item->save();
+        }
+        $this->line(sprintf('Corrected %d recurring transaction amount(s).', $count));
+    }
+
+    /**
+     * @return void
+     */
+    private function fixRuleTriggers(): void
+    {
+        $set   = RuleTrigger::whereIn('trigger_type', ['amount_less', 'amount_more', 'amount_is'])->get();
+        $fixed = 0;
+        /** @var RuleTrigger $item */
+        foreach ($set as $item) {
+            // basic check:
+            if (-1 === bccomp((string)$item->trigger_value, '0')) {
+                $fixed++;
+                $item->trigger_value = app('steam')->positive((string)$item->trigger_value);
+                $item->save();
+            }
+        }
+        if (0 === $fixed) {
+            $this->info('Correct: All rule trigger amounts are positive.');
+            return;
+        }
+        $this->line(sprintf('Corrected %d rule trigger amount(s).', $fixed));
+    }
+
+}

app/Console/Commands/Correction/CorrectDatabase.php

@@ -52,8 +52,10 @@ class CorrectDatabase extends Command
      */
     public function handle(): int
     {
+        $this->line('Handle Firefly III database correction commands.');
         // if table does not exist, return false
         if (!Schema::hasTable('users')) {
+            $this->error('No "users"-table, will not continue.');
             return 1;
         }
         $commands = [
@@ -61,7 +63,7 @@ class CorrectDatabase extends Command
             'firefly-iii:create-link-types',
             'firefly-iii:create-access-tokens',
             'firefly-iii:remove-bills',
-            'firefly-iii:fix-negative-limits',
+            'firefly-iii:fix-amount-pos-neg',
             'firefly-iii:enable-currencies',
             'firefly-iii:fix-transfer-budgets',
             'firefly-iii:fix-uneven-amount',
@@ -76,16 +78,16 @@ class CorrectDatabase extends Command
             'firefly-iii:fix-ob-currencies',
             'firefly-iii:fix-long-descriptions',
             'firefly-iii:fix-recurring-transactions',
-            'firefly-iii:restore-oauth-keys',
             'firefly-iii:upgrade-group-information',
             'firefly-iii:fix-transaction-types',
             'firefly-iii:fix-frontpage-accounts',
+            // new!
+            'firefly-iii:unify-group-accounts',
+            'firefly-iii:trigger-credit-recalculation'
         ];
         foreach ($commands as $command) {
-            $this->line(sprintf('Now executing %s', $command));
+            $this->line(sprintf('Now executing command "%s"', $command));
-            Artisan::call($command);
+            $this->call($command);
-            $result = Artisan::output();
-            echo $result;
         }
 
         return 0;

app/Console/Commands/Correction/FixBudgetLimits.php

@@ -1,67 +0,0 @@
-<?php
-
-/*
- * FixBudgetLimits.php
- * Copyright (c) 2022 james@firefly-iii.org
- *
- * This file is part of Firefly III (https://github.com/firefly-iii).
- *
- * This program is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Affero General Public License as
- * published by the Free Software Foundation, either version 3 of the
- * License, or (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Affero General Public License for more details.
- *
- * You should have received a copy of the GNU Affero General Public License
- * along with this program.  If not, see <https://www.gnu.org/licenses/>.
- */
-
-declare(strict_types=1);
-
-namespace FireflyIII\Console\Commands\Correction;
-
-use DB;
-use FireflyIII\Models\BudgetLimit;
-use Illuminate\Console\Command;
-
-/**
- * Class CorrectionSkeleton
- */
-class FixBudgetLimits extends Command
-{
-    /**
-     * The console command description.
-     *
-     * @var string
-     */
-    protected $description = 'Fixes negative budget limits';
-    /**
-     * The name and signature of the console command.
-     *
-     * @var string
-     */
-    protected $signature = 'firefly-iii:fix-negative-limits';
-
-    /**
-     * Execute the console command.
-     *
-     * @return int
-     */
-    public function handle(): int
-    {
-        $set = BudgetLimit::where('amount', '<', '0')->get();
-        if (0 === $set->count()) {
-            $this->info('All budget limits are OK.');
-            return 0;
-        }
-        $count = BudgetLimit::where('amount', '<', '0')->update(['amount' => DB::raw('amount * -1')]);
-
-        $this->info(sprintf('Fixed %d budget limit(s)', $count));
-
-        return 0;
-    }
-}

app/Console/Commands/Correction/FixIbans.php

@@ -25,7 +25,9 @@ declare(strict_types=1);
 namespace FireflyIII\Console\Commands\Correction;
 
 use FireflyIII\Models\Account;
+use FireflyIII\Models\AccountType;
 use Illuminate\Console\Command;
+use Illuminate\Support\Collection;
 
 /**
  * Class FixIbans
@@ -53,6 +55,52 @@ class FixIbans extends Command
     public function handle(): int
     {
         $accounts = Account::whereNotNull('iban')->get();
+        $this->filterIbans($accounts);
+        $this->countAndCorrectIbans($accounts);
+
+        return 0;
+    }
+
+    /**
+     * @param  Collection  $accounts
+     * @return void
+     */
+    private function countAndCorrectIbans(Collection $accounts): void
+    {
+        $set = [];
+        /** @var Account $account */
+        foreach($accounts as $account) {
+            $userId = (int)$account->user_id;
+            $set[$userId] = $set[$userId] ?? [];
+            $iban = (string)$account->iban;
+            $type = $account->accountType->type;
+            if(in_array($type, [AccountType::LOAN, AccountType::DEBT, AccountType::MORTGAGE], true)) {
+                $type = 'liabilities';
+            }
+            if(array_key_exists($iban, $set[$userId])) {
+                // iban already in use! two exceptions exist:
+                if(
+                    !(AccountType::EXPENSE === $set[$userId][$iban] && AccountType::REVENUE === $type) && // allowed combination
+                    !(AccountType::REVENUE === $set[$userId][$iban] && AccountType::EXPENSE === $type) // also allowed combination.
+                ) {
+                    $this->line(sprintf('IBAN "%s" is used more than once and will be removed from %s #%d ("%s")', $iban, $account->accountType->type, $account->id, $account->name));
+                    $account->iban = null;
+                    $account->save();
+                }
+            }
+
+            if(!array_key_exists($iban, $set[$userId])) {
+                $set[$userId][$iban] = $type;
+            }
+        }
+    }
+
+    /**
+     * @param  Collection  $accounts
+     * @return void
+     */
+    private function filterIbans(Collection $accounts): void
+    {
         /** @var Account $account */
         foreach ($accounts as $account) {
             $iban = $account->iban;
@@ -65,7 +113,5 @@ class FixIbans extends Command
                 }
             }
         }
-
-        return 0;
     }
 }

app/Console/Commands/Integrity/ReportIntegrity.php

@@ -57,14 +57,15 @@ class ReportIntegrity extends Command
             return 1;
         }
         $commands = [
+            'firefly-iii:create-group-memberships',
             'firefly-iii:report-empty-objects',
             'firefly-iii:report-sum',
+            'firefly-iii:restore-oauth-keys',
+            'firefly-iii:upgrade-group-information'
         ];
         foreach ($commands as $command) {
             $this->line(sprintf('Now executing %s', $command));
-            Artisan::call($command);
+            $this->call($command);
-            $result = Artisan::output();
-            echo $result;
         }
 
         return 0;

app/Console/Commands/System/CreateDatabase.php

@@ -1,8 +1,8 @@
 <?php
 
-/**
+/*
  * CreateDatabase.php
- * Copyright (c) 2020 james@firefly-iii.org
+ * Copyright (c) 2023 james@firefly-iii.org
  *
  * This file is part of Firefly III (https://github.com/firefly-iii).
  *
@@ -22,7 +22,7 @@
 
 declare(strict_types=1);
 
-namespace FireflyIII\Console\Commands;
+namespace FireflyIII\Console\Commands\System;
 
 use Illuminate\Console\Command;
 use PDO;

app/Console/Commands/System/CreateFirstUser.php

@@ -2,7 +2,7 @@
 
 /*
  * CreateFirstUser.php
- * Copyright (c) 2021 james@firefly-iii.org
+ * Copyright (c) 2023 james@firefly-iii.org
  *
  * This file is part of Firefly III (https://github.com/firefly-iii).
  *
@@ -22,7 +22,7 @@
 
 declare(strict_types=1);
 
-namespace FireflyIII\Console\Commands;
+namespace FireflyIII\Console\Commands\System;
 
 use FireflyIII\Repositories\User\UserRepositoryInterface;
 use Illuminate\Console\Command;

app/Console/Commands/System/ForceDecimalSize.php

@@ -0,0 +1,107 @@
+<?php
+/*
+ * ForceDecimalSize.php
+ * Copyright (c) 2023 james@firefly-iii.org
+ *
+ * This file is part of Firefly III (https://github.com/firefly-iii).
+ *
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU Affero General Public License as
+ * published by the Free Software Foundation, either version 3 of the
+ * License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU Affero General Public License for more details.
+ *
+ * You should have received a copy of the GNU Affero General Public License
+ * along with this program.  If not, see <https://www.gnu.org/licenses/>.
+ */
+
+declare(strict_types=1);
+
+namespace FireflyIII\Console\Commands\System;
+
+use FireflyIII\Console\Commands\VerifiesAccessToken;
+use FireflyIII\Exceptions\FireflyException;
+use Illuminate\Console\Command;
+use Illuminate\Support\Facades\DB;
+use Illuminate\Support\Facades\Log;
+
+class ForceDecimalSize extends Command
+{
+    use VerifiesAccessToken;
+    /**
+     * The name and signature of the console command.
+     *
+     * @var string
+     */
+    protected $signature = 'firefly-iii:force-decimal-size
+                                {--user=1 : The user ID.}
+                            {--token= : The user\'s access token.}';
+
+    /**
+     * The console command description.
+     *
+     * @var string
+     */
+    protected $description = 'This command resizes DECIMAL columns in MySQL or PostgreSQL.';
+
+    /**
+     * Execute the console command.
+     * @throws FireflyException
+     */
+    public function handle(): int
+    {
+        if (!$this->verifyAccessToken()) {
+            $this->error('Invalid access token.');
+
+            return 1;
+        }
+
+        $this->error('Running this command is dangerous and can cause data loss.');
+        $this->error('Please do not continue.');
+        $question = $this->confirm('Do you want to continue?');
+        if (true === $question) {
+            $user = $this->getUser();
+            Log::channel('audit')->info(sprintf('User #%d ("%s") forced DECIMAL size.', $user->id, $user->email));
+            $this->updateDecimals();
+            return 0;
+        }
+        $this->line('Done!');
+        return 0;
+    }
+
+    private function updateDecimals(): void
+    {
+        $this->info('Going to force the size of DECIMAL columns. Please hold.');
+        $tables = [
+            'accounts'                 => ['virtual_balance'],
+            'auto_budgets'             => ['amount'],
+            'available_budgets'        => ['amount'],
+            'bills'                    => ['amount_min', 'amount_max'],
+            'budget_limits'            => ['amount'],
+            'currency_exchange_rates'  => ['rate', 'user_rate'],
+            'limit_repetitions'        => ['amount'],
+            'piggy_bank_events'        => ['amount'],
+            'piggy_bank_repetitions'   => ['currentamount'],
+            'piggy_banks'              => ['targetamount'],
+            'recurrences_transactions' => ['amount', 'foreign_amount'],
+            'transactions'             => ['amount', 'foreign_amount'],
+        ];
+        /**
+         * @var string $name
+         * @var array $fields
+         */
+        foreach($tables as $name => $fields) {
+            /** @var string $field */
+            foreach($fields as $field) {
+                $this->line(sprintf('Updating table "%s", field "%s"...', $name, $field));
+                $query = sprintf('ALTER TABLE %s CHANGE COLUMN %s %s DECIMAL(32, 12);', $name, $field, $field);
+                DB::select($query);
+                sleep(1);
+            }
+        }
+    }
+}

app/Console/Commands/System/ForceMigration.php

@@ -1,9 +1,29 @@
 <?php
+/*
+ * ForceMigration.php
+ * Copyright (c) 2023 james@firefly-iii.org
+ *
+ * This file is part of Firefly III (https://github.com/firefly-iii).
+ *
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU Affero General Public License as
+ * published by the Free Software Foundation, either version 3 of the
+ * License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU Affero General Public License for more details.
+ *
+ * You should have received a copy of the GNU Affero General Public License
+ * along with this program.  If not, see <https://www.gnu.org/licenses/>.
+ */
 
 declare(strict_types=1);
 
-namespace FireflyIII\Console\Commands;
+namespace FireflyIII\Console\Commands\System;
 
+use FireflyIII\Console\Commands\VerifiesAccessToken;
 use FireflyIII\Exceptions\FireflyException;
 use Illuminate\Console\Command;
 use Illuminate\Support\Facades\Artisan;
@@ -57,6 +77,7 @@ class ForceMigration extends Command
 
     private function forceMigration(): void
     {
+        DB::commit();
         $this->line('Dropping "migrations" table...');
         sleep(2);
         Schema::dropIfExists('migrations');

app/Console/Commands/System/ScanAttachments.php

@@ -1,7 +1,7 @@
 <?php
-/**
+/*
  * ScanAttachments.php
- * Copyright (c) 2020 james@firefly-iii.org
+ * Copyright (c) 2023 james@firefly-iii.org
  *
  * This file is part of Firefly III (https://github.com/firefly-iii).
  *
@@ -21,7 +21,7 @@
 
 declare(strict_types=1);
 
-namespace FireflyIII\Console\Commands;
+namespace FireflyIII\Console\Commands\System;
 
 use Crypt;
 use FireflyIII\Models\Attachment;
@@ -42,7 +42,7 @@ class ScanAttachments extends Command
      *
      * @var string
      */
-    protected $description = 'Rescan all attachments and re-set the MD5 hash and mime.';
+    protected $description = 'Rescan all attachments and re-set the correct MD5 hash and mime.';
 
     /**
      * The name and signature of the console command.

app/Console/Commands/System/SetLatestVersion.php

@@ -1,8 +1,8 @@
 <?php
 
-/**
+/*
  * SetLatestVersion.php
- * Copyright (c) 2020 james@firefly-iii.org
+ * Copyright (c) 2023 james@firefly-iii.org
  *
  * This file is part of Firefly III (https://github.com/firefly-iii).
  *
@@ -22,7 +22,7 @@
 
 declare(strict_types=1);
 
-namespace FireflyIII\Console\Commands;
+namespace FireflyIII\Console\Commands\System;
 
 use Illuminate\Console\Command;
 

app/Console/Commands/System/UpgradeFireflyInstructions.php

@@ -1,7 +1,7 @@
 <?php
-/**
+/*
  * UpgradeFireflyInstructions.php
- * Copyright (c) 2020 james@firefly-iii.org
+ * Copyright (c) 2023 james@firefly-iii.org
  *
  * This file is part of Firefly III (https://github.com/firefly-iii).
  *
@@ -21,11 +21,13 @@
 
 declare(strict_types=1);
 
-namespace FireflyIII\Console\Commands;
+namespace FireflyIII\Console\Commands\System;
 
 use FireflyIII\Support\System\GeneratesInstallationId;
 use Illuminate\Console\Command;
 
+use function FireflyIII\Console\Commands\str_starts_with;
+
 /**
  * Class UpgradeFireflyInstructions.
  *
@@ -75,14 +77,15 @@ class UpgradeFireflyInstructions extends Command
         $text    = '';
         foreach (array_keys($config) as $compare) {
             // if string starts with:
-            if (str_starts_with($version, $compare)) {
+            if (\str_starts_with($version, $compare)) {
                 $text = $config[$compare];
             }
+
         }
 
         $this->showLine();
         $this->boxed('');
-        if (null === $text) {
+        if (null === $text || '' === $text) {
             $this->boxed(sprintf('Thank you for updating to Firefly III, v%s', $version));
             $this->boxedInfo('There are no extra upgrade instructions.');
             $this->boxed('Firefly III should be ready for use.');
@@ -146,13 +149,13 @@ class UpgradeFireflyInstructions extends Command
         $text    = '';
         foreach (array_keys($config) as $compare) {
             // if string starts with:
-            if (str_starts_with($version, $compare)) {
+            if (\str_starts_with($version, $compare)) {
                 $text = $config[$compare];
             }
         }
         $this->showLine();
         $this->boxed('');
-        if (null === $text) {
+        if (null === $text || '' === $text) {
             $this->boxed(sprintf('Thank you for installing Firefly III, v%s!', $version));
             $this->boxedInfo('There are no extra installation instructions.');
             $this->boxed('Firefly III should be ready for use.');

app/Console/Commands/System/VerifySecurityAlerts.php

@@ -2,7 +2,7 @@
 
 /*
  * VerifySecurityAlerts.php
- * Copyright (c) 2021 james@firefly-iii.org
+ * Copyright (c) 2023 james@firefly-iii.org
  *
  * This file is part of Firefly III (https://github.com/firefly-iii).
  *
@@ -22,12 +22,12 @@
 
 declare(strict_types=1);
 
-namespace FireflyIII\Console\Commands;
+namespace FireflyIII\Console\Commands\System;
 
 use Illuminate\Console\Command;
 use Illuminate\Database\QueryException;
-use League\Flysystem\FilesystemException;
 use Illuminate\Support\Facades\Log;
+use League\Flysystem\FilesystemException;
 use Storage;
 
 /**

app/Console/Commands/Upgrade/DecryptDatabase.php

@@ -1,8 +1,8 @@
 <?php
 
-/**
+/*
  * DecryptDatabase.php
- * Copyright (c) 2020 james@firefly-iii.org
+ * Copyright (c) 2023 james@firefly-iii.org
  *
  * This file is part of Firefly III (https://github.com/firefly-iii).
  *
@@ -22,7 +22,7 @@
 
 declare(strict_types=1);
 
-namespace FireflyIII\Console\Commands;
+namespace FireflyIII\Console\Commands\Upgrade;
 
 use Crypt;
 use DB;
@@ -30,8 +30,8 @@ use FireflyIII\Exceptions\FireflyException;
 use FireflyIII\Models\Preference;
 use Illuminate\Console\Command;
 use Illuminate\Contracts\Encryption\DecryptException;
-use JsonException;
 use Illuminate\Support\Facades\Log;
+use JsonException;
 use Psr\Container\ContainerExceptionInterface;
 use Psr\Container\NotFoundExceptionInterface;
 use stdClass;

app/Console/Commands/Upgrade/FixPostgresSequences.php

@@ -2,7 +2,7 @@
 
 /*
  * FixPostgresSequences.php
- * Copyright (c) 2021 james@firefly-iii.org
+ * Copyright (c) 2023 james@firefly-iii.org
  *
  * This file is part of Firefly III (https://github.com/firefly-iii).
  *
@@ -22,7 +22,7 @@
 
 declare(strict_types=1);
 
-namespace FireflyIII\Console\Commands\Correction;
+namespace FireflyIII\Console\Commands\Upgrade;
 
 use DB;
 use Illuminate\Console\Command;
@@ -76,10 +76,8 @@ class FixPostgresSequences extends Command
             'category_transaction_journal',
             'configuration',
             'currency_exchange_rates',
-            'export_jobs',
             'failed_jobs',
             'group_journals',
-            'import_jobs',
             'jobs',
             'journal_links',
             'journal_meta',

app/Console/Commands/Upgrade/UpgradeDatabase.php

@@ -57,7 +57,6 @@ class UpgradeDatabase extends Command
     {
         $this->callInitialCommands();
         $commands = [
-            // there are 14 upgrade commands.
             'firefly-iii:transaction-identifiers',
             'firefly-iii:migrate-to-groups',
             'firefly-iii:account-currencies',
@@ -75,41 +74,7 @@ class UpgradeDatabase extends Command
             'firefly-iii:migrate-recurrence-type',
             'firefly-iii:upgrade-liabilities',
             'firefly-iii:liabilities-600',
-
+            'firefly-iii:budget-limit-periods',
-            // there are 16 verify commands.
-            'firefly-iii:fix-piggies',
-            'firefly-iii:create-link-types',
-            'firefly-iii:create-access-tokens',
-            'firefly-iii:remove-bills',
-            'firefly-iii:fix-negative-limits',
-            'firefly-iii:enable-currencies',
-            'firefly-iii:fix-transfer-budgets',
-            'firefly-iii:fix-uneven-amount',
-            'firefly-iii:delete-zero-amount',
-            'firefly-iii:delete-orphaned-transactions',
-            'firefly-iii:delete-empty-journals',
-            'firefly-iii:delete-empty-groups',
-            'firefly-iii:fix-account-types',
-            'firefly-iii:fix-account-order',
-            'firefly-iii:rename-meta-fields',
-            'firefly-iii:fix-ob-currencies',
-            'firefly-iii:fix-long-descriptions',
-            'firefly-iii:fix-recurring-transactions',
-            'firefly-iii:unify-group-accounts',
-            'firefly-iii:fix-transaction-types',
-            'firefly-iii:fix-frontpage-accounts',
-            'firefly-iii:fix-ibans',
-            'firefly-iii:create-group-memberships',
-            'firefly-iii:upgrade-group-information',
-
-            // two report commands
-            'firefly-iii:report-empty-objects',
-            'firefly-iii:report-sum',
-            'firefly-iii:restore-oauth-keys',
-
-            // instructions
-            'firefly:instructions update',
-            'firefly-iii:verify-security-alerts',
         ];
         $args     = [];
         if ($this->option('force')) {
@@ -117,9 +82,7 @@ class UpgradeDatabase extends Command
         }
         foreach ($commands as $command) {
             $this->line(sprintf('Now executing %s', $command));
-            Artisan::call($command, $args);
+            $this->call($command, $args);
-            $result = Artisan::output();
-            echo $result;
         }
         // set new DB version.
         app('fireflyconfig')->set('db_version', (int)config('firefly.db_version'));
@@ -129,22 +92,19 @@ class UpgradeDatabase extends Command
         return 0;
     }
 
+    /**
+     * @return void
+     */
     private function callInitialCommands(): void
     {
         $this->line('Now seeding the database...');
-        Artisan::call('migrate', ['--seed' => true, '--force' => true]);
+        $this->call('migrate', ['--seed' => true, '--force' => true,'--no-interaction' => true]);
-        $result = Artisan::output();
-        echo $result;
 
         $this->line('Fix PostgreSQL sequences.');
-        Artisan::call('firefly-iii:fix-pgsql-sequences');
+        $this->call('firefly-iii:fix-pgsql-sequences');
-        $result = Artisan::output();
-        echo $result;
 
         $this->line('Now decrypting the database (if necessary)...');
-        Artisan::call('firefly-iii:decrypt-all');
+        $this->call('firefly-iii:decrypt-all');
-        $result = Artisan::output();
-        echo $result;
 
         $this->line('Done!');
     }

app/Events/Model/BudgetLimit/Created.php

@@ -0,0 +1,46 @@
+<?php
+
+declare(strict_types=1);
+/*
+ * Created.php
+ * Copyright (c) 2023 james@firefly-iii.org
+ *
+ * This file is part of Firefly III (https://github.com/firefly-iii).
+ *
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU Affero General Public License as
+ * published by the Free Software Foundation, either version 3 of the
+ * License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU Affero General Public License for more details.
+ *
+ * You should have received a copy of the GNU Affero General Public License
+ * along with this program.  If not, see <https://www.gnu.org/licenses/>.
+ */
+
+namespace FireflyIII\Events\Model\BudgetLimit;
+
+use FireflyIII\Events\Event;
+use FireflyIII\Models\BudgetLimit;
+use Illuminate\Queue\SerializesModels;
+
+/**
+ * Class Created
+ */
+class Created extends Event
+{
+    use SerializesModels;
+
+    public BudgetLimit $budgetLimit;
+
+    /**
+     * @param  BudgetLimit  $budgetLimit
+     */
+    public function __construct(BudgetLimit $budgetLimit)
+    {
+        $this->budgetLimit = $budgetLimit;
+    }
+}

app/Events/Model/BudgetLimit/Deleted.php

@@ -0,0 +1,46 @@
+<?php
+
+declare(strict_types=1);
+/*
+ * Created.php
+ * Copyright (c) 2023 james@firefly-iii.org
+ *
+ * This file is part of Firefly III (https://github.com/firefly-iii).
+ *
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU Affero General Public License as
+ * published by the Free Software Foundation, either version 3 of the
+ * License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU Affero General Public License for more details.
+ *
+ * You should have received a copy of the GNU Affero General Public License
+ * along with this program.  If not, see <https://www.gnu.org/licenses/>.
+ */
+
+namespace FireflyIII\Events\Model\BudgetLimit;
+
+use FireflyIII\Events\Event;
+use FireflyIII\Models\BudgetLimit;
+use Illuminate\Queue\SerializesModels;
+
+/**
+ * Class Deleted
+ */
+class Deleted extends Event
+{
+    use SerializesModels;
+
+    public BudgetLimit $budgetLimit;
+
+    /**
+     * @param  BudgetLimit  $budgetLimit
+     */
+    public function __construct(BudgetLimit $budgetLimit)
+    {
+        $this->budgetLimit = $budgetLimit;
+    }
+}

app/Events/Model/BudgetLimit/Updated.php

@@ -0,0 +1,46 @@
+<?php
+
+declare(strict_types=1);
+/*
+ * Created.php
+ * Copyright (c) 2023 james@firefly-iii.org
+ *
+ * This file is part of Firefly III (https://github.com/firefly-iii).
+ *
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU Affero General Public License as
+ * published by the Free Software Foundation, either version 3 of the
+ * License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU Affero General Public License for more details.
+ *
+ * You should have received a copy of the GNU Affero General Public License
+ * along with this program.  If not, see <https://www.gnu.org/licenses/>.
+ */
+
+namespace FireflyIII\Events\Model\BudgetLimit;
+
+use FireflyIII\Events\Event;
+use FireflyIII\Models\BudgetLimit;
+use Illuminate\Queue\SerializesModels;
+
+/**
+ * Class Updated
+ */
+class Updated extends Event
+{
+    use SerializesModels;
+
+    public BudgetLimit $budgetLimit;
+
+    /**
+     * @param  BudgetLimit  $budgetLimit
+     */
+    public function __construct(BudgetLimit $budgetLimit)
+    {
+        $this->budgetLimit = $budgetLimit;
+    }
+}

app/Generator/Chart/Basic/ChartJsGenerator.php

@@ -122,7 +122,7 @@ class ChartJsGenerator implements GeneratorInterface
 
         foreach ($data as $set) {
             $currentSet = [
-                'label' => $set['label'],
+                'label' => $set['label'] ?? '(no label)',
                 'type'  => $set['type'] ?? 'line',
                 'data'  => array_values($set['entries']),
             ];

app/Handlers/Events/Model/BudgetLimitHandler.php

@@ -0,0 +1,218 @@
+<?php
+
+declare(strict_types=1);
+/*
+ * BudgetLimitHandler.php
+ * Copyright (c) 2023 james@firefly-iii.org
+ *
+ * This file is part of Firefly III (https://github.com/firefly-iii).
+ *
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU Affero General Public License as
+ * published by the Free Software Foundation, either version 3 of the
+ * License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU Affero General Public License for more details.
+ *
+ * You should have received a copy of the GNU Affero General Public License
+ * along with this program.  If not, see <https://www.gnu.org/licenses/>.
+ */
+
+namespace FireflyIII\Handlers\Events\Model;
+
+use FireflyIII\Events\Model\BudgetLimit\Created;
+use FireflyIII\Events\Model\BudgetLimit\Deleted;
+use FireflyIII\Events\Model\BudgetLimit\Updated;
+use FireflyIII\Models\AvailableBudget;
+use FireflyIII\Models\BudgetLimit;
+use FireflyIII\Repositories\Budget\BudgetLimitRepositoryInterface;
+use Illuminate\Support\Facades\Log;
+use Spatie\Period\Boundaries;
+use Spatie\Period\Period;
+use Spatie\Period\Precision;
+
+/**
+ * Class BudgetLimitHandler
+ */
+class BudgetLimitHandler
+{
+    /**
+     * @param  Created  $event
+     * @return void
+     */
+    public function created(Created $event): void
+    {
+        Log::debug(sprintf('BudgetLimitHandler::created(%s)', $event->budgetLimit->id));
+        $this->updateAvailableBudget($event->budgetLimit);
+    }
+
+    /**
+     * @param  Updated  $event
+     * @return void
+     */
+    public function updated(Updated $event): void
+    {
+        Log::debug(sprintf('BudgetLimitHandler::updated(%s)', $event->budgetLimit->id));
+        $this->updateAvailableBudget($event->budgetLimit);
+    }
+
+    /**
+     * @param  Deleted  $event
+     * @return void
+     */
+    public function deleted(Deleted $event): void
+    {
+        Log::debug(sprintf('BudgetLimitHandler::deleted(%s)', $event->budgetLimit->id));
+        $this->updateAvailableBudget($event->budgetLimit);
+    }
+
+    /**
+     * @param  AvailableBudget  $availableBudget
+     * @return void
+     */
+    private function calculateAmount(AvailableBudget $availableBudget): void
+    {
+        $repository = app(BudgetLimitRepositoryInterface::class);
+        $repository->setUser($availableBudget->user);
+        $newAmount = '0';
+        $abPeriod  = Period::make($availableBudget->start_date, $availableBudget->end_date, Precision::DAY());
+        Log::debug(
+            sprintf(
+                'Now at AB #%d, ("%s" to "%s")',
+                $availableBudget->id,
+                $availableBudget->start_date->format('Y-m-d'),
+                $availableBudget->end_date->format('Y-m-d')
+            )
+        );
+        // have to recalc everything just in case.
+        $set = $repository->getAllBudgetLimitsByCurrency($availableBudget->transactionCurrency, $availableBudget->start_date, $availableBudget->end_date);
+        Log::debug(sprintf('Found %d interesting budget limit(s).', $set->count()));
+        /** @var BudgetLimit $budgetLimit */
+        foreach ($set as $budgetLimit) {
+            Log::debug(
+                sprintf(
+                    'Found interesting budget limit #%d ("%s" to "%s")',
+                    $budgetLimit->id,
+                    $budgetLimit->start_date->format('Y-m-d'),
+                    $budgetLimit->end_date->format('Y-m-d')
+                )
+            );
+            // overlap in days:
+            $limitPeriod = Period::make(
+                $budgetLimit->start_date,
+                $budgetLimit->end_date,
+                precision: Precision::DAY(),
+                boundaries: Boundaries::EXCLUDE_NONE()
+            );
+            // if both equal eachother, amount from this BL must be added to the AB
+            if ($limitPeriod->equals($abPeriod)) {
+                $newAmount = bcadd($newAmount, $budgetLimit->amount);
+            }
+            // if budget limit period inside AB period, can be added in full.
+            if (!$limitPeriod->equals($abPeriod) && $abPeriod->contains($limitPeriod)) {
+                $newAmount = bcadd($newAmount, $budgetLimit->amount);
+            }
+            if (!$limitPeriod->equals($abPeriod) && $abPeriod->overlapsWith($limitPeriod)) {
+                $overlap = $abPeriod->overlap($limitPeriod);
+                if (null !== $overlap) {
+                    $length    = $overlap->length();
+                    $daily     = bcmul($this->getDailyAmount($budgetLimit), (string)$length);
+                    $newAmount = bcadd($newAmount, $daily);
+                }
+            }
+        }
+        Log::debug(sprintf('Concluded new amount for this AB must be %s', $newAmount));
+        $availableBudget->amount = $newAmount;
+        $availableBudget->save();
+    }
+
+    /**
+     * @param  BudgetLimit  $budgetLimit
+     * @return string
+     */
+    private function getDailyAmount(BudgetLimit $budgetLimit): string
+    {
+        $limitPeriod = Period::make(
+            $budgetLimit->start_date,
+            $budgetLimit->end_date,
+            precision: Precision::DAY(),
+            boundaries: Boundaries::EXCLUDE_NONE()
+        );
+        $days        = $limitPeriod->length();
+        $amount      = bcdiv((string)$budgetLimit->amount, (string)$days, 12);
+        Log::debug(
+            sprintf('Total amount for budget limit #%d is %s. Nr. of days is %d. Amount per day is %s', $budgetLimit->id, $budgetLimit->amount, $days, $amount)
+        );
+        return $amount;
+    }
+
+    /**
+     * @param  BudgetLimit  $budgetLimit
+     * @return void
+     * @throws \FireflyIII\Exceptions\FireflyException
+     * @throws \Psr\Container\ContainerExceptionInterface
+     * @throws \Psr\Container\NotFoundExceptionInterface
+     */
+    private function updateAvailableBudget(BudgetLimit $budgetLimit): void
+    {
+        Log::debug(sprintf('Now in updateAvailableBudget(#%d)', $budgetLimit->id));
+
+        // based on the view range of the user (month week quarter etc) the budget limit could
+        // either overlap multiple available budget periods or be contained in a single one.
+        // all have to be created or updated.
+        $viewRange = app('preferences')->get('viewRange', '1M')->data;
+        $start     = app('navigation')->startOfPeriod($budgetLimit->start_date, $viewRange);
+        $end       = app('navigation')->startOfPeriod($budgetLimit->end_date, $viewRange);
+        $end       = app('navigation')->endOfPeriod($end, $viewRange);
+        $user      = $budgetLimit->budget->user;
+
+        // limit period in total is:
+        $limitPeriod = Period::make($start, $end, precision: Precision::DAY(), boundaries: Boundaries::EXCLUDE_NONE());
+
+        // from the start until the end of the budget limit, need to loop!
+        $current = clone $start;
+        while ($current <= $end) {
+            $currentEnd = app('navigation')->endOfPeriod($current, $viewRange);
+
+            // create or find AB for this particular period, and set the amount accordingly.
+            /** @var AvailableBudget $availableBudget */
+            $availableBudget = $user->availableBudgets()->where('start_date', $current->format('Y-m-d'))->where(
+                'end_date',
+                $currentEnd->format('Y-m-d')
+            )->where('transaction_currency_id', $budgetLimit->transaction_currency_id)->first();
+            if (null !== $availableBudget) {
+                Log::debug('Found 1 AB, will update.');
+                $this->calculateAmount($availableBudget);
+            }
+            if (null === $availableBudget) {
+                // if not exists:
+                $currentPeriod = Period::make($current, $currentEnd, precision: Precision::DAY(), boundaries: Boundaries::EXCLUDE_NONE());
+                $daily         = $this->getDailyAmount($budgetLimit);
+                $amount        = bcmul($daily, (string)$currentPeriod->length(), 12);
+
+                // no need to calculate if period is equal.
+                if ($currentPeriod->equals($limitPeriod)) {
+                    $amount = $budgetLimit->amount;
+                }
+                Log::debug(sprintf('Will create AB for period %s to %s', $current->format('Y-m-d'), $currentEnd->format('Y-m-d')));
+                $availableBudget = new AvailableBudget(
+                    [
+                        'user_id'                 => $budgetLimit->budget->user->id,
+                        'transaction_currency_id' => $budgetLimit->transaction_currency_id,
+                        'start_date'              => $current,
+                        'end_date'                => $currentEnd,
+                        'amount'                  => $amount,
+                    ]
+                );
+                $availableBudget->save();
+            }
+
+            // prep for next loop
+            $current = app('navigation')->addPeriod($current, $viewRange, 0);
+        }
+    }
+
+}

app/Helpers/Collector/Extensions/AttachmentCollection.php

@@ -92,7 +92,8 @@ trait AttachmentCollection
                         ->where(
                             static function (EloquentBuilder $q1) {
                                 $q1->where('attachments.attachable_type', TransactionJournal::class);
-                                //$q1->where('attachments.uploaded', true);
+                                $q1->where('attachments.uploaded', true);
+                                $q1->whereNull('attachments.deleted_at');
                                 $q1->orWhereNull('attachments.attachable_type');
                             }
                         );

app/Helpers/Update/UpdateTrait.php

@@ -41,7 +41,6 @@ trait UpdateTrait
      * 'level' => 'info' / 'success' / 'error'
      *
      * @return array
-     * @throws FireflyException
      * @throws ContainerExceptionInterface
      * @throws NotFoundExceptionInterface
      */

app/Http/Controllers/Budget/BudgetLimitController.php

@@ -89,12 +89,13 @@ class BudgetLimitController extends Controller
         $collection   = $this->currencyRepos->get();
         $budgetLimits = $this->blRepository->getBudgetLimits($budget, $start, $end);
 
-        // remove already budgeted currencies:
+        // remove already budgeted currencies with the same date range
         $currencies = $collection->filter(
-            static function (TransactionCurrency $currency) use ($budgetLimits) {
+            static function (TransactionCurrency $currency) use ($budgetLimits, $start, $end) {
-                /** @var AvailableBudget $budget */
+                /** @var BudgetLimit $limit */
-                foreach ($budgetLimits as $budget) {
+                foreach ($budgetLimits as $limit) {
-                    if ($budget->transaction_currency_id === $currency->id) {
+                    if ($limit->transaction_currency_id === $currency->id && $limit->start_date->isSameDay($start) && $limit->end_date->isSameDay($end)
+                    ) {
                         return false;
                     }
                 }

app/Http/Controllers/Budget/IndexController.php

@@ -40,9 +40,9 @@ use Illuminate\Contracts\View\Factory;
 use Illuminate\Http\JsonResponse;
 use Illuminate\Http\Request;
 use Illuminate\Support\Collection;
+use Illuminate\Support\Facades\Log;
 use Illuminate\View\View;
 use JsonException;
-use Illuminate\Support\Facades\Log;
 use Psr\Container\ContainerExceptionInterface;
 use Psr\Container\NotFoundExceptionInterface;
 
@@ -101,12 +101,23 @@ class IndexController extends Controller
      */
     public function index(Request $request, Carbon $start = null, Carbon $end = null)
     {
-        Log::debug('Start of IndexController::index()');
+        Log::debug(sprintf('Start of IndexController::index("%s", "%s")', $start?->format('Y-m-d'), $end?->format('Y-m-d')));
 
         // collect some basic vars:
-        $range           = app('navigation')->getViewRange(true);
+        $range         = app('navigation')->getViewRange(true);
-        $start           = $start ?? session('start', today(config('app.timezone'))->startOfMonth());
+        $isCustomRange = session('is_custom_range', false);
-        $end             = $end ?? app('navigation')->endOfPeriod($start, $range);
+        if (false === $isCustomRange) {
+            $start = $start ?? session('start', today(config('app.timezone'))->startOfMonth());
+            $end   = $end ?? app('navigation')->endOfPeriod($start, $range);
+        }
+
+        // overrule start and end if necessary:
+        if (true === $isCustomRange) {
+            $start = $start ?? session('start', today(config('app.timezone'))->startOfMonth());
+            $end   = $end ?? session('end', today(config('app.timezone'))->endOfMonth());
+        }
+
+
         $defaultCurrency = app('amount')->getDefaultCurrency();
         $currencies      = $this->currencyRepository->get();
         $budgeted        = '0';

app/Http/Controllers/Json/BudgetController.php

@@ -82,8 +82,10 @@ class BudgetController extends Controller
         $percentage      = '0';
 
         if (null !== $availableBudget) {
-            $available  = $availableBudget->amount;
+            $available = $availableBudget->amount;
-            $percentage = bcmul(bcdiv($budgeted, $available), '100');
+            if (0 !== bccomp($available, '0')) {
+                $percentage = bcmul(bcdiv($budgeted, $available), '100');
+            }
         }
 
         // if available, get the AB for this period + currency, so the bar can be redrawn.

app/Http/Controllers/System/InstallController.php

@@ -33,9 +33,9 @@ use FireflyIII\Support\Http\Controllers\GetConfigurationData;
 use Illuminate\Contracts\View\Factory;
 use Illuminate\Http\JsonResponse;
 use Illuminate\Http\Request;
+use Illuminate\Support\Facades\Log;
 use Illuminate\View\View;
 use Laravel\Passport\Passport;
-use Illuminate\Support\Facades\Log;
 use phpseclib3\Crypt\RSA;
 
 /**
@@ -60,61 +60,15 @@ class InstallController extends Controller
     {
         // empty on purpose.
         $this->upgradeCommands = [
-            // there are 3 initial commands
+            // there are 5 initial commands
-            'migrate'                                  => ['--seed' => true, '--force' => true],
+            // Check 4 places: InstallController, Docker image, UpgradeDatabase, composer.json
-            'firefly-iii:fix-pgsql-sequences'          => [],
+            'migrate'                            => ['--seed' => true, '--force' => true],
-            'firefly-iii:decrypt-all'                  => [],
+            'generate-keys'                      => [], // an exception :(
-            'firefly-iii:restore-oauth-keys'           => [],
+            'firefly-iii:upgrade-database'       => [],
-            'generate-keys'                            => [], // an exception :(
+            'firefly-iii:correct-database'       => [],
-
+            'firefly-iii:report-integrity'       => [],
-            // upgrade commands
+            'firefly-iii:set-latest-version'     => ['--james-is-cool' => true],
-            'firefly-iii:transaction-identifiers'      => [],
+            'firefly-iii:verify-security-alerts' => [],
-            'firefly-iii:migrate-to-groups'            => [],
-            'firefly-iii:account-currencies'           => [],
-            'firefly-iii:transfer-currencies'          => [],
-            'firefly-iii:other-currencies'             => [],
-            'firefly-iii:migrate-notes'                => [],
-            'firefly-iii:migrate-attachments'          => [],
-            'firefly-iii:bills-to-rules'               => [],
-            'firefly-iii:bl-currency'                  => [],
-            'firefly-iii:cc-liabilities'               => [],
-            'firefly-iii:back-to-journals'             => [],
-            'firefly-iii:rename-account-meta'          => [],
-            'firefly-iii:migrate-recurrence-meta'      => [],
-            'firefly-iii:migrate-tag-locations'        => [],
-            'firefly-iii:migrate-recurrence-type'      => [],
-            'firefly-iii:upgrade-liabilities'          => [],
-            'firefly-iii:liabilities-600'              => [],
-
-            // verify commands
-            'firefly-iii:fix-piggies'                  => [],
-            'firefly-iii:create-link-types'            => [],
-            'firefly-iii:create-access-tokens'         => [],
-            'firefly-iii:remove-bills'                 => [],
-            'firefly-iii:fix-negative-limits'          => [],
-            'firefly-iii:enable-currencies'            => [],
-            'firefly-iii:fix-transfer-budgets'         => [],
-            'firefly-iii:fix-uneven-amount'            => [],
-            'firefly-iii:delete-zero-amount'           => [],
-            'firefly-iii:delete-orphaned-transactions' => [],
-            'firefly-iii:delete-empty-journals'        => [],
-            'firefly-iii:delete-empty-groups'          => [],
-            'firefly-iii:fix-account-types'            => [],
-            'firefly-iii:fix-account-order'            => [],
-            'firefly-iii:rename-meta-fields'           => [],
-            'firefly-iii:fix-ob-currencies'            => [],
-            'firefly-iii:fix-long-descriptions'        => [],
-            'firefly-iii:fix-recurring-transactions'   => [],
-            'firefly-iii:unify-group-accounts'         => [],
-            'firefly-iii:fix-transaction-types'        => [],
-            'firefly-iii:fix-frontpage-accounts'       => [],
-            'firefly-iii:fix-ibans'                    => [],
-            'firefly-iii:create-group-memberships'     => [],
-            'firefly-iii:upgrade-group-information'    => [],
-
-            // final command to set the latest version in DB
-            'firefly-iii:set-latest-version'           => ['--james-is-cool' => true],
-            'firefly-iii:verify-security-alerts'       => [],
         ];
 
         $this->lastError = '';
@@ -127,6 +81,7 @@ class InstallController extends Controller
      */
     public function index()
     {
+        app('view')->share('FF_VERSION', config('firefly.version'));
         // index will set FF3 version.
         app('fireflyconfig')->set('ff3_version', (string)config('firefly.version'));
 
@@ -147,27 +102,19 @@ class InstallController extends Controller
         $response     = [
             'hasNextCommand' => false,
             'done'           => true,
-            'next'           => 0,
             'previous'       => null,
             'error'          => false,
             'errorMessage'   => null,
         ];
 
         Log::debug(sprintf('Will now run commands. Request index is %d', $requestIndex));
-        $index = 0;
+        $indexes = array_values(array_keys($this->upgradeCommands));
-        /**
+        if (array_key_exists($requestIndex, $indexes)) {
-         * @var string $command
+            $command    = $indexes[$requestIndex];
-         * @var array $args
+            $parameters = $this->upgradeCommands[$command];
-         */
+            Log::debug(sprintf('Will now execute command "%s" with parameters', $command), $parameters);
-        foreach ($this->upgradeCommands as $command => $args) {
-            Log::debug(sprintf('Current command is "%s", index is %d', $command, $index));
-            if ($index < $requestIndex) {
-                Log::debug('Will not execute.');
-                $index++;
-                continue;
-            }
             try {
-                $result = $this->executeCommand($command, $args);
+                $result = $this->executeCommand($command, $parameters);
             } catch (FireflyException $e) {
                 Log::error($e->getMessage());
                 Log::error($e->getTraceAsString());
@@ -180,15 +127,11 @@ class InstallController extends Controller
             if (false === $result) {
                 $response['errorMessage'] = $this->lastError;
                 $response['error']        = true;
-
                 return response()->json($response);
             }
-            $index++;
+            $response['hasNextCommand'] = array_key_exists($requestIndex + 1, $indexes);
-            $response['hasNextCommand'] = true;
             $response['previous']       = $command;
         }
-        $response['next'] = $index;
-
         return response()->json($response);
     }
 

app/Http/Middleware/Authenticate.php

@@ -87,13 +87,10 @@ class Authenticate
      */
     protected function authenticate($request, array $guards)
     {
-        Log::debug(sprintf('Now in %s', __METHOD__));
         if (0 === count($guards)) {
-            Log::debug('No guards present.');
             // go for default guard:
             /** @noinspection PhpUndefinedMethodInspection */
             if ($this->auth->check()) {
-                Log::debug('Default guard says user is authenticated.');
                 // do an extra check on user object.
                 /** @noinspection PhpUndefinedMethodInspection */
                 /** @var User $user */
@@ -104,18 +101,13 @@ class Authenticate
             /** @noinspection PhpUndefinedMethodInspection */
             return $this->auth->authenticate();
         }
-        Log::debug('Guard array is not empty.');
 
         foreach ($guards as $guard) {
-            Log::debug(sprintf('Now in guard loop, guard is "%s"', $guard));
             if ('api' !== $guard) {
-                Log::debug('Guard is "api", call authenticate()');
                 $this->auth->guard($guard)->authenticate();
             }
             $result = $this->auth->guard($guard)->check();
-            Log::debug(sprintf('Result is %s', var_export($result, true)));
             if ($result) {
-                Log::debug('Guard says user is authenticated.');
                 $user = $this->auth->guard($guard)->user();
                 $this->validateBlockedUser($user, $guards);
                 // According to PHPstan the method returns void, but we'll see.
@@ -134,7 +126,6 @@ class Authenticate
      */
     private function validateBlockedUser(?User $user, array $guards): void
     {
-        Log::debug(sprintf('Now in %s', __METHOD__));
         if (null === $user) {
             Log::warning('User is null, throw exception?');
         }

app/Http/Requests/AccountFormRequest.php

@@ -110,7 +110,7 @@ class AccountFormRequest extends FormRequest
         $ccPaymentTypes = implode(',', array_keys(config('firefly.ccTypes')));
         $rules          = [
             'administration_id'                  => 'min:1|max:16777216|numeric',
-            'name'                               => 'required|min:1|uniqueAccountForUser',
+            'name'                               => 'required|max:1024|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->convertString('objectType'))],
@@ -133,7 +133,7 @@ class AccountFormRequest extends FormRequest
         if (null !== $account) {
             // add rules:
             $rules['id']   = 'belongsToUser:accounts';
-            $rules['name'] = 'required|min:1|uniqueAccountForUser:'.$account->id;
+            $rules['name'] = 'required|max:1024|min:1|uniqueAccountForUser:'.$account->id;
             $rules['iban'] = ['iban', 'nullable', new UniqueIban($account, $account->accountType->type)];
         }
 

app/Http/Requests/BillUpdateRequest.php

@@ -81,6 +81,7 @@ class BillUpdateRequest extends FormRequest
             'repeat_freq'             => sprintf('required|in:%s', join(',', config('firefly.bill_periods'))),
             'skip'                    => 'required|integer|gte:0|lte:31',
             'active'                  => 'boolean',
+            'notes'                   => 'between:1,65536|nullable',
         ];
     }
 }

app/Http/Requests/LinkTypeFormRequest.php

@@ -43,7 +43,7 @@ class LinkTypeFormRequest extends FormRequest
     public function rules(): array
     {
         // fixed
-        $nameRule = 'required|min:1|unique:link_types,name';
+        $nameRule = 'required|max:255|min:1|unique:link_types,name';
         $idRule   = '';
 
         // get parameter link:
@@ -51,14 +51,14 @@ class LinkTypeFormRequest extends FormRequest
 
         if (null !== $link) {
             $idRule   = 'exists:link_types,id';
-            $nameRule = 'required|min:1';
+            $nameRule = 'required|max:255|min:1';
         }
 
         return [
             'id'      => $idRule,
             'name'    => $nameRule,
-            'inward'  => 'required|min:1|different:outward',
+            'inward'  => 'required|max:255|min:1|different:outward',
-            'outward' => 'required|min:1|different:inward',
+            'outward' => 'required|max:255|min:1|different:inward',
         ];
     }
 }

app/Http/Requests/MassEditJournalRequest.php

@@ -45,7 +45,7 @@ class MassEditJournalRequest extends FormRequest
         // fixed
 
         return [
-            'description.*'    => 'required|min:1,max:255',
+            'description.*'    => 'required|min:1|max:255',
             'source_id.*'      => 'numeric|belongsToUser:accounts,id',
             'destination_id.*' => 'numeric|belongsToUser:accounts,id',
             'journals.*'       => 'numeric|belongsToUser:transaction_journals,id',

app/Http/Requests/PiggyBankUpdateRequest.php

@@ -70,7 +70,7 @@ class PiggyBankUpdateRequest extends FormRequest
             'targetamount' => 'nullable|numeric|max:1000000000',
             'startdate'    => 'date',
             'targetdate'   => 'date|nullable',
-            'order'        => 'integer|min:1',
+            'order'        => 'integer|max:65536|min:1',
             'object_group' => 'min:0|max:255',
         ];
     }

app/Http/Requests/RuleFormRequest.php

@@ -157,9 +157,9 @@ class RuleFormRequest extends FormRequest
             'rule_group_id'    => 'required|belongsToUser:rule_groups',
             'trigger'          => 'required|in:store-journal,update-journal',
             'triggers.*.type'  => 'required|in:'.implode(',', $validTriggers),
-            'triggers.*.value' => sprintf('required_if:triggers.*.type,%s|min:1|ruleTriggerValue', $contextTriggers),
+            'triggers.*.value' => sprintf('required_if:triggers.*.type,%s|max:1024|min:1|ruleTriggerValue', $contextTriggers),
             'actions.*.type'   => 'required|in:'.implode(',', $validActions),
-            'actions.*.value'  => sprintf('required_if:actions.*.type,%s|min:0|max:255|ruleActionValue', $contextActions),
+            'actions.*.value'  => sprintf('required_if:actions.*.type,%s|min:0|max:1024|ruleActionValue', $contextActions),
             'strict'           => 'in:0,1',
         ];
 

app/Http/Requests/TagFormRequest.php

@@ -66,17 +66,18 @@ class TagFormRequest extends FormRequest
 
         /** @var Tag $tag */
         $tag     = $this->route()->parameter('tag');
-        $tagRule = 'required|min:1|uniqueObjectForUser:tags,tag';
+        $tagRule = 'required|max:1024|min:1|uniqueObjectForUser:tags,tag';
         if (null !== $tag) {
             $idRule  = 'belongsToUser:tags';
-            $tagRule = 'required|min:1|uniqueObjectForUser:tags,tag,'.$tag->id;
+            $tagRule = 'required|max:1024|min:1|uniqueObjectForUser:tags,tag,'.$tag->id;
         }
 
         $rules = [
             'tag'         => $tagRule,
             'id'          => $idRule,
-            'description' => 'min:1|nullable',
+            'description' => 'max:65536|min:1|nullable',
             'date'        => 'date|nullable',
+
         ];
 
         return Location::requestRules($rules);

app/Http/Requests/TestRuleFormRequest.php

@@ -49,8 +49,8 @@ class TestRuleFormRequest extends FormRequest
         $validTriggers = $this->getTriggers();
 
         return [
-            'rule-trigger.*'       => 'required|min:1|in:'.implode(',', $validTriggers),
+            'rule-trigger.*'       => 'required|max:1024|min:1|in:'.implode(',', $validTriggers),
-            'rule-trigger-value.*' => 'required|min:1|ruleTriggerValue',
+            'rule-trigger-value.*' => 'required|max:1024|min:1|ruleTriggerValue',
         ];
     }
 }

app/Models/BudgetLimit.php

@@ -24,6 +24,9 @@ declare(strict_types=1);
 namespace FireflyIII\Models;
 
 use Eloquent;
+use FireflyIII\Events\Model\BudgetLimit\Created;
+use FireflyIII\Events\Model\BudgetLimit\Deleted;
+use FireflyIII\Events\Model\BudgetLimit\Updated;
 use Illuminate\Database\Eloquent\Builder;
 use Illuminate\Database\Eloquent\Casts\Attribute;
 use Illuminate\Database\Eloquent\Model;
@@ -81,6 +84,12 @@ class BudgetLimit extends Model
     /** @var array Fields that can be filled */
     protected $fillable = ['budget_id', 'start_date', 'end_date', 'amount', 'transaction_currency_id'];
 
+    protected $dispatchesEvents = [
+        'created' => Created::class,
+        'updated' => Updated::class,
+        'deleted' => Deleted::class,
+    ];
+
     /**
      * Route binder. Converts the key in the URL to the specified object (or throw 404).
      *

app/Models/UserGroup.php

@@ -52,6 +52,8 @@ use Illuminate\Support\Carbon;
  * @property-read Collection<int, \FireflyIII\Models\Account> $accounts
  * @property-read int|null $accounts_count
  * @property-read Collection<int, \FireflyIII\Models\Account> $accounts
+ * @property-read Collection<int, \FireflyIII\Models\Account> $accounts
+ * @property-read Collection<int, \FireflyIII\Models\Account> $accounts
  * @mixin Eloquent
  */
 class UserGroup extends Model

app/Providers/EventServiceProvider.php

@@ -29,6 +29,9 @@ use FireflyIII\Events\AdminRequestedTestMessage;
 use FireflyIII\Events\ChangedPiggyBankAmount;
 use FireflyIII\Events\DestroyedTransactionGroup;
 use FireflyIII\Events\DetectedNewIPAddress;
+use FireflyIII\Events\Model\BudgetLimit\Created;
+use FireflyIII\Events\Model\BudgetLimit\Deleted;
+use FireflyIII\Events\Model\BudgetLimit\Updated;
 use FireflyIII\Events\NewVersionAvailable;
 use FireflyIII\Events\RegisteredUser;
 use FireflyIII\Events\RequestedNewPassword;
@@ -42,6 +45,7 @@ use FireflyIII\Events\UpdatedAccount;
 use FireflyIII\Events\UpdatedTransactionGroup;
 use FireflyIII\Events\UserChangedEmail;
 use FireflyIII\Events\WarnUserAboutBill;
+use FireflyIII\Models\Budget;
 use FireflyIII\Models\BudgetLimit;
 use FireflyIII\Models\PiggyBank;
 use FireflyIII\Models\PiggyBankRepetition;
@@ -160,6 +164,17 @@ class EventServiceProvider extends ServiceProvider
             ChangedPiggyBankAmount::class       => [
                 'FireflyIII\Handlers\Events\PiggyBankEventHandler@changePiggyAmount',
             ],
+            // budget related events: CRUD budget limit
+            Created::class          => [
+                'FireflyIII\Handlers\Events\Model\BudgetLimitHandler@created',
+            ],
+            Updated::class          => [
+                'FireflyIII\Handlers\Events\Model\BudgetLimitHandler@updated',
+            ],
+            Deleted::class          => [
+                'FireflyIII\Handlers\Events\Model\BudgetLimitHandler@deleted',
+            ],
+
         ];
 
     /**
@@ -169,7 +184,6 @@ class EventServiceProvider extends ServiceProvider
     {
         parent::boot();
         $this->registerCreateEvents();
-        $this->registerBudgetEvents();
     }
 
     /**
@@ -188,57 +202,4 @@ class EventServiceProvider extends ServiceProvider
             }
         );
     }
-
-    /**
-     * TODO needs a dedicated method.
-     */
-    protected function registerBudgetEvents(): void
-    {
-        $func = static function (BudgetLimit $limit) {
-            Log::debug('Trigger budget limit event.');
-            // find available budget with same period and same currency or create it.
-            // then set it or add money:
-            $user            = $limit->budget->user;
-            $availableBudget = $user
-                ->availableBudgets()
-                ->where('start_date', $limit->start_date->format('Y-m-d'))
-                ->where('end_date', $limit->end_date->format('Y-m-d'))
-                ->where('transaction_currency_id', $limit->transaction_currency_id)
-                ->first();
-            // update!
-            if (null !== $availableBudget) {
-                $repository = app(BudgetLimitRepositoryInterface::class);
-                $repository->setUser($user);
-                $set = $repository->getAllBudgetLimitsByCurrency($limit->transactionCurrency, $limit->start_date, $limit->end_date);
-                $sum = (string)$set->sum('amount');
-
-
-                Log::debug(
-                    sprintf(
-                        'Because budget limit #%d had its amount changed to %s, available budget limit #%d will be updated.',
-                        $limit->id,
-                        $limit->amount,
-                        $availableBudget->id
-                    )
-                );
-                $availableBudget->amount = $sum;
-                $availableBudget->save();
-                return;
-            }
-            Log::debug('Does not exist, create it.');
-            // create it.
-            $data       = [
-                'amount'      => $limit->amount,
-                'start'       => $limit->start_date,
-                'end'         => $limit->end_date,
-                'currency_id' => $limit->transaction_currency_id,
-            ];
-            $repository = app(AvailableBudgetRepositoryInterface::class);
-            $repository->setUser($user);
-            $repository->store($data);
-        };
-
-        BudgetLimit::created($func);
-        BudgetLimit::updated($func);
-    }
 }

app/Repositories/Attachment/AttachmentRepository.php

@@ -84,10 +84,10 @@ class AttachmentRepository implements AttachmentRepositoryInterface
 
         if ($disk->exists($file)) {
             $encryptedContent = (string)$disk->get($file);
-
             try {
                 $unencryptedContent = Crypt::decrypt($encryptedContent); // verified
             } catch (DecryptException $e) {
+                Log::debug(sprintf('Could not decrypt attachment #%d but this is fine: %s', $attachment->id, $e->getMessage()));
                 $unencryptedContent = $encryptedContent;
             }
         }

app/Rules/UniqueIban.php

@@ -23,18 +23,20 @@ declare(strict_types=1);
 
 namespace FireflyIII\Rules;
 
+use Closure;
 use FireflyIII\Models\Account;
 use FireflyIII\Models\AccountType;
 use Illuminate\Contracts\Validation\Rule;
+use Illuminate\Contracts\Validation\ValidationRule;
 use Illuminate\Support\Facades\Log;
 
 /**
  * Class UniqueIban
  */
-class UniqueIban implements Rule
+class UniqueIban implements ValidationRule
 {
     private ?Account $account;
-    private ?string  $expectedType;
+    private array    $expectedTypes;
 
     /**
      * Create a new rule instance.
@@ -45,17 +47,24 @@ class UniqueIban implements Rule
      */
     public function __construct(?Account $account, ?string $expectedType)
     {
-        $this->account      = $account;
+        $this->account       = $account;
-        $this->expectedType = $expectedType;
+        $this->expectedTypes = [];
+        if (null === $expectedType) {
+            return;
+        }
+        $this->expectedTypes = [$expectedType];
         // a very basic fix to make sure we get the correct account type:
         if ('expense' === $expectedType) {
-            $this->expectedType = AccountType::EXPENSE;
+            $this->expectedTypes = [AccountType::EXPENSE];
         }
         if ('revenue' === $expectedType) {
-            $this->expectedType = AccountType::REVENUE;
+            $this->expectedTypes = [AccountType::REVENUE];
         }
         if ('asset' === $expectedType) {
-            $this->expectedType = AccountType::ASSET;
+            $this->expectedTypes = [AccountType::ASSET];
+        }
+        if ('liabilities' === $expectedType) {
+            $this->expectedTypes = [AccountType::LOAN, AccountType::DEBT, AccountType::MORTGAGE];
         }
     }
 
@@ -84,7 +93,7 @@ class UniqueIban implements Rule
         if (!auth()->check()) {
             return true;
         }
-        if (null === $this->expectedType) {
+        if (0 === count($this->expectedTypes)) {
             return true;
         }
         $maxCounts = $this->getMaxOccurrences();
@@ -95,11 +104,11 @@ class UniqueIban implements Rule
             if ($count > $max) {
                 Log::debug(
                     sprintf(
-                        'IBAN "%s" is in use with %d account(s) of type "%s", which is too much for expected type "%s"',
+                        'IBAN "%s" is in use with %d account(s) of type "%s", which is too much for expected types "%s"',
                         $value,
                         $count,
                         $type,
-                        $this->expectedType
+                        join(', ', $this->expectedTypes)
                     )
                 );
 
@@ -120,14 +129,15 @@ class UniqueIban implements Rule
             AccountType::ASSET   => 0,
             AccountType::EXPENSE => 0,
             AccountType::REVENUE => 0,
+            'liabilities'        => 0,
         ];
 
-        if ('expense' === $this->expectedType || AccountType::EXPENSE === $this->expectedType) {
+        if (in_array('expense', $this->expectedTypes, true) || in_array(AccountType::EXPENSE, $this->expectedTypes, true)) {
             // IBAN should be unique amongst expense and asset accounts.
             // may appear once in revenue accounts
             $maxCounts[AccountType::REVENUE] = 1;
         }
-        if ('revenue' === $this->expectedType || AccountType::REVENUE === $this->expectedType) {
+        if (in_array('revenue', $this->expectedTypes, true) || in_array(AccountType::REVENUE, $this->expectedTypes, true)) {
             // IBAN should be unique amongst revenue and asset accounts.
             // may appear once in expense accounts
             $maxCounts[AccountType::EXPENSE] = 1;
@@ -144,12 +154,16 @@ class UniqueIban implements Rule
      */
     private function countHits(string $type, string $iban): int
     {
+        $typesArray = [$type];
+        if ('liabilities' === $type) {
+            $typesArray = [AccountType::LOAN, AccountType::DEBT, AccountType::MORTGAGE];
+        }
         $query
             = auth()->user()
                     ->accounts()
                     ->leftJoin('account_types', 'account_types.id', '=', 'accounts.account_type_id')
                     ->where('accounts.iban', $iban)
-                    ->where('account_types.type', $type);
+                    ->whereIn('account_types.type', $typesArray);
 
         if (null !== $this->account) {
             $query->where('accounts.id', '!=', $this->account->id);
@@ -157,4 +171,14 @@ class UniqueIban implements Rule
 
         return $query->count();
     }
+
+    /**
+     * @inheritDoc
+     */
+    public function validate(string $attribute, mixed $value, Closure $fail): void
+    {
+        if (!$this->passes($attribute, $value)) {
+            $fail((string)trans('validation.unique_iban_for_user'));
+        }
+    }
 }

app/Support/Search/OperatorQuerySearch.php

@@ -856,12 +856,12 @@ class OperatorQuerySearch implements SearchInterface
                 break;
             case '-tag_is_not':
             case 'tag_is':
-                $result = $this->tagRepository->searchTag($value);
+                $result = $this->tagRepository->findByTag($value);
-                if ($result->count() > 0) {
+                if (null !== $result) {
-                    $this->collector->setTags($result);
+                    $this->collector->setTags(new Collection([$result]));
                 }
                 // no tags found means search must result in nothing.
-                if (0 === $result->count()) {
+                if (null === $result) {
                     Log::info(sprintf('No valid tags in "%s"-operator, so search will not return ANY results.', $operator));
                     $this->collector->findNothing();
                 }

app/Validation/GroupValidation.php

@@ -24,6 +24,7 @@ declare(strict_types=1);
 
 namespace FireflyIII\Validation;
 
+use FireflyIII\Exceptions\FireflyException;
 use FireflyIII\Models\TransactionGroup;
 use Illuminate\Validation\Validator;
 use Illuminate\Support\Facades\Log;
@@ -53,6 +54,9 @@ trait GroupValidation
         ];
         /** @var array $transaction */
         foreach ($transactions as $index => $transaction) {
+            if(!is_array($transaction)) {
+                throw new FireflyException('Invalid data submitted: transaction is not array.');
+            }
             $hasAccountInfo = false;
             $hasJournalId   = array_key_exists('transaction_journal_id', $transaction);
             foreach ($keys as $key) {

app/Validation/TransactionValidation.php

@@ -23,6 +23,7 @@ declare(strict_types=1);
 
 namespace FireflyIII\Validation;
 
+use FireflyIII\Exceptions\FireflyException;
 use FireflyIII\Models\Account;
 use FireflyIII\Models\AccountType;
 use FireflyIII\Models\Transaction;
@@ -30,8 +31,8 @@ use FireflyIII\Models\TransactionGroup;
 use FireflyIII\Models\TransactionJournal;
 use FireflyIII\Models\TransactionType;
 use FireflyIII\Repositories\Account\AccountRepositoryInterface;
-use Illuminate\Validation\Validator;
 use Illuminate\Support\Facades\Log;
+use Illuminate\Validation\Validator;
 
 /**
  * Trait TransactionValidation
@@ -74,19 +75,22 @@ trait TransactionValidation
      */
     protected function getTransactionsArray(Validator $validator): array
     {
+        Log::debug('Now in getTransactionsArray');
         $data         = $validator->getData();
-        $transactions = $data['transactions'] ?? [];
+        $transactions = [];
+        if (is_array($data) && array_key_exists('transactions', $data) && is_array($data['transactions'])) {
+            Log::debug('Transactions key exists and is array.');
+            $transactions = $data['transactions'];
+        }
+        if (is_array($data) && array_key_exists('transactions', $data) && !is_array($data['transactions'])) {
+            Log::debug(sprintf('Transactions key exists but is NOT array,  its a %s', gettype($data['transactions'])));
+        }
+        // should be impossible to hit this:
         if (!is_countable($transactions)) {
             Log::error(sprintf('Transactions array is not countable, because its a %s', gettype($transactions)));
-
-            return [];
-        }
-        // a superfluous check but you never know.
-        if (!is_array($transactions)) {
-            Log::error(sprintf('Transactions array is not an array, because its a %s', gettype($transactions)));
-
             return [];
         }
+        Log::debug('Returning transactions.', $transactions);
 
         return $transactions;
     }
@@ -355,6 +359,9 @@ trait TransactionValidation
          * @var array $transaction
          */
         foreach ($transactions as $index => $transaction) {
+            if(!is_int($index)) {
+                throw new FireflyException('Invalid data submitted: transaction is not array.');
+            }
             $this->validateSingleUpdate($validator, $index, $transaction, $transactionGroup);
         }
     }
@@ -489,10 +496,11 @@ trait TransactionValidation
      */
     public function validateOneTransaction(Validator $validator): void
     {
+        Log::debug('Now in validateOneTransaction');
         if ($validator->errors()->count() > 0) {
+            Log::debug('Validator already has errors, so return.');
             return;
         }
-        Log::debug('Now in validateOneTransaction()');
         $transactions = $this->getTransactionsArray($validator);
         // need at least one transaction
         if (0 === count($transactions)) {

changelog.md

@@ -2,6 +2,39 @@
 All notable changes to this project will be documented in this file.
 This project adheres to [Semantic Versioning](http://semver.org/).
 
+## 6.0.9 - 2023-04-29
+
+### Added
+- Better length validation for text fields.
+
+### Changed
+- Better calculation of available budget
+
+### Fixed
+- [Issue 7377](https://github.com/firefly-iii/firefly-iii/issues/7377) Tag search was broken
+- [Issue 7389](https://github.com/firefly-iii/firefly-iii/issues/7389) Bug in charts
+- [Issue 7394](https://github.com/firefly-iii/firefly-iii/issues/7394) unique iban check was broken
+- [Issue 7427](https://github.com/firefly-iii/firefly-iii/issues/7427) API would not accept page 18 and up.
+- [Issue 7410](https://github.com/firefly-iii/firefly-iii/issues/7410) Various dark mode color fixes
+- Old documentation links fixed by @mindlessroman and @noxonad!
+
+## 6.0.8 - 2023-04-16
+
+### Added
+- [Issue 7351](https://github.com/firefly-iii/firefly-iii/issues/7351) Optional command to force the decimal size.
+- [Issue 7352](https://github.com/firefly-iii/firefly-iii/issues/7352) Optional command to force the migrations.
+- [Issue 7354](https://github.com/firefly-iii/firefly-iii/issues/7354) The new v3 layout will redirect to the index when unauthenticated, thanks @corcom!
+
+### Fixed
+- [Issue 7349](https://github.com/firefly-iii/firefly-iii/issues/7349) Missing tables in PostgreSQL script.
+- [Issue 7358](https://github.com/firefly-iii/firefly-iii/issues/7358) Could not create liabilities with a pre-set amount.
+- Fix date field in bill warning mail.
+- Fix installer script.
+- Remove attachment paperclip from transactions with deleted attachments.
+
+### API
+- [Issue 7347](https://github.com/firefly-iii/firefly-iii/issues/7347) API made rules would be inactive by default.
+
 ## v6.0.7 - 2023-04-09
 
 ### Added

composer.json

@@ -82,7 +82,7 @@
     "ext-xml": "*",
     "ext-xmlwriter": "*",
     "bacon/bacon-qr-code": "2.*",
-    "diglactic/laravel-breadcrumbs": "^8.0",
+    "diglactic/laravel-breadcrumbs": "^8.1",
     "doctrine/dbal": "3.*",
     "gdbots/query-parser": "^3.0",
     "guzzlehttp/guzzle": "^7.5",
@@ -104,6 +104,7 @@
     "ramsey/uuid": "^4.7",
     "rcrowe/twigbridge": "^0.14",
     "spatie/laravel-ignition": "^2",
+    "spatie/period": "^2.4",
     "symfony/http-client": "^6.0",
     "symfony/mailgun-mailer": "^6.0",
     "therobfonz/laravel-mandrill-driver": "^5.0"
@@ -114,7 +115,7 @@
     "fakerphp/faker": "1.*",
     "filp/whoops": "2.*",
     "mockery/mockery": "1.*",
-    "nunomaduro/larastan": "^2.5",
+    "nunomaduro/larastan": "^2.6",
     "phpstan/phpstan": "^1.10",
     "phpstan/phpstan-deprecation-rules": "^1.1",
     "phpstan/phpstan-strict-rules": "^1.4",
@@ -155,57 +156,17 @@
       "Illuminate\\Foundation\\ComposerScripts::postAutoloadDump"
     ],
     "post-update-cmd": [
+      "@php artisan config:clear",
+      "@php artisan route:clear",
+      "@php artisan twig:clean",
+      "@php artisan view:clear",
+      "@php artisan clear-compiled",
       "@php artisan cache:clear",
-      "@php artisan firefly-iii:fix-pgsql-sequences",
+      "@php artisan firefly-iii:upgrade-database",
-      "@php artisan firefly-iii:decrypt-all",
+      "@php artisan firefly-iii:correct-database",
-      "@php artisan firefly-iii:transaction-identifiers",
+      "@php artisan firefly-iii:report-integrity",
-      "@php artisan firefly-iii:migrate-to-groups",
+      "@php artisan passport:install",
-      "@php artisan firefly-iii:account-currencies",
+      "@php artisan firefly:instructions update"
-      "@php artisan firefly-iii:transfer-currencies",
-      "@php artisan firefly-iii:other-currencies",
-      "@php artisan firefly-iii:migrate-notes",
-      "@php artisan firefly-iii:migrate-attachments",
-      "@php artisan firefly-iii:bills-to-rules",
-      "@php artisan firefly-iii:bl-currency",
-      "@php artisan firefly-iii:cc-liabilities",
-      "@php artisan firefly-iii:back-to-journals",
-      "@php artisan firefly-iii:rename-account-meta",
-      "@php artisan firefly-iii:migrate-recurrence-meta",
-      "@php artisan firefly-iii:migrate-tag-locations",
-      "@php artisan firefly-iii:migrate-recurrence-type",
-      "@php artisan firefly-iii:upgrade-liabilities",
-      "@php artisan firefly-iii:liabilities-600",
-      "@php artisan firefly-iii:fix-piggies",
-      "@php artisan firefly-iii:create-link-types",
-      "@php artisan firefly-iii:create-access-tokens",
-      "@php artisan firefly-iii:remove-bills",
-      "@php artisan firefly-iii:fix-negative-limits",
-      "@php artisan firefly-iii:enable-currencies",
-      "@php artisan firefly-iii:fix-transfer-budgets",
-      "@php artisan firefly-iii:fix-uneven-amount",
-      "@php artisan firefly-iii:delete-zero-amount",
-      "@php artisan firefly-iii:delete-orphaned-transactions",
-      "@php artisan firefly-iii:delete-empty-journals",
-      "@php artisan firefly-iii:delete-empty-groups",
-      "@php artisan firefly-iii:fix-account-types",
-      "@php artisan firefly-iii:fix-account-order",
-      "@php artisan firefly-iii:rename-meta-fields",
-      "@php artisan firefly-iii:fix-ob-currencies",
-      "@php artisan firefly-iii:fix-long-descriptions",
-      "@php artisan firefly-iii:fix-recurring-transactions",
-      "@php artisan firefly-iii:unify-group-accounts",
-      "@php artisan firefly-iii:fix-transaction-types",
-      "@php artisan firefly-iii:fix-frontpage-accounts",
-      "@php artisan firefly-iii:fix-ibans",
-      "@php artisan firefly-iii:create-group-memberships",
-      "@php artisan firefly-iii:report-empty-objects",
-      "@php artisan firefly-iii:report-sum",
-      "@php artisan firefly-iii:restore-oauth-keys",
-      "@php artisan firefly-iii:upgrade-group-information",
-      "@php artisan firefly-iii:set-latest-version --james-is-cool",
-      "@php artisan firefly:instructions update",
-      "@php artisan firefly-iii:verify-security-alerts",
-      "@php artisan passport:install"
     ],
     "post-install-cmd": [
       "@php artisan firefly:instructions install",

config/firefly.php

@@ -107,7 +107,7 @@ return [
         'webhooks'     => true,
         'handle_debts' => true,
     ],
-    'version'                      => '6.0.7',
+    'version'                      => '6.0.9',
     'api_version'                  => '2.0.1',
     'db_version'                   => 19,
 
@@ -879,7 +879,7 @@ return [
     ],
     'can_have_virtual_amounts'  => [AccountType::ASSET],
     'can_have_opening_balance'  => [AccountType::ASSET, AccountType::LOAN, AccountType::DEBT, AccountType::MORTGAGE],
-    'dynamic_creation_allowed'  => [AccountType::EXPENSE, AccountType::REVENUE, AccountType::INITIAL_BALANCE, AccountType::RECONCILIATION],
+    'dynamic_creation_allowed'  => [AccountType::EXPENSE, AccountType::REVENUE, AccountType::INITIAL_BALANCE, AccountType::RECONCILIATION, AccountType::LIABILITY_CREDIT],
     'valid_asset_fields'        => ['account_role', 'account_number', 'currency_id', 'BIC', 'include_net_worth'],
     'valid_cc_fields'           => ['account_role', 'cc_monthly_payment_date', 'cc_type', 'account_number', 'currency_id', 'BIC', 'include_net_worth'],
     'valid_account_fields'      => ['account_number', 'currency_id', 'BIC', 'interest', 'interest_period', 'include_net_worth', 'liability_direction'],

database/migrations/2016_09_12_121359_fix_nullables.php

@@ -45,28 +45,32 @@ class FixNullables extends Migration
      */
     public function up(): void
     {
-        try {
+        if (!Schema::hasColumn('rule_groups', 'description')) {
-            Schema::table(
+            try {
-                'rule_groups',
+                Schema::table(
-                static function (Blueprint $table) {
+                    'rule_groups',
-                    $table->text('description')->nullable()->change();
+                    static function (Blueprint $table) {
-                }
+                        $table->text('description')->nullable()->change();
-            );
+                    }
-        } catch (QueryException $e) {
+                );
-            Log::error(sprintf('Could not update table: %s', $e->getMessage()));
+            } catch (QueryException $e) {
-            Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+                Log::error(sprintf('Could not update table: %s', $e->getMessage()));
+                Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+            }
         }
 
-        try {
+        if (!Schema::hasColumn('rules', 'description')) {
-            Schema::table(
+            try {
-                'rules',
+                Schema::table(
-                static function (Blueprint $table) {
+                    'rules',
-                    $table->text('description')->nullable()->change();
+                    static function (Blueprint $table) {
-                }
+                        $table->text('description')->nullable()->change();
-            );
+                    }
-        } catch (QueryException $e) {
+                );
-            Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+            } catch (QueryException $e) {
-            Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+                Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+                Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+            }
         }
     }
 }

database/migrations/2016_10_09_150037_expand_transactions_table.php

@@ -38,16 +38,18 @@ class ExpandTransactionsTable extends Migration
      */
     public function down(): void
     {
-        try {
+        if (Schema::hasColumn('transactions', 'identifier')) {
-            Schema::table(
+            try {
-                'transactions',
+                Schema::table(
-                static function (Blueprint $table) {
+                    'transactions',
-                    $table->dropColumn('identifier');
+                    static function (Blueprint $table) {
-                }
+                        $table->dropColumn('identifier');
-            );
+                    }
-        } catch (QueryException|ColumnDoesNotExist $e) {
+                );
-            Log::error(sprintf('Could not drop column "extended_status": %s', $e->getMessage()));
+            } catch (QueryException|ColumnDoesNotExist $e) {
-            Log::error('If the column does not exist, this is not an problem. Otherwise, please open a GitHub discussion.');
+                Log::error(sprintf('Could not drop column "identifier": %s', $e->getMessage()));
+                Log::error('If the column does not exist, this is not an problem. Otherwise, please open a GitHub discussion.');
+            }
         }
     }
 
@@ -57,16 +59,18 @@ class ExpandTransactionsTable extends Migration
      */
     public function up(): void
     {
-        try {
+        if (!Schema::hasColumn('transactions', 'identifier')) {
-            Schema::table(
+            try {
-                'transactions',
+                Schema::table(
-                static function (Blueprint $table) {
+                    'transactions',
-                    $table->smallInteger('identifier', false, true)->default(0);
+                    static function (Blueprint $table) {
-                }
+                        $table->smallInteger('identifier', false, true)->default(0);
-            );
+                    }
-        } catch (QueryException $e) {
+                );
-            Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+            } catch (QueryException $e) {
-            Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+                Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+                Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+            }
         }
     }
 }

database/migrations/2016_10_22_075804_changes_for_v410.php

@@ -46,22 +46,24 @@ class ChangesForV410 extends Migration
      */
     public function up(): void
     {
-        try {
+        if(!Schema::hasTable('notes')) {
-            Schema::create(
+            try {
-                'notes',
+                Schema::create(
-                static function (Blueprint $table) {
+                    'notes',
-                    $table->increments('id');
+                    static function (Blueprint $table) {
-                    $table->timestamps();
+                        $table->increments('id');
-                    $table->softDeletes();
+                        $table->timestamps();
-                    $table->integer('noteable_id', false, true);
+                        $table->softDeletes();
-                    $table->string('noteable_type');
+                        $table->integer('noteable_id', false, true);
-                    $table->string('title')->nullable();
+                        $table->string('noteable_type');
-                    $table->text('text')->nullable();
+                        $table->string('title')->nullable();
-                }
+                        $table->text('text')->nullable();
-            );
+                    }
-        } catch (QueryException $e) {
+                );
-            Log::error(sprintf('Could not create table "notes": %s', $e->getMessage()));
+            } catch (QueryException $e) {
-            Log::error('If this table exists already (see the error message), this is not a problem. Other errors? Please open a discussion on GitHub.');
+                Log::error(sprintf('Could not create table "notes": %s', $e->getMessage()));
+                Log::error('If this table exists already (see the error message), this is not a problem. Other errors? Please open a discussion on GitHub.');
+            }
         }
     }
 }

database/migrations/2016_11_24_210552_changes_for_v420.php

@@ -37,16 +37,18 @@ class ChangesForV420 extends Migration
      */
     public function down(): void
     {
-        try {
+        if (Schema::hasColumn('journal_meta', 'deleted_at')) {
-            Schema::table(
+            try {
-                'journal_meta',
+                Schema::table(
-                static function (Blueprint $table) {
+                    'journal_meta',
-                    $table->dropSoftDeletes();
+                    static function (Blueprint $table) {
-                }
+                        $table->dropSoftDeletes();
-            );
+                    }
-        } catch (QueryException $e) {
+                );
-            Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+            } catch (QueryException $e) {
-            Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+                Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+                Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+            }
         }
     }
 
@@ -56,16 +58,18 @@ class ChangesForV420 extends Migration
      */
     public function up(): void
     {
-        try {
+        if (!Schema::hasColumn('journal_meta', 'deleted_at')) {
-            Schema::table(
+            try {
-                'journal_meta',
+                Schema::table(
-                static function (Blueprint $table) {
+                    'journal_meta',
-                    $table->softDeletes();
+                    static function (Blueprint $table) {
-                }
+                        $table->softDeletes();
-            );
+                    }
-        } catch (QueryException $e) {
+                );
-            Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+            } catch (QueryException $e) {
-            Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+                Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+                Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+            }
         }
     }
 }

database/migrations/2016_12_22_150431_changes_for_v430.php

@@ -46,26 +46,28 @@ class ChangesForV430 extends Migration
      */
     public function up(): void
     {
-        try {
+        if (!Schema::hasTable('available_budgets')) {
-            Schema::create(
+            try {
-                'available_budgets',
+                Schema::create(
-                static function (Blueprint $table) {
+                    'available_budgets',
-                    $table->increments('id');
+                    static function (Blueprint $table) {
-                    $table->timestamps();
+                        $table->increments('id');
-                    $table->softDeletes();
+                        $table->timestamps();
-                    $table->integer('user_id', false, true);
+                        $table->softDeletes();
-                    $table->integer('transaction_currency_id', false, true);
+                        $table->integer('user_id', false, true);
-                    $table->decimal('amount', 32, 12);
+                        $table->integer('transaction_currency_id', false, true);
-                    $table->date('start_date');
+                        $table->decimal('amount', 32, 12);
-                    $table->date('end_date');
+                        $table->date('start_date');
+                        $table->date('end_date');
 
-                    $table->foreign('transaction_currency_id')->references('id')->on('transaction_currencies')->onDelete('cascade');
+                        $table->foreign('transaction_currency_id')->references('id')->on('transaction_currencies')->onDelete('cascade');
-                    $table->foreign('user_id')->references('id')->on('users')->onDelete('cascade');
+                        $table->foreign('user_id')->references('id')->on('users')->onDelete('cascade');
-                }
+                    }
-            );
+                );
-        } catch (QueryException $e) {
+            } catch (QueryException $e) {
-            Log::error(sprintf('Could not create table "available_budgets": %s', $e->getMessage()));
+                Log::error(sprintf('Could not create table "available_budgets": %s', $e->getMessage()));
-            Log::error('If this table exists already (see the error message), this is not a problem. Other errors? Please open a discussion on GitHub.');
+                Log::error('If this table exists already (see the error message), this is not a problem. Other errors? Please open a discussion on GitHub.');
+            }
         }
     }
 }

database/migrations/2016_12_28_203205_changes_for_v431.php

@@ -39,65 +39,74 @@ class ChangesForV431 extends Migration
     public function down(): void
     {
         // reinstate "repeats" and "repeat_freq".
-        try {
+        if (!Schema::hasColumn('budget_limits', 'repeat_freq')) {
-            Schema::table(
+            try {
-                'budget_limits',
+                Schema::table(
-                static function (Blueprint $table) {
+                    'budget_limits',
-                    $table->string('repeat_freq', 30)->nullable();
+                    static function (Blueprint $table) {
-                }
+                        $table->string('repeat_freq', 30)->nullable();
-            );
+                    }
-        } catch (QueryException $e) {
+                );
-            Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+            } catch (QueryException $e) {
-            Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+                Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+                Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+            }
         }
-        try {
+        if (!Schema::hasColumn('budget_limits', 'repeats')) {
-            Schema::table(
+            try {
-                'budget_limits',
+                Schema::table(
-                static function (Blueprint $table) {
+                    'budget_limits',
-                    $table->boolean('repeats')->default(0);
+                    static function (Blueprint $table) {
-                }
+                        $table->boolean('repeats')->default(0);
-            );
+                    }
-        } catch (QueryException $e) {
+                );
-            Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+            } catch (QueryException $e) {
-            Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+                Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+                Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+            }
         }
-
         // change field "start_date" to "startdate"
-        try {
+        if (Schema::hasColumn('budget_limits', 'start_date') && !Schema::hasColumn('budget_limits', 'startdate')) {
-            Schema::table(
+            try {
-                'budget_limits',
+                Schema::table(
-                static function (Blueprint $table) {
+                    'budget_limits',
-                    $table->renameColumn('start_date', 'startdate');
+                    static function (Blueprint $table) {
-                }
+                        $table->renameColumn('start_date', 'startdate');
-            );
+                    }
-        } catch (QueryException|ColumnDoesNotExist $e) {
+                );
-            Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+            } catch (QueryException|ColumnDoesNotExist $e) {
-            Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+                Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+                Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+            }
         }
 
         // remove date field "end_date"
-        try {
+        if (Schema::hasColumn('budget_limits', 'end_date')) {
-            Schema::table(
+            try {
-                'budget_limits',
+                Schema::table(
-                static function (Blueprint $table) {
+                    'budget_limits',
-                    $table->dropColumn('end_date');
+                    static function (Blueprint $table) {
-                }
+                        $table->dropColumn('end_date');
-            );
+                    }
-        } catch (QueryException|ColumnDoesNotExist $e) {
+                );
-            Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+            } catch (QueryException|ColumnDoesNotExist $e) {
-            Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+                Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+                Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+            }
         }
         // remove decimal places
-        try {
+        if (Schema::hasColumn('transaction_currencies', 'decimal_places')) {
-            Schema::table(
+            try {
-                'transaction_currencies',
+                Schema::table(
-                static function (Blueprint $table) {
+                    'transaction_currencies',
-                    $table->dropColumn('decimal_places');
+                    static function (Blueprint $table) {
-                }
+                        $table->dropColumn('decimal_places');
-            );
+                    }
-        } catch (QueryException|ColumnDoesNotExist $e) {
+                );
-            Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+            } catch (QueryException|ColumnDoesNotExist $e) {
-            Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+                Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+                Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+            }
         }
     }
 
@@ -108,66 +117,76 @@ class ChangesForV431 extends Migration
     public function up(): void
     {
         // add decimal places to "transaction currencies".
-        try {
+        if (!Schema::hasColumn('transaction_currencies', 'decimal_places')) {
-            Schema::table(
+            try {
-                'transaction_currencies',
+                Schema::table(
-                static function (Blueprint $table) {
+                    'transaction_currencies',
-                    $table->smallInteger('decimal_places', false, true)->default(2);
+                    static function (Blueprint $table) {
-                }
+                        $table->smallInteger('decimal_places', false, true)->default(2);
-            );
+                    }
-        } catch (QueryException $e) {
+                );
-            Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+            } catch (QueryException $e) {
-            Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+                Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+                Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+            }
         }
 
         // change field "startdate" to "start_date"
-        try {
+        if (Schema::hasColumn('budget_limits', 'startdate') && !Schema::hasColumn('budget_limits', 'start_date')) {
-            Schema::table(
+            try {
-                'budget_limits',
+                Schema::table(
-                static function (Blueprint $table) {
+                    'budget_limits',
-                    $table->renameColumn('startdate', 'start_date');
+                    static function (Blueprint $table) {
-                }
+                        $table->renameColumn('startdate', 'start_date');
-            );
+                    }
-        } catch (QueryException|ColumnDoesNotExist $e) {
+                );
-            Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+            } catch (QueryException|ColumnDoesNotExist $e) {
-            Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+                Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+                Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+            }
         }
 
         // add date field "end_date" after "start_date"
-        try {
+        if (!Schema::hasColumn('budget_limits', 'end_date')) {
-            Schema::table(
+            try {
-                'budget_limits',
+                Schema::table(
-                static function (Blueprint $table) {
+                    'budget_limits',
-                    $table->date('end_date')->nullable()->after('start_date');
+                    static function (Blueprint $table) {
-                }
+                        $table->date('end_date')->nullable()->after('start_date');
-            );
+                    }
-        } catch (QueryException $e) {
+                );
-            Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+            } catch (QueryException $e) {
-            Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+                Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+                Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+            }
         }
 
         // drop "repeats" and "repeat_freq".
-        try {
+        if (Schema::hasColumn('budget_limits', 'repeats')) {
-            Schema::table(
+            try {
-                'budget_limits',
+                Schema::table(
-                static function (Blueprint $table) {
+                    'budget_limits',
-                    $table->dropColumn('repeats');
+                    static function (Blueprint $table) {
-                }
+                        $table->dropColumn('repeats');
-            );
+                    }
-        } catch (QueryException|ColumnDoesNotExist $e) {
+                );
-            Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+            } catch (QueryException|ColumnDoesNotExist $e) {
-            Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+                Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+                Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+            }
         }
-        try {
+        if (Schema::hasColumn('budget_limits', 'repeat_freq')) {
-            Schema::table(
+            try {
-                'budget_limits',
+                Schema::table(
-                static function (Blueprint $table) {
+                    'budget_limits',
-                    $table->dropColumn('repeat_freq');
+                    static function (Blueprint $table) {
-                }
+                        $table->dropColumn('repeat_freq');
-            );
+                    }
-        } catch (QueryException|ColumnDoesNotExist $e) {
+                );
-            Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+            } catch (QueryException|ColumnDoesNotExist $e) {
-            Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+                Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+                Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+            }
         }
     }
 }

database/migrations/2017_04_13_163623_changes_for_v440.php

@@ -89,20 +89,21 @@ class ChangesForV440 extends Migration
                 Log::error('If this table exists already (see the error message), this is not a problem. Other errors? Please open a discussion on GitHub.');
             }
         }
-
+        if(!Schema::hasColumn('transactions', 'transaction_currency_id')) {
-        try {
+            try {
-            Schema::table(
+                Schema::table(
-                'transactions',
+                    'transactions',
-                static function (Blueprint $table) {
+                    static function (Blueprint $table) {
-                    if (!Schema::hasColumn('transactions', 'transaction_currency_id')) {
+                        if (!Schema::hasColumn('transactions', 'transaction_currency_id')) {
-                        $table->integer('transaction_currency_id', false, true)->after('description')->nullable();
+                            $table->integer('transaction_currency_id', false, true)->after('description')->nullable();
-                        $table->foreign('transaction_currency_id')->references('id')->on('transaction_currencies')->onDelete('set null');
+                            $table->foreign('transaction_currency_id')->references('id')->on('transaction_currencies')->onDelete('set null');
+                        }
                     }
-                }
+                );
-            );
+            } catch (QueryException $e) {
-        } catch (QueryException $e) {
+                Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
-            Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+                Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
-            Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+            }
         }
     }
 }

database/migrations/2017_06_02_105232_changes_for_v450.php

@@ -39,16 +39,18 @@ class ChangesForV450 extends Migration
     public function down(): void
     {
         // split up for sqlite compatibility
-        try {
+        if(Schema::hasColumn('transactions', 'foreign_amount')) {
-            Schema::table(
+            try {
-                'transactions',
+                Schema::table(
-                static function (Blueprint $table) {
+                    'transactions',
-                    $table->dropColumn('foreign_amount');
+                    static function (Blueprint $table) {
-                }
+                        $table->dropColumn('foreign_amount');
-            );
+                    }
-        } catch (QueryException|ColumnDoesNotExist $e) {
+                );
-            Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+            } catch (QueryException|ColumnDoesNotExist $e) {
-            Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+                Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+                Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+            }
         }
 
         try {
@@ -65,17 +67,18 @@ class ChangesForV450 extends Migration
             Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
             Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
         }
-
+        if(Schema::hasColumn('transactions', 'foreign_currency_id')) {
-        try {
+            try {
-            Schema::table(
+                Schema::table(
-                'transactions',
+                    'transactions',
-                static function (Blueprint $table) {
+                    static function (Blueprint $table) {
-                    $table->dropColumn('foreign_currency_id');
+                        $table->dropColumn('foreign_currency_id');
-                }
+                    }
-            );
+                );
-        } catch (QueryException|ColumnDoesNotExist $e) {
+            } catch (QueryException|ColumnDoesNotExist $e) {
-            Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+                Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
-            Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+                Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+            }
         }
     }
 
@@ -86,30 +89,34 @@ class ChangesForV450 extends Migration
     public function up(): void
     {
         // add "foreign_amount" to transactions
-        try {
+        if(!Schema::hasColumn('transactions', 'foreign_amount')) {
-            Schema::table(
+            try {
-                'transactions',
+                Schema::table(
-                static function (Blueprint $table) {
+                    'transactions',
-                    $table->decimal('foreign_amount', 32, 12)->nullable()->after('amount');
+                    static function (Blueprint $table) {
-                }
+                        $table->decimal('foreign_amount', 32, 12)->nullable()->after('amount');
-            );
+                    }
-        } catch (QueryException $e) {
+                );
-            Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+            } catch (QueryException $e) {
-            Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+                Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+                Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+            }
         }
 
         // add foreign transaction currency id to transactions (is nullable):
-        try {
+        if(!Schema::hasColumn('transactions', 'foreign_currency_id')) {
-            Schema::table(
+            try {
-                'transactions',
+                Schema::table(
-                static function (Blueprint $table) {
+                    'transactions',
-                    $table->integer('foreign_currency_id', false, true)->default(null)->after('foreign_amount')->nullable();
+                    static function (Blueprint $table) {
-                    $table->foreign('foreign_currency_id')->references('id')->on('transaction_currencies')->onDelete('set null');
+                        $table->integer('foreign_currency_id', false, true)->default(null)->after('foreign_amount')->nullable();
-                }
+                        $table->foreign('foreign_currency_id')->references('id')->on('transaction_currencies')->onDelete('set null');
-            );
+                    }
-        } catch (QueryException $e) {
+                );
-            Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+            } catch (QueryException $e) {
-            Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+                Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+                Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+            }
         }
     }
 }

database/migrations/2017_11_04_170844_changes_for_v470a.php

@@ -39,16 +39,18 @@ class ChangesForV470a extends Migration
      */
     public function down(): void
     {
-        try {
+        if (Schema::hasColumn('transactions', 'reconciled')) {
-            Schema::table(
+            try {
-                'transactions',
+                Schema::table(
-                static function (Blueprint $table) {
+                    'transactions',
-                    $table->dropColumn('reconciled');
+                    static function (Blueprint $table) {
-                }
+                        $table->dropColumn('reconciled');
-            );
+                    }
-        } catch (QueryException|ColumnDoesNotExist $e) {
+                );
-            Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+            } catch (QueryException|ColumnDoesNotExist $e) {
-            Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+                Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+                Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+            }
         }
     }
 
@@ -58,16 +60,18 @@ class ChangesForV470a extends Migration
      */
     public function up(): void
     {
-        try {
+        if (!Schema::hasColumn('transactions', 'reconciled')) {
-            Schema::table(
+            try {
-                'transactions',
+                Schema::table(
-                static function (Blueprint $table) {
+                    'transactions',
-                    $table->boolean('reconciled')->after('deleted_at')->default(0);
+                    static function (Blueprint $table) {
-                }
+                        $table->boolean('reconciled')->after('deleted_at')->default(0);
-            );
+                    }
-        } catch (QueryException $e) {
+                );
-            Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+            } catch (QueryException $e) {
-            Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+                Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+                Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+            }
         }
     }
 }

database/migrations/2018_01_01_000001_create_oauth_auth_codes_table.php

@@ -47,21 +47,23 @@ class CreateOauthAuthCodesTable extends Migration
      */
     public function up(): void
     {
-        try {
+        if(!Schema::hasTable('oauth_auth_codes')) {
-            Schema::create(
+            try {
-                'oauth_auth_codes',
+                Schema::create(
-                static function (Blueprint $table) {
+                    'oauth_auth_codes',
-                    $table->string('id', 100)->primary();
+                    static function (Blueprint $table) {
-                    $table->integer('user_id');
+                        $table->string('id', 100)->primary();
-                    $table->integer('client_id');
+                        $table->integer('user_id');
-                    $table->text('scopes')->nullable();
+                        $table->integer('client_id');
-                    $table->boolean('revoked');
+                        $table->text('scopes')->nullable();
-                    $table->dateTime('expires_at')->nullable();
+                        $table->boolean('revoked');
-                }
+                        $table->dateTime('expires_at')->nullable();
-            );
+                    }
-        } catch (QueryException $e) {
+                );
-            Log::error(sprintf('Could not create table "oauth_auth_codes": %s', $e->getMessage()));
+            } catch (QueryException $e) {
-            Log::error('If this table exists already (see the error message), this is not a problem. Other errors? Please open a discussion on GitHub.');
+                Log::error(sprintf('Could not create table "oauth_auth_codes": %s', $e->getMessage()));
+                Log::error('If this table exists already (see the error message), this is not a problem. Other errors? Please open a discussion on GitHub.');
+            }
         }
     }
 }

database/migrations/2018_01_01_000002_create_oauth_access_tokens_table.php

@@ -47,23 +47,25 @@ class CreateOauthAccessTokensTable extends Migration
      */
     public function up(): void
     {
-        try {
+        if(!Schema::hasTable('oauth_access_tokens')) {
-            Schema::create(
+            try {
-                'oauth_access_tokens',
+                Schema::create(
-                static function (Blueprint $table) {
+                    'oauth_access_tokens',
-                    $table->string('id', 100)->primary();
+                    static function (Blueprint $table) {
-                    $table->integer('user_id')->index()->nullable();
+                        $table->string('id', 100)->primary();
-                    $table->integer('client_id');
+                        $table->integer('user_id')->index()->nullable();
-                    $table->string('name')->nullable();
+                        $table->integer('client_id');
-                    $table->text('scopes')->nullable();
+                        $table->string('name')->nullable();
-                    $table->boolean('revoked');
+                        $table->text('scopes')->nullable();
-                    $table->timestamps();
+                        $table->boolean('revoked');
-                    $table->dateTime('expires_at')->nullable();
+                        $table->timestamps();
-                }
+                        $table->dateTime('expires_at')->nullable();
-            );
+                    }
-        } catch (QueryException $e) {
+                );
-            Log::error(sprintf('Could not create table "oauth_access_tokens": %s', $e->getMessage()));
+            } catch (QueryException $e) {
-            Log::error('If this table exists already (see the error message), this is not a problem. Other errors? Please open a discussion on GitHub.');
+                Log::error(sprintf('Could not create table "oauth_access_tokens": %s', $e->getMessage()));
+                Log::error('If this table exists already (see the error message), this is not a problem. Other errors? Please open a discussion on GitHub.');
+            }
         }
     }
 }

database/migrations/2018_01_01_000003_create_oauth_refresh_tokens_table.php

@@ -47,19 +47,21 @@ class CreateOauthRefreshTokensTable extends Migration
      */
     public function up(): void
     {
-        try {
+        if(!Schema::hasTable('oauth_refresh_tokens')) {
-            Schema::create(
+            try {
-                'oauth_refresh_tokens',
+                Schema::create(
-                static function (Blueprint $table) {
+                    'oauth_refresh_tokens',
-                    $table->string('id', 100)->primary();
+                    static function (Blueprint $table) {
-                    $table->string('access_token_id', 100)->index();
+                        $table->string('id', 100)->primary();
-                    $table->boolean('revoked');
+                        $table->string('access_token_id', 100)->index();
-                    $table->dateTime('expires_at')->nullable();
+                        $table->boolean('revoked');
-                }
+                        $table->dateTime('expires_at')->nullable();
-            );
+                    }
-        } catch (QueryException $e) {
+                );
-            Log::error(sprintf('Could not create table "oauth_refresh_tokens": %s', $e->getMessage()));
+            } catch (QueryException $e) {
-            Log::error('If this table exists already (see the error message), this is not a problem. Other errors? Please open a discussion on GitHub.');
+                Log::error(sprintf('Could not create table "oauth_refresh_tokens": %s', $e->getMessage()));
+                Log::error('If this table exists already (see the error message), this is not a problem. Other errors? Please open a discussion on GitHub.');
+            }
         }
     }
 }

database/migrations/2018_01_01_000004_create_oauth_clients_table.php

@@ -47,24 +47,26 @@ class CreateOauthClientsTable extends Migration
      */
     public function up(): void
     {
-        try {
+        if(!Schema::hasTable('oauth_clients')) {
-            Schema::create(
+            try {
-                'oauth_clients',
+                Schema::create(
-                static function (Blueprint $table) {
+                    'oauth_clients',
-                    $table->increments('id');
+                    static function (Blueprint $table) {
-                    $table->integer('user_id')->index()->nullable();
+                        $table->increments('id');
-                    $table->string('name');
+                        $table->integer('user_id')->index()->nullable();
-                    $table->string('secret', 100);
+                        $table->string('name');
-                    $table->text('redirect');
+                        $table->string('secret', 100);
-                    $table->boolean('personal_access_client');
+                        $table->text('redirect');
-                    $table->boolean('password_client');
+                        $table->boolean('personal_access_client');
-                    $table->boolean('revoked');
+                        $table->boolean('password_client');
-                    $table->timestamps();
+                        $table->boolean('revoked');
-                }
+                        $table->timestamps();
-            );
+                    }
-        } catch (QueryException $e) {
+                );
-            Log::error(sprintf('Could not create table "oauth_clients": %s', $e->getMessage()));
+            } catch (QueryException $e) {
-            Log::error('If this table exists already (see the error message), this is not a problem. Other errors? Please open a discussion on GitHub.');
+                Log::error(sprintf('Could not create table "oauth_clients": %s', $e->getMessage()));
+                Log::error('If this table exists already (see the error message), this is not a problem. Other errors? Please open a discussion on GitHub.');
+            }
         }
     }
 }

database/migrations/2018_01_01_000005_create_oauth_personal_access_clients_table.php

@@ -47,18 +47,20 @@ class CreateOauthPersonalAccessClientsTable extends Migration
      */
     public function up(): void
     {
-        try {
+        if(!Schema::hasTable('oauth_personal_access_clients')) {
-            Schema::create(
+            try {
-                'oauth_personal_access_clients',
+                Schema::create(
-                static function (Blueprint $table) {
+                    'oauth_personal_access_clients',
-                    $table->increments('id');
+                    static function (Blueprint $table) {
-                    $table->integer('client_id')->index();
+                        $table->increments('id');
-                    $table->timestamps();
+                        $table->integer('client_id')->index();
-                }
+                        $table->timestamps();
-            );
+                    }
-        } catch (QueryException $e) {
+                );
-            Log::error(sprintf('Could not create table "oauth_personal_access_clients": %s', $e->getMessage()));
+            } catch (QueryException $e) {
-            Log::error('If this table exists already (see the error message), this is not a problem. Other errors? Please open a discussion on GitHub.');
+                Log::error(sprintf('Could not create table "oauth_personal_access_clients": %s', $e->getMessage()));
+                Log::error('If this table exists already (see the error message), this is not a problem. Other errors? Please open a discussion on GitHub.');
+            }
         }
     }
 }

database/migrations/2018_03_19_141348_changes_for_v472.php

@@ -41,28 +41,32 @@ class ChangesForV472 extends Migration
      */
     public function down(): void
     {
-        try {
+        if(!Schema::hasColumn('attachments', 'notes')) {
-            Schema::table(
+            try {
-                'attachments',
+                Schema::table(
-                static function (Blueprint $table) {
+                    'attachments',
-                    $table->text('notes')->nullable();
+                    static function (Blueprint $table) {
-                }
+                        $table->text('notes')->nullable();
-            );
+                    }
-        } catch (QueryException $e) {
+                );
-            Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+            } catch (QueryException $e) {
-            Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+                Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+                Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+            }
         }
 
-        try {
+        if(Schema::hasColumn('transactions', 'order')) {
-            Schema::table(
+            try {
-                'budgets',
+                Schema::table(
-                static function (Blueprint $table) {
+                    'budgets',
-                    $table->dropColumn('order');
+                    static function (Blueprint $table) {
-                }
+                        $table->dropColumn('order');
-            );
+                    }
-        } catch (QueryException|ColumnDoesNotExist $e) {
+                );
-            Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+            } catch (QueryException|ColumnDoesNotExist $e) {
-            Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+                Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+                Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+            }
         }
     }
 
@@ -73,28 +77,32 @@ class ChangesForV472 extends Migration
      */
     public function up(): void
     {
-        try {
+        if(Schema::hasColumn('attachments', 'notes')) {
-            Schema::table(
+            try {
-                'attachments',
+                Schema::table(
-                static function (Blueprint $table) {
+                    'attachments',
-                    $table->dropColumn('notes');
+                    static function (Blueprint $table) {
-                }
+                        $table->dropColumn('notes');
-            );
+                    }
-        } catch (QueryException|ColumnDoesNotExist $e) {
+                );
-            Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+            } catch (QueryException|ColumnDoesNotExist $e) {
-            Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+                Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+                Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+            }
         }
 
-        try {
+        if(!Schema::hasColumn('budgets', 'order')) {
-            Schema::table(
+            try {
-                'budgets',
+                Schema::table(
-                static function (Blueprint $table) {
+                    'budgets',
-                    $table->mediumInteger('order', false, true)->default(0);
+                    static function (Blueprint $table) {
-                }
+                        $table->mediumInteger('order', false, true)->default(0);
-            );
+                    }
-        } catch (QueryException $e) {
+                );
-            Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+            } catch (QueryException $e) {
-            Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+                Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+                Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+            }
         }
     }
 }

database/migrations/2018_04_07_210913_changes_for_v473.php

@@ -42,33 +42,36 @@ class ChangesForV473 extends Migration
      */
     public function down(): void
     {
-        try {
+        if(!Schema::hasColumn('bills', 'transaction_currency_id')) {
-            Schema::table(
+            try {
-                'bills',
+                Schema::table(
-                static function (Blueprint $table) {
+                    'bills',
-                    // cannot drop foreign keys in SQLite:
+                    static function (Blueprint $table) {
-                    if ('sqlite' !== config('database.default')) {
+                        // cannot drop foreign keys in SQLite:
-                        $table->dropForeign('bills_transaction_currency_id_foreign');
+                        if ('sqlite' !== config('database.default')) {
+                            $table->dropForeign('bills_transaction_currency_id_foreign');
+                        }
+                        $table->dropColumn('transaction_currency_id');
                     }
-                    $table->dropColumn('transaction_currency_id');
+                );
-                }
+            } catch (QueryException|ColumnDoesNotExist $e) {
-            );
+                Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
-        } catch (QueryException|ColumnDoesNotExist $e) {
+                Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
-            Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+            }
-            Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
         }
 
-
+        if(!Schema::hasColumn('rules', 'strict')) {
-        try {
+            try {
-            Schema::table(
+                Schema::table(
-                'rules',
+                    'rules',
-                static function (Blueprint $table) {
+                    static function (Blueprint $table) {
-                    $table->dropColumn('strict');
+                        $table->dropColumn('strict');
-                }
+                    }
-            );
+                );
-        } catch (QueryException|ColumnDoesNotExist $e) {
+            } catch (QueryException|ColumnDoesNotExist $e) {
-            Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+                Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
-            Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+                Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+            }
         }
     }
 
@@ -79,28 +82,32 @@ class ChangesForV473 extends Migration
      */
     public function up(): void
     {
-        try {
+        if(!Schema::hasColumn('bills', 'transaction_currency_id')) {
-            Schema::table(
+            try {
-                'bills',
+                Schema::table(
-                static function (Blueprint $table) {
+                    'bills',
-                    $table->integer('transaction_currency_id', false, true)->nullable()->after('user_id');
+                    static function (Blueprint $table) {
-                    $table->foreign('transaction_currency_id')->references('id')->on('transaction_currencies')->onDelete('set null');
+                        $table->integer('transaction_currency_id', false, true)->nullable()->after('user_id');
-                }
+                        $table->foreign('transaction_currency_id')->references('id')->on('transaction_currencies')->onDelete('set null');
-            );
+                    }
-        } catch (QueryException $e) {
+                );
-            Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+            } catch (QueryException $e) {
-            Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+                Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+                Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+            }
         }
-        try {
+        if(!Schema::hasColumn('rules', 'strict')) {
-            Schema::table(
+            try {
-                'rules',
+                Schema::table(
-                static function (Blueprint $table) {
+                    'rules',
-                    $table->boolean('strict')->default(true);
+                    static function (Blueprint $table) {
-                }
+                        $table->boolean('strict')->default(true);
-            );
+                    }
-        } catch (QueryException $e) {
+                );
-            Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+            } catch (QueryException $e) {
-            Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+                Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+                Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+            }
         }
     }
 }

database/migrations/2018_06_08_200526_changes_for_v475.php

@@ -54,122 +54,133 @@ class ChangesForV475 extends Migration
      */
     public function up(): void
     {
-        try {
+        if (!Schema::hasTable('recurrences')) {
-            Schema::create(
+            try {
-                'recurrences',
+                Schema::create(
-                static function (Blueprint $table) {
+                    'recurrences',
-                    $table->increments('id');
+                    static function (Blueprint $table) {
-                    $table->timestamps();
+                        $table->increments('id');
-                    $table->softDeletes();
+                        $table->timestamps();
-                    $table->integer('user_id', false, true);
+                        $table->softDeletes();
-                    $table->integer('transaction_type_id', false, true);
+                        $table->integer('user_id', false, true);
+                        $table->integer('transaction_type_id', false, true);
 
-                    $table->string('title', 1024);
+                        $table->string('title', 1024);
-                    $table->text('description');
+                        $table->text('description');
 
-                    $table->date('first_date');
+                        $table->date('first_date');
-                    $table->date('repeat_until')->nullable();
+                        $table->date('repeat_until')->nullable();
-                    $table->date('latest_date')->nullable();
+                        $table->date('latest_date')->nullable();
-                    $table->smallInteger('repetitions', false, true);
+                        $table->smallInteger('repetitions', false, true);
 
-                    $table->boolean('apply_rules')->default(true);
+                        $table->boolean('apply_rules')->default(true);
-                    $table->boolean('active')->default(true);
+                        $table->boolean('active')->default(true);
 
-                    $table->foreign('user_id')->references('id')->on('users')->onDelete('cascade');
+                        $table->foreign('user_id')->references('id')->on('users')->onDelete('cascade');
-                    $table->foreign('transaction_type_id')->references('id')->on('transaction_types')->onDelete('cascade');
+                        $table->foreign('transaction_type_id')->references('id')->on('transaction_types')->onDelete('cascade');
-                }
+                    }
-            );
+                );
-        } catch (QueryException $e) {
+            } catch (QueryException $e) {
-            Log::error(sprintf('Could not create table "recurrences": %s', $e->getMessage()));
+                Log::error(sprintf('Could not create table "recurrences": %s', $e->getMessage()));
-            Log::error('If this table exists already (see the error message), this is not a problem. Other errors? Please open a discussion on GitHub.');
+                Log::error('If this table exists already (see the error message), this is not a problem. Other errors? Please open a discussion on GitHub.');
+            }
         }
-        try {
+        if (!Schema::hasTable('recurrences_transactions')) {
-            Schema::create(
+            try {
-                'recurrences_transactions',
+                Schema::create(
-                static function (Blueprint $table) {
+                    'recurrences_transactions',
-                    $table->increments('id');
+                    static function (Blueprint $table) {
-                    $table->timestamps();
+                        $table->increments('id');
-                    $table->softDeletes();
+                        $table->timestamps();
-                    $table->integer('recurrence_id', false, true);
+                        $table->softDeletes();
-                    $table->integer('transaction_currency_id', false, true);
+                        $table->integer('recurrence_id', false, true);
-                    $table->integer('foreign_currency_id', false, true)->nullable();
+                        $table->integer('transaction_currency_id', false, true);
-                    $table->integer('source_id', false, true);
+                        $table->integer('foreign_currency_id', false, true)->nullable();
-                    $table->integer('destination_id', false, true);
+                        $table->integer('source_id', false, true);
+                        $table->integer('destination_id', false, true);
 
-                    $table->decimal('amount', 32, 12);
+                        $table->decimal('amount', 32, 12);
-                    $table->decimal('foreign_amount', 32, 12)->nullable();
+                        $table->decimal('foreign_amount', 32, 12)->nullable();
-                    $table->string('description', 1024);
+                        $table->string('description', 1024);
 
-                    $table->foreign('recurrence_id')->references('id')->on('recurrences')->onDelete('cascade');
+                        $table->foreign('recurrence_id')->references('id')->on('recurrences')->onDelete('cascade');
-                    $table->foreign('transaction_currency_id')->references('id')->on('transaction_currencies')->onDelete('cascade');
+                        $table->foreign('transaction_currency_id')->references('id')->on('transaction_currencies')->onDelete('cascade');
-                    $table->foreign('foreign_currency_id')->references('id')->on('transaction_currencies')->onDelete('set null');
+                        $table->foreign('foreign_currency_id')->references('id')->on('transaction_currencies')->onDelete('set null');
-                    $table->foreign('source_id')->references('id')->on('accounts')->onDelete('cascade');
+                        $table->foreign('source_id')->references('id')->on('accounts')->onDelete('cascade');
-                    $table->foreign('destination_id')->references('id')->on('accounts')->onDelete('cascade');
+                        $table->foreign('destination_id')->references('id')->on('accounts')->onDelete('cascade');
-                }
+                    }
-            );
+                );
-        } catch (QueryException $e) {
+            } catch (QueryException $e) {
-            Log::error(sprintf('Could not create table "recurrences_transactions": %s', $e->getMessage()));
+                Log::error(sprintf('Could not create table "recurrences_transactions": %s', $e->getMessage()));
-            Log::error('If this table exists already (see the error message), this is not a problem. Other errors? Please open a discussion on GitHub.');
+                Log::error('If this table exists already (see the error message), this is not a problem. Other errors? Please open a discussion on GitHub.');
+            }
         }
 
-        try {
+        if (!Schema::hasTable('recurrences_repetitions')) {
-            Schema::create(
+            try {
-                'recurrences_repetitions',
+                Schema::create(
-                static function (Blueprint $table) {
+                    'recurrences_repetitions',
-                    $table->increments('id');
+                    static function (Blueprint $table) {
-                    $table->timestamps();
+                        $table->increments('id');
-                    $table->softDeletes();
+                        $table->timestamps();
-                    $table->integer('recurrence_id', false, true);
+                        $table->softDeletes();
-                    $table->string('repetition_type', 50);
+                        $table->integer('recurrence_id', false, true);
-                    $table->string('repetition_moment', 50);
+                        $table->string('repetition_type', 50);
-                    $table->smallInteger('repetition_skip', false, true);
+                        $table->string('repetition_moment', 50);
-                    $table->smallInteger('weekend', false, true);
+                        $table->smallInteger('repetition_skip', false, true);
+                        $table->smallInteger('weekend', false, true);
 
-                    $table->foreign('recurrence_id')->references('id')->on('recurrences')->onDelete('cascade');
+                        $table->foreign('recurrence_id')->references('id')->on('recurrences')->onDelete('cascade');
-                }
+                    }
-            );
+                );
-        } catch (QueryException $e) {
+            } catch (QueryException $e) {
-            Log::error(sprintf('Could not create table "recurrences_repetitions": %s', $e->getMessage()));
+                Log::error(sprintf('Could not create table "recurrences_repetitions": %s', $e->getMessage()));
-            Log::error('If this table exists already (see the error message), this is not a problem. Other errors? Please open a discussion on GitHub.');
+                Log::error('If this table exists already (see the error message), this is not a problem. Other errors? Please open a discussion on GitHub.');
+            }
         }
 
-        try {
+        if (!Schema::hasTable('recurrences_meta')) {
-            Schema::create(
+            try {
-                'recurrences_meta',
+                Schema::create(
-                static function (Blueprint $table) {
+                    'recurrences_meta',
-                    $table->increments('id');
+                    static function (Blueprint $table) {
-                    $table->timestamps();
+                        $table->increments('id');
-                    $table->softDeletes();
+                        $table->timestamps();
-                    $table->integer('recurrence_id', false, true);
+                        $table->softDeletes();
+                        $table->integer('recurrence_id', false, true);
 
-                    $table->string('name', 50);
+                        $table->string('name', 50);
-                    $table->text('value');
+                        $table->text('value');
 
-                    $table->foreign('recurrence_id')->references('id')->on('recurrences')->onDelete('cascade');
+                        $table->foreign('recurrence_id')->references('id')->on('recurrences')->onDelete('cascade');
-                }
+                    }
-            );
+                );
-        } catch (QueryException $e) {
+            } catch (QueryException $e) {
-            Log::error(sprintf('Could not create table "recurrences_meta": %s', $e->getMessage()));
+                Log::error(sprintf('Could not create table "recurrences_meta": %s', $e->getMessage()));
-            Log::error('If this table exists already (see the error message), this is not a problem. Other errors? Please open a discussion on GitHub.');
+                Log::error('If this table exists already (see the error message), this is not a problem. Other errors? Please open a discussion on GitHub.');
+            }
         }
-        try {
-            Schema::create(
-                'rt_meta',
-                static function (Blueprint $table) {
-                    $table->increments('id');
-                    $table->timestamps();
-                    $table->softDeletes();
-                    $table->integer('rt_id', false, true);
 
-                    $table->string('name', 50);
+        if (!Schema::hasTable('rt_meta')) {
-                    $table->text('value');
+            try {
+                Schema::create(
+                    'rt_meta',
+                    static function (Blueprint $table) {
+                        $table->increments('id');
+                        $table->timestamps();
+                        $table->softDeletes();
+                        $table->integer('rt_id', false, true);
 
-                    $table->foreign('rt_id')->references('id')->on('recurrences_transactions')->onDelete('cascade');
+                        $table->string('name', 50);
-                }
+                        $table->text('value');
-            );
+
-        } catch (QueryException $e) {
+                        $table->foreign('rt_id')->references('id')->on('recurrences_transactions')->onDelete('cascade');
-            Log::error(sprintf('Could not create table "rt_meta": %s', $e->getMessage()));
+                    }
-            Log::error('If this table exists already (see the error message), this is not a problem. Other errors? Please open a discussion on GitHub.');
+                );
+            } catch (QueryException $e) {
+                Log::error(sprintf('Could not create table "rt_meta": %s', $e->getMessage()));
+                Log::error('If this table exists already (see the error message), this is not a problem. Other errors? Please open a discussion on GitHub.');
+            }
         }
     }
 }

database/migrations/2018_09_05_195147_changes_for_v477.php

@@ -41,21 +41,23 @@ class ChangesForV477 extends Migration
      */
     public function down(): void
     {
-        try {
+        if(Schema::hasColumn('budget_limits', 'transaction_currency_id')) {
-            Schema::table(
+            try {
-                'budget_limits',
+                Schema::table(
-                static function (Blueprint $table) {
+                    'budget_limits',
-                    // cannot drop foreign keys in SQLite:
+                    static function (Blueprint $table) {
-                    if ('sqlite' !== config('database.default')) {
+                        // cannot drop foreign keys in SQLite:
-                        $table->dropForeign('budget_limits_transaction_currency_id_foreign');
+                        if ('sqlite' !== config('database.default')) {
-                    }
+                            $table->dropForeign('budget_limits_transaction_currency_id_foreign');
+                        }
 
-                    $table->dropColumn(['transaction_currency_id']);
+                        $table->dropColumn(['transaction_currency_id']);
-                }
+                    }
-            );
+                );
-        } catch (QueryException|ColumnDoesNotExist $e) {
+            } catch (QueryException|ColumnDoesNotExist $e) {
-            Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+                Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
-            Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+                Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+            }
         }
     }
 
@@ -66,17 +68,19 @@ class ChangesForV477 extends Migration
      */
     public function up(): void
     {
-        try {
+        if(!Schema::hasColumn('budget_limits', 'transaction_currency_id')) {
-            Schema::table(
+            try {
-                'budget_limits',
+                Schema::table(
-                static function (Blueprint $table) {
+                    'budget_limits',
-                    $table->integer('transaction_currency_id', false, true)->nullable()->after('budget_id');
+                    static function (Blueprint $table) {
-                    $table->foreign('transaction_currency_id')->references('id')->on('transaction_currencies')->onDelete('set null');
+                        $table->integer('transaction_currency_id', false, true)->nullable()->after('budget_id');
-                }
+                        $table->foreign('transaction_currency_id')->references('id')->on('transaction_currencies')->onDelete('set null');
-            );
+                    }
-        } catch (QueryException $e) {
+                );
-            Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+            } catch (QueryException $e) {
-            Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+                Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+                Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+            }
         }
     }
 }

database/migrations/2018_11_06_172532_changes_for_v479.php

@@ -41,16 +41,18 @@ class ChangesForV479 extends Migration
      */
     public function down(): void
     {
-        try {
+        if(Schema::hasColumn('transaction_currencies', 'enabled')) {
-            Schema::table(
+            try {
-                'transaction_currencies',
+                Schema::table(
-                static function (Blueprint $table) {
+                    'transaction_currencies',
-                    $table->dropColumn(['enabled']);
+                    static function (Blueprint $table) {
-                }
+                        $table->dropColumn(['enabled']);
-            );
+                    }
-        } catch (QueryException|ColumnDoesNotExist $e) {
+                );
-            Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+            } catch (QueryException|ColumnDoesNotExist $e) {
-            Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+                Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+                Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+            }
         }
     }
 
@@ -61,16 +63,18 @@ class ChangesForV479 extends Migration
      */
     public function up(): void
     {
-        try {
+        if(!Schema::hasColumn('transaction_currencies', 'enabled')) {
-            Schema::table(
+            try {
-                'transaction_currencies',
+                Schema::table(
-                static function (Blueprint $table) {
+                    'transaction_currencies',
-                    $table->boolean('enabled')->default(0)->after('deleted_at');
+                    static function (Blueprint $table) {
-                }
+                        $table->boolean('enabled')->default(0)->after('deleted_at');
-            );
+                    }
-        } catch (QueryException $e) {
+                );
-            Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+            } catch (QueryException $e) {
-            Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+                Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+                Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+            }
         }
     }
 }

database/migrations/2019_03_11_223700_fix_ldap_configuration.php

@@ -41,16 +41,18 @@ class FixLdapConfiguration extends Migration
      */
     public function down(): void
     {
-        try {
+        if(Schema::hasColumn('users', 'objectguid')) {
-            Schema::table(
+            try {
-                'users',
+                Schema::table(
-                static function (Blueprint $table) {
+                    'users',
-                    $table->dropColumn(['objectguid']);
+                    static function (Blueprint $table) {
-                }
+                        $table->dropColumn(['objectguid']);
-            );
+                    }
-        } catch (QueryException|ColumnDoesNotExist $e) {
+                );
-            Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+            } catch (QueryException|ColumnDoesNotExist $e) {
-            Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+                Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+                Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+            }
         }
     }
 
@@ -65,16 +67,18 @@ class FixLdapConfiguration extends Migration
          * ADLdap2 appears to require the ability to store an objectguid for LDAP users
          * now. To support this, we add the column.
          */
-        try {
+        if(!Schema::hasColumn('users', 'objectguid')) {
-            Schema::table(
+            try {
-                'users',
+                Schema::table(
-                static function (Blueprint $table) {
+                    'users',
-                    $table->uuid('objectguid')->nullable()->after('id');
+                    static function (Blueprint $table) {
-                }
+                        $table->uuid('objectguid')->nullable()->after('id');
-            );
+                    }
-        } catch (QueryException $e) {
+                );
-            Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+            } catch (QueryException $e) {
-            Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+                Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+                Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+            }
         }
     }
 }

database/migrations/2019_03_22_183214_changes_for_v480.php

@@ -40,65 +40,74 @@ class ChangesForV480 extends Migration
      */
     public function down(): void
     {
-        try {
+        // remove group ID
-            Schema::table(
+        if(Schema::hasColumn('transaction_journals', 'transaction_group_id')) {
-                'transaction_journals',
+            try {
-                static function (Blueprint $table) {
+                Schema::table(
-                    // drop transaction_group_id + foreign key.
+                    'transaction_journals',
-                    // cannot drop foreign keys in SQLite:
+                    static function (Blueprint $table) {
-                    if ('sqlite' !== config('database.default')) {
+                        // drop transaction_group_id + foreign key.
+                        // cannot drop foreign keys in SQLite:
+                        if ('sqlite' !== config('database.default')) {
+                            try {
+                                $table->dropForeign('transaction_journals_transaction_group_id_foreign');
+                            } catch (QueryException $e) {
+                                Log::error(sprintf('Could not drop foreign ID: %s', $e->getMessage()));
+                                Log::error('If the foreign ID does not exist (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+                            }
+                        }
                         try {
-                            $table->dropForeign('transaction_journals_transaction_group_id_foreign');
+                            $table->dropColumn('transaction_group_id');
-                        } catch (QueryException $e) {
+                        } catch (QueryException|ColumnDoesNotExist $e) {
-                            Log::error(sprintf('Could not drop foreign ID: %s', $e->getMessage()));
+                            Log::error(sprintf('Could not drop column: %s', $e->getMessage()));
-                            Log::error('If the foreign ID does not exist (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+                            Log::error('If the column does not exist, this is not an problem. Otherwise, please open a GitHub discussion.');
                         }
                     }
-                    try {
+                );
-                        $table->dropColumn('transaction_group_id');
+            } catch (QueryException $e) {
-                    } catch (QueryException|ColumnDoesNotExist $e) {
+                Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
-                        Log::error(sprintf('Could not drop column: %s', $e->getMessage()));
+                Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
-                        Log::error('If the column does not exist, this is not an problem. Otherwise, please open a GitHub discussion.');
+            }
-                    }
-                }
-            );
-        } catch (QueryException $e) {
-            Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
-            Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
         }
 
-        try {
+        // remove 'stop processing' column
-            Schema::table(
+        if(Schema::hasColumn('rule_groups', 'stop_processing')) {
-                'rule_groups',
+            try {
-                static function (Blueprint $table) {
+                Schema::table(
-                    try {
+                    'rule_groups',
-                        $table->dropColumn('stop_processing');
+                    static function (Blueprint $table) {
-                    } catch (QueryException|ColumnDoesNotExist $e) {
+                        try {
-                        Log::error(sprintf('Could not drop column: %s', $e->getMessage()));
+                            $table->dropColumn('stop_processing');
-                        Log::error('If the column does not exist, this is not an problem. Otherwise, please open a GitHub discussion.');
+                        } catch (QueryException|ColumnDoesNotExist $e) {
+                            Log::error(sprintf('Could not drop column: %s', $e->getMessage()));
+                            Log::error('If the column does not exist, this is not an problem. Otherwise, please open a GitHub discussion.');
+                        }
                     }
-                }
+                );
-            );
+            } catch (QueryException $e) {
-        } catch (QueryException $e) {
+                Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
-            Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+                Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
-            Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+            }
         }
 
-        try {
+        // remove 'mfa_secret' column
-            Schema::table(
+        if(Schema::hasColumn('users', 'mfa_secret')) {
-                'users',
+            try {
-                static function (Blueprint $table) {
+                Schema::table(
-                    try {
+                    'users',
-                        $table->dropColumn('mfa_secret');
+                    static function (Blueprint $table) {
-                    } catch (QueryException|ColumnDoesNotExist $e) {
+                        try {
-                        Log::error(sprintf('Could not drop column: %s', $e->getMessage()));
+                            $table->dropColumn('mfa_secret');
-                        Log::error('If the column does not exist, this is not an problem. Otherwise, please open a GitHub discussion.');
+                        } catch (QueryException|ColumnDoesNotExist $e) {
+                            Log::error(sprintf('Could not drop column: %s', $e->getMessage()));
+                            Log::error('If the column does not exist, this is not an problem. Otherwise, please open a GitHub discussion.');
+                        }
                     }
-                }
+                );
-            );
+            } catch (QueryException $e) {
-        } catch (QueryException $e) {
+                Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
-            Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+                Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
-            Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+            }
         }
     }
 
@@ -109,52 +118,63 @@ class ChangesForV480 extends Migration
      */
     public function up(): void
     {
-        try {
+        // add currency_id
-            Schema::table(
+        if(!Schema::hasColumn('transaction_journals', 'transaction_group_id')) {
-                'transaction_journals',
+            try {
-                static function (Blueprint $table) {
+                Schema::table(
-                    $table->integer('transaction_currency_id', false, true)->nullable()->change();
+                    'transaction_journals',
+                    static function (Blueprint $table) {
+                        $table->integer('transaction_currency_id', false, true)->nullable()->change();
 
-                    // add column "group_id" after "transaction_type_id"
+                        // add column "group_id" after "transaction_type_id"
-                    $table->integer('transaction_group_id', false, true)
+                        $table->integer('transaction_group_id', false, true)
-                          ->nullable()->default(null)->after('transaction_type_id');
+                              ->nullable()->default(null)->after('transaction_type_id');
 
-                    // add foreign key for "transaction_group_id"
+                        // add foreign key for "transaction_group_id"
-                    try {
+                        try {
-                        $table->foreign('transaction_group_id')->references('id')->on('transaction_groups')->onDelete('cascade');
+                            $table->foreign('transaction_group_id')->references('id')->on('transaction_groups')->onDelete('cascade');
-                    } catch (QueryException $e) {
+                        } catch (QueryException $e) {
-                        Log::error(sprintf('Could not create foreign index: %s', $e->getMessage()));
+                            Log::error(sprintf('Could not create foreign index: %s', $e->getMessage()));
-                        Log::error(
+                            Log::error(
-                            'If this table exists already (see the error message), this is not a problem. Other errors? Please open a discussion on GitHub.'
+                                'If this table exists already (see the error message), this is not a problem. Other errors? Please open a discussion on GitHub.'
-                        );
+                            );
+                        }
                     }
-                }
+                );
-            );
+            } catch (QueryException $e) {
-        } catch (QueryException $e) {
+                Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
-            Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+                Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
-            Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+            }
         }
-        try {
+
-            Schema::table(
+        // add 'stop processing' column
-                'rule_groups',
+        if(!Schema::hasColumn('rule_groups', 'stop_processing')) {
-                static function (Blueprint $table) {
+            try {
-                    $table->boolean('stop_processing')->default(false);
+                Schema::table(
-                }
+                    'rule_groups',
-            );
+                    static function (Blueprint $table) {
-        } catch (QueryException $e) {
+                        $table->boolean('stop_processing')->default(false);
-            Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+                    }
-            Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+                );
+            } catch (QueryException $e) {
+                Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+                Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+            }
         }
-        try {
+
-            Schema::table(
+        // add 'mfa_secret' column
-                'users',
+        if(!Schema::hasColumn('users', 'mfa_secret')) {
-                static function (Blueprint $table) {
+            try {
-                    $table->string('mfa_secret', 50)->nullable();
+                Schema::table(
-                }
+                    'users',
-            );
+                    static function (Blueprint $table) {
-        } catch (QueryException $e) {
+                        $table->string('mfa_secret', 50)->nullable();
-            Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+                    }
-            Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+                );
+            } catch (QueryException $e) {
+                Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+                Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+            }
         }
     }
 }

database/migrations/2019_12_28_191351_make_locations_table.php

@@ -51,25 +51,27 @@ class MakeLocationsTable extends Migration
      */
     public function up(): void
     {
-        try {
+        if (!Schema::hasTable('locations')) {
-            Schema::create(
+            try {
-                'locations',
+                Schema::create(
-                static function (Blueprint $table) {
+                    'locations',
-                    $table->bigIncrements('id');
+                    static function (Blueprint $table) {
-                    $table->timestamps();
+                        $table->bigIncrements('id');
-                    $table->softDeletes();
+                        $table->timestamps();
+                        $table->softDeletes();
 
-                    $table->integer('locatable_id', false, true);
+                        $table->integer('locatable_id', false, true);
-                    $table->string('locatable_type', 255);
+                        $table->string('locatable_type', 255);
 
-                    $table->decimal('latitude', 12, 8)->nullable();
+                        $table->decimal('latitude', 12, 8)->nullable();
-                    $table->decimal('longitude', 12, 8)->nullable();
+                        $table->decimal('longitude', 12, 8)->nullable();
-                    $table->smallInteger('zoom_level', false, true)->nullable();
+                        $table->smallInteger('zoom_level', false, true)->nullable();
-                }
+                    }
-            );
+                );
-        } catch (QueryException $e) {
+            } catch (QueryException $e) {
-            Log::error(sprintf('Could not create table "locations": %s', $e->getMessage()));
+                Log::error(sprintf('Could not create table "locations": %s', $e->getMessage()));
-            Log::error('If this table exists already (see the error message), this is not a problem. Other errors? Please open a discussion on GitHub.');
+                Log::error('If this table exists already (see the error message), this is not a problem. Other errors? Please open a discussion on GitHub.');
+            }
         }
     }
 }

database/migrations/2020_06_30_202620_changes_for_v530a.php

@@ -42,16 +42,18 @@ class ChangesForV530a extends Migration
      */
     public function down(): void
     {
-        try {
+        if(Schema::hasColumn('bills', 'order')) {
-            Schema::table(
+            try {
-                'bills',
+                Schema::table(
-                static function (Blueprint $table) {
+                    'bills',
-                    $table->dropColumn('order');
+                    static function (Blueprint $table) {
-                }
+                        $table->dropColumn('order');
-            );
+                    }
-        } catch (QueryException|ColumnDoesNotExist $e) {
+                );
-            Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+            } catch (QueryException|ColumnDoesNotExist $e) {
-            Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+                Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+                Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+            }
         }
     }
 
@@ -62,16 +64,18 @@ class ChangesForV530a extends Migration
      */
     public function up(): void
     {
-        try {
+        if(!Schema::hasColumn('bills', 'order')) {
-            Schema::table(
+            try {
-                'bills',
+                Schema::table(
-                static function (Blueprint $table) {
+                    'bills',
-                    $table->integer('order', false, true)->default(0);
+                    static function (Blueprint $table) {
-                }
+                        $table->integer('order', false, true)->default(0);
-            );
+                    }
-        } catch (QueryException $e) {
+                );
-            Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+            } catch (QueryException $e) {
-            Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+                Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+                Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+            }
         }
     }
 }

database/migrations/2020_07_24_162820_changes_for_v540.php

@@ -42,42 +42,47 @@ class ChangesForV540 extends Migration
      */
     public function down(): void
     {
-        try {
+        if(Schema::hasColumn('oauth_clients', 'provider')) {
-            Schema::table(
+            try {
-                'oauth_clients',
+                Schema::table(
-                static function (Blueprint $table) {
+                    'oauth_clients',
-                    $table->dropColumn('provider');
+                    static function (Blueprint $table) {
-                }
+                        $table->dropColumn('provider');
-            );
+                    }
-        } catch (QueryException|ColumnDoesNotExist $e) {
+                );
-            Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+            } catch (QueryException|ColumnDoesNotExist $e) {
-            Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+                Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+                Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+            }
         }
 
-        try {
+        if(Schema::hasColumn('accounts', 'order')) {
-            Schema::table(
+            try {
-                'accounts',
+                Schema::table(
-                static function (Blueprint $table) {
+                    'accounts',
-                    $table->dropColumn('order');
+                    static function (Blueprint $table) {
-                }
+                        $table->dropColumn('order');
-            );
+                    }
-        } catch (QueryException|ColumnDoesNotExist $e) {
+                );
-            Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+            } catch (QueryException|ColumnDoesNotExist $e) {
-            Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+                Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+                Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+            }
         }
+        if(Schema::hasColumn('bills', 'end_date') && Schema::hasColumn('bills', 'extension_date')) {
+            try {
+                Schema::table(
+                    'bills',
+                    static function (Blueprint $table) {
+                        $table->dropColumn('end_date');
 
-        try {
+                        $table->dropColumn('extension_date');
-            Schema::table(
+                    }
-                'bills',
+                );
-                static function (Blueprint $table) {
+            } catch (QueryException|ColumnDoesNotExist $e) {
-                    $table->dropColumn('end_date');
+                Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
-
+                Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
-                    $table->dropColumn('extension_date');
+            }
-                }
-            );
-        } catch (QueryException|ColumnDoesNotExist $e) {
-            Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
-            Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
         }
     }
 
@@ -88,40 +93,49 @@ class ChangesForV540 extends Migration
      */
     public function up(): void
     {
-        try {
+        if(!Schema::hasColumn('accounts', 'order')) {
-            Schema::table(
+            try {
-                'accounts',
+                Schema::table(
-                static function (Blueprint $table) {
+                    'accounts',
-                    $table->integer('order', false, true)->default(0);
+                    static function (Blueprint $table) {
-                }
+                        $table->integer('order', false, true)->default(0);
-            );
+                    }
-        } catch (QueryException $e) {
+                );
-            Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+            } catch (QueryException $e) {
-            Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+                Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+                Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+            }
         }
-        try {
+
-            Schema::table(
+        if(!Schema::hasColumn('oauth_clients', 'provider')) {
-                'oauth_clients',
+            try {
-                static function (Blueprint $table) {
+                Schema::table(
-                    $table->string('provider')->nullable();
+                    'oauth_clients',
-                }
+                    static function (Blueprint $table) {
-            );
+                        $table->string('provider')->nullable();
-        } catch (QueryException $e) {
+                    }
-            Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+                );
-            Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+            } catch (QueryException $e) {
+                Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+                Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+            }
         }
-        try {
+
-            Schema::table(
+        if(!Schema::hasColumn('bills', 'end_date') && !Schema::hasColumn('bills', 'extension_date')) {
-                'bills',
+            try {
-                static function (Blueprint $table) {
+                Schema::table(
-                    $table->date('end_date')->nullable()->after('date');
+                    'bills',
-                    $table->date('extension_date')->nullable()->after('end_date');
+                    static function (Blueprint $table) {
-                }
+                        $table->date('end_date')->nullable()->after('date');
-            );
+                        $table->date('extension_date')->nullable()->after('end_date');
-        } catch (QueryException $e) {
+                    }
-            Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+                );
-            Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+            } catch (QueryException $e) {
+                Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+                Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+            }
         }
+
         // make column nullable:
         try {
             Schema::table(

database/migrations/2020_11_12_070604_changes_for_v550.php

@@ -43,57 +43,65 @@ class ChangesForV550 extends Migration
         // recreate jobs table.
         Schema::dropIfExists('jobs');
 
-        try {
+        if (!Schema::hasTable('jobs')) {
-            Schema::create(
+            try {
-                'jobs',
+                Schema::create(
-                static function (Blueprint $table) {
+                    'jobs',
-                    // straight from Laravel (this is the OLD table)
+                    static function (Blueprint $table) {
-                    $table->bigIncrements('id');
+                        // straight from Laravel (this is the OLD table)
-                    $table->string('queue');
+                        $table->bigIncrements('id');
-                    $table->longText('payload');
+                        $table->string('queue');
-                    $table->tinyInteger('attempts')->unsigned();
+                        $table->longText('payload');
-                    $table->tinyInteger('reserved')->unsigned();
+                        $table->tinyInteger('attempts')->unsigned();
-                    $table->unsignedInteger('reserved_at')->nullable();
+                        $table->tinyInteger('reserved')->unsigned();
-                    $table->unsignedInteger('available_at');
+                        $table->unsignedInteger('reserved_at')->nullable();
-                    $table->unsignedInteger('created_at');
+                        $table->unsignedInteger('available_at');
-                    $table->index(['queue', 'reserved', 'reserved_at']);
+                        $table->unsignedInteger('created_at');
-                }
+                        $table->index(['queue', 'reserved', 'reserved_at']);
-            );
+                    }
-        } catch (QueryException $e) {
+                );
-            Log::error(sprintf('Could not create table "jobs": %s', $e->getMessage()));
+            } catch (QueryException $e) {
-            Log::error('If this table exists already (see the error message), this is not a problem. Other errors? Please open a discussion on GitHub.');
+                Log::error(sprintf('Could not create table "jobs": %s', $e->getMessage()));
+                Log::error('If this table exists already (see the error message), this is not a problem. Other errors? Please open a discussion on GitHub.');
+            }
         }
 
         // expand budget / transaction journal table.
-        try {
+        if(Schema::hasColumn('budget_transaction_journal', 'budget_limit_id')) {
-            Schema::table(
+            try {
-                'budget_transaction_journal',
+                Schema::table(
-                function (Blueprint $table) {
+                    'budget_transaction_journal',
-                    $table->dropForeign('budget_id_foreign');
+                    function (Blueprint $table) {
-                    $table->dropColumn('budget_limit_id');
+                        $table->dropForeign('budget_id_foreign');
-                }
+                        $table->dropColumn('budget_limit_id');
-            );
+                    }
-        } catch (QueryException|ColumnDoesNotExist $e) {
+                );
-            Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+            } catch (QueryException|ColumnDoesNotExist $e) {
-            Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+                Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+                Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+            }
         }
 
         // drop failed jobs table.
         Schema::dropIfExists('failed_jobs');
 
         // drop fields from budget limits
-        try {
+        if(Schema::hasColumn('budget_limits', 'period') && Schema::hasColumn('budget_limits', 'generated')) {
-            Schema::table(
+            try {
-                'budget_limits',
+                Schema::table(
-                static function (Blueprint $table) {
+                    'budget_limits',
-                    $table->dropColumn('period');
+                    static function (Blueprint $table) {
-                    $table->dropColumn('generated');
+                        $table->dropColumn('period');
-                }
+                        $table->dropColumn('generated');
-            );
+                    }
-        } catch (QueryException|ColumnDoesNotExist $e) {
+                );
-            Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+            } catch (QueryException|ColumnDoesNotExist $e) {
-            Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+                Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+                Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+            }
         }
+
+        // drop other tables
         Schema::dropIfExists('webhook_attempts');
         Schema::dropIfExists('webhook_messages');
         Schema::dropIfExists('webhooks');
@@ -109,63 +117,70 @@ class ChangesForV550 extends Migration
         // drop and recreate jobs table.
         Schema::dropIfExists('jobs');
         // this is the NEW table
-        try {
+        if (!Schema::hasTable('jobs')) {
-            Schema::create(
+            try {
-                'jobs',
+                Schema::create(
-                function (Blueprint $table) {
+                    'jobs',
-                    $table->bigIncrements('id');
+                    function (Blueprint $table) {
-                    $table->string('queue')->index();
+                        $table->bigIncrements('id');
-                    $table->longText('payload');
+                        $table->string('queue')->index();
-                    $table->unsignedTinyInteger('attempts');
+                        $table->longText('payload');
-                    $table->unsignedInteger('reserved_at')->nullable();
+                        $table->unsignedTinyInteger('attempts');
-                    $table->unsignedInteger('available_at');
+                        $table->unsignedInteger('reserved_at')->nullable();
-                    $table->unsignedInteger('created_at');
+                        $table->unsignedInteger('available_at');
-                }
+                        $table->unsignedInteger('created_at');
-            );
+                    }
-        } catch (QueryException $e) {
+                );
-            Log::error(sprintf('Could not create table "jobs": %s', $e->getMessage()));
+            } catch (QueryException $e) {
-            Log::error('If this table exists already (see the error message), this is not a problem. Other errors? Please open a discussion on GitHub.');
+                Log::error(sprintf('Could not create table "jobs": %s', $e->getMessage()));
+                Log::error('If this table exists already (see the error message), this is not a problem. Other errors? Please open a discussion on GitHub.');
+            }
         }
         // drop failed jobs table.
         Schema::dropIfExists('failed_jobs');
 
         // create new failed_jobs table.
-        try {
+        if (!Schema::hasTable('failed_jobs')) {
-            Schema::create(
+            try {
-                'failed_jobs',
+                Schema::create(
-                function (Blueprint $table) {
+                    'failed_jobs',
-                    $table->bigIncrements('id');
+                    function (Blueprint $table) {
-                    $table->string('uuid')->unique();
+                        $table->bigIncrements('id');
-                    $table->text('connection');
+                        $table->string('uuid')->unique();
-                    $table->text('queue');
+                        $table->text('connection');
-                    $table->longText('payload');
+                        $table->text('queue');
-                    $table->longText('exception');
+                        $table->longText('payload');
-                    $table->timestamp('failed_at')->useCurrent();
+                        $table->longText('exception');
-                }
+                        $table->timestamp('failed_at')->useCurrent();
-            );
+                    }
-        } catch (QueryException $e) {
+                );
-            Log::error(sprintf('Could not create table "failed_jobs": %s', $e->getMessage()));
+            } catch (QueryException $e) {
-            Log::error('If this table exists already (see the error message), this is not a problem. Other errors? Please open a discussion on GitHub.');
+                Log::error(sprintf('Could not create table "failed_jobs": %s', $e->getMessage()));
+                Log::error('If this table exists already (see the error message), this is not a problem. Other errors? Please open a discussion on GitHub.');
+            }
         }
 
         // update budget / transaction journal table.
-        try {
+        if(!Schema::hasColumn('budget_transaction_journal', 'budget_limit_id')) {
-            Schema::table(
+            try {
-                'budget_transaction_journal',
+                Schema::table(
-                function (Blueprint $table) {
+                    'budget_transaction_journal',
-                    if (!Schema::hasColumn('budget_transaction_journal', 'budget_limit_id')) {
+                    function (Blueprint $table) {
-                        $table->integer('budget_limit_id', false, true)->nullable()->default(null)->after('budget_id');
+                        if (!Schema::hasColumn('budget_transaction_journal', 'budget_limit_id')) {
-                        $table->foreign('budget_limit_id', 'budget_id_foreign')->references('id')->on('budget_limits')->onDelete('set null');
+                            $table->integer('budget_limit_id', false, true)->nullable()->default(null)->after('budget_id');
+                            $table->foreign('budget_limit_id', 'budget_id_foreign')->references('id')->on('budget_limits')->onDelete('set null');
+                        }
                     }
-                }
+                );
-            );
+            } catch (QueryException $e) {
-        } catch (QueryException $e) {
+                Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
-            Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+                Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
-            Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+            }
         }
 
         // append budget limits table.
-        // i swear I dropped & recreated this field 15 times already.
+        // I swear I dropped & recreated this field 15 times already.
+
         try {
             Schema::table(
                 'budget_limits',

database/migrations/2021_03_12_061213_changes_for_v550b2.php

@@ -40,19 +40,21 @@ class ChangesForV550b2 extends Migration
      */
     public function down(): void
     {
-        try {
+        if(Schema::hasColumn('recurrences_transactions', 'transaction_type_id')) {
-            Schema::table(
+            try {
-                'recurrences_transactions',
+                Schema::table(
-                function (Blueprint $table) {
+                    'recurrences_transactions',
-                    $table->dropForeign('type_foreign');
+                    function (Blueprint $table) {
-                    if (Schema::hasColumn('recurrences_transactions', 'transaction_type_id')) {
+                        $table->dropForeign('type_foreign');
-                        $table->dropColumn('transaction_type_id');
+                        if (Schema::hasColumn('recurrences_transactions', 'transaction_type_id')) {
+                            $table->dropColumn('transaction_type_id');
+                        }
                     }
-                }
+                );
-            );
+            } catch (QueryException|ColumnDoesNotExist $e) {
-        } catch (QueryException|ColumnDoesNotExist $e) {
+                Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
-            Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+                Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
-            Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+            }
         }
     }
 
@@ -64,19 +66,21 @@ class ChangesForV550b2 extends Migration
     public function up(): void
     {
         // expand recurrence transaction table
-        try {
+        if(!Schema::hasColumn('recurrences_transactions', 'transaction_type_id')) {
-            Schema::table(
+            try {
-                'recurrences_transactions',
+                Schema::table(
-                function (Blueprint $table) {
+                    'recurrences_transactions',
-                    if (!Schema::hasColumn('recurrences_transactions', 'transaction_type_id')) {
+                    function (Blueprint $table) {
-                        $table->integer('transaction_type_id', false, true)->nullable()->after('transaction_currency_id');
+                        if (!Schema::hasColumn('recurrences_transactions', 'transaction_type_id')) {
-                        $table->foreign('transaction_type_id', 'type_foreign')->references('id')->on('transaction_types')->onDelete('set null');
+                            $table->integer('transaction_type_id', false, true)->nullable()->after('transaction_currency_id');
+                            $table->foreign('transaction_type_id', 'type_foreign')->references('id')->on('transaction_types')->onDelete('set null');
+                        }
                     }
-                }
+                );
-            );
+            } catch (QueryException $e) {
-        } catch (QueryException $e) {
+                Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
-            Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+                Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
-            Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+            }
         }
     }
 }

database/migrations/2021_05_09_064644_add_ldap_columns_to_users_table.php

@@ -35,16 +35,18 @@ class AddLdapColumnsToUsersTable extends Migration
      */
     public function down(): void
     {
-        try {
+        if(Schema::hasColumn('users', 'domain')) {
-            Schema::table(
+            try {
-                'users',
+                Schema::table(
-                function (Blueprint $table) {
+                    'users',
-                    $table->dropColumn(['domain']);
+                    function (Blueprint $table) {
-                }
+                        $table->dropColumn(['domain']);
-            );
+                    }
-        } catch (QueryException|ColumnDoesNotExist $e) {
+                );
-            Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+            } catch (QueryException|ColumnDoesNotExist $e) {
-            Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+                Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+                Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+            }
         }
     }
 
@@ -53,16 +55,18 @@ class AddLdapColumnsToUsersTable extends Migration
      */
     public function up(): void
     {
-        try {
+        if(!Schema::hasColumn('users', 'domain')) {
-            Schema::table(
+            try {
-                'users',
+                Schema::table(
-                function (Blueprint $table) {
+                    'users',
-                    $table->string('domain')->nullable();
+                    function (Blueprint $table) {
-                }
+                        $table->string('domain')->nullable();
-            );
+                    }
-        } catch (QueryException $e) {
+                );
-            Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+            } catch (QueryException $e) {
-            Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+                Log::error(sprintf('Could not execute query: %s', $e->getMessage()));
+                Log::error('If the column or index already exists (see error), this is not an problem. Otherwise, please open a GitHub discussion.');
+            }
         }
     }
 }