search

magento / data-migration-tool

Magento Data Migration Tool
https://devdocs.magento.com/guides/v2.3/migration/bk-migration-guide.html
Open Software License 3.0
338 stars 199 forks source link

Php 8.2 Compatibility #914

Open fooman opened 1 year ago

fooman commented 1 year ago

Preconditions

  1. php 8.2
  2. Magento 2.4.6

Steps to reproduce

  1. install data migration tool
  2. run Magento compilation: bin/magento setup:di:compile

Expected result

  1. no error

Actual result

1.

In ErrorHandler.php line 62:

Deprecated Functionality: Creation of dynamic property Migration\Logger\FileHandler::$filesystem is deprecated in /var/www/vendor/magento/data-migration-tool/src/Migration/Logger/FileHandler.php on line 50

Additional notes

m2-assistant[bot] commented 1 year ago

Hi @fooman. Thank you for your report. To speed up processing of this issue, make sure that you provided sufficient information. Add a comment to assign the issue: @magento I am working on this

Join Magento Community Engineering Slack and ask your questions in #github channel.

oceane410 commented 1 year ago

I had the same problem and hot fix it on my side (by declaring these missing properties) There are also issue when launching php bin/magento migrate:data

mustafaslk commented 1 year ago

php bin/magento migrate:data -a app/code/Vendor/Migration/etc/opensource-to-opensource/1.9.4.0/config.xml

Screenshot_5

jesse-deboer commented 1 year ago

@magento I am working on this

Created a PR for this: https://github.com/magento/data-migration-tool/pull/917. In the meantime you can use this as a patch: https://patch-diff.githubusercontent.com/raw/magento/data-migration-tool/pull/917.patch

m2-assistant[bot] commented 1 year ago

Hi @jesse-deboer! :wave: Thank you for collaboration. Only members of Community Contributors Team are allowed to be assigned to the issue. Please use @magento add to contributors team command to join Contributors team.

jesse-deboer commented 1 year ago

@victor-v-rad is it possible to merge this one to also give support for PHP 8.2?

deemantha22 commented 1 year ago

Hi,

I am getting the same error on M 246 PHP 8.2

Deprecated Functionality: Creation of dynamic property Migration\Logger\FileHandler::$filesystem is deprecated

added protected $filesystem; and it fixed it