← All changes
|
modules/woocommerce-analytics/classes/class-jetpack-woocommerce-analytics-universal.php
+28
-25
13.4.5
→
16.3-a.7
View file →
| @@ -11,9 +11,9 @@ | ||
| 11 | 11 | /** |
| 12 | 12 | * Bail if accessed directly |
| 13 | 13 | */ |
| 14 | 14 | if ( ! defined( 'ABSPATH' ) ) { |
| 15 | - exit; | |
| 15 | + exit( 0 ); | |
| 16 | 16 | } |
| 17 | 17 | |
| 18 | 18 | /** |
| 19 | 19 | * Class Jetpack_WooCommerce_Analytics_Universal |
| @@ -94,10 +94,16 @@ | ||
| 94 | 94 | * |
| 95 | 95 | * @deprecated 13.3 |
| 96 | 96 | */ |
| 97 | 97 | public function remove_from_cart() { |
| 98 | - $common_props = $this->render_properties_as_js( | |
| 99 | - $this->get_common_properties() | |
| 98 | + $common_props = wp_json_encode( | |
| 99 | + array_merge( | |
| 100 | + array( | |
| 101 | + '_en' => 'woocommerceanalytics_remove_from_cart', | |
| 102 | + ), | |
| 103 | + $this->get_common_properties() | |
| 104 | + ), | |
| 105 | + JSON_UNESCAPED_SLASHES | JSON_HEX_TAG | JSON_HEX_AMP | |
| 100 | 106 | ); |
| 101 | 107 | |
| 102 | 108 | // We listen at div.woocommerce because the cart 'form' contents get forcibly |
| 103 | 109 | // updated and subsequent removals from cart would then not have this click |
| @@ -105,19 +111,13 @@ | ||
| 105 | 111 | wc_enqueue_js( |
| 106 | 112 | "jQuery( 'div.woocommerce' ).on( 'click', 'a.remove', function() { |
| 107 | 113 | var productID = jQuery( this ).data( 'product_id' ); |
| 108 | 114 | var quantity = jQuery( this ).parent().parent().find( '.qty' ).val() |
| 109 | - var productDetails = { | |
| 110 | - 'id': productID, | |
| 111 | - 'quantity': quantity ? quantity : '1', | |
| 112 | - }; | |
| 113 | - _wca.push( { | |
| 114 | - '_en': 'woocommerceanalytics_remove_from_cart', | |
| 115 | - 'pi': productDetails.id, | |
| 116 | - 'pq': productDetails.quantity, " . | |
| 117 | - $common_props . ' | |
| 118 | - } ); | |
| 119 | - } );' | |
| 115 | + var common_props = $common_props; | |
| 116 | + common_props.pi = productID; | |
| 117 | + common_props.pq = quantity ? quantity : '1'; | |
| 118 | + _wca.push( common_props ); | |
| 119 | + } );" | |
| 120 | 120 | ); |
| 121 | 121 | } |
| 122 | 122 | |
| 123 | 123 | /** |
| @@ -353,17 +353,16 @@ | ||
| 353 | 353 | $checkout_page_contains_checkout_shortcode = '0'; |
| 354 | 354 | |
| 355 | 355 | $order_source = $order->get_created_via(); |
| 356 | 356 | if ( 'store-api' === $order_source ) { |
| 357 | - $checkout_page_contains_checkout_block = '1'; | |
| 358 | - $checkout_page_contains_checkout_shortcode = '0'; | |
| 357 | + $checkout_page_contains_checkout_block = '1'; | |
| 359 | 358 | } elseif ( 'checkout' === $order_source ) { |
| 360 | - $checkout_page_contains_checkout_block = '0'; | |
| 361 | 359 | $checkout_page_contains_checkout_shortcode = '1'; |
| 362 | 360 | } |
| 363 | 361 | |
| 364 | 362 | // loop through products in the order and queue a purchase event. |
| 365 | 363 | foreach ( $order->get_items() as $order_item ) { |
| 364 | + // @phan-suppress-next-line PhanUndeclaredMethod -- Checked before being called. See also https://github.com/phan/phan/issues/1204. | |
| 366 | 365 | $product_id = is_callable( array( $order_item, 'get_product_id' ) ) ? $order_item->get_product_id() : -1; |
| 367 | 366 | |
| 368 | 367 | $order_items = $order->get_items(); |
| 369 | 368 | $order_items_count = 0; |
| @@ -402,10 +401,16 @@ | ||
| 402 | 401 | * |
| 403 | 402 | * @deprecated 13.3 |
| 404 | 403 | */ |
| 405 | 404 | public function remove_from_cart_via_quantity() { |
| 406 | - $common_props = $this->render_properties_as_js( | |
| 407 | - $this->get_common_properties() | |
| 405 | + $common_props = wp_json_encode( | |
| 406 | + array_merge( | |
| 407 | + array( | |
| 408 | + '_en' => 'woocommerceanalytics_remove_from_cart', | |
| 409 | + ), | |
| 410 | + $this->get_common_properties() | |
| 411 | + ), | |
| 412 | + JSON_UNESCAPED_SLASHES | JSON_HEX_TAG | JSON_HEX_AMP | |
| 408 | 413 | ); |
| 409 | 414 | |
| 410 | 415 | wc_enqueue_js( |
| 411 | 416 | " |
| @@ -414,16 +419,14 @@ | ||
| 414 | 419 | cartItems.each( function( item ) { |
| 415 | 420 | var qty = jQuery( this ).find( 'input.qty' ); |
| 416 | 421 | if ( qty && qty.val() === '0' ) { |
| 417 | 422 | var productID = jQuery( this ).find( '.product-remove a' ).data( 'product_id' ); |
| 418 | - _wca.push( { | |
| 419 | - '_en': 'woocommerceanalytics_remove_from_cart', | |
| 420 | - 'pi': productID, " . | |
| 421 | - $common_props . ' | |
| 422 | - } ); | |
| 423 | + var common_props = $common_props; | |
| 424 | + common_props.pi = productID; | |
| 425 | + _wca.push( common_props ); | |
| 423 | 426 | } |
| 424 | 427 | } ); |
| 425 | - } );' | |
| 428 | + } );" | |
| 426 | 429 | ); |
| 427 | 430 | } |
| 428 | 431 | |
| 429 | 432 | /** |