WPSC_Purchase_Log

Complexity

Total Complexity

197

Size/Duplication

Total Lines	1467
Duplicated Lines	0 %

Coupling/Cohesion

Components	11
Dependencies	5

Importance

Changes

0

65 Methods

Rating	Name	Duplication	Size	Complexity
A	get_column_format()	0	11	3
F	fetch_stats()	0	134	16
A	convert_date_args()	0	4	1
A	get_stats_for_product()	0	15	2
A	is_order_status_completed()	0	9	1
A	update_cache()	0	3	1
A	update_caches()	0	14	2
A	delete_cache()	0	5	1
A	delete_caches()	0	16	3
C	delete()	0	50	8
C	__construct()	0	42	8
A	set_total_shipping()	0	9	1
A	set_gateway_name()	0	13	3
A	set_shipping_method_names()	0	16	2
A	set_meta_props()	0	10	2
A	get_meta()	0	8	3
B	fetch()	0	33	5
A	prepare_get()	0	3	1
A	prepare_get_data()	0	3	1
A	prepare_get_meta()	0	3	1
A	get_cart_contents()	0	4	1
B	get_items()	0	25	6
A	get_cart_item()	0	10	2
A	get_cart_item_from_product_id()	0	12	3
B	update_cart_item()	0	25	3
B	remove_cart_item()	0	24	3
B	get_gateway_data()	0	44	5
C	set()	0	36	8
A	get_data_format()	0	9	2
B	save()	0	62	6
A	save_meta()	0	13	2
A	update_downloadable_status()	0	16	2
A	have_downloads_locked()	0	8	1
C	get_log_by_meta()	0	24	8
A	get_next_log_id()	0	14	2
A	get_previous_log_id()	0	14	2
A	is_transaction_completed()	0	3	1
A	can_edit()	0	8	3
A	is_order_received()	0	3	1
A	is_incomplete_sale()	0	3	1
A	is_accepted_payment()	0	3	1
A	is_job_dispatched()	0	3	1
A	is_closed_order()	0	3	1
A	is_payment_declined()	0	3	1
A	is_refunded()	0	3	1
A	is_refund_pending()	0	3	1
A	init_items()	0	9	2
A	buyers_name()	0	19	4
A	buyers_city()	0	9	3
A	buyers_email()	0	9	3
A	buyers_address()	0	9	3
C	buyers_state_and_postcode()	0	26	7
A	buyers_country()	0	9	3
A	buyers_phone()	0	9	3
A	shipping_name()	0	9	3
A	shipping_city()	0	9	3
A	shipping_address()	0	9	3
B	shipping_state_and_postcode()	0	24	6
A	shipping_country()	0	11	3
B	payment_method()	0	21	7
A	shipping_method()	0	15	3
A	discount()	0	8	2
A	shipping()	0	13	3
A	taxes()	0	9	2
A	total_price()	0	6	1

<?php
// by default, expire stats cache after 48 hours
// this doesn't have any effect if you're not using APC or memcached

if ( ! defined( 'WPSC_PURCHASE_LOG_STATS_CACHE_EXPIRE' ) ) {
	define( 'WPSC_PURCHASE_LOG_STATS_CACHE_EXPIRE', DAY_IN_SECONDS * 2 );
}

class WPSC_Purchase_Log extends WPSC_Query_Base {
	const INCOMPLETE_SALE  		= 1;
	const ORDER_RECEIVED  	 	= 2;
	const ACCEPTED_PAYMENT		= 3;
	const JOB_DISPATCHED   		= 4;
	const CLOSED_ORDER     		= 5;
	const PAYMENT_DECLINED 		= 6;
	const REFUNDED         		= 7;
	const REFUND_PENDING   		= 8;
	const PARTIALLY_REFUNDED 	= 9;

	/**
	 * Names of column that requires escaping values as strings before being inserted
	 * into the database
	 *
	 * @access private
	 * @static
	 * @since 3.8.9
	 *
	 * @var array
	 */
	private static $string_cols = array(
		'sessionid',
		'transactid',
		'authcode',
		'date',
		'gateway',
		'billing_country',
		'shipping_country',
		'email_sent',
		'stock_adjusted',
		'discount_data',
		'track_id',
		'billing_region',
		'shipping_region',
		'find_us',
		'engrave_text',
		'shipping_method',
		'shipping_option',
		'affiliate_id',
		'plugin_version',
		'notes',
	);

	/**
	 * Names of column that requires escaping values as integers before being inserted
	 * into the database
	 *
	 * @static
	 * @since 3.8.9
	 * @var array
	 */
	private static $int_cols = array(
		'id',
		'statusno',
		'processed',
		'user_ID',
	);

	/**
	 * Names of column that requires escaping values as float before being inserted
	 * into the database
	 *
	 * @static
	 * @since 4.0
	 * @var array
	 */
	private static $float_cols = array(
		'totalprice',
		'base_shipping',
		'discount_value',
		'wpec_taxes_total',
		'wpec_taxes_rate',
	);

	/**
	 * Array of metadata
	 *
	 * @static
	 * @since 4.0
	 * @var array
	 */
	private static $metadata = array(

The property $metadata is not used and could be removed.

		'totalprice',
		'base_shipping',
		'discount_value',
		'wpec_taxes_total',
		'wpec_taxes_rate',
	);

	private $gateway_data = array();

	private $is_status_changed = false;
	private $previous_status   = false;

	private $cart_contents = array();
	private $cart_ids = array();
	private $can_edit = null;

	/**
	 * Contains the constructor arguments. This array is necessary because we will
	 * lazy load the DB row into $this->data whenever necessary. Lazy loading is,
	 * in turn, necessary because sometimes right after saving a new record, we need
	 * to fetch a property with the same object.
	 *
	 * @access private
	 * @since 3.8.9
	 *
	 * @var array
	 */
	private $args = array(
		'col'   => '',
		'value' => '',
	);

   protected $buyers_name = null;
   protected $buyers_city = null;
   protected $buyers_email = null;
   protected $buyers_address = null;
   protected $buyers_state_and_postcode = null;
   protected $buyers_country = null;
   protected $buyers_phone = null;
   protected $shipping_name = null;
   protected $shipping_address = null;
   protected $shipping_city = null;
   protected $shipping_state_and_postcode = null;
   protected $shipping_country = null;
   protected $payment_method = null;
   protected $shipping_method = null;

