LSX\Classes\Frontend

Complexity

Total Complexity

14

Size/Duplication

Total Lines	98
Duplicated Lines	0 %

Importance

Changes	12
Bugs	1	Features	0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	1	1
A	init()	0	10	1
A	woocommerce_account_menu_items_fix()	0	12	4
A	cf_async_disable()	0	5	2
A	enqueue_styles()	0	2	1
A	woo_asset_files()	0	3	2
A	wpforms_match_button_block()	0	3	1
A	yoast_faq_asset_files()	0	4	2

<?php

This file is missing a doc comment.

namespace LSX\Classes;

/**
 * All the functions for setting up the theme.
 *
 * @package   LSX
 * @author    LightSpeed
 * @license   GPL3
 * @link
 * @copyright 2023 LightSpeed
 */
class Frontend {

	/**
	 * Contructor
	 */
	public function __construct() {
	}

	/**
	 * Initiate our class.
	 *
	 * @return void
	 */
	public function init() {
		// Styles and Scripts

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

		add_action( 'wp_enqueue_scripts', array( $this, 'enqueue_styles' ) );
		add_action( 'wp_enqueue_scripts', array( $this, 'yoast_faq_asset_files' ) );
		add_action( 'wp_enqueue_scripts', array( $this, 'woo_asset_files' ) );
		add_filter( 'script_loader_tag', array( $this, 'cf_async_disable' ), 10, 3);

Expected 1 spaces before closing parenthesis; 0 found

		//Output on the frontend.

No space found before comment text; expected "// Output on the frontend." but found "//Output on the frontend."

		add_filter( 'wpforms_frontend_form_data', array( $this, 'wpforms_match_button_block' ) );
		add_filter( 'woocommerce_account_menu_items', array( $this, 'woocommerce_account_menu_items_fix' ), 10, 2 );
		
	}

	/**
	 * Enqueues the frontend styles
	 *
	 * @return void
	 */
	public function enqueue_styles() {
		wp_enqueue_style( 'lsx-styles', get_template_directory_uri() . '/style.css', array(), wp_get_theme()->get( 'Version' ) );

It seems like wp_get_theme()->get('Version') can also be of type array; however, parameter $ver of wp_enqueue_style() does only seem to accept boolean|null|string, maybe add an additional type check?

	}


	/**
	 * Load the assets files for Yoast
	 *
	 * @return void
	 */
	public function yoast_faq_asset_files() {
		if ( function_exists( 'wpseo_init' ) ) {
			wp_enqueue_style( 'lsx-yoast-faq-css', get_template_directory_uri() . '/assets/css/faq/style.css', array() );

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

			wp_enqueue_script( 'lsx-yoast-faq-js', get_template_directory_uri()  . '/assets/js/faq.js', array( "jquery" ), "1.0", true );

Concat operator must be surrounded by a single space

The string literal jquery does not require double quotes, as per coding-style, please use single quotes.

The string literal 1.0 does not require double quotes, as per coding-style, please use single quotes.

		}
	}

	/**
	 * Load the assets files for Yoast
	 *
	 * @return void
	 */
	public function woo_asset_files() {
		if ( class_exists( 'woocommerce' ) ) {
			wp_enqueue_style( 'lsx-woo-css', get_template_directory_uri() . '/assets/css/woocommerce.css', array() );

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

		}
	}

	/**
	 * WPForms submit button, match Gutenberg button block
	 *
	 * @param [type] $form_data

The doc comment [type] at position 0 could not be parsed: Unknown type name '[' at position 0 in [type].

Missing parameter comment

	 * @return void

Function return type is void, but function contains return statement

	 */
	function wpforms_match_button_block( $form_data ) {

It is generally recommended to explicitly declare the visibility for methods.

It is recommend to declare an explicit visibility for wpforms_match_button_block.

		$form_data['settings']['submit_class'] .= ' btn';
		return $form_data;
	}

	/**

Doc comment for parameter "$items" missing

Doc comment for parameter "$endpoints" missing

	 * Fixes the plural for the edit address my account menu.
	 *
	 * @return array
	 */
	public function woocommerce_account_menu_items_fix( $items, $endpoints ) {

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

		if ( ! isset( $items['edit-address'] ) || '' !== $items['edit-address'] ) {
			return $items;
		}

		if ( true === wc_shipping_enabled() ) {

The function wc_shipping_enabled was not found. Maybe you did not declare it correctly or list all dependencies?

			$items['edit-address'] = __( 'Addresses', 'woocommerce' );
		} else {
			$items['edit-address'] = __( 'Address', 'woocommerce' );
		}

		return $items;
	}

	/**

Doc comment for parameter "$tag" missing

Doc comment for parameter "$handle" missing

Doc comment for parameter "$src" missing

	 * Make comment reply button work with CloudFlare Rocket Loader
	 * @see https://support.cloudflare.com/hc/en-us/articles/200169436-How-can-I-have-Rocket-Loader-ignore-specific-JavaScripts-

There must be exactly one blank line before the tags in a doc comment

	 */
	public function cf_async_disable( $tag, $handle, $src ) {

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

		if ( 'comment-reply' !== $handle ) {
			return $tag;
		}
		return str_replace( ' src', ' data-cfasync="false" src', $tag );
	}
}