wpinv-upgrade-functions.php ➔ wpinv_update_new_email_settings() - Code Metrics - Inspection of "modify upgrade function to fix issues with activat..." - AyeCode/invoicing - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#171)

unknown

created 2018-06-02 09:47 UTC

wpinv-upgrade-functions.php ➔ wpinv_update_new_email_settings() B

↳ Parent: Project

Complexity

Conditions	3
Paths	3

Size

Total Lines	26
Code Lines	18

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	3
eloc	18
nc	3
nop	0
dl	0
loc	26
rs	8.8571
c	0
b	0
f	0

<?php
/**
 * Upgrade related functions.
 *
 * @since 1.0.0
 */

/**
 * Perform automatic upgrades when necessary.
 *
 * @since 1.0.0
*/
function wpinv_automatic_upgrade() {
    $wpi_version = get_option( 'wpinv_version' );

    if ( $wpi_version == WPINV_VERSION ) {
        return;
    }
    
    if ( version_compare( $wpi_version, '0.0.5', '<' ) ) {
        wpinv_v005_upgrades();
    }
    
    if ( version_compare( $wpi_version, '1.0.3', '<' ) ) {
        wpinv_v110_upgrades();
    }
    
    update_option( 'wpinv_version', WPINV_VERSION );
}
add_action( 'admin_init', 'wpinv_automatic_upgrade' );

function wpinv_v005_upgrades() {
    global $wpdb;
    
    // Invoices status
    $results = $wpdb->get_results( "SELECT ID FROM " . $wpdb->posts . " WHERE post_type = 'wpi_invoice' AND post_status IN( 'pending', 'processing', 'onhold', 'refunded', 'cancelled', 'failed', 'renewal' )" );
    if ( !empty( $results ) ) {
        $wpdb->query( "UPDATE " . $wpdb->posts . " SET post_status = CONCAT( 'wpi-', post_status ) WHERE post_type = 'wpi_invoice' AND post_status IN( 'pending', 'processing', 'onhold', 'refunded', 'cancelled', 'failed', 'renewal' )" );
        
        // Clean post cache
        foreach ( $results as $row ) {
            clean_post_cache( $row->ID );
        }
    }
    
    // Item meta key changes
    $query = "SELECT DISTINCT post_id FROM " . $wpdb->postmeta . " WHERE meta_key IN( '_wpinv_item_id', '_wpinv_package_id', '_wpinv_post_id', '_wpinv_cpt_name', '_wpinv_cpt_singular_name' )";
    $results = $wpdb->get_results( $query );
    
    if ( !empty( $results ) ) {
        $wpdb->query( "UPDATE " . $wpdb->postmeta . " SET meta_key = '_wpinv_custom_id' WHERE meta_key IN( '_wpinv_item_id', '_wpinv_package_id', '_wpinv_post_id' )" );
        $wpdb->query( "UPDATE " . $wpdb->postmeta . " SET meta_key = '_wpinv_custom_name' WHERE meta_key = '_wpinv_cpt_name'" );
        $wpdb->query( "UPDATE " . $wpdb->postmeta . " SET meta_key = '_wpinv_custom_singular_name' WHERE meta_key = '_wpinv_cpt_singular_name'" );
        
        foreach ( $results as $row ) {
            clean_post_cache( $row->post_id );
        }
    }

    wpinv_add_admin_caps();
}

function wpinv_v110_upgrades() {
    // Upgrade email settings
    wpinv_update_new_email_settings();
    
    // Add Subscription tables
    $db = new WPInv_Subscriptions_DB;
    @$db->create_table();
// For example instead of
@mkdir($dir);

// Better use
if (@mkdir($dir) === false) {
    throw new \RuntimeException('The directory '.$dir.' could not be created.');
}

    wpinv_convert_old_subscriptions();
}

