lightspeeddevelopment / lsx-search

classes/class-lsx-search-shortcode.php

<?php
/**
 * LSX Search Shortcode Class.
 *
 * @package lsx-search
 */
class LSX_Search_Shortcode {

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

Class name "LSX_Search_Shortcode" is not in PascalCase format

	/**
	 * Construct method.
	 */
	public function __construct() {

Expected 2 blank lines before function; 1 found

		add_shortcode( 'lsx_search_form', array( $this, 'search_form' ) );
	}

Expected 1 blank line before closing function brace; 0 found

Expected 2 blank lines after function; 1 found

	/**
	 * Outputs the appropriate search form
	 */
	public function search_form( $atts = array() ) {

Method name "LSX_Search_Shortcode::search_form" is not in camel caps format

		$classes = 'search-form lsx-search-form form-inline';

		if ( isset( $atts['class'] ) ) {

Expected 0 spaces before closing bracket; 1 found

			$classes .= $atts['class'];
		}

		$placeholder = __( 'Where do you want to go?', 'lsx-search' );

		if ( isset( $atts['placeholder'] ) ) {

Expected 0 spaces before closing bracket; 1 found

			$placeholder = $atts['placeholder'];
		}

		$action = '/';

		if ( isset( $atts['action'] ) ) {

Expected 0 spaces before closing bracket; 1 found

			$action = $atts['action'];
		}

		$method = 'get';

		if ( isset( $atts['method'] ) ) {

Expected 0 spaces before closing bracket; 1 found

			$method = $atts['method'];
		}

		$button_label = __( 'Search', 'lsx-search' );

		if ( isset( $atts['button_label'] ) ) {

Expected 0 spaces before closing bracket; 1 found

			$button_label = $atts['button_label'];
		}

		$button_class = 'btn cta-btn ';

		if ( isset( $atts['button_class'] ) ) {

Expected 0 spaces before closing bracket; 1 found

			$button_class .= $atts['button_class'];
		}

		$engine = false;

		if ( isset( $atts['engine'] ) ) {

Expected 0 spaces before closing bracket; 1 found

			$engine = $atts['engine'];
		}

		$engine_select = false;

		if ( isset( $atts['engine_select'] ) ) {

Expected 0 spaces before closing bracket; 1 found

			$engine_select = true;
		}

		$display_search_field = true;

		if ( isset( $atts['search_field'] ) ) {

Expected 0 spaces before closing bracket; 1 found

			$display_search_field = (boolean) $atts['search_field'];

Short form type keywords must be used. Found: (boolean)

		}

		$facets = false;

		if ( isset( $atts['facets'] ) ) {

Expected 0 spaces before closing bracket; 1 found

			$facets = $atts['facets'];
		}

		$combo_box = false;

		if ( isset( $atts['combo_box'] ) ) {

Expected 0 spaces before closing bracket; 1 found

			$combo_box = true;
		}

		$return = '';

		ob_start(); ?>

		<?php do_action( 'lsx_search_form_before' ); ?>

		<nav class="navbar navbar-light bg-light">

			<form class="<?php echo esc_attr( $classes ); ?>" action="<?php echo esc_attr( $action ); ?>" method="<?php echo esc_attr( $method ); ?>">

				<?php do_action( 'lsx_search_form_top' ); ?>

				<div class="input-group navbar-nav">
					<?php if ( true === $display_search_field ) : ?>

Expected 0 spaces before closing bracket; 1 found

						<div class="field">
							<input class="search-field form-control" name="s" type="search" placeholder="<?php echo esc_attr( $placeholder ); ?>" autocomplete="off">
						</div>
					<?php endif; ?>

					<?php if ( false !== $engine_select && false !== $engine && 'default' !== $engine ) :

Opening PHP tag must be on a line by itself

Expected 0 spaces before closing bracket; 1 found

						$engines = explode( '|',$engine ); ?>

Closing PHP tag must be on a line by itself

						<div class="field engine-select">
							<div class="dropdown nav-item">
								<?php
								$plural = 's';
								if ( 'accommodation' === $engine[0] ) {

Expected 0 spaces before closing bracket; 1 found

									$plural = '';
								}
								?>
								<button id="engine" data-selection="<?php echo esc_attr( $engines[0] ); ?>" class="btn border-btn btn-dropdown dropdown-toggle" type="button" data-toggle="dropdown" aria-haspopup="true" aria-expanded="false"><?php echo esc_html( ucwords( str_replace( '_', ' ',$engines[0] ) ) . $plural ); ?> <span class="caret"></span></button>
								<ul class="dropdown-menu">
									<?php
									foreach ( $engines as $engine ) {

Expected 0 spaces before closing bracket; 1 found

										$plural = 's';
										if ( 'accommodation' === $engine ) {

Expected 0 spaces before closing bracket; 1 found

											$plural = '';
										}
										echo '<li><a data-value="' . esc_attr( $engine ) . '" href="#">' . esc_html( ucfirst( str_replace( '_', ' ',$engine ) ) . $plural ) . '</a></li>';
									}
									?>
								</ul>
							</div>
						</div>
					<?php endif; ?>

					<?php if ( false !== $facets ) {

Opening PHP tag must be on a line by itself

Expected 0 spaces before closing bracket; 1 found

						$facets = explode( '|',$facets );

						if ( ! is_array( $facets ) ) {

The condition is_array($facets) is always true.

Expected 0 spaces before closing bracket; 1 found

							$facets = array( $facets );
						}

						$field_class = 'field';

						if ( false !== $combo_box ) {

Expected 0 spaces before closing bracket; 1 found

							$this->combo_box( $facets );
							$field_class .= ' combination-toggle hidden';
						}

						foreach ( $facets as $facet ) {

Expected 0 spaces before closing bracket; 1 found

							?>
							<div class="<?php echo wp_kses_post( $field_class ); ?>">
								<?php
								$facet = FWP()->helper->get_facet_by_name( $facet );

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

								if ( isset( $facet['source'] ) ) {

Expected 0 spaces before closing bracket; 1 found

									$values = $this->get_form_facet( $facet['source'] );
								} else {
									$values = array();
								}
								$facet_display_type = apply_filters( 'lsx_search_form_field_type', 'select', $facet );
								$this->display_form_field( $facet_display_type,$facet,$values,$combo_box );
								?>
							</div>
							<?php
						}
					} ?>

Closing PHP tag must be on a line by itself

					<div class="field submit-button">
						<button class="<?php echo esc_attr( $button_class ); ?>" type="submit"><?php echo wp_kses_post( $button_label ); ?></button>
					</div>

					<?php if ( false === $engine_select && false !== $engine && 'default' !== $engine ) : ?>

Expected 0 spaces before closing bracket; 1 found

						<input name="engine" type="hidden" value="<?php echo esc_attr( $engine ); ?>">
					<?php endif; ?>
				</div>

				<?php do_action( 'lsx_search_form_bottom' ); ?>

			</form>

		</nav>

		<?php do_action( 'lsx_search_form_after' ); ?>
		<?php
		$return = ob_get_clean();

		$return = preg_replace( '/[\n]+/', ' ', $return );
		$return = preg_replace( '/[\t]+/', ' ', $return );

		return $return;
	}

Expected 1 blank line before closing function brace; 0 found

Expected 2 blank lines after function; 1 found

