I'm working on a site where OG home pages function as wikis and are editable by all group members. However, I don't want every group member to be able to e-mail all group members (using OG's E-mail tab), and the access check for the menu callback that defines the tab is node_access('update', $node).

I propose that we add a permission to go: 'email all members of a group' and check for both permissions as the access check. This patch against the dev version implements this.

CommentFileSizeAuthor
og_email_members_perm.patch946 bytesezra-g

Comments

moshe weitzman’s picture

moshe weitzman commented
Status: Active » Needs work

i'm not too comfortable with this. i would rather give some pref to each group so that they can name an alternate page as their group home page. in this case, create a wiki node and make that the home page. just thinking aloud. this is how core handles its custom home page and 403/404 error pages.

Christefano-oldaccount’s picture

Christefano-oldaccount commented

I think ezra-g's approach makes more sense. If a group home page is a wiki node, then it still has an "Email" tab. If a group home page is something else like a group node, however, and group members are simply told that the wiki node is their group home page, then all the links that point to their group (breadcrumbs, groups listing on user profile, etc.) will eventually lead them to the "Email" tab.

cindyr’s picture

cindyr commented

I would really love to have this ability to define which roles can email to the group. I have a closed community where the siteadmin is the owner of all the groups, I don't want the layman "owner" (the person who manages the content for that group) to have full admin privileges to change their group, I only want them to have the email tab.

I tried out the patch and it adds the email as a new permission in the access control table, but doesn't seem to actually work. Can someone point me in the right direction to making it function?

Thanks,
~cindyr

mightyiam’s picture

mightyiam commented

I like ezra-g's idea.

cindyr’s picture

cindyr commented

Sorry, my bad. It works just fine, it just wasn't the way I wanted it to. I had to take a break from it, get some sleep. Went back and read it through again and figured out what all you wrote meant. I want those who do NOT have access to edit the group to have the ability to email, so I changed to have it check for only the email part.

shmatty29’s picture

shmatty29 commented

I also require the email tab to have assigned control.

How and where would I load the attached patch.

Thank, Matt

shmatty29’s picture

shmatty29 commented

Hi I have added the patch and the permission appears in the access control area.

I have ticked the box in access control to allow the appropriate role (Group Manager). The group manager can see the email tab, but I am still getting access denied when clicking the tab logged in as group manager.

Any ideas

Matt

shmatty29’s picture

shmatty29 commented
Status: Needs work » Closed (fixed)

I went over it again, and it is working fine, My bad.

Many thanks,

Matt

ezra-g’s picture

ezra-g commented
Status: Closed (fixed) » Needs review

This issue is not closed.

ezra-g’s picture

ezra-g commented
Status: Needs review » Needs work

This issue is not closed.

aschiwi’s picture

aschiwi commented

Can this patch be released for 5.x-7.3? I am also looking to remove the "email" tab. Actually I don't need it at all, but I can't seem to find another way to remove it...

Thanks
Anja

Grayside’s picture

Grayside commented
Status: Needs work » Closed (won't fix)

D5 no longer supported. There are now multiple techniques for doing this in OG6/7.