Merge branch 'master' into update/14848

# Conflicts:
#	includes/class-wc-structured-data.php

assets/css/twenty-seventeen.scss

@@ -388,6 +388,10 @@ table.variations {
 			cursor: pointer;
 		}
 
+		li:nth-child(4n+1) {
+			clear: left;
+		}
+
 		img {
 			opacity: .5;
 

assets/css/woocommerce.scss

@@ -245,6 +245,10 @@ p.demo_store,
 						}
 					}
 				}
+
+				li:nth-child(4n+1) {
+					clear: left;
+				}
 			}
 		}
 

includes/abstracts/abstract-wc-order.php

@@ -853,7 +853,7 @@ abstract class WC_Abstract_Order extends WC_Abstract_Legacy_Order {
 		if ( $item_id = $item->get_id() ) {
 			$this->items[ $items_key ][ $item_id ] = $item;
 		} else {
-			$this->items[ $items_key ][ 'new:' . sizeof( $this->items[ $items_key ] ) ] = $item;
+			$this->items[ $items_key ][ 'new:' . $items_key . sizeof( $this->items[ $items_key ] ) ] = $item;
 		}
 	}
 

includes/api/legacy/v2/class-wc-api-orders.php

@@ -939,12 +939,17 @@ class WC_API_Orders extends WC_API_Resource {
 			$line_item->set_variation( $item['variations'] );
 		}
 
+		// Save or add to order.
+		if ( $creating ) {
+			$order->add_item( $line_item );
+		} else {
 			$item_id = $line_item->save();
 
 			if ( ! $item_id ) {
 				throw new WC_API_Exception( 'woocommerce_cannot_create_line_item', __( 'Cannot create line item, try again.', 'woocommerce' ), 500 );
 			}
 		}
