Issues (1057)

templates/content-widget-review.php (23 issues)

1
<?php
2
/**
3
 * Reviews Widget Content Part
4
 *
5
 * @package 	tour-operators
0 ignored issues
show
Spaces must be used for mid-line alignment; tabs are not allowed
Loading history...
6
 * @category	review
0 ignored issues
show
Spaces must be used for mid-line alignment; tabs are not allowed
Loading history...
7
 * @subpackage	widget
0 ignored issues
show
Spaces must be used for mid-line alignment; tabs are not allowed
Loading history...
8
 */
9
10
global $disable_placeholder, $disable_text, $post;
11
12
$has_single = ! lsx_to_is_single_disabled();
13
$permalink = '';
0 ignored issues
show
Equals sign not aligned with surrounding assignments; expected 2 spaces but found 1 space

This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line.

To visualize

$a = "a";
$ab = "ab";
$abc = "abc";

will produce issues in the first and second line, while this second example

$a   = "a";
$ab  = "ab";
$abc = "abc";

will produce no issues.

Loading history...
14
15
if ( $has_single ) {
0 ignored issues
show
Expected 0 spaces after opening bracket; 1 found
Loading history...
Expected 0 spaces before closing bracket; 1 found
Loading history...
16
	$permalink = get_the_permalink();
17
} elseif ( ! is_post_type_archive( 'review' ) ) {
0 ignored issues
show
Expected 0 spaces after opening bracket; 1 found
Loading history...
Expected 0 spaces before closing bracket; 1 found
Loading history...
18
	$has_single = true;
19
	$permalink = get_post_type_archive_link( 'review' ) . '#review-' . $post->post_name;
0 ignored issues
show
Are you sure get_post_type_archive_link('review') of type false|string can be used in concatenation? ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-type  annotation

19
	$permalink = /** @scrutinizer ignore-type */ get_post_type_archive_link( 'review' ) . '#review-' . $post->post_name;
Loading history...
Equals sign not aligned with surrounding assignments; expected 2 spaces but found 1 space

This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line.

To visualize

$a = "a";
$ab = "ab";
$abc = "abc";

will produce issues in the first and second line, while this second example

$a   = "a";
$ab  = "ab";
$abc = "abc";

will produce no issues.

Loading history...
20
}
21
?>
22
<article <?php post_class(); ?>>
23
	<?php if ( empty( $disable_placeholder ) ) { ?>
0 ignored issues
show
Expected 0 spaces after opening bracket; 1 found
Loading history...
Expected 0 spaces before closing bracket; 1 found
Loading history...
24
		<div class="lsx-to-widget-thumb">
25
			<?php if ( $has_single ) { ?><a href="<?php echo esc_url( $permalink ); ?>"><?php } ?>
0 ignored issues
show
Expected 0 spaces after opening bracket; 1 found
Loading history...
Expected 0 spaces before closing bracket; 1 found
Loading history...
It seems like $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? ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-type  annotation

25
			<?php if ( $has_single ) { ?><a href="<?php echo esc_url( /** @scrutinizer ignore-type */ $permalink ); ?>"><?php } ?>
Loading history...
26
				<?php lsx_thumbnail( 'lsx-thumbnail-wide' ); ?>
27
			<?php if ( $has_single ) { ?></a><?php } ?>
0 ignored issues
show
Expected 0 spaces after opening bracket; 1 found
Loading history...
Expected 0 spaces before closing bracket; 1 found
Loading history...
28
		</div>
29
	<?php } ?>
30
31
	<h4 class="lsx-to-widget-title text-center">
32
		<?php if ( $has_single ) { ?><a href="<?php echo esc_url( $permalink ); ?>"><?php } ?>
0 ignored issues
show
Expected 0 spaces after opening bracket; 1 found
Loading history...
Expected 0 spaces before closing bracket; 1 found
Loading history...
33
			<?php the_title(); ?>
34
		<?php if ( $has_single ) { ?></a><?php } ?>
0 ignored issues
show
Expected 0 spaces after opening bracket; 1 found
Loading history...
Expected 0 spaces before closing bracket; 1 found
Loading history...
35
	</h4>
36
37
	<?php if ( empty( $disable_text ) ) { ?>
0 ignored issues
show
Expected 0 spaces after opening bracket; 1 found
Loading history...
Expected 0 spaces before closing bracket; 1 found
Loading history...
38
		<blockquote class="lsx-to-widget-blockquote">
39
			<?php
40
			$excerpt = get_the_excerpt();
41
			if ( empty( $excerpt ) || '' === $excerpt ) {
42
				$tooltip = apply_filters( 'get_the_excerpt', get_the_content() );
43
				$tooltip = strip_tags( $tooltip );
44
				echo wp_kses_post( wpautop( $tooltip ) );
45
			} else {
46
				echo wp_kses_post( $excerpt );
47
			}
48
			?>
49
		</blockquote>
50
	<?php } ?>
51
52
	<div class="lsx-to-widget-meta-data">
53
		<?php
54
			$meta_class = 'lsx-to-meta-data lsx-to-meta-data-';
55
56
			lsx_to_connected_accommodation( '<span class="' . $meta_class . 'accommodations"><span class="lsx-to-meta-data-key">' . __( 'Accommodation', 'to-reviews' ) . ':</span> ', '</span>' );
57
			lsx_to_connected_tours( '<span class="' . $meta_class . 'tours"><span class="lsx-to-meta-data-key">' . __( 'Tours', 'to-reviews' ) . ':</span> ', '</span>' );
58
			lsx_to_connected_destinations( '<span class="' . $meta_class . 'destinations"><span class="lsx-to-meta-data-key">' . __( 'Destinations', 'to-reviews' ) . ':</span> ', '</span>' );
59
			if ( function_exists( 'lsx_to_connected_team' ) ) {
60
				lsx_to_connected_team( '<span class="' . $meta_class . 'team"><span class="lsx-to-meta-data-key">' . __( 'Advised by', 'to-reviews' ) . ':</span> ', '</span>' );
61
			}
62
		?>
63
	</div>
64
65
</article>
66