chore(quality): phpcbf auto-fix + @author email typo

lib/AppInfo/Application.php

@@ -8,7 +8,7 @@
  * @category AppInfo
  * @package  OCA\Procest\AppInfo
  *
- * @author    Conduction Development Team <dev@conductio.nl>
+ * @author    Conduction Development Team <info@conduction.nl>
  * @copyright 2024 Conduction B.V.
  * @license   EUPL-1.2 https://joinup.ec.europa.eu/collection/eupl/eupl-text-eupl-12
  *

lib/BackgroundJob/AppointmentReminderJob.php

@@ -21,8 +21,9 @@
         private LoggerInterface $logger,
     ) {
         parent::__construct($time);
-        $this->setInterval(86400); // Daily.
-    }
+        $this->setInterval(86400);
+        // Daily.
+    }//end __construct()
 
     protected function run($argument): void
     {
@@ -50,19 +51,22 @@
             );
 
             foreach (($result['objects'] ?? []) as $apt) {
-                $data = is_object($apt) ? $apt->jsonSerialize() : $apt;
+                $data    = is_object($apt) ? $apt->jsonSerialize() : $apt;
                 $aptDate = substr($data['dateTime'] ?? '', 0, 10);
 
                 if ($aptDate === $tomorrow && empty($data['reminderSent'])) {
                     $data['reminderSent'] = true;
                     $objectService->saveObject((int) $register, (int) $schema, $data);
-                    $this->logger->info('Procest: Reminder sent for appointment', [
-                        'appointmentId' => $data['uuid'] ?? $data['id'] ?? '',
-                    ]);
+                    $this->logger->info(
+                            'Procest: Reminder sent for appointment',
+                            [
+                                'appointmentId' => $data['uuid'] ?? $data['id'] ?? '',
+                            ]
+                            );
                 }
             }
         } catch (\Exception $e) {
             $this->logger->error('Procest: Reminder job error: '.$e->getMessage());
-        }
-    }
-}
+        }//end try
+    }//end run()
+}//end class

lib/BackgroundJob/BerichtenboxReadStatusJob.php

@@ -13,17 +13,18 @@
 {
     public function __construct(
         ITimeFactory $time,
         private BerichtenboxService $berichtenboxService,
         private LoggerInterface $logger,
     ) {
         parent::__construct($time);
-        $this->setInterval(86400); // Daily.
-    }
+        $this->setInterval(86400);
+        // Daily.
+    }//end __construct()
 
     protected function run($argument): void
     {
         $this->logger->info('Procest: Running Berichtenbox read status poll');
         // The actual polling happens in BerichtenboxService::pollReadStatus
         // This job would iterate unread messages and poll each one.
-    }
-}
+    }//end run()
+}//end class

lib/BackgroundJob/ShareMaintenanceJob.php

@@ -8,7 +8,7 @@
  * @category BackgroundJob
  * @package  OCA\Procest\BackgroundJob
  *
- * @author    Conduction Development Team <dev@conductio.nl>
+ * @author    Conduction Development Team <info@conduction.nl>
  * @copyright 2024 Conduction B.V.
  * @license   EUPL-1.2 https://joinup.ec.europa.eu/collection/eupl/eupl-text-eupl-12
  *
@@ -47,7 +47,7 @@ class ShareMaintenanceJob extends TimedJob
      * @param ITimeFactory       $time            The time factory
      * @param SettingsService    $settingsService The settings service
      * @param IAppManager        $appManager      The app manager
-     * @param ContainerInterface $container        The DI container
+     * @param ContainerInterface $container       The DI container
      * @param LoggerInterface    $logger          The logger
      *
      * @return void

lib/Controller/AcController.php

@@ -10,7 +10,7 @@
  * @category Controller
  * @package  OCA\Procest\Controller
  *
- * @author    Conduction Development Team <dev@conductio.nl>
+ * @author    Conduction Development Team <info@conduction.nl>
  * @copyright 2024 Conduction B.V.
  * @license   EUPL-1.2 https://joinup.ec.europa.eu/collection/eupl/eupl-text-eupl-12
  *

lib/Controller/AiController.php

@@ -10,7 +10,7 @@
  * @category Controller
  * @package  OCA\Procest\Controller
  *
- * @author    Conduction Development Team <dev@conductio.nl>
+ * @author    Conduction Development Team <info@conduction.nl>
  * @copyright 2024 Conduction B.V.
  * @license   EUPL-1.2 https://joinup.ec.europa.eu/collection/eupl/eupl-text-eupl-12
  *
@@ -47,12 +47,12 @@
     /**
      * Constructor for AiController.
      *
-     * @param string           $appName         The application name
-     * @param IRequest         $request         The request object
-     * @param AiService        $aiService       The AI service
-     * @param SettingsService  $settingsService The settings service
-     * @param IUserSession     $userSession     The user session
-     * @param LoggerInterface  $logger          The logger interface
+     * @param string          $appName         The application name
+     * @param IRequest        $request         The request object
+     * @param AiService       $aiService       The AI service
+     * @param SettingsService $settingsService The settings service
+     * @param IUserSession    $userSession     The user session
+     * @param LoggerInterface $logger          The logger interface
      *
      * @return void
      */