	/**
	 * Outputs the combination selector
	 */
	public function combo_box( $facets ) {

Method name "LSX_Search_Shortcode::combo_box" is not in camel caps format

		?>
		<div class="field combination-dropdown">
			<div class="dropdown">
				<button data-selection="0" class="btn border-btn btn-dropdown dropdown-toggle btn-combination" type="button" data-toggle="dropdown" aria-haspopup="true" aria-expanded="true">
					<?php esc_attr_e( 'Select', 'lsx-search' ); ?>
					<span class="caret"></span>
				</button>
				<ul class="dropdown-menu">

					<li style="display: none;"><a class="default" data-value="0" href="#"><?php esc_attr_e( 'Select ', 'lsx-search' ); ?></a></li>

					<?php foreach ( $facets as $facet ) {

Opening PHP tag must be on a line by itself

Expected 0 spaces before closing bracket; 1 found

						$facet = FWP()->helper->get_facet_by_name( $facet );

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

						?>
						<li><a data-value="fwp_<?php echo wp_kses_post( $facet['name'] ); ?>" href="#"><?php echo wp_kses_post( $facet['label'] ); ?></a></li>
					<?php } ?>
				</ul>
			</div>
		</div>
		<?php
	}

Expected 1 blank line before closing function brace; 0 found

Expected 2 blank lines after function; 1 found

