Merge pull request #615 from eerison/bug/remove-entity-data

Bug/Remove entity data

composer.json

@@ -101,6 +101,7 @@
             "@db:migration:apply"
         ],
         "lint": [
+            "@lint:phpstan",
             "@lint:ecs",
             "@lint:rector",
             "@lint:twig",

config/packages/prod/monolog.yaml

@@ -4,7 +4,6 @@ monolog:
             type: fingers_crossed
             action_level: error
             handler: sentry
-            excluded_http_codes: [404, 405]
             buffer_size: 50 # How many messages should be saved? Prevent memory leaks
         sentry:
             type: service

phpstan.neon

@@ -2,5 +2,3 @@ parameters:
     level: 1
     paths:
         - src
-    ignoreErrors:
-            - "#has an uninitialized readonly property \\$id. Assign it in the constructor.#"

src/Controller/Profile/CertificationController.php

@@ -43,7 +43,7 @@ public function editAction(Request $request, Certification $certification): Resp
      *
      * @IsGranted("ROLE_USER", subject="certification")
      */
-    #[Route(path: '/{id}', name: 'delete', methods: ['DELETE'])]
+    #[Route(path: '/{id}/del', name: 'delete', methods: ['POST'])]
     public function deleteAction(Request $request, Certification $certification): Response
     {
         return $this->delete($request, $certification);

src/Controller/Profile/EducationController.php

@@ -46,7 +46,7 @@ public function editAction(Request $request, Education $education): Response
      *
      * @Security("user == education.getUser()")
      */
-    #[Route(path: '/{id}', name: 'delete', methods: ['DELETE'])]
+    #[Route(path: '/{id}/del', name: 'delete', methods: ['POST'])]
     public function deleteAction(Request $request, Education $education): Response
     {
         return $this->delete($request, $education);

src/Controller/Profile/ExperienceController.php

@@ -75,7 +75,7 @@ public function editAction(Request $request, Experience $experience): Response
     /**
      * @Security("user == experience.getUser()")
      */
-    #[Route(path: '/{id}', name: 'delete', methods: ['DELETE'])]
+    #[Route(path: '/{id}/del', name: 'delete', methods: ['POST'])]
     public function deleteAction(Request $request, Experience $experience): Response
     {
         if ($this->isCsrfTokenValid('delete' . $experience->getId(), $request->request->get('_token'))) {

src/Controller/Profile/SkillController.php

@@ -72,7 +72,7 @@ public function editAction(Request $request, Skill $skill): Response
     /**
      * @Security("user == skill.getUser()")
      */
-    #[Route(path: '/{id}', name: 'delete', methods: ['DELETE'])]
+    #[Route(path: '/{id}/del', name: 'delete', methods: ['POST'])]
     public function deleteAction(Request $request, Skill $skill): Response
     {
         if ($this->isCsrfTokenValid('delete' . $skill->getId(), $request->request->get('_token'))) {

src/Controller/Profile/UserLanguageController.php

@@ -72,7 +72,7 @@ public function edit(Request $request, UserLanguage $userLanguage): Response
     /**
      * @IsGranted("ROLE_USER", subject="userLanguage")
      */
-    #[Route(path: '/{id}', name: 'delete', methods: ['DELETE'])]
+    #[Route(path: '/{id}/del', name: 'delete', methods: ['POST'])]
     public function delete(Request $request, UserLanguage $userLanguage): Response
     {
         if ($this->isCsrfTokenValid('delete' . $userLanguage->getId(), $request->request->get('_token'))) {

src/Controller/Profile/UserSocialNetworkingController.php

@@ -87,7 +87,7 @@ public function editAction(Request $request, UserUserSocialNetworking $userSocia
      *
      * @Security("user == userSocialNetworking.getUser()")
      */
-    #[Route(path: '/{id}', name: 'delete', methods: ['DELETE'])]
+    #[Route(path: '/{id}/del', name: 'delete', methods: ['POST'])]
     public function deleteAction(Request $request, UserUserSocialNetworking $userSocialNetworking): Response
     {
         if ($this->isCsrfTokenValid('delete' . $userSocialNetworking->getId(), $request->request->get('_token'))) {

src/Entity/Certification.php

@@ -21,7 +21,7 @@ class Certification implements EntityInterface, HasUserInterface, CertificationI
     #[ORM\Column(name: 'id', type: Types::INTEGER)]
     #[ORM\Id]
     #[ORM\GeneratedValue]
-    private readonly int $id;
+    private int $id;
 
     #[ORM\ManyToOne(targetEntity: User::class, inversedBy: 'certifications')]
     #[ORM\JoinColumn(name: 'user_id')]

src/Entity/Skill.php

@@ -17,7 +17,7 @@ class Skill implements SkillInterface
     #[ORM\Column(name: 'id', type: Types::INTEGER)]
     #[ORM\Id]
     #[ORM\GeneratedValue]
-    private readonly int $id;
+    private int $id;
 
     #[ORM\ManyToOne(targetEntity: 'User', inversedBy: 'skills')]
     #[ORM\JoinColumn(name: 'user_id')]