diff --git a/database/migrations/2016_06_16_000000_create_support_tables.php b/database/migrations/2016_06_16_000000_create_support_tables.php index a063f16276..d1580d5f4d 100644 --- a/database/migrations/2016_06_16_000000_create_support_tables.php +++ b/database/migrations/2016_06_16_000000_create_support_tables.php @@ -77,7 +77,7 @@ class CreateSupportTables extends Migration try { Schema::create( 'account_types', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->increments('id'); $table->timestamps(); $table->string('type', 50); @@ -99,7 +99,7 @@ class CreateSupportTables extends Migration try { Schema::create( 'transaction_currencies', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->increments('id'); $table->timestamps(); $table->softDeletes(); @@ -124,7 +124,7 @@ class CreateSupportTables extends Migration try { Schema::create( 'transaction_types', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->increments('id'); $table->timestamps(); $table->softDeletes(); @@ -147,7 +147,7 @@ class CreateSupportTables extends Migration try { Schema::create( 'jobs', - static function (Blueprint $table) { + static function (Blueprint $table): void { // straight from Laravel $table->bigIncrements('id'); $table->string('queue'); @@ -173,7 +173,7 @@ class CreateSupportTables extends Migration try { Schema::create( 'password_resets', - static function (Blueprint $table) { + static function (Blueprint $table): void { // straight from laravel $table->string('email')->index(); $table->string('token')->index(); @@ -193,7 +193,7 @@ class CreateSupportTables extends Migration try { Schema::create( 'permissions', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->increments('id'); $table->timestamps(); $table->string('name')->unique(); @@ -214,7 +214,7 @@ class CreateSupportTables extends Migration try { Schema::create( 'roles', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->increments('id'); $table->timestamps(); $table->string('name')->unique(); @@ -235,7 +235,7 @@ class CreateSupportTables extends Migration try { Schema::create( 'permission_role', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->integer('permission_id')->unsigned(); $table->integer('role_id')->unsigned(); @@ -258,7 +258,7 @@ class CreateSupportTables extends Migration try { Schema::create( 'sessions', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->string('id')->unique(); $table->integer('user_id')->nullable(); $table->string('ip_address', 45)->nullable(); @@ -280,7 +280,7 @@ class CreateSupportTables extends Migration try { Schema::create( 'configuration', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->increments('id'); $table->timestamps(); $table->softDeletes(); diff --git a/database/migrations/2016_06_16_000001_create_users_table.php b/database/migrations/2016_06_16_000001_create_users_table.php index 7d3d372efa..5002e26302 100644 --- a/database/migrations/2016_06_16_000001_create_users_table.php +++ b/database/migrations/2016_06_16_000001_create_users_table.php @@ -54,7 +54,7 @@ class CreateUsersTable extends Migration try { Schema::create( 'users', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->increments('id'); $table->timestamps(); $table->string('email', 255); diff --git a/database/migrations/2016_06_16_000002_create_main_tables.php b/database/migrations/2016_06_16_000002_create_main_tables.php index 1a94293c6a..2a0208a514 100644 --- a/database/migrations/2016_06_16_000002_create_main_tables.php +++ b/database/migrations/2016_06_16_000002_create_main_tables.php @@ -96,7 +96,7 @@ class CreateMainTables extends Migration try { Schema::create( 'accounts', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->increments('id'); $table->timestamps(); $table->softDeletes(); @@ -121,7 +121,7 @@ class CreateMainTables extends Migration try { Schema::create( 'account_meta', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->increments('id'); $table->timestamps(); $table->integer('account_id', false, true); @@ -143,7 +143,7 @@ class CreateMainTables extends Migration try { Schema::create( 'piggy_banks', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->increments('id'); $table->timestamps(); $table->softDeletes(); @@ -168,7 +168,7 @@ class CreateMainTables extends Migration try { Schema::create( 'piggy_bank_repetitions', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->increments('id'); $table->timestamps(); $table->integer('piggy_bank_id', false, true); @@ -191,7 +191,7 @@ class CreateMainTables extends Migration try { Schema::create( 'attachments', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->increments('id'); $table->timestamps(); $table->softDeletes(); @@ -224,7 +224,7 @@ class CreateMainTables extends Migration try { Schema::create( 'bills', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->increments('id'); $table->timestamps(); $table->softDeletes(); @@ -258,7 +258,7 @@ class CreateMainTables extends Migration try { Schema::create( 'budgets', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->increments('id'); $table->timestamps(); $table->softDeletes(); @@ -278,7 +278,7 @@ class CreateMainTables extends Migration try { Schema::create( 'budget_limits', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->increments('id'); $table->timestamps(); $table->integer('budget_id', false, true); @@ -298,7 +298,7 @@ class CreateMainTables extends Migration try { Schema::create( 'limit_repetitions', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->increments('id'); $table->timestamps(); $table->integer('budget_limit_id', false, true); @@ -321,7 +321,7 @@ class CreateMainTables extends Migration try { Schema::create( 'categories', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->increments('id'); $table->timestamps(); $table->softDeletes(); @@ -346,7 +346,7 @@ class CreateMainTables extends Migration try { Schema::create( 'preferences', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->increments('id'); $table->timestamps(); $table->integer('user_id', false, true); @@ -369,7 +369,7 @@ class CreateMainTables extends Migration try { Schema::create( 'role_user', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->integer('user_id', false, true); $table->integer('role_id', false, true); @@ -392,7 +392,7 @@ class CreateMainTables extends Migration try { Schema::create( 'rule_groups', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->increments('id'); $table->timestamps(); $table->softDeletes(); @@ -415,7 +415,7 @@ class CreateMainTables extends Migration try { Schema::create( 'rules', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->increments('id'); $table->timestamps(); $table->softDeletes(); @@ -443,7 +443,7 @@ class CreateMainTables extends Migration try { Schema::create( 'rule_actions', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->increments('id'); $table->timestamps(); $table->integer('rule_id', false, true); @@ -468,7 +468,7 @@ class CreateMainTables extends Migration try { Schema::create( 'rule_triggers', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->increments('id'); $table->timestamps(); $table->integer('rule_id', false, true); @@ -497,7 +497,7 @@ class CreateMainTables extends Migration try { Schema::create( 'tags', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->increments('id'); $table->timestamps(); $table->softDeletes(); @@ -528,7 +528,7 @@ class CreateMainTables extends Migration try { Schema::create( 'transaction_journals', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->increments('id'); $table->timestamps(); $table->softDeletes(); @@ -561,7 +561,7 @@ class CreateMainTables extends Migration try { Schema::create( 'journal_meta', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->increments('id'); $table->timestamps(); $table->integer('transaction_journal_id', false, true); @@ -581,7 +581,7 @@ class CreateMainTables extends Migration try { Schema::create( 'tag_transaction_journal', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->increments('id'); $table->integer('tag_id', false, true); $table->integer('transaction_journal_id', false, true); @@ -602,7 +602,7 @@ class CreateMainTables extends Migration try { Schema::create( 'budget_transaction_journal', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->increments('id'); $table->integer('budget_id', false, true); $table->integer('transaction_journal_id', false, true); @@ -620,7 +620,7 @@ class CreateMainTables extends Migration try { Schema::create( 'category_transaction_journal', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->increments('id'); $table->integer('category_id', false, true); $table->integer('transaction_journal_id', false, true); @@ -638,7 +638,7 @@ class CreateMainTables extends Migration try { Schema::create( 'piggy_bank_events', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->increments('id'); $table->timestamps(); $table->integer('piggy_bank_id', false, true); @@ -660,7 +660,7 @@ class CreateMainTables extends Migration try { Schema::create( 'transactions', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->increments('id'); $table->timestamps(); $table->softDeletes(); @@ -683,7 +683,7 @@ class CreateMainTables extends Migration try { Schema::create( 'budget_transaction', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->increments('id'); $table->integer('budget_id', false, true); $table->integer('transaction_id', false, true); @@ -702,7 +702,7 @@ class CreateMainTables extends Migration try { Schema::create( 'category_transaction', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->increments('id'); $table->integer('category_id', false, true); $table->integer('transaction_id', false, true); diff --git a/database/migrations/2016_09_12_121359_fix_nullables.php b/database/migrations/2016_09_12_121359_fix_nullables.php index 90277e8c08..3f8d858915 100644 --- a/database/migrations/2016_09_12_121359_fix_nullables.php +++ b/database/migrations/2016_09_12_121359_fix_nullables.php @@ -51,7 +51,7 @@ class FixNullables extends Migration try { Schema::table( 'rule_groups', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->text('description')->nullable()->change(); } ); @@ -65,7 +65,7 @@ class FixNullables extends Migration try { Schema::table( 'rules', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->text('description')->nullable()->change(); } ); diff --git a/database/migrations/2016_10_09_150037_expand_transactions_table.php b/database/migrations/2016_10_09_150037_expand_transactions_table.php index fa5dc6bb8f..29a356f96d 100644 --- a/database/migrations/2016_10_09_150037_expand_transactions_table.php +++ b/database/migrations/2016_10_09_150037_expand_transactions_table.php @@ -42,7 +42,7 @@ class ExpandTransactionsTable extends Migration try { Schema::table( 'transactions', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->dropColumn('identifier'); } ); @@ -64,7 +64,7 @@ class ExpandTransactionsTable extends Migration try { Schema::table( 'transactions', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->smallInteger('identifier', false, true)->default(0); } ); diff --git a/database/migrations/2016_10_22_075804_changes_for_v410.php b/database/migrations/2016_10_22_075804_changes_for_v410.php index cb722eae7f..417bc70afe 100644 --- a/database/migrations/2016_10_22_075804_changes_for_v410.php +++ b/database/migrations/2016_10_22_075804_changes_for_v410.php @@ -51,7 +51,7 @@ class ChangesForV410 extends Migration try { Schema::create( 'notes', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->increments('id'); $table->timestamps(); $table->softDeletes(); diff --git a/database/migrations/2016_11_24_210552_changes_for_v420.php b/database/migrations/2016_11_24_210552_changes_for_v420.php index f66cce7988..54a8c898c3 100644 --- a/database/migrations/2016_11_24_210552_changes_for_v420.php +++ b/database/migrations/2016_11_24_210552_changes_for_v420.php @@ -41,7 +41,7 @@ class ChangesForV420 extends Migration try { Schema::table( 'journal_meta', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->dropSoftDeletes(); } ); @@ -63,7 +63,7 @@ class ChangesForV420 extends Migration try { Schema::table( 'journal_meta', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->softDeletes(); } ); diff --git a/database/migrations/2016_12_22_150431_changes_for_v430.php b/database/migrations/2016_12_22_150431_changes_for_v430.php index 39538f84b7..dce36a973d 100644 --- a/database/migrations/2016_12_22_150431_changes_for_v430.php +++ b/database/migrations/2016_12_22_150431_changes_for_v430.php @@ -51,7 +51,7 @@ class ChangesForV430 extends Migration try { Schema::create( 'available_budgets', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->increments('id'); $table->timestamps(); $table->softDeletes(); diff --git a/database/migrations/2016_12_28_203205_changes_for_v431.php b/database/migrations/2016_12_28_203205_changes_for_v431.php index e33c166654..8c20187715 100644 --- a/database/migrations/2016_12_28_203205_changes_for_v431.php +++ b/database/migrations/2016_12_28_203205_changes_for_v431.php @@ -43,7 +43,7 @@ class ChangesForV431 extends Migration try { Schema::table( 'budget_limits', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->string('repeat_freq', 30)->nullable(); } ); @@ -56,7 +56,7 @@ class ChangesForV431 extends Migration try { Schema::table( 'budget_limits', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->boolean('repeats')->default(0); } ); @@ -70,7 +70,7 @@ class ChangesForV431 extends Migration try { Schema::table( 'budget_limits', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->renameColumn('start_date', 'startdate'); } ); @@ -85,7 +85,7 @@ class ChangesForV431 extends Migration try { Schema::table( 'budget_limits', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->dropColumn('end_date'); } ); @@ -99,7 +99,7 @@ class ChangesForV431 extends Migration try { Schema::table( 'transaction_currencies', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->dropColumn('decimal_places'); } ); @@ -122,7 +122,7 @@ class ChangesForV431 extends Migration try { Schema::table( 'transaction_currencies', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->smallInteger('decimal_places', false, true)->default(2); } ); @@ -137,7 +137,7 @@ class ChangesForV431 extends Migration try { Schema::table( 'budget_limits', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->renameColumn('startdate', 'start_date'); } ); @@ -152,7 +152,7 @@ class ChangesForV431 extends Migration try { Schema::table( 'budget_limits', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->date('end_date')->nullable()->after('start_date'); } ); @@ -167,7 +167,7 @@ class ChangesForV431 extends Migration try { Schema::table( 'budget_limits', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->dropColumn('repeats'); } ); @@ -180,7 +180,7 @@ class ChangesForV431 extends Migration try { Schema::table( 'budget_limits', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->dropColumn('repeat_freq'); } ); diff --git a/database/migrations/2017_04_13_163623_changes_for_v440.php b/database/migrations/2017_04_13_163623_changes_for_v440.php index 39970e7b17..075c03181c 100644 --- a/database/migrations/2017_04_13_163623_changes_for_v440.php +++ b/database/migrations/2017_04_13_163623_changes_for_v440.php @@ -43,7 +43,7 @@ class ChangesForV440 extends Migration try { Schema::table( 'transactions', - static function (Blueprint $table) { + static function (Blueprint $table): void { if (Schema::hasColumn('transactions', 'transaction_currency_id')) { // cannot drop foreign keys in SQLite: if ('sqlite' !== config('database.default')) { @@ -70,7 +70,7 @@ class ChangesForV440 extends Migration try { Schema::create( 'currency_exchange_rates', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->increments('id'); $table->timestamps(); $table->softDeletes(); @@ -95,7 +95,7 @@ class ChangesForV440 extends Migration try { Schema::table( 'transactions', - static function (Blueprint $table) { + static function (Blueprint $table): void { if (!Schema::hasColumn('transactions', 'transaction_currency_id')) { $table->integer('transaction_currency_id', false, true)->after('description')->nullable(); $table->foreign('transaction_currency_id')->references('id')->on('transaction_currencies')->onDelete('set null'); diff --git a/database/migrations/2017_06_02_105232_changes_for_v450.php b/database/migrations/2017_06_02_105232_changes_for_v450.php index 8fbb89c9a1..b1e5267036 100644 --- a/database/migrations/2017_06_02_105232_changes_for_v450.php +++ b/database/migrations/2017_06_02_105232_changes_for_v450.php @@ -43,7 +43,7 @@ class ChangesForV450 extends Migration try { Schema::table( 'transactions', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->dropColumn('foreign_amount'); } ); @@ -56,7 +56,7 @@ class ChangesForV450 extends Migration try { Schema::table( 'transactions', - static function (Blueprint $table) { + static function (Blueprint $table): void { // cannot drop foreign keys in SQLite: if ('sqlite' !== config('database.default')) { $table->dropForeign('transactions_foreign_currency_id_foreign'); @@ -71,7 +71,7 @@ class ChangesForV450 extends Migration try { Schema::table( 'transactions', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->dropColumn('foreign_currency_id'); } ); @@ -94,7 +94,7 @@ class ChangesForV450 extends Migration try { Schema::table( 'transactions', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->decimal('foreign_amount', 32, 12)->nullable()->after('amount'); } ); @@ -109,7 +109,7 @@ class ChangesForV450 extends Migration try { Schema::table( 'transactions', - static function (Blueprint $table) { + static function (Blueprint $table): void { $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'); } diff --git a/database/migrations/2017_08_20_062014_changes_for_v470.php b/database/migrations/2017_08_20_062014_changes_for_v470.php index 3ac4a9d78f..0c3794cda4 100644 --- a/database/migrations/2017_08_20_062014_changes_for_v470.php +++ b/database/migrations/2017_08_20_062014_changes_for_v470.php @@ -53,7 +53,7 @@ class ChangesForV470 extends Migration try { Schema::create( 'link_types', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->increments('id'); $table->timestamps(); $table->softDeletes(); @@ -75,7 +75,7 @@ class ChangesForV470 extends Migration try { Schema::create( 'journal_links', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->increments('id'); $table->timestamps(); $table->integer('link_type_id', false, true); diff --git a/database/migrations/2017_11_04_170844_changes_for_v470a.php b/database/migrations/2017_11_04_170844_changes_for_v470a.php index b174b72aad..01e7460537 100644 --- a/database/migrations/2017_11_04_170844_changes_for_v470a.php +++ b/database/migrations/2017_11_04_170844_changes_for_v470a.php @@ -43,7 +43,7 @@ class ChangesForV470a extends Migration try { Schema::table( 'transactions', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->dropColumn('reconciled'); } ); @@ -65,7 +65,7 @@ class ChangesForV470a extends Migration try { Schema::table( 'transactions', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->boolean('reconciled')->after('deleted_at')->default(0); } ); diff --git a/database/migrations/2018_01_01_000001_create_oauth_auth_codes_table.php b/database/migrations/2018_01_01_000001_create_oauth_auth_codes_table.php index 1f77365235..cb70131bf2 100644 --- a/database/migrations/2018_01_01_000001_create_oauth_auth_codes_table.php +++ b/database/migrations/2018_01_01_000001_create_oauth_auth_codes_table.php @@ -53,7 +53,7 @@ class CreateOauthAuthCodesTable extends Migration try { Schema::create( 'oauth_auth_codes', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->string('id', 100)->primary(); $table->integer('user_id'); $table->integer('client_id'); diff --git a/database/migrations/2018_01_01_000002_create_oauth_access_tokens_table.php b/database/migrations/2018_01_01_000002_create_oauth_access_tokens_table.php index 846d6cded8..cc41b105bd 100644 --- a/database/migrations/2018_01_01_000002_create_oauth_access_tokens_table.php +++ b/database/migrations/2018_01_01_000002_create_oauth_access_tokens_table.php @@ -53,7 +53,7 @@ class CreateOauthAccessTokensTable extends Migration try { Schema::create( 'oauth_access_tokens', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->string('id', 100)->primary(); $table->integer('user_id')->index()->nullable(); $table->integer('client_id'); diff --git a/database/migrations/2018_01_01_000003_create_oauth_refresh_tokens_table.php b/database/migrations/2018_01_01_000003_create_oauth_refresh_tokens_table.php index 26a44fdd0e..3b91d74abc 100644 --- a/database/migrations/2018_01_01_000003_create_oauth_refresh_tokens_table.php +++ b/database/migrations/2018_01_01_000003_create_oauth_refresh_tokens_table.php @@ -53,7 +53,7 @@ class CreateOauthRefreshTokensTable extends Migration try { Schema::create( 'oauth_refresh_tokens', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->string('id', 100)->primary(); $table->string('access_token_id', 100)->index(); $table->boolean('revoked'); diff --git a/database/migrations/2018_01_01_000004_create_oauth_clients_table.php b/database/migrations/2018_01_01_000004_create_oauth_clients_table.php index ca86388fd2..7c75715cb0 100644 --- a/database/migrations/2018_01_01_000004_create_oauth_clients_table.php +++ b/database/migrations/2018_01_01_000004_create_oauth_clients_table.php @@ -53,7 +53,7 @@ class CreateOauthClientsTable extends Migration try { Schema::create( 'oauth_clients', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->increments('id'); $table->integer('user_id')->index()->nullable(); $table->string('name'); diff --git a/database/migrations/2018_01_01_000005_create_oauth_personal_access_clients_table.php b/database/migrations/2018_01_01_000005_create_oauth_personal_access_clients_table.php index 1c6c711835..2bdfa66075 100644 --- a/database/migrations/2018_01_01_000005_create_oauth_personal_access_clients_table.php +++ b/database/migrations/2018_01_01_000005_create_oauth_personal_access_clients_table.php @@ -53,7 +53,7 @@ class CreateOauthPersonalAccessClientsTable extends Migration try { Schema::create( 'oauth_personal_access_clients', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->increments('id'); $table->integer('client_id')->index(); $table->timestamps(); diff --git a/database/migrations/2018_03_19_141348_changes_for_v472.php b/database/migrations/2018_03_19_141348_changes_for_v472.php index e0fc018f3b..b6919563bc 100644 --- a/database/migrations/2018_03_19_141348_changes_for_v472.php +++ b/database/migrations/2018_03_19_141348_changes_for_v472.php @@ -43,7 +43,7 @@ class ChangesForV472 extends Migration try { Schema::table( 'attachments', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->text('notes')->nullable(); } ); @@ -57,7 +57,7 @@ class ChangesForV472 extends Migration try { Schema::table( 'budgets', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->dropColumn('order'); } ); @@ -79,7 +79,7 @@ class ChangesForV472 extends Migration try { Schema::table( 'attachments', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->dropColumn('notes'); } ); @@ -93,7 +93,7 @@ class ChangesForV472 extends Migration try { Schema::table( 'budgets', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->mediumInteger('order', false, true)->default(0); } ); diff --git a/database/migrations/2018_04_07_210913_changes_for_v473.php b/database/migrations/2018_04_07_210913_changes_for_v473.php index 80add3a888..b9ba91e78c 100644 --- a/database/migrations/2018_04_07_210913_changes_for_v473.php +++ b/database/migrations/2018_04_07_210913_changes_for_v473.php @@ -44,7 +44,7 @@ class ChangesForV473 extends Migration try { Schema::table( 'bills', - static function (Blueprint $table) { + static function (Blueprint $table): void { // cannot drop foreign keys in SQLite: if ('sqlite' !== config('database.default')) { $table->dropForeign('bills_transaction_currency_id_foreign'); @@ -62,7 +62,7 @@ class ChangesForV473 extends Migration try { Schema::table( 'rules', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->dropColumn('strict'); } ); @@ -84,7 +84,7 @@ class ChangesForV473 extends Migration try { Schema::table( 'bills', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->integer('transaction_currency_id', false, true)->nullable()->after('user_id'); $table->foreign('transaction_currency_id')->references('id')->on('transaction_currencies')->onDelete('set null'); } @@ -98,7 +98,7 @@ class ChangesForV473 extends Migration try { Schema::table( 'rules', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->boolean('strict')->default(true); } ); diff --git a/database/migrations/2018_06_08_200526_changes_for_v475.php b/database/migrations/2018_06_08_200526_changes_for_v475.php index 85384c9376..6575103788 100644 --- a/database/migrations/2018_06_08_200526_changes_for_v475.php +++ b/database/migrations/2018_06_08_200526_changes_for_v475.php @@ -56,7 +56,7 @@ class ChangesForV475 extends Migration try { Schema::create( 'recurrences', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->increments('id'); $table->timestamps(); $table->softDeletes(); @@ -87,7 +87,7 @@ class ChangesForV475 extends Migration try { Schema::create( 'recurrences_transactions', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->increments('id'); $table->timestamps(); $table->softDeletes(); @@ -118,7 +118,7 @@ class ChangesForV475 extends Migration try { Schema::create( 'recurrences_repetitions', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->increments('id'); $table->timestamps(); $table->softDeletes(); @@ -141,7 +141,7 @@ class ChangesForV475 extends Migration try { Schema::create( 'recurrences_meta', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->increments('id'); $table->timestamps(); $table->softDeletes(); @@ -163,7 +163,7 @@ class ChangesForV475 extends Migration try { Schema::create( 'rt_meta', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->increments('id'); $table->timestamps(); $table->softDeletes(); diff --git a/database/migrations/2018_09_05_195147_changes_for_v477.php b/database/migrations/2018_09_05_195147_changes_for_v477.php index 04da77b3da..b4c1e8c9fb 100644 --- a/database/migrations/2018_09_05_195147_changes_for_v477.php +++ b/database/migrations/2018_09_05_195147_changes_for_v477.php @@ -43,7 +43,7 @@ class ChangesForV477 extends Migration try { Schema::table( 'budget_limits', - static function (Blueprint $table) { + static function (Blueprint $table): void { // cannot drop foreign keys in SQLite: if ('sqlite' !== config('database.default')) { $table->dropForeign('budget_limits_transaction_currency_id_foreign'); @@ -70,7 +70,7 @@ class ChangesForV477 extends Migration try { Schema::table( 'budget_limits', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->integer('transaction_currency_id', false, true)->nullable()->after('budget_id'); $table->foreign('transaction_currency_id')->references('id')->on('transaction_currencies')->onDelete('set null'); } diff --git a/database/migrations/2018_11_06_172532_changes_for_v479.php b/database/migrations/2018_11_06_172532_changes_for_v479.php index 2e7c4bf568..f391bc5431 100644 --- a/database/migrations/2018_11_06_172532_changes_for_v479.php +++ b/database/migrations/2018_11_06_172532_changes_for_v479.php @@ -43,7 +43,7 @@ class ChangesForV479 extends Migration try { Schema::table( 'transaction_currencies', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->dropColumn(['enabled']); } ); @@ -65,7 +65,7 @@ class ChangesForV479 extends Migration try { Schema::table( 'transaction_currencies', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->boolean('enabled')->default(0)->after('deleted_at'); } ); diff --git a/database/migrations/2019_01_28_193833_changes_for_v4710.php b/database/migrations/2019_01_28_193833_changes_for_v4710.php index f9c8e46ae9..26d1423964 100644 --- a/database/migrations/2019_01_28_193833_changes_for_v4710.php +++ b/database/migrations/2019_01_28_193833_changes_for_v4710.php @@ -54,7 +54,7 @@ class ChangesForV4710 extends Migration try { Schema::create( 'transaction_groups', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->increments('id'); $table->timestamps(); $table->softDeletes(); @@ -74,7 +74,7 @@ class ChangesForV4710 extends Migration try { Schema::create( 'group_journals', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->increments('id'); $table->integer('transaction_group_id', false, true); $table->integer('transaction_journal_id', false, true); diff --git a/database/migrations/2019_02_05_055516_changes_for_v4711.php b/database/migrations/2019_02_05_055516_changes_for_v4711.php index d3329eb39b..b9b4d8a7de 100644 --- a/database/migrations/2019_02_05_055516_changes_for_v4711.php +++ b/database/migrations/2019_02_05_055516_changes_for_v4711.php @@ -57,7 +57,7 @@ class ChangesForV4711 extends Migration try { Schema::table( 'transaction_journals', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->dateTime('date')->change(); } ); @@ -69,7 +69,7 @@ class ChangesForV4711 extends Migration try { Schema::table( 'preferences', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->text('data')->nullable()->change(); } ); diff --git a/database/migrations/2019_02_11_170529_changes_for_v4712.php b/database/migrations/2019_02_11_170529_changes_for_v4712.php index 1a796723ef..118d6c63c7 100644 --- a/database/migrations/2019_02_11_170529_changes_for_v4712.php +++ b/database/migrations/2019_02_11_170529_changes_for_v4712.php @@ -56,7 +56,7 @@ class ChangesForV4712 extends Migration try { Schema::table( 'transaction_journals', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->dateTime('date')->change(); } ); diff --git a/database/migrations/2019_03_11_223700_fix_ldap_configuration.php b/database/migrations/2019_03_11_223700_fix_ldap_configuration.php index b51861d508..c31cb12910 100644 --- a/database/migrations/2019_03_11_223700_fix_ldap_configuration.php +++ b/database/migrations/2019_03_11_223700_fix_ldap_configuration.php @@ -43,7 +43,7 @@ class FixLdapConfiguration extends Migration try { Schema::table( 'users', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->dropColumn(['objectguid']); } ); @@ -69,7 +69,7 @@ class FixLdapConfiguration extends Migration try { Schema::table( 'users', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->uuid('objectguid')->nullable()->after('id'); } ); diff --git a/database/migrations/2019_03_22_183214_changes_for_v480.php b/database/migrations/2019_03_22_183214_changes_for_v480.php index 142e77cf94..bdd332e7d4 100644 --- a/database/migrations/2019_03_22_183214_changes_for_v480.php +++ b/database/migrations/2019_03_22_183214_changes_for_v480.php @@ -43,7 +43,7 @@ class ChangesForV480 extends Migration try { Schema::table( 'transaction_journals', - static function (Blueprint $table) { + static function (Blueprint $table): void { // drop transaction_group_id + foreign key. // cannot drop foreign keys in SQLite: if ('sqlite' !== config('database.default')) { @@ -74,7 +74,7 @@ class ChangesForV480 extends Migration try { Schema::table( 'rule_groups', - static function (Blueprint $table) { + static function (Blueprint $table): void { try { $table->dropColumn('stop_processing'); } catch (ColumnDoesNotExist|QueryException $e) { @@ -94,7 +94,7 @@ class ChangesForV480 extends Migration try { Schema::table( 'users', - static function (Blueprint $table) { + static function (Blueprint $table): void { try { $table->dropColumn('mfa_secret'); } catch (ColumnDoesNotExist|QueryException $e) { @@ -122,7 +122,7 @@ class ChangesForV480 extends Migration try { Schema::table( 'transaction_journals', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->integer('transaction_currency_id', false, true)->nullable()->change(); // add column "group_id" after "transaction_type_id" @@ -152,7 +152,7 @@ class ChangesForV480 extends Migration try { Schema::table( 'rule_groups', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->boolean('stop_processing')->default(false); } ); @@ -167,7 +167,7 @@ class ChangesForV480 extends Migration try { Schema::table( 'users', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->string('mfa_secret', 50)->nullable(); } ); diff --git a/database/migrations/2019_12_28_191351_make_locations_table.php b/database/migrations/2019_12_28_191351_make_locations_table.php index 72ea6cc018..b9125a2de6 100644 --- a/database/migrations/2019_12_28_191351_make_locations_table.php +++ b/database/migrations/2019_12_28_191351_make_locations_table.php @@ -53,7 +53,7 @@ class MakeLocationsTable extends Migration try { Schema::create( 'locations', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->bigIncrements('id'); $table->timestamps(); $table->softDeletes(); diff --git a/database/migrations/2020_03_13_201950_changes_for_v520.php b/database/migrations/2020_03_13_201950_changes_for_v520.php index 71ea92a1a0..7bbc6f1fba 100644 --- a/database/migrations/2020_03_13_201950_changes_for_v520.php +++ b/database/migrations/2020_03_13_201950_changes_for_v520.php @@ -52,7 +52,7 @@ class ChangesForV520 extends Migration try { Schema::create( 'auto_budgets', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->increments('id'); $table->timestamps(); $table->softDeletes(); diff --git a/database/migrations/2020_06_07_063612_changes_for_v530.php b/database/migrations/2020_06_07_063612_changes_for_v530.php index d07a4048bc..42c3d53903 100644 --- a/database/migrations/2020_06_07_063612_changes_for_v530.php +++ b/database/migrations/2020_06_07_063612_changes_for_v530.php @@ -53,7 +53,7 @@ class ChangesForV530 extends Migration try { Schema::create( 'object_groups', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->increments('id'); $table->integer('user_id', false, true); $table->timestamps(); @@ -73,7 +73,7 @@ class ChangesForV530 extends Migration try { Schema::create( 'object_groupables', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->integer('object_group_id'); $table->integer('object_groupable_id', false, true); $table->string('object_groupable_type', 255); diff --git a/database/migrations/2020_06_30_202620_changes_for_v530a.php b/database/migrations/2020_06_30_202620_changes_for_v530a.php index 545c82cc8c..b9b6f0925f 100644 --- a/database/migrations/2020_06_30_202620_changes_for_v530a.php +++ b/database/migrations/2020_06_30_202620_changes_for_v530a.php @@ -44,7 +44,7 @@ class ChangesForV530a extends Migration try { Schema::table( 'bills', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->dropColumn('order'); } ); @@ -66,7 +66,7 @@ class ChangesForV530a extends Migration try { Schema::table( 'bills', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->integer('order', false, true)->default(0); } ); diff --git a/database/migrations/2020_07_24_162820_changes_for_v540.php b/database/migrations/2020_07_24_162820_changes_for_v540.php index f71a4c8f7f..7fe7afd7a5 100644 --- a/database/migrations/2020_07_24_162820_changes_for_v540.php +++ b/database/migrations/2020_07_24_162820_changes_for_v540.php @@ -44,7 +44,7 @@ class ChangesForV540 extends Migration try { Schema::table( 'oauth_clients', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->dropColumn('provider'); } ); @@ -58,7 +58,7 @@ class ChangesForV540 extends Migration try { Schema::table( 'accounts', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->dropColumn('order'); } ); @@ -72,7 +72,7 @@ class ChangesForV540 extends Migration try { Schema::table( 'bills', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->dropColumn('end_date'); } ); @@ -85,7 +85,7 @@ class ChangesForV540 extends Migration try { Schema::table( 'bills', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->dropColumn('extension_date'); } ); @@ -107,7 +107,7 @@ class ChangesForV540 extends Migration try { Schema::table( 'accounts', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->integer('order', false, true)->default(0); } ); @@ -121,7 +121,7 @@ class ChangesForV540 extends Migration try { Schema::table( 'oauth_clients', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->string('provider')->nullable(); } ); @@ -135,7 +135,7 @@ class ChangesForV540 extends Migration try { Schema::table( 'bills', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->date('end_date')->nullable()->after('date'); $table->date('extension_date')->nullable()->after('end_date'); } @@ -150,7 +150,7 @@ class ChangesForV540 extends Migration try { Schema::table( 'oauth_clients', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->string('secret', 100)->nullable()->change(); } ); diff --git a/database/migrations/2020_11_12_070604_changes_for_v550.php b/database/migrations/2020_11_12_070604_changes_for_v550.php index 7440735421..8364c8224b 100644 --- a/database/migrations/2020_11_12_070604_changes_for_v550.php +++ b/database/migrations/2020_11_12_070604_changes_for_v550.php @@ -45,7 +45,7 @@ class ChangesForV550 extends Migration try { Schema::create( 'jobs', - static function (Blueprint $table) { + static function (Blueprint $table): void { // straight from Laravel (this is the OLD table) $table->bigIncrements('id'); $table->string('queue'); @@ -69,7 +69,7 @@ class ChangesForV550 extends Migration try { Schema::table( 'budget_transaction_journal', - static function (Blueprint $table) { + static function (Blueprint $table): void { if ('sqlite' !== config('database.default')) { $table->dropForeign('budget_id_foreign'); } @@ -91,7 +91,7 @@ class ChangesForV550 extends Migration try { Schema::table( 'budget_limits', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->dropColumn('period'); } ); @@ -104,7 +104,7 @@ class ChangesForV550 extends Migration try { Schema::table( 'budget_limits', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->dropColumn('generated'); } ); @@ -134,7 +134,7 @@ class ChangesForV550 extends Migration try { Schema::create( 'jobs', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->bigIncrements('id'); $table->string('queue')->index(); $table->longText('payload'); @@ -157,7 +157,7 @@ class ChangesForV550 extends Migration try { Schema::create( 'failed_jobs', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->bigIncrements('id'); $table->string('uuid')->unique(); $table->text('connection'); @@ -178,7 +178,7 @@ class ChangesForV550 extends Migration try { Schema::table( 'budget_transaction_journal', - static function (Blueprint $table) { + static function (Blueprint $table): void { if (!Schema::hasColumn('budget_transaction_journal', 'budget_limit_id')) { $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'); @@ -197,7 +197,7 @@ class ChangesForV550 extends Migration try { Schema::table( 'budget_limits', - static function (Blueprint $table) { + static function (Blueprint $table): void { if (!Schema::hasColumn('budget_limits', 'period')) { $table->string('period', 12)->nullable(); } @@ -216,7 +216,7 @@ class ChangesForV550 extends Migration try { Schema::create( 'webhooks', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->increments('id'); $table->timestamps(); $table->softDeletes(); @@ -243,7 +243,7 @@ class ChangesForV550 extends Migration try { Schema::create( 'webhook_messages', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->increments('id'); $table->timestamps(); $table->softDeletes(); @@ -267,7 +267,7 @@ class ChangesForV550 extends Migration try { Schema::create( 'webhook_attempts', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->increments('id'); $table->timestamps(); $table->softDeletes(); diff --git a/database/migrations/2021_03_12_061213_changes_for_v550b2.php b/database/migrations/2021_03_12_061213_changes_for_v550b2.php index a9736d6613..5e1f30eda4 100644 --- a/database/migrations/2021_03_12_061213_changes_for_v550b2.php +++ b/database/migrations/2021_03_12_061213_changes_for_v550b2.php @@ -42,7 +42,7 @@ class ChangesForV550b2 extends Migration try { Schema::table( 'recurrences_transactions', - static function (Blueprint $table) { + static function (Blueprint $table): void { if ('sqlite' !== config('database.default')) { $table->dropForeign('type_foreign'); } @@ -70,7 +70,7 @@ class ChangesForV550b2 extends Migration try { Schema::table( 'recurrences_transactions', - static function (Blueprint $table) { + static function (Blueprint $table): void { if (!Schema::hasColumn('recurrences_transactions', 'transaction_type_id')) { $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'); diff --git a/database/migrations/2021_05_09_064644_add_ldap_columns_to_users_table.php b/database/migrations/2021_05_09_064644_add_ldap_columns_to_users_table.php index 624b332e77..8d96ab23f4 100644 --- a/database/migrations/2021_05_09_064644_add_ldap_columns_to_users_table.php +++ b/database/migrations/2021_05_09_064644_add_ldap_columns_to_users_table.php @@ -39,7 +39,7 @@ class AddLdapColumnsToUsersTable extends Migration try { Schema::table( 'users', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->dropColumn(['domain']); } ); @@ -61,7 +61,7 @@ class AddLdapColumnsToUsersTable extends Migration try { Schema::table( 'users', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->string('domain')->nullable(); } ); diff --git a/database/migrations/2021_05_13_053836_extend_currency_info.php b/database/migrations/2021_05_13_053836_extend_currency_info.php index 081e931c5a..888bbab2fa 100644 --- a/database/migrations/2021_05_13_053836_extend_currency_info.php +++ b/database/migrations/2021_05_13_053836_extend_currency_info.php @@ -47,7 +47,7 @@ class ExtendCurrencyInfo extends Migration try { Schema::table( 'transaction_currencies', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->string('code', 51)->change(); $table->string('symbol', 51)->change(); } diff --git a/database/migrations/2021_08_28_073733_user_groups.php b/database/migrations/2021_08_28_073733_user_groups.php index 0e48c875fc..9a9a37e249 100644 --- a/database/migrations/2021_08_28_073733_user_groups.php +++ b/database/migrations/2021_08_28_073733_user_groups.php @@ -63,7 +63,7 @@ class UserGroups extends Migration try { Schema::table( $tableName, - static function (Blueprint $table) use ($tableName) { + static function (Blueprint $table) use ($tableName): void { if ('sqlite' !== config('database.default')) { $table->dropForeign(sprintf('%s_to_ugi', $tableName)); } @@ -83,7 +83,7 @@ class UserGroups extends Migration try { Schema::table( 'users', - static function (Blueprint $table) { + static function (Blueprint $table): void { if ('sqlite' !== config('database.default')) { $table->dropForeign('type_user_group_id'); } @@ -118,7 +118,7 @@ class UserGroups extends Migration try { Schema::create( 'user_groups', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->bigIncrements('id'); $table->timestamps(); $table->softDeletes(); @@ -136,7 +136,7 @@ class UserGroups extends Migration try { Schema::create( 'user_roles', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->bigIncrements('id'); $table->timestamps(); $table->softDeletes(); @@ -154,7 +154,7 @@ class UserGroups extends Migration try { Schema::create( 'group_memberships', - static function (Blueprint $table) { + static function (Blueprint $table): void { $table->bigIncrements('id'); $table->timestamps(); $table->softDeletes(); @@ -177,7 +177,7 @@ class UserGroups extends Migration try { Schema::table( 'users', - static function (Blueprint $table) { + static function (Blueprint $table): void { if (!Schema::hasColumn('users', 'user_group_id')) { $table->bigInteger('user_group_id', false, true)->nullable(); $table->foreign('user_group_id', 'type_user_group_id')->references('id')->on('user_groups')->onDelete('set null')->onUpdate( @@ -197,7 +197,7 @@ class UserGroups extends Migration try { Schema::table( $tableName, - static function (Blueprint $table) use ($tableName) { + static function (Blueprint $table) use ($tableName): void { if (!Schema::hasColumn($tableName, 'user_group_id')) { $table->bigInteger('user_group_id', false, true)->nullable()->after('user_id'); $table->foreign('user_group_id', sprintf('%s_to_ugi', $tableName))->references('id')->on('user_groups')->onDelete( diff --git a/database/migrations/2021_12_27_000001_create_local_personal_access_tokens_table.php b/database/migrations/2021_12_27_000001_create_local_personal_access_tokens_table.php index b341a65e5c..c551a9fe43 100644 --- a/database/migrations/2021_12_27_000001_create_local_personal_access_tokens_table.php +++ b/database/migrations/2021_12_27_000001_create_local_personal_access_tokens_table.php @@ -49,7 +49,7 @@ class CreateLocalPersonalAccessTokensTable extends Migration { if (!Schema::hasTable('personal_access_tokens')) { try { - Schema::create('personal_access_tokens', static function (Blueprint $table) { + Schema::create('personal_access_tokens', static function (Blueprint $table): void { $table->bigIncrements('id'); $table->morphs('tokenable'); $table->string('name'); diff --git a/database/migrations/2022_08_21_104626_add_user_groups.php b/database/migrations/2022_08_21_104626_add_user_groups.php index 3b348e0fd7..552c66bcc1 100644 --- a/database/migrations/2022_08_21_104626_add_user_groups.php +++ b/database/migrations/2022_08_21_104626_add_user_groups.php @@ -39,7 +39,7 @@ return new class() extends Migration { try { Schema::table( 'currency_exchange_rates', - static function (Blueprint $table) { + static function (Blueprint $table): void { if (!Schema::hasColumn('currency_exchange_rates', 'user_group_id')) { $table->bigInteger('user_group_id', false, true)->nullable()->after('user_id'); $table->foreign('user_group_id', 'cer_to_ugi')->references('id')->on('user_groups')->onDelete('set null')->onUpdate('cascade'); @@ -60,7 +60,7 @@ return new class() extends Migration { try { Schema::table( 'currency_exchange_rates', - static function (Blueprint $table) { + static function (Blueprint $table): void { if ('sqlite' !== config('database.default')) { $table->dropForeign('cer_to_ugi'); } diff --git a/database/migrations/2022_09_18_123911_create_notifications_table.php b/database/migrations/2022_09_18_123911_create_notifications_table.php index 862bbb94fd..034f4625fe 100644 --- a/database/migrations/2022_09_18_123911_create_notifications_table.php +++ b/database/migrations/2022_09_18_123911_create_notifications_table.php @@ -37,7 +37,7 @@ return new class() extends Migration { { if (!Schema::hasTable('notifications')) { try { - Schema::create('notifications', static function (Blueprint $table) { + Schema::create('notifications', static function (Blueprint $table): void { $table->uuid('id')->primary(); $table->string('type'); $table->morphs('notifiable'); diff --git a/database/migrations/2022_10_01_074908_invited_users.php b/database/migrations/2022_10_01_074908_invited_users.php index 84c40cddca..f802f72179 100644 --- a/database/migrations/2022_10_01_074908_invited_users.php +++ b/database/migrations/2022_10_01_074908_invited_users.php @@ -37,7 +37,7 @@ return new class() extends Migration { { if (!Schema::hasTable('invited_users')) { try { - Schema::create('invited_users', static function (Blueprint $table) { + Schema::create('invited_users', static function (Blueprint $table): void { $table->id(); $table->timestamps(); $table->integer('user_id', false, true); diff --git a/database/migrations/2022_10_01_210238_audit_log_entries.php b/database/migrations/2022_10_01_210238_audit_log_entries.php index 57ff51c5a5..bf6cf14f60 100644 --- a/database/migrations/2022_10_01_210238_audit_log_entries.php +++ b/database/migrations/2022_10_01_210238_audit_log_entries.php @@ -37,7 +37,7 @@ return new class() extends Migration { { if (!Schema::hasTable('audit_log_entries')) { try { - Schema::create('audit_log_entries', static function (Blueprint $table) { + Schema::create('audit_log_entries', static function (Blueprint $table): void { $table->id(); $table->timestamps(); $table->softDeletes(); diff --git a/database/migrations/2023_08_11_192521_upgrade_og_table.php b/database/migrations/2023_08_11_192521_upgrade_og_table.php index f5e37d2a6a..aea11c646f 100644 --- a/database/migrations/2023_08_11_192521_upgrade_og_table.php +++ b/database/migrations/2023_08_11_192521_upgrade_og_table.php @@ -39,7 +39,7 @@ return new class() extends Migration { try { Schema::table( 'object_groups', - static function (Blueprint $table) { + static function (Blueprint $table): void { if (!Schema::hasColumn('object_groups', 'user_group_id')) { $table->bigInteger('user_group_id', false, true)->nullable()->after('user_id'); $table->foreign('user_group_id', sprintf('%s_to_ugi', 'object_groups'))->references('id')->on('user_groups')->onDelete( @@ -62,7 +62,7 @@ return new class() extends Migration { try { Schema::table( 'object_groups', - static function (Blueprint $table) { + static function (Blueprint $table): void { if ('sqlite' !== config('database.default')) { $table->dropForeign(sprintf('%s_to_ugi', 'object_groups')); } diff --git a/database/migrations/2023_10_21_113213_add_currency_pivot_tables.php b/database/migrations/2023_10_21_113213_add_currency_pivot_tables.php index d32ece5411..ee53d2c3e5 100644 --- a/database/migrations/2023_10_21_113213_add_currency_pivot_tables.php +++ b/database/migrations/2023_10_21_113213_add_currency_pivot_tables.php @@ -37,7 +37,7 @@ return new class() extends Migration { // transaction_currency_user if (!Schema::hasTable('transaction_currency_user')) { try { - Schema::create('transaction_currency_user', static function (Blueprint $table) { + Schema::create('transaction_currency_user', static function (Blueprint $table): void { $table->id(); $table->timestamps(); $table->integer('user_id', false, true); @@ -56,7 +56,7 @@ return new class() extends Migration { // transaction_currency_user_group if (!Schema::hasTable('transaction_currency_user_group')) { try { - Schema::create('transaction_currency_user_group', static function (Blueprint $table) { + Schema::create('transaction_currency_user_group', static function (Blueprint $table): void { $table->id(); $table->timestamps(); $table->bigInteger('user_group_id', false, true); diff --git a/database/seeders/UserRoleSeeder.php b/database/seeders/UserRoleSeeder.php index ee9a680bdb..76d1c2084f 100644 --- a/database/seeders/UserRoleSeeder.php +++ b/database/seeders/UserRoleSeeder.php @@ -36,7 +36,7 @@ class UserRoleSeeder extends Seeder /** * Run the database seeds. */ - public function run() + public function run(): void { $roles = []; foreach (UserRoleEnum::cases() as $role) { diff --git a/routes/api-noauth.php b/routes/api-noauth.php index cd7f16f571..b062cf43cc 100644 --- a/routes/api-noauth.php +++ b/routes/api-noauth.php @@ -28,7 +28,7 @@ Route::group( 'prefix' => '', 'as' => 'api.v1.cron.', ], - static function () { + static function (): void { Route::get('{cliToken}', ['uses' => 'CronController@cron', 'as' => 'index']); } ); diff --git a/routes/api.php b/routes/api.php index 213c90d828..32ffe97a1f 100644 --- a/routes/api.php +++ b/routes/api.php @@ -30,7 +30,7 @@ Route::group( 'prefix' => 'v2/summary', 'as' => 'api.v2.summary.', ], - static function () { + static function (): void { Route::get('basic', ['uses' => 'BasicController@basic', 'as' => 'basic']); } ); @@ -41,7 +41,7 @@ Route::group( 'prefix' => 'v2', 'as' => 'api.v2.', ], - static function () { + static function (): void { Route::get('transactions', ['uses' => 'TransactionController@list', 'as' => 'transactions.list']); Route::get('accounts/{userGroupAccount}/transactions', ['uses' => 'AccountController@list', 'as' => 'accounts.transactions']); } @@ -54,7 +54,7 @@ Route::group( 'prefix' => 'v2/autocomplete', 'as' => 'api.v2.autocomplete.', ], - static function () { + static function (): void { // Auto complete routes Route::get('accounts', ['uses' => 'AccountController@accounts', 'as' => 'accounts']); Route::get('transaction-descriptions', ['uses' => 'TransactionController@transactionDescriptions', 'as' => 'transaction-descriptions']); @@ -68,7 +68,7 @@ Route::group( 'prefix' => 'v2/net-worth', 'as' => 'api.v2.net-worth.', ], - static function () { + static function (): void { Route::get('', ['uses' => 'NetWorthController@get', 'as' => 'index']); } ); @@ -80,7 +80,7 @@ Route::group( 'prefix' => 'v2/chart', 'as' => 'api.v1.chart.', ], - static function () { + static function (): void { Route::get('account/dashboard', ['uses' => 'AccountController@dashboard', 'as' => 'account.dashboard']); Route::get('budget/dashboard', ['uses' => 'BudgetController@dashboard', 'as' => 'budget.dashboard']); Route::get('category/dashboard', ['uses' => 'CategoryController@dashboard', 'as' => 'category.dashboard']); @@ -95,7 +95,7 @@ Route::group( 'prefix' => 'v2/accounts', 'as' => 'api.v2.accounts.', ], - static function () { + static function (): void { Route::get('{account}', ['uses' => 'ShowController@show', 'as' => 'show']); } ); @@ -107,7 +107,7 @@ Route::group( 'prefix' => 'v2/subscriptions', 'as' => 'api.v2.subscriptions.', ], - static function () { + static function (): void { Route::get('', ['uses' => 'IndexController@index', 'as' => 'index']); Route::get('{userGroupBill}', ['uses' => 'ShowController@show', 'as' => 'show']); Route::get('sum/paid', ['uses' => 'SumController@paid', 'as' => 'sum.paid']); @@ -122,7 +122,7 @@ Route::group( 'prefix' => 'v2/piggy-banks', 'as' => 'api.v2.piggy-banks.', ], - static function () { + static function (): void { Route::get('', ['uses' => 'IndexController@index', 'as' => 'index']); } ); @@ -134,7 +134,7 @@ Route::group( 'prefix' => 'v2/currencies', 'as' => 'api.v2.currencies.', ], - static function () { + static function (): void { Route::get('', ['uses' => 'IndexController@index', 'as' => 'index']); } ); @@ -146,7 +146,7 @@ Route::group( 'prefix' => 'v2/transactions', 'as' => 'api.v2.transactions.', ], - static function () { + static function (): void { Route::post('', ['uses' => 'StoreController@post', 'as' => 'store']); } ); @@ -158,7 +158,7 @@ Route::group( 'prefix' => 'v2/budgets', 'as' => 'api.v2.budgets', ], - static function () { + static function (): void { Route::get('', ['uses' => 'Budget\IndexController@index', 'as' => 'index']); Route::get('{budget}', ['uses' => 'Budget\ShowController@show', 'as' => 'show']); Route::get('{budget}/limits', ['uses' => 'BudgetLimit\IndexController@index', 'as' => 'budget-limits.index']); @@ -176,7 +176,7 @@ Route::group( 'prefix' => 'v2', 'as' => 'api.v2.system.', ], - static function () { + static function (): void { Route::get('preferences/{preference}', ['uses' => 'PreferencesController@get', 'as' => 'preferences.get']); } ); @@ -188,7 +188,7 @@ Route::group( 'prefix' => 'v2/user-groups', 'as' => 'api.v2.user-groups.', ], - static function () { + static function (): void { Route::get('', ['uses' => 'IndexController@index', 'as' => 'index']); Route::post('', ['uses' => 'StoreController@store', 'as' => 'store']); Route::get('{userGroup}', ['uses' => 'ShowController@show', 'as' => 'show']); @@ -207,7 +207,7 @@ Route::group( 'prefix' => 'v1/autocomplete', 'as' => 'api.v1.autocomplete.', ], - static function () { + static function (): void { // Auto complete routes Route::get('accounts', ['uses' => 'AccountController@accounts', 'as' => 'accounts']); Route::get('bills', ['uses' => 'BillController@bills', 'as' => 'bills']); @@ -236,7 +236,7 @@ Route::group( 'prefix' => 'v1/chart/account', 'as' => 'api.v1.chart.account.', ], - static function () { + static function (): void { Route::get('overview', ['uses' => 'AccountController@overview', 'as' => 'overview']); } ); @@ -249,7 +249,7 @@ Route::group( 'prefix' => 'v1/data/export', 'as' => 'api.v1.data.export.', ], - static function () { + static function (): void { Route::get('accounts', ['uses' => 'ExportController@accounts', 'as' => 'accounts']); Route::get('bills', ['uses' => 'ExportController@bills', 'as' => 'bills']); Route::get('budgets', ['uses' => 'ExportController@budgets', 'as' => 'budgets']); @@ -268,7 +268,7 @@ Route::group( 'prefix' => 'v1/data/destroy', 'as' => 'api.v1.data.', ], - static function () { + static function (): void { Route::delete('', ['uses' => 'DestroyController@destroy', 'as' => 'destroy']); } ); @@ -278,7 +278,7 @@ Route::group( 'prefix' => 'v1/data/purge', 'as' => 'api.v1.data.', ], - static function () { + static function (): void { Route::delete('', ['uses' => 'PurgeController@purge', 'as' => 'purge']); } ); @@ -290,7 +290,7 @@ Route::group( 'prefix' => 'v1/data/bulk', 'as' => 'api.v1.data.bulk.', ], - static function () { + static function (): void { Route::post('transactions', ['uses' => 'TransactionController@update', 'as' => 'transactions']); } ); @@ -304,7 +304,7 @@ Route::group( 'prefix' => 'v1/insight/expense', 'as' => 'api.v1.insight.expense.', ], - static function () { + static function (): void { // Insight in expenses per account: Route::get('expense', ['uses' => 'AccountController@expense', 'as' => 'expense']); Route::get('asset', ['uses' => 'AccountController@asset', 'as' => 'asset']); @@ -331,7 +331,7 @@ Route::group( 'prefix' => 'v1/insight/income', 'as' => 'api.v1.insight.income.', ], - static function () { + static function (): void { // Insight in expenses per account: Route::get('revenue', ['uses' => 'AccountController@revenue', 'as' => 'revenue']); Route::get('asset', ['uses' => 'AccountController@asset', 'as' => 'asset']); @@ -356,7 +356,7 @@ Route::group( 'prefix' => 'v1/insight/transfer', 'as' => 'api.v1.insight.transfer.', ], - static function () { + static function (): void { // Insight in expenses per account: Route::get('asset', ['uses' => 'AccountController@asset', 'as' => 'asset']); Route::get('category', ['uses' => 'CategoryController@category', 'as' => 'category']); @@ -375,7 +375,7 @@ Route::group( 'prefix' => 'v1/summary', 'as' => 'api.v1.summary.', ], - static function () { + static function (): void { Route::get('basic', ['uses' => 'BasicController@basic', 'as' => 'basic']); } ); @@ -388,7 +388,7 @@ Route::group( 'prefix' => 'v1/accounts', 'as' => 'api.v1.accounts.', ], - static function () { + static function (): void { Route::get('', ['uses' => 'ShowController@index', 'as' => 'index']); Route::post('', ['uses' => 'StoreController@store', 'as' => 'store']); Route::get('{account}', ['uses' => 'ShowController@show', 'as' => 'show']); @@ -408,7 +408,7 @@ Route::group( 'prefix' => 'v1/attachments', 'as' => 'api.v1.attachments.', ], - static function () { + static function (): void { Route::get('', ['uses' => 'ShowController@index', 'as' => 'index']); Route::post('', ['uses' => 'StoreController@store', 'as' => 'store']); Route::get('{attachment}', ['uses' => 'ShowController@show', 'as' => 'show']); @@ -426,7 +426,7 @@ Route::group( 'prefix' => 'v1/bills', 'as' => 'api.v1.bills.', ], - static function () { + static function (): void { Route::get('', ['uses' => 'ShowController@index', 'as' => 'index']); Route::post('', ['uses' => 'StoreController@store', 'as' => 'store']); Route::get('{bill}', ['uses' => 'ShowController@show', 'as' => 'show']); @@ -446,7 +446,7 @@ Route::group( 'prefix' => 'v1/available-budgets', 'as' => 'api.v1.available-budgets.', ], - static function () { + static function (): void { Route::get('', ['uses' => 'ShowController@index', 'as' => 'index']); // Route::post('', ['uses' => 'StoreController@store', 'as' => 'store']); Route::get('{availableBudget}', ['uses' => 'ShowController@show', 'as' => 'show']); @@ -462,7 +462,7 @@ Route::group( 'prefix' => 'v1/budgets', 'as' => 'api.v1.budgets.', ], - static function () { + static function (): void { Route::get('', ['uses' => 'Budget\ShowController@index', 'as' => 'index']); Route::post('', ['uses' => 'Budget\StoreController@store', 'as' => 'store']); Route::get('transactions-without-budget', ['uses' => 'Budget\ListController@withoutBudget', 'as' => 'without-budget']); @@ -490,7 +490,7 @@ Route::group( 'prefix' => 'v1/budget-limits', 'as' => 'api.v1.budget-limits.', ], - static function () { + static function (): void { Route::get('', ['uses' => 'ShowController@indexAll', 'as' => 'index']); } ); @@ -502,7 +502,7 @@ Route::group( 'prefix' => 'v1/categories', 'as' => 'api.v1.categories.', ], - static function () { + static function (): void { Route::get('', ['uses' => 'ShowController@index', 'as' => 'index']); Route::post('', ['uses' => 'StoreController@store', 'as' => 'store']); Route::get('{category}', ['uses' => 'ShowController@show', 'as' => 'show']); @@ -521,7 +521,7 @@ Route::group( 'prefix' => 'v1/object-groups', 'as' => 'api.v1.object-groups.', ], - static function () { + static function (): void { Route::get('', ['uses' => 'ShowController@index', 'as' => 'index']); Route::get('{objectGroup}', ['uses' => 'ShowController@show', 'as' => 'show']); Route::put('{objectGroup}', ['uses' => 'UpdateController@update', 'as' => 'update']); @@ -539,7 +539,7 @@ Route::group( 'prefix' => 'v1/piggy-banks', 'as' => 'api.v1.piggy-banks.', ], - static function () { + static function (): void { Route::get('', ['uses' => 'ShowController@index', 'as' => 'index']); Route::post('', ['uses' => 'StoreController@store', 'as' => 'store']); Route::get('{piggyBank}', ['uses' => 'ShowController@show', 'as' => 'show']); @@ -558,7 +558,7 @@ Route::group( 'prefix' => 'v1/recurrences', 'as' => 'api.v1.recurrences.', ], - static function () { + static function (): void { Route::get('', ['uses' => 'ShowController@index', 'as' => 'index']); Route::post('', ['uses' => 'StoreController@store', 'as' => 'store']); Route::get('{recurrence}', ['uses' => 'ShowController@show', 'as' => 'show']); @@ -577,7 +577,7 @@ Route::group( 'prefix' => 'v1/rules', 'as' => 'api.v1.rules.', ], - static function () { + static function (): void { Route::get('', ['uses' => 'ShowController@index', 'as' => 'index']); Route::post('', ['uses' => 'StoreController@store', 'as' => 'store']); Route::get('{rule}', ['uses' => 'ShowController@show', 'as' => 'show']); @@ -598,7 +598,7 @@ Route::group( 'prefix' => 'v1/rule-groups', 'as' => 'api.v1.rule-groups.', ], - static function () { + static function (): void { Route::get('', ['uses' => 'ShowController@index', 'as' => 'index']); Route::post('', ['uses' => 'StoreController@store', 'as' => 'store']); Route::get('{ruleGroup}', ['uses' => 'ShowController@show', 'as' => 'show']); @@ -618,7 +618,7 @@ Route::group( 'prefix' => 'v1/tags', 'as' => 'api.v1.tags.', ], - static function () { + static function (): void { Route::get('', ['uses' => 'ShowController@index', 'as' => 'index']); Route::post('', ['uses' => 'StoreController@store', 'as' => 'store']); Route::get('{tagOrId}', ['uses' => 'ShowController@show', 'as' => 'show']); @@ -636,7 +636,7 @@ Route::group( 'prefix' => 'v1/transactions', 'as' => 'api.v1.transactions.', ], - static function () { + static function (): void { Route::get('', ['uses' => 'ShowController@index', 'as' => 'index']); Route::post('', ['uses' => 'StoreController@store', 'as' => 'store']); Route::get('{transactionGroup}', ['uses' => 'ShowController@show', 'as' => 'show']); @@ -654,7 +654,7 @@ Route::group( 'prefix' => 'v1/transaction-journals', 'as' => 'api.v1.transaction-journals.', ], - static function () { + static function (): void { Route::get('{tj}', ['uses' => 'ShowController@showJournal', 'as' => 'show']); Route::delete('{tj}', ['uses' => 'DestroyController@destroyJournal', 'as' => 'delete']); @@ -669,7 +669,7 @@ Route::group( 'prefix' => 'v1/currencies', 'as' => 'api.v1.currencies.', ], - static function () { + static function (): void { Route::get('', ['uses' => 'ShowController@index', 'as' => 'index']); Route::post('', ['uses' => 'StoreController@store', 'as' => 'store']); Route::get('default', ['uses' => 'ShowController@showDefault', 'as' => 'show.default']); @@ -699,7 +699,7 @@ Route::group( 'prefix' => 'v1/transaction-links', 'as' => 'api.v1.transaction-links.', ], - static function () { + static function (): void { Route::get('', ['uses' => 'ShowController@index', 'as' => 'index']); Route::post('', ['uses' => 'StoreController@store', 'as' => 'store']); Route::get('{journalLink}', ['uses' => 'ShowController@show', 'as' => 'show']); @@ -715,7 +715,7 @@ Route::group( 'prefix' => 'v1/link-types', 'as' => 'api.v1.link-types.', ], - static function () { + static function (): void { Route::get('', ['uses' => 'ShowController@index', 'as' => 'index']); Route::post('', ['uses' => 'StoreController@store', 'as' => 'store']); Route::get('{linkType}', ['uses' => 'ShowController@show', 'as' => 'show']); @@ -732,7 +732,7 @@ Route::group( 'prefix' => 'v1/search', 'as' => 'api.v1.search.', ], - static function () { + static function (): void { Route::get('transactions', ['uses' => 'TransactionController@search', 'as' => 'transactions']); Route::get('accounts', ['uses' => 'AccountController@search', 'as' => 'accounts']); } @@ -746,7 +746,7 @@ Route::group( 'prefix' => 'v1/about', 'as' => 'api.v1.about.', ], - static function () { + static function (): void { Route::get('', ['uses' => 'AboutController@about', 'as' => 'index']); Route::get('user', ['uses' => 'AboutController@user', 'as' => 'user']); } @@ -758,7 +758,7 @@ Route::group( 'prefix' => 'v1/configuration', 'as' => 'api.v1.configuration.', ], - static function () { + static function (): void { Route::get('', ['uses' => 'ConfigurationController@index', 'as' => 'index']); Route::get('{eitherConfigKey}', ['uses' => 'ConfigurationController@show', 'as' => 'show']); Route::put('{dynamicConfigKey}', ['uses' => 'ConfigurationController@update', 'as' => 'update']); @@ -772,7 +772,7 @@ Route::group( 'prefix' => 'v1/users', 'as' => 'api.v1.users.', ], - static function () { + static function (): void { Route::get('', ['uses' => 'UserController@index', 'as' => 'index']); Route::post('', ['uses' => 'UserController@store', 'as' => 'store']); Route::get('{user}', ['uses' => 'UserController@show', 'as' => 'show']); @@ -790,7 +790,7 @@ Route::group( 'prefix' => 'v1/preferences', 'as' => 'api.v1.preferences.', ], - static function () { + static function (): void { Route::get('', ['uses' => 'PreferencesController@index', 'as' => 'index']); Route::post('', ['uses' => 'PreferencesController@store', 'as' => 'store']); Route::get('{preference}', ['uses' => 'PreferencesController@show', 'as' => 'show']); @@ -805,7 +805,7 @@ Route::group( 'prefix' => 'v1/webhooks', 'as' => 'api.v1.webhooks.', ], - static function () { + static function (): void { Route::get('', ['uses' => 'ShowController@index', 'as' => 'index']); Route::post('', ['uses' => 'StoreController@store', 'as' => 'store']); Route::get('{webhook}', ['uses' => 'ShowController@show', 'as' => 'show']); diff --git a/routes/breadcrumbs.php b/routes/breadcrumbs.php index f6be8c699a..2b17956637 100644 --- a/routes/breadcrumbs.php +++ b/routes/breadcrumbs.php @@ -65,14 +65,14 @@ if (!function_exists('limitStringLength')) { // HOME Breadcrumbs::for( 'home', - static function (Generator $breadcrumbs) { + static function (Generator $breadcrumbs): void { $breadcrumbs->push(trans('breadcrumbs.home'), route('index')); } ); Breadcrumbs::for( 'index', - static function (Generator $breadcrumbs) { + static function (Generator $breadcrumbs): void { $breadcrumbs->push(trans('breadcrumbs.home'), route('index')); } ); @@ -80,14 +80,14 @@ Breadcrumbs::for( // ACCOUNTS Breadcrumbs::for( 'accounts.index', - static function (Generator $breadcrumbs, string $what) { + static function (Generator $breadcrumbs, string $what): void { $breadcrumbs->parent('home'); $breadcrumbs->push(trans('firefly.'.strtolower(e($what)).'_accounts'), route('accounts.index', [$what])); } ); Breadcrumbs::for( // inactive 'accounts.inactive.index', - static function (Generator $breadcrumbs, string $what) { + static function (Generator $breadcrumbs, string $what): void { $breadcrumbs->parent('home'); $breadcrumbs->push(trans('firefly.'.strtolower(e($what)).'_accounts_inactive'), route('accounts.inactive.index', [$what])); } @@ -95,7 +95,7 @@ Breadcrumbs::for( // inactive Breadcrumbs::for( 'accounts.create', - static function (Generator $breadcrumbs, string $what) { + static function (Generator $breadcrumbs, string $what): void { $breadcrumbs->parent('accounts.index', $what); $breadcrumbs->push(trans('firefly.new_'.strtolower(e($what)).'_account'), route('accounts.create', [$what])); } @@ -103,7 +103,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'accounts.show', - static function (Generator $breadcrumbs, Account $account, Carbon $start = null, Carbon $end = null) { + static function (Generator $breadcrumbs, Account $account, Carbon $start = null, Carbon $end = null): void { $what = config('firefly.shortNamesByFullName.'.$account->accountType->type); $breadcrumbs->parent('accounts.index', $what); @@ -123,7 +123,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'accounts.show.all', - static function (Generator $breadcrumbs, Account $account) { + static function (Generator $breadcrumbs, Account $account): void { $what = config('firefly.shortNamesByFullName.'.$account->accountType->type); $breadcrumbs->parent('accounts.index', $what); @@ -133,7 +133,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'accounts.reconcile', - static function (Generator $breadcrumbs, Account $account) { + static function (Generator $breadcrumbs, Account $account): void { $breadcrumbs->parent('accounts.show', $account); $breadcrumbs->push(trans('firefly.reconcile_account', ['account' => $account->name]), route('accounts.reconcile', [$account->id])); } @@ -141,7 +141,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'accounts.reconcile.show', - static function (Generator $breadcrumbs, Account $account, TransactionJournal $journal) { + static function (Generator $breadcrumbs, Account $account, TransactionJournal $journal): void { $breadcrumbs->parent('accounts.show', $account); $title = trans('firefly.reconciliation').' "'.$journal->description.'"'; $breadcrumbs->push($title, route('accounts.reconcile.show', [$journal->id])); @@ -150,7 +150,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'accounts.delete', - static function (Generator $breadcrumbs, Account $account) { + static function (Generator $breadcrumbs, Account $account): void { $breadcrumbs->parent('accounts.show', $account); $breadcrumbs->push(trans('firefly.delete_account', ['name' => limitStringLength($account->name)]), route('accounts.delete', [$account->id])); } @@ -158,7 +158,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'accounts.edit', - static function (Generator $breadcrumbs, Account $account) { + static function (Generator $breadcrumbs, Account $account): void { $breadcrumbs->parent('accounts.show', $account); $what = config('firefly.shortNamesByFullName.'.$account->accountType->type); @@ -172,7 +172,7 @@ Breadcrumbs::for( // ADMIN Breadcrumbs::for( 'admin.index', - static function (Generator $breadcrumbs) { + static function (Generator $breadcrumbs): void { $breadcrumbs->parent('home'); $breadcrumbs->push(trans('firefly.administration'), route('admin.index')); } @@ -180,7 +180,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'admin.users', - static function (Generator $breadcrumbs) { + static function (Generator $breadcrumbs): void { $breadcrumbs->parent('admin.index'); $breadcrumbs->push(trans('firefly.list_all_users'), route('admin.users')); } @@ -188,21 +188,21 @@ Breadcrumbs::for( Breadcrumbs::for( 'admin.users.show', - static function (Generator $breadcrumbs, User $user) { + static function (Generator $breadcrumbs, User $user): void { $breadcrumbs->parent('admin.users'); $breadcrumbs->push(trans('firefly.single_user_administration', ['email' => $user->email]), route('admin.users.show', [$user->id])); } ); Breadcrumbs::for( 'admin.users.edit', - static function (Generator $breadcrumbs, User $user) { + static function (Generator $breadcrumbs, User $user): void { $breadcrumbs->parent('admin.users'); $breadcrumbs->push(trans('firefly.edit_user', ['email' => $user->email]), route('admin.users.edit', [$user->id])); } ); Breadcrumbs::for( 'admin.users.delete', - static function (Generator $breadcrumbs, User $user) { + static function (Generator $breadcrumbs, User $user): void { $breadcrumbs->parent('admin.users'); $breadcrumbs->push(trans('firefly.delete_user', ['email' => $user->email]), route('admin.users.delete', [$user->id])); } @@ -210,7 +210,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'admin.users.domains', - static function (Generator $breadcrumbs) { + static function (Generator $breadcrumbs): void { $breadcrumbs->parent('admin.index'); $breadcrumbs->push(trans('firefly.blocked_domains'), route('admin.users.domains')); } @@ -218,14 +218,14 @@ Breadcrumbs::for( Breadcrumbs::for( 'admin.configuration.index', - static function (Generator $breadcrumbs) { + static function (Generator $breadcrumbs): void { $breadcrumbs->parent('admin.index'); $breadcrumbs->push(trans('firefly.instance_configuration'), route('admin.configuration.index')); } ); Breadcrumbs::for( 'admin.update-check', - static function (Generator $breadcrumbs) { + static function (Generator $breadcrumbs): void { $breadcrumbs->parent('admin.index'); $breadcrumbs->push(trans('firefly.update_check_title'), route('admin.update-check')); } @@ -233,7 +233,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'admin.links.index', - static function (Generator $breadcrumbs) { + static function (Generator $breadcrumbs): void { $breadcrumbs->parent('admin.index'); $breadcrumbs->push(trans('firefly.journal_link_configuration'), route('admin.links.index')); } @@ -241,7 +241,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'admin.links.create', - static function (Generator $breadcrumbs) { + static function (Generator $breadcrumbs): void { $breadcrumbs->parent('admin.links.index'); $breadcrumbs->push(trans('firefly.create_new_link_type'), route('admin.links.create')); } @@ -249,7 +249,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'admin.links.show', - static function (Generator $breadcrumbs, LinkType $linkType) { + static function (Generator $breadcrumbs, LinkType $linkType): void { $breadcrumbs->parent('admin.links.index'); $breadcrumbs->push(trans('firefly.overview_for_link', ['name' => limitStringLength($linkType->name)]), route('admin.links.show', [$linkType->id])); } @@ -257,7 +257,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'admin.links.edit', - static function (Generator $breadcrumbs, LinkType $linkType) { + static function (Generator $breadcrumbs, LinkType $linkType): void { $breadcrumbs->parent('admin.links.index'); $breadcrumbs->push(trans('firefly.edit_link_type', ['name' => limitStringLength($linkType->name)]), route('admin.links.edit', [$linkType->id])); } @@ -265,7 +265,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'admin.links.delete', - static function (Generator $breadcrumbs, LinkType $linkType) { + static function (Generator $breadcrumbs, LinkType $linkType): void { $breadcrumbs->parent('admin.links.index'); $breadcrumbs->push(trans('firefly.delete_link_type', ['name' => limitStringLength($linkType->name)]), route('admin.links.delete', [$linkType->id])); } @@ -273,7 +273,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'transactions.link.delete', - static function (Generator $breadcrumbs, TransactionJournalLink $link) { + static function (Generator $breadcrumbs, TransactionJournalLink $link): void { $breadcrumbs->parent('home'); $breadcrumbs->push(trans('breadcrumbs.delete_journal_link'), route('transactions.link.delete', $link->id)); } @@ -282,7 +282,7 @@ Breadcrumbs::for( // ATTACHMENTS Breadcrumbs::for( 'attachments.index', - static function (Generator $breadcrumbs) { + static function (Generator $breadcrumbs): void { $breadcrumbs->parent('home'); $breadcrumbs->push(trans('firefly.attachments'), route('attachments.index')); } @@ -290,7 +290,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'attachments.edit', - static function (Generator $breadcrumbs, Attachment $attachment) { + static function (Generator $breadcrumbs, Attachment $attachment): void { $object = $attachment->attachable; if ($object instanceof TransactionJournal) { $group = $object->transactionGroup; @@ -310,7 +310,7 @@ Breadcrumbs::for( ); Breadcrumbs::for( 'attachments.delete', - static function (Generator $breadcrumbs, Attachment $attachment) { + static function (Generator $breadcrumbs, Attachment $attachment): void { $object = $attachment->attachable; if ($object instanceof TransactionJournal) { $breadcrumbs->parent('transactions.show', $object->transactionGroup); @@ -328,14 +328,14 @@ Breadcrumbs::for( // BILLS Breadcrumbs::for( 'bills.index', - static function (Generator $breadcrumbs) { + static function (Generator $breadcrumbs): void { $breadcrumbs->parent('home'); $breadcrumbs->push(trans('breadcrumbs.bills'), route('bills.index')); } ); Breadcrumbs::for( 'bills.create', - static function (Generator $breadcrumbs) { + static function (Generator $breadcrumbs): void { $breadcrumbs->parent('bills.index'); $breadcrumbs->push(trans('breadcrumbs.newBill'), route('bills.create')); } @@ -343,14 +343,14 @@ Breadcrumbs::for( Breadcrumbs::for( 'bills.edit', - static function (Generator $breadcrumbs, Bill $bill) { + static function (Generator $breadcrumbs, Bill $bill): void { $breadcrumbs->parent('bills.show', $bill); $breadcrumbs->push(trans('breadcrumbs.edit_bill', ['name' => limitStringLength($bill->name)]), route('bills.edit', [$bill->id])); } ); Breadcrumbs::for( 'bills.delete', - static function (Generator $breadcrumbs, Bill $bill) { + static function (Generator $breadcrumbs, Bill $bill): void { $breadcrumbs->parent('bills.show', $bill); $breadcrumbs->push(trans('breadcrumbs.delete_bill', ['name' => limitStringLength($bill->name)]), route('bills.delete', [$bill->id])); } @@ -358,7 +358,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'bills.show', - static function (Generator $breadcrumbs, Bill $bill) { + static function (Generator $breadcrumbs, Bill $bill): void { $breadcrumbs->parent('bills.index'); $breadcrumbs->push(limitStringLength($bill->name), route('bills.show', [$bill->id])); } @@ -367,14 +367,14 @@ Breadcrumbs::for( // SUBSCRIPTIONS Breadcrumbs::for( 'subscriptions.index', - static function (Generator $breadcrumbs) { + static function (Generator $breadcrumbs): void { $breadcrumbs->parent('home'); $breadcrumbs->push(trans('breadcrumbs.bills'), route('subscriptions.index')); } ); Breadcrumbs::for( 'subscriptions.create', - static function (Generator $breadcrumbs) { + static function (Generator $breadcrumbs): void { $breadcrumbs->parent('subscriptions.index'); $breadcrumbs->push(trans('breadcrumbs.newBill'), route('subscriptions.create')); } @@ -382,14 +382,14 @@ Breadcrumbs::for( Breadcrumbs::for( 'subscriptions.edit', - static function (Generator $breadcrumbs, Bill $bill) { + static function (Generator $breadcrumbs, Bill $bill): void { $breadcrumbs->parent('subscriptions.show', $bill); $breadcrumbs->push(trans('breadcrumbs.edit_bill', ['name' => limitStringLength($bill->name)]), route('subscriptions.edit', [$bill->id])); } ); Breadcrumbs::for( 'subscriptions.delete', - static function (Generator $breadcrumbs, Bill $bill) { + static function (Generator $breadcrumbs, Bill $bill): void { $breadcrumbs->parent('subscriptions.show', $bill); $breadcrumbs->push(trans('breadcrumbs.delete_bill', ['name' => limitStringLength($bill->name)]), route('subscriptions.delete', [$bill->id])); } @@ -397,7 +397,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'subscriptions.show', - static function (Generator $breadcrumbs, Bill $bill) { + static function (Generator $breadcrumbs, Bill $bill): void { $breadcrumbs->parent('subscriptions.index'); $breadcrumbs->push(limitStringLength($bill->name), route('subscriptions.show', [$bill->id])); } @@ -406,14 +406,14 @@ Breadcrumbs::for( // BUDGETS Breadcrumbs::for( 'budgets.index', - static function (Generator $breadcrumbs) { + static function (Generator $breadcrumbs): void { $breadcrumbs->parent('home'); $breadcrumbs->push(trans('firefly.budgets'), route('budgets.index')); } ); Breadcrumbs::for( 'budgets.create', - static function (Generator $breadcrumbs) { + static function (Generator $breadcrumbs): void { $breadcrumbs->parent('budgets.index'); $breadcrumbs->push(trans('firefly.create_new_budget'), route('budgets.create')); } @@ -421,14 +421,14 @@ Breadcrumbs::for( Breadcrumbs::for( 'budgets.edit', - static function (Generator $breadcrumbs, Budget $budget) { + static function (Generator $breadcrumbs, Budget $budget): void { $breadcrumbs->parent('budgets.show', $budget); $breadcrumbs->push(trans('firefly.edit_budget', ['name' => limitStringLength($budget->name)]), route('budgets.edit', [$budget->id])); } ); Breadcrumbs::for( 'budgets.delete', - static function (Generator $breadcrumbs, Budget $budget) { + static function (Generator $breadcrumbs, Budget $budget): void { $breadcrumbs->parent('budgets.show', $budget); $breadcrumbs->push(trans('firefly.delete_budget', ['name' => limitStringLength($budget->name)]), route('budgets.delete', [$budget->id])); } @@ -436,7 +436,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'budgets.no-budget', - static function (Generator $breadcrumbs, Carbon $start = null, Carbon $end = null) { + static function (Generator $breadcrumbs, Carbon $start = null, Carbon $end = null): void { $breadcrumbs->parent('budgets.index'); $breadcrumbs->push(trans('firefly.journals_without_budget'), route('budgets.no-budget')); if (null !== $start && null !== $end) { @@ -454,7 +454,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'budgets.no-budget-all', - static function (Generator $breadcrumbs) { + static function (Generator $breadcrumbs): void { $breadcrumbs->parent('budgets.index'); $breadcrumbs->push(trans('firefly.journals_without_budget'), route('budgets.no-budget')); $breadcrumbs->push(trans('firefly.everything'), route('budgets.no-budget-all')); @@ -463,7 +463,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'budgets.show', - static function (Generator $breadcrumbs, Budget $budget) { + static function (Generator $breadcrumbs, Budget $budget): void { $breadcrumbs->parent('budgets.index'); $breadcrumbs->push(limitStringLength($budget->name), route('budgets.show', [$budget->id])); $breadcrumbs->push(trans('firefly.everything'), route('budgets.show', [$budget->id])); @@ -472,7 +472,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'budgets.show.limit', - static function (Generator $breadcrumbs, Budget $budget, BudgetLimit $budgetLimit) { + static function (Generator $breadcrumbs, Budget $budget, BudgetLimit $budgetLimit): void { $breadcrumbs->parent('budgets.index'); $breadcrumbs->push(limitStringLength($budget->name), route('budgets.show', [$budget->id])); @@ -494,14 +494,14 @@ Breadcrumbs::for( // CATEGORIES Breadcrumbs::for( 'categories.index', - static function (Generator $breadcrumbs) { + static function (Generator $breadcrumbs): void { $breadcrumbs->parent('home'); $breadcrumbs->push(trans('firefly.categories'), route('categories.index')); } ); Breadcrumbs::for( 'categories.create', - static function (Generator $breadcrumbs) { + static function (Generator $breadcrumbs): void { $breadcrumbs->parent('categories.index'); $breadcrumbs->push(trans('firefly.new_category'), route('categories.create')); } @@ -509,14 +509,14 @@ Breadcrumbs::for( Breadcrumbs::for( 'categories.edit', - static function (Generator $breadcrumbs, Category $category) { + static function (Generator $breadcrumbs, Category $category): void { $breadcrumbs->parent('categories.show.all', $category); $breadcrumbs->push(trans('firefly.edit_category', ['name' => limitStringLength($category->name)]), route('categories.edit', [$category->id])); } ); Breadcrumbs::for( 'categories.delete', - static function (Generator $breadcrumbs, Category $category) { + static function (Generator $breadcrumbs, Category $category): void { $breadcrumbs->parent('categories.show', $category); $breadcrumbs->push(trans('firefly.delete_category', ['name' => limitStringLength($category->name)]), route('categories.delete', [$category->id])); } @@ -524,7 +524,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'categories.show', - static function (Generator $breadcrumbs, Category $category, Carbon $start = null, Carbon $end = null) { + static function (Generator $breadcrumbs, Category $category, Carbon $start = null, Carbon $end = null): void { $breadcrumbs->parent('categories.index'); $breadcrumbs->push(limitStringLength($category->name), route('categories.show', [$category->id])); if (null !== $start && null !== $end) { @@ -542,7 +542,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'categories.show.all', - static function (Generator $breadcrumbs, Category $category) { + static function (Generator $breadcrumbs, Category $category): void { $breadcrumbs->parent('categories.index'); $breadcrumbs->push(limitStringLength($category->name), route('categories.show', [$category->id])); $breadcrumbs->push(trans('firefly.everything'), route('categories.show.all', [$category->id])); @@ -551,7 +551,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'categories.no-category', - static function (Generator $breadcrumbs, Carbon $start = null, Carbon $end = null) { + static function (Generator $breadcrumbs, Carbon $start = null, Carbon $end = null): void { $breadcrumbs->parent('categories.index'); $breadcrumbs->push(trans('firefly.journals_without_category'), route('categories.no-category')); if (null !== $start && null !== $end) { @@ -569,7 +569,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'categories.no-category.all', - static function (Generator $breadcrumbs) { + static function (Generator $breadcrumbs): void { $breadcrumbs->parent('categories.index'); $breadcrumbs->push(trans('firefly.journals_without_category'), route('categories.no-category')); $breadcrumbs->push(trans('firefly.everything'), route('categories.no-category.all')); @@ -579,7 +579,7 @@ Breadcrumbs::for( // CURRENCIES Breadcrumbs::for( 'currencies.index', - static function (Generator $breadcrumbs) { + static function (Generator $breadcrumbs): void { $breadcrumbs->parent('home'); $breadcrumbs->push(trans('firefly.currencies'), route('currencies.index')); } @@ -587,7 +587,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'currencies.create', - static function (Generator $breadcrumbs) { + static function (Generator $breadcrumbs): void { $breadcrumbs->parent('currencies.index'); $breadcrumbs->push(trans('firefly.create_currency'), route('currencies.create')); } @@ -595,14 +595,14 @@ Breadcrumbs::for( Breadcrumbs::for( 'currencies.edit', - static function (Generator $breadcrumbs, TransactionCurrency $currency) { + static function (Generator $breadcrumbs, TransactionCurrency $currency): void { $breadcrumbs->parent('currencies.index'); $breadcrumbs->push(trans('breadcrumbs.edit_currency', ['name' => $currency->name]), route('currencies.edit', [$currency->id])); } ); Breadcrumbs::for( 'currencies.delete', - static function (Generator $breadcrumbs, TransactionCurrency $currency) { + static function (Generator $breadcrumbs, TransactionCurrency $currency): void { $breadcrumbs->parent('currencies.index'); $breadcrumbs->push(trans('breadcrumbs.delete_currency', ['name' => $currency->name]), route('currencies.delete', [$currency->id])); } @@ -611,7 +611,7 @@ Breadcrumbs::for( // EXPORT Breadcrumbs::for( 'export.index', - static function (Generator $breadcrumbs) { + static function (Generator $breadcrumbs): void { $breadcrumbs->parent('home'); $breadcrumbs->push(trans('firefly.export_data_bc'), route('export.index')); } @@ -620,14 +620,14 @@ Breadcrumbs::for( // PIGGY BANKS Breadcrumbs::for( 'piggy-banks.index', - static function (Generator $breadcrumbs) { + static function (Generator $breadcrumbs): void { $breadcrumbs->parent('home'); $breadcrumbs->push(trans('firefly.piggyBanks'), route('piggy-banks.index')); } ); Breadcrumbs::for( 'piggy-banks.create', - static function (Generator $breadcrumbs) { + static function (Generator $breadcrumbs): void { $breadcrumbs->parent('piggy-banks.index'); $breadcrumbs->push(trans('breadcrumbs.newPiggyBank'), route('piggy-banks.create')); } @@ -635,14 +635,14 @@ Breadcrumbs::for( Breadcrumbs::for( 'piggy-banks.edit', - static function (Generator $breadcrumbs, PiggyBank $piggyBank) { + static function (Generator $breadcrumbs, PiggyBank $piggyBank): void { $breadcrumbs->parent('piggy-banks.show', $piggyBank); $breadcrumbs->push(trans('breadcrumbs.edit_piggyBank', ['name' => $piggyBank->name]), route('piggy-banks.edit', [$piggyBank->id])); } ); Breadcrumbs::for( 'piggy-banks.delete', - static function (Generator $breadcrumbs, PiggyBank $piggyBank) { + static function (Generator $breadcrumbs, PiggyBank $piggyBank): void { $breadcrumbs->parent('piggy-banks.show', $piggyBank); $breadcrumbs->push(trans('firefly.delete_piggy_bank', ['name' => $piggyBank->name]), route('piggy-banks.delete', [$piggyBank->id])); } @@ -650,7 +650,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'piggy-banks.show', - static function (Generator $breadcrumbs, PiggyBank $piggyBank) { + static function (Generator $breadcrumbs, PiggyBank $piggyBank): void { $breadcrumbs->parent('piggy-banks.index'); $breadcrumbs->push($piggyBank->name, route('piggy-banks.show', [$piggyBank->id])); } @@ -658,7 +658,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'piggy-banks.add-money-mobile', - static function (Generator $breadcrumbs, PiggyBank $piggyBank) { + static function (Generator $breadcrumbs, PiggyBank $piggyBank): void { $breadcrumbs->parent('piggy-banks.show', $piggyBank); $breadcrumbs->push(trans('firefly.add_money_to_piggy', ['name' => $piggyBank->name]), route('piggy-banks.add-money-mobile', [$piggyBank->id])); } @@ -666,7 +666,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'piggy-banks.remove-money-mobile', - static function (Generator $breadcrumbs, PiggyBank $piggyBank) { + static function (Generator $breadcrumbs, PiggyBank $piggyBank): void { $breadcrumbs->parent('piggy-banks.show', $piggyBank); $breadcrumbs->push( trans('firefly.remove_money_from_piggy_title', ['name' => $piggyBank->name]), @@ -678,7 +678,7 @@ Breadcrumbs::for( // PREFERENCES Breadcrumbs::for( 'preferences.index', - static function (Generator $breadcrumbs) { + static function (Generator $breadcrumbs): void { $breadcrumbs->parent('home'); $breadcrumbs->push(trans('breadcrumbs.preferences'), route('preferences.index')); } @@ -686,7 +686,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'profile.code', - static function (Generator $breadcrumbs) { + static function (Generator $breadcrumbs): void { $breadcrumbs->parent('home'); $breadcrumbs->push(trans('breadcrumbs.profile'), route('profile.index')); } @@ -694,7 +694,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'profile.new-backup-codes', - static function (Generator $breadcrumbs) { + static function (Generator $breadcrumbs): void { $breadcrumbs->parent('home'); $breadcrumbs->push(trans('breadcrumbs.profile'), route('profile.index')); } @@ -702,7 +702,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'profile.logout-others', - static function (Generator $breadcrumbs) { + static function (Generator $breadcrumbs): void { $breadcrumbs->parent('home'); $breadcrumbs->push(trans('breadcrumbs.logout_others'), route('profile.logout-others')); } @@ -711,14 +711,14 @@ Breadcrumbs::for( // PROFILE Breadcrumbs::for( 'profile.index', - static function (Generator $breadcrumbs) { + static function (Generator $breadcrumbs): void { $breadcrumbs->parent('home'); $breadcrumbs->push(trans('breadcrumbs.profile'), route('profile.index')); } ); Breadcrumbs::for( 'profile.change-password', - static function (Generator $breadcrumbs) { + static function (Generator $breadcrumbs): void { $breadcrumbs->parent('profile.index'); $breadcrumbs->push(trans('breadcrumbs.changePassword'), route('profile.change-password')); } @@ -726,7 +726,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'profile.change-email', - static function (Generator $breadcrumbs) { + static function (Generator $breadcrumbs): void { $breadcrumbs->parent('profile.index'); $breadcrumbs->push(trans('breadcrumbs.change_email'), route('profile.change-email')); } @@ -734,7 +734,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'profile.delete-account', - static function (Generator $breadcrumbs) { + static function (Generator $breadcrumbs): void { $breadcrumbs->parent('profile.index'); $breadcrumbs->push(trans('firefly.delete_account'), route('profile.delete-account')); } @@ -743,7 +743,7 @@ Breadcrumbs::for( // REPORTS Breadcrumbs::for( 'reports.index', - static function (Generator $breadcrumbs) { + static function (Generator $breadcrumbs): void { $breadcrumbs->parent('home'); $breadcrumbs->push(trans('breadcrumbs.reports'), route('reports.index')); } @@ -751,7 +751,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'reports.report.audit', - static function (Generator $breadcrumbs, string $accountIds, Carbon $start, Carbon $end) { + static function (Generator $breadcrumbs, string $accountIds, Carbon $start, Carbon $end): void { $breadcrumbs->parent('reports.index'); $monthFormat = (string)trans('config.month_and_day_js'); @@ -764,7 +764,7 @@ Breadcrumbs::for( ); Breadcrumbs::for( 'reports.report.budget', - static function (Generator $breadcrumbs, string $accountIds, string $budgetIds, Carbon $start, Carbon $end) { + static function (Generator $breadcrumbs, string $accountIds, string $budgetIds, Carbon $start, Carbon $end): void { $breadcrumbs->parent('reports.index'); $monthFormat = (string)trans('config.month_and_day_js'); @@ -778,7 +778,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'reports.report.tag', - static function (Generator $breadcrumbs, string $accountIds, string $tagTags, Carbon $start, Carbon $end) { + static function (Generator $breadcrumbs, string $accountIds, string $tagTags, Carbon $start, Carbon $end): void { $breadcrumbs->parent('reports.index'); $monthFormat = (string)trans('config.month_and_day_js'); @@ -792,7 +792,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'reports.report.category', - static function (Generator $breadcrumbs, string $accountIds, string $categoryIds, Carbon $start, Carbon $end) { + static function (Generator $breadcrumbs, string $accountIds, string $categoryIds, Carbon $start, Carbon $end): void { $breadcrumbs->parent('reports.index'); $monthFormat = (string)trans('config.month_and_day_js'); @@ -806,7 +806,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'reports.report.double', - static function (Generator $breadcrumbs, string $accountIds, string $doubleIds, Carbon $start, Carbon $end) { + static function (Generator $breadcrumbs, string $accountIds, string $doubleIds, Carbon $start, Carbon $end): void { $breadcrumbs->parent('reports.index'); $monthFormat = (string)trans('config.month_and_day_js'); @@ -820,7 +820,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'reports.report.default', - static function (Generator $breadcrumbs, string $accountIds, Carbon $start, Carbon $end) { + static function (Generator $breadcrumbs, string $accountIds, Carbon $start, Carbon $end): void { $breadcrumbs->parent('reports.index'); $monthFormat = (string)trans('config.month_and_day_js'); @@ -835,7 +835,7 @@ Breadcrumbs::for( // New user Controller Breadcrumbs::for( 'new-user.index', - static function (Generator $breadcrumbs) { + static function (Generator $breadcrumbs): void { $breadcrumbs->parent('home'); $breadcrumbs->push(trans('firefly.getting_started'), route('new-user.index')); } @@ -844,14 +844,14 @@ Breadcrumbs::for( // Recurring transactions controller: Breadcrumbs::for( 'recurring.index', - static function (Generator $breadcrumbs) { + static function (Generator $breadcrumbs): void { $breadcrumbs->parent('home'); $breadcrumbs->push(trans('firefly.recurrences'), route('recurring.index')); } ); Breadcrumbs::for( 'recurring.show', - static function (Generator $breadcrumbs, Recurrence $recurrence) { + static function (Generator $breadcrumbs, Recurrence $recurrence): void { $breadcrumbs->parent('recurring.index'); $breadcrumbs->push($recurrence->title, route('recurring.show', [$recurrence->id])); } @@ -859,7 +859,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'recurring.delete', - static function (Generator $breadcrumbs, Recurrence $recurrence) { + static function (Generator $breadcrumbs, Recurrence $recurrence): void { $breadcrumbs->parent('recurring.index'); $breadcrumbs->push(trans('firefly.delete_recurring', ['title' => $recurrence->title]), route('recurring.delete', [$recurrence->id])); } @@ -867,7 +867,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'recurring.edit', - static function (Generator $breadcrumbs, Recurrence $recurrence) { + static function (Generator $breadcrumbs, Recurrence $recurrence): void { $breadcrumbs->parent('recurring.index'); $breadcrumbs->push(trans('firefly.edit_recurrence', ['title' => $recurrence->title]), route('recurring.edit', [$recurrence->id])); } @@ -875,7 +875,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'recurring.create', - static function (Generator $breadcrumbs) { + static function (Generator $breadcrumbs): void { $breadcrumbs->parent('recurring.index'); $breadcrumbs->push(trans('firefly.create_new_recurrence'), route('recurring.create')); } @@ -883,7 +883,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'recurring.create-from-journal', - static function (Generator $breadcrumbs) { + static function (Generator $breadcrumbs): void { $breadcrumbs->parent('recurring.index'); $breadcrumbs->push(trans('firefly.create_new_recurrence'), route('recurring.create')); } @@ -892,7 +892,7 @@ Breadcrumbs::for( // Rules Breadcrumbs::for( 'rules.index', - static function (Generator $breadcrumbs) { + static function (Generator $breadcrumbs): void { $breadcrumbs->parent('home'); $breadcrumbs->push(trans('firefly.rules'), route('rules.index')); } @@ -900,7 +900,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'rules.create', - static function (Generator $breadcrumbs, RuleGroup $ruleGroup = null) { + static function (Generator $breadcrumbs, RuleGroup $ruleGroup = null): void { $breadcrumbs->parent('rules.index'); if (null === $ruleGroup) { $breadcrumbs->push(trans('firefly.make_new_rule_no_group'), route('rules.create')); @@ -913,7 +913,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'rules.create-from-bill', - static function (Generator $breadcrumbs) { + static function (Generator $breadcrumbs): void { $breadcrumbs->parent('rules.index'); $breadcrumbs->push(trans('firefly.make_new_rule_no_group'), route('rules.create')); } @@ -921,7 +921,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'rules.create-from-journal', - static function (Generator $breadcrumbs) { + static function (Generator $breadcrumbs): void { $breadcrumbs->parent('rules.index'); $breadcrumbs->push(trans('firefly.make_new_rule_no_group'), route('rules.create')); } @@ -929,35 +929,35 @@ Breadcrumbs::for( Breadcrumbs::for( 'rules.edit', - static function (Generator $breadcrumbs, Rule $rule) { + static function (Generator $breadcrumbs, Rule $rule): void { $breadcrumbs->parent('rules.index'); $breadcrumbs->push(trans('firefly.edit_rule', ['title' => $rule->title]), route('rules.edit', [$rule])); } ); Breadcrumbs::for( 'rules.delete', - static function (Generator $breadcrumbs, Rule $rule) { + static function (Generator $breadcrumbs, Rule $rule): void { $breadcrumbs->parent('rules.index'); $breadcrumbs->push(trans('firefly.delete_rule', ['title' => $rule->title]), route('rules.delete', [$rule])); } ); Breadcrumbs::for( 'rule-groups.create', - static function (Generator $breadcrumbs) { + static function (Generator $breadcrumbs): void { $breadcrumbs->parent('rules.index'); $breadcrumbs->push(trans('firefly.make_new_rule_group'), route('rule-groups.create')); } ); Breadcrumbs::for( 'rule-groups.edit', - static function (Generator $breadcrumbs, RuleGroup $ruleGroup) { + static function (Generator $breadcrumbs, RuleGroup $ruleGroup): void { $breadcrumbs->parent('rules.index'); $breadcrumbs->push(trans('firefly.edit_rule_group', ['title' => $ruleGroup->title]), route('rule-groups.edit', [$ruleGroup])); } ); Breadcrumbs::for( 'rule-groups.delete', - static function (Generator $breadcrumbs, RuleGroup $ruleGroup) { + static function (Generator $breadcrumbs, RuleGroup $ruleGroup): void { $breadcrumbs->parent('rules.index'); $breadcrumbs->push(trans('firefly.delete_rule_group', ['title' => $ruleGroup->title]), route('rule-groups.delete', [$ruleGroup])); } @@ -965,7 +965,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'rules.select-transactions', - static function (Generator $breadcrumbs, Rule $rule) { + static function (Generator $breadcrumbs, Rule $rule): void { $breadcrumbs->parent('rules.index'); $breadcrumbs->push( trans('firefly.rule_select_transactions', ['title' => $rule->title]), @@ -976,7 +976,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'rule-groups.select-transactions', - static function (Generator $breadcrumbs, RuleGroup $ruleGroup) { + static function (Generator $breadcrumbs, RuleGroup $ruleGroup): void { $breadcrumbs->parent('rules.index'); $breadcrumbs->push( trans('firefly.rule_group_select_transactions', ['title' => $ruleGroup->title]), @@ -988,7 +988,7 @@ Breadcrumbs::for( // SEARCH Breadcrumbs::for( 'search.index', - static function (Generator $breadcrumbs, $query) { + static function (Generator $breadcrumbs, $query): void { $breadcrumbs->parent('home'); $breadcrumbs->push(trans('breadcrumbs.search_result', ['query' => $query]), route('search.index')); } @@ -997,7 +997,7 @@ Breadcrumbs::for( // TAGS Breadcrumbs::for( 'tags.index', - static function (Generator $breadcrumbs) { + static function (Generator $breadcrumbs): void { $breadcrumbs->parent('home'); $breadcrumbs->push(trans('breadcrumbs.tags'), route('tags.index')); } @@ -1005,7 +1005,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'tags.create', - static function (Generator $breadcrumbs) { + static function (Generator $breadcrumbs): void { $breadcrumbs->parent('tags.index'); $breadcrumbs->push(trans('breadcrumbs.createTag'), route('tags.create')); } @@ -1013,7 +1013,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'tags.edit', - static function (Generator $breadcrumbs, Tag $tag) { + static function (Generator $breadcrumbs, Tag $tag): void { $breadcrumbs->parent('tags.show', $tag); $breadcrumbs->push(trans('breadcrumbs.edit_tag', ['tag' => $tag->tag]), route('tags.edit', [$tag->id])); } @@ -1021,7 +1021,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'tags.delete', - static function (Generator $breadcrumbs, Tag $tag) { + static function (Generator $breadcrumbs, Tag $tag): void { $breadcrumbs->parent('tags.show', $tag); $breadcrumbs->push(trans('breadcrumbs.delete_tag', ['tag' => $tag->tag]), route('tags.delete', [$tag->id])); } @@ -1029,7 +1029,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'tags.show', - static function (Generator $breadcrumbs, Tag $tag, Carbon $start = null, Carbon $end = null) { + static function (Generator $breadcrumbs, Tag $tag, Carbon $start = null, Carbon $end = null): void { $breadcrumbs->parent('tags.index'); $breadcrumbs->push($tag->tag, route('tags.show', [$tag->id, $start, $end])); @@ -1048,7 +1048,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'tags.show.all', - static function (Generator $breadcrumbs, Tag $tag) { + static function (Generator $breadcrumbs, Tag $tag): void { $breadcrumbs->parent('tags.index'); $breadcrumbs->push($tag->tag, route('tags.show', [$tag->id])); $title = (string)trans('firefly.all_journals_for_tag', ['tag' => $tag->tag]); @@ -1060,7 +1060,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'transactions.index', - static function (Generator $breadcrumbs, string $what, Carbon $start = null, Carbon $end = null) { + static function (Generator $breadcrumbs, string $what, Carbon $start = null, Carbon $end = null): void { $breadcrumbs->parent('home'); $breadcrumbs->push(trans('breadcrumbs.'.$what.'_list'), route('transactions.index', [$what])); @@ -1080,7 +1080,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'transactions.index.all', - static function (Generator $breadcrumbs, string $what) { + static function (Generator $breadcrumbs, string $what): void { $breadcrumbs->parent('home'); $breadcrumbs->push(trans('breadcrumbs.'.$what.'_list'), route('transactions.index', [$what])); } @@ -1088,7 +1088,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'transactions.create', - static function (Generator $breadcrumbs, string $objectType) { + static function (Generator $breadcrumbs, string $objectType): void { $breadcrumbs->parent('transactions.index', $objectType); $breadcrumbs->push(trans(sprintf('breadcrumbs.create_%s', strtolower($objectType))), route('transactions.create', [$objectType])); } @@ -1096,7 +1096,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'transactions.edit', - static function (Generator $breadcrumbs, TransactionGroup $group) { + static function (Generator $breadcrumbs, TransactionGroup $group): void { $breadcrumbs->parent('transactions.show', $group); /** @var TransactionJournal $first */ @@ -1112,7 +1112,7 @@ Breadcrumbs::for( // also edit reconciliations: Breadcrumbs::for( 'accounts.reconcile.edit', - static function (Generator $breadcrumbs, TransactionJournal $journal) { + static function (Generator $breadcrumbs, TransactionJournal $journal): void { $breadcrumbs->parent('transactions.show', $journal); $breadcrumbs->push( trans('breadcrumbs.edit_reconciliation', ['description' => limitStringLength($journal->description)]), @@ -1123,7 +1123,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'transactions.delete', - static function (Generator $breadcrumbs, TransactionGroup $group) { + static function (Generator $breadcrumbs, TransactionGroup $group): void { $breadcrumbs->parent('transactions.show', $group); $journal = $group->transactionJournals->first(); @@ -1136,7 +1136,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'transactions.show', - static function (Generator $breadcrumbs, TransactionGroup $group) { + static function (Generator $breadcrumbs, TransactionGroup $group): void { /** @var TransactionJournal $first */ $first = $group->transactionJournals()->first(); $type = strtolower($first->transactionType->type); @@ -1164,7 +1164,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'transactions.convert.index', - static function (Generator $breadcrumbs, TransactionGroup $group, string $groupTitle) { + static function (Generator $breadcrumbs, TransactionGroup $group, string $groupTitle): void { $breadcrumbs->parent('transactions.show', $group); $breadcrumbs->push( trans('firefly.breadcrumb_convert_group', ['description' => limitStringLength($groupTitle)]), @@ -1190,7 +1190,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'transactions.mass.delete', - static function (Generator $breadcrumbs, array $journals) { + static function (Generator $breadcrumbs, array $journals): void { $objectType = strtolower(reset($journals)['transaction_type_type']); $breadcrumbs->parent('transactions.index', $objectType); $breadcrumbs->push(trans('firefly.mass_edit_journals'), route('transactions.mass.delete', [''])); @@ -1225,7 +1225,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'object-groups.edit', - static function (Generator $breadcrumbs, ObjectGroup $objectGroup) { + static function (Generator $breadcrumbs, ObjectGroup $objectGroup): void { $breadcrumbs->parent('object-groups.index'); $breadcrumbs->push(trans('breadcrumbs.edit_object_group', ['title' => $objectGroup->title]), route('object-groups.edit', [$objectGroup->id])); } @@ -1233,7 +1233,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'object-groups.delete', - static function (Generator $breadcrumbs, ObjectGroup $objectGroup) { + static function (Generator $breadcrumbs, ObjectGroup $objectGroup): void { $breadcrumbs->parent('object-groups.index'); $breadcrumbs->push(trans('breadcrumbs.delete_object_group', ['title' => $objectGroup->title]), route('object-groups.delete', [$objectGroup->id])); } @@ -1257,7 +1257,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'webhooks.show', - static function (Generator $breadcrumbs, Webhook $webhook) { + static function (Generator $breadcrumbs, Webhook $webhook): void { $breadcrumbs->parent('webhooks.index'); $breadcrumbs->push(limitStringLength($webhook->title), route('webhooks.show', [$webhook->id])); } @@ -1265,7 +1265,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'webhooks.delete', - static function (Generator $breadcrumbs, Webhook $webhook) { + static function (Generator $breadcrumbs, Webhook $webhook): void { $breadcrumbs->parent('webhooks.show', $webhook); $breadcrumbs->push(trans('firefly.delete_webhook', ['title' => limitStringLength($webhook->title)]), route('webhooks.delete', [$webhook->id])); } @@ -1273,7 +1273,7 @@ Breadcrumbs::for( Breadcrumbs::for( 'webhooks.edit', - static function (Generator $breadcrumbs, Webhook $webhook) { + static function (Generator $breadcrumbs, Webhook $webhook): void { $breadcrumbs->parent('webhooks.show', $webhook); $breadcrumbs->push(trans('firefly.edit_webhook', ['title' => limitStringLength($webhook->title)]), route('webhooks.edit', [$webhook->id])); } diff --git a/routes/web.php b/routes/web.php index d1a342679c..69a96d1ab8 100644 --- a/routes/web.php +++ b/routes/web.php @@ -31,7 +31,7 @@ Route::group( 'as' => 'installer.', 'prefix' => 'install', ], - static function () { + static function (): void { Route::get('', ['uses' => 'InstallController@index', 'as' => 'index']); Route::post('runCommand', ['uses' => 'InstallController@runCommand', 'as' => 'runCommand']); } @@ -39,14 +39,14 @@ Route::group( Route::group( ['middleware' => 'binders-only', 'namespace' => 'FireflyIII\Http\Controllers\System', 'as' => 'cron.', 'prefix' => 'cron'], - static function () { + static function (): void { Route::get('run/{cliToken}', ['uses' => 'CronController@cron', 'as' => 'cron']); } ); Route::group( ['middleware' => 'binders-only', 'namespace' => 'FireflyIII\Http\Controllers\System'], - static function () { + static function (): void { Route::get('offline', static fn () => view('errors.offline')); Route::get('health', ['uses' => 'HealthcheckController@check', 'as' => 'healthcheck']); } @@ -55,7 +55,7 @@ Route::group( // These routes only work when the user is NOT logged in. Route::group( ['middleware' => 'user-not-logged-in', 'namespace' => 'FireflyIII\Http\Controllers'], - static function () { + static function (): void { // Authentication Routes... Route::get('login', ['uses' => 'Auth\LoginController@showLoginForm', 'as' => 'login']); Route::post('login', ['uses' => 'Auth\LoginController@login', 'as' => 'login.post']); @@ -80,7 +80,7 @@ Route::group( // For some other routes, it is only relevant that the user is authenticated. Route::group( ['middleware' => 'user-simple-auth', 'namespace' => 'FireflyIII\Http\Controllers'], - static function () { + static function (): void { Route::get('error', ['uses' => 'DebugController@displayError', 'as' => 'error']); Route::post('logout', ['uses' => 'Auth\LoginController@logout', 'as' => 'logout']); Route::get('flush', ['uses' => 'DebugController@flush', 'as' => 'flush']); @@ -92,7 +92,7 @@ Route::group( // For the two factor routes, the user must be logged in, but NOT 2FA. Account confirmation does not matter here. Route::group( ['middleware' => 'user-logged-in-no-2fa', 'prefix' => 'two-factor', 'as' => 'two-factor.', 'namespace' => 'FireflyIII\Http\Controllers\Auth'], - static function () { + static function (): void { Route::post('submit', ['uses' => 'TwoFactorController@submitMFA', 'as' => 'submit']); Route::get('lost', ['uses' => 'TwoFactorController@lostTwoFactor', 'as' => 'lost']); // can be removed when v2 is live. } @@ -103,7 +103,7 @@ Route::group( // Home Controller. Route::group( ['middleware' => ['user-full-auth'], 'namespace' => 'FireflyIII\Http\Controllers'], - static function () { + static function (): void { Route::get('/', ['uses' => 'HomeController@index', 'as' => 'index']); Route::get('/flash', ['uses' => 'DebugController@testFlash', 'as' => 'test-flash']); Route::get('/home', ['uses' => 'HomeController@index', 'as' => 'home']); @@ -119,7 +119,7 @@ Route::group( */ Route::group( ['middleware' => 'user-full-auth', 'namespace' => 'FireflyIII\Http\Controllers', 'prefix' => 'inactive-accounts', 'as' => 'accounts.'], - static function () { + static function (): void { Route::get('{objectType}', ['uses' => 'Account\IndexController@inactive', 'as' => 'inactive.index'])->where( 'objectType', 'revenue|asset|expense|liabilities' @@ -128,7 +128,7 @@ Route::group( ); Route::group( ['middleware' => 'user-full-auth', 'namespace' => 'FireflyIII\Http\Controllers', 'prefix' => 'accounts', 'as' => 'accounts.'], - static function () { + static function (): void { // show: Route::get('{objectType}', ['uses' => 'Account\IndexController@index', 'as' => 'index'])->where('objectType', 'revenue|asset|expense|liabilities'); @@ -182,7 +182,7 @@ Route::group( // Attachment Controller. Route::group( ['middleware' => 'user-full-auth', 'namespace' => 'FireflyIII\Http\Controllers', 'prefix' => 'attachments', 'as' => 'attachments.'], - static function () { + static function (): void { Route::get('', ['uses' => 'AttachmentController@index', 'as' => 'index']); Route::get('edit/{attachment}', ['uses' => 'AttachmentController@edit', 'as' => 'edit']); Route::get('delete/{attachment}', ['uses' => 'AttachmentController@delete', 'as' => 'delete']); @@ -197,7 +197,7 @@ Route::group( // Bills Controller. Route::group( ['middleware' => 'user-full-auth', 'namespace' => 'FireflyIII\Http\Controllers', 'prefix' => 'bills', 'as' => 'bills.'], - static function () { + static function (): void { Route::get('', ['uses' => 'Bill\IndexController@index', 'as' => 'index']); Route::post('rescan/{bill}', ['uses' => 'Bill\ShowController@rescan', 'as' => 'rescan']); Route::get('create', ['uses' => 'Bill\CreateController@create', 'as' => 'create']); @@ -215,7 +215,7 @@ Route::group( Route::group( ['middleware' => 'user-full-auth', 'namespace' => 'FireflyIII\Http\Controllers', 'prefix' => 'subscriptions', 'as' => 'subscriptions.'], - static function () { + static function (): void { Route::get('', ['uses' => 'Bill\IndexController@index', 'as' => 'index']); Route::post('rescan/{bill}', ['uses' => 'Bill\ShowController@rescan', 'as' => 'rescan']); Route::get('create', ['uses' => 'Bill\CreateController@create', 'as' => 'create']); @@ -234,7 +234,7 @@ Route::group( // Budget Controller. Route::group( ['middleware' => 'user-full-auth', 'namespace' => 'FireflyIII\Http\Controllers', 'prefix' => 'budgets', 'as' => 'budgets.'], - static function () { + static function (): void { // delete Route::get('delete/{budget}', ['uses' => 'Budget\DeleteController@delete', 'as' => 'delete']); Route::post('destroy/{budget}', ['uses' => 'Budget\DeleteController@destroy', 'as' => 'destroy']); @@ -270,7 +270,7 @@ Route::group( // Budget Limit Controller. Route::group( ['middleware' => 'user-full-auth', 'namespace' => 'FireflyIII\Http\Controllers', 'prefix' => 'budget-limits', 'as' => 'budget-limits.'], - static function () { + static function (): void { Route::get('create/{budget}/{start_date}/{end_date}', ['uses' => 'Budget\BudgetLimitController@create', 'as' => 'create']) ->where(['start_date' => DATEFORMAT]) ->where(['end_date' => DATEFORMAT]) @@ -286,7 +286,7 @@ Route::group( // Category Controller. Route::group( ['middleware' => 'user-full-auth', 'namespace' => 'FireflyIII\Http\Controllers', 'prefix' => 'categories', 'as' => 'categories.'], - static function () { + static function (): void { // index: Route::get('', ['uses' => 'Category\IndexController@index', 'as' => 'index']); @@ -321,7 +321,7 @@ Route::group( // Currency Controller. Route::group( ['middleware' => 'user-full-auth', 'namespace' => 'FireflyIII\Http\Controllers\TransactionCurrency', 'prefix' => 'currencies', 'as' => 'currencies.'], - static function () { + static function (): void { Route::get('', ['uses' => 'IndexController@index', 'as' => 'index']); Route::get('create', ['uses' => 'CreateController@create', 'as' => 'create']); Route::get('edit/{currency}', ['uses' => 'EditController@edit', 'as' => 'edit']); @@ -336,7 +336,7 @@ Route::group( // Chart\Account Controller (default report). Route::group( ['middleware' => 'user-full-auth', 'namespace' => 'FireflyIII\Http\Controllers\Chart', 'prefix' => 'chart/account', 'as' => 'chart.account.'], - static function () { + static function (): void { Route::get('frontpage', ['uses' => 'AccountController@frontpage', 'as' => 'frontpage']); Route::get('expense', ['uses' => 'AccountController@expenseAccounts', 'as' => 'expense']); Route::get('revenue', ['uses' => 'AccountController@revenueAccounts', 'as' => 'revenue']); @@ -371,7 +371,7 @@ Route::group( // Chart\Bill Controller. Route::group( ['middleware' => 'user-full-auth', 'namespace' => 'FireflyIII\Http\Controllers\Chart', 'prefix' => 'chart/bill', 'as' => 'chart.bill.'], - static function () { + static function (): void { Route::get('frontpage', ['uses' => 'BillController@frontpage', 'as' => 'frontpage']); Route::get('single/{bill}', ['uses' => 'BillController@single', 'as' => 'single']); } @@ -380,7 +380,7 @@ Route::group( // Chart\Budget Controller. Route::group( ['middleware' => 'user-full-auth', 'namespace' => 'FireflyIII\Http\Controllers\Chart', 'prefix' => 'chart/budget', 'as' => 'chart.budget.'], - static function () { + static function (): void { Route::get('frontpage', ['uses' => 'BudgetController@frontpage', 'as' => 'frontpage']); Route::get('period/0/{currency}/{accountList}/{start_date}/{end_date}', ['uses' => 'BudgetController@periodNoBudget', 'as' => 'period.no-budget']) ->where(['start_date' => DATEFORMAT]) @@ -431,7 +431,7 @@ Route::group( // Chart\Category Controller. Route::group( ['middleware' => 'user-full-auth', 'namespace' => 'FireflyIII\Http\Controllers\Chart', 'prefix' => 'chart/category', 'as' => 'chart.category.'], - static function () { + static function (): void { Route::get('frontpage', ['uses' => 'CategoryController@frontPage', 'as' => 'frontpage']); Route::get('period/{category}', ['uses' => 'CategoryController@currentPeriod', 'as' => 'current']); Route::get('period/{category}/{date}', ['uses' => 'CategoryController@specificPeriod', 'as' => 'specific']); @@ -501,7 +501,7 @@ Route::group( // Chart\Tag Controller. Route::group( ['middleware' => 'user-full-auth', 'namespace' => 'FireflyIII\Http\Controllers\Chart', 'prefix' => 'chart/tag', 'as' => 'chart.tag.'], - static function () { + static function (): void { Route::get('tag/expense/{accountList}/{tagList}/{start_date}/{end_date}', ['uses' => 'TagReportController@tagExpense', 'as' => 'tag-expense']) ->where(['start_date' => DATEFORMAT]) ->where(['end_date' => DATEFORMAT]) @@ -565,7 +565,7 @@ Route::group( // Chart\Double Controller (for expense/revenue report). Route::group( ['middleware' => 'user-full-auth', 'namespace' => 'FireflyIII\Http\Controllers\Chart', 'prefix' => 'chart/double', 'as' => 'chart.double.'], - static function () { + static function (): void { Route::get('main/{accountList}/{account}/{start_date}/{end_date}', ['uses' => 'DoubleReportController@mainChart', 'as' => 'main'])->where( ['start_date' => DATEFORMAT] ) @@ -609,7 +609,7 @@ Route::group( // Chart\PiggyBank Controller. Route::group( ['middleware' => 'user-full-auth', 'namespace' => 'FireflyIII\Http\Controllers\Chart', 'prefix' => 'chart/piggy-bank', 'as' => 'chart.piggy-bank.'], - static function () { + static function (): void { Route::get('{piggyBank}', ['uses' => 'PiggyBankController@history', 'as' => 'history']); } ); @@ -617,7 +617,7 @@ Route::group( // Chart\Report Controller. Route::group( ['middleware' => 'user-full-auth', 'namespace' => 'FireflyIII\Http\Controllers\Chart', 'prefix' => 'chart/report', 'as' => 'chart.report.'], - static function () { + static function (): void { Route::get('operations/{accountList}/{start_date}/{end_date}', ['uses' => 'ReportController@operations', 'as' => 'operations'])->where( ['start_date' => DATEFORMAT] ) @@ -634,7 +634,7 @@ Route::group( // Chart\Transactions Controller. Route::group( ['middleware' => 'user-full-auth', 'namespace' => 'FireflyIII\Http\Controllers\Chart', 'prefix' => 'chart/transactions', 'as' => 'chart.transactions.'], - static function () { + static function (): void { Route::get('categories/{objectType}/{start_date}/{end_date}', ['uses' => 'TransactionController@categories', 'as' => 'categories'])->where( ['start_date' => DATEFORMAT] ) @@ -660,7 +660,7 @@ Route::group( // Export controller. Route::group( ['middleware' => 'user-full-auth', 'namespace' => 'FireflyIII\Http\Controllers', 'prefix' => 'export', 'as' => 'export.'], - static function () { + static function (): void { // index Route::get('', ['uses' => 'Export\IndexController@index', 'as' => 'index']); Route::post('export', ['uses' => 'Export\IndexController@export', 'as' => 'export']); @@ -669,7 +669,7 @@ Route::group( // Object group controller. Route::group( ['middleware' => 'user-full-auth', 'namespace' => 'FireflyIII\Http\Controllers', 'prefix' => 'groups', 'as' => 'object-groups.'], - static function () { + static function (): void { // index Route::get('', ['uses' => 'ObjectGroup\IndexController@index', 'as' => 'index']); Route::post('set-order/{objectGroup}', ['uses' => 'ObjectGroup\IndexController@setOrder', 'as' => 'set-order']); @@ -687,7 +687,7 @@ Route::group( // JScript Controller. Route::group( ['middleware' => 'user-full-auth', 'namespace' => 'FireflyIII\Http\Controllers', 'prefix' => 'v1/jscript', 'as' => 'javascript.'], - static function () { + static function (): void { Route::get('variables', ['uses' => 'JavascriptController@variables', 'as' => 'variables']); Route::get('accounts', ['uses' => 'JavascriptController@accounts', 'as' => 'accounts']); Route::get('currencies', ['uses' => 'JavascriptController@currencies', 'as' => 'currencies']); @@ -697,7 +697,7 @@ Route::group( // JScript Controller. Route::group( ['middleware' => 'user-full-auth', 'namespace' => 'FireflyIII\Http\Controllers', 'prefix' => 'v2/jscript', 'as' => 'javascript.v2.'], - static function () { + static function (): void { Route::get('variables', ['uses' => 'JavascriptController@variablesV2', 'as' => 'variables']); } ); @@ -705,7 +705,7 @@ Route::group( // JSON Controller(s). Route::group( ['middleware' => 'user-full-auth', 'namespace' => 'FireflyIII\Http\Controllers', 'prefix' => 'json', 'as' => 'json.'], - static function () { + static function (): void { // budgets: Route::get( 'budget/total-budgeted/{currency}/{start_date}/{end_date}', @@ -738,7 +738,7 @@ Route::group( // NewUser Controller. Route::group( ['middleware' => 'user-full-auth', 'namespace' => 'FireflyIII\Http\Controllers', 'prefix' => 'new-user', 'as' => 'new-user.'], - static function () { + static function (): void { Route::get('', ['uses' => 'NewUserController@index', 'as' => 'index']); Route::post('submit', ['uses' => 'NewUserController@submit', 'as' => 'submit']); } @@ -747,7 +747,7 @@ Route::group( // Piggy Bank Controller. Route::group( ['middleware' => 'user-full-auth', 'namespace' => 'FireflyIII\Http\Controllers', 'prefix' => 'piggy-banks', 'as' => 'piggy-banks.'], - static function () { + static function (): void { Route::get('', ['uses' => 'PiggyBank\IndexController@index', 'as' => 'index']); Route::get('add/{piggyBank}', ['uses' => 'PiggyBank\AmountController@add', 'as' => 'add-money']); Route::get('remove/{piggyBank}', ['uses' => 'PiggyBank\AmountController@remove', 'as' => 'remove-money']); @@ -770,7 +770,7 @@ Route::group( // Preferences Controller. Route::group( ['middleware' => 'user-full-auth', 'namespace' => 'FireflyIII\Http\Controllers', 'prefix' => 'preferences', 'as' => 'preferences.'], - static function () { + static function (): void { Route::get('', ['uses' => 'PreferencesController@index', 'as' => 'index']); Route::post('', ['uses' => 'PreferencesController@postIndex', 'as' => 'update']); } @@ -779,7 +779,7 @@ Route::group( // Profile Controller. Route::group( ['middleware' => 'user-full-auth', 'namespace' => 'FireflyIII\Http\Controllers', 'prefix' => 'profile', 'as' => 'profile.'], - static function () { + static function (): void { Route::get('', ['uses' => 'ProfileController@index', 'as' => 'index']); Route::get('change-email', ['uses' => 'ProfileController@changeEmail', 'as' => 'change-email']); Route::get('change-password', ['uses' => 'ProfileController@changePassword', 'as' => 'change-password']); @@ -805,7 +805,7 @@ Route::group( // Recurring Transactions Controller. Route::group( ['middleware' => 'user-full-auth', 'namespace' => 'FireflyIII\Http\Controllers', 'prefix' => 'recurring', 'as' => 'recurring.'], - static function () { + static function (): void { Route::get('', ['uses' => 'Recurring\IndexController@index', 'as' => 'index']); Route::get('show/{recurrence}', ['uses' => 'Recurring\ShowController@show', 'as' => 'show']); @@ -828,7 +828,7 @@ Route::group( // Report Controller. Route::group( ['middleware' => 'user-full-auth', 'namespace' => 'FireflyIII\Http\Controllers', 'prefix' => 'reports', 'as' => 'reports.'], - static function () { + static function (): void { Route::get('', ['uses' => 'ReportController@index', 'as' => 'index']); Route::get('options/{reportType}', ['uses' => 'ReportController@options', 'as' => 'options']); Route::get('default/{accountList}/{start_date}/{end_date}', ['uses' => 'ReportController@defaultReport', 'as' => 'report.default']); @@ -848,7 +848,7 @@ Route::group( // Report Data AccountController. Route::group( ['middleware' => 'user-full-auth', 'namespace' => 'FireflyIII\Http\Controllers\Report', 'prefix' => 'report-data/account', 'as' => 'report-data.account.'], - static function () { + static function (): void { Route::get('general/{accountList}/{start_date}/{end_date}', ['uses' => 'AccountController@general', 'as' => 'general'])->where( ['start_date' => DATEFORMAT] ) @@ -860,7 +860,7 @@ Route::group( // Report Data Bill Controller. Route::group( ['middleware' => 'user-full-auth', 'namespace' => 'FireflyIII\Http\Controllers\Report', 'prefix' => 'report-data/bill', 'as' => 'report-data.bills.'], - static function () { + static function (): void { Route::get('overview/{accountList}/{start_date}/{end_date}', ['uses' => 'BillController@overview', 'as' => 'overview'])->where( ['start_date' => DATEFORMAT] ) @@ -872,7 +872,7 @@ Route::group( // Report Double Data Expense / Revenue Account Controller. Route::group( ['middleware' => 'user-full-auth', 'namespace' => 'FireflyIII\Http\Controllers\Report', 'prefix' => 'report-data/double', 'as' => 'report-data.double.'], - static function () { + static function (): void { // spent + earned per combination. Route::get('operations/{accountList}/{doubleList}/{start_date}/{end_date}', ['uses' => 'DoubleController@operations', 'as' => 'operations'])->where( ['start_date' => DATEFORMAT] @@ -920,7 +920,7 @@ Route::group( 'prefix' => 'report-data/operations', 'as' => 'report-data.operations.', ], - static function () { + static function (): void { Route::get('operations/{accountList}/{start_date}/{end_date}', ['uses' => 'OperationsController@operations', 'as' => 'operations'])->where( ['start_date' => DATEFORMAT] ) @@ -947,7 +947,7 @@ Route::group( 'prefix' => 'report-data/category', 'as' => 'report-data.category.', ], - static function () { + static function (): void { // TODO three routes still in use? Route::get('operations/{accountList}/{start_date}/{end_date}', ['uses' => 'CategoryController@operations', 'as' => 'operations'])->where( ['start_date' => DATEFORMAT] @@ -1016,7 +1016,7 @@ Route::group( 'prefix' => 'report-data/tag', 'as' => 'report-data.tag.', ], - static function () { + static function (): void { Route::get('accounts/{accountList}/{tagList}/{start_date}/{end_date}', ['uses' => 'TagController@accounts', 'as' => 'accounts'])->where( ['start_date' => DATEFORMAT] ) @@ -1061,7 +1061,7 @@ Route::group( // Report Data Balance Controller. Route::group( ['middleware' => 'user-full-auth', 'namespace' => 'FireflyIII\Http\Controllers\Report', 'prefix' => 'report-data/balance', 'as' => 'report-data.balance.'], - static function () { + static function (): void { Route::get('general/{accountList}/{start_date}/{end_date}', ['uses' => 'BalanceController@general', 'as' => 'general'])->where( ['start_date' => DATEFORMAT] ) @@ -1073,7 +1073,7 @@ Route::group( // Report Data Budget Controller. Route::group( ['middleware' => 'user-full-auth', 'namespace' => 'FireflyIII\Http\Controllers\Report', 'prefix' => 'report-data/budget', 'as' => 'report-data.budget.'], - static function () { + static function (): void { Route::get('general/{accountList}/{start_date}/{end_date}/', ['uses' => 'BudgetController@general', 'as' => 'general'])->where( ['start_date' => DATEFORMAT] ) @@ -1118,7 +1118,7 @@ Route::group( // Rules Controller. Route::group( ['middleware' => 'user-full-auth', 'namespace' => 'FireflyIII\Http\Controllers', 'prefix' => 'rules', 'as' => 'rules.'], - static function () { + static function (): void { // create controller Route::get('create/{ruleGroup?}', ['uses' => 'Rule\CreateController@create', 'as' => 'create']); Route::get('create-from-bill/{bill}', ['uses' => 'Rule\CreateController@createFromBill', 'as' => 'create-from-bill']); @@ -1150,7 +1150,7 @@ Route::group( // Rule Groups Controller. Route::group( ['middleware' => 'user-full-auth', 'namespace' => 'FireflyIII\Http\Controllers', 'prefix' => 'rule-groups', 'as' => 'rule-groups.'], - static function () { + static function (): void { Route::get('create', ['uses' => 'RuleGroup\CreateController@create', 'as' => 'create']); Route::get('edit/{ruleGroup}', ['uses' => 'RuleGroup\EditController@edit', 'as' => 'edit']); Route::get('delete/{ruleGroup}', ['uses' => 'RuleGroup\DeleteController@delete', 'as' => 'delete']); @@ -1169,7 +1169,7 @@ Route::group( // Search Controller. Route::group( ['middleware' => 'user-full-auth', 'namespace' => 'FireflyIII\Http\Controllers', 'prefix' => 'search', 'as' => 'search.'], - static function () { + static function (): void { Route::get('', ['uses' => 'SearchController@index', 'as' => 'index']); Route::any('search', ['uses' => 'SearchController@search', 'as' => 'search']); } @@ -1178,7 +1178,7 @@ Route::group( // Tag Controller. Route::group( ['middleware' => 'user-full-auth', 'namespace' => 'FireflyIII\Http\Controllers', 'prefix' => 'tags', 'as' => 'tags.'], - static function () { + static function (): void { Route::get('', ['uses' => 'TagController@index', 'as' => 'index']); Route::get('create', ['uses' => 'TagController@create', 'as' => 'create']); @@ -1200,7 +1200,7 @@ Route::group( // Transaction Controller. Route::group( ['middleware' => 'user-full-auth', 'namespace' => 'FireflyIII\Http\Controllers', 'prefix' => 'transactions', 'as' => 'transactions.'], - static function () { + static function (): void { // show groups: // TODO improve these routes Route::get('{what}/all', ['uses' => 'Transaction\IndexController@indexAll', 'as' => 'index.all'])->where( @@ -1239,7 +1239,7 @@ Route::group( // Transaction Mass Controller. Route::group( ['middleware' => 'user-full-auth', 'namespace' => 'FireflyIII\Http\Controllers\Transaction', 'prefix' => 'transactions/mass', 'as' => 'transactions.mass.'], - static function () { + static function (): void { Route::get('edit/{journalList}', ['uses' => 'MassController@edit', 'as' => 'edit']); Route::get('delete/{journalList}', ['uses' => 'MassController@delete', 'as' => 'delete']); Route::post('update', ['uses' => 'MassController@update', 'as' => 'update']); @@ -1250,7 +1250,7 @@ Route::group( // Transaction Bulk Controller. Route::group( ['middleware' => 'user-full-auth', 'namespace' => 'FireflyIII\Http\Controllers\Transaction', 'prefix' => 'transactions/bulk', 'as' => 'transactions.bulk.'], - static function () { + static function (): void { Route::get('edit/{journalList}', ['uses' => 'BulkController@edit', 'as' => 'edit']); Route::post('update', ['uses' => 'BulkController@update', 'as' => 'update']); } @@ -1264,7 +1264,7 @@ Route::group( 'prefix' => 'transactions/convert', 'as' => 'transactions.convert.', ], - static function () { + static function (): void { Route::get('{transactionType}/{transactionGroup}', ['uses' => 'ConvertController@index', 'as' => 'index']); Route::post('{transactionType}/{transactionGroup}', ['uses' => 'ConvertController@postIndex', 'as' => 'index.post']); } @@ -1273,7 +1273,7 @@ Route::group( // Transaction Link Controller. Route::group( ['middleware' => 'user-full-auth', 'namespace' => 'FireflyIII\Http\Controllers\Transaction', 'prefix' => 'transactions/link', 'as' => 'transactions.link.'], - static function () { + static function (): void { Route::get('modal/{tj}', ['uses' => 'LinkController@modal', 'as' => 'modal']); // TODO improve this route @@ -1287,7 +1287,7 @@ Route::group( // Report Popup Controller. Route::group( ['middleware' => 'user-full-auth', 'namespace' => 'FireflyIII\Http\Controllers\Popup', 'prefix' => 'popup', 'as' => 'popup.'], - static function () { + static function (): void { Route::get('general', ['uses' => 'ReportController@general', 'as' => 'general']); } ); @@ -1295,7 +1295,7 @@ Route::group( // Webhooks management Route::group( ['middleware' => 'user-full-auth', 'namespace' => 'FireflyIII\Http\Controllers\Webhooks', 'prefix' => 'webhooks', 'as' => 'webhooks.'], - static function () { + static function (): void { Route::get('index', ['uses' => 'IndexController@index', 'as' => 'index']); Route::get('create', ['uses' => 'CreateController@index', 'as' => 'create']); Route::get('edit/{webhook}', ['uses' => 'EditController@index', 'as' => 'edit']); @@ -1307,7 +1307,7 @@ Route::group( // For the admin routes, the user must be logged in and have the role of 'owner'. Route::group( ['middleware' => 'admin', 'namespace' => 'FireflyIII\Http\Controllers\Admin', 'prefix' => 'admin', 'as' => 'admin.'], - static function () { + static function (): void { // admin home Route::get('', ['uses' => 'HomeController@index', 'as' => 'index']); Route::post('test-message', ['uses' => 'HomeController@testMessage', 'as' => 'test-message']); diff --git a/tests/integration/Support/NavigationCustomEndOfPeriodTest.php b/tests/integration/Support/NavigationCustomEndOfPeriodTest.php index 7dfbb974de..7feba5061a 100644 --- a/tests/integration/Support/NavigationCustomEndOfPeriodTest.php +++ b/tests/integration/Support/NavigationCustomEndOfPeriodTest.php @@ -38,7 +38,7 @@ final class NavigationCustomEndOfPeriodTest extends TestCase /** * @preserveGlobalState disabled */ - public function testGivenADateAndCustomFrequencyWhenCalculateTheDateThenReturnsTheEndOfMonthSuccessful() + public function testGivenADateAndCustomFrequencyWhenCalculateTheDateThenReturnsTheEndOfMonthSuccessful(): void { $from = Carbon::parse('2023-08-05'); $expected = Carbon::parse('2023-09-04'); diff --git a/tests/unit/Support/Calendar/CalculatorTest.php b/tests/unit/Support/Calendar/CalculatorTest.php index 711eaee1f6..ed9f1537c3 100644 --- a/tests/unit/Support/Calendar/CalculatorTest.php +++ b/tests/unit/Support/Calendar/CalculatorTest.php @@ -78,7 +78,7 @@ final class CalculatorTest extends TestCase * * @throws IntervalException */ - public function testGivenADailyPeriodicityWhenCallTheNextDateByIntervalMethodThenReturnsTheExpectedDateSuccessful(CalculatorProvider $provider) + public function testGivenADailyPeriodicityWhenCallTheNextDateByIntervalMethodThenReturnsTheExpectedDateSuccessful(CalculatorProvider $provider): void { $calculator = new Calculator(); $period = $calculator->nextDateByInterval($provider->epoch(), $provider->periodicity); @@ -90,7 +90,7 @@ final class CalculatorTest extends TestCase * * @throws IntervalException */ - public function testGivenAnEpochWithSkipIntervalNumberWhenCallTheNextDateBySkippedIntervalMethodThenReturnsTheExpectedDateSuccessful(CalculatorProvider $provider) + public function testGivenAnEpochWithSkipIntervalNumberWhenCallTheNextDateBySkippedIntervalMethodThenReturnsTheExpectedDateSuccessful(CalculatorProvider $provider): void { $calculator = new Calculator(); $period = $calculator->nextDateByInterval($provider->epoch(), $provider->periodicity, $provider->skip); diff --git a/tests/unit/Support/NavigationAddPeriodTest.php b/tests/unit/Support/NavigationAddPeriodTest.php index bd91acf40b..70bce9defc 100644 --- a/tests/unit/Support/NavigationAddPeriodTest.php +++ b/tests/unit/Support/NavigationAddPeriodTest.php @@ -166,7 +166,7 @@ final class NavigationAddPeriodTest extends TestCase /** * @dataProvider providePeriodsWithSkippingParam */ - public function testGivenAFrequencyAndSkipIntervalWhenCalculateTheDateThenReturnsTheSkippedDateSuccessful(int $skip, string $frequency, Carbon $from, Carbon $expected) + public function testGivenAFrequencyAndSkipIntervalWhenCalculateTheDateThenReturnsTheSkippedDateSuccessful(int $skip, string $frequency, Carbon $from, Carbon $expected): void { $period = $this->navigation->addPeriod($from, $frequency, $skip); self::assertSame($expected->toDateString(), $period->toDateString()); @@ -175,7 +175,7 @@ final class NavigationAddPeriodTest extends TestCase /** * @dataProvider providePeriods */ - public function testGivenAFrequencyWhenCalculateTheDateThenReturnsTheExpectedDateSuccessful(string $frequency, Carbon $from, Carbon $expected) + public function testGivenAFrequencyWhenCalculateTheDateThenReturnsTheExpectedDateSuccessful(string $frequency, Carbon $from, Carbon $expected): void { $period = $this->navigation->addPeriod($from, $frequency, 0); self::assertSame($expected->toDateString(), $period->toDateString()); @@ -184,7 +184,7 @@ final class NavigationAddPeriodTest extends TestCase /** * @dataProvider provideFrequencies */ - public function testGivenAIntervalWhenCallTheNextDateByIntervalMethodThenReturnsTheExpectedDateSuccessful(Periodicity $periodicity, Carbon $from, Carbon $expected) + public function testGivenAIntervalWhenCallTheNextDateByIntervalMethodThenReturnsTheExpectedDateSuccessful(Periodicity $periodicity, Carbon $from, Carbon $expected): void { $period = $this->navigation->nextDateByInterval($from, $periodicity); self::assertSame($expected->toDateString(), $period->toDateString()); @@ -193,7 +193,7 @@ final class NavigationAddPeriodTest extends TestCase /** * @dataProvider provideMonthPeriods */ - public function testGivenAMonthFrequencyWhenCalculateTheDateThenReturnsTheLastDayOfMonthSuccessful(string $frequency, Carbon $from, Carbon $expected) + public function testGivenAMonthFrequencyWhenCalculateTheDateThenReturnsTheLastDayOfMonthSuccessful(string $frequency, Carbon $from, Carbon $expected): void { $period = $this->navigation->addPeriod($from, $frequency, 0); self::assertSame($expected->toDateString(), $period->toDateString()); diff --git a/tests/unit/Support/NavigationEndOfPeriodTest.php b/tests/unit/Support/NavigationEndOfPeriodTest.php index 0474c981a5..0666106b21 100644 --- a/tests/unit/Support/NavigationEndOfPeriodTest.php +++ b/tests/unit/Support/NavigationEndOfPeriodTest.php @@ -80,7 +80,7 @@ final class NavigationEndOfPeriodTest extends TestCase /** * @dataProvider provideDates */ - public function testGivenADateAndFrequencyWhenCalculateTheDateThenReturnsTheExpectedDateSuccessful(string $frequency, Carbon $from, Carbon $expected) + public function testGivenADateAndFrequencyWhenCalculateTheDateThenReturnsTheExpectedDateSuccessful(string $frequency, Carbon $from, Carbon $expected): void { $period = $this->navigation->endOfPeriod($from, $frequency); self::assertSame($expected->toDateString(), $period->toDateString()); @@ -98,7 +98,7 @@ final class NavigationEndOfPeriodTest extends TestCase /** * @dataProvider provideUnknownFrequencies */ - public function testGivenADateAndUnknownFrequencyWhenCalculateTheDateThenReturnsTheSameDateSuccessful(string $frequency, Carbon $from, Carbon $expected) + public function testGivenADateAndUnknownFrequencyWhenCalculateTheDateThenReturnsTheSameDateSuccessful(string $frequency, Carbon $from, Carbon $expected): void { Log::spy(); diff --git a/tests/unit/Support/NavigationPreferredCarbonFormatByPeriodTest.php b/tests/unit/Support/NavigationPreferredCarbonFormatByPeriodTest.php index 9ecb8af6b6..986644aab1 100644 --- a/tests/unit/Support/NavigationPreferredCarbonFormatByPeriodTest.php +++ b/tests/unit/Support/NavigationPreferredCarbonFormatByPeriodTest.php @@ -61,7 +61,7 @@ final class NavigationPreferredCarbonFormatByPeriodTest extends TestCase /** * @dataProvider providePeriods */ - public function testGivenAPeriodWhenCallPreferredCarbonFormatByPeriodThenReturnsExpectedFormat(string $period, string $expected) + public function testGivenAPeriodWhenCallPreferredCarbonFormatByPeriodThenReturnsExpectedFormat(string $period, string $expected): void { $formatPeriod = $this->navigation->preferredCarbonFormatByPeriod($period); self::assertSame($expected, $formatPeriod); diff --git a/tests/unit/Support/NavigationPreferredCarbonFormatTest.php b/tests/unit/Support/NavigationPreferredCarbonFormatTest.php index 099fe70342..304d6b9bbf 100644 --- a/tests/unit/Support/NavigationPreferredCarbonFormatTest.php +++ b/tests/unit/Support/NavigationPreferredCarbonFormatTest.php @@ -68,7 +68,7 @@ final class NavigationPreferredCarbonFormatTest extends TestCase /** * @dataProvider providePeriods */ - public function testGivenStartAndEndDatesWhenCallPreferredCarbonFormatThenReturnsTheExpectedFormatSuccessful(Carbon $start, Carbon $end, string $expected) + public function testGivenStartAndEndDatesWhenCallPreferredCarbonFormatThenReturnsTheExpectedFormatSuccessful(Carbon $start, Carbon $end, string $expected): void { $carbonFormat = $this->navigation->preferredCarbonFormat($start, $end); self::assertSame($expected, $carbonFormat); diff --git a/tests/unit/Support/NavigationPreferredEndOfPeriodTest.php b/tests/unit/Support/NavigationPreferredEndOfPeriodTest.php index f0c937699b..54e8eb132c 100644 --- a/tests/unit/Support/NavigationPreferredEndOfPeriodTest.php +++ b/tests/unit/Support/NavigationPreferredEndOfPeriodTest.php @@ -68,7 +68,7 @@ final class NavigationPreferredEndOfPeriodTest extends TestCase /** * @dataProvider providePeriods */ - public function testGivenStartAndEndDatesWhenCallPreferredEndOfPeriodThenReturnsTheExpectedFormatSuccessful(Carbon $start, Carbon $end, string $expected) + public function testGivenStartAndEndDatesWhenCallPreferredEndOfPeriodThenReturnsTheExpectedFormatSuccessful(Carbon $start, Carbon $end, string $expected): void { $formatPeriod = $this->navigation->preferredEndOfPeriod($start, $end); self::assertSame($expected, $formatPeriod); diff --git a/tests/unit/Support/NavigationPreferredRangeFormatTest.php b/tests/unit/Support/NavigationPreferredRangeFormatTest.php index e6ac727a16..3d328e8e5c 100644 --- a/tests/unit/Support/NavigationPreferredRangeFormatTest.php +++ b/tests/unit/Support/NavigationPreferredRangeFormatTest.php @@ -68,7 +68,7 @@ final class NavigationPreferredRangeFormatTest extends TestCase /** * @dataProvider providePeriods */ - public function testGivenStartAndEndDatesWhenCallPreferredRangeFormatThenReturnsTheExpectedFormatSuccessful(Carbon $start, Carbon $end, string $expected) + public function testGivenStartAndEndDatesWhenCallPreferredRangeFormatThenReturnsTheExpectedFormatSuccessful(Carbon $start, Carbon $end, string $expected): void { $formatPeriod = $this->navigation->preferredRangeFormat($start, $end); self::assertSame($expected, $formatPeriod); diff --git a/tests/unit/Support/NavigationPreferredSqlFormatTest.php b/tests/unit/Support/NavigationPreferredSqlFormatTest.php index 3b2ac677e7..6e1d1eac32 100644 --- a/tests/unit/Support/NavigationPreferredSqlFormatTest.php +++ b/tests/unit/Support/NavigationPreferredSqlFormatTest.php @@ -68,7 +68,7 @@ final class NavigationPreferredSqlFormatTest extends TestCase /** * @dataProvider provideDates */ - public function testGivenStartAndEndDatesWhenCallPreferredSqlFormatThenReturnsTheExpectedFormatSuccessful(Carbon $start, Carbon $end, string $expected) + public function testGivenStartAndEndDatesWhenCallPreferredSqlFormatThenReturnsTheExpectedFormatSuccessful(Carbon $start, Carbon $end, string $expected): void { $formatPeriod = $this->navigation->preferredSqlFormat($start, $end); self::assertSame($expected, $formatPeriod); diff --git a/tests/unit/Support/NavigationStartOfPeriodTest.php b/tests/unit/Support/NavigationStartOfPeriodTest.php index 9bc3bf3e85..f2e2528742 100644 --- a/tests/unit/Support/NavigationStartOfPeriodTest.php +++ b/tests/unit/Support/NavigationStartOfPeriodTest.php @@ -90,7 +90,7 @@ final class NavigationStartOfPeriodTest extends TestCase /** * @dataProvider provideDates */ - public function testGivenADateAndFrequencyWhenCalculateTheDateThenReturnsTheExpectedDateSuccessful(string $frequency, Carbon $from, Carbon $expected) + public function testGivenADateAndFrequencyWhenCalculateTheDateThenReturnsTheExpectedDateSuccessful(string $frequency, Carbon $from, Carbon $expected): void { $period = $this->navigation->startOfPeriod($from, $frequency); self::assertSame($expected->toDateString(), $period->toDateString()); @@ -99,7 +99,7 @@ final class NavigationStartOfPeriodTest extends TestCase /** * @dataProvider provideUnknownFrequencies */ - public function testGivenADateAndUnknownFrequencyWhenCalculateTheDateThenReturnsTheSameDateSuccessful(string $frequency, Carbon $from, Carbon $expected) + public function testGivenADateAndUnknownFrequencyWhenCalculateTheDateThenReturnsTheSameDateSuccessful(string $frequency, Carbon $from, Carbon $expected): void { Log::shouldReceive('error') ->with(sprintf('Cannot do startOfPeriod for $repeat_freq "%s"', $frequency))