lightspeeddevelopment / lsx

partials/content-related.php

<?php
/**
 * Template used to display related post content.
 *
 * @package lsx
 */

?>

<?php lsx_entry_before(); ?>

<?php
	$no_thumb_post_types   = array(
		'audio'   => 'audio',
		'gallery' => 'gallery',
		'image'   => 'image',
		'link'    => 'link',
		'quote'   => 'quote',
		'video'   => 'video',
	);
	$no_thumb_post_formats = apply_filters( 'lsx_no_thumb_post_formats', $no_thumb_post_types );

	$has_thumb = has_post_thumbnail() && ! has_post_format( $no_thumb_post_formats );

	if ( $has_thumb ) {
		$thumb_class = 'has-thumb';
	} else {
		$thumb_class = 'no-thumb';
	}

	$blog_layout = apply_filters( 'lsx_blog_layout', 'default' );

	$image_class = '';

	$thumbnail_id = get_post_thumbnail_id( get_the_ID() );

It seems like get_the_ID() can also be of type false; however, parameter $post of get_post_thumbnail_id() does only seem to accept WP_Post|integer, maybe add an additional type check?

	$image_arr    = wp_get_attachment_image_src( $thumbnail_id, 'lsx-thumbnail-single' );

	if ( is_array( $image_arr ) ) {
		$image_src = $image_arr[0];
	}
?>

<article id="post-<?php the_ID(); ?>" <?php post_class( array( 'lsx-slot', $thumb_class ) ); ?>>
	<?php lsx_entry_top(); ?>

	<div class="entry-layout">
		<div class="entry-layout-content entry-layout-content-<?php echo has_post_thumbnail() ? '67' : '100'; ?>">
			<header class="entry-header">
				<?php if ( $has_thumb ) : ?>
					<div class="entry-image <?php echo esc_attr( $image_class ); ?>">
						<a class="thumbnail" href="<?php the_permalink(); ?>">
							<?php lsx_thumbnail( 'lsx-thumbnail-single' ); ?>
						</a>
					</div>
				<?php endif; ?>

				<?php
				$format = get_post_format();

				if ( false === $format ) {
					$format        = 'standard';
					$show_on_front = get_option( 'show_on_front', 'posts' );

					if ( 'page' === $show_on_front ) {
						$archive_link = get_permalink( get_option( 'page_for_posts' ) );
					} else {
						$archive_link = home_url();
					}
				} else {
					$archive_link = get_post_format_link( $format );
				}

				$format = lsx_translate_format_to_fontawesome( $format );
				?>

				<h2 class="entry-title">
					<?php if ( has_post_thumbnail() ) : ?>
						<a href="<?php echo esc_url( $archive_link ); ?>" class="format-link has-thumb fa fa-<?php echo esc_attr( $format ); ?>"></a>
					<?php else : ?>
						<a href="<?php echo esc_url( $archive_link ); ?>" class="format-link fa fa-<?php echo esc_attr( $format ); ?>"></a>
					<?php endif; ?>

					<?php if ( has_post_format( array( 'link' ) ) ) : ?>
						<a href="<?php echo esc_url( lsx_get_my_url() ); ?>" rel="bookmark"><?php the_title(); ?> <span class="fa fa-external-link"></span></a>

It seems like lsx_get_my_url() 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 else : ?>
						<a href="<?php the_permalink(); ?>" rel="bookmark"><?php the_title(); ?></a>
					<?php endif; ?>

					<?php if ( is_sticky() ) : ?>
						<span class="label label-default label-sticky"><?php esc_html_e( 'Featured', 'lsx' ); ?></span>
					<?php endif; ?>
				</h2>
				<div class="entry-meta">
					<?php lsx_post_meta_list_top(); ?>
				</div><!-- .entry-meta -->

				<?php lsx_post_meta_category(); ?>

			</header><!-- .entry-header -->

			<?php if ( has_post_format( array( 'quote' ) ) || apply_filters( 'lsx_blog_display_text_on_list', true ) ) : ?>

				<?php if ( lsx_post_format_force_content_on_list() && ! apply_filters( 'lsx_blog_force_content_on_list', false ) ) : ?>

					<div class="entry-summary">
					<?php
					if ( ! has_excerpt() ) {

						$excerpt_more = '<p><a class="moretag" href="' . esc_url( get_permalink() ) . '">' . esc_html__( 'Read More', 'lsx' ) . '</a></p>';
						$content      = wp_trim_words( get_the_content(), 50 );
						$content      = '<p>' . $content . '</p>' . $excerpt_more;
						echo wp_kses_post( $content );
					} else {
						the_excerpt();
					}
					?>
					</div><!-- .entry-summary -->

				<?php elseif ( has_post_format( array( 'link' ) ) ) : ?>

				<?php elseif ( apply_filters( 'lsx_blog_force_content_on_list', false ) ) : ?>

					<div class="entry-content">
						<?php the_content(); ?>
					</div><!-- .entry-content -->

				<?php else : ?>

					<div class="entry-content">
						<?php
							the_content();

							wp_link_pages( array(

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

								'before'      => '<div class="lsx-postnav-wrapper"><div class="lsx-postnav">',
								'after'       => '</div></div>',
								'link_before' => '<span>',
								'link_after'  => '</span>',
							) );

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

						?>
					</div><!-- .entry-content -->

				<?php endif; ?>

			<?php endif; ?>

			<div class="clearfix"></div>

			<?php $comments_number = get_comments_number(); ?>

			<div class="post-tags-wrapper">

				<?php lsx_content_post_tags(); ?>

				<?php if ( comments_open() && ! empty( $comments_number ) ) : ?>
					<div class="post-comments">
						<a href="<?php the_permalink(); ?>#comments">
							<?php
							if ( '1' === $comments_number ) {
								echo esc_html_x( 'One Comment', 'content.php', 'lsx' );
							} else {
								printf(
									/* Translators: %s: number of comments */
									esc_html( _nx(
										'%s Comment',
										'%s Comments',
										$comments_number,
										'content.php',
										'lsx'
									) ),
									esc_html( number_format_i18n( $comments_number ) )
								);
							}
							?>
						</a>
					</div>
				<?php endif ?>
			</div>
		</div>

		<?php if ( has_post_thumbnail() ) : ?>

			<div class="entry-image hidden-xs">
				<a class="thumbnail" href="<?php the_permalink(); ?>" style="background-image:url(<?php echo esc_url( $image_src ); ?>);">
					<?php lsx_thumbnail( 'lsx-thumbnail-single' ); ?>
				</a>
			</div>

		<?php endif; ?>
	</div>

	<?php lsx_entry_bottom(); ?>

	<div class="clearfix"></div>

	<div class="lsx-breaker"></div>
</article>

<?php
lsx_entry_after();