Why is getting transaction id from GET params commented in order received?

  • Resolved nejcr

    (@nejcr)


    4 years, 11 months ago

    Hello,

    is there a reason you decided to comment lines in file /inc/main.inc.php/ from 2736:2740?

    Because we are using preauthorization, the transaction id is not set in the $order object, so the value is not getting in the SecureModeReturnURL in function create_preauthorization and is throwing fatal error on page /order-received/ (because it has no transaction id in URL).

    When I uncommented the mentioned lines, it works fine. But I would not like to uncomment lines every time we update the plugin.

    Thank you!

    • This topic was modified 4 years, 11 months ago by nejcr.
Viewing 3 replies - 1 through 3 (of 3 total)
  • Plugin Author Yann at WP&Co

    (@ydubois)

    Hi @nejcr

    Could you please confirm that you are using the latest version of the plugin (ie version 3.2.1)?

    Because if this is the case, what you are reporting is very odd: the commented code at lines 2736:2740 is present just above at lines 2726:2728.

    So I don’t see how uncommenting the same code after the returned error would change anything.

    However we will investigate into the reason why the transaction_id is not present or causing an error.

    Thread Starter nejcr

    (@nejcr)

    Hello @yann, thank you for the response. Yes I am using latest release. Hmm you are right, it seems that it went into first if on 2726 and skip the if on the 2731, because it did not compare okay. But now I commented the block of code again and it seems to be working… Strange. Thanks for your help. You can delete the thread. Will open new one If it happens again.

    Plugin Author Yann at WP&Co

    (@ydubois)

    @nejcr with further investigation we determined that there was indeed a bug crashing the order-received page after transaction processing when dealing with a card pre-authorization.

    Please update to the just released latest version 3.2.3 to prevent this bug.

    Thank you again for your report.

Viewing 3 replies - 1 through 3 (of 3 total)
  • The topic ‘Why is getting transaction id from GET params commented in order received?’ is closed to new replies.