lightspeeddevelopment / to-vehicles

templates/content-widget-vehicle.php

<?php
/**
 * Vehicle Widget Content Part
 *
 * @package    lsx-tour-operators
 * @category   vehicle
 * @subpackage widget
 */

There must be exactly one blank line after the file comment

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( 'vehicle' ) ) {

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( 'vehicle' ) . '#vehicle-' . $post->post_name;

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

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

}
?>
<article id="post-<?php the_ID(); ?>" <?php post_class(); ?>>

	<?php lsx_widget_entry_top(); ?>

	<?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-single' ); ?>
			<?php if ( $has_single ) { ?></a><?php } ?>

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

		</div>
	<?php } ?>

	<div class="lsx-to-widget-content">
		<?php lsx_widget_entry_content_top(); ?>

		<h4 class="lsx-to-widget-title text-center">
			<?php if ( false !== $has_single ) { ?>

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

				<a href="<?php echo esc_url( $permalink ); ?>"><?php } ?>
				<?php the_title(); ?>
			<?php if ( false !== $has_single ) { ?>

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

				</a>
			<?php } ?>
		</h4>

		<div class="lsx-to-widget-meta-data">
			<?php
				$meta_class = 'lsx-to-meta-data lsx-to-meta-data-';
			?>
			<?php if ( false !== get_post_meta( get_the_ID(), 'vehicle_type', true ) ) { ?>

It seems like get_the_ID() can also be of type false; however, parameter $post_id of get_post_meta() does only seem to accept integer, maybe add an additional type check?

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

				<div <?php echo 'class="' . esc_html( $meta_class ) . 'type"'; ?>><span class="lsx-to-meta-data-key"><?php esc_html_e( 'Type', 'to-vehicles' ); ?>:</span> <?php echo esc_attr( get_post_meta( get_the_ID(), 'vehicle_type', true ) ); ?></div>

It seems like get_post_meta(get_the_ID(), 'vehicle_type', true) can also be of type false; however, parameter $text of esc_attr() does only seem to accept string, maybe add an additional type check?

			<?php } ?>
			<?php if ( false !== get_post_meta( get_the_ID(), 'seating', true ) ) { ?>

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

				<div <?php echo 'class="' . esc_html( $meta_class ) . 'seating"'; ?>><span class="lsx-to-meta-data-key"><?php esc_html_e( 'Seats', 'to-vehicles' ); ?>:</span> <?php echo esc_attr( get_post_meta( get_the_ID(), 'seating', true ) ); ?></div>
			<?php } ?>
			<?php if ( false !== get_post_meta( get_the_ID(), 'price', true ) ) { ?>

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

				<div <?php echo 'class="' . esc_html( $meta_class ) . 'price"'; ?>><span class="lsx-to-meta-data-key"><?php esc_html_e( 'Price Guide', 'to-vehicles' ); ?>:</span> <?php echo esc_attr( get_post_meta( get_the_ID(), 'price', true ) ); ?></div>
			<?php } ?>
		</div>
		<?php
			ob_start();
			lsx_to_widget_entry_content_top();
			the_excerpt();
			lsx_to_widget_entry_content_bottom();
			$excerpt = ob_get_clean();

			if ( empty( $disable_text ) && ! empty( $excerpt ) ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

				echo wp_kses_post( $excerpt );
			} elseif ( $has_single ) { ?>

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

Closing PHP tag must be on a line by itself

				<p><a href="<?php echo esc_url( $permalink ); ?>" class="moretag"><?php esc_html_e( 'View more', 'tour-operator' ); ?></a></p>
			<?php
		}

Closing brace indented incorrectly; expected 12 spaces, found 8

		?>
		<?php lsx_widget_entry_content_bottom(); ?>
	</div>	
	<?php lsx_widget_entry_bottom(); ?>

</article>
<?php lsx_widget_entry_after(); ?>