Problem/Motivation

After upgrading from 8.x-1.15 to 2.0.0 all of the breadcrumbs after "Home" appear empty in the UI. The links are there but the text appears to be truncated to zero. This appears to be a regression from #3203311: The current page title is not truncated when selecting "Truncate the page's title to a maximum number." because if I enable the "Truncate the page's title to a maximum number." option and set its value the issue goes away.

Steps to reproduce

Tested with Drupal 9.2.1 --

  1. Install Easy Breadcrumb 2.0.0
  2. Navigate to config (/admin/config/user-interface/easy-breadcrumb).
  3. Ensure Set the limit of truncation has no value and Truncate the page's title to a maximum number. is unchecked.
  4. Save configuration
  5. Observe that breadcrumbs have empty text values.

Proposed resolution

Fix changes from #3203311: The current page title is not truncated when selecting "Truncate the page's title to a maximum number." to not truncate at all if Truncate the page's title to a maximum number. is not enabled.

Remaining tasks

Create patch.

User interface changes

None.

API changes

None.

Data model changes

None.

Issue fork easy_breadcrumb-3223467

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

wells created an issue. See original summary.

wells opened merge request !19

wells’s picture

wells
he/him/his
Primary language English
Location Seattle, WA
commented
Status: Active » Needs review

Looks like @nicole.harnish already identified the issue in comment #11 of #3203311: The current page title is not truncated when selecting "Truncate the page's title to a maximum number.". Branch added here with the fix so folks can patch.

  • wells committed 7c8ca13 on 2.x 
    Issue #3223467: Breadcrumb item strings truncated to zero after 2.0.0...
parthabapari’s picture

parthabapari commented

Thanks for the patch. Its working like charm.

hunterbuchanan’s picture

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

Upgrading to latest 2.x-dev with this patch committed fixes this issue for us as well. If possible, please include this in a 2.0.1 release soon, we'd like to move our production sites to the new branch in our next round of updates if this is all wrapped up. Thanks a bunch!

greg boggs’s picture

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

Release created. :}

Status: Fixed » Closed (fixed)

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