- Issue created by @sker101
- πΊπΈUnited States sker101 NYC
here's a patch that replaces the jQuery once with the new once library from core.
- πΊπΈUnited States sker101 NYC
Not sure if it's worth but hopefully this patch could address most of the tests.
There are two functional javascript tests that might fail due to upstream issue from Panels module
See https://www.drupal.org/project/panels/issues/3352958#comment-15002421 π Functional Javascript Test Failures Active
- last update
over 1 year ago Patch Failed to Apply - last update
over 1 year ago run-tests.sh fatal error - π¦πΊAustralia mingsong π¦πΊ
Thanks @Shen for the patch.
May I ask a question about the access check in following codes?
/** @var \Drupal\panelizer\Plugin\PanelizerEntityManager $panelizer_manager */ $panelizer_manager = \Drupal::service('plugin.manager.panelizer_entity'); @@ -31,6 +31,7 @@ function panelizer_post_update_rename_layout_machine_names(&$sandbox) { $storage = \Drupal::entityTypeManager()->getStorage($entity_type); $ids = $storage->getQuery() ->condition('panelizer', serialize([]), '<>') + ->accessCheck(FALSE) ->execute();
Do we need to check the access in this case?
- πΊπΈUnited States sker101 NYC
@Mingsong
I believe not because this query is triggered during a "post_udpate" hook.
I added the `accessCheck` method because it's required in Drupal 10. - First commit to issue fork.
- Status changed to RTBC
over 1 year ago 2:46am 6 July 2023 - First commit to issue fork.
- @codebymikey opened merge request.
- π©πͺGermany jan kellermann
Thank you very much! I can confirm this patch also.
Notice for people who want to add this patch: Remove #3325442 before because it is included in this patch already.
- last update
about 1 year ago 31 pass, 19 fail - Issue was unassigned.
- Status changed to Fixed
about 1 year ago 10:03pm 10 October 2023 -
DamienMcKenna β
committed 23c2f969 on 8.x-4.x authored by
sker101 β
Issue #3352704 by sker101, codebymikey, hamzadwaya, Mingsong, joseph....
-
DamienMcKenna β
committed 23c2f969 on 8.x-4.x authored by
sker101 β
- πΊπΈUnited States zanematthew Baltimore, MD
Note for others, patch does not apply and current 4.x version still produces various once errors in the console `$(...).once is not a function`.
- π¨π¦Canada joseph.olstad
@zanematthew, I'd say try the latest tagged release, no need for a patch now that it's released and included in the build, please rebuild your cache after upgrading to the latest release
composer require 'drupal/panelizer:^4.5'
This will get you 4.5 or newer, 4.5 being just released today. - πΊπΈUnited States zanematthew Baltimore, MD
@joseph.olstad I revisited this and it was another module causing the .once error. This update works as expected, thanks!
Automatically closed - issue fixed for 2 weeks with no activity.