WC_Order_Item_Product - Code Metrics - Inspection of "Order CRUD - data manipulation and abstraction" - woothemes/woocommerce - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#10259)

by Mike

created 2016-02-16 13:53 UTC

WC_Order_Item_Product B

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	319
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Metric	Value
wmc	41
lcom	1
cbo	2
dl	0
loc	319
rs	8.2769

28 Methods

Rating	Name	Size	Complexity
B	offsetGet()	18	6
A	offsetExists()	6	2
A	read()	14	2
A	save()	14	2
A	get_internal_meta_keys()	3	1
A	get_product()	7	2
A	get_tax_status()	4	2
A	set_qty()	3	1
A	set_tax_class()	3	1
A	set_product_id()	3	1
A	set_variation_id()	3	1
A	set_subtotal()	3	1
A	set_total()	3	1
A	set_subtotal_tax()	3	1
A	set_total_tax()	3	1
A	set_taxes()	12	3
A	set_variations()	5	2
A	get_type()	3	1
A	get_product_id()	3	1
A	get_variation_id()	3	1
A	get_qty()	3	1
A	get_tax_class()	3	1
A	get_subtotal()	3	1
A	get_subtotal_tax()	3	1
A	get_total()	3	1
A	get_total_tax()	3	1
A	get_taxes()	3	1
A	get_meta_data()	3	1

How to fix Complexity

<?php
/**
 * Order Line Item (product).
 *
 * @class 		WC_Order_Item_Product
 * @version		2.6.0
 * @since       2.6.0
 * @package		WooCommerce/Classes
 * @author 		WooThemes
 */
class WC_Order_Item_Product extends WC_Order_Item {

    /**
	 * Data properties of this order item object.
	 * @since 2.6.0
	 * @var array
	 */
    protected $data = array(
        'order_id'      => 0,
		'order_item_id' => 0,
        'name'          => '',
        'product_id'    => 0,
        'variation_id'  => 0,
        'qty'           => 0,
        'tax_class'     => '',
        'subtotal'      => 0,
        'subtotal_tax'  => 0,
        'total'         => 0,
        'total_tax'     => 0,
        'taxes'         => array(
            'subtotal' => array(),
            'total'    => array()
        ),
        'meta_data'     => array(),
    );

    /**
     * offsetGet for ArrayAccess/Backwards compatibility.
     * @todo Add deprecation notices in future release.
     * @param string $offset
     * @return mixed
     */
    public function offsetGet( $offset ) {
        if ( 'line_subtotal' === $offset ) {
            $offset = 'subtotal';
        }
        elseif ( 'line_subtotal_tax' === $offset ) {
            $offset = 'subtotal_tax';
        }
        elseif ( 'line_total' === $offset ) {
            $offset = 'total';
        }
        elseif ( 'line_tax' === $offset ) {
            $offset = 'total_tax';
        }
        elseif ( 'line_tax_data' === $offset ) {
            $offset = 'taxes';
        }
        return parent::offsetGet( $offset );
    }

    /**
     * offsetExists for ArrayAccess
     * @param string $offset
     * @return bool
     */
    public function offsetExists( $offset ) {
        if ( in_array( $offset, array( 'line_subtotal', 'line_subtotal_tax', 'line_total', 'line_tax', 'line_tax_data', 'item_meta_array', 'item_meta' ) ) ) {
            return true;
        }
        return parent::offsetExists( $offset );
    }

    /**
     * Read/populate data properties specific to this order item.
     */
    protected function read( $id ) {
        parent::read( $id );
        if ( $this->get_order_item_id() ) {
            $this->set_product_id( get_metadata( 'order_item', $this->get_order_item_id(), '_product_id', true ) );
            $this->set_variation_id( get_metadata( 'order_item', $this->get_order_item_id(), '_variation_id', true ) );
            $this->set_qty( get_metadata( 'order_item', $this->get_order_item_id(), '_qty', true ) );
            $this->set_tax_class( get_metadata( 'order_item', $this->get_order_item_id(), '_tax_class', true ) );
            $this->set_subtotal( get_metadata( 'order_item', $this->get_order_item_id(), '_line_subtotal', true ) );
            $this->set_subtotal_tax( get_metadata( 'order_item', $this->get_order_item_id(), '_line_subtotal_tax', true ) );
            $this->set_total( get_metadata( 'order_item', $this->get_order_item_id(), '_line_total', true ) );
            $this->set_total_tax( get_metadata( 'order_item', $this->get_order_item_id(), '_line_tax', true ) );
            $this->set_taxes( get_metadata( 'order_item', $this->get_order_item_id(), '_line_tax_data', true ) );
        }
    }

