The \Commerce\Adyen\Payment\Modification abstract class has a constructor with three arguments: $order, $remote_transaction_status and $modification_type. Inside of construct there is a call of commerce_adyen_get_transaction_instance(), which can accept three arguments: $type, $order and $remote_status. As you can see the $remote_transaction_status and $remote_status are the same and the problem is that the commerce_adyen_get_transaction_instance() function has no chance to receive the $remote_status argument, since $remote_transaction_status not in use in \Commerce\Adyen\Payment\Modification::__construct().

CommentFileSizeAuthor
#2 2863695-2.patch594 bytesBR0kEN
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

BR0kEN created an issue. See original summary.

BR0kEN’s picture

Status: Active » Needs review
FileSize
594 bytes

  • BR0kEN committed d9af033 on 7.x-2.x
    Issue #2863695 by BR0kEN: Impossible to affect on transaction loading/...
BR0kEN’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.