hey
looks like i'm getting two variables stored by content type:
- [content_type]_menu_settings
- menu_[content_type]

example:
- book_menu_settings

a:6:{
s:18:"menu-footer-menu-1";s:18:"menu-footer-menu-1";
s:18:"menu-footer-menu-2";s:18:"menu-footer-menu-2";
s:18:"menu-footer-menu-3";s:18:"menu-footer-menu-3";
s:10:"navigation";s:10:"navigation";
s:13:"primary-links";s:13:"primary-links";
s:15:"secondary-links";s:15:"secondary-links";
}

- menu_book

a:6:{
i:0;s:18:"menu-footer-menu-1";
i:1;s:18:"menu-footer-menu-2";
i:2;s:18:"menu-footer-menu-3";
i:3;s:10:"navigation";
i:4;s:13:"primary-links";
i:5;s:15:"secondary-links";
}

this normal? is a second variable really needed?

Comments

lpalgarvio’s picture

check on this,
http://drupal.org/coding-standards

Persistent Variables

Persistent variables (variables/settings defined using Drupal's variable_get()/variable_set() functions) should be named using all lowercase letters, and words should be separated with an underscore. They should use the grouping/module name as a prefix, to avoid name collisions between modules.

vasike’s picture

Version: 6.x-1.1 » 6.x-2.x-dev

moved to the new branch

vasike’s picture

Issue summary: View changes
Status: Active » Closed (outdated)