From 4e618a4d0bb2297f9031bbf378c1f1cd2a38ac3e Mon Sep 17 00:00:00 2001 From: James Cole Date: Mon, 31 Oct 2022 05:40:59 +0100 Subject: [PATCH] Remove phpstan comments. --- .../Correction/DeleteEmptyJournals.php | 4 +-- .../Correction/DeleteOrphanedTransactions.php | 2 +- .../Commands/Correction/DeleteZeroAmount.php | 2 +- .../Commands/Upgrade/BackToJournals.php | 7 ++-- .../Commands/Upgrade/MigrateJournalNotes.php | 2 +- app/Factory/AccountMetaFactory.php | 2 +- app/Factory/TransactionJournalFactory.php | 2 +- app/Factory/TransactionJournalMetaFactory.php | 4 +-- .../Report/Account/MonthReportGenerator.php | 2 +- .../Report/Audit/MonthReportGenerator.php | 2 +- .../Report/Budget/MonthReportGenerator.php | 2 +- .../Report/Category/MonthReportGenerator.php | 2 +- .../Report/Standard/MonthReportGenerator.php | 2 +- .../Standard/MultiYearReportGenerator.php | 2 +- .../Report/Standard/YearReportGenerator.php | 2 +- .../Report/Tag/MonthReportGenerator.php | 2 +- app/Handlers/Events/UserEventHandler.php | 6 ++-- app/Helpers/Collector/GroupCollector.php | 2 +- .../Controllers/Chart/CategoryController.php | 2 +- app/Http/Controllers/DebugController.php | 4 +-- .../Controllers/Json/FrontpageController.php | 2 +- .../Controllers/Json/ReconcileController.php | 4 +-- app/Http/Controllers/Json/RuleController.php | 4 +-- .../Controllers/Report/AccountController.php | 2 +- .../Controllers/Report/BalanceController.php | 2 +- .../Controllers/Report/BillController.php | 2 +- .../Controllers/Report/BudgetController.php | 6 ++-- .../Controllers/Report/CategoryController.php | 14 ++++---- .../Controllers/Report/DoubleController.php | 8 ++--- .../Report/OperationsController.php | 6 ++-- app/Http/Controllers/Report/TagController.php | 8 ++--- app/Http/Controllers/Rule/EditController.php | 2 +- .../Controllers/Rule/SelectController.php | 4 +-- app/Http/Controllers/SearchController.php | 2 +- .../Controllers/System/InstallController.php | 4 +-- app/Http/Middleware/Authenticate.php | 2 +- app/Jobs/MailError.php | 2 +- app/Providers/AccountServiceProvider.php | 6 ++-- app/Providers/AdminServiceProvider.php | 2 +- app/Providers/AttachmentServiceProvider.php | 2 +- app/Providers/BillServiceProvider.php | 2 +- app/Providers/BudgetServiceProvider.php | 10 +++--- app/Providers/CategoryServiceProvider.php | 6 ++-- app/Providers/CurrencyServiceProvider.php | 2 +- app/Providers/FireflyServiceProvider.php | 6 ++-- app/Providers/JournalServiceProvider.php | 10 +++--- app/Providers/PiggyBankServiceProvider.php | 2 +- app/Providers/RecurringServiceProvider.php | 2 +- app/Providers/RuleGroupServiceProvider.php | 2 +- app/Providers/RuleServiceProvider.php | 2 +- app/Providers/SearchServiceProvider.php | 2 +- app/Providers/TagServiceProvider.php | 4 +-- .../Attachment/AttachmentRepository.php | 4 +-- .../Budget/AvailableBudgetRepository.php | 2 +- .../Budget/BudgetLimitRepository.php | 4 +-- app/Repositories/Budget/BudgetRepository.php | 4 +-- .../Journal/JournalCLIRepository.php | 6 ++-- .../LinkType/LinkTypeRepository.php | 2 +- .../PiggyBank/ModifiesPiggyBanks.php | 2 +- .../TransactionGroupRepository.php | 2 +- .../Destroy/AccountDestroyService.php | 4 +-- .../Internal/Destroy/BillDestroyService.php | 2 +- .../Internal/Destroy/BudgetDestroyService.php | 2 +- .../Destroy/CategoryDestroyService.php | 2 +- .../Destroy/CurrencyDestroyService.php | 2 +- .../Destroy/JournalDestroyService.php | 2 +- .../Destroy/RecurrenceDestroyService.php | 6 ++-- .../TransactionGroupDestroyService.php | 2 +- .../Internal/Support/AccountServiceTrait.php | 2 +- .../Internal/Support/BillServiceTrait.php | 2 +- .../Internal/Support/JournalServiceTrait.php | 2 +- .../Support/RecurringTransactionTrait.php | 4 +-- .../Internal/Update/CategoryUpdateService.php | 2 +- .../Internal/Update/JournalUpdateService.php | 2 +- .../Update/RecurrenceUpdateService.php | 2 +- app/Services/Password/PwndVerifierV2.php | 2 +- .../Authentication/RemoteUserGuard.php | 3 +- app/Support/Binder/Date.php | 2 +- app/Support/ExpandedForm.php | 34 +++++++++---------- app/Support/FireflyConfig.php | 6 ++-- app/Support/Form/AccountForm.php | 2 +- app/Support/Form/CurrencyForm.php | 4 +-- app/Support/Form/FormSupport.php | 6 ++-- .../Http/Controllers/ModelInformation.php | 6 ++-- .../Http/Controllers/RenderPartialViews.php | 24 ++++++------- .../Http/Controllers/RuleManagement.php | 6 ++-- app/Support/Search/OperatorQuerySearch.php | 2 +- app/Support/Steam.php | 6 ++-- 88 files changed, 177 insertions(+), 179 deletions(-) diff --git a/app/Console/Commands/Correction/DeleteEmptyJournals.php b/app/Console/Commands/Correction/DeleteEmptyJournals.php index 45edc7ab93..1c625a7064 100644 --- a/app/Console/Commands/Correction/DeleteEmptyJournals.php +++ b/app/Console/Commands/Correction/DeleteEmptyJournals.php @@ -77,7 +77,7 @@ class DeleteEmptyJournals extends Command // uneven number, delete journal and transactions: try { TransactionJournal::find((int)$row->transaction_journal_id)->delete(); - } catch (Exception $e) { // @phpstan-ignore-line + } catch (Exception $e) { Log::info(sprintf('Could not delete journal: %s', $e->getMessage())); } @@ -104,7 +104,7 @@ class DeleteEmptyJournals extends Command foreach ($set as $entry) { try { TransactionJournal::find($entry->id)->delete(); - } catch (Exception $e) { // @phpstan-ignore-line + } catch (Exception $e) { Log::info(sprintf('Could not delete entry: %s', $e->getMessage())); } diff --git a/app/Console/Commands/Correction/DeleteOrphanedTransactions.php b/app/Console/Commands/Correction/DeleteOrphanedTransactions.php index f3fb23a901..61b933296c 100644 --- a/app/Console/Commands/Correction/DeleteOrphanedTransactions.php +++ b/app/Console/Commands/Correction/DeleteOrphanedTransactions.php @@ -147,7 +147,7 @@ class DeleteOrphanedTransactions extends Command if ($journal) { try { $journal->delete(); - } catch (Exception $e) { // @phpstan-ignore-line + } catch (Exception $e) { Log::info(sprintf('Could not delete journal %s', $e->getMessage())); } } diff --git a/app/Console/Commands/Correction/DeleteZeroAmount.php b/app/Console/Commands/Correction/DeleteZeroAmount.php index 67fe97aef3..3cbb79fb35 100644 --- a/app/Console/Commands/Correction/DeleteZeroAmount.php +++ b/app/Console/Commands/Correction/DeleteZeroAmount.php @@ -62,7 +62,7 @@ class DeleteZeroAmount extends Command $this->info(sprintf('Deleted transaction journal #%d because the amount is zero (0.00).', $journal->id)); try { $journal->delete(); - } catch (Exception $e) { // @phpstan-ignore-line + } catch (Exception $e) { $this->line($e->getMessage()); } diff --git a/app/Console/Commands/Upgrade/BackToJournals.php b/app/Console/Commands/Upgrade/BackToJournals.php index 6150410571..cc262bc2a2 100644 --- a/app/Console/Commands/Upgrade/BackToJournals.php +++ b/app/Console/Commands/Upgrade/BackToJournals.php @@ -148,8 +148,7 @@ class BackToJournals extends Command */ private function getIdsForBudgets(): array { - $transactions = DB::table('budget_transaction')->distinct()->get(['transaction_id'])-> - toCollection()->pluck('transaction_id')->toArray(); // @phpstan-ignore-line + $transactions = DB::table('budget_transaction')->distinct()->get(['transaction_id'])->pluck('transaction_id')->toArray(); $array = []; $chunks = array_chunk($transactions, 500); @@ -223,12 +222,12 @@ class BackToJournals extends Command */ private function getIdsForCategories(): array { - $transactions = DB::table('category_transaction')->distinct()->get(['transaction_id'])->pluck('transaction_id')->toArray(); // @phpstan-ignore-line + $transactions = DB::table('category_transaction')->distinct()->get(['transaction_id'])->pluck('transaction_id')->toArray(); $array = []; $chunks = array_chunk($transactions, 500); foreach ($chunks as $chunk) { - $set = DB::table('transactions') // @phpstan-ignore-line + $set = DB::table('transactions') ->whereIn('transactions.id', $chunk) ->get(['transaction_journal_id'])->pluck('transaction_journal_id')->toArray(); $array = array_merge($array, $set); diff --git a/app/Console/Commands/Upgrade/MigrateJournalNotes.php b/app/Console/Commands/Upgrade/MigrateJournalNotes.php index 7bb3b8929d..20b10a1709 100644 --- a/app/Console/Commands/Upgrade/MigrateJournalNotes.php +++ b/app/Console/Commands/Upgrade/MigrateJournalNotes.php @@ -83,7 +83,7 @@ class MigrateJournalNotes extends Command Log::debug(sprintf('Migrated meta note #%d to Note #%d', $meta->id, $note->id)); try { $meta->delete(); - } catch (Exception $e) { // @phpstan-ignore-line + } catch (Exception $e) { Log::error(sprintf('Could not delete old meta entry #%d: %s', $meta->id, $e->getMessage())); } diff --git a/app/Factory/AccountMetaFactory.php b/app/Factory/AccountMetaFactory.php index 5e92d1ae10..fc1f73468c 100644 --- a/app/Factory/AccountMetaFactory.php +++ b/app/Factory/AccountMetaFactory.php @@ -64,7 +64,7 @@ class AccountMetaFactory if ('' === $value && null !== $entry) { try { $entry->delete(); - } catch (Exception $e) { // @phpstan-ignore-line + } catch (Exception $e) { Log::debug(sprintf('Could not delete entry: %s', $e->getMessage())); } diff --git a/app/Factory/TransactionJournalFactory.php b/app/Factory/TransactionJournalFactory.php index d78ee5fc62..22b8a21d71 100644 --- a/app/Factory/TransactionJournalFactory.php +++ b/app/Factory/TransactionJournalFactory.php @@ -510,7 +510,7 @@ class TransactionJournalFactory { try { $transaction->delete(); - } catch (Exception $e) { // @phpstan-ignore-line + } catch (Exception $e) { Log::error($e->getMessage()); Log::error($e->getTraceAsString()); Log::error('Could not delete negative transaction.'); diff --git a/app/Factory/TransactionJournalMetaFactory.php b/app/Factory/TransactionJournalMetaFactory.php index 4bcb5b36ca..136a9c29b4 100644 --- a/app/Factory/TransactionJournalMetaFactory.php +++ b/app/Factory/TransactionJournalMetaFactory.php @@ -48,7 +48,7 @@ class TransactionJournalMetaFactory //Log::debug('Value is empty, delete meta value.'); try { $entry->delete(); - } catch (Exception $e) { // @phpstan-ignore-line + } catch (Exception $e) { Log::error(sprintf('Could not delete transaction journal meta: %s', $e->getMessage())); } @@ -66,7 +66,7 @@ class TransactionJournalMetaFactory Log::debug('Will not store empty strings, delete meta value'); try { $entry->delete(); - } catch (Exception $e) { // @phpstan-ignore-line + } catch (Exception $e) { Log::error(sprintf('Could not delete transaction journal meta: %s', $e->getMessage())); } } diff --git a/app/Generator/Report/Account/MonthReportGenerator.php b/app/Generator/Report/Account/MonthReportGenerator.php index 937548f453..7eea275513 100644 --- a/app/Generator/Report/Account/MonthReportGenerator.php +++ b/app/Generator/Report/Account/MonthReportGenerator.php @@ -56,7 +56,7 @@ class MonthReportGenerator implements ReportGeneratorInterface ->with('start', $this->start)->with('end', $this->end) ->with('doubles', $this->expense) ->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::error(sprintf('Cannot render reports.double.report: %s', $e->getMessage())); $result = sprintf('Could not render report view: %s', $e->getMessage()); } diff --git a/app/Generator/Report/Audit/MonthReportGenerator.php b/app/Generator/Report/Audit/MonthReportGenerator.php index 4ad3090221..50af19975b 100644 --- a/app/Generator/Report/Audit/MonthReportGenerator.php +++ b/app/Generator/Report/Audit/MonthReportGenerator.php @@ -83,7 +83,7 @@ class MonthReportGenerator implements ReportGeneratorInterface $result = view('reports.audit.report', compact('reportType', 'accountIds', 'auditData', 'hideable', 'defaultShow')) ->with('start', $this->start)->with('end', $this->end)->with('accounts', $this->accounts) ->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::error(sprintf('Cannot render reports.audit.report: %s', $e->getMessage())); Log::error($e->getTraceAsString()); $result = sprintf('Could not render report view: %s', $e->getMessage()); diff --git a/app/Generator/Report/Budget/MonthReportGenerator.php b/app/Generator/Report/Budget/MonthReportGenerator.php index 07d8f96dbf..ff637ad5b0 100644 --- a/app/Generator/Report/Budget/MonthReportGenerator.php +++ b/app/Generator/Report/Budget/MonthReportGenerator.php @@ -71,7 +71,7 @@ class MonthReportGenerator implements ReportGeneratorInterface ->with('budgets', $this->budgets) ->with('accounts', $this->accounts) ->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::error(sprintf('Cannot render reports.account.report: %s', $e->getMessage())); $result = sprintf('Could not render report view: %s', $e->getMessage()); } diff --git a/app/Generator/Report/Category/MonthReportGenerator.php b/app/Generator/Report/Category/MonthReportGenerator.php index fb0c5a3cba..afede2a346 100644 --- a/app/Generator/Report/Category/MonthReportGenerator.php +++ b/app/Generator/Report/Category/MonthReportGenerator.php @@ -80,7 +80,7 @@ class MonthReportGenerator implements ReportGeneratorInterface ->with('categories', $this->categories) ->with('accounts', $this->accounts) ->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::error(sprintf('Cannot render reports.category.month: %s', $e->getMessage())); $result = sprintf('Could not render report view: %s', $e->getMessage()); } diff --git a/app/Generator/Report/Standard/MonthReportGenerator.php b/app/Generator/Report/Standard/MonthReportGenerator.php index 8d787e5c8d..4dfd585758 100644 --- a/app/Generator/Report/Standard/MonthReportGenerator.php +++ b/app/Generator/Report/Standard/MonthReportGenerator.php @@ -54,7 +54,7 @@ class MonthReportGenerator implements ReportGeneratorInterface try { return view('reports.default.month', compact('accountIds', 'reportType'))->with('start', $this->start)->with('end', $this->end)->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::error(sprintf('Cannot render reports.default.month: %s', $e->getMessage())); $result = 'Could not render report view.'; } diff --git a/app/Generator/Report/Standard/MultiYearReportGenerator.php b/app/Generator/Report/Standard/MultiYearReportGenerator.php index 09794671e0..6211528ce5 100644 --- a/app/Generator/Report/Standard/MultiYearReportGenerator.php +++ b/app/Generator/Report/Standard/MultiYearReportGenerator.php @@ -58,7 +58,7 @@ class MultiYearReportGenerator implements ReportGeneratorInterface 'reports.default.multi-year', compact('accountIds', 'reportType') )->with('start', $this->start)->with('end', $this->end)->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::error(sprintf('Cannot render reports.default.multi-year: %s', $e->getMessage())); $result = sprintf('Could not render report view: %s', $e->getMessage()); } diff --git a/app/Generator/Report/Standard/YearReportGenerator.php b/app/Generator/Report/Standard/YearReportGenerator.php index 95f52190db..03cd0628b4 100644 --- a/app/Generator/Report/Standard/YearReportGenerator.php +++ b/app/Generator/Report/Standard/YearReportGenerator.php @@ -58,7 +58,7 @@ class YearReportGenerator implements ReportGeneratorInterface 'reports.default.year', compact('accountIds', 'reportType') )->with('start', $this->start)->with('end', $this->end)->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::error(sprintf('Cannot render reports.account.report: %s', $e->getMessage())); $result = 'Could not render report view.'; } diff --git a/app/Generator/Report/Tag/MonthReportGenerator.php b/app/Generator/Report/Tag/MonthReportGenerator.php index c7cfa7b563..ae729661bc 100644 --- a/app/Generator/Report/Tag/MonthReportGenerator.php +++ b/app/Generator/Report/Tag/MonthReportGenerator.php @@ -78,7 +78,7 @@ class MonthReportGenerator implements ReportGeneratorInterface 'reports.tag.month', compact('accountIds', 'reportType', 'tagIds') )->with('start', $this->start)->with('end', $this->end)->with('tags', $this->tags)->with('accounts', $this->accounts)->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::error(sprintf('Cannot render reports.tag.month: %s', $e->getMessage())); $result = sprintf('Could not render report view: %s', $e->getMessage()); } diff --git a/app/Handlers/Events/UserEventHandler.php b/app/Handlers/Events/UserEventHandler.php index 6d4cb7c51d..2d05c1dfa0 100644 --- a/app/Handlers/Events/UserEventHandler.php +++ b/app/Handlers/Events/UserEventHandler.php @@ -89,7 +89,7 @@ class UserEventHandler $url = route('invite', [$event->invitee->invite_code]); try { Mail::to($invitee)->send(new InvitationMail($invitee, $admin, $url)); - } catch (Exception $e) { // @phpstan-ignore-line + } catch (Exception $e) { Log::error($e->getMessage()); } } @@ -245,7 +245,7 @@ class UserEventHandler try { Mail::to($newEmail)->send(new ConfirmEmailChangeMail($newEmail, $oldEmail, $url)); - } catch (Exception $e) { // @phpstan-ignore-line + } catch (Exception $e) { Log::error($e->getMessage()); } } @@ -268,7 +268,7 @@ class UserEventHandler $url = route('profile.undo-email-change', [$token->data, $hashed]); try { Mail::to($oldEmail)->send(new UndoEmailChangeMail($newEmail, $oldEmail, $url)); - } catch (Exception $e) { // @phpstan-ignore-line + } catch (Exception $e) { Log::error($e->getMessage()); } } diff --git a/app/Helpers/Collector/GroupCollector.php b/app/Helpers/Collector/GroupCollector.php index 8b655012b6..fd69ea5745 100644 --- a/app/Helpers/Collector/GroupCollector.php +++ b/app/Helpers/Collector/GroupCollector.php @@ -572,7 +572,7 @@ class GroupCollector implements GroupCollectorInterface $result['date']->setTimezone(config('app.timezone')); $result['created_at']->setTimezone(config('app.timezone')); $result['updated_at']->setTimezone(config('app.timezone')); - } catch (Exception $e) { // @phpstan-ignore-line + } catch (Exception $e) { Log::error($e->getMessage()); } diff --git a/app/Http/Controllers/Chart/CategoryController.php b/app/Http/Controllers/Chart/CategoryController.php index 7d34da8e3c..e00bcb4f77 100644 --- a/app/Http/Controllers/Chart/CategoryController.php +++ b/app/Http/Controllers/Chart/CategoryController.php @@ -109,7 +109,7 @@ class CategoryController extends Controller $carbon = null; try { $carbon = today(config('app.timezone')); - } catch (Exception $e) { // @phpstan-ignore-line + } catch (Exception $e) { // @ignoreException } diff --git a/app/Http/Controllers/DebugController.php b/app/Http/Controllers/DebugController.php index cb01d5040a..9a8b96a06b 100644 --- a/app/Http/Controllers/DebugController.php +++ b/app/Http/Controllers/DebugController.php @@ -98,7 +98,7 @@ class DebugController extends Controller Log::debug('Call twig:clean...'); try { Artisan::call('twig:clean'); - } catch (Exception $e) { // @phpstan-ignore-line + } catch (Exception $e) { // @ignoreException } @@ -188,7 +188,7 @@ class DebugController extends Controller if (null !== $logFile) { try { $logContent = file_get_contents($logFile); - } catch (Exception $e) { // @phpstan-ignore-line + } catch (Exception $e) { // @ignoreException } } diff --git a/app/Http/Controllers/Json/FrontpageController.php b/app/Http/Controllers/Json/FrontpageController.php index 5ea174dd99..3924db14fb 100644 --- a/app/Http/Controllers/Json/FrontpageController.php +++ b/app/Http/Controllers/Json/FrontpageController.php @@ -70,7 +70,7 @@ class FrontpageController extends Controller if (!empty($info)) { try { $html = view('json.piggy-banks', compact('info'))->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::error(sprintf('Cannot render json.piggy-banks: %s', $e->getMessage())); $html = 'Could not render view.'; } diff --git a/app/Http/Controllers/Json/ReconcileController.php b/app/Http/Controllers/Json/ReconcileController.php index 7c8426f9ca..0287d1886d 100644 --- a/app/Http/Controllers/Json/ReconcileController.php +++ b/app/Http/Controllers/Json/ReconcileController.php @@ -161,7 +161,7 @@ class ReconcileController extends Controller 'selectedIds' ) )->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::debug(sprintf('View error: %s', $e->getMessage())); $view = sprintf('Could not render accounts.reconcile.overview: %s', $e->getMessage()); } @@ -261,7 +261,7 @@ class ReconcileController extends Controller 'accounts.reconcile.transactions', compact('account', 'journals', 'currency', 'start', 'end', 'selectionStart', 'selectionEnd') )->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::debug(sprintf('Could not render: %s', $e->getMessage())); $html = sprintf('Could not render accounts.reconcile.transactions: %s', $e->getMessage()); } diff --git a/app/Http/Controllers/Json/RuleController.php b/app/Http/Controllers/Json/RuleController.php index c41d856ec4..956f145dbf 100644 --- a/app/Http/Controllers/Json/RuleController.php +++ b/app/Http/Controllers/Json/RuleController.php @@ -51,7 +51,7 @@ class RuleController extends Controller } try { $view = view('rules.partials.action', compact('actions', 'count'))->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::error(sprintf('Cannot render rules.partials.action: %s', $e->getMessage())); $view = 'Could not render view.'; } @@ -80,7 +80,7 @@ class RuleController extends Controller try { $view = view('rules.partials.trigger', compact('triggers', 'count'))->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::error(sprintf('Cannot render rules.partials.trigger: %s', $e->getMessage())); $view = 'Could not render view.'; } diff --git a/app/Http/Controllers/Report/AccountController.php b/app/Http/Controllers/Report/AccountController.php index 03d73c7625..32034dca8b 100644 --- a/app/Http/Controllers/Report/AccountController.php +++ b/app/Http/Controllers/Report/AccountController.php @@ -62,7 +62,7 @@ class AccountController extends Controller $accountReport = $accountTasker->getAccountReport($accounts, $start, $end); try { $result = view('reports.partials.accounts', compact('accountReport'))->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::debug(sprintf('Could not render reports.partials.accounts: %s', $e->getMessage())); $result = 'Could not render view.'; } diff --git a/app/Http/Controllers/Report/BalanceController.php b/app/Http/Controllers/Report/BalanceController.php index 504a5715c1..7ec7d90578 100644 --- a/app/Http/Controllers/Report/BalanceController.php +++ b/app/Http/Controllers/Report/BalanceController.php @@ -137,7 +137,7 @@ class BalanceController extends Controller } try { $result = view('reports.partials.balance', compact('report'))->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::debug(sprintf('Could not render reports.partials.balance: %s', $e->getMessage())); $result = 'Could not render view.'; } diff --git a/app/Http/Controllers/Report/BillController.php b/app/Http/Controllers/Report/BillController.php index f9194c0b4d..f1aca86cd0 100644 --- a/app/Http/Controllers/Report/BillController.php +++ b/app/Http/Controllers/Report/BillController.php @@ -59,7 +59,7 @@ class BillController extends Controller $report = $helper->getBillReport($accounts, $start, $end); try { $result = view('reports.partials.bills', compact('report'))->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::debug(sprintf('Could not render reports.partials.budgets: %s', $e->getMessage())); $result = 'Could not render view.'; } diff --git a/app/Http/Controllers/Report/BudgetController.php b/app/Http/Controllers/Report/BudgetController.php index 0a09509915..0b7858319f 100644 --- a/app/Http/Controllers/Report/BudgetController.php +++ b/app/Http/Controllers/Report/BudgetController.php @@ -198,7 +198,7 @@ class BudgetController extends Controller try { $result = view('reports.budget.partials.avg-expenses', compact('result'))->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::debug(sprintf('Could not render reports.partials.budget-period: %s', $e->getMessage())); $result = sprintf('Could not render view: %s', $e->getMessage()); } @@ -353,7 +353,7 @@ class BudgetController extends Controller } try { $result = view('reports.partials.budget-period', compact('report', 'periods'))->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::debug(sprintf('Could not render reports.partials.budget-period: %s', $e->getMessage())); $result = 'Could not render view.'; } @@ -404,7 +404,7 @@ class BudgetController extends Controller try { $result = view('reports.budget.partials.top-expenses', compact('result'))->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::debug(sprintf('Could not render reports.partials.budget-period: %s', $e->getMessage())); $result = sprintf('Could not render view: %s', $e->getMessage()); } diff --git a/app/Http/Controllers/Report/CategoryController.php b/app/Http/Controllers/Report/CategoryController.php index 39a2c1af6d..81f04870a9 100644 --- a/app/Http/Controllers/Report/CategoryController.php +++ b/app/Http/Controllers/Report/CategoryController.php @@ -312,7 +312,7 @@ class CategoryController extends Controller try { $result = view('reports.category.partials.avg-expenses', compact('result'))->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::debug(sprintf('Could not render reports.partials.budget-period: %s', $e->getMessage())); $result = sprintf('Could not render view: %s', $e->getMessage()); } @@ -363,7 +363,7 @@ class CategoryController extends Controller try { $result = view('reports.category.partials.avg-income', compact('result'))->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::debug(sprintf('Could not render reports.partials.budget-period: %s', $e->getMessage())); $result = sprintf('Could not render view: %s', $e->getMessage()); } @@ -547,7 +547,7 @@ class CategoryController extends Controller try { $result = view('reports.partials.category-period', compact('report', 'periods'))->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::error(sprintf('Could not render category::expenses: %s', $e->getMessage())); $result = sprintf('An error prevented Firefly III from rendering: %s. Apologies.', $e->getMessage()); } @@ -626,7 +626,7 @@ class CategoryController extends Controller try { $result = view('reports.partials.category-period', compact('report', 'periods'))->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::error(sprintf('Could not render category::expenses: %s', $e->getMessage())); $result = sprintf('An error prevented Firefly III from rendering: %s. Apologies.', $e->getMessage()); } @@ -670,7 +670,7 @@ class CategoryController extends Controller try { $result = (string) view('reports.partials.categories', compact('report'))->render(); $cache->store($result); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::error(sprintf('Could not render category::expenses: %s', $e->getMessage())); $result = sprintf('An error prevented Firefly III from rendering: %s. Apologies.', $e->getMessage()); } @@ -719,7 +719,7 @@ class CategoryController extends Controller try { $result = view('reports.category.partials.top-expenses', compact('result'))->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::debug(sprintf('Could not render reports.partials.budget-period: %s', $e->getMessage())); $result = sprintf('Could not render view: %s', $e->getMessage()); } @@ -768,7 +768,7 @@ class CategoryController extends Controller try { $result = view('reports.category.partials.top-income', compact('result'))->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::debug(sprintf('Could not render reports.partials.budget-period: %s', $e->getMessage())); $result = sprintf('Could not render view: %s', $e->getMessage()); } diff --git a/app/Http/Controllers/Report/DoubleController.php b/app/Http/Controllers/Report/DoubleController.php index b46ca85d95..f4156e997a 100644 --- a/app/Http/Controllers/Report/DoubleController.php +++ b/app/Http/Controllers/Report/DoubleController.php @@ -111,7 +111,7 @@ class DoubleController extends Controller try { $result = view('reports.double.partials.avg-expenses', compact('result'))->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::debug(sprintf('Could not render reports.partials.budget-period: %s', $e->getMessage())); $result = sprintf('Could not render view: %s', $e->getMessage()); } @@ -162,7 +162,7 @@ class DoubleController extends Controller try { $result = view('reports.double.partials.avg-income', compact('result'))->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::debug(sprintf('Could not render reports.partials.budget-period: %s', $e->getMessage())); $result = sprintf('Could not render view: %s', $e->getMessage()); } @@ -455,7 +455,7 @@ class DoubleController extends Controller try { $result = view('reports.double.partials.top-expenses', compact('result'))->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::debug(sprintf('Could not render reports.partials.budget-period: %s', $e->getMessage())); $result = sprintf('Could not render view: %s', $e->getMessage()); } @@ -504,7 +504,7 @@ class DoubleController extends Controller try { $result = view('reports.double.partials.top-income', compact('result'))->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::debug(sprintf('Could not render reports.partials.budget-period: %s', $e->getMessage())); $result = sprintf('Could not render view: %s', $e->getMessage()); } diff --git a/app/Http/Controllers/Report/OperationsController.php b/app/Http/Controllers/Report/OperationsController.php index 189338dbd4..78043a110c 100644 --- a/app/Http/Controllers/Report/OperationsController.php +++ b/app/Http/Controllers/Report/OperationsController.php @@ -82,7 +82,7 @@ class OperationsController extends Controller $type = 'expense-entry'; try { $result = view('reports.partials.income-expenses', compact('report', 'type'))->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::debug(sprintf('Could not render reports.partials.income-expense: %s', $e->getMessage())); $result = 'Could not render view.'; } @@ -116,7 +116,7 @@ class OperationsController extends Controller $type = 'income-entry'; try { $result = view('reports.partials.income-expenses', compact('report', 'type'))->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::debug(sprintf('Could not render reports.partials.income-expenses: %s', $e->getMessage())); $result = 'Could not render view.'; } @@ -170,7 +170,7 @@ class OperationsController extends Controller try { $result = view('reports.partials.operations', compact('sums'))->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::debug(sprintf('Could not render reports.partials.operations: %s', $e->getMessage())); $result = 'Could not render view.'; } diff --git a/app/Http/Controllers/Report/TagController.php b/app/Http/Controllers/Report/TagController.php index 2e8ffdf93a..aebe580a28 100644 --- a/app/Http/Controllers/Report/TagController.php +++ b/app/Http/Controllers/Report/TagController.php @@ -305,7 +305,7 @@ class TagController extends Controller try { $result = view('reports.tag.partials.avg-expenses', compact('result'))->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::debug(sprintf('Could not render reports.partials.budget-period: %s', $e->getMessage())); $result = sprintf('Could not render view: %s', $e->getMessage()); } @@ -356,7 +356,7 @@ class TagController extends Controller try { $result = view('reports.tag.partials.avg-income', compact('result'))->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::debug(sprintf('Could not render reports.partials.budget-period: %s', $e->getMessage())); $result = sprintf('Could not render view: %s', $e->getMessage()); } @@ -513,7 +513,7 @@ class TagController extends Controller try { $result = view('reports.tag.partials.top-expenses', compact('result'))->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::debug(sprintf('Could not render reports.partials.budget-period: %s', $e->getMessage())); $result = sprintf('Could not render view: %s', $e->getMessage()); } @@ -562,7 +562,7 @@ class TagController extends Controller try { $result = view('reports.tag.partials.top-income', compact('result'))->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::debug(sprintf('Could not render reports.partials.budget-period: %s', $e->getMessage())); $result = sprintf('Could not render view: %s', $e->getMessage()); } diff --git a/app/Http/Controllers/Rule/EditController.php b/app/Http/Controllers/Rule/EditController.php index 995a2294e4..326a96c260 100644 --- a/app/Http/Controllers/Rule/EditController.php +++ b/app/Http/Controllers/Rule/EditController.php @@ -168,7 +168,7 @@ class EditController extends Controller 'triggers' => $triggers, ] )->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::debug(sprintf('Throwable was thrown in getPreviousTriggers(): %s', $e->getMessage())); Log::error($e->getTraceAsString()); } diff --git a/app/Http/Controllers/Rule/SelectController.php b/app/Http/Controllers/Rule/SelectController.php index a3ff6c8e47..ac5150dc78 100644 --- a/app/Http/Controllers/Rule/SelectController.php +++ b/app/Http/Controllers/Rule/SelectController.php @@ -176,7 +176,7 @@ class SelectController extends Controller $view = 'ERROR, see logs.'; try { $view = view('list.journals-array-tiny', ['groups' => $collection])->render(); - } catch (Throwable $exception) { // @phpstan-ignore-line + } catch (Throwable $exception) { Log::error(sprintf('Could not render view in testTriggers(): %s', $exception->getMessage())); Log::error($exception->getTraceAsString()); $view = sprintf('Could not render list.journals-tiny: %s', $exception->getMessage()); @@ -218,7 +218,7 @@ class SelectController extends Controller $view = 'ERROR, see logs.'; try { $view = view('list.journals-array-tiny', ['groups' => $collection])->render(); - } catch (Throwable $exception) { // @phpstan-ignore-line + } catch (Throwable $exception) { Log::error(sprintf('Could not render view in testTriggersByRule(): %s', $exception->getMessage())); Log::error($exception->getTraceAsString()); } diff --git a/app/Http/Controllers/SearchController.php b/app/Http/Controllers/SearchController.php index 87da6d22dd..18619ac3a7 100644 --- a/app/Http/Controllers/SearchController.php +++ b/app/Http/Controllers/SearchController.php @@ -119,7 +119,7 @@ class SearchController extends Controller try { $html = view('search.search', compact('groups', 'hasPages', 'searchTime'))->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::error(sprintf('Cannot render search.search: %s', $e->getMessage())); $html = 'Could not render view.'; } diff --git a/app/Http/Controllers/System/InstallController.php b/app/Http/Controllers/System/InstallController.php index 94d0257c84..fa50e8d056 100644 --- a/app/Http/Controllers/System/InstallController.php +++ b/app/Http/Controllers/System/InstallController.php @@ -198,7 +198,7 @@ class InstallController extends Controller Artisan::call($command, $args); Log::debug(Artisan::output()); } - } catch (Exception $e) { // @phpstan-ignore-line + } catch (Exception $e) { Log::error($e->getMessage()); Log::error($e->getTraceAsString()); if (strpos($e->getMessage(), 'open_basedir restriction in effect')) { @@ -211,7 +211,7 @@ class InstallController extends Controller return false; } // clear cache as well. - Cache::clear(); // @phpstan-ignore-line + Cache::clear(); Preferences::mark(); return true; diff --git a/app/Http/Middleware/Authenticate.php b/app/Http/Middleware/Authenticate.php index 6c6402b496..7eaa66ff99 100644 --- a/app/Http/Middleware/Authenticate.php +++ b/app/Http/Middleware/Authenticate.php @@ -122,7 +122,7 @@ class Authenticate foreach ($guards as $guard) { if ($this->auth->guard($guard)->check()) { - return $this->auth->shouldUse($guard); // @phpstan-ignore-line + return $this->auth->shouldUse($guard); } } diff --git a/app/Jobs/MailError.php b/app/Jobs/MailError.php index b431a4a2cd..8398de7080 100644 --- a/app/Jobs/MailError.php +++ b/app/Jobs/MailError.php @@ -87,7 +87,7 @@ class MailError extends Job implements ShouldQueue } } ); - } catch (Exception $e) { // @phpstan-ignore-line + } catch (Exception $e) { Log::error('Exception when mailing: ' . $e->getMessage()); } } diff --git a/app/Providers/AccountServiceProvider.php b/app/Providers/AccountServiceProvider.php index c6fae4af38..71e4c22eb2 100644 --- a/app/Providers/AccountServiceProvider.php +++ b/app/Providers/AccountServiceProvider.php @@ -64,7 +64,7 @@ class AccountServiceProvider extends ServiceProvider /** @var AccountRepositoryInterface $repository */ $repository = app(AccountRepository::class); - if ($app->auth->check()) { // @phpstan-ignore-line + if ($app->auth->check()) { $repository->setUser(auth()->user()); } @@ -78,7 +78,7 @@ class AccountServiceProvider extends ServiceProvider /** @var OperationsRepository $repository */ $repository = app(OperationsRepository::class); - if ($app->auth->check()) { // @phpstan-ignore-line + if ($app->auth->check()) { $repository->setUser(auth()->user()); } @@ -98,7 +98,7 @@ class AccountServiceProvider extends ServiceProvider /** @var AccountTaskerInterface $tasker */ $tasker = app(AccountTasker::class); - if ($app->auth->check()) { // @phpstan-ignore-line + if ($app->auth->check()) { $tasker->setUser(auth()->user()); } diff --git a/app/Providers/AdminServiceProvider.php b/app/Providers/AdminServiceProvider.php index 41b2c87f8e..47a7d3d763 100644 --- a/app/Providers/AdminServiceProvider.php +++ b/app/Providers/AdminServiceProvider.php @@ -58,7 +58,7 @@ class AdminServiceProvider extends ServiceProvider function (Application $app) { /** @var LinkTypeRepository $repository */ $repository = app(LinkTypeRepository::class); - if ($app->auth->check()) { // @phpstan-ignore-line + if ($app->auth->check()) { $repository->setUser(auth()->user()); } diff --git a/app/Providers/AttachmentServiceProvider.php b/app/Providers/AttachmentServiceProvider.php index adf299f740..68b1b2fc32 100644 --- a/app/Providers/AttachmentServiceProvider.php +++ b/app/Providers/AttachmentServiceProvider.php @@ -50,7 +50,7 @@ class AttachmentServiceProvider extends ServiceProvider function (Application $app) { /** @var AttachmentRepositoryInterface $repository */ $repository = app(AttachmentRepository::class); - if ($app->auth->check()) { // @phpstan-ignore-line + if ($app->auth->check()) { $repository->setUser(auth()->user()); } diff --git a/app/Providers/BillServiceProvider.php b/app/Providers/BillServiceProvider.php index ecaefa8edc..f366036704 100644 --- a/app/Providers/BillServiceProvider.php +++ b/app/Providers/BillServiceProvider.php @@ -51,7 +51,7 @@ class BillServiceProvider extends ServiceProvider /** @var BillRepositoryInterface $repository */ $repository = app(BillRepository::class); - if ($app->auth->check()) { // @phpstan-ignore-line + if ($app->auth->check()) { $repository->setUser(auth()->user()); } diff --git a/app/Providers/BudgetServiceProvider.php b/app/Providers/BudgetServiceProvider.php index 6ac056d9a0..6ad26488cd 100644 --- a/app/Providers/BudgetServiceProvider.php +++ b/app/Providers/BudgetServiceProvider.php @@ -58,7 +58,7 @@ class BudgetServiceProvider extends ServiceProvider static function (Application $app) { /** @var BudgetRepositoryInterface $repository */ $repository = app(BudgetRepository::class); - if ($app->auth->check()) { // @phpstan-ignore-line + if ($app->auth->check()) { $repository->setUser(auth()->user()); } @@ -72,7 +72,7 @@ class BudgetServiceProvider extends ServiceProvider static function (Application $app) { /** @var AvailableBudgetRepositoryInterface $repository */ $repository = app(AvailableBudgetRepository::class); - if ($app->auth->check()) { // @phpstan-ignore-line + if ($app->auth->check()) { $repository->setUser(auth()->user()); } @@ -86,7 +86,7 @@ class BudgetServiceProvider extends ServiceProvider static function (Application $app) { /** @var BudgetLimitRepositoryInterface $repository */ $repository = app(BudgetLimitRepository::class); - if ($app->auth->check()) { // @phpstan-ignore-line + if ($app->auth->check()) { $repository->setUser(auth()->user()); } @@ -100,7 +100,7 @@ class BudgetServiceProvider extends ServiceProvider static function (Application $app) { /** @var NoBudgetRepositoryInterface $repository */ $repository = app(NoBudgetRepository::class); - if ($app->auth->check()) { // @phpstan-ignore-line + if ($app->auth->check()) { $repository->setUser(auth()->user()); } @@ -114,7 +114,7 @@ class BudgetServiceProvider extends ServiceProvider static function (Application $app) { /** @var OperationsRepositoryInterface $repository */ $repository = app(OperationsRepository::class); - if ($app->auth->check()) { // @phpstan-ignore-line + if ($app->auth->check()) { $repository->setUser(auth()->user()); } diff --git a/app/Providers/CategoryServiceProvider.php b/app/Providers/CategoryServiceProvider.php index 0df0767cd1..cf02dd1e85 100644 --- a/app/Providers/CategoryServiceProvider.php +++ b/app/Providers/CategoryServiceProvider.php @@ -54,7 +54,7 @@ class CategoryServiceProvider extends ServiceProvider static function (Application $app) { /** @var CategoryRepository $repository */ $repository = app(CategoryRepository::class); - if ($app->auth->check()) { // @phpstan-ignore-line + if ($app->auth->check()) { $repository->setUser(auth()->user()); } @@ -67,7 +67,7 @@ class CategoryServiceProvider extends ServiceProvider static function (Application $app) { /** @var OperationsRepository $repository */ $repository = app(OperationsRepository::class); - if ($app->auth->check()) { // @phpstan-ignore-line + if ($app->auth->check()) { $repository->setUser(auth()->user()); } @@ -80,7 +80,7 @@ class CategoryServiceProvider extends ServiceProvider static function (Application $app) { /** @var NoCategoryRepository $repository */ $repository = app(NoCategoryRepository::class); - if ($app->auth->check()) { // @phpstan-ignore-line + if ($app->auth->check()) { $repository->setUser(auth()->user()); } diff --git a/app/Providers/CurrencyServiceProvider.php b/app/Providers/CurrencyServiceProvider.php index ae527f4026..33206e8937 100644 --- a/app/Providers/CurrencyServiceProvider.php +++ b/app/Providers/CurrencyServiceProvider.php @@ -50,7 +50,7 @@ class CurrencyServiceProvider extends ServiceProvider function (Application $app) { /** @var CurrencyRepository $repository */ $repository = app(CurrencyRepository::class); - if ($app->auth->check()) { // @phpstan-ignore-line + if ($app->auth->check()) { $repository->setUser(auth()->user()); } diff --git a/app/Providers/FireflyServiceProvider.php b/app/Providers/FireflyServiceProvider.php index a4c198319a..2bb77cdb2d 100644 --- a/app/Providers/FireflyServiceProvider.php +++ b/app/Providers/FireflyServiceProvider.php @@ -180,7 +180,7 @@ class FireflyServiceProvider extends ServiceProvider static function (Application $app) { /** @var ObjectGroupRepository $repository */ $repository = app(ObjectGroupRepository::class); - if ($app->auth->check()) { // @phpstan-ignore-line + if ($app->auth->check()) { $repository->setUser(auth()->user()); } @@ -193,7 +193,7 @@ class FireflyServiceProvider extends ServiceProvider static function (Application $app) { /** @var WebhookRepository $repository */ $repository = app(WebhookRepository::class); - if ($app->auth->check()) { // @phpstan-ignore-line + if ($app->auth->check()) { $repository->setUser(auth()->user()); } @@ -206,7 +206,7 @@ class FireflyServiceProvider extends ServiceProvider static function (Application $app) { /** @var SearchRuleEngine $engine */ $engine = app(SearchRuleEngine::class); - if ($app->auth->check()) { // @phpstan-ignore-line + if ($app->auth->check()) { $engine->setUser(auth()->user()); } diff --git a/app/Providers/JournalServiceProvider.php b/app/Providers/JournalServiceProvider.php index b2c6e2672f..02bd305047 100644 --- a/app/Providers/JournalServiceProvider.php +++ b/app/Providers/JournalServiceProvider.php @@ -68,7 +68,7 @@ class JournalServiceProvider extends ServiceProvider static function (Application $app) { /** @var JournalRepositoryInterface $repository */ $repository = app(JournalRepository::class); - if ($app->auth->check()) { // @phpstan-ignore-line + if ($app->auth->check()) { $repository->setUser(auth()->user()); } @@ -82,7 +82,7 @@ class JournalServiceProvider extends ServiceProvider static function (Application $app) { /** @var JournalAPIRepositoryInterface $repository */ $repository = app(JournalAPIRepository::class); - if ($app->auth->check()) { // @phpstan-ignore-line + if ($app->auth->check()) { $repository->setUser(auth()->user()); } @@ -96,7 +96,7 @@ class JournalServiceProvider extends ServiceProvider static function (Application $app) { /** @var JournalCLIRepositoryInterface $repository */ $repository = app(JournalCLIRepository::class); - if ($app->auth->check()) { // @phpstan-ignore-line + if ($app->auth->check()) { $repository->setUser(auth()->user()); } @@ -115,7 +115,7 @@ class JournalServiceProvider extends ServiceProvider static function (Application $app) { /** @var TransactionGroupRepositoryInterface $repository */ $repository = app(TransactionGroupRepository::class); - if ($app->auth->check()) { // @phpstan-ignore-line + if ($app->auth->check()) { $repository->setUser(auth()->user()); } @@ -134,7 +134,7 @@ class JournalServiceProvider extends ServiceProvider static function (Application $app) { /** @var GroupCollectorInterface $collector */ $collector = app(GroupCollector::class); - if ($app->auth->check()) { // @phpstan-ignore-line + if ($app->auth->check()) { $collector->setUser(auth()->user()); } diff --git a/app/Providers/PiggyBankServiceProvider.php b/app/Providers/PiggyBankServiceProvider.php index 59501483f7..3c07247255 100644 --- a/app/Providers/PiggyBankServiceProvider.php +++ b/app/Providers/PiggyBankServiceProvider.php @@ -50,7 +50,7 @@ class PiggyBankServiceProvider extends ServiceProvider function (Application $app) { /** @var PiggyBankRepository $repository */ $repository = app(PiggyBankRepository::class); - if ($app->auth->check()) { // @phpstan-ignore-line + if ($app->auth->check()) { $repository->setUser(auth()->user()); } diff --git a/app/Providers/RecurringServiceProvider.php b/app/Providers/RecurringServiceProvider.php index 85d2572c2c..f8cbfd0bee 100644 --- a/app/Providers/RecurringServiceProvider.php +++ b/app/Providers/RecurringServiceProvider.php @@ -51,7 +51,7 @@ class RecurringServiceProvider extends ServiceProvider /** @var RecurringRepositoryInterface $repository */ $repository = app(RecurringRepository::class); - if ($app->auth->check()) { // @phpstan-ignore-line + if ($app->auth->check()) { $repository->setUser(auth()->user()); } diff --git a/app/Providers/RuleGroupServiceProvider.php b/app/Providers/RuleGroupServiceProvider.php index 8c4f96374b..ac18086099 100644 --- a/app/Providers/RuleGroupServiceProvider.php +++ b/app/Providers/RuleGroupServiceProvider.php @@ -50,7 +50,7 @@ class RuleGroupServiceProvider extends ServiceProvider function (Application $app) { /** @var RuleGroupRepository $repository */ $repository = app(RuleGroupRepository::class); - if ($app->auth->check()) { // @phpstan-ignore-line + if ($app->auth->check()) { $repository->setUser(auth()->user()); } diff --git a/app/Providers/RuleServiceProvider.php b/app/Providers/RuleServiceProvider.php index 41bc572893..fd0c76e758 100644 --- a/app/Providers/RuleServiceProvider.php +++ b/app/Providers/RuleServiceProvider.php @@ -50,7 +50,7 @@ class RuleServiceProvider extends ServiceProvider function (Application $app) { /** @var RuleRepository $repository */ $repository = app(RuleRepository::class); - if ($app->auth->check()) { // @phpstan-ignore-line + if ($app->auth->check()) { $repository->setUser(auth()->user()); } diff --git a/app/Providers/SearchServiceProvider.php b/app/Providers/SearchServiceProvider.php index 5e49a10c5a..0136b244ea 100644 --- a/app/Providers/SearchServiceProvider.php +++ b/app/Providers/SearchServiceProvider.php @@ -50,7 +50,7 @@ class SearchServiceProvider extends ServiceProvider function (Application $app) { /** @var OperatorQuerySearch $search */ $search = app(OperatorQuerySearch::class); - if ($app->auth->check()) { // @phpstan-ignore-line + if ($app->auth->check()) { $search->setUser(auth()->user()); } diff --git a/app/Providers/TagServiceProvider.php b/app/Providers/TagServiceProvider.php index 4bcf89b463..fba7c066b1 100644 --- a/app/Providers/TagServiceProvider.php +++ b/app/Providers/TagServiceProvider.php @@ -53,7 +53,7 @@ class TagServiceProvider extends ServiceProvider /** @var TagRepository $repository */ $repository = app(TagRepository::class); - if ($app->auth->check()) { // @phpstan-ignore-line + if ($app->auth->check()) { $repository->setUser(auth()->user()); } @@ -67,7 +67,7 @@ class TagServiceProvider extends ServiceProvider /** @var OperationsRepository $repository */ $repository = app(OperationsRepository::class); - if ($app->auth->check()) { // @phpstan-ignore-line + if ($app->auth->check()) { $repository->setUser(auth()->user()); } diff --git a/app/Repositories/Attachment/AttachmentRepository.php b/app/Repositories/Attachment/AttachmentRepository.php index 74f31ccd65..0dee38ec6f 100644 --- a/app/Repositories/Attachment/AttachmentRepository.php +++ b/app/Repositories/Attachment/AttachmentRepository.php @@ -57,7 +57,7 @@ class AttachmentRepository implements AttachmentRepositoryInterface $path = $helper->getAttachmentLocation($attachment); try { Storage::disk('upload')->delete($path); - } catch (Exception $e) { // @phpstan-ignore-line + } catch (Exception $e) { // @ignoreException } $attachment->delete(); @@ -201,7 +201,7 @@ class AttachmentRepository implements AttachmentRepositoryInterface if (null !== $dbNote) { try { $dbNote->delete(); - } catch (Exception $e) { // @phpstan-ignore-line + } catch (Exception $e) { // @ignoreException } } diff --git a/app/Repositories/Budget/AvailableBudgetRepository.php b/app/Repositories/Budget/AvailableBudgetRepository.php index fd578c920c..9dc4518860 100644 --- a/app/Repositories/Budget/AvailableBudgetRepository.php +++ b/app/Repositories/Budget/AvailableBudgetRepository.php @@ -54,7 +54,7 @@ class AvailableBudgetRepository implements AvailableBudgetRepositoryInterface { try { $availableBudget->delete(); - } catch (Exception $e) { // @phpstan-ignore-line + } catch (Exception $e) { // @ignoreException } } diff --git a/app/Repositories/Budget/BudgetLimitRepository.php b/app/Repositories/Budget/BudgetLimitRepository.php index 3673e42a7d..3d9c23771d 100644 --- a/app/Repositories/Budget/BudgetLimitRepository.php +++ b/app/Repositories/Budget/BudgetLimitRepository.php @@ -126,7 +126,7 @@ class BudgetLimitRepository implements BudgetLimitRepositoryInterface { try { $budgetLimit->delete(); - } catch (Exception $e) { // @phpstan-ignore-line + } catch (Exception $e) { // @ignoreException } } @@ -426,7 +426,7 @@ class BudgetLimitRepository implements BudgetLimitRepositoryInterface Log::debug(sprintf('%s is zero, delete budget limit #%d', $amount, $limit->id)); try { $limit->delete(); - } catch (Exception $e) { // @phpstan-ignore-line + } catch (Exception $e) { // @ignoreException } diff --git a/app/Repositories/Budget/BudgetRepository.php b/app/Repositories/Budget/BudgetRepository.php index ffa1f0e04e..521aaf9a12 100644 --- a/app/Repositories/Budget/BudgetRepository.php +++ b/app/Repositories/Budget/BudgetRepository.php @@ -198,7 +198,7 @@ class BudgetRepository implements BudgetRepositoryInterface // delete limits with amount 0: try { BudgetLimit::where('amount', 0)->delete(); - } catch (Exception $e) { // @phpstan-ignore-line + } catch (Exception $e) { // @ignoreException } $budgets = $this->getActiveBudgets(); @@ -605,7 +605,7 @@ class BudgetRepository implements BudgetRepositoryInterface if (null !== $dbNote) { try { $dbNote->delete(); - } catch (Exception $e) { // @phpstan-ignore-line + } catch (Exception $e) { // @ignoreException } } diff --git a/app/Repositories/Journal/JournalCLIRepository.php b/app/Repositories/Journal/JournalCLIRepository.php index b292ee1232..37c1a3ed77 100644 --- a/app/Repositories/Journal/JournalCLIRepository.php +++ b/app/Repositories/Journal/JournalCLIRepository.php @@ -127,7 +127,7 @@ class JournalCLIRepository implements JournalCLIRepositoryInterface $result = null; try { $result = new Carbon($cache->get()); - } catch (Exception $e) { // @phpstan-ignore-line + } catch (Exception $e) { // @ignoreException } @@ -141,7 +141,7 @@ class JournalCLIRepository implements JournalCLIRepositoryInterface $value = null; try { $value = new Carbon($entry->data); - } catch (Exception $e) { // @phpstan-ignore-line + } catch (Exception $e) { // @ignoreException } if (null !== $value) { @@ -188,7 +188,7 @@ class JournalCLIRepository implements JournalCLIRepositoryInterface $return = (string) $value; try { $cache->store($return); - } catch (Exception $e) { // @phpstan-ignore-line + } catch (Exception $e) { // @ignoreException } diff --git a/app/Repositories/LinkType/LinkTypeRepository.php b/app/Repositories/LinkType/LinkTypeRepository.php index 3958a012c1..7d73f9926d 100644 --- a/app/Repositories/LinkType/LinkTypeRepository.php +++ b/app/Repositories/LinkType/LinkTypeRepository.php @@ -311,7 +311,7 @@ class LinkTypeRepository implements LinkTypeRepositoryInterface if (null !== $dbNote && '' === $text) { try { $dbNote->delete(); - } catch (Exception $e) { // @phpstan-ignore-line + } catch (Exception $e) { // @ignoreException } } diff --git a/app/Repositories/PiggyBank/ModifiesPiggyBanks.php b/app/Repositories/PiggyBank/ModifiesPiggyBanks.php index b64729e75c..881960611f 100644 --- a/app/Repositories/PiggyBank/ModifiesPiggyBanks.php +++ b/app/Repositories/PiggyBank/ModifiesPiggyBanks.php @@ -366,7 +366,7 @@ trait ModifiesPiggyBanks if (null !== $dbNote) { try { $dbNote->delete(); - } catch (Exception $e) { // @phpstan-ignore-line + } catch (Exception $e) { // @ignoreException } } diff --git a/app/Repositories/TransactionGroup/TransactionGroupRepository.php b/app/Repositories/TransactionGroup/TransactionGroupRepository.php index d48696838c..cfbb2681a1 100644 --- a/app/Repositories/TransactionGroup/TransactionGroupRepository.php +++ b/app/Repositories/TransactionGroup/TransactionGroupRepository.php @@ -178,7 +178,7 @@ class TransactionGroupRepository implements TransactionGroupRepositoryInterface $current = $attachment->toArray(); $current['file_exists'] = true; $current['notes'] = $repository->getNoteText($attachment); - $current['journal_title'] = $attachment->attachable->description; // @phpstan-ignore-line + $current['journal_title'] = $attachment->attachable->description; $result[$journalId][] = $current; } diff --git a/app/Services/Internal/Destroy/AccountDestroyService.php b/app/Services/Internal/Destroy/AccountDestroyService.php index 85f733529e..b7557aa1c6 100644 --- a/app/Services/Internal/Destroy/AccountDestroyService.php +++ b/app/Services/Internal/Destroy/AccountDestroyService.php @@ -70,7 +70,7 @@ class AccountDestroyService // delete account. try { $account->delete(); - } catch (Exception $e) { // @phpstan-ignore-line + } catch (Exception $e) { // @ignoreException } } @@ -122,7 +122,7 @@ class AccountDestroyService $collection = Transaction::groupBy('transaction_journal_id', 'account_id') ->where('account_id', $moveTo->id) - ->get(['transaction_journal_id', 'account_id', DB::raw('count(*) as the_count')]); // @phpstan-ignore-line + ->get(['transaction_journal_id', 'account_id', DB::raw('count(*) as the_count')]); if (0 === $collection->count()) { return; } diff --git a/app/Services/Internal/Destroy/BillDestroyService.php b/app/Services/Internal/Destroy/BillDestroyService.php index 6f2f7df2ba..71a2a9e711 100644 --- a/app/Services/Internal/Destroy/BillDestroyService.php +++ b/app/Services/Internal/Destroy/BillDestroyService.php @@ -39,7 +39,7 @@ class BillDestroyService { try { $bill->delete(); - } catch (Exception $e) { // @phpstan-ignore-line + } catch (Exception $e) { // @ignoreException } } diff --git a/app/Services/Internal/Destroy/BudgetDestroyService.php b/app/Services/Internal/Destroy/BudgetDestroyService.php index 307ea0a7ef..946c894b58 100644 --- a/app/Services/Internal/Destroy/BudgetDestroyService.php +++ b/app/Services/Internal/Destroy/BudgetDestroyService.php @@ -41,7 +41,7 @@ class BudgetDestroyService { try { $budget->delete(); - } catch (Exception $e) { // @phpstan-ignore-line + } catch (Exception $e) { // @ignoreException } diff --git a/app/Services/Internal/Destroy/CategoryDestroyService.php b/app/Services/Internal/Destroy/CategoryDestroyService.php index 63c97ec7a2..f836b85e07 100644 --- a/app/Services/Internal/Destroy/CategoryDestroyService.php +++ b/app/Services/Internal/Destroy/CategoryDestroyService.php @@ -41,7 +41,7 @@ class CategoryDestroyService { try { $category->delete(); - } catch (Exception $e) { // @phpstan-ignore-line + } catch (Exception $e) { // @ignoreException } diff --git a/app/Services/Internal/Destroy/CurrencyDestroyService.php b/app/Services/Internal/Destroy/CurrencyDestroyService.php index 0cb278d5ea..9810dc6856 100644 --- a/app/Services/Internal/Destroy/CurrencyDestroyService.php +++ b/app/Services/Internal/Destroy/CurrencyDestroyService.php @@ -40,7 +40,7 @@ class CurrencyDestroyService { try { $currency->delete(); - } catch (Exception $e) { // @phpstan-ignore-line + } catch (Exception $e) { // @ignoreException } } diff --git a/app/Services/Internal/Destroy/JournalDestroyService.php b/app/Services/Internal/Destroy/JournalDestroyService.php index 36d32873ef..99fdfdd823 100644 --- a/app/Services/Internal/Destroy/JournalDestroyService.php +++ b/app/Services/Internal/Destroy/JournalDestroyService.php @@ -95,7 +95,7 @@ class JournalDestroyService $group->delete(); } } - } catch (Exception $e) { // @phpstan-ignore-line + } catch (Exception $e) { // @ignoreException } } diff --git a/app/Services/Internal/Destroy/RecurrenceDestroyService.php b/app/Services/Internal/Destroy/RecurrenceDestroyService.php index c86b10d249..bca7bfda85 100644 --- a/app/Services/Internal/Destroy/RecurrenceDestroyService.php +++ b/app/Services/Internal/Destroy/RecurrenceDestroyService.php @@ -58,7 +58,7 @@ class RecurrenceDestroyService try { // delete all meta data $recurrence->recurrenceMeta()->delete(); - } catch (Exception $e) { // @phpstan-ignore-line + } catch (Exception $e) { // @ignoreException } // delete all transactions. @@ -67,7 +67,7 @@ class RecurrenceDestroyService $transaction->recurrenceTransactionMeta()->delete(); try { $transaction->delete(); - } catch (Exception $e) { // @phpstan-ignore-line + } catch (Exception $e) { // @ignoreException } } @@ -77,7 +77,7 @@ class RecurrenceDestroyService // delete recurrence try { $recurrence->delete(); - } catch (Exception $e) { // @phpstan-ignore-line + } catch (Exception $e) { // @ignoreException } } diff --git a/app/Services/Internal/Destroy/TransactionGroupDestroyService.php b/app/Services/Internal/Destroy/TransactionGroupDestroyService.php index efc2806178..4d7847b130 100644 --- a/app/Services/Internal/Destroy/TransactionGroupDestroyService.php +++ b/app/Services/Internal/Destroy/TransactionGroupDestroyService.php @@ -45,7 +45,7 @@ class TransactionGroupDestroyService } try { $transactionGroup->delete(); - } catch (Exception $e) { // @phpstan-ignore-line + } catch (Exception $e) { // @ignoreException } } diff --git a/app/Services/Internal/Support/AccountServiceTrait.php b/app/Services/Internal/Support/AccountServiceTrait.php index 3809d579b6..7b44610389 100644 --- a/app/Services/Internal/Support/AccountServiceTrait.php +++ b/app/Services/Internal/Support/AccountServiceTrait.php @@ -174,7 +174,7 @@ trait AccountServiceTrait if (null !== $dbNote) { try { $dbNote->delete(); - } catch (Exception $e) { // @phpstan-ignore-line + } catch (Exception $e) { // @ignoreException } } diff --git a/app/Services/Internal/Support/BillServiceTrait.php b/app/Services/Internal/Support/BillServiceTrait.php index 9aba82b7a0..1e5a7eccbf 100644 --- a/app/Services/Internal/Support/BillServiceTrait.php +++ b/app/Services/Internal/Support/BillServiceTrait.php @@ -72,7 +72,7 @@ trait BillServiceTrait if (null !== $dbNote) { try { $dbNote->delete(); - } catch (Exception $e) { // @phpstan-ignore-line + } catch (Exception $e) { // @ignoreException } } diff --git a/app/Services/Internal/Support/JournalServiceTrait.php b/app/Services/Internal/Support/JournalServiceTrait.php index 0fa4a89b3a..c9e653b1fc 100644 --- a/app/Services/Internal/Support/JournalServiceTrait.php +++ b/app/Services/Internal/Support/JournalServiceTrait.php @@ -397,7 +397,7 @@ trait JournalServiceTrait // try to delete existing notes. try { $note->delete(); - } catch (Exception $e) { // @phpstan-ignore-line + } catch (Exception $e) { // @ignoreException } } diff --git a/app/Services/Internal/Support/RecurringTransactionTrait.php b/app/Services/Internal/Support/RecurringTransactionTrait.php index 9ca2fc2154..95e08cec2d 100644 --- a/app/Services/Internal/Support/RecurringTransactionTrait.php +++ b/app/Services/Internal/Support/RecurringTransactionTrait.php @@ -62,7 +62,7 @@ trait RecurringTransactionTrait if (null !== $dbNote) { try { $dbNote->delete(); - } catch (Exception $e) { // @phpstan-ignore-line + } catch (Exception $e) { // @ignoreException } } @@ -377,7 +377,7 @@ trait RecurringTransactionTrait $transaction->recurrenceTransactionMeta()->delete(); try { $transaction->delete(); - } catch (Exception $e) { // @phpstan-ignore-line + } catch (Exception $e) { // @ignoreException } } diff --git a/app/Services/Internal/Update/CategoryUpdateService.php b/app/Services/Internal/Update/CategoryUpdateService.php index d7ce1936f5..c13bc3c5a9 100644 --- a/app/Services/Internal/Update/CategoryUpdateService.php +++ b/app/Services/Internal/Update/CategoryUpdateService.php @@ -155,7 +155,7 @@ class CategoryUpdateService if (null !== $dbNote) { try { $dbNote->delete(); - } catch (Exception $e) { // @phpstan-ignore-line + } catch (Exception $e) { // @ignoreException } } diff --git a/app/Services/Internal/Update/JournalUpdateService.php b/app/Services/Internal/Update/JournalUpdateService.php index 8b28b2b562..87c9ec168b 100644 --- a/app/Services/Internal/Update/JournalUpdateService.php +++ b/app/Services/Internal/Update/JournalUpdateService.php @@ -640,7 +640,7 @@ class JournalUpdateService if ($this->hasFields([$field])) { try { $value = '' === (string) $this->data[$field] ? null : new Carbon($this->data[$field]); - } catch (Exception $e) { // @phpstan-ignore-line + } catch (Exception $e) { Log::debug(sprintf('%s is not a valid date value: %s', $this->data[$field], $e->getMessage())); return; diff --git a/app/Services/Internal/Update/RecurrenceUpdateService.php b/app/Services/Internal/Update/RecurrenceUpdateService.php index 55518862c4..605d4e1ec4 100644 --- a/app/Services/Internal/Update/RecurrenceUpdateService.php +++ b/app/Services/Internal/Update/RecurrenceUpdateService.php @@ -134,7 +134,7 @@ class RecurrenceUpdateService if (null !== $dbNote && '' === $text) { try { $dbNote->delete(); - } catch (Exception $e) { // @phpstan-ignore-line + } catch (Exception $e) { // @ignoreException } } diff --git a/app/Services/Password/PwndVerifierV2.php b/app/Services/Password/PwndVerifierV2.php index 8dc0f56784..fe25aba448 100644 --- a/app/Services/Password/PwndVerifierV2.php +++ b/app/Services/Password/PwndVerifierV2.php @@ -73,7 +73,7 @@ class PwndVerifierV2 implements Verifier } try { $strpos = stripos($res->getBody()->getContents(), $rest); - } catch (RuntimeException $e) { // @phpstan-ignore-line + } catch (RuntimeException $e) { Log::error(sprintf('Could not get body from Pwnd result: %s', $e->getMessage())); $strpos = false; } diff --git a/app/Support/Authentication/RemoteUserGuard.php b/app/Support/Authentication/RemoteUserGuard.php index 2044856b99..76d4fc0f24 100644 --- a/app/Support/Authentication/RemoteUserGuard.php +++ b/app/Support/Authentication/RemoteUserGuard.php @@ -47,8 +47,7 @@ class RemoteUserGuard implements Guard * @param UserProvider $provider * @param Application $app */ - // @phpstan-ignore-next-line - public function __construct(UserProvider $provider, Application $app) // @phpstan-ignore-line + public function __construct(UserProvider $provider, Application $app) { $this->application = $app; $this->provider = $provider; diff --git a/app/Support/Binder/Date.php b/app/Support/Binder/Date.php index ed021b72f3..745b2d6d54 100644 --- a/app/Support/Binder/Date.php +++ b/app/Support/Binder/Date.php @@ -71,7 +71,7 @@ class Date implements BinderInterface try { $result = new Carbon($value); - } catch (Exception $e) { // @phpstan-ignore-line + } catch (Exception $e) { Log::error(sprintf('Could not parse date "%s" for user #%d: %s', $value, auth()->user()->id, $e->getMessage())); throw new NotFoundHttpException(); } diff --git a/app/Support/ExpandedForm.php b/app/Support/ExpandedForm.php index 9f7150f7ce..812e6e73d5 100644 --- a/app/Support/ExpandedForm.php +++ b/app/Support/ExpandedForm.php @@ -63,7 +63,7 @@ class ExpandedForm //} try { $html = view('form.amount-no-currency', compact('classes', 'name', 'label', 'value', 'options'))->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::error(sprintf('Could not render amountNoCurrency(): %s', $e->getMessage())); $html = 'Could not render amountNoCurrency.'; } @@ -98,7 +98,7 @@ class ExpandedForm unset($options['placeholder'], $options['autocomplete'], $options['class']); try { $html = view('form.checkbox', compact('classes', 'name', 'label', 'value', 'options'))->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::debug(sprintf('Could not render checkbox(): %s', $e->getMessage())); $html = 'Could not render checkbox.'; } @@ -122,7 +122,7 @@ class ExpandedForm unset($options['placeholder']); try { $html = view('form.date', compact('classes', 'name', 'label', 'value', 'options'))->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::debug(sprintf('Could not render date(): %s', $e->getMessage())); $html = 'Could not render date.'; } @@ -144,7 +144,7 @@ class ExpandedForm $classes = $this->getHolderClasses($name); try { $html = view('form.file', compact('classes', 'name', 'label', 'options'))->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::debug(sprintf('Could not render file(): %s', $e->getMessage())); $html = 'Could not render file.'; } @@ -169,7 +169,7 @@ class ExpandedForm $options['step'] = '1'; try { $html = view('form.integer', compact('classes', 'name', 'label', 'value', 'options'))->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::debug(sprintf('Could not render integer(): %s', $e->getMessage())); $html = 'Could not render integer.'; } @@ -193,7 +193,7 @@ class ExpandedForm $value = $this->fillFieldValue($name, $value); try { $html = view('form.location', compact('classes', 'name', 'label', 'value', 'options'))->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::debug(sprintf('Could not render location(): %s', $e->getMessage())); $html = 'Could not render location.'; } @@ -214,12 +214,12 @@ class ExpandedForm $fields = ['title', 'name', 'description']; /** @var Eloquent $entry */ foreach ($set as $entry) { - $entryId = (int) $entry->id; // @phpstan-ignore-line + $entryId = (int) $entry->id; $current = $entry->toArray(); $title = null; foreach ($fields as $field) { if (array_key_exists($field, $current) && null === $title) { - $title = $current[$field]; // @phpstan-ignore-line + $title = $current[$field]; } } $selectList[$entryId] = $title; @@ -251,7 +251,7 @@ class ExpandedForm } try { $html = view('form.non-selectable-amount', compact('selectedCurrency', 'classes', 'name', 'label', 'value', 'options'))->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::debug(sprintf('Could not render nonSelectableAmount(): %s', $e->getMessage())); $html = 'Could not render nonSelectableAmount.'; } @@ -276,7 +276,7 @@ class ExpandedForm unset($options['placeholder']); try { $html = view('form.number', compact('classes', 'name', 'label', 'value', 'options'))->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::debug(sprintf('Could not render number(): %s', $e->getMessage())); $html = 'Could not render number.'; } @@ -305,7 +305,7 @@ class ExpandedForm try { $html = view('form.object_group', compact('classes', 'name', 'label', 'value', 'options'))->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::debug(sprintf('Could not render objectGroup(): %s', $e->getMessage())); $html = 'Could not render objectGroup.'; } @@ -324,7 +324,7 @@ class ExpandedForm { try { $html = view('form.options', compact('type', 'name'))->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::debug(sprintf('Could not render select(): %s', $e->getMessage())); $html = 'Could not render optionsList.'; } @@ -345,7 +345,7 @@ class ExpandedForm $classes = $this->getHolderClasses($name); try { $html = view('form.password', compact('classes', 'name', 'label', 'options'))->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::debug(sprintf('Could not render password(): %s', $e->getMessage())); $html = 'Could not render password.'; } @@ -372,7 +372,7 @@ class ExpandedForm unset($options['placeholder']); try { $html = view('form.percentage', compact('classes', 'name', 'label', 'value', 'options'))->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::debug(sprintf('Could not render percentage(): %s', $e->getMessage())); $html = 'Could not render percentage.'; } @@ -394,7 +394,7 @@ class ExpandedForm $classes = $this->getHolderClasses($name); try { $html = view('form.static', compact('classes', 'name', 'label', 'value', 'options'))->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::debug(sprintf('Could not render staticText(): %s', $e->getMessage())); $html = 'Could not render staticText.'; } @@ -417,7 +417,7 @@ class ExpandedForm $value = $this->fillFieldValue($name, $value); try { $html = view('form.text', compact('classes', 'name', 'label', 'value', 'options'))->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::debug(sprintf('Could not render text(): %s', $e->getMessage())); $html = 'Could not render text.'; } @@ -446,7 +446,7 @@ class ExpandedForm try { $html = view('form.textarea', compact('classes', 'name', 'label', 'value', 'options'))->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::debug(sprintf('Could not render textarea(): %s', $e->getMessage())); $html = 'Could not render textarea.'; } diff --git a/app/Support/FireflyConfig.php b/app/Support/FireflyConfig.php index de1de44014..d126b69465 100644 --- a/app/Support/FireflyConfig.php +++ b/app/Support/FireflyConfig.php @@ -46,7 +46,7 @@ class FireflyConfig } try { Configuration::where('name', $name)->forceDelete(); - } catch (Exception $e) { // @phpstan-ignore-line + } catch (Exception $e) { // @ignoreException } } @@ -68,7 +68,7 @@ class FireflyConfig try { /** @var Configuration|null $config */ $config = Configuration::where('name', $name)->first(['id', 'name', 'data']); - } catch (QueryException|Exception $e) { // @phpstan-ignore-line + } catch (QueryException|Exception $e) { throw new FireflyException(sprintf('Could not poll the database: %s', $e->getMessage())); } @@ -95,7 +95,7 @@ class FireflyConfig { try { $config = Configuration::whereName($name)->whereNull('deleted_at')->first(); - } catch (QueryException|Exception $e) { // @phpstan-ignore-line + } catch (QueryException|Exception $e) { $item = new Configuration(); $item->name = $name; $item->data = $value; diff --git a/app/Support/Form/AccountForm.php b/app/Support/Form/AccountForm.php index 6f8a0fdb2b..55bc076a89 100644 --- a/app/Support/Form/AccountForm.php +++ b/app/Support/Form/AccountForm.php @@ -137,7 +137,7 @@ class AccountForm unset($options['class']); try { $html = view('form.assetAccountCheckList', compact('classes', 'selected', 'name', 'label', 'options', 'grouped'))->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::debug(sprintf('Could not render assetAccountCheckList(): %s', $e->getMessage())); $html = 'Could not render assetAccountCheckList.'; } diff --git a/app/Support/Form/CurrencyForm.php b/app/Support/Form/CurrencyForm.php index 8151f9da3d..0468136c3f 100644 --- a/app/Support/Form/CurrencyForm.php +++ b/app/Support/Form/CurrencyForm.php @@ -96,7 +96,7 @@ class CurrencyForm } try { $html = view('form.' . $view, compact('defaultCurrency', 'currencies', 'classes', 'name', 'label', 'value', 'options'))->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::debug(sprintf('Could not render currencyField(): %s', $e->getMessage())); $html = 'Could not render currencyField.'; } @@ -165,7 +165,7 @@ class CurrencyForm } try { $html = view('form.' . $view, compact('defaultCurrency', 'currencies', 'classes', 'name', 'label', 'value', 'options'))->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::debug(sprintf('Could not render currencyField(): %s', $e->getMessage())); $html = 'Could not render currencyField.'; } diff --git a/app/Support/Form/FormSupport.php b/app/Support/Form/FormSupport.php index 0abf922be0..4291aa1442 100644 --- a/app/Support/Form/FormSupport.php +++ b/app/Support/Form/FormSupport.php @@ -54,7 +54,7 @@ trait FormSupport unset($options['autocomplete'], $options['placeholder']); try { $html = view('form.select', compact('classes', 'name', 'label', 'selected', 'options', 'list'))->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::debug(sprintf('Could not render select(): %s', $e->getMessage())); $html = 'Could not render select.'; } @@ -134,7 +134,7 @@ trait FormSupport if (null !== request()->old($name)) { $value = request()->old($name); } - } catch (RuntimeException $e) { // @phpstan-ignore-line + } catch (RuntimeException $e) { // don't care about session errors. Log::debug(sprintf('Run time: %s', $e->getMessage())); } @@ -163,7 +163,7 @@ trait FormSupport $date = null; try { $date = today(config('app.timezone')); - } catch (Exception $e) { // @phpstan-ignore-line + } catch (Exception $e) { // @ignoreException } diff --git a/app/Support/Http/Controllers/ModelInformation.php b/app/Support/Http/Controllers/ModelInformation.php index 6183de164c..a53fda9510 100644 --- a/app/Support/Http/Controllers/ModelInformation.php +++ b/app/Support/Http/Controllers/ModelInformation.php @@ -57,7 +57,7 @@ trait ModelInformation 'count' => 1, ] )->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::error(sprintf('Throwable was thrown in getActionsForBill(): %s', $e->getMessage())); Log::error($e->getTraceAsString()); $result = 'Could not render view. See log files.'; @@ -144,7 +144,7 @@ trait ModelInformation 'triggers' => $triggers, ] )->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::debug(sprintf('Throwable was thrown in getTriggersForBill(): %s', $e->getMessage())); Log::debug($e->getTraceAsString()); $string = ''; @@ -258,7 +258,7 @@ trait ModelInformation 'triggers' => $triggers, ] )->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::debug(sprintf('Throwable was thrown in getTriggersForJournal(): %s', $e->getMessage())); Log::debug($e->getTraceAsString()); $string = ''; diff --git a/app/Support/Http/Controllers/RenderPartialViews.php b/app/Support/Http/Controllers/RenderPartialViews.php index 48ce1f8f3c..321f0d8b29 100644 --- a/app/Support/Http/Controllers/RenderPartialViews.php +++ b/app/Support/Http/Controllers/RenderPartialViews.php @@ -67,7 +67,7 @@ trait RenderPartialViews try { $view = view('popup.report.balance-amount', compact('journals', 'budget', 'account'))->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::error(sprintf('Could not render: %s', $e->getMessage())); $view = 'Firefly III could not render the view. Please see the log files.'; } @@ -88,7 +88,7 @@ trait RenderPartialViews try { $result = view('reports.options.budget', compact('budgets'))->render(); - } catch (Throwable $e) {// @phpstan-ignore-line + } catch (Throwable $e) { Log::error(sprintf('Cannot render reports.options.tag: %s', $e->getMessage())); $result = 'Could not render view.'; } @@ -119,7 +119,7 @@ trait RenderPartialViews try { $view = view('popup.report.budget-spent-amount', compact('journals', 'budget'))->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::error(sprintf('Could not render: %s', $e->getMessage())); $view = 'Firefly III could not render the view. Please see the log files.'; } @@ -146,7 +146,7 @@ trait RenderPartialViews try { $view = view('popup.report.category-entry', compact('journals', 'category'))->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::error(sprintf('Could not render: %s', $e->getMessage())); $view = 'Firefly III could not render the view. Please see the log files.'; } @@ -167,7 +167,7 @@ trait RenderPartialViews try { $result = view('reports.options.category', compact('categories'))->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::error(sprintf('Cannot render reports.options.category: %s', $e->getMessage())); $result = 'Could not render view.'; } @@ -208,7 +208,7 @@ trait RenderPartialViews try { $result = view('reports.options.double', compact('set'))->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::error(sprintf('Cannot render reports.options.tag: %s', $e->getMessage())); $result = 'Could not render view.'; } @@ -241,7 +241,7 @@ trait RenderPartialViews try { $view = view('popup.report.expense-entry', compact('journals', 'account'))->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::error(sprintf('Could not render: %s', $e->getMessage())); $view = 'Firefly III could not render the view. Please see the log files.'; } @@ -275,7 +275,7 @@ trait RenderPartialViews 'count' => $count, ] )->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::debug(sprintf('Throwable was thrown in getCurrentActions(): %s', $e->getMessage())); Log::error($e->getTraceAsString()); } @@ -329,7 +329,7 @@ trait RenderPartialViews 'triggers' => $triggers, ] )->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::debug(sprintf('Throwable was thrown in getCurrentTriggers(): %s', $e->getMessage())); Log::error($e->getTraceAsString()); } @@ -365,7 +365,7 @@ trait RenderPartialViews try { $view = view('popup.report.income-entry', compact('journals', 'account'))->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::error(sprintf('Could not render: %s', $e->getMessage())); $view = 'Firefly III could not render the view. Please see the log files.'; } @@ -382,7 +382,7 @@ trait RenderPartialViews { try { $result = view('reports.options.no-options')->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::error(sprintf('Cannot render reports.options.no-options: %s', $e->getMessage())); $result = 'Could not render view.'; } @@ -404,7 +404,7 @@ trait RenderPartialViews try { $result = view('reports.options.tag', compact('tags'))->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::error(sprintf('Cannot render reports.options.tag: %s', $e->getMessage())); $result = 'Could not render view.'; } diff --git a/app/Support/Http/Controllers/RuleManagement.php b/app/Support/Http/Controllers/RuleManagement.php index b77cdad3d9..1a360cc3d0 100644 --- a/app/Support/Http/Controllers/RuleManagement.php +++ b/app/Support/Http/Controllers/RuleManagement.php @@ -58,7 +58,7 @@ trait RuleManagement 'count' => $index + 1, ] )->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::debug(sprintf('Throwable was thrown in getPreviousActions(): %s', $e->getMessage())); Log::error($e->getTraceAsString()); } @@ -104,7 +104,7 @@ trait RuleManagement 'triggers' => $triggers, ] )->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::debug(sprintf('Throwable was thrown in getPreviousTriggers(): %s', $e->getMessage())); Log::error($e->getTraceAsString()); } @@ -146,7 +146,7 @@ trait RuleManagement 'triggers' => $triggers, ] )->render(); - } catch (Throwable $e) { // @phpstan-ignore-line + } catch (Throwable $e) { Log::debug(sprintf('Throwable was thrown in getPreviousTriggers(): %s', $e->getMessage())); Log::error($e->getTraceAsString()); } diff --git a/app/Support/Search/OperatorQuerySearch.php b/app/Support/Search/OperatorQuerySearch.php index 813c7e4e27..31724595e5 100644 --- a/app/Support/Search/OperatorQuerySearch.php +++ b/app/Support/Search/OperatorQuerySearch.php @@ -189,7 +189,7 @@ class OperatorQuerySearch implements SearchInterface throw new FireflyException(sprintf('Firefly III search cant handle "%s"-nodes', $class)); case Subquery::class: // loop all notes in subquery: - foreach ($searchNode->getNodes() as $subNode) { // @phpstan-ignore-line + foreach ($searchNode->getNodes() as $subNode) { $this->handleSearchNode($subNode); // let's hope it's not too recursive } break; diff --git a/app/Support/Steam.php b/app/Support/Steam.php index 8b3bd2270d..789989497b 100644 --- a/app/Support/Steam.php +++ b/app/Support/Steam.php @@ -160,7 +160,7 @@ class Steam ->orderBy('transaction_journals.date', 'ASC') ->whereNull('transaction_journals.deleted_at') ->get( - [ // @phpstan-ignore-line + [ 'transaction_journals.date', 'transactions.transaction_currency_id', DB::raw('SUM(transactions.amount) AS modified'), @@ -331,7 +331,7 @@ class Steam ->leftJoin('transaction_journals', 'transaction_journals.id', '=', 'transactions.transaction_journal_id') ->where('transaction_journals.date', '<=', $date->format('Y-m-d 23:59:59')) ->groupBy('transactions.transaction_currency_id'); - $balances = $query->get(['transactions.transaction_currency_id', DB::raw('SUM(transactions.amount) as sum_for_currency')]); // @phpstan-ignore-line + $balances = $query->get(['transactions.transaction_currency_id', DB::raw('SUM(transactions.amount) as sum_for_currency')]); $return = []; /** @var stdClass $entry */ foreach ($balances as $entry) { @@ -416,7 +416,7 @@ class Steam $set = auth()->user()->transactions() ->whereIn('transactions.account_id', $accounts) ->groupBy(['transactions.account_id', 'transaction_journals.user_id']) - ->get(['transactions.account_id', DB::raw('MAX(transaction_journals.date) AS max_date')]); // @phpstan-ignore-line + ->get(['transactions.account_id', DB::raw('MAX(transaction_journals.date) AS max_date')]); foreach ($set as $entry) { $date = new Carbon($entry->max_date, config('app.timezone'));