Passed
Push — master ( fbb7de...43cb65 )
by Paul
25:46 queued 14:45
created
plugin/Blocks/SiteReviewsBlock.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -73,7 +73,7 @@
 block discarded – undo
73 73
 			$this->filterShortcodeClass();
74 74
 			$this->filterShowMoreLinks( 'content' );
75 75
 			$this->filterShowMoreLinks( 'response' );
76
-			if( !$this->hasVisibleFields( $shortcode, $attributes )) {
76
+			if( !$this->hasVisibleFields( $shortcode, $attributes ) ) {
77 77
 				$this->filterInterpolation();
78 78
 			}
79 79
 		}
Please login to merge, or discard this patch.
plugin/Blocks/SiteReviewsFormBlock.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -48,7 +48,7 @@  discard block
 block discarded – undo
48 48
 			$this->filterRatingField();
49 49
 			$this->filterShortcodeClass();
50 50
 			$this->filterSubmitButton();
51
-			if( !$this->hasVisibleFields( $shortcode, $attributes )) {
51
+			if( !$this->hasVisibleFields( $shortcode, $attributes ) ) {
52 52
 				$this->filterInterpolation();
53 53
 			}
54 54
 		}
@@ -91,8 +91,8 @@  discard block
 block discarded – undo
91 91
 		add_filter( 'site-reviews/rendered/field', function( $html, $type, $args ) {
92 92
 			if( $args['path'] == 'rating' ) {
93 93
 				$stars = '<span class="glsr-stars">';
94
-				$stars.= str_repeat( '<span class="glsr-star glsr-star-empty" aria-hidden="true"></span>', 5 );
95
-				$stars.= '</span>';
94
+				$stars .= str_repeat( '<span class="glsr-star glsr-star-empty" aria-hidden="true"></span>', 5 );
95
+				$stars .= '</span>';
96 96
 				$html = preg_replace( '/(.*)(<select.*)(<\/select>)(.*)/', '$1'.$stars.'$4', $html );
97 97
 			}
98 98
 			return $html;
Please login to merge, or discard this patch.
plugin/Blocks/SiteReviewsSummaryBlock.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -58,7 +58,7 @@
 block discarded – undo
58 58
 		if( filter_input( INPUT_GET, 'context' ) == 'edit' ) {
59 59
 			$attributes = $this->normalize( $attributes );
60 60
 			$this->filterShortcodeClass();
61
-			if( !$this->hasVisibleFields( $shortcode, $attributes )) {
61
+			if( !$this->hasVisibleFields( $shortcode, $attributes ) ) {
62 62
 				$this->filterInterpolation();
63 63
 			}
64 64
 		}
Please login to merge, or discard this patch.
plugin/Modules/Html/Partials/SiteReviewsForm.php 2 patches
Spacing   +38 added lines, -38 removed lines patch added patch discarded remove patch
@@ -47,7 +47,7 @@  discard block
 block discarded – undo
47 47
 	public function build( array $args = [] )
48 48
 	{
49 49
 		$this->args = $args;
50
-		if( !is_user_logged_in() && glsr( OptionManager::class )->getBool( 'settings.general.require.login' )) {
50
+		if( !is_user_logged_in() && glsr( OptionManager::class )->getBool( 'settings.general.require.login' ) ) {
51 51
 			return $this->buildLoginRegister();
52 52
 		}
53 53
 		$this->errors = glsr( Session::class )->get( $args['id'].'errors', [], true );
@@ -66,7 +66,7 @@  discard block
 block discarded – undo
66 66
 				'response' => $this->buildResponse(),
67 67
 				'submit_button' => $this->buildSubmitButton().$this->buildRecaptcha(),
68 68
 			],
69
-		]);
69
+		] );
70 70
 	}
71 71
 
