Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.There is what I consider a small bug that's bad for the usability:
When a group admin opens his og/users list, the tab "Configure Roles" isn't instantly visible, only when the admin opens "Add Members" - then the tab appears.
My programmer came up with the following bugfix that works well in our tests. Add the bold line as line 162 in this context:
elseif ((!empty($item['map'][0]) && $item['map'][0] == 'og' && !empty($item['map'][2])) || $path == 'comment/reply/%') {
$node = og_user_roles_menu_get_object('node', 2);
$node = $node ? $node : node_load($item['map'][2]);
}
Hope that's all that's needed.
I did quite some testing yesterday and will probably implement the module in my live project soon, as it solves several prio 1 user demands for me. Thank you!
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | missing-tab-571306-6.patch | 687 bytes | pwolanin |
Comments
Comment #1
itsnotme commentedA related problem showed up when og_user_roles didn't "know" that certain views were in the group context (in my case, I had extended the group tabs with views, e.g. "pages" of the group). So a content type permission that was connected to a user role in group was missing and the user could not create that content type at that point.
The solution my programmer found was to extend line 160 with a more complex check for the group context.
(This doesn't solve the problem yet that on a similar tab for Forums (which comes with og_forum), the user role context and all group tabs are lost again, possibly because the forums paths coming with og_forum start with forums/... No fix for this one yet, as it's a lower priority.)
Comment #2
sunPlease provide a proper patch.
Comment #3
pwolanin commentedI'm observing this same bug
Comment #4
pwolanin commentedwith the latest dev code
Comment #5
pwolanin commentedIt looks like the problem is that the og/users/% path is created by Views module and hence there is no associated node_load load function
Comment #6
pwolanin commentedHere's a patch which takes a slightly different approach, but essentially the same fix as suggested in the initial post.
Comment #7
sunIs that "menu system maintainer" approved? :)
Comment #8
pwolanin commentedIt's a bit of a hack either way - less overhead I think to check if it's numeric then running through all the rest of the logic.
Comment #9
sunThanks for reporting, reviewing, and testing! Committed to all branches.
A new development snapshot will be available within the next 12 hours. This improvement will be available in the next official release.