lightspeeddevelopment / lsx-health-plan

templates/single-plan.php

<?php
/**
 * The Template for displaying the single day plan and its connected items
 *
 * @package lsx-health-plan
 */

get_header(); ?>

<?php lsx_content_wrap_before(); ?>

<?php
$args = array(
	'post_parent' => get_the_ID(),
	'post_type'   => 'plan',
);

$plan_id      = get_the_ID();
$has_sections = \lsx_health_plan\functions\plan\has_sections( $plan_id );

It seems like $plan_id can also be of type false; however, parameter $plan_id of lsx_health_plan\functions\plan\has_sections() does only seem to accept integer, maybe add an additional type check?

$restricted   = false;
$is_section   = get_query_var( 'section', false );

// Getting translated endpoint.
$plan = \lsx_health_plan\functions\get_option( 'endpoint_plan', 'plan' );

$connected_members  = get_post_meta( get_the_ID(), ( $plan . '_connected_team_member' ), true );

Are you sure $plan 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?

$connected_articles = get_post_meta( get_the_ID(), ( $plan . '_connected_articles' ), true );
$small_description  = get_post_meta( get_the_ID(), ( $plan . '_short_description' ), true );

if ( ! empty( $has_sections ) && empty( $is_section ) ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

	$plan_type_class = 'parent-plan';
}

No blank line found after control structure

if ( ! empty( $has_sections ) && ! empty( $is_section ) ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

	$plan_type_class = 'child-plan';
}

// Get the plan restrictions.
if ( function_exists( 'wc_memberships_is_post_content_restricted' ) && wc_memberships_is_post_content_restricted( get_the_ID() ) ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

	$restricted = ! current_user_can( 'wc_memberships_view_restricted_post_content', get_the_ID() );
}

No blank line found after control structure

if ( false === $restricted ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

	$round_progress = round( \lsx_health_plan\functions\get_progress( get_the_ID() ), 0 );

It seems like get_the_ID() can also be of type false; however, parameter $plan_id of lsx_health_plan\functions\get_progress() does only seem to accept integer, maybe add an additional type check?

}
?>

<div id="primary" class="content-area <?php echo esc_attr( lsx_main_class() ); ?>">

	<?php lsx_content_before(); ?>

	<main id="main" class="site-main <?php echo esc_html( $plan_type_class ); ?>" role="main">

		<?php lsx_content_top(); ?>

		<div class="post-wrapper">
			<?php if ( ! empty( $has_sections ) && empty( $is_section ) ) { ?>

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

				<div class="plan-content">
					<?php the_content(); ?>
				</div>
			<?php } ?>

			<?php
			if ( ! empty( $has_sections ) ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

				if ( false === $is_section ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

					?>
					<article id="post-<?php the_ID(); ?>" <?php post_class(); ?>>
						<div class="entry-content">
							<div class="single-plan-inner main-plan-content">
								<div class="single-plan-section-title title-lined">
									<?php lsx_get_svg_icon( 'my-plan.svg' ); ?>
									<h2><?php echo esc_html_e( 'Your Plan', 'lsx-health-plan' ); ?></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>
								<div class="plan">
									<div class="set-box set content-box entry-content">
										<div class="plan-top-content">
										<?php
										if ( $connected_members ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

											echo wp_kses_post( lsx_hp_member_connected( $connected_members, $plan ) );

lsx_hp_member_connected($connected_members, $plan) of type void is incompatible with the type string expected by parameter $data of wp_kses_post().

										}

No blank line found after control structure

										if ( false === $restricted ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

											echo wp_kses_post( '<span class="progress"><progress class="bar" value="' . $round_progress . '" max="100"> ' . $round_progress . '% </progress><span>' . $round_progress . '%</span></span>' );
										}

No blank line found after control structure

										if ( $small_description ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

											?>
											<div class="the-content">
												<span><?php echo esc_html( $small_description ); ?></span>
											</div>
											<?php
										}
										?>
										</div>
										<?php
										if ( lsx_health_plan_has_tips() ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

											echo wp_kses_post( do_shortcode( '[lsx_health_plan_featured_tips_block]' ) );
										} ?>

Closing PHP tag must be on a line by itself

									</div>
									<div class="the-plan-content">
										<?php
										echo do_shortcode( '[lsx_health_plan_day_plan_block week_view="true" show_downloads="true" plan="' . get_the_ID() . '"]' );

Are you sure get_the_ID() of type false|integer can be used in concatenation?

										?>
										<div class="row status-plan-buttons main-plan-btn">
											<?php
											if ( function_exists( 'wc_get_page_id' ) ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

												?>
												<a class="btn border-btn" href="<?php echo wp_kses_post( get_permalink( wc_get_page_id( 'myaccount' ) ) ); ?>"><?php esc_html_e( 'My Plans', 'lsx-health-plan' ); ?></a>

It seems like get_permalink(wc_get_page_id('myaccount')) 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?

												<?php
											}
											?>
										</div>
									</div>
								</div>

							</div>
						</div><!-- .entry-content -->
					</article>
					<?php
				} else {
					lsx_health_plan_single_nav();
					lsx_health_plan_single_tabs();
				}
			}
			?>
		</div>

		<?php
		// Show the buttons on the single plan tabs.
		if ( ! empty( $has_sections ) && false !== $is_section ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

			?>
			<div class="row status-plan-buttons">
				<?php lsx_health_plan_day_button(); ?>
			</div>
			<?php
		}
		?>

		<?php lsx_content_bottom(); ?>

		<?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, __( 'Latest articles', 'lsx-health-plan' ) );
		}
		?>


	</main><!-- #main -->

	<?php lsx_content_after(); ?>

</div><!-- #primary -->

<?php lsx_content_wrap_after(); ?>

<?php get_sidebar(); ?>

<?php
get_footer();