On the add forum topic page, if the gids argument is supplied as 'gids[]', all forum containers are visible in the 'Forums' drop down box, regardless of whether the logged in user actually has access to see them.

The attached patch checks to see if the user is a member of the supplied group. So if 'gids[]' is the argument, no query is returned for the list of forums, since the empty string is not a valid group id.

See the patch. This is the first one I've made so let me know if I've not explained it very well here!

CommentFileSizeAuthor
og_forum-term-access.patch1.54 KBjames.williams
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Steven Jones’s picture

Steven Jones CreditAttribution: Steven Jones commented

subscribing.

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented

Great catch, ill get that committed today.

Best,
Paul Booker

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented
Status: Needs review » Closed (fixed)
Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented
Assigned: Unassigned »
Status: Closed (fixed) » Fixed

Oops!

Status: Fixed » Closed (fixed)

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