Problem/Motivation

Viewing a node on Drupal 11.2 correctly displays the node's title as the last breadcrumb.
Update to Drupal core 11.3.0 and the node title no longer displays.

CommentFileSizeAuthor
#7 3565400-menu_breadcrumb-missing_curretn_page_title-7.patch2.15 KBludo.r

Issue fork menu_breadcrumb-3565400

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

nicrodgers created an issue. See original summary.

nicrodgers opened merge request !38

nicrodgers’s picture

nicrodgers
Primary language English
Location Monmouthshire, UK
commented
Title: Current page missing since updating to Drupal Core 11.3 » Current page node title missing since updating to Drupal Core 11.3
Status: Active » Needs review
eddylbs’s picture

eddylbs
Location Paris
commented

It's Works for me on Drupal 11.3.1 and php 8.4.10
Thanks

plektr0’s picture

plektr0 commented

It's Works for me on Drupal 11.3.1 and PHP 8.3.28

tikaszvince’s picture

tikaszvince commented
Status: Needs review » Reviewed & tested by the community

I beleive this is ready for merge

ludo.r’s picture

ludo.r
Location Brussels
commented
StatusFileSize
new 3565400-menu_breadcrumb-missing_curretn_page_title-7.patch2.15 KB

I'm having the same issue with version 2.0.1 and D10.5.

Attaching patch of MR #38 here.

rossb89’s picture

rossb89 commented

Confirmed the fix is good. This can cause a fatal php error as well if $links is empty by the time $first_url = $links[0]->getUrl(); is called in MenuBasedBreadcrumbBuilder

xurizaemon’s picture

xurizaemon
he/him
Location Ōtepoti, Aotearoa 🏝
commented
Status: Reviewed & tested by the community » Fixed

Thanks, merging.

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

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

klemendev’s picture

klemendev commented

Any news when this patch is planned to be merged in release?