72 72
 	/**
@@ -78,7 +78,7 @@  discard block
 block discarded – undo
78 78
 			'context' => [
79 79
 				'text' => trim( $this->getLoginText().' '.$this->getRegisterText() ),
80 80
 			],
81
-		]);
81
+		] );
82 82
 	}
83 83
 
84 84
 	/**
@@ -87,12 +87,12 @@  discard block
 block discarded – undo
87 87
 	protected function buildRecaptcha()
88 88
 	{
89 89
 		if( !glsr( OptionManager::class )->isRecaptchaEnabled() )return;
90
-		return glsr( Builder::class )->div([
90
+		return glsr( Builder::class )->div( [
91 91
 			'class' => 'glsr-recaptcha-holder',
92 92
 			'data-badge' => glsr( OptionManager::class )->get( 'settings.submissions.recaptcha.position' ),
93
-			'data-sitekey' => sanitize_text_field( glsr( OptionManager::class )->get( 'settings.submissions.recaptcha.key' )),
93
+			'data-sitekey' => sanitize_text_field( glsr( OptionManager::class )->get( 'settings.submissions.recaptcha.key' ) ),
94 94
 			'data-size' => 'invisible',
95
-		]);
95
+		] );
96 96
 	}
97 97
 
98 98
 	/**
@@ -100,7 +100,7 @@  discard block
 block discarded – undo
100 100
 	 */
101 101
 	protected function buildResponse()
102 102
 	{
103
-		$classes = !empty( $this->errors )
103
+		$classes = !empty($this->errors)
104 104
 			? glsr( StyleValidationDefaults::class )->defaults()['message_error_class']
105 105
 			: '';
106 106
 		return glsr( Template::class )->build( 'templates/form/response', [
@@ -108,8 +108,8 @@  discard block
 block discarded – undo
108 108
 				'class' => $classes,
109 109
 				'message' => wpautop( $this->message ),
110 110
 			],
111
-			'has_errors' => !empty( $this->errors ),
112
-		]);
111
+			'has_errors' => !empty($this->errors),
112
+		] );
113 113
 	}
114 114
 
115 115
 	/**
@@ -121,7 +121,7 @@  discard block
 block discarded – undo
121 121
 			'context' => [
122 122
 				'text' => __( 'Submit your review', 'site-reviews' ),
123 123
 			],
124
-		]);
124
+		] );
125 125
 	}
126 126
 
127 127
 	/**
@@ -139,14 +139,14 @@  discard block
 block discarded – undo
139 139
 	{
140 140
 		$hiddenFields = $this->getHiddenFields();
141 141
 		$hiddenFields[] = $this->getHoneypotField();
142
-		$fields = $this->normalizeFields( glsr( Form::class )->getFields( 'submission-form' ));
142
+		$fields = $this->normalizeFields( glsr( Form::class )->getFields( 'submission-form' ) );
143 143
 		$paths = array_map( function( $obj ) {
144 144
 			return $obj->field['path'];
145 145
 		}, $hiddenFields );
146 146
 		foreach( $fields as $field ) {
147 147
 			$index = array_search( $field->field['path'], $paths );
148 148
 			if( $index === false )continue;
149
-			unset( $hiddenFields[$index] );
149
+			unset($hiddenFields[$index]);
150 150
 		}
151 151
 		return array_merge( $hiddenFields, $fields );
152 152
 	}
@@ -156,10 +156,10 @@  discard block
 block discarded – undo
156 156
 	 */
157 157
 	protected function getLoginText()
158 158
 	{
159
-		$loginLink = glsr( Builder::class )->a([
160
-			'href' => wp_login_url( strval( get_permalink() )),
159
+		$loginLink = glsr( Builder::class )->a( [
160
+			'href' => wp_login_url( strval( get_permalink() ) ),
161 161
 			'text' => __( 'logged in', 'site-reviews' ),
162
-		]);
162
+		] );
163 163
 		return sprintf( __( 'You must be %s to submit a review.', 'site-reviews' ), $loginLink );
164 164
 	}
165 165
 
@@ -168,11 +168,11 @@  discard block
 block discarded – undo
168 168
 	 */
169 169
 	protected function getRegisterText()
170 170
 	{
171
-		if( !get_option( 'users_can_register' ) || !glsr( OptionManager::class )->getBool( 'settings.general.require.login' ))return;
172
-		$registerLink = glsr( Builder::class )->a([
171
+		if( !get_option( 'users_can_register' ) || !glsr( OptionManager::class )->getBool( 'settings.general.require.login' ) )return;
172
+		$registerLink = glsr( Builder::class )->a( [
173 173
 			'href' => wp_registration_url(),
174 174
 			'text' => __( 'register', 'site-reviews' ),
175
-		]);
175
+		] );
176 176
 		return sprintf( __( 'You may also %s for an account.', 'site-reviews' ), $registerLink );
177 177
 	}