	/**
	 * Get the SQL query format for a column
	 *
	 * @since 3.8.9
	 * @param  string $col Name of the column
	 * @return string      Placeholder
	 */
	private static function get_column_format( $col ) {
		if ( in_array( $col, self::$string_cols ) ) {
			return '%s';
		}

		if ( in_array( $col, self::$int_cols ) ) {
			return '%d';
		}

		return '%f';
	}

	/**
	 * Query the purchase log table to get sales and earning stats
	 *
	 * Accepts an array of arguments:
	 * 	- 'ids': IDs of products for which you want to get stats
	 * 	- 'products': array of WPSC_Product objects for which you want to get stats
	 * 	- 'start' and 'end': the timestamp range (integers) during which you want
	 * 	                     to collect the stats.
	 * 	                     You can use none, either, or both of them.
	 * 	                     Note that the [start, end) interval is a left-closed,
	 * 	                     right-open.
	 * 	                     E.g.: to get stats from Jan 1st, 2013 to
	 * 	                     Dec 31st, 2013 (23:59:59),
	 * 	                     set "start" to the timestamp for Jan 1st, 2013, set
	 * 	                     "end" to the timestamp for Jan 1st, 2014
	 *  - 'order': what to sort the results by, defaults to 'id'.
	 *            Can be 'ids', 'sales', 'earnings' or empty string to disable sorting
	 *  - 'orderby': how to sort the results, defaults to 'ASC'.
	 *              Can be 'ASC', 'DESC' (lowercase is fine too)
	 *  - 'per_page': How many items to fetch, defaults to 0, which fetches all results
	 *  - 'page': Which page of the results to fetch, defaults to 1.
	 *            Has no effect if per_page is set to 0.
	 *
	 * @since 3.8.14
	 * @param  array|string $args Arguments
	 * @return array       Array containing 'sales' and 'earnings' stats
	 */
	public static function fetch_stats( $args ) {
		global $wpdb;

		$defaults = array(
			'ids'      => array(), // IDs of the products to be queried
			'products' => array(), // Array of WPSC_Products objects
			'start'    => 0,       // Int. timestamp, has to be UTC
			'end'      => 0,       // Int. timestamp, has to be UTC
			'order'    => 'ASC',
			'orderby'  => 'id',
			'per_page' => 0,
			'page'     => 1,
		);

		$args = wp_parse_args( $args, $defaults );

		// convert more advanced date / time args into "start" and "end"
		$args = self::convert_date_args( $args );

		// build an array of WPSC_Product objects based on 'ids' and 'products' args
		$products = array_merge(
			$args['products'],
			array_map( array( 'WPSC_Product', 'get_instance' ), $args['ids'] )
		);

		// eliminate duplicates (Note: usage of this requires PHP 5.2.9)
		$products = array_unique( $products, SORT_REGULAR );

		if ( empty( $products ) ) {
			return null;
		}

		$needs_fetching = array();

		$stats = array(
			'sales'    => 0,
			'earnings' => 0,
		);

		// if we have date restriction, that means we won't be able to rely upon
		// individual stats cache inside WPSC_Product objects
		$has_date_restriction = ! ( empty( $args['start'] ) && empty( $args['end'] ) );

		// if we do NOT have date restriction, find out which WPSC_Product object
		// has stats cache, and which don't
		if ( ! $has_date_restriction ) {
			foreach ( $products as $product ) {
				// store the ID if this product doesn't have a stats cache yet
				if ( $product->post->_wpsc_stats === '' ) {
					$needs_fetching[] = $product->post->ID;
				} else {

					// tally up the sales and earnings if this one has cache already
					$prod_meta = get_post_meta( $product->post->ID, '_wpsc_stats', true );

					if ( isset( $prod_meta['sales'] ) && isset( $prod_meta['earnings'] ) ) {
						$stats['sales']    += $prod_meta['sales'];
						$stats['earnings'] += $prod_meta['earnings'];
					}
					$needs_fetching[]   = $product->post->ID;
				}
			}
		}

		// never hurts to make sure
		$needs_fetching = array_map( 'absint', $needs_fetching );

		// pagination arguments
		$limit = '';

		if ( ! empty( $args['per_page'] ) ) {
			$offset = ( $args['page'] - 1 ) * $args['per_page'];
			$limit = "LIMIT " . absint( $args['per_page'] ) . " OFFSET " . absint( $offset );
		}

		// sorting
		$order = '';

		if ( ! empty( $args['orderby'] ) )
			$order = "ORDER BY " . esc_sql( $args['orderby'] ) . " " . esc_sql( $args['order'] );

		// date
		$where = "WHERE p.processed IN (3, 4, 5)";

		if ( $has_date_restriction ) {
			// start date equal or larger than $args['start']
			if ( ! empty( $args['start'] ) )
				$where .= " AND CAST(p.date AS UNSIGNED) >= " . absint( $args['start'] );

			// end date less than $args['end'].
			// the "<" sign is not a typo, such upper limit makes it easier for
			// people to specify range.
			// E.g.: [1/1/2013 - 1/1/2014) rather than:
			//       [1/1/2013 - 31/12/2013 23:59:59]
			if ( ! empty( $args['end'] ) )
				$where .= " AND CAST(p.date AS UNSIGNED) < " . absint( $args['end'] );
		}

		// assemble the SQL query
		$sql = "
			SELECT cc.prodid AS id, SUM(cc.quantity) AS sales, SUM(cc.quantity * cc.price) AS earnings
			FROM $wpdb->wpsc_purchase_logs AS p
			INNER JOIN
				$wpdb->wpsc_cart_contents AS cc
				ON p.id = cc.purchaseid AND cc.prodid IN (" . implode( ', ', $needs_fetching ) . ")
			{$where}
			GROUP BY cc.prodid
			{$order}
			{$limit}
		";

		// if the result is cached, don't bother querying the database
		$cache_key = md5( $sql );
		$results   = wp_cache_get( $cache_key, 'wpsc_purchase_logs_stats' );

		if ( false === $results ) {
			$results = $wpdb->get_results( $sql );
			wp_cache_set( $cache_key, $results, 'wpsc_purchase_logs_stats', WPSC_PURCHASE_LOG_STATS_CACHE_EXPIRE );
		}

		// tally up the sales and earnings from the query results
		foreach ( $results as $row ) {
			if ( ! $has_date_restriction ) {
				$product           = WPSC_Product::get_instance( $row->id );
				$product->sales    = $row->sales;
				$product->earnings = $row->earnings;
			}

			$stats['sales']    += $row->sales;
			$stats['earnings'] += $row->earnings;
		}

		return $stats;
	}

