OXDEV-8786 Add multiple error messages in one exception

Author: TitaKoleva

src/PasswordPolicy/Intrastructure/ExceptionFactory.php

@@ -10,6 +10,7 @@
 namespace OxidEsales\SecurityModule\PasswordPolicy\Intrastructure;
 
 use OxidEsales\Eshop\Core\Exception\InputException;
+use OxidEsales\SecurityModule\PasswordPolicy\Validation\Exception\PasswordCollectionException;
 use OxidEsales\SecurityModule\PasswordPolicy\Validation\Exception\PasswordValidateException;
 
 class ExceptionFactory implements ExceptionFactoryInterface
@@ -21,6 +22,10 @@ public function __construct(
 
     public function create(PasswordValidateException $exception): InputException
     {
+        if ($exception instanceof PasswordCollectionException) {
+            return $this->createCollection($exception);
+        }
+
         $exception = oxNew(
             InputException::class,
             sprintf(
@@ -32,4 +37,20 @@ public function create(PasswordValidateException $exception): InputException
 
         return $exception;
     }
+
+    private function createCollection(PasswordCollectionException $collection): InputException
+    {
+        $lines = [];
+        foreach ($collection->getValidationExceptions() as $ex) {
+            $lines[] = sprintf(
+                /** @phpstan-ignore-next-line */
+                $this->language->translateString($ex->getMessage()),
+                ...$ex->getTranslationParameters()
+            );
+        }
+        /** @phpstan-ignore-next-line */
+        $message = '<ul><li>' . implode('</li><li>', $lines) . '</li></ul>';
+
+        return oxNew(InputException::class, $message);
+    }
 }

src/PasswordPolicy/Validation/Exception/PasswordCollectionException.php

@@ -9,9 +9,7 @@
 
 namespace OxidEsales\SecurityModule\PasswordPolicy\Validation\Exception;
 
-use OxidEsales\Eshop\Core\Exception\InputException;
-
-class PasswordCollectionException extends InputException
+class PasswordCollectionException extends PasswordValidateException
 {
     /**
      * @var PasswordValidateException[]
@@ -20,6 +18,7 @@ class PasswordCollectionException extends InputException
 
     public function __construct(array $exceptions)
     {
+        parent::__construct();
         $this->exceptions = $exceptions;
     }
 

src/Shared/Core/InputValidator.php

@@ -39,16 +39,11 @@ public function checkPassword($user, $newPassword, $confirmationPassword, $shoul
             $passwordValidator->validatePassword($newPassword);
         } catch (PasswordCollectionException $e) {
             $exceptionFactory = $this->getService(ExceptionFactoryInterface::class);
-            foreach ($e->getValidationExceptions() as $key => $error) {
-                if ($key === count($e->getValidationExceptions()) - 1) {
-                    return $this->addValidationError(
-                        "oxuser__oxpassword",
-                        $exceptionFactory->create($error)
-                    );
-                }
-
-                Registry::getUtilsView()->addErrorToDisplay($exceptionFactory->create($error));
-            }
+
+            return $this->addValidationError(
+                "oxuser__oxpassword",
+                $exceptionFactory->create($e)
+            );
         }
 
         return parent::checkPassword($user, $newPassword, $confirmationPassword, $shouldCheckPasswordLength);

tests/Integration/Shared/Core/InputValidatorTest.php

@@ -32,7 +32,7 @@ public function testInputValidationError($password, $expectedException): void
         $exception = $validator->checkPassword($userModelMock, $password, $password);
 
         $this->assertInstanceOf(InputException::class, $exception);
-        $this->assertSame($expectedException, $exception->getMessage());
+        $this->assertStringContainsString($expectedException, $exception->getMessage());
     }
 
     public static function dataProviderPasswordError(): iterable