$this->siteConfig->get('page.front'); fails when front page is set to /

Hrm, try clearing the cache after switching themes. Easy Breadcrumb has code to expire the cache when things change, but Bootstrap does not.

Have you set a value for the front page crumb to Home or some other string? If you leave that field blank, that's the only thing I can think of. Here's the code in question

   $front = $this->siteConfig->get('page.front');

    // Give the option to keep the breadcrumb on the front page.
    $keep_front = !empty($this->config->get(EasyBreadcrumbConstants::HOME_SEGMENT_TITLE))
                  && $this->config->get(EasyBreadcrumbConstants::HOME_SEGMENT_KEEP);
    $exclude[$front] = !$keep_front;
    $exclude[''] = !$keep_front;

Did that work?

I'm out of ideas on where the home crumb could be coming from!

Are you able to replicate the issue on Simply Test me?

https://simplytest.me/

That's pretty weird.
I started to debug it locally and my first conclusion is: it only happens if the frontpage global site setting is set to simply"/"
If I change to any other path then everything is working as expected.
I recon that it's not easy breadcrumb related bug because if I set slash as value of frontpage it stores a totally different value (/node/2 in my case) so strange.
I will continue the debugging soon.

Any ideas are welcome!

Cheers,

Bálint

Sounds like a bug in core $this->siteConfig->get('page.front'); to me!

If that is indeed the cause, we can work around it could be somewhere lower in the code doesn't like the value of / for front page.

Well if I try to get the frontpage url in a different way then it's fine for me with a slash as well!

current code:
$front = $this->siteConfig->get('page.front');

my change:
$front = Url::fromRoute('<front>')->toString();

I know a static call is not the best inside this class but seems okay.

Probably a bit better idea to inject UrlGeneratorInterface to the class as a dependency.

You could have a look if you have time!

Thanks!

My configuration like the image upload below.
I want to hide the breadcrumb in the font, but it still shows in the front.
I have checked the code and find that it is caused by the front page path. drupal 8 set the real node path in 'page.front', eg: node/1.
When the code below executes, $front is not equal to the path to the front page.

if ($path && '/' . $path != $front && $path != $curr_lang) {
  $links[] = Link::createFromRoute($this->config->get(EasyBreadcrumbConstants::HOME_SEGMENT_TITLE), '<front>');
}
if ($this->config->get(EasyBreadcrumbConstants::HIDE_SINGLE_HOME_ITEM) && count($links) === 1) {
  return $breadcrumb->setLinks([]);
}

I solve this by add aliasManage and get the front page path alias for $front.

@guaneagler Yes your patch really solves the path problem BUT I found another one which is also an issue.
There is a foreach loop which iterates on the the path_crumb_rows which are set on the settings form IF it is set!
Currently this loop runs there is no replace path set and it's not fine that's the reason why the code never reaches the frontpage related part (in the foreach loop there is a return $breadcrumb->setLinks($links);)

I've attached a patch, please have look and test it yourself.

Hope that helps!

Bálint

Sorry this one should be okay let's see the tests.

This is a lot of complex code that will give folks a white screen of death on module upgrade to work around a pretty obscure bug with front page settings. Can we solve this just by updating the help text on the front field?

That’s okay from my perspective.

I've added a note to the config form @Greg Boggs I hope you thought so. :)
Patch is attached.

The message can help other users. Makes sense

Committed to the dev branch.

Thank you all for helping