	/**
	 * Convert advanced date/time arguments like year, month, day, 'ago' etc.
	 * into basic arguments which are "start" and "end".
	 *
	 * @since  3.8.14
	 * @param  array $args Arguments
	 * @return array       Arguments after converting
	 */
	private static function convert_date_args( $args ) {
		// TODO: Implement this
		return $args;
	}

	/**
	 * Get overall sales and earning stats for just one product
	 *
	 * @since 3.8.14
	 * @param  int $id ID of the product
	 * @return array   Array containing 'sales' and 'earnings' stats
	 */
	public static function get_stats_for_product( $id, $args = '' ) {

		$product = WPSC_Product::get_instance( $id );

		// if this product has variations
		if ( $product->has_variations ) {
			// get total stats of variations
			$args['products'] = $product->variations;
		} else {
			// otherwise, get stats of only this product
			$args['products'] = array( $product );
		}

		return self::fetch_stats( $args );
	}

	/**
	 * Check whether the status code indicates a completed status
	 *
	 * @since 3.8.13
	 * @param int  $status Status code
	 * @return boolean
	 */
	public static function is_order_status_completed( $status ) {
		$completed_status = apply_filters( 'wpsc_order_status_completed', array(
			self::ACCEPTED_PAYMENT,
			self::JOB_DISPATCHED,
			self::CLOSED_ORDER,
		) );

		return in_array( $status, $completed_status );
	}

	/**
	 * Update cache of the passed log object
	 *
	 * @access public
	 * @static
	 * @since 3.8.9
	 *
	 * @param WPSC_Purchase_Log $log The log object that you want to store into cache
	 * @return void
	 */
	public static function update_cache( &$log ) {
		return $log->update_caches();
	}

	/**
	 * Update caches.
	 *
	 * @access public
	 * @static
	 * @since 4.0
	 *
	 * @return void
	 */
	public function update_caches() {

		// wpsc_purchase_logs stores the data array, while wpsc_purchase_logs_sessionid stores the
		// log id that's associated with the sessionid
		$id = $this->get( 'id' );
		wp_cache_set( $id, $this->data, 'wpsc_purchase_logs' );

		if ( $sessionid = $this->get( 'sessionid' ) ) {
			wp_cache_set( $sessionid, $id, 'wpsc_purchase_logs_sessionid' );
		}

		wp_cache_set( $id, $this->cart_contents, 'wpsc_purchase_log_cart_contents' );
		do_action( 'wpsc_purchase_log_update_cache', $this );
	}

	/**
	 * Deletes cache of a log (either by using the log ID or sessionid)
	 *
	 * @access public
	 * @static
	 * @since 3.8.9
	 *
	 * @param string $value The value to query
	 * @param string $col Optional. Defaults to 'id'. Whether to delete cache by using
	 *                    a purchase log ID or sessionid
	 * @return void
	 */
	public static function delete_cache( $value, $col = 'id' ) {
		// this will pull from the old cache, so no worries there
		$log = new WPSC_Purchase_Log( $value, $col );
		$log->delete_caches( $value, $col );
	}

	/**
	 * Deletes caches.
	 *
	 * @access public
	 * @static
	 * @since 4.0
	 *
	 * @param string|null $value Optional (left for back-compatibility). The value which was queried.
	 * @param string|null $col   Optional (left for back-compatibility). The column used as the identifier.
	 *
	 * @return void
	 */
	public function delete_caches( $value = null, $col = null ) {
		wp_cache_delete( $this->get( 'id' ), 'wpsc_purchase_logs' );
		wp_cache_delete( $this->get( 'sessionid' ), 'wpsc_purchase_logs_sessionid' );
		wp_cache_delete( $this->get( 'id' ), 'wpsc_purchase_log_cart_contents' );
		wp_cache_delete( $this->get( 'id' ), 'wpsc_purchase_meta' );

		if ( null === $value ) {
			$value = $this->args['value'];
		}

		if ( null === $col ) {
			$col = $this->args['col'];
		}

		do_action( 'wpsc_purchase_log_delete_cache', $this, $value, $col );
	}

	/**
	 * Deletes a log from the database.
	 *
	 * @access  public
	 * @since   3.8.9
	 *
	 * @uses  $wpdb                              Global database instance.
	 * @uses  wpsc_is_store_admin()              Check user has admin capabilities.
	 * @uses  WPSC_Purchase_Log::delete_cache()  Delete purchaselog cache.
	 * @uses  WPSC_Claimed_Stock                 Claimed Stock class.
	 *
	 * @param   string   $log_id   ID of the log.

Should the type for parameter $log_id not be false|string?

	 * @return  boolean            Deleted successfully.
	 */
	public function delete( $log_id = false ) {

		global $wpdb;

		if ( ! ( isset( $this ) && get_class( $this ) == __CLASS__ ) ) {
			_wpsc_doing_it_wrong( 'WPSC_Purchase_Log::delete', __( 'WPSC_Purchase_Log::delete() is no longer a static method and should not be called statically.', 'wp-e-commerce' ), '3.9.0' );
		}

		if ( false !== $log_id ) {
			_wpsc_deprecated_argument( __FUNCTION__, '3.9.0', 'The $log_id param is not used. You must first create an instance of WPSC_Purchase_Log before calling this method.' );
		}

		if ( ! wpsc_is_store_admin() ) {
			return false;
		}

		$log_id = $this->get( 'id' );

		if ( $log_id > 0 ) {

			do_action( 'wpsc_purchase_log_before_delete', $log_id );

			$this->delete_caches();

			// Delete claimed stock
			$purchlog_status = $wpdb->get_var( $wpdb->prepare( "SELECT `processed` FROM `" . WPSC_TABLE_PURCHASE_LOGS . "` WHERE `id`= %d", $log_id ) );
			if ( $purchlog_status == WPSC_Purchase_Log::CLOSED_ORDER || $purchlog_status == WPSC_Purchase_Log::INCOMPLETE_SALE ) {
				$claimed_query = new WPSC_Claimed_Stock( array(
					'cart_id'        => $log_id,
					'cart_submitted' => 1
				) );
				$claimed_query->clear_claimed_stock( 0 );
			}

			// Delete cart content, submitted data, then purchase log
			$wpdb->query( $wpdb->prepare( "DELETE FROM `" . WPSC_TABLE_CART_CONTENTS . "` WHERE `purchaseid` = %d", $log_id ) );
			$wpdb->query( $wpdb->prepare( "DELETE FROM `" . WPSC_TABLE_SUBMITTED_FORM_DATA . "` WHERE `log_id` IN (%d)", $log_id ) );
			$wpdb->query( $wpdb->prepare( "DELETE FROM `" . WPSC_TABLE_PURCHASE_LOGS . "` WHERE `id` = %d LIMIT 1", $log_id ) );
			$wpdb->query( $wpdb->prepare( "DELETE FROM `" . WPSC_TABLE_PURCHASE_META . "` WHERE `wpsc_purchase_id` = %d", $log_id ) );
			$wpdb->query( $wpdb->prepare( "DELETE FROM `" . WPSC_TABLE_DOWNLOAD_STATUS . "` WHERE `purchid` = %d ", $log_id ) );

			do_action( 'wpsc_purchase_log_delete', $log_id );

			return true;

		}

		return false;

	}

