Skip to content

Fixed PHP 8.2 deprecation warning: 'creation of dynamic property'. #1

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversation

hostep
Copy link
Contributor

@hostep hostep commented Jun 3, 2023

Fixes deprecation warning with PHP 8.2: Creation of dynamic property xxx is deprecated
I've used old PHP array syntax, just to stay consistent with the rest of the code.

Found with phpstan:

  1. Have latest version of phpstan installed somewhere (I've used version 1.10.15)
  2. Clone this repo
  3. Inside the repo, run:
$ /path/to/phpstan analyse --level=0 . | grep 'Access to an undefined property'
  1. Expected to see no errors, but seeing:
  468    Access to an undefined property Zend_Loader_Autoloader_Resource::$_resources.

PR fixes:

  • missing member

This fixes one of the warnings mentioned in https://github.com/magento/adobe-commerce-beta/issues/61, If I find more time and this PR actually gets approved, I might try to tackle the others as well one day.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants