Currently all the settings for forward are in one admin page. This is not so great when you have multiple developers working on a site and they need to change the HTML / text for the thank you, but you don't want them mucking around with your flood control or the email address that it gets sent from.
There are permissions for override flood control and override email address, but it doesn't look like they actually *do* anything. It seems like it would make more sense to remove those 2 permissions in favor of 2 more general permissions.. one for admin settings, one for the html portions of emails/replies.
Comment | File | Size | Author |
---|---|---|---|
#3 | forward-permissions-breakout-965564-2.patch | 4.97 KB | rfelsburg |
#1 | forward-permissions-breakout-965564-1.patch | 5.26 KB | rfelsburg |
Comments
Comment #1
rfelsburg CreditAttribution: rfelsburg commentedBroke out permissions for admin settings, versus email settings.
I also changed permission names to reflect what they actually do. Such that people can recognize that 'access forward admin pages' means allowing access to the forward module in the admin menu, versus 'forward admin settings' which actually gives permissions to change the admin settings.
Currently the forward admin menu populates two hardcoded menu options for 'Admin settings' and 'Email settings'. I'm working towards making this more dynamic like the 'Logging and Alerts' admin menu, which uses menu_get_item and a series of sql queries to populate a block.
Comment #2
rfelsburg CreditAttribution: rfelsburg commentedUpdated Assigned: unassigned
Comment #3
rfelsburg CreditAttribution: rfelsburg commentedFixed the display menu to dynamically output the sub pages of the forward module. At this point 'Admin Settings' and 'Email Settings'. Both dynamically generated off of the plid for the forward module.
This patch is a replacement. This is not a patch to the initial patch.
Comment #4
crimsondryad CreditAttribution: crimsondryad commentedThis patch works for me.
Comment #5
randompants CreditAttribution: randompants commentedI've tested it and it works for me. Nice job!
Comment #6
crimsondryad CreditAttribution: crimsondryad commentedMarking RBTC
Comment #7
john.oltman CreditAttribution: john.oltman commented