@@ -62,7 +62,7 @@
         private AiService $aiService,
         private SettingsService $settingsService,
         private IUserSession $userSession,
         private LoggerInterface $logger,
     ) {
         parent::__construct($appName, $request);
     }//end __construct()
@@ -277,11 +277,13 @@
             'offset' => (int) $this->request->getParam('offset', '0'),
         ];
 
-        return new JSONResponse([
-            'success' => true,
-            'filters' => array_filter($filters),
-            'message' => 'Audit trail query — implement with OpenRegister object listing',
-        ]);
+        return new JSONResponse(
+                [
+                    'success' => true,
+                    'filters' => array_filter($filters),
+                    'message' => 'Audit trail query — implement with OpenRegister object listing',
+                ]
+                );
     }//end auditIndex()
 
     /**

lib/Controller/AppointmentController.php

@@ -17,17 +17,21 @@ public function __construct(
         private AppointmentService $appointmentService,
     ) {
         parent::__construct(appName: Application::APP_ID, request: $request);
-    }
+    }//end __construct()
 
-    /** @NoAdminRequired */
+    /**
+     * @NoAdminRequired
+     */
     public function index(): JSONResponse
     {
         $caseId       = $this->request->getParam('caseId');
         $appointments = $this->appointmentService->getAppointmentsForCase($caseId ?? '');
         return new JSONResponse(['success' => true, 'appointments' => $appointments]);
-    }
+    }//end index()
 
-    /** @NoAdminRequired */
+    /**
+     * @NoAdminRequired
+     */
     public function create(): JSONResponse
     {
         $caseId = $this->request->getParam('caseId');
@@ -48,23 +52,29 @@ public function create(): JSONResponse
 
         $result = $this->appointmentService->bookAppointment($caseId, $data);
         return new JSONResponse(['success' => true, 'appointment' => $result]);
-    }
+    }//end create()
 
-    /** @NoAdminRequired */
+    /**
+     * @NoAdminRequired
+     */
     public function cancel(string $appointmentId): JSONResponse
     {
         $result = $this->appointmentService->cancelAppointment($appointmentId);
         return new JSONResponse(['success' => true, 'appointment' => $result]);
-    }
+    }//end cancel()
 
-    /** @NoAdminRequired */
+    /**
+     * @NoAdminRequired
+     */
     public function noShow(string $appointmentId): JSONResponse
     {
         $result = $this->appointmentService->markNoShow($appointmentId);
         return new JSONResponse(['success' => true, 'appointment' => $result]);
-    }
+    }//end noShow()
 
-    /** @NoAdminRequired */
+    /**
+     * @NoAdminRequired
+     */
     public function timeslots(): JSONResponse
     {
         $productId  = $this->request->getParam('productId', '');
@@ -73,5 +83,5 @@ public function timeslots(): JSONResponse
 
         $slots = $this->appointmentService->getTimeslots($productId, $locationId, $date);
         return new JSONResponse(['success' => true, 'timeslots' => $slots]);
-    }
-}
+    }//end timeslots()
+}//end class

lib/Controller/BerichtenboxController.php

@@ -17,45 +17,56 @@ public function __construct(
         private BerichtenboxService $berichtenboxService,
     ) {
         parent::__construct(appName: Application::APP_ID, request: $request);
-    }
+    }//end __construct()
 
-    /** @NoAdminRequired */
+    /**
+     * @NoAdminRequired
+     */
     public function send(): JSONResponse
     {
-        $caseId           = $this->request->getParam('caseId');
-        $bsn              = $this->request->getParam('bsn', '');
-        $subject          = $this->request->getParam('subject', '');
-        $body             = $this->request->getParam('body', '');
-        $typeCode         = $this->request->getParam('berichtTypeCode', '');
+        $caseId   = $this->request->getParam('caseId');
+        $bsn      = $this->request->getParam('bsn', '');
+        $subject  = $this->request->getParam('subject', '');
+        $body     = $this->request->getParam('body', '');
+        $typeCode = $this->request->getParam('berichtTypeCode', '');
         $attachmentFileId = $this->request->getParam('attachmentFileId');
 
         if (empty($caseId) === true) {
             return new JSONResponse(['success' => false, 'error' => 'caseId is required'], 400);
         }
 
         $result = $this->berichtenboxService->sendMessage(
-            $caseId, $bsn, $subject, $body, $typeCode, $attachmentFileId
+            $caseId,
+                $bsn,
+                $subject,
+                $body,
+                $typeCode,
+                $attachmentFileId
         );
 
         if (isset($result['error']) === true) {
             return new JSONResponse(['success' => false, 'error' => $result['error']], 400);
         }
 
         return new JSONResponse(['success' => true, 'message' => $result]);
-    }
+    }//end send()
 
-    /** @NoAdminRequired */
+    /**
+     * @NoAdminRequired
+     */
     public function messages(): JSONResponse
     {
         $caseId   = $this->request->getParam('caseId', '');
         $messages = $this->berichtenboxService->getMessagesForCase($caseId);
         return new JSONResponse(['success' => true, 'messages' => $messages]);
-    }
+    }//end messages()
 
