Passed
Push — master ( fbb7de...43cb65 )
by Paul
25:46 queued 14:45
created
plugin/Modules/Email.php 2 patches
Braces   +9 added lines, -3 removed lines patch added patch discarded remove patch
@@ -72,7 +72,9 @@  discard block
 block discarded – undo
72 72
 	 */
73 73
 	public function send()
74 74
 	{
75
-		if( !$this->message || !$this->subject || !$this->to )return;
75
+		if( !$this->message || !$this->subject || !$this->to ) {
76
+			return;
77
+		}
76 78
 		$sent = wp_mail(
77 79
 			$this->to,
78 80
 			$this->subject,
@@ -90,8 +92,12 @@  discard block
 block discarded – undo
90 92
 	 */
91 93
 	public function buildPlainTextMessage( PHPMailer $phpmailer )
92 94
 	{
93
-		if( empty( $this->email ))return;
94
-		if( $phpmailer->ContentType === 'text/plain' || !empty( $phpmailer->AltBody ))return;
95
+		if( empty( $this->email )) {
96
+			return;
97
+		}
98
+		if( $phpmailer->ContentType === 'text/plain' || !empty( $phpmailer->AltBody )) {
99
+			return;
100
+		}
95 101
 		$message = $this->stripHtmlTags( $phpmailer->Body );
96 102
 		$phpmailer->AltBody = apply_filters( 'site-reviews/email/message', $message, 'text', $this );
97 103
 	}
Please login to merge, or discard this patch.
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -90,8 +90,8 @@  discard block
 block discarded – undo
90 90
 	 */
91 91
 	public function buildPlainTextMessage( PHPMailer $phpmailer )
92 92
 	{
93
-		if( empty( $this->email ))return;
94
-		if( $phpmailer->ContentType === 'text/plain' || !empty( $phpmailer->AltBody ))return;
93
+		if( empty($this->email) )return;
94
+		if( $phpmailer->ContentType === 'text/plain' || !empty($phpmailer->AltBody) )return;
95 95
 		$message = $this->stripHtmlTags( $phpmailer->Body );
96 96
 		$phpmailer->AltBody = apply_filters( 'site-reviews/email/message', $message, 'text', $this );
97 97
 	}
@@ -104,10 +104,10 @@  discard block
 block discarded – undo
104 104
 		$allowed = [
105 105
 			'bcc', 'cc', 'from', 'reply-to',
106 106
 		];
107
-		$headers = array_intersect_key( $this->email, array_flip( $allowed ));
107
+		$headers = array_intersect_key( $this->email, array_flip( $allowed ) );
108 108
 		$headers = array_filter( $headers );
109 109
 		foreach( $headers as $key => $value ) {
110
-			unset( $headers[$key] );
110
+			unset($headers[$key]);
111 111
 			$headers[] = $key.': '.$value;
112 112
 		}
113 113
 		$headers[] = 'Content-Type: text/html';
@@ -119,16 +119,16 @@  discard block
 block discarded – undo
119 119
 	 */
120 120
 	protected function buildHtmlMessage()
121 121
 	{
122
-		$template = trim( glsr( OptionManager::class )->get( 'settings.general.notification_message' ));
123
-		if( !empty( $template )) {
122
+		$template = trim( glsr( OptionManager::class )->get( 'settings.general.notification_message' ) );
123
+		if( !empty($template) ) {
124 124
 			$message = glsr( Template::class )->interpolate( $template, $this->email['template-tags'], $this->email['template'] );
125 125
 		}
126 126
 		else if( $this->email['template'] ) {
127 127
 			$message = glsr( Template::class )->build( 'templates/'.$this->email['template'], [
128 128
 				'context' => $this->email['template-tags'],
129
-			]);
129
+			] );
130 130
 		}
131
-		if( !isset( $message )) {
131
+		if( !isset($message) ) {
132 132
 			$message = $this->email['message'];
133 133
 		}
134 134
 		$message = $this->email['before'].$message.$this->email['after'];
@@ -139,7 +139,7 @@  discard block
 block discarded – undo
139 139
 		$message = str_replace( ']]>', ']]>', $message );
140 140
 		$message = glsr( Template::class )->build( 'partials/email/index', [
141 141
 			'context' => ['message' => $message],
142
-		]);
142
+		] );
143 143
 		return apply_filters( 'site-reviews/email/message', stripslashes( $message ), 'html', $this );
144 144
 	}
145 145
 
@@ -149,7 +149,7 @@  discard block
 block discarded – undo
149 149
 	protected function normalize( array $email = [] )
