WPInv_Subscriptions::wpinv_add_update_subscription() - Code Metrics - Inspection of "Merge pull request #153 from kprajapatii/master" - AyeCode/invoicing - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 011c37...24d4c3 )

by Stiofan

created 2018-04-25 13:33 UTC

WPInv_Subscriptions::wpinv_add_update_subscription() C

↳ Parent: Project

Complexity

Conditions	13
Paths	9

Size

Total Lines	72
Code Lines	49

Duplication

Lines	8
Ratio	11.11 %

Importance

Changes

Metric	Value
cc	13
eloc	49
nc	9
nop	2
dl	8
loc	72
rs	5.5073
c	0
b	0
f	0

How to fix Long Method Complexity

<?php
// Exit if accessed directly.
if (!defined( 'ABSPATH' ) ) exit;

function wpinv_subscription_init() {
    return WPInv_Subscriptions::instance();
}
add_action( 'plugins_loaded', 'wpinv_subscription_init', 100 );

/**
 * WPInv_Subscriptions Class.
 *
 * @since 1.0.0
 */
class WPInv_Subscriptions {

    private static $instance;

    /**
     * Main WPInv_Subscriptions Instance
     */
    public static function instance() {
        if ( ! isset( self::$instance ) ) {
            self::$instance = new WPInv_Subscriptions;

            self::$instance->init();
        }

        return self::$instance;
    }

    /**
     * Constructor -- prevent new instances
     *
     * @since 1.0.0
     */
    private function __construct(){

    }

    /**
     * Get things started
     *
     * Sets up inits actions and filters
     *
     * @since 1.0.0
     */
    function init() {


        self::setup_constants();
        self::actions();
        self::filters();
class User
{
    private $email;

    public function getEmail()
    {
        return $this->email;
    }

    public function setEmail($email)
    {
        $this->email = $email;
    }
}

    }

    /**
     * Setup plugin constants.
     *
     * @access private
     * @since 1.0.0
     * @return void
     */
    private function setup_constants() {

        // Make sure CAL_GREGORIAN is defined.
        if ( ! defined( 'CAL_GREGORIAN' ) ) {
            define( 'CAL_GREGORIAN', 1 );
        }
    }

    /**
     * Add our actions
     *
     * @since  1.0.0
     * @return void
     */
    private function actions() {

        add_action( 'admin_menu', array( $this, 'wpinv_subscriptions_list' ), 10 );
        add_action( 'admin_notices', array( $this, 'notices' ) );
        add_action( 'init', array( $this, 'wpinv_post_actions' ) );
        add_action( 'init', array( $this, 'wpinv_get_actions' ) );
        add_action( 'wpinv_cancel_subscription', array( $this, 'wpinv_process_cancellation' ) );
        add_action( 'wpinv_checkout_before_send_to_gateway', array( $this, 'wpinv_checkout_add_subscription' ), -999, 2 );
        add_action( 'wpinv_subscriptions_front_notices', array( $this, 'notices' ) );
    }

    /**
     * Add our filters
     *
     * @since  1.0
     * @return void
     */
    private function filters() {

    }

    /**
     * Register our Subscriptions submenu
     *
     * @since  2.4
     * @return void
     */
    public function wpinv_subscriptions_list() {
        add_submenu_page(
            'wpinv',
            __( 'Subscriptions', 'invoicing' ),
            __( 'Subscriptions', 'invoicing' ),
            'manage_invoicing',
            'wpinv-subscriptions',
            'wpinv_subscriptions_page'
        );
    }

    public function notices() {

        if( empty( $_GET['wpinv-message'] ) ) {
            return;
        }

        $type    = 'updated';
        $message = '';

        switch( strtolower( $_GET['wpinv-message'] ) ) {

            case 'updated' :

                $message = __( 'Subscription updated successfully.', 'invoicing' );

                break;

            case 'deleted' :

                $message = __( 'Subscription deleted successfully.', 'invoicing' );

                break;

            case 'cancelled' :

                $message = __( 'Subscription cancelled successfully.', 'invoicing' );

                break;

        }

        if ( ! empty( $message ) ) {
            echo '<div class="' . esc_attr( $type ) . '"><p>' . $message . '</p></div>';
        }

    }

