Merge pull request #5275 from usu/chore/remove-delete-listener

chore(api): remove ValidateDeleteRequestsListener
ecamp · Jun 2, 2024 · fe4a8da · fe4a8da
2 parents c4181e1 + 554ed48
commit fe4a8da
Show file tree

Hide file tree

Showing 8 changed files with 12 additions and 168 deletions.
diff --git a/api/src/Entity/ActivityProgressLabel.php b/api/src/Entity/ActivityProgressLabel.php
@@ -36,6 +36,8 @@
             validationContext: ['groups' => ['Default', 'update']]
         ),
         new Delete(
+            validate: true,
+            validationContext: ['groups' => ['delete']],
             security: 'is_granted("CAMP_MANAGER", object)'
         ),
         new GetCollection(

diff --git a/api/src/Entity/CampCollaboration.php b/api/src/Entity/CampCollaboration.php
@@ -44,6 +44,8 @@
             validationContext: ['groups' => ['Default', 'update']]
         ),
         new Delete(
+            validate: true,
+            validationContext: ['groups' => ['delete']],
             security: 'is_granted("CAMP_MEMBER", object) or is_granted("CAMP_MANAGER", object)'
         ),
         new Patch(

diff --git a/api/src/Entity/Category.php b/api/src/Entity/Category.php
@@ -44,7 +44,9 @@
         ),
         new Delete(
             processor: CategoryRemoveProcessor::class,
-            security: 'is_granted("CAMP_MEMBER", object) or is_granted("CAMP_MANAGER", object)'
+            security: 'is_granted("CAMP_MEMBER", object) or is_granted("CAMP_MANAGER", object)',
+            validate: true,
+            validationContext: ['groups' => ['delete']],
         ),
         new GetCollection(
             security: 'is_authenticated()'

diff --git a/api/src/Entity/MaterialList.php b/api/src/Entity/MaterialList.php
@@ -34,7 +34,9 @@
             security: 'is_granted("CAMP_MEMBER", object) or is_granted("CAMP_MANAGER", object)'
         ),
         new Delete(
-            security: 'is_granted("CAMP_MEMBER", object) or is_granted("CAMP_MANAGER", object)'
+            security: 'is_granted("CAMP_MEMBER", object) or is_granted("CAMP_MANAGER", object)',
+            validate: true,
+            validationContext: ['groups' => ['delete']]
         ),
         new GetCollection(
             security: 'is_authenticated()'

diff --git a/api/src/Entity/Period.php b/api/src/Entity/Period.php
@@ -43,6 +43,7 @@
         ),
         new Delete(
             security: 'is_granted("CAMP_MEMBER", object) or is_granted("CAMP_MANAGER", object)',
+            validate: true,
             validationContext: ['groups' => ['delete', 'Period:delete']]
         ),
         new GetCollection(

diff --git a/api/src/Entity/ScheduleEntry.php b/api/src/Entity/ScheduleEntry.php
@@ -38,6 +38,7 @@
         ),
         new Delete(
             security: 'is_granted("CAMP_MEMBER", object) or is_granted("CAMP_MANAGER", object)',
+            validate: true,
             validationContext: ['groups' => ['delete', 'ScheduleEntry:delete']]
         ),
         new GetCollection(

diff --git a/api/src/EventListener/ValidateDeleteRequestsListener.php b/api/src/EventListener/ValidateDeleteRequestsListener.php