Problem/Motivation

Clearly environment with d9

Steps to reproduce

1. standard install with bartik.
2. enable easy_breadcrumb, report the error.

CommentFileSizeAuthor
#13 easy_breadcrumb-3202162-13.patch911 bytesreszli
#3 easy_breadcrumb-3202162-1.patch427 bytesjohn.nie

Comments

John.nie created an issue. See original summary.

greg boggs’s picture

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

Perfect. Thanks John.Nie.

john.nie’s picture

john.nie commented
StatusFileSize
new easy_breadcrumb-3202162-1.patch427 bytes
steveoriol’s picture

steveoriol
Location Grenoble 🇫🇷
commented

Thanks John.Nie, "drush updb" PASS now :-)

neslee canil pinto’s picture

neslee canil pinto
he/him
Primary language English
commented
Status: Active » Reviewed & tested by the community
neslee canil pinto’s picture

neslee canil pinto
he/him
Primary language English
commented
Title: $string ("100") must be a string. » Can't update DB after updating Easy Breadcrumb - ($string ("100") must be a string)

@here thanks for the patch and it solves the issue. but not sure why this didn't occur for me or Greg when we tested it before the release.

FY: This will go into a new release soon

hassebasse’s picture

hassebasse commented

The patch worked fine for me as well. Thanks.

neslee canil pinto’s picture

neslee canil pinto
he/him
Primary language English
commented
Status: Reviewed & tested by the community » Fixed
karimbou’s picture

karimbou commented

Thanks this fixed the issue for me when trying to update the module. Using 8.9.13 at the time

kaipipek’s picture

kaipipek commented

Just to let you know that I got the same error with the latest production version 8.x-1.14.

neslee canil pinto’s picture

neslee canil pinto
he/him
Primary language English
commented

@kaipipek yes, we are planning to release it soon, till then you can use this patch.

reszli’s picture

reszli
he/him
Primary language Hungarian
Location Târgu Mureș
commented
StatusFileSize
new easy_breadcrumb-3202162-13.patch911 bytes

Although this patch solves the error, I think it would make more sense to change the schema type from text to integer for the truncator limit, since it's a numeric value that does not require to be translatable.

zebda’s picture

zebda commented

#3 is working for me 8.x-1.14 on Drupal 9.1.5
#13 failed to patch

Status: Fixed » Closed (fixed)

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

karimbou’s picture

karimbou commented

Hello i'm now unabled to apply this patch which was working fine previously ? :/

Applying patches for drupal/easy_breadcrumb
https://www.drupal.org/files/issues/2021-03-08/easy_breadcrumb-3202162-1... (Allowing change of truncate size from url to breadcrumb)
Could not apply patch! Skipping. The error was: Cannot apply patch https://www.drupal.org/files/issues/2021-03-08/easy_breadcrumb-3202162-1...