lightspeeddevelopment / lsx-health-plan

templates/partials/workout-table.php

<?php
/**
 * Template used to display the workout content in table form
 *
 * @package lsx-health-plan
 */

global $group_name;
$groups = get_post_meta( get_the_ID(), $group_name, 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 ( is_singular( 'workout' ) ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

	$groups = get_post_meta( get_queried_object_id(), $group_name, true );
}

No blank line found after control structure

if ( ! empty( $groups ) ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

	?>
	<div class="set-table">
		<table class="workout-table">
			<?php
			$table_headers = array();
			$table_body    = array();

			foreach ( $groups as $group ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

				$this_row = array();

				$this_row[] = '<tr>';

				// Getting the connected exercise.
				if ( post_type_exists( 'exercise' ) ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

					if ( isset( $group['connected_exercises'] ) && '' !== $group['connected_exercises'] && ! empty( \lsx_health_plan\functions\check_posts_exist( array( $group['connected_exercises'] ) ) ) ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

						$exercise    = $group['connected_exercises'];
						$exercise_id = get_post( $exercise );
					}
				} else {
					$exercise    = false;
					$exercise_id = false;
				}

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

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

					$exercise_name         = get_the_title( $exercise_id );
					$this_row[]            = '<td class="workout-title-item">' . esc_html( $exercise_name ) . '</td>';
					$table_headers['name'] = true;
				} else {
					if ( isset( $group['name'] ) && '' !== $group['name'] ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

						$this_row[]            = '<td class="workout-title-item">' . esc_html( $group['name'] ) . '</td>';
						$table_headers['name'] = true;
					}
				}

No blank line found after control structure

				// Only display this is exercise is disabled.
				if ( false === $exercise && isset( $group['description'] ) && '' !== $group['description'] ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

					$this_row[]                   = '<td class="workout-desc-item"><p>' . esc_html( $group['description'] ) . '</td>';
					$table_headers['description'] = true;
				}

				if ( isset( $group['reps'] ) && '' !== $group['reps'] ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

					$this_row[]            = '<td class="reps-field-item center-mobile">' . esc_html( $group['reps'] ) . '</td>';
					$table_headers['reps'] = true;
				}

				// Only display this is exercise is disabled.
				if ( false === $exercise ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

					if ( isset( $group['equipment'] ) && '' !== $group['equipment'] ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

						$this_row[]                 = '<td class="equipment-field-item center-mobile">' . esc_html( $group['equipment'] ) . '</td>';
						$table_headers['equipment'] = true;
					}

No blank line found after control structure

					if ( isset( $group['muscle'] ) && '' !== $group['muscle'] ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

						$this_row[]              = '<td class="muscle-field-item center-mobile">' . esc_html( $group['muscle'] ) . '</td>';
						$table_headers['muscle'] = true;
					}
				}

No blank line found after control structure

				if ( post_type_exists( 'video' ) && isset( $group['connected_videos'] ) && '' !== $group['connected_videos'] && ! empty( \lsx_health_plan\functions\check_posts_exist( array( $group['connected_videos'] ) ) ) ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

					$this_row[]             = '<td class="video-button-item center-mobile">' . lsx_health_plan_workout_video_play_button( $m, $group, false ) . '</td>';

Are you sure lsx_health_plan_workout_video_play_button($m, $group, false) of type void can be used in concatenation?

					$table_headers['video'] = true;
				}

No blank line found after control structure

				if ( post_type_exists( 'exercise' ) && isset( $group['connected_exercises'] ) && '' !== $group['connected_exercises'] && ! empty( \lsx_health_plan\functions\check_posts_exist( array( $group['connected_exercises'] ) ) ) ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

					$this_row[]             = '<td class="video-button-item center-mobile">' . lsx_health_plan_workout_exercise_button( $m, $group, false ) . '</td>';

Are you sure lsx_health_plan_workout_exercise_button($m, $group, false) of type void can be used in concatenation?

Equals sign not aligned with surrounding assignments; expected 16 spaces but found 13 spaces

					$table_headers['exercise'] = true;
				}

No blank line found after control structure

				$this_row[] = '</tr>';

				$table_body[] = implode( '', $this_row );
				$m++;
			}

			// Now we build the table header.
			$table_header   = array();
			$table_header[] = '<tr>';
			if ( isset( $table_headers['name'] ) ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

				$table_header[] = '<th class="center-mobile">' . __( 'Workout', 'lsx-health-plan' ) . '</th>';
			}

No blank line found after control structure

			if ( isset( $table_headers['description'] ) ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

				$table_header[] = '<th class="center-mobile">' . __( 'Description', 'lsx-health-plan' ) . '</th>';
			}

No blank line found after control structure

			if ( isset( $table_headers['reps'] ) ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

				$table_header[] = '<th class="center-mobile">' . __( 'Reps / Time / Distance', 'lsx-health-plan' ) . '</th>';
			}

No blank line found after control structure

			if ( isset( $table_headers['equipment'] ) ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

				$table_header[] = '<th class="center-mobile">' . __( 'Equipment', 'lsx-health-plan' ) . '</th>';
			}

No blank line found after control structure

			if ( isset( $table_headers['muscle'] ) ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

				$table_header[] = '<th class="center-mobile">' . __( 'Muscle', 'lsx-health-plan' ) . '</th>';
			}

No blank line found after control structure

			if ( isset( $table_headers['video'] ) ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

				$table_header[] = '<th class="center-mobile">' . __( 'How To', 'lsx-health-plan' ) . '</th>';
			}

No blank line found after control structure

			if ( isset( $table_headers['exercise'] ) ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

				$table_header[] = '<th class="center-mobile">' . __( 'How To', 'lsx-health-plan' ) . '</th>';
			}

No blank line found after control structure

			$table_header[] = '</tr>';
			?>
			<thead>
				<?php echo wp_kses_post( implode( '', $table_header ) ); ?>
			</thead>
			<tbody>
				<?php echo wp_kses_post( implode( '', $table_body ) ); ?>
			</tbody>
		</table>
	</div>
	<?php
}