Could you add another permission so we can allow only certain roles to administer the premium messages. According to the code you need 'administer permissions' to access admin/settings/nopremium, not sure what the thinking was to use that permission to administer the premium messages. Thanks
Comments
Comment #1
anrikun commentedWell, administering Node Option Premium is basically setting permissions, which requires to have the "Administer permissions" permission. That's why I used the same permission for the settings page.
Doesn't it make sense?
I'm OK to add a new permission, but I'd like other users' opinions.
Comment #2
drupalgideonI think it makes sense to allow an editor to set whether the content should be premium content or not in the same way they can set an article to be sticky, published to front page etc. but I wouldn’t want to give an editor access to all the permissions on the site.
Comment #3
drupalgideon...and I meant that it would be handy if that editor could also set the message for the premium content rather than having to get a site admin to change it.
Thanks
Comment #4
anrikun commented@SkidNCrashwell: so you're OK with the way Node Option Premium works now, right?
Comment #5
drupalgideonI think @thetoast is right and it needs another permission to allow an editor to change the messages in the same way that they can set if the node is premium or not.
Comment #6
anrikun commentedChanging the messages is an administrative task.
So if I add a new permission, it will be called "administer Node Option Premium".
Comment #7
Taxoman commented+1 for an extra permission. Does this go into 7.x first?
Comment #8
mandus.cz commented+1