Use vocabulary machine name in variable 'notifications_tags_vocabularies' for better features support [#1870148]

I noticed that after i exported the variable 'notifications_tags_vocabularies' into features it's like this in code:

$strongarm = new stdClass();
  $strongarm->disabled = FALSE; /* Edit this to true to make a default strongarm disabled initially */
  $strongarm->api_version = 1;
  $strongarm->name = 'notifications_tags_vocabularies';
  $strongarm->value = array(
    2 => '2',
    3 => 0,
    4 => 0,
    5 => 0,
    6 => 0,
    7 => 0,
    1 => 0,
  );
  $export['notifications_tags_vocabularies'] = $strongarm;

The vocabulary's vid is used rather than the machine name. I think it would be better to use the vocabulary machine name. As the vid will change after the site reinstalled.

Comment	File	Size	Author
#2	notifications_tags-use_vocabulary_machine_name-1870148-2.patch	2.46 KB	kurtzhong
#1	notifications_tags-use_vocabulary_machine_name-1870148-1.patch	2.38 KB	kurtzhong

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

kurtzhong CreditAttribution: kurtzhong commented 19 December 2012 at 11:04

Status:

Active

» Needs review

File	Size
notifications_tags-use_vocabulary_machine_name-1870148-1.patch	2.38 KB

A patch, now the exported variable looks like this:

$strongarm = new stdClass();
  $strongarm->disabled = FALSE; /* Edit this to true to make a default strongarm disabled initially */
  $strongarm->api_version = 1;
  $strongarm->name = 'notifications_tags_vocabularies';
  $strongarm->value = array(
    'taxonomy_categories' => 'taxonomy_categories',
    'compatibilities' => 0,
    'question_type' => 0,
    'release_type' => 0,
    'themes' => 0,
    'tool' => 0,
    'userpoints' => 0,
  );
  $export['notifications_tags_vocabularies'] = $strongarm;