- Issue created by @catch
- 🇬🇧United Kingdom catch
It's not a 100% duplicate, e.g. query_node_access_alter only appears to be in this file. Might be necessary to merge it with NodeHooks.
- 🇧🇷Brazil charlliequadros
Hi @catch
Is the file name "nodehooks1"?
I couldn't find it. Could you tell me where it is located? - 🇦🇺Australia mstrelan
I don't think it's a duplicate at all? The original
NodeHooks.phpwas added in 📌 OOP hooks using event dispatcher Needs review , I believe as a demonstration of implementing the same hook twice (hook_user_cancel). The newerNodeHooks1.phpwas added in 📌 [ignore] Convert everything everywhere all at once Active when all the remaining hooks were converted. I think we should move the 2 hooks in NodeHooks to NodeHooks1, delete NodeHooks and then rename NodeHooks1 to NodeHooks. Any further refactoring should be done later. - 🇬🇧United Kingdom catch
Given it's not a duplicate it's also not a novice issue.