AyeCode / invoicing

includes/api/class-wpinv-rest-items-controller.php

Spacing +24 added lines, -24 removed lines

@@ -5,7 +5,7 @@ 
  * @version 1.0.19
  */
 
-defined( 'ABSPATH' ) || exit;
+defined('ABSPATH') || exit;
 
 /**
  * REST API items controller class.
@@ -42,9 +42,9 @@ 
 	 *
 	 * @see register_rest_route()
 	 */
-	public function register_namespace_routes( $namespace ) {
+	public function register_namespace_routes($namespace) {
 
-		parent::register_namespace_routes( $namespace );
+		parent::register_namespace_routes($namespace);
 
 		register_rest_route(
 			$this->namespace,
@@ -52,8 +52,8 @@ 
 			array(
 				array(
 					'methods'             => WP_REST_Server::READABLE,
-					'callback'            => array( $this, 'get_item_types' ),
-					'permission_callback' => array( $this, 'get_items_permissions_check' ),
+					'callback'            => array($this, 'get_item_types'),
+					'permission_callback' => array($this, 'get_items_permissions_check'),
 				),
 			)
 		);
@@ -69,7 +69,7 @@ 
 	 * @return WP_REST_Response|WP_Error Response object on success, or WP_Error object on failure.
 	 */
 	public function get_item_types() {
-		return rest_ensure_response( wpinv_get_item_types() );
+		return rest_ensure_response(wpinv_get_item_types());
 	}
 
     /**
@@ -87,13 +87,13 @@ 
 
 				// Item types
 				'type' => array(
-					'description'       => __( 'Type of items to fetch.', 'invoicing' ),
-					'type'              => array( 'array', 'string' ),
+					'description'       => __('Type of items to fetch.', 'invoicing'),
+					'type'              => array('array', 'string'),
 					'default'           => 'any',
 					'validate_callback' => 'rest_validate_request_arg',
 					'sanitize_callback' => 'wpinv_parse_list',
 					'items'             => array(
-						'enum' => array_merge( array( 'any' ), wpinv_item_types() ),
+						'enum' => array_merge(array('any'), wpinv_item_types()),
 						'type' => 'string',
 					),
 				),
@@ -102,7 +102,7 @@ 
 		);
 
 		// Filter collection parameters for the items controller.
-		return apply_filters( 'getpaid_rest_items_collection_params', $params, $this );
+		return apply_filters('getpaid_rest_items_collection_params', $params, $this);
 
 	}
 
@@ -114,26 +114,26 @@ 
 	 * @param WP_REST_Request $request Request object.
 	 * @return array          $query_args
 	 */
-	protected function prepare_items_query( $prepared_args = array(), $request = null ) {
+	protected function prepare_items_query($prepared_args = array(), $request = null) {
 
-		$query_args = parent::prepare_items_query( $prepared_args );
+		$query_args = parent::prepare_items_query($prepared_args);
 
 		// Retrieve items by type.
-		if ( ! in_array( 'any', $request['type'] ) ) {
+		if (!in_array('any', $request['type'])) {
 
-			if ( empty( $query_args['meta_query'] ) ) {
+			if (empty($query_args['meta_query'])) {
 				$query_args['meta_query'] = array();
 			}
 
 			$query_args['meta_query'][] = array(
 				'key'     => '_wpinv_type',
-				'value'   => implode( ',', $request['type'] ),
+				'value'   => implode(',', $request['type']),
 				'compare' => 'IN',
 			);
 
 		}
 
-		return apply_filters( 'getpaid_rest_items_prepare_items_query', $query_args, $request, $this );
+		return apply_filters('getpaid_rest_items_prepare_items_query', $query_args, $request, $this);
 
 	}
 
@@ -145,7 +145,7 @@ 
 	 * @return array A list of registered item statuses.
 	 */
 	public function get_post_statuses() {
-		return array( 'draft', 'pending', 'publish' );
+		return array('draft', 'pending', 'publish');
 	}
 
 	/**
@@ -156,27 +156,27 @@ 
 	 * @param  string       $field_key The key to check for.
 	 * @return bool
 	 */
-	public function object_supports_field( $item, $field_key ) {
+	public function object_supports_field($item, $field_key) {
 
-		if ( 'minimum_price' == $field_key && ! $item->user_can_set_their_price() ) {
+		if ('minimum_price' == $field_key && !$item->user_can_set_their_price()) {
 			return false;
 		}
 
-		foreach ( wpinv_parse_list( 'initial_price the_initial_price recurring_price the_recurring_price recurring_period recurring_interval recurring_limit is_free_trial trial_period trial_interval first_renewal_date' ) as $key ) {
+		foreach (wpinv_parse_list('initial_price the_initial_price recurring_price the_recurring_price recurring_period recurring_interval recurring_limit is_free_trial trial_period trial_interval first_renewal_date') as $key) {
 
-			if ( $key == $field_key && ! $item->is_recurring() ) {
+			if ($key == $field_key && !$item->is_recurring()) {
 				return false;
 			}
 }
 
-		foreach ( wpinv_parse_list( 'trial_period trial_interval' ) as $key ) {
+		foreach (wpinv_parse_list('trial_period trial_interval') as $key) {
 
-			if ( $key == $field_key && ! $item->has_free_trial() ) {
+			if ($key == $field_key && !$item->has_free_trial()) {
 				return false;
 			}
 }
 
-		return parent::object_supports_field( $item, $field_key );
+		return parent::object_supports_field($item, $field_key);
 	}
 
 }

includes/api/class-getpaid-rest-report-sales-controller.php

Spacing +154 added lines, -154 removed lines

@@ -9,7 +9,7 @@ 
  * @since   2.0.0
  */
 
-defined( 'ABSPATH' ) || exit;
+defined('ABSPATH') || exit;
 
 /**
  * GetPaid REST reports controller class.
@@ -46,7 +46,7 @@ 
 	 *
 	 * @see register_rest_route()
 	 */
-	public function register_namespace_routes( $namespace ) {
+	public function register_namespace_routes($namespace) {
 
 		// Get sales report.
 		register_rest_route(
@@ -55,11 +55,11 @@ 
 			array(
 				array(
 					'methods'             => WP_REST_Server::READABLE,
-					'callback'            => array( $this, 'get_items' ),
-					'permission_callback' => array( $this, 'get_items_permissions_check' ),
+					'callback'            => array($this, 'get_items'),
+					'permission_callback' => array($this, 'get_items_permissions_check'),
 					'args'                => $this->get_collection_params(),
 				),
-				'schema' => array( $this, 'get_public_item_schema' ),
+				'schema' => array($this, 'get_public_item_schema'),
 			)
 		);
 
@@ -72,10 +72,10 @@ 
 	 * @param WP_REST_Request $request Full data about the request.
 	 * @return WP_Error|boolean
 	 */
-	public function get_items_permissions_check( $request ) {
+	public function get_items_permissions_check($request) {
 
-		if ( ! wpinv_current_user_can_manage_invoicing() ) {
-			return new WP_Error( 'rest_cannot_view', __( 'Sorry, you cannot list resources.', 'invoicing' ), array( 'status' => rest_authorization_required_code() ) );
+		if (!wpinv_current_user_can_manage_invoicing()) {
+			return new WP_Error('rest_cannot_view', __('Sorry, you cannot list resources.', 'invoicing'), array('status' => rest_authorization_required_code()));
 		}
 
 		return true;
@@ -87,12 +87,12 @@ 
 	 * @param WP_REST_Request $request
 	 * @return array|WP_Error
 	 */
-	public function get_items( $request ) {
+	public function get_items($request) {
 		$data   = array();
-		$item   = $this->prepare_item_for_response( null, $request );
-		$data[] = $this->prepare_response_for_collection( $item );
+		$item   = $this->prepare_item_for_response(null, $request);
+		$data[] = $this->prepare_response_for_collection($item);
 
-		return rest_ensure_response( $data );
+		return rest_ensure_response($data);
 	}
 
 	/**
@@ -102,164 +102,164 @@ 
 	 * @param WP_REST_Request $request Request object.
 	 * @return WP_REST_Response $response Response data.
 	 */
-	public function prepare_item_for_response( $_, $request ) {
+	public function prepare_item_for_response($_, $request) {
 
 		// Set report range.
-		$this->report_range = $this->get_date_range( $request );
+		$this->report_range = $this->get_date_range($request);
 
 		$report_data     = $this->get_report_data();
 		$period_totals   = array();
 
 		// Setup period totals by ensuring each period in the interval has data.
-		$start_date      = strtotime( $this->report_range['after'] ) + DAY_IN_SECONDS;
+		$start_date      = strtotime($this->report_range['after']) + DAY_IN_SECONDS;
 
-		if ( 'month' === $this->groupby ) {
-			$start_date      = strtotime( date( 'Y-m-01', $start_date ) );
+		if ('month' === $this->groupby) {
+			$start_date = strtotime(date('Y-m-01', $start_date));
 		}
 
-		for ( $i = 0; $i < $this->interval; $i++ ) {
+		for ($i = 0; $i < $this->interval; $i++) {
 
-			switch ( $this->groupby ) {
+			switch ($this->groupby) {
 				case 'day':
-					$time = date( 'Y-m-d', strtotime( "+{$i} DAY", $start_date ) );
+					$time = date('Y-m-d', strtotime("+{$i} DAY", $start_date));
 					break;
 				default:
-					$time = date( 'Y-m', strtotime( "+{$i} MONTH", $start_date ) );
+					$time = date('Y-m', strtotime("+{$i} MONTH", $start_date));
 					break;
 			}
 
 			// Set the defaults for each period.
-			$period_totals[ $time ] = array(
+			$period_totals[$time] = array(
 				'invoices'          => 0,
 				'items'             => 0,
 				'refunded_items'    => 0,
-				'refunded_tax'      => wpinv_round_amount( 0.00 ),
-				'subtotal'          => wpinv_round_amount( 0.00 ),
-				'refunded_subtotal' => wpinv_round_amount( 0.00 ),
-				'refunded_fees'     => wpinv_round_amount( 0.00 ),
-				'discount'          => wpinv_round_amount( 0.00 ),
+				'refunded_tax'      => wpinv_round_amount(0.00),
+				'subtotal'          => wpinv_round_amount(0.00),
+				'refunded_subtotal' => wpinv_round_amount(0.00),
+				'refunded_fees'     => wpinv_round_amount(0.00),
+				'discount'          => wpinv_round_amount(0.00),
 			);
 
-			foreach ( array_keys( wpinv_get_report_graphs() ) as $key ) {
-				if ( ! isset( $period_totals[ $time ][ $key ] ) ) {
-					$period_totals[ $time ][ $key ] = wpinv_round_amount( 0.00 );
+			foreach (array_keys(wpinv_get_report_graphs()) as $key) {
+				if (!isset($period_totals[$time][$key])) {
+					$period_totals[$time][$key] = wpinv_round_amount(0.00);
 				}
 			}
 }
 
 		// add total sales, total invoice count, total tax for each period
-		$date_format = ( 'day' === $this->groupby ) ? 'Y-m-d' : 'Y-m';
-		foreach ( $report_data->invoices as $invoice ) {
-			$time = date( $date_format, strtotime( $invoice->post_date ) );
+		$date_format = ('day' === $this->groupby) ? 'Y-m-d' : 'Y-m';
+		foreach ($report_data->invoices as $invoice) {
+			$time = date($date_format, strtotime($invoice->post_date));
 
-			if ( ! isset( $period_totals[ $time ] ) ) {
+			if (!isset($period_totals[$time])) {
 				continue;
 			}
 
-			$period_totals[ $time ]['sales']    = wpinv_round_amount( $invoice->total_sales );
-			$period_totals[ $time ]['tax']      = wpinv_round_amount( $invoice->total_tax );
-			$period_totals[ $time ]['subtotal'] = wpinv_round_amount( $invoice->subtotal );
-			$period_totals[ $time ]['fees']     = wpinv_round_amount( $invoice->total_fees );
+			$period_totals[$time]['sales']    = wpinv_round_amount($invoice->total_sales);
+			$period_totals[$time]['tax']      = wpinv_round_amount($invoice->total_tax);
+			$period_totals[$time]['subtotal'] = wpinv_round_amount($invoice->subtotal);
+			$period_totals[$time]['fees']     = wpinv_round_amount($invoice->total_fees);
 
 		}
 
-		foreach ( $report_data->refunds as $invoice ) {
-			$time = date( $date_format, strtotime( $invoice->post_date ) );
+		foreach ($report_data->refunds as $invoice) {
+			$time = date($date_format, strtotime($invoice->post_date));
 
-			if ( ! isset( $period_totals[ $time ] ) ) {
+			if (!isset($period_totals[$time])) {
 				continue;
 			}
 
-			$period_totals[ $time ]['refunds']           = wpinv_round_amount( $invoice->total_sales );
-			$period_totals[ $time ]['refunded_tax']      = wpinv_round_amount( $invoice->total_tax );
-			$period_totals[ $time ]['refunded_subtotal'] = wpinv_round_amount( $invoice->subtotal );
-			$period_totals[ $time ]['refunded_fees']     = wpinv_round_amount( $invoice->total_fees );
+			$period_totals[$time]['refunds']           = wpinv_round_amount($invoice->total_sales);
+			$period_totals[$time]['refunded_tax']      = wpinv_round_amount($invoice->total_tax);
+			$period_totals[$time]['refunded_subtotal'] = wpinv_round_amount($invoice->subtotal);
+			$period_totals[$time]['refunded_fees']     = wpinv_round_amount($invoice->total_fees);
 
 		}
 
-		foreach ( $report_data->invoice_counts as $invoice ) {
-			$time = date( $date_format, strtotime( $invoice->post_date ) );
+		foreach ($report_data->invoice_counts as $invoice) {
+			$time = date($date_format, strtotime($invoice->post_date));
 
-			if ( isset( $period_totals[ $time ] ) ) {
-				$period_totals[ $time ]['invoices']   = (int) $invoice->count;
+			if (isset($period_totals[$time])) {
+				$period_totals[$time]['invoices'] = (int) $invoice->count;
 			}
 }
 
 		// Add total invoice items for each period.
-		foreach ( $report_data->invoice_items as $invoice_item ) {
-			$time = ( 'day' === $this->groupby ) ? date( 'Y-m-d', strtotime( $invoice_item->post_date ) ) : date( 'Y-m', strtotime( $invoice_item->post_date ) );
+		foreach ($report_data->invoice_items as $invoice_item) {
+			$time = ('day' === $this->groupby) ? date('Y-m-d', strtotime($invoice_item->post_date)) : date('Y-m', strtotime($invoice_item->post_date));
 
-			if ( isset( $period_totals[ $time ] ) ) {
-				$period_totals[ $time ]['items'] = (int) $invoice_item->invoice_item_count;
+			if (isset($period_totals[$time])) {
+				$period_totals[$time]['items'] = (int) $invoice_item->invoice_item_count;
 			}
 }
 
 		// Add total discount for each period.
-		foreach ( $report_data->coupons as $discount ) {
-			$time = ( 'day' === $this->groupby ) ? date( 'Y-m-d', strtotime( $discount->post_date ) ) : date( 'Y-m', strtotime( $discount->post_date ) );
+		foreach ($report_data->coupons as $discount) {
+			$time = ('day' === $this->groupby) ? date('Y-m-d', strtotime($discount->post_date)) : date('Y-m', strtotime($discount->post_date));
 
-			if ( isset( $period_totals[ $time ] ) ) {
-				$period_totals[ $time ]['discount'] = wpinv_round_amount( $discount->discount_amount );
+			if (isset($period_totals[$time])) {
+				$period_totals[$time]['discount'] = wpinv_round_amount($discount->discount_amount);
 			}
 }
 
 		// Extra fields.
-		foreach ( array_keys( wpinv_get_report_graphs() ) as $key ) {
+		foreach (array_keys(wpinv_get_report_graphs()) as $key) {
 
 			// Abort unprepared.
-			if ( ! isset( $report_data->$key ) ) {
+			if (!isset($report_data->$key)) {
 				continue;
 			}
 
 			// Abort defaults.
-			if ( in_array( $key, array( 'sales', 'refunds', 'tax', 'fees', 'discount', 'invoices', 'items' ) ) ) {
+			if (in_array($key, array('sales', 'refunds', 'tax', 'fees', 'discount', 'invoices', 'items'))) {
 				continue;
 			}
 
 			// Set values.
-			foreach ( $report_data->$key as $item ) {
-				$time = ( 'day' === $this->groupby ) ? date( 'Y-m-d', strtotime( $item->date ) ) : date( 'Y-m', strtotime( $item->date ) );
+			foreach ($report_data->$key as $item) {
+				$time = ('day' === $this->groupby) ? date('Y-m-d', strtotime($item->date)) : date('Y-m', strtotime($item->date));
 
-				if ( isset( $period_totals[ $time ] ) ) {
-					$period_totals[ $time ][ $key ] = wpinv_round_amount( $item->val );
+				if (isset($period_totals[$time])) {
+					$period_totals[$time][$key] = wpinv_round_amount($item->val);
 				}
 			}
 
-			unset( $report_data->$key );
+			unset($report_data->$key);
 		}
 
 		$report_data->totals            = $period_totals;
 		$report_data->grouped_by        = $this->groupby;
-		$report_data->interval          = max( $this->interval, 1 );
+		$report_data->interval          = max($this->interval, 1);
 		$report_data->currency          = wpinv_get_currency();
 		$report_data->currency_symbol   = wpinv_currency_symbol();
 		$report_data->currency_position = wpinv_currency_position();
 		$report_data->decimal_places    = wpinv_decimals();
 		$report_data->thousands_sep     = wpinv_thousands_separator();
 		$report_data->decimals_sep      = wpinv_decimal_separator();
-		$report_data->start_date        = date( 'Y-m-d', strtotime( $this->report_range['after'] ) + DAY_IN_SECONDS );
-		$report_data->end_date          = date( 'Y-m-d', strtotime( $this->report_range['before'] ) - DAY_IN_SECONDS );
-		$report_data->start_date_locale = getpaid_format_date( date( 'Y-m-d', strtotime( $this->report_range['after'] ) + DAY_IN_SECONDS ) );
-		$report_data->end_date_locale   = getpaid_format_date( date( 'Y-m-d', strtotime( $this->report_range['before'] ) - DAY_IN_SECONDS ) );
+		$report_data->start_date        = date('Y-m-d', strtotime($this->report_range['after']) + DAY_IN_SECONDS);
+		$report_data->end_date          = date('Y-m-d', strtotime($this->report_range['before']) - DAY_IN_SECONDS);
+		$report_data->start_date_locale = getpaid_format_date(date('Y-m-d', strtotime($this->report_range['after']) + DAY_IN_SECONDS));
+		$report_data->end_date_locale   = getpaid_format_date(date('Y-m-d', strtotime($this->report_range['before']) - DAY_IN_SECONDS));
 		$report_data->decimals_sep      = wpinv_decimal_separator();
 
-		$context = ! empty( $request['context'] ) ? $request['context'] : 'view';
+		$context = !empty($request['context']) ? $request['context'] : 'view';
 		$data    = $report_data;
-		unset( $data->invoice_counts, $data->invoices, $data->coupons, $data->refunds, $data->invoice_items );
-		$data    = $this->add_additional_fields_to_object( (array) $data, $request );
-		$data    = $this->filter_response_by_context( $data, $context );
+		unset($data->invoice_counts, $data->invoices, $data->coupons, $data->refunds, $data->invoice_items);
+		$data    = $this->add_additional_fields_to_object((array) $data, $request);
+		$data    = $this->filter_response_by_context($data, $context);
 
 		// Wrap the data in a response object.
-		$response = rest_ensure_response( $data );
+		$response = rest_ensure_response($data);
 		$response->add_links(
             array(
 				'about' => array(
-					'href' => rest_url( sprintf( '%s/reports', $this->namespace ) ),
+					'href' => rest_url(sprintf('%s/reports', $this->namespace)),
 				),
             )
         );
 
-		return apply_filters( 'getpaid_rest_prepare_report_sales', $response, $report_data, $request );
+		return apply_filters('getpaid_rest_prepare_report_sales', $response, $report_data, $request);
 	}
 
 	/**
@@ -268,7 +268,7 @@ 
 	 * @return stdClass
 	 */
 	public function get_report_data() {
-		if ( empty( $this->report_data ) ) {
+		if (empty($this->report_data)) {
 			$this->query_report_data();
 		}
 		return $this->report_data;
@@ -291,31 +291,31 @@ 
 		);
 
 		// Calculated totals.
-		$this->report_data->total_tax          = wpinv_round_amount( array_sum( wp_list_pluck( $this->report_data->invoices, 'total_tax' ) ) );
-		$this->report_data->total_sales        = wpinv_round_amount( array_sum( wp_list_pluck( $this->report_data->invoices, 'total_sales' ) ) );
-		$this->report_data->total_discount     = wpinv_round_amount( array_sum( wp_list_pluck( $this->report_data->invoices, 'total_discount' ) ) );
-		$this->report_data->total_fees         = wpinv_round_amount( array_sum( wp_list_pluck( $this->report_data->invoices, 'total_fees' ) ) );
-		$this->report_data->subtotal           = wpinv_round_amount( array_sum( wp_list_pluck( $this->report_data->invoices, 'subtotal' ) ) );
-		$this->report_data->net_sales          = wpinv_round_amount( $this->report_data->total_sales - max( 0, $this->report_data->total_tax ) );
-		$this->report_data->total_refunded_tax = wpinv_round_amount( array_sum( wp_list_pluck( $this->report_data->refunds, 'total_tax' ) ) );
-		$this->report_data->total_refunds      = wpinv_round_amount( array_sum( wp_list_pluck( $this->report_data->refunds, 'total_sales' ) ) );
-		$this->report_data->refunded_discount  = wpinv_round_amount( array_sum( wp_list_pluck( $this->report_data->refunds, 'total_discount' ) ) );
-		$this->report_data->refunded_fees      = wpinv_round_amount( array_sum( wp_list_pluck( $this->report_data->refunds, 'total_fees' ) ) );
-		$this->report_data->refunded_subtotal  = wpinv_round_amount( array_sum( wp_list_pluck( $this->report_data->refunds, 'subtotal' ) ) );
-		$this->report_data->net_refunds        = wpinv_round_amount( $this->report_data->total_refunds + max( 0, $this->report_data->total_refunded_tax ) );
+		$this->report_data->total_tax          = wpinv_round_amount(array_sum(wp_list_pluck($this->report_data->invoices, 'total_tax')));
+		$this->report_data->total_sales        = wpinv_round_amount(array_sum(wp_list_pluck($this->report_data->invoices, 'total_sales')));
+		$this->report_data->total_discount     = wpinv_round_amount(array_sum(wp_list_pluck($this->report_data->invoices, 'total_discount')));
+		$this->report_data->total_fees         = wpinv_round_amount(array_sum(wp_list_pluck($this->report_data->invoices, 'total_fees')));
+		$this->report_data->subtotal           = wpinv_round_amount(array_sum(wp_list_pluck($this->report_data->invoices, 'subtotal')));
+		$this->report_data->net_sales          = wpinv_round_amount($this->report_data->total_sales - max(0, $this->report_data->total_tax));
+		$this->report_data->total_refunded_tax = wpinv_round_amount(array_sum(wp_list_pluck($this->report_data->refunds, 'total_tax')));
+		$this->report_data->total_refunds      = wpinv_round_amount(array_sum(wp_list_pluck($this->report_data->refunds, 'total_sales')));
+		$this->report_data->refunded_discount  = wpinv_round_amount(array_sum(wp_list_pluck($this->report_data->refunds, 'total_discount')));
+		$this->report_data->refunded_fees      = wpinv_round_amount(array_sum(wp_list_pluck($this->report_data->refunds, 'total_fees')));
+		$this->report_data->refunded_subtotal  = wpinv_round_amount(array_sum(wp_list_pluck($this->report_data->refunds, 'subtotal')));
+		$this->report_data->net_refunds        = wpinv_round_amount($this->report_data->total_refunds + max(0, $this->report_data->total_refunded_tax));
 
 		// Calculate average based on net.
-		$this->report_data->average_sales       = wpinv_round_amount( $this->report_data->net_sales / max( $this->interval, 1 ), 2 );
-		$this->report_data->average_total_sales = wpinv_round_amount( $this->report_data->total_sales / max( $this->interval, 1 ), 2 );
+		$this->report_data->average_sales       = wpinv_round_amount($this->report_data->net_sales / max($this->interval, 1), 2);
+		$this->report_data->average_total_sales = wpinv_round_amount($this->report_data->total_sales / max($this->interval, 1), 2);
 
 		// Total invoices in this period, even if refunded.
-		$this->report_data->total_invoices = absint( array_sum( wp_list_pluck( $this->report_data->invoice_counts, 'count' ) ) );
+		$this->report_data->total_invoices = absint(array_sum(wp_list_pluck($this->report_data->invoice_counts, 'count')));
 
 		// Items invoiced in this period, even if refunded.
-		$this->report_data->total_items = absint( array_sum( wp_list_pluck( $this->report_data->invoice_items, 'invoice_item_count' ) ) );
+		$this->report_data->total_items = absint(array_sum(wp_list_pluck($this->report_data->invoice_items, 'invoice_item_count')));
 
 		// 3rd party filtering of report data
-		$this->report_data = apply_filters( 'getpaid_rest_api_filter_report_data', $this->report_data, $this );
+		$this->report_data = apply_filters('getpaid_rest_api_filter_report_data', $this->report_data, $this);
 	}
 
 	/**
@@ -340,11 +340,11 @@ 
 						'name'     => 'post_date',
 					),
 				),
-				'group_by'       => $this->get_group_by_sql( 'posts.post_date' ),
+				'group_by'       => $this->get_group_by_sql('posts.post_date'),
 				'order_by'       => 'post_date ASC',
 				'query_type'     => 'get_results',
 				'filter_range'   => $this->report_range,
-				'invoice_status' => array( 'publish', 'wpi-processing', 'wpi-onhold', 'wpi-refunded', 'wpi-renewal' ),
+				'invoice_status' => array('publish', 'wpi-processing', 'wpi-onhold', 'wpi-refunded', 'wpi-renewal'),
 			)
 		);
 
@@ -371,11 +371,11 @@ 
 						'name'     => 'post_date',
 					),
 				),
-				'group_by'       => $this->get_group_by_sql( 'posts.post_date' ),
+				'group_by'       => $this->get_group_by_sql('posts.post_date'),
 				'order_by'       => 'post_date ASC',
 				'query_type'     => 'get_results',
 				'filter_range'   => $this->report_range,
-				'invoice_status' => array( 'publish', 'wpi-processing', 'wpi-onhold', 'wpi-refunded', 'wpi-renewal' ),
+				'invoice_status' => array('publish', 'wpi-processing', 'wpi-onhold', 'wpi-refunded', 'wpi-renewal'),
 			)
 		);
 
@@ -402,11 +402,11 @@ 
 						'name'     => 'post_date',
 					),
 				),
-				'group_by'       => $this->get_group_by_sql( 'posts.post_date' ),
+				'group_by'       => $this->get_group_by_sql('posts.post_date'),
 				'order_by'       => 'post_date ASC',
 				'query_type'     => 'get_results',
 				'filter_range'   => $this->report_range,
-				'invoice_status' => array( 'publish', 'wpi-processing', 'wpi-onhold', 'wpi-refunded', 'wpi-renewal' ),
+				'invoice_status' => array('publish', 'wpi-processing', 'wpi-onhold', 'wpi-refunded', 'wpi-renewal'),
 			)
 		);
 
@@ -430,7 +430,7 @@ 
 				),
 				'query_type'     => 'get_var',
 				'filter_range'   => $this->report_range,
-				'invoice_status' => array( 'wpi-refunded' ),
+				'invoice_status' => array('wpi-refunded'),
 			)
 		);
 
@@ -477,11 +477,11 @@ 
 						'name'     => 'post_date',
 					),
 				),
-				'group_by'       => $this->get_group_by_sql( 'posts.post_date' ),
+				'group_by'       => $this->get_group_by_sql('posts.post_date'),
 				'order_by'       => 'post_date ASC',
 				'query_type'     => 'get_results',
 				'filter_range'   => $this->report_range,
-				'invoice_status' => array( 'publish', 'wpi-processing', 'wpi-onhold', 'wpi-renewal' ),
+				'invoice_status' => array('publish', 'wpi-processing', 'wpi-onhold', 'wpi-renewal'),
 			)
 		);
 
@@ -528,11 +528,11 @@ 
 						'name'     => 'post_date',
 					),
 				),
-				'group_by'       => $this->get_group_by_sql( 'posts.post_date' ),
+				'group_by'       => $this->get_group_by_sql('posts.post_date'),
 				'order_by'       => 'post_date ASC',
 				'query_type'     => 'get_results',
 				'filter_range'   => $this->report_range,
-				'invoice_status' => array( 'wpi-refunded' ),
+				'invoice_status' => array('wpi-refunded'),
 			)
 		);
 
@@ -551,154 +551,154 @@ 
 			'type'       => 'object',
 			'properties' => array(
 				'total_sales'         => array(
-					'description' => __( 'Gross sales in the period.', 'invoicing' ),
+					'description' => __('Gross sales in the period.', 'invoicing'),
 					'type'        => 'string',
-					'context'     => array( 'view' ),
+					'context'     => array('view'),
 					'readonly'    => true,
 				),
 				'net_sales'           => array(
-					'description' => __( 'Net sales in the period.', 'invoicing' ),
+					'description' => __('Net sales in the period.', 'invoicing'),
 					'type'        => 'string',
-					'context'     => array( 'view' ),
+					'context'     => array('view'),
 					'readonly'    => true,
 				),
 				'average_sales'       => array(
-					'description' => __( 'Average net daily sales.', 'invoicing' ),
+					'description' => __('Average net daily sales.', 'invoicing'),
 					'type'        => 'string',
-					'context'     => array( 'view' ),
+					'context'     => array('view'),
 					'readonly'    => true,
 				),
 				'average_total_sales' => array(
-					'description' => __( 'Average gross daily sales.', 'invoicing' ),
+					'description' => __('Average gross daily sales.', 'invoicing'),
 					'type'        => 'string',
-					'context'     => array( 'view' ),
+					'context'     => array('view'),
 					'readonly'    => true,
 				),
 				'total_invoices'      => array(
-					'description' => __( 'Number of paid invoices.', 'invoicing' ),
+					'description' => __('Number of paid invoices.', 'invoicing'),
 					'type'        => 'integer',
-					'context'     => array( 'view' ),
+					'context'     => array('view'),
 					'readonly'    => true,
 				),
 				'total_items'         => array(
-					'description' => __( 'Number of items purchased.', 'invoicing' ),
+					'description' => __('Number of items purchased.', 'invoicing'),
 					'type'        => 'integer',
-					'context'     => array( 'view' ),
+					'context'     => array('view'),
 					'readonly'    => true,
 				),
 				'refunded_items'      => array(
-					'description' => __( 'Number of items refunded.', 'invoicing' ),
+					'description' => __('Number of items refunded.', 'invoicing'),
 					'type'        => 'integer',
-					'context'     => array( 'view' ),
+					'context'     => array('view'),
 					'readonly'    => true,
 				),
 				'total_tax'           => array(
-					'description' => __( 'Total charged for taxes.', 'invoicing' ),
+					'description' => __('Total charged for taxes.', 'invoicing'),
 					'type'        => 'string',
-					'context'     => array( 'view' ),
+					'context'     => array('view'),
 					'readonly'    => true,
 				),
 				'total_refunded_tax'  => array(
-					'description' => __( 'Total refunded for taxes.', 'invoicing' ),
+					'description' => __('Total refunded for taxes.', 'invoicing'),
 					'type'        => 'string',
-					'context'     => array( 'view' ),
+					'context'     => array('view'),
 					'readonly'    => true,
 				),
 				'total_fees'          => array(
-					'description' => __( 'Total fees charged.', 'invoicing' ),
+					'description' => __('Total fees charged.', 'invoicing'),
 					'type'        => 'string',
-					'context'     => array( 'view' ),
+					'context'     => array('view'),
 					'readonly'    => true,
 				),
 				'total_refunds'       => array(
-					'description' => __( 'Total of refunded invoices.', 'invoicing' ),
+					'description' => __('Total of refunded invoices.', 'invoicing'),
 					'type'        => 'integer',
-					'context'     => array( 'view' ),
+					'context'     => array('view'),
 					'readonly'    => true,
 				),
 				'net_refunds'         => array(
-					'description' => __( 'Net of refunded invoices.', 'invoicing' ),
+					'description' => __('Net of refunded invoices.', 'invoicing'),
 					'type'        => 'integer',
-					'context'     => array( 'view' ),
+					'context'     => array('view'),
 					'readonly'    => true,
 				),
 				'total_discount'      => array(
-					'description' => __( 'Total of discounts used.', 'invoicing' ),
+					'description' => __('Total of discounts used.', 'invoicing'),
 					'type'        => 'integer',
-					'context'     => array( 'view' ),
+					'context'     => array('view'),
 					'readonly'    => true,
 				),
 				'totals'              => array(
-					'description' => __( 'Totals.', 'invoicing' ),
+					'description' => __('Totals.', 'invoicing'),
 					'type'        => 'array',
 					'items'       => array(
 						'type' => 'array',
 					),
-					'context'     => array( 'view' ),
+					'context'     => array('view'),
 					'readonly'    => true,
 				),
 				'interval'            => array(
-					'description' => __( 'Number of months/days in the report period.', 'invoicing' ),
+					'description' => __('Number of months/days in the report period.', 'invoicing'),
 					'type'        => 'integer',
-					'context'     => array( 'view' ),
+					'context'     => array('view'),
 					'readonly'    => true,
 				),
 				'previous_range'      => array(
-					'description' => __( 'The previous report period.', 'invoicing' ),
+					'description' => __('The previous report period.', 'invoicing'),
 					'type'        => 'array',
 					'items'       => array(
 						'type' => 'string',
 					),
-					'context'     => array( 'view' ),
+					'context'     => array('view'),
 					'readonly'    => true,
 				),
 				'grouped_by'          => array(
-					'description' => __( 'The period used to group the totals.', 'invoicing' ),
+					'description' => __('The period used to group the totals.', 'invoicing'),
 					'type'        => 'string',
-					'context'     => array( 'view' ),
-					'enum'        => array( 'day', 'month' ),
+					'context'     => array('view'),
+					'enum'        => array('day', 'month'),
 					'readonly'    => true,
 				),
 				'currency'            => array(
-					'description' => __( 'The default store currency.', 'invoicing' ),
+					'description' => __('The default store currency.', 'invoicing'),
 					'type'        => 'string',
-					'context'     => array( 'view' ),
+					'context'     => array('view'),
 					'readonly'    => true,
 				),
 				'currency_symbol'     => array(
-					'description' => __( 'The default store currency symbol.', 'invoicing' ),
+					'description' => __('The default store currency symbol.', 'invoicing'),
 					'type'        => 'string',
-					'context'     => array( 'view' ),
+					'context'     => array('view'),
 					'readonly'    => true,
 				),
 				'currency_position'   => array(
-					'description' => __( 'The default store currency position.', 'invoicing' ),
+					'description' => __('The default store currency position.', 'invoicing'),
 					'type'        => 'string',
-					'context'     => array( 'view' ),
+					'context'     => array('view'),
 					'readonly'    => true,
 				),
 				'decimal_places'      => array(
-					'description' => __( 'The default store decimal places.', 'invoicing' ),
+					'description' => __('The default store decimal places.', 'invoicing'),
 					'type'        => 'string',
-					'context'     => array( 'view' ),
+					'context'     => array('view'),
 					'readonly'    => true,
 				),
 				'thousands_sep'       => array(
-					'description' => __( 'The default store thousands separator.', 'invoicing' ),
+					'description' => __('The default store thousands separator.', 'invoicing'),
 					'type'        => 'string',
-					'context'     => array( 'view' ),
+					'context'     => array('view'),
 					'readonly'    => true,
 				),
 				'decimals_sep'        => array(
-					'description' => __( 'The default store decimals separator.', 'invoicing' ),
+					'description' => __('The default store decimals separator.', 'invoicing'),
 					'type'        => 'string',
-					'context'     => array( 'view' ),
+					'context'     => array('view'),
 					'readonly'    => true,
 				),
 			),
 		);
 
-		return $this->add_additional_fields_schema( $schema );
+		return $this->add_additional_fields_schema($schema);
 
 	}
 

includes/api/class-wpinv-rest-discounts-controller.php

Spacing +17 added lines, -17 removed lines

@@ -5,7 +5,7 @@ 
  * @version 1.0.19
  */
 
-defined( 'ABSPATH' ) || exit;
+defined('ABSPATH') || exit;
 
 /**
  * REST API discounts controller class.
@@ -42,9 +42,9 @@ 
 	 *
 	 * @see register_rest_route()
 	 */
-	public function register_namespace_routes( $namespace ) {
+	public function register_namespace_routes($namespace) {
 
-		parent::register_namespace_routes( $namespace );
+		parent::register_namespace_routes($namespace);
 
 		register_rest_route(
 			$this->namespace,
@@ -52,8 +52,8 @@ 
 			array(
 				array(
 					'methods'             => WP_REST_Server::READABLE,
-					'callback'            => array( $this, 'get_discount_types' ),
-					'permission_callback' => array( $this, 'get_items_permissions_check' ),
+					'callback'            => array($this, 'get_discount_types'),
+					'permission_callback' => array($this, 'get_items_permissions_check'),
 				),
 			)
 		);
@@ -69,7 +69,7 @@ 
 	 * @return WP_REST_Response|WP_Error Response object on success, or WP_Error object on failure.
 	 */
 	public function get_discount_types() {
-		return rest_ensure_response( wpinv_get_discount_types() );
+		return rest_ensure_response(wpinv_get_discount_types());
 	}
 
     /**
@@ -87,13 +87,13 @@ 
 
 				// Discount types
 				'type' => array(
-					'description'       => __( 'Type of discounts to fetch.', 'invoicing' ),
-					'type'              => array( 'array', 'string' ),
+					'description'       => __('Type of discounts to fetch.', 'invoicing'),
+					'type'              => array('array', 'string'),
 					'default'           => 'any',
 					'validate_callback' => 'rest_validate_request_arg',
 					'sanitize_callback' => 'wpinv_parse_list',
 					'items'             => array(
-						'enum' => array_merge( array( 'any' ), array_keys( wpinv_get_discount_types() ) ),
+						'enum' => array_merge(array('any'), array_keys(wpinv_get_discount_types())),
 						'type' => 'string',
 					),
 				),
@@ -102,7 +102,7 @@ 
 		);
 
 		// Filter collection parameters for the discounts controller.
-		return apply_filters( 'getpaid_rest_discounts_collection_params', $params, $this );
+		return apply_filters('getpaid_rest_discounts_collection_params', $params, $this);
 	}
 
 	/**
@@ -113,26 +113,26 @@ 
 	 * @param WP_REST_Request $request Request object.
 	 * @return array          $query_args
 	 */
-	protected function prepare_items_query( $prepared_args = array(), $request = null ) {
+	protected function prepare_items_query($prepared_args = array(), $request = null) {
 
-		$query_args = parent::prepare_items_query( $prepared_args );
+		$query_args = parent::prepare_items_query($prepared_args);
 
 		// Retrieve items by type.
-		if ( ! in_array( 'any', $request['type'] ) ) {
+		if (!in_array('any', $request['type'])) {
 
-			if ( empty( $query_args['meta_query'] ) ) {
+			if (empty($query_args['meta_query'])) {
 				$query_args['meta_query'] = array();
 			}
 
 			$query_args['meta_query'][] = array(
 				'key'     => '_wpi_discount_type',
-				'value'   => implode( ',', $request['type'] ),
+				'value'   => implode(',', $request['type']),
 				'compare' => 'IN',
 			);
 
 		}
 
-		return apply_filters( 'getpaid_rest_discounts_prepare_items_query', $query_args, $request, $this );
+		return apply_filters('getpaid_rest_discounts_prepare_items_query', $query_args, $request, $this);
 
 	}
 
@@ -144,7 +144,7 @@ 
 	 * @return array A list of registered item statuses.
 	 */
 	public function get_post_statuses() {
-		return array( 'publish', 'pending', 'draft', 'expired' );
+		return array('publish', 'pending', 'draft', 'expired');
 	}
 
 }

includes/api/class-getpaid-rest-crud-controller.php

Spacing +117 added lines, -117 removed lines

@@ -8,7 +8,7 @@ 
  * @version 1.0.19
  */
 
-defined( 'ABSPATH' ) || exit;
+defined('ABSPATH') || exit;
 
 /**
  * GetPaid REST CRUD controller class.
@@ -38,7 +38,7 @@ 
 	 *
 	 * @see register_rest_route()
 	 */
-	public function register_namespace_routes( $namespace ) {
+	public function register_namespace_routes($namespace) {
 
 		register_rest_route(
 			$namespace,
@@ -46,22 +46,22 @@ 
 			array(
 				array(
 					'methods'             => WP_REST_Server::READABLE,
-					'callback'            => array( $this, 'get_items' ),
-					'permission_callback' => array( $this, 'get_items_permissions_check' ),
+					'callback'            => array($this, 'get_items'),
+					'permission_callback' => array($this, 'get_items_permissions_check'),
 					'args'                => $this->get_collection_params(),
 				),
 				array(
 					'methods'             => WP_REST_Server::CREATABLE,
-					'callback'            => array( $this, 'create_item' ),
-					'permission_callback' => array( $this, 'create_item_permissions_check' ),
-					'args'                => $this->get_endpoint_args_for_item_schema( WP_REST_Server::CREATABLE ),
+					'callback'            => array($this, 'create_item'),
+					'permission_callback' => array($this, 'create_item_permissions_check'),
+					'args'                => $this->get_endpoint_args_for_item_schema(WP_REST_Server::CREATABLE),
 				),
-				'schema' => array( $this, 'get_public_item_schema' ),
+				'schema' => array($this, 'get_public_item_schema'),
 			)
 		);
 
 		$get_item_args = array(
-			'context' => $this->get_context_param( array( 'default' => 'view' ) ),
+			'context' => $this->get_context_param(array('default' => 'view')),
 		);
 
 		register_rest_route(
@@ -70,35 +70,35 @@ 
 			array(
 				'args'   => array(
 					'id' => array(
-						'description' => __( 'Unique identifier for the object.', 'invoicing' ),
+						'description' => __('Unique identifier for the object.', 'invoicing'),
 						'type'        => 'integer',
 					),
 				),
 				array(
 					'methods'             => WP_REST_Server::READABLE,
-					'callback'            => array( $this, 'get_item' ),
-					'permission_callback' => array( $this, 'get_item_permissions_check' ),
+					'callback'            => array($this, 'get_item'),
+					'permission_callback' => array($this, 'get_item_permissions_check'),
 					'args'                => $get_item_args,
 				),
 				array(
 					'methods'             => WP_REST_Server::EDITABLE,
-					'callback'            => array( $this, 'update_item' ),
-					'permission_callback' => array( $this, 'update_item_permissions_check' ),
-					'args'                => $this->get_endpoint_args_for_item_schema( WP_REST_Server::EDITABLE ),
+					'callback'            => array($this, 'update_item'),
+					'permission_callback' => array($this, 'update_item_permissions_check'),
+					'args'                => $this->get_endpoint_args_for_item_schema(WP_REST_Server::EDITABLE),
 				),
 				array(
 					'methods'             => WP_REST_Server::DELETABLE,
-					'callback'            => array( $this, 'delete_item' ),
-					'permission_callback' => array( $this, 'delete_item_permissions_check' ),
+					'callback'            => array($this, 'delete_item'),
+					'permission_callback' => array($this, 'delete_item_permissions_check'),
 					'args'                => array(
 						'force' => array(
 							'type'        => 'boolean',
 							'default'     => false,
-							'description' => __( 'Whether to bypass Trash and force deletion.', 'invoicing' ),
+							'description' => __('Whether to bypass Trash and force deletion.', 'invoicing'),
 						),
 					),
 				),
-				'schema' => array( $this, 'get_public_item_schema' ),
+				'schema' => array($this, 'get_public_item_schema'),
 			)
 		);
 
@@ -110,14 +110,14 @@ 
 	 * @param GetPaid_Data $object Object to save.
 	 * @return WP_Error|GetPaid_Data
 	 */
-	protected function save_object( $object ) {
+	protected function save_object($object) {
 		$object->save();
 
-		if ( ! empty( $object->last_error ) ) {
-			return new WP_Error( 'rest_cannot_save', $object->last_error, array( 'status' => 400 ) );
+		if (!empty($object->last_error)) {
+			return new WP_Error('rest_cannot_save', $object->last_error, array('status' => 400));
 		}
 
-		return new $this->crud_class( $object->get_id() );
+		return new $this->crud_class($object->get_id());
 	}
 
 	/**
@@ -128,21 +128,21 @@ 
 	 * @param int|WP_Post $object_id Supplied ID.
 	 * @return GetPaid_Data|WP_Error GetPaid_Data object if ID is valid, WP_Error otherwise.
 	 */
-	protected function get_object( $object_id ) {
+	protected function get_object($object_id) {
 
 		// Do we have an object?
-		if ( empty( $this->crud_class ) || ! class_exists( $this->crud_class ) ) {
-			return new WP_Error( 'no_crud_class', __( 'You need to specify a CRUD class for this controller', 'invoicing' ) );
+		if (empty($this->crud_class) || !class_exists($this->crud_class)) {
+			return new WP_Error('no_crud_class', __('You need to specify a CRUD class for this controller', 'invoicing'));
 		}
 
 		// Fetch the object.
-		$object = new $this->crud_class( $object_id );
-		if ( ! empty( $object->last_error ) ) {
-			return new WP_Error( 'rest_object_invalid_id', $object->last_error, array( 'status' => 404 ) );
+		$object = new $this->crud_class($object_id);
+		if (!empty($object->last_error)) {
+			return new WP_Error('rest_object_invalid_id', $object->last_error, array('status' => 404));
 		}
 
 		$this->data_object = $object;
-		return $object->get_id() ? $object : new WP_Error( 'rest_object_invalid_id', __( 'Invalid ID.', 'invoicing' ), array( 'status' => 404 ) );
+		return $object->get_id() ? $object : new WP_Error('rest_object_invalid_id', __('Invalid ID.', 'invoicing'), array('status' => 404));
 
 	}
 
@@ -152,17 +152,17 @@ 
 	 * @param WP_REST_Request $request Full details about the request.
 	 * @return WP_Error|WP_REST_Response
 	 */
-	public function get_item( $request ) {
+	public function get_item($request) {
 
 		// Fetch the item.
-		$object = $this->get_object( $request['id'] );
+		$object = $this->get_object($request['id']);
 
-		if ( is_wp_error( $object ) ) {
+		if (is_wp_error($object)) {
 			return $object;
 		}
 
 		// Generate a response.
-		return rest_ensure_response( $this->prepare_item_for_response( $object, $request ) );
+		return rest_ensure_response($this->prepare_item_for_response($object, $request));
 
 	}
 
@@ -172,38 +172,38 @@ 
 	 * @param WP_REST_Request $request Full details about the request.
 	 * @return WP_Error|WP_REST_Response
 	 */
-	public function create_item( $request ) {
+	public function create_item($request) {
 
 		// Can not create an existing item.
-		if ( ! empty( $request['id'] ) ) {
+		if (!empty($request['id'])) {
 			/* translators: %s: post type */
-			return new WP_Error( "getpaid_rest_{$this->rest_base}_exists", __( 'Cannot create existing resource.', 'invoicing' ), array( 'status' => 400 ) );
+			return new WP_Error("getpaid_rest_{$this->rest_base}_exists", __('Cannot create existing resource.', 'invoicing'), array('status' => 400));
 		}
 
 		// Generate a GetPaid_Data object from the request.
-		$object = $this->prepare_item_for_database( $request );
-		if ( is_wp_error( $object ) ) {
+		$object = $this->prepare_item_for_database($request);
+		if (is_wp_error($object)) {
 			return $object;
 		}
 
 		// Save the object.
-		$object = $this->save_object( $object );
-		if ( is_wp_error( $object ) ) {
+		$object = $this->save_object($object);
+		if (is_wp_error($object)) {
 			return $object;
 		}
 
 		// Save special fields.
-		$save_special = $this->update_additional_fields_for_object( $object, $request );
-		if ( is_wp_error( $save_special ) ) {
-			$object->delete( true );
+		$save_special = $this->update_additional_fields_for_object($object, $request);
+		if (is_wp_error($save_special)) {
+			$object->delete(true);
 			return $save_special;
 		}
 
-		$request->set_param( 'context', 'edit' );
-		$response = $this->prepare_item_for_response( $object, $request );
-		$response = rest_ensure_response( $response );
-		$response->set_status( 201 );
-		$response->header( 'Location', rest_url( sprintf( '/%s/%s/%d', $this->namespace, $this->rest_base, $object->get_id() ) ) );
+		$request->set_param('context', 'edit');
+		$response = $this->prepare_item_for_response($object, $request);
+		$response = rest_ensure_response($response);
+		$response->set_status(201);
+		$response->header('Location', rest_url(sprintf('/%s/%s/%d', $this->namespace, $this->rest_base, $object->get_id())));
 
 		return $response;
 	}
@@ -214,35 +214,35 @@ 
 	 * @param WP_REST_Request $request Full details about the request.
 	 * @return WP_Error|WP_REST_Response
 	 */
-	public function update_item( $request ) {
+	public function update_item($request) {
 
 		// Fetch the item.
-		$object = $this->get_object( $request['id'] );
-		if ( is_wp_error( $object ) ) {
+		$object = $this->get_object($request['id']);
+		if (is_wp_error($object)) {
 			return $object;
 		}
 
 		// Prepare the item for saving.
-		$object = $this->prepare_item_for_database( $request );
-		if ( is_wp_error( $object ) ) {
+		$object = $this->prepare_item_for_database($request);
+		if (is_wp_error($object)) {
 			return $object;
 		}
 
 		// Save the item.
-		$object = $this->save_object( $object );
-		if ( is_wp_error( $object ) ) {
+		$object = $this->save_object($object);
+		if (is_wp_error($object)) {
 			return $object;
 		}
 
 		// Save special fields (those added via hooks).
-		$save_special = $this->update_additional_fields_for_object( $object, $request );
-		if ( is_wp_error( $save_special ) ) {
+		$save_special = $this->update_additional_fields_for_object($object, $request);
+		if (is_wp_error($save_special)) {
 			return $save_special;
 		}
 
-		$request->set_param( 'context', 'edit' );
-		$response = $this->prepare_item_for_response( $object, $request );
-		return rest_ensure_response( $response );
+		$request->set_param('context', 'edit');
+		$response = $this->prepare_item_for_response($object, $request);
+		return rest_ensure_response($response);
 	}
 
 	/**
@@ -251,14 +251,14 @@ 
 	 * @param GetPaid_Data    $object GetPaid_Data object.
 	 * @return array Links for the given object.
 	 */
-	protected function prepare_links( $object ) {
+	protected function prepare_links($object) {
 
 		$links = array(
 			'self'       => array(
-				'href' => rest_url( sprintf( '/%s/%s/%d', $this->namespace, $this->rest_base, $object->get_id() ) ),
+				'href' => rest_url(sprintf('/%s/%s/%d', $this->namespace, $this->rest_base, $object->get_id())),
 			),
 			'collection' => array(
-				'href' => rest_url( sprintf( '/%s/%s', $this->namespace, $this->rest_base ) ),
+				'href' => rest_url(sprintf('/%s/%s', $this->namespace, $this->rest_base)),
 			),
 		);
 
@@ -282,8 +282,8 @@ 
 	 * @param  array $schema Schema.
 	 * @return bool
 	 */
-	public function filter_writable_props( $schema ) {
-		return empty( $schema['readonly'] );
+	public function filter_writable_props($schema) {
+		return empty($schema['readonly']);
 	}
 
 	/**
@@ -293,43 +293,43 @@ 
 	 * @param  WP_REST_Request $request Request object.
 	 * @return GetPaid_Data|WP_Error Data object or WP_Error.
 	 */
-	protected function prepare_item_for_database( $request ) {
+	protected function prepare_item_for_database($request) {
 
 		// Do we have an object?
-		if ( empty( $this->crud_class ) || ! class_exists( $this->crud_class ) ) {
-			return new WP_Error( 'no_crud_class', __( 'You need to specify a CRUD class for this controller', 'invoicing' ) );
+		if (empty($this->crud_class) || !class_exists($this->crud_class)) {
+			return new WP_Error('no_crud_class', __('You need to specify a CRUD class for this controller', 'invoicing'));
 		}
 
 		// Prepare the object.
-		$id        = isset( $request['id'] ) ? absint( $request['id'] ) : 0;
-		$object    = new $this->crud_class( $id );
+		$id        = isset($request['id']) ? absint($request['id']) : 0;
+		$object    = new $this->crud_class($id);
 
 		// Abort if an error exists.
-		if ( ! empty( $object->last_error ) ) {
-			return new WP_Error( 'invalid_item', $object->last_error );
+		if (!empty($object->last_error)) {
+			return new WP_Error('invalid_item', $object->last_error);
 		}
 
 		$schema    = $this->get_item_schema();
-		$data_keys = array_keys( array_filter( $schema['properties'], array( $this, 'filter_writable_props' ) ) );
+		$data_keys = array_keys(array_filter($schema['properties'], array($this, 'filter_writable_props')));
 
 		// Handle all writable props.
-		foreach ( $data_keys as $key ) {
-			$value = $request[ $key ];
+		foreach ($data_keys as $key) {
+			$value = $request[$key];
 
-			if ( ! is_null( $value ) ) {
-				switch ( $key ) {
+			if (!is_null($value)) {
+				switch ($key) {
 
 					case 'meta_data':
-						if ( is_array( $value ) ) {
-							foreach ( $value as $meta ) {
-								$object->update_meta_data( $meta['key'], $meta['value'], isset( $meta['id'] ) ? $meta['id'] : '' );
+						if (is_array($value)) {
+							foreach ($value as $meta) {
+								$object->update_meta_data($meta['key'], $meta['value'], isset($meta['id']) ? $meta['id'] : '');
 							}
 						}
 						break;
 
 					default:
-						if ( is_callable( array( $object, "set_{$key}" ) ) ) {
-							$object->{"set_{$key}"}( $value );
+						if (is_callable(array($object, "set_{$key}"))) {
+							$object->{"set_{$key}"}($value);
 						}
 						break;
 				}
@@ -337,7 +337,7 @@ 
 }
 
 		// Filters an object before it is inserted via the REST API..
-		return apply_filters( "getpaid_rest_pre_insert_{$this->rest_base}_object", $object, $request );
+		return apply_filters("getpaid_rest_pre_insert_{$this->rest_base}_object", $object, $request);
 	}
 
 	/**
@@ -347,10 +347,10 @@ 
 	 * @param  GetPaid_Meta_Data[]    $meta_data  meta data objects.
 	 * @return array
 	 */
-	protected function prepare_object_meta_data( $meta_data ) {
+	protected function prepare_object_meta_data($meta_data) {
 		$meta = array();
 
-		foreach ( $meta_data as $object ) {
+		foreach ($meta_data as $object) {
 			$meta[] = $object->get_data();
 		}
 
@@ -365,14 +365,14 @@ 
 	 * @param array            $fields Fields to include.
 	 * @return array
 	 */
-	protected function prepare_invoice_items( $invoice ) {
+	protected function prepare_invoice_items($invoice) {
 		$items = array();
 
-		foreach ( $invoice->get_items() as $item ) {
+		foreach ($invoice->get_items() as $item) {
 
 			$item_data = $item->prepare_data_for_saving();
 
-			if ( 'amount' == $invoice->get_template() ) {
+			if ('amount' == $invoice->get_template()) {
 				$item_data['quantity'] = 1;
 			}
 
@@ -391,52 +391,52 @@ 
 	 * @param string           $context either view or edit.
 	 * @return array
 	 */
-	protected function prepare_object_data( $object, $fields, $context = 'view' ) {
+	protected function prepare_object_data($object, $fields, $context = 'view') {
 
 		$data = array();
 
 		// Handle all writable props.
-		foreach ( array_keys( $this->get_schema_properties() ) as $key ) {
+		foreach (array_keys($this->get_schema_properties()) as $key) {
 
 			// Abort if it is not included.
-			if ( ! empty( $fields ) && ! $this->is_field_included( $key, $fields ) ) {
+			if (!empty($fields) && !$this->is_field_included($key, $fields)) {
 				continue;
 			}
 
 			// Or this current object does not support the field.
-			if ( ! $this->object_supports_field( $object, $key ) ) {
+			if (!$this->object_supports_field($object, $key)) {
 				continue;
 			}
 
 			// Handle meta data.
-			if ( $key == 'meta_data' ) {
-				$data['meta_data'] = $this->prepare_object_meta_data( $object->get_meta_data() );
+			if ($key == 'meta_data') {
+				$data['meta_data'] = $this->prepare_object_meta_data($object->get_meta_data());
 				continue;
 			}
 
 			// Handle items.
-			if ( $key == 'items' && is_a( $object, 'WPInv_Invoice' ) ) {
-				$data['items'] = $this->prepare_invoice_items( $object );
+			if ($key == 'items' && is_a($object, 'WPInv_Invoice')) {
+				$data['items'] = $this->prepare_invoice_items($object);
 				continue;
 			}
 
 			// Booleans.
-			if ( is_callable( array( $object, $key ) ) ) {
-				$data[ $key ] = $object->$key( $context );
+			if (is_callable(array($object, $key))) {
+				$data[$key] = $object->$key($context);
 				continue;
 			}
 
 			// Get object value.
-			if ( is_callable( array( $object, "get_{$key}" ) ) ) {
-				$value = $object->{"get_{$key}"}( $context );
+			if (is_callable(array($object, "get_{$key}"))) {
+				$value = $object->{"get_{$key}"}($context);
 
 				// If the value is an instance of GetPaid_Data...
-				if ( is_a( $value, 'GetPaid_Data' ) ) {
-					$value = $value->get_data( $context );
+				if (is_a($value, 'GetPaid_Data')) {
+					$value = $value->get_data($context);
 				}
 
 				// For objects, retrieves it's properties.
-				$data[ $key ] = is_object( $value ) ? get_object_vars( $value ) : $value;
+				$data[$key] = is_object($value) ? get_object_vars($value) : $value;
 				continue;
 			}
 }
@@ -452,8 +452,8 @@ 
 	 * @param  string       $field_key The key to check for.
 	 * @return bool
 	 */
-	public function object_supports_field( $object, $field_key ) {
-		return apply_filters( 'getpaid_rest_object_supports_key', true, $object, $field_key );
+	public function object_supports_field($object, $field_key) {
+		return apply_filters('getpaid_rest_object_supports_key', true, $object, $field_key);
 	}
 
 	/**
@@ -464,27 +464,27 @@ 
 	 * @param  WP_REST_Request $request Request object.
 	 * @return WP_REST_Response
 	 */
-	public function prepare_item_for_response( $object, $request ) {
-		remove_filter( 'rest_post_dispatch', 'rest_filter_response_fields', 10 );
+	public function prepare_item_for_response($object, $request) {
+		remove_filter('rest_post_dispatch', 'rest_filter_response_fields', 10);
 
 		$this->data_object = $object;
 
 		// Fetch the fields to include in this response.
-		$fields = $this->get_fields_for_response( $request );
+		$fields = $this->get_fields_for_response($request);
 
 		// Prepare object data.
-		$context = ! empty( $request['context'] ) ? $request['context'] : 'view';
-		$data    = $this->prepare_object_data( $object, $fields, $context );
-		$data    = $this->add_additional_fields_to_object( $data, $request );
-		$data    = $this->limit_object_to_requested_fields( $data, $fields );
-		$data    = $this->filter_response_by_context( $data, $context );
+		$context = !empty($request['context']) ? $request['context'] : 'view';
+		$data    = $this->prepare_object_data($object, $fields, $context);
+		$data    = $this->add_additional_fields_to_object($data, $request);
+		$data    = $this->limit_object_to_requested_fields($data, $fields);
+		$data    = $this->filter_response_by_context($data, $context);
 
 		// Prepare the response.
-		$response = rest_ensure_response( $data );
-		$response->add_links( $this->prepare_links( $object, $request ) );
+		$response = rest_ensure_response($data);
+		$response->add_links($this->prepare_links($object, $request));
 
 		// Filter item response.
-		return apply_filters( "getpaid_rest_prepare_{$this->rest_base}_object", $response, $object, $request );
+		return apply_filters("getpaid_rest_prepare_{$this->rest_base}_object", $response, $object, $request);
 	}
 
 }

includes/api/class-getpaid-rest-date-based-controller.php

Spacing +93 added lines, -93 removed lines

@@ -7,7 +7,7 @@ 
  * @since   2.0.0
  */
 
-defined( 'ABSPATH' ) || exit;
+defined('ABSPATH') || exit;
 
 /**
  * GetPaid REST date based controller class.
@@ -43,15 +43,15 @@ 
 	 * @param WP_REST_Request $request Request object.
 	 * @return array The appropriate date range.
 	 */
-	public function get_date_range( $request ) {
+	public function get_date_range($request) {
 
 		// If not supported, assume all time.
-		if ( ! in_array( $request['period'], array( 'custom', 'today', 'yesterday', 'week', 'last_week', '7_days', '30_days', '60_days', '90_days', '180_days', 'month', 'last_month', 'quarter', 'last_quarter', 'year', 'last_year' ) ) ) {
+		if (!in_array($request['period'], array('custom', 'today', 'yesterday', 'week', 'last_week', '7_days', '30_days', '60_days', '90_days', '180_days', 'month', 'last_month', 'quarter', 'last_quarter', 'year', 'last_year'))) {
 			$request['period'] = '7_days';
 		}
 
-		$date_range = call_user_func( array( $this, 'get_' . $request['period'] . '_date_range' ), $request );
-		$this->prepare_interval( $date_range );
+		$date_range = call_user_func(array($this, 'get_' . $request['period'] . '_date_range'), $request);
+		$this->prepare_interval($date_range);
 
 		return $date_range;
 
@@ -63,25 +63,25 @@ 
 	 * @param array $range Date range.
 	 * @return array The appropriate date range.
 	 */
-	public function prepare_interval( $range ) {
+	public function prepare_interval($range) {
 
-		$before = strtotime( $range['before'] ) - DAY_IN_SECONDS;
-		$after  = strtotime( $range['after'] ) + DAY_IN_SECONDS;
-		if ( 'day' === $this->groupby ) {
-			$difference     = max( DAY_IN_SECONDS, ( DAY_IN_SECONDS + $before - $after ) ); // Prevent division by 0;
-			$this->interval = absint( ceil( max( 1, $difference / DAY_IN_SECONDS ) ) );
+		$before = strtotime($range['before']) - DAY_IN_SECONDS;
+		$after  = strtotime($range['after']) + DAY_IN_SECONDS;
+		if ('day' === $this->groupby) {
+			$difference     = max(DAY_IN_SECONDS, (DAY_IN_SECONDS + $before - $after)); // Prevent division by 0;
+			$this->interval = absint(ceil(max(1, $difference / DAY_IN_SECONDS)));
 			return;
 		}
 
 		$this->interval = 0;
-		$min_date       = strtotime( date( 'Y-m-01', $after ) );
+		$min_date       = strtotime(date('Y-m-01', $after));
 
-		while ( $min_date <= $before ) {
-			$this->interval ++;
-			$min_date = strtotime( '+1 MONTH', $min_date );
+		while ($min_date <= $before) {
+			$this->interval++;
+			$min_date = strtotime('+1 MONTH', $min_date);
 		}
 
-		$this->interval = max( 1, $this->interval );
+		$this->interval = max(1, $this->interval);
 
 	}
 
@@ -91,17 +91,17 @@ 
 	 * @param WP_REST_Request $request Request object.
 	 * @return array The appropriate date range.
 	 */
-	public function get_custom_date_range( $request ) {
+	public function get_custom_date_range($request) {
 
-		$after  = max( strtotime( '-20 years' ), strtotime( sanitize_text_field( $request['after'] ) ) );
-		$before = strtotime( '+1 day', current_time( 'timestamp' ) );
+		$after  = max(strtotime('-20 years'), strtotime(sanitize_text_field($request['after'])));
+		$before = strtotime('+1 day', current_time('timestamp'));
 
-		if ( ! empty( $request['before'] ) ) {
-			$before  = min( $before, strtotime( sanitize_text_field( $request['before'] ) ) );
+		if (!empty($request['before'])) {
+			$before = min($before, strtotime(sanitize_text_field($request['before'])));
 		}
 
 		// 3 months max for day view
-		if ( floor( ( $before - $after ) / MONTH_IN_SECONDS ) > 3 ) {
+		if (floor(($before - $after) / MONTH_IN_SECONDS) > 3) {
 			$this->groupby = 'month';
 		}
 
@@ -109,14 +109,14 @@ 
 		$difference           = $before - $after;
 		$this->previous_range = array(
 			'period' => 'custom',
-			'before' => date( 'Y-m-d', $before - $difference ),
-			'after'  => date( 'Y-m-d', $after - $difference ),
+			'before' => date('Y-m-d', $before - $difference),
+			'after'  => date('Y-m-d', $after - $difference),
 		);
 
 		// Generate the report.
 		return array(
-			'before' => date( 'Y-m-d', $before ),
-			'after'  => date( 'Y-m-d', $after ),
+			'before' => date('Y-m-d', $before),
+			'after'  => date('Y-m-d', $after),
 		);
 
 	}
@@ -135,8 +135,8 @@ 
 
 		// Generate the report.
 		return array(
-			'before' => date( 'Y-m-d', strtotime( '+1 day', current_time( 'timestamp' ) ) ),
-			'after'  => date( 'Y-m-d', strtotime( '-1 day', current_time( 'timestamp' ) ) ),
+			'before' => date('Y-m-d', strtotime('+1 day', current_time('timestamp'))),
+			'after'  => date('Y-m-d', strtotime('-1 day', current_time('timestamp'))),
 		);
 
 	}
@@ -151,14 +151,14 @@ 
 		// Set the previous date range.
 		$this->previous_range = array(
 			'period' => 'custom',
-			'before' => date( 'Y-m-d', strtotime( '-1 day', current_time( 'timestamp' ) ) ),
-			'after'  => date( 'Y-m-d', strtotime( '-3 days', current_time( 'timestamp' ) ) ),
+			'before' => date('Y-m-d', strtotime('-1 day', current_time('timestamp'))),
+			'after'  => date('Y-m-d', strtotime('-3 days', current_time('timestamp'))),
 		);
 
 		// Generate the report.
 		return array(
-			'before' => date( 'Y-m-d', current_time( 'timestamp' ) ),
-			'after'  => date( 'Y-m-d', strtotime( '-2 days', current_time( 'timestamp' ) ) ),
+			'before' => date('Y-m-d', current_time('timestamp')),
+			'after'  => date('Y-m-d', strtotime('-2 days', current_time('timestamp'))),
 		);
 
 	}
@@ -177,8 +177,8 @@ 
 
 		// Generate the report.
 		return array(
-			'before' => date( 'Y-m-d', strtotime( 'sunday last week', current_time( 'timestamp' ) ) + 8 * DAY_IN_SECONDS ),
-			'after'  => date( 'Y-m-d', strtotime( 'sunday last week', current_time( 'timestamp' ) ) ),
+			'before' => date('Y-m-d', strtotime('sunday last week', current_time('timestamp')) + 8 * DAY_IN_SECONDS),
+			'after'  => date('Y-m-d', strtotime('sunday last week', current_time('timestamp'))),
 		);
 
 	}
@@ -193,14 +193,14 @@ 
 		// Set the previous date range.
 		$this->previous_range = array(
 			'period' => 'custom',
-			'before' => date( 'Y-m-d', strtotime( 'monday last week', current_time( 'timestamp' ) ) ),
-			'after'  => date( 'Y-m-d', strtotime( 'monday last week', current_time( 'timestamp' ) ) - 8 * DAY_IN_SECONDS ),
+			'before' => date('Y-m-d', strtotime('monday last week', current_time('timestamp'))),
+			'after'  => date('Y-m-d', strtotime('monday last week', current_time('timestamp')) - 8 * DAY_IN_SECONDS),
 		);
 
 		// Generate the report.
 		return array(
-			'before' => date( 'Y-m-d', strtotime( 'monday this week', current_time( 'timestamp' ) ) ),
-			'after'  => date( 'Y-m-d', strtotime( 'monday last week', current_time( 'timestamp' ) ) - DAY_IN_SECONDS ),
+			'before' => date('Y-m-d', strtotime('monday this week', current_time('timestamp'))),
+			'after'  => date('Y-m-d', strtotime('monday last week', current_time('timestamp')) - DAY_IN_SECONDS),
 		);
 
 	}
@@ -215,14 +215,14 @@ 
 		// Set the previous date range.
 		$this->previous_range = array(
 			'period' => 'custom',
-			'before' => date( 'Y-m-d', strtotime( '-7 days', current_time( 'timestamp' ) ) ),
-			'after'  => date( 'Y-m-d', strtotime( '-15 days', current_time( 'timestamp' ) ) ),
+			'before' => date('Y-m-d', strtotime('-7 days', current_time('timestamp'))),
+			'after'  => date('Y-m-d', strtotime('-15 days', current_time('timestamp'))),
 		);
 
 		// Generate the report.
 		return array(
-			'before' => date( 'Y-m-d', current_time( 'timestamp' ) ),
-			'after'  => date( 'Y-m-d', strtotime( '-8 days', current_time( 'timestamp' ) ) ),
+			'before' => date('Y-m-d', current_time('timestamp')),
+			'after'  => date('Y-m-d', strtotime('-8 days', current_time('timestamp'))),
 		);
 
 	}
@@ -237,14 +237,14 @@ 
 		// Set the previous date range.
 		$this->previous_range = array(
 			'period' => 'custom',
-			'before' => date( 'Y-m-d', strtotime( '-30 days', current_time( 'timestamp' ) ) ),
-			'after'  => date( 'Y-m-d', strtotime( '-61 days', current_time( 'timestamp' ) ) ),
+			'before' => date('Y-m-d', strtotime('-30 days', current_time('timestamp'))),
+			'after'  => date('Y-m-d', strtotime('-61 days', current_time('timestamp'))),
 		);
 
 		// Generate the report.
 		return array(
-			'before' => date( 'Y-m-d', current_time( 'timestamp' ) ),
-			'after'  => date( 'Y-m-d', strtotime( '-31 days', current_time( 'timestamp' ) ) ),
+			'before' => date('Y-m-d', current_time('timestamp')),
+			'after'  => date('Y-m-d', strtotime('-31 days', current_time('timestamp'))),
 		);
 
 	}
@@ -261,14 +261,14 @@ 
 		// Set the previous date range.
 		$this->previous_range = array(
 			'period' => 'custom',
-			'before' => date( 'Y-m-d', strtotime( '-90 days', current_time( 'timestamp' ) ) ),
-			'after'  => date( 'Y-m-d', strtotime( '-181 days', current_time( 'timestamp' ) ) ),
+			'before' => date('Y-m-d', strtotime('-90 days', current_time('timestamp'))),
+			'after'  => date('Y-m-d', strtotime('-181 days', current_time('timestamp'))),
 		);
 
 		// Generate the report.
 		return array(
-			'before' => date( 'Y-m-d', current_time( 'timestamp' ) ),
-			'after'  => date( 'Y-m-d', strtotime( '-91 days', current_time( 'timestamp' ) ) ),
+			'before' => date('Y-m-d', current_time('timestamp')),
+			'after'  => date('Y-m-d', strtotime('-91 days', current_time('timestamp'))),
 		);
 
 	}
@@ -285,14 +285,14 @@ 
 		// Set the previous date range.
 		$this->previous_range = array(
 			'period' => 'custom',
-			'before' => date( 'Y-m-d', strtotime( '-180 days', current_time( 'timestamp' ) ) ),
-			'after'  => date( 'Y-m-d', strtotime( '-361 days', current_time( 'timestamp' ) ) ),
+			'before' => date('Y-m-d', strtotime('-180 days', current_time('timestamp'))),
+			'after'  => date('Y-m-d', strtotime('-361 days', current_time('timestamp'))),
 		);
 
 		// Generate the report.
 		return array(
-			'before' => date( 'Y-m-d', current_time( 'timestamp' ) ),
-			'after'  => date( 'Y-m-d', strtotime( '-181 days', current_time( 'timestamp' ) ) ),
+			'before' => date('Y-m-d', current_time('timestamp')),
+			'after'  => date('Y-m-d', strtotime('-181 days', current_time('timestamp'))),
 		);
 
 	}
@@ -307,14 +307,14 @@ 
 		// Set the previous date range.
 		$this->previous_range = array(
 			'period' => 'custom',
-			'before' => date( 'Y-m-d', strtotime( '-60 days', current_time( 'timestamp' ) ) ),
-			'after'  => date( 'Y-m-d', strtotime( '-121 days', current_time( 'timestamp' ) ) ),
+			'before' => date('Y-m-d', strtotime('-60 days', current_time('timestamp'))),
+			'after'  => date('Y-m-d', strtotime('-121 days', current_time('timestamp'))),
 		);
 
 		// Generate the report.
 		return array(
-			'before' => date( 'Y-m-d', current_time( 'timestamp' ) ),
-			'after'  => date( 'Y-m-d', strtotime( '-61 days', current_time( 'timestamp' ) ) ),
+			'before' => date('Y-m-d', current_time('timestamp')),
+			'after'  => date('Y-m-d', strtotime('-61 days', current_time('timestamp'))),
 		);
 
 	}
@@ -333,8 +333,8 @@ 
 
 		// Generate the report.
 		return array(
-			'before' => date( 'Y-m-01', strtotime( 'next month', current_time( 'timestamp' ) ) ),
-			'after'  => date( 'Y-m-t', strtotime( 'last month', current_time( 'timestamp' ) ) ),
+			'before' => date('Y-m-01', strtotime('next month', current_time('timestamp'))),
+			'after'  => date('Y-m-t', strtotime('last month', current_time('timestamp'))),
 		);
 
 	}
@@ -349,14 +349,14 @@ 
 		// Set the previous date range.
 		$this->previous_range = array(
 			'period' => 'custom',
-			'before' => date( 'Y-m-1', strtotime( 'last month', current_time( 'timestamp' ) ) ),
-			'after'  => date( 'Y-m-t', strtotime( '-3 months', current_time( 'timestamp' ) ) ),
+			'before' => date('Y-m-1', strtotime('last month', current_time('timestamp'))),
+			'after'  => date('Y-m-t', strtotime('-3 months', current_time('timestamp'))),
 		);
 
 		// Generate the report.
 		return array(
-			'before' => date( 'Y-m-1', current_time( 'timestamp' ) ),
-			'after'  => date( 'Y-m-t', strtotime( '-2 months', current_time( 'timestamp' ) ) ),
+			'before' => date('Y-m-1', current_time('timestamp')),
+			'after'  => date('Y-m-t', strtotime('-2 months', current_time('timestamp'))),
 		);
 
 	}
@@ -368,9 +368,9 @@ 
 	 */
 	public function get_quarters() {
 
-		$last_year = (int) date( 'Y' ) - 1;
-		$next_year = (int) date( 'Y' ) + 1;
-		$year      = (int) date( 'Y' );
+		$last_year = (int) date('Y') - 1;
+		$next_year = (int) date('Y') + 1;
+		$year      = (int) date('Y');
 		return array(
 
 			array(
@@ -414,9 +414,9 @@ 
 	 */
 	public function get_quarter() {
 
-		$month    = (int) date( 'n', current_time( 'timestamp' ) );
-		$quarters = array( 1, 1, 1, 2, 2, 2, 3, 3, 3, 4, 4, 4 );
-		return $quarters[ $month - 1 ];
+		$month    = (int) date('n', current_time('timestamp'));
+		$quarters = array(1, 1, 1, 2, 2, 2, 3, 3, 3, 4, 4, 4);
+		return $quarters[$month - 1];
 
 	}
 
@@ -434,7 +434,7 @@ 
 
 		// Generate the report.
 		$quarters = $this->get_quarters();
-		return $quarters[ $this->get_quarter() + 1 ];
+		return $quarters[$this->get_quarter() + 1];
 
 	}
 
@@ -450,12 +450,12 @@ 
 
 		// Set the previous date range.
 		$this->previous_range = array_merge(
-			$quarters[ $quarter - 1 ],
-			array( 'period' => 'custom' )
+			$quarters[$quarter - 1],
+			array('period' => 'custom')
 		);
 
 		// Generate the report.
-		return $quarters[ $quarter ];
+		return $quarters[$quarter];
 
 	}
 
@@ -475,8 +475,8 @@ 
 
 		// Generate the report.
 		return array(
-			'before' => date( 'Y-m-d', strtotime( 'next year January 1st', current_time( 'timestamp' ) ) ),
-			'after'  => date( 'Y-m-d', strtotime( 'last year December 31st', current_time( 'timestamp' ) ) ),
+			'before' => date('Y-m-d', strtotime('next year January 1st', current_time('timestamp'))),
+			'after'  => date('Y-m-d', strtotime('last year December 31st', current_time('timestamp'))),
 		);
 
 	}
@@ -491,18 +491,18 @@ 
 		$this->groupby = 'month';
 
 		// Set the previous date range.
-		$year          = (int) date( 'Y' ) - 3;
+		$year          = (int) date('Y') - 3;
 		$this->previous_range = array(
 			'period' => 'custom',
-			'before' => date( 'Y-m-d', strtotime( 'first day of january last year', current_time( 'timestamp' ) ) ),
+			'before' => date('Y-m-d', strtotime('first day of january last year', current_time('timestamp'))),
 			'after'  => "$year-12-31",
 		);
 
 		// Generate the report.
-		$year          = (int) date( 'Y' ) - 2;
+		$year = (int) date('Y') - 2;
 		return array(
 			'after'  => "$year-12-31",
-			'before' => date( 'Y-m-d', strtotime( 'first day of january this year', current_time( 'timestamp' ) ) ),
+			'before' => date('Y-m-d', strtotime('first day of january this year', current_time('timestamp'))),
 		);
 
 	}
@@ -514,20 +514,20 @@ 
 	 * @param string $date_field The date field.
 	 * @return string The appropriate SQL.
 	 */
-	public function get_date_range_sql( $request, $date_field ) {
+	public function get_date_range_sql($request, $date_field) {
 		global $wpdb;
 
 		$sql = '1=1';
-		$range = $this->get_date_range( $request );
+		$range = $this->get_date_range($request);
 
-		if ( ! empty( $range['after'] ) ) {
+		if (!empty($range['after'])) {
 			$sql .= ' AND ' . $wpdb->prepare(
 				"$date_field > %s",
 				$range['after']
 			);
 		}
 
-		if ( ! empty( $range['before'] ) ) {
+		if (!empty($range['before'])) {
 			$sql .= ' AND ' . $wpdb->prepare(
 				"$date_field < %s",
 				$range['before']
@@ -544,9 +544,9 @@ 
 	 * @param string $date_field The date field.
 	 * @return string The appropriate SQL.
 	 */
-	public function get_group_by_sql( $date_field ) {
+	public function get_group_by_sql($date_field) {
 
-		if ( 'day' === $this->groupby ) {
+		if ('day' === $this->groupby) {
 			return "YEAR($date_field), MONTH($date_field), DAY($date_field)";
 		}
 
@@ -560,32 +560,32 @@ 
 	 */
 	public function get_collection_params() {
 		return array(
-			'context' => $this->get_context_param( array( 'default' => 'view' ) ),
+			'context' => $this->get_context_param(array('default' => 'view')),
 			'period'  => array(
-				'description'       => __( 'Limit to results of a specific period.', 'invoicing' ),
+				'description'       => __('Limit to results of a specific period.', 'invoicing'),
 				'type'              => 'string',
-				'enum'              => array( 'custom', 'today', 'yesterday', 'week', 'last_week', '7_days', '30_days', '60_days', '90_days', '180_days', 'month', 'last_month', 'quarter', 'last_quarter', 'year', 'last_year', 'quarter', 'last_quarter' ),
+				'enum'              => array('custom', 'today', 'yesterday', 'week', 'last_week', '7_days', '30_days', '60_days', '90_days', '180_days', 'month', 'last_month', 'quarter', 'last_quarter', 'year', 'last_year', 'quarter', 'last_quarter'),
 				'validate_callback' => 'rest_validate_request_arg',
 				'sanitize_callback' => 'sanitize_text_field',
 				'default'           => '7_days',
 			),
 			'after'   => array(
 				/* translators: %s: date format */
-				'description'       => sprintf( __( 'Limit to results after a specific date, the date needs to be in the %s format.', 'invoicing' ), 'YYYY-MM-DD' ),
+				'description'       => sprintf(__('Limit to results after a specific date, the date needs to be in the %s format.', 'invoicing'), 'YYYY-MM-DD'),
 				'type'              => 'string',
 				'format'            => 'date',
 				'validate_callback' => 'rest_validate_request_arg',
 				'sanitize_callback' => 'sanitize_text_field',
-				'default'           => date( 'Y-m-d', strtotime( '-8 days', current_time( 'timestamp' ) ) ),
+				'default'           => date('Y-m-d', strtotime('-8 days', current_time('timestamp'))),
 			),
 			'before'  => array(
 				/* translators: %s: date format */
-				'description'       => sprintf( __( 'Limit to results before a specific date, the date needs to be in the %s format.', 'invoicing' ), 'YYYY-MM-DD' ),
+				'description'       => sprintf(__('Limit to results before a specific date, the date needs to be in the %s format.', 'invoicing'), 'YYYY-MM-DD'),
 				'type'              => 'string',
 				'format'            => 'date',
 				'validate_callback' => 'rest_validate_request_arg',
 				'sanitize_callback' => 'sanitize_text_field',
-				'default'           => date( 'Y-m-d', current_time( 'timestamp' ) ),
+				'default'           => date('Y-m-d', current_time('timestamp')),
 			),
 		);
 	}

includes/api/class-wpinv-rest-invoice-controller.php

Spacing +14 added lines, -14 removed lines

@@ -5,7 +5,7 @@ 
  * @version 1.0.19
  */
 
-defined( 'ABSPATH' ) || exit;
+defined('ABSPATH') || exit;
 
 /**
  * REST API invoices controller class.
@@ -49,7 +49,7 @@ 
 			array(
 
 				'customers'         => array(
-					'description'       => __( 'Limit result set to invoices for specific user ids.', 'invoicing' ),
+					'description'       => __('Limit result set to invoices for specific user ids.', 'invoicing'),
 					'type'              => 'array',
 					'items'             => array(
 						'type' => 'integer',
@@ -59,7 +59,7 @@ 
 				),
 
 				'exclude_customers' => array(
-					'description'       => __( 'Exclude invoices to specific users.', 'invoicing' ),
+					'description'       => __('Exclude invoices to specific users.', 'invoicing'),
 					'type'              => 'array',
 					'items'             => array(
 						'type' => 'integer',
@@ -69,7 +69,7 @@ 
 				),
 
 				'parent'            => array(
-					'description'       => __( 'Limit result set to those of particular parent IDs.', 'invoicing' ),
+					'description'       => __('Limit result set to those of particular parent IDs.', 'invoicing'),
 					'type'              => 'array',
 					'items'             => array(
 						'type' => 'integer',
@@ -79,7 +79,7 @@ 
 				),
 
 				'parent_exclude'    => array(
-					'description'       => __( 'Limit result set to all items except those of a particular parent ID.', 'invoicing' ),
+					'description'       => __('Limit result set to all items except those of a particular parent ID.', 'invoicing'),
 					'type'              => 'array',
 					'items'             => array(
 						'type' => 'integer',
@@ -92,7 +92,7 @@ 
 		);
 
 		// Filter collection parameters for the invoices controller.
-		return apply_filters( 'getpaid_rest_invoices_collection_params', $params, $this );
+		return apply_filters('getpaid_rest_invoices_collection_params', $params, $this);
 	}
 
 	/**
@@ -103,21 +103,21 @@ 
 	 * @param WP_REST_Request $request Request object.
 	 * @return array          $query_args
 	 */
-	protected function prepare_items_query( $prepared_args = array(), $request = null ) {
+	protected function prepare_items_query($prepared_args = array(), $request = null) {
 
-		$query_args = parent::prepare_items_query( $prepared_args );
+		$query_args = parent::prepare_items_query($prepared_args);
 
 		// Retrieve invoices for specific customers.
-		if ( ! empty( $request['customers'] ) ) {
+		if (!empty($request['customers'])) {
 			$query_args['author__in'] = $request['customers'];
 		}
 
 		// Skip invoices for specific customers.
-		if ( ! empty( $request['exclude_customers'] ) ) {
+		if (!empty($request['exclude_customers'])) {
 			$query_args['author__not_in'] = $request['exclude_customers'];
 		}
 
-		return apply_filters( 'getpaid_rest_invoices_prepare_items_query', $query_args, $request, $this );
+		return apply_filters('getpaid_rest_invoices_prepare_items_query', $query_args, $request, $this);
 
 	}
 
@@ -129,7 +129,7 @@ 
 	 * @return array A list of registered item statuses.
 	 */
 	public function get_post_statuses() {
-		return array_keys( wpinv_get_invoice_statuses( true, false, $this->post_type ) );
+		return array_keys(wpinv_get_invoice_statuses(true, false, $this->post_type));
 	}
 
 	/**
@@ -138,9 +138,9 @@ 
 	 * @param WPInv_Invoice $invoice Invoice to save.
 	 * @return WP_Error|WPInv_Invoice
 	 */
-	protected function save_object( $invoice ) {
+	protected function save_object($invoice) {
 		$invoice->recalculate_total();
-		return parent::save_object( $invoice );
+		return parent::save_object($invoice);
 	}
 
 }

includes/api/class-getpaid-rest-reports-controller.php

Spacing +32 added lines, -32 removed lines

@@ -9,7 +9,7 @@ 
  * @since   2.0.0
  */
 
-defined( 'ABSPATH' ) || exit;
+defined('ABSPATH') || exit;
 
 /**
  * GetPaid REST reports controller class.
@@ -32,7 +32,7 @@ 
 	 *
 	 * @see register_rest_route()
 	 */
-	public function register_namespace_routes( $namespace ) {
+	public function register_namespace_routes($namespace) {
 
 		// List all available reports.
 		register_rest_route(
@@ -41,11 +41,11 @@ 
 			array(
 				array(
 					'methods'             => WP_REST_Server::READABLE,
-					'callback'            => array( $this, 'get_items' ),
-					'permission_callback' => array( $this, 'get_items_permissions_check' ),
+					'callback'            => array($this, 'get_items'),
+					'permission_callback' => array($this, 'get_items_permissions_check'),
 					'args'                => $this->get_collection_params(),
 				),
-				'schema' => array( $this, 'get_public_item_schema' ),
+				'schema' => array($this, 'get_public_item_schema'),
 			)
 		);
 
@@ -58,10 +58,10 @@ 
 	 * @param WP_REST_Request $request Full data about the request.
 	 * @return WP_Error|boolean
 	 */
-	public function get_items_permissions_check( $request ) {
+	public function get_items_permissions_check($request) {
 
-		if ( ! wpinv_current_user_can_manage_invoicing() ) {
-			return new WP_Error( 'rest_cannot_view', __( 'Sorry, you cannot list resources.', 'invoicing' ), array( 'status' => rest_authorization_required_code() ) );
+		if (!wpinv_current_user_can_manage_invoicing()) {
+			return new WP_Error('rest_cannot_view', __('Sorry, you cannot list resources.', 'invoicing'), array('status' => rest_authorization_required_code()));
 		}
 
 		return true;
@@ -78,23 +78,23 @@ 
 		$reports = array(
 			array(
 				'slug'        => 'sales',
-				'description' => __( 'List of sales reports.', 'invoicing' ),
+				'description' => __('List of sales reports.', 'invoicing'),
 			),
 			array(
 				'slug'        => 'top_sellers',
-				'description' => __( 'List of top selling items.', 'invoicing' ),
+				'description' => __('List of top selling items.', 'invoicing'),
 			),
 			array(
 				'slug'        => 'top_earners',
-				'description' => __( 'List of top earning items.', 'invoicing' ),
+				'description' => __('List of top earning items.', 'invoicing'),
 			),
 			array(
 				'slug'        => 'invoices/counts',
-				'description' => __( 'Invoice counts.', 'invoicing' ),
+				'description' => __('Invoice counts.', 'invoicing'),
 			),
 		);
 
-		return apply_filters( 'getpaid_available_api_reports', $reports );
+		return apply_filters('getpaid_available_api_reports', $reports);
 
 	}
 
@@ -105,16 +105,16 @@ 
 	 * @param WP_REST_Request $request
 	 * @return array|WP_Error
 	 */
-	public function get_items( $request ) {
+	public function get_items($request) {
 		$data    = array();
 		$reports = $this->get_reports();
 
-		foreach ( $reports as $report ) {
-			$item   = $this->prepare_item_for_response( (object) $report, $request );
-			$data[] = $this->prepare_response_for_collection( $item );
+		foreach ($reports as $report) {
+			$item   = $this->prepare_item_for_response((object) $report, $request);
+			$data[] = $this->prepare_response_for_collection($item);
 		}
 
-		return rest_ensure_response( $data );
+		return rest_ensure_response($data);
 	}
 
 	/**
@@ -125,30 +125,30 @@ 
 	 * @param WP_REST_Request $request Request object.
 	 * @return WP_REST_Response $response Response data.
 	 */
-	public function prepare_item_for_response( $report, $request ) {
+	public function prepare_item_for_response($report, $request) {
 		$data = array(
 			'slug'        => $report->slug,
 			'description' => $report->description,
 		);
 
-		$context = ! empty( $request['context'] ) ? $request['context'] : 'view';
-		$data = $this->add_additional_fields_to_object( $data, $request );
-		$data = $this->filter_response_by_context( $data, $context );
+		$context = !empty($request['context']) ? $request['context'] : 'view';
+		$data = $this->add_additional_fields_to_object($data, $request);
+		$data = $this->filter_response_by_context($data, $context);
 
 		// Wrap the data in a response object.
-		$response = rest_ensure_response( $data );
+		$response = rest_ensure_response($data);
 		$response->add_links(
             array(
 				'self'       => array(
-					'href' => rest_url( sprintf( '/%s/%s/%s', $this->namespace, $this->rest_base, $report->slug ) ),
+					'href' => rest_url(sprintf('/%s/%s/%s', $this->namespace, $this->rest_base, $report->slug)),
 				),
 				'collection' => array(
-					'href' => rest_url( sprintf( '%s/%s', $this->namespace, $this->rest_base ) ),
+					'href' => rest_url(sprintf('%s/%s', $this->namespace, $this->rest_base)),
 				),
             )
         );
 
-		return apply_filters( 'getpaid_rest_prepare_report', $response, $report, $request );
+		return apply_filters('getpaid_rest_prepare_report', $response, $report, $request);
 	}
 
 	/**
@@ -164,21 +164,21 @@ 
 			'type'       => 'object',
 			'properties' => array(
 				'slug'        => array(
-					'description' => __( 'An alphanumeric identifier for the resource.', 'invoicing' ),
+					'description' => __('An alphanumeric identifier for the resource.', 'invoicing'),
 					'type'        => 'string',
-					'context'     => array( 'view' ),
+					'context'     => array('view'),
 					'readonly'    => true,
 				),
 				'description' => array(
-					'description' => __( 'A human-readable description of the resource.', 'invoicing' ),
+					'description' => __('A human-readable description of the resource.', 'invoicing'),
 					'type'        => 'string',
-					'context'     => array( 'view' ),
+					'context'     => array('view'),
 					'readonly'    => true,
 				),
 			),
 		);
 
-		return $this->add_additional_fields_schema( $schema );
+		return $this->add_additional_fields_schema($schema);
 	}
 
 	/**
@@ -189,7 +189,7 @@ 
 	 */
 	public function get_collection_params() {
 		return array(
-			'context' => $this->get_context_param( array( 'default' => 'view' ) ),
+			'context' => $this->get_context_param(array('default' => 'view')),
 		);
 	}
 }

includes/api/class-getpaid-rest-posts-controller.php

Spacing +127 added lines, -127 removed lines

@@ -8,7 +8,7 @@ 
  * @version 1.0.19
  */
 
-defined( 'ABSPATH' ) || exit;
+defined('ABSPATH') || exit;
 
 /**
  * GetPaid REST Posts controller class.
@@ -38,9 +38,9 @@ 
 	 *
 	 * @see register_rest_route()
 	 */
-	public function register_namespace_routes( $namespace ) {
+	public function register_namespace_routes($namespace) {
 
-		parent::register_namespace_routes( $namespace );
+		parent::register_namespace_routes($namespace);
 
 		register_rest_route(
 			$namespace,
@@ -48,11 +48,11 @@ 
 			array(
 				array(
 					'methods'             => WP_REST_Server::EDITABLE,
-					'callback'            => array( $this, 'batch_items' ),
-					'permission_callback' => array( $this, 'batch_items_permissions_check' ),
-					'args'                => $this->get_endpoint_args_for_item_schema( WP_REST_Server::EDITABLE ),
+					'callback'            => array($this, 'batch_items'),
+					'permission_callback' => array($this, 'batch_items_permissions_check'),
+					'args'                => $this->get_endpoint_args_for_item_schema(WP_REST_Server::EDITABLE),
 				),
-				'schema' => array( $this, 'get_public_batch_schema' ),
+				'schema' => array($this, 'get_public_batch_schema'),
 			)
 		);
 
@@ -66,7 +66,7 @@ 
 	 * @param int    $object_id Post ID.
 	 * @return bool
 	 */
-	public function check_post_permissions( $context = 'read', $object_id = 0 ) {
+	public function check_post_permissions($context = 'read', $object_id = 0) {
 
 		$contexts = array(
 			'read'   => 'read_private_posts',
@@ -76,11 +76,11 @@ 
 			'batch'  => 'edit_others_posts',
 		);
 
-		$cap              = $contexts[ $context ];
-		$post_type_object = get_post_type_object( $this->post_type );
-		$permission       = current_user_can( $post_type_object->cap->$cap, $object_id );
+		$cap              = $contexts[$context];
+		$post_type_object = get_post_type_object($this->post_type);
+		$permission       = current_user_can($post_type_object->cap->$cap, $object_id);
 
-		return apply_filters( 'getpaid_rest_check_permissions', $permission, $context, $object_id, $this->post_type );
+		return apply_filters('getpaid_rest_check_permissions', $permission, $context, $object_id, $this->post_type);
 	}
 
 	/**
@@ -89,8 +89,8 @@ 
 	 * @param  WP_REST_Request $request Full details about the request.
 	 * @return WP_Error|boolean
 	 */
-	public function get_items_permissions_check( $request ) {
-		return $this->check_post_permissions() ? true : new WP_Error( 'rest_cannot_view', __( 'Sorry, you cannot list resources.', 'invoicing' ), array( 'status' => rest_authorization_required_code() ) );
+	public function get_items_permissions_check($request) {
+		return $this->check_post_permissions() ? true : new WP_Error('rest_cannot_view', __('Sorry, you cannot list resources.', 'invoicing'), array('status' => rest_authorization_required_code()));
 	}
 
 	/**
@@ -99,8 +99,8 @@ 
 	 * @param  WP_REST_Request $request Full details about the request.
 	 * @return WP_Error|boolean
 	 */
-	public function create_item_permissions_check( $request ) {
-		return $this->check_post_permissions( 'create' ) ? true : new WP_Error( 'rest_cannot_create', __( 'Sorry, you are not allowed to create resources.', 'invoicing' ), array( 'status' => rest_authorization_required_code() ) );
+	public function create_item_permissions_check($request) {
+		return $this->check_post_permissions('create') ? true : new WP_Error('rest_cannot_create', __('Sorry, you are not allowed to create resources.', 'invoicing'), array('status' => rest_authorization_required_code()));
 	}
 
 	/**
@@ -109,11 +109,11 @@ 
 	 * @param  WP_REST_Request $request Full details about the request.
 	 * @return WP_Error|boolean
 	 */
-	public function get_item_permissions_check( $request ) {
-		$post = get_post( (int) $request['id'] );
+	public function get_item_permissions_check($request) {
+		$post = get_post((int) $request['id']);
 
-		if ( $post && ! $this->check_post_permissions( 'read', $post->ID ) ) {
-			return new WP_Error( 'rest_cannot_view', __( 'Sorry, you cannot view this resource.', 'invoicing' ), array( 'status' => rest_authorization_required_code() ) );
+		if ($post && !$this->check_post_permissions('read', $post->ID)) {
+			return new WP_Error('rest_cannot_view', __('Sorry, you cannot view this resource.', 'invoicing'), array('status' => rest_authorization_required_code()));
 		}
 
 		return true;
@@ -125,11 +125,11 @@ 
 	 * @param  WP_REST_Request $request Full details about the request.
 	 * @return WP_Error|boolean
 	 */
-	public function update_item_permissions_check( $request ) {
-		$post = get_post( (int) $request['id'] );
+	public function update_item_permissions_check($request) {
+		$post = get_post((int) $request['id']);
 
-		if ( $post && ! $this->check_post_permissions( 'edit', $post->ID ) ) {
-			return new WP_Error( 'rest_cannot_edit', __( 'Sorry, you are not allowed to edit this resource.', 'invoicing' ), array( 'status' => rest_authorization_required_code() ) );
+		if ($post && !$this->check_post_permissions('edit', $post->ID)) {
+			return new WP_Error('rest_cannot_edit', __('Sorry, you are not allowed to edit this resource.', 'invoicing'), array('status' => rest_authorization_required_code()));
 		}
 
 		return true;
@@ -141,11 +141,11 @@ 
 	 * @param  WP_REST_Request $request Full details about the request.
 	 * @return bool|WP_Error
 	 */
-	public function delete_item_permissions_check( $request ) {
-		$post = get_post( (int) $request['id'] );
+	public function delete_item_permissions_check($request) {
+		$post = get_post((int) $request['id']);
 
-		if ( $post && ! $this->check_post_permissions( 'delete', $post->ID ) ) {
-			return new WP_Error( 'rest_cannot_delete', __( 'Sorry, you are not allowed to delete this resource.', 'invoicing' ), array( 'status' => rest_authorization_required_code() ) );
+		if ($post && !$this->check_post_permissions('delete', $post->ID)) {
+			return new WP_Error('rest_cannot_delete', __('Sorry, you are not allowed to delete this resource.', 'invoicing'), array('status' => rest_authorization_required_code()));
 		}
 
 		return true;
@@ -158,15 +158,15 @@ 
 	 *
 	 * @return boolean|WP_Error
 	 */
-	public function batch_items_permissions_check( $request ) {
-		return $this->check_post_permissions( 'batch' ) ? true : new WP_Error( 'rest_cannot_batch', __( 'Sorry, you are not allowed to batch manipulate this resource.', 'invoicing' ), array( 'status' => rest_authorization_required_code() ) );
+	public function batch_items_permissions_check($request) {
+		return $this->check_post_permissions('batch') ? true : new WP_Error('rest_cannot_batch', __('Sorry, you are not allowed to batch manipulate this resource.', 'invoicing'), array('status' => rest_authorization_required_code()));
 	}
 
 	/**
 	 * @deprecated
 	 */
-	public function get_post( $object_id ) {
-		return $this->get_object( $object_id );
+	public function get_post($object_id) {
+		return $this->get_object($object_id);
     }
 
 	/**
@@ -175,14 +175,14 @@ 
 	 * @param WP_REST_Request $request Full details about the request.
 	 * @return WP_Error|WP_REST_Response
 	 */
-	public function get_item( $request ) {
+	public function get_item($request) {
 
 		// Fetch item.
-		$response = parent::get_item( $request );
+		$response = parent::get_item($request);
 
 		// (Maybe) add a link to the html pagee.
-		if ( $this->public && ! is_wp_error( $response ) ) {
-			$response->link_header( 'alternate', get_permalink( $this->data_object->get_id() ), array( 'type' => 'text/html' ) );
+		if ($this->public && !is_wp_error($response)) {
+			$response->link_header('alternate', get_permalink($this->data_object->get_id()), array('type' => 'text/html'));
 		}
 
 		return $response;
@@ -194,13 +194,13 @@ 
 	 * @param WP_REST_Request $request Full details about the request.
 	 * @return WP_Error|WP_REST_Response
 	 */
-	public function create_item( $request ) {
+	public function create_item($request) {
 
 		// Create item.
-		$response = parent::create_item( $request );
+		$response = parent::create_item($request);
 
 		// Fire a hook after an item is created.
-		if ( ! is_wp_error( $response ) ) {
+		if (!is_wp_error($response)) {
 
 			/**
 			 * Fires after a single item is created or updated via the REST API.
@@ -209,7 +209,7 @@ 
 			 * @param WP_REST_Request $request   Request object.
 			 * @param boolean         $creating  True when creating item, false when updating.
 			 */
-			do_action( "getpaid_rest_insert_{$this->post_type}", $this->data_object, $request, true );
+			do_action("getpaid_rest_insert_{$this->post_type}", $this->data_object, $request, true);
 
 		}
 
@@ -223,13 +223,13 @@ 
 	 * @param WP_REST_Request $request Full details about the request.
 	 * @return WP_Error|WP_REST_Response
 	 */
-	public function update_item( $request ) {
+	public function update_item($request) {
 
 		// Create item.
-		$response = parent::update_item( $request );
+		$response = parent::update_item($request);
 
 		// Fire a hook after an item is created.
-		if ( ! is_wp_error( $response ) ) {
+		if (!is_wp_error($response)) {
 
 			/**
 			 * Fires after a single item is created or updated via the REST API.
@@ -238,7 +238,7 @@ 
 			 * @param WP_REST_Request $request   Request object.
 			 * @param boolean         $creating  True when creating item, false when updating.
 			 */
-			do_action( "getpaid_rest_insert_{$this->post_type}", $this->data_object, $request, false );
+			do_action("getpaid_rest_insert_{$this->post_type}", $this->data_object, $request, false);
 
 		}
 
@@ -252,7 +252,7 @@ 
 	 * @param WP_REST_Request $request Full details about the request.
 	 * @return WP_Error|WP_REST_Response
 	 */
-	public function get_items( $request ) {
+	public function get_items($request) {
 
 		$args                         = array();
 		$args['offset']               = $request['offset'];
@@ -266,17 +266,17 @@ 
 		$args['post_parent__in']      = $request['parent'];
 		$args['post_parent__not_in']  = $request['parent_exclude'];
 		$args['s']                    = $request['search'];
-		$args['post_status']          = wpinv_parse_list( $request['status'] );
+		$args['post_status']          = wpinv_parse_list($request['status']);
 
 		$args['date_query'] = array();
 
 		// Set before into date query. Date query must be specified as an array of an array.
-		if ( isset( $request['before'] ) ) {
+		if (isset($request['before'])) {
 			$args['date_query'][0]['before'] = $request['before'];
 		}
 
 		// Set after into date query. Date query must be specified as an array of an array.
-		if ( isset( $request['after'] ) ) {
+		if (isset($request['after'])) {
 			$args['date_query'][0]['after'] = $request['after'];
 		}
 
@@ -285,54 +285,54 @@ 
 		$args['fields']    = 'ids';
 
 		// Filter the query arguments for a request.
-		$args       = apply_filters( "getpaid_rest_{$this->post_type}_query", $args, $request );
-		$query_args = $this->prepare_items_query( $args, $request );
+		$args       = apply_filters("getpaid_rest_{$this->post_type}_query", $args, $request);
+		$query_args = $this->prepare_items_query($args, $request);
 
 		$posts_query = new WP_Query();
-		$query_result = $posts_query->query( $query_args );
+		$query_result = $posts_query->query($query_args);
 
 		$posts = array();
-		foreach ( $query_result as $post_id ) {
-			if ( ! $this->check_post_permissions( 'read', $post_id ) ) {
+		foreach ($query_result as $post_id) {
+			if (!$this->check_post_permissions('read', $post_id)) {
 				continue;
 			}
 
-			$data    = $this->prepare_item_for_response( $this->get_object( $post_id ), $request );
-			$posts[] = $this->prepare_response_for_collection( $data );
+			$data    = $this->prepare_item_for_response($this->get_object($post_id), $request);
+			$posts[] = $this->prepare_response_for_collection($data);
 		}
 
 		$page        = (int) $query_args['paged'];
 		$total_posts = $posts_query->found_posts;
 
-		if ( $total_posts < 1 ) {
+		if ($total_posts < 1) {
 			// Out-of-bounds, run the query again without LIMIT for total count.
-			unset( $query_args['paged'] );
+			unset($query_args['paged']);
 			$count_query = new WP_Query();
-			$count_query->query( $query_args );
+			$count_query->query($query_args);
 			$total_posts = $count_query->found_posts;
 		}
 
-		$max_pages = ceil( $total_posts / (int) $query_args['posts_per_page'] );
+		$max_pages = ceil($total_posts / (int) $query_args['posts_per_page']);
 
-		$response = rest_ensure_response( $posts );
-		$response->header( 'X-WP-Total', (int) $total_posts );
-		$response->header( 'X-WP-TotalPages', (int) $max_pages );
+		$response = rest_ensure_response($posts);
+		$response->header('X-WP-Total', (int) $total_posts);
+		$response->header('X-WP-TotalPages', (int) $max_pages);
 
 		$request_params = $request->get_query_params();
-		$base = add_query_arg( $request_params, rest_url( sprintf( '/%s/%s', $this->namespace, $this->rest_base ) ) );
+		$base = add_query_arg($request_params, rest_url(sprintf('/%s/%s', $this->namespace, $this->rest_base)));
 
-		if ( $page > 1 ) {
+		if ($page > 1) {
 			$prev_page = $page - 1;
-			if ( $prev_page > $max_pages ) {
+			if ($prev_page > $max_pages) {
 				$prev_page = $max_pages;
 			}
-			$prev_link = add_query_arg( 'page', $prev_page, $base );
-			$response->link_header( 'prev', $prev_link );
+			$prev_link = add_query_arg('page', $prev_page, $base);
+			$response->link_header('prev', $prev_link);
 		}
-		if ( $max_pages > $page ) {
+		if ($max_pages > $page) {
 			$next_page = $page + 1;
-			$next_link = add_query_arg( 'page', $next_page, $base );
-			$response->link_header( 'next', $next_link );
+			$next_link = add_query_arg('page', $next_page, $base);
+			$response->link_header('next', $next_link);
 		}
 
 		return $response;
@@ -344,26 +344,26 @@ 
 	 * @param WP_REST_Request $request Full details about the request.
 	 * @return WP_REST_Response|WP_Error
 	 */
-	public function delete_item( $request ) {
+	public function delete_item($request) {
 
 		// Fetch the item.
-		$item = $this->get_object( $request['id'] );
-		if ( is_wp_error( $item ) ) {
+		$item = $this->get_object($request['id']);
+		if (is_wp_error($item)) {
 			return $item;
 		}
 
 		$supports_trash = EMPTY_TRASH_DAYS > 0;
 		$force          = $supports_trash && (bool) $request['force'];
 
-		if ( ! $this->check_post_permissions( 'delete', $item->ID ) ) {
-			return new WP_Error( 'cannot_delete', __( 'Sorry, you are not allowed to delete this resource.', 'invoicing' ), array( 'status' => rest_authorization_required_code() ) );
+		if (!$this->check_post_permissions('delete', $item->ID)) {
+			return new WP_Error('cannot_delete', __('Sorry, you are not allowed to delete this resource.', 'invoicing'), array('status' => rest_authorization_required_code()));
 		}
 
-		$request->set_param( 'context', 'edit' );
-		$response = $this->prepare_item_for_response( $item, $request );
+		$request->set_param('context', 'edit');
+		$response = $this->prepare_item_for_response($item, $request);
 
-		if ( ! wp_delete_post( $item->ID, $force ) ) {
-			return new WP_Error( 'rest_cannot_delete', sprintf( __( 'The resource cannot be deleted.', 'invoicing' ), $this->post_type ), array( 'status' => 500 ) );
+		if (!wp_delete_post($item->ID, $force)) {
+			return new WP_Error('rest_cannot_delete', sprintf(__('The resource cannot be deleted.', 'invoicing'), $this->post_type), array('status' => 500));
 		}
 
 		return $response;
@@ -375,27 +375,27 @@ 
 	 * @param GetPaid_Data    $object GetPaid_Data object.
 	 * @return array Links for the given object.
 	 */
-	protected function prepare_links( $object ) {
+	protected function prepare_links($object) {
 
-		$links = parent::prepare_links( $object );
+		$links = parent::prepare_links($object);
 
-		if ( is_callable( array( $object, 'get_user_id' ) ) ) {
+		if (is_callable(array($object, 'get_user_id'))) {
 			$links['user'] = array(
-				'href'       => rest_url( 'wp/v2/users/' . call_user_func( array( $object, 'get_user_id' ) ) ),
+				'href'       => rest_url('wp/v2/users/' . call_user_func(array($object, 'get_user_id'))),
 				'embeddable' => true,
 			);
 		}
 
-		if ( is_callable( array( $object, 'get_owner' ) ) ) {
-			$links['owner']  = array(
-				'href'       => rest_url( 'wp/v2/users/' . call_user_func( array( $object, 'get_owner' ) ) ),
+		if (is_callable(array($object, 'get_owner'))) {
+			$links['owner'] = array(
+				'href'       => rest_url('wp/v2/users/' . call_user_func(array($object, 'get_owner'))),
 				'embeddable' => true,
 			);
 		}
 
-		if ( is_callable( array( $object, 'get_parent_id' ) ) && call_user_func( array( $object, 'get_parent_id' ) ) ) {
-			$links['parent']  = array(
-				'href'       => rest_url( "$this->namespace/$this->rest_base/" . call_user_func( array( $object, 'get_parent_id' ) ) ),
+		if (is_callable(array($object, 'get_parent_id')) && call_user_func(array($object, 'get_parent_id'))) {
+			$links['parent'] = array(
+				'href'       => rest_url("$this->namespace/$this->rest_base/" . call_user_func(array($object, 'get_parent_id'))),
 				'embeddable' => true,
 			);
 		}
@@ -411,27 +411,27 @@ 
 	 * @param WP_REST_Request $request Request object.
 	 * @return array          $query_args
 	 */
-	protected function prepare_items_query( $prepared_args = array(), $request = null ) {
+	protected function prepare_items_query($prepared_args = array(), $request = null) {
 
-		$valid_vars = array_flip( $this->get_allowed_query_vars() );
+		$valid_vars = array_flip($this->get_allowed_query_vars());
 		$query_args = array();
-		foreach ( $valid_vars as $var => $index ) {
-			if ( isset( $prepared_args[ $var ] ) ) {
-				$query_args[ $var ] = apply_filters( "getpaid_rest_query_var-{$var}", $prepared_args[ $var ], $index );
+		foreach ($valid_vars as $var => $index) {
+			if (isset($prepared_args[$var])) {
+				$query_args[$var] = apply_filters("getpaid_rest_query_var-{$var}", $prepared_args[$var], $index);
 			}
 		}
 
 		$query_args['ignore_sticky_posts'] = true;
 
-		if ( 'include' === $query_args['orderby'] ) {
+		if ('include' === $query_args['orderby']) {
 			$query_args['orderby'] = 'post__in';
-		} elseif ( 'id' === $query_args['orderby'] ) {
+		} elseif ('id' === $query_args['orderby']) {
 			$query_args['orderby'] = 'ID'; // ID must be capitalized.
-		} elseif ( 'slug' === $query_args['orderby'] ) {
+		} elseif ('slug' === $query_args['orderby']) {
 			$query_args['orderby'] = 'name';
 		}
 
-		return apply_filters( 'getpaid_rest_prepare_items_query', $query_args, $request, $this );
+		return apply_filters('getpaid_rest_prepare_items_query', $query_args, $request, $this);
 
 	}
 
@@ -450,12 +450,12 @@ 
 		 *
 		 * @param array  Array of allowed WP_Query query vars.
 		 */
-		$valid_vars = apply_filters( 'query_vars', $wp->public_query_vars );
+		$valid_vars = apply_filters('query_vars', $wp->public_query_vars);
 
-		$post_type_obj = get_post_type_object( $this->post_type );
-		if ( current_user_can( $post_type_obj->cap->edit_posts ) ) {
-			$private = apply_filters( 'getpaid_rest_private_query_vars', $wp->private_query_vars );
-			$valid_vars = array_merge( $valid_vars, $private );
+		$post_type_obj = get_post_type_object($this->post_type);
+		if (current_user_can($post_type_obj->cap->edit_posts)) {
+			$private = apply_filters('getpaid_rest_private_query_vars', $wp->private_query_vars);
+			$valid_vars = array_merge($valid_vars, $private);
 		}
 
 		// Define our own in addition to WP's normal vars.
@@ -477,10 +477,10 @@ 
 			'meta_compare',
 			'meta_value_num',
 		);
-		$valid_vars = array_merge( $valid_vars, $rest_valid );
+		$valid_vars = array_merge($valid_vars, $rest_valid);
 
 		// Filter allowed query vars for the REST API.
-		$valid_vars = apply_filters( 'getpaid_rest_query_vars', $valid_vars, $this );
+		$valid_vars = apply_filters('getpaid_rest_query_vars', $valid_vars, $this);
 
 		return $valid_vars;
 	}
@@ -497,31 +497,31 @@ 
 			array(
 				'status'  => array(
 					'default'           => $this->get_post_statuses(),
-					'description'       => __( 'Limit result set to resources assigned one or more statuses.', 'invoicing' ),
-					'type'              => array( 'array', 'string' ),
+					'description'       => __('Limit result set to resources assigned one or more statuses.', 'invoicing'),
+					'type'              => array('array', 'string'),
 					'items'             => array(
 						'enum' => $this->get_post_statuses(),
 						'type' => 'string',
 					),
 					'validate_callback' => 'rest_validate_request_arg',
-					'sanitize_callback' => array( $this, 'sanitize_post_statuses' ),
+					'sanitize_callback' => array($this, 'sanitize_post_statuses'),
 				),
 				'after'   => array(
-					'description'       => __( 'Limit response to resources created after a given ISO8601 compliant date.', 'invoicing' ),
+					'description'       => __('Limit response to resources created after a given ISO8601 compliant date.', 'invoicing'),
 					'type'              => 'string',
 					'format'            => 'string',
 					'validate_callback' => 'rest_validate_request_arg',
 					'sanitize_callback' => 'sanitize_text_field',
 				),
 				'before'  => array(
-					'description'       => __( 'Limit response to resources created before a given ISO8601 compliant date.', 'invoicing' ),
+					'description'       => __('Limit response to resources created before a given ISO8601 compliant date.', 'invoicing'),
 					'type'              => 'string',
 					'format'            => 'string',
 					'validate_callback' => 'rest_validate_request_arg',
 					'sanitize_callback' => 'sanitize_text_field',
 				),
 				'exclude' => array(
-					'description'       => __( 'Ensure result set excludes specific IDs.', 'invoicing' ),
+					'description'       => __('Ensure result set excludes specific IDs.', 'invoicing'),
 					'type'              => 'array',
 					'items'             => array(
 						'type' => 'integer',
@@ -531,7 +531,7 @@ 
 					'validate_callback' => 'rest_validate_request_arg',
 				),
 				'include' => array(
-					'description'       => __( 'Limit result set to specific ids.', 'invoicing' ),
+					'description'       => __('Limit result set to specific ids.', 'invoicing'),
 					'type'              => 'array',
 					'items'             => array(
 						'type' => 'integer',
@@ -541,20 +541,20 @@ 
 					'validate_callback' => 'rest_validate_request_arg',
 				),
 				'offset'  => array(
-					'description'       => __( 'Offset the result set by a specific number of items.', 'invoicing' ),
+					'description'       => __('Offset the result set by a specific number of items.', 'invoicing'),
 					'type'              => 'integer',
 					'sanitize_callback' => 'absint',
 					'validate_callback' => 'rest_validate_request_arg',
 				),
 				'order'   => array(
-					'description'       => __( 'Order sort attribute ascending or descending.', 'invoicing' ),
+					'description'       => __('Order sort attribute ascending or descending.', 'invoicing'),
 					'type'              => 'string',
 					'default'           => 'desc',
-					'enum'              => array( 'asc', 'desc' ),
+					'enum'              => array('asc', 'desc'),
 					'validate_callback' => 'rest_validate_request_arg',
 				),
 				'orderby' => array(
-					'description'       => __( 'Sort collection by object attribute.', 'invoicing' ),
+					'description'       => __('Sort collection by object attribute.', 'invoicing'),
 					'type'              => 'string',
 					'default'           => 'date',
 					'enum'              => array(
@@ -581,25 +581,25 @@ 
 	public function get_item_schema() {
 
 		// Maybe retrieve the schema from cache.
-		if ( ! empty( $this->schema ) ) {
-			return $this->add_additional_fields_schema( $this->schema );
+		if (!empty($this->schema)) {
+			return $this->add_additional_fields_schema($this->schema);
 		}
 
-		$type   = str_replace( 'wpi_', '', $this->post_type );
+		$type   = str_replace('wpi_', '', $this->post_type);
 		$schema = array(
 			'$schema'    => 'http://json-schema.org/draft-04/schema#',
 			'title'      => $this->post_type,
 			'type'       => 'object',
-			'properties' => wpinv_get_data( "$type-schema" ),
+			'properties' => wpinv_get_data("$type-schema"),
 		);
 
 		// Filters the invoice schema for the REST API.
-        $schema = apply_filters( "wpinv_rest_{$type}_schema", $schema );
+        $schema = apply_filters("wpinv_rest_{$type}_schema", $schema);
 
 		// Cache the invoice schema.
 		$this->schema = $schema;
 
-		return $this->add_additional_fields_schema( $this->schema );
+		return $this->add_additional_fields_schema($this->schema);
 	}
 
 	/**
@@ -612,8 +612,8 @@ 
 	 * @param string          $parameter Additional parameter to pass to validation.
 	 * @return array|WP_Error A list of valid statuses, otherwise WP_Error object.
 	 */
-	public function sanitize_post_statuses( $statuses, $request, $parameter ) {
-		return array_intersect( wp_parse_slug_list( $statuses ), $this->get_post_statuses() );
+	public function sanitize_post_statuses($statuses, $request, $parameter) {
+		return array_intersect(wp_parse_slug_list($statuses), $this->get_post_statuses());
 	}
 
 	/**
@@ -635,9 +635,9 @@ 
 	 * @param  string       $field_key The key to check for.
 	 * @return bool
 	 */
-	public function object_supports_field( $object, $field_key ) {
-		$supports = parent::object_supports_field( $object, $field_key );
-		return apply_filters( "getpaid_rest_{$this->post_type}_object_supports_key", $supports, $object, $field_key );
+	public function object_supports_field($object, $field_key) {
+		$supports = parent::object_supports_field($object, $field_key);
+		return apply_filters("getpaid_rest_{$this->post_type}_object_supports_key", $supports, $object, $field_key);
 	}
 
 }

includes/class-wpinv-cli.php

Spacing +16 added lines, -16 removed lines

@@ -7,7 +7,7 @@ 
  */
 
 // MUST have WordPress.
-if ( ! defined( 'WPINC' ) ) {
+if (!defined('WPINC')) {
     exit;
 }
 
@@ -24,40 +24,40 @@ 
      *  @param Array $assoc_args Key value arguments stored in associated array format.
      *  @since 1.0.13
      */
-    public function insert_invoice( $args, $assoc_args ) {
+    public function insert_invoice($args, $assoc_args) {
 
         // Fetch invoice data from the args
-        $invoice_data = wp_unslash( $assoc_args );
+        $invoice_data = wp_unslash($assoc_args);
 
         // Abort if no invoice data is provided
-        if ( empty( $invoice_data ) ) {
-            return WP_CLI::error( __( 'Invoice data not provided', 'invoicing' ) );
+        if (empty($invoice_data)) {
+            return WP_CLI::error(__('Invoice data not provided', 'invoicing'));
         }
 
         //Cart details
-        if ( ! empty( $invoice_data['cart_details'] ) ) {
-            $invoice_data['cart_details'] = json_decode( $invoice_data['cart_details'], true );
+        if (!empty($invoice_data['cart_details'])) {
+            $invoice_data['cart_details'] = json_decode($invoice_data['cart_details'], true);
         }
 
         //User details
-        if ( ! empty( $invoice_data['user_info'] ) ) {
-            $invoice_data['user_info'] = json_decode( $invoice_data['user_info'], true );
+        if (!empty($invoice_data['user_info'])) {
+            $invoice_data['user_info'] = json_decode($invoice_data['user_info'], true);
         }
 
         //Payment info
-        if ( ! empty( $invoice_data['payment_details'] ) ) {
-            $invoice_data['payment_details'] = json_decode( $invoice_data['payment_details'], true );
+        if (!empty($invoice_data['payment_details'])) {
+            $invoice_data['payment_details'] = json_decode($invoice_data['payment_details'], true);
         }
 
         // Try creating the invoice
-        $invoice = wpinv_insert_invoice( $invoice_data, true );
+        $invoice = wpinv_insert_invoice($invoice_data, true);
 
-        if ( is_wp_error( $invoice ) ) {
-            return WP_CLI::error( $invoice->get_error_message() );
+        if (is_wp_error($invoice)) {
+            return WP_CLI::error($invoice->get_error_message());
         }
 
-        $message = sprintf( __( 'Invoice %s created', 'invoicing' ), $invoice->get_id() );
-        WP_CLI::success( $message );
+        $message = sprintf(__('Invoice %s created', 'invoicing'), $invoice->get_id());
+        WP_CLI::success($message);
     }