mollie / Shopware6

MIT License
51 stars 53 forks source link

Not working on Shopware 6.5 #576

Closed drieken closed 1 year ago

drieken commented 1 year ago

I can't enable the plugin on Shopware 6.5 because it's not supportet yet.

boxblinkracer commented 1 year ago

Hi

thank you for your message yes, it's not available at the moment

we are working really really hard since a while on it, and are (hopefully) almost at the end now, then we start a more intense testing phase and release it

drieken commented 1 year ago

Great to hear!

Tim-VL commented 1 year ago

Hi @boxblinkracer, any updates?

weiterfunken commented 1 year ago

Hey @boxblinkracer , any updates for 6.5? Our Clients still waiting :-(

boxblinkracer commented 1 year ago

hi. yes so latest master is working. we are already in the testing phase and test everything in 6.4 and 6.5. so should be released very soon :)

weiterfunken commented 1 year ago

@boxblinkracer perfect :-)

AndreasA commented 1 year ago

Hi @boxblinkracer Any news regarding a 6.5 supported official release - without the need to use dev-master 😄 ?

BlackScorp commented 1 year ago

hi @AndreasA , @weiterfunken @Tim-VL @drieken

we have released the new version today, now the plugin works with shopware 6.5.

TanjaBayer commented 1 year ago

I installed the new version of the plugin (4.0), and when doing the upgrade it says compatible

But when upgrading to 6.5.0.0 the upgrade fails with the following error leaving the shop in a broken state. Any ideas?

PHP Fatal error:  Class Kiener\MolliePayments\Compatibility\Bundles\FlowBuilder\Actions\ShipOrderAction contains 1 abstract method and must therefore be declared abstract or implement the remaining methods (Shopware\Core\Content\Flow\Dispatching\Action\FlowAction::handleFlow) in /sw6/vendor/store.shopware.com/molliepayments/src/Compatibility/Bundles/FlowBuilder/Actions/ShipOrderAction.php on line 14
Symfony\Component\ErrorHandler\Error\FatalError {
#12400
  #message: "Error: Class Kiener\MolliePayments\Compatibility\Bundles\FlowBuilder\Actions\ShipOrderAction contains 1 abstract method and must therefore be declared abstract or implement the remaining methods (Shopware\Core\Content\Flow\Dispatching\Action\FlowAction::handleFlow)"
  #code: 0
  #file: "/sw6/vendor/store.shopware.com/molliepayments/src/Compatibility/Bundles/FlowBuilder/Actions/ShipOrderAction.php"
  #line: 14
  -error: array:4 [
    "type" => 1
    "message" => "Class Kiener\MolliePayments\Compatibility\Bundles\FlowBuilder\Actions\ShipOrderAction contains 1 abstract method and must therefore be declared abstract or implement the remaining methods (Shopware\Core\Content\Flow\Dispatching\Action\FlowAction::handleFlow)"
    "file" => "/sw6/vendor/store.shopware.com/molliepayments/src/Compatibility/Bundles/FlowBuilder/Actions/ShipOrderAction.php"
    "line" => 14
  ]
}
boxblinkracer commented 1 year ago

Hi there

huch....thats totally weird, we even have cypress tests in there... and its been manually tested (like really all tests that we have have been manually tested in 6.4 and 6.5, 200 to 300 e2e tests)

wondering if this is only in the upgrade phase. how do you do the upgrade? CLI or admin? the error just says that the code is wrong...but thats completely weird. would you mind trying it in a fresh installation, just to see if anything is weird on your side?

do you know dockware? "docker run --rm -p 80:80 -p 443:443 dockware/dev:6.5.0.0" that runs a fresh shopware 6.5

thank you

TanjaBayer commented 1 year ago

I did the upgrade via the admin ui.

docker run --rm -p 80:80 -p 443:443 dockware/dev:6.5.0.0 and than doing a mollie install you mean?

boxblinkracer commented 1 year ago

thanks for the feedback, ok so when its also in the new shopware version, then it might really be an upgrade problem (like a race condition) or caching stuff....phu...

uninstalling seems like a good solution, although annoying

