og_user_access_entity() can be used to check organic groups access for a user on an entity based on the group of the entity.

This finds the groups for the entity passed in and then calls og_user_access(). og_user_access_entity() does not use the $skip_alter or $ignore_admin parameters of og_user_access meaning that these can never be set to TRUE when using this function.

I propose adding the two parameters to og_user_access_entity() with the same defaults as og_user_access() which will allow this functionality to be used without changing any existing behaviour.

Patch to follow.

CommentFileSizeAuthor
#1 og-extend-access-entity-2451857-1.patch1.92 KBjustanothermark
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

justanothermark’s picture

justanothermark CreditAttribution: justanothermark commented
FileSize
og-extend-access-entity-2451857-1.patch1.92 KB

Patch to extend og_user_access_entity().

amitaibu’s picture

amitaibu
Primary language Hebrew
Location Israel
CreditAttribution: amitaibu commented
Status: Active » Needs review
brt’s picture

brt CreditAttribution: brt commented

The patch from #1 works perfectly for me

brt’s picture

brt CreditAttribution: brt commented
Status: Needs review » Reviewed & tested by the community

amitaibu’s picture

amitaibu
Primary language Hebrew
Location Israel
CreditAttribution: amitaibu commented
Status: Reviewed & tested by the community » Fixed

Merged, thanks.

Status: Fixed » Closed (fixed)

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