lsx_health_plan\classes\frontend\Plan_Status

Complexity

Total Complexity

9

Size/Duplication

Total Lines	49
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	2	1
A	get_instance()	0	6	2
A	handle_day_action()	0	13	6

<?php
namespace lsx_health_plan\classes\frontend;

/**
 * Holds the functionality to check and control your status with the current plan.
 *
 * @package lsx-health-plan
 */
class Plan_Status {

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

	/**
	 * Contructor
	 */
	public function __construct() {
		add_action( 'init', array( $this, 'handle_day_action' ), 100 );
	}

	/**
	 * Return an instance of this class.
	 *
	 * @since 1.0.0
	 *
	 * @return    object \lsx_health_plan\classes\frontend\Plan_Status()    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;
	}

	/**
	 * Registers the rewrites.
	 */
	public function handle_day_action() {
		if ( isset( $_POST['lsx-health-plan-actions'] ) && wp_verify_nonce( $_POST['lsx-health-plan-actions'], 'complete' ) ) {
			update_user_meta( get_current_user_id(), 'day_' . sanitize_key( $_POST['lsx-health-plan-id'] ) . '_complete', true );
			$plan_id     = sanitize_key( $_POST['lsx-health-plan-id'] );
			$plan_parent = wp_get_post_parent_id( $plan_id );

$plan_id of type string is incompatible with the type WP_Post|integer expected by parameter $post of wp_get_post_parent_id().

			if ( 0 !== $plan_parent ) {
				$plan_id = $plan_parent;
			}
			wp_safe_redirect( get_permalink( $plan_id ) );

It seems like $plan_id can also be of type false and string; however, parameter $post of get_permalink() does only seem to accept WP_Post|integer, maybe add an additional type check?

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

		}

		if ( isset( $_POST['lsx-health-plan-actions'] ) && wp_verify_nonce( $_POST['lsx-health-plan-actions'], 'unlock' ) ) {
			delete_user_meta( get_current_user_id(), 'day_' . sanitize_key( $_POST['lsx-health-plan-id'] ) . '_complete' );
		}
	}
}