Hello gentelmen,

I've faced with following problem, using your module.
I have Public and Private groups on my site. I use "group_access" field for this.
Problem appears when manager of group tries to invite somebody into the private group.
User could not view group into which he was invited.
I have already raised similar question #1847830: Accessing to private groups for members with "Pending status", but it was explained that hide private groups from pending members it's expected and needed behavior.

I'm not sure that it's really good way to invite people into groups not giving them no any information about group.
It should be at least name or some other limited set of group fields.

What do ou think about it?

CommentFileSizeAuthor
#6 og_invite_people-1848436-6807864.patch6.6 KBsadashiv
#6 og_invite_people.admin_.inc_.txt1.21 KBsadashiv
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

sadashiv’s picture

sadashiv CreditAttribution: sadashiv commented

I think we should respect the privacy of OG's and add a admin.inc for this module and allow admins to edit the message body which is send and add tokens for node title of node teaser, etc which will solve this problem.

Thanks,
Sadashiv.

vloviagin’s picture

vloviagin CreditAttribution: vloviagin commented

It's really helpful think to add admin.inc!
Could you please provide a patch or new build of dev. version with it?

But actually it solves original porblem only particulary.
Yes, we could add information about group to letter, but we could not show it on site.

I think it's quite reasonable feautre on site with groups - list of all invitations to group.
It's really easy to do with views, but it's denied to view provate groups for users, whose membership in private groups is pending (they are invited).
This way we could see invitations only for open groups.
It's qute strange to see invitations only into groups in which user could subscribe by himself.

Sorry for so wordy comment, but i really need your module on my site.
Thanks a lot for your help and your work!

vloviagin’s picture

vloviagin CreditAttribution: vloviagin commented

Sorry, could is anything new related to this request?
Could someone help with this problem?

sadashiv’s picture

sadashiv CreditAttribution: sadashiv commented

I am waiting for previous patch submitted to be added in the module and for alesr to update what he thinks about this feature.

alesr’s picture

alesr CreditAttribution: alesr commented

A patch is welcome...

sadashiv’s picture

sadashiv CreditAttribution: sadashiv commented
Status: Active » Needs review
FileSize
og_invite_people.admin_.inc_.txt1.21 KB
og_invite_people-1848436-6807864.patch6.6 KB

I am attaching the patch which includes the patch at http://drupal.org/node/1841648#comment-6743716 and plus token support with admin.inc file

Please download a fresh copy of module before applying the patch on it.
Download the og_invite_people.admin_.inc_.txt file in attachment and rename it to og_invite_people.admin.inc and copy it to the modules folder and flush cache so that menus can get rebuild.

Hth,
Sadashiv.

drasgardian’s picture

drasgardian CreditAttribution: drasgardian commented

I have added a patch here http://drupal.org/node/1841648#comment-6944120

This exposes details of the group to rules, which can be used to send invitation emails including the group name etc.

It would still be good for users to be able to view the invitation details from within the site rather than just in emails (as mentioned in #2), but a simple solution there might just be to create a view of pending group memberships and set 'Disable SQL rewriting' = true on the view so that it bypasses the normal node_access checks.

Another solution could be to use the privatemsg module and alter the rules to send private messages instead of emails. Then the invitations are visible within the site in the privatemsg inbox.

stefank’s picture

stefank CreditAttribution: stefank commented

Actually they can. http://drupal.org/node/720252 You can create a view with pending invitations and the trick in link.
I tried it and its working even for the 7 version. Works with the patch above.