Closed (fixed)
Project:
Chaos Tool Suite (ctools)
Version:
6.x-1.3
Component:
Page Manager
Priority:
Critical
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
9 Feb 2010 at 10:50 UTC
Updated:
10 Jun 2010 at 21:30 UTC
Using Page Manager without Panels enable, I ran into this the following error while trying to change the context settings of a page:
Fatal error in /home/robert/Sites/sites-6/all/modules/ctools/includes/context-task-handler.inc on line 364: Call to undefined function panels_get_path()
I suppose the line 364 of context-task-handler.inc
// @todo -- this CSS is actually dependent upon the plugins which means
// the plugins need to be able to add it.
drupal_add_css(panels_get_path('css/panels_admin.css')); should implies a dependencies[] = panels within the ctools.info file or (better to my own opinion) should be removed/avoid using a Panels function within CTools.
As a quick fix I wrapped the lines above with a if(module_exists('panels')) { and it works again like a charm.
All the best and thanks for the awesome modules,
Valentin
Comments
Comment #1
merlinofchaos commentedOh yeah, that should definitely not be in there.
Comment #2
suzanne.aldrich commentedI was trying to un-install Page Manager and I got to that fatal error. I tried wrapping the code referenced above in the if(){} statement, but that didn't work. I re-enabled Panels, un-installed Page Manager, got another error:
Invalid argument supplied for foreach() in /var/www/mysite.com/sites/all/modules/ctools/includes/export.inc on line 322.But Page Manager got un-installed successfully, it looks like. I'll check my db tables and variables to make sure it's really gone. Seems like Panels should be a dependency, for sure, but there's another bug in there too.
Comment #3
sdboyer commentedFixed this by simply removing the offending line. The css it's trying to add is added via a preprocess function by looking at the task handler's 'admin css' value.