@@ -55,7 +55,9 @@ discard block |
||
| 55 | 55 | */ |
| 56 | 56 | public function enqueuePolyfillService() |
| 57 | 57 | { |
| 58 | - if( !apply_filters( 'site-reviews/assets/polyfill', true ))return; |
|
| 58 | + if( !apply_filters( 'site-reviews/assets/polyfill', true )) { |
|
| 59 | + return; |
|
| 60 | + } |
|
| 59 | 61 | wp_enqueue_script( Application::ID.'/polyfill', add_query_arg([ |
| 60 | 62 | 'features' => 'CustomEvent,Element.prototype.closest,Element.prototype.dataset,Event,XMLHttpRequest,MutationObserver', |
| 61 | 63 | 'flags' => 'gated', |
@@ -70,7 +72,9 @@ discard block |
||
| 70 | 72 | // wpforms-recaptcha |
| 71 | 73 | // google-recaptcha |
| 72 | 74 | // nf-google-recaptcha |
| 73 | - if( !glsr( OptionManager::class )->isRecaptchaEnabled() )return; |
|
| 75 | + if( !glsr( OptionManager::class )->isRecaptchaEnabled() ) { |
|
| 76 | + return; |
|
| 77 | + } |
|
| 74 | 78 | $language = apply_filters( 'site-reviews/recaptcha/language', get_locale() ); |
| 75 | 79 | wp_enqueue_script( Application::ID.'/google-recaptcha', add_query_arg([ |
| 76 | 80 | 'hl' => $language, |
@@ -84,7 +88,9 @@ discard block |
||
| 84 | 88 | public function inlineStyles() |
| 85 | 89 | { |
| 86 | 90 | $inlineStylesheetPath = glsr()->path( 'assets/styles/inline-styles.css' ); |
| 87 | - if( !apply_filters( 'site-reviews/assets/css', true ))return; |
|
| 91 | + if( !apply_filters( 'site-reviews/assets/css', true )) { |
|
| 92 | + return; |
|
| 93 | + } |
|
| 88 | 94 | if( !file_exists( $inlineStylesheetPath )) { |
| 89 | 95 | glsr_log()->error( 'Inline stylesheet is missing: '.$inlineStylesheetPath ); |
| 90 | 96 | return; |
@@ -27,7 +27,7 @@ discard block |
||
| 27 | 27 | */ |
| 28 | 28 | public function enqueueAssets() |
| 29 | 29 | { |
| 30 | - if( apply_filters( 'site-reviews/assets/css', true )) { |
|
| 30 | + if( apply_filters( 'site-reviews/assets/css', true ) ) { |
|
| 31 | 31 | wp_enqueue_style( |
| 32 | 32 | Application::ID, |
| 33 | 33 | $this->getStylesheet(), |
@@ -35,7 +35,7 @@ discard block |
||
| 35 | 35 | glsr()->version |
| 36 | 36 | ); |
| 37 | 37 | } |
| 38 | - if( apply_filters( 'site-reviews/assets/js', true )) { |
|
| 38 | + if( apply_filters( 'site-reviews/assets/js', true ) ) { |
|
| 39 | 39 | $dependencies = apply_filters( 'site-reviews/assets/polyfill', true ) |
| 40 | 40 | ? [Application::ID.'/polyfill'] |
| 41 | 41 | : []; |
@@ -55,11 +55,11 @@ discard block |
||
| 55 | 55 | */ |
| 56 | 56 | public function enqueuePolyfillService() |
| 57 | 57 | { |
| 58 | - if( !apply_filters( 'site-reviews/assets/polyfill', true ))return; |
|
| 59 | - wp_enqueue_script( Application::ID.'/polyfill', add_query_arg([ |
|
| 58 | + if( !apply_filters( 'site-reviews/assets/polyfill', true ) )return; |
|
| 59 | + wp_enqueue_script( Application::ID.'/polyfill', add_query_arg( [ |
|
| 60 | 60 | 'features' => 'CustomEvent,Element.prototype.closest,Element.prototype.dataset,Event,XMLHttpRequest,MutationObserver', |
| 61 | 61 | 'flags' => 'gated', |
| 62 | - ], 'https://polyfill.io/v3/polyfill.min.js' )); |
|
| 62 | + ], 'https://polyfill.io/v3/polyfill.min.js' ) ); |
|
| 63 | 63 | } |
| 64 | 64 | |
| 65 | 65 | /** |
@@ -72,10 +72,10 @@ discard block |
||
| 72 | 72 | // nf-google-recaptcha |
| 73 | 73 | if( !glsr( OptionManager::class )->isRecaptchaEnabled() )return; |
| 74 | 74 | $language = apply_filters( 'site-reviews/recaptcha/language', get_locale() ); |
| 75 | - wp_enqueue_script( Application::ID.'/google-recaptcha', add_query_arg([ |
|
| 75 | + wp_enqueue_script( Application::ID.'/google-recaptcha', add_query_arg( [ |
|
| 76 | 76 | 'hl' => $language, |
| 77 | 77 | 'render' => 'explicit', |
| 78 | - ], 'https://www.google.com/recaptcha/api.js' )); |
|
| 78 | + ], 'https://www.google.com/recaptcha/api.js' ) ); |
|
| 79 | 79 | } |
| 80 | 80 | |
| 81 | 81 | /** |
@@ -101,8 +101,8 @@ discard block |
||
| 101 | 101 | public function inlineStyles() |
| 102 | 102 | { |
| 103 | 103 | $inlineStylesheetPath = glsr()->path( 'assets/styles/inline-styles.css' ); |
| 104 | - if( !apply_filters( 'site-reviews/assets/css', true ))return; |
|
| 105 | - if( !file_exists( $inlineStylesheetPath )) { |
|
| 104 | + if( !apply_filters( 'site-reviews/assets/css', true ) )return; |
|
| 105 | + if( !file_exists( $inlineStylesheetPath ) ) { |
|
| 106 | 106 | glsr_log()->error( 'Inline stylesheet is missing: '.$inlineStylesheetPath ); |
| 107 | 107 | return; |
| 108 | 108 | } |
@@ -122,7 +122,7 @@ discard block |
||
| 122 | 122 | { |
| 123 | 123 | $script = 'window.hasOwnProperty("GLSR")||(window.GLSR={});'; |
| 124 | 124 | foreach( $variables as $key => $value ) { |
| 125 | - $script.= sprintf( 'GLSR.%s=%s;', $key, json_encode( $value, JSON_UNESCAPED_SLASHES|JSON_UNESCAPED_UNICODE )); |
|
| 125 | + $script .= sprintf( 'GLSR.%s=%s;', $key, json_encode( $value, JSON_UNESCAPED_SLASHES | JSON_UNESCAPED_UNICODE ) ); |
|
| 126 | 126 | } |
| 127 | 127 | $pattern = '/\"([^ \-\"]+)\"(:[{\[\"])/'; // removes unnecessary quotes surrounding object keys |
| 128 | 128 | $optimizedScript = preg_replace( $pattern, '$1$2', $script ); |
@@ -134,7 +134,7 @@ discard block |
||
| 134 | 134 | */ |
| 135 | 135 | protected function getFixedSelectorsForPagination() |
| 136 | 136 | { |
| 137 | - $selectors = ['#wpadminbar','.site-navigation-fixed']; |
|
| 137 | + $selectors = ['#wpadminbar', '.site-navigation-fixed']; |
|
| 138 | 138 | return apply_filters( 'site-reviews/enqueue/public/localize/ajax-pagination', $selectors ); |
| 139 | 139 | } |
| 140 | 140 | |
@@ -144,7 +144,7 @@ discard block |
||
| 144 | 144 | protected function getStylesheet() |
| 145 | 145 | { |
| 146 | 146 | $currentStyle = glsr( Style::class )->style; |
| 147 | - return file_exists( glsr()->path( 'assets/styles/custom/'.$currentStyle.'.css' )) |
|
| 147 | + return file_exists( glsr()->path( 'assets/styles/custom/'.$currentStyle.'.css' ) ) |
|
| 148 | 148 | ? glsr()->url( 'assets/styles/custom/'.$currentStyle.'.css' ) |
| 149 | 149 | : glsr()->url( 'assets/styles/'.Application::ID.'.css' ); |
| 150 | 150 | } |
@@ -22,11 +22,11 @@ discard block |
||
| 22 | 22 | public function approve() |
| 23 | 23 | { |
| 24 | 24 | if( filter_input( INPUT_GET, 'plugin' ) != Application::ID )return; |
| 25 | - check_admin_referer( 'approve-review_'.( $postId = $this->getPostId() )); |
|
| 26 | - wp_update_post([ |
|
| 25 | + check_admin_referer( 'approve-review_'.($postId = $this->getPostId()) ); |
|
| 26 | + wp_update_post( [ |
|
| 27 | 27 | 'ID' => $postId, |
| 28 | 28 | 'post_status' => 'publish', |
| 29 | - ]); |
|
| 29 | + ] ); |
|
| 30 | 30 | wp_safe_redirect( wp_get_referer() ); |
| 31 | 31 | exit; |
| 32 | 32 | } |
@@ -59,11 +59,11 @@ discard block |
||
| 59 | 59 | $columns = glsr( Helper::class )->consolidateArray( $columns ); |
| 60 | 60 | $postTypeColumns = glsr()->postTypeColumns[Application::POST_TYPE]; |
| 61 | 61 | foreach( $postTypeColumns as $key => &$value ) { |
| 62 | - if( !array_key_exists( $key, $columns ) || !empty( $value ))continue; |
|
| 62 | + if( !array_key_exists( $key, $columns ) || !empty($value) )continue; |
|
| 63 | 63 | $value = $columns[$key]; |
| 64 | 64 | } |
| 65 | - if( count( glsr( Database::class )->getReviewsMeta( 'review_type' )) < 2 ) { |
|
| 66 | - unset( $postTypeColumns['review_type'] ); |
|
| 65 | + if( count( glsr( Database::class )->getReviewsMeta( 'review_type' ) ) < 2 ) { |
|
| 66 | + unset($postTypeColumns['review_type']); |
|
| 67 | 67 | } |
| 68 | 68 | return array_filter( $postTypeColumns, 'strlen' ); |
| 69 | 69 | } |
@@ -105,7 +105,7 @@ discard block |
||
| 105 | 105 | */ |
| 106 | 106 | public function filterPostStates( $postStates, $post ) { |
| 107 | 107 | $postStates = glsr( Helper::class )->consolidateArray( $postStates ); |
| 108 | - if( glsr_get( $post, 'post_type' ) == Application::POST_TYPE && array_key_exists( 'pending', $postStates )) { |
|
| 108 | + if( glsr_get( $post, 'post_type' ) == Application::POST_TYPE && array_key_exists( 'pending', $postStates ) ) { |
|
| 109 | 109 | $postStates['pending'] = __( 'Unapproved', 'site-reviews' ); |
| 110 | 110 | } |
| 111 | 111 | return $postStates; |
@@ -122,7 +122,7 @@ discard block |
||
| 122 | 122 | if( glsr_get( $post, 'post_type' ) != Application::POST_TYPE || $post->post_status == 'trash' ) { |
| 123 | 123 | return $actions; |
| 124 | 124 | } |
| 125 | - unset( $actions['inline hide-if-no-js'] ); //Remove Quick-edit |
|
| 125 | + unset($actions['inline hide-if-no-js']); //Remove Quick-edit |
|
| 126 | 126 | $rowActions = [ |
| 127 | 127 | 'approve' => esc_attr__( 'Approve', 'site-reviews' ), |
| 128 | 128 | 'unapprove' => esc_attr__( 'Unapprove', 'site-reviews' ), |
@@ -136,7 +136,7 @@ discard block |
||
| 136 | 136 | admin_url( 'post.php?post='.$post->ID.'&action='.$key.'&plugin='.Application::ID ), |
| 137 | 137 | $key.'-review_'.$post->ID |
| 138 | 138 | ), |
| 139 | - ]); |
|
| 139 | + ] ); |
|
| 140 | 140 | } |
| 141 | 141 | return $newActions + glsr( Helper::class )->consolidateArray( $actions ); |
| 142 | 142 | } |
@@ -150,9 +150,9 @@ discard block |
||
| 150 | 150 | { |
| 151 | 151 | $columns = glsr( Helper::class )->consolidateArray( $columns ); |
| 152 | 152 | $postTypeColumns = glsr()->postTypeColumns[Application::POST_TYPE]; |
| 153 | - unset( $postTypeColumns['cb'] ); |
|
| 153 | + unset($postTypeColumns['cb']); |
|
| 154 | 154 | foreach( $postTypeColumns as $key => $value ) { |
| 155 | - if( glsr( Helper::class )->startsWith( 'taxonomy', $key ))continue; |
|
| 155 | + if( glsr( Helper::class )->startsWith( 'taxonomy', $key ) )continue; |
|
| 156 | 156 | $columns[$key] = $key; |
| 157 | 157 | } |
| 158 | 158 | return $columns; |
@@ -170,18 +170,18 @@ discard block |
||
| 170 | 170 | */ |
| 171 | 171 | public function filterStatusText( $translation, $single, $plural, $number, $domain ) |
| 172 | 172 | { |
| 173 | - if( $this->canModifyTranslation( $domain )) { |
|
| 173 | + if( $this->canModifyTranslation( $domain ) ) { |
|
| 174 | 174 | $strings = [ |
| 175 | 175 | 'Published' => __( 'Approved', 'site-reviews' ), |
| 176 | 176 | 'Pending' => __( 'Unapproved', 'site-reviews' ), |
| 177 | 177 | ]; |
| 178 | 178 | foreach( $strings as $search => $replace ) { |
| 179 | 179 | if( strpos( $single, $search ) === false )continue; |
| 180 | - $translation = $this->getTranslation([ |
|
| 180 | + $translation = $this->getTranslation( [ |
|
| 181 | 181 | 'number' => $number, |
| 182 | 182 | 'plural' => str_replace( $search, $replace, $plural ), |
| 183 | 183 | 'single' => str_replace( $search, $replace, $single ), |
| 184 | - ]); |
|
| 184 | + ] ); |
|
| 185 | 185 | } |
| 186 | 186 | } |
| 187 | 187 | return $translation; |
@@ -228,9 +228,9 @@ discard block |
||
| 228 | 228 | */ |
| 229 | 229 | public function saveBulkEditFields( $postId ) |
| 230 | 230 | { |
| 231 | - if( !current_user_can( 'edit_posts' ))return; |
|
| 231 | + if( !current_user_can( 'edit_posts' ) )return; |
|
| 232 | 232 | $assignedTo = filter_input( INPUT_GET, 'assigned_to' ); |
| 233 | - if( $assignedTo && get_post( $assignedTo )) { |
|
| 233 | + if( $assignedTo && get_post( $assignedTo ) ) { |
|
| 234 | 234 | update_post_meta( $postId, 'assigned_to', $assignedTo ); |
| 235 | 235 | } |
| 236 | 236 | } |
@@ -241,10 +241,10 @@ discard block |
||
| 241 | 241 | */ |
| 242 | 242 | public function setQueryForColumn( WP_Query $query ) |
| 243 | 243 | { |
| 244 | - if( !$this->hasPermission( $query ))return; |
|
| 244 | + if( !$this->hasPermission( $query ) )return; |
|
| 245 | 245 | $this->setMetaQuery( $query, [ |
| 246 | 246 | 'rating', 'review_type', |
| 247 | - ]); |
|
| 247 | + ] ); |
|
| 248 | 248 | $this->setOrderby( $query ); |
| 249 | 249 | } |
| 250 | 250 | |
@@ -255,11 +255,11 @@ discard block |
||
| 255 | 255 | public function unapprove() |
| 256 | 256 | { |
| 257 | 257 | if( filter_input( INPUT_GET, 'plugin' ) != Application::ID )return; |
| 258 | - check_admin_referer( 'unapprove-review_'.( $postId = $this->getPostId() )); |
|
| 259 | - wp_update_post([ |
|
| 258 | + check_admin_referer( 'unapprove-review_'.($postId = $this->getPostId()) ); |
|
| 259 | + wp_update_post( [ |
|
| 260 | 260 | 'ID' => $postId, |
| 261 | 261 | 'post_status' => 'pending', |
| 262 | - ]); |
|
| 262 | + ] ); |
|
| 263 | 263 | wp_safe_redirect( wp_get_referer() ); |
| 264 | 264 | exit; |
| 265 | 265 | } |
@@ -289,7 +289,7 @@ discard block |
||
| 289 | 289 | 'single' => '', |
| 290 | 290 | 'text' => '', |
| 291 | 291 | ]; |
| 292 | - $args = (object) wp_parse_args( $args, $defaults ); |
|
| 292 | + $args = (object)wp_parse_args( $args, $defaults ); |
|
| 293 | 293 | $translations = get_translations_for_domain( Application::ID ); |
| 294 | 294 | return $args->text |
| 295 | 295 | ? $translations->translate( $args->text ) |
@@ -314,7 +314,7 @@ discard block |
||
| 314 | 314 | protected function setMetaQuery( WP_Query $query, array $metaKeys ) |
| 315 | 315 | { |
| 316 | 316 | foreach( $metaKeys as $key ) { |
| 317 | - if( !( $value = filter_input( INPUT_GET, $key )))continue; |
|
| 317 | + if( !($value = filter_input( INPUT_GET, $key )) )continue; |
|
| 318 | 318 | $metaQuery = (array)$query->get( 'meta_query' ); |
| 319 | 319 | $metaQuery[] = [ |
| 320 | 320 | 'key' => $key, |
@@ -331,8 +331,8 @@ discard block |
||
| 331 | 331 | { |
| 332 | 332 | $orderby = $query->get( 'orderby' ); |
| 333 | 333 | $columns = glsr()->postTypeColumns[Application::POST_TYPE]; |
| 334 | - unset( $columns['cb'], $columns['title'], $columns['date'] ); |
|
| 335 | - if( in_array( $orderby, array_keys( $columns ))) { |
|
| 334 | + unset($columns['cb'], $columns['title'], $columns['date']); |
|
| 335 | + if( in_array( $orderby, array_keys( $columns ) ) ) { |
|
| 336 | 336 | if( $orderby == 'reviewer' ) { |
| 337 | 337 | $orderby = 'author'; |
| 338 | 338 | } |
@@ -21,7 +21,9 @@ discard block |
||
| 21 | 21 | */ |
| 22 | 22 | public function approve() |
| 23 | 23 | { |
| 24 | - if( filter_input( INPUT_GET, 'plugin' ) != Application::ID )return; |
|
| 24 | + if( filter_input( INPUT_GET, 'plugin' ) != Application::ID ) { |
|
| 25 | + return; |
|
| 26 | + } |
|
| 25 | 27 | check_admin_referer( 'approve-review_'.( $postId = $this->getPostId() )); |
| 26 | 28 | wp_update_post([ |
| 27 | 29 | 'ID' => $postId, |
@@ -59,7 +61,9 @@ discard block |
||
| 59 | 61 | $columns = glsr( Helper::class )->consolidateArray( $columns ); |
| 60 | 62 | $postTypeColumns = glsr()->postTypeColumns[Application::POST_TYPE]; |
| 61 | 63 | foreach( $postTypeColumns as $key => &$value ) { |
| 62 | - if( !array_key_exists( $key, $columns ) || !empty( $value ))continue; |
|
| 64 | + if( !array_key_exists( $key, $columns ) || !empty( $value )) { |
|
| 65 | + continue; |
|
| 66 | + } |
|
| 63 | 67 | $value = $columns[$key]; |
| 64 | 68 | } |
| 65 | 69 | if( count( glsr( Database::class )->getReviewsMeta( 'review_type' )) < 2 ) { |
@@ -103,7 +107,8 @@ discard block |
||
| 103 | 107 | * @return array |
| 104 | 108 | * @filter display_post_states |
| 105 | 109 | */ |
| 106 | - public function filterPostStates( $postStates, $post ) { |
|
| 110 | + public function filterPostStates( $postStates, $post ) |
|
| 111 | + { |
|
| 107 | 112 | $postStates = glsr( Helper::class )->consolidateArray( $postStates ); |
| 108 | 113 | if( glsr_get( $post, 'post_type' ) == Application::POST_TYPE && array_key_exists( 'pending', $postStates )) { |
| 109 | 114 | $postStates['pending'] = __( 'Unapproved', 'site-reviews' ); |
@@ -152,7 +157,9 @@ discard block |
||
| 152 | 157 | $postTypeColumns = glsr()->postTypeColumns[Application::POST_TYPE]; |
| 153 | 158 | unset( $postTypeColumns['cb'] ); |
| 154 | 159 | foreach( $postTypeColumns as $key => $value ) { |
| 155 | - if( glsr( Helper::class )->startsWith( 'taxonomy', $key ))continue; |
|
| 160 | + if( glsr( Helper::class )->startsWith( 'taxonomy', $key )) { |
|
| 161 | + continue; |
|
| 162 | + } |
|
| 156 | 163 | $columns[$key] = $key; |
| 157 | 164 | } |
| 158 | 165 | return $columns; |
@@ -176,7 +183,9 @@ discard block |
||
| 176 | 183 | 'Pending' => __( 'Unapproved', 'site-reviews' ), |
| 177 | 184 | ]; |
| 178 | 185 | foreach( $strings as $search => $replace ) { |
| 179 | - if( strpos( $single, $search ) === false )continue; |
|
| 186 | + if( strpos( $single, $search ) === false ) { |
|
| 187 | + continue; |
|
| 188 | + } |
|
| 180 | 189 | $translation = $this->getTranslation([ |
| 181 | 190 | 'number' => $number, |
| 182 | 191 | 'plural' => str_replace( $search, $replace, $plural ), |
@@ -228,7 +237,9 @@ discard block |
||
| 228 | 237 | */ |
| 229 | 238 | public function saveBulkEditFields( $postId ) |
| 230 | 239 | { |
| 231 | - if( !current_user_can( 'edit_posts' ))return; |
|
| 240 | + if( !current_user_can( 'edit_posts' )) { |
|
| 241 | + return; |
|
| 242 | + } |
|
| 232 | 243 | $assignedTo = filter_input( INPUT_GET, 'assigned_to' ); |
| 233 | 244 | if( $assignedTo && get_post( $assignedTo )) { |
| 234 | 245 | update_post_meta( $postId, 'assigned_to', $assignedTo ); |
@@ -241,7 +252,9 @@ discard block |
||
| 241 | 252 | */ |
| 242 | 253 | public function setQueryForColumn( WP_Query $query ) |
| 243 | 254 | { |
| 244 | - if( !$this->hasPermission( $query ))return; |
|
| 255 | + if( !$this->hasPermission( $query )) { |
|
| 256 | + return; |
|
| 257 | + } |
|
| 245 | 258 | $this->setMetaQuery( $query, [ |
| 246 | 259 | 'rating', 'review_type', |
| 247 | 260 | ]); |
@@ -254,7 +267,9 @@ discard block |
||
| 254 | 267 | */ |
| 255 | 268 | public function unapprove() |
| 256 | 269 | { |
| 257 | - if( filter_input( INPUT_GET, 'plugin' ) != Application::ID )return; |
|
| 270 | + if( filter_input( INPUT_GET, 'plugin' ) != Application::ID ) { |
|
| 271 | + return; |
|
| 272 | + } |
|
| 258 | 273 | check_admin_referer( 'unapprove-review_'.( $postId = $this->getPostId() )); |
| 259 | 274 | wp_update_post([ |
| 260 | 275 | 'ID' => $postId, |
@@ -314,7 +329,9 @@ discard block |
||
| 314 | 329 | protected function setMetaQuery( WP_Query $query, array $metaKeys ) |
| 315 | 330 | { |
| 316 | 331 | foreach( $metaKeys as $key ) { |
| 317 | - if( !( $value = filter_input( INPUT_GET, $key )))continue; |
|
| 332 | + if( !( $value = filter_input( INPUT_GET, $key ))) { |
|
| 333 | + continue; |
|
| 334 | + } |
|
| 318 | 335 | $metaQuery = (array)$query->get( 'meta_query' ); |
| 319 | 336 | $metaQuery[] = [ |
| 320 | 337 | 'key' => $key, |
@@ -133,13 +133,13 @@ discard block |
||
| 133 | 133 | */ |
| 134 | 134 | public function renderAssignedToMetabox( $post ) |
| 135 | 135 | { |
| 136 | - if( !$this->isReviewPostType( $post ))return; |
|
| 136 | + if( !$this->isReviewPostType( $post ) )return; |
|
| 137 | 137 | $assignedTo = (string)get_post_meta( $post->ID, 'assigned_to', true ); |
| 138 | 138 | wp_nonce_field( 'assigned_to', '_nonce-assigned-to', false ); |
| 139 | 139 | glsr()->render( 'partials/editor/metabox-assigned-to', [ |
| 140 | 140 | 'id' => $assignedTo, |
| 141 | 141 | 'template' => $this->buildAssignedToTemplate( $assignedTo, $post ), |
| 142 | - ]); |
|
| 142 | + ] ); |
|
| 143 | 143 | } |
| 144 | 144 | |
| 145 | 145 | /** |
@@ -149,12 +149,12 @@ discard block |
||
| 149 | 149 | */ |
| 150 | 150 | public function renderDetailsMetaBox( $post ) |
| 151 | 151 | { |
| 152 | - if( !$this->isReviewPostType( $post ))return; |
|
| 152 | + if( !$this->isReviewPostType( $post ) )return; |
|
| 153 | 153 | $review = glsr( ReviewManager::class )->single( $post ); |
| 154 | 154 | glsr()->render( 'partials/editor/metabox-details', [ |
| 155 | 155 | 'button' => $this->buildDetailsMetaBoxRevertButton( $review, $post ), |
| 156 | 156 | 'metabox' => $this->normalizeDetailsMetaBox( $review ), |
| 157 | - ]); |
|
| 157 | + ] ); |
|
| 158 | 158 | } |
| 159 | 159 | |
| 160 | 160 | /** |
@@ -163,14 +163,14 @@ discard block |
||
| 163 | 163 | */ |
| 164 | 164 | public function renderPinnedInPublishMetaBox() |
| 165 | 165 | { |
| 166 | - if( !$this->isReviewPostType( get_post() ))return; |
|
| 166 | + if( !$this->isReviewPostType( get_post() ) )return; |
|
| 167 | 167 | glsr( Template::class )->render( 'partials/editor/pinned', [ |
| 168 | 168 | 'context' => [ |
| 169 | 169 | 'no' => __( 'No', 'site-reviews' ), |
| 170 | 170 | 'yes' => __( 'Yes', 'site-reviews' ), |
| 171 | 171 | ], |
| 172 | - 'pinned' => wp_validate_boolean( get_post_meta( intval( get_the_ID() ), 'pinned', true )), |
|
| 173 | - ]); |
|
| 172 | + 'pinned' => wp_validate_boolean( get_post_meta( intval( get_the_ID() ), 'pinned', true ) ), |
|
| 173 | + ] ); |
|
| 174 | 174 | } |
| 175 | 175 | |
| 176 | 176 | /** |
@@ -180,11 +180,11 @@ discard block |
||
| 180 | 180 | */ |
| 181 | 181 | public function renderResponseMetaBox( $post ) |
| 182 | 182 | { |
| 183 | - if( !$this->isReviewPostType( $post ))return; |
|
| 183 | + if( !$this->isReviewPostType( $post ) )return; |
|
| 184 | 184 | wp_nonce_field( 'response', '_nonce-response', false ); |
| 185 | 185 | glsr()->render( 'partials/editor/metabox-response', [ |
| 186 | 186 | 'response' => get_post_meta( $post->ID, 'response', true ), |
| 187 | - ]); |
|
| 187 | + ] ); |
|
| 188 | 188 | } |
| 189 | 189 | |
| 190 | 190 | /** |
@@ -194,11 +194,11 @@ discard block |
||
| 194 | 194 | */ |
| 195 | 195 | public function renderReviewEditor( $post ) |
| 196 | 196 | { |
| 197 | - if( !$this->isReviewPostType( $post ) || $this->isReviewEditable( $post ))return; |
|
| 197 | + if( !$this->isReviewPostType( $post ) || $this->isReviewEditable( $post ) )return; |
|
| 198 | 198 | glsr()->render( 'partials/editor/review', [ |
| 199 | 199 | 'post' => $post, |
| 200 | 200 | 'response' => get_post_meta( $post->ID, 'response', true ), |
| 201 | - ]); |
|
| 201 | + ] ); |
|
| 202 | 202 | } |
| 203 | 203 | |
| 204 | 204 | /** |
@@ -210,7 +210,7 @@ discard block |
||
| 210 | 210 | $screen = glsr_current_screen(); |
| 211 | 211 | if( $screen->base != 'post' || $screen->post_type != Application::POST_TYPE )return; |
| 212 | 212 | add_action( 'edit_form_after_title', [$this, 'renderReviewEditor'] ); |
| 213 | - add_action( 'edit_form_top', [$this, 'renderReviewNotice'] ); |
|
| 213 | + add_action( 'edit_form_top', [$this, 'renderReviewNotice'] ); |
|
| 214 | 214 | } |
| 215 | 215 | |
| 216 | 216 | /** |
@@ -220,16 +220,16 @@ discard block |
||
| 220 | 220 | */ |
| 221 | 221 | public function renderReviewNotice( $post ) |
| 222 | 222 | { |
| 223 | - if( !$this->isReviewPostType( $post ) || $this->isReviewEditable( $post ))return; |
|
| 223 | + if( !$this->isReviewPostType( $post ) || $this->isReviewEditable( $post ) )return; |
|
| 224 | 224 | glsr( Notice::class )->addWarning( sprintf( |
| 225 | 225 | __( '%s reviews are read-only.', 'site-reviews' ), |
| 226 | 226 | glsr( Columns::class )->buildColumnReviewType( $post->ID ) |
| 227 | - )); |
|
| 227 | + ) ); |
|
| 228 | 228 | glsr( Template::class )->render( 'partials/editor/notice', [ |
| 229 | 229 | 'context' => [ |
| 230 | 230 | 'notices' => glsr( Notice::class )->get(), |
| 231 | 231 | ], |
| 232 | - ]); |
|
| 232 | + ] ); |
|
| 233 | 233 | } |
| 234 | 234 | |
| 235 | 235 | /** |
@@ -240,12 +240,12 @@ discard block |
||
| 240 | 240 | */ |
| 241 | 241 | public function renderTaxonomyMetabox( $post ) |
| 242 | 242 | { |
| 243 | - if( !$this->isReviewPostType( $post ))return; |
|
| 243 | + if( !$this->isReviewPostType( $post ) )return; |
|
| 244 | 244 | glsr()->render( 'partials/editor/metabox-categories', [ |
| 245 | 245 | 'post' => $post, |
| 246 | 246 | 'tax_name' => Application::TAXONOMY, |
| 247 | 247 | 'taxonomy' => get_taxonomy( Application::TAXONOMY ), |
| 248 | - ]); |
|
| 248 | + ] ); |
|
| 249 | 249 | } |
| 250 | 250 | |
| 251 | 251 | /** |
@@ -256,7 +256,7 @@ discard block |
||
| 256 | 256 | public function revertReview() |
| 257 | 257 | { |
| 258 | 258 | if( filter_input( INPUT_GET, 'plugin' ) != Application::ID )return; |
| 259 | - check_admin_referer( 'revert-review_'.( $postId = $this->getPostId() )); |
|
| 259 | + check_admin_referer( 'revert-review_'.($postId = $this->getPostId()) ); |
|
| 260 | 260 | glsr( ReviewManager::class )->revert( $postId ); |
| 261 | 261 | $this->redirect( $postId, 52 ); |
| 262 | 262 | } |
@@ -279,13 +279,13 @@ discard block |
||
| 279 | 279 | protected function buildAssignedToTemplate( $assignedTo, WP_Post $post ) |
| 280 | 280 | { |
| 281 | 281 | $assignedPost = glsr( Database::class )->getAssignedToPost( $post->ID, $assignedTo ); |
| 282 | - if( !( $assignedPost instanceof WP_Post ))return; |
|
| 282 | + if( !($assignedPost instanceof WP_Post) )return; |
|
| 283 | 283 | return glsr( Template::class )->build( 'partials/editor/assigned-post', [ |
| 284 | 284 | 'context' => [ |
| 285 | 285 | 'data.url' => (string)get_permalink( $assignedPost ), |
| 286 | 286 | 'data.title' => get_the_title( $assignedPost ), |
| 287 | 287 | ], |
| 288 | - ]); |
|
| 288 | + ] ); |
|
| 289 | 289 | } |
| 290 | 290 | |
| 291 | 291 | /** |
@@ -310,13 +310,13 @@ discard block |
||
| 310 | 310 | 'class' => 'button button-large', |
| 311 | 311 | 'href' => $revertUrl, |
| 312 | 312 | 'id' => 'revert', |
| 313 | - ]); |
|
| 313 | + ] ); |
|
| 314 | 314 | } |
| 315 | 315 | return glsr( Builder::class )->button( __( 'Nothing to Revert', 'site-reviews' ), [ |
| 316 | 316 | 'class' => 'button button-large', |
| 317 | 317 | 'disabled' => true, |
| 318 | 318 | 'id' => 'revert', |
| 319 | - ]); |
|
| 319 | + ] ); |
|
| 320 | 320 | } |
| 321 | 321 | |
| 322 | 322 | /** |
@@ -329,11 +329,11 @@ discard block |
||
| 329 | 329 | $reviewType = array_key_exists( $review->review_type, glsr()->reviewTypes ) |
| 330 | 330 | ? glsr()->reviewTypes[$review->review_type] |
| 331 | 331 | : __( 'Unknown', 'site-reviews' ); |
| 332 | - if( !empty( $review->url )) { |
|
| 332 | + if( !empty($review->url) ) { |
|
| 333 | 333 | $reviewType = glsr( Builder::class )->a( $reviewType, [ |
| 334 | 334 | 'href' => $review->url, |
| 335 | 335 | 'target' => '_blank', |
| 336 | - ]); |
|
| 336 | + ] ); |
|
| 337 | 337 | } |
| 338 | 338 | return $reviewType; |
| 339 | 339 | } |
@@ -362,16 +362,16 @@ discard block |
||
| 362 | 362 | */ |
| 363 | 363 | protected function normalizeDetailsMetaBox( Review $review ) |
| 364 | 364 | { |
| 365 | - $user = empty( $review->user_id ) |
|
| 365 | + $user = empty($review->user_id) |
|
| 366 | 366 | ? __( 'Unregistered user', 'site-reviews' ) |
| 367 | 367 | : glsr( Builder::class )->a( get_the_author_meta( 'display_name', $review->user_id ), [ |
| 368 | 368 | 'href' => get_author_posts_url( $review->user_id ), |
| 369 | - ]); |
|
| 370 | - $email = empty( $review->email ) |
|
| 369 | + ] ); |
|
| 370 | + $email = empty($review->email) |
|
| 371 | 371 | ? '—' |
| 372 | 372 | : glsr( Builder::class )->a( $review->email, [ |
| 373 | 373 | 'href' => 'mailto:'.$review->email.'?subject='.esc_attr( __( 'RE:', 'site-reviews' ).' '.$review->title ), |
| 374 | - ]); |
|
| 374 | + ] ); |
|
| 375 | 375 | $metabox = [ |
| 376 | 376 | __( 'Rating', 'site-reviews' ) => glsr_star_rating( $review->rating ), |
| 377 | 377 | __( 'Type', 'site-reviews' ) => $this->getReviewType( $review ), |
@@ -382,7 +382,7 @@ discard block |
||
| 382 | 382 | __( 'IP Address', 'site-reviews' ) => $review->ip_address, |
| 383 | 383 | __( 'Avatar', 'site-reviews' ) => sprintf( '<img src="%s" width="96">', $review->avatar ), |
| 384 | 384 | ]; |
| 385 | - return array_filter( apply_filters( 'site-reviews/metabox/details', $metabox, $review )); |
|
| 385 | + return array_filter( apply_filters( 'site-reviews/metabox/details', $metabox, $review ) ); |
|
| 386 | 386 | } |
| 387 | 387 | |
| 388 | 388 | /** |
@@ -399,7 +399,7 @@ discard block |
||
| 399 | 399 | $redirectUri = $hasReferer |
| 400 | 400 | ? remove_query_arg( ['deleted', 'ids', 'trashed', 'untrashed'], $referer ) |
| 401 | 401 | : get_edit_post_link( $postId ); |
| 402 | - wp_safe_redirect( add_query_arg( ['message' => $messageIndex], $redirectUri )); |
|
| 402 | + wp_safe_redirect( add_query_arg( ['message' => $messageIndex], $redirectUri ) ); |
|
| 403 | 403 | exit; |
| 404 | 404 | } |
| 405 | 405 | } |
@@ -96,7 +96,9 @@ discard block |
||
| 96 | 96 | { |
| 97 | 97 | add_meta_box( Application::ID.'_assigned_to', __( 'Assigned To', 'site-reviews' ), [$this, 'renderAssignedToMetabox'], null, 'side' ); |
| 98 | 98 | add_meta_box( Application::ID.'_review', __( 'Details', 'site-reviews' ), [$this, 'renderDetailsMetaBox'], null, 'side' ); |
| 99 | - if( get_post_meta( $post->ID, 'review_type', true ) != 'local' )return; |
|
| 99 | + if( get_post_meta( $post->ID, 'review_type', true ) != 'local' ) { |
|
| 100 | + return; |
|
| 101 | + } |
|
| 100 | 102 | add_meta_box( Application::ID.'_response', __( 'Respond Publicly', 'site-reviews' ), [$this, 'renderResponseMetaBox'], null, 'normal' ); |
| 101 | 103 | } |
| 102 | 104 | |
@@ -133,7 +135,9 @@ discard block |
||
| 133 | 135 | */ |
| 134 | 136 | public function renderAssignedToMetabox( $post ) |
| 135 | 137 | { |
| 136 | - if( !$this->isReviewPostType( $post ))return; |
|
| 138 | + if( !$this->isReviewPostType( $post )) { |
|
| 139 | + return; |
|
| 140 | + } |
|
| 137 | 141 | $assignedTo = (string)get_post_meta( $post->ID, 'assigned_to', true ); |
| 138 | 142 | wp_nonce_field( 'assigned_to', '_nonce-assigned-to', false ); |
| 139 | 143 | glsr()->render( 'partials/editor/metabox-assigned-to', [ |
@@ -149,7 +153,9 @@ discard block |
||
| 149 | 153 | */ |
| 150 | 154 | public function renderDetailsMetaBox( $post ) |
| 151 | 155 | { |
| 152 | - if( !$this->isReviewPostType( $post ))return; |
|
| 156 | + if( !$this->isReviewPostType( $post )) { |
|
| 157 | + return; |
|
| 158 | + } |
|
| 153 | 159 | $review = glsr( ReviewManager::class )->single( $post ); |
| 154 | 160 | glsr()->render( 'partials/editor/metabox-details', [ |
| 155 | 161 | 'button' => $this->buildDetailsMetaBoxRevertButton( $review, $post ), |
@@ -163,7 +169,9 @@ discard block |
||
| 163 | 169 | */ |
| 164 | 170 | public function renderPinnedInPublishMetaBox() |
| 165 | 171 | { |
| 166 | - if( !$this->isReviewPostType( get_post() ))return; |
|
| 172 | + if( !$this->isReviewPostType( get_post() )) { |
|
| 173 | + return; |
|
| 174 | + } |
|
| 167 | 175 | glsr( Template::class )->render( 'partials/editor/pinned', [ |
| 168 | 176 | 'context' => [ |
| 169 | 177 | 'no' => __( 'No', 'site-reviews' ), |
@@ -180,7 +188,9 @@ discard block |
||
| 180 | 188 | */ |
| 181 | 189 | public function renderResponseMetaBox( $post ) |
| 182 | 190 | { |
| 183 | - if( !$this->isReviewPostType( $post ))return; |
|
| 191 | + if( !$this->isReviewPostType( $post )) { |
|
| 192 | + return; |
|
| 193 | + } |
|
| 184 | 194 | wp_nonce_field( 'response', '_nonce-response', false ); |
| 185 | 195 | glsr()->render( 'partials/editor/metabox-response', [ |
| 186 | 196 | 'response' => get_post_meta( $post->ID, 'response', true ), |
@@ -194,7 +204,9 @@ discard block |
||
| 194 | 204 | */ |
| 195 | 205 | public function renderReviewEditor( $post ) |
| 196 | 206 | { |
| 197 | - if( !$this->isReviewPostType( $post ) || $this->isReviewEditable( $post ))return; |
|
| 207 | + if( !$this->isReviewPostType( $post ) || $this->isReviewEditable( $post )) { |
|
| 208 | + return; |
|
| 209 | + } |
|
| 198 | 210 | glsr()->render( 'partials/editor/review', [ |
| 199 | 211 | 'post' => $post, |
| 200 | 212 | 'response' => get_post_meta( $post->ID, 'response', true ), |
@@ -208,7 +220,9 @@ discard block |
||
| 208 | 220 | public function renderReviewFields() |
| 209 | 221 | { |
| 210 | 222 | $screen = glsr_current_screen(); |
| 211 | - if( $screen->base != 'post' || $screen->post_type != Application::POST_TYPE )return; |
|
| 223 | + if( $screen->base != 'post' || $screen->post_type != Application::POST_TYPE ) { |
|
| 224 | + return; |
|
| 225 | + } |
|
| 212 | 226 | add_action( 'edit_form_after_title', [$this, 'renderReviewEditor'] ); |
| 213 | 227 | add_action( 'edit_form_top', [$this, 'renderReviewNotice'] ); |
| 214 | 228 | } |
@@ -220,7 +234,9 @@ discard block |
||
| 220 | 234 | */ |
| 221 | 235 | public function renderReviewNotice( $post ) |
| 222 | 236 | { |
| 223 | - if( !$this->isReviewPostType( $post ) || $this->isReviewEditable( $post ))return; |
|
| 237 | + if( !$this->isReviewPostType( $post ) || $this->isReviewEditable( $post )) { |
|
| 238 | + return; |
|
| 239 | + } |
|
| 224 | 240 | glsr( Notice::class )->addWarning( sprintf( |
| 225 | 241 | __( '%s reviews are read-only.', 'site-reviews' ), |
| 226 | 242 | glsr( Columns::class )->buildColumnReviewType( $post->ID ) |
@@ -240,7 +256,9 @@ discard block |
||
| 240 | 256 | */ |
| 241 | 257 | public function renderTaxonomyMetabox( $post ) |
| 242 | 258 | { |
| 243 | - if( !$this->isReviewPostType( $post ))return; |
|
| 259 | + if( !$this->isReviewPostType( $post )) { |
|
| 260 | + return; |
|
| 261 | + } |
|
| 244 | 262 | glsr()->render( 'partials/editor/metabox-categories', [ |
| 245 | 263 | 'post' => $post, |
| 246 | 264 | 'tax_name' => Application::TAXONOMY, |
@@ -255,7 +273,9 @@ discard block |
||
| 255 | 273 | */ |
| 256 | 274 | public function revertReview() |
| 257 | 275 | { |
| 258 | - if( filter_input( INPUT_GET, 'plugin' ) != Application::ID )return; |
|
| 276 | + if( filter_input( INPUT_GET, 'plugin' ) != Application::ID ) { |
|
| 277 | + return; |
|
| 278 | + } |
|
| 259 | 279 | check_admin_referer( 'revert-review_'.( $postId = $this->getPostId() )); |
| 260 | 280 | glsr( ReviewManager::class )->revert( $postId ); |
| 261 | 281 | $this->redirect( $postId, 52 ); |
@@ -279,7 +299,9 @@ discard block |
||
| 279 | 299 | protected function buildAssignedToTemplate( $assignedTo, WP_Post $post ) |
| 280 | 300 | { |
| 281 | 301 | $assignedPost = glsr( Database::class )->getAssignedToPost( $post->ID, $assignedTo ); |
| 282 | - if( !( $assignedPost instanceof WP_Post ))return; |
|
| 302 | + if( !( $assignedPost instanceof WP_Post )) { |
|
| 303 | + return; |
|
| 304 | + } |
|
| 283 | 305 | return glsr( Template::class )->build( 'partials/editor/assigned-post', [ |
| 284 | 306 | 'context' => [ |
| 285 | 307 | 'data.url' => (string)get_permalink( $assignedPost ), |
@@ -325,7 +347,9 @@ discard block |
||
| 325 | 347 | */ |
| 326 | 348 | protected function getReviewType( $review ) |
| 327 | 349 | { |
| 328 | - if( count( glsr()->reviewTypes ) < 2 )return; |
|
| 350 | + if( count( glsr()->reviewTypes ) < 2 ) { |
|
| 351 | + return; |
|
| 352 | + } |
|
| 329 | 353 | $reviewType = array_key_exists( $review->review_type, glsr()->reviewTypes ) |
| 330 | 354 | ? glsr()->reviewTypes[$review->review_type] |
| 331 | 355 | : __( 'Unknown', 'site-reviews' ); |
@@ -21,7 +21,7 @@ discard block |
||
| 21 | 21 | */ |
| 22 | 22 | public function build() |
| 23 | 23 | { |
| 24 | - glsr_log()->error( 'Build method is not implemented for '.get_class( $this )); |
|
| 24 | + glsr_log()->error( 'Build method is not implemented for '.get_class( $this ) ); |
|
| 25 | 25 | } |
| 26 | 26 | |
| 27 | 27 | /** |
@@ -41,8 +41,8 @@ discard block |
||
| 41 | 41 | wp_parse_args( $args, static::defaults() ), |
| 42 | 42 | static::required() |
| 43 | 43 | ); |
| 44 | - $merged['class'] = implode( ' ', static::mergedAttribute( 'class', ' ', $args )); |
|
| 45 | - $merged['style'] = implode( ';', static::mergedAttribute( 'style', ';', $args )); |
|
| 44 | + $merged['class'] = implode( ' ', static::mergedAttribute( 'class', ' ', $args ) ); |
|
| 45 | + $merged['style'] = implode( ';', static::mergedAttribute( 'style', ';', $args ) ); |
|
| 46 | 46 | return $merged; |
| 47 | 47 | } |
| 48 | 48 | |
@@ -54,10 +54,10 @@ discard block |
||
| 54 | 54 | public static function mergedAttribute( $key, $delimiter, array $args ) |
| 55 | 55 | { |
| 56 | 56 | return array_filter( array_merge( |
| 57 | - explode( $delimiter, glsr_get( $args, $key )), |
|
| 58 | - explode( $delimiter, glsr_get( static::defaults(), $key )), |
|
| 59 | - explode( $delimiter, glsr_get( static::required(), $key )) |
|
| 60 | - )); |
|
| 57 | + explode( $delimiter, glsr_get( $args, $key ) ), |
|
| 58 | + explode( $delimiter, glsr_get( static::defaults(), $key ) ), |
|
| 59 | + explode( $delimiter, glsr_get( static::required(), $key ) ) |
|
| 60 | + ) ); |
|
| 61 | 61 | } |
| 62 | 62 | |
| 63 | 63 | /** |
@@ -50,12 +50,12 @@ discard block |
||
| 50 | 50 | 'templates/form/submit-button', |
| 51 | 51 | 'templates/reviews-form', |
| 52 | 52 | ]; |
| 53 | - if( !preg_match( '('.implode( '|', $styledViews ).')', $view )) { |
|
| 53 | + if( !preg_match( '('.implode( '|', $styledViews ).')', $view ) ) { |
|
| 54 | 54 | return $view; |
| 55 | 55 | } |
| 56 | 56 | $views = $this->generatePossibleViews( $view ); |
| 57 | 57 | foreach( $views as $possibleView ) { |
| 58 | - if( !file_exists( glsr()->file( $possibleView )))continue; |
|
| 58 | + if( !file_exists( glsr()->file( $possibleView ) ) )continue; |
|
| 59 | 59 | return glsr( Helper::class )->removePrefix( 'views/', $possibleView ); |
| 60 | 60 | } |
| 61 | 61 | return $view; |
@@ -88,7 +88,7 @@ discard block |
||
| 88 | 88 | */ |
| 89 | 89 | public function modifyField( Builder $instance ) |
| 90 | 90 | { |
| 91 | - if( !$this->isPublicInstance( $instance ) || empty( array_filter( $this->fields )))return; |
|
| 91 | + if( !$this->isPublicInstance( $instance ) || empty(array_filter( $this->fields )) )return; |
|
| 92 | 92 | call_user_func_array( [$this, 'customize'], [$instance] ); |
| 93 | 93 | } |
| 94 | 94 | |
@@ -105,10 +105,10 @@ discard block |
||
| 105 | 105 | */ |
| 106 | 106 | protected function customize( Builder $instance ) |
| 107 | 107 | { |
| 108 | - if( !array_key_exists( $instance->tag, $this->fields ))return; |
|
| 109 | - $args = wp_parse_args( $instance->args, array_fill_keys( ['class', 'type'], '' )); |
|
| 108 | + if( !array_key_exists( $instance->tag, $this->fields ) )return; |
|
| 109 | + $args = wp_parse_args( $instance->args, array_fill_keys( ['class', 'type'], '' ) ); |
|
| 110 | 110 | $key = $instance->tag.'_'.$args['type']; |
| 111 | - $classes = glsr_get( $this->fields, $key, glsr_get( $this->fields, $instance->tag )); |
|
| 111 | + $classes = glsr_get( $this->fields, $key, glsr_get( $this->fields, $instance->tag ) ); |
|
| 112 | 112 | $instance->args['class'] = trim( $args['class'].' '.$classes ); |
| 113 | 113 | do_action_ref_array( 'site-reviews/customize/'.$this->style, [$instance] ); |
| 114 | 114 | } |
@@ -140,7 +140,7 @@ discard block |
||
| 140 | 140 | $args = wp_parse_args( $instance->args, [ |
| 141 | 141 | 'is_public' => false, |
| 142 | 142 | 'is_raw' => false, |
| 143 | - ]); |
|
| 143 | + ] ); |
|
| 144 | 144 | if( is_admin() || !$args['is_public'] || $args['is_raw'] ) { |
| 145 | 145 | return false; |
| 146 | 146 | } |
@@ -55,7 +55,9 @@ discard block |
||
| 55 | 55 | } |
| 56 | 56 | $views = $this->generatePossibleViews( $view ); |
| 57 | 57 | foreach( $views as $possibleView ) { |
| 58 | - if( !file_exists( glsr()->file( $possibleView )))continue; |
|
| 58 | + if( !file_exists( glsr()->file( $possibleView ))) { |
|
| 59 | + continue; |
|
| 60 | + } |
|
| 59 | 61 | return glsr( Helper::class )->removePrefix( 'views/', $possibleView ); |
| 60 | 62 | } |
| 61 | 63 | return $view; |
@@ -88,7 +90,9 @@ discard block |
||
| 88 | 90 | */ |
| 89 | 91 | public function modifyField( Builder $instance ) |
| 90 | 92 | { |
| 91 | - if( !$this->isPublicInstance( $instance ) || empty( array_filter( $this->fields )))return; |
|
| 93 | + if( !$this->isPublicInstance( $instance ) || empty( array_filter( $this->fields ))) { |
|
| 94 | + return; |
|
| 95 | + } |
|
| 92 | 96 | call_user_func_array( [$this, 'customize'], [$instance] ); |
| 93 | 97 | } |
| 94 | 98 | |
@@ -105,7 +109,9 @@ discard block |
||
| 105 | 109 | */ |
| 106 | 110 | protected function customize( Builder $instance ) |
| 107 | 111 | { |
| 108 | - if( !array_key_exists( $instance->tag, $this->fields ))return; |
|
| 112 | + if( !array_key_exists( $instance->tag, $this->fields )) { |
|
| 113 | + return; |
|
| 114 | + } |
|
| 109 | 115 | $args = wp_parse_args( $instance->args, array_fill_keys( ['class', 'type'], '' )); |
| 110 | 116 | $key = $instance->tag.'_'.$args['type']; |
| 111 | 117 | $classes = glsr_get( $this->fields, $key, glsr_get( $this->fields, $instance->tag )); |
@@ -26,15 +26,15 @@ discard block |
||
| 26 | 26 | { |
| 27 | 27 | sort( $newTTIds ); |
| 28 | 28 | sort( $oldTTIds ); |
| 29 | - if( $newTTIds === $oldTTIds || !$this->isReviewPostId( $postId ))return; |
|
| 30 | - $review = glsr( ReviewManager::class )->single( get_post( $postId )); |
|
| 29 | + if( $newTTIds === $oldTTIds || !$this->isReviewPostId( $postId ) )return; |
|
| 30 | + $review = glsr( ReviewManager::class )->single( get_post( $postId ) ); |
|
| 31 | 31 | $ignoredIds = array_intersect( $oldTTIds, $newTTIds ); |
| 32 | 32 | $decreasedIds = array_diff( $oldTTIds, $ignoredIds ); |
| 33 | 33 | $increasedIds = array_diff( $newTTIds, $ignoredIds ); |
| 34 | - if( $review->term_ids = glsr( Database::class )->getTermIds( $decreasedIds, 'term_taxonomy_id' )) { |
|
| 34 | + if( $review->term_ids = glsr( Database::class )->getTermIds( $decreasedIds, 'term_taxonomy_id' ) ) { |
|
| 35 | 35 | glsr( CountsManager::class )->decreaseTermCounts( $review ); |
| 36 | 36 | } |
| 37 | - if( $review->term_ids = glsr( Database::class )->getTermIds( $increasedIds, 'term_taxonomy_id' )) { |
|
| 37 | + if( $review->term_ids = glsr( Database::class )->getTermIds( $increasedIds, 'term_taxonomy_id' ) ) { |
|
| 38 | 38 | glsr( CountsManager::class )->increaseTermCounts( $review ); |
| 39 | 39 | } |
| 40 | 40 | } |
@@ -48,8 +48,8 @@ discard block |
||
| 48 | 48 | */ |
| 49 | 49 | public function onAfterChangeStatus( $newStatus, $oldStatus, $post ) |
| 50 | 50 | { |
| 51 | - if( glsr_get( $post, 'post_type') != Application::POST_TYPE || in_array( $oldStatus, ['new', $newStatus] ))return; |
|
| 52 | - $review = glsr( ReviewManager::class )->single( get_post( $post->ID )); |
|
| 51 | + if( glsr_get( $post, 'post_type' ) != Application::POST_TYPE || in_array( $oldStatus, ['new', $newStatus] ) )return; |
|
| 52 | + $review = glsr( ReviewManager::class )->single( get_post( $post->ID ) ); |
|
| 53 | 53 | if( $post->post_status == 'publish' ) { |
| 54 | 54 | glsr( CountsManager::class )->increase( $review ); |
| 55 | 55 | } |
@@ -75,8 +75,8 @@ discard block |
||
| 75 | 75 | */ |
| 76 | 76 | public function onBeforeDelete( $postId ) |
| 77 | 77 | { |
| 78 | - if( !$this->isReviewPostId( $postId ))return; |
|
| 79 | - $review = glsr( ReviewManager::class )->single( get_post( $postId )); |
|
| 78 | + if( !$this->isReviewPostId( $postId ) )return; |
|
| 79 | + $review = glsr( ReviewManager::class )->single( get_post( $postId ) ); |
|
| 80 | 80 | glsr( CountsManager::class )->decrease( $review ); |
| 81 | 81 | } |
| 82 | 82 | |
@@ -93,7 +93,7 @@ discard block |
||
| 93 | 93 | if( !$this->isReviewPostId( $postId ) |
| 94 | 94 | || !in_array( $metaKey, ['assigned_to', 'rating', 'review_type'] ) |
| 95 | 95 | )return; |
| 96 | - $review = glsr( ReviewManager::class )->single( get_post( $postId )); |
|
| 96 | + $review = glsr( ReviewManager::class )->single( get_post( $postId ) ); |
|
| 97 | 97 | if( $review->$metaKey == $metaValue )return; |
| 98 | 98 | $method = glsr( Helper::class )->buildMethodName( $metaKey, 'onBeforeChange' ); |
| 99 | 99 | call_user_func( [$this, $method], $review, $metaValue ); |
@@ -26,7 +26,9 @@ discard block |
||
| 26 | 26 | { |
| 27 | 27 | sort( $newTTIds ); |
| 28 | 28 | sort( $oldTTIds ); |
| 29 | - if( $newTTIds === $oldTTIds || !$this->isReviewPostId( $postId ))return; |
|
| 29 | + if( $newTTIds === $oldTTIds || !$this->isReviewPostId( $postId )) { |
|
| 30 | + return; |
|
| 31 | + } |
|
| 30 | 32 | $review = glsr( ReviewManager::class )->single( get_post( $postId )); |
| 31 | 33 | $ignoredIds = array_intersect( $oldTTIds, $newTTIds ); |
| 32 | 34 | $decreasedIds = array_diff( $oldTTIds, $ignoredIds ); |
@@ -48,7 +50,9 @@ discard block |
||
| 48 | 50 | */ |
| 49 | 51 | public function onAfterChangeStatus( $newStatus, $oldStatus, $post ) |
| 50 | 52 | { |
| 51 | - if( glsr_get( $post, 'post_type') != Application::POST_TYPE || in_array( $oldStatus, ['new', $newStatus] ))return; |
|
| 53 | + if( glsr_get( $post, 'post_type') != Application::POST_TYPE || in_array( $oldStatus, ['new', $newStatus] )) { |
|
| 54 | + return; |
|
| 55 | + } |
|
| 52 | 56 | $review = glsr( ReviewManager::class )->single( get_post( $post->ID )); |
| 53 | 57 | if( $post->post_status == 'publish' ) { |
| 54 | 58 | glsr( CountsManager::class )->increase( $review ); |
@@ -64,7 +68,9 @@ discard block |
||
| 64 | 68 | */ |
| 65 | 69 | public function onAfterCreate( Review $review ) |
| 66 | 70 | { |
| 67 | - if( $review->status !== 'publish' )return; |
|
| 71 | + if( $review->status !== 'publish' ) { |
|
| 72 | + return; |
|
| 73 | + } |
|
| 68 | 74 | glsr( CountsManager::class )->increase( $review ); |
| 69 | 75 | } |
| 70 | 76 | |
@@ -75,7 +81,9 @@ discard block |
||
| 75 | 81 | */ |
| 76 | 82 | public function onBeforeDelete( $postId ) |
| 77 | 83 | { |
| 78 | - if( !$this->isReviewPostId( $postId ))return; |
|
| 84 | + if( !$this->isReviewPostId( $postId )) { |
|
| 85 | + return; |
|
| 86 | + } |
|
| 79 | 87 | $review = glsr( ReviewManager::class )->single( get_post( $postId )); |
| 80 | 88 | glsr( CountsManager::class )->decrease( $review ); |
| 81 | 89 | } |
@@ -92,9 +100,13 @@ discard block |
||
| 92 | 100 | { |
| 93 | 101 | if( !$this->isReviewPostId( $postId ) |
| 94 | 102 | || !in_array( $metaKey, ['assigned_to', 'rating', 'review_type'] ) |
| 95 | - )return; |
|
| 103 | + ) { |
|
| 104 | + return; |
|
| 105 | + } |
|
| 96 | 106 | $review = glsr( ReviewManager::class )->single( get_post( $postId )); |
| 97 | - if( $review->$metaKey == $metaValue )return; |
|
| 107 | + if( $review->$metaKey == $metaValue ) { |
|
| 108 | + return; |
|
| 109 | + } |
|
| 98 | 110 | $method = glsr( Helper::class )->buildMethodName( $metaKey, 'onBeforeChange' ); |
| 99 | 111 | call_user_func( [$this, $method], $review, $metaValue ); |
| 100 | 112 | } |
@@ -35,12 +35,12 @@ discard block |
||
| 35 | 35 | */ |
| 36 | 36 | public function renderTaxonomyFilter() |
| 37 | 37 | { |
| 38 | - if( !is_object_in_taxonomy( glsr_current_screen()->post_type, Application::TAXONOMY ))return; |
|
| 38 | + if( !is_object_in_taxonomy( glsr_current_screen()->post_type, Application::TAXONOMY ) )return; |
|
| 39 | 39 | echo glsr( Builder::class )->label( __( 'Filter by category', 'site-reviews' ), [ |
| 40 | 40 | 'class' => 'screen-reader-text', |
| 41 | 41 | 'for' => Application::TAXONOMY, |
| 42 | - ]); |
|
| 43 | - wp_dropdown_categories([ |
|
| 42 | + ] ); |
|
| 43 | + wp_dropdown_categories( [ |
|
| 44 | 44 | 'depth' => 3, |
| 45 | 45 | 'hide_empty' => true, |
| 46 | 46 | 'hide_if_empty' => true, |
@@ -52,7 +52,7 @@ discard block |
||
| 52 | 52 | 'show_option_all' => $this->getShowOptionAll(), |
| 53 | 53 | 'taxonomy' => Application::TAXONOMY, |
| 54 | 54 | 'value_field' => 'slug', |
| 55 | - ]); |
|
| 55 | + ] ); |
|
| 56 | 56 | } |
| 57 | 57 | |
| 58 | 58 | /** |
@@ -69,7 +69,7 @@ discard block |
||
| 69 | 69 | { |
| 70 | 70 | if( $taxonomy != Application::TAXONOMY || count( $newTTIds ) <= 1 )return; |
| 71 | 71 | $diff = array_diff( $newTTIds, $oldTTIds ); |
| 72 | - if( empty( $newTerm = array_shift( $diff ))) { |
|
| 72 | + if( empty($newTerm = array_shift( $diff )) ) { |
|
| 73 | 73 | $newTerm = array_shift( $newTTIds ); |
| 74 | 74 | } |
| 75 | 75 | if( $newTerm ) { |
@@ -93,7 +93,7 @@ discard block |
||
| 93 | 93 | { |
| 94 | 94 | $taxonomy = get_taxonomy( Application::TAXONOMY ); |
| 95 | 95 | return $taxonomy |
| 96 | - ? ucfirst( strtolower( $taxonomy->labels->all_items )) |
|
| 96 | + ? ucfirst( strtolower( $taxonomy->labels->all_items ) ) |
|
| 97 | 97 | : ''; |
| 98 | 98 | } |
| 99 | 99 | } |
@@ -35,7 +35,9 @@ discard block |
||
| 35 | 35 | */ |
| 36 | 36 | public function renderTaxonomyFilter() |
| 37 | 37 | { |
| 38 | - if( !is_object_in_taxonomy( glsr_current_screen()->post_type, Application::TAXONOMY ))return; |
|
| 38 | + if( !is_object_in_taxonomy( glsr_current_screen()->post_type, Application::TAXONOMY )) { |
|
| 39 | + return; |
|
| 40 | + } |
|
| 39 | 41 | echo glsr( Builder::class )->label( __( 'Filter by category', 'site-reviews' ), [ |
| 40 | 42 | 'class' => 'screen-reader-text', |
| 41 | 43 | 'for' => Application::TAXONOMY, |
@@ -67,7 +69,9 @@ discard block |
||
| 67 | 69 | */ |
| 68 | 70 | public function restrictTermSelection( $postId, $terms, $newTTIds, $taxonomy, $append, $oldTTIds ) |
| 69 | 71 | { |
| 70 | - if( $taxonomy != Application::TAXONOMY || count( $newTTIds ) <= 1 )return; |
|
| 72 | + if( $taxonomy != Application::TAXONOMY || count( $newTTIds ) <= 1 ) { |
|
| 73 | + return; |
|
| 74 | + } |
|
| 71 | 75 | $diff = array_diff( $newTTIds, $oldTTIds ); |
| 72 | 76 | if( empty( $newTerm = array_shift( $diff ))) { |
| 73 | 77 | $newTerm = array_shift( $newTTIds ); |
@@ -73,7 +73,7 @@ discard block |
||
| 73 | 73 | public function send() |
| 74 | 74 | { |
| 75 | 75 | if( !$this->message || !$this->subject || !$this->to )return; |
| 76 | - add_action( 'wp_mail_failed', [$this, 'logMailError']); |
|
| 76 | + add_action( 'wp_mail_failed', [$this, 'logMailError'] ); |
|
| 77 | 77 | $sent = wp_mail( |
| 78 | 78 | $this->to, |
| 79 | 79 | $this->subject, |
@@ -81,7 +81,7 @@ discard block |
||
| 81 | 81 | $this->headers, |
| 82 | 82 | $this->attachments |
| 83 | 83 | ); |
| 84 | - remove_action( 'wp_mail_failed', [$this, 'logMailError']); |
|
| 84 | + remove_action( 'wp_mail_failed', [$this, 'logMailError'] ); |
|
| 85 | 85 | $this->reset(); |
| 86 | 86 | return $sent; |
| 87 | 87 | } |
@@ -92,8 +92,8 @@ discard block |
||
| 92 | 92 | */ |
| 93 | 93 | public function buildPlainTextMessage( PHPMailer $phpmailer ) |
| 94 | 94 | { |
| 95 | - if( empty( $this->email ))return; |
|
| 96 | - if( $phpmailer->ContentType === 'text/plain' || !empty( $phpmailer->AltBody ))return; |
|
| 95 | + if( empty($this->email) )return; |
|
| 96 | + if( $phpmailer->ContentType === 'text/plain' || !empty($phpmailer->AltBody) )return; |
|
| 97 | 97 | $message = $this->stripHtmlTags( $phpmailer->Body ); |
| 98 | 98 | $phpmailer->AltBody = apply_filters( 'site-reviews/email/message', $message, 'text', $this ); |
| 99 | 99 | } |
@@ -106,10 +106,10 @@ discard block |
||
| 106 | 106 | $allowed = [ |
| 107 | 107 | 'bcc', 'cc', 'from', 'reply-to', |
| 108 | 108 | ]; |
| 109 | - $headers = array_intersect_key( $this->email, array_flip( $allowed )); |
|
| 109 | + $headers = array_intersect_key( $this->email, array_flip( $allowed ) ); |
|
| 110 | 110 | $headers = array_filter( $headers ); |
| 111 | 111 | foreach( $headers as $key => $value ) { |
| 112 | - unset( $headers[$key] ); |
|
| 112 | + unset($headers[$key]); |
|
| 113 | 113 | $headers[] = $key.': '.$value; |
| 114 | 114 | } |
| 115 | 115 | $headers[] = 'Content-Type: text/html'; |
@@ -121,16 +121,16 @@ discard block |
||
| 121 | 121 | */ |
| 122 | 122 | protected function buildHtmlMessage() |
| 123 | 123 | { |
| 124 | - $template = trim( glsr( OptionManager::class )->get( 'settings.general.notification_message' )); |
|
| 125 | - if( !empty( $template )) { |
|
| 124 | + $template = trim( glsr( OptionManager::class )->get( 'settings.general.notification_message' ) ); |
|
| 125 | + if( !empty($template) ) { |
|
| 126 | 126 | $message = glsr( Template::class )->interpolate( $template, $this->email['template-tags'], $this->email['template'] ); |
| 127 | 127 | } |
| 128 | 128 | else if( $this->email['template'] ) { |
| 129 | 129 | $message = glsr( Template::class )->build( 'templates/'.$this->email['template'], [ |
| 130 | 130 | 'context' => $this->email['template-tags'], |
| 131 | - ]); |
|
| 131 | + ] ); |
|
| 132 | 132 | } |
| 133 | - if( !isset( $message )) { |
|
| 133 | + if( !isset($message) ) { |
|
| 134 | 134 | $message = $this->email['message']; |
| 135 | 135 | } |
| 136 | 136 | $message = $this->email['before'].$message.$this->email['after']; |
@@ -141,7 +141,7 @@ discard block |
||
| 141 | 141 | $message = str_replace( ']]>', ']]>', $message ); |
| 142 | 142 | $message = glsr( Template::class )->build( 'partials/email/index', [ |
| 143 | 143 | 'context' => ['message' => $message], |
| 144 | - ]); |
|
| 144 | + ] ); |
|
| 145 | 145 | return apply_filters( 'site-reviews/email/message', stripslashes( $message ), 'html', $this ); |
| 146 | 146 | } |
| 147 | 147 | |
@@ -162,7 +162,7 @@ discard block |
||
| 162 | 162 | protected function normalize( array $email = [] ) |
| 163 | 163 | { |
| 164 | 164 | $email = shortcode_atts( glsr( EmailDefaults::class )->defaults(), $email ); |
| 165 | - if( empty( $email['reply-to'] )) { |
|
| 165 | + if( empty($email['reply-to']) ) { |
|
| 166 | 166 | $email['reply-to'] = $email['from']; |
| 167 | 167 | } |
| 168 | 168 | $this->email = apply_filters( 'site-reviews/email/compose', $email, $this ); |
@@ -72,7 +72,9 @@ discard block |
||
| 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 | add_action( 'wp_mail_failed', [$this, 'logMailError']); |
| 77 | 79 | $sent = wp_mail( |
| 78 | 80 | $this->to, |
@@ -92,8 +94,12 @@ discard block |
||
| 92 | 94 | */ |
| 93 | 95 | public function buildPlainTextMessage( PHPMailer $phpmailer ) |
| 94 | 96 | { |
| 95 | - if( empty( $this->email ))return; |
|
| 96 | - if( $phpmailer->ContentType === 'text/plain' || !empty( $phpmailer->AltBody ))return; |
|
| 97 | + if( empty( $this->email )) { |
|
| 98 | + return; |
|
| 99 | + } |
|
| 100 | + if( $phpmailer->ContentType === 'text/plain' || !empty( $phpmailer->AltBody )) { |
|
| 101 | + return; |
|
| 102 | + } |
|
| 97 | 103 | $message = $this->stripHtmlTags( $phpmailer->Body ); |
| 98 | 104 | $phpmailer->AltBody = apply_filters( 'site-reviews/email/message', $message, 'text', $this ); |
| 99 | 105 | } |
@@ -33,7 +33,7 @@ discard block |
||
| 33 | 33 | public function __construct() |
| 34 | 34 | { |
| 35 | 35 | $types = glsr( OptionManager::class )->get( 'settings.general.notifications', [] ); |
| 36 | - $this->email = count( array_intersect( ['admin', 'custom'], $types )) > 0; |
|
| 36 | + $this->email = count( array_intersect( ['admin', 'custom'], $types ) ) > 0; |
|
| 37 | 37 | $this->slack = in_array( 'slack', $types ); |
| 38 | 38 | $this->types = $types; |
| 39 | 39 | } |
@@ -43,7 +43,7 @@ discard block |
||
| 43 | 43 | */ |
| 44 | 44 | public function send( Review $review ) |
| 45 | 45 | { |
| 46 | - if( empty( $this->types ))return; |
|
| 46 | + if( empty($this->types) )return; |
|
| 47 | 47 | $this->review = $review; |
| 48 | 48 | $args = [ |
| 49 | 49 | 'link' => $this->getLink(), |
@@ -58,7 +58,7 @@ discard block |
||
| 58 | 58 | */ |
| 59 | 59 | protected function buildEmail( array $args ) |
| 60 | 60 | { |
| 61 | - return glsr( Email::class )->compose([ |
|
| 61 | + return glsr( Email::class )->compose( [ |
|
| 62 | 62 | 'to' => $this->getEmailAddresses(), |
| 63 | 63 | 'subject' => $args['title'], |
| 64 | 64 | 'template' => 'email-notification', |
@@ -71,7 +71,7 @@ discard block |
||
| 71 | 71 | 'review_rating' => $this->review->rating, |
| 72 | 72 | 'review_title' => $this->review->title, |
| 73 | 73 | ], |
| 74 | - ]); |
|
| 74 | + ] ); |
|
| 75 | 75 | } |
| 76 | 76 | |
| 77 | 77 | /** |
@@ -83,7 +83,7 @@ discard block |
||
| 83 | 83 | 'button_url' => $args['link'], |
| 84 | 84 | 'fallback' => $this->buildEmail( $args )->read( 'plaintext' ), |
| 85 | 85 | 'pretext' => $args['title'], |
| 86 | - ]); |
|
| 86 | + ] ); |
|
| 87 | 87 | } |
| 88 | 88 | |
| 89 | 89 | /** |
@@ -92,23 +92,23 @@ discard block |
||
| 92 | 92 | protected function getEmailAddresses() |
| 93 | 93 | { |
| 94 | 94 | $emails = []; |
| 95 | - if( in_array( 'admin', $this->types )) { |
|
| 95 | + if( in_array( 'admin', $this->types ) ) { |
|
| 96 | 96 | $emails[] = get_option( 'admin_email' ); |
| 97 | 97 | } |
| 98 | - if( in_array( 'author', $this->types )) { |
|
| 99 | - $assignedPost = get_post( intval( $this->review->assigned_to )); |
|
| 98 | + if( in_array( 'author', $this->types ) ) { |
|
| 99 | + $assignedPost = get_post( intval( $this->review->assigned_to ) ); |
|
| 100 | 100 | if( $assignedPost instanceof WP_Post ) { |
| 101 | 101 | $this->email = true; |
| 102 | - $emails[] = get_the_author_meta( 'user_email', intval( $assignedPost->post_author )); |
|
| 102 | + $emails[] = get_the_author_meta( 'user_email', intval( $assignedPost->post_author ) ); |
|
| 103 | 103 | } |
| 104 | 104 | } |
| 105 | - if( in_array( 'custom', $this->types )) { |
|
| 105 | + if( in_array( 'custom', $this->types ) ) { |
|
| 106 | 106 | $customEmails = glsr( OptionManager::class )->get( 'settings.general.notification_email' ); |
| 107 | 107 | $customEmails = str_replace( [' ', ',', ';'], ',', $customEmails ); |
| 108 | 108 | $customEmails = explode( ',', $customEmails ); |
| 109 | 109 | $emails = array_merge( $emails, $customEmails ); |
| 110 | 110 | } |
| 111 | - $emails = array_filter( array_keys( array_flip( $emails ))); |
|
| 111 | + $emails = array_filter( array_keys( array_flip( $emails ) ) ); |
|
| 112 | 112 | return apply_filters( 'site-reviews/notification/emails', $emails, $this->review ); |
| 113 | 113 | } |
| 114 | 114 | |
@@ -125,16 +125,16 @@ discard block |
||
| 125 | 125 | */ |
| 126 | 126 | protected function getTitle() |
| 127 | 127 | { |
| 128 | - $assignedTitle = get_the_title( intval( $this->review->assigned_to )); |
|
| 128 | + $assignedTitle = get_the_title( intval( $this->review->assigned_to ) ); |
|
| 129 | 129 | $title = _nx( |
| 130 | 130 | 'New %s-star review', |
| 131 | 131 | 'New %s-star review of: %s', |
| 132 | - intval( empty( $assignedTitle )), |
|
| 132 | + intval( empty($assignedTitle) ), |
|
| 133 | 133 | 'This string differs depending on whether or not the review has been assigned to a post.', |
| 134 | 134 | 'site-reviews' |
| 135 | 135 | ); |
| 136 | 136 | $title = sprintf( '[%s] %s', |
| 137 | - wp_specialchars_decode( strval( get_option( 'blogname' )), ENT_QUOTES ), |
|
| 137 | + wp_specialchars_decode( strval( get_option( 'blogname' ) ), ENT_QUOTES ), |
|
| 138 | 138 | sprintf( $title, $this->review->rating, $assignedTitle ) |
| 139 | 139 | ); |
| 140 | 140 | return apply_filters( 'site-reviews/notification/title', $title, $this->review ); |
@@ -146,7 +146,7 @@ discard block |
||
| 146 | 146 | protected function sendToEmail( array $args ) |
| 147 | 147 | { |
| 148 | 148 | $email = $this->buildEmail( $args ); |
| 149 | - if( empty( $email->to )) { |
|
| 149 | + if( empty($email->to) ) { |
|
| 150 | 150 | glsr_log()->error( 'Email notification was not sent (missing email address)' ); |
| 151 | 151 | return; |
| 152 | 152 | } |
@@ -161,7 +161,7 @@ discard block |
||
| 161 | 161 | if( !$this->slack )return; |
| 162 | 162 | $notification = $this->buildSlackNotification( $args ); |
| 163 | 163 | $result = $notification->send(); |
| 164 | - if( is_wp_error( $result )) { |
|
| 164 | + if( is_wp_error( $result ) ) { |
|
| 165 | 165 | $notification->review = null; |
| 166 | 166 | glsr_log()->error( $result->get_error_message() )->debug( $notification ); |
| 167 | 167 | } |
@@ -43,7 +43,9 @@ discard block |
||
| 43 | 43 | */ |
| 44 | 44 | public function send( Review $review ) |
| 45 | 45 | { |
| 46 | - if( empty( $this->types ))return; |
|
| 46 | + if( empty( $this->types )) { |
|
| 47 | + return; |
|
| 48 | + } |
|
| 47 | 49 | $this->review = $review; |
| 48 | 50 | $args = [ |
| 49 | 51 | 'link' => $this->getLink(), |
@@ -158,7 +160,9 @@ discard block |
||
| 158 | 160 | */ |
| 159 | 161 | protected function sendToSlack( array $args ) |
| 160 | 162 | { |
| 161 | - if( !$this->slack )return; |
|
| 163 | + if( !$this->slack ) { |
|
| 164 | + return; |
|
| 165 | + } |
|
| 162 | 166 | $notification = $this->buildSlackNotification( $args ); |
| 163 | 167 | $result = $notification->send(); |
| 164 | 168 | if( is_wp_error( $result )) { |