This patch adds a permission manage own openid sites that is needed to access a user's own OpenID Sites tab.

CommentFileSizeAuthor
#3 867390-3.patch1.98 KBroderik
#2 867390.patch1.66 KBroderik
#1 867390_user_perm.patch989 bytesyhahn
openid_provider_user_perm.patch1.27 KByhahn

Comments

yhahn’s picture

yhahn commented
StatusFileSize
new 867390_user_perm.patch989 bytes

Rerolled without the ending cruft.

roderik’s picture

roderik
Primary language Dutch
Location Amsterdam,NL / Budapest,HU
commented
StatusFileSize
new 867390.patch1.66 KB

Great, I needed that. (To hide that tab from regular users who wouldn't understand it anyway, on some site I'm tinkering with.)

But "disallowing people to administer their own OpenID sites" got me thinking... Do I want those users to be able to use the site as an OpenID provider, in the first place? ...no, I don't, in this case.

Maybe we need another extra permission?

(And if we do, we(I)'ll want to add code to an .install file, which sets that permission to TRUE by default for registered users...)

roderik’s picture

roderik
Primary language Dutch
Location Amsterdam,NL / Budapest,HU
commented
StatusFileSize
new 867390-3.patch1.98 KB

Added necessary modification.
(If user does not have access to "use openid provider", don't show them their identity on their account page)

walkah’s picture

walkah commented
Status: Needs review » Fixed

committed yhan's patch from #1 . I'm not sure how common it is to turn on openid_provider and not want users to use it...

Status: Fixed » Closed (fixed)

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