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.
I am working on an aegir server that will only have server nodes.
I am unable to only enable hosting module without causing errors, as it calls functions that depend on hosting_site to be enabled.
The patch moved three functions to hosting_site.module: hosting_get_hostmaster_site_nid(), hosting_get_hostmaster_nid(), hosting_get_hostmaster_platform_nid()
It also checks to see if hosting_site module is enabled before trying to create a verify task for hostmaster from the site in hosting_feature_rebuild_caches()
Comments
Comment #3
Jon PughBranch '2704247-hostmaster-get-nid'
Comment #4
ergonlogicShouldn't
hosting_get_hostmaster_site_nid()
andhosting_get_hostmaster_platform_nid()
now be calledhosting_site_...
?hosting_get_hostmaster_nid()
should probably stay where it it, but callhosting_site_get_hostmaster_site_nid()
.I'm diffident about the
module_exists('hosting_site')
, though. Seems like it would be cleaner to add support for a new hook:hook_post_hosting_feature_rebuild_caches()
. This would allow for implementinghosting_site_post_hosting_feature_rebuild_caches()
that could contain everything aftermenu_rebuild()
.But this verify task is how Hosting Features' backends get registered with Drush. So this change would mean that any module that didn't depend on
hosting_site.module
would have to find another way to register itsdrush/
dir as a Drush include. At leasthook_post_hosting_feature_rebuild_caches()
should allow such alternatives. But maybe we should give this whole mechanism further thought. Either way, we should probably find a way to document it. Maybe add a couple lines to the example modules.Comment #5
ergonlogic