diff --git a/commerce_vipps.inc b/commerce_vipps.inc
index cbdbe79..a68e7d6 100644
--- a/commerce_vipps.inc
+++ b/commerce_vipps.inc
@@ -238,7 +238,7 @@ function _commerce_vipps_get_current_transaction($order, $payment_method) {
       $transaction->remote_id,
       (int) $mobile,
       (int) $transaction->amount,
-      t('Payment for order @order_id', array('@order_id' => $wrapper->getIdentifier())),
+      t('Payment for order @order_id', array('@order_id' => $transaction->order_id)),
       url('commerce_vipps/' . $transaction->transaction_id . '/transaction/' . $order->data['payment_redirect_key'], array('absolute' => TRUE, 'https' => TRUE))
     );
CommentFileSizeAuthor
#2 3032079-fix-id.patch667 bytesfarald

Comments

erlendstromsvik created an issue. See original summary.

farald’s picture

Status: Active » Needs review
StatusFileSize
new667 bytes

Patch attached.

  • farald authored 2b254ba on 7.x-2.x
    Issue #3032079 by farald, erlendstromsvik: Wrong ID sent as order ID in...
zaporylie’s picture

Status: Needs review » Fixed

Committed and pushed to 7.x-2.x.

Made me wonder - should we actually use Order ID rather than Order Number? IIRC order number is not available before the order is completed, which happens after the payment is fully registered, but this probably should be based on the research, not me remembering stuff :)

Status: Fixed » Closed (fixed)

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