Hi,

nice module - have wondered for some time why the admin permissions were so coarse - was going to write something similar for my next project, but you've saved me the trouble.

Only problem I see so far is that disabling the module doesn't clean up the menu_router table after it's disabled - producing errors when the config_perms callback is still seen. Although this is fixed by running update.php, it would be more friendly to have the module clean up after itself - I've written a patch to do this (attached).

I looked at just using menu_rebuild - but this in turn calls the menu_alter hook - so the config_perm entries get added back in again!

Cheers

Mike R

CommentFileSizeAuthor
install.patch846 bytesfat-eddie
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mrthumpz’s picture

Status: Needs review » Fixed

Thanks for the patch! committed.

Status: Fixed » Closed (fixed)

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