    /**
     * Every wpinv_action present in $_GET is called using WordPress's do_action function.
     * These functions are called on init.
     *
     * @since 1.0.0
     * @return void
     */
    function wpinv_get_actions() {

        if ( isset( $_GET['wpinv_action'] ) ) {
            do_action( 'wpinv_' . $_GET['wpinv_action'], $_GET );
        }
    }

    /**
     * Every wpinv_action present in $_POST is called using WordPress's do_action function.
     * These functions are called on init.
     *
     * @since 1.0.0
     * @return void
     */
    function wpinv_post_actions() {

        if ( isset( $_POST['wpinv_action'] ) ) {
            do_action( 'wpinv_' . $_POST['wpinv_action'], $_POST );
        }
    }

    /**
     * Get pretty subscription frequency
     *
     * @param $period
     * @param int $frequency_count The frequency of the period.
     * @return mixed|string|void
     */
    public static function wpinv_get_pretty_subscription_frequency( $period, $frequency_count = 1) {
        $frequency = '';
        //Format period details
        switch ( $period ) {
            case 'day' :
                $frequency = sprintf( _n('%d Day', '%d Days', $frequency_count, 'invoicing'), $frequency_count);
                break;
            case 'week' :
                $frequency = sprintf( _n('%d Week', '%d Weeks', $frequency_count, 'invoicing'), $frequency_count);
                break;
            case 'month' :
                $frequency = sprintf( _n('%d Month', '%d Months', $frequency_count, 'invoicing'), $frequency_count);
                break;
            case 'year' :
                $frequency = sprintf( _n('%d Year', '%d Years', $frequency_count, 'invoicing'), $frequency_count);
                break;
            default :
                $frequency = apply_filters( 'wpinv_recurring_subscription_frequency', $frequency, $period, $frequency_count );
                break;
        }

        return $frequency;

    }

    /**
     * Handles cancellation requests for a subscription
     *
     * @access      public
     * @since       1.0.0
     * @return      void
     */
    public function wpinv_process_cancellation( $data ) {


        if( empty( $data['sub_id'] ) ) {
            return;
        }

        if( ! is_user_logged_in() ) {
            return;
        }

        if( ! wp_verify_nonce( $data['_wpnonce'], 'wpinv-recurring-cancel' ) ) {
            wp_die( __( 'Error', 'invoicing' ), __( 'Nonce verification failed', 'invoicing' ), array( 'response' => 403 ) );
        }

        $data['sub_id'] = absint( $data['sub_id'] );
        $subscription   = new WPInv_Subscription( $data['sub_id'] );

        if( ! $subscription->can_cancel() ) {
            wp_die( __( 'Error', 'invoicing' ), __( 'This subscription cannot be cancelled', 'invoicing' ), array( 'response' => 403 ) );
        }

        try {

            do_action( 'wpinv_recurring_cancel_' . $subscription->gateway . '_subscription', $subscription, true );

            $subscription->cancel();

            if( is_admin() ) {

                wp_redirect( admin_url( 'admin.php?page=wpinv-subscriptions&wpinv-message=cancelled&id=' . $subscription->id ) );
                exit;

            } else {

                $redirect = remove_query_arg( array( '_wpnonce', 'wpinv_action', 'sub_id' ), add_query_arg( array( 'wpinv-message' => 'cancelled' ) ) );
                $redirect = apply_filters( 'wpinv_recurring_cancellation_redirect', $redirect, $subscription );
                wp_safe_redirect( $redirect );
                exit;

            }

        } catch ( Exception $e ) {
            wp_die( __( 'Error', 'invoicing' ), $e->getMessage(), array( 'response' => 403 ) );
        }

    }

