Skip to content
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

FrontendProxyController.php - change CartOrderRoute to AbstractCartOrderRoute #492

Open
sascha-seyfert opened this issue May 15, 2024 · 4 comments
Assignees
Labels
Bug report Indicates that issue has been marked as a possible bug

Comments

@sascha-seyfert
Copy link

Hello dear Adyen team,

There is a bug in the Adyen Shopware 6 plugin.

This error occurs in connection with a third-party plugin.

Name of the third-party plugin: Conversion Checkout for Shopware 6
Store: https://store.shopware.com/en/loyxx57434896565m/conversion-checkout-for-shopware-6.html

Description of the problem / clear instructions for reproduction:

[2024-02-09T08:06:36.056488+00:00] request.CRITICAL: Uncaught PHP Exception TypeError: "Cannot assign LoyxxSW6ConversionCheckout\Core\Checkout\Cart\SalesChannel\CartOrderRoute to property Adyen\Shopware\Storefront\Controller\FrontendProxyController: :$cartOrderRoute of type Shopware\Core\Checkout\Cart\SalesChannel\CartOrderRoute" at FrontendProxyController.php line 71 {"exception":"[object] (TypeError(code: 0): Cannot assign LoyxxSW6ConversionCheckout\Core\Checkout\ \Cart\SalesChannel\CartOrderRoute to property Adyen\Shopware\Storefront\Controller\FrontendProxyController::$cartOrderRoute of type Shopware\Core\Checkout\Cart\SalesChannel\CartOrderRoute at /var/www /clients/client1/web2/web/production/releases/193/vendor/adyen/adyen-shopware6/src/Storefront/Controller/FrontendProxyController.php:71)"} []

PHP class:
Adyen\Shopware\Storefront\Controller\FrontendProxyController

Change:

  • private CartOrderRoute $cartOrderRoute;
  • private AbstractCartOrderRoute $cartOrderRoute;

private CartOrderRoute $cartOrderRoute;

@RokPopov
Copy link
Contributor

Hi @sascha-seyfert,

Thank you for reporting this issue. I have created an internal ticket for handling this. Once the fix is merged with the develop branch, this issue will automatically be closed.

Kind regards,
Rok

@RokPopov RokPopov added the Bug report Indicates that issue has been marked as a possible bug label May 16, 2024
@RokPopov RokPopov self-assigned this May 16, 2024
@MelvinAchterhuis
Copy link
Contributor

MelvinAchterhuis commented May 21, 2024

@RokPopov , same goes for PostNL:

Cannot assign PostNL\Shopware6\Controller\StoreApi\ContextSwitchRoute to property Adyen\Shopware\Storefront\Controller\FrontendProxyController::$contextSwitchRoute of type Shopware\Core\System\SalesChannel\SalesChannel\ContextSwitchRoute

See: https://github.com/Adyen/adyen-shopware6/pull/499/files

@AndreasA
Copy link
Contributor

AndreasA commented Aug 2, 2024

It seems like this was fixed with 4.0.0
https://github.com/Adyen/adyen-shopware6/blob/4.0.0/src/Storefront/Controller/FrontendProxyController.php

but not with 3.16.0 https://github.com/Adyen/adyen-shopware6/blob/3.16.0/src/Storefront/Controller/FrontendProxyController.php

which means that for 6.5 which is the LTS version already the fix is not available.

@rick7240
Copy link

rick7240 commented Sep 9, 2024

@RokPopov The problem was fixed in the 4.X version, but not in the 3.X version. Due to the Shopware Version not everyone can update to the 4.X version and is dependent on a bugfix in the 3.X version.

We now have to downgrade to a previous 3.X version. Actually we need the 3.16.1 but because of the ECP-9382-v3 Visa Compliance issue.

Can THIS bug also be fixed in the 3.X version in the short term?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug report Indicates that issue has been marked as a possible bug
Projects
None yet
Development

No branches or pull requests

5 participants