lsx_health_plan\classes\Frontend

Complexity

Total Complexity

14

Size/Duplication

Total Lines	103
Duplicated Lines	0 %

Importance

Changes	3
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	get_instance()	0	6	2
A	load_classes()	0	18	1
A	__construct()	0	4	2
B	redirect()	0	13	9

<?php
namespace lsx_health_plan\classes;

/**
 * LSX Health Plan Frontend Class.
 *
 * @package lsx-health-plan
 */
class Frontend {

	/**
	 * Holds class instance
	 *
	 * @since 1.0.0
	 *
	 * @var      object \lsx_health_plan\classes\Frontend()
	 */
	protected static $instance = null;

	/**
	 * @var object \lsx_health_plan\classes\frontend\Endpoints();
	 */
	public $endpoints;

	/**
	 * @var object \lsx_health_plan\classes\frontend\Gallery();
	 */
	public $gallery;

	/**
	 * @var object \lsx_health_plan\classes\frontend\Plan_Status();
	 */
	public $plan_status;

	/**
	 * @var object \lsx_health_plan\classes\frontend\General();
	 */
	public $general;

	/**
	 * @var object \lsx_health_plan\classes\frontend\Template_Redirects();
	 */
	public $template_redirects;

	/**
	 * Contructor
	 */
	public function __construct() {
		if ( ! is_admin() ) {
			$this->load_classes();
			add_action( 'template_redirect', array( $this, 'redirect' ) );
			
		}	
	}

	/**
	 * Return an instance of this class.
	 *
	 * @since 1.0.0
	 *
	 * @return    object \lsx_health_plan\classes\Frontend()    A single instance of this class.
	 */
	public static function get_instance() {
		// If the single instance hasn't been set, set it now.
		if ( null === self::$instance ) {
			self::$instance = new self();
		}
		return self::$instance;
	}

	/**
	 * Loads the variable classes and the static classes.
	 */
	private function load_classes() {
		require_once LSX_HEALTH_PLAN_PATH . 'classes/frontend/class-endpoints.php';
		$this->endpoints = frontend\Endpoints::get_instance();

		require_once LSX_HEALTH_PLAN_PATH . 'classes/frontend/class-modals.php';
		$this->modals = Modals::get_instance();

The property modals does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

		require_once LSX_HEALTH_PLAN_PATH . 'classes/frontend/class-gallery.php';
		$this->gallery = frontend\Gallery::get_instance();

		require_once LSX_HEALTH_PLAN_PATH . 'classes/frontend/class-plan-status.php';
		$this->plan_status = frontend\Plan_Status::get_instance();

		require_once LSX_HEALTH_PLAN_PATH . 'classes/frontend/class-general.php';
		$this->general = frontend\General::get_instance();

		require_once LSX_HEALTH_PLAN_PATH . 'classes/frontend/class-template-redirects.php';
		$this->template_redirects = frontend\Template_Redirects::get_instance();
	}

	/**
	 * Redirect the user from the cart or checkout page if they have purchased the product already.
	 *
	 * @return void
	 */
	public function redirect() {
		if ( ! is_user_logged_in() || ! function_exists( 'wc_get_page_id' ) || is_home() ) {
			return;
		}
		if ( lsx_health_plan_user_has_purchase() && ( is_page( wc_get_page_id( 'cart' ) ) || is_page( wc_get_page_id( 'checkout' ) ) ) ) {
			wp_redirect( get_permalink( wc_get_page_id( 'myaccount' ) ) );

It seems like get_permalink(wc_get_page_id('myaccount')) can also be of type false; however, parameter $location of wp_redirect() does only seem to accept string, maybe add an additional type check?

			die;

Using exit here is not recommended.

		}

		$product_id = \lsx_health_plan\functions\get_option( 'membership_product', false );
		if ( false !== $product_id && is_single( $product_id ) ) {
			wp_redirect( home_url() );
			die;

Using exit here is not recommended.

		}
	}
}