Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
It looks like version 2.1 is only compatible with Drupal 8.8.x and above as that is when TrustedCallbackInterface.php was introduced. May need to document somewhere as it killed a deployment on me.
Comment | File | Size | Author |
---|---|---|---|
#7 | 3121816-07.patch | 756 bytes | wengerk |
Comments
Comment #2
adriancidComment #4
adriancidComment #5
kbrinnerMe too. I noticed that the contrib module address anticipated such an issue in its AddressDefaultFormatter class:
For now I'm keeping the admin_toolbar at 2.0 while looking in to a more complete fix.
Comment #6
adriancid@Kbrinner I release yesterday another version with this patch, so if you're using Drupal 8.8.0 you can install it. I really miss this when I commit the patch in #3117452: Replace admin_toolbar_prerender_toolbar_administration_tray by trusted callback
Comment #7
wengerkComment #8
wengerkFor people trying/needing to use
8.2.1
with a Drupal8.7.x
- here is a patch applying the same trick as the address module suggested in #5.This patch only works if you use the version 8.2.1 of Admin Toolbar.
You can apply the patch using the "extra > patch" section of your composer.json
PS: re-add the original issue summary ... was not awake this morning...
Comment #9
s-jack CreditAttribution: s-jack commentedAfter editing the composer.json file directly, how do I execute the composer command to apply the extra section?
Comment #10
esolitos@s-jack: A safe way to do it is to run
composer update nothing
.Comment #11
s-jack CreditAttribution: s-jack commentedHi, esolitos.
I ran composer update because there were updates for other modules and libraries.
It was successful, thank you.
Is there a composer command that can only apply patches without updating modules or libraries?
Comment #12
xmacinfoYes, it is:
composer update --lock
Comment #13
s-jack CreditAttribution: s-jack commentedHi, xmacinfo.
Thanks for telling me useful commands.
Comment #15
jndvrk CreditAttribution: jndvrk commentedSorry for adding onto a closed issue. Has version 2.1 been updated with the above fix? I just tried installing version 2.1 onto a Drupal 8.7.8 site and all I was seeing is 'skip to main content' and nothing else. Version 2.0 works without issues.
(was updating from 1.x)
Comment #16
lithiumlab CreditAttribution: lithiumlab commentedFound this issue while updating to 2.1 on a Drupal 8.7.10 site.