Closed (fixed)
Project:
Forum email integration
Version:
6.x-1.9
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
15 Mar 2012 at 21:20 UTC
Updated:
9 Apr 2012 at 09:00 UTC
The code in femail_nodeapi to look up the forum term id for the node we are operating does not work correctly. It does not locate the tid and therefor the post is not sent out via email as it should be.
The old code will only ever work if the forum term id value == the forum vocabulary id, a random occurrence.
I am including a patch that looks up the forum term id correctly.
| Comment | File | Size | Author |
|---|---|---|---|
| commit-6708770-form-tid.patch | 1005 bytes | labrown |
Comments
Comment #1
danzsmith commentedI was running into this same issue and this fix took care of it so I just wanted to say thank you!
Comment #2
sdrycroft commentedThanks for this labrown. The old code works in most cases, but clearly doesn't work in some situations (where the taxonomy array isn't an array of arrays keyed by the vocabulary ID). I've added this patch to the 6.x branch, and will do the same for the 7.x branch. Once that is done, I'll add a release.