	/**
	 * Constructor of the purchase log object. If no argument is passed, this simply
	 * create a new empty object. Otherwise, this will get the purchase log from the
	 * DB either by using purchase log id or sessionid (specified by the 2nd argument).
	 *
	 * Eg:
	 *
	 * // get purchase log with ID number 23
	 * $log = new WPSC_Purchase_Log( 23 );
	 *
	 * // get purchase log with sessionid "asdf"
	 * $log = new WPSC_Purchase_Log( 'asdf', 'sessionid' )
	 *
	 * @access public
	 * @since 3.8.9
	 *
	 * @param string $value Optional. Defaults to false.

Should the type for parameter $value not be false|string?

	 * @param string $col Optional. Defaults to 'id'.
	 */
	public function __construct( $value = false, $col = 'id' ) {
		if ( false === $value ) {
			return;
		}

		if ( is_array( $value ) ) {
			$this->set( $value );
			return;
		}

		global $wpdb;

		if ( ! in_array( $col, array( 'id', 'sessionid' ) ) ) {
			return;
		}

		// store the constructor args into an array so that later we can lazy load the data
		$this->args = array(
			'col'   => $col,
			'value' => $value,
		);

		// if the sessionid is in cache, pull out the id
		if ( $col == 'sessionid'  && $id = wp_cache_get( $value, 'wpsc_purchase_logs_sessionid' ) ) {
				$col = 'id';
				$value = $id;
		}

		// if the id is specified, try to get from cache
		if ( $col == 'id' ) {
			$this->data = wp_cache_get( $value, 'wpsc_purchase_logs' );
			$this->cart_contents = wp_cache_get( $value, 'wpsc_purchase_log_cart_contents' );
		}

		// cache exists
		if ( $this->data ) {
			$this->set_meta_props();
			$this->fetched = true;

The property $fetched was declared of type string, but true is of type boolean. Maybe add a type cast?

			$this->exists  = true;

The property $exists was declared of type string, but true is of type boolean. Maybe add a type cast?

			return;
		}
	}

	private function set_total_shipping() {

		$base_shipping  = $this->get( 'base_shipping' );
		$item_shipping  = wp_list_pluck( $this->get_items(), 'pnp' );

		$this->meta_data['total_shipping'] = $base_shipping + array_sum( $item_shipping );

		return $this->meta_data['total_shipping'];
	}

	private function set_gateway_name() {
		global $wpsc_gateways;
		$gateway = $this->get( 'gateway' );
		$gateway_name = $gateway;

		if( 'wpsc_merchant_testmode' == $gateway )
			$gateway_name = __( 'Manual Payment', 'wp-e-commerce' );
		elseif ( isset( $wpsc_gateways[$gateway] ) )
			$gateway_name = $wpsc_gateways[$gateway]['name'];

		$this->meta_data['gateway_name'] = $gateway_name;
		return $this->meta_data['gateway_name'];
	}

	private function set_shipping_method_names() {
		global $wpsc_shipping_modules;

		$shipping_method = $this->get( 'shipping_method' );
		$shipping_option = $this->get( 'shipping_option' );
		$shipping_method_name = $shipping_method;
		$shipping_option_name = $shipping_option;

		if ( ! empty ( $wpsc_shipping_modules[$shipping_method] ) ) {

Space before opening parenthesis of function call prohibited

			$shipping_class = $wpsc_shipping_modules[$shipping_method];
			$shipping_method_name = $shipping_class->name;
		}

		$this->meta_data['shipping_method_name'] = $shipping_method_name;
		$this->meta_data['shipping_option_name'] = $shipping_option_name;
	}

	private function set_meta_props() {

		foreach ( wpsc_get_purchase_custom( $this->get( 'id' ) ) as $key => $value  ) {
			$this->meta_data[ $key ] = wpsc_get_purchase_meta( $this->get( 'id' ), $key, true );
		}

		$this->set_total_shipping();
		$this->set_gateway_name();
		$this->set_shipping_method_names();
	}

public function get_meta() {

		if ( empty( $this->data ) || empty( $this->meta_data ) ) {
			$this->fetch();
		}

		return (array) apply_filters( 'wpsc_purchase_log_meta_data', $this->meta_data );
	}

