@@ -212,7 +212,9 @@ discard block |
||
| 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 |
||
| 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 | } |
@@ -39,7 +39,7 @@ discard block |
||
| 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 |
||
| 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 |
||
| 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 |
||
| 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 |
||
| 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 |
||
| 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; |
@@ -91,7 +91,9 @@ discard block |
||
| 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 |
||
| 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 |
||
| 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 |
||
| 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 |
||
| 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 | } |
@@ -38,7 +38,7 @@ discard block |
||
| 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 |
||
| 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 |
||
| 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 |
||
| 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 |
||
| 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 |
||
| 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 |
||
| 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 |
||
| 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 |
||
| 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 |
||
| 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 |
||
| 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 | } |
@@ -10,13 +10,13 @@ |
||
| 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; ?> |
@@ -15,14 +15,20 @@ |
||
| 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> |
@@ -38,7 +38,7 @@ |
||
| 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> |
@@ -46,9 +46,12 @@ |
||
| 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"> |
@@ -13,7 +13,7 @@ discard block |
||
| 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 |
||
| 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> |
@@ -60,7 +60,9 @@ |
||
| 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 |
@@ -30,7 +30,7 @@ discard block |
||
| 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 |
||
| 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 |
||
| 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 | } |
@@ -3,7 +3,9 @@ |
||
| 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' ); |
@@ -34,7 +34,9 @@ discard block |
||
| 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 |
||
| 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 |
||
| 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 |
||
| 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(); |
@@ -38,10 +38,10 @@ discard block |
||
| 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 |
||
| 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 |
||
| 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 |
||
| 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 |
||
| 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 |
||
| 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 |
||
| 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 |
||
| 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 |
||
| 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 |
||
| 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 |
||
| 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 |
||
| 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 |
||
| 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 |
||
| 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 |
||
| 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 |
||
| 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 | /** |
@@ -21,11 +21,11 @@ discard block |
||
| 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 |
||
| 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 |
||
| 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 |
||
| 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 |
||
| 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 |
||
| 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 |
||
| 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 |
||
| 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 |
||
| 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 |
||
| 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 |
||
| 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 |
||
| 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 |
||
| 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 | } |
@@ -54,7 +54,9 @@ discard block |
||
| 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 |
||
| 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 |
||
| 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 |
||
| 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 |
||
| 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 |
||
| 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 |
||
| 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, |