I just noticed that breadcrumbs for comment reply links "comment/reply/1302" are only one level deep
(e.g.
Home » Forum Topic Title
)

rather than including the forum breadcrumb path like the forum topic display does (node/1302)

CommentFileSizeAuthor
#1 40989.patch2.46 KBWesley Tanaka

Comments

Wesley Tanaka’s picture

Wesley Tanaka commented
StatusFileSize
new 40989.patch2.46 KB
Wesley Tanaka’s picture

Wesley Tanaka commented
Status: Active » Needs review
dries’s picture

dries commented
Status: Needs review » Needs work

We won't put forum.module specific code into the comment.module.

Jaza’s picture

Jaza commented
Version: x.y.z » 6.x-dev
Category: bug » feature

-1 from me. Using menu_set_location() is evil IMHO, as it interferes with user-defined menu items.

I also don't think that this is a bug as such, since technically the 'comment' page is not the child of its associated 'node' page in the Drupal menu hierarchy, so it's expected behaviour for it to not appear as such in the breadcrumbs. Changing the menu structure from 'comment/reply/x' to 'node/x/reply' would be a better way to fix this.

Moving to 6.x-dev queue.

pancho’s picture

pancho
Location UTC+2 🇪🇺 EU
commented
Category: feature » bug

This inconsistancy still needs to be fixed.

dpearcefl’s picture

dpearcefl commented

Is this still a need in current D6?

Status: Needs work » Closed (outdated)

Automatically closed because Drupal 6 is no longer supported. If the issue verifiably applies to later versions, please reopen with details and update the version.