	/**
	 * Fetches the actual record from the database
	 *
	 * @access protected
	 * @since 3.8.9
	 *
	 * @return void
	 */
	protected function fetch() {
		global $wpdb;

		if ( $this->fetched ) {
			return;
		}

		// If $this->args is not set yet, it means the object contains a new unsaved
		// row so we don't need to fetch from DB
		if ( ! $this->args['col'] || ! $this->args['value'] ) {
			return;
		}

		extract( $this->args );

extract() usage is highly discouraged, due to the complexity and unintended issues it might cause.

		$format = self::get_column_format( $col );
		$sql    = $wpdb->prepare( "SELECT * FROM " . WPSC_TABLE_PURCHASE_LOGS . " WHERE {$col} = {$format}", $value );

		$this->exists = false;

The property $exists was declared of type string, but false is of type false. Maybe add a type cast?

		if ( $data = $wpdb->get_row( $sql, ARRAY_A ) ) {
			$this->exists        = true;

The property $exists was declared of type string, but true is of type boolean. Maybe add a type cast?

			$this->data          = apply_filters( 'wpsc_purchase_log_data', $data );
			$this->cart_contents = $this->get_items();

			$this->set_meta_props();
			$this->update_caches( $this );

The call to WPSC_Purchase_Log::update_caches() has too many arguments starting with $this.

		}

		do_action( 'wpsc_purchase_log_fetched', $this );

		$this->fetched = true;

The property $fetched was declared of type string, but true is of type boolean. Maybe add a type cast?

	}

	/**
	 * Prepares the return value for get() (apply_filters, etc).
	 *
	 * @access protected
	 * @since  4.0
	 *
	 * @param  mixed  $value Value fetched
	 * @param  string $key   Key for $data.
	 *
	 * @return mixed
	 */
	protected function prepare_get( $value, $key ) {
		return apply_filters( 'wpsc_purchase_log_get_property', $value, $key, $this );
	}

	/**
	 * Prepares the return value for get_data() (apply_filters, etc).
	 *
	 * @access protected
	 * @since  4.0
	 *
	 * @return mixed
	 */
	protected function prepare_get_data() {
		return apply_filters( 'wpsc_purchase_log_get_data', $this->data, $this );
	}

	/**
	 * Prepares the return value for get_meta() (apply_filters, etc).
	 *
	 * @access protected
	 * @since  4.0
	 *
	 * @return mixed
	 */
	protected function prepare_get_meta() {
		return (array) apply_filters( 'wpsc_purchase_log_meta_data', $this->meta_data );
	}

	public function get_cart_contents() {
		_wpsc_doing_it_wrong( __FUNCTION__, __( 'This function has been deprecated in favor of the get_items() method.', 'wp-e-commerce' ), '4.0' );
		return $this->get_items();
	}

	public function get_items() {
		global $wpdb;

		if ( ! empty( $this->cart_contents ) && $this->fetched ) {
			return $this->cart_contents;
		}

		$id = $this->get( 'id' );

		// Bail if we don't have a log object yet (no id).
		if ( empty( $id ) ) {
			return $this->cart_contents;
		}

		$sql = $wpdb->prepare( "SELECT * FROM " . WPSC_TABLE_CART_CONTENTS . " WHERE purchaseid = %d", $id );
		$this->cart_contents = $wpdb->get_results( $sql );

		if ( is_array( $this->cart_contents ) ) {
			foreach ( $this->cart_contents as $index => $item ) {
				$this->cart_ids[ absint( $item->id ) ] = $index;
			}
		}

		return $this->cart_contents;
	}



	public function get_cart_item( $item_id ) {
		$item_id = absint( $item_id );
		$cart    = $this->get_items();

		if ( isset( $this->cart_ids[ $item_id ] ) ) {
			return $cart[ $this->cart_ids[ $item_id ] ];
		}

		return false;
	}

	public function get_cart_item_from_product_id( $product_id ) {
		$product_id = absint( $product_id );
		$cart       = $this->get_items();

		foreach ( $cart as $item ) {
			if ( $product_id === absint( $item->prodid ) ) {
				return $item;
			}
		}

		return false;
	}

	public function update_cart_item( $item_id, $data ) {
		global $wpdb;

		$item_id = absint( $item_id );
		$item = $this->get_cart_item( $item_id );

		if ( $item ) {
			do_action( 'wpsc_purchase_log_before_update_cart_item', $item_id );

			$data = wp_unslash( $data );
			$result = $wpdb->update( WPSC_TABLE_CART_CONTENTS, $data, array( 'id' => $item_id  ) );

			if ( $result ) {

				$this->cart_contents = array();
				$this->get_cart_item( $item_id );

				do_action( 'wpsc_purchase_log_update_cart_item', $item_id );
			}

			return $result;
		}

		return false;
	}

	public function remove_cart_item( $item_id ) {
		global $wpdb;

		$item_id = absint( $item_id );
		$item = $this->get_cart_item( $item_id );

		if ( $item ) {
			do_action( 'wpsc_purchase_log_before_remove_cart_item', $item_id );

			$result = $wpdb->delete( WPSC_TABLE_CART_CONTENTS, array( 'id' => $item_id ) );

			if ( $result ) {

				unset( $this->cart_contents[ $this->cart_ids[ $item_id ] ] );
				unset( $this->cart_ids[ $item_id ] );

				do_action( 'wpsc_purchase_log_remove_cart_item', $item_id );
			}

			return $result;
		}

		return false;
	}

	public function get_gateway_data( $from_currency = false, $to_currency = false ) {
		if ( ! $this->exists() ) {
			return array();
		}

		$subtotal = 0;
		$shipping = wpsc_convert_currency( (float) $this->get( 'base_shipping' ), $from_currency, $to_currency );
		$items    = array();

		$this->gateway_data = array(
			'amount'  => wpsc_convert_currency( $this->get( 'totalprice' ), $from_currency, $to_currency ),
			'invoice' => $this->get( 'sessionid' ),
			'tax'     => wpsc_convert_currency( $this->get( 'wpec_taxes_total' ), $from_currency, $to_currency ),
		);

		foreach ( $this->cart_contents as $item ) {
			$item_price = wpsc_convert_currency( $item->price, $from_currency, $to_currency );
			$items[] = array(
				'name'     => $item->name,
				'amount'   => $item_price,
				'tax'      => wpsc_convert_currency( $item->tax_charged, $from_currency, $to_currency ),
				'quantity' => $item->quantity,
			);
			$subtotal += $item_price * $item->quantity;
			$shipping += wpsc_convert_currency( $item->pnp, $from_currency, $to_currency );
		}

		$this->gateway_data['discount'] = wpsc_convert_currency( (float) $this->get( 'discount_value' ), $from_currency, $to_currency );

		$this->gateway_data['items'] = $items;
		$this->gateway_data['shipping'] = $shipping;
		$this->gateway_data['subtotal'] = $subtotal;

		if ( $from_currency ) {
			// adjust total amount in case there's slight decimal error
			$total = $subtotal + $shipping + $this->gateway_data['tax'] - $this->gateway_data['discount'];
			if ( $this->gateway_data['amount'] != $total ) {
				$this->gateway_data['amount'] = $total;
			}
		}

		$this->gateway_data = apply_filters( 'wpsc_purchase_log_gateway_data', $this->gateway_data, $this->get_data() );
		return $this->gateway_data;
	}

