Problem/Motivation

error message with PHP 8.2 : Deprecated: Using ${var} in strings is deprecated, use {$var} instead in /matomo/modules/matomo_tagmanager/matomo_tagmanager.install on line 43

Proposed resolution

change line 43 to be compliant with php8.2

CommentFileSizeAuthor
#2 deprecated_php8.2_-3414796-1.patch728 bytesnicoc

Issue fork matomo-3414796

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

Nicoc created an issue. See original summary.

nicoc’s picture

nicoc commented
StatusFileSize
new deprecated_php8.2_-3414796-1.patch728 bytes
andreasderijcke’s picture

andreasderijcke
Location Antwerpen / Gent
commented
Status: Active » Reviewed & tested by the community

Thanks, small fix, lot less warnings :)

finex’s picture

finex commented

Hi, the patch works fine. Thank you.

shelane’s picture

shelane
Primary language English
commented
Status: Reviewed & tested by the community » Closed (duplicate)
Related issues: +#3390345: Using ${var} string is deprecated
timohuisman’s picture

timohuisman
Primary language Dutch
Location Leiden, Netherlands
commented
Status: Closed (duplicate) » Reviewed & tested by the community
Related issues: +#3441281: PHP 8.3 Deprecated: Using ${var} in strings is deprecated, use {$var} instead

I see that this issue is closed because it seems like its a duplicate of #3390345: Using ${var} string is deprecated, but that fix is only committed on the 2.0 version. The 8.x-1.23 version is as far as I can tell still the recommend release, so it would be nice if this can also get fixed in that release.

The patch from #2 does still apply against 8.x-1.23 and resolves the warnings.

#3441281: PHP 8.3 Deprecated: Using ${var} in strings is deprecated, use {$var} instead is actually a duplicate of this issues.

grimreaper’s picture

grimreaper
Primary language French
Location France 🇫🇷
commented
Assigned: Unassigned » grimreaper

grimreaper opened merge request !67

  • grimreaper committed 7db7ef6e on 8.x-1.x 
    Issue #3414796 by nicoc, andreasderijcke, finex, shelane, grimreaper,...
grimreaper’s picture

grimreaper
Primary language French
Location France 🇫🇷
commented
Assigned: grimreaper » Unassigned
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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