Issue is related to #2607920: Breadcrumb render cache not invalidated when entity label changes

If you save an entity and change the title, the breadcrumb is not updated.

The mentioned issue fixes this in core. But even with the patch applied its not working when using easy_breadcrumb.

The reason for this is a missing addCacheableDependency call in the build method. This is present in cores PathBasedBreadcrumbBuilder::build.

CommentFileSizeAuthor
#2 easy_breadcrumb-2.patch699 byteskfritsche

Comments

kfritsche created an issue. See original summary.

kfritsche’s picture

kfritsche
Primary language German
Location 🇩🇪🇪🇺
commented
Status: Active » Needs review
StatusFileSize
new easy_breadcrumb-2.patch699 bytes

Here the patch. This can be applied before the core patch is applied as other cache related issues could be fixed with that.

As soon as core patch lands this will be fix the issue with breadcrumbs not updating if you change the title of an entity.

tstoeckler’s picture

tstoeckler
he/him
Primary language German
Location Essen, Germany
commented
Status: Needs review » Reviewed & tested by the community

Yup, totally makes sense.

  • Greg Boggs committed 770562d on 8.x-1.x authored by kfritsche 
    Issue #2939498 by kfritsche, tstoeckler: Missing CacheContext from...
greg boggs’s picture

greg boggs
he/him
Primary language English
Location Portland Oregon
commented

This will be in the next release. Thanks!

greg boggs’s picture

greg boggs
he/him
Primary language English
Location Portland Oregon
commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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