	/**
	 * Sets a property to a certain value. This function accepts a key and a value
	 * as arguments, or an associative array containing key value pairs.
	 *
	 * Loops through data, comparing against database, and saves as meta if not found in purchase log table.
	 *
	 * @access public
	 * @since 3.8.9
	 *
	 * @param mixed $key Name of the property (column), or an array containing key
	 *                   value pairs
	 * @param string|int $value Optional. Defaults to false. In case $key is a string,

Should the type for parameter $value not be string|integer|null?

	 *                          this should be specified.
	 * @return WPSC_Purchase_Log The current object (for method chaining)
	 */
	public function set( $key, $value = null ) {
		if ( is_array( $key ) ) {
			$properties = $key;
		} else {
			if ( is_null( $value ) ) {
				return $this;
			}

			$properties = array( $key => $value );
		}

		$properties = apply_filters( 'wpsc_purchase_log_set_properties', $properties, $this );

		if ( array_key_exists( 'processed', $properties ) ) {
			$this->previous_status = $this->get( 'processed' );

			if ( $properties['processed'] != $this->previous_status ) {
				$this->is_status_changed = true;
			}
		}

		if ( ! is_array( $this->data ) ) {
			$this->data = array();
		}

		foreach ( $properties as $key => $value ) {
			if ( ! in_array( $key, array_merge( self::$string_cols, self::$int_cols, self::$float_cols ) ) ) {
				$this->meta_data[ $key ] = $value;
				unset( $properties[ $key ] );
			}

Blank line found after control structure

		}

		$this->data = array_merge( $this->data, $properties );
		return $this;
	}

	/**
	 * Returns an array containing the parameter format so that this can be used in
	 * $wpdb methods (update, insert etc.)
	 *
	 * @access private
	 * @since 3.8.9
	 *
	 * @param array $data
	 * @return array
	 */
	private function get_data_format( $data ) {
		$format = array();

		foreach ( $data as $key => $value ) {
			$format[] = self::get_column_format( $key );
		}

		return $format;
	}

	/**
	 * Saves the purchase log back to the database
	 *
	 * @access public
	 * @since 3.8.9
	 *
	 * @return void
	 */
	public function save() {
		global $wpdb;

		do_action( 'wpsc_purchase_log_pre_save', $this );

		// $this->args['col'] is empty when the record is new and needs to
		// be inserted. Otherwise, it means we're performing an update
		$where_col = $this->args['col'];

		$result = false;

		if ( $where_col ) {
			$where_val = $this->args['value'];
			$where_format = self::get_column_format( $where_col );
			do_action( 'wpsc_purchase_log_pre_update', $this );
			self::delete_cache( $where_val, $where_col );
			$data = apply_filters( 'wpsc_purchase_log_update_data', $this->data );
			$format = $this->get_data_format( $data );
			$result = $wpdb->update( WPSC_TABLE_PURCHASE_LOGS, $data, array( $where_col => $where_val ), $format, array( $where_format ) );
			do_action( 'wpsc_purchase_log_update', $this );
		} else {
			do_action( 'wpsc_purchase_log_pre_insert', $this );
			$data = apply_filters( 'wpsc_purchase_log_insert_data', $this->data );
			$format = $this->get_data_format( $data );
			$result = $wpdb->insert( WPSC_TABLE_PURCHASE_LOGS, $data, $format );

			if ( $result ) {
				$this->set( 'id', $wpdb->insert_id );

				// set $this->args so that properties can be lazy loaded right after
				// the row is inserted into the db
				$this->args = array(
					'col'   => 'id',
					'value' => $this->get( 'id' ),
				);
			}

			do_action( 'wpsc_purchase_log_insert', $this );
		}

		if ( $this->is_status_changed ) {

			if ( $this->is_transaction_completed() ) {
				$this->update_downloadable_status();
			}

			$current_status          = $this->get( 'processed' );
			$previous_status         = $this->previous_status;
			$this->previous_status   = $current_status;
			$this->is_status_changed = false;

			do_action( 'wpsc_update_purchase_log_status', $this->get( 'id' ), $current_status, $previous_status, $this );
		}

		if ( ! empty( $this->meta_data ) ) {
			$this->save_meta();
		}

		do_action( 'wpsc_purchase_log_save', $this );

		return $result;
	}

	/**
	 * Save meta data for purchase log, if any was set via set().
	 *
	 * @access public
	 * @since  4.0
	 *
	 * @return WPSC_Query_Base  The current object (for method chaining)

Consider making the return type a bit more specific; maybe use WPSC_Purchase_Log.

	 */
	public function save_meta() {
		do_action( 'wpsc_purchase_log_pre_save_meta', $this );

		$meta = $this->get_meta();

		foreach ( $meta as $key => $value ) {
			wpsc_update_purchase_meta( $this->get( 'id' ), $key, $value );
		}

		do_action( 'wpsc_purchase_log_save_meta', $this );

		return $this;
	}

	private function update_downloadable_status() {
		global $wpdb;

		foreach ( $this->get_items() as $item ) {
			$wpdb->update(
				WPSC_TABLE_DOWNLOAD_STATUS,
				array(
					'active' => '1'
				),
				array(
					'cartid'  => $item->id,
					'purchid' => $this->get( 'id' ),
				)
			);
		}
	}

	public function have_downloads_locked() {
		global $wpdb;

		$sql = $wpdb->prepare( "SELECT `ip_number` FROM `" . WPSC_TABLE_DOWNLOAD_STATUS . "` WHERE `purchid` = %d ", $this->get( 'id' ) );
		$ip_number = $wpdb->get_var( $sql );

		return $ip_number;
	}

