lightspeeddevelopment / lsx-health-plan

templates/tab-content-warm-up.php

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

There must be exactly one blank line after the file comment

global $group_name, $connected_workouts, $shortcode_args;

$warm_up = get_post_meta( get_the_ID(), 'plan_warmup', 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?

if ( false === $warm_up || '' === $warm_up ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

	$options = \lsx_health_plan\functions\get_option( 'all' );
	if ( isset( $options['plan_warmup'] ) && '' !== $options['plan_warmup'] && ! empty( $options['plan_warmup'] ) ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

		$warm_up = $options['plan_warmup'];
	}
}

?>
<div class="warmup-container">
<?php

if ( false !== $warm_up && '' !== $warm_up ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

	if ( ! is_array( $warm_up ) ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

		$warm_up = array( $warm_up );
	}

	$warmup_type  = array( 'page', 'workout', 'exercise' );
	$warmup_query = new WP_Query(
		array(
			'post__in'  => $warm_up,
			'post_type' => $warmup_type,
		)
	);

	if ( $warmup_query->have_posts() ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

		while ( $warmup_query->have_posts() ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

			$warmup_query->the_post();
			lsx_entry_before();
			if ( 'workout' === get_post_type() ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

				$connected_workouts = array( get_the_ID() );
				?>
				<article id="post-<?php the_ID(); ?>" <?php post_class(); ?>>
					<?php lsx_entry_top(); ?>
					<div class="entry-content">
						<div class="single-plan-inner warmup-content">
							<div class="single-plan-section-title warmup-plan title-lined">
								<?php lsx_get_svg_icon( 'warm.svg' ); ?>
								<h2><?php esc_html_e( 'Warm Up', 'lsx-health-plan' ); ?></h2>
							</div>
						</div>
					</div><!-- .entry-content -->
					<?php lsx_entry_bottom(); ?>
				</article><!-- #post-## -->
				<?php
				lsx_health_plan_warmup_sets( $connected_workouts );
			} else {
				?>
				<article id="post-<?php the_ID(); ?>" <?php post_class(); ?>>
					<?php lsx_entry_top(); ?>
					<div class="entry-content">
						<div class="single-plan-inner warmup-content">
							<div class="single-plan-section-title warmup-plan title-lined">
								<?php lsx_get_svg_icon( 'warm.svg' ); ?>
								<h2><?php esc_html_e( 'Warm Up', 'lsx-health-plan' ); ?></h2>
							</div>
						<?php
							the_content();
							wp_link_pages( array(

The opening parenthesis of a multi-line function call should be the last content on the line.

								'before'      => '<div class="lsx-postnav-wrapper"><div class="lsx-postnav">',
								'after'       => '</div></div>',
								'link_before' => '<span>',
								'link_after'  => '</span>',
							) );

For multi-line function calls, the closing parenthesis should be on a new line.

						?>
						</div>
					</div><!-- .entry-content -->
					<?php lsx_entry_bottom(); ?>
				</article><!-- #post-## -->
				<?php
			}

Blank line found after control structure

Blank line found at end of control structure

		}

No blank line found after control structure

		wp_reset_postdata();
	}
}
?>
</div>
<?php