search

Adyen / adyen-shopware5

MIT License
11 stars 12 forks source link

Cronjobs are not running #54

Closed EggersS1 closed 4 years ago

EggersS1 commented 4 years ago

Describe the bug Recurring payments are not going out from the AboCommerce (Shopware 5) to Adyen to support subscription payments.

PHP Fatal error: Uncaught ArgumentCountError: Too few arguments to function AdyenPayment\Components\NotificationProcessor\Authorisation::__construct(), 3 passed in /var/customers/webs/draeger/ppe-shop/var/cache/production_202005221140/proxies/ShopwareProduction3362b35eaa965cff4f4fccda9aa2ef9f3ec56040ProjectContainer.php on line 6526 and exactly 4 expected in /var/customers/webs/draeger/ppe-shop/custom/plugins/AdyenPayment/Components/NotificationProcessor/Authorisation.php:49 Stack trace:

0 /var/customers/webs/draeger/ppe-shop/var/cache/production_202005221140/proxies/ShopwareProduction3362b35eaa965cff4f4fccda9aa2ef9f3ec56040ProjectContainer.php(6526): AdyenPayment\Components\NotificationProcessor\Authorisation->__construct()

1 /var/customers/webs/draeger/ppe-shop/vendor/symfony/dependency-injection/Container.php(306): ShopwareProduction3362b35eaa965cff4f4fccda9aa2ef9f3ec56040ProjectContainer->getAdyenPayment_Components_NotificationProcessor_AuthorisationService()

2 /var/customers/webs/draeger/ppe-shop/engine/Shopware/Compo in /var/customers/webs/draeger/ppe-shop/custom/plugins/AdyenPayment/Components/NotificationProcessor/Authorisation.php on line 49

To Reproduce Steps to reproduce the behavior:

  1. Adyen plug in approved in Shopware 5 backend
  2. Cronjub is runnning
  3. Payment trigger isn't sent to Adyen
  4. See error

Expected behavior Subscription payments are able to process via digital payment provider

Desktop (please complete the following information):

Additional context Add any other context about the problem here.

Byorun commented 4 years ago

The component.xml is missing the new 4th parameter https://github.com/Adyen/adyen-shopware5/blob/8c3b08f3d613fe07b364b66c74d36240d73f9d81/Resources/services/components.xml#L81l

EggersS1 commented 4 years ago

Hi Byorun, many thanks for the reply. we are using the standard plug in from adyen. Has adyen to change it in the plug in or is supposed to change something?

Byorun commented 4 years ago

@EggersS1 This is a bug in the current version of the plugin, we already notified adyen about it, my guess is that we will soon see a 1.5.1 hotfix.

If you can't wait for a fix you can add <argument type="service" id="models"/> as the 4th argument parameter for the service registration in Resources/services/component.xml for adyen_payment.components.notification_processor.authorisation and adyen_payment.components.notification_processor.capture

EggersS1 commented 4 years ago

Hi acampos1916, does it mean that cronjobs now are possible to make? and that it support subscritpion models?

EggersS1 commented 4 years ago

Greate to see is there already a date published when the release with bug fixes will be available?

cyattilakiss commented 4 years ago

Greate to see is there already a date published when the release with bug fixes will be available?

Hi @EggersS1 , @acampos1916 is working on the release at the moment, hopefully it will be published today 👍 Best, Attila Adyen