Passed
Push — hotfix/fix-counts ( 0ee906...e9420b )
by Paul
03:57
created
plugin/Shortcodes/SiteReviewsPopup.php 1 patch
Spacing   +17 added lines, -17 removed lines patch added patch discarded remove patch
@@ -13,15 +13,15 @@  discard block
 block discarded – undo
13 13
 	public function fields()
14 14
 	{
15 15
 		return [[
16
-			'html' => sprintf( '<p class="strong">%s</p>', esc_html__( 'All settings are optional.', 'site-reviews' )),
16
+			'html' => sprintf( '<p class="strong">%s</p>', esc_html__( 'All settings are optional.', 'site-reviews' ) ),
17 17
 			'minWidth' => 320,
18 18
 			'type' => 'container',
19
-		],[
19
+		], [
20 20
 			'label' => esc_html__( 'Title', 'site-reviews' ),
21 21
 			'name' => 'title',
22 22
 			'tooltip' => __( 'Enter a custom shortcode heading.', 'site-reviews' ),
23 23
 			'type' => 'textbox',
24
-		],[
24
+		], [
25 25
 			'label' => esc_html__( 'Count', 'site-reviews' ),
26 26
 			'maxLength' => 5,
27 27
 			'name' => 'count',
@@ -29,20 +29,20 @@  discard block
 block discarded – undo
29 29
 			'text' => '10',
30 30
 			'tooltip' => __( 'How many reviews would you like to display (default: 10)?', 'site-reviews' ),
31 31
 			'type' => 'textbox',
32
-		],[
32
+		], [
33 33
 			'label' => esc_html__( 'Rating', 'site-reviews' ),
34 34
 			'name' => 'rating',
35 35
 			'options' => [
36
-				'5' => esc_html( sprintf( _n( '%s star', '%s stars', 5, 'site-reviews' ), 5 )),
37
-				'4' => esc_html( sprintf( _n( '%s star', '%s stars', 4, 'site-reviews' ), 4 )),
38
-				'3' => esc_html( sprintf( _n( '%s star', '%s stars', 3, 'site-reviews' ), 3 )),
39
-				'2' => esc_html( sprintf( _n( '%s star', '%s stars', 2, 'site-reviews' ), 2 )),
40
-				'1' => esc_html( sprintf( _n( '%s star', '%s stars', 1, 'site-reviews' ), 1 )),
41
-				'0' => esc_html( __( 'Unrated', 'site-reviews' )),
36
+				'5' => esc_html( sprintf( _n( '%s star', '%s stars', 5, 'site-reviews' ), 5 ) ),
37
+				'4' => esc_html( sprintf( _n( '%s star', '%s stars', 4, 'site-reviews' ), 4 ) ),
38
+				'3' => esc_html( sprintf( _n( '%s star', '%s stars', 3, 'site-reviews' ), 3 ) ),
39
+				'2' => esc_html( sprintf( _n( '%s star', '%s stars', 2, 'site-reviews' ), 2 ) ),
40
+				'1' => esc_html( sprintf( _n( '%s star', '%s stars', 1, 'site-reviews' ), 1 ) ),
41
+				'0' => esc_html( __( 'Unrated', 'site-reviews' ) ),
42 42
 			],
43 43
 			'tooltip' => __( 'What is the minimum rating to display (default: 1 star)?', 'site-reviews' ),
44 44
 			'type' => 'listbox',
45
-		],[
45
+		], [
46 46
 			'label' => esc_html__( 'Pagination', 'site-reviews' ),
47 47
 			'name' => 'pagination',
48 48
 			'options' => [
@@ -53,14 +53,14 @@  discard block
 block discarded – undo
53 53
 			'tooltip' => __( 'When using pagination this shortcode can only be used once on a page. (default: disable)', 'site-reviews' ),
54 54
 			'type' => 'listbox',
55 55
 		],
56
-		$this->getTypes( __( 'Which type of review would you like to display?', 'site-reviews' )),
57
-		$this->getCategories( __( 'Limit reviews to this category.', 'site-reviews' )),
56
+		$this->getTypes( __( 'Which type of review would you like to display?', 'site-reviews' ) ),
57
+		$this->getCategories( __( 'Limit reviews to this category.', 'site-reviews' ) ),
58 58
 		[
59 59
 			'label' => esc_html__( 'Assigned To', 'site-reviews' ),
60 60
 			'name' => 'assigned_to',
61 61
 			'tooltip' => __( 'Limit reviews to those assigned to this post ID (separate multiple IDs with a comma). You can also enter "post_id" to use the ID of the current page, or "parent_id" to use the ID of the parent page.', 'site-reviews' ),
62 62
 			'type' => 'textbox',
63
-		],[
63
+		], [
64 64
 			'label' => esc_html__( 'Schema', 'site-reviews' ),
65 65
 			'name' => 'schema',
66 66
 			'options' => [
@@ -69,19 +69,19 @@  discard block
 block discarded – undo
69 69
 			],
70 70
 			'tooltip' => __( 'Rich snippets are disabled by default.', 'site-reviews' ),
71 71
 			'type' => 'listbox',
72
-		],[
72
+		], [
73 73
 			'label' => esc_html__( 'Classes', 'site-reviews' ),
74 74
 			'name' => 'class',
75 75
 			'tooltip' => __( 'Add custom CSS classes to the shortcode.', 'site-reviews' ),
76 76
 			'type' => 'textbox',
77
-		],[
77
+		], [
78 78
 			'columns' => 2,
79 79
 			'items' => $this->getHideOptions(),
80 80
 			'label' => esc_html__( 'Hide', 'site-reviews' ),
81 81
 			'layout' => 'grid',
82 82
 			'spacing' => 5,
83 83
 			'type' => 'container',
84
-		],[
84
+		], [
85 85
 			'hidden' => true,
86 86
 			'name' => 'id',
87 87
 			'type' => 'textbox',
Please login to merge, or discard this patch.
plugin/Shortcodes/SiteReviewsSummaryPopup.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -12,23 +12,23 @@  discard block
 block discarded – undo
12 12
 	public function fields()
13 13
 	{
14 14
 		return [[
15
-			'html' => sprintf( '<p class="strong">%s</p>', esc_html__( 'All settings are optional.', 'site-reviews' )),
15
+			'html' => sprintf( '<p class="strong">%s</p>', esc_html__( 'All settings are optional.', 'site-reviews' ) ),
16 16
 			'minWidth' => 320,
17 17
 			'type' => 'container',
18
-		],[
18
+		], [
19 19
 			'label' => esc_html__( 'Title', 'site-reviews' ),
20 20
 			'name' => 'title',
21 21
 			'tooltip' => __( 'Enter a custom shortcode heading.', 'site-reviews' ),
22 22
 			'type' => 'textbox',
23 23
 		],
24
-		$this->getTypes( __( 'Which type of review would you like to use?', 'site-reviews' )),
25
-		$this->getCategories( __( 'Limit reviews to this category.', 'site-reviews' )),
24
+		$this->getTypes( __( 'Which type of review would you like to use?', 'site-reviews' ) ),
25
+		$this->getCategories( __( 'Limit reviews to this category.', 'site-reviews' ) ),
26 26
 		[
27 27
 			'label' => esc_html__( 'Assigned To', 'site-reviews' ),
28 28
 			'name' => 'assigned_to',
29 29
 			'tooltip' => __( 'Limit reviews to those assigned to this post ID (separate multiple IDs with a comma). You can also enter "post_id" to use the ID of the current page, or "parent_id" to use the ID of the parent page.', 'site-reviews' ),
30 30
 			'type' => 'textbox',
31
-		],[
31
+		], [
32 32
 			'label' => esc_html__( 'Schema', 'site-reviews' ),
33 33
 			'name' => 'schema',
34 34
 			'options' => [
@@ -37,12 +37,12 @@  discard block
 block discarded – undo
37 37
 			],
38 38
 			'tooltip' => __( 'Rich snippets are disabled by default.', 'site-reviews' ),
39 39
 			'type' => 'listbox',
40
-		],[
40
+		], [
41 41
 			'label' => esc_html__( 'Classes', 'site-reviews' ),
42 42
 			'name' => 'class',
43 43
 			'tooltip' => __( 'Add custom CSS classes to the shortcode.', 'site-reviews' ),
44 44
 			'type' => 'textbox',
45
-		],[
45
+		], [
46 46
 			'columns' => 2,
47 47
 			'items' => $this->getHideOptions(),
48 48
 			'label' => esc_html__( 'Hide', 'site-reviews' ),
Please login to merge, or discard this patch.
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.
site-reviews.php 2 patches
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -18,7 +18,7 @@  discard block
 block discarded – undo
18 18
 
19 19
 defined( 'WPINC' ) || die;
20 20
 
21
-if( !class_exists( 'GL_Plugin_Check_v3' )) {
21
+if( !class_exists( 'GL_Plugin_Check_v3' ) ) {
22 22
 	require_once __DIR__.'/activate.php';
23 23
 }
24 24
 $check = new GL_Plugin_Check_v3( __FILE__ );
@@ -30,7 +30,7 @@  discard block
 block discarded – undo
30 30
 
31 31
 $app = new GeminiLabs\SiteReviews\Application;
32 32
 $app->make( 'Provider' )->register( $app );
33
-register_activation_hook( __FILE__, array( $app, 'activate' ));
34
-register_deactivation_hook( __FILE__, array( $app, 'deactivate' ));
35
-register_shutdown_function( array( $app, 'catchFatalError' ));
33
+register_activation_hook( __FILE__, array( $app, 'activate' ) );
34
+register_deactivation_hook( __FILE__, array( $app, 'deactivate' ) );
35
+register_shutdown_function( array( $app, 'catchFatalError' ) );
36 36
 $app->init();
Please login to merge, or discard this patch.
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -22,7 +22,9 @@
 block discarded – undo
22 22
 	require_once __DIR__.'/activate.php';
23 23
 }
24 24
 $check = new GL_Plugin_Check_v3( __FILE__ );
25
-if( !$check->canProceed() )return;
25
+if( !$check->canProceed() ) {
26
+	return;
27
+}
26 28
 require_once __DIR__.'/autoload.php';
27 29
 require_once __DIR__.'/compatibility.php';
28 30
 require_once __DIR__.'/deprecated.php';
Please login to merge, or discard this patch.
helpers.php 2 patches
Spacing   +12 added lines, -12 removed lines patch added patch discarded remove patch
@@ -20,8 +20,8 @@  discard block
 block discarded – undo
20 20
 		'glsr_log',
21 21
 		'glsr_star_rating',
22 22
 	);
23
-	if( !in_array( $hook, $hooks ) || !function_exists( $hook ))return;
24
-	add_filter( $hook, function() use( $hook, $args ) {
23
+	if( !in_array( $hook, $hooks ) || !function_exists( $hook ) )return;
24
+	add_filter( $hook, function() use($hook, $args) {
25 25
 		array_shift( $args ); // remove the fallback value
26 26
 		return call_user_func_array( $hook, $args );
27 27
 	});
@@ -32,7 +32,7 @@  discard block
 block discarded – undo
32 32
  */
33 33
 function glsr( $alias = null ) {
34 34
 	$app = \GeminiLabs\SiteReviews\Application::load();
35
-	return !empty( $alias )
35
+	return !empty($alias)
36 36
 		? $app->make( $alias )
37 37
 		: $app;
38 38
 }
@@ -46,7 +46,7 @@  discard block
 block discarded – undo
46 46
 	$result = array();
47 47
 	foreach( $array as $subarray ) {
48 48
 		$subarray = (array)$subarray;
49
-		if( !isset( $subarray[$column] ))continue;
49
+		if( !isset($subarray[$column]) )continue;
50 50
 		$result[] = $subarray[$column];
51 51
 	}
52 52
 	return $result;
@@ -57,7 +57,7 @@  discard block
 block discarded – undo
57 57
  */
58 58
 function glsr_calculate_ratings() {
59 59
 	glsr( 'Controllers\AdminController' )->routerCountReviews( false );
60
-	glsr_log()->notice( __( 'Recalculated rating counts.', 'site-reviews' ));
60
+	glsr_log()->notice( __( 'Recalculated rating counts.', 'site-reviews' ) );
61 61
 }
62 62
 
63 63
 /**
@@ -74,10 +74,10 @@  discard block
 block discarded – undo
74 74
  * @return \WP_Screen|object
75 75
  */
76 76
 function glsr_current_screen() {
77
-	if( function_exists( 'get_current_screen' )) {
77
+	if( function_exists( 'get_current_screen' ) ) {
78 78
 		$screen = get_current_screen();
79 79
 	}
80
-	return empty( $screen )
80
+	return empty($screen)
81 81
 		? (object)array_fill_keys( ['base', 'id', 'post_type'], null )
82 82
 		: $screen;
83 83
 }
@@ -134,10 +134,10 @@  discard block
 block discarded – undo
134 134
  */
135 135
 function glsr_get_review( $post_id ) {
136 136
 	$post = null;
137
-	if( is_numeric( $post_id )) {
137
+	if( is_numeric( $post_id ) ) {
138 138
 		$post = get_post( $post_id );
139 139
 	}
140
-	if( !( $post instanceof WP_Post )) {
140
+	if( !($post instanceof WP_Post) ) {
141 141
 		$post = new WP_Post( (object)[] );
142 142
 	}
143 143
 	return glsr( 'Database\ReviewManager' )->single( $post );
@@ -147,7 +147,7 @@  discard block
 block discarded – undo
147 147
  * @return array
148 148
  */
149 149
 function glsr_get_reviews( $args = array() ) {
150
-	return glsr( 'Database\ReviewManager' )->get( glsr( 'Helper' )->consolidateArray( $args ));
150
+	return glsr( 'Database\ReviewManager' )->get( glsr( 'Helper' )->consolidateArray( $args ) );
151 151
 }
152 152
 
153 153
 /**
@@ -157,8 +157,8 @@  discard block
 block discarded – undo
157 157
 	$args = func_get_args();
158 158
 	$console = glsr( 'Modules\Console' );
159 159
 	$value = glsr( 'Helper' )->dataGet( $args, '0' );
160
-	return !empty( $value )
161
-		? $console->log( 'debug', $value, glsr( 'Helper' )->dataGet( $args, '1', [] ))
160
+	return !empty($value)
161
+		? $console->log( 'debug', $value, glsr( 'Helper' )->dataGet( $args, '1', [] ) )
162 162
 		: $console;
163 163
 }
164 164
 
Please login to merge, or discard this patch.
Braces   +32 added lines, -15 removed lines patch added patch discarded remove patch
@@ -20,7 +20,9 @@  discard block
 block discarded – undo
20 20
 		'glsr_log',
21 21
 		'glsr_star_rating',
22 22
 	);
23
-	if( !in_array( $hook, $hooks ) || !function_exists( $hook ))return;
23
+	if( !in_array( $hook, $hooks ) || !function_exists( $hook )) {
24
+		return;
25
+	}
24 26
 	add_filter( $hook, function() use( $hook, $args ) {
25 27
 		array_shift( $args ); // remove the fallback value
26 28
 		return call_user_func_array( $hook, $args );
@@ -30,7 +32,8 @@  discard block
 block discarded – undo
30 32
 /**
31 33
  * @return mixed
32 34
  */
33
-function glsr( $alias = null ) {
35
+function glsr( $alias = null )
36
+{
34 37
 	$app = \GeminiLabs\SiteReviews\Application::load();
35 38
 	return !empty( $alias )
36 39
 		? $app->make( $alias )
@@ -42,11 +45,14 @@  discard block
 block discarded – undo
42 45
  * @param $column string
43 46
  * @return array
44 47
  */
45
-function glsr_array_column( array $array, $column ) {
48
+function glsr_array_column( array $array, $column )
49
+{
46 50
 	$result = array();
47 51
 	foreach( $array as $subarray ) {
48 52
 		$subarray = (array)$subarray;
49
-		if( !isset( $subarray[$column] ))continue;
53
+		if( !isset( $subarray[$column] )) {
54
+			continue;
55
+		}
50 56
 		$result[] = $subarray[$column];
51 57
 	}
52 58
 	return $result;
@@ -55,7 +61,8 @@  discard block
 block discarded – undo
55 61
 /**
56 62
  * @return void
57 63
  */
58
-function glsr_calculate_ratings() {
64
+function glsr_calculate_ratings()
65
+{
59 66
 	glsr( 'Controllers\AdminController' )->routerCountReviews( false );
60 67
 	glsr_log()->notice( __( 'Recalculated rating counts.', 'site-reviews' ));
61 68
 }
@@ -63,7 +70,8 @@  discard block
 block discarded – undo
63 70
 /**
64 71
  * @return \GeminiLabs\SiteReviews\Review|false
65 72
  */
66
-function glsr_create_review( $reviewValues = array() ) {
73
+function glsr_create_review( $reviewValues = array() )
74
+{
67 75
 	$review = new \GeminiLabs\SiteReviews\Commands\CreateReview(
68 76
 		glsr( 'Helper' )->consolidateArray( $reviewValues )
69 77
 	);
@@ -73,7 +81,8 @@  discard block
 block discarded – undo
73 81
 /**
74 82
  * @return \WP_Screen|object
75 83
  */
76
-function glsr_current_screen() {
84
+function glsr_current_screen()
85
+{
77 86
 	if( function_exists( 'get_current_screen' )) {
78 87
 		$screen = get_current_screen();
79 88
 	}
@@ -86,7 +95,8 @@  discard block
 block discarded – undo
86 95
  * @param mixed ...$vars
87 96
  * @return void
88 97
  */
89
-function glsr_debug( ...$vars ) {
98
+function glsr_debug( ...$vars )
99
+{
90 100
 	if( count( $vars ) == 1 ) {
91 101
 		$value = htmlspecialchars( print_r( $vars[0], true ), ENT_QUOTES, 'UTF-8' );
92 102
 		printf( '<div class="glsr-debug"><pre>%s</pre></div>', $value );
@@ -106,7 +116,8 @@  discard block
 block discarded – undo
106 116
  * @param mixed $fallback
107 117
  * @return mixed
108 118
  */
109
-function glsr_get( $array, $path = '', $fallback = '' ) {
119
+function glsr_get( $array, $path = '', $fallback = '' )
120
+{
110 121
 	return glsr( 'Helper' )->dataGet( $array, $path, $fallback );
111 122
 }
112 123
 
@@ -115,7 +126,8 @@  discard block
 block discarded – undo
115 126
  * @param mixed $fallback
116 127
  * @return string|array
117 128
  */
118
-function glsr_get_option( $path = '', $fallback = '' ) {
129
+function glsr_get_option( $path = '', $fallback = '' )
130
+{
119 131
 	return is_string( $path )
120 132
 		? glsr( 'Database\OptionManager' )->get( 'settings.'.$path, $fallback )
121 133
 		: $fallback;
@@ -124,7 +136,8 @@  discard block
 block discarded – undo
124 136
 /**
125 137
  * @return array
126 138
  */
127
-function glsr_get_options() {
139
+function glsr_get_options()
140
+{
128 141
 	return glsr( 'Database\OptionManager' )->get( 'settings' );
129 142
 }
130 143
 
@@ -132,7 +145,8 @@  discard block
 block discarded – undo
132 145
  * @param int $post_id
133 146
  * @return \GeminiLabs\SiteReviews\Review
134 147
  */
135
-function glsr_get_review( $post_id ) {
148
+function glsr_get_review( $post_id )
149
+{
136 150
 	$post = null;
137 151
 	if( is_numeric( $post_id )) {
138 152
 		$post = get_post( $post_id );
@@ -146,14 +160,16 @@  discard block
 block discarded – undo
146 160
 /**
147 161
  * @return array
148 162
  */
149
-function glsr_get_reviews( $args = array() ) {
163
+function glsr_get_reviews( $args = array() )
164
+{
150 165
 	return glsr( 'Database\ReviewManager' )->get( glsr( 'Helper' )->consolidateArray( $args ));
151 166
 }
152 167
 
153 168
 /**
154 169
  * @return \GeminiLabs\SiteReviews\Modules\Console
155 170
  */
156
-function glsr_log() {
171
+function glsr_log()
172
+{
157 173
 	$args = func_get_args();
158 174
 	$console = glsr( 'Modules\Console' );
159 175
 	$value = glsr( 'Helper' )->dataGet( $args, '0' );
@@ -165,6 +181,7 @@  discard block
 block discarded – undo
165 181
 /**
166 182
  * @return string
167 183
  */
168
-function glsr_star_rating( $rating ) {
184
+function glsr_star_rating( $rating )
185
+{
169 186
 	return glsr( 'Modules\Html\Partial' )->build( 'star-rating', ['rating' => $rating] );
170 187
 }
Please login to merge, or discard this patch.