AyeCode / invoicing

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

Spacing +14 added lines, -14 removed lines

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

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

Spacing +32 added lines, -32 removed lines

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

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

Spacing +127 added lines, -127 removed lines

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

includes/class-wpinv-cli.php

Spacing +16 added lines, -16 removed lines

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

includes/class-getpaid-template.php

Spacing +45 added lines, -45 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if ( ! defined( 'ABSPATH' ) ) {
+if (!defined('ABSPATH')) {
 	exit;
 }
 
@@ -26,12 +26,12 @@ 
 	 */
 	public function __construct() {
 
-        $this->templates_dir = apply_filters( 'getpaid_default_templates_dir', WPINV_PLUGIN_DIR . 'templates' );
-        $this->templates_url = apply_filters( 'getpaid_default_templates_url', WPINV_PLUGIN_URL . 'templates' );
+        $this->templates_dir = apply_filters('getpaid_default_templates_dir', WPINV_PLUGIN_DIR . 'templates');
+        $this->templates_url = apply_filters('getpaid_default_templates_url', WPINV_PLUGIN_URL . 'templates');
 
         // Oxygen plugin
-		if ( defined( 'CT_VERSION' ) ) {
-			add_filter( 'wpinv_locate_template', array( $this, 'oxygen_override_template' ), 11, 4 );
+		if (defined('CT_VERSION')) {
+			add_filter('wpinv_locate_template', array($this, 'oxygen_override_template'), 11, 4);
 		}
 
     }
@@ -59,7 +59,7 @@ 
 	 * @return bool
 	 */
 	public function is_elementor_preview() {
-		return isset( $_REQUEST['elementor-preview'] ) || ( is_admin() && isset( $_REQUEST['action'] ) && $_REQUEST['action'] == 'elementor' ) || ( isset( $_REQUEST['action'] ) && $_REQUEST['action'] == 'elementor_ajax' );
+		return isset($_REQUEST['elementor-preview']) || (is_admin() && isset($_REQUEST['action']) && $_REQUEST['action'] == 'elementor') || (isset($_REQUEST['action']) && $_REQUEST['action'] == 'elementor_ajax');
 	}
 
 	/**
@@ -69,7 +69,7 @@ 
 	 * @return bool
 	 */
 	public function is_divi_preview() {
-		return isset( $_REQUEST['et_fb'] ) || isset( $_REQUEST['et_pb_preview'] ) || ( is_admin() && isset( $_REQUEST['action'] ) && $_REQUEST['action'] == 'et_pb' );
+		return isset($_REQUEST['et_fb']) || isset($_REQUEST['et_pb_preview']) || (is_admin() && isset($_REQUEST['action']) && $_REQUEST['action'] == 'et_pb');
 	}
 
 	/**
@@ -79,7 +79,7 @@ 
 	 * @return bool
 	 */
 	public function is_beaver_preview() {
-		return isset( $_REQUEST['fl_builder'] );
+		return isset($_REQUEST['fl_builder']);
 	}
 
 	/**
@@ -89,7 +89,7 @@ 
 	 * @return bool
 	 */
 	public function is_siteorigin_preview() {
-		return ! empty( $_REQUEST['siteorigin_panels_live_editor'] );
+		return !empty($_REQUEST['siteorigin_panels_live_editor']);
 	}
 
 	/**
@@ -99,7 +99,7 @@ 
 	 * @return bool
 	 */
 	public function is_cornerstone_preview() {
-		return ! empty( $_REQUEST['cornerstone_preview'] ) || basename( $_SERVER['REQUEST_URI'] ) == 'cornerstone-endpoint';
+		return !empty($_REQUEST['cornerstone_preview']) || basename($_SERVER['REQUEST_URI']) == 'cornerstone-endpoint';
 	}
 
 	/**
@@ -109,7 +109,7 @@ 
 	 * @return bool
 	 */
 	public function is_fusion_preview() {
-		return ! empty( $_REQUEST['fb-edit'] ) || ! empty( $_REQUEST['fusion_load_nonce'] );
+		return !empty($_REQUEST['fb-edit']) || !empty($_REQUEST['fusion_load_nonce']);
 	}
 
 	/**
@@ -119,7 +119,7 @@ 
 	 * @return bool
 	 */
 	public function is_oxygen_preview() {
-		return ! empty( $_REQUEST['ct_builder'] ) || ( ! empty( $_REQUEST['action'] ) && ( substr( $_REQUEST['action'], 0, 11 ) === 'oxy_render_' || substr( $_REQUEST['action'], 0, 10 ) === 'ct_render_' ) );
+		return !empty($_REQUEST['ct_builder']) || (!empty($_REQUEST['action']) && (substr($_REQUEST['action'], 0, 11) === 'oxy_render_' || substr($_REQUEST['action'], 0, 10) === 'ct_render_'));
     }
 
     /**
@@ -129,24 +129,24 @@ 
      * @param string $template_path The template path relative to the theme's root dir. Defaults to 'invoicing'.
      * @param string $default_path The root path to the default template. Defaults to invoicing/templates
      */
-	public function locate_template( $template_name, $template_path = '', $default_path = '' ) {
+	public function locate_template($template_name, $template_path = '', $default_path = '') {
 
         // Load the defaults for the template path and default path.
-        $template_path = empty( $template_path ) ? 'invoicing' : $template_path;
-        $default_path  = empty( $default_path ) ? $this->templates_dir : $default_path;
-        $default_path  = apply_filters( 'getpaid_template_default_template_path', $default_path, $template_name );
+        $template_path = empty($template_path) ? 'invoicing' : $template_path;
+        $default_path  = empty($default_path) ? $this->templates_dir : $default_path;
+        $default_path  = apply_filters('getpaid_template_default_template_path', $default_path, $template_name);
 
         // Is it overidden?
         $template = locate_template(
-            array( trailingslashit( $template_path ) . $template_name, 'wpinv-' . $template_name )
+            array(trailingslashit($template_path) . $template_name, 'wpinv-' . $template_name)
         );
 
         // If not, load the default template.
-        if ( empty( $template ) ) {
-            $template = trailingslashit( $default_path ) . $template_name;
+        if (empty($template)) {
+            $template = trailingslashit($default_path) . $template_name;
         }
 
-        return apply_filters( 'wpinv_locate_template', $template, $template_name, $template_path, $default_path );
+        return apply_filters('wpinv_locate_template', $template, $template_name, $template_path, $default_path);
     }
 
     /**
@@ -155,20 +155,20 @@ 
 	 * @since 1.0.19
 	 * @return bool
 	 */
-	protected function load_template( $template_name, $template_path, $args ) {
+	protected function load_template($template_name, $template_path, $args) {
 
-        if ( is_array( $args ) ) {
-            extract( $args );
+        if (is_array($args)) {
+            extract($args);
         }
 
         // Fires before loading a template.
-	    do_action( 'wpinv_before_template_part', $template_name, $template_path, $args );
+	    do_action('wpinv_before_template_part', $template_name, $template_path, $args);
 
         // Load the template.
 	    include $template_path;
 
         // Fires after loading a template.
-        do_action( 'wpinv_after_template_part', $template_name, $template_path, $args );
+        do_action('wpinv_after_template_part', $template_name, $template_path, $args);
 
     }
 
@@ -182,18 +182,18 @@ 
      * @param string $template_path The templates directory relative to the theme's root dir. Defaults to 'invoicing'.
      * @param string $default_path The root path to the default template. Defaults to invoicing/templates
      */
-	public function display_template( $template_name, $args = array(), $template_path = '', $default_path = '' ) {
+	public function display_template($template_name, $args = array(), $template_path = '', $default_path = '') {
 
         // Locate the template.
-        $located = $this->locate_template( $template_name, $template_path, $default_path );
+        $located = $this->locate_template($template_name, $template_path, $default_path);
 
         // Abort if the file does not exist.
-        if ( ! file_exists( $located ) ) {
-            getpaid_doing_it_wrong( __METHOD__, sprintf( '<code>%s</code> does not exist.', $located ), '2.0.0' );
+        if (!file_exists($located)) {
+            getpaid_doing_it_wrong(__METHOD__, sprintf('<code>%s</code> does not exist.', $located), '2.0.0');
             return;
         }
 
-        $this->load_template( $template_name, $located, $args );
+        $this->load_template($template_name, $located, $args);
 
     }
 
@@ -207,9 +207,9 @@ 
      * @param string $template_path The templates directory relative to the theme's root dir. Defaults to 'invoicing'.
      * @param string $default_path The root path to the default template. Defaults to invoicing/templates
      */
-	public function get_template( $template_name, $args = array(), $template_path = '', $default_path = '' ) {
+	public function get_template($template_name, $args = array(), $template_path = '', $default_path = '') {
         ob_start();
-        $this->display_template( $template_name, $args, $template_path, $default_path );
+        $this->display_template($template_name, $args, $template_path, $default_path);
         return ob_get_clean();
     }
 
@@ -221,9 +221,9 @@ 
 	 */
 	public static function get_theme_template_path() {
 		$template   = get_template();
-		$theme_root = get_theme_root( $template );
+		$theme_root = get_theme_root($template);
 
-		return $theme_root . '/' . $template . '/' . untrailingslashit( wpinv_get_theme_template_dir_name() );
+		return $theme_root . '/' . $template . '/' . untrailingslashit(wpinv_get_theme_template_dir_name());
 
 	}
 
@@ -233,28 +233,28 @@ 
 	 * @param string $template The template.
 	 * @return string The theme template.
 	 */
-	public static function oxygen_locate_template( $template ) {
+	public static function oxygen_locate_template($template) {
 
-		if ( empty( $template ) ) {
+		if (empty($template)) {
 			return '';
 		}
 
-		$has_filter = has_filter( 'template', 'ct_oxygen_template_name' );
+		$has_filter = has_filter('template', 'ct_oxygen_template_name');
 
 		// Remove template filter
-		if ( $has_filter ) {
-			remove_filter( 'template', 'ct_oxygen_template_name' );
+		if ($has_filter) {
+			remove_filter('template', 'ct_oxygen_template_name');
 		}
 
 		$template = self::get_theme_template_path() . '/' . $template;
 
-		if ( ! file_exists( $template ) ) {
+		if (!file_exists($template)) {
 			$template = '';
 		}
 
 		// Add template filter
-		if ( $has_filter ) {
-			add_filter( 'template', 'ct_oxygen_template_name' );
+		if ($has_filter) {
+			add_filter('template', 'ct_oxygen_template_name');
 		}
 
 		return $template;
@@ -267,10 +267,10 @@ 
 	 * @param string $template_name Template name.
 	 * @return string Located template.
 	 */
-	public function oxygen_override_template( $located, $template_name ) {
+	public function oxygen_override_template($located, $template_name) {
 
-        $oxygen_overide = self::oxygen_locate_template( $template_name );
-		if ( ! empty( $oxygen_overide ) ) {
+        $oxygen_overide = self::oxygen_locate_template($template_name);
+		if (!empty($oxygen_overide)) {
 			return $oxygen_overide;
 		}
 

includes/class-wpinv-db.php

Spacing +39 added lines, -39 removed lines

@@ -1,7 +1,7 @@ 
 <?php
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if (!defined('ABSPATH')) {
 exit;
 }
 
@@ -68,9 +68,9 @@ 
 	 * @since   1.0.0
 	 * @return  object
 	 */
-	public function get( $row_id ) {
+	public function get($row_id) {
 		global $wpdb;
-		return $wpdb->get_row( $wpdb->prepare( "SELECT * FROM $this->table_name WHERE $this->primary_key = %s LIMIT 1;", $row_id ) );
+		return $wpdb->get_row($wpdb->prepare("SELECT * FROM $this->table_name WHERE $this->primary_key = %s LIMIT 1;", $row_id));
 	}
 
 	/**
@@ -80,10 +80,10 @@ 
 	 * @since   1.0.0
 	 * @return  object
 	 */
-	public function get_by( $column, $row_id ) {
+	public function get_by($column, $row_id) {
 		global $wpdb;
-		$column = esc_sql( $column );
-		return $wpdb->get_row( $wpdb->prepare( "SELECT * FROM $this->table_name WHERE $column = %s LIMIT 1;", $row_id ) );
+		$column = esc_sql($column);
+		return $wpdb->get_row($wpdb->prepare("SELECT * FROM $this->table_name WHERE $column = %s LIMIT 1;", $row_id));
 	}
 
 	/**
@@ -93,10 +93,10 @@ 
 	 * @since   1.0.0
 	 * @return  string
 	 */
-	public function get_column( $column, $row_id ) {
+	public function get_column($column, $row_id) {
 		global $wpdb;
-		$column = esc_sql( $column );
-		return $wpdb->get_var( $wpdb->prepare( "SELECT $column FROM $this->table_name WHERE $this->primary_key = %s LIMIT 1;", $row_id ) );
+		$column = esc_sql($column);
+		return $wpdb->get_var($wpdb->prepare("SELECT $column FROM $this->table_name WHERE $this->primary_key = %s LIMIT 1;", $row_id));
 	}
 
 	/**
@@ -106,11 +106,11 @@ 
 	 * @since   1.0.0
 	 * @return  string
 	 */
-	public function get_column_by( $column, $column_where, $column_value ) {
+	public function get_column_by($column, $column_where, $column_value) {
 		global $wpdb;
-		$column_where = esc_sql( $column_where );
-		$column       = esc_sql( $column );
-		return $wpdb->get_var( $wpdb->prepare( "SELECT $column FROM $this->table_name WHERE $column_where = %s LIMIT 1;", $column_value ) );
+		$column_where = esc_sql($column_where);
+		$column       = esc_sql($column);
+		return $wpdb->get_var($wpdb->prepare("SELECT $column FROM $this->table_name WHERE $column_where = %s LIMIT 1;", $column_value));
 	}
 
 	/**
@@ -120,31 +120,31 @@ 
 	 * @since   1.0.0
 	 * @return  int
 	 */
-	public function insert( $data, $type = '' ) {
+	public function insert($data, $type = '') {
 		global $wpdb;
 
 		// Set default values
-		$data = wp_parse_args( $data, $this->get_column_defaults() );
+		$data = wp_parse_args($data, $this->get_column_defaults());
 
-		do_action( 'wpinv_pre_insert_' . $type, $data );
+		do_action('wpinv_pre_insert_' . $type, $data);
 
 		// Initialise column format array
 		$column_formats = $this->get_columns();
 
 		// Force fields to lower case
-		$data = array_change_key_case( $data );
+		$data = array_change_key_case($data);
 
 		// White list columns
-		$data = array_intersect_key( $data, $column_formats );
+		$data = array_intersect_key($data, $column_formats);
 
 		// Reorder $column_formats to match the order of columns given in $data
-		$data_keys = array_keys( $data );
-		$column_formats = array_merge( array_flip( $data_keys ), $column_formats );
+		$data_keys = array_keys($data);
+		$column_formats = array_merge(array_flip($data_keys), $column_formats);
 
-		$wpdb->insert( $this->table_name, $data, $column_formats );
+		$wpdb->insert($this->table_name, $data, $column_formats);
 		$wpdb_insert_id = $wpdb->insert_id;
 
-		do_action( 'wpinv_post_insert_' . $type, $wpdb_insert_id, $data );
+		do_action('wpinv_post_insert_' . $type, $wpdb_insert_id, $data);
 
 		return $wpdb_insert_id;
 	}
@@ -156,18 +156,18 @@ 
 	 * @since   1.0.0
 	 * @return  bool
 	 */
-	public function update( $row_id, $data = array(), $where = '' ) {
+	public function update($row_id, $data = array(), $where = '') {
 
 		global $wpdb;
 
 		// Row ID must be positive integer
-		$row_id = absint( $row_id );
+		$row_id = absint($row_id);
 
-		if ( empty( $row_id ) ) {
+		if (empty($row_id)) {
 			return false;
 		}
 
-		if ( empty( $where ) ) {
+		if (empty($where)) {
 			$where = $this->primary_key;
 		}
 
@@ -175,16 +175,16 @@ 
 		$column_formats = $this->get_columns();
 
 		// Force fields to lower case
-		$data = array_change_key_case( $data );
+		$data = array_change_key_case($data);
 
 		// White list columns
-		$data = array_intersect_key( $data, $column_formats );
+		$data = array_intersect_key($data, $column_formats);
 
 		// Reorder $column_formats to match the order of columns given in $data
-		$data_keys = array_keys( $data );
-		$column_formats = array_merge( array_flip( $data_keys ), $column_formats );
+		$data_keys = array_keys($data);
+		$column_formats = array_merge(array_flip($data_keys), $column_formats);
 
-		if ( false === $wpdb->update( $this->table_name, $data, array( $where => $row_id ), $column_formats ) ) {
+		if (false === $wpdb->update($this->table_name, $data, array($where => $row_id), $column_formats)) {
 			return false;
 		}
 
@@ -198,18 +198,18 @@ 
 	 * @since   1.0.0
 	 * @return  bool
 	 */
-	public function delete( $row_id = 0 ) {
+	public function delete($row_id = 0) {
 
 		global $wpdb;
 
 		// Row ID must be positive integer
-		$row_id = absint( $row_id );
+		$row_id = absint($row_id);
 
-		if ( empty( $row_id ) ) {
+		if (empty($row_id)) {
 			return false;
 		}
 
-		if ( false === $wpdb->query( $wpdb->prepare( "DELETE FROM $this->table_name WHERE $this->primary_key = %d", $row_id ) ) ) {
+		if (false === $wpdb->query($wpdb->prepare("DELETE FROM $this->table_name WHERE $this->primary_key = %d", $row_id))) {
 			return false;
 		}
 
@@ -223,11 +223,11 @@ 
 	 * @param  string $table The table name
 	 * @return bool          If the table name exists
 	 */
-	public function table_exists( $table ) {
+	public function table_exists($table) {
 		global $wpdb;
-		$table = sanitize_text_field( $table );
+		$table = sanitize_text_field($table);
 
-		return $wpdb->get_var( $wpdb->prepare( "SHOW TABLES LIKE '%s'", $table ) ) === $table;
+		return $wpdb->get_var($wpdb->prepare("SHOW TABLES LIKE '%s'", $table)) === $table;
 	}
 
 	/**
@@ -237,7 +237,7 @@ 
 	 * @return bool Returns if the customers table was installed and upgrade routine run
 	 */
 	public function installed() {
-		return $this->table_exists( $this->table_name );
+		return $this->table_exists($this->table_name);
 	}
 
 }

includes/class-getpaid-daily-maintenance.php

Spacing +34 added lines, -34 removed lines

@@ -4,7 +4,7 @@ 
  *
  */
 
-defined( 'ABSPATH' ) || exit;
+defined('ABSPATH') || exit;
 
 /**
  * Daily maintenance class.
@@ -18,17 +18,17 @@ 
 	public function __construct() {
 
 		// Clear deprecated events.
-		add_action( 'wp', array( $this, 'maybe_clear_deprecated_events' ) );
+		add_action('wp', array($this, 'maybe_clear_deprecated_events'));
 
 		// (Maybe) schedule a cron that runs daily.
-		add_action( 'wp', array( $this, 'maybe_create_scheduled_event' ) );
+		add_action('wp', array($this, 'maybe_create_scheduled_event'));
 
 		// Fired everyday at 7 a.m (this might vary for sites with few visitors)
-		add_action( 'getpaid_daily_maintenance', array( $this, 'log_cron_run' ) );
-		add_action( 'getpaid_daily_maintenance', array( $this, 'backwards_compat' ) );
-		add_action( 'getpaid_daily_maintenance', array( $this, 'maybe_expire_subscriptions' ) );
-		add_action( 'getpaid_daily_maintenance', array( $this, 'check_renewing_subscriptions' ) );
-		add_action( 'getpaid_daily_maintenance', array( $this, 'maybe_update_geoip_databases' ) );
+		add_action('getpaid_daily_maintenance', array($this, 'log_cron_run'));
+		add_action('getpaid_daily_maintenance', array($this, 'backwards_compat'));
+		add_action('getpaid_daily_maintenance', array($this, 'maybe_expire_subscriptions'));
+		add_action('getpaid_daily_maintenance', array($this, 'check_renewing_subscriptions'));
+		add_action('getpaid_daily_maintenance', array($this, 'maybe_update_geoip_databases'));
 
 	}
 
@@ -38,9 +38,9 @@ 
 	 */
 	public function maybe_create_scheduled_event() {
 
-		if ( ! wp_next_scheduled( 'getpaid_daily_maintenance' ) ) {
-			$timestamp = strtotime( 'tomorrow 07:00:00', current_time( 'timestamp' ) );
-			wp_schedule_event( $timestamp, 'daily', 'getpaid_daily_maintenance' );
+		if (!wp_next_scheduled('getpaid_daily_maintenance')) {
+			$timestamp = strtotime('tomorrow 07:00:00', current_time('timestamp'));
+			wp_schedule_event($timestamp, 'daily', 'getpaid_daily_maintenance');
 		}
 
 	}
@@ -51,10 +51,10 @@ 
 	 */
 	public function maybe_clear_deprecated_events() {
 
-		if ( ! get_option( 'wpinv_cleared_old_events' ) ) {
-			wp_clear_scheduled_hook( 'wpinv_register_schedule_event_twicedaily' );
-			wp_clear_scheduled_hook( 'wpinv_register_schedule_event_daily' );
-			update_option( 'wpinv_cleared_old_events', 1 );
+		if (!get_option('wpinv_cleared_old_events')) {
+			wp_clear_scheduled_hook('wpinv_register_schedule_event_twicedaily');
+			wp_clear_scheduled_hook('wpinv_register_schedule_event_daily');
+			update_option('wpinv_cleared_old_events', 1);
 		}
 
 	}
@@ -64,7 +64,7 @@ 
 	 *
 	 */
 	public function backwards_compat() {
-		do_action( 'wpinv_register_schedule_event_daily' );
+		do_action('wpinv_register_schedule_event_daily');
 	}
 
 	/**
@@ -74,29 +74,29 @@ 
 	public function check_renewing_subscriptions() {
 
 		// Fetch subscriptions that expire today.
-		$args  = array(
+		$args = array(
 			'number'             => -1,
 			'count_total'        => false,
 			'status'             => 'trialling active',
 			'date_expires_query' => array(
 				array(
-					'year'    => date( 'Y', current_time( 'timestamp' ) ),
-					'month'   => date( 'n', current_time( 'timestamp' ) ),
-					'day'     => date( 'j', current_time( 'timestamp' ) ),
+					'year'    => date('Y', current_time('timestamp')),
+					'month'   => date('n', current_time('timestamp')),
+					'day'     => date('j', current_time('timestamp')),
 					'compare' => '=',
 				),
 			),
 		);
 
-		$subscriptions = new GetPaid_Subscriptions_Query( $args );
+		$subscriptions = new GetPaid_Subscriptions_Query($args);
 
-		foreach ( $subscriptions->get_results() as $subscription ) {
+		foreach ($subscriptions->get_results() as $subscription) {
 
 			/** @var WPInv_Subscription $subscription */
-			if ( $subscription->is_last_renewal() ) {
+			if ($subscription->is_last_renewal()) {
 				$subscription->complete();
 			} else {
-				do_action( 'getpaid_should_renew_subscription', $subscription );
+				do_action('getpaid_should_renew_subscription', $subscription);
 			}
 }
 
@@ -109,7 +109,7 @@ 
 	public function maybe_expire_subscriptions() {
 
 		// Fetch expired subscriptions (skips those that expire today).
-		$args  = array(
+		$args = array(
 			'number'             => -1,
 			'count_total'        => false,
 			'status'             => 'trialling active failing cancelled',
@@ -119,11 +119,11 @@ 
 			),
 		);
 
-		$subscriptions = new GetPaid_Subscriptions_Query( $args );
+		$subscriptions = new GetPaid_Subscriptions_Query($args);
 
-		foreach ( $subscriptions->get_results() as $subscription ) {
-			if ( apply_filters( 'getpaid_daily_maintenance_should_expire_subscription', false, $subscription ) ) {
-				$subscription->set_status( 'expired' );
+		foreach ($subscriptions->get_results() as $subscription) {
+			if (apply_filters('getpaid_daily_maintenance_should_expire_subscription', false, $subscription)) {
+				$subscription->set_status('expired');
 				$subscription->save();
 			}
 		}
@@ -135,7 +135,7 @@ 
 	 *
 	 */
 	public function log_cron_run() {
-		wpinv_error_log( 'GetPaid Daily Cron', false );
+		wpinv_error_log('GetPaid Daily Cron', false);
 	}
 
 	/**
@@ -143,11 +143,11 @@ 
 	 *
 	 */
 	public function maybe_update_geoip_databases() {
-		$updated = get_transient( 'getpaid_updated_geoip_databases' );
+		$updated = get_transient('getpaid_updated_geoip_databases');
 
-		if ( false === $updated ) {
-			set_transient( 'getpaid_updated_geoip_databases', 1, 15 * DAY_IN_SECONDS );
-			do_action( 'getpaid_update_geoip_databases' );
+		if (false === $updated) {
+			set_transient('getpaid_updated_geoip_databases', 1, 15 * DAY_IN_SECONDS);
+			do_action('getpaid_update_geoip_databases');
 		}
 
 	}

includes/class-wpinv-privacy-exporters.php

Spacing +59 added lines, -59 removed lines

@@ -3,7 +3,7 @@ 
  * Personal data exporters.
  */
 
-defined( 'ABSPATH' ) || exit;
+defined('ABSPATH') || exit;
 
 /**
  * WPInv_Privacy_Exporters Class.
@@ -17,39 +17,39 @@ 
      * @param int    $page  Page.
      * @return array An array of invoice data in name value pairs
      */
-    public static function customer_invoice_data_exporter( $email_address, $page ) {
+    public static function customer_invoice_data_exporter($email_address, $page) {
         $done           = false;
         $page           = (int) $page;
         $data_to_export = array();
 
-        $user           = get_user_by( 'email', $email_address );
-        if ( ! $user instanceof WP_User ) {
+        $user           = get_user_by('email', $email_address);
+        if (!$user instanceof WP_User) {
             return array(
                 'data' => $data_to_export,
                 'done' => true,
             );
         }
 
-        $args    = array(
-            'limit'    => get_option( 'posts_per_page' ),
+        $args = array(
+            'limit'    => get_option('posts_per_page'),
             'page'     => $page,
             'user'     => $user->ID,
             'paginate' => false,
         );
 
-        $invoices = wpinv_get_invoices( $args );
+        $invoices = wpinv_get_invoices($args);
 
-        if ( 0 < count( $invoices ) ) {
-            foreach ( $invoices as $invoice ) {
+        if (0 < count($invoices)) {
+            foreach ($invoices as $invoice) {
                 $data_to_export[] = array(
                     'group_id'          => 'customer_invoices',
-                    'group_label'       => __( 'GetPaid: Invoices', 'invoicing' ),
-                    'group_description' => __( 'Customer invoices.', 'invoicing' ),
+                    'group_label'       => __('GetPaid: Invoices', 'invoicing'),
+                    'group_description' => __('Customer invoices.', 'invoicing'),
                     'item_id'           => "wpinv-{$invoice->get_id()}",
-                    'data'              => self::get_customer_invoice_data( $invoice ),
+                    'data'              => self::get_customer_invoice_data($invoice),
                 );
             }
-            $done = get_option( 'posts_per_page' ) > count( $invoices );
+            $done = get_option('posts_per_page') > count($invoices);
         } else {
             $done = true;
         }
@@ -67,116 +67,116 @@ 
      * @param WPInv_Invoice $invoice invoice object.
      * @return array
      */
-    public static function get_customer_invoice_data( $invoice ) {
+    public static function get_customer_invoice_data($invoice) {
 
         // Prepare basic properties.
         $props_to_export = array(
             'number'       => array(
-                'name'  => __( 'Invoice Number', 'invoicing' ),
+                'name'  => __('Invoice Number', 'invoicing'),
                 'value' => $invoice->get_number(),
             ),
             'created_date' => array(
-                'name'  => __( 'Created Date', 'invoicing' ),
+                'name'  => __('Created Date', 'invoicing'),
                 'value' => $invoice->get_date_created(),
             ),
             'due_date'     => array(
-                'name'  => __( 'Due Date', 'invoicing' ),
+                'name'  => __('Due Date', 'invoicing'),
                 'value' => $invoice->get_due_date(),
             ),
             'items'        => array(
-                'name'  => __( 'Invoice Items', 'invoicing' ),
-                'value' => self::process_invoice_items( $invoice ),
+                'name'  => __('Invoice Items', 'invoicing'),
+                'value' => self::process_invoice_items($invoice),
             ),
             'discount'     => array(
-                'name'  => __( 'Invoice Discount', 'invoicing' ),
-                'value' => wpinv_price( $invoice->get_total_discount(), $invoice->get_currency() ),
+                'name'  => __('Invoice Discount', 'invoicing'),
+                'value' => wpinv_price($invoice->get_total_discount(), $invoice->get_currency()),
             ),
             'total'        => array(
-                'name'  => __( 'Invoice Total', 'invoicing' ),
-                'value' => wpinv_price( $invoice->get_total(), $invoice->get_currency() ),
+                'name'  => __('Invoice Total', 'invoicing'),
+                'value' => wpinv_price($invoice->get_total(), $invoice->get_currency()),
             ),
             'status'       => array(
-                'name'  => __( 'Invoice Status', 'invoicing' ),
+                'name'  => __('Invoice Status', 'invoicing'),
                 'value' => $invoice->get_status_nicename(),
             ),
             'first_name'   => array(
-                'name'  => __( 'First Name', 'invoicing' ),
+                'name'  => __('First Name', 'invoicing'),
                 'value' => $invoice->get_first_name(),
             ),
             'last_name'    => array(
-                'name'  => __( 'Last Name', 'invoicing' ),
+                'name'  => __('Last Name', 'invoicing'),
                 'value' => $invoice->get_last_name(),
             ),
             'email'        => array(
-                'name'  => __( 'Email Address', 'invoicing' ),
+                'name'  => __('Email Address', 'invoicing'),
                 'value' => $invoice->get_email(),
             ),
             'company'      => array(
-                'name'  => __( 'Company', 'invoicing' ),
+                'name'  => __('Company', 'invoicing'),
                 'value' => $invoice->get_company(),
             ),
             'phone'        => array(
-                'name'  => __( 'Phone Number', 'invoicing' ),
+                'name'  => __('Phone Number', 'invoicing'),
                 'value' => $invoice->get_phone(),
             ),
             'address'      => array(
-                'name'  => __( 'Address', 'invoicing' ),
+                'name'  => __('Address', 'invoicing'),
                 'value' => $invoice->get_address(),
             ),
             'city'         => array(
-                'name'  => __( 'City', 'invoicing' ),
+                'name'  => __('City', 'invoicing'),
                 'value' => $invoice->get_city(),
             ),
             'state'        => array(
-                'name'  => __( 'State', 'invoicing' ),
+                'name'  => __('State', 'invoicing'),
                 'value' => $invoice->get_state(),
             ),
             'zip'          => array(
-                'name'  => __( 'Zip', 'invoicing' ),
+                'name'  => __('Zip', 'invoicing'),
                 'value' => $invoice->get_zip(),
             ),
             'vat_number'   => array(
-                'name'  => __( 'VAT Number', 'invoicing' ),
+                'name'  => __('VAT Number', 'invoicing'),
                 'value' => $invoice->get_vat_number(),
             ),
             'description'  => array(
-                'name'  => __( 'Description', 'invoicing' ),
+                'name'  => __('Description', 'invoicing'),
                 'value' => $invoice->get_description(),
             ),
         );
 
         // In case the invoice is paid, add the payment date and gateway.
-        if ( $invoice->is_paid() ) {
+        if ($invoice->is_paid()) {
 
             $props_to_export['completed_date'] = array(
-                'name'  => __( 'Completed Date', 'invoicing' ),
+                'name'  => __('Completed Date', 'invoicing'),
                 'value' => $invoice->get_completed_date(),
             );
 
             $props_to_export['gateway'] = array(
-                'name'  => __( 'Paid Via', 'invoicing' ),
+                'name'  => __('Paid Via', 'invoicing'),
                 'value' => $invoice->get_gateway(),
             );
 
         }
 
         // Maybe add subscription details.
-        $props_to_export = self::process_subscription( $invoice, $props_to_export );
+        $props_to_export = self::process_subscription($invoice, $props_to_export);
 
         // Add the ip address.
         $props_to_export['ip'] = array(
-            'name'  => __( 'IP Address', 'invoicing' ),
+            'name'  => __('IP Address', 'invoicing'),
             'value' => $invoice->get_ip(),
         );
 
         // Add the invoice url.
         $props_to_export['view_url'] = array(
-            'name'  => __( 'Invoice URL', 'invoicing' ),
+            'name'  => __('Invoice URL', 'invoicing'),
             'value' => $invoice->get_view_url(),
         );
 
         // Return the values.
-        return apply_filters( 'getpaid_privacy_export_invoice_personal_data', array_values( $props_to_export ), $invoice );
+        return apply_filters('getpaid_privacy_export_invoice_personal_data', array_values($props_to_export), $invoice);
 
     }
 
@@ -188,40 +188,40 @@ 
      * @param array $props invoice props.
      * @return array
      */
-    public static function process_subscription( $invoice, $props ) {
+    public static function process_subscription($invoice, $props) {
 
-        $subscription = wpinv_get_subscription( $invoice );
-        if ( ! empty( $subscription ) ) {
+        $subscription = wpinv_get_subscription($invoice);
+        if (!empty($subscription)) {
 
-            $frequency    = getpaid_get_subscription_period_label( $subscription->get_period(), $subscription->get_frequency() );
-            $period       = wpinv_price( $subscription->get_recurring_amount(), $subscription->get_parent_payment()->get_currency() ) . ' / ' . $frequency;
-            $initial_amt  = wpinv_price( $subscription->get_initial_amount(), $subscription->get_parent_payment()->get_currency() );
-            $bill_times   = $subscription->get_times_billed() . ' / ' . ( ( $subscription->get_bill_times() == 0 ) ? __( 'Until Cancelled', 'invoicing' ) : $subscription->get_bill_times() );
-            $renewal_date = getpaid_format_date_value( $subscription->get_expiration() );
+            $frequency    = getpaid_get_subscription_period_label($subscription->get_period(), $subscription->get_frequency());
+            $period       = wpinv_price($subscription->get_recurring_amount(), $subscription->get_parent_payment()->get_currency()) . ' / ' . $frequency;
+            $initial_amt  = wpinv_price($subscription->get_initial_amount(), $subscription->get_parent_payment()->get_currency());
+            $bill_times   = $subscription->get_times_billed() . ' / ' . (($subscription->get_bill_times() == 0) ? __('Until Cancelled', 'invoicing') : $subscription->get_bill_times());
+            $renewal_date = getpaid_format_date_value($subscription->get_expiration());
 
             // Billing cycle.
             $props['period'] = array(
-                'name'  => __( 'Billing Cycle', 'invoicing' ),
+                'name'  => __('Billing Cycle', 'invoicing'),
                 'value' => $period,
             );
 
             // Initial amount.
             $props['initial_amount'] = array(
-                'name'  => __( 'Initial Amount', 'invoicing' ),
+                'name'  => __('Initial Amount', 'invoicing'),
                 'value' => $initial_amt,
             );
 
             // Bill times.
             $props['bill_times'] = array(
-                'name'  => __( 'Times Billed', 'invoicing' ),
+                'name'  => __('Times Billed', 'invoicing'),
                 'value' => $bill_times,
             );
 
             // Add expiry date.
-            if ( $subscription->is_active() ) {
+            if ($subscription->is_active()) {
 
                 $props['renewal_date'] = array(
-                    'name'  => __( 'Expires', 'invoicing' ),
+                    'name'  => __('Expires', 'invoicing'),
                     'value' => $renewal_date,
                 );
 
@@ -239,19 +239,19 @@ 
      * @param WPInv_Invoice $invoice invoice object.
      * @return array
      */
-    public static function process_invoice_items( $invoice ) {
+    public static function process_invoice_items($invoice) {
 
         $item_names = array();
-        foreach ( $invoice->get_items() as $cart_item ) {
+        foreach ($invoice->get_items() as $cart_item) {
             $item_names[] = sprintf(
                 '%s x %s - %s',
                 $cart_item->get_name(),
                 $cart_item->get_quantity(),
-                wpinv_price( $invoice->is_renewal() ? $cart_item->get_recurring_sub_total() : $cart_item->get_sub_total(), $invoice->get_currency() )
+                wpinv_price($invoice->is_renewal() ? $cart_item->get_recurring_sub_total() : $cart_item->get_sub_total(), $invoice->get_currency())
             );
         }
 
-        return implode( ', ', $item_names );
+        return implode(', ', $item_names);
 
     }
 

includes/wpinv-discount-functions.php

Spacing +44 added lines, -44 removed lines

@@ -6,7 +6,7 @@ 
  * @package Invoicing
  */
 
-defined( 'ABSPATH' ) || exit;
+defined('ABSPATH') || exit;
 
 /**
  * Returns an array of discount type.
@@ -17,8 +17,8 @@ 
     return apply_filters(
         'wpinv_discount_types',
         array(
-            'percent' => __( 'Percentage', 'invoicing' ),
-            'flat'    => __( 'Flat Amount', 'invoicing' ),
+            'percent' => __('Percentage', 'invoicing'),
+            'flat'    => __('Flat Amount', 'invoicing'),
         )
     );
 }
@@ -28,46 +28,46 @@ 
  *
  * @return string
  */
-function wpinv_get_discount_type_name( $type = '' ) {
+function wpinv_get_discount_type_name($type = '') {
     $types = wpinv_get_discount_types();
-    return isset( $types[ $type ] ) ? $types[ $type ] : $type;
+    return isset($types[$type]) ? $types[$type] : $type;
 }
 
 /**
  * Deletes a discount via the admin page.
  *
  */
-function wpinv_delete_discount( $data ) {
+function wpinv_delete_discount($data) {
 
-    $discount = new WPInv_Discount( absint( $data['discount'] ) );
-    $discount->delete( true );
+    $discount = new WPInv_Discount(absint($data['discount']));
+    $discount->delete(true);
 
 }
-add_action( 'getpaid_authenticated_admin_action_delete_discount', 'wpinv_delete_discount' );
+add_action('getpaid_authenticated_admin_action_delete_discount', 'wpinv_delete_discount');
 
 /**
  * Deactivates a discount via the admin page.
  */
-function wpinv_activate_discount( $data ) {
+function wpinv_activate_discount($data) {
 
-    $discount = new WPInv_Discount( absint( $data['discount'] ) );
-    $discount->set_status( 'publish' );
+    $discount = new WPInv_Discount(absint($data['discount']));
+    $discount->set_status('publish');
     $discount->save();
 
 }
-add_action( 'getpaid_authenticated_admin_action_activate_discount', 'wpinv_activate_discount' );
+add_action('getpaid_authenticated_admin_action_activate_discount', 'wpinv_activate_discount');
 
 /**
  * Activates a discount via the admin page.
  */
-function wpinv_deactivate_discount( $data ) {
+function wpinv_deactivate_discount($data) {
 
-    $discount = new WPInv_Discount( absint( $data['discount'] ) );
-    $discount->set_status( 'pending' );
+    $discount = new WPInv_Discount(absint($data['discount']));
+    $discount->set_status('pending');
     $discount->save();
 
 }
-add_action( 'getpaid_authenticated_admin_action_deactivate_discount', 'wpinv_deactivate_discount' );
+add_action('getpaid_authenticated_admin_action_deactivate_discount', 'wpinv_deactivate_discount');
 
 /**
  * Fetches a discount object.
@@ -76,8 +76,8 @@ 
  * @since 1.0.15
  * @return WPInv_Discount
  */
-function wpinv_get_discount( $discount ) {
-    return new WPInv_Discount( $discount );
+function wpinv_get_discount($discount) {
+    return new WPInv_Discount($discount);
 }
 
 /**
@@ -87,8 +87,8 @@ 
  * @since 1.0.15
  * @return WPInv_Discount
  */
-function wpinv_get_discount_obj( $discount = 0 ) {
-    return new WPInv_Discount( $discount );
+function wpinv_get_discount_obj($discount = 0) {
+    return new WPInv_Discount($discount);
 }
 
 /**
@@ -98,10 +98,10 @@ 
  * @param string|int $value The field value
  * @return bool|WPInv_Discount
  */
-function wpinv_get_discount_by( $deprecated = null, $value = '' ) {
-    $discount = new WPInv_Discount( $value );
+function wpinv_get_discount_by($deprecated = null, $value = '') {
+    $discount = new WPInv_Discount($value);
 
-    if ( $discount->get_id() != 0 ) {
+    if ($discount->get_id() != 0) {
         return $discount;
     }
 
@@ -116,9 +116,9 @@ 
 function wpinv_get_discount_statuses() {
 
     return array(
-        'expired'  => __( 'Expired', 'invoicing' ),
-        'publish'  => __( 'Active', 'invoicing' ),
-        'inactive' => __( 'Inactive', 'invoicing' ),
+        'expired'  => __('Expired', 'invoicing'),
+        'publish'  => __('Active', 'invoicing'),
+        'inactive' => __('Inactive', 'invoicing'),
     );
 
 }
@@ -126,9 +126,9 @@ 
 /**
  * Retrieves an invoice status label.
  */
-function wpinv_discount_status( $status ) {
+function wpinv_discount_status($status) {
     $statuses = wpinv_get_discount_statuses();
-    return isset( $statuses[ $status ] ) ? $statuses[ $status ] : __( 'Inactive', 'invoicing' );
+    return isset($statuses[$status]) ? $statuses[$status] : __('Inactive', 'invoicing');
 }
 
 /**
@@ -138,12 +138,12 @@ 
  * @param int|array|string|WPInv_Discount $code discount data, object, ID or code.
  * @return bool
  */
-function wpinv_discount_is_recurring( $discount = 0, $code = 0 ) {
+function wpinv_discount_is_recurring($discount = 0, $code = 0) {
 
-    if ( ! empty( $discount ) ) {
-        $discount    = wpinv_get_discount_obj( $discount );
+    if (!empty($discount)) {
+        $discount    = wpinv_get_discount_obj($discount);
     } else {
-        $discount    = wpinv_get_discount_obj( $code );
+        $discount    = wpinv_get_discount_obj($code);
     }
 
     return $discount->get_is_recurring();
@@ -158,35 +158,35 @@ 
  * @param WPInv_Discount $discount
  * @return array
  */
-function getpaid_calculate_invoice_discount( $invoice, $discount ) {
+function getpaid_calculate_invoice_discount($invoice, $discount) {
 
 	$initial_discount   = 0;
 	$recurring_discount = 0;
 
-	foreach ( $invoice->get_items() as $item ) {
+	foreach ($invoice->get_items() as $item) {
 
 		// Abort if it is not valid for this item.
-		if ( ! $discount->is_valid_for_items( array( $item->get_id() ) ) ) {
+		if (!$discount->is_valid_for_items(array($item->get_id()))) {
 			continue;
 		}
 
 		// Calculate the initial amount...
-		$item_discount           = $discount->get_discounted_amount( $item->get_sub_total() );
+		$item_discount           = $discount->get_discounted_amount($item->get_sub_total());
 		$recurring_item_discount = 0;
 
 		// ... and maybe the recurring amount.
-		if ( $item->is_recurring() && $discount->is_recurring() ) {
-			$recurring_item_discount = $discount->get_discounted_amount( $item->get_recurring_sub_total() );
+		if ($item->is_recurring() && $discount->is_recurring()) {
+			$recurring_item_discount = $discount->get_discounted_amount($item->get_recurring_sub_total());
 		}
 
 		// Discount should not exceed discounted amount.
-		if ( ! $discount->is_type( 'percent' ) ) {
+		if (!$discount->is_type('percent')) {
 
-			if ( ( $initial_discount + $item_discount ) > $discount->get_amount() ) {
+			if (($initial_discount + $item_discount) > $discount->get_amount()) {
 				$item_discount = $discount->get_amount() - $initial_discount;
 			}
 
-			if ( ( $recurring_discount + $recurring_item_discount ) > $discount->get_amount() ) {
+			if (($recurring_discount + $recurring_item_discount) > $discount->get_amount()) {
 				$recurring_item_discount = $discount->get_amount() - $recurring_discount;
 			}
 }
@@ -218,10 +218,10 @@ 
         array(
             'post_type'   => 'wpi_discount',
             'numberposts' => 1,
-            'fields'      => array( 'ids' ),
+            'fields'      => array('ids'),
         )
     );
 
-    return ! empty( $discounts );
+    return !empty($discounts);
 
 }