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.Change notice: http://drupal.org/node/1894902 - Module/hook system functions replaced with module_handler service
We need to replace all our module_invoke_all, module_list, etc adapted to the new system.
| Comment | File | Size | Author |
|---|---|---|---|
| #7 | module_handlers-1895468-7.patch | 4.51 KB | plopesc |
Comments
Comment #1
plopescHello
I was working on this replacement when you created the issue :)
Patch pass tests in my local environment and I've made some other manual tests without any problem.
Maybe more manual testing is necessary.
Regards.
Comment #2
moshe weitzman CreditAttribution: moshe weitzman commentedPatch looks good. Lets commit it after some manual testing.
We should start thinking about devel_generate includes as D8 plugins (text, image, etc.).
Comment #3
pcambraI've got an UI proposal for the devel generate rewrite in here: #1877460: Rearchitect devel generate to allow generation of any kind of content entity
Comment #4
parthipanramesh CreditAttribution: parthipanramesh commentedWorks fine. Thanks.
Comment #5
pcambraI really want #2154141: DevelGenerate as a D8 plugin type in and this might conflict, I rather mark it postponed and re roll it after the first gets in.
Comment #6
pcambraD8 generation tools are plugins now, getting this back to needs work.
Probably most of the patch already went in, though.
Comment #7
plopescRe-rolling.
Comment #9
moshe weitzman CreditAttribution: moshe weitzman commentedCommitted. Thanks.