When the Breadcrumb pane (available under "Page elements") is used to render the breadcrumbs, ctools_page_breadcrumb_content_type_render() is executed before panels_breadcrumbs_ctools_render_alter(), and therefore this module has no effect on the output.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

devuo’s picture

Assigned: Unassigned » devuo

This is a known issue. There's no way to have our code execute before Breadcrumb content pane. The solution I used was to write my own implementation of the Breadcrumb content pane, which I called "Panels Breadcrumb" in a project I worked in recently. I'll add this in 7.x-1.5.

ghindle’s picture

Any chance you can attach this as patch now?

devuo’s picture

I was looking into this today, and apparently the proper way to get this to work is by having Panels panel_context.inc patched and having it saving the current page handler in a static variable much like it does in panels_get_current_page_display() to something like panels_get_current_page_handler() (which currently doesn't exists) otherwise there's simply no way to get the "Panels Breadcrumbs" configuration except in a very complex, weird sort of way. I'll create a new issue on the Panels project issue queue with a feature request patch that adds this required functionality.

devuo’s picture

Version: 7.x-1.4 » 7.x-1.5
Status: Active » Postponed

I'll set this as postponed until the patch to Panels has been commited.

devuo’s picture

Status: Postponed » Needs review
FileSize
6.24 KB

The following patch adds the required functionality. Notice however it requires #1418598: Set page_manager_get_current_page() before the page render to be applied as well to Panels.

devuo’s picture

Status: Needs review » Postponed

Postponed until we've got news from Panels side.

merlinofchaos’s picture

The patch to Panels should be unnecessary as page_manager_get_current_page() can be used instead.

devuo’s picture

Merlin please see the referenced issue in Panels for the response.

dubois’s picture

Version: 7.x-1.5 » 7.x-1.6

FYI @devuo, your abovementioned patches for Panels #1418598 & Panels Breadcrumbs apply cleanly to Panels 3.2 and Panels Breadcrumbs 1.6 to yield the desired behaviour.

My users find it very handy to use the breadcrumb pane as suggested by jweowu. Hope you and merlinofchaos agree on a way to implement this without patches. Thank you for a very useful little module.

devuo’s picture

Status: Postponed » Closed (fixed)

This has been fixed in the latest 7.x-2.x-dev version.