Issues (617)

templates/content-vehicle.php (40 issues)

1
<?php
2
/**
3
 * Vehicle Content Part
4
 *
5
 * @package    lsx-tour-operators
6
 * @category   vehicle
7
 */
8
9
global $lsx_archive, $lsx_to_archive, $post;
10
11
if ( 1 !== $lsx_archive ) {
0 ignored issues
show
Expected 0 spaces after opening bracket; 1 found
Loading history...
Expected 0 spaces before closing bracket; 1 found
Loading history...
12
	$lsx_archive = false;
13
}
14
?>
15
16
<?php lsx_entry_before(); ?>
17
18
	<article id="tour-<?php echo esc_attr( $post->post_name ); ?>" <?php post_class( 'lsx-to-archive-container' ); ?>>
19
20
		<?php if ( is_archive() ) { ?>
0 ignored issues
show
Expected 0 spaces after opening bracket; 1 found
Loading history...
Expected 0 spaces before closing bracket; 1 found
Loading history...
21
		<div class="lsx-to-archive-thumb lsx-to-widget-thumb">
22
			<a href="<?php the_permalink(); ?>">
23
				<?php lsx_thumbnail( 'lsx-thumbnail-wide' ); ?>
24
			</a>
25
		</div>
26
27
		<div class="lsx-to-archive-wrapper">
28
			<div class="lsx-to-archive-content">
29
				<h3 class="lsx-to-archive-content-title">
30
					<a href="<?php the_permalink(); ?>" title="<?php esc_html_e( 'Read more', 'tour-operator' ); ?>">
31
						<?php
32
						the_title();
33
						do_action( 'lsx_to_the_title_end', get_the_ID() );
34
						?>
35
					</a>
36
				</h3>
37
				<!-- .entry-header -->
38
				<?php } ?>
0 ignored issues
show
Closing brace indented incorrectly; expected 8 spaces, found 16
Loading history...
39
40
				<div <?php lsx_to_entry_class( 'entry-content' ); ?>>
