@@ -27,9 +27,9 @@ |
||
27 | 27 | public function getFields( $id ) |
28 | 28 | { |
29 | 29 | $fields = []; |
30 | - $values = apply_filters( 'site-reviews/form/fields', glsr()->config( 'forms/'.$id )); |
|
30 | + $values = apply_filters( 'site-reviews/form/fields', glsr()->config( 'forms/'.$id ) ); |
|
31 | 31 | foreach( $values as $name => $field ) { |
32 | - $fields[] = new Field( wp_parse_args( $field, ['name' => $name] )); |
|
32 | + $fields[] = new Field( wp_parse_args( $field, ['name' => $name] ) ); |
|
33 | 33 | } |
34 | 34 | return $fields; |
35 | 35 | } |
@@ -55,7 +55,7 @@ discard block |
||
55 | 55 | { |
56 | 56 | $instance = new static; |
57 | 57 | $instance->setTagFromMethod( $method ); |
58 | - call_user_func_array( [$instance, 'normalize'], $args += ['',''] ); |
|
58 | + call_user_func_array( [$instance, 'normalize'], $args += ['', ''] ); |
|
59 | 59 | $tags = array_merge( static::TAGS_FORM, static::TAGS_SINGLE, static::TAGS_STRUCTURE, static::TAGS_TEXT ); |
60 | 60 | do_action_ref_array( 'site-reviews/builder', [&$instance] ); |
61 | 61 | $generatedTag = in_array( $instance->tag, $tags ) |
@@ -79,8 +79,8 @@ discard block |
||
79 | 79 | 'render' => 'is_bool', |
80 | 80 | 'tag' => 'is_string', |
81 | 81 | ]; |
82 | - if( !isset( $properties[$property] ) |
|
83 | - || empty( array_filter( [$value], $properties[$property] )) |
|
82 | + if( !isset($properties[$property]) |
|
83 | + || empty(array_filter( [$value], $properties[$property] )) |
|
84 | 84 | )return; |
85 | 85 | $this->$property = $value; |
86 | 86 | } |
@@ -90,7 +90,7 @@ discard block |
||
90 | 90 | */ |
91 | 91 | public function getClosingTag() |
92 | 92 | { |
93 | - if( empty( $this->tag ))return; |
|
93 | + if( empty($this->tag) )return; |
|
94 | 94 | return '</'.$this->tag.'>'; |
95 | 95 | } |
96 | 96 | |
@@ -99,8 +99,8 @@ discard block |
||
99 | 99 | */ |
100 | 100 | public function getOpeningTag() |
101 | 101 | { |
102 | - if( empty( $this->tag ))return; |
|
103 | - $attributes = glsr( Attributes::class )->{$this->tag}( $this->args )->toString(); |
|
102 | + if( empty($this->tag) )return; |
|
103 | + $attributes = glsr( Attributes::class )->{$this->tag}($this->args)->toString(); |
|
104 | 104 | return '<'.trim( $this->tag.' '.$attributes ).'>'; |
105 | 105 | } |
106 | 106 | |
@@ -109,8 +109,8 @@ discard block |
||
109 | 109 | */ |
110 | 110 | public function raw( array $field ) |
111 | 111 | { |
112 | - unset( $field['label'] ); |
|
113 | - return $this->{$field['type']}( $field ); |
|
112 | + unset($field['label']); |
|
113 | + return $this->{$field['type']}($field); |
|
114 | 114 | } |
115 | 115 | |
116 | 116 | /** |
@@ -119,7 +119,7 @@ discard block |
||
119 | 119 | protected function buildCustomField() |
120 | 120 | { |
121 | 121 | $className = $this->getCustomFieldClassName(); |
122 | - if( class_exists( $className )) { |
|
122 | + if( class_exists( $className ) ) { |
|
123 | 123 | return (new $className( $this ))->build(); |
124 | 124 | } |
125 | 125 | glsr_log()->error( 'Field missing: '.$className ); |
@@ -130,7 +130,7 @@ discard block |
||
130 | 130 | */ |
131 | 131 | protected function buildDefaultTag( $text = '' ) |
132 | 132 | { |
133 | - if( empty( $text )) { |
|
133 | + if( empty($text) ) { |
|
134 | 134 | $text = $this->args['text']; |
135 | 135 | } |
136 | 136 | return $this->getOpeningTag().$text.$this->getClosingTag(); |
@@ -141,7 +141,7 @@ discard block |
||
141 | 141 | */ |
142 | 142 | protected function buildFieldDescription() |
143 | 143 | { |
144 | - if( empty( $this->args['description'] ))return; |
|
144 | + if( empty($this->args['description']) )return; |
|
145 | 145 | if( $this->args['is_widget'] ) { |
146 | 146 | return $this->small( $this->args['description'] ); |
147 | 147 | } |
@@ -153,13 +153,13 @@ discard block |
||
153 | 153 | */ |
154 | 154 | protected function buildFormInput() |
155 | 155 | { |
156 | - if( !in_array( $this->args['type'], ['checkbox', 'radio'] )) { |
|
157 | - if( isset( $this->args['multiple'] )) { |
|
158 | - $this->args['name'].= '[]'; |
|
156 | + if( !in_array( $this->args['type'], ['checkbox', 'radio'] ) ) { |
|
157 | + if( isset($this->args['multiple']) ) { |
|
158 | + $this->args['name'] .= '[]'; |
|
159 | 159 | } |
160 | 160 | return $this->buildFormLabel().$this->getOpeningTag(); |
161 | 161 | } |
162 | - return empty( $this->args['options'] ) |
|
162 | + return empty($this->args['options']) |
|
163 | 163 | ? $this->buildFormInputChoice() |
164 | 164 | : $this->buildFormInputMultiChoice(); |
165 | 165 | } |
@@ -169,13 +169,13 @@ discard block |
||
169 | 169 | */ |
170 | 170 | protected function buildFormInputChoice() |
171 | 171 | { |
172 | - if( !empty( $this->args['text'] )) { |
|
172 | + if( !empty($this->args['text']) ) { |
|
173 | 173 | $this->args['label'] = $this->args['text']; |
174 | 174 | } |
175 | - return $this->getOpeningTag().$this->buildFormLabel([ |
|
175 | + return $this->getOpeningTag().$this->buildFormLabel( [ |
|
176 | 176 | 'class' => 'glsr-'.$this->args['type'].'-label', |
177 | 177 | 'text' => $this->args['label'].'<span></span>', |
178 | - ]); |
|
178 | + ] ); |
|
179 | 179 | } |
180 | 180 | |
181 | 181 | /** |
@@ -184,22 +184,22 @@ discard block |
||
184 | 184 | protected function buildFormInputMultiChoice() |
185 | 185 | { |
186 | 186 | if( $this->args['type'] == 'checkbox' ) { |
187 | - $this->args['name'].= '[]'; |
|
187 | + $this->args['name'] .= '[]'; |
|
188 | 188 | } |
189 | 189 | $index = 0; |
190 | - $options = array_reduce( array_keys( $this->args['options'] ), function( $carry, $key ) use( &$index ) { |
|
190 | + $options = array_reduce( array_keys( $this->args['options'] ), function( $carry, $key ) use(&$index) { |
|
191 | 191 | return $carry.$this->li( $this->{$this->args['type']}([ |
192 | 192 | 'checked' => in_array( $key, (array)$this->args['value'] ), |
193 | 193 | 'id' => $this->args['id'].'-'.$index++, |
194 | 194 | 'name' => $this->args['name'], |
195 | 195 | 'text' => $this->args['options'][$key], |
196 | 196 | 'value' => $key, |
197 | - ])); |
|
197 | + ]) ); |
|
198 | 198 | }); |
199 | 199 | return $this->ul( $options, [ |
200 | 200 | 'class' => $this->args['class'], |
201 | 201 | 'id' => $this->args['id'], |
202 | - ]); |
|
202 | + ] ); |
|
203 | 203 | } |
204 | 204 | |
205 | 205 | /** |
@@ -207,13 +207,13 @@ discard block |
||
207 | 207 | */ |
208 | 208 | protected function buildFormLabel( array $customArgs = [] ) |
209 | 209 | { |
210 | - if( empty( $this->args['label'] ) || $this->args['type'] == 'hidden' )return; |
|
210 | + if( empty($this->args['label']) || $this->args['type'] == 'hidden' )return; |
|
211 | 211 | return $this->label( wp_parse_args( $customArgs, [ |
212 | 212 | 'for' => $this->args['id'], |
213 | 213 | 'is_public' => $this->args['is_public'], |
214 | 214 | 'text' => $this->args['label'], |
215 | 215 | 'type' => $this->args['type'], |
216 | - ])); |
|
216 | + ] ) ); |
|
217 | 217 | } |
218 | 218 | |
219 | 219 | /** |
@@ -230,11 +230,11 @@ discard block |
||
230 | 230 | protected function buildFormSelectOptions() |
231 | 231 | { |
232 | 232 | return array_reduce( array_keys( $this->args['options'] ), function( $carry, $key ) { |
233 | - return $carry.$this->option([ |
|
233 | + return $carry.$this->option( [ |
|
234 | 234 | 'selected' => $this->args['value'] == $key, |
235 | 235 | 'text' => $this->args['options'][$key], |
236 | 236 | 'value' => $key, |
237 | - ]); |
|
237 | + ] ); |
|
238 | 238 | }); |
239 | 239 | } |
240 | 240 | |
@@ -252,10 +252,10 @@ discard block |
||
252 | 252 | protected function buildTag() |
253 | 253 | { |
254 | 254 | $this->mergeArgsWithRequiredDefaults(); |
255 | - if( in_array( $this->tag, static::TAGS_SINGLE )) { |
|
255 | + if( in_array( $this->tag, static::TAGS_SINGLE ) ) { |
|
256 | 256 | return $this->getOpeningTag(); |
257 | 257 | } |
258 | - if( !in_array( $this->tag, static::TAGS_FORM )) { |
|
258 | + if( !in_array( $this->tag, static::TAGS_FORM ) ) { |
|
259 | 259 | return $this->buildDefaultTag(); |
260 | 260 | } |
261 | 261 | return call_user_func( [$this, 'buildForm'.ucfirst( $this->tag )] ).$this->buildFieldDescription(); |
@@ -275,7 +275,7 @@ discard block |
||
275 | 275 | protected function mergeArgsWithRequiredDefaults() |
276 | 276 | { |
277 | 277 | $className = $this->getCustomFieldClassName(); |
278 | - if( class_exists( $className )) { |
|
278 | + if( class_exists( $className ) ) { |
|
279 | 279 | $this->args = array_merge( |
280 | 280 | wp_parse_args( $this->args, $className::defaults() ), |
281 | 281 | $className::required() |
@@ -290,16 +290,16 @@ discard block |
||
290 | 290 | */ |
291 | 291 | protected function normalize( ...$params ) |
292 | 292 | { |
293 | - if( is_string( $params[0] ) || is_numeric( $params[0] )) { |
|
293 | + if( is_string( $params[0] ) || is_numeric( $params[0] ) ) { |
|
294 | 294 | $this->setNameOrTextAttributeForTag( $params[0] ); |
295 | 295 | } |
296 | - if( is_array( $params[0] )) { |
|
296 | + if( is_array( $params[0] ) ) { |
|
297 | 297 | $this->args += $params[0]; |
298 | 298 | } |
299 | - else if( is_array( $params[1] )) { |
|
299 | + else if( is_array( $params[1] ) ) { |
|
300 | 300 | $this->args += $params[1]; |
301 | 301 | } |
302 | - if( !isset( $this->args['is_public'] )) { |
|
302 | + if( !isset($this->args['is_public']) ) { |
|
303 | 303 | $this->args['is_public'] = false; |
304 | 304 | } |
305 | 305 | } |
@@ -323,7 +323,7 @@ discard block |
||
323 | 323 | protected function setTagFromMethod( $method ) |
324 | 324 | { |
325 | 325 | $this->tag = strtolower( $method ); |
326 | - if( in_array( $this->tag, static::INPUT_TYPES )) { |
|
326 | + if( in_array( $this->tag, static::INPUT_TYPES ) ) { |
|
327 | 327 | $this->args['type'] = $this->tag; |
328 | 328 | $this->tag = 'input'; |
329 | 329 | } |
@@ -81,7 +81,9 @@ discard block |
||
81 | 81 | ]; |
82 | 82 | if( !isset( $properties[$property] ) |
83 | 83 | || empty( array_filter( [$value], $properties[$property] )) |
84 | - )return; |
|
84 | + ) { |
|
85 | + return; |
|
86 | + } |
|
85 | 87 | $this->$property = $value; |
86 | 88 | } |
87 | 89 | |
@@ -90,7 +92,9 @@ discard block |
||
90 | 92 | */ |
91 | 93 | public function getClosingTag() |
92 | 94 | { |
93 | - if( empty( $this->tag ))return; |
|
95 | + if( empty( $this->tag )) { |
|
96 | + return; |
|
97 | + } |
|
94 | 98 | return '</'.$this->tag.'>'; |
95 | 99 | } |
96 | 100 | |
@@ -99,7 +103,9 @@ discard block |
||
99 | 103 | */ |
100 | 104 | public function getOpeningTag() |
101 | 105 | { |
102 | - if( empty( $this->tag ))return; |
|
106 | + if( empty( $this->tag )) { |
|
107 | + return; |
|
108 | + } |
|
103 | 109 | $attributes = glsr( Attributes::class )->{$this->tag}( $this->args )->toString(); |
104 | 110 | return '<'.trim( $this->tag.' '.$attributes ).'>'; |
105 | 111 | } |
@@ -141,7 +147,9 @@ discard block |
||
141 | 147 | */ |
142 | 148 | protected function buildFieldDescription() |
143 | 149 | { |
144 | - if( empty( $this->args['description'] ))return; |
|
150 | + if( empty( $this->args['description'] )) { |
|
151 | + return; |
|
152 | + } |
|
145 | 153 | if( $this->args['is_widget'] ) { |
146 | 154 | return $this->small( $this->args['description'] ); |
147 | 155 | } |
@@ -207,7 +215,9 @@ discard block |
||
207 | 215 | */ |
208 | 216 | protected function buildFormLabel( array $customArgs = [] ) |
209 | 217 | { |
210 | - if( empty( $this->args['label'] ) || $this->args['type'] == 'hidden' )return; |
|
218 | + if( empty( $this->args['label'] ) || $this->args['type'] == 'hidden' ) { |
|
219 | + return; |
|
220 | + } |
|
211 | 221 | return $this->label( wp_parse_args( $customArgs, [ |
212 | 222 | 'for' => $this->args['id'], |
213 | 223 | 'is_public' => $this->args['is_public'], |
@@ -37,12 +37,12 @@ discard block |
||
37 | 37 | 'templates/form/submit-button', |
38 | 38 | 'templates/reviews-form', |
39 | 39 | ]; |
40 | - if( !preg_match( '('.implode( '|', $styledViews ).')', $view )) { |
|
40 | + if( !preg_match( '('.implode( '|', $styledViews ).')', $view ) ) { |
|
41 | 41 | return $view; |
42 | 42 | } |
43 | 43 | $views = $this->generatePossibleViews( $view ); |
44 | 44 | foreach( $views as $possibleView ) { |
45 | - if( !file_exists( glsr()->path( 'views/'.$possibleView.'.php' )))continue; |
|
45 | + if( !file_exists( glsr()->path( 'views/'.$possibleView.'.php' ) ) )continue; |
|
46 | 46 | return $possibleView; |
47 | 47 | } |
48 | 48 | return $view; |
@@ -76,7 +76,7 @@ discard block |
||
76 | 76 | */ |
77 | 77 | public function modifyField( Builder $instance ) |
78 | 78 | { |
79 | - if( !$this->isPublicInstance( $instance ) || empty( array_filter( $this->styles )))return; |
|
79 | + if( !$this->isPublicInstance( $instance ) || empty(array_filter( $this->styles )) )return; |
|
80 | 80 | call_user_func_array( [$this, 'customize'], [&$instance] ); |
81 | 81 | } |
82 | 82 | |
@@ -85,9 +85,9 @@ discard block |
||
85 | 85 | */ |
86 | 86 | protected function customize( Builder $instance ) |
87 | 87 | { |
88 | - $args = wp_parse_args( $instance->args, array_fill_keys( ['class', 'type'], '' )); |
|
88 | + $args = wp_parse_args( $instance->args, array_fill_keys( ['class', 'type'], '' ) ); |
|
89 | 89 | $key = $instance->tag.'_'.$args['type']; |
90 | - $classes = !isset( $this->styles[$key] ) |
|
90 | + $classes = !isset($this->styles[$key]) |
|
91 | 91 | ? $this->styles[$instance->tag] |
92 | 92 | : $this->styles[$key]; |
93 | 93 | $instance->args['class'] = trim( $args['class'].' '.$classes ); |
@@ -121,7 +121,7 @@ discard block |
||
121 | 121 | $args = wp_parse_args( $instance->args, [ |
122 | 122 | 'is_public' => false, |
123 | 123 | 'is_raw' => false, |
124 | - ]); |
|
124 | + ] ); |
|
125 | 125 | if( is_admin() || !$args['is_public'] || $args['is_raw'] ) { |
126 | 126 | return false; |
127 | 127 | } |
@@ -42,7 +42,9 @@ discard block |
||
42 | 42 | } |
43 | 43 | $views = $this->generatePossibleViews( $view ); |
44 | 44 | foreach( $views as $possibleView ) { |
45 | - if( !file_exists( glsr()->path( 'views/'.$possibleView.'.php' )))continue; |
|
45 | + if( !file_exists( glsr()->path( 'views/'.$possibleView.'.php' ))) { |
|
46 | + continue; |
|
47 | + } |
|
46 | 48 | return $possibleView; |
47 | 49 | } |
48 | 50 | return $view; |
@@ -76,7 +78,9 @@ discard block |
||
76 | 78 | */ |
77 | 79 | public function modifyField( Builder $instance ) |
78 | 80 | { |
79 | - if( !$this->isPublicInstance( $instance ) || empty( array_filter( $this->styles )))return; |
|
81 | + if( !$this->isPublicInstance( $instance ) || empty( array_filter( $this->styles ))) { |
|
82 | + return; |
|
83 | + } |
|
80 | 84 | call_user_func_array( [$this, 'customize'], [&$instance] ); |
81 | 85 | } |
82 | 86 |
@@ -9,7 +9,7 @@ |
||
9 | 9 | */ |
10 | 10 | add_filter( 'sbp_exclude_defer_scripts', function( $scriptHandles ) { |
11 | 11 | $scriptHandles[] = 'site-reviews/google-recaptcha'; |
12 | - return array_keys( array_flip( $scriptHandles )); |
|
12 | + return array_keys( array_flip( $scriptHandles ) ); |
|
13 | 13 | }); |
14 | 14 | |
15 | 15 | /** |
@@ -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 | }); |
@@ -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 pm ON p.ID = pm.post_id |
110 | 110 | WHERE p.post_type = '{$this->postType}' |
111 | 111 | AND pm.meta_key = 'review_id' |
112 | 112 | AND pm.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( $termId, $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 |
@@ -62,13 +62,13 @@ discard block |
||
62 | 62 | 'review_type' => '', |
63 | 63 | ]; |
64 | 64 | $meta = array_filter( |
65 | - array_map( 'array_shift', (array)get_post_meta( $post->ID )), |
|
65 | + array_map( 'array_shift', (array)get_post_meta( $post->ID ) ), |
|
66 | 66 | 'strlen' |
67 | 67 | ); |
68 | - $properties = glsr( CreateReviewDefaults::class )->restrict( array_merge( $defaults, $meta )); |
|
68 | + $properties = glsr( CreateReviewDefaults::class )->restrict( array_merge( $defaults, $meta ) ); |
|
69 | 69 | $this->modified = $this->isModified( $properties ); |
70 | 70 | array_walk( $properties, function( $value, $key ) { |
71 | - if( !property_exists( $this, $key ) || isset( $this->$key ))return; |
|
71 | + if( !property_exists( $this, $key ) || isset($this->$key) )return; |
|
72 | 72 | $this->$key = $value; |
73 | 73 | }); |
74 | 74 | } |
@@ -79,7 +79,7 @@ discard block |
||
79 | 79 | protected function setTermIds( WP_Post $post ) |
80 | 80 | { |
81 | 81 | $this->term_ids = []; |
82 | - if( !is_array( $terms = get_the_terms( $post, Application::TAXONOMY )))return; |
|
82 | + if( !is_array( $terms = get_the_terms( $post, Application::TAXONOMY ) ) )return; |
|
83 | 83 | foreach( $terms as $term ) { |
84 | 84 | $this->term_ids[] = $term->term_id; |
85 | 85 | } |
@@ -68,7 +68,9 @@ discard block |
||
68 | 68 | $properties = glsr( CreateReviewDefaults::class )->restrict( array_merge( $defaults, $meta )); |
69 | 69 | $this->modified = $this->isModified( $properties ); |
70 | 70 | array_walk( $properties, function( $value, $key ) { |
71 | - if( !property_exists( $this, $key ) || isset( $this->$key ))return; |
|
71 | + if( !property_exists( $this, $key ) || isset( $this->$key )) { |
|
72 | + return; |
|
73 | + } |
|
72 | 74 | $this->$key = $value; |
73 | 75 | }); |
74 | 76 | } |
@@ -79,7 +81,9 @@ discard block |
||
79 | 81 | protected function setTermIds( WP_Post $post ) |
80 | 82 | { |
81 | 83 | $this->term_ids = []; |
82 | - if( !is_array( $terms = get_the_terms( $post, Application::TAXONOMY )))return; |
|
84 | + if( !is_array( $terms = get_the_terms( $post, Application::TAXONOMY ))) { |
|
85 | + return; |
|
86 | + } |
|
83 | 87 | foreach( $terms as $term ) { |
84 | 88 | $this->term_ids[] = $term->term_id; |
85 | 89 | } |
@@ -63,7 +63,7 @@ discard block |
||
63 | 63 | $this->getCounts(), |
64 | 64 | $review->review_type, |
65 | 65 | $review->rating |
66 | - )); |
|
66 | + ) ); |
|
67 | 67 | } |
68 | 68 | |
69 | 69 | /** |
@@ -71,7 +71,7 @@ discard block |
||
71 | 71 | */ |
72 | 72 | public function decreasePostCounts( Review $review ) |
73 | 73 | { |
74 | - if( empty( $counts = $this->getPostCounts( $review->assigned_to )))return; |
|
74 | + if( empty($counts = $this->getPostCounts( $review->assigned_to )) )return; |
|
75 | 75 | $counts = $this->decreaseRating( $counts, $review->review_type, $review->rating ); |
76 | 76 | $this->setPostCounts( $review->assigned_to, $counts ); |
77 | 77 | } |
@@ -82,7 +82,7 @@ discard block |
||
82 | 82 | public function decreaseTermCounts( Review $review ) |
83 | 83 | { |
84 | 84 | foreach( $review->term_ids as $termId ) { |
85 | - if( empty( $counts = $this->getTermCounts( $termId )))continue; |
|
85 | + if( empty($counts = $this->getTermCounts( $termId )) )continue; |
|
86 | 86 | $counts = $this->decreaseRating( $counts, $review->review_type, $review->rating ); |
87 | 87 | $this->setTermCounts( $termId, $counts ); |
88 | 88 | } |
@@ -94,8 +94,8 @@ discard block |
||
94 | 94 | public function flatten( array $reviewCounts ) |
95 | 95 | { |
96 | 96 | $counts = []; |
97 | - array_walk_recursive( $reviewCounts, function( $num, $index ) use( &$counts ) { |
|
98 | - $counts[$index] = isset( $counts[$index] ) |
|
97 | + array_walk_recursive( $reviewCounts, function( $num, $index ) use(&$counts) { |
|
98 | + $counts[$index] = isset($counts[$index]) |
|
99 | 99 | ? $num + $counts[$index] |
100 | 100 | : $num; |
101 | 101 | }); |
@@ -143,14 +143,14 @@ discard block |
||
143 | 143 | */ |
144 | 144 | public function increaseCounts( Review $review ) |
145 | 145 | { |
146 | - if( empty( $counts = $this->getCounts() )) { |
|
146 | + if( empty($counts = $this->getCounts()) ) { |
|
147 | 147 | $counts = $this->buildCounts(); |
148 | 148 | } |
149 | 149 | $this->setCounts( $this->increaseRating( |
150 | 150 | $counts, |
151 | 151 | $review->review_type, |
152 | 152 | $review->rating |
153 | - )); |
|
153 | + ) ); |
|
154 | 154 | } |
155 | 155 | |
156 | 156 | /** |
@@ -158,8 +158,8 @@ discard block |
||
158 | 158 | */ |
159 | 159 | public function increasePostCounts( Review $review ) |
160 | 160 | { |
161 | - if( !( get_post( $review->assigned_to ) instanceof WP_Post ))return; |
|
162 | - if( empty( $counts = $this->getPostCounts( $review->assigned_to ))) { |
|
161 | + if( !(get_post( $review->assigned_to ) instanceof WP_Post) )return; |
|
162 | + if( empty($counts = $this->getPostCounts( $review->assigned_to )) ) { |
|
163 | 163 | $counts = $this->buildPostCounts( $review->assigned_to ); |
164 | 164 | } |
165 | 165 | $counts = $this->increaseRating( $counts, $review->review_type, $review->rating ); |
@@ -172,8 +172,8 @@ discard block |
||
172 | 172 | public function increaseTermCounts( Review $review ) |
173 | 173 | { |
174 | 174 | foreach( $review->term_ids as $termId ) { |
175 | - if( !( get_term( $termId, Application::TAXONOMY ) instanceof WP_Term ))continue; |
|
176 | - if( empty( $counts = $this->getTermCounts( $termId ))) { |
|
175 | + if( !(get_term( $termId, Application::TAXONOMY ) instanceof WP_Term) )continue; |
|
176 | + if( empty($counts = $this->getTermCounts( $termId )) ) { |
|
177 | 177 | $counts = $this->buildTermCounts( $termId ); |
178 | 178 | } |
179 | 179 | $counts = $this->increaseRating( $counts, $review->review_type, $review->rating ); |
@@ -197,8 +197,8 @@ discard block |
||
197 | 197 | { |
198 | 198 | $ratingCounts = glsr( CountsManager::class )->flatten( $reviewCounts ); |
199 | 199 | update_post_meta( $postId, static::META_COUNT, $reviewCounts ); |
200 | - update_post_meta( $postId, static::META_AVERAGE, glsr( Rating::class )->getAverage( $ratingCounts )); |
|
201 | - update_post_meta( $postId, static::META_RANKING, glsr( Rating::class )->getRanking( $ratingCounts )); |
|
200 | + update_post_meta( $postId, static::META_AVERAGE, glsr( Rating::class )->getAverage( $ratingCounts ) ); |
|
201 | + update_post_meta( $postId, static::META_RANKING, glsr( Rating::class )->getRanking( $ratingCounts ) ); |
|
202 | 202 | } |
203 | 203 | |
204 | 204 | /** |
@@ -209,8 +209,8 @@ discard block |
||
209 | 209 | { |
210 | 210 | $ratingCounts = glsr( CountsManager::class )->flatten( $reviewCounts ); |
211 | 211 | update_term_meta( $termId, static::META_COUNT, $reviewCounts ); |
212 | - update_term_meta( $termId, static::META_AVERAGE, glsr( Rating::class )->getAverage( $ratingCounts )); |
|
213 | - update_term_meta( $termId, static::META_RANKING, glsr( Rating::class )->getRanking( $ratingCounts )); |
|
212 | + update_term_meta( $termId, static::META_AVERAGE, glsr( Rating::class )->getAverage( $ratingCounts ) ); |
|
213 | + update_term_meta( $termId, static::META_RANKING, glsr( Rating::class )->getRanking( $ratingCounts ) ); |
|
214 | 214 | } |
215 | 215 | |
216 | 216 | /** |
@@ -221,10 +221,10 @@ discard block |
||
221 | 221 | { |
222 | 222 | $counts = []; |
223 | 223 | $lastPostId = 0; |
224 | - while( $reviews = $this->queryReviews( $args, $lastPostId, $limit )) { |
|
225 | - $types = array_keys( array_flip( array_column( $reviews, 'type' ))); |
|
224 | + while( $reviews = $this->queryReviews( $args, $lastPostId, $limit ) ) { |
|
225 | + $types = array_keys( array_flip( array_column( $reviews, 'type' ) ) ); |
|
226 | 226 | foreach( $types as $type ) { |
227 | - if( isset( $counts[$type] ))continue; |
|
227 | + if( isset($counts[$type]) )continue; |
|
228 | 228 | $counts[$type] = array_fill_keys( range( 0, Rating::MAX_RATING ), 0 ); |
229 | 229 | } |
230 | 230 | foreach( $reviews as $review ) { |
@@ -242,7 +242,7 @@ discard block |
||
242 | 242 | */ |
243 | 243 | protected function decreaseRating( array $reviewCounts, $type, $rating ) |
244 | 244 | { |
245 | - if( isset( $reviewCounts[$type][$rating] )) { |
|
245 | + if( isset($reviewCounts[$type][$rating]) ) { |
|
246 | 246 | $reviewCounts[$type][$rating] = max( 0, $reviewCounts[$type][$rating] - 1 ); |
247 | 247 | } |
248 | 248 | return $reviewCounts; |
@@ -255,10 +255,10 @@ discard block |
||
255 | 255 | */ |
256 | 256 | protected function increaseRating( array $reviewCounts, $type, $rating ) |
257 | 257 | { |
258 | - if( !array_key_exists( $type, glsr()->reviewTypes )) { |
|
258 | + if( !array_key_exists( $type, glsr()->reviewTypes ) ) { |
|
259 | 259 | return $reviewCounts; |
260 | 260 | } |
261 | - if( !array_key_exists( $type, $reviewCounts )) { |
|
261 | + if( !array_key_exists( $type, $reviewCounts ) ) { |
|
262 | 262 | $reviewCounts[$type] = []; |
263 | 263 | } |
264 | 264 | $reviewCounts = $this->normalize( $reviewCounts ); |
@@ -273,7 +273,7 @@ discard block |
||
273 | 273 | { |
274 | 274 | foreach( $reviewCounts as &$counts ) { |
275 | 275 | foreach( range( 0, Rating::MAX_RATING ) as $index ) { |
276 | - if( isset( $counts[$index] ))continue; |
|
276 | + if( isset($counts[$index]) )continue; |
|
277 | 277 | $counts[$index] = 0; |
278 | 278 | } |
279 | 279 | ksort( $counts ); |
@@ -288,14 +288,14 @@ discard block |
||
288 | 288 | */ |
289 | 289 | protected function queryReviews( array $args = [], $lastPostId, $limit ) |
290 | 290 | { |
291 | - $args = wp_parse_args( $args, array_fill_keys( ['post_id', 'term_id'], '' )); |
|
292 | - if( empty( array_filter( $args ))) { |
|
291 | + $args = wp_parse_args( $args, array_fill_keys( ['post_id', 'term_id'], '' ) ); |
|
292 | + if( empty(array_filter( $args )) ) { |
|
293 | 293 | return glsr( SqlQueries::class )->getReviewCounts( $lastPostId, $limit ); |
294 | 294 | } |
295 | - if( !empty( $args['post_id'] )) { |
|
295 | + if( !empty($args['post_id']) ) { |
|
296 | 296 | return glsr( SqlQueries::class )->getReviewPostCounts( $args['post_id'], $lastPostId, $limit ); |
297 | 297 | } |
298 | - if( !empty( $args['term_id'] )) { |
|
298 | + if( !empty($args['term_id']) ) { |
|
299 | 299 | return glsr( SqlQueries::class )->getReviewTermCounts( $args['term_id'], $lastPostId, $limit ); |
300 | 300 | } |
301 | 301 | } |
@@ -71,7 +71,9 @@ discard block |
||
71 | 71 | */ |
72 | 72 | public function decreasePostCounts( Review $review ) |
73 | 73 | { |
74 | - if( empty( $counts = $this->getPostCounts( $review->assigned_to )))return; |
|
74 | + if( empty( $counts = $this->getPostCounts( $review->assigned_to ))) { |
|
75 | + return; |
|
76 | + } |
|
75 | 77 | $counts = $this->decreaseRating( $counts, $review->review_type, $review->rating ); |
76 | 78 | $this->setPostCounts( $review->assigned_to, $counts ); |
77 | 79 | } |
@@ -82,7 +84,9 @@ discard block |
||
82 | 84 | public function decreaseTermCounts( Review $review ) |
83 | 85 | { |
84 | 86 | foreach( $review->term_ids as $termId ) { |
85 | - if( empty( $counts = $this->getTermCounts( $termId )))continue; |
|
87 | + if( empty( $counts = $this->getTermCounts( $termId ))) { |
|
88 | + continue; |
|
89 | + } |
|
86 | 90 | $counts = $this->decreaseRating( $counts, $review->review_type, $review->rating ); |
87 | 91 | $this->setTermCounts( $termId, $counts ); |
88 | 92 | } |
@@ -158,7 +162,9 @@ discard block |
||
158 | 162 | */ |
159 | 163 | public function increasePostCounts( Review $review ) |
160 | 164 | { |
161 | - if( !( get_post( $review->assigned_to ) instanceof WP_Post ))return; |
|
165 | + if( !( get_post( $review->assigned_to ) instanceof WP_Post )) { |
|
166 | + return; |
|
167 | + } |
|
162 | 168 | if( empty( $counts = $this->getPostCounts( $review->assigned_to ))) { |
163 | 169 | $counts = $this->buildPostCounts( $review->assigned_to ); |
164 | 170 | } |
@@ -172,7 +178,9 @@ discard block |
||
172 | 178 | public function increaseTermCounts( Review $review ) |
173 | 179 | { |
174 | 180 | foreach( $review->term_ids as $termId ) { |
175 | - if( !( get_term( $termId, Application::TAXONOMY ) instanceof WP_Term ))continue; |
|
181 | + if( !( get_term( $termId, Application::TAXONOMY ) instanceof WP_Term )) { |
|
182 | + continue; |
|
183 | + } |
|
176 | 184 | if( empty( $counts = $this->getTermCounts( $termId ))) { |
177 | 185 | $counts = $this->buildTermCounts( $termId ); |
178 | 186 | } |
@@ -224,7 +232,9 @@ discard block |
||
224 | 232 | while( $reviews = $this->queryReviews( $args, $lastPostId, $limit )) { |
225 | 233 | $types = array_keys( array_flip( array_column( $reviews, 'type' ))); |
226 | 234 | foreach( $types as $type ) { |
227 | - if( isset( $counts[$type] ))continue; |
|
235 | + if( isset( $counts[$type] )) { |
|
236 | + continue; |
|
237 | + } |
|
228 | 238 | $counts[$type] = array_fill_keys( range( 0, Rating::MAX_RATING ), 0 ); |
229 | 239 | } |
230 | 240 | foreach( $reviews as $review ) { |
@@ -273,7 +283,9 @@ discard block |
||
273 | 283 | { |
274 | 284 | foreach( $reviewCounts as &$counts ) { |
275 | 285 | foreach( range( 0, Rating::MAX_RATING ) as $index ) { |
276 | - if( isset( $counts[$index] ))continue; |
|
286 | + if( isset( $counts[$index] )) { |
|
287 | + continue; |
|
288 | + } |
|
277 | 289 | $counts[$index] = 0; |
278 | 290 | } |
279 | 291 | ksort( $counts ); |
@@ -22,8 +22,8 @@ discard block |
||
22 | 22 | */ |
23 | 23 | public function onAfterChangeCategory( $postId, $terms, $termIds, $taxonomySlug, $append, $oldTermIds ) |
24 | 24 | { |
25 | - if( !$this->isReviewPostType( $postId ))return; |
|
26 | - $review = glsr( ReviewManager::class )->single( get_post( $postId )); |
|
25 | + if( !$this->isReviewPostType( $postId ) )return; |
|
26 | + $review = glsr( ReviewManager::class )->single( get_post( $postId ) ); |
|
27 | 27 | $ignoredTerms = array_intersect( $oldTermIds, $termIds ); |
28 | 28 | $review->term_ids = array_diff( $oldTermIds, $ignoredTerms ); |
29 | 29 | glsr( CountsManager::class )->decreaseTermCounts( $review ); |
@@ -40,8 +40,8 @@ discard block |
||
40 | 40 | */ |
41 | 41 | public function onAfterCreate( $postData, $meta, $postId ) |
42 | 42 | { |
43 | - if( !$this->isReviewPostType( $postId ))return; |
|
44 | - $review = glsr( ReviewManager::class )->single( get_post( $postId )); |
|
43 | + if( !$this->isReviewPostType( $postId ) )return; |
|
44 | + $review = glsr( ReviewManager::class )->single( get_post( $postId ) ); |
|
45 | 45 | glsr( CountsManager::class )->increase( $review ); |
46 | 46 | } |
47 | 47 | |
@@ -52,8 +52,8 @@ discard block |
||
52 | 52 | */ |
53 | 53 | public function onBeforeDelete( $postId ) |
54 | 54 | { |
55 | - if( !$this->isReviewPostType( $postId ))return; |
|
56 | - $review = glsr( ReviewManager::class )->single( get_post( $postId )); |
|
55 | + if( !$this->isReviewPostType( $postId ) )return; |
|
56 | + $review = glsr( ReviewManager::class )->single( get_post( $postId ) ); |
|
57 | 57 | glsr( CountsManager::class )->decrease( $review ); |
58 | 58 | } |
59 | 59 | |
@@ -70,7 +70,7 @@ discard block |
||
70 | 70 | if( !$this->isReviewPostType( $postId ) |
71 | 71 | || !in_array( $metaKey, ['assigned_to', 'rating', 'review_type'] ) |
72 | 72 | )return; |
73 | - $review = glsr( ReviewManager::class )->single( get_post( $postId )); |
|
73 | + $review = glsr( ReviewManager::class )->single( get_post( $postId ) ); |
|
74 | 74 | if( $review->$metaKey == $metaValue )return; |
75 | 75 | $method = glsr( Helper::class )->buildMethodName( $metaKey, 'onBeforeChange' ); |
76 | 76 | call_user_func( [$this, $method], $review, $metaValue ); |
@@ -96,8 +96,8 @@ discard block |
||
96 | 96 | */ |
97 | 97 | public function onChangeStatus( $newStatus, $oldStatus, WP_Post $post ) |
98 | 98 | { |
99 | - if( $post->post_type != Application::POST_TYPE || in_array( $oldStatus, ['new', $newStatus] ))return; |
|
100 | - $review = glsr( ReviewManager::class )->single( get_post( $postId )); |
|
99 | + if( $post->post_type != Application::POST_TYPE || in_array( $oldStatus, ['new', $newStatus] ) )return; |
|
100 | + $review = glsr( ReviewManager::class )->single( get_post( $postId ) ); |
|
101 | 101 | if( $status == 'publish' ) { |
102 | 102 | glsr( CountsManager::class )->increase( $review ); |
103 | 103 | } |
@@ -22,7 +22,9 @@ discard block |
||
22 | 22 | */ |
23 | 23 | public function onAfterChangeCategory( $postId, $terms, $termIds, $taxonomySlug, $append, $oldTermIds ) |
24 | 24 | { |
25 | - if( !$this->isReviewPostType( $postId ))return; |
|
25 | + if( !$this->isReviewPostType( $postId )) { |
|
26 | + return; |
|
27 | + } |
|
26 | 28 | $review = glsr( ReviewManager::class )->single( get_post( $postId )); |
27 | 29 | $ignoredTerms = array_intersect( $oldTermIds, $termIds ); |
28 | 30 | $review->term_ids = array_diff( $oldTermIds, $ignoredTerms ); |
@@ -40,7 +42,9 @@ discard block |
||
40 | 42 | */ |
41 | 43 | public function onAfterCreate( $postData, $meta, $postId ) |
42 | 44 | { |
43 | - if( !$this->isReviewPostType( $postId ))return; |
|
45 | + if( !$this->isReviewPostType( $postId )) { |
|
46 | + return; |
|
47 | + } |
|
44 | 48 | $review = glsr( ReviewManager::class )->single( get_post( $postId )); |
45 | 49 | glsr( CountsManager::class )->increase( $review ); |
46 | 50 | } |
@@ -52,7 +56,9 @@ discard block |
||
52 | 56 | */ |
53 | 57 | public function onBeforeDelete( $postId ) |
54 | 58 | { |
55 | - if( !$this->isReviewPostType( $postId ))return; |
|
59 | + if( !$this->isReviewPostType( $postId )) { |
|
60 | + return; |
|
61 | + } |
|
56 | 62 | $review = glsr( ReviewManager::class )->single( get_post( $postId )); |
57 | 63 | glsr( CountsManager::class )->decrease( $review ); |
58 | 64 | } |
@@ -69,9 +75,13 @@ discard block |
||
69 | 75 | { |
70 | 76 | if( !$this->isReviewPostType( $postId ) |
71 | 77 | || !in_array( $metaKey, ['assigned_to', 'rating', 'review_type'] ) |
72 | - )return; |
|
78 | + ) { |
|
79 | + return; |
|
80 | + } |
|
73 | 81 | $review = glsr( ReviewManager::class )->single( get_post( $postId )); |
74 | - if( $review->$metaKey == $metaValue )return; |
|
82 | + if( $review->$metaKey == $metaValue ) { |
|
83 | + return; |
|
84 | + } |
|
75 | 85 | $method = glsr( Helper::class )->buildMethodName( $metaKey, 'onBeforeChange' ); |
76 | 86 | call_user_func( [$this, $method], $review, $metaValue ); |
77 | 87 | } |
@@ -96,7 +106,9 @@ discard block |
||
96 | 106 | */ |
97 | 107 | public function onChangeStatus( $newStatus, $oldStatus, WP_Post $post ) |
98 | 108 | { |
99 | - if( $post->post_type != Application::POST_TYPE || in_array( $oldStatus, ['new', $newStatus] ))return; |
|
109 | + if( $post->post_type != Application::POST_TYPE || in_array( $oldStatus, ['new', $newStatus] )) { |
|
110 | + return; |
|
111 | + } |
|
100 | 112 | $review = glsr( ReviewManager::class )->single( get_post( $postId )); |
101 | 113 | if( $status == 'publish' ) { |
102 | 114 | glsr( CountsManager::class )->increase( $review ); |
@@ -13,10 +13,10 @@ 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; |
|
17 | - $assignedTo = strval( glsr( Helper::class )->filterInput( 'assigned_to' )); |
|
16 | + if( !wp_verify_nonce( glsr( Helper::class )->filterInput( '_nonce-assigned-to' ), 'assigned_to' ) )return; |
|
17 | + $assignedTo = strval( glsr( Helper::class )->filterInput( 'assigned_to' ) ); |
|
18 | 18 | if( get_post_meta( $postId, 'assigned_to', true ) != $assignedTo ) { |
19 | - $review = glsr( ReviewManager::class )->single( get_post( $postId )); |
|
19 | + $review = glsr( ReviewManager::class )->single( get_post( $postId ) ); |
|
20 | 20 | glsr( CountsManager::class )->decreasePostCounts( $review ); |
21 | 21 | } |
22 | 22 | update_post_meta( $postId, 'assigned_to', $assignedTo ); |
@@ -28,12 +28,12 @@ discard block |
||
28 | 28 | */ |
29 | 29 | public function saveResponseMetabox( $postId ) |
30 | 30 | { |
31 | - if( !wp_verify_nonce( glsr( Helper::class )->filterInput( '_nonce-response' ), 'response' ))return; |
|
32 | - $response = strval( glsr( Helper::class )->filterInput( 'response' ))c; |
|
31 | + if( !wp_verify_nonce( glsr( Helper::class )->filterInput( '_nonce-response' ), 'response' ) )return; |
|
32 | + $response = strval( glsr( Helper::class )->filterInput( 'response' ) )c; |
|
33 | 33 | update_post_meta( $postId, 'response', trim( wp_kses( $response, [ |
34 | 34 | 'a' => ['href' => [], 'title' => []], |
35 | 35 | 'em' => [], |
36 | 36 | 'strong' => [], |
37 | - ]))); |
|
37 | + ] ) ) ); |
|
38 | 38 | } |
39 | 39 | } |
@@ -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' ))c; |
33 | 37 | update_post_meta( $postId, 'response', trim( wp_kses( $response, [ |
34 | 38 | 'a' => ['href' => [], 'title' => []], |