AyeCode / invoicing

includes/admin/subscriptions.php

Indentation +415 added lines, -415 removed lines

@@ -14,7 +14,7 @@ 
  */
 function wpinv_subscriptions_page() {
 
-	?>
+    ?>
 
 	<div class="wrap">
 		<h1><?php echo esc_html( get_admin_page_title() ); ?></h1>
@@ -22,28 +22,28 @@ 
 
 			<?php
 
-				// Verify user permissions.
-				if ( ! wpinv_current_user_can_manage_invoicing() ) {
+                // Verify user permissions.
+                if ( ! wpinv_current_user_can_manage_invoicing() ) {
 
-				aui()->alert(
+                aui()->alert(
                     array(
-						'type'    => 'danger',
-						'content' => __( 'You are not permitted to view this page.', 'invoicing' ),
-					),
-					true
+                        'type'    => 'danger',
+                        'content' => __( 'You are not permitted to view this page.', 'invoicing' ),
+                    ),
+                    true
                 );
 
-				} elseif ( ! empty( $_GET['id'] ) && is_numeric( $_GET['id'] ) ) {
+                } elseif ( ! empty( $_GET['id'] ) && is_numeric( $_GET['id'] ) ) {
 
-				// Display a single subscription.
-				wpinv_recurring_subscription_details();
-				} else {
+                // Display a single subscription.
+                wpinv_recurring_subscription_details();
+                } else {
 
-				// Display a list of available subscriptions.
-				getpaid_print_subscriptions_list();
-				}
+                // Display a list of available subscriptions.
+                getpaid_print_subscriptions_list();
+                }
 
-			?>
+            ?>
 
 		</div>
 	</div>
@@ -60,10 +60,10 @@ 
  */
 function getpaid_print_subscriptions_list() {
 
-	$subscribers_table = new WPInv_Subscriptions_List_Table();
-	$subscribers_table->prepare_items();
+    $subscribers_table = new WPInv_Subscriptions_List_Table();
+    $subscribers_table->prepare_items();
 
-	?>
+    ?>
 	<?php $subscribers_table->views(); ?>
 	<form id="subscribers-filter" class="bsui" method="get">
 		<input type="hidden" name="page" value="wpinv-subscriptions" />
@@ -82,42 +82,42 @@ 
  */
 function wpinv_recurring_subscription_details() {
 
-	// Fetch the subscription.
-	$sub = new WPInv_Subscription( (int) $_GET['id'] );
-	if ( ! $sub->exists() ) {
+    // Fetch the subscription.
+    $sub = new WPInv_Subscription( (int) $_GET['id'] );
+    if ( ! $sub->exists() ) {
 
-		aui()->alert(
-			array(
-				'type'    => 'danger',
-				'content' => __( 'Subscription not found.', 'invoicing' ),
-			),
-			true
-		);
+        aui()->alert(
+            array(
+                'type'    => 'danger',
+                'content' => __( 'Subscription not found.', 'invoicing' ),
+            ),
+            true
+        );
 
-		return;
-	}
+        return;
+    }
 
-	// Use metaboxes to display the subscription details.
-	add_meta_box( 'getpaid_admin_subscription_details_metabox', __( 'Subscription Details', 'invoicing' ), 'getpaid_admin_subscription_details_metabox', get_current_screen(), 'normal', 'high' );
-	add_meta_box( 'getpaid_admin_subscription_update_metabox', __( 'Change Status', 'invoicing' ), 'getpaid_admin_subscription_update_metabox', get_current_screen(), 'side' );
+    // Use metaboxes to display the subscription details.
+    add_meta_box( 'getpaid_admin_subscription_details_metabox', __( 'Subscription Details', 'invoicing' ), 'getpaid_admin_subscription_details_metabox', get_current_screen(), 'normal', 'high' );
+    add_meta_box( 'getpaid_admin_subscription_update_metabox', __( 'Change Status', 'invoicing' ), 'getpaid_admin_subscription_update_metabox', get_current_screen(), 'side' );
 
-	$subscription_id     = $sub->get_id();
-	$subscription_groups = getpaid_get_invoice_subscription_groups( $sub->get_parent_invoice_id() );
-	$subscription_group  = wp_list_filter( $subscription_groups, compact( 'subscription_id' ) );
+    $subscription_id     = $sub->get_id();
+    $subscription_groups = getpaid_get_invoice_subscription_groups( $sub->get_parent_invoice_id() );
+    $subscription_group  = wp_list_filter( $subscription_groups, compact( 'subscription_id' ) );
 
-	if ( 1 < count( $subscription_groups ) ) {
-		add_meta_box( 'getpaid_admin_subscription_related_subscriptions_metabox', __( 'Related Subscriptions', 'invoicing' ), 'getpaid_admin_subscription_related_subscriptions_metabox', get_current_screen(), 'advanced' );
-	}
+    if ( 1 < count( $subscription_groups ) ) {
+        add_meta_box( 'getpaid_admin_subscription_related_subscriptions_metabox', __( 'Related Subscriptions', 'invoicing' ), 'getpaid_admin_subscription_related_subscriptions_metabox', get_current_screen(), 'advanced' );
+    }
 
-	if ( ! empty( $subscription_group ) ) {
-		add_meta_box( 'getpaid_admin_subscription_item_details_metabox', __( 'Subscription Items', 'invoicing' ), 'getpaid_admin_subscription_item_details_metabox', get_current_screen(), 'normal', 'low' );
-	}
+    if ( ! empty( $subscription_group ) ) {
+        add_meta_box( 'getpaid_admin_subscription_item_details_metabox', __( 'Subscription Items', 'invoicing' ), 'getpaid_admin_subscription_item_details_metabox', get_current_screen(), 'normal', 'low' );
+    }
 
-	add_meta_box( 'getpaid_admin_subscription_invoice_details_metabox', __( 'Related Invoices', 'invoicing' ), 'getpaid_admin_subscription_invoice_details_metabox', get_current_screen(), 'advanced' );
+    add_meta_box( 'getpaid_admin_subscription_invoice_details_metabox', __( 'Related Invoices', 'invoicing' ), 'getpaid_admin_subscription_invoice_details_metabox', get_current_screen(), 'advanced' );
 
-	do_action( 'getpaid_admin_single_subscription_register_metabox', $sub );
+    do_action( 'getpaid_admin_single_subscription_register_metabox', $sub );
 
-	?>
+    ?>
 
 		<form method="post" action="<?php echo esc_url( admin_url( 'admin.php?page=wpinv-subscriptions&id=' . absint( $sub->get_id() ) ) ); ?>">
 
@@ -157,44 +157,44 @@ 
  */
 function getpaid_admin_subscription_details_metabox( $sub ) {
 
-	// Subscription items.
-	$subscription_group = getpaid_get_invoice_subscription_group( $sub->get_parent_invoice_id(), $sub->get_id() );
-	$items_count        = empty( $subscription_group ) ? 1 : count( $subscription_group['items'] );
-
-	// Prepare subscription detail columns.
-	$fields = apply_filters(
-		'getpaid_subscription_admin_page_fields',
-		array(
-			'subscription' => __( 'Subscription', 'invoicing' ),
-			'customer'     => __( 'Customer', 'invoicing' ),
-			'amount'       => __( 'Amount', 'invoicing' ),
-			'start_date'   => __( 'Start Date', 'invoicing' ),
-			'renews_on'    => __( 'Next Payment', 'invoicing' ),
-			'renewals'     => __( 'Payments', 'invoicing' ),
-			'item'         => _n( 'Item', 'Items', $items_count, 'invoicing' ),
-			'gateway'      => __( 'Payment Method', 'invoicing' ),
-			'profile_id'   => __( 'Profile ID', 'invoicing' ),
-			'status'       => __( 'Status', 'invoicing' ),
-		)
-	);
-
-	if ( ! $sub->is_active() ) {
-
-		if ( isset( $fields['renews_on'] ) ) {
-			unset( $fields['renews_on'] );
-		}
-
-		if ( isset( $fields['gateway'] ) ) {
-			unset( $fields['gateway'] );
-		}
+    // Subscription items.
+    $subscription_group = getpaid_get_invoice_subscription_group( $sub->get_parent_invoice_id(), $sub->get_id() );
+    $items_count        = empty( $subscription_group ) ? 1 : count( $subscription_group['items'] );
+
+    // Prepare subscription detail columns.
+    $fields = apply_filters(
+        'getpaid_subscription_admin_page_fields',
+        array(
+            'subscription' => __( 'Subscription', 'invoicing' ),
+            'customer'     => __( 'Customer', 'invoicing' ),
+            'amount'       => __( 'Amount', 'invoicing' ),
+            'start_date'   => __( 'Start Date', 'invoicing' ),
+            'renews_on'    => __( 'Next Payment', 'invoicing' ),
+            'renewals'     => __( 'Payments', 'invoicing' ),
+            'item'         => _n( 'Item', 'Items', $items_count, 'invoicing' ),
+            'gateway'      => __( 'Payment Method', 'invoicing' ),
+            'profile_id'   => __( 'Profile ID', 'invoicing' ),
+            'status'       => __( 'Status', 'invoicing' ),
+        )
+    );
+
+    if ( ! $sub->is_active() ) {
+
+        if ( isset( $fields['renews_on'] ) ) {
+            unset( $fields['renews_on'] );
+        }
+
+        if ( isset( $fields['gateway'] ) ) {
+            unset( $fields['gateway'] );
+        }
 }
 
-	$profile_id = $sub->get_profile_id();
-	if ( empty( $profile_id ) && isset( $fields['profile_id'] ) ) {
-		unset( $fields['profile_id'] );
-	}
+    $profile_id = $sub->get_profile_id();
+    if ( empty( $profile_id ) && isset( $fields['profile_id'] ) ) {
+        unset( $fields['profile_id'] );
+    }
 
-	?>
+    ?>
 
 		<table class="table table-borderless" style="font-size: 14px;">
 			<tbody>
@@ -228,20 +228,20 @@ 
  */
 function getpaid_admin_subscription_metabox_display_customer( $subscription ) {
 
-	$username = __( '(Missing User)', 'invoicing' );
+    $username = __( '(Missing User)', 'invoicing' );
 
-	$user = get_userdata( $subscription->get_customer_id() );
-	if ( $user ) {
+    $user = get_userdata( $subscription->get_customer_id() );
+    if ( $user ) {
 
-		$username = sprintf(
-			'<a href="user-edit.php?user_id=%s">%s</a>',
-			absint( $user->ID ),
-			! empty( $user->display_name ) ? esc_html( $user->display_name ) : sanitize_email( $user->user_email )
-		);
+        $username = sprintf(
+            '<a href="user-edit.php?user_id=%s">%s</a>',
+            absint( $user->ID ),
+            ! empty( $user->display_name ) ? esc_html( $user->display_name ) : sanitize_email( $user->user_email )
+        );
 
-	}
+    }
 
-	echo wp_kses_post( $username );
+    echo wp_kses_post( $username );
 }
 add_action( 'getpaid_subscription_admin_display_customer', 'getpaid_admin_subscription_metabox_display_customer' );
 
@@ -251,8 +251,8 @@ 
  * @param WPInv_Subscription $subscription
  */
 function getpaid_admin_subscription_metabox_display_amount( $subscription ) {
-	$amount    = getpaid_get_formatted_subscription_amount( $subscription );
-	echo wp_kses_post( "<span>$amount</span>" );
+    $amount    = getpaid_get_formatted_subscription_amount( $subscription );
+    echo wp_kses_post( "<span>$amount</span>" );
 }
 add_action( 'getpaid_subscription_admin_display_amount', 'getpaid_admin_subscription_metabox_display_amount' );
 
@@ -263,11 +263,11 @@ 
  */
 function getpaid_admin_subscription_metabox_display_id( $subscription ) {
 
-	printf(
-		'<a href="%s">#%s</a>',
-		esc_url( admin_url( 'admin.php?page=wpinv-subscriptions&id=' . absint( $subscription->get_id() ) ) ),
-		absint( $subscription->get_id() )
-	);
+    printf(
+        '<a href="%s">#%s</a>',
+        esc_url( admin_url( 'admin.php?page=wpinv-subscriptions&id=' . absint( $subscription->get_id() ) ) ),
+        absint( $subscription->get_id() )
+    );
 
 }
 add_action( 'getpaid_subscription_admin_display_subscription', 'getpaid_admin_subscription_metabox_display_id' );
@@ -278,7 +278,7 @@ 
  * @param WPInv_Subscription $subscription
  */
 function getpaid_admin_subscription_metabox_display_start_date( $subscription ) {
-	echo esc_html( getpaid_format_date_value( $subscription->get_date_created() ) );
+    echo esc_html( getpaid_format_date_value( $subscription->get_date_created() ) );
 }
 add_action( 'getpaid_subscription_admin_display_start_date', 'getpaid_admin_subscription_metabox_display_start_date' );
 
@@ -288,7 +288,7 @@ 
  * @param WPInv_Subscription $subscription
  */
 function getpaid_admin_subscription_metabox_display_renews_on( $subscription ) {
-	echo esc_html( getpaid_format_date_value( $subscription->get_expiration() ) );
+    echo esc_html( getpaid_format_date_value( $subscription->get_expiration() ) );
 }
 add_action( 'getpaid_subscription_admin_display_renews_on', 'getpaid_admin_subscription_metabox_display_renews_on' );
 
@@ -298,8 +298,8 @@ 
  * @param WPInv_Subscription $subscription
  */
 function getpaid_admin_subscription_metabox_display_renewals( $subscription ) {
-	$max_bills = $subscription->get_bill_times();
-	echo ( (int) $subscription->get_times_billed() ) . ' / ' . ( empty( $max_bills ) ? '&infin;' : (int) $max_bills );
+    $max_bills = $subscription->get_bill_times();
+    echo ( (int) $subscription->get_times_billed() ) . ' / ' . ( empty( $max_bills ) ? '&infin;' : (int) $max_bills );
 }
 add_action( 'getpaid_subscription_admin_display_renewals', 'getpaid_admin_subscription_metabox_display_renewals' );
 /**
@@ -310,13 +310,13 @@ 
  */
 function getpaid_admin_subscription_metabox_display_item( $subscription, $subscription_group = false ) {
 
-	if ( empty( $subscription_group ) ) {
-		echo wp_kses_post( WPInv_Subscriptions_List_Table::generate_item_markup( $subscription->get_product_id() ) );
-		return;
-	}
+    if ( empty( $subscription_group ) ) {
+        echo wp_kses_post( WPInv_Subscriptions_List_Table::generate_item_markup( $subscription->get_product_id() ) );
+        return;
+    }
 
-	$markup = array_map( array( 'WPInv_Subscriptions_List_Table', 'generate_item_markup' ), array_keys( $subscription_group['items'] ) );
-	echo wp_kses_post( implode( ' | ', $markup ) );
+    $markup = array_map( array( 'WPInv_Subscriptions_List_Table', 'generate_item_markup' ), array_keys( $subscription_group['items'] ) );
+    echo wp_kses_post( implode( ' | ', $markup ) );
 
 }
 add_action( 'getpaid_subscription_admin_display_item', 'getpaid_admin_subscription_metabox_display_item', 10, 2 );
@@ -328,13 +328,13 @@ 
  */
 function getpaid_admin_subscription_metabox_display_gateway( $subscription ) {
 
-	$gateway = $subscription->get_gateway();
+    $gateway = $subscription->get_gateway();
 
-	if ( ! empty( $gateway ) ) {
-		echo esc_html( wpinv_get_gateway_admin_label( $gateway ) );
-	} else {
-		echo '&mdash;';
-	}
+    if ( ! empty( $gateway ) ) {
+        echo esc_html( wpinv_get_gateway_admin_label( $gateway ) );
+    } else {
+        echo '&mdash;';
+    }
 
 }
 add_action( 'getpaid_subscription_admin_display_gateway', 'getpaid_admin_subscription_metabox_display_gateway' );
@@ -345,7 +345,7 @@ 
  * @param WPInv_Subscription $subscription
  */
 function getpaid_admin_subscription_metabox_display_status( $subscription ) {
-	echo wp_kses_post( $subscription->get_status_label_html() );
+    echo wp_kses_post( $subscription->get_status_label_html() );
 }
 add_action( 'getpaid_subscription_admin_display_status', 'getpaid_admin_subscription_metabox_display_status' );
 
@@ -356,27 +356,27 @@ 
  */
 function getpaid_admin_subscription_metabox_display_profile_id( $subscription ) {
 
-	$profile_id = $subscription->get_profile_id();
-
-	aui()->input(
-		array(
-			'type'              => 'text',
-			'id'                => 'wpinv_subscription_profile_id',
-			'name'              => 'wpinv_subscription_profile_id',
-			'label'             => __( 'Profile Id', 'invoicing' ),
-			'label_type'        => 'hidden',
-			'placeholder'       => __( 'Profile Id', 'invoicing' ),
-			'value'             => esc_attr( $profile_id ),
-			'input_group_right' => '',
-			'no_wrap'           => true,
-		),
-		true
-	);
-
-	$url = apply_filters( 'getpaid_remote_subscription_profile_url', '', $subscription );
-	if ( ! empty( $url ) ) {
-		echo '&nbsp;<a href="' . esc_url_raw( $url ) . '" title="' . esc_attr__( 'View in Gateway', 'invoicing' ) . '" target="_blank"><i class="fas fa-external-link-alt fa-xs fa-fw align-top"></i></a>';
-	}
+    $profile_id = $subscription->get_profile_id();
+
+    aui()->input(
+        array(
+            'type'              => 'text',
+            'id'                => 'wpinv_subscription_profile_id',
+            'name'              => 'wpinv_subscription_profile_id',
+            'label'             => __( 'Profile Id', 'invoicing' ),
+            'label_type'        => 'hidden',
+            'placeholder'       => __( 'Profile Id', 'invoicing' ),
+            'value'             => esc_attr( $profile_id ),
+            'input_group_right' => '',
+            'no_wrap'           => true,
+        ),
+        true
+    );
+
+    $url = apply_filters( 'getpaid_remote_subscription_profile_url', '', $subscription );
+    if ( ! empty( $url ) ) {
+        echo '&nbsp;<a href="' . esc_url_raw( $url ) . '" title="' . esc_attr__( 'View in Gateway', 'invoicing' ) . '" target="_blank"><i class="fas fa-external-link-alt fa-xs fa-fw align-top"></i></a>';
+    }
 
 }
 add_action( 'getpaid_subscription_admin_display_profile_id', 'getpaid_admin_subscription_metabox_display_profile_id' );
@@ -388,40 +388,40 @@ 
  */
 function getpaid_admin_subscription_update_metabox( $subscription ) {
 
-	?>
+    ?>
 	<div class="mt-3">
 
 		<?php
-			aui()->select(
-				array(
-					'options'   => getpaid_get_subscription_statuses(),
-					'name'      => 'subscription_status',
-					'id'        => 'subscription_status_update_select',
-					'required'  => true,
-					'no_wrap'   => false,
-					'label'     => __( 'Subscription Status', 'invoicing' ),
-					'help_text' => __( 'Updating the status will trigger related actions and hooks', 'invoicing' ),
-					'select2'   => true,
-					'value'     => $subscription->get_status( 'edit' ),
-				),
-				true
-			);
-		?>
+            aui()->select(
+                array(
+                    'options'   => getpaid_get_subscription_statuses(),
+                    'name'      => 'subscription_status',
+                    'id'        => 'subscription_status_update_select',
+                    'required'  => true,
+                    'no_wrap'   => false,
+                    'label'     => __( 'Subscription Status', 'invoicing' ),
+                    'help_text' => __( 'Updating the status will trigger related actions and hooks', 'invoicing' ),
+                    'select2'   => true,
+                    'value'     => $subscription->get_status( 'edit' ),
+                ),
+                true
+            );
+        ?>
 
 		<div class="mt-2 px-3 py-2 bg-light border-top" style="margin: -12px;">
 
 		<?php
-			submit_button( __( 'Update', 'invoicing' ), 'primary', 'submit', false );
+            submit_button( __( 'Update', 'invoicing' ), 'primary', 'submit', false );
 
-			$url    = wp_nonce_url( add_query_arg( 'getpaid-admin-action', 'subscription_manual_renew' ), 'getpaid-nonce', 'getpaid-nonce' );
-			$anchor = __( 'Renew Subscription', 'invoicing' );
-			$title  = esc_attr__( 'Are you sure you want to extend the subscription and generate a new invoice that will be automatically marked as paid?', 'invoicing' );
+            $url    = wp_nonce_url( add_query_arg( 'getpaid-admin-action', 'subscription_manual_renew' ), 'getpaid-nonce', 'getpaid-nonce' );
+            $anchor = __( 'Renew Subscription', 'invoicing' );
+            $title  = esc_attr__( 'Are you sure you want to extend the subscription and generate a new invoice that will be automatically marked as paid?', 'invoicing' );
 
-			if ( $subscription->is_active() ) {
-			echo "<a href='" . esc_url( $url ) . "' class='float-right text-muted' onclick='return confirm(\"" . esc_attr( $title ) . "\")'>" . esc_html( $anchor ) . "</a>";
-			}
+            if ( $subscription->is_active() ) {
+            echo "<a href='" . esc_url( $url ) . "' class='float-right text-muted' onclick='return confirm(\"" . esc_attr( $title ) . "\")'>" . esc_html( $anchor ) . "</a>";
+            }
 
-	echo '</div></div>';
+    echo '</div></div>';
 }
 
 /**
@@ -432,33 +432,33 @@ 
  */
 function getpaid_admin_subscription_invoice_details_metabox( $subscription, $strict = true ) {
 
-	$columns = apply_filters(
-		'getpaid_subscription_related_invoices_columns',
-		array(
-			'invoice'      => __( 'Invoice', 'invoicing' ),
-			'relationship' => __( 'Relationship', 'invoicing' ),
-			'date'         => __( 'Date', 'invoicing' ),
-			'status'       => __( 'Status', 'invoicing' ),
-			'total'        => __( 'Total', 'invoicing' ),
-		),
-		$subscription
-	);
-
-	// Prepare the invoices.
-	$payments = $subscription->get_child_payments( ! is_admin() );
-	$parent   = $subscription->get_parent_invoice();
-
-	if ( $parent->exists() ) {
-		$payments = array_merge( array( $parent ), $payments );
-	}
-
-	$table_class = 'w-100 bg-white';
-
-	if ( ! is_admin() ) {
-		$table_class = 'table table-bordered';
-	}
-
-	?>
+    $columns = apply_filters(
+        'getpaid_subscription_related_invoices_columns',
+        array(
+            'invoice'      => __( 'Invoice', 'invoicing' ),
+            'relationship' => __( 'Relationship', 'invoicing' ),
+            'date'         => __( 'Date', 'invoicing' ),
+            'status'       => __( 'Status', 'invoicing' ),
+            'total'        => __( 'Total', 'invoicing' ),
+        ),
+        $subscription
+    );
+
+    // Prepare the invoices.
+    $payments = $subscription->get_child_payments( ! is_admin() );
+    $parent   = $subscription->get_parent_invoice();
+
+    if ( $parent->exists() ) {
+        $payments = array_merge( array( $parent ), $payments );
+    }
+
+    $table_class = 'w-100 bg-white';
+
+    if ( ! is_admin() ) {
+        $table_class = 'table table-bordered';
+    }
+
+    ?>
 		<div class="m-0" style="overflow: auto;">
 
 			<table class="<?php echo esc_attr( $table_class ); ?>">
@@ -466,10 +466,10 @@ 
 				<thead>
 					<tr>
 						<?php
-							foreach ( $columns as $key => $label ) {
-							echo "<th class='subscription-invoice-field-" . esc_attr( $key ) . " bg-light p-2 text-left color-dark font-weight-bold'>" . esc_html( $label ) . "</th>";
-							}
-						?>
+                            foreach ( $columns as $key => $label ) {
+                            echo "<th class='subscription-invoice-field-" . esc_attr( $key ) . " bg-light p-2 text-left color-dark font-weight-bold'>" . esc_html( $label ) . "</th>";
+                            }
+                        ?>
 					</tr>
 				</thead>
 
@@ -485,72 +485,72 @@ 
 
 					<?php
 
-						foreach ( $payments as $payment ) :
+                        foreach ( $payments as $payment ) :
 
-						// Ensure that we have an invoice.
-						$payment = new WPInv_Invoice( $payment );
+                        // Ensure that we have an invoice.
+                        $payment = new WPInv_Invoice( $payment );
 
-						// Abort if the invoice is invalid...
-						if ( ! $payment->exists() ) {
-							continue;
-							}
+                        // Abort if the invoice is invalid...
+                        if ( ! $payment->exists() ) {
+                            continue;
+                            }
 
-						// ... or belongs to a different subscription.
-						if ( $strict && $payment->is_renewal() && $payment->get_subscription_id() && $payment->get_subscription_id() != $subscription->get_id() ) {
-							continue;
-							}
+                        // ... or belongs to a different subscription.
+                        if ( $strict && $payment->is_renewal() && $payment->get_subscription_id() && $payment->get_subscription_id() != $subscription->get_id() ) {
+                            continue;
+                            }
 
-						echo '<tr>';
+                        echo '<tr>';
 
-						foreach ( array_keys( $columns ) as $key ) {
+                        foreach ( array_keys( $columns ) as $key ) {
 
-							echo "<td class='p-2 text-left'>";
+                            echo "<td class='p-2 text-left'>";
 
-								switch ( $key ) {
+                                switch ( $key ) {
 
-								case 'total':
-										echo '<strong>';
-										wpinv_the_price( $payment->get_total(), $payment->get_currency() );
-										echo '</strong>';
-									break;
+                                case 'total':
+                                        echo '<strong>';
+                                        wpinv_the_price( $payment->get_total(), $payment->get_currency() );
+                                        echo '</strong>';
+                                    break;
 
-								case 'relationship':
-										echo $payment->is_renewal() ? esc_html__( 'Renewal Invoice', 'invoicing' ) : esc_html__( 'Initial Invoice', 'invoicing' );
-									break;
+                                case 'relationship':
+                                        echo $payment->is_renewal() ? esc_html__( 'Renewal Invoice', 'invoicing' ) : esc_html__( 'Initial Invoice', 'invoicing' );
+                                    break;
 
-								case 'date':
-									echo esc_html( getpaid_format_date_value( $payment->get_date_created() ) );
-									break;
+                                case 'date':
+                                    echo esc_html( getpaid_format_date_value( $payment->get_date_created() ) );
+                                    break;
 
-								case 'status':
-										$status = $payment->get_status_nicename();
-										if ( is_admin() ) {
-										$status = $payment->get_status_label_html();
-										}
+                                case 'status':
+                                        $status = $payment->get_status_nicename();
+                                        if ( is_admin() ) {
+                                        $status = $payment->get_status_label_html();
+                                        }
 
-										echo wp_kses_post( $status );
-									break;
+                                        echo wp_kses_post( $status );
+                                    break;
 
-								case 'invoice':
-										$link    = esc_url( get_edit_post_link( $payment->get_id() ) );
+                                case 'invoice':
+                                        $link    = esc_url( get_edit_post_link( $payment->get_id() ) );
 
-										if ( ! is_admin() ) {
-										$link = esc_url( $payment->get_view_url() );
-										}
+                                        if ( ! is_admin() ) {
+                                        $link = esc_url( $payment->get_view_url() );
+                                        }
 
-										$invoice = esc_html( $payment->get_number() );
-										echo wp_kses_post( "<a href='$link'>$invoice</a>" );
-									break;
-										}
+                                        $invoice = esc_html( $payment->get_number() );
+                                        echo wp_kses_post( "<a href='$link'>$invoice</a>" );
+                                    break;
+                                        }
 
-								echo '</td>';
+                                echo '</td>';
 
-							}
+                            }
 
-						echo '</tr>';
+                        echo '</tr>';
 
-						endforeach;
-					?>
+                        endforeach;
+                    ?>
 
 				</tbody>
 
@@ -568,42 +568,42 @@ 
  */
 function getpaid_admin_subscription_item_details_metabox( $subscription ) {
 
-	// Fetch the subscription group.
-	$subscription_group = getpaid_get_invoice_subscription_group( $subscription->get_parent_payment_id(), $subscription->get_id() );
+    // Fetch the subscription group.
+    $subscription_group = getpaid_get_invoice_subscription_group( $subscription->get_parent_payment_id(), $subscription->get_id() );
 
-	if ( empty( $subscription_group ) || empty( $subscription_group['items'] ) ) {
-		return;
-	}
+    if ( empty( $subscription_group ) || empty( $subscription_group['items'] ) ) {
+        return;
+    }
 
-	// Prepare table columns.
-	$columns = apply_filters(
-		'getpaid_subscription_item_details_columns',
-		array(
-			'item_name' => __( 'Item', 'invoicing' ),
-			'price'     => __( 'Price', 'invoicing' ),
-			'tax'       => __( 'Tax', 'invoicing' ),
-			'discount'  => __( 'Discount', 'invoicing' ),
-			//'initial'      => __( 'Initial Amount', 'invoicing' ),
-			'recurring' => __( 'Subtotal', 'invoicing' ),
-		),
-		$subscription
-	);
+    // Prepare table columns.
+    $columns = apply_filters(
+        'getpaid_subscription_item_details_columns',
+        array(
+            'item_name' => __( 'Item', 'invoicing' ),
+            'price'     => __( 'Price', 'invoicing' ),
+            'tax'       => __( 'Tax', 'invoicing' ),
+            'discount'  => __( 'Discount', 'invoicing' ),
+            //'initial'      => __( 'Initial Amount', 'invoicing' ),
+            'recurring' => __( 'Subtotal', 'invoicing' ),
+        ),
+        $subscription
+    );
 
-	// Prepare the invoices.
+    // Prepare the invoices.
 
-	$invoice = $subscription->get_parent_invoice();
+    $invoice = $subscription->get_parent_invoice();
 
-	if ( ( ! wpinv_use_taxes() || ! $invoice->is_taxable() ) && isset( $columns['tax'] ) ) {
-		unset( $columns['tax'] );
-	}
+    if ( ( ! wpinv_use_taxes() || ! $invoice->is_taxable() ) && isset( $columns['tax'] ) ) {
+        unset( $columns['tax'] );
+    }
 
-	$table_class = 'w-100 bg-white';
+    $table_class = 'w-100 bg-white';
 
-	if ( ! is_admin() ) {
-		$table_class = 'table table-bordered';
-	}
+    if ( ! is_admin() ) {
+        $table_class = 'table table-bordered';
+    }
 
-	?>
+    ?>
 		<div class="m-0" style="overflow: auto;">
 
 			<table class="<?php echo esc_attr( $table_class ); ?>">
@@ -612,10 +612,10 @@ 
 					<tr>
 						<?php
 
-							foreach ( $columns as $key => $label ) {
-							echo "<th class='subscription-item-field-" . esc_attr( $key ) . " bg-light p-2 text-left color-dark font-weight-bold'>" . esc_html( $label ) . "</th>";
-							}
-						?>
+                            foreach ( $columns as $key => $label ) {
+                            echo "<th class='subscription-item-field-" . esc_attr( $key ) . " bg-light p-2 text-left color-dark font-weight-bold'>" . esc_html( $label ) . "</th>";
+                            }
+                        ?>
 					</tr>
 				</thead>
 
@@ -623,106 +623,106 @@ 
 
 					<?php
 
-						foreach ( $subscription_group['items'] as $subscription_group_item ) :
+                        foreach ( $subscription_group['items'] as $subscription_group_item ) :
 
-						echo '<tr>';
+                        echo '<tr>';
 
-						foreach ( array_keys( $columns ) as $key ) {
+                        foreach ( array_keys( $columns ) as $key ) {
 
-							$class = 'text-left';
+                            $class = 'text-left';
 
-							echo "<td class='p-2 text-left'>";
+                            echo "<td class='p-2 text-left'>";
 
-								switch ( $key ) {
+                                switch ( $key ) {
 
-								case 'item_name':
-										$item_name = get_the_title( $subscription_group_item['item_id'] );
-										$item_name = empty( $item_name ) ? $subscription_group_item['item_name'] : $item_name;
+                                case 'item_name':
+                                        $item_name = get_the_title( $subscription_group_item['item_id'] );
+                                        $item_name = empty( $item_name ) ? $subscription_group_item['item_name'] : $item_name;
 
-										if ( $invoice->get_template() == 'amount' || 1 == (float) $subscription_group_item['quantity'] ) {
-										echo esc_html( $item_name );
-										} else {
-										printf( '%1$s x %2$d', esc_html( $item_name ), (float) $subscription_group_item['quantity'] );
-											}
+                                        if ( $invoice->get_template() == 'amount' || 1 == (float) $subscription_group_item['quantity'] ) {
+                                        echo esc_html( $item_name );
+                                        } else {
+                                        printf( '%1$s x %2$d', esc_html( $item_name ), (float) $subscription_group_item['quantity'] );
+                                            }
 
-									break;
+                                    break;
 
-								case 'price':
-									wpinv_the_price( $subscription_group_item['item_price'], $invoice->get_currency() );
-									break;
+                                case 'price':
+                                    wpinv_the_price( $subscription_group_item['item_price'], $invoice->get_currency() );
+                                    break;
 
-								case 'tax':
-									wpinv_the_price( $subscription_group_item['tax'], $invoice->get_currency() );
-									break;
+                                case 'tax':
+                                    wpinv_the_price( $subscription_group_item['tax'], $invoice->get_currency() );
+                                    break;
 
-								case 'discount':
-									wpinv_the_price( $subscription_group_item['discount'], $invoice->get_currency() );
-									break;
+                                case 'discount':
+                                    wpinv_the_price( $subscription_group_item['discount'], $invoice->get_currency() );
+                                    break;
 
-								case 'initial':
-									wpinv_the_price( $subscription_group_item['price'] * $subscription_group_item['quantity'], $invoice->get_currency() );
-									break;
+                                case 'initial':
+                                    wpinv_the_price( $subscription_group_item['price'] * $subscription_group_item['quantity'], $invoice->get_currency() );
+                                    break;
 
-								case 'recurring':
-										echo wp_kses_post( '<strong>' . wpinv_price( $subscription_group_item['price'] * $subscription_group_item['quantity'], $invoice->get_currency() ) . '</strong>' );
-									break;
+                                case 'recurring':
+                                        echo wp_kses_post( '<strong>' . wpinv_price( $subscription_group_item['price'] * $subscription_group_item['quantity'], $invoice->get_currency() ) . '</strong>' );
+                                    break;
 
-										}
+                                        }
 
-								echo '</td>';
+                                echo '</td>';
 
-							}
+                            }
 
-						echo '</tr>';
+                        echo '</tr>';
 
-						endforeach;
+                        endforeach;
 
-						foreach ( $subscription_group['fees'] as $subscription_group_fee ) :
+                        foreach ( $subscription_group['fees'] as $subscription_group_fee ) :
 
-						echo '<tr>';
+                        echo '<tr>';
 
-						foreach ( array_keys( $columns ) as $key ) {
+                        foreach ( array_keys( $columns ) as $key ) {
 
-							$class = 'text-left';
+                            $class = 'text-left';
 
-							echo "<td class='p-2 text-left'>";
+                            echo "<td class='p-2 text-left'>";
 
-								switch ( $key ) {
+                                switch ( $key ) {
 
-								case 'item_name':
-										echo esc_html( $subscription_group_fee['name'] );
-									break;
+                                case 'item_name':
+                                        echo esc_html( $subscription_group_fee['name'] );
+                                    break;
 
-								case 'price':
-									wpinv_the_price( $subscription_group_fee['initial_fee'], $invoice->get_currency() );
-									break;
+                                case 'price':
+                                    wpinv_the_price( $subscription_group_fee['initial_fee'], $invoice->get_currency() );
+                                    break;
 
-								case 'tax':
-									echo '&mdash;';
-									break;
+                                case 'tax':
+                                    echo '&mdash;';
+                                    break;
 
-								case 'discount':
-										echo '&mdash;';
-									break;
+                                case 'discount':
+                                        echo '&mdash;';
+                                    break;
 
-								case 'initial':
-									wpinv_the_price( $subscription_group_fee['initial_fee'], $invoice->get_currency() );
-									break;
+                                case 'initial':
+                                    wpinv_the_price( $subscription_group_fee['initial_fee'], $invoice->get_currency() );
+                                    break;
 
-								case 'recurring':
-										echo wp_kses_post( '<strong>' . wpinv_price( $subscription_group_fee['recurring_fee'], $invoice->get_currency() ) . '</strong>' );
-									break;
+                                case 'recurring':
+                                        echo wp_kses_post( '<strong>' . wpinv_price( $subscription_group_fee['recurring_fee'], $invoice->get_currency() ) . '</strong>' );
+                                    break;
 
-										}
+                                        }
 
-								echo '</td>';
+                                echo '</td>';
 
-							}
+                            }
 
-						echo '</tr>';
+                        echo '</tr>';
 
-						endforeach;
-					?>
+                        endforeach;
+                    ?>
 
 				</tbody>
 
@@ -741,38 +741,38 @@ 
  */
 function getpaid_admin_subscription_related_subscriptions_metabox( $subscription, $skip_current = true ) {
 
-	// Fetch the subscription groups.
-	$subscription_groups = getpaid_get_invoice_subscription_groups( $subscription->get_parent_payment_id() );
-
-	if ( empty( $subscription_groups ) ) {
-		return;
-	}
-
-	// Prepare table columns.
-	$columns = apply_filters(
-		'getpaid_subscription_related_subscriptions_columns',
-		array(
-			'subscription' => __( 'Subscription', 'invoicing' ),
-			'start_date'   => __( 'Start Date', 'invoicing' ),
-			'renewal_date' => __( 'Next Payment', 'invoicing' ),
-			'renewals'     => __( 'Payments', 'invoicing' ),
-			'item'         => __( 'Items', 'invoicing' ),
-			'status'       => __( 'Status', 'invoicing' ),
-		),
-		$subscription
-	);
-
-	if ( $subscription->get_status() == 'pending' ) {
-		unset( $columns['start_date'], $columns['renewal_date'] );
-	}
-
-	$table_class = 'w-100 bg-white';
-
-	if ( ! is_admin() ) {
-		$table_class = 'table table-bordered';
-	}
-
-	?>
+    // Fetch the subscription groups.
+    $subscription_groups = getpaid_get_invoice_subscription_groups( $subscription->get_parent_payment_id() );
+
+    if ( empty( $subscription_groups ) ) {
+        return;
+    }
+
+    // Prepare table columns.
+    $columns = apply_filters(
+        'getpaid_subscription_related_subscriptions_columns',
+        array(
+            'subscription' => __( 'Subscription', 'invoicing' ),
+            'start_date'   => __( 'Start Date', 'invoicing' ),
+            'renewal_date' => __( 'Next Payment', 'invoicing' ),
+            'renewals'     => __( 'Payments', 'invoicing' ),
+            'item'         => __( 'Items', 'invoicing' ),
+            'status'       => __( 'Status', 'invoicing' ),
+        ),
+        $subscription
+    );
+
+    if ( $subscription->get_status() == 'pending' ) {
+        unset( $columns['start_date'], $columns['renewal_date'] );
+    }
+
+    $table_class = 'w-100 bg-white';
+
+    if ( ! is_admin() ) {
+        $table_class = 'table table-bordered';
+    }
+
+    ?>
 		<div class="m-0" style="overflow: auto;">
 
 			<table class="<?php echo esc_attr( $table_class ); ?>">
@@ -781,10 +781,10 @@ 
 					<tr>
 						<?php
 
-							foreach ( $columns as $key => $label ) {
-							echo "<th class='related-subscription-field-" . esc_attr( $key ) . " bg-light p-2 text-left color-dark font-weight-bold'>" . esc_html( $label ) . "</th>";
-							}
-						?>
+                            foreach ( $columns as $key => $label ) {
+                            echo "<th class='related-subscription-field-" . esc_attr( $key ) . " bg-light p-2 text-left color-dark font-weight-bold'>" . esc_html( $label ) . "</th>";
+                            }
+                        ?>
 					</tr>
 				</thead>
 
@@ -792,74 +792,74 @@ 
 
 					<?php
 
-						foreach ( $subscription_groups as $subscription_group ) :
+                        foreach ( $subscription_groups as $subscription_group ) :
 
-						// Do not list current subscription.
-						if ( $skip_current && (int) $subscription_group['subscription_id'] === $subscription->get_id() ) {
-							continue;
-							}
+                        // Do not list current subscription.
+                        if ( $skip_current && (int) $subscription_group['subscription_id'] === $subscription->get_id() ) {
+                            continue;
+                            }
 
-						// Ensure the subscription exists.
-						$_suscription = new WPInv_Subscription( $subscription_group['subscription_id'] );
+                        // Ensure the subscription exists.
+                        $_suscription = new WPInv_Subscription( $subscription_group['subscription_id'] );
 
-						if ( ! $_suscription->exists() ) {
-							continue;
-							}
+                        if ( ! $_suscription->exists() ) {
+                            continue;
+                            }
 
-						echo '<tr>';
+                        echo '<tr>';
 
-						foreach ( array_keys( $columns ) as $key ) {
+                        foreach ( array_keys( $columns ) as $key ) {
 
-							$class = 'text-left';
+                            $class = 'text-left';
 
-							echo "<td class='p-2 text-left'>";
+                            echo "<td class='p-2 text-left'>";
 
-								switch ( $key ) {
+                                switch ( $key ) {
 
-								case 'status':
-										echo wp_kses_post( $_suscription->get_status_label_html() );
-									break;
+                                case 'status':
+                                        echo wp_kses_post( $_suscription->get_status_label_html() );
+                                    break;
 
-								case 'item':
-											$markup = array_map( array( 'WPInv_Subscriptions_List_Table', 'generate_item_markup' ), array_keys( $subscription_group['items'] ) );
-											echo wp_kses_post( implode( ' | ', $markup ) );
-									break;
+                                case 'item':
+                                            $markup = array_map( array( 'WPInv_Subscriptions_List_Table', 'generate_item_markup' ), array_keys( $subscription_group['items'] ) );
+                                            echo wp_kses_post( implode( ' | ', $markup ) );
+                                    break;
 
-								case 'renewals':
-									$max_bills = $_suscription->get_bill_times();
-									echo ( (int) $_suscription->get_times_billed() ) . ' / ' . ( empty( $max_bills ) ? '&infin;' : (int) $max_bills );
-									break;
+                                case 'renewals':
+                                    $max_bills = $_suscription->get_bill_times();
+                                    echo ( (int) $_suscription->get_times_billed() ) . ' / ' . ( empty( $max_bills ) ? '&infin;' : (int) $max_bills );
+                                    break;
 
-								case 'renewal_date':
-										echo $_suscription->is_active() ? esc_html( getpaid_format_date_value( $_suscription->get_expiration() ) ) : '&mdash;';
-									break;
+                                case 'renewal_date':
+                                        echo $_suscription->is_active() ? esc_html( getpaid_format_date_value( $_suscription->get_expiration() ) ) : '&mdash;';
+                                    break;
 
-								case 'start_date':
-										echo 'pending' == $_suscription->get_status() ? '&mdash;' : esc_html( getpaid_format_date_value( $_suscription->get_date_created() ) );
-									break;
+                                case 'start_date':
+                                        echo 'pending' == $_suscription->get_status() ? '&mdash;' : esc_html( getpaid_format_date_value( $_suscription->get_date_created() ) );
+                                    break;
 
-								case 'subscription':
-										$url = is_admin() ? admin_url( 'admin.php?page=wpinv-subscriptions&id=' . absint( $_suscription->get_id() ) ) : $_suscription->get_view_url();
-										printf(
+                                case 'subscription':
+                                        $url = is_admin() ? admin_url( 'admin.php?page=wpinv-subscriptions&id=' . absint( $_suscription->get_id() ) ) : $_suscription->get_view_url();
+                                        printf(
                                             '%1$s#%2$s%3$s',
                                             '<a href="' . esc_url( $url ) . '">',
                                             '<strong>' . intval( $_suscription->get_id() ) . '</strong>',
-											'</a>'
+                                            '</a>'
                                         );
 
-											echo wp_kses_post( WPInv_Subscriptions_List_Table::column_amount( $_suscription ) );
-									break;
+                                            echo wp_kses_post( WPInv_Subscriptions_List_Table::column_amount( $_suscription ) );
+                                    break;
 
-										}
+                                        }
 
-								echo '</td>';
+                                echo '</td>';
 
-							}
+                            }
 
-						echo '</tr>';
+                        echo '</tr>';
 
-						endforeach;
-					?>
+                        endforeach;
+                    ?>
 
 				</tbody>
 

Switch Indentation +108 added lines, -108 removed lines

@@ -508,39 +508,39 @@ 
 
 								switch ( $key ) {
 
-								case 'total':
-										echo '<strong>';
-										wpinv_the_price( $payment->get_total(), $payment->get_currency() );
-										echo '</strong>';
-									break;
-
-								case 'relationship':
-										echo $payment->is_renewal() ? esc_html__( 'Renewal Invoice', 'invoicing' ) : esc_html__( 'Initial Invoice', 'invoicing' );
-									break;
-
-								case 'date':
-									echo esc_html( getpaid_format_date_value( $payment->get_date_created() ) );
-									break;
-
-								case 'status':
-										$status = $payment->get_status_nicename();
-										if ( is_admin() ) {
-										$status = $payment->get_status_label_html();
-										}
-
-										echo wp_kses_post( $status );
-									break;
-
-								case 'invoice':
-										$link    = esc_url( get_edit_post_link( $payment->get_id() ) );
-
-										if ( ! is_admin() ) {
-										$link = esc_url( $payment->get_view_url() );
-										}
-
-										$invoice = esc_html( $payment->get_number() );
-										echo wp_kses_post( "<a href='$link'>$invoice</a>" );
-									break;
+								    case 'total':
+										    echo '<strong>';
+										    wpinv_the_price( $payment->get_total(), $payment->get_currency() );
+										    echo '</strong>';
+									    break;
+
+								    case 'relationship':
+										    echo $payment->is_renewal() ? esc_html__( 'Renewal Invoice', 'invoicing' ) : esc_html__( 'Initial Invoice', 'invoicing' );
+									    break;
+
+								    case 'date':
+									    echo esc_html( getpaid_format_date_value( $payment->get_date_created() ) );
+									    break;
+
+								    case 'status':
+										    $status = $payment->get_status_nicename();
+										    if ( is_admin() ) {
+										    $status = $payment->get_status_label_html();
+										    }
+
+										    echo wp_kses_post( $status );
+									    break;
+
+								    case 'invoice':
+										    $link    = esc_url( get_edit_post_link( $payment->get_id() ) );
+
+										    if ( ! is_admin() ) {
+										    $link = esc_url( $payment->get_view_url() );
+										    }
+
+										    $invoice = esc_html( $payment->get_number() );
+										    echo wp_kses_post( "<a href='$link'>$invoice</a>" );
+									    break;
 										}
 
 								echo '</td>';
@@ -635,37 +635,37 @@ 
 
 								switch ( $key ) {
 
-								case 'item_name':
-										$item_name = get_the_title( $subscription_group_item['item_id'] );
-										$item_name = empty( $item_name ) ? $subscription_group_item['item_name'] : $item_name;
+								    case 'item_name':
+										    $item_name = get_the_title( $subscription_group_item['item_id'] );
+										    $item_name = empty( $item_name ) ? $subscription_group_item['item_name'] : $item_name;
 
-										if ( $invoice->get_template() == 'amount' || 1 == (float) $subscription_group_item['quantity'] ) {
-										echo esc_html( $item_name );
-										} else {
-										printf( '%1$s x %2$d', esc_html( $item_name ), (float) $subscription_group_item['quantity'] );
-											}
+										    if ( $invoice->get_template() == 'amount' || 1 == (float) $subscription_group_item['quantity'] ) {
+										    echo esc_html( $item_name );
+										    } else {
+										    printf( '%1$s x %2$d', esc_html( $item_name ), (float) $subscription_group_item['quantity'] );
+											    }
 
-									break;
+									    break;
 
-								case 'price':
-									wpinv_the_price( $subscription_group_item['item_price'], $invoice->get_currency() );
-									break;
+								    case 'price':
+									    wpinv_the_price( $subscription_group_item['item_price'], $invoice->get_currency() );
+									    break;
 
-								case 'tax':
-									wpinv_the_price( $subscription_group_item['tax'], $invoice->get_currency() );
-									break;
+								    case 'tax':
+									    wpinv_the_price( $subscription_group_item['tax'], $invoice->get_currency() );
+									    break;
 
-								case 'discount':
-									wpinv_the_price( $subscription_group_item['discount'], $invoice->get_currency() );
-									break;
+								    case 'discount':
+									    wpinv_the_price( $subscription_group_item['discount'], $invoice->get_currency() );
+									    break;
 
-								case 'initial':
-									wpinv_the_price( $subscription_group_item['price'] * $subscription_group_item['quantity'], $invoice->get_currency() );
-									break;
+								    case 'initial':
+									    wpinv_the_price( $subscription_group_item['price'] * $subscription_group_item['quantity'], $invoice->get_currency() );
+									    break;
 
-								case 'recurring':
-										echo wp_kses_post( '<strong>' . wpinv_price( $subscription_group_item['price'] * $subscription_group_item['quantity'], $invoice->get_currency() ) . '</strong>' );
-									break;
+								    case 'recurring':
+										    echo wp_kses_post( '<strong>' . wpinv_price( $subscription_group_item['price'] * $subscription_group_item['quantity'], $invoice->get_currency() ) . '</strong>' );
+									    break;
 
 										}
 
@@ -689,29 +689,29 @@ 
 
 								switch ( $key ) {
 
-								case 'item_name':
-										echo esc_html( $subscription_group_fee['name'] );
-									break;
+								    case 'item_name':
+										    echo esc_html( $subscription_group_fee['name'] );
+									    break;
 
-								case 'price':
-									wpinv_the_price( $subscription_group_fee['initial_fee'], $invoice->get_currency() );
-									break;
+								    case 'price':
+									    wpinv_the_price( $subscription_group_fee['initial_fee'], $invoice->get_currency() );
+									    break;
 
-								case 'tax':
-									echo '&mdash;';
-									break;
+								    case 'tax':
+									    echo '&mdash;';
+									    break;
 
-								case 'discount':
-										echo '&mdash;';
-									break;
+								    case 'discount':
+										    echo '&mdash;';
+									    break;
 
-								case 'initial':
-									wpinv_the_price( $subscription_group_fee['initial_fee'], $invoice->get_currency() );
-									break;
+								    case 'initial':
+									    wpinv_the_price( $subscription_group_fee['initial_fee'], $invoice->get_currency() );
+									    break;
 
-								case 'recurring':
-										echo wp_kses_post( '<strong>' . wpinv_price( $subscription_group_fee['recurring_fee'], $invoice->get_currency() ) . '</strong>' );
-									break;
+								    case 'recurring':
+										    echo wp_kses_post( '<strong>' . wpinv_price( $subscription_group_fee['recurring_fee'], $invoice->get_currency() ) . '</strong>' );
+									    break;
 
 										}
 
@@ -816,39 +816,39 @@ 
 
 								switch ( $key ) {
 
-								case 'status':
-										echo wp_kses_post( $_suscription->get_status_label_html() );
-									break;
-
-								case 'item':
-											$markup = array_map( array( 'WPInv_Subscriptions_List_Table', 'generate_item_markup' ), array_keys( $subscription_group['items'] ) );
-											echo wp_kses_post( implode( ' | ', $markup ) );
-									break;
-
-								case 'renewals':
-									$max_bills = $_suscription->get_bill_times();
-									echo ( (int) $_suscription->get_times_billed() ) . ' / ' . ( empty( $max_bills ) ? '&infin;' : (int) $max_bills );
-									break;
-
-								case 'renewal_date':
-										echo $_suscription->is_active() ? esc_html( getpaid_format_date_value( $_suscription->get_expiration() ) ) : '&mdash;';
-									break;
-
-								case 'start_date':
-										echo 'pending' == $_suscription->get_status() ? '&mdash;' : esc_html( getpaid_format_date_value( $_suscription->get_date_created() ) );
-									break;
-
-								case 'subscription':
-										$url = is_admin() ? admin_url( 'admin.php?page=wpinv-subscriptions&id=' . absint( $_suscription->get_id() ) ) : $_suscription->get_view_url();
-										printf(
-                                            '%1$s#%2$s%3$s',
-                                            '<a href="' . esc_url( $url ) . '">',
-                                            '<strong>' . intval( $_suscription->get_id() ) . '</strong>',
-											'</a>'
-                                        );
-
-											echo wp_kses_post( WPInv_Subscriptions_List_Table::column_amount( $_suscription ) );
-									break;
+								    case 'status':
+										    echo wp_kses_post( $_suscription->get_status_label_html() );
+									    break;
+
+								    case 'item':
+											    $markup = array_map( array( 'WPInv_Subscriptions_List_Table', 'generate_item_markup' ), array_keys( $subscription_group['items'] ) );
+											    echo wp_kses_post( implode( ' | ', $markup ) );
+									    break;
+
+								    case 'renewals':
+									    $max_bills = $_suscription->get_bill_times();
+									    echo ( (int) $_suscription->get_times_billed() ) . ' / ' . ( empty( $max_bills ) ? '&infin;' : (int) $max_bills );
+									    break;
+
+								    case 'renewal_date':
+										    echo $_suscription->is_active() ? esc_html( getpaid_format_date_value( $_suscription->get_expiration() ) ) : '&mdash;';
+									    break;
+
+								    case 'start_date':
+										    echo 'pending' == $_suscription->get_status() ? '&mdash;' : esc_html( getpaid_format_date_value( $_suscription->get_date_created() ) );
+									    break;
+
+								    case 'subscription':
+										    $url = is_admin() ? admin_url( 'admin.php?page=wpinv-subscriptions&id=' . absint( $_suscription->get_id() ) ) : $_suscription->get_view_url();
+										    printf(
+                                                '%1$s#%2$s%3$s',
+                                                '<a href="' . esc_url( $url ) . '">',
+                                                '<strong>' . intval( $_suscription->get_id() ) . '</strong>',
+											    '</a>'
+                                            );
+
+											    echo wp_kses_post( WPInv_Subscriptions_List_Table::column_amount( $_suscription ) );
+									    break;
 
 										}
 

Spacing +203 added lines, -203 removed lines

@@ -3,7 +3,7 @@ 
  * Contains functions that display the subscriptions admin page.
  */
 
-defined( 'ABSPATH' ) || exit;
+defined('ABSPATH') || exit;
 
 /**
  * Render the Subscriptions page
@@ -17,23 +17,23 @@ 
 	?>
 
 	<div class="wrap">
-		<h1><?php echo esc_html( get_admin_page_title() ); ?></h1>
+		<h1><?php echo esc_html(get_admin_page_title()); ?></h1>
 		<div class="bsui">
 
 			<?php
 
 				// Verify user permissions.
-				if ( ! wpinv_current_user_can_manage_invoicing() ) {
+				if (!wpinv_current_user_can_manage_invoicing()) {
 
 				aui()->alert(
                     array(
 						'type'    => 'danger',
-						'content' => __( 'You are not permitted to view this page.', 'invoicing' ),
+						'content' => __('You are not permitted to view this page.', 'invoicing'),
 					),
 					true
                 );
 
-				} elseif ( ! empty( $_GET['id'] ) && is_numeric( $_GET['id'] ) ) {
+				} elseif (!empty($_GET['id']) && is_numeric($_GET['id'])) {
 
 				// Display a single subscription.
 				wpinv_recurring_subscription_details();
@@ -67,7 +67,7 @@ 
 	<?php $subscribers_table->views(); ?>
 	<form id="subscribers-filter" class="bsui" method="get">
 		<input type="hidden" name="page" value="wpinv-subscriptions" />
-		<?php $subscribers_table->search_box( __( 'Search Subscriptions', 'invoicing' ), 'getpaid-search-subscriptions' ); ?>
+		<?php $subscribers_table->search_box(__('Search Subscriptions', 'invoicing'), 'getpaid-search-subscriptions'); ?>
 		<?php $subscribers_table->display(); ?>
 	</form>
 	<?php
@@ -83,13 +83,13 @@ 
 function wpinv_recurring_subscription_details() {
 
 	// Fetch the subscription.
-	$sub = new WPInv_Subscription( (int) $_GET['id'] );
-	if ( ! $sub->exists() ) {
+	$sub = new WPInv_Subscription((int) $_GET['id']);
+	if (!$sub->exists()) {
 
 		aui()->alert(
 			array(
 				'type'    => 'danger',
-				'content' => __( 'Subscription not found.', 'invoicing' ),
+				'content' => __('Subscription not found.', 'invoicing'),
 			),
 			true
 		);
@@ -98,32 +98,32 @@ 
 	}
 
 	// Use metaboxes to display the subscription details.
-	add_meta_box( 'getpaid_admin_subscription_details_metabox', __( 'Subscription Details', 'invoicing' ), 'getpaid_admin_subscription_details_metabox', get_current_screen(), 'normal', 'high' );
-	add_meta_box( 'getpaid_admin_subscription_update_metabox', __( 'Change Status', 'invoicing' ), 'getpaid_admin_subscription_update_metabox', get_current_screen(), 'side' );
+	add_meta_box('getpaid_admin_subscription_details_metabox', __('Subscription Details', 'invoicing'), 'getpaid_admin_subscription_details_metabox', get_current_screen(), 'normal', 'high');
+	add_meta_box('getpaid_admin_subscription_update_metabox', __('Change Status', 'invoicing'), 'getpaid_admin_subscription_update_metabox', get_current_screen(), 'side');
 
 	$subscription_id     = $sub->get_id();
-	$subscription_groups = getpaid_get_invoice_subscription_groups( $sub->get_parent_invoice_id() );
-	$subscription_group  = wp_list_filter( $subscription_groups, compact( 'subscription_id' ) );
+	$subscription_groups = getpaid_get_invoice_subscription_groups($sub->get_parent_invoice_id());
+	$subscription_group  = wp_list_filter($subscription_groups, compact('subscription_id'));
 
-	if ( 1 < count( $subscription_groups ) ) {
-		add_meta_box( 'getpaid_admin_subscription_related_subscriptions_metabox', __( 'Related Subscriptions', 'invoicing' ), 'getpaid_admin_subscription_related_subscriptions_metabox', get_current_screen(), 'advanced' );
+	if (1 < count($subscription_groups)) {
+		add_meta_box('getpaid_admin_subscription_related_subscriptions_metabox', __('Related Subscriptions', 'invoicing'), 'getpaid_admin_subscription_related_subscriptions_metabox', get_current_screen(), 'advanced');
 	}
 
-	if ( ! empty( $subscription_group ) ) {
-		add_meta_box( 'getpaid_admin_subscription_item_details_metabox', __( 'Subscription Items', 'invoicing' ), 'getpaid_admin_subscription_item_details_metabox', get_current_screen(), 'normal', 'low' );
+	if (!empty($subscription_group)) {
+		add_meta_box('getpaid_admin_subscription_item_details_metabox', __('Subscription Items', 'invoicing'), 'getpaid_admin_subscription_item_details_metabox', get_current_screen(), 'normal', 'low');
 	}
 
-	add_meta_box( 'getpaid_admin_subscription_invoice_details_metabox', __( 'Related Invoices', 'invoicing' ), 'getpaid_admin_subscription_invoice_details_metabox', get_current_screen(), 'advanced' );
+	add_meta_box('getpaid_admin_subscription_invoice_details_metabox', __('Related Invoices', 'invoicing'), 'getpaid_admin_subscription_invoice_details_metabox', get_current_screen(), 'advanced');
 
-	do_action( 'getpaid_admin_single_subscription_register_metabox', $sub );
+	do_action('getpaid_admin_single_subscription_register_metabox', $sub);
 
 	?>
 
-		<form method="post" action="<?php echo esc_url( admin_url( 'admin.php?page=wpinv-subscriptions&id=' . absint( $sub->get_id() ) ) ); ?>">
+		<form method="post" action="<?php echo esc_url(admin_url('admin.php?page=wpinv-subscriptions&id=' . absint($sub->get_id()))); ?>">
 
-			<?php wp_nonce_field( 'getpaid-nonce', 'getpaid-nonce' ); ?>
-			<?php wp_nonce_field( 'meta-box-order', 'meta-box-order-nonce', false ); ?>
-			<?php wp_nonce_field( 'closedpostboxes', 'closedpostboxesnonce', false ); ?>
+			<?php wp_nonce_field('getpaid-nonce', 'getpaid-nonce'); ?>
+			<?php wp_nonce_field('meta-box-order', 'meta-box-order-nonce', false); ?>
+			<?php wp_nonce_field('closedpostboxes', 'closedpostboxesnonce', false); ?>
 			<input type="hidden" name="getpaid-admin-action" value="update_single_subscription" />
 			<input type="hidden" name="subscription_id" value="<?php echo (int) $sub->get_id(); ?>" />
 
@@ -131,12 +131,12 @@ 
 				<div id="post-body" class="metabox-holder columns-<?php echo 1 == get_current_screen()->get_columns() ? '1' : '2'; ?>">
 
 					<div id="postbox-container-1" class="postbox-container">
-						<?php do_meta_boxes( get_current_screen(), 'side', $sub ); ?>
+						<?php do_meta_boxes(get_current_screen(), 'side', $sub); ?>
 					</div>
 
 					<div id="postbox-container-2" class="postbox-container">
-						<?php do_meta_boxes( get_current_screen(), 'normal', $sub ); ?>
-						<?php do_meta_boxes( get_current_screen(), 'advanced', $sub ); ?>
+						<?php do_meta_boxes(get_current_screen(), 'normal', $sub); ?>
+						<?php do_meta_boxes(get_current_screen(), 'advanced', $sub); ?>
 					</div>
 
 				</div>
@@ -155,43 +155,43 @@ 
  *
  * @param WPInv_Subscription $sub
  */
-function getpaid_admin_subscription_details_metabox( $sub ) {
+function getpaid_admin_subscription_details_metabox($sub) {
 
 	// Subscription items.
-	$subscription_group = getpaid_get_invoice_subscription_group( $sub->get_parent_invoice_id(), $sub->get_id() );
-	$items_count        = empty( $subscription_group ) ? 1 : count( $subscription_group['items'] );
+	$subscription_group = getpaid_get_invoice_subscription_group($sub->get_parent_invoice_id(), $sub->get_id());
+	$items_count        = empty($subscription_group) ? 1 : count($subscription_group['items']);
 
 	// Prepare subscription detail columns.
 	$fields = apply_filters(
 		'getpaid_subscription_admin_page_fields',
 		array(
-			'subscription' => __( 'Subscription', 'invoicing' ),
-			'customer'     => __( 'Customer', 'invoicing' ),
-			'amount'       => __( 'Amount', 'invoicing' ),
-			'start_date'   => __( 'Start Date', 'invoicing' ),
-			'renews_on'    => __( 'Next Payment', 'invoicing' ),
-			'renewals'     => __( 'Payments', 'invoicing' ),
-			'item'         => _n( 'Item', 'Items', $items_count, 'invoicing' ),
-			'gateway'      => __( 'Payment Method', 'invoicing' ),
-			'profile_id'   => __( 'Profile ID', 'invoicing' ),
-			'status'       => __( 'Status', 'invoicing' ),
+			'subscription' => __('Subscription', 'invoicing'),
+			'customer'     => __('Customer', 'invoicing'),
+			'amount'       => __('Amount', 'invoicing'),
+			'start_date'   => __('Start Date', 'invoicing'),
+			'renews_on'    => __('Next Payment', 'invoicing'),
+			'renewals'     => __('Payments', 'invoicing'),
+			'item'         => _n('Item', 'Items', $items_count, 'invoicing'),
+			'gateway'      => __('Payment Method', 'invoicing'),
+			'profile_id'   => __('Profile ID', 'invoicing'),
+			'status'       => __('Status', 'invoicing'),
 		)
 	);
 
-	if ( ! $sub->is_active() ) {
+	if (!$sub->is_active()) {
 
-		if ( isset( $fields['renews_on'] ) ) {
-			unset( $fields['renews_on'] );
+		if (isset($fields['renews_on'])) {
+			unset($fields['renews_on']);
 		}
 
-		if ( isset( $fields['gateway'] ) ) {
-			unset( $fields['gateway'] );
+		if (isset($fields['gateway'])) {
+			unset($fields['gateway']);
 		}
 }
 
 	$profile_id = $sub->get_profile_id();
-	if ( empty( $profile_id ) && isset( $fields['profile_id'] ) ) {
-		unset( $fields['profile_id'] );
+	if (empty($profile_id) && isset($fields['profile_id'])) {
+		unset($fields['profile_id']);
 	}
 
 	?>
@@ -199,16 +199,16 @@ 
 		<table class="table table-borderless" style="font-size: 14px;">
 			<tbody>
 
-				<?php foreach ( $fields as $key => $label ) : ?>
+				<?php foreach ($fields as $key => $label) : ?>
 
-					<tr class="getpaid-subscription-meta-<?php echo esc_attr( $key ); ?>">
+					<tr class="getpaid-subscription-meta-<?php echo esc_attr($key); ?>">
 
 						<th class="w-25" style="font-weight: 500;">
-							<?php echo esc_html( $label ); ?>
+							<?php echo esc_html($label); ?>
 						</th>
 
 						<td class="w-75 text-muted">
-							<?php do_action( 'getpaid_subscription_admin_display_' . sanitize_key( $key ), $sub, $subscription_group ); ?>
+							<?php do_action('getpaid_subscription_admin_display_' . sanitize_key($key), $sub, $subscription_group); ?>
 						</td>
 
 					</tr>
@@ -226,135 +226,135 @@ 
  *
  * @param WPInv_Subscription $subscription
  */
-function getpaid_admin_subscription_metabox_display_customer( $subscription ) {
+function getpaid_admin_subscription_metabox_display_customer($subscription) {
 
-	$username = __( '(Missing User)', 'invoicing' );
+	$username = __('(Missing User)', 'invoicing');
 
-	$user = get_userdata( $subscription->get_customer_id() );
-	if ( $user ) {
+	$user = get_userdata($subscription->get_customer_id());
+	if ($user) {
 
 		$username = sprintf(
 			'<a href="user-edit.php?user_id=%s">%s</a>',
-			absint( $user->ID ),
-			! empty( $user->display_name ) ? esc_html( $user->display_name ) : sanitize_email( $user->user_email )
+			absint($user->ID),
+			!empty($user->display_name) ? esc_html($user->display_name) : sanitize_email($user->user_email)
 		);
 
 	}
 
-	echo wp_kses_post( $username );
+	echo wp_kses_post($username);
 }
-add_action( 'getpaid_subscription_admin_display_customer', 'getpaid_admin_subscription_metabox_display_customer' );
+add_action('getpaid_subscription_admin_display_customer', 'getpaid_admin_subscription_metabox_display_customer');
 
 /**
  * Displays the subscription amount.
  *
  * @param WPInv_Subscription $subscription
  */
-function getpaid_admin_subscription_metabox_display_amount( $subscription ) {
-	$amount    = getpaid_get_formatted_subscription_amount( $subscription );
-	echo wp_kses_post( "<span>$amount</span>" );
+function getpaid_admin_subscription_metabox_display_amount($subscription) {
+	$amount = getpaid_get_formatted_subscription_amount($subscription);
+	echo wp_kses_post("<span>$amount</span>");
 }
-add_action( 'getpaid_subscription_admin_display_amount', 'getpaid_admin_subscription_metabox_display_amount' );
+add_action('getpaid_subscription_admin_display_amount', 'getpaid_admin_subscription_metabox_display_amount');
 
 /**
  * Displays the subscription id.
  *
  * @param WPInv_Subscription $subscription
  */
-function getpaid_admin_subscription_metabox_display_id( $subscription ) {
+function getpaid_admin_subscription_metabox_display_id($subscription) {
 
 	printf(
 		'<a href="%s">#%s</a>',
-		esc_url( admin_url( 'admin.php?page=wpinv-subscriptions&id=' . absint( $subscription->get_id() ) ) ),
-		absint( $subscription->get_id() )
+		esc_url(admin_url('admin.php?page=wpinv-subscriptions&id=' . absint($subscription->get_id()))),
+		absint($subscription->get_id())
 	);
 
 }
-add_action( 'getpaid_subscription_admin_display_subscription', 'getpaid_admin_subscription_metabox_display_id' );
+add_action('getpaid_subscription_admin_display_subscription', 'getpaid_admin_subscription_metabox_display_id');
 
 /**
  * Displays the subscription renewal date.
  *
  * @param WPInv_Subscription $subscription
  */
-function getpaid_admin_subscription_metabox_display_start_date( $subscription ) {
-	echo esc_html( getpaid_format_date_value( $subscription->get_date_created() ) );
+function getpaid_admin_subscription_metabox_display_start_date($subscription) {
+	echo esc_html(getpaid_format_date_value($subscription->get_date_created()));
 }
-add_action( 'getpaid_subscription_admin_display_start_date', 'getpaid_admin_subscription_metabox_display_start_date' );
+add_action('getpaid_subscription_admin_display_start_date', 'getpaid_admin_subscription_metabox_display_start_date');
 
 /**
  * Displays the subscription renewal date.
  *
  * @param WPInv_Subscription $subscription
  */
-function getpaid_admin_subscription_metabox_display_renews_on( $subscription ) {
-	echo esc_html( getpaid_format_date_value( $subscription->get_expiration() ) );
+function getpaid_admin_subscription_metabox_display_renews_on($subscription) {
+	echo esc_html(getpaid_format_date_value($subscription->get_expiration()));
 }
-add_action( 'getpaid_subscription_admin_display_renews_on', 'getpaid_admin_subscription_metabox_display_renews_on' );
+add_action('getpaid_subscription_admin_display_renews_on', 'getpaid_admin_subscription_metabox_display_renews_on');
 
 /**
  * Displays the subscription renewal count.
  *
  * @param WPInv_Subscription $subscription
  */
-function getpaid_admin_subscription_metabox_display_renewals( $subscription ) {
+function getpaid_admin_subscription_metabox_display_renewals($subscription) {
 	$max_bills = $subscription->get_bill_times();
-	echo ( (int) $subscription->get_times_billed() ) . ' / ' . ( empty( $max_bills ) ? '&infin;' : (int) $max_bills );
+	echo ((int) $subscription->get_times_billed()) . ' / ' . (empty($max_bills) ? '&infin;' : (int) $max_bills);
 }
-add_action( 'getpaid_subscription_admin_display_renewals', 'getpaid_admin_subscription_metabox_display_renewals' );
+add_action('getpaid_subscription_admin_display_renewals', 'getpaid_admin_subscription_metabox_display_renewals');
 /**
  * Displays the subscription item.
  *
  * @param WPInv_Subscription $subscription
  * @param false|array $subscription_group
  */
-function getpaid_admin_subscription_metabox_display_item( $subscription, $subscription_group = false ) {
+function getpaid_admin_subscription_metabox_display_item($subscription, $subscription_group = false) {
 
-	if ( empty( $subscription_group ) ) {
-		echo wp_kses_post( WPInv_Subscriptions_List_Table::generate_item_markup( $subscription->get_product_id() ) );
+	if (empty($subscription_group)) {
+		echo wp_kses_post(WPInv_Subscriptions_List_Table::generate_item_markup($subscription->get_product_id()));
 		return;
 	}
 
-	$markup = array_map( array( 'WPInv_Subscriptions_List_Table', 'generate_item_markup' ), array_keys( $subscription_group['items'] ) );
-	echo wp_kses_post( implode( ' | ', $markup ) );
+	$markup = array_map(array('WPInv_Subscriptions_List_Table', 'generate_item_markup'), array_keys($subscription_group['items']));
+	echo wp_kses_post(implode(' | ', $markup));
 
 }
-add_action( 'getpaid_subscription_admin_display_item', 'getpaid_admin_subscription_metabox_display_item', 10, 2 );
+add_action('getpaid_subscription_admin_display_item', 'getpaid_admin_subscription_metabox_display_item', 10, 2);
 
 /**
  * Displays the subscription gateway.
  *
  * @param WPInv_Subscription $subscription
  */
-function getpaid_admin_subscription_metabox_display_gateway( $subscription ) {
+function getpaid_admin_subscription_metabox_display_gateway($subscription) {
 
 	$gateway = $subscription->get_gateway();
 
-	if ( ! empty( $gateway ) ) {
-		echo esc_html( wpinv_get_gateway_admin_label( $gateway ) );
+	if (!empty($gateway)) {
+		echo esc_html(wpinv_get_gateway_admin_label($gateway));
 	} else {
 		echo '&mdash;';
 	}
 
 }
-add_action( 'getpaid_subscription_admin_display_gateway', 'getpaid_admin_subscription_metabox_display_gateway' );
+add_action('getpaid_subscription_admin_display_gateway', 'getpaid_admin_subscription_metabox_display_gateway');
 
 /**
  * Displays the subscription status.
  *
  * @param WPInv_Subscription $subscription
  */
-function getpaid_admin_subscription_metabox_display_status( $subscription ) {
-	echo wp_kses_post( $subscription->get_status_label_html() );
+function getpaid_admin_subscription_metabox_display_status($subscription) {
+	echo wp_kses_post($subscription->get_status_label_html());
 }
-add_action( 'getpaid_subscription_admin_display_status', 'getpaid_admin_subscription_metabox_display_status' );
+add_action('getpaid_subscription_admin_display_status', 'getpaid_admin_subscription_metabox_display_status');
 
 /**
  * Displays the subscription profile id.
  *
  * @param WPInv_Subscription $subscription
  */
-function getpaid_admin_subscription_metabox_display_profile_id( $subscription ) {
+function getpaid_admin_subscription_metabox_display_profile_id($subscription) {
 
 	$profile_id = $subscription->get_profile_id();
 
@@ -363,30 +363,30 @@ 
 			'type'              => 'text',
 			'id'                => 'wpinv_subscription_profile_id',
 			'name'              => 'wpinv_subscription_profile_id',
-			'label'             => __( 'Profile Id', 'invoicing' ),
+			'label'             => __('Profile Id', 'invoicing'),
 			'label_type'        => 'hidden',
-			'placeholder'       => __( 'Profile Id', 'invoicing' ),
-			'value'             => esc_attr( $profile_id ),
+			'placeholder'       => __('Profile Id', 'invoicing'),
+			'value'             => esc_attr($profile_id),
 			'input_group_right' => '',
 			'no_wrap'           => true,
 		),
 		true
 	);
 
-	$url = apply_filters( 'getpaid_remote_subscription_profile_url', '', $subscription );
-	if ( ! empty( $url ) ) {
-		echo '&nbsp;<a href="' . esc_url_raw( $url ) . '" title="' . esc_attr__( 'View in Gateway', 'invoicing' ) . '" target="_blank"><i class="fas fa-external-link-alt fa-xs fa-fw align-top"></i></a>';
+	$url = apply_filters('getpaid_remote_subscription_profile_url', '', $subscription);
+	if (!empty($url)) {
+		echo '&nbsp;<a href="' . esc_url_raw($url) . '" title="' . esc_attr__('View in Gateway', 'invoicing') . '" target="_blank"><i class="fas fa-external-link-alt fa-xs fa-fw align-top"></i></a>';
 	}
 
 }
-add_action( 'getpaid_subscription_admin_display_profile_id', 'getpaid_admin_subscription_metabox_display_profile_id' );
+add_action('getpaid_subscription_admin_display_profile_id', 'getpaid_admin_subscription_metabox_display_profile_id');
 
 /**
  * Displays the subscriptions update metabox.
  *
  * @param WPInv_Subscription $subscription
  */
-function getpaid_admin_subscription_update_metabox( $subscription ) {
+function getpaid_admin_subscription_update_metabox($subscription) {
 
 	?>
 	<div class="mt-3">
@@ -399,10 +399,10 @@ 
 					'id'        => 'subscription_status_update_select',
 					'required'  => true,
 					'no_wrap'   => false,
-					'label'     => __( 'Subscription Status', 'invoicing' ),
-					'help_text' => __( 'Updating the status will trigger related actions and hooks', 'invoicing' ),
+					'label'     => __('Subscription Status', 'invoicing'),
+					'help_text' => __('Updating the status will trigger related actions and hooks', 'invoicing'),
 					'select2'   => true,
-					'value'     => $subscription->get_status( 'edit' ),
+					'value'     => $subscription->get_status('edit'),
 				),
 				true
 			);
@@ -411,14 +411,14 @@ 
 		<div class="mt-2 px-3 py-2 bg-light border-top" style="margin: -12px;">
 
 		<?php
-			submit_button( __( 'Update', 'invoicing' ), 'primary', 'submit', false );
+			submit_button(__('Update', 'invoicing'), 'primary', 'submit', false);
 
-			$url    = wp_nonce_url( add_query_arg( 'getpaid-admin-action', 'subscription_manual_renew' ), 'getpaid-nonce', 'getpaid-nonce' );
-			$anchor = __( 'Renew Subscription', 'invoicing' );
-			$title  = esc_attr__( 'Are you sure you want to extend the subscription and generate a new invoice that will be automatically marked as paid?', 'invoicing' );
+			$url    = wp_nonce_url(add_query_arg('getpaid-admin-action', 'subscription_manual_renew'), 'getpaid-nonce', 'getpaid-nonce');
+			$anchor = __('Renew Subscription', 'invoicing');
+			$title  = esc_attr__('Are you sure you want to extend the subscription and generate a new invoice that will be automatically marked as paid?', 'invoicing');
 
-			if ( $subscription->is_active() ) {
-			echo "<a href='" . esc_url( $url ) . "' class='float-right text-muted' onclick='return confirm(\"" . esc_attr( $title ) . "\")'>" . esc_html( $anchor ) . "</a>";
+			if ($subscription->is_active()) {
+			echo "<a href='" . esc_url($url) . "' class='float-right text-muted' onclick='return confirm(\"" . esc_attr($title) . "\")'>" . esc_html($anchor) . "</a>";
 			}
 
 	echo '</div></div>';
@@ -430,44 +430,44 @@ 
  * @param WPInv_Subscription $subscription
  * @param bool $strict Whether or not to skip invoices of sibling subscriptions
  */
-function getpaid_admin_subscription_invoice_details_metabox( $subscription, $strict = true ) {
+function getpaid_admin_subscription_invoice_details_metabox($subscription, $strict = true) {
 
 	$columns = apply_filters(
 		'getpaid_subscription_related_invoices_columns',
 		array(
-			'invoice'      => __( 'Invoice', 'invoicing' ),
-			'relationship' => __( 'Relationship', 'invoicing' ),
-			'date'         => __( 'Date', 'invoicing' ),
-			'status'       => __( 'Status', 'invoicing' ),
-			'total'        => __( 'Total', 'invoicing' ),
+			'invoice'      => __('Invoice', 'invoicing'),
+			'relationship' => __('Relationship', 'invoicing'),
+			'date'         => __('Date', 'invoicing'),
+			'status'       => __('Status', 'invoicing'),
+			'total'        => __('Total', 'invoicing'),
 		),
 		$subscription
 	);
 
 	// Prepare the invoices.
-	$payments = $subscription->get_child_payments( ! is_admin() );
+	$payments = $subscription->get_child_payments(!is_admin());
 	$parent   = $subscription->get_parent_invoice();
 
-	if ( $parent->exists() ) {
-		$payments = array_merge( array( $parent ), $payments );
+	if ($parent->exists()) {
+		$payments = array_merge(array($parent), $payments);
 	}
 
 	$table_class = 'w-100 bg-white';
 
-	if ( ! is_admin() ) {
+	if (!is_admin()) {
 		$table_class = 'table table-bordered';
 	}
 
 	?>
 		<div class="m-0" style="overflow: auto;">
 
-			<table class="<?php echo esc_attr( $table_class ); ?>">
+			<table class="<?php echo esc_attr($table_class); ?>">
 
 				<thead>
 					<tr>
 						<?php
-							foreach ( $columns as $key => $label ) {
-							echo "<th class='subscription-invoice-field-" . esc_attr( $key ) . " bg-light p-2 text-left color-dark font-weight-bold'>" . esc_html( $label ) . "</th>";
+							foreach ($columns as $key => $label) {
+							echo "<th class='subscription-invoice-field-" . esc_attr($key) . " bg-light p-2 text-left color-dark font-weight-bold'>" . esc_html($label) . "</th>";
 							}
 						?>
 					</tr>
@@ -475,71 +475,71 @@ 
 
 				<tbody>
 
-					<?php if ( empty( $payments ) ) : ?>
+					<?php if (empty($payments)) : ?>
 						<tr>
-							<td colspan="<?php echo count( $columns ); ?>" class="p-2 text-left text-muted">
-								<?php esc_html_e( 'This subscription has no invoices.', 'invoicing' ); ?>
+							<td colspan="<?php echo count($columns); ?>" class="p-2 text-left text-muted">
+								<?php esc_html_e('This subscription has no invoices.', 'invoicing'); ?>
 							</td>
 						</tr>
 					<?php endif; ?>
 
 					<?php
 
-						foreach ( $payments as $payment ) :
+						foreach ($payments as $payment) :
 
 						// Ensure that we have an invoice.
-						$payment = new WPInv_Invoice( $payment );
+						$payment = new WPInv_Invoice($payment);
 
 						// Abort if the invoice is invalid...
-						if ( ! $payment->exists() ) {
+						if (!$payment->exists()) {
 							continue;
 							}
 
 						// ... or belongs to a different subscription.
-						if ( $strict && $payment->is_renewal() && $payment->get_subscription_id() && $payment->get_subscription_id() != $subscription->get_id() ) {
+						if ($strict && $payment->is_renewal() && $payment->get_subscription_id() && $payment->get_subscription_id() != $subscription->get_id()) {
 							continue;
 							}
 
 						echo '<tr>';
 
-						foreach ( array_keys( $columns ) as $key ) {
+						foreach (array_keys($columns) as $key) {
 
 							echo "<td class='p-2 text-left'>";
 
-								switch ( $key ) {
+								switch ($key) {
 
 								case 'total':
 										echo '<strong>';
-										wpinv_the_price( $payment->get_total(), $payment->get_currency() );
+										wpinv_the_price($payment->get_total(), $payment->get_currency());
 										echo '</strong>';
 									break;
 
 								case 'relationship':
-										echo $payment->is_renewal() ? esc_html__( 'Renewal Invoice', 'invoicing' ) : esc_html__( 'Initial Invoice', 'invoicing' );
+										echo $payment->is_renewal() ? esc_html__('Renewal Invoice', 'invoicing') : esc_html__('Initial Invoice', 'invoicing');
 									break;
 
 								case 'date':
-									echo esc_html( getpaid_format_date_value( $payment->get_date_created() ) );
+									echo esc_html(getpaid_format_date_value($payment->get_date_created()));
 									break;
 
 								case 'status':
 										$status = $payment->get_status_nicename();
-										if ( is_admin() ) {
+										if (is_admin()) {
 										$status = $payment->get_status_label_html();
 										}
 
-										echo wp_kses_post( $status );
+										echo wp_kses_post($status);
 									break;
 
 								case 'invoice':
-										$link    = esc_url( get_edit_post_link( $payment->get_id() ) );
+										$link = esc_url(get_edit_post_link($payment->get_id()));
 
-										if ( ! is_admin() ) {
-										$link = esc_url( $payment->get_view_url() );
+										if (!is_admin()) {
+										$link = esc_url($payment->get_view_url());
 										}
 
-										$invoice = esc_html( $payment->get_number() );
-										echo wp_kses_post( "<a href='$link'>$invoice</a>" );
+										$invoice = esc_html($payment->get_number());
+										echo wp_kses_post("<a href='$link'>$invoice</a>");
 									break;
 										}
 
@@ -566,12 +566,12 @@ 
  *
  * @param WPInv_Subscription $subscription
  */
-function getpaid_admin_subscription_item_details_metabox( $subscription ) {
+function getpaid_admin_subscription_item_details_metabox($subscription) {
 
 	// Fetch the subscription group.
-	$subscription_group = getpaid_get_invoice_subscription_group( $subscription->get_parent_payment_id(), $subscription->get_id() );
+	$subscription_group = getpaid_get_invoice_subscription_group($subscription->get_parent_payment_id(), $subscription->get_id());
 
-	if ( empty( $subscription_group ) || empty( $subscription_group['items'] ) ) {
+	if (empty($subscription_group) || empty($subscription_group['items'])) {
 		return;
 	}
 
@@ -579,12 +579,12 @@ 
 	$columns = apply_filters(
 		'getpaid_subscription_item_details_columns',
 		array(
-			'item_name' => __( 'Item', 'invoicing' ),
-			'price'     => __( 'Price', 'invoicing' ),
-			'tax'       => __( 'Tax', 'invoicing' ),
-			'discount'  => __( 'Discount', 'invoicing' ),
+			'item_name' => __('Item', 'invoicing'),
+			'price'     => __('Price', 'invoicing'),
+			'tax'       => __('Tax', 'invoicing'),
+			'discount'  => __('Discount', 'invoicing'),
 			//'initial'      => __( 'Initial Amount', 'invoicing' ),
-			'recurring' => __( 'Subtotal', 'invoicing' ),
+			'recurring' => __('Subtotal', 'invoicing'),
 		),
 		$subscription
 	);
@@ -593,27 +593,27 @@ 
 
 	$invoice = $subscription->get_parent_invoice();
 
-	if ( ( ! wpinv_use_taxes() || ! $invoice->is_taxable() ) && isset( $columns['tax'] ) ) {
-		unset( $columns['tax'] );
+	if ((!wpinv_use_taxes() || !$invoice->is_taxable()) && isset($columns['tax'])) {
+		unset($columns['tax']);
 	}
 
 	$table_class = 'w-100 bg-white';
 
-	if ( ! is_admin() ) {
+	if (!is_admin()) {
 		$table_class = 'table table-bordered';
 	}
 
 	?>
 		<div class="m-0" style="overflow: auto;">
 
-			<table class="<?php echo esc_attr( $table_class ); ?>">
+			<table class="<?php echo esc_attr($table_class); ?>">
 
 				<thead>
 					<tr>
 						<?php
 
-							foreach ( $columns as $key => $label ) {
-							echo "<th class='subscription-item-field-" . esc_attr( $key ) . " bg-light p-2 text-left color-dark font-weight-bold'>" . esc_html( $label ) . "</th>";
+							foreach ($columns as $key => $label) {
+							echo "<th class='subscription-item-field-" . esc_attr($key) . " bg-light p-2 text-left color-dark font-weight-bold'>" . esc_html($label) . "</th>";
 							}
 						?>
 					</tr>
@@ -623,48 +623,48 @@ 
 
 					<?php
 
-						foreach ( $subscription_group['items'] as $subscription_group_item ) :
+						foreach ($subscription_group['items'] as $subscription_group_item) :
 
 						echo '<tr>';
 
-						foreach ( array_keys( $columns ) as $key ) {
+						foreach (array_keys($columns) as $key) {
 
 							$class = 'text-left';
 
 							echo "<td class='p-2 text-left'>";
 
-								switch ( $key ) {
+								switch ($key) {
 
 								case 'item_name':
-										$item_name = get_the_title( $subscription_group_item['item_id'] );
-										$item_name = empty( $item_name ) ? $subscription_group_item['item_name'] : $item_name;
+										$item_name = get_the_title($subscription_group_item['item_id']);
+										$item_name = empty($item_name) ? $subscription_group_item['item_name'] : $item_name;
 
-										if ( $invoice->get_template() == 'amount' || 1 == (float) $subscription_group_item['quantity'] ) {
-										echo esc_html( $item_name );
+										if ($invoice->get_template() == 'amount' || 1 == (float) $subscription_group_item['quantity']) {
+										echo esc_html($item_name);
 										} else {
-										printf( '%1$s x %2$d', esc_html( $item_name ), (float) $subscription_group_item['quantity'] );
+										printf('%1$s x %2$d', esc_html($item_name), (float) $subscription_group_item['quantity']);
 											}
 
 									break;
 
 								case 'price':
-									wpinv_the_price( $subscription_group_item['item_price'], $invoice->get_currency() );
+									wpinv_the_price($subscription_group_item['item_price'], $invoice->get_currency());
 									break;
 
 								case 'tax':
-									wpinv_the_price( $subscription_group_item['tax'], $invoice->get_currency() );
+									wpinv_the_price($subscription_group_item['tax'], $invoice->get_currency());
 									break;
 
 								case 'discount':
-									wpinv_the_price( $subscription_group_item['discount'], $invoice->get_currency() );
+									wpinv_the_price($subscription_group_item['discount'], $invoice->get_currency());
 									break;
 
 								case 'initial':
-									wpinv_the_price( $subscription_group_item['price'] * $subscription_group_item['quantity'], $invoice->get_currency() );
+									wpinv_the_price($subscription_group_item['price'] * $subscription_group_item['quantity'], $invoice->get_currency());
 									break;
 
 								case 'recurring':
-										echo wp_kses_post( '<strong>' . wpinv_price( $subscription_group_item['price'] * $subscription_group_item['quantity'], $invoice->get_currency() ) . '</strong>' );
+										echo wp_kses_post('<strong>' . wpinv_price($subscription_group_item['price'] * $subscription_group_item['quantity'], $invoice->get_currency()) . '</strong>');
 									break;
 
 										}
@@ -677,24 +677,24 @@ 
 
 						endforeach;
 
-						foreach ( $subscription_group['fees'] as $subscription_group_fee ) :
+						foreach ($subscription_group['fees'] as $subscription_group_fee) :
 
 						echo '<tr>';
 
-						foreach ( array_keys( $columns ) as $key ) {
+						foreach (array_keys($columns) as $key) {
 
 							$class = 'text-left';
 
 							echo "<td class='p-2 text-left'>";
 
-								switch ( $key ) {
+								switch ($key) {
 
 								case 'item_name':
-										echo esc_html( $subscription_group_fee['name'] );
+										echo esc_html($subscription_group_fee['name']);
 									break;
 
 								case 'price':
-									wpinv_the_price( $subscription_group_fee['initial_fee'], $invoice->get_currency() );
+									wpinv_the_price($subscription_group_fee['initial_fee'], $invoice->get_currency());
 									break;
 
 								case 'tax':
@@ -706,11 +706,11 @@ 
 									break;
 
 								case 'initial':
-									wpinv_the_price( $subscription_group_fee['initial_fee'], $invoice->get_currency() );
+									wpinv_the_price($subscription_group_fee['initial_fee'], $invoice->get_currency());
 									break;
 
 								case 'recurring':
-										echo wp_kses_post( '<strong>' . wpinv_price( $subscription_group_fee['recurring_fee'], $invoice->get_currency() ) . '</strong>' );
+										echo wp_kses_post('<strong>' . wpinv_price($subscription_group_fee['recurring_fee'], $invoice->get_currency()) . '</strong>');
 									break;
 
 										}
@@ -739,12 +739,12 @@ 
  * @param WPInv_Subscription $subscription
  * @param bool $skip_current
  */
-function getpaid_admin_subscription_related_subscriptions_metabox( $subscription, $skip_current = true ) {
+function getpaid_admin_subscription_related_subscriptions_metabox($subscription, $skip_current = true) {
 
 	// Fetch the subscription groups.
-	$subscription_groups = getpaid_get_invoice_subscription_groups( $subscription->get_parent_payment_id() );
+	$subscription_groups = getpaid_get_invoice_subscription_groups($subscription->get_parent_payment_id());
 
-	if ( empty( $subscription_groups ) ) {
+	if (empty($subscription_groups)) {
 		return;
 	}
 
@@ -752,37 +752,37 @@ 
 	$columns = apply_filters(
 		'getpaid_subscription_related_subscriptions_columns',
 		array(
-			'subscription' => __( 'Subscription', 'invoicing' ),
-			'start_date'   => __( 'Start Date', 'invoicing' ),
-			'renewal_date' => __( 'Next Payment', 'invoicing' ),
-			'renewals'     => __( 'Payments', 'invoicing' ),
-			'item'         => __( 'Items', 'invoicing' ),
-			'status'       => __( 'Status', 'invoicing' ),
+			'subscription' => __('Subscription', 'invoicing'),
+			'start_date'   => __('Start Date', 'invoicing'),
+			'renewal_date' => __('Next Payment', 'invoicing'),
+			'renewals'     => __('Payments', 'invoicing'),
+			'item'         => __('Items', 'invoicing'),
+			'status'       => __('Status', 'invoicing'),
 		),
 		$subscription
 	);
 
-	if ( $subscription->get_status() == 'pending' ) {
-		unset( $columns['start_date'], $columns['renewal_date'] );
+	if ($subscription->get_status() == 'pending') {
+		unset($columns['start_date'], $columns['renewal_date']);
 	}
 
 	$table_class = 'w-100 bg-white';
 
-	if ( ! is_admin() ) {
+	if (!is_admin()) {
 		$table_class = 'table table-bordered';
 	}
 
 	?>
 		<div class="m-0" style="overflow: auto;">
 
-			<table class="<?php echo esc_attr( $table_class ); ?>">
+			<table class="<?php echo esc_attr($table_class); ?>">
 
 				<thead>
 					<tr>
 						<?php
 
-							foreach ( $columns as $key => $label ) {
-							echo "<th class='related-subscription-field-" . esc_attr( $key ) . " bg-light p-2 text-left color-dark font-weight-bold'>" . esc_html( $label ) . "</th>";
+							foreach ($columns as $key => $label) {
+							echo "<th class='related-subscription-field-" . esc_attr($key) . " bg-light p-2 text-left color-dark font-weight-bold'>" . esc_html($label) . "</th>";
 							}
 						?>
 					</tr>
@@ -792,62 +792,62 @@ 
 
 					<?php
 
-						foreach ( $subscription_groups as $subscription_group ) :
+						foreach ($subscription_groups as $subscription_group) :
 
 						// Do not list current subscription.
-						if ( $skip_current && (int) $subscription_group['subscription_id'] === $subscription->get_id() ) {
+						if ($skip_current && (int) $subscription_group['subscription_id'] === $subscription->get_id()) {
 							continue;
 							}
 
 						// Ensure the subscription exists.
-						$_suscription = new WPInv_Subscription( $subscription_group['subscription_id'] );
+						$_suscription = new WPInv_Subscription($subscription_group['subscription_id']);
 
-						if ( ! $_suscription->exists() ) {
+						if (!$_suscription->exists()) {
 							continue;
 							}
 
 						echo '<tr>';
 
-						foreach ( array_keys( $columns ) as $key ) {
+						foreach (array_keys($columns) as $key) {
 
 							$class = 'text-left';
 
 							echo "<td class='p-2 text-left'>";
 
-								switch ( $key ) {
+								switch ($key) {
 
 								case 'status':
-										echo wp_kses_post( $_suscription->get_status_label_html() );
+										echo wp_kses_post($_suscription->get_status_label_html());
 									break;
 
 								case 'item':
-											$markup = array_map( array( 'WPInv_Subscriptions_List_Table', 'generate_item_markup' ), array_keys( $subscription_group['items'] ) );
-											echo wp_kses_post( implode( ' | ', $markup ) );
+											$markup = array_map(array('WPInv_Subscriptions_List_Table', 'generate_item_markup'), array_keys($subscription_group['items']));
+											echo wp_kses_post(implode(' | ', $markup));
 									break;
 
 								case 'renewals':
 									$max_bills = $_suscription->get_bill_times();
-									echo ( (int) $_suscription->get_times_billed() ) . ' / ' . ( empty( $max_bills ) ? '&infin;' : (int) $max_bills );
+									echo ((int) $_suscription->get_times_billed()) . ' / ' . (empty($max_bills) ? '&infin;' : (int) $max_bills);
 									break;
 
 								case 'renewal_date':
-										echo $_suscription->is_active() ? esc_html( getpaid_format_date_value( $_suscription->get_expiration() ) ) : '&mdash;';
+										echo $_suscription->is_active() ? esc_html(getpaid_format_date_value($_suscription->get_expiration())) : '&mdash;';
 									break;
 
 								case 'start_date':
-										echo 'pending' == $_suscription->get_status() ? '&mdash;' : esc_html( getpaid_format_date_value( $_suscription->get_date_created() ) );
+										echo 'pending' == $_suscription->get_status() ? '&mdash;' : esc_html(getpaid_format_date_value($_suscription->get_date_created()));
 									break;
 
 								case 'subscription':
-										$url = is_admin() ? admin_url( 'admin.php?page=wpinv-subscriptions&id=' . absint( $_suscription->get_id() ) ) : $_suscription->get_view_url();
+										$url = is_admin() ? admin_url('admin.php?page=wpinv-subscriptions&id=' . absint($_suscription->get_id())) : $_suscription->get_view_url();
 										printf(
                                             '%1$s#%2$s%3$s',
-                                            '<a href="' . esc_url( $url ) . '">',
-                                            '<strong>' . intval( $_suscription->get_id() ) . '</strong>',
+                                            '<a href="' . esc_url($url) . '">',
+                                            '<strong>' . intval($_suscription->get_id()) . '</strong>',
 											'</a>'
                                         );
 
-											echo wp_kses_post( WPInv_Subscriptions_List_Table::column_amount( $_suscription ) );
+											echo wp_kses_post(WPInv_Subscriptions_List_Table::column_amount($_suscription));
 									break;
 
 										}

includes/admin/meta-boxes/class-mb-invoice-notes.php

Spacing +11 added lines, -11 removed lines

@@ -1,38 +1,38 @@
 <?php
 // MUST have WordPress.
-if ( ! defined( 'WPINC' ) ) {
+if (!defined('WPINC')) {
     exit;
 }
 
 class WPInv_Meta_Box_Notes {
-    public static function output( $post ) {
+    public static function output($post) {
         global $post;
 
-        $notes = wpinv_get_invoice_notes( $post->ID );
+        $notes = wpinv_get_invoice_notes($post->ID);
 
         echo '<ul class="invoice_notes">';
 
-        if ( $notes ) {
-            foreach ( $notes as $note ) {
-                wpinv_get_invoice_note_line_item( $note );
+        if ($notes) {
+            foreach ($notes as $note) {
+                wpinv_get_invoice_note_line_item($note);
             }
 } else {
-            echo '<li>' . esc_html__( 'There are no notes yet.', 'invoicing' ) . '</li>';
+            echo '<li>' . esc_html__('There are no notes yet.', 'invoicing') . '</li>';
         }
 
         echo '</ul>';
         ?>
         <div class="add_note">
-            <h4><?php esc_html_e( 'Add note', 'invoicing' ); ?></h4>
+            <h4><?php esc_html_e('Add note', 'invoicing'); ?></h4>
             <p>
                 <textarea type="text" name="invoice_note" id="add_invoice_note" class="input-text" cols="20" rows="5"></textarea>
             </p>
             <p>
                 <select name="invoice_note_type" id="invoice_note_type" class="regular-text">
-                    <option value=""><?php esc_html_e( 'Private note', 'invoicing' ); ?></option>
-                    <option value="customer"><?php esc_html_e( 'Note to customer', 'invoicing' ); ?></option>
+                    <option value=""><?php esc_html_e('Private note', 'invoicing'); ?></option>
+                    <option value="customer"><?php esc_html_e('Note to customer', 'invoicing'); ?></option>
                 </select>
-                <a href="#" class="add_note button"><?php esc_html_e( 'Add', 'invoicing' ); ?></a> <span class="description"><?php esc_html_e( 'Add a note for your reference, or add a customer note (the user will be notified).', 'invoicing' ); ?></span>
+                <a href="#" class="add_note button"><?php esc_html_e('Add', 'invoicing'); ?></a> <span class="description"><?php esc_html_e('Add a note for your reference, or add a customer note (the user will be notified).', 'invoicing'); ?></span>
             </p>
         </div>
         <?php

includes/error-functions.php

Indentation +8 added lines, -8 removed lines

@@ -126,14 +126,14 @@
  */
 function getpaid_doing_it_wrong( $function, $message, $version ) {
 
-	$message .= ' Backtrace: ' . wp_debug_backtrace_summary();
-
-	if ( wp_doing_ajax() || defined( 'REST_REQUEST' ) ) {
-		do_action( 'doing_it_wrong_run', $function, $message, $version );
-		error_log( "{$function} was called incorrectly. {$message}. This message was added in version {$version}." );
-	} else {
-		_doing_it_wrong( esc_html( $function ), wp_kses_post( $message ), esc_html( $version ) );
-	}
+    $message .= ' Backtrace: ' . wp_debug_backtrace_summary();
+
+    if ( wp_doing_ajax() || defined( 'REST_REQUEST' ) ) {
+        do_action( 'doing_it_wrong_run', $function, $message, $version );
+        error_log( "{$function} was called incorrectly. {$message}. This message was added in version {$version}." );
+    } else {
+        _doing_it_wrong( esc_html( $function ), wp_kses_post( $message ), esc_html( $version ) );
+    }
 
 }
 

Spacing +49 added lines, -49 removed lines

@@ -6,7 +6,7 @@ 
  * @package Invoicing
  */
 
-defined( 'ABSPATH' ) || exit;
+defined('ABSPATH') || exit;
 
 /**
  * Returns the errors as html
@@ -15,35 +15,35 @@ 
  * @param bool $wrap whether or not to wrap the errors.
  * @since  1.0.19
  */
-function getpaid_get_errors_html( $clear = true, $wrap = true ) {
+function getpaid_get_errors_html($clear = true, $wrap = true) {
 
     $errors = '';
-    foreach ( wpinv_get_errors() as $id => $error ) {
-        $type     = 'error';
+    foreach (wpinv_get_errors() as $id => $error) {
+        $type = 'error';
 
-        if ( is_array( $error ) ) {
+        if (is_array($error)) {
             $type  = $error['type'];
             $error = $error['text'];
         }
 
-        if ( $wrap ) {
+        if ($wrap) {
 
             $errors .= aui()->alert(
                 array(
-                    'content' => wp_kses_post( $error ),
+                    'content' => wp_kses_post($error),
                     'type'    => $type,
                 )
             );
 
         } else {
 
-            $id      = esc_attr( $id );
-            $error   = wp_kses_post( $error );
+            $id      = esc_attr($id);
+            $error   = wp_kses_post($error);
             $errors .= "<div data-code='$id'>$error</div>";
         }
     }
 
-    if ( $clear ) {
+    if ($clear) {
         wpinv_clear_errors();
     }
 
@@ -55,7 +55,7 @@ 
  * Prints (then clears) all available errors.
  */
 function wpinv_print_errors() {
-    echo wp_kses_post( getpaid_get_errors_html() );
+    echo wp_kses_post(getpaid_get_errors_html());
 }
 
 /**
@@ -69,54 +69,54 @@ 
     $all_errors = array(
         'perm_cancel_subscription'   => array(
             'type' => 'error',
-            'text' => __( 'You do not have permission to cancel this subscription', 'invoicing' ),
+            'text' => __('You do not have permission to cancel this subscription', 'invoicing'),
         ),
         'cannot_cancel_subscription' => array(
             'type' => 'error',
-            'text' => __( 'This subscription cannot be cancelled as it is not active.', 'invoicing' ),
+            'text' => __('This subscription cannot be cancelled as it is not active.', 'invoicing'),
         ),
         'cancelled_subscription'     => array(
             'type' => 'success',
-            'text' => __( 'Subscription cancelled successfully.', 'invoicing' ),
+            'text' => __('Subscription cancelled successfully.', 'invoicing'),
         ),
         'address_updated'            => array(
             'type' => 'success',
-            'text' => __( 'Address updated successfully.', 'invoicing' ),
+            'text' => __('Address updated successfully.', 'invoicing'),
         ),
         'perm_delete_invoice'        => array(
             'type' => 'error',
-            'text' => __( 'You do not have permission to delete this invoice', 'invoicing' ),
+            'text' => __('You do not have permission to delete this invoice', 'invoicing'),
         ),
         'cannot_delete_invoice'      => array(
             'type' => 'error',
-            'text' => __( 'This invoice cannot be deleted as it has already been paid.', 'invoicing' ),
+            'text' => __('This invoice cannot be deleted as it has already been paid.', 'invoicing'),
         ),
         'deleted_invoice'            => array(
             'type' => 'success',
-            'text' => __( 'Invoice deleted successfully.', 'invoicing' ),
+            'text' => __('Invoice deleted successfully.', 'invoicing'),
         ),
         'card_declined'              => array(
             'type' => 'error',
-            'text' => __( 'Your card was declined.', 'invoicing' ),
+            'text' => __('Your card was declined.', 'invoicing'),
         ),
         'invalid_currency'           => array(
             'type' => 'error',
-            'text' => __( 'The chosen payment gateway does not support this currency.', 'invoicing' ),
+            'text' => __('The chosen payment gateway does not support this currency.', 'invoicing'),
         ),
     );
 
-    $errors = apply_filters( 'wpinv_errors', array() );
+    $errors = apply_filters('wpinv_errors', array());
 
-    if ( isset( $_GET['wpinv-notice'] ) && isset( $all_errors[ $_GET['wpinv-notice'] ] ) ) { // phpcs:ignore WordPress.Security.NonceVerification.Recommended
-        $errors[ $_GET['wpinv-notice'] ] = $all_errors[ $_GET['wpinv-notice'] ]; // phpcs:ignore WordPress.Security.NonceVerification.Recommended
+    if (isset($_GET['wpinv-notice']) && isset($all_errors[$_GET['wpinv-notice']])) { // phpcs:ignore WordPress.Security.NonceVerification.Recommended
+        $errors[$_GET['wpinv-notice']] = $all_errors[$_GET['wpinv-notice']]; // phpcs:ignore WordPress.Security.NonceVerification.Recommended
     }
 
-    if ( isset( $GLOBALS['wpinv_notice'] ) && isset( $all_errors[ $GLOBALS['wpinv_notice'] ] ) ) {
-        $errors[ $GLOBALS['wpinv_notice'] ] = $all_errors[ $GLOBALS['wpinv_notice'] ];
+    if (isset($GLOBALS['wpinv_notice']) && isset($all_errors[$GLOBALS['wpinv_notice']])) {
+        $errors[$GLOBALS['wpinv_notice']] = $all_errors[$GLOBALS['wpinv_notice']];
     }
 
-    if ( isset( $GLOBALS['wpinv_custom_notice'] ) ) {
-        $errors[ $GLOBALS['wpinv_custom_notice']['code'] ] = $GLOBALS['wpinv_custom_notice'];
+    if (isset($GLOBALS['wpinv_custom_notice'])) {
+        $errors[$GLOBALS['wpinv_custom_notice']['code']] = $GLOBALS['wpinv_custom_notice'];
     }
 
     return $errors;
@@ -129,9 +129,9 @@ 
  * @param string $error The error message.
  * @param string $type The error type.
  */
-function wpinv_set_error( $error_id, $message = '', $type = 'error' ) {
+function wpinv_set_error($error_id, $message = '', $type = 'error') {
 
-    if ( ! empty( $message ) ) {
+    if (!empty($message)) {
         $GLOBALS['wpinv_custom_notice'] = array(
             'code' => $error_id,
             'type' => $type,
@@ -147,7 +147,7 @@ 
  *
  */
 function wpinv_has_errors() {
-    return count( wpinv_get_errors() ) > 0;
+    return count(wpinv_get_errors()) > 0;
 }
 
 /**
@@ -155,7 +155,7 @@ 
  *
  */
 function wpinv_clear_errors() {
-    unset( $GLOBALS['wpinv_notice'] );
+    unset($GLOBALS['wpinv_notice']);
 }
 
 /**
@@ -163,7 +163,7 @@ 
  *
  */
 function wpinv_unset_error() {
-    unset( $GLOBALS['wpinv_notice'] );
+    unset($GLOBALS['wpinv_notice']);
 }
 
 /**
@@ -174,15 +174,15 @@ 
  * @param string $message Message to log.
  * @param string $version Version the message was added in.
  */
-function getpaid_doing_it_wrong( $function, $message, $version ) {
+function getpaid_doing_it_wrong($function, $message, $version) {
 
 	$message .= ' Backtrace: ' . wp_debug_backtrace_summary();
 
-	if ( wp_doing_ajax() || defined( 'REST_REQUEST' ) ) {
-		do_action( 'doing_it_wrong_run', $function, $message, $version );
-		error_log( "{$function} was called incorrectly. {$message}. This message was added in version {$version}." );
+	if (wp_doing_ajax() || defined('REST_REQUEST')) {
+		do_action('doing_it_wrong_run', $function, $message, $version);
+		error_log("{$function} was called incorrectly. {$message}. This message was added in version {$version}.");
 	} else {
-		_doing_it_wrong( esc_html( $function ), wp_kses_post( $message ), esc_html( $version ) );
+		_doing_it_wrong(esc_html($function), wp_kses_post($message), esc_html($version));
 	}
 
 }
@@ -196,41 +196,41 @@ 
  * @param string $line The line that contains the error.
  * @param bool $exit Whether or not to exit function execution.
  */
-function wpinv_error_log( $log, $title = '', $file = '', $line = '', $exit = false ) {
+function wpinv_error_log($log, $title = '', $file = '', $line = '', $exit = false) {
 
-    if ( true === apply_filters( 'wpinv_log_errors', true ) ) {
+    if (true === apply_filters('wpinv_log_errors', true)) {
 
         // Ensure the log is a scalar.
-        if ( ! is_scalar( $log ) ) {
-            $log = print_r( $log, true );
+        if (!is_scalar($log)) {
+            $log = print_r($log, true);
         }
 
         // Add title.
-        if ( ! empty( $title ) ) {
-            $log  = $title . ' ' . trim( $log );
+        if (!empty($title)) {
+            $log = $title . ' ' . trim($log);
         }
 
         // Add the file to the label.
-        if ( ! empty( $file ) ) {
+        if (!empty($file)) {
             $log .= ' in ' . $file;
         }
 
         // Add the line number to the label.
-        if ( ! empty( $line ) ) {
+        if (!empty($line)) {
             $log .= ' on line ' . $line;
         }
 
         // Log the message.
-        error_log( trim( $log ) );
+        error_log(trim($log));
 
         // ... and a backtrace.
-        if ( false !== $title && false !== $file ) {
-            error_log( 'Backtrace ' . wp_debug_backtrace_summary() );
+        if (false !== $title && false !== $file) {
+            error_log('Backtrace ' . wp_debug_backtrace_summary());
         }
 }
 
     // Maybe exit.
-    if ( $exit ) {
+    if ($exit) {
         exit;
     }
 

includes/reports/class-getpaid-reports.php

Indentation +100 added lines, -100 removed lines

@@ -12,49 +12,49 @@ 
  */
 class GetPaid_Reports {
 
-	/**
-	 * Class constructor.
-	 *
-	 */
-	public function __construct() {
-		add_action( 'admin_menu', array( $this, 'register_reports_page' ), 20 );
-		add_action( 'wpinv_reports_tab_reports', array( $this, 'display_reports_tab' ) );
-		add_action( 'wpinv_reports_tab_export', array( $this, 'display_exports_tab' ) );
-		add_action( 'getpaid_authenticated_admin_action_download_graph', array( $this, 'download_graph' ) );
-		add_action( 'getpaid_authenticated_admin_action_export_invoices', array( $this, 'export_invoices' ) );
-
-	}
-
-	/**
-	 * Registers the reports page.
-	 *
-	 */
-	public function register_reports_page() {
-
-		add_submenu_page(
+    /**
+     * Class constructor.
+     *
+     */
+    public function __construct() {
+        add_action( 'admin_menu', array( $this, 'register_reports_page' ), 20 );
+        add_action( 'wpinv_reports_tab_reports', array( $this, 'display_reports_tab' ) );
+        add_action( 'wpinv_reports_tab_export', array( $this, 'display_exports_tab' ) );
+        add_action( 'getpaid_authenticated_admin_action_download_graph', array( $this, 'download_graph' ) );
+        add_action( 'getpaid_authenticated_admin_action_export_invoices', array( $this, 'export_invoices' ) );
+
+    }
+
+    /**
+     * Registers the reports page.
+     *
+     */
+    public function register_reports_page() {
+
+        add_submenu_page(
             'wpinv',
             __( 'Reports', 'invoicing' ),
             __( 'Reports', 'invoicing' ),
             wpinv_get_capability(),
             'wpinv-reports',
             array( $this, 'display_reports_page' )
-		);
+        );
 
-	}
+    }
 
-	/**
-	 * Displays the reports page.
-	 *
-	 */
-	public function display_reports_page() {
+    /**
+     * Displays the reports page.
+     *
+     */
+    public function display_reports_page() {
 
-		// Prepare variables.
-		$tabs        = $this->get_tabs();
-		$current_tab = isset( $_GET['tab'] ) ? sanitize_text_field( $_GET['tab'] ) : 'reports';
-		$current_tab = array_key_exists( $current_tab, $tabs ) ? $current_tab : 'reports';
+        // Prepare variables.
+        $tabs        = $this->get_tabs();
+        $current_tab = isset( $_GET['tab'] ) ? sanitize_text_field( $_GET['tab'] ) : 'reports';
+        $current_tab = array_key_exists( $current_tab, $tabs ) ? $current_tab : 'reports';
 
-		// Display the current tab.
-		?>
+        // Display the current tab.
+        ?>
 
         <div class="wrap">
 
@@ -63,19 +63,19 @@ 
 			<nav class="nav-tab-wrapper">
 
 				<?php
-					foreach ( $tabs as $key => $label ) {
+                    foreach ( $tabs as $key => $label ) {
 
-					$key   = sanitize_key( $key );
-					$label = esc_html( $label );
-					$class = $key == $current_tab ? 'nav-tab nav-tab-active' : 'nav-tab';
-					$url   = esc_url(
+                    $key   = sanitize_key( $key );
+                    $label = esc_html( $label );
+                    $class = $key == $current_tab ? 'nav-tab nav-tab-active' : 'nav-tab';
+                    $url   = esc_url(
                         add_query_arg( 'tab', $key, admin_url( 'admin.php?page=wpinv-reports' ) )
                     );
 
-				echo wp_kses_post( "\n\t\t\t<a href='$url' class='" . $class . "'>$label</a>" );
+                echo wp_kses_post( "\n\t\t\t<a href='$url' class='" . $class . "'>$label</a>" );
 
-					}
-				?>
+                    }
+                ?>
 
 			</nav>
 
@@ -86,82 +86,82 @@ 
         </div>
 		<?php
 
-			// Wordfence loads an unsupported version of chart js on our page.
-			wp_deregister_style( 'chart-js' );
-			wp_deregister_script( 'chart-js' );
-			wp_enqueue_script( 'chart-js', WPINV_PLUGIN_URL . 'assets/js/chart.min.js', array( 'jquery' ), '3.7.1', true );
+            // Wordfence loads an unsupported version of chart js on our page.
+            wp_deregister_style( 'chart-js' );
+            wp_deregister_script( 'chart-js' );
+            wp_enqueue_script( 'chart-js', WPINV_PLUGIN_URL . 'assets/js/chart.min.js', array( 'jquery' ), '3.7.1', true );
 
-	}
+    }
 
-	/**
-	 * Retrieves reports page tabs.
-	 *
-	 * @return array
-	 */
-	public function get_tabs() {
+    /**
+     * Retrieves reports page tabs.
+     *
+     * @return array
+     */
+    public function get_tabs() {
 
-		$tabs = array(
-			'reports' => __( 'Reports', 'invoicing' ),
-			'export'  => __( 'Export', 'invoicing' ),
-		);
+        $tabs = array(
+            'reports' => __( 'Reports', 'invoicing' ),
+            'export'  => __( 'Export', 'invoicing' ),
+        );
 
-		return apply_filters( 'getpaid_report_tabs', $tabs );
-	}
+        return apply_filters( 'getpaid_report_tabs', $tabs );
+    }
 
-	/**
-	 * Displays the reports tab.
-	 *
-	 */
-	public function display_reports_tab() {
+    /**
+     * Displays the reports tab.
+     *
+     */
+    public function display_reports_tab() {
 
-		$reports = new GetPaid_Reports_Report();
-		$reports->display();
+        $reports = new GetPaid_Reports_Report();
+        $reports->display();
 
-	}
+    }
 
-	/**
-	 * Displays the exports tab.
-	 *
-	 */
-	public function display_exports_tab() {
+    /**
+     * Displays the exports tab.
+     *
+     */
+    public function display_exports_tab() {
 
-		$exports = new GetPaid_Reports_Export();
-		$exports->display();
+        $exports = new GetPaid_Reports_Export();
+        $exports->display();
 
-	}
+    }
 
-	/**
-	 * Donwnloads a graph.
-	 *
-	 * @param array $args
-	 */
-	public function download_graph( $args ) {
+    /**
+     * Donwnloads a graph.
+     *
+     * @param array $args
+     */
+    public function download_graph( $args ) {
 
-		if ( ! empty( $args['graph'] ) ) {
-			$downloader = new GetPaid_Graph_Downloader();
-			$downloader->download( $args['graph'] );
-		}
+        if ( ! empty( $args['graph'] ) ) {
+            $downloader = new GetPaid_Graph_Downloader();
+            $downloader->download( $args['graph'] );
+        }
 
-	}
+    }
 
-	/**
-	 * Exports invoices.
-	 *
-	 * @param array $args
-	 */
-	public function export_invoices( $args ) {
+    /**
+     * Exports invoices.
+     *
+     * @param array $args
+     */
+    public function export_invoices( $args ) {
 
-		if ( ! empty( $args['post_type'] ) ) {
+        if ( ! empty( $args['post_type'] ) ) {
 
-			if ( 'subscriptions' === $args['post_type'] ) {
-				$downloader = new GetPaid_Subscription_Exporter();
-			} else {
-				$downloader = new GetPaid_Invoice_Exporter();
-			}
+            if ( 'subscriptions' === $args['post_type'] ) {
+                $downloader = new GetPaid_Subscription_Exporter();
+            } else {
+                $downloader = new GetPaid_Invoice_Exporter();
+            }
 
-			$downloader->export( $args['post_type'], $args );
-		}
+            $downloader->export( $args['post_type'], $args );
+        }
 
-	}
+    }
 
 }

Spacing +32 added lines, -32 removed lines

@@ -5,7 +5,7 @@ 
  *
  */
 
-defined( 'ABSPATH' ) || exit;
+defined('ABSPATH') || exit;
 
 /**
  * GetPaid_Reports Class.
@@ -17,11 +17,11 @@ 
 	 *
 	 */
 	public function __construct() {
-		add_action( 'admin_menu', array( $this, 'register_reports_page' ), 20 );
-		add_action( 'wpinv_reports_tab_reports', array( $this, 'display_reports_tab' ) );
-		add_action( 'wpinv_reports_tab_export', array( $this, 'display_exports_tab' ) );
-		add_action( 'getpaid_authenticated_admin_action_download_graph', array( $this, 'download_graph' ) );
-		add_action( 'getpaid_authenticated_admin_action_export_invoices', array( $this, 'export_invoices' ) );
+		add_action('admin_menu', array($this, 'register_reports_page'), 20);
+		add_action('wpinv_reports_tab_reports', array($this, 'display_reports_tab'));
+		add_action('wpinv_reports_tab_export', array($this, 'display_exports_tab'));
+		add_action('getpaid_authenticated_admin_action_download_graph', array($this, 'download_graph'));
+		add_action('getpaid_authenticated_admin_action_export_invoices', array($this, 'export_invoices'));
 
 	}
 
@@ -33,11 +33,11 @@ 
 
 		add_submenu_page(
             'wpinv',
-            __( 'Reports', 'invoicing' ),
-            __( 'Reports', 'invoicing' ),
+            __('Reports', 'invoicing'),
+            __('Reports', 'invoicing'),
             wpinv_get_capability(),
             'wpinv-reports',
-            array( $this, 'display_reports_page' )
+            array($this, 'display_reports_page')
 		);
 
 	}
@@ -50,46 +50,46 @@ 
 
 		// Prepare variables.
 		$tabs        = $this->get_tabs();
-		$current_tab = isset( $_GET['tab'] ) ? sanitize_text_field( $_GET['tab'] ) : 'reports';
-		$current_tab = array_key_exists( $current_tab, $tabs ) ? $current_tab : 'reports';
+		$current_tab = isset($_GET['tab']) ? sanitize_text_field($_GET['tab']) : 'reports';
+		$current_tab = array_key_exists($current_tab, $tabs) ? $current_tab : 'reports';
 
 		// Display the current tab.
 		?>
 
         <div class="wrap">
 
-			<h1><?php echo esc_html( $tabs[ $current_tab ] ); ?></h1>
+			<h1><?php echo esc_html($tabs[$current_tab]); ?></h1>
 
 			<nav class="nav-tab-wrapper">
 
 				<?php
-					foreach ( $tabs as $key => $label ) {
+					foreach ($tabs as $key => $label) {
 
-					$key   = sanitize_key( $key );
-					$label = esc_html( $label );
+					$key   = sanitize_key($key);
+					$label = esc_html($label);
 					$class = $key == $current_tab ? 'nav-tab nav-tab-active' : 'nav-tab';
 					$url   = esc_url(
-                        add_query_arg( 'tab', $key, admin_url( 'admin.php?page=wpinv-reports' ) )
+                        add_query_arg('tab', $key, admin_url('admin.php?page=wpinv-reports'))
                     );
 
-				echo wp_kses_post( "\n\t\t\t<a href='$url' class='" . $class . "'>$label</a>" );
+				echo wp_kses_post("\n\t\t\t<a href='$url' class='" . $class . "'>$label</a>");
 
 					}
 				?>
 
 			</nav>
 
-			<div class="bsui <?php echo esc_attr( $current_tab ); ?>">
-				<?php do_action( "wpinv_reports_tab_{$current_tab}" ); ?>
+			<div class="bsui <?php echo esc_attr($current_tab); ?>">
+				<?php do_action("wpinv_reports_tab_{$current_tab}"); ?>
 			</div>
 
         </div>
 		<?php
 
 			// Wordfence loads an unsupported version of chart js on our page.
-			wp_deregister_style( 'chart-js' );
-			wp_deregister_script( 'chart-js' );
-			wp_enqueue_script( 'chart-js', WPINV_PLUGIN_URL . 'assets/js/chart.min.js', array( 'jquery' ), '3.7.1', true );
+			wp_deregister_style('chart-js');
+			wp_deregister_script('chart-js');
+			wp_enqueue_script('chart-js', WPINV_PLUGIN_URL . 'assets/js/chart.min.js', array('jquery'), '3.7.1', true);
 
 	}
 
@@ -101,11 +101,11 @@ 
 	public function get_tabs() {
 
 		$tabs = array(
-			'reports' => __( 'Reports', 'invoicing' ),
-			'export'  => __( 'Export', 'invoicing' ),
+			'reports' => __('Reports', 'invoicing'),
+			'export'  => __('Export', 'invoicing'),
 		);
 
-		return apply_filters( 'getpaid_report_tabs', $tabs );
+		return apply_filters('getpaid_report_tabs', $tabs);
 	}
 
 	/**
@@ -135,11 +135,11 @@ 
 	 *
 	 * @param array $args
 	 */
-	public function download_graph( $args ) {
+	public function download_graph($args) {
 
-		if ( ! empty( $args['graph'] ) ) {
+		if (!empty($args['graph'])) {
 			$downloader = new GetPaid_Graph_Downloader();
-			$downloader->download( $args['graph'] );
+			$downloader->download($args['graph']);
 		}
 
 	}
@@ -149,17 +149,17 @@ 
 	 *
 	 * @param array $args
 	 */
-	public function export_invoices( $args ) {
+	public function export_invoices($args) {
 
-		if ( ! empty( $args['post_type'] ) ) {
+		if (!empty($args['post_type'])) {
 
-			if ( 'subscriptions' === $args['post_type'] ) {
+			if ('subscriptions' === $args['post_type']) {
 				$downloader = new GetPaid_Subscription_Exporter();
 			} else {
 				$downloader = new GetPaid_Invoice_Exporter();
 			}
 
-			$downloader->export( $args['post_type'], $args );
+			$downloader->export($args['post_type'], $args);
 		}
 
 	}

includes/reports/class-getpaid-graph-downloader.php

Indentation +202 added lines, -202 removed lines

@@ -12,218 +12,218 @@
  */
 class GetPaid_Graph_Downloader {
 
-	/**
-	 * @var GetPaid_Reports_Report
-	 */
-	public $handler;
-
-	/**
-	 * Class constructor.
-	 *
-	 */
-	public function __construct() {
-		$this->handler = new GetPaid_Reports_Report();
-	}
-
-	/**
-	 * Prepares the datastore handler.
-	 *
-	 * @return GetPaid_Reports_Report_Items|GetPaid_Reports_Report_Gateways|GetPaid_Reports_Report_Discounts
-	 */
-	public function prepare_handler( $graph ) {
-
-		if ( empty( $this->handler->views[ $graph ] ) ) {
-			wp_die( esc_html__( 'Invalid Graph', 'invoicing' ), 400 );
-		}
-
-		return new $this->handler->views[ $graph ]['class']();
-
-	}
-
-	/**
-	 * Prepares the output stream.
-	 *
-	 * @return resource
-	 */
-	public function prepare_output() {
-
-		$output  = fopen( 'php://output', 'w' );
-
-		if ( false === $output ) {
-			wp_die( esc_html__( 'Unsupported server', 'invoicing' ), 500 );
-		}
-
-		return $output;
-	}
-
-	/**
-	 * Prepares the file type.
-	 *
-	 * @return string
-	 */
-	public function prepare_file_type( $graph ) {
-
-		$file_type = empty( $_REQUEST['file_type'] ) ? 'csv' : sanitize_text_field( $_REQUEST['file_type'] );
-		$file_name = wpinv_sanitize_key( "getpaid-$graph-" . current_time( 'Y-m-d' ) );
-
-		header( "Content-Type:application/$file_type" );
-		header( "Content-Disposition:attachment;filename=$file_name.$file_type" );
-
-		return $file_type;
-	}
-
-	/**
-	 * Handles the actual download.
-	 *
-	 */
-	public function download( $graph ) {
-		global $wpdb;
-
-		$handler   = $this->prepare_handler( $graph );
-		$stream    = $this->prepare_output();
-		$stats     = $wpdb->get_results( $handler->get_sql( $handler->get_range() ) );
-		$headers   = array( $handler->field, 'total', 'total_raw' );
-		$file_type = $this->prepare_file_type( $graph );
-
-		if ( 'csv' == $file_type ) {
-			$this->download_csv( $stats, $stream, $headers );
-		} elseif ( 'xml' == $file_type ) {
-			$this->download_xml( $stats, $stream, $headers );
-		} else {
-			$this->download_json( $stats, $stream, $headers );
-		}
-
-		fclose( $stream );
-		exit;
-	}
-
-	/**
-	 * Downloads graph as csv
-	 *
-	 * @param array $stats The stats being downloaded.
-	 * @param resource $stream The stream to output to.
-	 * @param array $headers The fields to stream.
-	 * @since       1.0.19
-	 */
-	public function download_csv( $stats, $stream, $headers ) {
-
-		// Output the csv column headers.
-		fputcsv( $stream, $headers );
-
-		// Loop through
-		foreach ( $stats as $stat ) {
-			$row  = array_values( $this->prepare_row( $stat, $headers ) );
-			$row  = array_map( 'maybe_serialize', $row );
-			fputcsv( $stream, $row );
-		}
-
-	}
-
-	/**
-	 * Downloads graph as json
-	 *
-	 * @param array $stats The stats being downloaded.
-	 * @param resource $stream The stream to output to.
-	 * @param array $headers The fields to stream.
-	 * @since       1.0.19
-	 */
-	public function download_json( $stats, $stream, $headers ) {
-
-		$prepared = array();
-
-		// Loop through
-		foreach ( $stats as $stat ) {
-			$prepared[] = $this->prepare_row( $stat, $headers );
-		}
-
-		fwrite( $stream, wp_json_encode( $prepared ) );
-
-	}
-
-	/**
-	 * Downloads graph as xml
-	 *
-	 * @param array $stats The stats being downloaded.
-	 * @param resource $stream The stream to output to.
-	 * @param array $headers The fields to stream.
-	 * @since       1.0.19
-	 */
-	public function download_xml( $stats, $stream, $headers ) {
-
-		$prepared = array();
-
-		// Loop through
-		foreach ( $stats as $stat ) {
-			$prepared[] = $this->prepare_row( $stat, $headers );
-		}
-
-		$xml = new SimpleXMLElement( '<?xml version="1.0"?><data></data>' );
-		$this->convert_array_xml( $prepared, $xml );
-
-		fwrite( $stream, $xml->asXML() );
-
-	}
-
-	/**
-	 * Converts stats array to xml
-	 *
-	 * @access      public
-	 * @since      1.0.19
-	 */
-	public function convert_array_xml( $data, $xml ) {
-
-		// Loop through
-		foreach ( $data as $key => $value ) {
-
-			$key = preg_replace( '/[^A-Za-z0-9_\-]/', '', $key );
-
-			if ( is_array( $value ) ) {
-
-				if ( is_numeric( $key ) ) {
-					$key = 'item' . $key; //dealing with <0/>..<n/> issues
-				}
-
-				$subnode = $xml->addChild( $key );
-				$this->convert_array_xml( $value, $subnode );
-
-			} else {
-				$xml->addChild( $key, htmlspecialchars( $value ) );
-			}
+    /**
+     * @var GetPaid_Reports_Report
+     */
+    public $handler;
+
+    /**
+     * Class constructor.
+     *
+     */
+    public function __construct() {
+        $this->handler = new GetPaid_Reports_Report();
+    }
+
+    /**
+     * Prepares the datastore handler.
+     *
+     * @return GetPaid_Reports_Report_Items|GetPaid_Reports_Report_Gateways|GetPaid_Reports_Report_Discounts
+     */
+    public function prepare_handler( $graph ) {
+
+        if ( empty( $this->handler->views[ $graph ] ) ) {
+            wp_die( esc_html__( 'Invalid Graph', 'invoicing' ), 400 );
+        }
+
+        return new $this->handler->views[ $graph ]['class']();
+
+    }
+
+    /**
+     * Prepares the output stream.
+     *
+     * @return resource
+     */
+    public function prepare_output() {
+
+        $output  = fopen( 'php://output', 'w' );
+
+        if ( false === $output ) {
+            wp_die( esc_html__( 'Unsupported server', 'invoicing' ), 500 );
+        }
+
+        return $output;
+    }
+
+    /**
+     * Prepares the file type.
+     *
+     * @return string
+     */
+    public function prepare_file_type( $graph ) {
+
+        $file_type = empty( $_REQUEST['file_type'] ) ? 'csv' : sanitize_text_field( $_REQUEST['file_type'] );
+        $file_name = wpinv_sanitize_key( "getpaid-$graph-" . current_time( 'Y-m-d' ) );
+
+        header( "Content-Type:application/$file_type" );
+        header( "Content-Disposition:attachment;filename=$file_name.$file_type" );
+
+        return $file_type;
+    }
+
+    /**
+     * Handles the actual download.
+     *
+     */
+    public function download( $graph ) {
+        global $wpdb;
+
+        $handler   = $this->prepare_handler( $graph );
+        $stream    = $this->prepare_output();
+        $stats     = $wpdb->get_results( $handler->get_sql( $handler->get_range() ) );
+        $headers   = array( $handler->field, 'total', 'total_raw' );
+        $file_type = $this->prepare_file_type( $graph );
+
+        if ( 'csv' == $file_type ) {
+            $this->download_csv( $stats, $stream, $headers );
+        } elseif ( 'xml' == $file_type ) {
+            $this->download_xml( $stats, $stream, $headers );
+        } else {
+            $this->download_json( $stats, $stream, $headers );
+        }
+
+        fclose( $stream );
+        exit;
+    }
+
+    /**
+     * Downloads graph as csv
+     *
+     * @param array $stats The stats being downloaded.
+     * @param resource $stream The stream to output to.
+     * @param array $headers The fields to stream.
+     * @since       1.0.19
+     */
+    public function download_csv( $stats, $stream, $headers ) {
+
+        // Output the csv column headers.
+        fputcsv( $stream, $headers );
+
+        // Loop through
+        foreach ( $stats as $stat ) {
+            $row  = array_values( $this->prepare_row( $stat, $headers ) );
+            $row  = array_map( 'maybe_serialize', $row );
+            fputcsv( $stream, $row );
+        }
+
+    }
+
+    /**
+     * Downloads graph as json
+     *
+     * @param array $stats The stats being downloaded.
+     * @param resource $stream The stream to output to.
+     * @param array $headers The fields to stream.
+     * @since       1.0.19
+     */
+    public function download_json( $stats, $stream, $headers ) {
+
+        $prepared = array();
+
+        // Loop through
+        foreach ( $stats as $stat ) {
+            $prepared[] = $this->prepare_row( $stat, $headers );
+        }
+
+        fwrite( $stream, wp_json_encode( $prepared ) );
+
+    }
+
+    /**
+     * Downloads graph as xml
+     *
+     * @param array $stats The stats being downloaded.
+     * @param resource $stream The stream to output to.
+     * @param array $headers The fields to stream.
+     * @since       1.0.19
+     */
+    public function download_xml( $stats, $stream, $headers ) {
+
+        $prepared = array();
+
+        // Loop through
+        foreach ( $stats as $stat ) {
+            $prepared[] = $this->prepare_row( $stat, $headers );
+        }
+
+        $xml = new SimpleXMLElement( '<?xml version="1.0"?><data></data>' );
+        $this->convert_array_xml( $prepared, $xml );
+
+        fwrite( $stream, $xml->asXML() );
+
+    }
+
+    /**
+     * Converts stats array to xml
+     *
+     * @access      public
+     * @since      1.0.19
+     */
+    public function convert_array_xml( $data, $xml ) {
+
+        // Loop through
+        foreach ( $data as $key => $value ) {
+
+            $key = preg_replace( '/[^A-Za-z0-9_\-]/', '', $key );
+
+            if ( is_array( $value ) ) {
+
+                if ( is_numeric( $key ) ) {
+                    $key = 'item' . $key; //dealing with <0/>..<n/> issues
+                }
+
+                $subnode = $xml->addChild( $key );
+                $this->convert_array_xml( $value, $subnode );
+
+            } else {
+                $xml->addChild( $key, htmlspecialchars( $value ) );
+            }
 }
 
-	}
+    }
 
-	/**
-	 * Prepares a single row for download.
-	 *
-	 * @param stdClass|array $row The row to prepare..
-	 * @param array $fields The fields to stream.
-	 * @since       1.0.19
-	 * @return array
-	 */
-	public function prepare_row( $row, $fields ) {
+    /**
+     * Prepares a single row for download.
+     *
+     * @param stdClass|array $row The row to prepare..
+     * @param array $fields The fields to stream.
+     * @since       1.0.19
+     * @return array
+     */
+    public function prepare_row( $row, $fields ) {
 
-		$prepared = array();
-		$row      = (array) $row;
+        $prepared = array();
+        $row      = (array) $row;
 
-		foreach ( $fields as $field ) {
+        foreach ( $fields as $field ) {
 
-			if ( $field === 'total' ) {
-				$prepared[ $field ] = html_entity_decode( strip_tags( wpinv_price( $row['total'] ) ), ENT_QUOTES );
-				continue;
-			}
+            if ( $field === 'total' ) {
+                $prepared[ $field ] = html_entity_decode( strip_tags( wpinv_price( $row['total'] ) ), ENT_QUOTES );
+                continue;
+            }
 
-			if ( $field === 'total_raw' ) {
-				$prepared[ $field ] = wpinv_round_amount( wpinv_sanitize_amount( $row['total'] ) );
-				continue;
-			}
+            if ( $field === 'total_raw' ) {
+                $prepared[ $field ] = wpinv_round_amount( wpinv_sanitize_amount( $row['total'] ) );
+                continue;
+            }
 
-			$prepared[ $field ] = strip_tags( $row[ $field ] );
+            $prepared[ $field ] = strip_tags( $row[ $field ] );
 
-		}
+        }
 
-		return $prepared;
-	}
+        return $prepared;
+    }
 
 
 }

Spacing +56 added lines, -56 removed lines

@@ -5,7 +5,7 @@ 
  *
  */
 
-defined( 'ABSPATH' ) || exit;
+defined('ABSPATH') || exit;
 
 /**
  * GetPaid_Graph_Downloader Class.
@@ -30,13 +30,13 @@ 
 	 *
 	 * @return GetPaid_Reports_Report_Items|GetPaid_Reports_Report_Gateways|GetPaid_Reports_Report_Discounts
 	 */
-	public function prepare_handler( $graph ) {
+	public function prepare_handler($graph) {
 
-		if ( empty( $this->handler->views[ $graph ] ) ) {
-			wp_die( esc_html__( 'Invalid Graph', 'invoicing' ), 400 );
+		if (empty($this->handler->views[$graph])) {
+			wp_die(esc_html__('Invalid Graph', 'invoicing'), 400);
 		}
 
-		return new $this->handler->views[ $graph ]['class']();
+		return new $this->handler->views[$graph]['class']();
 
 	}
 
@@ -47,10 +47,10 @@ 
 	 */
 	public function prepare_output() {
 
-		$output  = fopen( 'php://output', 'w' );
+		$output = fopen('php://output', 'w');
 
-		if ( false === $output ) {
-			wp_die( esc_html__( 'Unsupported server', 'invoicing' ), 500 );
+		if (false === $output) {
+			wp_die(esc_html__('Unsupported server', 'invoicing'), 500);
 		}
 
 		return $output;
@@ -61,13 +61,13 @@ 
 	 *
 	 * @return string
 	 */
-	public function prepare_file_type( $graph ) {
+	public function prepare_file_type($graph) {
 
-		$file_type = empty( $_REQUEST['file_type'] ) ? 'csv' : sanitize_text_field( $_REQUEST['file_type'] );
-		$file_name = wpinv_sanitize_key( "getpaid-$graph-" . current_time( 'Y-m-d' ) );
+		$file_type = empty($_REQUEST['file_type']) ? 'csv' : sanitize_text_field($_REQUEST['file_type']);
+		$file_name = wpinv_sanitize_key("getpaid-$graph-" . current_time('Y-m-d'));
 
-		header( "Content-Type:application/$file_type" );
-		header( "Content-Disposition:attachment;filename=$file_name.$file_type" );
+		header("Content-Type:application/$file_type");
+		header("Content-Disposition:attachment;filename=$file_name.$file_type");
 
 		return $file_type;
 	}
@@ -76,24 +76,24 @@ 
 	 * Handles the actual download.
 	 *
 	 */
-	public function download( $graph ) {
+	public function download($graph) {
 		global $wpdb;
 
-		$handler   = $this->prepare_handler( $graph );
+		$handler   = $this->prepare_handler($graph);
 		$stream    = $this->prepare_output();
-		$stats     = $wpdb->get_results( $handler->get_sql( $handler->get_range() ) );
-		$headers   = array( $handler->field, 'total', 'total_raw' );
-		$file_type = $this->prepare_file_type( $graph );
-
-		if ( 'csv' == $file_type ) {
-			$this->download_csv( $stats, $stream, $headers );
-		} elseif ( 'xml' == $file_type ) {
-			$this->download_xml( $stats, $stream, $headers );
+		$stats     = $wpdb->get_results($handler->get_sql($handler->get_range()));
+		$headers   = array($handler->field, 'total', 'total_raw');
+		$file_type = $this->prepare_file_type($graph);
+
+		if ('csv' == $file_type) {
+			$this->download_csv($stats, $stream, $headers);
+		} elseif ('xml' == $file_type) {
+			$this->download_xml($stats, $stream, $headers);
 		} else {
-			$this->download_json( $stats, $stream, $headers );
+			$this->download_json($stats, $stream, $headers);
 		}
 
-		fclose( $stream );
+		fclose($stream);
 		exit;
 	}
 
@@ -105,16 +105,16 @@ 
 	 * @param array $headers The fields to stream.
 	 * @since       1.0.19
 	 */
-	public function download_csv( $stats, $stream, $headers ) {
+	public function download_csv($stats, $stream, $headers) {
 
 		// Output the csv column headers.
-		fputcsv( $stream, $headers );
+		fputcsv($stream, $headers);
 
 		// Loop through
-		foreach ( $stats as $stat ) {
-			$row  = array_values( $this->prepare_row( $stat, $headers ) );
-			$row  = array_map( 'maybe_serialize', $row );
-			fputcsv( $stream, $row );
+		foreach ($stats as $stat) {
+			$row  = array_values($this->prepare_row($stat, $headers));
+			$row  = array_map('maybe_serialize', $row);
+			fputcsv($stream, $row);
 		}
 
 	}
@@ -127,16 +127,16 @@ 
 	 * @param array $headers The fields to stream.
 	 * @since       1.0.19
 	 */
-	public function download_json( $stats, $stream, $headers ) {
+	public function download_json($stats, $stream, $headers) {
 
 		$prepared = array();
 
 		// Loop through
-		foreach ( $stats as $stat ) {
-			$prepared[] = $this->prepare_row( $stat, $headers );
+		foreach ($stats as $stat) {
+			$prepared[] = $this->prepare_row($stat, $headers);
 		}
 
-		fwrite( $stream, wp_json_encode( $prepared ) );
+		fwrite($stream, wp_json_encode($prepared));
 
 	}
 
@@ -148,19 +148,19 @@ 
 	 * @param array $headers The fields to stream.
 	 * @since       1.0.19
 	 */
-	public function download_xml( $stats, $stream, $headers ) {
+	public function download_xml($stats, $stream, $headers) {
 
 		$prepared = array();
 
 		// Loop through
-		foreach ( $stats as $stat ) {
-			$prepared[] = $this->prepare_row( $stat, $headers );
+		foreach ($stats as $stat) {
+			$prepared[] = $this->prepare_row($stat, $headers);
 		}
 
-		$xml = new SimpleXMLElement( '<?xml version="1.0"?><data></data>' );
-		$this->convert_array_xml( $prepared, $xml );
+		$xml = new SimpleXMLElement('<?xml version="1.0"?><data></data>');
+		$this->convert_array_xml($prepared, $xml);
 
-		fwrite( $stream, $xml->asXML() );
+		fwrite($stream, $xml->asXML());
 
 	}
 
@@ -170,24 +170,24 @@ 
 	 * @access      public
 	 * @since      1.0.19
 	 */
-	public function convert_array_xml( $data, $xml ) {
+	public function convert_array_xml($data, $xml) {
 
 		// Loop through
-		foreach ( $data as $key => $value ) {
+		foreach ($data as $key => $value) {
 
-			$key = preg_replace( '/[^A-Za-z0-9_\-]/', '', $key );
+			$key = preg_replace('/[^A-Za-z0-9_\-]/', '', $key);
 
-			if ( is_array( $value ) ) {
+			if (is_array($value)) {
 
-				if ( is_numeric( $key ) ) {
+				if (is_numeric($key)) {
 					$key = 'item' . $key; //dealing with <0/>..<n/> issues
 				}
 
-				$subnode = $xml->addChild( $key );
-				$this->convert_array_xml( $value, $subnode );
+				$subnode = $xml->addChild($key);
+				$this->convert_array_xml($value, $subnode);
 
 			} else {
-				$xml->addChild( $key, htmlspecialchars( $value ) );
+				$xml->addChild($key, htmlspecialchars($value));
 			}
 }
 
@@ -201,24 +201,24 @@ 
 	 * @since       1.0.19
 	 * @return array
 	 */
-	public function prepare_row( $row, $fields ) {
+	public function prepare_row($row, $fields) {
 
 		$prepared = array();
 		$row      = (array) $row;
 
-		foreach ( $fields as $field ) {
+		foreach ($fields as $field) {
 
-			if ( $field === 'total' ) {
-				$prepared[ $field ] = html_entity_decode( strip_tags( wpinv_price( $row['total'] ) ), ENT_QUOTES );
+			if ($field === 'total') {
+				$prepared[$field] = html_entity_decode(strip_tags(wpinv_price($row['total'])), ENT_QUOTES);
 				continue;
 			}
 
-			if ( $field === 'total_raw' ) {
-				$prepared[ $field ] = wpinv_round_amount( wpinv_sanitize_amount( $row['total'] ) );
+			if ($field === 'total_raw') {
+				$prepared[$field] = wpinv_round_amount(wpinv_sanitize_amount($row['total']));
 				continue;
 			}
 
-			$prepared[ $field ] = strip_tags( $row[ $field ] );
+			$prepared[$field] = strip_tags($row[$field]);
 
 		}
 

includes/payments/class-getpaid-payment-form-submission-items.php

Indentation +73 added lines, -73 removed lines

@@ -12,38 +12,38 @@ 
  */
 class GetPaid_Payment_Form_Submission_Items {
 
-	/**
-	 * Submission items.
-	 * @var GetPaid_Form_Item[]
-	 */
-	public $items = array();
+    /**
+     * Submission items.
+     * @var GetPaid_Form_Item[]
+     */
+    public $items = array();
 
     /**
-	 * Class constructor
-	 *
-	 * @param GetPaid_Payment_Form_Submission $submission
-	 */
-	public function __construct( $submission ) {
-
-		$data         = $submission->get_data();
-		$payment_form = $submission->get_payment_form();
-
-		// Prepare the selected items.
-		$selected_items = array();
-		if ( ! empty( $data['getpaid-items'] ) ) {
-			$selected_items = wpinv_clean( $data['getpaid-items'] );
-		}
-
-		// (Maybe) set form items.
-		if ( isset( $data['getpaid-form-items'] ) ) {
-
-			// Confirm items key.
-			$form_items = wpinv_clean( $data['getpaid-form-items'] );
-			if ( ! isset( $data['getpaid-form-items-key'] ) || $data['getpaid-form-items-key'] !== md5( NONCE_KEY . AUTH_KEY . $form_items ) ) {
-				throw new Exception( __( 'We could not validate the form items. Please reload the page and try again.', 'invoicing' ) );
-			}
-
-			$items    = array();
+     * Class constructor
+     *
+     * @param GetPaid_Payment_Form_Submission $submission
+     */
+    public function __construct( $submission ) {
+
+        $data         = $submission->get_data();
+        $payment_form = $submission->get_payment_form();
+
+        // Prepare the selected items.
+        $selected_items = array();
+        if ( ! empty( $data['getpaid-items'] ) ) {
+            $selected_items = wpinv_clean( $data['getpaid-items'] );
+        }
+
+        // (Maybe) set form items.
+        if ( isset( $data['getpaid-form-items'] ) ) {
+
+            // Confirm items key.
+            $form_items = wpinv_clean( $data['getpaid-form-items'] );
+            if ( ! isset( $data['getpaid-form-items-key'] ) || $data['getpaid-form-items-key'] !== md5( NONCE_KEY . AUTH_KEY . $form_items ) ) {
+                throw new Exception( __( 'We could not validate the form items. Please reload the page and try again.', 'invoicing' ) );
+            }
+
+            $items    = array();
             $item_ids = array();
 
             foreach ( getpaid_convert_items_to_array( $form_items ) as $item_id => $qty ) {
@@ -69,61 +69,61 @@ 
                         $items[]    = $item;
                     }
                 }
-			}
+            }
 
             $payment_form->set_items( $items );
 
-		}
-
-		// Process each individual item.
-		foreach ( $payment_form->get_items() as $item ) {
-			$this->process_item( $item, $selected_items, $submission );
-		}
-
-	}
+        }
 
-	/**
-	 * Process a single item.
-	 *
-	 * @param GetPaid_Form_Item $item
-	 * @param array $selected_items
-	 * @param GetPaid_Payment_Form_Submission $submission
-	 */
-	public function process_item( $item, $selected_items, $submission ) {
+        // Process each individual item.
+        foreach ( $payment_form->get_items() as $item ) {
+            $this->process_item( $item, $selected_items, $submission );
+        }
 
-		// Abort if this is an optional item and it has not been selected.
-		if ( ! $item->is_required() && ! isset( $selected_items[ $item->get_id() ] ) ) {
-			return;
-		}
+    }
 
-		// (maybe) let customers change the quantities and prices.
-		if ( isset( $selected_items[ $item->get_id() ] ) ) {
-
-			// Maybe change the quantities.
-			if ( $item->allows_quantities() ) {
-				$item->set_quantity( (float) $selected_items[ $item->get_id() ]['quantity'] );
-			}
+    /**
+     * Process a single item.
+     *
+     * @param GetPaid_Form_Item $item
+     * @param array $selected_items
+     * @param GetPaid_Payment_Form_Submission $submission
+     */
+    public function process_item( $item, $selected_items, $submission ) {
+
+        // Abort if this is an optional item and it has not been selected.
+        if ( ! $item->is_required() && ! isset( $selected_items[ $item->get_id() ] ) ) {
+            return;
+        }
+
+        // (maybe) let customers change the quantities and prices.
+        if ( isset( $selected_items[ $item->get_id() ] ) ) {
+
+            // Maybe change the quantities.
+            if ( $item->allows_quantities() ) {
+                $item->set_quantity( (float) $selected_items[ $item->get_id() ]['quantity'] );
+            }
 
-			// Maybe change the price.
-			if ( $item->user_can_set_their_price() ) {
-				$price = (float) wpinv_sanitize_amount( $selected_items[ $item->get_id() ]['price'] );
+            // Maybe change the price.
+            if ( $item->user_can_set_their_price() ) {
+                $price = (float) wpinv_sanitize_amount( $selected_items[ $item->get_id() ]['price'] );
 
-				if ( $item->get_minimum_price() > $price ) {
-					throw new Exception( sprintf( __( 'The minimum allowed amount is %s', 'invoicing' ), getpaid_unstandardize_amount( $item->get_minimum_price() ) ) );
-				}
+                if ( $item->get_minimum_price() > $price ) {
+                    throw new Exception( sprintf( __( 'The minimum allowed amount is %s', 'invoicing' ), getpaid_unstandardize_amount( $item->get_minimum_price() ) ) );
+                }
 
-				$item->set_price( $price );
+                $item->set_price( $price );
 
-			}
-		}
+            }
+        }
 
-		if ( 0 == $item->get_quantity() ) {
-			return;
-		}
+        if ( 0 == $item->get_quantity() ) {
+            return;
+        }
 
-		// Save the item.
-		$this->items[] = apply_filters( 'getpaid_payment_form_submission_processed_item', $item, $submission );
+        // Save the item.
+        $this->items[] = apply_filters( 'getpaid_payment_form_submission_processed_item', $item, $submission );
 
-	}
+    }
 
 }

Spacing +34 added lines, -34 removed lines

@@ -4,7 +4,7 @@ 
  *
  */
 
-defined( 'ABSPATH' ) || exit;
+defined('ABSPATH') || exit;
 
 /**
  * Payment form submission itemss class
@@ -23,37 +23,37 @@ 
 	 *
 	 * @param GetPaid_Payment_Form_Submission $submission
 	 */
-	public function __construct( $submission ) {
+	public function __construct($submission) {
 
 		$data         = $submission->get_data();
 		$payment_form = $submission->get_payment_form();
 
 		// Prepare the selected items.
 		$selected_items = array();
-		if ( ! empty( $data['getpaid-items'] ) ) {
-			$selected_items = wpinv_clean( $data['getpaid-items'] );
+		if (!empty($data['getpaid-items'])) {
+			$selected_items = wpinv_clean($data['getpaid-items']);
 		}
 
 		// (Maybe) set form items.
-		if ( isset( $data['getpaid-form-items'] ) ) {
+		if (isset($data['getpaid-form-items'])) {
 
 			// Confirm items key.
-			$form_items = wpinv_clean( $data['getpaid-form-items'] );
-			if ( ! isset( $data['getpaid-form-items-key'] ) || $data['getpaid-form-items-key'] !== md5( NONCE_KEY . AUTH_KEY . $form_items ) ) {
-				throw new Exception( __( 'We could not validate the form items. Please reload the page and try again.', 'invoicing' ) );
+			$form_items = wpinv_clean($data['getpaid-form-items']);
+			if (!isset($data['getpaid-form-items-key']) || $data['getpaid-form-items-key'] !== md5(NONCE_KEY . AUTH_KEY . $form_items)) {
+				throw new Exception(__('We could not validate the form items. Please reload the page and try again.', 'invoicing'));
 			}
 
-			$items    = array();
+			$items = array();
             $item_ids = array();
 
-            foreach ( getpaid_convert_items_to_array( $form_items ) as $item_id => $qty ) {
-                if ( ! in_array( $item_id, $item_ids ) ) {
-                    $item = new GetPaid_Form_Item( $item_id );
-                    $item->set_quantity( $qty );
+            foreach (getpaid_convert_items_to_array($form_items) as $item_id => $qty) {
+                if (!in_array($item_id, $item_ids)) {
+                    $item = new GetPaid_Form_Item($item_id);
+                    $item->set_quantity($qty);
 
-                    if ( 0 == $qty ) {
-                        $item->set_allow_quantities( true );
-                        $item->set_is_required( false );
+                    if (0 == $qty) {
+                        $item->set_allow_quantities(true);
+                        $item->set_is_required(false);
                     }
 
                     $item_ids[] = $item->get_id();
@@ -61,23 +61,23 @@ 
                 }
             }
 
-            if ( ! $payment_form->is_default() ) {
+            if (!$payment_form->is_default()) {
 
-                foreach ( $payment_form->get_items() as $item ) {
-                    if ( ! in_array( $item->get_id(), $item_ids ) ) {
+                foreach ($payment_form->get_items() as $item) {
+                    if (!in_array($item->get_id(), $item_ids)) {
                         $item_ids[] = $item->get_id();
                         $items[]    = $item;
                     }
                 }
 			}
 
-            $payment_form->set_items( $items );
+            $payment_form->set_items($items);
 
 		}
 
 		// Process each individual item.
-		foreach ( $payment_form->get_items() as $item ) {
-			$this->process_item( $item, $selected_items, $submission );
+		foreach ($payment_form->get_items() as $item) {
+			$this->process_item($item, $selected_items, $submission);
 		}
 
 	}
@@ -89,40 +89,40 @@ 
 	 * @param array $selected_items
 	 * @param GetPaid_Payment_Form_Submission $submission
 	 */
-	public function process_item( $item, $selected_items, $submission ) {
+	public function process_item($item, $selected_items, $submission) {
 
 		// Abort if this is an optional item and it has not been selected.
-		if ( ! $item->is_required() && ! isset( $selected_items[ $item->get_id() ] ) ) {
+		if (!$item->is_required() && !isset($selected_items[$item->get_id()])) {
 			return;
 		}
 
 		// (maybe) let customers change the quantities and prices.
-		if ( isset( $selected_items[ $item->get_id() ] ) ) {
+		if (isset($selected_items[$item->get_id()])) {
 
 			// Maybe change the quantities.
-			if ( $item->allows_quantities() ) {
-				$item->set_quantity( (float) $selected_items[ $item->get_id() ]['quantity'] );
+			if ($item->allows_quantities()) {
+				$item->set_quantity((float) $selected_items[$item->get_id()]['quantity']);
 			}
 
 			// Maybe change the price.
-			if ( $item->user_can_set_their_price() ) {
-				$price = (float) wpinv_sanitize_amount( $selected_items[ $item->get_id() ]['price'] );
+			if ($item->user_can_set_their_price()) {
+				$price = (float) wpinv_sanitize_amount($selected_items[$item->get_id()]['price']);
 
-				if ( $item->get_minimum_price() > $price ) {
-					throw new Exception( sprintf( __( 'The minimum allowed amount is %s', 'invoicing' ), getpaid_unstandardize_amount( $item->get_minimum_price() ) ) );
+				if ($item->get_minimum_price() > $price) {
+					throw new Exception(sprintf(__('The minimum allowed amount is %s', 'invoicing'), getpaid_unstandardize_amount($item->get_minimum_price())));
 				}
 
-				$item->set_price( $price );
+				$item->set_price($price);
 
 			}
 		}
 
-		if ( 0 == $item->get_quantity() ) {
+		if (0 == $item->get_quantity()) {
 			return;
 		}
 
 		// Save the item.
-		$this->items[] = apply_filters( 'getpaid_payment_form_submission_processed_item', $item, $submission );
+		$this->items[] = apply_filters('getpaid_payment_form_submission_processed_item', $item, $submission);
 
 	}
 

includes/payments/class-getpaid-payment-form.php

Indentation +571 added lines, -571 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 if ( ! defined( 'ABSPATH' ) ) {
-	exit;
+    exit;
 }
 
 /**
@@ -10,55 +10,55 @@ 
 class GetPaid_Payment_Form extends GetPaid_Data {
 
     /**
-	 * Which data store to load.
-	 *
-	 * @var string
-	 */
+     * Which data store to load.
+     *
+     * @var string
+     */
     protected $data_store_name = 'payment_form';
 
     /**
-	 * This is the name of this object type.
-	 *
-	 * @var string
-	 */
-	protected $object_type = 'payment_form';
+     * This is the name of this object type.
+     *
+     * @var string
+     */
+    protected $object_type = 'payment_form';
 
     /**
-	 * Form Data array. This is the core form data exposed in APIs.
-	 *
-	 * @since 1.0.19
-	 * @var array
-	 */
-	protected $data = array(
-		'status'        => 'draft',
-		'version'       => '',
-		'date_created'  => null,
+     * Form Data array. This is the core form data exposed in APIs.
+     *
+     * @since 1.0.19
+     * @var array
+     */
+    protected $data = array(
+        'status'        => 'draft',
+        'version'       => '',
+        'date_created'  => null,
         'date_modified' => null,
         'name'          => '',
         'author'        => 1,
         'elements'      => null,
-		'items'         => null,
-		'earned'        => 0,
-		'refunded'      => 0,
-		'cancelled'     => 0,
-		'failed'        => 0,
-	);
-
-    /**
-	 * Stores meta in cache for future reads.
-	 *
-	 * A group must be set to to enable caching.
-	 *
-	 * @var string
-	 */
-	protected $cache_group = 'getpaid_forms';
-
-	/**
-	 * Stores a reference to the invoice if the form is for an invoice..
-	 *
-	 * @var WPInv_Invoice
-	 */
-	public $invoice = 0;
+        'items'         => null,
+        'earned'        => 0,
+        'refunded'      => 0,
+        'cancelled'     => 0,
+        'failed'        => 0,
+    );
+
+    /**
+     * Stores meta in cache for future reads.
+     *
+     * A group must be set to to enable caching.
+     *
+     * @var string
+     */
+    protected $cache_group = 'getpaid_forms';
+
+    /**
+     * Stores a reference to the invoice if the form is for an invoice..
+     *
+     * @var WPInv_Invoice
+     */
+    public $invoice = 0;
 
     /**
      * Stores a reference to the original WP_Post object
@@ -68,35 +68,35 @@ 
     protected $post = null;
 
     /**
-	 * Get the form if ID is passed, otherwise the form is new and empty.
-	 *
-	 * @param  int|object|GetPaid_Payment_Form|WP_Post $form Form to read.
-	 */
-	public function __construct( $form = 0 ) {
-		parent::__construct( $form );
+     * Get the form if ID is passed, otherwise the form is new and empty.
+     *
+     * @param  int|object|GetPaid_Payment_Form|WP_Post $form Form to read.
+     */
+    public function __construct( $form = 0 ) {
+        parent::__construct( $form );
 
-		if ( is_numeric( $form ) && $form > 0 ) {
-			$this->set_id( $form );
-		} elseif ( $form instanceof self ) {
+        if ( is_numeric( $form ) && $form > 0 ) {
+            $this->set_id( $form );
+        } elseif ( $form instanceof self ) {
 
-			$this->set_id( $form->get_id() );
-			$this->invoice = $form->invoice;
+            $this->set_id( $form->get_id() );
+            $this->invoice = $form->invoice;
 
-		} elseif ( ! empty( $form->ID ) ) {
-			$this->set_id( $form->ID );
-		} else {
-			$this->set_object_read( true );
-		}
+        } elseif ( ! empty( $form->ID ) ) {
+            $this->set_id( $form->ID );
+        } else {
+            $this->set_object_read( true );
+        }
 
         // Load the datastore.
-		$this->data_store = GetPaid_Data_Store::load( $this->data_store_name );
+        $this->data_store = GetPaid_Data_Store::load( $this->data_store_name );
 
-		if ( $this->get_id() > 0 ) {
+        if ( $this->get_id() > 0 ) {
             $this->post = get_post( $this->get_id() );
-			$this->data_store->read( $this );
+            $this->data_store->read( $this );
         }
 
-	}
+    }
 
     /*
 	|--------------------------------------------------------------------------
@@ -114,356 +114,356 @@ 
     */
 
     /**
-	 * Get plugin version when the form was created.
-	 *
-	 * @since 1.0.19
-	 * @param  string $context View or edit context.
-	 * @return string
-	 */
-	public function get_version( $context = 'view' ) {
-		return $this->get_prop( 'version', $context );
+     * Get plugin version when the form was created.
+     *
+     * @since 1.0.19
+     * @param  string $context View or edit context.
+     * @return string
+     */
+    public function get_version( $context = 'view' ) {
+        return $this->get_prop( 'version', $context );
     }
 
     /**
-	 * Get date when the form was created.
-	 *
-	 * @since 1.0.19
-	 * @param  string $context View or edit context.
-	 * @return string
-	 */
-	public function get_date_created( $context = 'view' ) {
-		return $this->get_prop( 'date_created', $context );
+     * Get date when the form was created.
+     *
+     * @since 1.0.19
+     * @param  string $context View or edit context.
+     * @return string
+     */
+    public function get_date_created( $context = 'view' ) {
+        return $this->get_prop( 'date_created', $context );
     }
 
     /**
-	 * Get GMT date when the form was created.
-	 *
-	 * @since 1.0.19
-	 * @param  string $context View or edit context.
-	 * @return string
-	 */
-	public function get_date_created_gmt( $context = 'view' ) {
+     * Get GMT date when the form was created.
+     *
+     * @since 1.0.19
+     * @param  string $context View or edit context.
+     * @return string
+     */
+    public function get_date_created_gmt( $context = 'view' ) {
         $date = $this->get_date_created( $context );
 
         if ( $date ) {
             $date = get_gmt_from_date( $date );
         }
-		return $date;
+        return $date;
     }
 
     /**
-	 * Get date when the form was last modified.
-	 *
-	 * @since 1.0.19
-	 * @param  string $context View or edit context.
-	 * @return string
-	 */
-	public function get_date_modified( $context = 'view' ) {
-		return $this->get_prop( 'date_modified', $context );
+     * Get date when the form was last modified.
+     *
+     * @since 1.0.19
+     * @param  string $context View or edit context.
+     * @return string
+     */
+    public function get_date_modified( $context = 'view' ) {
+        return $this->get_prop( 'date_modified', $context );
     }
 
     /**
-	 * Get GMT date when the form was last modified.
-	 *
-	 * @since 1.0.19
-	 * @param  string $context View or edit context.
-	 * @return string
-	 */
-	public function get_date_modified_gmt( $context = 'view' ) {
+     * Get GMT date when the form was last modified.
+     *
+     * @since 1.0.19
+     * @param  string $context View or edit context.
+     * @return string
+     */
+    public function get_date_modified_gmt( $context = 'view' ) {
         $date = $this->get_date_modified( $context );
 
         if ( $date ) {
             $date = get_gmt_from_date( $date );
         }
-		return $date;
+        return $date;
     }
 
     /**
-	 * Get the form name.
-	 *
-	 * @since 1.0.19
-	 * @param  string $context View or edit context.
-	 * @return string
-	 */
-	public function get_name( $context = 'view' ) {
-		return $this->get_prop( 'name', $context );
+     * Get the form name.
+     *
+     * @since 1.0.19
+     * @param  string $context View or edit context.
+     * @return string
+     */
+    public function get_name( $context = 'view' ) {
+        return $this->get_prop( 'name', $context );
     }
 
     /**
-	 * Alias of self::get_name().
-	 *
-	 * @since 1.0.19
-	 * @param  string $context View or edit context.
-	 * @return string
-	 */
-	public function get_title( $context = 'view' ) {
-		return $this->get_name( $context );
-	}
+     * Alias of self::get_name().
+     *
+     * @since 1.0.19
+     * @param  string $context View or edit context.
+     * @return string
+     */
+    public function get_title( $context = 'view' ) {
+        return $this->get_name( $context );
+    }
 
     /**
-	 * Get the owner of the form.
-	 *
-	 * @since 1.0.19
-	 * @param  string $context View or edit context.
-	 * @return int
-	 */
-	public function get_author( $context = 'view' ) {
-		return (int) $this->get_prop( 'author', $context );
+     * Get the owner of the form.
+     *
+     * @since 1.0.19
+     * @param  string $context View or edit context.
+     * @return int
+     */
+    public function get_author( $context = 'view' ) {
+        return (int) $this->get_prop( 'author', $context );
     }
 
     /**
-	 * Get the elements that make up the form.
-	 *
-	 * @since 1.0.19
-	 * @param  string $context View or edit context.
-	 * @return array
-	 */
-	public function get_elements( $context = 'view' ) {
-		$elements = $this->get_prop( 'elements', $context );
+     * Get the elements that make up the form.
+     *
+     * @since 1.0.19
+     * @param  string $context View or edit context.
+     * @return array
+     */
+    public function get_elements( $context = 'view' ) {
+        $elements = $this->get_prop( 'elements', $context );
 
-		if ( empty( $elements ) || ! is_array( $elements ) ) {
+        if ( empty( $elements ) || ! is_array( $elements ) ) {
             return wpinv_get_data( 'sample-payment-form' );
-		}
+        }
 
-		// Ensure that all required elements exist.
-		$_elements = array();
-		foreach ( $elements as $element ) {
+        // Ensure that all required elements exist.
+        $_elements = array();
+        foreach ( $elements as $element ) {
 
-			if ( $element['type'] == 'pay_button' && ! $this->has_element_type( 'gateway_select' ) ) {
+            if ( $element['type'] == 'pay_button' && ! $this->has_element_type( 'gateway_select' ) ) {
 
-				$_elements[] = array(
-					'text'    => __( 'Select Payment Method', 'invoicing' ),
-					'id'      => 'gtscicd',
-					'name'    => 'gtscicd',
-					'type'    => 'gateway_select',
-					'premade' => true,
+                $_elements[] = array(
+                    'text'    => __( 'Select Payment Method', 'invoicing' ),
+                    'id'      => 'gtscicd',
+                    'name'    => 'gtscicd',
+                    'type'    => 'gateway_select',
+                    'premade' => true,
 
-				);
+                );
 
-			}
+            }
 
-			$_elements[] = $element;
+            $_elements[] = $element;
 
-		}
+        }
 
         return $_elements;
-	}
-
-	/**
-	 * Get the items sold via the form.
-	 *
-	 * @since 1.0.19
-	 * @param  string $context View or edit context.
-	 * @param  string $return objects or arrays.
-	 * @return GetPaid_Form_Item[]
-	 */
-	public function get_items( $context = 'view', $return = 'objects' ) {
-		$items = $this->get_prop( 'items', $context );
-
-		if ( empty( $items ) || ! is_array( $items ) ) {
+    }
+
+    /**
+     * Get the items sold via the form.
+     *
+     * @since 1.0.19
+     * @param  string $context View or edit context.
+     * @param  string $return objects or arrays.
+     * @return GetPaid_Form_Item[]
+     */
+    public function get_items( $context = 'view', $return = 'objects' ) {
+        $items = $this->get_prop( 'items', $context );
+
+        if ( empty( $items ) || ! is_array( $items ) ) {
             $items = wpinv_get_data( 'sample-payment-form-items' );
-		}
+        }
 
-		// Convert the items.
-		$prepared = array();
+        // Convert the items.
+        $prepared = array();
 
-		foreach ( $items as $key => $value ) {
+        foreach ( $items as $key => $value ) {
 
-			// Form items.
-			if ( $value instanceof GetPaid_Form_Item ) {
+            // Form items.
+            if ( $value instanceof GetPaid_Form_Item ) {
 
-				if ( $value->can_purchase() ) {
-					$prepared[] = $value;
-				}
+                if ( $value->can_purchase() ) {
+                    $prepared[] = $value;
+                }
 
-				continue;
+                continue;
 
-			}
+            }
 
-			// $item_id => $quantity (buy buttons)
-			if ( is_numeric( $key ) && is_numeric( $value ) ) {
-				$item = new GetPaid_Form_Item( $key );
+            // $item_id => $quantity (buy buttons)
+            if ( is_numeric( $key ) && is_numeric( $value ) ) {
+                $item = new GetPaid_Form_Item( $key );
 
-				if ( $item->can_purchase() ) {
+                if ( $item->can_purchase() ) {
 
-					$value = (float) $value;
-					$item->set_quantity( $value );
-					if ( 0 == $value ) {
-						$item->set_quantity( 1 );
-						$item->set_allow_quantities( true );
-					}
+                    $value = (float) $value;
+                    $item->set_quantity( $value );
+                    if ( 0 == $value ) {
+                        $item->set_quantity( 1 );
+                        $item->set_allow_quantities( true );
+                    }
 
-					$prepared[] = $item;
-				}
+                    $prepared[] = $item;
+                }
 
-				continue;
-			}
+                continue;
+            }
 
-			// Items saved via payment forms editor.
-			if ( is_array( $value ) && isset( $value['id'] ) ) {
+            // Items saved via payment forms editor.
+            if ( is_array( $value ) && isset( $value['id'] ) ) {
 
-				$item = new GetPaid_Form_Item( $value['id'] );
+                $item = new GetPaid_Form_Item( $value['id'] );
 
-				if ( ! $item->can_purchase() ) {
-					continue;
-				}
+                if ( ! $item->can_purchase() ) {
+                    continue;
+                }
 
-				// Sub-total (Cart items).
-				if ( isset( $value['subtotal'] ) ) {
-					$item->set_price( $value['subtotal'] );
-				}
+                // Sub-total (Cart items).
+                if ( isset( $value['subtotal'] ) ) {
+                    $item->set_price( $value['subtotal'] );
+                }
 
-				if ( isset( $value['quantity'] ) ) {
-					$item->set_quantity( $value['quantity'] );
-				}
+                if ( isset( $value['quantity'] ) ) {
+                    $item->set_quantity( $value['quantity'] );
+                }
 
-				if ( isset( $value['allow_quantities'] ) ) {
-					$item->set_allow_quantities( $value['allow_quantities'] );
-				}
+                if ( isset( $value['allow_quantities'] ) ) {
+                    $item->set_allow_quantities( $value['allow_quantities'] );
+                }
 
-				if ( isset( $value['required'] ) ) {
-					$item->set_is_required( $value['required'] );
-				}
+                if ( isset( $value['required'] ) ) {
+                    $item->set_is_required( $value['required'] );
+                }
 
-				if ( isset( $value['description'] ) ) {
-					$item->set_custom_description( $value['description'] );
-				}
+                if ( isset( $value['description'] ) ) {
+                    $item->set_custom_description( $value['description'] );
+                }
 
-				$prepared[] = $item;
-				continue;
+                $prepared[] = $item;
+                continue;
 
-			}
+            }
 
-			// $item_id => array( 'price' => 10 ) (item variations)
-			if ( is_numeric( $key ) && is_array( $value ) ) {
-				$item = new GetPaid_Form_Item( $key );
+            // $item_id => array( 'price' => 10 ) (item variations)
+            if ( is_numeric( $key ) && is_array( $value ) ) {
+                $item = new GetPaid_Form_Item( $key );
 
-				if ( isset( $value['price'] ) && $item->user_can_set_their_price() ) {
-					$item->set_price( $value['price'] );
-				}
+                if ( isset( $value['price'] ) && $item->user_can_set_their_price() ) {
+                    $item->set_price( $value['price'] );
+                }
 
-				if ( $item->can_purchase() ) {
-					$prepared[] = $item;
-				}
+                if ( $item->can_purchase() ) {
+                    $prepared[] = $item;
+                }
 
-				continue;
-			}
+                continue;
+            }
 }
 
-		if ( 'objects' == $return && 'view' == $context ) {
-			return $prepared;
-		}
-
-		$items = array();
-		foreach ( $prepared as $item ) {
-			$items[] = $item->prepare_data_for_use();
-		}
-
-		return $items;
-	}
-
-	/**
-	 * Get a single item belonging to the form.
-	 *
-	 * @since 1.0.19
-	 * @param  int $item_id The item id to return.
-	 * @return GetPaid_Form_Item|bool
-	 */
-	public function get_item( $item_id ) {
-
-		if ( empty( $item_id ) || ! is_numeric( $item_id ) ) {
-			return false;
-		}
-
-		foreach ( $this->get_items() as $item ) {
-			if ( $item->get_id() == (int) $item_id ) {
-				return $item;
-			}
-		}
-
-		return false;
-
-	}
-
-	/**
-	 * Gets a single element.
-	 *
-	 * @since 1.0.19
-	 * @param  string $element_type The element type to return.
-	 * @return array|bool
-	 */
-	public function get_element_type( $element_type ) {
-
-		if ( empty( $element_type ) || ! is_scalar( $element_type ) ) {
-			return false;
-		}
-
-		foreach ( $this->get_prop( 'elements' ) as $element ) {
-
-			if ( $element['type'] == $element_type ) {
-				return $element;
-			}
+        if ( 'objects' == $return && 'view' == $context ) {
+            return $prepared;
+        }
+
+        $items = array();
+        foreach ( $prepared as $item ) {
+            $items[] = $item->prepare_data_for_use();
+        }
+
+        return $items;
+    }
+
+    /**
+     * Get a single item belonging to the form.
+     *
+     * @since 1.0.19
+     * @param  int $item_id The item id to return.
+     * @return GetPaid_Form_Item|bool
+     */
+    public function get_item( $item_id ) {
+
+        if ( empty( $item_id ) || ! is_numeric( $item_id ) ) {
+            return false;
+        }
+
+        foreach ( $this->get_items() as $item ) {
+            if ( $item->get_id() == (int) $item_id ) {
+                return $item;
+            }
+        }
+
+        return false;
+
+    }
+
+    /**
+     * Gets a single element.
+     *
+     * @since 1.0.19
+     * @param  string $element_type The element type to return.
+     * @return array|bool
+     */
+    public function get_element_type( $element_type ) {
+
+        if ( empty( $element_type ) || ! is_scalar( $element_type ) ) {
+            return false;
+        }
+
+        foreach ( $this->get_prop( 'elements' ) as $element ) {
+
+            if ( $element['type'] == $element_type ) {
+                return $element;
+            }
 }
 
-		return false;
-
-	}
-
-	/**
-	 * Get the total amount earned via this form.
-	 *
-	 * @since 1.0.19
-	 * @param  string $context View or edit context.
-	 * @return float
-	 */
-	public function get_earned( $context = 'view' ) {
-		return $this->get_prop( 'earned', $context );
-	}
-
-	/**
-	 * Get the total amount refunded via this form.
-	 *
-	 * @since 1.0.19
-	 * @param  string $context View or edit context.
-	 * @return float
-	 */
-	public function get_refunded( $context = 'view' ) {
-		return $this->get_prop( 'refunded', $context );
-	}
-
-	/**
-	 * Get the total amount cancelled via this form.
-	 *
-	 * @since 1.0.19
-	 * @param  string $context View or edit context.
-	 * @return float
-	 */
-	public function get_cancelled( $context = 'view' ) {
-		return $this->get_prop( 'cancelled', $context );
-	}
-
-	/**
-	 * Get the total amount failed via this form.
-	 *
-	 * @since 1.0.19
-	 * @param  string $context View or edit context.
-	 * @return float
-	 */
-	public function get_failed( $context = 'view' ) {
-		return $this->get_prop( 'failed', $context );
-	}
-
-	/**
-	 * Get the currency.
-	 *
-	 * @since 1.0.19
-	 * @param  string $context View or edit context.
-	 * @return string
-	 */
-	public function get_currency() {
-		$currency = empty( $this->invoice ) ? wpinv_get_currency() : $this->invoice->get_currency();
-		return apply_filters( 'getpaid-payment-form-currency', $currency, $this );
-	}
+        return false;
+
+    }
+
+    /**
+     * Get the total amount earned via this form.
+     *
+     * @since 1.0.19
+     * @param  string $context View or edit context.
+     * @return float
+     */
+    public function get_earned( $context = 'view' ) {
+        return $this->get_prop( 'earned', $context );
+    }
+
+    /**
+     * Get the total amount refunded via this form.
+     *
+     * @since 1.0.19
+     * @param  string $context View or edit context.
+     * @return float
+     */
+    public function get_refunded( $context = 'view' ) {
+        return $this->get_prop( 'refunded', $context );
+    }
+
+    /**
+     * Get the total amount cancelled via this form.
+     *
+     * @since 1.0.19
+     * @param  string $context View or edit context.
+     * @return float
+     */
+    public function get_cancelled( $context = 'view' ) {
+        return $this->get_prop( 'cancelled', $context );
+    }
+
+    /**
+     * Get the total amount failed via this form.
+     *
+     * @since 1.0.19
+     * @param  string $context View or edit context.
+     * @return float
+     */
+    public function get_failed( $context = 'view' ) {
+        return $this->get_prop( 'failed', $context );
+    }
+
+    /**
+     * Get the currency.
+     *
+     * @since 1.0.19
+     * @param  string $context View or edit context.
+     * @return string
+     */
+    public function get_currency() {
+        $currency = empty( $this->invoice ) ? wpinv_get_currency() : $this->invoice->get_currency();
+        return apply_filters( 'getpaid-payment-form-currency', $currency, $this );
+    }
 
     /*
 	|--------------------------------------------------------------------------
@@ -476,22 +476,22 @@ 
     */
 
     /**
-	 * Set plugin version when the item was created.
-	 *
-	 * @since 1.0.19
-	 */
-	public function set_version( $value ) {
-		$this->set_prop( 'version', $value );
+     * Set plugin version when the item was created.
+     *
+     * @since 1.0.19
+     */
+    public function set_version( $value ) {
+        $this->set_prop( 'version', $value );
     }
 
     /**
-	 * Set date when the item was created.
-	 *
-	 * @since 1.0.19
-	 * @param string $value Value to set.
+     * Set date when the item was created.
+     *
+     * @since 1.0.19
+     * @param string $value Value to set.
      * @return bool Whether or not the date was set.
-	 */
-	public function set_date_created( $value ) {
+     */
+    public function set_date_created( $value ) {
         $date = strtotime( $value );
 
         if ( $date ) {
@@ -504,13 +504,13 @@ 
     }
 
     /**
-	 * Set date when the item was last modified.
-	 *
-	 * @since 1.0.19
-	 * @param string $value Value to set.
+     * Set date when the item was last modified.
+     *
+     * @since 1.0.19
+     * @param string $value Value to set.
      * @return bool Whether or not the date was set.
-	 */
-	public function set_date_modified( $value ) {
+     */
+    public function set_date_modified( $value ) {
         $date = strtotime( $value );
 
         if ( $date ) {
@@ -523,164 +523,164 @@ 
     }
 
     /**
-	 * Set the item name.
-	 *
-	 * @since 1.0.19
-	 * @param  string $value New name.
-	 */
-	public function set_name( $value ) {
-		$this->set_prop( 'name', sanitize_text_field( $value ) );
-    }
-
-    /**
-	 * Alias of self::set_name().
-	 *
-	 * @since 1.0.19
-	 * @param  string $value New name.
-	 */
-	public function set_title( $value ) {
-		$this->set_name( $value );
-    }
-
-    /**
-	 * Set the owner of the item.
-	 *
-	 * @since 1.0.19
-	 * @param  int $value New author.
-	 */
-	public function set_author( $value ) {
-		$this->set_prop( 'author', (int) $value );
-	}
-
-	/**
-	 * Set the form elements.
-	 *
-	 * @since 1.0.19
-	 * @sinve 2.3.4 Array values sanitized.
-	 * @param  array $value Form elements.
-	 */
-	public function set_elements( $value ) {
-		if ( is_array( $value ) ) {
-			$this->set_prop( 'elements', wp_kses_post_deep( $value ) );
-		}
-	}
-
-	/**
-	 * Sanitize array values.
-	 *
-	 * @param $value
-	 *
-	 * @return mixed
-	 */
-	public function sanitize_array_values( $value ) {
-
-		// sanitize
-		if ( ! empty( $value ) ) {
-
-			foreach ( $value as $key => $val_arr ) {
-
-				if ( is_array( $val_arr ) ) {
-					// check if we have sub array items.
-					$sub_arr = array();
-					foreach ( $val_arr as $key2 => $val2 ) {
-						if ( is_array( $val2 ) ) {
-							$sub_arr[ $key2 ] = $this->sanitize_array_values( $val2 );
-							unset( $val_arr[ $key ][ $key2 ] );
-						}
-					}
-
-					// we allow some html in description so we sanitize it separately.
-					$help_text = ! empty( $val_arr['description'] ) ? wp_kses_post( $val_arr['description'] ) : '';
-
-					// sanitize array elements
-					$value[ $key ] = array_map( 'sanitize_text_field', $val_arr );
-
-					// add back the description if set
-					if ( isset( $val_arr['description'] ) ) {
+     * Set the item name.
+     *
+     * @since 1.0.19
+     * @param  string $value New name.
+     */
+    public function set_name( $value ) {
+        $this->set_prop( 'name', sanitize_text_field( $value ) );
+    }
+
+    /**
+     * Alias of self::set_name().
+     *
+     * @since 1.0.19
+     * @param  string $value New name.
+     */
+    public function set_title( $value ) {
+        $this->set_name( $value );
+    }
+
+    /**
+     * Set the owner of the item.
+     *
+     * @since 1.0.19
+     * @param  int $value New author.
+     */
+    public function set_author( $value ) {
+        $this->set_prop( 'author', (int) $value );
+    }
+
+    /**
+     * Set the form elements.
+     *
+     * @since 1.0.19
+     * @sinve 2.3.4 Array values sanitized.
+     * @param  array $value Form elements.
+     */
+    public function set_elements( $value ) {
+        if ( is_array( $value ) ) {
+            $this->set_prop( 'elements', wp_kses_post_deep( $value ) );
+        }
+    }
+
+    /**
+     * Sanitize array values.
+     *
+     * @param $value
+     *
+     * @return mixed
+     */
+    public function sanitize_array_values( $value ) {
+
+        // sanitize
+        if ( ! empty( $value ) ) {
+
+            foreach ( $value as $key => $val_arr ) {
+
+                if ( is_array( $val_arr ) ) {
+                    // check if we have sub array items.
+                    $sub_arr = array();
+                    foreach ( $val_arr as $key2 => $val2 ) {
+                        if ( is_array( $val2 ) ) {
+                            $sub_arr[ $key2 ] = $this->sanitize_array_values( $val2 );
+                            unset( $val_arr[ $key ][ $key2 ] );
+                        }
+                    }
+
+                    // we allow some html in description so we sanitize it separately.
+                    $help_text = ! empty( $val_arr['description'] ) ? wp_kses_post( $val_arr['description'] ) : '';
+
+                    // sanitize array elements
+                    $value[ $key ] = array_map( 'sanitize_text_field', $val_arr );
+
+                    // add back the description if set
+                    if ( isset( $val_arr['description'] ) ) {
 $value[ $key ]['description'] = $help_text;}
 
-					// add back sub array items after its been sanitized.
-					if ( ! empty( $sub_arr ) ) {
-						$value[ $key ] = array_merge( $value[ $key ], $sub_arr );
-					}
-				}
+                    // add back sub array items after its been sanitized.
+                    if ( ! empty( $sub_arr ) ) {
+                        $value[ $key ] = array_merge( $value[ $key ], $sub_arr );
+                    }
+                }
 }
 }
 
-		return $value;
-	}
-
-	/**
-	 * Set the form items.
-	 *
-	 * @since 1.0.19
-	 * @param  array $value Form elements.
-	 */
-	public function set_items( $value ) {
-		if ( is_array( $value ) ) {
-			$this->set_prop( 'items', $value );
-		}
-	}
-
-	/**
-	 * Set the total amount earned via this form.
-	 *
-	 * @since 1.0.19
-	 * @param  float $value Amount earned.
-	 */
-	public function set_earned( $value ) {
-		$value = max( (float) $value, 0 );
-		$this->set_prop( 'earned', $value );
-	}
-
-	/**
-	 * Set the total amount refunded via this form.
-	 *
-	 * @since 1.0.19
-	 * @param  float $value Amount refunded.
-	 */
-	public function set_refunded( $value ) {
-		$value = max( (float) $value, 0 );
-		$this->set_prop( 'refunded', $value );
-	}
-
-	/**
-	 * Set the total amount cancelled via this form.
-	 *
-	 * @since 1.0.19
-	 * @param  float $value Amount cancelled.
-	 */
-	public function set_cancelled( $value ) {
-		$value = max( (float) $value, 0 );
-		$this->set_prop( 'cancelled', $value );
-	}
-
-	/**
-	 * Set the total amount failed via this form.
-	 *
-	 * @since 1.0.19
-	 * @param  float $value Amount cancelled.
-	 */
-	public function set_failed( $value ) {
-		$value = max( (float) $value, 0 );
-		$this->set_prop( 'failed', $value );
-	}
+        return $value;
+    }
+
+    /**
+     * Set the form items.
+     *
+     * @since 1.0.19
+     * @param  array $value Form elements.
+     */
+    public function set_items( $value ) {
+        if ( is_array( $value ) ) {
+            $this->set_prop( 'items', $value );
+        }
+    }
+
+    /**
+     * Set the total amount earned via this form.
+     *
+     * @since 1.0.19
+     * @param  float $value Amount earned.
+     */
+    public function set_earned( $value ) {
+        $value = max( (float) $value, 0 );
+        $this->set_prop( 'earned', $value );
+    }
+
+    /**
+     * Set the total amount refunded via this form.
+     *
+     * @since 1.0.19
+     * @param  float $value Amount refunded.
+     */
+    public function set_refunded( $value ) {
+        $value = max( (float) $value, 0 );
+        $this->set_prop( 'refunded', $value );
+    }
+
+    /**
+     * Set the total amount cancelled via this form.
+     *
+     * @since 1.0.19
+     * @param  float $value Amount cancelled.
+     */
+    public function set_cancelled( $value ) {
+        $value = max( (float) $value, 0 );
+        $this->set_prop( 'cancelled', $value );
+    }
+
+    /**
+     * Set the total amount failed via this form.
+     *
+     * @since 1.0.19
+     * @param  float $value Amount cancelled.
+     */
+    public function set_failed( $value ) {
+        $value = max( (float) $value, 0 );
+        $this->set_prop( 'failed', $value );
+    }
 
     /**
      * Create an item. For backwards compatibilty.
      *
      * @deprecated
-	 * @return int item id
+     * @return int item id
      */
     public function create( $data = array() ) {
 
-		// Set the properties.
-		if ( is_array( $data ) ) {
-			$this->set_props( $data );
-		}
+        // Set the properties.
+        if ( is_array( $data ) ) {
+            $this->set_props( $data );
+        }
 
-		// Save the item.
-		return $this->save();
+        // Save the item.
+        return $this->save();
 
     }
 
@@ -688,7 +688,7 @@ 
      * Updates an item. For backwards compatibilty.
      *
      * @deprecated
-	 * @return int item id
+     * @return int item id
      */
     public function update( $data = array() ) {
         return $this->create( $data );
@@ -704,22 +704,22 @@ 
 	*/
 
     /**
-	 * Checks whether this is the default payment form.
-	 *
-	 * @since 1.0.19
-	 * @return bool
-	 */
+     * Checks whether this is the default payment form.
+     *
+     * @since 1.0.19
+     * @return bool
+     */
     public function is_default() {
         $is_default = $this->get_id() == wpinv_get_default_payment_form();
         return (bool) apply_filters( 'wpinv_is_default_payment_form', $is_default, $this->get_id(), $this );
-	}
+    }
 
     /**
-	 * Checks whether the form is active.
-	 *
-	 * @since 1.0.19
-	 * @return bool
-	 */
+     * Checks whether the form is active.
+     *
+     * @since 1.0.19
+     * @return bool
+     */
     public function is_active() {
         $is_active = 0 !== (int) $this->get_id();
 
@@ -728,81 +728,81 @@ 
         }
 
         return (bool) apply_filters( 'wpinv_is_payment_form_active', $is_active, $this );
-	}
-
-	/**
-	 * Checks whether the form has a given item.
-	 *
-	 * @since 1.0.19
-	 * @return bool
-	 */
+    }
+
+    /**
+     * Checks whether the form has a given item.
+     *
+     * @since 1.0.19
+     * @return bool
+     */
     public function has_item( $item_id ) {
         return false !== $this->get_item( $item_id );
-	}
-
-	/**
-	 * Checks whether the form has a given element.
-	 *
-	 * @since 1.0.19
-	 * @return bool
-	 */
+    }
+
+    /**
+     * Checks whether the form has a given element.
+     *
+     * @since 1.0.19
+     * @return bool
+     */
     public function has_element_type( $element_type ) {
         return false !== $this->get_element_type( $element_type );
-	}
-
-	/**
-	 * Checks whether this form is recurring or not.
-	 *
-	 * @since 1.0.19
-	 * @return bool
-	 */
+    }
+
+    /**
+     * Checks whether this form is recurring or not.
+     *
+     * @since 1.0.19
+     * @return bool
+     */
     public function is_recurring() {
 
-		if ( ! empty( $this->invoice ) ) {
-			return $this->invoice->is_recurring();
-		}
+        if ( ! empty( $this->invoice ) ) {
+            return $this->invoice->is_recurring();
+        }
 
-		foreach ( $this->get_items() as $item ) {
+        foreach ( $this->get_items() as $item ) {
 
-			if ( $item->is_recurring() ) {
-				return true;
-			}
+            if ( $item->is_recurring() ) {
+                return true;
+            }
 }
 
         return false;
-	}
+    }
 
-	/**
-	 * Retrieves the form's html.
-	 *
-	 * @since 1.0.19
-	 */
+    /**
+     * Retrieves the form's html.
+     *
+     * @since 1.0.19
+     */
     public function get_html( $extra_markup = '' ) {
 
-		// Return the HTML.
-		return wpinv_get_template_html(
-			'payment-forms/form.php',
-			array(
-				'form'         => $this,
-				'extra_markup' => $extra_markup,
-			)
-		);
-
-	}
-
-	/**
-	 * Displays the payment form.
-	 *
-	 * @since 1.0.19
-	 */
+        // Return the HTML.
+        return wpinv_get_template_html(
+            'payment-forms/form.php',
+            array(
+                'form'         => $this,
+                'extra_markup' => $extra_markup,
+            )
+        );
+
+    }
+
+    /**
+     * Displays the payment form.
+     *
+     * @since 1.0.19
+     */
     public function display( $extra_markup = '' ) {
-		wpinv_get_template(
-			'payment-forms/form.php',
-			array(
-				'form'         => $this,
-				'extra_markup' => $extra_markup,
-			)
-		);
+        wpinv_get_template(
+            'payment-forms/form.php',
+            array(
+                'form'         => $this,
+                'extra_markup' => $extra_markup,
+            )
+        );
     }
 
 }

Spacing +158 added lines, -158 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if ( ! defined( 'ABSPATH' ) ) {
+if (!defined('ABSPATH')) {
 	exit;
 }
 
@@ -72,28 +72,28 @@ 
 	 *
 	 * @param  int|object|GetPaid_Payment_Form|WP_Post $form Form to read.
 	 */
-	public function __construct( $form = 0 ) {
-		parent::__construct( $form );
+	public function __construct($form = 0) {
+		parent::__construct($form);
 
-		if ( is_numeric( $form ) && $form > 0 ) {
-			$this->set_id( $form );
-		} elseif ( $form instanceof self ) {
+		if (is_numeric($form) && $form > 0) {
+			$this->set_id($form);
+		} elseif ($form instanceof self) {
 
-			$this->set_id( $form->get_id() );
+			$this->set_id($form->get_id());
 			$this->invoice = $form->invoice;
 
-		} elseif ( ! empty( $form->ID ) ) {
-			$this->set_id( $form->ID );
+		} elseif (!empty($form->ID)) {
+			$this->set_id($form->ID);
 		} else {
-			$this->set_object_read( true );
+			$this->set_object_read(true);
 		}
 
         // Load the datastore.
-		$this->data_store = GetPaid_Data_Store::load( $this->data_store_name );
+		$this->data_store = GetPaid_Data_Store::load($this->data_store_name);
 
-		if ( $this->get_id() > 0 ) {
-            $this->post = get_post( $this->get_id() );
-			$this->data_store->read( $this );
+		if ($this->get_id() > 0) {
+            $this->post = get_post($this->get_id());
+			$this->data_store->read($this);
         }
 
 	}
@@ -120,8 +120,8 @@ 
 	 * @param  string $context View or edit context.
 	 * @return string
 	 */
-	public function get_version( $context = 'view' ) {
-		return $this->get_prop( 'version', $context );
+	public function get_version($context = 'view') {
+		return $this->get_prop('version', $context);
     }
 
     /**
@@ -131,8 +131,8 @@ 
 	 * @param  string $context View or edit context.
 	 * @return string
 	 */
-	public function get_date_created( $context = 'view' ) {
-		return $this->get_prop( 'date_created', $context );
+	public function get_date_created($context = 'view') {
+		return $this->get_prop('date_created', $context);
     }
 
     /**
@@ -142,11 +142,11 @@ 
 	 * @param  string $context View or edit context.
 	 * @return string
 	 */
-	public function get_date_created_gmt( $context = 'view' ) {
-        $date = $this->get_date_created( $context );
+	public function get_date_created_gmt($context = 'view') {
+        $date = $this->get_date_created($context);
 
-        if ( $date ) {
-            $date = get_gmt_from_date( $date );
+        if ($date) {
+            $date = get_gmt_from_date($date);
         }
 		return $date;
     }
@@ -158,8 +158,8 @@ 
 	 * @param  string $context View or edit context.
 	 * @return string
 	 */
-	public function get_date_modified( $context = 'view' ) {
-		return $this->get_prop( 'date_modified', $context );
+	public function get_date_modified($context = 'view') {
+		return $this->get_prop('date_modified', $context);
     }
 
     /**
@@ -169,11 +169,11 @@ 
 	 * @param  string $context View or edit context.
 	 * @return string
 	 */
-	public function get_date_modified_gmt( $context = 'view' ) {
-        $date = $this->get_date_modified( $context );
+	public function get_date_modified_gmt($context = 'view') {
+        $date = $this->get_date_modified($context);
 
-        if ( $date ) {
-            $date = get_gmt_from_date( $date );
+        if ($date) {
+            $date = get_gmt_from_date($date);
         }
 		return $date;
     }
@@ -185,8 +185,8 @@ 
 	 * @param  string $context View or edit context.
 	 * @return string
 	 */
-	public function get_name( $context = 'view' ) {
-		return $this->get_prop( 'name', $context );
+	public function get_name($context = 'view') {
+		return $this->get_prop('name', $context);
     }
 
     /**
@@ -196,8 +196,8 @@ 
 	 * @param  string $context View or edit context.
 	 * @return string
 	 */
-	public function get_title( $context = 'view' ) {
-		return $this->get_name( $context );
+	public function get_title($context = 'view') {
+		return $this->get_name($context);
 	}
 
     /**
@@ -207,8 +207,8 @@ 
 	 * @param  string $context View or edit context.
 	 * @return int
 	 */
-	public function get_author( $context = 'view' ) {
-		return (int) $this->get_prop( 'author', $context );
+	public function get_author($context = 'view') {
+		return (int) $this->get_prop('author', $context);
     }
 
     /**
@@ -218,21 +218,21 @@ 
 	 * @param  string $context View or edit context.
 	 * @return array
 	 */
-	public function get_elements( $context = 'view' ) {
-		$elements = $this->get_prop( 'elements', $context );
+	public function get_elements($context = 'view') {
+		$elements = $this->get_prop('elements', $context);
 
-		if ( empty( $elements ) || ! is_array( $elements ) ) {
-            return wpinv_get_data( 'sample-payment-form' );
+		if (empty($elements) || !is_array($elements)) {
+            return wpinv_get_data('sample-payment-form');
 		}
 
 		// Ensure that all required elements exist.
 		$_elements = array();
-		foreach ( $elements as $element ) {
+		foreach ($elements as $element) {
 
-			if ( $element['type'] == 'pay_button' && ! $this->has_element_type( 'gateway_select' ) ) {
+			if ($element['type'] == 'pay_button' && !$this->has_element_type('gateway_select')) {
 
 				$_elements[] = array(
-					'text'    => __( 'Select Payment Method', 'invoicing' ),
+					'text'    => __('Select Payment Method', 'invoicing'),
 					'id'      => 'gtscicd',
 					'name'    => 'gtscicd',
 					'type'    => 'gateway_select',
@@ -257,22 +257,22 @@ 
 	 * @param  string $return objects or arrays.
 	 * @return GetPaid_Form_Item[]
 	 */
-	public function get_items( $context = 'view', $return = 'objects' ) {
-		$items = $this->get_prop( 'items', $context );
+	public function get_items($context = 'view', $return = 'objects') {
+		$items = $this->get_prop('items', $context);
 
-		if ( empty( $items ) || ! is_array( $items ) ) {
-            $items = wpinv_get_data( 'sample-payment-form-items' );
+		if (empty($items) || !is_array($items)) {
+            $items = wpinv_get_data('sample-payment-form-items');
 		}
 
 		// Convert the items.
 		$prepared = array();
 
-		foreach ( $items as $key => $value ) {
+		foreach ($items as $key => $value) {
 
 			// Form items.
-			if ( $value instanceof GetPaid_Form_Item ) {
+			if ($value instanceof GetPaid_Form_Item) {
 
-				if ( $value->can_purchase() ) {
+				if ($value->can_purchase()) {
 					$prepared[] = $value;
 				}
 
@@ -281,16 +281,16 @@ 
 			}
 
 			// $item_id => $quantity (buy buttons)
-			if ( is_numeric( $key ) && is_numeric( $value ) ) {
-				$item = new GetPaid_Form_Item( $key );
+			if (is_numeric($key) && is_numeric($value)) {
+				$item = new GetPaid_Form_Item($key);
 
-				if ( $item->can_purchase() ) {
+				if ($item->can_purchase()) {
 
 					$value = (float) $value;
-					$item->set_quantity( $value );
-					if ( 0 == $value ) {
-						$item->set_quantity( 1 );
-						$item->set_allow_quantities( true );
+					$item->set_quantity($value);
+					if (0 == $value) {
+						$item->set_quantity(1);
+						$item->set_allow_quantities(true);
 					}
 
 					$prepared[] = $item;
@@ -300,33 +300,33 @@ 
 			}
 
 			// Items saved via payment forms editor.
-			if ( is_array( $value ) && isset( $value['id'] ) ) {
+			if (is_array($value) && isset($value['id'])) {
 
-				$item = new GetPaid_Form_Item( $value['id'] );
+				$item = new GetPaid_Form_Item($value['id']);
 
-				if ( ! $item->can_purchase() ) {
+				if (!$item->can_purchase()) {
 					continue;
 				}
 
 				// Sub-total (Cart items).
-				if ( isset( $value['subtotal'] ) ) {
-					$item->set_price( $value['subtotal'] );
+				if (isset($value['subtotal'])) {
+					$item->set_price($value['subtotal']);
 				}
 
-				if ( isset( $value['quantity'] ) ) {
-					$item->set_quantity( $value['quantity'] );
+				if (isset($value['quantity'])) {
+					$item->set_quantity($value['quantity']);
 				}
 
-				if ( isset( $value['allow_quantities'] ) ) {
-					$item->set_allow_quantities( $value['allow_quantities'] );
+				if (isset($value['allow_quantities'])) {
+					$item->set_allow_quantities($value['allow_quantities']);
 				}
 
-				if ( isset( $value['required'] ) ) {
-					$item->set_is_required( $value['required'] );
+				if (isset($value['required'])) {
+					$item->set_is_required($value['required']);
 				}
 
-				if ( isset( $value['description'] ) ) {
-					$item->set_custom_description( $value['description'] );
+				if (isset($value['description'])) {
+					$item->set_custom_description($value['description']);
 				}
 
 				$prepared[] = $item;
@@ -335,14 +335,14 @@ 
 			}
 
 			// $item_id => array( 'price' => 10 ) (item variations)
-			if ( is_numeric( $key ) && is_array( $value ) ) {
-				$item = new GetPaid_Form_Item( $key );
+			if (is_numeric($key) && is_array($value)) {
+				$item = new GetPaid_Form_Item($key);
 
-				if ( isset( $value['price'] ) && $item->user_can_set_their_price() ) {
-					$item->set_price( $value['price'] );
+				if (isset($value['price']) && $item->user_can_set_their_price()) {
+					$item->set_price($value['price']);
 				}
 
-				if ( $item->can_purchase() ) {
+				if ($item->can_purchase()) {
 					$prepared[] = $item;
 				}
 
@@ -350,12 +350,12 @@ 
 			}
 }
 
-		if ( 'objects' == $return && 'view' == $context ) {
+		if ('objects' == $return && 'view' == $context) {
 			return $prepared;
 		}
 
 		$items = array();
-		foreach ( $prepared as $item ) {
+		foreach ($prepared as $item) {
 			$items[] = $item->prepare_data_for_use();
 		}
 
@@ -369,14 +369,14 @@ 
 	 * @param  int $item_id The item id to return.
 	 * @return GetPaid_Form_Item|bool
 	 */
-	public function get_item( $item_id ) {
+	public function get_item($item_id) {
 
-		if ( empty( $item_id ) || ! is_numeric( $item_id ) ) {
+		if (empty($item_id) || !is_numeric($item_id)) {
 			return false;
 		}
 
-		foreach ( $this->get_items() as $item ) {
-			if ( $item->get_id() == (int) $item_id ) {
+		foreach ($this->get_items() as $item) {
+			if ($item->get_id() == (int) $item_id) {
 				return $item;
 			}
 		}
@@ -392,15 +392,15 @@ 
 	 * @param  string $element_type The element type to return.
 	 * @return array|bool
 	 */
-	public function get_element_type( $element_type ) {
+	public function get_element_type($element_type) {
 
-		if ( empty( $element_type ) || ! is_scalar( $element_type ) ) {
+		if (empty($element_type) || !is_scalar($element_type)) {
 			return false;
 		}
 
-		foreach ( $this->get_prop( 'elements' ) as $element ) {
+		foreach ($this->get_prop('elements') as $element) {
 
-			if ( $element['type'] == $element_type ) {
+			if ($element['type'] == $element_type) {
 				return $element;
 			}
 }
@@ -416,8 +416,8 @@ 
 	 * @param  string $context View or edit context.
 	 * @return float
 	 */
-	public function get_earned( $context = 'view' ) {
-		return $this->get_prop( 'earned', $context );
+	public function get_earned($context = 'view') {
+		return $this->get_prop('earned', $context);
 	}
 
 	/**
@@ -427,8 +427,8 @@ 
 	 * @param  string $context View or edit context.
 	 * @return float
 	 */
-	public function get_refunded( $context = 'view' ) {
-		return $this->get_prop( 'refunded', $context );
+	public function get_refunded($context = 'view') {
+		return $this->get_prop('refunded', $context);
 	}
 
 	/**
@@ -438,8 +438,8 @@ 
 	 * @param  string $context View or edit context.
 	 * @return float
 	 */
-	public function get_cancelled( $context = 'view' ) {
-		return $this->get_prop( 'cancelled', $context );
+	public function get_cancelled($context = 'view') {
+		return $this->get_prop('cancelled', $context);
 	}
 
 	/**
@@ -449,8 +449,8 @@ 
 	 * @param  string $context View or edit context.
 	 * @return float
 	 */
-	public function get_failed( $context = 'view' ) {
-		return $this->get_prop( 'failed', $context );
+	public function get_failed($context = 'view') {
+		return $this->get_prop('failed', $context);
 	}
 
 	/**
@@ -461,8 +461,8 @@ 
 	 * @return string
 	 */
 	public function get_currency() {
-		$currency = empty( $this->invoice ) ? wpinv_get_currency() : $this->invoice->get_currency();
-		return apply_filters( 'getpaid-payment-form-currency', $currency, $this );
+		$currency = empty($this->invoice) ? wpinv_get_currency() : $this->invoice->get_currency();
+		return apply_filters('getpaid-payment-form-currency', $currency, $this);
 	}
 
     /*
@@ -480,8 +480,8 @@ 
 	 *
 	 * @since 1.0.19
 	 */
-	public function set_version( $value ) {
-		$this->set_prop( 'version', $value );
+	public function set_version($value) {
+		$this->set_prop('version', $value);
     }
 
     /**
@@ -491,11 +491,11 @@ 
 	 * @param string $value Value to set.
      * @return bool Whether or not the date was set.
 	 */
-	public function set_date_created( $value ) {
-        $date = strtotime( $value );
+	public function set_date_created($value) {
+        $date = strtotime($value);
 
-        if ( $date ) {
-            $this->set_prop( 'date_created', date( 'Y-m-d H:i:s', $date ) );
+        if ($date) {
+            $this->set_prop('date_created', date('Y-m-d H:i:s', $date));
             return true;
         }
 
@@ -510,11 +510,11 @@ 
 	 * @param string $value Value to set.
      * @return bool Whether or not the date was set.
 	 */
-	public function set_date_modified( $value ) {
-        $date = strtotime( $value );
+	public function set_date_modified($value) {
+        $date = strtotime($value);
 
-        if ( $date ) {
-            $this->set_prop( 'date_modified', date( 'Y-m-d H:i:s', $date ) );
+        if ($date) {
+            $this->set_prop('date_modified', date('Y-m-d H:i:s', $date));
             return true;
         }
 
@@ -528,8 +528,8 @@ 
 	 * @since 1.0.19
 	 * @param  string $value New name.
 	 */
-	public function set_name( $value ) {
-		$this->set_prop( 'name', sanitize_text_field( $value ) );
+	public function set_name($value) {
+		$this->set_prop('name', sanitize_text_field($value));
     }
 
     /**
@@ -538,8 +538,8 @@ 
 	 * @since 1.0.19
 	 * @param  string $value New name.
 	 */
-	public function set_title( $value ) {
-		$this->set_name( $value );
+	public function set_title($value) {
+		$this->set_name($value);
     }
 
     /**
@@ -548,8 +548,8 @@ 
 	 * @since 1.0.19
 	 * @param  int $value New author.
 	 */
-	public function set_author( $value ) {
-		$this->set_prop( 'author', (int) $value );
+	public function set_author($value) {
+		$this->set_prop('author', (int) $value);
 	}
 
 	/**
@@ -559,9 +559,9 @@ 
 	 * @sinve 2.3.4 Array values sanitized.
 	 * @param  array $value Form elements.
 	 */
-	public function set_elements( $value ) {
-		if ( is_array( $value ) ) {
-			$this->set_prop( 'elements', wp_kses_post_deep( $value ) );
+	public function set_elements($value) {
+		if (is_array($value)) {
+			$this->set_prop('elements', wp_kses_post_deep($value));
 		}
 	}
 
@@ -572,36 +572,36 @@ 
 	 *
 	 * @return mixed
 	 */
-	public function sanitize_array_values( $value ) {
+	public function sanitize_array_values($value) {
 
 		// sanitize
-		if ( ! empty( $value ) ) {
+		if (!empty($value)) {
 
-			foreach ( $value as $key => $val_arr ) {
+			foreach ($value as $key => $val_arr) {
 
-				if ( is_array( $val_arr ) ) {
+				if (is_array($val_arr)) {
 					// check if we have sub array items.
 					$sub_arr = array();
-					foreach ( $val_arr as $key2 => $val2 ) {
-						if ( is_array( $val2 ) ) {
-							$sub_arr[ $key2 ] = $this->sanitize_array_values( $val2 );
-							unset( $val_arr[ $key ][ $key2 ] );
+					foreach ($val_arr as $key2 => $val2) {
+						if (is_array($val2)) {
+							$sub_arr[$key2] = $this->sanitize_array_values($val2);
+							unset($val_arr[$key][$key2]);
 						}
 					}
 
 					// we allow some html in description so we sanitize it separately.
-					$help_text = ! empty( $val_arr['description'] ) ? wp_kses_post( $val_arr['description'] ) : '';
+					$help_text = !empty($val_arr['description']) ? wp_kses_post($val_arr['description']) : '';
 
 					// sanitize array elements
-					$value[ $key ] = array_map( 'sanitize_text_field', $val_arr );
+					$value[$key] = array_map('sanitize_text_field', $val_arr);
 
 					// add back the description if set
-					if ( isset( $val_arr['description'] ) ) {
-$value[ $key ]['description'] = $help_text;}
+					if (isset($val_arr['description'])) {
+$value[$key]['description'] = $help_text; }
 
 					// add back sub array items after its been sanitized.
-					if ( ! empty( $sub_arr ) ) {
-						$value[ $key ] = array_merge( $value[ $key ], $sub_arr );
+					if (!empty($sub_arr)) {
+						$value[$key] = array_merge($value[$key], $sub_arr);
 					}
 				}
 }
@@ -616,9 +616,9 @@ 
 	 * @since 1.0.19
 	 * @param  array $value Form elements.
 	 */
-	public function set_items( $value ) {
-		if ( is_array( $value ) ) {
-			$this->set_prop( 'items', $value );
+	public function set_items($value) {
+		if (is_array($value)) {
+			$this->set_prop('items', $value);
 		}
 	}
 
@@ -628,9 +628,9 @@ 
 	 * @since 1.0.19
 	 * @param  float $value Amount earned.
 	 */
-	public function set_earned( $value ) {
-		$value = max( (float) $value, 0 );
-		$this->set_prop( 'earned', $value );
+	public function set_earned($value) {
+		$value = max((float) $value, 0);
+		$this->set_prop('earned', $value);
 	}
 
 	/**
@@ -639,9 +639,9 @@ 
 	 * @since 1.0.19
 	 * @param  float $value Amount refunded.
 	 */
-	public function set_refunded( $value ) {
-		$value = max( (float) $value, 0 );
-		$this->set_prop( 'refunded', $value );
+	public function set_refunded($value) {
+		$value = max((float) $value, 0);
+		$this->set_prop('refunded', $value);
 	}
 
 	/**
@@ -650,9 +650,9 @@ 
 	 * @since 1.0.19
 	 * @param  float $value Amount cancelled.
 	 */
-	public function set_cancelled( $value ) {
-		$value = max( (float) $value, 0 );
-		$this->set_prop( 'cancelled', $value );
+	public function set_cancelled($value) {
+		$value = max((float) $value, 0);
+		$this->set_prop('cancelled', $value);
 	}
 
 	/**
@@ -661,9 +661,9 @@ 
 	 * @since 1.0.19
 	 * @param  float $value Amount cancelled.
 	 */
-	public function set_failed( $value ) {
-		$value = max( (float) $value, 0 );
-		$this->set_prop( 'failed', $value );
+	public function set_failed($value) {
+		$value = max((float) $value, 0);
+		$this->set_prop('failed', $value);
 	}
 
     /**
@@ -672,11 +672,11 @@ 
      * @deprecated
 	 * @return int item id
      */
-    public function create( $data = array() ) {
+    public function create($data = array()) {
 
 		// Set the properties.
-		if ( is_array( $data ) ) {
-			$this->set_props( $data );
+		if (is_array($data)) {
+			$this->set_props($data);
 		}
 
 		// Save the item.
@@ -690,8 +690,8 @@ 
      * @deprecated
 	 * @return int item id
      */
-    public function update( $data = array() ) {
-        return $this->create( $data );
+    public function update($data = array()) {
+        return $this->create($data);
     }
 
     /*
@@ -711,7 +711,7 @@ 
 	 */
     public function is_default() {
         $is_default = $this->get_id() == wpinv_get_default_payment_form();
-        return (bool) apply_filters( 'wpinv_is_default_payment_form', $is_default, $this->get_id(), $this );
+        return (bool) apply_filters('wpinv_is_default_payment_form', $is_default, $this->get_id(), $this);
 	}
 
     /**
@@ -723,11 +723,11 @@ 
     public function is_active() {
         $is_active = 0 !== (int) $this->get_id();
 
-        if ( $is_active && ! current_user_can( 'edit_post', $this->get_id() ) && $this->get_status() != 'publish' ) {
+        if ($is_active && !current_user_can('edit_post', $this->get_id()) && $this->get_status() != 'publish') {
             $is_active = false;
         }
 
-        return (bool) apply_filters( 'wpinv_is_payment_form_active', $is_active, $this );
+        return (bool) apply_filters('wpinv_is_payment_form_active', $is_active, $this);
 	}
 
 	/**
@@ -736,8 +736,8 @@ 
 	 * @since 1.0.19
 	 * @return bool
 	 */
-    public function has_item( $item_id ) {
-        return false !== $this->get_item( $item_id );
+    public function has_item($item_id) {
+        return false !== $this->get_item($item_id);
 	}
 
 	/**
@@ -746,8 +746,8 @@ 
 	 * @since 1.0.19
 	 * @return bool
 	 */
-    public function has_element_type( $element_type ) {
-        return false !== $this->get_element_type( $element_type );
+    public function has_element_type($element_type) {
+        return false !== $this->get_element_type($element_type);
 	}
 
 	/**
@@ -758,13 +758,13 @@ 
 	 */
     public function is_recurring() {
 
-		if ( ! empty( $this->invoice ) ) {
+		if (!empty($this->invoice)) {
 			return $this->invoice->is_recurring();
 		}
 
-		foreach ( $this->get_items() as $item ) {
+		foreach ($this->get_items() as $item) {
 
-			if ( $item->is_recurring() ) {
+			if ($item->is_recurring()) {
 				return true;
 			}
 }
@@ -777,7 +777,7 @@ 
 	 *
 	 * @since 1.0.19
 	 */
-    public function get_html( $extra_markup = '' ) {
+    public function get_html($extra_markup = '') {
 
 		// Return the HTML.
 		return wpinv_get_template_html(
@@ -795,7 +795,7 @@ 
 	 *
 	 * @since 1.0.19
 	 */
-    public function display( $extra_markup = '' ) {
+    public function display($extra_markup = '') {
 		wpinv_get_template(
 			'payment-forms/form.php',
 			array(

templates/invoice-history.php

Indentation +73 added lines, -73 removed lines

@@ -42,92 +42,92 @@ 
 					<tr class="wpinv-item wpinv-item-<?php echo esc_attr( $invoice->get_status() ); ?>">
 						<?php
 
-							foreach ( wpinv_get_user_invoices_columns( $post_type ) as $column_id => $column_name ) :
+                            foreach ( wpinv_get_user_invoices_columns( $post_type ) as $column_id => $column_name ) :
 
-							$column_id = sanitize_html_class( $column_id );
-							$class     = empty( $column_name['class'] ) ? '' : sanitize_html_class( $column_name['class'] );
+                            $column_id = sanitize_html_class( $column_id );
+                            $class     = empty( $column_name['class'] ) ? '' : sanitize_html_class( $column_name['class'] );
 
-							echo "<td class='" . esc_attr( $column_id . ' ' . $class ) . "'>";
-							switch ( $column_id ) {
+                            echo "<td class='" . esc_attr( $column_id . ' ' . $class ) . "'>";
+                            switch ( $column_id ) {
 
-								case 'invoice-number':
-									echo wp_kses_post( wpinv_invoice_link( $invoice ) );
-									break;
+                                case 'invoice-number':
+                                    echo wp_kses_post( wpinv_invoice_link( $invoice ) );
+                                    break;
 
-								case 'created-date':
-									echo esc_html( getpaid_format_date_value( $invoice->get_date_created() ) );
-									break;
+                                case 'created-date':
+                                    echo esc_html( getpaid_format_date_value( $invoice->get_date_created() ) );
+                                    break;
 
-								case 'payment-date':
-									if ( $invoice->needs_payment() ) {
-										echo '&mdash;';
-										} else {
-										echo esc_html( getpaid_format_date_value( $invoice->get_date_completed() ) );
-										}
+                                case 'payment-date':
+                                    if ( $invoice->needs_payment() ) {
+                                        echo '&mdash;';
+                                        } else {
+                                        echo esc_html( getpaid_format_date_value( $invoice->get_date_completed() ) );
+                                        }
 
-									break;
+                                    break;
 
-								case 'invoice-status':
-									echo wp_kses_post( $invoice->get_status_label_html() );
+                                case 'invoice-status':
+                                    echo wp_kses_post( $invoice->get_status_label_html() );
 
-									break;
+                                    break;
 
-								case 'invoice-total':
-									wpinv_the_price( $invoice->get_total(), $invoice->get_currency() );
+                                case 'invoice-total':
+                                    wpinv_the_price( $invoice->get_total(), $invoice->get_currency() );
 
-									break;
+                                    break;
 
-								case 'invoice-actions':
-									$actions = array(
+                                case 'invoice-actions':
+                                    $actions = array(
 
-										'pay'   => array(
-											'url'   => $invoice->get_checkout_payment_url(),
-											'name'  => __( 'Pay Now', 'invoicing' ),
-											'class' => 'btn-success',
-										),
+                                        'pay'   => array(
+                                            'url'   => $invoice->get_checkout_payment_url(),
+                                            'name'  => __( 'Pay Now', 'invoicing' ),
+                                            'class' => 'btn-success',
+                                        ),
 
-										'print' => array(
-											'url'   => $invoice->get_view_url(),
-											'name'  => __( 'View', 'invoicing' ),
-											'class' => 'btn-secondary',
-											'attrs' => 'target="_blank"',
-										),
-									);
+                                        'print' => array(
+                                            'url'   => $invoice->get_view_url(),
+                                            'name'  => __( 'View', 'invoicing' ),
+                                            'class' => 'btn-secondary',
+                                            'attrs' => 'target="_blank"',
+                                        ),
+                                    );
 
-									if ( ! $invoice->needs_payment() ) {
-										unset( $actions['pay'] );
-										}
+                                    if ( ! $invoice->needs_payment() ) {
+                                        unset( $actions['pay'] );
+                                        }
 
-									if ( $invoice->needs_payment() ) {
-										$actions['delete'] = array(
-											'url'   => getpaid_get_authenticated_action_url( 'delete_invoice', add_query_arg( 'invoice_id', $invoice->get_id() ) ),
-											'name'  => __( 'Delete', 'invoicing' ),
-											'class' => 'btn-danger',
-										);
-										}
+                                    if ( $invoice->needs_payment() ) {
+                                        $actions['delete'] = array(
+                                            'url'   => getpaid_get_authenticated_action_url( 'delete_invoice', add_query_arg( 'invoice_id', $invoice->get_id() ) ),
+                                            'name'  => __( 'Delete', 'invoicing' ),
+                                            'class' => 'btn-danger',
+                                        );
+                                        }
 
-									$actions = apply_filters( 'wpinv_user_invoices_actions', $actions, $invoice, $post_type );
+                                    $actions = apply_filters( 'wpinv_user_invoices_actions', $actions, $invoice, $post_type );
 
-									foreach ( $actions as $key => $action ) {
-										$class = ! empty( $action['class'] ) ? sanitize_html_class( $action['class'] ) : '';
-										echo '<a href="' . esc_url( $action['url'] ) . '" class="btn btn-sm btn-block ' . esc_attr( $class . ' ' . sanitize_html_class( $key ) ) . '" ' . ( ! empty( $action['attrs'] ) ? esc_html( $action['attrs'] ) : '' ) . '>' . esc_attr( $action['name'] ) . '</a>';
-										}
+                                    foreach ( $actions as $key => $action ) {
+                                        $class = ! empty( $action['class'] ) ? sanitize_html_class( $action['class'] ) : '';
+                                        echo '<a href="' . esc_url( $action['url'] ) . '" class="btn btn-sm btn-block ' . esc_attr( $class . ' ' . sanitize_html_class( $key ) ) . '" ' . ( ! empty( $action['attrs'] ) ? esc_html( $action['attrs'] ) : '' ) . '>' . esc_attr( $action['name'] ) . '</a>';
+                                        }
 
-									break;
+                                    break;
 
-								default:
-									do_action( "wpinv_user_invoices_column_$column_id", $invoice );
-									break;
+                                default:
+                                    do_action( "wpinv_user_invoices_column_$column_id", $invoice );
+                                    break;
 
 
-								}
+                                }
 
-							do_action( "wpinv_user_invoices_column_after_$column_id", $invoice );
+                            do_action( "wpinv_user_invoices_column_after_$column_id", $invoice );
 
-							echo '</td>';
+                            echo '</td>';
 
-							endforeach;
-						?>
+                            endforeach;
+                        ?>
 					</tr>
 
 				<?php endforeach; ?>
@@ -141,18 +141,18 @@ 
 	<?php if ( 1 < $invoices->max_num_pages ) : ?>
 		<div class="invoicing-Pagination">
 			<?php
-			$big = 999999;
-
-			echo wp_kses_post(
-				paginate_links(
-					array(
-						'base'   => str_replace( $big, '%#%', esc_url( get_pagenum_link( $big ) ) ),
-						'format' => '?paged=%#%',
-						'total'  => $invoices->max_num_pages,
-					)
-				)
+            $big = 999999;
+
+            echo wp_kses_post(
+                paginate_links(
+                    array(
+                        'base'   => str_replace( $big, '%#%', esc_url( get_pagenum_link( $big ) ) ),
+                        'format' => '?paged=%#%',
+                        'total'  => $invoices->max_num_pages,
+                    )
+                )
             );
-			?>
+            ?>
 		</div>
 	<?php endif; ?>
 

Spacing +37 added lines, -37 removed lines

@@ -7,13 +7,13 @@ 
  * @version 1.0.19
  */
 
-defined( 'ABSPATH' ) || exit;
+defined('ABSPATH') || exit;
 
 // Current page.
-$current_page   = empty( $_GET['page'] ) ? 1 : absint( $_GET['page'] );
+$current_page = empty($_GET['page']) ? 1 : absint($_GET['page']);
 
 // Fires before displaying user invoices.
-do_action( 'wpinv_before_user_invoices', $invoices->invoices, $invoices->total, $invoices->max_num_pages, $post_type );
+do_action('wpinv_before_user_invoices', $invoices->invoices, $invoices->total, $invoices->max_num_pages, $post_type);
 
 wpinv_print_errors();
 
@@ -21,15 +21,15 @@ 
 
 
 	<div class="table-responsive">
-		<table class="table table-bordered table-hover getpaid-user-invoices <?php echo esc_attr( $post_type ); ?>">
+		<table class="table table-bordered table-hover getpaid-user-invoices <?php echo esc_attr($post_type); ?>">
 
 
 			<thead>
 				<tr>
 
-					<?php foreach ( wpinv_get_user_invoices_columns( $post_type ) as $column_id => $column_name ) : ?>
-						<th class="<?php echo esc_attr( $column_id ); ?> <?php echo ( ! empty( $column_name['class'] ) ? sanitize_html_class( $column_name['class'] ) : ''); ?> border-bottom-0">
-							<span class="nobr"><?php echo esc_html( $column_name['title'] ); ?></span>
+					<?php foreach (wpinv_get_user_invoices_columns($post_type) as $column_id => $column_name) : ?>
+						<th class="<?php echo esc_attr($column_id); ?> <?php echo (!empty($column_name['class']) ? sanitize_html_class($column_name['class']) : ''); ?> border-bottom-0">
+							<span class="nobr"><?php echo esc_html($column_name['title']); ?></span>
 						</th>
 					<?php endforeach; ?>
 
@@ -39,43 +39,43 @@ 
 
 
 			<tbody>
-				<?php foreach ( $invoices->invoices as $invoice ) : ?>
+				<?php foreach ($invoices->invoices as $invoice) : ?>
 
-					<tr class="wpinv-item wpinv-item-<?php echo esc_attr( $invoice->get_status() ); ?>">
+					<tr class="wpinv-item wpinv-item-<?php echo esc_attr($invoice->get_status()); ?>">
 						<?php
 
-							foreach ( wpinv_get_user_invoices_columns( $post_type ) as $column_id => $column_name ) :
+							foreach (wpinv_get_user_invoices_columns($post_type) as $column_id => $column_name) :
 
-							$column_id = sanitize_html_class( $column_id );
-							$class     = empty( $column_name['class'] ) ? '' : sanitize_html_class( $column_name['class'] );
+							$column_id = sanitize_html_class($column_id);
+							$class     = empty($column_name['class']) ? '' : sanitize_html_class($column_name['class']);
 
-							echo "<td class='" . esc_attr( $column_id . ' ' . $class ) . "'>";
-							switch ( $column_id ) {
+							echo "<td class='" . esc_attr($column_id . ' ' . $class) . "'>";
+							switch ($column_id) {
 
 								case 'invoice-number':
-									echo wp_kses_post( wpinv_invoice_link( $invoice ) );
+									echo wp_kses_post(wpinv_invoice_link($invoice));
 									break;
 
 								case 'created-date':
-									echo esc_html( getpaid_format_date_value( $invoice->get_date_created() ) );
+									echo esc_html(getpaid_format_date_value($invoice->get_date_created()));
 									break;
 
 								case 'payment-date':
-									if ( $invoice->needs_payment() ) {
+									if ($invoice->needs_payment()) {
 										echo '&mdash;';
 										} else {
-										echo esc_html( getpaid_format_date_value( $invoice->get_date_completed() ) );
+										echo esc_html(getpaid_format_date_value($invoice->get_date_completed()));
 										}
 
 									break;
 
 								case 'invoice-status':
-									echo wp_kses_post( $invoice->get_status_label_html() );
+									echo wp_kses_post($invoice->get_status_label_html());
 
 									break;
 
 								case 'invoice-total':
-									wpinv_the_price( $invoice->get_total(), $invoice->get_currency() );
+									wpinv_the_price($invoice->get_total(), $invoice->get_currency());
 
 									break;
 
@@ -84,47 +84,47 @@ 
 
 										'pay'   => array(
 											'url'   => $invoice->get_checkout_payment_url(),
-											'name'  => __( 'Pay Now', 'invoicing' ),
+											'name'  => __('Pay Now', 'invoicing'),
 											'class' => 'btn-success',
 										),
 
 										'print' => array(
 											'url'   => $invoice->get_view_url(),
-											'name'  => __( 'View', 'invoicing' ),
+											'name'  => __('View', 'invoicing'),
 											'class' => 'btn-secondary',
 											'attrs' => 'target="_blank"',
 										),
 									);
 
-									if ( ! $invoice->needs_payment() ) {
-										unset( $actions['pay'] );
+									if (!$invoice->needs_payment()) {
+										unset($actions['pay']);
 										}
 
-									if ( $invoice->needs_payment() ) {
+									if ($invoice->needs_payment()) {
 										$actions['delete'] = array(
-											'url'   => getpaid_get_authenticated_action_url( 'delete_invoice', add_query_arg( 'invoice_id', $invoice->get_id() ) ),
-											'name'  => __( 'Delete', 'invoicing' ),
+											'url'   => getpaid_get_authenticated_action_url('delete_invoice', add_query_arg('invoice_id', $invoice->get_id())),
+											'name'  => __('Delete', 'invoicing'),
 											'class' => 'btn-danger',
 										);
 										}
 
-									$actions = apply_filters( 'wpinv_user_invoices_actions', $actions, $invoice, $post_type );
+									$actions = apply_filters('wpinv_user_invoices_actions', $actions, $invoice, $post_type);
 
-									foreach ( $actions as $key => $action ) {
-										$class = ! empty( $action['class'] ) ? sanitize_html_class( $action['class'] ) : '';
-										echo '<a href="' . esc_url( $action['url'] ) . '" class="btn btn-sm btn-block ' . esc_attr( $class . ' ' . sanitize_html_class( $key ) ) . '" ' . ( ! empty( $action['attrs'] ) ? esc_html( $action['attrs'] ) : '' ) . '>' . esc_attr( $action['name'] ) . '</a>';
+									foreach ($actions as $key => $action) {
+										$class = !empty($action['class']) ? sanitize_html_class($action['class']) : '';
+										echo '<a href="' . esc_url($action['url']) . '" class="btn btn-sm btn-block ' . esc_attr($class . ' ' . sanitize_html_class($key)) . '" ' . (!empty($action['attrs']) ? esc_html($action['attrs']) : '') . '>' . esc_attr($action['name']) . '</a>';
 										}
 
 									break;
 
 								default:
-									do_action( "wpinv_user_invoices_column_$column_id", $invoice );
+									do_action("wpinv_user_invoices_column_$column_id", $invoice);
 									break;
 
 
 								}
 
-							do_action( "wpinv_user_invoices_column_after_$column_id", $invoice );
+							do_action("wpinv_user_invoices_column_after_$column_id", $invoice);
 
 							echo '</td>';
 
@@ -138,9 +138,9 @@ 
 		</table>
 	</div>
 
-	<?php do_action( 'wpinv_before_user_invoices_pagination' ); ?>
+	<?php do_action('wpinv_before_user_invoices_pagination'); ?>
 
-	<?php if ( 1 < $invoices->max_num_pages ) : ?>
+	<?php if (1 < $invoices->max_num_pages) : ?>
 		<div class="invoicing-Pagination">
 			<?php
 			$big = 999999;
@@ -148,7 +148,7 @@ 
 			echo wp_kses_post(
 				paginate_links(
 					array(
-						'base'   => str_replace( $big, '%#%', esc_url( get_pagenum_link( $big ) ) ),
+						'base'   => str_replace($big, '%#%', esc_url(get_pagenum_link($big))),
 						'format' => '?paged=%#%',
 						'total'  => $invoices->max_num_pages,
 					)
@@ -158,4 +158,4 @@ 
 		</div>
 	<?php endif; ?>
 
-<?php do_action( 'wpinv_after_user_invoices', $invoices->invoices, $invoices->total, $invoices->max_num_pages, $post_type ); ?>
+<?php do_action('wpinv_after_user_invoices', $invoices->invoices, $invoices->total, $invoices->max_num_pages, $post_type); ?>

templates/emails/invoice-details.php

Spacing +13 added lines, -13 removed lines

@@ -8,45 +8,45 @@ 
  * @var WPInv_Invoice $invoice
  * @var array $columns
  */
-defined( 'ABSPATH' ) || exit;
+defined('ABSPATH') || exit;
 
-$meta_data = getpaid_get_invoice_meta( $invoice );
+$meta_data = getpaid_get_invoice_meta($invoice);
 
-if ( isset( $meta_data['status'] ) ) {
+if (isset($meta_data['status'])) {
     $meta_data['status']['value'] = $invoice->get_status_nicename();
 }
 
-do_action( 'wpinv_email_before_invoice_details', $invoice, $sent_to_admin );
+do_action('wpinv_email_before_invoice_details', $invoice, $sent_to_admin);
 
 ?>
 
 <div id="wpinv-email-details">
 
     <h3 class="invoice-details-title">
-        <?php echo sprintf( esc_html__( '%s Details', 'invoicing' ), esc_html( ucfirst( $invoice->get_invoice_quote_type() ) ) ); ?>
+        <?php echo sprintf(esc_html__('%s Details', 'invoicing'), esc_html(ucfirst($invoice->get_invoice_quote_type()))); ?>
     </h3>
 
     <table class="table table-bordered table-sm">
 
-        <?php foreach ( $meta_data as $key => $data ) : ?>
+        <?php foreach ($meta_data as $key => $data) : ?>
 
-            <?php if ( ! empty( $data['value'] ) ) : ?>
+            <?php if (!empty($data['value'])) : ?>
 
-                <?php do_action( "getpaid_before_email_details_$key", $invoice, $data ); ?>
+                <?php do_action("getpaid_before_email_details_$key", $invoice, $data); ?>
 
-                <tr class="getpaid-email-details-<?php echo esc_attr( $key ); ?>">
+                <tr class="getpaid-email-details-<?php echo esc_attr($key); ?>">
 
                     <td class="getpaid-lable-td">
-                        <?php echo esc_html( $data['label'] ); ?>
+                        <?php echo esc_html($data['label']); ?>
                     </td>
 
                     <td class="getpaid-value-td">
-                        <span class="getpaid-invoice-meta-<?php echo esc_attr( $key ); ?>-value"><?php echo wp_kses_post( $data['value'] ); ?></span>
+                        <span class="getpaid-invoice-meta-<?php echo esc_attr($key); ?>-value"><?php echo wp_kses_post($data['value']); ?></span>
                     </td>
 
                 </tr>
 
-                <?php do_action( "getpaid_after_email_details_$key", $invoice, $data ); ?>
+                <?php do_action("getpaid_after_email_details_$key", $invoice, $data); ?>
 
             <?php endif; ?>
 
@@ -56,4 +56,4 @@ 
 
 </div>
 
-<?php do_action( 'wpinv_email_after_invoice_details', $invoice, $sent_to_admin ); ?>
+<?php do_action('wpinv_email_after_invoice_details', $invoice, $sent_to_admin); ?>