    /**
     * Save properties specific to this order item.
     */
    protected function save() {
        parent::save();
        if ( $this->get_order_item_id() ) {
            wc_update_order_item_meta( $this->get_order_item_id(), '_product_id', $this->get_product_id() );
            wc_update_order_item_meta( $this->get_order_item_id(), '_variation_id', $this->get_variation_id() );
            wc_update_order_item_meta( $this->get_order_item_id(), '_qty', $this->get_qty() );
            wc_update_order_item_meta( $this->get_order_item_id(), '_tax_class', $this->get_tax_class() );
            wc_update_order_item_meta( $this->get_order_item_id(), '_line_subtotal', $this->get_subtotal() );
            wc_update_order_item_meta( $this->get_order_item_id(), '_line_subtotal_tax', $this->get_subtotal_tax() );
            wc_update_order_item_meta( $this->get_order_item_id(), '_line_total', $this->get_total() );
            wc_update_order_item_meta( $this->get_order_item_id(), '_line_tax', $this->get_total_tax() );
            wc_update_order_item_meta( $this->get_order_item_id(), '_line_tax_data', $this->get_taxes() );
        }
    }

    /**
     * Internal meta keys we don't want exposed as part of meta_data.
     * @return array()

     */
    protected function get_internal_meta_keys() {
        return array( '_product_id', '_variation_id', '_qty', '_tax_class', '_line_subtotal', '_line_subtotal_tax', '_line_total', '_line_tax', '_line_tax_data' );
    }

    /**
     * Get the associated product.
     * @return WC_Product|bool
     */
    public function get_product() {
        if ( $this->get_variation_id() ) {
			return wc_get_product( $this->get_variation_id() );
		} else {
			return wc_get_product( $this->get_product_id() );
		}
    }

    /**
     * Get tax status.
     * @return string
     */
    public function get_tax_status() {
        $product = $this->get_product();
        return $product ? $product->get_tax_class() : 'taxable';
    }

    /*
	|--------------------------------------------------------------------------
	| Setters
	|--------------------------------------------------------------------------
	*/

    /**
     * Set qty.
     * @param int $value
     */
    public function set_qty( $value ) {
        $this->data['qty'] = wc_stock_amount( $value );
    }

    /**
     * Set tax class.
     * @param string $value
     */
    public function set_tax_class( $value ) {
        $this->data['tax_class'] = $value;
    }

    /**
     * Set Product ID
     * @param int $value
     */
    public function set_product_id( $value ) {
        $this->data['product_id'] = absint( $value );
    }

    /**
     * Set variation ID.
     * @param int $value
     */
    public function set_variation_id( $value ) {
        $this->data['variation_id'] = absint( $value );
    }

    /**
     * Line subtotal (before discounts).
     * @param string $value
     */
    public function set_subtotal( $value ) {
        $this->data['subtotal'] = wc_format_decimal( $value );
    }

    /**
     * Line total (after discounts).
     * @param string $value
     */
    public function set_total( $value ) {
        $this->data['total'] = wc_format_decimal( $value );
    }

    /**
     * Line subtotal tax (before discounts).
     * @param string $value
     */
    public function set_subtotal_tax( $value ) {
        $this->data['subtotal_tax'] = wc_format_decimal( $value );
    }

    /**
     * Line total tax (after discounts).
     * @param string $value
     */
    public function set_total_tax( $value ) {
        $this->data['total_tax'] = wc_format_decimal( $value );
    }

    /**
     * Set line taxes.
     * @param array $raw_tax_data
     */
    public function set_taxes( $raw_tax_data ) {
        $raw_tax_data = maybe_unserialize( $raw_tax_data );
        $tax_data     = array(
            'total'    => array(),
            'subtotal' => array()
        );
        if ( ! empty( $raw_tax_data['total'] ) && ! empty( $raw_tax_data['subtotal'] ) ) {
            $tax_data['total']    = array_map( 'wc_format_decimal', $raw_tax_data['total'] );
            $tax_data['subtotal'] = array_map( 'wc_format_decimal', $raw_tax_data['subtotal'] );
        }
        $this->data['taxes'] = $tax_data;
    }

