Merge branch 'feature/SF-6.2' into develop

.env

@@ -51,7 +51,10 @@ WKHTMLTOIMAGE_PATH=/usr/local/bin/wkhtmltoimage
 ###< knplabs/knp-snappy-bundle ###
 
 ###> symfony/messenger ###
-MESSENGER_TRANSPORT_DSN=doctrine://default
+# Choose one of the transports below
+# MESSENGER_TRANSPORT_DSN=amqp://guest:guest@localhost:5672/%2f/messages
+# MESSENGER_TRANSPORT_DSN=redis://localhost:6379/messages
+MESSENGER_TRANSPORT_DSN=doctrine://default?auto_setup=0
 ###< symfony/messenger ###
 
 ###> AdminAssos configuration ###

composer.json

@@ -15,7 +15,6 @@
         "beberlei/doctrineextensions": "^1.3",
         "blackfire/php-sdk": "^1.23",
         "composer/package-versions-deprecated": "^1.11",
-        "damienharper/auditor-bundle": "^5.0",
         "doctrine/dbal": "^2.6",
         "doctrine/doctrine-bundle": "^2.1",
         "doctrine/doctrine-migrations-bundle": "^3.0",
@@ -33,30 +32,32 @@
         "phpdocumentor/reflection-docblock": "^5.2",
         "ralouphie/mimey": "^1.0",
         "ramsey/uuid": "^4.2",
-        "symfony/asset": "5.4.*",
-        "symfony/console": "5.4.*",
-        "symfony/doctrine-messenger": "5.4.*",
-        "symfony/dotenv": "5.4.*",
-        "symfony/expression-language": "5.4.*",
+        "symfony/asset": "6.2.*",
+        "symfony/console": "6.2.*",
+        "symfony/doctrine-messenger": "6.2.*",
+        "symfony/dotenv": "6.2.*",
+        "symfony/error-handler": "6.2.*",
+        "symfony/expression-language": "6.2.*",
         "symfony/flex": "^1.3.1",
-        "symfony/framework-bundle": "5.4.*",
-        "symfony/http-client": "5.4.*",
-        "symfony/intl": "5.4.*",
-        "symfony/lock": "5.4.*",
-        "symfony/mailer": "^5.4",
+        "symfony/framework-bundle": "6.2.*",
+        "symfony/http-client": "6.2.*",
+        "symfony/intl": "6.2.*",
+        "symfony/lock": "6.2.*",
+        "symfony/mailer": "6.2.*",
         "symfony/mercure": "^0.6.1",
         "symfony/mercure-bundle": "^0.3.4",
         "symfony/monolog-bundle": "^3.7",
+        "symfony/polyfill-intl-icu": "^1.21",
         "symfony/polyfill-intl-messageformatter": "^1.24",
-        "symfony/property-access": "5.4.*",
-        "symfony/property-info": "5.4.*",
-        "symfony/security-bundle": "5.4.*",
-        "symfony/serializer": "5.4.*",
-        "symfony/translation": "5.4.*",
-        "symfony/twig-bundle": "^5.4",
-        "symfony/uid": "5.4.*",
-        "symfony/validator": "5.4.*",
-        "symfony/yaml": "5.4.*",
+        "symfony/property-access": "6.2.*",
+        "symfony/property-info": "6.2.*",
+        "symfony/security-bundle": "6.2.*",
+        "symfony/serializer": "6.2.*",
+        "symfony/translation": "6.2.*",
+        "symfony/twig-bundle": "6.2.*",
+        "symfony/uid": "6.2.*",
+        "symfony/validator": "6.2.*",
+        "symfony/yaml": "6.2.*",
         "twig/cssinliner-extra": "^3.4",
         "twig/extra-bundle": "^3.4",
         "twig/inky-extra": "^3.4",
@@ -66,12 +67,12 @@
     "require-dev": {
         "cyclonedx/cyclonedx-php-composer": "^3.4",
         "phpstan/phpstan": "^1.9",
-        "rector/rector": "^0.14",
-        "symfony/debug-bundle": "5.4.*",
+        "rector/rector": "^0.15.13",
+        "symfony/debug-bundle": "6.2.*",
         "symfony/maker-bundle": "^1.21",
-        "symfony/phpunit-bridge": "^5.4",
-        "symfony/stopwatch": "^5.4",
-        "symfony/web-profiler-bundle": "^5.4"
+        "symfony/phpunit-bridge": "^6.2",
+        "symfony/stopwatch": "6.2.*",
+        "symfony/web-profiler-bundle": "6.2.*"
     },
     "config": {
         "optimize-autoloader": true,
@@ -121,7 +122,7 @@
     "extra": {
         "symfony": {
             "allow-contrib": false,
-            "require": "5.4.*"
+            "require": "6.2.*"
         }
     }
 }

config/bundles.php

@@ -20,5 +20,4 @@ return [
     Symfony\Bundle\DebugBundle\DebugBundle::class => ['docker' => true],
     Symfony\Bundle\MercureBundle\MercureBundle::class => ['all' => true],
     Twig\Extra\TwigExtraBundle\TwigExtraBundle::class => ['all' => true],
-    DH\AuditorBundle\DHAuditorBundle::class => ['all' => true],
 ];

config/packages/api_platform.yaml

@@ -10,9 +10,7 @@ api_platform:
         json: ['application/merge-patch+json']
     swagger:
         versions: [3]
-    collection:
-        pagination:
-            # The default number of items per page.
-            items_per_page: 20
     resource_class_directories:
         - '%kernel.project_dir%/src/Entity'
+    defaults:
+        pagination_items_per_page: 20

config/packages/docker/security.yaml

@@ -0,0 +1,4 @@
+security:
+  password_hashers:
+    App\Entity\Person\Person:
+      algorithm: plaintext

config/packages/framework.yaml

@@ -2,7 +2,8 @@
 framework:
     secret: '%env(APP_SECRET)%'
     #csrf_protection: true
-    #http_method_override: true
+    http_method_override: false
+    handle_all_throwables: true
 
     # Enables session support. Note that the session will ONLY be started if you read or write from it.
     # Remove or comment this section to explicitly disable session support.
@@ -10,31 +11,15 @@ framework:
         handler_id: null
         cookie_secure: auto
         cookie_samesite: lax
+        storage_factory_id: session.storage.factory.native
 
     #esi: true
     #fragments: true
     php_errors:
         log: true
 
-    http_client:
-        scoped_clients:
-            siret_checking:
-                base_uri: 'https://entreprise.data.gouv.fr/api/sirene/v3/etablissements/'
-            openstreetmap:
-                base_uri: 'https://nominatim.openstreetmap.org/'
-            dolibarr_client:
-                base_uri: '%env(DOLIBARR_API_BASE_URI)%'
-                headers:
-                    DOLAPIKEY: '%env(DOLIBARR_API_TOKEN)%'
-                    Accept: 'application/json'
-            typo3_client:
-                base_uri: '%env(TYPO3_BASE_URI)%'
-                headers:
-                    Content-Type: 'application/json'
-                    Accept: 'application/json'
-            mobyt_client:
-                base_uri: '%env(MOBYT_API_BASE_URI)%'
-                headers:
-                    Content-Type: 'application/json'
-            apiLegacyClient:
-                base_uri: '%env(API_LEG_BASE_URL)%'
+when@test:
+    framework:
+        test: true
+        session:
+            storage_factory_id: session.storage.factory.mock_file

config/packages/messenger.yaml

@@ -1,13 +1,13 @@
 framework:
     messenger:
         # Uncomment this (and the failed transport below) to send failed messages to this transport for later handling.
-        failure_transport: failed
+        # failure_transport: failed
 
         transports:
             # https://symfony.com/doc/current/messenger.html#transport-configuration
-            async: '%env(MESSENGER_TRANSPORT_DSN)%'
-            failed: '%env(MESSENGER_TRANSPORT_DSN)%?queue_name=failed'
-            sync: 'sync://'
+            # async: '%env(MESSENGER_TRANSPORT_DSN)%'
+            # failed: 'doctrine://default?queue_name=failed'
+            # sync: 'sync://'
 
         routing:
             # Route your messages to the transports
@@ -16,4 +16,13 @@ framework:
             'App\Message\Command\Export': async
             'App\Message\Command\Typo3\Typo3UpdateCommand': async
             'App\Message\Command\Typo3\Typo3DeleteCommand': async
-            'App\Message\Command\Typo3\Typo3UndeleteCommand': async
+            'App\Message\Command\Typo3\Typo3UndeleteCommand': async
+
+
+# when@test:
+#    framework:
+#        messenger:
+#            transports:
+#                # replace with your transport name here (e.g., my_transport: 'in-memory://')
+#                # For more Messenger testing tools, see https://github.com/zenstruck/messenger-test
+#                async: 'in-memory://'

config/packages/security.yaml

@@ -3,9 +3,9 @@ imports:
 
 security:
     role_hierarchy:
-        ROLE_SUPER_ADMIN: [ROLE_ADMIN, ROLE-ALLOWED-TO-SWITCH, ROLE_TIPS, ROLE_NETWORK]
+        ROLE_SUPER_ADMIN: [ ROLE_ADMIN, ROLE-ALLOWED-TO-SWITCH, ROLE_TIPS, ROLE_NETWORK ]
 
-        BASE_ROLE_ADMINISTRATION_CORE : &BASE_ROLE_ADMINISTRATION_CORE
+        BASE_ROLE_ADMINISTRATION_CORE: &BASE_ROLE_ADMINISTRATION_CORE
             - ROLE_MEMBER_CORE
             - ROLE_ORGANIZATION
 
@@ -91,7 +91,7 @@ security:
         ROLE_CA_CORE:
             - ROLE_MEMBER_CORE
 
-        ROLE_STUDENT :
+        ROLE_STUDENT:
             - ROLE_STUDENT_CORE
 
         ROLE_STUDENT_CORE:
@@ -119,11 +119,11 @@ security:
             - ROLE_CORE
             - ROLE_RULERZ_ACTION
 
-    encoders:
+    password_hashers:
         App\Entity\Person\Person:
-            algorithm: plaintext
+            algorithm: bcrypt
 
-    # https://symfony.com/doc/current/security.html#where-do-users-come-from-user-providers
+    # https://symfony.com/doc/current/security.html#loading-the-user-the-user-provider
     providers:
         # used to reload user from session & other features (e.g. switch_user)
         access_provider:
@@ -142,7 +142,7 @@ security:
             security: false
 
         login:
-            pattern:  ^/login_check
+            pattern: ^/login_check
             stateless: true
             json_login:
                 provider: person_provider
@@ -162,10 +162,26 @@ security:
 
         main:
             # activate different ways to authenticate
-            # https://symfony.com/doc/current/security.html#firewalls-authentication
+            # https://symfony.com/doc/current/security.html#the-firewall
+
+            # https://symfony.com/doc/current/security/impersonating_user.html
+            # switch_user: true
 
     # Easy way to control access for large sections of your site
     # Note: Only the *first* access control that matches will be used
     access_control:
         - { path: ^/api/public, roles: PUBLIC_ACCESS }
         - { path: ^/api/, roles: IS_HAVING_MODULE }
+
+when@test:
+    security:
+        password_hashers:
+            # By default, password hashers are resource intensive and take time. This is
+            # important to generate secure password hashes. In tests however, secure hashes
+            # are not important, waste resources and increase test times. The following
+            # reduces the work factor to the lowest possible values.
+            Symfony\Component\Security\Core\User\PasswordAuthenticatedUserInterface:
+                algorithm: auto
+                cost: 4 # Lowest possible value for bcrypt
+                time_cost: 3 # Lowest possible value for argon
+                memory_cost: 10 # Lowest possible value for argon

config/packages/test/framework.yaml

@@ -1,4 +1,4 @@
 framework:
     test: true
     session:
-        storage_id: session.storage.mock_file
+        storage_factory_id: session.storage.mock_file

config/packages/uid.yaml

@@ -0,0 +1,4 @@
+framework:
+    uid:
+        default_uuid_version: 7
+        time_based_uuid_version: 7

config/packages/web_profiler.yaml

@@ -4,7 +4,9 @@ when@dev:
         intercept_redirects: false
 
     framework:
-        profiler: { only_exceptions: false }
+        profiler:
+            only_exceptions: false
+            collect_serializer_data: true
 
 when@test:
     web_profiler:

config/routes.yaml

@@ -1,3 +1,9 @@
+controllers:
+    resource:
+        path: ../src/Controller/
+        namespace: App\Controller
+    type: attribute
+
 login_check:
   path: /login_check
   methods: ['POST']

config/routes/annotations.yaml

@@ -1,7 +0,0 @@
-controllers:
-    resource: ../../src/Controller/
-    type: annotation
-
-kernel:
-    resource: ../../src/Kernel.php
-    type: annotation

config/services.yaml

@@ -104,3 +104,9 @@ services:
     App\Service\Elasticsearch\EducationNotationUpdater:
         arguments:
             - '@fos_elastica.object_persister.search.educationNotation'
+
+    #########################################
+    ## AutoWiring du service container ##
+    Symfony\Component\DependencyInjection\ContainerInterface: '@service_container'
+    #########################################
+

rector.php

@@ -2,30 +2,24 @@
 
 declare(strict_types=1);
 
