FS_Subscription

Complexity

Total Complexity

6

Size/Duplication

Total Lines	113
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	1

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
A	get_type()	0	3	1
A	is_active()	0	4	2
A	is_first_payment_pending()	0	3	1
A	has_trial()	0	3	1

<?php

For compatibility and reusability of your code, PSR1 recommends that a file should introduce either new symbols (like classes, functions, etc.) or have side-effects (like outputting something, or including other files), but not both at the same time. The first symbol is defined on line 13 and the first side effect is on line 10.

	/**
	 * @package     Freemius
	 * @copyright   Copyright (c) 2015, Freemius, Inc.
	 * @license     https://www.gnu.org/licenses/gpl-3.0.html GNU General Public License Version 3
	 * @since       1.0.9
	 */

	if ( ! defined( 'ABSPATH' ) ) {
		exit;
	}

	class FS_Subscription extends FS_Entity {

PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions.

		#region Properties

		/**
		 * @var number
		 */
		public $user_id;
		/**
		 * @var number
		 */
		public $install_id;
		/**
		 * @var number
		 */
		public $plan_id;
		/**
		 * @var number
		 */
		public $license_id;
		/**
		 * @var float
		 */
		public $total_gross;
		/**
		 * @var float
		 */
		public $amount_per_cycle;
		/**
		 * @var int # of months
		 */
		public $billing_cycle;
		/**
		 * @var float
		 */
		public $outstanding_balance;
		/**
		 * @var int
		 */
		public $failed_payments;
		/**
		 * @var string
		 */
		public $gateway;
		/**
		 * @var string
		 */
		public $external_id;
		/**
		 * @var string|null
		 */
		public $trial_ends;
		/**
		 * @var string|null Datetime of the next payment, or null if cancelled
		 */
		public $next_payment;
		/**
		 * @var string|null
		 */
		public $vat_id;
		/**
		 * @var string Two characters country code
		 */
		public $country_code;

		#endregion Properties

		/**
		 * @param object|bool $subscription
		 */
		function __construct( $subscription = false ) {

It is generally recommended to explicitly declare the visibility for methods.

			parent::__construct( $subscription );
		}

		static function get_type() {

It is generally recommended to explicitly declare the visibility for methods.

			return 'subscription';
		}

		/**
		 * Check if subscription is active.
		 *
		 * @author Vova Feldman (@svovaf)
		 * @since  1.0.9
		 *
		 * @return bool
		 */
		function is_active() {

It is generally recommended to explicitly declare the visibility for methods.

			return ! empty( $this->next_payment ) &&
			       ( strtotime( $this->next_payment ) > WP_FS__SCRIPT_START_TIME );
		}

		/**
		 * Subscription considered to be new without any payments
		 * if the next payment should be made within less than 24 hours
		 * from the subscription creation.
		 *
		 * @author Vova Feldman (@svovaf)
		 * @since  1.0.9
		 *
		 * @return bool
		 */
		function is_first_payment_pending() {

It is generally recommended to explicitly declare the visibility for methods.

			return ( WP_FS__TIME_24_HOURS_IN_SEC >= strtotime( $this->next_payment ) - strtotime( $this->created ) );
		}

		/**
		 * @author Vova Feldman (@svovaf)
		 * @since  1.1.7
		 */
		function has_trial() {

It is generally recommended to explicitly declare the visibility for methods.

			return ! is_null( $this->trial_ends );
		}
	}