Problem/Motivation

Using PHP 8.2 I get the following warnings in my log:
"Using ${var} in strings is deprecated, use {$var} instead."

CommentFileSizeAuthor
#6 3340222-6.patch3.7 KBhchonov
#6 interdiff-2-6.diff519 byteshchonov
#2 3340222-php-8.2-deprecations.patch3.7 KBZsuffa Dávid

Issue fork lazy-3340222

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

Zsuffa Dávid created an issue. See original summary.

Zsuffa Dávid’s picture

Zsuffa Dávid commented
Status: Active » Needs review
StatusFileSize
new 3340222-php-8.2-deprecations.patch3.7 KB

Patch attached.

dpi’s picture

dpi
Location Perth, Australia
commented
Issue tags: -deprecated +PHP 8.2
urvashi_vora’s picture

urvashi_vora commented
Assigned: Unassigned » urvashi_vora

urvashi_vora opened merge request !11

hchonov’s picture

hchonov
Primary language Bulgarian
Location Burgas
commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new interdiff-2-6.diff519 bytes
new 3340222-6.patch3.7 KB

The patch had only one small issue with a double dollar sign. I fixed it and I feel comfortable RTBCing this small issue.

urvashi_vora’s picture

urvashi_vora commented
Assigned: urvashi_vora » Unassigned

Unassigning.

  • osman committed 51100a3c on 8.x-3.x authored by hchonov 
    Issue #3340222 by hchonov, Zsuffa Dávid: php 8.2 deprecations warnings
osman’s picture

osman
Location Copenhagen
commented
Status: Reviewed & tested by the community » Fixed

Thanks Hristo, and everyone : )

osman’s picture

osman
Location Copenhagen
commented
Status: Fixed » Closed (fixed)
taran2l’s picture

taran2l
Primary language Ukrainian
Location Lviv
commented

@osman, can you tag a new release ? 3.12 does not contain this fix. thanks

jddh’s picture

jddh commented

Can this please be released? 3.12 throws these warnings all over the place.

nikral’s picture

nikral commented

Hello @osman,
Do you have any news for the release please?
The latest tag does not yet contain this fix
Thanks

socialnicheguru’s picture

socialnicheguru commented

Just to note for others. this is in dev not the main version even if they have the saem date.