AyeCode / invoicing

includes/gateways/authorizenet.php

Spacing +211 added lines, -211 removed lines

@@ -1,60 +1,60 @@ 
 <?php
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) exit;
+if (!defined('ABSPATH')) exit;
 
-add_filter( 'wpinv_authorizenet_support_subscription', '__return_true' );
+add_filter('wpinv_authorizenet_support_subscription', '__return_true');
 
-function wpinv_authorizenet_cc_form( $invoice_id ) {
-    $invoice = wpinv_get_invoice( $invoice_id );
-    $cc_owner = !empty( $invoice ) ? esc_attr( $invoice->get_user_full_name() ) : '';
+function wpinv_authorizenet_cc_form($invoice_id) {
+    $invoice = wpinv_get_invoice($invoice_id);
+    $cc_owner = !empty($invoice) ? esc_attr($invoice->get_user_full_name()) : '';
     ?>
     <div id="authorizenet_cc_form" class="form-horizontal wpi-cc-form panel panel-default">
-        <div class="panel-heading"><h3 class="panel-title"><?php _e( 'Card Details', 'invoicing' ) ;?></h3></div>
+        <div class="panel-heading"><h3 class="panel-title"><?php _e('Card Details', 'invoicing'); ?></h3></div>
         <div class="panel-body">
             <div class="form-group required">
-              <label for="auth-input-cc-owner" class="col-sm-4 control-label"><?php _e( 'Card Owner', 'invoicing' ) ;?></label>
+              <label for="auth-input-cc-owner" class="col-sm-4 control-label"><?php _e('Card Owner', 'invoicing'); ?></label>
               <div class="col-sm-8">
-                <input type="text" class="form-control" id="auth-input-cc-owner" placeholder="<?php esc_attr_e( 'Card Owner', 'invoicing' ) ;?>" value="<?php echo $cc_owner;?>" name="authorizenet[cc_owner]">
+                <input type="text" class="form-control" id="auth-input-cc-owner" placeholder="<?php esc_attr_e('Card Owner', 'invoicing'); ?>" value="<?php echo $cc_owner; ?>" name="authorizenet[cc_owner]">
               </div>
             </div>
             <div class="form-group required">
-              <label for="auth-input-cc-number" class="col-sm-4 control-label"><?php _e( 'Card Number', 'invoicing' ) ;?></label>
+              <label for="auth-input-cc-number" class="col-sm-4 control-label"><?php _e('Card Number', 'invoicing'); ?></label>
               <div class="col-sm-8">
-                <input type="text" class="form-control" id="auth-input-cc-number" placeholder="<?php esc_attr_e( 'Card Number', 'invoicing' ) ;?>" value="" name="authorizenet[cc_number]">
+                <input type="text" class="form-control" id="auth-input-cc-number" placeholder="<?php esc_attr_e('Card Number', 'invoicing'); ?>" value="" name="authorizenet[cc_number]">
               </div>
             </div>
             <div class="form-group required">
-              <label for="auth-input-cc-expire-date" class="col-sm-4 control-label"><?php _e( 'Card Expiry Date', 'invoicing' ) ;?></label>
+              <label for="auth-input-cc-expire-date" class="col-sm-4 control-label"><?php _e('Card Expiry Date', 'invoicing'); ?></label>
               <div class="col-sm-2">
                 <select class="form-control" id="auth-input-cc-expire-date" name="authorizenet[cc_expire_month]">
-                    <?php for ( $i = 1; $i <= 12; $i++ ) { $value = str_pad( $i, 2, '0', STR_PAD_LEFT ); ?>
-                    <option value="<?php echo $value;?>"><?php echo $value;?></option>
+                    <?php for ($i = 1; $i <= 12; $i++) { $value = str_pad($i, 2, '0', STR_PAD_LEFT); ?>
+                    <option value="<?php echo $value; ?>"><?php echo $value; ?></option>
                     <?php } ?>
                 </select>
                </div>
                <div class="col-sm-2">
                 <select class="form-control" name="authorizenet[cc_expire_year]">
-                    <?php $year = date( 'Y' ); for ( $i = $year; $i <= ( $year + 10 ); $i++ ) { ?>
-                    <option value="<?php echo $i;?>"><?php echo $i;?></option>
+                    <?php $year = date('Y'); for ($i = $year; $i <= ($year + 10); $i++) { ?>
+                    <option value="<?php echo $i; ?>"><?php echo $i; ?></option>
                     <?php } ?>
                 </select>
               </div>
             </div>
             <div class="form-group required">
-              <label for="auth-input-cc-cvv2" class="col-sm-4 control-label"><?php _e( 'Card Security Code (CVV2)', 'invoicing' ) ;?></label>
+              <label for="auth-input-cc-cvv2" class="col-sm-4 control-label"><?php _e('Card Security Code (CVV2)', 'invoicing'); ?></label>
               <div class="col-sm-8">
-                <input type="text" class="form-control" id="auth-input-cc-cvv2" placeholder="<?php esc_attr_e( 'Card Security Code (CVV2)', 'invoicing' ) ;?>" value="" name="authorizenet[cc_cvv2]"">
+                <input type="text" class="form-control" id="auth-input-cc-cvv2" placeholder="<?php esc_attr_e('Card Security Code (CVV2)', 'invoicing'); ?>" value="" name="authorizenet[cc_cvv2]"">
               </div>
             </div>
       </div>
     </div>
     <?php
 }
-add_action( 'wpinv_authorizenet_cc_form', 'wpinv_authorizenet_cc_form', 10, 1 );
+add_action('wpinv_authorizenet_cc_form', 'wpinv_authorizenet_cc_form', 10, 1);
 
-function wpinv_process_authorizenet_payment( $purchase_data ) {
-    if( ! wp_verify_nonce( $purchase_data['gateway_nonce'], 'wpi-gateway' ) ) {
-        wp_die( __( 'Nonce verification has failed', 'invoicing' ), __( 'Error', 'invoicing' ), array( 'response' => 403 ) );
+function wpinv_process_authorizenet_payment($purchase_data) {
+    if (!wp_verify_nonce($purchase_data['gateway_nonce'], 'wpi-gateway')) {
+        wp_die(__('Nonce verification has failed', 'invoicing'), __('Error', 'invoicing'), array('response' => 403));
     }
     
     // Collect payment data
@@ -72,10 +72,10 @@ 
     );
 
     // Record the pending payment
-    $invoice = wpinv_get_invoice( $purchase_data['invoice_id'] );
+    $invoice = wpinv_get_invoice($purchase_data['invoice_id']);
 
-    if ( !empty( $invoice ) ) {
-        $authorizenet_card  = !empty( $_POST['authorizenet'] ) ? $_POST['authorizenet'] : array();
+    if (!empty($invoice)) {
+        $authorizenet_card  = !empty($_POST['authorizenet']) ? $_POST['authorizenet'] : array();
         $card_defaults      = array(
             'cc_owner'          => $invoice->get_user_full_name(),
             'cc_number'         => false,
@@ -83,174 +83,174 @@ 
             'cc_expire_year'    => false,
             'cc_cvv2'           => false,
         );
-        $authorizenet_card = wp_parse_args( $authorizenet_card, $card_defaults );
+        $authorizenet_card = wp_parse_args($authorizenet_card, $card_defaults);
         
-        if ( empty( $authorizenet_card['cc_owner'] ) ) {
-            wpinv_set_error( 'empty_card_name', __( 'You must enter the name on your card!', 'invoicing'));
+        if (empty($authorizenet_card['cc_owner'])) {
+            wpinv_set_error('empty_card_name', __('You must enter the name on your card!', 'invoicing'));
         }
-        if ( empty( $authorizenet_card['cc_number'] ) ) {
-            wpinv_set_error( 'empty_card', __( 'You must enter a card number!', 'invoicing'));
+        if (empty($authorizenet_card['cc_number'])) {
+            wpinv_set_error('empty_card', __('You must enter a card number!', 'invoicing'));
         }
-        if ( empty( $authorizenet_card['cc_expire_month'] ) ) {
-            wpinv_set_error( 'empty_month', __( 'You must enter an card expiration month!', 'invoicing'));
+        if (empty($authorizenet_card['cc_expire_month'])) {
+            wpinv_set_error('empty_month', __('You must enter an card expiration month!', 'invoicing'));
         }
-        if ( empty( $authorizenet_card['cc_expire_year'] ) ) {
-            wpinv_set_error( 'empty_year', __( 'You must enter an card expiration year!', 'invoicing'));
+        if (empty($authorizenet_card['cc_expire_year'])) {
+            wpinv_set_error('empty_year', __('You must enter an card expiration year!', 'invoicing'));
         }
-        if ( empty( $authorizenet_card['cc_cvv2'] ) ) {
-            wpinv_set_error( 'empty_cvv2', __( 'You must enter a valid CVV2!', 'invoicing' ) );
+        if (empty($authorizenet_card['cc_cvv2'])) {
+            wpinv_set_error('empty_cvv2', __('You must enter a valid CVV2!', 'invoicing'));
         }
         
         $errors = wpinv_get_errors();
     
-        if ( empty( $errors ) ) {
+        if (empty($errors)) {
             $invoice_id = $invoice->ID;
             $quantities_enabled = wpinv_item_quantities_enabled();
             $use_taxes          = wpinv_use_taxes();
             
             $authorizeAIM = wpinv_authorizenet_AIM();
-            $authorizeAIM->first_name       = wpinv_utf8_substr( $invoice->get_first_name(), 0, 50 );
-            $authorizeAIM->last_name        = wpinv_utf8_substr( $invoice->get_last_name(), 0, 50 );
-            $authorizeAIM->company          = wpinv_utf8_substr( $invoice->company, 0, 50 );
-            $authorizeAIM->address          = wpinv_utf8_substr( wp_strip_all_tags( $invoice->get_address(), true ), 0, 60 );
-            $authorizeAIM->city             = wpinv_utf8_substr( $invoice->city, 0, 40 );
-            $authorizeAIM->state            = wpinv_utf8_substr( $invoice->state, 0, 40 );
-            $authorizeAIM->zip              = wpinv_utf8_substr( $invoice->zip, 0, 40 );
-            $authorizeAIM->country          = wpinv_utf8_substr( $invoice->country, 0, 60 );
-            $authorizeAIM->phone            = wpinv_utf8_substr( $invoice->phone, 0, 25 );
-            $authorizeAIM->email            = wpinv_utf8_substr( $invoice->get_email(), 0, 255 );
-            $authorizeAIM->amount           = wpinv_sanitize_amount( $invoice->get_total() );
-            $authorizeAIM->card_num         = str_replace( ' ', '', sanitize_text_field( $authorizenet_card['cc_number'] ) );
-            $authorizeAIM->exp_date         = sanitize_text_field( $authorizenet_card['cc_expire_month'] ) . sanitize_text_field( $authorizenet_card['cc_expire_year'] );
-            $authorizeAIM->card_code        = sanitize_text_field( $authorizenet_card['cc_cvv2'] );
+            $authorizeAIM->first_name       = wpinv_utf8_substr($invoice->get_first_name(), 0, 50);
+            $authorizeAIM->last_name        = wpinv_utf8_substr($invoice->get_last_name(), 0, 50);
+            $authorizeAIM->company          = wpinv_utf8_substr($invoice->company, 0, 50);
+            $authorizeAIM->address          = wpinv_utf8_substr(wp_strip_all_tags($invoice->get_address(), true), 0, 60);
+            $authorizeAIM->city             = wpinv_utf8_substr($invoice->city, 0, 40);
+            $authorizeAIM->state            = wpinv_utf8_substr($invoice->state, 0, 40);
+            $authorizeAIM->zip              = wpinv_utf8_substr($invoice->zip, 0, 40);
+            $authorizeAIM->country          = wpinv_utf8_substr($invoice->country, 0, 60);
+            $authorizeAIM->phone            = wpinv_utf8_substr($invoice->phone, 0, 25);
+            $authorizeAIM->email            = wpinv_utf8_substr($invoice->get_email(), 0, 255);
+            $authorizeAIM->amount           = wpinv_sanitize_amount($invoice->get_total());
+            $authorizeAIM->card_num         = str_replace(' ', '', sanitize_text_field($authorizenet_card['cc_number']));
+            $authorizeAIM->exp_date         = sanitize_text_field($authorizenet_card['cc_expire_month']) . sanitize_text_field($authorizenet_card['cc_expire_year']);
+            $authorizeAIM->card_code        = sanitize_text_field($authorizenet_card['cc_cvv2']);
             $authorizeAIM->invoice_num      = $invoice->ID;
             
             $item_desc = array();
-            foreach ( $invoice->get_cart_details() as $item ) {            
-                $quantity       = $quantities_enabled && !empty( $item['quantity'] ) && $item['quantity'] > 0 ? $item['quantity'] : 1;
-                $item_name      = wpinv_utf8_substr( $item['name'], 0, 31 );
-                $item_desc[]    = $item_name . ' (' . $quantity . 'x ' . wpinv_price( wpinv_format_amount( $item['item_price'] ) ) . ')';
+            foreach ($invoice->get_cart_details() as $item) {            
+                $quantity       = $quantities_enabled && !empty($item['quantity']) && $item['quantity'] > 0 ? $item['quantity'] : 1;
+                $item_name      = wpinv_utf8_substr($item['name'], 0, 31);
+                $item_desc[]    = $item_name . ' (' . $quantity . 'x ' . wpinv_price(wpinv_format_amount($item['item_price'])) . ')';
                 
-                $authorizeAIM->addLineItem( $item['id'], $item_name, '', $quantity, $item['item_price'], ( $use_taxes && !empty( $item['tax'] ) && $item['tax'] > 0 ? 'Y' : 'N' ) );
+                $authorizeAIM->addLineItem($item['id'], $item_name, '', $quantity, $item['item_price'], ($use_taxes && !empty($item['tax']) && $item['tax'] > 0 ? 'Y' : 'N'));
             }
             
-            $item_desc = '#' . $invoice->get_number() . ': ' . implode( ', ', $item_desc );
+            $item_desc = '#' . $invoice->get_number() . ': ' . implode(', ', $item_desc);
             
-            if ( $use_taxes && $invoice->get_tax() > 0 ) {
-                $authorizeAIM->tax  = $invoice->get_tax();
+            if ($use_taxes && $invoice->get_tax() > 0) {
+                $authorizeAIM->tax = $invoice->get_tax();
                 
-                $item_desc .= ', ' . wp_sprintf( __( 'Tax: %s', 'invoicing' ), $invoice->get_tax( true ) );
+                $item_desc .= ', ' . wp_sprintf(__('Tax: %s', 'invoicing'), $invoice->get_tax(true));
             }
             
-            if ( $invoice->get_discount() > 0 ) {
-                $item_desc .= ', ' . wp_sprintf( __( 'Discount: %s', 'invoicing' ), $invoice->get_discount( true ) );
+            if ($invoice->get_discount() > 0) {
+                $item_desc .= ', ' . wp_sprintf(__('Discount: %s', 'invoicing'), $invoice->get_discount(true));
             }
             
-            $item_description = wpinv_utf8_substr( $item_desc, 0, 255 );
-            $item_description = html_entity_decode( $item_desc , ENT_QUOTES, 'UTF-8' );
+            $item_description = wpinv_utf8_substr($item_desc, 0, 255);
+            $item_description = html_entity_decode($item_desc, ENT_QUOTES, 'UTF-8');
             
-            $authorizeAIM->description  = wpinv_utf8_substr( $item_description, 0, 255 );
+            $authorizeAIM->description = wpinv_utf8_substr($item_description, 0, 255);
             
             $is_recurring = $invoice->is_recurring(); // Recurring payment.
             
-            if ( $is_recurring ) {
+            if ($is_recurring) {
                 $authorizeAIM->recurring_billing = true;
             }
             
             try {
-                if ( $is_recurring ) {
+                if ($is_recurring) {
                     $response = $authorizeAIM->authorizeOnly();
                 } else {
                     $response = $authorizeAIM->authorizeAndCapture();
                 }
                 
-                if ( $response->approved || $response->held ) {
-                    if ( $response->approved ) {
-                        wpinv_update_payment_status( $invoice_id, 'publish' );
+                if ($response->approved || $response->held) {
+                    if ($response->approved) {
+                        wpinv_update_payment_status($invoice_id, 'publish');
                     }
-                    wpinv_set_payment_transaction_id( $invoice_id, $response->transaction_id );
+                    wpinv_set_payment_transaction_id($invoice_id, $response->transaction_id);
                     
-                    $message = wp_sprintf( __( 'Authorize.Net Payment: %s with transaction id %s using %s and authorization code %s', 'invoicing' ), $response->response_reason_text, $response->transaction_id, strtoupper( $response->transaction_type ), $response->authorization_code );
+                    $message = wp_sprintf(__('Authorize.Net Payment: %s with transaction id %s using %s and authorization code %s', 'invoicing'), $response->response_reason_text, $response->transaction_id, strtoupper($response->transaction_type), $response->authorization_code);
                     
-                    wpinv_insert_payment_note( $invoice_id, $message );
+                    wpinv_insert_payment_note($invoice_id, $message);
                     
-                    do_action( 'wpinv_authorizenet_handle_response', $response, $invoice, $authorizenet_card );
+                    do_action('wpinv_authorizenet_handle_response', $response, $invoice, $authorizenet_card);
                     
                     wpinv_clear_errors();
                     wpinv_empty_cart();
                     
-                    wpinv_send_to_success_page( array( 'invoice_key' => $invoice->get_key() ) );
+                    wpinv_send_to_success_page(array('invoice_key' => $invoice->get_key()));
                 } else {
-                    if ( !empty( $response->response_reason_text ) ) {
-                        $error = __( $response->response_reason_text, 'invoicing' );
-                    } else if ( !empty( $response->error_message ) ) {
-                        $error = __( $response->error_message, 'invoicing' );
+                    if (!empty($response->response_reason_text)) {
+                        $error = __($response->response_reason_text, 'invoicing');
+                    } else if (!empty($response->error_message)) {
+                        $error = __($response->error_message, 'invoicing');
                     } else {
-                        $error = wp_sprintf( __( 'Error data: %s', 'invoicing' ), print_r( $response, true ) );
+                        $error = wp_sprintf(__('Error data: %s', 'invoicing'), print_r($response, true));
                     } 
                     
-                    $error = wp_sprintf( __( 'Authorize.Net payment error occurred. %s', 'invoicing' ), $error );
+                    $error = wp_sprintf(__('Authorize.Net payment error occurred. %s', 'invoicing'), $error);
                     
-                    wpinv_set_error( 'payment_error', $error );
-                    wpinv_record_gateway_error( $error, $response );
-                    wpinv_insert_payment_note( $invoice_id, $error );
+                    wpinv_set_error('payment_error', $error);
+                    wpinv_record_gateway_error($error, $response);
+                    wpinv_insert_payment_note($invoice_id, $error);
                     
-                    wpinv_send_back_to_checkout( '?payment-mode=' . $purchase_data['post_data']['wpi-gateway'] );
+                    wpinv_send_back_to_checkout('?payment-mode=' . $purchase_data['post_data']['wpi-gateway']);
                 }
-            } catch ( AuthorizeNetException $e ) {
-                wpinv_set_error( 'request_error', $e->getMessage() );
-                wpinv_record_gateway_error( wp_sprintf( __( 'Authorize.Net payment error occurred. %s', 'invoicing' ), $e->getMessage() ) );
-                wpinv_send_back_to_checkout( '?payment-mode=' . $purchase_data['post_data']['wpi-gateway'] );
+            } catch (AuthorizeNetException $e) {
+                wpinv_set_error('request_error', $e->getMessage());
+                wpinv_record_gateway_error(wp_sprintf(__('Authorize.Net payment error occurred. %s', 'invoicing'), $e->getMessage()));
+                wpinv_send_back_to_checkout('?payment-mode=' . $purchase_data['post_data']['wpi-gateway']);
             }
         } else {
-            wpinv_send_back_to_checkout( '?payment-mode=' . $purchase_data['post_data']['wpi-gateway'] );
+            wpinv_send_back_to_checkout('?payment-mode=' . $purchase_data['post_data']['wpi-gateway']);
         }
     } else {
-        wpinv_record_gateway_error( wp_sprintf( __( 'Authorize.Net payment error occurred. Payment creation failed while processing a Authorize.net payment. Payment data: %s', 'invoicing' ), print_r( $payment_data, true ) ), $invoice );
-        wpinv_send_back_to_checkout( '?payment-mode=' . $purchase_data['post_data']['wpi-gateway'] );
+        wpinv_record_gateway_error(wp_sprintf(__('Authorize.Net payment error occurred. Payment creation failed while processing a Authorize.net payment. Payment data: %s', 'invoicing'), print_r($payment_data, true)), $invoice);
+        wpinv_send_back_to_checkout('?payment-mode=' . $purchase_data['post_data']['wpi-gateway']);
     }
 }
-add_action( 'wpinv_gateway_authorizenet', 'wpinv_process_authorizenet_payment' );
+add_action('wpinv_gateway_authorizenet', 'wpinv_process_authorizenet_payment');
 
-function wpinv_authorizenet_cancel_subscription( $subscription_id = '' ) {
-    if ( empty( $subscription_id ) ) {
+function wpinv_authorizenet_cancel_subscription($subscription_id = '') {
+    if (empty($subscription_id)) {
         return false;
     }
     
     try {
         $authnetXML = wpinv_authorizenet_XML();
-        $authnetXML->ARBCancelSubscriptionRequest( array( 'subscriptionId' => $subscription_id ) );
+        $authnetXML->ARBCancelSubscriptionRequest(array('subscriptionId' => $subscription_id));
 
         return $authnetXML->isSuccessful();
-    } catch( Exception $e ) {
-        wpinv_error_log( $e->getMessage(), __( 'Authorize.Net cancel subscription', 'invoicing' ) );
+    } catch (Exception $e) {
+        wpinv_error_log($e->getMessage(), __('Authorize.Net cancel subscription', 'invoicing'));
     }
     
     return false;
 }
 
-function wpinv_authorizenet_valid_ipn( $md5_hash, $transaction_id, $amount ) {
-    $authorizenet_md5_hash = wpinv_get_option( 'authorizenet_md5_hash' );
-    if ( empty( $authorizenet_md5_hash ) ) {
+function wpinv_authorizenet_valid_ipn($md5_hash, $transaction_id, $amount) {
+    $authorizenet_md5_hash = wpinv_get_option('authorizenet_md5_hash');
+    if (empty($authorizenet_md5_hash)) {
         return true;
     }
     
-    $compare_md5 = strtoupper( md5( $authorizenet_md5_hash . $transaction_id . $amount ) );
+    $compare_md5 = strtoupper(md5($authorizenet_md5_hash . $transaction_id . $amount));
     
-    return hash_equals( $compare_md5, $md5_hash );
+    return hash_equals($compare_md5, $md5_hash);
 }
 
 function wpinv_authorizenet_AIM() {
-    if ( !class_exists( 'AuthorizeNetException' ) ) {
-        require_once plugin_dir_path( WPINV_PLUGIN_FILE ) . 'includes/gateways/authorizenet/anet_php_sdk/AuthorizeNet.php';
+    if (!class_exists('AuthorizeNetException')) {
+        require_once plugin_dir_path(WPINV_PLUGIN_FILE) . 'includes/gateways/authorizenet/anet_php_sdk/AuthorizeNet.php';
     }
     
-    $authorizeAIM = new AuthorizeNetAIM( wpinv_get_option( 'authorizenet_login_id' ), wpinv_get_option( 'authorizenet_transaction_key' ) );
+    $authorizeAIM = new AuthorizeNetAIM(wpinv_get_option('authorizenet_login_id'), wpinv_get_option('authorizenet_transaction_key'));
     
-    if ( wpinv_is_test_mode( 'authorizenet' ) ) {
-        $authorizeAIM->setSandbox( true );
+    if (wpinv_is_test_mode('authorizenet')) {
+        $authorizeAIM->setSandbox(true);
     } else {
-        $authorizeAIM->setSandbox( false );
+        $authorizeAIM->setSandbox(false);
     }
     
     $authorizeAIM->customer_ip = wpinv_get_ip();
@@ -259,164 +259,164 @@ 
 }
 
 function wpinv_authorizenet_XML() {
-    if ( !class_exists( 'AuthnetXML' ) ) {
-        require_once plugin_dir_path( WPINV_PLUGIN_FILE ) . 'includes/gateways/authorizenet/Authorize.Net-XML/AuthnetXML.class.php';
+    if (!class_exists('AuthnetXML')) {
+        require_once plugin_dir_path(WPINV_PLUGIN_FILE) . 'includes/gateways/authorizenet/Authorize.Net-XML/AuthnetXML.class.php';
     }
     
-    $authnetXML = new AuthnetXML( wpinv_get_option( 'authorizenet_login_id' ), wpinv_get_option( 'authorizenet_transaction_key' ), (bool)wpinv_is_test_mode( 'authorizenet' ) );
+    $authnetXML = new AuthnetXML(wpinv_get_option('authorizenet_login_id'), wpinv_get_option('authorizenet_transaction_key'), (bool)wpinv_is_test_mode('authorizenet'));
     
     return $authnetXML;
 }
 
-function wpinv_authorizenet_handle_response( $response, $invoice, $card_info = array() ) {
-    if ( empty( $response ) || empty( $invoice ) ) {
+function wpinv_authorizenet_handle_response($response, $invoice, $card_info = array()) {
+    if (empty($response) || empty($invoice)) {
         return false;
     }
     
-    if ( $invoice->is_recurring() && !empty( $response->approved ) ) {
-        $subscription = wpinv_authorizenet_create_new_subscription( $invoice, $response, $card_info );
+    if ($invoice->is_recurring() && !empty($response->approved)) {
+        $subscription = wpinv_authorizenet_create_new_subscription($invoice, $response, $card_info);
 
-        if ( !empty( $subscription ) && $subscription->isSuccessful() ) {
-            do_action( 'wpinv_recurring_post_create_subscription', $subscription, $invoice, 'authorizenet' );
+        if (!empty($subscription) && $subscription->isSuccessful()) {
+            do_action('wpinv_recurring_post_create_subscription', $subscription, $invoice, 'authorizenet');
             
-            wpinv_authorizenet_subscription_record_signup( $subscription, $invoice );
+            wpinv_authorizenet_subscription_record_signup($subscription, $invoice);
             
-            do_action( 'wpinv_recurring_post_record_signup', $subscription, $invoice, 'authorizenet' );
+            do_action('wpinv_recurring_post_record_signup', $subscription, $invoice, 'authorizenet');
         } else {
-            if ( isset( $subscription->messages->message ) ) {
+            if (isset($subscription->messages->message)) {
                 $error = $subscription->messages->message->code . ': ' . $subscription->messages->message->text;
-                wpinv_set_error( 'wpinv_authorize_recurring_error', $error, 'invoicing' );
+                wpinv_set_error('wpinv_authorize_recurring_error', $error, 'invoicing');
             } else {
-                $error = __( 'Your subscription cannot be created due to an error.', 'invoicing' );
-                wpinv_set_error( 'wpinv_authorize_recurring_error', $error );
+                $error = __('Your subscription cannot be created due to an error.', 'invoicing');
+                wpinv_set_error('wpinv_authorize_recurring_error', $error);
             }
             
-            wpinv_record_gateway_error( $error, $subscription );
+            wpinv_record_gateway_error($error, $subscription);
             
-            wpinv_insert_payment_note( $invoice->ID, wp_sprintf( __( 'Authorize.Net subscription error occurred. %s', 'invoicing' ), $error ) );
+            wpinv_insert_payment_note($invoice->ID, wp_sprintf(__('Authorize.Net subscription error occurred. %s', 'invoicing'), $error));
         }
     }
 }
-add_action( 'wpinv_authorizenet_handle_response', 'wpinv_authorizenet_handle_response', 10, 3 );
+add_action('wpinv_authorizenet_handle_response', 'wpinv_authorizenet_handle_response', 10, 3);
 
-function wpinv_authorizenet_create_new_subscription( $invoice, $response = array(), $card_info = array() ) {
-    if ( empty( $invoice ) ) {
+function wpinv_authorizenet_create_new_subscription($invoice, $response = array(), $card_info = array()) {
+    if (empty($invoice)) {
         return false;
     }
     
-    $params = wpinv_authorizenet_generate_subscription_params( $invoice, $card_info, $response );
+    $params = wpinv_authorizenet_generate_subscription_params($invoice, $card_info, $response);
     
     try {
         $authnetXML = wpinv_authorizenet_XML();
-        $authnetXML->ARBCreateSubscriptionRequest( $params );
-    } catch( Exception $e ) {
+        $authnetXML->ARBCreateSubscriptionRequest($params);
+    } catch (Exception $e) {
         $authnetXML = array();
-        wpinv_error_log( $e->getMessage(), __( 'Authorize.Net cancel subscription', 'invoicing' ) );
+        wpinv_error_log($e->getMessage(), __('Authorize.Net cancel subscription', 'invoicing'));
     }
     
     return $authnetXML;
 }
 
-function wpinv_authorizenet_generate_subscription_params( $invoice, $card_info = array(), $response = array() ) {
-    if ( empty( $invoice ) ) {
+function wpinv_authorizenet_generate_subscription_params($invoice, $card_info = array(), $response = array()) {
+    if (empty($invoice)) {
         return false;
     }
     
-    $subscription_item = $invoice->get_recurring( true );
-    if ( empty( $subscription_item ) ) {
+    $subscription_item = $invoice->get_recurring(true);
+    if (empty($subscription_item)) {
         return false;
     }
     
-    $card_details       = wpinv_authorizenet_generate_card_info( $card_info );
+    $card_details       = wpinv_authorizenet_generate_card_info($card_info);
     $subscription_name  = $invoice->get_subscription_name();
-    $initial_amount     = wpinv_round_amount( $invoice->get_total() );
-    $recurring_amount   = wpinv_round_amount( $invoice->get_recurring_details( 'total' ) );
+    $initial_amount     = wpinv_round_amount($invoice->get_total());
+    $recurring_amount   = wpinv_round_amount($invoice->get_recurring_details('total'));
     $interval           = $subscription_item->get_recurring_interval();
     $period             = $subscription_item->get_recurring_period();
     $bill_times         = (int)$subscription_item->get_recurring_limit();
     $bill_times         = $bill_times > 0 ? $bill_times : 9999;
     
-    $time_period        = wpinv_authorizenet_get_time_period( $interval, $period );
+    $time_period        = wpinv_authorizenet_get_time_period($interval, $period);
     $interval           = $time_period['interval'];
     $period             = $time_period['period'];
     
     $current_tz = date_default_timezone_get();
-    date_default_timezone_set( 'America/Denver' ); // Set same timezone as Authorize's server (Mountain Time) to prevent conflicts.
-    $today = date( 'Y-m-d' );
-    date_default_timezone_set( $current_tz );
+    date_default_timezone_set('America/Denver'); // Set same timezone as Authorize's server (Mountain Time) to prevent conflicts.
+    $today = date('Y-m-d');
+    date_default_timezone_set($current_tz);
     
     $free_trial = $invoice->is_free_trial();
-    if ( $free_trial && $subscription_item->has_free_trial() ) {
+    if ($free_trial && $subscription_item->has_free_trial()) {
         $trial_interval    = $subscription_item->get_trial_interval();
-        $trial_period      = $subscription_item->get_trial_period( true );
+        $trial_period      = $subscription_item->get_trial_period(true);
     }
     
     $subscription = array();
     $subscription['name'] = $subscription_name;
     
     $subscription['paymentSchedule'] = array(
-        'interval'         => array( 'length' => $interval, 'unit' => $period ),
+        'interval'         => array('length' => $interval, 'unit' => $period),
         'startDate'        => $today,
         'totalOccurrences' => $bill_times,
-        'trialOccurrences' => $free_trial || ( $initial_amount != $recurring_amount ) ? 1 : 0,
+        'trialOccurrences' => $free_trial || ($initial_amount != $recurring_amount) ? 1 : 0,
     );
     
     $subscription['amount'] = $recurring_amount;
     $subscription['trialAmount'] = $initial_amount;
-    $subscription['payment'] = array( 'creditCard' => $card_details );
-    $subscription['order'] = array( 'invoiceNumber' => $invoice->ID, 'description' => '#' . $invoice->get_number() );
-    $subscription['customer'] = array( 'id' => $invoice->get_user_id(), 'email' => $invoice->get_email(), 'phoneNumber' => $invoice->phone );
+    $subscription['payment'] = array('creditCard' => $card_details);
+    $subscription['order'] = array('invoiceNumber' => $invoice->ID, 'description' => '#' . $invoice->get_number());
+    $subscription['customer'] = array('id' => $invoice->get_user_id(), 'email' => $invoice->get_email(), 'phoneNumber' => $invoice->phone);
     
     $subscription['billTo'] = array(
         'firstName' => $invoice->get_first_name(),
         'lastName'  => $invoice->get_last_name(),
         'company'   => $invoice->company,
-        'address'   => wp_strip_all_tags( $invoice->get_address(), true ),
+        'address'   => wp_strip_all_tags($invoice->get_address(), true),
         'city'      => $invoice->city,
         'state'     => $invoice->state,
         'zip'       => $invoice->zip,
         'country'   => $invoice->country,
     );
     
-    $params = array( 'subscription' => $subscription );
+    $params = array('subscription' => $subscription);
     
-    return apply_filters( 'wpinv_authorizenet_generate_subscription_params', $params, $invoice, $card_info, $response );
+    return apply_filters('wpinv_authorizenet_generate_subscription_params', $params, $invoice, $card_info, $response);
 }
 
-function wpinv_authorizenet_generate_card_info( $card_info = array() ) {
-    $card_defaults      = array(
+function wpinv_authorizenet_generate_card_info($card_info = array()) {
+    $card_defaults = array(
         'cc_owner'          => null,
         'cc_number'         => null,
         'cc_expire_month'   => null,
         'cc_expire_year'    => null,
         'cc_cvv2'           => null,
     );
-    $card_info = wp_parse_args( $card_info, $card_defaults );
+    $card_info = wp_parse_args($card_info, $card_defaults);
         
     $card_details = array(
-        'cardNumber'     => str_replace( ' ', '', sanitize_text_field( $card_info['cc_number'] ) ),
-        'expirationDate' => sanitize_text_field( $card_info['cc_expire_month'] ) . sanitize_text_field( $card_info['cc_expire_year'] ),
-        'cardCode'       => sanitize_text_field( $card_info['cc_cvv2'] ),
+        'cardNumber'     => str_replace(' ', '', sanitize_text_field($card_info['cc_number'])),
+        'expirationDate' => sanitize_text_field($card_info['cc_expire_month']) . sanitize_text_field($card_info['cc_expire_year']),
+        'cardCode'       => sanitize_text_field($card_info['cc_cvv2']),
     );
 
     return $card_details;
 }
 
-function wpinv_authorizenet_subscription_record_signup( $subscription, $invoice ) {
-    if ( empty( $invoice ) || empty( $subscription ) ) {
+function wpinv_authorizenet_subscription_record_signup($subscription, $invoice) {
+    if (empty($invoice) || empty($subscription)) {
         return false;
     }
     
-    $subscription_item = $invoice->get_recurring( true );
-    if ( empty( $subscription_item ) ) {
+    $subscription_item = $invoice->get_recurring(true);
+    if (empty($subscription_item)) {
         return false;
     }
     
     $invoice_id         = $invoice->ID;
     $subscriptionId     = (array)$subscription->subscriptionId;
-    $subscription_id    = !empty( $subscriptionId[0] ) ? $subscriptionId[0] : $invoice_id;
+    $subscription_id    = !empty($subscriptionId[0]) ? $subscriptionId[0] : $invoice_id;
 
-    wpinv_insert_payment_note( $invoice_id, sprintf( __( 'Authorize.Net Subscription ID: %s', 'invoicing' ) , $subscription_id ) );
+    wpinv_insert_payment_note($invoice_id, sprintf(__('Authorize.Net Subscription ID: %s', 'invoicing'), $subscription_id));
     
     $status = $invoice->is_free_trial() && $subscription_item->has_free_trial() ? 'trialing' : 'active';
     
@@ -424,16 +424,16 @@ 
         'profile_id'        => $subscription_id,
         'item_id'           => $subscription_item->ID,
         'initial_amount'    => $invoice->get_total(),
-        'recurring_amount'  => $invoice->get_recurring_details( 'total' ),
+        'recurring_amount'  => $invoice->get_recurring_details('total'),
         'period'            => $subscription_item->get_recurring_period(),
         'interval'          => $subscription_item->get_recurring_interval(),
         'bill_times'        => $subscription_item->get_recurring_limit(),
-        'expiration'        => $invoice->get_new_expiration( $subscription_item->ID ),
+        'expiration'        => $invoice->get_new_expiration($subscription_item->ID),
         'status'            => $status,
-        'created'           => current_time( 'mysql', 0 )
+        'created'           => current_time('mysql', 0)
     );
     
-    if ( $invoice->is_free_trial() && $subscription_item->has_free_trial() ) {
+    if ($invoice->is_free_trial() && $subscription_item->has_free_trial()) {
         $args['trial_period']      = $subscription_item->get_trial_period();
         $args['trial_interval']    = $subscription_item->get_trial_interval();
     } else {
@@ -441,58 +441,58 @@ 
         $args['trial_interval']    = 0;
     }
     
-    return $invoice->update_subscription( $args );
+    return $invoice->update_subscription($args);
 }
 
-function wpinv_authorizenet_validate_checkout( $valid_data, $post ) {
-    if ( !empty( $post['wpi-gateway'] ) && $post['wpi-gateway'] == 'authorizenet' ) {
+function wpinv_authorizenet_validate_checkout($valid_data, $post) {
+    if (!empty($post['wpi-gateway']) && $post['wpi-gateway'] == 'authorizenet') {
         $error = false;
         
-        if ( empty( $post['authorizenet']['cc_owner'] ) ) {
+        if (empty($post['authorizenet']['cc_owner'])) {
             $error = true;
-            wpinv_set_error( 'empty_card_name', __( 'You must enter the name on your card!', 'invoicing'));
+            wpinv_set_error('empty_card_name', __('You must enter the name on your card!', 'invoicing'));
         }
-        if ( empty( $post['authorizenet']['cc_number'] ) ) {
+        if (empty($post['authorizenet']['cc_number'])) {
             $error = true;
-            wpinv_set_error( 'empty_card', __( 'You must enter a card number!', 'invoicing'));
+            wpinv_set_error('empty_card', __('You must enter a card number!', 'invoicing'));
         }
-        if ( empty( $post['authorizenet']['cc_expire_month'] ) ) {
+        if (empty($post['authorizenet']['cc_expire_month'])) {
             $error = true;
-            wpinv_set_error( 'empty_month', __( 'You must enter an card expiration month!', 'invoicing'));
+            wpinv_set_error('empty_month', __('You must enter an card expiration month!', 'invoicing'));
         }
-        if ( empty( $post['authorizenet']['cc_expire_year'] ) ) {
+        if (empty($post['authorizenet']['cc_expire_year'])) {
             $error = true;
-            wpinv_set_error( 'empty_year', __( 'You must enter an card expiration year!', 'invoicing'));
+            wpinv_set_error('empty_year', __('You must enter an card expiration year!', 'invoicing'));
         }
-        if ( empty( $post['authorizenet']['cc_cvv2'] ) ) {
+        if (empty($post['authorizenet']['cc_cvv2'])) {
             $error = true;
-            wpinv_set_error( 'empty_cvv2', __( 'You must enter a valid CVV2!', 'invoicing' ) );
+            wpinv_set_error('empty_cvv2', __('You must enter a valid CVV2!', 'invoicing'));
         }
         
-        if ( $error ) {
+        if ($error) {
             return;
         }
         
         $invoice = wpinv_get_invoice_cart();
         
-        if ( !empty( $invoice ) && $subscription_item = $invoice->get_recurring( true ) ) {
-            $subscription_item = $invoice->get_recurring( true );
+        if (!empty($invoice) && $subscription_item = $invoice->get_recurring(true)) {
+            $subscription_item = $invoice->get_recurring(true);
             
             $interval   = $subscription_item->get_recurring_interval();
             $period     = $subscription_item->get_recurring_period();
             
-            if ( $period == 'D' && ( $interval < 7 || $interval > 365 ) ) {
-                wpinv_set_error( 'authorizenet_subscription_error', __( 'Interval Length must be a value from 7 through 365 for day based subscriptions.', 'invoicing' ) );
+            if ($period == 'D' && ($interval < 7 || $interval > 365)) {
+                wpinv_set_error('authorizenet_subscription_error', __('Interval Length must be a value from 7 through 365 for day based subscriptions.', 'invoicing'));
             }
         }
     }
 }
-add_action( 'wpinv_checkout_error_checks', 'wpinv_authorizenet_validate_checkout', 11, 2 );
+add_action('wpinv_checkout_error_checks', 'wpinv_authorizenet_validate_checkout', 11, 2);
 
-function wpinv_authorizenet_get_time_period( $subscription_interval, $subscription_period ) {
-    $subscription_interval = absint( $subscription_interval );
+function wpinv_authorizenet_get_time_period($subscription_interval, $subscription_period) {
+    $subscription_interval = absint($subscription_interval);
 
-    switch( $subscription_period ) {
+    switch ($subscription_period) {
         case 'W':
         case 'week':
         case 'weeks':
@@ -502,14 +502,14 @@ 
         case 'M':
         case 'month':
         case 'months':
-            if ( $subscription_interval > 12 ) {
+            if ($subscription_interval > 12) {
                 $subscription_interval = 12;
             }
             
             $interval = $subscription_interval;
             $period   = 'months';
             
-            if ( !( $subscription_interval === 1 || $subscription_interval === 2 || $subscription_interval === 3 || $subscription_interval === 6 || $subscription_interval === 12 ) ) {
+            if (!($subscription_interval === 1 || $subscription_interval === 2 || $subscription_interval === 3 || $subscription_interval === 6 || $subscription_interval === 12)) {
                 $interval = $subscription_interval * 30;
                 $period   = 'days';
             }
@@ -526,40 +526,40 @@ 
             break;
     }
 
-    return compact( 'interval', 'period' );
+    return compact('interval', 'period');
 }
 
 function wpinv_authorizenet_process_ipn() {
-    if ( !( !empty( $_REQUEST['wpi-gateway'] ) && $_REQUEST['wpi-gateway'] == 'authorizenet' ) ) {
+    if (!(!empty($_REQUEST['wpi-gateway']) && $_REQUEST['wpi-gateway'] == 'authorizenet')) {
         return;
     }
     
-    $subscription_id = intval( $_POST['x_subscription_id'] );
+    $subscription_id = intval($_POST['x_subscription_id']);
     
-    if ( $subscription_id ) {
-        $transaction_id = sanitize_text_field( $_POST['x_trans_id'] );
-        $renewal_amount = sanitize_text_field( $_POST['x_amount'] );
-        $response_code  = intval( $_POST['x_response_code'] );
-        $reason_code    = intval( $_POST['x_response_reason_code'] );
+    if ($subscription_id) {
+        $transaction_id = sanitize_text_field($_POST['x_trans_id']);
+        $renewal_amount = sanitize_text_field($_POST['x_amount']);
+        $response_code  = intval($_POST['x_response_code']);
+        $reason_code    = intval($_POST['x_response_reason_code']);
 
-        if ( 1 == $response_code ) {
+        if (1 == $response_code) {
             // Approved
-            do_action( 'wpinv_authorizenet_renewal_payment', $transaction_id );
-        } else if ( 2 == $response_code ) {
+            do_action('wpinv_authorizenet_renewal_payment', $transaction_id);
+        } else if (2 == $response_code) {
             // Declined
-            do_action( 'wpinv_authorizenet_renewal_payment_failed', $transaction_id );
-            do_action( 'wpinv_authorizenet_renewal_error', $transaction_id );
-        } else if ( 3 == $response_code || 8 == $reason_code ) {
+            do_action('wpinv_authorizenet_renewal_payment_failed', $transaction_id);
+            do_action('wpinv_authorizenet_renewal_error', $transaction_id);
+        } else if (3 == $response_code || 8 == $reason_code) {
             // An expired card
-            do_action( 'wpinv_authorizenet_renewal_payment_failed', $transaction_id );
-            do_action( 'wpinv_authorizenet_renewal_payment_error', $transaction_id );
+            do_action('wpinv_authorizenet_renewal_payment_failed', $transaction_id);
+            do_action('wpinv_authorizenet_renewal_payment_error', $transaction_id);
 
         } else {
             // Other Error
-            do_action( 'wpinv_authorizenet_renewal_payment_error', $subscription );
+            do_action('wpinv_authorizenet_renewal_payment_error', $subscription);
         }
         
         exit;
     }
 }
-add_action( 'wpinv_verify_authorizenet_ipn', 'wpinv_authorizenet_process_ipn' );
\ No newline at end of file
+add_action('wpinv_verify_authorizenet_ipn', 'wpinv_authorizenet_process_ipn');
\ No newline at end of file

includes/class-wpinv-api.php

Spacing +67 added lines, -67 removed lines

@@ -7,51 +7,51 @@ 
  */
  
 // MUST have WordPress.
-if ( !defined( 'WPINC' ) ) {
-    exit( 'Do NOT access this file directly: ' . basename( __FILE__ ) );
+if (!defined('WPINC')) {
+    exit('Do NOT access this file directly: ' . basename(__FILE__));
 }
 
 class WPInv_API {
     protected $post_type = 'wpi_invoice';
     
-    public function __construct( $params = array() ) {
+    public function __construct($params = array()) {
     }
-    public function insert_invoice( $data ) {
+    public function insert_invoice($data) {
         global $wpdb;
         //wpinv_transaction_query( 'start' );
 
         try {
-            if ( empty( $data['invoice'] ) ) {
-                throw new WPInv_API_Exception( 'wpinv_api_missing_invoice_data', sprintf( __( 'No %1$s data specified to create %1$s', 'invoicing' ), 'invoice' ), 400 );
+            if (empty($data['invoice'])) {
+                throw new WPInv_API_Exception('wpinv_api_missing_invoice_data', sprintf(__('No %1$s data specified to create %1$s', 'invoicing'), 'invoice'), 400);
             }
 
-            $data = apply_filters( 'wpinv_api_create_invoice_data', $data['invoice'], $this );
+            $data = apply_filters('wpinv_api_create_invoice_data', $data['invoice'], $this);
 
-            $invoice = wpinv_insert_invoice( $data, true );
-            if ( empty( $invoice->ID ) || is_wp_error( $invoice ) ) {
-                throw new WPInv_API_Exception( 'wpinv_api_cannot_create_invoice', sprintf( __( 'Cannot create invoice: %s', 'invoicing' ), implode( ', ', $invoice->get_error_messages() ) ), 400 );
+            $invoice = wpinv_insert_invoice($data, true);
+            if (empty($invoice->ID) || is_wp_error($invoice)) {
+                throw new WPInv_API_Exception('wpinv_api_cannot_create_invoice', sprintf(__('Cannot create invoice: %s', 'invoicing'), implode(', ', $invoice->get_error_messages())), 400);
             }
 
             // HTTP 201 Created
-            $this->send_status( 201 );
+            $this->send_status(201);
 
-            do_action( 'wpinv_api_create_invoice', $invoice->ID, $data, $this );
+            do_action('wpinv_api_create_invoice', $invoice->ID, $data, $this);
 
             //wpinv_transaction_query( 'commit' );
 
-            return wpinv_get_invoice( $invoice->ID );
-        } catch ( WPInv_API_Exception $e ) {
+            return wpinv_get_invoice($invoice->ID);
+        } catch (WPInv_API_Exception $e) {
             //wpinv_transaction_query( 'rollback' );
 
-            return new WP_Error( $e->getErrorCode(), $e->getMessage(), array( 'status' => $e->getCode() ) );
+            return new WP_Error($e->getErrorCode(), $e->getMessage(), array('status' => $e->getCode()));
         }
     }
     
-    public function send_status( $code ) {
-        status_header( $code );
+    public function send_status($code) {
+        status_header($code);
     }
     
-    protected function set_billing_details( $invoice, $data ) {
+    protected function set_billing_details($invoice, $data) {
         $address_fields = array(
             'user_id',
             'first_name',
@@ -70,66 +70,66 @@ 
         $billing_details = array();
         $user_id = $invoice->get_user_id();
         
-        foreach ( $address_fields as $field ) {
-            if ( isset( $data['billing_details'][ $field ] ) ) {
-                $value = sanitize_text_field( $data['billing_details'][ $field ] );
+        foreach ($address_fields as $field) {
+            if (isset($data['billing_details'][$field])) {
+                $value = sanitize_text_field($data['billing_details'][$field]);
                 
-                if ( $field == 'country' && empty( $value ) ) {
-                    if ( !empty( $invoice->country ) ) {
+                if ($field == 'country' && empty($value)) {
+                    if (!empty($invoice->country)) {
                         $value = $invoice->country;
                     } else {
-                        $value = wpinv_default_billing_country( '', $user_id );
+                        $value = wpinv_default_billing_country('', $user_id);
                     }
                 }
                 
-                if ( $field == 'state' && empty( $value ) ) {
-                    if ( !empty( $invoice->state ) ) {
+                if ($field == 'state' && empty($value)) {
+                    if (!empty($invoice->state)) {
                         $value = $invoice->state;
                     } else {
                         $value = wpinv_get_default_state();
                     }
                 }
                 
-                $invoice->set( $field, $value );
+                $invoice->set($field, $value);
                 
-                update_post_meta( $invoice->ID, '_wpinv_' . $field, $value );
+                update_post_meta($invoice->ID, '_wpinv_' . $field, $value);
             }
         }
         
         return $invoice;
     }
     
-    protected function set_discount( $invoice, $data ) {
-        if ( isset( $data['discount'] ) ) {
-            $invoice->set( 'discount', wpinv_round_amount( $data['discount'] ) );
+    protected function set_discount($invoice, $data) {
+        if (isset($data['discount'])) {
+            $invoice->set('discount', wpinv_round_amount($data['discount']));
             
-            update_post_meta( $invoice->ID, '_wpinv_discount', wpinv_round_amount( $data['discount'] ) );
+            update_post_meta($invoice->ID, '_wpinv_discount', wpinv_round_amount($data['discount']));
             
-            if ( isset( $data['discount_code'] ) ) {
-                $invoice->set( 'discount_code', $data['discount_code'] );
+            if (isset($data['discount_code'])) {
+                $invoice->set('discount_code', $data['discount_code']);
                 
-                update_post_meta( $invoice->ID, '_wpinv_discount_code', $data['discount_code'] );
+                update_post_meta($invoice->ID, '_wpinv_discount_code', $data['discount_code']);
             }
         }
         
         return $invoice;
     }
     
-    protected function set_items( $invoice, $data ) {
-        if ( !empty( $data['items'] ) && is_array( $data['items'] ) ) {
+    protected function set_items($invoice, $data) {
+        if (!empty($data['items']) && is_array($data['items'])) {
             $items_array = array();
            
-            if ( !empty( $invoice->country ) ) {
+            if (!empty($invoice->country)) {
                 $country = $invoice->country;
-            } else if ( !empty( $data['billing_details']['country'] ) ) {
+            } else if (!empty($data['billing_details']['country'])) {
                 $country = $data['billing_details']['country'];
             } else {
-                $country = wpinv_default_billing_country( '', $invoice->get_user_id() );
+                $country = wpinv_default_billing_country('', $invoice->get_user_id());
             }
             
-            if ( !empty( $invoice->state ) ) {
+            if (!empty($invoice->state)) {
                 $state = $invoice->state;
-            } else if ( !empty( $data['billing_details']['state'] ) ) {
+            } else if (!empty($data['billing_details']['state'])) {
                 $state = $data['billing_details']['state'];
             } else {
                 $state = wpinv_get_default_state();
@@ -138,54 +138,54 @@ 
             $_POST['country']   = $country;
             $_POST['state']     = $state;
             
-            $rate = wpinv_get_tax_rate( $country, $state, 'global' );
+            $rate = wpinv_get_tax_rate($country, $state, 'global');
             
             $total_tax = 0;
-            foreach ( $data['items'] as $item ) {
-                $id                 = isset( $item['id'] ) ? sanitize_text_field( $item['id'] ) : '';
-                $title              = isset( $item['title'] ) ? sanitize_text_field( $item['title'] ) : '';
-                $desc               = isset( $item['description'] ) ? sanitize_text_field( $item['description'] ) : '';
-                $amount             = isset( $item['amount'] ) ? wpinv_round_amount( $item['amount'] ) : 0;
+            foreach ($data['items'] as $item) {
+                $id                 = isset($item['id']) ? sanitize_text_field($item['id']) : '';
+                $title              = isset($item['title']) ? sanitize_text_field($item['title']) : '';
+                $desc               = isset($item['description']) ? sanitize_text_field($item['description']) : '';
+                $amount             = isset($item['amount']) ? wpinv_round_amount($item['amount']) : 0;
                 
-                if ( !empty( $item['vat_rates_class'] ) ) {
+                if (!empty($item['vat_rates_class'])) {
                     $vat_rates_class = $item['vat_rates_class'];
                 } else {
                     $vat_rates_class = '_standard';
                 }
-                $vat_rate = wpinv_get_tax_rate( $country, $state, $id );
+                $vat_rate = wpinv_get_tax_rate($country, $state, $id);
                 
-                $tax = $amount > 0 ? ( $amount * 0.01 * (float)$vat_rate ) : 0;
+                $tax = $amount > 0 ? ($amount * 0.01 * (float)$vat_rate) : 0;
                 $total_tax += $tax;
                 
                 $items_array[] = array(
                     'id'                => $id,
-                    'title'             => esc_html( $title ),
-                    'description'       => esc_html( $desc ),
-                    'amount'            => $amount > 0 ? wpinv_round_amount( $amount ) : 0,
-                    'subtotal'          => $amount > 0 ? wpinv_round_amount( $amount ) : 0,
+                    'title'             => esc_html($title),
+                    'description'       => esc_html($desc),
+                    'amount'            => $amount > 0 ? wpinv_round_amount($amount) : 0,
+                    'subtotal'          => $amount > 0 ? wpinv_round_amount($amount) : 0,
                     'vat_rates_class'   => $vat_rates_class,
                     'vat_rate'          => $vat_rate,
-                    'tax'               => $tax > 0 ? wpinv_round_amount( $tax ) : 0,
+                    'tax'               => $tax > 0 ? wpinv_round_amount($tax) : 0,
                 );
             }
 
-            update_post_meta( $invoice->ID, '_wpinv_tax', wpinv_round_amount( $total_tax ) );
-            $invoice->set( 'tax', wpinv_round_amount( $total_tax ) );
+            update_post_meta($invoice->ID, '_wpinv_tax', wpinv_round_amount($total_tax));
+            $invoice->set('tax', wpinv_round_amount($total_tax));
             
-            $items_array = apply_filters( 'wpinv_save_invoice_items', $items_array, $data['items'], $invoice );
+            $items_array = apply_filters('wpinv_save_invoice_items', $items_array, $data['items'], $invoice);
             
-            $invoice->set( 'items', $items_array );
-            update_post_meta( $invoice->ID, '_wpinv_items', $items_array );
+            $invoice->set('items', $items_array);
+            update_post_meta($invoice->ID, '_wpinv_items', $items_array);
         }
         
         return $invoice;
     }
     
-    protected function set_invoice_meta( $invoice_id, $invoice_meta ) {
-        foreach ( $invoice_meta as $meta_key => $meta_value ) {
+    protected function set_invoice_meta($invoice_id, $invoice_meta) {
+        foreach ($invoice_meta as $meta_key => $meta_value) {
 
-            if ( is_string( $meta_key) && ! is_protected_meta( $meta_key ) && is_scalar( $meta_value ) ) {
-                update_post_meta( $invoice_id, $meta_key, $meta_value );
+            if (is_string($meta_key) && !is_protected_meta($meta_key) && is_scalar($meta_value)) {
+                update_post_meta($invoice_id, $meta_key, $meta_value);
             }
         }
     }
@@ -195,9 +195,9 @@ 
 class WPInv_API_Exception extends Exception {
     protected $error_code;
 
-    public function __construct( $error_code, $error_message, $http_status_code ) {
+    public function __construct($error_code, $error_message, $http_status_code) {
         $this->error_code = $error_code;
-        parent::__construct( $error_message, $http_status_code );
+        parent::__construct($error_message, $http_status_code);
     }
 
     public function getErrorCode() {

templates/emails/wpinv-email-invoice-items.php

Spacing +70 added lines, -70 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 // don't load directly
-if ( !defined('ABSPATH') )
+if (!defined('ABSPATH'))
     die('-1');
 
 global $wpinv_euvat, $ajax_cart_details;
@@ -11,158 +11,158 @@ 
 $quantities_enabled = wpinv_item_quantities_enabled();
 $use_taxes          = wpinv_use_taxes();
 $zero_tax           = !(float)$invoice->get_tax() > 0 ? true : false;
-$tax_label          = $use_taxes && $invoice->has_vat() ? $wpinv_euvat->get_vat_name() : __( 'Tax', 'invoicing' );
-$tax_title          = !$zero_tax && $use_taxes ? ( wpinv_prices_include_tax() ? wp_sprintf( __( '(%s Incl.)', 'invoicing' ), $tax_label ) : wp_sprintf( __( '(%s Excl.)', 'invoicing' ), $tax_label ) ) : '';
+$tax_label          = $use_taxes && $invoice->has_vat() ? $wpinv_euvat->get_vat_name() : __('Tax', 'invoicing');
+$tax_title          = !$zero_tax && $use_taxes ? (wpinv_prices_include_tax() ? wp_sprintf(__('(%s Incl.)', 'invoicing'), $tax_label) : wp_sprintf(__('(%s Excl.)', 'invoicing'), $tax_label)) : '';
 
-do_action( 'wpinv_before_email_items', $invoice ); ?>
+do_action('wpinv_before_email_items', $invoice); ?>
 <div id="wpinv-email-items">
-    <h3 class="wpinv-items-t"><?php echo apply_filters( 'wpinv_email_items_title', __( 'Items', 'invoicing' ) ); ?></h3>
+    <h3 class="wpinv-items-t"><?php echo apply_filters('wpinv_email_items_title', __('Items', 'invoicing')); ?></h3>
     <table id="wpinv_checkout_cart" class="table table-bordered table-hover">
         <thead>
             <tr class="wpinv_cart_header_row">
-                <?php do_action( 'wpinv_email_items_table_header_first' ); ?>
-                <th class="wpinv_cart_item_name text-left"><?php _e( 'Item Name', 'invoicing' ); ?></th>
-                <th class="wpinv_cart_item_price text-right"><?php _e( 'Item Price', 'invoicing' ); ?></th>
-                <?php if ( $quantities_enabled ) { ?>
-                <th class="wpinv_cart_item_qty text-right"><?php _e( 'Qty', 'invoicing' ); ?></th>
+                <?php do_action('wpinv_email_items_table_header_first'); ?>
+                <th class="wpinv_cart_item_name text-left"><?php _e('Item Name', 'invoicing'); ?></th>
+                <th class="wpinv_cart_item_price text-right"><?php _e('Item Price', 'invoicing'); ?></th>
+                <?php if ($quantities_enabled) { ?>
+                <th class="wpinv_cart_item_qty text-right"><?php _e('Qty', 'invoicing'); ?></th>
                 <?php } ?>
-                <?php if ( !$zero_tax && $use_taxes ) { ?>
+                <?php if (!$zero_tax && $use_taxes) { ?>
                 <th class="wpinv_cart_item_tax text-right"><?php echo $tax_label . ' <span class="normal small">(%)</span>'; ?></th>
                 <?php } ?>
-                <th class="wpinv_cart_item_subtotal text-right"><?php echo __( 'Item Total', 'invoicing' ) . ' <span class="normal small">' . $tax_title . '<span>'; ?></th>
-                <?php do_action( 'wpinv_email_items_table_header_last' ); ?>
+                <th class="wpinv_cart_item_subtotal text-right"><?php echo __('Item Total', 'invoicing') . ' <span class="normal small">' . $tax_title . '<span>'; ?></th>
+                <?php do_action('wpinv_email_items_table_header_last'); ?>
             </tr>
         </thead>
         <tbody>
             <?php
-                do_action( 'wpinv_email_items_before' );
-                if ( $cart_items ) {
-                    foreach ( $cart_items as $key => $item ) {
-                        $wpi_item = $item['id'] ? new WPInv_Item( $item['id'] ) : NULL;
+                do_action('wpinv_email_items_before');
+                if ($cart_items) {
+                    foreach ($cart_items as $key => $item) {
+                        $wpi_item = $item['id'] ? new WPInv_Item($item['id']) : NULL;
                     ?>
-                    <tr class="wpinv_cart_item" id="wpinv_cart_item_<?php echo esc_attr( $key ) . '_' . esc_attr( $item['id'] ); ?>" data-item-id="<?php echo esc_attr( $item['id'] ); ?>">
-                        <?php do_action( 'wpinv_email_items_table_body_first', $item ); ?>
+                    <tr class="wpinv_cart_item" id="wpinv_cart_item_<?php echo esc_attr($key) . '_' . esc_attr($item['id']); ?>" data-item-id="<?php echo esc_attr($item['id']); ?>">
+                        <?php do_action('wpinv_email_items_table_body_first', $item); ?>
                         <td class="wpinv_cart_item_name text-left">
                             <?php
-                                if ( current_theme_supports( 'post-thumbnails' ) && has_post_thumbnail( $item['id'] ) ) {
+                                if (current_theme_supports('post-thumbnails') && has_post_thumbnail($item['id'])) {
                                     echo '<div class="wpinv_cart_item_image">';
-                                        echo get_the_post_thumbnail( $item['id'], apply_filters( 'wpinv_checkout_image_size', array( 25,25 ) ) );
+                                        echo get_the_post_thumbnail($item['id'], apply_filters('wpinv_checkout_image_size', array(25, 25)));
                                     echo '</div>';
                                 }
-                                $item_title = esc_html( wpinv_get_cart_item_name( $item ) ) . wpinv_get_item_suffix( $wpi_item );
+                                $item_title = esc_html(wpinv_get_cart_item_name($item)) . wpinv_get_item_suffix($wpi_item);
                                 echo '<span class="wpinv_email_cart_item_title">' . $item_title . '</span>';
                                 
                                 $summary = '';
-                                if ( !empty( $wpi_item ) && $wpi_item->is_package() && !empty( $item['meta']['post_id'] ) ) {
-                                    $post_link = '<a href="' . get_permalink( $item['meta']['post_id'] ) .'" target="_blank">' . (!empty($item['meta']['invoice_title']) ? $item['meta']['invoice_title'] : get_the_title( $item['meta']['post_id']) ) . '</a>';
-                                    $summary = wp_sprintf( __( '%s: %s', 'invoicing' ), $wpi_item->get_custom_singular_name(), $post_link );
+                                if (!empty($wpi_item) && $wpi_item->is_package() && !empty($item['meta']['post_id'])) {
+                                    $post_link = '<a href="' . get_permalink($item['meta']['post_id']) . '" target="_blank">' . (!empty($item['meta']['invoice_title']) ? $item['meta']['invoice_title'] : get_the_title($item['meta']['post_id'])) . '</a>';
+                                    $summary = wp_sprintf(__('%s: %s', 'invoicing'), $wpi_item->get_custom_singular_name(), $post_link);
                                 }
                                 
-                                $summary = apply_filters( 'wpinv_email_invoice_line_item_summary', $summary, $item, $wpi_item, $invoice );
-                                if ( !empty( $summary ) ) {
+                                $summary = apply_filters('wpinv_email_invoice_line_item_summary', $summary, $item, $wpi_item, $invoice);
+                                if (!empty($summary)) {
                                     echo '<p class="small">' . $summary . '</p>';
                                 }
     
-                                do_action( 'wpinv_email_cart_item_title_after', $item, $key );
+                                do_action('wpinv_email_cart_item_title_after', $item, $key);
                             ?>
                         </td>
                         <td class="wpinv_cart_item_price text-right">
                             <?php 
-                            echo wpinv_cart_item_price( $item );
-                            do_action( 'wpinv_email_cart_item_price_after', $item, $key );
+                            echo wpinv_cart_item_price($item);
+                            do_action('wpinv_email_cart_item_price_after', $item, $key);
                             ?>
                         </td>
-                        <?php if ( $quantities_enabled ) { ?>
+                        <?php if ($quantities_enabled) { ?>
                         <td class="wpinv_cart_item_qty text-right">
                             <?php
-                            echo wpinv_get_cart_item_quantity( $item );
-                            do_action( 'wpinv_email_item_quantitiy', $item, $key );
+                            echo wpinv_get_cart_item_quantity($item);
+                            do_action('wpinv_email_item_quantitiy', $item, $key);
                             ?>
                         </td>
                         <?php } ?>
-                        <?php if ( !$zero_tax && $use_taxes ) { ?>
+                        <?php if (!$zero_tax && $use_taxes) { ?>
                         <td class="wpinv_cart_item_tax text-right">
                             <?php
-                            echo wpinv_cart_item_tax( $item );
-                            do_action( 'wpinv_email_item_tax', $item, $key );
+                            echo wpinv_cart_item_tax($item);
+                            do_action('wpinv_email_item_tax', $item, $key);
                             ?>
                         </td>
                         <?php } ?>
                         <td class="wpinv_cart_item_subtotal text-right">
                             <?php
-                            echo wpinv_cart_item_subtotal( $item );
-                            do_action( 'wpinv_email_item_subtotal', $item, $key );
+                            echo wpinv_cart_item_subtotal($item);
+                            do_action('wpinv_email_item_subtotal', $item, $key);
                             ?>
                         </td>
-                        <?php do_action( 'wpinv_email_items_table_body_last', $item, $key ); ?>
+                        <?php do_action('wpinv_email_items_table_body_last', $item, $key); ?>
                     </tr>
                 <?php } ?>
             <?php } ?>
-            <?php do_action( 'wpinv_email_items_middle' ); ?>
-            <?php do_action( 'wpinv_email_items_after' ); ?>
+            <?php do_action('wpinv_email_items_middle'); ?>
+            <?php do_action('wpinv_email_items_after'); ?>
         </tbody>
         <tfoot>
-            <?php $cart_columns = wpinv_checkout_cart_columns(); if ( $zero_tax && $use_taxes ) { $cart_columns--; } ?>
-            <?php if ( has_action( 'wpinv_email_footer_buttons' ) ) { ?>
+            <?php $cart_columns = wpinv_checkout_cart_columns(); if ($zero_tax && $use_taxes) { $cart_columns--; } ?>
+            <?php if (has_action('wpinv_email_footer_buttons')) { ?>
                 <tr class="wpinv_cart_footer_row">
-                    <?php do_action( 'wpinv_email_items_table_buttons_first', $cart_items ); ?>
-                    <td colspan="<?php echo ( $cart_columns ); ?>">
-                        <?php do_action( 'wpinv_email_footer_buttons' ); ?>
+                    <?php do_action('wpinv_email_items_table_buttons_first', $cart_items); ?>
+                    <td colspan="<?php echo ($cart_columns); ?>">
+                        <?php do_action('wpinv_email_footer_buttons'); ?>
                     </td>
-                    <?php do_action( 'wpinv_email_items_table_buttons_first', $cart_items ); ?>
+                    <?php do_action('wpinv_email_items_table_buttons_first', $cart_items); ?>
                 </tr>
             <?php } ?>
 
-            <?php if ( !$zero_tax && $use_taxes && !wpinv_prices_include_tax() && wpinv_is_cart_taxed() ) { ?>
+            <?php if (!$zero_tax && $use_taxes && !wpinv_prices_include_tax() && wpinv_is_cart_taxed()) { ?>
                 <tr class="wpinv_cart_footer_row wpinv_cart_subtotal_row">
-                    <?php do_action( 'wpinv_email_items_table_subtotal_first', $cart_items ); ?>
-                    <td colspan="<?php echo ( $cart_columns - 1 ); ?>" class="wpinv_cart_subtotal_label text-right">
-                        <strong><?php _e( 'Sub-Total', 'invoicing' ); ?>:</strong>
+                    <?php do_action('wpinv_email_items_table_subtotal_first', $cart_items); ?>
+                    <td colspan="<?php echo ($cart_columns - 1); ?>" class="wpinv_cart_subtotal_label text-right">
+                        <strong><?php _e('Sub-Total', 'invoicing'); ?>:</strong>
                     </td>
                     <td class="wpinv_cart_subtotal text-right">
-                        <span class="wpinv_cart_subtotal_amount bold"><?php echo $invoice->get_subtotal( true ); ?></span>
+                        <span class="wpinv_cart_subtotal_amount bold"><?php echo $invoice->get_subtotal(true); ?></span>
                     </td>
-                    <?php do_action( 'wpinv_email_items_table_subtotal_last', $cart_items, $invoice ); ?>
+                    <?php do_action('wpinv_email_items_table_subtotal_last', $cart_items, $invoice); ?>
                 </tr>
             <?php } ?>
             
-            <?php if ( wpinv_discount( $invoice_id, false ) > 0 ) { ?>
+            <?php if (wpinv_discount($invoice_id, false) > 0) { ?>
                 <tr class="wpinv_cart_footer_row wpinv_cart_discount_row">
-                    <?php do_action( 'wpinv_receipt_items_table_discount_first', $cart_items, $invoice ); ?>
-                    <td colspan="<?php echo ( $cart_columns - 1 ); ?>" class="wpinv_cart_discount_label text-right">
-                        <strong><?php wpinv_get_discount_label( wpinv_discount_code( $invoice_id ) ); ?>:</strong>
+                    <?php do_action('wpinv_receipt_items_table_discount_first', $cart_items, $invoice); ?>
+                    <td colspan="<?php echo ($cart_columns - 1); ?>" class="wpinv_cart_discount_label text-right">
+                        <strong><?php wpinv_get_discount_label(wpinv_discount_code($invoice_id)); ?>:</strong>
                     </td>
                     <td class="wpinv_cart_discount text-right">
-                        <span class="wpinv_cart_discount_amount"><?php echo wpinv_discount( $invoice_id, true, true ); ?></span>
+                        <span class="wpinv_cart_discount_amount"><?php echo wpinv_discount($invoice_id, true, true); ?></span>
                     </td>
-                    <?php do_action( 'wpinv_receipt_items_table_discount_last', $cart_items, $invoice ); ?>
+                    <?php do_action('wpinv_receipt_items_table_discount_last', $cart_items, $invoice); ?>
                 </tr>
             <?php } ?>
 
-            <?php if ( !$zero_tax && $use_taxes && wpinv_is_cart_taxed() ) { ?>
+            <?php if (!$zero_tax && $use_taxes && wpinv_is_cart_taxed()) { ?>
                 <tr class="wpinv_cart_footer_row wpinv_cart_tax_row">
-                    <?php do_action( 'wpinv_email_items_table_tax_first', $cart_items, $invoice ); ?>
-                    <td colspan="<?php echo ( $cart_columns - 1 ); ?>" class="wpinv_cart_tax_label text-right">
+                    <?php do_action('wpinv_email_items_table_tax_first', $cart_items, $invoice); ?>
+                    <td colspan="<?php echo ($cart_columns - 1); ?>" class="wpinv_cart_tax_label text-right">
                         <strong><?php echo $tax_label; ?>:</strong>
                     </td>
                     <td class="wpinv_cart_tax text-right">
-                        <span class="wpinv_cart_tax_amount"><?php echo $invoice->get_tax( true ); ?></span>
+                        <span class="wpinv_cart_tax_amount"><?php echo $invoice->get_tax(true); ?></span>
                     </td>
-                    <?php do_action( 'wpinv_email_items_table_tax_last', $cart_items, $invoice ); ?>
+                    <?php do_action('wpinv_email_items_table_tax_last', $cart_items, $invoice); ?>
                 </tr>
             <?php } ?>
 
             <tr class="wpinv_cart_footer_row">
-                <?php do_action( 'wpinv_email_items_table_footer_first', $cart_items, $invoice ); ?>
-                <td colspan="<?php echo ( $cart_columns - 1 ); ?>" class="wpinv_cart_total_label text-right">
-                    <?php echo apply_filters( 'wpinv_email_cart_total_label', '<strong>' . __( 'Total', 'invoicing' ) . ':</strong>', $invoice ); ?>
+                <?php do_action('wpinv_email_items_table_footer_first', $cart_items, $invoice); ?>
+                <td colspan="<?php echo ($cart_columns - 1); ?>" class="wpinv_cart_total_label text-right">
+                    <?php echo apply_filters('wpinv_email_cart_total_label', '<strong>' . __('Total', 'invoicing') . ':</strong>', $invoice); ?>
                 </td>
                 <td class="wpinv_cart_total text-right">
-                    <span class="wpinv_cart_amount bold"><?php echo $invoice->get_total( true ); ?></span>
+                    <span class="wpinv_cart_amount bold"><?php echo $invoice->get_total(true); ?></span>
                 </td>
-                <?php do_action( 'wpinv_email_items_table_footer_last', $cart_items, $invoice ); ?>
+                <?php do_action('wpinv_email_items_table_footer_last', $cart_items, $invoice); ?>
             </tr>
         </tfoot>
     </table>
 </div>
-<?php do_action( 'wpinv_after_email_items', $invoice ); ?>
\ No newline at end of file
+<?php do_action('wpinv_after_email_items', $invoice); ?>
\ No newline at end of file

templates/wpinv-checkout-cart.php

Spacing +61 added lines, -61 removed lines

@@ -5,140 +5,140 @@
 
 global $wpinv_euvat, $post, $ajax_cart_details, $wpi_cart_columns, $wpi_session;
 $invoice            = wpinv_get_invoice_cart();
-$cart_items         = !empty( $ajax_cart_details ) ? $ajax_cart_details : wpinv_get_cart_content_details();
+$cart_items         = !empty($ajax_cart_details) ? $ajax_cart_details : wpinv_get_cart_content_details();
 $quantities_enabled = wpinv_item_quantities_enabled();
 $use_taxes          = wpinv_use_taxes();
 $tax_label          = $wpinv_euvat->tax_label();
-$tax_title          = $use_taxes ? ( wpinv_prices_include_tax() ? wp_sprintf( __( '(%s Incl.)', 'invoicing' ), $tax_label ) : wp_sprintf( __( '(%s Excl.)', 'invoicing' ), $tax_label ) ) : '';
+$tax_title          = $use_taxes ? (wpinv_prices_include_tax() ? wp_sprintf(__('(%s Incl.)', 'invoicing'), $tax_label) : wp_sprintf(__('(%s Excl.)', 'invoicing'), $tax_label)) : '';
 ?>
 <table id="wpinv_checkout_cart" class="table table-bordered table-hover">
     <thead>
         <tr class="wpinv_cart_header_row">
-            <?php do_action( 'wpinv_checkout_table_header_first' ); ?>
-            <th class="wpinv_cart_item_name text-left"><?php _e( 'Item Name', 'invoicing' ); ?></th>
-            <th class="wpinv_cart_item_price text-right"><?php _e( 'Item Price', 'invoicing' ); ?></th>
-            <?php if ( $quantities_enabled ) { ?>
-            <th class="wpinv_cart_item_qty text-right"><?php _e( 'Qty', 'invoicing' ); ?></th>
+            <?php do_action('wpinv_checkout_table_header_first'); ?>
+            <th class="wpinv_cart_item_name text-left"><?php _e('Item Name', 'invoicing'); ?></th>
+            <th class="wpinv_cart_item_price text-right"><?php _e('Item Price', 'invoicing'); ?></th>
+            <?php if ($quantities_enabled) { ?>
+            <th class="wpinv_cart_item_qty text-right"><?php _e('Qty', 'invoicing'); ?></th>
             <?php } ?>
-            <?php if ( $use_taxes ) { ?>
+            <?php if ($use_taxes) { ?>
             <th class="wpinv_cart_item_tax text-right"><?php echo $tax_label . ' <span class="normal small">(%)</span>'; ?></th>
             <?php } ?>
-            <th class="wpinv_cart_item_subtotal text-right"><?php echo __( 'Item Total', 'invoicing' ) . ' <span class="normal small">' . $tax_title . '<span>'; ?></th>
-            <?php do_action( 'wpinv_checkout_table_header_last' ); ?>
+            <th class="wpinv_cart_item_subtotal text-right"><?php echo __('Item Total', 'invoicing') . ' <span class="normal small">' . $tax_title . '<span>'; ?></th>
+            <?php do_action('wpinv_checkout_table_header_last'); ?>
         </tr>
     </thead>
     <tbody>
         <?php
-            do_action( 'wpinv_cart_items_before' );
+            do_action('wpinv_cart_items_before');
             
-            if ( $cart_items ) {
-                foreach ( $cart_items as $key => $item ) {
-                    $wpi_item = !empty( $item['id'] ) ? new WPInv_Item( $item['id'] ) : NULL;
+            if ($cart_items) {
+                foreach ($cart_items as $key => $item) {
+                    $wpi_item = !empty($item['id']) ? new WPInv_Item($item['id']) : NULL;
                 ?>
-                <tr class="wpinv_cart_item" id="wpinv_cart_item_<?php echo esc_attr( $key ) . '_' . esc_attr( $item['id'] ); ?>" data-item-id="<?php echo esc_attr( $item['id'] ); ?>">
-                    <?php do_action( 'wpinv_checkout_table_body_first', $item ); ?>
+                <tr class="wpinv_cart_item" id="wpinv_cart_item_<?php echo esc_attr($key) . '_' . esc_attr($item['id']); ?>" data-item-id="<?php echo esc_attr($item['id']); ?>">
+                    <?php do_action('wpinv_checkout_table_body_first', $item); ?>
                     <td class="wpinv_cart_item_name text-left">
                         <?php
-                            if ( current_theme_supports( 'post-thumbnails' ) && has_post_thumbnail( $item['id'] ) ) {
+                            if (current_theme_supports('post-thumbnails') && has_post_thumbnail($item['id'])) {
                                 echo '<div class="wpinv_cart_item_image">';
-                                    echo get_the_post_thumbnail( $item['id'], apply_filters( 'wpinv_checkout_image_size', array( 25,25 ) ) );
+                                    echo get_the_post_thumbnail($item['id'], apply_filters('wpinv_checkout_image_size', array(25, 25)));
                                 echo '</div>';
                             }
-                            $item_title = esc_html( wpinv_get_cart_item_name( $item ) ) . wpinv_get_item_suffix( $wpi_item );
+                            $item_title = esc_html(wpinv_get_cart_item_name($item)) . wpinv_get_item_suffix($wpi_item);
                             echo '<span class="wpinv_checkout_cart_item_title">' . $item_title . '</span>';
-                            if ( !empty( $wpi_item ) && $wpi_item->is_package() && !empty( $item['meta']['post_id'] ) ) {
-                                $post_link = !empty( $item['meta']['invoice_title'] ) ? $item['meta']['invoice_title'] : get_the_title( $item['meta']['post_id'] );
-                                $summary = wp_sprintf( __( '%s: %s', 'invoicing' ), $wpi_item->get_custom_singular_name(), $post_link );
-                                echo '<small class="meta">' . wpautop( wp_kses_post( $summary ) ) . '</small>';
+                            if (!empty($wpi_item) && $wpi_item->is_package() && !empty($item['meta']['post_id'])) {
+                                $post_link = !empty($item['meta']['invoice_title']) ? $item['meta']['invoice_title'] : get_the_title($item['meta']['post_id']);
+                                $summary = wp_sprintf(__('%s: %s', 'invoicing'), $wpi_item->get_custom_singular_name(), $post_link);
+                                echo '<small class="meta">' . wpautop(wp_kses_post($summary)) . '</small>';
                             }
-                            do_action( 'wpinv_checkout_cart_item_title_after', $item, $key );
+                            do_action('wpinv_checkout_cart_item_title_after', $item, $key);
                         ?>
                     </td>
                     <td class="wpinv_cart_item_price text-right">
                         <?php 
-                        echo wpinv_cart_item_price( $item );
-                        do_action( 'wpinv_checkout_cart_item_price_after', $item, $key );
+                        echo wpinv_cart_item_price($item);
+                        do_action('wpinv_checkout_cart_item_price_after', $item, $key);
                         ?>
                     </td>
-                    <?php if ( $quantities_enabled ) { ?>
+                    <?php if ($quantities_enabled) { ?>
                     <td class="wpinv_cart_item_qty text-right">
                         <?php
-                        echo wpinv_get_cart_item_quantity( $item );
-                        do_action( 'wpinv_cart_item_quantitiy', $item, $key );
+                        echo wpinv_get_cart_item_quantity($item);
+                        do_action('wpinv_cart_item_quantitiy', $item, $key);
                         ?>
                     </td>
                     <?php } ?>
-                    <?php if ( $use_taxes ) { ?>
+                    <?php if ($use_taxes) { ?>
                     <td class="wpinv_cart_item_tax text-right">
                         <?php
-                        echo wpinv_cart_item_tax( $item );
+                        echo wpinv_cart_item_tax($item);
                         //echo wpinv_get_cart_item_tax( $wpi_item->ID, $subtotal = '', $options = array() );
-                        do_action( 'wpinv_cart_item_tax', $item, $key );
+                        do_action('wpinv_cart_item_tax', $item, $key);
                         ?>
                     </td>
                     <?php } ?>
                     <td class="wpinv_cart_item_subtotal text-right">
                         <?php
-                        echo wpinv_cart_item_subtotal( $item );
-                        do_action( 'wpinv_cart_item_subtotal', $item, $key );
+                        echo wpinv_cart_item_subtotal($item);
+                        do_action('wpinv_cart_item_subtotal', $item, $key);
                         ?>
                     </td>
-                    <?php do_action( 'wpinv_checkout_table_body_last', $item, $key ); ?>
+                    <?php do_action('wpinv_checkout_table_body_last', $item, $key); ?>
                 </tr>
             <?php } ?>
         <?php } ?>
-        <?php do_action( 'wpinv_cart_items_middle' ); ?>
-        <?php do_action( 'wpinv_cart_items_after' ); ?>
+        <?php do_action('wpinv_cart_items_middle'); ?>
+        <?php do_action('wpinv_cart_items_after'); ?>
     </tbody>
     <tfoot>
         <?php $cart_columns = wpinv_checkout_cart_columns(); ?>
-        <?php if ( has_action( 'wpinv_cart_footer_buttons' ) ) { ?>
+        <?php if (has_action('wpinv_cart_footer_buttons')) { ?>
             <tr class="wpinv_cart_footer_row">
-                <?php do_action( 'wpinv_checkout_table_buttons_first', $cart_items ); ?>
-                <td colspan="<?php echo ( $cart_columns ); ?>">
-                    <?php do_action( 'wpinv_cart_footer_buttons' ); ?>
+                <?php do_action('wpinv_checkout_table_buttons_first', $cart_items); ?>
+                <td colspan="<?php echo ($cart_columns); ?>">
+                    <?php do_action('wpinv_cart_footer_buttons'); ?>
                 </td>
-                <?php do_action( 'wpinv_checkout_table_buttons_first', $cart_items ); ?>
+                <?php do_action('wpinv_checkout_table_buttons_first', $cart_items); ?>
             </tr>
         <?php } ?>
 
-        <?php if ( $use_taxes && !wpinv_prices_include_tax() ) { ?>
-            <tr class="wpinv_cart_footer_row wpinv_cart_subtotal_row"<?php if ( !wpinv_is_cart_taxed() ) echo ' style="display:none;"'; ?>>
-                <?php do_action( 'wpinv_checkout_table_subtotal_first', $cart_items ); ?>
-                <td colspan="<?php echo ( $cart_columns - 1 ); ?>" class="wpinv_cart_subtotal_label text-right">
-                    <strong><?php _e( 'Sub-Total', 'invoicing' ); ?>:</strong>
+        <?php if ($use_taxes && !wpinv_prices_include_tax()) { ?>
+            <tr class="wpinv_cart_footer_row wpinv_cart_subtotal_row"<?php if (!wpinv_is_cart_taxed()) echo ' style="display:none;"'; ?>>
+                <?php do_action('wpinv_checkout_table_subtotal_first', $cart_items); ?>
+                <td colspan="<?php echo ($cart_columns - 1); ?>" class="wpinv_cart_subtotal_label text-right">
+                    <strong><?php _e('Sub-Total', 'invoicing'); ?>:</strong>
                 </td>
                 <td class="wpinv_cart_subtotal text-right">
-                    <span class="wpinv_cart_subtotal_amount bold"><?php echo wpinv_cart_subtotal( $cart_items ); ?></span>
+                    <span class="wpinv_cart_subtotal_amount bold"><?php echo wpinv_cart_subtotal($cart_items); ?></span>
                 </td>
-                <?php do_action( 'wpinv_checkout_table_subtotal_last', $cart_items ); ?>
+                <?php do_action('wpinv_checkout_table_subtotal_last', $cart_items); ?>
             </tr>
         <?php } ?>
         
-        <?php $wpi_cart_columns = $cart_columns - 1; wpinv_cart_discounts_html( $cart_items ); ?>
+        <?php $wpi_cart_columns = $cart_columns - 1; wpinv_cart_discounts_html($cart_items); ?>
 
-        <?php if ( $use_taxes ) { ?>
-            <tr class="wpinv_cart_footer_row wpinv_cart_tax_row"<?php if( !wpinv_is_cart_taxed() ) echo ' style="display:none;"'; ?>>
-                <?php do_action( 'wpinv_checkout_table_tax_first' ); ?>
-                <td colspan="<?php echo ( $cart_columns - 1 ); ?>" class="wpinv_cart_tax_label text-right">
+        <?php if ($use_taxes) { ?>
+            <tr class="wpinv_cart_footer_row wpinv_cart_tax_row"<?php if (!wpinv_is_cart_taxed()) echo ' style="display:none;"'; ?>>
+                <?php do_action('wpinv_checkout_table_tax_first'); ?>
+                <td colspan="<?php echo ($cart_columns - 1); ?>" class="wpinv_cart_tax_label text-right">
                     <strong><?php echo $tax_label; ?>:</strong>
                 </td>
                 <td class="wpinv_cart_tax text-right">
-                    <span class="wpinv_cart_tax_amount" data-tax="<?php echo wpinv_get_cart_tax( $cart_items ); ?>"><?php echo esc_html( wpinv_cart_tax( $cart_items ) ); ?></span>
+                    <span class="wpinv_cart_tax_amount" data-tax="<?php echo wpinv_get_cart_tax($cart_items); ?>"><?php echo esc_html(wpinv_cart_tax($cart_items)); ?></span>
                 </td>
-                <?php do_action( 'wpinv_checkout_table_tax_last' ); ?>
+                <?php do_action('wpinv_checkout_table_tax_last'); ?>
             </tr>
         <?php } ?>
 
         <tr class="wpinv_cart_footer_row wpinv_cart_total_row">
-            <?php do_action( 'wpinv_checkout_table_footer_first' ); ?>
-            <td colspan="<?php echo ( $cart_columns - 1 ); ?>" class="wpinv_cart_total_label text-right">
-                <?php echo apply_filters( 'wpinv_cart_total_label', '<strong>' . __( 'Total', 'invoicing' ) . ':</strong>', $invoice ); ?>
+            <?php do_action('wpinv_checkout_table_footer_first'); ?>
+            <td colspan="<?php echo ($cart_columns - 1); ?>" class="wpinv_cart_total_label text-right">
+                <?php echo apply_filters('wpinv_cart_total_label', '<strong>' . __('Total', 'invoicing') . ':</strong>', $invoice); ?>
             </td>
             <td class="wpinv_cart_total text-right">
-                <span class="wpinv_cart_amount bold" data-subtotal="<?php echo wpinv_get_cart_total( $cart_items ); ?>" data-total="<?php echo wpinv_get_cart_total( NULL, NULL, $invoice ); ?>"><?php wpinv_cart_total( $cart_items, true, $invoice ); ?></span>
+                <span class="wpinv_cart_amount bold" data-subtotal="<?php echo wpinv_get_cart_total($cart_items); ?>" data-total="<?php echo wpinv_get_cart_total(NULL, NULL, $invoice); ?>"><?php wpinv_cart_total($cart_items, true, $invoice); ?></span>
             </td>
-            <?php do_action( 'wpinv_checkout_table_footer_last' ); ?>
+            <?php do_action('wpinv_checkout_table_footer_last'); ?>
         </tr>
     </tfoot>
 </table>

includes/class-wpinv-reports.php

Spacing +169 added lines, -169 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if ( ! defined( 'ABSPATH' ) ) {
+if (!defined('ABSPATH')) {
     exit; // Exit if accessed directly
 }
 
@@ -21,68 +21,68 @@ 
     public function init() {
         global $wp_filesystem;
 
-        if ( empty( $wp_filesystem ) ) {
-            require_once( ABSPATH . '/wp-admin/includes/file.php' );
+        if (empty($wp_filesystem)) {
+            require_once(ABSPATH . '/wp-admin/includes/file.php');
             WP_Filesystem();
             global $wp_filesystem;
         }
         $this->wp_filesystem    = $wp_filesystem;
         
         $this->export_dir       = $this->export_location();
-        $this->export_url       = $this->export_location( true );
+        $this->export_url       = $this->export_location(true);
         $this->export           = 'invoicing';
         $this->filetype         = 'csv';
         $this->per_page         = 20;
         
-        do_action( 'wpinv_class_reports_init', $this );
+        do_action('wpinv_class_reports_init', $this);
     }
     
     public function includes() {
-        do_action( 'wpinv_class_reports_includes', $this );
+        do_action('wpinv_class_reports_includes', $this);
     }
     
     public function actions() {
-        if ( is_admin() ) {
-            add_action( 'admin_menu', array( $this, 'add_submenu' ), 10 );
-            add_action( 'wpinv_reports_tab_export', array( $this, 'export' ) );
-            add_action( 'wp_ajax_wpinv_ajax_export', array( $this, 'ajax_export' ) );
+        if (is_admin()) {
+            add_action('admin_menu', array($this, 'add_submenu'), 10);
+            add_action('wpinv_reports_tab_export', array($this, 'export'));
+            add_action('wp_ajax_wpinv_ajax_export', array($this, 'ajax_export'));
             
             // Export Invoices.
-            add_action( 'wpinv_export_set_params_invoices', array( $this, 'set_invoices_export' ) );
-            add_filter( 'wpinv_export_get_columns_invoices', array( $this, 'get_invoices_columns' ) );
-            add_filter( 'wpinv_export_get_data_invoices', array( $this, 'get_invoices_data' ) );
-            add_filter( 'wpinv_get_export_status_invoices', array( $this, 'invoices_export_status' ) );
+            add_action('wpinv_export_set_params_invoices', array($this, 'set_invoices_export'));
+            add_filter('wpinv_export_get_columns_invoices', array($this, 'get_invoices_columns'));
+            add_filter('wpinv_export_get_data_invoices', array($this, 'get_invoices_data'));
+            add_filter('wpinv_get_export_status_invoices', array($this, 'invoices_export_status'));
         }
-        do_action( 'wpinv_class_reports_actions', $this );
+        do_action('wpinv_class_reports_actions', $this);
     }
     
     public function add_submenu() {
         global $wpi_reports_page;
-        $wpi_reports_page = add_submenu_page( 'wpinv', __( 'Reports', 'invoicing' ), __( 'Reports', 'invoicing' ), 'manage_options', 'wpinv-reports', array( $this, 'reports_page' ) );
+        $wpi_reports_page = add_submenu_page('wpinv', __('Reports', 'invoicing'), __('Reports', 'invoicing'), 'manage_options', 'wpinv-reports', array($this, 'reports_page'));
     }
     
     public function reports_page() {
-        if ( !wp_script_is( 'postbox', 'enqueued' ) ) {
-            wp_enqueue_script( 'postbox' );
+        if (!wp_script_is('postbox', 'enqueued')) {
+            wp_enqueue_script('postbox');
         }
-        if ( !wp_script_is( 'jquery-ui-datepicker', 'enqueued' ) ) {
-            wp_enqueue_script( 'jquery-ui-datepicker' );
+        if (!wp_script_is('jquery-ui-datepicker', 'enqueued')) {
+            wp_enqueue_script('jquery-ui-datepicker');
         }
         
-        $current_page = admin_url( 'admin.php?page=wpinv-reports' );
-        $active_tab = isset( $_GET['tab'] ) ? sanitize_text_field( $_GET['tab'] ) : 'export';
+        $current_page = admin_url('admin.php?page=wpinv-reports');
+        $active_tab = isset($_GET['tab']) ? sanitize_text_field($_GET['tab']) : 'export';
         ?>
         <div class="wrap wpi-reports-wrap">
-            <h1><?php echo esc_html( __( 'Reports', 'invoicing' ) ); ?></h1>
+            <h1><?php echo esc_html(__('Reports', 'invoicing')); ?></h1>
             <h2 class="nav-tab-wrapper wp-clearfix">
-                <a href="<?php echo add_query_arg( array( 'tab' => 'export', 'settings-updated' => false ), $current_page ); ?>" class="nav-tab <?php echo $active_tab == 'export' ? 'nav-tab-active' : ''; ?>"><?php _e( 'Export', 'invoicing' ); ?></a>
-                <?php do_action( 'wpinv_reports_page_tabs' ); ;?>
+                <a href="<?php echo add_query_arg(array('tab' => 'export', 'settings-updated' => false), $current_page); ?>" class="nav-tab <?php echo $active_tab == 'export' ? 'nav-tab-active' : ''; ?>"><?php _e('Export', 'invoicing'); ?></a>
+                <?php do_action('wpinv_reports_page_tabs'); ;?>
             </h2>
             <div class="wpi-reports-content wpi-reports-<?php echo $active_tab; ?>">
             <?php
-                do_action( 'wpinv_reports_page_top' );
-                do_action( 'wpinv_reports_tab_' . $active_tab );
-                do_action( 'wpinv_reports_page_bottom' );
+                do_action('wpinv_reports_page_top');
+                do_action('wpinv_reports_tab_' . $active_tab);
+                do_action('wpinv_reports_page_bottom');
             ?>
         </div>
         <?php
@@ -90,97 +90,97 @@ 
     
     public function export() {
         $statuses = wpinv_get_invoice_statuses();
-        $statuses = array_merge( array( 'any' => __( 'All Statuses', 'invoicing' ) ), $statuses );
+        $statuses = array_merge(array('any' => __('All Statuses', 'invoicing')), $statuses);
         ?>
         <div class="metabox-holder">
             <div id="post-body">
                 <div id="post-body-content">
-                    <?php do_action( 'wpinv_reports_tab_export_content_top' ); ?>
+                    <?php do_action('wpinv_reports_tab_export_content_top'); ?>
                     
                     <div class="postbox wpi-export-invoices">
-                        <h2 class="hndle ui-sortabled-handle"><span><?php _e( 'Invoices','invoicing' ); ?></span></h2>
+                        <h2 class="hndle ui-sortabled-handle"><span><?php _e('Invoices', 'invoicing'); ?></span></h2>
                         <div class="inside">
-                            <p><?php _e( 'Download a CSV of all payment invoices.', 'invoicing' ); ?></p>
+                            <p><?php _e('Download a CSV of all payment invoices.', 'invoicing'); ?></p>
                             <form id="wpi-export-invoices" class="wpi-export-form" method="post">
-                                <?php echo wpinv_html_date_field( array( 
+                                <?php echo wpinv_html_date_field(array( 
                                     'id' => 'wpi_export_from_date', 
                                     'name' => 'from_date',
                                     'data' => array(
                                         'dateFormat' => 'yy-mm-dd'
                                     ),
-                                    'placeholder' => __( 'From date', 'invoicing' ) )
+                                    'placeholder' => __('From date', 'invoicing') )
                                 ); ?>
-                                <?php echo wpinv_html_date_field( array( 
+                                <?php echo wpinv_html_date_field(array( 
                                     'id' => 'wpi_export_to_date',
                                     'name' => 'to_date',
                                     'data' => array(
                                         'dateFormat' => 'yy-mm-dd'
                                     ),
-                                    'placeholder' => __( 'To date', 'invoicing' ) )
+                                    'placeholder' => __('To date', 'invoicing') )
                                 ); ?>
                                 <span id="wpinv-status-wrap">
-                                <?php echo wpinv_html_select( array(
+                                <?php echo wpinv_html_select(array(
                                     'options'          => $statuses,
                                     'name'             => 'status',
                                     'id'               => 'wpi_export_status',
                                     'show_option_all'  => false,
                                     'show_option_none' => false,
                                     'class'            => '',
-                                ) ); ?>
-                                <?php wp_nonce_field( 'wpi_ajax_export', 'wpi_ajax_export' ); ?>
+                                )); ?>
+                                <?php wp_nonce_field('wpi_ajax_export', 'wpi_ajax_export'); ?>
                                 </span>
                                 <span id="wpinv-submit-wrap">
                                     <input type="hidden" value="invoices" name="export" />
-                                    <input type="submit" value="<?php _e( 'Generate CSV', 'invoicing' ); ?>" class="button-primary" />
+                                    <input type="submit" value="<?php _e('Generate CSV', 'invoicing'); ?>" class="button-primary" />
                                 </span>
                             </form>
                         </div>
                     </div>
                     
-                    <?php do_action( 'wpinv_reports_tab_export_content_bottom' ); ?>
+                    <?php do_action('wpinv_reports_tab_export_content_bottom'); ?>
                 </div>
             </div>
         </div>
         <?php
     }
     
-    public function export_location( $relative = false ) {
+    public function export_location($relative = false) {
         $upload_dir         = wp_upload_dir();
-        $export_location    = $relative ? trailingslashit( $upload_dir['baseurl'] ) . 'cache' : trailingslashit( $upload_dir['basedir'] ) . 'cache';
-        $export_location    = apply_filters( 'wpinv_export_location', $export_location, $relative );
+        $export_location    = $relative ? trailingslashit($upload_dir['baseurl']) . 'cache' : trailingslashit($upload_dir['basedir']) . 'cache';
+        $export_location    = apply_filters('wpinv_export_location', $export_location, $relative);
         
-        return trailingslashit( $export_location );
+        return trailingslashit($export_location);
     }
     
     public function check_export_location() {
         try {
-            if ( empty( $this->wp_filesystem ) ) {
-                return __( 'Filesystem ERROR: Could not access filesystem.', 'invoicing' );
+            if (empty($this->wp_filesystem)) {
+                return __('Filesystem ERROR: Could not access filesystem.', 'invoicing');
             }
 
-            if ( is_wp_error( $this->wp_filesystem ) ) {
-                return __( 'Filesystem ERROR: ' . $this->wp_filesystem->get_error_message(), 'invoicing' );
+            if (is_wp_error($this->wp_filesystem)) {
+                return __('Filesystem ERROR: ' . $this->wp_filesystem->get_error_message(), 'invoicing');
             }
         
-            $is_dir         = $this->wp_filesystem->is_dir( $this->export_dir );
-            $is_writeable   = $is_dir && is_writeable( $this->export_dir );
+            $is_dir         = $this->wp_filesystem->is_dir($this->export_dir);
+            $is_writeable   = $is_dir && is_writeable($this->export_dir);
             
-            if ( $is_dir && $is_writeable ) {
+            if ($is_dir && $is_writeable) {
                return true;
-            } else if ( $is_dir && !$is_writeable ) {
-               if ( !$this->wp_filesystem->chmod( $this->export_dir, FS_CHMOD_DIR ) ) {
-                   return wp_sprintf( __( 'Filesystem ERROR: Export location %s is not writable, check your file permissions.', 'invoicing' ), $this->export_dir );
+            } else if ($is_dir && !$is_writeable) {
+               if (!$this->wp_filesystem->chmod($this->export_dir, FS_CHMOD_DIR)) {
+                   return wp_sprintf(__('Filesystem ERROR: Export location %s is not writable, check your file permissions.', 'invoicing'), $this->export_dir);
                }
                
                return true;
             } else {
-                if ( !$this->wp_filesystem->mkdir( $this->export_dir, FS_CHMOD_DIR ) ) {
-                    return wp_sprintf( __( 'Filesystem ERROR: Could not create directory %s. This is usually due to inconsistent file permissions.', 'invoicing' ), $this->export_dir );
+                if (!$this->wp_filesystem->mkdir($this->export_dir, FS_CHMOD_DIR)) {
+                    return wp_sprintf(__('Filesystem ERROR: Could not create directory %s. This is usually due to inconsistent file permissions.', 'invoicing'), $this->export_dir);
                 }
                 
                 return true;
             }
-        } catch ( Exception $e ) {
+        } catch (Exception $e) {
             return $e->getMessage();
         }
     }
@@ -188,130 +188,130 @@ 
     public function ajax_export() {
         $response               = array();
         $response['success']    = false;
-        $response['msg']        = __( 'Invalid export request found.', 'invoicing' );
+        $response['msg']        = __('Invalid export request found.', 'invoicing');
         
-        if ( empty( $_POST['data'] ) || !current_user_can( 'manage_options' ) ) {
-            wp_send_json( $response );
+        if (empty($_POST['data']) || !current_user_can('manage_options')) {
+            wp_send_json($response);
         }
 
-        parse_str( $_POST['data'], $data );
+        parse_str($_POST['data'], $data);
         
-        $data['step']   = !empty( $_POST['step'] ) ? absint( $_POST['step'] ) : 1;
+        $data['step'] = !empty($_POST['step']) ? absint($_POST['step']) : 1;
 
         $_REQUEST = (array)$data;
-        if ( !( !empty( $_REQUEST['wpi_ajax_export'] ) && wp_verify_nonce( $_REQUEST['wpi_ajax_export'], 'wpi_ajax_export' ) ) ) {
-            $response['msg']    = __( 'Security check failed.', 'invoicing' );
-            wp_send_json( $response );
+        if (!(!empty($_REQUEST['wpi_ajax_export']) && wp_verify_nonce($_REQUEST['wpi_ajax_export'], 'wpi_ajax_export'))) {
+            $response['msg'] = __('Security check failed.', 'invoicing');
+            wp_send_json($response);
         }
         
-        if ( ( $error = $this->check_export_location( true ) ) !== true ) {
-            $response['msg'] = __( 'Filesystem ERROR: ' . $error, 'invoicing' );
-            wp_send_json( $response );
+        if (($error = $this->check_export_location(true)) !== true) {
+            $response['msg'] = __('Filesystem ERROR: ' . $error, 'invoicing');
+            wp_send_json($response);
         }
                         
-        $this->set_export_params( $_REQUEST );
+        $this->set_export_params($_REQUEST);
         
         $return = $this->process_export_step();
         $done   = $this->get_export_status();
         
-        if ( $return ) {
+        if ($return) {
             $this->step += 1;
             
             $response['success']    = true;
             $response['msg']        = '';
             
-            if ( $done >= 100 ) {
+            if ($done >= 100) {
                 $this->step     = 'done';
-                $new_filename   = 'wpi-' . $this->export . '-' . date( 'y-m-d-H-i' ) . '.' . $this->filetype;
+                $new_filename   = 'wpi-' . $this->export . '-' . date('y-m-d-H-i') . '.' . $this->filetype;
                 $new_file       = $this->export_dir . $new_filename;
                 
-                if ( file_exists( $this->file ) ) {
-                    $this->wp_filesystem->move( $this->file, $new_file, true );
+                if (file_exists($this->file)) {
+                    $this->wp_filesystem->move($this->file, $new_file, true);
                 }
                 
-                if ( file_exists( $new_file ) ) {
-                    $response['data']['file'] = array( 'u' => $this->export_url . $new_filename, 's' => size_format( filesize( $new_file ), 2 ) );
+                if (file_exists($new_file)) {
+                    $response['data']['file'] = array('u' => $this->export_url . $new_filename, 's' => size_format(filesize($new_file), 2));
                 }
             }
             
             $response['data']['step']   = $this->step;
             $response['data']['done']   = $done;
         } else {
-            $response['msg']    = __( 'No data found for export.', 'invoicing' );
+            $response['msg'] = __('No data found for export.', 'invoicing');
         }
 
-        wp_send_json( $response );
+        wp_send_json($response);
     }
     
-    public function set_export_params( $request ) {
+    public function set_export_params($request) {
         $this->empty    = false;
-        $this->step     = !empty( $request['step'] ) ? absint( $request['step'] ) : 1;
-        $this->export   = !empty( $request['export'] ) ? $request['export'] : $this->export;
+        $this->step     = !empty($request['step']) ? absint($request['step']) : 1;
+        $this->export   = !empty($request['export']) ? $request['export'] : $this->export;
         $this->filename = 'wpi-' . $this->export . '-' . $request['wpi_ajax_export'] . '.' . $this->filetype;
         $this->file     = $this->export_dir . $this->filename;
         
-        do_action( 'wpinv_export_set_params_' . $this->export, $request );
+        do_action('wpinv_export_set_params_' . $this->export, $request);
     }
     
     public function get_columns() {
         $columns = array(
-            'id'   => __( 'ID',   'invoicing' ),
-            'date' => __( 'Date', 'invoicing' )
+            'id'   => __('ID', 'invoicing'),
+            'date' => __('Date', 'invoicing')
         );
         
-        return apply_filters( 'wpinv_export_get_columns_' . $this->export, $columns );
+        return apply_filters('wpinv_export_get_columns_' . $this->export, $columns);
     }
     
     protected function get_export_file() {
         $file = '';
 
-        if ( $this->wp_filesystem->exists( $this->file ) ) {
-            $file = $this->wp_filesystem->get_contents( $this->file );
+        if ($this->wp_filesystem->exists($this->file)) {
+            $file = $this->wp_filesystem->get_contents($this->file);
         } else {
-            $this->wp_filesystem->put_contents( $this->file, '' );
+            $this->wp_filesystem->put_contents($this->file, '');
         }
 
         return $file;
     }
     
-    protected function attach_export_data( $data = '' ) {
-        $filedata   = $this->get_export_file();
-        $filedata   .= $data;
+    protected function attach_export_data($data = '') {
+        $filedata = $this->get_export_file();
+        $filedata .= $data;
         
-        $this->wp_filesystem->put_contents( $this->file, $filedata );
+        $this->wp_filesystem->put_contents($this->file, $filedata);
 
-        $rows       = file( $this->file, FILE_SKIP_EMPTY_LINES );
+        $rows       = file($this->file, FILE_SKIP_EMPTY_LINES);
         $columns    = $this->get_columns();
-        $columns    = empty( $columns ) ? 0 : 1;
+        $columns    = empty($columns) ? 0 : 1;
 
-        $this->empty = count( $rows ) == $columns ? true : false;
+        $this->empty = count($rows) == $columns ? true : false;
     }
     
     public function print_columns() {
         $column_data    = '';
         $columns        = $this->get_columns();
         $i              = 1;
-        foreach( $columns as $key => $column ) {
-            $column_data .= '"' . addslashes( $column ) . '"';
-            $column_data .= $i == count( $columns ) ? '' : ',';
+        foreach ($columns as $key => $column) {
+            $column_data .= '"' . addslashes($column) . '"';
+            $column_data .= $i == count($columns) ? '' : ',';
             $i++;
         }
         $column_data .= "\r\n";
 
-        $this->attach_export_data( $column_data );
+        $this->attach_export_data($column_data);
 
         return $column_data;
     }
     
     public function process_export_step() {
-        if ( $this->step < 2 ) {
-            @unlink( $this->file );
+        if ($this->step < 2) {
+            @unlink($this->file);
             $this->print_columns();
         }
         
         $return = $this->print_rows();
         
-        if ( $return ) {
+        if ($return) {
             return true;
         } else {
             return false;
@@ -320,23 +320,23 @@ 
     
     public function get_export_status() {
         $status = 100;
-        return apply_filters( 'wpinv_get_export_status_' . $this->export, $status );
+        return apply_filters('wpinv_get_export_status_' . $this->export, $status);
     }
     
     public function get_export_data() {
         $data = array(
             0 => array(
                 'id'   => '',
-                'data' => date( 'F j, Y' )
+                'data' => date('F j, Y')
             ),
             1 => array(
                 'id'   => '',
-                'data' => date( 'F j, Y' )
+                'data' => date('F j, Y')
             )
         );
 
-        $data = apply_filters( 'wpinv_export_get_data', $data );
-        $data = apply_filters( 'wpinv_export_get_data_' . $this->export, $data );
+        $data = apply_filters('wpinv_export_get_data', $data);
+        $data = apply_filters('wpinv_export_get_data_' . $this->export, $data);
 
         return $data;
     }
@@ -346,20 +346,20 @@ 
         $data       = $this->get_export_data();
         $columns    = $this->get_columns();
 
-        if ( $data ) {
-            foreach ( $data as $row ) {
+        if ($data) {
+            foreach ($data as $row) {
                 $i = 1;
-                foreach ( $row as $key => $column ) {
-                    if ( array_key_exists( $key, $columns ) ) {
-                        $row_data .= '"' . addslashes( preg_replace( "/\"/","'", $column ) ) . '"';
-                        $row_data .= $i == count( $columns ) ? '' : ',';
+                foreach ($row as $key => $column) {
+                    if (array_key_exists($key, $columns)) {
+                        $row_data .= '"' . addslashes(preg_replace("/\"/", "'", $column)) . '"';
+                        $row_data .= $i == count($columns) ? '' : ',';
                         $i++;
                     }
                 }
                 $row_data .= "\r\n";
             }
 
-            $this->attach_export_data( $row_data );
+            $this->attach_export_data($row_data);
 
             return $row_data;
         }
@@ -368,46 +368,46 @@ 
     }
     
     // Export Invoices.
-    public function set_invoices_export( $request ) {
-        $this->from_date    = isset( $request['from_date'] ) ? sanitize_text_field( $request['from_date'] ) : '';
-        $this->to_date      = isset( $request['to_date'] ) ? sanitize_text_field( $request['to_date'] ) : '';
-        $this->status       = isset( $request['status'] ) ? sanitize_text_field( $request['status'] ) : 'publish';
+    public function set_invoices_export($request) {
+        $this->from_date    = isset($request['from_date']) ? sanitize_text_field($request['from_date']) : '';
+        $this->to_date      = isset($request['to_date']) ? sanitize_text_field($request['to_date']) : '';
+        $this->status       = isset($request['status']) ? sanitize_text_field($request['status']) : 'publish';
     }
     
-    public function get_invoices_columns( $columns = array() ) {
+    public function get_invoices_columns($columns = array()) {
         $columns = array(
-            'id'            => __( 'ID',   'invoicing' ),
-            'number'        => __( 'Number',   'invoicing' ),
-            'date'          => __( 'Date', 'invoicing' ),
-            'amount'        => __( 'Amount', 'invoicing' ),
-            'status_nicename'  => __( 'Status Nicename', 'invoicing' ),
-            'status'        => __( 'Status', 'invoicing' ),
-            'tax'           => __( 'Tax', 'invoicing' ),
-            'discount'      => __( 'Discount', 'invoicing' ),
-            'user_id'       => __( 'User ID', 'invoicing' ),
-            'email'         => __( 'Email', 'invoicing' ),
-            'first_name'    => __( 'First Name', 'invoicing' ),
-            'last_name'     => __( 'Last Name', 'invoicing' ),
-            'address'       => __( 'Address', 'invoicing' ),
-            'city'          => __( 'City', 'invoicing' ),
-            'state'         => __( 'State', 'invoicing' ),
-            'country'       => __( 'Country', 'invoicing' ),
-            'zip'           => __( 'Zipcode', 'invoicing' ),
-            'phone'         => __( 'Phone', 'invoicing' ),
-            'company'       => __( 'Company', 'invoicing' ),
-            'vat_number'    => __( 'Vat Number', 'invoicing' ),
-            'ip'            => __( 'IP', 'invoicing' ),
-            'gateway'       => __( 'Gateway', 'invoicing' ),
-            'gateway_nicename'       => __( 'Gateway Nicename', 'invoicing' ),
-            'transaction_id'=> __( 'Transaction ID', 'invoicing' ),
-            'currency'      => __( 'Currency', 'invoicing' ),
-            'due_date'      => __( 'Due Date', 'invoicing' ),
+            'id'            => __('ID', 'invoicing'),
+            'number'        => __('Number', 'invoicing'),
+            'date'          => __('Date', 'invoicing'),
+            'amount'        => __('Amount', 'invoicing'),
+            'status_nicename'  => __('Status Nicename', 'invoicing'),
+            'status'        => __('Status', 'invoicing'),
+            'tax'           => __('Tax', 'invoicing'),
+            'discount'      => __('Discount', 'invoicing'),
+            'user_id'       => __('User ID', 'invoicing'),
+            'email'         => __('Email', 'invoicing'),
+            'first_name'    => __('First Name', 'invoicing'),
+            'last_name'     => __('Last Name', 'invoicing'),
+            'address'       => __('Address', 'invoicing'),
+            'city'          => __('City', 'invoicing'),
+            'state'         => __('State', 'invoicing'),
+            'country'       => __('Country', 'invoicing'),
+            'zip'           => __('Zipcode', 'invoicing'),
+            'phone'         => __('Phone', 'invoicing'),
+            'company'       => __('Company', 'invoicing'),
+            'vat_number'    => __('Vat Number', 'invoicing'),
+            'ip'            => __('IP', 'invoicing'),
+            'gateway'       => __('Gateway', 'invoicing'),
+            'gateway_nicename'       => __('Gateway Nicename', 'invoicing'),
+            'transaction_id'=> __('Transaction ID', 'invoicing'),
+            'currency'      => __('Currency', 'invoicing'),
+            'due_date'      => __('Due Date', 'invoicing'),
         );
 
         return $columns;
     }
         
-    public function get_invoices_data( $response = array() ) {
+    public function get_invoices_data($response = array()) {
         $args = array(
             'limit'    => $this->per_page,
             'page'     => $this->step,
@@ -415,35 +415,35 @@ 
             'orderby'  => 'date',
         );
         
-        if ( $this->status != 'any' ) {
+        if ($this->status != 'any') {
             $args['status'] = $this->status;
         }
 
-        if ( !empty( $this->from_date ) || !empty( $this->to_date ) ) {
+        if (!empty($this->from_date) || !empty($this->to_date)) {
             $args['date_query'] = array(
                 array(
-                    'after'     => date( 'Y-n-d 00:00:00', strtotime( $this->from_date ) ),
-                    'before'    => date( 'Y-n-d 23:59:59', strtotime( $this->to_date ) ),
+                    'after'     => date('Y-n-d 00:00:00', strtotime($this->from_date)),
+                    'before'    => date('Y-n-d 23:59:59', strtotime($this->to_date)),
                     'inclusive' => true
                 )
             );
         }
 
-        $invoices = wpinv_get_invoices( $args );
+        $invoices = wpinv_get_invoices($args);
         
         $data = array();
         
-        if ( !empty( $invoices ) ) {
-            foreach ( $invoices as $invoice ) {
+        if (!empty($invoices)) {
+            foreach ($invoices as $invoice) {
                 $row = array(
                     'id'            => $invoice->ID,
                     'number'        => $invoice->get_number(),
-                    'date'          => $invoice->get_invoice_date( false ),
-                    'amount'        => wpinv_round_amount( $invoice->get_total() ),
-                    'status_nicename' => $invoice->get_status( true ),
+                    'date'          => $invoice->get_invoice_date(false),
+                    'amount'        => wpinv_round_amount($invoice->get_total()),
+                    'status_nicename' => $invoice->get_status(true),
                     'status'        => $invoice->get_status(),
-                    'tax'           => $invoice->get_tax() > 0 ? wpinv_round_amount( $invoice->get_tax() ) : '',
-                    'discount'      => $invoice->get_discount() > 0 ? wpinv_round_amount( $invoice->get_discount() ) : '',
+                    'tax'           => $invoice->get_tax() > 0 ? wpinv_round_amount($invoice->get_tax()) : '',
+                    'discount'      => $invoice->get_discount() > 0 ? wpinv_round_amount($invoice->get_discount()) : '',
                     'user_id'       => $invoice->get_user_id(),
                     'email'         => $invoice->get_email(),
                     'first_name'    => $invoice->get_first_name(),
@@ -464,7 +464,7 @@ 
                     'due_date'      => $invoice->needs_payment() ? $invoice->get_due_date() : '',
                 );
                 
-                $data[] = apply_filters( 'wpinv_export_invoice_row', $row, $invoice );
+                $data[] = apply_filters('wpinv_export_invoice_row', $row, $invoice);
             }
 
             return $data;
@@ -480,29 +480,29 @@ 
             'return'   => 'ids',
         );
         
-        if ( $this->status != 'any' ) {
+        if ($this->status != 'any') {
             $args['status'] = $this->status;
         }
 
-        if ( !empty( $this->from_date ) || !empty( $this->to_date ) ) {
+        if (!empty($this->from_date) || !empty($this->to_date)) {
             $args['date_query'] = array(
                 array(
-                    'after'     => date( 'Y-n-d 00:00:00', strtotime( $this->from_date ) ),
-                    'before'    => date( 'Y-n-d 23:59:59', strtotime( $this->to_date ) ),
+                    'after'     => date('Y-n-d 00:00:00', strtotime($this->from_date)),
+                    'before'    => date('Y-n-d 23:59:59', strtotime($this->to_date)),
                     'inclusive' => true
                 )
             );
         }
 
-        $invoices   = wpinv_get_invoices( $args );
-        $total      = !empty( $invoices ) ? count( $invoices ) : 0;
+        $invoices   = wpinv_get_invoices($args);
+        $total      = !empty($invoices) ? count($invoices) : 0;
         $status     = 100;
 
-        if ( $total > 0 ) {
-            $status = ( ( $this->per_page * $this->step ) / $total ) * 100;
+        if ($total > 0) {
+            $status = (($this->per_page * $this->step) / $total) * 100;
         }
 
-        if ( $status > 100 ) {
+        if ($status > 100) {
             $status = 100;
         }
 

templates/emails/wpinv-email-user_note.php

Spacing +8 added lines, -8 removed lines

@@ -1,18 +1,18 @@
 <?php
 // don't load directly
-if ( !defined('ABSPATH') )
+if (!defined('ABSPATH'))
     die('-1');
 
-do_action( 'wpinv_email_header', $email_heading, $invoice, $email_type, $sent_to_admin );
+do_action('wpinv_email_header', $email_heading, $invoice, $email_type, $sent_to_admin);
 
-do_action( 'wpinv_email_before_note_details', $invoice, $email_type, $sent_to_admin, $customer_note );
+do_action('wpinv_email_before_note_details', $invoice, $email_type, $sent_to_admin, $customer_note);
 
-do_action( 'wpinv_email_invoice_details', $invoice, $email_type, $sent_to_admin );
+do_action('wpinv_email_invoice_details', $invoice, $email_type, $sent_to_admin);
 
-do_action( 'wpinv_email_invoice_items', $invoice, $email_type, $sent_to_admin );
+do_action('wpinv_email_invoice_items', $invoice, $email_type, $sent_to_admin);
 
-do_action( 'wpinv_email_billing_details', $invoice, $email_type, $sent_to_admin );
+do_action('wpinv_email_billing_details', $invoice, $email_type, $sent_to_admin);
 
-do_action( 'wpinv_email_after_note_details', $invoice, $email_type, $sent_to_admin, $customer_note );
+do_action('wpinv_email_after_note_details', $invoice, $email_type, $sent_to_admin, $customer_note);
 
-do_action( 'wpinv_email_footer', $invoice, $email_type, $sent_to_admin );
\ No newline at end of file
+do_action('wpinv_email_footer', $invoice, $email_type, $sent_to_admin);
\ No newline at end of file

includes/class-wpinv-notes.php

Spacing +39 added lines, -39 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if ( ! defined( 'ABSPATH' ) ) {
+if (!defined('ABSPATH')) {
     exit; // Exit if accessed directly
 }
 
@@ -14,73 +14,73 @@ 
     }
     
     public function init() {
-        do_action( 'wpinv_class_notes_init', $this );
+        do_action('wpinv_class_notes_init', $this);
     }
     
     public function includes() {
-        do_action( 'wpinv_class_notes_includes', $this );
+        do_action('wpinv_class_notes_includes', $this);
     }
     
     public function actions() {
         // Secure inovice notes
-        add_action( 'pre_get_comments', array( $this, 'set_invoice_note_type' ), 11, 1 );
+        add_action('pre_get_comments', array($this, 'set_invoice_note_type'), 11, 1);
         
         // Count comments
-        add_filter( 'wp_count_comments', array( $this, 'wp_count_comments' ), 11, 2 );
+        add_filter('wp_count_comments', array($this, 'wp_count_comments'), 11, 2);
         
         // Delete comments count cache whenever there is a new comment or a comment status changes
-        add_action( 'wp_insert_comment', array( $this, 'delete_comments_count_cache' ) );
-        add_action( 'wp_set_comment_status', array( $this, 'delete_comments_count_cache' ) );
+        add_action('wp_insert_comment', array($this, 'delete_comments_count_cache'));
+        add_action('wp_set_comment_status', array($this, 'delete_comments_count_cache'));
         
-        do_action( 'wpinv_class_notes_actions', $this );
+        do_action('wpinv_class_notes_actions', $this);
     }
         
-    public function set_invoice_note_type( $query ) {
-        $post_ID        = !empty( $query->query_vars['post_ID'] ) ? $query->query_vars['post_ID'] : $query->query_vars['post_id'];
+    public function set_invoice_note_type($query) {
+        $post_ID = !empty($query->query_vars['post_ID']) ? $query->query_vars['post_ID'] : $query->query_vars['post_id'];
         
-        if ( $post_ID && in_array(get_post_type( $post_ID ), array($this->invoice_post_type, 'wpi_quote' )) ) {
+        if ($post_ID && in_array(get_post_type($post_ID), array($this->invoice_post_type, 'wpi_quote'))) {
             $query->query_vars['type__in']      = $this->comment_type;
             $query->query_vars['type__not_in']  = '';
         } else {        
-            if ( isset( $query->query_vars['type__in'] ) && $type_in = $query->query_vars['type__in'] ) {
-                if ( is_array( $type_in ) && in_array( $this->comment_type, $type_in ) ) {
-                    $key = array_search( $this->comment_type, $type_in );
-                    unset( $query->query_vars['type__in'][$key] );
-                } else if ( !is_array( $type_in ) && $type_in == $this->comment_type ) {
+            if (isset($query->query_vars['type__in']) && $type_in = $query->query_vars['type__in']) {
+                if (is_array($type_in) && in_array($this->comment_type, $type_in)) {
+                    $key = array_search($this->comment_type, $type_in);
+                    unset($query->query_vars['type__in'][$key]);
+                } else if (!is_array($type_in) && $type_in == $this->comment_type) {
                     $query->query_vars['type__in'] = '';
                 }
             }
             
-            if ( isset( $query->query_vars['type__not_in'] ) && $type_not_in = $query->query_vars['type__not_in'] ) {
-                if ( is_array( $type_not_in ) && !in_array( $this->comment_type, $type_not_in ) ) {
+            if (isset($query->query_vars['type__not_in']) && $type_not_in = $query->query_vars['type__not_in']) {
+                if (is_array($type_not_in) && !in_array($this->comment_type, $type_not_in)) {
                     $query->query_vars['type__not_in'][] = $this->comment_type;
-                } else if ( !is_array( $type_not_in ) && $type_not_in != $this->comment_type ) {
+                } else if (!is_array($type_not_in) && $type_not_in != $this->comment_type) {
                     $query->query_vars['type__not_in'] = (array)$query->query_vars['type__not_in'];
                     $query->query_vars['type__not_in'][] = $this->comment_type;
                 }
             } else {
-                $query->query_vars['type__not_in']  = $this->comment_type;
+                $query->query_vars['type__not_in'] = $this->comment_type;
             }
         }
         
         return $query;
     }
     
-    public function get_invoice_notes( $invoice_id = 0, $type = '' ) {
+    public function get_invoice_notes($invoice_id = 0, $type = '') {
         $args = array( 
             'post_id'   => $invoice_id,
             'orderby'   => 'comment_ID',
             'order'     => 'ASC',
         );
         
-        if ( $type == 'customer' ) {
+        if ($type == 'customer') {
             $args['meta_key']   = '_wpi_customer_note';
             $args['meta_value'] = 1;
         }
         
-        $args   = apply_filters( 'wpinv_invoice_notes_args', $args, $this, $invoice_id, $type );
+        $args = apply_filters('wpinv_invoice_notes_args', $args, $this, $invoice_id, $type);
         
-        return get_comments( $args );
+        return get_comments($args);
     }
     
     /**
@@ -91,7 +91,7 @@ 
      * @return void
      */
     public function delete_comments_count_cache() {
-        delete_transient( 'wpinv_count_comments' );
+        delete_transient('wpinv_count_comments');
     }
     
     /**
@@ -102,16 +102,16 @@ 
      * @param  int    $post_id Post ID.
      * @return object
      */
-    public function wp_count_comments( $stats, $post_id ) {
+    public function wp_count_comments($stats, $post_id) {
         global $wpdb;
 
-        if ( 0 === $post_id ) {
-            $stats = get_transient( 'wpinv_count_comments' );
+        if (0 === $post_id) {
+            $stats = get_transient('wpinv_count_comments');
 
-            if ( ! $stats ) {
+            if (!$stats) {
                 $stats = array();
 
-                $count = $wpdb->get_results( "SELECT comment_approved, COUNT(*) AS num_comments FROM {$wpdb->comments} WHERE comment_type NOT IN ('" . $this->comment_type . "') GROUP BY comment_approved", ARRAY_A );
+                $count = $wpdb->get_results("SELECT comment_approved, COUNT(*) AS num_comments FROM {$wpdb->comments} WHERE comment_type NOT IN ('" . $this->comment_type . "') GROUP BY comment_approved", ARRAY_A);
 
                 $total = 0;
                 $approved = array(
@@ -122,26 +122,26 @@ 
                     'post-trashed' => 'post-trashed',
                 );
 
-                foreach ( (array) $count as $row ) {
+                foreach ((array)$count as $row) {
                     // Do not count post-trashed toward totals.
-                    if ( 'post-trashed' !== $row['comment_approved'] && 'trash' !== $row['comment_approved'] ) {
+                    if ('post-trashed' !== $row['comment_approved'] && 'trash' !== $row['comment_approved']) {
                         $total += $row['num_comments'];
                     }
-                    if ( isset( $approved[ $row['comment_approved'] ] ) ) {
-                        $stats[ $approved[ $row['comment_approved'] ] ] = $row['num_comments'];
+                    if (isset($approved[$row['comment_approved']])) {
+                        $stats[$approved[$row['comment_approved']]] = $row['num_comments'];
                     }
                 }
 
                 $stats['total_comments'] = $total;
                 $stats['all'] = $total;
-                foreach ( $approved as $key ) {
-                    if ( empty( $stats[ $key ] ) ) {
-                        $stats[ $key ] = 0;
+                foreach ($approved as $key) {
+                    if (empty($stats[$key])) {
+                        $stats[$key] = 0;
                     }
                 }
 
-                $stats = (object) $stats;
-                set_transient( 'wpinv_count_comments', $stats );
+                $stats = (object)$stats;
+                set_transient('wpinv_count_comments', $stats);
             }
         }
 

templates/emails/wpinv-email-new_invoice.php

Spacing +7 added lines, -7 removed lines

@@ -1,18 +1,18 @@
 <?php
 // don't load directly
-if ( !defined('ABSPATH') )
+if (!defined('ABSPATH'))
     die('-1');
 
-do_action( 'wpinv_email_header', $email_heading, $invoice, $email_type, $sent_to_admin );
+do_action('wpinv_email_header', $email_heading, $invoice, $email_type, $sent_to_admin);
 ?>
 
-<p><?php printf( __( 'You have received payment from %s. The invoice is as follows:', 'invoicing' ), $invoice->get_user_full_name() ); ?></p>
+<p><?php printf(__('You have received payment from %s. The invoice is as follows:', 'invoicing'), $invoice->get_user_full_name()); ?></p>
 
 <?php
-do_action( 'wpinv_email_invoice_details', $invoice, $email_type, $sent_to_admin );
+do_action('wpinv_email_invoice_details', $invoice, $email_type, $sent_to_admin);
 
-do_action( 'wpinv_email_invoice_items', $invoice, $email_type, $sent_to_admin );
+do_action('wpinv_email_invoice_items', $invoice, $email_type, $sent_to_admin);
 
-do_action( 'wpinv_email_billing_details', $invoice, $email_type, $sent_to_admin );
+do_action('wpinv_email_billing_details', $invoice, $email_type, $sent_to_admin);
 
-do_action( 'wpinv_email_footer', $invoice, $email_type, $sent_to_admin );
\ No newline at end of file
+do_action('wpinv_email_footer', $invoice, $email_type, $sent_to_admin);
\ No newline at end of file

includes/admin/admin-meta-boxes.php

Spacing +126 added lines, -126 removed lines

@@ -1,65 +1,65 @@ 
 <?php
 // MUST have WordPress.
-if ( !defined( 'WPINC' ) ) {
-    exit( 'Do NOT access this file directly: ' . basename( __FILE__ ) );
+if (!defined('WPINC')) {
+    exit('Do NOT access this file directly: ' . basename(__FILE__));
 }
 
-function wpinv_add_meta_boxes( $post_type, $post ) {
+function wpinv_add_meta_boxes($post_type, $post) {
     global $wpi_mb_invoice;
-    if ( $post_type == 'wpi_invoice' && !empty( $post->ID ) ) {
-        $wpi_mb_invoice = wpinv_get_invoice( $post->ID );
+    if ($post_type == 'wpi_invoice' && !empty($post->ID)) {
+        $wpi_mb_invoice = wpinv_get_invoice($post->ID);
     }
     
-    if ( !empty( $wpi_mb_invoice ) && !$wpi_mb_invoice->has_status( array( 'draft', 'auto-draft' ) ) ) {
-        add_meta_box( 'wpinv-mb-resend-invoice', __( 'Resend Invoice', 'invoicing' ), 'WPInv_Meta_Box_Details::resend_invoice', 'wpi_invoice', 'side', 'high' );
+    if (!empty($wpi_mb_invoice) && !$wpi_mb_invoice->has_status(array('draft', 'auto-draft'))) {
+        add_meta_box('wpinv-mb-resend-invoice', __('Resend Invoice', 'invoicing'), 'WPInv_Meta_Box_Details::resend_invoice', 'wpi_invoice', 'side', 'high');
     }
     
-    if ( !empty( $wpi_mb_invoice ) && $wpi_mb_invoice->is_recurring() && $wpi_mb_invoice->is_parent() ) {
-        add_meta_box( 'wpinv-mb-subscriptions', __( 'Subscriptions', 'invoicing' ), 'WPInv_Meta_Box_Details::subscriptions', 'wpi_invoice', 'side', 'high' );
+    if (!empty($wpi_mb_invoice) && $wpi_mb_invoice->is_recurring() && $wpi_mb_invoice->is_parent()) {
+        add_meta_box('wpinv-mb-subscriptions', __('Subscriptions', 'invoicing'), 'WPInv_Meta_Box_Details::subscriptions', 'wpi_invoice', 'side', 'high');
     }
     
-    if ( wpinv_is_subscription_payment( $wpi_mb_invoice ) ) {
-        add_meta_box( 'wpinv-mb-renewals', __( 'Renewal Payment', 'invoicing' ), 'WPInv_Meta_Box_Details::renewals', 'wpi_invoice', 'side', 'high' );
+    if (wpinv_is_subscription_payment($wpi_mb_invoice)) {
+        add_meta_box('wpinv-mb-renewals', __('Renewal Payment', 'invoicing'), 'WPInv_Meta_Box_Details::renewals', 'wpi_invoice', 'side', 'high');
     }
     
-    add_meta_box( 'wpinv-details', __( 'Invoice Details', 'invoicing' ), 'WPInv_Meta_Box_Details::output', 'wpi_invoice', 'side', 'default' );
-    add_meta_box( 'wpinv-payment-meta', __( 'Payment Meta', 'invoicing' ), 'WPInv_Meta_Box_Details::payment_meta', 'wpi_invoice', 'side', 'default' );
+    add_meta_box('wpinv-details', __('Invoice Details', 'invoicing'), 'WPInv_Meta_Box_Details::output', 'wpi_invoice', 'side', 'default');
+    add_meta_box('wpinv-payment-meta', __('Payment Meta', 'invoicing'), 'WPInv_Meta_Box_Details::payment_meta', 'wpi_invoice', 'side', 'default');
    
-    add_meta_box( 'wpinv-address', __( 'Billing Details', 'invoicing' ), 'WPInv_Meta_Box_Billing_Details::output', 'wpi_invoice', 'normal', 'high' );
-    add_meta_box( 'wpinv-items', __( 'Invoice Items', 'invoicing' ), 'WPInv_Meta_Box_Items::output', 'wpi_invoice', 'normal', 'high' );
-    add_meta_box( 'wpinv-notes', __( 'Invoice Notes', 'invoicing' ), 'WPInv_Meta_Box_Notes::output', 'wpi_invoice', 'normal', 'high' );
+    add_meta_box('wpinv-address', __('Billing Details', 'invoicing'), 'WPInv_Meta_Box_Billing_Details::output', 'wpi_invoice', 'normal', 'high');
+    add_meta_box('wpinv-items', __('Invoice Items', 'invoicing'), 'WPInv_Meta_Box_Items::output', 'wpi_invoice', 'normal', 'high');
+    add_meta_box('wpinv-notes', __('Invoice Notes', 'invoicing'), 'WPInv_Meta_Box_Notes::output', 'wpi_invoice', 'normal', 'high');
 }
-add_action( 'add_meta_boxes', 'wpinv_add_meta_boxes', 30, 2 );
+add_action('add_meta_boxes', 'wpinv_add_meta_boxes', 30, 2);
 
-function wpinv_save_meta_boxes( $post_id, $post, $update = false ) {
-    remove_action( 'save_post', __FUNCTION__ );
+function wpinv_save_meta_boxes($post_id, $post, $update = false) {
+    remove_action('save_post', __FUNCTION__);
     
     // $post_id and $post are required
-    if ( empty( $post_id ) || empty( $post ) ) {
+    if (empty($post_id) || empty($post)) {
         return;
     }
         
-    if ( !current_user_can( 'edit_post', $post_id ) || empty( $post->post_type ) ) {
+    if (!current_user_can('edit_post', $post_id) || empty($post->post_type)) {
         return;
     }
     
     // Dont' save meta boxes for revisions or autosaves
-    if ( defined( 'DOING_AUTOSAVE' ) || is_int( wp_is_post_revision( $post ) ) || is_int( wp_is_post_autosave( $post ) ) ) {
+    if (defined('DOING_AUTOSAVE') || is_int(wp_is_post_revision($post)) || is_int(wp_is_post_autosave($post))) {
         return;
     }
         
-    if ( $post->post_type == 'wpi_invoice' or $post->post_type == 'wpi_quote' ) {
-        if ( ( defined( 'DOING_AJAX') && DOING_AJAX ) || isset( $_REQUEST['bulk_edit'] ) ) {
+    if ($post->post_type == 'wpi_invoice' or $post->post_type == 'wpi_quote') {
+        if ((defined('DOING_AJAX') && DOING_AJAX) || isset($_REQUEST['bulk_edit'])) {
             return;
         }
     
-        if ( isset( $_POST['wpinv_save_invoice'] ) && wp_verify_nonce( $_POST['wpinv_save_invoice'], 'wpinv_save_invoice' ) ) {
-            WPInv_Meta_Box_Items::save( $post_id, $_POST, $post );
+        if (isset($_POST['wpinv_save_invoice']) && wp_verify_nonce($_POST['wpinv_save_invoice'], 'wpinv_save_invoice')) {
+            WPInv_Meta_Box_Items::save($post_id, $_POST, $post);
         }
-    } else if ( $post->post_type == 'wpi_item' ) {
+    } else if ($post->post_type == 'wpi_item') {
         // verify nonce
-        if ( isset( $_POST['wpinv_vat_meta_box_nonce'] ) && wp_verify_nonce( $_POST['wpinv_vat_meta_box_nonce'], 'wpinv_item_meta_box_save' ) ) {
-            $fields                                 = array();
+        if (isset($_POST['wpinv_vat_meta_box_nonce']) && wp_verify_nonce($_POST['wpinv_vat_meta_box_nonce'], 'wpinv_item_meta_box_save')) {
+            $fields = array();
             $fields['_wpinv_price']              = 'wpinv_item_price';
             $fields['_wpinv_vat_class']          = 'wpinv_vat_class';
             $fields['_wpinv_vat_rule']           = 'wpinv_vat_rules';
@@ -72,256 +72,256 @@ 
             $fields['_wpinv_trial_period']       = 'wpinv_trial_period';
             $fields['_wpinv_trial_interval']     = 'wpinv_trial_interval';
             
-            if ( !isset( $_POST['wpinv_is_recurring'] ) ) {
+            if (!isset($_POST['wpinv_is_recurring'])) {
                 $_POST['wpinv_is_recurring'] = 0;
             }
             
-            if ( !isset( $_POST['wpinv_free_trial'] ) || empty( $_POST['wpinv_is_recurring'] ) ) {
+            if (!isset($_POST['wpinv_free_trial']) || empty($_POST['wpinv_is_recurring'])) {
                 $_POST['wpinv_free_trial'] = 0;
             }
             
-            foreach ( $fields as $field => $name ) {
-                if ( isset( $_POST[ $name ] ) ) {
-                    if ( $field == '_wpinv_price' ) {
-                        if ( get_post_meta( $post_id, '_wpinv_type', true ) === 'package' ) {
-                            $value = wpinv_sanitize_amount( get_post_meta( $post_id, '_wpinv_price', true ) ); // Don't allow edit GD package item price.
+            foreach ($fields as $field => $name) {
+                if (isset($_POST[$name])) {
+                    if ($field == '_wpinv_price') {
+                        if (get_post_meta($post_id, '_wpinv_type', true) === 'package') {
+                            $value = wpinv_sanitize_amount(get_post_meta($post_id, '_wpinv_price', true)); // Don't allow edit GD package item price.
                         } else {
-                            $value = wpinv_sanitize_amount( $_POST[ $name ] );
+                            $value = wpinv_sanitize_amount($_POST[$name]);
                         }
                     } else {
-                        $value = is_string( $_POST[ $name ] ) ? sanitize_text_field( $_POST[ $name ] ) : $_POST[ $name ];
+                        $value = is_string($_POST[$name]) ? sanitize_text_field($_POST[$name]) : $_POST[$name];
                     }
                     
-                    $value = apply_filters( 'wpinv_item_metabox_save_' . $field, $value, $name );
-                    update_post_meta( $post_id, $field, $value );
+                    $value = apply_filters('wpinv_item_metabox_save_' . $field, $value, $name);
+                    update_post_meta($post_id, $field, $value);
                 }
             }
             
-            if ( !get_post_meta( $post_id, '_wpinv_custom_id', true ) ) {
-                update_post_meta( $post_id, '_wpinv_custom_id', $post_id );
+            if (!get_post_meta($post_id, '_wpinv_custom_id', true)) {
+                update_post_meta($post_id, '_wpinv_custom_id', $post_id);
             }
         }
     }
 }
-add_action( 'save_post', 'wpinv_save_meta_boxes', 10, 3 );
+add_action('save_post', 'wpinv_save_meta_boxes', 10, 3);
 
 function wpinv_register_item_meta_boxes() {    
     global $wpinv_euvat;
     
-    add_meta_box( 'wpinv_field_prices', __( 'Item Price', 'invoicing' ), 'WPInv_Meta_Box_Items::prices', 'wpi_item', 'normal', 'high' );
+    add_meta_box('wpinv_field_prices', __('Item Price', 'invoicing'), 'WPInv_Meta_Box_Items::prices', 'wpi_item', 'normal', 'high');
 
-    if ( $wpinv_euvat->allow_vat_rules() ) {
-        add_meta_box( 'wpinv_field_vat_rules', __( 'VAT rules type to use', 'invoicing' ), 'WPInv_Meta_Box_Items::vat_rules', 'wpi_item', 'normal', 'high' );
+    if ($wpinv_euvat->allow_vat_rules()) {
+        add_meta_box('wpinv_field_vat_rules', __('VAT rules type to use', 'invoicing'), 'WPInv_Meta_Box_Items::vat_rules', 'wpi_item', 'normal', 'high');
     }
     
-    if ( $wpinv_euvat->allow_vat_classes() ) {
-        add_meta_box( 'wpinv_field_vat_classes', __( 'VAT rates class to use', 'invoicing' ), 'WPInv_Meta_Box_Items::vat_classes', 'wpi_item', 'normal', 'high' );
+    if ($wpinv_euvat->allow_vat_classes()) {
+        add_meta_box('wpinv_field_vat_classes', __('VAT rates class to use', 'invoicing'), 'WPInv_Meta_Box_Items::vat_classes', 'wpi_item', 'normal', 'high');
     }
     
-    add_meta_box( 'wpinv_field_item_info', __( 'Item info', 'invoicing' ), 'WPInv_Meta_Box_Items::item_info', 'wpi_item', 'side', 'core' );
-    add_meta_box( 'wpinv_field_meta_values', __( 'Item Meta Values', 'invoicing' ), 'WPInv_Meta_Box_Items::meta_values', 'wpi_item', 'side', 'core' );
+    add_meta_box('wpinv_field_item_info', __('Item info', 'invoicing'), 'WPInv_Meta_Box_Items::item_info', 'wpi_item', 'side', 'core');
+    add_meta_box('wpinv_field_meta_values', __('Item Meta Values', 'invoicing'), 'WPInv_Meta_Box_Items::meta_values', 'wpi_item', 'side', 'core');
 }
 
 function wpinv_register_discount_meta_boxes() {
-    add_meta_box( 'wpinv_discount_fields', __( 'Discount Details', 'invoicing' ), 'wpinv_discount_metabox_details', 'wpi_discount', 'normal', 'high' );
+    add_meta_box('wpinv_discount_fields', __('Discount Details', 'invoicing'), 'wpinv_discount_metabox_details', 'wpi_discount', 'normal', 'high');
 }
 
-function wpinv_discount_metabox_details( $post ) {
+function wpinv_discount_metabox_details($post) {
     $discount_id    = $post->ID;
-    $discount       = wpinv_get_discount( $discount_id );
+    $discount       = wpinv_get_discount($discount_id);
     
-    $type           = wpinv_get_discount_type( $discount_id );
-    $item_reqs      = wpinv_get_discount_item_reqs( $discount_id );
-    $excluded_items = wpinv_get_discount_excluded_items( $discount_id );
-    $min_total      = wpinv_get_discount_min_total( $discount_id );
-    $max_total      = wpinv_get_discount_max_total( $discount_id );
-    $max_uses       = wpinv_get_discount_max_uses( $discount_id );
-    $single_use     = wpinv_discount_is_single_use( $discount_id );
-    $recurring      = (bool)wpinv_discount_is_recurring( $discount_id );
+    $type           = wpinv_get_discount_type($discount_id);
+    $item_reqs      = wpinv_get_discount_item_reqs($discount_id);
+    $excluded_items = wpinv_get_discount_excluded_items($discount_id);
+    $min_total      = wpinv_get_discount_min_total($discount_id);
+    $max_total      = wpinv_get_discount_max_total($discount_id);
+    $max_uses       = wpinv_get_discount_max_uses($discount_id);
+    $single_use     = wpinv_discount_is_single_use($discount_id);
+    $recurring      = (bool)wpinv_discount_is_recurring($discount_id);
     
     $min_total      = $min_total > 0 ? $min_total : '';
     $max_total      = $max_total > 0 ? $max_total : '';
     $max_uses       = $max_uses > 0 ? $max_uses : '';
 ?>
-<?php do_action( 'wpinv_discount_form_top', $post ); ?>
-<?php wp_nonce_field( 'wpinv_discount_metabox_nonce', 'wpinv_discount_metabox_nonce' ); ;?>
+<?php do_action('wpinv_discount_form_top', $post); ?>
+<?php wp_nonce_field('wpinv_discount_metabox_nonce', 'wpinv_discount_metabox_nonce'); ;?>
 <table class="form-table wpi-form-table">
     <tbody>
-        <?php do_action( 'wpinv_discount_form_first', $post ); ?>
-        <?php do_action( 'wpinv_discount_form_before_code', $post ); ?>
+        <?php do_action('wpinv_discount_form_first', $post); ?>
+        <?php do_action('wpinv_discount_form_before_code', $post); ?>
         <tr>
             <th valign="top" scope="row">
-                <label for="wpinv_discount_code"><?php _e( 'Discount Code', 'invoicing' ); ?></label>
+                <label for="wpinv_discount_code"><?php _e('Discount Code', 'invoicing'); ?></label>
             </th>
             <td>
-                <input type="text" name="code" id="wpinv_discount_code" class="medium-text" value="<?php echo esc_attr( wpinv_get_discount_code( $discount_id ) ); ?>" required>
-                <p class="description"><?php _e( 'Enter a code for this discount, such as 10OFF', 'invoicing' ); ?></p>
+                <input type="text" name="code" id="wpinv_discount_code" class="medium-text" value="<?php echo esc_attr(wpinv_get_discount_code($discount_id)); ?>" required>
+                <p class="description"><?php _e('Enter a code for this discount, such as 10OFF', 'invoicing'); ?></p>
             </td>
         </tr>
-        <?php do_action( 'wpinv_discount_form_before_type', $post ); ?>
+        <?php do_action('wpinv_discount_form_before_type', $post); ?>
         <tr>
             <th valign="top" scope="row">
-                <label for="wpinv_discount_type"><?php _e( 'Discount Type', 'invoicing' ); ?></label>
+                <label for="wpinv_discount_type"><?php _e('Discount Type', 'invoicing'); ?></label>
             </th>
             <td>
                 <select id="wpinv_discount_type" name="type" class="medium-text">
-                    <?php foreach ( wpinv_get_discount_types() as $value => $label ) { ?>
-                    <option value="<?php echo $value ;?>" <?php selected( $type, $value ); ?>><?php echo $label; ?></option>
+                    <?php foreach (wpinv_get_discount_types() as $value => $label) { ?>
+                    <option value="<?php echo $value; ?>" <?php selected($type, $value); ?>><?php echo $label; ?></option>
                     <?php } ?>
                 </select>
-                <p class="description"><?php _e( 'The kind of discount to apply for this discount.', 'invoicing' ); ?></p>
+                <p class="description"><?php _e('The kind of discount to apply for this discount.', 'invoicing'); ?></p>
             </td>
         </tr>
-        <?php do_action( 'wpinv_discount_form_before_amount', $post ); ?>
+        <?php do_action('wpinv_discount_form_before_amount', $post); ?>
         <tr>
             <th valign="top" scope="row">
-                <label for="wpinv_discount_amount"><?php _e( 'Amount', 'invoicing' ); ?></label>
+                <label for="wpinv_discount_amount"><?php _e('Amount', 'invoicing'); ?></label>
             </th>
             <td>
-                <input type="text" name="amount" id="wpinv_discount_amount" class="wpi-field-price wpi-price" value="<?php echo esc_attr( wpinv_get_discount_amount( $discount_id ) ); ?>" required> <font class="wpi-discount-p">%</font><font class="wpi-discount-f" style="display:none;"><?php echo wpinv_currency_symbol() ;?></font>
-                <p style="display:none;" class="description"><?php _e( 'Enter the discount amount in USD', 'invoicing' ); ?></p>
-                <p class="description"><?php _e( 'Enter the discount value. Ex: 10', 'invoicing' ); ?></p>
+                <input type="text" name="amount" id="wpinv_discount_amount" class="wpi-field-price wpi-price" value="<?php echo esc_attr(wpinv_get_discount_amount($discount_id)); ?>" required> <font class="wpi-discount-p">%</font><font class="wpi-discount-f" style="display:none;"><?php echo wpinv_currency_symbol(); ?></font>
+                <p style="display:none;" class="description"><?php _e('Enter the discount amount in USD', 'invoicing'); ?></p>
+                <p class="description"><?php _e('Enter the discount value. Ex: 10', 'invoicing'); ?></p>
             </td>
         </tr>
-        <?php do_action( 'wpinv_discount_form_before_items', $post ); ?>
+        <?php do_action('wpinv_discount_form_before_items', $post); ?>
         <tr>
             <th valign="top" scope="row">
-                <label for="wpinv_discount_items"><?php _e( 'Items', 'invoicing' ); ?></label>
+                <label for="wpinv_discount_items"><?php _e('Items', 'invoicing'); ?></label>
             </th>
             <td>
-                <p><?php echo wpinv_item_dropdown( array(
+                <p><?php echo wpinv_item_dropdown(array(
                         'name'              => 'items[]',
                         'id'                => 'items',
                         'selected'          => $item_reqs,
                         'multiple'          => true,
                         'chosen'            => true,
                         'class'             => 'medium-text',
-                        'placeholder'       => __( 'Select one or more Items', 'invoicing' ),
+                        'placeholder'       => __('Select one or more Items', 'invoicing'),
                         'show_recurring'    => true,
-                    ) ); ?>
+                    )); ?>
                 </p>
-                <p class="description"><?php _e( 'Items which need to be in the cart to use this discount or, for "Item Discounts", which items are discounted. If left blank, this discount can be used on any item.', 'invoicing' ); ?></p>
+                <p class="description"><?php _e('Items which need to be in the cart to use this discount or, for "Item Discounts", which items are discounted. If left blank, this discount can be used on any item.', 'invoicing'); ?></p>
             </td>
         </tr>
-        <?php do_action( 'wpinv_discount_form_before_excluded_items', $post ); ?>
+        <?php do_action('wpinv_discount_form_before_excluded_items', $post); ?>
         <tr>
             <th valign="top" scope="row">
-                <label for="wpinv_discount_excluded_items"><?php _e( 'Excluded Items', 'invoicing' ); ?></label>
+                <label for="wpinv_discount_excluded_items"><?php _e('Excluded Items', 'invoicing'); ?></label>
             </th>
             <td>
-                <p><?php echo wpinv_item_dropdown( array(
+                <p><?php echo wpinv_item_dropdown(array(
                         'name'              => 'excluded_items[]',
                         'id'                => 'excluded_items',
                         'selected'          => $excluded_items,
                         'multiple'          => true,
                         'chosen'            => true,
                         'class'             => 'medium-text',
-                        'placeholder'       => __( 'Select one or more Items', 'invoicing' ),
+                        'placeholder'       => __('Select one or more Items', 'invoicing'),
                         'show_recurring'    => true,
-                    ) ); ?>
+                    )); ?>
                 </p>
-                <p class="description"><?php _e( 'Items which are NOT allowed to use this discount.', 'invoicing' ); ?></p>
+                <p class="description"><?php _e('Items which are NOT allowed to use this discount.', 'invoicing'); ?></p>
             </td>
         </tr>
-        <?php do_action( 'wpinv_discount_form_before_start', $post ); ?>
+        <?php do_action('wpinv_discount_form_before_start', $post); ?>
         <tr>
             <th valign="top" scope="row">
-                <label for="wpinv_discount_start"><?php _e( 'Start Date', 'invoicing' ); ?></label>
+                <label for="wpinv_discount_start"><?php _e('Start Date', 'invoicing'); ?></label>
             </th>
             <td>
-                <input type="text" class="medium-text wpiDatepicker" id="wpinv_discount_start" data-dateFormat="yy-mm-dd" name="start" value="<?php echo esc_attr( wpinv_get_discount_start_date( $discount_id ) ); ?>">
-                <p class="description"><?php _e( 'Enter the start date for this discount code in the format of yyyy-mm-dd. For no start date, leave blank. If entered, the discount can only be used after or on this date.', 'invoicing' ); ?></p>
+                <input type="text" class="medium-text wpiDatepicker" id="wpinv_discount_start" data-dateFormat="yy-mm-dd" name="start" value="<?php echo esc_attr(wpinv_get_discount_start_date($discount_id)); ?>">
+                <p class="description"><?php _e('Enter the start date for this discount code in the format of yyyy-mm-dd. For no start date, leave blank. If entered, the discount can only be used after or on this date.', 'invoicing'); ?></p>
             </td>
         </tr>
-        <?php do_action( 'wpinv_discount_form_before_expiration', $post ); ?>
+        <?php do_action('wpinv_discount_form_before_expiration', $post); ?>
         <tr>
             <th valign="top" scope="row">
-                <label for="wpinv_discount_expiration"><?php _e( 'Expiration Date', 'invoicing' ); ?></label>
+                <label for="wpinv_discount_expiration"><?php _e('Expiration Date', 'invoicing'); ?></label>
             </th>
             <td>
-                <input type="text" class="medium-text wpiDatepicker" id="wpinv_discount_expiration" data-dateFormat="yy-mm-dd" name="expiration" value="<?php echo esc_attr( wpinv_get_discount_expiration( $discount_id ) ); ?>">
-                <p class="description"><?php _e( 'Enter the expiration date for this discount code in the format of yyyy-mm-dd. Leave blank for no expiration.', 'invoicing' ); ?></p>
+                <input type="text" class="medium-text wpiDatepicker" id="wpinv_discount_expiration" data-dateFormat="yy-mm-dd" name="expiration" value="<?php echo esc_attr(wpinv_get_discount_expiration($discount_id)); ?>">
+                <p class="description"><?php _e('Enter the expiration date for this discount code in the format of yyyy-mm-dd. Leave blank for no expiration.', 'invoicing'); ?></p>
             </td>
         </tr>
-        <?php do_action( 'wpinv_discount_form_before_min_total', $post ); ?>
+        <?php do_action('wpinv_discount_form_before_min_total', $post); ?>
         <tr>
             <th valign="top" scope="row">
-                <label for="wpinv_discount_min_total"><?php _e( 'Minimum Amount', 'invoicing' ); ?></label>
+                <label for="wpinv_discount_min_total"><?php _e('Minimum Amount', 'invoicing'); ?></label>
             </th>
             <td>
                 <input type="text" name="min_total" id="wpinv_discount_min_total" class="wpi-field-price wpi-price" value="<?php echo $min_total; ?>">
-                <p class="description"><?php _e( 'This allows you to set the minimum amount (subtotal, including taxes) allowed when using the discount.', 'invoicing' ); ?></p>
+                <p class="description"><?php _e('This allows you to set the minimum amount (subtotal, including taxes) allowed when using the discount.', 'invoicing'); ?></p>
             </td>
         </tr>
-        <?php do_action( 'wpinv_discount_form_before_max_total', $post ); ?>
+        <?php do_action('wpinv_discount_form_before_max_total', $post); ?>
         <tr>
             <th valign="top" scope="row">
-                <label for="wpinv_discount_max_total"><?php _e( 'Maximum Amount', 'invoicing' ); ?></label>
+                <label for="wpinv_discount_max_total"><?php _e('Maximum Amount', 'invoicing'); ?></label>
             </th>
             <td>
                 <input type="text" name="max_total" id="wpinv_discount_max_total" class="wpi-field-price wpi-price" value="<?php echo $max_total; ?>">
-                <p class="description"><?php _e( 'This allows you to set the maximum amount (subtotal, including taxes) allowed when using the discount.', 'invoicing' ); ?></p>
+                <p class="description"><?php _e('This allows you to set the maximum amount (subtotal, including taxes) allowed when using the discount.', 'invoicing'); ?></p>
             </td>
         </tr>
-        <?php do_action( 'wpinv_discount_form_before_recurring', $post ); ?>
+        <?php do_action('wpinv_discount_form_before_recurring', $post); ?>
         <tr>
             <th valign="top" scope="row">
-                <label for="wpinv_discount_recurring"><?php _e( 'For recurring apply to', 'invoicing' ); ?></label>
+                <label for="wpinv_discount_recurring"><?php _e('For recurring apply to', 'invoicing'); ?></label>
             </th>
             <td>
                 <select id="wpinv_discount_recurring" name="recurring" class="medium-text">
-                    <option value="0" <?php selected( false, $recurring ); ?>><?php _e( 'All payments', 'invoicing' ); ?></option>
-                    <option value="1" <?php selected( true, $recurring ); ?>><?php _e( 'First payment only', 'invoicing' ); ?></option>
+                    <option value="0" <?php selected(false, $recurring); ?>><?php _e('All payments', 'invoicing'); ?></option>
+                    <option value="1" <?php selected(true, $recurring); ?>><?php _e('First payment only', 'invoicing'); ?></option>
                 </select>
-                <p class="description"><?php _e( '<b>All payments:</b> Apply this discount to all recurring payments of the recurring invoice. <br><b>First payment only:</b> Apply this discount to only first payment of the recurring invoice.', 'invoicing' ); ?></p>
+                <p class="description"><?php _e('<b>All payments:</b> Apply this discount to all recurring payments of the recurring invoice. <br><b>First payment only:</b> Apply this discount to only first payment of the recurring invoice.', 'invoicing'); ?></p>
             </td>
         </tr>
-        <?php do_action( 'wpinv_discount_form_before_max_uses', $post ); ?>
+        <?php do_action('wpinv_discount_form_before_max_uses', $post); ?>
         <tr>
             <th valign="top" scope="row">
-                <label for="wpinv_discount_max_uses"><?php _e( 'Max Uses', 'invoicing' ); ?></label>
+                <label for="wpinv_discount_max_uses"><?php _e('Max Uses', 'invoicing'); ?></label>
             </th>
             <td>
                 <input type="number" min="0" step="1" id="wpinv_discount_max_uses" name="max_uses" class="medium-text" value="<?php echo $max_uses; ?>">
-                <p class="description"><?php _e( 'The maximum number of times this discount can be used. Leave blank for unlimited.', 'invoicing' ); ?></p>
+                <p class="description"><?php _e('The maximum number of times this discount can be used. Leave blank for unlimited.', 'invoicing'); ?></p>
             </td>
         </tr>
-        <?php do_action( 'wpinv_discount_form_before_single_use', $post ); ?>
+        <?php do_action('wpinv_discount_form_before_single_use', $post); ?>
         <tr>
             <th valign="top" scope="row">
-                <label for="wpinv_discount_single_use"><?php _e( 'Use Once Per User', 'invoicing' ); ?></label>
+                <label for="wpinv_discount_single_use"><?php _e('Use Once Per User', 'invoicing'); ?></label>
             </th>
             <td>
-                <input type="checkbox" value="1" name="single_use" id="wpinv_discount_single_use" <?php checked( true, $single_use ); ?>>
-                <span class="description"><?php _e( 'Limit this discount to a single use per user?', 'invoicing' ); ?></span>
+                <input type="checkbox" value="1" name="single_use" id="wpinv_discount_single_use" <?php checked(true, $single_use); ?>>
+                <span class="description"><?php _e('Limit this discount to a single use per user?', 'invoicing'); ?></span>
             </td>
         </tr>
-        <?php do_action( 'wpinv_discount_form_last', $post ); ?>
+        <?php do_action('wpinv_discount_form_last', $post); ?>
     </tbody>
 </table>
-<?php do_action( 'wpinv_discount_form_bottom', $post ); ?>
+<?php do_action('wpinv_discount_form_bottom', $post); ?>
     <?php
 }
 
-function wpinv_discount_metabox_save( $post_id, $post, $update = false ) {
-    $post_type = !empty( $post ) ? $post->post_type : '';
+function wpinv_discount_metabox_save($post_id, $post, $update = false) {
+    $post_type = !empty($post) ? $post->post_type : '';
     
-    if ( $post_type != 'wpi_discount' ) {
+    if ($post_type != 'wpi_discount') {
         return;
     }
     
-    if ( !isset( $_POST['wpinv_discount_metabox_nonce'] ) || ( isset( $_POST['wpinv_discount_metabox_nonce'] ) && !wp_verify_nonce( $_POST['wpinv_discount_metabox_nonce'], 'wpinv_discount_metabox_nonce' ) ) ) {
+    if (!isset($_POST['wpinv_discount_metabox_nonce']) || (isset($_POST['wpinv_discount_metabox_nonce']) && !wp_verify_nonce($_POST['wpinv_discount_metabox_nonce'], 'wpinv_discount_metabox_nonce'))) {
         return;
     }
     
-    if ( ( defined( 'DOING_AUTOSAVE' ) && DOING_AUTOSAVE ) || ( defined( 'DOING_AJAX') && DOING_AJAX ) || isset( $_REQUEST['bulk_edit'] ) ) {
+    if ((defined('DOING_AUTOSAVE') && DOING_AUTOSAVE) || (defined('DOING_AJAX') && DOING_AJAX) || isset($_REQUEST['bulk_edit'])) {
         return;
     }
     
-    if ( !current_user_can( 'manage_options', $post_id ) ) {
+    if (!current_user_can('manage_options', $post_id)) {
         return;
     }
     
-    return wpinv_store_discount( $post_id, $_POST, $post, $update );
+    return wpinv_store_discount($post_id, $_POST, $post, $update);
 }
-add_action( 'save_post', 'wpinv_discount_metabox_save', 10, 3 );
\ No newline at end of file
+add_action('save_post', 'wpinv_discount_metabox_save', 10, 3);
\ No newline at end of file