- Issue created by @slucero
- last update
over 1 year ago 358 pass, 2 fail - @slucero opened merge request.
- last update
over 1 year ago 359 pass - Status changed to Needs review
over 1 year ago 4:15pm 23 August 2023 - last update
over 1 year ago 359 pass - 🇺🇸United States slucero Arkansas
I'm uploading a patch of the current state of the MR for convenient access. This may be applied in a
composer.json
file with the following snippet:{ [...] "extra": { "patches": { "drupal/patternkit": { "#3380008: Saved Pattern Entities Include Drupal Root In Cached Asset Paths": "https://www.drupal.org/files/issues/2023-09-08/patternkit-3380008--fix-cached-asset-paths--4.patch" } } } }
- 🇺🇸United States slucero Arkansas
Testing revealed that the testing instructions I originally posted don't successfully reproduce the issue. Below is a revised set of instructions that should reproduce the issue as expected:
- Install Patternkit, Patternkit Example, and Layout Builder modules
- Configure a Node type to display with Layout Builder and per-node layout customizations
- Create and save a new test Node
- Edit the layout for the new Node, and create a new [Patternkit] Example block with test content
- Save the layout
- Move the Patternkit Example module folder to a new location in the Drupal installation, such as
modules/contrib/patternkit_example/
- Run the following Drush command to remove cached asset values:
drush sqlq "UPDATE pattern_revision AS revision SET revision.schema=NULL, revision.template=NULL, revision.hash=NULL WHERE revision.revision IN (SELECT revision FROM pattern_revision)"
- Rebuild all caches
- Attempt to view the test page
- Observe failed rendering for affected blocks
At this point, the resolution for this issue may be tested:
- Install the patch for this issue
- Run update hooks
- View the test page
- Expect to see the failed blocks loading successfully
- 🇮🇳India minsharm India
Thanks for the thorough explanation and updating the instructions Stephen !!
I tried to reproduce as well re-verify the issue. Result looks good to me now.
Steps :
- Install Patternkit, Patternkit Example, and Layout Builder modules
- Configure a Node type to display with Layout Builder and per-node layout customizations
- Create and save a new test Node
- Edit the layout for the new Node, and create a new [Patternkit] Example block with test content
- Save the layout
- Move the Patternkit Example module folder to a new location in the Drupal installation, such as modules/contrib/patternkit_example/
- Run the following Drush command to remove cached asset values.
- drush sqlq "UPDATE pattern_revision AS revision SET revision.schema=NULL, revision.template=NULL, revision.hash=NULL WHERE revision.revision IN (SELECT revision FROM pattern_revision)"
- Rebuild all caches
- Attempt to view the test page
- Observe failed rendering for affected blocks
Recreation Results (Before Fix)
Getting this error : Unable to decode the schema for pattern @my/pattern/name: Syntax errorRetest Results (After Fix)
Tested the issue with merge_requests/82 and failed block has been loaded successfully - Status changed to RTBC
over 1 year ago 3:15pm 18 September 2023 - last update
over 1 year ago 359 pass - last update
over 1 year ago 359 pass - Status changed to Fixed
over 1 year ago 3:27pm 18 September 2023 - 🇺🇸United States slucero Arkansas
Merged for inclusion in the Beta 8 release: 🌱 Patternkit Beta 8 Release Plan Active
Automatically closed - issue fixed for 2 weeks with no activity.