jbtronics / StuRa-Finanzsoftware

ecs.php

Spacing +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
 use Symplify\EasyCodingStandard\ValueObject\Set\SetList;
 use Symplify\CodingStandard\Fixer\LineLength\LineLengthFixer;
 
-return static function (ContainerConfigurator $containerConfigurator): void {
+return static function(ContainerConfigurator $containerConfigurator): void {
     $parameters = $containerConfigurator->parameters();
     $parameters->set(Option::SETS, [
         SetList::CLEAN_CODE,

config/preload.php

Spacing +2 added lines, -2 removed lines

@@ -1,5 +1,5 @@
 <?php
 
-if (file_exists(dirname(__DIR__).'/var/cache/prod/App_KernelProdContainer.preload.php')) {
-    require dirname(__DIR__).'/var/cache/prod/App_KernelProdContainer.preload.php';
+if (file_exists(dirname(__DIR__) . '/var/cache/prod/App_KernelProdContainer.preload.php')) {
+    require dirname(__DIR__) . '/var/cache/prod/App_KernelProdContainer.preload.php';
 }

src/EventSubscriber/Fail2BanSubscriber.php

Spacing +1 added lines, -1 removed lines

@@ -49,7 +49,7 @@
     {
         $ipAddress = $this->request->getCurrentRequest()
             ->getClientIp();
-        $this->logger->error('Authentication failed for IP: '.$ipAddress);
+        $this->logger->error('Authentication failed for IP: ' . $ipAddress);
     }
 
     public static function getSubscribedEvents(): array

src/Controller/ExportController.php

Spacing +4 added lines, -4 removed lines

@@ -99,7 +99,7 @@ 
                 //Download as file
                 if (1 === count($xml_files)) {
                     $xml_string = array_values($xml_files)[0];
-                    $filename = 'export_'.date('Y-m-d_H-i-s').'.xml';
+                    $filename = 'export_' . date('Y-m-d_H-i-s') . '.xml';
                     $response = $this->getDownloadResponse($xml_string, $filename);
                 } else {
                     $zip = new ZipArchive();
@@ -113,7 +113,7 @@ 
                         $response = new BinaryFileResponse($file_path);
                         $response->deleteFileAfterSend();
                         $response->setPrivate();
-                        $response->setContentDisposition(ResponseHeaderBag::DISPOSITION_ATTACHMENT, 'export_'.date('Y-m-d_H-i-s').'.zip');
+                        $response->setContentDisposition(ResponseHeaderBag::DISPOSITION_ATTACHMENT, 'export_' . date('Y-m-d_H-i-s') . '.zip');
                     } else {
                         throw new RuntimeException('Could not create a ZIP Archive.');
                     }
@@ -130,7 +130,7 @@ 
                 //Show error if auto mode is not possible
                 $this->addFlash('danger',
                                 $this->translator->trans('sepa_export.error.department_missing_account')
-                                .': '.$exception->getWrongDepartment()->getName());
+                                .': ' . $exception->getWrongDepartment()->getName());
             }
         }
 
@@ -145,7 +145,7 @@ 
         $response = new Response();
         $response->headers->set('Cache-Control', 'private');
         $response->headers->set('Content-type', $mime_type);
-        $response->headers->set('Content-Disposition', 'attachment; filename="'.$filename.'";');
+        $response->headers->set('Content-Disposition', 'attachment; filename="' . $filename . '";');
         $response->headers->set('Content-length', strlen($content));
         $response->setContent($content);
 

src/Controller/UserSettingsController.php

Spacing +2 added lines, -2 removed lines

@@ -148,7 +148,7 @@ 
             throw new RuntimeException('This controller only works only for Part-DB User objects!');
         }
 
-        if ($this->isCsrfTokenValid('regenerate_backup_codes'.$user->getId(), $request->request->get('_token'))) {
+        if ($this->isCsrfTokenValid('regenerate_backup_codes' . $user->getId(), $request->request->get('_token'))) {
             $backupCodeManager->regenerateBackupCodes($user);
             $entityManager->flush();
             $this->addFlash('success', 'user.settings.2fa.backup_codes.regenerated');
@@ -175,7 +175,7 @@ 
             throw new RuntimeException('This controller only works only for Part-DB User objects!');
         }
 
-        if ($this->isCsrfTokenValid('devices_reset'.$user->getId(), $request->request->get('_token'))) {
+        if ($this->isCsrfTokenValid('devices_reset' . $user->getId(), $request->request->get('_token'))) {
             $user->invalidateTrustedDevices();
             $entityManager->flush();
             $this->addFlash('success', 'tfa_trustedDevice.invalidate.success');

src/Controller/Admin/DashboardController.php

Spacing +5 added lines, -5 removed lines

@@ -68,7 +68,7 @@ 
         //$menuItem->setQueryParameter('referrer', $referrer);
 
         foreach ($filters as $filter => $value) {
-            $menuItem->setQueryParameter('filters['.$filter.']', $value);
+            $menuItem->setQueryParameter('filters[' . $filter . ']', $value);
         }
 
         $menuItem->setQueryParameter('crudAction', 'index');
@@ -159,8 +159,8 @@ 
         yield MenuItem::linkToCrud('user.labelp', 'fas fa-user', User::class)
             ->setPermission('ROLE_READ_USER');
 
-        $version = $this->app_version.'-'.$this->gitVersionInfo->getGitCommitHash() ?? '';
-        yield MenuItem::section('Version '.$version, 'fas fa-info');
+        $version = $this->app_version . '-' . $this->gitVersionInfo->getGitCommitHash() ?? '';
+        yield MenuItem::section('Version ' . $version, 'fas fa-info');
         yield MenuItem::linktoRoute('dashboard.menu.homepage', 'fas fa-home', 'homepage');
         yield MenuItem::linkToUrl('dashboard.menu.stura', 'fab fa-rebel', 'https://www.stura.uni-jena.de/');
         yield MenuItem::linkToUrl('dashboard.menu.github', 'fab fa-github', 'https://github.com/jbtronics/StuRa-Finanzsoftware');
@@ -175,8 +175,8 @@ 
             ->displayUserName(true)
             ->addMenuItems([
                 MenuItem::linktoRoute('user.settings.title', 'fas fa-user-cog', 'user_settings'),
-                MenuItem::linktoRoute(Languages::getName('de', 'de').' (DE)', '', 'admin_dashboard.de'),
-                MenuItem::linktoRoute(Languages::getName('en', 'en').' (EN)', '', 'admin_dashboard.en'),
+                MenuItem::linktoRoute(Languages::getName('de', 'de') . ' (DE)', '', 'admin_dashboard.de'),
+                MenuItem::linktoRoute(Languages::getName('en', 'en') . ' (EN)', '', 'admin_dashboard.en'),
             ]);
     }
 

src/Controller/Admin/DepartmentCrudController.php

Spacing +1 added lines, -1 removed lines

@@ -78,7 +78,7 @@
     {
         $choices = [];
         foreach (Department::ALLOWED_TYPES as $type) {
-            $choices['department.type.'.$type] = $type;
+            $choices['department.type.' . $type] = $type;
         }
 
         return [

src/Controller/Admin/PaymentOrderCrudController.php

Spacing +6 added lines, -6 removed lines

@@ -179,32 +179,32 @@ 
         ]);
 
         $emailAction = Action::new('sendEmail', 'payment_order.action.email', 'fas fa-envelope')
-            ->linkToUrl(function (PaymentOrder $paymentOrder) {
+            ->linkToUrl(function(PaymentOrder $paymentOrder) {
                 return $this->mailToGenerator->generateContactMailLink($paymentOrder);
             })
             ->setCssClass('text-dark');
 
         //Hide action if no contact emails are associated with department
-        $emailAction->displayIf(function (PaymentOrder $paymentOrder) {
+        $emailAction->displayIf(function(PaymentOrder $paymentOrder) {
             return null !== $this->mailToGenerator->generateContactMailLink($paymentOrder);
         });
 
         $hhv_action = Action::new('contactHHV', 'payment_order.action.contact_hhv', 'fas fa-comment-dots')
-            ->linkToUrl(function (PaymentOrder $paymentOrder) {
+            ->linkToUrl(function(PaymentOrder $paymentOrder) {
                 return $this->mailToGenerator->getHHVMailLink($paymentOrder);
             })
             ->setCssClass('mr-2 text-dark');
 
         $resend_confirmation_action = Action::new('resendConfirmation', 'payment_order.action.resend_confirmation', 'fas fa-redo')
             ->linkToCrudAction('resendConfirmationEmail')
-            ->displayIf(function (PaymentOrder $paymentOrder) {
+            ->displayIf(function(PaymentOrder $paymentOrder) {
                 return $this->isGranted('ROLE_EDIT_PAYMENT_ORDERS') && (null === $paymentOrder->getConfirm2Timestamp() || null === $paymentOrder->getConfirm1Timestamp());
             })
             ->setCssClass('mr-2 text-dark');
 
         $mathematically_correct_action = Action::new('mathematicallyCorrect', 'payment_order.action.mathematically_correct', 'fas fa-check')
             ->linkToCrudAction('checkMathematicallyCorrect')
-            ->displayIf(function (PaymentOrder $paymentOrder) {
+            ->displayIf(function(PaymentOrder $paymentOrder) {
                 return $this->isGranted('ROLE_PO_MATHEMATICALLY')
                     && $paymentOrder->isConfirmed()
                     && !$paymentOrder->isMathematicallyCorrect();
@@ -213,7 +213,7 @@ 
 
         $factually_correct_action = Action::new('factuallyCorrect', 'payment_order.action.factually_correct', 'fas fa-check')
             ->linkToCrudAction('checkFactuallyCorrect')
-            ->displayIf(function (PaymentOrder $paymentOrder) {
+            ->displayIf(function(PaymentOrder $paymentOrder) {
                 return $this->isGranted('ROLE_PO_FACTUALLY')
                     && $paymentOrder->isConfirmed()
                     && !$paymentOrder->isFactuallyCorrect()

src/Entity/User.php

Spacing +2 added lines, -2 removed lines

@@ -311,7 +311,7 @@ 
             return $this->getFirstName();
         }
 
-        return $this->getFirstName().' '.$this->getLastName();
+        return $this->getFirstName() . ' ' . $this->getLastName();
     }
 
     /**
@@ -460,6 +460,6 @@ 
 
     public function __toString(): string
     {
-        return $this->getFullName().' ('.$this->username.')';
+        return $this->getFullName() . ' (' . $this->username . ')';
     }
 }