Closed (fixed)
Project:
Sites
Version:
6.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
5 Apr 2010 at 04:24 UTC
Updated:
3 May 2010 at 15:40 UTC
Jump to comment: Most recent file
Some theme or module, does not use $vars['primary_links'] but load the menu directly from variables table (such as dhtml_menu). My suggestion is to use strongarm to override the menu on configuration level.
I will upload a patch soon.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | sites_strongarm.patch | 3.15 KB | nquocbao |
Comments
Comment #1
nquocbao commentedHere is the patch
Comment #2
brynbellomy commentedReviewed this -- looks really good! This has been integrated into 2.x.
A couple of minor changes. I removed the .info file dependency on Strongarm. I think it makes more sense to make Strongarm optional. If you have a theme (like Zen) that doesn't require Strongarm in order to change the primary and secondary links menus, then you shouldn't have to install an extra module that you're not going to use. My solution was simply to wrap the call to strongarm_set_conf() in sites_init() in a "function_exists()" call. We just need to make sure we document this well so users understand both paths that are open to them.
Comment #3
brynbellomy commentedHere is the commit: http://drupal.org/cvs?commit=352036
Comment #4
brynbellomy commented