-    /** @NoAdminRequired */
+    /**
+     * @NoAdminRequired
+     */
     public function poll(string $messageId): JSONResponse
     {
         $result = $this->berichtenboxService->pollReadStatus($messageId);
         return new JSONResponse(['success' => true, 'message' => $result]);
-    }
-}
+    }//end poll()
+}//end class

lib/Controller/BrcController.php

@@ -10,7 +10,7 @@
  * @category Controller
  * @package  OCA\Procest\Controller
  *
- * @author    Conduction Development Team <dev@conductio.nl>
+ * @author    Conduction Development Team <info@conduction.nl>
  * @copyright 2024 Conduction B.V.
  * @license   EUPL-1.2 https://joinup.ec.europa.eu/collection/eupl/eupl-text-eupl-12
  *

lib/Controller/CaseDefinitionController.php

@@ -9,7 +9,7 @@
  * @category Controller
  * @package  OCA\Procest\Controller
  *
- * @author    Conduction Development Team <dev@conductio.nl>
+ * @author    Conduction Development Team <info@conduction.nl>
  * @copyright 2024 Conduction B.V.
  * @license   EUPL-1.2 https://joinup.ec.europa.eu/collection/eupl/eupl-text-eupl-12
  *
@@ -41,11 +41,11 @@
     /**
      * Constructor.
      *
-     * @param string                        $appName       The app name.
-     * @param IRequest                      $request       The request object.
-     * @param CaseDefinitionExportService   $exportService The export service.
-     * @param CaseDefinitionImportService   $importService The import service.
-     * @param LoggerInterface               $logger        The logger.
+     * @param string                      $appName       The app name.
+     * @param IRequest                    $request       The request object.
+     * @param CaseDefinitionExportService $exportService The export service.
+     * @param CaseDefinitionImportService $importService The import service.
+     * @param LoggerInterface             $logger        The logger.
      */
     public function __construct(
         string $appName,
@@ -55,7 +55,7 @@
         private readonly LoggerInterface $logger,
     ) {
         parent::__construct($appName, $request);
-    }
+    }//end __construct()
 
     /**
      * Export a case definition as a ZIP archive.
@@ -104,13 +104,13 @@
                 Http::STATUS_BAD_REQUEST
             );
         } catch (\Throwable $e) {
-            $this->logger->error('Case definition export failed: ' . $e->getMessage());
+            $this->logger->error('Case definition export failed: '.$e->getMessage());
             return new JSONResponse(
-                ['error' => 'Export failed: ' . $e->getMessage()],
+                ['error' => 'Export failed: '.$e->getMessage()],
                 Http::STATUS_INTERNAL_SERVER_ERROR
             );
-        }
-    }
+        }//end try
+    }//end export()
 
     /**
      * Validate a case definition package without importing it.
@@ -124,7 +124,7 @@
         try {
             $file = $this->request->getUploadedFile('package');
 
             if ($file === null || !isset($file['tmp_name'])) {
                 return new JSONResponse(
                     ['error' => 'No package file uploaded'],
                     Http::STATUS_BAD_REQUEST
@@ -135,13 +135,13 @@
 
             return new JSONResponse($result);
         } catch (\Throwable $e) {
-            $this->logger->error('Case definition validation failed: ' . $e->getMessage());
+            $this->logger->error('Case definition validation failed: '.$e->getMessage());
             return new JSONResponse(
-                ['error' => 'Validation failed: ' . $e->getMessage()],
+                ['error' => 'Validation failed: '.$e->getMessage()],
                 Http::STATUS_INTERNAL_SERVER_ERROR
             );
         }
-    }
+    }//end validate()
 
     /**
      * Import a case definition package.
@@ -153,10 +153,10 @@
     public function import(): JSONResponse
     {
         try {
-            $file = $this->request->getUploadedFile('package');
+            $file     = $this->request->getUploadedFile('package');
             $strategy = $this->request->getParam('strategy', 'skip');
 
             if ($file === null || !isset($file['tmp_name'])) {
                 return new JSONResponse(
                     ['error' => 'No package file uploaded'],
                     Http::STATUS_BAD_REQUEST
@@ -175,17 +175,15 @@
                 $strategy
             );
 
-            $statusCode = $result['success']
-                ? Http::STATUS_OK
-                : Http::STATUS_UNPROCESSABLE_ENTITY;
+            $statusCode = $result['success'] ? Http::STATUS_OK : Http::STATUS_UNPROCESSABLE_ENTITY;
 
             return new JSONResponse($result, $statusCode);
         } catch (\Throwable $e) {
-            $this->logger->error('Case definition import failed: ' . $e->getMessage());
+            $this->logger->error('Case definition import failed: '.$e->getMessage());
             return new JSONResponse(
-                ['error' => 'Import failed: ' . $e->getMessage()],
+                ['error' => 'Import failed: '.$e->getMessage()],
                 Http::STATUS_INTERNAL_SERVER_ERROR
             );
-        }
-    }
-}
+        }//end try
+    }//end import()
+}//end class