Wrong hook called in domain_nav_render() ? [#284398]

Heya,

I just tried using hook_domainnav() as described in the domain API documentation (API.php) but it turns out that the hook never actually gets invoked.

Function domain_nav_render() around line 188:

      $extra = array();
      $extra = module_invoke_all('domainlist', $value);
      $value = array_merge($value, $extra);
      $options[$value['domain_id']] = $value;

hook_domainlist() is invoked instead of hook_domainnav()... is this a typo or the API documentation out of date?

(I'm working with HEAD: v1.9 2008/05/18 14:37:13)

Comments

Comment #1

agentrickard

he/him

English

Georgia (US)

commented 18 July 2008 at 16:15

Status:

Active

» Needs review

This is a case where I changed the name of the hook -- for clarity, since it does not generate a list -- and forgot to change the function.

It should be:

      $extra = array();
      $extra = module_invoke_all('domainnav', $value);
      $value = array_merge($value, $extra);
      $options[$value['domain_id']] = $value;

This would apply to D5 and D6.