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.
Hey !
When I updated drupal to the version 8.3.1, I also updated display suite. The update failed because of this module, I had to uninstall it, do the update, and reinstall it at the end.
It seems that the need of the CSS rebuild when the cache is cleared make the update crash.
Comment | File | Size | Author |
---|---|---|---|
#3 | simple_menu_icons-change_hook_not_rebuild_on_update-2871491-3.patch | 428 bytes | sylvainar |
bug.txt | 1.48 KB | sylvainar |
Comments
Comment #2
sylvainar CreditAttribution: sylvainar commentedIt seems that it's not the good hook.
According to Drupal.org drupal API, the cache flush hook shouldn't be used to flush custom cache.
The hook_rebuild should be used instead.
I'm working on a patch, I'll keep you informed.
Comment #3
sylvainar CreditAttribution: sylvainar commentedYup, changing the hook was the solution. I retried all the update procedure and it doesn't fail anymore, and the module is still working properly.
Comment #4
sylvainar CreditAttribution: sylvainar commentedComment #5
pifagorhttps://www.drupal.org/project/simple_menu_icons/issues/2880544#comment-...
Comment #6
pifagor