Skip to content

Commit dfca1ed

Browse files
rthideawayfubhy
authored andcommitted
Fix access checking. (#921)
1 parent 526eeea commit dfca1ed

4 files changed

Lines changed: 4 additions & 4 deletions

File tree

src/Plugin/GraphQL/DataProducer/Entity/EntityLoad.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -159,7 +159,7 @@ public function resolve($type, $id, $language, $bundles, ?bool $access, ?Account
159159
/* @var $accessResult \Drupal\Core\Access\AccessResultInterface */
160160
$accessResult = $entity->access($accessOperation, $accessUser, TRUE);
161161
$context->addCacheableDependency($accessResult);
162-
if ($accessResult->isForbidden()) {
162+
if (!$accessResult->isAllowed()) {
163163
return NULL;
164164
}
165165
}

src/Plugin/GraphQL/DataProducer/Entity/EntityLoadByUuid.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -159,7 +159,7 @@ public function resolve($type, $uuid, $language, $bundles, ?bool $access, ?Accou
159159
/* @var $accessResult \Drupal\Core\Access\AccessResultInterface */
160160
$accessResult = $entity->access($accessOperation, $accessUser, TRUE);
161161
$context->addCacheableDependency($accessResult);
162-
if ($accessResult->isForbidden()) {
162+
if (!$accessResult->isAllowed()) {
163163
return NULL;
164164
}
165165
}

src/Plugin/GraphQL/DataProducer/Entity/EntityTranslation.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -105,7 +105,7 @@ public function resolve(EntityInterface $entity, $language, ?bool $access, ?Acco
105105
if ($access) {
106106
/* @var $accessResult \Drupal\Core\Access\AccessResultInterface */
107107
$accessResult = $entity->access($accessOperation, $accessUser, TRUE);
108-
if ($accessResult->isForbidden()) {
108+
if (!$accessResult->isAllowed()) {
109109
return NULL;
110110
}
111111
}

src/Plugin/GraphQL/DataProducer/Entity/EntityTranslations.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -106,7 +106,7 @@ public function resolve(EntityInterface $entity, ?bool $access , ?AccountInterfa
106106
if ($access) {
107107
/* @var $accessResult \Drupal\Core\Access\AccessResultInterface */
108108
$accessResult = $entity->access($accessOperation, $accessUser, TRUE);
109-
if ($accessResult->isForbidden()) {
109+
if (!$accessResult->isAllowed()) {
110110
return NULL;
111111
}
112112
}

0 commit comments

Comments
 (0)