install_taxonomy_get_tid() needs to search by vid
DamienMcKenna - March 11, 2009 - 03:17
| Project: | Install Profile API |
| Version: | 6.x-2.0 |
| Component: | CRUD functions and includes |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed |
Description
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.
#1
PLEASE IGNORE.
I mixed up two issues, will post this to a separate issue.
#2
Please ignore the previous attachment, it was for the wrong issue.
#3
The attached patch limits install_taxonomy_get_tid() to the vid if passed in through the optional argument.
#4
Patch actually attached this time.
#5
Seems sensible to me, committing with a small change (keeping vid numeric, as done in taxonomy.module with optional arguments).
#6
#7
Automatically closed -- issue fixed for 2 weeks with no activity.