Geo Taxonomy saves the status - enabled or not - of taxonomies with a variable called geotaxonomy_(vid). This is a problem for exports, in which vids cannot be relied upon.

Comments

secoif’s picture

Noob here, what's the alternative method?

tmcw’s picture

Using the same strategy as the features module - this bit of code, one can make machine names for vocabularies which are less prone to clashes than numeric ids.

Will White’s picture

Status: Active » Needs review
StatusFileSize
new13.17 KB

Here's a patch that allows vocabularies to be referenced by module name in addition to vid.

Will White’s picture

StatusFileSize
new4.91 KB

Last patch file was really quite wrong. Here's the correct one.

Will White’s picture

StatusFileSize
new13.54 KB

New version that updates more code to be compatible.

Will White’s picture

StatusFileSize
new4.68 KB

Minor patch adjustment.

alex_b’s picture

Status: Needs review » Needs work

_geotaxnomy_vocabulary_enabled()

taxnomy.

Needs an upgrade path.

Will White’s picture

Status: Needs work » Needs review

The storage method upgrade path was committed as a part of this: http://drupal.org/node/869004

Are you seeing a need for an additional update hook that migrates settings to use vocab machine name? I'm not sure how to approach this because we can be sure that all vocabs on the site will have a machine name.

David Goode’s picture

StatusFileSize
new4.99 KB

Fix to support new FeedsTermProcessor spec of having non-numeric vid config. Uses callback function instead. Also fixed typo in function name.

David Goode’s picture

StatusFileSize
new4.62 KB

Removed part of another patch from this one; don't use post #9, use this instead.

David Goode’s picture

StatusFileSize
new4.62 KB

Fix to previous handmade patch; bear with me :-(

David Goode’s picture

Category: bug » feature

Note; these patches require the patches to feeds located here: http://drupal.org/node/889196. Also, this doesn't seem like a bug report to me...

alex_b’s picture

Issue tags: +6.x-2.0-beta3

To be released with beta3

alex_b’s picture

Status: Needs review » Reviewed & tested by the community
StatusFileSize
new5.16 KB

Added static caching to _geotaxonomy_vocabulary_enabled().

alex_b’s picture

Title: Geo Taxonomy uses numeric vids in variables » Use vocabulary.module field as unique identifier
Status: Reviewed & tested by the community » Fixed

This is committed now, thank you everybody.

http://drupal.org/cvs?commit=417698

Status: Fixed » Closed (fixed)
Issue tags: -6.x-2.0-beta3

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