41
42
					<?php if ( is_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...
43
						<div <?php lsx_to_entry_class( 'entry-content' ); ?>>
44
							<div class="col-xs-12 col-sm-12 col-md-7">
45
								<div class="lsx-to-summary">
46
									<h2 class="lsx-to-summary-title"><?php the_title(); ?></h2>
47
								</div>
48
49
								<?php lsx_to_price( '<p class="lsx-to-meta-data lsx-to-meta-data-big lsx-to-meta-data-price"><span class="lsx-to-meta-data-key">' . esc_html__( 'From price', 'tour-operator' ) . ':</span> ', '</p>' ); ?>
50
								<?php the_content(); ?>
51
								<div class="key-fetures">
52
									<?php lsx_vehicle_features( '<div class="meta features">' . __( 'Features', 'to-vehicles' ) . ': ', '</div>' ); ?>
0 ignored issues
show
The call to lsx_vehicle_features() has too many arguments starting with '<div class="meta featur..., 'to-vehicles') . ': '. ( Ignorable by Annotation )

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

52
									<?php /** @scrutinizer ignore-call */ lsx_vehicle_features( '<div class="meta features">' . __( 'Features', 'to-vehicles' ) . ': ', '</div>' ); ?>

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress. Please note the @ignore annotation hint above.

Loading history...
53
								</div>
54
								<?php lsx_to_sharing(); ?>
55
							</div>
56
						</div>
57
						<div class="col-xs-12 col-sm-12 col-md-5">
58
							<section id="fast-facts">
59
								<div class="vehicle-summary">
60
									<div class="lsx-to-archive-thumb lsx-to-widget-thumb">
61
										<a href="<?php the_permalink(); ?>">
62
											<?php the_post_thumbnail( 'full' ); ?>
63
										</a>
64
									</div>
65
									<div class="lsx-to-section-inner">
66
										<h3 class="lsx-to-section-title"><?php esc_html_e( 'Vehicle Summary', 'tour-operator' ); ?></h3>
67
										<div class="lsx-to-single-meta-data">
68
											<div class="meta taxonomies">
69
												<?php //lsx_vehicle_code( '<div class="meta code">' . __( 'Code', 'to-vehicles' ) . ': ', '</div>' ); ?>
0 ignored issues
show
Unused Code Comprehensibility introduced by
47% of this comment could be valid code. Did you maybe forget this after debugging?

Sometimes obsolete code just ends up commented out instead of removed. In this case it is better to remove the code once you have checked you do not need it.

The code might also have been commented out for debugging purposes. In this case it is vital that someone uncomments it again or your project may behave in very unexpected ways in production.

This check looks for comments that seem to be mostly valid code and reports them.

Loading history...
No space found before comment text; expected "// lsx_vehicle_code( '<div class="meta code">' . __( 'Code', 'to-vehicles' ) . ': ', '</div>' );" but found "//lsx_vehicle_code( '<div class="meta code">' . __( 'Code', 'to-vehicles' ) . ': ', '</div>' );"
Loading history...
Inline comments must end in full-stops, exclamation marks, or question marks
Loading history...
70
												<?php lsx_vehicle_type( '<div class="meta vehicle-type"><span class="entry-meta-key">' . __( 'Type', 'to-vehicles' ) . '</span>: ', '</div>' ); ?>
71
												<?php lsx_vehicle_seating( '<div class="meta seating"><span class="entry-meta-key">' . __( 'Seats', 'to-vehicles' ) . '</span>: ', '</div>' ); ?>
72
												<?php lsx_vehicle_price( '<div class="meta seating"><span class="entry-meta-key">' . __( 'Price Guide', 'to-vehicles' ) . '</span>: ', '</div>' ); ?>
73
												<?php //lsx_vehicle_engine_type( '<div class="meta vehicle-type"><span class="entry-meta-key">' . __( 'Engine Type', 'to-vehicles' ) . '</span>: ', '</div>' ); ?>
0 ignored issues
show
Unused Code Comprehensibility introduced by
47% of this comment could be valid code. Did you maybe forget this after debugging?

Sometimes obsolete code just ends up commented out instead of removed. In this case it is better to remove the code once you have checked you do not need it.

The code might also have been commented out for debugging purposes. In this case it is vital that someone uncomments it again or your project may behave in very unexpected ways in production.

This check looks for comments that seem to be mostly valid code and reports them.

Loading history...
No space found before comment text; expected "// lsx_vehicle_engine_type( '<div class="meta vehicle-type"><span class="entry-meta-key">' . __( 'Engine Type', 'to-vehicles' ) . '</span>: ', '</div>' );" but found "//lsx_vehicle_engine_type( '<div class="meta vehicle-type"><span class="entry-meta-key">' . __( 'Engine Type', 'to-vehicles' ) . '</span>: ', '</div>' );"
Loading history...
Inline comments must end in full-stops, exclamation marks, or question marks
Loading history...
74
												<?php //lsx_vehicle_gearbox( '<div class="meta gearbox"><span class="entry-meta-key">' . __( 'Gearbox', 'to-vehicles' ) . '</span>: ', '</div>' ); ?>
0 ignored issues
show
Unused Code Comprehensibility introduced by
47% of this comment could be valid code. Did you maybe forget this after debugging?

Sometimes obsolete code just ends up commented out instead of removed. In this case it is better to remove the code once you have checked you do not need it.

The code might also have been commented out for debugging purposes. In this case it is vital that someone uncomments it again or your project may behave in very unexpected ways in production.

This check looks for comments that seem to be mostly valid code and reports them.

Loading history...
No space found before comment text; expected "// lsx_vehicle_gearbox( '<div class="meta gearbox"><span class="entry-meta-key">' . __( 'Gearbox', 'to-vehicles' ) . '</span>: ', '</div>' );" but found "//lsx_vehicle_gearbox( '<div class="meta gearbox"><span class="entry-meta-key">' . __( 'Gearbox', 'to-vehicles' ) . '</span>: ', '</div>' );"
Loading history...
Inline comments must end in full-stops, exclamation marks, or question marks
Loading history...
75
												<?php //lsx_vehicle_engine_size( '<div class="meta engine-size"><span class="entry-meta-key">' . __( 'Engine-size', 'to-vehicles' ) . '</span>: ', '</div>' ); ?>
0 ignored issues
show
Unused Code Comprehensibility introduced by
47% of this comment could be valid code. Did you maybe forget this after debugging?

Sometimes obsolete code just ends up commented out instead of removed. In this case it is better to remove the code once you have checked you do not need it.

The code might also have been commented out for debugging purposes. In this case it is vital that someone uncomments it again or your project may behave in very unexpected ways in production.

This check looks for comments that seem to be mostly valid code and reports them.

Loading history...
No space found before comment text; expected "// lsx_vehicle_engine_size( '<div class="meta engine-size"><span class="entry-meta-key">' . __( 'Engine-size', 'to-vehicles' ) . '</span>: ', '</div>' );" but found "//lsx_vehicle_engine_size( '<div class="meta engine-size"><span class="entry-meta-key">' . __( 'Engine-size', 'to-vehicles' ) . '</span>: ', '</div>' );"
Loading history...
Inline comments must end in full-stops, exclamation marks, or question marks
Loading history...
76
												<?php //lsx_vehicle_gears( '<div class="meta gears"><span class="entry-meta-key">' . __( 'Gears', 'to-vehicles' ) . '</span>: ', '</div>' ); ?>
0 ignored issues
show
Unused Code Comprehensibility introduced by
47% of this comment could be valid code. Did you maybe forget this after debugging?

Sometimes obsolete code just ends up commented out instead of removed. In this case it is better to remove the code once you have checked you do not need it.

The code might also have been commented out for debugging purposes. In this case it is vital that someone uncomments it again or your project may behave in very unexpected ways in production.

This check looks for comments that seem to be mostly valid code and reports them.

Loading history...
No space found before comment text; expected "// lsx_vehicle_gears( '<div class="meta gears"><span class="entry-meta-key">' . __( 'Gears', 'to-vehicles' ) . '</span>: ', '</div>' );" but found "//lsx_vehicle_gears( '<div class="meta gears"><span class="entry-meta-key">' . __( 'Gears', 'to-vehicles' ) . '</span>: ', '</div>' );"
Loading history...
Inline comments must end in full-stops, exclamation marks, or question marks
Loading history...
77
78
											</div>
79
										</div>
80
									</div>
81
								</div>
82
							</section>
83
							<?php if ( lsx_to_has_enquiry_contact() ) : ?>
0 ignored issues
show
Expected 0 spaces after opening bracket; 1 found
Loading history...
Expected 0 spaces before closing bracket; 1 found
Loading history...
84
								<div class="lsx-to-contact-widget">
85
									<?php
86
									if ( function_exists( 'lsx_to_has_team_member' ) && lsx_to_has_team_member() ) {
0 ignored issues
show
Expected 0 spaces after opening bracket; 1 found
Loading history...
Expected 0 spaces before closing bracket; 1 found
Loading history...
87
										lsx_to_team_member_panel( '<div class="lsx-to-contact">', '</div>' );
0 ignored issues
show
The function lsx_to_team_member_panel was not found. Maybe you did not declare it correctly or list all dependencies? ( Ignorable by Annotation )

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

87
										/** @scrutinizer ignore-call */ 
88
          lsx_to_team_member_panel( '<div class="lsx-to-contact">', '</div>' );
Loading history...
88
									} else {
89
										lsx_to_enquiry_contact( '<div class="lsx-to-contact">', '</div>' );
90
									}
91
92
									lsx_to_enquire_modal();
93
									?>
94
								</div>
95
							<?php endif ?>
96
						</div>
97
					<?php } else { ?>
98
						<div class="lsx-to-archive-meta-data lsx-to-archive-meta-data-grid-mode">
99
							<div class="vehicle-details">
100
								<?php if ( false !== get_post_meta( get_the_ID(), 'type', true ) ) { ?>
0 ignored issues
show
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? ( Ignorable by Annotation )

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

100
								<?php if ( false !== get_post_meta( /** @scrutinizer ignore-type */ get_the_ID(), 'type', true ) ) { ?>
Loading history...
Expected 0 spaces after opening bracket; 1 found
Loading history...
Expected 0 spaces before closing bracket; 1 found
Loading history...
101
									<div class="meta type lsx-to-meta-data lsx-to-meta-data-type"><?php esc_html_e( 'Type', 'to-vehicles' ); ?>: <span><?php echo esc_attr( get_post_meta( get_the_ID(), 'vehicle_type', true ) ); ?></span></div>
0 ignored issues
show
It seems like get_post_meta(get_the_ID(), 'vehicle_type', true) can also be of type false; however, parameter $text of esc_attr() 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

101
									<div class="meta type lsx-to-meta-data lsx-to-meta-data-type"><?php esc_html_e( 'Type', 'to-vehicles' ); ?>: <span><?php echo esc_attr( /** @scrutinizer ignore-type */ get_post_meta( get_the_ID(), 'vehicle_type', true ) ); ?></span></div>
Loading history...
102
								<?php } ?>
103
								<?php if ( false !== get_post_meta( get_the_ID(), 'seating', true ) ) { ?>
0 ignored issues
show
Expected 0 spaces after opening bracket; 1 found
Loading history...
Expected 0 spaces before closing bracket; 1 found
Loading history...
104
									<div class="meta seats lsx-to-meta-data lsx-to-meta-data-seating"><?php esc_html_e( 'Seats', 'to-vehicles' ); ?>: <span><?php echo esc_attr( get_post_meta( get_the_ID(), 'seating', true ) ); ?></span></div>
105
								<?php } ?>
106
								<?php if ( false !== get_post_meta( get_the_ID(), 'price', true ) ) { ?>
0 ignored issues
show
Expected 0 spaces after opening bracket; 1 found
Loading history...
Expected 0 spaces before closing bracket; 1 found
Loading history...
107
									<div class="meta price lsx-to-meta-data lsx-to-meta-data-price"><?php esc_html_e( 'Price guide', 'to-vehicles' ); ?>: <span><?php echo esc_attr( get_post_meta( get_the_ID(), 'price', true ) ); ?></span></div>
108
								<?php } ?>
109
							</div>
110
						</div>
111
						<div class="entry-content">
112
							<?php the_excerpt(); ?>
113
						</div>
114
					<?php } ?>
115
				</div><!-- .entry-content -->
116
117
				<?php if ( is_archive() ) { ?>
0 ignored issues
show
Expected 0 spaces after opening bracket; 1 found
Loading history...
Expected 0 spaces before closing bracket; 1 found
Loading history...
118
			</div>
119
		</div>
120
	<?php echo '<a class="moretag" href="' . wp_kses_post( get_permalink() ) . '" title="View more">', 'View More</a>'; ?>
0 ignored issues
show
It seems like get_permalink() 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? ( Ignorable by Annotation )

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

120
	<?php echo '<a class="moretag" href="' . wp_kses_post( /** @scrutinizer ignore-type */ get_permalink() ) . '" title="View more">', 'View More</a>'; ?>
Loading history...
121
	<?php } ?>
0 ignored issues
show
Closing brace indented incorrectly; expected 16 spaces, found 4
Loading history...
122
123
		<?php
124
		/**
125
		 * Hooked
126
		 *
127
		 *  - lsx_vehicle_metabox() - 10
128
		 */
129
		lsx_entry_bottom();
130
		?>
131
132
	</article><!-- #post-## -->
133
134
<?php
135
lsx_entry_after();
136