	/**
	 * Grabs the Values for the Facet in Question.
	 */
	protected function get_form_facet( $facet_source = false ) {

Method name "LSX_Search_Shortcode::get_form_facet" is not in camel caps format

		global $wpdb;

		$values = array();
		$select = 'f.facet_value, f.facet_display_value';
		$from = "{$wpdb->prefix}facetwp_index f";

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

		$where = "f.facet_source = '{$facet_source}'";

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

		//Check if the current facet is showing destinations.
		if ( stripos( $facet_source, 'destination_to' ) ) {

Expected 0 spaces before closing bracket; 1 found

			$from .= " INNER JOIN {$wpdb->posts} p ON f.facet_value = p.ID";

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

			$where .= " AND p.post_parent = '0'";

Blank line found at end of control structure

		}

		$response = $wpdb->prepare( "SELECT {$select} FROM {$from} WHERE {$where}" );// WPCS: unprepared SQL OK.

Using the WPCS native whitelist comments is deprecated. Please use the PHPCS native "phpcs:ignore Standard.Category.SniffName.ErrorCode" annotations instead. Found: // WPCS: unprepared SQL OK.

		if ( ! empty( $response ) ) {

Expected 0 spaces before closing bracket; 1 found

			foreach ( $response as $re ) {

Expected 0 spaces before closing bracket; 1 found

				$display_value = $re->facet_display_value;
				if ( function_exists( 'pll_translate_string' ) ) {

Expected 0 spaces before closing bracket; 1 found

					$current_lang = pll_current_language();

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

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

					$display_value = pll_translate_string( $display_value, $current_lang );
				}
				$display_value = apply_filters( 'lsx_search_facetwp_display_value', $display_value, $re->facet_value );

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

				$values[ $re->facet_value ] = $display_value;
			}
		}

		asort( $values );
		return $values;
	}

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 main pagination (WP-PageNavi)
	 */
	public function display_form_field( $type = 'select', $facet = array(), $values = array(), $combo = false ) {

Method name "LSX_Search_Shortcode::display_form_field" is not in camel caps format

		if ( empty( $facet ) ) {

Expected 0 spaces before closing bracket; 1 found

			return;
		}

		$source = 'fwp_' . $facet['name'];

		switch ( $type ) {

Blank line found at start of control structure

Expected 0 spaces before closing bracket; 1 found

			case 'select': ?>

The case body in a switch statement must start on the line following the statement.

Closing PHP tag must be on a line by itself

				<div class="dropdown nav-item <?php if ( true === $combo ) { echo 'combination-dropdown'; } ?>">

Expected 0 spaces before closing bracket; 1 found

					<button data-selection="0" class="btn border-btn btn-dropdown dropdown-toggle" type="button" id="<?php echo wp_kses_post( $source ); ?>" data-toggle="dropdown" aria-haspopup="true" aria-expanded="true">
						<?php echo esc_attr( apply_filters( 'lsx_search_facet_label', __( 'Select', 'lsx-search' ) . ' ' . wp_kses_post( $facet['label'] ) ) ); ?>
						<span class="caret"></span>
					</button>
					<ul class="dropdown-menu" aria-labelledby="<?php echo wp_kses_post( $source ); ?>">
						<?php if ( ! empty( $values ) ) { ?>

Blank line found at start of control structure

Expected 0 spaces before closing bracket; 1 found

							<li style="display: none;">
								<a class="default" data-value="0" href="#">
									<?php
										$facet_label = __( 'Select ', 'lsx-search' ) . ' ' . wp_kses_post( $facet['label'] );
										$facet_label = apply_filters( 'lsx_search_facet_label', $facet_label );
										echo esc_attr( $facet_label );
									?>
								</a>
							</li>

							<?php foreach ( $values as $key => $value ) { ?>

Expected 0 spaces before closing bracket; 1 found

								<li><a data-value="<?php echo wp_kses_post( $key ); ?>" href="#"><?php echo wp_kses_post( $value ); ?></a></li>
							<?php } ?>
						<?php } else { ?>
							<li><a data-value="0" href="#"><?php esc_attr_e( 'Please re-index your facets.', 'lsx-search' ); ?></a></li>
						<?php } ?>
					</ul>
				</div>
				<?php
				break;

			case 'datepicker': ?>

The case body in a switch statement must start on the line following the statement.

Closing PHP tag must be on a line by itself

				<div class="datepicker nav-item">
					<input autocomplete="off" class="datepicker-value" placeholder="<?php echo wp_kses_post( apply_filters( 'lsx_search_facet_label' , $facet['label'] ) ); ?>" name="<?php echo wp_kses_post( $source ); ?>"  id="<?php echo wp_kses_post( $source ); ?>" type="text" value="" />

Space found before comma in argument list

				</div>
			<?php
				break;
		}

		?>

	<?php }

Opening PHP tag must be on a line by itself

Expected 1 blank line before closing function brace; 0 found

Expected 2 blank lines after function; 0 found

}