AyeCode / invoicing

includes/admin/class-wpinv-subscriptions-list-table.php

Spacing +94 added lines, -94 removed lines

@@ -3,11 +3,11 @@ 
  * Displays a list of all subscriptions rules
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if (!defined('ABSPATH')) {
 exit;
 }
 
-if ( ! class_exists( 'WP_List_Table' ) ) {
+if (!class_exists('WP_List_Table')) {
 	include_once ABSPATH . 'wp-admin/includes/class-wp-list-table.php';
 }
 
@@ -80,7 +80,7 @@ 
 
 		$this->prepare_query();
 
-		$this->base_url = remove_query_arg( 'status' );
+		$this->base_url = remove_query_arg('status');
 
 	}
 
@@ -93,13 +93,13 @@ 
 		$query = array(
 			'number'      => $this->per_page,
 			'paged'       => $this->get_paged(),
-			'status'      => ( isset( $_GET['status'] ) && array_key_exists( $_GET['status'], getpaid_get_subscription_statuses() ) ) ? sanitize_text_field( $_GET['status'] ) : 'all',
-			'orderby'     => ( isset( $_GET['orderby'] ) ) ? sanitize_text_field( $_GET['orderby'] ) : 'id',
-			'order'       => ( isset( $_GET['order'] ) ) ? sanitize_text_field( $_GET['order'] ) : 'DESC',
+			'status'      => (isset($_GET['status']) && array_key_exists($_GET['status'], getpaid_get_subscription_statuses())) ? sanitize_text_field($_GET['status']) : 'all',
+			'orderby'     => (isset($_GET['orderby'])) ? sanitize_text_field($_GET['orderby']) : 'id',
+			'order'       => (isset($_GET['order'])) ? sanitize_text_field($_GET['order']) : 'DESC',
 			'customer_in' => $this->get_user_in(),
 		);
 
