The extractor module currently stores and refers to the lookup vocabulary by its serial vid. This prevents you from consistently referring a particular vocabulary that has been exported using the Features module. This module should be patched to allow the option to use the non-serial "module name" to identify the vocabulary.

CommentFileSizeAuthor
#5 836878_7.patch7.87 KBalex_b
#3 875212_2.patch2.16 KBWill White
#1 875212_1.patch2.16 KBWill White

Comments

Will White’s picture

Status: Active » Needs review
StatusFileSize
new2.16 KB

Patch attached.

alex_b’s picture

Status: Needs review » Needs work

double quotes (") in SQL statements don't conform to the standard, this would throw off PGSQL.

Have you done any performance testing on the WHERE v.module = "%s" query? The number of selected items overall is high, while the number of vocabularies is usually very slow. I guess we are fine - unless the overall number of selected items requires an index no matter how low the number of items of the table we join to and filter by is (vocabulary)...

Will White’s picture

StatusFileSize
new2.16 KB

New patch fixes the double-quotes issue. Still needs a performance review.

alex_b’s picture

Issue tags: +6.x-1.0-alpha4

To be released with alpha 4.

alex_b’s picture

Status: Needs work » Reviewed & tested by the community
StatusFileSize
new7.87 KB

Addressed performance issue by resolving the vid before using it in the query.

alex_b’s picture

Status: Reviewed & tested by the community » Fixed

Committed, thank you.

Status: Fixed » Closed (fixed)
Issue tags: -6.x-1.0-alpha4

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