Bug with multiple dictionaries

olivo.marco - October 21, 2008 - 18:07
Project:Yahoo Terms
Version:5.x-1.x-dev
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:needs review
Description

I have found a bug with multiple dictionaries: Yahoo terms overwrites what is already stored in the node (I found this by using Leech + Yahoo Terms)
The problem seems to be in the lines:

    case 'insert':
      $vid = _yahoo_terms_which_vocab($node->type);

which, when modified as:

    case 'insert':
      $node = node_load($node->nid);
      $vid = _yahoo_terms_which_vocab($node->type);

seem to solve the problem. The bug is probably in the fact that the node is not "fully loaded" when yahoo_terms_nodeapi() is invoked.

For me, it worked (TM)

Bye,
Marco

#1

Shiny - November 25, 2008 - 19:49

I'm trying to replicate this to work out if the patch is needed.
When you say "dictionary" do you mean "vocabulary"?
at node insert, are you seeing tags you entered manually disappearing?

 
 

Drupal is a registered trademark of Dries Buytaert.