wpinv_convert_old_subscriptions()

Complexity

Conditions	23
Paths	545

Size

Total Lines	107
Code Lines	70

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
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();

The assignment to $is_free_trial is dead and can be removed.

The function WPInv_Invoice::is_free_trial() has been deprecated.

Are you sure the assignment to $is_free_trial is correct as $invoice->is_free_trial() targeting WPInv_Invoice::is_free_trial() seems to always return null.

        $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 );

The function WPInv_Invoice::get_invoice_date() has been deprecated.

        }
        
        $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() ) {

The function WPInv_Invoice::is_free_trial() has been deprecated.

Are you sure the usage of $invoice->is_free_trial() targeting WPInv_Invoice::is_free_trial() seems to always return null.

            $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,

The property user_id does not exist on WPInv_Invoice. Since you implemented __get, consider adding a @property annotation.

            '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 ) ) {

The method get_times_billed() does not exist on integer.

The property status does not exist on integer.

                $new_sub->complete(); // Mark completed if all times billed
            }