The 2026 Drupal Business Survey is open. Your response will help shape the future of the Drupal business ecosystem.Incorrect use of vid instead of nid in term_node table
The INSERT INTO term_node query assign $node->vid to the nid field of term_node table. It should assign $node->nid.
It works until a revision is created (when $nid and $vid are no more synchronised with each other).
Patch is supplied.
Have I got it right ?
| Comment | File | Size | Author |
|---|---|---|---|
| patch_0_0.txt | 488 bytes | Anonymous (not verified) |
Comments
Comment #1
Anonymous (not verified) commentedIt is apparently the same for the 5.x-1.x-dev version of the module.
Comment #2
bjaspan commentedYou appear to be correct. Primary Term assigns the PT to a vid but taxonomy.module (in D5) assigns terms to nids; this is probably what led to the confusion. Anyway, I fixed the module not to muck directly with the term_node table at all, using nodeapi('submit') instead, which also fixed this bug.
I am not maintaining the 4.7 branch of PT any more; if someone comes along that wants to back-port fixes, I'll add them to the CVS access list.
Comment #3
(not verified) commented