lightspeeddevelopment / lsx-health-plan

templates/tab-content-workout.php

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

global $shortcode_args;

// Getting translated endpoint.
$archive_workout = \lsx_health_plan\functions\get_option( 'endpoint_workout_archive', 'workout' );
$workout         = \lsx_health_plan\functions\get_option( 'endpoint_workout', 'workout' );

$connected_articles = get_post_meta( get_the_ID(), ( $workout . '_connected_articles' ), true );

Are you sure $workout of type array|mixed can be used in concatenation?

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?

?>

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

	<?php lsx_entry_top(); ?>

	<div class="entry-meta">
		<?php lsx_post_meta_single_bottom(); ?>
	</div><!-- .entry-meta -->
	<?php
	if ( is_singular( 'workout' ) ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

		the_content();
	}
	?>
	<div class="entry-content">
		<div class="single-plan-inner workout-content">
			<?php
			if ( is_singular( 'workout' ) ) { ?>

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

				<div class="single-plan-section-title workout title-lined">
					<?php lsx_get_svg_icon( 'work.svg' ); ?>
					<h2><?php the_title(); ?></h2>
					<?php if ( class_exists( 'LSX_Sharing' ) ) {

Opening PHP tag must be on a line by itself

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

						lsx_content_sharing();
					} ?>

Closing PHP tag must be on a line by itself

				</div>
			<?php } else { ?>
				<div class="single-plan-section-title workout title-lined">
					<?php lsx_get_svg_icon( 'work.svg' ); ?>
					<h2><?php esc_html_e( 'My Workout', 'lsx-health-plan' ); ?></h2>
				</div>
			<?php } ?>
			<?php lsx_health_plan_workout_box(); ?>
			<?php lsx_health_plan_workout_sets(); ?>
		</div>
	</div><!-- .entry-content -->

	<?php lsx_entry_bottom(); ?>

</article><!-- #post-## -->
<?php if ( is_singular( $workout ) ) { ?>

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

	<div  class="back-plan-btn">
		<a class="btn" href="/<?php echo $archive_workout; ?>"><?php esc_html_e( 'Back to workouts', 'lsx-health-plan' ); ?></a>

Are you sure $archive_workout of type array|mixed can be used in echo?

All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$archive_workout'.

	</div>
<?php } ?>
<?php
if ( ! empty( $connected_articles ) ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

	lsx_hp_single_related( $connected_articles, __( 'Related articles', 'lsx-health-plan' ) );
}