function wpinv_convert_old_subscriptions() {
    global $wpdb;

    $query = "SELECT ". $wpdb->posts .".ID FROM ". $wpdb->posts ." INNER JOIN ". $wpdb->postmeta ." ON ( ". $wpdb->posts .".ID = ". $wpdb->postmeta .".post_id ) WHERE 1=1  AND ". $wpdb->postmeta .".meta_key = '_wpinv_subscr_status' AND (". $wpdb->postmeta .".meta_value = 'pending' OR ". $wpdb->postmeta .".meta_value = 'active' OR ". $wpdb->postmeta .".meta_value = 'cancelled' OR ". $wpdb->postmeta .".meta_value = 'completed' OR ". $wpdb->postmeta .".meta_value = 'expired' OR ". $wpdb->postmeta .".meta_value = 'trialling' OR ". $wpdb->postmeta .".meta_value = 'failing') AND ". $wpdb->posts .".post_type = 'wpi_invoice' GROUP BY ". $wpdb->posts .".ID ORDER BY ". $wpdb->posts .".ID ASC";

    $results = $wpdb->get_results( $query );

    if ( empty( $results ) ) {
        return;
    }

    foreach ( $results as $row ) {
        $invoice = new WPInv_Invoice( $row->ID );

        if ( empty( $invoice->ID ) ) {
            continue;
        }

        if ( $invoice->has_status( 'wpi-renewal' ) ) {
            continue;
        }
        
        $item = $invoice->get_recurring( true );

        if ( empty( $item ) ) {
            continue;
        }

        $is_free_trial          = $invoice->is_free_trial();
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
        $profile_id             = get_post_meta( $invoice->ID, '_wpinv_subscr_profile_id', true );
        $subscription_status    = get_post_meta( $invoice->ID, '_wpinv_subscr_status', true );
        $transaction_id         = $invoice->get_transaction_id();

        // Last invoice
        $query          = "SELECT ID, post_date FROM ". $wpdb->posts ." WHERE post_type = 'wpi_invoice' AND post_parent = '" . $invoice->ID . "' ORDER BY ID DESC LIMIT 1";
        $last_payment   = $wpdb->get_row( $query );

        if ( !empty( $last_payment ) ) {
            $invoice_date       = $last_payment->post_date;
            
            $meta_profile_id     = get_post_meta( $last_payment->ID, '_wpinv_subscr_profile_id', true );
            $meta_transaction_id = get_post_meta( $last_payment->ID, '_wpinv_transaction_id', true );

            if ( !empty( $meta_profile_id ) ) {
                $profile_id  = $meta_profile_id;
            }

            if ( !empty( $meta_transaction_id ) ) {
                $transaction_id  = $meta_transaction_id;
            }
        } else {
            $invoice_date       = $invoice->get_invoice_date( false );
        }
        
        $profile_id             = empty( $profile_id ) ? $invoice->ID : $profile_id;
        $status                 = empty( $subscription_status ) ? 'pending' : $subscription_status;
        
        $period                 = $item->get_recurring_period( true );
        $interval               = $item->get_recurring_interval();
        $bill_times             = (int)$item->get_recurring_limit();
        $add_period             = $interval . ' ' . $period;
        $trial_period           = '';

        if ( $invoice->is_free_trial() ) {
            $trial_period       = $item->get_trial_period( true );
            $free_interval      = $item->get_trial_interval();
            $trial_period       = $free_interval . ' ' . $trial_period;

            if ( empty( $last_payment ) ) {
                $add_period     = $trial_period;
            }
        }

        $expiration             = date_i18n( 'Y-m-d H:i:s', strtotime( '+' . $add_period  . ' 23:59:59', strtotime( $invoice_date ) ) );
        if ( strtotime( $expiration ) <  strtotime( date_i18n( 'Y-m-d' ) ) ) {
            if ( $status == 'active' || $status == 'trialling' || $status == 'pending' ) {
                $status = 'expired';
            }
        }

        $args = array(
            'product_id'        => $item->ID,
            'customer_id'       => $invoice->user_id,
            'parent_payment_id' => $invoice->ID,
            'status'            => $status,
            'frequency'         => $interval,
            'period'            => $period,
            'initial_amount'    => $invoice->get_total(),
            'recurring_amount'  => $invoice->get_recurring_details( 'total' ),
            'bill_times'        => $bill_times,
            'created'           => $invoice_date,
            'expiration'        => $expiration,
            'trial_period'      => $trial_period,
            'profile_id'        => $profile_id,
            'transaction_id'    => $transaction_id,
        );

        $subs_db      = new WPInv_Subscriptions_DB;
        $subs         = $subs_db->get_subscriptions( array( 'parent_payment_id' => $invoice->ID, 'number' => 1 ) );
        $subscription = reset( $subs );

        if ( empty( $subscription ) || $subscription->id <= 0 ) {
            $subscription = new WPInv_Subscription();
            $new_sub = $subscription->create( $args );

            if ( !empty( $bill_times ) && $new_sub->get_times_billed() >= $bill_times && ( 'active' == $new_sub->status || 'trialling' == $new_sub->status ) ) {
                $new_sub->complete(); // Mark completed if all times billed
            }
        }
    }
}

