Deprecate addValidationErrors

CHANGELOG.md

@@ -94,6 +94,7 @@ Complete refactoring of Fortress. Mostly enforcing strict types, updating PHPDoc
 ### Alert
 - Messages are now translated at read time ([#1156](https://github.com/userfrosting/UserFrosting/pull/1156), [#811](https://github.com/userfrosting/UserFrosting/issues/811)). Messages will be translated when using `messages` and `getAndClearMessages`. `addMessage` now accept the optional placeholders, which will be stored with the alert message. `addMessageTranslated` is **deprecated**. 
 - Translator is not optional anymore. `setTranslator` method has been removed.
+- `addValidationErrors` is deprecated (N.B.: It can't accept the new `\UserFrosting\Fortress\Validator\ServerSideValidatorInterface`)
 
 ## [5.0.0](https://github.com/userfrosting/framework/compare/4.6.1...5.0.0)
 With version 5, this repo can be used as a bare bone Slim & Symfony Console application. It include the necessary routing class, [PHP-DI](https://php-di.org) as the Dependency Injection Container, a PSR EventDispatcher, etc. SprinkleManager has also been moved from Core/System Sprinkle and completely rewritten. 

src/Alert/AlertStream.php

@@ -100,6 +100,8 @@ public function messages(): array
      * Add error messages from a ServerSideValidator object to the message stream.
      *
      * @param ServerSideValidator $validator
+     *
+     * @deprecated 5.1 This should be manually done in code.
      */
     public function addValidationErrors(ServerSideValidator $validator): void
     {

tests/Alert/CacheAlertStreamTest.php

@@ -197,6 +197,7 @@ public function testGetAndClearMessages(): void
 
     /**
      * @depends testAddMessage
+     * @deprecated 5.1
      */
     public function testAddValidationErrors(): void
     {

tests/Alert/SessionAlertStreamTest.php

@@ -158,6 +158,7 @@ public function testGetAndClearMessages(): void
 
     /**
      * @depends testAddMessage
+     * @deprecated 5.1
      */
     public function testAddValidationErrors(): void
     {