Originally, Panels IPE was more or less an on or off deal. As Panels has progressed, there have become more places where the IPE might be used in more localized areas. Unfortunately, with IPE on, you either get it or you don't with Page Manager. This can be a problem if you want content people to be able to use IPE on things that aren't Page Manager (particularly Panelizer) but not on page manager pages. We need an additional permission to allow this to be possible. Patch forthcoming.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

merlinofchaos’s picture

Status: Active » Needs review
FileSize
1.44 KB

Here is the patch. It needs to be tested.

beeradb’s picture

Status: Needs review » Reviewed & tested by the community

I tested this and it works as expected. My only nitpick would be on the description of the new permission, "Allows users who do not normally have access to administer page manager to use the IPE with page manager.". The new permission still requires the "Use in-place editor" permission to take effect, which I understand and agree with, but I feel like the description text makes it sounds as if you can edit page-manager pages with this permission alone.

Here's an alternate description suggestion:
"Allows users with access to the In-Place editor to administer page manager pages."

Going to mark RTBC as the functionality works as expected, and I personally don't see a reason to hold it up based on a small language issue.

merlinofchaos’s picture

Status: Reviewed & tested by the community » Fixed

Fixed up the wording a bit and committed. I added a line to make it clear that it is only needed without "use page manager" permission.

Status: Fixed » Closed (fixed)

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

  • Commit cf8fb3f on 7.x-3.x, 7.x-3.x-i18n, 8.x-3.x by merlinofchaos:
    Issue #1854374: Add additional permission to make it possible to...