With the Panels storage change, we're checking the specific permissions for each entity type, bundle and operation, but not the full 'administer panelizer' permission.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dsnopek created an issue. See original summary.

dsnopek’s picture

Status: Active » Needs review
FileSize
786 bytes

Here's a patch!

I'm not sure if it makes sense to do the same thing in the 'panelizer_entity' plugin. At the moment, I'm leaning to not doing it, because you still should have permission to edit the entity (I think). Any other opinions?

DamienMcKenna’s picture

Issue tags: +Needs tests

Lets add some tests.

DamienMcKenna’s picture

Working on some tests.

DamienMcKenna’s picture

DamienMcKenna’s picture

The test-only patch should fail while the full patch should pass.

The last submitted patch, 4: panelizer-n2788851-4-testsonly.patch, failed testing.

The last submitted patch, 5: panelizer-n2788851-5-testsonly.patch, failed testing.

DamienMcKenna’s picture

Status: Needs review » Fixed

Committed. Thanks dsnopek!

dsnopek’s picture

Woohoo! Thanks for the tests and the fast commit :-)

Status: Fixed » Closed (fixed)

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