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

Merge pull request #7201 from thenbrent/fix_wc_product__isset 

[2.3] Do not set WC_Product property when calling __get()
This commit is contained in:
Mike Jolley 2015-01-26 11:38:21 +00:00
parent 07c863e6dd 95d87e28d6
commit 2a95402d22
1 changed files with 12 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
includes/abstracts/abstract-wc-product.php
View File

@ -91,30 +91,34 @@ class WC_Product {
* @return mixed
*/
public function __get( $key ) {
$this->$key = get_post_meta( $this->id, '_' . $key, true );
$value = get_post_meta( $this->id, '_' . $key, true );
// Get values or default if not set
if ( in_array( $key, array( 'downloadable', 'virtual', 'backorders', 'manage_stock', 'featured', 'sold_individually' ) ) ) {
$this->$key = $this->$key ? $this->$key : 'no';
$value = $value ? $value : 'no';
} elseif ( in_array( $key, array( 'product_attributes', 'crosssell_ids', 'upsell_ids' ) ) ) {
$this->$key = $this->$key ? $this->$key : array();
$value = $value ? $value : array();
} elseif ( 'visibility' === $key ) {
$this->$key = $this->$key ? $this->$key : 'hidden';
$value = $value ? $value : 'hidden';
} elseif ( 'stock' === $key ) {
$this->$key = $this->$key ? $this->$key : 0;
$value = $value ? $value : 0;
} elseif ( 'stock_status' === $key ) {
$this->$key = $this->$key ? $this->$key : 'instock';
$value = $value ? $value : 'instock';
} elseif ( 'tax_status' === $key ) {
$this->$key = $this->$key ? $this->$key : 'taxable';
$value = $value ? $value : 'taxable';
}
return $this->$key;
if ( ! empty( $value ) ) {
$this->$key = $value;
}
return $value;
}
/**