Fix deprecation warnings on PHP 8.2

[jnamenyi]

The ${variable} styled string interpolation and dynamic property creation became deprecated in PHP 8.2, because of which we get a deprecation warning using Zen.

This change fixes these issues by replacing ${variable} to {$variable} and checking the existence of the property before assigning it in AbstractCompiledContainer

[codecov]

Codecov Report

Merging #16 (e694e95) into master (a7abc84) will decrease coverage by 0.08%.
The diff coverage is 92.85%.

❗ Current head e694e95 differs from pull request most recent head 535428d. Consider uploading reports for the commit 535428d to get more accurate results

@@             Coverage Diff              @@
##             master      #16      +/-   ##
============================================
- Coverage     89.16%   89.08%   -0.08%     
- Complexity      437      438       +1     
============================================
  Files            33       33              
  Lines          1117     1118       +1     
============================================
  Hits            996      996              
- Misses          121      122       +1     

Impacted Files	Coverage Δ
src/AbstractCompiledContainer.php	90.00% <50.00%> (-10.00%)	⬇️
src/Container/Definition/ClassDefinition.php	95.55% <100.00%> (ø)
src/Container/Definition/ReferenceDefinition.php	85.36% <100.00%> (ø)
src/Container/Definition/SelfDefinition.php	89.47% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[ellisgl]

@jnamenyi Can you make a signed commit for this?

[szepeviktor]

Here is how to find them all: git grep -e '\${[A-Z_a-z]' -- '*.php'

source

[kocsismate]

Thanks for the PR! I prefer not to add the property_exists check because facing a dynamic property creation deprecation notice should be a configuration error (as in maybe you have a wrong property override in your ContainerConfig?). If it's not the case, I'd be eager to receive some kind of reproducer/excerpt from your code for debugging if possible.

For reference: this is the only place where a property may be added to the injectable properties array via the #[Inject] attribute, and they are added here via the user config.

[kocsismate]

I'm merging the PR as is, but will remove the property_exists() call in a followup commit.