yes, the docker line is just for testing...just upload the zip in the admin for testing and see if it works

please let me know what happens, thanks

AndreasA commented 1 year ago

@TanjaBayer Did you check your PHP version? I remember running into a similar issue when switching to dev-master, but I accidentally used the forgot to change the PHP version in my Docker container (which was still 7.4). Shopware 6.5+ only supports PHP 8.1 or 8.2

however, it is possible that it might be some opcache (or similar) as handleFlow was changed in the later version in Shopware 6.5 to be abstract but it has been implemented in ShipOrderAction with plugin version 4.0, so it would be a bit weird but not impossible.

TanjaBayer commented 1 year ago

@TanjaBayer Did you check your PHP version? I remember running into a similar issue when switching to dev-master, but I accidentally used the forgot to change the PHP version in my Docker container (which was still 7.4). Shopware 6.5+ only supports PHP 8.1 or 8.2

I installed PHP 8.1, When doing the upgrade via the admin-ui I think there is no way doing it with an older version, they check the version before letting you do the upgrade

@boxblinkracer

yes, the docker line is just for testing...just upload the zip in the admin for testing and see if it works

Installing mollie via zip upload in a fresh install of shopware 6.5.0.0 works without a problem. I also think it is more like a race condition during the upgrade.

Will give an update once I have news about the deployed system with the upgrade and the uninstalled mollie.

AndreasA commented 1 year ago

@TanjaBayer

TanjaBayer commented 1 year ago

@AndreasA

AndreasA commented 1 year ago

@TanjaBayer One last question - though probably not a solution - but from your comment above the plugin code used that produces the failure is in /sw6/vendor/store.shopware.com/molliepayments/.

Did you originally install the plugin using composer? If so, I am acutally not sure what happens if a plugin update is started in the administration as I think Shopware installs plugin updates (if installed through the administration) to custom/plugins. However, It is possible that the class autoloading, still loads various files from the old folder in that scenario. But you would need to check the classloader config of composer and custom/plugins folder on the server to verify. In that case it is probably a Shopware bug that this is even possible 😄

Otherwise it could still be an opcache/realpath issue.

TanjaBayer commented 1 year ago

Did you originally install the plugin using composer

Yes that is how it was installed. But the updates of the plugins are also done via cli during the build/startup phase. Only the shopware update itself is done via UI

Btw. upgrade with uninstall/reinstall of the mollie extension worked in the dev environment so I guess I will go this way.

boxblinkracer commented 1 year ago

hi all, and thanks @AndreasA for your great input :) i guess the most efficient way to get you going is, that you uninstall/reinstall as you just mentioned with the dev environment. can you post if it worked in here (once you do it on production)

i would then make sure shopware is notified about this thread, maybe they can give some input too, or even discover anything wrong in shopware

AndreasA commented 1 year ago

Btw. upgrade with uninstall/reinstall of the mollie extension worked in the dev environment so I guess I will go this way.

great. At least it is working now. Maybe a info on the store page could potentially help others.

Yes that is how it was installed. But the updates of the plugins are also done via cli during the build/startup phase. Only the shopware update itself is done via UI

Ah. OK. Then this shouldn't be an issue. Though I have never done a Shopware update using the UI, so I have no idea what might go wrong there. But I guess some kind of caching issue is still the most likely scenario, be it that a stale Symfony cache, that is used during the upgrade, or maybe opcache etc. related.

TanjaBayer commented 1 year ago

Hi @boxblinkracer @AndreasA thanks for your fast feedback. I will give an update once we do the upgrade on the other environments, however as another plugin is at the moment blocking the upgrade I can not tell when that will be.

boxblinkracer commented 1 year ago

Hi ok now worries, just let us know once its done :) ill pass this thread on to shopware in the meantime

boxblinkracer commented 1 year ago

ok shopware told me, it might be due to the opcache

boxblinkracer commented 1 year ago

so we have a lot of positive feedback on 6.5 by now i hope its ok that i close this issue

feel free to open a new one or this one again :) thank you

lchhieu commented 1 year ago

I dont think this plugin is ready for Shopware 6.5. I created a new issue https://github.com/mollie/Shopware6/issues/632