lightspeeddevelopment / to-reviews

templates/content-widget-review.php

<?php
/**
 * Reviews Widget Content Part
 *
 * @package 	tour-operators

Spaces must be used for mid-line alignment; tabs are not allowed

 * @category	review

Spaces must be used for mid-line alignment; tabs are not allowed

 * @subpackage	widget

Spaces must be used for mid-line alignment; tabs are not allowed

 */

global $disable_placeholder, $disable_text, $post;

$has_single = ! lsx_to_is_single_disabled();
$permalink = '';

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

if ( $has_single ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

	$permalink = get_the_permalink();
} elseif ( ! is_post_type_archive( 'review' ) ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

	$has_single = true;
	$permalink = get_post_type_archive_link( 'review' ) . '#review-' . $post->post_name;

Are you sure get_post_type_archive_link('review') of type false|string can be used in concatenation?

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

}
?>
<article <?php post_class(); ?>>
	<?php if ( empty( $disable_placeholder ) ) { ?>

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

		<div class="lsx-to-widget-thumb">
			<?php if ( $has_single ) { ?><a href="<?php echo esc_url( $permalink ); ?>"><?php } ?>

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

It seems like $permalink can also be of type false; however, parameter $url of esc_url() does only seem to accept string, maybe add an additional type check?

				<?php lsx_thumbnail( 'lsx-thumbnail-wide' ); ?>
			<?php if ( $has_single ) { ?></a><?php } ?>

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

		</div>
	<?php } ?>

	<h4 class="lsx-to-widget-title text-center">
		<?php if ( $has_single ) { ?><a href="<?php echo esc_url( $permalink ); ?>"><?php } ?>

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

			<?php the_title(); ?>
		<?php if ( $has_single ) { ?></a><?php } ?>

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

	</h4>

	<?php if ( empty( $disable_text ) ) { ?>

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

		<blockquote class="lsx-to-widget-blockquote">
			<?php
			$excerpt = get_the_excerpt();
			if ( empty( $excerpt ) || '' === $excerpt ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

				$tooltip = apply_filters( 'get_the_excerpt', get_the_content() );
				$tooltip = strip_tags( $tooltip );

strip_tags() is discouraged. Use the more comprehensive wp_strip_all_tags() instead.

				echo wp_kses_post( wpautop( $tooltip ) );
			} else {
				echo wp_kses_post( $excerpt );
			}
			?>
		</blockquote>
	<?php } ?>

	<div class="lsx-to-widget-meta-data">
		<?php
			$meta_class = 'lsx-to-meta-data lsx-to-meta-data-';

			lsx_to_connected_accommodation( '<span class="' . $meta_class . 'accommodations"><span class="lsx-to-meta-data-key">' . __( 'Accommodation', 'to-reviews' ) . ':</span> ', '</span>' );
			lsx_to_connected_tours( '<span class="' . $meta_class . 'tours"><span class="lsx-to-meta-data-key">' . __( 'Tours', 'to-reviews' ) . ':</span> ', '</span>' );
			lsx_to_connected_destinations( '<span class="' . $meta_class . 'destinations"><span class="lsx-to-meta-data-key">' . __( 'Destinations', 'to-reviews' ) . ':</span> ', '</span>' );
			if ( function_exists( 'lsx_to_connected_team' ) ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

				lsx_to_connected_team( '<span class="' . $meta_class . 'team"><span class="lsx-to-meta-data-key">' . __( 'Advised by', 'to-reviews' ) . ':</span> ', '</span>' );
			}
		?>
	</div>

</article>