150 150
 	{
151 151
 		$email = shortcode_atts( glsr( EmailDefaults::class )->defaults(), $email );
152
-		if( empty( $email['reply-to'] )) {
152
+		if( empty($email['reply-to']) ) {
153 153
 			$email['reply-to'] = $email['from'];
154 154
 		}
155 155
 		$this->email = apply_filters( 'site-reviews/email/compose', $email, $this );
Please login to merge, or discard this patch.
plugin/Modules/Schema/BaseType.php 2 patches
Braces   +6 added lines, -2 removed lines patch added patch discarded remove patch
@@ -212,7 +212,9 @@  discard block
 block discarded – undo
212 212
 		$newParents = $parents;
213 213
 		foreach( $parents as $parent ) {
214 214
 			$parentClass = glsr( Helper::class )->buildClassName( $parent, __NAMESPACE__ );
215
-			if( !class_exists( $parentClass ))continue;
215
+			if( !class_exists( $parentClass )) {
216
+				continue;
217
+			}
216 218
 			$newParents = array_merge( $newParents, $this->getParents( (new $parentClass)->parents ));
217 219
 		}
218 220
 		return array_values( array_unique( $newParents ));
@@ -226,7 +228,9 @@  discard block
 block discarded – undo
226 228
 		$parents = $this->getParents();
227 229
 		foreach( $parents as $parent ) {
228 230
 			$parentClass = glsr( Helper::class )->buildClassName( $parent, __NAMESPACE__ );
229
-			if( !class_exists( $parentClass ))continue;
231
+			if( !class_exists( $parentClass )) {
232
+				continue;
233
+			}
230 234
 			$this->allowed = array_values( array_unique( array_merge( (new $parentClass)->allowed, $this->allowed )));
231 235
 		}
232 236
 	}
Please login to merge, or discard this patch.
Spacing   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -39,7 +39,7 @@  discard block
 block discarded – undo
39 39
 	 */
40 40
 	public function __call( $method, array $arguments )
41 41
 	{
42
-		$value = isset( $arguments[0] )
42
+		$value = isset($arguments[0])
43 43
 			? $arguments[0]
44 44
 			: '';
45 45
 		return $this->setProperty( $method, $value );
@@ -96,9 +96,9 @@  discard block
 block discarded – undo
96 96
 	 * @param mixed $default
97 97
 	 * @return mixed
98 98
 	 */
99
-	public function getProperty( $property, $default = null)
99
+	public function getProperty( $property, $default = null )
100 100
 	{
101
-		return isset( $this->properties[$property] )
101
+		return isset($this->properties[$property])
102 102
 			? $this->properties[$property]
103 103
 			: $default;
104 104
 	}
@@ -166,7 +166,7 @@  discard block
 block discarded – undo
166 166
 	 */
167 167
 	public function offsetUnset( $offset )
168 168
 	{
169
-		unset( $this->properties[$offset] );
169
+		unset($this->properties[$offset]);
170 170
 	}
171 171
 
172 172
 	/**
@@ -212,16 +212,16 @@  discard block
 block discarded – undo
212 212
 	 */
213 213
 	protected function getParents( $parents = null )
214 214
 	{
215
-		if( !isset( $parents )) {
215
+		if( !isset($parents) ) {
216 216
 			$parents = $this->parents;
217 217
 		}
218 218
 		$newParents = $parents;
219 219
 		foreach( $parents as $parent ) {
220 220
 			$parentClass = glsr( Helper::class )->buildClassName( $parent, __NAMESPACE__ );
221
-			if( !class_exists( $parentClass ))continue;
222
-			$newParents = array_merge( $newParents, $this->getParents( (new $parentClass)->parents ));
221
+			if( !class_exists( $parentClass ) )continue;
222
+			$newParents = array_merge( $newParents, $this->getParents( (new $parentClass)->parents ) );
223 223
 		}
224
-		return array_values( array_unique( $newParents ));
224
+		return array_values( array_unique( $newParents ) );
225 225
 	}
226 226
 
227 227
 	/**
@@ -232,8 +232,8 @@  discard block
 block discarded – undo
232 232
 		$parents = $this->getParents();
233 233
 		foreach( $parents as $parent ) {
234 234
 			$parentClass = glsr( Helper::class )->buildClassName( $parent, __NAMESPACE__ );
235
-			if( !class_exists( $parentClass ))continue;
236
-			$this->allowed = array_values( array_unique( array_merge( (new $parentClass)->allowed, $this->allowed )));
235
+			if( !class_exists( $parentClass ) )continue;
236
+			$this->allowed = array_values( array_unique( array_merge( (new $parentClass)->allowed, $this->allowed ) ) );
237 237
 		}
238 238
 	}
239 239
 
@@ -243,17 +243,17 @@  discard block
 block discarded – undo
243 243
 	 */
244 244
 	protected function serializeProperty( $property )
245 245
 	{
246
-		if( is_array( $property )) {
246
+		if( is_array( $property ) ) {
247 247
 			return array_map( [$this, 'serializeProperty'], $property );
248 248
 		}
249 249
 		if( $property instanceof Type ) {
250 250
 			$property = $property->toArray();
251
-			unset( $property['@context'] );
251
+			unset($property['@context']);
252 252
 		}
253 253
 		if( $property instanceof DateTimeInterface ) {
254 254
 			$property = $property->format( DateTime::ATOM );
255 255
 		}
256
-		if( is_object( $property )) {
256
+		if( is_object( $property ) ) {
257 257
 			throw new InvalidProperty();
258 258
 		}
259 259
 		return $property;
Please login to merge, or discard this patch.
plugin/Modules/System.php 2 patches
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
 
@@ -157,7 +161,9 @@  discard block
 block discarded – undo
157 161
 		ksort( $settings );
158 162
 		$details = [];
159 163
 		foreach( $settings as $key => $value ) {
160
-			if( $helper->startsWith( 'strings', $key ) && $helper->endsWith( 'id', $key ))continue;
164
+			if( $helper->startsWith( 'strings', $key ) && $helper->endsWith( 'id', $key )) {
165
+				continue;
166
+			}
161 167
 			$value = htmlspecialchars( trim( preg_replace('/\s\s+/', '\\n', $value )), ENT_QUOTES, 'UTF-8' );
162 168
 			$details[$key] = $value;
163 169
 		}
@@ -230,7 +236,9 @@  discard block
 block discarded – undo
230 236
 			'WPE_APIKEY' => 'WP Engine',
231 237
 		];
232 238
 		foreach( $checks as $key => $value ) {
233
-			if( !$this->isWebhostCheckValid( $key ))continue;
239
+			if( !$this->isWebhostCheckValid( $key )) {
240
+				continue;
241
+			}
234 242
 			return $value;
235 243
 		}
236 244
 		return implode( ',', array_filter( [DB_HOST, filter_input( INPUT_SERVER, 'SERVER_NAME' )] ));
@@ -311,7 +319,9 @@  discard block
 block discarded – undo
311 319
 		];
312 320
 		array_walk( $settings, function( &$value, $setting ) use( $keys ) {
313 321
 			foreach( $keys as $key ) {
314
-				if( !glsr( Helper::class )->startsWith( $key, $setting ) || empty( $value ))continue;
322
+				if( !glsr( Helper::class )->startsWith( $key, $setting ) || empty( $value )) {
323
+					continue;
324
+				}
315 325
 				$value = str_repeat( '•', 13 );
316 326
 				return;
317 327
 			}
Please login to merge, or discard this patch.
Spacing   +24 added lines, -24 removed lines patch added patch discarded remove patch
@@ -40,7 +40,7 @@  discard block
 block discarded – undo
40 40
 			'setting' => 'Plugin Settings',
41 41
 			'reviews' => 'Review Counts',
42 42
 		];
43
-		$systemInfo = array_reduce( array_keys( $details ), function( $carry, $key ) use( $details ) {
43
+		$systemInfo = array_reduce( array_keys( $details ), function( $carry, $key ) use($details) {
44 44
 			$methodName = glsr( Helper::class )->buildMethodName( 'get-'.$key.'-details' );
45 45
 			if( method_exists( $this, $methodName ) && $systemDetails = $this->$methodName() ) {
46 46
 				return $carry.$this->implode(
@@ -60,8 +60,8 @@  discard block
 block discarded – undo
60 60
 	{
61 61
 		$plugins = get_plugins();
62 62
 		$activePlugins = (array)get_option( 'active_plugins', [] );
63
-		$inactive = array_diff_key( $plugins, array_flip( $activePlugins ));
64
-		return $this->normalizePluginList( array_diff_key( $plugins, $inactive ));
63
+		$inactive = array_diff_key( $plugins, array_flip( $activePlugins ) );
64
+		return $this->normalizePluginList( array_diff_key( $plugins, $inactive ) );
65 65
 	}
66 66
 
67 67
 	/**
@@ -95,7 +95,7 @@  discard block
 block discarded – undo
95 95
 	public function getInactivePluginDetails()
96 96
 	{
97 97
 		$activePlugins = (array)get_option( 'active_plugins', [] );
98
-		return $this->normalizePluginList( array_diff_key( get_plugins(), array_flip( $activePlugins )));
98
+		return $this->normalizePluginList( array_diff_key( get_plugins(), array_flip( $activePlugins ) ) );
99 99
 	}
100 100
 
101 101
 	/**
@@ -105,9 +105,9 @@  discard block
 block discarded – undo
105 105
 	{
106 106
 		$plugins = array_merge(
107 107
 			get_mu_plugins(),
108
-			get_plugins( '/../'.basename( WPMU_PLUGIN_DIR ))
108
+			get_plugins( '/../'.basename( WPMU_PLUGIN_DIR ) )
109 109
 		);
110
-		if( empty( $plugins ))return;
110
+		if( empty($plugins) )return;
111 111
 		return $this->normalizePluginList( $plugins );
112 112
 	}
113 113
 
@@ -116,7 +116,7 @@  discard block
 block discarded – undo
116 116
 	 */
117 117
 	public function getMultisitePluginDetails()
118 118
 	{
119
-		if( !is_multisite() || empty( get_site_option( 'active_sitewide_plugins', [] )))return;
119
+		if( !is_multisite() || empty(get_site_option( 'active_sitewide_plugins', [] )) )return;
120 120
 		return $this->normalizePluginList( wp_get_active_network_plugins() );
121 121
 	}
122 122
 
@@ -144,11 +144,11 @@  discard block
 block discarded – undo
144 144
 			'Memory Limit' => ini_get( 'memory_limit' ),
145 145
 			'Post Max Size' => ini_get( 'post_max_size' ),
146 146
 			'Sendmail Path' => ini_get( 'sendmail_path' ),
147
-			'Session Cookie Path' => esc_html( ini_get( 'session.cookie_path' )),
148
-			'Session Name' => esc_html( ini_get( 'session.name' )),
149
-			'Session Save Path' => esc_html( ini_get( 'session.save_path' )),
150
-			'Session Use Cookies' => var_export( wp_validate_boolean( ini_get( 'session.use_cookies' )), true ),
151
-			'Session Use Only Cookies' => var_export( wp_validate_boolean( ini_get( 'session.use_only_cookies' )), true ),
147
+			'Session Cookie Path' => esc_html( ini_get( 'session.cookie_path' ) ),
148
+			'Session Name' => esc_html( ini_get( 'session.name' ) ),
149
+			'Session Save Path' => esc_html( ini_get( 'session.save_path' ) ),
150
+			'Session Use Cookies' => var_export( wp_validate_boolean( ini_get( 'session.use_cookies' ) ), true ),
151
+			'Session Use Only Cookies' => var_export( wp_validate_boolean( ini_get( 'session.use_only_cookies' ) ), true ),
152 152
 			'Upload Max Filesize' => ini_get( 'upload_max_filesize' ),
153 153
 		];
154 154
 	}
@@ -160,8 +160,8 @@  discard block
 block discarded – undo
160 160
 	{
161 161
 		$counts = glsr( CountsManager::class )->getCounts();
162 162
 		$counts = glsr( Helper::class )->flattenArray( $counts );
163
- 		array_walk( $counts, function( &$ratings ) use( $counts ) {
164
-			if( !is_array( $ratings )) {
163
+ 		array_walk( $counts, function( &$ratings ) use($counts) {
164
+			if( !is_array( $ratings ) ) {
165 165
 				glsr_log()
166 166
 					->error( '$ratings is not an array, possibly due to incorrectly imported reviews.' )
167 167
 					->debug( $ratings )
@@ -200,8 +200,8 @@  discard block
 block discarded – undo
200 200
 		ksort( $settings );
201 201
 		$details = [];
202 202
 		foreach( $settings as $key => $value ) {
203
-			if( $helper->startsWith( 'strings', $key ) && $helper->endsWith( 'id', $key ))continue;
204
-			$value = htmlspecialchars( trim( preg_replace('/\s\s+/', '\\n', $value )), ENT_QUOTES, 'UTF-8' );
203
+			if( $helper->startsWith( 'strings', $key ) && $helper->endsWith( 'id', $key ) )continue;
204
+			$value = htmlspecialchars( trim( preg_replace( '/\s\s+/', '\\n', $value ) ), ENT_QUOTES, 'UTF-8' );
205 205
 			$details[$key] = $value;
206 206
 		}
207 207
 		return $details;
@@ -214,7 +214,7 @@  discard block
 block discarded – undo
214 214
 	{
215 215
 		return [
216 216
 			'Console size' => glsr( Console::class )->humanSize( '0' ),
217
-			'Last Rating Count' => date_i18n( 'Y-m-d H:i', glsr( OptionManager::class )->get( 'last_review_count' )),
217
+			'Last Rating Count' => date_i18n( 'Y-m-d H:i', glsr( OptionManager::class )->get( 'last_review_count' ) ),
218 218
 			'Version (current)' => glsr()->version,
219 219
 			'Version (previous)' => glsr( OptionManager::class )->get( 'version_upgraded_from' ),
220 220
 		];
@@ -274,10 +274,10 @@  discard block
 block discarded – undo
274 274
 			'WPE_APIKEY' => 'WP Engine',
275 275
 		];
276 276
 		foreach( $checks as $key => $value ) {
277
-			if( !$this->isWebhostCheckValid( $key ))continue;
277
+			if( !$this->isWebhostCheckValid( $key ) )continue;
278 278
 			return $value;
279 279
 		}
280
-		return implode( ',', array_filter( [DB_HOST, filter_input( INPUT_SERVER, 'SERVER_NAME' )] ));
280
+		return implode( ',', array_filter( [DB_HOST, filter_input( INPUT_SERVER, 'SERVER_NAME' )] ) );
281 281
 	}
282 282
 
283 283
 	/**
@@ -298,8 +298,8 @@  discard block
 block discarded – undo
298 298
 	{
299 299
 		$plugins = get_plugins();
300 300
 		$activePlugins = (array)get_option( 'active_plugins', [] );
301
-		$inactive = $this->normalizePluginList( array_diff_key( $plugins, array_flip( $activePlugins )));
302
-		$active = $this->normalizePluginList( array_diff_key( $plugins, $inactive ));
301
+		$inactive = $this->normalizePluginList( array_diff_key( $plugins, array_flip( $activePlugins ) ) );
302
+		$active = $this->normalizePluginList( array_diff_key( $plugins, $inactive ) );
303 303
 		return $active + $inactive;
304 304
 	}
305 305
 
@@ -310,7 +310,7 @@  discard block
 block discarded – undo
310 310
 	protected function implode( $title, array $details )
311 311
 	{
312 312
 		$strings = ['['.$title.']'];
313
-		$padding = max( array_map( 'strlen', array_keys( $details )) );
313
+		$padding = max( array_map( 'strlen', array_keys( $details ) ) );
314 314
 		$padding = max( [$padding, static::PAD] );
315 315
 		foreach( $details as $key => $value ) {
316 316
 			$strings[] = is_string( $key )
@@ -353,9 +353,9 @@  discard block
 block discarded – undo
353 353
 		$keys = [
354 354
 			'licenses.', 'submissions.recaptcha.key', 'submissions.recaptcha.secret',
355 355
 		];
356
-		array_walk( $settings, function( &$value, $setting ) use( $keys ) {
356
+		array_walk( $settings, function( &$value, $setting ) use($keys) {
357 357
 			foreach( $keys as $key ) {
358
-				if( !glsr( Helper::class )->startsWith( $key, $setting ) || empty( $value ))continue;
358
+				if( !glsr( Helper::class )->startsWith( $key, $setting ) || empty($value) )continue;
359 359
 				$value = str_repeat( '•', 13 );
360 360
 				return;
361 361
 			}
Please login to merge, or discard this patch.
views/partials/addons/addon.php 2 patches
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -10,13 +10,13 @@
 block discarded – undo
10 10
 			<p>{{ description }}</p>
11 11
 		</div>
12 12
 		<div class="glsr-addon-footer">
13
-		<?php if( !is_wp_error( validate_plugin( $plugin ))) : ?>
14
-			<?php if( is_plugin_active( $plugin )) : ?>
13
+		<?php if( !is_wp_error( validate_plugin( $plugin ) ) ) : ?>
14
+			<?php if( is_plugin_active( $plugin ) ) : ?>
15 15
 			<span class="glsr-addon-link button button-secondary" disabled>
16 16
 				<?= __( 'Installed', 'site-reviews' ); ?>
17 17
 			</span>
18 18
 			<?php else: ?>
19
-			<a href="<?= wp_nonce_url( self_admin_url( 'plugins.php?action=activate&plugin='.$plugin ), 'activate-plugin_'.$plugin );?>" class="glsr-addon-link button button-secondary">
19
+			<a href="<?= wp_nonce_url( self_admin_url( 'plugins.php?action=activate&plugin='.$plugin ), 'activate-plugin_'.$plugin ); ?>" class="glsr-addon-link button button-secondary">
20 20
 				<?= __( 'Activate', 'site-reviews' ); ?>
21 21
 			</a>
22 22
 			<?php endif; ?>
Please login to merge, or discard this patch.
Braces   +10 added lines, -4 removed lines patch added patch discarded remove patch
@@ -15,14 +15,20 @@
 block discarded – undo
15 15
 			<span class="glsr-addon-link button button-secondary" disabled>
16 16
 				<?= __( 'Installed', 'site-reviews' ); ?>
17 17
 			</span>
18
-			<?php else: ?>
19
-			<a href="<?= wp_nonce_url( self_admin_url( 'plugins.php?action=activate&plugin='.$plugin ), 'activate-plugin_'.$plugin );?>" class="glsr-addon-link button button-secondary">
18
+			<?php else {
19
+	: ?>
20
+			<a href="<?= wp_nonce_url( self_admin_url( 'plugins.php?action=activate&plugin='.$plugin ), 'activate-plugin_'.$plugin );
21
+}
22
+?>" class="glsr-addon-link button button-secondary">
20 23
 				<?= __( 'Activate', 'site-reviews' ); ?>
21 24
 			</a>
22 25
 			<?php endif; ?>
23
-		<?php else: ?>
26
+		<?php else {
27
+	: ?>
24 28
 			<a href="{{ link }}" class="glsr-addon-link glsr-external button button-secondary">
25
-				<?= __( 'More Info', 'site-reviews' ); ?>
29
+				<?= __( 'More Info', 'site-reviews' );
30
+}
31
+?>
26 32
 			</a>
27 33
 		<?php endif; ?>
28 34
 		</div>
Please login to merge, or discard this patch.
activate.php 2 patches
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -60,7 +60,9 @@
 block discarded – undo
60 60
 	 */
61 61
 	public function deactivate( $plugin )
62 62
 	{
63
-		if( $this->isValid() )return;
63
+		if( $this->isValid() ) {
64
+			return;
65
+		}
64 66
 		$pluginSlug = plugin_basename( $this->file );
65 67
 		if( $plugin == $pluginSlug ) {
66 68
 			$this->redirect(); //exit
Please login to merge, or discard this patch.
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -30,7 +30,7 @@  discard block
 block discarded – undo
30 30
 		$this->versions = wp_parse_args( $versions, array(
31 31
 			'php' => static::MIN_PHP_VERSION,
32 32
 			'wordpress' => static::MIN_WORDPRESS_VERSION,
33
-		));
33
+		) );
34 34
 	}
35 35
 
36 36
 	/**
@@ -41,8 +41,8 @@  discard block
 block discarded – undo
41 41
 		if( $this->isValid() ) {
42 42
 			return true;
43 43
 		}
44
-		add_action( 'activated_plugin', array( $this, 'deactivate' ));
45
-		add_action( 'admin_notices', array( $this, 'deactivate' ));
44
+		add_action( 'activated_plugin', array( $this, 'deactivate' ) );
45
+		add_action( 'admin_notices', array( $this, 'deactivate' ) );
46 46
 		return false;
47 47
 	}
48 48
 
@@ -137,7 +137,7 @@  discard block
 block discarded – undo
137 137
 			filter_input( INPUT_GET, 'plugin_status' ),
138 138
 			filter_input( INPUT_GET, 'paged' ),
139 139
 			filter_input( INPUT_GET, 's' )
140
-		)));
140
+		) ) );
141 141
 		exit;
142 142
 	}
143 143
 }
Please login to merge, or discard this patch.
uninstall.php 1 patch
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -3,7 +3,9 @@
 block discarded – undo
3 3
 defined( 'WP_UNINSTALL_PLUGIN' ) || die;
4 4
 
5 5
 require_once __DIR__.'/site-reviews.php';
6
-if( !(new GL_Plugin_Check_v3( __FILE__ ))->isValid() )return;
6
+if( !(new GL_Plugin_Check_v3( __FILE__ ))->isValid() ) {
7
+	return;
8
+}
7 9
 
8 10
 delete_option( GeminiLabs\SiteReviews\Database\OptionManager::databaseKey() );
9 11
 delete_option( 'widget_'.glsr()->id.'_site-reviews' );
Please login to merge, or discard this patch.
plugin/Modules/Rating.php 2 patches
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -90,7 +90,9 @@
 block discarded – undo
90 90
 	{
91 91
 		$total = array_sum( $ratingCounts );
92 92
 		foreach( $ratingCounts as $index => $count ) {
93
-			if( empty( $count ))continue;
93
+			if( empty( $count )) {
94
+				continue;
95
+			}
94 96
 			$ratingCounts[$index] = $count / $total * 100;
95 97
 		}
96 98
 		return $this->getRoundedPercentages( $ratingCounts );
Please login to merge, or discard this patch.
Spacing   +15 added lines, -15 removed lines patch added patch discarded remove patch
@@ -49,9 +49,9 @@  discard block
 block discarded – undo
49 49
 	{
50 50
 		$average = array_sum( $ratingCounts );
51 51
 		if( $average > 0 ) {
52
-			$average = round( $this->getTotalSum( $ratingCounts ) / $average, intval( $roundBy ));
52
+			$average = round( $this->getTotalSum( $ratingCounts ) / $average, intval( $roundBy ) );
53 53
 		}
54
-		return floatval( apply_filters( 'site-reviews/rating/average', $average, $ratingCounts ));
54
+		return floatval( apply_filters( 'site-reviews/rating/average', $average, $ratingCounts ) );
55 55
 	}
56 56
 
57 57
 	/**
@@ -72,7 +72,7 @@  discard block
 block discarded – undo
72 72
 		}
73 73
 		$z = static::CONFIDENCE_LEVEL_Z_SCORES[$confidencePercentage];
74 74
 		$phat = 1 * $upDownCounts[1] / $numRatings;
75
-		return ( $phat + $z * $z / ( 2 * $numRatings ) - $z * sqrt(( $phat * ( 1 - $phat ) + $z * $z / ( 4 * $numRatings )) / $numRatings )) / ( 1 + $z * $z / $numRatings );
75
+		return ($phat + $z * $z / (2 * $numRatings) - $z * sqrt( ($phat * (1 - $phat) + $z * $z / (4 * $numRatings)) / $numRatings )) / (1 + $z * $z / $numRatings);
76 76
 	}
77 77
 
78 78
 	/**
@@ -90,7 +90,7 @@  discard block
 block discarded – undo
90 90
 	{
91 91
 		$total = array_sum( $ratingCounts );
92 92
 		foreach( $ratingCounts as $index => $count ) {
93
-			if( empty( $count ))continue;
93
+			if( empty($count) )continue;
94 94
 			$ratingCounts[$index] = $count / $total * 100;
95 95
 		}
96 96
 		return $this->getRoundedPercentages( $ratingCounts );
@@ -105,7 +105,7 @@  discard block
 block discarded – undo
105 105
 			$this->getRankingUsingImdb( $ratingCounts ),
106 106
 			$ratingCounts,
107 107
 			$this
108
-		));
108
+		) );
109 109
 	}
110 110
 
111 111
 	/**
@@ -123,12 +123,12 @@  discard block
 block discarded – undo
123 123
 		$avgRating = $this->getAverage( $ratingCounts );
124 124
 		// Represents a prior (your prior opinion without data) for the average star rating. A higher prior also means a higher margin for error.
125 125
 		// This could also be the average score of all items instead of a fixed value.
126
-		$bayesMean = ( $confidencePercentage / 100 ) * static::MAX_RATING; // prior, 70% = 3.5
126
+		$bayesMean = ($confidencePercentage / 100) * static::MAX_RATING; // prior, 70% = 3.5
127 127
 		// Represents the number of ratings expected to begin observing a pattern that would put confidence in the prior.
128 128
 		$bayesMinimal = 10; // confidence
129 129
 		$numOfReviews = array_sum( $ratingCounts );
130 130
 		return $avgRating > 0
131
-			? (( $bayesMinimal * $bayesMean ) + ( $avgRating * $numOfReviews )) / ( $bayesMinimal + $numOfReviews )
131
+			? (($bayesMinimal * $bayesMean) + ($avgRating * $numOfReviews)) / ($bayesMinimal + $numOfReviews)
132 132
 			: 0;
133 133
 	}
134 134
 
@@ -148,7 +148,7 @@  discard block
 block discarded – undo
148 148
 		$weight = $this->getWeight( $ratingCounts, $ratingCountsSum );
149 149
 		$weightPow2 = $this->getWeight( $ratingCounts, $ratingCountsSum, true );
150 150
 		$zScore = static::CONFIDENCE_LEVEL_Z_SCORES[$confidencePercentage];
151
-		return $weight - $zScore * sqrt(( $weightPow2 - pow( $weight, 2 )) / ( $ratingCountsSum + 1 ));
151
+		return $weight - $zScore * sqrt( ($weightPow2 - pow( $weight, 2 )) / ($ratingCountsSum + 1) );
152 152
 	}
153 153
 
154 154
 	/**
@@ -168,14 +168,14 @@  discard block
 block discarded – undo
168 168
 		$remainders = glsr_array_column( $percentages, 'remainder' );
169 169
 		array_multisort( $remainders, SORT_DESC, SORT_STRING, $indexes, SORT_DESC, $percentages );
170 170
 		$i = 0;
171
-		if( array_sum( glsr_array_column( $percentages, 'percent' )) > 0 ) {
172
-			while( array_sum( glsr_array_column( $percentages, 'percent' )) < $totalPercent ) {
171
+		if( array_sum( glsr_array_column( $percentages, 'percent' ) ) > 0 ) {
172
+			while( array_sum( glsr_array_column( $percentages, 'percent' ) ) < $totalPercent ) {
173 173
 				$percentages[$i]['percent']++;
174 174
 				$i++;
175 175
 			}
176 176
 		}
177 177
 		array_multisort( $indexes, SORT_DESC, $percentages );
178
-		return array_combine( $indexes, glsr_array_column( $percentages, 'percent' ));
178
+		return array_combine( $indexes, glsr_array_column( $percentages, 'percent' ) );
179 179
 	}
180 180
 
181 181
 	/**
@@ -183,8 +183,8 @@  discard block
 block discarded – undo
183 183
 	 */
184 184
 	protected function getTotalSum( array $ratingCounts )
185 185
 	{
186
-		return array_reduce( array_keys( $ratingCounts ), function( $carry, $index ) use( $ratingCounts ) {
187
-			return $carry + ( $index * $ratingCounts[$index] );
186
+		return array_reduce( array_keys( $ratingCounts ), function( $carry, $index ) use($ratingCounts) {
187
+			return $carry + ($index * $ratingCounts[$index]);
188 188
 		});
189 189
 	}
190 190
 
@@ -196,11 +196,11 @@  discard block
 block discarded – undo
196 196
 	protected function getWeight( array $ratingCounts, $ratingCountsSum, $powerOf2 = false )
197 197
 	{
198 198
 		return array_reduce( array_keys( $ratingCounts ),
199
-			function( $count, $rating ) use( $ratingCounts, $ratingCountsSum, $powerOf2 ) {
199
+			function( $count, $rating ) use($ratingCounts, $ratingCountsSum, $powerOf2) {
200 200
 				$ratingLevel = $powerOf2
201 201
 					? pow( $rating, 2 )
202 202
 					: $rating;
203
-				return $count + ( $ratingLevel * ( $ratingCounts[$rating] + 1 )) / $ratingCountsSum;
203
+				return $count + ($ratingLevel * ($ratingCounts[$rating] + 1)) / $ratingCountsSum;
204 204
 			}
205 205
 		);
206 206
 	}
Please login to merge, or discard this patch.
plugin/Modules/Html/Builder.php 2 patches
Braces   +15 added lines, -5 removed lines patch added patch discarded remove patch
@@ -81,7 +81,9 @@  discard block
 block discarded – undo
81 81
 		];
82 82
 		if( !isset( $properties[$property] )
83 83
 			|| empty( array_filter( [$value], $properties[$property] ))
84
-		)return;
84
+		) {
85
+			return;
86
+		}
85 87
 		$this->$property = $value;
86 88
 	}
87 89
 
@@ -90,7 +92,9 @@  discard block
 block discarded – undo
90 92
 	 */
91 93
 	public function getClosingTag()
92 94
 	{
93
-		if( empty( $this->tag ))return;
95
+		if( empty( $this->tag )) {
96
+			return;
97
+		}
94 98
 		return '</'.$this->tag.'>';
95 99
 	}
96 100
 
@@ -99,7 +103,9 @@  discard block
 block discarded – undo
99 103
 	 */
100 104
 	public function getOpeningTag()
101 105
 	{
102
-		if( empty( $this->tag ))return;
106
+		if( empty( $this->tag )) {
107
+			return;
108
+		}
103 109
 		$attributes = glsr( Attributes::class )->{$this->tag}( $this->args )->toString();
104 110
 		return '<'.trim( $this->tag.' '.$attributes ).'>';
105 111
 	}
@@ -141,7 +147,9 @@  discard block
 block discarded – undo
141 147
 	 */
142 148
 	protected function buildFieldDescription()
143 149
 	{
144
-		if( empty( $this->args['description'] ))return;
150
+		if( empty( $this->args['description'] )) {
151
+			return;
152
+		}
145 153
 		if( $this->args['is_widget'] ) {
146 154
 			return $this->small( $this->args['description'] );
147 155
 		}
@@ -207,7 +215,9 @@  discard block
 block discarded – undo
207 215
 	 */
208 216
 	protected function buildFormLabel( array $customArgs = [] )
209 217
 	{
210
-		if( empty( $this->args['label'] ) || $this->args['type'] == 'hidden' )return;
218
+		if( empty( $this->args['label'] ) || $this->args['type'] == 'hidden' ) {
219
+			return;
220
+		}
211 221
 		return $this->label( wp_parse_args( $customArgs, [
212 222
 			'for' => $this->args['id'],
213 223
 			'is_public' => $this->args['is_public'],
Please login to merge, or discard this patch.
Spacing   +36 added lines, -36 removed lines patch added patch discarded remove patch
@@ -66,7 +66,7 @@  discard block
 block discarded – undo
66 66
 	{
67 67
 		$instance = new static;
68 68
 		$instance->setTagFromMethod( $method );
69
-		call_user_func_array( [$instance, 'normalize'], $args += ['',''] );
69
+		call_user_func_array( [$instance, 'normalize'], $args += ['', ''] );
70 70
 		$tags = array_merge( static::TAGS_FORM, static::TAGS_SINGLE, static::TAGS_STRUCTURE, static::TAGS_TEXT );
71 71
 		do_action_ref_array( 'site-reviews/builder', [&$instance] );
72 72
 		$generatedTag = in_array( $instance->tag, $tags )
@@ -91,8 +91,8 @@  discard block
 block discarded – undo
91 91
 			'render' => 'is_bool',
92 92
 			'tag' => 'is_string',
93 93
 		];
94
-		if( !isset( $properties[$property] )
95
-			|| empty( array_filter( [$value], $properties[$property] ))
94
+		if( !isset($properties[$property])
95
+			|| empty(array_filter( [$value], $properties[$property] ))
96 96
 		)return;
97 97
 		$this->$property = $value;
98 98
 	}
@@ -102,7 +102,7 @@  discard block
 block discarded – undo
102 102
 	 */
103 103
 	public function getClosingTag()
104 104
 	{
105
-		if( empty( $this->tag ))return;
105
+		if( empty($this->tag) )return;
106 106
 		return '</'.$this->tag.'>';
107 107
 	}
108 108
 
@@ -111,8 +111,8 @@  discard block
 block discarded – undo
111 111
 	 */
112 112
 	public function getOpeningTag()
113 113
 	{
114
-		if( empty( $this->tag ))return;
115
-		$attributes = glsr( Attributes::class )->{$this->tag}( $this->args )->toString();
114
+		if( empty($this->tag) )return;
115
+		$attributes = glsr( Attributes::class )->{$this->tag}($this->args)->toString();
116 116
 		return '<'.trim( $this->tag.' '.$attributes ).'>';
117 117
 	}
118 118
 
@@ -121,10 +121,10 @@  discard block
 block discarded – undo
121 121
 	 */
122 122
 	public function getTag()
123 123
 	{
124
-		if( in_array( $this->tag, static::TAGS_SINGLE )) {
124
+		if( in_array( $this->tag, static::TAGS_SINGLE ) ) {
125 125
 			return $this->getOpeningTag();
126 126
 		}
127
-		if( !in_array( $this->tag, static::TAGS_FORM )) {
127
+		if( !in_array( $this->tag, static::TAGS_FORM ) ) {
128 128
 			return $this->buildDefaultTag();
129 129
 		}
130 130
 		return call_user_func( [$this, 'buildForm'.ucfirst( $this->tag )] ).$this->buildFieldDescription();
@@ -135,8 +135,8 @@  discard block
 block discarded – undo
135 135
 	 */
136 136
 	public function raw( array $field )
137 137
 	{
138
-		unset( $field['label'] );
139
-		return $this->{$field['type']}( $field );
138
+		unset($field['label']);
139
+		return $this->{$field['type']}($field);
140 140
 	}
141 141
 
142 142
 	/**
@@ -145,7 +145,7 @@  discard block
 block discarded – undo
145 145
 	protected function buildCustomField()
146 146
 	{
147 147
 		$className = $this->getCustomFieldClassName();
148
-		if( class_exists( $className )) {
148
+		if( class_exists( $className ) ) {
149 149
 			return (new $className( $this ))->build();
150 150
 		}
151 151
 		glsr_log()->error( 'Field missing: '.$className );
@@ -156,7 +156,7 @@  discard block
 block discarded – undo
156 156
 	 */
157 157
 	protected function buildDefaultTag( $text = '' )
158 158
 	{
159
-		if( empty( $text )) {
159
+		if( empty($text) ) {
160 160
 			$text = $this->args['text'];
161 161
 		}
162 162
 		return $this->getOpeningTag().$text.$this->getClosingTag();
@@ -167,7 +167,7 @@  discard block
 block discarded – undo
167 167
 	 */
168 168
 	protected function buildFieldDescription()
169 169
 	{
170
-		if( empty( $this->args['description'] ))return;
170
+		if( empty($this->args['description']) )return;
171 171
 		if( $this->args['is_widget'] ) {
172 172
 			return $this->small( $this->args['description'] );
173 173
 		}
@@ -179,13 +179,13 @@  discard block
 block discarded – undo
179 179
 	 */
180 180
 	protected function buildFormInput()
181 181
 	{
182
-		if( !in_array( $this->args['type'], ['checkbox', 'radio'] )) {
183
-			if( isset( $this->args['multiple'] )) {
184
-				$this->args['name'].= '[]';
182
+		if( !in_array( $this->args['type'], ['checkbox', 'radio'] ) ) {
183
+			if( isset($this->args['multiple']) ) {
184
+				$this->args['name'] .= '[]';
185 185
 			}
186 186
 			return $this->buildFormLabel().$this->getOpeningTag();
187 187
 		}
188
-		return empty( $this->args['options'] )
188
+		return empty($this->args['options'])
189 189
 			? $this->buildFormInputChoice()
190 190
 			: $this->buildFormInputMultiChoice();
191 191
 	}
@@ -195,19 +195,19 @@  discard block
 block discarded – undo
195 195
 	 */
196 196
 	protected function buildFormInputChoice()
197 197
 	{
198
-		if( !empty( $this->args['text'] )) {
198
+		if( !empty($this->args['text']) ) {
199 199
 			$this->args['label'] = $this->args['text'];
200 200
 		}
201 201
 		if( !$this->args['is_public'] ) {
202
-			return $this->buildFormLabel([
202
+			return $this->buildFormLabel( [
203 203
 				'class' => 'glsr-'.$this->args['type'].'-label',
204 204
 				'text' => $this->getOpeningTag().' '.$this->args['label'].'<span></span>',
205
-			]);
205
+			] );
206 206
 		}
207
-		return $this->getOpeningTag().$this->buildFormLabel([
207
+		return $this->getOpeningTag().$this->buildFormLabel( [
208 208
 			'class' => 'glsr-'.$this->args['type'].'-label',
209 209
 			'text' => $this->args['label'].'<span></span>',
210
-		]);
210
+		] );
211 211
 	}
212 212
 
213 213
 	/**
@@ -216,22 +216,22 @@  discard block
 block discarded – undo
216 216
 	protected function buildFormInputMultiChoice()
217 217
 	{
218 218
 		if( $this->args['type'] == 'checkbox' ) {
219
-			$this->args['name'].= '[]';
219
+			$this->args['name'] .= '[]';
220 220
 		}
221 221
 		$index = 0;
222
-		$options = array_reduce( array_keys( $this->args['options'] ), function( $carry, $key ) use( &$index ) {
222
+		$options = array_reduce( array_keys( $this->args['options'] ), function( $carry, $key ) use(&$index) {
223 223
 			return $carry.$this->li( $this->{$this->args['type']}([
224 224
 				'checked' => in_array( $key, (array)$this->args['value'] ),
225 225
 				'id' => $this->args['id'].'-'.$index++,
226 226
 				'name' => $this->args['name'],
227 227
 				'text' => $this->args['options'][$key],
228 228
 				'value' => $key,
229
-			]));
229
+			]) );
230 230
 		});
231 231
 		return $this->ul( $options, [
232 232
 			'class' => $this->args['class'],
233 233
 			'id' => $this->args['id'],
234
-		]);
234
+		] );
235 235
 	}
236 236
 
237 237
 	/**
@@ -239,13 +239,13 @@  discard block
 block discarded – undo
239 239
 	 */
240 240
 	protected function buildFormLabel( array $customArgs = [] )
241 241
 	{
242
-		if( empty( $this->args['label'] ) || $this->args['type'] == 'hidden' )return;
242
+		if( empty($this->args['label']) || $this->args['type'] == 'hidden' )return;
243 243
 		return $this->label( wp_parse_args( $customArgs, [
244 244
 			'for' => $this->args['id'],
245 245
 			'is_public' => $this->args['is_public'],
246 246
 			'text' => $this->args['label'],
247 247
 			'type' => $this->args['type'],
248
-		]));
248
+		] ) );
249 249
 	}
250 250
 
251 251
 	/**
@@ -262,11 +262,11 @@  discard block
 block discarded – undo
262 262
 	protected function buildFormSelectOptions()
263 263
 	{
264 264
 		return array_reduce( array_keys( $this->args['options'] ), function( $carry, $key ) {
265
-			return $carry.$this->option([
265
+			return $carry.$this->option( [
266 266
 				'selected' => $this->args['value'] == $key,
267 267
 				'text' => $this->args['options'][$key],
268 268
 				'value' => $key,
269
-			]);
269
+			] );
270 270
 		});
271 271
 	}
272 272
 
@@ -302,7 +302,7 @@  discard block
 block discarded – undo
302 302
 	protected function mergeArgsWithRequiredDefaults()
303 303
 	{
304 304
 		$className = $this->getCustomFieldClassName();
305
-		if( class_exists( $className )) {
305
+		if( class_exists( $className ) ) {
306 306
 			$this->args = array_merge(
307 307
 				wp_parse_args( $this->args, $className::defaults() ),
308 308
 				$className::required()
@@ -317,16 +317,16 @@  discard block
 block discarded – undo
317 317
 	 */
318 318
 	protected function normalize( ...$params )
319 319
 	{
320
-		if( is_string( $params[0] ) || is_numeric( $params[0] )) {
320
+		if( is_string( $params[0] ) || is_numeric( $params[0] ) ) {
321 321
 			$this->setNameOrTextAttributeForTag( $params[0] );
322 322
 		}
323
-		if( is_array( $params[0] )) {
323
+		if( is_array( $params[0] ) ) {
324 324
 			$this->args += $params[0];
325 325
 		}
326
-		else if( is_array( $params[1] )) {
326
+		else if( is_array( $params[1] ) ) {
327 327
 			$this->args += $params[1];
328 328
 		}
329
-		if( !isset( $this->args['is_public'] )) {
329
+		if( !isset($this->args['is_public']) ) {
330 330
 			$this->args['is_public'] = false;
331 331
 		}
332 332
 	}
@@ -350,7 +350,7 @@  discard block
 block discarded – undo
350 350
 	protected function setTagFromMethod( $method )
351 351
 	{
352 352
 		$this->tag = strtolower( $method );
353
-		if( in_array( $this->tag, static::INPUT_TYPES )) {
353
+		if( in_array( $this->tag, static::INPUT_TYPES ) ) {
354 354
 			$this->args['type'] = $this->tag;
355 355
 			$this->tag = 'input';
356 356
 		}
Please login to merge, or discard this patch.
plugin/Controllers/EditorController/Metaboxes.php 2 patches
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -13,10 +13,10 @@  discard block
 block discarded – undo
13 13
 	 */
14 14
 	public function saveAssignedToMetabox( $postId )
15 15
 	{
16
-		if( !wp_verify_nonce( glsr( Helper::class )->filterInput( '_nonce-assigned-to' ), 'assigned_to' ))return;
17
-		$assignedTo = strval( glsr( Helper::class )->filterInput( 'assigned_to' ));
16
+		if( !wp_verify_nonce( glsr( Helper::class )->filterInput( '_nonce-assigned-to' ), 'assigned_to' ) )return;
17
+		$assignedTo = strval( glsr( Helper::class )->filterInput( 'assigned_to' ) );
18 18
 		if( get_post_meta( $postId, 'assigned_to', true ) != $assignedTo ) {
19
-			$review = glsr( ReviewManager::class )->single( get_post( $postId ));
19
+			$review = glsr( ReviewManager::class )->single( get_post( $postId ) );
20 20
 			glsr( CountsManager::class )->decreasePostCounts( $review );
21 21
 		}
22 22
 		update_post_meta( $postId, 'assigned_to', $assignedTo );
@@ -28,12 +28,12 @@  discard block
 block discarded – undo
28 28
 	 */
29 29
 	public function saveResponseMetabox( $postId )
30 30
 	{
31
-		if( !wp_verify_nonce( glsr( Helper::class )->filterInput( '_nonce-response' ), 'response' ))return;
32
-		$response = strval( glsr( Helper::class )->filterInput( 'response' ));
31
+		if( !wp_verify_nonce( glsr( Helper::class )->filterInput( '_nonce-response' ), 'response' ) )return;
32
+		$response = strval( glsr( Helper::class )->filterInput( 'response' ) );
33 33
 		update_post_meta( $postId, 'response', trim( wp_kses( $response, [
34 34
 			'a' => ['href' => [], 'title' => []],
35 35
 			'em' => [],
36 36
 			'strong' => [],
37
-		])));
37
+		] ) ) );
38 38
 	}
39 39
 }
Please login to merge, or discard this patch.
Braces   +6 added lines, -2 removed lines patch added patch discarded remove patch
@@ -13,7 +13,9 @@  discard block
 block discarded – undo
13 13
 	 */
14 14
 	public function saveAssignedToMetabox( $postId )
15 15
 	{
16
-		if( !wp_verify_nonce( glsr( Helper::class )->filterInput( '_nonce-assigned-to' ), 'assigned_to' ))return;
16
+		if( !wp_verify_nonce( glsr( Helper::class )->filterInput( '_nonce-assigned-to' ), 'assigned_to' )) {
17
+			return;
18
+		}
17 19
 		$assignedTo = strval( glsr( Helper::class )->filterInput( 'assigned_to' ));
18 20
 		if( get_post_meta( $postId, 'assigned_to', true ) != $assignedTo ) {
19 21
 			$review = glsr( ReviewManager::class )->single( get_post( $postId ));
@@ -28,7 +30,9 @@  discard block
 block discarded – undo
28 30
 	 */
29 31
 	public function saveResponseMetabox( $postId )
30 32
 	{
31
-		if( !wp_verify_nonce( glsr( Helper::class )->filterInput( '_nonce-response' ), 'response' ))return;
33
+		if( !wp_verify_nonce( glsr( Helper::class )->filterInput( '_nonce-response' ), 'response' )) {
34
+			return;
35
+		}
32 36
 		$response = strval( glsr( Helper::class )->filterInput( 'response' ));
33 37
 		update_post_meta( $postId, 'response', trim( wp_kses( $response, [
34 38
 			'a' => ['href' => [], 'title' => []],
Please login to merge, or discard this patch.