Cleaned up

OS2Forms · Sep 29, 2023 · 93cb526 · 93cb526
1 parent a0904ab
commit 93cb526
Showing 1 changed file with 37 additions and 4 deletions.
diff --git a/modules/os2forms_forloeb/src/Controller/MaestroNotificationController.php b/modules/os2forms_forloeb/src/Controller/MaestroNotificationController.php
@@ -3,7 +3,6 @@
 namespace Drupal\os2forms_forloeb\Controller;
 
 use Drupal\Core\Controller\ControllerBase;
-use Drupal\Core\Routing\RouteMatchInterface;
 use Drupal\Core\Url;
 use Drupal\os2forms_digital_post\Model\Document;
 use Drupal\os2forms_forloeb\MaestroHelper;
@@ -39,9 +38,23 @@ public static function create(ContainerInterface $container) {
   }
 
   /**
-   * Preview action.
+   * Preview notification action.
+   *
+   * @param Request $request
+   *    The request.
+   * @param WebformInterface $webform
+   *    The webform.
+   * @param string $handler
+   *    The handler ID.
+   * @param string $notification_type
+   *    The notification type.
+   * @param string $content_type
+   *    The content type.
+   *
+   * @return array
+   *   A render array.
    */
-  public function preview(Request $request, WebformInterface $webform, string $handler, string $notification_type, string $content_type, RouteMatchInterface $routeMatch) {
+  public function preview(Request $request, WebformInterface $webform, string $handler, string $notification_type, string $content_type): array {
     $handler = $webform->getHandler($handler);
     $submissionIds = array_keys($this->webformSubmissionStorage->getQuery()
       ->condition('webform_id', $webform->id())
@@ -103,8 +116,20 @@ public function preview(Request $request, WebformInterface $webform, string $han
 
   /**
    * Render notification preview.
+   *
+   * @param string $handler
+   *   The handler ID.
+   * @param string $notification_type
+   *   The notification type.
+   * @param string $content_type
+   *   The content type.
+   * @param WebformSubmissionInterface $submission
+   *   The webform submission.
+   *
+   * @return Response
+   *   The response.
    */
-  public function previewRender(Request $request, WebformInterface $webform, string $handler, string $notification_type, string $content_type, WebformSubmissionInterface $submission) {
+  public function previewRender(string $handler, string $notification_type, string $content_type, WebformSubmissionInterface $submission) {
     $templateTask = [];
     $maestroQueueID = 0;
     [
@@ -122,6 +147,14 @@ public function previewRender(Request $request, WebformInterface $webform, strin
 
   /**
    * Message action.
+   *
+   * Used only to show a message when a Maestro task link from a notification preview is clicked.
+   *
+   * @param Request $request
+   *   The request.
+   *
+   * @return Response
+   *   The response.
    */
   public function message(Request $request): Response {
     $content[] = '<h1>' . $request->get('message') . '</h1>';