Problem/Motivation
Deprecated function: str_replace(): Passing null to parameter #2 ($replace) of type array|string is deprecated in require_once() (line 19 of /var/www/hfspo/web/modules/contrib/taxonomy_term_depth/constants.inc)
| Comment | File | Size | Author |
|---|---|---|---|
| #15 | taxonomy_term_depth-null_param-3283096-15.patch | 598 bytes | liam morland |
Issue fork taxonomy_term_depth-3283096
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:
Comments
Comment #3
renrhafSetting to review
Comment #4
hstoellinger commentedHello,
I have just upgraded to PHP 8.1 (Kubuntu 22.04) and Drupal 9.3.16. Now I see the message
Might a patch be available in the meantime?
Regards
H. Stoellinger
Comment #5
renrhafHi, you can use patch from the merge request as follow : https://git.drupalcode.org/project/taxonomy_term_depth/-/merge_requests/...
Comment #6
hstoellinger commentedThanks! Seems to work for me (DR 9.4.3)
Comment #7
karimbou commentedWe're having this issue aswell on a recent upgrade of D9.4.5 PHP8.1 every module updated, and it's messing views with elastic search passing taxonomy parameters.
Comment #8
googletorp commentedWorking for me as well.
Comment #9
smulvih2This works for me as well, would be nice to get this into a new release.
Comment #10
rar9 commented+1 working
Comment #11
jasjeet kaur brar commentedTested, working for me on a Drupal version 9.5.1 . Thanks
Comment #12
shahzad-anwar commentedRTBC.
Tested and worked for me on Drupal 9.5.3 , PHP 8.1.13
Comment #13
joseph.olstadComment #14
liam morlandSimple patch; works correctly. +1 for commit.
Comment #15
liam morlandFix from merge request as a patch.
Comment #17
joseph.olstadFixed in dev branch, dev release
Comment #20
joseph.olstad@Liam Morland, great work, on the rc1 release , I suggest tagging it as 7.x-1.6, it's been about 9 weeks.
Comment #21
liam morlandThe RC was released on May 9. I typically wait until an RC is two weeks old with no release-blockers having been found before making a full release.
Comment #22
joseph.olstad@Liam Morland, ok ya I must have been confused, I see the latest dev commit was last month but the RC1 release was yesterday, thanks!