-		if ( is_array( $query['customer_in'] ) && empty( $query['customer_in'] ) ) {
+		if (is_array($query['customer_in']) && empty($query['customer_in'])) {
 			$this->total_count         = 0;
 			$this->current_total_count = 0;
 			$this->items               = array();
@@ -108,15 +108,15 @@ 
 		}
 
 		// Prepare class properties.
-		$this->query               = new GetPaid_Subscriptions_Query( $query );
+		$this->query               = new GetPaid_Subscriptions_Query($query);
 		$this->total_count         = $this->query->get_total();
 		$this->current_total_count = $this->query->get_total();
 		$this->items               = $this->query->get_results();
-		$this->status_counts       = getpaid_get_subscription_status_counts( $query );
+		$this->status_counts       = getpaid_get_subscription_status_counts($query);
 
-		if ( 'all' != $query['status'] ) {
-			unset( $query['status'] );
-			$this->total_count   = getpaid_get_subscriptions( $query, 'count' );
+		if ('all' != $query['status']) {
+			unset($query['status']);
+			$this->total_count = getpaid_get_subscriptions($query, 'count');
 		}
 
 	}
@@ -128,14 +128,14 @@ 
 	public function get_user_in() {
 
 		// Abort if no user.
-		if ( empty( $_GET['s'] ) ) {
+		if (empty($_GET['s'])) {
 			return null;
 		}
 
 		// Or invalid user.
-		$user = wp_unslash( sanitize_text_field( $_REQUEST['s'] ) );
+		$user = wp_unslash(sanitize_text_field($_REQUEST['s']));
 
-		if ( empty( $user ) ) {
+		if (empty($user)) {
 			return null;
 		}
 
@@ -164,26 +164,26 @@ 
 	 */
 	public function get_views() {
 
-		$current  = isset( $_GET['status'] ) ? sanitize_text_field( $_GET['status'] ) : 'all';
+		$current  = isset($_GET['status']) ? sanitize_text_field($_GET['status']) : 'all';
 		$views    = array(
 
 			'all' => sprintf(
 				'<a href="%s" %s>%s&nbsp;<span class="count">(%d)</span></a>',
-				esc_url( add_query_arg( 'status', false, $this->base_url ) ),
+				esc_url(add_query_arg('status', false, $this->base_url)),
 				$current === 'all' ? ' class="current"' : '',
-				__( 'All', 'invoicing' ),
+				__('All', 'invoicing'),
 				$this->total_count
 			),
 
 		);
 
-		foreach ( array_filter( $this->status_counts ) as $status => $count ) {
+		foreach (array_filter($this->status_counts) as $status => $count) {
 
-			$views[ $status ] = sprintf(
+			$views[$status] = sprintf(
 				'<a href="%s" %s>%s&nbsp;<span class="count">(%d)</span></a>',
-				esc_url( add_query_arg( 'status', urlencode( $status ), $this->base_url ) ),
+				esc_url(add_query_arg('status', urlencode($status), $this->base_url)),
 				$current === $status ? ' class="current"' : '',
-				esc_html( getpaid_get_subscription_status_label( $status ) ),
+				esc_html(getpaid_get_subscription_status_label($status)),
 				$count
 			);
 
@@ -200,8 +200,8 @@ 
 	 * @since       1.0.0
 	 * @return      string
 	 */
-	public function column_default( $item, $column_name ) {
-		return apply_filters( "getpaid_subscriptions_table_column_$column_name", $item->$column_name );
+	public function column_default($item, $column_name) {
+		return apply_filters("getpaid_subscriptions_table_column_$column_name", $item->$column_name);
 	}
 
 	/**
@@ -210,8 +210,8 @@ 
 	 * @param WPInv_Subscription $item
 	 * @return string
 	 */
-	public function column_cb( $item ) {
-		return sprintf( '<input type="checkbox" name="id[]" value="%s" />', esc_html( $item->get_id() ) );
+	public function column_cb($item) {
+		return sprintf('<input type="checkbox" name="id[]" value="%s" />', esc_html($item->get_id()));
 	}
 
 	/**
@@ -221,7 +221,7 @@ 
 	 * @since       1.0.0
 	 * @return      string
 	 */
-	public function column_status( $item ) {
+	public function column_status($item) {
 		return $item->get_status_label_html();
 	}
 
@@ -232,26 +232,26 @@ 
 	 * @since       1.0.0
 	 * @return      string
 	 */
-	public function column_subscription( $item ) {
+	public function column_subscription($item) {
 
-		$username = __( '(Missing User)', 'invoicing' );
+		$username = __('(Missing User)', 'invoicing');
 
-		$user = get_userdata( $item->get_customer_id() );
-		if ( $user ) {
+		$user = get_userdata($item->get_customer_id());
+		if ($user) {
 
 			$username = sprintf(
 				'<a href="user-edit.php?user_id=%s">%s</a>',
-				absint( $user->ID ),
-				! empty( $user->display_name ) ? esc_html( $user->display_name ) : sanitize_email( $user->user_email )
+				absint($user->ID),
+				!empty($user->display_name) ? esc_html($user->display_name) : sanitize_email($user->user_email)
 			);
 
 		}
 
 		// translators: $1: is opening link, $2: is subscription id number, $3: is closing link tag, $4: is user's name
 		$column_content = sprintf(
-			_x( '%1$s#%2$s%3$s for %4$s', 'Subscription title on admin table. (e.g.: #211 for John Doe)', 'invoicing' ),
-			'<a href="' . esc_url( admin_url( 'admin.php?page=wpinv-subscriptions&id=' . absint( $item->get_id() ) ) ) . '">',
-			'<strong>' . esc_attr( $item->get_id() ) . '</strong>',
+			_x('%1$s#%2$s%3$s for %4$s', 'Subscription title on admin table. (e.g.: #211 for John Doe)', 'invoicing'),
+			'<a href="' . esc_url(admin_url('admin.php?page=wpinv-subscriptions&id=' . absint($item->get_id()))) . '">',
+			'<strong>' . esc_attr($item->get_id()) . '</strong>',
             '</a>',
 			$username
 		);
@@ -259,18 +259,18 @@ 
 		$row_actions = array();
 
 		// View subscription.
-		$view_url    = esc_url( add_query_arg( 'id', $item->get_id(), admin_url( 'admin.php?page=wpinv-subscriptions' ) ) );
-		$row_actions['view'] = '<a href="' . $view_url . '">' . __( 'View Subscription', 'invoicing' ) . '</a>';
+		$view_url    = esc_url(add_query_arg('id', $item->get_id(), admin_url('admin.php?page=wpinv-subscriptions')));
+		$row_actions['view'] = '<a href="' . $view_url . '">' . __('View Subscription', 'invoicing') . '</a>';
 
 		// View invoice.
-		$invoice = get_post( $item->get_parent_invoice_id() );
+		$invoice = get_post($item->get_parent_invoice_id());
 
-		if ( ! empty( $invoice ) ) {
-			$invoice_url            = get_edit_post_link( $invoice );
-			$row_actions['invoice'] = '<a href="' . $invoice_url . '">' . __( 'View Invoice', 'invoicing' ) . '</a>';
+		if (!empty($invoice)) {
+			$invoice_url            = get_edit_post_link($invoice);
+			$row_actions['invoice'] = '<a href="' . $invoice_url . '">' . __('View Invoice', 'invoicing') . '</a>';
 		}
 
-		$delete_url            = esc_url(
+		$delete_url = esc_url(
 			wp_nonce_url(
 				add_query_arg(
 					array(
@@ -282,11 +282,11 @@ 
 				'getpaid-nonce'
 			)
 		);
-		$row_actions['delete'] = '<a class="text-danger" href="' . $delete_url . '">' . __( 'Delete Subscription', 'invoicing' ) . '</a>';
+		$row_actions['delete'] = '<a class="text-danger" href="' . $delete_url . '">' . __('Delete Subscription', 'invoicing') . '</a>';
 
-		$row_actions = $this->row_actions( apply_filters( 'getpaid_subscription_table_row_actions', $row_actions, $item ) );
+		$row_actions = $this->row_actions(apply_filters('getpaid_subscription_table_row_actions', $row_actions, $item));
 
-		return "<strong>$column_content</strong>" . $this->column_amount( $item ) . $row_actions;
+		return "<strong>$column_content</strong>" . $this->column_amount($item) . $row_actions;
 	}
 
 	/**
@@ -296,8 +296,8 @@ 
 	 * @since       1.0.0
 	 * @return      string
 	 */
-	public function column_renewal_date( $item ) {
-		return getpaid_format_date_value( $item->get_expiration() );
+	public function column_renewal_date($item) {
+		return getpaid_format_date_value($item->get_expiration());
 	}
 
 	/**
@@ -307,27 +307,27 @@ 
 	 * @since       1.0.0
 	 * @return      string
 	 */
-	public function column_start_date( $item ) {
+	public function column_start_date($item) {
 
 		$gateway = $item->get_parent_invoice()->get_gateway_title();
 
-		if ( empty( $gateway ) ) {
-			return getpaid_format_date_value( $item->get_date_created() );
+		if (empty($gateway)) {
+			return getpaid_format_date_value($item->get_date_created());
 		}
 
-		$url = apply_filters( 'getpaid_remote_subscription_profile_url', '', $item );
-		if ( ! empty( $url ) ) {
+		$url = apply_filters('getpaid_remote_subscription_profile_url', '', $item);
+		if (!empty($url)) {
 
-			return getpaid_format_date_value( $item->get_date_created() ) . '<br>' . sprintf(
-				__( 'Via %s', 'invoicing' ),
-				'<strong><a href="' . esc_url( $url ) . '" target="_blank">' . esc_html( $item->get_parent_invoice()->get_gateway_title() ) . '</a></strong>'
+			return getpaid_format_date_value($item->get_date_created()) . '<br>' . sprintf(
+				__('Via %s', 'invoicing'),
+				'<strong><a href="' . esc_url($url) . '" target="_blank">' . esc_html($item->get_parent_invoice()->get_gateway_title()) . '</a></strong>'
 			);
 
 		}
 
-		return getpaid_format_date_value( $item->get_date_created() ) . '<br>' . sprintf(
-			__( 'Via %s', 'invoicing' ),
-			'<strong>' . esc_html( $item->get_parent_invoice()->get_gateway_title() ) . '</strong>'
+		return getpaid_format_date_value($item->get_date_created()) . '<br>' . sprintf(
+			__('Via %s', 'invoicing'),
+			'<strong>' . esc_html($item->get_parent_invoice()->get_gateway_title()) . '</strong>'
 		);
 
 	}
@@ -339,8 +339,8 @@ 
 	 * @since       1.0.19
 	 * @return      string
 	 */
-	public static function column_amount( $item ) {
-		$amount = getpaid_get_formatted_subscription_amount( $item );
+	public static function column_amount($item) {
+		$amount = getpaid_get_formatted_subscription_amount($item);
 		return "<span class='text-muted form-text mt-2 mb-2'>$amount</span>";
 	}
 
@@ -351,9 +351,9 @@ 
 	 * @since       1.0.0
 	 * @return      string
 	 */
-	public function column_renewals( $item ) {
+	public function column_renewals($item) {
 		$max_bills = $item->get_bill_times();
-		return $item->get_times_billed() . ' / ' . ( empty( $max_bills ) ? '&infin;' : $max_bills );
+		return $item->get_times_billed() . ' / ' . (empty($max_bills) ? '&infin;' : $max_bills);
 	}
 
 	/**
@@ -363,15 +363,15 @@ 
 	 * @since       1.0.0
 	 * @return      string
 	 */
-	public function column_item( $item ) {
-		$subscription_group = getpaid_get_invoice_subscription_group( $item->get_parent_invoice_id(), $item->get_id() );
+	public function column_item($item) {
+		$subscription_group = getpaid_get_invoice_subscription_group($item->get_parent_invoice_id(), $item->get_id());
 
-		if ( empty( $subscription_group ) ) {
-			return $this->generate_item_markup( $item->get_product_id() );
+		if (empty($subscription_group)) {
+			return $this->generate_item_markup($item->get_product_id());
 		}
 
-		$markup = array_map( array( $this, 'generate_item_markup' ), array_keys( $subscription_group['items'] ) );
-		return implode( ' | ', $markup );
+		$markup = array_map(array($this, 'generate_item_markup'), array_keys($subscription_group['items']));
+		return implode(' | ', $markup);
 
 	}
 
@@ -382,16 +382,16 @@ 
 	 * @since       1.0.0
 	 * @return      string
 	 */
-	public static function generate_item_markup( $item_id ) {
-		$item = get_post( $item_id );
+	public static function generate_item_markup($item_id) {
+		$item = get_post($item_id);
 
-		if ( ! empty( $item ) ) {
-			$link = get_edit_post_link( $item );
-			$link = esc_url( $link );
-			$name = esc_html( get_the_title( $item ) );
+		if (!empty($item)) {
+			$link = get_edit_post_link($item);
+			$link = esc_url($link);
+			$name = esc_html(get_the_title($item));
 			return wpinv_current_user_can_manage_invoicing() ? "<a href='$link'>$name</a>" : $name;
 		} else {
-			return sprintf( __( 'Item #%s', 'invoicing' ), $item_id );
+			return sprintf(__('Item #%s', 'invoicing'), $item_id);
 		}
 
 	}
@@ -402,7 +402,7 @@ 
 	 * @return      int
 	 */
 	public function get_paged() {
-		return isset( $_GET['paged'] ) ? absint( $_GET['paged'] ) : 1;
+		return isset($_GET['paged']) ? absint($_GET['paged']) : 1;
 	}
 
 	/**
@@ -415,13 +415,13 @@ 
 		$hidden   = array();
 		$sortable = $this->get_sortable_columns();
 
-		$this->_column_headers = array( $columns, $hidden, $sortable );
+		$this->_column_headers = array($columns, $hidden, $sortable);
 
 		$this->set_pagination_args(
 			array(
 				'total_items' => $this->current_total_count,
 				'per_page'    => $this->per_page,
-				'total_pages' => ceil( $this->current_total_count / $this->per_page ),
+				'total_pages' => ceil($this->current_total_count / $this->per_page),
 			)
 		);
 	}
@@ -434,15 +434,15 @@ 
 	public function get_columns() {
 		$columns = array(
 			'cb'           => '<input type="checkbox" />',
-			'subscription' => __( 'Subscription', 'invoicing' ),
-			'start_date'   => __( 'Start Date', 'invoicing' ),
-			'renewal_date' => __( 'Next Payment', 'invoicing' ),
-			'renewals'     => __( 'Payments', 'invoicing' ),
-			'item'         => __( 'Items', 'invoicing' ),
-			'status'       => __( 'Status', 'invoicing' ),
+			'subscription' => __('Subscription', 'invoicing'),
+			'start_date'   => __('Start Date', 'invoicing'),
+			'renewal_date' => __('Next Payment', 'invoicing'),
+			'renewals'     => __('Payments', 'invoicing'),
+			'item'         => __('Items', 'invoicing'),
+			'status'       => __('Status', 'invoicing'),
 		);
 
-		return apply_filters( 'manage_getpaid_subscriptions_table_columns', $columns );
+		return apply_filters('manage_getpaid_subscriptions_table_columns', $columns);
 	}
 
 	/**
@@ -452,15 +452,15 @@ 
 	 */
 	public function get_sortable_columns() {
 		$sortable = array(
-			'subscription' => array( 'id', true ),
-			'start_date'   => array( 'created', true ),
-			'renewal_date' => array( 'expiration', true ),
-			'renewals'     => array( 'bill_times', true ),
-			'item'         => array( 'product_id', true ),
-			'status'       => array( 'status', true ),
+			'subscription' => array('id', true),
+			'start_date'   => array('created', true),
+			'renewal_date' => array('expiration', true),
+			'renewals'     => array('bill_times', true),
+			'item'         => array('product_id', true),
+			'status'       => array('status', true),
 		);
 
-		return apply_filters( 'manage_getpaid_subscriptions_sortable_table_columns', $sortable );
+		return apply_filters('manage_getpaid_subscriptions_sortable_table_columns', $sortable);
 	}
 
 	/**
@@ -469,7 +469,7 @@ 
 	 * @return bool
 	 */
 	public function has_items() {
-		return ! empty( $this->current_total_count );
+		return !empty($this->current_total_count);
 	}
 
 	/**

includes/admin/class-getpaid-metaboxes.php

Spacing +62 added lines, -62 removed lines

@@ -4,7 +4,7 @@ 
  *
  */
 
-defined( 'ABSPATH' ) || exit;
+defined('ABSPATH') || exit;
 
 /**
  * Metaboxes Admin Class
@@ -25,35 +25,35 @@ 
 	public static function init() {
 
 		// Register metaboxes.
-		add_action( 'add_meta_boxes', 'GetPaid_Metaboxes::add_meta_boxes', 5, 2 );
+		add_action('add_meta_boxes', 'GetPaid_Metaboxes::add_meta_boxes', 5, 2);
 
 		// Remove metaboxes.
-		add_action( 'add_meta_boxes', 'GetPaid_Metaboxes::remove_meta_boxes', 30 );
+		add_action('add_meta_boxes', 'GetPaid_Metaboxes::remove_meta_boxes', 30);
 
 		// Rename metaboxes.
-		add_action( 'add_meta_boxes', 'GetPaid_Metaboxes::rename_meta_boxes', 45 );
+		add_action('add_meta_boxes', 'GetPaid_Metaboxes::rename_meta_boxes', 45);
 
 		// Save metaboxes.
-		add_action( 'save_post', 'GetPaid_Metaboxes::save_meta_boxes', 1, 2 );
+		add_action('save_post', 'GetPaid_Metaboxes::save_meta_boxes', 1, 2);
 	}
 
 	/**
 	 * Register core metaboxes.
 	 */
-	public static function add_meta_boxes( $post_type, $post ) {
+	public static function add_meta_boxes($post_type, $post) {
 
 		// For invoices...
-		self::add_invoice_meta_boxes( $post_type, $post );
+		self::add_invoice_meta_boxes($post_type, $post);
 
 		// For payment forms.
-		self::add_payment_form_meta_boxes( $post_type, $post );
+		self::add_payment_form_meta_boxes($post_type, $post);
 
 		// For invoice items.
-		self::add_item_meta_boxes( $post_type );
+		self::add_item_meta_boxes($post_type);
 
 		// For invoice discounts.
-		if ( $post_type == 'wpi_discount' ) {
-			add_meta_box( 'wpinv_discount_details', __( 'Discount Details', 'invoicing' ), 'GetPaid_Meta_Box_Discount_Details::output', 'wpi_discount', 'normal', 'high' );
+		if ($post_type == 'wpi_discount') {
+			add_meta_box('wpinv_discount_details', __('Discount Details', 'invoicing'), 'GetPaid_Meta_Box_Discount_Details::output', 'wpi_discount', 'normal', 'high');
 		}
 
 	}
@@ -61,17 +61,17 @@ 
 	/**
 	 * Register core metaboxes.
 	 */
-	protected static function add_payment_form_meta_boxes( $post_type, $post ) {
+	protected static function add_payment_form_meta_boxes($post_type, $post) {
 
 		// For payment forms.
-		if ( $post_type == 'wpi_payment_form' ) {
+		if ($post_type == 'wpi_payment_form') {
 
 			// Design payment form.
-			add_meta_box( 'wpinv-payment-form-design', __( 'Payment Form', 'invoicing' ), 'GetPaid_Meta_Box_Payment_Form::output', 'wpi_payment_form', 'normal' );
+			add_meta_box('wpinv-payment-form-design', __('Payment Form', 'invoicing'), 'GetPaid_Meta_Box_Payment_Form::output', 'wpi_payment_form', 'normal');
 
 			// Payment form information.
-			if ( $post->ID != wpinv_get_default_payment_form() ) {
-				add_meta_box( 'wpinv-payment-form-info', __( 'Details', 'invoicing' ), 'GetPaid_Meta_Box_Payment_Form_Info::output', 'wpi_payment_form', 'side' );
+			if ($post->ID != wpinv_get_default_payment_form()) {
+				add_meta_box('wpinv-payment-form-info', __('Details', 'invoicing'), 'GetPaid_Meta_Box_Payment_Form_Info::output', 'wpi_payment_form', 'side');
 			}
 }
 
@@ -80,20 +80,20 @@ 
 	/**
 	 * Register core metaboxes.
 	 */
-	protected static function add_item_meta_boxes( $post_type ) {
+	protected static function add_item_meta_boxes($post_type) {
 
-		if ( $post_type == 'wpi_item' ) {
+		if ($post_type == 'wpi_item') {
 
 			// Item details.
-			add_meta_box( 'wpinv_item_details', __( 'Item Details', 'invoicing' ), 'GetPaid_Meta_Box_Item_Details::output', 'wpi_item', 'normal', 'high' );
+			add_meta_box('wpinv_item_details', __('Item Details', 'invoicing'), 'GetPaid_Meta_Box_Item_Details::output', 'wpi_item', 'normal', 'high');
 
 			// If taxes are enabled, register the tax metabox.
-			if ( wpinv_use_taxes() ) {
-				add_meta_box( 'wpinv_item_vat', __( 'Tax', 'invoicing' ), 'GetPaid_Meta_Box_Item_VAT::output', 'wpi_item', 'normal', 'high' );
+			if (wpinv_use_taxes()) {
+				add_meta_box('wpinv_item_vat', __('Tax', 'invoicing'), 'GetPaid_Meta_Box_Item_VAT::output', 'wpi_item', 'normal', 'high');
 			}
 
 			// Item info.
-			add_meta_box( 'wpinv_field_item_info', __( 'Item info', 'invoicing' ), 'GetPaid_Meta_Box_Item_Info::output', 'wpi_item', 'side', 'core' );
+			add_meta_box('wpinv_field_item_info', __('Item info', 'invoicing'), 'GetPaid_Meta_Box_Item_Info::output', 'wpi_item', 'side', 'core');
 
 		}
 
@@ -102,20 +102,20 @@ 
 	/**
 	 * Register invoice metaboxes.
 	 */
-	protected static function add_invoice_meta_boxes( $post_type, $post ) {
+	protected static function add_invoice_meta_boxes($post_type, $post) {
 
 		// For invoices...
-		if ( getpaid_is_invoice_post_type( $post_type ) ) {
-			$invoice = new WPInv_Invoice( $post );
+		if (getpaid_is_invoice_post_type($post_type)) {
+			$invoice = new WPInv_Invoice($post);
 
 			// Resend invoice.
-			if ( ! $invoice->is_draft() ) {
+			if (!$invoice->is_draft()) {
 
 				add_meta_box(
 					'wpinv-mb-resend-invoice',
 					sprintf(
-						__( 'Resend %s', 'invoicing' ),
-						ucfirst( $invoice->get_invoice_quote_type() )
+						__('Resend %s', 'invoicing'),
+						ucfirst($invoice->get_invoice_quote_type())
 					),
 					'GetPaid_Meta_Box_Resend_Invoice::output',
 					$post_type,
@@ -126,17 +126,17 @@ 
 			}
 
 			// Subscriptions.
-			$subscriptions = getpaid_get_invoice_subscriptions( $invoice );
-			if ( ! empty( $subscriptions ) ) {
+			$subscriptions = getpaid_get_invoice_subscriptions($invoice);
+			if (!empty($subscriptions)) {
 
-				if ( is_array( $subscriptions ) ) {
-					add_meta_box( 'wpinv-mb-subscriptions', __( 'Related Subscriptions', 'invoicing' ), 'GetPaid_Meta_Box_Invoice_Subscription::output_related', $post_type, 'advanced' );
+				if (is_array($subscriptions)) {
+					add_meta_box('wpinv-mb-subscriptions', __('Related Subscriptions', 'invoicing'), 'GetPaid_Meta_Box_Invoice_Subscription::output_related', $post_type, 'advanced');
 				} else {
-					add_meta_box( 'wpinv-mb-subscriptions', __( 'Subscription Details', 'invoicing' ), 'GetPaid_Meta_Box_Invoice_Subscription::output', $post_type, 'advanced' );
+					add_meta_box('wpinv-mb-subscriptions', __('Subscription Details', 'invoicing'), 'GetPaid_Meta_Box_Invoice_Subscription::output', $post_type, 'advanced');
 				}
 
-				if ( getpaid_count_subscription_invoices( $invoice->is_renewal() ? $invoice->get_parent_id() : $invoice->get_id() ) > 1 ) {
-					add_meta_box( 'wpinv-mb-subscription-invoices', __( 'Related Payments', 'invoicing' ), 'GetPaid_Meta_Box_Invoice_Subscription::output_invoices', $post_type, 'advanced' );
+				if (getpaid_count_subscription_invoices($invoice->is_renewal() ? $invoice->get_parent_id() : $invoice->get_id()) > 1) {
+					add_meta_box('wpinv-mb-subscription-invoices', __('Related Payments', 'invoicing'), 'GetPaid_Meta_Box_Invoice_Subscription::output_invoices', $post_type, 'advanced');
 				}
 }
 
@@ -144,8 +144,8 @@ 
 			add_meta_box(
 				'wpinv-details',
 				sprintf(
-					__( '%s Details', 'invoicing' ),
-					ucfirst( $invoice->get_invoice_quote_type() )
+					__('%s Details', 'invoicing'),
+					ucfirst($invoice->get_invoice_quote_type())
 				),
 				'GetPaid_Meta_Box_Invoice_Details::output',
 				$post_type,
@@ -153,17 +153,17 @@ 
 			);
 
 			// Payment details.
-			add_meta_box( 'wpinv-payment-meta', __( 'Payment Meta', 'invoicing' ), 'GetPaid_Meta_Box_Invoice_Payment_Meta::output', $post_type, 'side', 'default' );
+			add_meta_box('wpinv-payment-meta', __('Payment Meta', 'invoicing'), 'GetPaid_Meta_Box_Invoice_Payment_Meta::output', $post_type, 'side', 'default');
 
 			// Billing details.
-			add_meta_box( 'wpinv-address', __( 'Billing Details', 'invoicing' ), 'GetPaid_Meta_Box_Invoice_Address::output', $post_type, 'normal', 'high' );
+			add_meta_box('wpinv-address', __('Billing Details', 'invoicing'), 'GetPaid_Meta_Box_Invoice_Address::output', $post_type, 'normal', 'high');
 
 			// Invoice items.
 			add_meta_box(
 				'wpinv-items',
 				sprintf(
-					__( '%s Items', 'invoicing' ),
-					ucfirst( $invoice->get_invoice_quote_type() )
+					__('%s Items', 'invoicing'),
+					ucfirst($invoice->get_invoice_quote_type())
 				),
 				'GetPaid_Meta_Box_Invoice_Items::output',
 				$post_type,
@@ -175,8 +175,8 @@ 
 			add_meta_box(
 				'wpinv-notes',
 				sprintf(
-					__( '%s Notes', 'invoicing' ),
-					ucfirst( $invoice->get_invoice_quote_type() )
+					__('%s Notes', 'invoicing'),
+					ucfirst($invoice->get_invoice_quote_type())
 				),
 				'WPInv_Meta_Box_Notes::output',
 				$post_type,
@@ -185,13 +185,13 @@ 
 			);
 
 			// Shipping Address.
-			if ( get_post_meta( $invoice->get_id(), 'shipping_address', true ) ) {
-				add_meta_box( 'wpinv-invoice-shipping-details', __( 'Shipping Address', 'invoicing' ), 'GetPaid_Meta_Box_Invoice_Shipping_Address::output', $post_type, 'side', 'high' );
+			if (get_post_meta($invoice->get_id(), 'shipping_address', true)) {
+				add_meta_box('wpinv-invoice-shipping-details', __('Shipping Address', 'invoicing'), 'GetPaid_Meta_Box_Invoice_Shipping_Address::output', $post_type, 'side', 'high');
 			}
 
 			// Payment form information.
-			if ( get_post_meta( $invoice->get_id(), 'payment_form_data', true ) ) {
-				add_meta_box( 'wpinv-invoice-payment-form-details', __( 'Payment Form Details', 'invoicing' ), 'WPInv_Meta_Box_Payment_Form::output_details', $post_type, 'side', 'high' );
+			if (get_post_meta($invoice->get_id(), 'payment_form_data', true)) {
+				add_meta_box('wpinv-invoice-payment-form-details', __('Payment Form Details', 'invoicing'), 'WPInv_Meta_Box_Payment_Form::output_details', $post_type, 'side', 'high');
 			}
 }
 
@@ -201,7 +201,7 @@ 
 	 * Remove some metaboxes.
 	 */
 	public static function remove_meta_boxes() {
-		remove_meta_box( 'wpseo_meta', 'wpi_invoice', 'normal' );
+		remove_meta_box('wpseo_meta', 'wpi_invoice', 'normal');
 	}
 
 	/**
@@ -217,46 +217,46 @@ 
 	 * @param  int    $post_id Post ID.
 	 * @param  object $post Post object.
 	 */
-	public static function save_meta_boxes( $post_id, $post ) {
-		$post_id = absint( $post_id );
-		$data    = wp_kses_post_deep( wp_unslash( $_POST ) );
+	public static function save_meta_boxes($post_id, $post) {
+		$post_id = absint($post_id);
+		$data    = wp_kses_post_deep(wp_unslash($_POST));
 
 		// Do not save for ajax requests.
-		if ( ( defined( 'DOING_AJAX' ) && DOING_AJAX ) || isset( $_REQUEST['bulk_edit'] ) ) {
+		if ((defined('DOING_AJAX') && DOING_AJAX) || isset($_REQUEST['bulk_edit'])) {
 			return;
 		}
 
 		// $post_id and $post are required
-		if ( empty( $post_id ) || empty( $post ) || self::$saved_meta_boxes ) {
+		if (empty($post_id) || empty($post) || self::$saved_meta_boxes) {
 			return;
 		}
 
 		// Dont' save meta boxes for revisions or autosaves.
-		if ( ( defined( 'DOING_AUTOSAVE' ) && DOING_AUTOSAVE ) || is_int( wp_is_post_revision( $post ) ) || is_int( wp_is_post_autosave( $post ) ) ) {
+		if ((defined('DOING_AUTOSAVE') && DOING_AUTOSAVE) || is_int(wp_is_post_revision($post)) || is_int(wp_is_post_autosave($post))) {
 			return;
 		}
 
 		// Check the nonce.
-		if ( empty( $data['getpaid_meta_nonce'] ) || ! wp_verify_nonce( $data['getpaid_meta_nonce'], 'getpaid_meta_nonce' ) ) {
+		if (empty($data['getpaid_meta_nonce']) || !wp_verify_nonce($data['getpaid_meta_nonce'], 'getpaid_meta_nonce')) {
 			return;
 		}
 
 		// Check the post being saved == the $post_id to prevent triggering this call for other save_post events.
-		if ( empty( $data['post_ID'] ) || absint( $data['post_ID'] ) !== $post_id ) {
+		if (empty($data['post_ID']) || absint($data['post_ID']) !== $post_id) {
 			return;
 		}
 
 		// Check user has permission to edit.
-		if ( ! current_user_can( 'edit_post', $post_id ) ) {
+		if (!current_user_can('edit_post', $post_id)) {
 			return;
 		}
 
-		if ( getpaid_is_invoice_post_type( $post->post_type ) ) {
+		if (getpaid_is_invoice_post_type($post->post_type)) {
 
 			// We need this save event to run once to avoid potential endless loops.
 			self::$saved_meta_boxes = true;
 
-			return GetPaid_Meta_Box_Invoice_Address::save( $post_id );
+			return GetPaid_Meta_Box_Invoice_Address::save($post_id);
 
 		}
 
@@ -268,7 +268,7 @@ 
 		);
 
 		// Is this our post type?
-		if ( ! isset( $post_types_map[ $post->post_type ] ) ) {
+		if (!isset($post_types_map[$post->post_type])) {
 			return;
 		}
 
@@ -276,8 +276,8 @@ 
 		self::$saved_meta_boxes = true;
 
 		// Save the post.
-		$class = $post_types_map[ $post->post_type ];
-		$class::save( $post_id, wp_kses_post_deep( $_POST ), $post );
+		$class = $post_types_map[$post->post_type];
+		$class::save($post_id, wp_kses_post_deep($_POST), $post);
 
 	}
 

includes/admin/class-getpaid-admin-profile.php

Spacing +49 added lines, -49 removed lines

@@ -4,11 +4,11 @@ 
  *
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if (!defined('ABSPATH')) {
 	exit; // Exit if accessed directly
 }
 
-if ( ! class_exists( 'GetPaid_Admin_Profile', false ) ) :
+if (!class_exists('GetPaid_Admin_Profile', false)) :
 
 	/**
 	 * GetPaid_Admin_Profile Class.
@@ -19,11 +19,11 @@ 
 		 * Hook in tabs.
 		 */
 		public function __construct() {
-			add_action( 'show_user_profile', array( $this, 'add_customer_meta_fields' ), 100 );
-			add_action( 'edit_user_profile', array( $this, 'add_customer_meta_fields' ), 100 );
+			add_action('show_user_profile', array($this, 'add_customer_meta_fields'), 100);
+			add_action('edit_user_profile', array($this, 'add_customer_meta_fields'), 100);
 
-			add_action( 'personal_options_update', array( $this, 'save_customer_meta_fields' ) );
-			add_action( 'edit_user_profile_update', array( $this, 'save_customer_meta_fields' ) );
+			add_action('personal_options_update', array($this, 'save_customer_meta_fields'));
+			add_action('edit_user_profile_update', array($this, 'save_customer_meta_fields'));
 		}
 
 		/**
@@ -37,54 +37,54 @@ 
 				'getpaid_customer_meta_fields',
 				array(
 					'billing' => array(
-						'title'  => __( 'Billing Details (GetPaid)', 'invoicing' ),
+						'title'  => __('Billing Details (GetPaid)', 'invoicing'),
 						'fields' => array(
 							'_wpinv_first_name' => array(
-								'label'       => __( 'First name', 'invoicing' ),
+								'label'       => __('First name', 'invoicing'),
 								'description' => '',
 							),
 							'_wpinv_last_name'  => array(
-								'label'       => __( 'Last name', 'invoicing' ),
+								'label'       => __('Last name', 'invoicing'),
 								'description' => '',
 							),
 							'_wpinv_company'    => array(
-								'label'       => __( 'Company', 'invoicing' ),
+								'label'       => __('Company', 'invoicing'),
 								'description' => '',
 							),
 							'_wpinv_company_id' => array(
-								'label'       => __( 'Company ID', 'invoicing' ),
+								'label'       => __('Company ID', 'invoicing'),
 								'description' => '',
 							),
 							'_wpinv_address'    => array(
-								'label'       => __( 'Address', 'invoicing' ),
+								'label'       => __('Address', 'invoicing'),
 								'description' => '',
 							),
 							'_wpinv_city'       => array(
-								'label'       => __( 'City', 'invoicing' ),
+								'label'       => __('City', 'invoicing'),
 								'description' => '',
 							),
 							'_wpinv_zip'        => array(
-								'label'       => __( 'Postcode / ZIP', 'invoicing' ),
+								'label'       => __('Postcode / ZIP', 'invoicing'),
 								'description' => '',
 							),
 							'_wpinv_country'    => array(
-								'label'       => __( 'Country / Region', 'invoicing' ),
+								'label'       => __('Country / Region', 'invoicing'),
 								'description' => '',
 								'class'       => 'getpaid_js_field-country',
 								'type'        => 'select',
-								'options'     => array( '' => __( 'Select a country / region…', 'invoicing' ) ) + wpinv_get_country_list(),
+								'options'     => array('' => __('Select a country / region…', 'invoicing')) + wpinv_get_country_list(),
 							),
 							'_wpinv_state'      => array(
-								'label'       => __( 'State / County', 'invoicing' ),
-								'description' => __( 'State / County or state code', 'invoicing' ),
+								'label'       => __('State / County', 'invoicing'),
+								'description' => __('State / County or state code', 'invoicing'),
 								'class'       => 'getpaid_js_field-state regular-text',
 							),
 							'_wpinv_phone'      => array(
-								'label'       => __( 'Phone', 'invoicing' ),
+								'label'       => __('Phone', 'invoicing'),
 								'description' => '',
 							),
 							'_wpinv_vat_number' => array(
-								'label'       => __( 'VAT Number', 'invoicing' ),
+								'label'       => __('VAT Number', 'invoicing'),
 								'description' => '',
 							),
 						),
@@ -99,39 +99,39 @@ 
 		 *
 		 * @param WP_User $user
 		 */
-		public function add_customer_meta_fields( $user ) {
+		public function add_customer_meta_fields($user) {
 
-			if ( ! apply_filters( 'getpaid_current_user_can_edit_customer_meta_fields', current_user_can( 'manage_options' ), $user->ID ) ) {
+			if (!apply_filters('getpaid_current_user_can_edit_customer_meta_fields', current_user_can('manage_options'), $user->ID)) {
 				return;
 			}
 
 			$show_fields = $this->get_customer_meta_fields();
 
-			foreach ( $show_fields as $fieldset_key => $fieldset ) :
+			foreach ($show_fields as $fieldset_key => $fieldset) :
 				?>
-				<h2><?php echo esc_html( $fieldset['title'] ); ?></h2>
-				<table class="form-table" id="<?php echo esc_attr( 'getpaid-fieldset-' . $fieldset_key ); ?>">
-					<?php foreach ( $fieldset['fields'] as $key => $field ) : ?>
+				<h2><?php echo esc_html($fieldset['title']); ?></h2>
+				<table class="form-table" id="<?php echo esc_attr('getpaid-fieldset-' . $fieldset_key); ?>">
+					<?php foreach ($fieldset['fields'] as $key => $field) : ?>
 						<tr>
 							<th>
-								<label for="<?php echo esc_attr( $key ); ?>"><?php echo esc_html( $field['label'] ); ?></label>
+								<label for="<?php echo esc_attr($key); ?>"><?php echo esc_html($field['label']); ?></label>
 							</th>
 							<td>
-								<?php if ( ! empty( $field['type'] ) && 'select' === $field['type'] ) : ?>
-									<select name="<?php echo esc_attr( $key ); ?>" id="<?php echo esc_attr( $key ); ?>" class="<?php echo esc_attr( $field['class'] ); ?> wpi_select2" style="width: 25em;">
+								<?php if (!empty($field['type']) && 'select' === $field['type']) : ?>
+									<select name="<?php echo esc_attr($key); ?>" id="<?php echo esc_attr($key); ?>" class="<?php echo esc_attr($field['class']); ?> wpi_select2" style="width: 25em;">
 										<?php
-											$selected = esc_attr( get_user_meta( $user->ID, $key, true ) );
-										foreach ( $field['options'] as $option_key => $option_value ) :
+											$selected = esc_attr(get_user_meta($user->ID, $key, true));
+										foreach ($field['options'] as $option_key => $option_value) :
 											?>
-											<option value="<?php echo esc_attr( $option_key ); ?>" <?php selected( $selected, $option_key, true ); ?>><?php echo esc_html( $option_value ); ?></option>
+											<option value="<?php echo esc_attr($option_key); ?>" <?php selected($selected, $option_key, true); ?>><?php echo esc_html($option_value); ?></option>
 										<?php endforeach; ?>
 									</select>
-								<?php elseif ( ! empty( $field['type'] ) && 'checkbox' === $field['type'] ) : ?>
-									<input type="checkbox" name="<?php echo esc_attr( $key ); ?>" id="<?php echo esc_attr( $key ); ?>" value="1" class="<?php echo esc_attr( $field['class'] ); ?>" <?php checked( (int) get_user_meta( $user->ID, $key, true ), 1, true ); ?> />
+								<?php elseif (!empty($field['type']) && 'checkbox' === $field['type']) : ?>
+									<input type="checkbox" name="<?php echo esc_attr($key); ?>" id="<?php echo esc_attr($key); ?>" value="1" class="<?php echo esc_attr($field['class']); ?>" <?php checked((int) get_user_meta($user->ID, $key, true), 1, true); ?> />
 								<?php else : ?>
-									<input type="text" name="<?php echo esc_attr( $key ); ?>" id="<?php echo esc_attr( $key ); ?>" value="<?php echo esc_attr( $this->get_user_meta( $user->ID, $key ) ); ?>" class="<?php echo ( ! empty( $field['class'] ) ? esc_attr( $field['class'] ) : 'regular-text' ); ?>" />
+									<input type="text" name="<?php echo esc_attr($key); ?>" id="<?php echo esc_attr($key); ?>" value="<?php echo esc_attr($this->get_user_meta($user->ID, $key)); ?>" class="<?php echo (!empty($field['class']) ? esc_attr($field['class']) : 'regular-text'); ?>" />
 								<?php endif; ?>
-								<p class="description"><?php echo wp_kses_post( $field['description'] ); ?></p>
+								<p class="description"><?php echo wp_kses_post($field['description']); ?></p>
 							</td>
 						</tr>
 					<?php endforeach; ?>
@@ -145,21 +145,21 @@ 
 		 *
 		 * @param int $user_id User ID of the user being saved
 		 */
-		public function save_customer_meta_fields( $user_id ) {
-			if ( ! apply_filters( 'getpaid_current_user_can_edit_customer_meta_fields', current_user_can( 'manage_options' ), $user_id ) ) {
+		public function save_customer_meta_fields($user_id) {
+			if (!apply_filters('getpaid_current_user_can_edit_customer_meta_fields', current_user_can('manage_options'), $user_id)) {
 				return;
 			}
 
 			$save_fields = $this->get_customer_meta_fields();
 
-			foreach ( $save_fields as $fieldset ) {
+			foreach ($save_fields as $fieldset) {
 
-				foreach ( $fieldset['fields'] as $key => $field ) {
+				foreach ($fieldset['fields'] as $key => $field) {
 
-					if ( isset( $field['type'] ) && 'checkbox' === $field['type'] ) {
-						update_user_meta( $user_id, $key, ! empty( $_POST[ $key ] ) );
-					} elseif ( isset( $_POST[ $key ] ) ) {
-						update_user_meta( $user_id, $key, wpinv_clean( $_POST[ $key ] ) );
+					if (isset($field['type']) && 'checkbox' === $field['type']) {
+						update_user_meta($user_id, $key, !empty($_POST[$key]));
+					} elseif (isset($_POST[$key])) {
+						update_user_meta($user_id, $key, wpinv_clean($_POST[$key]));
 					}
 				}
 			}
@@ -173,11 +173,11 @@ 
 		 * @param string $key     Key for user meta field
 		 * @return string
 		 */
-		protected function get_user_meta( $user_id, $key ) {
-			$value           = get_user_meta( $user_id, $key, true );
-			$existing_fields = array( '_wpinv_first_name', '_wpinv_last_name' );
-			if ( ! $value && in_array( $key, $existing_fields ) ) {
-				$value = get_user_meta( $user_id, str_replace( '_wpinv_', '', $key ), true );
+		protected function get_user_meta($user_id, $key) {
+			$value           = get_user_meta($user_id, $key, true);
+			$existing_fields = array('_wpinv_first_name', '_wpinv_last_name');
+			if (!$value && in_array($key, $existing_fields)) {
+				$value = get_user_meta($user_id, str_replace('_wpinv_', '', $key), true);
 			}
 
 			return $value;

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

Spacing +95 added lines, -95 removed lines

@@ -8,7 +8,7 @@ 
  * @version 1.0.19
  */
 
-defined( 'ABSPATH' ) || exit;
+defined('ABSPATH') || exit;
 
 /**
  * Core class to access posts via the REST API.
@@ -64,7 +64,7 @@ 
 		);
 
 		// Register REST routes.
-        add_action( 'rest_api_init', array( $this, 'register_routes' ) );
+        add_action('rest_api_init', array($this, 'register_routes'));
 
 	}
 
@@ -76,8 +76,8 @@ 
 	 */
 	public function register_routes() {
 
-		foreach ( $this->namespaces as $namespace ) {
-			$this->register_namespace_routes( $namespace );
+		foreach ($this->namespaces as $namespace) {
+			$this->register_namespace_routes($namespace);
 		}
 
 	}
@@ -89,12 +89,12 @@ 
 	 *
 	 * @param string $namespace
 	 */
-	public function register_namespace_routes( /** @scrutinizer ignore-unused */ $namespace ) {
+	public function register_namespace_routes(/** @scrutinizer ignore-unused */ $namespace) {
 
 		getpaid_doing_it_wrong(
 			__CLASS__ . '::' . __METHOD__,
 			/* translators: %s: register_namespace_routes() */
-			sprintf( __( "Method '%s' must be overridden." ), __METHOD__ ),
+			sprintf(__("Method '%s' must be overridden."), __METHOD__),
 			'1.0.19'
 		);
 
@@ -106,7 +106,7 @@ 
 	 * @return string
 	 */
 	protected function get_normalized_rest_base() {
-		return preg_replace( '/\(.*\)\//i', '', $this->rest_base );
+		return preg_replace('/\(.*\)\//i', '', $this->rest_base);
 	}
 
 	/**
@@ -115,11 +115,11 @@ 
 	 * @param array array of request items.
 	 * @return array
 	 */
-	protected function fill_batch_keys( $items ) {
+	protected function fill_batch_keys($items) {
 
-		$items['create'] = empty( $items['create'] ) ? array() : $items['create'];
-		$items['update'] = empty( $items['update'] ) ? array() : $items['update'];
-		$items['delete'] = empty( $items['delete'] ) ? array() : wp_parse_id_list( $items['delete'] );
+		$items['create'] = empty($items['create']) ? array() : $items['create'];
+		$items['update'] = empty($items['update']) ? array() : $items['update'];
+		$items['delete'] = empty($items['delete']) ? array() : wp_parse_id_list($items['delete']);
 		return $items;
 
 	}
@@ -130,13 +130,13 @@ 
 	 * @param array $items Request items.
 	 * @return bool|WP_Error
 	 */
-	protected function check_batch_limit( $items ) {
-		$limit = apply_filters( 'getpaid_rest_batch_items_limit', 100, $this->get_normalized_rest_base() );
-		$total = count( $items['create'] ) + count( $items['update'] ) + count( $items['delete'] );
+	protected function check_batch_limit($items) {
+		$limit = apply_filters('getpaid_rest_batch_items_limit', 100, $this->get_normalized_rest_base());
+		$total = count($items['create']) + count($items['update']) + count($items['delete']);
 
-		if ( $total > $limit ) {
+		if ($total > $limit) {
 			/* translators: %s: items limit */
-			return new WP_Error( 'getpaid_rest_request_entity_too_large', sprintf( __( 'Unable to accept more than %s items for this request.', 'invoicing' ), $limit ), array( 'status' => 413 ) );
+			return new WP_Error('getpaid_rest_request_entity_too_large', sprintf(__('Unable to accept more than %s items for this request.', 'invoicing'), $limit), array('status' => 413));
 		}
 
 		return true;
@@ -150,37 +150,37 @@ 
 	 * @param WP_REST_Server $wp_rest_server
 	 * @return array()
 	 */
-	protected function batch_create_items( $items, $request, $wp_rest_server ) {
+	protected function batch_create_items($items, $request, $wp_rest_server) {
 
 		$query  = $request->get_query_params();
 		$create = array();
 
-		foreach ( $items as $item ) {
-			$_item = new WP_REST_Request( 'POST' );
+		foreach ($items as $item) {
+			$_item = new WP_REST_Request('POST');
 
 			// Default parameters.
 			$defaults = array();
 			$schema   = $this->get_public_item_schema();
-			foreach ( $schema['properties'] as $arg => $options ) {
-				if ( isset( $options['default'] ) ) {
-					$defaults[ $arg ] = $options['default'];
+			foreach ($schema['properties'] as $arg => $options) {
+				if (isset($options['default'])) {
+					$defaults[$arg] = $options['default'];
 				}
 			}
-			$_item->set_default_params( $defaults );
+			$_item->set_default_params($defaults);
 
 			// Set request parameters.
-			$_item->set_body_params( $item );
+			$_item->set_body_params($item);
 
 			// Set query (GET) parameters.
-			$_item->set_query_params( $query );
+			$_item->set_query_params($query);
 
 			// Create the item.
-			$_response = $this->create_item( $_item );
+			$_response = $this->create_item($_item);
 
 			// If an error occured...
-			if ( is_wp_error( $_response ) ) {
+			if (is_wp_error($_response)) {
 
-				$create[]   = array(
+				$create[] = array(
 					'id'    => 0,
 					'error' => array(
 						'code'    => $_response->get_error_code(),
@@ -192,7 +192,7 @@ 
 				continue;
 			}
 
-			$create[] = $wp_rest_server->response_to_data( /** @scrutinizer ignore-type */ $_response, false );
+			$create[] = $wp_rest_server->response_to_data(/** @scrutinizer ignore-type */ $_response, false);
 
 		}
 
@@ -208,27 +208,27 @@ 
 	 * @param WP_REST_Server $wp_rest_server
 	 * @return array()
 	 */
-	protected function batch_update_items( $items, $request, $wp_rest_server ) {
+	protected function batch_update_items($items, $request, $wp_rest_server) {
 
 		$query  = $request->get_query_params();
 		$update = array();
 
-		foreach ( $items as $item ) {
+		foreach ($items as $item) {
 
 			// Create a dummy request.
-			$_item = new WP_REST_Request( 'PUT' );
+			$_item = new WP_REST_Request('PUT');
 
 			// Add body params.
-			$_item->set_body_params( $item );
+			$_item->set_body_params($item);
 
 			// Set query (GET) parameters.
-			$_item->set_query_params( $query );
+			$_item->set_query_params($query);
 
 			// Update the item.
-			$_response = $this->update_item( $_item );
+			$_response = $this->update_item($_item);
 
 			// If an error occured...
-			if ( is_wp_error( $_response ) ) {
+			if (is_wp_error($_response)) {
 
 				$update[] = array(
 					'id'    => $item['id'],
@@ -243,7 +243,7 @@ 
 
 			}
 
-			$update[] = $wp_rest_server->response_to_data( /** @scrutinizer ignore-type */ $_response, false );
+			$update[] = $wp_rest_server->response_to_data(/** @scrutinizer ignore-type */ $_response, false);
 
 		}
 
@@ -258,14 +258,14 @@ 
 	 * @param WP_REST_Server $wp_rest_server
 	 * @return array()
 	 */
-	protected function batch_delete_items( $items, $wp_rest_server ) {
+	protected function batch_delete_items($items, $wp_rest_server) {
 
 		$delete = array();
 
-		foreach ( array_filter( $items ) as $id ) {
+		foreach (array_filter($items) as $id) {
 
 			// Prepare the request.
-			$_item = new WP_REST_Request( 'DELETE' );
+			$_item = new WP_REST_Request('DELETE');
 			$_item->set_query_params(
 				array(
 					'id'    => $id,
@@ -274,9 +274,9 @@ 
 			);
 
 			// Delete the item.
-			$_response = $this->delete_item( $_item );
+			$_response = $this->delete_item($_item);
 
-			if ( is_wp_error( $_response ) ) {
+			if (is_wp_error($_response)) {
 
 				$delete[] = array(
 					'id'    => $id,
@@ -290,7 +290,7 @@ 
 				continue;
 			}
 
-			$delete[] = $wp_rest_server->response_to_data( /** @scrutinizer ignore-type */ $_response, false );
+			$delete[] = $wp_rest_server->response_to_data(/** @scrutinizer ignore-type */ $_response, false);
 
 		}
 
@@ -304,23 +304,23 @@ 
 	 * @param WP_REST_Request $request Full details about the request.
 	 * @return WP_Error|array.
 	 */
-	public function batch_items( $request ) {
+	public function batch_items($request) {
 		global $wp_rest_server;
 
 		// Prepare the batch items.
-		$items = $this->fill_batch_keys( array_filter( $request->get_params() ) );
+		$items = $this->fill_batch_keys(array_filter($request->get_params()));
 
 		// Ensure that the batch has not exceeded the limit to prevent abuse.
-		$limit = $this->check_batch_limit( $items );
-		if ( is_wp_error( $limit ) ) {
+		$limit = $this->check_batch_limit($items);
+		if (is_wp_error($limit)) {
 			return $limit;
 		}
 
 		// Process the items.
 		return array(
-			'create' => $this->batch_create_items( $items['create'], $request, $wp_rest_server ),
-			'update' => $this->batch_update_items( $items['update'], $request, $wp_rest_server ),
-			'delete' => $this->batch_delete_items( $items['delete'], $wp_rest_server ),
+			'create' => $this->batch_create_items($items['create'], $request, $wp_rest_server),
+			'update' => $this->batch_update_items($items['update'], $request, $wp_rest_server),
+			'delete' => $this->batch_delete_items($items['delete'], $wp_rest_server),
 		);
 
 	}
@@ -333,8 +333,8 @@ 
 	 * @param array $meta_query Meta query.
 	 * @return array
 	 */
-	protected function add_meta_query( $args, $meta_query ) {
-		if ( empty( $args['meta_query'] ) ) {
+	protected function add_meta_query($args, $meta_query) {
+		if (empty($args['meta_query'])) {
 			$args['meta_query'] = array();
 		}
 
@@ -356,25 +356,25 @@ 
 			'type'       => 'object',
 			'properties' => array(
 				'create' => array(
-					'description' => __( 'List of created resources.', 'invoicing' ),
+					'description' => __('List of created resources.', 'invoicing'),
 					'type'        => 'array',
-					'context'     => array( 'view', 'edit' ),
+					'context'     => array('view', 'edit'),
 					'items'       => array(
 						'type' => 'object',
 					),
 				),
 				'update' => array(
-					'description' => __( 'List of updated resources.', 'invoicing' ),
+					'description' => __('List of updated resources.', 'invoicing'),
 					'type'        => 'array',
-					'context'     => array( 'view', 'edit' ),
+					'context'     => array('view', 'edit'),
 					'items'       => array(
 						'type' => 'object',
 					),
 				),
 				'delete' => array(
-					'description' => __( 'List of deleted resources.', 'invoicing' ),
+					'description' => __('List of deleted resources.', 'invoicing'),
 					'type'        => 'array',
-					'context'     => array( 'view', 'edit' ),
+					'context'     => array('view', 'edit'),
 					'items'       => array(
 						'type' => 'integer',
 					),
@@ -395,13 +395,13 @@ 
 	protected function get_schema_properties() {
 
 		$schema     = $this->get_item_schema();
-		$properties = isset( $schema['properties'] ) ? $schema['properties'] : array();
+		$properties = isset($schema['properties']) ? $schema['properties'] : array();
 
 		// For back-compat, include any field with an empty schema
 		// because it won't be present in $this->get_item_schema().
-		foreach ( $this->get_additional_fields() as $field_name => $field_options ) {
-			if ( is_null( $field_options['schema'] ) ) {
-				$properties[ $field_name ] = $field_options;
+		foreach ($this->get_additional_fields() as $field_name => $field_options) {
+			if (is_null($field_options['schema'])) {
+				$properties[$field_name] = $field_options;
 			}
 		}
 
@@ -416,15 +416,15 @@ 
 	 * @since 1.0.19
 	 * @return array
 	 */
-	protected function filter_response_fields_by_context( $fields, $context ) {
+	protected function filter_response_fields_by_context($fields, $context) {
 
-		if ( empty( $context ) ) {
+		if (empty($context)) {
 			return $fields;
 		}
 
-		foreach ( $fields as $name => $options ) {
-			if ( ! empty( $options['context'] ) && ! in_array( $context, $options['context'], true ) ) {
-				unset( $fields[ $name ] );
+		foreach ($fields as $name => $options) {
+			if (!empty($options['context']) && !in_array($context, $options['context'], true)) {
+				unset($fields[$name]);
 			}
 		}
 
@@ -440,35 +440,35 @@ 
 	 * @since 1.0.19
 	 * @return array
 	 */
-	protected function filter_response_fields_by_array( $fields, $requested ) {
+	protected function filter_response_fields_by_array($fields, $requested) {
 
 		// Trim off any whitespace from the list array.
-		$requested = array_map( 'trim', $requested );
+		$requested = array_map('trim', $requested);
 
 		// Always persist 'id', because it can be needed for add_additional_fields_to_object().
-		if ( in_array( 'id', $fields, true ) ) {
+		if (in_array('id', $fields, true)) {
 			$requested[] = 'id';
 		}
 
 		// Get rid of duplicate fields.
-		$requested = array_unique( $requested );
+		$requested = array_unique($requested);
 
 		// Return the list of all included fields which are available.
 		return array_reduce(
 			$requested,
-			function( $response_fields, $field ) use ( $fields ) {
+			function($response_fields, $field) use ($fields) {
 
-				if ( in_array( $field, $fields, true ) ) {
+				if (in_array($field, $fields, true)) {
 					$response_fields[] = $field;
 					return $response_fields;
 				}
 
 				// Check for nested fields if $field is not a direct match.
-				$nested_fields = explode( '.', $field );
+				$nested_fields = explode('.', $field);
 
 				// A nested field is included so long as its top-level property is
 				// present in the schema.
-				if ( in_array( $nested_fields[0], $fields, true ) ) {
+				if (in_array($nested_fields[0], $fields, true)) {
 					$response_fields[] = $field;
 				}
 
@@ -489,23 +489,23 @@ 
 	 * @param WP_REST_Request $request Full details about the request.
 	 * @return array Fields to be included in the response.
 	 */
-	public function get_fields_for_response( $request ) {
+	public function get_fields_for_response($request) {
 
 		// Retrieve fields in the schema.
 		$properties = $this->get_schema_properties();
 
 		// Exclude fields that specify a different context than the request context.
-		$properties = $this->filter_response_fields_by_context( $properties, $request['context'] );
+		$properties = $this->filter_response_fields_by_context($properties, $request['context']);
 
 		// We only need the field keys.
-		$fields = array_keys( $properties );
+		$fields = array_keys($properties);
 
 		// Is the user filtering the response fields??
-		if ( empty( $request['_fields'] ) ) {
+		if (empty($request['_fields'])) {
 			return $fields;
 		}
 
-		return $this->filter_response_fields_by_array( $fields, wpinv_parse_list( $request['_fields'] ) );
+		return $this->filter_response_fields_by_array($fields, wpinv_parse_list($request['_fields']));
 
 	}
 
@@ -519,32 +519,32 @@ 
 	 * @param array $fields Requested fields.
 	 * @return array Fields to be included in the response.
 	 */
-	public function limit_object_to_requested_fields( $data, $fields, $prefix = '' ) {
+	public function limit_object_to_requested_fields($data, $fields, $prefix = '') {
 
 		// Is the user filtering the response fields??
-		if ( empty( $fields ) ) {
+		if (empty($fields)) {
 			return $data;
 		}
 
-		foreach ( $data as $key => $value ) {
+		foreach ($data as $key => $value) {
 
 			// Numeric arrays.
-			if ( is_numeric( $key ) && is_array( $value ) ) {
-				$data[ $key ] = $this->limit_object_to_requested_fields( $value, $fields, $prefix );
+			if (is_numeric($key) && is_array($value)) {
+				$data[$key] = $this->limit_object_to_requested_fields($value, $fields, $prefix);
 				continue;
 			}
 
 			// Generate a new prefix.
-			$new_prefix = empty( $prefix ) ? $key : "$prefix.$key";
+			$new_prefix = empty($prefix) ? $key : "$prefix.$key";
 
 			// Check if it was requested.
-			if ( ! empty( $key ) && ! $this->is_field_included( $new_prefix, $fields ) ) {
-				unset( $data[ $key ] );
+			if (!empty($key) && !$this->is_field_included($new_prefix, $fields)) {
+				unset($data[$key]);
 				continue;
 			}
 
-			if ( $key != 'meta_data' && is_array( $value ) ) {
-				$data[ $key ] = $this->limit_object_to_requested_fields( $value, $fields, $new_prefix );
+			if ($key != 'meta_data' && is_array($value)) {
+				$data[$key] = $this->limit_object_to_requested_fields($value, $fields, $new_prefix);
 			}
 }
 
@@ -565,20 +565,20 @@ 
 	 * @return bool Whether to include the field or not.
 	 * @see rest_is_field_included()
 	 */
-	public function is_field_included( $field, $fields ) {
-		if ( in_array( $field, $fields, true ) ) {
+	public function is_field_included($field, $fields) {
+		if (in_array($field, $fields, true)) {
 			return true;
 		}
 
-		foreach ( $fields as $accepted_field ) {
+		foreach ($fields as $accepted_field) {
 			// Check to see if $field is the parent of any item in $fields.
 			// A field "parent" should be accepted if "parent.child" is accepted.
-			if ( strpos( $accepted_field, "$field." ) === 0 ) {
+			if (strpos($accepted_field, "$field.") === 0) {
 				return true;
 			}
 			// Conversely, if "parent" is accepted, all "parent.child" fields
 			// should also be accepted.
-			if ( strpos( $field, "$accepted_field." ) === 0 ) {
+			if (strpos($field, "$accepted_field.") === 0) {
 				return true;
 			}
 		}

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

Spacing +184 added lines, -184 removed lines

@@ -9,7 +9,7 @@ 
  * @since   2.0.0
  */
 
-defined( 'ABSPATH' ) || exit;
+defined('ABSPATH') || exit;
 
 /**
  * GetPaid REST Setting controller class.
@@ -39,7 +39,7 @@ 
 	 *
 	 * @see register_rest_route()
 	 */
-	public function register_namespace_routes( $namespace ) {
+	public function register_namespace_routes($namespace) {
 
 		// List all registered tabs.
 		register_rest_route(
@@ -48,8 +48,8 @@ 
 			array(
 				array(
 					'methods'             => WP_REST_Server::READABLE,
-					'callback'            => array( $this, 'get_tabs' ),
-					'permission_callback' => array( $this, 'get_items_permissions_check' ),
+					'callback'            => array($this, 'get_tabs'),
+					'permission_callback' => array($this, 'get_items_permissions_check'),
 				),
 				'schema' => '__return_empty_array',
 			)
@@ -62,23 +62,23 @@ 
 			array(
 				'args'   => array(
 					'id' => array(
-						'description' => __( 'Unique identifier for the setting.', 'invoicing' ),
+						'description' => __('Unique identifier for the setting.', 'invoicing'),
 						'type'        => 'string',
 						'required'    => true,
 					),
 				),
 				array(
 					'methods'             => WP_REST_Server::READABLE,
-					'callback'            => array( $this, 'get_item' ),
-					'permission_callback' => array( $this, 'get_items_permissions_check' ),
+					'callback'            => array($this, 'get_item'),
+					'permission_callback' => array($this, 'get_items_permissions_check'),
 				),
 				array(
 					'methods'             => WP_REST_Server::EDITABLE,
-					'callback'            => array( $this, 'update_item' ),
-					'permission_callback' => array( $this, 'update_items_permissions_check' ),
-					'args'                => $this->get_endpoint_args_for_item_schema( WP_REST_Server::EDITABLE ),
+					'callback'            => array($this, 'update_item'),
+					'permission_callback' => array($this, 'update_items_permissions_check'),
+					'args'                => $this->get_endpoint_args_for_item_schema(WP_REST_Server::EDITABLE),
 				),
-				'schema' => array( $this, 'get_public_item_schema' ),
+				'schema' => array($this, 'get_public_item_schema'),
 			)
 		);
 
@@ -89,16 +89,16 @@ 
 			array(
 				'args'   => array(
 					'tab' => array(
-						'description' => __( 'Unique identifier for the tab whose sections should be retrieved.', 'invoicing' ),
+						'description' => __('Unique identifier for the tab whose sections should be retrieved.', 'invoicing'),
 						'type'        => 'string',
 						'required'    => true,
-						'enum'        => array_keys( wpinv_get_settings_tabs() ),
+						'enum'        => array_keys(wpinv_get_settings_tabs()),
 					),
 				),
 				array(
 					'methods'             => WP_REST_Server::READABLE,
-					'callback'            => array( $this, 'get_sections' ),
-					'permission_callback' => array( $this, 'get_items_permissions_check' ),
+					'callback'            => array($this, 'get_sections'),
+					'permission_callback' => array($this, 'get_items_permissions_check'),
 				),
 				'schema' => '__return_empty_array',
 			)
@@ -111,23 +111,23 @@ 
 			array(
 				'args'   => array(
 					'tab'     => array(
-						'description' => __( 'Unique identifier for the tab whose settings should be retrieved.', 'invoicing' ),
+						'description' => __('Unique identifier for the tab whose settings should be retrieved.', 'invoicing'),
 						'type'        => 'string',
 						'required'    => true,
-						'enum'        => array_keys( wpinv_get_settings_tabs() ),
+						'enum'        => array_keys(wpinv_get_settings_tabs()),
 					),
 					'section' => array(
-						'description' => __( 'The section in the tab whose settings should be retrieved.', 'invoicing' ),
+						'description' => __('The section in the tab whose settings should be retrieved.', 'invoicing'),
 						'type'        => 'string',
 						'required'    => true,
 					),
 				),
 				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'),
 				),
-				'schema' => array( $this, 'get_public_item_schema' ),
+				'schema' => array($this, 'get_public_item_schema'),
 			)
 		);
 
@@ -137,17 +137,17 @@ 
 			array(
 				'args'   => array(
 					'id' => array(
-						'description' => __( 'Setting ID.', 'invoicing' ),
+						'description' => __('Setting ID.', 'invoicing'),
 						'type'        => 'string',
 					),
 				),
 				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'),
 			)
 		);
 
@@ -160,33 +160,33 @@ 
 	 * @param  WP_REST_Request $request Request data.
 	 * @return WP_Error|WP_REST_Response
 	 */
-	public function get_items( $request ) {
+	public function get_items($request) {
 
 		$settings = $this->get_settings();
 
-		if ( ! isset( $settings[ $request['tab'] ] ) ) {
-			return new WP_Error( 'rest_invalid_tab', __( 'Invalid tab.', 'invoicing' ), array( 'status' => 400 ) );
+		if (!isset($settings[$request['tab']])) {
+			return new WP_Error('rest_invalid_tab', __('Invalid tab.', 'invoicing'), array('status' => 400));
 		}
 
-		if ( ! isset( $settings[ $request['tab'] ][ $request['section'] ] ) ) {
-			return new WP_Error( 'rest_invalid_section', __( 'Invalid section.', 'invoicing' ), array( 'status' => 400 ) );
+		if (!isset($settings[$request['tab']][$request['section']])) {
+			return new WP_Error('rest_invalid_section', __('Invalid section.', 'invoicing'), array('status' => 400));
 		}
 
-		$settings = $settings[ $request['tab'] ][ $request['section'] ];
+		$settings = $settings[$request['tab']][$request['section']];
 		$prepared = array();
 
-		foreach ( $settings as $setting ) {
+		foreach ($settings as $setting) {
 
-			$setting      = $this->sanitize_setting( $setting );
-			$setting_data = $this->prepare_item_for_response( $setting, $request );
-			$setting_data = $this->prepare_response_for_collection( $setting_data );
+			$setting      = $this->sanitize_setting($setting);
+			$setting_data = $this->prepare_item_for_response($setting, $request);
+			$setting_data = $this->prepare_response_for_collection($setting_data);
 
-			if ( $this->is_setting_type_valid( $setting['type'] ) ) {
-				$prepared[]   = $setting_data;
+			if ($this->is_setting_type_valid($setting['type'])) {
+				$prepared[] = $setting_data;
 			}
 }
 
-		return rest_ensure_response( $prepared );
+		return rest_ensure_response($prepared);
 	}
 
 	/**
@@ -196,16 +196,16 @@ 
 	 * @param  WP_REST_Request $request Request data.
 	 * @return WP_Error|WP_REST_Response
 	 */
-	public function get_item( $request ) {
-		$setting  = $this->get_setting( $request['id'] );
+	public function get_item($request) {
+		$setting = $this->get_setting($request['id']);
 
-		if ( is_wp_error( $setting ) ) {
+		if (is_wp_error($setting)) {
 			return $setting;
 		}
 
-		$setting  = $this->sanitize_setting( $setting );
-		$response = $this->prepare_item_for_response( $setting, $request );
-		return rest_ensure_response( $response );
+		$setting  = $this->sanitize_setting($setting);
+		$response = $this->prepare_item_for_response($setting, $request);
+		return rest_ensure_response($response);
 	}
 
 	/**
@@ -215,29 +215,29 @@ 
 	 * @param  WP_REST_Request $request Request data.
 	 * @return WP_Error|WP_REST_Response
 	 */
-	public function update_item( $request ) {
-		$setting = $this->get_setting( $request['id'] );
+	public function update_item($request) {
+		$setting = $this->get_setting($request['id']);
 
-		if ( is_wp_error( $setting ) ) {
+		if (is_wp_error($setting)) {
 			return $setting;
 		}
 
-		if ( is_callable( array( $this, 'validate_setting_' . $setting['type'] . '_field' ) ) ) {
-			$value = $this->{'validate_setting_' . $setting['type'] . '_field'}( $request['value'], $setting );
+		if (is_callable(array($this, 'validate_setting_' . $setting['type'] . '_field'))) {
+			$value = $this->{'validate_setting_' . $setting['type'] . '_field'}($request['value'], $setting);
 		} else {
-			$value = $this->validate_setting_text_field( $request['value'], $setting );
+			$value = $this->validate_setting_text_field($request['value'], $setting);
 		}
 
-		if ( is_wp_error( $value ) ) {
+		if (is_wp_error($value)) {
 			return $value;
 		}
 
-		wpinv_update_option( $request['id'], $value );
+		wpinv_update_option($request['id'], $value);
 		$setting['value'] = $value;
-		$setting          = $this->sanitize_setting( $setting );
-		$response         = $this->prepare_item_for_response( $setting, $request );
+		$setting          = $this->sanitize_setting($setting);
+		$response         = $this->prepare_item_for_response($setting, $request);
 
-		return rest_ensure_response( $response );
+		return rest_ensure_response($response);
 	}
 
 	/**
@@ -247,9 +247,9 @@ 
 	 * @param WP_REST_Request $request Full data about the request.
 	 * @return WP_Error|boolean
 	 */
-	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() ) );
+	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()));
 		}
 
 		return true;
@@ -262,9 +262,9 @@ 
 	 * @param WP_REST_Request $request Full data about the request.
 	 * @return WP_Error|boolean
 	 */
-	public function update_items_permissions_check( $request ) {
-		if ( ! wpinv_current_user_can_manage_invoicing() ) {
-			return new WP_Error( 'rest_cannot_edit', __( 'Sorry, you cannot edit this resource.', 'invoicing' ), array( 'status' => rest_authorization_required_code() ) );
+	public function update_items_permissions_check($request) {
+		if (!wpinv_current_user_can_manage_invoicing()) {
+			return new WP_Error('rest_cannot_edit', __('Sorry, you cannot edit this resource.', 'invoicing'), array('status' => rest_authorization_required_code()));
 		}
 
 		return true;
@@ -277,8 +277,8 @@ 
 	 *
 	 * @return boolean|WP_Error
 	 */
-	public function batch_items_permissions_check( $request ) {
-		return wpinv_current_user_can_manage_invoicing() ? 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 wpinv_current_user_can_manage_invoicing() ? true : new WP_Error('rest_cannot_batch', __('Sorry, you are not allowed to batch manipulate this resource.', 'invoicing'), array('status' => rest_authorization_required_code()));
 	}
 
 	/**
@@ -287,14 +287,14 @@ 
 	 * @param string $setting_id Setting ID.
 	 * @return array Links for the given setting.
 	 */
-	protected function prepare_links( $setting_id ) {
+	protected function prepare_links($setting_id) {
 
 		$links = array(
 			'self'       => array(
-				'href' => rest_url( sprintf( '/%s/%s/setting/%s', $this->namespace, $this->rest_base, $setting_id ) ),
+				'href' => rest_url(sprintf('/%s/%s/setting/%s', $this->namespace, $this->rest_base, $setting_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)),
 			),
 		);
 
@@ -309,14 +309,14 @@ 
 	 * @param WP_REST_Request $request Request object.
 	 * @return WP_REST_Response $response Response data.
 	 */
-	public function prepare_item_for_response( $item, $request ) {
-		$context = empty( $request['context'] ) ? 'view' : $request['context'];
-		$data    = $this->add_additional_fields_to_object( $item, $request );
-		$data    = $this->filter_response_by_context( $data, $context );
+	public function prepare_item_for_response($item, $request) {
+		$context = empty($request['context']) ? 'view' : $request['context'];
+		$data    = $this->add_additional_fields_to_object($item, $request);
+		$data    = $this->filter_response_by_context($data, $context);
 
-		$response = rest_ensure_response( $data );
+		$response = rest_ensure_response($data);
 
-		$response->add_links( $this->prepare_links( $item['id'] ) );
+		$response->add_links($this->prepare_links($item['id']));
 
 		return $response;
 	}
@@ -329,10 +329,10 @@ 
 	 * @param  array $setting Setting.
 	 * @return array
 	 */
-	public function filter_setting( $setting ) {
+	public function filter_setting($setting) {
 		return array_intersect_key(
 			$setting,
-			array_flip( array_filter( array_keys( $setting ), array( $this, 'allowed_setting_keys' ) ) )
+			array_flip(array_filter(array_keys($setting), array($this, 'allowed_setting_keys')))
 		);
 	}
 
@@ -342,8 +342,8 @@ 
 	 * @param  string $key Key to check.
 	 * @return boolean
 	 */
-	public function allowed_setting_keys( $key ) {
-		return in_array( $key, array_keys( $this->setting_defaults() ), true );
+	public function allowed_setting_keys($key) {
+		return in_array($key, array_keys($this->setting_defaults()), true);
 	}
 
 	/**
@@ -375,11 +375,11 @@ 
 	 * @param  array $setting The setting to sanitize.
 	 * @return array
 	 */
-	public function sanitize_setting( $setting ) {
+	public function sanitize_setting($setting) {
 
-		$setting          = wp_parse_args( $setting, $this->setting_defaults() );
-		$setting['value'] = wpinv_get_option( $setting['id'], $setting['std'] );
-		return $this->filter_setting( $setting );
+		$setting          = wp_parse_args($setting, $this->setting_defaults());
+		$setting['value'] = wpinv_get_option($setting['id'], $setting['std']);
+		return $this->filter_setting($setting);
 
 	}
 
@@ -390,29 +390,29 @@ 
 	 * @param string $setting_id Setting ID.
 	 * @return array|WP_Error
 	 */
-	public function get_setting( $setting_id ) {
+	public function get_setting($setting_id) {
 
-		if ( empty( $setting_id ) ) {
-			return new WP_Error( 'rest_setting_setting_invalid', __( 'Invalid setting.', 'invoicing' ), array( 'status' => 404 ) );
+		if (empty($setting_id)) {
+			return new WP_Error('rest_setting_setting_invalid', __('Invalid setting.', 'invoicing'), array('status' => 404));
 		}
 
-		$settings  = $this->get_settings();
+		$settings = $this->get_settings();
 
-		foreach ( $settings as $tabs ) {
+		foreach ($settings as $tabs) {
 
-			foreach ( $tabs as $sections ) {
+			foreach ($tabs as $sections) {
 
-				if ( isset( $sections[ $setting_id ] ) ) {
-					if ( ! $this->is_setting_type_valid( $sections[ $setting_id ]['type'] ) ) {
-						return new WP_Error( 'rest_setting_setting_type_invalid', __( 'Invalid setting type.', 'invoicing' ), array( 'status' => 404 ) );
+				if (isset($sections[$setting_id])) {
+					if (!$this->is_setting_type_valid($sections[$setting_id]['type'])) {
+						return new WP_Error('rest_setting_setting_type_invalid', __('Invalid setting type.', 'invoicing'), array('status' => 404));
 					}
 
-					return $sections[ $setting_id ];
+					return $sections[$setting_id];
 				}
 }
 }
 
-		return new WP_Error( 'rest_setting_setting_invalid', __( 'Invalid setting.', 'invoicing' ), array( 'status' => 404 ) );
+		return new WP_Error('rest_setting_setting_invalid', __('Invalid setting.', 'invoicing'), array('status' => 404));
 	}
 
 	/**
@@ -421,41 +421,41 @@ 
 	 * @param  WP_REST_Request $request Request data.
 	 * @return array
 	 */
-	public function get_tabs( $request ) {
+	public function get_tabs($request) {
 		$tabs     = wpinv_get_settings_tabs();
 		$prepared = array();
 
-		foreach ( $tabs as $id => $tab ) {
+		foreach ($tabs as $id => $tab) {
 
 			$_request        = $request;
-			$_request['tab'] = sanitize_title( $id );
+			$_request['tab'] = sanitize_title($id);
 			$data            = array(
-				'id'       => sanitize_title( $id ),
-				'label'    => sanitize_text_field( $tab ),
-				'sections' => $this->get_sections( $_request ),
+				'id'       => sanitize_title($id),
+				'label'    => sanitize_text_field($tab),
+				'sections' => $this->get_sections($_request),
 			);
 
-			$data     = $this->add_additional_fields_to_object( $data, $request );
-			$response = rest_ensure_response( $data );
+			$data     = $this->add_additional_fields_to_object($data, $request);
+			$response = rest_ensure_response($data);
 
-			if ( ! is_wp_error( $response ) ) {
+			if (!is_wp_error($response)) {
 				$links = array(
 					'sections'   => array(
-						'href' => rest_url( sprintf( '/%s/%s/%s', $this->namespace, $this->rest_base, $id ) ),
+						'href' => rest_url(sprintf('/%s/%s/%s', $this->namespace, $this->rest_base, $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)),
 					),
 				);
-				$response->add_links( $links );
-				$response = $this->prepare_response_for_collection( $response );
+				$response->add_links($links);
+				$response = $this->prepare_response_for_collection($response);
 			}
 
 			$prepared[] = $response;
 
 		}
 
-		return rest_ensure_response( $prepared );
+		return rest_ensure_response($prepared);
 	}
 
 	/**
@@ -464,43 +464,43 @@ 
 	 * @param  WP_REST_Request $request Request data.
 	 * @return array
 	 */
-	public function get_sections( $request ) {
+	public function get_sections($request) {
 
-		$tab      = sanitize_title( $request['tab'] );
-		$sections = wpinv_get_settings_tab_sections( $tab );
+		$tab      = sanitize_title($request['tab']);
+		$sections = wpinv_get_settings_tab_sections($tab);
 		$prepared = array();
 
-		foreach ( $sections as $id => $section ) {
+		foreach ($sections as $id => $section) {
 
-			$data            = array(
-				'id'    => sanitize_title( $id ),
-				'label' => sanitize_text_field( $section ),
+			$data = array(
+				'id'    => sanitize_title($id),
+				'label' => sanitize_text_field($section),
 			);
 
-			$data     = $this->add_additional_fields_to_object( $data, $request );
-			$response = rest_ensure_response( $data );
+			$data     = $this->add_additional_fields_to_object($data, $request);
+			$response = rest_ensure_response($data);
 
-			if ( ! is_wp_error( $response ) ) {
+			if (!is_wp_error($response)) {
 				$links = array(
 					'settings'   => array(
-						'href' => rest_url( sprintf( '/%s/%s/%s/%s', $this->namespace, $this->rest_base, $tab, $id ) ),
+						'href' => rest_url(sprintf('/%s/%s/%s/%s', $this->namespace, $this->rest_base, $tab, $id)),
 					),
 					'collection' => array(
-						'href' => rest_url( sprintf( '/%s/%s/%s', $this->namespace, $this->rest_base, $tab ) ),
+						'href' => rest_url(sprintf('/%s/%s/%s', $this->namespace, $this->rest_base, $tab)),
 					),
 					'tabs'       => array(
-						'href' => rest_url( sprintf( '/%s/%s', $this->namespace, $this->rest_base ) ),
+						'href' => rest_url(sprintf('/%s/%s', $this->namespace, $this->rest_base)),
 					),
 				);
-				$response->add_links( $links );
-				$response = $this->prepare_response_for_collection( $response );
+				$response->add_links($links);
+				$response = $this->prepare_response_for_collection($response);
 			}
 
 			$prepared[] = $response;
 
 		}
 
-		return rest_ensure_response( $prepared );
+		return rest_ensure_response($prepared);
 	}
 
 	/**
@@ -510,7 +510,7 @@ 
 	 */
 	public function get_settings() {
 
-		if ( empty( $this->settings ) ) {
+		if (empty($this->settings)) {
 			$this->settings = wpinv_get_registered_settings();
 		}
 
@@ -525,22 +525,22 @@ 
 	 * @param  string $type Type.
 	 * @return bool
 	 */
-	public function is_setting_type_valid( $type ) {
+	public function is_setting_type_valid($type) {
 
 		return in_array(
 			$type,
             array(
-				'text',         // Validates with validate_setting_text_field.
-				'email',        // Validates with validate_setting_text_field.
-				'number',       // Validates with validate_setting_text_field.
-				'color',        // Validates with validate_setting_text_field.
-				'password',     // Validates with validate_setting_text_field.
-				'textarea',     // Validates with validate_setting_textarea_field.
-				'select',       // Validates with validate_setting_select_field.
-				'multiselect',  // Validates with validate_setting_multiselect_field.
-				'radio',        // Validates with validate_setting_radio_field (-> validate_setting_select_field).
-				'checkbox',     // Validates with validate_setting_checkbox_field.
-				'header',       // Validates with validate_setting_text_field.
+				'text', // Validates with validate_setting_text_field.
+				'email', // Validates with validate_setting_text_field.
+				'number', // Validates with validate_setting_text_field.
+				'color', // Validates with validate_setting_text_field.
+				'password', // Validates with validate_setting_text_field.
+				'textarea', // Validates with validate_setting_textarea_field.
+				'select', // Validates with validate_setting_select_field.
+				'multiselect', // Validates with validate_setting_multiselect_field.
+				'radio', // Validates with validate_setting_radio_field (-> validate_setting_select_field).
+				'checkbox', // Validates with validate_setting_checkbox_field.
+				'header', // Validates with validate_setting_text_field.
 			)
 		);
 
@@ -554,8 +554,8 @@ 
 	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);
 		}
 
 		$schema = array(
@@ -564,105 +564,105 @@ 
 			'type'       => 'object',
 			'properties' => array(
 				'id'          => array(
-					'description' => __( 'A unique identifier for the setting.', 'invoicing' ),
+					'description' => __('A unique identifier for the setting.', 'invoicing'),
 					'type'        => 'string',
 					'arg_options' => array(
 						'sanitize_callback' => 'sanitize_title',
 					),
-					'context'     => array( 'view', 'edit' ),
+					'context'     => array('view', 'edit'),
 					'readonly'    => true,
 				),
 				'tab'         => array(
-					'description' => __( 'An identifier for the tab this setting belongs to.', 'invoicing' ),
+					'description' => __('An identifier for the tab this setting belongs to.', 'invoicing'),
 					'type'        => 'string',
 					'arg_options' => array(
 						'sanitize_callback' => 'sanitize_title',
 					),
-					'context'     => array( 'view', 'edit' ),
+					'context'     => array('view', 'edit'),
 					'readonly'    => true,
 				),
 				'section'     => array(
-					'description' => __( 'An identifier for the section this setting belongs to.', 'invoicing' ),
+					'description' => __('An identifier for the section this setting belongs to.', 'invoicing'),
 					'type'        => 'string',
 					'arg_options' => array(
 						'sanitize_callback' => 'sanitize_title',
 					),
-					'context'     => array( 'view', 'edit' ),
+					'context'     => array('view', 'edit'),
 					'readonly'    => true,
 				),
 				'name'        => array(
-					'description' => __( 'A human readable label for the setting used in interfaces.', 'invoicing' ),
+					'description' => __('A human readable label for the setting used in interfaces.', 'invoicing'),
 					'type'        => 'string',
 					'arg_options' => array(
 						'sanitize_callback' => 'sanitize_text_field',
 					),
-					'context'     => array( 'view', 'edit' ),
+					'context'     => array('view', 'edit'),
 					'readonly'    => true,
 				),
 				'desc'        => array(
-					'description' => __( 'A human readable description for the setting used in interfaces.', 'invoicing' ),
+					'description' => __('A human readable description for the setting used in interfaces.', 'invoicing'),
 					'type'        => 'string',
-					'context'     => array( 'view', 'edit' ),
+					'context'     => array('view', 'edit'),
 					'readonly'    => true,
 				),
 				'value'       => array(
-					'description' => __( 'The current value of this setting.', 'invoicing' ),
+					'description' => __('The current value of this setting.', 'invoicing'),
 					'type'        => 'mixed',
-					'context'     => array( 'view', 'edit' ),
+					'context'     => array('view', 'edit'),
 				),
 				'default'     => array(
-					'description' => __( 'Default value for the setting.', 'invoicing' ),
+					'description' => __('Default value for the setting.', 'invoicing'),
 					'type'        => 'mixed',
-					'context'     => array( 'view', 'edit' ),
+					'context'     => array('view', 'edit'),
 					'readonly'    => true,
 				),
 				'placeholder' => array(
-					'description' => __( 'Placeholder text to be displayed in text inputs.', 'invoicing' ),
+					'description' => __('Placeholder text to be displayed in text inputs.', 'invoicing'),
 					'type'        => 'string',
 					'arg_options' => array(
 						'sanitize_callback' => 'sanitize_text_field',
 					),
-					'context'     => array( 'view', 'edit' ),
+					'context'     => array('view', 'edit'),
 					'readonly'    => true,
 				),
 				'type'        => array(
-					'description' => __( 'Type of setting.', 'invoicing' ),
+					'description' => __('Type of setting.', 'invoicing'),
 					'type'        => 'string',
 					'arg_options' => array(
 						'sanitize_callback' => 'sanitize_text_field',
 					),
-					'context'     => array( 'view', 'edit' ),
-					'enum'        => array( 'text', 'email', 'number', 'color', 'password', 'textarea', 'select', 'multiselect', 'radio', 'image_width', 'checkbox', 'raw_html' ),
+					'context'     => array('view', 'edit'),
+					'enum'        => array('text', 'email', 'number', 'color', 'password', 'textarea', 'select', 'multiselect', 'radio', 'image_width', 'checkbox', 'raw_html'),
 					'readonly'    => true,
 				),
 				'options'     => array(
-					'description' => __( 'Array of options (key value pairs) for inputs such as select, multiselect, and radio buttons.', 'invoicing' ),
+					'description' => __('Array of options (key value pairs) for inputs such as select, multiselect, and radio buttons.', 'invoicing'),
 					'type'        => 'object',
-					'context'     => array( 'view', 'edit' ),
+					'context'     => array('view', 'edit'),
 					'readonly'    => true,
 				),
 				'readonly'    => array(
-					'description' => __( 'Whether or not this setting is readonly', 'invoicing' ),
+					'description' => __('Whether or not this setting is readonly', 'invoicing'),
 					'type'        => 'string',
-					'context'     => array( 'view' ),
+					'context'     => array('view'),
 					'readonly'    => true,
 				),
 				'faux'        => array(
-					'description' => __( 'Whether or not this setting is readonly/faux', 'invoicing' ),
+					'description' => __('Whether or not this setting is readonly/faux', 'invoicing'),
 					'type'        => 'string',
-					'context'     => array( 'view' ),
+					'context'     => array('view'),
 					'readonly'    => true,
 				),
 			),
 		);
 
 		// Filters the settings schema for the REST API.
-        $schema = apply_filters( 'getpaid_rest_settings_schema', $schema );
+        $schema = apply_filters('getpaid_rest_settings_schema', $schema);
 
 		// Cache the settings schema.
 		$this->schema = $schema;
 
-		return $this->add_additional_fields_schema( $this->schema );
+		return $this->add_additional_fields_schema($this->schema);
 
 	}
 
@@ -674,9 +674,9 @@ 
 	 * @param array  $setting Setting.
 	 * @return string
 	 */
-	public function validate_setting_text_field( $value ) {
-		$value = is_null( $value ) ? '' : $value;
-		return wp_kses_post( trim( stripslashes( $value ) ) );
+	public function validate_setting_text_field($value) {
+		$value = is_null($value) ? '' : $value;
+		return wp_kses_post(trim(stripslashes($value)));
 	}
 
 	/**
@@ -687,11 +687,11 @@ 
 	 * @param array  $setting Setting.
 	 * @return string|WP_Error
 	 */
-	public function validate_setting_select_field( $value, $setting ) {
-		if ( array_key_exists( $value, $setting['options'] ) ) {
+	public function validate_setting_select_field($value, $setting) {
+		if (array_key_exists($value, $setting['options'])) {
 			return $value;
 		} else {
-			return new WP_Error( 'rest_setting_value_invalid', __( 'An invalid setting value was passed.', 'invoicing' ), array( 'status' => 400 ) );
+			return new WP_Error('rest_setting_value_invalid', __('An invalid setting value was passed.', 'invoicing'), array('status' => 400));
 		}
 	}
 
@@ -703,18 +703,18 @@ 
 	 * @param array $setting Setting.
 	 * @return array|WP_Error
 	 */
-	public function validate_setting_multiselect_field( $values, $setting ) {
-		if ( empty( $values ) ) {
+	public function validate_setting_multiselect_field($values, $setting) {
+		if (empty($values)) {
 			return array();
 		}
 
-		if ( ! is_array( $values ) ) {
-			return new WP_Error( 'rest_setting_value_invalid', __( 'An invalid setting value was passed.', 'invoicing' ), array( 'status' => 400 ) );
+		if (!is_array($values)) {
+			return new WP_Error('rest_setting_value_invalid', __('An invalid setting value was passed.', 'invoicing'), array('status' => 400));
 		}
 
 		$final_values = array();
-		foreach ( $values as $value ) {
-			if ( array_key_exists( $value, $setting['options'] ) ) {
+		foreach ($values as $value) {
+			if (array_key_exists($value, $setting['options'])) {
 				$final_values[] = $value;
 			}
 		}
@@ -730,8 +730,8 @@ 
 	 * @param array  $setting Setting.
 	 * @return string|WP_Error
 	 */
-	public function validate_setting_radio_field( $value, $setting ) {
-		return $this->validate_setting_select_field( $value, $setting );
+	public function validate_setting_radio_field($value, $setting) {
+		return $this->validate_setting_select_field($value, $setting);
 	}
 
 	/**
@@ -741,8 +741,8 @@ 
 	 * @param string $value Value.
 	 * @return int
 	 */
-	public function validate_setting_checkbox_field( $value ) {
-		return (int) ! empty( $value );
+	public function validate_setting_checkbox_field($value) {
+		return (int) !empty($value);
 	}
 
 	/**
@@ -752,10 +752,10 @@ 
 	 * @param string $value Value.
 	 * @return string
 	 */
-	public function validate_setting_textarea_field( $value ) {
-		$value = is_null( $value ) ? '' : $value;
+	public function validate_setting_textarea_field($value) {
+		$value = is_null($value) ? '' : $value;
 		return wp_kses(
-			trim( stripslashes( $value ) ),
+			trim(stripslashes($value)),
 			array_merge(
 				array(
 					'iframe' => array(
@@ -765,7 +765,7 @@ 
 						'class' => true,
 					),
 				),
-				wp_kses_allowed_html( 'post' )
+				wp_kses_allowed_html('post')
 			)
 		);
 	}

includes/api/class-getpaid-rest-report-top-sellers-controller.php

Spacing +39 added lines, -39 removed lines

@@ -9,7 +9,7 @@ 
  * @since   2.0.0
  */
 
-defined( 'ABSPATH' ) || exit;
+defined('ABSPATH') || exit;
 
 /**
  * GetPaid REST top sellers controller class.
@@ -31,47 +31,47 @@ 
 	 * @param WP_REST_Request $request
 	 * @return array|WP_Error
 	 */
-	public function get_items( $request ) {
+	public function get_items($request) {
 
 		// Prepare items.
-		$this->report_range = $this->get_date_range( $request );
+		$this->report_range = $this->get_date_range($request);
 		$report_data        = $this->get_report_data();
 
 		$top_sellers = array();
 
-		foreach ( $report_data as $item ) {
+		foreach ($report_data as $item) {
 
-			$item_obj  = new WPInv_Item( $item );
+			$item_obj  = new WPInv_Item($item);
 			$item_name = $item->invoice_item_name;
-			$item_qty  = floatval( $item->invoice_item_qty );
-			$item_id   = absint( $item->invoice_item_id );
-			$price     = sanitize_text_field( wpinv_price( $item->invoice_item_price ) );
+			$item_qty  = floatval($item->invoice_item_qty);
+			$item_id   = absint($item->invoice_item_id);
+			$price     = sanitize_text_field(wpinv_price($item->invoice_item_price));
 
-			$item_obj  = new WPInv_Item( $item_id );
+			$item_obj  = new WPInv_Item($item_id);
 
-			if ( $item_obj->exists() ) {
+			if ($item_obj->exists()) {
 				$item_name = $item_obj->get_name();
 			} else {
 				$item_id   = 0;
 			}
 
 			$top_sellers[] = array(
-				'name'               => sanitize_text_field( $item_name ),
+				'name'               => sanitize_text_field($item_name),
 				'item_id'            => $item_id,
 				'quantity'           => $item_qty,
-				'earnings'           => wpinv_round_amount( $item->invoice_item_price ),
-				'earnings_formatted' => sanitize_text_field( wpinv_price( $price ) ),
+				'earnings'           => wpinv_round_amount($item->invoice_item_price),
+				'earnings_formatted' => sanitize_text_field(wpinv_price($price)),
 			);
 
 		}
 
 		$data = array();
-		foreach ( $top_sellers as $top_seller ) {
-			$item   = $this->prepare_item_for_response( (object) $top_seller, $request );
-			$data[] = $this->prepare_response_for_collection( $item );
+		foreach ($top_sellers as $top_seller) {
+			$item   = $this->prepare_item_for_response((object) $top_seller, $request);
+			$data[] = $this->prepare_response_for_collection($item);
 		}
 
-		return rest_ensure_response( $data );
+		return rest_ensure_response($data);
 
 	}
 
@@ -82,30 +82,30 @@ 
 	 * @param WP_REST_Request $request Request object.
 	 * @return WP_REST_Response $response Response data.
 	 */
-	public function prepare_item_for_response( $top_seller, $request ) {
+	public function prepare_item_for_response($top_seller, $request) {
 		$data    = (array) $top_seller;
 
-		$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);
 		$links = array(
 			'about' => array(
-				'href' => rest_url( sprintf( '%s/reports', $this->namespace ) ),
+				'href' => rest_url(sprintf('%s/reports', $this->namespace)),
 			),
 		);
 
-		if ( ! empty( $top_seller->item_id ) ) {
-			$links['item']   = array(
-				'href'       => rest_url( sprintf( '/%s/items/%s', $this->namespace, $top_seller->item_id ) ),
+		if (!empty($top_seller->item_id)) {
+			$links['item'] = array(
+				'href'       => rest_url(sprintf('/%s/items/%s', $this->namespace, $top_seller->item_id)),
 				'embeddable' => true,
 			);
 		}
 
-		$response->add_links( $links );
-		return apply_filters( 'getpaid_rest_prepare_report_' . $this->rest_base, $response, $top_seller, $request );
+		$response->add_links($links);
+		return apply_filters('getpaid_rest_prepare_report_' . $this->rest_base, $response, $top_seller, $request);
 	}
 
 	/**
@@ -159,38 +159,38 @@ 
 			'type'       => 'object',
 			'properties' => array(
 				'name'                => array(
-					'description' => __( 'Item name.', 'invoicing' ),
+					'description' => __('Item name.', 'invoicing'),
 					'type'        => 'string',
-					'context'     => array( 'view' ),
+					'context'     => array('view'),
 					'readonly'    => true,
 				),
 				'item_id'             => array(
-					'description' => __( 'Item ID.', 'invoicing' ),
+					'description' => __('Item ID.', 'invoicing'),
 					'type'        => 'integer',
-					'context'     => array( 'view' ),
+					'context'     => array('view'),
 					'readonly'    => true,
 				),
 				'quantity'            => array(
-					'description' => __( 'Total number of purchases.', 'invoicing' ),
+					'description' => __('Total number of purchases.', 'invoicing'),
 					'type'        => 'number',
-					'context'     => array( 'view' ),
+					'context'     => array('view'),
 					'readonly'    => true,
 				),
 				'earnings'            => array(
-					'description' => __( 'Total earnings for the item.', 'invoicing' ),
+					'description' => __('Total earnings for the item.', 'invoicing'),
 					'type'        => 'number',
-					'context'     => array( 'view' ),
+					'context'     => array('view'),
 					'readonly'    => true,
 				),
 				'earnings_formatted"' => array(
-					'description' => __( 'Total earnings (formatted) for the item.', 'invoicing' ),
+					'description' => __('Total earnings (formatted) for the item.', '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-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');
 	}
 
 }