lightspeeddevelopment / lsx-health-plan

templates/content-archive-plan.php

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

There must be exactly one blank line after the file comment

global $shortcode_args, $product;
?>

<?php lsx_entry_before(); ?>

<?php
$content_setting = '';
$column_class    = '4';
$completed_class = '4';
$link_html       = '';
$link_close      = '';
$linked_product  = false;
$restricted      = '';
if ( function_exists( 'lsx_health_plan\functions\woocommerce\plan_has_products' ) && \lsx_health_plan\functions\woocommerce\plan_has_products() ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

	$products       = \lsx_health_plan\functions\woocommerce\get_plan_products();
	$linked_product = wc_get_product( $products[0] );

The function wc_get_product was not found. Maybe you did not declare it correctly or list all dependencies?

	$product        = $linked_product;

	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() );
	}
}

// Check for shortcode overrides.
if ( null !== $shortcode_args ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

	if ( isset( $shortcode_args['columns'] ) ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

		$column_class = $shortcode_args['columns'];
		$column_class = \lsx_health_plan\functions\column_class( $column_class );
	}

No blank line found after control structure

	if ( isset( $shortcode_args['link'] ) ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

		$link_setting = $shortcode_args['link'];
		// Setup our link and content.
		switch ( $link_setting ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

			case 'single':
				$link_html  = '<a href="' . get_permalink( $group['connected_plans'] ) . '">';

Are you sure get_permalink($group['connected_plans']) of type false|string can be used in concatenation?

				$link_close = '</a>';
				break;

			case 'modal':
				$link_html  = '<a data-toggle="modal" href="#workout-plan-modal-' . $group['connected_plans'] . '">';

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

				$link_close = '</a>';

Equals sign not aligned with surrounding assignments; expected 12 spaces but found 1 space

				$modal_content_setting = \lsx_health_plan\functions\get_option( 'workout_tab_modal_content', 'excerpt' );
				$modal_args = array(

Equals sign not aligned with surrounding assignments; expected 12 spaces but found 1 space

					'modal_content' => $modal_content_setting,
				);
				// We call the button to register the modal, but we do not output it.
				lsx_health_plan_workout_plan_button( $group['connected_plans'], $group, false, $modal_args );

The function lsx_health_plan_workout_plan_button was not found. Maybe you did not declare it correctly or list all dependencies?

				break;

			case 'none':
			default:
				$link_html  = '';
				$link_close = '';
				break;
		}
	}

No blank line found after control structure

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

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

		$content_setting = $shortcode_args['description'];
	}
}

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

Equals sign not aligned correctly; expected 1 space but found 6 spaces

?>

<div class="lsx-plan-column col-xs-12 col-sm-6 col-md-<?php echo esc_attr( $column_class ); ?> <?php echo esc_attr( lsx_hp_plan_get_classes() ); ?>">
	<article class="lsx-slot box-shadow">
		<?php lsx_entry_top(); ?>

		<div class="plan-feature-img">
			<?php if ( $featured ) { ?>

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

				<span class="featured-plan"><?php //lsx_get_svg_icon( 'icon-featured.svg' ); ?></span>

58% of this comment could be valid code. Did you maybe forget this after debugging?

No space found before comment text; expected "// lsx_get_svg_icon( 'icon-featured.svg' );" but found "//lsx_get_svg_icon( 'icon-featured.svg' );"

Inline comments must end in full-stops, exclamation marks, or question marks

			<?php } ?>
			<a href="<?php echo esc_url( get_permalink() ); ?>">

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

			<?php
			$featured_image = get_the_post_thumbnail();
			if ( ! empty( $featured_image ) && '' !== $featured_image ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

				the_post_thumbnail( 'lsx-thumbnail', array(

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

For multi-line function calls, each argument should be on a separate line.

					'class' => 'aligncenter',
				) );

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

			} else {
				?>
				<img loading="lazy" class="placeholder" src="<?php echo esc_attr( plugin_dir_url( __FILE__ ) . '../assets/images/placeholder.jpg' ); ?>">
				<?php
			}
			?>
			</a>
		</div>
		<div class="content-box plan-content-box white-bg">
			<h3 class="plan id-<?php the_ID(); ?> <?php echo esc_attr( $completed_class ); ?>"><a href="<?php echo esc_url( get_permalink() ); ?>"><?php the_title(); ?></a></h3>
			<?php
				echo wp_kses_post( \lsx_health_plan\functions\hp_get_plan_type_meta( $post ) );

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

			?>
			<?php
			if ( false !== $linked_product ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

				echo '<div class="plan-price">';
				echo wp_kses_post( $linked_product->get_price_html() );
				echo '</div>';
			}
			?>
			<div class="excerpt">
				<?php
				if ( ! has_excerpt() ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

					$content = wp_trim_words( get_the_content(), 20 );
					$content = '<p>' . $content . '</p>';
				} else {
					$content = apply_filters( 'the_excerpt', get_the_excerpt() );
				}

No blank line found after control structure

				echo wp_kses_post( $content );
				?>
			</div>

			<?php
			$button_link  = get_permalink();
			$button_text  = __( 'Sign Up', 'lsx-health-plan' );
			$button_class = '';

			if ( true === $restricted ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

				if ( 1 < count( $products ) ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

					$button_text = __( 'Select Options', 'lsx-health-plan' );
				} else {
					$button_link = $linked_product->add_to_cart_url() . '&plan_id=' . get_the_ID();

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

					$button_text = $linked_product->add_to_cart_text();
				}
				?>
				<?php
			} elseif ( false === $restricted ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

				$button_text  = __( 'Already Signed Up', 'lsx-health-plan' );
				$button_class = 'btn-disabled';
			}
			?>
			<a class="btn <?php echo esc_attr( $button_class ); ?>" href="<?php echo esc_attr( $button_link ); ?>"><?php echo esc_attr( $button_text ); ?></a>

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

		</div>
		<?php lsx_entry_bottom(); ?>
	</article>
</div>

<?php
lsx_entry_after();