+	}
 
 	/**
 	 * Given a product ID & API provided variations, find the correct variation ID to use for calculation
@@ -1023,11 +1028,7 @@ class WC_API_Orders extends WC_API_Resource {
 			$item = new WC_Order_Item_Shipping();
 			$item->set_order_id( $order->get_id() );
 			$item->set_shipping_rate( $rate );
-			$shipping_id = $item->save();
+			$order->add_item( $item );
-
-			if ( ! $shipping_id ) {
-				throw new WC_API_Exception( 'woocommerce_cannot_create_shipping', __( 'Cannot create shipping method, try again.', 'woocommerce' ), 500 );
-			}
 		} else {
 
 			$item = new WC_Order_Item_Shipping( $shipping['id'] );
@@ -1094,11 +1095,7 @@ class WC_API_Orders extends WC_API_Resource {
 				}
 			}
 
-			$fee_id = $item->save();
+			$order->add_item( $item );
-
-			if ( ! $fee_id ) {
-				throw new WC_API_Exception( 'woocommerce_cannot_create_fee', __( 'Cannot create fee, try again.', 'woocommerce' ), 500 );
-			}
 		} else {
 
 			$item = new WC_Order_Item_Fee( $fee['id'] );
@@ -1157,11 +1154,7 @@ class WC_API_Orders extends WC_API_Resource {
 				'discount_tax' => 0,
 				'order_id'     => $order->get_id(),
 			) );
-			$coupon_id = $item->save();
+			$order->add_item( $item );
-
-			if ( ! $coupon_id ) {
-				throw new WC_API_Exception( 'woocommerce_cannot_create_order_coupon', __( 'Cannot create coupon, try again.', 'woocommerce' ), 500 );
-			}
 		} else {
 
 			$item = new WC_Order_Item_Coupon( $coupon['id'] );

includes/api/legacy/v3/class-wc-api-orders.php

@@ -988,12 +988,17 @@ class WC_API_Orders extends WC_API_Resource {
 			$line_item->set_variation( $item['variations'] );
 		}
 
+		// Save or add to order.
+		if ( $creating ) {
+			$order->add_item( $line_item );
+		} else {
 			$item_id = $line_item->save();
 
 			if ( ! $item_id ) {
 				throw new WC_API_Exception( 'woocommerce_cannot_create_line_item', __( 'Cannot create line item, try again.', 'woocommerce' ), 500 );
 			}
 		}
+	}
 
 	/**
 	 * Given a product ID & API provided variations, find the correct variation ID to use for calculation
@@ -1073,11 +1078,7 @@ class WC_API_Orders extends WC_API_Resource {
 			$item = new WC_Order_Item_Shipping();
 			$item->set_order_id( $order->get_id() );
 			$item->set_shipping_rate( $rate );
-			$shipping_id = $item->save();
+			$order->add_item( $item );
-
-			if ( ! $shipping_id ) {
-				throw new WC_API_Exception( 'woocommerce_cannot_create_shipping', __( 'Cannot create shipping method, try again.', 'woocommerce' ), 500 );
-			}
 		} else {
 
 			$item = new WC_Order_Item_Shipping( $shipping['id'] );
@@ -1144,11 +1145,7 @@ class WC_API_Orders extends WC_API_Resource {
 				}
 			}
 
-			$fee_id = $item->save();
+			$order->add_item( $item );
-
-			if ( ! $fee_id ) {
-				throw new WC_API_Exception( 'woocommerce_cannot_create_fee', __( 'Cannot create fee, try again.', 'woocommerce' ), 500 );
-			}
 		} else {
 
 			$item = new WC_Order_Item_Fee( $fee['id'] );
@@ -1207,11 +1204,7 @@ class WC_API_Orders extends WC_API_Resource {
 				'discount_tax' => 0,
 				'order_id'     => $order->get_id(),
 			) );
-			$coupon_id = $item->save();
+			$order->add_item( $item );
-
-			if ( ! $coupon_id ) {
-				throw new WC_API_Exception( 'woocommerce_cannot_create_order_coupon', __( 'Cannot create coupon, try again.', 'woocommerce' ), 500 );
-			}
 		} else {
 
 			$item = new WC_Order_Item_Coupon( $coupon['id'] );

includes/class-wc-structured-data.php

@@ -200,6 +200,7 @@ class WC_Structured_Data {
 			return;
 		}
 
+		if ( '' !== $product->get_price() ) {
 			$markup_offer = array(
 				'@type'         => 'Offer',
 				'priceCurrency' => $currency,
@@ -230,8 +231,7 @@ class WC_Structured_Data {
 			} else {
 				$markup_offer['price'] = wc_format_decimal( $product->get_price(), wc_get_price_decimals() );
 			}
-
+		}
-		$markup['offers'] = array( apply_filters( 'woocommerce_structured_data_product_offer', $markup_offer, $product ) );
 
 		if ( $product->get_rating_count() ) {
 			$markup['aggregateRating'] = array(
@@ -402,7 +402,7 @@ class WC_Structured_Data {
 		$markup                       = array();
 		$markup['@type']              = 'Order';
 		$markup['url']                = $order_url;
-		$markup['orderStatus']        = isset( $order_status[ $order->get_status() ] ) ? $order_status[ $order->get_status() ] : '';
+		$markup['orderStatus']        = isset( $order_statuses[ $order->get_status() ] ) ? $order_statuses[ $order->get_status() ] : '';
 		$markup['orderNumber']        = $order->get_order_number();
 		$markup['orderDate']          = $order->get_date_created()->format( 'c' );
 		$markup['acceptedOffer']      = $markup_offers;

includes/data-stores/class-wc-product-data-store-cpt.php

@@ -997,51 +997,47 @@ class WC_Product_Data_Store_CPT extends WC_Data_Store_WP implements WC_Object_Da
 	public function get_related_products_query( $cats_array, $tags_array, $exclude_ids, $limit ) {
 		global $wpdb;
 
-		// Arrays to string.
+		$include_term_ids            = array_merge( $cats_array, $tags_array );
-		$exclude_ids = implode( ',', array_map( 'absint', $exclude_ids ) );
+		$exclude_term_ids            = array();
-		$cats_array  = implode( ',', array_map( 'absint', $cats_array ) );
-		$tags_array  = implode( ',', array_map( 'absint', $tags_array ) );
-
-		$limit           = absint( $limit );
-		$query           = array();
-		$query['fields'] = "SELECT DISTINCT ID FROM {$wpdb->posts} p";
-		$query['join']   = " INNER JOIN {$wpdb->term_relationships} tr ON (p.ID = tr.object_id)";
-		$query['join']  .= " INNER JOIN {$wpdb->term_taxonomy} tt ON (tr.term_taxonomy_id = tt.term_taxonomy_id)";
-		$query['join']  .= " INNER JOIN {$wpdb->terms} t ON (t.term_id = tt.term_id)";
-		$query['where']  = ' WHERE 1=1';
-		$query['where'] .= " AND p.post_status = 'publish'";
-		$query['where'] .= " AND p.post_type = 'product'";
-		$query['where'] .= " AND p.ID NOT IN ( {$exclude_ids} )";
-
 		$product_visibility_term_ids = wc_get_product_visibility_term_ids();
 
 		if ( $product_visibility_term_ids['exclude-from-catalog'] ) {
-			$query['where'] .= " AND t.term_id !=" . $product_visibility_term_ids['exclude-from-catalog'];
+			$exclude_term_ids[] = $product_visibility_term_ids['exclude-from-catalog'];
 		}
 
 		if ( 'yes' === get_option( 'woocommerce_hide_out_of_stock_items' ) && $product_visibility_term_ids['outofstock'] ) {
-			$query['where'] .= " AND t.term_id !=" . $product_visibility_term_ids['outofstock'];
+			$exclude_term_ids[] = $product_visibility_term_ids['outofstock'];
 		}
 
-		if ( $cats_array || $tags_array ) {
+		$query = array(
-			$query['where'] .= ' AND (';
+			'fields' => "
+				SELECT DISTINCT ID FROM {$wpdb->posts} p
+			",
+			'join'   => '',
+			'where'  => "
+				WHERE 1=1
+				AND p.post_status = 'publish'
+				AND p.post_type = 'product'
 
-			if ( $cats_array ) {
+			",
-				$query['where'] .= " ( tt.taxonomy = 'product_cat' AND t.term_id IN ( {$cats_array} ) ) ";
+			'limits' => "
-				if ( $tags_array ) {
+				LIMIT " . absint( $limit ) . "
-					$query['where'] .= ' OR ';
+			",
-				}
+		);
+
+		if ( $exclude_term_ids ) {
+			$query['join']  .= " LEFT JOIN ( SELECT object_id FROM {$wpdb->term_relationships} WHERE term_taxonomy_id IN ( " . implode( ',', array_map( 'absint', $exclude_term_ids ) ) . " ) ) AS exclude_join ON exclude_join.object_id = p.ID";
+			$query['where'] .= " AND exclude_join.object_id IS NULL";
 		}
 
-			if ( $tags_array ) {
+		if ( $include_term_ids ) {
-				$query['where'] .= " ( tt.taxonomy = 'product_tag' AND t.term_id IN ( {$tags_array} ) ) ";
+			$query['join']  .= " INNER JOIN ( SELECT object_id FROM {$wpdb->term_relationships} LEFT JOIN wp_term_taxonomy AS tax using( term_taxonomy_id ) WHERE term_id IN ( " . implode( ',', array_map( 'absint', $include_term_ids ) ) . " ) ) AS include_join ON include_join.object_id = p.ID";
 		}
 
-			$query['where'] .= ')';
+		if ( $exclude_ids ) {
+			$query['where'] .= " AND p.ID NOT IN ( " . implode( ',', array_map( 'absint', $exclude_ids ) ) . " )";
 		}
 
-		$query['limits'] = " LIMIT {$limit} ";
-
 		return $query;
 	}
 

includes/emails/class-wc-email.php

@@ -280,7 +280,7 @@ class WC_Email extends WC_Settings_API {
 	public function get_headers() {
 		$header = "Content-Type: " . $this->get_content_type() . "\r\n";
 
-		if ( 'new_order' === $this->id ) {
+		if ( 'new_order' === $this->id && $this->object && $this->object->get_billing_email() && ( $this->object->get_billing_first_name() || $this->object->get_billing_last_name() ) ) {
 			$header .= 'Reply-to: ' . $this->object->get_billing_first_name() . ' ' . $this->object->get_billing_last_name() . ' <' . $this->object->get_billing_email() . ">\r\n";
 		}
 

includes/wc-product-functions.php

@@ -707,7 +707,7 @@ function wc_get_product_attachment_props( $attachment_id = null, $product = fals
 		$props['alt']     = trim( strip_tags( get_post_meta( $attachment_id, '_wp_attachment_image_alt', true ) ) );
 
 		// Large version.
-		$src                 = wp_get_attachment_image_src( $attachment_id, 'large' );
+		$src                 = wp_get_attachment_image_src( $attachment_id, 'full' );
 		$props['full_src']   = $src[0];
 		$props['full_src_w'] = $src[1];
 		$props['full_src_h'] = $src[2];

includes/wc-template-functions.php

@@ -106,6 +106,18 @@ function wc_prevent_adjacent_posts_rel_link_wp_head() {
 }
 add_action( 'template_redirect', 'wc_prevent_adjacent_posts_rel_link_wp_head' );
 
+/**
+ * Show the gallery if JS is disabled.
+ *
+ * @since 3.0.6
+ */
+function wc_gallery_noscript() {
+	?>
+	<noscript><style>.woocommerce-product-gallery{ opacity: 1 !important; }</style></noscript>
+	<?php
+}
+add_action( 'wp_head', 'wc_gallery_noscript' );
+
 /**
  * When the_post is called, put product data into a global.
  *

templates/single-product/add-to-cart/grouped.php

@@ -13,7 +13,7 @@
  * @see 	    https://docs.woocommerce.com/document/template-structure/
  * @author 		WooThemes
  * @package 	WooCommerce/Templates
- * @version     3.0.3
+ * @version     3.0.6
  */
 if ( ! defined( 'ABSPATH' ) ) {
 	exit;
@@ -28,12 +28,13 @@ do_action( 'woocommerce_before_add_to_cart_form' ); ?>
 		<tbody>
 			<?php
 				$quantites_required = false;
+				$previous_post      = $post;
 
 				foreach ( $grouped_products as $grouped_product ) {
 					$post_object        = get_post( $grouped_product->get_id() );
 					$quantites_required = $quantites_required || ( $grouped_product->is_purchasable() && ! $grouped_product->has_options() );
 
-					setup_postdata( $GLOBALS['post'] =& $post_object );
+					setup_postdata( $post =& $post_object );
 					?>
 					<tr id="product-<?php the_ID(); ?>" <?php post_class(); ?>>
 						<td>
@@ -79,7 +80,7 @@ do_action( 'woocommerce_before_add_to_cart_form' ); ?>
 					</tr>
 					<?php
 				}
-				wp_reset_postdata();
+				$post = $previous_post;
 			?>
 		</tbody>
 	</table>

tests/unit-tests/order/crud.php

@@ -273,6 +273,29 @@ class WC_Tests_CRUD_Orders extends WC_Unit_Test_Case {
 		$this->assertCount( 2, $object->get_items() );
 	}
 	
+	/**
+	 * Test: get_different_items
+	 */
+	function test_get_different_items() {
+		$object  = new WC_Order();
+		$item_1  = new WC_Order_Item_Product();
+		$item_1->set_props( array(
+			'product'  => WC_Helper_Product::create_simple_product(),
+			'quantity' => 4,
+		) );
+		$item_2  = new WC_Order_Item_Fee();
+		$item_2->set_props( array(
+			'name'       => 'Some Fee',
+			'tax_status' => 'taxable',
+			'total'      => '100',
+			'tax_class'  => '',
+		) );
+		$object->add_item( $item_1 );
+		$object->add_item( $item_2 );
+		// $object->save();
+		$this->assertCount( 2, $object->get_items( array( 'line_item', 'fee' ) ) );
+	}
+
 	/**
 	 * Test: get_fees
 	 */