jbtronics / StuRa-Finanzsoftware

src/Controller/Admin/DashboardController.php

Spacing +8 added lines, -8 removed lines

@@ -75,10 +75,10 @@ 
         foreach ($filters as $filter => $value) {
             if (is_array($value)) {
                 foreach ($value as $subfilter => $subvalue) {
-                    $menuItem->setQueryParameter('filters['.$filter.']['.$subfilter.']', $subvalue);
+                    $menuItem->setQueryParameter('filters[' . $filter . '][' . $subfilter . ']', $subvalue);
                 }
             } else {
-                $menuItem->setQueryParameter('filters['.$filter.']', $value);
+                $menuItem->setQueryParameter('filters[' . $filter . ']', $value);
             }
         }
 
@@ -92,11 +92,11 @@ 
         $actions = parent::configureActions();
 
         $showLog = Action::new('showLog', 'action.show_logs', 'fas fa-binoculars')
-            ->displayIf(function (DBElementInterface $entity) {
+            ->displayIf(function(DBElementInterface $entity) {
                 return $this->isGranted('ROLE_VIEW_AUDITS');
             })
             ->setCssClass('btn btn-secondary')
-            ->linkToRoute('dh_auditor_show_entity_history', function (DBElementInterface $entity) {
+            ->linkToRoute('dh_auditor_show_entity_history', function(DBElementInterface $entity) {
                 return [
                     'entity' => str_replace('\\', '-', get_class($entity)),
                     'id' => $entity->getId(),
@@ -242,8 +242,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.audits', 'fas fa-binoculars', 'dh_auditor_list_audits')
             ->setPermission('ROLE_VIEW_AUDITS');
         yield MenuItem::linktoRoute('dashboard.menu.homepage', 'fas fa-home', 'homepage');
@@ -261,8 +261,8 @@ 
             ->addMenuItems([
                 MenuItem::linktoRoute('user.settings.title', 'fas fa-user-cog', 'user_settings'),
                 //It is important to use LinkToUrl here. LinkToCrud will put the route name into a param, but does not change the prefix
-                MenuItem::linkToUrl(Languages::getName('de', 'de').' (DE)', '', '/de/admin'),
-                MenuItem::linkToUrl(Languages::getName('en', 'en').' (EN)', '', '/en/admin'),
+                MenuItem::linkToUrl(Languages::getName('de', 'de') . ' (DE)', '', '/de/admin'),
+                MenuItem::linkToUrl(Languages::getName('en', 'en') . ' (EN)', '', '/en/admin'),
             ]);
     }
 

src/Controller/Admin/DepartmentCrudController.php

Spacing +2 added lines, -2 removed lines

@@ -90,7 +90,7 @@ 
         ]);
 
         $generateTokenAction = Action::new('generateSkipToken', 'department.action.generate_skip_token', 'fas fa-award')
-            ->displayIf(function (Department $paymentOrder) {
+            ->displayIf(function(Department $paymentOrder) {
                 return $this->isGranted('ROLE_EDIT_ORGANISATIONS');
             })
             ->setCssClass('btn btn-secondary')
@@ -114,7 +114,7 @@ 
     {
         $choices = [];
         foreach (Department::ALLOWED_TYPES as $type) {
-            $choices['department.type.'.$type] = $type;
+            $choices['department.type.' . $type] = $type;
         }
 
         return [

src/Controller/PaymentOrderController.php

Spacing +1 added lines, -1 removed lines

@@ -177,7 +177,7 @@
      */
     public function confirmation(?PaymentOrder $paymentOrder, Request $request, EntityManagerInterface $em): Response
     {
-        if($paymentOrder === null) {
+        if ($paymentOrder === null) {
             $this->addFlash('error', 'payment_order.can_not_be_found');
             return $this->redirectToRoute('homepage');
         }

src/Controller/Admin/PaymentOrderCrudController.php

Spacing +12 added lines, -12 removed lines

@@ -119,7 +119,7 @@ 
 
             if ($payment_order->getDepartment() !== null && $payment_order->getDepartment()->getBankAccount() !== null) {
                 //First folder for each bank account
-                $prefix = $payment_order->getDepartment()->getBankAccount()->getName() . ' [' . $payment_order->getDepartment()->getBankAccount()->getIban() .']' . '/';
+                $prefix = $payment_order->getDepartment()->getBankAccount()->getName() . ' [' . $payment_order->getDepartment()->getBankAccount()->getIban() . ']' . '/';
 
                 //A sub folder for each department
                 $prefix .= $payment_order->getDepartment()->getName() . '/';
@@ -131,7 +131,7 @@ 
             $project_name = mb_ereg_replace("([^\w\s\d\-_~,;\[\]\(\).])", '', $project_name);
             $project_name = mb_ereg_replace("([\.]{2,})", '', $project_name);
             //Format: "ZA000001 Project Name.pdf"
-            $filename = $prefix.$payment_order->getIDString().' '.$project_name.'.'.$extension;
+            $filename = $prefix . $payment_order->getIDString() . ' ' . $project_name . '.' . $extension;
 
             $data[$filename] = $path;
 
@@ -148,7 +148,7 @@ 
 
         return ZIPBinaryFileResponseFacade::createZIPResponseFromFiles(
             $data,
-            'Belege_'.date('Y-m-d_H-i-s').'.zip');
+            'Belege_' . date('Y-m-d_H-i-s') . '.zip');
     }
 
     public function configureCrud(Crud $crud): Crud
@@ -317,32 +317,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('btn btn-secondary 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('btn btn-secondary 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') && !$paymentOrder->isConfirmed();
             })
             ->setCssClass('btn btn-secondary 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();
@@ -351,7 +351,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()
@@ -361,12 +361,12 @@ 
 
         $manual_confirmation = Action::new('manual_confirmation', 'payment_order.action.manual_confirmation', 'fas fa-exclamation-triangle')
             ->setCssClass('btn btn-secondary')
-            ->linkToRoute('payment_order_manual_confirm', function (PaymentOrder $paymentOrder) {
+            ->linkToRoute('payment_order_manual_confirm', function(PaymentOrder $paymentOrder) {
                 return [
                     'id' => $paymentOrder->getId(),
                 ];
             })
-            ->displayIf(function (PaymentOrder $paymentOrder) {
+            ->displayIf(function(PaymentOrder $paymentOrder) {
                 return $this->isGranted('ROLE_MANUAL_CONFIRMATION')
                     && !$paymentOrder->isConfirmed();
             });
@@ -379,7 +379,7 @@ 
 
         $actions->disable(Crud::PAGE_NEW);
 
-        if(!$this->isGranted('ROLE_EDIT_PAYMENT_ORDERS')) {
+        if (!$this->isGranted('ROLE_EDIT_PAYMENT_ORDERS')) {
             $actions->disable('batchDelete');
         }
 

src/Services/Upload/SEPAExportFileNamer.php

Spacing +3 added lines, -3 removed lines

@@ -58,12 +58,12 @@
         //TODO: Improve this
         $filename = sprintf("XML%06d", $object->getId());
 
-        $filename .= '_'.date('ymd-His');
+        $filename .= '_' . date('ymd-His');
 
-        $filename .= '_'.bin2hex(random_bytes(5));
+        $filename .= '_' . bin2hex(random_bytes(5));
 
         //Add original extension
-        $filename .= '.'.$originalExtension;
+        $filename .= '.' . $originalExtension;
 
         return $filename;
     }

src/Helpers/SEPAXML/SEPAXMLParser.php

Spacing +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
         return [
             'msg_id' => (string) ($group_header->MsgId),
             'number_of_payments' => (int) ($group_header->NbOfTxs),
-            'total_sum' => (int) str_replace('.','',$group_header->CtrlSum),
+            'total_sum' => (int) str_replace('.', '', $group_header->CtrlSum),
             'initiator_iban' => (string) $payment_info->DbtrAcct->Id->IBAN,
             'initiator_bic' => (string) $payment_info->DbtrAgt->FinInstnId->BIC,
         ];

src/Helpers/SEPAXML/SEPAXMLExportResult.php

Spacing +2 added lines, -2 removed lines

@@ -44,7 +44,7 @@ 
         }
 
         //Perform some checks on the sepa exports
-        foreach($sepa_exports as $sepa_export) {
+        foreach ($sepa_exports as $sepa_export) {
             if (!$sepa_export instanceof SEPAExport) {
                 throw new \InvalidArgumentException('$sepa_exports must all be of type SEPAExport!');
             }
@@ -151,7 +151,7 @@ 
      */
     public function persistSEPAExports(EntityManagerInterface $entityManager): void
     {
-        foreach($this->sepa_exports as $sepa_export) {
+        foreach ($this->sepa_exports as $sepa_export) {
             $entityManager->persist($sepa_export);
         }
     }

src/Entity/SEPAExport.php

Spacing +1 added lines, -1 removed lines

@@ -277,7 +277,7 @@
     public function setXmlFile(\Symfony\Component\HttpFoundation\File\File $xml_file): SEPAExport
     {
         //Check if $xml_file was changed
-        if($this->xml_file === null || $xml_file->getPathname() !== $this->xml_file->getPathname()) {
+        if ($this->xml_file === null || $xml_file->getPathname() !== $this->xml_file->getPathname()) {
             $this->xml_file = $xml_file;
             $this->updateFromFile();
         } else {