Steve-Dee-Designs
/
woocommerce
mirror of https://github.com/woocommerce/woocommerce.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Remove Customer Order Count from Order Attribution checkout data (#44359) 

* Remove Customer Order Count from Checkout Tracks data
This commit is contained in:
Justin Palmer 2024-02-05 16:06:49 +01:00 committed by GitHub
parent a943de24fa
commit cd7f429bc1
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 18 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
plugins/woocommerce/changelog/tweak-remove-order-attribution-tracks-cust-order-count Normal file
View File

@ -0,0 +1,4 @@
Significance: patch
Type: performance
Remove customer order count from order attribution checkout data.

30
plugins/woocommerce/src/Internal/Orders/OrderAttributionController.php
View File

@ -416,27 +416,25 @@ class OrderAttributionController implements RegisterHooksInterface {
* @return void * @return void
*/ */
private function send_order_tracks( array $source_data, WC_Order $order ) { private function send_order_tracks( array $source_data, WC_Order $order ) {
$origin_label = $this->get_origin_label( $origin_label = $this->get_origin_label(
$source_data['source_type'] ?? '', $source_data['source_type'] ?? '',
$source_data['utm_source'] ?? '', $source_data['utm_source'] ?? '',
false false
); );
$customer_identifier = $order->get_customer_id() ? $order->get_customer_id() : $order->get_billing_email();
$customer_info = $this->get_customer_history( $customer_identifier ); $tracks_data = array(
$tracks_data = array( 'order_id' => $order->get_id(),
'order_id' => $order->get_id(), 'source_type' => $source_data['source_type'] ?? '',
'source_type' => $source_data['source_type'] ?? '', 'medium' => $source_data['utm_medium'] ?? '',
'medium' => $source_data['utm_medium'] ?? '', 'source' => $source_data['utm_source'] ?? '',
'source' => $source_data['utm_source'] ?? '', 'device_type' => strtolower( $source_data['device_type'] ?? 'unknown' ),
'device_type' => strtolower( $source_data['device_type'] ?? 'unknown' ), 'origin_label' => strtolower( $origin_label ),
'origin_label' => strtolower( $origin_label ), 'session_pages' => $source_data['session_pages'] ?? 0,
'session_pages' => $source_data['session_pages'] ?? 0, 'session_count' => $source_data['session_count'] ?? 0,
'session_count' => $source_data['session_count'] ?? 0, 'order_total' => $order->get_total(),
'order_total' => $order->get_total(), 'customer_registered' => $order->get_customer_id() ? 'yes' : 'no',
// Add 1 to include the current order (which is currently still Pending when the event is sent).
'customer_order_count' => $customer_info['order_count'] + 1,
'customer_registered' => $order->get_customer_id() ? 'yes' : 'no',
); );
$this->proxy->call_static( WC_Tracks::class, 'record_event', 'order_attribution', $tracks_data ); $this->proxy->call_static( WC_Tracks::class, 'record_event', 'order_attribution', $tracks_data );
} }