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

[Product CRUD] Getter setter proxy methods (#12236) 

* Started on variation changes

* Stock functions

* Variation class

* Bulk change ->id to get_id() to fix variation form display

* Missing status

* Fix add to cart

* Start on stored data save

* save variation

* Save_variations

* Variation edit panel

* Save variations code works.

* Remove stored data code and fix save

* Improve legacy class

* wc_bool_to_string

* prepare_set_attributes

* Use wc_get_products

* More feedback fixes

* get_prop implementation in abstract and data classes

* Implement set_prop

* Change handling

* Array key exists

* set_object_read
This commit is contained in:
Mike Jolley 2016-11-03 12:03:19 +00:00
parent 9072b1da43
commit 31e5882bb8
8 changed files with 311 additions and 246 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
includes/abstracts/abstract-wc-data.php
View File

@ -465,7 +465,7 @@ abstract class WC_Data {
* @param array $props Key value pairs to set. Key is the prop and should map to a setter function name.
* @return WP_Error|bool
*/
public function set_props( $props ) {
public function set_props( $props, $context = 'set' ) {
$errors = new WP_Error();
foreach ( $props as $prop => $value ) {

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

File diff suppressed because it is too large Load Diff

50
includes/class-wc-product-external.php
View File

@ -35,6 +35,14 @@ class WC_Product_External extends WC_Product {
parent::__construct( $product );
}
/**
* Get internal type.
* @return string
*/
public function get_type() {
return 'external';
}
/*
|--------------------------------------------------------------------------
| Getters
@ -43,30 +51,24 @@ class WC_Product_External extends WC_Product {
| Methods for getting data from the product object.
*/
/**
* Get internal type.
* @return string
*/
public function get_type() {
return 'external';
}
/**
* Get product url.
*
* @param string $context
* @return string
*/
public function get_product_url() {
return esc_url( $this->data['product_url'] );
public function get_product_url( $context = 'view' ) {
return esc_url( $this->get_prop( 'product_url', $context ) );
}
/**
* Get button text.
*
* @param string $context
* @return string
*/
public function get_button_text() {
return $this->data['button_text'] ? $this->data['button_text'] : __( 'Buy product', 'woocommerce' );
public function get_button_text( $context = 'view' ) {
return $this->get_prop( 'button_text', $context );
}
/*
@ -86,7 +88,7 @@ class WC_Product_External extends WC_Product {
* @param string $product_url Product URL.
*/
public function set_product_url( $product_url ) {
$this->data['product_url'] = $product_url;
$this->set_prop( 'product_url', $product_url );
}
/**
@ -96,7 +98,7 @@ class WC_Product_External extends WC_Product {
* @param string $button_text Button text.
*/
public function set_button_text( $button_text ) {
$this->data['button_text'] = $button_text;
$this->set_prop( 'button_text', $button_text );
}
/**
@ -106,7 +108,7 @@ class WC_Product_External extends WC_Product {
* @param bool
*/
public function set_manage_stock( $manage_stock ) {
$this->data['manage_stock'] = false;
$this->set_prop( 'manage_stock', false );
if ( true === $manage_stock ) {
$this->error( 'product_external_invalid_manage_stock', __( 'External products cannot be stock managed.', 'woocommerce' ) );
@ -120,7 +122,7 @@ class WC_Product_External extends WC_Product {
* @param bool
*/
public function set_stock_status( $stock_status ) {
$this->data['stock_status'] = 'instock';
$this->set_prop( 'stock_status', 'instock' );
if ( 'instock' !== $stock_status ) {
$this->error( 'product_external_invalid_stock_status', __( 'External products cannot be stock managed.', 'woocommerce' ) );
@ -134,7 +136,7 @@ class WC_Product_External extends WC_Product {
* @param string $backorders Options: 'yes', 'no' or 'notify'.
*/
public function set_backorders( $backorders ) {
$this->data['backorders'] = 'no';
$this->set_prop( 'backorders', 'no' );
if ( 'no' !== $backorders ) {
$this->error( 'product_external_invalid_backorders', __( 'External products cannot be backordered.', 'woocommerce' ) );
@ -194,19 +196,17 @@ class WC_Product_External extends WC_Product {
*/
/**
* Reads a product from the database and sets its data to the class.
* Read post data.
*
* @since 2.7.0
* @param int $id Product ID.
*/
public function read( $id ) {
parent::read( $id );
protected function read_product_data() {
parent::read_product_data();
$this->set_props( array(
'product_url' => get_post_meta( $id, '_product_url', true ),
'button_text' => get_post_meta( $id, '_button_text', true ),
'product_url' => get_post_meta( $this->get_id(), '_product_url', true ),
'button_text' => get_post_meta( $this->get_id(), '_button_text', true ) ? get_post_meta( $this->get_id(), '_button_text', true ) : __( 'Buy product', 'woocommerce' ),
) );
do_action( 'woocommerce_product_loaded', $this );
do_action( 'woocommerce_product_' . $this->get_type() . '_loaded', $this );
}
/**

64
includes/class-wc-product-grouped.php
View File

@ -34,14 +34,6 @@ class WC_Product_Grouped extends WC_Product {
parent::__construct( $product );
}
/*
|--------------------------------------------------------------------------
| Getters
|--------------------------------------------------------------------------
|
| Methods for getting data from the product object.
*/
/**
* Get internal type.
* @return string
@ -50,15 +42,6 @@ class WC_Product_Grouped extends WC_Product {
return 'grouped';
}
/**
* Return the children of this product.
*
* @return array
*/
public function get_children() {
return $this->data['children'];
}
/**
* Get the add to cart button text.
*
@ -131,6 +114,24 @@ class WC_Product_Grouped extends WC_Product {
return apply_filters( 'woocommerce_get_price_html', $price, $this );
}
/*
|--------------------------------------------------------------------------
| Getters
|--------------------------------------------------------------------------
|
| Methods for getting data from the product object.
*/
/**
* Return the children of this product.
*
* @param string $context
* @return array
*/
public function get_children( $context = 'view' ) {
return $this->get_prop( 'children', $context );
}
/*
|--------------------------------------------------------------------------
| Setters
@ -145,7 +146,7 @@ class WC_Product_Grouped extends WC_Product {
* @param array $children
*/
public function set_children( $children ) {
$this->data['children'] = array_filter( wp_parse_id_list( (array) $children ) );
$this->set_prop( 'children', array_filter( wp_parse_id_list( (array) $children ) ) );
}
/*
@ -155,35 +156,16 @@ class WC_Product_Grouped extends WC_Product {
*/
/**
* Reads a product from the database and sets its data to the class.
* Read post data.
*
* @since 2.7.0
* @param int $id Product ID.
*/
public function read( $id ) {
parent::read( $id );
$transient_name = 'wc_product_children_' . $this->get_id();
$grouped_products = array_filter( wp_parse_id_list( (array) get_transient( $transient_name ) ) );
if ( empty( $grouped_products ) ) {
$grouped_products = get_posts( apply_filters( 'woocommerce_grouped_children_args', array(
'post_parent' => $this->get_id(),
'post_type' => 'product',
'orderby' => 'menu_order',
'order' => 'ASC',
'fields' => 'ids',
'post_status' => 'publish',
'numberposts' => -1,
) ) );
set_transient( $transient_name, $grouped_products, DAY_IN_SECONDS * 30 );
}
protected function read_product_data() {
parent::read_product_data();
$this->set_props( array(
'children' => $grouped_products,
'children' => wp_parse_id_list( get_post_meta( $this->get_id(), '_children', true ) ),
) );
do_action( 'woocommerce_product_loaded', $this );
do_action( 'woocommerce_product_' . $this->get_type() . '_loaded', $this );
}
/**

2
includes/class-wc-product-simple.php
View File

@ -57,7 +57,7 @@ class WC_Product_Simple extends WC_Product {
}
/**
* Get the title of the product.
* Get the title of the post. @todo should this be deprecated or not? It's in deprecated class and needs review.
*
* @return string
*/

2
includes/class-wc-product-variable.php
View File

@ -4,7 +4,7 @@ if ( ! defined( 'ABSPATH' ) ) {
}
/**
* Variable Product Class.
* Variable Product Class. @todo needs new getters/setters/changes code
*
* The WooCommerce product class handles individual product data.
*

9
includes/class-wc-product-variation.php
View File

@ -4,7 +4,14 @@ if ( ! defined( 'ABSPATH' ) ) {
}
/**
* Product Variation Class.
* Product Variation Class. @todo needs new getters/setters/changes code
*
* @todo removed filters need to be mapped via add_action to the product actions of similar naming.
* woocommerce_variation_is_in_stock
* woocommerce_variation_sale_price_html
* woocommerce_variation_price_html
* woocommerce_variation_free_price_html
* woocommerce_get_variation_price_html
*
* @todo removed filters need to be mapped via add_action to the product actions of similar naming.
* woocommerce_variation_is_in_stock

22
includes/wc-deprecated-functions.php
View File

@ -508,13 +508,21 @@ function woocommerce_list_pages( $pages ) {
global $wc_map_deprecated_filters;
$wc_map_deprecated_filters = array(
'woocommerce_add_to_cart_fragments' => 'add_to_cart_fragments',
'woocommerce_add_to_cart_redirect' => 'add_to_cart_redirect',
'woocommerce_structured_data_order' => 'woocommerce_email_order_schema_markup',
'woocommerce_product_get_width' => 'woocommerce_product_width',
'woocommerce_product_get_height' => 'woocommerce_product_height',
'woocommerce_product_get_length' => 'woocommerce_product_length',
'woocommerce_product_get_weight' => 'woocommerce_product_weight',
'woocommerce_structured_data_order' => 'woocommerce_email_order_schema_markup',
'woocommerce_add_to_cart_fragments' => 'add_to_cart_fragments',
'woocommerce_add_to_cart_redirect' => 'add_to_cart_redirect',
'woocommerce_product_get_width' => 'woocommerce_product_width',
'woocommerce_product_get_height' => 'woocommerce_product_height',
'woocommerce_product_get_length' => 'woocommerce_product_length',
'woocommerce_product_get_weight' => 'woocommerce_product_weight',
'woocommerce_product_get_sku' => 'woocommerce_get_sku',
'woocommerce_product_get_price' => 'woocommerce_get_price',
'woocommerce_product_get_regular_price' => 'woocommerce_get_regular_price',
'woocommerce_product_get_sale_price' => 'woocommerce_get_sale_price',
'woocommerce_product_get_tax_class' => 'woocommerce_product_tax_class',
'woocommerce_product_get_stock_quantity' => 'woocommerce_get_stock_quantity',
'woocommerce_product_get_attributes' => 'woocommerce_get_product_attributes',
'woocommerce_product_get_gallery_image_ids' => 'woocommerce_product_gallery_attachment_ids',
);
foreach ( $wc_map_deprecated_filters as $new => $old ) {