Passed
Push — master ( 75834f...73b77e )
by Paul
04:12
created
plugin/Modules/Html/Builder.php 1 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/Handlers/EnqueueAdminAssets.php 1 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.
plugin/Modules/System.php 1 patch
Braces   +15 added lines, -5 removed lines patch added patch discarded remove patch
@@ -91,7 +91,9 @@  discard block
 block discarded – undo
91 91
 			get_mu_plugins(),
92 92
 			get_plugins( '/../'.basename( WPMU_PLUGIN_DIR ))
93 93
 		);
94
-		if( empty( $plugins ))return;
94
+		if( empty( $plugins )) {
95
+			return;
96
+		}
95 97
 		return $this->normalizePluginList( $plugins );
96 98
 	}
97 99
 
@@ -100,7 +102,9 @@  discard block
 block discarded – undo
100 102
 	 */
101 103
 	public function getMultisitePluginDetails()
102 104
 	{
103
-		if( !is_multisite() || empty( get_site_option( 'active_sitewide_plugins', [] )))return;
105
+		if( !is_multisite() || empty( get_site_option( 'active_sitewide_plugins', [] ))) {
106
+			return;
107
+		}
104 108
 		return $this->normalizePluginList( wp_get_active_network_plugins() );
105 109
 	}
106 110
 
@@ -154,12 +158,16 @@  discard block
 block discarded – undo
154 158
 		$settings = glsr( OptionManager::class )->get( 'settings', [] );
155 159
 		$settings = $helper->flattenArray( $settings, true );
156 160
 		foreach( ['submissions.recaptcha.key', 'submissions.recaptcha.secret'] as $key ) {
157
-			if( empty( $settings[$key] ))continue;
161
+			if( empty( $settings[$key] )) {
162
+				continue;
163
+			}
158 164
 			$settings[$key] = str_repeat( '*', 10 );
159 165
 		}
160 166
 		$details = [];
161 167
 		foreach( $settings as $key => $value ) {
162
-			if( $helper->startsWith( 'strings', $key ) && $helper->endsWith( 'id', $key ))continue;
168
+			if( $helper->startsWith( 'strings', $key ) && $helper->endsWith( 'id', $key )) {
169
+				continue;
170
+			}
163 171
 			$value = htmlspecialchars( trim( preg_replace('/\s\s+/', '\\n', $value )), ENT_QUOTES, 'UTF-8' );
164 172
 			$details[$key] = $value;
165 173
 		}
@@ -231,7 +239,9 @@  discard block
 block discarded – undo
231 239
 			'WPE_APIKEY' => 'WP Engine',
232 240
 		];
233 241
 		foreach( $checks as $key => $value ) {
234
-			if( !$this->isWebhostCheckValid( $key ))continue;
242
+			if( !$this->isWebhostCheckValid( $key )) {
243
+				continue;
244
+			}
235 245
 			return $value;
236 246
 		}
237 247
 		return implode( ',', array_filter( [DB_HOST, filter_input( INPUT_SERVER, 'SERVER_NAME' )] ));
Please login to merge, or discard this patch.
autoload.php 1 patch
Braces   +6 added lines, -2 removed lines patch added patch discarded remove patch
@@ -15,9 +15,13 @@
 block discarded – undo
15 15
 	];
16 16
 	foreach( $namespaces as $prefix => $baseDir ) {
17 17
 		$len = strlen( $prefix );
18
-		if( strncmp( $prefix, $className, $len ) !== 0 )continue;
18
+		if( strncmp( $prefix, $className, $len ) !== 0 ) {
19
+			continue;
20
+		}
19 21
 		$file = $baseDir.str_replace( '\\', '/', substr( $className, $len )).'.php';
20
-		if( !file_exists( $file ))continue;
22
+		if( !file_exists( $file )) {
23
+			continue;
24
+		}
21 25
 		require $file;
22 26
 		break;
23 27
 	}
Please login to merge, or discard this patch.
plugin/Database.php 1 patch
Braces   +18 added lines, -6 removed lines patch added patch discarded remove patch
@@ -64,7 +64,9 @@  discard block
 block discarded – undo
64 64
 	public function getAssignedToPost( $post, $assignedTo = '' )
