Steps to reproduce:

1. Create new content type. Save it.
2. I've exported content type to Features (probably not relevant).
3. Edit again and enable two checkboxes at /admin/structure/types/manage/%: 'Panelize' and 'Full page override'. Save it.
4. Additional Panelizer tab is available, so I'm going to: Administration » Structure » Content types » MyContentType » Panelizer » Full page override
5. The page is not found: The requested page "/admin/structure/types/manage/%/panelizer/page_manager" could not be found.

Workaround is to enable: 'Provide an initial display named "Default"' at /admin/structure/types/manage/%, then then 'Full page override' is available as expected.

Disabling the option, goes off again and at /admin/structure/types/manage/%/panelizer/page_manager/content it gives: Access denied (You are not authorized to access this page.) which is pretty weird, as I'm logged-in as admin.

This was reproduced on two different environments.

CommentFileSizeAuthor
#9 panelizer-n2546906-9.patch14.74 KBDamienMcKenna
#7 panelizer-n2546906-7.patch13.02 KBDamienMcKenna
#6 panelizer-n2546906-6.patch2.7 KBDamienMcKenna
#5 panelizer-n2546906-5.patch472 bytesDamienMcKenna
#4 panelizer-n2546906-4.patch446 bytesDamienMcKenna
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

kenorb created an issue. See original summary.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
Parent issue: » #2155813: Plan for Panelizer 7.x-3.2 release
DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented

The logic that controls whether this page is available is too closely tied to whether there's an existing display, it should be tied to what the configuration options are.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Status: Active » Needs review
FileSize
panelizer-n2546906-4.patch446 bytes

So the problem is that the access is determined by panelizer_default_settings_page(), which returns MENU_NOT_FOUND if no page is suitable for the given path, instead it'd be better to return a simple message.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
FileSize
panelizer-n2546906-5.patch472 bytes

A slightly better message.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
FileSize
panelizer-n2546906-6.patch2.7 KB

This fixes the subpages so they give the same message if no default display is available, rather than throwing 403 errors.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
FileSize
panelizer-n2546906-7.patch13.02 KB

Tests!

Status: Needs review » Needs work

The last submitted patch, 7: panelizer-n2546906-7.patch, failed testing.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Status: Needs work » Needs review
FileSize
panelizer-n2546906-9.patch14.74 KB

Reverted changes to the node_revisions test file.

  • DamienMcKenna committed fed2039 on 7.x-3.x 
    Issue #2546906 by DamienMcKenna: If a default display is not available...
DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Status: Needs review » Fixed

Committed.

Status: Fixed » Closed (fixed)

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