When you set layout to bundle default storage id gets changed to [entity_type]:[bundle]:default, but storage type remains "panelizer_entity" which isn't correct. Later when you try to edit this display it will check access using panelizer_entity_panels_storage_access which expect storage id to have entity id and you'll get access denied. Apparently we need to add code that changes storage type after saving display as default so it checks for access using panelizer_default_panels_storage_access.
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | panelizer-storage_type_issue-2864607-1.patch | 463 bytes | sickness29 |
Comments
Comment #2
sickness29 commentedPatch for the issue.
Comment #3
sickness29 commentedComment #4
damienmckennaThat's an interesting bug, thanks for providing a patch.
I think we'll need to put together tests to confirm the bug and confirm the patch fixes it.
Comment #5
jorrit commentedI can reproduce the problem and confirm that this patch fixes the problem.
Comment #6
jorrit commented@sickness29: do you have time to add a test to panelizer.node.test?
Comment #7
aubjr_drupal commentedExtremely late response on this, but I've had to fix this issue in many databases across sites (after a database update in Panopoly that addressed other issues).
If nobody added a test to panelizer.node.test yet, and it'd mean that this gets ported into Panelizer as a patch upstream for sure, I'd be willing to give it a shot. Is that's all that is holding this up (no .test update)?