Test Failed
Push — master ( 653e6c...3b4aec )
by Paul
03:40
created
plugin/Controllers/SettingsController.php 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -16,7 +16,7 @@  discard block
 block discarded – undo
16 16
 	 */
17 17
 	public function callbackRegisterSettings( $input )
18 18
 	{
19
-		if( !is_array( $input )) {
19
+		if( !is_array( $input ) ) {
20 20
 			$input = ['settings' => []];
21 21
 		}
22 22
 		if( key( $input ) == 'settings' ) {
@@ -24,7 +24,7 @@  discard block
 block discarded – undo
24 24
 			$options = $this->sanitizeGeneral( $input, $options );
25 25
 			$options = $this->sanitizeSubmissions( $input, $options );
26 26
 			$options = $this->sanitizeTranslations( $input, $options );
27
-			glsr( Notice::class )->addSuccess( __( 'Settings updated.', 'site-reviews' ));
27
+			glsr( Notice::class )->addSuccess( __( 'Settings updated.', 'site-reviews' ) );
28 28
 			return $options;
29 29
 		}
30 30
 		return $input;
@@ -38,7 +38,7 @@  discard block
 block discarded – undo
38 38
 	{
39 39
 		register_setting( Application::ID.'-settings', OptionManager::databaseKey(), [
40 40
 			'sanitize_callback' => [$this, 'callbackRegisterSettings'],
41
-		]);
41
+		] );
42 42
 	}
43 43
 
44 44
 	/**
@@ -58,7 +58,7 @@  discard block
 block discarded – undo
58 58
 	protected function sanitizeSubmissions( array $input, array $options )
59 59
 	{
60 60
 		$inputForm = $input['settings']['submissions'];
61
-		$options['settings']['submissions']['required'] = isset( $inputForm['required'] )
61
+		$options['settings']['submissions']['required'] = isset($inputForm['required'])
62 62
 			? $inputForm['required']
63 63
 			: [];
64 64
 		return $options;
@@ -69,13 +69,13 @@  discard block
 block discarded – undo
69 69
 	 */
70 70
 	protected function sanitizeTranslations( array $input, array $options )
