There is an issue in mini panels:

Undefined variable: current_page in panels_mini_block_list_alter()

This does not throw an error but rather a notice in PHP.

The culprit code is here:

  if (module_exists('page_manager')) {
    $current_page = page_manager_get_current_page();
  }

Which throws the notice here:

if (!$current_page || empty($current_page['contexts'])) {

Where the variable should still be instantiated even if page_manager doesn't get picked up by module_exists().

A possible fix:

  $current_page = false;
  if (module_exists('page_manager')) {
    $current_page = page_manager_get_current_page();
  }

Comments

totipotek created an issue. See original summary.

totipotek’s picture

Assigned: totipotek » Unassigned
totipotek’s picture

StatusFileSize
new630 bytes
totipotek’s picture

Status: Active » Needs review

This patch is ready. Needs to be reviewed by the community for inclusion in next release.

heykarthikwithu’s picture

hi @totipotek, i tried to reproduce the bug, but was not able to..

please provide the steps to reproduce the bug.

jacob.embree’s picture

Issue summary: View changes
StatusFileSize
new498 bytes

To reproduce this page_manager needs to be disabled. I am not sure of all conditions or steps to reproduce.

This patch uses an all-capital "FALSE".

joelpittet’s picture

Version: 7.x-3.5 » 7.x-3.x-dev
Status: Needs review » Fixed

Thanks I've committed this to the dev branch.

  • joelpittet committed 7ea6a84 on 7.x-3.x authored by totipotek
    Issue #2614220 by totipotek, jacob.embree, heykarthikwithu: Undefined...

Status: Fixed » Closed (fixed)

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