We have a lot of hooks in core/modules/system/system.api.php that do not relate to the System module specifically. See parent issue #2299715: [meta] Move core hooks from system.api.php to core.api.php or other files for more discussion.
For this issue, the task is to move a bunch of module-related hooks to a new file called module.api.php in the same directory.
The file needs to have a @file block at the top with an appropriate description, probably something like:
Hooks related to the module install/uninstall and hook systems.
Then this file should have:
/**
* @addtogroup hooks
* @{
*/
The end of the file should also have
/**
* @} End of "addtogroup hooks".
*/
And then in between the addtogroup blocks, the following hooks should be removed from system.api.php and moved to this new file (probably in this order):
hook_hook_info()
hook_module_implements_alter()
hook_system_info_alter()
hook_module_preinstall()
hook_modules_installed()
hook_install()
hook_module_preuninstall()
hook_modules_uninstalled()
hook_uninstall()
hook_install_tasks()
hook_install_tasks_alter()
hook_update_N()
hook_update_dependencies()
hook_update_last_removed()
hook_updater_info()
hook_updater_info_alter()
hook_requirements()
Comment | File | Size | Author |
---|---|---|---|
#2 | 2366583-2.patch | 71.5 KB | rpayanm |
Comments
Comment #1
jhodgdonComment #2
rpayanmComment #3
jhodgdonLooks great except for
So I fixed that with the commit to 8.0.x. Thanks!