function wpinv_update_new_email_settings() {
    global $wpinv_options;

    $current_options = get_option( 'wpinv_settings', array() );
    $options = array(
        'email_new_invoice_body' => __( '<p>Hi Admin,</p><p>You have received payment invoice from {name}.</p>', 'invoicing' ),
        'email_cancelled_invoice_body' => __( '<p>Hi Admin,</p><p>The invoice #{invoice_number} from {site_title} has been cancelled.</p>', 'invoicing' ),
        'email_failed_invoice_body' => __( '<p>Hi Admin,</p><p>Payment for invoice #{invoice_number} from {site_title} has been failed.</p>', 'invoicing' ),
        'email_onhold_invoice_body' => __( '<p>Hi {name},</p><p>Your invoice is on-hold until we confirm your payment has been received.</p>', 'invoicing' ),
        'email_processing_invoice_body' => __( '<p>Hi {name},</p><p>Your invoice has been received at {site_title} and is now being processed.</p>', 'invoicing' ),
        'email_refunded_invoice_body' => __( '<p>Hi {name},</p><p>Your invoice on {site_title} has been refunded.</p>', 'invoicing' ),
        'email_user_invoice_body' => __( '<p>Hi {name},</p><p>An invoice has been created for you on {site_title}. To view / pay for this invoice please use the following link: <a class="btn btn-success" href="{invoice_link}">View / Pay</a></p>', 'invoicing' ),
        'email_user_note_body' => __( '<p>Hi {name},</p><p>Following note has been added to your {invoice_label}:</p><blockquote class="wpinv-note">{customer_note}</blockquote>', 'invoicing' ),
        'email_overdue_body' => __( '<p>Hi {full_name},</p><p>This is just a friendly reminder that your invoice <a href="{invoice_link}">#{invoice_number}</a> {is_was} due on {invoice_due_date}.</p><p>The total of this invoice is {invoice_total}</p><p>To view / pay now for this invoice please use the following link: <a class="btn btn-success" href="{invoice_link}">View / Pay</a></p>', 'invoicing' ),
    );

    foreach ($options as $option => $value){
        if (!isset($current_options[$option])) {
            $current_options[$option] = $value;
        }
    }

    $wpinv_options = $current_options;

    update_option( 'wpinv_settings', $current_options );
}

