the rules module includes a module named rules_admin this module has a hook_theme function

the admin_theme module has a hook_admin_theme but it also loads the hook_theme function of the rules_admin module.

therefore a number of empty checkboxes are created on the admin theme settings page

perhaps the hook_admin_theme method should have another name to prevent namespace collisions as there are probably more modules with a *_admin_theme function name

Comments

Johnny vd Laar’s picture

Johnny vd Laar commented

for now i've modified the admin_theme_list function into this:

/**
 * Get all module defined options.
 *
 * @return
 *   Array. All options.
 */
function admin_theme_list() {
  $options = array();
  foreach (module_list() as $module) {
    $module_options = module_invoke($module, 'admin_theme', 'info');
    if (count($module_options) > 0) {
      foreach ($module_options as $option => $info) {
        if(isset($info['title']) && isset($info['description'])) {
          $info['option'] = $option;
          $info['module'] = $module;
          $options[] = $info;
        }
      }
    }
  }
  return $options;
}
davyvdb’s picture

davyvdb commented
Status: Active » Fixed

This has been fixed in CVS.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.