    /**
     * Create subscription on checkout
     *
     * @access      public
     * @since       1.0.0
     * @return      void
     */
    public function wpinv_checkout_add_subscription( $invoice, $invoice_data ) {

        if ( ! ( ! empty( $invoice->ID ) && $invoice->is_recurring() ) ) {
            return;
        }
        
        $item               = $invoice->get_recurring( true );
        if ( empty( $item ) ) {
            return;
        }

        $invoice_date       = $invoice->get_invoice_date( false );
        $status             = 'pending';

        $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() ) {
            $status         = 'trialling';
            $trial_period   = $item->get_trial_period( true );
            $free_interval  = $item->get_trial_interval();
            $trial_period   = $free_interval . ' ' . $trial_period;

            $add_period     = $trial_period;
        }

        $expiration         = date_i18n( 'Y-m-d H:i:s', strtotime( '+' . $add_period  . ' 23:59:59', strtotime( $invoice_date ) ) );

        $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'        => '',
            'transaction_id'    => '',
        );

        $subscription = wpinv_get_subscription( $invoice );

        if ( empty( $subscription ) ) {
            $subscription = new WPInv_Subscription();
            $subscription->create( $args );
        }
        
        return $subscription;
    }
}

1		<?php
2		// Exit if accessed directly.
3		if (!defined( 'ABSPATH' ) ) exit;
4
5		function wpinv_subscription_init() {
6		return WPInv_Subscriptions::instance();
7		}
8		add_action( 'plugins_loaded', 'wpinv_subscription_init', 100 );
9
10		/**
11		* WPInv_Subscriptions Class.
12		*
13		* @since 1.0.0
14		*/
15		class WPInv_Subscriptions {
16
17		private static $instance;
18
19		/**
20		* Main WPInv_Subscriptions Instance
21		*/
22		public static function instance() {
23		if ( ! isset( self::$instance ) ) {
24		self::$instance = new WPInv_Subscriptions;
25
26		self::$instance->init();
27		}
28
29		return self::$instance;
30		}
31
32		/**
33		* Constructor -- prevent new instances
34		*
35		* @since 1.0.0
36		*/
37		private function __construct(){
38
39		}
40
41		/**
42		* Get things started
43		*
44		* Sets up inits actions and filters
45		*
46		* @since 1.0.0
47		*/
48		function init() {
		0 ignored issues – show Best Practice introduced 2017-08-25 14:11 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history...
49
50		self::setup_constants();
51		self::actions();
52		self::filters();
		0 ignored issues – show Unused Code introduced 2017-08-25 14:11 UTC by Report Bug Copy Issue Report The call to the method `WPInv_Subscriptions::filters()` seems un-needed as the method has no side-effects. PHP Analyzer performs a side-effects analysis of your code. A side-effect is basically anything that might be visible after the scope of the method is left. Let’s take a look at an example: class User { private $email; public function getEmail() { return $this->email; } public function setEmail($email) { $this->email = $email; } } If we look at the `getEmail()` method, we can see that it has no side-effect. Whether you call this method or not, no future calls to other methods are affected by this. As such code as the following is useless: $user = new User(); $user->getEmail(); // This line could safely be removed as it has no effect. On the hand, if we look at the `setEmail()`, this method _has_ side-effects. In the following case, we could not remove the method call: $user = new User(); $user->setEmail('email@domain'); // This line has a side-effect (it changes an // instance variable). Loading history...
53
54		}
55
56		/**
57		* Setup plugin constants.
58		*
59		* @access private
60		* @since 1.0.0
61		* @return void
62		*/
63		private function setup_constants() {
64
65		// Make sure CAL_GREGORIAN is defined.
66		if ( ! defined( 'CAL_GREGORIAN' ) ) {
67		define( 'CAL_GREGORIAN', 1 );
68		}
69		}
70
71		/**
72		* Add our actions
73		*
74		* @since 1.0.0
75		* @return void
76		*/
77		private function actions() {
78
79		add_action( 'admin_menu', array( $this, 'wpinv_subscriptions_list' ), 10 );
80		add_action( 'admin_notices', array( $this, 'notices' ) );
81		add_action( 'init', array( $this, 'wpinv_post_actions' ) );
82		add_action( 'init', array( $this, 'wpinv_get_actions' ) );
83		add_action( 'wpinv_cancel_subscription', array( $this, 'wpinv_process_cancellation' ) );
84		add_action( 'wpinv_checkout_before_send_to_gateway', array( $this, 'wpinv_checkout_add_subscription' ), -999, 2 );
85		add_action( 'wpinv_subscriptions_front_notices', array( $this, 'notices' ) );
86		}
87
88		/**
89		* Add our filters
90		*
91		* @since 1.0
92		* @return void
93		*/
94		private function filters() {
95
96		}
97
98		/**
99		* Register our Subscriptions submenu
100		*
101		* @since 2.4
102		* @return void
103		*/
104		public function wpinv_subscriptions_list() {
105		add_submenu_page(
106		'wpinv',
107		__( 'Subscriptions', 'invoicing' ),
108		__( 'Subscriptions', 'invoicing' ),
109		'manage_invoicing',
110		'wpinv-subscriptions',
111		'wpinv_subscriptions_page'
112		);
113		}
114
115		public function notices() {
116
117		if( empty( $_GET['wpinv-message'] ) ) {
118		return;
119		}
120
121		$type = 'updated';
122		$message = '';
123
124		switch( strtolower( $_GET['wpinv-message'] ) ) {
125
126		case 'updated' :
127
128		$message = __( 'Subscription updated successfully.', 'invoicing' );
129
130		break;
131
132		case 'deleted' :
133
134		$message = __( 'Subscription deleted successfully.', 'invoicing' );
135
136		break;
137
138		case 'cancelled' :
139
140		$message = __( 'Subscription cancelled successfully.', 'invoicing' );
141
142		break;
143
144		}
145
146		if ( ! empty( $message ) ) {
147		echo '<div class="' . esc_attr( $type ) . '"><p>' . $message . '</p></div>';
148		}
149
150		}
151
152		/**
153		* Every wpinv_action present in $_GET is called using WordPress's do_action function.
154		* These functions are called on init.
155		*
156		* @since 1.0.0
157		* @return void
158		*/
159		function wpinv_get_actions() {
		0 ignored issues – show Best Practice introduced 2017-08-25 14:11 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history...
160		if ( isset( $_GET['wpinv_action'] ) ) {
161		do_action( 'wpinv_' . $_GET['wpinv_action'], $_GET );
162		}
163		}
164
165		/**
166		* Every wpinv_action present in $_POST is called using WordPress's do_action function.
167		* These functions are called on init.
168		*
169		* @since 1.0.0
170		* @return void
171		*/
172		function wpinv_post_actions() {
		0 ignored issues – show Best Practice introduced 2017-08-25 14:11 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history...
173		if ( isset( $_POST['wpinv_action'] ) ) {
174		do_action( 'wpinv_' . $_POST['wpinv_action'], $_POST );
175		}
176		}
177
178		/**
179		* Get pretty subscription frequency
180		*
181		* @param $period
182		* @param int $frequency_count The frequency of the period.
183		* @return mixed\|string\|void
184		*/
185		public static function wpinv_get_pretty_subscription_frequency( $period, $frequency_count = 1) {
186		$frequency = '';
187		//Format period details
188		switch ( $period ) {
189		case 'day' :
190		$frequency = sprintf( _n('%d Day', '%d Days', $frequency_count, 'invoicing'), $frequency_count);
191		break;
192		case 'week' :
193		$frequency = sprintf( _n('%d Week', '%d Weeks', $frequency_count, 'invoicing'), $frequency_count);
194		break;
195		case 'month' :
196		$frequency = sprintf( _n('%d Month', '%d Months', $frequency_count, 'invoicing'), $frequency_count);
197		break;
198		case 'year' :
199		$frequency = sprintf( _n('%d Year', '%d Years', $frequency_count, 'invoicing'), $frequency_count);
200		break;
201		default :
202		$frequency = apply_filters( 'wpinv_recurring_subscription_frequency', $frequency, $period, $frequency_count );
203		break;
204		}
205
206		return $frequency;
207
208		}
209
210		/**
211		* Handles cancellation requests for a subscription
212		*
213		* @access public
214		* @since 1.0.0
215		* @return void
216		*/
217		public function wpinv_process_cancellation( $data ) {
218
219
220		if( empty( $data['sub_id'] ) ) {
221		return;
222		}
223
224		if( ! is_user_logged_in() ) {
225		return;
226		}
227
228		if( ! wp_verify_nonce( $data['_wpnonce'], 'wpinv-recurring-cancel' ) ) {
229		wp_die( __( 'Error', 'invoicing' ), __( 'Nonce verification failed', 'invoicing' ), array( 'response' => 403 ) );
230		}
231
232		$data['sub_id'] = absint( $data['sub_id'] );
233		$subscription = new WPInv_Subscription( $data['sub_id'] );
234
235		if( ! $subscription->can_cancel() ) {
236		wp_die( __( 'Error', 'invoicing' ), __( 'This subscription cannot be cancelled', 'invoicing' ), array( 'response' => 403 ) );
237		}
238
239		try {
240
241		do_action( 'wpinv_recurring_cancel_' . $subscription->gateway . '_subscription', $subscription, true );
242
243		$subscription->cancel();
244
245		if( is_admin() ) {
246
247		wp_redirect( admin_url( 'admin.php?page=wpinv-subscriptions&wpinv-message=cancelled&id=' . $subscription->id ) );
248		exit;
249
250		} else {
251
252		$redirect = remove_query_arg( array( '_wpnonce', 'wpinv_action', 'sub_id' ), add_query_arg( array( 'wpinv-message' => 'cancelled' ) ) );
253		$redirect = apply_filters( 'wpinv_recurring_cancellation_redirect', $redirect, $subscription );
254		wp_safe_redirect( $redirect );
255		exit;
256
257		}
258
259		} catch ( Exception $e ) {
260		wp_die( __( 'Error', 'invoicing' ), $e->getMessage(), array( 'response' => 403 ) );
261		}
262
263		}
264
265		/**
266		* Create subscription on checkout
267		*
268		* @access public
269		* @since 1.0.0
270		* @return void
271		*/
272		public function wpinv_checkout_add_subscription( $invoice, $invoice_data ) {
		0 ignored issues – show Unused Code introduced 2017-08-25 14:11 UTC by Report Bug Copy Issue Report The parameter `$invoice_data` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history...
273		if ( ! ( ! empty( $invoice->ID ) && $invoice->is_recurring() ) ) {
274		return;
275		}
276
277		$item = $invoice->get_recurring( true );
278		if ( empty( $item ) ) {
279		return;
280		}
281
282		$invoice_date = $invoice->get_invoice_date( false );
283		$status = 'pending';
284
285		$period = $item->get_recurring_period( true );
286		$interval = $item->get_recurring_interval();
287		$bill_times = (int)$item->get_recurring_limit();
288		$add_period = $interval . ' ' . $period;
289		$trial_period = '';
290
291	View Code Duplication	if ( $invoice->is_free_trial() ) {
292		$status = 'trialling';
293		$trial_period = $item->get_trial_period( true );
294		$free_interval = $item->get_trial_interval();
295		$trial_period = $free_interval . ' ' . $trial_period;
296
297		$add_period = $trial_period;
298		}
299
300		$expiration = date_i18n( 'Y-m-d H:i:s', strtotime( '+' . $add_period . ' 23:59:59', strtotime( $invoice_date ) ) );
301
302		$args = array(
303		'product_id' => $item->ID,
304		'customer_id' => $invoice->user_id,
305		'parent_payment_id' => $invoice->ID,
306		'status' => $status,
307		'frequency' => $interval,
308		'period' => $period,
309		'initial_amount' => $invoice->get_total(),
310		'recurring_amount' => $invoice->get_recurring_details( 'total' ),
311		'bill_times' => $bill_times,
312		'created' => $invoice_date,
313		'expiration' => $expiration,
314		'trial_period' => $trial_period,
315		'profile_id' => '',
316		'transaction_id' => '',
317		);
318
319		$subscription = wpinv_get_subscription( $invoice );
320
321		if ( empty( $subscription ) ) {
322		$subscription = new WPInv_Subscription();
323		$subscription->create( $args );
324		}
325
326		return $subscription;
327		}
328		}

AyeCode / invoicing

Push — master ( 011c37...24d4c3 )

WPInv_Subscriptions::wpinv_add_update_subscription() C

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

Duplication Side-by-Side

Filter issues like