PHP return type

composer.json

@@ -36,6 +36,7 @@
         "symfony/console": "6.0.*",
         "symfony/doctrine-messenger": "6.0.*",
         "symfony/dotenv": "6.0.*",
+        "symfony/error-handler": "6.0.*",
         "symfony/expression-language": "6.0.*",
         "symfony/flex": "^1.3.1",
         "symfony/framework-bundle": "6.0.*",
@@ -67,7 +68,7 @@
         "cyclonedx/cyclonedx-php-composer": "^3.4",
         "phpstan/phpstan": "^1.9",
         "rector/rector": "^0.15.13",
-        "symfony/debug-bundle": "5.4.*",
+        "symfony/debug-bundle": "6.0.*",
         "symfony/maker-bundle": "^1.21",
         "symfony/phpunit-bridge": "^6.2",
         "symfony/stopwatch": "6.0.*",

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/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/Person/Person.php

@@ -226,7 +226,7 @@ class Person implements UserInterface, PasswordAuthenticatedUserInterface
     }
 
     /**
-     * @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/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

@@ -29,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): ?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));
@@ -79,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/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/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;
 

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/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);