function page_manager_http_response_admin_summary(...) sometimes generates function 'panels_panel_context_get_display' not found errors when trying to acces the edit panel page (user/%user) for a Panel containing a Http response Variant.

The solution is to load the panel_context.inc file just before the function call, like this:
module_load_include('inc', 'panels', 'plugins/task_handlers/panel_context');

CommentFileSizeAuthor
#3 967590-not-panels-at-all.patch4.38 KBmerlinofchaos
patch.diff633 bytesz7
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

merlinofchaos’s picture

Status: Needs review » Active

That solution can't be correct.

1) Page Manager has no dependency on Panels, so directly referencing one if its files is wrong.
2) That's a task handler plugin. Even if we were to directly reference a Panels file, we'd use page_manager_get_task_handler(). That said, 1) is more important.

So that leads us to the question...why is a Panels function getting called when a page manager task handler is in place?

un11imig’s picture

The patch works for me.

Thanks!

merlinofchaos’s picture

Status: Active » Fixed
FileSize
4.38 KB

The problem is that the HTTP Response task handler is cut & pasted from Panels and I didn't sufficiently scrub it. Sorry about that.

Patch attach. Fixes already committed.

Status: Fixed » Closed (fixed)

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

netsensei’s picture

Status: Closed (fixed) » Reviewed & tested by the community

Just ran into this problem while tinkering with the page manager without having panels enabled. Patch fixes the issue. Thanks!

merlinofchaos’s picture

Status: Reviewed & tested by the community » Active

Why did you re-open this? I said in #3 it was committed. :/

merlinofchaos’s picture

Status: Active » Closed (fixed)
Pls’s picture

It would be great to have new version of ctools as we now have to use 1.x-dev version, in order to enjoy this commit and patch. Thanks!