lightspeeddevelopment / lsx-health-plan

templates/partials/content-shortcode-exercise.php

<?php
/**
 * Template used to display post content on widgets and archive pages.
 *
 * @package lsx-health-plan
 */

There must be exactly one blank line after the file comment

global $shortcode_args;

$column_class = '4';
// Check for shortcode overrides.
if ( null !== $shortcode_args ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

	if ( isset( $shortcode_args['columns'] ) ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

		$column_class = $shortcode_args['columns'];
		$column_class = \lsx_health_plan\functions\column_class( $column_class );
	}
}

lsx_entry_before();

$post_id        = get_the_id();
$featured_image = get_the_post_thumbnail_url( $post_id, $image_size );

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

?>
<div class="col-xs-12 col-sm-6 col-md-<?php echo esc_attr( $column_class ); ?>">
	<div style="background-image:url('<?php echo esc_url( $featured_image ); ?>')" class="lsx-exercises-item bg-<?php echo esc_html( $image_size ); ?>">

It seems like $featured_image 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
		if ( isset( $link ) && ( 'modal' === $link ) ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

			echo wp_kses_post( lsx_health_plan_shortcode_exercise_button( $post_id, true ) );

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

lsx_health_plan_shortcode_exercise_button($post_id, true) of type void is incompatible with the type string expected by parameter $data of wp_kses_post().

		}

		if ( 'item' === $link ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

			?>
			<a href="<?php echo esc_url( get_permalink() ); ?>" class="exercise-link excerpt-<?php echo esc_html( $description ); ?>">
			<?php
		} else {
			?>
			<div class="exercise-link excerpt-<?php echo esc_html( $description ); ?>">
			<?php
		}
		?>

		<?php lsx_health_plan_exercise_title( '<h4 class="lsx-exercises-title">', '</h4>', false ); ?>
		<?php if ( isset( $description ) && ( 'none' !== $description ) ) { ?>

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

			<?php
			if ( 'excerpt' === $description ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

				$excerpt = \lsx_health_plan\functions\hp_excerpt( $post_id );
				?>
				<p class="lsx-exercises-excerpt"><?php echo wp_kses_post( $excerpt ); ?></p>
			<?php } ?>
			<?php if ( 'full' === $description ) { ?>

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

				<?php echo wp_kses_post( get_the_content() ); ?>
			<?php } ?>
		<?php } ?>
		<?php
		if ( isset( $link ) && ( 'item' === $link ) ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

		?>
			</a>
		<?php } else { ?>
			</div>
		<?php } ?>
	</div>
</div>
<?php
lsx_entry_after();