Issue with hook_domain_conf() invoke

Created on 1 August 2012, almost 13 years ago
Updated 2 May 2025, about 1 month ago

Hi,

There is a bug in hook invoke or I think it is. If it is not I think this should be included as a feature in next release.
The problem is following:

When you call the function $extra = domain_conf_api(TRUE, $settings); you do not pass the current loaded domain ( current arg(4) ). In this function you are invoking the hook_domain_conf() with parameter current domain that is actually a global variable so it can be accessed everywhere and I personally cannot see why we need this. So if we pass the loaded by menu system domain parameter to domain_conf_api() and after that we pass this variable to module_invoke it makes much more sense. I've made a patch for this.

Regards.

✨ Feature request
Status

Closed: outdated

Version

3.4

Component

Code

Created by

πŸ‡§πŸ‡¬Bulgaria skek

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Production build 0.71.5 2024