Hi there,

activation of the log module causes views with path "/log" or "/logentries" being displayed in admin theme.

The module should not interfere with other views.

Comments

e5sego created an issue. See original summary.

m.stenta’s picture

m.stenta
he/him
Primary language English
CreditAttribution: m.stenta commented
Title: Module causes views using specific path identifiers to be displayed in admin theme » Make log module's hook_admin_paths() more specific

There is a settings page at admin/config/log/settings with an option to disable that.

The actual code that makes it happen is in log.module:

/**
 * Implements hook_admin_paths().
 */
function log_admin_paths() {

  // Display log paths with the admin theme, if specified.
  $paths = array();
  if (variable_get('log_admin_theme', TRUE)) {
    $paths = array(
      'log*' => TRUE,
    );
  }
  return $paths;
}

Perhaps we can modify that to be a little more specific.

Maybe just changing log* to log/* would do it. That should prevent both /log and /logentries from matching.

  • m.stenta committed baa7565 on 7.x-1.x 
    Issue #2982585: Make log module's hook_admin_paths() more specific
m.stenta’s picture

m.stenta
he/him
Primary language English
CreditAttribution: m.stenta commented
Status: Active » Fixed

I pushed a commit that changes log* to log/*.

Status: Fixed » Closed (fixed)

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