From be11d56256c916b4a6c1edf9cd0016cf280c9238 Mon Sep 17 00:00:00 2001 From: octoseth <71984208+octoseth@users.noreply.github.com> Date: Wed, 29 Jan 2025 23:36:46 +0100 Subject: [PATCH 1/2] Check if entity is Accessible before processActions in EntityFactory --- src/Factory/EntityFactory.php | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/Factory/EntityFactory.php b/src/Factory/EntityFactory.php index 30b5d0189d..4e4585f508 100644 --- a/src/Factory/EntityFactory.php +++ b/src/Factory/EntityFactory.php @@ -59,6 +59,9 @@ public function processActions(EntityDto $entityDto, ActionConfigDto $actionConf public function processActionsForAll(EntityCollection $entities, ActionConfigDto $actionConfigDto): ActionCollection { foreach ($entities as $entity) { + if (!$entity->isAccessible()) { + continue; + } $this->processActions($entity, clone $actionConfigDto); } From bfb47300f6cfcf3f88ce1a325f160fe522273135 Mon Sep 17 00:00:00 2001 From: octoseth <71984208+octoseth@users.noreply.github.com> Date: Sat, 1 Feb 2025 22:59:24 +0100 Subject: [PATCH 2/2] check if entityDto is accessible in processActions instead inside processActionsForAll --- src/Factory/EntityFactory.php | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/Factory/EntityFactory.php b/src/Factory/EntityFactory.php index 4e4585f508..808845beee 100644 --- a/src/Factory/EntityFactory.php +++ b/src/Factory/EntityFactory.php @@ -53,15 +53,15 @@ public function processFieldsForAll(EntityCollection $entities, FieldCollection public function processActions(EntityDto $entityDto, ActionConfigDto $actionConfigDto): void { + if (!$entityDto->isAccessible()) { + return; + } $this->actionFactory->processEntityActions($entityDto, $actionConfigDto); } public function processActionsForAll(EntityCollection $entities, ActionConfigDto $actionConfigDto): ActionCollection { foreach ($entities as $entity) { - if (!$entity->isAccessible()) { - continue; - } $this->processActions($entity, clone $actionConfigDto); }