Needs review
Project:
OG Access Roles
Version:
7.x-2.x-dev
Component:
Code
Priority:
Major
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
11 Oct 2016 at 11:51 UTC
Updated:
13 Oct 2016 at 12:16 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
mxr576I've made some coding standards and logic fixes.
Comment #3
mxr576Lemme fix myself :)
https://www.drupal.org/node/2816547
Comment #4
mxr576Another issue which needs to be fixed before the stable release. I haven't created a new issue for this with separate patch, because my modifications uses constants which renamed in this issue.
Background story
I have a group, but I haven't assigned a "Group Visibility Roles" field to it. However, I have added a "Group Content Visibility Roles" field to one of its group content. (The module allows this, I do not see any reasons why I should not have done that, besides this is an important point one of our feature requests.)
Then I have created a group with private visibility and after that I've added a group content, where I've selected "Groups Content Visility: Use groups default" and then I've selected a custom role in "Group Content Visibility Roles".
Problem
Because my group does not have "Group Visibility Roles" field, the current implementation of og_access_roles_node_access_records() returns empty roles when I'm creating a group content with the settings that I've mentioned above therefore users with the custom role that I've selected do not get access to the group content.