Passed
Push — master ( b096e4...dcfecd )
by Chris
05:13
created

LSX_Schema_Graph_Piece::is_needed()   A

Complexity

Conditions 3
Paths 3

Size

Total Lines 8
Code Lines 5

Duplication

Lines 0
Ratio 0 %

Importance

Changes 0
Metric Value
cc 3
eloc 5
nc 3
nop 0
dl 0
loc 8
rs 10
c 0
b 0
f 0
1
<?php
2
/**
3
 * Schema for LSX
4
 *
5
 * @package lsx
6
 */
7
/**
8
 * Returns schema Review data.
9
 *
10
 * @since 10.2
11
 */
12
if ( interface_exists( 'WPSEO_Graph_Piece' ) ) {
13
14
	class LSX_Schema_Graph_Piece implements WPSEO_Graph_Piece {
0 ignored issues
show
Bug introduced by
The type WPSEO_Graph_Piece was not found. Maybe you did not declare it correctly or list all dependencies?

The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. excluded_paths: ["lib/*"], you can move it to the dependency path list as follows:

filter:
    dependency_paths: ["lib/*"]

For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths

Loading history...
15
		/**
16
		 * A value object with context variables.
17
		 *
18
		 * @var \WPSEO_Schema_Context
0 ignored issues
show
Bug introduced by
The type WPSEO_Schema_Context was not found. Maybe you did not declare it correctly or list all dependencies?

The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. excluded_paths: ["lib/*"], you can move it to the dependency path list as follows:

filter:
    dependency_paths: ["lib/*"]

For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths

Loading history...
19
		 */
20
		public $context;
21
		/**
22
		 * This is the post type that you want the piece to output for.
23
		 *
24
		 * @var string;
25
		 */
26
		public $post_type;
27
		/**
28
		 * If this is a top level parent
29
		 *
30
		 * @var boolean
31
		 */
32
		public $is_top_level;
33
		/**
34
		 * This holds the meta_key => scehma_type of the fields you want to add to your subtrip.
35
		 *
36
		 * @var array()
37
		 */
38
		public $place_ids;
39
		/**
40
		 * This holds an object or the current trip post.
41
		 *
42
		 * @var WP_Post();
43
		 */
44
		public $post;
45
		/**
46
		 * This holds URL for the trip
47
		 *
48
		 * @var string
49
		 */
50
		public $post_url;
51
		/**
52
		 * Constructor.
53
		 *
54
		 * @param \WPSEO_Schema_Context $context A value object with context variables.
55
		 */
56
		public function __construct( WPSEO_Schema_Context $context ) {
57
			$this->context      = $context;
58
			$this->place_ids    = array();
59
			$this->post         = get_post( $this->context->id );
60
			$this->post_url     = get_permalink( $this->context->id );
0 ignored issues
show
Documentation Bug introduced by
It seems like get_permalink($this->context->id) can also be of type false. However, the property $post_url is declared as type string. Maybe add an additional type check?

Our type inference engine has found a suspicous assignment of a value to a property. This check raises an issue when a value that can be of a mixed type is assigned to a property that is type hinted more strictly.

For example, imagine you have a variable $accountId that can either hold an Id object or false (if there is no account id yet). Your code now assigns that value to the id property of an instance of the Account class. This class holds a proper account, so the id value must no longer be false.

Either this assignment is in error or a type check should be added for that assignment.

class Id
{
    public $id;

    public function __construct($id)
    {
        $this->id = $id;
    }

}

class Account
{
    /** @var  Id $id */
    public $id;
}

$account_id = false;

if (starsAreRight()) {
    $account_id = new Id(42);
}

$account = new Account();
if ($account instanceof Id)
{
    $account->id = $account_id;
}
Loading history...
61
			$this->is_top_level = false;
62
			if ( is_object( $this->post ) && isset( $this->post->post_parent ) && ( false === $this->post->post_parent || 0 === $this->post->post_parent || '' === $this->post->post_parent ) ) {
63
				$this->is_top_level = true;
64
			}
65
		}
66
		/**
67
		 * Determines whether or not a piece should be added to the graph.
68
		 *
69
		 * @return bool
70
		 */
71
		public function is_needed() {
72
			if ( ! is_singular() ) {
73
				return false;
74
			}
75
			if ( false === $this->context->site_represents ) {
76
				return false;
77
			}
78
			return LSX_Schema_Utils::is_type( get_post_type(), $this->post_type );
79
		}
80
		/**
81
		 * Returns Review data.
82
		 *
83
		 * @return array $data Review data.
84
		 */
85
		public function generate() {
86
			$data = array();
87
			return $data;
88
		}
89
		/**
90
		 * Gets the connected reviews post type and set it as the "Review" schema
91
		 *
92
		 * @param  array    $data An array of offers already added.
93
		 * @param  string   $data_key
94
		 * @param  boolean  $include_aggregate
95
		 * @return array    $data
96
		 */
97
		public function add_reviews( $data, $data_key = 'reviews', $include_aggregate = true ) {
98
			$reviews       = get_post_meta( $this->context->id, 'review_to_' . $this->post_type, false );
99
			$reviews_array = array();
100
			if ( ! empty( $reviews ) ) {
101
				$aggregate_value = 1;
102
				$review_count    = 0;
103
				foreach ( $reviews as $review_id ) {
104
					$rating      = get_post_meta( $review_id, 'rating', true );
105
					$author      = get_post_meta( $review_id, 'reviewer_name', true );
106
					$description = wp_strip_all_tags( get_the_excerpt( $review_id ) );
107
					$review_args = array(
108
						'author'     => $author,
109
						'reviewBody' => $description,
110
					);
111
					// Add in the review rating.
112
					if ( false !== $rating && '' !== $rating && '0' !== $rating && 0 !== $rating ) {
113
						$review_args['reviewRating'] = array(
114
							'@type'       => 'Rating',
115
							'ratingValue' => $rating,
116
						);
117
					}
118
					$reviews_array = LSX_Schema_Utils::add_review( $reviews_array, $review_id, $this->context, $review_args );
0 ignored issues
show
Bug introduced by
The method add_review() does not exist on LSX_Schema_Utils. ( Ignorable by Annotation )

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

118
					/** @scrutinizer ignore-call */ 
119
     $reviews_array = LSX_Schema_Utils::add_review( $reviews_array, $review_id, $this->context, $review_args );

This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces.

This is most likely a typographical error or the method has been renamed.

Loading history...
119
					$review_count++;
120
				}
121
				if ( ! empty( $reviews_array ) ) {
122
					if ( true === $include_aggregate ) {
123
						$data['aggregateRating'] = array(
124
							'@type'       => 'AggregateRating',
125
							'ratingValue' => (string) $aggregate_value,
126
							'reviewCount' => (string) $review_count,
127
							'bestRating'  => '5',
128
							'worstRating' => '1',
129
						);
130
					}
131
					$data[ $data_key ] = $reviews_array;
132
				}
133
			}
134
			return $data;
135
		}
136
		/**
137
		 * Gets the connected posts and set it as the "Article" schema
138
		 *
139
		 * @param  array  $data An array of offers already added.
140
		 * @param  string $data_key
141
		 * @return array  $data
142
		 */
143
		public function add_articles( $data, $data_key = 'subjectOf' ) {
144
			$posts       = get_post_meta( $this->context->id, 'post_to_' . $this->post_type, false );
145
			$posts_array = array();
146
			if ( ! empty( $posts ) ) {
147
				foreach ( $posts as $post_id ) {
148
					$post_args = array(
149
						'articleBody' => wp_strip_all_tags( get_the_excerpt( $post_id ) ),
150
						'headline'    => get_the_title( $post_id ),
151
					);
152
					$section   = get_the_term_list( $post_id, 'category' );
153
					if ( ! is_wp_error( $section ) && '' !== $section && false !== $section ) {
154
						$post_args['articleSection'] = wp_strip_all_tags( $section );
0 ignored issues
show
Bug introduced by
It seems like $section can also be of type WP_Error; however, parameter $string of wp_strip_all_tags() 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

154
						$post_args['articleSection'] = wp_strip_all_tags( /** @scrutinizer ignore-type */ $section );
Loading history...
155
					}
156
					if ( $this->context->site_represents_reference ) {
157
						$post_args['publisher'] = $this->context->site_represents_reference;
158
					}
159
					$image_url = get_the_post_thumbnail_url( $post_id, 'lsx-thumbnail-wide' );
160
					if ( false !== $image_url ) {
161
						$post_args['image'] = $image_url;
162
					}
163
					$posts_array = LSX_Schema_Utils::add_article( $posts_array, $post_id, $this->context, $post_args );
0 ignored issues
show
Bug introduced by
The method add_article() does not exist on LSX_Schema_Utils. ( Ignorable by Annotation )

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

163
					/** @scrutinizer ignore-call */ 
164
     $posts_array = LSX_Schema_Utils::add_article( $posts_array, $post_id, $this->context, $post_args );

This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces.

This is most likely a typographical error or the method has been renamed.

Loading history...
164
				}
165
				if ( ! empty( $posts_array ) ) {
166
					$data[ $data_key ] = $posts_array;
167
				}
168
			}
169
			return $data;
170
		}
171
		/**
172
		 * Adds the Project and Testimonials attached to the Team Member
173
		 *
174
		 * @param array $data
175
		 *
176
		 * @return array $data
177
		 */
178
		public function add_connections( $data ) {
179
			$connections_array = array();
180
			if ( $this->is_top_level ) {
181
				$connections_array = $this->add_regions( $connections_array );
182
				$connections_array = $this->add_accommodation( $connections_array );
183
				if ( ! empty( $connections_array ) ) {
184
					$data['containsPlace'] = $connections_array;
185
				}
186
			} else {
187
				$connections_array             = $this->add_countries( $connections_array );
188
				$data['containedInPlace'] = $connections_array;
189
				$connections_array          = array();
190
				$connections_array          = $this->add_accommodation( $connections_array );
191
				$data['containsPlace'] = $connections_array;
192
			}
193
			return $data;
194
		}
195
		/**
196
		 * Adds the terms for the taxonomy
197
		 *
198
		 * @param array $data     Review data.
199
		 * @param array $data_key the parameter name you wish to assign it to.
200
		 * @param array $taxonomy the taxonomy to grab terms for.
201
		 *
202
		 * @return array $data Review data.
203
		 */
204
		public function add_taxonomy_terms( $data, $data_key, $taxonomy ) {
205
			/**
206
			 * Filter: 'lsx_schema_' . $this->post_type . '_' . $data_key . '_taxonomy' - Allow changing the taxonomy used to assign keywords to a post type Review data.
207
			 *
208
			 * @api string $taxonomy The chosen taxonomy.
209
			 */
210
			$taxonomy = apply_filters( 'lsx_schema_' . $this->post_type . '_' . $data_key . '_taxonomy', $taxonomy );
0 ignored issues
show
Bug introduced by
Are you sure $data_key of type array 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

210
			$taxonomy = apply_filters( 'lsx_schema_' . $this->post_type . '_' . /** @scrutinizer ignore-type */ $data_key . '_taxonomy', $taxonomy );
Loading history...
211
			return LSX_Schema_Utils::add_terms( $data, $this->context->id, $data_key, $taxonomy );
0 ignored issues
show
Bug introduced by
$data_key of type array is incompatible with the type string expected by parameter $key of LSX_Schema_Utils::add_terms(). ( Ignorable by Annotation )

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

211
			return LSX_Schema_Utils::add_terms( $data, $this->context->id, /** @scrutinizer ignore-type */ $data_key, $taxonomy );
Loading history...
Bug introduced by
It seems like $taxonomy can also be of type array; however, parameter $taxonomy of LSX_Schema_Utils::add_terms() 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

211
			return LSX_Schema_Utils::add_terms( $data, $this->context->id, $data_key, /** @scrutinizer ignore-type */ $taxonomy );
Loading history...
212
		}
213
		/**
214
		 * Adds the custom field value for the supplied key
215
		 *
216
		 * @param array   $data     Schema data.
217
		 * @param string  $data_key the parameter name you wish to assign it to.
218
		 * @param string  $meta_key the taxonomy to grab terms for.
219
		 * @param boolean $single   A single custom field or an array
220
		 *
221
		 * @return array $data Review data.
222
		 */
223
		public function add_custom_field( $data, $data_key, $meta_key, $single = true ) {
224
			$value = get_post_meta( $this->context->id, $meta_key, $single );
225
			if ( '' !== $value && false !== $value ) {
226
				$data[ $data_key ] = $value;
227
			}
228
			return $data;
229
		}
230
	}
231
}
232