We should use the order number for payment description and not the order ID, as the number is what Commerce itself shows to users. However, this may break BC. See if we can do this.

CommentFileSizeAuthor
#4 payment_commerce_2060873_4.patch1.08 KBxano
#3 payment_commerce_2060873_3.patch1.08 KBxano

Comments

dariogcode’s picture

dariogcode commented

Order number isn't the order id? May be I don't know about revision or some like that but in my system I only see order id.

xano’s picture

xano
he/they
Primary language English
Location Southampton
commented

The ID is the internal entity ID. The number is different in that it is only set once the order becomes final.

xano’s picture

xano
he/they
Primary language English
Location Southampton
commented
Status: Active » Needs review
StatusFileSize
new payment_commerce_2060873_3.patch1.08 KB
xano’s picture

xano
he/they
Primary language English
Location Southampton
commented
StatusFileSize
new payment_commerce_2060873_4.patch1.08 KB

Let's get rid of the hash/pound before the order number as well. It was only done in one of the two cases anyway.

xano’s picture

xano
he/they
Primary language English
Location Southampton
commented
Issue summary: View changes
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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