lsx_health_plan\classes\FacetWP

Complexity

Total Complexity

5

Size/Duplication

Total Lines	62
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	load_classes()	0	3	1
A	register_sources()	0	9	1
A	__construct()	0	3	1
A	get_instance()	0	6	2

<?php

This file is missing a doc comment.

namespace lsx_health_plan\classes;

/**
 * Contains the downloads functions post type
 *
 * @package lsx-health-plan
 */
class FacetWP {

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

	/**
	 * Holds the indexer filters for the workouts.
	 *
	 * @var      object \lsx_health_plan\classes\integrations\FacetWP\Workouts_Indexer()
	 */
	public $workouts = null;

	/**
	 * Constructor
	 */
	public function __construct() {

Expected 2 blank lines before function; 1 found

		$this->load_classes();
		add_filter( 'facetwp_facet_sources', array( $this, 'register_sources' ) );
	}

Expected 2 blank lines after function; 1 found

Expected 1 blank line before closing function brace; 0 found

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

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

			self::$instance = new self();
		}

No blank line found after control structure

		return self::$instance;
	}

Expected 1 blank line before closing function brace; 0 found

Expected 2 blank lines after function; 1 found

	/**
	 * Loads the variable classes and the static classes.
	 */
	private function load_classes() {
		require_once LSX_HEALTH_PLAN_PATH . 'classes/integrations/facetwp/class-connected-plans.php';
		$this->connected_plans = integrations\facetwp\Connected_Plans::get_instance();

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

	}

Expected 2 blank lines after function; 1 found

Expected 1 blank line before closing function brace; 0 found

	/**
	 * Registers the custom sources.
	 *
	 * @param array $sources

Missing parameter comment

	 * @return array
	 */
	public function register_sources( $sources ) {
		$sources['lsx_health_plan'] = array(
			'label'   => __( 'LSX Health Plan', 'lsx-health-plan' ),
			'choices' => array(
				'lsx_hp/connected_plans' => 'Connected Plans',
			),
		);

		return $sources;
	}

Expected 2 blank lines after function; 0 found

Expected 1 blank line before closing function brace; 0 found

}