Problem/Motivation

There are a number of deprecated constructs in the module beta2 release.

  • Deprecated: Using ${var} in strings is deprecated, use {$var} instead in (site)/web/modules/contrib/g2/src/Controller/Main.php on line 145
  • Deprecated: Using ${var} in strings is deprecated, use {$var} instead in (site)/web/modules/contrib/g2/src/Controller/Main.php on line 163
  • Deprecated: Using ${var} in strings is deprecated, use {$var} instead in (site)/web/modules/contrib/g2/src/Controller/Main.php on line 163
  • Deprecated: Using ${var} in strings is deprecated, use {$var} instead in (site)/web/modules/contrib/g2/src/Form/SettingsForm.php on line 554

Steps to reproduce

Run the site on PHP 8.2

Proposed resolution

Fix them.

Remaining tasks

Do it.

User interface changes

None.

API changes

None.

Data model changes

None.

CommentFileSizeAuthor
#5 g2-3382156-2.patch1.35 KBmohd sahzad

Issue fork g2-3382156

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

fgm created an issue. See original summary.

fgm opened merge request !10

fgm opened merge request !11

fgm closed merge request !10

mohd sahzad’s picture

mohd sahzad commented
StatusFileSize
new g2-3382156-2.patch1.35 KB

i have fixed deprecated: Using ${var} in strings is deprecated, use {$var} instead

fgm’s picture

fgm
Primary language French
Location Paris, France
commented
Status: Active » Fixed

Thanks, but that was already done in the MR above, so there was no reason to add a patch; and your patch is missing about half the cases.

There's more work available in the other issues, if you are interested, though, especially reviewing / completing #3378056: Support incremental footnotes block update and #3382138: Add new "Help topics" support.

  • fgm committed 9c58a0d5 on 8.x-1.x 
    Issue #3382156: Remove PHP 8.2 deprecations

Status: Fixed » Closed (fixed)

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