search

tanoconsulting / ezmigrationbundle2

This bundle makes it easy to handle Ibexa DXP 3 (aka. eZPlatform 3) content upgrades/migrations
GNU General Public License v2.0
10 stars 7 forks source link

Fix the "FixExecutedMigrationsPaths" for Ibexa 3.x #23

Closed McLone closed 1 year ago

McLone commented 1 year ago

Hi!

Those changes aim to fix an error that occur at the first "kaliop:migration:migrate" : the bundled migration "FixExecutedMigrationsPaths" crash with a "Migration failed! Reason: Error in execution of step 1: You have requested a non-existent service "ezpublish.connection". in file /var/www/app/vendor/symfony/dependency-injection/Container.php line 271", followed if fixed by another error about the console_io service not being public.

You might feel a déjà-vu of https://github.com/tanoconsulting/ibexa-migration-bundle/pull/1

Regards

gggeek commented 1 year ago

The changes look fine. I wonder when ezpublish.connection stopped being a public service though. Is it in eZPlatform 3.0, or maybe 3.3?

And, more to the point: is ezpublish.persistence.connection available since 3.0?

McLone commented 1 year ago

ezpublish.connection was remove on ezsystems/ezplatform-kernel since 1.0 (https://github.com/ezsystems/ezplatform-kernel/blob/f5393e79d9abe73f1f0d1bc90b0fc8a18922c11a/doc/bc/changes-1.0.md#removed-services) which is first used in eZPlatform 3.0, as far as I know.

ezpublish.persistence.connection is indeed available since 3.0, and even earlier (eZPublish 5.3, starting from here https://github.com/ezsystems/ezplatform-kernel/commit/9b01725e9802874558977eeca1d793cc03b6ae56)

gggeek commented 1 year ago

Thx.

Released as 1.0.3