Passed
Push — master ( 32d871...06bde6 )
by Paul
06:00
created
plugin/Modules/Html/Partials/SiteReviews.php 2 patches
Braces   +33 added lines, -11 removed lines patch added patch discarded remove patch
@@ -83,7 +83,9 @@  discard block
 block discarded – undo
83 83
 		$generatedReview = [];
84 84
 		foreach( $review as $key => $value ) {
85 85
 			$method = glsr( Helper::class )->buildMethodName( $key, 'buildOption' );
86
-			if( !method_exists( $this, $method ))continue;
86
+			if( !method_exists( $this, $method )) {
87
+				continue;
88
+			}
87 89
 			$generatedReview[$key] = $this->$method( $key, $value );
88 90
 		}
89 91
 		return (object)$generatedReview;
@@ -96,9 +98,13 @@  discard block
 block discarded – undo
96 98
 	 */
97 99
 	protected function buildOptionAssignedTo( $key, $value )
98 100
 	{
99
-		if( $this->isHiddenOrEmpty( $key, 'settings.reviews.assigned_links.enabled' ))return;
101
+		if( $this->isHiddenOrEmpty( $key, 'settings.reviews.assigned_links.enabled' )) {
102
+			return;
103
+		}
100 104
 		$post = get_post( intval( $value ));
101
-		if( !( $post instanceof WP_Post ))return;
105
+		if( !( $post instanceof WP_Post )) {
106
+			return;
107
+		}
102 108
 		$permalink = glsr( Builder::class )->a( get_the_title( $post->ID ), [
103 109
 			'href' => get_the_permalink( $post->ID ),
104 110
 		]);
@@ -113,7 +119,9 @@  discard block
 block discarded – undo
113 119
 	 */
114 120
 	protected function buildOptionAuthor( $key, $value )
115 121
 	{
116
-		if( $this->isHidden( $key ))return;
122
+		if( $this->isHidden( $key )) {
123
+			return;
124
+		}
117 125
 		$prefix = !$this->isOptionEnabled( 'settings.reviews.avatars.enabled' )
118 126
 			? apply_filters( 'site-reviews/review/author/prefix', '—' )
119 127
 			: '';
@@ -127,7 +135,9 @@  discard block
 block discarded – undo
127 135
 	 */
128 136
 	protected function buildOptionAvatar( $key, $value )
129 137
 	{
130
-		if( $this->isHidden( $key, 'settings.reviews.avatars.enabled' ))return;
138
+		if( $this->isHidden( $key, 'settings.reviews.avatars.enabled' )) {
139
+			return;
140
+		}
131 141
 		$size = $this->getOption( 'settings.reviews.avatars.size', 40 );
132 142
 		return $this->wrap( $key, glsr( Builder::class )->img([
133 143
 			'src' => $this->generateAvatar( $value ),
@@ -144,7 +154,9 @@  discard block
 block discarded – undo
144 154
 	protected function buildOptionContent( $key, $value )
145 155
 	{
146 156
 		$text = $this->normalizeText( $value );
147
-		if( $this->isHiddenOrEmpty( $key, $text ))return;
157
+		if( $this->isHiddenOrEmpty( $key, $text )) {
158
+			return;
159
+		}
148 160
 		return $this->wrap( $key, '<p>'.$text.'</p>' );
149 161
 	}
150 162
 
@@ -155,7 +167,9 @@  discard block
 block discarded – undo
155 167
 	 */
156 168
 	protected function buildOptionDate( $key, $value )
157 169
 	{
158
-		if( $this->isHidden( $key ))return;
170
+		if( $this->isHidden( $key )) {
171
+			return;
172
+		}
159 173
 		$dateFormat = $this->getOption( 'settings.reviews.date.format', 'default' );
160 174
 		if( $dateFormat == 'relative' ) {
161 175
 			$date = glsr( Date::class )->relative( $value );
@@ -176,7 +190,9 @@  discard block
 block discarded – undo
176 190
 	 */
177 191
 	protected function buildOptionRating( $key, $value )
178 192
 	{
179
-		if( $this->isHiddenOrEmpty( $key, $value ))return;
193
+		if( $this->isHiddenOrEmpty( $key, $value )) {
194
+			return;
195
+		}
180 196
 		$rating = glsr( Html::class )->buildPartial( 'star-rating', [
181 197
 			'rating' => $value,
182 198
 		]);
@@ -190,7 +206,9 @@  discard block
 block discarded – undo
190 206
 	 */
191 207
 	protected function buildOptionResponse( $key, $value )
192 208
 	{
193
-		if( $this->isHiddenOrEmpty( $key, $value ))return;
209
+		if( $this->isHiddenOrEmpty( $key, $value )) {
210
+			return;
211
+		}
194 212
 		$title = sprintf( __( 'Response from %s', 'site-reviews' ), get_bloginfo( 'name' ));
195 213
 		$text = $this->normalizeText( $value );
196 214
 		$text = '<p><strong>'.$title.'</strong></p><p>'.$text.'</p>';
@@ -207,7 +225,9 @@  discard block
 block discarded – undo
207 225
 	 */
208 226
 	protected function buildOptionTitle( $key, $value )
209 227
 	{
210
-		if( $this->isHidden( $key ))return;
228
+		if( $this->isHidden( $key )) {
229
+			return;
230
+		}
211 231
 		if( empty( $value )) {
212 232
 			$value = __( 'No Title', 'site-reviews' );
213 233
 		}
@@ -237,7 +257,9 @@  discard block
 block discarded – undo
237 257
 	 */
238 258
 	protected function generateSchema()
239 259
 	{
240
-		if( !wp_validate_boolean( $this->args['schema'] ))return;
260
+		if( !wp_validate_boolean( $this->args['schema'] )) {
261
+			return;
262
+		}
241 263
 		glsr( Schema::class )->store(
242 264
 			glsr( Schema::class )->build( $this->args )
243 265
 		);
Please login to merge, or discard this patch.
Spacing   +30 added lines, -30 removed lines patch added patch discarded remove patch
@@ -55,7 +55,7 @@  discard block
 block discarded – undo
55 55
 				'navigation' => $navigation,
56 56
 			],
57 57
 			'reviews' => $this->buildReviews(),
58
-		]);
58
+		] );
59 59
 	}
60 60
 
61 61
 	/**
@@ -83,7 +83,7 @@  discard block
 block discarded – undo
83 83
 		$generatedReview = [];
84 84
 		foreach( $review as $key => $value ) {
85 85
 			$method = glsr( Helper::class )->buildMethodName( $key, 'buildOption' );
86
-			if( !method_exists( $this, $method ))continue;
86
+			if( !method_exists( $this, $method ) )continue;
87 87
 			$generatedReview[$key] = $this->$method( $key, $value );
88 88
 		}
89 89
 		return (object)$generatedReview;
@@ -96,12 +96,12 @@  discard block
 block discarded – undo
96 96
 	 */
97 97
 	protected function buildOptionAssignedTo( $key, $value )
98 98
 	{
99
-		if( $this->isHiddenOrEmpty( $key, 'settings.reviews.assigned_links.enabled' ))return;
100
-		$post = get_post( intval( $value ));
101
-		if( !( $post instanceof WP_Post ))return;
99
+		if( $this->isHiddenOrEmpty( $key, 'settings.reviews.assigned_links.enabled' ) )return;
100
+		$post = get_post( intval( $value ) );
101
+		if( !($post instanceof WP_Post) )return;
102 102
 		$permalink = glsr( Builder::class )->a( get_the_title( $post->ID ), [
103 103
 			'href' => get_the_permalink( $post->ID ),
104
-		]);
104
+		] );
105 105
 		$assignedTo = sprintf( __( 'Review of %s', 'site-reviews' ), $permalink );
106 106
 		return $this->wrap( $key, '<span>'.$assignedTo.'</span>' );
107 107
 	}
@@ -113,7 +113,7 @@  discard block
 block discarded – undo
113 113
 	 */
114 114
 	protected function buildOptionAuthor( $key, $value )
115 115
 	{
116
-		if( $this->isHidden( $key ))return;
116
+		if( $this->isHidden( $key ) )return;
117 117
 		$prefix = !$this->isOptionEnabled( 'settings.reviews.avatars.enabled' )
118 118
 			? apply_filters( 'site-reviews/review/author/prefix', '&mdash;' )
119 119
 			: '';
@@ -127,13 +127,13 @@  discard block
 block discarded – undo
127 127
 	 */
128 128
 	protected function buildOptionAvatar( $key, $value )
129 129
 	{
130
-		if( $this->isHidden( $key, 'settings.reviews.avatars.enabled' ))return;
130
+		if( $this->isHidden( $key, 'settings.reviews.avatars.enabled' ) )return;
131 131
 		$size = $this->getOption( 'settings.reviews.avatars.size', 40 );
132
-		return $this->wrap( $key, glsr( Builder::class )->img([
132
+		return $this->wrap( $key, glsr( Builder::class )->img( [
133 133
 			'src' => $this->generateAvatar( $value ),
134 134
 			'height' => $size,
135 135
 			'width' => $size,
136
-		]));
136
+		] ) );
137 137
 	}
138 138
 
139 139
 	/**
@@ -144,7 +144,7 @@  discard block
 block discarded – undo
144 144
 	protected function buildOptionContent( $key, $value )
145 145
 	{
146 146
 		$text = $this->normalizeText( $value );
147
-		if( $this->isHiddenOrEmpty( $key, $text ))return;
147
+		if( $this->isHiddenOrEmpty( $key, $text ) )return;
148 148
 		return $this->wrap( $key, '<p>'.$text.'</p>' );
149 149
 	}
150 150
 
@@ -155,7 +155,7 @@  discard block
 block discarded – undo
155 155
 	 */
156 156
 	protected function buildOptionDate( $key, $value )
157 157
 	{
158
-		if( $this->isHidden( $key ))return;
158
+		if( $this->isHidden( $key ) )return;
159 159
 		$dateFormat = $this->getOption( 'settings.reviews.date.format', 'default' );
160 160
 		if( $dateFormat == 'relative' ) {
161 161
 			$date = glsr( Date::class )->relative( $value );
@@ -164,7 +164,7 @@  discard block
 block discarded – undo
164 164
 			$format = $dateFormat == 'custom'
165 165
 				? $this->getOption( 'settings.reviews.date.custom', 'M j, Y' )
166 166
 				: (string)get_option( 'date_format' );
167
-			$date = date_i18n( $format, strtotime( $value ));
167
+			$date = date_i18n( $format, strtotime( $value ) );
168 168
 		}
169 169
 		return $this->wrap( $key, '<span>'.$date.'</span>' );
170 170
 	}
@@ -176,10 +176,10 @@  discard block
 block discarded – undo
176 176
 	 */
177 177
 	protected function buildOptionRating( $key, $value )
178 178
 	{
179
-		if( $this->isHiddenOrEmpty( $key, $value ))return;
179
+		if( $this->isHiddenOrEmpty( $key, $value ) )return;
180 180
 		$rating = glsr( Html::class )->buildPartial( 'star-rating', [
181 181
 			'rating' => $value,
182
-		]);
182
+		] );
183 183
 		return $this->wrap( $key, $rating );
184 184
 	}
185 185
 
@@ -190,8 +190,8 @@  discard block
 block discarded – undo
190 190
 	 */
191 191
 	protected function buildOptionResponse( $key, $value )
192 192
 	{
193
-		if( $this->isHiddenOrEmpty( $key, $value ))return;
194
-		$title = sprintf( __( 'Response from %s', 'site-reviews' ), get_bloginfo( 'name' ));
193
+		if( $this->isHiddenOrEmpty( $key, $value ) )return;
194
+		$title = sprintf( __( 'Response from %s', 'site-reviews' ), get_bloginfo( 'name' ) );
195 195
 		$text = $this->normalizeText( $value );
196 196
 		$text = '<p><strong>'.$title.'</strong></p><p>'.$text.'</p>';
197 197
 		return $this->wrap( $key,
@@ -207,8 +207,8 @@  discard block
 block discarded – undo
207 207
 	 */
208 208
 	protected function buildOptionTitle( $key, $value )
209 209
 	{
210
-		if( $this->isHidden( $key ))return;
211
-		if( empty( $value )) {
210
+		if( $this->isHidden( $key ) )return;
211
+		if( empty($value) ) {
212 212
 			$value = __( 'No Title', 'site-reviews' );
213 213
 		}
214 214
 		return $this->wrap( $key, '<h3>'.$value.'</h3>' );
@@ -226,7 +226,7 @@  discard block
 block discarded – undo
226 226
 			return $avatarUrl;
227 227
 		}
228 228
 		$authorIdOrEmail = get_the_author_meta( 'ID', $review->user_id );
229
-		if( empty( $authorIdOrEmail )) {
229
+		if( empty($authorIdOrEmail) ) {
230 230
 			$authorIdOrEmail = $review->email;
231 231
 		}
232 232
 		return (string)get_avatar_url( $authorIdOrEmail );
@@ -237,7 +237,7 @@  discard block
 block discarded – undo
237 237
 	 */
238 238
 	protected function generateSchema()
239 239
 	{
240
-		if( !wp_validate_boolean( $this->args['schema'] ))return;
240
+		if( !wp_validate_boolean( $this->args['schema'] ) )return;
241 241
 		glsr( Schema::class )->store(
242 242
 			glsr( Schema::class )->build( $this->args )
243 243
 		);
@@ -263,14 +263,14 @@  discard block
 block discarded – undo
263 263
 	{
264 264
 		$limit = $this->getOption( 'settings.reviews.excerpt.length', 55 );
265 265
 		if( str_word_count( $text, 0 ) > $limit ) {
266
-			$words = array_keys( str_word_count( $text, 2 ));
267
-			$excerpt = ltrim( substr( $text, 0, $words[$limit] ));
268
-			$hiddenText = substr( $text, strlen( $excerpt ));
266
+			$words = array_keys( str_word_count( $text, 2 ) );
267
+			$excerpt = ltrim( substr( $text, 0, $words[$limit] ) );
268
+			$hiddenText = substr( $text, strlen( $excerpt ) );
269 269
 			$showMore = glsr( Builder::class )->span( $hiddenText, [
270 270
 				'class' => 'glsr-hidden glsr-hidden-text',
271 271
 				'data-show-less' => __( 'Show less', 'site-reviews' ),
272 272
 				'data-show-more' => __( 'Show more', 'site-reviews' ),
273
-			]);
273
+			] );
274 274
 			$text = $excerpt.$showMore;
275 275
 		}
276 276
 		return nl2br( $text );
@@ -283,7 +283,7 @@  discard block
 block discarded – undo
283 283
 	 */
284 284
 	protected function getOption( $path, $fallback = '' )
285 285
 	{
286
-		if( array_key_exists( $path, $this->options )) {
286
+		if( array_key_exists( $path, $this->options ) ) {
287 287
 			return $this->options[$path];
288 288
 		}
289 289
 		return $fallback;
@@ -296,7 +296,7 @@  discard block
 block discarded – undo
296 296
 	 */
297 297
 	protected function isHidden( $key, $path = '' )
298 298
 	{
299
-		$isOptionEnabled = !empty( $path )
299
+		$isOptionEnabled = !empty($path)
300 300
 			? $this->isOptionEnabled( $path )
301 301
 			: true;
302 302
 		return in_array( $key, $this->args['hide'] ) || !$isOptionEnabled;
@@ -309,7 +309,7 @@  discard block
 block discarded – undo
309 309
 	 */
310 310
 	protected function isHiddenOrEmpty( $key, $value )
311 311
 	{
312
-		return $this->isHidden( $key ) || empty( $value );
312
+		return $this->isHidden( $key ) || empty($value);
313 313
 	}
314 314
 
315 315
 	/**
@@ -328,7 +328,7 @@  discard block
 block discarded – undo
328 328
 	protected function normalizeText( $text )
329 329
 	{
330 330
 		$text = wp_kses( $text, wp_kses_allowed_html() );
331
-		$text = convert_smilies( wptexturize( strip_shortcodes( $text )));
331
+		$text = convert_smilies( wptexturize( strip_shortcodes( $text ) ) );
332 332
 		$text = str_replace( ']]>', ']]&gt;', $text );
333 333
 		$text = preg_replace( '/(\R){2,}/', '$1', $text );
334 334
 		return $this->isOptionEnabled( 'settings.reviews.excerpt.enabled' )
@@ -345,6 +345,6 @@  discard block
 block discarded – undo
345 345
 	{
346 346
 		return glsr( Builder::class )->div( $value, [
347 347
 			'class' => 'glsr-review-'.$key,
348
-		]);
348
+		] );
349 349
 	}
350 350
 }
Please login to merge, or discard this patch.
plugin/Modules/Html/Partials/SiteReviewsForm.php 1 patch
Spacing   +11 added lines, -11 removed lines patch added patch discarded remove patch
@@ -30,7 +30,7 @@  discard block
 block discarded – undo
30 30
 				'submit_button' => $this->buildSubmitButton(),
31 31
 			],
32 32
 			'fields' => $this->getFields(),
33
-		]);
33
+		] );
34 34
 	}
35 35
 
36 36
 	/**
@@ -48,7 +48,7 @@  discard block
 block discarded – undo
48 48
 	{
49 49
 		return glsr( Builder::class )->button( '<span></span>'.__( 'Submit your review', 'site-reviews' ), [
50 50
 			'type' => 'submit',
51
-		]);
51
+		] );
52 52
 	}
53 53
 
54 54
 	/**
@@ -82,25 +82,25 @@  discard block
 block discarded – undo
82 82
 		$fields = [[
83 83
 			'name' => 'assign_to',
84 84
 			'value' => $this->args['assign_to'],
85
-		],[
85
+		], [
86 86
 			'name' => 'category',
87 87
 			'value' => $this->args['category'],
88
-		],[
88
+		], [
89 89
 			'name' => 'excluded',
90 90
 			'value' => $this->args['excluded'],
91
-		],[
91
+		], [
92 92
 			'name' => 'id',
93 93
 			'value' => $this->args['id'],
94
-		],[
94
+		], [
95 95
 			'name' => '_wp_http_referer',
96 96
 			'value' => wp_get_referer(),
97
-		],[
97
+		], [
98 98
 			'name' => '_wpnonce',
99 99
 			'value' => wp_create_nonce( 'post-review' ),
100 100
 		]];
101 101
 		return array_values( array_map( function( $field ) {
102
-			return new Field( wp_parse_args( $field, ['type' => 'hidden'] ));
103
-		}, $fields ));
102
+			return new Field( wp_parse_args( $field, ['type' => 'hidden'] ) );
103
+		}, $fields ) );
104 104
 	}
105 105
 
106 106
 	/**
@@ -108,9 +108,9 @@  discard block
 block discarded – undo
108 108
 	 */
109 109
 	protected function getHoneypotField()
110 110
 	{
111
-		return new Field([
111
+		return new Field( [
112 112
 			'name' => 'gotcha',
113 113
 			'type' => 'honeypot',
114
-		]);
114
+		] );
115 115
 	}
116 116
 }
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,13 +27,13 @@
 block discarded – undo
27 27
 	public function getFields( $id )
28 28
 	{
29 29
 		$fields = [];
30
-		$configPath = glsr()->path( 'config/'.$id.'.php' );;
30
+		$configPath = glsr()->path( 'config/'.$id.'.php' ); ;
31 31
 		$values = file_exists( $configPath )
32 32
 			? include $configPath
33 33
 			: [];
34 34
 		$values = apply_filters( 'site-reviews/form/fields', $values );
35 35
 		foreach( $values as $name => $field ) {
36
-			$fields[] = new Field( wp_parse_args( $field, ['name' => $name] ));
36
+			$fields[] = new Field( wp_parse_args( $field, ['name' => $name] ) );
37 37
 		}
38 38
 		return $fields;
39 39
 	}
Please login to merge, or discard this patch.
plugin/Modules/Html/Settings.php 1 patch
Spacing   +14 added lines, -14 removed lines patch added patch discarded remove patch
@@ -23,7 +23,7 @@  discard block
 block discarded – undo
23 23
 			: $this->$method( $id );
24 24
 		return glsr( Template::class )->build( 'pages/settings/'.$id, [
25 25
 			'context' => $context,
26
-		]);
26
+		] );
27 27
 	}
28 28
 
29 29
 	/**
@@ -31,7 +31,7 @@  discard block
 block discarded – undo
31 31
 	 */
32 32
 	protected function getFieldDefault( array $field )
33 33
 	{
34
-		return isset( $field['default'] )
34
+		return isset($field['default'])
35 35
 			? $field['default']
36 36
 			: '';
37 37
 	}
@@ -42,7 +42,7 @@  discard block
 block discarded – undo
42 42
 	protected function getSettingFields( $path )
43 43
 	{
44 44
 		$settings = glsr( DefaultsManager::class )->settings();
45
-		return array_filter( $settings, function( $key ) use( $path ) {
45
+		return array_filter( $settings, function( $key ) use($path) {
46 46
 			return glsr( Helper::class )->startsWith( $path, $key );
47 47
 		}, ARRAY_FILTER_USE_KEY );
48 48
 	}
@@ -53,14 +53,14 @@  discard block
 block discarded – undo
53 53
 	 */
54 54
 	protected function getTemplateContext( $id )
55 55
 	{
56
-		$fields = $this->getSettingFields( $this->normalizeSettingPath( $id ));
56
+		$fields = $this->getSettingFields( $this->normalizeSettingPath( $id ) );
57 57
 		$rows = '';
58 58
 		foreach( $fields as $name => $field ) {
59 59
 			$field = wp_parse_args( $field, [
60 60
 				'is_setting' => true,
61 61
 				'name' => $name,
62
-			]);
63
-			$rows.= new Field( $this->normalize( $field ));
62
+			] );
63
+			$rows .= new Field( $this->normalize( $field ) );
64 64
 		}
65 65
 		return [
66 66
 			'rows' => $rows,
@@ -73,7 +73,7 @@  discard block
 block discarded – undo
73 73
 	protected function getTemplateContextForTranslations()
74 74
 	{
75 75
 		$translations = glsr( Translator::class )->renderAll();
76
-		$class = empty( $translations )
76
+		$class = empty($translations)
77 77
 			? 'glsr-hidden'
78 78
 			: '';
79 79
 		return [
@@ -94,7 +94,7 @@  discard block
 block discarded – undo
94 94
 			$path,
95 95
 			glsr( Helper::class )->getPathValue( $path, glsr()->defaults )
96 96
 		);
97
-		if( is_array( $expectedValue )) {
97
+		if( is_array( $expectedValue ) ) {
98 98
 			return !in_array( $optionValue, $expectedValue );
99 99
 		}
100 100
 		return $optionValue != $expectedValue;
@@ -116,13 +116,13 @@  discard block
 block discarded – undo
116 116
 	 */
117 117
 	protected function normalizeDependsOn( array $field )
118 118
 	{
119
-		if( !empty( $field['depends_on'] ) && is_array( $field['depends_on'] )) {
119
+		if( !empty($field['depends_on']) && is_array( $field['depends_on'] ) ) {
120 120
 			$path = key( $field['depends_on'] );
121 121
 			$expectedValue = $field['depends_on'][$path];
122
-			$field['data-depends'] = json_encode([
122
+			$field['data-depends'] = json_encode( [
123 123
 				'name' => glsr( Helper::class )->convertPathToName( $path, OptionManager::databaseKey() ),
124 124
 				'value' => $expectedValue,
125
-			], JSON_HEX_APOS|JSON_HEX_QUOT );
125
+			], JSON_HEX_APOS | JSON_HEX_QUOT );
126 126
 			$field['is_hidden'] = $this->isFieldHidden( $path, $expectedValue );
127 127
 		}
128 128
 		return $field;
@@ -133,9 +133,9 @@  discard block
 block discarded – undo
133 133
 	 */
134 134
 	protected function normalizeLabelAndLegend( array $field )
135 135
 	{
136
-		if( isset( $field['label'] )) {
136
+		if( isset($field['label']) ) {
137 137
 			$field['legend'] = $field['label'];
138
-			unset( $field['label'] );
138
+			unset($field['label']);
139 139
 		}
140 140
 		else {
141 141
 			$field['is_valid'] = false;
@@ -149,7 +149,7 @@  discard block
 block discarded – undo
149 149
 	 */
150 150
 	protected function normalizeValue( array $field )
151 151
 	{
152
-		if( !isset( $field['value'] )) {
152
+		if( !isset($field['value']) ) {
153 153
 			$field['value'] = glsr( OptionManager::class )->get(
154 154
 				$field['name'],
155 155
 				$this->getFieldDefault( $field )
Please login to merge, or discard this patch.
plugin/Modules/Html/Fields/Honeypot.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -13,7 +13,7 @@
 block discarded – undo
13 13
 	{
14 14
 		$this->builder->args = wp_parse_args( $this->builder->args, [
15 15
 			'name' => $this->builder->args['text'],
16
-		]);
16
+		] );
17 17
 		$this->builder->tag = 'input';
18 18
 		$this->mergeFieldArgs();
19 19
 		return $this->builder->getOpeningTag();
Please login to merge, or discard this patch.
plugin/Modules/Html/Builder.php 2 patches
Spacing   +24 added lines, -24 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
 		$generatedTag = in_array( $instance->tag, $tags )
61 61
 			? $instance->buildTag()
@@ -78,8 +78,8 @@  discard block
 block discarded – undo
78 78
 			'render' => 'is_bool',
79 79
 			'tag' => 'is_string',
80 80
 		];
81
-		if( !isset( $properties[$property] )
82
-			|| empty( array_filter( [$value], $properties[$property] ))
81
+		if( !isset($properties[$property])
82
+			|| empty(array_filter( [$value], $properties[$property] ))
83 83
 		)return;
84 84
 		$this->$property = $value;
85 85
 	}
@@ -97,7 +97,7 @@  discard block
 block discarded – undo
97 97
 	 */
98 98
 	public function getOpeningTag()
99 99
 	{
100
-		$attributes = glsr( Attributes::class )->{$this->tag}( $this->args )->toString();
100
+		$attributes = glsr( Attributes::class )->{$this->tag}($this->args)->toString();
101 101
 		return '<'.trim( $this->tag.' '.$attributes ).'>';
102 102
 	}
103 103
 
@@ -107,7 +107,7 @@  discard block
 block discarded – undo
107 107
 	protected function buildCustomField()
108 108
 	{
109 109
 		$className = $this->getCustomFieldClassName();
110
-		if( class_exists( $className )) {
110
+		if( class_exists( $className ) ) {
111 111
 			return (new $className( $this ))->build();
112 112
 		}
113 113
 		glsr_log()->error( 'Field missing: '.$className );
@@ -118,7 +118,7 @@  discard block
 block discarded – undo
118 118
 	 */
119 119
 	protected function buildDefaultTag( $text = '' )
120 120
 	{
121
-		if( empty( $text )) {
121
+		if( empty($text) ) {
122 122
 			$text = $this->args['text'];
123 123
 		}
124 124
 		return $this->getOpeningTag().$text.$this->getClosingTag();
@@ -129,7 +129,7 @@  discard block
 block discarded – undo
129 129
 	 */
130 130
 	protected function buildFieldDescription()
131 131
 	{
132
-		if( empty( $this->args['description'] ))return;
132
+		if( empty($this->args['description']) )return;
133 133
 		if( $this->args['is_widget'] ) {
134 134
 			return $this->small( $this->args['description'] );
135 135
 		}
@@ -141,10 +141,10 @@  discard block
 block discarded – undo
141 141
 	 */
142 142
 	protected function buildFormInput()
143 143
 	{
144
-		if( !in_array( $this->args['type'], ['checkbox', 'radio'] )) {
144
+		if( !in_array( $this->args['type'], ['checkbox', 'radio'] ) ) {
145 145
 			return $this->buildFormLabel().$this->getOpeningTag();
146 146
 		}
147
-		return empty( $this->args['options'] )
147
+		return empty($this->args['options'])
148 148
 			? $this->buildFormInputChoice()
149 149
 			: $this->buildFormInputMultiChoice();
150 150
 	}
@@ -163,7 +163,7 @@  discard block
 block discarded – undo
163 163
 	protected function buildFormInputMultiChoice()
164 164
 	{
165 165
 		if( $this->args['type'] == 'checkbox' ) {
166
-			$this->args['name'].= '[]';
166
+			$this->args['name'] .= '[]';
167 167
 		}
168 168
 		$options = array_reduce( array_keys( $this->args['options'] ), function( $carry, $key ) {
169 169
 			return $carry.$this->li( $this->{$this->args['type']}([
@@ -171,12 +171,12 @@  discard block
 block discarded – undo
171 171
 				'name' => $this->args['name'],
172 172
 				'text' => $this->args['options'][$key],
173 173
 				'value' => $key,
174
-			]));
174
+			]) );
175 175
 		});
176 176
 		return $this->ul( $options, [
177 177
 			'class' => $this->args['class'],
178 178
 			'id' => $this->args['id'],
179
-		]);
179
+		] );
180 180
 	}
181 181
 
182 182
 	/**
@@ -184,11 +184,11 @@  discard block
 block discarded – undo
184 184
 	 */
185 185
 	protected function buildFormLabel()
186 186
 	{
187
-		if( empty( $this->args['label'] ) || $this->args['type'] == 'hidden' )return;
188
-		return $this->label([
187
+		if( empty($this->args['label']) || $this->args['type'] == 'hidden' )return;
188
+		return $this->label( [
189 189
 			'for' => $this->args['id'],
190 190
 			'text' => $this->args['label'],
191
-		]);
191
+		] );
192 192
 	}
193 193
 
194 194
 	/**
@@ -205,11 +205,11 @@  discard block
 block discarded – undo
205 205
 	protected function buildFormSelectOptions()
206 206
 	{
207 207
 		return array_reduce( array_keys( $this->args['options'] ), function( $carry, $key ) {
208
-			return $carry.$this->option([
208
+			return $carry.$this->option( [
209 209
 				'selected' => $this->args['value'] == $key,
210 210
 				'text' => $this->args['options'][$key],
211 211
 				'value' => $key,
212
-			]);
212
+			] );
213 213
 		});
214 214
 	}
215 215
 
@@ -227,10 +227,10 @@  discard block
 block discarded – undo
227 227
 	protected function buildTag()
228 228
 	{
229 229
 		$this->mergeArgsWithRequiredDefaults();
230
-		if( in_array( $this->tag, static::TAGS_SINGLE )) {
230
+		if( in_array( $this->tag, static::TAGS_SINGLE ) ) {
231 231
 			return $this->getOpeningTag();
232 232
 		}
233
-		if( !in_array( $this->tag, static::TAGS_FORM )) {
233
+		if( !in_array( $this->tag, static::TAGS_FORM ) ) {
234 234
 			return $this->buildDefaultTag();
235 235
 		}
236 236
 		return call_user_func( [$this, 'buildForm'.ucfirst( $this->tag )] ).$this->buildFieldDescription();
@@ -251,7 +251,7 @@  discard block
 block discarded – undo
251 251
 	{
252 252
 		$args = glsr( BuilderDefaults::class )->merge( $this->args );
253 253
 		$className = $this->getCustomFieldClassName();
254
-		if( class_exists( $className )) {
254
+		if( class_exists( $className ) ) {
255 255
 			$args = array_merge(
256 256
 				wp_parse_args( $args, $className::defaults() ),
257 257
 				$className::required()
@@ -266,13 +266,13 @@  discard block
 block discarded – undo
266 266
 	 */
267 267
 	protected function normalize( ...$params )
268 268
 	{
269
-		if( is_string( $params[0] ) || is_numeric( $params[0] )) {
269
+		if( is_string( $params[0] ) || is_numeric( $params[0] ) ) {
270 270
 			$this->setNameOrTextAttributeForTag( $params[0] );
271 271
 		}
272
-		if( is_array( $params[0] )) {
272
+		if( is_array( $params[0] ) ) {
273 273
 			$this->args += $params[0];
274 274
 		}
275
-		else if( is_array( $params[1] )) {
275
+		else if( is_array( $params[1] ) ) {
276 276
 			$this->args += $params[1];
277 277
 		}
278 278
 	}
@@ -296,7 +296,7 @@  discard block
 block discarded – undo
296 296
 	protected function setTagFromMethod( $method )
297 297
 	{
298 298
 		$this->tag = strtolower( $method );
299
-		if( in_array( $this->tag, static::INPUT_TYPES )) {
299
+		if( in_array( $this->tag, static::INPUT_TYPES ) ) {
300 300
 			$this->args['type'] = $this->tag;
301 301
 			$this->tag = 'input';
302 302
 		}
Please login to merge, or discard this patch.
Braces   +9 added lines, -3 removed lines patch added patch discarded remove patch
@@ -80,7 +80,9 @@  discard block
 block discarded – undo
80 80
 		];
81 81
 		if( !isset( $properties[$property] )
82 82
 			|| empty( array_filter( [$value], $properties[$property] ))
83
-		)return;
83
+		) {
84
+			return;
85
+		}
84 86
 		$this->$property = $value;
85 87
 	}
86 88
 
@@ -129,7 +131,9 @@  discard block
 block discarded – undo
129 131
 	 */
130 132
 	protected function buildFieldDescription()
131 133
 	{
132
-		if( empty( $this->args['description'] ))return;
134
+		if( empty( $this->args['description'] )) {
135
+			return;
136
+		}
133 137
 		if( $this->args['is_widget'] ) {
134 138
 			return $this->small( $this->args['description'] );
135 139
 		}
@@ -184,7 +188,9 @@  discard block
 block discarded – undo
184 188
 	 */
185 189
 	protected function buildFormLabel()
186 190
 	{
187
-		if( empty( $this->args['label'] ) || $this->args['type'] == 'hidden' )return;
191
+		if( empty( $this->args['label'] ) || $this->args['type'] == 'hidden' ) {
192
+			return;
193
+		}
188 194
 		return $this->label([
189 195
 			'for' => $this->args['id'],
190 196
 			'text' => $this->args['label'],
Please login to merge, or discard this patch.
plugin/Modules/Html/Field.php 2 patches
Spacing   +15 added lines, -15 removed lines patch added patch discarded remove patch
@@ -25,7 +25,7 @@  discard block
 block discarded – undo
25 25
 			'is_valid' => true,
26 26
 			'is_widget' => false,
27 27
 			'path' => '',
28
-		]);
28
+		] );
29 29
 		$this->normalize();
30 30
 	}
31 31
 
@@ -71,9 +71,9 @@  discard block
 block discarded – undo
71 71
 		return glsr( Template::class )->build( 'partials/form/field', [
72 72
 			'context' => [
73 73
 				'class' => $this->getFieldClass(),
74
-				'field' => glsr( Builder::class )->{$this->field['type']}( $this->field ),
74
+				'field' => glsr( Builder::class )->{$this->field['type']}($this->field),
75 75
 			],
76
-		]);
76
+		] );
77 77
 	}
78 78
 
79 79
 	/**
@@ -84,10 +84,10 @@  discard block
 block discarded – undo
84 84
 		return glsr( Template::class )->build( 'partials/form/table-row', [
85 85
 			'context' => [
86 86
 				'class' => $this->getFieldClass(),
87
-				'field' => glsr( Builder::class )->{$this->field['type']}( $this->field ),
87
+				'field' => glsr( Builder::class )->{$this->field['type']}($this->field),
88 88
 				'label' => glsr( Builder::class )->label( $this->field['legend'], ['for' => $this->field['id']] ),
89 89
 			],
90
-		]);
90
+		] );
91 91
 	}
92 92
 
93 93
 	/**
@@ -96,16 +96,16 @@  discard block
 block discarded – undo
96 96
 	protected function buildSettingMultiField()
97 97
 	{
98 98
 		$dependsOn = $this->getFieldDependsOn();
99
-		unset( $this->field['data-depends'] );
99
+		unset($this->field['data-depends']);
100 100
 		return glsr( Template::class )->build( 'partials/form/table-row-multiple', [
101 101
 			'context' => [
102 102
 				'class' => $this->getFieldClass(),
103 103
 				'depends_on' => $dependsOn,
104
-				'field' => glsr( Builder::class )->{$this->field['type']}( $this->field ),
104
+				'field' => glsr( Builder::class )->{$this->field['type']}($this->field),
105 105
 				'label' => glsr( Builder::class )->label( $this->field['legend'], ['for' => $this->field['id']] ),
106 106
 				'legend' => $this->field['legend'],
107 107
 			],
108
-		]);
108
+		] );
109 109
 	}
110 110
 
111 111
 	/**
@@ -117,7 +117,7 @@  discard block
 block discarded – undo
117 117
 		if( $this->field['is_hidden'] ) {
118 118
 			$classes[] = 'hidden';
119 119
 		}
120
-		if( !empty( $this->field['required'] )) {
120
+		if( !empty($this->field['required']) ) {
121 121
 			$classes[] = 'glsr-required';
122 122
 		}
123 123
 		return implode( ' ', $classes );
@@ -128,7 +128,7 @@  discard block
 block discarded – undo
128 128
 	 */
129 129
 	protected function getFieldDependsOn()
130 130
 	{
131
-		return !empty( $this->field['data-depends'] )
131
+		return !empty($this->field['data-depends'])
132 132
 			? $this->field['data-depends']
133 133
 			: '';
134 134
 	}
@@ -153,13 +153,13 @@  discard block
 block discarded – undo
153 153
 			'name', 'type',
154 154
 		];
155 155
 		foreach( $requiredValues as $value ) {
156
-			if( isset( $this->field[$value] ))continue;
156
+			if( isset($this->field[$value]) )continue;
157 157
 			$missingValues[] = $value;
158 158
 			$this->field['is_valid'] = false;
159 159
 		}
160
-		if( !empty( $missingValues )) {
160
+		if( !empty($missingValues) ) {
161 161
 			glsr_log()
162
-				->warning( 'Field is missing: '.implode( ', ', $missingValues ))
162
+				->warning( 'Field is missing: '.implode( ', ', $missingValues ) )
163 163
 				->info( $this->field );
164 164
 		}
165 165
 		return $this->field['is_valid'];
@@ -173,7 +173,7 @@  discard block
 block discarded – undo
173 173
 		if( !$this->isFieldValid() )return;
174 174
 		$this->field['path'] = $this->field['name'];
175 175
 		$className = glsr( Helper::class )->buildClassName( $this->field['type'], __NAMESPACE__.'\Fields' );
176
-		if( class_exists( $className )) {
176
+		if( class_exists( $className ) ) {
177 177
 			$this->field = array_merge(
178 178
 				wp_parse_args( $this->field, $className::defaults() ),
179 179
 				$className::required()
@@ -188,7 +188,7 @@  discard block
 block discarded – undo
188 188
 	 */
189 189
 	protected function normalizeFieldId()
190 190
 	{
191
-		if( isset( $this->field['id'] ) || $this->field['is_raw'] )return;
191
+		if( isset($this->field['id']) || $this->field['is_raw'] )return;
192 192
 		$this->field['id'] = glsr( Helper::class )->convertPathToId(
193 193
 			$this->field['path'],
194 194
 			$this->getFieldPrefix()
Please login to merge, or discard this patch.
Braces   +12 added lines, -4 removed lines patch added patch discarded remove patch
@@ -42,7 +42,9 @@  discard block
 block discarded – undo
42 42
 	 */
43 43
 	public function build()
44 44
 	{
45
-		if( !$this->field['is_valid'] )return;
45
+		if( !$this->field['is_valid'] ) {
46
+			return;
47
+		}
46 48
 		if( $this->field['is_raw'] ) {
47 49
 			return glsr( Builder::class )->hidden( $this->field );
48 50
 		}
@@ -153,7 +155,9 @@  discard block
 block discarded – undo
153 155
 			'name', 'type',
154 156
 		];
155 157
 		foreach( $requiredValues as $value ) {
156
-			if( isset( $this->field[$value] ))continue;
158
+			if( isset( $this->field[$value] )) {
159
+				continue;
160
+			}
157 161
 			$missingValues[] = $value;
158 162
 			$this->field['is_valid'] = false;
159 163
 		}
@@ -170,7 +174,9 @@  discard block
 block discarded – undo
170 174
 	 */
171 175
 	protected function normalize()
172 176
 	{
173
-		if( !$this->isFieldValid() )return;
177
+		if( !$this->isFieldValid() ) {
178
+			return;
179
+		}
174 180
 		$this->field['path'] = $this->field['name'];
175 181
 		$className = glsr( Helper::class )->buildClassName( $this->field['type'], __NAMESPACE__.'\Fields' );
176 182
 		if( class_exists( $className )) {
@@ -188,7 +194,9 @@  discard block
 block discarded – undo
188 194
 	 */
189 195
 	protected function normalizeFieldId()
190 196
 	{
191
-		if( isset( $this->field['id'] ) || $this->field['is_raw'] )return;
197
+		if( isset( $this->field['id'] ) || $this->field['is_raw'] ) {
198
+			return;
199
+		}
192 200
 		$this->field['id'] = glsr( Helper::class )->convertPathToId(
193 201
 			$this->field['path'],
194 202
 			$this->getFieldPrefix()
Please login to merge, or discard this patch.
plugin/Handlers/EnqueueAdminAssets.php 2 patches
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -44,7 +44,7 @@  discard block
 block discarded – undo
44 44
 			glsr()->version,
45 45
 			true
46 46
 		);
47
-		if( !empty( $this->pointers )) {
47
+		if( !empty($this->pointers) ) {
48 48
 			wp_enqueue_style( 'wp-pointer' );
49 49
 			wp_enqueue_script( 'wp-pointer' );
50 50
 		}
@@ -84,7 +84,7 @@  discard block
 block discarded – undo
84 84
 		$dependencies = apply_filters( 'site-reviews/enqueue/admin/dependencies', [] );
85 85
 		$dependencies = array_merge( $dependencies, [
86 86
 			'jquery', 'jquery-ui-sortable', 'underscore', 'wp-util',
87
-		]);
87
+		] );
88 88
 		return $dependencies;
89 89
 	}
90 90
 
@@ -114,7 +114,7 @@  discard block
 block discarded – undo
114 114
 		$generatedPointers = [];
115 115
 		foreach( $pointers as $pointer ) {
116 116
 			if( $pointer['screen'] != glsr_current_screen()->id )continue;
117
-			if( in_array( $pointer['id'], $dismissedPointers ))continue;
117
+			if( in_array( $pointer['id'], $dismissedPointers ) )continue;
118 118
 			$generatedPointers[] = $this->generatePointer( $pointer );
119 119
 		}
120 120
 		$this->pointers = $generatedPointers;
@@ -126,7 +126,7 @@  discard block
 block discarded – undo
126 126
 	protected function isCurrentScreen()
127 127
 	{
128 128
 		$screen = glsr_current_screen();
129
-		return $screen && ( $screen->post_type == Application::POST_TYPE
129
+		return $screen && ($screen->post_type == Application::POST_TYPE
130 130
 			|| $screen->base == 'post'
131 131
 			|| $screen->id == 'dashboard'
132 132
 			|| $screen->id == 'widgets'
@@ -140,7 +140,7 @@  discard block
 block discarded – undo
140 140
 	{
141 141
 		$variables = [];
142 142
 		foreach( glsr()->mceShortcodes as $tag => $args ) {
143
-			if( empty( $args['required'] ))continue;
143
+			if( empty($args['required']) )continue;
144 144
 			$variables[$tag] = $args['required'];
145 145
 		}
146 146
 		return $variables;
Please login to merge, or discard this patch.
Braces   +12 added lines, -4 removed lines patch added patch discarded remove patch
@@ -36,7 +36,9 @@  discard block
 block discarded – undo
36 36
 			[],
37 37
 			glsr()->version
38 38
 		);
39
-		if( !$this->isCurrentScreen() )return;
39
+		if( !$this->isCurrentScreen() ) {
40
+			return;
41
+		}
40 42
 		wp_enqueue_script(
41 43
 			Application::ID,
42 44
 			glsr()->url( 'assets/scripts/'.Application::ID.'-admin.js' ),
@@ -113,8 +115,12 @@  discard block
 block discarded – undo
113 115
 		$dismissedPointers = explode( ',', (string)$dismissedPointers );
114 116
 		$generatedPointers = [];
115 117
 		foreach( $pointers as $pointer ) {
116
-			if( $pointer['screen'] != glsr_current_screen()->id )continue;
117
-			if( in_array( $pointer['id'], $dismissedPointers ))continue;
118
+			if( $pointer['screen'] != glsr_current_screen()->id ) {
119
+				continue;
120
+			}
121
+			if( in_array( $pointer['id'], $dismissedPointers )) {
122
+				continue;
123
+			}
118 124
 			$generatedPointers[] = $this->generatePointer( $pointer );
119 125
 		}
120 126
 		$this->pointers = $generatedPointers;
@@ -140,7 +146,9 @@  discard block
 block discarded – undo
140 146
 	{
141 147
 		$variables = [];
142 148
 		foreach( glsr()->mceShortcodes as $tag => $args ) {
143
-			if( empty( $args['required'] ))continue;
149
+			if( empty( $args['required'] )) {
150
+				continue;
151
+			}
144 152
 			$variables[$tag] = $args['required'];
145 153
 		}
146 154
 		return $variables;
Please login to merge, or discard this patch.