diff --git a/.ci/phpstan.neon b/.ci/phpstan.neon index 3dfe2330a9..2582567d74 100644 --- a/.ci/phpstan.neon +++ b/.ci/phpstan.neon @@ -1,11 +1,22 @@ parameters: + scanFiles: + - ../_ide_helper_models.php + - ../_ide_helper.php + paths: + - ../app + - ../database + - ../routes + - ../config + - ../bootstrap/app.php universalObjectCratesClasses: - Illuminate\Database\Eloquent\Model # TODO: slowly remove these parameters and fix the issues found. reportUnmatchedIgnoredErrors: false ignoreErrors: # TODO: slowly remove these exceptions and fix the issues found. - # - '#Dynamic call to static method#' # all the Laravel ORM things depend on this. + - '#Dynamic call to static method#' # all the Laravel ORM things depend on this. + - identifier: varTag.nativeType + - identifier: varTag.type # - '#Control structures using switch should not be used.#' # switch is fine in some cases. # - '#with no value type specified in iterable type array#' # remove this rule when all other issues are solved. # - '#has no value type specified in iterable type array#' # remove this rule when all other issues are solved. @@ -48,14 +59,9 @@ parameters: # message: '#Either catch a more specific exception#' # paths: # - ../app/Support/Form/FormSupport.php - paths: - - ../app - - ../database - - ../routes - - ../config - - ../bootstrap/app.php + # The level 8 is the highest level. original was 5 # 7 is more than enough, higher just leaves NULL things. - level: 1 + level: 2 diff --git a/app/Api/V1/Controllers/Data/Export/ExportController.php b/app/Api/V1/Controllers/Data/Export/ExportController.php index cfbe4414b1..b4decbf49f 100644 --- a/app/Api/V1/Controllers/Data/Export/ExportController.php +++ b/app/Api/V1/Controllers/Data/Export/ExportController.php @@ -59,7 +59,7 @@ class ExportController extends Controller * * @throws FireflyException * - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function accounts(ExportRequest $request): LaravelResponse { @@ -100,7 +100,7 @@ class ExportController extends Controller * * @throws FireflyException * - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function bills(ExportRequest $request): LaravelResponse { @@ -115,7 +115,7 @@ class ExportController extends Controller * * @throws FireflyException * - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function budgets(ExportRequest $request): LaravelResponse { @@ -130,7 +130,7 @@ class ExportController extends Controller * * @throws FireflyException * - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function categories(ExportRequest $request): LaravelResponse { @@ -145,7 +145,7 @@ class ExportController extends Controller * * @throws FireflyException * - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function piggyBanks(ExportRequest $request): LaravelResponse { @@ -160,7 +160,7 @@ class ExportController extends Controller * * @throws FireflyException * - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function recurring(ExportRequest $request): LaravelResponse { @@ -175,7 +175,7 @@ class ExportController extends Controller * * @throws FireflyException * - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function rules(ExportRequest $request): LaravelResponse { @@ -190,7 +190,7 @@ class ExportController extends Controller * * @throws FireflyException * - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function tags(ExportRequest $request): LaravelResponse { diff --git a/app/Api/V1/Controllers/Models/BudgetLimit/ShowController.php b/app/Api/V1/Controllers/Models/BudgetLimit/ShowController.php index ddf404a1e3..9be210853c 100644 --- a/app/Api/V1/Controllers/Models/BudgetLimit/ShowController.php +++ b/app/Api/V1/Controllers/Models/BudgetLimit/ShowController.php @@ -100,7 +100,7 @@ class ShowController extends Controller * * Display a listing of the budget limits for this budget. * - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function indexAll(SameDateRequest $request): JsonResponse { diff --git a/app/Api/V1/Controllers/Models/Rule/ExpressionController.php b/app/Api/V1/Controllers/Models/Rule/ExpressionController.php index 5272c14ff3..2187dd7fa5 100644 --- a/app/Api/V1/Controllers/Models/Rule/ExpressionController.php +++ b/app/Api/V1/Controllers/Models/Rule/ExpressionController.php @@ -37,7 +37,7 @@ class ExpressionController extends Controller * This endpoint is documented at: * https://api-docs.firefly-iii.org/?urls.primaryName=2.0.0%20(v1)#/rules/validateExpression * - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function validateExpression(ValidateExpressionRequest $request): JsonResponse { diff --git a/app/Api/V2/Controllers/Chart/CategoryController.php b/app/Api/V2/Controllers/Chart/CategoryController.php index 990b9de811..6bc6267f9e 100644 --- a/app/Api/V2/Controllers/Chart/CategoryController.php +++ b/app/Api/V2/Controllers/Chart/CategoryController.php @@ -71,7 +71,7 @@ class CategoryController extends Controller * * @throws FireflyException * - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function dashboard(DateRequest $request): JsonResponse { diff --git a/app/Api/V2/Controllers/Controller.php b/app/Api/V2/Controllers/Controller.php index ee8d782e48..88f74a0e6b 100644 --- a/app/Api/V2/Controllers/Controller.php +++ b/app/Api/V2/Controllers/Controller.php @@ -74,7 +74,7 @@ class Controller extends BaseController * TODO duplicate from V1 controller * Method to grab all parameters from the URL. * - * @SuppressWarnings(PHPMD.NPathComplexity) + * @SuppressWarnings("PHPMD.NPathComplexity") */ private function getParameters(): ParameterBag { diff --git a/app/Api/V2/Controllers/Model/Bill/SumController.php b/app/Api/V2/Controllers/Model/Bill/SumController.php index 6995d51729..3f8b61bdba 100644 --- a/app/Api/V2/Controllers/Model/Bill/SumController.php +++ b/app/Api/V2/Controllers/Model/Bill/SumController.php @@ -58,7 +58,7 @@ class SumController extends Controller * * TODO see autocomplete/accountcontroller for list. * - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function paid(DateRequest $request): JsonResponse { @@ -74,7 +74,7 @@ class SumController extends Controller * * TODO see autocomplete/accountcontroller for list. * - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function unpaid(DateRequest $request): JsonResponse { diff --git a/app/Api/V2/Controllers/Summary/BasicController.php b/app/Api/V2/Controllers/Summary/BasicController.php index c7875165b7..f54b659f19 100644 --- a/app/Api/V2/Controllers/Summary/BasicController.php +++ b/app/Api/V2/Controllers/Summary/BasicController.php @@ -95,7 +95,7 @@ class BasicController extends Controller * * @throws \Exception * - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function basic(DateRequest $request): JsonResponse { @@ -223,7 +223,7 @@ class BasicController extends Controller } /** - * @SuppressWarnings(PHPMD.ExcessiveMethodLength) + * @SuppressWarnings("PHPMD.ExcessiveMethodLength") */ private function getLeftToSpendInfo(Carbon $start, Carbon $end): array { diff --git a/app/Console/Commands/Upgrade/UpgradesToGroups.php b/app/Console/Commands/Upgrade/UpgradesToGroups.php index bf3113f788..e5304d8c39 100644 --- a/app/Console/Commands/Upgrade/UpgradesToGroups.php +++ b/app/Console/Commands/Upgrade/UpgradesToGroups.php @@ -185,7 +185,7 @@ class UpgradesToGroups extends Command } /** - * @SuppressWarnings(PHPMD.ExcessiveMethodLength) + * @SuppressWarnings("PHPMD.ExcessiveMethodLength") */ private function generateTransaction(TransactionJournal $journal, Transaction $transaction): array { diff --git a/app/Events/TriggeredAuditLog.php b/app/Events/TriggeredAuditLog.php index 19d66b07da..11febd18ff 100644 --- a/app/Events/TriggeredAuditLog.php +++ b/app/Events/TriggeredAuditLog.php @@ -43,7 +43,7 @@ class TriggeredAuditLog extends Event /** * Create a new event instance. * - * @SuppressWarnings(PHPMD.ExcessiveParameterList) + * @SuppressWarnings("PHPMD.ExcessiveParameterList") */ public function __construct(Model $changer, Model $auditable, string $field, mixed $before, mixed $after) { diff --git a/app/Exceptions/GracefulNotFoundHandler.php b/app/Exceptions/GracefulNotFoundHandler.php index 35059c8367..0e953f02ce 100644 --- a/app/Exceptions/GracefulNotFoundHandler.php +++ b/app/Exceptions/GracefulNotFoundHandler.php @@ -48,7 +48,7 @@ class GracefulNotFoundHandler extends ExceptionHandler * * @throws \Throwable * - * @SuppressWarnings(PHPMD.CyclomaticComplexity) + * @SuppressWarnings("PHPMD.CyclomaticComplexity") */ public function render($request, \Throwable $e): Response { diff --git a/app/Exceptions/Handler.php b/app/Exceptions/Handler.php index 3c3aacb68e..d8c48cb6c8 100644 --- a/app/Exceptions/Handler.php +++ b/app/Exceptions/Handler.php @@ -78,8 +78,8 @@ class Handler extends ExceptionHandler * * @throws \Throwable * - * @SuppressWarnings(PHPMD.NPathComplexity) - * @SuppressWarnings(PHPMD.CyclomaticComplexity) + * @SuppressWarnings("PHPMD.NPathComplexity") + * @SuppressWarnings("PHPMD.CyclomaticComplexity") */ public function render($request, \Throwable $e): Response { diff --git a/app/Factory/RecurrenceFactory.php b/app/Factory/RecurrenceFactory.php index e0cf5ce9d7..8df3357b9b 100644 --- a/app/Factory/RecurrenceFactory.php +++ b/app/Factory/RecurrenceFactory.php @@ -54,7 +54,7 @@ class RecurrenceFactory /** * @throws FireflyException * - * @SuppressWarnings(PHPMD.NPathComplexity) + * @SuppressWarnings("PHPMD.NPathComplexity") */ public function create(array $data): Recurrence { diff --git a/app/Factory/TransactionFactory.php b/app/Factory/TransactionFactory.php index d39f31699b..78def382e2 100644 --- a/app/Factory/TransactionFactory.php +++ b/app/Factory/TransactionFactory.php @@ -218,7 +218,7 @@ class TransactionFactory } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function setUser(User $user): void { diff --git a/app/Factory/TransactionJournalFactory.php b/app/Factory/TransactionJournalFactory.php index 9ff2a464ed..b77d292c11 100644 --- a/app/Factory/TransactionJournalFactory.php +++ b/app/Factory/TransactionJournalFactory.php @@ -151,7 +151,7 @@ class TransactionJournalFactory * @throws DuplicateTransactionException * @throws FireflyException * - * @SuppressWarnings(PHPMD.ExcessiveMethodLength) + * @SuppressWarnings("PHPMD.ExcessiveMethodLength") */ private function createJournal(NullArrayObject $row): ?TransactionJournal { diff --git a/app/Handlers/Events/UserEventHandler.php b/app/Handlers/Events/UserEventHandler.php index 91efe68017..14bb41a4a8 100644 --- a/app/Handlers/Events/UserEventHandler.php +++ b/app/Handlers/Events/UserEventHandler.php @@ -110,7 +110,7 @@ class UserEventHandler } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function createExchangeRates(RegisteredUser $event): void { diff --git a/app/Helpers/Collector/Extensions/AttachmentCollection.php b/app/Helpers/Collector/Extensions/AttachmentCollection.php index ab7e420cd5..2217b40c83 100644 --- a/app/Helpers/Collector/Extensions/AttachmentCollection.php +++ b/app/Helpers/Collector/Extensions/AttachmentCollection.php @@ -124,7 +124,7 @@ trait AttachmentCollection * * @return bool * - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ $filter = static function (array $object) use ($name): bool { /** @var array $transaction */ @@ -159,7 +159,7 @@ trait AttachmentCollection * * @return bool * - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ $filter = static function (array $object) use ($name): bool { /** @var array $transaction */ @@ -194,7 +194,7 @@ trait AttachmentCollection * * @return bool * - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ $filter = static function (array $object) use ($name): bool { /** @var array $transaction */ diff --git a/app/Http/Controllers/Account/EditController.php b/app/Http/Controllers/Account/EditController.php index a72c579179..503aae8ad6 100644 --- a/app/Http/Controllers/Account/EditController.php +++ b/app/Http/Controllers/Account/EditController.php @@ -71,7 +71,7 @@ class EditController extends Controller /** * Edit account overview. It's complex, but it just has a lot of if/then/else. * - * @SuppressWarnings(PHPMD.NPathComplexity) + * @SuppressWarnings("PHPMD.NPathComplexity") * * @return Factory|Redirector|RedirectResponse|View */ diff --git a/app/Http/Controllers/Auth/LoginController.php b/app/Http/Controllers/Auth/LoginController.php index 6496591e6b..d464b78b8d 100644 --- a/app/Http/Controllers/Auth/LoginController.php +++ b/app/Http/Controllers/Auth/LoginController.php @@ -163,7 +163,7 @@ class LoginController extends Controller /** * Get the failed login response instance. * - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") * * @throws ValidationException */ diff --git a/app/Http/Controllers/Chart/BudgetController.php b/app/Http/Controllers/Chart/BudgetController.php index 22a2c9b004..996a7432da 100644 --- a/app/Http/Controllers/Chart/BudgetController.php +++ b/app/Http/Controllers/Chart/BudgetController.php @@ -483,7 +483,7 @@ class BudgetController extends Controller * * Suppress warning because this method will be replaced by API calls. * - * @SuppressWarnings(PHPMD.ExcessiveParameterList) + * @SuppressWarnings("PHPMD.ExcessiveParameterList") */ public function period(Budget $budget, TransactionCurrency $currency, Collection $accounts, Carbon $start, Carbon $end): JsonResponse { diff --git a/app/Http/Controllers/Chart/ExpenseReportController.php b/app/Http/Controllers/Chart/ExpenseReportController.php index 7b3997759a..d3806d846e 100644 --- a/app/Http/Controllers/Chart/ExpenseReportController.php +++ b/app/Http/Controllers/Chart/ExpenseReportController.php @@ -69,7 +69,7 @@ class ExpenseReportController extends Controller * * TODO this chart is not multi currency aware. * - * @SuppressWarnings(PHPMD.ExcessiveMethodLength) + * @SuppressWarnings("PHPMD.ExcessiveMethodLength") */ public function mainChart(Collection $accounts, Collection $expense, Carbon $start, Carbon $end): JsonResponse { diff --git a/app/Http/Controllers/Chart/ReportController.php b/app/Http/Controllers/Chart/ReportController.php index cecdde2826..0bf9177164 100644 --- a/app/Http/Controllers/Chart/ReportController.php +++ b/app/Http/Controllers/Chart/ReportController.php @@ -133,7 +133,7 @@ class ReportController extends Controller /** * Shows income and expense, debit/credit: operations. * - * @SuppressWarnings(PHPMD.ExcessiveMethodLength) + * @SuppressWarnings("PHPMD.ExcessiveMethodLength") */ public function operations(Collection $accounts, Carbon $start, Carbon $end): JsonResponse { diff --git a/app/Http/Controllers/Json/RecurrenceController.php b/app/Http/Controllers/Json/RecurrenceController.php index b423ec53ea..10fdb61b67 100644 --- a/app/Http/Controllers/Json/RecurrenceController.php +++ b/app/Http/Controllers/Json/RecurrenceController.php @@ -60,8 +60,8 @@ class RecurrenceController extends Controller /** * Shows all events for a repetition. Used in calendar. * - * @SuppressWarnings(PHPMD.NPathComplexity) - * @SuppressWarnings(PHPMD.CyclomaticComplexity) + * @SuppressWarnings("PHPMD.NPathComplexity") + * @SuppressWarnings("PHPMD.CyclomaticComplexity") * * @throws FireflyException */ diff --git a/app/Http/Controllers/PreferencesController.php b/app/Http/Controllers/PreferencesController.php index c2f9d2b244..c9813b1226 100644 --- a/app/Http/Controllers/PreferencesController.php +++ b/app/Http/Controllers/PreferencesController.php @@ -211,8 +211,8 @@ class PreferencesController extends Controller * * @throws FireflyException * - * @SuppressWarnings(PHPMD.ExcessiveMethodLength) - * @SuppressWarnings(PHPMD.NPathComplexity) + * @SuppressWarnings("PHPMD.ExcessiveMethodLength") + * @SuppressWarnings("PHPMD.NPathComplexity") */ public function postIndex(PreferencesRequest $request) { diff --git a/app/Http/Controllers/Recurring/CreateController.php b/app/Http/Controllers/Recurring/CreateController.php index 21c306d233..9e2e014be9 100644 --- a/app/Http/Controllers/Recurring/CreateController.php +++ b/app/Http/Controllers/Recurring/CreateController.php @@ -124,7 +124,7 @@ class CreateController extends Controller /** * @return Factory|\Illuminate\Contracts\View\View * - * @SuppressWarnings(PHPMD.ExcessiveMethodLength) + * @SuppressWarnings("PHPMD.ExcessiveMethodLength") */ public function createFromJournal(Request $request, TransactionJournal $journal) { diff --git a/app/Http/Controllers/Report/CategoryController.php b/app/Http/Controllers/Report/CategoryController.php index 907525f85f..258dba2d6a 100644 --- a/app/Http/Controllers/Report/CategoryController.php +++ b/app/Http/Controllers/Report/CategoryController.php @@ -156,7 +156,7 @@ class CategoryController extends Controller /** * @return Factory|View * - * @SuppressWarnings(PHPMD.ExcessiveMethodLength) + * @SuppressWarnings("PHPMD.ExcessiveMethodLength") */ public function accounts(Collection $accounts, Collection $categories, Carbon $start, Carbon $end) { @@ -358,7 +358,7 @@ class CategoryController extends Controller /** * @return Factory|View * - * @SuppressWarnings(PHPMD.ExcessiveMethodLength) + * @SuppressWarnings("PHPMD.ExcessiveMethodLength") */ public function categories(Collection $accounts, Collection $categories, Carbon $start, Carbon $end) { diff --git a/app/Http/Controllers/Report/DoubleController.php b/app/Http/Controllers/Report/DoubleController.php index 75840fcef4..d8a76e3f78 100644 --- a/app/Http/Controllers/Report/DoubleController.php +++ b/app/Http/Controllers/Report/DoubleController.php @@ -165,7 +165,7 @@ class DoubleController extends Controller /** * @return Factory|View * - * @SuppressWarnings(PHPMD.ExcessiveMethodLength) + * @SuppressWarnings("PHPMD.ExcessiveMethodLength") */ public function operations(Collection $accounts, Collection $double, Carbon $start, Carbon $end) { diff --git a/app/Http/Controllers/Report/TagController.php b/app/Http/Controllers/Report/TagController.php index fdae8faec2..a60387539a 100644 --- a/app/Http/Controllers/Report/TagController.php +++ b/app/Http/Controllers/Report/TagController.php @@ -59,7 +59,7 @@ class TagController extends Controller /** * @return Factory|View * - * @SuppressWarnings(PHPMD.ExcessiveMethodLength) + * @SuppressWarnings("PHPMD.ExcessiveMethodLength") */ public function accountPerTag(Collection $accounts, Collection $tags, Carbon $start, Carbon $end) { @@ -153,7 +153,7 @@ class TagController extends Controller /** * @return Factory|View * - * @SuppressWarnings(PHPMD.ExcessiveMethodLength) + * @SuppressWarnings("PHPMD.ExcessiveMethodLength") */ public function accounts(Collection $accounts, Collection $tags, Carbon $start, Carbon $end) { @@ -355,7 +355,7 @@ class TagController extends Controller /** * @return Factory|View * - * @SuppressWarnings(PHPMD.ExcessiveMethodLength) + * @SuppressWarnings("PHPMD.ExcessiveMethodLength") */ public function tags(Collection $accounts, Collection $tags, Carbon $start, Carbon $end) { diff --git a/app/Http/Middleware/Authenticate.php b/app/Http/Middleware/Authenticate.php index eec83beed2..4dafb75854 100644 --- a/app/Http/Middleware/Authenticate.php +++ b/app/Http/Middleware/Authenticate.php @@ -76,7 +76,7 @@ class Authenticate * @throws FireflyException * @throws AuthenticationException * - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ protected function authenticate($request, array $guards) { diff --git a/app/Http/Requests/RecurrenceFormRequest.php b/app/Http/Requests/RecurrenceFormRequest.php index 88bf53358c..3d82eb6633 100644 --- a/app/Http/Requests/RecurrenceFormRequest.php +++ b/app/Http/Requests/RecurrenceFormRequest.php @@ -52,7 +52,7 @@ class RecurrenceFormRequest extends FormRequest * * @throws FireflyException * - * @SuppressWarnings(PHPMD.ExcessiveMethodLength) + * @SuppressWarnings("PHPMD.ExcessiveMethodLength") */ public function getAll(): array { diff --git a/app/Models/UserGroup.php b/app/Models/UserGroup.php index b3ed2716a9..05c7509a2c 100644 --- a/app/Models/UserGroup.php +++ b/app/Models/UserGroup.php @@ -26,6 +26,7 @@ namespace FireflyIII\Models; use FireflyIII\Enums\UserRoleEnum; use FireflyIII\Exceptions\FireflyException; +use FireflyIII\Repositories\PiggyBank\PiggyBankRepositoryInterface; use FireflyIII\Support\Models\ReturnsIntegerIdTrait; use FireflyIII\User; use Illuminate\Database\Eloquent\Model; @@ -151,7 +152,7 @@ class UserGroup extends Model */ public function piggyBanks(): HasManyThrough { - throw new FireflyException('This user group method is EOL.'); + return $this->hasManyThrough( PiggyBank::class, Account::class); } public function recurrences(): HasMany diff --git a/app/Notifications/Admin/UnknownUserLoginAttempt.php b/app/Notifications/Admin/UnknownUserLoginAttempt.php index 5409b18ee9..6fa01590fe 100644 --- a/app/Notifications/Admin/UnknownUserLoginAttempt.php +++ b/app/Notifications/Admin/UnknownUserLoginAttempt.php @@ -48,7 +48,7 @@ class UnknownUserLoginAttempt extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toArray(OwnerNotifiable $notifiable) { @@ -57,7 +57,7 @@ class UnknownUserLoginAttempt extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toMail(OwnerNotifiable $notifiable): MailMessage { @@ -73,7 +73,7 @@ class UnknownUserLoginAttempt extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toNtfy(OwnerNotifiable $notifiable): Message { @@ -87,7 +87,7 @@ class UnknownUserLoginAttempt extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toPushover(OwnerNotifiable $notifiable): PushoverMessage { @@ -97,7 +97,7 @@ class UnknownUserLoginAttempt extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toSlack(OwnerNotifiable $notifiable): SlackMessage { @@ -107,7 +107,7 @@ class UnknownUserLoginAttempt extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function via(OwnerNotifiable $notifiable) { diff --git a/app/Notifications/Admin/UserInvitation.php b/app/Notifications/Admin/UserInvitation.php index 88a51f691a..339cfef394 100644 --- a/app/Notifications/Admin/UserInvitation.php +++ b/app/Notifications/Admin/UserInvitation.php @@ -55,7 +55,7 @@ class UserInvitation extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toArray(OwnerNotifiable $notifiable) { @@ -64,7 +64,7 @@ class UserInvitation extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toMail(OwnerNotifiable $notifiable) { @@ -81,7 +81,7 @@ class UserInvitation extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toNtfy(OwnerNotifiable $notifiable): Message { @@ -96,7 +96,7 @@ class UserInvitation extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toPushover(OwnerNotifiable $notifiable): PushoverMessage { @@ -108,7 +108,7 @@ class UserInvitation extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toSlack(OwnerNotifiable $notifiable) { @@ -118,7 +118,7 @@ class UserInvitation extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function via(OwnerNotifiable $notifiable) { diff --git a/app/Notifications/Admin/UserRegistration.php b/app/Notifications/Admin/UserRegistration.php index b3410f2a29..32930bb0a6 100644 --- a/app/Notifications/Admin/UserRegistration.php +++ b/app/Notifications/Admin/UserRegistration.php @@ -55,7 +55,7 @@ class UserRegistration extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toArray(OwnerNotifiable $notifiable) { @@ -64,7 +64,7 @@ class UserRegistration extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toMail(OwnerNotifiable $notifiable) { @@ -80,7 +80,7 @@ class UserRegistration extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toNtfy(OwnerNotifiable $notifiable): Message { @@ -95,7 +95,7 @@ class UserRegistration extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toPushover(OwnerNotifiable $notifiable): PushoverMessage { @@ -107,7 +107,7 @@ class UserRegistration extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toSlack(OwnerNotifiable $notifiable) { @@ -115,7 +115,7 @@ class UserRegistration extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function via(OwnerNotifiable $notifiable) { diff --git a/app/Notifications/Admin/VersionCheckResult.php b/app/Notifications/Admin/VersionCheckResult.php index c4a4e783b9..3408257e4a 100644 --- a/app/Notifications/Admin/VersionCheckResult.php +++ b/app/Notifications/Admin/VersionCheckResult.php @@ -50,7 +50,7 @@ class VersionCheckResult extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toArray(OwnerNotifiable $notifiable) { @@ -59,7 +59,7 @@ class VersionCheckResult extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toMail(OwnerNotifiable $notifiable) { @@ -70,7 +70,7 @@ class VersionCheckResult extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toNtfy(OwnerNotifiable $notifiable): Message { @@ -85,7 +85,7 @@ class VersionCheckResult extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toPushover(OwnerNotifiable $notifiable): PushoverMessage { @@ -97,7 +97,7 @@ class VersionCheckResult extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toSlack(OwnerNotifiable $notifiable) { @@ -109,7 +109,7 @@ class VersionCheckResult extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function via(OwnerNotifiable $notifiable) { diff --git a/app/Notifications/Security/DisabledMFANotification.php b/app/Notifications/Security/DisabledMFANotification.php index c467c478bb..f5b7af8c63 100644 --- a/app/Notifications/Security/DisabledMFANotification.php +++ b/app/Notifications/Security/DisabledMFANotification.php @@ -48,7 +48,7 @@ class DisabledMFANotification extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toArray(User $notifiable) { @@ -57,7 +57,7 @@ class DisabledMFANotification extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toMail(User $notifiable) { @@ -82,7 +82,7 @@ class DisabledMFANotification extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toPushover(User $notifiable): PushoverMessage { @@ -92,7 +92,7 @@ class DisabledMFANotification extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toSlack(User $notifiable) { @@ -102,7 +102,7 @@ class DisabledMFANotification extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function via(User $notifiable) { diff --git a/app/Notifications/Security/EnabledMFANotification.php b/app/Notifications/Security/EnabledMFANotification.php index daac64653b..3074c89121 100644 --- a/app/Notifications/Security/EnabledMFANotification.php +++ b/app/Notifications/Security/EnabledMFANotification.php @@ -48,7 +48,7 @@ class EnabledMFANotification extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toArray(User $notifiable) { @@ -57,7 +57,7 @@ class EnabledMFANotification extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toMail(User $notifiable) { @@ -82,7 +82,7 @@ class EnabledMFANotification extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toPushover(User $notifiable): PushoverMessage { @@ -92,7 +92,7 @@ class EnabledMFANotification extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toSlack(User $notifiable) { @@ -102,7 +102,7 @@ class EnabledMFANotification extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function via(User $notifiable) { diff --git a/app/Notifications/Security/MFABackupFewLeftNotification.php b/app/Notifications/Security/MFABackupFewLeftNotification.php index 60ef94e2f0..99f75b861c 100644 --- a/app/Notifications/Security/MFABackupFewLeftNotification.php +++ b/app/Notifications/Security/MFABackupFewLeftNotification.php @@ -50,7 +50,7 @@ class MFABackupFewLeftNotification extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toArray(User $notifiable) { @@ -59,7 +59,7 @@ class MFABackupFewLeftNotification extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toMail(User $notifiable) { @@ -84,7 +84,7 @@ class MFABackupFewLeftNotification extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toPushover(User $notifiable): PushoverMessage { @@ -94,7 +94,7 @@ class MFABackupFewLeftNotification extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toSlack(User $notifiable) { @@ -104,7 +104,7 @@ class MFABackupFewLeftNotification extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function via(User $notifiable) { diff --git a/app/Notifications/Security/MFABackupNoLeftNotification.php b/app/Notifications/Security/MFABackupNoLeftNotification.php index 7c0344d9ce..bcbd8d0cfb 100644 --- a/app/Notifications/Security/MFABackupNoLeftNotification.php +++ b/app/Notifications/Security/MFABackupNoLeftNotification.php @@ -48,7 +48,7 @@ class MFABackupNoLeftNotification extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toArray(User $notifiable) { @@ -57,7 +57,7 @@ class MFABackupNoLeftNotification extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toMail(User $notifiable) { @@ -82,7 +82,7 @@ class MFABackupNoLeftNotification extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toPushover(User $notifiable): PushoverMessage { @@ -92,7 +92,7 @@ class MFABackupNoLeftNotification extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toSlack(User $notifiable) { @@ -102,7 +102,7 @@ class MFABackupNoLeftNotification extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function via(User $notifiable) { diff --git a/app/Notifications/Security/MFAManyFailedAttemptsNotification.php b/app/Notifications/Security/MFAManyFailedAttemptsNotification.php index 499c4d7ff8..17638358c7 100644 --- a/app/Notifications/Security/MFAManyFailedAttemptsNotification.php +++ b/app/Notifications/Security/MFAManyFailedAttemptsNotification.php @@ -56,7 +56,7 @@ class MFAManyFailedAttemptsNotification extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toMail(User $notifiable) { @@ -81,7 +81,7 @@ class MFAManyFailedAttemptsNotification extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toPushover(User $notifiable): PushoverMessage { @@ -91,7 +91,7 @@ class MFAManyFailedAttemptsNotification extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toSlack(User $notifiable) { @@ -101,7 +101,7 @@ class MFAManyFailedAttemptsNotification extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function via(User $notifiable) { diff --git a/app/Notifications/Security/MFAUsedBackupCodeNotification.php b/app/Notifications/Security/MFAUsedBackupCodeNotification.php index 7c4b754c5c..5eba3c43c8 100644 --- a/app/Notifications/Security/MFAUsedBackupCodeNotification.php +++ b/app/Notifications/Security/MFAUsedBackupCodeNotification.php @@ -48,7 +48,7 @@ class MFAUsedBackupCodeNotification extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toArray(User $notifiable) { @@ -57,7 +57,7 @@ class MFAUsedBackupCodeNotification extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toMail(User $notifiable) { @@ -82,7 +82,7 @@ class MFAUsedBackupCodeNotification extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toPushover(User $notifiable): PushoverMessage { @@ -92,7 +92,7 @@ class MFAUsedBackupCodeNotification extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toSlack(User $notifiable) { @@ -102,7 +102,7 @@ class MFAUsedBackupCodeNotification extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function via(User $notifiable) { diff --git a/app/Notifications/Security/NewBackupCodesNotification.php b/app/Notifications/Security/NewBackupCodesNotification.php index c3a7e72144..1251c6cf88 100644 --- a/app/Notifications/Security/NewBackupCodesNotification.php +++ b/app/Notifications/Security/NewBackupCodesNotification.php @@ -48,7 +48,7 @@ class NewBackupCodesNotification extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toArray(User $notifiable) { @@ -57,7 +57,7 @@ class NewBackupCodesNotification extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toMail(User $notifiable) { @@ -82,7 +82,7 @@ class NewBackupCodesNotification extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toPushover(User $notifiable): PushoverMessage { @@ -92,7 +92,7 @@ class NewBackupCodesNotification extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toSlack(User $notifiable) { @@ -102,7 +102,7 @@ class NewBackupCodesNotification extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function via(User $notifiable) { diff --git a/app/Notifications/Security/UserFailedLoginAttempt.php b/app/Notifications/Security/UserFailedLoginAttempt.php index 25d02183e2..7ed59f4127 100644 --- a/app/Notifications/Security/UserFailedLoginAttempt.php +++ b/app/Notifications/Security/UserFailedLoginAttempt.php @@ -54,7 +54,7 @@ class UserFailedLoginAttempt extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toMail(User $notifiable) { @@ -79,7 +79,7 @@ class UserFailedLoginAttempt extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toPushover(User $notifiable): PushoverMessage { @@ -89,7 +89,7 @@ class UserFailedLoginAttempt extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toSlack(User $notifiable) { @@ -99,7 +99,7 @@ class UserFailedLoginAttempt extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function via(User $notifiable) { diff --git a/app/Notifications/Test/OwnerTestNotificationEmail.php b/app/Notifications/Test/OwnerTestNotificationEmail.php index 72b1c85443..ac377bb552 100644 --- a/app/Notifications/Test/OwnerTestNotificationEmail.php +++ b/app/Notifications/Test/OwnerTestNotificationEmail.php @@ -44,7 +44,7 @@ class OwnerTestNotificationEmail extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toArray(OwnerNotifiable $notifiable) { @@ -53,7 +53,7 @@ class OwnerTestNotificationEmail extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toMail(OwnerNotifiable $notifiable) { @@ -66,7 +66,7 @@ class OwnerTestNotificationEmail extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function via(OwnerNotifiable $notifiable) { diff --git a/app/Notifications/Test/OwnerTestNotificationNtfy.php b/app/Notifications/Test/OwnerTestNotificationNtfy.php index 36ea816bf9..ef308f018a 100644 --- a/app/Notifications/Test/OwnerTestNotificationNtfy.php +++ b/app/Notifications/Test/OwnerTestNotificationNtfy.php @@ -48,7 +48,7 @@ class OwnerTestNotificationNtfy extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") * * @param mixed $notifiable */ @@ -59,7 +59,7 @@ class OwnerTestNotificationNtfy extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toNtfy(OwnerNotifiable $notifiable): Message { @@ -74,7 +74,7 @@ class OwnerTestNotificationNtfy extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function via(OwnerNotifiable $notifiable) { diff --git a/app/Notifications/Test/OwnerTestNotificationPushover.php b/app/Notifications/Test/OwnerTestNotificationPushover.php index 876d81f038..b6a4640485 100644 --- a/app/Notifications/Test/OwnerTestNotificationPushover.php +++ b/app/Notifications/Test/OwnerTestNotificationPushover.php @@ -48,7 +48,7 @@ class OwnerTestNotificationPushover extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toArray(OwnerNotifiable $notifiable) { @@ -57,7 +57,7 @@ class OwnerTestNotificationPushover extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toPushover(OwnerNotifiable $notifiable): PushoverMessage { @@ -69,7 +69,7 @@ class OwnerTestNotificationPushover extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function via(OwnerNotifiable $notifiable) { diff --git a/app/Notifications/Test/OwnerTestNotificationSlack.php b/app/Notifications/Test/OwnerTestNotificationSlack.php index a8dab9aa0e..efcad83144 100644 --- a/app/Notifications/Test/OwnerTestNotificationSlack.php +++ b/app/Notifications/Test/OwnerTestNotificationSlack.php @@ -46,7 +46,7 @@ class OwnerTestNotificationSlack extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toArray(OwnerNotifiable $notifiable) { @@ -55,7 +55,7 @@ class OwnerTestNotificationSlack extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toSlack(OwnerNotifiable $notifiable) { @@ -64,7 +64,7 @@ class OwnerTestNotificationSlack extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function via(OwnerNotifiable $notifiable) { diff --git a/app/Notifications/Test/UserTestNotificationEmail.php b/app/Notifications/Test/UserTestNotificationEmail.php index 6001bcb69c..5cd791005c 100644 --- a/app/Notifications/Test/UserTestNotificationEmail.php +++ b/app/Notifications/Test/UserTestNotificationEmail.php @@ -44,7 +44,7 @@ class UserTestNotificationEmail extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toArray(User $notifiable) { @@ -63,7 +63,7 @@ class UserTestNotificationEmail extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function via(User $notifiable) { diff --git a/app/Notifications/Test/UserTestNotificationNtfy.php b/app/Notifications/Test/UserTestNotificationNtfy.php index 0305c40775..34862a2089 100644 --- a/app/Notifications/Test/UserTestNotificationNtfy.php +++ b/app/Notifications/Test/UserTestNotificationNtfy.php @@ -48,7 +48,7 @@ class UserTestNotificationNtfy extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toArray(User $notifiable) { @@ -57,7 +57,7 @@ class UserTestNotificationNtfy extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toNtfy(User $user): Message { @@ -72,7 +72,7 @@ class UserTestNotificationNtfy extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function via(User $user) { diff --git a/app/Notifications/Test/UserTestNotificationPushover.php b/app/Notifications/Test/UserTestNotificationPushover.php index d4da811e72..9cbe6df75e 100644 --- a/app/Notifications/Test/UserTestNotificationPushover.php +++ b/app/Notifications/Test/UserTestNotificationPushover.php @@ -48,7 +48,7 @@ class UserTestNotificationPushover extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toArray(User $notifiable) { @@ -57,7 +57,7 @@ class UserTestNotificationPushover extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toPushover(User $notifiable): PushoverMessage { @@ -69,7 +69,7 @@ class UserTestNotificationPushover extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function via(User $notifiable) { diff --git a/app/Notifications/Test/UserTestNotificationSlack.php b/app/Notifications/Test/UserTestNotificationSlack.php index 150ee0dd62..1c267265fa 100644 --- a/app/Notifications/Test/UserTestNotificationSlack.php +++ b/app/Notifications/Test/UserTestNotificationSlack.php @@ -46,7 +46,7 @@ class UserTestNotificationSlack extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toArray(User $user) { @@ -55,7 +55,7 @@ class UserTestNotificationSlack extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toSlack(User $user) { @@ -64,7 +64,7 @@ class UserTestNotificationSlack extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function via(User $user) { diff --git a/app/Notifications/User/BillReminder.php b/app/Notifications/User/BillReminder.php index 4e65a0b55b..eadde2d75c 100644 --- a/app/Notifications/User/BillReminder.php +++ b/app/Notifications/User/BillReminder.php @@ -54,7 +54,7 @@ class BillReminder extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toArray(User $notifiable) { @@ -63,7 +63,7 @@ class BillReminder extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toMail(User $notifiable) { @@ -95,7 +95,7 @@ class BillReminder extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toPushover(User $notifiable): PushoverMessage { @@ -105,7 +105,7 @@ class BillReminder extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toSlack(User $notifiable) { @@ -122,7 +122,7 @@ class BillReminder extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function via(User $notifiable) { diff --git a/app/Notifications/User/NewAccessToken.php b/app/Notifications/User/NewAccessToken.php index dd8873fb43..9e659ac40d 100644 --- a/app/Notifications/User/NewAccessToken.php +++ b/app/Notifications/User/NewAccessToken.php @@ -52,7 +52,7 @@ class NewAccessToken extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toMail(User $notifiable) { @@ -79,7 +79,7 @@ class NewAccessToken extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toPushover(User $notifiable): PushoverMessage { @@ -89,7 +89,7 @@ class NewAccessToken extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toSlack(User $notifiable) { @@ -97,7 +97,7 @@ class NewAccessToken extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function via(User $notifiable) { diff --git a/app/Notifications/User/RuleActionFailed.php b/app/Notifications/User/RuleActionFailed.php index d798ff666f..b9f3ed9f05 100644 --- a/app/Notifications/User/RuleActionFailed.php +++ b/app/Notifications/User/RuleActionFailed.php @@ -57,7 +57,7 @@ class RuleActionFailed extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toArray(User $notifiable) { @@ -76,7 +76,7 @@ class RuleActionFailed extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toPushover(User $notifiable): PushoverMessage { @@ -84,7 +84,7 @@ class RuleActionFailed extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toSlack(User $notifiable) { @@ -101,7 +101,7 @@ class RuleActionFailed extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function via(User $notifiable) { diff --git a/app/Notifications/User/TransactionCreation.php b/app/Notifications/User/TransactionCreation.php index 034ffc39c8..82a5606e86 100644 --- a/app/Notifications/User/TransactionCreation.php +++ b/app/Notifications/User/TransactionCreation.php @@ -44,7 +44,7 @@ class TransactionCreation extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toArray(User $notifiable) { @@ -53,7 +53,7 @@ class TransactionCreation extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toMail(User $notifiable) { @@ -64,7 +64,7 @@ class TransactionCreation extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function via(User $notifiable) { diff --git a/app/Notifications/User/UserLogin.php b/app/Notifications/User/UserLogin.php index 95e5bcba13..3cadefb41c 100644 --- a/app/Notifications/User/UserLogin.php +++ b/app/Notifications/User/UserLogin.php @@ -50,7 +50,7 @@ class UserLogin extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toMail(User $notifiable) { @@ -79,7 +79,7 @@ class UserLogin extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toPushover(User $notifiable): PushoverMessage { @@ -92,7 +92,7 @@ class UserLogin extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toSlack(User $notifiable) { @@ -103,7 +103,7 @@ class UserLogin extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function via(User $notifiable) { diff --git a/app/Notifications/User/UserNewPassword.php b/app/Notifications/User/UserNewPassword.php index ca14855de5..372e8c0df3 100644 --- a/app/Notifications/User/UserNewPassword.php +++ b/app/Notifications/User/UserNewPassword.php @@ -51,7 +51,7 @@ class UserNewPassword extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toArray(User $notifiable) { @@ -60,7 +60,7 @@ class UserNewPassword extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toMail(User $notifiable) { @@ -86,7 +86,7 @@ class UserNewPassword extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toPushover(User $notifiable): PushoverMessage { @@ -94,7 +94,7 @@ class UserNewPassword extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toSlack(User $notifiable) { diff --git a/app/Notifications/User/UserRegistration.php b/app/Notifications/User/UserRegistration.php index 2d7c32a07c..bcc8947993 100644 --- a/app/Notifications/User/UserRegistration.php +++ b/app/Notifications/User/UserRegistration.php @@ -39,7 +39,7 @@ class UserRegistration extends Notification public function __construct() {} /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toArray(User $notifiable) { @@ -48,7 +48,7 @@ class UserRegistration extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function toMail(User $notifiable) { @@ -59,7 +59,7 @@ class UserRegistration extends Notification } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function via(User $notifiable) { diff --git a/app/Providers/AuthServiceProvider.php b/app/Providers/AuthServiceProvider.php index 6ad1fc165b..e562dd71de 100644 --- a/app/Providers/AuthServiceProvider.php +++ b/app/Providers/AuthServiceProvider.php @@ -42,7 +42,7 @@ class AuthServiceProvider extends ServiceProvider /** * Register any authentication / authorization services. * - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function boot(): void { diff --git a/app/Providers/BudgetServiceProvider.php b/app/Providers/BudgetServiceProvider.php index 1c31ea805f..69350c0977 100644 --- a/app/Providers/BudgetServiceProvider.php +++ b/app/Providers/BudgetServiceProvider.php @@ -55,7 +55,7 @@ class BudgetServiceProvider extends ServiceProvider /** * Register the application services. * - * @SuppressWarnings(PHPMD.ExcessiveMethodLength) + * @SuppressWarnings("PHPMD.ExcessiveMethodLength") */ public function register(): void { diff --git a/app/Providers/FireflyServiceProvider.php b/app/Providers/FireflyServiceProvider.php index 752fb21309..cbb3fd8d0c 100644 --- a/app/Providers/FireflyServiceProvider.php +++ b/app/Providers/FireflyServiceProvider.php @@ -98,7 +98,7 @@ class FireflyServiceProvider extends ServiceProvider /** * Register stuff. * - * @SuppressWarnings(PHPMD.ExcessiveMethodLength) + * @SuppressWarnings("PHPMD.ExcessiveMethodLength") */ public function register(): void { diff --git a/app/Repositories/Account/OperationsRepository.php b/app/Repositories/Account/OperationsRepository.php index d7959127d8..ab37ea4b87 100644 --- a/app/Repositories/Account/OperationsRepository.php +++ b/app/Repositories/Account/OperationsRepository.php @@ -122,7 +122,7 @@ class OperationsRepository implements OperationsRepositoryInterface } /** - * @SuppressWarnings(PHPMD.ExcessiveParameterList) + * @SuppressWarnings("PHPMD.ExcessiveParameterList") */ public function sumExpenses( Carbon $start, @@ -137,8 +137,8 @@ class OperationsRepository implements OperationsRepositoryInterface } /** - * @SuppressWarnings(PHPMD.ExcessiveParameterList) - * @SuppressWarnings(PHPMD.NPathComplexity) + * @SuppressWarnings("PHPMD.ExcessiveParameterList") + * @SuppressWarnings("PHPMD.NPathComplexity") */ private function getTransactionsForSum( string $type, @@ -223,7 +223,7 @@ class OperationsRepository implements OperationsRepositoryInterface } /** - * @SuppressWarnings(PHPMD.ExcessiveParameterList) + * @SuppressWarnings("PHPMD.ExcessiveParameterList") */ public function sumExpensesByDestination( Carbon $start, @@ -245,7 +245,7 @@ class OperationsRepository implements OperationsRepositoryInterface } /** - * @SuppressWarnings(PHPMD.ExcessiveParameterList) + * @SuppressWarnings("PHPMD.ExcessiveParameterList") */ public function sumExpensesBySource( Carbon $start, @@ -260,7 +260,7 @@ class OperationsRepository implements OperationsRepositoryInterface } /** - * @SuppressWarnings(PHPMD.ExcessiveParameterList) + * @SuppressWarnings("PHPMD.ExcessiveParameterList") */ public function sumIncome( Carbon $start, @@ -275,7 +275,7 @@ class OperationsRepository implements OperationsRepositoryInterface } /** - * @SuppressWarnings(PHPMD.ExcessiveParameterList) + * @SuppressWarnings("PHPMD.ExcessiveParameterList") */ public function sumIncomeByDestination( Carbon $start, @@ -290,7 +290,7 @@ class OperationsRepository implements OperationsRepositoryInterface } /** - * @SuppressWarnings(PHPMD.ExcessiveParameterList) + * @SuppressWarnings("PHPMD.ExcessiveParameterList") */ public function sumIncomeBySource( Carbon $start, diff --git a/app/Repositories/Account/OperationsRepositoryInterface.php b/app/Repositories/Account/OperationsRepositoryInterface.php index 63574ae97c..aea8031c7c 100644 --- a/app/Repositories/Account/OperationsRepositoryInterface.php +++ b/app/Repositories/Account/OperationsRepositoryInterface.php @@ -54,7 +54,7 @@ interface OperationsRepositoryInterface /** * Sum of withdrawal journals in period for a set of accounts, grouped per currency. Amounts are always negative. * - * @SuppressWarnings(PHPMD.ExcessiveParameterList) + * @SuppressWarnings("PHPMD.ExcessiveParameterList") */ public function sumExpenses( Carbon $start, @@ -68,7 +68,7 @@ interface OperationsRepositoryInterface * Sum of withdrawal journals in period for a set of accounts, grouped per destination / currency. Amounts are * always negative. * - * @SuppressWarnings(PHPMD.ExcessiveParameterList) + * @SuppressWarnings("PHPMD.ExcessiveParameterList") */ public function sumExpensesByDestination( Carbon $start, @@ -82,7 +82,7 @@ interface OperationsRepositoryInterface * Sum of withdrawal journals in period for a set of accounts, grouped per source / currency. Amounts are always * negative. * - * @SuppressWarnings(PHPMD.ExcessiveParameterList) + * @SuppressWarnings("PHPMD.ExcessiveParameterList") */ public function sumExpensesBySource( Carbon $start, @@ -95,7 +95,7 @@ interface OperationsRepositoryInterface /** * Sum of income journals in period for a set of accounts, grouped per currency. Amounts are always positive. * - * @SuppressWarnings(PHPMD.ExcessiveParameterList) + * @SuppressWarnings("PHPMD.ExcessiveParameterList") */ public function sumIncome( Carbon $start, @@ -109,7 +109,7 @@ interface OperationsRepositoryInterface * Sum of income journals in period for a set of accounts, grouped per destination + currency. Amounts are always * positive. * - * @SuppressWarnings(PHPMD.ExcessiveParameterList) + * @SuppressWarnings("PHPMD.ExcessiveParameterList") */ public function sumIncomeByDestination( Carbon $start, @@ -123,7 +123,7 @@ interface OperationsRepositoryInterface * Sum of income journals in period for a set of accounts, grouped per source + currency. Amounts are always * positive. * - * @SuppressWarnings(PHPMD.ExcessiveParameterList) + * @SuppressWarnings("PHPMD.ExcessiveParameterList") */ public function sumIncomeBySource( Carbon $start, diff --git a/app/Repositories/Budget/BudgetRepository.php b/app/Repositories/Budget/BudgetRepository.php index 82c36a2c64..096ffc32fd 100644 --- a/app/Repositories/Budget/BudgetRepository.php +++ b/app/Repositories/Budget/BudgetRepository.php @@ -721,7 +721,7 @@ class BudgetRepository implements BudgetRepositoryInterface /** * @throws FireflyException * - * @SuppressWarnings(PHPMD.NPathComplexity) + * @SuppressWarnings("PHPMD.NPathComplexity") */ public function store(array $data): Budget { diff --git a/app/Repositories/Budget/OperationsRepository.php b/app/Repositories/Budget/OperationsRepository.php index 2f05c7ecaa..25ad300558 100644 --- a/app/Repositories/Budget/OperationsRepository.php +++ b/app/Repositories/Budget/OperationsRepository.php @@ -203,7 +203,7 @@ class OperationsRepository implements OperationsRepositoryInterface } /** - * @SuppressWarnings(PHPMD.ExcessiveParameterList) + * @SuppressWarnings("PHPMD.ExcessiveParameterList") */ public function sumExpenses( Carbon $start, diff --git a/app/Repositories/Budget/OperationsRepositoryInterface.php b/app/Repositories/Budget/OperationsRepositoryInterface.php index f472f99feb..f35973432e 100644 --- a/app/Repositories/Budget/OperationsRepositoryInterface.php +++ b/app/Repositories/Budget/OperationsRepositoryInterface.php @@ -57,7 +57,7 @@ interface OperationsRepositoryInterface public function setUser(null|Authenticatable|User $user): void; /** - * @SuppressWarnings(PHPMD.ExcessiveParameterList) + * @SuppressWarnings("PHPMD.ExcessiveParameterList") */ public function sumExpenses( Carbon $start, diff --git a/app/Repositories/UserGroup/UserGroupRepository.php b/app/Repositories/UserGroup/UserGroupRepository.php index 89baea8799..164caff7da 100644 --- a/app/Repositories/UserGroup/UserGroupRepository.php +++ b/app/Repositories/UserGroup/UserGroupRepository.php @@ -203,7 +203,7 @@ class UserGroupRepository implements UserGroupRepositoryInterface } /** - * @SuppressWarnings(PHPMD.NPathComplexity) + * @SuppressWarnings("PHPMD.NPathComplexity") * * @throws FireflyException */ diff --git a/app/Rules/Admin/IsValidDiscordUrl.php b/app/Rules/Admin/IsValidDiscordUrl.php index df670a60c6..9d328612d5 100644 --- a/app/Rules/Admin/IsValidDiscordUrl.php +++ b/app/Rules/Admin/IsValidDiscordUrl.php @@ -34,7 +34,7 @@ class IsValidDiscordUrl implements ValidationRule use ValidatesAmountsTrait; /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function validate(string $attribute, mixed $value, \Closure $fail): void { diff --git a/app/Rules/Admin/IsValidSlackOrDiscordUrl.php b/app/Rules/Admin/IsValidSlackOrDiscordUrl.php index 429f5654fa..866127f988 100644 --- a/app/Rules/Admin/IsValidSlackOrDiscordUrl.php +++ b/app/Rules/Admin/IsValidSlackOrDiscordUrl.php @@ -34,7 +34,7 @@ class IsValidSlackOrDiscordUrl implements ValidationRule use ValidatesAmountsTrait; /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function validate(string $attribute, mixed $value, \Closure $fail): void { diff --git a/app/Rules/Admin/IsValidSlackUrl.php b/app/Rules/Admin/IsValidSlackUrl.php index 86908f086c..e7e12fd464 100644 --- a/app/Rules/Admin/IsValidSlackUrl.php +++ b/app/Rules/Admin/IsValidSlackUrl.php @@ -34,7 +34,7 @@ class IsValidSlackUrl implements ValidationRule use ValidatesAmountsTrait; /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function validate(string $attribute, mixed $value, \Closure $fail): void { diff --git a/app/Rules/IsAssetAccountId.php b/app/Rules/IsAssetAccountId.php index 273a5b3f26..d5b44d4aef 100644 --- a/app/Rules/IsAssetAccountId.php +++ b/app/Rules/IsAssetAccountId.php @@ -34,7 +34,7 @@ use Illuminate\Contracts\Validation\ValidationRule; class IsAssetAccountId implements ValidationRule { /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function validate(string $attribute, mixed $value, \Closure $fail): void { diff --git a/app/Rules/IsBoolean.php b/app/Rules/IsBoolean.php index 4ca0ea2c63..9ffd5a7366 100644 --- a/app/Rules/IsBoolean.php +++ b/app/Rules/IsBoolean.php @@ -32,7 +32,7 @@ use Illuminate\Contracts\Validation\ValidationRule; class IsBoolean implements ValidationRule { /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function validate(string $attribute, mixed $value, \Closure $fail): void { diff --git a/app/Rules/IsDateOrTime.php b/app/Rules/IsDateOrTime.php index 32d0176aad..f976cdaac8 100644 --- a/app/Rules/IsDateOrTime.php +++ b/app/Rules/IsDateOrTime.php @@ -35,7 +35,7 @@ use Illuminate\Contracts\Validation\ValidationRule; class IsDateOrTime implements ValidationRule { /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function validate(string $attribute, mixed $value, \Closure $fail): void { diff --git a/app/Rules/IsDefaultUserGroupName.php b/app/Rules/IsDefaultUserGroupName.php index 09a94329e7..4113d222b7 100644 --- a/app/Rules/IsDefaultUserGroupName.php +++ b/app/Rules/IsDefaultUserGroupName.php @@ -42,7 +42,7 @@ class IsDefaultUserGroupName implements ValidationRule } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function validate(string $attribute, mixed $value, \Closure $fail): void { diff --git a/app/Rules/IsDuplicateTransaction.php b/app/Rules/IsDuplicateTransaction.php index 94d60222ec..b8076b20f6 100644 --- a/app/Rules/IsDuplicateTransaction.php +++ b/app/Rules/IsDuplicateTransaction.php @@ -36,7 +36,7 @@ class IsDuplicateTransaction implements ValidationRule private string $value; /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function validate(string $attribute, mixed $value, \Closure $fail): void { diff --git a/app/Rules/IsFilterValueIn.php b/app/Rules/IsFilterValueIn.php index 2e4b92d0d6..3a08483811 100644 --- a/app/Rules/IsFilterValueIn.php +++ b/app/Rules/IsFilterValueIn.php @@ -38,7 +38,7 @@ class IsFilterValueIn implements ValidationRule } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function validate(string $attribute, mixed $value, \Closure $fail): void { diff --git a/app/Rules/IsTransferAccount.php b/app/Rules/IsTransferAccount.php index 8a825ebd02..ae678863db 100644 --- a/app/Rules/IsTransferAccount.php +++ b/app/Rules/IsTransferAccount.php @@ -35,7 +35,7 @@ use Illuminate\Contracts\Validation\ValidationRule; class IsTransferAccount implements ValidationRule { /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function validate(string $attribute, mixed $value, \Closure $fail): void { diff --git a/app/Rules/IsValidActionExpression.php b/app/Rules/IsValidActionExpression.php index dc79a129eb..48fcb02efc 100644 --- a/app/Rules/IsValidActionExpression.php +++ b/app/Rules/IsValidActionExpression.php @@ -36,7 +36,7 @@ class IsValidActionExpression implements ValidationRule * * @param \Closure(string): PotentiallyTranslatedString $fail * - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function validate(string $attribute, mixed $value, \Closure $fail): void { diff --git a/app/Rules/IsValidAmount.php b/app/Rules/IsValidAmount.php index ca8b5b429b..b39dfa13ab 100644 --- a/app/Rules/IsValidAmount.php +++ b/app/Rules/IsValidAmount.php @@ -34,7 +34,7 @@ class IsValidAmount implements ValidationRule use ValidatesAmountsTrait; /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function validate(string $attribute, mixed $value, \Closure $fail): void { diff --git a/app/Rules/IsValidAttachmentModel.php b/app/Rules/IsValidAttachmentModel.php index dcdec6409f..5a547d092a 100644 --- a/app/Rules/IsValidAttachmentModel.php +++ b/app/Rules/IsValidAttachmentModel.php @@ -68,7 +68,7 @@ class IsValidAttachmentModel implements ValidationRule } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function validate(string $attribute, mixed $value, \Closure $fail): void { diff --git a/app/Rules/IsValidBulkClause.php b/app/Rules/IsValidBulkClause.php index 8250c169ac..30d108edd1 100644 --- a/app/Rules/IsValidBulkClause.php +++ b/app/Rules/IsValidBulkClause.php @@ -47,7 +47,7 @@ class IsValidBulkClause implements ValidationRule } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function validate(string $attribute, mixed $value, \Closure $fail): void { diff --git a/app/Rules/IsValidDateRange.php b/app/Rules/IsValidDateRange.php index c66eedbbac..3d35f011be 100644 --- a/app/Rules/IsValidDateRange.php +++ b/app/Rules/IsValidDateRange.php @@ -32,7 +32,7 @@ use Illuminate\Contracts\Validation\ValidationRule; class IsValidDateRange implements ValidationRule { /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function validate(string $attribute, mixed $value, \Closure $fail): void { diff --git a/app/Rules/IsValidPositiveAmount.php b/app/Rules/IsValidPositiveAmount.php index f6b0c1abe3..233fab613d 100644 --- a/app/Rules/IsValidPositiveAmount.php +++ b/app/Rules/IsValidPositiveAmount.php @@ -34,7 +34,7 @@ class IsValidPositiveAmount implements ValidationRule use ValidatesAmountsTrait; /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function validate(string $attribute, mixed $value, \Closure $fail): void { diff --git a/app/Rules/IsValidZeroOrMoreAmount.php b/app/Rules/IsValidZeroOrMoreAmount.php index 784bf518cb..da87757a6a 100644 --- a/app/Rules/IsValidZeroOrMoreAmount.php +++ b/app/Rules/IsValidZeroOrMoreAmount.php @@ -34,7 +34,7 @@ class IsValidZeroOrMoreAmount implements ValidationRule use ValidatesAmountsTrait; /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function validate(string $attribute, mixed $value, \Closure $fail): void { diff --git a/app/Rules/LessThanPiggyTarget.php b/app/Rules/LessThanPiggyTarget.php index a0fc41c38c..2fa6a729eb 100644 --- a/app/Rules/LessThanPiggyTarget.php +++ b/app/Rules/LessThanPiggyTarget.php @@ -40,7 +40,7 @@ class LessThanPiggyTarget implements ValidationRule } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function validate(string $attribute, mixed $value, \Closure $fail): void { diff --git a/app/Rules/UniqueAccountNumber.php b/app/Rules/UniqueAccountNumber.php index a45a8d0467..c44ffb87b5 100644 --- a/app/Rules/UniqueAccountNumber.php +++ b/app/Rules/UniqueAccountNumber.php @@ -70,7 +70,7 @@ class UniqueAccountNumber implements ValidationRule } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function validate(string $attribute, mixed $value, \Closure $fail): void { diff --git a/app/Rules/UniqueIban.php b/app/Rules/UniqueIban.php index 5f2116dcad..54b71a44d1 100644 --- a/app/Rules/UniqueIban.php +++ b/app/Rules/UniqueIban.php @@ -85,7 +85,7 @@ class UniqueIban implements ValidationRule * @param string $attribute * @param mixed $value * - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function passes($attribute, $value): bool { diff --git a/app/Rules/ValidJournals.php b/app/Rules/ValidJournals.php index a73ba98f2b..68368d21fd 100644 --- a/app/Rules/ValidJournals.php +++ b/app/Rules/ValidJournals.php @@ -33,7 +33,7 @@ use Illuminate\Contracts\Validation\ValidationRule; class ValidJournals implements ValidationRule { /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function validate(string $attribute, mixed $value, \Closure $fail): void { diff --git a/app/Rules/ValidRecurrenceRepetitionType.php b/app/Rules/ValidRecurrenceRepetitionType.php index 06a762cc15..d11eded70c 100644 --- a/app/Rules/ValidRecurrenceRepetitionType.php +++ b/app/Rules/ValidRecurrenceRepetitionType.php @@ -34,7 +34,7 @@ class ValidRecurrenceRepetitionType implements ValidationRule /** * Determine if the validation rule passes. * - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function validate(string $attribute, mixed $value, \Closure $fail): void { diff --git a/app/Rules/ValidRecurrenceRepetitionValue.php b/app/Rules/ValidRecurrenceRepetitionValue.php index 046d7bcf8d..e724c98540 100644 --- a/app/Rules/ValidRecurrenceRepetitionValue.php +++ b/app/Rules/ValidRecurrenceRepetitionValue.php @@ -33,7 +33,7 @@ use Illuminate\Contracts\Validation\ValidationRule; class ValidRecurrenceRepetitionValue implements ValidationRule { /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function validate(string $attribute, mixed $value, \Closure $fail): void { diff --git a/app/Services/Internal/Support/AccountServiceTrait.php b/app/Services/Internal/Support/AccountServiceTrait.php index 863ceb1224..d1963563e9 100644 --- a/app/Services/Internal/Support/AccountServiceTrait.php +++ b/app/Services/Internal/Support/AccountServiceTrait.php @@ -90,7 +90,7 @@ trait AccountServiceTrait /** * Update metadata for account. Depends on type which fields are valid. * - * @SuppressWarnings(PHPMD.NPathComplexity) + * @SuppressWarnings("PHPMD.NPathComplexity") * * TODO this method treats expense accounts and liabilities the same way (tries to save interest) */ diff --git a/app/Services/Internal/Support/CreditRecalculateService.php b/app/Services/Internal/Support/CreditRecalculateService.php index ed0dd16dc7..ead0369f79 100644 --- a/app/Services/Internal/Support/CreditRecalculateService.php +++ b/app/Services/Internal/Support/CreditRecalculateService.php @@ -236,9 +236,9 @@ class CreditRecalculateService /** * A complex and long method, but rarely used luckily. * - * @SuppressWarnings(PHPMD.ExcessiveMethodLength) - * @SuppressWarnings(PHPMD.NPathComplexity) - * @SuppressWarnings(PHPMD.CyclomaticComplexity) + * @SuppressWarnings("PHPMD.ExcessiveMethodLength") + * @SuppressWarnings("PHPMD.NPathComplexity") + * @SuppressWarnings("PHPMD.CyclomaticComplexity") */ private function processTransaction(Account $account, string $direction, Transaction $transaction, string $leftOfDebt): string { diff --git a/app/Services/Internal/Support/RecurringTransactionTrait.php b/app/Services/Internal/Support/RecurringTransactionTrait.php index 0b0c21a0d7..e772e57d85 100644 --- a/app/Services/Internal/Support/RecurringTransactionTrait.php +++ b/app/Services/Internal/Support/RecurringTransactionTrait.php @@ -91,7 +91,7 @@ trait RecurringTransactionTrait * * @throws FireflyException * - * @SuppressWarnings(PHPMD.NPathComplexity) + * @SuppressWarnings("PHPMD.NPathComplexity") */ protected function createTransactions(Recurrence $recurrence, array $transactions): void { diff --git a/app/Services/Internal/Update/BillUpdateService.php b/app/Services/Internal/Update/BillUpdateService.php index ddd23ddcd1..e2f6044b87 100644 --- a/app/Services/Internal/Update/BillUpdateService.php +++ b/app/Services/Internal/Update/BillUpdateService.php @@ -131,7 +131,7 @@ class BillUpdateService } /** - * @SuppressWarnings(PHPMD.NPathComplexity) + * @SuppressWarnings("PHPMD.NPathComplexity") */ private function updateBillProperties(Bill $bill, array $data): Bill { diff --git a/app/Services/Internal/Update/RecurrenceUpdateService.php b/app/Services/Internal/Update/RecurrenceUpdateService.php index 90575b664d..60598355f7 100644 --- a/app/Services/Internal/Update/RecurrenceUpdateService.php +++ b/app/Services/Internal/Update/RecurrenceUpdateService.php @@ -250,8 +250,8 @@ class RecurrenceUpdateService /** * It's a complex method but nothing surprising. * - * @SuppressWarnings(PHPMD.NPathComplexity) - * @SuppressWarnings(PHPMD.CyclomaticComplexity) + * @SuppressWarnings("PHPMD.NPathComplexity") + * @SuppressWarnings("PHPMD.CyclomaticComplexity") */ private function updateCombination(Recurrence $recurrence, array $combination): void { diff --git a/app/Services/Webhook/StandardWebhookSender.php b/app/Services/Webhook/StandardWebhookSender.php index 720d9bea8e..911fde75d9 100644 --- a/app/Services/Webhook/StandardWebhookSender.php +++ b/app/Services/Webhook/StandardWebhookSender.php @@ -49,7 +49,7 @@ class StandardWebhookSender implements WebhookSenderInterface /** * @throws GuzzleException * - * @SuppressWarnings(PHPMD.ExcessiveMethodLength) + * @SuppressWarnings("PHPMD.ExcessiveMethodLength") */ public function send(): void { diff --git a/app/Support/Authentication/RemoteUserGuard.php b/app/Support/Authentication/RemoteUserGuard.php index 61b6d4c88b..a6c787a8d8 100644 --- a/app/Support/Authentication/RemoteUserGuard.php +++ b/app/Support/Authentication/RemoteUserGuard.php @@ -137,7 +137,7 @@ class RemoteUserGuard implements Guard } /** - * @SuppressWarnings(PHPMD.ShortMethodName) + * @SuppressWarnings("PHPMD.ShortMethodName") */ public function id(): null|int|string { @@ -160,7 +160,7 @@ class RemoteUserGuard implements Guard /** * @throws FireflyException * - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function validate(array $credentials = []): bool { diff --git a/app/Support/Authentication/RemoteUserProvider.php b/app/Support/Authentication/RemoteUserProvider.php index 37364be807..1249e62c65 100644 --- a/app/Support/Authentication/RemoteUserProvider.php +++ b/app/Support/Authentication/RemoteUserProvider.php @@ -47,7 +47,7 @@ class RemoteUserProvider implements UserProvider /** * @throws FireflyException * - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function retrieveByCredentials(array $credentials): ?Authenticatable { @@ -95,7 +95,7 @@ class RemoteUserProvider implements UserProvider * * @throws FireflyException * - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function retrieveByToken($identifier, $token): ?Authenticatable { @@ -105,7 +105,7 @@ class RemoteUserProvider implements UserProvider } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") * * @param mixed $token * @@ -121,7 +121,7 @@ class RemoteUserProvider implements UserProvider /** * @throws FireflyException * - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function validateCredentials(Authenticatable $user, array $credentials): bool { diff --git a/app/Support/Binder/DynamicConfigKey.php b/app/Support/Binder/DynamicConfigKey.php index 3165192461..e2524c81ec 100644 --- a/app/Support/Binder/DynamicConfigKey.php +++ b/app/Support/Binder/DynamicConfigKey.php @@ -43,7 +43,7 @@ class DynamicConfigKey /** * @throws NotFoundHttpException * - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public static function routeBinder(string $value, Route $route): string { diff --git a/app/Support/Binder/EitherConfigKey.php b/app/Support/Binder/EitherConfigKey.php index 17394866a8..9df265c348 100644 --- a/app/Support/Binder/EitherConfigKey.php +++ b/app/Support/Binder/EitherConfigKey.php @@ -62,7 +62,7 @@ class EitherConfigKey /** * @throws NotFoundHttpException * - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public static function routeBinder(string $value, Route $route): string { diff --git a/app/Support/Export/ExportDataGenerator.php b/app/Support/Export/ExportDataGenerator.php index 7fb74e610b..6c6dc13ca9 100644 --- a/app/Support/Export/ExportDataGenerator.php +++ b/app/Support/Export/ExportDataGenerator.php @@ -713,7 +713,7 @@ class ExportDataGenerator } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function get(string $key, mixed $default = null): mixed { @@ -815,7 +815,7 @@ class ExportDataGenerator } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function has(mixed $key): mixed { diff --git a/app/Support/Http/Controllers/ModelInformation.php b/app/Support/Http/Controllers/ModelInformation.php index dfff32f1c4..5130ecb238 100644 --- a/app/Support/Http/Controllers/ModelInformation.php +++ b/app/Support/Http/Controllers/ModelInformation.php @@ -159,7 +159,7 @@ trait ModelInformation /** * @throws FireflyException * - * @SuppressWarnings(PHPMD.ExcessiveMethodLength) + * @SuppressWarnings("PHPMD.ExcessiveMethodLength") */ private function getTriggersForJournal(TransactionJournal $journal): array { diff --git a/app/Support/Models/BillDateCalculator.php b/app/Support/Models/BillDateCalculator.php index b8ff0c57cc..038f2b0364 100644 --- a/app/Support/Models/BillDateCalculator.php +++ b/app/Support/Models/BillDateCalculator.php @@ -37,7 +37,7 @@ class BillDateCalculator /** * Returns the dates a bill needs to be paid. * - * @SuppressWarnings(PHPMD.ExcessiveParameterList) + * @SuppressWarnings("PHPMD.ExcessiveParameterList") */ public function getPayDates(Carbon $earliest, Carbon $latest, Carbon $billStart, string $period, int $skip, ?Carbon $lastPaid): array { diff --git a/app/Support/Models/ReturnsIntegerIdTrait.php b/app/Support/Models/ReturnsIntegerIdTrait.php index 4b1fe3affc..804b9be384 100644 --- a/app/Support/Models/ReturnsIntegerIdTrait.php +++ b/app/Support/Models/ReturnsIntegerIdTrait.php @@ -34,7 +34,7 @@ trait ReturnsIntegerIdTrait /** * Get the ID * - * @SuppressWarnings(PHPMD.ShortMethodName) + * @SuppressWarnings("PHPMD.ShortMethodName") */ protected function id(): Attribute { diff --git a/app/Support/ParseDateString.php b/app/Support/ParseDateString.php index 9919f9fb0e..8cfee34c10 100644 --- a/app/Support/ParseDateString.php +++ b/app/Support/ParseDateString.php @@ -72,7 +72,7 @@ class ParseDateString /** * @throws FireflyException * - * @SuppressWarnings(PHPMD.NPathComplexity) + * @SuppressWarnings("PHPMD.NPathComplexity") */ public function parseDate(string $date): Carbon { diff --git a/app/Support/Repositories/Recurring/CalculateXOccurrencesSince.php b/app/Support/Repositories/Recurring/CalculateXOccurrencesSince.php index 45a246f9cc..7a1d34e39e 100644 --- a/app/Support/Repositories/Recurring/CalculateXOccurrencesSince.php +++ b/app/Support/Repositories/Recurring/CalculateXOccurrencesSince.php @@ -58,7 +58,7 @@ trait CalculateXOccurrencesSince * Calculates the number of monthly occurrences for a recurring transaction, starting at the date, until $count is * reached. It will skip over $skipMod -1 recurrences. * - * @SuppressWarnings(PHPMD.ExcessiveParameterList) + * @SuppressWarnings("PHPMD.ExcessiveParameterList") */ protected function getXMonthlyOccurrencesSince(Carbon $date, Carbon $afterDate, int $count, int $skipMod, string $moment): array { @@ -96,7 +96,7 @@ trait CalculateXOccurrencesSince * Calculates the number of NDOM occurrences for a recurring transaction, starting at the date, until $count is * reached. It will skip over $skipMod -1 recurrences. * - * @SuppressWarnings(PHPMD.ExcessiveParameterList) + * @SuppressWarnings("PHPMD.ExcessiveParameterList") */ protected function getXNDomOccurrencesSince(Carbon $date, Carbon $afterDate, int $count, int $skipMod, string $moment): array { @@ -130,7 +130,7 @@ trait CalculateXOccurrencesSince * Calculates the number of weekly occurrences for a recurring transaction, starting at the date, until $count is * reached. It will skip over $skipMod -1 recurrences. * - * @SuppressWarnings(PHPMD.ExcessiveParameterList) + * @SuppressWarnings("PHPMD.ExcessiveParameterList") */ protected function getXWeeklyOccurrencesSince(Carbon $date, Carbon $afterDate, int $count, int $skipMod, string $moment): array { @@ -169,7 +169,7 @@ trait CalculateXOccurrencesSince * Calculates the number of yearly occurrences for a recurring transaction, starting at the date, until $count is * reached. It will skip over $skipMod -1 recurrences. * - * @SuppressWarnings(PHPMD.ExcessiveParameterList) + * @SuppressWarnings("PHPMD.ExcessiveParameterList") */ protected function getXYearlyOccurrencesSince(Carbon $date, Carbon $afterDate, int $count, int $skipMod, string $moment): array { diff --git a/app/Support/Request/AppendsLocationData.php b/app/Support/Request/AppendsLocationData.php index 4329f865bb..149898f986 100644 --- a/app/Support/Request/AppendsLocationData.php +++ b/app/Support/Request/AppendsLocationData.php @@ -194,7 +194,7 @@ trait AppendsLocationData * * @return mixed * - * @SuppressWarnings(PHPMD.BooleanArgumentFlag) + * @SuppressWarnings("PHPMD.BooleanArgumentFlag") */ abstract public function boolean($key = null, $default = false); diff --git a/app/Support/Search/OperatorQuerySearch.php b/app/Support/Search/OperatorQuerySearch.php index 50dbff217a..69e55922b8 100644 --- a/app/Support/Search/OperatorQuerySearch.php +++ b/app/Support/Search/OperatorQuerySearch.php @@ -172,7 +172,7 @@ class OperatorQuerySearch implements SearchInterface /** * @throws FireflyException * - * @SuppressWarnings(PHPMD.CyclomaticComplexity) + * @SuppressWarnings("PHPMD.CyclomaticComplexity") */ private function handleSearchNode(Node $searchNode): void { @@ -255,8 +255,8 @@ class OperatorQuerySearch implements SearchInterface /** * @throws FireflyException * - * @SuppressWarnings(PHPMD.ExcessiveMethodLength) - * @SuppressWarnings(PHPMD.CyclomaticComplexity) + * @SuppressWarnings("PHPMD.ExcessiveMethodLength") + * @SuppressWarnings("PHPMD.CyclomaticComplexity") */ private function updateCollector(string $operator, string $value, bool $prohibited): bool { @@ -1931,8 +1931,8 @@ class OperatorQuerySearch implements SearchInterface * searchDirection: 1 = source (default), 2 = destination, 3 = both * stringPosition: 1 = start (default), 2 = end, 3 = contains, 4 = is * - * @SuppressWarnings(PHPMD.BooleanArgumentFlag) - * @SuppressWarnings(PHPMD.NPathComplexity) + * @SuppressWarnings("PHPMD.BooleanArgumentFlag") + * @SuppressWarnings("PHPMD.NPathComplexity") */ private function searchAccount(string $value, SearchDirection $searchDirection, StringPosition $stringPosition, bool $prohibited = false): void { @@ -2011,8 +2011,8 @@ class OperatorQuerySearch implements SearchInterface * searchDirection: 1 = source (default), 2 = destination, 3 = both * stringPosition: 1 = start (default), 2 = end, 3 = contains, 4 = is * - * @SuppressWarnings(PHPMD.BooleanArgumentFlag) - * @SuppressWarnings(PHPMD.NPathComplexity) + * @SuppressWarnings("PHPMD.BooleanArgumentFlag") + * @SuppressWarnings("PHPMD.NPathComplexity") */ private function searchAccountNr(string $value, SearchDirection $searchDirection, StringPosition $stringPosition, bool $prohibited = false): void { @@ -2146,7 +2146,7 @@ class OperatorQuerySearch implements SearchInterface /** * @throws FireflyException * - * @SuppressWarnings(PHPMD.BooleanArgumentFlag) + * @SuppressWarnings("PHPMD.BooleanArgumentFlag") */ private function setExactDateParams(array $range, bool $prohibited = false): void { @@ -2238,7 +2238,7 @@ class OperatorQuerySearch implements SearchInterface /** * @throws FireflyException * - * @SuppressWarnings(PHPMD.BooleanArgumentFlag) + * @SuppressWarnings("PHPMD.BooleanArgumentFlag") */ private function setDateBeforeParams(array $range, bool $prohibited = false): void { @@ -2294,7 +2294,7 @@ class OperatorQuerySearch implements SearchInterface /** * @throws FireflyException * - * @SuppressWarnings(PHPMD.BooleanArgumentFlag) + * @SuppressWarnings("PHPMD.BooleanArgumentFlag") */ private function setDateAfterParams(array $range, bool $prohibited = false): void { @@ -2350,7 +2350,7 @@ class OperatorQuerySearch implements SearchInterface /** * @throws FireflyException * - * @SuppressWarnings(PHPMD.BooleanArgumentFlag) + * @SuppressWarnings("PHPMD.BooleanArgumentFlag") */ private function setExactMetaDateParams(string $field, array $range, bool $prohibited = false): void { @@ -2445,7 +2445,7 @@ class OperatorQuerySearch implements SearchInterface /** * @throws FireflyException * - * @SuppressWarnings(PHPMD.BooleanArgumentFlag) + * @SuppressWarnings("PHPMD.BooleanArgumentFlag") */ private function setMetaDateBeforeParams(string $field, array $range, bool $prohibited = false): void { @@ -2501,7 +2501,7 @@ class OperatorQuerySearch implements SearchInterface /** * @throws FireflyException * - * @SuppressWarnings(PHPMD.BooleanArgumentFlag) + * @SuppressWarnings("PHPMD.BooleanArgumentFlag") */ private function setMetaDateAfterParams(string $field, array $range, bool $prohibited = false): void { @@ -2557,7 +2557,7 @@ class OperatorQuerySearch implements SearchInterface /** * @throws FireflyException * - * @SuppressWarnings(PHPMD.BooleanArgumentFlag) + * @SuppressWarnings("PHPMD.BooleanArgumentFlag") */ private function setExactObjectDateParams(string $field, array $range, bool $prohibited = false): void { @@ -2650,7 +2650,7 @@ class OperatorQuerySearch implements SearchInterface /** * @throws FireflyException * - * @SuppressWarnings(PHPMD.BooleanArgumentFlag) + * @SuppressWarnings("PHPMD.BooleanArgumentFlag") */ private function setObjectDateBeforeParams(string $field, array $range, bool $prohibited = false): void { @@ -2706,7 +2706,7 @@ class OperatorQuerySearch implements SearchInterface /** * @throws FireflyException * - * @SuppressWarnings(PHPMD.BooleanArgumentFlag) + * @SuppressWarnings("PHPMD.BooleanArgumentFlag") */ private function setObjectDateAfterParams(string $field, array $range, bool $prohibited = false): void { diff --git a/app/Support/Twig/General.php b/app/Support/Twig/General.php index 18b3d5bac7..31c57ca5e7 100644 --- a/app/Support/Twig/General.php +++ b/app/Support/Twig/General.php @@ -128,7 +128,7 @@ class General extends AbstractExtension /** * Show icon with attachment. * - * @SuppressWarnings(PHPMD.CyclomaticComplexity) + * @SuppressWarnings("PHPMD.CyclomaticComplexity") */ protected function mimeIcon(): TwigFilter { diff --git a/app/TransactionRules/Actions/AppendNotesToDescription.php b/app/TransactionRules/Actions/AppendNotesToDescription.php index 3402a6e8a8..5d8c4584fb 100644 --- a/app/TransactionRules/Actions/AppendNotesToDescription.php +++ b/app/TransactionRules/Actions/AppendNotesToDescription.php @@ -88,7 +88,7 @@ class AppendNotesToDescription implements ActionInterface } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function get(string $key, mixed $default = null): mixed { @@ -96,7 +96,7 @@ class AppendNotesToDescription implements ActionInterface } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function has(mixed $key): mixed { diff --git a/app/TransactionRules/Actions/ConvertToTransfer.php b/app/TransactionRules/Actions/ConvertToTransfer.php index 96ef106bdb..3a88c9bf39 100644 --- a/app/TransactionRules/Actions/ConvertToTransfer.php +++ b/app/TransactionRules/Actions/ConvertToTransfer.php @@ -52,8 +52,8 @@ class ConvertToTransfer implements ActionInterface } /** - * @SuppressWarnings(PHPMD.ExcessiveMethodLength) - * @SuppressWarnings(PHPMD.NPathComplexity) + * @SuppressWarnings("PHPMD.ExcessiveMethodLength") + * @SuppressWarnings("PHPMD.NPathComplexity") */ public function actOnArray(array $journal): bool { diff --git a/app/TransactionRules/Actions/MoveNotesToDescription.php b/app/TransactionRules/Actions/MoveNotesToDescription.php index d073cce16b..108d3ce499 100644 --- a/app/TransactionRules/Actions/MoveNotesToDescription.php +++ b/app/TransactionRules/Actions/MoveNotesToDescription.php @@ -89,7 +89,7 @@ class MoveNotesToDescription implements ActionInterface } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function get(string $key, mixed $default = null): mixed { @@ -97,7 +97,7 @@ class MoveNotesToDescription implements ActionInterface } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function has(mixed $key): mixed { diff --git a/app/TransactionRules/Expressions/ActionExpressionLanguageProvider.php b/app/TransactionRules/Expressions/ActionExpressionLanguageProvider.php index f871b3390a..a097377487 100644 --- a/app/TransactionRules/Expressions/ActionExpressionLanguageProvider.php +++ b/app/TransactionRules/Expressions/ActionExpressionLanguageProvider.php @@ -28,7 +28,7 @@ use Symfony\Component\ExpressionLanguage\ExpressionFunction; use Symfony\Component\ExpressionLanguage\ExpressionFunctionProviderInterface; /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ class ActionExpressionLanguageProvider implements ExpressionFunctionProviderInterface { diff --git a/app/Transformers/BillTransformer.php b/app/Transformers/BillTransformer.php index 9c66a1f8c6..760459a1dc 100644 --- a/app/Transformers/BillTransformer.php +++ b/app/Transformers/BillTransformer.php @@ -54,8 +54,8 @@ class BillTransformer extends AbstractTransformer /** * Transform the bill. * - * @SuppressWarnings(PHPMD.ExcessiveMethodLength) - * @SuppressWarnings(PHPMD.NPathComplexity) + * @SuppressWarnings("PHPMD.ExcessiveMethodLength") + * @SuppressWarnings("PHPMD.NPathComplexity") */ public function transform(Bill $bill): array { diff --git a/app/Transformers/TransactionGroupTransformer.php b/app/Transformers/TransactionGroupTransformer.php index 79fa691cdc..2683727631 100644 --- a/app/Transformers/TransactionGroupTransformer.php +++ b/app/Transformers/TransactionGroupTransformer.php @@ -109,7 +109,7 @@ class TransactionGroupTransformer extends AbstractTransformer } /** - * @SuppressWarnings(PHPMD.ExcessiveMethodLength) + * @SuppressWarnings("PHPMD.ExcessiveMethodLength") */ private function transformTransaction(array $transaction): array { @@ -323,7 +323,7 @@ class TransactionGroupTransformer extends AbstractTransformer /** * @throws FireflyException * - * @SuppressWarnings(PHPMD.ExcessiveMethodLength) + * @SuppressWarnings("PHPMD.ExcessiveMethodLength") */ private function transformJournal(TransactionJournal $journal): array { diff --git a/app/Transformers/V2/BillTransformer.php b/app/Transformers/V2/BillTransformer.php index 76f58e2e34..e52d17d38d 100644 --- a/app/Transformers/V2/BillTransformer.php +++ b/app/Transformers/V2/BillTransformer.php @@ -53,7 +53,7 @@ class BillTransformer extends AbstractTransformer /** * @throws FireflyException * - * @SuppressWarnings(PHPMD.ExcessiveMethodLength) + * @SuppressWarnings("PHPMD.ExcessiveMethodLength") */ public function collectMetaData(Collection $objects): Collection { diff --git a/app/Transformers/V2/TransactionGroupTransformer.php b/app/Transformers/V2/TransactionGroupTransformer.php index f244943b1f..a78c645d57 100644 --- a/app/Transformers/V2/TransactionGroupTransformer.php +++ b/app/Transformers/V2/TransactionGroupTransformer.php @@ -311,7 +311,7 @@ class TransactionGroupTransformer extends AbstractTransformer /** * @throws FireflyException * - * @SuppressWarnings(PHPMD.ExcessiveMethodLength) + * @SuppressWarnings("PHPMD.ExcessiveMethodLength") */ private function transformTransaction(array $transaction): array { @@ -505,7 +505,7 @@ class TransactionGroupTransformer extends AbstractTransformer /** * @throws FireflyException * - * @SuppressWarnings(PHPMD.ExcessiveMethodLength) + * @SuppressWarnings("PHPMD.ExcessiveMethodLength") */ private function transformJournal(TransactionJournal $journal): array { diff --git a/app/User.php b/app/User.php index c4fa2727b1..ffd1568ba0 100644 --- a/app/User.php +++ b/app/User.php @@ -36,6 +36,7 @@ use FireflyIII\Models\Category; use FireflyIII\Models\CurrencyExchangeRate; use FireflyIII\Models\GroupMembership; use FireflyIII\Models\ObjectGroup; +use FireflyIII\Models\PiggyBank; use FireflyIII\Models\Preference; use FireflyIII\Models\Recurrence; use FireflyIII\Models\Role; @@ -51,6 +52,7 @@ use FireflyIII\Models\UserRole; use FireflyIII\Models\Webhook; use FireflyIII\Notifications\Admin\UserRegistration; use FireflyIII\Notifications\Admin\VersionCheckResult; +use FireflyIII\Support\Models\ReturnsIntegerIdTrait; use Illuminate\Database\Eloquent\Relations\BelongsTo; use Illuminate\Database\Eloquent\Relations\BelongsToMany; use Illuminate\Database\Eloquent\Relations\HasMany; @@ -71,6 +73,7 @@ class User extends Authenticatable { use HasApiTokens; use Notifiable; + use ReturnsIntegerIdTrait; protected $casts = [ @@ -329,9 +332,12 @@ class User extends Authenticatable return $this->hasMany(ObjectGroup::class); } - public function piggyBanks(): void + /** + * Link to piggy banks. + */ + public function piggyBanks(): HasManyThrough { - throw new FireflyException('Method no longer supported.'); + return $this->hasManyThrough( PiggyBank::class, Account::class); } /** diff --git a/app/Validation/Account/DepositValidation.php b/app/Validation/Account/DepositValidation.php index 0f9336e2b9..9f7b67f39a 100644 --- a/app/Validation/Account/DepositValidation.php +++ b/app/Validation/Account/DepositValidation.php @@ -83,8 +83,8 @@ trait DepositValidation /** * Pretty complex unfortunately. * - * @SuppressWarnings(PHPMD.NPathComplexity) - * @SuppressWarnings(PHPMD.CyclomaticComplexity) + * @SuppressWarnings("PHPMD.NPathComplexity") + * @SuppressWarnings("PHPMD.CyclomaticComplexity") */ protected function validateDepositSource(array $array): bool { diff --git a/app/Validation/AccountValidator.php b/app/Validation/AccountValidator.php index a40bb2f79c..093fa479a9 100644 --- a/app/Validation/AccountValidator.php +++ b/app/Validation/AccountValidator.php @@ -251,9 +251,9 @@ class AccountValidator /** * It's a long and fairly complex method, but I don't mind. * - * @SuppressWarnings(PHPMD.CyclomaticComplexity) - * @SuppressWarnings(PHPMD.BooleanArgumentFlag) - * @SuppressWarnings(PHPMD.NPathComplexity) + * @SuppressWarnings("PHPMD.CyclomaticComplexity") + * @SuppressWarnings("PHPMD.BooleanArgumentFlag") + * @SuppressWarnings("PHPMD.NPathComplexity") */ protected function findExistingAccount(array $validTypes, array $data, bool $inverse = false): ?Account { diff --git a/app/Validation/AutoBudget/ValidatesAutoBudgetRequest.php b/app/Validation/AutoBudget/ValidatesAutoBudgetRequest.php index 493c0d03ac..581e0da61c 100644 --- a/app/Validation/AutoBudget/ValidatesAutoBudgetRequest.php +++ b/app/Validation/AutoBudget/ValidatesAutoBudgetRequest.php @@ -32,7 +32,7 @@ use Illuminate\Validation\Validator; trait ValidatesAutoBudgetRequest { /** - * @SuppressWarnings(PHPMD.NPathComplexity) + * @SuppressWarnings("PHPMD.NPathComplexity") */ protected function validateAutoBudgetAmount(Validator $validator): void { diff --git a/app/Validation/FireflyValidator.php b/app/Validation/FireflyValidator.php index 5b32fb8df0..9f3951168a 100644 --- a/app/Validation/FireflyValidator.php +++ b/app/Validation/FireflyValidator.php @@ -57,7 +57,7 @@ class FireflyValidator extends Validator * @throws InvalidCharactersException * @throws SecretKeyTooShortException * - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function validate2faCode($attribute, $value): bool { @@ -85,7 +85,7 @@ class FireflyValidator extends Validator * @param mixed $value * @param mixed $parameters * - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function validateBelongsToUser($attribute, $value, $parameters): bool { @@ -103,7 +103,7 @@ class FireflyValidator extends Validator * @param mixed $attribute * @param mixed $value * - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function validateBic($attribute, $value): bool { @@ -133,8 +133,8 @@ class FireflyValidator extends Validator } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) - * @SuppressWarnings(PHPMD.ExcessiveMethodLength) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") + * @SuppressWarnings("PHPMD.ExcessiveMethodLength") */ public function validateIban(mixed $attribute, mixed $value): bool { @@ -229,7 +229,7 @@ class FireflyValidator extends Validator * @param mixed $value * @param mixed $parameters * - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function validateLess($attribute, $value, $parameters): bool { @@ -244,7 +244,7 @@ class FireflyValidator extends Validator * @param mixed $value * @param mixed $parameters * - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function validateMore($attribute, $value, $parameters): bool { @@ -259,7 +259,7 @@ class FireflyValidator extends Validator * @param mixed $value * @param mixed $parameters * - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function validateMustExist($attribute, $value, $parameters): bool { @@ -443,7 +443,7 @@ class FireflyValidator extends Validator * @param mixed $attribute * @param mixed $value * - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function validateSecurePassword($attribute, $value): bool { @@ -466,7 +466,7 @@ class FireflyValidator extends Validator * @param mixed $value * @param mixed $parameters * - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function validateUniqueAccountForUser($attribute, $value, $parameters): bool { @@ -614,7 +614,7 @@ class FireflyValidator extends Validator * @param mixed $value * @param mixed $parameters * - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function validateUniqueAccountNumberForUser($attribute, $value, $parameters): bool { @@ -668,7 +668,7 @@ class FireflyValidator extends Validator } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function validateUniqueCurrencyCode(?string $attribute, ?string $value): bool { @@ -676,7 +676,7 @@ class FireflyValidator extends Validator } /** - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function validateUniqueCurrency(string $field, string $attribute, string $value): bool { @@ -698,7 +698,7 @@ class FireflyValidator extends Validator * @param mixed $parameters * @param mixed $something * - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function validateUniqueExistingWebhook($value, $parameters, $something): bool { @@ -753,7 +753,7 @@ class FireflyValidator extends Validator * @param mixed $value * @param mixed $parameters * - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function validateUniqueObjectForUser($attribute, $value, $parameters): bool { @@ -787,7 +787,7 @@ class FireflyValidator extends Validator * @param mixed $value * @param mixed $parameters * - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function validateUniqueObjectGroup($attribute, $value, $parameters): bool { @@ -809,7 +809,7 @@ class FireflyValidator extends Validator * @param mixed $value * @param mixed $parameters * - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function validateUniquePiggyBankForUser($attribute, $value, $parameters): bool { @@ -830,7 +830,7 @@ class FireflyValidator extends Validator * @param mixed $value * @param mixed $parameters * - * @SuppressWarnings(PHPMD.UnusedFormalParameter) + * @SuppressWarnings("PHPMD.UnusedFormalParameter") */ public function validateUniqueWebhook($value, $parameters): bool { diff --git a/app/Validation/RecurrenceValidation.php b/app/Validation/RecurrenceValidation.php index 81d74cae6a..ebb910bd0e 100644 --- a/app/Validation/RecurrenceValidation.php +++ b/app/Validation/RecurrenceValidation.php @@ -301,7 +301,7 @@ trait RecurrenceValidation } /** - * @SuppressWarnings(PHPMD.NPathComplexity) + * @SuppressWarnings("PHPMD.NPathComplexity") */ protected function validateTransactionId(Recurrence $recurrence, Validator $validator): void { diff --git a/app/Validation/TransactionValidation.php b/app/Validation/TransactionValidation.php index 4e5f390861..edeb5907b9 100644 --- a/app/Validation/TransactionValidation.php +++ b/app/Validation/TransactionValidation.php @@ -34,6 +34,7 @@ use FireflyIII\Models\TransactionGroup; use FireflyIII\Models\TransactionJournal; use FireflyIII\Models\TransactionType; use FireflyIII\Models\UserGroup; +use FireflyIII\Repositories\Account\AccountRepository; use FireflyIII\Repositories\Account\AccountRepositoryInterface; use FireflyIII\User; use Illuminate\Validation\Validator; @@ -61,7 +62,7 @@ trait TransactionValidation app('log')->debug(sprintf('Going to loop %d transaction(s)', count($transactions))); /** - * @var null|int $index + * @var string|int $index * @var array $transaction */ foreach ($transactions as $index => $transaction) { @@ -91,7 +92,7 @@ trait TransactionValidation } /** - * @SuppressWarnings(PHPMD.NPathComplexity) + * @SuppressWarnings("PHPMD.NPathComplexity") */ protected function validateSingleAccount(Validator $validator, int $index, string $transactionType, array $transaction): void { @@ -155,7 +156,7 @@ trait TransactionValidation } /** - * @SuppressWarnings(PHPMD.ExcessiveParameterList) + * @SuppressWarnings("PHPMD.ExcessiveParameterList") */ protected function sanityCheckReconciliation(Validator $validator, string $transactionType, int $index, array $source, array $destination): void { @@ -184,8 +185,8 @@ trait TransactionValidation /** * TODO describe this method. * - * @SuppressWarnings(PHPMD.ExcessiveParameterList) - * @SuppressWarnings(PHPMD.NPathComplexity) + * @SuppressWarnings("PHPMD.ExcessiveParameterList") + * @SuppressWarnings("PHPMD.NPathComplexity") */ private function sanityCheckForeignCurrency( Validator $validator, @@ -222,7 +223,7 @@ trait TransactionValidation return; } - /** @var AccountRepositoryInterface $accountRepository */ + /** @var AccountRepository $accountRepository */ $accountRepository = app(AccountRepositoryInterface::class); $defaultCurrency = app('amount')->getDefaultCurrency(); $sourceCurrency = $accountRepository->getAccountCurrency($source) ?? $defaultCurrency; @@ -347,7 +348,7 @@ trait TransactionValidation $transactions = $this->getTransactionsArray($validator); /** - * @var null|int $index + * @var string|int $index * @var array $transaction */ foreach ($transactions as $index => $transaction) { 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 d1580d5f4d..fd5c08dcd6 100644 --- a/database/migrations/2016_06_16_000000_create_support_tables.php +++ b/database/migrations/2016_06_16_000000_create_support_tables.php @@ -55,7 +55,7 @@ class CreateSupportTables extends Migration /** * Run the migrations. * - * @SuppressWarnings(PHPMD.ShortMethodName) + * @SuppressWarnings("PHPMD.ShortMethodName") */ public function up(): void { 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 5002e26302..98df97b1a8 100644 --- a/database/migrations/2016_06_16_000001_create_users_table.php +++ b/database/migrations/2016_06_16_000001_create_users_table.php @@ -46,7 +46,7 @@ class CreateUsersTable extends Migration /** * Run the migrations. * - * @SuppressWarnings(PHPMD.ShortMethodName) + * @SuppressWarnings("PHPMD.ShortMethodName") */ public function up(): void { 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 80000d31f4..f2d76e5c53 100644 --- a/database/migrations/2016_06_16_000002_create_main_tables.php +++ b/database/migrations/2016_06_16_000002_create_main_tables.php @@ -73,7 +73,7 @@ class CreateMainTables extends Migration /** * Run the migrations. * - * @SuppressWarnings(PHPMD.ShortMethodName) + * @SuppressWarnings("PHPMD.ShortMethodName") */ public function up(): void { @@ -368,7 +368,7 @@ class CreateMainTables extends Migration } /** - * @SuppressWarnings(PHPMD.ExcessiveMethodLength) + * @SuppressWarnings("PHPMD.ExcessiveMethodLength") */ private function createRuleTables(): void { @@ -507,8 +507,8 @@ class CreateMainTables extends Migration } /** - * @SuppressWarnings(PHPMD.ExcessiveMethodLength) - * @SuppressWarnings(PHPMD.NPathComplexity) + * @SuppressWarnings("PHPMD.ExcessiveMethodLength") + * @SuppressWarnings("PHPMD.NPathComplexity") */ private function createTransactionTables(): void { diff --git a/database/migrations/2016_08_25_091522_changes_for_3101.php b/database/migrations/2016_08_25_091522_changes_for_3101.php index 2743c4fad5..ce78bf0dfe 100644 --- a/database/migrations/2016_08_25_091522_changes_for_3101.php +++ b/database/migrations/2016_08_25_091522_changes_for_3101.php @@ -38,7 +38,7 @@ class ChangesFor3101 extends Migration /** * Run the migrations. * - * @SuppressWarnings(PHPMD.ShortMethodName) + * @SuppressWarnings("PHPMD.ShortMethodName") */ public function up(): void {} } diff --git a/database/migrations/2016_09_12_121359_fix_nullables.php b/database/migrations/2016_09_12_121359_fix_nullables.php index 3f8d858915..8862bdd1cb 100644 --- a/database/migrations/2016_09_12_121359_fix_nullables.php +++ b/database/migrations/2016_09_12_121359_fix_nullables.php @@ -43,7 +43,7 @@ class FixNullables extends Migration /** * Run the migrations. * - * @SuppressWarnings(PHPMD.ShortMethodName) + * @SuppressWarnings("PHPMD.ShortMethodName") */ public function up(): void { 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 d5af9d1602..bd86fcb336 100644 --- a/database/migrations/2016_10_09_150037_expand_transactions_table.php +++ b/database/migrations/2016_10_09_150037_expand_transactions_table.php @@ -55,7 +55,7 @@ class ExpandTransactionsTable extends Migration /** * Run the migrations. * - * @SuppressWarnings(PHPMD.ShortMethodName) + * @SuppressWarnings("PHPMD.ShortMethodName") */ public function up(): void { 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 417bc70afe..42002388fe 100644 --- a/database/migrations/2016_10_22_075804_changes_for_v410.php +++ b/database/migrations/2016_10_22_075804_changes_for_v410.php @@ -43,7 +43,7 @@ class ChangesForV410 extends Migration /** * Run the migrations. * - * @SuppressWarnings(PHPMD.ShortMethodName) + * @SuppressWarnings("PHPMD.ShortMethodName") */ public function up(): void { 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 54a8c898c3..21d2d34410 100644 --- a/database/migrations/2016_11_24_210552_changes_for_v420.php +++ b/database/migrations/2016_11_24_210552_changes_for_v420.php @@ -55,7 +55,7 @@ class ChangesForV420 extends Migration /** * Run the migrations. * - * @SuppressWarnings(PHPMD.ShortMethodName) + * @SuppressWarnings("PHPMD.ShortMethodName") */ public function up(): void { 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 dce36a973d..a4ee8dd1e6 100644 --- a/database/migrations/2016_12_22_150431_changes_for_v430.php +++ b/database/migrations/2016_12_22_150431_changes_for_v430.php @@ -43,7 +43,7 @@ class ChangesForV430 extends Migration /** * Run the migrations. * - * @SuppressWarnings(PHPMD.ShortMethodName) + * @SuppressWarnings("PHPMD.ShortMethodName") */ public function up(): void { 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 030c5bd3c0..1243266588 100644 --- a/database/migrations/2016_12_28_203205_changes_for_v431.php +++ b/database/migrations/2016_12_28_203205_changes_for_v431.php @@ -112,7 +112,7 @@ class ChangesForV431 extends Migration /** * Run the migrations. * - * @SuppressWarnings(PHPMD.ShortMethodName) + * @SuppressWarnings("PHPMD.ShortMethodName") */ public function up(): void { 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 0326323a4b..0232cc68fe 100644 --- a/database/migrations/2017_04_13_163623_changes_for_v440.php +++ b/database/migrations/2017_04_13_163623_changes_for_v440.php @@ -61,7 +61,7 @@ class ChangesForV440 extends Migration /** * Run the migrations. * - * @SuppressWarnings(PHPMD.ShortMethodName) + * @SuppressWarnings("PHPMD.ShortMethodName") */ public function up(): void { 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 92a3ffa889..8151926cc0 100644 --- a/database/migrations/2017_06_02_105232_changes_for_v450.php +++ b/database/migrations/2017_06_02_105232_changes_for_v450.php @@ -84,7 +84,7 @@ class ChangesForV450 extends Migration /** * Run the migrations. * - * @SuppressWarnings(PHPMD.ShortMethodName) + * @SuppressWarnings("PHPMD.ShortMethodName") */ public function up(): void { 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 0c3794cda4..ded20b566b 100644 --- a/database/migrations/2017_08_20_062014_changes_for_v470.php +++ b/database/migrations/2017_08_20_062014_changes_for_v470.php @@ -45,7 +45,7 @@ class ChangesForV470 extends Migration /** * Run the migrations. * - * @SuppressWarnings(PHPMD.ShortMethodName) + * @SuppressWarnings("PHPMD.ShortMethodName") */ public function up(): void { 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 2beafc4f7e..40c2aa3df0 100644 --- a/database/migrations/2017_11_04_170844_changes_for_v470a.php +++ b/database/migrations/2017_11_04_170844_changes_for_v470a.php @@ -56,7 +56,7 @@ class ChangesForV470a extends Migration /** * Run the migrations. * - * @SuppressWarnings(PHPMD.ShortMethodName) + * @SuppressWarnings("PHPMD.ShortMethodName") */ public function up(): void { 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 cb70131bf2..e38a8854a5 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 @@ -45,7 +45,7 @@ class CreateOauthAuthCodesTable extends Migration /** * Run the migrations. * - * @SuppressWarnings(PHPMD.ShortMethodName) + * @SuppressWarnings("PHPMD.ShortMethodName") */ public function up(): void { 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 cc41b105bd..16c80af3b4 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 @@ -45,7 +45,7 @@ class CreateOauthAccessTokensTable extends Migration /** * Run the migrations. * - * @SuppressWarnings(PHPMD.ShortMethodName) + * @SuppressWarnings("PHPMD.ShortMethodName") */ public function up(): void { 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 3b91d74abc..4e0b8b69a7 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 @@ -45,7 +45,7 @@ class CreateOauthRefreshTokensTable extends Migration /** * Run the migrations. * - * @SuppressWarnings(PHPMD.ShortMethodName) + * @SuppressWarnings("PHPMD.ShortMethodName") */ public function up(): void { 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 7c75715cb0..f4f2d8b02b 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 @@ -45,7 +45,7 @@ class CreateOauthClientsTable extends Migration /** * Run the migrations. * - * @SuppressWarnings(PHPMD.ShortMethodName) + * @SuppressWarnings("PHPMD.ShortMethodName") */ public function up(): void { 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 2bdfa66075..d9fae563d7 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 @@ -45,7 +45,7 @@ class CreateOauthPersonalAccessClientsTable extends Migration /** * Run the migrations. * - * @SuppressWarnings(PHPMD.ShortMethodName) + * @SuppressWarnings("PHPMD.ShortMethodName") */ public function up(): void { 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 94c7ae935b..e1548f08a7 100644 --- a/database/migrations/2018_03_19_141348_changes_for_v472.php +++ b/database/migrations/2018_03_19_141348_changes_for_v472.php @@ -70,7 +70,7 @@ class ChangesForV472 extends Migration /** * Run the migrations. * - * @SuppressWarnings(PHPMD.ShortMethodName) + * @SuppressWarnings("PHPMD.ShortMethodName") */ public function up(): void { 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 41f6dc57b5..7c18633029 100644 --- a/database/migrations/2018_04_07_210913_changes_for_v473.php +++ b/database/migrations/2018_04_07_210913_changes_for_v473.php @@ -75,7 +75,7 @@ class ChangesForV473 extends Migration /** * Run the migrations. * - * @SuppressWarnings(PHPMD.ShortMethodName) + * @SuppressWarnings("PHPMD.ShortMethodName") */ public function up(): void { diff --git a/database/migrations/2018_04_29_174524_changes_for_v474.php b/database/migrations/2018_04_29_174524_changes_for_v474.php index d78013e8ec..a3415a224d 100644 --- a/database/migrations/2018_04_29_174524_changes_for_v474.php +++ b/database/migrations/2018_04_29_174524_changes_for_v474.php @@ -39,7 +39,7 @@ class ChangesForV474 extends Migration /** * Run the migrations. * - * @SuppressWarnings(PHPMD.ShortMethodName) + * @SuppressWarnings("PHPMD.ShortMethodName") */ public function up(): void {} } 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 7a73769960..d0b33bd88b 100644 --- a/database/migrations/2018_06_08_200526_changes_for_v475.php +++ b/database/migrations/2018_06_08_200526_changes_for_v475.php @@ -48,8 +48,8 @@ class ChangesForV475 extends Migration /** * Run the migrations. * - * @SuppressWarnings(PHPMD.ShortMethodName) - * @SuppressWarnings(PHPMD.ExcessiveMethodLength) + * @SuppressWarnings("PHPMD.ShortMethodName") + * @SuppressWarnings("PHPMD.ExcessiveMethodLength") */ public function up(): void { 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 654081e97a..4a5c82b061 100644 --- a/database/migrations/2018_09_05_195147_changes_for_v477.php +++ b/database/migrations/2018_09_05_195147_changes_for_v477.php @@ -61,7 +61,7 @@ class ChangesForV477 extends Migration /** * Run the migrations. * - * @SuppressWarnings(PHPMD.ShortMethodName) + * @SuppressWarnings("PHPMD.ShortMethodName") */ public function up(): void { 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 fc33d5c261..4c74ffcb7b 100644 --- a/database/migrations/2018_11_06_172532_changes_for_v479.php +++ b/database/migrations/2018_11_06_172532_changes_for_v479.php @@ -56,7 +56,7 @@ class ChangesForV479 extends Migration /** * Run the migrations. * - * @SuppressWarnings(PHPMD.ShortMethodName) + * @SuppressWarnings("PHPMD.ShortMethodName") */ public function up(): void { 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 26d1423964..c95b150d30 100644 --- a/database/migrations/2019_01_28_193833_changes_for_v4710.php +++ b/database/migrations/2019_01_28_193833_changes_for_v4710.php @@ -46,7 +46,7 @@ class ChangesForV4710 extends Migration /** * Run the migrations. * - * @SuppressWarnings(PHPMD.ShortMethodName) + * @SuppressWarnings("PHPMD.ShortMethodName") */ public function up(): void { 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 b9b4d8a7de..25d18d1a8e 100644 --- a/database/migrations/2019_02_05_055516_changes_for_v4711.php +++ b/database/migrations/2019_02_05_055516_changes_for_v4711.php @@ -42,7 +42,7 @@ class ChangesForV4711 extends Migration /** * Run the migrations. * - * @SuppressWarnings(PHPMD.ShortMethodName) + * @SuppressWarnings("PHPMD.ShortMethodName") */ public function up(): void { 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 70f6f113c7..1afb19649e 100644 --- a/database/migrations/2019_02_11_170529_changes_for_v4712.php +++ b/database/migrations/2019_02_11_170529_changes_for_v4712.php @@ -42,7 +42,7 @@ class ChangesForV4712 extends Migration /** * Run the migrations. * - * @SuppressWarnings(PHPMD.ShortMethodName) + * @SuppressWarnings("PHPMD.ShortMethodName") */ public function up(): void { 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 f8063bbeda..acbc06bad7 100644 --- a/database/migrations/2019_03_11_223700_fix_ldap_configuration.php +++ b/database/migrations/2019_03_11_223700_fix_ldap_configuration.php @@ -57,7 +57,7 @@ class FixLdapConfiguration extends Migration /** * Run the migrations. * - * @SuppressWarnings(PHPMD.ShortMethodName) + * @SuppressWarnings("PHPMD.ShortMethodName") */ public function up(): void { 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 1228f38c05..ec5d0fa0e4 100644 --- a/database/migrations/2019_03_22_183214_changes_for_v480.php +++ b/database/migrations/2019_03_22_183214_changes_for_v480.php @@ -113,7 +113,7 @@ class ChangesForV480 extends Migration /** * Run the migrations. * - * @SuppressWarnings(PHPMD.ShortMethodName) + * @SuppressWarnings("PHPMD.ShortMethodName") */ public function up(): void { 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 b9125a2de6..3c707b5218 100644 --- a/database/migrations/2019_12_28_191351_make_locations_table.php +++ b/database/migrations/2019_12_28_191351_make_locations_table.php @@ -45,7 +45,7 @@ class MakeLocationsTable extends Migration /** * Run the migrations. * - * @SuppressWarnings(PHPMD.ShortMethodName) + * @SuppressWarnings("PHPMD.ShortMethodName") */ public function up(): void { 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 66169c6ad3..3787265066 100644 --- a/database/migrations/2020_03_13_201950_changes_for_v520.php +++ b/database/migrations/2020_03_13_201950_changes_for_v520.php @@ -45,7 +45,7 @@ class ChangesForV520 extends Migration /** * Run the migrations. * - * @SuppressWarnings(PHPMD.ShortMethodName) + * @SuppressWarnings("PHPMD.ShortMethodName") */ public function up(): void { 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 42c3d53903..5992d70862 100644 --- a/database/migrations/2020_06_07_063612_changes_for_v530.php +++ b/database/migrations/2020_06_07_063612_changes_for_v530.php @@ -45,7 +45,7 @@ class ChangesForV530 extends Migration /** * Run the migrations. * - * @SuppressWarnings(PHPMD.ShortMethodName) + * @SuppressWarnings("PHPMD.ShortMethodName") */ public function up(): void { 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 88e7e94050..87dc66bf79 100644 --- a/database/migrations/2020_06_30_202620_changes_for_v530a.php +++ b/database/migrations/2020_06_30_202620_changes_for_v530a.php @@ -57,7 +57,7 @@ class ChangesForV530a extends Migration /** * Run the migrations. * - * @SuppressWarnings(PHPMD.ShortMethodName) + * @SuppressWarnings("PHPMD.ShortMethodName") */ public function up(): void { 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 abc76317a2..c161a6ef1c 100644 --- a/database/migrations/2020_07_24_162820_changes_for_v540.php +++ b/database/migrations/2020_07_24_162820_changes_for_v540.php @@ -98,7 +98,7 @@ class ChangesForV540 extends Migration /** * Run the migrations. * - * @SuppressWarnings(PHPMD.ShortMethodName) + * @SuppressWarnings("PHPMD.ShortMethodName") */ public function up(): void { 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 b210f4c25c..48876943fa 100644 --- a/database/migrations/2020_11_12_070604_changes_for_v550.php +++ b/database/migrations/2020_11_12_070604_changes_for_v550.php @@ -122,9 +122,9 @@ class ChangesForV550 extends Migration /** * Run the migrations. * - * @SuppressWarnings(PHPMD.ShortMethodName) - * @SuppressWarnings(PHPMD.ExcessiveMethodLength) - * @SuppressWarnings(PHPMD.NPathComplexity) + * @SuppressWarnings("PHPMD.ShortMethodName") + * @SuppressWarnings("PHPMD.ExcessiveMethodLength") + * @SuppressWarnings("PHPMD.NPathComplexity") */ public function up(): void { 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 689f91b760..0eac9b1dc1 100644 --- a/database/migrations/2021_03_12_061213_changes_for_v550b2.php +++ b/database/migrations/2021_03_12_061213_changes_for_v550b2.php @@ -60,7 +60,7 @@ class ChangesForV550b2 extends Migration /** * Run the migrations. * - * @SuppressWarnings(PHPMD.ShortMethodName) + * @SuppressWarnings("PHPMD.ShortMethodName") */ public function up(): void { 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 354d04f90c..1ac8a7bc7b 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 @@ -52,7 +52,7 @@ class AddLdapColumnsToUsersTable extends Migration /** * Run the migrations. * - * @SuppressWarnings(PHPMD.ShortMethodName) + * @SuppressWarnings("PHPMD.ShortMethodName") */ public function up(): void { 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 888bbab2fa..5393100696 100644 --- a/database/migrations/2021_05_13_053836_extend_currency_info.php +++ b/database/migrations/2021_05_13_053836_extend_currency_info.php @@ -40,7 +40,7 @@ class ExtendCurrencyInfo extends Migration /** * Run the migrations. * - * @SuppressWarnings(PHPMD.ShortMethodName) + * @SuppressWarnings("PHPMD.ShortMethodName") */ public function up(): void { diff --git a/database/migrations/2021_07_05_193044_drop_tele_table.php b/database/migrations/2021_07_05_193044_drop_tele_table.php index 105cf474d6..115ba6258f 100644 --- a/database/migrations/2021_07_05_193044_drop_tele_table.php +++ b/database/migrations/2021_07_05_193044_drop_tele_table.php @@ -41,7 +41,7 @@ class DropTeleTable extends Migration /** * Run the migrations. * - * @SuppressWarnings(PHPMD.ShortMethodName) + * @SuppressWarnings("PHPMD.ShortMethodName") */ public function up(): void { diff --git a/database/migrations/2021_08_28_073733_user_groups.php b/database/migrations/2021_08_28_073733_user_groups.php index 1111ddc671..6f6f7c0404 100644 --- a/database/migrations/2021_08_28_073733_user_groups.php +++ b/database/migrations/2021_08_28_073733_user_groups.php @@ -106,8 +106,8 @@ class UserGroups extends Migration /** * Run the migrations. * - * @SuppressWarnings(PHPMD.ShortMethodName) - * @SuppressWarnings(PHPMD.ExcessiveMethodLength) + * @SuppressWarnings("PHPMD.ShortMethodName") + * @SuppressWarnings("PHPMD.ExcessiveMethodLength") */ public function up(): void { 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 c551a9fe43..2ea68f9fa3 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 @@ -43,7 +43,7 @@ class CreateLocalPersonalAccessTokensTable extends Migration /** * Run the migrations. * - * @SuppressWarnings(PHPMD.ShortMethodName) + * @SuppressWarnings("PHPMD.ShortMethodName") */ public function up(): void { 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 4129828f03..a4d2ba8b96 100644 --- a/database/migrations/2022_08_21_104626_add_user_groups.php +++ b/database/migrations/2022_08_21_104626_add_user_groups.php @@ -31,7 +31,7 @@ return new class () extends Migration { /** * Run the migrations. * - * @SuppressWarnings(PHPMD.ShortMethodName) + * @SuppressWarnings("PHPMD.ShortMethodName") */ public function up(): void { 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 fb0bd31fe0..fdd7c43c03 100644 --- a/database/migrations/2022_09_18_123911_create_notifications_table.php +++ b/database/migrations/2022_09_18_123911_create_notifications_table.php @@ -31,7 +31,7 @@ return new class () extends Migration { /** * Run the migrations. * - * @SuppressWarnings(PHPMD.ShortMethodName) + * @SuppressWarnings("PHPMD.ShortMethodName") */ public function up(): void { diff --git a/database/migrations/2022_10_01_074908_invited_users.php b/database/migrations/2022_10_01_074908_invited_users.php index eed71a1bc7..638c823980 100644 --- a/database/migrations/2022_10_01_074908_invited_users.php +++ b/database/migrations/2022_10_01_074908_invited_users.php @@ -31,7 +31,7 @@ return new class () extends Migration { /** * Run the migrations. * - * @SuppressWarnings(PHPMD.ShortMethodName) + * @SuppressWarnings("PHPMD.ShortMethodName") */ public function up(): void { 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 a0d18c182a..cc3d3afaf8 100644 --- a/database/migrations/2022_10_01_210238_audit_log_entries.php +++ b/database/migrations/2022_10_01_210238_audit_log_entries.php @@ -31,7 +31,7 @@ return new class () extends Migration { /** * Run the migrations. * - * @SuppressWarnings(PHPMD.ShortMethodName) + * @SuppressWarnings("PHPMD.ShortMethodName") */ public function up(): void { 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 54349c2db6..47ca29b340 100644 --- a/database/migrations/2023_08_11_192521_upgrade_og_table.php +++ b/database/migrations/2023_08_11_192521_upgrade_og_table.php @@ -31,7 +31,7 @@ return new class () extends Migration { /** * Run the migrations. * - * @SuppressWarnings(PHPMD.ShortMethodName) + * @SuppressWarnings("PHPMD.ShortMethodName") */ public function up(): void { 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 65107b5610..75d1b94c45 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 @@ -29,7 +29,7 @@ use Illuminate\Support\Facades\Schema; return new class () extends Migration { /** - * @SuppressWarnings(PHPMD.ShortMethodName) + * @SuppressWarnings("PHPMD.ShortMethodName") * Run the migrations. */ public function up(): void diff --git a/database/seeders/ExchangeRateSeeder.php b/database/seeders/ExchangeRateSeeder.php index 06e1a20335..d61a5f6101 100644 --- a/database/seeders/ExchangeRateSeeder.php +++ b/database/seeders/ExchangeRateSeeder.php @@ -87,7 +87,7 @@ class ExchangeRateSeeder extends Seeder } /** - * @SuppressWarnings(PHPMD.ExcessiveParameterList) + * @SuppressWarnings("PHPMD.ExcessiveParameterList") */ private function addRate(User $user, TransactionCurrency $from, TransactionCurrency $to, string $date, float $rate): void {