lightspeeddevelopment / lsx-search

classes/facetwp/class-hierarchy.php

<?php
/**
 * LSX_Search_FacetWP_Hierarchy Frontend Main Class
 */

namespace lsx\search\classes\facetwp;

class Hierarchy {

	/**
	 * Holds class instance
	 *
	 * @since 1.0.0
	 *
	 * @var      object \lsx\search\classes\facetwp\Hierarchy()
	 */
	protected static $instance = null;

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

Expected 2 blank lines before function; 1 found

		add_filter( 'facetwp_facet_html', array( $this, 'checkbox_facet_html' ), 100, 2 );
	}

Expected 1 blank line before closing function brace; 0 found

Expected 2 blank lines after function; 1 found

	/**
	 * Return an instance of this class.
	 *
	 * @since 1.0.0
	 *
	 * @return    object \lsx\search\classes\facetwp\Hierarchy()    A single instance of this class.
	 */
	public static function get_instance() {

Method name "Hierarchy::get_instance" is not in camel caps format

		// If the single instance hasn't been set, set it now.
		if ( null === self::$instance ) {

Expected 0 spaces before closing bracket; 1 found

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

Expected 1 blank line before closing function brace; 0 found

Expected 2 blank lines after function; 1 found

	public function checkbox_facet_html( $output, $params ) {

Method name "Hierarchy::checkbox_facet_html" is not in camel caps format

		if ( 'checkboxes' === $params['facet']['type'] && 'yes' === $params['facet']['hierarchical'] ) {

Expected 0 spaces before closing bracket; 1 found

			$output = $this->render_hierarchy( $params );
		}
		return $output;
	}

Expected 1 blank line before closing function brace; 0 found

Expected 2 blank lines after function; 1 found

	/**
	 * Generate the facet HTML (hierarchical taxonomies)
	 */
	function render_hierarchy( $params ) {

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

It is recommend to declare an explicit visibility for render_hierarchy.

Expected 0 blank lines after opening function brace; 1 found

Method name "Hierarchy::render_hierarchy" is not in camel caps format

		$output = '';

Equals sign not aligned with surrounding assignments; expected 10 spaces but found 1 space

		$facet = $params['facet'];

Equals sign not aligned with surrounding assignments; expected 11 spaces but found 1 space

		$selected_values = (array) $params['selected_values'];
		$values = FWP()->helper->sort_taxonomy_values( $params['values'], $facet['orderby'] );

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

Equals sign not aligned with surrounding assignments; expected 10 spaces but found 1 space

		$init_depth = -1;
		$last_depth = -1;

		foreach ( $values as $result ) {

Expected 0 spaces before closing bracket; 1 found

			$depth = (int) $result['depth'];

			/*if ( -1 == $last_depth ) {

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

				$init_depth = $depth;
			}
			elseif ( $depth > $last_depth ) {
				$output .= '<div class="facetwp-depth">';
			}
			elseif ( $depth < $last_depth ) {
				for ( $i = $last_depth; $i > $depth; $i-- ) {
					$output .= '</div>';
				}
			}*/

			$selected = in_array( $result['facet_value'], $selected_values ) ? ' checked' : '';

Equals sign not aligned with surrounding assignments; expected 2 spaces but found 1 space

Not using strict comparison for in_array; supply true for third argument.

			$selected .= ( 0 == $result['counter'] && '' == $selected ) ? ' disabled' : '';

Found: ==. Use strict comparisons (=== or !==).

			$is_child = ( 0 == $result['parent_id'] && '0' == $result['parent_id'] ) ? ' is-child' : '';

Equals sign not aligned with surrounding assignments; expected 2 spaces but found 1 space

Found: ==. Use strict comparisons (=== or !==).

			$depth_css = ' depth-' . $result['depth'];

			$output .= '<div class="facetwp-checkbox' . $selected . $is_child . $depth_css . '" data-parent-id="' . esc_attr( $result['parent_id'] ) . '" data-value="' . esc_attr( $result['facet_value'] ) . '">';
			$output .= esc_html( $result['facet_display_value'] ) . ' <span class="facetwp-counter">(' . $result['counter'] . ')</span>';
			$output .= '</div>';

			$last_depth = $depth;
		}

		for ( $i = $last_depth; $i > $init_depth; $i-- ) {

Expected 0 spaces before closing bracket; 1 found

			$output .= '</div>';
		}

		return $output;
	}

Expected 1 blank line before closing function brace; 0 found

Expected 2 blank lines after function; 0 found

}