Updated WC_Product_Variable::get_available_variation() to handle cases when variation ID provided is deleted/invalid. Throwing PHP fatal error on the next line where $variation->get_price()
This commit is contained in:
Amritansh_tact9
parent
c2519e9d7d
commit
da5e31ce2c
|
|
@ -8,10 +8,10 @@ if ( ! defined( 'ABSPATH' ) ) {
|
|||
*
|
||||
* The WooCommerce product class handles individual product data.
|
||||
*
|
||||
* @version 3.0.0
|
||||
* @package WooCommerce/Classes/Products
|
||||
* @category Class
|
||||
* @author WooThemes
|
||||
* @version 3.0.0
|
||||
* @package WooCommerce/Classes/Products
|
||||
* @category Class
|
||||
* @author WooThemes
|
||||
*/
|
||||
class WC_Product_Variable extends WC_Product {
|
||||
|
||||
|
|
@ -74,7 +74,8 @@ class WC_Product_Variable extends WC_Product {
|
|||
/**
|
||||
* Get an array of all sale and regular prices from all variations. This is used for example when displaying the price range at variable product level or seeing if the variable product is on sale.
|
||||
*
|
||||
* @param bool $for_display If true, prices will be adapted for display based on the `woocommerce_tax_display_shop` setting (including or excluding taxes).
|
||||
* @param bool $for_display If true, prices will be adapted for display based on the `woocommerce_tax_display_shop` setting (including or excluding taxes).
|
||||
*
|
||||
* @return array Array of RAW prices, regular prices, and sale prices with keys set to variation ID.
|
||||
*/
|
||||
public function get_variation_prices( $for_display = false ) {
|
||||
|
|
@ -90,39 +91,45 @@ class WC_Product_Variable extends WC_Product {
|
|||
/**
|
||||
* Get the min or max variation regular price.
|
||||
*
|
||||
* @param string $min_or_max Min or max price.
|
||||
* @param string $min_or_max Min or max price.
|
||||
* @param boolean $for_display If true, prices will be adapted for display based on the `woocommerce_tax_display_shop` setting (including or excluding taxes).
|
||||
*
|
||||
* @return string
|
||||
*/
|
||||
public function get_variation_regular_price( $min_or_max = 'min', $for_display = false ) {
|
||||
$prices = $this->get_variation_prices( $for_display );
|
||||
$price = 'min' === $min_or_max ? current( $prices['regular_price'] ) : end( $prices['regular_price'] );
|
||||
|
||||
return apply_filters( 'woocommerce_get_variation_regular_price', $price, $this, $min_or_max, $for_display );
|
||||
}
|
||||
|
||||
/**
|
||||
* Get the min or max variation sale price.
|
||||
*
|
||||
* @param string $min_or_max Min or max price.
|
||||
* @param string $min_or_max Min or max price.
|
||||
* @param boolean $for_display If true, prices will be adapted for display based on the `woocommerce_tax_display_shop` setting (including or excluding taxes).
|
||||
*
|
||||
* @return string
|
||||
*/
|
||||
public function get_variation_sale_price( $min_or_max = 'min', $for_display = false ) {
|
||||
$prices = $this->get_variation_prices( $for_display );
|
||||
$price = 'min' === $min_or_max ? current( $prices['sale_price'] ) : end( $prices['sale_price'] );
|
||||
|
||||
return apply_filters( 'woocommerce_get_variation_sale_price', $price, $this, $min_or_max, $for_display );
|
||||
}
|
||||
|
||||
/**
|
||||
* Get the min or max variation (active) price.
|
||||
*
|
||||
* @param string $min_or_max Min or max price.
|
||||
* @param string $min_or_max Min or max price.
|
||||
* @param boolean $for_display If true, prices will be adapted for display based on the `woocommerce_tax_display_shop` setting (including or excluding taxes).
|
||||
*
|
||||
* @return string
|
||||
*/
|
||||
public function get_variation_price( $min_or_max = 'min', $for_display = false ) {
|
||||
$prices = $this->get_variation_prices( $for_display );
|
||||
$price = 'min' === $min_or_max ? current( $prices['price'] ) : end( $prices['price'] );
|
||||
|
||||
return apply_filters( 'woocommerce_get_variation_price', $price, $this, $min_or_max, $for_display );
|
||||
}
|
||||
|
||||
|
|
@ -140,6 +147,7 @@ class WC_Product_Variable extends WC_Product {
|
|||
* is shown instead.
|
||||
*
|
||||
* @param string $price (default: '')
|
||||
*
|
||||
* @return string
|
||||
*/
|
||||
public function get_price_html( $price = '' ) {
|
||||
|
|
@ -174,8 +182,9 @@ class WC_Product_Variable extends WC_Product {
|
|||
* has dynamic values such as {price_excluding_tax} for variable products.
|
||||
* @see get_price_html for an explanation as to why.
|
||||
*
|
||||
* @param string $price to calculate, left blank to just use get_price()
|
||||
* @param integer $qty passed on to get_price_including_tax() or get_price_excluding_tax()
|
||||
* @param string $price to calculate, left blank to just use get_price()
|
||||
* @param integer $qty passed on to get_price_including_tax() or get_price_excluding_tax()
|
||||
*
|
||||
* @return string
|
||||
*/
|
||||
public function get_price_suffix( $price = '', $qty = 1 ) {
|
||||
|
|
@ -198,8 +207,10 @@ class WC_Product_Variable extends WC_Product {
|
|||
public function get_children( $visible_only = '' ) {
|
||||
if ( is_bool( $visible_only ) ) {
|
||||
wc_deprecated_argument( 'visible_only', '3.0', 'WC_Product_Variable::get_visible_children' );
|
||||
|
||||
return $visible_only ? $this->get_visible_children() : $this->get_children();
|
||||
}
|
||||
|
||||
return apply_filters( 'woocommerce_get_children', $this->children, $this, false );
|
||||
}
|
||||
|
||||
|
|
@ -226,11 +237,13 @@ class WC_Product_Variable extends WC_Product {
|
|||
* If set, get the default attributes for a variable product.
|
||||
*
|
||||
* @param string $attribute_name
|
||||
*
|
||||
* @return string
|
||||
*/
|
||||
public function get_variation_default_attribute( $attribute_name ) {
|
||||
$defaults = $this->get_default_attributes();
|
||||
$attribute_name = sanitize_title( $attribute_name );
|
||||
|
||||
return isset( $defaults[ $attribute_name ] ) ? $defaults[ $attribute_name ] : '';
|
||||
}
|
||||
|
||||
|
|
@ -286,14 +299,18 @@ class WC_Product_Variable extends WC_Product {
|
|||
/**
|
||||
* Returns an array of data for a variation. Used in the add to cart form.
|
||||
* @since 2.4.0
|
||||
*
|
||||
* @param WC_Product $variation Variation product object or ID
|
||||
*
|
||||
* @return array
|
||||
*/
|
||||
public function get_available_variation( $variation ) {
|
||||
if ( is_numeric( $variation ) ) {
|
||||
$variation = wc_get_product( $variation );
|
||||
}
|
||||
|
||||
if ( ! $variation instanceof WC_Product ) {
|
||||
return false;
|
||||
}
|
||||
// See if prices should be shown for each variation after selection.
|
||||
$show_variation_price = apply_filters( 'woocommerce_show_variation_price', $variation->get_price() === "" || $this->get_variation_sale_price( 'min' ) !== $this->get_variation_sale_price( 'max' ) || $this->get_variation_regular_price( 'min' ) !== $this->get_variation_regular_price( 'max' ), $this, $variation );
|
||||
|
||||
|
|
@ -335,6 +352,7 @@ class WC_Product_Variable extends WC_Product {
|
|||
* Sets an array of variation attributes.
|
||||
*
|
||||
* @since 3.0.0
|
||||
*
|
||||
* @param array
|
||||
*/
|
||||
public function set_variation_attributes( $variation_attributes ) {
|
||||
|
|
@ -345,6 +363,7 @@ class WC_Product_Variable extends WC_Product {
|
|||
* Sets an array of children for the product.
|
||||
*
|
||||
* @since 3.0.0
|
||||
*
|
||||
* @param array
|
||||
*/
|
||||
public function set_children( $children ) {
|
||||
|
|
@ -355,6 +374,7 @@ class WC_Product_Variable extends WC_Product {
|
|||
* Sets an array of visible children only.
|
||||
*
|
||||
* @since 3.0.0
|
||||
*
|
||||
* @param array
|
||||
*/
|
||||
public function set_visible_children( $visible_children ) {
|
||||
|
|
@ -434,6 +454,7 @@ class WC_Product_Variable extends WC_Product {
|
|||
* Returns whether or not the product is on sale.
|
||||
*
|
||||
* @param string $context What the value is for. Valid values are view and edit.
|
||||
*
|
||||
* @return bool
|
||||
*/
|
||||
public function is_on_sale( $context = 'view' ) {
|
||||
|
|
@ -473,6 +494,7 @@ class WC_Product_Variable extends WC_Product {
|
|||
$has_weight = $this->data_store->child_has_weight( $this );
|
||||
set_transient( $transient_name, $has_weight, DAY_IN_SECONDS * 30 );
|
||||
}
|
||||
|
||||
return (bool) $has_weight;
|
||||
}
|
||||
|
||||
|
|
@ -488,6 +510,7 @@ class WC_Product_Variable extends WC_Product {
|
|||
$has_dimension = $this->data_store->child_has_dimensions( $this );
|
||||
set_transient( $transient_name, $has_dimension, DAY_IN_SECONDS * 30 );
|
||||
}
|
||||
|
||||
return (bool) $has_dimension;
|
||||
}
|
||||
|
||||
|
|
@ -532,6 +555,7 @@ class WC_Product_Variable extends WC_Product {
|
|||
*
|
||||
* @param WC_Product|int $product Product object or ID for which you wish to sync.
|
||||
* @param bool $save If true, the product object will be saved to the DB before returning it.
|
||||
*
|
||||
* @return WC_Product Synced product object.
|
||||
*/
|
||||
public static function sync( $product, $save = true ) {
|
||||
|
|
@ -550,8 +574,12 @@ class WC_Product_Variable extends WC_Product {
|
|||
$product->save();
|
||||
}
|
||||
|
||||
wc_do_deprecated_action( 'woocommerce_variable_product_sync', array( $product->get_id(), $product->get_visible_children() ), '3.0', 'woocommerce_variable_product_sync_data, woocommerce_new_product or woocommerce_update_product' );
|
||||
wc_do_deprecated_action( 'woocommerce_variable_product_sync', array(
|
||||
$product->get_id(),
|
||||
$product->get_visible_children()
|
||||
), '3.0', 'woocommerce_variable_product_sync_data, woocommerce_new_product or woocommerce_update_product' );
|
||||
}
|
||||
|
||||
return $product;
|
||||
}
|
||||
|
||||
|
|
@ -560,6 +588,7 @@ class WC_Product_Variable extends WC_Product {
|
|||
*
|
||||
* @param WC_Product|int $product Product object or ID for which you wish to sync.
|
||||
* @param bool $save If true, the product object will be saved to the DB before returning it.
|
||||
*
|
||||
* @return WC_Product Synced product object.
|
||||
*/
|
||||
public static function sync_stock_status( $product, $save = true ) {
|
||||
|
|
@ -574,6 +603,7 @@ class WC_Product_Variable extends WC_Product {
|
|||
$product->save();
|
||||
}
|
||||
}
|
||||
|
||||
return $product;
|
||||
}
|
||||
|
||||
|
|
@ -581,10 +611,12 @@ class WC_Product_Variable extends WC_Product {
|
|||
* Sort an associativate array of $variation_id => $price pairs in order of min and max prices.
|
||||
*
|
||||
* @param array $prices Associativate array of $variation_id => $price pairs
|
||||
*
|
||||
* @return array
|
||||
*/
|
||||
protected function sort_variation_prices( $prices ) {
|
||||
asort( $prices );
|
||||
|
||||
return $prices;
|
||||
}
|
||||
}
|
||||
|
|
|
|||
Loading…
Reference in New Issue