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

Comments

jon pugh’s picture

jon pugh
he/him
Primary language English
Location Newburgh, NY
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
commented
StatusFileSize
new 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 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
commented
Title: Style option only appears for the super user account » 'administer panels styles' missing from hook_permission(), only user 1 can edit styles
StatusFileSize
new panels-style-perms.1313642.patch527 bytes

Re-rolled on latest dev release

droath’s picture

droath commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new 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 commented
Assigned: Unassigned » merlinofchaos
BenK’s picture

BenK commented

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

Letharion’s picture

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 commented

Patch works for me.

Letharion’s picture

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

qylie-1 commented

Thanks! Worked like a charm!

Letharion’s picture

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

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

Letharion’s picture

Letharion commented

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

Letharion’s picture

Letharion commented
Assigned: merlinofchaos » Unassigned

Status: Fixed » Closed (fixed)

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

steveoliver’s picture

steveoliver commented
StatusFileSize
new 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 commented
Status: Closed (fixed) » Needs review

Marking nr to prevent loss

pcorbett’s picture

pcorbett commented

Works for 6.x-3.10.

merlinofchaos’s picture

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
commented

This problem persist in Drupal 6 version.

merlinofchaos’s picture

merlinofchaos commented

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

EvanDonovan’s picture

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...