Skip to content

[CFM-861] Adjust group permissions and align groups presentation.#1520

Merged
ordavidil merged 4 commits into
developfrom
feature/CFM-861-new-pr
Nov 14, 2016
Merged

[CFM-861] Adjust group permissions and align groups presentation.#1520
ordavidil merged 4 commits into
developfrom
feature/CFM-861-new-pr

Conversation

foreach (c4m_features_homepage_get_user_groups_grouped_by_membership($user) as $group_list) {
foreach ($group_list as $group) {
// Deleted groups should not appear on the top bar list.
if (c4m_og_get_group_status($group) == 'deleted') {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@anvmn
is this taking care somewhere else ?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

c4m_features_homepage_get_user_groups_grouped_by_membership() uses
c4m_features_homepage_get_user_groups() that filters group by 'view' access:

// Removes groups where the logged in user has no access. $groups = array_filter($groups, 'c4m_features_homepage_node_access_view_filter');

@anvmn anvmn changed the title WIP: feature/CFM-861-adjust-permissions [CFM-861] Adjust group permissions and align groups presentation. Nov 13, 2016
@anvmn anvmn changed the title [CFM-861] Adjust group permissions and align groups presentation. WIP: [CFM-861] Adjust group permissions and align groups presentation. Nov 13, 2016
@ordavidil ordavidil changed the title WIP: [CFM-861] Adjust group permissions and align groups presentation. [CFM-861] Adjust group permissions and align groups presentation. Nov 14, 2016
@ordavidil ordavidil merged commit 449ecc4 into develop Nov 14, 2016
@soniCaH soniCaH deleted the feature/CFM-861-new-pr branch November 22, 2016 14:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants