I looked at your module and while I like the theory behind it, I really don't want to implement it unless I know it's going to be used in Drupal 6. Any plans for doing this?

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

edhel’s picture

Try this. It was upgraded by one of module's users... and it's not tested by me yet.

lonehorseend’s picture

I will once I reach Druapl 6. Right now, I'm waiting for other modules to be completed as well. I was just so exicted that I had to make sure that this module was included. And my excitement was well-founded, I tried it last night on a site that I'm working on and I absolutely love it. It solves a major issue where I don't have the nodes in the menu system, yet still want the breadcrumbs to reflect the node's terms.

Thank you for this wonderful module.

encho’s picture

subscribing

josuealcalde’s picture

subscribing

TuWebO’s picture

Hi,
It is the first time that I use this module "node_breadcrumb-6.x-1.x-dev.zip", and it looks good for me. I haven´t done too much testing, but installing it and set some "node breadcrumb" options works on Drupal 6.2, Content Construction Kit 6.x-2.0-beta, Views 6.x-2.0-beta3 (I don´t know if it is something to do with views and cck but just in case).
At least I looks good for testing and maybe somebody with more skills can try it and report another opinion.
Thank you.

demeester_roel’s picture

The module in #1 seems to work well except.
Even if the user has the correct permission to edit a node that matches any of the rules, The 'view'|'edit' tab menu's are no longer displayed.
I think this might have to do with the way the new menu system works with menu patterns like /node/%node/edit etc.
Since the node_breadcrumb will use 'menu_set_active_item' to make another menu-item active, this new url no longer matches (internally) to /node/%node/edit.

Anyone has second thoughts about this?

demeester_roel’s picture

FileSize
36.45 KB

At line 35 i've inserted the following ..
As a result this module also works for submenu's generated by the taxonomy menu module
To get it working:
Make a rule for the content-type that is also configured for display by taxonomy.
Select the top-level of the menu generated by taxonomy (typically the one which has the same name as your vocabulary).

Browser through the taxomenu and select a node.. The breadcrumb will match the correct path.

        if (module_exists('taxonomy_menu')) {
          $top_menu = menu_link_load($rule->mid);
          //the given menu is the top parent of a taxonomy_menu
          //it's submenus are generated dynamically by taxonomy_menu
          //try generate a new FULL link_path
          $first_item = variable_get('taxonomy_menu_display_page', 'category');
          $taxonomy=$node->taxonomy;
          $vid_link=str_replace($first_item.'/','',$top_menu['link_path']);
          foreach($taxonomy as $term) {
            if ($term->vid == $vid_link) {
              $parents = taxonomy_get_parents($term->tid);
              foreach ($parents as $parent) {
                $depth[] = $parent->tid;
              }
              $path = $first_item .'/'. $vid_link .'/' .implode('/', $depth) .'/'. $term->tid;
              if ($item = db_fetch_array(db_query("SELECT ml.mlid FROM {menu_links} ml LEFT JOIN {menu_router} m ON m.path = ml.router_path WHERE ml.link_path = '%s'", $path))) {
                //this submenu is the correct one to highlight
                $rule->mid = $item[mlid];
              }
            }
          }
        }
andypost’s picture

up

Mattias-J’s picture

subscribing

roitsch’s picture

i had a problem with node bradcrumb (6.x) and displaying the right local tasks (tabs). i solved it by forcing the local tasks to be loaded ("menu_local_tasks") before the new active menu-item is set.

while ($mid && ($item = menu_link_load($mid))) {
    if ($item['link_path']) {
      if ($i == 0) {
         menu_local_tasks(0);
         menu_set_active_item($item['link_path']);
      }

sorry for my hobby-english... ;-(

jvinci’s picture

subscribing

edhel’s picture

http://drupal.org/node/333917 - beta1 for 6.x

ps: it's not tested with local tasks yet

edhel’s picture

----

andypost’s picture

Version: 5.x-1.0-rc1 » 6.x-1.x-dev
Component: Miscellaneous » Code
Status: Active » Fixed

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.