@@ -17,6 +17,8 @@ |
||
| 17 | 17 | * @return void |
| 18 | 18 | */ |
| 19 | 19 | add_action( 'site-reviews/customize/divi', function( $instance ) { |
| 20 | - if( $instance->tag != 'label' || $instance->args['type'] != 'checkbox' )return; |
|
| 20 | + if( $instance->tag != 'label' || $instance->args['type'] != 'checkbox' ) { |
|
| 21 | + return; |
|
| 22 | + } |
|
| 21 | 23 | $instance->args['text'] = '<i></i>'.$instance->args['text']; |
| 22 | 24 | }); |
@@ -17,7 +17,9 @@ |
||
| 17 | 17 | natsort( $routines ); |
| 18 | 18 | array_walk( $routines, function( $routine ) { |
| 19 | 19 | $parts = explode( '__', $routine ); |
| 20 | - if( version_compare( glsr()->version, end( $parts ), '<' ))return; |
|
| 20 | + if( version_compare( glsr()->version, end( $parts ), '<' )) { |
|
| 21 | + return; |
|
| 22 | + } |
|
| 21 | 23 | call_user_func( [$this, $routine] ); |
| 22 | 24 | }); |
| 23 | 25 | $this->updateVersion(); |
@@ -13,7 +13,9 @@ discard block |
||
| 13 | 13 | */ |
| 14 | 14 | public function saveAssignedToMetabox( $postId ) |
| 15 | 15 | { |
| 16 | - if( !wp_verify_nonce( glsr( Helper::class )->filterInput( '_nonce-assigned-to' ), 'assigned_to' ))return; |
|
| 16 | + if( !wp_verify_nonce( glsr( Helper::class )->filterInput( '_nonce-assigned-to' ), 'assigned_to' )) { |
|
| 17 | + return; |
|
| 18 | + } |
|
| 17 | 19 | $assignedTo = strval( glsr( Helper::class )->filterInput( 'assigned_to' )); |
| 18 | 20 | if( get_post_meta( $postId, 'assigned_to', true ) != $assignedTo ) { |
| 19 | 21 | $review = glsr( ReviewManager::class )->single( get_post( $postId )); |
@@ -28,7 +30,9 @@ discard block |
||
| 28 | 30 | */ |
| 29 | 31 | public function saveResponseMetabox( $postId ) |
| 30 | 32 | { |
| 31 | - if( !wp_verify_nonce( glsr( Helper::class )->filterInput( '_nonce-response' ), 'response' ))return; |
|
| 33 | + if( !wp_verify_nonce( glsr( Helper::class )->filterInput( '_nonce-response' ), 'response' )) { |
|
| 34 | + return; |
|
| 35 | + } |
|
| 32 | 36 | $response = strval( glsr( Helper::class )->filterInput( 'response' )); |
| 33 | 37 | update_post_meta( $postId, 'response', trim( wp_kses( $response, [ |
| 34 | 38 | 'a' => ['href' => [], 'title' => []], |
@@ -73,7 +73,9 @@ discard block |
||
| 73 | 73 | */ |
| 74 | 74 | public function decreasePostCounts( Review $review ) |
| 75 | 75 | { |
| 76 | - if( empty( $counts = $this->getPostCounts( $review->assigned_to )))return; |
|
| 76 | + if( empty( $counts = $this->getPostCounts( $review->assigned_to ))) { |
|
| 77 | + return; |
|
| 78 | + } |
|
| 77 | 79 | $counts = $this->decreaseRating( $counts, $review->review_type, $review->rating ); |
| 78 | 80 | $this->setPostCounts( $review->assigned_to, $counts ); |
| 79 | 81 | } |
@@ -84,7 +86,9 @@ discard block |
||
| 84 | 86 | public function decreaseTermCounts( Review $review ) |
| 85 | 87 | { |
| 86 | 88 | foreach( $review->term_ids as $termId ) { |
| 87 | - if( empty( $counts = $this->getTermCounts( $termId )))continue; |
|
| 89 | + if( empty( $counts = $this->getTermCounts( $termId ))) { |
|
| 90 | + continue; |
|
| 91 | + } |
|
| 88 | 92 | $counts = $this->decreaseRating( $counts, $review->review_type, $review->rating ); |
| 89 | 93 | $this->setTermCounts( $termId, $counts ); |
| 90 | 94 | } |
@@ -106,7 +110,9 @@ discard block |
||
| 106 | 110 | 'min' => Rating::MIN_RATING, |
| 107 | 111 | ]); |
| 108 | 112 | foreach( $counts as $index => &$num ) { |
| 109 | - if( $index >= intval( $args['min'] ) && $index <= intval( $args['max'] ))continue; |
|
| 113 | + if( $index >= intval( $args['min'] ) && $index <= intval( $args['max'] )) { |
|
| 114 | + continue; |
|
| 115 | + } |
|
| 110 | 116 | $num = 0; |
| 111 | 117 | } |
| 112 | 118 | return $counts; |
@@ -191,7 +197,9 @@ discard block |
||
| 191 | 197 | */ |
| 192 | 198 | public function increasePostCounts( Review $review ) |
| 193 | 199 | { |
| 194 | - if( !( get_post( $review->assigned_to ) instanceof WP_Post ))return; |
|
| 200 | + if( !( get_post( $review->assigned_to ) instanceof WP_Post )) { |
|
| 201 | + return; |
|
| 202 | + } |
|
| 195 | 203 | $counts = $this->getPostCounts( $review->assigned_to ); |
| 196 | 204 | $counts = empty( $counts ) |
| 197 | 205 | ? $this->buildPostCounts( $review->assigned_to ) |
@@ -240,7 +248,9 @@ discard block |
||
| 240 | 248 | */ |
| 241 | 249 | public function setTermCounts( $termId, array $reviewCounts ) |
| 242 | 250 | { |
| 243 | - if( !term_exists( $termId ))return; |
|
| 251 | + if( !term_exists( $termId )) { |
|
| 252 | + return; |
|
| 253 | + } |
|
| 244 | 254 | $ratingCounts = glsr( CountsManager::class )->flatten( $reviewCounts ); |
| 245 | 255 | update_term_meta( $termId, static::META_COUNT, $reviewCounts ); |
| 246 | 256 | update_term_meta( $termId, static::META_AVERAGE, glsr( Rating::class )->getAverage( $ratingCounts )); |
@@ -258,7 +268,9 @@ discard block |
||
| 258 | 268 | while( $reviews = $this->queryReviews( $args, $lastPostId, $limit )) { |
| 259 | 269 | $types = array_keys( array_flip( array_column( $reviews, 'type' ))); |
| 260 | 270 | foreach( $types as $type ) { |
| 261 | - if( isset( $counts[$type] ))continue; |
|
| 271 | + if( isset( $counts[$type] )) { |
|
| 272 | + continue; |
|
| 273 | + } |
|
| 262 | 274 | $counts[$type] = array_fill_keys( range( 0, Rating::MAX_RATING ), 0 ); |
| 263 | 275 | } |
| 264 | 276 | foreach( $reviews as $review ) { |
@@ -307,7 +319,9 @@ discard block |
||
| 307 | 319 | { |
| 308 | 320 | foreach( $reviewCounts as &$counts ) { |
| 309 | 321 | foreach( range( 0, Rating::MAX_RATING ) as $index ) { |
| 310 | - if( isset( $counts[$index] ))continue; |
|
| 322 | + if( isset( $counts[$index] )) { |
|
| 323 | + continue; |
|
| 324 | + } |
|
| 311 | 325 | $counts[$index] = 0; |
| 312 | 326 | } |
| 313 | 327 | ksort( $counts ); |
@@ -111,7 +111,9 @@ discard block |
||
| 111 | 111 | $termIds = array_filter( array_map( 'trim', explode( ',', $commaSeparatedTermIds ))); |
| 112 | 112 | foreach( $termIds as $termId ) { |
| 113 | 113 | $term = get_term( $termId, Application::TAXONOMY ); |
| 114 | - if( !isset( $term->term_id ))continue; |
|
| 114 | + if( !isset( $term->term_id )) { |
|
| 115 | + continue; |
|
| 116 | + } |
|
| 115 | 117 | $terms[] = $term->term_id; |
| 116 | 118 | } |
| 117 | 119 | return $terms; |
@@ -123,7 +125,9 @@ discard block |
||
| 123 | 125 | */ |
| 124 | 126 | public function revert( $postId ) |
| 125 | 127 | { |
| 126 | - if( get_post_field( 'post_type', $postId ) != Application::POST_TYPE )return; |
|
| 128 | + if( get_post_field( 'post_type', $postId ) != Application::POST_TYPE ) { |
|
| 129 | + return; |
|
| 130 | + } |
|
| 127 | 131 | delete_post_meta( $postId, '_edit_last' ); |
| 128 | 132 | $result = wp_update_post([ |
| 129 | 133 | 'ID' => $postId, |
@@ -141,7 +145,9 @@ discard block |
||
| 141 | 145 | */ |
| 142 | 146 | public function single( WP_Post $post ) |
| 143 | 147 | { |
| 144 | - if( $post->post_type != Application::POST_TYPE )return; |
|
| 148 | + if( $post->post_type != Application::POST_TYPE ) { |
|
| 149 | + return; |
|
| 150 | + } |
|
| 145 | 151 | $review = new Review( $post ); |
| 146 | 152 | return apply_filters( 'site-reviews/get/review', $review, $post ); |
| 147 | 153 | } |
@@ -166,7 +172,9 @@ discard block |
||
| 166 | 172 | protected function setTerms( $postId, $termIds ) |
| 167 | 173 | { |
| 168 | 174 | $terms = $this->normalizeTerms( $termIds ); |
| 169 | - if( empty( $terms ))return; |
|
| 175 | + if( empty( $terms )) { |
|
| 176 | + return; |
|
| 177 | + } |
|
| 170 | 178 | $result = wp_set_object_terms( $postId, $terms, Application::TAXONOMY ); |
| 171 | 179 | if( is_wp_error( $result )) { |
| 172 | 180 | glsr_log()->error( $result->get_error_message() ); |
@@ -25,7 +25,9 @@ discard block |
||
| 25 | 25 | { |
| 26 | 26 | sort( $termIds ); |
| 27 | 27 | sort( $oldTermIds ); |
| 28 | - if( $termIds === $oldTermIds || !$this->isReviewPostId( $postId ))return; |
|
| 28 | + if( $termIds === $oldTermIds || !$this->isReviewPostId( $postId )) { |
|
| 29 | + return; |
|
| 30 | + } |
|
| 29 | 31 | $review = glsr( ReviewManager::class )->single( get_post( $postId )); |
| 30 | 32 | $ignoredTerms = array_intersect( $oldTermIds, $termIds ); |
| 31 | 33 | if( $review->term_ids = array_diff( $oldTermIds, $ignoredTerms )) { |
@@ -45,7 +47,9 @@ discard block |
||
| 45 | 47 | */ |
| 46 | 48 | public function onAfterCreate( $postData, $meta, $postId ) |
| 47 | 49 | { |
| 48 | - if( !$this->isReviewPostId( $postId ))return; |
|
| 50 | + if( !$this->isReviewPostId( $postId )) { |
|
| 51 | + return; |
|
| 52 | + } |
|
| 49 | 53 | $review = glsr( ReviewManager::class )->single( get_post( $postId )); |
| 50 | 54 | glsr( CountsManager::class )->increase( $review ); |
| 51 | 55 | } |
@@ -57,7 +61,9 @@ discard block |
||
| 57 | 61 | */ |
| 58 | 62 | public function onBeforeDelete( $postId ) |
| 59 | 63 | { |
| 60 | - if( !$this->isReviewPostId( $postId ))return; |
|
| 64 | + if( !$this->isReviewPostId( $postId )) { |
|
| 65 | + return; |
|
| 66 | + } |
|
| 61 | 67 | $review = glsr( ReviewManager::class )->single( get_post( $postId )); |
| 62 | 68 | glsr( CountsManager::class )->decrease( $review ); |
| 63 | 69 | } |
@@ -74,9 +80,13 @@ discard block |
||
| 74 | 80 | { |
| 75 | 81 | if( !$this->isReviewPostId( $postId ) |
| 76 | 82 | || !in_array( $metaKey, ['assigned_to', 'rating', 'review_type'] ) |
| 77 | - )return; |
|
| 83 | + ) { |
|
| 84 | + return; |
|
| 85 | + } |
|
| 78 | 86 | $review = glsr( ReviewManager::class )->single( get_post( $postId )); |
| 79 | - if( $review->$metaKey == $metaValue )return; |
|
| 87 | + if( $review->$metaKey == $metaValue ) { |
|
| 88 | + return; |
|
| 89 | + } |
|
| 80 | 90 | $method = glsr( Helper::class )->buildMethodName( $metaKey, 'onBeforeChange' ); |
| 81 | 91 | call_user_func( [$this, $method], $review, $metaValue ); |
| 82 | 92 | } |
@@ -122,7 +132,9 @@ discard block |
||
| 122 | 132 | */ |
| 123 | 133 | public function onChangeStatus( $newStatus, $oldStatus, WP_Post $post ) |
| 124 | 134 | { |
| 125 | - if( $post->post_type != Application::POST_TYPE || in_array( $oldStatus, ['new', $newStatus] ))return; |
|
| 135 | + if( $post->post_type != Application::POST_TYPE || in_array( $oldStatus, ['new', $newStatus] )) { |
|
| 136 | + return; |
|
| 137 | + } |
|
| 126 | 138 | $review = glsr( ReviewManager::class )->single( get_post( $post->ID )); |
| 127 | 139 | if( $post->post_status == 'publish' ) { |
| 128 | 140 | glsr( CountsManager::class )->increase( $review ); |
@@ -40,7 +40,9 @@ discard block |
||
| 40 | 40 | $this->ratingCounts = glsr( CountsManager::class )->flatten( $counts, [ |
| 41 | 41 | 'min' => $args['rating'], |
| 42 | 42 | ]); |
| 43 | - if( !array_sum( $this->ratingCounts ) && $this->isHidden( 'if_empty' ))return; |
|
| 43 | + if( !array_sum( $this->ratingCounts ) && $this->isHidden( 'if_empty' )) { |
|
| 44 | + return; |
|
| 45 | + } |
|
| 44 | 46 | $this->averageRating = glsr( Rating::class )->getAverage( $this->ratingCounts ); |
| 45 | 47 | $this->generateSchema(); |
| 46 | 48 | return glsr( Template::class )->build( 'templates/reviews-summary', [ |
@@ -60,7 +62,9 @@ discard block |
||
| 60 | 62 | */ |
| 61 | 63 | protected function buildPercentage() |
| 62 | 64 | { |
| 63 | - if( $this->isHidden( 'bars' ))return; |
|
| 65 | + if( $this->isHidden( 'bars' )) { |
|
| 66 | + return; |
|
| 67 | + } |
|
| 64 | 68 | $range = range( Rating::MAX_RATING, 1 ); |
| 65 | 69 | $percentages = preg_filter( '/$/', '%', glsr( Rating::class )->getPercentages( $this->ratingCounts )); |
| 66 | 70 | $bars = array_reduce( $range, function( $carry, $level ) use( $percentages ) { |
@@ -110,7 +114,9 @@ discard block |
||
| 110 | 114 | */ |
| 111 | 115 | protected function buildRating() |
| 112 | 116 | { |
| 113 | - if( $this->isHidden( 'rating' ))return; |
|
| 117 | + if( $this->isHidden( 'rating' )) { |
|
| 118 | + return; |
|
| 119 | + } |
|
| 114 | 120 | return $this->wrap( 'rating', '<span>'.$this->averageRating.'</span>' ); |
| 115 | 121 | } |
| 116 | 122 | |
@@ -119,7 +125,9 @@ discard block |
||
| 119 | 125 | */ |
| 120 | 126 | protected function buildStars() |
| 121 | 127 | { |
| 122 | - if( $this->isHidden( 'stars' ))return; |
|
| 128 | + if( $this->isHidden( 'stars' )) { |
|
| 129 | + return; |
|
| 130 | + } |
|
| 123 | 131 | $stars = glsr( Partial::class )->build( 'star-rating', ['rating' => $this->averageRating] ); |
| 124 | 132 | return $this->wrap( 'stars', $stars ); |
| 125 | 133 | } |
@@ -129,7 +137,9 @@ discard block |
||
| 129 | 137 | */ |
| 130 | 138 | protected function buildText() |
| 131 | 139 | { |
| 132 | - if( $this->isHidden( 'summary' ))return; |
|
| 140 | + if( $this->isHidden( 'summary' )) { |
|
| 141 | + return; |
|
| 142 | + } |
|
| 133 | 143 | $count = intval( array_sum( $this->ratingCounts )); |
| 134 | 144 | if( empty( $this->args['text'] )) { |
| 135 | 145 | // @todo document this change |
@@ -154,7 +164,9 @@ discard block |
||
| 154 | 164 | */ |
| 155 | 165 | protected function generateSchema() |
| 156 | 166 | { |
| 157 | - if( !wp_validate_boolean( $this->args['schema'] ))return; |
|
| 167 | + if( !wp_validate_boolean( $this->args['schema'] )) { |
|
| 168 | + return; |
|
| 169 | + } |
|
| 158 | 170 | glsr( Schema::class )->store( |
| 159 | 171 | glsr( Schema::class )->buildSummary( $this->args ) |
| 160 | 172 | ); |
@@ -47,7 +47,9 @@ discard block |
||
| 47 | 47 | } |
| 48 | 48 | $views = $this->generatePossibleViews( $view ); |
| 49 | 49 | foreach( $views as $possibleView ) { |
| 50 | - if( !file_exists( glsr()->path( 'views/'.$possibleView.'.php' )))continue; |
|
| 50 | + if( !file_exists( glsr()->path( 'views/'.$possibleView.'.php' ))) { |
|
| 51 | + continue; |
|
| 52 | + } |
|
| 51 | 53 | return $possibleView; |
| 52 | 54 | } |
| 53 | 55 | return $view; |
@@ -84,7 +86,9 @@ discard block |
||
| 84 | 86 | */ |
| 85 | 87 | public function modifyField( Builder $instance ) |
| 86 | 88 | { |
| 87 | - if( !$this->isPublicInstance( $instance ) || empty( array_filter( $this->fields )))return; |
|
| 89 | + if( !$this->isPublicInstance( $instance ) || empty( array_filter( $this->fields ))) { |
|
| 90 | + return; |
|
| 91 | + } |
|
| 88 | 92 | call_user_func_array( [$this, 'customize'], [&$instance] ); |
| 89 | 93 | } |
| 90 | 94 | |
@@ -93,7 +93,9 @@ discard block |
||
| 93 | 93 | foreach( $this->rules as $attribute => $rules ) { |
| 94 | 94 | foreach( $rules as $rule ) { |
| 95 | 95 | $this->validateAttribute( $attribute, $rule ); |
| 96 | - if( $this->shouldStopValidating( $attribute ))break; |
|
| 96 | + if( $this->shouldStopValidating( $attribute )) { |
|
| 97 | + break; |
|
| 98 | + } |
|
| 97 | 99 | } |
| 98 | 100 | } |
| 99 | 101 | return $this->errors; |
@@ -109,7 +111,9 @@ discard block |
||
| 109 | 111 | public function validateAttribute( $attribute, $rule ) |
| 110 | 112 | { |
| 111 | 113 | list( $rule, $parameters ) = $this->parseRule( $rule ); |
| 112 | - if( $rule == '' )return; |
|
| 114 | + if( $rule == '' ) { |
|
| 115 | + return; |
|
| 116 | + } |
|
| 113 | 117 | $value = $this->getValue( $attribute ); |
| 114 | 118 | if( !method_exists( $this, $method = 'validate'.$rule )) { |
| 115 | 119 | throw new BadMethodCallException( "Method [$method] does not exist." ); |
@@ -178,7 +182,9 @@ discard block |
||
| 178 | 182 | */ |
| 179 | 183 | protected function getRule( $attribute, $rules ) |
| 180 | 184 | { |
| 181 | - if( !array_key_exists( $attribute, $this->rules ))return; |
|
| 185 | + if( !array_key_exists( $attribute, $this->rules )) { |
|
| 186 | + return; |
|
| 187 | + } |
|
| 182 | 188 | $rules = (array)$rules; |
| 183 | 189 | foreach( $this->rules[$attribute] as $rule ) { |
| 184 | 190 | list( $rule, $parameters ) = $this->parseRule( $rule ); |