FEATURE: Make it possible to validate host, source and target path

At the moment we can validate only the source path with a regex in the configuration. This change extends the function to be able to also validate the host and target path.

Author: markusguenther

Classes/Controller/ModuleController.php

@@ -582,17 +582,30 @@ protected function deleteRedirect(string $sourceUriPath, ?string $host = null):
 
     protected function validateRedirectAttributes(?string $host, string $sourceUriPath, string $targetUriPath): bool
     {
+        $valid = true;
+
         if ($sourceUriPath === $targetUriPath) {
-            $this->addFlashMessage('', $this->translateById('error.sameSourceAndTarget'),
-                Message::SEVERITY_WARNING);
-        } elseif (!preg_match($this->validationOptions['sourceUriPath'], $sourceUriPath)) {
-            $this->addFlashMessage('',
-                $this->translateById('error.sourceUriPathNotValid', [$this->validationOptions['sourceUriPath']]),
-                Message::SEVERITY_WARNING);
-        } else {
-            return true;
+            $valid = false;
+            $errorMessages[] = $this->translateById('error.sameSourceAndTarget');
+        }
+        if (isset($this->validationOptions['host']) && !preg_match($this->validationOptions['host'], $host)) {
+            $valid = false;
+            $errorMessages[] = $this->translateById('error.hostNotValid');
+        }
+        if (isset($this->validationOptions['sourceUriPath']) && !preg_match($this->validationOptions['sourceUriPath'], $sourceUriPath)) {
+            $valid = false;
+            $errorMessages[] = $this->translateById('error.sourceUriPathNotValid', [$this->validationOptions['sourceUriPath']]);
         }
-        return false;
+        if (isset($this->validationOptions['targetUriPath']) && !preg_match($this->validationOptions['targetUriPath'], $targetUriPath)) {
+            $valid = false;
+            $errorMessages[] = $this->translateById('error.targetUriPathNotValid', [$this->validationOptions['targetUriPath']]);
+        }
+
+        if (!$valid) {
+            $this->addFlashMessage('', implode('<br>', $errorMessages), Message::SEVERITY_WARNING);
+        }
+
+        return $valid;
     }
 
     protected function isSame(

Resources/Private/Translations/de/Modules.xlf

@@ -163,6 +163,14 @@
                 <source>The source path doesn't match the expression {0}</source>
         <target>Der Quellpfad entspricht nicht dem Muster {0}</target>
       </trans-unit>
+      <trans-unit id="error.targetUriPathNotValid" xml:space="preserve">
+        <source>The target path doesn't match the expression {0}</source>
+        <target>Der Zielpfad entspricht nicht dem Muster {0}</target>
+      </trans-unit>
+      <trans-unit id="error.hostNotValid" xml:space="preserve">
+        <source>The host doesn't match the expression {0}</source>
+        <target>Der Host entspricht nicht dem Muster {0}</target>
+      </trans-unit>
       <trans-unit id="host" xml:space="preserve">
                 <source>Origin domain</source>
         <target>Ursprungsdomäne</target>

Resources/Private/Translations/en/Modules.xlf

@@ -128,6 +128,12 @@
             <trans-unit id="error.sourceUriPathNotValid" xml:space="preserve">
                 <source>The source path doesn't match the expression {0}</source>
             </trans-unit>
+            <trans-unit id="error.targetUriPathNotValid" xml:space="preserve">
+                <source>The target path doesn't match the expression {0}</source>
+            </trans-unit>
+            <trans-unit id="error.hostNotValid" xml:space="preserve">
+                <source>The host doesn't match the expression {0}</source>
+            </trans-unit>
 
             <trans-unit id="host" xml:space="preserve">
                 <source>Origin domain</source>