Closed (fixed)
Project:
Panelizer (obsolete)
Version:
7.x-3.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
24 Oct 2012 at 17:43 UTC
Updated:
27 Apr 2014 at 13:00 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
populist commentedI did some more research here and can report the following:
Layouts for the IPE are set in ajax_change_layout() of plugins/display_renderers/panels_renderer_ipe.class.php in the Panels.module. The logic there goes like this:
My guess the problem here is that the $this->display->allowed_layouts's isn't properly getting set in panelizer_panels_cache_get() since a quick look at that logic shows we have explicit handling of the allowed content types, but not the layout.
Comment #2
populist commentedHere is a patch that likely does the trick. Uses the same pattern as the content types, but adds in the layouts too. It requires you clear your CTools cache to make it work, but I tested it on Panopoly and saw it setting correctly.
Comment #3
merlinofchaos commentedThis is a simple fix. Committed and pushed.
Comment #4
damienmckennaShould we backport this to v2 too?
Comment #5
tim.plunkettI'm seeing the opposite problem. I can open a new issue later, but this touches some of the same code, and seems like just a follow-up.
Basically, the "Use the same allowed layouts as standard Panels pages?" setting is ignored, and all layouts are presented.
Comment #6
ec1ipsis commentedI'm seeing the exact same issue mentioned in #5. Will test the patch and report back.
Comment #7
ec1ipsis commentedThe patch applied cleanly and solves the problem for me using Drupal version 7.22, Panelizer version 7.x-3.1.
Comment #8
damienmckennaJust a quick reroll.
Comment #9
damienmckennaCommitted.