65 65
 	{
66 66
 		$post = get_post( $post );
67
-		if( !( $post instanceof WP_Post ))return;
67
+		if( !( $post instanceof WP_Post )) {
68
+			return;
69
+		}
68 70
 		if( empty( $assignedTo )) {
69 71
 			$assignedTo = get_post_meta( $post->ID, 'assigned_to', true );
70 72
 		}
@@ -82,7 +84,9 @@  discard block
 block discarded – undo
82 84
 	 */
83 85
 	public function getReview( $post )
84 86
 	{
85
-		if( !( $post instanceof WP_Post ) || $post->post_type != Application::POST_TYPE )return;
87
+		if( !( $post instanceof WP_Post ) || $post->post_type != Application::POST_TYPE ) {
88
+			return;
89
+		}
86 90
 		$review = $this->getReviewMeta( $post->ID );
87 91
 		$modified = $this->isReviewModified( $post, $review );
88 92
 		$review->content = $post->post_content;
@@ -256,7 +260,9 @@  discard block
 block discarded – undo
256 260
 		$termIds = array_map( 'trim', explode( ',', $termIds ));
257 261
 		foreach( $termIds as $termId ) {
258 262
 			$term = term_exists( $termId, Application::TAXONOMY );
259
-			if( !isset( $term['term_id'] ))continue;
263
+			if( !isset( $term['term_id'] )) {
264
+				continue;
265
+			}
260 266
 			$terms[] = intval( $term['term_id'] );
261 267
 		}
262 268
 		return $terms;
@@ -269,7 +275,9 @@  discard block
 block discarded – undo
269 275
 	public function revertReview( $postId )
270 276
 	{
271 277
 		$post = get_post( $postId );
272
-		if( !( $post instanceof WP_Post ) || $post->post_type != Application::POST_TYPE )return;
278
+		if( !( $post instanceof WP_Post ) || $post->post_type != Application::POST_TYPE ) {
279
+			return;
280
+		}
273 281
 		delete_post_meta( $post->ID, '_edit_last' );
274 282
 		$result = wp_update_post([
275 283
 			'ID' => $post->ID,
@@ -304,7 +312,9 @@  discard block
 block discarded – undo
304 312
 		add_filter( 'posts_search', [$queryBuilder, 'filterSearchByTitle'], 500, 2 );
305 313
 		$search = new WP_Query( $args );
306 314
 		remove_filter( 'posts_search', [$queryBuilder, 'filterSearchByTitle'], 500 );
307
-		if( !$search->have_posts() )return;
315
+		if( !$search->have_posts() ) {
316
+			return;
317
+		}
308 318
 		$results = '';
309 319
 		while( $search->have_posts() ) {
310 320
 			$search->the_post();
@@ -331,7 +341,9 @@  discard block
 block discarded – undo
331 341
 			update_post_meta( $postId, $metaKey, $metaValue );
332 342
 		}
333 343
 		$terms = $this->normalizeTerms( $termIds );
334
-		if( empty( $terms ))return;
344
+		if( empty( $terms )) {
345
+			return;
346
+		}
335 347
 		$result = wp_set_object_terms( $postId, $terms, Application::TAXONOMY );
336 348
 		if( is_wp_error( $result )) {
337 349
 			glsr_log()->error( $result->get_error_message() );
Please login to merge, or discard this patch.
plugin/Controllers/PublicController.php 1 patch
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -50,7 +50,9 @@
 block discarded – undo
50 50
 	public function routerSubmitReview( array $request )
51 51
 	{
52 52
 		$validated = glsr( ValidateReview::class )->validate( $request );
53
-		if( !empty( $validated->error ) || $validated->recaptchaIsUnset )return;
53
+		if( !empty( $validated->error ) || $validated->recaptchaIsUnset ) {
54
+			return;
55
+		}
54 56
 		$this->execute( new CreateReview( $validated->request ));
55 57
 	}
56 58
 }
Please login to merge, or discard this patch.
plugin/Modules/Html/Partials/FormResults.php 1 patch
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -19,7 +19,9 @@
 block discarded – undo
19 19
 	public function build( array $args = [] )
20 20
 	{
21 21
 		$this->errors = $args['errors'];
22
-		if( empty( $args['message'] ))return;
22
+		if( empty( $args['message'] )) {
23
+			return;
24
+		}
23 25
 		return glsr( Builder::class )->div( wpautop( $args['message'] ), [
24 26
 			'class' => $this->getClass(),
25 27
 		]);
Please login to merge, or discard this patch.
plugin/Modules/Html/Partials/SiteReviewsForm.php 1 patch
Braces   +9 added lines, -3 removed lines patch added patch discarded remove patch
@@ -152,7 +152,9 @@  discard block
 block discarded – undo
152 152
 	 */
153 153
 	protected function normalizeFieldErrors( Field &$field )
154 154
 	{
155
-		if( !array_key_exists( $field->field['path'], $this->errors ))return;
155
+		if( !array_key_exists( $field->field['path'], $this->errors )) {
156
+			return;
157
+		}
156 158
 		$field->field['errors'] = $this->errors[$field->field['path']];
157 159
 	}
158 160
 
@@ -161,7 +163,9 @@  discard block
 block discarded – undo
161 163
 	 */
162 164
 	protected function normalizeFieldRequired( Field &$field )
163 165
 	{
164
-		if( !in_array( $field->field['path'], $this->required ))return;
166
+		if( !in_array( $field->field['path'], $this->required )) {
167
+			return;
168
+		}
165 169
 		$field->field['required'] = true;
166 170
 	}
167 171
 
@@ -183,7 +187,9 @@  discard block
 block discarded – undo
183 187
 	 */
184 188
 	protected function normalizeFieldValue( Field &$field )
185 189
 	{
186
-		if( !array_key_exists( $field->field['path'], $this->values ))return;
190
+		if( !array_key_exists( $field->field['path'], $this->values )) {
191
+			return;
192
+		}
187 193
 		if( in_array( $field->field['type'], ['radio', 'checkbox'] )) {
188 194
 			$field->field['checked'] = $field->field['value'] == $this->values[$field->field['path']];
189 195
 		}
Please login to merge, or discard this patch.
plugin/Modules/Html/Field.php 1 patch
Braces   +15 added lines, -5 removed lines patch added patch discarded remove patch
@@ -43,7 +43,9 @@  discard block
 block discarded – undo
43 43
 	 */
44 44
 	public function build()
45 45
 	{
46
-		if( !$this->field['is_valid'] )return;
46
+		if( !$this->field['is_valid'] ) {
47
+			return;
48
+		}
47 49
 		if( $this->field['is_raw'] ) {
48 50
 			return glsr( Builder::class )->{$this->field['type']}( $this->field );
49 51
 		}
@@ -143,7 +145,9 @@  discard block
 block discarded – undo
143 145
 	 */
144 146
 	protected function getFieldErrors()
145 147
 	{
146
-		if( empty( $this->field['errors'] ) || !is_array( $this->field['errors'] ))return;
148
+		if( empty( $this->field['errors'] ) || !is_array( $this->field['errors'] )) {
149
+			return;
150
+		}
147 151
 		$errors = array_reduce( $this->field['errors'], function( $carry, $error ) {
148 152
 			return $carry.glsr( Builder::class )->span( $error, ['class' => 'glsr-field-error'] );
149 153
 		});
@@ -170,7 +174,9 @@  discard block
 block discarded – undo
170 174
 			'name', 'type',
171 175
 		];
172 176
 		foreach( $requiredValues as $value ) {
173
-			if( isset( $this->field[$value] ))continue;
177
+			if( isset( $this->field[$value] )) {
178
+				continue;
179
+			}
174 180
 			$missingValues[] = $value;
175 181
 			$this->field['is_valid'] = false;
176 182
 		}
@@ -187,7 +193,9 @@  discard block
 block discarded – undo
187 193
 	 */
188 194
 	protected function normalize()
189 195
 	{
190
-		if( !$this->isFieldValid() )return;
196
+		if( !$this->isFieldValid() ) {
197
+			return;
198
+		}
191 199
 		$this->field['path'] = $this->field['name'];
192 200
 		$className = glsr( Helper::class )->buildClassName( $this->field['type'], __NAMESPACE__.'\Fields' );
193 201
 		if( class_exists( $className )) {
@@ -205,7 +213,9 @@  discard block
 block discarded – undo
205 213
 	 */
206 214
 	protected function normalizeFieldId()
207 215
 	{
208
-		if( isset( $this->field['id'] ) || $this->field['is_raw'] )return;
216
+		if( isset( $this->field['id'] ) || $this->field['is_raw'] ) {
217
+			return;
218
+		}
209 219
 		$this->field['id'] = glsr( Helper::class )->convertPathToId(
210 220
 			$this->field['path'],
211 221
 			$this->getFieldPrefix()
Please login to merge, or discard this patch.