AyeCode / invoicing

includes/admin/class-getpaid-admin.php

Spacing +186 added lines, -186 removed lines

@@ -4,7 +4,7 @@ 
  *
  */
 
-defined( 'ABSPATH' ) || exit;
+defined('ABSPATH') || exit;
 
 /**
  * The main admin class.
@@ -37,13 +37,13 @@ 
     /**
 	 * Class constructor.
 	 */
-	public function __construct(){
+	public function __construct() {
 
-        $this->admin_path  = plugin_dir_path( __FILE__ );
-		$this->admin_url   = plugins_url( '/', __FILE__ );
+        $this->admin_path = plugin_dir_path(__FILE__);
+		$this->admin_url   = plugins_url('/', __FILE__);
 		$this->reports     = new GetPaid_Reports();
 
-        if ( is_admin() ) {
+        if (is_admin()) {
 			$this->init_admin_hooks();
         }
 
@@ -54,22 +54,22 @@ 
 	 *
 	 */
 	private function init_admin_hooks() {
-        add_action( 'admin_enqueue_scripts', array( $this, 'enqeue_scripts' ) );
-        add_filter( 'admin_body_class', array( $this, 'admin_body_class' ) );
-        add_action( 'admin_init', array( $this, 'init_ayecode_connect_helper' ) );
-        add_action( 'admin_init', array( $this, 'activation_redirect') );
-        add_action( 'admin_init', array( $this, 'maybe_do_admin_action') );
-		add_action( 'admin_notices', array( $this, 'show_notices' ) );
-		add_action( 'getpaid_authenticated_admin_action_rate_plugin', array( $this, 'redirect_to_wordpress_rating_page' ) );
-		add_action( 'getpaid_authenticated_admin_action_send_invoice', array( $this, 'send_customer_invoice' ) );
-		add_action( 'getpaid_authenticated_admin_action_send_invoice_reminder', array( $this, 'send_customer_payment_reminder' ) );
-        add_action( 'getpaid_authenticated_admin_action_reset_tax_rates', array( $this, 'admin_reset_tax_rates' ) );
-		add_action( 'getpaid_authenticated_admin_action_create_missing_pages', array( $this, 'admin_create_missing_pages' ) );
-		add_action( 'getpaid_authenticated_admin_action_create_missing_tables', array( $this, 'admin_create_missing_tables' ) );
-		add_action( 'getpaid_authenticated_admin_action_migrate_old_invoices', array( $this, 'admin_migrate_old_invoices' ) );
-		add_action( 'getpaid_authenticated_admin_action_recalculate_discounts', array( $this, 'admin_recalculate_discounts' ) );
-		add_filter( 'admin_footer_text', array( $this, 'admin_footer_text' ) );
-		do_action( 'getpaid_init_admin_hooks', $this );
+        add_action('admin_enqueue_scripts', array($this, 'enqeue_scripts'));
+        add_filter('admin_body_class', array($this, 'admin_body_class'));
+        add_action('admin_init', array($this, 'init_ayecode_connect_helper'));
+        add_action('admin_init', array($this, 'activation_redirect'));
+        add_action('admin_init', array($this, 'maybe_do_admin_action'));
+		add_action('admin_notices', array($this, 'show_notices'));
+		add_action('getpaid_authenticated_admin_action_rate_plugin', array($this, 'redirect_to_wordpress_rating_page'));
+		add_action('getpaid_authenticated_admin_action_send_invoice', array($this, 'send_customer_invoice'));
+		add_action('getpaid_authenticated_admin_action_send_invoice_reminder', array($this, 'send_customer_payment_reminder'));
+        add_action('getpaid_authenticated_admin_action_reset_tax_rates', array($this, 'admin_reset_tax_rates'));
+		add_action('getpaid_authenticated_admin_action_create_missing_pages', array($this, 'admin_create_missing_pages'));
+		add_action('getpaid_authenticated_admin_action_create_missing_tables', array($this, 'admin_create_missing_tables'));
+		add_action('getpaid_authenticated_admin_action_migrate_old_invoices', array($this, 'admin_migrate_old_invoices'));
+		add_action('getpaid_authenticated_admin_action_recalculate_discounts', array($this, 'admin_recalculate_discounts'));
+		add_filter('admin_footer_text', array($this, 'admin_footer_text'));
+		do_action('getpaid_init_admin_hooks', $this);
 
     }
 
@@ -80,37 +80,37 @@ 
 	public function enqeue_scripts() {
         global $current_screen, $pagenow;
 
-		$page    = isset( $_GET['page'] ) ? $_GET['page'] : '';
+		$page    = isset($_GET['page']) ? $_GET['page'] : '';
 		$editing = $pagenow == 'post.php' || $pagenow == 'post-new.php';
 
-        if ( ! empty( $current_screen->post_type ) ) {
+        if (!empty($current_screen->post_type)) {
 			$page = $current_screen->post_type;
         }
 
         // General styles.
-        if ( false !== stripos( $page, 'wpi' ) ) {
+        if (false !== stripos($page, 'wpi')) {
 
             // Styles.
-            $version = filemtime( WPINV_PLUGIN_DIR . 'assets/css/admin.css' );
-            wp_enqueue_style( 'wpinv_admin_style', WPINV_PLUGIN_URL . 'assets/css/admin.css', array( 'wp-color-picker' ), $version );
-            wp_enqueue_style( 'select2', WPINV_PLUGIN_URL . 'assets/css/select2/select2.min.css', array(), '4.0.13', 'all' );
+            $version = filemtime(WPINV_PLUGIN_DIR . 'assets/css/admin.css');
+            wp_enqueue_style('wpinv_admin_style', WPINV_PLUGIN_URL . 'assets/css/admin.css', array('wp-color-picker'), $version);
+            wp_enqueue_style('select2', WPINV_PLUGIN_URL . 'assets/css/select2/select2.min.css', array(), '4.0.13', 'all');
 
             // Scripts.
-            wp_enqueue_script('select2', WPINV_PLUGIN_URL . 'assets/js/select2/select2.full.min.js', array( 'jquery' ), WPINV_VERSION );
+            wp_enqueue_script('select2', WPINV_PLUGIN_URL . 'assets/js/select2/select2.full.min.js', array('jquery'), WPINV_VERSION);
 
-            $version = filemtime( WPINV_PLUGIN_DIR . 'assets/js/admin.js' );
-            wp_enqueue_script( 'wpinv-admin-script', WPINV_PLUGIN_URL . 'assets/js/admin.js', array( 'jquery', 'wp-color-picker' ),  $version );
-            wp_localize_script( 'wpinv-admin-script', 'WPInv_Admin', apply_filters( 'wpinv_admin_js_localize', $this->get_admin_i18() ) );
+            $version = filemtime(WPINV_PLUGIN_DIR . 'assets/js/admin.js');
+            wp_enqueue_script('wpinv-admin-script', WPINV_PLUGIN_URL . 'assets/js/admin.js', array('jquery', 'wp-color-picker'), $version);
+            wp_localize_script('wpinv-admin-script', 'WPInv_Admin', apply_filters('wpinv_admin_js_localize', $this->get_admin_i18()));
 
         }
 
         // Payment form scripts.
-		if ( 'wpi_payment_form' == $page && $editing ) {
+		if ('wpi_payment_form' == $page && $editing) {
             $this->load_payment_form_scripts();
         }
 
-		if ( $page == 'wpinv-subscriptions' ) {
-			wp_enqueue_script( 'postbox' );
+		if ($page == 'wpinv-subscriptions') {
+			wp_enqueue_script('postbox');
 		}
 
     }
@@ -123,32 +123,32 @@ 
         global $post;
 
 		$date_range = array(
-			'period' => isset( $_GET['date_range'] ) ? sanitize_text_field( $_GET['date_range'] ) : '7_days'
+			'period' => isset($_GET['date_range']) ? sanitize_text_field($_GET['date_range']) : '7_days'
 		);
 
-		if ( $date_range['period'] == 'custom' ) {
+		if ($date_range['period'] == 'custom') {
 			
-			if ( isset( $_GET['from'] ) ) {
-				$date_range[ 'after' ] = date( 'Y-m-d', strtotime( sanitize_text_field( $_GET['from'] ), current_time( 'timestamp' ) ) - DAY_IN_SECONDS );
+			if (isset($_GET['from'])) {
+				$date_range['after'] = date('Y-m-d', strtotime(sanitize_text_field($_GET['from']), current_time('timestamp')) - DAY_IN_SECONDS);
 			}
 
-			if ( isset( $_GET['to'] ) ) {
-				$date_range[ 'before' ] = date( 'Y-m-d', strtotime( sanitize_text_field( $_GET['to'] ), current_time( 'timestamp' ) ) + DAY_IN_SECONDS );
+			if (isset($_GET['to'])) {
+				$date_range['before'] = date('Y-m-d', strtotime(sanitize_text_field($_GET['to']), current_time('timestamp')) + DAY_IN_SECONDS);
 			}
 
 		}
 
         $i18n = array(
-            'ajax_url'                  => admin_url( 'admin-ajax.php' ),
-            'post_ID'                   => isset( $post->ID ) ? $post->ID : '',
-			'wpinv_nonce'               => wp_create_nonce( 'wpinv-nonce' ),
-			'rest_nonce'                => wp_create_nonce( 'wp_rest' ),
-			'rest_root'                 => esc_url_raw( rest_url() ),
+            'ajax_url'                  => admin_url('admin-ajax.php'),
+            'post_ID'                   => isset($post->ID) ? $post->ID : '',
+			'wpinv_nonce'               => wp_create_nonce('wpinv-nonce'),
+			'rest_nonce'                => wp_create_nonce('wp_rest'),
+			'rest_root'                 => esc_url_raw(rest_url()),
 			'date_range'                => $date_range,
-            'add_invoice_note_nonce'    => wp_create_nonce( 'add-invoice-note' ),
-            'delete_invoice_note_nonce' => wp_create_nonce( 'delete-invoice-note' ),
-            'invoice_item_nonce'        => wp_create_nonce( 'invoice-item' ),
-            'billing_details_nonce'     => wp_create_nonce( 'get-billing-details' ),
+            'add_invoice_note_nonce'    => wp_create_nonce('add-invoice-note'),
+            'delete_invoice_note_nonce' => wp_create_nonce('delete-invoice-note'),
+            'invoice_item_nonce'        => wp_create_nonce('invoice-item'),
+            'billing_details_nonce'     => wp_create_nonce('get-billing-details'),
             'tax'                       => wpinv_tax_amount(),
             'discount'                  => 0,
 			'currency_symbol'           => wpinv_currency_symbol(),
@@ -157,38 +157,38 @@ 
             'thousand_sep'              => wpinv_thousands_separator(),
             'decimal_sep'               => wpinv_decimal_separator(),
             'decimals'                  => wpinv_decimals(),
-            'save_invoice'              => __( 'Save Invoice', 'invoicing' ),
-            'status_publish'            => wpinv_status_nicename( 'publish' ),
-            'status_pending'            => wpinv_status_nicename( 'wpi-pending' ),
-            'delete_tax_rate'           => __( 'Are you sure you wish to delete this tax rate?', 'invoicing' ),
-            'status_pending'            => wpinv_status_nicename( 'wpi-pending' ),
-            'FillBillingDetails'        => __( 'Fill the user\'s billing information? This will remove any currently entered billing information', 'invoicing' ),
-            'confirmCalcTotals'         => __( 'Recalculate totals? This will recalculate totals based on the user billing country. If no billing country is set it will use the base country.', 'invoicing' ),
-            'AreYouSure'                => __( 'Are you sure?', 'invoicing' ),
-            'errDeleteItem'             => __( 'This item is in use! Before delete this item, you need to delete all the invoice(s) using this item.', 'invoicing' ),
-            'delete_subscription'       => __( 'Are you sure you want to delete this subscription?', 'invoicing' ),
-            'action_edit'               => __( 'Edit', 'invoicing' ),
-            'action_cancel'             => __( 'Cancel', 'invoicing' ),
-            'item_description'          => __( 'Item Description', 'invoicing' ),
-            'invoice_description'       => __( 'Invoice Description', 'invoicing' ),
-            'discount_description'      => __( 'Discount Description', 'invoicing' ),
-			'searching'                 => __( 'Searching', 'invoicing' ),
-			'loading'                   => __( 'Loading...', 'invoicing' ),
-			'search_customers'          => __( 'Enter customer name or email', 'invoicing' ),
-			'search_items'              => __( 'Enter item name', 'invoicing' ),
+            'save_invoice'              => __('Save Invoice', 'invoicing'),
+            'status_publish'            => wpinv_status_nicename('publish'),
+            'status_pending'            => wpinv_status_nicename('wpi-pending'),
+            'delete_tax_rate'           => __('Are you sure you wish to delete this tax rate?', 'invoicing'),
+            'status_pending'            => wpinv_status_nicename('wpi-pending'),
+            'FillBillingDetails'        => __('Fill the user\'s billing information? This will remove any currently entered billing information', 'invoicing'),
+            'confirmCalcTotals'         => __('Recalculate totals? This will recalculate totals based on the user billing country. If no billing country is set it will use the base country.', 'invoicing'),
+            'AreYouSure'                => __('Are you sure?', 'invoicing'),
+            'errDeleteItem'             => __('This item is in use! Before delete this item, you need to delete all the invoice(s) using this item.', 'invoicing'),
+            'delete_subscription'       => __('Are you sure you want to delete this subscription?', 'invoicing'),
+            'action_edit'               => __('Edit', 'invoicing'),
+            'action_cancel'             => __('Cancel', 'invoicing'),
+            'item_description'          => __('Item Description', 'invoicing'),
+            'invoice_description'       => __('Invoice Description', 'invoicing'),
+            'discount_description'      => __('Discount Description', 'invoicing'),
+			'searching'                 => __('Searching', 'invoicing'),
+			'loading'                   => __('Loading...', 'invoicing'),
+			'search_customers'          => __('Enter customer name or email', 'invoicing'),
+			'search_items'              => __('Enter item name', 'invoicing'),
         );
 
-		if ( ! empty( $post ) && getpaid_is_invoice_post_type( $post->post_type ) ) {
+		if (!empty($post) && getpaid_is_invoice_post_type($post->post_type)) {
 
-			$invoice              = new WPInv_Invoice( $post );
+			$invoice              = new WPInv_Invoice($post);
 			$i18n['save_invoice'] = sprintf(
-				__( 'Save %s', 'invoicing' ),
-				ucfirst( $invoice->get_invoice_quote_type() )
+				__('Save %s', 'invoicing'),
+				ucfirst($invoice->get_invoice_quote_type())
 			);
 
 			$i18n['invoice_description'] = sprintf(
-				__( '%s Description', 'invoicing' ),
-				ucfirst( $invoice->get_invoice_quote_type() )
+				__('%s Description', 'invoicing'),
+				ucfirst($invoice->get_invoice_quote_type())
 			);
 
 		}
@@ -202,24 +202,24 @@ 
 	 * @param  string $footer_text
 	 * @return string
 	 */
-	public function admin_footer_text( $footer_text ) {
+	public function admin_footer_text($footer_text) {
 		global $current_screen;
 
-		$page    = isset( $_GET['page'] ) ? $_GET['page'] : '';
+		$page = isset($_GET['page']) ? $_GET['page'] : '';
 
-        if ( ! empty( $current_screen->post_type ) ) {
+        if (!empty($current_screen->post_type)) {
 			$page = $current_screen->post_type;
         }
 
         // General styles.
-        if ( apply_filters( 'getpaid_display_admin_footer_text', wpinv_current_user_can_manage_invoicing() ) && false !== stripos( $page, 'wpi' ) ) {
+        if (apply_filters('getpaid_display_admin_footer_text', wpinv_current_user_can_manage_invoicing()) && false !== stripos($page, 'wpi')) {
 
 			// Change the footer text
-			if ( ! get_user_meta( get_current_user_id(), 'getpaid_admin_footer_text_rated', true ) ) {
+			if (!get_user_meta(get_current_user_id(), 'getpaid_admin_footer_text_rated', true)) {
 
-				$rating_url  = esc_url(
+				$rating_url = esc_url(
 					wp_nonce_url(
-						admin_url( 'admin.php?page=wpinv-reports&getpaid-admin-action=rate_plugin' ),
+						admin_url('admin.php?page=wpinv-reports&getpaid-admin-action=rate_plugin'),
 						'getpaid-nonce',
 						'getpaid-nonce'
 						)
@@ -227,7 +227,7 @@ 
 
 				$footer_text = sprintf(
 					/* translators: %s: five stars */
-					__( 'If you like <strong>GetPaid</strong>, please leave us a %s rating. A huge thanks in advance!', 'invoicing' ),
+					__('If you like <strong>GetPaid</strong>, please leave us a %s rating. A huge thanks in advance!', 'invoicing'),
 					"<a href='$rating_url'>&#9733;&#9733;&#9733;&#9733;&#9733;</a>"
 				);
 
@@ -235,7 +235,7 @@ 
 
 				$footer_text = sprintf(
 					/* translators: %s: GetPaid */
-					__( 'Thank you for using %s!', 'invoicing' ),
+					__('Thank you for using %s!', 'invoicing'),
 					"<a href='https://wpgetpaid.com/' target='_blank'><strong>GetPaid</strong></a>"
 				);
 
@@ -252,8 +252,8 @@ 
 	 * @since  2.0.0
 	 */
 	public function redirect_to_wordpress_rating_page() {
-		update_user_meta( get_current_user_id(), 'getpaid_admin_footer_text_rated', 1 );
-		wp_redirect( 'https://wordpress.org/support/plugin/invoicing/reviews?rate=5#new-post' );
+		update_user_meta(get_current_user_id(), 'getpaid_admin_footer_text_rated', 1);
+		wp_redirect('https://wordpress.org/support/plugin/invoicing/reviews?rate=5#new-post');
 		exit;
 	}
 
@@ -264,30 +264,30 @@ 
 	protected function load_payment_form_scripts() {
         global $post;
 
-        wp_enqueue_script( 'vue', WPINV_PLUGIN_URL . 'assets/js/vue/vue.js', array(), WPINV_VERSION );
-		wp_enqueue_script( 'sortable', WPINV_PLUGIN_URL . 'assets/js/sortable.min.js', array(), WPINV_VERSION );
-		wp_enqueue_script( 'vue_draggable', WPINV_PLUGIN_URL . 'assets/js/vue/vuedraggable.min.js', array( 'sortable', 'vue' ), WPINV_VERSION );
+        wp_enqueue_script('vue', WPINV_PLUGIN_URL . 'assets/js/vue/vue.js', array(), WPINV_VERSION);
+		wp_enqueue_script('sortable', WPINV_PLUGIN_URL . 'assets/js/sortable.min.js', array(), WPINV_VERSION);
+		wp_enqueue_script('vue_draggable', WPINV_PLUGIN_URL . 'assets/js/vue/vuedraggable.min.js', array('sortable', 'vue'), WPINV_VERSION);
 
-		$version = filemtime( WPINV_PLUGIN_DIR . 'assets/js/admin-payment-forms.js' );
-		wp_register_script( 'wpinv-admin-payment-form-script', WPINV_PLUGIN_URL . 'assets/js/admin-payment-forms.js', array( 'wpinv-admin-script', 'vue_draggable' ),  $version );
+		$version = filemtime(WPINV_PLUGIN_DIR . 'assets/js/admin-payment-forms.js');
+		wp_register_script('wpinv-admin-payment-form-script', WPINV_PLUGIN_URL . 'assets/js/admin-payment-forms.js', array('wpinv-admin-script', 'vue_draggable'), $version);
 
 		wp_localize_script(
             'wpinv-admin-payment-form-script',
             'wpinvPaymentFormAdmin',
             array(
-				'elements'      => wpinv_get_data( 'payment-form-elements' ),
-				'form_elements' => getpaid_get_payment_form_elements( $post->ID ),
+				'elements'      => wpinv_get_data('payment-form-elements'),
+				'form_elements' => getpaid_get_payment_form_elements($post->ID),
 				'currency'      => wpinv_currency_symbol(),
 				'position'      => wpinv_currency_position(),
 				'decimals'      => (int) wpinv_decimals(),
 				'thousands_sep' => wpinv_thousands_separator(),
 				'decimals_sep'  => wpinv_decimal_separator(),
-				'form_items'    => gepaid_get_form_items( $post->ID ),
+				'form_items'    => gepaid_get_form_items($post->ID),
 				'is_default'    => $post->ID == wpinv_get_default_payment_form(),
             )
         );
 
-        wp_enqueue_script( 'wpinv-admin-payment-form-script' );
+        wp_enqueue_script('wpinv-admin-payment-form-script');
 
     }
 
@@ -298,25 +298,25 @@ 
      * @return string
 	 *
 	 */
-    public function admin_body_class( $classes ) {
+    public function admin_body_class($classes) {
 		global $pagenow, $post, $current_screen;
 
 
-        $page = isset( $_GET['page'] ) ? $_GET['page'] : '';
+        $page = isset($_GET['page']) ? $_GET['page'] : '';
 
-        if ( ! empty( $current_screen->post_type ) ) {
+        if (!empty($current_screen->post_type)) {
 			$page = $current_screen->post_type;
         }
 
-        if ( false !== stripos( $page, 'wpi' ) ) {
-            $classes .= ' wpi-' . sanitize_key( $page );
+        if (false !== stripos($page, 'wpi')) {
+            $classes .= ' wpi-' . sanitize_key($page);
         }
 
-        if ( in_array( $page, wpinv_parse_list( 'wpi_invoice wpi_payment_form wpi_quote' ) ) ) {
+        if (in_array($page, wpinv_parse_list('wpi_invoice wpi_payment_form wpi_quote'))) {
             $classes .= ' wpinv-cpt wpinv';
 		}
 		
-		if ( getpaid_is_invoice_post_type( $page ) ) {
+		if (getpaid_is_invoice_post_type($page)) {
             $classes .= ' getpaid-is-invoice-cpt';
         }
 
@@ -326,19 +326,19 @@ 
     /**
 	 * Maybe show the AyeCode Connect Notice.
 	 */
-	public function init_ayecode_connect_helper(){
+	public function init_ayecode_connect_helper() {
 
         new AyeCode_Connect_Helper(
             array(
-				'connect_title' => __("WP Invoicing - an AyeCode product!","invoicing"),
-				'connect_external'  => __( "Please confirm you wish to connect your site?","invoicing" ),
-				'connect'           => sprintf( __( "<strong>Have a license?</strong> Forget about entering license keys or downloading zip files, connect your site for instant access. %slearn more%s","invoicing" ),"<a href='https://ayecode.io/introducing-ayecode-connect/' target='_blank'>","</a>" ),
-				'connect_button'    => __("Connect Site","invoicing"),
-				'connecting_button'    => __("Connecting...","invoicing"),
-				'error_localhost'   => __( "This service will only work with a live domain, not a localhost.","invoicing" ),
-				'error'             => __( "Something went wrong, please refresh and try again.","invoicing" ),
+				'connect_title' => __("WP Invoicing - an AyeCode product!", "invoicing"),
+				'connect_external'  => __("Please confirm you wish to connect your site?", "invoicing"),
+				'connect'           => sprintf(__("<strong>Have a license?</strong> Forget about entering license keys or downloading zip files, connect your site for instant access. %slearn more%s", "invoicing"), "<a href='https://ayecode.io/introducing-ayecode-connect/' target='_blank'>", "</a>"),
+				'connect_button'    => __("Connect Site", "invoicing"),
+				'connecting_button'    => __("Connecting...", "invoicing"),
+				'error_localhost'   => __("This service will only work with a live domain, not a localhost.", "invoicing"),
+				'error'             => __("Something went wrong, please refresh and try again.", "invoicing"),
             ),
-            array( 'wpi-addons' )
+            array('wpi-addons')
         );
 
     }
@@ -350,20 +350,20 @@ 
 	 */
 	public function activation_redirect() {
 
-		$redirected = get_option( 'wpinv_redirected_to_settings' );
+		$redirected = get_option('wpinv_redirected_to_settings');
 
-		if ( ! empty( $redirected ) || wp_doing_ajax() || ! current_user_can( 'manage_options' ) ) {
+		if (!empty($redirected) || wp_doing_ajax() || !current_user_can('manage_options')) {
 			return;
 		}
 
 		// Bail if activating from network, or bulk
-		if ( is_network_admin() || isset( $_GET['activate-multi'] ) ) {
+		if (is_network_admin() || isset($_GET['activate-multi'])) {
 			return;
 		}
 
-	    update_option( 'wpinv_redirected_to_settings', 1 );
+	    update_option('wpinv_redirected_to_settings', 1);
 
-        wp_safe_redirect( admin_url( 'admin.php?page=wpinv-settings&tab=general' ) );
+        wp_safe_redirect(admin_url('admin.php?page=wpinv-settings&tab=general'));
         exit;
 
 	}
@@ -373,9 +373,9 @@ 
      */
     public function maybe_do_admin_action() {
 
-        if ( wpinv_current_user_can_manage_invoicing() && isset( $_REQUEST['getpaid-admin-action'] ) && isset( $_REQUEST['getpaid-nonce'] ) && wp_verify_nonce( $_REQUEST['getpaid-nonce'], 'getpaid-nonce' ) ) {
-            $key = sanitize_key( $_REQUEST['getpaid-admin-action'] );
-            do_action( "getpaid_authenticated_admin_action_$key", $_REQUEST );
+        if (wpinv_current_user_can_manage_invoicing() && isset($_REQUEST['getpaid-admin-action']) && isset($_REQUEST['getpaid-nonce']) && wp_verify_nonce($_REQUEST['getpaid-nonce'], 'getpaid-nonce')) {
+            $key = sanitize_key($_REQUEST['getpaid-admin-action']);
+            do_action("getpaid_authenticated_admin_action_$key", $_REQUEST);
         }
 
     }
@@ -385,16 +385,16 @@ 
 	 * 
 	 * @param array $args
      */
-    public function send_customer_invoice( $args ) {
-		$sent = getpaid()->get( 'invoice_emails' )->user_invoice( new WPInv_Invoice( $args['invoice_id'] ), true );
+    public function send_customer_invoice($args) {
+		$sent = getpaid()->get('invoice_emails')->user_invoice(new WPInv_Invoice($args['invoice_id']), true);
 
-		if ( $sent ) {
-			$this->show_success( __( 'Invoice was successfully sent to the customer', 'invoicing' ) );
+		if ($sent) {
+			$this->show_success(__('Invoice was successfully sent to the customer', 'invoicing'));
 		} else {
-			$this->show_error( __( 'Could not send the invoice to the customer', 'invoicing' ) );
+			$this->show_error(__('Could not send the invoice to the customer', 'invoicing'));
 		}
 
-		wp_safe_redirect( remove_query_arg( array( 'getpaid-admin-action', 'getpaid-nonce', 'invoice_id' ) ) );
+		wp_safe_redirect(remove_query_arg(array('getpaid-admin-action', 'getpaid-nonce', 'invoice_id')));
 		exit;
 	}
 
@@ -403,16 +403,16 @@ 
 	 * 
 	 * @param array $args
      */
-    public function send_customer_payment_reminder( $args ) {
-		$sent = getpaid()->get( 'invoice_emails' )->force_send_overdue_notice( new WPInv_Invoice( $args['invoice_id'] ) );
+    public function send_customer_payment_reminder($args) {
+		$sent = getpaid()->get('invoice_emails')->force_send_overdue_notice(new WPInv_Invoice($args['invoice_id']));
 
-		if ( $sent ) {
-			$this->show_success( __( 'Payment reminder was successfully sent to the customer', 'invoicing' ) );
+		if ($sent) {
+			$this->show_success(__('Payment reminder was successfully sent to the customer', 'invoicing'));
 		} else {
-			$this->show_error( __( 'Could not sent payment reminder to the customer', 'invoicing' ) );
+			$this->show_error(__('Could not sent payment reminder to the customer', 'invoicing'));
 		}
 
-		wp_safe_redirect( remove_query_arg( array( 'getpaid-admin-action', 'getpaid-nonce', 'invoice_id' ) ) );
+		wp_safe_redirect(remove_query_arg(array('getpaid-admin-action', 'getpaid-nonce', 'invoice_id')));
 		exit;
 	}
 
@@ -422,8 +422,8 @@ 
      */
     public function admin_reset_tax_rates() {
 
-		update_option( 'wpinv_tax_rates', wpinv_get_data( 'tax-rates' ) );
-		wp_safe_redirect( remove_query_arg( array( 'getpaid-admin-action', 'getpaid-nonce' ) ) );
+		update_option('wpinv_tax_rates', wpinv_get_data('tax-rates'));
+		wp_safe_redirect(remove_query_arg(array('getpaid-admin-action', 'getpaid-nonce')));
 		exit;
 
 	}
@@ -435,8 +435,8 @@ 
     public function admin_create_missing_pages() {
 		$installer = new GetPaid_Installer();
 		$installer->create_pages();
-		$this->show_success( __( 'GetPaid pages updated.', 'invoicing' ) );
-		wp_safe_redirect( remove_query_arg( array( 'getpaid-admin-action', 'getpaid-nonce' ) ) );
+		$this->show_success(__('GetPaid pages updated.', 'invoicing'));
+		wp_safe_redirect(remove_query_arg(array('getpaid-admin-action', 'getpaid-nonce')));
 		exit;
 	}
 
@@ -448,35 +448,35 @@ 
 		global $wpdb;
 		$installer = new GetPaid_Installer();
 
-		if ( $wpdb->get_var( "SHOW TABLES LIKE '{$wpdb->prefix}wpinv_subscriptions'" ) != $wpdb->prefix . 'wpinv_subscriptions' ) {
+		if ($wpdb->get_var("SHOW TABLES LIKE '{$wpdb->prefix}wpinv_subscriptions'") != $wpdb->prefix . 'wpinv_subscriptions') {
 			$installer->create_subscriptions_table();
 
-			if ( $wpdb->last_error !== '' ) {
-				$this->show_error( __( 'Your GetPaid tables have been updated:', 'invoicing' ) . ' ' . $wpdb->last_error );
+			if ($wpdb->last_error !== '') {
+				$this->show_error(__('Your GetPaid tables have been updated:', 'invoicing') . ' ' . $wpdb->last_error);
 			}
 		}
 
-		if ( $wpdb->get_var( "SHOW TABLES LIKE '{$wpdb->prefix}getpaid_invoices'" ) != $wpdb->prefix . 'getpaid_invoices' ) {
+		if ($wpdb->get_var("SHOW TABLES LIKE '{$wpdb->prefix}getpaid_invoices'") != $wpdb->prefix . 'getpaid_invoices') {
 			$installer->create_invoices_table();
 
-			if ( $wpdb->last_error !== '' ) {
-				$this->show_error( __( 'Your GetPaid tables have been updated:', 'invoicing' ) . ' ' . $wpdb->last_error );
+			if ($wpdb->last_error !== '') {
+				$this->show_error(__('Your GetPaid tables have been updated:', 'invoicing') . ' ' . $wpdb->last_error);
 			}
 		}
 
-		if ( $wpdb->get_var( "SHOW TABLES LIKE '{$wpdb->prefix}getpaid_invoice_items'" ) != $wpdb->prefix . 'getpaid_invoice_items' ) {
+		if ($wpdb->get_var("SHOW TABLES LIKE '{$wpdb->prefix}getpaid_invoice_items'") != $wpdb->prefix . 'getpaid_invoice_items') {
 			$installer->create_invoice_items_table();
 
-			if ( $wpdb->last_error !== '' ) {
-				$this->show_error( __( 'Your GetPaid tables have been updated:', 'invoicing' ) . ' ' . $wpdb->last_error );
+			if ($wpdb->last_error !== '') {
+				$this->show_error(__('Your GetPaid tables have been updated:', 'invoicing') . ' ' . $wpdb->last_error);
 			}
 		}
 
-		if ( ! $this->has_notices() ) {
-			$this->show_success( __( 'Your GetPaid tables have been updated.', 'invoicing' ) );
+		if (!$this->has_notices()) {
+			$this->show_success(__('Your GetPaid tables have been updated.', 'invoicing'));
 		}
 
-		wp_safe_redirect( remove_query_arg( array( 'getpaid-admin-action', 'getpaid-nonce' ) ) );
+		wp_safe_redirect(remove_query_arg(array('getpaid-admin-action', 'getpaid-nonce')));
 		exit;
 	}
 
@@ -491,10 +491,10 @@ 
 		$installer->migrate_old_invoices();
 
 		// Show an admin message.
-		$this->show_success( __( 'Your invoices have been migrated.', 'invoicing' ) );
+		$this->show_success(__('Your invoices have been migrated.', 'invoicing'));
 
 		// Redirect the admin.
-		wp_safe_redirect( remove_query_arg( array( 'getpaid-admin-action', 'getpaid-nonce' ) ) );
+		wp_safe_redirect(remove_query_arg(array('getpaid-admin-action', 'getpaid-nonce')));
 		exit;
 
 	}
@@ -508,36 +508,36 @@ 
 
 		// Fetch all invoices that have discount codes.
 		$table    = $wpdb->prefix . 'getpaid_invoices';
-		$invoices = $wpdb->get_col( "SELECT `post_id` FROM `$table` WHERE `discount` = 0 && `discount_code` <> ''" );
+		$invoices = $wpdb->get_col("SELECT `post_id` FROM `$table` WHERE `discount` = 0 && `discount_code` <> ''");
 
-		foreach ( $invoices as $invoice ) {
+		foreach ($invoices as $invoice) {
 
-			$invoice = new WPInv_Invoice( $invoice );
+			$invoice = new WPInv_Invoice($invoice);
 
-			if ( ! $invoice->exists() ) {
+			if (!$invoice->exists()) {
 				continue;
 			}
 
 			// Abort if the discount does not exist or does not apply here.
-			$discount = new WPInv_Discount( $invoice->get_discount_code() );
-			if ( ! $discount->exists() ) {
+			$discount = new WPInv_Discount($invoice->get_discount_code());
+			if (!$discount->exists()) {
 				continue;
 			}
 
-			$invoice->add_discount( getpaid_calculate_invoice_discount( $invoice, $discount ) );
+			$invoice->add_discount(getpaid_calculate_invoice_discount($invoice, $discount));
 			$invoice->recalculate_total();
 
-			if ( $invoice->get_total_discount() > 0 ) {
+			if ($invoice->get_total_discount() > 0) {
 				$invoice->save();
 			}
 
 		}
 
 		// Show an admin message.
-		$this->show_success( __( 'Discounts have been recalculated.', 'invoicing' ) );
+		$this->show_success(__('Discounts have been recalculated.', 'invoicing'));
 
 		// Redirect the admin.
-		wp_safe_redirect( remove_query_arg( array( 'getpaid-admin-action', 'getpaid-nonce' ) ) );
+		wp_safe_redirect(remove_query_arg(array('getpaid-admin-action', 'getpaid-nonce')));
 		exit;
 
 	}
@@ -549,8 +549,8 @@ 
      * @return array
 	 */
 	public function get_notices() {
-		$notices = get_option( 'wpinv_admin_notices' );
-        return is_array( $notices ) ? $notices : array();
+		$notices = get_option('wpinv_admin_notices');
+        return is_array($notices) ? $notices : array();
 	}
 
 	/**
@@ -560,7 +560,7 @@ 
      * @return array
 	 */
 	public function has_notices() {
-		return count( $this->get_notices() ) > 0;
+		return count($this->get_notices()) > 0;
 	}
 
 	/**
@@ -570,7 +570,7 @@ 
 	 * @since       1.0.19
 	 */
 	public function clear_notices() {
-		delete_option( 'wpinv_admin_notices' );
+		delete_option('wpinv_admin_notices');
 	}
 
 	/**
@@ -579,16 +579,16 @@ 
 	 * @access      public
 	 * @since       1.0.19
 	 */
-	public function save_notice( $type, $message ) {
+	public function save_notice($type, $message) {
 		$notices = $this->get_notices();
 
-		if ( empty( $notices[ $type ] ) || ! is_array( $notices[ $type ]) ) {
-			$notices[ $type ] = array();
+		if (empty($notices[$type]) || !is_array($notices[$type])) {
+			$notices[$type] = array();
 		}
 
-		$notices[ $type ][] = $message;
+		$notices[$type][] = $message;
 
-		update_option( 'wpinv_admin_notices', $notices );
+		update_option('wpinv_admin_notices', $notices);
 	}
 
 	/**
@@ -598,8 +598,8 @@ 
 	 * @access      public
 	 * @since       1.0.19
 	 */
-	public function show_success( $msg ) {
-		$this->save_notice( 'success', $msg );
+	public function show_success($msg) {
+		$this->save_notice('success', $msg);
 	}
 
 	/**
@@ -609,8 +609,8 @@ 
 	 * @param       string $msg The message to qeue.
 	 * @since       1.0.19
 	 */
-	public function show_error( $msg ) {
-		$this->save_notice( 'error', $msg );
+	public function show_error($msg) {
+		$this->save_notice('error', $msg);
 	}
 
 	/**
@@ -620,8 +620,8 @@ 
 	 * @param       string $msg The message to qeue.
 	 * @since       1.0.19
 	 */
-	public function show_warning( $msg ) {
-		$this->save_notice( 'warning', $msg );
+	public function show_warning($msg) {
+		$this->save_notice('warning', $msg);
 	}
 
 	/**
@@ -631,8 +631,8 @@ 
 	 * @param       string $msg The message to qeue.
 	 * @since       1.0.19
 	 */
-	public function show_info( $msg ) {
-		$this->save_notice( 'info', $msg );
+	public function show_info($msg) {
+		$this->save_notice('info', $msg);
 	}
 
 	/**
@@ -646,32 +646,32 @@ 
         $notices = $this->get_notices();
         $this->clear_notices();
 
-		foreach ( $notices as $type => $messages ) {
+		foreach ($notices as $type => $messages) {
 
-			if ( ! is_array( $messages ) ) {
+			if (!is_array($messages)) {
 				continue;
 			}
 
-            $type  = sanitize_key( $type );
-			foreach ( $messages as $message ) {
-                $message = wp_kses_post( $message );
+            $type = sanitize_key($type);
+			foreach ($messages as $message) {
+                $message = wp_kses_post($message);
 				echo "<div class='notice notice-$type is-dismissible'><p>$message</p></div>";
             }
 
         }
 
-		foreach ( array( 'checkout_page', 'invoice_history_page', 'success_page', 'failure_page', 'invoice_subscription_page' ) as $page ) {
+		foreach (array('checkout_page', 'invoice_history_page', 'success_page', 'failure_page', 'invoice_subscription_page') as $page) {
 
-			if ( ! is_numeric( wpinv_get_option( $page, false ) ) ) {
-				$url     = esc_url(
+			if (!is_numeric(wpinv_get_option($page, false))) {
+				$url = esc_url(
 					wp_nonce_url(
-						add_query_arg( 'getpaid-admin-action', 'create_missing_pages' ),
+						add_query_arg('getpaid-admin-action', 'create_missing_pages'),
 						'getpaid-nonce',
 						'getpaid-nonce'
 					)
 				);
-				$message  = __( 'Some GetPaid pages are missing. To use GetPaid without any issues, click the button below to generate the missing pages.', 'invoicing' );
-				$message2 = __( 'Generate Pages', 'invoicing' );
+				$message  = __('Some GetPaid pages are missing. To use GetPaid without any issues, click the button below to generate the missing pages.', 'invoicing');
+				$message2 = __('Generate Pages', 'invoicing');
 				echo "<div class='notice notice-warning is-dismissible'><p>$message<br><br><a href='$url' class='button button-primary'>$message2</a></p></div>";
 				break;
 			}

includes/admin/meta-boxes/class-getpaid-meta-box-invoice-details.php

Spacing +55 added lines, -55 removed lines

@@ -7,7 +7,7 @@ 
  *
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if (!defined('ABSPATH')) {
 	exit; // Exit if accessed directly
 }
 
@@ -21,13 +21,13 @@ 
 	 *
 	 * @param WP_Post $post
 	 */
-    public static function output( $post ) {
+    public static function output($post) {
 
         // Prepare the invoice.
-        $invoice = new WPInv_Invoice( $post );
+        $invoice = new WPInv_Invoice($post);
 
         // Nonce field.
-        wp_nonce_field( 'wpinv_details', 'wpinv_details_nonce' ) ;
+        wp_nonce_field('wpinv_details', 'wpinv_details_nonce');
 
 
         ?>
@@ -46,11 +46,11 @@ 
 
                 <div class="bsui" style="margin-top: 1.5rem">
 
-                    <?php do_action( 'getpaid_invoice_edit_before_viewed_by_customer', $invoice ); ?>
-                    <?php if ( ! $invoice->is_draft() ) : ?>
+                    <?php do_action('getpaid_invoice_edit_before_viewed_by_customer', $invoice); ?>
+                    <?php if (!$invoice->is_draft()) : ?>
                         <div class="form-group">
-                            <strong><?php _e( 'Viewed by Customer:', 'invoicing' );?></strong>
-                            <?php ( $invoice->get_is_viewed() ) ? _e( 'Yes', 'invoicing' ) : _e( 'No', 'invoicing' ); ?>
+                            <strong><?php _e('Viewed by Customer:', 'invoicing'); ?></strong>
+                            <?php ($invoice->get_is_viewed()) ? _e('Yes', 'invoicing') : _e('No', 'invoicing'); ?>
                         </div>
                     <?php endif; ?>
 
@@ -58,13 +58,13 @@ 
 
                         // Date created.
                         $label = sprintf(
-                            __( '%s Date:', 'invoicing' ),
-                            ucfirst( $invoice->get_invoice_quote_type() )
+                            __('%s Date:', 'invoicing'),
+                            ucfirst($invoice->get_invoice_quote_type())
                         );
 
-                        $info  = sprintf(
-                            __( 'The date this %s was created.', 'invoicing' ),
-                            strtolower( $invoice->get_invoice_quote_type() )
+                        $info = sprintf(
+                            __('The date this %s was created.', 'invoicing'),
+                            strtolower($invoice->get_invoice_quote_type())
                         );
 
                         echo aui()->input(
@@ -72,11 +72,11 @@ 
                                 'type'        => 'datepicker',
                                 'id'          => 'wpinv_date_created',
                                 'name'        => 'date_created',
-                                'label'       => $label . getpaid_get_help_tip( $info ),
+                                'label'       => $label . getpaid_get_help_tip($info),
                                 'label_type'  => 'vertical',
                                 'placeholder' => 'YYYY-MM-DD 00:00',
                                 'class'       => 'form-control-sm',
-                                'value'       => $invoice->get_date_created( 'edit' ),
+                                'value'       => $invoice->get_date_created('edit'),
                                 'extra_attributes' => array(
                                     'data-enable-time' => 'true',
                                     'data-time_24hr'   => 'true',
@@ -87,15 +87,15 @@ 
                         );
 
                         // Date paid.
-                        $date_paid = $invoice->get_date_completed( 'edit' );
-                        if ( ! empty( $date_paid ) && $invoice->is_paid() ) {
+                        $date_paid = $invoice->get_date_completed('edit');
+                        if (!empty($date_paid) && $invoice->is_paid()) {
 
                             echo aui()->input(
                                 array(
                                     'type'        => 'text',
                                     'id'          => 'wpinv_date_completed',
                                     'name'        => 'wpinv_date_completed',
-                                    'label'       => __( 'Date Completed:', 'invoicing' ),
+                                    'label'       => __('Date Completed:', 'invoicing'),
                                     'label_type'  => 'vertical',
                                     'class'       => 'form-control-sm',
                                     'value'       => $date_paid,
@@ -109,18 +109,18 @@ 
                         }
 
                         // Due date.
-                        if ( $invoice->is_type( 'invoice' ) && wpinv_get_option( 'overdue_active' ) && ( ! $invoice->is_paid() || $invoice->is_draft() ) ) {
+                        if ($invoice->is_type('invoice') && wpinv_get_option('overdue_active') && (!$invoice->is_paid() || $invoice->is_draft())) {
 
                             echo aui()->input(
                                 array(
                                     'type'        => 'datepicker',
                                     'id'          => 'wpinv_due_date',
                                     'name'        => 'wpinv_due_date',
-                                    'label'       => __( 'Due Date:', 'invoicing' ) . getpaid_get_help_tip( __( 'Leave blank to disable automated reminder emails for this invoice.', 'invoicing' ) ),
+                                    'label'       => __('Due Date:', 'invoicing') . getpaid_get_help_tip(__('Leave blank to disable automated reminder emails for this invoice.', 'invoicing')),
                                     'label_type'  => 'vertical',
-                                    'placeholder' => __( 'No due date', 'invoicing' ),
+                                    'placeholder' => __('No due date', 'invoicing'),
                                     'class'       => 'form-control-sm',
-                                    'value'       => $invoice->get_due_date( 'edit' ),
+                                    'value'       => $invoice->get_due_date('edit'),
                                     'extra_attributes' => array(
                                         'data-enable-time' => 'true',
                                         'data-time_24hr'   => 'true',
@@ -132,39 +132,39 @@ 
 
                         }
 
-                        do_action( 'wpinv_meta_box_details_after_due_date', $invoice->get_id() );
-                        do_action( 'getpaid_metabox_after_due_date', $invoice );
+                        do_action('wpinv_meta_box_details_after_due_date', $invoice->get_id());
+                        do_action('getpaid_metabox_after_due_date', $invoice);
 
                         // Status.
                         $label = sprintf(
-                            __( '%s Status:', 'invoicing' ),
-                            ucfirst( $invoice->get_invoice_quote_type() )
+                            __('%s Status:', 'invoicing'),
+                            ucfirst($invoice->get_invoice_quote_type())
                         );
 
-                        $status = $invoice->get_status( 'edit' );
+                        $status = $invoice->get_status('edit');
                         echo aui()->select(
                             array(
                                 'id'               => 'wpinv_status',
                                 'name'             => 'wpinv_status',
                                 'label'            => $label,
                                 'label_type'       => 'vertical',
-                                'placeholder'      => __( 'Select Status', 'invoicing' ),
-                                'value'            => array_key_exists( $status, $invoice->get_all_statuses() ) ? $status : $invoice->get_default_status(),
+                                'placeholder'      => __('Select Status', 'invoicing'),
+                                'value'            => array_key_exists($status, $invoice->get_all_statuses()) ? $status : $invoice->get_default_status(),
                                 'select2'          => true,
                                 'data-allow-clear' => 'false',
-                                'options'          => wpinv_get_invoice_statuses( true, false, $invoice )
+                                'options'          => wpinv_get_invoice_statuses(true, false, $invoice)
                             )
                         );
 
                         // Invoice number.
                         $label = sprintf(
-                            __( '%s Number:', 'invoicing' ),
-                            ucfirst( $invoice->get_invoice_quote_type() )
+                            __('%s Number:', 'invoicing'),
+                            ucfirst($invoice->get_invoice_quote_type())
                         );
 
-                        $info  = sprintf(
-                            __( 'Each %s number must be unique.', 'invoicing' ),
-                            strtolower( $invoice->get_invoice_quote_type() )
+                        $info = sprintf(
+                            __('Each %s number must be unique.', 'invoicing'),
+                            strtolower($invoice->get_invoice_quote_type())
                         );
 
                         echo aui()->input(
@@ -172,11 +172,11 @@ 
                                 'type'        => 'text',
                                 'id'          => 'wpinv_number',
                                 'name'        => 'wpinv_number',
-                                'label'       => $label . getpaid_get_help_tip( $info ),
+                                'label'       => $label . getpaid_get_help_tip($info),
                                 'label_type'  => 'vertical',
-                                'placeholder' => __( 'Autogenerate', 'invoicing' ),
+                                'placeholder' => __('Autogenerate', 'invoicing'),
                                 'class'       => 'form-control-sm',
-                                'value'       => $invoice->get_number( 'edit' ),
+                                'value'       => $invoice->get_number('edit'),
                             )
                         );
 
@@ -186,15 +186,15 @@ 
                                 'type'        => 'text',
                                 'id'          => 'wpinv_cc',
                                 'name'        => 'wpinv_cc',
-                                'label'       => __( 'Email CC:', 'invoicing' ) . getpaid_get_help_tip( __( 'Enter a comma separated list of other emails that should be notified about the invoice.', 'invoicing' ) ),
+                                'label'       => __('Email CC:', 'invoicing') . getpaid_get_help_tip(__('Enter a comma separated list of other emails that should be notified about the invoice.', 'invoicing')),
                                 'label_type'  => 'vertical',
-                                'placeholder' => __( 'example@gmail.com, example@yahoo.com', 'invoicing' ),
+                                'placeholder' => __('example@gmail.com, example@yahoo.com', 'invoicing'),
                                 'class'       => 'form-control-sm',
-                                'value'       => $invoice->get_email_cc( 'edit' ),
+                                'value'       => $invoice->get_email_cc('edit'),
                             )
                         );
 
-                        if ( ! $invoice->is_paid() && ! $invoice->is_refunded() ) {
+                        if (!$invoice->is_paid() && !$invoice->is_refunded()) {
 
                             // Apply a discount.
                             echo aui()->input(
@@ -202,25 +202,25 @@ 
                                     'type'        => 'text',
                                     'id'          => 'wpinv_discount_code',
                                     'name'        => 'wpinv_discount_code',
-                                    'label'       => __( 'Discount Code:', 'invoicing' ),
-                                    'placeholder' => __( 'Apply Discount', 'invoicing' ),
+                                    'label'       => __('Discount Code:', 'invoicing'),
+                                    'placeholder' => __('Apply Discount', 'invoicing'),
                                     'label_type'  => 'vertical',
                                     'class'       => 'form-control-sm',
-                                    'value'       => $invoice->get_discount_code( 'edit' ),
+                                    'value'       => $invoice->get_discount_code('edit'),
                                 )
                             );
 
-                        } else if ( $invoice->get_discount_code( 'edit' ) ) {
+                        } else if ($invoice->get_discount_code('edit')) {
 
                             echo aui()->input(
                                 array(
                                     'type'        => 'text',
                                     'id'          => 'wpinv_discount_code',
                                     'name'        => 'wpinv_discount_code',
-                                    'label'       => __( 'Discount Code:', 'invoicing' ),
+                                    'label'       => __('Discount Code:', 'invoicing'),
                                     'label_type'  => 'vertical',
                                     'class'       => 'form-control-sm',
-                                    'value'       => $invoice->get_discount_code( 'edit' ),
+                                    'value'       => $invoice->get_discount_code('edit'),
                                     'extra_attributes' => array(
                                         'onclick'  => 'this.select();',
                                         'readonly' => 'true',
@@ -230,17 +230,17 @@ 
 
                         }
 
-                        do_action( 'wpinv_meta_box_details_inner', $invoice->get_id() );
+                        do_action('wpinv_meta_box_details_inner', $invoice->get_id());
 
                         // Disable taxes.
-                        if ( wpinv_use_taxes() && ! ( $invoice->is_paid() || $invoice->is_refunded() ) ) {
+                        if (wpinv_use_taxes() && !($invoice->is_paid() || $invoice->is_refunded())) {
 
                             echo aui()->input(
                                 array(
                                     'id'          => 'wpinv_taxable',
                                     'name'        => 'disable_taxes',
                                     'type'        => 'checkbox',
-                                    'label'       => __( 'Disable taxes', 'invoicing' ),
+                                    'label'       => __('Disable taxes', 'invoicing'),
                                     'value'       => '1',
                                     'checked'     => (bool) $invoice->get_disable_taxes(),
                                 )
@@ -248,7 +248,7 @@ 
 
                         }
 
-                        if ( $invoice->is_type( 'invoice' ) ) {
+                        if ($invoice->is_type('invoice')) {
 
                             // Send to customer.
                             echo aui()->input(
@@ -256,15 +256,15 @@ 
                                     'id'          => 'wpinv_send_to_customer',
                                     'name'        => 'send_to_customer',
                                     'type'        => 'checkbox',
-                                    'label'       => __( 'Send invoice to customer after saving', 'invoicing' ),
+                                    'label'       => __('Send invoice to customer after saving', 'invoicing'),
                                     'value'       => '1',
-                                    'checked'     => $invoice->is_draft() && (bool) wpinv_get_option( 'email_user_invoice_active', true ),
+                                    'checked'     => $invoice->is_draft() && (bool) wpinv_get_option('email_user_invoice_active', true),
                                 )
                             );
 
                         }
 
-                        do_action( 'getpaid_metabox_after_invoice_details', $invoice );
+                        do_action('getpaid_metabox_after_invoice_details', $invoice);
 
                     ?>
 

includes/class-getpaid-template.php

Spacing +46 added lines, -46 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);
 		}
 
     }
@@ -60,7 +60,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');
 	}
 
 	/**
@@ -70,7 +70,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');
 	}
 
 	/**
@@ -80,7 +80,7 @@ 
 	 * @return bool
 	 */
 	public function is_beaver_preview() {
-		return isset( $_REQUEST['fl_builder'] );
+		return isset($_REQUEST['fl_builder']);
 	}
 
 	/**
@@ -90,7 +90,7 @@ 
 	 * @return bool
 	 */
 	public function is_siteorigin_preview() {
-		return ! empty( $_REQUEST['siteorigin_panels_live_editor'] );
+		return !empty($_REQUEST['siteorigin_panels_live_editor']);
 	}
 
 	/**
@@ -100,7 +100,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';
 	}
 
 	/**
@@ -110,7 +110,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']);
 	}
 
 	/**
@@ -120,7 +120,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_"));
     }
 
     /**
@@ -130,24 +130,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);
     }
     
     /**
@@ -156,20 +156,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 );
+	    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);
 
     }
 
@@ -183,18 +183,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);
 
     }
     
@@ -208,9 +208,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();
     }
 
@@ -222,9 +222,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());
 
 	}
 
@@ -234,28 +234,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;
@@ -268,10 +268,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;
 		}
 

templates/invoice/invoice-logo.php

Spacing +9 added lines, -9 removed lines

@@ -7,24 +7,24 @@
  * @version 1.0.19
  */
 
-defined( 'ABSPATH' ) || exit;
+defined('ABSPATH') || exit;
 
-$logo_width  = wpinv_get_option( 'logo_width' );
-$logo_height = wpinv_get_option( 'logo_height' );
+$logo_width  = wpinv_get_option('logo_width');
+$logo_height = wpinv_get_option('logo_height');
 
 ?>
-<a target="_blank" class="logo-link text-dark" href="<?php echo esc_url( wpinv_get_business_website() ); ?>">
+<a target="_blank" class="logo-link text-dark" href="<?php echo esc_url(wpinv_get_business_website()); ?>">
 
-    <?php if ( $logo = wpinv_get_business_logo() ) : ?>
+    <?php if ($logo = wpinv_get_business_logo()) : ?>
 
-        <?php if ( ! empty( $logo_width ) && ! empty( $logo_height ) ) : ?>
-            <img class="logo" style="max-width:100%; width:<?php echo absint( $logo_width ); ?>px; height:<?php echo absint( $logo_height ); ?>px;" src="<?php echo esc_url( $logo ); ?>">
+        <?php if (!empty($logo_width) && !empty($logo_height)) : ?>
+            <img class="logo" style="max-width:100%; width:<?php echo absint($logo_width); ?>px; height:<?php echo absint($logo_height); ?>px;" src="<?php echo esc_url($logo); ?>">
         <?php else: ?>
-            <img class="logo" style="max-width:100%;" src="<?php echo esc_url( $logo ); ?>">
+            <img class="logo" style="max-width:100%;" src="<?php echo esc_url($logo); ?>">
         <?php endif; ?>
 
     <?php else: ?>
-        <h1 class="h3"><?php echo esc_html( wpinv_get_business_name() ); ?></h1>
+        <h1 class="h3"><?php echo esc_html(wpinv_get_business_name()); ?></h1>
     <?php endif; ?>
 
 </a>

includes/wpinv-tax-functions.php

Spacing +120 added lines, -120 removed lines

@@ -4,7 +4,7 @@ 
  *
  */
 
-defined( 'ABSPATH' ) || exit;
+defined('ABSPATH') || exit;
 
 /**
  * Returns an array of eu states.
@@ -12,7 +12,7 @@ 
  * @return array
  */
 function getpaid_get_eu_states() {
-    return wpinv_get_data( 'eu-states' );
+    return wpinv_get_data('eu-states');
 }
 
 /**
@@ -20,8 +20,8 @@ 
  * 
  * @return bool
  */
-function getpaid_is_eu_state( $country ) {
-    return ! empty( $country ) && in_array( strtoupper( $country ), getpaid_get_eu_states() ) ? true : false;
+function getpaid_is_eu_state($country) {
+    return !empty($country) && in_array(strtoupper($country), getpaid_get_eu_states()) ? true : false;
 }
 
 /**
@@ -30,7 +30,7 @@ 
  * @return array
  */
 function getpaid_get_gst_states() {
-    return array( 'AU', 'NZ', 'CA', 'CN' );
+    return array('AU', 'NZ', 'CA', 'CN');
 }
 
 /**
@@ -38,8 +38,8 @@ 
  * 
  * @return bool
  */
-function getpaid_is_gst_country( $country ) {
-    return ! empty( $country ) && in_array( strtoupper( $country ), getpaid_get_gst_states() ) ? true : false;
+function getpaid_is_gst_country($country) {
+    return !empty($country) && in_array(strtoupper($country), getpaid_get_gst_states()) ? true : false;
 }
 
 /**
@@ -49,8 +49,8 @@ 
  */
 function wpinv_use_taxes() {
 
-    $ret = wpinv_get_option( 'enable_taxes', false );
-    return (bool) apply_filters( 'wpinv_use_taxes', ! empty( $ret ) );
+    $ret = wpinv_get_option('enable_taxes', false);
+    return (bool) apply_filters('wpinv_use_taxes', !empty($ret));
 
 }
 
@@ -60,7 +60,7 @@ 
  * @param WPInv_Invoice $invoice
  * @return bool
  */
-function wpinv_is_invoice_taxable( $invoice ) {
+function wpinv_is_invoice_taxable($invoice) {
     return $invoice->is_taxable();
 }
 
@@ -70,11 +70,11 @@ 
  * @param string $country
  * @return bool
  */
-function wpinv_is_country_taxable( $country ) {
-    $is_eu     = getpaid_is_eu_state( $country );
-    $is_exempt = ! $is_eu && wpinv_is_base_country( $country ) && wpinv_same_country_exempt_vat();
+function wpinv_is_country_taxable($country) {
+    $is_eu     = getpaid_is_eu_state($country);
+    $is_exempt = !$is_eu && wpinv_is_base_country($country) && wpinv_same_country_exempt_vat();
 
-    return (bool) apply_filters( 'wpinv_is_country_taxable', ! $is_exempt, $country ); 
+    return (bool) apply_filters('wpinv_is_country_taxable', !$is_exempt, $country); 
 
 }
 
@@ -84,7 +84,7 @@ 
  * @param WPInv_Item|GetPaid_Form_Item $item
  * @return bool
  */
-function wpinv_is_item_taxable( $item ) {
+function wpinv_is_item_taxable($item) {
     return '_exempt' != $item->get_vat_rule();
 }
 
@@ -94,8 +94,8 @@ 
  * @return bool
  */
 function wpinv_use_store_address_as_tax_base() {
-    $use_base = wpinv_get_option( 'tax_base', 'billing' ) == 'base';
-    return (bool) apply_filters( 'wpinv_use_store_address_as_tax_base', $use_base );
+    $use_base = wpinv_get_option('tax_base', 'billing') == 'base';
+    return (bool) apply_filters('wpinv_use_store_address_as_tax_base', $use_base);
 }
 
 /**
@@ -104,8 +104,8 @@ 
  * @return bool
  */
 function wpinv_prices_include_tax() {
-    $is_inclusive = wpinv_get_option( 'prices_include_tax', 'no' ) == 'yes';
-    return (bool) apply_filters( 'wpinv_prices_include_tax', $is_inclusive );
+    $is_inclusive = wpinv_get_option('prices_include_tax', 'no') == 'yes';
+    return (bool) apply_filters('wpinv_prices_include_tax', $is_inclusive);
 }
 
 /**
@@ -114,8 +114,8 @@ 
  * @return bool
  */
 function wpinv_round_tax_per_tax_rate() {
-    $subtotal_rounding = wpinv_get_option( 'tax_subtotal_rounding', 1 );
-    return (bool) apply_filters( 'wpinv_round_tax_per_tax_rate', empty( $subtotal_rounding ) );
+    $subtotal_rounding = wpinv_get_option('tax_subtotal_rounding', 1);
+    return (bool) apply_filters('wpinv_round_tax_per_tax_rate', empty($subtotal_rounding));
 }
 
 /**
@@ -124,8 +124,8 @@ 
  * @return bool
  */
 function wpinv_display_individual_tax_rates() {
-    $individual = wpinv_get_option( 'tax_display_totals', 'single' ) == 'individual';
-    return (bool) apply_filters( 'wpinv_display_individual_tax_rates', $individual );
+    $individual = wpinv_get_option('tax_display_totals', 'single') == 'individual';
+    return (bool) apply_filters('wpinv_display_individual_tax_rates', $individual);
 }
 
 /**
@@ -134,8 +134,8 @@ 
  * @return float
  */
 function wpinv_get_default_tax_rate() {
-    $rate = wpinv_get_option( 'tax_rate', 0 );
-    return (float) apply_filters( 'wpinv_get_default_tax_rate', floatval( $rate ) );
+    $rate = wpinv_get_option('tax_rate', 0);
+    return (float) apply_filters('wpinv_get_default_tax_rate', floatval($rate));
 }
 
 /**
@@ -144,7 +144,7 @@ 
  * @return bool
  */
 function wpinv_same_country_exempt_vat() {
-    return 'no' == wpinv_get_option( 'vat_same_country_rule', 'vat_too' );
+    return 'no' == wpinv_get_option('vat_same_country_rule', 'vat_too');
 }
 
 /**
@@ -164,28 +164,28 @@ 
  * @param string $state
  * @return array
  */
-function getpaid_get_item_tax_rates( $item, $country = '', $state = '' ) {
+function getpaid_get_item_tax_rates($item, $country = '', $state = '') {
 
     // Abort if the item is not taxable.
-    if ( ! wpinv_is_item_taxable( $item ) ) {
+    if (!wpinv_is_item_taxable($item)) {
         return array();
     }
 
     // Maybe use the store address.
-    if ( wpinv_use_store_address_as_tax_base() ) {
+    if (wpinv_use_store_address_as_tax_base()) {
         $country = wpinv_get_default_country();
         $state   = wpinv_get_default_state();
     }
 
     // Retrieve tax rates.
-    $tax_rates = GetPaid_Tax::get_address_tax_rates( $country, $state );
+    $tax_rates = GetPaid_Tax::get_address_tax_rates($country, $state);
 
     // Fallback to the default tax rates if non were found.
-    if ( empty( $tax_rates ) ) {
+    if (empty($tax_rates)) {
         $tax_rates = GetPaid_Tax::get_default_tax_rates();
     }
 
-    return apply_filters( 'getpaid_get_item_tax_rates', $tax_rates, $item, $country, $state );
+    return apply_filters('getpaid_get_item_tax_rates', $tax_rates, $item, $country, $state);
 }
 
 /**
@@ -195,23 +195,23 @@ 
  * @param array $rates
  * @return array
  */
-function getpaid_filter_item_tax_rates( $item, $rates ) {
+function getpaid_filter_item_tax_rates($item, $rates) {
 
     $tax_class = $item->get_vat_class();
 
-    foreach ( $rates as $i => $rate ) {
+    foreach ($rates as $i => $rate) {
 
-        if ( $tax_class == '_reduced' ) {
-            $rates[ $i ]['rate'] = empty( $rate['reduced_rate'] ) ? 0 : $rate['reduced_rate'];
+        if ($tax_class == '_reduced') {
+            $rates[$i]['rate'] = empty($rate['reduced_rate']) ? 0 : $rate['reduced_rate'];
         }
 
-        if ( $tax_class == '_exempt' ) {
-            $rates[ $i ]['rate'] = 0;
+        if ($tax_class == '_exempt') {
+            $rates[$i]['rate'] = 0;
         }
 
     }
 
-    return apply_filters( 'getpaid_filter_item_tax_rates', $rates, $item );
+    return apply_filters('getpaid_filter_item_tax_rates', $rates, $item);
 }
 
 /**
@@ -221,12 +221,12 @@ 
  * @param array $rates
  * @return array
  */
-function getpaid_calculate_item_taxes( $amount, $rates ) {
+function getpaid_calculate_item_taxes($amount, $rates) {
 
     $is_inclusive = wpinv_prices_include_tax();
-    $taxes        = GetPaid_Tax::calc_tax( $amount, $rates, $is_inclusive );
+    $taxes        = GetPaid_Tax::calc_tax($amount, $rates, $is_inclusive);
 
-    return apply_filters( 'getpaid_calculate_taxes', $taxes, $amount, $rates );
+    return apply_filters('getpaid_calculate_taxes', $taxes, $amount, $rates);
 }
 
 /**
@@ -238,17 +238,17 @@ 
  * @param float $recurring_tax_amount
  * @return array
  */
-function getpaid_prepare_item_tax( $item, $tax_name, $tax_amount, $recurring_tax_amount ) {
+function getpaid_prepare_item_tax($item, $tax_name, $tax_amount, $recurring_tax_amount) {
 
-    $initial_tax   = $tax_amount;
+    $initial_tax = $tax_amount;
 	$recurring_tax = 0;
 
-    if ( $item->is_recurring() ) {
+    if ($item->is_recurring()) {
 		$recurring_tax = $recurring_tax_amount;
 	}
 
 	return array(
-		'name'          => sanitize_text_field( $tax_name ),
+		'name'          => sanitize_text_field($tax_name),
 		'initial_tax'   => $initial_tax,
 		'recurring_tax' => $recurring_tax,
     );
@@ -261,8 +261,8 @@ 
  * @param string $vat_number
  * @return string
  */
-function wpinv_sanitize_vat_number( $vat_number ) {
-    return str_replace( array(' ', '.', '-', '_', ',' ), '', strtoupper( trim( $vat_number ) ) );
+function wpinv_sanitize_vat_number($vat_number) {
+    return str_replace(array(' ', '.', '-', '_', ','), '', strtoupper(trim($vat_number)));
 }
 
 /**
@@ -271,22 +271,22 @@ 
  * @param string $vat_number
  * @return bool
  */
-function wpinv_regex_validate_vat_number( $vat_number ) {
+function wpinv_regex_validate_vat_number($vat_number) {
 
-    $country    = substr( $vat_number, 0, 2 );
-    $vatin      = substr( $vat_number, 2 );
-    $regexes    = wpinv_get_data( 'vat-number-regexes' );
+    $country    = substr($vat_number, 0, 2);
+    $vatin      = substr($vat_number, 2);
+    $regexes    = wpinv_get_data('vat-number-regexes');
 
-    if ( isset( $regexes[ $country ] ) ) {
+    if (isset($regexes[$country])) {
 
-        $regex = $regexes[ $country ];
+        $regex = $regexes[$country];
         $regex = '/^(?:' . $regex . ')$/';
-        return 1 === preg_match( $regex, $vatin );
+        return 1 === preg_match($regex, $vatin);
 
     }
 
     // Not an EU state, use filters to validate the number.
-    return apply_filters( 'wpinv_regex_validate_vat_number', true, $vat_number );
+    return apply_filters('wpinv_regex_validate_vat_number', true, $vat_number);
 }
 
 /**
@@ -295,29 +295,29 @@ 
  * @param string $vat_number
  * @return bool
  */
-function wpinv_vies_validate_vat_number( $vat_number ) {
+function wpinv_vies_validate_vat_number($vat_number) {
 
-    $country    = substr( $vat_number, 0, 2 );
-    $vatin      = substr( $vat_number, 2 );
+    $country    = substr($vat_number, 0, 2);
+    $vatin      = substr($vat_number, 2);
 
     $url        = add_query_arg(
         array(
-            'ms'  => urlencode( $country ),
-            'iso' => urlencode( $country ),
-            'vat' => urlencode( $vatin ),
+            'ms'  => urlencode($country),
+            'iso' => urlencode($country),
+            'vat' => urlencode($vatin),
         ),
         'http://ec.europa.eu/taxation_customs/vies/viesquer.do'
     );
 
-    $response   = wp_remote_get( $url );
-    $response   = wp_remote_retrieve_body( $response );
+    $response   = wp_remote_get($url);
+    $response   = wp_remote_retrieve_body($response);
 
     // Fallback gracefully if the VIES website is down.
-    if ( empty( $response ) ) {
+    if (empty($response)) {
         return true;
     }
 
-    return 1 !== preg_match( '/invalid VAT number/i', $response );
+    return 1 !== preg_match('/invalid VAT number/i', $response);
 
 }
 
@@ -328,18 +328,18 @@ 
  * @param string $country
  * @return bool
  */
-function wpinv_validate_vat_number( $vat_number, $country ) {
+function wpinv_validate_vat_number($vat_number, $country) {
 
     // In case the vat number does not have a country code...
-    $vat_number = wpinv_sanitize_vat_number( $vat_number );
-    $_country   = substr( $vat_number, 0, 2 );
-    $_country   = $_country == wpinv_country_name( $_country );
+    $vat_number = wpinv_sanitize_vat_number($vat_number);
+    $_country   = substr($vat_number, 0, 2);
+    $_country   = $_country == wpinv_country_name($_country);
 
-    if ( $_country ) {
-        $vat_number = strtoupper( $country ) . $vat_number;
+    if ($_country) {
+        $vat_number = strtoupper($country) . $vat_number;
     }
 
-    return wpinv_regex_validate_vat_number( $vat_number ) && wpinv_vies_validate_vat_number( $vat_number );
+    return wpinv_regex_validate_vat_number($vat_number) && wpinv_vies_validate_vat_number($vat_number);
 }
 
 /**
@@ -348,40 +348,40 @@ 
  * @return bool
  */
 function wpinv_should_validate_vat_number() {
-    $validate = wpinv_get_option( 'validate_vat_number' );
-	return ! empty( $validate );
+    $validate = wpinv_get_option('validate_vat_number');
+	return !empty($validate);
 }
 
-function wpinv_sales_tax_for_year( $year = null ) {
-    return wpinv_price( wpinv_get_sales_tax_for_year( $year ) );
+function wpinv_sales_tax_for_year($year = null) {
+    return wpinv_price(wpinv_get_sales_tax_for_year($year));
 }
 
-function wpinv_get_sales_tax_for_year( $year = null ) {
+function wpinv_get_sales_tax_for_year($year = null) {
     global $wpdb;
 
     // Start at zero
     $tax = 0;
 
-    if ( ! empty( $year ) ) {
+    if (!empty($year)) {
         $args = array(
             'post_type'      => 'wpi_invoice',
-            'post_status'    => array( 'publish' ),
+            'post_status'    => array('publish'),
             'posts_per_page' => -1,
             'year'           => $year,
             'fields'         => 'ids'
         );
 
-        $payments    = get_posts( $args );
-        $payment_ids = implode( ',', $payments );
+        $payments    = get_posts($args);
+        $payment_ids = implode(',', $payments);
 
-        if ( count( $payments ) > 0 ) {
+        if (count($payments) > 0) {
             $sql = "SELECT SUM( meta_value ) FROM $wpdb->postmeta WHERE meta_key = '_wpinv_tax' AND post_id IN( $payment_ids )";
-            $tax = $wpdb->get_var( $sql );
+            $tax = $wpdb->get_var($sql);
         }
 
     }
 
-    return apply_filters( 'wpinv_get_sales_tax_for_year', $tax, $year );
+    return apply_filters('wpinv_get_sales_tax_for_year', $tax, $year);
 }
 
 function wpinv_is_cart_taxed() {
@@ -390,33 +390,33 @@ 
 
 function wpinv_prices_show_tax_on_checkout() {
     return false; // TODO
-    $ret = ( wpinv_get_option( 'checkout_include_tax', false ) == 'yes' && wpinv_use_taxes() );
+    $ret = (wpinv_get_option('checkout_include_tax', false) == 'yes' && wpinv_use_taxes());
 
-    return apply_filters( 'wpinv_taxes_on_prices_on_checkout', $ret );
+    return apply_filters('wpinv_taxes_on_prices_on_checkout', $ret);
 }
 
 function wpinv_display_tax_rate() {
-    $ret = wpinv_use_taxes() && wpinv_get_option( 'display_tax_rate', false );
+    $ret = wpinv_use_taxes() && wpinv_get_option('display_tax_rate', false);
 
-    return apply_filters( 'wpinv_display_tax_rate', $ret );
+    return apply_filters('wpinv_display_tax_rate', $ret);
 }
 
 function wpinv_cart_needs_tax_address_fields() {
-    if( !wpinv_is_cart_taxed() )
+    if (!wpinv_is_cart_taxed())
         return false;
 
-    return ! did_action( 'wpinv_after_cc_fields', 'wpinv_default_cc_address_fields' );
+    return !did_action('wpinv_after_cc_fields', 'wpinv_default_cc_address_fields');
 }
 
-function wpinv_item_is_tax_exclusive( $item_id = 0 ) {
-    $ret = (bool)get_post_meta( $item_id, '_wpinv_tax_exclusive', false );
-    return apply_filters( 'wpinv_is_tax_exclusive', $ret, $item_id );
+function wpinv_item_is_tax_exclusive($item_id = 0) {
+    $ret = (bool) get_post_meta($item_id, '_wpinv_tax_exclusive', false);
+    return apply_filters('wpinv_is_tax_exclusive', $ret, $item_id);
 }
 
-function wpinv_currency_decimal_filter( $decimals = 2 ) {
+function wpinv_currency_decimal_filter($decimals = 2) {
     $currency = wpinv_get_currency();
 
-    switch ( $currency ) {
+    switch ($currency) {
         case 'RIAL' :
         case 'JPY' :
         case 'TWD' :
@@ -425,13 +425,13 @@ 
             break;
     }
 
-    return apply_filters( 'wpinv_currency_decimal_count', $decimals, $currency );
+    return apply_filters('wpinv_currency_decimal_count', $decimals, $currency);
 }
 
 function wpinv_tax_amount() {
     $output = 0.00;
     
-    return apply_filters( 'wpinv_tax_amount', $output );
+    return apply_filters('wpinv_tax_amount', $output);
 }
 
 /**
@@ -439,25 +439,25 @@ 
  * 
  * @param string|bool|null $vat_rule
  */
-function getpaid_filter_vat_rule( $vat_rule ) {
+function getpaid_filter_vat_rule($vat_rule) {
 
-    if ( empty( $vat_rule ) ) {        
+    if (empty($vat_rule)) {        
         return 'digital';
     }
 
     return $vat_rule;
 }
-add_filter( 'wpinv_get_item_vat_rule', 'getpaid_filter_vat_rule' );
+add_filter('wpinv_get_item_vat_rule', 'getpaid_filter_vat_rule');
 
 /**
  * Filters the VAT class to ensure that each item has a VAT class.
  * 
  * @param string|bool|null $vat_rule
  */
-function getpaid_filter_vat_class( $vat_class ) {
-    return empty( $vat_class ) ? '_standard' : $vat_class;
+function getpaid_filter_vat_class($vat_class) {
+    return empty($vat_class) ? '_standard' : $vat_class;
 }
-add_filter( 'wpinv_get_item_vat_class', 'getpaid_filter_vat_class' );
+add_filter('wpinv_get_item_vat_class', 'getpaid_filter_vat_class');
 
 /**
  * Returns a list of all tax classes.
@@ -469,9 +469,9 @@ 
     return apply_filters(
         'getpaid_tax_classes',
         array(
-            '_standard' => __( 'Standard Tax Rate', 'invoicing' ),
-            '_reduced'  => __( 'Reduced Tax Rate', 'invoicing' ),
-            '_exempt'   => __( 'Tax Exempt', 'invoicing' ),
+            '_standard' => __('Standard Tax Rate', 'invoicing'),
+            '_reduced'  => __('Reduced Tax Rate', 'invoicing'),
+            '_exempt'   => __('Tax Exempt', 'invoicing'),
         )
     );
 
@@ -487,8 +487,8 @@ 
     return apply_filters(
         'getpaid_tax_rules',
         array(
-            'physical' => __( 'Physical Item', 'invoicing' ),
-            'digital'  => __( 'Digital Item', 'invoicing' ),
+            'physical' => __('Physical Item', 'invoicing'),
+            'digital'  => __('Digital Item', 'invoicing'),
         )
     );
 
@@ -500,15 +500,15 @@ 
  * @param string $tax_class
  * @return string
  */
-function getpaid_get_tax_class_label( $tax_class ) {
+function getpaid_get_tax_class_label($tax_class) {
 
     $classes = getpaid_get_tax_classes();
 
-    if ( isset( $classes[ $tax_class ] ) ) {
-        return sanitize_text_field( $classes[ $tax_class ] );
+    if (isset($classes[$tax_class])) {
+        return sanitize_text_field($classes[$tax_class]);
     }
 
-    return sanitize_text_field( $tax_class );
+    return sanitize_text_field($tax_class);
 
 }
 
@@ -518,15 +518,15 @@ 
  * @param string $tax_rule
  * @return string
  */
-function getpaid_get_tax_rule_label( $tax_rule ) {
+function getpaid_get_tax_rule_label($tax_rule) {
 
     $rules = getpaid_get_tax_rules();
 
-    if ( isset( $rules[ $tax_rule ] ) ) {
-        return sanitize_text_field( $rules[ $tax_rule ] );
+    if (isset($rules[$tax_rule])) {
+        return sanitize_text_field($rules[$tax_rule]);
     }
 
-    return sanitize_text_field( $tax_rule );
+    return sanitize_text_field($tax_rule);
 
 }
 
@@ -537,11 +537,11 @@ 
  * @param string $recurring
  * @return string
  */
-function getpaid_get_taxable_amount( $item, $recurring = false ) {
+function getpaid_get_taxable_amount($item, $recurring = false) {
 
     $taxable_amount  = $recurring ? $item->get_recurring_sub_total() : $item->get_sub_total();
     $taxable_amount -= $recurring ? $item->recurring_item_discount : $item->item_discount;
-    $taxable_amount  = max( 0, $taxable_amount );
-    return apply_filters( 'getpaid_taxable_amount', $taxable_amount, $item, $recurring );
+    $taxable_amount  = max(0, $taxable_amount);
+    return apply_filters('getpaid_taxable_amount', $taxable_amount, $item, $recurring);
 
 }

includes/class-wpinv-discount.php

Spacing +288 added lines, -288 removed lines

@@ -5,7 +5,7 @@ 
  * @since   1.0.15
  */
 
-defined( 'ABSPATH' ) || exit;
+defined('ABSPATH') || exit;
 
 /**
  * Discount class.
@@ -13,7 +13,7 @@ 
  * @since 1.0.15
  *
  */
-class WPInv_Discount extends GetPaid_Data  {
+class WPInv_Discount extends GetPaid_Data {
 
 	/**
 	 * Which data store to load.
@@ -79,35 +79,35 @@ 
 	 *
 	 * @param int|array|string|WPInv_Discount|WP_Post $discount discount data, object, ID or code.
 	 */
-	public function __construct( $discount = 0 ) {
-		parent::__construct( $discount );
+	public function __construct($discount = 0) {
+		parent::__construct($discount);
 
-		if ( is_numeric( $discount ) && 'wpi_discount' === get_post_type( $discount ) ) {
-			$this->set_id( $discount );
-		} elseif ( $discount instanceof self ) {
-			$this->set_id( $discount->get_id() );
-		} elseif ( ! empty( $discount->ID ) ) {
-			$this->set_id( $discount->ID );
-		} elseif ( is_array( $discount ) ) {
-			$this->set_props( $discount );
+		if (is_numeric($discount) && 'wpi_discount' === get_post_type($discount)) {
+			$this->set_id($discount);
+		} elseif ($discount instanceof self) {
+			$this->set_id($discount->get_id());
+		} elseif (!empty($discount->ID)) {
+			$this->set_id($discount->ID);
+		} elseif (is_array($discount)) {
+			$this->set_props($discount);
 
-			if ( isset( $discount['ID'] ) ) {
-				$this->set_id( $discount['ID'] );
+			if (isset($discount['ID'])) {
+				$this->set_id($discount['ID']);
 			}
 
-		} elseif ( is_scalar( $discount ) && $discount = self::get_discount_id_by_code( $discount ) ) {
-			$this->set_id( $discount );
+		} elseif (is_scalar($discount) && $discount = self::get_discount_id_by_code($discount)) {
+			$this->set_id($discount);
 		} else {
-			$this->set_object_read( true );
+			$this->set_object_read(true);
 		}
 
         // Load the datastore.
-		$this->data_store = GetPaid_Data_Store::load( $this->data_store_name );
+		$this->data_store = GetPaid_Data_Store::load($this->data_store_name);
 
-		if ( $this->get_id() > 0 ) {
-            $this->post = get_post( $this->get_id() );
+		if ($this->get_id() > 0) {
+            $this->post = get_post($this->get_id());
             $this->ID   = $this->get_id();
-			$this->data_store->read( $this );
+			$this->data_store->read($this);
         }
 
 	}
@@ -123,50 +123,50 @@ 
 	 * @since 1.0.15
 	 * @return array|bool array of discount details on success. False otherwise.
 	 */
-	public static function get_data_by( $field, $value ) {
+	public static function get_data_by($field, $value) {
 
-		if ( 'id' == strtolower( $field ) ) {
+		if ('id' == strtolower($field)) {
 			// Make sure the value is numeric to avoid casting objects, for example,
 			// to int 1.
-			if ( ! is_numeric( $value ) )
+			if (!is_numeric($value))
 				return false;
-			$value = intval( $value );
-			if ( $value < 1 )
+			$value = intval($value);
+			if ($value < 1)
 				return false;
 		}
 
-		if ( ! $value || ! is_string( $field ) ) {
+		if (!$value || !is_string($field)) {
 			return false;
 		}
 
-		$field = trim( $field );
+		$field = trim($field);
 
 		// prepare query args
-		switch ( strtolower( $field ) ) {
+		switch (strtolower($field)) {
 			case 'id':
 				$discount_id = $value;
-				$args		 = array( 'include' => array( $value ) );
+				$args = array('include' => array($value));
 				break;
 			case 'discount_code':
 			case 'code':
-				$value       = trim( $value );
-				$discount_id = wp_cache_get( $value, 'WPInv_Discount_Codes' );
-				$args		 = array( 'meta_key' => '_wpi_discount_code', 'meta_value' => $value );
+				$value       = trim($value);
+				$discount_id = wp_cache_get($value, 'WPInv_Discount_Codes');
+				$args = array('meta_key' => '_wpi_discount_code', 'meta_value' => $value);
 				break;
 			case 'name':
 				$discount_id = 0;
-				$args		 = array( 'name' => trim( $value ) );
+				$args		 = array('name' => trim($value));
 				break;
 			default:
-				$args		 = apply_filters( "wpinv_discount_get_data_by_{$field}_args", null, $value );
-				if ( ! is_array( $args ) ) {
-					return apply_filters( "wpinv_discount_get_data_by_$field", false, $value );
+				$args		 = apply_filters("wpinv_discount_get_data_by_{$field}_args", null, $value);
+				if (!is_array($args)) {
+					return apply_filters("wpinv_discount_get_data_by_$field", false, $value);
 				}
 
 		}
 
 		// Check if there is a cached value.
-		if ( ! empty( $discount_id ) && $discount = wp_cache_get( (int) $discount_id, 'WPInv_Discounts' ) ) {
+		if (!empty($discount_id) && $discount = wp_cache_get((int) $discount_id, 'WPInv_Discounts')) {
 			return $discount;
 		}
 
@@ -175,13 +175,13 @@ 
 			array(
 				'post_type'      => 'wpi_discount',
 				'posts_per_page' => 1,
-				'post_status'    => array( 'publish', 'pending', 'draft', 'expired' )
+				'post_status'    => array('publish', 'pending', 'draft', 'expired')
 			)
 		);
 
-		$discount = get_posts( $args );
+		$discount = get_posts($args);
 
-		if( empty( $discount ) ) {
+		if (empty($discount)) {
 			return false;
 		}
 
@@ -190,30 +190,30 @@ 
 		// Prepare the return data.
 		$return = array(
             'ID'                          => $discount->ID,
-            'code'                        => get_post_meta( $discount->ID, '_wpi_discount_code', true ),
-            'amount'                      => get_post_meta( $discount->ID, '_wpi_discount_amount', true ),
+            'code'                        => get_post_meta($discount->ID, '_wpi_discount_code', true),
+            'amount'                      => get_post_meta($discount->ID, '_wpi_discount_amount', true),
             'date_created'                => $discount->post_date,
 			'date_modified'               => $discount->post_modified,
 			'status'               		  => $discount->post_status,
-			'start'                  	  => get_post_meta( $discount->ID, '_wpi_discount_start', true ),
-            'expiration'                  => get_post_meta( $discount->ID, '_wpi_discount_expiration', true ),
-            'type'               		  => get_post_meta( $discount->ID, '_wpi_discount_type', true ),
+			'start'                  	  => get_post_meta($discount->ID, '_wpi_discount_start', true),
+            'expiration'                  => get_post_meta($discount->ID, '_wpi_discount_expiration', true),
+            'type'               		  => get_post_meta($discount->ID, '_wpi_discount_type', true),
             'description'                 => $discount->post_excerpt,
-            'uses'                 		  => get_post_meta( $discount->ID, '_wpi_discount_uses', true ),
-            'is_single_use'               => get_post_meta( $discount->ID, '_wpi_discount_is_single_use', true ),
-            'items'              	      => get_post_meta( $discount->ID, '_wpi_discount_items', true ),
-            'excluded_items'              => get_post_meta( $discount->ID, '_wpi_discount_excluded_items', true ),
-            'max_uses'                    => get_post_meta( $discount->ID, '_wpi_discount_max_uses', true ),
-            'is_recurring'                => get_post_meta( $discount->ID, '_wpi_discount_is_recurring', true ),
-            'min_total'                   => get_post_meta( $discount->ID, '_wpi_discount_min_total', true ),
-            'max_total'                   => get_post_meta( $discount->ID, '_wpi_discount_max_total', true ),
+            'uses'                 		  => get_post_meta($discount->ID, '_wpi_discount_uses', true),
+            'is_single_use'               => get_post_meta($discount->ID, '_wpi_discount_is_single_use', true),
+            'items'              	      => get_post_meta($discount->ID, '_wpi_discount_items', true),
+            'excluded_items'              => get_post_meta($discount->ID, '_wpi_discount_excluded_items', true),
+            'max_uses'                    => get_post_meta($discount->ID, '_wpi_discount_max_uses', true),
+            'is_recurring'                => get_post_meta($discount->ID, '_wpi_discount_is_recurring', true),
+            'min_total'                   => get_post_meta($discount->ID, '_wpi_discount_min_total', true),
+            'max_total'                   => get_post_meta($discount->ID, '_wpi_discount_max_total', true),
         );
 
-		$return = apply_filters( 'wpinv_discount_properties', $return );
+		$return = apply_filters('wpinv_discount_properties', $return);
 
 		// Update the cache with our data
-		wp_cache_add( $discount->ID, $return, 'WPInv_Discounts' );
-		wp_cache_add( $return['code'], $discount->ID, 'WPInv_Discount_Codes' );
+		wp_cache_add($discount->ID, $return, 'WPInv_Discounts');
+		wp_cache_add($return['code'], $discount->ID, 'WPInv_Discount_Codes');
 
 		return $return;
 	}
@@ -227,19 +227,19 @@ 
 	 * @since 1.0.15
 	 * @return int
 	 */
-	public static function get_discount_id_by_code( $discount_code ) {
+	public static function get_discount_id_by_code($discount_code) {
 
 		// Trim the code.
-		$discount_code = trim( $discount_code );
+		$discount_code = trim($discount_code);
 
 		// Ensure a value has been passed.
-		if ( empty( $discount_code ) ) {
+		if (empty($discount_code)) {
 			return 0;
 		}
 
 		// Maybe retrieve from the cache.
-		$discount_id   = wp_cache_get( $discount_code, 'getpaid_discount_codes' );
-		if ( ! empty( $discount_id ) ) {
+		$discount_id = wp_cache_get($discount_code, 'getpaid_discount_codes');
+		if (!empty($discount_id)) {
 			return $discount_id;
 		}
 
@@ -250,19 +250,19 @@ 
 				'meta_value'     => $discount_code,
 				'post_type'      => 'wpi_discount',
 				'posts_per_page' => 1,
-				'post_status'    => array( 'publish', 'pending', 'draft', 'expired' ),
+				'post_status'    => array('publish', 'pending', 'draft', 'expired'),
 				'fields'         => 'ids',
 			)
 		);
 
-		if ( empty( $discounts ) ) {
+		if (empty($discounts)) {
 			return 0;
 		}
 
 		$discount_id = $discounts[0];
 
 		// Update the cache with our data
-		wp_cache_add( get_post_meta( $discount_id, '_wpi_discount_code', true ), $discount_id, 'getpaid_discount_codes' );
+		wp_cache_add(get_post_meta($discount_id, '_wpi_discount_code', true), $discount_id, 'getpaid_discount_codes');
 
 		return $discount_id;
 	}
@@ -275,8 +275,8 @@ 
 	 *
 	 * @return bool Whether the given discount field is set.
 	 */
-	public function __isset( $key ){
-		return isset( $this->data[$key] ) || method_exists( $this, "get_$key");
+	public function __isset($key) {
+		return isset($this->data[$key]) || method_exists($this, "get_$key");
 	}
 
 	/*
@@ -301,8 +301,8 @@ 
 	 * @param  string $context View or edit context.
 	 * @return string
 	 */
-	public function get_status( $context = 'view' ) {
-		return $this->get_prop( 'status', $context );
+	public function get_status($context = 'view') {
+		return $this->get_prop('status', $context);
     }
 
     /**
@@ -312,8 +312,8 @@ 
 	 * @param  string $context View or edit context.
 	 * @return string
 	 */
-	public function get_version( $context = 'view' ) {
-		return $this->get_prop( 'version', $context );
+	public function get_version($context = 'view') {
+		return $this->get_prop('version', $context);
     }
 
     /**
@@ -323,8 +323,8 @@ 
 	 * @param  string $context View or edit context.
 	 * @return string
 	 */
-	public function get_date_created( $context = 'view' ) {
-		return $this->get_prop( 'date_created', $context );
+	public function get_date_created($context = 'view') {
+		return $this->get_prop('date_created', $context);
     }
 
     /**
@@ -334,11 +334,11 @@ 
 	 * @param  string $context View or edit context.
 	 * @return string
 	 */
-	public function get_date_created_gmt( $context = 'view' ) {
-        $date = $this->get_date_created( $context );
+	public function get_date_created_gmt($context = 'view') {
+        $date = $this->get_date_created($context);
 
-        if ( $date ) {
-            $date = get_gmt_from_date( $date );
+        if ($date) {
+            $date = get_gmt_from_date($date);
         }
 		return $date;
     }
@@ -350,8 +350,8 @@ 
 	 * @param  string $context View or edit context.
 	 * @return string
 	 */
-	public function get_date_modified( $context = 'view' ) {
-		return $this->get_prop( 'date_modified', $context );
+	public function get_date_modified($context = 'view') {
+		return $this->get_prop('date_modified', $context);
     }
 
     /**
@@ -361,11 +361,11 @@ 
 	 * @param  string $context View or edit context.
 	 * @return string
 	 */
-	public function get_date_modified_gmt( $context = 'view' ) {
-        $date = $this->get_date_modified( $context );
+	public function get_date_modified_gmt($context = 'view') {
+        $date = $this->get_date_modified($context);
 
-        if ( $date ) {
-            $date = get_gmt_from_date( $date );
+        if ($date) {
+            $date = get_gmt_from_date($date);
         }
 		return $date;
     }
@@ -377,8 +377,8 @@ 
 	 * @param  string $context View or edit context.
 	 * @return string
 	 */
-	public function get_name( $context = 'view' ) {
-		return $this->get_prop( 'name', $context );
+	public function get_name($context = 'view') {
+		return $this->get_prop('name', $context);
     }
 
     /**
@@ -388,8 +388,8 @@ 
 	 * @param  string $context View or edit context.
 	 * @return string
 	 */
-	public function get_title( $context = 'view' ) {
-		return $this->get_name( $context );
+	public function get_title($context = 'view') {
+		return $this->get_name($context);
     }
 
     /**
@@ -399,8 +399,8 @@ 
 	 * @param  string $context View or edit context.
 	 * @return string
 	 */
-	public function get_description( $context = 'view' ) {
-		return $this->get_prop( 'description', $context );
+	public function get_description($context = 'view') {
+		return $this->get_prop('description', $context);
     }
 
     /**
@@ -410,8 +410,8 @@ 
 	 * @param  string $context View or edit context.
 	 * @return string
 	 */
-	public function get_excerpt( $context = 'view' ) {
-		return $this->get_description( $context );
+	public function get_excerpt($context = 'view') {
+		return $this->get_description($context);
     }
 
     /**
@@ -421,8 +421,8 @@ 
 	 * @param  string $context View or edit context.
 	 * @return string
 	 */
-	public function get_summary( $context = 'view' ) {
-		return $this->get_description( $context );
+	public function get_summary($context = 'view') {
+		return $this->get_description($context);
     }
 
     /**
@@ -432,8 +432,8 @@ 
 	 * @param  string $context View or edit context.
 	 * @return string
 	 */
-	public function get_author( $context = 'view' ) {
-		return (int) $this->get_prop( 'author', $context );
+	public function get_author($context = 'view') {
+		return (int) $this->get_prop('author', $context);
 	}
 	
 	/**
@@ -443,8 +443,8 @@ 
 	 * @param  string $context View or edit context.
 	 * @return string
 	 */
-	public function get_code( $context = 'view' ) {
-		return $this->get_prop( 'code', $context );
+	public function get_code($context = 'view') {
+		return $this->get_prop('code', $context);
 	}
 	
 	/**
@@ -454,8 +454,8 @@ 
 	 * @param  string $context View or edit context.
 	 * @return string
 	 */
-	public function get_coupon_code( $context = 'view' ) {
-		return $this->get_code( $context );
+	public function get_coupon_code($context = 'view') {
+		return $this->get_code($context);
 	}
 	
 	/**
@@ -465,8 +465,8 @@ 
 	 * @param  string $context View or edit context.
 	 * @return string
 	 */
-	public function get_discount_code( $context = 'view' ) {
-		return $this->get_code( $context );
+	public function get_discount_code($context = 'view') {
+		return $this->get_code($context);
 	}
 	
 	/**
@@ -476,8 +476,8 @@ 
 	 * @param  string $context View or edit context.
 	 * @return float
 	 */
-	public function get_amount( $context = 'view' ) {
-		return $context == 'view' ? floatval( $this->get_prop( 'amount', $context ) ) : $this->get_prop( 'amount', $context );
+	public function get_amount($context = 'view') {
+		return $context == 'view' ? floatval($this->get_prop('amount', $context)) : $this->get_prop('amount', $context);
 	}
 
 	/**
@@ -488,13 +488,13 @@ 
 	 */
 	public function get_formatted_amount() {
 
-		if ( $this->is_type( 'flat' ) ) {
-			$rate = wpinv_price( $this->get_amount() );
+		if ($this->is_type('flat')) {
+			$rate = wpinv_price($this->get_amount());
 		} else {
 			$rate = $this->get_amount() . '%';
 		}
 
-		return apply_filters( 'wpinv_format_discount_rate', $rate, $this->get_type(), $this->get_amount() );
+		return apply_filters('wpinv_format_discount_rate', $rate, $this->get_type(), $this->get_amount());
 	}
 	
 	/**
@@ -504,8 +504,8 @@ 
 	 * @param  string $context View or edit context.
 	 * @return string
 	 */
-	public function get_start( $context = 'view' ) {
-		return $this->get_prop( 'start', $context );
+	public function get_start($context = 'view') {
+		return $this->get_prop('start', $context);
 	}
 	
 	/**
@@ -515,8 +515,8 @@ 
 	 * @param  string $context View or edit context.
 	 * @return string
 	 */
-	public function get_start_date( $context = 'view' ) {
-		return $this->get_start( $context );
+	public function get_start_date($context = 'view') {
+		return $this->get_start($context);
 	}
 	
 	/**
@@ -526,8 +526,8 @@ 
 	 * @param  string $context View or edit context.
 	 * @return string
 	 */
-	public function get_expiration( $context = 'view' ) {
-		return $this->get_prop( 'expiration', $context );
+	public function get_expiration($context = 'view') {
+		return $this->get_prop('expiration', $context);
 	}
 	
 	/**
@@ -537,8 +537,8 @@ 
 	 * @param  string $context View or edit context.
 	 * @return string
 	 */
-	public function get_expiration_date( $context = 'view' ) {
-		return $this->get_expiration( $context );
+	public function get_expiration_date($context = 'view') {
+		return $this->get_expiration($context);
 	}
 
 	/**
@@ -548,8 +548,8 @@ 
 	 * @param  string $context View or edit context.
 	 * @return string
 	 */
-	public function get_end_date( $context = 'view' ) {
-		return $this->get_expiration( $context );
+	public function get_end_date($context = 'view') {
+		return $this->get_expiration($context);
 	}
 	
 	/**
@@ -559,8 +559,8 @@ 
 	 * @param  string $context View or edit context.
 	 * @return string
 	 */
-	public function get_type( $context = 'view' ) {
-		return $this->get_prop( 'type', $context );
+	public function get_type($context = 'view') {
+		return $this->get_prop('type', $context);
 	}
 
 	/**
@@ -570,8 +570,8 @@ 
 	 * @param  string $context View or edit context.
 	 * @return int
 	 */
-	public function get_uses( $context = 'view' ) {
-		return (int) $this->get_prop( 'uses', $context );
+	public function get_uses($context = 'view') {
+		return (int) $this->get_prop('uses', $context);
 	}
 
 	/**
@@ -582,7 +582,7 @@ 
 	 */
 	public function get_usage() {
 
-		if ( ! $this->has_limit() ) {
+		if (!$this->has_limit()) {
 			return $this->get_uses() . ' / ' . ' &infin;';
 		}
 
@@ -597,9 +597,9 @@ 
 	 * @param  string $context View or edit context.
 	 * @return int
 	 */
-	public function get_max_uses( $context = 'view' ) {
-		$max_uses = $this->get_prop( 'max_uses', $context );
-		return empty( $max_uses ) ? null : $max_uses;
+	public function get_max_uses($context = 'view') {
+		$max_uses = $this->get_prop('max_uses', $context);
+		return empty($max_uses) ? null : $max_uses;
 	}
 
 	/**
@@ -609,8 +609,8 @@ 
 	 * @param  string $context View or edit context.
 	 * @return bool
 	 */
-	public function get_is_single_use( $context = 'view' ) {
-		return $this->get_prop( 'is_single_use', $context );
+	public function get_is_single_use($context = 'view') {
+		return $this->get_prop('is_single_use', $context);
 	}
 
 	/**
@@ -620,8 +620,8 @@ 
 	 * @param  string $context View or edit context.
 	 * @return array
 	 */
-	public function get_items( $context = 'view' ) {
-		return wpinv_parse_list( $this->get_prop( 'items', $context ) );
+	public function get_items($context = 'view') {
+		return wpinv_parse_list($this->get_prop('items', $context));
 	}
 
 	/**
@@ -631,8 +631,8 @@ 
 	 * @param  string $context View or edit context.
 	 * @return array
 	 */
-	public function get_allowed_items( $context = 'view' ) {
-		return $this->get_items( $context );
+	public function get_allowed_items($context = 'view') {
+		return $this->get_items($context);
 	}
 
 	/**
@@ -642,8 +642,8 @@ 
 	 * @param  string $context View or edit context.
 	 * @return array
 	 */
-	public function get_excluded_items( $context = 'view' ) {
-		return wpinv_parse_list( $this->get_prop( 'excluded_items', $context ) );
+	public function get_excluded_items($context = 'view') {
+		return wpinv_parse_list($this->get_prop('excluded_items', $context));
 	}
 
 	/**
@@ -653,8 +653,8 @@ 
 	 * @param  string $context View or edit context.
 	 * @return int|string|bool
 	 */
-	public function get_is_recurring( $context = 'view' ) {
-		return $this->get_prop( 'is_recurring', $context );
+	public function get_is_recurring($context = 'view') {
+		return $this->get_prop('is_recurring', $context);
 	}
 
 	/**
@@ -664,9 +664,9 @@ 
 	 * @param  string $context View or edit context.
 	 * @return float
 	 */
-	public function get_min_total( $context = 'view' ) {
-		$minimum = $this->get_prop( 'min_total', $context );
-		return empty( $minimum ) ? null : $minimum;
+	public function get_min_total($context = 'view') {
+		$minimum = $this->get_prop('min_total', $context);
+		return empty($minimum) ? null : $minimum;
 	}
 
 	/**
@@ -676,8 +676,8 @@ 
 	 * @param  string $context View or edit context.
 	 * @return float
 	 */
-	public function get_minimum_total( $context = 'view' ) {
-		return $this->get_min_total( $context );
+	public function get_minimum_total($context = 'view') {
+		return $this->get_min_total($context);
 	}
 
 	/**
@@ -687,9 +687,9 @@ 
 	 * @param  string $context View or edit context.
 	 * @return float
 	 */
-	public function get_max_total( $context = 'view' ) {
-		$maximum = $this->get_prop( 'max_total', $context );
-		return empty( $maximum ) ? null : $maximum;
+	public function get_max_total($context = 'view') {
+		$maximum = $this->get_prop('max_total', $context);
+		return empty($maximum) ? null : $maximum;
 	}
 
 	/**
@@ -699,8 +699,8 @@ 
 	 * @param  string $context View or edit context.
 	 * @return float
 	 */
-	public function get_maximum_total( $context = 'view' ) {
-		return $this->get_max_total( $context );
+	public function get_maximum_total($context = 'view') {
+		return $this->get_max_total($context);
 	}
 
 	/**
@@ -713,8 +713,8 @@ 
 	 * @param  string $context View or edit context.
 	 * @return mixed Value of the given discount property (if set).
 	 */
-	public function get( $key, $context = 'view' ) {
-        return $this->get_prop( $key, $context );
+	public function get($key, $context = 'view') {
+        return $this->get_prop($key, $context);
 	}
 
 	/*
@@ -734,10 +734,10 @@ 
 	 * @param  string $status New status.
 	 * @return array details of change.
 	 */
-	public function set_status( $status ) {
+	public function set_status($status) {
         $old_status = $this->get_status();
 
-        $this->set_prop( 'status', $status );
+        $this->set_prop('status', $status);
 
 		return array(
 			'from' => $old_status,
@@ -750,8 +750,8 @@ 
 	 *
 	 * @since 1.0.19
 	 */
-	public function set_version( $value ) {
-		$this->set_prop( 'version', $value );
+	public function set_version($value) {
+		$this->set_prop('version', $value);
     }
 
     /**
@@ -761,11 +761,11 @@ 
 	 * @param string $value Value to set.
      * @return bool Whether or not the date was set.
 	 */
-	public function set_date_created( $value ) {
-        $date = strtotime( $value );
+	public function set_date_created($value) {
+        $date = strtotime($value);
 
-        if ( $date ) {
-            $this->set_prop( 'date_created', date( 'Y-m-d H:i:s', $date ) );
+        if ($date) {
+            $this->set_prop('date_created', date('Y-m-d H:i:s', $date));
             return true;
         }
 
@@ -780,11 +780,11 @@ 
 	 * @param string $value Value to set.
      * @return bool Whether or not the date was set.
 	 */
-	public function set_date_modified( $value ) {
-        $date = strtotime( $value );
+	public function set_date_modified($value) {
+        $date = strtotime($value);
 
-        if ( $date ) {
-            $this->set_prop( 'date_modified', date( 'Y-m-d H:i:s', $date ) );
+        if ($date) {
+            $this->set_prop('date_modified', date('Y-m-d H:i:s', $date));
             return true;
         }
 
@@ -798,9 +798,9 @@ 
 	 * @since 1.0.19
 	 * @param  string $value New name.
 	 */
-	public function set_name( $value ) {
-        $name = sanitize_text_field( $value );
-		$this->set_prop( 'name', $name );
+	public function set_name($value) {
+        $name = sanitize_text_field($value);
+		$this->set_prop('name', $name);
     }
 
     /**
@@ -809,8 +809,8 @@ 
 	 * @since 1.0.19
 	 * @param  string $value New name.
 	 */
-	public function set_title( $value ) {
-		$this->set_name( $value );
+	public function set_title($value) {
+		$this->set_name($value);
     }
 
     /**
@@ -819,9 +819,9 @@ 
 	 * @since 1.0.19
 	 * @param  string $value New description.
 	 */
-	public function set_description( $value ) {
-        $description = wp_kses_post( $value );
-		return $this->set_prop( 'description', $description );
+	public function set_description($value) {
+        $description = wp_kses_post($value);
+		return $this->set_prop('description', $description);
     }
 
     /**
@@ -830,8 +830,8 @@ 
 	 * @since 1.0.19
 	 * @param  string $value New description.
 	 */
-	public function set_excerpt( $value ) {
-		$this->set_description( $value );
+	public function set_excerpt($value) {
+		$this->set_description($value);
     }
 
     /**
@@ -840,8 +840,8 @@ 
 	 * @since 1.0.19
 	 * @param  string $value New description.
 	 */
-	public function set_summary( $value ) {
-		$this->set_description( $value );
+	public function set_summary($value) {
+		$this->set_description($value);
     }
 
     /**
@@ -850,8 +850,8 @@ 
 	 * @since 1.0.19
 	 * @param  int $value New author.
 	 */
-	public function set_author( $value ) {
-		$this->set_prop( 'author', (int) $value );
+	public function set_author($value) {
+		$this->set_prop('author', (int) $value);
 	}
 	
 	/**
@@ -860,9 +860,9 @@ 
 	 * @since 1.0.19
 	 * @param string $value New discount code.
 	 */
-	public function set_code( $value ) {
-		$code = sanitize_text_field( $value );
-		$this->set_prop( 'code', $code );
+	public function set_code($value) {
+		$code = sanitize_text_field($value);
+		$this->set_prop('code', $code);
 	}
 	
 	/**
@@ -871,8 +871,8 @@ 
 	 * @since 1.0.19
 	 * @param string $value New discount code.
 	 */
-	public function set_coupon_code( $value ) {
-		$this->set_code( $value );
+	public function set_coupon_code($value) {
+		$this->set_code($value);
 	}
 	
 	/**
@@ -881,8 +881,8 @@ 
 	 * @since 1.0.19
 	 * @param string $value New discount code.
 	 */
-	public function set_discount_code( $value ) {
-		$this->set_code( $value );
+	public function set_discount_code($value) {
+		$this->set_code($value);
 	}
 	
 	/**
@@ -891,9 +891,9 @@ 
 	 * @since 1.0.19
 	 * @param float $value New discount code.
 	 */
-	public function set_amount( $value ) {
-		$amount = floatval( wpinv_sanitize_amount( $value ) );
-		$this->set_prop( 'amount', $amount );
+	public function set_amount($value) {
+		$amount = floatval(wpinv_sanitize_amount($value));
+		$this->set_prop('amount', $amount);
 	}
 
 	/**
@@ -902,15 +902,15 @@ 
 	 * @since 1.0.19
 	 * @param float $value New start date.
 	 */
-	public function set_start( $value ) {
-		$date = strtotime( $value );
+	public function set_start($value) {
+		$date = strtotime($value);
 
-        if ( $date ) {
-            $this->set_prop( 'start', date( 'Y-m-d H:i', $date ) );
+        if ($date) {
+            $this->set_prop('start', date('Y-m-d H:i', $date));
             return true;
 		}
 		
-		$this->set_prop( 'start', '' );
+		$this->set_prop('start', '');
 
         return false;
 	}
@@ -921,8 +921,8 @@ 
 	 * @since 1.0.19
 	 * @param string $value New start date.
 	 */
-	public function set_start_date( $value ) {
-		$this->set_start( $value );
+	public function set_start_date($value) {
+		$this->set_start($value);
 	}
 
 	/**
@@ -931,15 +931,15 @@ 
 	 * @since 1.0.19
 	 * @param float $value New expiration date.
 	 */
-	public function set_expiration( $value ) {
-		$date = strtotime( $value );
+	public function set_expiration($value) {
+		$date = strtotime($value);
 
-        if ( $date ) {
-            $this->set_prop( 'expiration', date( 'Y-m-d H:i', $date ) );
+        if ($date) {
+            $this->set_prop('expiration', date('Y-m-d H:i', $date));
             return true;
         }
 
-		$this->set_prop( 'expiration', '' );
+		$this->set_prop('expiration', '');
         return false;
 	}
 
@@ -949,8 +949,8 @@ 
 	 * @since 1.0.19
 	 * @param string $value New expiration date.
 	 */
-	public function set_expiration_date( $value ) {
-		$this->set_expiration( $value );
+	public function set_expiration_date($value) {
+		$this->set_expiration($value);
 	}
 
 	/**
@@ -959,8 +959,8 @@ 
 	 * @since 1.0.19
 	 * @param string $value New expiration date.
 	 */
-	public function set_end_date( $value ) {
-		$this->set_expiration( $value );
+	public function set_end_date($value) {
+		$this->set_expiration($value);
 	}
 
 	/**
@@ -969,9 +969,9 @@ 
 	 * @since 1.0.19
 	 * @param string $value New discount type.
 	 */
-	public function set_type( $value ) {
-		if ( $value && array_key_exists( sanitize_text_field( $value ), wpinv_get_discount_types() ) ) {
-			$this->set_prop( 'type', sanitize_text_field( $value ) );
+	public function set_type($value) {
+		if ($value && array_key_exists(sanitize_text_field($value), wpinv_get_discount_types())) {
+			$this->set_prop('type', sanitize_text_field($value));
 		}
 	}
 
@@ -981,15 +981,15 @@ 
 	 * @since 1.0.19
 	 * @param int $value usage count.
 	 */
-	public function set_uses( $value ) {
+	public function set_uses($value) {
 
 		$value = (int) $value;
 
-		if ( $value < 0 ) {
+		if ($value < 0) {
 			$value = 0;
 		}
 
-		$this->set_prop( 'uses', (int) $value );
+		$this->set_prop('uses', (int) $value);
 	}
 
 	/**
@@ -998,8 +998,8 @@ 
 	 * @since 1.0.19
 	 * @param int $value maximum usage count.
 	 */
-	public function set_max_uses( $value ) {
-		$this->set_prop( 'max_uses', absint( $value ) );
+	public function set_max_uses($value) {
+		$this->set_prop('max_uses', absint($value));
 	}
 
 	/**
@@ -1008,8 +1008,8 @@ 
 	 * @since 1.0.19
 	 * @param int|bool $value is single use.
 	 */
-	public function set_is_single_use( $value ) {
-		$this->set_prop( 'is_single_use', (bool) $value );
+	public function set_is_single_use($value) {
+		$this->set_prop('is_single_use', (bool) $value);
 	}
 
 	/**
@@ -1018,8 +1018,8 @@ 
 	 * @since 1.0.19
 	 * @param array $value items.
 	 */
-	public function set_items( $value ) {
-		$this->set_prop( 'items', wpinv_parse_list( $value ) );
+	public function set_items($value) {
+		$this->set_prop('items', wpinv_parse_list($value));
 	}
 
 	/**
@@ -1028,8 +1028,8 @@ 
 	 * @since 1.0.19
 	 * @param array $value items.
 	 */
-	public function set_allowed_items( $value ) {
-		$this->set_items( $value );
+	public function set_allowed_items($value) {
+		$this->set_items($value);
 	}
 
 	/**
@@ -1038,8 +1038,8 @@ 
 	 * @since 1.0.19
 	 * @param array $value items.
 	 */
-	public function set_excluded_items( $value ) {
-		$this->set_prop( 'excluded_items', wpinv_parse_list( $value ) );
+	public function set_excluded_items($value) {
+		$this->set_prop('excluded_items', wpinv_parse_list($value));
 	}
 
 	/**
@@ -1048,8 +1048,8 @@ 
 	 * @since 1.0.19
 	 * @param int|bool $value is recurring.
 	 */
-	public function set_is_recurring( $value ) {
-		$this->set_prop( 'is_recurring', (bool) $value );
+	public function set_is_recurring($value) {
+		$this->set_prop('is_recurring', (bool) $value);
 	}
 
 	/**
@@ -1058,8 +1058,8 @@ 
 	 * @since 1.0.19
 	 * @param float $value minimum total.
 	 */
-	public function set_min_total( $value ) {
-		$this->set_prop( 'min_total', (float) wpinv_sanitize_amount( $value ) );
+	public function set_min_total($value) {
+		$this->set_prop('min_total', (float) wpinv_sanitize_amount($value));
 	}
 
 	/**
@@ -1068,8 +1068,8 @@ 
 	 * @since 1.0.19
 	 * @param float $value minimum total.
 	 */
-	public function set_minimum_total( $value ) {
-		$this->set_min_total( $value );
+	public function set_minimum_total($value) {
+		$this->set_min_total($value);
 	}
 
 	/**
@@ -1078,8 +1078,8 @@ 
 	 * @since 1.0.19
 	 * @param float $value maximum total.
 	 */
-	public function set_max_total( $value ) {
-		$this->set_prop( 'max_total', (float) wpinv_sanitize_amount( $value ) );
+	public function set_max_total($value) {
+		$this->set_prop('max_total', (float) wpinv_sanitize_amount($value));
 	}
 
 	/**
@@ -1088,23 +1088,23 @@ 
 	 * @since 1.0.19
 	 * @param float $value maximum total.
 	 */
-	public function set_maximum_total( $value ) {
-		$this->set_max_total( $value );
+	public function set_maximum_total($value) {
+		$this->set_max_total($value);
 	}
 
 	/**
 	 * @deprecated
 	 */
-	public function refresh(){}
+	public function refresh() {}
 
 	/**
 	 * @deprecated
 	 *
 	 */
-	public function update_status( $status = 'publish' ){
+	public function update_status($status = 'publish') {
 
-		if ( $this->exists() && $this->get_status() != $status ) {
-			$this->set_status( $status );
+		if ($this->exists() && $this->get_status() != $status) {
+			$this->set_status($status);
 			$this->save();
 		}
 
@@ -1124,9 +1124,9 @@ 
 	 *
 	 * @since 1.0.15
 	 */
-	public function exists(){
+	public function exists() {
 		$id = $this->get_id();
-		return ! empty( $id );
+		return !empty($id);
 	}
 
 	/**
@@ -1137,7 +1137,7 @@ 
 	 * @since 1.0.15
 	 * @return bool
 	 */
-	public function is_type( $type ) {
+	public function is_type($type) {
 		return $this->get_type() == $type;
 	}
 
@@ -1159,7 +1159,7 @@ 
 	 */
 	public function has_limit() {
 		$limit = $this->get_max_uses();
-		return ! empty( $limit );
+		return !empty($limit);
 	}
 
 	/**
@@ -1180,13 +1180,13 @@ 
 	 */
 	public function has_exceeded_limit() {
 
-		if ( ! $this->has_limit() || ! $this->has_uses() ) {
-			$exceeded = false ;
+		if (!$this->has_limit() || !$this->has_uses()) {
+			$exceeded = false;
 		} else {
 			$exceeded = (int) $this->get_max_uses() <= $this->get_uses();
 		}
 
-		return apply_filters( 'wpinv_is_discount_maxed_out', $exceeded, $this->get_id(), $this, $this->get_code() );
+		return apply_filters('wpinv_is_discount_maxed_out', $exceeded, $this->get_id(), $this, $this->get_code());
 	}
 
 	/**
@@ -1197,7 +1197,7 @@ 
 	 */
 	public function has_expiration_date() {
 		$date = $this->get_expiration_date();
-		return ! empty( $date );
+		return !empty($date);
 	}
 
 	/**
@@ -1207,8 +1207,8 @@ 
 	 * @return bool
 	 */
 	public function is_expired() {
-		$expired = $this->has_expiration_date() ? current_time( 'timestamp' ) > strtotime( $this->get_expiration_date() ) : false;
-		return apply_filters( 'wpinv_is_discount_expired', $expired, $this->get_id(), $this, $this->get_code() );
+		$expired = $this->has_expiration_date() ? current_time('timestamp') > strtotime($this->get_expiration_date()) : false;
+		return apply_filters('wpinv_is_discount_expired', $expired, $this->get_id(), $this, $this->get_code());
 	}
 
 	/**
@@ -1219,7 +1219,7 @@ 
 	 */
 	public function has_start_date() {
 		$date = $this->get_start_date();
-		return ! empty( $date );
+		return !empty($date);
 	}
 
 	/**
@@ -1229,8 +1229,8 @@ 
 	 * @return bool
 	 */
 	public function has_started() {
-		$started = $this->has_start_date() ? true : current_time( 'timestamp' ) > strtotime( $this->get_start_date() );
-		return apply_filters( 'wpinv_is_discount_started', $started, $this->get_id(), $this, $this->get_code() );
+		$started = $this->has_start_date() ? true : current_time('timestamp') > strtotime($this->get_start_date());
+		return apply_filters('wpinv_is_discount_started', $started, $this->get_id(), $this, $this->get_code());
 	}
 
 	/**
@@ -1241,7 +1241,7 @@ 
 	 */
 	public function has_allowed_items() {
 		$allowed_items = $this->get_allowed_items();
-		return ! empty( $allowed_items );
+		return !empty($allowed_items);
 	}
 
 	/**
@@ -1252,7 +1252,7 @@ 
 	 */
 	public function has_excluded_items() {
 		$excluded_items = $this->get_excluded_items();
-		return ! empty( $excluded_items );
+		return !empty($excluded_items);
 	}
 
 	/**
@@ -1262,17 +1262,17 @@ 
 	 * @since 1.0.15
 	 * @return boolean
 	 */
-	public function is_valid_for_items( $item_ids ) {
+	public function is_valid_for_items($item_ids) {
 
-		$item_ids = wp_parse_id_list( $item_ids );
-		$included = array_intersect( $item_ids, $this->get_allowed_items() );
-		$excluded = array_intersect( $item_ids, $this->get_excluded_items() );
+		$item_ids = wp_parse_id_list($item_ids);
+		$included = array_intersect($item_ids, $this->get_allowed_items());
+		$excluded = array_intersect($item_ids, $this->get_excluded_items());
 
-		if ( $this->has_excluded_items() && ! empty( $excluded ) ) {
+		if ($this->has_excluded_items() && !empty($excluded)) {
 			return false;
 		}
 
-		if ( $this->has_allowed_items() && empty( $included ) ) {
+		if ($this->has_allowed_items() && empty($included)) {
 			return false;
 		}
 
@@ -1286,8 +1286,8 @@ 
 	 * @since 1.0.15
 	 * @return boolean
 	 */
-	public function is_valid_for_amount( $amount ) {
-		return $this->is_minimum_amount_met( $amount ) && $this->is_maximum_amount_met( $amount );
+	public function is_valid_for_amount($amount) {
+		return $this->is_minimum_amount_met($amount) && $this->is_maximum_amount_met($amount);
 	}
 
 	/**
@@ -1298,7 +1298,7 @@ 
 	 */
 	public function has_minimum_amount() {
 		$minimum = $this->get_minimum_total();
-		return ! empty( $minimum );
+		return !empty($minimum);
 	}
 
 	/**
@@ -1308,10 +1308,10 @@ 
 	 * @since 1.0.15
 	 * @return boolean
 	 */
-	public function is_minimum_amount_met( $amount ) {
-		$amount = floatval( wpinv_sanitize_amount( $amount ) );
-		$min_met= ! ( $this->has_minimum_amount() && $amount < floatval( wpinv_sanitize_amount( $this->get_minimum_total() ) ) );
-		return apply_filters( 'wpinv_is_discount_min_met', $min_met, $this->get_id(), $this, $this->get_code(), $amount );
+	public function is_minimum_amount_met($amount) {
+		$amount = floatval(wpinv_sanitize_amount($amount));
+		$min_met = !($this->has_minimum_amount() && $amount < floatval(wpinv_sanitize_amount($this->get_minimum_total())));
+		return apply_filters('wpinv_is_discount_min_met', $min_met, $this->get_id(), $this, $this->get_code(), $amount);
 	}
 
 	/**
@@ -1322,7 +1322,7 @@ 
 	 */
 	public function has_maximum_amount() {
 		$maximum = $this->get_maximum_total();
-		return ! empty( $maximum );
+		return !empty($maximum);
 	}
 
 	/**
@@ -1332,10 +1332,10 @@ 
 	 * @since 1.0.15
 	 * @return boolean
 	 */
-	public function is_maximum_amount_met( $amount ) {
-		$amount = floatval( wpinv_sanitize_amount( $amount ) );
-		$max_met= ! ( $this->has_maximum_amount() && $amount > floatval( wpinv_sanitize_amount( $this->get_maximum_total() ) ) );
-		return apply_filters( 'wpinv_is_discount_max_met', $max_met, $this->get_id(), $this, $this->get_code(), $amount );
+	public function is_maximum_amount_met($amount) {
+		$amount = floatval(wpinv_sanitize_amount($amount));
+		$max_met = !($this->has_maximum_amount() && $amount > floatval(wpinv_sanitize_amount($this->get_maximum_total())));
+		return apply_filters('wpinv_is_discount_max_met', $max_met, $this->get_id(), $this, $this->get_code(), $amount);
 	}
 
 	/**
@@ -1346,7 +1346,7 @@ 
 	 */
 	public function is_recurring() {
 		$recurring = $this->get_is_recurring();
-		return ! empty( $recurring );
+		return !empty($recurring);
 	}
 
 	/**
@@ -1357,7 +1357,7 @@ 
 	 */
 	public function is_single_use() {
 		$usage = $this->get_is_single_use();
-		return ! empty( $usage );
+		return !empty($usage);
 	}
 
 	/**
@@ -1367,37 +1367,37 @@ 
 	 * @since 1.0.15
 	 * @return boolean
 	 */
-	public function is_valid_for_user( $user ) {
+	public function is_valid_for_user($user) {
 
 		// Ensure that the discount is single use.
-		if ( empty( $user ) || ! $this->is_single_use() ) {
+		if (empty($user) || !$this->is_single_use()) {
 			return true;
 		}
 
 		// Prepare the user id.
 		$user_id = 0;
-        if ( is_numeric( $user ) ) {
-            $user_id = absint( $user );
-        } else if ( is_email( $user ) && $user_data = get_user_by( 'email', $user ) ) {
+        if (is_numeric($user)) {
+            $user_id = absint($user);
+        } else if (is_email($user) && $user_data = get_user_by('email', $user)) {
             $user_id = $user_data->ID;
-        } else if ( $user_data = get_user_by( 'login', $user ) ) {
+        } else if ($user_data = get_user_by('login', $user)) {
             $user_id = $user_data->ID;
         }
 
 		// Ensure that we have a user.
-		if ( empty( $user_id ) ) {
+		if (empty($user_id)) {
 			return true;
 		}
 
 		// Get all payments with matching user id.
-        $payments = wpinv_get_invoices( array( 'user' => $user_id, 'limit' => false, 'paginate' => false ) );
-		$code     = strtolower( $this->get_code() );
+        $payments = wpinv_get_invoices(array('user' => $user_id, 'limit' => false, 'paginate' => false));
+		$code = strtolower($this->get_code());
 
 		// For each payment...
-		foreach ( $payments as $payment ) {
+		foreach ($payments as $payment) {
 
 			// Only check for paid invoices.
-			if ( $payment->is_paid() && strtolower( $payment->get_discount_code() ) == $code ) {
+			if ($payment->is_paid() && strtolower($payment->get_discount_code()) == $code) {
 				return false;
 			}
 
@@ -1423,24 +1423,24 @@ 
 	 * @param int $by The number of usages to increas by.
 	 * @return int
 	 */
-	public function increase_usage( $by = 1 ) {
+	public function increase_usage($by = 1) {
 
 		// Abort if zero.
-		if ( empty( $by ) ) {
+		if (empty($by)) {
 			return;
 		}
 
 		// Increase the usage.
-		$this->set_uses( $this->get_uses() + (int) $by );
+		$this->set_uses($this->get_uses() + (int) $by);
 
 		// Save the discount.
 		$this->save();
 
 		// Fire relevant hooks.
-		if( (int) $by > 0 ) {
-			do_action( 'wpinv_discount_increase_use_count', $this->get_uses(), $this->get_id(), $this->get_code(),  absint( $by ) );
+		if ((int) $by > 0) {
+			do_action('wpinv_discount_increase_use_count', $this->get_uses(), $this->get_id(), $this->get_code(), absint($by));
 		} else {
-			do_action( 'wpinv_discount_decrease_use_count', $this->get_uses(), $this->get_id(), $this->get_code(), absint( $by ) );
+			do_action('wpinv_discount_decrease_use_count', $this->get_uses(), $this->get_id(), $this->get_code(), absint($by));
 		}
 
 		// Return the number of times the discount has been used.
@@ -1464,7 +1464,7 @@ 
 	 * @param float $amount
 	 * @return float
 	 */
-	public function get_discounted_amount( $amount ) {
+	public function get_discounted_amount($amount) {
 
 		// Convert amount to float.
 		$amount = (float) $amount;
@@ -1472,29 +1472,29 @@ 
 		// Get discount amount.
 		$discount_amount = $this->get_amount();
 
-		if ( empty( $discount_amount ) ) {
+		if (empty($discount_amount)) {
 			return 0;
 		}
 
 		// Format the amount.
-		$discount_amount = floatval( wpinv_sanitize_amount( $discount_amount ) );
+		$discount_amount = floatval(wpinv_sanitize_amount($discount_amount));
 		
 		// If this is a percentage discount.
-		if ( $this->is_type( 'percent' ) ) {
-            $discount_amount = $amount * ( $discount_amount / 100 );
+		if ($this->is_type('percent')) {
+            $discount_amount = $amount * ($discount_amount / 100);
 		}
 
 		// Discount can not be less than zero...
-		if ( $discount_amount < 0 ) {
+		if ($discount_amount < 0) {
 			$discount_amount = 0;
 		}
 
 		// ... or more than the amount.
-		if ( $discount_amount > $amount ) {
+		if ($discount_amount > $amount) {
 			$discount_amount = $amount;
 		}
 
-		return apply_filters( 'wpinv_discount_total_discount_amount', $discount_amount, $amount, $this );
+		return apply_filters('wpinv_discount_total_discount_amount', $discount_amount, $amount, $this);
 	}
 
 }

includes/data-stores/class-getpaid-data.php

Spacing +161 added lines, -161 removed lines

@@ -8,7 +8,7 @@ 
  *
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if (!defined('ABSPATH')) {
 	exit;
 }
 
@@ -117,8 +117,8 @@ 
 	 *
 	 * @param int|object|array|string $read ID to load from the DB (optional) or already queried data.
 	 */
-	public function __construct( $read = 0 ) {
-		$this->data         = array_merge( $this->data, $this->extra_data );
+	public function __construct($read = 0) {
+		$this->data         = array_merge($this->data, $this->extra_data);
 		$this->default_data = $this->data;
 	}
 
@@ -128,7 +128,7 @@ 
 	 * @return array
 	 */
 	public function __sleep() {
-		return array( 'id' );
+		return array('id');
 	}
 
 	/**
@@ -137,10 +137,10 @@ 
 	 * If the object no longer exists, remove the ID.
 	 */
 	public function __wakeup() {
-		$this->__construct( absint( $this->id ) );
+		$this->__construct(absint($this->id));
 
-		if ( ! empty( $this->last_error ) ) {
-			$this->set_id( 0 );
+		if (!empty($this->last_error)) {
+			$this->set_id(0);
 		}
 
 	}
@@ -152,11 +152,11 @@ 
 	 */
 	public function __clone() {
 		$this->maybe_read_meta_data();
-		if ( ! empty( $this->meta_data ) ) {
-			foreach ( $this->meta_data as $array_key => $meta ) {
-				$this->meta_data[ $array_key ] = clone $meta;
-				if ( ! empty( $meta->id ) ) {
-					$this->meta_data[ $array_key ]->id = null;
+		if (!empty($this->meta_data)) {
+			foreach ($this->meta_data as $array_key => $meta) {
+				$this->meta_data[$array_key] = clone $meta;
+				if (!empty($meta->id)) {
+					$this->meta_data[$array_key]->id = null;
 				}
 			}
 		}
@@ -199,8 +199,8 @@ 
 	 * @param  string $context View or edit context.
 	 * @return string
 	 */
-	public function get_status( $context = 'view' ) {
-		return $this->get_prop( 'status', $context );
+	public function get_status($context = 'view') {
+		return $this->get_prop('status', $context);
     }
 
 	/**
@@ -210,10 +210,10 @@ 
 	 * @param  bool $force_delete Should the data be deleted permanently.
 	 * @return bool result
 	 */
-	public function delete( $force_delete = false ) {
-		if ( $this->data_store && $this->exists() ) {
-			$this->data_store->delete( $this, array( 'force_delete' => $force_delete ) );
-			$this->set_id( 0 );
+	public function delete($force_delete = false) {
+		if ($this->data_store && $this->exists()) {
+			$this->data_store->delete($this, array('force_delete' => $force_delete));
+			$this->set_id(0);
 			return true;
 		}
 		return false;
@@ -226,7 +226,7 @@ 
 	 * @return int
 	 */
 	public function save() {
-		if ( ! $this->data_store ) {
+		if (!$this->data_store) {
 			return $this->get_id();
 		}
 
@@ -236,12 +236,12 @@ 
 		 * @param GetPaid_Data          $this The object being saved.
 		 * @param GetPaid_Data_Store_WP $data_store The data store persisting the data.
 		 */
-		do_action( 'getpaid_before_' . $this->object_type . '_object_save', $this, $this->data_store );
+		do_action('getpaid_before_' . $this->object_type . '_object_save', $this, $this->data_store);
 
-		if ( $this->get_id() ) {
-			$this->data_store->update( $this );
+		if ($this->get_id()) {
+			$this->data_store->update($this);
 		} else {
-			$this->data_store->create( $this );
+			$this->data_store->create($this);
 		}
 
 		/**
@@ -250,7 +250,7 @@ 
 		 * @param GetPaid_Data          $this The object being saved.
 		 * @param GetPaid_Data_Store_WP $data_store The data store persisting the data.
 		 */
-		do_action( 'getpaid_after_' . $this->object_type . '_object_save', $this, $this->data_store );
+		do_action('getpaid_after_' . $this->object_type . '_object_save', $this, $this->data_store);
 
 		return $this->get_id();
 	}
@@ -262,7 +262,7 @@ 
 	 * @return string Data in JSON format.
 	 */
 	public function __toString() {
-		return wp_json_encode( $this->get_data() );
+		return wp_json_encode($this->get_data());
 	}
 
 	/**
@@ -272,7 +272,7 @@ 
 	 * @return array
 	 */
 	public function get_data() {
-		return array_merge( array( 'id' => $this->get_id() ), $this->data, array( 'meta_data' => $this->get_meta_data() ) );
+		return array_merge(array('id' => $this->get_id()), $this->data, array('meta_data' => $this->get_meta_data()));
 	}
 
 	/**
@@ -282,7 +282,7 @@ 
 	 * @return array
 	 */
 	public function get_data_keys() {
-		return array_keys( $this->data );
+		return array_keys($this->data);
 	}
 
 	/**
@@ -292,7 +292,7 @@ 
 	 * @return array
 	 */
 	public function get_extra_data_keys() {
-		return array_keys( $this->extra_data );
+		return array_keys($this->extra_data);
 	}
 
 	/**
@@ -302,8 +302,8 @@ 
 	 * @param mixed $meta Meta value to check.
 	 * @return bool
 	 */
-	protected function filter_null_meta( $meta ) {
-		return ! is_null( $meta->value );
+	protected function filter_null_meta($meta) {
+		return !is_null($meta->value);
 	}
 
 	/**
@@ -314,7 +314,7 @@ 
 	 */
 	public function get_meta_data() {
 		$this->maybe_read_meta_data();
-		return array_values( array_filter( $this->meta_data, array( $this, 'filter_null_meta' ) ) );
+		return array_values(array_filter($this->meta_data, array($this, 'filter_null_meta')));
 	}
 
 	/**
@@ -324,21 +324,21 @@ 
 	 * @param  string $key Key to check.
 	 * @return bool   true if it's an internal key, false otherwise
 	 */
-	protected function is_internal_meta_key( $key ) {
-		$internal_meta_key = ! empty( $key ) && $this->data_store && in_array( $key, $this->data_store->get_internal_meta_keys(), true );
+	protected function is_internal_meta_key($key) {
+		$internal_meta_key = !empty($key) && $this->data_store && in_array($key, $this->data_store->get_internal_meta_keys(), true);
 
-		if ( ! $internal_meta_key ) {
+		if (!$internal_meta_key) {
 			return false;
 		}
 
-		$has_setter_or_getter = is_callable( array( $this, 'set_' . $key ) ) || is_callable( array( $this, 'get_' . $key ) );
+		$has_setter_or_getter = is_callable(array($this, 'set_' . $key)) || is_callable(array($this, 'get_' . $key));
 
-		if ( ! $has_setter_or_getter ) {
+		if (!$has_setter_or_getter) {
 			return false;
 		}
 
 		/* translators: %s: $key Key to check */
-		getpaid_doing_it_wrong( __FUNCTION__, sprintf( __( 'Generic add/update/get meta methods should not be used for internal meta data, including "%s". Use getters and setters.', 'invoicing' ), $key ), '1.0.19' );
+		getpaid_doing_it_wrong(__FUNCTION__, sprintf(__('Generic add/update/get meta methods should not be used for internal meta data, including "%s". Use getters and setters.', 'invoicing'), $key), '1.0.19');
 
 		return true;
 	}
@@ -352,20 +352,20 @@ 
 	 * @access public
 	 *
 	 */
-	public function __set( $key, $value ) {
+	public function __set($key, $value) {
 
-		if ( 'id' == strtolower( $key ) ) {
-			return $this->set_id( $value );
+		if ('id' == strtolower($key)) {
+			return $this->set_id($value);
 		}
 
-		if ( method_exists( $this, "set_$key") ) {
+		if (method_exists($this, "set_$key")) {
 
 			/* translators: %s: $key Key to set */
-			getpaid_doing_it_wrong( __FUNCTION__, sprintf( __( 'Object data such as "%s" should not be accessed directly. Use getters and setters.', 'invoicing' ), $key ), '1.0.19' );
+			getpaid_doing_it_wrong(__FUNCTION__, sprintf(__('Object data such as "%s" should not be accessed directly. Use getters and setters.', 'invoicing'), $key), '1.0.19');
 
-			call_user_func( array( $this, "set_$key" ), $value );
+			call_user_func(array($this, "set_$key"), $value);
 		} else {
-			$this->set_prop( $key, $value );
+			$this->set_prop($key, $value);
 		}
 
 	}
@@ -373,25 +373,25 @@ 
 	/**
      * Margic method for retrieving a property.
      */
-    public function __get( $key ) {
+    public function __get($key) {
 
         // Check if we have a helper method for that.
-        if ( method_exists( $this, 'get_' . $key ) ) {
+        if (method_exists($this, 'get_' . $key)) {
 
-			if ( 'post_type' != $key ) {
+			if ('post_type' != $key) {
 				/* translators: %s: $key Key to set */
-				getpaid_doing_it_wrong( __FUNCTION__, sprintf( __( 'Object data such as "%s" should not be accessed directly. Use getters and setters.', 'invoicing' ), $key ), '1.0.19' );
+				getpaid_doing_it_wrong(__FUNCTION__, sprintf(__('Object data such as "%s" should not be accessed directly. Use getters and setters.', 'invoicing'), $key), '1.0.19');
 			}
 
-            return call_user_func( array( $this, 'get_' . $key ) );
+            return call_user_func(array($this, 'get_' . $key));
         }
 
         // Check if the key is in the associated $post object.
-        if ( ! empty( $this->post ) && isset( $this->post->$key ) ) {
+        if (!empty($this->post) && isset($this->post->$key)) {
             return $this->post->$key;
         }
 
-		return $this->get_prop( $key );
+		return $this->get_prop($key);
 
     }
 
@@ -404,15 +404,15 @@ 
 	 * @param  string $context What the value is for. Valid values are view and edit.
 	 * @return mixed
 	 */
-	public function get_meta( $key = '', $single = true, $context = 'view' ) {
+	public function get_meta($key = '', $single = true, $context = 'view') {
 
 		// Check if this is an internal meta key.
-		$_key = str_replace( '_wpinv', '', $key );
-		$_key = str_replace( 'wpinv', '', $_key );
-		if ( $this->is_internal_meta_key( $key ) ) {
+		$_key = str_replace('_wpinv', '', $key);
+		$_key = str_replace('wpinv', '', $_key);
+		if ($this->is_internal_meta_key($key)) {
 			$function = 'get_' . $_key;
 
-			if ( is_callable( array( $this, $function ) ) ) {
+			if (is_callable(array($this, $function))) {
 				return $this->{$function}();
 			}
 		}
@@ -420,20 +420,20 @@ 
 		// Read the meta data if not yet read.
 		$this->maybe_read_meta_data();
 		$meta_data  = $this->get_meta_data();
-		$array_keys = array_keys( wp_list_pluck( $meta_data, 'key' ), $key, true );
+		$array_keys = array_keys(wp_list_pluck($meta_data, 'key'), $key, true);
 		$value      = $single ? '' : array();
 
-		if ( ! empty( $array_keys ) ) {
+		if (!empty($array_keys)) {
 			// We don't use the $this->meta_data property directly here because we don't want meta with a null value (i.e. meta which has been deleted via $this->delete_meta_data()).
-			if ( $single ) {
-				$value = $meta_data[ current( $array_keys ) ]->value;
+			if ($single) {
+				$value = $meta_data[current($array_keys)]->value;
 			} else {
-				$value = array_intersect_key( $meta_data, array_flip( $array_keys ) );
+				$value = array_intersect_key($meta_data, array_flip($array_keys));
 			}
 		}
 
-		if ( 'view' === $context ) {
-			$value = apply_filters( $this->get_hook_prefix() . $key, $value, $this );
+		if ('view' === $context) {
+			$value = apply_filters($this->get_hook_prefix() . $key, $value, $this);
 		}
 
 		return $value;
@@ -446,10 +446,10 @@ 
 	 * @param  string $key Meta Key.
 	 * @return boolean
 	 */
-	public function meta_exists( $key = '' ) {
+	public function meta_exists($key = '') {
 		$this->maybe_read_meta_data();
-		$array_keys = wp_list_pluck( $this->get_meta_data(), 'key' );
-		return in_array( $key, $array_keys, true );
+		$array_keys = wp_list_pluck($this->get_meta_data(), 'key');
+		return in_array($key, $array_keys, true);
 	}
 
 	/**
@@ -458,12 +458,12 @@ 
 	 * @since 1.0.19
 	 * @param array $data Key/Value pairs.
 	 */
-	public function set_meta_data( $data ) {
-		if ( ! empty( $data ) && is_array( $data ) ) {
+	public function set_meta_data($data) {
+		if (!empty($data) && is_array($data)) {
 			$this->maybe_read_meta_data();
-			foreach ( $data as $meta ) {
+			foreach ($data as $meta) {
 				$meta = (array) $meta;
-				if ( isset( $meta['key'], $meta['value'], $meta['id'] ) ) {
+				if (isset($meta['key'], $meta['value'], $meta['id'])) {
 					$this->meta_data[] = new GetPaid_Meta_Data(
 						array(
 							'id'    => $meta['id'],
@@ -485,18 +485,18 @@ 
 	 * @param string|array $value Meta value.
 	 * @param bool         $unique Should this be a unique key?.
 	 */
-	public function add_meta_data( $key, $value, $unique = false ) {
-		if ( $this->is_internal_meta_key( $key ) ) {
+	public function add_meta_data($key, $value, $unique = false) {
+		if ($this->is_internal_meta_key($key)) {
 			$function = 'set_' . $key;
 
-			if ( is_callable( array( $this, $function ) ) ) {
-				return $this->{$function}( $value );
+			if (is_callable(array($this, $function))) {
+				return $this->{$function}($value);
 			}
 		}
 
 		$this->maybe_read_meta_data();
-		if ( $unique ) {
-			$this->delete_meta_data( $key );
+		if ($unique) {
+			$this->delete_meta_data($key);
 		}
 		$this->meta_data[] = new GetPaid_Meta_Data(
 			array(
@@ -517,12 +517,12 @@ 
 	 * @param  string|array $value Meta value.
 	 * @param  int          $meta_id Meta ID.
 	 */
-	public function update_meta_data( $key, $value, $meta_id = 0 ) {
-		if ( $this->is_internal_meta_key( $key ) ) {
+	public function update_meta_data($key, $value, $meta_id = 0) {
+		if ($this->is_internal_meta_key($key)) {
 			$function = 'set_' . $key;
 
-			if ( is_callable( array( $this, $function ) ) ) {
-				return $this->{$function}( $value );
+			if (is_callable(array($this, $function))) {
+				return $this->{$function}($value);
 			}
 		}
 
@@ -530,33 +530,33 @@ 
 
 		$array_key = false;
 
-		if ( $meta_id ) {
-			$array_keys = array_keys( wp_list_pluck( $this->meta_data, 'id' ), $meta_id, true );
-			$array_key  = $array_keys ? current( $array_keys ) : false;
+		if ($meta_id) {
+			$array_keys = array_keys(wp_list_pluck($this->meta_data, 'id'), $meta_id, true);
+			$array_key  = $array_keys ? current($array_keys) : false;
 		} else {
 			// Find matches by key.
 			$matches = array();
-			foreach ( $this->meta_data as $meta_data_array_key => $meta ) {
-				if ( $meta->key === $key ) {
+			foreach ($this->meta_data as $meta_data_array_key => $meta) {
+				if ($meta->key === $key) {
 					$matches[] = $meta_data_array_key;
 				}
 			}
 
-			if ( ! empty( $matches ) ) {
+			if (!empty($matches)) {
 				// Set matches to null so only one key gets the new value.
-				foreach ( $matches as $meta_data_array_key ) {
-					$this->meta_data[ $meta_data_array_key ]->value = null;
+				foreach ($matches as $meta_data_array_key) {
+					$this->meta_data[$meta_data_array_key]->value = null;
 				}
-				$array_key = current( $matches );
+				$array_key = current($matches);
 			}
 		}
 
-		if ( false !== $array_key ) {
-			$meta        = $this->meta_data[ $array_key ];
+		if (false !== $array_key) {
+			$meta        = $this->meta_data[$array_key];
 			$meta->key   = $key;
 			$meta->value = $value;
 		} else {
-			$this->add_meta_data( $key, $value, true );
+			$this->add_meta_data($key, $value, true);
 		}
 	}
 
@@ -566,13 +566,13 @@ 
 	 * @since 1.0.19
 	 * @param string $key Meta key.
 	 */
-	public function delete_meta_data( $key ) {
+	public function delete_meta_data($key) {
 		$this->maybe_read_meta_data();
-		$array_keys = array_keys( wp_list_pluck( $this->meta_data, 'key' ), $key, true );
+		$array_keys = array_keys(wp_list_pluck($this->meta_data, 'key'), $key, true);
 
-		if ( $array_keys ) {
-			foreach ( $array_keys as $array_key ) {
-				$this->meta_data[ $array_key ]->value = null;
+		if ($array_keys) {
+			foreach ($array_keys as $array_key) {
+				$this->meta_data[$array_key]->value = null;
 			}
 		}
 	}
@@ -583,13 +583,13 @@ 
 	 * @since 1.0.19
 	 * @param int $mid Meta ID.
 	 */
-	public function delete_meta_data_by_mid( $mid ) {
+	public function delete_meta_data_by_mid($mid) {
 		$this->maybe_read_meta_data();
-		$array_keys = array_keys( wp_list_pluck( $this->meta_data, 'id' ), (int) $mid, true );
+		$array_keys = array_keys(wp_list_pluck($this->meta_data, 'id'), (int) $mid, true);
 
-		if ( $array_keys ) {
-			foreach ( $array_keys as $array_key ) {
-				$this->meta_data[ $array_key ]->value = null;
+		if ($array_keys) {
+			foreach ($array_keys as $array_key) {
+				$this->meta_data[$array_key]->value = null;
 			}
 		}
 	}
@@ -600,7 +600,7 @@ 
 	 * @since 1.0.19
 	 */
 	protected function maybe_read_meta_data() {
-		if ( is_null( $this->meta_data ) ) {
+		if (is_null($this->meta_data)) {
 			$this->read_meta_data();
 		}
 	}
@@ -612,42 +612,42 @@ 
 	 * @since 1.0.19
 	 * @param bool $force_read True to force a new DB read (and update cache).
 	 */
-	public function read_meta_data( $force_read = false ) {
+	public function read_meta_data($force_read = false) {
 
 		// Reset meta data.
 		$this->meta_data = array();
 
 		// Maybe abort early.
-		if ( ! $this->get_id() || ! $this->data_store ) {
+		if (!$this->get_id() || !$this->data_store) {
 			return;
 		}
 
 		// Only read from cache if the cache key is set.
 		$cache_key = null;
-		if ( ! $force_read && ! empty( $this->cache_group ) ) {
-			$cache_key     = GetPaid_Cache_Helper::get_cache_prefix( $this->cache_group ) . GetPaid_Cache_Helper::get_cache_prefix( 'object_' . $this->get_id() ) . 'object_meta_' . $this->get_id();
-			$raw_meta_data = wp_cache_get( $cache_key, $this->cache_group );
+		if (!$force_read && !empty($this->cache_group)) {
+			$cache_key     = GetPaid_Cache_Helper::get_cache_prefix($this->cache_group) . GetPaid_Cache_Helper::get_cache_prefix('object_' . $this->get_id()) . 'object_meta_' . $this->get_id();
+			$raw_meta_data = wp_cache_get($cache_key, $this->cache_group);
 		}
 
 		// Should we force read?
-		if ( empty( $raw_meta_data ) ) {
-			$raw_meta_data = $this->data_store->read_meta( $this );
+		if (empty($raw_meta_data)) {
+			$raw_meta_data = $this->data_store->read_meta($this);
 
-			if ( ! empty( $cache_key ) ) {
-				wp_cache_set( $cache_key, $raw_meta_data, $this->cache_group );
+			if (!empty($cache_key)) {
+				wp_cache_set($cache_key, $raw_meta_data, $this->cache_group);
 			}
 
 		}
 
 		// Set meta data.
-		if ( is_array( $raw_meta_data ) ) {
+		if (is_array($raw_meta_data)) {
 
-			foreach ( $raw_meta_data as $meta ) {
+			foreach ($raw_meta_data as $meta) {
 				$this->meta_data[] = new GetPaid_Meta_Data(
 					array(
 						'id'    => (int) $meta->meta_id,
 						'key'   => $meta->meta_key,
-						'value' => maybe_unserialize( $meta->meta_value ),
+						'value' => maybe_unserialize($meta->meta_value),
 					)
 				);
 			}
@@ -662,28 +662,28 @@ 
 	 * @since 1.0.19
 	 */
 	public function save_meta_data() {
-		if ( ! $this->data_store || is_null( $this->meta_data ) ) {
+		if (!$this->data_store || is_null($this->meta_data)) {
 			return;
 		}
-		foreach ( $this->meta_data as $array_key => $meta ) {
-			if ( is_null( $meta->value ) ) {
-				if ( ! empty( $meta->id ) ) {
-					$this->data_store->delete_meta( $this, $meta );
-					unset( $this->meta_data[ $array_key ] );
+		foreach ($this->meta_data as $array_key => $meta) {
+			if (is_null($meta->value)) {
+				if (!empty($meta->id)) {
+					$this->data_store->delete_meta($this, $meta);
+					unset($this->meta_data[$array_key]);
 				}
-			} elseif ( empty( $meta->id ) ) {
-				$meta->id = $this->data_store->add_meta( $this, $meta );
+			} elseif (empty($meta->id)) {
+				$meta->id = $this->data_store->add_meta($this, $meta);
 				$meta->apply_changes();
 			} else {
-				if ( $meta->get_changes() ) {
-					$this->data_store->update_meta( $this, $meta );
+				if ($meta->get_changes()) {
+					$this->data_store->update_meta($this, $meta);
 					$meta->apply_changes();
 				}
 			}
 		}
-		if ( ! empty( $this->cache_group ) ) {
-			$cache_key = GetPaid_Cache_Helper::get_cache_prefix( $this->cache_group ) . GetPaid_Cache_Helper::get_cache_prefix( 'object_' . $this->get_id() ) . 'object_meta_' . $this->get_id();
-			wp_cache_delete( $cache_key, $this->cache_group );
+		if (!empty($this->cache_group)) {
+			$cache_key = GetPaid_Cache_Helper::get_cache_prefix($this->cache_group) . GetPaid_Cache_Helper::get_cache_prefix('object_' . $this->get_id()) . 'object_meta_' . $this->get_id();
+			wp_cache_delete($cache_key, $this->cache_group);
 		}
 	}
 
@@ -693,8 +693,8 @@ 
 	 * @since 1.0.19
 	 * @param int $id ID.
 	 */
-	public function set_id( $id ) {
-		$this->id = absint( $id );
+	public function set_id($id) {
+		$this->id = absint($id);
 	}
 
 	/**
@@ -704,10 +704,10 @@ 
 	 * @param string $status New status.
 	 * @return array details of change.
 	 */
-	public function set_status( $status ) {
+	public function set_status($status) {
         $old_status = $this->get_status();
 
-		$this->set_prop( 'status', $status );
+		$this->set_prop('status', $status);
 
 		return array(
 			'from' => $old_status,
@@ -723,7 +723,7 @@ 
 	public function set_defaults() {
 		$this->data    = $this->default_data;
 		$this->changes = array();
-		$this->set_object_read( false );
+		$this->set_object_read(false);
 	}
 
 	/**
@@ -732,7 +732,7 @@ 
 	 * @since 1.0.19
 	 * @param boolean $read Should read?.
 	 */
-	public function set_object_read( $read = true ) {
+	public function set_object_read($read = true) {
 		$this->object_read = (bool) $read;
 	}
 
@@ -757,32 +757,32 @@ 
 	 *
 	 * @return bool|WP_Error
 	 */
-	public function set_props( $props, $context = 'set' ) {
+	public function set_props($props, $context = 'set') {
 		$errors = false;
 
-		foreach ( $props as $prop => $value ) {
+		foreach ($props as $prop => $value) {
 			try {
 				/**
 				 * Checks if the prop being set is allowed, and the value is not null.
 				 */
-				if ( is_null( $value ) || in_array( $prop, array( 'prop', 'date_prop', 'meta_data' ), true ) ) {
+				if (is_null($value) || in_array($prop, array('prop', 'date_prop', 'meta_data'), true)) {
 					continue;
 				}
 				$setter = "set_$prop";
 
-				if ( is_callable( array( $this, $setter ) ) ) {
-					$this->{$setter}( $value );
+				if (is_callable(array($this, $setter))) {
+					$this->{$setter}($value);
 				}
-			} catch ( Exception $e ) {
-				if ( ! $errors ) {
+			} catch (Exception $e) {
+				if (!$errors) {
 					$errors = new WP_Error();
 				}
-				$errors->add( $e->getCode(), $e->getMessage() );
+				$errors->add($e->getCode(), $e->getMessage());
 				$this->last_error = $e->getMessage();
 			}
 		}
 
-		return $errors && count( $errors->get_error_codes() ) ? $errors : true;
+		return $errors && count($errors->get_error_codes()) ? $errors : true;
 	}
 
 	/**
@@ -795,14 +795,14 @@ 
 	 * @param string $prop Name of prop to set.
 	 * @param mixed  $value Value of the prop.
 	 */
-	protected function set_prop( $prop, $value ) {
-		if ( array_key_exists( $prop, $this->data ) ) {
-			if ( true === $this->object_read ) {
-				if ( $value !== $this->data[ $prop ] || array_key_exists( $prop, $this->changes ) ) {
-					$this->changes[ $prop ] = $value;
+	protected function set_prop($prop, $value) {
+		if (array_key_exists($prop, $this->data)) {
+			if (true === $this->object_read) {
+				if ($value !== $this->data[$prop] || array_key_exists($prop, $this->changes)) {
+					$this->changes[$prop] = $value;
 				}
 			} else {
-				$this->data[ $prop ] = $value;
+				$this->data[$prop] = $value;
 			}
 		}
 	}
@@ -823,7 +823,7 @@ 
 	 * @since 1.0.19
 	 */
 	public function apply_changes() {
-		$this->data    = array_replace( $this->data, $this->changes );
+		$this->data    = array_replace($this->data, $this->changes);
 		$this->changes = array();
 	}
 
@@ -848,14 +848,14 @@ 
 	 * @param  string $context What the value is for. Valid values are view and edit.
 	 * @return mixed
 	 */
-	protected function get_prop( $prop, $context = 'view' ) {
+	protected function get_prop($prop, $context = 'view') {
 		$value = null;
 
-		if ( array_key_exists( $prop, $this->data ) ) {
-			$value = array_key_exists( $prop, $this->changes ) ? $this->changes[ $prop ] : $this->data[ $prop ];
+		if (array_key_exists($prop, $this->data)) {
+			$value = array_key_exists($prop, $this->changes) ? $this->changes[$prop] : $this->data[$prop];
 
-			if ( 'view' === $context ) {
-				$value = apply_filters( $this->get_hook_prefix() . $prop, $value, $this );
+			if ('view' === $context) {
+				$value = apply_filters($this->get_hook_prefix() . $prop, $value, $this);
 			}
 		}
 
@@ -869,13 +869,13 @@ 
 	 * @param string         $prop Name of prop to set.
 	 * @param string|integer $value Value of the prop.
 	 */
-	protected function set_date_prop( $prop, $value ) {
+	protected function set_date_prop($prop, $value) {
 
-		if ( empty( $value ) ) {
-			$this->set_prop( $prop, null );
+		if (empty($value)) {
+			$this->set_prop($prop, null);
 			return;
 		}
-		$this->set_prop( $prop, $value );
+		$this->set_prop($prop, $value);
 
 	}
 
@@ -886,7 +886,7 @@ 
 	 * @param string $code             Error code.
 	 * @param string $message          Error message.
 	 */
-	protected function error( $code, $message ) {
+	protected function error($code, $message) {
 		$this->last_error = $message;
 	}
 
@@ -898,7 +898,7 @@ 
 	 */
 	public function exists() {
 		$id = $this->get_id();
-		return ! empty( $id );
+		return !empty($id);
 	}
 
 }

includes/class-getpaid-subscription-notification-emails.php

Spacing +65 added lines, -65 removed lines

@@ -4,7 +4,7 @@ 
  *
  */
 
-defined( 'ABSPATH' ) || exit;
+defined('ABSPATH') || exit;
 
 /**
  * This class handles subscription notificaiton emails.
@@ -45,21 +45,21 @@ 
 	 */
 	public function init_hooks() {
 
-		add_filter( 'getpaid_get_email_merge_tags', array( $this, 'subscription_merge_tags' ), 10, 2 );
-		foreach ( $this->subscription_actions as $hook => $email_type ) {
+		add_filter('getpaid_get_email_merge_tags', array($this, 'subscription_merge_tags'), 10, 2);
+		foreach ($this->subscription_actions as $hook => $email_type) {
 
-			$email = new GetPaid_Notification_Email( $email_type );
+			$email = new GetPaid_Notification_Email($email_type);
 
-			if ( ! $email->is_active() ) {
+			if (!$email->is_active()) {
 				continue;
 			}
 
-			if ( method_exists( $this, $email_type ) ) {
-				add_action( $hook, array( $this, $email_type ), 100, 2 );
+			if (method_exists($this, $email_type)) {
+				add_action($hook, array($this, $email_type), 100, 2);
 				continue;
 			}
 
-			do_action( 'getpaid_subscription_notification_email_register_hook', $email_type, $hook );
+			do_action('getpaid_subscription_notification_email_register_hook', $email_type, $hook);
 
 		}
 
@@ -71,12 +71,12 @@ 
 	 * @param array $merge_tags
 	 * @param mixed|WPInv_Invoice|WPInv_Subscription $object
 	 */
-	public function subscription_merge_tags( $merge_tags, $object ) {
+	public function subscription_merge_tags($merge_tags, $object) {
 
-		if ( is_a( $object, 'WPInv_Subscription' ) ) {
+		if (is_a($object, 'WPInv_Subscription')) {
 			$merge_tags = array_merge(
 				$merge_tags,
-				$this->get_subscription_merge_tags( $object )
+				$this->get_subscription_merge_tags($object)
 			);
 		}
 
@@ -90,25 +90,25 @@ 
 	 * @param WPInv_Subscription $subscription
 	 * @return array
 	 */
-	public function get_subscription_merge_tags( $subscription ) {
+	public function get_subscription_merge_tags($subscription) {
 
 		// Abort if it does not exist.
-		if ( ! $subscription->get_id() ) {
+		if (!$subscription->get_id()) {
 			return array();
 		}
 
-		$invoice    = $subscription->get_parent_invoice();
+		$invoice = $subscription->get_parent_invoice();
 		return array(
-			'{subscription_renewal_date}'     => getpaid_format_date_value( $subscription->get_next_renewal_date(), __( 'Never', 'invoicing' ) ),
-			'{subscription_created}'          => getpaid_format_date_value( $subscription->get_date_created() ),
-			'{subscription_status}'           => sanitize_text_field( $subscription->get_status_label() ),
-			'{subscription_profile_id}'       => sanitize_text_field( $subscription->get_profile_id() ),
-			'{subscription_id}'               => absint( $subscription->get_id() ),
-			'{subscription_recurring_amount}' => sanitize_text_field( wpinv_price( $subscription->get_recurring_amount(), $invoice->get_currency() ) ),
-			'{subscription_initial_amount}'   => sanitize_text_field( wpinv_price( $subscription->get_initial_amount(), $invoice->get_currency() ) ),
-			'{subscription_recurring_period}' => getpaid_get_subscription_period_label( $subscription->get_period(), $subscription->get_frequency(), '' ),
+			'{subscription_renewal_date}'     => getpaid_format_date_value($subscription->get_next_renewal_date(), __('Never', 'invoicing')),
+			'{subscription_created}'          => getpaid_format_date_value($subscription->get_date_created()),
+			'{subscription_status}'           => sanitize_text_field($subscription->get_status_label()),
+			'{subscription_profile_id}'       => sanitize_text_field($subscription->get_profile_id()),
+			'{subscription_id}'               => absint($subscription->get_id()),
+			'{subscription_recurring_amount}' => sanitize_text_field(wpinv_price($subscription->get_recurring_amount(), $invoice->get_currency())),
+			'{subscription_initial_amount}'   => sanitize_text_field(wpinv_price($subscription->get_initial_amount(), $invoice->get_currency())),
+			'{subscription_recurring_period}' => getpaid_get_subscription_period_label($subscription->get_period(), $subscription->get_frequency(), ''),
 			'{subscription_bill_times}'       => $subscription->get_bill_times(),
-			'{subscription_url}'              => esc_url( $subscription->get_view_url() ),
+			'{subscription_url}'              => esc_url($subscription->get_view_url()),
 		);
 
 	}
@@ -119,7 +119,7 @@ 
 	 * @param WPInv_Invoice $invoice
 	 * @return bool
 	 */
-	public function should_send_notification( $invoice ) {
+	public function should_send_notification($invoice) {
 		return 0 != $invoice->get_id();
 	}
 
@@ -129,14 +129,14 @@ 
 	 * @param WPInv_Invoice $invoice
 	 * @return array
 	 */
-	public function get_recipients( $invoice ) {
-		$recipients = array( $invoice->get_email() );
+	public function get_recipients($invoice) {
+		$recipients = array($invoice->get_email());
 
 		$cc = $invoice->get_email_cc();
 
-		if ( ! empty( $cc ) ) {
-			$cc = array_map( 'sanitize_email', wpinv_parse_list( $cc ) );
-			$recipients = array_filter( array_unique( array_merge( $recipients, $cc ) ) );
+		if (!empty($cc)) {
+			$cc = array_map('sanitize_email', wpinv_parse_list($cc));
+			$recipients = array_filter(array_unique(array_merge($recipients, $cc)));
 		}
 
 		return $recipients;
@@ -150,51 +150,51 @@ 
 	 * @param string $type
 	 * @param array $extra_args Extra template args.
 	 */
-	public function send_email( $subscription, $email, $type, $extra_args = array() ) {
+	public function send_email($subscription, $email, $type, $extra_args = array()) {
 
 		// Abort in case the parent invoice does not exist.
 		$invoice = $subscription->get_parent_invoice();
-		if ( ! $this->should_send_notification( $invoice ) ) {
+		if (!$this->should_send_notification($invoice)) {
 			return;
 		}
 
-		if ( apply_filters( 'getpaid_skip_subscription_email', false, $type, $subscription ) ) {
+		if (apply_filters('getpaid_skip_subscription_email', false, $type, $subscription)) {
 			return;
 		}
 
-		do_action( 'getpaid_before_send_subscription_notification', $type, $subscription, $email );
+		do_action('getpaid_before_send_subscription_notification', $type, $subscription, $email);
 
-		$recipients  = $this->get_recipients( $invoice );
+		$recipients  = $this->get_recipients($invoice);
 		$mailer      = new GetPaid_Notification_Email_Sender();
 		$merge_tags  = $email->get_merge_tags();
-		$content     = $email->get_content( $merge_tags, $extra_args );
-		$subject     = $email->add_merge_tags( $email->get_subject(), $merge_tags );
+		$content     = $email->get_content($merge_tags, $extra_args);
+		$subject     = $email->add_merge_tags($email->get_subject(), $merge_tags);
 		$attachments = $email->get_attachments();
 
 		$result = $mailer->send(
-			apply_filters( 'getpaid_subscription_email_recipients', wpinv_parse_list( $recipients ), $email ),
+			apply_filters('getpaid_subscription_email_recipients', wpinv_parse_list($recipients), $email),
 			$subject,
 			$content,
 			$attachments
 		);
 
 		// Maybe send a copy to the admin.
-		if ( $email->include_admin_bcc() ) {
+		if ($email->include_admin_bcc()) {
 			$mailer->send(
 				wpinv_get_admin_email(),
-				$subject . __( ' - ADMIN BCC COPY', 'invoicing' ),
+				$subject . __(' - ADMIN BCC COPY', 'invoicing'),
 				$content,
 				$attachments
 			);
 		}
 
-		if ( $result ) {
-			$subscription->get_parent_invoice()->add_note( sprintf( __( 'Successfully sent %s notification email.', 'invoicing' ), sanitize_key( $type ) ), false, false, true );
+		if ($result) {
+			$subscription->get_parent_invoice()->add_note(sprintf(__('Successfully sent %s notification email.', 'invoicing'), sanitize_key($type)), false, false, true);
 		} else {
-			$subscription->get_parent_invoice()->add_note( sprintf( __( 'Failed sending %s notification email.', 'invoicing' ), sanitize_key( $type ) ), false, false, true );
+			$subscription->get_parent_invoice()->add_note(sprintf(__('Failed sending %s notification email.', 'invoicing'), sanitize_key($type)), false, false, true);
 		}
 
-		do_action( 'getpaid_after_send_subscription_notification', $type, $subscription, $email );
+		do_action('getpaid_after_send_subscription_notification', $type, $subscription, $email);
 
 	}
 
@@ -203,10 +203,10 @@ 
 	 *
 	 * @param WPInv_Subscription $subscription
 	 */
-	public function subscription_trial( $subscription ) {
+	public function subscription_trial($subscription) {
 
-		$email     = new GetPaid_Notification_Email( __FUNCTION__, $subscription );
-		$this->send_email( $subscription, $email, __FUNCTION__ );
+		$email = new GetPaid_Notification_Email(__FUNCTION__, $subscription);
+		$this->send_email($subscription, $email, __FUNCTION__);
 
 	}
 
@@ -215,10 +215,10 @@ 
 	 *
 	 * @param WPInv_Subscription $subscription
 	 */
-	public function subscription_cancelled( $subscription ) {
+	public function subscription_cancelled($subscription) {
 
-		$email     = new GetPaid_Notification_Email( __FUNCTION__, $subscription );
-		$this->send_email( $subscription, $email, __FUNCTION__ );
+		$email = new GetPaid_Notification_Email(__FUNCTION__, $subscription);
+		$this->send_email($subscription, $email, __FUNCTION__);
 
 	}
 
@@ -227,10 +227,10 @@ 
 	 *
 	 * @param WPInv_Subscription $subscription
 	 */
-	public function subscription_expired( $subscription ) {
+	public function subscription_expired($subscription) {
 
-		$email     = new GetPaid_Notification_Email( __FUNCTION__, $subscription );
-		$this->send_email( $subscription, $email, __FUNCTION__ );
+		$email = new GetPaid_Notification_Email(__FUNCTION__, $subscription);
+		$this->send_email($subscription, $email, __FUNCTION__);
 
 	}
 
@@ -239,10 +239,10 @@ 
 	 *
 	 * @param WPInv_Subscription $subscription
 	 */
-	public function subscription_complete( $subscription ) {
+	public function subscription_complete($subscription) {
 
-		$email     = new GetPaid_Notification_Email( __FUNCTION__, $subscription );
-		$this->send_email( $subscription, $email, __FUNCTION__ );
+		$email = new GetPaid_Notification_Email(__FUNCTION__, $subscription);
+		$this->send_email($subscription, $email, __FUNCTION__);
 
 	}
 
@@ -252,18 +252,18 @@ 
 	 */
 	public function renewal_reminder() {
 
-		$email = new GetPaid_Notification_Email( __FUNCTION__ );
+		$email = new GetPaid_Notification_Email(__FUNCTION__);
 
 		// Fetch reminder days.
-		$reminder_days = array_unique( wp_parse_id_list( $email->get_option( 'days' ) ) );
+		$reminder_days = array_unique(wp_parse_id_list($email->get_option('days')));
 
 		// Abort if non is set.
-		if ( empty( $reminder_days ) ) {
+		if (empty($reminder_days)) {
 			return;
 		}
 
 		// Fetch matching subscriptions.
-        $args  = array(
+        $args = array(
             'number'             => -1,
 			'count_total'        => false,
 			'status'             => 'trialling active',
@@ -272,8 +272,8 @@ 
             ),
 		);
 
-		foreach ( $reminder_days as $days ) {
-			$date = date_parse( date( 'Y-m-d', strtotime( "+$days days", current_time( 'timestamp' ) ) ) );
+		foreach ($reminder_days as $days) {
+			$date = date_parse(date('Y-m-d', strtotime("+$days days", current_time('timestamp'))));
 
 			$args['date_expires_query'][] = array(
 				'year'  => $date['year'],
@@ -283,14 +283,14 @@ 
 
 		}
 
-		$subscriptions = new GetPaid_Subscriptions_Query( $args );
+		$subscriptions = new GetPaid_Subscriptions_Query($args);
 
-        foreach ( $subscriptions as $subscription ) {
+        foreach ($subscriptions as $subscription) {
 
 			// Skip packages.
-			if ( get_post_meta( $subscription->get_product_id(), '_wpinv_type', true ) != 'package' ) {
+			if (get_post_meta($subscription->get_product_id(), '_wpinv_type', true) != 'package') {
 				$email->object = $subscription;
-            	$this->send_email( $subscription, $email, __FUNCTION__ );
+            	$this->send_email($subscription, $email, __FUNCTION__);
 			}
 
 		}

includes/wpinv-item-functions.php

Spacing +131 added lines, -131 removed lines

@@ -6,7 +6,7 @@ 
  * @package Invoicing
  */
 
-defined( 'ABSPATH' ) || exit;
+defined('ABSPATH') || exit;
 
 /**
  * Retrieves an item by it's ID.
@@ -14,9 +14,9 @@ 
  * @param int the item ID to retrieve.
  * @return WPInv_Item|false
  */
-function wpinv_get_item_by_id( $id ) {
-    $item = wpinv_get_item( $id );
-    return empty( $item ) || $id != $item->get_id() ? false : $item;
+function wpinv_get_item_by_id($id) {
+    $item = wpinv_get_item($id);
+    return empty($item) || $id != $item->get_id() ? false : $item;
 }
 
 /**
@@ -24,14 +24,14 @@ 
  *
  * @return WPInv_Item|false
  */
-function wpinv_get_item_by( $field = '', $value = '', $type = '' ) {
+function wpinv_get_item_by($field = '', $value = '', $type = '') {
 
-    if ( 'id' == strtolower( $field ) ) {
-        return wpinv_get_item_by_id( $field );
+    if ('id' == strtolower($field)) {
+        return wpinv_get_item_by_id($field);
     }
 
-    $id = WPInv_Item::get_item_id_by_field( $value, strtolower( $field ), $type );
-    return empty( $id ) ? false : wpinv_get_item( $id );
+    $id = WPInv_Item::get_item_id_by_field($value, strtolower($field), $type);
+    return empty($id) ? false : wpinv_get_item($id);
 
 }
 
@@ -41,22 +41,22 @@ 
  * @param int|WPInv_Item the item to retrieve.
  * @return WPInv_Item|false
  */
-function wpinv_get_item( $item = 0 ) {
+function wpinv_get_item($item = 0) {
 
-    if ( empty( $item ) ) {
+    if (empty($item)) {
         return false;
     }
 
-    $item = new WPInv_Item( $item );
+    $item = new WPInv_Item($item);
     return $item->exists() ? $item : false;
 
 }
 
-function wpinv_get_all_items( $args = array() ) {
+function wpinv_get_all_items($args = array()) {
 
-    $args = wp_parse_args( $args, array(
-        'status'         => array( 'publish' ),
-        'limit'          => get_option( 'posts_per_page' ),
+    $args = wp_parse_args($args, array(
+        'status'         => array('publish'),
+        'limit'          => get_option('posts_per_page'),
         'page'           => 1,
         'exclude'        => array(),
         'orderby'        => 'date',
@@ -65,7 +65,7 @@ 
         'meta_query'     => array(),
         'return'         => 'objects',
         'paginate'       => false,
-    ) );
+    ));
 
     $wp_query_args = array(
         'post_type'      => 'wpi_item',
@@ -75,26 +75,26 @@ 
         'fields'         => 'ids',
         'orderby'        => $args['orderby'],
         'order'          => $args['order'],
-        'paged'          => absint( $args['page'] ),
+        'paged'          => absint($args['page']),
     );
 
-    if ( ! empty( $args['exclude'] ) ) {
-        $wp_query_args['post__not_in'] = array_map( 'absint', $args['exclude'] );
+    if (!empty($args['exclude'])) {
+        $wp_query_args['post__not_in'] = array_map('absint', $args['exclude']);
     }
 
-    if ( ! $args['paginate' ] ) {
+    if (!$args['paginate']) {
         $wp_query_args['no_found_rows'] = true;
     }
 
-    if ( ! empty( $args['search'] ) ) {
+    if (!empty($args['search'])) {
         $wp_query_args['s'] = $args['search'];
     }
 
-    if ( ! empty( $args['type'] ) && $args['type'] !== wpinv_item_types() ) {
-        $types = wpinv_parse_list( $args['type'] );
+    if (!empty($args['type']) && $args['type'] !== wpinv_item_types()) {
+        $types = wpinv_parse_list($args['type']);
         $wp_query_args['meta_query'][] = array(
             'key'     => '_wpinv_type',
-            'value'   => implode( ',', $types ),
+            'value'   => implode(',', $types),
             'compare' => 'IN',
         );
     }
@@ -102,17 +102,17 @@ 
     $wp_query_args = apply_filters('wpinv_get_items_args', $wp_query_args, $args);
 
     // Get results.
-    $items = new WP_Query( $wp_query_args );
+    $items = new WP_Query($wp_query_args);
 
-    if ( 'objects' === $args['return'] ) {
-        $return = array_map( 'wpinv_get_item_by_id', $items->posts );
-    } elseif ( 'self' === $args['return'] ) {
+    if ('objects' === $args['return']) {
+        $return = array_map('wpinv_get_item_by_id', $items->posts);
+    } elseif ('self' === $args['return']) {
         return $items;
     } else {
         $return = $items->posts;
     }
 
-    if ( $args['paginate' ] ) {
+    if ($args['paginate']) {
         return (object) array(
             'items'      => $return,
             'total'         => $items->found_posts,
@@ -124,12 +124,12 @@ 
 
 }
 
-function wpinv_is_free_item( $item_id = 0 ) {
-    if( empty( $item_id ) ) {
+function wpinv_is_free_item($item_id = 0) {
+    if (empty($item_id)) {
         return false;
     }
 
-    $item = new WPInv_Item( $item_id );
+    $item = new WPInv_Item($item_id);
 
     return $item->is_free();
 }
@@ -139,21 +139,21 @@ 
  *
  * @param WP_Post|WPInv_Item|Int $item The item to check for.
  */
-function wpinv_item_is_editable( $item = 0 ) {
+function wpinv_item_is_editable($item = 0) {
 
     // Fetch the item.
-    $item = new WPInv_Item( $item );
+    $item = new WPInv_Item($item);
 
     // Check if it is editable.
     return $item->is_editable();
 }
 
-function wpinv_get_item_price( $item_id = 0 ) {
-    if( empty( $item_id ) ) {
+function wpinv_get_item_price($item_id = 0) {
+    if (empty($item_id)) {
         return false;
     }
 
-    $item = new WPInv_Item( $item_id );
+    $item = new WPInv_Item($item_id);
 
     return $item->get_price();
 }
@@ -163,96 +163,96 @@ 
  *
  * @param WPInv_Item|int $item
  */
-function wpinv_is_recurring_item( $item = 0 ) {
-    $item = new WPInv_Item( $item );
+function wpinv_is_recurring_item($item = 0) {
+    $item = new WPInv_Item($item);
     return $item->is_recurring();
 }
 
-function wpinv_item_price( $item_id = 0 ) {
-    if( empty( $item_id ) ) {
+function wpinv_item_price($item_id = 0) {
+    if (empty($item_id)) {
         return false;
     }
 
-    $price = wpinv_get_item_price( $item_id );
-    $price = wpinv_price( $price );
+    $price = wpinv_get_item_price($item_id);
+    $price = wpinv_price($price);
 
-    return apply_filters( 'wpinv_item_price', $price, $item_id );
+    return apply_filters('wpinv_item_price', $price, $item_id);
 }
 
-function wpinv_get_item_final_price( $item_id = 0, $amount_override = null ) {
-    if ( is_null( $amount_override ) ) {
-        $original_price = get_post_meta( $item_id, '_wpinv_price', true );
+function wpinv_get_item_final_price($item_id = 0, $amount_override = null) {
+    if (is_null($amount_override)) {
+        $original_price = get_post_meta($item_id, '_wpinv_price', true);
     } else {
         $original_price = $amount_override;
     }
 
     $price = $original_price;
 
-    return apply_filters( 'wpinv_get_item_final_price', $price, $item_id );
+    return apply_filters('wpinv_get_item_final_price', $price, $item_id);
 }
 
-function wpinv_item_custom_singular_name( $item_id ) {
-    if( empty( $item_id ) ) {
+function wpinv_item_custom_singular_name($item_id) {
+    if (empty($item_id)) {
         return false;
     }
 
-    $item = new WPInv_Item( $item_id );
+    $item = new WPInv_Item($item_id);
 
     return $item->get_custom_singular_name();
 }
 
 function wpinv_get_item_types() {
     $item_types = array(
-            'custom'    => __( 'Standard', 'invoicing' ),
-            'fee'       => __( 'Fee', 'invoicing' ),
+            'custom'    => __('Standard', 'invoicing'),
+            'fee'       => __('Fee', 'invoicing'),
         );
-    return apply_filters( 'wpinv_get_item_types', $item_types );
+    return apply_filters('wpinv_get_item_types', $item_types);
 }
 
 function wpinv_item_types() {
     $item_types = wpinv_get_item_types();
 
-    return ( !empty( $item_types ) ? array_keys( $item_types ) : array() );
+    return (!empty($item_types) ? array_keys($item_types) : array());
 }
 
-function wpinv_get_item_type( $item_id ) {
-    if( empty( $item_id ) ) {
+function wpinv_get_item_type($item_id) {
+    if (empty($item_id)) {
         return false;
     }
 
-    $item = new WPInv_Item( $item_id );
+    $item = new WPInv_Item($item_id);
 
     return $item->get_type();
 }
 
-function wpinv_item_type( $item_id ) {
+function wpinv_item_type($item_id) {
     $item_types = wpinv_get_item_types();
 
-    $item_type = wpinv_get_item_type( $item_id );
+    $item_type = wpinv_get_item_type($item_id);
 
-    if ( empty( $item_type ) ) {
+    if (empty($item_type)) {
         $item_type = '-';
     }
 
-    $item_type = isset( $item_types[$item_type] ) ? $item_types[$item_type] : __( $item_type, 'invoicing' );
+    $item_type = isset($item_types[$item_type]) ? $item_types[$item_type] : __($item_type, 'invoicing');
 
-    return apply_filters( 'wpinv_item_type', $item_type, $item_id );
+    return apply_filters('wpinv_item_type', $item_type, $item_id);
 }
 
-function wpinv_get_random_item( $post_ids = true ) {
-    wpinv_get_random_items( 1, $post_ids );
+function wpinv_get_random_item($post_ids = true) {
+    wpinv_get_random_items(1, $post_ids);
 }
 
-function wpinv_get_random_items( $num = 3, $post_ids = true ) {
-    if ( $post_ids ) {
-        $args = array( 'post_type' => 'wpi_item', 'orderby' => 'rand', 'post_count' => $num, 'fields' => 'ids' );
+function wpinv_get_random_items($num = 3, $post_ids = true) {
+    if ($post_ids) {
+        $args = array('post_type' => 'wpi_item', 'orderby' => 'rand', 'post_count' => $num, 'fields' => 'ids');
     } else {
-        $args = array( 'post_type' => 'wpi_item', 'orderby' => 'rand', 'post_count' => $num );
+        $args = array('post_type' => 'wpi_item', 'orderby' => 'rand', 'post_count' => $num);
     }
 
-    $args  = apply_filters( 'wpinv_get_random_items', $args );
+    $args = apply_filters('wpinv_get_random_items', $args);
 
-    return get_posts( $args );
+    return get_posts($args);
 }
 
 /**
@@ -261,13 +261,13 @@ 
  * @param WPInv_Item|int $item
  * @param bool $html
  */
-function wpinv_get_item_suffix( $item, $html = true ) {
+function wpinv_get_item_suffix($item, $html = true) {
 
-    $item   = new WPInv_Item( $item );
-    $suffix = $item->is_recurring() ? ' <span class="wpi-suffix">' . __( '(r)', 'invoicing' ) . '</span>' : '';
-    $suffix = $html ? $suffix : strip_tags( $suffix );
+    $item   = new WPInv_Item($item);
+    $suffix = $item->is_recurring() ? ' <span class="wpi-suffix">' . __('(r)', 'invoicing') . '</span>' : '';
+    $suffix = $html ? $suffix : strip_tags($suffix);
 
-    return apply_filters( 'wpinv_get_item_suffix', $suffix, $item, $html );
+    return apply_filters('wpinv_get_item_suffix', $suffix, $item, $html);
 }
 
 /**
@@ -276,9 +276,9 @@ 
  * @param WPInv_Item|int $item
  * @param bool $force_delete
  */
-function wpinv_remove_item( $item = 0, $force_delete = false ) {
-    $item = new WPInv_Item( $item );
-    $item->delete( $force_delete );
+function wpinv_remove_item($item = 0, $force_delete = false) {
+    $item = new WPInv_Item($item);
+    $item->delete($force_delete);
 }
 
 /**
@@ -317,45 +317,45 @@ 
  * @param bool $wp_error whether or not to return a WP_Error on failure.
  * @return bool|WP_Error|WPInv_Item
  */
-function wpinv_create_item( $args = array(), $wp_error = false ) {
+function wpinv_create_item($args = array(), $wp_error = false) {
 
     // Prepare the item.
-    if ( ! empty( $args['custom_id'] ) && empty( $args['ID'] ) ) {
-        $type = empty( $args['type'] ) ? 'custom' : $args['type'];
-        $item = wpinv_get_item_by( 'custom_id', $args['custom_id'], $type );
+    if (!empty($args['custom_id']) && empty($args['ID'])) {
+        $type = empty($args['type']) ? 'custom' : $args['type'];
+        $item = wpinv_get_item_by('custom_id', $args['custom_id'], $type);
 
-        if ( ! empty( $item ) ) {
+        if (!empty($item)) {
             $args['ID'] = $item->get_id();
         }
 
     }
 
     // Do we have an item?
-    if ( ! empty( $args['ID'] ) ) {
-        $item = new WPInv_Item( $args['ID'] );
+    if (!empty($args['ID'])) {
+        $item = new WPInv_Item($args['ID']);
     } else {
         $item = new WPInv_Item();
     }
 
     // Do we have an error?
-    if ( ! empty( $item->last_error ) ) {
-        return $wp_error ? new WP_Error( 'invalid_item', $item->last_error ) : false;
+    if (!empty($item->last_error)) {
+        return $wp_error ? new WP_Error('invalid_item', $item->last_error) : false;
     }
 
     // Update item props.
-    $item->set_props( $args );
+    $item->set_props($args);
 
     // Save the item.
     $item->save();
 
     // Do we have an error?
-    if ( ! empty( $item->last_error ) ) {
-        return $wp_error ? new WP_Error( 'not_saved', $item->last_error ) : false;
+    if (!empty($item->last_error)) {
+        return $wp_error ? new WP_Error('not_saved', $item->last_error) : false;
     }
 
     // Was the item saved?
-    if ( ! $item->get_id() ) {
-        return $wp_error ? new WP_Error( 'not_saved', __( 'An error occured while saving the item', 'invoicing' ) ) : false;
+    if (!$item->get_id()) {
+        return $wp_error ? new WP_Error('not_saved', __('An error occured while saving the item', 'invoicing')) : false;
     }
 
     return $item;
@@ -367,14 +367,14 @@ 
  *
  * @see wpinv_create_item()
  */
-function wpinv_update_item( $args = array(), $wp_error = false ) {
-    return wpinv_create_item( $args, $wp_error );
+function wpinv_update_item($args = array(), $wp_error = false) {
+    return wpinv_create_item($args, $wp_error);
 }
 
 /**
  * Sanitizes a recurring period
  */
-function getpaid_sanitize_recurring_period( $period, $full = false ) {
+function getpaid_sanitize_recurring_period($period, $full = false) {
 
     $periods = array(
         'D' => 'day',
@@ -383,11 +383,11 @@ 
         'Y' => 'year',
     );
 
-    if ( ! isset( $periods[ $period ] ) ) {
+    if (!isset($periods[$period])) {
         $period = 'D';
     }
 
-    return $full ? $periods[ $period ] : $period;
+    return $full ? $periods[$period] : $period;
 
 }
 
@@ -396,46 +396,46 @@ 
  *
  * @param WPInv_Item|GetPaid_Form_Item $item
  */
-function getpaid_item_recurring_price_help_text( $item, $currency = '', $_initial_price = false, $_recurring_price = false ) {
+function getpaid_item_recurring_price_help_text($item, $currency = '', $_initial_price = false, $_recurring_price = false) {
 
     // Abort if it is not recurring.
-    if ( ! $item->is_recurring() ) {
+    if (!$item->is_recurring()) {
         return '';
     }
 
-    $initial_price   = false === $_initial_price ? wpinv_price( $item->get_initial_price(), $currency ) : $_initial_price;
-    $recurring_price = false === $_recurring_price ? wpinv_price( $item->get_recurring_price(), $currency ) : $_recurring_price;
-    $period          = getpaid_get_subscription_period_label( $item->get_recurring_period(), $item->get_recurring_interval(), '' );
+    $initial_price   = false === $_initial_price ? wpinv_price($item->get_initial_price(), $currency) : $_initial_price;
+    $recurring_price = false === $_recurring_price ? wpinv_price($item->get_recurring_price(), $currency) : $_recurring_price;
+    $period          = getpaid_get_subscription_period_label($item->get_recurring_period(), $item->get_recurring_interval(), '');
     $initial_class   = 'getpaid-item-initial-price';
     $recurring_class = 'getpaid-item-recurring-price';
     $bill_times      = $item->get_recurring_limit();
 
-    if ( ! empty( $bill_times ) ) {
+    if (!empty($bill_times)) {
 		$bill_times = $item->get_recurring_interval() * $bill_times;
-		$bill_times = getpaid_get_subscription_period_label( $item->get_recurring_period(), $bill_times );
+		$bill_times = getpaid_get_subscription_period_label($item->get_recurring_period(), $bill_times);
 	}
 
-    if ( $item instanceof GetPaid_Form_Item && false === $_initial_price ) {
-        $initial_price   = wpinv_price( $item->get_sub_total(), $currency );
-        $recurring_price = wpinv_price( $item->get_recurring_sub_total(), $currency );
+    if ($item instanceof GetPaid_Form_Item && false === $_initial_price) {
+        $initial_price   = wpinv_price($item->get_sub_total(), $currency);
+        $recurring_price = wpinv_price($item->get_recurring_sub_total(), $currency);
     }
 
-    if ( wpinv_price( 0, $currency ) == $initial_price && wpinv_price( 0, $currency ) == $recurring_price ) {
-        return __( 'Free forever', 'invoicing' );
+    if (wpinv_price(0, $currency) == $initial_price && wpinv_price(0, $currency) == $recurring_price) {
+        return __('Free forever', 'invoicing');
     }
 
     // For free trial items.
-    if ( $item->has_free_trial() ) {
-        $trial_period = getpaid_get_subscription_period_label( $item->get_trial_period(), $item->get_trial_interval() );
+    if ($item->has_free_trial()) {
+        $trial_period = getpaid_get_subscription_period_label($item->get_trial_period(), $item->get_trial_interval());
 
-        if ( wpinv_price( 0, $currency ) == $initial_price ) {
+        if (wpinv_price(0, $currency) == $initial_price) {
 
-            if ( empty( $bill_times ) ) {
+            if (empty($bill_times)) {
 
                 return sprintf(
 
                     // translators: $1: is the trial period, $2: is the recurring price, $3: is the susbcription period
-                    _x( 'Free for %1$s then %2$s / %3$s', 'Item subscription amount. (e.g.: Free for 1 month then $120 / year)', 'invoicing' ),
+                    _x('Free for %1$s then %2$s / %3$s', 'Item subscription amount. (e.g.: Free for 1 month then $120 / year)', 'invoicing'),
                     "<span class='getpaid-item-trial-period'>$trial_period</span>",
                     "<span class='$recurring_class'>$recurring_price</span>",
                     "<span class='getpaid-item-recurring-period'>$period</span>"
@@ -447,7 +447,7 @@ 
             return sprintf(
 
                 // translators: $1: is the trial period, $2: is the recurring price, $3: is the susbcription period, $4: is the bill times
-                _x( 'Free for %1$s then %2$s / %3$s for %4$s', 'Item subscription amount. (e.g.: Free for 1 month then $120 / year for 4 years)', 'invoicing' ),
+                _x('Free for %1$s then %2$s / %3$s for %4$s', 'Item subscription amount. (e.g.: Free for 1 month then $120 / year for 4 years)', 'invoicing'),
                 "<span class='getpaid-item-trial-period'>$trial_period</span>",
                 "<span class='$recurring_class'>$recurring_price</span>",
                 "<span class='getpaid-item-recurring-period'>$period</span>",
@@ -457,12 +457,12 @@ 
 
         }
 
-        if ( empty( $bill_times ) ) {
+        if (empty($bill_times)) {
 
             return sprintf(
 
                 // translators: $1: is the initial price, $2: is the trial period, $3: is the recurring price, $4: is the susbcription period
-                _x( '%1$s for %2$s then %3$s / %4$s', 'Item subscription amount. (e.g.: $7 for 1 month then $120 / year)', 'invoicing' ),
+                _x('%1$s for %2$s then %3$s / %4$s', 'Item subscription amount. (e.g.: $7 for 1 month then $120 / year)', 'invoicing'),
                 "<span class='$initial_class'>$initial_price</span>",
                 "<span class='getpaid-item-trial-period'>$trial_period</span>",
                 "<span class='$recurring_class'>$recurring_price</span>",
@@ -475,7 +475,7 @@ 
         return sprintf(
 
             // translators: $1: is the initial price, $2: is the trial period, $3: is the recurring price, $4: is the susbcription period, $4: is the susbcription bill times
-            _x( '%1$s for %2$s then %3$s / %4$s for %5$s', 'Item subscription amount. (e.g.: $7 for 1 month then $120 / year for 5 years)', 'invoicing' ),
+            _x('%1$s for %2$s then %3$s / %4$s for %5$s', 'Item subscription amount. (e.g.: $7 for 1 month then $120 / year for 5 years)', 'invoicing'),
             "<span class='$initial_class'>$initial_price</span>",
             "<span class='getpaid-item-trial-period'>$trial_period</span>",
             "<span class='$recurring_class'>$recurring_price</span>",
@@ -486,14 +486,14 @@ 
 
     }
 
-    if ( $initial_price == $recurring_price ) {
+    if ($initial_price == $recurring_price) {
 
-        if ( empty( $bill_times ) ) {
+        if (empty($bill_times)) {
 
             return sprintf(
 
                 // translators: $1: is the recurring price, $2: is the susbcription period
-                _x( '%1$s / %2$s', 'Item subscription amount. (e.g.: $120 / year)', 'invoicing' ),
+                _x('%1$s / %2$s', 'Item subscription amount. (e.g.: $120 / year)', 'invoicing'),
                 "<span class='$recurring_class'>$recurring_price</span>",
                 "<span class='getpaid-item-recurring-period'>$period</span>"
 
@@ -504,7 +504,7 @@ 
         return sprintf(
 
             // translators: $1: is the recurring price, $2: is the susbcription period, $3: is the susbcription bill times
-            _x( '%1$s / %2$s for %3$s', 'Item subscription amount. (e.g.: $120 / year for 5 years)', 'invoicing' ),
+            _x('%1$s / %2$s for %3$s', 'Item subscription amount. (e.g.: $120 / year for 5 years)', 'invoicing'),
             "<span class='$recurring_class'>$recurring_price</span>",
             "<span class='getpaid-item-recurring-period'>$period</span>",
             "<span class='getpaid-item-recurring-bill-times'>$bill_times</span>"
@@ -513,14 +513,14 @@ 
 
     }
 
-    if ( $initial_price == wpinv_price( 0, $currency ) ) {
+    if ($initial_price == wpinv_price(0, $currency)) {
 
-        if ( empty( $bill_times ) ) {
+        if (empty($bill_times)) {
 
             return sprintf(
 
                 // translators: $1: is the recurring period, $2: is the recurring price
-                _x( 'Free for %1$s then %2$s / %1$s', 'Item subscription amount. (e.g.: Free for 3 months then $7 / 3 months)', 'invoicing' ),
+                _x('Free for %1$s then %2$s / %1$s', 'Item subscription amount. (e.g.: Free for 3 months then $7 / 3 months)', 'invoicing'),
                 "<span class='getpaid-item-recurring-period'>$period</span>",
                 "<span class='$recurring_class'>$recurring_price</span>"
 
@@ -531,7 +531,7 @@ 
         return sprintf(
 
             // translators: $1: is the recurring period, $2: is the recurring price, $3: is the bill times
-            _x( 'Free for %1$s then %2$s / %1$s for %3$s', 'Item subscription amount. (e.g.: Free for 3 months then $7 / 3 months for 12 months)', 'invoicing' ),
+            _x('Free for %1$s then %2$s / %1$s for %3$s', 'Item subscription amount. (e.g.: Free for 3 months then $7 / 3 months for 12 months)', 'invoicing'),
             "<span class='getpaid-item-recurring-period'>$period</span>",
             "<span class='$recurring_class'>$recurring_price</span>",
             "<span class='getpaid-item-recurring-bill-times'>$bill_times</span>"
@@ -540,12 +540,12 @@ 
 
     }
 
-    if ( empty( $bill_times ) ) {
+    if (empty($bill_times)) {
 
         return sprintf(
 
             // translators: $1: is the initial price, $2: is the recurring price, $3: is the susbcription period
-            _x( 'Initial payment of %1$s then %2$s / %3$s', 'Item subscription amount. (e.g.: Initial payment of $7 then $120 / year)', 'invoicing' ),
+            _x('Initial payment of %1$s then %2$s / %3$s', 'Item subscription amount. (e.g.: Initial payment of $7 then $120 / year)', 'invoicing'),
             "<span class='$initial_class'>$initial_price</span>",
             "<span class='$recurring_class'>$recurring_price</span>",
             "<span class='getpaid-item-recurring-period'>$period</span>"
@@ -557,7 +557,7 @@ 
     return sprintf(
 
         // translators: $1: is the initial price, $2: is the recurring price, $3: is the susbcription period, $4: is the susbcription bill times
-        _x( 'Initial payment of %1$s then %2$s / %3$s for %4$s', 'Item subscription amount. (e.g.: Initial payment of $7 then $120 / year for 4 years)', 'invoicing' ),
+        _x('Initial payment of %1$s then %2$s / %3$s for %4$s', 'Item subscription amount. (e.g.: Initial payment of $7 then $120 / year for 4 years)', 'invoicing'),
         "<span class='$initial_class'>$initial_price</span>",
         "<span class='$recurring_class'>$recurring_price</span>",
         "<span class='getpaid-item-recurring-period'>$period</span>",