This is cloned issue from Commerce Payment Suomen Verkkomaksut -module.

Please see https://www.drupal.org/node/2402753

Patch created to commerce_paytrail module.

CommentFileSizeAuthor
#4 commerce_paytrail-notifications_handler-2552353-4.patch8.18 KBtuutti
#3 commerce_paytrail-notifications_handler-2552353--3.patch8.2 KBtuutti
#3 interdiff-2552353-2--3.txt7.12 KBtuutti
#2 interdiff.txt660 bytesmitrpaka
#2 commerce_paytrail-notifications_handler-2552353-2.patch5.73 KBmitrpaka
commerce_paytrail-notification-handler.patch5.69 KBmitrpaka

Comments

mitrpaka created an issue. See original summary.

mitrpaka’s picture

mitrpaka commented
StatusFileSize
new commerce_paytrail-notifications_handler-2552353-2.patch5.73 KB
new interdiff.txt660 bytes

Removed duplication of constant.

tuutti’s picture

tuutti commented
StatusFileSize
new interdiff-2552353-2--3.txt7.12 KB
new commerce_paytrail-notifications_handler-2552353--3.patch8.2 KB

I added some missing validations to notification callback, removed duplicate code and changed commerce_paytrail_notify_save() to match the previous transaction logic.

tuutti’s picture

tuutti commented
StatusFileSize
new commerce_paytrail-notifications_handler-2552353-4.patch8.18 KB

Looks like this does not apply anymore.

  • tuutti committed 88d3493 on 7.x-1.x 
    Issue #2552353 by mitrpaka, tuutti: Implement notification handler [...
tuutti’s picture

tuutti commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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