- Issue created by @mandclu
- Open on Drupal.org โCore: 10.0.7 + Environment: PHP 8.1 & MySQL 5.7last update
over 1 year ago Waiting for branch to pass - @mandclu opened merge request.
- Status changed to Needs review
over 1 year ago 10:43pm 13 June 2023 - ๐จ๐ฆCanada mandclu
The simple change in the MR resolved the error for me.
- Status changed to RTBC
over 1 year ago 7:41pm 27 June 2023 - ๐บ๐ธUnited States ctrladel North Carolina, USA
Patch applied and Enqueue content is working as expected.
- Open on Drupal.org โCore: 10.0.7 + Environment: PHP 8.1 & MySQL 5.7last update
over 1 year ago Waiting for branch to pass - ๐ฎ๐ณIndia kk5190
Removing the call to accessCheck(TRUE) will still result in an error for regular Drupal entities. Itโs important to note that explicitly invoking accessCheck is necessary. Refer to this source: Access checking must be explicitly specified on content entity queries โ
The error in the current code arises because the code checks for a non-empty $bundle_property_name before calling accessCheck. However, for Site Studio content components, $bundle_property_name is empty, causing our code to skip adding the access check and resulting in a fatal error. To resolve this, the correct approach is to ensure the access check is called before executing the query, regardless of the value of $property_bundle_name. Therefore, in addition to Martinโs changes, we should also include a call to accessCheck(TRUE). Please review the updated patch provided below. While potential test case failures are uncertain, this solution will function correctly for both component contents and standard Drupal entities.
- Status changed to Needs review
over 1 year ago 8:57am 3 August 2023 - Status changed to RTBC
over 1 year ago 9:06am 3 August 2023 -
scor โ
committed c0d6efa0 on 4.x authored by
mandclu โ
Issue #3366589 by mandclu: Fatal error when using the "Enqueue Content"...
-
scor โ
committed c0d6efa0 on 4.x authored by
mandclu โ
- Status changed to Fixed
about 1 year ago 10:22pm 17 October 2023 - ๐จ๐ฆCanada scor Toronto
Fixed in https://git.drupalcode.org/project/acquia_perz/-/commit/c0d6efa0c266bfbd... and released in latest stable release.
Automatically closed - issue fixed for 2 weeks with no activity.