Problem/Motivation

- We get this warning with PHP 8.2:
Deprecated function: Creation of dynamic property Drupal\conditional_fields\DependencyHelper::$module_handler is deprecated in Drupal\conditional_fields\DependencyHelper->__construct() (line 98 of modules/contrib/conditional_fields/src/DependencyHelper.php).

Proposed resolution

- Attached MR

CommentFileSizeAuthor
#3 3345459-Fix-PHP-8.2-deprecation-issue.diff491 bytesmaunm

Issue fork conditional_fields-3345459

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

j-vee created an issue. See original summary.

j-vee opened merge request !24

maunm’s picture

maunm commented
StatusFileSize
new 3345459-Fix-PHP-8.2-deprecation-issue.diff491 bytes

Attach the patch to quickly remediate on D10

paulocs’s picture

paulocs
Primary language Portuguese, Brazil
Location Belo Horizonte
commented
Status: Needs review » Reviewed & tested by the community

Merge request looks good to me.

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

thalles’s picture

thalles
Primary language Portuguese, Brazil
Location Teófilo Otoni - MG
commented
umekikazuya’s picture

umekikazuya commented

Patch #3 has been confirmed to be working properly on Version 10.0.7.

rishi.kulshreshtha’s picture

rishi.kulshreshtha
He/Him
Primary language English
Location Portland, Maine
commented
Assigned: j-vee » Unassigned
heddn’s picture

heddn
Primary language English
Location Illinois
commented

+1 on RTBC.

yovanny.gomez.oyola’s picture

yovanny.gomez.oyola commented

Patch #3 works as expected.

Drupal core version: 10.1.2
conditional_fields version: 4.0.0-alpha3

anybody’s picture

anybody
Primary language German
Location Porta Westfalica
commented
Priority: Normal » Major

Same here. Any chance to merge this to get the warnings away, finally? :)
(And a tagged release, please)

  • heddn committed 1a59d230 on 4.x authored by maunm 
    Issue #3345459 by maunm, heddn: Fix PHP 8.2 deprecation issue with...
heddn’s picture

heddn
Primary language English
Location Illinois
commented
Status: Reviewed & tested by the community » Fixed

Pushed and released https://www.drupal.org/project/conditional_fields/releases/4.0.0-alpha4.

Thanks everyone for your contributions.

anybody’s picture

anybody
Primary language German
Location Porta Westfalica
commented

@heddn thank you, a thousand times :)

Status: Fixed » Closed (fixed)

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

heddn closed merge request !24