jbtronics / StuRa-Finanzsoftware

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 {

src/Controller/ExportController.php

Spacing +3 added lines, -3 removed lines

@@ -84,7 +84,7 @@ 
 
             try {
                 //Call function depending on the selected mode
-                switch($data['mode']) {
+                switch ($data['mode']) {
                     case 'auto':
                             $result = $this->sepaExporter->exportAuto($payment_orders);
                         break;
@@ -113,13 +113,13 @@ 
                 $this->entityManager->flush();
 
                 //Return the download
-                return $result->getDownloadResponse('export_'.date('Y-m-d_H-i-s'));
+                return $result->getDownloadResponse('export_' . date('Y-m-d_H-i-s'));
 
             } catch (SEPAExportAutoModeNotPossible $exception) {
                 //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());
             }
         }
 

public/index.php

Spacing +2 added lines, -2 removed lines

@@ -2,8 +2,8 @@
 
 use App\Kernel;
 
-require_once dirname(__DIR__).'/vendor/autoload_runtime.php';
+require_once dirname(__DIR__) . '/vendor/autoload_runtime.php';
 
-return function (array $context) {
+return function(array $context) {
     return new Kernel($context['APP_ENV'], (bool) $context['APP_DEBUG']);
 };

src/Services/SEPAExport/SEPAExportGroupAndSplitHelper.php

Spacing +5 added lines, -5 removed lines

@@ -83,7 +83,7 @@ 
             }
 
             //If no array object is existing yet, create the array
-            if(!isset($grouped[$bank_account])) {
+            if (!isset($grouped[$bank_account])) {
                 $grouped[$bank_account] = [];
             }
 
@@ -128,7 +128,7 @@ 
         $groups_exceeding_limit = $tmp;
         /** @var PaymentOrder[][] $output */
         $output = [];
-        while(!empty($groups_exceeding_limit)) {
+        while (!empty($groups_exceeding_limit)) {
             foreach ($groups_exceeding_limit as $key => $group) {
                 /** @var PaymentOrder[] $group */
                 //If the group does not exceed the limit, then remove it from the bad list and put it to output array
@@ -146,7 +146,7 @@ 
                     $part = array_slice($group, 0, $n + 1);
                     if ($this->calculateSumAmountOfPaymentOrders($part) > $limit_max_amount) {
                         //If our group contains just a single element which exceed the limit, then throw an exception, as we can not split it further.
-                        if(count($part) === 1) {
+                        if (count($part) === 1) {
                             throw new SinglePaymentOrderExceedsLimit($part[0], $limit_max_amount);
                         }
 
@@ -156,7 +156,7 @@ 
                 }
 
                 //Split group into our two subgroups of which at least one is below the limit
-                $a = array_slice($group, 0 , $split_index);
+                $a = array_slice($group, 0, $split_index);
                 $b = array_slice($group, $split_index);
 
                 //Remove the old group from list and add the new split groups
@@ -193,7 +193,7 @@ 
      */
     public function sortPaymentOrderArrayByAmount(array $payment_orders, bool $ascending = true): array
     {
-        usort($payment_orders, function (PaymentOrder $a, PaymentOrder $b) {
+        usort($payment_orders, function(PaymentOrder $a, PaymentOrder $b) {
             //When amounts are the same, use the ID to get a consistent sorting
             if ($a->getAmount() === $b->getAmount()) {
                 return $a->getId() <=> $b->getAmount();

src/Services/Upload/ExtendedFileSystemStorage.php

Spacing +5 added lines, -5 removed lines

@@ -52,9 +52,9 @@ 
         }
 
         $uploadDir = $this->convertWindowsDirectorySeparator($mapping->getUploadDir($obj));
-        $uploadDir = empty($uploadDir) ? '' : $uploadDir.'/';
+        $uploadDir = empty($uploadDir) ? '' : $uploadDir . '/';
 
-        $tmp = \sprintf('%s/%s', $mapping->getUriPrefix(), $uploadDir.$name);
+        $tmp = \sprintf('%s/%s', $mapping->getUriPrefix(), $uploadDir . $name);
 
         if (null !== $tmp && $obj instanceof PaymentOrder) {
             if ('printed_form' === $fieldName || 'printed_form_file' === $fieldName) {
@@ -75,7 +75,7 @@ 
 
     protected function doUpload(PropertyMapping $mapping, UploadedFile $file, ?string $dir, string $name): ?File
     {
-        $uploadDir = $mapping->getUploadDestination().\DIRECTORY_SEPARATOR.$dir;
+        $uploadDir = $mapping->getUploadDestination() . \DIRECTORY_SEPARATOR . $dir;
 
         return $file->move($uploadDir, $name);
     }
@@ -89,13 +89,13 @@ 
 
     protected function doResolvePath(PropertyMapping $mapping, ?string $dir, string $name, ?bool $relative = false): string
     {
-        $path = !empty($dir) ? $dir.\DIRECTORY_SEPARATOR.$name : $name;
+        $path = !empty($dir) ? $dir . \DIRECTORY_SEPARATOR . $name : $name;
 
         if ($relative) {
             return $path;
         }
 
-        return $mapping->getUploadDestination().\DIRECTORY_SEPARATOR.$path;
+        return $mapping->getUploadDestination() . \DIRECTORY_SEPARATOR . $path;
     }
 
     private function convertWindowsDirectorySeparator(string $string): string

src/Admin/Filter/ULIDFilter.php

Spacing +1 added lines, -1 removed lines

@@ -59,7 +59,7 @@
         $value = $filterDataDto->getValue();
 
         //Only apply filter if provided string is a valid ULID, otherwise we run into an exception
-        if(Ulid::isValid($value)) {
+        if (Ulid::isValid($value)) {
             $queryBuilder->andWhere(sprintf('%s.%s = :%s', $alias, $property, $parameterName))
                 ->setParameter($parameterName, Ulid::fromString($value), 'ulid');
         } else { //Otherwise just ensure no results are returned using an impossible SQL statement

src/Services/UserSystem/EnforceTFARedirectHelper.php

Spacing +1 added lines, -1 removed lines

@@ -85,7 +85,7 @@
 
         foreach ($roles as $role) {
             foreach ($risky_roles as $risky_role) {
-                if (preg_match('/'. $risky_role . '/', $role)) {
+                if (preg_match('/' . $risky_role . '/', $role)) {
                     return true;
                 }
             }