@@ -17,9 +17,13 @@ discard block |
||
17 | 17 | { |
18 | 18 | $queries = []; |
19 | 19 | foreach( $keys as $key ) { |
20 | - if( !array_key_exists( $key, $values ))continue; |
|
20 | + if( !array_key_exists( $key, $values )) { |
|
21 | + continue; |
|
22 | + } |
|
21 | 23 | $methodName = glsr( Helper::class )->buildMethodName( $key, __FUNCTION__ ); |
22 | - if( !method_exists( $this, $methodName ))continue; |
|
24 | + if( !method_exists( $this, $methodName )) { |
|
25 | + continue; |
|
26 | + } |
|
23 | 27 | $query = call_user_func( [$this, $methodName], $values[$key] ); |
24 | 28 | if( is_array( $query )) { |
25 | 29 | $queries[] = $query; |
@@ -36,7 +40,9 @@ discard block |
||
36 | 40 | $string = ''; |
37 | 41 | $values = array_filter( $values ); |
38 | 42 | foreach( $conditions as $key => $value ) { |
39 | - if( !isset( $values[$key] ))continue; |
|
43 | + if( !isset( $values[$key] )) { |
|
44 | + continue; |
|
45 | + } |
|
40 | 46 | $values[$key] = implode( ',', (array)$values[$key] ); |
41 | 47 | $string .= strpos( $value, '%s' ) !== false |
42 | 48 | ? sprintf( $value, strval( $values[$key] )) |
@@ -110,7 +116,9 @@ discard block |
||
110 | 116 | */ |
111 | 117 | protected function buildQueryAssignedTo( $value ) |
112 | 118 | { |
113 | - if( empty( $value ))return; |
|
119 | + if( empty( $value )) { |
|
120 | + return; |
|
121 | + } |
|
114 | 122 | $postIds = glsr( Helper::class )->convertStringToArray( $value, 'is_numeric' ); |
115 | 123 | return [ |
116 | 124 | 'compare' => 'IN', |
@@ -125,7 +133,9 @@ discard block |
||
125 | 133 | */ |
126 | 134 | protected function buildQueryCategory( $value ) |
127 | 135 | { |
128 | - if( empty( $value ))return; |
|
136 | + if( empty( $value )) { |
|
137 | + return; |
|
138 | + } |
|
129 | 139 | return [ |
130 | 140 | 'field' => 'term_id', |
131 | 141 | 'taxonomy' => Application::TAXONOMY, |
@@ -139,7 +149,9 @@ discard block |
||
139 | 149 | */ |
140 | 150 | protected function buildQueryRating( $value ) |
141 | 151 | { |
142 | - if( !is_numeric( $value ) || !in_array( intval( $value ), range( 1, 5 )))return; |
|
152 | + if( !is_numeric( $value ) || !in_array( intval( $value ), range( 1, 5 ))) { |
|
153 | + return; |
|
154 | + } |
|
143 | 155 | return [ |
144 | 156 | 'compare' => '>=', |
145 | 157 | 'key' => 'rating', |
@@ -153,7 +165,9 @@ discard block |
||
153 | 165 | */ |
154 | 166 | protected function buildQueryType( $value ) |
155 | 167 | { |
156 | - if( in_array( $value, ['','all'] ))return; |
|
168 | + if( in_array( $value, ['','all'] )) { |
|
169 | + return; |
|
170 | + } |
|
157 | 171 | return [ |
158 | 172 | 'key' => 'review_type', |
159 | 173 | 'value' => $value, |
@@ -74,7 +74,9 @@ discard block |
||
74 | 74 | */ |
75 | 75 | public function decreasePostCounts( Review $review ) |
76 | 76 | { |
77 | - if( empty( $counts = $this->getPostCounts( $review->assigned_to )))return; |
|
77 | + if( empty( $counts = $this->getPostCounts( $review->assigned_to ))) { |
|
78 | + return; |
|
79 | + } |
|
78 | 80 | $counts = $this->decreaseRating( $counts, $review->review_type, $review->rating ); |
79 | 81 | $this->setPostCounts( $review->assigned_to, $counts ); |
80 | 82 | } |
@@ -85,7 +87,9 @@ discard block |
||
85 | 87 | public function decreaseTermCounts( Review $review ) |
86 | 88 | { |
87 | 89 | foreach( $review->term_ids as $termId ) { |
88 | - if( empty( $counts = $this->getTermCounts( $termId )))continue; |
|
90 | + if( empty( $counts = $this->getTermCounts( $termId ))) { |
|
91 | + continue; |
|
92 | + } |
|
89 | 93 | $counts = $this->decreaseRating( $counts, $review->review_type, $review->rating ); |
90 | 94 | $this->setTermCounts( $termId, $counts ); |
91 | 95 | } |
@@ -107,7 +111,9 @@ discard block |
||
107 | 111 | 'min' => Rating::MIN_RATING, |
108 | 112 | ]); |
109 | 113 | foreach( $counts as $index => &$num ) { |
110 | - if( $index >= intval( $args['min'] ) && $index <= intval( $args['max'] ))continue; |
|
114 | + if( $index >= intval( $args['min'] ) && $index <= intval( $args['max'] )) { |
|
115 | + continue; |
|
116 | + } |
|
111 | 117 | $num = 0; |
112 | 118 | } |
113 | 119 | return $counts; |
@@ -192,7 +198,9 @@ discard block |
||
192 | 198 | */ |
193 | 199 | public function increasePostCounts( Review $review ) |
194 | 200 | { |
195 | - if( !( get_post( $review->assigned_to ) instanceof WP_Post ))return; |
|
201 | + if( !( get_post( $review->assigned_to ) instanceof WP_Post )) { |
|
202 | + return; |
|
203 | + } |
|
196 | 204 | $counts = $this->getPostCounts( $review->assigned_to ); |
197 | 205 | $counts = empty( $counts ) |
198 | 206 | ? $this->buildPostCounts( $review->assigned_to ) |
@@ -241,7 +249,9 @@ discard block |
||
241 | 249 | */ |
242 | 250 | public function setTermCounts( $termId, array $reviewCounts ) |
243 | 251 | { |
244 | - if( !term_exists( $termId ))return; |
|
252 | + if( !term_exists( $termId )) { |
|
253 | + return; |
|
254 | + } |
|
245 | 255 | $ratingCounts = glsr( CountsManager::class )->flatten( $reviewCounts ); |
246 | 256 | update_term_meta( $termId, static::META_COUNT, $reviewCounts ); |
247 | 257 | update_term_meta( $termId, static::META_AVERAGE, glsr( Rating::class )->getAverage( $ratingCounts )); |
@@ -259,7 +269,9 @@ discard block |
||
259 | 269 | while( $reviews = $this->queryReviews( $args, $lastPostId, $limit )) { |
260 | 270 | $types = array_keys( array_flip( array_column( $reviews, 'type' ))); |
261 | 271 | foreach( $types as $type ) { |
262 | - if( isset( $counts[$type] ))continue; |
|
272 | + if( isset( $counts[$type] )) { |
|
273 | + continue; |
|
274 | + } |
|
263 | 275 | $counts[$type] = array_fill_keys( range( 0, Rating::MAX_RATING ), 0 ); |
264 | 276 | } |
265 | 277 | foreach( $reviews as $review ) { |
@@ -311,7 +323,9 @@ discard block |
||
311 | 323 | } |
312 | 324 | foreach( $reviewCounts as &$counts ) { |
313 | 325 | foreach( range( 0, Rating::MAX_RATING ) as $index ) { |
314 | - if( isset( $counts[$index] ))continue; |
|
326 | + if( isset( $counts[$index] )) { |
|
327 | + continue; |
|
328 | + } |
|
315 | 329 | $counts[$index] = 0; |
316 | 330 | } |
317 | 331 | ksort( $counts ); |
@@ -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()->path( $possibleView.'.php' )))continue; |
|
58 | + if( !file_exists( glsr()->path( $possibleView.'.php' ))) { |
|
59 | + continue; |
|
60 | + } |
|
59 | 61 | return $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 |
@@ -41,7 +41,9 @@ discard block |
||
41 | 41 | $this->ratingCounts = glsr( CountsManager::class )->flatten( $counts, [ |
42 | 42 | 'min' => $args['rating'], |
43 | 43 | ]); |
44 | - if( !array_sum( $this->ratingCounts ) && $this->isHidden( 'if_empty' ))return; |
|
44 | + if( !array_sum( $this->ratingCounts ) && $this->isHidden( 'if_empty' )) { |
|
45 | + return; |
|
46 | + } |
|
45 | 47 | $this->averageRating = glsr( Rating::class )->getAverage( $this->ratingCounts ); |
46 | 48 | $this->generateSchema(); |
47 | 49 | return glsr( Template::class )->build( 'templates/reviews-summary', [ |
@@ -61,7 +63,9 @@ discard block |
||
61 | 63 | */ |
62 | 64 | protected function buildPercentage() |
63 | 65 | { |
64 | - if( $this->isHidden( 'bars' ))return; |
|
66 | + if( $this->isHidden( 'bars' )) { |
|
67 | + return; |
|
68 | + } |
|
65 | 69 | $percentages = preg_filter( '/$/', '%', glsr( Rating::class )->getPercentages( $this->ratingCounts )); |
66 | 70 | $bars = array_reduce( range( Rating::MAX_RATING, 1 ), function( $carry, $level ) use( $percentages ) { |
67 | 71 | $label = $this->buildPercentageLabel( $this->args['labels'][$level] ); |
@@ -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 | ); |
@@ -20,7 +20,9 @@ |
||
20 | 20 | public function build( array $args = [] ) |
21 | 21 | { |
22 | 22 | $this->args = $this->normalize( $args ); |
23 | - if( $this->args['total'] < 2 )return; |
|
23 | + if( $this->args['total'] < 2 ) { |
|
24 | + return; |
|
25 | + } |
|
24 | 26 | return glsr( Template::class )->build( 'templates/pagination', [ |
25 | 27 | 'context' => [ |
26 | 28 | 'links' => apply_filters( 'site-reviews/paginate_links', $this->buildLinks(), $this->args ), |
@@ -85,7 +85,9 @@ discard block |
||
85 | 85 | $renderedFields = []; |
86 | 86 | foreach( $review as $key => $value ) { |
87 | 87 | $method = glsr( Helper::class )->buildMethodName( $key, 'buildOption' ); |
88 | - if( !method_exists( $this, $method ))continue; |
|
88 | + if( !method_exists( $this, $method )) { |
|
89 | + continue; |
|
90 | + } |
|
89 | 91 | $renderedFields[$key] = $this->$method( $key, $value ); |
90 | 92 | } |
91 | 93 | $renderedFields = apply_filters( 'site-reviews/review/build/after', (array)$renderedFields ); |
@@ -99,9 +101,13 @@ discard block |
||
99 | 101 | */ |
100 | 102 | protected function buildOptionAssignedTo( $key, $value ) |
101 | 103 | { |
102 | - if( $this->isHidden( $key, 'settings.reviews.assigned_links' ))return; |
|
104 | + if( $this->isHidden( $key, 'settings.reviews.assigned_links' )) { |
|
105 | + return; |
|
106 | + } |
|
103 | 107 | $post = glsr( Polylang::class )->getPost( $value ); |
104 | - if( !( $post instanceof WP_Post ))return; |
|
108 | + if( !( $post instanceof WP_Post )) { |
|
109 | + return; |
|
110 | + } |
|
105 | 111 | $permalink = glsr( Builder::class )->a( get_the_title( $post->ID ), [ |
106 | 112 | 'href' => get_the_permalink( $post->ID ), |
107 | 113 | ]); |
@@ -116,7 +122,9 @@ discard block |
||
116 | 122 | */ |
117 | 123 | protected function buildOptionAuthor( $key, $value ) |
118 | 124 | { |
119 | - if( $this->isHidden( $key ))return; |
|
125 | + if( $this->isHidden( $key )) { |
|
126 | + return; |
|
127 | + } |
|
120 | 128 | return $this->wrap( $key, '<span>'.$value.'</span>' ); |
121 | 129 | } |
122 | 130 | |
@@ -127,7 +135,9 @@ discard block |
||
127 | 135 | */ |
128 | 136 | protected function buildOptionAvatar( $key, $value ) |
129 | 137 | { |
130 | - if( $this->isHidden( $key, 'settings.reviews.avatars' ))return; |
|
138 | + if( $this->isHidden( $key, 'settings.reviews.avatars' )) { |
|
139 | + return; |
|
140 | + } |
|
131 | 141 | $size = $this->getOption( 'settings.reviews.avatars_size', 40 ); |
132 | 142 | return $this->wrap( $key, glsr( Builder::class )->img([ |
133 | 143 | 'src' => $this->generateAvatar( $value ), |
@@ -144,7 +154,9 @@ discard block |
||
144 | 154 | protected function buildOptionContent( $key, $value ) |
145 | 155 | { |
146 | 156 | $text = $this->normalizeText( $value ); |
147 | - if( $this->isHiddenOrEmpty( $key, $text ))return; |
|
157 | + if( $this->isHiddenOrEmpty( $key, $text )) { |
|
158 | + return; |
|
159 | + } |
|
148 | 160 | return $this->wrap( $key, '<p>'.$text.'</p>' ); |
149 | 161 | } |
150 | 162 | |
@@ -155,7 +167,9 @@ discard block |
||
155 | 167 | */ |
156 | 168 | protected function buildOptionDate( $key, $value ) |
157 | 169 | { |
158 | - if( $this->isHidden( $key ))return; |
|
170 | + if( $this->isHidden( $key )) { |
|
171 | + return; |
|
172 | + } |
|
159 | 173 | $dateFormat = $this->getOption( 'settings.reviews.date.format', 'default' ); |
160 | 174 | if( $dateFormat == 'relative' ) { |
161 | 175 | $date = glsr( Date::class )->relative( $value ); |
@@ -176,7 +190,9 @@ discard block |
||
176 | 190 | */ |
177 | 191 | protected function buildOptionRating( $key, $value ) |
178 | 192 | { |
179 | - if( $this->isHiddenOrEmpty( $key, $value ))return; |
|
193 | + if( $this->isHiddenOrEmpty( $key, $value )) { |
|
194 | + return; |
|
195 | + } |
|
180 | 196 | $rating = glsr( Partial::class )->build( 'star-rating', [ |
181 | 197 | 'rating' => $value, |
182 | 198 | ]); |
@@ -190,7 +206,9 @@ discard block |
||
190 | 206 | */ |
191 | 207 | protected function buildOptionResponse( $key, $value ) |
192 | 208 | { |
193 | - if( $this->isHiddenOrEmpty( $key, $value ))return; |
|
209 | + if( $this->isHiddenOrEmpty( $key, $value )) { |
|
210 | + return; |
|
211 | + } |
|
194 | 212 | $title = sprintf( __( 'Response from %s', 'site-reviews' ), get_bloginfo( 'name' )); |
195 | 213 | $text = $this->normalizeText( $value ); |
196 | 214 | $text = '<p><strong>'.$title.'</strong></p><p>'.$text.'</p>'; |
@@ -207,7 +225,9 @@ discard block |
||
207 | 225 | */ |
208 | 226 | protected function buildOptionTitle( $key, $value ) |
209 | 227 | { |
210 | - if( $this->isHidden( $key ))return; |
|
228 | + if( $this->isHidden( $key )) { |
|
229 | + return; |
|
230 | + } |
|
211 | 231 | if( empty( $value )) { |
212 | 232 | $value = __( 'No Title', 'site-reviews' ); |
213 | 233 | } |
@@ -239,7 +259,9 @@ discard block |
||
239 | 259 | */ |
240 | 260 | protected function generateSchema() |
241 | 261 | { |
242 | - if( !wp_validate_boolean( $this->args['schema'] ))return; |
|
262 | + if( !wp_validate_boolean( $this->args['schema'] )) { |
|
263 | + return; |
|
264 | + } |
|
243 | 265 | glsr( Schema::class )->store( |
244 | 266 | glsr( Schema::class )->build( $this->args ) |
245 | 267 | ); |
@@ -289,9 +311,13 @@ discard block |
||
289 | 311 | $words->setText( $text ); |
290 | 312 | $count = 0; |
291 | 313 | foreach( $words as $offset ){ |
292 | - if( $words->getRuleStatus() === IntlRuleBasedBreakIterator::WORD_NONE )continue; |
|
314 | + if( $words->getRuleStatus() === IntlRuleBasedBreakIterator::WORD_NONE ) { |
|
315 | + continue; |
|
316 | + } |
|
293 | 317 | $count++; |
294 | - if( $count != $limit )continue; |
|
318 | + if( $count != $limit ) { |
|
319 | + continue; |
|
320 | + } |
|
295 | 321 | return $offset; |
296 | 322 | } |
297 | 323 | return strlen( $text ); |
@@ -22,7 +22,9 @@ |
||
22 | 22 | } |
23 | 23 | glsr( Session::class )->set( $command->form_id.'message', __( 'Your review has been submitted!', 'site-reviews' )); |
24 | 24 | glsr( Notification::class )->send( $review ); |
25 | - if( $command->ajax_request )return; |
|
25 | + if( $command->ajax_request ) { |
|
26 | + return; |
|
27 | + } |
|
26 | 28 | wp_safe_redirect( $this->getReferer( $command )); |
27 | 29 | exit; |
28 | 30 | } |
@@ -60,7 +60,9 @@ discard block |
||
60 | 60 | public function catchFatalError() |
61 | 61 | { |
62 | 62 | $error = error_get_last(); |
63 | - if( $error['type'] !== E_ERROR )return; |
|
63 | + if( $error['type'] !== E_ERROR ) { |
|
64 | + return; |
|
65 | + } |
|
64 | 66 | glsr_log()->error( $error['message'] ); |
65 | 67 | } |
66 | 68 | |
@@ -204,7 +206,9 @@ discard block |
||
204 | 206 | */ |
205 | 207 | public function scheduleCronJob() |
206 | 208 | { |
207 | - if( wp_next_scheduled( static::CRON_EVENT ))return; |
|
209 | + if( wp_next_scheduled( static::CRON_EVENT )) { |
|
210 | + return; |
|
211 | + } |
|
208 | 212 | wp_schedule_event( time(), 'twicedaily', static::CRON_EVENT ); |
209 | 213 | } |
210 | 214 | |
@@ -235,7 +239,9 @@ discard block |
||
235 | 239 | || !in_array( plugin_basename( $this->file ), $data['plugins'] ) |
236 | 240 | || $data['action'] != 'update' |
237 | 241 | || $data['type'] != 'plugin' |
238 | - )return; |
|
242 | + ) { |
|
243 | + return; |
|
244 | + } |
|
239 | 245 | $this->upgrade(); |
240 | 246 | } |
241 | 247 |
@@ -20,7 +20,9 @@ discard block |
||
20 | 20 | $filenames = []; |
21 | 21 | $iterator = new DirectoryIterator( dirname( __FILE__ ).'/Upgrader' ); |
22 | 22 | foreach( $iterator as $fileinfo ) { |
23 | - if( !$fileinfo->isFile() )continue; |
|
23 | + if( !$fileinfo->isFile() ) { |
|
24 | + continue; |
|
25 | + } |
|
24 | 26 | $filenames[] = $fileinfo->getFilename(); |
25 | 27 | } |
26 | 28 | natsort( $filenames ); |
@@ -28,7 +30,9 @@ discard block |
||
28 | 30 | $className = str_replace( '.php', '', $file ); |
29 | 31 | $version = str_replace( ['Upgrade_', '_'], ['', '.'], $className ); |
30 | 32 | $versionSuffix = preg_replace( '/[\d.]+(.+)?/', '${1}', glsr()->version ); // allow alpha/beta versions |
31 | - if( version_compare( $this->currentVersion(), $version.$versionSuffix, '>=' ))return; |
|
33 | + if( version_compare( $this->currentVersion(), $version.$versionSuffix, '>=' )) { |
|
34 | + return; |
|
35 | + } |
|
32 | 36 | glsr( 'Modules\\Upgrader\\'.$className ); |
33 | 37 | glsr_log()->info( 'Completed Upgrade for v'.$version.$versionSuffix ); |
34 | 38 | }); |