Module:
Chaos tool - 7.x-1.0-rc1

Description:
When on the panel content section, you're able to click on the pane or regions setting gear to show you the configuration options (styles, visibility rules, etc). If you're not logged in as a super user (uid 1) then the style option doesn't appear in the configuration menu for both pane or regions.

The account that's not able to see the style options, has all permissions granted. Is there something I'm missing?

Thanks,
Droath

CommentFileSizeAuthor
#16 panels-style-perms-1313642-15.patch374 bytessteveoliver
#5 panels-style-perms.1313642-5.patch537 bytesdroath
#4 panels-style-perms.1313642.patch527 bytesJon Pugh
#2 panels-style-perms.1313642.patch920 bytesJon Pugh
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Jon Pugh’s picture

Jon Pugh
he/him
Primary language English
Location Newburgh, NY
CreditAttribution: Jon Pugh commented

Missing 'administer panels styles' in hook_perm... patch on the way...

Jon Pugh’s picture

Jon Pugh
he/him
Primary language English
Location Newburgh, NY
CreditAttribution: Jon Pugh commented
FileSize
panels-style-perms.1313642.patch920 bytes

Patch attached.

@droath: to fix your problem you'll have to apply the patch and then give a role the permission "administer panels styles"

merlinofchaos’s picture

merlinofchaos CreditAttribution: merlinofchaos commented
Status: Active » Needs review

DOH. I really forgot to put the perm in? Man that's sad!

Jon Pugh’s picture

Jon Pugh
he/him
Primary language English
Location Newburgh, NY
CreditAttribution: Jon Pugh commented
Title: Style option only appears for the super user account » 'administer panels styles' missing from hook_permission(), only user 1 can edit styles
FileSize
panels-style-perms.1313642.patch527 bytes

Re-rolled on latest dev release

droath’s picture

droath CreditAttribution: droath commented
Status: Needs review » Reviewed & tested by the community
FileSize
panels-style-perms.1313642-5.patch537 bytes

@careernerd, Thanks!

fatal: git diff header lacks filename information when removing 1 leading pathname components (line 5)

I got an error when applying the patch. So I re-rolled patch #4, if anyone else is having problems.

Letharion’s picture

Letharion CreditAttribution: Letharion commented
Assigned: Unassigned » merlinofchaos
BenK’s picture

BenK CreditAttribution: BenK commented

Just checking back in.... any progress on this?

Letharion’s picture

Letharion CreditAttribution: Letharion commented
Status: Reviewed & tested by the community » Needs review

@BenK You can clearly see the progress in this issue, so I don't know what you are asking for.
You are welcome to try out the patch and report back how it works for you.

EmanueleQuinto’s picture

EmanueleQuinto CreditAttribution: EmanueleQuinto commented

Patch works for me.

Letharion’s picture

Letharion CreditAttribution: Letharion commented
Status: Needs review » Reviewed & tested by the community
qylie-1’s picture

qylie-1 CreditAttribution: qylie-1 commented

Thanks! Worked like a charm!

Letharion’s picture

Letharion CreditAttribution: Letharion commented
Status: Reviewed & tested by the community » Fixed

http://drupalcode.org/project/panels.git/commit/5bdd2f069c91add0d07e9d6e...

Letharion’s picture

Letharion CreditAttribution: Letharion commented

http://drupalcode.org/project/panels.git/commit/5bdd2f069c91add0d07e9d6e...

Letharion’s picture

Letharion CreditAttribution: Letharion commented
Assigned: merlinofchaos » Unassigned

Status: Fixed » Closed (fixed)

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

steveoliver’s picture

steveoliver CreditAttribution: steveoliver commented
FileSize
panels-style-perms-1313642-15.patch374 bytes

I didn't see a 6.x variant of this issue, so here is a patch for the same issue in D6 (against 6.x-3.9).

merlinofchaos’s picture

merlinofchaos CreditAttribution: merlinofchaos commented
Status: Closed (fixed) » Needs review

Marking nr to prevent loss

pcorbett’s picture

pcorbett CreditAttribution: pcorbett commented

Works for 6.x-3.10.

merlinofchaos’s picture

merlinofchaos CreditAttribution: merlinofchaos commented
Version: 7.x-3.0-alpha3 » 6.x-3.x-dev
Status: Needs review » Fixed

Committed to D6.

Status: Fixed » Closed (fixed)

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

mgerbault’s picture

mgerbault
Primary language French
CreditAttribution: mgerbault commented

This problem persist in Drupal 6 version.

merlinofchaos’s picture

merlinofchaos CreditAttribution: merlinofchaos commented

It was committed here: http://drupalcode.org/project/panels.git/commit/bab933ea5fd682a06eb5fe8d...

EvanDonovan’s picture

EvanDonovan CreditAttribution: EvanDonovan commented

Could a new release of Panels be made that includes this patch? People are missing out on a crucial bit of functionality because they assume the stable release has all the required permissions but it is from before this patch was committed.

  • Commit 5bdd2f0 on 7.x-3.x, 7.x-3.x-i18n, 8.x-3.x by Letharion: 
    Issue #1313642 by careernerd, droath. Adding a 'Administer panels styles...