Active
Project:
Hooks
Version:
8.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
18 Oct 2015 at 00:29 UTC
Updated:
17 Nov 2015 at 03:52 UTC
Jump to comment: Most recent
This module started with only alter(), It would be good to support invoke and invokeAll although that may require many core patches.
The biggest problem to solve here is that core and contrib assume that hooks are always functions, and they construct their names with $module . '_' . $hook which makes it impossible for us.
Decide how to handle the $function = $module . '_' . $hook pattern.
Comments
Comment #2
benjy commentedComment #3
benjy commentedComment #4
benjy commentedComment #5
benjy commentedComment #6
cweagansI don't think there's really any way around using PhpStorage or some other clever thing like that for the $module . '_' . $hook case until #2616814: Delegate all hook invocations to ModuleHandler is solved.
That's possibly a separate issue, though. Making ->invoke() and ->invokeAll() work should be reasonably straightforward, even if they aren't called appropriately in some cases.