Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

main links source menu is hardcoded #192

Open
soniktrooth opened this issue Aug 13, 2015 · 1 comment
Open

main links source menu is hardcoded #192

soniktrooth opened this issue Aug 13, 2015 · 1 comment

Comments

@soniktrooth
Copy link
Contributor

The following code lives in kalatheme_process_page(). It is rendering the core 'main-menu' menu which is then printed in page.tpl as the main links. This is incorrect. What should be happening is this:

  1. get site variable for main links source as set in config.
  2. pass THAT menu name to menu_tree_all_data()
  3. ???
  4. Profit
  // Get the entire main menu tree.
  $main_menu_tree = array();
  $main_menu_tree = menu_tree_all_data('main-menu', NULL, 2);
  // Add the rendered output to the $main_menu_expanded variable.
  $variables['main_menu_expanded'] = menu_tree_output($main_menu_tree);
@soniktrooth soniktrooth added this to the 7.x-3.2 milestone Aug 13, 2015
@pirog
Copy link
Contributor

pirog commented Aug 19, 2015

+1 this def seems much better

@soniktrooth soniktrooth self-assigned this Aug 27, 2015
soniktrooth pushed a commit that referenced this issue Aug 27, 2015
@andrewmallis andrewmallis assigned RobLoach and unassigned soniktrooth Sep 2, 2015
soniktrooth pushed a commit that referenced this issue Apr 18, 2016
soniktrooth added a commit that referenced this issue Apr 18, 2016
…nks-source

#192: Switch to pulling main menu from what was set in config instead…
@RobLoach RobLoach removed their assignment Oct 9, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants