Closed (fixed)
Project:
Menu Position
Version:
7.x-1.x-dev
Component:
Code
Priority:
Major
Category:
Task
Assigned:
Reporter:
Created:
23 Nov 2010 at 09:08 UTC
Updated:
25 Oct 2011 at 07:30 UTC
Jump to comment: Most recent file
I'm trying to get this patch to land before Drupal 7.0-RC1: #520106: Allow setting the active menu trail for dynamically-generated menu paths.
Basically, that patch will allow this module to set the active trail for any menu tree without having to do the brittle, ungodly cache pre-setting that it does now.
When that patch lands, we'll need to replace the cache pre-setting code with the new menu_tree_set_path() function.
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | 979464-2-menu-tree-set-path.patch | 6.11 KB | johnalbin |
| #1 | 979464-1-menu-tree-set-path.patch | 5.69 KB | johnalbin |
Comments
Comment #1
johnalbin#520106: Allow setting the active menu trail for dynamically-generated menu paths. has landed, but won't be in an official release until Drupal 7.9 comes out at the end of October.
I've completed the work, but I can't commit it to the 7.x-1.x branch because Drupal 7.9 is not out yet. I'll have to release a 7.x-1.0-beta2 version without this patch first. And then I can commit it and release a 7.x-1.0 to coincide with Drupal 7.9.
Here's the patch. The amount of minuses in this patch makes me sooooo happy.
Comment #2
johnalbinAdded .info file dependency on Drupal 7.9-dev or later.
Comment #3
johnalbinOldest issue in the queue FIXED!
HUZZAH!
Comment #4
hyperglide commentedGreat! -- Thanks!
So the patch is committed or pending D7.9?
Comment #5
johnalbinIts committed, so you'll find it in 7.x-1.x-dev (and it will only work with Drupal 7.9-dev). I won't release a stable release with this code until after Drupal 7.9 comes out.
Comment #6
hyperglide commented@johnAlbin -- thanks for the reply!
Looking forward to D7.9!