lsx_health_plan\classes\frontend\General

Complexity

Total Complexity

31

Size/Duplication

Total Lines	166
Duplicated Lines	0 %

Importance

Changes	7
Bugs	0	Features	0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	assets()	0	9	3
B	body_classes()	0	31	8
A	allow_html_tags_attributes()	0	16	2
A	get_the_archive_title()	0	14	5
A	get_instance()	0	6	2
A	single_title()	0	13	5
A	__construct()	0	10	1
A	remove_single_footer()	0	3	5

<?php

This file is missing a doc comment.

namespace lsx_health_plan\classes\frontend;

/**
 * Holds the functions and actions which are shared accross the post types.
 *
 * @package lsx-health-plan
 */
class General {

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

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

Expected 2 blank lines before function; 1 found

		// Before Output.
		add_action( 'wp_enqueue_scripts', array( $this, 'assets' ), 5 );
		add_filter( 'wp_kses_allowed_html', array( $this, 'allow_html_tags_attributes' ), 100, 2 );

		// Output.
		add_action( 'body_class', array( $this, 'body_classes' ) );
		add_filter( 'lsx_global_header_title',  array( $this, 'single_title' ), 200, 1 );

Expected 1 space after comma in argument list; 2 found

		add_action( 'wp_head', array( $this, 'remove_single_footer' ), 99 );
		add_filter( 'get_the_archive_title', array( $this, 'get_the_archive_title' ), 9 );
	}

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\frontend\General()    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 2 blank lines after function; 1 found

Expected 1 blank line before closing function brace; 0 found

	/**
	 * Registers the plugin frontend assets
	 *
	 * @return void
	 */
	public function assets() {

Expected 0 blank lines after opening function brace; 1 found

		if ( is_post_type_archive( 'plan' ) && false === \lsx_health_plan\functions\plan\is_filters_disabled() ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

			wp_enqueue_script( 'isotope', LSX_HEALTH_PLAN_URL . 'assets/js/vendor/isotope.pkgd.min.js', array( 'jquery' ), null, LSX_HEALTH_PLAN_URL, true );

The call to wp_enqueue_script() has too many arguments starting with true.

Resource version not set in call to wp_enqueue_script(). This means new versions of the script will not always be loaded due to browser caching.

		}

		wp_enqueue_style( 'lsx-health-plan', LSX_HEALTH_PLAN_URL . 'assets/css/lsx-health-plan.css', array(), LSX_HEALTH_PLAN_VER );
		wp_style_add_data( 'lsx-health-plan', 'rtl', 'replace' );
		wp_enqueue_script( 'lsx-health-plan-scripts', LSX_HEALTH_PLAN_URL . 'assets/js/src/lsx-health-plan-admin.js', array( 'jquery' ) );

In footer ($in_footer) is not set explicitly wp_enqueue_script; It is recommended to load scripts in the footer. Please set this value to true to load it in the footer, or explicitly false if it should be loaded in the header.

Resource version not set in call to wp_enqueue_script(). This means new versions of the script will not always be loaded due to browser caching.

	}

Expected 2 blank lines after function; 1 found

	/**

Doc comment for parameter "$tags" missing

Doc comment for parameter "$context" missing

	 * Adds the iframe and the progress HTML tags to the allowed WordPress list.
	 */
	public function allow_html_tags_attributes( $tags, $context ) {
		if ( 'post' === $context ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

			$tags['iframe'] = array(
				'src'             => true,
				'height'          => true,
				'width'           => true,
				'frameborder'     => true,
				'allowfullscreen' => true,
			);
		}

No blank line found after control structure

		$tags['progress'] = array(
			'id'    => true,
			'value' => true,
			'max'   => true,
		);
		return $tags;
	}

Expected 2 blank lines after function; 1 found

Expected 1 blank line before closing function brace; 0 found

	/**
	 * Add body classes to body.
	 *
	 * @param array $classes

Missing parameter comment

	 * @return void

Function return type is void, but function contains return statement

	 */
	public function body_classes( $classes = array() ) {
		global $post;

		if ( isset( $post->post_content ) && has_shortcode( $post->post_content, 'lsx_health_plan_my_profile_block' ) ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

			$classes[] = 'my-plan-shortcode';
		}

		if ( is_single() && is_singular( 'plan' ) ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

			$args = array(
				'post_parent' => get_the_ID(),
				'post_type'   => 'plan',
			);

			$post_id      = get_the_ID();
			$has_children = get_children( $args );
			$has_parent   = wp_get_post_parent_id( $post_id );

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

			if ( ! empty( $has_children ) ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

				$plan_type_class = 'parent-plan-page';
				if ( 0 !== $has_parent ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

					$plan_type_class = 'parent-sub-plan-page';
				}
			} else {
				$plan_type_class = 'unique-plan-page';
				if ( 0 !== $has_parent ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

					$plan_type_class = 'child-plan-page';
				}
			}

No blank line found after control structure

			$classes[] = $plan_type_class;
		}

No blank line found after control structure

		return $classes;
	}

Expected 2 blank lines after function; 1 found

Expected 1 blank line before closing function brace; 0 found

	/**

Doc comment for parameter "$title" missing

	 * Remove the single recipe and exercise title
	 */
	public function single_title( $title ) {

Expected 0 blank lines after opening function brace; 1 found

		if ( is_single() && is_singular( 'recipe' ) ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

Blank line found at start of control structure

			$title = __( 'Recipe', 'lsx-health-plan' );
		}

		if ( is_single() && is_singular( 'exercise' ) ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

Blank line found at start of control structure

			$title = __( 'Exercise', 'lsx-health-plan' );
		}

		return $title;
	}

Expected 2 blank lines after function; 1 found

Expected 1 blank line before closing function brace; 0 found

	/**
	 * Removing footer for HP single pages.
	 *
	 * @return void
	 */
	public function remove_single_footer() {
		if ( ( is_single() && is_singular( array( 'exercise', 'recipe', 'workout', 'meal' ) ) ) || ( is_archive() && is_post_type_archive( array( 'exercise', 'recipe', 'workout', 'meal' ) ) )) {

Expected 0 spaces after opening bracket; 1 found

No space before closing parenthesis is prohibited

			remove_action( 'lsx_footer_before', 'lsx_add_footer_sidebar_area' );
		}
	}

Expected 2 blank lines after function; 0 found

Expected 1 blank line before closing function brace; 0 found

	/**
	 * Remove the "Archives:" from the post type recipes.
	 *
	 * @param string $title the term title.
	 * @return string
	 */
	public function get_the_archive_title( $title ) {
		if ( is_post_type_archive( 'recipe' ) ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

			$title = __( 'Recipes', 'lsx-health-plan' );
		}

No blank line found after control structure

		if ( is_post_type_archive( 'exercise' ) ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

			$title = __( 'Exercises', 'lsx-health-plan' );
		}

No blank line found after control structure

		if ( is_tax() ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

			$queried_object = get_queried_object();
			if ( isset( $queried_object->name ) ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

				$title = $queried_object->name;
			}
		}

No blank line found after control structure

		return $title;
	}

Expected 2 blank lines after function; 0 found

Expected 1 blank line before closing function brace; 0 found

}