Closed ghost closed 3 years ago
@gett-thijssimonis awesome PR! Thanks a lot!
In the configuration page there is still the check for the module position in actionValidateOrder
, this is obsolete now.
If you like you can create another pull request for this. If not, I will remove it and commit this.
If you like you can create another pull request for this. If not, I will remove it and commit this
@muyncky see #6
With the current hooks the order reference is not changed when we creating a order payment transaction. If we change the order reference after the order is created with the
actionObjectOrderAddAfter
hook, the order payment transaction is also created with the new order reference.This hooks are not needed anymore and are unregistered in the upgrade file:
actionObjectOrderUpdateAfter
actionPaymentConfirmation
actionValidateOrder
actionOrderStatusUpdate
(not even used in the prev code)This also fix #2