When clicking on the group forum link (web_site/taxonomy/term/28) for the first time (after cache clearing) I am redirected to the /forum/28 as expected.
However, any further attempts fail without any redirecting. This fails with three different browsers so it is not a browser caching issue.
There are no errors logged. Result is blank page.
Looks like Drupal caching issue.
Clearing cache will help, again, only once.

Actually some other might be related issues:

If I clear the "Taxonomy Term Redirect" box I go to the taxonomy term page but not seeing the group forum at tall.
Not sure why the choice for this settings if it only works when ticking the box.
Another question is why keep the taxonomy/term id as the group forum target url instead of using the forum/term id as the target url and save the redirection.

CommentFileSizeAuthor
#5 Redirect_from_term_to_forum_fails-2881703-2881703.patch598 bytesbehindthepage
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

taitai created an issue. See original summary.

taitai’s picture

taitai CreditAttribution: taitai as a volunteer commented
Issue summary: View changes
arnoldbird’s picture

arnoldbird CreditAttribution: arnoldbird commented
Status: Active » Closed (cannot reproduce)

I can't reproduce the problem. I haven't encountered this issue during my work on the module. Please provide steps to reproduce, if you can, and re-open.

The checkbox in the settings is working as designed in my testing today.

Please report one issue at a time. Create a separate ticket for any support request.

Thank you

behindthepage’s picture

behindthepage CreditAttribution: behindthepage as a volunteer commented

I think it is a bug.

behindthepage’s picture

behindthepage CreditAttribution: behindthepage as a volunteer commented
FileSize
Redirect_from_term_to_forum_fails-2881703-2881703.patch598 bytes

Patch attached

behindthepage’s picture

behindthepage CreditAttribution: behindthepage as a volunteer commented
Status: Closed (cannot reproduce) » Needs review

Sorry couldn't seem to to do these three things on one post.

arnoldbird’s picture

arnoldbird CreditAttribution: arnoldbird commented
Status: Needs review » Fixed

Yes, this was a bug. Fixed in latest dev commit.

@behindthepage -- Thanks for the patch. That helped me find the problem. Though a different fix was needed...

elseif ('taxonomy' == $parts[0] && 'term' == $parts[1] && !isset($parts[3])) {

We have to check for $parts[3] so we don't redirect away from taxonomy/term/123/edit and taxonomy/term/123/delete.

behindthepage’s picture

behindthepage CreditAttribution: behindthepage as a volunteer commented

Thanks @arnoldbird your solution covered all the possibilities. Well done.

I helped fixed a bug and learnt something ;-)

arnoldbird’s picture

arnoldbird CreditAttribution: arnoldbird commented
Status: Fixed » Closed (fixed)