    /**
     * Set variation data (stored as meta data - write only).
     * @param array $data Key/Value pairs
     */
    public function set_variations( $data ) {
        foreach ( $data as $key => $value ) {
            $this->data['meta_data'][ str_replace( 'attribute_', '', $key ) ] = $value;
        }
    }

    /*
	|--------------------------------------------------------------------------
	| Getters
	|--------------------------------------------------------------------------
	*/

    /**
     * Get order item type.
     * @return string
     */
    public function get_type() {
        return 'line_item';
    }

    /**
     * Get product ID.
     * @return int
     */
    public function get_product_id() {
        return absint( $this->data['product_id'] );
    }

    /**
     * Get variation ID.
     * @return int
     */
    public function get_variation_id() {
        return absint( $this->data['variation_id'] );
    }

    /**
     * Get qty.
     * @return int
     */
    public function get_qty() {
        return wc_stock_amount( $this->data['qty'] );
    }

    /**
     * Get tax class.
     * @return string
     */
    public function get_tax_class() {
        return $this->data['tax_class'];
    }

    /**
     * Get subtotal.
     * @return string
     */
    public function get_subtotal() {
        return wc_format_decimal( $this->data['subtotal'] );
    }

    /**
     * Get subtotal tax.
     * @return string
     */
    public function get_subtotal_tax() {
        return wc_format_decimal( $this->data['subtotal_tax'] );
    }

    /**
     * Get total.
     * @return string
     */
    public function get_total() {
        return wc_format_decimal( $this->data['total'] );
    }

    /**
     * Get total tax.
     * @return string
     */
    public function get_total_tax() {
        return wc_format_decimal( $this->data['total_tax'] );
    }

    /**
     * Get fee taxes.
     * @return array
     */
    public function get_taxes() {
        return $this->data['taxes'];
    }

    /**
     * Get meta data.
     * @return array of key/value pairs
     */
    public function get_meta_data() {
        return $this->data['meta_data'];
    }
}


