Passed
Push — add/cover-template-update ( ca0bef...ba8c8e )
by Virginia
09:49 queued 04:07
created

LSX_Schema_Graph_Piece::add_connections()   A

Complexity

Conditions 3
Paths 3

Size

Total Lines 16
Code Lines 13

Duplication

Lines 0
Ratio 0 %

Importance

Changes 0
Metric Value
cc 3
eloc 13
nc 3
nop 1
dl 0
loc 16
rs 9.8333
c 0
b 0
f 0
1
<?php
2
/**
3
 * Schema for LSX
4
 *
5
 * @package lsx
6
 */
0 ignored issues
show
Coding Style introduced by
There must be exactly one blank line after the file comment
Loading history...
7
/**
8
 * Returns schema Review data.
9
 *
10
 * @since 10.2
11
 */
12
use \Yoast\WP\SEO\Generators\Schema\Abstract_Schema_Piece;
13
14
if ( class_exists( 'Abstract_Schema_Piece' ) ) {
0 ignored issues
show
Coding Style introduced by
Expected 0 spaces after opening bracket; 1 found
Loading history...
Coding Style introduced by
Expected 0 spaces before closing bracket; 1 found
Loading history...
15
	class LSX_Schema_Graph_Piece extends Abstract_Schema_Piece {
0 ignored issues
show
Coding Style Documentation introduced by
Missing doc comment for class LSX_Schema_Graph_Piece
Loading history...
16
		/**
17
		 * A value object with context variables.
18
		 *
19
		 * @var \WPSEO_Schema_Context
20
		 */
21
		public $context;
0 ignored issues
show
Coding Style introduced by
Expected 1 blank line(s) before first member var; 0 found
Loading history...
22
		/**
23
		 * This is the post type that you want the piece to output for.
24
		 *
25
		 * @var string;
26
		 */
27
		public $post_type;
0 ignored issues
show
Coding Style introduced by
Expected 1 blank line(s) before member var; 0 found
Loading history...
28
		/**
29
		 * If this is a top level parent
30
		 *
31
		 * @var boolean
32
		 */
33
		public $is_top_level;
0 ignored issues
show
Coding Style introduced by
Expected 1 blank line(s) before member var; 0 found
Loading history...
34
		/**
35
		 * This holds the meta_key => scehma_type of the fields you want to add to your subtrip.
36
		 *
37
		 * @var array()
38
		 */
39
		public $place_ids;
0 ignored issues
show
Coding Style introduced by
Expected 1 blank line(s) before member var; 0 found
Loading history...
40
		/**
41
		 * This holds an object or the current trip post.
42
		 *
43
		 * @var WP_Post();
44
		 */
45
		public $post;
0 ignored issues
show
Coding Style introduced by
Expected 1 blank line(s) before member var; 0 found
Loading history...
46
		/**
47
		 * This holds URL for the trip
48
		 *
49
		 * @var string
50
		 */
51
		public $post_url;
0 ignored issues
show
Coding Style introduced by
Expected 1 blank line(s) before member var; 0 found
Loading history...
52
		/**
53
		 * Constructor.
54
		 *
55
		 * @param \WPSEO_Schema_Context $context A value object with context variables.
56
		 */
57
		public function __construct( WPSEO_Schema_Context $context ) {
0 ignored issues
show
Coding Style introduced by
Expected 2 blank lines before function; 0 found
Loading history...
58
			$this->context      = $context;
59
			$this->place_ids    = array();
60
			$this->post         = get_post( $this->context->id );
0 ignored issues
show
Bug introduced by
$this->context->id of type string is incompatible with the type WP_Post|integer|null expected by parameter $post of get_post(). ( Ignorable by Annotation )

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

60
			$this->post         = get_post( /** @scrutinizer ignore-type */ $this->context->id );
Loading history...
61
			$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...
Bug introduced by
$this->context->id of type string is incompatible with the type WP_Post|integer expected by parameter $post of get_permalink(). ( Ignorable by Annotation )

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

61
			$this->post_url     = get_permalink( /** @scrutinizer ignore-type */ $this->context->id );
Loading history...
62
			$this->is_top_level = false;
63
			if ( is_object( $this->post ) && isset( $this->post->post_parent ) && ( false === $this->post->post_parent || 0 === $this->post->post_parent || '' === $this->post->post_parent ) ) {
0 ignored issues
show
Coding Style introduced by
Expected 0 spaces after opening bracket; 1 found
Loading history...
Coding Style introduced by
Expected 0 spaces before closing bracket; 1 found
Loading history...
64
				$this->is_top_level = true;
65
			}
66
		}
0 ignored issues
show
Coding Style introduced by
Expected 2 blank lines after function; 0 found
Loading history...
Coding Style introduced by
Expected 1 blank line before closing function brace; 0 found
Loading history...
67
		/**
68
		 * Determines whether or not a piece should be added to the graph.
69
		 *
70
		 * @return bool
71
		 */
72
		public function is_needed() {
73
			if ( ! is_singular() ) {
0 ignored issues
show
Coding Style introduced by
Expected 0 spaces after opening bracket; 1 found
Loading history...
Coding Style introduced by
Expected 0 spaces before closing bracket; 1 found
Loading history...
74
				return false;
75
			}
0 ignored issues
show
Coding Style introduced by
No blank line found after control structure
Loading history...
76
			if ( false === $this->context->site_represents ) {
0 ignored issues
show
introduced by
The condition false === $this->context->site_represents is always false.
Loading history...
Coding Style introduced by
Expected 0 spaces after opening bracket; 1 found
Loading history...
Coding Style introduced by
Expected 0 spaces before closing bracket; 1 found
Loading history...
77
				return false;
78
			}
0 ignored issues
show
Coding Style introduced by
No blank line found after control structure
Loading history...
79
			return LSX_Schema_Utils::is_type( get_post_type(), $this->post_type );
80
		}
0 ignored issues
show
Coding Style introduced by
Expected 2 blank lines after function; 0 found
Loading history...
Coding Style introduced by
Expected 1 blank line before closing function brace; 0 found
Loading history...
81
		/**
82
		 * Returns Review data.
83
		 *
84
		 * @return array $data Review data.
85
		 */
86
		public function generate() {
87
			$data = array();
88
			return $data;
89
		}
0 ignored issues
show
Coding Style introduced by
Expected 2 blank lines after function; 0 found
Loading history...
Coding Style introduced by
Expected 1 blank line before closing function brace; 0 found
Loading history...
90
		/**
91
		 * Gets the connected reviews post type and set it as the "Review" schema
92
		 *
93
		 * @param  array    $data An array of offers already added.
0 ignored issues
show
Coding Style introduced by
Expected 3 spaces after parameter type; 4 found
Loading history...
94
		 * @param  string   $data_key
0 ignored issues
show
Coding Style Documentation introduced by
Missing parameter comment
Loading history...
Coding Style introduced by
Expected 2 spaces after parameter type; 3 found
Loading history...
95
		 * @param  boolean  $include_aggregate
0 ignored issues
show
Coding Style Documentation introduced by
Missing parameter comment
Loading history...
Coding Style introduced by
Expected 1 spaces after parameter type; 2 found
Loading history...
96
		 * @return array    $data
97
		 */
98
		public function add_reviews( $data, $data_key = 'reviews', $include_aggregate = true ) {
99
			$reviews       = get_post_meta( $this->context->id, 'review_to_' . $this->post_type, false );
0 ignored issues
show
Bug introduced by
$this->context->id of type string is incompatible with the type integer expected by parameter $post_id of get_post_meta(). ( Ignorable by Annotation )

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

99
			$reviews       = get_post_meta( /** @scrutinizer ignore-type */ $this->context->id, 'review_to_' . $this->post_type, false );
Loading history...
100
			$reviews_array = array();
101
			if ( ! empty( $reviews ) ) {
0 ignored issues
show
Coding Style introduced by
Expected 0 spaces after opening bracket; 1 found
Loading history...
Coding Style introduced by
Expected 0 spaces before closing bracket; 1 found
Loading history...
102
				$aggregate_value = 1;
103
				$review_count    = 0;
104
				foreach ( $reviews as $review_id ) {
0 ignored issues
show
Coding Style introduced by
Expected 0 spaces after opening bracket; 1 found
Loading history...
Coding Style introduced by
Expected 0 spaces before closing bracket; 1 found
Loading history...
105
					$rating      = get_post_meta( $review_id, 'rating', true );
106
					$author      = get_post_meta( $review_id, 'reviewer_name', true );
107
					$description = wp_strip_all_tags( get_the_excerpt( $review_id ) );
108
					$review_args = array(
109
						'author'     => $author,
110
						'reviewBody' => $description,
111
					);
112
					// Add in the review rating.
113
					if ( false !== $rating && '' !== $rating && '0' !== $rating && 0 !== $rating ) {
0 ignored issues
show
Coding Style introduced by
Expected 0 spaces after opening bracket; 1 found
Loading history...
Coding Style introduced by
Expected 0 spaces before closing bracket; 1 found
Loading history...
114
						$review_args['reviewRating'] = array(
115
							'@type'       => 'Rating',
116
							'ratingValue' => $rating,
117
						);
118
					}
0 ignored issues
show
Coding Style introduced by
No blank line found after control structure
Loading history...
119
					$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

119
					/** @scrutinizer ignore-call */ 
120
     $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...
120
					$review_count++;
121
				}
0 ignored issues
show
Coding Style introduced by
No blank line found after control structure
Loading history...
122
				if ( ! empty( $reviews_array ) ) {
0 ignored issues
show
Coding Style introduced by
Expected 0 spaces after opening bracket; 1 found
Loading history...
Coding Style introduced by
Expected 0 spaces before closing bracket; 1 found
Loading history...
123
					if ( true === $include_aggregate ) {
0 ignored issues
show
Coding Style introduced by
Expected 0 spaces after opening bracket; 1 found
Loading history...
Coding Style introduced by
Expected 0 spaces before closing bracket; 1 found
Loading history...
124
						$data['aggregateRating'] = array(
125
							'@type'       => 'AggregateRating',
126
							'ratingValue' => (string) $aggregate_value,
127
							'reviewCount' => (string) $review_count,
128
							'bestRating'  => '5',
129
							'worstRating' => '1',
130
						);
131
					}
0 ignored issues
show
Coding Style introduced by
No blank line found after control structure
Loading history...
132
					$data[ $data_key ] = $reviews_array;
133
				}
134
			}
0 ignored issues
show
Coding Style introduced by
No blank line found after control structure
Loading history...
135
			return $data;
136
		}
0 ignored issues
show
Coding Style introduced by
Expected 2 blank lines after function; 0 found
Loading history...
Coding Style introduced by
Expected 1 blank line before closing function brace; 0 found
Loading history...
137
		/**
138
		 * Gets the connected posts and set it as the "Article" schema
139
		 *
140
		 * @param  array  $data An array of offers already added.
141
		 * @param  string $data_key
0 ignored issues
show
Coding Style Documentation introduced by
Missing parameter comment
Loading history...
142
		 * @return array  $data
143
		 */
144
		public function add_articles( $data, $data_key = 'subjectOf' ) {
145
			$posts       = get_post_meta( $this->context->id, 'post_to_' . $this->post_type, false );
0 ignored issues
show
Bug introduced by
$this->context->id of type string is incompatible with the type integer expected by parameter $post_id of get_post_meta(). ( Ignorable by Annotation )

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

145
			$posts       = get_post_meta( /** @scrutinizer ignore-type */ $this->context->id, 'post_to_' . $this->post_type, false );
Loading history...
146
			$posts_array = array();
147
			if ( ! empty( $posts ) ) {
0 ignored issues
show
Coding Style introduced by
Expected 0 spaces after opening bracket; 1 found
Loading history...
Coding Style introduced by
Expected 0 spaces before closing bracket; 1 found
Loading history...
148
				foreach ( $posts as $post_id ) {
0 ignored issues
show
Coding Style introduced by
Expected 0 spaces after opening bracket; 1 found
Loading history...
Coding Style introduced by
Expected 0 spaces before closing bracket; 1 found
Loading history...
149
					$post_args = array(
150
						'articleBody' => wp_strip_all_tags( get_the_excerpt( $post_id ) ),
151
						'headline'    => get_the_title( $post_id ),
152
					);
153
					$section   = get_the_term_list( $post_id, 'category' );
154
					if ( ! is_wp_error( $section ) && '' !== $section && false !== $section ) {
0 ignored issues
show
Coding Style introduced by
Expected 0 spaces after opening bracket; 1 found
Loading history...
Coding Style introduced by
Expected 0 spaces before closing bracket; 1 found
Loading history...
155
						$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

155
						$post_args['articleSection'] = wp_strip_all_tags( /** @scrutinizer ignore-type */ $section );
Loading history...
156
					}
0 ignored issues
show
Coding Style introduced by
No blank line found after control structure
Loading history...
157
					if ( $this->context->site_represents_reference ) {
0 ignored issues
show
Coding Style introduced by
Expected 0 spaces after opening bracket; 1 found
Loading history...
Coding Style introduced by
Expected 0 spaces before closing bracket; 1 found
Loading history...
158
						$post_args['publisher'] = $this->context->site_represents_reference;
0 ignored issues
show
Documentation Bug introduced by
It seems like $this->context->site_represents_reference can also be of type true. However, the property $site_represents_reference is declared as type array|false. 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...
159
					}
0 ignored issues
show
Coding Style introduced by
No blank line found after control structure
Loading history...
160
					$image_url = get_the_post_thumbnail_url( $post_id, 'lsx-thumbnail-wide' );
161
					if ( false !== $image_url ) {
0 ignored issues
show
Coding Style introduced by
Expected 0 spaces after opening bracket; 1 found
Loading history...
Coding Style introduced by
Expected 0 spaces before closing bracket; 1 found
Loading history...
162
						$post_args['image'] = $image_url;
163
					}
0 ignored issues
show
Coding Style introduced by
No blank line found after control structure
Loading history...
164
					$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

164
					/** @scrutinizer ignore-call */ 
165
     $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...
165
				}
0 ignored issues
show
Coding Style introduced by
No blank line found after control structure
Loading history...
166
				if ( ! empty( $posts_array ) ) {
0 ignored issues
show
Coding Style introduced by
Expected 0 spaces after opening bracket; 1 found
Loading history...
Coding Style introduced by
Expected 0 spaces before closing bracket; 1 found
Loading history...
167
					$data[ $data_key ] = $posts_array;
168
				}
169
			}
0 ignored issues
show
Coding Style introduced by
No blank line found after control structure
Loading history...
170
			return $data;
171
		}
0 ignored issues
show
Coding Style introduced by
Expected 2 blank lines after function; 0 found
Loading history...
Coding Style introduced by
Expected 1 blank line before closing function brace; 0 found
Loading history...
172
		/**
173
		 * Adds the Project and Testimonials attached to the Team Member
174
		 *
175
		 * @param array $data
0 ignored issues
show
Coding Style Documentation introduced by
Missing parameter comment
Loading history...
176
		 *
177
		 * @return array $data
178
		 */
179
		public function add_connections( $data ) {
180
			$connections_array = array();
181
			if ( $this->is_top_level ) {
0 ignored issues
show
Coding Style introduced by
Expected 0 spaces after opening bracket; 1 found
Loading history...
Coding Style introduced by
Expected 0 spaces before closing bracket; 1 found
Loading history...
182
				$connections_array = $this->add_regions( $connections_array );
0 ignored issues
show
Bug introduced by
The method add_regions() does not exist on LSX_Schema_Graph_Piece. Did you maybe mean add_reviews()? ( Ignorable by Annotation )

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

182
				/** @scrutinizer ignore-call */ 
183
    $connections_array = $this->add_regions( $connections_array );

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...
183
				$connections_array = $this->add_accommodation( $connections_array );
0 ignored issues
show
Bug introduced by
The method add_accommodation() does not exist on LSX_Schema_Graph_Piece. ( Ignorable by Annotation )

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

183
				/** @scrutinizer ignore-call */ 
184
    $connections_array = $this->add_accommodation( $connections_array );

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...
184
				if ( ! empty( $connections_array ) ) {
0 ignored issues
show
Coding Style introduced by
Expected 0 spaces after opening bracket; 1 found
Loading history...
Coding Style introduced by
Expected 0 spaces before closing bracket; 1 found
Loading history...
185
					$data['containsPlace'] = $connections_array;
186
				}
187
			} else {
188
				$connections_array             = $this->add_countries( $connections_array );
0 ignored issues
show
Coding Style introduced by
Equals sign not aligned with surrounding assignments; expected 8 spaces but found 13 spaces

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...
Bug introduced by
The method add_countries() does not exist on LSX_Schema_Graph_Piece. ( Ignorable by Annotation )

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

188
				/** @scrutinizer ignore-call */ 
189
    $connections_array             = $this->add_countries( $connections_array );

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...
189
				$data['containedInPlace'] = $connections_array;
190
				$connections_array          = array();
0 ignored issues
show
Coding Style introduced by
Equals sign not aligned with surrounding assignments; expected 8 spaces but found 10 spaces

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...
191
				$connections_array          = $this->add_accommodation( $connections_array );
0 ignored issues
show
Coding Style introduced by
Equals sign not aligned with surrounding assignments; expected 8 spaces but found 10 spaces

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...
192
				$data['containsPlace'] = $connections_array;
0 ignored issues
show
Coding Style introduced by
Equals sign not aligned with surrounding assignments; expected 4 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...
193
			}
0 ignored issues
show
Coding Style introduced by
No blank line found after control structure
Loading history...
194
			return $data;
195
		}
0 ignored issues
show
Coding Style introduced by
Expected 2 blank lines after function; 0 found
Loading history...
Coding Style introduced by
Expected 1 blank line before closing function brace; 0 found
Loading history...
196
		/**
197
		 * Adds the terms for the taxonomy
198
		 *
199
		 * @param array $data     Review data.
200
		 * @param array $data_key the parameter name you wish to assign it to.
201
		 * @param array $taxonomy the taxonomy to grab terms for.
202
		 *
203
		 * @return array $data Review data.
204
		 */
205
		public function add_taxonomy_terms( $data, $data_key, $taxonomy ) {
206
			/**
207
			 * Filter: 'lsx_schema_' . $this->post_type . '_' . $data_key . '_taxonomy' - Allow changing the taxonomy used to assign keywords to a post type Review data.
208
			 *
209
			 * @api string $taxonomy The chosen taxonomy.
210
			 */
211
			$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

211
			$taxonomy = apply_filters( 'lsx_schema_' . $this->post_type . '_' . /** @scrutinizer ignore-type */ $data_key . '_taxonomy', $taxonomy );
Loading history...
212
			return LSX_Schema_Utils::add_terms( $data, $this->context->id, $data_key, $taxonomy );
0 ignored issues
show
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

212
			return LSX_Schema_Utils::add_terms( $data, $this->context->id, $data_key, /** @scrutinizer ignore-type */ $taxonomy );
Loading history...
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

212
			return LSX_Schema_Utils::add_terms( $data, $this->context->id, /** @scrutinizer ignore-type */ $data_key, $taxonomy );
Loading history...
213
		}
0 ignored issues
show
Coding Style introduced by
Expected 2 blank lines after function; 0 found
Loading history...
Coding Style introduced by
Expected 1 blank line before closing function brace; 0 found
Loading history...
214
		/**
215
		 * Adds the custom field value for the supplied key
216
		 *
217
		 * @param array   $data     Schema data.
218
		 * @param string  $data_key the parameter name you wish to assign it to.
219
		 * @param string  $meta_key the taxonomy to grab terms for.
220
		 * @param boolean $single   A single custom field or an array
0 ignored issues
show
Coding Style Documentation introduced by
Parameter comment must end with a full stop
Loading history...
221
		 *
222
		 * @return array $data Review data.
223
		 */
224
		public function add_custom_field( $data, $data_key, $meta_key, $single = true ) {
225
			$value = get_post_meta( $this->context->id, $meta_key, $single );
0 ignored issues
show
Bug introduced by
$this->context->id of type string is incompatible with the type integer expected by parameter $post_id of get_post_meta(). ( Ignorable by Annotation )

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

225
			$value = get_post_meta( /** @scrutinizer ignore-type */ $this->context->id, $meta_key, $single );
Loading history...
226
			if ( '' !== $value && false !== $value ) {
0 ignored issues
show
Coding Style introduced by
Expected 0 spaces after opening bracket; 1 found
Loading history...
Coding Style introduced by
Expected 0 spaces before closing bracket; 1 found
Loading history...
227
				$data[ $data_key ] = $value;
228
			}
0 ignored issues
show
Coding Style introduced by
No blank line found after control structure
Loading history...
229
			return $data;
230
		}
0 ignored issues
show
Coding Style introduced by
Expected 2 blank lines after function; 0 found
Loading history...
Coding Style introduced by
Expected 1 blank line before closing function brace; 0 found
Loading history...
231
	}
232
}
233