Internal: Add @implements ProcessorInterface specifying their generic type parameters.

Author: AngelFQC

src/CoreBundle/State/AccessUrlRelColorThemeStateProcessor.php

@@ -12,6 +12,9 @@
 use Chamilo\CoreBundle\ServiceHelper\AccessUrlHelper;
 use Doctrine\ORM\EntityManagerInterface;
 
+/**
+ * @implements ProcessorInterface<AccessUrlRelColorTheme, AccessUrlRelColorTheme>
+ */
 final class AccessUrlRelColorThemeStateProcessor implements ProcessorInterface
 {
     public function __construct(

src/CoreBundle/State/CAttendanceStateProcessor.php

@@ -18,6 +18,9 @@
 use Symfony\Component\HttpFoundation\RequestStack;
 use Symfony\Component\HttpKernel\Exception\BadRequestHttpException;
 
+/**
+ * @implements ProcessorInterface<void, void>
+ */
 final class CAttendanceStateProcessor implements ProcessorInterface
 {
     public function __construct(

src/CoreBundle/State/CCalendarEventStateProcessor.php

@@ -14,6 +14,9 @@
 use Exception;
 use Symfony\Bundle\SecurityBundle\Security;
 
+/**
+ * @implements ProcessorInterface<CCalendarEvent, CCalendarEvent>
+ */
 final class CCalendarEventStateProcessor implements ProcessorInterface
 {
     public function __construct(

src/CoreBundle/State/CStudentPublicationPostStateProcessor.php

@@ -23,6 +23,9 @@
 use Symfony\Component\Routing\RouterInterface;
 use Symfony\Contracts\Translation\TranslatorInterface;
 
+/**
+ * @implements ProcessorInterface<CStudentPublication, CStudentPublication>
+ */
 final class CStudentPublicationPostStateProcessor implements ProcessorInterface
 {
     public function __construct(
@@ -34,8 +37,12 @@ public function __construct(
         private readonly SettingsManager $settingsManager,
     ) {}
 
-    public function process($data, Operation $operation, array $uriVariables = [], array $context = [])
-    {
+    public function process(
+        $data,
+        Operation $operation,
+        array $uriVariables = [],
+        array $context = []
+    ): CStudentPublication {
         $result = $this->persistProcessor->process($data, $operation, $uriVariables, $context);
 
         /** @var CStudentPublication $publication */

src/CoreBundle/State/ColorThemeStateProcessor.php

@@ -19,6 +19,9 @@
 use const DIRECTORY_SEPARATOR;
 use const PHP_EOL;
 
+/**
+ * @implements ProcessorInterface<ColorTheme, ColorTheme|void>
+ */
 final class ColorThemeStateProcessor implements ProcessorInterface
 {
     public function __construct(
@@ -29,12 +32,7 @@ public function __construct(
         private readonly FilesystemOperator $filesystem,
     ) {}
 
-    /**
-     * @param mixed $data
-     *
-     * @throws FilesystemException
-     */
-    public function process($data, Operation $operation, array $uriVariables = [], array $context = [])
+    public function process($data, Operation $operation, array $uriVariables = [], array $context = []): ?ColorTheme
     {
         \assert($data instanceof ColorTheme);
 

src/CoreBundle/State/MessageProcessor.php

@@ -12,27 +12,27 @@
 use Chamilo\CoreBundle\Entity\Message;
 use Chamilo\CoreBundle\Entity\MessageAttachment;
 use Chamilo\CoreBundle\Entity\MessageRelUser;
+use Chamilo\CoreBundle\Entity\User;
 use Chamilo\CoreBundle\Repository\ResourceNodeRepository;
 use Doctrine\ORM\EntityManagerInterface;
 use LogicException;
 use Notification;
 use Symfony\Bundle\SecurityBundle\Security;
-use Symfony\Component\HttpFoundation\RequestStack;
-use Vich\UploaderBundle\Storage\FlysystemStorage;
 
-final class MessageProcessor implements ProcessorInterface
+/**
+ * @implements ProcessorInterface<Message, Message|void>
+ */
+final readonly class MessageProcessor implements ProcessorInterface
 {
     public function __construct(
-        private readonly ProcessorInterface $persistProcessor,
-        private readonly ProcessorInterface $removeProcessor,
-        private readonly FlysystemStorage $storage,
-        private readonly EntityManagerInterface $entityManager,
-        private readonly ResourceNodeRepository $resourceNodeRepository,
-        private readonly Security $security,
-        private readonly RequestStack $requestStack
+        private ProcessorInterface $persistProcessor,
+        private ProcessorInterface $removeProcessor,
+        private EntityManagerInterface $entityManager,
+        private ResourceNodeRepository $resourceNodeRepository,
+        private Security $security,
     ) {}
 
-    public function process($data, Operation $operation, array $uriVariables = [], array $context = [])
+    public function process($data, Operation $operation, array $uriVariables = [], array $context = []): ?Message
     {
         if ($operation instanceof DeleteOperationInterface) {
             return $this->removeProcessor->process($data, $operation, $uriVariables, $context);
@@ -51,6 +51,7 @@ public function process($data, Operation $operation, array $uriVariables = [], a
             }
         }
 
+        /** @var User $user */
         $user = $this->security->getUser();
         if (!$user) {
             throw new LogicException('User not found.');

src/CoreBundle/State/UserRelUserStateProcessor.php

@@ -13,6 +13,9 @@
 use Chamilo\CoreBundle\Entity\UserRelUser;
 use Doctrine\ORM\EntityManagerInterface;
 
+/**
+ * @implements ProcessorInterface<UserRelUser, UserRelUser|void>
+ */
 final class UserRelUserStateProcessor implements ProcessorInterface
 {
     public function __construct(

src/CoreBundle/State/UsergroupPostStateProcessor.php

@@ -12,6 +12,9 @@
 use Symfony\Bundle\SecurityBundle\Security;
 use Symfony\Component\HttpFoundation\RequestStack;
 
+/**
+ * @implements ProcessorInterface<Usergroup, Usergroup>
+ */
 final class UsergroupPostStateProcessor implements ProcessorInterface
 {
     private ProcessorInterface $processor;
@@ -31,7 +34,7 @@ public function __construct(
         $this->requestStack = $requestStack;
     }
 
-    public function process($data, Operation $operation, array $uriVariables = [], array $context = [])
+    public function process($data, Operation $operation, array $uriVariables = [], array $context = []): Usergroup
     {
         /** @var Usergroup $usergroup */
         $usergroup = $this->processor->process($data, $operation, $uriVariables, $context);