1			<?php
2			/**
3			* Order Line Item (product).
4			*
5			* @class WC_Order_Item_Product
6			* @version 2.6.0
7			* @since 2.6.0
8			* @package WooCommerce/Classes
9			* @author WooThemes
10			*/
11			class WC_Order_Item_Product extends WC_Order_Item {
12
13			/**
14			* Data properties of this order item object.
15			* @since 2.6.0
16			* @var array
17			*/
18			protected $data = array(
19			'order_id' => 0,
20			'order_item_id' => 0,
21			'name' => '',
22			'product_id' => 0,
23			'variation_id' => 0,
24			'qty' => 0,
25			'tax_class' => '',
26			'subtotal' => 0,
27			'subtotal_tax' => 0,
28			'total' => 0,
29			'total_tax' => 0,
30			'taxes' => array(
31			'subtotal' => array(),
32			'total' => array()
33			),
34			'meta_data' => array(),
35			);
36
37			/**
38			* offsetGet for ArrayAccess/Backwards compatibility.
39			* @todo Add deprecation notices in future release.
40			* @param string $offset
41			* @return mixed
42			*/
43			public function offsetGet( $offset ) {
44			if ( 'line_subtotal' === $offset ) {
45			$offset = 'subtotal';
46			}
47			elseif ( 'line_subtotal_tax' === $offset ) {
48			$offset = 'subtotal_tax';
49			}
50			elseif ( 'line_total' === $offset ) {
51			$offset = 'total';
52			}
53			elseif ( 'line_tax' === $offset ) {
54			$offset = 'total_tax';
55			}
56			elseif ( 'line_tax_data' === $offset ) {
57			$offset = 'taxes';
58			}
59			return parent::offsetGet( $offset );
60			}
61
62			/**
63			* offsetExists for ArrayAccess
64			* @param string $offset
65			* @return bool
66			*/
67			public function offsetExists( $offset ) {
68			if ( in_array( $offset, array( 'line_subtotal', 'line_subtotal_tax', 'line_total', 'line_tax', 'line_tax_data', 'item_meta_array', 'item_meta' ) ) ) {
69			return true;
70			}
71			return parent::offsetExists( $offset );
72			}
73
74			/**
75			* Read/populate data properties specific to this order item.
76			*/
77			protected function read( $id ) {
78			parent::read( $id );
79			if ( $this->get_order_item_id() ) {
80			$this->set_product_id( get_metadata( 'order_item', $this->get_order_item_id(), '_product_id', true ) );
81			$this->set_variation_id( get_metadata( 'order_item', $this->get_order_item_id(), '_variation_id', true ) );
82			$this->set_qty( get_metadata( 'order_item', $this->get_order_item_id(), '_qty', true ) );
83			$this->set_tax_class( get_metadata( 'order_item', $this->get_order_item_id(), '_tax_class', true ) );
84			$this->set_subtotal( get_metadata( 'order_item', $this->get_order_item_id(), '_line_subtotal', true ) );
85			$this->set_subtotal_tax( get_metadata( 'order_item', $this->get_order_item_id(), '_line_subtotal_tax', true ) );
86			$this->set_total( get_metadata( 'order_item', $this->get_order_item_id(), '_line_total', true ) );
87			$this->set_total_tax( get_metadata( 'order_item', $this->get_order_item_id(), '_line_tax', true ) );
88			$this->set_taxes( get_metadata( 'order_item', $this->get_order_item_id(), '_line_tax_data', true ) );
89			}
90			}
91
92			/**
93			* Save properties specific to this order item.
94			*/
95			protected function save() {
96			parent::save();
97			if ( $this->get_order_item_id() ) {
98			wc_update_order_item_meta( $this->get_order_item_id(), '_product_id', $this->get_product_id() );
99			wc_update_order_item_meta( $this->get_order_item_id(), '_variation_id', $this->get_variation_id() );
100			wc_update_order_item_meta( $this->get_order_item_id(), '_qty', $this->get_qty() );
101			wc_update_order_item_meta( $this->get_order_item_id(), '_tax_class', $this->get_tax_class() );
102			wc_update_order_item_meta( $this->get_order_item_id(), '_line_subtotal', $this->get_subtotal() );
103			wc_update_order_item_meta( $this->get_order_item_id(), '_line_subtotal_tax', $this->get_subtotal_tax() );
104			wc_update_order_item_meta( $this->get_order_item_id(), '_line_total', $this->get_total() );
105			wc_update_order_item_meta( $this->get_order_item_id(), '_line_tax', $this->get_total_tax() );
106			wc_update_order_item_meta( $this->get_order_item_id(), '_line_tax_data', $this->get_taxes() );
107			}
108			}
109
110			/**
111			* Internal meta keys we don't want exposed as part of meta_data.
112			* @return array()
			0 ignored issues – show Documentation introduced 2016-02-16 14:01 UTC by Report Bug Copy Issue Report The doc-type `array()` could not be parsed: Expected "\|" or "end of type", but got "(" at position 5. (view supported doc-types) This check marks PHPDoc comments that could not be parsed by our parser. To see which comment annotations we can parse, please refer to our documentation on supported doc-types. Loading history...
113			*/
114			protected function get_internal_meta_keys() {
115			return array( '_product_id', '_variation_id', '_qty', '_tax_class', '_line_subtotal', '_line_subtotal_tax', '_line_total', '_line_tax', '_line_tax_data' );
116			}
117
118			/**
119			* Get the associated product.
120			* @return WC_Product\|bool
121			*/
122			public function get_product() {
123			if ( $this->get_variation_id() ) {
124			return wc_get_product( $this->get_variation_id() );
125			} else {
126			return wc_get_product( $this->get_product_id() );
127			}
128			}
129
130			/**
131			* Get tax status.
132			* @return string
133			*/
134			public function get_tax_status() {
135			$product = $this->get_product();
136			return $product ? $product->get_tax_class() : 'taxable';
137			}
138
139			/*
140			\|--------------------------------------------------------------------------
141			\| Setters
142			\|--------------------------------------------------------------------------
143			*/
144
145			/**
146			* Set qty.
147			* @param int $value
148			*/
149			public function set_qty( $value ) {
150			$this->data['qty'] = wc_stock_amount( $value );
151			}
152
153			/**
154			* Set tax class.
155			* @param string $value
156			*/
157			public function set_tax_class( $value ) {
158			$this->data['tax_class'] = $value;
159			}
160
161			/**
162			* Set Product ID
163			* @param int $value
164			*/
165			public function set_product_id( $value ) {
166			$this->data['product_id'] = absint( $value );
167			}
168
169			/**
170			* Set variation ID.
171			* @param int $value
172			*/
173			public function set_variation_id( $value ) {
174			$this->data['variation_id'] = absint( $value );
175			}
176
177			/**
178			* Line subtotal (before discounts).
179			* @param string $value
180			*/
181			public function set_subtotal( $value ) {
182			$this->data['subtotal'] = wc_format_decimal( $value );
183			}
184
185			/**
186			* Line total (after discounts).
187			* @param string $value
188			*/
189			public function set_total( $value ) {
190			$this->data['total'] = wc_format_decimal( $value );
191			}
192
193			/**
194			* Line subtotal tax (before discounts).
195			* @param string $value
196			*/
197			public function set_subtotal_tax( $value ) {
198			$this->data['subtotal_tax'] = wc_format_decimal( $value );
199			}
200
201			/**
202			* Line total tax (after discounts).
203			* @param string $value
204			*/
205			public function set_total_tax( $value ) {
206			$this->data['total_tax'] = wc_format_decimal( $value );
207			}
208
209			/**
210			* Set line taxes.
211			* @param array $raw_tax_data
212			*/
213			public function set_taxes( $raw_tax_data ) {
214			$raw_tax_data = maybe_unserialize( $raw_tax_data );
215			$tax_data = array(
216			'total' => array(),
217			'subtotal' => array()
218			);
219			if ( ! empty( $raw_tax_data['total'] ) && ! empty( $raw_tax_data['subtotal'] ) ) {
220			$tax_data['total'] = array_map( 'wc_format_decimal', $raw_tax_data['total'] );
221			$tax_data['subtotal'] = array_map( 'wc_format_decimal', $raw_tax_data['subtotal'] );
222			}
223			$this->data['taxes'] = $tax_data;
224			}
225
226			/**
227			* Set variation data (stored as meta data - write only).
228			* @param array $data Key/Value pairs
229			*/
230			public function set_variations( $data ) {
231			foreach ( $data as $key => $value ) {
232			$this->data['meta_data'][ str_replace( 'attribute_', '', $key ) ] = $value;
233			}
234			}
235
236			/*
237			\|--------------------------------------------------------------------------
238			\| Getters
239			\|--------------------------------------------------------------------------
240			*/
241
242			/**
243			* Get order item type.
244			* @return string
245			*/
246			public function get_type() {
247			return 'line_item';
248			}
249
250			/**
251			* Get product ID.
252			* @return int
253			*/
254			public function get_product_id() {
255			return absint( $this->data['product_id'] );
256			}
257
258			/**
259			* Get variation ID.
260			* @return int
261			*/
262			public function get_variation_id() {
263			return absint( $this->data['variation_id'] );
264			}
265
266			/**
267			* Get qty.
268			* @return int
269			*/
270			public function get_qty() {
271			return wc_stock_amount( $this->data['qty'] );
272			}
273
274			/**
275			* Get tax class.
276			* @return string
277			*/
278			public function get_tax_class() {
279			return $this->data['tax_class'];
280			}
281
282			/**
283			* Get subtotal.
284			* @return string
285			*/
286			public function get_subtotal() {
287			return wc_format_decimal( $this->data['subtotal'] );
288			}
289
290			/**
291			* Get subtotal tax.
292			* @return string
293			*/
294			public function get_subtotal_tax() {
295			return wc_format_decimal( $this->data['subtotal_tax'] );
296			}
297
298			/**
299			* Get total.
300			* @return string
301			*/
302			public function get_total() {
303			return wc_format_decimal( $this->data['total'] );
304			}
305
306			/**
307			* Get total tax.
308			* @return string
309			*/
310			public function get_total_tax() {
311			return wc_format_decimal( $this->data['total_tax'] );
312			}
313
314			/**
315			* Get fee taxes.
316			* @return array
317			*/
318			public function get_taxes() {
319			return $this->data['taxes'];
320			}
321
322			/**
323			* Get meta data.
324			* @return array of key/value pairs
325			*/
326			public function get_meta_data() {
327			return $this->data['meta_data'];
328			}
329			}
330

woothemes / woocommerce

Pull Request — master (#10259)

WC_Order_Item_Product B

Complexity

Size/Duplication

Coupling/Cohesion

28 Methods

How to fix Complexity

Complex Class

Duplication Side-by-Side

Filter issues like