Hi,

I installed the new release, and enabled the Extended primary links block. But my breadcrumbs are not showing, and I only noticed this because I'm using the [menupath] token for pathauto, and saw that my path reset to node/xx after edit. Then I noticed the breadcrumb isn't showing at all.

Have you experienced this? I don't see an existing issue, but wondering if there's anything I can do about it?

Cheers!
Scott

CommentFileSizeAuthor
#18 nodehierarchy-6.x-2.1-breadcrumbs-fix.patch1016 bytesestepix
#12 nodehierarchy-6.x-1.3-Stop_Breaking_Breadcrumbs.patch476 bytessgtpep

Comments

bengtan’s picture

bengtan commented

Hi,

I think you're running into a Drupal 6 behaviour which is addressed by menu_breadcrumb. Have a look at that module, give it a try and see if it solves your problem.

bengtan’s picture

bengtan commented
Status: Active » Fixed

Assumed to be fixed since I haven't heard anything further.

scottrigby’s picture

scottrigby
he/him
Primary language English
Location Brooklyn, NY
commented
Title: Breadcrumbs don't seem to be working? » Menu Breadcrumbs doesn't work when Node Hierarchy is enabled
Project: Submenu Tree » Node Hierarchy
Version: 6.x-1.3 » 6.x-1.0
Component: Miscellaneous » User interface
Status: Fixed » Closed (fixed)

Hello...

Menu Breadcrumbs doesn't work when Node Hierarchy is enabled.
(I'm moving this issue from Submenu Tree, because I ruled out that module's involvement).

I had a new D6.3 (later D6.4) installation, running Submenu Tree & Menu Breadcrumbs too, but the breadcrumbs didn't reflect the menu hierarchy when on those pages.

At first I thought maybe it was the Zen theme's Breadcrumb function maybe getting in the way - but even in Garland, this was still an issue.

I then tried to rule out any other module that might be conflicting.
* I removed Submenu Tree, and guess what? The issue was still there.
* I removed Menu Breadcrumbs, and (of course) that didn't fix my problem because what I wanted was non-default behavoir - making breadcrumbs reflect primary links menu hierarchy.
* Then I disabled the Node Hierarchy module, and the issue went away!

But is this an issue in Node Hierarchy, or Menu Breadcrumbs? For now moving to Node Hierarchy, though maybe the issue lies with Menu Breadcrumbs instead (if so please let me know?).

Thanks! It seems not many people have run into this problem yet, but ideally these 2 mods would play nice togehter

:) Scott

scottrigby’s picture

scottrigby
he/him
Primary language English
Location Brooklyn, NY
commented
Status: Closed (fixed) » Active

whoops - meant to set this to 'active' ;)

adraskoy’s picture

adraskoy commented
Version: 6.x-1.0 » 6.x-1.2

I just tried node hierarchy (6.x-1.2) for the first time, on a vanilla install of drupal 6.9, and ran into the same problem. I set up some content under the primary links, and set that as the default menu for node hierarchy. It is all working well, except that all the breadcrumbs only have "Home". I added menu_breadcrumb and turned on everything in its configuration. No change in breadcrumb behaviour for nodes created via node hierarchy before or after that change.

I also tried turning off pathauto and menu_block, and I tried setting the parent menu to Navigation rather than primary links. Neither worked. No matter what I do, I can't get breadcrumbs under node hierarchy.

Extra modules enabled:

menu_block menu_breadcrumb nodehierarchy pathauto token

adraskoy’s picture

adraskoy commented

On further examination, I see that it is creating a breadcrumb, but incorrectly. The breadcrumb does not include the current level, only the parents. This is true even if menu_breadcrumb is enabled and "Append page title to breadcrumb" is selected.

If I turn off the node_hierachy module and look at the nodes that it had created, then the breadcrumbs look fine. So the problem would seem to be in this module.

ronan’s picture

ronan commented

Node hierarchy manipulates the breadcrumbs so that they reflect the current hierarchy (in case it is different from the menu structure). I don't use menu_breadcrumb, but I assume that if both modules are trying to set the breadcrumb, somebody's going to lose. Do you guys think an appropriate solution would be for me to leave the breadcrumbs as is if menu_breadcrumb is installed?

Thanks
Ronan

brisath’s picture

brisath commented

Subscribing

asb’s picture

asb commented

Subscribing

joln’s picture

joln commented

Subscribing

bergie3000’s picture

bergie3000 commented

subscribing

sgtpep’s picture

sgtpep commented
StatusFileSize
new nodehierarchy-6.x-1.3-Stop_Breaking_Breadcrumbs.patch476 bytes

Suggesting patch.

djac’s picture

djac commented
Status: Active » Needs review

Patch #12 fixes the issue for me.

Anyone else care to review/test?

djac’s picture

djac commented
Status: Needs review » Needs work

I spoke too soon. The patch breaks expanding the 'Node Hierarchy' section on the node/edit page.

bkosborne’s picture

bkosborne
Primary language English
Location New Jersey, USA
commented

Subscribing

radu.cocian’s picture

radu.cocian commented
Version: 6.x-1.2 » 6.x-2.1
Component: User interface » Drupal/PHP Code

I had the same problem. I like node hierarchy but I really need the breadcrumbs.

What I did to fix the problem:
- installed Menu Breadcrumb 6.x.1.3
- commented line 1373 from nodehierarchy.module ( drupal_set_breadcrumb($breadcrumb); )

jordotech’s picture

jordotech commented

This fixed it for me as well.

estepix’s picture

estepix commented
Issue summary: View changes
StatusFileSize
new nodehierarchy-6.x-2.1-breadcrumbs-fix.patch1016 bytes

Patch #12 does not work properly since it is looking for two variables which do not exist: nh_parent_node-type and nh_child_node-type

if (!variable_get('nh_parent_' . $node->type, 0) && !variable_get('nh_child_' . $node->type, 0)) return;

Instead we should be checking the following variable: nh_allowchild_node-type:

variable_get('nh_allowchild_' . $node->type, 0);

The attached patch works fine for me.