@@ -108,12 +108,16 @@ |
||
| 108 | 108 | public function renderTinymceButton( $editorId ) |
| 109 | 109 | { |
| 110 | 110 | $allowedEditors = apply_filters( 'site-reviews/tinymce/editor-ids', ['content'], $editorId ); |
| 111 | - if( glsr_current_screen()->base != 'post' || !in_array( $editorId, $allowedEditors ))return; |
|
| 111 | + if( glsr_current_screen()->base != 'post' || !in_array( $editorId, $allowedEditors )) { |
|
| 112 | + return; |
|
| 113 | + } |
|
| 112 | 114 | $shortcodes = []; |
| 113 | 115 | foreach( glsr()->mceShortcodes as $shortcode => $values ) { |
| 114 | 116 | $shortcodes[$shortcode] = $values; |
| 115 | 117 | } |
| 116 | - if( empty( $shortcodes ))return; |
|
| 118 | + if( empty( $shortcodes )) { |
|
| 119 | + return; |
|
| 120 | + } |
|
| 117 | 121 | glsr()->render( 'partials/editor/tinymce', [ |
| 118 | 122 | 'shortcodes' => $shortcodes, |
| 119 | 123 | ]); |
@@ -24,7 +24,7 @@ discard block |
||
| 24 | 24 | */ |
| 25 | 25 | public function enqueueAssets() |
| 26 | 26 | { |
| 27 | - $command = new EnqueueAdminAssets([ |
|
| 27 | + $command = new EnqueueAdminAssets( [ |
|
| 28 | 28 | 'pointers' => [[ |
| 29 | 29 | 'content' => __( 'You can pin exceptional reviews so that they are always shown first.', 'site-reviews' ), |
| 30 | 30 | 'id' => 'glsr-pointer-pinned', |
@@ -36,7 +36,7 @@ discard block |
||
| 36 | 36 | 'target' => '#misc-pub-pinned', |
| 37 | 37 | 'title' => __( 'Pin Your Reviews', 'site-reviews' ), |
| 38 | 38 | ]], |
| 39 | - ]); |
|
| 39 | + ] ); |
|
| 40 | 40 | $this->execute( $command ); |
| 41 | 41 | } |
| 42 | 42 | |
@@ -48,7 +48,7 @@ discard block |
||
| 48 | 48 | { |
| 49 | 49 | $links['settings'] = glsr( Builder::class )->a( __( 'Settings', 'site-reviews' ), [ |
| 50 | 50 | 'href' => admin_url( 'edit.php?post_type='.Application::POST_TYPE.'&page=settings' ), |
| 51 | - ]); |
|
| 51 | + ] ); |
|
| 52 | 52 | return $links; |
| 53 | 53 | } |
| 54 | 54 | |
@@ -59,19 +59,19 @@ discard block |
||
| 59 | 59 | public function filterDashboardGlanceItems( array $items ) |
| 60 | 60 | { |
| 61 | 61 | $postCount = wp_count_posts( Application::POST_TYPE ); |
| 62 | - if( empty( $postCount->publish )) { |
|
| 62 | + if( empty($postCount->publish) ) { |
|
| 63 | 63 | return $items; |
| 64 | 64 | } |
| 65 | 65 | $text = _n( '%s Review', '%s Reviews', $postCount->publish, 'site-reviews' ); |
| 66 | - $text = sprintf( $text, number_format_i18n( $postCount->publish )); |
|
| 66 | + $text = sprintf( $text, number_format_i18n( $postCount->publish ) ); |
|
| 67 | 67 | $items[] = current_user_can( get_post_type_object( Application::POST_TYPE )->cap->edit_posts ) |
| 68 | 68 | ? glsr( Builder::class )->a( $text, [ |
| 69 | 69 | 'class' => 'glsr-review-count', |
| 70 | 70 | 'href' => 'edit.php?post_type='.Application::POST_TYPE, |
| 71 | - ]) |
|
| 71 | + ] ) |
|
| 72 | 72 | : glsr( Builder::class )->span( $text, [ |
| 73 | 73 | 'class' => 'glsr-review-count', |
| 74 | - ]); |
|
| 74 | + ] ); |
|
| 75 | 75 | return $items; |
| 76 | 76 | } |
| 77 | 77 | |
@@ -81,7 +81,7 @@ discard block |
||
| 81 | 81 | */ |
| 82 | 82 | public function filterTinymcePlugins( array $plugins ) |
| 83 | 83 | { |
| 84 | - if( current_user_can( 'edit_posts' ) || current_user_can( 'edit_pages' )) { |
|
| 84 | + if( current_user_can( 'edit_posts' ) || current_user_can( 'edit_pages' ) ) { |
|
| 85 | 85 | $plugins['glsr_shortcode'] = glsr()->url( 'assets/scripts/mce-plugin.js' ); |
| 86 | 86 | } |
| 87 | 87 | return $plugins; |
@@ -93,11 +93,11 @@ discard block |
||
| 93 | 93 | */ |
| 94 | 94 | public function registerTinymcePopups() |
| 95 | 95 | { |
| 96 | - $command = new RegisterTinymcePopups([ |
|
| 96 | + $command = new RegisterTinymcePopups( [ |
|
| 97 | 97 | 'site_reviews' => esc_html__( 'Recent Reviews', 'site-reviews' ), |
| 98 | 98 | 'site_reviews_form' => esc_html__( 'Submit a Review', 'site-reviews' ), |
| 99 | 99 | 'site_reviews_summary' => esc_html__( 'Summary of Reviews', 'site-reviews' ), |
| 100 | - ]); |
|
| 100 | + ] ); |
|
| 101 | 101 | $this->execute( $command ); |
| 102 | 102 | } |
| 103 | 103 | |
@@ -109,15 +109,15 @@ discard block |
||
| 109 | 109 | public function renderTinymceButton( $editorId ) |
| 110 | 110 | { |
| 111 | 111 | $allowedEditors = apply_filters( 'site-reviews/tinymce/editor-ids', ['content'], $editorId ); |
| 112 | - if( glsr_current_screen()->base != 'post' || !in_array( $editorId, $allowedEditors ))return; |
|
| 112 | + if( glsr_current_screen()->base != 'post' || !in_array( $editorId, $allowedEditors ) )return; |
|
| 113 | 113 | $shortcodes = []; |
| 114 | 114 | foreach( glsr()->mceShortcodes as $shortcode => $values ) { |
| 115 | 115 | $shortcodes[$shortcode] = $values; |
| 116 | 116 | } |
| 117 | - if( empty( $shortcodes ))return; |
|
| 117 | + if( empty($shortcodes) )return; |
|
| 118 | 118 | glsr()->render( 'partials/editor/tinymce', [ |
| 119 | 119 | 'shortcodes' => $shortcodes, |
| 120 | - ]); |
|
| 120 | + ] ); |
|
| 121 | 121 | } |
| 122 | 122 | |
| 123 | 123 | /** |
@@ -126,7 +126,7 @@ discard block |
||
| 126 | 126 | public function routerClearConsole() |
| 127 | 127 | { |
| 128 | 128 | glsr( Console::class )->clear(); |
| 129 | - glsr( Notice::class )->addSuccess( __( 'Console cleared.', 'site-reviews' )); |
|
| 129 | + glsr( Notice::class )->addSuccess( __( 'Console cleared.', 'site-reviews' ) ); |
|
| 130 | 130 | } |
| 131 | 131 | |
| 132 | 132 | /** |
@@ -134,7 +134,7 @@ discard block |
||
| 134 | 134 | */ |
| 135 | 135 | public function routerFetchConsole() |
| 136 | 136 | { |
| 137 | - glsr( Notice::class )->addSuccess( __( 'Console reloaded.', 'site-reviews' )); |
|
| 137 | + glsr( Notice::class )->addSuccess( __( 'Console reloaded.', 'site-reviews' ) ); |
|
| 138 | 138 | } |
| 139 | 139 | |
| 140 | 140 | /** |
@@ -153,13 +153,13 @@ discard block |
||
| 153 | 153 | } |
| 154 | 154 | $postIds = glsr( SqlQueries::class )->getReviewsMeta( 'assigned_to' ); |
| 155 | 155 | foreach( $postIds as $postId ) { |
| 156 | - $countManager->setPostCounts( $postId, $countManager->buildPostCounts( $postId )); |
|
| 156 | + $countManager->setPostCounts( $postId, $countManager->buildPostCounts( $postId ) ); |
|
| 157 | 157 | } |
| 158 | 158 | $countManager->setCounts( $countManager->buildCounts() ); |
| 159 | 159 | if( $showNotice ) { |
| 160 | - glsr( Notice::class )->clear()->addSuccess( __( 'Recalculated rating counts.', 'site-reviews' )); |
|
| 160 | + glsr( Notice::class )->clear()->addSuccess( __( 'Recalculated rating counts.', 'site-reviews' ) ); |
|
| 161 | 161 | } |
| 162 | - glsr( OptionManager::class )->set( 'last_review_count', current_time( 'timestamp' )); |
|
| 162 | + glsr( OptionManager::class )->set( 'last_review_count', current_time( 'timestamp' ) ); |
|
| 163 | 163 | } |
| 164 | 164 | |
| 165 | 165 | /** |
@@ -193,17 +193,17 @@ discard block |
||
| 193 | 193 | { |
| 194 | 194 | $file = $_FILES['import-file']; |
| 195 | 195 | if( $file['error'] !== UPLOAD_ERR_OK ) { |
| 196 | - return glsr( Notice::class )->addError( $this->getUploadError( $file['error'] )); |
|
| 196 | + return glsr( Notice::class )->addError( $this->getUploadError( $file['error'] ) ); |
|
| 197 | 197 | } |
| 198 | - if( $file['type'] !== 'application/json' || !glsr( Helper::class )->endsWith( '.json', $file['name'] )) { |
|
| 199 | - return glsr( Notice::class )->addError( __( 'Please use a valid Site Reviews settings file.', 'site-reviews' )); |
|
| 198 | + if( $file['type'] !== 'application/json' || !glsr( Helper::class )->endsWith( '.json', $file['name'] ) ) { |
|
| 199 | + return glsr( Notice::class )->addError( __( 'Please use a valid Site Reviews settings file.', 'site-reviews' ) ); |
|
| 200 | 200 | } |
| 201 | 201 | $settings = json_decode( file_get_contents( $file['tmp_name'] ), true ); |
| 202 | - if( empty( $settings )) { |
|
| 203 | - return glsr( Notice::class )->addWarning( __( 'There were no settings found to import.', 'site-reviews' )); |
|
| 202 | + if( empty($settings) ) { |
|
| 203 | + return glsr( Notice::class )->addWarning( __( 'There were no settings found to import.', 'site-reviews' ) ); |
|
| 204 | 204 | } |
| 205 | - glsr( OptionManager::class )->set( glsr( OptionManager::class )->normalize( $settings )); |
|
| 206 | - glsr( Notice::class )->addSuccess( __( 'Settings imported.', 'site-reviews' )); |
|
| 205 | + glsr( OptionManager::class )->set( glsr( OptionManager::class )->normalize( $settings ) ); |
|
| 206 | + glsr( Notice::class )->addSuccess( __( 'Settings imported.', 'site-reviews' ) ); |
|
| 207 | 207 | } |
| 208 | 208 | |
| 209 | 209 | /** |
@@ -221,7 +221,7 @@ discard block |
||
| 221 | 221 | UPLOAD_ERR_CANT_WRITE => __( 'Failed to write file to disk.', 'site-reviews' ), |
| 222 | 222 | UPLOAD_ERR_EXTENSION => __( 'A PHP extension stopped the file upload.', 'site-reviews' ), |
| 223 | 223 | ]; |
| 224 | - return !isset( $errors[$errorCode] ) |
|
| 224 | + return !isset($errors[$errorCode]) |
|
| 225 | 225 | ? __( 'Unknown upload error.', 'site-reviews' ) |
| 226 | 226 | : $errors[$errorCode]; |
| 227 | 227 | } |
@@ -13,11 +13,11 @@ discard block |
||
| 13 | 13 | */ |
| 14 | 14 | public function handle( Command $command ) |
| 15 | 15 | { |
| 16 | - $postId = wp_update_post([ |
|
| 16 | + $postId = wp_update_post( [ |
|
| 17 | 17 | 'ID' => $command->id, |
| 18 | 18 | 'post_status' => $command->status, |
| 19 | - ]); |
|
| 20 | - if( is_wp_error( $postId )) { |
|
| 19 | + ] ); |
|
| 20 | + if( is_wp_error( $postId ) ) { |
|
| 21 | 21 | glsr_log()->error( $postId->get_error_message() ); |
| 22 | 22 | return []; |
| 23 | 23 | } |
@@ -39,7 +39,7 @@ discard block |
||
| 39 | 39 | 'aria-label' => '“'.esc_attr( $title ).'” ('.__( 'Edit', 'site-reviews' ).')', |
| 40 | 40 | 'class' => 'row-title', |
| 41 | 41 | 'href' => get_edit_post_link( $postId ), |
| 42 | - ]); |
|
| 42 | + ] ); |
|
| 43 | 43 | } |
| 44 | 44 | |
| 45 | 45 | /** |
@@ -49,7 +49,7 @@ discard block |
||
| 49 | 49 | protected function getPostState( $postId ) |
| 50 | 50 | { |
| 51 | 51 | ob_start(); |
| 52 | - _post_states( get_post( $postId )); |
|
| 52 | + _post_states( get_post( $postId ) ); |
|
| 53 | 53 | return ob_get_clean(); |
| 54 | 54 | } |
| 55 | 55 | |
@@ -59,13 +59,13 @@ discard block |
||
| 59 | 59 | protected function getStatusLinks() |
| 60 | 60 | { |
| 61 | 61 | global $avail_post_stati; |
| 62 | - require_once( ABSPATH.'wp-admin/includes/class-wp-posts-list-table.php' ); |
|
| 62 | + require_once(ABSPATH.'wp-admin/includes/class-wp-posts-list-table.php'); |
|
| 63 | 63 | $hookName = 'edit-'.Application::POST_TYPE; |
| 64 | 64 | set_current_screen( $hookName ); |
| 65 | 65 | $avail_post_stati = get_available_post_statuses( Application::POST_TYPE ); |
| 66 | 66 | $table = new \WP_Posts_List_Table( ['screen' => $hookName] ); |
| 67 | 67 | $views = apply_filters( 'views_'.$hookName, $table->get_views() ); // uses compat get_views() |
| 68 | - if( empty( $views ))return; |
|
| 68 | + if( empty($views) )return; |
|
| 69 | 69 | foreach( $views as $class => $view ) { |
| 70 | 70 | $views[$class] = "\t<li class='$class'>$view"; |
| 71 | 71 | } |
@@ -65,7 +65,9 @@ |
||
| 65 | 65 | $avail_post_stati = get_available_post_statuses( Application::POST_TYPE ); |
| 66 | 66 | $table = new \WP_Posts_List_Table( ['screen' => $hookName] ); |
| 67 | 67 | $views = apply_filters( 'views_'.$hookName, $table->get_views() ); // uses compat get_views() |
| 68 | - if( empty( $views ))return; |
|
| 68 | + if( empty( $views )) { |
|
| 69 | + return; |
|
| 70 | + } |
|
| 69 | 71 | foreach( $views as $class => $view ) { |
| 70 | 72 | $views[$class] = "\t<li class='$class'>$view"; |
| 71 | 73 | } |
@@ -20,9 +20,9 @@ discard block |
||
| 20 | 20 | * @see https://wordpress.org/plugins/wp-super-cache/ |
| 21 | 21 | */ |
| 22 | 22 | add_action( 'site-reviews/review/created', function( $review, $request ) { |
| 23 | - if( !function_exists( 'wp_cache_post_change' ))return; |
|
| 23 | + if( !function_exists( 'wp_cache_post_change' ) )return; |
|
| 24 | 24 | wp_cache_post_change( $request->post_id ); |
| 25 | - if( empty( $review->assigned_to ) || $review->assigned_to == $request->post_id )return; |
|
| 25 | + if( empty($review->assigned_to) || $review->assigned_to == $request->post_id )return; |
|
| 26 | 26 | wp_cache_post_change( $review->assigned_to ); |
| 27 | 27 | }, 10, 2 ); |
| 28 | 28 | |
@@ -33,7 +33,7 @@ discard block |
||
| 33 | 33 | */ |
| 34 | 34 | add_filter( 'sbp_exclude_defer_scripts', function( $scriptHandles ) { |
| 35 | 35 | $scriptHandles[] = 'site-reviews/google-recaptcha'; |
| 36 | - return array_keys( array_flip( $scriptHandles )); |
|
| 36 | + return array_keys( array_flip( $scriptHandles ) ); |
|
| 37 | 37 | }); |
| 38 | 38 | |
| 39 | 39 | /** |
@@ -43,8 +43,8 @@ discard block |
||
| 43 | 43 | * @see https://searchandfilter.com/ |
| 44 | 44 | */ |
| 45 | 45 | add_filter( 'sf_edit_query_args', function( $query ) { |
| 46 | - if( !empty( $query['meta_key'] ) && $query['meta_key'] == '_glsr_ranking' ) { |
|
| 47 | - unset( $query['meta_key'] ); |
|
| 46 | + if( !empty($query['meta_key']) && $query['meta_key'] == '_glsr_ranking' ) { |
|
| 47 | + unset($query['meta_key']); |
|
| 48 | 48 | $query['meta_query'] = [ |
| 49 | 49 | 'relation' => 'OR', |
| 50 | 50 | ['key' => '_glsr_ranking', 'compare' => 'EXISTS'], |
@@ -8,7 +8,9 @@ discard block |
||
| 8 | 8 | * @see https://www.elegantthemes.com/gallery/divi/ |
| 9 | 9 | */ |
| 10 | 10 | add_action( 'site-reviews/customize/divi', function( $instance ) { |
| 11 | - if( $instance->tag != 'label' || $instance->args['type'] != 'checkbox' )return; |
|
| 11 | + if( $instance->tag != 'label' || $instance->args['type'] != 'checkbox' ) { |
|
| 12 | + return; |
|
| 13 | + } |
|
| 12 | 14 | $instance->args['text'] = '<i></i>'.$instance->args['text']; |
| 13 | 15 | }); |
| 14 | 16 | |
@@ -20,9 +22,13 @@ discard block |
||
| 20 | 22 | * @see https://wordpress.org/plugins/wp-super-cache/ |
| 21 | 23 | */ |
| 22 | 24 | add_action( 'site-reviews/review/created', function( $review, $request ) { |
| 23 | - if( !function_exists( 'wp_cache_post_change' ))return; |
|
| 25 | + if( !function_exists( 'wp_cache_post_change' )) { |
|
| 26 | + return; |
|
| 27 | + } |
|
| 24 | 28 | wp_cache_post_change( $request->post_id ); |
| 25 | - if( empty( $review->assigned_to ) || $review->assigned_to == $request->post_id )return; |
|
| 29 | + if( empty( $review->assigned_to ) || $review->assigned_to == $request->post_id ) { |
|
| 30 | + return; |
|
| 31 | + } |
|
| 26 | 32 | wp_cache_post_change( $review->assigned_to ); |
| 27 | 33 | }, 10, 2 ); |
| 28 | 34 | |
@@ -56,9 +56,13 @@ discard block |
||
| 56 | 56 | protected function generateFields( array $fields ) |
| 57 | 57 | { |
| 58 | 58 | $generatedFields = array_map( function( $field ) { |
| 59 | - if( empty( $field ))return; |
|
| 59 | + if( empty( $field )) { |
|
| 60 | + return; |
|
| 61 | + } |
|
| 60 | 62 | $field = $this->normalize( $field ); |
| 61 | - if( !method_exists( $this, $method = 'normalize'.ucfirst( $field['type'] )))return; |
|
| 63 | + if( !method_exists( $this, $method = 'normalize'.ucfirst( $field['type'] ))) { |
|
| 64 | + return; |
|
| 65 | + } |
|
| 62 | 66 | return $this->$method( $field ); |
| 63 | 67 | }, $fields ); |
| 64 | 68 | return array_values( array_filter( $generatedFields )); |
@@ -73,7 +77,9 @@ discard block |
||
| 73 | 77 | if( !empty( $this->errors )) { |
| 74 | 78 | $errors = []; |
| 75 | 79 | foreach( $this->required as $name => $alert ) { |
| 76 | - if( false !== array_search( $name, glsr_array_column( $fields, 'name' )))continue; |
|
| 80 | + if( false !== array_search( $name, glsr_array_column( $fields, 'name' ))) { |
|
| 81 | + continue; |
|
| 82 | + } |
|
| 77 | 83 | $errors[] = $this->errors[$name]; |
| 78 | 84 | } |
| 79 | 85 | $this->errors = $errors; |
@@ -117,7 +123,9 @@ discard block |
||
| 117 | 123 | */ |
| 118 | 124 | protected function normalizeContainer( array $field ) |
| 119 | 125 | { |
| 120 | - if( !array_key_exists( 'html', $field ) && !array_key_exists( 'items', $field ))return; |
|
| 126 | + if( !array_key_exists( 'html', $field ) && !array_key_exists( 'items', $field )) { |
|
| 127 | + return; |
|
| 128 | + } |
|
| 121 | 129 | $field['items'] = $this->generateFields( $field['items'] ); |
| 122 | 130 | return $field; |
| 123 | 131 | } |
@@ -127,7 +135,9 @@ discard block |
||
| 127 | 135 | */ |
| 128 | 136 | protected function normalizeField( array $field, array $defaults ) |
| 129 | 137 | { |
| 130 | - if( !$this->validate( $field ))return; |
|
| 138 | + if( !$this->validate( $field )) { |
|
| 139 | + return; |
|
| 140 | + } |
|
| 131 | 141 | return array_filter( shortcode_atts( $defaults, $field ), function( $value ) { |
| 132 | 142 | return $value !== ''; |
| 133 | 143 | }); |
@@ -148,7 +158,9 @@ discard block |
||
| 148 | 158 | 'type' => '', |
| 149 | 159 | 'value' => '', |
| 150 | 160 | ]); |
| 151 | - if( !is_array( $listbox ))return; |
|
| 161 | + if( !is_array( $listbox )) { |
|
| 162 | + return; |
|
| 163 | + } |
|
| 152 | 164 | if( !array_key_exists( '', $listbox['options'] )) { |
| 153 | 165 | $listbox['options'] = ['' => $listbox['placeholder']] + $listbox['options']; |
| 154 | 166 | } |
@@ -48,7 +48,7 @@ discard block |
||
| 48 | 48 | 'label' => '['.$tag.']', |
| 49 | 49 | 'required' => $this->required, |
| 50 | 50 | 'title' => esc_html__( 'Shortcode', 'site-reviews' ), |
| 51 | - ]); |
|
| 51 | + ] ); |
|
| 52 | 52 | return $this; |
| 53 | 53 | } |
| 54 | 54 | |
@@ -58,12 +58,12 @@ discard block |
||
| 58 | 58 | protected function generateFields( array $fields ) |
| 59 | 59 | { |
| 60 | 60 | $generatedFields = array_map( function( $field ) { |
| 61 | - if( empty( $field ))return; |
|
| 61 | + if( empty($field) )return; |
|
| 62 | 62 | $field = $this->normalize( $field ); |
| 63 | - if( !method_exists( $this, $method = 'normalize'.ucfirst( $field['type'] )))return; |
|
| 63 | + if( !method_exists( $this, $method = 'normalize'.ucfirst( $field['type'] ) ) )return; |
|
| 64 | 64 | return $this->$method( $field ); |
| 65 | 65 | }, $fields ); |
| 66 | - return array_values( array_filter( $generatedFields )); |
|
| 66 | + return array_values( array_filter( $generatedFields ) ); |
|
| 67 | 67 | } |
| 68 | 68 | |
| 69 | 69 | /** |
@@ -73,7 +73,7 @@ discard block |
||
| 73 | 73 | protected function getCategories( $tooltip = '' ) |
| 74 | 74 | { |
| 75 | 75 | $terms = glsr( Database::class )->getTerms(); |
| 76 | - if( empty( $terms )) { |
|
| 76 | + if( empty($terms) ) { |
|
| 77 | 77 | return []; |
| 78 | 78 | } |
| 79 | 79 | return [ |
@@ -91,15 +91,15 @@ discard block |
||
| 91 | 91 | protected function getFields() |
| 92 | 92 | { |
| 93 | 93 | $fields = $this->generateFields( $this->fields() ); |
| 94 | - if( !empty( $this->errors )) { |
|
| 94 | + if( !empty($this->errors) ) { |
|
| 95 | 95 | $errors = []; |
| 96 | 96 | foreach( $this->required as $name => $alert ) { |
| 97 | - if( false !== array_search( $name, glsr_array_column( $fields, 'name' )))continue; |
|
| 97 | + if( false !== array_search( $name, glsr_array_column( $fields, 'name' ) ) )continue; |
|
| 98 | 98 | $errors[] = $this->errors[$name]; |
| 99 | 99 | } |
| 100 | 100 | $this->errors = $errors; |
| 101 | 101 | } |
| 102 | - return empty( $this->errors ) |
|
| 102 | + return empty($this->errors) |
|
| 103 | 103 | ? $fields |
| 104 | 104 | : $this->errors; |
| 105 | 105 | } |
@@ -109,7 +109,7 @@ discard block |
||
| 109 | 109 | */ |
| 110 | 110 | protected function getHideOptions() |
| 111 | 111 | { |
| 112 | - $classname = str_replace( 'Popup', 'Shortcode', get_class( $this )); |
|
| 112 | + $classname = str_replace( 'Popup', 'Shortcode', get_class( $this ) ); |
|
| 113 | 113 | $hideOptions = glsr( $classname )->getHideOptions(); |
| 114 | 114 | $options = []; |
| 115 | 115 | foreach( $hideOptions as $name => $tooltip ) { |
@@ -149,7 +149,7 @@ discard block |
||
| 149 | 149 | return wp_parse_args( $field, [ |
| 150 | 150 | 'items' => [], |
| 151 | 151 | 'type' => '', |
| 152 | - ]); |
|
| 152 | + ] ); |
|
| 153 | 153 | } |
| 154 | 154 | |
| 155 | 155 | /** |
@@ -167,7 +167,7 @@ discard block |
||
| 167 | 167 | 'tooltip' => '', |
| 168 | 168 | 'type' => '', |
| 169 | 169 | 'value' => '', |
| 170 | - ]); |
|
| 170 | + ] ); |
|
| 171 | 171 | } |
| 172 | 172 | |
| 173 | 173 | /** |
@@ -175,7 +175,7 @@ discard block |
||
| 175 | 175 | */ |
| 176 | 176 | protected function normalizeContainer( array $field ) |
| 177 | 177 | { |
| 178 | - if( !array_key_exists( 'html', $field ) && !array_key_exists( 'items', $field ))return; |
|
| 178 | + if( !array_key_exists( 'html', $field ) && !array_key_exists( 'items', $field ) )return; |
|
| 179 | 179 | $field['items'] = $this->generateFields( $field['items'] ); |
| 180 | 180 | return $field; |
| 181 | 181 | } |
@@ -185,7 +185,7 @@ discard block |
||
| 185 | 185 | */ |
| 186 | 186 | protected function normalizeField( array $field, array $defaults ) |
| 187 | 187 | { |
| 188 | - if( !$this->validate( $field ))return; |
|
| 188 | + if( !$this->validate( $field ) )return; |
|
| 189 | 189 | return array_filter( shortcode_atts( $defaults, $field ), function( $value ) { |
| 190 | 190 | return $value !== ''; |
| 191 | 191 | }); |
@@ -205,9 +205,9 @@ discard block |
||
| 205 | 205 | 'tooltip' => '', |
| 206 | 206 | 'type' => '', |
| 207 | 207 | 'value' => '', |
| 208 | - ]); |
|
| 209 | - if( !is_array( $listbox ))return; |
|
| 210 | - if( !array_key_exists( '', $listbox['options'] )) { |
|
| 208 | + ] ); |
|
| 209 | + if( !is_array( $listbox ) )return; |
|
| 210 | + if( !array_key_exists( '', $listbox['options'] ) ) { |
|
| 211 | 211 | $listbox['options'] = ['' => $listbox['placeholder']] + $listbox['options']; |
| 212 | 212 | } |
| 213 | 213 | foreach( $listbox['options'] as $value => $text ) { |
@@ -224,7 +224,7 @@ discard block |
||
| 224 | 224 | */ |
| 225 | 225 | protected function normalizePost( array $field ) |
| 226 | 226 | { |
| 227 | - if( !is_array( $field['query_args'] )) { |
|
| 227 | + if( !is_array( $field['query_args'] ) ) { |
|
| 228 | 228 | $field['query_args'] = []; |
| 229 | 229 | } |
| 230 | 230 | $posts = get_posts( wp_parse_args( $field['query_args'], [ |
@@ -232,8 +232,8 @@ discard block |
||
| 232 | 232 | 'orderby' => 'title', |
| 233 | 233 | 'post_type' => 'post', |
| 234 | 234 | 'posts_per_page' => 30, |
| 235 | - ])); |
|
| 236 | - if( !empty( $posts )) { |
|
| 235 | + ] ) ); |
|
| 236 | + if( !empty($posts) ) { |
|
| 237 | 237 | $options = []; |
| 238 | 238 | foreach( $posts as $post ) { |
| 239 | 239 | $options[$post->ID] = esc_html( $post->post_title ); |
@@ -263,7 +263,7 @@ discard block |
||
| 263 | 263 | 'tooltip' => '', |
| 264 | 264 | 'type' => '', |
| 265 | 265 | 'value' => '', |
| 266 | - ]); |
|
| 266 | + ] ); |
|
| 267 | 267 | } |
| 268 | 268 | |
| 269 | 269 | /** |
@@ -271,7 +271,7 @@ discard block |
||
| 271 | 271 | */ |
| 272 | 272 | protected function validate( array $field ) |
| 273 | 273 | { |
| 274 | - $args = shortcode_atts([ |
|
| 274 | + $args = shortcode_atts( [ |
|
| 275 | 275 | 'label' => '', |
| 276 | 276 | 'name' => false, |
| 277 | 277 | 'required' => false, |
@@ -287,13 +287,13 @@ discard block |
||
| 287 | 287 | */ |
| 288 | 288 | protected function validateErrors( array $args ) |
| 289 | 289 | { |
| 290 | - if( !isset( $args['required']['error'] )) { |
|
| 290 | + if( !isset($args['required']['error']) ) { |
|
| 291 | 291 | return true; |
| 292 | 292 | } |
| 293 | - $this->errors[$args['name']] = $this->normalizeContainer([ |
|
| 293 | + $this->errors[$args['name']] = $this->normalizeContainer( [ |
|
| 294 | 294 | 'html' => $args['required']['error'], |
| 295 | 295 | 'type' => 'container', |
| 296 | - ]); |
|
| 296 | + ] ); |
|
| 297 | 297 | return false; |
| 298 | 298 | } |
| 299 | 299 | |
@@ -306,10 +306,10 @@ discard block |
||
| 306 | 306 | return true; |
| 307 | 307 | } |
| 308 | 308 | $alert = esc_html__( 'Some of the shortcode options are required.', 'site-reviews' ); |
| 309 | - if( isset( $args['required']['alert'] )) { |
|
| 309 | + if( isset($args['required']['alert']) ) { |
|
| 310 | 310 | $alert = $args['required']['alert']; |
| 311 | 311 | } |
| 312 | - else if( !empty( $args['label'] )) { |
|
| 312 | + else if( !empty($args['label']) ) { |
|
| 313 | 313 | $alert = sprintf( |
| 314 | 314 | esc_html_x( 'The "%s" option is required.', 'the option label', 'site-reviews' ), |
| 315 | 315 | str_replace( ':', '', $args['label'] ) |
@@ -30,7 +30,7 @@ discard block |
||
| 30 | 30 | { |
| 31 | 31 | $translations = $this->translations(); |
| 32 | 32 | $entries = $this->filter( $translations, $this->entries() )->results(); |
| 33 | - array_walk( $translations, function( &$entry ) use( $entries ) { |
|
| 33 | + array_walk( $translations, function( &$entry ) use($entries) { |
|
| 34 | 34 | $entry['desc'] = array_key_exists( $entry['id'], $entries ) |
| 35 | 35 | ? $this->getEntryString( $entries[$entry['id']], 'msgctxt' ) |
| 36 | 36 | : ''; |
@@ -43,7 +43,7 @@ discard block |
||
| 43 | 43 | */ |
| 44 | 44 | public function entries() |
| 45 | 45 | { |
| 46 | - if( !isset( $this->entries )) { |
|
| 46 | + if( !isset($this->entries) ) { |
|
| 47 | 47 | $potFile = glsr()->path( glsr()->languages.'/'.Application::ID.'.pot' ); |
| 48 | 48 | $entries = $this->extractEntriesFromPotFile( $potFile ); |
| 49 | 49 | $entries = apply_filters( 'site-reviews/translation/entries', $entries ); |
@@ -88,13 +88,13 @@ discard block |
||
| 88 | 88 | */ |
| 89 | 89 | public function filter( $filterWith = null, $entries = null, $intersect = true ) |
| 90 | 90 | { |
| 91 | - if( !is_array( $entries )) { |
|
| 91 | + if( !is_array( $entries ) ) { |
|
| 92 | 92 | $entries = $this->results; |
| 93 | 93 | } |
| 94 | - if( !is_array( $filterWith )) { |
|
| 94 | + if( !is_array( $filterWith ) ) { |
|
| 95 | 95 | $filterWith = $this->translations(); |
| 96 | 96 | } |
| 97 | - $keys = array_flip( glsr_array_column( $filterWith, 'id' )); |
|
| 97 | + $keys = array_flip( glsr_array_column( $filterWith, 'id' ) ); |
|
| 98 | 98 | $this->results = $intersect |
| 99 | 99 | ? array_intersect_key( $entries, $keys ) |
| 100 | 100 | : array_diff_key( $entries, $keys ); |
@@ -108,17 +108,17 @@ discard block |
||
| 108 | 108 | public function render( $template, array $entry ) |
| 109 | 109 | { |
| 110 | 110 | $data = array_combine( |
| 111 | - array_map( function( $key ) { return 'data.'.$key; }, array_keys( $entry )), |
|
| 111 | + array_map( function( $key ) { return 'data.'.$key; }, array_keys( $entry ) ), |
|
| 112 | 112 | $entry |
| 113 | 113 | ); |
| 114 | 114 | $data['data.class'] = $data['data.error'] = ''; |
| 115 | - if( array_search( $entry['s1'], glsr_array_column( $this->entries(), 'msgid' )) === false ) { |
|
| 115 | + if( array_search( $entry['s1'], glsr_array_column( $this->entries(), 'msgid' ) ) === false ) { |
|
| 116 | 116 | $data['data.class'] = 'is-invalid'; |
| 117 | 117 | $data['data.error'] = __( 'This custom translation is no longer valid as the original text has been changed or removed.', 'site-reviews' ); |
| 118 | 118 | } |
| 119 | 119 | return glsr( Template::class )->build( 'partials/translations/'.$template, [ |
| 120 | 120 | 'context' => $data, |
| 121 | - ]); |
|
| 121 | + ] ); |
|
| 122 | 122 | } |
| 123 | 123 | |
| 124 | 124 | /** |
@@ -150,13 +150,13 @@ discard block |
||
| 150 | 150 | 'p1' => $this->getEntryString( $entry, 'msgid_plural' ), |
| 151 | 151 | 's1' => $this->getEntryString( $entry, 'msgid' ), |
| 152 | 152 | ]; |
| 153 | - $text = !empty( $data['p1'] ) |
|
| 153 | + $text = !empty($data['p1']) |
|
| 154 | 154 | ? sprintf( '%s | %s', $data['s1'], $data['p1'] ) |
| 155 | 155 | : $data['s1']; |
| 156 | 156 | $rendered .= $this->render( 'result', [ |
| 157 | - 'entry' => json_encode( $data, JSON_HEX_APOS|JSON_HEX_QUOT|JSON_HEX_TAG|JSON_UNESCAPED_SLASHES|JSON_UNESCAPED_UNICODE ), |
|
| 157 | + 'entry' => json_encode( $data, JSON_HEX_APOS | JSON_HEX_QUOT | JSON_HEX_TAG | JSON_UNESCAPED_SLASHES | JSON_UNESCAPED_UNICODE ), |
|
| 158 | 158 | 'text' => wp_strip_all_tags( $text ), |
| 159 | - ]); |
|
| 159 | + ] ); |
|
| 160 | 160 | } |
| 161 | 161 | if( $resetAfterRender ) { |
| 162 | 162 | $this->reset(); |
@@ -189,12 +189,12 @@ discard block |
||
| 189 | 189 | public function search( $needle = '' ) |
| 190 | 190 | { |
| 191 | 191 | $this->reset(); |
| 192 | - $needle = trim( strtolower( $needle )); |
|
| 192 | + $needle = trim( strtolower( $needle ) ); |
|
| 193 | 193 | foreach( $this->entries() as $key => $entry ) { |
| 194 | - $single = strtolower( $this->getEntryString( $entry, 'msgid' )); |
|
| 195 | - $plural = strtolower( $this->getEntryString( $entry, 'msgid_plural' )); |
|
| 194 | + $single = strtolower( $this->getEntryString( $entry, 'msgid' ) ); |
|
| 195 | + $plural = strtolower( $this->getEntryString( $entry, 'msgid_plural' ) ); |
|
| 196 | 196 | if( strlen( $needle ) < static::SEARCH_THRESHOLD ) { |
| 197 | - if( in_array( $needle, [$single, $plural] )) { |
|
| 197 | + if( in_array( $needle, [$single, $plural] ) ) { |
|
| 198 | 198 | $this->results[$key] = $entry; |
| 199 | 199 | } |
| 200 | 200 | } |
@@ -212,10 +212,10 @@ discard block |
||
| 212 | 212 | public function translations() |
| 213 | 213 | { |
| 214 | 214 | static $translations; |
| 215 | - if( empty( $translations )) { |
|
| 215 | + if( empty($translations) ) { |
|
| 216 | 216 | $settings = glsr( OptionManager::class )->get( 'settings' ); |
| 217 | - $translations = isset( $settings['strings'] ) |
|
| 218 | - ? $this->normalizeSettings( (array) $settings['strings'] ) |
|
| 217 | + $translations = isset($settings['strings']) |
|
| 218 | + ? $this->normalizeSettings( (array)$settings['strings'] ) |
|
| 219 | 219 | : []; |
| 220 | 220 | } |
| 221 | 221 | return $translations; |
@@ -227,8 +227,8 @@ discard block |
||
| 227 | 227 | */ |
| 228 | 228 | protected function getEntryString( array $entry, $key ) |
| 229 | 229 | { |
| 230 | - return isset( $entry[$key] ) |
|
| 231 | - ? implode( '', (array) $entry[$key] ) |
|
| 230 | + return isset($entry[$key]) |
|
| 231 | + ? implode( '', (array)$entry[$key] ) |
|
| 232 | 232 | : ''; |
| 233 | 233 | } |
| 234 | 234 | |
@@ -240,7 +240,7 @@ discard block |
||
| 240 | 240 | $keys = [ |
| 241 | 241 | 'msgctxt', 'msgid', 'msgid_plural', 'msgstr', 'msgstr[0]', 'msgstr[1]', |
| 242 | 242 | ]; |
| 243 | - array_walk( $entries, function( &$entry ) use( $keys ) { |
|
| 243 | + array_walk( $entries, function( &$entry ) use($keys) { |
|
| 244 | 244 | foreach( $keys as $key ) { |
| 245 | 245 | $entry = $this->normalizeEntryString( $entry, $key ); |
| 246 | 246 | } |
@@ -254,7 +254,7 @@ discard block |
||
| 254 | 254 | */ |
| 255 | 255 | protected function normalizeEntryString( array $entry, $key ) |
| 256 | 256 | { |
| 257 | - if( isset( $entry[$key] )) { |
|
| 257 | + if( isset($entry[$key]) ) { |
|
| 258 | 258 | $entry[$key] = $this->getEntryString( $entry, $key ); |
| 259 | 259 | } |
| 260 | 260 | return $entry; |
@@ -268,11 +268,11 @@ discard block |
||
| 268 | 268 | $defaultString = array_fill_keys( ['id', 's1', 's2', 'p1', 'p2'], '' ); |
| 269 | 269 | $strings = array_filter( $strings, 'is_array' ); |
| 270 | 270 | foreach( $strings as &$string ) { |
| 271 | - $string['type'] = isset( $string['p1'] ) ? 'plural' : 'single'; |
|
| 271 | + $string['type'] = isset($string['p1']) ? 'plural' : 'single'; |
|
| 272 | 272 | $string = wp_parse_args( $string, $defaultString ); |
| 273 | 273 | } |
| 274 | 274 | return array_filter( $strings, function( $string ) { |
| 275 | - return !empty( $string['id'] ); |
|
| 275 | + return !empty($string['id']); |
|
| 276 | 276 | }); |
| 277 | 277 | } |
| 278 | 278 | } |
@@ -19,8 +19,8 @@ discard block |
||
| 19 | 19 | 'glsr_get_review', 'glsr_get_reviews', |
| 20 | 20 | 'glsr_log', |
| 21 | 21 | ); |
| 22 | - if( !in_array( $hook, $hooks ) || !function_exists( $hook ))return; |
|
| 23 | - add_filter( $hook, function() use( $hook, $args ) { |
|
| 22 | + if( !in_array( $hook, $hooks ) || !function_exists( $hook ) )return; |
|
| 23 | + add_filter( $hook, function() use($hook, $args) { |
|
| 24 | 24 | array_shift( $args ); // remove the fallback value |
| 25 | 25 | return call_user_func_array( $hook, $args ); |
| 26 | 26 | }); |
@@ -31,7 +31,7 @@ discard block |
||
| 31 | 31 | */ |
| 32 | 32 | function glsr( $alias = null ) { |
| 33 | 33 | $app = \GeminiLabs\SiteReviews\Application::load(); |
| 34 | - return !empty( $alias ) |
|
| 34 | + return !empty($alias) |
|
| 35 | 35 | ? $app->make( $alias ) |
| 36 | 36 | : $app; |
| 37 | 37 | } |
@@ -45,7 +45,7 @@ discard block |
||
| 45 | 45 | $result = array(); |
| 46 | 46 | foreach( $array as $subarray ) { |
| 47 | 47 | $subarray = (array)$subarray; |
| 48 | - if( !isset( $subarray[$column] ))continue; |
|
| 48 | + if( !isset($subarray[$column]) )continue; |
|
| 49 | 49 | $result[] = $subarray[$column]; |
| 50 | 50 | } |
| 51 | 51 | return $result; |
@@ -56,19 +56,19 @@ discard block |
||
| 56 | 56 | */ |
| 57 | 57 | function glsr_calculate_ratings() { |
| 58 | 58 | glsr( 'Controllers\AdminController' )->routerCountReviews( false ); |
| 59 | - glsr_log()->info( __( 'Recalculated rating counts.', 'site-reviews' )); |
|
| 59 | + glsr_log()->info( __( 'Recalculated rating counts.', 'site-reviews' ) ); |
|
| 60 | 60 | } |
| 61 | 61 | |
| 62 | 62 | /** |
| 63 | 63 | * @return null|\GeminiLabs\SiteReviews\Review |
| 64 | 64 | */ |
| 65 | 65 | function glsr_create_review( $reviewValues = array() ) { |
| 66 | - if( !is_array( $reviewValues )) { |
|
| 66 | + if( !is_array( $reviewValues ) ) { |
|
| 67 | 67 | $reviewValues = array(); |
| 68 | 68 | } |
| 69 | 69 | $review = new \GeminiLabs\SiteReviews\Commands\CreateReview( $reviewValues ); |
| 70 | 70 | $result = glsr( 'Database\ReviewManager' )->create( $review ); |
| 71 | - return !empty( $result ) |
|
| 71 | + return !empty($result) |
|
| 72 | 72 | ? $result |
| 73 | 73 | : null; |
| 74 | 74 | } |
@@ -77,10 +77,10 @@ discard block |
||
| 77 | 77 | * @return \WP_Screen|object |
| 78 | 78 | */ |
| 79 | 79 | function glsr_current_screen() { |
| 80 | - if( function_exists( 'get_current_screen' )) { |
|
| 80 | + if( function_exists( 'get_current_screen' ) ) { |
|
| 81 | 81 | $screen = get_current_screen(); |
| 82 | 82 | } |
| 83 | - return empty( $screen ) |
|
| 83 | + return empty($screen) |
|
| 84 | 84 | ? (object)array_fill_keys( ['base', 'id', 'post_type'], null ) |
| 85 | 85 | : $screen; |
| 86 | 86 | } |
@@ -126,7 +126,7 @@ discard block |
||
| 126 | 126 | * @return \GeminiLabs\SiteReviews\Review|void |
| 127 | 127 | */ |
| 128 | 128 | function glsr_get_review( $post_id ) { |
| 129 | - if( !is_numeric( $post_id ))return; |
|
| 129 | + if( !is_numeric( $post_id ) )return; |
|
| 130 | 130 | $post = get_post( $post_id ); |
| 131 | 131 | if( $post instanceof WP_Post ) { |
| 132 | 132 | return glsr( 'Database\ReviewManager' )->single( $post ); |
@@ -138,7 +138,7 @@ discard block |
||
| 138 | 138 | * @todo document change of $reviews->reviews to $reviews->results |
| 139 | 139 | */ |
| 140 | 140 | function glsr_get_reviews( $args = array() ) { |
| 141 | - if( !is_array( $args )) { |
|
| 141 | + if( !is_array( $args ) ) { |
|
| 142 | 142 | $args = []; |
| 143 | 143 | } |
| 144 | 144 | return glsr( 'Database\ReviewManager' )->get( $args ); |
@@ -149,11 +149,11 @@ discard block |
||
| 149 | 149 | */ |
| 150 | 150 | function glsr_log() { |
| 151 | 151 | $args = func_get_args(); |
| 152 | - $context = isset( $args[1] ) |
|
| 152 | + $context = isset($args[1]) |
|
| 153 | 153 | ? $args[1] |
| 154 | 154 | : []; |
| 155 | 155 | $console = glsr( 'Modules\Console' ); |
| 156 | - return !empty( $args ) |
|
| 156 | + return !empty($args) |
|
| 157 | 157 | ? $console->log( 'debug', $args[0], $context ) |
| 158 | 158 | : $console; |
| 159 | 159 | } |
@@ -19,7 +19,9 @@ discard block |
||
| 19 | 19 | 'glsr_get_review', 'glsr_get_reviews', |
| 20 | 20 | 'glsr_log', |
| 21 | 21 | ); |
| 22 | - if( !in_array( $hook, $hooks ) || !function_exists( $hook ))return; |
|
| 22 | + if( !in_array( $hook, $hooks ) || !function_exists( $hook )) { |
|
| 23 | + return; |
|
| 24 | + } |
|
| 23 | 25 | add_filter( $hook, function() use( $hook, $args ) { |
| 24 | 26 | array_shift( $args ); // remove the fallback value |
| 25 | 27 | return call_user_func_array( $hook, $args ); |
@@ -29,7 +31,8 @@ discard block |
||
| 29 | 31 | /** |
| 30 | 32 | * @return mixed |
| 31 | 33 | */ |
| 32 | -function glsr( $alias = null ) { |
|
| 34 | +function glsr( $alias = null ) |
|
| 35 | +{ |
|
| 33 | 36 | $app = \GeminiLabs\SiteReviews\Application::load(); |
| 34 | 37 | return !empty( $alias ) |
| 35 | 38 | ? $app->make( $alias ) |
@@ -41,11 +44,14 @@ discard block |
||
| 41 | 44 | * @param $column string |
| 42 | 45 | * @return array |
| 43 | 46 | */ |
| 44 | -function glsr_array_column( array $array, $column ) { |
|
| 47 | +function glsr_array_column( array $array, $column ) |
|
| 48 | +{ |
|
| 45 | 49 | $result = array(); |
| 46 | 50 | foreach( $array as $subarray ) { |
| 47 | 51 | $subarray = (array)$subarray; |
| 48 | - if( !isset( $subarray[$column] ))continue; |
|
| 52 | + if( !isset( $subarray[$column] )) { |
|
| 53 | + continue; |
|
| 54 | + } |
|
| 49 | 55 | $result[] = $subarray[$column]; |
| 50 | 56 | } |
| 51 | 57 | return $result; |
@@ -54,7 +60,8 @@ discard block |
||
| 54 | 60 | /** |
| 55 | 61 | * @return void |
| 56 | 62 | */ |
| 57 | -function glsr_calculate_ratings() { |
|
| 63 | +function glsr_calculate_ratings() |
|
| 64 | +{ |
|
| 58 | 65 | glsr( 'Controllers\AdminController' )->routerCountReviews( false ); |
| 59 | 66 | glsr_log()->info( __( 'Recalculated rating counts.', 'site-reviews' )); |
| 60 | 67 | } |
@@ -62,7 +69,8 @@ discard block |
||
| 62 | 69 | /** |
| 63 | 70 | * @return null|\GeminiLabs\SiteReviews\Review |
| 64 | 71 | */ |
| 65 | -function glsr_create_review( $reviewValues = array() ) { |
|
| 72 | +function glsr_create_review( $reviewValues = array() ) |
|
| 73 | +{ |
|
| 66 | 74 | if( !is_array( $reviewValues )) { |
| 67 | 75 | $reviewValues = array(); |
| 68 | 76 | } |
@@ -76,7 +84,8 @@ discard block |
||
| 76 | 84 | /** |
| 77 | 85 | * @return \WP_Screen|object |
| 78 | 86 | */ |
| 79 | -function glsr_current_screen() { |
|
| 87 | +function glsr_current_screen() |
|
| 88 | +{ |
|
| 80 | 89 | if( function_exists( 'get_current_screen' )) { |
| 81 | 90 | $screen = get_current_screen(); |
| 82 | 91 | } |
@@ -89,7 +98,8 @@ discard block |
||
| 89 | 98 | * @param mixed ...$vars |
| 90 | 99 | * @return void |
| 91 | 100 | */ |
| 92 | -function glsr_debug( ...$vars ) { |
|
| 101 | +function glsr_debug( ...$vars ) |
|
| 102 | +{ |
|
| 93 | 103 | if( count( $vars ) == 1 ) { |
| 94 | 104 | $value = htmlspecialchars( print_r( $vars[0], true ), ENT_QUOTES, 'UTF-8' ); |
| 95 | 105 | printf( '<div class="glsr-debug"><pre>%s</pre></div>', $value ); |
@@ -108,7 +118,8 @@ discard block |
||
| 108 | 118 | * @param mixed $fallback |
| 109 | 119 | * @return string|array |
| 110 | 120 | */ |
| 111 | -function glsr_get_option( $path = '', $fallback = '' ) { |
|
| 121 | +function glsr_get_option( $path = '', $fallback = '' ) |
|
| 122 | +{ |
|
| 112 | 123 | return is_string( $path ) |
| 113 | 124 | ? glsr( 'Database\OptionManager' )->get( 'settings.'.$path, $fallback ) |
| 114 | 125 | : $fallback; |
@@ -117,7 +128,8 @@ discard block |
||
| 117 | 128 | /** |
| 118 | 129 | * @return array |
| 119 | 130 | */ |
| 120 | -function glsr_get_options() { |
|
| 131 | +function glsr_get_options() |
|
| 132 | +{ |
|
| 121 | 133 | return glsr( 'Database\OptionManager' )->get( 'settings' ); |
| 122 | 134 | } |
| 123 | 135 | |
@@ -125,8 +137,11 @@ discard block |
||
| 125 | 137 | * @param int $post_id |
| 126 | 138 | * @return \GeminiLabs\SiteReviews\Review|void |
| 127 | 139 | */ |
| 128 | -function glsr_get_review( $post_id ) { |
|
| 129 | - if( !is_numeric( $post_id ))return; |
|
| 140 | +function glsr_get_review( $post_id ) |
|
| 141 | +{ |
|
| 142 | + if( !is_numeric( $post_id )) { |
|
| 143 | + return; |
|
| 144 | + } |
|
| 130 | 145 | $post = get_post( $post_id ); |
| 131 | 146 | if( $post instanceof WP_Post ) { |
| 132 | 147 | return glsr( 'Database\ReviewManager' )->single( $post ); |
@@ -137,7 +152,8 @@ discard block |
||
| 137 | 152 | * @return array |
| 138 | 153 | * @todo document change of $reviews->reviews to $reviews->results |
| 139 | 154 | */ |
| 140 | -function glsr_get_reviews( $args = array() ) { |
|
| 155 | +function glsr_get_reviews( $args = array() ) |
|
| 156 | +{ |
|
| 141 | 157 | if( !is_array( $args )) { |
| 142 | 158 | $args = []; |
| 143 | 159 | } |
@@ -147,7 +163,8 @@ discard block |
||
| 147 | 163 | /** |
| 148 | 164 | * @return \GeminiLabs\SiteReviews\Modules\Console |
| 149 | 165 | */ |
| 150 | -function glsr_log() { |
|
| 166 | +function glsr_log() |
|
| 167 | +{ |
|
| 151 | 168 | $args = func_get_args(); |
| 152 | 169 | $context = isset( $args[1] ) |
| 153 | 170 | ? $args[1] |
@@ -23,11 +23,11 @@ discard block |
||
| 23 | 23 | */ |
| 24 | 24 | public function deleteAllSessions( $sessionCookiePrefix ) |
| 25 | 25 | { |
| 26 | - return $this->db->query(" |
|
| 26 | + return $this->db->query( " |
|
| 27 | 27 | DELETE |
| 28 | 28 | FROM {$this->db->options} |
| 29 | 29 | WHERE option_name LIKE '{$sessionCookiePrefix}_%' |
| 30 | - "); |
|
| 30 | + " ); |
|
| 31 | 31 | } |
| 32 | 32 | |
| 33 | 33 | /** |
@@ -36,11 +36,11 @@ discard block |
||
| 36 | 36 | */ |
| 37 | 37 | public function deleteExpiredSessions( $expiredSessions ) |
| 38 | 38 | { |
| 39 | - return $this->db->query(" |
|
| 39 | + return $this->db->query( " |
|
| 40 | 40 | DELETE |
| 41 | 41 | FROM {$this->db->options} |
| 42 | 42 | WHERE option_name IN ('{$expiredSessions}') |
| 43 | - "); |
|
| 43 | + " ); |
|
| 44 | 44 | } |
| 45 | 45 | |
| 46 | 46 | /** |
@@ -49,14 +49,14 @@ discard block |
||
| 49 | 49 | */ |
| 50 | 50 | public function getReviewCountsFor( $metaKey ) |
| 51 | 51 | { |
| 52 | - return (array) $this->db->get_results(" |
|
| 52 | + return (array)$this->db->get_results( " |
|
| 53 | 53 | SELECT m.meta_value AS name, COUNT(*) num_posts |
| 54 | 54 | FROM {$this->db->posts} AS p |
| 55 | 55 | INNER JOIN {$this->db->postmeta} AS m ON p.ID = m.post_id |
| 56 | 56 | WHERE p.post_type = '{$this->postType}' |
| 57 | 57 | AND m.meta_key = '{$metaKey}' |
| 58 | 58 | GROUP BY name |
| 59 | - "); |
|
| 59 | + " ); |
|
| 60 | 60 | } |
| 61 | 61 | |
| 62 | 62 | /** |
@@ -66,13 +66,13 @@ discard block |
||
| 66 | 66 | */ |
| 67 | 67 | public function getExpiredSessions( $sessionCookiePrefix, $limit ) |
| 68 | 68 | { |
| 69 | - return $this->db->get_results(" |
|
| 69 | + return $this->db->get_results( " |
|
| 70 | 70 | SELECT option_name AS name, option_value AS expiration |
| 71 | 71 | FROM {$this->db->options} |
| 72 | 72 | WHERE option_name LIKE '{$sessionCookiePrefix}_expires_%' |
| 73 | 73 | ORDER BY option_value ASC |
| 74 | 74 | LIMIT 0, {$limit} |
| 75 | - "); |
|
| 75 | + " ); |
|
| 76 | 76 | } |
| 77 | 77 | |
| 78 | 78 | /** |
@@ -82,7 +82,7 @@ discard block |
||
| 82 | 82 | */ |
| 83 | 83 | public function getReviewCounts( $lastPostId = 0, $limit = 500 ) |
| 84 | 84 | { |
| 85 | - return $this->db->get_results(" |
|
| 85 | + return $this->db->get_results( " |
|
| 86 | 86 | SELECT p.ID, m1.meta_value AS rating, m2.meta_value AS type |
| 87 | 87 | FROM {$this->db->posts} AS p |
| 88 | 88 | INNER JOIN {$this->db->postmeta} AS m1 ON p.ID = m1.post_id |
@@ -103,14 +103,14 @@ discard block |
||
| 103 | 103 | */ |
| 104 | 104 | public function getReviewPostId( $metaReviewId ) |
| 105 | 105 | { |
| 106 | - $postId = $this->db->get_var(" |
|
| 106 | + $postId = $this->db->get_var( " |
|
| 107 | 107 | SELECT p.ID |
| 108 | 108 | FROM {$this->db->posts} AS p |
| 109 | 109 | INNER JOIN {$this->db->postmeta} AS m ON p.ID = m.post_id |
| 110 | 110 | WHERE p.post_type = '{$this->postType}' |
| 111 | 111 | AND m.meta_key = 'review_id' |
| 112 | 112 | AND m.meta_value = '{$metaReviewId}' |
| 113 | - "); |
|
| 113 | + " ); |
|
| 114 | 114 | return intval( $postId ); |
| 115 | 115 | } |
| 116 | 116 | |
@@ -120,7 +120,7 @@ discard block |
||
| 120 | 120 | */ |
| 121 | 121 | public function getReviewIdsByType( $reviewType ) |
| 122 | 122 | { |
| 123 | - $query = $this->db->get_col(" |
|
| 123 | + $query = $this->db->get_col( " |
|
| 124 | 124 | SELECT m1.meta_value AS review_id |
| 125 | 125 | FROM {$this->db->posts} AS p |
| 126 | 126 | INNER JOIN {$this->db->postmeta} AS m1 ON p.ID = m1.post_id |
@@ -129,8 +129,8 @@ discard block |
||
| 129 | 129 | AND m1.meta_key = 'review_id' |
| 130 | 130 | AND m2.meta_key = 'review_type' |
| 131 | 131 | AND m2.meta_value = '{$reviewType}' |
| 132 | - "); |
|
| 133 | - return array_keys( array_flip( $query )); |
|
| 132 | + " ); |
|
| 133 | + return array_keys( array_flip( $query ) ); |
|
| 134 | 134 | } |
| 135 | 135 | |
| 136 | 136 | /** |
@@ -141,7 +141,7 @@ discard block |
||
| 141 | 141 | */ |
| 142 | 142 | public function getReviewPostCounts( $postId, $lastPostId = 0, $limit = 500 ) |
| 143 | 143 | { |
| 144 | - return $this->db->get_results(" |
|
| 144 | + return $this->db->get_results( " |
|
| 145 | 145 | SELECT p.ID, m1.meta_value AS rating, m2.meta_value AS type |
| 146 | 146 | FROM {$this->db->posts} AS p |
| 147 | 147 | INNER JOIN {$this->db->postmeta} AS m1 ON p.ID = m1.post_id |
@@ -167,9 +167,9 @@ discard block |
||
| 167 | 167 | public function getReviewRatingsFromIds( array $postIds, $greaterThanId = 0, $limit = 100 ) |
| 168 | 168 | { |
| 169 | 169 | sort( $postIds ); |
| 170 | - $postIds = array_slice( $postIds, intval( array_search( $greaterThanId, $postIds )), $limit ); |
|
| 170 | + $postIds = array_slice( $postIds, intval( array_search( $greaterThanId, $postIds ) ), $limit ); |
|
| 171 | 171 | $postIds = implode( ',', $postIds ); |
| 172 | - return $this->db->get_results(" |
|
| 172 | + return $this->db->get_results( " |
|
| 173 | 173 | SELECT p.ID, m.meta_value AS rating |
| 174 | 174 | FROM {$this->db->posts} AS p |
| 175 | 175 | INNER JOIN {$this->db->postmeta} AS m ON p.ID = m.post_id |
@@ -193,7 +193,7 @@ discard block |
||
| 193 | 193 | $queryBuilder = glsr( QueryBuilder::class ); |
| 194 | 194 | $key = $queryBuilder->buildSqlOr( $key, "m.meta_key = '%s'" ); |
| 195 | 195 | $status = $queryBuilder->buildSqlOr( $status, "p.post_status = '%s'" ); |
| 196 | - return $this->db->get_col(" |
|
| 196 | + return $this->db->get_col( " |
|
| 197 | 197 | SELECT DISTINCT m.meta_value |
| 198 | 198 | FROM {$this->db->postmeta} m |
| 199 | 199 | LEFT JOIN {$this->db->posts} p ON p.ID = m.post_id |
@@ -201,7 +201,7 @@ discard block |
||
| 201 | 201 | AND ({$key}) |
| 202 | 202 | AND ({$status}) |
| 203 | 203 | ORDER BY m.meta_value |
| 204 | - "); |
|
| 204 | + " ); |
|
| 205 | 205 | } |
| 206 | 206 | |
| 207 | 207 | /** |
@@ -212,7 +212,7 @@ discard block |
||
| 212 | 212 | */ |
| 213 | 213 | public function getReviewTermCounts( $termTaxonomyId, $lastPostId = 0, $limit = 500 ) |
| 214 | 214 | { |
| 215 | - return $this->db->get_results(" |
|
| 215 | + return $this->db->get_results( " |
|
| 216 | 216 | SELECT p.ID, m1.meta_value AS rating, m2.meta_value AS type |
| 217 | 217 | FROM {$this->db->posts} AS p |
| 218 | 218 | INNER JOIN {$this->db->postmeta} AS m1 ON p.ID = m1.post_id |
@@ -35,12 +35,12 @@ discard block |
||
| 35 | 35 | 'post_type' => Application::POST_TYPE, |
| 36 | 36 | ]; |
| 37 | 37 | $postId = wp_insert_post( $postValues, true ); |
| 38 | - if( is_wp_error( $postId )) { |
|
| 38 | + if( is_wp_error( $postId ) ) { |
|
| 39 | 39 | glsr_log()->error( $postId->get_error_message() )->debug( $postValues ); |
| 40 | 40 | return false; |
| 41 | 41 | } |
| 42 | 42 | $this->setTerms( $postId, $command->category ); |
| 43 | - $review = $this->single( get_post( $postId )); |
|
| 43 | + $review = $this->single( get_post( $postId ) ); |
|
| 44 | 44 | do_action( 'site-reviews/review/created', $review, $command ); |
| 45 | 45 | return $review; |
| 46 | 46 | } |
@@ -51,7 +51,7 @@ discard block |
||
| 51 | 51 | */ |
| 52 | 52 | public function delete( $metaReviewId ) |
| 53 | 53 | { |
| 54 | - if( $postId = $this->getPostId( $metaReviewId )) { |
|
| 54 | + if( $postId = $this->getPostId( $metaReviewId ) ) { |
|
| 55 | 55 | wp_delete_post( $postId, true ); |
| 56 | 56 | } |
| 57 | 57 | } |
@@ -73,7 +73,7 @@ discard block |
||
| 73 | 73 | $paged = glsr( QueryBuilder::class )->getPaged( |
| 74 | 74 | wp_validate_boolean( $args['pagination'] ) |
| 75 | 75 | ); |
| 76 | - $reviews = new WP_Query([ |
|
| 76 | + $reviews = new WP_Query( [ |
|
| 77 | 77 | 'meta_key' => 'pinned', |
| 78 | 78 | 'meta_query' => $metaQuery, |
| 79 | 79 | 'offset' => $args['offset'], |
@@ -86,7 +86,7 @@ discard block |
||
| 86 | 86 | 'post_type' => Application::POST_TYPE, |
| 87 | 87 | 'posts_per_page' => $args['count'], |
| 88 | 88 | 'tax_query' => $taxQuery, |
| 89 | - ]); |
|
| 89 | + ] ); |
|
| 90 | 90 | return (object)[ |
| 91 | 91 | 'results' => array_map( [$this, 'single'], $reviews->posts ), |
| 92 | 92 | 'max_num_pages' => $reviews->max_num_pages, |
@@ -120,11 +120,11 @@ discard block |
||
| 120 | 120 | $terms = []; |
| 121 | 121 | $termIds = glsr( Helper::class )->convertStringToArray( $commaSeparatedTermIds ); |
| 122 | 122 | foreach( $termIds as $termId ) { |
| 123 | - if( is_numeric( $termId )) { |
|
| 123 | + if( is_numeric( $termId ) ) { |
|
| 124 | 124 | $termId = intval( $termId ); |
| 125 | 125 | } |
| 126 | 126 | $term = term_exists( $termId, Application::TAXONOMY ); |
| 127 | - if( !isset( $term['term_id'] ))continue; |
|
| 127 | + if( !isset($term['term_id']) )continue; |
|
| 128 | 128 | $terms[] = $term; |
| 129 | 129 | } |
| 130 | 130 | return $terms; |
@@ -138,13 +138,13 @@ discard block |
||
| 138 | 138 | { |
| 139 | 139 | if( get_post_field( 'post_type', $postId ) != Application::POST_TYPE )return; |
| 140 | 140 | delete_post_meta( $postId, '_edit_last' ); |
| 141 | - $result = wp_update_post([ |
|
| 141 | + $result = wp_update_post( [ |
|
| 142 | 142 | 'ID' => $postId, |
| 143 | 143 | 'post_content' => get_post_meta( $postId, 'content', true ), |
| 144 | 144 | 'post_date' => get_post_meta( $postId, 'date', true ), |
| 145 | 145 | 'post_title' => get_post_meta( $postId, 'title', true ), |
| 146 | - ]); |
|
| 147 | - if( is_wp_error( $result )) { |
|
| 146 | + ] ); |
|
| 147 | + if( is_wp_error( $result ) ) { |
|
| 148 | 148 | glsr_log()->error( $result->get_error_message() ); |
| 149 | 149 | } |
| 150 | 150 | } |
@@ -166,7 +166,7 @@ discard block |
||
| 166 | 166 | protected function getNewPostStatus( array $review, $isBlacklisted ) |
| 167 | 167 | { |
| 168 | 168 | $requireApprovalOption = glsr( OptionManager::class )->get( 'settings.general.require.approval' ); |
| 169 | - return $review['review_type'] == 'local' && ( $requireApprovalOption == 'yes' || $isBlacklisted ) |
|
| 169 | + return $review['review_type'] == 'local' && ($requireApprovalOption == 'yes' || $isBlacklisted) |
|
| 170 | 170 | ? 'pending' |
| 171 | 171 | : 'publish'; |
| 172 | 172 | } |
@@ -179,9 +179,9 @@ discard block |
||
| 179 | 179 | protected function setTerms( $postId, $termIds ) |
| 180 | 180 | { |
| 181 | 181 | $terms = $this->normalizeTermIds( $termIds ); |
| 182 | - if( empty( $terms ))return; |
|
| 182 | + if( empty($terms) )return; |
|
| 183 | 183 | $result = wp_set_object_terms( $postId, $terms, Application::TAXONOMY ); |
| 184 | - if( is_wp_error( $result )) { |
|
| 184 | + if( is_wp_error( $result ) ) { |
|
| 185 | 185 | glsr_log()->error( $result->get_error_message() ); |
| 186 | 186 | } |
| 187 | 187 | } |
@@ -124,7 +124,9 @@ discard block |
||
| 124 | 124 | $termId = intval( $termId ); |
| 125 | 125 | } |
| 126 | 126 | $term = term_exists( $termId, Application::TAXONOMY ); |
| 127 | - if( !isset( $term['term_id'] ))continue; |
|
| 127 | + if( !isset( $term['term_id'] )) { |
|
| 128 | + continue; |
|
| 129 | + } |
|
| 128 | 130 | $terms[] = $term; |
| 129 | 131 | } |
| 130 | 132 | return $terms; |
@@ -136,7 +138,9 @@ discard block |
||
| 136 | 138 | */ |
| 137 | 139 | public function revert( $postId ) |
| 138 | 140 | { |
| 139 | - if( get_post_field( 'post_type', $postId ) != Application::POST_TYPE )return; |
|
| 141 | + if( get_post_field( 'post_type', $postId ) != Application::POST_TYPE ) { |
|
| 142 | + return; |
|
| 143 | + } |
|
| 140 | 144 | delete_post_meta( $postId, '_edit_last' ); |
| 141 | 145 | $result = wp_update_post([ |
| 142 | 146 | 'ID' => $postId, |
@@ -154,7 +158,9 @@ discard block |
||
| 154 | 158 | */ |
| 155 | 159 | public function single( WP_Post $post ) |
| 156 | 160 | { |
| 157 | - if( $post->post_type != Application::POST_TYPE )return; |
|
| 161 | + if( $post->post_type != Application::POST_TYPE ) { |
|
| 162 | + return; |
|
| 163 | + } |
|
| 158 | 164 | $review = new Review( $post ); |
| 159 | 165 | return apply_filters( 'site-reviews/get/review', $review, $post ); |
| 160 | 166 | } |
@@ -179,7 +185,9 @@ discard block |
||
| 179 | 185 | protected function setTerms( $postId, $termIds ) |
| 180 | 186 | { |
| 181 | 187 | $terms = $this->normalizeTermIds( $termIds ); |
| 182 | - if( empty( $terms ))return; |
|
| 188 | + if( empty( $terms )) { |
|
| 189 | + return; |
|
| 190 | + } |
|
| 183 | 191 | $result = wp_set_object_terms( $postId, $terms, Application::TAXONOMY ); |
| 184 | 192 | if( is_wp_error( $result )) { |
| 185 | 193 | glsr_log()->error( $result->get_error_message() ); |
@@ -26,15 +26,15 @@ discard block |
||
| 26 | 26 | { |
| 27 | 27 | sort( $termTaxonomyIds ); |
| 28 | 28 | sort( $oldTermTaxonomyIds ); |
| 29 | - if( $termTaxonomyIds === $oldTermTaxonomyIds || !$this->isReviewPostId( $postId ))return; |
|
| 30 | - $review = glsr( ReviewManager::class )->single( get_post( $postId )); |
|
| 29 | + if( $termTaxonomyIds === $oldTermTaxonomyIds || !$this->isReviewPostId( $postId ) )return; |
|
| 30 | + $review = glsr( ReviewManager::class )->single( get_post( $postId ) ); |
|
| 31 | 31 | $ignoredIds = array_intersect( $oldTermTaxonomyIds, $termTaxonomyIds ); |
| 32 | 32 | $decreasedIds = array_diff( $oldTermTaxonomyIds, $ignoredIds ); |
| 33 | 33 | $increasedIds = array_diff( $termTaxonomyIds, $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 | } |
@@ -47,8 +47,8 @@ discard block |
||
| 47 | 47 | */ |
| 48 | 48 | public function onAfterChangeStatus( $newStatus, $oldStatus, WP_Post $post ) |
| 49 | 49 | { |
| 50 | - if( $post->post_type != Application::POST_TYPE || in_array( $oldStatus, ['new', $newStatus] ))return; |
|
| 51 | - $review = glsr( ReviewManager::class )->single( get_post( $post->ID )); |
|
| 50 | + if( $post->post_type != Application::POST_TYPE || in_array( $oldStatus, ['new', $newStatus] ) )return; |
|
| 51 | + $review = glsr( ReviewManager::class )->single( get_post( $post->ID ) ); |
|
| 52 | 52 | if( $post->post_status == 'publish' ) { |
| 53 | 53 | glsr( CountsManager::class )->increase( $review ); |
| 54 | 54 | } |
@@ -74,8 +74,8 @@ discard block |
||
| 74 | 74 | */ |
| 75 | 75 | public function onBeforeDelete( $postId ) |
| 76 | 76 | { |
| 77 | - if( !$this->isReviewPostId( $postId ))return; |
|
| 78 | - $review = glsr( ReviewManager::class )->single( get_post( $postId )); |
|
| 77 | + if( !$this->isReviewPostId( $postId ) )return; |
|
| 78 | + $review = glsr( ReviewManager::class )->single( get_post( $postId ) ); |
|
| 79 | 79 | glsr( CountsManager::class )->decrease( $review ); |
| 80 | 80 | } |
| 81 | 81 | |
@@ -92,7 +92,7 @@ discard block |
||
| 92 | 92 | if( !$this->isReviewPostId( $postId ) |
| 93 | 93 | || !in_array( $metaKey, ['assigned_to', 'rating', 'review_type'] ) |
| 94 | 94 | )return; |
| 95 | - $review = glsr( ReviewManager::class )->single( get_post( $postId )); |
|
| 95 | + $review = glsr( ReviewManager::class )->single( get_post( $postId ) ); |
|
| 96 | 96 | if( $review->$metaKey == $metaValue )return; |
| 97 | 97 | $method = glsr( Helper::class )->buildMethodName( $metaKey, 'onBeforeChange' ); |
| 98 | 98 | call_user_func( [$this, $method], $review, $metaValue ); |
@@ -26,7 +26,9 @@ discard block |
||
| 26 | 26 | { |
| 27 | 27 | sort( $termTaxonomyIds ); |
| 28 | 28 | sort( $oldTermTaxonomyIds ); |
| 29 | - if( $termTaxonomyIds === $oldTermTaxonomyIds || !$this->isReviewPostId( $postId ))return; |
|
| 29 | + if( $termTaxonomyIds === $oldTermTaxonomyIds || !$this->isReviewPostId( $postId )) { |
|
| 30 | + return; |
|
| 31 | + } |
|
| 30 | 32 | $review = glsr( ReviewManager::class )->single( get_post( $postId )); |
| 31 | 33 | $ignoredIds = array_intersect( $oldTermTaxonomyIds, $termTaxonomyIds ); |
| 32 | 34 | $decreasedIds = array_diff( $oldTermTaxonomyIds, $ignoredIds ); |
@@ -47,7 +49,9 @@ discard block |
||
| 47 | 49 | */ |
| 48 | 50 | public function onAfterChangeStatus( $newStatus, $oldStatus, WP_Post $post ) |
| 49 | 51 | { |
| 50 | - if( $post->post_type != Application::POST_TYPE || in_array( $oldStatus, ['new', $newStatus] ))return; |
|
| 52 | + if( $post->post_type != Application::POST_TYPE || in_array( $oldStatus, ['new', $newStatus] )) { |
|
| 53 | + return; |
|
| 54 | + } |
|
| 51 | 55 | $review = glsr( ReviewManager::class )->single( get_post( $post->ID )); |
| 52 | 56 | if( $post->post_status == 'publish' ) { |
| 53 | 57 | glsr( CountsManager::class )->increase( $review ); |
@@ -63,7 +67,9 @@ discard block |
||
| 63 | 67 | */ |
| 64 | 68 | public function onAfterCreate( Review $review ) |
| 65 | 69 | { |
| 66 | - if( $review->status !== 'publish' )return; |
|
| 70 | + if( $review->status !== 'publish' ) { |
|
| 71 | + return; |
|
| 72 | + } |
|
| 67 | 73 | glsr( CountsManager::class )->increase( $review ); |
| 68 | 74 | } |
| 69 | 75 | |
@@ -74,7 +80,9 @@ discard block |
||
| 74 | 80 | */ |
| 75 | 81 | public function onBeforeDelete( $postId ) |
| 76 | 82 | { |
| 77 | - if( !$this->isReviewPostId( $postId ))return; |
|
| 83 | + if( !$this->isReviewPostId( $postId )) { |
|
| 84 | + return; |
|
| 85 | + } |
|
| 78 | 86 | $review = glsr( ReviewManager::class )->single( get_post( $postId )); |
| 79 | 87 | glsr( CountsManager::class )->decrease( $review ); |
| 80 | 88 | } |
@@ -91,9 +99,13 @@ discard block |
||
| 91 | 99 | { |
| 92 | 100 | if( !$this->isReviewPostId( $postId ) |
| 93 | 101 | || !in_array( $metaKey, ['assigned_to', 'rating', 'review_type'] ) |
| 94 | - )return; |
|
| 102 | + ) { |
|
| 103 | + return; |
|
| 104 | + } |
|
| 95 | 105 | $review = glsr( ReviewManager::class )->single( get_post( $postId )); |
| 96 | - if( $review->$metaKey == $metaValue )return; |
|
| 106 | + if( $review->$metaKey == $metaValue ) { |
|
| 107 | + return; |
|
| 108 | + } |
|
| 97 | 109 | $method = glsr( Helper::class )->buildMethodName( $metaKey, 'onBeforeChange' ); |
| 98 | 110 | call_user_func( [$this, $method], $review, $metaValue ); |
| 99 | 111 | } |