Upon update to version 7.x-2.13, the separator began appearing at the end of the breadcrumb even though the option was not selected on the configuration page ("Show segments separator at the end of the breadcrumb"). It seems to be an issue with the new easy-breadcrumb.tpl.php file which replaced the output that was previous handled by the .module file in function theme_easy_breadcrumb($variables).

After looking at the old .module file, I changed line 5 from the new easy-breadcrumb.tple.php file from
if ($i < $segments_quantity - $separator_ending):
to
if ($i < $segments_quantity - 1):

That fixed the problem for my configuration at least. But it broke the functionality of using the configuration checkbox to the purposely include the separator at the end; that checkbox doesn't work with my temporary fix.

CommentFileSizeAuthor
#6 separator_appears_at_end-2894016-6.patch659 bytestatarbj

Comments

gubbeleye created an issue. See original summary.

greg boggs’s picture

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

Thanks for the report. We'll get this fixed and a new release out shortly!

tatarbj’s picture

tatarbj
Primary language Hungarian
Location Brussels
commented
Assigned: Unassigned » tatarbj
Status: Active » Needs work

Thanks for the report, i'm picking up the issue and will post here a patch for that - as also other issues got reported, for improvements too, i would publish a new release with them too!
I'm gonna get back to you here with results in at most in a few days!
Thank you again,
Bests,
Balazs.

mitchelljj’s picture

mitchelljj commented

@tatarbj any estimate of date when a new version that fixes this issue will be released? Thanks, John

tatarbj’s picture

tatarbj
Primary language Hungarian
Location Brussels
commented

Hi @mitchelljj,
i would say beginning of next week as in the weekend it seems i'll have more time to test it deeper to avoid other issues popping up!
Bests,
Balazs.

tatarbj’s picture

tatarbj
Primary language Hungarian
Location Brussels
commented
Assigned: tatarbj » Unassigned
Status: Needs work » Needs review
StatusFileSize
new separator_appears_at_end-2894016-6.patch659 bytes

Hi @mitchelljj,
thank you again for the reported issue, i've found the original of it that was a typo :)
Please use the patch until we release the new version at latest next week!
Bests,
Balazs.

  • tatarbj committed d11b02b on 7.x-2.x 
    Issue #2894016 by tatarbj, gubbeleye: Separator appears at end despite...
brianlp’s picture

brianlp commented

Works, thanks.

tatarbj’s picture

tatarbj
Primary language Hungarian
Location Brussels
commented
Status: Needs review » Reviewed & tested by the community

Thanks for the feedbacks, i'm putting it to RTBC

jenlampton’s picture

jenlampton
she / her
commented
Status: Reviewed & tested by the community » Fixed

Looks like this was already committed to 2.x. Changing status to fixed.

Status: Fixed » Closed (fixed)

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

tatarbj’s picture

tatarbj
Primary language Hungarian
Location Brussels
commented
Parent issue: » #2938503: New minor release of 7.x