feat(ZMS-1808): add validation

it-at-m · Jan 17, 2024 · 8533473 · 8533473
1 parent 0a95e61
commit 8533473
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 4 deletions.
diff --git a/zmsadmin/src/Zmsadmin/WorkstationProcessRedirect.php b/zmsadmin/src/Zmsadmin/WorkstationProcessRedirect.php
@@ -29,9 +29,24 @@ public function readResponse(
         $process = $workstation->process;
 
         if ($request->getMethod() === 'POST') {
+            $validator = $request->getAttribute('validator');
+            $selectedLocation = $validator->getParameter('location')->isNumber()->getValue();
+
+            if (empty($selectedLocation)) {
+                return \BO\Slim\Render::redirect(
+                    'workstationProcessRedirect',
+                    [],
+                    [
+                        'errors' => [
+                            'location' => 'Bitte wählen Sie einen Standort aus.'
+                        ]
+                    ]
+                );
+            }
+
             $scope = \App::$http
                 ->readGetResult(
-                    '/scope/' . $input['location'] . '/',
+                    '/scope/' . $selectedLocation . '/',
                     ['resolveReferences' => 2]
                 )->getEntity();
 
@@ -54,7 +69,7 @@ public function readResponse(
                 array()
             );
         }
- 
+
         return \BO\Slim\Render::withHtml(
             $response,
             'page/workstationProcessRedirect.twig',
@@ -64,7 +79,8 @@ public function readResponse(
                 'department' => $department,
                 'scope' => $workstation->scope,
                 'scopes' => $department->getScopeList(),
-                'menuActive' => 'workstation'
+                'menuActive' => 'workstation',
+                'errors' => $request->getParam('errors')
             )
         );
     }

diff --git a/zmsadmin/templates/block/appointment/formRedirect.twig b/zmsadmin/templates/block/appointment/formRedirect.twig
@@ -21,9 +21,10 @@
             {% set scopeOptionsGroup = scopeOptionsGroup|merge([{ 'name' : department.name, 'options' : scopeOptionsGroupOptions|msort('name') }]) %}
 
             {{ formgroup(
-                {"label": "Weiterleiten zu Standort"},
+                {"label": "Weiterleiten zu Standort", "required": true, "errors": [errors.location]},
                 [{
                     "type": "select",
+                    "required": true,
                     "parameter": {
                         "name": "location",
                         "options": scopeOptionsGroup