	/**
	 * Adds ability to retrieve a purchase log by a meta key or value.
	 *
	 * @since  4.0
	 *
	 * @param  string $key   Meta key. Optional.
	 * @param  string $value Meta value. Optional.
	 *
	 * @return false|WPSC_Purchase_Log  False if no log is found or meta key and value are both not provided. WPSC_Purchase_Log object if found.
	 */
	public static function get_log_by_meta( $key = '', $value = '' ) {

		if ( empty( $key ) && empty( $value ) ) {
			return false;
		}

		global $wpdb;

		if ( ! empty( $key ) && empty( $value ) ) {
			$sql = $wpdb->prepare( 'SELECT wpsc_purchase_id FROM ' . WPSC_TABLE_PURCHASE_META . ' WHERE meta_key = %s', $key );
		} else if ( empty( $key ) && ! empty( $value ) ) {
			$sql = $wpdb->prepare( 'SELECT wpsc_purchase_id FROM ' . WPSC_TABLE_PURCHASE_META . ' WHERE meta_value = %s', $value );
		} else {
			$sql = $wpdb->prepare( 'SELECT wpsc_purchase_id FROM ' . WPSC_TABLE_PURCHASE_META . ' WHERE meta_key = %s AND meta_value = %s', $key, $value );
		}

		$id = $wpdb->get_var( $sql );

		if ( $id ) {
			return new WPSC_Purchase_Log( $id );
		} else {
			return false;
		}
	}

	public function get_next_log_id() {
		if ( ! $this->exists() ) {
			return false;
		}

		global $wpdb;

		$sql = $wpdb->prepare(
			"SELECT MIN(id) FROM " . WPSC_TABLE_PURCHASE_LOGS . " WHERE id > %d",
			$this->get( 'id' )
		);

		return $wpdb->get_var( $sql );
	}

	public function get_previous_log_id() {
		if ( ! $this->exists() ) {
			return false;
		}

		global $wpdb;

		$sql = $wpdb->prepare(
			"SELECT MAX(id) FROM " . WPSC_TABLE_PURCHASE_LOGS . " WHERE id < %d",
			$this->get( 'id' )
		);

		return $wpdb->get_var( $sql );
	}

	public function is_transaction_completed() {
		return WPSC_Purchase_Log::is_order_status_completed( $this->get( 'processed' ) );
	}

	public function can_edit() {
		if ( null === $this->can_edit ) {
			$can_edit = current_user_can( 'edit_others_posts' ) && ! $this->is_transaction_completed();
			$this->can_edit = apply_filters( 'wpsc_can_edit_order', $can_edit, $this );
		}

		return $this->can_edit;
	}

	public function is_order_received() {
		return $this->get( 'processed' ) == self::ORDER_RECEIVED;
	}

	public function is_incomplete_sale() {
		return $this->get( 'processed' ) == self::INCOMPLETE_SALE;
	}

	public function is_accepted_payment() {
		return $this->get( 'processed' ) == self::ACCEPTED_PAYMENT;
	}

	public function is_job_dispatched() {
		return $this->get( 'processed' ) == self::JOB_DISPATCHED;
	}

	public function is_closed_order() {
		return $this->get( 'processed' ) == self::CLOSED_ORDER;
	}

	public function is_payment_declined() {
		return $this->get( 'processed' ) == self::PAYMENT_DECLINED;
	}

	public function is_refunded() {
		return $this->get( 'processed' ) == self::REFUNDED;
	}

	public function is_refund_pending() {
		return $this->get( 'processed' ) == self::REFUND_PENDING;
	}

	/*
	 * Utility methods using the $purchlogitem global.. Global usage to be replaced in the future.
	 *
	 * TODO: seriously get rid of all these badly coded purchaselogs.functions.php functions
	 * and wpsc_purchaselogs/wpsc_purchaselogs_items classes.
	 */

	/**
	 * Init the purchase log items for this purchase log.
	 *
	 * @since  4.0
	 *
	 * @return wpsc_purchaselogs_items|false The purhchase log item object or false.
	 */
	public function init_items() {
		global $purchlogitem;
		if ( ! $this->exists() ) {
			return false;
		}

		$form_data_obj = new WPSC_Checkout_Form_Data( $this->get( 'id' ) );
		$purchlogitem = new wpsc_purchaselogs_items( $this->get( 'id' ), $this, $form_data_obj );
	}

	public function buyers_name() {
		global $purchlogitem;

		if ( null === $this->buyers_name ) {
			$first_name = $last_name = '';

			if ( isset( $purchlogitem->userinfo['billingfirstname'] ) ) {
				$first_name = $purchlogitem->userinfo['billingfirstname']['value'];
			}

			if ( isset( $purchlogitem->userinfo['billinglastname'] ) ) {
				$last_name = ' ' . $purchlogitem->userinfo['billinglastname']['value'];
			}

			$this->buyers_name = trim( $first_name . $last_name );
		}

		return $this->buyers_name;
	}

	public function buyers_city() {
		global $purchlogitem;

		if ( null === $this->buyers_city ) {
			$this->buyers_city = isset( $purchlogitem->userinfo['billingcity']['value'] ) ? $purchlogitem->userinfo['billingcity']['value'] : '';
		}

		return $this->buyers_city;
	}

	public function buyers_email() {
		global $purchlogitem;

		if ( null === $this->buyers_email ) {
			$this->buyers_email = isset( $purchlogitem->userinfo['billingemail']['value'] ) ? $purchlogitem->userinfo['billingemail']['value'] : '';
		}

		return $this->buyers_email;
	}

	public function buyers_address() {
		global $purchlogitem;

		if ( null === $this->buyers_address ) {
			$this->buyers_address = isset( $purchlogitem->userinfo['billingaddress']['value'] ) ? nl2br( esc_html( $purchlogitem->userinfo['billingaddress']['value'] ) ) : '';
		}

		return $this->buyers_address;
	}

	public function buyers_state_and_postcode() {
		global $purchlogitem;

		if ( null === $this->buyers_state_and_postcode ) {

			if ( is_numeric( $this->get( 'billing_region' ) ) ) {
				$state = wpsc_get_region( $this->get( 'billing_region' ) );
			} else {
				$state = $purchlogitem->userinfo['billingstate']['value'];
				$state = is_numeric( $state ) ? wpsc_get_region( $state ) : $state;
			}

			$output = esc_html( $state );

			if ( isset( $purchlogitem->userinfo['billingpostcode']['value'] ) && ! empty( $purchlogitem->userinfo['billingpostcode']['value'] ) ) {
				if ( $output ) {
					$output .= ', '; // TODO determine if it's ok to make this a space only (like shipping_state_and_postcode)
				}
				$output .= $purchlogitem->userinfo['billingpostcode']['value'];
			}

			$this->buyers_state_and_postcode = $output;
		}

		return $this->buyers_state_and_postcode;
	}

