Due to the fact that multiple vocabularies can have the same term name, install_taxonomy_get_tid() really needs to limit the query to vid.

CommentFileSizeAuthor
#5 install_profile_api-n398022-get_tid_vid_v2.patch920 bytesJames Andres
#4 install_profile_api-n398022-get_tid_vid.patch956 bytesdamienmckenna
#1 install_profile_api-n398022-taxonomy_tid_fix.patch987 bytesdamienmckenna

Comments

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Active » Needs review
StatusFileSize
new install_profile_api-n398022-taxonomy_tid_fix.patch987 bytes

PLEASE IGNORE.
I mixed up two issues, will post this to a separate issue.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Needs review » Active

Please ignore the previous attachment, it was for the wrong issue.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Active » Needs review

The attached patch limits install_taxonomy_get_tid() to the vid if passed in through the optional argument.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
StatusFileSize
new install_profile_api-n398022-get_tid_vid.patch956 bytes

Patch actually attached this time.

James Andres’s picture

James Andres commented
StatusFileSize
new install_profile_api-n398022-get_tid_vid_v2.patch920 bytes

Seems sensible to me, committing with a small change (keeping vid numeric, as done in taxonomy.module with optional arguments).

James Andres’s picture

James Andres commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.