178 178
 
@@ -184,32 +184,32 @@  discard block
 block discarded – undo
184 184
 		$fields = [[
185 185
 			'name' => '_action',
186 186
 			'value' => 'submit-review',
187
-		],[
187
+		], [
188 188
 			'name' => '_counter',
189
-		],[
189
+		], [
190 190
 			'name' => '_nonce',
191 191
 			'value' => wp_create_nonce( 'submit-review' ),
192
-		],[
192
+		], [
193 193
 			'name' => '_post_id',
194 194
 			'value' => get_the_ID(),
195
-		],[
195
+		], [
196 196
 			'name' => '_referer',
197
-			'value' => wp_unslash( filter_input( INPUT_SERVER, 'REQUEST_URI' )),
198
-		],[
197
+			'value' => wp_unslash( filter_input( INPUT_SERVER, 'REQUEST_URI' ) ),
198
+		], [
199 199
 			'name' => 'assign_to',
200 200
 			'value' => $this->args['assign_to'],
201
-		],[
201
+		], [
202 202
 			'name' => 'category',
203 203
 			'value' => $this->args['category'],
204
-		],[
204
+		], [
205 205
 			'name' => 'excluded',
206 206
 			'value' => $this->args['hide'],
207
-		],[
207
+		], [
208 208
 			'name' => 'form_id',
209 209
 			'value' => $this->args['id'],
210 210
 		]];
211 211
 		return array_map( function( $field ) {
212
-			return new Field( wp_parse_args( $field, ['type' => 'hidden'] ));
212
+			return new Field( wp_parse_args( $field, ['type' => 'hidden'] ) );
213 213
 		}, $fields );
214 214
 	}
215 215
 
@@ -218,10 +218,10 @@  discard block
 block discarded – undo
218 218
 	 */
219 219
 	protected function getHoneypotField()
220 220
 	{
221
-		return new Field([
221
+		return new Field( [
222 222
 			'name' => 'gotcha',
223 223
 			'type' => 'honeypot',
224
-		]);
224
+		] );
225 225
 	}
226 226
 
227 227
 	/**
@@ -229,8 +229,8 @@  discard block
 block discarded – undo
229 229
 	 */
230 230
 	protected function normalizeFieldId( Field &$field )
231 231
 	{
232
-		if( empty( $this->args['id'] ) || empty( $field->field['id'] ))return;
233
-		$field->field['id'].= '-'.$this->args['id'];
232
+		if( empty($this->args['id']) || empty($field->field['id']) )return;
233
+		$field->field['id'] .= '-'.$this->args['id'];
234 234
 	}
235 235
 
236 236
 	/**
@@ -238,7 +238,7 @@  discard block
 block discarded – undo
238 238
 	 */
239 239
 	protected function normalizeFieldClass( Field &$field )
