Download & Extend
Taxonomy Node » Issues

Review or remove the possibly redundant vid field in the taxonomynode table

Posted by DamienMcKenna on May 14, 2010 at 2:03pm
Project:Taxonomy Node
Version:6.x-1.x-dev
Component:Code
Category:task
Priority:normal
Assigned:Unassigned
Status:postponed

Issue Summary

In the taxonomynode table there's a field for the vocabulary ID, vid. Given this can be obtained from the tid and that no term can be in more than one vocabulary, is there really a need for this field? It seems redundant.

Login or register to post comments

Comments

#1

Posted by hanoii on May 14, 2010 at 6:39pm
Title:Why is the vid field needed in the taxonomynode table?» Review or remove the possibly redundant vid field in the taxonomynode table
Category:support request» task
Status:active» postponed

I've just gave a very quick review to the code and I think you might be right.

Although redundant, not sure if it's worth, at the moment, working on removing that, the module seems pretty stable so far for what it does, so when I schedule more work on this module, I might take a look on this.

Feel free to provide a patch if you feel like it.

nobody click here