I would like to see this seperated permission because obviously i might permit my users to override the options on node creation but not to change the settings under admin/settings/override_node_options.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mrfelton’s picture

at least make it use 'administer site configuration' rather than 'access administration pages'. I want to allow certain users into /admin but don't want them messing with the override_node_options settings.

mrfelton’s picture

marcushenningsen’s picture

Status: Active » Reviewed & tested by the community

I noticed this small "bug" as well. The patch is a very simple to solution (changing one small line of code) and IMHO might as well be committed. I'd like to hear what the maintainer thinks?

Bartezz’s picture

Tested and working fine with D6.14...

Cheers

joachim’s picture

Tested and works fine.
+1 for this patch -- keeps non-techy admins away from settings!

Bartezz’s picture

Priority: Minor » Critical

Changing priority so hopefully a maintainer will pick this up and commit.

Cheers

joachim’s picture

I agree this is critical and as a new comaintainer I'd love to commit it!
However, we have this problem to fix before anything can be committed on this project: #621030: Branch hell and I don't have a couple of hours free to untangle it.
If anyone can help out with that, please do!

joachim’s picture

Status: Reviewed & tested by the community » Fixed

Committed.

Thanks for the patch.

Status: Fixed » Closed (fixed)

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