When a user clicks on an "Add topic" link in a group forum the group context is not set. Since the url contains the taxonomy term id; ie: node/add/forum/123 we should be able to set the group context on that page.

CommentFileSizeAuthor
#1 og_forum-set-group-context-add-topic-2283229-1.patch1.15 KBjojonaloha

Comments

jojonaloha’s picture

jojonaloha commented
Status: Active » Needs review
StatusFileSize
new og_forum-set-group-context-add-topic-2283229-1.patch1.15 KB

Attached is a patch that updates og_forum_forum_group_handler() to set the group context on node/add/forum/[tid]

vegantriathlete’s picture

vegantriathlete
he/him/his
Primary language English
Location Lakewood, CO
commented

This will likely end up applying with a hunk #?? successful, offset . . . since the code will have changed as a result of applying #1844104: non-group-member can post in private forum

vegantriathlete’s picture

vegantriathlete
he/him/his
Primary language English
Location Lakewood, CO
commented
Status: Needs review » Closed (cannot reproduce)

It appears that the Forum module itself is already taking care of this. It sure seems to be working fine to me without this patch applied :)

vegantriathlete’s picture

vegantriathlete
he/him/his
Primary language English
Location Lakewood, CO
commented

I was confused. The Forum module sets the forum based on context. It is not taking care of the Group context. Perhaps patching with[#2266851] fixed this issue? I haven't dug into the details to figure it out, nor had I done a before / after test. Nonetheless, it appears things are working fine without this patch.