- Issue created by @ptmkenny
- 🇯🇵Japan ptmkenny
ptmkenny → changed the visibility of the branch oop_hooks4 to hidden.
- 🇺🇸United States mlncn Minneapolis, MN, USA
Major, because a priority major is postponed for this currently. (This and 📌 Support OOP hooks Active (or combining them into one issue again) are needed for Field Encrypt to work on Drupal 11.1.)
Definitely do think the hooks should be in separate files based on usage, loading only what is needed when it is needed is a big benefit of OOP hooks, but that can be a later code reorganization.
- 🇬🇧United Kingdom alexpott 🇪🇺🌍
We should definitely make 4.x only 11.1.x and up. Trying to support 11.0.x and 11.1.x on the same branch will be extremely hard and not worth the work.
- 🇯🇵Japan ptmkenny
After some thought, I'm going to close this in favor of the parent issue 📌 Support OOP hooks Active .
I originally spun off this issue because I thought there would be a way to support both 10 and 11.1+, but as pointed out in #9, because of the changes to the handling of eval(), there's really no good way to do this. So all work on hooks will go forward in the parent issue, and we will leave the 3.x branch with the conventional hooks.