Inspection of "2.8.29" - AyeCode/invoicing - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#857)

by Kiran

created 2025-07-31 16:43 UTC

Status

Indentation +11 added lines, -11 removed lines patch added patch discarded remove patch

@@ -40,19 +40,19 @@
 block discarded – undo
         <?php
 
             if ( ! $invoice->exists() || $invoice->is_draft() ) {
-			$error = __( 'This invoice was deleted or is not visible.', 'invoicing' );
+            $error = __( 'This invoice was deleted or is not visible.', 'invoicing' );
             } else {
 
-			$user_id = get_current_user_id();
-			if ( wpinv_require_login_to_checkout() && empty( $user_id ) ) {
-				$error  = __( 'You must be logged in to view this invoice.', 'invoicing' );
-				$error .= sprintf(
-					' <a href="%s">%s</a>',
-					wp_login_url( $invoice->get_view_url() ),
-					__( 'Login.', 'invoicing' )
-				);
-			} else {
-				$error = __( 'This invoice is only viewable by clicking on the invoice link that was sent to you via email.', 'invoicing' );
+            $user_id = get_current_user_id();
+            if ( wpinv_require_login_to_checkout() && empty( $user_id ) ) {
+                $error  = __( 'You must be logged in to view this invoice.', 'invoicing' );
+                $error .= sprintf(
+                    ' <a href="%s">%s</a>',
+                    wp_login_url( $invoice->get_view_url() ),
+                    __( 'Login.', 'invoicing' )
+                );
+            } else {
+                $error = __( 'This invoice is only viewable by clicking on the invoice link that was sent to you via email.', 'invoicing' );
                 }
 }
 

Please login to merge, or discard this patch.

Spacing +15 added lines, -15 removed lines patch added patch discarded remove patch

@@ -7,11 +7,11 @@  discard block
 block discarded – undo
  * @version 1.0.19
  */
 
-defined( 'ABSPATH' ) || exit;
+defined('ABSPATH') || exit;
 
 // Fetch the invoice.
-if ( empty( $invoice ) ) {
-    $invoice = new WPInv_Invoice( $GLOBALS['post'] );
+if (empty($invoice)) {
+    $invoice = new WPInv_Invoice($GLOBALS['post']);
 }
 
 ?><!DOCTYPE html>
@@ -21,16 +21,16 @@  discard block
 block discarded – undo
 
     <head>
 
-		<meta charset="<?php bloginfo( 'charset' ); ?>">
+		<meta charset="<?php bloginfo('charset'); ?>">
         <meta name="viewport" content="width=device-width, initial-scale=1.0" >
 
         <meta name="robots" content="noindex,nofollow">
 
 		<link rel="profile" href="https://gmpg.org/xfn/11">
 
-        <title><?php esc_html_e( 'Invalid Access', 'invoicing' ); ?></title>
+        <title><?php esc_html_e('Invalid Access', 'invoicing'); ?></title>
 
-        <?php do_action( 'wpinv_invoice_print_head', $invoice ); ?>
+        <?php do_action('wpinv_invoice_print_head', $invoice); ?>
 
     </head>
 
@@ -39,20 +39,20 @@  discard block
 block discarded – undo
 
         <?php
 
-            if ( ! $invoice->exists() || $invoice->is_draft() ) {
-			$error = __( 'This invoice was deleted or is not visible.', 'invoicing' );
+            if (!$invoice->exists() || $invoice->is_draft()) {
+			$error = __('This invoice was deleted or is not visible.', 'invoicing');
             } else {
 
 			$user_id = get_current_user_id();
-			if ( wpinv_require_login_to_checkout() && empty( $user_id ) ) {
-				$error  = __( 'You must be logged in to view this invoice.', 'invoicing' );
+			if (wpinv_require_login_to_checkout() && empty($user_id)) {
+				$error  = __('You must be logged in to view this invoice.', 'invoicing');
 				$error .= sprintf(
 					' <a href="%s">%s</a>',
-					wp_login_url( $invoice->get_view_url() ),
-					__( 'Login.', 'invoicing' )
+					wp_login_url($invoice->get_view_url()),
+					__('Login.', 'invoicing')
 				);
 			} else {
-				$error = __( 'This invoice is only viewable by clicking on the invoice link that was sent to you via email.', 'invoicing' );
+				$error = __('This invoice is only viewable by clicking on the invoice link that was sent to you via email.', 'invoicing');
                 }
 }
 
@@ -60,8 +60,8 @@  discard block
 block discarded – undo
 
         <div class="container">
             <div class="alert alert-danger m-5" role="alert">
-                <h4 class="alert-heading"><?php esc_html_e( 'Access Denied', 'invoicing' ); ?></h4>
-                <p><?php echo wp_kses_post( $error ); ?></p>
+                <h4 class="alert-heading"><?php esc_html_e('Access Denied', 'invoicing'); ?></h4>
+                <p><?php echo wp_kses_post($error); ?></p>
             </div>
         </div>
 

Please login to merge, or discard this patch.

templates/payment-forms-admin/previews/separator.php 1 patch

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

@@ -7,7 +7,7 @@
 block discarded – undo
  * @version 1.0.19
  */
 
-defined( 'ABSPATH' ) || exit;
+defined('ABSPATH') || exit;
 ?>
 
 <hr class="featurette-divider" />

Please login to merge, or discard this patch.

templates/payment-forms-admin/previews/heading.php 1 patch

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

@@ -8,7 +8,7 @@
 block discarded – undo
  */
 
 
-defined( 'ABSPATH' ) || exit;
+defined('ABSPATH') || exit;
 
 ?>
 <component :is='form_element.level' v-html='form_element.text'></component>

Please login to merge, or discard this patch.

templates/payment-forms/elements/items.php 1 patch

Spacing +13 added lines, -13 removed lines patch added patch discarded remove patch

@@ -7,44 +7,44 @@
 block discarded – undo
  * @version 1.0.19
  */
 
-defined( 'ABSPATH' ) || exit;
+defined('ABSPATH') || exit;
 
-if ( empty( $form->get_items() ) ) {
+if (empty($form->get_items())) {
     return;
 }
 
-if ( ! empty( $GLOBALS['getpaid_force_checkbox'] ) ) {
+if (!empty($GLOBALS['getpaid_force_checkbox'])) {
     $items_type = 'checkbox';
 }
 
-if ( empty( $items_type ) ) {
+if (empty($items_type)) {
     $items_type = 'total';
 }
 
-do_action( 'getpaid_before_payment_form_items', $form );
+do_action('getpaid_before_payment_form_items', $form);
 
-switch ( $items_type ) {
+switch ($items_type) {
     case 'radio':
-        wpinv_get_template( 'payment-forms/variations/radio.php', compact( 'form', 'items_type' ) );
+        wpinv_get_template('payment-forms/variations/radio.php', compact('form', 'items_type'));
         break;
     case 'checkbox':
-        wpinv_get_template( 'payment-forms/variations/checkbox.php', compact( 'form', 'items_type' ) );
+        wpinv_get_template('payment-forms/variations/checkbox.php', compact('form', 'items_type'));
         break;
     case 'select':
-        wpinv_get_template( 'payment-forms/variations/select.php', compact( 'form', 'items_type' ) );
+        wpinv_get_template('payment-forms/variations/select.php', compact('form', 'items_type'));
         break;
 }
 
-do_action( 'getpaid_before_payment_form_cart', $form );
+do_action('getpaid_before_payment_form_cart', $form);
 
 // Display the cart totals.
-if ( ! empty( $hide_cart ) ) {
+if (!empty($hide_cart)) {
     echo '<div class="d-none">';
 }
 
 // Display the cart totals.
-wpinv_get_template( 'payment-forms/cart.php', compact( 'form', 'items_type' ) );
+wpinv_get_template('payment-forms/cart.php', compact('form', 'items_type'));
 
-if ( ! empty( $hide_cart ) ) {
+if (!empty($hide_cart)) {
     echo '</div>';
 }

Please login to merge, or discard this patch.

templates/payment-forms/cart.php 2 patches

Indentation +1 added lines, -1 removed lines patch added patch discarded remove patch

@@ -44,7 +44,7 @@
 block discarded – undo
 
             // Display the item totals.
             foreach ( $form->get_items() as $item ) {
-			wpinv_get_template( 'payment-forms/cart-item.php', compact( 'form', 'item', 'columns' ) );
+            wpinv_get_template( 'payment-forms/cart-item.php', compact( 'form', 'item', 'columns' ) );
             }
 
             // Display the cart totals.

Please login to merge, or discard this patch.

Spacing +17 added lines, -17 removed lines patch added patch discarded remove patch

@@ -7,34 +7,34 @@  discard block
 block discarded – undo
  * @version 1.0.19
  */
 
-defined( 'ABSPATH' ) || exit;
+defined('ABSPATH') || exit;
 
 // Cart table columns.
 $columns = array(
-    'name'     => __( 'Item', 'invoicing' ),
-    'price'    => __( 'Price', 'invoicing' ),
-    'quantity' => __( 'Qty', 'invoicing' ),
-    'subtotal' => __( 'Subtotal', 'invoicing' ),
+    'name'     => __('Item', 'invoicing'),
+    'price'    => __('Price', 'invoicing'),
+    'quantity' => __('Qty', 'invoicing'),
+    'subtotal' => __('Subtotal', 'invoicing'),
 );
 
-if ( ! empty( $form->invoice ) ) {
-    $columns = getpaid_invoice_item_columns( $form->invoice );
+if (!empty($form->invoice)) {
+    $columns = getpaid_invoice_item_columns($form->invoice);
 }
 
-if ( isset( $columns['tax_rate'] ) ) {
-    unset( $columns['tax_rate'] );
+if (isset($columns['tax_rate'])) {
+    unset($columns['tax_rate']);
 }
 
-$columns = apply_filters( 'getpaid_payment_form_cart_table_columns', $columns, $form );
+$columns = apply_filters('getpaid_payment_form_cart_table_columns', $columns, $form);
 
 ?>
     <div class="getpaid-payment-form-items-cart border form-group mb-3">
 
         <div class="getpaid-payment-form-items-cart-header font-weight-bold bg-light border-bottom py-2 px-3">
             <div class="form-row row">
-                <?php foreach ( $columns as $key => $label ) : ?>
-                    <div class="<?php echo 'name' == $key ? 'col-6' : 'col'; ?> <?php echo ( in_array( $key, array( 'subtotal', 'quantity', 'tax_rate' ) ) ) ? 'd-none d-sm-block' : ''; ?> getpaid-form-cart-item-<?php echo esc_attr( $key ); ?>">
-                        <span><?php echo esc_html( $label ); ?></span>
+                <?php foreach ($columns as $key => $label) : ?>
+                    <div class="<?php echo 'name' == $key ? 'col-6' : 'col'; ?> <?php echo (in_array($key, array('subtotal', 'quantity', 'tax_rate'))) ? 'd-none d-sm-block' : ''; ?> getpaid-form-cart-item-<?php echo esc_attr($key); ?>">
+                        <span><?php echo esc_html($label); ?></span>
                     </div>
                 <?php endforeach; ?>
             </div>
@@ -43,16 +43,16 @@  discard block
 block discarded – undo
         <?php
 
             // Display the item totals.
-            foreach ( $form->get_items() as $item ) {
-			wpinv_get_template( 'payment-forms/cart-item.php', compact( 'form', 'item', 'columns' ) );
+            foreach ($form->get_items() as $item) {
+			wpinv_get_template('payment-forms/cart-item.php', compact('form', 'item', 'columns'));
             }
 
             // Display the cart totals.
-            wpinv_get_template( 'payment-forms/cart-totals.php', compact( 'form' ) );
+            wpinv_get_template('payment-forms/cart-totals.php', compact('form'));
 
         ?>
     </div>
 
 <?php
 
-do_action( 'getpaid_after_payment_form_cart', $form );
+do_action('getpaid_after_payment_form_cart', $form);

Please login to merge, or discard this patch.

templates/emails/invoice-items.php 2 patches

Indentation +2 added lines, -2 removed lines patch added patch discarded remove patch

@@ -44,12 +44,12 @@
 block discarded – undo
 
                 // Display the item totals.
                 foreach ( $invoice->get_items() as $item ) {
-				wpinv_get_template( 'emails/invoice-item.php', compact( 'invoice', 'item', 'columns' ) );
+                wpinv_get_template( 'emails/invoice-item.php', compact( 'invoice', 'item', 'columns' ) );
                 }
 
                 // Display the fee totals.
                 foreach ( $invoice->get_fees() as $fee ) {
-				wpinv_get_template( 'emails/fee-item.php', compact( 'invoice', 'fee', 'columns' ) );
+                wpinv_get_template( 'emails/fee-item.php', compact( 'invoice', 'fee', 'columns' ) );
                 }
 
             ?>

Please login to merge, or discard this patch.

Spacing +12 added lines, -12 removed lines patch added patch discarded remove patch

@@ -8,18 +8,18 @@  discard block
 block discarded – undo
  * @var WPInv_Invoice $invoice
  */
 
-defined( 'ABSPATH' ) || exit;
+defined('ABSPATH') || exit;
 
-$column_count = count( $columns );
+$column_count = count($columns);
 ?>
 
-<?php do_action( 'wpinv_before_email_items', $invoice ); ?>
+<?php do_action('wpinv_before_email_items', $invoice); ?>
 
 
 <div id="wpinv-email-items">
 
     <h3 class="invoice-items-title">
-        <?php echo sprintf( esc_html__( '%s Items', 'invoicing' ), esc_html( ucfirst( $invoice->get_invoice_quote_type() ) ) ); ?>
+        <?php echo sprintf(esc_html__('%s Items', 'invoicing'), esc_html(ucfirst($invoice->get_invoice_quote_type()))); ?>
     </h3>
 
     <table class="table table-bordered table-hover">
@@ -28,9 +28,9 @@  discard block
 block discarded – undo
 
             <tr class="wpinv_cart_header_row">
 
-                <?php foreach ( $columns as $key => $label ) : ?>
-                    <th class="<?php echo 'name' == $key ? 'text-left' : 'text-right'; ?> wpinv_cart_item_<?php echo esc_attr( $key ); ?>">
-                        <?php echo esc_html( $label ); ?>
+                <?php foreach ($columns as $key => $label) : ?>
+                    <th class="<?php echo 'name' == $key ? 'text-left' : 'text-right'; ?> wpinv_cart_item_<?php echo esc_attr($key); ?>">
+                        <?php echo esc_html($label); ?>
                     </th>
                 <?php endforeach; ?>
 
@@ -43,13 +43,13 @@  discard block
 block discarded – undo
             <?php
 
                 // Display the item totals.
-                foreach ( $invoice->get_items() as $item ) {
-				wpinv_get_template( 'emails/invoice-item.php', compact( 'invoice', 'item', 'columns' ) );
+                foreach ($invoice->get_items() as $item) {
+				wpinv_get_template('emails/invoice-item.php', compact('invoice', 'item', 'columns'));
                 }
 
                 // Display the fee totals.
-                foreach ( $invoice->get_fees() as $fee ) {
-				wpinv_get_template( 'emails/fee-item.php', compact( 'invoice', 'fee', 'columns' ) );
+                foreach ($invoice->get_fees() as $fee) {
+				wpinv_get_template('emails/fee-item.php', compact('invoice', 'fee', 'columns'));
                 }
 
             ?>
@@ -57,7 +57,7 @@  discard block
 block discarded – undo
         </tbody>
 
         <tfoot>
-            <?php wpinv_get_template( 'emails/invoice-totals.php', compact( 'invoice', 'column_count' ) ); ?>
+            <?php wpinv_get_template('emails/invoice-totals.php', compact('invoice', 'column_count')); ?>
         </tfoot>
     
     </table>

Please login to merge, or discard this patch.

templates/emails/wpinv-email-billing-details.php 2 patches

Indentation +1 added lines, -1 removed lines patch added patch discarded remove patch

@@ -20,7 +20,7 @@
 block discarded – undo
 <a href="<?php echo esc_url( add_query_arg( 'user_id', $invoice->get_user_id(), self_admin_url( 'user-edit.php' ) ) ); ?>"><?php echo esc_html( $invoice->get_user_full_name() ); ?></a>
                     <?php
 } else {
-					echo esc_html( $invoice->get_user_full_name() ); }
+                    echo esc_html( $invoice->get_user_full_name() ); }
 ?>
 </td>
             </tr>

Please login to merge, or discard this patch.

Spacing +21 added lines, -21 removed lines patch added patch discarded remove patch

@@ -1,51 +1,51 @@
 block discarded – undo
 <?php
 // don't load directly
-if ( ! defined( 'ABSPATH' ) ) {
-    die( '-1' );
+if (!defined('ABSPATH')) {
+    die('-1');
 }
 
-do_action( 'wpinv_email_before_billing_details', $invoice ); ?>
+do_action('wpinv_email_before_billing_details', $invoice); ?>
 <div id="wpinv-email-billing">
-    <h3 class="wpinv-address-t"><?php echo esc_html( apply_filters( 'wpinv_email_billing_title', __( 'Billing Details', 'invoicing' ) ) ); ?></h3>
+    <h3 class="wpinv-address-t"><?php echo esc_html(apply_filters('wpinv_email_billing_title', __('Billing Details', 'invoicing'))); ?></h3>
 
     <table class="table table-bordered table-sm wpi-billing-details">
         <tbody>
-            <?php do_action( 'wpinv_email_billing_fields_first', $invoice ); ?>
+            <?php do_action('wpinv_email_billing_fields_first', $invoice); ?>
             <tr class="wpi-receipt-name">
-                <th class="text-left"><?php esc_html_e( 'Name', 'invoicing' ); ?></th>
+                <th class="text-left"><?php esc_html_e('Name', 'invoicing'); ?></th>
                 <td>
                 <?php
-                if ( $sent_to_admin && $invoice->get_user_id() ) {
+                if ($sent_to_admin && $invoice->get_user_id()) {
 ?>
-<a href="<?php echo esc_url( add_query_arg( 'user_id', $invoice->get_user_id(), self_admin_url( 'user-edit.php' ) ) ); ?>"><?php echo esc_html( $invoice->get_user_full_name() ); ?></a>
+<a href="<?php echo esc_url(add_query_arg('user_id', $invoice->get_user_id(), self_admin_url('user-edit.php'))); ?>"><?php echo esc_html($invoice->get_user_full_name()); ?></a>
                     <?php
 } else {
-					echo esc_html( $invoice->get_user_full_name() ); }
+					echo esc_html($invoice->get_user_full_name()); }
 ?>
 </td>
             </tr>
             <tr class="wpi-receipt-email">
-                <th class="text-left"><?php esc_html_e( 'Email', 'invoicing' ); ?></th>
-                <td><?php echo esc_html( sanitize_email( $invoice->get_email() ) ); ?></td>
+                <th class="text-left"><?php esc_html_e('Email', 'invoicing'); ?></th>
+                <td><?php echo esc_html(sanitize_email($invoice->get_email())); ?></td>
             </tr>
-            <?php if ( $invoice->get_company() ) { ?>
+            <?php if ($invoice->get_company()) { ?>
             <tr class="wpi-receipt-company">
-                <th class="text-left"><?php esc_html_e( 'Company', 'invoicing' ); ?></th>
-                <td><?php echo esc_html( $invoice->get_company() ); ?></td>
+                <th class="text-left"><?php esc_html_e('Company', 'invoicing'); ?></th>
+                <td><?php echo esc_html($invoice->get_company()); ?></td>
             </tr>
             <?php } ?>
             <tr class="wpi-receipt-address">
-                <th class="text-left"><?php esc_html_e( 'Address', 'invoicing' ); ?></th>
-                <td><?php echo wp_kses_post( wpinv_get_invoice_address_markup( $invoice->get_user_info() ) ); ?></td>
+                <th class="text-left"><?php esc_html_e('Address', 'invoicing'); ?></th>
+                <td><?php echo wp_kses_post(wpinv_get_invoice_address_markup($invoice->get_user_info())); ?></td>
             </tr>
-            <?php if ( $invoice->get_phone() ) { ?>
+            <?php if ($invoice->get_phone()) { ?>
             <tr class="wpi-receipt-phone">
-                <th class="text-left"><?php esc_html_e( 'Phone', 'invoicing' ); ?></th>
-                <td><?php echo esc_html( $invoice->get_phone() ); ?></td>
+                <th class="text-left"><?php esc_html_e('Phone', 'invoicing'); ?></th>
+                <td><?php echo esc_html($invoice->get_phone()); ?></td>
             </tr>
             <?php } ?>
-            <?php do_action( 'wpinv_email_billing_fields_last', $invoice ); ?>
+            <?php do_action('wpinv_email_billing_fields_last', $invoice); ?>
         </tbody>
     </table>
 </div>
-<?php do_action( 'wpinv_email_after_billing_details', $invoice ); ?>
+<?php do_action('wpinv_email_after_billing_details', $invoice); ?>

Please login to merge, or discard this patch.

includes/data-stores/class-getpaid-payment-form-data-store.php 2 patches

Indentation +169 added lines, -169 removed lines patch added patch discarded remove patch

@@ -4,7 +4,7 @@  discard block
 block discarded – undo
  *
  */
 if ( ! defined( 'ABSPATH' ) ) {
-	exit;
+    exit;
 }
 
 /**
@@ -14,179 +14,179 @@  discard block
 block discarded – undo
  */
 class GetPaid_Payment_Form_Data_Store extends GetPaid_Data_Store_WP {
 
-	/**
-	 * Data stored in meta keys, but not considered "meta" for a form.
-	 *
-	 * @since 1.0.19
-	 * @var array
-	 */
-	protected $internal_meta_keys = array(
-		'wpinv_form_elements',
-		'wpinv_form_items',
-		'wpinv_form_earned',
-		'wpinv_form_refunded',
-		'wpinv_form_cancelled',
-		'wpinv_form_failed',
-	);
-
-	/**
-	 * A map of meta keys to data props.
-	 *
-	 * @since 1.0.19
-	 *
-	 * @var array
-	 */
-	protected $meta_key_to_props = array(
-		'wpinv_form_elements'  => 'elements',
-		'wpinv_form_items'     => 'items',
-		'wpinv_form_earned'    => 'earned',
-		'wpinv_form_refunded'  => 'refunded',
-		'wpinv_form_cancelled' => 'cancelled',
-		'wpinv_form_failed'    => 'failed',
-	);
-
-	/*
+    /**
+     * Data stored in meta keys, but not considered "meta" for a form.
+     *
+     * @since 1.0.19
+     * @var array
+     */
+    protected $internal_meta_keys = array(
+        'wpinv_form_elements',
+        'wpinv_form_items',
+        'wpinv_form_earned',
+        'wpinv_form_refunded',
+        'wpinv_form_cancelled',
+        'wpinv_form_failed',
+    );
+
+    /**
+     * A map of meta keys to data props.
+     *
+     * @since 1.0.19
+     *
+     * @var array
+     */
+    protected $meta_key_to_props = array(
+        'wpinv_form_elements'  => 'elements',
+        'wpinv_form_items'     => 'items',
+        'wpinv_form_earned'    => 'earned',
+        'wpinv_form_refunded'  => 'refunded',
+        'wpinv_form_cancelled' => 'cancelled',
+        'wpinv_form_failed'    => 'failed',
+    );
+
+    /*
 	|--------------------------------------------------------------------------
 	| CRUD Methods
 	|--------------------------------------------------------------------------
 	*/
 
-	/**
-	 * Method to create a new form in the database.
-	 *
-	 * @param GetPaid_Payment_Form $form Form object.
-	 */
-	public function create( &$form ) {
-		$form->set_version( WPINV_VERSION );
-		$form->set_date_created( current_time( 'mysql' ) );
-
-		// Create a new post.
-		$id = wp_insert_post(
-			apply_filters(
-				'getpaid_new_payment_form_data',
-				array(
-					'post_date'   => $form->get_date_created( 'edit' ),
-					'post_type'   => 'wpi_payment_form',
-					'post_status' => $this->get_post_status( $form ),
-					'ping_status' => 'closed',
-					'post_author' => $form->get_author( 'edit' ),
-					'post_title'  => $form->get_name( 'edit' ),
-				)
-			),
-			true
-		);
-
-		if ( $id && ! is_wp_error( $id ) ) {
-			$form->set_id( $id );
-			$this->update_post_meta( $form );
-			$form->save_meta_data();
-			$form->apply_changes();
-			$this->clear_caches( $form );
-			do_action( 'getpaid_create_payment_form', $form );
-			return true;
-		}
-
-		if ( is_wp_error( $id ) ) {
-			$form->last_error = $id->get_error_message();
-		}
-
-		return false;
-	}
-
-	/**
-	 * Method to read a form from the database.
-	 *
-	 * @param GetPaid_Payment_Form $form Form object.
-	 *
-	 */
-	public function read( &$form ) {
-
-		$form->set_defaults();
-		$form_object = get_post( $form->get_id() );
-
-		if ( ! $form->get_id() || ! $form_object || $form_object->post_type != 'wpi_payment_form' ) {
-			$form->last_error = __( 'Invalid form.', 'invoicing' );
-			$form->set_id( 0 );
-			return false;
-		}
-
-		$form->set_props(
-			array(
-				'date_created'  => 0 < $form_object->post_date ? $form_object->post_date : null,
-				'date_modified' => 0 < $form_object->post_modified ? $form_object->post_modified : null,
-				'status'        => $form_object->post_status,
-				'name'          => $form_object->post_title,
-				'author'        => $form_object->post_author,
-			)
-		);
-
-		$this->read_object_data( $form, $form_object );
-		$form->read_meta_data();
-		$form->set_object_read( true );
-		do_action( 'getpaid_read_payment_form', $form );
-
-	}
-
-	/**
-	 * Method to update a form in the database.
-	 *
-	 * @param GetPaid_Payment_Form $form Form object.
-	 */
-	public function update( &$form ) {
-		$form->save_meta_data();
-		$form->set_version( WPINV_VERSION );
-
-		if ( null === $form->get_date_created( 'edit' ) ) {
-			$form->set_date_created( current_time( 'mysql' ) );
-		}
-
-		// Grab the current status so we can compare.
-		$previous_status = get_post_status( $form->get_id() );
-
-		$changes = $form->get_changes();
-
-		// Only update the post when the post data changes.
-		if ( array_intersect( array( 'date_created', 'date_modified', 'status', 'name', 'author' ), array_keys( $changes ) ) ) {
-			$post_data = array(
-				'post_date'     => $form->get_date_created( 'edit' ),
-				'post_status'   => $form->get_status( 'edit' ),
-				'post_title'    => $form->get_name( 'edit' ),
-				'post_author'   => $form->get_author( 'edit' ),
-				'post_modified' => $form->get_date_modified( 'edit' ),
-			);
-
-			/**
-			 * When updating this object, to prevent infinite loops, use $wpdb
-			 * to update data, since wp_update_post spawns more calls to the
-			 * save_post action.
-			 *
-			 * This ensures hooks are fired by either WP itself (admin screen save),
-			 * or an update purely from CRUD.
-			 */
-			if ( doing_action( 'save_post' ) ) {
-				$GLOBALS['wpdb']->update( $GLOBALS['wpdb']->posts, $post_data, array( 'ID' => $form->get_id() ) );
-				clean_post_cache( $form->get_id() );
-			} else {
-				wp_update_post( array_merge( array( 'ID' => $form->get_id() ), $post_data ) );
-			}
-			$form->read_meta_data( true ); // Refresh internal meta data, in case things were hooked into `save_post` or another WP hook.
-		}
-		$this->update_post_meta( $form );
-		$form->apply_changes();
-		$this->clear_caches( $form );
-
-		// Fire a hook depending on the status - this should be considered a creation if it was previously draft status.
-		$new_status = $form->get_status( 'edit' );
-
-		if ( $new_status !== $previous_status && in_array( $previous_status, array( 'new', 'auto-draft', 'draft' ), true ) ) {
-			do_action( 'getpaid_new_payment_form', $form );
-		} else {
-			do_action( 'getpaid_update_payment_form', $form );
-		}
-
-	}
-
-	/*
+    /**
+     * Method to create a new form in the database.
+     *
+     * @param GetPaid_Payment_Form $form Form object.
+     */
+    public function create( &$form ) {
+        $form->set_version( WPINV_VERSION );
+        $form->set_date_created( current_time( 'mysql' ) );
+
+        // Create a new post.
+        $id = wp_insert_post(
+            apply_filters(
+                'getpaid_new_payment_form_data',
+                array(
+                    'post_date'   => $form->get_date_created( 'edit' ),
+                    'post_type'   => 'wpi_payment_form',
+                    'post_status' => $this->get_post_status( $form ),
+                    'ping_status' => 'closed',
+                    'post_author' => $form->get_author( 'edit' ),
+                    'post_title'  => $form->get_name( 'edit' ),
+                )
+            ),
+            true
+        );
+
+        if ( $id && ! is_wp_error( $id ) ) {
+            $form->set_id( $id );
+            $this->update_post_meta( $form );
+            $form->save_meta_data();
+            $form->apply_changes();
+            $this->clear_caches( $form );
+            do_action( 'getpaid_create_payment_form', $form );
+            return true;
+        }
+
+        if ( is_wp_error( $id ) ) {
+            $form->last_error = $id->get_error_message();
+        }
+
+        return false;
+    }
+
+    /**
+     * Method to read a form from the database.
+     *
+     * @param GetPaid_Payment_Form $form Form object.
+     *
+     */
+    public function read( &$form ) {
+
+        $form->set_defaults();
+        $form_object = get_post( $form->get_id() );
+
+        if ( ! $form->get_id() || ! $form_object || $form_object->post_type != 'wpi_payment_form' ) {
+            $form->last_error = __( 'Invalid form.', 'invoicing' );
+            $form->set_id( 0 );
+            return false;
+        }
+
+        $form->set_props(
+            array(
+                'date_created'  => 0 < $form_object->post_date ? $form_object->post_date : null,
+                'date_modified' => 0 < $form_object->post_modified ? $form_object->post_modified : null,
+                'status'        => $form_object->post_status,
+                'name'          => $form_object->post_title,
+                'author'        => $form_object->post_author,
+            )
+        );
+
+        $this->read_object_data( $form, $form_object );
+        $form->read_meta_data();
+        $form->set_object_read( true );
+        do_action( 'getpaid_read_payment_form', $form );
+
+    }
+
+    /**
+     * Method to update a form in the database.
+     *
+     * @param GetPaid_Payment_Form $form Form object.
+     */
+    public function update( &$form ) {
+        $form->save_meta_data();
+        $form->set_version( WPINV_VERSION );
+
+        if ( null === $form->get_date_created( 'edit' ) ) {
+            $form->set_date_created( current_time( 'mysql' ) );
+        }
+
+        // Grab the current status so we can compare.
+        $previous_status = get_post_status( $form->get_id() );
+
+        $changes = $form->get_changes();
+
+        // Only update the post when the post data changes.
+        if ( array_intersect( array( 'date_created', 'date_modified', 'status', 'name', 'author' ), array_keys( $changes ) ) ) {
+            $post_data = array(
+                'post_date'     => $form->get_date_created( 'edit' ),
+                'post_status'   => $form->get_status( 'edit' ),
+                'post_title'    => $form->get_name( 'edit' ),
+                'post_author'   => $form->get_author( 'edit' ),
+                'post_modified' => $form->get_date_modified( 'edit' ),
+            );
+
+            /**
+             * When updating this object, to prevent infinite loops, use $wpdb
+             * to update data, since wp_update_post spawns more calls to the
+             * save_post action.
+             *
+             * This ensures hooks are fired by either WP itself (admin screen save),
+             * or an update purely from CRUD.
+             */
+            if ( doing_action( 'save_post' ) ) {
+                $GLOBALS['wpdb']->update( $GLOBALS['wpdb']->posts, $post_data, array( 'ID' => $form->get_id() ) );
+                clean_post_cache( $form->get_id() );
+            } else {
+                wp_update_post( array_merge( array( 'ID' => $form->get_id() ), $post_data ) );
+            }
+            $form->read_meta_data( true ); // Refresh internal meta data, in case things were hooked into `save_post` or another WP hook.
+        }
+        $this->update_post_meta( $form );
+        $form->apply_changes();
+        $this->clear_caches( $form );
+
+        // Fire a hook depending on the status - this should be considered a creation if it was previously draft status.
+        $new_status = $form->get_status( 'edit' );
+
+        if ( $new_status !== $previous_status && in_array( $previous_status, array( 'new', 'auto-draft', 'draft' ), true ) ) {
+            do_action( 'getpaid_new_payment_form', $form );
+        } else {
+            do_action( 'getpaid_update_payment_form', $form );
+        }
+
+    }
+
+    /*
 	|--------------------------------------------------------------------------
 	| Additional Methods
 	|--------------------------------------------------------------------------

Please login to merge, or discard this patch.

Spacing +44 added lines, -44 removed lines patch added patch discarded remove patch

@@ -3,7 +3,7 @@  discard block
 block discarded – undo
  * GetPaid_Payment_Form_Data_Store class file.
  *
  */
-if ( ! defined( 'ABSPATH' ) ) {
+if (!defined('ABSPATH')) {
 	exit;
 }
 
@@ -56,37 +56,37 @@  discard block
 block discarded – undo
 	 *
 	 * @param GetPaid_Payment_Form $form Form object.
 	 */
-	public function create( &$form ) {
-		$form->set_version( WPINV_VERSION );
-		$form->set_date_created( current_time( 'mysql' ) );
+	public function create(&$form) {
+		$form->set_version(WPINV_VERSION);
+		$form->set_date_created(current_time('mysql'));
 
 		// Create a new post.
 		$id = wp_insert_post(
 			apply_filters(
 				'getpaid_new_payment_form_data',
 				array(
-					'post_date'   => $form->get_date_created( 'edit' ),
+					'post_date'   => $form->get_date_created('edit'),
 					'post_type'   => 'wpi_payment_form',
-					'post_status' => $this->get_post_status( $form ),
+					'post_status' => $this->get_post_status($form),
 					'ping_status' => 'closed',
-					'post_author' => $form->get_author( 'edit' ),
-					'post_title'  => $form->get_name( 'edit' ),
+					'post_author' => $form->get_author('edit'),
+					'post_title'  => $form->get_name('edit'),
 				)
 			),
 			true
 		);
 
-		if ( $id && ! is_wp_error( $id ) ) {
-			$form->set_id( $id );
-			$this->update_post_meta( $form );
+		if ($id && !is_wp_error($id)) {
+			$form->set_id($id);
+			$this->update_post_meta($form);
 			$form->save_meta_data();
 			$form->apply_changes();
-			$this->clear_caches( $form );
-			do_action( 'getpaid_create_payment_form', $form );
+			$this->clear_caches($form);
+			do_action('getpaid_create_payment_form', $form);
 			return true;
 		}
 
-		if ( is_wp_error( $id ) ) {
+		if (is_wp_error($id)) {
 			$form->last_error = $id->get_error_message();
 		}
 
@@ -99,14 +99,14 @@  discard block
 block discarded – undo
 	 * @param GetPaid_Payment_Form $form Form object.
 	 *
 	 */
-	public function read( &$form ) {
+	public function read(&$form) {
 
 		$form->set_defaults();
-		$form_object = get_post( $form->get_id() );
+		$form_object = get_post($form->get_id());
 
-		if ( ! $form->get_id() || ! $form_object || $form_object->post_type != 'wpi_payment_form' ) {
-			$form->last_error = __( 'Invalid form.', 'invoicing' );
-			$form->set_id( 0 );
+		if (!$form->get_id() || !$form_object || $form_object->post_type != 'wpi_payment_form') {
+			$form->last_error = __('Invalid form.', 'invoicing');
+			$form->set_id(0);
 			return false;
 		}
 
@@ -120,10 +120,10 @@  discard block
 block discarded – undo
 			)
 		);
 
-		$this->read_object_data( $form, $form_object );
+		$this->read_object_data($form, $form_object);
 		$form->read_meta_data();
-		$form->set_object_read( true );
-		do_action( 'getpaid_read_payment_form', $form );
+		$form->set_object_read(true);
+		do_action('getpaid_read_payment_form', $form);
 
 	}
 
@@ -132,27 +132,27 @@  discard block
 block discarded – undo
 	 *
 	 * @param GetPaid_Payment_Form $form Form object.
 	 */
-	public function update( &$form ) {
+	public function update(&$form) {
 		$form->save_meta_data();
-		$form->set_version( WPINV_VERSION );
+		$form->set_version(WPINV_VERSION);
 
-		if ( null === $form->get_date_created( 'edit' ) ) {
-			$form->set_date_created( current_time( 'mysql' ) );
+		if (null === $form->get_date_created('edit')) {
+			$form->set_date_created(current_time('mysql'));
 		}
 
 		// Grab the current status so we can compare.
-		$previous_status = get_post_status( $form->get_id() );
+		$previous_status = get_post_status($form->get_id());
 
 		$changes = $form->get_changes();
 
 		// Only update the post when the post data changes.
-		if ( array_intersect( array( 'date_created', 'date_modified', 'status', 'name', 'author' ), array_keys( $changes ) ) ) {
+		if (array_intersect(array('date_created', 'date_modified', 'status', 'name', 'author'), array_keys($changes))) {
 			$post_data = array(
-				'post_date'     => $form->get_date_created( 'edit' ),
-				'post_status'   => $form->get_status( 'edit' ),
-				'post_title'    => $form->get_name( 'edit' ),
-				'post_author'   => $form->get_author( 'edit' ),
-				'post_modified' => $form->get_date_modified( 'edit' ),
+				'post_date'     => $form->get_date_created('edit'),
+				'post_status'   => $form->get_status('edit'),
+				'post_title'    => $form->get_name('edit'),
+				'post_author'   => $form->get_author('edit'),
+				'post_modified' => $form->get_date_modified('edit'),
 			);
 
 			/**
@@ -163,25 +163,25 @@  discard block
 block discarded – undo
 			 * This ensures hooks are fired by either WP itself (admin screen save),
 			 * or an update purely from CRUD.
 			 */
-			if ( doing_action( 'save_post' ) ) {
-				$GLOBALS['wpdb']->update( $GLOBALS['wpdb']->posts, $post_data, array( 'ID' => $form->get_id() ) );
-				clean_post_cache( $form->get_id() );
+			if (doing_action('save_post')) {
+				$GLOBALS['wpdb']->update($GLOBALS['wpdb']->posts, $post_data, array('ID' => $form->get_id()));
+				clean_post_cache($form->get_id());
 			} else {
-				wp_update_post( array_merge( array( 'ID' => $form->get_id() ), $post_data ) );
+				wp_update_post(array_merge(array('ID' => $form->get_id()), $post_data));
 			}
-			$form->read_meta_data( true ); // Refresh internal meta data, in case things were hooked into `save_post` or another WP hook.
+			$form->read_meta_data(true); // Refresh internal meta data, in case things were hooked into `save_post` or another WP hook.
 		}
-		$this->update_post_meta( $form );
+		$this->update_post_meta($form);
 		$form->apply_changes();
-		$this->clear_caches( $form );
+		$this->clear_caches($form);
 
 		// Fire a hook depending on the status - this should be considered a creation if it was previously draft status.
-		$new_status = $form->get_status( 'edit' );
+		$new_status = $form->get_status('edit');
 
-		if ( $new_status !== $previous_status && in_array( $previous_status, array( 'new', 'auto-draft', 'draft' ), true ) ) {
-			do_action( 'getpaid_new_payment_form', $form );
+		if ($new_status !== $previous_status && in_array($previous_status, array('new', 'auto-draft', 'draft'), true)) {
+			do_action('getpaid_new_payment_form', $form);
 		} else {
-			do_action( 'getpaid_update_payment_form', $form );
+			do_action('getpaid_update_payment_form', $form);
 		}
 
 	}

Please login to merge, or discard this patch.

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

Indentation +857 added lines, -857 removed lines patch added patch discarded remove patch

@@ -9,7 +9,7 @@  discard block
 block discarded – undo
  */
 
 if ( ! defined( 'ABSPATH' ) ) {
-	exit;
+    exit;
 }
 
 /**
@@ -21,356 +21,356 @@  discard block
 block discarded – undo
  */
 abstract class GetPaid_Data {
 
-	/**
-	 * ID for this object.
-	 *
-	 * @since 1.0.19
-	 * @var int
-	 */
-	protected $id = 0;
-
-	/**
-	 * Core data for this object. Name value pairs (name + default value).
-	 *
-	 * @since 1.0.19
-	 * @var array
-	 */
-	protected $data = array();
-
-	/**
-	 * Core data changes for this object.
-	 *
-	 * @since 1.0.19
-	 * @var array
-	 */
-	protected $changes = array();
-
-	/**
-	 * This is false until the object is read from the DB.
-	 *
-	 * @since 1.0.19
-	 * @var bool
-	 */
-	protected $object_read = false;
-
-	/**
-	 * This is the name of this object type.
-	 *
-	 * @since 1.0.19
-	 * @var string
-	 */
-	protected $object_type = 'data';
-
-	/**
-	 * Extra data for this object. Name value pairs (name + default value).
-	 * Used as a standard way for sub classes (like item types) to add
-	 * additional information to an inherited class.
-	 *
-	 * @since 1.0.19
-	 * @var array
-	 */
-	protected $extra_data = array();
-
-	/**
-	 * Set to _data on construct so we can track and reset data if needed.
-	 *
-	 * @since 1.0.19
-	 * @var array
-	 */
-	protected $default_data = array();
-
-	/**
-	 * Contains a reference to the data store for this class.
-	 *
-	 * @since 1.0.19
-	 * @var GetPaid_Data_Store
-	 */
-	protected $data_store;
-
-	/**
-	 * Stores meta in cache for future reads.
-	 * A group must be set to to enable caching.
-	 *
-	 * @since 1.0.19
-	 * @var string
-	 */
-	protected $cache_group = '';
-
-	/**
-	 * Stores the last error.
-	 *
-	 * @since 1.0.19
-	 * @var string
-	 */
-	public $last_error = '';
-
-	/**
-	 * Stores additional meta data.
-	 *
-	 * @since 1.0.19
-	 * @var array
-	 */
-	protected $meta_data = null;
-
-	/**
-	 * Default constructor.
-	 *
-	 * @param int|object|array|string $read ID to load from the DB (optional) or already queried data.
-	 */
-	public function __construct( $read = 0 ) {
-		$this->data         = array_merge( $this->data, $this->extra_data );
-		$this->default_data = $this->data;
-	}
-
-	/**
-	 * Only store the object ID to avoid serializing the data object instance.
-	 *
-	 * @return array
-	 */
-	public function __sleep() {
-		return array( 'id' );
-	}
-
-	/**
-	 * Re-run the constructor with the object ID.
-	 *
-	 * If the object no longer exists, remove the ID.
-	 */
-	public function __wakeup() {
-		$this->__construct( absint( $this->id ) );
-
-		if ( ! empty( $this->last_error ) ) {
-			$this->set_id( 0 );
-		}
-
-	}
-
-	/**
-	 * When the object is cloned, make sure meta is duplicated correctly.
-	 *
-	 * @since 1.0.19
-	 */
-	public function __clone() {
-		$this->maybe_read_meta_data();
-		if ( ! empty( $this->meta_data ) ) {
-			foreach ( $this->meta_data as $array_key => $meta ) {
-				$this->meta_data[ $array_key ] = clone $meta;
-				if ( ! empty( $meta->id ) ) {
-					$this->meta_data[ $array_key ]->id = null;
-				}
-			}
-		}
-	}
-
-	/**
-	 * Get the data store.
-	 *
-	 * @since  1.0.19
-	 * @return object
-	 */
-	public function get_data_store() {
-		return $this->data_store;
-	}
-
-	/**
-	 * Get the object type.
-	 *
-	 * @since  1.0.19
-	 * @return string
-	 */
-	public function get_object_type() {
-		return $this->object_type;
-	}
-
-	/**
-	 * Returns the unique ID for this object.
-	 *
-	 * @since  1.0.19
-	 * @return int
-	 */
-	public function get_id() {
-		return $this->id;
-	}
-
-	/**
-	 * Get form status.
-	 *
-	 * @since 1.0.19
-	 * @param  string $context View or edit context.
-	 * @return string
-	 */
-	public function get_status( $context = 'view' ) {
-		return $this->get_prop( 'status', $context );
-    }
-
-	/**
-	 * Delete an object, set the ID to 0, and return result.
-	 *
-	 * @since  1.0.19
-	 * @param  bool $force_delete Should the data be deleted permanently.
-	 * @return bool result
-	 */
-	public function delete( $force_delete = false ) {
-		if ( $this->data_store && $this->exists() ) {
-			$this->data_store->delete( $this, array( 'force_delete' => $force_delete ) );
-			$this->set_id( 0 );
-			return true;
-		}
-		return false;
-	}
-
-	/**
-	 * Save should create or update based on object existence.
-	 *
-	 * @since  1.0.19
-	 * @return int
-	 */
-	public function save() {
-		if ( ! $this->data_store ) {
-			return $this->get_id();
-		}
-
-		/**
-		 * Trigger action before saving to the DB. Allows you to adjust object props before save.
-		 *
-		 * @param GetPaid_Data          $this The object being saved.
-		 * @param GetPaid_Data_Store_WP $data_store The data store persisting the data.
-		 */
-		do_action( 'getpaid_before_' . $this->object_type . '_object_save', $this, $this->data_store );
-
-		if ( $this->get_id() ) {
-			$this->data_store->update( $this );
-		} else {
-			$this->data_store->create( $this );
-		}
-
-		/**
-		 * Trigger action after saving to the DB.
-		 *
-		 * @param GetPaid_Data          $this The object being saved.
-		 * @param GetPaid_Data_Store_WP $data_store The data store persisting the data.
-		 */
-		do_action( 'getpaid_after_' . $this->object_type . '_object_save', $this, $this->data_store );
-
-		return $this->get_id();
-	}
-
-	/**
-	 * Change data to JSON format.
-	 *
-	 * @since  1.0.19
-	 * @return string Data in JSON format.
-	 */
-	public function __toString() {
-		return wp_json_encode( $this->get_data() );
-	}
-
-	/**
-	 * Returns all data for this object.
-	 *
-	 * @since  1.0.19
-	 * @return array
-	 */
-	public function get_data() {
-		return array_merge( array( 'id' => $this->get_id() ), $this->data, array( 'meta_data' => $this->get_meta_data() ) );
-	}
-
-	/**
-	 * Returns array of expected data keys for this object.
-	 *
-	 * @since   1.0.19
-	 * @return array
-	 */
-	public function get_data_keys() {
-		return array_keys( $this->data );
-	}
-
-	/**
-	 * Returns all "extra" data keys for an object (for sub objects like item types).
-	 *
-	 * @since  1.0.19
-	 * @return array
-	 */
-	public function get_extra_data_keys() {
-		return array_keys( $this->extra_data );
-	}
-
-	/**
-	 * Filter null meta values from array.
-	 *
-	 * @since  1.0.19
-	 * @param mixed $meta Meta value to check.
-	 * @return bool
-	 */
-	protected function filter_null_meta( $meta ) {
-		return ! is_null( $meta->value );
-	}
-
-	/**
-	 * Get All Meta Data.
-	 *
-	 * @since 1.0.19
-	 * @return array of objects.
-	 */
-	public function get_meta_data() {
-		$this->maybe_read_meta_data();
-		return array_values( array_filter( $this->meta_data, array( $this, 'filter_null_meta' ) ) );
-	}
-
-	/**
-	 * Check if the key is an internal one.
-	 *
-	 * @since  1.0.19
-	 * @param  string $key Key to check.
-	 * @return bool   true if it's an internal key, false otherwise
-	 */
-	protected function is_internal_meta_key( $key ) {
-		$internal_meta_key = ! empty( $key ) && $this->data_store && in_array( $key, $this->data_store->get_internal_meta_keys(), true );
-
-		if ( ! $internal_meta_key ) {
-			return false;
-		}
-
-		$has_setter_or_getter = is_callable( array( $this, 'set_' . $key ) ) || is_callable( array( $this, 'get_' . $key ) );
-
-		if ( ! $has_setter_or_getter ) {
-			return false;
-		}
-
-		/* translators: %s: $key Key to check */
-		getpaid_doing_it_wrong( __FUNCTION__, sprintf( __( 'Generic add/update/get meta methods should not be used for internal meta data, including "%s". Use getters and setters.', 'invoicing' ), $key ), '1.0.19' );
-
-		return true;
-	}
-
-	/**
-	 * Magic method for setting data fields.
-	 *
-	 * This method does not update custom fields in the database.
-	 *
-	 * @since 1.0.19
-	 * @access public
-	 *
-	 */
-	public function __set( $key, $value ) {
-
-		if ( 'id' == strtolower( $key ) ) {
-			return $this->set_id( $value );
-		}
-
-		if ( method_exists( $this, "set_$key" ) ) {
-
-			/* translators: %s: $key Key to set */
-			getpaid_doing_it_wrong( __FUNCTION__, sprintf( __( 'Object data such as "%s" should not be accessed directly. Use getters and setters.', 'invoicing' ), $key ), '1.0.19' );
-
-			call_user_func( array( $this, "set_$key" ), $value );
-		} else {
-			$this->set_prop( $key, $value );
-		}
-
-	}
-
-	/**
+    /**
+     * ID for this object.
+     *
+     * @since 1.0.19
+     * @var int
+     */
+    protected $id = 0;
+
+    /**
+     * Core data for this object. Name value pairs (name + default value).
+     *
+     * @since 1.0.19
+     * @var array
+     */
+    protected $data = array();
+
+    /**
+     * Core data changes for this object.
+     *
+     * @since 1.0.19
+     * @var array
+     */
+    protected $changes = array();
+
+    /**
+     * This is false until the object is read from the DB.
+     *
+     * @since 1.0.19
+     * @var bool
+     */
+    protected $object_read = false;
+
+    /**
+     * This is the name of this object type.
+     *
+     * @since 1.0.19
+     * @var string
+     */
+    protected $object_type = 'data';
+
+    /**
+     * Extra data for this object. Name value pairs (name + default value).
+     * Used as a standard way for sub classes (like item types) to add
+     * additional information to an inherited class.
+     *
+     * @since 1.0.19
+     * @var array
+     */
+    protected $extra_data = array();
+
+    /**
+     * Set to _data on construct so we can track and reset data if needed.
+     *
+     * @since 1.0.19
+     * @var array
+     */
+    protected $default_data = array();
+
+    /**
+     * Contains a reference to the data store for this class.
+     *
+     * @since 1.0.19
+     * @var GetPaid_Data_Store
+     */
+    protected $data_store;
+
+    /**
+     * Stores meta in cache for future reads.
+     * A group must be set to to enable caching.
+     *
+     * @since 1.0.19
+     * @var string
+     */
+    protected $cache_group = '';
+
+    /**
+     * Stores the last error.
+     *
+     * @since 1.0.19
+     * @var string
+     */
+    public $last_error = '';
+
+    /**
+     * Stores additional meta data.
+     *
+     * @since 1.0.19
+     * @var array
+     */
+    protected $meta_data = null;
+
+    /**
+     * Default constructor.
+     *
+     * @param int|object|array|string $read ID to load from the DB (optional) or already queried data.
+     */
+    public function __construct( $read = 0 ) {
+        $this->data         = array_merge( $this->data, $this->extra_data );
+        $this->default_data = $this->data;
+    }
+
+    /**
+     * Only store the object ID to avoid serializing the data object instance.
+     *
+     * @return array
+     */
+    public function __sleep() {
+        return array( 'id' );
+    }
+
+    /**
+     * Re-run the constructor with the object ID.
+     *
+     * If the object no longer exists, remove the ID.
+     */
+    public function __wakeup() {
+        $this->__construct( absint( $this->id ) );
+
+        if ( ! empty( $this->last_error ) ) {
+            $this->set_id( 0 );
+        }
+
+    }
+
+    /**
+     * When the object is cloned, make sure meta is duplicated correctly.
+     *
+     * @since 1.0.19
+     */
+    public function __clone() {
+        $this->maybe_read_meta_data();
+        if ( ! empty( $this->meta_data ) ) {
+            foreach ( $this->meta_data as $array_key => $meta ) {
+                $this->meta_data[ $array_key ] = clone $meta;
+                if ( ! empty( $meta->id ) ) {
+                    $this->meta_data[ $array_key ]->id = null;
+                }
+            }
+        }
+    }
+
+    /**
+     * Get the data store.
+     *
+     * @since  1.0.19
+     * @return object
+     */
+    public function get_data_store() {
+        return $this->data_store;
+    }
+
+    /**
+     * Get the object type.
+     *
+     * @since  1.0.19
+     * @return string
+     */
+    public function get_object_type() {
+        return $this->object_type;
+    }
+
+    /**
+     * Returns the unique ID for this object.
+     *
+     * @since  1.0.19
+     * @return int
+     */
+    public function get_id() {
+        return $this->id;
+    }
+
+    /**
+     * Get form status.
+     *
+     * @since 1.0.19
+     * @param  string $context View or edit context.
+     * @return string
+     */
+    public function get_status( $context = 'view' ) {
+        return $this->get_prop( 'status', $context );
+    }
+
+    /**
+     * Delete an object, set the ID to 0, and return result.
+     *
+     * @since  1.0.19
+     * @param  bool $force_delete Should the data be deleted permanently.
+     * @return bool result
+     */
+    public function delete( $force_delete = false ) {
+        if ( $this->data_store && $this->exists() ) {
+            $this->data_store->delete( $this, array( 'force_delete' => $force_delete ) );
+            $this->set_id( 0 );
+            return true;
+        }
+        return false;
+    }
+
+    /**
+     * Save should create or update based on object existence.
+     *
+     * @since  1.0.19
+     * @return int
+     */
+    public function save() {
+        if ( ! $this->data_store ) {
+            return $this->get_id();
+        }
+
+        /**
+         * Trigger action before saving to the DB. Allows you to adjust object props before save.
+         *
+         * @param GetPaid_Data          $this The object being saved.
+         * @param GetPaid_Data_Store_WP $data_store The data store persisting the data.
+         */
+        do_action( 'getpaid_before_' . $this->object_type . '_object_save', $this, $this->data_store );
+
+        if ( $this->get_id() ) {
+            $this->data_store->update( $this );
+        } else {
+            $this->data_store->create( $this );
+        }
+
+        /**
+         * Trigger action after saving to the DB.
+         *
+         * @param GetPaid_Data          $this The object being saved.
+         * @param GetPaid_Data_Store_WP $data_store The data store persisting the data.
+         */
+        do_action( 'getpaid_after_' . $this->object_type . '_object_save', $this, $this->data_store );
+
+        return $this->get_id();
+    }
+
+    /**
+     * Change data to JSON format.
+     *
+     * @since  1.0.19
+     * @return string Data in JSON format.
+     */
+    public function __toString() {
+        return wp_json_encode( $this->get_data() );
+    }
+
+    /**
+     * Returns all data for this object.
+     *
+     * @since  1.0.19
+     * @return array
+     */
+    public function get_data() {
+        return array_merge( array( 'id' => $this->get_id() ), $this->data, array( 'meta_data' => $this->get_meta_data() ) );
+    }
+
+    /**
+     * Returns array of expected data keys for this object.
+     *
+     * @since   1.0.19
+     * @return array
+     */
+    public function get_data_keys() {
+        return array_keys( $this->data );
+    }
+
+    /**
+     * Returns all "extra" data keys for an object (for sub objects like item types).
+     *
+     * @since  1.0.19
+     * @return array
+     */
+    public function get_extra_data_keys() {
+        return array_keys( $this->extra_data );
+    }
+
+    /**
+     * Filter null meta values from array.
+     *
+     * @since  1.0.19
+     * @param mixed $meta Meta value to check.
+     * @return bool
+     */
+    protected function filter_null_meta( $meta ) {
+        return ! is_null( $meta->value );
+    }
+
+    /**
+     * Get All Meta Data.
+     *
+     * @since 1.0.19
+     * @return array of objects.
+     */
+    public function get_meta_data() {
+        $this->maybe_read_meta_data();
+        return array_values( array_filter( $this->meta_data, array( $this, 'filter_null_meta' ) ) );
+    }
+
+    /**
+     * Check if the key is an internal one.
+     *
+     * @since  1.0.19
+     * @param  string $key Key to check.
+     * @return bool   true if it's an internal key, false otherwise
+     */
+    protected function is_internal_meta_key( $key ) {
+        $internal_meta_key = ! empty( $key ) && $this->data_store && in_array( $key, $this->data_store->get_internal_meta_keys(), true );
+
+        if ( ! $internal_meta_key ) {
+            return false;
+        }
+
+        $has_setter_or_getter = is_callable( array( $this, 'set_' . $key ) ) || is_callable( array( $this, 'get_' . $key ) );
+
+        if ( ! $has_setter_or_getter ) {
+            return false;
+        }
+
+        /* translators: %s: $key Key to check */
+        getpaid_doing_it_wrong( __FUNCTION__, sprintf( __( 'Generic add/update/get meta methods should not be used for internal meta data, including "%s". Use getters and setters.', 'invoicing' ), $key ), '1.0.19' );
+
+        return true;
+    }
+
+    /**
+     * Magic method for setting data fields.
+     *
+     * This method does not update custom fields in the database.
+     *
+     * @since 1.0.19
+     * @access public
+     *
+     */
+    public function __set( $key, $value ) {
+
+        if ( 'id' == strtolower( $key ) ) {
+            return $this->set_id( $value );
+        }
+
+        if ( method_exists( $this, "set_$key" ) ) {
+
+            /* translators: %s: $key Key to set */
+            getpaid_doing_it_wrong( __FUNCTION__, sprintf( __( 'Object data such as "%s" should not be accessed directly. Use getters and setters.', 'invoicing' ), $key ), '1.0.19' );
+
+            call_user_func( array( $this, "set_$key" ), $value );
+        } else {
+            $this->set_prop( $key, $value );
+        }
+
+    }
+
+    /**
      * Margic method for retrieving a property.
      */
     public function __get( $key ) {
@@ -378,10 +378,10 @@  discard block
 block discarded – undo
         // Check if we have a helper method for that.
         if ( method_exists( $this, 'get_' . $key ) ) {
 
-			if ( 'post_type' != $key ) {
-				/* translators: %s: $key Key to set */
-				getpaid_doing_it_wrong( __FUNCTION__, sprintf( __( 'Object data such as "%s" should not be accessed directly. Use getters and setters.', 'invoicing' ), $key ), '1.0.19' );
-			}
+            if ( 'post_type' != $key ) {
+                /* translators: %s: $key Key to set */
+                getpaid_doing_it_wrong( __FUNCTION__, sprintf( __( 'Object data such as "%s" should not be accessed directly. Use getters and setters.', 'invoicing' ), $key ), '1.0.19' );
+            }
 
             return call_user_func( array( $this, 'get_' . $key ) );
         }
@@ -391,513 +391,513 @@  discard block
 block discarded – undo
             return $this->post->$key;
         }
 
-		return $this->get_prop( $key );
-
-    }
-
-	/**
-	 * Get Meta Data by Key.
-	 *
-	 * @since  1.0.19
-	 * @param  string $key Meta Key.
-	 * @param  bool   $single return first found meta with key, or all with $key.
-	 * @param  string $context What the value is for. Valid values are view and edit.
-	 * @return mixed
-	 */
-	public function get_meta( $key = '', $single = true, $context = 'view' ) {
-
-		// Check if this is an internal meta key.
-		$_key = str_replace( '_wpinv', '', $key );
-		$_key = str_replace( 'wpinv', '', $_key );
-		if ( $this->is_internal_meta_key( $key ) ) {
-			$function = 'get_' . $_key;
-
-			if ( is_callable( array( $this, $function ) ) ) {
-				return $this->{$function}();
-			}
-		}
-
-		// Read the meta data if not yet read.
-		$this->maybe_read_meta_data();
-		$meta_data  = $this->get_meta_data();
-		$array_keys = array_keys( wp_list_pluck( $meta_data, 'key' ), $key, true );
-		$value      = $single ? '' : array();
-
-		if ( ! empty( $array_keys ) ) {
-			// We don't use the $this->meta_data property directly here because we don't want meta with a null value (i.e. meta which has been deleted via $this->delete_meta_data()).
-			if ( $single ) {
-				$value = $meta_data[ current( $array_keys ) ]->value;
-			} else {
-				$value = array_intersect_key( $meta_data, array_flip( $array_keys ) );
-			}
-		}
-
-		if ( 'view' === $context ) {
-			$value = apply_filters( $this->get_hook_prefix() . $key, $value, $this );
-		}
-
-		return $value;
-	}
-
-	/**
-	 * See if meta data exists, since get_meta always returns a '' or array().
-	 *
-	 * @since  1.0.19
-	 * @param  string $key Meta Key.
-	 * @return boolean
-	 */
-	public function meta_exists( $key = '' ) {
-		$this->maybe_read_meta_data();
-		$array_keys = wp_list_pluck( $this->get_meta_data(), 'key' );
-		return in_array( $key, $array_keys, true );
-	}
-
-	/**
-	 * Set all meta data from array.
-	 *
-	 * @since 1.0.19
-	 * @param array $data Key/Value pairs.
-	 */
-	public function set_meta_data( $data ) {
-		if ( ! empty( $data ) && is_array( $data ) ) {
-			$this->maybe_read_meta_data();
-			foreach ( $data as $meta ) {
-				$meta = (array) $meta;
-				if ( isset( $meta['key'], $meta['value'], $meta['id'] ) ) {
-					$this->meta_data[] = new GetPaid_Meta_Data(
-						array(
-							'id'    => $meta['id'],
-							'key'   => $meta['key'],
-							'value' => $meta['value'],
-						)
-					);
-				}
-			}
-		}
-	}
-
-	/**
-	 * Add meta data.
-	 *
-	 * @since 1.0.19
-	 *
-	 * @param string       $key Meta key.
-	 * @param string|array $value Meta value.
-	 * @param bool         $unique Should this be a unique key?.
-	 */
-	public function add_meta_data( $key, $value, $unique = false ) {
-		if ( $this->is_internal_meta_key( $key ) ) {
-			$function = 'set_' . $key;
-
-			if ( is_callable( array( $this, $function ) ) ) {
-				return $this->{$function}( $value );
-			}
-		}
-
-		$this->maybe_read_meta_data();
-		if ( $unique ) {
-			$this->delete_meta_data( $key );
-		}
-		$this->meta_data[] = new GetPaid_Meta_Data(
-			array(
-				'key'   => $key,
-				'value' => $value,
-			)
-		);
-
-		$this->save();
-	}
-
-	/**
-	 * Update meta data by key or ID, if provided.
-	 *
-	 * @since  1.0.19
-	 *
-	 * @param  string       $key Meta key.
-	 * @param  string|array $value Meta value.
-	 * @param  int          $meta_id Meta ID.
-	 */
-	public function update_meta_data( $key, $value, $meta_id = 0 ) {
-		if ( $this->is_internal_meta_key( $key ) ) {
-			$function = 'set_' . $key;
-
-			if ( is_callable( array( $this, $function ) ) ) {
-				return $this->{$function}( $value );
-			}
-		}
-
-		$this->maybe_read_meta_data();
-
-		$array_key = false;
-
-		if ( $meta_id ) {
-			$array_keys = array_keys( wp_list_pluck( $this->meta_data, 'id' ), $meta_id, true );
-			$array_key  = $array_keys ? current( $array_keys ) : false;
-		} else {
-			// Find matches by key.
-			$matches = array();
-			foreach ( $this->meta_data as $meta_data_array_key => $meta ) {
-				if ( $meta->key === $key ) {
-					$matches[] = $meta_data_array_key;
-				}
-			}
-
-			if ( ! empty( $matches ) ) {
-				// Set matches to null so only one key gets the new value.
-				foreach ( $matches as $meta_data_array_key ) {
-					$this->meta_data[ $meta_data_array_key ]->value = null;
-				}
-				$array_key = current( $matches );
-			}
-		}
-
-		if ( false !== $array_key ) {
-			$meta        = $this->meta_data[ $array_key ];
-			$meta->key   = $key;
-			$meta->value = $value;
-		} else {
-			$this->add_meta_data( $key, $value, true );
-		}
-	}
-
-	/**
-	 * Delete meta data.
-	 *
-	 * @since 1.0.19
-	 * @param string $key Meta key.
-	 */
-	public function delete_meta_data( $key ) {
-		$this->maybe_read_meta_data();
-		$array_keys = array_keys( wp_list_pluck( $this->meta_data, 'key' ), $key, true );
-
-		if ( $array_keys ) {
-			foreach ( $array_keys as $array_key ) {
-				$this->meta_data[ $array_key ]->value = null;
-			}
-		}
-	}
-
-	/**
-	 * Delete meta data.
-	 *
-	 * @since 1.0.19
-	 * @param int $mid Meta ID.
-	 */
-	public function delete_meta_data_by_mid( $mid ) {
-		$this->maybe_read_meta_data();
-		$array_keys = array_keys( wp_list_pluck( $this->meta_data, 'id' ), (int) $mid, true );
-
-		if ( $array_keys ) {
-			foreach ( $array_keys as $array_key ) {
-				$this->meta_data[ $array_key ]->value = null;
-			}
-		}
-	}
-
-	/**
-	 * Read meta data if null.
-	 *
-	 * @since 1.0.19
-	 */
-	protected function maybe_read_meta_data() {
-		if ( is_null( $this->meta_data ) ) {
-			$this->read_meta_data();
-		}
-	}
-
-	/**
-	 * Read Meta Data from the database. Ignore any internal properties.
-	 * Uses it's own caches because get_metadata does not provide meta_ids.
-	 *
-	 * @since 1.0.19
-	 * @param bool $force_read True to force a new DB read (and update cache).
-	 */
-	public function read_meta_data( $force_read = false ) {
-
-		// Reset meta data.
-		$this->meta_data = array();
-
-		// Maybe abort early.
-		if ( ! $this->get_id() || ! $this->data_store ) {
-			return;
-		}
-
-		// Only read from cache if the cache key is set.
-		$cache_key = null;
-		if ( ! $force_read && ! empty( $this->cache_group ) ) {
-			$cache_key     = GetPaid_Cache_Helper::get_cache_prefix( $this->cache_group ) . GetPaid_Cache_Helper::get_cache_prefix( 'object_' . $this->get_id() ) . 'object_meta_' . $this->get_id();
-			$raw_meta_data = wp_cache_get( $cache_key, $this->cache_group );
-		}
-
-		// Should we force read?
-		if ( empty( $raw_meta_data ) ) {
-			$raw_meta_data = $this->data_store->read_meta( $this );
-
-			if ( ! empty( $cache_key ) ) {
-				wp_cache_set( $cache_key, $raw_meta_data, $this->cache_group );
-			}
+        return $this->get_prop( $key );
+
+    }
+
+    /**
+     * Get Meta Data by Key.
+     *
+     * @since  1.0.19
+     * @param  string $key Meta Key.
+     * @param  bool   $single return first found meta with key, or all with $key.
+     * @param  string $context What the value is for. Valid values are view and edit.
+     * @return mixed
+     */
+    public function get_meta( $key = '', $single = true, $context = 'view' ) {
+
+        // Check if this is an internal meta key.
+        $_key = str_replace( '_wpinv', '', $key );
+        $_key = str_replace( 'wpinv', '', $_key );
+        if ( $this->is_internal_meta_key( $key ) ) {
+            $function = 'get_' . $_key;
+
+            if ( is_callable( array( $this, $function ) ) ) {
+                return $this->{$function}();
+            }
+        }
+
+        // Read the meta data if not yet read.
+        $this->maybe_read_meta_data();
+        $meta_data  = $this->get_meta_data();
+        $array_keys = array_keys( wp_list_pluck( $meta_data, 'key' ), $key, true );
+        $value      = $single ? '' : array();
+
+        if ( ! empty( $array_keys ) ) {
+            // We don't use the $this->meta_data property directly here because we don't want meta with a null value (i.e. meta which has been deleted via $this->delete_meta_data()).
+            if ( $single ) {
+                $value = $meta_data[ current( $array_keys ) ]->value;
+            } else {
+                $value = array_intersect_key( $meta_data, array_flip( $array_keys ) );
+            }
+        }
+
+        if ( 'view' === $context ) {
+            $value = apply_filters( $this->get_hook_prefix() . $key, $value, $this );
+        }
+
+        return $value;
+    }
+
+    /**
+     * See if meta data exists, since get_meta always returns a '' or array().
+     *
+     * @since  1.0.19
+     * @param  string $key Meta Key.
+     * @return boolean
+     */
+    public function meta_exists( $key = '' ) {
+        $this->maybe_read_meta_data();
+        $array_keys = wp_list_pluck( $this->get_meta_data(), 'key' );
+        return in_array( $key, $array_keys, true );
+    }
+
+    /**
+     * Set all meta data from array.
+     *
+     * @since 1.0.19
+     * @param array $data Key/Value pairs.
+     */
+    public function set_meta_data( $data ) {
+        if ( ! empty( $data ) && is_array( $data ) ) {
+            $this->maybe_read_meta_data();
+            foreach ( $data as $meta ) {
+                $meta = (array) $meta;
+                if ( isset( $meta['key'], $meta['value'], $meta['id'] ) ) {
+                    $this->meta_data[] = new GetPaid_Meta_Data(
+                        array(
+                            'id'    => $meta['id'],
+                            'key'   => $meta['key'],
+                            'value' => $meta['value'],
+                        )
+                    );
+                }
+            }
+        }
+    }
+
+    /**
+     * Add meta data.
+     *
+     * @since 1.0.19
+     *
+     * @param string       $key Meta key.
+     * @param string|array $value Meta value.
+     * @param bool         $unique Should this be a unique key?.
+     */
+    public function add_meta_data( $key, $value, $unique = false ) {
+        if ( $this->is_internal_meta_key( $key ) ) {
+            $function = 'set_' . $key;
+
+            if ( is_callable( array( $this, $function ) ) ) {
+                return $this->{$function}( $value );
+            }
+        }
+
+        $this->maybe_read_meta_data();
+        if ( $unique ) {
+            $this->delete_meta_data( $key );
+        }
+        $this->meta_data[] = new GetPaid_Meta_Data(
+            array(
+                'key'   => $key,
+                'value' => $value,
+            )
+        );
+
+        $this->save();
+    }
+
+    /**
+     * Update meta data by key or ID, if provided.
+     *
+     * @since  1.0.19
+     *
+     * @param  string       $key Meta key.
+     * @param  string|array $value Meta value.
+     * @param  int          $meta_id Meta ID.
+     */
+    public function update_meta_data( $key, $value, $meta_id = 0 ) {
+        if ( $this->is_internal_meta_key( $key ) ) {
+            $function = 'set_' . $key;
+
+            if ( is_callable( array( $this, $function ) ) ) {
+                return $this->{$function}( $value );
+            }
+        }
+
+        $this->maybe_read_meta_data();
+
+        $array_key = false;
+
+        if ( $meta_id ) {
+            $array_keys = array_keys( wp_list_pluck( $this->meta_data, 'id' ), $meta_id, true );
+            $array_key  = $array_keys ? current( $array_keys ) : false;
+        } else {
+            // Find matches by key.
+            $matches = array();
+            foreach ( $this->meta_data as $meta_data_array_key => $meta ) {
+                if ( $meta->key === $key ) {
+                    $matches[] = $meta_data_array_key;
+                }
+            }
+
+            if ( ! empty( $matches ) ) {
+                // Set matches to null so only one key gets the new value.
+                foreach ( $matches as $meta_data_array_key ) {
+                    $this->meta_data[ $meta_data_array_key ]->value = null;
+                }
+                $array_key = current( $matches );
+            }
+        }
+
+        if ( false !== $array_key ) {
+            $meta        = $this->meta_data[ $array_key ];
+            $meta->key   = $key;
+            $meta->value = $value;
+        } else {
+            $this->add_meta_data( $key, $value, true );
+        }
+    }
+
+    /**
+     * Delete meta data.
+     *
+     * @since 1.0.19
+     * @param string $key Meta key.
+     */
+    public function delete_meta_data( $key ) {
+        $this->maybe_read_meta_data();
+        $array_keys = array_keys( wp_list_pluck( $this->meta_data, 'key' ), $key, true );
+
+        if ( $array_keys ) {
+            foreach ( $array_keys as $array_key ) {
+                $this->meta_data[ $array_key ]->value = null;
+            }
+        }
+    }
+
+    /**
+     * Delete meta data.
+     *
+     * @since 1.0.19
+     * @param int $mid Meta ID.
+     */
+    public function delete_meta_data_by_mid( $mid ) {
+        $this->maybe_read_meta_data();
+        $array_keys = array_keys( wp_list_pluck( $this->meta_data, 'id' ), (int) $mid, true );
+
+        if ( $array_keys ) {
+            foreach ( $array_keys as $array_key ) {
+                $this->meta_data[ $array_key ]->value = null;
+            }
+        }
+    }
+
+    /**
+     * Read meta data if null.
+     *
+     * @since 1.0.19
+     */
+    protected function maybe_read_meta_data() {
+        if ( is_null( $this->meta_data ) ) {
+            $this->read_meta_data();
+        }
+    }
+
+    /**
+     * Read Meta Data from the database. Ignore any internal properties.
+     * Uses it's own caches because get_metadata does not provide meta_ids.
+     *
+     * @since 1.0.19
+     * @param bool $force_read True to force a new DB read (and update cache).
+     */
+    public function read_meta_data( $force_read = false ) {
+
+        // Reset meta data.
+        $this->meta_data = array();
+
+        // Maybe abort early.
+        if ( ! $this->get_id() || ! $this->data_store ) {
+            return;
+        }
+
+        // Only read from cache if the cache key is set.
+        $cache_key = null;
+        if ( ! $force_read && ! empty( $this->cache_group ) ) {
+            $cache_key     = GetPaid_Cache_Helper::get_cache_prefix( $this->cache_group ) . GetPaid_Cache_Helper::get_cache_prefix( 'object_' . $this->get_id() ) . 'object_meta_' . $this->get_id();
+            $raw_meta_data = wp_cache_get( $cache_key, $this->cache_group );
+        }
+
+        // Should we force read?
+        if ( empty( $raw_meta_data ) ) {
+            $raw_meta_data = $this->data_store->read_meta( $this );
+
+            if ( ! empty( $cache_key ) ) {
+                wp_cache_set( $cache_key, $raw_meta_data, $this->cache_group );
+            }
 }
 
-		// Set meta data.
-		if ( is_array( $raw_meta_data ) ) {
-
-			foreach ( $raw_meta_data as $meta ) {
-				$this->meta_data[] = new GetPaid_Meta_Data(
-					array(
-						'id'    => (int) $meta->meta_id,
-						'key'   => $meta->meta_key,
-						'value' => maybe_unserialize( $meta->meta_value ),
-					)
-				);
-			}
+        // Set meta data.
+        if ( is_array( $raw_meta_data ) ) {
+
+            foreach ( $raw_meta_data as $meta ) {
+                $this->meta_data[] = new GetPaid_Meta_Data(
+                    array(
+                        'id'    => (int) $meta->meta_id,
+                        'key'   => $meta->meta_key,
+                        'value' => maybe_unserialize( $meta->meta_value ),
+                    )
+                );
+            }
 }
 
-	}
-
-	/**
-	 * Update Meta Data in the database.
-	 *
-	 * @since 1.0.19
-	 */
-	public function save_meta_data() {
-		if ( ! $this->data_store || is_null( $this->meta_data ) ) {
-			return;
-		}
-		foreach ( $this->meta_data as $array_key => $meta ) {
-			if ( is_null( $meta->value ) ) {
-				if ( ! empty( $meta->id ) ) {
-					$this->data_store->delete_meta( $this, $meta );
-					unset( $this->meta_data[ $array_key ] );
-				}
-			} elseif ( empty( $meta->id ) ) {
-				$meta->id = $this->data_store->add_meta( $this, $meta );
-				$meta->apply_changes();
-			} else {
-				if ( $meta->get_changes() ) {
-					$this->data_store->update_meta( $this, $meta );
-					$meta->apply_changes();
-				}
-			}
-		}
-		if ( ! empty( $this->cache_group ) ) {
-			$cache_key = GetPaid_Cache_Helper::get_cache_prefix( $this->cache_group ) . GetPaid_Cache_Helper::get_cache_prefix( 'object_' . $this->get_id() ) . 'object_meta_' . $this->get_id();
-			wp_cache_delete( $cache_key, $this->cache_group );
-		}
-	}
-
-	/**
-	 * Set ID.
-	 *
-	 * @since 1.0.19
-	 * @param int $id ID.
-	 */
-	public function set_id( $id ) {
-		$this->id = absint( $id );
-	}
-
-	/**
-	 * Sets item status.
-	 *
-	 * @since 1.0.19
-	 * @param string $status New status.
-	 * @return array details of change.
-	 */
-	public function set_status( $status ) {
+    }
+
+    /**
+     * Update Meta Data in the database.
+     *
+     * @since 1.0.19
+     */
+    public function save_meta_data() {
+        if ( ! $this->data_store || is_null( $this->meta_data ) ) {
+            return;
+        }
+        foreach ( $this->meta_data as $array_key => $meta ) {
+            if ( is_null( $meta->value ) ) {
+                if ( ! empty( $meta->id ) ) {
+                    $this->data_store->delete_meta( $this, $meta );
+                    unset( $this->meta_data[ $array_key ] );
+                }
+            } elseif ( empty( $meta->id ) ) {
+                $meta->id = $this->data_store->add_meta( $this, $meta );
+                $meta->apply_changes();
+            } else {
+                if ( $meta->get_changes() ) {
+                    $this->data_store->update_meta( $this, $meta );
+                    $meta->apply_changes();
+                }
+            }
+        }
+        if ( ! empty( $this->cache_group ) ) {
+            $cache_key = GetPaid_Cache_Helper::get_cache_prefix( $this->cache_group ) . GetPaid_Cache_Helper::get_cache_prefix( 'object_' . $this->get_id() ) . 'object_meta_' . $this->get_id();
+            wp_cache_delete( $cache_key, $this->cache_group );
+        }
+    }
+
+    /**
+     * Set ID.
+     *
+     * @since 1.0.19
+     * @param int $id ID.
+     */
+    public function set_id( $id ) {
+        $this->id = absint( $id );
+    }
+
+    /**
+     * Sets item status.
+     *
+     * @since 1.0.19
+     * @param string $status New status.
+     * @return array details of change.
+     */
+    public function set_status( $status ) {
         $old_status = $this->get_status();
 
-		$this->set_prop( 'status', $status );
-
-		return array(
-			'from' => $old_status,
-			'to'   => $status,
-		);
-    }
-
-	/**
-	 * Set all props to default values.
-	 *
-	 * @since 1.0.19
-	 */
-	public function set_defaults() {
-		$this->data    = $this->default_data;
-		$this->changes = array();
-		$this->set_object_read( false );
-	}
-
-	/**
-	 * Set object read property.
-	 *
-	 * @since 1.0.19
-	 * @param boolean $read Should read?.
-	 */
-	public function set_object_read( $read = true ) {
-		$this->object_read = (bool) $read;
-	}
-
-	/**
-	 * Get object read property.
-	 *
-	 * @since  1.0.19
-	 * @return boolean
-	 */
-	public function get_object_read() {
-		return (bool) $this->object_read;
-	}
-
-	/**
-	 * Set a collection of props in one go, collect any errors, and return the result.
-	 * Only sets using public methods.
-	 *
-	 * @since  1.0.19
-	 *
-	 * @param array  $props Key value pairs to set. Key is the prop and should map to a setter function name.
-	 * @param string $context In what context to run this.
-	 *
-	 * @return bool|WP_Error
-	 */
-	public function set_props( $props, $context = 'set' ) {
-		$errors = false;
-
-		$props = wp_unslash( $props );
-		foreach ( $props as $prop => $value ) {
-			try {
-				/**
-				 * Checks if the prop being set is allowed, and the value is not null.
-				 */
-				if ( is_null( $value ) || in_array( $prop, array( 'prop', 'date_prop', 'meta_data' ), true ) ) {
-					continue;
-				}
-				$setter = "set_$prop";
-
-				if ( is_callable( array( $this, $setter ) ) ) {
-					$this->{$setter}( $value );
-				}
-			} catch ( Exception $e ) {
-				if ( ! $errors ) {
-					$errors = new WP_Error();
-				}
-				$errors->add( $e->getCode(), $e->getMessage() );
-				$this->last_error = $e->getMessage();
-			}
-		}
-
-		return $errors && count( $errors->get_error_codes() ) ? $errors : true;
-	}
-
-	/**
-	 * Sets a prop for a setter method.
-	 *
-	 * This stores changes in a special array so we can track what needs saving
-	 * the the DB later.
-	 *
-	 * @since 1.0.19
-	 * @param string $prop Name of prop to set.
-	 * @param mixed  $value Value of the prop.
-	 */
-	protected function set_prop( $prop, $value ) {
-		if ( array_key_exists( $prop, $this->data ) ) {
-			if ( true === $this->object_read ) {
-				if ( $value !== $this->data[ $prop ] || array_key_exists( $prop, $this->changes ) ) {
-					$this->changes[ $prop ] = $value;
-				}
-			} else {
-				$this->data[ $prop ] = $value;
-			}
-		}
-	}
-
-	/**
-	 * Return data changes only.
-	 *
-	 * @since 1.0.19
-	 * @return array
-	 */
-	public function get_changes() {
-		return $this->changes;
-	}
-
-	/**
-	 * Merge changes with data and clear.
-	 *
-	 * @since 1.0.19
-	 */
-	public function apply_changes() {
-		$this->data    = array_replace( $this->data, $this->changes );
-		$this->changes = array();
-	}
-
-	/**
-	 * Prefix for action and filter hooks on data.
-	 *
-	 * @since  1.0.19
-	 * @return string
-	 */
-	protected function get_hook_prefix() {
-		return 'wpinv_get_' . $this->object_type . '_';
-	}
-
-	/**
-	 * Gets a prop for a getter method.
-	 *
-	 * Gets the value from either current pending changes, or the data itself.
-	 * Context controls what happens to the value before it's returned.
-	 *
-	 * @since  1.0.19
-	 * @param  string $prop Name of prop to get.
-	 * @param  string $context What the value is for. Valid values are view and edit.
-	 * @return mixed
-	 */
-	protected function get_prop( $prop, $context = 'view' ) {
-		$value = null;
-
-		if ( array_key_exists( $prop, $this->data ) ) {
-			$value = array_key_exists( $prop, $this->changes ) ? $this->changes[ $prop ] : $this->data[ $prop ];
-
-			if ( 'view' === $context ) {
-				$value = apply_filters( $this->get_hook_prefix() . $prop, $value, $this );
-			}
-		}
-
-		return $value;
-	}
-
-	/**
-	 * Sets a date prop whilst handling formatting and datetime objects.
-	 *
-	 * @since 1.0.19
-	 * @param string         $prop Name of prop to set.
-	 * @param string|integer $value Value of the prop.
-	 */
-	protected function set_date_prop( $prop, $value ) {
-
-		if ( empty( $value ) ) {
-			$this->set_prop( $prop, null );
-			return;
-		}
-		$this->set_prop( $prop, $value );
-
-	}
-
-	/**
-	 * When invalid data is found, throw an exception unless reading from the DB.
-	 *
-	 * @since 1.0.19
-	 * @param string $code             Error code.
-	 * @param string $message          Error message.
-	 */
-	protected function error( $code, $message ) {
-		$this->last_error = $message;
-	}
-
-	/**
-	 * Checks if the object is saved in the database
-	 *
-	 * @since 1.0.19
-	 * @return bool
-	 */
-	public function exists() {
-		$id = $this->get_id();
-		return ! empty( $id );
-	}
+        $this->set_prop( 'status', $status );
+
+        return array(
+            'from' => $old_status,
+            'to'   => $status,
+        );
+    }
+
+    /**
+     * Set all props to default values.
+     *
+     * @since 1.0.19
+     */
+    public function set_defaults() {
+        $this->data    = $this->default_data;
+        $this->changes = array();
+        $this->set_object_read( false );
+    }
+
+    /**
+     * Set object read property.
+     *
+     * @since 1.0.19
+     * @param boolean $read Should read?.
+     */
+    public function set_object_read( $read = true ) {
+        $this->object_read = (bool) $read;
+    }
+
+    /**
+     * Get object read property.
+     *
+     * @since  1.0.19
+     * @return boolean
+     */
+    public function get_object_read() {
+        return (bool) $this->object_read;
+    }
+
+    /**
+     * Set a collection of props in one go, collect any errors, and return the result.
+     * Only sets using public methods.
+     *
+     * @since  1.0.19
+     *
+     * @param array  $props Key value pairs to set. Key is the prop and should map to a setter function name.
+     * @param string $context In what context to run this.
+     *
+     * @return bool|WP_Error
+     */
+    public function set_props( $props, $context = 'set' ) {
+        $errors = false;
+
+        $props = wp_unslash( $props );
+        foreach ( $props as $prop => $value ) {
+            try {
+                /**
+                 * Checks if the prop being set is allowed, and the value is not null.
+                 */
+                if ( is_null( $value ) || in_array( $prop, array( 'prop', 'date_prop', 'meta_data' ), true ) ) {
+                    continue;
+                }
+                $setter = "set_$prop";
+
+                if ( is_callable( array( $this, $setter ) ) ) {
+                    $this->{$setter}( $value );
+                }
+            } catch ( Exception $e ) {
+                if ( ! $errors ) {
+                    $errors = new WP_Error();
+                }
+                $errors->add( $e->getCode(), $e->getMessage() );
+                $this->last_error = $e->getMessage();
+            }
+        }
+
+        return $errors && count( $errors->get_error_codes() ) ? $errors : true;
+    }
+
+    /**
+     * Sets a prop for a setter method.
+     *
+     * This stores changes in a special array so we can track what needs saving
+     * the the DB later.
+     *
+     * @since 1.0.19
+     * @param string $prop Name of prop to set.
+     * @param mixed  $value Value of the prop.
+     */
+    protected function set_prop( $prop, $value ) {
+        if ( array_key_exists( $prop, $this->data ) ) {
+            if ( true === $this->object_read ) {
+                if ( $value !== $this->data[ $prop ] || array_key_exists( $prop, $this->changes ) ) {
+                    $this->changes[ $prop ] = $value;
+                }
+            } else {
+                $this->data[ $prop ] = $value;
+            }
+        }
+    }
+
+    /**
+     * Return data changes only.
+     *
+     * @since 1.0.19
+     * @return array
+     */
+    public function get_changes() {
+        return $this->changes;
+    }
+
+    /**
+     * Merge changes with data and clear.
+     *
+     * @since 1.0.19
+     */
+    public function apply_changes() {
+        $this->data    = array_replace( $this->data, $this->changes );
+        $this->changes = array();
+    }
+
+    /**
+     * Prefix for action and filter hooks on data.
+     *
+     * @since  1.0.19
+     * @return string
+     */
+    protected function get_hook_prefix() {
+        return 'wpinv_get_' . $this->object_type . '_';
+    }
+
+    /**
+     * Gets a prop for a getter method.
+     *
+     * Gets the value from either current pending changes, or the data itself.
+     * Context controls what happens to the value before it's returned.
+     *
+     * @since  1.0.19
+     * @param  string $prop Name of prop to get.
+     * @param  string $context What the value is for. Valid values are view and edit.
+     * @return mixed
+     */
+    protected function get_prop( $prop, $context = 'view' ) {
+        $value = null;
+
+        if ( array_key_exists( $prop, $this->data ) ) {
+            $value = array_key_exists( $prop, $this->changes ) ? $this->changes[ $prop ] : $this->data[ $prop ];
+
+            if ( 'view' === $context ) {
+                $value = apply_filters( $this->get_hook_prefix() . $prop, $value, $this );
+            }
+        }
+
+        return $value;
+    }
+
+    /**
+     * Sets a date prop whilst handling formatting and datetime objects.
+     *
+     * @since 1.0.19
+     * @param string         $prop Name of prop to set.
+     * @param string|integer $value Value of the prop.
+     */
+    protected function set_date_prop( $prop, $value ) {
+
+        if ( empty( $value ) ) {
+            $this->set_prop( $prop, null );
+            return;
+        }
+        $this->set_prop( $prop, $value );
+
+    }
+
+    /**
+     * When invalid data is found, throw an exception unless reading from the DB.
+     *
+     * @since 1.0.19
+     * @param string $code             Error code.
+     * @param string $message          Error message.
+     */
+    protected function error( $code, $message ) {
+        $this->last_error = $message;
+    }
+
+    /**
+     * Checks if the object is saved in the database
+     *
+     * @since 1.0.19
+     * @return bool
+     */
+    public function exists() {
+        $id = $this->get_id();
+        return ! empty( $id );
+    }
 
 }

Please login to merge, or discard this patch.

Spacing +162 added lines, -162 removed lines patch added patch discarded remove patch

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

Please login to merge, or discard this patch.

		@@ -40,19 +40,19 @@
		block discarded – undo
40	40	<?php
41	41
42	42	if ( ! $invoice->exists() \|\| $invoice->is_draft() ) {
43		- $error = __( 'This invoice was deleted or is not visible.', 'invoicing' );
	43	+ $error = __( 'This invoice was deleted or is not visible.', 'invoicing' );
44	44	} else {
45	45
46		- $user_id = get_current_user_id();
47		- if ( wpinv_require_login_to_checkout() && empty( $user_id ) ) {
48		- $error = __( 'You must be logged in to view this invoice.', 'invoicing' );
49		- $error .= sprintf(
50		- ' <a href="%s">%s</a>',
51		- wp_login_url( $invoice->get_view_url() ),
52		- __( 'Login.', 'invoicing' )
53		- );
54		- } else {
55		- $error = __( 'This invoice is only viewable by clicking on the invoice link that was sent to you via email.', 'invoicing' );
	46	+ $user_id = get_current_user_id();
	47	+ if ( wpinv_require_login_to_checkout() && empty( $user_id ) ) {
	48	+ $error = __( 'You must be logged in to view this invoice.', 'invoicing' );
	49	+ $error .= sprintf(
	50	+ ' <a href="%s">%s</a>',
	51	+ wp_login_url( $invoice->get_view_url() ),
	52	+ __( 'Login.', 'invoicing' )
	53	+ );
	54	+ } else {
	55	+ $error = __( 'This invoice is only viewable by clicking on the invoice link that was sent to you via email.', 'invoicing' );
56	56	}
57	57	}
58	58

		@@ -7,11 +7,11 @@ discard block
		block discarded – undo
7	7	* @version 1.0.19
8	8	*/
9	9
10		-defined( 'ABSPATH' ) \|\| exit;
	10	+defined('ABSPATH') \|\| exit;
11	11
12	12	// Fetch the invoice.
13		-if ( empty( $invoice ) ) {
14		- $invoice = new WPInv_Invoice( $GLOBALS['post'] );
	13	+if (empty($invoice)) {
	14	+ $invoice = new WPInv_Invoice($GLOBALS['post']);
15	15	}
16	16
17	17	?><!DOCTYPE html>
		@@ -21,16 +21,16 @@ discard block
		block discarded – undo
21	21
22	22	<head>
23	23
24		- <meta charset="<?php bloginfo( 'charset' ); ?>">
	24	+ <meta charset="<?php bloginfo('charset'); ?>">
25	25	<meta name="viewport" content="width=device-width, initial-scale=1.0" >
26	26
27	27	<meta name="robots" content="noindex,nofollow">
28	28
29	29	<link rel="profile" href="https://gmpg.org/xfn/11">
30	30
31		- <title><?php esc_html_e( 'Invalid Access', 'invoicing' ); ?></title>
	31	+ <title><?php esc_html_e('Invalid Access', 'invoicing'); ?></title>
32	32
33		- <?php do_action( 'wpinv_invoice_print_head', $invoice ); ?>
	33	+ <?php do_action('wpinv_invoice_print_head', $invoice); ?>
34	34
35	35	</head>
36	36
		@@ -39,20 +39,20 @@ discard block
		block discarded – undo
39	39
40	40	<?php
41	41
42		- if ( ! $invoice->exists() \|\| $invoice->is_draft() ) {
43		- $error = __( 'This invoice was deleted or is not visible.', 'invoicing' );
	42	+ if (!$invoice->exists() \|\| $invoice->is_draft()) {
	43	+ $error = __('This invoice was deleted or is not visible.', 'invoicing');
44	44	} else {
45	45
46	46	$user_id = get_current_user_id();
47		- if ( wpinv_require_login_to_checkout() && empty( $user_id ) ) {
48		- $error = __( 'You must be logged in to view this invoice.', 'invoicing' );
	47	+ if (wpinv_require_login_to_checkout() && empty($user_id)) {
	48	+ $error = __('You must be logged in to view this invoice.', 'invoicing');
49	49	$error .= sprintf(
50	50	' <a href="%s">%s</a>',
51		- wp_login_url( $invoice->get_view_url() ),
52		- __( 'Login.', 'invoicing' )
	51	+ wp_login_url($invoice->get_view_url()),
	52	+ __('Login.', 'invoicing')
53	53	);
54	54	} else {
55		- $error = __( 'This invoice is only viewable by clicking on the invoice link that was sent to you via email.', 'invoicing' );
	55	+ $error = __('This invoice is only viewable by clicking on the invoice link that was sent to you via email.', 'invoicing');
56	56	}
57	57	}
58	58
		@@ -60,8 +60,8 @@ discard block
		block discarded – undo
60	60
61	61	<div class="container">
62	62	<div class="alert alert-danger m-5" role="alert">
63		- <h4 class="alert-heading"><?php esc_html_e( 'Access Denied', 'invoicing' ); ?></h4>
64		- <p><?php echo wp_kses_post( $error ); ?></p>
	63	+ <h4 class="alert-heading"><?php esc_html_e('Access Denied', 'invoicing'); ?></h4>
	64	+ <p><?php echo wp_kses_post($error); ?></p>
65	65	</div>
66	66	</div>
67	67

		@@ -7,7 +7,7 @@
		block discarded – undo
7	7	* @version 1.0.19
8	8	*/
9	9
10		-defined( 'ABSPATH' ) \|\| exit;
	10	+defined('ABSPATH') \|\| exit;
11	11	?>
12	12
13	13	<hr class="featurette-divider" />

		@@ -8,7 +8,7 @@
		block discarded – undo
8	8	*/
9	9
10	10
11		-defined( 'ABSPATH' ) \|\| exit;
	11	+defined('ABSPATH') \|\| exit;
12	12
13	13	?>
14	14	<component :is='form_element.level' v-html='form_element.text'></component>

		@@ -7,44 +7,44 @@
		block discarded – undo
7	7	* @version 1.0.19
8	8	*/
9	9
10		-defined( 'ABSPATH' ) \|\| exit;
	10	+defined('ABSPATH') \|\| exit;
11	11
12		-if ( empty( $form->get_items() ) ) {
	12	+if (empty($form->get_items())) {
13	13	return;
14	14	}
15	15
16		-if ( ! empty( $GLOBALS['getpaid_force_checkbox'] ) ) {
	16	+if (!empty($GLOBALS['getpaid_force_checkbox'])) {
17	17	$items_type = 'checkbox';
18	18	}
19	19
20		-if ( empty( $items_type ) ) {
	20	+if (empty($items_type)) {
21	21	$items_type = 'total';
22	22	}
23	23
24		-do_action( 'getpaid_before_payment_form_items', $form );
	24	+do_action('getpaid_before_payment_form_items', $form);
25	25
26		-switch ( $items_type ) {
	26	+switch ($items_type) {
27	27	case 'radio':
28		- wpinv_get_template( 'payment-forms/variations/radio.php', compact( 'form', 'items_type' ) );
	28	+ wpinv_get_template('payment-forms/variations/radio.php', compact('form', 'items_type'));
29	29	break;
30	30	case 'checkbox':
31		- wpinv_get_template( 'payment-forms/variations/checkbox.php', compact( 'form', 'items_type' ) );
	31	+ wpinv_get_template('payment-forms/variations/checkbox.php', compact('form', 'items_type'));
32	32	break;
33	33	case 'select':
34		- wpinv_get_template( 'payment-forms/variations/select.php', compact( 'form', 'items_type' ) );
	34	+ wpinv_get_template('payment-forms/variations/select.php', compact('form', 'items_type'));
35	35	break;
36	36	}
37	37
38		-do_action( 'getpaid_before_payment_form_cart', $form );
	38	+do_action('getpaid_before_payment_form_cart', $form);
39	39
40	40	// Display the cart totals.
41		-if ( ! empty( $hide_cart ) ) {
	41	+if (!empty($hide_cart)) {
42	42	echo '<div class="d-none">';
43	43	}
44	44
45	45	// Display the cart totals.
46		-wpinv_get_template( 'payment-forms/cart.php', compact( 'form', 'items_type' ) );
	46	+wpinv_get_template('payment-forms/cart.php', compact('form', 'items_type'));
47	47
48		-if ( ! empty( $hide_cart ) ) {
	48	+if (!empty($hide_cart)) {
49	49	echo '</div>';
50	50	}

		@@ -44,7 +44,7 @@
		block discarded – undo
44	44
45	45	// Display the item totals.
46	46	foreach ( $form->get_items() as $item ) {
47		- wpinv_get_template( 'payment-forms/cart-item.php', compact( 'form', 'item', 'columns' ) );
	47	+ wpinv_get_template( 'payment-forms/cart-item.php', compact( 'form', 'item', 'columns' ) );
48	48	}
49	49
50	50	// Display the cart totals.

		@@ -7,34 +7,34 @@ discard block
		block discarded – undo
7	7	* @version 1.0.19
8	8	*/
9	9
10		-defined( 'ABSPATH' ) \|\| exit;
	10	+defined('ABSPATH') \|\| exit;
11	11
12	12	// Cart table columns.
13	13	$columns = array(
14		- 'name' => __( 'Item', 'invoicing' ),
15		- 'price' => __( 'Price', 'invoicing' ),
16		- 'quantity' => __( 'Qty', 'invoicing' ),
17		- 'subtotal' => __( 'Subtotal', 'invoicing' ),
	14	+ 'name' => __('Item', 'invoicing'),
	15	+ 'price' => __('Price', 'invoicing'),
	16	+ 'quantity' => __('Qty', 'invoicing'),
	17	+ 'subtotal' => __('Subtotal', 'invoicing'),
18	18	);
19	19
20		-if ( ! empty( $form->invoice ) ) {
21		- $columns = getpaid_invoice_item_columns( $form->invoice );
	20	+if (!empty($form->invoice)) {
	21	+ $columns = getpaid_invoice_item_columns($form->invoice);
22	22	}
23	23
24		-if ( isset( $columns['tax_rate'] ) ) {
25		- unset( $columns['tax_rate'] );
	24	+if (isset($columns['tax_rate'])) {
	25	+ unset($columns['tax_rate']);
26	26	}
27	27
28		-$columns = apply_filters( 'getpaid_payment_form_cart_table_columns', $columns, $form );
	28	+$columns = apply_filters('getpaid_payment_form_cart_table_columns', $columns, $form);
29	29
30	30	?>
31	31	<div class="getpaid-payment-form-items-cart border form-group mb-3">
32	32
33	33	<div class="getpaid-payment-form-items-cart-header font-weight-bold bg-light border-bottom py-2 px-3">
34	34	<div class="form-row row">
35		- <?php foreach ( $columns as $key => $label ) : ?>
36		- <div class="<?php echo 'name' == $key ? 'col-6' : 'col'; ?> <?php echo ( in_array( $key, array( 'subtotal', 'quantity', 'tax_rate' ) ) ) ? 'd-none d-sm-block' : ''; ?> getpaid-form-cart-item-<?php echo esc_attr( $key ); ?>">
37		- <span><?php echo esc_html( $label ); ?></span>
	35	+ <?php foreach ($columns as $key => $label) : ?>
	36	+ <div class="<?php echo 'name' == $key ? 'col-6' : 'col'; ?> <?php echo (in_array($key, array('subtotal', 'quantity', 'tax_rate'))) ? 'd-none d-sm-block' : ''; ?> getpaid-form-cart-item-<?php echo esc_attr($key); ?>">
	37	+ <span><?php echo esc_html($label); ?></span>
38	38	</div>
39	39	<?php endforeach; ?>
40	40	</div>
		@@ -43,16 +43,16 @@ discard block
		block discarded – undo
43	43	<?php
44	44
45	45	// Display the item totals.
46		- foreach ( $form->get_items() as $item ) {
47		- wpinv_get_template( 'payment-forms/cart-item.php', compact( 'form', 'item', 'columns' ) );
	46	+ foreach ($form->get_items() as $item) {
	47	+ wpinv_get_template('payment-forms/cart-item.php', compact('form', 'item', 'columns'));
48	48	}
49	49
50	50	// Display the cart totals.
51		- wpinv_get_template( 'payment-forms/cart-totals.php', compact( 'form' ) );
	51	+ wpinv_get_template('payment-forms/cart-totals.php', compact('form'));
52	52
53	53	?>
54	54	</div>
55	55
56	56	<?php
57	57
58		-do_action( 'getpaid_after_payment_form_cart', $form );
	58	+do_action('getpaid_after_payment_form_cart', $form);

		@@ -44,12 +44,12 @@
		block discarded – undo
44	44
45	45	// Display the item totals.
46	46	foreach ( $invoice->get_items() as $item ) {
47		- wpinv_get_template( 'emails/invoice-item.php', compact( 'invoice', 'item', 'columns' ) );
	47	+ wpinv_get_template( 'emails/invoice-item.php', compact( 'invoice', 'item', 'columns' ) );
48	48	}
49	49
50	50	// Display the fee totals.
51	51	foreach ( $invoice->get_fees() as $fee ) {
52		- wpinv_get_template( 'emails/fee-item.php', compact( 'invoice', 'fee', 'columns' ) );
	52	+ wpinv_get_template( 'emails/fee-item.php', compact( 'invoice', 'fee', 'columns' ) );
53	53	}
54	54
55	55	?>

		@@ -8,18 +8,18 @@ discard block
		block discarded – undo
8	8	* @var WPInv_Invoice $invoice
9	9	*/
10	10
11		-defined( 'ABSPATH' ) \|\| exit;
	11	+defined('ABSPATH') \|\| exit;
12	12
13		-$column_count = count( $columns );
	13	+$column_count = count($columns);
14	14	?>
15	15
16		-<?php do_action( 'wpinv_before_email_items', $invoice ); ?>
	16	+<?php do_action('wpinv_before_email_items', $invoice); ?>
17	17
18	18
19	19	<div id="wpinv-email-items">
20	20
21	21	<h3 class="invoice-items-title">
22		- <?php echo sprintf( esc_html__( '%s Items', 'invoicing' ), esc_html( ucfirst( $invoice->get_invoice_quote_type() ) ) ); ?>
	22	+ <?php echo sprintf(esc_html__('%s Items', 'invoicing'), esc_html(ucfirst($invoice->get_invoice_quote_type()))); ?>
23	23	</h3>
24	24
25	25	<table class="table table-bordered table-hover">
		@@ -28,9 +28,9 @@ discard block
		block discarded – undo
28	28
29	29	<tr class="wpinv_cart_header_row">
30	30
31		- <?php foreach ( $columns as $key => $label ) : ?>
32		- <th class="<?php echo 'name' == $key ? 'text-left' : 'text-right'; ?> wpinv_cart_item_<?php echo esc_attr( $key ); ?>">
33		- <?php echo esc_html( $label ); ?>
	31	+ <?php foreach ($columns as $key => $label) : ?>
	32	+ <th class="<?php echo 'name' == $key ? 'text-left' : 'text-right'; ?> wpinv_cart_item_<?php echo esc_attr($key); ?>">
	33	+ <?php echo esc_html($label); ?>
34	34	</th>
35	35	<?php endforeach; ?>
36	36
		@@ -43,13 +43,13 @@ discard block
		block discarded – undo
43	43	<?php
44	44
45	45	// Display the item totals.
46		- foreach ( $invoice->get_items() as $item ) {
47		- wpinv_get_template( 'emails/invoice-item.php', compact( 'invoice', 'item', 'columns' ) );
	46	+ foreach ($invoice->get_items() as $item) {
	47	+ wpinv_get_template('emails/invoice-item.php', compact('invoice', 'item', 'columns'));
48	48	}
49	49
50	50	// Display the fee totals.
51		- foreach ( $invoice->get_fees() as $fee ) {
52		- wpinv_get_template( 'emails/fee-item.php', compact( 'invoice', 'fee', 'columns' ) );
	51	+ foreach ($invoice->get_fees() as $fee) {
	52	+ wpinv_get_template('emails/fee-item.php', compact('invoice', 'fee', 'columns'));
53	53	}
54	54
55	55	?>
		@@ -57,7 +57,7 @@ discard block
		block discarded – undo
57	57	</tbody>
58	58
59	59	<tfoot>
60		- <?php wpinv_get_template( 'emails/invoice-totals.php', compact( 'invoice', 'column_count' ) ); ?>
	60	+ <?php wpinv_get_template('emails/invoice-totals.php', compact('invoice', 'column_count')); ?>
61	61	</tfoot>
62	62
63	63	</table>

		@@ -20,7 +20,7 @@
		block discarded – undo
20	20	<a href="<?php echo esc_url( add_query_arg( 'user_id', $invoice->get_user_id(), self_admin_url( 'user-edit.php' ) ) ); ?>"><?php echo esc_html( $invoice->get_user_full_name() ); ?></a>
21	21	<?php
22	22	} else {
23		- echo esc_html( $invoice->get_user_full_name() ); }
	23	+ echo esc_html( $invoice->get_user_full_name() ); }
24	24	?>
25	25	</td>
26	26	</tr>

		@@ -1,51 +1,51 @@
		block discarded – undo
1	1	<?php
2	2	// don't load directly
3		-if ( ! defined( 'ABSPATH' ) ) {
4		- die( '-1' );
	3	+if (!defined('ABSPATH')) {
	4	+ die('-1');
5	5	}
6	6
7		-do_action( 'wpinv_email_before_billing_details', $invoice ); ?>
	7	+do_action('wpinv_email_before_billing_details', $invoice); ?>
8	8	<div id="wpinv-email-billing">
9		- <h3 class="wpinv-address-t"><?php echo esc_html( apply_filters( 'wpinv_email_billing_title', __( 'Billing Details', 'invoicing' ) ) ); ?></h3>
	9	+ <h3 class="wpinv-address-t"><?php echo esc_html(apply_filters('wpinv_email_billing_title', __('Billing Details', 'invoicing'))); ?></h3>
10	10
11	11	<table class="table table-bordered table-sm wpi-billing-details">
12	12	<tbody>
13		- <?php do_action( 'wpinv_email_billing_fields_first', $invoice ); ?>
	13	+ <?php do_action('wpinv_email_billing_fields_first', $invoice); ?>
14	14	<tr class="wpi-receipt-name">
15		- <th class="text-left"><?php esc_html_e( 'Name', 'invoicing' ); ?></th>
	15	+ <th class="text-left"><?php esc_html_e('Name', 'invoicing'); ?></th>
16	16	<td>
17	17	<?php
18		- if ( $sent_to_admin && $invoice->get_user_id() ) {
	18	+ if ($sent_to_admin && $invoice->get_user_id()) {
19	19	?>
20		-<a href="<?php echo esc_url( add_query_arg( 'user_id', $invoice->get_user_id(), self_admin_url( 'user-edit.php' ) ) ); ?>"><?php echo esc_html( $invoice->get_user_full_name() ); ?></a>
	20	+<a href="<?php echo esc_url(add_query_arg('user_id', $invoice->get_user_id(), self_admin_url('user-edit.php'))); ?>"><?php echo esc_html($invoice->get_user_full_name()); ?></a>
21	21	<?php
22	22	} else {
23		- echo esc_html( $invoice->get_user_full_name() ); }
	23	+ echo esc_html($invoice->get_user_full_name()); }
24	24	?>
25	25	</td>
26	26	</tr>
27	27	<tr class="wpi-receipt-email">
28		- <th class="text-left"><?php esc_html_e( 'Email', 'invoicing' ); ?></th>
29		- <td><?php echo esc_html( sanitize_email( $invoice->get_email() ) ); ?></td>
	28	+ <th class="text-left"><?php esc_html_e('Email', 'invoicing'); ?></th>
	29	+ <td><?php echo esc_html(sanitize_email($invoice->get_email())); ?></td>
30	30	</tr>
31		- <?php if ( $invoice->get_company() ) { ?>
	31	+ <?php if ($invoice->get_company()) { ?>
32	32	<tr class="wpi-receipt-company">
33		- <th class="text-left"><?php esc_html_e( 'Company', 'invoicing' ); ?></th>
34		- <td><?php echo esc_html( $invoice->get_company() ); ?></td>
	33	+ <th class="text-left"><?php esc_html_e('Company', 'invoicing'); ?></th>
	34	+ <td><?php echo esc_html($invoice->get_company()); ?></td>
35	35	</tr>
36	36	<?php } ?>
37	37	<tr class="wpi-receipt-address">
38		- <th class="text-left"><?php esc_html_e( 'Address', 'invoicing' ); ?></th>
39		- <td><?php echo wp_kses_post( wpinv_get_invoice_address_markup( $invoice->get_user_info() ) ); ?></td>
	38	+ <th class="text-left"><?php esc_html_e('Address', 'invoicing'); ?></th>
	39	+ <td><?php echo wp_kses_post(wpinv_get_invoice_address_markup($invoice->get_user_info())); ?></td>
40	40	</tr>
41		- <?php if ( $invoice->get_phone() ) { ?>
	41	+ <?php if ($invoice->get_phone()) { ?>
42	42	<tr class="wpi-receipt-phone">
43		- <th class="text-left"><?php esc_html_e( 'Phone', 'invoicing' ); ?></th>
44		- <td><?php echo esc_html( $invoice->get_phone() ); ?></td>
	43	+ <th class="text-left"><?php esc_html_e('Phone', 'invoicing'); ?></th>
	44	+ <td><?php echo esc_html($invoice->get_phone()); ?></td>
45	45	</tr>
46	46	<?php } ?>
47		- <?php do_action( 'wpinv_email_billing_fields_last', $invoice ); ?>
	47	+ <?php do_action('wpinv_email_billing_fields_last', $invoice); ?>
48	48	</tbody>
49	49	</table>
50	50	</div>
51		-<?php do_action( 'wpinv_email_after_billing_details', $invoice ); ?>
	51	+<?php do_action('wpinv_email_after_billing_details', $invoice); ?>

		@@ -4,7 +4,7 @@ discard block
		block discarded – undo
4	4	*
5	5	*/
6	6	if ( ! defined( 'ABSPATH' ) ) {
7		- exit;
	7	+ exit;
8	8	}
9	9
10	10	/**
		@@ -14,179 +14,179 @@ discard block
		block discarded – undo
14	14	*/
15	15	class GetPaid_Payment_Form_Data_Store extends GetPaid_Data_Store_WP {
16	16
17		- /**
18		- * Data stored in meta keys, but not considered "meta" for a form.
19		- *
20		- * @since 1.0.19
21		- * @var array
22		- */
23		- protected $internal_meta_keys = array(
24		- 'wpinv_form_elements',
25		- 'wpinv_form_items',
26		- 'wpinv_form_earned',
27		- 'wpinv_form_refunded',
28		- 'wpinv_form_cancelled',
29		- 'wpinv_form_failed',
30		- );
31		-
32		- /**
33		- * A map of meta keys to data props.
34		- *
35		- * @since 1.0.19
36		- *
37		- * @var array
38		- */
39		- protected $meta_key_to_props = array(
40		- 'wpinv_form_elements' => 'elements',
41		- 'wpinv_form_items' => 'items',
42		- 'wpinv_form_earned' => 'earned',
43		- 'wpinv_form_refunded' => 'refunded',
44		- 'wpinv_form_cancelled' => 'cancelled',
45		- 'wpinv_form_failed' => 'failed',
46		- );
47		-
48		- /*
	17	+ /**
	18	+ * Data stored in meta keys, but not considered "meta" for a form.
	19	+ *
	20	+ * @since 1.0.19
	21	+ * @var array
	22	+ */
	23	+ protected $internal_meta_keys = array(
	24	+ 'wpinv_form_elements',
	25	+ 'wpinv_form_items',
	26	+ 'wpinv_form_earned',
	27	+ 'wpinv_form_refunded',
	28	+ 'wpinv_form_cancelled',
	29	+ 'wpinv_form_failed',
	30	+ );
	31	+
	32	+ /**
	33	+ * A map of meta keys to data props.
	34	+ *
	35	+ * @since 1.0.19
	36	+ *
	37	+ * @var array
	38	+ */
	39	+ protected $meta_key_to_props = array(
	40	+ 'wpinv_form_elements' => 'elements',
	41	+ 'wpinv_form_items' => 'items',
	42	+ 'wpinv_form_earned' => 'earned',
	43	+ 'wpinv_form_refunded' => 'refunded',
	44	+ 'wpinv_form_cancelled' => 'cancelled',
	45	+ 'wpinv_form_failed' => 'failed',
	46	+ );
	47	+
	48	+ /*
49	49	\|--------------------------------------------------------------------------
50	50	\| CRUD Methods
51	51	\|--------------------------------------------------------------------------
52	52	*/
53	53
54		- /**
55		- * Method to create a new form in the database.
56		- *
57		- * @param GetPaid_Payment_Form $form Form object.
58		- */
59		- public function create( &$form ) {
60		- $form->set_version( WPINV_VERSION );
61		- $form->set_date_created( current_time( 'mysql' ) );
62		-
63		- // Create a new post.
64		- $id = wp_insert_post(
65		- apply_filters(
66		- 'getpaid_new_payment_form_data',
67		- array(
68		- 'post_date' => $form->get_date_created( 'edit' ),
69		- 'post_type' => 'wpi_payment_form',
70		- 'post_status' => $this->get_post_status( $form ),
71		- 'ping_status' => 'closed',
72		- 'post_author' => $form->get_author( 'edit' ),
73		- 'post_title' => $form->get_name( 'edit' ),
74		- )
75		- ),
76		- true
77		- );
78		-
79		- if ( $id && ! is_wp_error( $id ) ) {
80		- $form->set_id( $id );
81		- $this->update_post_meta( $form );
82		- $form->save_meta_data();
83		- $form->apply_changes();
84		- $this->clear_caches( $form );
85		- do_action( 'getpaid_create_payment_form', $form );
86		- return true;
87		- }
88		-
89		- if ( is_wp_error( $id ) ) {
90		- $form->last_error = $id->get_error_message();
91		- }
92		-
93		- return false;
94		- }
95		-
96		- /**
97		- * Method to read a form from the database.
98		- *
99		- * @param GetPaid_Payment_Form $form Form object.
100		- *
101		- */
102		- public function read( &$form ) {
103		-
104		- $form->set_defaults();
105		- $form_object = get_post( $form->get_id() );
106		-
107		- if ( ! $form->get_id() \|\| ! $form_object \|\| $form_object->post_type != 'wpi_payment_form' ) {
108		- $form->last_error = __( 'Invalid form.', 'invoicing' );
109		- $form->set_id( 0 );
110		- return false;
111		- }
112		-
113		- $form->set_props(
114		- array(
115		- 'date_created' => 0 < $form_object->post_date ? $form_object->post_date : null,
116		- 'date_modified' => 0 < $form_object->post_modified ? $form_object->post_modified : null,
117		- 'status' => $form_object->post_status,
118		- 'name' => $form_object->post_title,
119		- 'author' => $form_object->post_author,
120		- )
121		- );
122		-
123		- $this->read_object_data( $form, $form_object );
124		- $form->read_meta_data();
125		- $form->set_object_read( true );
126		- do_action( 'getpaid_read_payment_form', $form );
127		-
128		- }
129		-
130		- /**
131		- * Method to update a form in the database.
132		- *
133		- * @param GetPaid_Payment_Form $form Form object.
134		- */
135		- public function update( &$form ) {
136		- $form->save_meta_data();
137		- $form->set_version( WPINV_VERSION );
138		-
139		- if ( null === $form->get_date_created( 'edit' ) ) {
140		- $form->set_date_created( current_time( 'mysql' ) );
141		- }
142		-
143		- // Grab the current status so we can compare.
144		- $previous_status = get_post_status( $form->get_id() );
145		-
146		- $changes = $form->get_changes();
147		-
148		- // Only update the post when the post data changes.
149		- if ( array_intersect( array( 'date_created', 'date_modified', 'status', 'name', 'author' ), array_keys( $changes ) ) ) {
150		- $post_data = array(
151		- 'post_date' => $form->get_date_created( 'edit' ),
152		- 'post_status' => $form->get_status( 'edit' ),
153		- 'post_title' => $form->get_name( 'edit' ),
154		- 'post_author' => $form->get_author( 'edit' ),
155		- 'post_modified' => $form->get_date_modified( 'edit' ),
156		- );
157		-
158		- /**
159		- * When updating this object, to prevent infinite loops, use $wpdb
160		- * to update data, since wp_update_post spawns more calls to the
161		- * save_post action.
162		- *
163		- * This ensures hooks are fired by either WP itself (admin screen save),
164		- * or an update purely from CRUD.
165		- */
166		- if ( doing_action( 'save_post' ) ) {
167		- $GLOBALS['wpdb']->update( $GLOBALS['wpdb']->posts, $post_data, array( 'ID' => $form->get_id() ) );
168		- clean_post_cache( $form->get_id() );
169		- } else {
170		- wp_update_post( array_merge( array( 'ID' => $form->get_id() ), $post_data ) );
171		- }
172		- $form->read_meta_data( true ); // Refresh internal meta data, in case things were hooked into `save_post` or another WP hook.
173		- }
174		- $this->update_post_meta( $form );
175		- $form->apply_changes();
176		- $this->clear_caches( $form );
177		-
178		- // Fire a hook depending on the status - this should be considered a creation if it was previously draft status.
179		- $new_status = $form->get_status( 'edit' );
180		-
181		- if ( $new_status !== $previous_status && in_array( $previous_status, array( 'new', 'auto-draft', 'draft' ), true ) ) {
182		- do_action( 'getpaid_new_payment_form', $form );
183		- } else {
184		- do_action( 'getpaid_update_payment_form', $form );
185		- }
186		-
187		- }
188		-
189		- /*
	54	+ /**
	55	+ * Method to create a new form in the database.
	56	+ *
	57	+ * @param GetPaid_Payment_Form $form Form object.
	58	+ */
	59	+ public function create( &$form ) {
	60	+ $form->set_version( WPINV_VERSION );
	61	+ $form->set_date_created( current_time( 'mysql' ) );
	62	+
	63	+ // Create a new post.
	64	+ $id = wp_insert_post(
	65	+ apply_filters(
	66	+ 'getpaid_new_payment_form_data',
	67	+ array(
	68	+ 'post_date' => $form->get_date_created( 'edit' ),
	69	+ 'post_type' => 'wpi_payment_form',
	70	+ 'post_status' => $this->get_post_status( $form ),
	71	+ 'ping_status' => 'closed',
	72	+ 'post_author' => $form->get_author( 'edit' ),
	73	+ 'post_title' => $form->get_name( 'edit' ),
	74	+ )
	75	+ ),
	76	+ true
	77	+ );
	78	+
	79	+ if ( $id && ! is_wp_error( $id ) ) {
	80	+ $form->set_id( $id );
	81	+ $this->update_post_meta( $form );
	82	+ $form->save_meta_data();
	83	+ $form->apply_changes();
	84	+ $this->clear_caches( $form );
	85	+ do_action( 'getpaid_create_payment_form', $form );
	86	+ return true;
	87	+ }
	88	+
	89	+ if ( is_wp_error( $id ) ) {
	90	+ $form->last_error = $id->get_error_message();
	91	+ }
	92	+
	93	+ return false;
	94	+ }
	95	+
	96	+ /**
	97	+ * Method to read a form from the database.
	98	+ *
	99	+ * @param GetPaid_Payment_Form $form Form object.
	100	+ *
	101	+ */
	102	+ public function read( &$form ) {
	103	+
	104	+ $form->set_defaults();
	105	+ $form_object = get_post( $form->get_id() );
	106	+
	107	+ if ( ! $form->get_id() \|\| ! $form_object \|\| $form_object->post_type != 'wpi_payment_form' ) {
	108	+ $form->last_error = __( 'Invalid form.', 'invoicing' );
	109	+ $form->set_id( 0 );
	110	+ return false;
	111	+ }
	112	+
	113	+ $form->set_props(
	114	+ array(
	115	+ 'date_created' => 0 < $form_object->post_date ? $form_object->post_date : null,
	116	+ 'date_modified' => 0 < $form_object->post_modified ? $form_object->post_modified : null,
	117	+ 'status' => $form_object->post_status,
	118	+ 'name' => $form_object->post_title,
	119	+ 'author' => $form_object->post_author,
	120	+ )
	121	+ );
	122	+
	123	+ $this->read_object_data( $form, $form_object );
	124	+ $form->read_meta_data();
	125	+ $form->set_object_read( true );
	126	+ do_action( 'getpaid_read_payment_form', $form );
	127	+
	128	+ }
	129	+
	130	+ /**
	131	+ * Method to update a form in the database.
	132	+ *
	133	+ * @param GetPaid_Payment_Form $form Form object.
	134	+ */
	135	+ public function update( &$form ) {
	136	+ $form->save_meta_data();
	137	+ $form->set_version( WPINV_VERSION );
	138	+
	139	+ if ( null === $form->get_date_created( 'edit' ) ) {
	140	+ $form->set_date_created( current_time( 'mysql' ) );
	141	+ }
	142	+
	143	+ // Grab the current status so we can compare.
	144	+ $previous_status = get_post_status( $form->get_id() );
	145	+
	146	+ $changes = $form->get_changes();
	147	+
	148	+ // Only update the post when the post data changes.
	149	+ if ( array_intersect( array( 'date_created', 'date_modified', 'status', 'name', 'author' ), array_keys( $changes ) ) ) {
	150	+ $post_data = array(
	151	+ 'post_date' => $form->get_date_created( 'edit' ),
	152	+ 'post_status' => $form->get_status( 'edit' ),
	153	+ 'post_title' => $form->get_name( 'edit' ),
	154	+ 'post_author' => $form->get_author( 'edit' ),
	155	+ 'post_modified' => $form->get_date_modified( 'edit' ),
	156	+ );
	157	+
	158	+ /**
	159	+ * When updating this object, to prevent infinite loops, use $wpdb
	160	+ * to update data, since wp_update_post spawns more calls to the
	161	+ * save_post action.
	162	+ *
	163	+ * This ensures hooks are fired by either WP itself (admin screen save),
	164	+ * or an update purely from CRUD.
	165	+ */
	166	+ if ( doing_action( 'save_post' ) ) {
	167	+ $GLOBALS['wpdb']->update( $GLOBALS['wpdb']->posts, $post_data, array( 'ID' => $form->get_id() ) );
	168	+ clean_post_cache( $form->get_id() );
	169	+ } else {
	170	+ wp_update_post( array_merge( array( 'ID' => $form->get_id() ), $post_data ) );
	171	+ }
	172	+ $form->read_meta_data( true ); // Refresh internal meta data, in case things were hooked into `save_post` or another WP hook.
	173	+ }
	174	+ $this->update_post_meta( $form );
	175	+ $form->apply_changes();
	176	+ $this->clear_caches( $form );
	177	+
	178	+ // Fire a hook depending on the status - this should be considered a creation if it was previously draft status.
	179	+ $new_status = $form->get_status( 'edit' );
	180	+
	181	+ if ( $new_status !== $previous_status && in_array( $previous_status, array( 'new', 'auto-draft', 'draft' ), true ) ) {
	182	+ do_action( 'getpaid_new_payment_form', $form );
	183	+ } else {
	184	+ do_action( 'getpaid_update_payment_form', $form );
	185	+ }
	186	+
	187	+ }
	188	+
	189	+ /*
190	190	\|--------------------------------------------------------------------------
191	191	\| Additional Methods
192	192	\|--------------------------------------------------------------------------

		@@ -3,7 +3,7 @@ discard block
		block discarded – undo
3	3	* GetPaid_Payment_Form_Data_Store class file.
4	4	*
5	5	*/
6		-if ( ! defined( 'ABSPATH' ) ) {
	6	+if (!defined('ABSPATH')) {
7	7	exit;
8	8	}
9	9
		@@ -56,37 +56,37 @@ discard block
		block discarded – undo
56	56	*
57	57	* @param GetPaid_Payment_Form $form Form object.
58	58	*/
59		- public function create( &$form ) {
60		- $form->set_version( WPINV_VERSION );
61		- $form->set_date_created( current_time( 'mysql' ) );
	59	+ public function create(&$form) {
	60	+ $form->set_version(WPINV_VERSION);
	61	+ $form->set_date_created(current_time('mysql'));
62	62
63	63	// Create a new post.
64	64	$id = wp_insert_post(
65	65	apply_filters(
66	66	'getpaid_new_payment_form_data',
67	67	array(
68		- 'post_date' => $form->get_date_created( 'edit' ),
	68	+ 'post_date' => $form->get_date_created('edit'),
69	69	'post_type' => 'wpi_payment_form',
70		- 'post_status' => $this->get_post_status( $form ),
	70	+ 'post_status' => $this->get_post_status($form),
71	71	'ping_status' => 'closed',
72		- 'post_author' => $form->get_author( 'edit' ),
73		- 'post_title' => $form->get_name( 'edit' ),
	72	+ 'post_author' => $form->get_author('edit'),
	73	+ 'post_title' => $form->get_name('edit'),
74	74	)
75	75	),
76	76	true
77	77	);
78	78
79		- if ( $id && ! is_wp_error( $id ) ) {
80		- $form->set_id( $id );
81		- $this->update_post_meta( $form );
	79	+ if ($id && !is_wp_error($id)) {
	80	+ $form->set_id($id);
	81	+ $this->update_post_meta($form);
82	82	$form->save_meta_data();
83	83	$form->apply_changes();
84		- $this->clear_caches( $form );
85		- do_action( 'getpaid_create_payment_form', $form );
	84	+ $this->clear_caches($form);
	85	+ do_action('getpaid_create_payment_form', $form);
86	86	return true;
87	87	}
88	88
89		- if ( is_wp_error( $id ) ) {
	89	+ if (is_wp_error($id)) {
90	90	$form->last_error = $id->get_error_message();
91	91	}
92	92
		@@ -99,14 +99,14 @@ discard block
		block discarded – undo
99	99	* @param GetPaid_Payment_Form $form Form object.
100	100	*
101	101	*/
102		- public function read( &$form ) {
	102	+ public function read(&$form) {
103	103
104	104	$form->set_defaults();
105		- $form_object = get_post( $form->get_id() );
	105	+ $form_object = get_post($form->get_id());
106	106
107		- if ( ! $form->get_id() \|\| ! $form_object \|\| $form_object->post_type != 'wpi_payment_form' ) {
108		- $form->last_error = __( 'Invalid form.', 'invoicing' );
109		- $form->set_id( 0 );
	107	+ if (!$form->get_id() \|\| !$form_object \|\| $form_object->post_type != 'wpi_payment_form') {
	108	+ $form->last_error = __('Invalid form.', 'invoicing');
	109	+ $form->set_id(0);
110	110	return false;
111	111	}
112	112
		@@ -120,10 +120,10 @@ discard block
		block discarded – undo
120	120	)
121	121	);
122	122
123		- $this->read_object_data( $form, $form_object );
	123	+ $this->read_object_data($form, $form_object);
124	124	$form->read_meta_data();
125		- $form->set_object_read( true );
126		- do_action( 'getpaid_read_payment_form', $form );
	125	+ $form->set_object_read(true);
	126	+ do_action('getpaid_read_payment_form', $form);
127	127
128	128	}
129	129
		@@ -132,27 +132,27 @@ discard block
		block discarded – undo
132	132	*
133	133	* @param GetPaid_Payment_Form $form Form object.
134	134	*/
135		- public function update( &$form ) {
	135	+ public function update(&$form) {
136	136	$form->save_meta_data();
137		- $form->set_version( WPINV_VERSION );
	137	+ $form->set_version(WPINV_VERSION);
138	138
139		- if ( null === $form->get_date_created( 'edit' ) ) {
140		- $form->set_date_created( current_time( 'mysql' ) );
	139	+ if (null === $form->get_date_created('edit')) {
	140	+ $form->set_date_created(current_time('mysql'));
141	141	}
142	142
143	143	// Grab the current status so we can compare.
144		- $previous_status = get_post_status( $form->get_id() );
	144	+ $previous_status = get_post_status($form->get_id());
145	145
146	146	$changes = $form->get_changes();
147	147
148	148	// Only update the post when the post data changes.
149		- if ( array_intersect( array( 'date_created', 'date_modified', 'status', 'name', 'author' ), array_keys( $changes ) ) ) {
	149	+ if (array_intersect(array('date_created', 'date_modified', 'status', 'name', 'author'), array_keys($changes))) {
150	150	$post_data = array(
151		- 'post_date' => $form->get_date_created( 'edit' ),
152		- 'post_status' => $form->get_status( 'edit' ),
153		- 'post_title' => $form->get_name( 'edit' ),
154		- 'post_author' => $form->get_author( 'edit' ),
155		- 'post_modified' => $form->get_date_modified( 'edit' ),
	151	+ 'post_date' => $form->get_date_created('edit'),
	152	+ 'post_status' => $form->get_status('edit'),
	153	+ 'post_title' => $form->get_name('edit'),
	154	+ 'post_author' => $form->get_author('edit'),
	155	+ 'post_modified' => $form->get_date_modified('edit'),
156	156	);
157	157
158	158	/**
		@@ -163,25 +163,25 @@ discard block
		block discarded – undo
163	163	* This ensures hooks are fired by either WP itself (admin screen save),
164	164	* or an update purely from CRUD.
165	165	*/
166		- if ( doing_action( 'save_post' ) ) {
167		- $GLOBALS['wpdb']->update( $GLOBALS['wpdb']->posts, $post_data, array( 'ID' => $form->get_id() ) );
168		- clean_post_cache( $form->get_id() );
	166	+ if (doing_action('save_post')) {
	167	+ $GLOBALS['wpdb']->update($GLOBALS['wpdb']->posts, $post_data, array('ID' => $form->get_id()));
	168	+ clean_post_cache($form->get_id());
169	169	} else {
170		- wp_update_post( array_merge( array( 'ID' => $form->get_id() ), $post_data ) );
	170	+ wp_update_post(array_merge(array('ID' => $form->get_id()), $post_data));
171	171	}
172		- $form->read_meta_data( true ); // Refresh internal meta data, in case things were hooked into `save_post` or another WP hook.
	172	+ $form->read_meta_data(true); // Refresh internal meta data, in case things were hooked into `save_post` or another WP hook.
173	173	}
174		- $this->update_post_meta( $form );
	174	+ $this->update_post_meta($form);
175	175	$form->apply_changes();
176		- $this->clear_caches( $form );
	176	+ $this->clear_caches($form);
177	177
178	178	// Fire a hook depending on the status - this should be considered a creation if it was previously draft status.
179		- $new_status = $form->get_status( 'edit' );
	179	+ $new_status = $form->get_status('edit');
180	180
181		- if ( $new_status !== $previous_status && in_array( $previous_status, array( 'new', 'auto-draft', 'draft' ), true ) ) {
182		- do_action( 'getpaid_new_payment_form', $form );
	181	+ if ($new_status !== $previous_status && in_array($previous_status, array('new', 'auto-draft', 'draft'), true)) {
	182	+ do_action('getpaid_new_payment_form', $form);
183	183	} else {
184		- do_action( 'getpaid_update_payment_form', $form );
	184	+ do_action('getpaid_update_payment_form', $form);
185	185	}
186	186
187	187	}

AyeCode / invoicing

Pull Request — master (#857)

Status

Category

Indentation +11 added lines, -11 removed lines patch added patch discarded remove patch

Spacing +15 added lines, -15 removed lines patch added patch discarded remove patch

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

Spacing +13 added lines, -13 removed lines patch added patch discarded remove patch

Indentation +1 added lines, -1 removed lines patch added patch discarded remove patch

Spacing +17 added lines, -17 removed lines patch added patch discarded remove patch

Indentation +2 added lines, -2 removed lines patch added patch discarded remove patch

Spacing +12 added lines, -12 removed lines patch added patch discarded remove patch

Indentation +1 added lines, -1 removed lines patch added patch discarded remove patch

Spacing +21 added lines, -21 removed lines patch added patch discarded remove patch

Indentation +169 added lines, -169 removed lines patch added patch discarded remove patch

Spacing +44 added lines, -44 removed lines patch added patch discarded remove patch

Indentation +857 added lines, -857 removed lines patch added patch discarded remove patch

Spacing +162 added lines, -162 removed lines patch added patch discarded remove patch