	public function buyers_country() {
		global $purchlogitem;

		if ( null === $this->buyers_country ) {
			$this->buyers_country = isset( $purchlogitem->userinfo['billingcountry']['value'] ) ? wpsc_get_country( $purchlogitem->userinfo['billingcountry']['value'] ) : '';
		}

		return $this->buyers_country;
	}

	public function buyers_phone() {
		global $purchlogitem;

		if ( null === $this->buyers_phone ) {
			$this->buyers_phone = isset( $purchlogitem->userinfo['billingphone']['value'] ) ? $purchlogitem->userinfo['billingphone']['value'] : '';
		}

		return $this->buyers_phone;
	}

	public function shipping_name() {
		global $purchlogitem;

		if ( null === $this->shipping_name ) {
			$this->shipping_name = isset( $purchlogitem->shippinginfo['shippingfirstname']['value'] ) ? $purchlogitem->shippinginfo['shippingfirstname']['value'] : '';
		}

		return $this->shipping_name;
	}

	public function shipping_city() {
		global $purchlogitem;

		if ( null === $this->shipping_city ) {
			$this->shipping_city = isset( $purchlogitem->shippinginfo['shippingcity']['value'] ) ? $purchlogitem->shippinginfo['shippingcity']['value'] : '';
		}

		return $this->shipping_city;
	}

	public function shipping_address() {
		global $purchlogitem;

		if ( null === $this->shipping_address ) {
			$this->shipping_address = isset( $purchlogitem->shippinginfo['shippingaddress']['value'] ) ? nl2br( esc_html( $purchlogitem->shippinginfo['shippingaddress']['value'] ) ) : '';
		}

		return $this->shipping_address;
	}

	public function shipping_state_and_postcode() {
		global $purchlogitem;

		if ( null === $this->shipping_state_and_postcode ) {
			if ( is_numeric( $this->get( 'shipping_region' ) ) ) {
				$output = wpsc_get_region( $this->get( 'shipping_region' ) );
			} else {
				$state = $purchlogitem->shippinginfo['shippingstate']['value'];
				$output = is_numeric( $state ) ? wpsc_get_region( $state ) : $state;
			}

			if ( !empty( $purchlogitem->shippinginfo['shippingpostcode']['value'] ) ){
				if ( $output ) {
					$output .= ' ';
				}

				$output .= $purchlogitem->shippinginfo['shippingpostcode']['value'];
			}

			$this->shipping_state_and_postcode = $output;
		}

		return $this->shipping_state_and_postcode;
	}

	public function shipping_country() {
		global $purchlogitem;

		if ( null === $this->shipping_country ) {
			$this->shipping_country = isset( $purchlogitem->shippinginfo['shippingcountry'] )
				? wpsc_get_country( $purchlogitem->shippinginfo['shippingcountry']['value'] )
				: '';
		}

		return $this->shipping_country;
	}

	public function payment_method() {
		global $nzshpcrt_gateways;

		if ( null === $this->payment_method ) {
			if ( 'wpsc_merchant_testmode' == $this->get( 'gateway' ) ) {
				$this->payment_method = __( 'Manual Payment', 'wp-e-commerce' );
			} else {
				foreach ( (array) $nzshpcrt_gateways as $gateway ) {
					if ( isset( $gateway['internalname'] ) && $gateway['internalname'] == $this->get( 'gateway' ) ) {
						$this->payment_method = $gateway['name'];
					}
				}

				if ( ! $this->payment_method ) {
					$this->payment_method = $this->get( 'gateway' );
				}
			}
		}

		return $this->payment_method;
	}

	public function shipping_method() {
		global $wpsc_shipping_modules;

		if ( null === $this->shipping_method ) {

			if ( ! empty( $wpsc_shipping_modules[ $this->get( 'shipping_method' ) ] ) ) {
				$this->shipping_method = $wpsc_shipping_modules[ $this->get( 'shipping_method' ) ]->getName();
			} else {
				$this->shipping_method = $this->get( 'shipping_method' );
			}

Blank line found after control structure

		}

		return $this->shipping_method;
	}

	/**
	 * Returns base shipping should make a function to calculate items shipping as well
	 *
	 * @since  4.0
	 *
	 * @param  boolean $numeric Return numeric value.
	 *
	 * @return mixed
	 */
	public function discount( $numeric = false ) {
		$discount = $this->get( 'discount_value' );
		if ( ! $numeric ) {
			$discount = wpsc_currency_display( $discount, array( 'display_as_html' => false ) );
		}

		return $discount;
	}

	/**
	 * Returns base shipping should make a function to calculate items shipping as well
	 *
	 * @since  4.0
	 *
	 * @param  boolean $numeric       Return numeric value.
	 * @param  boolean $include_items Whether to calculate per-item-shipping.
	 *
	 * @return mixed
	 */
	public function shipping( $numeric = false, $include_items = false ) {
		$total_shipping = $this->get( 'base_shipping' );

		if ( $include_items ) {
			$total_shipping = $this->meta_data['total_shipping'];
		}

		if ( ! $numeric ) {
			$total_shipping = wpsc_currency_display( $total_shipping, array( 'display_as_html' => false ) );
		}

		return $total_shipping;
	}

	/**
	 * Returns taxes total.
	 *
	 * @since  4.0
	 *
	 * @param  boolean $numeric Return numeric value.
	 *
	 * @return mixed
	 */
	public function taxes( $numeric = false ) {
		$taxes = $this->get( 'wpec_taxes_total' );

		if ( ! $numeric ) {
			$taxes = wpsc_currency_display( $taxes, array( 'display_as_html' => false ) );
		}

		return $taxes;
	}

	public function total_price() {
		global $purchlogitem;

		$total = $purchlogitem->totalAmount - $this->discount( true ) + $this->shipping( true ) + $this->taxes( true );
		return wpsc_currency_display( $total, array( 'display_as_html' => false ) );
	}

}