Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
In the function forum_form there is a call to the taxonomy_node_get_terms_by_vocabulary function. This function expects the function parameters of $nid and $vid. The function call in forum function forum_form reverses the function parameters.
Is: $forum_terms = taxonomy_node_get_terms_by_vocabulary(_forum_get_vid(), $node->nid);
should be: $forum_terms = taxonomy_node_get_terms_by_vocabulary($node->nid, _forum_get_vid());
This is at line 395
Comment | File | Size | Author |
---|---|---|---|
#2 | 20080225.forum-terms-by-voc.patch | 1.06 KB | Bart Jansens |
Comments
Comment #1
Liam McDermott CreditAttribution: Liam McDermott commentedI've checked and this is still in 5.7, don't think it's a problem in 6.x however.
Comment #2
Bart Jansens CreditAttribution: Bart Jansens commentedconfirmed, and patch attached.
Shadow copies are still buggy though, but thats a different issue.
Comment #3
Liam McDermott CreditAttribution: Liam McDermott commentedI've tested this--on the 5.x-dev branch--by creating two forums, creating a new topic, then editing it and moving the topic to the other forum with the 'Leave shadow copy' box checked. When editing the moved topic, after moving it of course, the 'Leave shadow copy' is correctly checked.
This is a tiny fix and probably won't effect anything, but why leave the code wrong? This is ready to be committed, if there's another release of the 5 series it would be good to have this in there. :)
Comment #4
drummCommitted to 5.x.
Comment #5
Anonymous (not verified) CreditAttribution: Anonymous commentedAutomatically closed -- issue fixed for two weeks with no activity.