- Merge request !48Issue #3284123: PHP 8.1: Deprecated function: explode(): Passing null to parameter #2 ($string) of type string is deprecated → (Merged) created by gayatri chahar
Automatically closed - issue fixed for 2 weeks with no activity.
- Status changed to Fixed
over 1 year ago 8:30am 7 June 2023 - 🇺🇸United States diamondsea
Reiterating #21, after applying the updated code, you need to click Save on /admin/config/user-interface/easy-breadcrumb to update the settings and then re-export (drush cex) your config files. In my case this resulted in the config following changes (not having changed any settings, just clicking Save to update):
+alternative_title_field: ''
+menu_title_preferred_menu: ''
-capitalizator_ignored_words: null
+capitalizator_ignored_words: { }
+limit_segment_display: false
+segment_display_limit: null
+truncator_mode: false
+truncator_length: null
+truncator_dots: false
+remove_repeated_segments_text_only: 0
+home_segment_validation_skip: 0Perhaps there should be an update script that makes these changes automatically?
- 🇺🇸United States Greg Boggs Portland Oregon
Definately sounds like we need an update hook to save the form
- 🇺🇸United States joegraduate Arizona, USA
Was this ever actually committed/merged? I don't see the MR changes reflected in the latest 2.x branch.
- 🇬🇧United Kingdom Finn Lewis
I don't think this was ever committed or released.
Here's the merge request: https://git.drupalcode.org/project/easy_breadcrumb/-/merge_requests/48/d...
It doesn't look like it is on the 2.x branch: https://git.drupalcode.org/project/easy_breadcrumb/-/blob/2.x/src/EasyBr...
And not in the 2.0.5 tag: https://git.drupalcode.org/project/easy_breadcrumb/-/blob/2.0.5/src/Easy...
- 🇮🇹Italy apaderno Brescia, 🇮🇹
There should be a comment automatically posted in this issue whether a new commit with a #-reference to this issue in the commit summary, or the MR is merged with the project. I do not see such comment.
- Status changed to RTBC
11 months ago 5:03pm 3 January 2024 - last update
11 months ago Patch Failed to Apply - last update
11 months ago Patch Failed to Apply - last update
11 months ago Patch Failed to Apply - last update
11 months ago 6 pass - 🇮🇹Italy apaderno Brescia, 🇮🇹
Now the MR is Ready to merge by members who can write to the target branch.
- last update
11 months ago 6 pass -
apaderno →
committed e1655f42 on 2.x authored by
gayatri chahar →
Issue #3284123 by gayatri chahar, ptmkenny, philltran, HitchShock:...
-
apaderno →
committed e1655f42 on 2.x authored by
gayatri chahar →
- Status changed to Fixed
11 months ago 5:11pm 3 January 2024 - 🇺🇸United States Greg Boggs Portland Oregon
Thanks for catching this. Sometimes the merge fails and I have to repress the button, I must have missed it on this one.
Automatically closed - issue fixed for 2 weeks with no activity.