- Issue created by @mvogel
- Status changed to Needs review
about 2 years ago 7:25am 13 October 2023 - Status changed to Fixed
about 2 years ago 7:56am 13 October 2023 Automatically closed - issue fixed for 2 weeks with no activity.
Hi,
Context Breadcrumb has nine available rows to create a breadcrumb. The ContextBreadcrumbBuilder will loop through all nine rows, regardless of whether they are empty. With the change in https://git.drupalcode.org/project/context_breadcrumb/-/commit/fb1de0ac0... we changed the check from mb_strlen() to isset(). Because the title is set for every row but empty (see screenshot), the check will pass, and the statement Url::fromUserInput($contextBreadcrumb['url']) will throw an error The user-entered string '' must begin with a '/', '?', or '#'. because the URL is an empty string.
Change check from isset() to !empty()
Fixed
2.0
Code
Automatically closed - issue fixed for 2 weeks with no activity.