-use Rector\Core\Configuration\Option;
-use Rector\Core\ValueObject\PhpVersion;
-use Rector\Doctrine\Set\DoctrineSetList;
-use Rector\Php74\Rector\Property\TypedPropertyRector;
-use Rector\Set\ValueObject\LevelSetList;
-use Rector\Symfony\Set\SensiolabsSetList;
-use Rector\Symfony\Set\SymfonyLevelSetList;
-use Rector\Symfony\Set\SymfonySetList;
-use Symfony\Component\DependencyInjection\Loader\Configurator\ContainerConfigurator;
+use Rector\CodeQuality\Rector\Class_\InlineConstructorDefaultToPropertyRector;
+use Rector\Config\RectorConfig;
+use \Rector\Symfony\Set\SymfonySetList;
+
+return static function (RectorConfig $rectorConfig): void {
+    $rectorConfig->paths([
+        __DIR__ . '/config',
+        __DIR__ . '/public',
+        __DIR__ . '/src',
+    ]);
 
-return static function (ContainerConfigurator $containerConfigurator): void {
-    // region Symfony Container
-    $parameters = $containerConfigurator->parameters();
-    $parameters->set(
-        Option::SYMFONY_CONTAINER_XML_PATH_PARAMETER,
-        __DIR__ . '/var/cache/dev/App_KernelDevDebugContainer.xml'
-    );
-    // endregion
-    // Define what rule sets will be applied
-    $containerConfigurator->import(DoctrineSetList::ANNOTATIONS_TO_ATTRIBUTES);
-    $containerConfigurator->import(SymfonySetList::ANNOTATIONS_TO_ATTRIBUTES);
-    $containerConfigurator->import(SensiolabsSetList::FRAMEWORK_EXTRA_61);
-    // get services (needed for register a single rule)
-    // $services = $containerConfigurator->services();
     // register a single rule
-    // $services->set(TypedPropertyRector::class);
-};
+    $rectorConfig->rule(InlineConstructorDefaultToPropertyRector::class);
+    $rectorConfig->symfonyContainerXml(__DIR__ . '/var/cache/docker/App_KernelDockerDebugContainer.xml');
+
+    $rectorConfig->sets([
+        \Rector\Symfony\Set\SymfonyLevelSetList::UP_TO_SYMFONY_60,
+        SymfonySetList::SYMFONY_CODE_QUALITY,
+        SymfonySetList::SYMFONY_CONSTRUCTOR_INJECTION,
+    ]);
+};

src/Commands/CronCommand.php

@@ -143,7 +143,7 @@ class CronCommand extends Command
             $results[] = $this->runJob($job, $preview);
         }
 
