CiviCRM Community Forums (archive)

Hi,
this forum looks like the right place for a feature request.

Civicrm_og_sync (for Drupal) is a cool little module.

Unfortunately it lacks the ability to sync og administrators to the according civicrm acl group. It only syncs the group node author and it seems, it never removes them.

My suggestion would be to use an og permission check for the is_admin test.

Something like that in hook_og_membership_insert and hook_og_membership_update:
$is_admin = og_user_access_entity('administer group', 'node', $gid, $account);
$gid is of course the og group id.
$account ist the user object created with user_load($og_membership->etid).

So if a Drupal User has the og permission to administer members of his organic group, he would also be able to administer contacts of his group in CiviCRM.

There can be many og admins and they can always change, so just syncing the node author is probably not the best way.

Feedback is welcome.

Hi,
CiviCRM groups (2 per og group, one mailinglist and one ACL) are created and og membership is synced, that's right and it's pretty cool.

you add someone as Admin to OG and they get added to the civi ACL group

OG Admins are not synced, only the group node author is added to the ACL group.
If one changes the group node author, this user is also added to the ACL group.

My suggestion is to not rely on the node author but check if a a user has a specific permission in his og group with og_user_access_entity().

I solved that for my use case with a custom Drupal module but I would like to see a better admin-sync handling in CiviCRM.
So I decided to post a feature request here.

well i am not sure when things changed and i won't be able to check on the site we had this running on for a while.

I had a look at older civicrm source code on github.
Yes, this module changed significantly, probably because og-7.x-2.x was totally rewritten.
The check for og admin was different before and obviously synced correctly (it was working for you).

So this is not a feature request and actually misplaced in this forum because this functionality was already there in earlier releases, but broke.
I can't believe that there is no issue for that yet in https://issues.civicrm.org/
I'll see what I can do.