1		<?php
2		/**
3		* Upgrade related functions.
4		*
5		* @since 1.0.0
6		*/
7
8		/**
9		* Perform automatic upgrades when necessary.
10		*
11		* @since 1.0.0
12		*/
13		function wpinv_automatic_upgrade() {
14		$wpi_version = get_option( 'wpinv_version' );
15
16		if ( $wpi_version == WPINV_VERSION ) {
17		return;
18		}
19
20		if ( version_compare( $wpi_version, '0.0.5', '<' ) ) {
21		wpinv_v005_upgrades();
22		}
23
24		if ( version_compare( $wpi_version, '1.0.3', '<' ) ) {
25		wpinv_v110_upgrades();
26		}
27
28		update_option( 'wpinv_version', WPINV_VERSION );
29		}
30		add_action( 'admin_init', 'wpinv_automatic_upgrade' );
31
32		function wpinv_v005_upgrades() {
33		global $wpdb;
34
35		// Invoices status
36		$results = $wpdb->get_results( "SELECT ID FROM " . $wpdb->posts . " WHERE post_type = 'wpi_invoice' AND post_status IN( 'pending', 'processing', 'onhold', 'refunded', 'cancelled', 'failed', 'renewal' )" );
37		if ( !empty( $results ) ) {
38		$wpdb->query( "UPDATE " . $wpdb->posts . " SET post_status = CONCAT( 'wpi-', post_status ) WHERE post_type = 'wpi_invoice' AND post_status IN( 'pending', 'processing', 'onhold', 'refunded', 'cancelled', 'failed', 'renewal' )" );
39
40		// Clean post cache
41		foreach ( $results as $row ) {
42		clean_post_cache( $row->ID );
43		}
44		}
45
46		// Item meta key changes
47		$query = "SELECT DISTINCT post_id FROM " . $wpdb->postmeta . " WHERE meta_key IN( '_wpinv_item_id', '_wpinv_package_id', '_wpinv_post_id', '_wpinv_cpt_name', '_wpinv_cpt_singular_name' )";
48		$results = $wpdb->get_results( $query );
49
50		if ( !empty( $results ) ) {
51		$wpdb->query( "UPDATE " . $wpdb->postmeta . " SET meta_key = '_wpinv_custom_id' WHERE meta_key IN( '_wpinv_item_id', '_wpinv_package_id', '_wpinv_post_id' )" );
52		$wpdb->query( "UPDATE " . $wpdb->postmeta . " SET meta_key = '_wpinv_custom_name' WHERE meta_key = '_wpinv_cpt_name'" );
53		$wpdb->query( "UPDATE " . $wpdb->postmeta . " SET meta_key = '_wpinv_custom_singular_name' WHERE meta_key = '_wpinv_cpt_singular_name'" );
54
55		foreach ( $results as $row ) {
56		clean_post_cache( $row->post_id );
57		}
58		}
59
60		wpinv_add_admin_caps();
61		}
62
63		function wpinv_v110_upgrades() {
64		// Upgrade email settings
65		wpinv_update_new_email_settings();
66
67		// Add Subscription tables
68		$db = new WPInv_Subscriptions_DB;
69		@$db->create_table();
		0 ignored issues – show Security Best Practice introduced 2017-10-27 13:34 UTC by Report Bug Copy Issue Report It seems like you do not handle an error condition here. This can introduce security issues, and is generally not recommended. If you suppress an error, we recommend checking for the error condition explicitly: // For example instead of @mkdir($dir); // Better use if (@mkdir($dir) === false) { throw new \RuntimeException('The directory '.$dir.' could not be created.'); } Loading history...
70
71		wpinv_convert_old_subscriptions();
72		}
73
74		function wpinv_convert_old_subscriptions() {
75		global $wpdb;
76
77		$query = "SELECT ". $wpdb->posts .".ID FROM ". $wpdb->posts ." INNER JOIN ". $wpdb->postmeta ." ON ( ". $wpdb->posts .".ID = ". $wpdb->postmeta .".post_id ) WHERE 1=1 AND ". $wpdb->postmeta .".meta_key = '_wpinv_subscr_status' AND (". $wpdb->postmeta .".meta_value = 'pending' OR ". $wpdb->postmeta .".meta_value = 'active' OR ". $wpdb->postmeta .".meta_value = 'cancelled' OR ". $wpdb->postmeta .".meta_value = 'completed' OR ". $wpdb->postmeta .".meta_value = 'expired' OR ". $wpdb->postmeta .".meta_value = 'trialling' OR ". $wpdb->postmeta .".meta_value = 'failing') AND ". $wpdb->posts .".post_type = 'wpi_invoice' GROUP BY ". $wpdb->posts .".ID ORDER BY ". $wpdb->posts .".ID ASC";
78
79		$results = $wpdb->get_results( $query );
80
81		if ( empty( $results ) ) {
82		return;
83		}
84
85		foreach ( $results as $row ) {
86		$invoice = new WPInv_Invoice( $row->ID );
87
88		if ( empty( $invoice->ID ) ) {
89		continue;
90		}
91
92		if ( $invoice->has_status( 'wpi-renewal' ) ) {
93		continue;
94		}
95
96		$item = $invoice->get_recurring( true );
97
98		if ( empty( $item ) ) {
99		continue;
100		}
101
102		$is_free_trial = $invoice->is_free_trial();
		0 ignored issues – show Unused Code introduced 2017-12-04 13:49 UTC by Report Bug Copy Issue Report `$is_free_trial` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
103		$profile_id = get_post_meta( $invoice->ID, '_wpinv_subscr_profile_id', true );
104		$subscription_status = get_post_meta( $invoice->ID, '_wpinv_subscr_status', true );
105		$transaction_id = $invoice->get_transaction_id();
106
107		// Last invoice
108		$query = "SELECT ID, post_date FROM ". $wpdb->posts ." WHERE post_type = 'wpi_invoice' AND post_parent = '" . $invoice->ID . "' ORDER BY ID DESC LIMIT 1";
109		$last_payment = $wpdb->get_row( $query );
110
111		if ( !empty( $last_payment ) ) {
112		$invoice_date = $last_payment->post_date;
113
114		$meta_profile_id = get_post_meta( $last_payment->ID, '_wpinv_subscr_profile_id', true );
115		$meta_transaction_id = get_post_meta( $last_payment->ID, '_wpinv_transaction_id', true );
116
117		if ( !empty( $meta_profile_id ) ) {
118		$profile_id = $meta_profile_id;
119		}
120
121		if ( !empty( $meta_transaction_id ) ) {
122		$transaction_id = $meta_transaction_id;
123		}
124		} else {
125		$invoice_date = $invoice->get_invoice_date( false );
126		}
127
128		$profile_id = empty( $profile_id ) ? $invoice->ID : $profile_id;
129		$status = empty( $subscription_status ) ? 'pending' : $subscription_status;
130
131		$period = $item->get_recurring_period( true );
132		$interval = $item->get_recurring_interval();
133		$bill_times = (int)$item->get_recurring_limit();
134		$add_period = $interval . ' ' . $period;
135		$trial_period = '';
136
137	View Code Duplication	if ( $invoice->is_free_trial() ) {
138		$trial_period = $item->get_trial_period( true );
139		$free_interval = $item->get_trial_interval();
140		$trial_period = $free_interval . ' ' . $trial_period;
141
142		if ( empty( $last_payment ) ) {
143		$add_period = $trial_period;
144		}
145		}
146
147		$expiration = date_i18n( 'Y-m-d H:i:s', strtotime( '+' . $add_period . ' 23:59:59', strtotime( $invoice_date ) ) );
148		if ( strtotime( $expiration ) < strtotime( date_i18n( 'Y-m-d' ) ) ) {
149		if ( $status == 'active' \|\| $status == 'trialling' \|\| $status == 'pending' ) {
150		$status = 'expired';
151		}
152		}
153
154		$args = array(
155		'product_id' => $item->ID,
156		'customer_id' => $invoice->user_id,
157		'parent_payment_id' => $invoice->ID,
158		'status' => $status,
159		'frequency' => $interval,
160		'period' => $period,
161		'initial_amount' => $invoice->get_total(),
162		'recurring_amount' => $invoice->get_recurring_details( 'total' ),
163		'bill_times' => $bill_times,
164		'created' => $invoice_date,
165		'expiration' => $expiration,
166		'trial_period' => $trial_period,
167		'profile_id' => $profile_id,
168		'transaction_id' => $transaction_id,
169		);
170
171		$subs_db = new WPInv_Subscriptions_DB;
172		$subs = $subs_db->get_subscriptions( array( 'parent_payment_id' => $invoice->ID, 'number' => 1 ) );
173		$subscription = reset( $subs );
174
175		if ( empty( $subscription ) \|\| $subscription->id <= 0 ) {
176		$subscription = new WPInv_Subscription();
177		$new_sub = $subscription->create( $args );
178
179		if ( !empty( $bill_times ) && $new_sub->get_times_billed() >= $bill_times && ( 'active' == $new_sub->status \|\| 'trialling' == $new_sub->status ) ) {
180		$new_sub->complete(); // Mark completed if all times billed
181		}
182		}
183		}
184		}
185
186		function wpinv_update_new_email_settings() {
187		global $wpinv_options;
188
189		$current_options = get_option( 'wpinv_settings', array() );
190		$options = array(
191		'email_new_invoice_body' => __( '<p>Hi Admin,</p><p>You have received payment invoice from {name}.</p>', 'invoicing' ),
192		'email_cancelled_invoice_body' => __( '<p>Hi Admin,</p><p>The invoice #{invoice_number} from {site_title} has been cancelled.</p>', 'invoicing' ),
193		'email_failed_invoice_body' => __( '<p>Hi Admin,</p><p>Payment for invoice #{invoice_number} from {site_title} has been failed.</p>', 'invoicing' ),
194		'email_onhold_invoice_body' => __( '<p>Hi {name},</p><p>Your invoice is on-hold until we confirm your payment has been received.</p>', 'invoicing' ),
195		'email_processing_invoice_body' => __( '<p>Hi {name},</p><p>Your invoice has been received at {site_title} and is now being processed.</p>', 'invoicing' ),
196		'email_refunded_invoice_body' => __( '<p>Hi {name},</p><p>Your invoice on {site_title} has been refunded.</p>', 'invoicing' ),
197		'email_user_invoice_body' => __( '<p>Hi {name},</p><p>An invoice has been created for you on {site_title}. To view / pay for this invoice please use the following link: <a class="btn btn-success" href="{invoice_link}">View / Pay</a></p>', 'invoicing' ),
198		'email_user_note_body' => __( '<p>Hi {name},</p><p>Following note has been added to your {invoice_label}:</p><blockquote class="wpinv-note">{customer_note}</blockquote>', 'invoicing' ),
199		'email_overdue_body' => __( '<p>Hi {full_name},</p><p>This is just a friendly reminder that your invoice <a href="{invoice_link}">#{invoice_number}</a> {is_was} due on {invoice_due_date}.</p><p>The total of this invoice is {invoice_total}</p><p>To view / pay now for this invoice please use the following link: <a class="btn btn-success" href="{invoice_link}">View / Pay</a></p>', 'invoicing' ),
200		);
201
202		foreach ($options as $option => $value){
203		if (!isset($current_options[$option])) {
204		$current_options[$option] = $value;
205		}
206		}
207
208		$wpinv_options = $current_options;
209
210		update_option( 'wpinv_settings', $current_options );
211		}

AyeCode / invoicing

Pull Request — master (#171)

wpinv-upgrade-functions.php ➔ wpinv_update_new_email_settings() B

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like