lightspeeddevelopment / lsx-search

classes/class-lsx-search-facetwp.php

<?php
/**
 * LSX Search FacetWP filters and actions
 *
 * @package lsx-search
 */
class LSX_Search_FacetWP {

PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions.

Class name "LSX_Search_FacetWP" is not in PascalCase format

	/**
	 * @var object \lsx\search\classes\facetwp\Hierarchy()
	 */
	public $hierarchy;

	/**
	 * @var object \lsx\search\classes\facetwp\Post_Connections()
	 */
	public $post_connections;

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

Expected 2 blank lines before function; 1 found

Expected 0 blank lines after opening function brace; 1 found

		require_once LSX_SEARCH_PATH . '/classes/facetwp/class-hierarchy.php';
		$this->hierarchy = lsx\search\classes\facetwp\Hierarchy::get_instance();

		require_once LSX_SEARCH_PATH . '/classes/facetwp/class-post-connections.php';
		$this->post_connections = lsx\search\classes\facetwp\Post_Connections::get_instance();

		add_filter( 'facetwp_pager_html', array( $this, 'facetwp_pager_html' ), 10, 2 );
		add_filter( 'facetwp_result_count', array( $this, 'facetwp_result_count' ), 10, 2 );
		add_filter( 'facetwp_facet_html', array( $this, 'facetwp_slide_html' ), 10, 2 );
		add_filter( 'facetwp_load_css', array( $this, 'facetwp_load_css' ), 10, 1 );
		add_filter( 'facetwp_index_row', array( $this, 'index_row' ), 10, 2 );
	}

Expected 1 blank line before closing function brace; 0 found

Expected 2 blank lines after function; 1 found

	/**
	 * Change FaceWP pagination HTML to be equal LSX pagination.
	 */
	public function facetwp_pager_html( $output, $params ) {

Method name "LSX_Search_FacetWP::facetwp_pager_html" is not in camel caps format

		$output = '';

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

		$page = (int) $params['page'];

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

		$per_page = (int) $params['per_page'];

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

		$total_pages = (int) $params['total_pages'];

		if ( 1 < $total_pages ) {

Expected 0 spaces before closing bracket; 1 found

			$output .= '<div class="lsx-pagination-wrapper facetwp-custom">';
			$output .= '<div class="lsx-pagination">';
			// $output .= '<span class="pages">Page '. $page .' of '. $total_pages .'</span>';

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

			if ( 1 < $page ) {

Expected 0 spaces before closing bracket; 1 found

				$output .= '<a class="prev page-numbers facetwp-page" rel="prev" data-page="' . ( $page - 1 ) . '">«</a>';
			}

			$temp = false;

			for ( $i = 1; $i <= $total_pages; $i++ ) {

Expected 0 spaces before closing bracket; 1 found

				if ( $i == $page ) {

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

Expected 0 spaces before closing bracket; 1 found

					$output .= '<span class="page-numbers current">' . $i . '</span>';
				} elseif ( ( $page - 2 ) < $i && ( $page + 2 ) > $i ) {

Expected 0 spaces before closing bracket; 1 found

					$output .= '<a class="page-numbers facetwp-page" data-page="' . $i . '">' . $i . '</a>';
				} elseif ( ( $page - 2 ) >= $i && $page > 2 ) {

Expected 0 spaces before closing bracket; 1 found

					if ( ! $temp ) {

Expected 0 spaces before closing bracket; 1 found

						$output .= '<span class="page-numbers dots">...</span>';
						$temp = true;

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

					}
				} elseif ( ( $page + 2 ) <= $i && ( $page + 2 ) <= $total_pages ) {

Expected 0 spaces before closing bracket; 1 found

					$output .= '<span class="page-numbers dots">...</span>';
					break;
				}
			}

			if ( $page < $total_pages ) {

Expected 0 spaces before closing bracket; 1 found

				$output .= '<a class="next page-numbers facetwp-page" rel="next" data-page="' . ( $page + 1 ) . '">»</a>';
			}

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

		return $output;
	}

Expected 1 blank line before closing function brace; 0 found

Expected 2 blank lines after function; 1 found

	/**
	 * Change FaceWP result count HTML.
	 */
	public function facetwp_result_count( $output, $params ) {

Method name "LSX_Search_FacetWP::facetwp_result_count" is not in camel caps format

		$output = $params['total'];
		return $output;
	}

Expected 1 blank line before closing function brace; 0 found

Expected 2 blank lines after function; 1 found

	/**
	 * Change FaceWP slider HTML.
	 */
	public function facetwp_slide_html( $html, $args ) {

Method name "LSX_Search_FacetWP::facetwp_slide_html" is not in camel caps format

		if ( 'slider' === $args['facet']['type'] ) {

Expected 0 spaces before closing bracket; 1 found

			$html = str_replace( 'class="facetwp-slider-reset"', 'class="btn btn-md facetwp-slider-reset"', $html );
		}

		return $html;
	}

Expected 1 blank line before closing function brace; 0 found

Expected 2 blank lines after function; 1 found

	/**
	 * Change FaceWP slider HTML.
	 */
	public function facetwp_counts_html( $html, $args ) {

Method name "LSX_Search_FacetWP::facetwp_counts_html" is not in camel caps format

		if ( 'slider' === $args['facet']['type'] ) {

Expected 0 spaces before closing bracket; 1 found

			$html = str_replace( 'class="facetwp-slider-reset"', 'class="btn btn-md facetwp-slider-reset"', $html );
		}
		return $html;
	}

Expected 1 blank line before closing function brace; 0 found

Expected 2 blank lines after function; 1 found

	/**
	 * Disable FacetWP styles.
	 */
	public function facetwp_load_css( $boolean ) {

Method name "LSX_Search_FacetWP::facetwp_load_css" is not in camel caps format

		$boolean = false;
		return $boolean;
	}

Expected 1 blank line before closing function brace; 0 found

Expected 2 blank lines after function; 1 found

	/**
	 * Get the price including the tax
	 * @param $params
	 * @param $class
	 *
	 * @return mixed
	 */
	public function index_row( $params, $class ) {

Method name "LSX_Search_FacetWP::index_row" is not in camel caps format

		// Custom woo fields
		if ( 0 === strpos( $params['facet_source'], 'woo' ) ) {

Expected 0 spaces before closing bracket; 1 found

			$product = wc_get_product( $params['post_id'] );

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

			// Price
			if ( in_array( $params['facet_source'], array( 'woo/price', 'woo/sale_price', 'woo/regular_price' ) ) ) {

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

Expected 0 spaces before closing bracket; 1 found

				$price = $params['facet_value'];
				if ( $product->is_taxable() ) {

Expected 0 spaces before closing bracket; 1 found

					$price = wc_get_price_including_tax( $product );

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

				}
				$params['facet_value']    = $price;

Equals sign not aligned with surrounding assignments; expected 9 spaces but found 4 spaces

				$params['facet_display_value'] = $price;

Blank line found at end of control structure

			}
		}
		return $params;
	}

Expected 1 blank line before closing function brace; 0 found

Expected 2 blank lines after function; 1 found

}