-        return max($results); // If there is one failure result, the whole command is shown as a failure too
+        return (int) max($results); // If there is one failure result, the whole command is shown as a failure too
     }
 
     /**

src/Commands/Doctrine/SchemaUpdateCommand.php

@@ -14,7 +14,7 @@ use Symfony\Component\Console\Style\SymfonyStyle;
  */
 class SchemaUpdateCommand extends UpdateSchemaDoctrineCommand
 {
-    protected function executeSchemaCommand(InputInterface $input, OutputInterface $output, SchemaTool $schemaTool, array $metadatas, SymfonyStyle $ui)
+    protected function executeSchemaCommand(InputInterface $input, OutputInterface $output, SchemaTool $schemaTool, array $metadatas, SymfonyStyle $ui): ?int
     {
         $output->writeln('-- Executing pre-update scripts');
 

src/Controller/AuditController.php

@@ -1,26 +0,0 @@
-<?php
-
-namespace App\Controller;
-
-use App\Entity\Organization\Parameters;
-use DH\Auditor\Provider\Doctrine\DoctrineProvider;
-use DH\Auditor\Provider\Doctrine\Persistence\Reader\Filter\SimpleFilter;
-use DH\Auditor\Provider\Doctrine\Persistence\Reader\Reader;
-use Symfony\Bundle\FrameworkBundle\Controller\AbstractController;
-use Symfony\Component\HttpFoundation\Response;
-use Symfony\Component\Routing\Annotation\Route;
-
-class AuditController extends AbstractController
-{
-    #[Route('/audit_test', name: 'audit_test')]
-    public function index(DoctrineProvider $doctrineProvider): Response
-    {
-        $reader = new Reader($doctrineProvider);
-        $query = $reader
-            ->createQuery(Parameters::class)
-            ->addFilter(new SimpleFilter('object_id', 5755));
-
-        dd($query->execute());
-        return new Response('ok');
-    }
-}

src/Controller/AuditController.php.ori

@@ -0,0 +1,26 @@
+<?php
+//
+//namespace App\Controller;
+//
+//use App\Entity\Organization\Parameters;
+//use DH\Auditor\Provider\Doctrine\DoctrineProvider;
+//use DH\Auditor\Provider\Doctrine\Persistence\Reader\Filter\SimpleFilter;
+//use DH\Auditor\Provider\Doctrine\Persistence\Reader\Reader;
+//use Symfony\Bundle\FrameworkBundle\Controller\AbstractController;
+//use Symfony\Component\HttpFoundation\Response;
+//use Symfony\Component\Routing\Annotation\Route;
+//
+//class AuditController extends AbstractController
+//{
+//    #[Route('/audit_test', name: 'audit_test')]
+//    public function index(DoctrineProvider $doctrineProvider): Response
+//    {
+//        $reader = new Reader($doctrineProvider);
+//        $query = $reader
+//            ->createQuery(Parameters::class)
+//            ->addFilter(new SimpleFilter('object_id', 5755));
+//
+//        dd($query->execute());
+//        return new Response('ok');
+//    }
+//}

src/DataPersister/Access/AdminAccessDataPersister.php

@@ -36,7 +36,7 @@ class AdminAccessDataPersister implements ContextAwareDataPersisterInterface
      * @param array $context
      * @return void|null
      */
-    public function persist($data, array $context = [])
+    public function persist($data, array $context = []): void
     {
         /** @var Access $access */
         $access = $this->security->getUser();

src/DataPersister/EntityDataPersister.php

@@ -38,9 +38,9 @@ class EntityDataPersister implements ContextAwareDataPersisterInterface
      *
      * @param mixed $data
      * @param array $context
-     * @return object|void
+     * @return object
      */
-    public function persist($data, array $context = [])
+    public function persist($data, array $context = []): object
     {
         $onChangeContext = new OnChangeContext($context);
 

src/Entity/Access/Access.php

@@ -37,7 +37,6 @@ use App\Entity\Product\EquipmentLoan;
 use App\Filter\Person\FullNameFilter;
 use ApiPlatform\Core\Bridge\Doctrine\Orm\Filter\BooleanFilter;
 use ApiPlatform\Core\Annotation\ApiResource;
-use ApiPlatform\Core\Annotation\ApiSubresource;
 use App\Entity\Billing\AccessIntangible;
 use App\Entity\Billing\AccessPayer;
 use App\Entity\Core\Notification;
@@ -56,19 +55,21 @@ use Doctrine\Common\Collections\ArrayCollection;
 use Doctrine\Common\Collections\Collection;
 use Doctrine\ORM\Mapping as ORM;
 use JetBrains\PhpStorm\Pure;
+use Symfony\Component\Security\Core\User\PasswordAuthenticatedUserInterface;
 use Symfony\Component\Security\Core\User\UserInterface;
 use Symfony\Component\Serializer\Annotation\Groups;
 
 /**
  * Fais le lien entre une Person et une Organization
- * @ApiResource @see : config/api_platform/Access/access.yaml
+ * @see : config/api_platform/Access/access.yaml
  */
+#[ApiResource]
 #[Auditable]
 #[ORM\Entity(repositoryClass: AccessRepository::class)]
 #[ApiFilter(BooleanFilter::class, properties: ['person.isPhysical'])]
 #[ApiFilter(FullNameFilter::class)]
 #[ApiFilter(InFilter::class, properties: ['id'])]
-class Access implements UserInterface
+class Access implements UserInterface, PasswordAuthenticatedUserInterface
 {
     #[ORM\Id]
     #[ORM\Column]
@@ -110,11 +111,11 @@ class Access implements UserInterface
     private AccessBilling $accessBilling;
 
     #[ORM\OneToMany(mappedBy: 'access', targetEntity: PersonActivity::class, cascade: ['persist'], orphanRemoval: true)]
-    #[ApiSubresource]
+    #[ApiResource]
     private Collection $personActivity;
 
     #[ORM\OneToMany(mappedBy: 'access', targetEntity: OrganizationFunction::class, cascade: ['persist'], orphanRemoval: true)]
-    #[ApiSubresource]
+    #[ApiResource]
     private Collection $organizationFunction;
 
     #[ORM\OneToMany(mappedBy: 'licensee', targetEntity: OrganizationLicence::class, cascade: ['persist'], orphanRemoval: true)]
@@ -282,9 +283,6 @@ class Access implements UserInterface
     #[ORM\OneToMany(mappedBy: 'student', targetEntity: WorkByUser::class, cascade: ['persist'], orphanRemoval: true)]
     private Collection $workByUsers;
 
-    #[ORM\OneToMany(mappedBy: 'access', targetEntity: Email::class, cascade: ['persist'], orphanRemoval: true)]
-    private Collection $report;
-
     #[ORM\ManyToMany(targetEntity: Tagg::class, inversedBy: 'accesses', cascade: ['persist'])]
     #[ORM\JoinTable(name: 'tag_access')]
     #[ORM\JoinColumn(name: 'access_id', referencedColumnName: 'id')]
@@ -433,7 +431,7 @@ class Access implements UserInterface
         return $this;
     }
 
-    public function getRoles(): ?array
+    public function getRoles(): array
     {
         $roles = $this->roles;
         return array_unique($roles ?? []);
@@ -752,7 +750,7 @@ class Access implements UserInterface
 
     #[Pure] public function getUserIdentifier(): string
     {
-        return $this->person->getUsername();
+        return $this->person->getUserIdentifier();
     }
 
     public function getPassword(): ?string

src/Entity/Billing/BillingSetting.php

@@ -4,7 +4,6 @@ declare(strict_types=1);
 namespace App\Entity\Billing;
 
 use ApiPlatform\Core\Annotation\ApiResource;
-use ApiPlatform\Core\Annotation\ApiSubresource;
 use App\Entity\Organization\Organization;
 use App\Repository\Billing\BillingSettingRepository;
 use DH\Auditor\Provider\Doctrine\Auditing\Annotation\Auditable;

src/Entity/Core/File.php

@@ -264,7 +264,7 @@ class File
      * @param Person|null $person
      * @return File
      */
-    public function setPerson(?Person $person): self
+    public function setPerson(?Person $person): File
     {
         $this->person = $person;
 
@@ -283,7 +283,7 @@ class File
      * @param Organization|null $organization
      * @return File
      */
-    public function setOrganization(?Organization $organization): self
+    public function setOrganization(?Organization $organization): File
     {
         $this->organization = $organization;
 
@@ -383,7 +383,7 @@ class File
      * @param string $visibility
      * @return File
      */
-    public function setVisibility(string $visibility): self
+    public function setVisibility(string $visibility): File
     {
         $this->visibility = $visibility;
 
@@ -402,7 +402,7 @@ class File
      * @param string $type
      * @return File
      */
-    public function setType(string $type): self
+    public function setType(string $type): File
     {
         $this->type = $type;
 
@@ -421,7 +421,7 @@ class File
      * @param int|null $size
      * @return File
      */
-    public function setSize(?int $size): self
+    public function setSize(?int $size): File
     {
         $this->size = $size;
 
@@ -440,7 +440,7 @@ class File
      * @param bool $isTemporaryFile
      * @return File
      */
-    public function setIsTemporaryFile(bool $isTemporaryFile): self
+    public function setIsTemporaryFile(bool $isTemporaryFile): File
     {
         $this->isTemporaryFile = $isTemporaryFile;
 
@@ -459,7 +459,7 @@ class File
      * @param DateTime $createDate
      * @return File
      */
-    public function setCreateDate(DateTime $createDate): self
+    public function setCreateDate(DateTime $createDate): File
     {
         $this->createDate = $createDate;
 
@@ -478,7 +478,7 @@ class File
      * @param int|null $createdBy
      * @return File
      */
-    public function setCreatedBy(?int $createdBy): self
+    public function setCreatedBy(?int $createdBy): File
     {
         $this->createdBy = $createdBy;
 
@@ -497,7 +497,7 @@ class File
      * @param DateTime $updateDate
      * @return File
      */
-    public function setUpdateDate(DateTime $updateDate): self
+    public function setUpdateDate(DateTime $updateDate): File
     {
         $this->updateDate = $updateDate;
 
@@ -516,7 +516,7 @@ class File
      * @param int|null $updatedBy
      * @return File
      */
-    public function setUpdatedBy(?int $updatedBy): self
+    public function setUpdatedBy(?int $updatedBy): File
     {
         $this->updatedBy = $updatedBy;
 
@@ -535,7 +535,7 @@ class File
      * @param string|null $status
      * @return File
      */
-    public function setStatus(?string $status): self
+    public function setStatus(?string $status): File
     {
         $this->status = $status;
 

src/Entity/Core/Notification.php

@@ -125,7 +125,7 @@ class Notification
      * @param Organization|null $recipientOrganization
      * @return Notification
      */
-    public function setRecipientOrganization(?Organization $recipientOrganization): self
+    public function setRecipientOrganization(?Organization $recipientOrganization): Notification
     {
         $this->recipientOrganization = $recipientOrganization;
 

src/Entity/Education/CriteriaNotation.php

@@ -44,7 +44,7 @@ class CriteriaNotation
     private Collection $educationNotationCriteriaConfigs;
 
     #[ORM\Column]
-    #[Assert\Choice(callback: ['\AppBundle\Enum\Education\TypeCriteriaEnum', 'toArray'], message: 'invalid-type')]
+    #[Assert\Choice(callback: ['\App\Enum\Education\TypeCriteriaEnum', 'toArray'], message: 'invalid-type')]
     private string $type;
 
     #[ORM\Column]

src/Entity/Message/ReportSms.php

@@ -19,7 +19,7 @@ class ReportSms extends AbstractReport
     #[ORM\GeneratedValue]
     private ?int $id = null;
 
-    #[ORM\ManyToOne(inversedBy: 'reports')]
+    #[ORM\ManyToOne(targetEntity: Sms::class ,inversedBy: 'reports')]
     #[ORM\JoinColumn('message_id')]
     private Sms $sms;
 

src/Entity/Message/Sms.php

@@ -30,7 +30,7 @@ class Sms extends AbstractMessage
     private Access $author;
 
     #[ORM\OneToMany(mappedBy: 'sms', targetEntity: ReportSms::class, cascade: ['persist'], orphanRemoval: true)]
-    private Collection $report;
+    private Collection $reports;
 
     #[ORM\ManyToMany(targetEntity: Tagg::class, inversedBy: 'sms', cascade: ['persist'])]
     #[ORM\JoinTable(name: 'tag_message')]
@@ -41,7 +41,7 @@ class Sms extends AbstractMessage
     public function __construct()
     {
         $this->uuid = Uuid::uuid4();
-        $this->report = new ArrayCollection();
+        $this->reports = new ArrayCollection();
         $this->tags = new ArrayCollection();
     }
 
@@ -72,15 +72,15 @@ class Sms extends AbstractMessage
     /**
      * @return Collection<int, ReportSms>
      */
-    public function getReport(): Collection
+    public function getReports(): Collection
     {
-        return $this->report;
+        return $this->reports;
     }
 
     public function addReport(ReportSms $report): self
     {
-        if (!$this->report->contains($report)) {
-            $this->report[] = $report;
+        if (!$this->reports->contains($report)) {
+            $this->reports[] = $report;
             $report->setSms($this);
         }
 
@@ -89,7 +89,7 @@ class Sms extends AbstractMessage
 
     public function removeReport(ReportSms $report): self
     {
-        if ($this->report->removeElement($report)) {
+        if ($this->reports->removeElement($report)) {
             // set the owning side to null (unless already changed)
             if ($report->getSms() === $this) {
                 $report->setSms(null);

src/Entity/Organization/Organization.php

@@ -4,7 +4,6 @@ declare(strict_types=1);
 namespace App\Entity\Organization;
 
 use ApiPlatform\Core\Annotation\ApiResource;
-use ApiPlatform\Core\Annotation\ApiSubresource;
 use App\Entity\Access\Access;
 use App\Entity\Billing\BillingSetting;
 use App\Entity\Booking\Attendance;
@@ -84,7 +83,7 @@ class Organization
     private Collection $accesses;
 
     #[ORM\OneToMany(mappedBy: 'organization', targetEntity: NetworkOrganization::class, orphanRemoval: true)]
-    #[ApiSubresource]
+    #[ApiResource]
     private Collection $networkOrganizations;
 
     #[ORM\OneToMany(mappedBy: 'parent', targetEntity: NetworkOrganization::class, orphanRemoval: true)]
@@ -98,7 +97,7 @@ class Organization
     private Parameters $parameters;
 
     #[ORM\OneToOne(mappedBy: 'organization', cascade: ['persist', 'remove'], orphanRemoval: true)]
-    #[ApiSubresource]
+    #[ApiResource]
     private BillingSetting $billingSetting;
 
     #[ORM\Column(type: 'text', nullable: true)]
@@ -217,31 +216,31 @@ class Organization
     private ?File $image = null;
 
     #[ORM\ManyToMany(targetEntity: TypeOfPractice::class, mappedBy: 'organizations')]
-    #[ApiSubresource]
+    #[ApiResource]
     private Collection $typeOfPractices;
 
     #[ORM\Column(nullable: true)]
     private ?string $otherPractice = null;
 
     #[ORM\ManyToMany(targetEntity: ContactPoint::class, mappedBy: 'organization')]
-    #[ApiSubresource]
+    #[ApiResource]
     private Collection $contactPoints;
 
     #[ORM\ManyToMany(targetEntity: BankAccount::class, inversedBy: 'organization')]
     #[ORM\JoinColumn(name: 'organization_id', referencedColumnName: 'id', unique: true)]
     #[ORM\InverseJoinColumn(name: 'bankAccount_id', referencedColumnName: 'id')]
-    #[ApiSubresource]
+    #[ApiResource]
     private Collection $bankAccounts;
 
     #[ORM\OneToMany( mappedBy: 'organization', targetEntity: OrganizationAddressPostal::class, orphanRemoval: true)]
-    #[ApiSubresource]
+    #[ApiResource]
     private Collection $organizationAddressPostals;
 
     #[ORM\OneToMany(mappedBy: 'organization', targetEntity: OrganizationLicence::class, orphanRemoval: true)]
     private Collection $organizationLicences;
 
     #[ORM\OneToMany(mappedBy: 'organization', targetEntity: OrganizationArticle::class, orphanRemoval: true)]
-    #[ApiSubresource]
+    #[ApiResource]
     private Collection $organizationArticles;
 
     #[ORM\OneToMany(mappedBy: 'organization', targetEntity: Cycle::class, orphanRemoval: true)]
@@ -251,7 +250,7 @@ class Organization
     private Collection $educationTimings;
 
     #[ORM\OneToMany( mappedBy: 'organization', targetEntity: Subdomain::class)]
-    #[ApiSubresource]
+    #[ApiResource]
     private Collection $subdomains;
 
     #[ORM\ManyToOne(inversedBy: 'organizationContacts')]

src/Entity/Organization/Parameters.php

@@ -5,7 +5,6 @@ declare(strict_types=1);
 namespace App\Entity\Organization;
 
 use ApiPlatform\Core\Annotation\ApiResource;
-use ApiPlatform\Core\Annotation\ApiSubresource;
 use App\Entity\Access\Access;
 use App\Entity\Core\File;
 use App\Repository\Organization\ParametersRepository;
@@ -87,7 +86,7 @@ class Parameters
     private bool $desactivateOpentalentSiteWeb = false;
 
     #[ORM\OneToMany( mappedBy: 'publicationDirector', targetEntity: Access::class)]
-    #[ApiSubresource]
+    #[ApiResource]
     private Collection $publicationDirectors;
 
     #[ORM\Column(length: 255, nullable: true)]

src/Entity/Person/Person.php

@@ -16,6 +16,7 @@ use Doctrine\Common\Collections\ArrayCollection;
 use Doctrine\Common\Collections\Collection;
 use Doctrine\ORM\Mapping as ORM;
 use JetBrains\PhpStorm\Pure;
+use Symfony\Component\Security\Core\User\PasswordAuthenticatedUserInterface;
 use Symfony\Component\Security\Core\User\UserInterface;
 use Symfony\Component\Validator\Constraints as Assert;
 use Symfony\Component\Serializer\Annotation\Groups;
@@ -31,7 +32,7 @@ use Symfony\Component\Serializer\Annotation\Groups;
     ]
 )]
 #[ORM\Entity(repositoryClass: PersonRepository::class)]
-class Person implements UserInterface
+class Person implements UserInterface, PasswordAuthenticatedUserInterface
 {
     #[ORM\Id]
     #[ORM\Column]
@@ -130,7 +131,7 @@ class Person implements UserInterface
         return (string) $this->username;
     }
 
-    public function getUserIdentifier(): ?string
+    public function getUserIdentifier(): string
     {
         return (string) $this->username;
     }
@@ -225,7 +226,7 @@ class Person implements UserInterface
     }
 
     /**
-     * @return Collection|ContactPoint[]
+     * @return Collection
      */
     public function getContactPoints(): Collection
     {

src/Entity/Public/FederationStructure.php

@@ -135,7 +135,7 @@ class FederationStructure
      * @param int $id
      * @return FederationStructure
      */
-    public function setId(int $id): self
+    public function setId(int $id): FederationStructure
     {
         $this->id = $id;
         return $this;
@@ -153,7 +153,7 @@ class FederationStructure
      * @param string $name
      * @return FederationStructure
      */
-    public function setName(string $name): self
+    public function setName(string $name): FederationStructure
     {
         $this->name = $name;
 
@@ -172,7 +172,7 @@ class FederationStructure
      * @param int|null $logoId
      * @return FederationStructure
      */
-    public function setLogoId(?int $logoId): self
+    public function setLogoId(?int $logoId): FederationStructure
     {
         $this->logoId = $logoId;
         return $this;
@@ -190,7 +190,7 @@ class FederationStructure
      * @param string|null $description
      * @return FederationStructure
      */
-    public function setDescription(?string $description): self
+    public function setDescription(?string $description): FederationStructure
     {
         $this->description = $description;
 
@@ -209,7 +209,7 @@ class FederationStructure
      * @param int|null $imageId
      * @return FederationStructure
      */
-    public function setImageId(?int $imageId): self
+    public function setImageId(?int $imageId): FederationStructure
     {
         $this->imageId = $imageId;
 
@@ -228,7 +228,7 @@ class FederationStructure
      * @param string|null $type
      * @return FederationStructure
      */
-    public function setType(?string $type): self
+    public function setType(?string $type): FederationStructure
     {
         $this->type = $type;
         return $this;
@@ -246,7 +246,7 @@ class FederationStructure
      * @param string|null $website
      * @return FederationStructure
      */
-    public function setWebsite(?string $website): self
+    public function setWebsite(?string $website): FederationStructure
     {
         $this->website = $website;
         return $this;
@@ -264,7 +264,7 @@ class FederationStructure
      * @param array $addresses
      * @return FederationStructure
      */
-    public function setAddresses(array $addresses): self
+    public function setAddresses(array $addresses): FederationStructure
     {
         $this->addresses = $addresses;
         return $this;
@@ -282,7 +282,7 @@ class FederationStructure
      * @param string|null $phone
      * @return FederationStructure
      */
-    public function setPhone(?string $phone): self
+    public function setPhone(?string $phone): FederationStructure
     {
         $this->phone = $phone;
         return $this;
@@ -300,7 +300,7 @@ class FederationStructure
      * @param string|null $mobilePhone
      * @return FederationStructure
      */
-    public function setMobilePhone(?string $mobilePhone): self
+    public function setMobilePhone(?string $mobilePhone): FederationStructure
     {
         $this->mobilePhone = $mobilePhone;
         return $this;
@@ -390,7 +390,7 @@ class FederationStructure
      * @param string|null $youtube
      * @return FederationStructure
      */
-    public function setYoutube(?string $youtube): self
+    public function setYoutube(?string $youtube): FederationStructure
     {
         $this->youtube = $youtube;
 
@@ -409,7 +409,7 @@ class FederationStructure
      * @param array|null $articles
      * @return FederationStructure
      */
-    public function setArticles(?array $articles): self
+    public function setArticles(?array $articles): FederationStructure
     {
         $this->articles = $articles;
         return $this;
@@ -427,7 +427,7 @@ class FederationStructure
      * @param array|null $practices
      * @return FederationStructure
      */
-    public function setPractices(?array $practices): self
+    public function setPractices(?array $practices): FederationStructure
     {
         $this->practices = $practices;
         return $this;
@@ -445,7 +445,7 @@ class FederationStructure
      * @param int|null $parentId
      * @return FederationStructure
      */
-    public function setParentId(?int $parentId): self
+    public function setParentId(?int $parentId): FederationStructure
     {
         $this->parentId = $parentId;
         return $this;
@@ -463,7 +463,7 @@ class FederationStructure
      * @param string|null $parentName
      * @return FederationStructure
      */
-    public function setParentName(?string $parentName): self
+    public function setParentName(?string $parentName): FederationStructure
     {
         $this->parentName = $parentName;
         return $this;
@@ -481,7 +481,7 @@ class FederationStructure
      * @param array $parents
      * @return FederationStructure
      */
-    public function setParents(array $parents): self
+    public function setParents(array $parents): FederationStructure
     {
         $this->parents = $parents;
         return $this;

src/Entity/Public/PublicEvent.php

@@ -120,7 +120,7 @@ class PublicEvent
      * @param string $uuid
      * @return PublicEvent
      */
-    public function setUuid(string $uuid): self
+    public function setUuid(string $uuid): PublicEvent
     {
         $this->uuid = $uuid;
         return $this;
@@ -138,7 +138,7 @@ class PublicEvent
      * @param int|null $organizationId
      * @return PublicEvent
      */
-    public function setOrganizationId(?int $organizationId): self
+    public function setOrganizationId(?int $organizationId): PublicEvent
     {
         $this->organizationId = $organizationId;
         return $this;
@@ -156,7 +156,7 @@ class PublicEvent
      * @param string $name
      * @return PublicEvent
      */
-    public function setName(string $name): self
+    public function setName(string $name): PublicEvent
     {
         $this->name = $name;
         return $this;
@@ -174,7 +174,7 @@ class PublicEvent
      * @param string|null $description
      * @return PublicEvent
      */
-    public function setDescription(?string $description): self
+    public function setDescription(?string $description): PublicEvent
     {
         $this->description = $description;
         return $this;
@@ -192,7 +192,7 @@ class PublicEvent
      * @param string|null $url
      * @return PublicEvent
      */
-    public function setUrl(?string $url): self
+    public function setUrl(?string $url): PublicEvent
     {
         $this->url = $url;
         return $this;
@@ -210,7 +210,7 @@ class PublicEvent
      * @param \DateTime $datetimeStart
      * @return PublicEvent
      */
-    public function setDatetimeStart(\DateTime $datetimeStart): self
+    public function setDatetimeStart(\DateTime $datetimeStart): PublicEvent
     {
         $this->datetimeStart = $datetimeStart;
         return $this;
@@ -228,7 +228,7 @@ class PublicEvent
      * @param \DateTime $datetimeEnd
      * @return PublicEvent
      */
-    public function setDatetimeEnd(\DateTime $datetimeEnd): self
+    public function setDatetimeEnd(\DateTime $datetimeEnd): PublicEvent
     {
         $this->datetimeEnd = $datetimeEnd;
         return $this;
@@ -246,7 +246,7 @@ class PublicEvent
      * @param string|null $city
      * @return PublicEvent
      */
-    public function setCity(?string $city): self
+    public function setCity(?string $city): PublicEvent
     {
         $this->city = $city;
         return $this;
@@ -264,7 +264,7 @@ class PublicEvent
      * @param string|null $postalCode
      * @return PublicEvent
      */
-    public function setPostalCode(?string $postalCode): self
+    public function setPostalCode(?string $postalCode): PublicEvent
     {
         $this->postalCode = $postalCode;
         return $this;
@@ -282,7 +282,7 @@ class PublicEvent
      * @param string|null $streetAddress
      * @return PublicEvent
      */
-    public function setStreetAddress(?string $streetAddress): self
+    public function setStreetAddress(?string $streetAddress): PublicEvent
     {
         $this->streetAddress = $streetAddress;
         return $this;
@@ -300,7 +300,7 @@ class PublicEvent
      * @param float|null $longitude
      * @return PublicEvent
      */
-    public function setLongitude(?float $longitude): self
+    public function setLongitude(?float $longitude): PublicEvent
     {
         $this->longitude = $longitude;
         return $this;
@@ -318,7 +318,7 @@ class PublicEvent
      * @param float|null $latitude
      * @return PublicEvent
      */
-    public function setLatitude(?float $latitude): self
+    public function setLatitude(?float $latitude): PublicEvent
     {
         $this->latitude = $latitude;
         return $this;
@@ -336,7 +336,7 @@ class PublicEvent
      * @param string|null $roomName
      * @return PublicEvent
      */
-    public function setRoomName(?string $roomName): self
+    public function setRoomName(?string $roomName): PublicEvent
     {
         $this->roomName = $roomName;
         return $this;
@@ -354,7 +354,7 @@ class PublicEvent
      * @param string|null $roomDescription
      * @return PublicEvent
      */
-    public function setRoomDescription(?string $roomDescription): self
+    public function setRoomDescription(?string $roomDescription): PublicEvent
     {
         $this->roomDescription = $roomDescription;
         return $this;
@@ -372,7 +372,7 @@ class PublicEvent
      * @param string|null $roomLocalisation
      * @return PublicEvent
      */
-    public function setRoomLocalisation(?string $roomLocalisation): self
+    public function setRoomLocalisation(?string $roomLocalisation): PublicEvent
     {
         $this->roomLocalisation = $roomLocalisation;
         return $this;
@@ -390,7 +390,7 @@ class PublicEvent
      * @param string|null $roomCapacity
      * @return PublicEvent
      */
-    public function setRoomCapacity(?string $roomCapacity): self
+    public function setRoomCapacity(?string $roomCapacity): PublicEvent
     {
         $this->roomCapacity = $roomCapacity;
         return $this;
@@ -408,7 +408,7 @@ class PublicEvent
      * @param string|null $roomFloorSize
      * @return PublicEvent
      */
-    public function setRoomFloorSize(?string $roomFloorSize): self
+    public function setRoomFloorSize(?string $roomFloorSize): PublicEvent
     {
         $this->roomFloorSize = $roomFloorSize;
         return $this;
@@ -426,7 +426,7 @@ class PublicEvent
      * @param string|null $imageUrl
      * @return PublicEvent
      */
-    public function setImageUrl(?string $imageUrl): self
+    public function setImageUrl(?string $imageUrl): PublicEvent
     {
         $this->imageUrl = $imageUrl;
         return $this;
@@ -444,7 +444,7 @@ class PublicEvent
      * @param string|null $thumbnailUrl
      * @return PublicEvent
      */
-    public function setThumbnailUrl(?string $thumbnailUrl): self
+    public function setThumbnailUrl(?string $thumbnailUrl): PublicEvent
     {
         $this->thumbnailUrl = $thumbnailUrl;
         return $this;
@@ -462,7 +462,7 @@ class PublicEvent
      * @param array|null $categories
      * @return PublicEvent
      */
-    public function setCategories(?array $categories): self
+    public function setCategories(?array $categories): PublicEvent
     {
         $this->categories = $categories;
         return $this;
@@ -480,7 +480,7 @@ class PublicEvent
      * @param string $origin
      * @return PublicEvent
      */
-    public function setOrigin(string $origin): self
+    public function setOrigin(string $origin): PublicEvent
     {
         $this->origin = $origin;
         return $this;
@@ -498,7 +498,7 @@ class PublicEvent
      * @param int $entityId
      * @return PublicEvent
      */
-    public function setEntityId(int $entityId): self
+    public function setEntityId(int $entityId): PublicEvent
     {
         $this->entityId = $entityId;
         return $this;

src/EventSubscriber/ApiResourcesValidatorSubscriber.php

@@ -24,7 +24,7 @@ final class ApiResourcesValidatorSubscriber implements EventSubscriberInterface
      * ne se déclenche qu'après le post validate d'api platform
      * @return array[]
      */
-    public static function getSubscribedEvents()
+    public static function getSubscribedEvents(): array
     {
         return [
             KernelEvents::VIEW => ['validate', EventPriorities::POST_VALIDATE],

src/EventSubscriber/ElasticaPostTransformSubscriber.php

@@ -25,7 +25,7 @@ final class ElasticaPostTransformSubscriber implements EventSubscriberInterface
     /**
      * ne se déclenche qu'après le post validate d'api platform
      */
-    public static function getSubscribedEvents()
+    public static function getSubscribedEvents(): array
     {
         return [
             PreTransformEvent::class => 'doPostTransform',

src/Filter/DoctrineFilter/AbstractTimeFilter.php

@@ -22,7 +22,8 @@ abstract class AbstractTimeFilter extends SQLFilter
      * @param array $fields
      * @return string
      */
-    protected function constructQuery(array $constraints, string $targetTableAlias, array $fields): string{
+    protected function constructQuery(array $constraints, string $targetTableAlias, array $fields): string
+    {
         $queryConditionsAND = [];
         foreach ($constraints as $key => $constraint) {
             $queryConditionsOR = [];
@@ -48,7 +49,8 @@ abstract class AbstractTimeFilter extends SQLFilter
      * @return string|null
      * @see DateTimeFilterTest::testGetArithmeticValue()
      */
-    protected function getArithmeticValue($condition): string|null{
+    protected function getArithmeticValue($condition): ?string
+    {
         switch ($condition){
             case DateTimeConstraint::INF : return '<';
             case DateTimeConstraint::EQUAL :  return '=';

src/Repository/Access/AccessRepository.php

@@ -11,6 +11,7 @@ use Doctrine\Bundle\DoctrineBundle\Repository\ServiceEntityRepository;
 use Doctrine\Persistence\ManagerRegistry;
 use Symfony\Bridge\Doctrine\Security\User\UserLoaderInterface;
 use Symfony\Component\HttpFoundation\RequestStack;
+use Symfony\Component\Security\Core\User\UserInterface;
 
 /**
  * @method Access|null find($id, $lockMode = null, $lockVersion = null)
@@ -28,25 +29,15 @@ class AccessRepository extends ServiceEntityRepository implements UserLoaderInte
         parent::__construct($registry, Access::class);
     }
 
-    /**
-     * @deprecated  in symfony 6.0, replace by loadUserByIdentifier
-     * @param string $username
-     * @return \Symfony\Component\Security\Core\User\UserInterface|void|null
-     */
-    public function loadUserByUsername(string $username)
-    {}
-
     /**
      * Méthode permettant de fournir un userProvider custom (voir config provider : access_provider)
-     * @param string $username
-     * @return Access|mixed|object|\Symfony\Component\Security\Core\User\UserInterface|null
-     * @throws \Doctrine\ORM\ORMException
-     * @throws \Doctrine\ORM\OptimisticLockException
-     * @throws \Doctrine\ORM\TransactionRequiredException
+     * @param string $identifier
+     * @return UserInterface|null
+     * @throws \Doctrine\ORM\NonUniqueResultException
      */
-    public function loadUserByIdentifier($identifier): mixed
+    public function loadUserByIdentifier($identifier): ?UserInterface
     {
-        if($this->requestStack->getMainRequest()->headers->get(self::HTTP_X_SWITCH_USER) == $identifier)
+        if($this->requestStack->getMainRequest()->headers->get(self::HTTP_X_SWITCH_USER) === $identifier)
             return $this->getEntityManager()->find(Access::class, $identifier);
 
         return $this->findAccessByUsernameAndAccessId($identifier, $this->requestStack->getMainRequest()->headers->get(self::ACCESS_NAME_HEADER));
@@ -78,7 +69,7 @@ class AccessRepository extends ServiceEntityRepository implements UserLoaderInte
      * @return mixed
      * @throws \Exception
      */
-    public function findAllValidAccesses(Access $access): array
+    public function findAllValidAccesses(Access $access): mixed
     {
         $datetime = new \DateTime();
         $today = $datetime->format('Y-m-d');

src/Repository/Booking/CourseRepository.php

@@ -22,7 +22,8 @@ class CourseRepository extends ServiceEntityRepository
      * @param \DateTimeInterface $end
      * @return array
      */
-    public function getCoursesToFrom(Organization $organization, \DateTimeInterface $start, \DateTimeInterface $end): array{
+    public function getCoursesToFrom(Organization $organization, \DateTimeInterface $start, \DateTimeInterface $end): array
+    {
         $this->_em->getFilters()->disable('activity_year_filter');
 
         $queryBuilder = $this->createQueryBuilder('course');

src/Repository/Core/ContactPointRepository.php

@@ -28,7 +28,8 @@ class ContactPointRepository extends ServiceEntityRepository
      * @param Organization $organization
      * @return array
      */
-    public function getByTypeAndOrganization(String $type, Organization $organization): array{
+    public function getByTypeAndOrganization(String $type, Organization $organization): array
+    {
         return $this->createQueryBuilder('contact_point')
             ->innerJoin('contact_point.organization', 'organization')
             ->where('contact_point.contactType = :type')
@@ -46,7 +47,8 @@ class ContactPointRepository extends ServiceEntityRepository
      * @param Person $person
      * @return array
      */
-    public function getByTypeAndPerson(String $type, Person $person): array {
+    public function getByTypeAndPerson(String $type, Person $person): array
+    {
         return $this->createQueryBuilder('contact_point')
             ->innerJoin('contact_point.person', 'person')
             ->where('contact_point.contactType = :type')

src/Repository/Cotisation/CotisationApiResourcesRepository.php

@@ -19,7 +19,8 @@ class CotisationApiResourcesRepository
      * @param int $year
      * @return int|null
      */
-    public function getAffiliationState(int $organizationId, int $year): int|null {
+    public function getAffiliationState(int $organizationId, int $year): ?int
+    {
         $rsm = new ResultSetMapping();
         $rsm->addScalarResult('oa_miscellaneous_state_sta', 'oa_miscellaneous_state_sta');
 
@@ -30,7 +31,7 @@ class CotisationApiResourcesRepository
         $result = $query->getOneOrNullResult();
 
         if (!empty($result)) {
-            return intval($result['oa_miscellaneous_state_sta']);
+            return (int) $result['oa_miscellaneous_state_sta'];
         }
         return null;
     }
@@ -41,7 +42,8 @@ class CotisationApiResourcesRepository
      * @param int $year
      * @return bool
      */
-    public function isInsuranceNotDone(int $organizationId, int $year): bool {
+    public function isInsuranceNotDone(int $organizationId, int $year): bool
+    {
         $rsm = new ResultSetMapping();
         $rsm->addScalarResult('done', 'done');
 
@@ -54,11 +56,7 @@ class CotisationApiResourcesRepository
         $query = $this->adminassosEntityManager->createNativeQuery($sql, $rsm);
         $result = $query->getOneOrNullResult();
 
-        if (!empty($result)) {
-            return $result['done'] == 0;
-        } else {
-            return false;
-        }
+        return !empty($result) && $result['done'] === 0;
     }
 
     /**
@@ -66,7 +64,8 @@ class CotisationApiResourcesRepository
      * @param int $organizationId
      * @return bool
      */
-    public function isNotDGVCustomer(int $organizationId):bool {
+    public function isNotDGVCustomer(int $organizationId): bool
+    {
         $rsm = new ResultSetMapping();
         $rsm->addScalarResult('total', 'total');
 
@@ -78,10 +77,6 @@ class CotisationApiResourcesRepository
         $query = $this->adminassosEntityManager->createNativeQuery($sql, $rsm);
         $result = $query->getOneOrNullResult();
 
-        if (!empty($result)) {
-            return $result['total'] == 0;
-        } else {
-            return true;
-        }
+        return !empty($result) && $result['total'] === 0;
     }
 }

src/Repository/Network/NetworkOrganizationRepository.php

@@ -74,7 +74,7 @@ class NetworkOrganizationRepository extends ServiceEntityRepository
      *
      * @return array The organizations.
      */
-    public function findAllParents(Organization $organization): ?array
+    public function findAllParents(Organization $organization): array
     {
         $sql = sprintf("
             SELECT *

src/Repository/Organization/OrganizationAddressPostalRepository.php

@@ -27,7 +27,8 @@ class OrganizationAddressPostalRepository extends ServiceEntityRepository
      * @param Organization $organization
      * @return array|null
      */
-    public function getByType(String $type, Organization $organization): ?array{
+    public function getByType(String $type, Organization $organization): ?array
+    {
         return $this->createQueryBuilder('organizationAddressPostal')
             ->where('organizationAddressPostal.type = :type')
             ->andWhere('organizationAddressPostal.organization = :organization')

src/Repository/Person/PersonRepository.php

@@ -7,6 +7,7 @@ use App\Entity\Person\Person;
 use Doctrine\Bundle\DoctrineBundle\Repository\ServiceEntityRepository;
 use Doctrine\Persistence\ManagerRegistry;
 use Symfony\Component\Security\Core\Exception\UnsupportedUserException;
+use Symfony\Component\Security\Core\User\PasswordAuthenticatedUserInterface;
 use Symfony\Component\Security\Core\User\PasswordUpgraderInterface;
 use Symfony\Component\Security\Core\User\UserInterface;
 
@@ -26,7 +27,7 @@ final class PersonRepository extends ServiceEntityRepository implements Password
     /**
      * Used to upgrade (rehash) the user's password automatically over time.
      */
-    public function upgradePassword(UserInterface $user, string $newEncodedPassword): void
+    public function upgradePassword(PasswordAuthenticatedUserInterface $user, string $newEncodedPassword): void
     {
         if (!$user instanceof Person) {
             throw new UnsupportedUserException(sprintf('Instances of "%s" are not supported.', \get_class($user)));

src/Security/Voter/ModuleVoter.php

@@ -75,7 +75,8 @@ class ModuleVoter extends Voter
      * @param string $module
      * @return bool
      */
-    private function isOrganizationHaveThisModule(Organization $organization, string $module): bool{
+    private function isOrganizationHaveThisModule(Organization $organization, string $module): bool
+    {
         $organizationModules = $this->module->getOrganizationModules($organization);
         return in_array($module, $organizationModules);
     }

src/Serializer/DefaultNormalizer.php

@@ -27,23 +27,22 @@ final class DefaultNormalizer implements NormalizerInterface, DenormalizerInterf
         }
     }
 
-    public function supportsNormalization($data, $format = null)
+    public function supportsNormalization($data, $format = null): bool
     {
         return $this->decorated->supportsNormalization($data, $format);
     }
 
-    public function normalize($object, $format = null, array $context = [])
+    public function normalize($object, $format = null, array $context = []): array|string|int|float|bool|\ArrayObject|null
     {
-        $data = $this->decorated->normalize($object, $format, $context);
-        return $data;
+        return $this->decorated->normalize($object, $format, $context);
     }
 
-    public function supportsDenormalization($data, $type, $format = null)
+    public function supportsDenormalization($data, $type, $format = null): bool
     {
         return $this->decorated->supportsDenormalization($data, $type, $format);
     }
 
-    public function denormalize($data, $class, $format = null, array $context = [])
+    public function denormalize($data, $class, $format = null, array $context = []): mixed
     {
         $entity = $this->decorated->denormalize($data, $class, $format, $context);
 

src/Service/Access/AccessProfileCreator.php

@@ -31,7 +31,8 @@ class AccessProfileCreator
      * @throws \Exception
      * @see AccessProfileCreatorTest::testGetAccessProfileFailed
      */
-    public function getAccessProfile(Access $access, Access $originalAccess = null): AccessProfile{
+    public function getAccessProfile(Access $access, Access $originalAccess = null): AccessProfile
+    {
         $validAccesses = $this->accessRepository->findAllValidAccesses($access);
         if(empty($validAccesses))
             throw new AuthenticationException('no_valid_access', 401);
@@ -65,7 +66,8 @@ class AccessProfileCreator
      * @return AccessProfile
      * @see AccessProfileCreatorTest::testCreateAccessProfile
      */
-    public function createCompleteAccessProfile(Access $access): AccessProfile{
+    public function createCompleteAccessProfile(Access $access): AccessProfile
+    {
         $accessProfile = $this->createLightAccessProfile($access);
         return $accessProfile
             ->setIsAdminAccess($access->getAdminAccess())
@@ -86,7 +88,8 @@ class AccessProfileCreator
      * @return AccessProfile
      * @see AccessProfileCreatorTest::testCreateLightAccessProfile()
      */
-    public function createLightAccessProfile(Access $access): AccessProfile{
+    public function createLightAccessProfile(Access $access): AccessProfile
+    {
         $accessProfile = new AccessProfile();
         return $accessProfile
             ->setId($access->getId())

src/Service/Access/AdminAccessUtils.php

@@ -28,7 +28,8 @@ class AdminAccessUtils
      * @throws \Exception
      * @see AdminAccessUtilsTest::testGetAdminAccess()
      */
-    public function getAdminAccess(Organization $organization): ?AdminAccess{
+    public function getAdminAccess(Organization $organization): ?AdminAccess
+    {
         $administrator = $this->accessUtils->findAdminFor($organization);
         if(!$administrator) return null;
 
@@ -38,7 +39,7 @@ class AdminAccessUtils
         $adminAccess = new AdminAccess();
         $adminAccess
             ->setId($administrator->getId())
-            ->setUsername($administrator->getPerson()->getUsername())
+            ->setUsername($administrator->getPerson()->getUserIdentifier())
             ->setEmail($contactPoint->getEmail())
         ;
         return $adminAccess;

src/Service/Access/Utils.php

@@ -32,7 +32,8 @@ class Utils
      * @see UtilsTest::testFilterAccesses()
      * TODO: est-ce qu'on essaierait pas de trouver un nom plus explicite? j'ai pas trop d'idée la tout de suite cela dit
      */
-    public function filterAccesses(array $accesses, Access $access): array {
+    public function filterAccesses(array $accesses, Access $access): array
+    {
         return array_filter($accesses, function($a) use($access){
             /** @var Access $a */
             return $a->getId() !== $access->getId();
@@ -46,7 +47,8 @@ class Utils
      * @return bool
      * @see UtilsTest::testHasRoles()
      */
-    public function hasRoles(Access $access, string $roleToHave): bool{
+    public function hasRoles(Access $access, string $roleToHave): bool
+    {
         return in_array($roleToHave, $this->getAllRoles($access), true);
     }
 
@@ -56,7 +58,8 @@ class Utils
      * @return array
      * @see UtilsTest::testGetAllRoles()
      */
-    public function getAllRoles(Access $access): array {
+    public function getAllRoles(Access $access): array
+    {
         $roles = $this->optionalsRolesIterator->getOptionalsRoles($access);
         return $this->roleHierarchy->getReachableRoleNames(array_merge($access->getRoles(), $roles));
     }
@@ -68,7 +71,8 @@ class Utils
      * @return Access|null
      * @see UtilsTest::testGetAdminAccess()
      */
-    public function findAdminFor(Organization $organization): Access|null{
+    public function findAdminFor(Organization $organization): ?Access
+    {
         return $this->accessRepository->findOneBy([
             'adminAccess' => true,
             'organization' => $organization

src/Service/ApiResourceBuilder/Mobyt/MobytUserStatusBuilder.php

@@ -20,7 +20,8 @@ class MobytUserStatusBuilder
      * @return MobytUserStatus
      * @throws \JsonException
      */
-    public function getUserStatus(int $organizationId): MobytUserStatus {
+    public function getUserStatus(int $organizationId): MobytUserStatus
+    {
         $userStatus = new MobytUserStatus();
         $userStatus->setOrganizationId($organizationId);
 

src/Service/Constraint/AbstractTimeConstraintUtils.php

@@ -23,7 +23,8 @@ abstract class AbstractTimeConstraintUtils
      * @return bool
      * @see DateTimeConstraintTest::testHasCustomPeriods()
      */
-    protected function hasCustomPeriods($historical): bool{
+    protected function hasCustomPeriods($historical): bool
+    {
         return array_key_exists('dateStart', $historical) && $historical['dateStart'] && array_key_exists('dateEnd', $historical) && $historical['dateEnd'];
     }
 
@@ -34,7 +35,8 @@ abstract class AbstractTimeConstraintUtils
      * @return array
      * @see DateTimeConstraintTest::testAddConstraint()
      */
-    protected function addConstraint(array $contraints, array $newContraint): array {
+    protected function addConstraint(array $contraints, array $newContraint): array
+    {
         $contraints = $this->mergeConstraint($contraints,$newContraint,self::START_KEY);
         return $this->mergeConstraint($contraints,$newContraint,self::END_KEY);
     }
@@ -46,7 +48,8 @@ abstract class AbstractTimeConstraintUtils
      * @param string $key
      * @return array
      */
-    protected function mergeConstraint(array $contraints, array $newContraint, string $key): array{
+    protected function mergeConstraint(array $contraints, array $newContraint, string $key): array
+    {
         if(array_key_exists($key, $newContraint)){
             foreach ($newContraint[$key] as $dateKey => $arithmeticValue){
                 //Si la date à déjà des conditions
@@ -69,7 +72,8 @@ abstract class AbstractTimeConstraintUtils
      * @return array
      * @see DateTimeConstraintTest::testCleanConstraints()
      */
-    protected function cleanConstraints(array $constraints): array{
+    protected function cleanConstraints(array $constraints): array
+    {
         $constraints[self::START_KEY] = $this->filterConstraint($constraints, self::START_KEY);
         $constraints[self::START_KEY] = $this->clearNull($constraints, self::START_KEY);
 
@@ -86,7 +90,8 @@ abstract class AbstractTimeConstraintUtils
      * @return array
      * @see DateTimeConstraintTest::testFilterConstraint()
      */
-    protected function filterConstraint(array $constraints, string $key): array{
+    protected function filterConstraint(array $constraints, string $key): array
+    {
         return array_filter($constraints[$key], function($constraint){
             return array_sum($constraint) < self::CANCEL_OPERATION ;
         });
@@ -100,7 +105,8 @@ abstract class AbstractTimeConstraintUtils
      * @return array
      * @see DateTimeConstraintTest::testClearNull()
      */
-    protected function clearNull(array $constraints, string $key): array{
+    protected function clearNull(array $constraints, string $key): array
+    {
         if(count($constraints[$key]) === 1 && array_key_exists(self::NULL_VALUE, $constraints[$key]))
             $constraints[$key] = [];
         return $constraints[$key];

src/Service/Constraint/ActivityYearConstraint.php

@@ -55,7 +55,8 @@ class ActivityYearConstraint extends AbstractTimeConstraintUtils
      * @param string $dateEnd
      * @return string[]
      */
-    protected function getRangeYear(Access $access, string $dateStart, string $dateEnd): array {
+    protected function getRangeYear(Access $access, string $dateStart, string $dateEnd): array
+    {
         $organization = $access->getOrganization();
         return [
             OrganizationUtils::START_DATE_KEY => $this->organizationUtils->getActivityYearSwitchDate($organization, new \DateTime($dateStart)),
@@ -69,7 +70,8 @@ class ActivityYearConstraint extends AbstractTimeConstraintUtils
      * @param int $year
      * @return array
      */
-    protected function presentConstraint(int $year): array {
+    protected function presentConstraint(int $year): array
+    {
         return [
           self::START_KEY => [
               $year => self::EQUAL
@@ -83,7 +85,8 @@ class ActivityYearConstraint extends AbstractTimeConstraintUtils
      * @param $year
      * @return array
      */
-    protected function pastConstraint($year): array {
+    protected function pastConstraint($year): array
+    {
         return [
             self::END_KEY => [
                 $year => self::INF
@@ -97,7 +100,8 @@ class ActivityYearConstraint extends AbstractTimeConstraintUtils
      * @param $year
      * @return array
      */
-    protected function futureConstraint($year): array {
+    protected function futureConstraint($year): array
+    {
         return [
             self::START_KEY => [
                 $year => self::SUP
@@ -114,7 +118,8 @@ class ActivityYearConstraint extends AbstractTimeConstraintUtils
      * @param $years
      * @return array
      */
-    protected function customConstraint($years): array {
+    protected function customConstraint($years): array
+    {
         return [
             self::START_KEY => [
                 $years[OrganizationUtils::START_DATE_KEY]  => self::SUP + self::EQUAL,

src/Service/Constraint/DateTimeConstraint.php

@@ -27,7 +27,7 @@ class DateTimeConstraint extends AbstractTimeConstraintUtils
      * @return array
      * @throws \Exception
      */
-    public function invoke(int $accessID): array  // TODO: pas moyen de refactorer avec la méthode invoke de ActivityYearConstraint?
+    public function invoke(int $accessID): array
     {
         $access = $this->entityManager->getRepository(Access::class)->find($accessID);
         $historical = $access->getHistorical();
@@ -57,7 +57,8 @@ class DateTimeConstraint extends AbstractTimeConstraintUtils
      * @param string $dateEnd
      * @return string[]
      */
-    protected function getCustomPeriods(string $dateStart, string $dateEnd): array {
+    protected function getCustomPeriods(string $dateStart, string $dateEnd): array
+    {
         return [
             OrganizationUtils::START_DATE_KEY => $dateStart,
             OrganizationUtils::END_DATE_KEY => $dateEnd
@@ -72,7 +73,8 @@ class DateTimeConstraint extends AbstractTimeConstraintUtils
      * @throws \Exception
      * @see DateTimeConstraintTest::testGetPeriodsToday()
      */
-    protected function getPeriods(Access $access): array {
+    protected function getPeriods(Access $access): array
+    {
         $organization = $access->getOrganization();
         $activityYear = $access->getActivityYear();
         $currentActivityYear = $this->organizationUtils->getOrganizationCurrentActivityYear($organization);
@@ -99,7 +101,8 @@ class DateTimeConstraint extends AbstractTimeConstraintUtils
      *
      * @see DateTimeConstraintTest::testPresentConstrain()
      */
-    protected function presentConstraint(array $periods): array {
+    protected function presentConstraint(array $periods): array
+    {
         return [
           self::START_KEY => [
               $periods[OrganizationUtils::END_DATE_KEY] => self::INF + self::EQUAL
@@ -118,7 +121,8 @@ class DateTimeConstraint extends AbstractTimeConstraintUtils
      * @return array
      * @see DateTimeConstraintTest::testPastConstrain()
      */
-    protected function pastConstraint($periods): array{
+    protected function pastConstraint($periods): array
+    {
         return [
             self::END_KEY => [
                 $periods[OrganizationUtils::START_DATE_KEY] => self::INF
@@ -133,7 +137,8 @@ class DateTimeConstraint extends AbstractTimeConstraintUtils
      * @return array
      * @see DateTimeConstraintTest::testFuturConstrain()
      */
-    protected function futureConstraint($periods): array{
+    protected function futureConstraint($periods): array
+    {
         return [
             self::START_KEY => [
                 $periods[OrganizationUtils::END_DATE_KEY] => self::SUP

src/Service/Core/AddressPostalUtils.php

@@ -13,7 +13,8 @@ class AddressPostalUtils
      * @param string $separator
      * @return string
      */
-    public function getFullStreetAddress(AddressPostal $addressPostal, string $separator = "\n"): string {
+    public function getFullStreetAddress(AddressPostal $addressPostal, string $separator = "\n"): string
+    {
         return implode($separator, array_filter([
             trim($addressPostal->getStreetAddress()),
             trim($addressPostal->getStreetAddressSecond()),

src/Service/Core/ContactPointUtils.php

@@ -26,7 +26,8 @@ class ContactPointUtils
      * @return ContactPoint|null
      * @see ContactPointUtilsTest::testGetPersonContactPointPrincipal()
      */
-    public function getPersonContactPointPrincipal(Access $access): ?ContactPoint {
+    public function getPersonContactPointPrincipal(Access $access): ?ContactPoint
+    {
         $contactPoints = $this->contactPointRepository->getByTypeAndPerson(
             ContactPointTypeEnum::PRINCIPAL()->getValue(),
             $access->getPerson()

src/Service/Cotisation/Utils.php

@@ -93,7 +93,7 @@ class Utils
      * @return string|null
      * @see UtilsTest::testGetAlertStateAffiliation()
      */
-    public function getAlertState(Organization $organization, int $year)
+    public function getAlertState(Organization $organization, int $year): ?string
     {
         $state = $this->cotisationApiResourcesRepository->getAffiliationState($organization->getId(), $year);
 
@@ -119,7 +119,8 @@ class Utils
      * @throws \Exception
      * @see UtilsTest::testGetCurrentCotisationYear()
      */
-    public function getCurrentCotisationYear(): int {
+    public function getCurrentCotisationYear(): int
+    {
         $date = DatesUtils::new('now');
         $year = (int)$date->format('Y');
 

src/Service/Cron/Job/CleanDb.php

@@ -134,7 +134,8 @@ class CleanDb extends BaseCronJob implements CronjobInterface
      * @return int
      * @throws \Doctrine\DBAL\Exception
      */
-    protected function purgeMessages(DateTime $maxDate): int {
+    protected function purgeMessages(DateTime $maxDate): int
+    {
         $q = $this->connection->createQueryBuilder();
         $q->delete('Message')->where($q->expr()->lt('dateSent', $maxDate->format('Y-m-d')));
 
@@ -150,7 +151,8 @@ class CleanDb extends BaseCronJob implements CronjobInterface
      * @return int
      * @throws \Doctrine\DBAL\Exception
      */
-    protected function purgeLoginLog(DateTime $maxDate): int {
+    protected function purgeLoginLog(DateTime $maxDate): int
+    {
         $q = $this->connection->createQueryBuilder();
         $q->delete('LoginLog')->where($q->expr()->lt('date', $maxDate->format('Y-m-d')));
 

src/Service/Cron/Job/CleanTempFiles.php

@@ -95,7 +95,8 @@ class CleanTempFiles extends BaseCronJob implements CronjobInterface
      * @return Collection
      * @throws Exception
      */
-    protected function listFilesToDelete(): Collection {
+    protected function listFilesToDelete(): Collection
+    {
 
         $maxDate = DatesUtils::new();
         $maxDate->sub(new \DateInterval('P' . self::DELETE_OLDER_THAN . 'D'));

src/Service/Dolibarr/DolibarrApiService.php

@@ -29,7 +29,8 @@ class DolibarrApiService extends ApiRequestService
      * @return array|null
      * @throws \JsonException
      */
-    public function getSociety(int $organizationId): ?array {
+    public function getSociety(int $organizationId): ?array
+    {
         // impossible to retrieve a society by its extrafield 2iopen_organization_id (thanks dolibarr), so
         // we need to store the organization id in two fields: 2iopen_organization_id and ref_int :(
         try {
@@ -49,7 +50,8 @@ class DolibarrApiService extends ApiRequestService
      * @param int $socId
      * @return array|null
      */
-    public function getActiveContract(int $socId): ?array {
+    public function getActiveContract(int $socId): ?array
+    {
         try {
             return $this->getJsonContent(
                 "contracts",
@@ -70,7 +72,8 @@ class DolibarrApiService extends ApiRequestService
      * @param int $socId
      * @return array
      */
-    public function getBills(int $socId): array {
+    public function getBills(int $socId): array
+    {
         try {
             return $this->getJsonContent(
                 "invoices",

src/Service/Dolibarr/DolibarrSyncService.php

@@ -78,7 +78,8 @@ class DolibarrSyncService
      *
      * @noinspection NullPointerExceptionInspection
      */
-    public function scan(?callable $progressionCallback = null): array {
+    public function scan(?callable $progressionCallback = null): array
+    {
         $this->logger->info("-- Scan started --");
 
         // Index the dolibarr clients by organization ids
@@ -441,7 +442,8 @@ class DolibarrSyncService
      *
      * @return array
      */
-    protected function getActiveMembersIndex(): array {
+    protected function getActiveMembersIndex(): array
+    {
         $index = [];
         $results = $this->accessRepository->getAllActiveMembersAndMissions();
         foreach ($results as $row) {
@@ -469,7 +471,8 @@ class DolibarrSyncService
      * @param Person $person
      * @return array|null
      */
-    protected function findDolibarrContactFor(array $dolibarrContacts, Person $person): ?array {
+    protected function findDolibarrContactFor(array $dolibarrContacts, Person $person): ?array
+    {
         foreach ($dolibarrContacts as $contactData) {
             if (!empty($contactData["array_options"]["options_2iopen_person_id"])) {
                 $id = (int)$contactData["array_options"]["options_2iopen_person_id"];
@@ -504,7 +507,8 @@ class DolibarrSyncService
      * @param array|null $data
      * @return array|null
      */
-    protected function sanitizeDolibarrData(?array $data): ?array {
+    protected function sanitizeDolibarrData(?array $data): ?array
+    {
         if ($data === null) {
             return null;
         }
@@ -525,7 +529,8 @@ class DolibarrSyncService
      * @param Organization $organization
      * @return AddressPostal|null
      */
-    protected function getOrganizationPostalAddress(Organization $organization): ?AddressPostal {
+    protected function getOrganizationPostalAddress(Organization $organization): ?AddressPostal
+    {
         $addressPriorities = [
             AddressPostalOrganizationTypeEnum::ADDRESS_BILL()->getValue(),
             AddressPostalOrganizationTypeEnum::ADDRESS_CONTACT()->getValue(),
@@ -584,7 +589,8 @@ class DolibarrSyncService
      * @param Organization $organization
      * @return string|null
      */
-    protected function getOrganizationEmail(Organization $organization): ?string {
+    protected function getOrganizationEmail(Organization $organization): ?string
+    {
         $contactPriorities = [
             ContactPointTypeEnum::BILL()->getValue(),
             ContactPointTypeEnum::CONTACT()->getValue(),
@@ -610,7 +616,8 @@ class DolibarrSyncService
      * @param Organization $organization
      * @return int|null
      */
-    protected function getOrganizationNetworkId(Organization $organization): ?int {
+    protected function getOrganizationNetworkId(Organization $organization): ?int
+    {
         foreach ($organization->getNetworkOrganizations() as $networkOrganization) {
             if ($networkOrganization->getEndDate() !== null && $networkOrganization->getEndDate() < new \DateTime()) {
                 continue;
@@ -627,7 +634,8 @@ class DolibarrSyncService
      * @param array $members An organization members as returned by getActiveMembersIndex: [$accessID => [$missions...]]
      * @return int
      */
-    protected function countWithMission(array $missions, array $members): int {
+    protected function countWithMission(array $missions, array $members): int
+    {
         return count(array_filter(
             $members,
             static function ($actualMissions) use ($missions) { return !empty(array_intersect($actualMissions, $missions)); }
@@ -640,7 +648,8 @@ class DolibarrSyncService
      * @param Person $person
      * @return ContactPoint|null
      */
-    protected function getPersonContact(Person $person): ?ContactPoint {
+    protected function getPersonContact(Person $person): ?ContactPoint
+    {
         $contactPriorities = [
             ContactPointTypeEnum::PRINCIPAL()->getValue(),
             ContactPointTypeEnum::OTHER()->getValue()
@@ -665,7 +674,8 @@ class DolibarrSyncService
      * @param string|null $gender
      * @return string
      */
-    protected function formatContactPosition(array $missions, ?string $gender = 'X'): string {
+    protected function formatContactPosition(array $missions, ?string $gender = 'X'): string
+    {
         $to_exclude = [
             FunctionEnum::ADHERENT()->getValue(),
             FunctionEnum::STUDENT()->getValue(),
@@ -705,7 +715,8 @@ class DolibarrSyncService
      * @param PhoneNumber $phoneNumber
      * @return mixed
      */
-    protected function formatPhoneNumber(PhoneNumber $phoneNumber): string {
+    protected function formatPhoneNumber(PhoneNumber $phoneNumber): mixed
+    {
         $phoneUtil = PhoneNumberUtil::getInstance();
         return str_replace(
             ' ',

src/Service/Export/BaseExporter.php

@@ -112,7 +112,8 @@ abstract class BaseExporter implements ExporterInterface
      * @param bool $flushFile
      * @return File
      */
-    public function prepareFile(ExportRequest $exportRequest, bool $flushFile = true): File {
+    public function prepareFile(ExportRequest $exportRequest, bool $flushFile = true): File
+    {
 
         $requesterId = $exportRequest->getRequesterId();
         $requester = $this->accessRepository->find($requesterId);
@@ -172,7 +173,8 @@ abstract class BaseExporter implements ExporterInterface
      * Return the path of the twig template for this export
      * @return string
      */
-    protected function getTemplatePath(): string {
+    protected function getTemplatePath(): string
+    {
         return '@templates/export/' . $this->getBasename() . '.html.twig';
     }
 
@@ -227,7 +229,8 @@ abstract class BaseExporter implements ExporterInterface
      *
      * @return FileTypeEnum
      */
-    protected function getFileType(): FileTypeEnum {
+    protected function getFileType(): FileTypeEnum
+    {
         return FileTypeEnum::UNKNOWN();
     }
 }

src/Service/Export/LicenceCmfExporter.php

@@ -99,7 +99,8 @@ class LicenceCmfExporter extends BaseExporter
      *
      * @return FileTypeEnum
      */
-    protected function getFileType(): FileTypeEnum {
+    protected function getFileType(): FileTypeEnum
+    {
         return FileTypeEnum::LICENCE_CMF();
     }
 
@@ -109,7 +110,8 @@ class LicenceCmfExporter extends BaseExporter
      * @param int $year
      * @return string
      */
-    protected function getLicenceColor(int $year): string {
+    protected function getLicenceColor(int $year): string
+    {
         if (!($year > self::LICENCE_CMF_COLOR_START_YEAR)) {
             return self::LICENCE_CMF_COLOR[0];
         }

src/Service/Export/Model/LicenceCmf.php

@@ -107,7 +107,7 @@ class LicenceCmf implements ExportModelInterface
      * @param int $id
      * @return LicenceCmf
      */
-    public function setId(int $id): self
+    public function setId(int $id): LicenceCmf
     {
         $this->id = $id;
 
@@ -126,7 +126,7 @@ class LicenceCmf implements ExportModelInterface
      * @param int $year
      * @return LicenceCmf
      */
-    public function setYear(int $year): self
+    public function setYear(int $year): LicenceCmf
     {
         $this->year = $year;
 
@@ -145,7 +145,7 @@ class LicenceCmf implements ExportModelInterface
      * @param bool $isOrganizationLicence
      * @return LicenceCmf
      */
-    public function setIsOrganizationLicence(bool $isOrganizationLicence): self
+    public function setIsOrganizationLicence(bool $isOrganizationLicence): LicenceCmf
     {
         $this->isOrganizationLicence = $isOrganizationLicence;
 
@@ -164,7 +164,7 @@ class LicenceCmf implements ExportModelInterface
      * @param string $organizationName
      * @return LicenceCmf
      */
-    public function setOrganizationName(string $organizationName): self
+    public function setOrganizationName(string $organizationName): LicenceCmf
     {
         $this->organizationName = $organizationName;
 
@@ -183,7 +183,7 @@ class LicenceCmf implements ExportModelInterface
      * @param string|null $organizationIdentifier
      * @return LicenceCmf
      */
-    public function setOrganizationIdentifier(?string $organizationIdentifier): self
+    public function setOrganizationIdentifier(?string $organizationIdentifier): LicenceCmf
     {
         $this->organizationIdentifier = $organizationIdentifier;
 
@@ -202,7 +202,7 @@ class LicenceCmf implements ExportModelInterface
      * @param string $federationName
      * @return LicenceCmf
      */
-    public function setFederationName(string $federationName): self
+    public function setFederationName(string $federationName): LicenceCmf
     {
         $this->federationName = $federationName;
 
@@ -221,7 +221,7 @@ class LicenceCmf implements ExportModelInterface
      * @param string $color
      * @return LicenceCmf
      */
-    public function setColor(string $color): self
+    public function setColor(string $color): LicenceCmf
     {
         $this->color = $color;
 
@@ -278,7 +278,7 @@ class LicenceCmf implements ExportModelInterface
      * @param int|null $personId
      * @return LicenceCmf
      */
-    public function setPersonId(?int $personId): self
+    public function setPersonId(?int $personId): LicenceCmf
     {
         $this->personId = $personId;
 
@@ -297,7 +297,7 @@ class LicenceCmf implements ExportModelInterface
      * @param string $personGender
      * @return LicenceCmf
      */
-    public function setPersonGender(string $personGender): self
+    public function setPersonGender(string $personGender): LicenceCmf
     {
         $this->personGender = $personGender;
 
@@ -316,7 +316,7 @@ class LicenceCmf implements ExportModelInterface
      * @param string $personFirstName
      * @return LicenceCmf
      */
-    public function setPersonFirstName(string $personFirstName): self
+    public function setPersonFirstName(string $personFirstName): LicenceCmf
     {
         $this->personFirstName = $personFirstName;
 
@@ -335,7 +335,7 @@ class LicenceCmf implements ExportModelInterface
      * @param string $personLastName
      * @return LicenceCmf
      */
-    public function setPersonLastName(string $personLastName): self
+    public function setPersonLastName(string $personLastName): LicenceCmf
     {
         $this->personLastName = $personLastName;
 

src/Service/File/FileManager.php

@@ -54,7 +54,8 @@ class FileManager
      * @return string
      * @throws FileNotFoundException
      */
-    public function read(File $file, ?TokenInterface $token=null): string {
+    public function read(File $file, ?TokenInterface $token=null): string
+    {
         $storage = $this->getStorageFor($file);
         if ($storage instanceof ApiLegacyStorage && $token instanceof SwitchUserToken) {
             $storage->setSwitchAccount($token->getOriginalToken()->getUser()->getId(), $token->getUser()->getId());
@@ -83,7 +84,8 @@ class FileManager
      * @param string $ext
      * @return string|null
      */
-    public static function getMimeTypeFromExt(string $ext): string | null {
+    public static function getMimeTypeFromExt(string $ext): ?string
+    {
         return (new MimeTypes)->getMimeType(ltrim($ext, '.'));
     }
 
@@ -95,7 +97,8 @@ class FileManager
      * @param string $filename
      * @return string|null
      */
-    public static function guessMimeTypeFromFilename(string $filename): string | null {
+    public static function guessMimeTypeFromFilename(string $filename): ?string
+    {
         $ext = pathinfo($filename, PATHINFO_EXTENSION);
         if (empty($ext)) {
             return null;

src/Service/File/Storage/ApiLegacyStorage.php

@@ -20,14 +20,11 @@ class ApiLegacyStorage implements FileStorageInterface
     /**
      *  Si ces ids sont renseignés, on est en mode 'switch' et l'url de téléchargement est un peu différente
      */
-    private ?int $originalAccessId;
-    private ?int $asAccessId;
+    private ?int $originalAccessId = null;
+    private ?int $asAccessId = null;
 
-    public function __construct(
-        private ApiLegacyRequestService $apiLegacyRequestService
-    ) {
-        $this->originalAccessId = null;
-        $this->asAccessId = null;
+    public function __construct(private ApiLegacyRequestService $apiLegacyRequestService)
+    {
     }
 
     public function setSwitchAccount(int $originalAccessId, int $asAccessId) {

src/Service/File/Storage/LocalStorage.php

@@ -47,7 +47,8 @@ class LocalStorage implements FileStorageInterface
      * @param File $file
      * @return bool
      */
-    public function exists(File $file): bool {
+    public function exists(File $file): bool
+    {
         return $this->filesystem->has($file->getSlug());
     }
 
@@ -294,7 +295,8 @@ class LocalStorage implements FileStorageInterface
      * @return array
      */
     #[Pure]
-    protected function getOrganizationAndPersonFromOwner(Organization | Access | Person $owner): array {
+    protected function getOrganizationAndPersonFromOwner(Organization | Access | Person $owner): array
+    {
         if ($owner instanceof Access) {
             return [$owner->getOrganization(), $owner->getPerson()];
         }

src/Service/Mailer/Builder/AbstractBuilder.php

@@ -39,7 +39,8 @@ class AbstractBuilder implements AbstractBuilderInterface
      * @return EmailEntity
      * @see AbstractBuilderTest::testBuildEmailEntity()
      */
-    public function buildEmailEntity(string $subject, Access $author, string $content): EmailEntity{
+    public function buildEmailEntity(string $subject, Access $author, string $content): EmailEntity
+    {
         return (new EmailEntity())
             ->setAuthor($author)
             ->setAbout($subject)
@@ -62,7 +63,7 @@ class AbstractBuilder implements AbstractBuilderInterface
      *
      * @see AbstractBuilderTest::testRender()
      */
-    public function render(string $template, array $context)
+    public function render(string $template, array $context): string
     {
         return $this->twig->render(sprintf('@templates/emails/%s.html.twig', $template), $context);
     }
@@ -112,11 +113,12 @@ class AbstractBuilder implements AbstractBuilderInterface
 
     /**
      * @param $contactPoints
-     * @return ContactPoint
+     * @return mixed
      *
      * @see AbstractBuilderTest::testGetFirstContactPointWithEmail()
      */
-    public function getFirstContactPointWithEmail($contactPoints): ?ContactPoint{
+    public function getFirstContactPointWithEmail($contactPoints): mixed
+    {
         $find = false;
         $cmpt = 0;
         $contactPoint = null;

src/Service/Mailer/Mailer.php

@@ -49,7 +49,8 @@ class Mailer
      * @throws TransportExceptionInterface
      * @see MailerTest::testMain()
      */
-    public function main(MailerModelInterface $mailerModel): ArrayCollection {
+    public function main(MailerModelInterface $mailerModel): ArrayCollection
+    {
         // TODO: est-ce qu'on l'appellerait pas plutôt 'send' celle-ci? et est-ce qu'on ne passerait pas les autres en
         //       private? j'ai peur qu'on se trompe parfois entre l'une et l'autre
 
@@ -140,7 +141,8 @@ class Mailer
      *
      * @see MailerTest::testCreateSymfonyEmail()
      */
-    public function createSymfonyEmail(Email $email): SymfonyEmail{
+    public function createSymfonyEmail(Email $email): SymfonyEmail
+    {
         $addressMailFrom = new Address($email->getFrom(), $email->geFromName());
 
         return (new SymfonyEmail())
@@ -199,7 +201,8 @@ class Mailer
      *
      * @see MailerTest::testGetDeliveredAndUndelivered()
      */
-    function getDeliveredAndUndelivered(ArrayCollection $emails): array {
+    function getDeliveredAndUndelivered(ArrayCollection $emails): array
+    {
         $delivered = [];
         $unDelivered = [];
 
@@ -240,7 +243,8 @@ class Mailer
      * @return ReportEmail
      * @see MailerTest::testCreateReport()
      */
-    public function createReport(EmailRecipient $emailRecipient): ReportEmail{
+    public function createReport(EmailRecipient $emailRecipient): ReportEmail
+    {
         return (new ReportEmail())
             ->setAddressEmail($emailRecipient->getEmailAddress())
             ->setAccess($emailRecipient->getAccess())
@@ -257,7 +261,8 @@ class Mailer
      * @return ArrayCollection
      * @see MailerTest::testReduceEmailsCollectionInPreproduction()
      */
-    public function reduceEmailsCollectionInPreproduction(ArrayCollection $emailsCollection): ArrayCollection {
+    public function reduceEmailsCollectionInPreproduction(ArrayCollection $emailsCollection): ArrayCollection
+    {
         if($this->environnement->get(EnvironnementVarEnum::APP_ENV()->getValue()) !== 'prod' && $emailsCollection->count() > 20) {
             $startEmails = $emailsCollection->slice(0, 10);
             $endEmails = $emailsCollection->slice($emailsCollection->count() - 11, 10);

src/Service/Mobyt/MobytService.php

@@ -64,7 +64,8 @@ class MobytService extends ApiRequestService
      * @return array
      * @throws \JsonException
      */
-    public function getUserStatus(string $login, string $password): array {
+    public function getUserStatus(string $login, string $password): array
+    {
         $this->connect($login, $password);
         return $this->getJsonContent(
             'status',

src/Service/Network/Tree.php

@@ -23,7 +23,8 @@ class Tree
      * @param Organization $organization
      * @return array
      */
-    public function findAllParentsAndSortByType(Organization $organization): array {
+    public function findAllParentsAndSortByType(Organization $organization): array
+    {
         return $this->sortByType($this->networkOrganizationRepository->findAllParents($organization));
     }
 
@@ -34,7 +35,8 @@ class Tree
      * @return array
      * @see TreeTest::testSortByType()
      */
-    public function sortByType(array $organizations): array {
+    public function sortByType(array $organizations): array
+    {
         $typeOrder = [
             PrincipalTypeEnum::DELEGATION(),
             PrincipalTypeEnum::GROUPMENT(),

src/Service/Network/Utils.php

@@ -21,7 +21,8 @@ class Utils
      * @return bool
      * @see UtilsTest::testIsCmf()
      */
-    public function isCMF(Organization $organization): bool {
+    public function isCMF(Organization $organization): bool
+    {
         return $this->doesOrganizationBelongToTheNetwork($organization, NetworkEnum::CMF());
     }
 
@@ -31,7 +32,8 @@ class Utils
      * @return bool
      * @see UtilsTest::testIsCmfAndActiveNow()
      */
-    public function isCMFAndActiveNow(Organization $organization): bool {
+    public function isCMFAndActiveNow(Organization $organization): bool
+    {
         return $this->doesOrganizationBelongToTheNetwork($organization, NetworkEnum::CMF(), true);
     }
 
@@ -44,7 +46,8 @@ class Utils
      * @return boolean
      * @see UtilsTest::testIsOrganizationBelongToTheNetwork()
      */
-    public function doesOrganizationBelongToTheNetwork(Organization $organization, NetworkEnum $network, ?bool $activeNow = false): bool {
+    public function doesOrganizationBelongToTheNetwork(Organization $organization, NetworkEnum $network, ?bool $activeNow = false): bool
+    {
         $networksOrganizations = $organization->getNetworkOrganizations();
         /** @var NetworkOrganization $networksOrganization */
         foreach ($networksOrganizations as $networksOrganization) {
@@ -61,7 +64,8 @@ class Utils
      * @return bool
      * @throws \Exception
      */
-    public function isNetworkOrganizationActiveNow(NetworkOrganization $networksOrganization): bool{
+    public function isNetworkOrganizationActiveNow(NetworkOrganization $networksOrganization): bool
+    {
         return DatesUtils::isIntervalIsValidNow($networksOrganization->getStartDate(), $networksOrganization->getEndDate());
     }
 }

src/Service/OnChange/OnChangeContext.php

@@ -13,7 +13,8 @@ class OnChangeContext
      *
      * @return bool
      */
-    public function isPostRequest(): bool {
+    public function isPostRequest(): bool
+    {
         return $this->context['collection_operation_name'] ?? null === 'post';
     }
 
@@ -22,7 +23,8 @@ class OnChangeContext
      *
      * @return bool
      */
-    public function isPutRequest(): bool {
+    public function isPutRequest(): bool
+    {
         return $this->context['item_operation_name'] ?? null === 'put';
     }
 

src/Service/OnChange/Organization/OnSubdomainChange.php

@@ -91,7 +91,8 @@ class OnSubdomainChange extends OnChangeDefault
      * @return SubdomainChangeModel
      * @see OnSubdomainChangeTest::testGetMailModel()
      */
-    public function getMailModel(Subdomain $subdomain): SubdomainChangeModel{
+    public function getMailModel(Subdomain $subdomain): SubdomainChangeModel
+    {
         return (new SubdomainChangeModel())
             ->setSenderId($this->security->getUser()->getId())
             ->setOrganizationId($subdomain->getOrganization()->getId())

src/Service/Organization/OrganizationProfileCreator.php

@@ -30,7 +30,8 @@ class OrganizationProfileCreator
      * @return OrganizationProfile
      * @see OrganizationProfileCreatorTest::testCreateCompleteOrganizationProfile()
      */
-    public function createCompleteOrganizationProfile(Organization $organization): OrganizationProfile{
+    public function createCompleteOrganizationProfile(Organization $organization): OrganizationProfile
+    {
         $organizationProfile = $this->createLightOrganizationProfile($organization);
         $organizationProfile->setModules($this->module->getOrganizationModules($organization));
         $organizationProfile->setProduct($organization->getSettings()->getProduct());
@@ -62,7 +63,8 @@ class OrganizationProfileCreator
      * @param Organization $organization
      * @return OrganizationProfile
      */
-    public function createLightOrganizationProfile(Organization $organization): OrganizationProfile{
+    public function createLightOrganizationProfile(Organization $organization): OrganizationProfile
+    {
         $organizationProfile = new OrganizationProfile();
         $organizationProfile
             ->setId($organization->getId())

src/Service/Organization/Utils.php

@@ -85,7 +85,8 @@ class Utils
      * @throws \Exception
      * @see UtilsTest::testGetOrganizationCurrentActivityYear()
      */
-    public function getOrganizationCurrentActivityYear(Organization $organization): int{
+    public function getOrganizationCurrentActivityYear(Organization $organization): int
+    {
         $today = DatesUtils::new('now');
         $year = (int)$today->format('Y');
 
@@ -159,9 +160,10 @@ class Utils
      * Return the active subdomain of an organization as a string, or null
      *
      * @param Organization $organization
-     * @return string | null
+     * @return mixed
      */
-    public function getOrganizationActiveSubdomain(Organization $organization): ?string {
+    public function getOrganizationActiveSubdomain(Organization $organization): mixed
+    {
         foreach ($organization->getSubdomains() as $subdomain) {
             if ($subdomain->isActive()) {
                 return $subdomain->getSubdomain();
@@ -176,9 +178,11 @@ class Utils
      * @see https://ressources.opentalent.fr/display/SPEC/Preferences#Preferences-Siteinternet
      *
      * @param Organization $organization
-     * @return string | null
+     * @param Organization $organization
+     * @return mixed
      */
-    public function getOrganizationWebsite(Organization $organization): ?string {
+    public function getOrganizationWebsite(Organization $organization): mixed
+    {
         $parameters = $organization->getParameters();
 
         if ($parameters->getDesactivateOpentalentSiteWeb()) {

src/Service/Rest/ApiRequestService.php

@@ -31,7 +31,8 @@ class ApiRequestService implements ApiRequestInterface
      * @throws HttpException
      * @throws \JsonException
      */
-    public function getJsonContent(string $path, array $parameters = [], array $options = []): array {
+    public function getJsonContent(string $path, array $parameters = [], array $options = []): array
+    {
         return json_decode($this->getContent($path, $parameters, $options), true, 512, JSON_THROW_ON_ERROR);
     }
 
@@ -44,7 +45,8 @@ class ApiRequestService implements ApiRequestInterface
      * @return string
      * @throws HttpException
      */
-    public function getContent(string $path, array $parameters = [], array $options = []): string {
+    public function getContent(string $path, array $parameters = [], array $options = []): string
+    {
         try {
             return $this->get($path, $parameters, $options)->getContent();
         } catch (ClientExceptionInterface | TransportExceptionInterface | RedirectionExceptionInterface | ServerExceptionInterface $e) {
@@ -61,7 +63,8 @@ class ApiRequestService implements ApiRequestInterface
      * @return ResponseInterface
      * @throws HttpException
      */
-    public function get(string $path, array $parameters = [], array $options = []): ResponseInterface {
+    public function get(string $path, array $parameters = [], array $options = []): ResponseInterface
+    {
         return $this->request('GET', $path, $parameters, $options);
     }
 

src/Service/Rest/Operation/CreateOperation.php

@@ -58,7 +58,8 @@ class CreateOperation extends BaseRestOperation
      *
      * @return array
      */
-    public function getChangeLog(): array {
+    public function getChangeLog(): array
+    {
         $messages = [
             '[POST ' . $this->entityName . ']'
         ];

src/Service/Rest/Operation/DeleteOperation.php

@@ -52,7 +52,8 @@ class DeleteOperation extends BaseRestOperation
      *
      * @return array
      */
-    public function getChangeLog(): array {
+    public function getChangeLog(): array
+    {
         return [
             '[DELETE ' . $this->entityName . '/' . $this->id . ']'
         ];

src/Service/Rest/Operation/UpdateOperation.php

@@ -62,7 +62,8 @@ class UpdateOperation extends BaseRestOperation
      *
      * @return array
      */
-    public function getChangeLog(): array {
+    public function getChangeLog(): array
+    {
         $messages = [
             '[PUT ' . $this->entityName . '/' . $this->id . ']'
         ];

src/Service/Security/Module.php

@@ -29,7 +29,8 @@ class Module
      * @param Organization $organization
      * @return array
      */
-    public function getOrganizationModules(Organization $organization): array {
+    public function getOrganizationModules(Organization $organization): array
+    {
 //        $cacheDriver = new ApcuCache();
 //        //If the modules are all ready available inside the APCu cache
 //        if($cacheDriver->contains('organization_modules_' . $organization->getId())){
@@ -57,7 +58,8 @@ class Module
      * @return array
      * @see ModuleTest::testGetModuleBySettings()
      */
-    public function getModuleBySettings(Organization $organization): array{
+    public function getModuleBySettings(Organization $organization): array
+    {
         $moduleByOptions = [];
         $modulesOptions = $organization->getSettings()->getModules();
         if(!empty($modulesOptions)){
@@ -75,7 +77,8 @@ class Module
      * @return array
      * @see ModuleTest::testGetModulesByConditions()
      */
-    public function getModulesByConditions(Organization $organization): array {
+    public function getModulesByConditions(Organization $organization): array
+    {
         $modulesByConditions = [];
         $modules = $this->getModuleByConditionsConfig()['opentalent']['modulesbyconditions'];
         foreach ($modules as $moduleName => $module) {
@@ -101,7 +104,8 @@ class Module
      * @return array|null
      * @see ModuleTest::testGetModulesByProductConfiguration()
      */
-    public function getModulesByProductConfiguration(string $product): ?array {
+    public function getModulesByProductConfiguration(string $product): ?array
+    {
         $product = str_replace('-', '_', $product);
 
         if (!array_key_exists($product, $this->getModuleConfig()['opentalent']['products'])) {
@@ -130,7 +134,8 @@ class Module
      * @return array
      * @throws \Exception
      */
-    protected function loadModuleConfig(): array {
+    protected function loadModuleConfig(): array
+    {
         return $this->parser->yamlParser($this->opentalentConfig, 'products.yaml');
     }
 
@@ -146,7 +151,8 @@ class Module
      * @return array
      * @throws \Exception
      */
-    protected function loadModuleByConditionsConfig(): array{
+    protected function loadModuleByConditionsConfig(): array
+    {
         return $this->parser->yamlParser($this->opentalentConfig, 'modulesbyconditions.yaml');
     }
 
@@ -155,7 +161,8 @@ class Module
      * @param string $resource
      * @return int|null|string
      */
-    public function getModuleByResourceName(string $resource): ?string {
+    public function getModuleByResourceName(string $resource): int|string|null
+    {
         $modules = $this->getModuleConfig()['opentalent']['modules'];
         foreach ($modules as $module => $data) {
             if ($data['entities'] && in_array($resource, $data['entities'], true)) {

src/Service/Utils/ArrayUtils.php

@@ -58,7 +58,8 @@ class ArrayUtils
      * @param string $type
      * @return mixed
      */
-    public static function getAndCast(array $array, mixed $key, string $type): mixed {
+    public static function getAndCast(array $array, mixed $key, string $type): mixed
+    {
         $value = $array[$key] ?? null;
         if ($value !== null) {
             settype($value, $type);

src/Service/Utils/DatesUtils.php

@@ -49,7 +49,8 @@ class DatesUtils
      * @return bool
      * @see DatesUtilsTest::testIsIntervalIsValidNow()
      */
-    public static function isIntervalIsValidNow(\DateTimeInterface $dateStart, \DateTimeInterface $dateEnd = null): bool {
+    public static function isIntervalIsValidNow(\DateTimeInterface $dateStart, \DateTimeInterface $dateEnd = null): bool
+    {
         $now = new \DateTime('now');
         return $dateStart <= $now && (is_null($dateEnd) || $dateEnd >= $now);
     }

src/Service/Utils/GpsCoordinateUtils.php

@@ -35,7 +35,8 @@ class GpsCoordinateUtils
      * @throws \Symfony\Contracts\HttpClient\Exception\TransportExceptionInterface
      * @see GpsCoordinateUtilsTest::testSearchGpsCoordinates()
      */
-    public function searchGpsCoordinates(?string $street, ?string $cp, ?string $city): array {
+    public function searchGpsCoordinates(?string $street, ?string $cp, ?string $city): array
+    {
         try{
             $url = sprintf('search?addressdetails=1&format=json&limit=10&street=%s&postalcode=%s&city=%s', $street, $cp, $city);
             $response = $this->clientOpenStreetMap->request('GET', $url)->getContent();
@@ -56,7 +57,8 @@ class GpsCoordinateUtils
      * @throws \Symfony\Contracts\HttpClient\Exception\TransportExceptionInterface
      * @see GpsCoordinateUtilsTest::testReverseGpsCoordinates()
      */
-    public function reverseGpsCoordinates(float $latitude, float $longitude): array {
+    public function reverseGpsCoordinates(float $latitude, float $longitude): array
+    {
         try{
             $url = sprintf('reverse?addressdetails=1&format=json&lat=%s&lon=%s', $latitude, $longitude);
             $response = $this->clientOpenStreetMap->request('GET', $url)->getContent();
@@ -73,7 +75,8 @@ class GpsCoordinateUtils
      * @return GpsCoordinate
      * @see GpsCoordinateUtilsTest::testCreateGpsCoordinate()
      */
-    public function createGpsCoordinate(array $gpsApiResponse):GpsCoordinate{
+    public function createGpsCoordinate(array $gpsApiResponse): GpsCoordinate
+    {
         $address = $this->transformAddress($gpsApiResponse['address']);
         $gpsCoordinate = new GpsCoordinate();
         $gpsCoordinate
@@ -91,7 +94,8 @@ class GpsCoordinateUtils
      * @param array $address
      * @return array
      */
-    public function transformAddress(array $address): array{
+    public function transformAddress(array $address): array
+    {
         $addressTransformed['streetAddress'] = key_exists('road', $address) ? $address['road'] : (key_exists('hamlet', $address) ? $address['hamlet'] : null);
         $addressTransformed['streetAddressSecond'] =  key_exists('road', $address) && key_exists('hamlet', $address)  ? $address['hamlet'] : null;
         $addressTransformed['city'] = key_exists('town', $address) ? $address['town'] : (key_exists('village', $address) ? $address['village'] : null);

src/Service/Utils/Parser.php

@@ -19,7 +19,8 @@ class Parser
      * @return array
      * @see ParserTest::testYamlParser()
      */
-    public function yamlParser(string $directory, string $yamlFile): array {
+    public function yamlParser(string $directory, string $yamlFile): array
+    {
         $configDirectories = [$directory];
         $fileLocator = new FileLocator($configDirectories);
         $yamlConfig = $fileLocator->locate($yamlFile, null, false)[0];

src/Service/Utils/Path.php

@@ -51,7 +51,8 @@ class Path
      * @param string $glob
      * @return array
      */
-    public static function list(string $path, string $glob = '*'): array {
+    public static function list(string $path, string $glob = '*'): array
+    {
         return glob(self::join($path, $glob));
     }
 

src/Service/Utils/Siret.php

@@ -31,7 +31,8 @@ class Siret
      * @throws \Symfony\Contracts\HttpClient\Exception\TransportExceptionInterface
      * @see SiretTest::testIsSiretIsCorrect()
      */
-    public function isSiretCorrect(string $siret): bool {
+    public function isSiretCorrect(string $siret): bool
+    {
         $response = $this->clientSiret->request('GET', $siret);
         return $response->getStatusCode() === Response::HTTP_OK;
     }

src/Service/Utils/StringsUtils.php

@@ -17,7 +17,8 @@ class StringsUtils
      * @return string
      * @see StringsUtilsTest::testUnquote()
      */
-    public static function unquote(string $str): string {
+    public static function unquote(string $str): string
+    {
         return str_replace("'", "", $str);
     }
 
@@ -31,7 +32,8 @@ class StringsUtils
      *
      * @see StringsUtilsTest::testCamelToSnake()
      */
-    public static function camelToSnake(string $string, string $sep = "_"): string {
+    public static function camelToSnake(string $string, string $sep = "_"): string
+    {
         return strtolower(
             preg_replace(
             '/(?<=\d)(?=[A-Za-z])|(?<=[A-Za-z])(?=\d)|(?<=[a-z])(?=[A-Z])/',

src/Service/Utils/UrlBuilder.php

@@ -15,7 +15,8 @@ class UrlBuilder
      * @param string $path The following path
      * @return string
      */
-    public static function concatPath(string $url, string $path): string {
+    public static function concatPath(string $url, string $path): string
+    {
         return rtrim($url, '/') . '/' . ltrim($path, '/');
     }
 
@@ -26,7 +27,8 @@ class UrlBuilder
      * @param array $parameters
      * @return string
      */
-    public static function concatParameters(string $url, array $parameters = []): string {
+    public static function concatParameters(string $url, array $parameters = []): string
+    {
         if (!$parameters) { return $url; }
         $url = rtrim($url, '?&/');
         $query = join(
@@ -69,7 +71,8 @@ class UrlBuilder
      * @param bool $preprendHttps Should the 'https://' be prepended if missing
      * @return string
      */
-    public static function concat(string $url, string $path, array $parameters, bool $preprendHttps = false): string {
+    public static function concat(string $url, string $path, array $parameters, bool $preprendHttps = false): string
+    {
         $url = self::concatParameters(self::concatPath($url, $path), $parameters);
         if ($preprendHttps) {
             $url = self::prependHttps($url);

src/Validator/Core/ContactPoint.php

@@ -10,7 +10,7 @@ class ContactPoint extends Constraint
 {
     public $message = '{{ type }}_non_unique';
 
-    public function getTargets()
+    public function getTargets(): string|array
     {
         return self::CLASS_CONSTRAINT;
     }

src/Validator/Organization/OrganizationAddressPostal.php

@@ -10,7 +10,7 @@ class OrganizationAddressPostal extends Constraint
 {
     public $message = '{{ type }}_non_unique';
 
-    public function getTargets()
+    public function getTargets(): string|array
     {
         return self::CLASS_CONSTRAINT;
     }

src/Validator/Organization/Parameters/MobytCredentials.php

@@ -10,7 +10,7 @@ class MobytCredentials extends Constraint
 {
     public $message = 'wrong_mobyt_credentials';
 
-    public function getTargets()
+    public function getTargets(): string|array
     {
         return self::CLASS_CONSTRAINT;
     }

tests/Service/Cron/UI/ConsoleUITest.php

@@ -24,8 +24,6 @@ class ConsoleUITest extends TestCase
     public function setUp(): void
     {
         $this->output = $this->getMockBuilder(OutputInterface::class)->disableOriginalConstructor()->getMock();
-
-        $this->progressBar = $this->getMockBuilder(MockableProgressBar::class)->disableOriginalConstructor()->getMockForAbstractClass();
     }
 
     public function testPrint(): void