Cached breadcrumb not updating on node change

Hi Jhuhta,

Can you look at the addCacheTags call from easy_breadcrumb and try that code in here? I'd do it, but replicating your set up is a few steps, and I've got a super busy week ahead.

Hi!

Patch from #3 works for me for updating breadcrumb after editing node's title.

Thanks!

So, if I'm following this thread correctly, this will fail for entities, users and taxonomy terms as well? And it will fail in both Breadcrumb modules for D8?

Noting that I replicated this issue and that the patch in #3 fixed it for me, for node page breadcrumbs.

However, I imagine Greg is right in #5, that the problem would affect any entity whose label sets the page title... I have never fully understood where the entity view routes were provided, so I looked it up, and it seems this class does so - the `canonical` route:
https://api.drupal.org/api/drupal/core%21lib%21Drupal%21Core%21Entity%21...
So I think one way to be truly dynamic would be to check if the current route matches an `entity.[entity_type_id].canonical` route, and then add that entity's cache tags...?

        // Expire the cache per url.
        $breadcrumb->addCacheContexts(['url.path']);

        // Expire cache context for config changes.
        $breadcrumb->addCacheableDependency($this->config);

        return $breadcrumb->setLinks($links);
      }
    }

    // Handle views path expiration cache expiration.
    $parameters = $route_match->getParameters();
    foreach ($parameters as $key => $parameter) {
      if ($key === 'view_id') {
        $breadcrumb->addCacheTags(['config:views.view.' . $parameter]);
      }

      if ($parameter instanceof CacheableDependencyInterface) {
        $breadcrumb->addCacheableDependency($parameter);
      }
    }

    // Expire cache by languages and config changes.
    $breadcrumb->addCacheContexts(['url.path', 'languages']);
    $breadcrumb->addCacheableDependency($this->config);

Attaching a patch based off easy_breadcrumb implementation and comment #7.