@@ -3,7 +3,7 @@ |
||
3 | 3 | defined( 'WP_UNINSTALL_PLUGIN' ) || die; |
4 | 4 | |
5 | 5 | require_once __DIR__.'/site-reviews.php'; |
6 | -if( !GL_Plugin_Check_v2::isValid( array( 'wordpress' => '4.7.0' )))return; |
|
6 | +if( !GL_Plugin_Check_v2::isValid( array( 'wordpress' => '4.7.0' ) ) )return; |
|
7 | 7 | |
8 | 8 | delete_option( GeminiLabs\SiteReviews\Database\OptionManager::databaseKey() ); |
9 | 9 | delete_option( 'widget_'.glsr()->id.'_site-reviews' ); |
@@ -3,7 +3,9 @@ |
||
3 | 3 | defined( 'WP_UNINSTALL_PLUGIN' ) || die; |
4 | 4 | |
5 | 5 | require_once __DIR__.'/site-reviews.php'; |
6 | -if( !GL_Plugin_Check_v2::isValid( array( 'wordpress' => '4.7.0' )))return; |
|
6 | +if( !GL_Plugin_Check_v2::isValid( array( 'wordpress' => '4.7.0' ))) { |
|
7 | + return; |
|
8 | +} |
|
7 | 9 | |
8 | 10 | delete_option( GeminiLabs\SiteReviews\Database\OptionManager::databaseKey() ); |
9 | 11 | delete_option( 'widget_'.glsr()->id.'_site-reviews' ); |
@@ -18,15 +18,15 @@ |
||
18 | 18 | |
19 | 19 | defined( 'WPINC' ) || die; |
20 | 20 | |
21 | -if( !class_exists( 'GL_Plugin_Check_v2' )) { |
|
21 | +if( !class_exists( 'GL_Plugin_Check_v2' ) ) { |
|
22 | 22 | require_once __DIR__.'/activate.php'; |
23 | 23 | } |
24 | -if( GL_Plugin_Check_v2::shouldDeactivate( __FILE__, array( 'wordpress' => '4.7.0' )))return; |
|
24 | +if( GL_Plugin_Check_v2::shouldDeactivate( __FILE__, array( 'wordpress' => '4.7.0' ) ) )return; |
|
25 | 25 | require_once __DIR__.'/autoload.php'; |
26 | 26 | require_once __DIR__.'/helpers.php'; |
27 | 27 | |
28 | 28 | $app = new GeminiLabs\SiteReviews\Application; |
29 | 29 | $app->make( 'Provider' )->register( $app ); |
30 | -register_activation_hook( __FILE__, array( $app, 'activate' )); |
|
31 | -register_deactivation_hook( __FILE__, array( $app, 'deactivate' )); |
|
30 | +register_activation_hook( __FILE__, array( $app, 'activate' ) ); |
|
31 | +register_deactivation_hook( __FILE__, array( $app, 'deactivate' ) ); |
|
32 | 32 | $app->init(); |
@@ -21,7 +21,9 @@ |
||
21 | 21 | if( !class_exists( 'GL_Plugin_Check_v2' )) { |
22 | 22 | require_once __DIR__.'/activate.php'; |
23 | 23 | } |
24 | -if( GL_Plugin_Check_v2::shouldDeactivate( __FILE__, array( 'wordpress' => '4.7.0' )))return; |
|
24 | +if( GL_Plugin_Check_v2::shouldDeactivate( __FILE__, array( 'wordpress' => '4.7.0' ))) { |
|
25 | + return; |
|
26 | +} |
|
25 | 27 | require_once __DIR__.'/autoload.php'; |
26 | 28 | require_once __DIR__.'/helpers.php'; |
27 | 29 |
@@ -30,7 +30,7 @@ discard block |
||
30 | 30 | { |
31 | 31 | $translations = $this->getTranslations(); |
32 | 32 | $entries = $this->filter( $translations, $this->entries() )->results(); |
33 | - array_walk( $translations, function( &$entry ) use( $entries ) { |
|
33 | + array_walk( $translations, function( &$entry ) use($entries) { |
|
34 | 34 | $entry['desc'] = array_key_exists( $entry['id'], $entries ) |
35 | 35 | ? $this->getEntryString( $entries[$entry['id']], 'msgctxt' ) |
36 | 36 | : ''; |
@@ -43,7 +43,7 @@ discard block |
||
43 | 43 | */ |
44 | 44 | public function entries() |
45 | 45 | { |
46 | - if( !isset( $this->entries )) { |
|
46 | + if( !isset($this->entries) ) { |
|
47 | 47 | try { |
48 | 48 | $potFile = glsr()->path( glsr()->languages.'/'.Application::ID.'.pot' ); |
49 | 49 | $entries = $this->normalize( Parser::parseFile( $potFile )->getEntries() ); |
@@ -76,13 +76,13 @@ discard block |
||
76 | 76 | */ |
77 | 77 | public function filter( $filterWith = null, $entries = null, $intersect = true ) |
78 | 78 | { |
79 | - if( !is_array( $entries )) { |
|
79 | + if( !is_array( $entries ) ) { |
|
80 | 80 | $entries = $this->results; |
81 | 81 | } |
82 | - if( !is_array( $filterWith )) { |
|
82 | + if( !is_array( $filterWith ) ) { |
|
83 | 83 | $filterWith = $this->getTranslations(); |
84 | 84 | } |
85 | - $keys = array_flip( array_column( $filterWith, 'id' )); |
|
85 | + $keys = array_flip( array_column( $filterWith, 'id' ) ); |
|
86 | 86 | $this->results = $intersect |
87 | 87 | ? array_intersect_key( $entries, $keys ) |
88 | 88 | : array_diff_key( $entries, $keys ); |
@@ -99,7 +99,7 @@ discard block |
||
99 | 99 | { |
100 | 100 | return $this->translate( $translation, $domain, [ |
101 | 101 | 'single' => $text, |
102 | - ]); |
|
102 | + ] ); |
|
103 | 103 | } |
104 | 104 | |
105 | 105 | /** |
@@ -114,7 +114,7 @@ discard block |
||
114 | 114 | return $this->translate( $translation, $domain, [ |
115 | 115 | 'context' => $context, |
116 | 116 | 'single' => $text, |
117 | - ]); |
|
117 | + ] ); |
|
118 | 118 | } |
119 | 119 | |
120 | 120 | /** |
@@ -131,7 +131,7 @@ discard block |
||
131 | 131 | 'number' => $number, |
132 | 132 | 'plural' => $plural, |
133 | 133 | 'single' => $single, |
134 | - ]); |
|
134 | + ] ); |
|
135 | 135 | } |
136 | 136 | |
137 | 137 | /** |
@@ -150,7 +150,7 @@ discard block |
||
150 | 150 | 'number' => $number, |
151 | 151 | 'plural' => $plural, |
152 | 152 | 'single' => $single, |
153 | - ]); |
|
153 | + ] ); |
|
154 | 154 | } |
155 | 155 | |
156 | 156 | /** |
@@ -160,13 +160,13 @@ discard block |
||
160 | 160 | public function render( $template, array $entry ) |
161 | 161 | { |
162 | 162 | $data = array_combine( |
163 | - array_map( function( $key ) { return 'data.'.$key; }, array_keys( $entry )), |
|
163 | + array_map( function( $key ) { return 'data.'.$key; }, array_keys( $entry ) ), |
|
164 | 164 | $entry |
165 | 165 | ); |
166 | 166 | ob_start(); |
167 | 167 | glsr( Html::class )->renderTemplate( 'partials/translations/'.$template, [ |
168 | 168 | 'context' => $data, |
169 | - ]); |
|
169 | + ] ); |
|
170 | 170 | return ob_get_clean(); |
171 | 171 | } |
172 | 172 | |
@@ -199,13 +199,13 @@ discard block |
||
199 | 199 | 'p1' => $this->getEntryString( $entry, 'msgid_plural' ), |
200 | 200 | 's1' => $this->getEntryString( $entry, 'msgid' ), |
201 | 201 | ]; |
202 | - $text = !empty( $data['p1'] ) |
|
202 | + $text = !empty($data['p1']) |
|
203 | 203 | ? sprintf( '%s | %s', $data['s1'], $data['p1'] ) |
204 | 204 | : $data['s1']; |
205 | 205 | $rendered .= $this->render( 'result', [ |
206 | 206 | 'entry' => wp_json_encode( $data ), |
207 | 207 | 'text' => wp_strip_all_tags( $text ), |
208 | - ]); |
|
208 | + ] ); |
|
209 | 209 | } |
210 | 210 | if( $resetAfterRender ) { |
211 | 211 | $this->reset(); |
@@ -238,12 +238,12 @@ discard block |
||
238 | 238 | public function search( $needle = '' ) |
239 | 239 | { |
240 | 240 | $this->reset(); |
241 | - $needle = trim( strtolower( $needle )); |
|
241 | + $needle = trim( strtolower( $needle ) ); |
|
242 | 242 | foreach( $this->entries() as $key => $entry ) { |
243 | - $single = strtolower( $this->getEntryString( $entry, 'msgid' )); |
|
244 | - $plural = strtolower( $this->getEntryString( $entry, 'msgid_plural' )); |
|
243 | + $single = strtolower( $this->getEntryString( $entry, 'msgid' ) ); |
|
244 | + $plural = strtolower( $this->getEntryString( $entry, 'msgid_plural' ) ); |
|
245 | 245 | if( strlen( $needle ) < static::SEARCH_THRESHOLD ) { |
246 | - if( in_array( $needle, [$single, $plural] )) { |
|
246 | + if( in_array( $needle, [$single, $plural] ) ) { |
|
247 | 247 | $this->results[$key] = $entry; |
248 | 248 | } |
249 | 249 | } |
@@ -266,7 +266,7 @@ discard block |
||
266 | 266 | } |
267 | 267 | $args = $this->normalizeTranslationArgs( $args ); |
268 | 268 | $strings = $this->getTranslationStrings( $args['single'], $args['plural'] ); |
269 | - if( empty( $strings )) { |
|
269 | + if( empty($strings) ) { |
|
270 | 270 | return $original; |
271 | 271 | } |
272 | 272 | $string = current( $strings ); |
@@ -281,8 +281,8 @@ discard block |
||
281 | 281 | */ |
282 | 282 | protected function getEntryString( array $entry, $key ) |
283 | 283 | { |
284 | - return isset( $entry[$key] ) |
|
285 | - ? implode( '', (array) $entry[$key] ) |
|
284 | + return isset($entry[$key]) |
|
285 | + ? implode( '', (array)$entry[$key] ) |
|
286 | 286 | : ''; |
287 | 287 | } |
288 | 288 | |
@@ -293,10 +293,10 @@ discard block |
||
293 | 293 | protected function getTranslations() |
294 | 294 | { |
295 | 295 | static $translations; |
296 | - if( empty( $translations )) { |
|
296 | + if( empty($translations) ) { |
|
297 | 297 | $settings = glsr( OptionManager::class )->get( 'settings' ); |
298 | - $translations = isset( $settings['strings'] ) |
|
299 | - ? $this->normalizeSettings( (array) $settings['strings'] ) |
|
298 | + $translations = isset($settings['strings']) |
|
299 | + ? $this->normalizeSettings( (array)$settings['strings'] ) |
|
300 | 300 | : []; |
301 | 301 | } |
302 | 302 | return $translations; |
@@ -309,7 +309,7 @@ discard block |
||
309 | 309 | */ |
310 | 310 | protected function getTranslationStrings( $single, $plural ) |
311 | 311 | { |
312 | - return array_filter( $this->getTranslations(), function( $string ) use( $single, $plural ) { |
|
312 | + return array_filter( $this->getTranslations(), function( $string ) use($single, $plural) { |
|
313 | 313 | return $string['s1'] == html_entity_decode( $single, ENT_COMPAT, 'UTF-8' ) |
314 | 314 | && $string['p1'] == html_entity_decode( $plural, ENT_COMPAT, 'UTF-8' ); |
315 | 315 | }); |
@@ -323,7 +323,7 @@ discard block |
||
323 | 323 | $keys = [ |
324 | 324 | 'msgctxt', 'msgid', 'msgid_plural', 'msgstr', 'msgstr[0]', 'msgstr[1]', |
325 | 325 | ]; |
326 | - array_walk( $entries, function( &$entry ) use( $keys ) { |
|
326 | + array_walk( $entries, function( &$entry ) use($keys) { |
|
327 | 327 | foreach( $keys as $key ) { |
328 | 328 | $entry = $this->normalizeEntryString( $entry, $key ); |
329 | 329 | } |
@@ -337,7 +337,7 @@ discard block |
||
337 | 337 | */ |
338 | 338 | protected function normalizeEntryString( array $entry, $key ) |
339 | 339 | { |
340 | - if( isset( $entry[$key] )) { |
|
340 | + if( isset($entry[$key]) ) { |
|
341 | 341 | $entry[$key] = $this->getEntryString( $entry, $key ); |
342 | 342 | } |
343 | 343 | return $entry; |
@@ -351,11 +351,11 @@ discard block |
||
351 | 351 | $defaultString = array_fill_keys( ['id', 's1', 's2', 'p1', 'p2'], '' ); |
352 | 352 | $strings = array_filter( $strings, 'is_array' ); |
353 | 353 | foreach( $strings as &$string ) { |
354 | - $string['type'] = isset( $string['p1'] ) ? 'plural' : 'single'; |
|
354 | + $string['type'] = isset($string['p1']) ? 'plural' : 'single'; |
|
355 | 355 | $string = wp_parse_args( $string, $defaultString ); |
356 | 356 | } |
357 | 357 | return array_filter( $strings, function( $string ) { |
358 | - return !empty( $string['id'] ); |
|
358 | + return !empty($string['id']); |
|
359 | 359 | }); |
360 | 360 | } |
361 | 361 | |
@@ -379,10 +379,10 @@ discard block |
||
379 | 379 | */ |
380 | 380 | protected function translatePlural( $domain, array $string, array $args ) |
381 | 381 | { |
382 | - if( !empty( $string['s2'] )) { |
|
382 | + if( !empty($string['s2']) ) { |
|
383 | 383 | $args['single'] = $string['s2']; |
384 | 384 | } |
385 | - if( !empty( $string['p2'] )) { |
|
385 | + if( !empty($string['p2']) ) { |
|
386 | 386 | $args['plural'] = $string['p2']; |
387 | 387 | } |
388 | 388 | return get_translations_for_domain( $domain )->translate_plural( |
@@ -399,7 +399,7 @@ discard block |
||
399 | 399 | */ |
400 | 400 | protected function translateSingle( $domain, array $string, array $args ) |
401 | 401 | { |
402 | - if( !empty( $string['s2'] )) { |
|
402 | + if( !empty($string['s2']) ) { |
|
403 | 403 | $args['single'] = $string['s2']; |
404 | 404 | } |
405 | 405 | return get_translations_for_domain( $domain )->translate( |
@@ -34,7 +34,7 @@ discard block |
||
34 | 34 | { |
35 | 35 | $this->args = $args; |
36 | 36 | $this->reviews = glsr( Database::class )->getReviews( $args )->results; |
37 | - if( empty( $this->reviews ) && $this->isHidden( 'if_empty' ))return; |
|
37 | + if( empty($this->reviews) && $this->isHidden( 'if_empty' ) )return; |
|
38 | 38 | $this->rating = glsr( Rating::class )->getAverage( $this->reviews ); |
39 | 39 | $this->generateSchema(); |
40 | 40 | return glsr( Template::class )->build( 'templates/reviews-summary', [ |
@@ -46,7 +46,7 @@ discard block |
||
46 | 46 | 'stars' => $this->buildStars(), |
47 | 47 | 'text' => $this->buildText(), |
48 | 48 | ], |
49 | - ]); |
|
49 | + ] ); |
|
50 | 50 | } |
51 | 51 | |
52 | 52 | /** |
@@ -54,16 +54,16 @@ discard block |
||
54 | 54 | */ |
55 | 55 | protected function buildPercentage() |
56 | 56 | { |
57 | - if( $this->isHidden( 'bars' ))return; |
|
57 | + if( $this->isHidden( 'bars' ) )return; |
|
58 | 58 | $range = range( Rating::MAX_RATING, 1 ); |
59 | - $percentages = preg_filter( '/$/', '%', glsr( Rating::class )->getPercentages( $this->reviews )); |
|
60 | - $bars = array_reduce( $range, function( $carry, $level ) use( $percentages ) { |
|
59 | + $percentages = preg_filter( '/$/', '%', glsr( Rating::class )->getPercentages( $this->reviews ) ); |
|
60 | + $bars = array_reduce( $range, function( $carry, $level ) use($percentages) { |
|
61 | 61 | $label = $this->buildPercentageLabel( $this->args['labels'][$level] ); |
62 | 62 | $background = $this->buildPercentageBackground( $percentages[$level] ); |
63 | 63 | $percent = $this->buildPercentagePercent( $percentages[$level] ); |
64 | 64 | return $carry.glsr( Builder::class )->div( $label.$background.$percent, [ |
65 | 65 | 'class' => 'glsr-bar', |
66 | - ]); |
|
66 | + ] ); |
|
67 | 67 | }); |
68 | 68 | return $this->wrap( 'percentage', $bars ); |
69 | 69 | } |
@@ -74,10 +74,10 @@ discard block |
||
74 | 74 | */ |
75 | 75 | protected function buildPercentageBackground( $percent ) |
76 | 76 | { |
77 | - $backgroundPercent = glsr( Builder::class )->span([ |
|
77 | + $backgroundPercent = glsr( Builder::class )->span( [ |
|
78 | 78 | 'class' => 'glsr-bar-background-percent', |
79 | 79 | 'style' => 'width:'.$percent, |
80 | - ]); |
|
80 | + ] ); |
|
81 | 81 | return '<span class="glsr-bar-background">'.$backgroundPercent.'</span>'; |
82 | 82 | } |
83 | 83 | |
@@ -104,7 +104,7 @@ discard block |
||
104 | 104 | */ |
105 | 105 | protected function buildRating() |
106 | 106 | { |
107 | - if( $this->isHidden( 'rating' ))return; |
|
107 | + if( $this->isHidden( 'rating' ) )return; |
|
108 | 108 | return $this->wrap( 'rating', '<span>'.$this->rating.'</span>' ); |
109 | 109 | } |
110 | 110 | |
@@ -113,10 +113,10 @@ discard block |
||
113 | 113 | */ |
114 | 114 | protected function buildStars() |
115 | 115 | { |
116 | - if( $this->isHidden( 'stars' ))return; |
|
116 | + if( $this->isHidden( 'stars' ) )return; |
|
117 | 117 | $stars = glsr( Partial::class )->build( 'star-rating', [ |
118 | 118 | 'rating' => $this->rating, |
119 | - ]); |
|
119 | + ] ); |
|
120 | 120 | return $this->wrap( 'stars', $stars ); |
121 | 121 | } |
122 | 122 | |
@@ -125,9 +125,9 @@ discard block |
||
125 | 125 | */ |
126 | 126 | protected function buildText() |
127 | 127 | { |
128 | - if( $this->isHidden( 'summary' ))return; |
|
128 | + if( $this->isHidden( 'summary' ) )return; |
|
129 | 129 | $count = count( $this->reviews ); |
130 | - if( empty( $this->args['text'] )) { |
|
130 | + if( empty($this->args['text']) ) { |
|
131 | 131 | // @todo document this change |
132 | 132 | $this->args['text'] = _nx( |
133 | 133 | '{rating} out of {max} stars (based on {num} review)', |
@@ -138,7 +138,7 @@ discard block |
||
138 | 138 | ); |
139 | 139 | } |
140 | 140 | $summary = str_replace( |
141 | - ['{rating}','{max}', '{num}'], |
|
141 | + ['{rating}', '{max}', '{num}'], |
|
142 | 142 | [$this->rating, Rating::MAX_RATING, $count], |
143 | 143 | $this->args['text'] |
144 | 144 | ); |
@@ -150,7 +150,7 @@ discard block |
||
150 | 150 | */ |
151 | 151 | protected function generateSchema() |
152 | 152 | { |
153 | - if( !wp_validate_boolean( $this->args['schema'] ))return; |
|
153 | + if( !wp_validate_boolean( $this->args['schema'] ) )return; |
|
154 | 154 | glsr( Schema::class )->store( |
155 | 155 | glsr( Schema::class )->buildSummary( $this->args ) |
156 | 156 | ); |
@@ -183,6 +183,6 @@ discard block |
||
183 | 183 | { |
184 | 184 | return glsr( Builder::class )->div( $value, [ |
185 | 185 | 'class' => 'glsr-summary-'.$key, |
186 | - ]); |
|
186 | + ] ); |
|
187 | 187 | } |
188 | 188 | } |
@@ -34,7 +34,9 @@ discard block |
||
34 | 34 | { |
35 | 35 | $this->args = $args; |
36 | 36 | $this->reviews = glsr( Database::class )->getReviews( $args )->results; |
37 | - if( empty( $this->reviews ) && $this->isHidden( 'if_empty' ))return; |
|
37 | + if( empty( $this->reviews ) && $this->isHidden( 'if_empty' )) { |
|
38 | + return; |
|
39 | + } |
|
38 | 40 | $this->rating = glsr( Rating::class )->getAverage( $this->reviews ); |
39 | 41 | $this->generateSchema(); |
40 | 42 | return glsr( Template::class )->build( 'templates/reviews-summary', [ |
@@ -54,7 +56,9 @@ discard block |
||
54 | 56 | */ |
55 | 57 | protected function buildPercentage() |
56 | 58 | { |
57 | - if( $this->isHidden( 'bars' ))return; |
|
59 | + if( $this->isHidden( 'bars' )) { |
|
60 | + return; |
|
61 | + } |
|
58 | 62 | $range = range( Rating::MAX_RATING, 1 ); |
59 | 63 | $percentages = preg_filter( '/$/', '%', glsr( Rating::class )->getPercentages( $this->reviews )); |
60 | 64 | $bars = array_reduce( $range, function( $carry, $level ) use( $percentages ) { |
@@ -104,7 +108,9 @@ discard block |
||
104 | 108 | */ |
105 | 109 | protected function buildRating() |
106 | 110 | { |
107 | - if( $this->isHidden( 'rating' ))return; |
|
111 | + if( $this->isHidden( 'rating' )) { |
|
112 | + return; |
|
113 | + } |
|
108 | 114 | return $this->wrap( 'rating', '<span>'.$this->rating.'</span>' ); |
109 | 115 | } |
110 | 116 | |
@@ -113,7 +119,9 @@ discard block |
||
113 | 119 | */ |
114 | 120 | protected function buildStars() |
115 | 121 | { |
116 | - if( $this->isHidden( 'stars' ))return; |
|
122 | + if( $this->isHidden( 'stars' )) { |
|
123 | + return; |
|
124 | + } |
|
117 | 125 | $stars = glsr( Partial::class )->build( 'star-rating', [ |
118 | 126 | 'rating' => $this->rating, |
119 | 127 | ]); |
@@ -125,7 +133,9 @@ discard block |
||
125 | 133 | */ |
126 | 134 | protected function buildText() |
127 | 135 | { |
128 | - if( $this->isHidden( 'summary' ))return; |
|
136 | + if( $this->isHidden( 'summary' )) { |
|
137 | + return; |
|
138 | + } |
|
129 | 139 | $count = count( $this->reviews ); |
130 | 140 | if( empty( $this->args['text'] )) { |
131 | 141 | // @todo document this change |
@@ -150,7 +160,9 @@ discard block |
||
150 | 160 | */ |
151 | 161 | protected function generateSchema() |
152 | 162 | { |
153 | - if( !wp_validate_boolean( $this->args['schema'] ))return; |
|
163 | + if( !wp_validate_boolean( $this->args['schema'] )) { |
|
164 | + return; |
|
165 | + } |
|
154 | 166 | glsr( Schema::class )->store( |
155 | 167 | glsr( Schema::class )->buildSummary( $this->args ) |
156 | 168 | ); |
@@ -16,7 +16,7 @@ discard block |
||
16 | 16 | public function routeAdminPostRequest() |
17 | 17 | { |
18 | 18 | $request = $this->getRequest(); |
19 | - if( !$this->isValidPostRequest( $request ))return; |
|
19 | + if( !$this->isValidPostRequest( $request ) )return; |
|
20 | 20 | check_admin_referer( $request['action'] ); |
21 | 21 | $this->routeRequest( 'admin', $request['action'], $request ); |
22 | 22 | } |
@@ -42,8 +42,8 @@ discard block |
||
42 | 42 | { |
43 | 43 | if( is_admin() )return; |
44 | 44 | $request = $this->getRequest(); |
45 | - if( !$this->isValidPostRequest( $request ))return; |
|
46 | - if( !$this->isValidPublicNonce( $request ))return; |
|
45 | + if( !$this->isValidPostRequest( $request ) )return; |
|
46 | + if( !$this->isValidPublicNonce( $request ) )return; |
|
47 | 47 | $this->routeRequest( 'public', $request['action'], $request ); |
48 | 48 | } |
49 | 49 | |
@@ -53,11 +53,11 @@ discard block |
||
53 | 53 | protected function checkAjaxNonce( array $request ) |
54 | 54 | { |
55 | 55 | if( !is_user_logged_in() )return; |
56 | - if( !isset( $request['nonce'] )) { |
|
56 | + if( !isset($request['nonce']) ) { |
|
57 | 57 | glsr_log()->error( 'The AJAX request must include a nonce' )->info( $request ); |
58 | 58 | wp_die(); |
59 | 59 | } |
60 | - if( !wp_verify_nonce( $request['nonce'], $request['action'] )) { |
|
60 | + if( !wp_verify_nonce( $request['nonce'], $request['action'] ) ) { |
|
61 | 61 | glsr_log()->error( 'Nonce check failed for ajax request' )->info( $request ); |
62 | 62 | wp_die( -1, 403 ); |
63 | 63 | } |
@@ -68,11 +68,11 @@ discard block |
||
68 | 68 | */ |
69 | 69 | protected function checkAjaxRequest( array $request ) |
70 | 70 | { |
71 | - if( !isset( $request['action'] )) { |
|
71 | + if( !isset($request['action']) ) { |
|
72 | 72 | glsr_log()->error( 'The AJAX request must include an action' )->info( $request ); |
73 | 73 | wp_die(); |
74 | 74 | } |
75 | - if( empty( $request['ajax_request'] )) { |
|
75 | + if( empty($request['ajax_request']) ) { |
|
76 | 76 | glsr_log()->error( 'The AJAX request look invalid' )->info( $request ); |
77 | 77 | wp_die(); |
78 | 78 | } |
@@ -94,7 +94,7 @@ discard block |
||
94 | 94 | */ |
95 | 95 | protected function isValidPostRequest( array $request = [] ) |
96 | 96 | { |
97 | - return !empty( $request['action'] ) && empty( $request['ajax_request'] ); |
|
97 | + return !empty($request['action']) && empty($request['ajax_request']); |
|
98 | 98 | } |
99 | 99 | |
100 | 100 | /** |
@@ -102,7 +102,7 @@ discard block |
||
102 | 102 | */ |
103 | 103 | protected function isValidPublicNonce( array $request ) |
104 | 104 | { |
105 | - if( is_user_logged_in() && !wp_verify_nonce( $request['nonce'], $request['action'] )) { |
|
105 | + if( is_user_logged_in() && !wp_verify_nonce( $request['nonce'], $request['action'] ) ) { |
|
106 | 106 | glsr_log()->error( 'Nonce check failed for public request' )->info( $request ); |
107 | 107 | return false; |
108 | 108 | } |
@@ -131,11 +131,11 @@ discard block |
||
131 | 131 | protected function routeRequest( $type, $action, array $request = [] ) |
132 | 132 | { |
133 | 133 | $actionHook = 'site-reviews/route/'.$type.'/request'; |
134 | - $controller = glsr( glsr( Helper::class )->buildClassName( $type.'-controller', 'Controllers' )); |
|
134 | + $controller = glsr( glsr( Helper::class )->buildClassName( $type.'-controller', 'Controllers' ) ); |
|
135 | 135 | $method = glsr( Helper::class )->buildMethodName( $action, 'router' ); |
136 | 136 | $request = apply_filters( 'site-reviews/route/request', $request, $action, $type ); |
137 | 137 | do_action( $actionHook, $action, $request ); |
138 | - if( is_callable( [$controller, $method] )) { |
|
138 | + if( is_callable( [$controller, $method] ) ) { |
|
139 | 139 | call_user_func( [$controller, $method], $request ); |
140 | 140 | return; |
141 | 141 | } |
@@ -16,7 +16,9 @@ discard block |
||
16 | 16 | public function routeAdminPostRequest() |
17 | 17 | { |
18 | 18 | $request = $this->getRequest(); |
19 | - if( !$this->isValidPostRequest( $request ))return; |
|
19 | + if( !$this->isValidPostRequest( $request )) { |
|
20 | + return; |
|
21 | + } |
|
20 | 22 | check_admin_referer( $request['action'] ); |
21 | 23 | $this->routeRequest( 'admin', $request['action'], $request ); |
22 | 24 | } |
@@ -40,10 +42,16 @@ discard block |
||
40 | 42 | */ |
41 | 43 | public function routePublicPostRequest() |
42 | 44 | { |
43 | - if( is_admin() )return; |
|
45 | + if( is_admin() ) { |
|
46 | + return; |
|
47 | + } |
|
44 | 48 | $request = $this->getRequest(); |
45 | - if( !$this->isValidPostRequest( $request ))return; |
|
46 | - if( !$this->isValidPublicNonce( $request ))return; |
|
49 | + if( !$this->isValidPostRequest( $request )) { |
|
50 | + return; |
|
51 | + } |
|
52 | + if( !$this->isValidPublicNonce( $request )) { |
|
53 | + return; |
|
54 | + } |
|
47 | 55 | $this->routeRequest( 'public', $request['action'], $request ); |
48 | 56 | } |
49 | 57 | |
@@ -52,7 +60,9 @@ discard block |
||
52 | 60 | */ |
53 | 61 | protected function checkAjaxNonce( array $request ) |
54 | 62 | { |
55 | - if( !is_user_logged_in() )return; |
|
63 | + if( !is_user_logged_in() ) { |
|
64 | + return; |
|
65 | + } |
|
56 | 66 | if( !isset( $request['nonce'] )) { |
57 | 67 | glsr_log()->error( 'The AJAX request must include a nonce' )->info( $request ); |
58 | 68 | wp_die(); |
@@ -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 | $console = glsr( 'Modules\Console' ); |
60 | - return !empty( $args ) |
|
60 | + return !empty($args) |
|
61 | 61 | ? $console->log( 'debug', $args[0], $context ) |
62 | 62 | : $console; |
63 | 63 | } |
@@ -83,7 +83,7 @@ discard block |
||
83 | 83 | * @return void|object |
84 | 84 | */ |
85 | 85 | function glsr_get_review( $post_id ) { |
86 | - return glsr( 'Database' )->getReview( get_post( $post_id )); |
|
86 | + return glsr( 'Database' )->getReview( get_post( $post_id ) ); |
|
87 | 87 | } |
88 | 88 | |
89 | 89 | /** |
@@ -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\Console |
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 | |
@@ -82,7 +89,8 @@ discard block |
||
82 | 89 | * @param int $post_id |
83 | 90 | * @return void|object |
84 | 91 | */ |
85 | -function glsr_get_review( $post_id ) { |
|
92 | +function glsr_get_review( $post_id ) |
|
93 | +{ |
|
86 | 94 | return glsr( 'Database' )->getReview( get_post( $post_id )); |
87 | 95 | } |
88 | 96 | |
@@ -90,6 +98,7 @@ discard block |
||
90 | 98 | * @return array |
91 | 99 | * @todo document change of $reviews->reviews to $reviews->results |
92 | 100 | */ |
93 | -function glsr_get_reviews( array $args = array() ) { |
|
101 | +function glsr_get_reviews( array $args = array() ) |
|
102 | +{ |
|
94 | 103 | return glsr( 'Database' )->getReviews( $args ); |
95 | 104 | } |
@@ -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 | } |
@@ -157,8 +157,8 @@ discard block |
||
157 | 157 | ksort( $settings ); |
158 | 158 | $details = []; |
159 | 159 | foreach( $settings as $key => $value ) { |
160 | - if( $helper->startsWith( 'strings', $key ) && $helper->endsWith( 'id', $key ))continue; |
|
161 | - $value = htmlspecialchars( trim( preg_replace('/\s\s+/', '\\n', $value )), ENT_QUOTES, 'UTF-8' ); |
|
160 | + if( $helper->startsWith( 'strings', $key ) && $helper->endsWith( 'id', $key ) )continue; |
|
161 | + $value = htmlspecialchars( trim( preg_replace( '/\s\s+/', '\\n', $value ) ), ENT_QUOTES, 'UTF-8' ); |
|
162 | 162 | $details[$key] = $value; |
163 | 163 | } |
164 | 164 | return $details; |
@@ -230,10 +230,10 @@ discard block |
||
230 | 230 | 'WPE_APIKEY' => 'WP Engine', |
231 | 231 | ]; |
232 | 232 | foreach( $checks as $key => $value ) { |
233 | - if( !$this->isWebhostCheckValid( $key ))continue; |
|
233 | + if( !$this->isWebhostCheckValid( $key ) )continue; |
|
234 | 234 | return $value; |
235 | 235 | } |
236 | - return implode( ',', array_filter( [DB_HOST, filter_input( INPUT_SERVER, 'SERVER_NAME' )] )); |
|
236 | + return implode( ',', array_filter( [DB_HOST, filter_input( INPUT_SERVER, 'SERVER_NAME' )] ) ); |
|
237 | 237 | } |
238 | 238 | |
239 | 239 | /** |
@@ -254,8 +254,8 @@ discard block |
||
254 | 254 | { |
255 | 255 | $plugins = get_plugins(); |
256 | 256 | $activePlugins = (array)get_option( 'active_plugins', [] ); |
257 | - $inactive = $this->normalizePluginList( array_diff_key( $plugins, array_flip( $activePlugins ))); |
|
258 | - $active = $this->normalizePluginList( array_diff_key( $plugins, $inactive )); |
|
257 | + $inactive = $this->normalizePluginList( array_diff_key( $plugins, array_flip( $activePlugins ) ) ); |
|
258 | + $active = $this->normalizePluginList( array_diff_key( $plugins, $inactive ) ); |
|
259 | 259 | return $active + $inactive; |
260 | 260 | } |
261 | 261 | |
@@ -266,7 +266,7 @@ discard block |
||
266 | 266 | protected function implode( $title, array $details ) |
267 | 267 | { |
268 | 268 | $strings = ['['.$title.']']; |
269 | - $padding = max( array_map( 'strlen', array_keys( $details )) ); |
|
269 | + $padding = max( array_map( 'strlen', array_keys( $details ) ) ); |
|
270 | 270 | $padding = max( [$padding, static::PAD] ); |
271 | 271 | foreach( $details as $key => $value ) { |
272 | 272 | $strings[] = is_string( $key ) |
@@ -309,9 +309,9 @@ discard block |
||
309 | 309 | $keys = [ |
310 | 310 | 'licenses.', 'submissions.recaptcha.key', 'submissions.recaptcha.secret', |
311 | 311 | ]; |
312 | - array_walk( $settings, function( &$value, $setting ) use( $keys ) { |
|
312 | + array_walk( $settings, function( &$value, $setting ) use($keys) { |
|
313 | 313 | foreach( $keys as $key ) { |
314 | - if( !glsr( Helper::class )->startsWith( $key, $setting ) || empty( $value ))continue; |
|
314 | + if( !glsr( Helper::class )->startsWith( $key, $setting ) || empty($value) )continue; |
|
315 | 315 | $value = str_repeat( '•', 13 ); |
316 | 316 | return; |
317 | 317 | } |
@@ -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 | |
@@ -157,7 +161,9 @@ discard block |
||
157 | 161 | ksort( $settings ); |
158 | 162 | $details = []; |
159 | 163 | foreach( $settings as $key => $value ) { |
160 | - if( $helper->startsWith( 'strings', $key ) && $helper->endsWith( 'id', $key ))continue; |
|
164 | + if( $helper->startsWith( 'strings', $key ) && $helper->endsWith( 'id', $key )) { |
|
165 | + continue; |
|
166 | + } |
|
161 | 167 | $value = htmlspecialchars( trim( preg_replace('/\s\s+/', '\\n', $value )), ENT_QUOTES, 'UTF-8' ); |
162 | 168 | $details[$key] = $value; |
163 | 169 | } |
@@ -230,7 +236,9 @@ discard block |
||
230 | 236 | 'WPE_APIKEY' => 'WP Engine', |
231 | 237 | ]; |
232 | 238 | foreach( $checks as $key => $value ) { |
233 | - if( !$this->isWebhostCheckValid( $key ))continue; |
|
239 | + if( !$this->isWebhostCheckValid( $key )) { |
|
240 | + continue; |
|
241 | + } |
|
234 | 242 | return $value; |
235 | 243 | } |
236 | 244 | return implode( ',', array_filter( [DB_HOST, filter_input( INPUT_SERVER, 'SERVER_NAME' )] )); |
@@ -311,7 +319,9 @@ discard block |
||
311 | 319 | ]; |
312 | 320 | array_walk( $settings, function( &$value, $setting ) use( $keys ) { |
313 | 321 | foreach( $keys as $key ) { |
314 | - if( !glsr( Helper::class )->startsWith( $key, $setting ) || empty( $value ))continue; |
|
322 | + if( !glsr( Helper::class )->startsWith( $key, $setting ) || empty( $value )) { |
|
323 | + continue; |
|
324 | + } |
|
315 | 325 | $value = str_repeat( '•', 13 ); |
316 | 326 | return; |
317 | 327 | } |
@@ -29,7 +29,7 @@ discard block |
||
29 | 29 | */ |
30 | 30 | protected function getFieldDefault( array $field ) |
31 | 31 | { |
32 | - return isset( $field['default'] ) |
|
32 | + return isset($field['default']) |
|
33 | 33 | ? $field['default'] |
34 | 34 | : ''; |
35 | 35 | } |
@@ -40,7 +40,7 @@ discard block |
||
40 | 40 | protected function getSettingFields( $path ) |
41 | 41 | { |
42 | 42 | $settings = glsr( DefaultsManager::class )->settings(); |
43 | - return array_filter( $settings, function( $key ) use( $path ) { |
|
43 | + return array_filter( $settings, function( $key ) use($path) { |
|
44 | 44 | return glsr( Helper::class )->startsWith( $path, $key ); |
45 | 45 | }, ARRAY_FILTER_USE_KEY ); |
46 | 46 | } |
@@ -55,8 +55,8 @@ discard block |
||
55 | 55 | $field = wp_parse_args( $field, [ |
56 | 56 | 'is_setting' => true, |
57 | 57 | 'name' => $name, |
58 | - ]); |
|
59 | - $rows.= new Field( $this->normalize( $field )); |
|
58 | + ] ); |
|
59 | + $rows .= new Field( $this->normalize( $field ) ); |
|
60 | 60 | } |
61 | 61 | return $rows; |
62 | 62 | } |
@@ -67,7 +67,7 @@ discard block |
||
67 | 67 | */ |
68 | 68 | protected function getTemplateData( $id ) |
69 | 69 | { |
70 | - $fields = $this->getSettingFields( $this->normalizeSettingPath( $id )); |
|
70 | + $fields = $this->getSettingFields( $this->normalizeSettingPath( $id ) ); |
|
71 | 71 | return [ |
72 | 72 | 'context' => [ |
73 | 73 | 'rows' => $this->getSettingRows( $fields ), |
@@ -81,12 +81,12 @@ discard block |
||
81 | 81 | */ |
82 | 82 | protected function getTemplateDataForAddons( $id ) |
83 | 83 | { |
84 | - $fields = $this->getSettingFields( $this->normalizeSettingPath( $id )); |
|
84 | + $fields = $this->getSettingFields( $this->normalizeSettingPath( $id ) ); |
|
85 | 85 | $settings = glsr( Helper::class )->convertDotNotationArray( $fields ); |
86 | 86 | $settingKeys = array_keys( $settings['settings']['addons'] ); |
87 | 87 | $results = []; |
88 | 88 | foreach( $settingKeys as $key ) { |
89 | - $addonFields = array_filter( $fields, function( $path ) use( $key ) { |
|
89 | + $addonFields = array_filter( $fields, function( $path ) use($key) { |
|
90 | 90 | return glsr( Helper::class )->startsWith( 'settings.addons.'.$key, $path ); |
91 | 91 | }, ARRAY_FILTER_USE_KEY ); |
92 | 92 | $results[$key] = $this->getSettingRows( $addonFields ); |
@@ -102,7 +102,7 @@ discard block |
||
102 | 102 | protected function getTemplateDataForTranslations() |
103 | 103 | { |
104 | 104 | $translations = glsr( Translator::class )->renderAll(); |
105 | - $class = empty( $translations ) |
|
105 | + $class = empty($translations) |
|
106 | 106 | ? 'glsr-hidden' |
107 | 107 | : ''; |
108 | 108 | return [ |
@@ -125,7 +125,7 @@ discard block |
||
125 | 125 | $path, |
126 | 126 | glsr( Helper::class )->getPathValue( $path, glsr()->defaults ) |
127 | 127 | ); |
128 | - if( is_array( $expectedValue )) { |
|
128 | + if( is_array( $expectedValue ) ) { |
|
129 | 129 | return !in_array( $optionValue, $expectedValue ); |
130 | 130 | } |
131 | 131 | return $optionValue != $expectedValue; |
@@ -147,13 +147,13 @@ discard block |
||
147 | 147 | */ |
148 | 148 | protected function normalizeDependsOn( array $field ) |
149 | 149 | { |
150 | - if( !empty( $field['depends_on'] ) && is_array( $field['depends_on'] )) { |
|
150 | + if( !empty($field['depends_on']) && is_array( $field['depends_on'] ) ) { |
|
151 | 151 | $path = key( $field['depends_on'] ); |
152 | 152 | $expectedValue = $field['depends_on'][$path]; |
153 | - $field['data-depends'] = json_encode([ |
|
153 | + $field['data-depends'] = json_encode( [ |
|
154 | 154 | 'name' => glsr( Helper::class )->convertPathToName( $path, OptionManager::databaseKey() ), |
155 | 155 | 'value' => $expectedValue, |
156 | - ], JSON_HEX_APOS|JSON_HEX_QUOT ); |
|
156 | + ], JSON_HEX_APOS | JSON_HEX_QUOT ); |
|
157 | 157 | $field['is_hidden'] = $this->isFieldHidden( $path, $expectedValue ); |
158 | 158 | } |
159 | 159 | return $field; |
@@ -164,9 +164,9 @@ discard block |
||
164 | 164 | */ |
165 | 165 | protected function normalizeLabelAndLegend( array $field ) |
166 | 166 | { |
167 | - if( !empty( $field['label'] )) { |
|
167 | + if( !empty($field['label']) ) { |
|
168 | 168 | $field['legend'] = $field['label']; |
169 | - unset( $field['label'] ); |
|
169 | + unset($field['label']); |
|
170 | 170 | } |
171 | 171 | else { |
172 | 172 | $field['is_valid'] = false; |
@@ -180,7 +180,7 @@ discard block |
||
180 | 180 | */ |
181 | 181 | protected function normalizeValue( array $field ) |
182 | 182 | { |
183 | - if( !isset( $field['value'] )) { |
|
183 | + if( !isset($field['value']) ) { |
|
184 | 184 | $field['value'] = glsr( OptionManager::class )->get( |
185 | 185 | $field['name'], |
186 | 186 | $this->getFieldDefault( $field ) |
@@ -108,7 +108,7 @@ discard block |
||
108 | 108 | */ |
109 | 109 | public function get() |
110 | 110 | { |
111 | - return empty( $this->log ) |
|
111 | + return empty($this->log) |
|
112 | 112 | ? __( 'Log is empty', 'site-reviews' ) |
113 | 113 | : $this->log; |
114 | 114 | } |
@@ -120,11 +120,11 @@ discard block |
||
120 | 120 | public function humanSize( $valueIfEmpty = null ) |
121 | 121 | { |
122 | 122 | $bytes = $this->size(); |
123 | - if( empty( $bytes ) && is_string( $valueIfEmpty )) { |
|
123 | + if( empty($bytes) && is_string( $valueIfEmpty ) ) { |
|
124 | 124 | return $valueIfEmpty; |
125 | 125 | } |
126 | - $exponent = floor( log( max( $bytes, 1 ), 1024 )); |
|
127 | - return round( $bytes / pow( 1024, $exponent ), 2 ).' '.['bytes','KB','MB','GB'][$exponent]; |
|
126 | + $exponent = floor( log( max( $bytes, 1 ), 1024 ) ); |
|
127 | + return round( $bytes / pow( 1024, $exponent ), 2 ).' '.['bytes', 'KB', 'MB', 'GB'][$exponent]; |
|
128 | 128 | } |
129 | 129 | |
130 | 130 | /** |
@@ -148,9 +148,9 @@ discard block |
||
148 | 148 | { |
149 | 149 | $constants = (new ReflectionClass( __CLASS__ ))->getConstants(); |
150 | 150 | $constants = (array)apply_filters( 'site-reviews/log-levels', $constants ); |
151 | - if( in_array( $level, $constants, true )) { |
|
151 | + if( in_array( $level, $constants, true ) ) { |
|
152 | 152 | $entry = $this->buildLogEntry( $level, $message, $context ); |
153 | - file_put_contents( $this->file, $entry, FILE_APPEND|LOCK_EX ); |
|
153 | + file_put_contents( $this->file, $entry, FILE_APPEND | LOCK_EX ); |
|
154 | 154 | $this->reset(); |
155 | 155 | } |
156 | 156 | return $this; |
@@ -223,7 +223,7 @@ discard block |
||
223 | 223 | */ |
224 | 224 | protected function interpolate( $message, $context = [] ) |
225 | 225 | { |
226 | - if( $this->isObjectOrArray( $message ) || !is_array( $context )) { |
|
226 | + if( $this->isObjectOrArray( $message ) || !is_array( $context ) ) { |
|
227 | 227 | return print_r( $message, true ); |
228 | 228 | } |
229 | 229 | $replace = []; |
@@ -251,7 +251,7 @@ discard block |
||
251 | 251 | if( $value instanceof DateTime ) { |
252 | 252 | $value = $value->format( 'Y-m-d H:i:s' ); |
253 | 253 | } |
254 | - else if( $this->isObjectOrArray( $value )) { |
|
254 | + else if( $this->isObjectOrArray( $value ) ) { |
|
255 | 255 | $value = json_encode( $value ); |
256 | 256 | } |
257 | 257 | return (string)$value; |
@@ -266,7 +266,7 @@ discard block |
||
266 | 266 | $this->clear(); |
267 | 267 | file_put_contents( |
268 | 268 | $this->file, |
269 | - $this->buildLogEntry( 'info', __( 'Console was automatically reset (128 KB max size)', 'site-reviews' )) |
|
269 | + $this->buildLogEntry( 'info', __( 'Console was automatically reset (128 KB max size)', 'site-reviews' ) ) |
|
270 | 270 | ); |
271 | 271 | } |
272 | 272 | } |