Commerce order entities have a field "order_number" which is described as "The order number displayed to the customer". Currently, this module is sending the Order ID to Google analytics. However, this Order ID may be of little use to administrators/analysts if the order number is being generated separately from the ID. We should be using the 'getOrderNumber()' method rather than 'id()'.

CommentFileSizeAuthor
#2 send-order-number-3008291.patch1.01 KBmangy.fox
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mangy.fox created an issue. See original summary.

mangy.fox’s picture

mangy.fox CreditAttribution: mangy.fox at Investis Digital commented
Status: Active » Needs review
FileSize
send-order-number-3008291.patch1.01 KB

Attached patch sends order number rather than order ID.

Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir at MD Systems GmbH commented
Status: Needs review » Fixed

Makes sense, committed.

  • Berdir committed 45e6aef on 8.x-1.x authored by mangy.fox 
    Issue #3008291 by mangy.fox: Analytics should use Order Number not Order...

Status: Fixed » Closed (fixed)

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