Refactoring rector trait

[ThomasRitaine]

Description

Salut tout le monde ! Voici une grosse PR qui fait les modifications suivantes :

• Ajout d'un outil de refactoring Open Source : Rector PHP
• Mise à jour du code en accord avec les dernières conventions PHP, Symfony et Doctrine
• Typage de toutes les propriétés et méthodes
• Utilisation des nouveautés PHP 8.1
• Passage de toutes les annotations en attributs PHP
• Quelques modifications mineures
• Factorisation des UUID, Timestamps et Softdelete dans des traits Pour ce dernier points, il y a un petit inconvénient Quand une entité veut exposer via API platform un champ qui dans dans un trait, par exemple son UUID, il faut redéfinir la propriété id dans la classe en question pour y mettre l'attribut PHP Groups. Le id qu'on définit dans l'entité va redéfinir de celui du trait, donc on doit réécrire tous les #[ORM\] et #[Assert\]. Un exemple :

  #[Groups([
  'event:read:some',
  'event:read:one',
  'asso:read:one',
  ])]
  #[ORM\Id]
  #[ORM\Column(type: 'uuid', unique: true)]
  #[ORM\GeneratedValue(strategy: 'CUSTOM')]
  #[ORM\CustomIdGenerator(class: UuidGenerator::class)]
  private Uuid $id;

Test et review

Pour tester la branche, il faut :

1. Aller sur la branche
2. Supprimer sa base de donnée existante : php bin/console doctrine:database:drop --force
3. La recréer : php bin/console doctrine:database:create
4. Créer le schéma de la DB : php bin/console doctrine:schema:update --force
5. Lancer les fixtures : php bin/console doctrine:fixtures:load -n

Tout est fonctionnel, et j'ai regroupé les différentes fonctionnalités par thème et par commit, avec des noms très explicites afin de faciliter la review (mais encore une fois, tout est fonctionnel)

Checklist

Implementation

• [x] Files and variables have a name explaining what they do.
• [x] Updating the DB schemas and seeding the tables do not generates errors.
• [x] All code repeted more than 2 times has been refactored in a dedicated file or function.
• [x] Debugs have been removed : dd() | dump()

Tools

• [x] PHP CS Fixer has fixed all files inside the src folder.
• [x] All commits have been done using Commitizen.

Documentation

• [x] All new classes and non-trivial functions have a code documentation.
• [x] There are comments to explain complex parts inside your code.
• [x] If a new folder is created, it has been added and explained inside the "Folder structure" part of the README.md file.

[ThomasRitaine]

Salut ! C'est super ! Peut-être faudrait-il ajouter un test pour bloquer les pr si rector trouve des éléments à modifier ?

Salut ! Non, je pense qu'on devrait l'utiliser avant de commit, comme PHP-CS-Fixer. Aussi, cette techno sera utile pour les mises à niveau automatiques du code en fonction des versions PHP, Symfony et Doctrine

[ThomasRitaine]

@TeddyRoncin @larueli Si vous êtes d'accord, je vais merge. Cette PR ne modifie pas le fonctionnement de l'app, et le projet a grandement besoin d'une simplification comme celle-ci

[TeddyRoncin]

Je l'ai pas testée (je pourrai la tester demain ou après-demain si tu veux, mais je te fais confiance là-dessus, je pense que tu l'as testée beaucoup mieux que tout ce que je pourrais faire), mais j'ai vite fait regardé et les changements me paraissent cohérents

[ThomasRitaine]

Ok, je vais rebase et merge alors !