Problem/Motivation

With version 2.1.1, logs are filled up with the deprecation warnings related to PHP 8.2 for dynamic property creation.
Checklist deprecation

Steps to reproduce

With a using of GDPR and creating SARs will runa cron execution which will throw deprecation warnings coming from ChecklistapiChecklist.php and when user navigate to admin/people page deprecation of dynamic property creation are coming from ChecklistapiPermissions.php

Proposed resolution

Follow https://php.watch/versions/8.2/dynamic-properties-deprecated

CommentFileSizeAuthor
Screenshot 2023-03-26 at 8.10.01 PM.png357.78 KBankitv18

Issue fork checklistapi-3350474

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

ankitv18 created an issue. See original summary.

ankitv18 opened merge request !9

ankitv18’s picture

ankitv18 commented
Assigned: ankitv18 » Unassigned
Status: Active » Needs review

Can anyone review MR!9 and move this issue ahead??

rajeshreeputra’s picture

rajeshreeputra
he/him
Location Pune
commented
Status: Needs review » Reviewed & tested by the community

Changes looks good, hence moving ahead and RTBC.

rajeshreeputra’s picture

rajeshreeputra
he/him
Location Pune
commented
Version: 2.1.1 » 2.0.x-dev
vuil’s picture

vuil
Location Bulgaria 🇧🇬 🇪🇺 🌍
commented
vuil’s picture

vuil
Location Bulgaria 🇧🇬 🇪🇺 🌍
commented
Assigned: Unassigned » traviscarden
rajeshreeputra’s picture

rajeshreeputra
he/him
Location Pune
commented

+1 RTBC

TravisCarden made their first commit to this issue’s fork.

traviscarden’s picture

traviscarden commented
Status: Reviewed & tested by the community » Fixed

Merged. Thanks everyone!

tobiasb’s picture

tobiasb
Primary language German
Location Berlin
commented
Status: Fixed » Needs work

The new source translation is @description ;-)

Just copy t('String') to the permission.

traviscarden’s picture

traviscarden commented
Status: Needs work » Fixed

Thank you, @tobiasb. Would you please create a new issue for that? ...and include a little more detail? :)

vuil’s picture

vuil
Location Bulgaria 🇧🇬 🇪🇺 🌍
commented

Status: Fixed » Closed (fixed)

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

traviscarden’s picture

traviscarden commented
Assigned: traviscarden » Unassigned