@@ -16,7 +16,7 @@ discard block |
||
16 | 16 | */ |
17 | 17 | public function callbackRegisterSettings( $input ) |
18 | 18 | { |
19 | - if( !is_array( $input )) { |
|
19 | + if( !is_array( $input ) ) { |
|
20 | 20 | $input = ['settings' => []]; |
21 | 21 | } |
22 | 22 | if( key( $input ) == 'settings' ) { |
@@ -24,7 +24,7 @@ discard block |
||
24 | 24 | $options = $this->sanitizeGeneral( $input, $options ); |
25 | 25 | $options = $this->sanitizeSubmissions( $input, $options ); |
26 | 26 | $options = $this->sanitizeTranslations( $input, $options ); |
27 | - glsr( Notice::class )->addSuccess( __( 'Settings updated.', 'site-reviews' )); |
|
27 | + glsr( Notice::class )->addSuccess( __( 'Settings updated.', 'site-reviews' ) ); |
|
28 | 28 | return $options; |
29 | 29 | } |
30 | 30 | return $input; |
@@ -38,7 +38,7 @@ discard block |
||
38 | 38 | { |
39 | 39 | register_setting( Application::ID.'-settings', OptionManager::databaseKey(), [ |
40 | 40 | 'sanitize_callback' => [$this, 'callbackRegisterSettings'], |
41 | - ]); |
|
41 | + ] ); |
|
42 | 42 | } |
43 | 43 | |
44 | 44 | /** |
@@ -58,7 +58,7 @@ discard block |
||
58 | 58 | protected function sanitizeSubmissions( array $input, array $options ) |
59 | 59 | { |
60 | 60 | $inputForm = $input['settings']['submissions']; |
61 | - $options['settings']['submissions']['required'] = isset( $inputForm['required'] ) |
|
61 | + $options['settings']['submissions']['required'] = isset($inputForm['required']) |
|
62 | 62 | ? $inputForm['required'] |
63 | 63 | : []; |
64 | 64 | return $options; |
@@ -69,13 +69,13 @@ discard block |
||
69 | 69 | */ |
70 | 70 | protected function sanitizeTranslations( array $input, array $options ) |
71 | 71 | { |
72 | - if( isset( $input['settings']['strings'] )) { |
|
73 | - $options['settings']['strings'] = array_values( array_filter( $input['settings']['strings'] )); |
|
72 | + if( isset($input['settings']['strings']) ) { |
|
73 | + $options['settings']['strings'] = array_values( array_filter( $input['settings']['strings'] ) ); |
|
74 | 74 | array_walk( $options['settings']['strings'], function( &$string ) { |
75 | - if( isset( $string['s2'] )) { |
|
75 | + if( isset($string['s2']) ) { |
|
76 | 76 | $string['s2'] = wp_strip_all_tags( $string['s2'] ); |
77 | 77 | } |
78 | - if( isset( $string['p2'] )) { |
|
78 | + if( isset($string['p2']) ) { |
|
79 | 79 | $string['p2'] = wp_strip_all_tags( $string['p2'] ); |
80 | 80 | } |
81 | 81 | }); |
@@ -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 | } |
@@ -60,7 +60,7 @@ discard block |
||
60 | 60 | $this->getCounts(), |
61 | 61 | $review->review_type, |
62 | 62 | $review->rating |
63 | - )); |
|
63 | + ) ); |
|
64 | 64 | } |
65 | 65 | |
66 | 66 | /** |
@@ -68,12 +68,12 @@ discard block |
||
68 | 68 | */ |
69 | 69 | public function decreasePostCounts( Review $review ) |
70 | 70 | { |
71 | - if( empty( $counts = $this->getPostCounts( $review->assigned_to )))return; |
|
71 | + if( empty($counts = $this->getPostCounts( $review->assigned_to )) )return; |
|
72 | 72 | $this->setPostCounts( $review->assigned_to, $this->decreaseRating( |
73 | 73 | $counts, |
74 | 74 | $review->review_type, |
75 | 75 | $review->rating |
76 | - )); |
|
76 | + ) ); |
|
77 | 77 | } |
78 | 78 | |
79 | 79 | /** |
@@ -82,12 +82,12 @@ 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 | $this->setTermCounts( $termId, $this->decreaseRating( |
87 | 87 | $counts, |
88 | 88 | $review->review_type, |
89 | 89 | $review->rating |
90 | - )); |
|
90 | + ) ); |
|
91 | 91 | } |
92 | 92 | } |
93 | 93 | |
@@ -97,8 +97,8 @@ discard block |
||
97 | 97 | public function flatten( array $reviewCounts ) |
98 | 98 | { |
99 | 99 | $counts = []; |
100 | - array_walk_recursive( $reviewCounts, function( $num, $index ) use( &$counts ) { |
|
101 | - $counts[$index] = isset( $counts[$index] ) |
|
100 | + array_walk_recursive( $reviewCounts, function( $num, $index ) use(&$counts) { |
|
101 | + $counts[$index] = isset($counts[$index]) |
|
102 | 102 | ? $num + $counts[$index] |
103 | 103 | : $num; |
104 | 104 | }); |
@@ -146,14 +146,14 @@ discard block |
||
146 | 146 | */ |
147 | 147 | public function increaseCounts( Review $review ) |
148 | 148 | { |
149 | - if( empty( $counts = $this->getCounts() )) { |
|
149 | + if( empty($counts = $this->getCounts()) ) { |
|
150 | 150 | $counts = $this->buildCounts(); |
151 | 151 | } |
152 | 152 | $this->setCounts( $this->increaseRating( |
153 | 153 | $counts, |
154 | 154 | $review->review_type, |
155 | 155 | $review->rating |
156 | - )); |
|
156 | + ) ); |
|
157 | 157 | } |
158 | 158 | |
159 | 159 | /** |
@@ -161,14 +161,14 @@ discard block |
||
161 | 161 | */ |
162 | 162 | public function increasePostCounts( Review $review ) |
163 | 163 | { |
164 | - if( empty( $counts = $this->getPostCounts( $review->assigned_to ))) { |
|
164 | + if( empty($counts = $this->getPostCounts( $review->assigned_to )) ) { |
|
165 | 165 | $counts = $this->buildPostCounts( $review->assigned_to ); |
166 | 166 | } |
167 | 167 | $this->setPostCounts( $review->assigned_to, $this->increaseRating( |
168 | 168 | $counts, |
169 | 169 | $review->review_type, |
170 | 170 | $review->rating |
171 | - )); |
|
171 | + ) ); |
|
172 | 172 | } |
173 | 173 | |
174 | 174 | /** |
@@ -177,14 +177,14 @@ discard block |
||
177 | 177 | public function increaseTermCounts( Review $review ) |
178 | 178 | { |
179 | 179 | foreach( $review->term_ids as $termId ) { |
180 | - if( empty( $counts = $this->getTermCounts( $termId ))) { |
|
180 | + if( empty($counts = $this->getTermCounts( $termId )) ) { |
|
181 | 181 | $counts = $this->buildTermCounts( $termId ); |
182 | 182 | } |
183 | 183 | $this->setTermCounts( $termId, $this->increaseRating( |
184 | 184 | $counts, |
185 | 185 | $review->review_type, |
186 | 186 | $review->rating |
187 | - )); |
|
187 | + ) ); |
|
188 | 188 | } |
189 | 189 | } |
190 | 190 | |
@@ -222,10 +222,10 @@ discard block |
||
222 | 222 | { |
223 | 223 | $counts = []; |
224 | 224 | $lastPostId = 0; |
225 | - while( $reviews = $this->queryReviews( $args, $lastPostId, $limit )) { |
|
226 | - $types = array_keys( array_flip( array_column( $reviews, 'type' ))); |
|
225 | + while( $reviews = $this->queryReviews( $args, $lastPostId, $limit ) ) { |
|
226 | + $types = array_keys( array_flip( array_column( $reviews, 'type' ) ) ); |
|
227 | 227 | foreach( $types as $type ) { |
228 | - if( isset( $counts[$type] ))continue; |
|
228 | + if( isset($counts[$type]) )continue; |
|
229 | 229 | $counts[$type] = array_fill_keys( range( 0, Rating::MAX_RATING ), 0 ); |
230 | 230 | } |
231 | 231 | foreach( $reviews as $review ) { |
@@ -243,7 +243,7 @@ discard block |
||
243 | 243 | */ |
244 | 244 | protected function decreaseRating( array $reviewCounts, $type, $rating ) |
245 | 245 | { |
246 | - if( isset( $reviewCounts[$type][$rating] )) { |
|
246 | + if( isset($reviewCounts[$type][$rating]) ) { |
|
247 | 247 | $reviewCounts[$type][$rating] = max( 0, $reviewCounts[$type][$rating] - 1 ); |
248 | 248 | } |
249 | 249 | return $reviewCounts; |
@@ -256,10 +256,10 @@ discard block |
||
256 | 256 | */ |
257 | 257 | protected function increaseRating( array $reviewCounts, $type, $rating ) |
258 | 258 | { |
259 | - if( !array_key_exists( $type, glsr()->reviewTypes )) { |
|
259 | + if( !array_key_exists( $type, glsr()->reviewTypes ) ) { |
|
260 | 260 | return $reviewCounts; |
261 | 261 | } |
262 | - if( !array_key_exists( $type, $reviewCounts )) { |
|
262 | + if( !array_key_exists( $type, $reviewCounts ) ) { |
|
263 | 263 | $reviewCounts[$type] = []; |
264 | 264 | } |
265 | 265 | $reviewCounts = $this->normalize( $reviewCounts ); |
@@ -274,7 +274,7 @@ discard block |
||
274 | 274 | { |
275 | 275 | foreach( $reviewCounts as &$counts ) { |
276 | 276 | foreach( range( 0, Rating::MAX_RATING ) as $index ) { |
277 | - if( isset( $counts[$index] ))continue; |
|
277 | + if( isset($counts[$index]) )continue; |
|
278 | 278 | $counts[$index] = 0; |
279 | 279 | } |
280 | 280 | ksort( $counts ); |
@@ -289,14 +289,14 @@ discard block |
||
289 | 289 | */ |
290 | 290 | protected function queryReviews( array $args = [], $lastPostId, $limit ) |
291 | 291 | { |
292 | - $args = wp_parse_args( $args, array_fill_keys( ['post_id', 'term_id'], '' )); |
|
293 | - if( empty( array_filter( $args ))) { |
|
292 | + $args = wp_parse_args( $args, array_fill_keys( ['post_id', 'term_id'], '' ) ); |
|
293 | + if( empty(array_filter( $args )) ) { |
|
294 | 294 | return glsr( SqlQueries::class )->getReviewCounts( $lastPostId, $limit ); |
295 | 295 | } |
296 | - if( !empty( $args['post_id'] )) { |
|
296 | + if( !empty($args['post_id']) ) { |
|
297 | 297 | return glsr( SqlQueries::class )->getReviewPostCounts( $args['post_id'], $lastPostId, $limit ); |
298 | 298 | } |
299 | - if( !empty( $args['term_id'] )) { |
|
299 | + if( !empty($args['term_id']) ) { |
|
300 | 300 | return glsr( SqlQueries::class )->getReviewTermCounts( $args['term_id'], $lastPostId, $limit ); |
301 | 301 | } |
302 | 302 | } |
@@ -68,7 +68,9 @@ discard block |
||
68 | 68 | */ |
69 | 69 | public function decreasePostCounts( Review $review ) |
70 | 70 | { |
71 | - if( empty( $counts = $this->getPostCounts( $review->assigned_to )))return; |
|
71 | + if( empty( $counts = $this->getPostCounts( $review->assigned_to ))) { |
|
72 | + return; |
|
73 | + } |
|
72 | 74 | $this->setPostCounts( $review->assigned_to, $this->decreaseRating( |
73 | 75 | $counts, |
74 | 76 | $review->review_type, |
@@ -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 | $this->setTermCounts( $termId, $this->decreaseRating( |
87 | 91 | $counts, |
88 | 92 | $review->review_type, |
@@ -225,7 +229,9 @@ discard block |
||
225 | 229 | while( $reviews = $this->queryReviews( $args, $lastPostId, $limit )) { |
226 | 230 | $types = array_keys( array_flip( array_column( $reviews, 'type' ))); |
227 | 231 | foreach( $types as $type ) { |
228 | - if( isset( $counts[$type] ))continue; |
|
232 | + if( isset( $counts[$type] )) { |
|
233 | + continue; |
|
234 | + } |
|
229 | 235 | $counts[$type] = array_fill_keys( range( 0, Rating::MAX_RATING ), 0 ); |
230 | 236 | } |
231 | 237 | foreach( $reviews as $review ) { |
@@ -274,7 +280,9 @@ discard block |
||
274 | 280 | { |
275 | 281 | foreach( $reviewCounts as &$counts ) { |
276 | 282 | foreach( range( 0, Rating::MAX_RATING ) as $index ) { |
277 | - if( isset( $counts[$index] ))continue; |
|
283 | + if( isset( $counts[$index] )) { |
|
284 | + continue; |
|
285 | + } |
|
278 | 286 | $counts[$index] = 0; |
279 | 287 | } |
280 | 288 | ksort( $counts ); |