71 71
 	{
72
-		if( isset( $input['settings']['strings'] )) {
73
-			$options['settings']['strings'] = array_values( array_filter( $input['settings']['strings'] ));
72
+		if( isset($input['settings']['strings']) ) {
73
+			$options['settings']['strings'] = array_values( array_filter( $input['settings']['strings'] ) );
74 74
 			array_walk( $options['settings']['strings'], function( &$string ) {
75
-				if( isset( $string['s2'] )) {
75
+				if( isset($string['s2']) ) {
76 76
 					$string['s2'] = wp_strip_all_tags( $string['s2'] );
77 77
 				}
78
-				if( isset( $string['p2'] )) {
78
+				if( isset($string['p2']) ) {
79 79
 					$string['p2'] = wp_strip_all_tags( $string['p2'] );
80 80
 				}
81 81
 			});
Please login to merge, or discard this patch.
plugin/Modules/Html/Form.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -27,9 +27,9 @@
 block discarded – undo
27 27
 	public function getFields( $id )
28 28
 	{
29 29
 		$fields = [];
30
-		$values = apply_filters( 'site-reviews/form/fields', glsr()->config( 'forms/'.$id ));
30
+		$values = apply_filters( 'site-reviews/form/fields', glsr()->config( 'forms/'.$id ) );
31 31
 		foreach( $values as $name => $field ) {
32
-			$fields[] = new Field( wp_parse_args( $field, ['name' => $name] ));
32
+			$fields[] = new Field( wp_parse_args( $field, ['name' => $name] ) );
33 33
 		}
34 34
 		return $fields;
35 35
 	}
Please login to merge, or discard this patch.
plugin/Modules/Html/Builder.php 2 patches
Spacing   +34 added lines, -34 removed lines patch added patch discarded remove patch
@@ -55,7 +55,7 @@  discard block
 block discarded – undo
55 55
 	{
56 56
 		$instance = new static;
57 57
 		$instance->setTagFromMethod( $method );
58
-		call_user_func_array( [$instance, 'normalize'], $args += ['',''] );
58
+		call_user_func_array( [$instance, 'normalize'], $args += ['', ''] );
59 59
 		$tags = array_merge( static::TAGS_FORM, static::TAGS_SINGLE, static::TAGS_STRUCTURE, static::TAGS_TEXT );
60 60
 		do_action_ref_array( 'site-reviews/builder', [&$instance] );
61 61
 		$generatedTag = in_array( $instance->tag, $tags )
@@ -79,8 +79,8 @@  discard block
 block discarded – undo
79 79
 			'render' => 'is_bool',
80 80
 			'tag' => 'is_string',
81 81
 		];
82
-		if( !isset( $properties[$property] )
83
-			|| empty( array_filter( [$value], $properties[$property] ))
82
+		if( !isset($properties[$property])
83
+			|| empty(array_filter( [$value], $properties[$property] ))
84 84
 		)return;
85 85
 		$this->$property = $value;
86 86
 	}
@@ -90,7 +90,7 @@  discard block
 block discarded – undo
90 90
 	 */
91 91
 	public function getClosingTag()
92 92
 	{
93
-		if( empty( $this->tag ))return;
93
+		if( empty($this->tag) )return;
94 94
 		return '</'.$this->tag.'>';
95 95
 	}
96 96
 
@@ -99,8 +99,8 @@  discard block
 block discarded – undo
99 99
 	 */
100 100
 	public function getOpeningTag()
101 101
 	{
102
-		if( empty( $this->tag ))return;
103
-		$attributes = glsr( Attributes::class )->{$this->tag}( $this->args )->toString();
102
+		if( empty($this->tag) )return;
103
+		$attributes = glsr( Attributes::class )->{$this->tag}($this->args)->toString();
104 104
 		return '<'.trim( $this->tag.' '.$attributes ).'>';
105 105
 	}
106 106
 
@@ -109,8 +109,8 @@  discard block
 block discarded – undo
109 109
 	 */
110 110
 	public function raw( array $field )
111 111
 	{
112
-		unset( $field['label'] );
113
-		return $this->{$field['type']}( $field );
112
+		unset($field['label']);
113
+		return $this->{$field['type']}($field);
114 114
 	}
115 115
 
116 116
 	/**
@@ -119,7 +119,7 @@  discard block
 block discarded – undo
119 119
 	protected function buildCustomField()
120 120
 	{
121 121
 		$className = $this->getCustomFieldClassName();
122
-		if( class_exists( $className )) {
122
+		if( class_exists( $className ) ) {
123 123
 			return (new $className( $this ))->build();
124 124
 		}
125 125
 		glsr_log()->error( 'Field missing: '.$className );
@@ -130,7 +130,7 @@  discard block
 block discarded – undo
130 130
 	 */
131 131
 	protected function buildDefaultTag( $text = '' )
132 132
 	{
133
-		if( empty( $text )) {
133
+		if( empty($text) ) {
134 134
 			$text = $this->args['text'];
135 135
 		}
136 136
 		return $this->getOpeningTag().$text.$this->getClosingTag();
@@ -141,7 +141,7 @@  discard block
 block discarded – undo
141 141
 	 */
142 142
 	protected function buildFieldDescription()
143 143
 	{
144
-		if( empty( $this->args['description'] ))return;
144
+		if( empty($this->args['description']) )return;
145 145
 		if( $this->args['is_widget'] ) {
146 146
 			return $this->small( $this->args['description'] );
147 147
 		}
@@ -153,13 +153,13 @@  discard block
 block discarded – undo
153 153
 	 */
154 154
 	protected function buildFormInput()
155 155
 	{
156
-		if( !in_array( $this->args['type'], ['checkbox', 'radio'] )) {
157
-			if( isset( $this->args['multiple'] )) {
158
-				$this->args['name'].= '[]';
156
+		if( !in_array( $this->args['type'], ['checkbox', 'radio'] ) ) {
157
+			if( isset($this->args['multiple']) ) {
158
+				$this->args['name'] .= '[]';
159 159
 			}
160 160
 			return $this->buildFormLabel().$this->getOpeningTag();
161 161
 		}
162
-		return empty( $this->args['options'] )
162
+		return empty($this->args['options'])
163 163
 			? $this->buildFormInputChoice()
164 164
 			: $this->buildFormInputMultiChoice();
165 165
 	}
@@ -169,13 +169,13 @@  discard block
 block discarded – undo
169 169
 	 */
170 170
 	protected function buildFormInputChoice()
171 171
 	{
172
-		if( !empty( $this->args['text'] )) {
172
+		if( !empty($this->args['text']) ) {
173 173
 			$this->args['label'] = $this->args['text'];
174 174
 		}
175
-		return $this->getOpeningTag().$this->buildFormLabel([
175
+		return $this->getOpeningTag().$this->buildFormLabel( [
176 176
 			'class' => 'glsr-'.$this->args['type'].'-label',
177 177
 			'text' => $this->args['label'].'<span></span>',
178
-		]);
178
+		] );
179 179
 	}
180 180
 
181 181
 	/**
@@ -184,22 +184,22 @@  discard block
 block discarded – undo
184 184
 	protected function buildFormInputMultiChoice()
185 185
 	{
186 186
 		if( $this->args['type'] == 'checkbox' ) {
187
-			$this->args['name'].= '[]';
187
+			$this->args['name'] .= '[]';
188 188
 		}
189 189
 		$index = 0;
190
-		$options = array_reduce( array_keys( $this->args['options'] ), function( $carry, $key ) use( &$index ) {
190
+		$options = array_reduce( array_keys( $this->args['options'] ), function( $carry, $key ) use(&$index) {
191 191
 			return $carry.$this->li( $this->{$this->args['type']}([
192 192
 				'checked' => in_array( $key, (array)$this->args['value'] ),
193 193
 				'id' => $this->args['id'].'-'.$index++,
194 194
 				'name' => $this->args['name'],
195 195
 				'text' => $this->args['options'][$key],
196 196
 				'value' => $key,
197
-			]));
197
+			]) );
198 198
 		});
199 199
 		return $this->ul( $options, [
200 200
 			'class' => $this->args['class'],
201 201
 			'id' => $this->args['id'],
202
-		]);
202
+		] );
203 203
 	}
204 204
 
205 205
 	/**
@@ -207,13 +207,13 @@  discard block
 block discarded – undo
207 207
 	 */
208 208
 	protected function buildFormLabel( array $customArgs = [] )
209 209
 	{
210
-		if( empty( $this->args['label'] ) || $this->args['type'] == 'hidden' )return;
210
+		if( empty($this->args['label']) || $this->args['type'] == 'hidden' )return;
211 211
 		return $this->label( wp_parse_args( $customArgs, [
212 212
 			'for' => $this->args['id'],
213 213
 			'is_public' => $this->args['is_public'],
214 214
 			'text' => $this->args['label'],
215 215
 			'type' => $this->args['type'],
216
-		]));
216
+		] ) );
217 217
 	}
218 218
 
219 219
 	/**
@@ -230,11 +230,11 @@  discard block
 block discarded – undo
230 230
 	protected function buildFormSelectOptions()
231 231
 	{
232 232
 		return array_reduce( array_keys( $this->args['options'] ), function( $carry, $key ) {
233
-			return $carry.$this->option([
233
+			return $carry.$this->option( [
234 234
 				'selected' => $this->args['value'] == $key,
235 235
 				'text' => $this->args['options'][$key],
236 236
 				'value' => $key,
237
-			]);
237
+			] );
238 238
 		});
239 239
 	}
240 240
 
@@ -252,10 +252,10 @@  discard block
 block discarded – undo
252 252
 	protected function buildTag()
253 253
 	{
254 254
 		$this->mergeArgsWithRequiredDefaults();
255
-		if( in_array( $this->tag, static::TAGS_SINGLE )) {
255
+		if( in_array( $this->tag, static::TAGS_SINGLE ) ) {
256 256
 			return $this->getOpeningTag();
257 257
 		}
258
-		if( !in_array( $this->tag, static::TAGS_FORM )) {
258
+		if( !in_array( $this->tag, static::TAGS_FORM ) ) {
259 259
 			return $this->buildDefaultTag();
260 260
 		}
261 261
 		return call_user_func( [$this, 'buildForm'.ucfirst( $this->tag )] ).$this->buildFieldDescription();
@@ -275,7 +275,7 @@  discard block
 block discarded – undo
275 275
 	protected function mergeArgsWithRequiredDefaults()
276 276
 	{
277 277
 		$className = $this->getCustomFieldClassName();
278
-		if( class_exists( $className )) {
278
+		if( class_exists( $className ) ) {
279 279
 			$this->args = array_merge(
280 280
 				wp_parse_args( $this->args, $className::defaults() ),
281 281
 				$className::required()
@@ -290,16 +290,16 @@  discard block
 block discarded – undo
290 290
 	 */
291 291
 	protected function normalize( ...$params )
292 292
 	{
293
-		if( is_string( $params[0] ) || is_numeric( $params[0] )) {
293
+		if( is_string( $params[0] ) || is_numeric( $params[0] ) ) {
294 294
 			$this->setNameOrTextAttributeForTag( $params[0] );
295 295
 		}
296
-		if( is_array( $params[0] )) {
296
+		if( is_array( $params[0] ) ) {
297 297
 			$this->args += $params[0];
298 298
 		}
299
-		else if( is_array( $params[1] )) {
299
+		else if( is_array( $params[1] ) ) {
300 300
 			$this->args += $params[1];
301 301
 		}
302
-		if( !isset( $this->args['is_public'] )) {
302
+		if( !isset($this->args['is_public']) ) {
303 303
 			$this->args['is_public'] = false;
304 304
 		}
305 305
 	}
@@ -323,7 +323,7 @@  discard block
 block discarded – undo
323 323
 	protected function setTagFromMethod( $method )
324 324
 	{
325 325
 		$this->tag = strtolower( $method );
326
-		if( in_array( $this->tag, static::INPUT_TYPES )) {
326
+		if( in_array( $this->tag, static::INPUT_TYPES ) ) {
327 327
 			$this->args['type'] = $this->tag;
328 328
 			$this->tag = 'input';
329 329
 		}
Please login to merge, or discard this patch.
Braces   +15 added lines, -5 removed lines patch added patch discarded remove patch
@@ -81,7 +81,9 @@  discard block
 block discarded – undo
81 81
 		];
82 82
 		if( !isset( $properties[$property] )
83 83
 			|| empty( array_filter( [$value], $properties[$property] ))
84
-		)return;
84
+		) {
85
+			return;
86
+		}
85 87
 		$this->$property = $value;
86 88
 	}
87 89
 
@@ -90,7 +92,9 @@  discard block
 block discarded – undo
90 92
 	 */
91 93
 	public function getClosingTag()
92 94
 	{
93
-		if( empty( $this->tag ))return;
95
+		if( empty( $this->tag )) {
96
+			return;
97
+		}
94 98
 		return '</'.$this->tag.'>';
95 99
 	}
96 100
 
@@ -99,7 +103,9 @@  discard block
 block discarded – undo
99 103
 	 */
100 104
 	public function getOpeningTag()
101 105
 	{
102
-		if( empty( $this->tag ))return;
106
+		if( empty( $this->tag )) {
107
+			return;
108
+		}
103 109
 		$attributes = glsr( Attributes::class )->{$this->tag}( $this->args )->toString();
104 110
 		return '<'.trim( $this->tag.' '.$attributes ).'>';
105 111
 	}
@@ -141,7 +147,9 @@  discard block
 block discarded – undo
141 147
 	 */
142 148
 	protected function buildFieldDescription()
143 149
 	{
144
-		if( empty( $this->args['description'] ))return;
150
+		if( empty( $this->args['description'] )) {
151
+			return;
152
+		}
145 153
 		if( $this->args['is_widget'] ) {
146 154
 			return $this->small( $this->args['description'] );
147 155
 		}
@@ -207,7 +215,9 @@  discard block
 block discarded – undo
207 215
 	 */
208 216
 	protected function buildFormLabel( array $customArgs = [] )
209 217
 	{
210
-		if( empty( $this->args['label'] ) || $this->args['type'] == 'hidden' )return;
218
+		if( empty( $this->args['label'] ) || $this->args['type'] == 'hidden' ) {
219
+			return;
220
+		}
211 221
 		return $this->label( wp_parse_args( $customArgs, [
212 222
 			'for' => $this->args['id'],
213 223
 			'is_public' => $this->args['is_public'],
Please login to merge, or discard this patch.
plugin/Modules/Style.php 2 patches
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -37,12 +37,12 @@  discard block
 block discarded – undo
37 37
 			'templates/form/submit-button',
38 38
 			'templates/reviews-form',
39 39
 		];
40
-		if( !preg_match( '('.implode( '|', $styledViews ).')', $view )) {
40
+		if( !preg_match( '('.implode( '|', $styledViews ).')', $view ) ) {
41 41
 			return $view;
42 42
 		}
43 43
 		$views = $this->generatePossibleViews( $view );
44 44
 		foreach( $views as $possibleView ) {
45
-			if( !file_exists( glsr()->path( 'views/'.$possibleView.'.php' )))continue;
45
+			if( !file_exists( glsr()->path( 'views/'.$possibleView.'.php' ) ) )continue;
46 46
 			return $possibleView;
47 47
 		}
48 48
 		return $view;
@@ -76,7 +76,7 @@  discard block
 block discarded – undo
76 76
 	 */
77 77
 	public function modifyField( Builder $instance )
78 78
 	{
79
-		if( !$this->isPublicInstance( $instance ) || empty( array_filter( $this->styles )))return;
79
+		if( !$this->isPublicInstance( $instance ) || empty(array_filter( $this->styles )) )return;
80 80
 		call_user_func_array( [$this, 'customize'], [&$instance] );
81 81
 	}
82 82
 
@@ -85,9 +85,9 @@  discard block
 block discarded – undo
85 85
 	 */
86 86
 	protected function customize( Builder $instance )
87 87
 	{
88
-		$args = wp_parse_args( $instance->args, array_fill_keys( ['class', 'type'], '' ));
88
+		$args = wp_parse_args( $instance->args, array_fill_keys( ['class', 'type'], '' ) );
89 89
 		$key = $instance->tag.'_'.$args['type'];
90
-		$classes = !isset( $this->styles[$key] )
90
+		$classes = !isset($this->styles[$key])
91 91
 			? $this->styles[$instance->tag]
92 92
 			: $this->styles[$key];
93 93
 		$instance->args['class'] = trim( $args['class'].' '.$classes );
@@ -121,7 +121,7 @@  discard block
 block discarded – undo
121 121
 		$args = wp_parse_args( $instance->args, [
122 122
 			'is_public' => false,
123 123
 			'is_raw' => false,
124
-		]);
124
+		] );
125 125
 		if( is_admin() || !$args['is_public'] || $args['is_raw'] ) {
126 126
 			return false;
127 127
 		}
Please login to merge, or discard this patch.
Braces   +6 added lines, -2 removed lines patch added patch discarded remove patch
@@ -42,7 +42,9 @@  discard block
 block discarded – undo
42 42
 		}
43 43
 		$views = $this->generatePossibleViews( $view );
44 44
 		foreach( $views as $possibleView ) {
45
-			if( !file_exists( glsr()->path( 'views/'.$possibleView.'.php' )))continue;
45
+			if( !file_exists( glsr()->path( 'views/'.$possibleView.'.php' ))) {
46
+				continue;
47
+			}
46 48
 			return $possibleView;
47 49
 		}
48 50
 		return $view;
@@ -76,7 +78,9 @@  discard block
 block discarded – undo
76 78
 	 */
77 79
 	public function modifyField( Builder $instance )
78 80
 	{
79
-		if( !$this->isPublicInstance( $instance ) || empty( array_filter( $this->styles )))return;
81
+		if( !$this->isPublicInstance( $instance ) || empty( array_filter( $this->styles ))) {
82
+			return;
83
+		}
80 84
 		call_user_func_array( [$this, 'customize'], [&$instance] );
81 85
 	}
82 86
 
Please login to merge, or discard this patch.
compatibility.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -9,7 +9,7 @@
 block discarded – undo
9 9
  */
10 10
 add_filter( 'sbp_exclude_defer_scripts', function( $scriptHandles ) {
11 11
 	$scriptHandles[] = 'site-reviews/google-recaptcha';
12
-	return array_keys( array_flip( $scriptHandles ));
12
+	return array_keys( array_flip( $scriptHandles ) );
13 13
 });
14 14
 
15 15
 /**
Please login to merge, or discard this patch.
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -17,6 +17,8 @@
 block discarded – undo
17 17
  * @return void
18 18
  */
19 19
 add_action( 'site-reviews/customize/divi', function( $instance ) {
20
-	if( $instance->tag != 'label' || $instance->args['type'] != 'checkbox' )return;
20
+	if( $instance->tag != 'label' || $instance->args['type'] != 'checkbox' ) {
21
+		return;
22
+	}
21 23
 	$instance->args['text'] = '<i></i>'.$instance->args['text'];
22 24
 });
Please login to merge, or discard this patch.
plugin/Database/SqlQueries.php 1 patch
Spacing   +20 added lines, -20 removed lines patch added patch discarded remove patch
@@ -23,11 +23,11 @@  discard block
 block discarded – undo
23 23
 	 */
24 24
 	public function deleteAllSessions( $sessionCookiePrefix )
25 25
 	{
26
-		return $this->db->query("
26
+		return $this->db->query( "
27 27
 			DELETE
28 28
 			FROM {$this->db->options}
29 29
 			WHERE option_name LIKE '{$sessionCookiePrefix}_%'
30
-		");
30
+		" );
31 31
 	}
32 32
 
33 33
 	/**
@@ -36,11 +36,11 @@  discard block
 block discarded – undo
36 36
 	 */
37 37
 	public function deleteExpiredSessions( $expiredSessions )
38 38
 	{
39
-		return $this->db->query("
39
+		return $this->db->query( "
40 40
 			DELETE
41 41
 			FROM {$this->db->options}
42 42
 			WHERE option_name IN ('{$expiredSessions}')
43
-		");
43
+		" );
44 44
 	}
45 45
 
46 46
 	/**
@@ -49,14 +49,14 @@  discard block
 block discarded – undo
49 49
 	 */
50 50
 	public function getReviewCountsFor( $metaKey )
51 51
 	{
52
-		return (array) $this->db->get_results("
52
+		return (array)$this->db->get_results( "
53 53
 			SELECT m.meta_value AS name, COUNT(*) num_posts
54 54
 			FROM {$this->db->posts} AS p
55 55
 			INNER JOIN {$this->db->postmeta} AS m ON p.ID = m.post_id
56 56
 			WHERE p.post_type = '{$this->postType}'
57 57
 			AND m.meta_key = '{$metaKey}'
58 58
 			GROUP BY name
59
-		");
59
+		" );
60 60
 	}
61 61
 
62 62
 	/**
@@ -66,13 +66,13 @@  discard block
 block discarded – undo
66 66
 	 */
67 67
 	public function getExpiredSessions( $sessionCookiePrefix, $limit )
68 68
 	{
69
-		return $this->db->get_results("
69
+		return $this->db->get_results( "
70 70
 			SELECT option_name AS name, option_value AS expiration
71 71
 			FROM {$this->db->options}
72 72
 			WHERE option_name LIKE '{$sessionCookiePrefix}_expires_%'
73 73
 			ORDER BY option_value ASC
74 74
 			LIMIT 0, {$limit}
75
-		");
75
+		" );
76 76
 	}
77 77
 
78 78
 	/**
@@ -82,7 +82,7 @@  discard block
 block discarded – undo
82 82
 	 */
83 83
 	public function getReviewCounts( $lastPostId = 0, $limit = 500 )
84 84
 	{
85
-		return $this->db->get_results("
85
+		return $this->db->get_results( "
86 86
 			SELECT p.ID, m1.meta_value AS rating, m2.meta_value AS type
87 87
 			FROM {$this->db->posts} AS p
88 88
 			INNER JOIN {$this->db->postmeta} AS m1 ON p.ID = m1.post_id
@@ -103,14 +103,14 @@  discard block
 block discarded – undo
103 103
 	 */
104 104
 	public function getReviewPostId( $metaReviewId )
105 105
 	{
106
-		$postId = $this->db->get_var("
106
+		$postId = $this->db->get_var( "
107 107
 			SELECT p.ID
108 108
 			FROM {$this->db->posts} AS p
109 109
 			INNER JOIN {$this->db->postmeta} AS pm ON p.ID = pm.post_id
110 110
 			WHERE p.post_type = '{$this->postType}'
111 111
 			AND pm.meta_key = 'review_id'
112 112
 			AND pm.meta_value = '{$metaReviewId}'
113
-		");
113
+		" );
114 114
 		return intval( $postId );
115 115
 	}
116 116
 
@@ -120,7 +120,7 @@  discard block
 block discarded – undo
120 120
 	 */
121 121
 	public function getReviewIdsByType( $reviewType )
122 122
 	{
123
-		$query = $this->db->get_col("
123
+		$query = $this->db->get_col( "
124 124
 			SELECT m1.meta_value AS review_id
125 125
 			FROM {$this->db->posts} AS p
126 126
 			INNER JOIN {$this->db->postmeta} AS m1 ON p.ID = m1.post_id
@@ -129,8 +129,8 @@  discard block
 block discarded – undo
129 129
 			AND m1.meta_key = 'review_id'
130 130
 			AND m2.meta_key = 'review_type'
131 131
 			AND m2.meta_value = '{$reviewType}'
132
-		");
133
-		return array_keys( array_flip( $query ));
132
+		" );
133
+		return array_keys( array_flip( $query ) );
134 134
 	}
135 135
 
136 136
 	/**
@@ -141,7 +141,7 @@  discard block
 block discarded – undo
141 141
 	 */
142 142
 	public function getReviewPostCounts( $postId, $lastPostId = 0, $limit = 500 )
143 143
 	{
144
-		return $this->db->get_results("
144
+		return $this->db->get_results( "
145 145
 			SELECT p.ID, m1.meta_value AS rating, m2.meta_value AS type
146 146
 			FROM {$this->db->posts} AS p
147 147
 			INNER JOIN {$this->db->postmeta} AS m1 ON p.ID = m1.post_id
@@ -167,9 +167,9 @@  discard block
 block discarded – undo
167 167
 	public function getReviewRatingsFromIds( array $postIds, $greaterThanId = 0, $limit = 100 )
168 168
 	{
169 169
 		sort( $postIds );
170
-		$postIds = array_slice( $postIds, intval( array_search( $greaterThanId, $postIds )), $limit );
170
+		$postIds = array_slice( $postIds, intval( array_search( $greaterThanId, $postIds ) ), $limit );
171 171
 		$postIds = implode( ',', $postIds );
172
-		return $this->db->get_results("
172
+		return $this->db->get_results( "
173 173
 			SELECT p.ID, m.meta_value AS rating
174 174
 			FROM {$this->db->posts} AS p
175 175
 			INNER JOIN {$this->db->postmeta} AS m ON p.ID = m.post_id
@@ -193,7 +193,7 @@  discard block
 block discarded – undo
193 193
 		$queryBuilder = glsr( QueryBuilder::class );
194 194
 		$key = $queryBuilder->buildSqlOr( $key, "m.meta_key = '%s'" );
195 195
 		$status = $queryBuilder->buildSqlOr( $status, "p.post_status = '%s'" );
196
-		return $this->db->get_col("
196
+		return $this->db->get_col( "
197 197
 			SELECT DISTINCT m.meta_value
198 198
 			FROM {$this->db->postmeta} m
199 199
 			LEFT JOIN {$this->db->posts} p ON p.ID = m.post_id
@@ -201,7 +201,7 @@  discard block
 block discarded – undo
201 201
 			AND ({$key})
202 202
 			AND ({$status})
203 203
 			ORDER BY m.meta_value
204
-		");
204
+		" );
205 205
 	}
206 206
 
207 207
 	/**
@@ -212,7 +212,7 @@  discard block
 block discarded – undo
212 212
 	 */
213 213
 	public function getReviewTermCounts( $termId, $lastPostId = 0, $limit = 500 )
214 214
 	{
215
-		return $this->db->get_results("
215
+		return $this->db->get_results( "
216 216
 			SELECT p.ID, m1.meta_value AS rating, m2.meta_value AS type
217 217
 			FROM {$this->db->posts} AS p
218 218
 			INNER JOIN {$this->db->postmeta} AS m1 ON p.ID = m1.post_id
Please login to merge, or discard this patch.
plugin/Review.php 2 patches
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -62,13 +62,13 @@  discard block
 block discarded – undo
62 62
 			'review_type' => '',
63 63
 		];
64 64
 		$meta = array_filter(
65
-			array_map( 'array_shift', (array)get_post_meta( $post->ID )),
65
+			array_map( 'array_shift', (array)get_post_meta( $post->ID ) ),
66 66
 			'strlen'
67 67
 		);
68
-		$properties = glsr( CreateReviewDefaults::class )->restrict( array_merge( $defaults, $meta ));
68
+		$properties = glsr( CreateReviewDefaults::class )->restrict( array_merge( $defaults, $meta ) );
69 69
 		$this->modified = $this->isModified( $properties );
70 70
 		array_walk( $properties, function( $value, $key ) {
71
-			if( !property_exists( $this, $key ) || isset( $this->$key ))return;
71
+			if( !property_exists( $this, $key ) || isset($this->$key) )return;
72 72
 			$this->$key = $value;
73 73
 		});
74 74
 	}
@@ -79,7 +79,7 @@  discard block
 block discarded – undo
79 79
 	protected function setTermIds( WP_Post $post )
80 80
 	{
81 81
 		$this->term_ids = [];
82
-		if( !is_array( $terms = get_the_terms( $post, Application::TAXONOMY )))return;
82
+		if( !is_array( $terms = get_the_terms( $post, Application::TAXONOMY ) ) )return;
83 83
 		foreach( $terms as $term ) {
84 84
 			$this->term_ids[] = $term->term_id;
85 85
 		}
Please login to merge, or discard this patch.
Braces   +6 added lines, -2 removed lines patch added patch discarded remove patch
@@ -68,7 +68,9 @@  discard block
 block discarded – undo
68 68
 		$properties = glsr( CreateReviewDefaults::class )->restrict( array_merge( $defaults, $meta ));
69 69
 		$this->modified = $this->isModified( $properties );
70 70
 		array_walk( $properties, function( $value, $key ) {
71
-			if( !property_exists( $this, $key ) || isset( $this->$key ))return;
71
+			if( !property_exists( $this, $key ) || isset( $this->$key )) {
72
+				return;
73
+			}
72 74
 			$this->$key = $value;
73 75
 		});
74 76
 	}
@@ -79,7 +81,9 @@  discard block
 block discarded – undo
79 81
 	protected function setTermIds( WP_Post $post )
80 82
 	{
81 83
 		$this->term_ids = [];
82
-		if( !is_array( $terms = get_the_terms( $post, Application::TAXONOMY )))return;
84
+		if( !is_array( $terms = get_the_terms( $post, Application::TAXONOMY ))) {
85
+			return;
86
+		}
83 87
 		foreach( $terms as $term ) {
84 88
 			$this->term_ids[] = $term->term_id;
85 89
 		}
Please login to merge, or discard this patch.
plugin/Database/CountsManager.php 2 patches
Spacing   +24 added lines, -24 removed lines patch added patch discarded remove patch
@@ -60,7 +60,7 @@  discard block
 block discarded – undo
60 60
 			$this->getCounts(),
61 61
 			$review->review_type,
62 62
 			$review->rating
63
-		));
63
+		) );
64 64
 	}
65 65
 
66 66
 	/**
@@ -68,12 +68,12 @@  discard block
 block discarded – undo
68 68
 	 */
69 69
 	public function decreasePostCounts( Review $review )
70 70
 	{
71
-		if( empty( $counts = $this->getPostCounts( $review->assigned_to )))return;
71
+		if( empty($counts = $this->getPostCounts( $review->assigned_to )) )return;
72 72
 		$this->setPostCounts( $review->assigned_to, $this->decreaseRating(
73 73
 			$counts,
74 74
 			$review->review_type,
75 75
 			$review->rating
76
-		));
76
+		) );
77 77
 	}
78 78
 
79 79
 	/**
@@ -82,12 +82,12 @@  discard block
 block discarded – undo
82 82
 	public function decreaseTermCounts( Review $review )
83 83
 	{
84 84
 		foreach( $review->term_ids as $termId ) {
85
-			if( empty( $counts = $this->getTermCounts( $termId )))continue;
85
+			if( empty($counts = $this->getTermCounts( $termId )) )continue;
86 86
 			$this->setTermCounts( $termId, $this->decreaseRating(
87 87
 				$counts,
88 88
 				$review->review_type,
89 89
 				$review->rating
90
-			));
90
+			) );
91 91
 		}
92 92
 	}
93 93
 
@@ -97,8 +97,8 @@  discard block
 block discarded – undo
97 97
 	public function flatten( array $reviewCounts )
98 98
 	{
99 99
 		$counts = [];
100
-		array_walk_recursive( $reviewCounts, function( $num, $index ) use( &$counts ) {
101
-			$counts[$index] = isset( $counts[$index] )
100
+		array_walk_recursive( $reviewCounts, function( $num, $index ) use(&$counts) {
101
+			$counts[$index] = isset($counts[$index])
102 102
 				? $num + $counts[$index]
103 103
 				: $num;
104 104
 		});
@@ -146,14 +146,14 @@  discard block
 block discarded – undo
146 146
 	 */
147 147
 	public function increaseCounts( Review $review )
148 148
 	{
149
-		if( empty( $counts = $this->getCounts() )) {
149
+		if( empty($counts = $this->getCounts()) ) {
150 150
 			$counts = $this->buildCounts();
151 151
 		}
152 152
 		$this->setCounts( $this->increaseRating(
153 153
 			$counts,
154 154
 			$review->review_type,
155 155
 			$review->rating
156
-		));
156
+		) );
157 157
 	}
158 158
 
159 159
 	/**
@@ -161,14 +161,14 @@  discard block
 block discarded – undo
161 161
 	 */
162 162
 	public function increasePostCounts( Review $review )
163 163
 	{
164
-		if( empty( $counts = $this->getPostCounts( $review->assigned_to ))) {
164
+		if( empty($counts = $this->getPostCounts( $review->assigned_to )) ) {
165 165
 			$counts = $this->buildPostCounts( $review->assigned_to );
166 166
 		}
167 167
 		$this->setPostCounts( $review->assigned_to, $this->increaseRating(
168 168
 			$counts,
169 169
 			$review->review_type,
170 170
 			$review->rating
171
-		));
171
+		) );
172 172
 	}
173 173
 
174 174
 	/**
@@ -177,14 +177,14 @@  discard block
 block discarded – undo
177 177
 	public function increaseTermCounts( Review $review )
178 178
 	{
179 179
 		foreach( $review->term_ids as $termId ) {
180
-			if( empty( $counts = $this->getTermCounts( $termId ))) {
180
+			if( empty($counts = $this->getTermCounts( $termId )) ) {
181 181
 				$counts = $this->buildTermCounts( $termId );
182 182
 			}
183 183
 			$this->setTermCounts( $termId, $this->increaseRating(
184 184
 				$counts,
185 185
 				$review->review_type,
186 186
 				$review->rating
187
-			));
187
+			) );
188 188
 		}
189 189
 	}
190 190
 
@@ -222,10 +222,10 @@  discard block
 block discarded – undo
222 222
 	{
223 223
 		$counts = [];
224 224
 		$lastPostId = 0;
225
-		while( $reviews = $this->queryReviews( $args, $lastPostId, $limit )) {
226
-			$types = array_keys( array_flip( array_column( $reviews, 'type' )));
225
+		while( $reviews = $this->queryReviews( $args, $lastPostId, $limit ) ) {
226
+			$types = array_keys( array_flip( array_column( $reviews, 'type' ) ) );
227 227
 			foreach( $types as $type ) {
228
-				if( isset( $counts[$type] ))continue;
228
+				if( isset($counts[$type]) )continue;
229 229
 				$counts[$type] = array_fill_keys( range( 0, Rating::MAX_RATING ), 0 );
230 230
 			}
231 231
 			foreach( $reviews as $review ) {
@@ -243,7 +243,7 @@  discard block
 block discarded – undo
243 243
 	 */
244 244
 	protected function decreaseRating( array $reviewCounts, $type, $rating )
245 245
 	{
246
-		if( isset( $reviewCounts[$type][$rating] )) {
246
+		if( isset($reviewCounts[$type][$rating]) ) {
247 247
 			$reviewCounts[$type][$rating] = max( 0, $reviewCounts[$type][$rating] - 1 );
248 248
 		}
249 249
 		return $reviewCounts;
@@ -256,10 +256,10 @@  discard block
 block discarded – undo
256 256
 	 */
257 257
 	protected function increaseRating( array $reviewCounts, $type, $rating )
258 258
 	{
259
-		if( !array_key_exists( $type, glsr()->reviewTypes )) {
259
+		if( !array_key_exists( $type, glsr()->reviewTypes ) ) {
260 260
 			return $reviewCounts;
261 261
 		}
262
-		if( !array_key_exists( $type, $reviewCounts )) {
262
+		if( !array_key_exists( $type, $reviewCounts ) ) {
263 263
 			$reviewCounts[$type] = [];
264 264
 		}
265 265
 		$reviewCounts = $this->normalize( $reviewCounts );
@@ -274,7 +274,7 @@  discard block
 block discarded – undo
274 274
 	{
275 275
 		foreach( $reviewCounts as &$counts ) {
276 276
 			foreach( range( 0, Rating::MAX_RATING ) as $index ) {
277
-				if( isset( $counts[$index] ))continue;
277
+				if( isset($counts[$index]) )continue;
278 278
 				$counts[$index] = 0;
279 279
 			}
280 280
 			ksort( $counts );
@@ -289,14 +289,14 @@  discard block
 block discarded – undo
289 289
 	 */
290 290
 	protected function queryReviews( array $args = [], $lastPostId, $limit )
291 291
 	{
292
-		$args = wp_parse_args( $args, array_fill_keys( ['post_id', 'term_id'], '' ));
293
-		if( empty( array_filter( $args ))) {
292
+		$args = wp_parse_args( $args, array_fill_keys( ['post_id', 'term_id'], '' ) );
293
+		if( empty(array_filter( $args )) ) {
294 294
 			return glsr( SqlQueries::class )->getReviewCounts( $lastPostId, $limit );
295 295
 		}
296
-		if( !empty( $args['post_id'] )) {
296
+		if( !empty($args['post_id']) ) {
297 297
 			return glsr( SqlQueries::class )->getReviewPostCounts( $args['post_id'], $lastPostId, $limit );
298 298
 		}
299
-		if( !empty( $args['term_id'] )) {
299
+		if( !empty($args['term_id']) ) {
300 300
 			return glsr( SqlQueries::class )->getReviewTermCounts( $args['term_id'], $lastPostId, $limit );
301 301
 		}
302 302
 	}
Please login to merge, or discard this patch.
Braces   +12 added lines, -4 removed lines patch added patch discarded remove patch
@@ -68,7 +68,9 @@  discard block
 block discarded – undo
68 68
 	 */
69 69
 	public function decreasePostCounts( Review $review )
70 70
 	{
71
-		if( empty( $counts = $this->getPostCounts( $review->assigned_to )))return;
71
+		if( empty( $counts = $this->getPostCounts( $review->assigned_to ))) {
72
+			return;
73
+		}
72 74
 		$this->setPostCounts( $review->assigned_to, $this->decreaseRating(
73 75
 			$counts,
74 76
 			$review->review_type,
@@ -82,7 +84,9 @@  discard block
 block discarded – undo
82 84
 	public function decreaseTermCounts( Review $review )
83 85
 	{
84 86
 		foreach( $review->term_ids as $termId ) {
85
-			if( empty( $counts = $this->getTermCounts( $termId )))continue;
87
+			if( empty( $counts = $this->getTermCounts( $termId ))) {
88
+				continue;
89
+			}
86 90
 			$this->setTermCounts( $termId, $this->decreaseRating(
87 91
 				$counts,
88 92
 				$review->review_type,
@@ -225,7 +229,9 @@  discard block
 block discarded – undo
225 229
 		while( $reviews = $this->queryReviews( $args, $lastPostId, $limit )) {
226 230
 			$types = array_keys( array_flip( array_column( $reviews, 'type' )));
227 231
 			foreach( $types as $type ) {
228
-				if( isset( $counts[$type] ))continue;
232
+				if( isset( $counts[$type] )) {
233
+					continue;
234
+				}
229 235
 				$counts[$type] = array_fill_keys( range( 0, Rating::MAX_RATING ), 0 );
230 236
 			}
231 237
 			foreach( $reviews as $review ) {
@@ -274,7 +280,9 @@  discard block
 block discarded – undo
274 280
 	{
275 281
 		foreach( $reviewCounts as &$counts ) {
276 282
 			foreach( range( 0, Rating::MAX_RATING ) as $index ) {
277
-				if( isset( $counts[$index] ))continue;
283
+				if( isset( $counts[$index] )) {
284
+					continue;
285
+				}
278 286
 				$counts[$index] = 0;
279 287
 			}
280 288
 			ksort( $counts );
Please login to merge, or discard this patch.