The Doxygen comment indicates that the $direction parameter is optional. It's not: without passing it I get a "Notice: Undefined index: direction" warning.

See nice_menus.module:530:

/**
 * Theme the main menu as a Nice menu.
 *
 * @param $direction
 *   Optional. The direction the menu expands. Default is 'down'.
[...]
*/
function theme_nice_menus_main_menu($variables) {
  $direction = $variables['direction'];
[...]
}

The same applies for theme_nice_menus_secondary_menu().

Comments

vordude’s picture

Changing the comments would be the easy answer. But, probably not "right". This can likely be optional, but when the theme functions changed to one argument only, it was likely sloppily done.

vordude’s picture

Issue summary: View changes

Added markup for function name.

avpaderno’s picture

Version: 7.x-2.0 » 7.x-2.x-dev
Issue summary: View changes
astonvictor’s picture

Status: Active » Closed (outdated)

D7 reached its EOL back in January 2025, and there is no active release for D7 for this module anymore.
Development or support is not planned for D7. All D7-related issues are marked as outdated in a bunch.

Now that this issue is closed, please review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, please credit people who helped resolve this issue.