NEXT-00000 - Add missing transactions association

[jasperP98]

1. Why is this change necessary?

The \Shopware\Storefront\Controller\AccountOrderController::updateOrder function loads the order and checks the transaction state in the \Shopware\Core\Checkout\Order\SalesChannel\OrderService::isPaymentChangeableByTransactionState but the order transaction association is not loaded so this will always be null.

2. What does this change do, exactly?

This change adds the missing association when loading the order.

3. Describe each step to reproduce the issue or behaviour.

1. Create an order
2. Do a POST to the frontend.account.edit-order.update-order route
3. You can now see that the order transactions are always null

4. Please link to the relevant issues (if any).

/

5. Checklist

• I have rebased my changes to remove merge conflicts
• I have written tests and verified that they fail without my change
• I have created a changelog file with all necessary information about my changes
• I have written or adjusted the documentation according to my changes
• This change has comments for package types, values, functions, and non-obvious lines of code
• I have read the contribution requirements and fulfil them.

[mstegmeyer]

Hi @jasperP98, would you mind adding a unit test for the AccountOrderController that validates, that the Criteria passed to the order contains the transactions & stateMachineState?

[mstegmeyer]

@jasperP98 any update here?

[jasperP98]

@mstegmeyer Hi! Yes i had some trouble with setting up the test environment on my local machine. Will have a closer look at that ASAP

[jasperP98]

@mstegmeyer I added a test

[shopware-github-importer]

Hello,

thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/NEXT-36025

Please use this issue to track the state of your pull request.