Refund orders with several variants for one configurable about magento-2 HOT 9 CLOSED

For order items 0 and 1 in your getAllItems() list, the sku and productId are the same, thus I'd say that both items must be included. Or is that a typo? Or maybe I'm not quite understanding.

from magento-2.

Or is that a typo?

Sorry, I updated my post above. Items 1, 3, 5 are skipped as dummy order items in \Payone\Core\Model\Api\Invoice::addProductInfo.

Additional info: we use

    "name": "payone-gmbh/magento-2",
        "version": "1.3.5",

from magento-2.

Okay, I see :) Basically, this can be avoided by adding the sku as an additional condition for adding the product in the \Payone\Core\Model\Api\Invoice::addProductItem method, right? I'm not quite sure about possible side effects, but we'll check and come back to you.

from magento-2.

Thank you, that should do. Meanwhile I figured out, that in our case the configuration payone_general/invoicing/transmit_enabled = 0 is a suitable workaround for that issue.

from magento-2.

This workaround should work for most payment methods, but be advised that some methods require the transmission of a basket (e.g. factoring providers "safe invoice") and this transmit_enabled flag will be overridden for those.

from magento-2.

#127 is supposed to fix this issue. I can't really test this with our test data - can you confirm this fixes your issue?

from magento-2.

The code looks good to me. Currently I am not able to set up a test scenario either. I'll give an update, as soon as I can test it.

from magento-2.

Works fine for debits now, invoices/capture not explicitly tested as we have a different workflow for that. But debits (Gutschriften) work perfect, thanks.

from magento-2.

Thanks for the feedback!

from magento-2.