@@ -80,7 +80,9 @@ discard block |
||
80 | 80 | ]; |
81 | 81 | if( !isset( $properties[$property] ) |
82 | 82 | || empty( array_filter( [$value], $properties[$property] )) |
83 | - )return; |
|
83 | + ) { |
|
84 | + return; |
|
85 | + } |
|
84 | 86 | $this->$property = $value; |
85 | 87 | } |
86 | 88 | |
@@ -129,7 +131,9 @@ discard block |
||
129 | 131 | */ |
130 | 132 | protected function buildFieldDescription() |
131 | 133 | { |
132 | - if( empty( $this->args['description'] ))return; |
|
134 | + if( empty( $this->args['description'] )) { |
|
135 | + return; |
|
136 | + } |
|
133 | 137 | if( $this->args['is_widget'] ) { |
134 | 138 | return $this->small( $this->args['description'] ); |
135 | 139 | } |
@@ -184,7 +188,9 @@ discard block |
||
184 | 188 | */ |
185 | 189 | protected function buildFormLabel() |
186 | 190 | { |
187 | - if( empty( $this->args['label'] ) || $this->args['type'] == 'hidden' )return; |
|
191 | + if( empty( $this->args['label'] ) || $this->args['type'] == 'hidden' ) { |
|
192 | + return; |
|
193 | + } |
|
188 | 194 | return $this->label([ |
189 | 195 | 'for' => $this->args['id'], |
190 | 196 | 'text' => $this->args['label'], |
@@ -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 | $generatedTag = in_array( $instance->tag, $tags ) |
61 | 61 | ? $instance->buildTag() |
@@ -78,8 +78,8 @@ discard block |
||
78 | 78 | 'render' => 'is_bool', |
79 | 79 | 'tag' => 'is_string', |
80 | 80 | ]; |
81 | - if( !isset( $properties[$property] ) |
|
82 | - || empty( array_filter( [$value], $properties[$property] )) |
|
81 | + if( !isset($properties[$property]) |
|
82 | + || empty(array_filter( [$value], $properties[$property] )) |
|
83 | 83 | )return; |
84 | 84 | $this->$property = $value; |
85 | 85 | } |
@@ -97,7 +97,7 @@ discard block |
||
97 | 97 | */ |
98 | 98 | public function getOpeningTag() |
99 | 99 | { |
100 | - $attributes = glsr( Attributes::class )->{$this->tag}( $this->args )->toString(); |
|
100 | + $attributes = glsr( Attributes::class )->{$this->tag}($this->args)->toString(); |
|
101 | 101 | return '<'.trim( $this->tag.' '.$attributes ).'>'; |
102 | 102 | } |
103 | 103 | |
@@ -107,7 +107,7 @@ discard block |
||
107 | 107 | protected function buildCustomField() |
108 | 108 | { |
109 | 109 | $className = $this->getCustomFieldClassName(); |
110 | - if( class_exists( $className )) { |
|
110 | + if( class_exists( $className ) ) { |
|
111 | 111 | return (new $className( $this ))->build(); |
112 | 112 | } |
113 | 113 | glsr_log()->error( 'Field missing: '.$className ); |
@@ -118,7 +118,7 @@ discard block |
||
118 | 118 | */ |
119 | 119 | protected function buildDefaultTag( $text = '' ) |
120 | 120 | { |
121 | - if( empty( $text )) { |
|
121 | + if( empty($text) ) { |
|
122 | 122 | $text = $this->args['text']; |
123 | 123 | } |
124 | 124 | return $this->getOpeningTag().$text.$this->getClosingTag(); |
@@ -129,7 +129,7 @@ discard block |
||
129 | 129 | */ |
130 | 130 | protected function buildFieldDescription() |
131 | 131 | { |
132 | - if( empty( $this->args['description'] ))return; |
|
132 | + if( empty($this->args['description']) )return; |
|
133 | 133 | if( $this->args['is_widget'] ) { |
134 | 134 | return $this->small( $this->args['description'] ); |
135 | 135 | } |
@@ -141,10 +141,10 @@ discard block |
||
141 | 141 | */ |
142 | 142 | protected function buildFormInput() |
143 | 143 | { |
144 | - if( !in_array( $this->args['type'], ['checkbox', 'radio'] )) { |
|
144 | + if( !in_array( $this->args['type'], ['checkbox', 'radio'] ) ) { |
|
145 | 145 | return $this->buildFormLabel().$this->getOpeningTag(); |
146 | 146 | } |
147 | - return empty( $this->args['options'] ) |
|
147 | + return empty($this->args['options']) |
|
148 | 148 | ? $this->buildFormInputChoice() |
149 | 149 | : $this->buildFormInputMultiChoice(); |
150 | 150 | } |
@@ -154,13 +154,13 @@ discard block |
||
154 | 154 | */ |
155 | 155 | protected function buildFormInputChoice() |
156 | 156 | { |
157 | - $labelText = !empty( $this->args['text'] ) |
|
157 | + $labelText = !empty($this->args['text']) |
|
158 | 158 | ? $this->args['text'] |
159 | 159 | : $this->args['label']; |
160 | 160 | return $this->label( $this->getOpeningTag().' '.$labelText, [ |
161 | 161 | 'class' => 'glsr-'.$this->args['type'].'-label', |
162 | 162 | 'for' => $this->args['id'], |
163 | - ]); |
|
163 | + ] ); |
|
164 | 164 | } |
165 | 165 | |
166 | 166 | /** |
@@ -169,7 +169,7 @@ discard block |
||
169 | 169 | protected function buildFormInputMultiChoice() |
170 | 170 | { |
171 | 171 | if( $this->args['type'] == 'checkbox' ) { |
172 | - $this->args['name'].= '[]'; |
|
172 | + $this->args['name'] .= '[]'; |
|
173 | 173 | } |
174 | 174 | $options = array_reduce( array_keys( $this->args['options'] ), function( $carry, $key ) { |
175 | 175 | return $carry.$this->li( $this->{$this->args['type']}([ |
@@ -177,12 +177,12 @@ discard block |
||
177 | 177 | 'name' => $this->args['name'], |
178 | 178 | 'text' => $this->args['options'][$key], |
179 | 179 | 'value' => $key, |
180 | - ])); |
|
180 | + ]) ); |
|
181 | 181 | }); |
182 | 182 | return $this->ul( $options, [ |
183 | 183 | 'class' => $this->args['class'], |
184 | 184 | 'id' => $this->args['id'], |
185 | - ]); |
|
185 | + ] ); |
|
186 | 186 | } |
187 | 187 | |
188 | 188 | /** |
@@ -190,11 +190,11 @@ discard block |
||
190 | 190 | */ |
191 | 191 | protected function buildFormLabel() |
192 | 192 | { |
193 | - if( empty( $this->args['label'] ) || $this->args['type'] == 'hidden' )return; |
|
194 | - return $this->label([ |
|
193 | + if( empty($this->args['label']) || $this->args['type'] == 'hidden' )return; |
|
194 | + return $this->label( [ |
|
195 | 195 | 'for' => $this->args['id'], |
196 | 196 | 'text' => $this->args['label'], |
197 | - ]); |
|
197 | + ] ); |
|
198 | 198 | } |
199 | 199 | |
200 | 200 | /** |
@@ -211,11 +211,11 @@ discard block |
||
211 | 211 | protected function buildFormSelectOptions() |
212 | 212 | { |
213 | 213 | return array_reduce( array_keys( $this->args['options'] ), function( $carry, $key ) { |
214 | - return $carry.$this->option([ |
|
214 | + return $carry.$this->option( [ |
|
215 | 215 | 'selected' => $this->args['value'] == $key, |
216 | 216 | 'text' => $this->args['options'][$key], |
217 | 217 | 'value' => $key, |
218 | - ]); |
|
218 | + ] ); |
|
219 | 219 | }); |
220 | 220 | } |
221 | 221 | |
@@ -233,10 +233,10 @@ discard block |
||
233 | 233 | protected function buildTag() |
234 | 234 | { |
235 | 235 | $this->mergeArgsWithRequiredDefaults(); |
236 | - if( in_array( $this->tag, static::TAGS_SINGLE )) { |
|
236 | + if( in_array( $this->tag, static::TAGS_SINGLE ) ) { |
|
237 | 237 | return $this->getOpeningTag(); |
238 | 238 | } |
239 | - if( !in_array( $this->tag, static::TAGS_FORM )) { |
|
239 | + if( !in_array( $this->tag, static::TAGS_FORM ) ) { |
|
240 | 240 | return $this->buildDefaultTag(); |
241 | 241 | } |
242 | 242 | return call_user_func( [$this, 'buildForm'.ucfirst( $this->tag )] ).$this->buildFieldDescription(); |
@@ -256,7 +256,7 @@ discard block |
||
256 | 256 | protected function mergeArgsWithRequiredDefaults() |
257 | 257 | { |
258 | 258 | $className = $this->getCustomFieldClassName(); |
259 | - if( class_exists( $className )) { |
|
259 | + if( class_exists( $className ) ) { |
|
260 | 260 | $this->args = array_merge( |
261 | 261 | wp_parse_args( $this->args, $className::defaults() ), |
262 | 262 | $className::required() |
@@ -271,13 +271,13 @@ discard block |
||
271 | 271 | */ |
272 | 272 | protected function normalize( ...$params ) |
273 | 273 | { |
274 | - if( is_string( $params[0] ) || is_numeric( $params[0] )) { |
|
274 | + if( is_string( $params[0] ) || is_numeric( $params[0] ) ) { |
|
275 | 275 | $this->setNameOrTextAttributeForTag( $params[0] ); |
276 | 276 | } |
277 | - if( is_array( $params[0] )) { |
|
277 | + if( is_array( $params[0] ) ) { |
|
278 | 278 | $this->args += $params[0]; |
279 | 279 | } |
280 | - else if( is_array( $params[1] )) { |
|
280 | + else if( is_array( $params[1] ) ) { |
|
281 | 281 | $this->args += $params[1]; |
282 | 282 | } |
283 | 283 | } |
@@ -301,7 +301,7 @@ discard block |
||
301 | 301 | protected function setTagFromMethod( $method ) |
302 | 302 | { |
303 | 303 | $this->tag = strtolower( $method ); |
304 | - if( in_array( $this->tag, static::INPUT_TYPES )) { |
|
304 | + if( in_array( $this->tag, static::INPUT_TYPES ) ) { |
|
305 | 305 | $this->args['type'] = $this->tag; |
306 | 306 | $this->tag = 'input'; |
307 | 307 | } |
@@ -44,7 +44,7 @@ discard block |
||
44 | 44 | glsr()->version, |
45 | 45 | true |
46 | 46 | ); |
47 | - if( !empty( $this->pointers )) { |
|
47 | + if( !empty($this->pointers) ) { |
|
48 | 48 | wp_enqueue_style( 'wp-pointer' ); |
49 | 49 | wp_enqueue_script( 'wp-pointer' ); |
50 | 50 | } |
@@ -84,7 +84,7 @@ discard block |
||
84 | 84 | $dependencies = apply_filters( 'site-reviews/enqueue/admin/dependencies', [] ); |
85 | 85 | $dependencies = array_merge( $dependencies, [ |
86 | 86 | 'jquery', 'jquery-ui-sortable', 'underscore', 'wp-util', |
87 | - ]); |
|
87 | + ] ); |
|
88 | 88 | return $dependencies; |
89 | 89 | } |
90 | 90 | |
@@ -114,7 +114,7 @@ discard block |
||
114 | 114 | $generatedPointers = []; |
115 | 115 | foreach( $pointers as $pointer ) { |
116 | 116 | if( $pointer['screen'] != glsr_current_screen()->id )continue; |
117 | - if( in_array( $pointer['id'], $dismissedPointers ))continue; |
|
117 | + if( in_array( $pointer['id'], $dismissedPointers ) )continue; |
|
118 | 118 | $generatedPointers[] = $this->generatePointer( $pointer ); |
119 | 119 | } |
120 | 120 | $this->pointers = $generatedPointers; |
@@ -126,7 +126,7 @@ discard block |
||
126 | 126 | protected function isCurrentScreen() |
127 | 127 | { |
128 | 128 | $screen = glsr_current_screen(); |
129 | - return $screen && ( $screen->post_type == Application::POST_TYPE |
|
129 | + return $screen && ($screen->post_type == Application::POST_TYPE |
|
130 | 130 | || $screen->base == 'post' |
131 | 131 | || $screen->id == 'dashboard' |
132 | 132 | || $screen->id == 'widgets' |
@@ -140,7 +140,7 @@ discard block |
||
140 | 140 | { |
141 | 141 | $variables = []; |
142 | 142 | foreach( glsr()->mceShortcodes as $tag => $args ) { |
143 | - if( empty( $args['required'] ))continue; |
|
143 | + if( empty($args['required']) )continue; |
|
144 | 144 | $variables[$tag] = $args['required']; |
145 | 145 | } |
146 | 146 | return $variables; |
@@ -36,7 +36,9 @@ discard block |
||
36 | 36 | [], |
37 | 37 | glsr()->version |
38 | 38 | ); |
39 | - if( !$this->isCurrentScreen() )return; |
|
39 | + if( !$this->isCurrentScreen() ) { |
|
40 | + return; |
|
41 | + } |
|
40 | 42 | wp_enqueue_script( |
41 | 43 | Application::ID, |
42 | 44 | glsr()->url( 'assets/scripts/'.Application::ID.'-admin.js' ), |
@@ -113,8 +115,12 @@ discard block |
||
113 | 115 | $dismissedPointers = explode( ',', (string)$dismissedPointers ); |
114 | 116 | $generatedPointers = []; |
115 | 117 | foreach( $pointers as $pointer ) { |
116 | - if( $pointer['screen'] != glsr_current_screen()->id )continue; |
|
117 | - if( in_array( $pointer['id'], $dismissedPointers ))continue; |
|
118 | + if( $pointer['screen'] != glsr_current_screen()->id ) { |
|
119 | + continue; |
|
120 | + } |
|
121 | + if( in_array( $pointer['id'], $dismissedPointers )) { |
|
122 | + continue; |
|
123 | + } |
|
118 | 124 | $generatedPointers[] = $this->generatePointer( $pointer ); |
119 | 125 | } |
120 | 126 | $this->pointers = $generatedPointers; |
@@ -140,7 +146,9 @@ discard block |
||
140 | 146 | { |
141 | 147 | $variables = []; |
142 | 148 | foreach( glsr()->mceShortcodes as $tag => $args ) { |
143 | - if( empty( $args['required'] ))continue; |
|
149 | + if( empty( $args['required'] )) { |
|
150 | + continue; |
|
151 | + } |
|
144 | 152 | $variables[$tag] = $args['required']; |
145 | 153 | } |
146 | 154 | return $variables; |
@@ -24,7 +24,7 @@ discard block |
||
24 | 24 | { |
25 | 25 | wp_send_json( glsr( Html::class )->renderPartial( 'link', [ |
26 | 26 | 'post_id' => $request['ID'], |
27 | - ])); |
|
27 | + ] ) ); |
|
28 | 28 | } |
29 | 29 | |
30 | 30 | /** |
@@ -32,7 +32,7 @@ discard block |
||
32 | 32 | */ |
33 | 33 | public function routerChangeReviewStatus( array $request ) |
34 | 34 | { |
35 | - wp_send_json( $this->execute( new ChangeStatus( $request ))); |
|
35 | + wp_send_json( $this->execute( new ChangeStatus( $request ) ) ); |
|
36 | 36 | } |
37 | 37 | |
38 | 38 | /** |
@@ -41,10 +41,10 @@ discard block |
||
41 | 41 | public function routerClearLog() |
42 | 42 | { |
43 | 43 | glsr( AdminController::class )->routerClearLog(); |
44 | - wp_send_json([ |
|
44 | + wp_send_json( [ |
|
45 | 45 | 'logger' => glsr( Logger::class )->get(), |
46 | 46 | 'notices' => glsr( Notice::class )->get(), |
47 | - ]); |
|
47 | + ] ); |
|
48 | 48 | } |
49 | 49 | |
50 | 50 | /** |
@@ -54,9 +54,9 @@ discard block |
||
54 | 54 | { |
55 | 55 | $shortcode = $request['shortcode']; |
56 | 56 | $response = false; |
57 | - if( array_key_exists( $shortcode, glsr()->mceShortcodes )) { |
|
57 | + if( array_key_exists( $shortcode, glsr()->mceShortcodes ) ) { |
|
58 | 58 | $data = glsr()->mceShortcodes[$shortcode]; |
59 | - if( !empty( $data['errors'] )) { |
|
59 | + if( !empty($data['errors']) ) { |
|
60 | 60 | $data['btn_okay'] = [esc_html__( 'Okay', 'site-reviews' )]; |
61 | 61 | } |
62 | 62 | $response = [ |
@@ -76,10 +76,10 @@ discard block |
||
76 | 76 | public function routerSearchPosts( array $request ) |
77 | 77 | { |
78 | 78 | $results = glsr( Database::class )->searchPosts( $request['search'] ); |
79 | - wp_send_json_success([ |
|
79 | + wp_send_json_success( [ |
|
80 | 80 | 'empty' => '<div>'.__( 'Nothing found.', 'site-reviews' ).'</div>', |
81 | 81 | 'items' => $results, |
82 | - ]); |
|
82 | + ] ); |
|
83 | 83 | } |
84 | 84 | |
85 | 85 | /** |
@@ -87,7 +87,7 @@ discard block |
||
87 | 87 | */ |
88 | 88 | public function routerSearchTranslations( array $request ) |
89 | 89 | { |
90 | - if( empty( $request['exclude'] )) { |
|
90 | + if( empty($request['exclude']) ) { |
|
91 | 91 | $request['exclude'] = []; |
92 | 92 | } |
93 | 93 | $results = glsr( Translator::class ) |
@@ -95,10 +95,10 @@ discard block |
||
95 | 95 | ->exclude() |
96 | 96 | ->exclude( $request['exclude'] ) |
97 | 97 | ->renderResults(); |
98 | - wp_send_json_success([ |
|
98 | + wp_send_json_success( [ |
|
99 | 99 | 'empty' => '<div>'.__( 'Nothing found.', 'site-reviews' ).'</div>', |
100 | 100 | 'items' => $results, |
101 | - ]); |
|
101 | + ] ); |
|
102 | 102 | } |
103 | 103 | |
104 | 104 | /** |
@@ -108,11 +108,11 @@ discard block |
||
108 | 108 | { |
109 | 109 | $response = glsr( PublicController::class )->routerCreateReview( $request ); |
110 | 110 | $session = glsr( Session::class ); |
111 | - wp_send_json([ |
|
111 | + wp_send_json( [ |
|
112 | 112 | 'errors' => $session->get( $request['form_id'].'-errors', false, true ), |
113 | 113 | 'message' => $response, |
114 | 114 | 'recaptcha' => $session->get( $request['form_id'].'-recaptcha', false, true ), |
115 | - ]); |
|
115 | + ] ); |
|
116 | 116 | } |
117 | 117 | |
118 | 118 | /** |
@@ -120,10 +120,10 @@ discard block |
||
120 | 120 | */ |
121 | 121 | public function routerTogglePinned( array $request ) |
122 | 122 | { |
123 | - $isPinned = $this->execute( new TogglePinned( $request )); |
|
124 | - wp_send_json([ |
|
123 | + $isPinned = $this->execute( new TogglePinned( $request ) ); |
|
124 | + wp_send_json( [ |
|
125 | 125 | 'notices' => glsr( Notice::class )->get(), |
126 | 126 | 'pinned' => $isPinned, |
127 | - ]); |
|
127 | + ] ); |
|
128 | 128 | } |
129 | 129 | } |
@@ -37,7 +37,7 @@ discard block |
||
37 | 37 | 'inactive-plugin' => 'Inactive Plugins', |
38 | 38 | 'setting' => 'Plugin Settings', |
39 | 39 | ]; |
40 | - $systemInfo = array_reduce( array_keys( $details ), function( $carry, $key ) use( $details ) { |
|
40 | + $systemInfo = array_reduce( array_keys( $details ), function( $carry, $key ) use($details) { |
|
41 | 41 | $methodName = glsr( Helper::class )->buildMethodName( 'get-'.$key.'-details' ); |
42 | 42 | if( method_exists( $this, $methodName ) && $systemDetails = $this->$methodName() ) { |
43 | 43 | return $carry.$this->implode( $details[$key], $systemDetails ); |
@@ -54,8 +54,8 @@ discard block |
||
54 | 54 | { |
55 | 55 | $plugins = get_plugins(); |
56 | 56 | $activePlugins = (array)get_option( 'active_plugins', [] ); |
57 | - $inactive = array_diff_key( $plugins, array_flip( $activePlugins )); |
|
58 | - return $this->normalizePluginList( array_diff_key( $plugins, $inactive )); |
|
57 | + $inactive = array_diff_key( $plugins, array_flip( $activePlugins ) ); |
|
58 | + return $this->normalizePluginList( array_diff_key( $plugins, $inactive ) ); |
|
59 | 59 | } |
60 | 60 | |
61 | 61 | /** |
@@ -79,7 +79,7 @@ discard block |
||
79 | 79 | public function getInactivePluginDetails() |
80 | 80 | { |
81 | 81 | $activePlugins = (array)get_option( 'active_plugins', [] ); |
82 | - return $this->normalizePluginList( array_diff_key( get_plugins(), array_flip( $activePlugins ))); |
|
82 | + return $this->normalizePluginList( array_diff_key( get_plugins(), array_flip( $activePlugins ) ) ); |
|
83 | 83 | } |
84 | 84 | |
85 | 85 | /** |
@@ -89,9 +89,9 @@ discard block |
||
89 | 89 | { |
90 | 90 | $plugins = array_merge( |
91 | 91 | get_mu_plugins(), |
92 | - get_plugins( '/../'.basename( WPMU_PLUGIN_DIR )) |
|
92 | + get_plugins( '/../'.basename( WPMU_PLUGIN_DIR ) ) |
|
93 | 93 | ); |
94 | - if( empty( $plugins ))return; |
|
94 | + if( empty($plugins) )return; |
|
95 | 95 | return $this->normalizePluginList( $plugins ); |
96 | 96 | } |
97 | 97 | |
@@ -100,7 +100,7 @@ discard block |
||
100 | 100 | */ |
101 | 101 | public function getMultisitePluginDetails() |
102 | 102 | { |
103 | - if( !is_multisite() || empty( get_site_option( 'active_sitewide_plugins', [] )))return; |
|
103 | + if( !is_multisite() || empty(get_site_option( 'active_sitewide_plugins', [] )) )return; |
|
104 | 104 | return $this->normalizePluginList( wp_get_active_network_plugins() ); |
105 | 105 | } |
106 | 106 | |
@@ -122,11 +122,11 @@ discard block |
||
122 | 122 | 'Max Input Vars' => ini_get( 'max_input_vars' ), |
123 | 123 | 'Memory Limit' => ini_get( 'memory_limit' ), |
124 | 124 | 'Post Max Size' => ini_get( 'post_max_size' ), |
125 | - 'Session Cookie Path' => esc_html( ini_get( 'session.cookie_path' )), |
|
126 | - 'Session Name' => esc_html( ini_get( 'session.name' )), |
|
127 | - 'Session Save Path' => esc_html( ini_get( 'session.save_path' )), |
|
128 | - 'Session Use Cookies' => var_export( wp_validate_boolean( ini_get( 'session.use_cookies' )), true ), |
|
129 | - 'Session Use Only Cookies' => var_export( wp_validate_boolean( ini_get( 'session.use_only_cookies' )), true ), |
|
125 | + 'Session Cookie Path' => esc_html( ini_get( 'session.cookie_path' ) ), |
|
126 | + 'Session Name' => esc_html( ini_get( 'session.name' ) ), |
|
127 | + 'Session Save Path' => esc_html( ini_get( 'session.save_path' ) ), |
|
128 | + 'Session Use Cookies' => var_export( wp_validate_boolean( ini_get( 'session.use_cookies' ) ), true ), |
|
129 | + 'Session Use Only Cookies' => var_export( wp_validate_boolean( ini_get( 'session.use_only_cookies' ) ), true ), |
|
130 | 130 | 'Upload Max Filesize' => ini_get( 'upload_max_filesize' ), |
131 | 131 | ]; |
132 | 132 | } |
@@ -154,13 +154,13 @@ discard block |
||
154 | 154 | $settings = glsr( OptionManager::class )->get( 'settings', [] ); |
155 | 155 | $settings = $helper->flattenArray( $settings, true ); |
156 | 156 | foreach( ['submissions.recaptcha.key', 'submissions.recaptcha.secret'] as $key ) { |
157 | - if( empty( $settings[$key] ))continue; |
|
157 | + if( empty($settings[$key]) )continue; |
|
158 | 158 | $settings[$key] = str_repeat( '*', 10 ); |
159 | 159 | } |
160 | 160 | $details = []; |
161 | 161 | foreach( $settings as $key => $value ) { |
162 | - if( $helper->startsWith( 'strings', $key ) && $helper->endsWith( 'id', $key ))continue; |
|
163 | - $value = htmlspecialchars( trim( preg_replace('/\s\s+/', '\\n', $value )), ENT_QUOTES, 'UTF-8' ); |
|
162 | + if( $helper->startsWith( 'strings', $key ) && $helper->endsWith( 'id', $key ) )continue; |
|
163 | + $value = htmlspecialchars( trim( preg_replace( '/\s\s+/', '\\n', $value ) ), ENT_QUOTES, 'UTF-8' ); |
|
164 | 164 | $details[$key] = $value; |
165 | 165 | } |
166 | 166 | return $details; |
@@ -231,10 +231,10 @@ discard block |
||
231 | 231 | 'WPE_APIKEY' => 'WP Engine', |
232 | 232 | ]; |
233 | 233 | foreach( $checks as $key => $value ) { |
234 | - if( !$this->isWebhostCheckValid( $key ))continue; |
|
234 | + if( !$this->isWebhostCheckValid( $key ) )continue; |
|
235 | 235 | return $value; |
236 | 236 | } |
237 | - return implode( ',', array_filter( [DB_HOST, filter_input( INPUT_SERVER, 'SERVER_NAME' )] )); |
|
237 | + return implode( ',', array_filter( [DB_HOST, filter_input( INPUT_SERVER, 'SERVER_NAME' )] ) ); |
|
238 | 238 | } |
239 | 239 | |
240 | 240 | /** |
@@ -255,8 +255,8 @@ discard block |
||
255 | 255 | { |
256 | 256 | $plugins = get_plugins(); |
257 | 257 | $activePlugins = (array)get_option( 'active_plugins', [] ); |
258 | - $inactive = $this->normalizePluginList( array_diff_key( $plugins, array_flip( $activePlugins ))); |
|
259 | - $active = $this->normalizePluginList( array_diff_key( $plugins, $inactive )); |
|
258 | + $inactive = $this->normalizePluginList( array_diff_key( $plugins, array_flip( $activePlugins ) ) ); |
|
259 | + $active = $this->normalizePluginList( array_diff_key( $plugins, $inactive ) ); |
|
260 | 260 | return $active + $inactive; |
261 | 261 | } |
262 | 262 | |
@@ -267,7 +267,7 @@ discard block |
||
267 | 267 | protected function implode( $title, array $details ) |
268 | 268 | { |
269 | 269 | $strings = ['['.$title.']']; |
270 | - $padding = max( array_map( 'strlen', array_keys( $details )) ); |
|
270 | + $padding = max( array_map( 'strlen', array_keys( $details ) ) ); |
|
271 | 271 | $padding = max( [$padding, static::PAD] ); |
272 | 272 | foreach( $details as $key => $value ) { |
273 | 273 | $strings[] = is_string( $key ) |
@@ -91,7 +91,9 @@ discard block |
||
91 | 91 | get_mu_plugins(), |
92 | 92 | get_plugins( '/../'.basename( WPMU_PLUGIN_DIR )) |
93 | 93 | ); |
94 | - if( empty( $plugins ))return; |
|
94 | + if( empty( $plugins )) { |
|
95 | + return; |
|
96 | + } |
|
95 | 97 | return $this->normalizePluginList( $plugins ); |
96 | 98 | } |
97 | 99 | |
@@ -100,7 +102,9 @@ discard block |
||
100 | 102 | */ |
101 | 103 | public function getMultisitePluginDetails() |
102 | 104 | { |
103 | - if( !is_multisite() || empty( get_site_option( 'active_sitewide_plugins', [] )))return; |
|
105 | + if( !is_multisite() || empty( get_site_option( 'active_sitewide_plugins', [] ))) { |
|
106 | + return; |
|
107 | + } |
|
104 | 108 | return $this->normalizePluginList( wp_get_active_network_plugins() ); |
105 | 109 | } |
106 | 110 | |
@@ -154,12 +158,16 @@ discard block |
||
154 | 158 | $settings = glsr( OptionManager::class )->get( 'settings', [] ); |
155 | 159 | $settings = $helper->flattenArray( $settings, true ); |
156 | 160 | foreach( ['submissions.recaptcha.key', 'submissions.recaptcha.secret'] as $key ) { |
157 | - if( empty( $settings[$key] ))continue; |
|
161 | + if( empty( $settings[$key] )) { |
|
162 | + continue; |
|
163 | + } |
|
158 | 164 | $settings[$key] = str_repeat( '*', 10 ); |
159 | 165 | } |
160 | 166 | $details = []; |
161 | 167 | foreach( $settings as $key => $value ) { |
162 | - if( $helper->startsWith( 'strings', $key ) && $helper->endsWith( 'id', $key ))continue; |
|
168 | + if( $helper->startsWith( 'strings', $key ) && $helper->endsWith( 'id', $key )) { |
|
169 | + continue; |
|
170 | + } |
|
163 | 171 | $value = htmlspecialchars( trim( preg_replace('/\s\s+/', '\\n', $value )), ENT_QUOTES, 'UTF-8' ); |
164 | 172 | $details[$key] = $value; |
165 | 173 | } |
@@ -231,7 +239,9 @@ discard block |
||
231 | 239 | 'WPE_APIKEY' => 'WP Engine', |
232 | 240 | ]; |
233 | 241 | foreach( $checks as $key => $value ) { |
234 | - if( !$this->isWebhostCheckValid( $key ))continue; |
|
242 | + if( !$this->isWebhostCheckValid( $key )) { |
|
243 | + continue; |
|
244 | + } |
|
235 | 245 | return $value; |
236 | 246 | } |
237 | 247 | return implode( ',', array_filter( [DB_HOST, filter_input( INPUT_SERVER, 'SERVER_NAME' )] )); |
@@ -2,7 +2,7 @@ discard block |
||
2 | 2 | |
3 | 3 | defined( 'WPINC' ) || die; |
4 | 4 | |
5 | -require_once( ABSPATH.WPINC.'/class-phpass.php' ); |
|
5 | +require_once(ABSPATH.WPINC.'/class-phpass.php'); |
|
6 | 6 | |
7 | 7 | spl_autoload_register( function( $className ) { |
8 | 8 | $namespaces = [ |
@@ -16,8 +16,8 @@ discard block |
||
16 | 16 | foreach( $namespaces as $prefix => $baseDir ) { |
17 | 17 | $len = strlen( $prefix ); |
18 | 18 | if( strncmp( $prefix, $className, $len ) !== 0 )continue; |
19 | - $file = $baseDir.str_replace( '\\', '/', substr( $className, $len )).'.php'; |
|
20 | - if( !file_exists( $file ))continue; |
|
19 | + $file = $baseDir.str_replace( '\\', '/', substr( $className, $len ) ).'.php'; |
|
20 | + if( !file_exists( $file ) )continue; |
|
21 | 21 | require $file; |
22 | 22 | break; |
23 | 23 | } |
@@ -15,9 +15,13 @@ |
||
15 | 15 | ]; |
16 | 16 | foreach( $namespaces as $prefix => $baseDir ) { |
17 | 17 | $len = strlen( $prefix ); |
18 | - if( strncmp( $prefix, $className, $len ) !== 0 )continue; |
|
18 | + if( strncmp( $prefix, $className, $len ) !== 0 ) { |
|
19 | + continue; |
|
20 | + } |
|
19 | 21 | $file = $baseDir.str_replace( '\\', '/', substr( $className, $len )).'.php'; |
20 | - if( !file_exists( $file ))continue; |
|
22 | + if( !file_exists( $file )) { |
|
23 | + continue; |
|
24 | + } |
|
21 | 25 | require $file; |
22 | 26 | break; |
23 | 27 | } |
@@ -6,7 +6,7 @@ discard block |
||
6 | 6 | */ |
7 | 7 | function glsr( $alias = null ) { |
8 | 8 | $app = \GeminiLabs\SiteReviews\Application::load(); |
9 | - return !empty( $alias ) |
|
9 | + return !empty($alias) |
|
10 | 10 | ? $app->make( $alias ) |
11 | 11 | : $app; |
12 | 12 | } |
@@ -15,10 +15,10 @@ discard block |
||
15 | 15 | * @return \WP_Screen|object |
16 | 16 | */ |
17 | 17 | function glsr_current_screen() { |
18 | - if( function_exists( 'get_current_screen' )) { |
|
18 | + if( function_exists( 'get_current_screen' ) ) { |
|
19 | 19 | $screen = get_current_screen(); |
20 | 20 | } |
21 | - return empty( $screen ) |
|
21 | + return empty($screen) |
|
22 | 22 | ? (object)array_fill_keys( ['base', 'id', 'post_type'], null ) |
23 | 23 | : $screen; |
24 | 24 | } |
@@ -53,11 +53,11 @@ discard block |
||
53 | 53 | */ |
54 | 54 | function glsr_log() { |
55 | 55 | $args = func_get_args(); |
56 | - $context = isset( $args[1] ) |
|
56 | + $context = isset($args[1]) |
|
57 | 57 | ? $args[1] |
58 | 58 | : []; |
59 | 59 | $logger = glsr( 'Modules\Logger' ); |
60 | - return !empty( $args ) |
|
60 | + return !empty($args) |
|
61 | 61 | ? $logger->log( 'debug', $args[0], $context ) |
62 | 62 | : $logger; |
63 | 63 | } |
@@ -4,7 +4,8 @@ discard block |
||
4 | 4 | /** |
5 | 5 | * @return mixed |
6 | 6 | */ |
7 | -function glsr( $alias = null ) { |
|
7 | +function glsr( $alias = null ) |
|
8 | +{ |
|
8 | 9 | $app = \GeminiLabs\SiteReviews\Application::load(); |
9 | 10 | return !empty( $alias ) |
10 | 11 | ? $app->make( $alias ) |
@@ -14,7 +15,8 @@ discard block |
||
14 | 15 | /** |
15 | 16 | * @return \WP_Screen|object |
16 | 17 | */ |
17 | -function glsr_current_screen() { |
|
18 | +function glsr_current_screen() |
|
19 | +{ |
|
18 | 20 | if( function_exists( 'get_current_screen' )) { |
19 | 21 | $screen = get_current_screen(); |
20 | 22 | } |
@@ -26,7 +28,8 @@ discard block |
||
26 | 28 | /** |
27 | 29 | * @return \GeminiLabs\SiteReviews\Database |
28 | 30 | */ |
29 | -function glsr_db() { |
|
31 | +function glsr_db() |
|
32 | +{ |
|
30 | 33 | return glsr( 'Database' ); |
31 | 34 | } |
32 | 35 | |
@@ -34,7 +37,8 @@ discard block |
||
34 | 37 | * @param mixed ...$vars |
35 | 38 | * @return void |
36 | 39 | */ |
37 | -function glsr_debug( ...$vars ) { |
|
40 | +function glsr_debug( ...$vars ) |
|
41 | +{ |
|
38 | 42 | if( count( $vars ) == 1 ) { |
39 | 43 | $value = htmlspecialchars( print_r( $vars[0], true ), ENT_QUOTES, 'UTF-8' ); |
40 | 44 | printf( '<div class="glsr-debug"><pre>%s</pre></div>', $value ); |
@@ -51,7 +55,8 @@ discard block |
||
51 | 55 | /** |
52 | 56 | * @return \GeminiLabs\SiteReviews\Modules\Logger |
53 | 57 | */ |
54 | -function glsr_log() { |
|
58 | +function glsr_log() |
|
59 | +{ |
|
55 | 60 | $args = func_get_args(); |
56 | 61 | $context = isset( $args[1] ) |
57 | 62 | ? $args[1] |
@@ -67,14 +72,16 @@ discard block |
||
67 | 72 | * @param mixed $fallback |
68 | 73 | * @return string|array |
69 | 74 | */ |
70 | -function glsr_get_option( $path = '', $fallback = '' ) { |
|
75 | +function glsr_get_option( $path = '', $fallback = '' ) |
|
76 | +{ |
|
71 | 77 | return glsr( 'Database\OptionManager' )->get( 'settings.'.$path, $fallback ); |
72 | 78 | } |
73 | 79 | |
74 | 80 | /** |
75 | 81 | * @return array |
76 | 82 | */ |
77 | -function glsr_get_options() { |
|
83 | +function glsr_get_options() |
|
84 | +{ |
|
78 | 85 | return glsr( 'Database\OptionManager' )->get( 'settings' ); |
79 | 86 | } |
80 | 87 |
@@ -16,7 +16,7 @@ discard block |
||
16 | 16 | public function routeAdminPostRequest() |
17 | 17 | { |
18 | 18 | $request = filter_input( INPUT_POST, Application::ID, FILTER_DEFAULT, FILTER_REQUIRE_ARRAY ); |
19 | - if( !isset( $request['action'] ))return; |
|
19 | + if( !isset($request['action']) )return; |
|
20 | 20 | $this->checkNonce( $request['action'] ); |
21 | 21 | $this->routeRequest( 'admin', $request['action'], $request ); |
22 | 22 | } |
@@ -27,7 +27,7 @@ discard block |
||
27 | 27 | public function routeAjaxRequest() |
28 | 28 | { |
29 | 29 | $request = $this->normalizeAjaxRequest(); |
30 | - if( !wp_verify_nonce( $request['nonce'], $request['action'] )) { |
|
30 | + if( !wp_verify_nonce( $request['nonce'], $request['action'] ) ) { |
|
31 | 31 | glsr_log()->error( 'Nonce check failed for ajax request' )->info( $request ); |
32 | 32 | wp_die( -1, 403 ); |
33 | 33 | } |
@@ -41,8 +41,8 @@ discard block |
||
41 | 41 | public function routePublicPostRequest() |
42 | 42 | { |
43 | 43 | $request = filter_input( INPUT_POST, Application::ID, FILTER_DEFAULT, FILTER_REQUIRE_ARRAY ); |
44 | - if( !isset( $request['action'] ))return; |
|
45 | - if( !wp_verify_nonce( $request['_wpnonce'], $request['action'] )) { |
|
44 | + if( !isset($request['action']) )return; |
|
45 | + if( !wp_verify_nonce( $request['_wpnonce'], $request['action'] ) ) { |
|
46 | 46 | glsr_log()->error( 'Nonce check failed for public request' )->info( $request ); |
47 | 47 | return; |
48 | 48 | } |
@@ -57,9 +57,9 @@ discard block |
||
57 | 57 | */ |
58 | 58 | public function routeRequest( $type, $action, array $request = [] ) |
59 | 59 | { |
60 | - $controller = glsr( glsr( Helper::class )->buildClassName( $type.'-controller', 'Controllers' )); |
|
60 | + $controller = glsr( glsr( Helper::class )->buildClassName( $type.'-controller', 'Controllers' ) ); |
|
61 | 61 | $method = glsr( Helper::class )->buildMethodName( $action, 'router' ); |
62 | - if( is_callable( [$controller, $method] )) { |
|
62 | + if( is_callable( [$controller, $method] ) ) { |
|
63 | 63 | call_user_func( [$controller, $method], $request ); |
64 | 64 | return; |
65 | 65 | } |
@@ -75,7 +75,7 @@ discard block |
||
75 | 75 | */ |
76 | 76 | public function routeWebhookRequest() |
77 | 77 | { |
78 | - $request = filter_input( INPUT_GET, sprintf( '%s-hook', Application::ID )); |
|
78 | + $request = filter_input( INPUT_GET, sprintf( '%s-hook', Application::ID ) ); |
|
79 | 79 | if( !$request )return; |
80 | 80 | // @todo manage webhook here |
81 | 81 | } |
@@ -114,14 +114,14 @@ discard block |
||
114 | 114 | protected function normalizeAjaxRequest() |
115 | 115 | { |
116 | 116 | $request = filter_input( INPUT_POST, 'request', FILTER_DEFAULT, FILTER_REQUIRE_ARRAY ); |
117 | - if( isset( $request[Application::ID]['action'] )) { |
|
117 | + if( isset($request[Application::ID]['action']) ) { |
|
118 | 118 | $request = $request[Application::ID]; |
119 | 119 | } |
120 | - if( !isset( $request['action'] )) { |
|
120 | + if( !isset($request['action']) ) { |
|
121 | 121 | glsr_log()->error( 'The AJAX request must include an action' )->info( $request ); |
122 | 122 | wp_die(); |
123 | 123 | } |
124 | - if( !isset( $request['nonce'] )) { |
|
124 | + if( !isset($request['nonce']) ) { |
|
125 | 125 | glsr_log()->error( 'The AJAX request must include a nonce' )->info( $request ); |
126 | 126 | wp_die(); |
127 | 127 | } |
@@ -16,7 +16,9 @@ discard block |
||
16 | 16 | public function routeAdminPostRequest() |
17 | 17 | { |
18 | 18 | $request = filter_input( INPUT_POST, Application::ID, FILTER_DEFAULT, FILTER_REQUIRE_ARRAY ); |
19 | - if( !isset( $request['action'] ))return; |
|
19 | + if( !isset( $request['action'] )) { |
|
20 | + return; |
|
21 | + } |
|
20 | 22 | $this->checkNonce( $request['action'] ); |
21 | 23 | $this->routeRequest( 'admin', $request['action'], $request ); |
22 | 24 | } |
@@ -41,7 +43,9 @@ discard block |
||
41 | 43 | public function routePublicPostRequest() |
42 | 44 | { |
43 | 45 | $request = filter_input( INPUT_POST, Application::ID, FILTER_DEFAULT, FILTER_REQUIRE_ARRAY ); |
44 | - if( !isset( $request['action'] ))return; |
|
46 | + if( !isset( $request['action'] )) { |
|
47 | + return; |
|
48 | + } |
|
45 | 49 | if( !wp_verify_nonce( $request['_wpnonce'], $request['action'] )) { |
46 | 50 | glsr_log()->error( 'Nonce check failed for public request' )->info( $request ); |
47 | 51 | return; |
@@ -76,7 +80,9 @@ discard block |
||
76 | 80 | public function routeWebhookRequest() |
77 | 81 | { |
78 | 82 | $request = filter_input( INPUT_GET, sprintf( '%s-hook', Application::ID )); |
79 | - if( !$request )return; |
|
83 | + if( !$request ) { |
|
84 | + return; |
|
85 | + } |
|
80 | 86 | // @todo manage webhook here |
81 | 87 | } |
82 | 88 |
@@ -27,21 +27,21 @@ discard block |
||
27 | 27 | public function __construct( $input ) |
28 | 28 | { |
29 | 29 | $this->request = $input; |
30 | - $this->ajax_request = isset( $input['ajax_request'] ); |
|
30 | + $this->ajax_request = isset($input['ajax_request']); |
|
31 | 31 | $this->assigned_to = $this->getNumeric( 'assign_to' ); |
32 | - $this->author = sanitize_text_field( $this->get( 'name' )); |
|
33 | - $this->avatar = get_avatar_url( $this->get( 'email' )); |
|
34 | - $this->blacklisted = isset( $input['blacklisted'] ); |
|
35 | - $this->category = sanitize_key( $this->get( 'category' )); |
|
36 | - $this->content = sanitize_textarea_field( $this->get( 'content' )); |
|
32 | + $this->author = sanitize_text_field( $this->get( 'name' ) ); |
|
33 | + $this->avatar = get_avatar_url( $this->get( 'email' ) ); |
|
34 | + $this->blacklisted = isset($input['blacklisted']); |
|
35 | + $this->category = sanitize_key( $this->get( 'category' ) ); |
|
36 | + $this->content = sanitize_textarea_field( $this->get( 'content' ) ); |
|
37 | 37 | $this->custom = $this->getCustom(); |
38 | - $this->email = sanitize_email( $this->get( 'email' )); |
|
39 | - $this->form_id = sanitize_key( $this->get( 'form_id' )); |
|
38 | + $this->email = sanitize_email( $this->get( 'email' ) ); |
|
39 | + $this->form_id = sanitize_key( $this->get( 'form_id' ) ); |
|
40 | 40 | $this->ip_address = $this->get( 'ip_address' ); |
41 | - $this->rating = intval( $this->get( 'rating' )); |
|
41 | + $this->rating = intval( $this->get( 'rating' ) ); |
|
42 | 42 | $this->referrer = $this->get( '_wp_http_referer' ); |
43 | - $this->terms = isset( $input['terms'] ); |
|
44 | - $this->title = sanitize_text_field( $this->get( 'title' )); |
|
43 | + $this->terms = isset($input['terms']); |
|
44 | + $this->title = sanitize_text_field( $this->get( 'title' ) ); |
|
45 | 45 | } |
46 | 46 | |
47 | 47 | /** |
@@ -50,7 +50,7 @@ discard block |
||
50 | 50 | */ |
51 | 51 | protected function get( $key ) |
52 | 52 | { |
53 | - return isset( $this->request[$key] ) |
|
53 | + return isset($this->request[$key]) |
|
54 | 54 | ? (string)$this->request[$key] |
55 | 55 | : ''; |
56 | 56 | } |
@@ -67,7 +67,7 @@ discard block |
||
67 | 67 | ]; |
68 | 68 | $custom = $this->request; |
69 | 69 | foreach( $unset as $value ) { |
70 | - unset( $custom[$value] ); |
|
70 | + unset($custom[$value]); |
|
71 | 71 | } |
72 | 72 | return $custom; |
73 | 73 | } |
@@ -35,7 +35,7 @@ discard block |
||
35 | 35 | 'post_type' => Application::POST_TYPE, |
36 | 36 | ]; |
37 | 37 | $postId = wp_insert_post( $post, true ); |
38 | - if( is_wp_error( $postId )) { |
|
38 | + if( is_wp_error( $postId ) ) { |
|
39 | 39 | glsr_log()->error( $postId->get_error_message() ); |
40 | 40 | return false; |
41 | 41 | } |
@@ -51,7 +51,7 @@ discard block |
||
51 | 51 | */ |
52 | 52 | public function deleteReview( $metaReviewId ) |
53 | 53 | { |
54 | - if( $postId = $this->getReviewPostId( $metaReviewId )) { |
|
54 | + if( $postId = $this->getReviewPostId( $metaReviewId ) ) { |
|
55 | 55 | wp_delete_post( $postId, true ); |
56 | 56 | } |
57 | 57 | } |
@@ -64,12 +64,12 @@ discard block |
||
64 | 64 | public function getAssignedToPost( $post, $assignedTo = '' ) |
65 | 65 | { |
66 | 66 | $post = get_post( $post ); |
67 | - if( !( $post instanceof WP_Post ))return; |
|
68 | - if( empty( $assignedTo )) { |
|
67 | + if( !($post instanceof WP_Post) )return; |
|
68 | + if( empty($assignedTo) ) { |
|
69 | 69 | $assignedTo = get_post_meta( $post->ID, 'assigned_to', true ); |
70 | 70 | } |
71 | 71 | $assignedPost = get_post( $assignedTo ); |
72 | - if( !empty( $assignedTo ) |
|
72 | + if( !empty($assignedTo) |
|
73 | 73 | && $assignedPost instanceof WP_Post |
74 | 74 | && $assignedPost->ID != $post->ID ) { |
75 | 75 | return $assignedPost; |
@@ -82,7 +82,7 @@ discard block |
||
82 | 82 | */ |
83 | 83 | public function getReview( $post ) |
84 | 84 | { |
85 | - if( !( $post instanceof WP_Post ) || $post->post_type != Application::POST_TYPE )return; |
|
85 | + if( !($post instanceof WP_Post) || $post->post_type != Application::POST_TYPE )return; |
|
86 | 86 | $review = $this->getReviewMeta( $post->ID ); |
87 | 87 | $modified = $this->isReviewModified( $post, $review ); |
88 | 88 | $review->content = $post->post_content; |
@@ -104,13 +104,13 @@ discard block |
||
104 | 104 | { |
105 | 105 | $metaKey = $this->normalizeMetaKey( $metaKey ); |
106 | 106 | if( !$metaKey ) { |
107 | - return (array) wp_count_posts( Application::POST_TYPE ); |
|
107 | + return (array)wp_count_posts( Application::POST_TYPE ); |
|
108 | 108 | } |
109 | 109 | $counts = glsr( Cache::class )->getReviewCountsFor( $metaKey ); |
110 | 110 | if( !$metaValue ) { |
111 | 111 | return $counts; |
112 | 112 | } |
113 | - return isset( $counts[$metaValue] ) |
|
113 | + return isset($counts[$metaValue]) |
|
114 | 114 | ? $counts[$metaValue] |
115 | 115 | : 0; |
116 | 116 | } |
@@ -131,9 +131,9 @@ discard block |
||
131 | 131 | public function getReviewMeta( $postId ) |
132 | 132 | { |
133 | 133 | $meta = get_post_type( $postId ) == Application::POST_TYPE |
134 | - ? array_map( 'array_shift', (array) get_post_meta( $postId )) |
|
134 | + ? array_map( 'array_shift', (array)get_post_meta( $postId ) ) |
|
135 | 135 | : []; |
136 | - return (object) $this->normalizeMeta( array_filter( $meta, 'strlen' )); |
|
136 | + return (object)$this->normalizeMeta( array_filter( $meta, 'strlen' ) ); |
|
137 | 137 | } |
138 | 138 | |
139 | 139 | /** |
@@ -162,7 +162,7 @@ discard block |
||
162 | 162 | $paged = glsr( QueryBuilder::class )->getPaged( |
163 | 163 | wp_validate_boolean( $args['pagination'] ) |
164 | 164 | ); |
165 | - $reviews = new WP_Query([ |
|
165 | + $reviews = new WP_Query( [ |
|
166 | 166 | 'meta_key' => 'pinned', |
167 | 167 | 'meta_query' => $metaQuery, |
168 | 168 | 'offset' => $args['offset'], |
@@ -175,8 +175,8 @@ discard block |
||
175 | 175 | 'post_type' => Application::POST_TYPE, |
176 | 176 | 'posts_per_page' => $args['count'] ? $args['count'] : -1, |
177 | 177 | 'tax_query' => $taxQuery, |
178 | - ]); |
|
179 | - return (object) [ |
|
178 | + ] ); |
|
179 | + return (object)[ |
|
180 | 180 | 'results' => array_map( [$this, 'getReview'], $reviews->posts ), |
181 | 181 | 'max_num_pages' => $reviews->max_num_pages, |
182 | 182 | ]; |
@@ -190,7 +190,7 @@ discard block |
||
190 | 190 | public function getReviewsMeta( $keys, $status = 'publish' ) |
191 | 191 | { |
192 | 192 | $keys = array_map( [$this, 'normalizeMetaKey'], (array)$keys ); |
193 | - if( $status == 'all' || empty( $status )) { |
|
193 | + if( $status == 'all' || empty($status) ) { |
|
194 | 194 | $status = get_post_stati( ['exclude_from_search' => false] ); |
195 | 195 | } |
196 | 196 | return glsr( SqlQueries::class )->getReviewsMeta( $keys, $status ); |
@@ -206,10 +206,10 @@ discard block |
||
206 | 206 | 'fields' => 'id=>name', |
207 | 207 | 'hide_empty' => false, |
208 | 208 | 'taxonomy' => Application::TAXONOMY, |
209 | - ]); |
|
210 | - unset( $args['count'] ); //we don't want a term count |
|
209 | + ] ); |
|
210 | + unset($args['count']); //we don't want a term count |
|
211 | 211 | $terms = get_terms( $args ); |
212 | - if( is_wp_error( $terms )) { |
|
212 | + if( is_wp_error( $terms ) ) { |
|
213 | 213 | glsr_log()->error( $terms->get_error_message() ); |
214 | 214 | return []; |
215 | 215 | } |
@@ -221,7 +221,7 @@ discard block |
||
221 | 221 | */ |
222 | 222 | public function normalizeMeta( array $meta ) |
223 | 223 | { |
224 | - if( empty( $meta )) { |
|
224 | + if( empty($meta) ) { |
|
225 | 225 | return []; |
226 | 226 | } |
227 | 227 | $defaults = wp_parse_args( $meta, [ |
@@ -229,7 +229,7 @@ discard block |
||
229 | 229 | 'date' => '', |
230 | 230 | 'review_id' => '', |
231 | 231 | 'review_type' => '', |
232 | - ]); |
|
232 | + ] ); |
|
233 | 233 | return glsr( CreateReviewDefaults::class )->restrict( $defaults ); |
234 | 234 | } |
235 | 235 | |
@@ -240,7 +240,7 @@ discard block |
||
240 | 240 | public function normalizeMetaKey( $metaKey ) |
241 | 241 | { |
242 | 242 | $metaKey = strtolower( $metaKey ); |
243 | - if( in_array( $metaKey, ['id', 'type'] )) { |
|
243 | + if( in_array( $metaKey, ['id', 'type'] ) ) { |
|
244 | 244 | $metaKey = 'review_'.$metaKey; |
245 | 245 | } |
246 | 246 | return $metaKey; |
@@ -253,10 +253,10 @@ discard block |
||
253 | 253 | public function normalizeTerms( $termIds ) |
254 | 254 | { |
255 | 255 | $terms = []; |
256 | - $termIds = array_map( 'trim', explode( ',', $termIds )); |
|
256 | + $termIds = array_map( 'trim', explode( ',', $termIds ) ); |
|
257 | 257 | foreach( $termIds as $termId ) { |
258 | 258 | $term = term_exists( $termId, Application::TAXONOMY ); |
259 | - if( !isset( $term['term_id'] ))continue; |
|
259 | + if( !isset($term['term_id']) )continue; |
|
260 | 260 | $terms[] = intval( $term['term_id'] ); |
261 | 261 | } |
262 | 262 | return $terms; |
@@ -269,15 +269,15 @@ discard block |
||
269 | 269 | public function revertReview( $postId ) |
270 | 270 | { |
271 | 271 | $post = get_post( $postId ); |
272 | - if( !( $post instanceof WP_Post ) || $post->post_type != Application::POST_TYPE )return; |
|
272 | + if( !($post instanceof WP_Post) || $post->post_type != Application::POST_TYPE )return; |
|
273 | 273 | delete_post_meta( $post->ID, '_edit_last' ); |
274 | - $result = wp_update_post([ |
|
274 | + $result = wp_update_post( [ |
|
275 | 275 | 'ID' => $post->ID, |
276 | 276 | 'post_content' => get_post_meta( $post->ID, 'content', true ), |
277 | 277 | 'post_date' => get_post_meta( $post->ID, 'date', true ), |
278 | 278 | 'post_title' => get_post_meta( $post->ID, 'title', true ), |
279 | - ]); |
|
280 | - if( is_wp_error( $result )) { |
|
279 | + ] ); |
|
280 | + if( is_wp_error( $result ) ) { |
|
281 | 281 | glsr_log()->error( $result->get_error_message() ); |
282 | 282 | } |
283 | 283 | } |
@@ -292,7 +292,7 @@ discard block |
||
292 | 292 | 'post_status' => 'publish', |
293 | 293 | 'post_type' => 'any', |
294 | 294 | ]; |
295 | - if( is_numeric( $searchTerm )) { |
|
295 | + if( is_numeric( $searchTerm ) ) { |
|
296 | 296 | $args['post__in'] = [$searchTerm]; |
297 | 297 | } |
298 | 298 | else { |
@@ -311,9 +311,9 @@ discard block |
||
311 | 311 | ob_start(); |
312 | 312 | glsr()->render( 'partials/editor/search-result', [ |
313 | 313 | 'ID' => get_the_ID(), |
314 | - 'permalink' => esc_url( (string) get_permalink() ), |
|
314 | + 'permalink' => esc_url( (string)get_permalink() ), |
|
315 | 315 | 'title' => esc_attr( get_the_title() ), |
316 | - ]); |
|
316 | + ] ); |
|
317 | 317 | $results .= ob_get_clean(); |
318 | 318 | } |
319 | 319 | wp_reset_postdata(); |
@@ -331,9 +331,9 @@ discard block |
||
331 | 331 | update_post_meta( $postId, $metaKey, $metaValue ); |
332 | 332 | } |
333 | 333 | $terms = $this->normalizeTerms( $termIds ); |
334 | - if( empty( $terms ))return; |
|
334 | + if( empty($terms) )return; |
|
335 | 335 | $result = wp_set_object_terms( $postId, $terms, Application::TAXONOMY ); |
336 | - if( is_wp_error( $result )) { |
|
336 | + if( is_wp_error( $result ) ) { |
|
337 | 337 | glsr_log()->error( $result->get_error_message() ); |
338 | 338 | } |
339 | 339 | } |
@@ -345,7 +345,7 @@ discard block |
||
345 | 345 | protected function getNewPostStatus( array $review, $isBlacklisted ) |
346 | 346 | { |
347 | 347 | $requireApprovalOption = glsr( OptionManager::class )->get( 'settings.general.require.approval' ); |
348 | - return $review['review_type'] == 'local' && ( $requireApprovalOption == 'yes' || $isBlacklisted ) |
|
348 | + return $review['review_type'] == 'local' && ($requireApprovalOption == 'yes' || $isBlacklisted) |
|
349 | 349 | ? 'pending' |
350 | 350 | : 'publish'; |
351 | 351 | } |
@@ -64,7 +64,9 @@ discard block |
||
64 | 64 | public function getAssignedToPost( $post, $assignedTo = '' ) |
65 | 65 | { |
66 | 66 | $post = get_post( $post ); |
67 | - if( !( $post instanceof WP_Post ))return; |
|
67 | + if( !( $post instanceof WP_Post )) { |
|
68 | + return; |
|
69 | + } |
|
68 | 70 | if( empty( $assignedTo )) { |
69 | 71 | $assignedTo = get_post_meta( $post->ID, 'assigned_to', true ); |
70 | 72 | } |
@@ -82,7 +84,9 @@ discard block |
||
82 | 84 | */ |
83 | 85 | public function getReview( $post ) |
84 | 86 | { |
85 | - if( !( $post instanceof WP_Post ) || $post->post_type != Application::POST_TYPE )return; |
|
87 | + if( !( $post instanceof WP_Post ) || $post->post_type != Application::POST_TYPE ) { |
|
88 | + return; |
|
89 | + } |
|
86 | 90 | $review = $this->getReviewMeta( $post->ID ); |
87 | 91 | $modified = $this->isReviewModified( $post, $review ); |
88 | 92 | $review->content = $post->post_content; |
@@ -256,7 +260,9 @@ discard block |
||
256 | 260 | $termIds = array_map( 'trim', explode( ',', $termIds )); |
257 | 261 | foreach( $termIds as $termId ) { |
258 | 262 | $term = term_exists( $termId, Application::TAXONOMY ); |
259 | - if( !isset( $term['term_id'] ))continue; |
|
263 | + if( !isset( $term['term_id'] )) { |
|
264 | + continue; |
|
265 | + } |
|
260 | 266 | $terms[] = intval( $term['term_id'] ); |
261 | 267 | } |
262 | 268 | return $terms; |
@@ -269,7 +275,9 @@ discard block |
||
269 | 275 | public function revertReview( $postId ) |
270 | 276 | { |
271 | 277 | $post = get_post( $postId ); |
272 | - if( !( $post instanceof WP_Post ) || $post->post_type != Application::POST_TYPE )return; |
|
278 | + if( !( $post instanceof WP_Post ) || $post->post_type != Application::POST_TYPE ) { |
|
279 | + return; |
|
280 | + } |
|
273 | 281 | delete_post_meta( $post->ID, '_edit_last' ); |
274 | 282 | $result = wp_update_post([ |
275 | 283 | 'ID' => $post->ID, |
@@ -304,7 +312,9 @@ discard block |
||
304 | 312 | add_filter( 'posts_search', [$queryBuilder, 'filterSearchByTitle'], 500, 2 ); |
305 | 313 | $search = new WP_Query( $args ); |
306 | 314 | remove_filter( 'posts_search', [$queryBuilder, 'filterSearchByTitle'], 500 ); |
307 | - if( !$search->have_posts() )return; |
|
315 | + if( !$search->have_posts() ) { |
|
316 | + return; |
|
317 | + } |
|
308 | 318 | $results = ''; |
309 | 319 | while( $search->have_posts() ) { |
310 | 320 | $search->the_post(); |
@@ -331,7 +341,9 @@ discard block |
||
331 | 341 | update_post_meta( $postId, $metaKey, $metaValue ); |
332 | 342 | } |
333 | 343 | $terms = $this->normalizeTerms( $termIds ); |
334 | - if( empty( $terms ))return; |
|
344 | + if( empty( $terms )) { |
|
345 | + return; |
|
346 | + } |
|
335 | 347 | $result = wp_set_object_terms( $postId, $terms, Application::TAXONOMY ); |
336 | 348 | if( is_wp_error( $result )) { |
337 | 349 | glsr_log()->error( $result->get_error_message() ); |