LSX_Optimisation

Complexity

Total Complexity

12

Size/Duplication

Total Lines	63
Duplicated Lines	0 %

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	get_instance()	0	6	2
A	preload_css()	0	5	3
A	pum_remove_admin_bar_tools()	0	4	1
A	defer_parsing_of_js()	0	6	5
A	__construct()	0	4	1

<?php

This file is missing a doc comment.

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

/**
 * Holds the speed optimization functions for LSX.
 *
 * @author   LightSpeed
 * @category Widgets
 * @package  LSX
 * @return   LSX_Optimisation
 */
class LSX_Optimisation {

	/**
	 * Holds class instance
	 *
	 * @since 1.0.0
	 * @var      object
	 */
	protected static $instance = null;

	/**
	 * Constructor.
	 */
	public function __construct() {
		//add_filter( 'style_loader_tag', array( $this, 'preload_css' ), 100, 4 );

No space found before comment text; expected "// add_filter( 'style_loader_tag', array( $this, 'preload_css' ), 100, 4 );" but found "//add_filter( 'style_loader_tag', array( $this, 'preload_css' ), 100, 4 );"

61% of this comment could be valid code. Did you maybe forget this after debugging?

		//add_filter( 'script_loader_tag', array( $this, 'defer_parsing_of_js' ), 100, 3 );

No space found before comment text; expected "// add_filter( 'script_loader_tag', array( $this, 'defer_parsing_of_js' ), 100, 3 );" but found "//add_filter( 'script_loader_tag', array( $this, 'defer_parsing_of_js' ), 100, 3 );"

Inline comments must end in full-stops, exclamation marks, or question marks

		add_action( 'init', array( $this, 'pum_remove_admin_bar_tools' ), 100 );
	}
	/**
	 * Return an instance of this class.
	 *
	 * @since 1.0.0
	 * @return    object    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;

Parenthesis should always be used when instantiating a new object.

		}
		return self::$instance;
	}

	/**

Doc comment for parameter "$tag" missing

Doc comment for parameter "$handle" missing

Doc comment for parameter "$href" missing

Doc comment for parameter "$media" missing

	 * Defers the JS loading till Last
	 *
	 * @param  string $url The url to check and defer.

Doc comment for parameter $url does not match actual variable name $tag

	 * @return string
	 */
	public function preload_css( $tag, $handle, $href, $media ) {

The parameter $media is not used and could be removed.

The parameter $href is not used and could be removed.

		if ( 'lsx_fonts' === $handle || 'fontawesome' === $handle ) {
			$tag = str_replace( 'href', ' preload href', $tag );
		}
		return $tag;
	}

	/**

Doc comment for parameter "$tag" missing

Doc comment for parameter "$handle" missing

Doc comment for parameter "$href" missing

	 * Defers the JS loading till Last
	 *
	 * @param  string $url The url to check and defer.

Doc comment for parameter $url does not match actual variable name $tag

	 * @return string
	 */
	public function defer_parsing_of_js( $tag, $handle, $href ) {
		$skip_defer = apply_filters( 'lsx_defer_parsing_of_js', false, $tag, $handle, $href );
		if ( ! is_admin() && false !== stripos( $href, '.js' ) && false === stripos( $href, 'jquery.js' ) && false === $skip_defer ) {
			$tag = str_replace( 'src=', ' defer src=', $tag );
		}
		return $tag;
	}

	public function pum_remove_admin_bar_tools() {

Missing doc comment for function pum_remove_admin_bar_tools()

		remove_action( 'admin_bar_menu', array( 'PUM_Modules_Admin_Bar', 'toolbar_links' ), 999 );
		remove_action( 'wp_footer', array( 'PUM_Modules_Admin_Bar', 'admin_bar_styles' ), 999 );
		remove_action( 'init', array( 'PUM_Modules_Admin_Bar', 'show_debug_bar' ) );
	}
}
LSX_Optimisation::get_instance();