240 240
 	{
241
-		if( !isset( $field->field['class'] )) {
241
+		if( !isset($field->field['class']) ) {
242 242
 			$field->field['class'] = '';
243 243
 		}
244 244
 		$field->field['class'] = trim( $field->field['class'].' glsr-field-control' );
@@ -249,7 +249,7 @@  discard block
 block discarded – undo
249 249
 	 */
250 250
 	protected function normalizeFieldErrors( Field &$field )
251 251
 	{
252
-		if( !array_key_exists( $field->field['path'], $this->errors ))return;
252
+		if( !array_key_exists( $field->field['path'], $this->errors ) )return;
253 253
 		$field->field['errors'] = $this->errors[$field->field['path']];
254 254
 	}
255 255
 
@@ -258,7 +258,7 @@  discard block
 block discarded – undo
258 258
 	 */
259 259
 	protected function normalizeFieldRequired( Field &$field )
260 260
 	{
261
-		if( !in_array( $field->field['path'], $this->required ))return;
261
+		if( !in_array( $field->field['path'], $this->required ) )return;
262 262
 		$field->field['required'] = true;
263 263
 	}
264 264
 
@@ -269,7 +269,7 @@  discard block
 block discarded – undo
269 269
 	{
270 270
 		$normalizedFields = [];
271 271
 		foreach( $fields as $field ) {
272
-			if( in_array( $field->field['path'], $this->args['hide'] ))continue;
272
+			if( in_array( $field->field['path'], $this->args['hide'] ) )continue;
273 273
 			$field->field['is_public'] = true;
274 274
 			$this->normalizeFieldClass( $field );
275 275
 			$this->normalizeFieldErrors( $field );
@@ -286,8 +286,8 @@  discard block
 block discarded – undo
286 286
 	 */
287 287
 	protected function normalizeFieldValue( Field &$field )
288 288
 	{
289
-		if( !array_key_exists( $field->field['path'], $this->values ))return;
290
-		if( in_array( $field->field['type'], ['radio', 'checkbox'] )) {
289
+		if( !array_key_exists( $field->field['path'], $this->values ) )return;
290
+		if( in_array( $field->field['type'], ['radio', 'checkbox'] ) ) {
291 291
 			$field->field['checked'] = $field->field['value'] == $this->values[$field->field['path']];
292 292
 		}
293 293
 		else {
Please login to merge, or discard this patch.
Braces   +24 added lines, -8 removed lines patch added patch discarded remove patch
@@ -86,7 +86,9 @@  discard block
 block discarded – undo
86 86
 	 */
87 87
 	protected function buildRecaptcha()
88 88
 	{
89
-		if( !glsr( OptionManager::class )->isRecaptchaEnabled() )return;
89
+		if( !glsr( OptionManager::class )->isRecaptchaEnabled() ) {
90
+			return;
91
+		}
90 92
 		return glsr( Builder::class )->div([
91 93
 			'class' => 'glsr-recaptcha-holder',
92 94
 			'data-badge' => glsr( OptionManager::class )->get( 'settings.submissions.recaptcha.position' ),
@@ -145,7 +147,9 @@  discard block
 block discarded – undo
145 147
 		}, $hiddenFields );
146 148
 		foreach( $fields as $field ) {
147 149
 			$index = array_search( $field->field['path'], $paths );
148
-			if( $index === false )continue;
150
+			if( $index === false ) {
151
+				continue;
152
+			}
149 153
 			unset( $hiddenFields[$index] );
150 154
 		}
151 155
 		return array_merge( $hiddenFields, $fields );
@@ -168,7 +172,9 @@  discard block
 block discarded – undo
168 172
 	 */
169 173
 	protected function getRegisterText()
170 174
 	{
171
-		if( !get_option( 'users_can_register' ) || !glsr( OptionManager::class )->getBool( 'settings.general.require.login' ))return;
175
+		if( !get_option( 'users_can_register' ) || !glsr( OptionManager::class )->getBool( 'settings.general.require.login' )) {
176
+			return;
177
+		}
172 178
 		$registerLink = glsr( Builder::class )->a([
173 179
 			'href' => wp_registration_url(),
174 180
 			'text' => __( 'register', 'site-reviews' ),
@@ -229,7 +235,9 @@  discard block
 block discarded – undo
229 235
 	 */
230 236
 	protected function normalizeFieldId( Field &$field )
231 237
 	{
232
-		if( empty( $this->args['id'] ) || empty( $field->field['id'] ))return;
238
+		if( empty( $this->args['id'] ) || empty( $field->field['id'] )) {
239
+			return;
240
+		}
233 241
 		$field->field['id'].= '-'.$this->args['id'];
234 242
 	}
235 243
 
@@ -249,7 +257,9 @@  discard block
 block discarded – undo
249 257
 	 */
250 258
 	protected function normalizeFieldErrors( Field &$field )
251 259
 	{
252
-		if( !array_key_exists( $field->field['path'], $this->errors ))return;
260
+		if( !array_key_exists( $field->field['path'], $this->errors )) {
261
+			return;
262
+		}
253 263
 		$field->field['errors'] = $this->errors[$field->field['path']];
254 264
 	}
255 265
 
@@ -258,7 +268,9 @@  discard block
 block discarded – undo
258 268
 	 */
259 269
 	protected function normalizeFieldRequired( Field &$field )
260 270
 	{
261
-		if( !in_array( $field->field['path'], $this->required ))return;
271
+		if( !in_array( $field->field['path'], $this->required )) {
272
+			return;
273
+		}
262 274
 		$field->field['required'] = true;
263 275
 	}
264 276
 
@@ -269,7 +281,9 @@  discard block
 block discarded – undo
269 281
 	{
270 282
 		$normalizedFields = [];
271 283
 		foreach( $fields as $field ) {
272
-			if( in_array( $field->field['path'], $this->args['hide'] ))continue;
284
+			if( in_array( $field->field['path'], $this->args['hide'] )) {
285
+				continue;
286
+			}
273 287
 			$field->field['is_public'] = true;
274 288
 			$this->normalizeFieldClass( $field );
275 289
 			$this->normalizeFieldErrors( $field );
@@ -286,7 +300,9 @@  discard block
 block discarded – undo
286 300
 	 */
287 301
 	protected function normalizeFieldValue( Field &$field )
288 302
 	{
289
-		if( !array_key_exists( $field->field['path'], $this->values ))return;
303
+		if( !array_key_exists( $field->field['path'], $this->values )) {
304
+			return;
305
+		}
290 306
 		if( in_array( $field->field['type'], ['radio', 'checkbox'] )) {
291 307
 			$field->field['checked'] = $field->field['value'] == $this->values[$field->field['path']];
292 308
 		}
Please login to merge, or discard this patch.
plugin/Modules/Html/ReviewsHtml.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -31,8 +31,8 @@  discard block
 block discarded – undo
31 31
 		$this->reviews = $reviews;
32 32
 		$this->navigation = glsr( Partial::class )->build( 'pagination', [
33 33
 			'total' => $maxPageCount,
34
-		]);
35
-		parent::__construct( $reviews, ArrayObject::STD_PROP_LIST|ArrayObject::ARRAY_AS_PROPS );
34
+		] );
35
+		parent::__construct( $reviews, ArrayObject::STD_PROP_LIST | ArrayObject::ARRAY_AS_PROPS );
36 36
 	}
37 37
 
38 38
 	/**
@@ -58,7 +58,7 @@  discard block
 block discarded – undo
58 58
 				'navigation' => $this->getNavigation(),
59 59
 				'reviews' => $this->getReviews(),
60 60
 			],
61
-		]);
61
+		] );
62 62
 	}
63 63
 
64 64
 	/**
@@ -73,7 +73,7 @@  discard block
 block discarded – undo
73 73
 			$defaults[] = 'glsr-ajax-pagination';
74 74
 		}
75 75
 		$classes = explode( ' ', $this->args['class'] );
76
-		$classes = array_unique( array_merge( $defaults, $classes ));
76
+		$classes = array_unique( array_merge( $defaults, $classes ) );
77 77
 		return implode( ' ', $classes );
78 78
 	}
79 79
 
@@ -92,7 +92,7 @@  discard block
 block discarded – undo
92 92
 	 */
93 93
 	protected function getReviews()
94 94
 	{
95
-		return empty( $this->reviews )
95
+		return empty($this->reviews)
96 96
 			? $this->getReviewsFallback()
97 97
 			: implode( PHP_EOL, $this->reviews );
98 98
 	}
@@ -102,7 +102,7 @@  discard block
 block discarded – undo
102 102
 	 */
103 103
 	protected function getReviewsFallback()
104 104
 	{
105
-		if( empty( $this->args['fallback'] ) && glsr( OptionManager::class )->getBool( 'settings.reviews.fallback' )) {
105
+		if( empty($this->args['fallback']) && glsr( OptionManager::class )->getBool( 'settings.reviews.fallback' ) ) {
106 106
 			$this->args['fallback'] = __( 'There are no reviews yet. Be the first one to write one.', 'site-reviews' );
107 107
 		}
108 108
 		$fallback = '<p class="glsr-no-margins">'.$this->args['fallback'].'</p>';
Please login to merge, or discard this patch.