Passed
Push — master ( ed8579...f866b8 )
by Paul
05:30
created
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   +21 added lines, -21 removed lines patch added patch discarded remove patch
@@ -38,7 +38,7 @@  discard block
 block discarded – undo
38 38
 			'setting' => 'Plugin Settings',
39 39
 			'reviews' => 'Review Counts',
40 40
 		];
41
-		$systemInfo = array_reduce( array_keys( $details ), function( $carry, $key ) use( $details ) {
41
+		$systemInfo = array_reduce( array_keys( $details ), function( $carry, $key ) use($details) {
42 42
 			$methodName = glsr( Helper::class )->buildMethodName( 'get-'.$key.'-details' );
43 43
 			if( method_exists( $this, $methodName ) && $systemDetails = $this->$methodName() ) {
44 44
 				return $carry.$this->implode( strtoupper( $details[$key] ), $systemDetails );
@@ -55,8 +55,8 @@  discard block
 block discarded – undo
55 55
 	{
56 56
 		$plugins = get_plugins();
57 57
 		$activePlugins = (array)get_option( 'active_plugins', [] );
58
-		$inactive = array_diff_key( $plugins, array_flip( $activePlugins ));
59
-		return $this->normalizePluginList( array_diff_key( $plugins, $inactive ));
58
+		$inactive = array_diff_key( $plugins, array_flip( $activePlugins ) );
59
+		return $this->normalizePluginList( array_diff_key( $plugins, $inactive ) );
60 60
 	}
61 61
 
62 62
 	/**
@@ -80,7 +80,7 @@  discard block
 block discarded – undo
80 80
 	public function getInactivePluginDetails()
81 81
 	{
82 82
 		$activePlugins = (array)get_option( 'active_plugins', [] );
83
-		return $this->normalizePluginList( array_diff_key( get_plugins(), array_flip( $activePlugins )));
83
+		return $this->normalizePluginList( array_diff_key( get_plugins(), array_flip( $activePlugins ) ) );
84 84
 	}
85 85
 
86 86
 	/**
@@ -90,9 +90,9 @@  discard block
 block discarded – undo
90 90
 	{
91 91
 		$plugins = array_merge(
92 92
 			get_mu_plugins(),
93
-			get_plugins( '/../'.basename( WPMU_PLUGIN_DIR ))
93
+			get_plugins( '/../'.basename( WPMU_PLUGIN_DIR ) )
94 94
 		);
95
-		if( empty( $plugins ))return;
95
+		if( empty($plugins) )return;
96 96
 		return $this->normalizePluginList( $plugins );
97 97
 	}
98 98
 
@@ -101,7 +101,7 @@  discard block
 block discarded – undo
101 101
 	 */
102 102
 	public function getMultisitePluginDetails()
103 103
 	{
104
-		if( !is_multisite() || empty( get_site_option( 'active_sitewide_plugins', [] )))return;
104
+		if( !is_multisite() || empty(get_site_option( 'active_sitewide_plugins', [] )) )return;
105 105
 		return $this->normalizePluginList( wp_get_active_network_plugins() );
106 106
 	}
107 107
 
@@ -127,11 +127,11 @@  discard block
 block discarded – undo
127 127
 			'Max Input Vars' => ini_get( 'max_input_vars' ),
128 128
 			'Memory Limit' => ini_get( 'memory_limit' ),
129 129
 			'Post Max Size' => ini_get( 'post_max_size' ),
130
-			'Session Cookie Path' => esc_html( ini_get( 'session.cookie_path' )),
131
-			'Session Name' => esc_html( ini_get( 'session.name' )),
132
-			'Session Save Path' => esc_html( ini_get( 'session.save_path' )),
133
-			'Session Use Cookies' => var_export( wp_validate_boolean( ini_get( 'session.use_cookies' )), true ),
134
-			'Session Use Only Cookies' => var_export( wp_validate_boolean( ini_get( 'session.use_only_cookies' )), true ),
130
+			'Session Cookie Path' => esc_html( ini_get( 'session.cookie_path' ) ),
131
+			'Session Name' => esc_html( ini_get( 'session.name' ) ),
132
+			'Session Save Path' => esc_html( ini_get( 'session.save_path' ) ),
133
+			'Session Use Cookies' => var_export( wp_validate_boolean( ini_get( 'session.use_cookies' ) ), true ),
134
+			'Session Use Only Cookies' => var_export( wp_validate_boolean( ini_get( 'session.use_only_cookies' ) ), true ),
135 135
 			'Upload Max Filesize' => ini_get( 'upload_max_filesize' ),
136 136
 		];
137 137
 	}
@@ -176,8 +176,8 @@  discard block
 block discarded – undo
176 176
 		ksort( $settings );
177 177
 		$details = [];
178 178
 		foreach( $settings as $key => $value ) {
179
-			if( $helper->startsWith( 'strings', $key ) && $helper->endsWith( 'id', $key ))continue;
180
-			$value = htmlspecialchars( trim( preg_replace('/\s\s+/', '\\n', $value )), ENT_QUOTES, 'UTF-8' );
179
+			if( $helper->startsWith( 'strings', $key ) && $helper->endsWith( 'id', $key ) )continue;
180
+			$value = htmlspecialchars( trim( preg_replace( '/\s\s+/', '\\n', $value ) ), ENT_QUOTES, 'UTF-8' );
181 181
 			$details[$key] = $value;
182 182
 		}
183 183
 		return $details;
@@ -249,10 +249,10 @@  discard block
 block discarded – undo
249 249
 			'WPE_APIKEY' => 'WP Engine',
250 250
 		];
251 251
 		foreach( $checks as $key => $value ) {
252
-			if( !$this->isWebhostCheckValid( $key ))continue;
252
+			if( !$this->isWebhostCheckValid( $key ) )continue;
253 253
 			return $value;
254 254
 		}
255
-		return implode( ',', array_filter( [DB_HOST, filter_input( INPUT_SERVER, 'SERVER_NAME' )] ));
255
+		return implode( ',', array_filter( [DB_HOST, filter_input( INPUT_SERVER, 'SERVER_NAME' )] ) );
256 256
 	}
257 257
 
258 258
 	/**
@@ -273,8 +273,8 @@  discard block
 block discarded – undo
273 273
 	{
274 274
 		$plugins = get_plugins();
275 275
 		$activePlugins = (array)get_option( 'active_plugins', [] );
276
-		$inactive = $this->normalizePluginList( array_diff_key( $plugins, array_flip( $activePlugins )));
277
-		$active = $this->normalizePluginList( array_diff_key( $plugins, $inactive ));
276
+		$inactive = $this->normalizePluginList( array_diff_key( $plugins, array_flip( $activePlugins ) ) );
277
+		$active = $this->normalizePluginList( array_diff_key( $plugins, $inactive ) );
278 278
 		return $active + $inactive;
279 279
 	}
280 280
 
@@ -285,7 +285,7 @@  discard block
 block discarded – undo
285 285
 	protected function implode( $title, array $details )
286 286
 	{
287 287
 		$strings = ['['.$title.']'];
288
-		$padding = max( array_map( 'strlen', array_keys( $details )) );
288
+		$padding = max( array_map( 'strlen', array_keys( $details ) ) );
289 289
 		$padding = max( [$padding, static::PAD] );
290 290
 		foreach( $details as $key => $value ) {
291 291
 			$strings[] = is_string( $key )
@@ -328,9 +328,9 @@  discard block
 block discarded – undo
328 328
 		$keys = [
329 329
 			'licenses.', 'submissions.recaptcha.key', 'submissions.recaptcha.secret',
330 330
 		];
331
-		array_walk( $settings, function( &$value, $setting ) use( $keys ) {
331
+		array_walk( $settings, function( &$value, $setting ) use($keys) {
332 332
 			foreach( $keys as $key ) {
333
-				if( !glsr( Helper::class )->startsWith( $key, $setting ) || empty( $value ))continue;
333
+				if( !glsr( Helper::class )->startsWith( $key, $setting ) || empty($value) )continue;
334 334
 				$value = str_repeat( '•', 13 );
335 335
 				return;
336 336
 			}
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.
views/pages/tools/sync.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -38,7 +38,7 @@
 block discarded – undo
38 38
 <form method="post">
39 39
 	<div class="tablenav bottom">
40 40
 		<div class="actions">
41
-			<?php if( count( $sites ) > 1) : ?>
41
+			<?php if( count( $sites ) > 1 ) : ?>
42 42
 			<label for="bulk-action-selector-bottom" class="screen-reader-text">Select a site to sync</label>
43 43
 			<select name="{{ id }}[site]">
44 44
 				<option>- <?= __( 'Select a Site', 'site-reviews' ); ?> -</option>
Please login to merge, or discard this patch.
Braces   +5 added lines, -2 removed lines patch added patch discarded remove patch
@@ -46,9 +46,12 @@
 block discarded – undo
46 46
 				<option value="<?= $slug; ?>" class="hide-if-no-js"><?= $details['name']; ?></option>
47 47
 				<?php endforeach; ?>
48 48
 			</select>
49
-			<?php else: ?>
49
+			<?php else {
50
+	: ?>
50 51
 				<?php foreach( $sites as $slug => $details ) : ?>
51
-				<input type="hidden" name="{{ id }}[site]" value="<?= $slug; ?>">
52
+				<input type="hidden" name="{{ id }}[site]" value="<?= $slug;
53
+}
54
+?>">
52 55
 				<?php endforeach; ?>
53 56
 			<?php endif; ?>
54 57
 			<input type="hidden" name="{{ id }}[action]" value="sync-reviews">
Please login to merge, or discard this patch.
views/pages/addons/index.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -13,7 +13,7 @@  discard block
 block discarded – undo
13 13
 				'title' => 'Tripadvisor Reviews',
14 14
 			],
15 15
 			'plugin' => 'site-reviews-tripadvisor/site-reviews-tripadvisor.php',
16
-		]);
16
+		] );
17 17
 		$template->render( 'partials/addons/addon', [
18 18
 			'context' => [
19 19
 				'description' => __( 'Sync your Yelp reviews and display them on your site.', 'site-reviews' ),
@@ -22,7 +22,7 @@  discard block
 block discarded – undo
22 22
 				'title' => 'Yelp Reviews',
23 23
 			],
24 24
 			'plugin' => 'site-reviews-yelp/site-reviews-yelp.php',
25
-		]);
25
+		] );
26 26
 	?>
27 27
 	</div>
28 28
 </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/Schema.php 2 patches
Braces   +12 added lines, -4 removed lines patch added patch discarded remove patch
@@ -34,7 +34,9 @@  discard block
 block discarded – undo
34 34
 		foreach( glsr( ReviewManager::class )->get( $this->args )->results as $review ) {
35 35
 			// Only include critic reviews that have been directly produced by your site, not reviews from third- party sites or syndicated reviews.
36 36
 			// @see https://developers.google.com/search/docs/data-types/review
37
-			if( $review->review_type != 'local' )continue;
37
+			if( $review->review_type != 'local' ) {
38
+				continue;
39
+			}
38 40
 			$reviews[] = $this->buildReview( $review );
39 41
 		}
40 42
 		if( !empty( $reviews )) {
@@ -79,7 +81,9 @@  discard block
 block discarded – undo
79 81
 	 */
80 82
 	public function render()
81 83
 	{
82
-		if( is_null( glsr()->schemas ))return;
84
+		if( is_null( glsr()->schemas )) {
85
+			return;
86
+		}
83 87
 		printf( '<script type="application/ld+json">%s</script>', json_encode(
84 88
 			apply_filters( 'site-reviews/schema/all', glsr()->schemas ),
85 89
 			JSON_UNESCAPED_UNICODE | JSON_UNESCAPED_SLASHES
@@ -131,7 +135,9 @@  discard block
 block discarded – undo
131 135
 	{
132 136
 		foreach( $values as $value ) {
133 137
 			$option = $this->getSchemaOptionValue( $value );
134
-			if( empty( $option ))continue;
138
+			if( empty( $option )) {
139
+				continue;
140
+			}
135 141
 			$schema->$value( $option );
136 142
 		}
137 143
 		return $schema;
@@ -237,7 +243,9 @@  discard block
 block discarded – undo
237 243
 		if( $value != $fallback ) {
238 244
 			return $value;
239 245
 		}
240
-		if( !is_single() && !is_page() )return;
246
+		if( !is_single() && !is_page() ) {
247
+			return;
248
+		}
241 249
 		$method = glsr( Helper::class )->buildMethodName( $option, 'getThing' );
242 250
 		if( method_exists( $this, $method )) {
243 251
 			return $this->$method();
Please login to merge, or discard this patch.
Spacing   +32 added lines, -32 removed lines patch added patch discarded remove patch
@@ -38,10 +38,10 @@  discard block
 block discarded – undo
38 38
 			if( $review->review_type != 'local' )continue;
39 39
 			$reviews[] = $this->buildReview( $review );
40 40
 		}
41
-		if( !empty( $reviews )) {
41
+		if( !empty($reviews) ) {
42 42
 			array_walk( $reviews, function( &$review ) {
43
-				unset( $review['@context'] );
44
-				unset( $review['itemReviewed'] );
43
+				unset($review['@context']);
44
+				unset($review['itemReviewed']);
45 45
 			});
46 46
 			$schema['review'] = $reviews;
47 47
 		}
@@ -54,7 +54,7 @@  discard block
 block discarded – undo
54 54
 	 */
55 55
 	public function buildSummary( $args = null )
56 56
 	{
57
-		if( is_array( $args )) {
57
+		if( is_array( $args ) ) {
58 58
 			$this->args = $args;
59 59
 		}
60 60
 		$buildSummary = glsr( Helper::class )->buildMethodName( $this->getSchemaOptionValue( 'type' ), 'buildSummaryFor' );
@@ -62,7 +62,7 @@  discard block
 block discarded – undo
62 62
 		$schema = method_exists( $this, $buildSummary )
63 63
 			? $this->$buildSummary()
64 64
 			: $this->buildSummaryForCustom();
65
-		if( !empty( $count )) {
65
+		if( !empty($count) ) {
66 66
 			$schema->aggregateRating(
67 67
 				$this->getSchemaType( 'AggregateRating' )
68 68
 					->ratingValue( $this->getRatingValue() )
@@ -80,11 +80,11 @@  discard block
 block discarded – undo
80 80
 	 */
81 81
 	public function render()
82 82
 	{
83
-		if( is_null( glsr()->schemas ))return;
83
+		if( is_null( glsr()->schemas ) )return;
84 84
 		printf( '<script type="application/ld+json">%s</script>', json_encode(
85 85
 			apply_filters( 'site-reviews/schema/all', glsr()->schemas ),
86 86
 			JSON_UNESCAPED_UNICODE | JSON_UNESCAPED_SLASHES
87
-		));
87
+		) );
88 88
 	}
89 89
 
90 90
 	/**
@@ -94,7 +94,7 @@  discard block
 block discarded – undo
94 94
 	{
95 95
 		$schemas = glsr()->schemas;
96 96
 		$schemas[] = $schema;
97
-		glsr()->schemas = array_map( 'unserialize', array_unique( array_map( 'serialize', $schemas )));
97
+		glsr()->schemas = array_map( 'unserialize', array_unique( array_map( 'serialize', $schemas ) ) );
98 98
 	}
99 99
 
100 100
 	/**
@@ -104,16 +104,16 @@  discard block
 block discarded – undo
104 104
 	protected function buildReview( $review )
105 105
 	{
106 106
 		$schema = $this->getSchemaType( 'Review' )
107
-			->doIf( !in_array( 'title', $this->args['hide'] ), function( $schema ) use( $review ) {
107
+			->doIf( !in_array( 'title', $this->args['hide'] ), function( $schema ) use($review) {
108 108
 				$schema->name( $review->title );
109 109
 			})
110
-			->doIf( !in_array( 'excerpt', $this->args['hide'] ), function( $schema ) use( $review ) {
110
+			->doIf( !in_array( 'excerpt', $this->args['hide'] ), function( $schema ) use($review) {
111 111
 				$schema->reviewBody( $review->content );
112 112
 			})
113
-			->datePublished(( new DateTime( $review->date )))
114
-			->author( $this->getSchemaType( 'Person' )->name( $review->author ))
115
-			->itemReviewed( $this->getSchemaType()->name( $this->getSchemaOptionValue( 'name' )));
116
-		if( !empty( $review->rating )) {
113
+			->datePublished( (new DateTime( $review->date )) )
114
+			->author( $this->getSchemaType( 'Person' )->name( $review->author ) )
115
+			->itemReviewed( $this->getSchemaType()->name( $this->getSchemaOptionValue( 'name' ) ) );
116
+		if( !empty($review->rating) ) {
117 117
 			$schema->reviewRating(
118 118
 				$this->getSchemaType( 'Rating' )
119 119
 					->ratingValue( $review->rating )
@@ -132,7 +132,7 @@  discard block
 block discarded – undo
132 132
 	{
133 133
 		foreach( $values as $value ) {
134 134
 			$option = $this->getSchemaOptionValue( $value );
135
-			if( empty( $option ))continue;
135
+			if( empty($option) )continue;
136 136
 			$schema->$value( $option );
137 137
 		}
138 138
 		return $schema;
@@ -145,7 +145,7 @@  discard block
 block discarded – undo
145 145
 	{
146 146
 		return $this->buildSchemaValues( $this->getSchemaType(), [
147 147
 			'description', 'image', 'name', 'url',
148
-		]);
148
+		] );
149 149
 	}
150 150
 
151 151
 	/**
@@ -155,7 +155,7 @@  discard block
 block discarded – undo
155 155
 	{
156 156
 		return $this->buildSchemaValues( $this->buildSummaryForCustom(), [
157 157
 			'address', 'priceRange', 'telephone',
158
-		]);
158
+		] );
159 159
 	}
160 160
 
161 161
 	/**
@@ -165,10 +165,10 @@  discard block
 block discarded – undo
165 165
 	{
166 166
 		$offers = $this->buildSchemaValues( $this->getSchemaType( 'AggregateOffer' ), [
167 167
 			'highPrice', 'lowPrice', 'priceCurrency',
168
-		]);
168
+		] );
169 169
 		return $this->buildSummaryForCustom()
170 170
 			->offers( $offers )
171
-			->setProperty( '@id', $this->getSchemaOptionValue( 'url' ));
171
+			->setProperty( '@id', $this->getSchemaOptionValue( 'url' ) );
172 172
 	}
173 173
 
174 174
 	/**
@@ -176,14 +176,14 @@  discard block
 block discarded – undo
176 176
 	 */
177 177
 	protected function getRatingCounts()
178 178
 	{
179
-		if( !isset( $this->ratingCounts )) {
180
-			$counts = glsr( CountsManager::class )->get([
179
+		if( !isset($this->ratingCounts) ) {
180
+			$counts = glsr( CountsManager::class )->get( [
181 181
 				'post_ids' => glsr( Helper::class )->convertStringToArray( $this->args['assigned_to'] ),
182 182
 				'term_ids' => glsr( ReviewManager::class )->normalizeTerms( $this->args['category'] ),
183
-			]);
183
+			] );
184 184
 			$this->ratingCounts = glsr( CountsManager::class )->flatten( $counts, [
185 185
 				'min' => $this->args['rating'],
186
-			]);
186
+			] );
187 187
 		}
188 188
 		return $this->ratingCounts;
189 189
 	}
@@ -204,14 +204,14 @@  discard block
 block discarded – undo
204 204
 	protected function getSchemaOption( $option, $fallback )
205 205
 	{
206 206
 		$option = strtolower( $option );
207
-		if( $schemaOption = trim( (string)get_post_meta( intval( get_the_ID() ), 'schema_'.$option, true ))) {
207
+		if( $schemaOption = trim( (string)get_post_meta( intval( get_the_ID() ), 'schema_'.$option, true ) ) ) {
208 208
 			return $schemaOption;
209 209
 		}
210 210
 		$setting = glsr( OptionManager::class )->get( 'settings.schema.'.$option );
211
-		if( is_array( $setting )) {
211
+		if( is_array( $setting ) ) {
212 212
 			return $this->getSchemaOptionDefault( $setting, $fallback );
213 213
 		}
214
-		return !empty( $setting )
214
+		return !empty($setting)
215 215
 			? $setting
216 216
 			: $fallback;
217 217
 	}
@@ -225,7 +225,7 @@  discard block
 block discarded – undo
225 225
 		$setting = wp_parse_args( $setting, [
226 226
 			'custom' => '',
227 227
 			'default' => $fallback,
228
-		]);
228
+		] );
229 229
 		return $setting['default'] != 'custom'
230 230
 			? $setting['default']
231 231
 			: $setting['custom'];
@@ -244,7 +244,7 @@  discard block
 block discarded – undo
244 244
 		}
245 245
 		if( !is_single() && !is_page() )return;
246 246
 		$method = glsr( Helper::class )->buildMethodName( $option, 'getThing' );
247
-		if( method_exists( $this, $method )) {
247
+		if( method_exists( $this, $method ) ) {
248 248
 			return $this->$method();
249 249
 		}
250 250
 	}
@@ -255,7 +255,7 @@  discard block
 block discarded – undo
255 255
 	 */
256 256
 	protected function getSchemaType( $type = null )
257 257
 	{
258
-		if( !is_string( $type )) {
258
+		if( !is_string( $type ) ) {
259 259
 			$type = $this->getSchemaOption( 'type', 'LocalBusiness' );
260 260
 		}
261 261
 		$className = glsr( Helper::class )->buildClassName( $type, 'Modules\Schema' );
@@ -270,11 +270,11 @@  discard block
 block discarded – undo
270 270
 	protected function getThingDescription()
271 271
 	{
272 272
 		$post = get_post();
273
-		if( !( $post instanceof WP_Post )) {
273
+		if( !($post instanceof WP_Post) ) {
274 274
 			return '';
275 275
 		}
276
-		$text = strip_shortcodes( wp_strip_all_tags( $post->post_excerpt ));
277
-		return wp_trim_words( $text, apply_filters( 'excerpt_length', 55 ));
276
+		$text = strip_shortcodes( wp_strip_all_tags( $post->post_excerpt ) );
277
+		return wp_trim_words( $text, apply_filters( 'excerpt_length', 55 ) );
278 278
 	}
279 279
 
280 280
 	/**
Please login to merge, or discard this patch.
plugin/Controllers/ListTableController.php 2 patches
Spacing   +25 added lines, -25 removed lines patch added patch discarded remove patch
@@ -21,11 +21,11 @@  discard block
 block discarded – undo
21 21
 	 */
22 22
 	public function approve()
23 23
 	{
24
-		check_admin_referer( 'approve-review_'.( $postId = $this->getPostId() ));
25
-		wp_update_post([
24
+		check_admin_referer( 'approve-review_'.($postId = $this->getPostId()) );
25
+		wp_update_post( [
26 26
 			'ID' => $postId,
27 27
 			'post_status' => 'publish',
28
-		]);
28
+		] );
29 29
 		wp_safe_redirect( wp_get_referer() );
30 30
 		exit;
31 31
 	}
@@ -54,11 +54,11 @@  discard block
 block discarded – undo
54 54
 	{
55 55
 		$postTypeColumns = glsr()->postTypeColumns[Application::POST_TYPE];
56 56
 		foreach( $postTypeColumns as $key => &$value ) {
57
-			if( !array_key_exists( $key, $columns ) || !empty( $value ))continue;
57
+			if( !array_key_exists( $key, $columns ) || !empty($value) )continue;
58 58
 			$value = $columns[$key];
59 59
 		}
60
-		if( count( glsr( Database::class )->getReviewsMeta( 'review_type' )) < 2 ) {
61
-			unset( $postTypeColumns['review_type'] );
60
+		if( count( glsr( Database::class )->getReviewsMeta( 'review_type' ) ) < 2 ) {
61
+			unset($postTypeColumns['review_type']);
62 62
 		}
63 63
 		return array_filter( $postTypeColumns, 'strlen' );
64 64
 	}
@@ -94,7 +94,7 @@  discard block
 block discarded – undo
94 94
 	 */
95 95
 	public function filterPostStates( array $postStates, WP_Post $post ) {
96 96
 		if( $post->post_type == Application::POST_TYPE
97
-			&& array_key_exists( 'pending', $postStates )) {
97
+			&& array_key_exists( 'pending', $postStates ) ) {
98 98
 			$postStates['pending'] = __( 'Unapproved', 'site-reviews' );
99 99
 		}
100 100
 		return $postStates;
@@ -109,7 +109,7 @@  discard block
 block discarded – undo
109 109
 		if( $post->post_type != Application::POST_TYPE || $post->post_status == 'trash' ) {
110 110
 			return $actions;
111 111
 		}
112
-		unset( $actions['inline hide-if-no-js'] ); //Remove Quick-edit
112
+		unset($actions['inline hide-if-no-js']); //Remove Quick-edit
113 113
 		$rowActions = [
114 114
 			'approve' => esc_attr__( 'Approve', 'site-reviews' ),
115 115
 			'unapprove' => esc_attr__( 'Unapprove', 'site-reviews' ),
@@ -123,7 +123,7 @@  discard block
 block discarded – undo
123 123
 					admin_url( 'post.php?post='.$post->ID.'&action='.$key ),
124 124
 					$key.'-review_'.$post->ID
125 125
 				),
126
-			]);
126
+			] );
127 127
 		}
128 128
 		return $newActions + $actions;
129 129
 	}
@@ -135,9 +135,9 @@  discard block
 block discarded – undo
135 135
 	public function filterSortableColumns( array $columns )
136 136
 	{
137 137
 		$postTypeColumns = glsr()->postTypeColumns[Application::POST_TYPE];
138
-		unset( $postTypeColumns['cb'] );
138
+		unset($postTypeColumns['cb']);
139 139
 		foreach( $postTypeColumns as $key => $value ) {
140
-			if( glsr( Helper::class )->startsWith( 'taxonomy', $key ))continue;
140
+			if( glsr( Helper::class )->startsWith( 'taxonomy', $key ) )continue;
141 141
 			$columns[$key] = $key;
142 142
 		}
143 143
 		return $columns;
@@ -155,18 +155,18 @@  discard block
 block discarded – undo
155 155
 	 */
156 156
 	public function filterStatusText( $translation, $single, $plural, $number, $domain )
157 157
 	{
158
-		if( $this->canModifyTranslation( $domain )) {
158
+		if( $this->canModifyTranslation( $domain ) ) {
159 159
 			$strings = [
160 160
 				'Published' => __( 'Approved', 'site-reviews' ),
161 161
 				'Pending' => __( 'Unapproved', 'site-reviews' ),
162 162
 			];
163 163
 			foreach( $strings as $search => $replace ) {
164 164
 				if( strpos( $single, $search ) === false )continue;
165
-				$translation = $this->getTranslation([
165
+				$translation = $this->getTranslation( [
166 166
 					'number' => $number,
167 167
 					'plural' => str_replace( $search, $replace, $plural ),
168 168
 					'single' => str_replace( $search, $replace, $single ),
169
-				]);
169
+				] );
170 170
 			}
171 171
 		}
172 172
 		return $translation;
@@ -213,9 +213,9 @@  discard block
 block discarded – undo
213 213
 	 */
214 214
 	public function saveBulkEditFields( $postId )
215 215
 	{
216
-		if( !current_user_can( 'edit_posts' ))return;
216
+		if( !current_user_can( 'edit_posts' ) )return;
217 217
 		$assignedTo = filter_input( INPUT_GET, 'assigned_to' );
218
-		if( $assignedTo && get_post( $assignedTo )) {
218
+		if( $assignedTo && get_post( $assignedTo ) ) {
219 219
 			update_post_meta( $postId, 'assigned_to', $assignedTo );
220 220
 		}
221 221
 	}
@@ -226,10 +226,10 @@  discard block
 block discarded – undo
226 226
 	 */
227 227
 	public function setQueryForColumn( WP_Query $query )
228 228
 	{
229
-		if( !$this->hasPermission( $query ))return;
229
+		if( !$this->hasPermission( $query ) )return;
230 230
 		$this->setMetaQuery( $query, [
231 231
 			'rating', 'review_type',
232
-		]);
232
+		] );
233 233
 		$this->setOrderby( $query );
234 234
 	}
235 235
 
@@ -239,11 +239,11 @@  discard block
 block discarded – undo
239 239
 	 */
240 240
 	public function unapprove()
241 241
 	{
242
-		check_admin_referer( 'unapprove-review_'.( $postId = $this->getPostId() ));
243
-		wp_update_post([
242
+		check_admin_referer( 'unapprove-review_'.($postId = $this->getPostId()) );
243
+		wp_update_post( [
244 244
 			'ID' => $postId,
245 245
 			'post_status' => 'pending',
246
-		]);
246
+		] );
247 247
 		wp_safe_redirect( wp_get_referer() );
248 248
 		exit;
249 249
 	}
@@ -273,7 +273,7 @@  discard block
 block discarded – undo
273 273
 			'single' => '',
274 274
 			'text' => '',
275 275
 		];
276
-		$args = (object) wp_parse_args( $args, $defaults );
276
+		$args = (object)wp_parse_args( $args, $defaults );
277 277
 		$translations = get_translations_for_domain( Application::ID );
278 278
 		return $args->text
279 279
 			? $translations->translate( $args->text )
@@ -298,7 +298,7 @@  discard block
 block discarded – undo
298 298
 	protected function setMetaQuery( WP_Query $query, array $metaKeys )
299 299
 	{
300 300
 		foreach( $metaKeys as $key ) {
301
-			if( !( $value = filter_input( INPUT_GET, $key )))continue;
301
+			if( !($value = filter_input( INPUT_GET, $key )) )continue;
302 302
 			$metaQuery = (array)$query->get( 'meta_query' );
303 303
 			$metaQuery[] = [
304 304
 				'key' => $key,
@@ -315,8 +315,8 @@  discard block
 block discarded – undo
315 315
 	{
316 316
 		$orderby = $query->get( 'orderby' );
317 317
 		$columns = glsr()->postTypeColumns[Application::POST_TYPE];
318
-		unset( $columns['cb'], $columns['title'], $columns['date'] );
319
-		if( in_array( $orderby, array_keys( $columns ))) {
318
+		unset($columns['cb'], $columns['title'], $columns['date']);
319
+		if( in_array( $orderby, array_keys( $columns ) ) ) {
320 320
 			$query->set( 'meta_key', $orderby );
321 321
 			$query->set( 'orderby', 'meta_value' );
322 322
 		}
Please login to merge, or discard this patch.
Braces   +20 added lines, -7 removed lines patch added patch discarded remove patch
@@ -54,7 +54,9 @@  discard block
 block discarded – undo
54 54
 	{
55 55
 		$postTypeColumns = glsr()->postTypeColumns[Application::POST_TYPE];
56 56
 		foreach( $postTypeColumns as $key => &$value ) {
57
-			if( !array_key_exists( $key, $columns ) || !empty( $value ))continue;
57
+			if( !array_key_exists( $key, $columns ) || !empty( $value )) {
58
+				continue;
59
+			}
58 60
 			$value = $columns[$key];
59 61
 		}
60 62
 		if( count( glsr( Database::class )->getReviewsMeta( 'review_type' )) < 2 ) {
@@ -92,7 +94,8 @@  discard block
 block discarded – undo
92 94
 	 * @return array
93 95
 	 * @filter display_post_states
94 96
 	 */
95
-	public function filterPostStates( array $postStates, WP_Post $post ) {
97
+	public function filterPostStates( array $postStates, WP_Post $post )
98
+	{
96 99
 		if( $post->post_type == Application::POST_TYPE
97 100
 			&& array_key_exists( 'pending', $postStates )) {
98 101
 			$postStates['pending'] = __( 'Unapproved', 'site-reviews' );
@@ -137,7 +140,9 @@  discard block
 block discarded – undo
137 140
 		$postTypeColumns = glsr()->postTypeColumns[Application::POST_TYPE];
138 141
 		unset( $postTypeColumns['cb'] );
139 142
 		foreach( $postTypeColumns as $key => $value ) {
140
-			if( glsr( Helper::class )->startsWith( 'taxonomy', $key ))continue;
143
+			if( glsr( Helper::class )->startsWith( 'taxonomy', $key )) {
144
+				continue;
145
+			}
141 146
 			$columns[$key] = $key;
142 147
 		}
143 148
 		return $columns;
@@ -161,7 +166,9 @@  discard block
 block discarded – undo
161 166
 				'Pending' => __( 'Unapproved', 'site-reviews' ),
162 167
 			];
163 168
 			foreach( $strings as $search => $replace ) {
164
-				if( strpos( $single, $search ) === false )continue;
169
+				if( strpos( $single, $search ) === false ) {
170
+					continue;
171
+				}
165 172
 				$translation = $this->getTranslation([
166 173
 					'number' => $number,
167 174
 					'plural' => str_replace( $search, $replace, $plural ),
@@ -213,7 +220,9 @@  discard block
 block discarded – undo
213 220
 	 */
214 221
 	public function saveBulkEditFields( $postId )
215 222
 	{
216
-		if( !current_user_can( 'edit_posts' ))return;
223
+		if( !current_user_can( 'edit_posts' )) {
224
+			return;
225
+		}
217 226
 		$assignedTo = filter_input( INPUT_GET, 'assigned_to' );
218 227
 		if( $assignedTo && get_post( $assignedTo )) {
219 228
 			update_post_meta( $postId, 'assigned_to', $assignedTo );
@@ -226,7 +235,9 @@  discard block
 block discarded – undo
226 235
 	 */
227 236
 	public function setQueryForColumn( WP_Query $query )
228 237
 	{
229
-		if( !$this->hasPermission( $query ))return;
238
+		if( !$this->hasPermission( $query )) {
239
+			return;
240
+		}
230 241
 		$this->setMetaQuery( $query, [
231 242
 			'rating', 'review_type',
232 243
 		]);
@@ -298,7 +309,9 @@  discard block
 block discarded – undo
298 309
 	protected function setMetaQuery( WP_Query $query, array $metaKeys )
299 310
 	{
300 311
 		foreach( $metaKeys as $key ) {
301
-			if( !( $value = filter_input( INPUT_GET, $key )))continue;
312
+			if( !( $value = filter_input( INPUT_GET, $key ))) {
313
+				continue;
314
+			}
302 315
 			$metaQuery = (array)$query->get( 'meta_query' );
303 316
 			$metaQuery[] = [
304 317
 				'key' => $key,
Please login to merge, or discard this patch.