lightspeeddevelopment / lsx-health-plan

templates/partials/workout-sets.php

<?php
/**
 * Template used to loop through the workout sets and display the workouts.
 *
 * @package lsx-health-plan
 */

There must be exactly one blank line after the file comment

global $group_name, $connected_workouts, $shortcode_args;

// Check for any shortcode overrides.
if ( null !== $shortcode_args && isset( $shortcode_args['include'] ) ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

	$connected_workouts = array( get_the_ID() );
}
?>
<div class="sets-wrapper">
	<?php
	if ( empty( $connected_workouts ) || null === $connected_workouts ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

		$connected_workouts = get_post_meta( get_the_ID(), 'connected_workouts', 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 ( empty( $connected_workouts ) ) {

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['connected_workouts'] ) && '' !== $options['connected_workouts'] && ! empty( $options['connected_workouts'] ) ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

				$connected_workouts = $options['connected_workouts'];
				if ( ! array( $connected_workouts ) ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

					$connected_workouts = array( $connected_workouts );
				}
			}
		}
	}

No blank line found after control structure

	$args     = array(
		'orderby'   => 'date',
		'order'     => 'DESC',
		'post_type' => 'workout',
		'post__in'  => $connected_workouts,
	);
	$workouts = new WP_Query( $args );

	if ( $workouts->have_posts() ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

		while ( $workouts->have_posts() ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

			$workouts->the_post();
			// Brings the workout post content first.
			echo wp_kses_post( lsx_health_plan_workout_main_content() );

lsx_health_plan_workout_main_content() of type void is incompatible with the type string expected by parameter $data of wp_kses_post().

			$i               = 1;
			$section_counter = 6;
			echo '<div class="sets">';
			while ( $i <= $section_counter ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

Blank line found at start of control structure

				$workout_section = 'workout_section_' . ( $i ) . '_title';
				$workout_desc    = 'workout_section_' . ( $i ) . '_description';

				$section_title = get_post_meta( get_the_ID(), $workout_section, true );
				$description   = get_post_meta( get_the_ID(), $workout_desc, true );
				if ( is_singular( 'workout' ) ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

					$section_title = get_post_meta( get_queried_object_id(), $workout_section, true );
					$description   = get_post_meta( get_queried_object_id(), $workout_desc, true );
				}

				if ( '' === $section_title ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

					$i++;
					continue;
				}
				?>
				<div class="set-box set content-box">
					<h3 class="set-title"><?php echo esc_html( $section_title ); ?></h3>

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

					<div class="set-content">
						<p><?php echo wp_kses_post( apply_filters( 'the_content', $description ) ); ?></p>

It seems like apply_filters('the_content', $description) can also be of type false; however, parameter $data of wp_kses_post() does only seem to accept string, maybe add an additional type check?

					</div>
					<?php lsx_health_plan_workout_tab_content( $i ); ?>
				</div>
				<?php
				$i++;
			}

No blank line found after control structure

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