@@ -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 ); |