@@ -107,7 +107,9 @@ discard block |
||
107 | 107 | */ |
108 | 108 | public function renderReviewEditor( WP_Post $post ) |
109 | 109 | { |
110 | - if( !$this->isReviewPostType( $post ) || $this->isReviewEditable( $post ))return; |
|
110 | + if( !$this->isReviewPostType( $post ) || $this->isReviewEditable( $post )) { |
|
111 | + return; |
|
112 | + } |
|
111 | 113 | glsr()->render( 'partials/editor/review', [ |
112 | 114 | 'post' => $post, |
113 | 115 | ]); |
@@ -119,7 +121,9 @@ discard block |
||
119 | 121 | */ |
120 | 122 | public function renderReviewNotice( WP_Post $post ) |
121 | 123 | { |
122 | - if( !$this->isReviewPostType( $post ) || $this->isReviewEditable( $post ))return; |
|
124 | + if( !$this->isReviewPostType( $post ) || $this->isReviewEditable( $post )) { |
|
125 | + return; |
|
126 | + } |
|
123 | 127 | glsr( Notice::class )->addWarning( __( 'This review is read-only.', 'site-reviews' )); |
124 | 128 | glsr( Html::class )->renderTemplate( 'partials/editor/notice', [ |
125 | 129 | 'context' => [ |
@@ -134,13 +138,19 @@ discard block |
||
134 | 138 | */ |
135 | 139 | public function renderTinymceButton() |
136 | 140 | { |
137 | - if( glsr_current_screen()->base != 'post' )return; |
|
141 | + if( glsr_current_screen()->base != 'post' ) { |
|
142 | + return; |
|
143 | + } |
|
138 | 144 | $shortcodes = []; |
139 | 145 | foreach( glsr()->mceShortcodes as $shortcode => $values ) { |
140 | - if( !apply_filters( sanitize_title( $shortcode ).'_condition', true ))continue; |
|
146 | + if( !apply_filters( sanitize_title( $shortcode ).'_condition', true )) { |
|
147 | + continue; |
|
148 | + } |
|
141 | 149 | $shortcodes[$shortcode] = $values; |
142 | 150 | } |
143 | - if( empty( $shortcodes ))return; |
|
151 | + if( empty( $shortcodes )) { |
|
152 | + return; |
|
153 | + } |
|
144 | 154 | glsr()->render( 'partials/editor/tinymce', [ |
145 | 155 | 'shortcodes' => $shortcodes, |
146 | 156 | ]); |
@@ -72,7 +72,9 @@ discard block |
||
72 | 72 | */ |
73 | 73 | public function send() |
74 | 74 | { |
75 | - if( !$this->message || !$this->subject || !$this->to )return; |
|
75 | + if( !$this->message || !$this->subject || !$this->to ) { |
|
76 | + return; |
|
77 | + } |
|
76 | 78 | $sent = wp_mail( |
77 | 79 | $this->to, |
78 | 80 | $this->subject, |
@@ -90,8 +92,12 @@ discard block |
||
90 | 92 | */ |
91 | 93 | public function buildPlainTextMessage( PHPMailer $phpmailer ) |
92 | 94 | { |
93 | - if( empty( $this->email ))return; |
|
94 | - if( $phpmailer->ContentType === 'text/plain' || !empty( $phpmailer->AltBody ))return; |
|
95 | + if( empty( $this->email )) { |
|
96 | + return; |
|
97 | + } |
|
98 | + if( $phpmailer->ContentType === 'text/plain' || !empty( $phpmailer->AltBody )) { |
|
99 | + return; |
|
100 | + } |
|
95 | 101 | $message = $this->stripHtmlTags( $phpmailer->Body ); |
96 | 102 | $phpmailer->AltBody = apply_filters( 'site-reviews/email/message', $message, 'text', $this ); |
97 | 103 | } |
@@ -29,7 +29,8 @@ |
||
29 | 29 | protected $settings; |
30 | 30 | protected $taxonomy; |
31 | 31 | |
32 | - public function __construct( Application $app ) { |
|
32 | + public function __construct( Application $app ) |
|
33 | + { |
|
33 | 34 | $this->app = $app; |
34 | 35 | $this->admin = $app->make( AdminController::class ); |
35 | 36 | $this->editor = $app->make( EditorController::class ); |
@@ -16,9 +16,13 @@ discard block |
||
16 | 16 | { |
17 | 17 | $queries = []; |
18 | 18 | foreach( $keys as $key ) { |
19 | - if( !array_key_exists( $key, $values ))continue; |
|
19 | + if( !array_key_exists( $key, $values )) { |
|
20 | + continue; |
|
21 | + } |
|
20 | 22 | $methodName = glsr( Helper::class )->buildMethodName( $key, __FUNCTION__ ); |
21 | - if( !method_exists( $this, $methodName ))continue; |
|
23 | + if( !method_exists( $this, $methodName )) { |
|
24 | + continue; |
|
25 | + } |
|
22 | 26 | $query = call_user_func( [$this, $methodName], $values[$key] ); |
23 | 27 | if( is_array( $query )) { |
24 | 28 | $queries[] = $query; |
@@ -92,7 +96,9 @@ discard block |
||
92 | 96 | */ |
93 | 97 | protected function buildQueryAssignedTo( $value ) |
94 | 98 | { |
95 | - if( empty( $value ))return; |
|
99 | + if( empty( $value )) { |
|
100 | + return; |
|
101 | + } |
|
96 | 102 | return [ |
97 | 103 | 'compare' => 'IN', |
98 | 104 | 'key' => 'assigned_to', |
@@ -106,7 +112,9 @@ discard block |
||
106 | 112 | */ |
107 | 113 | protected function buildQueryCategory( $value ) |
108 | 114 | { |
109 | - if( empty( $value ))return; |
|
115 | + if( empty( $value )) { |
|
116 | + return; |
|
117 | + } |
|
110 | 118 | return [ |
111 | 119 | 'field' => 'term_id', |
112 | 120 | 'taxonomy' => Application::TAXONOMY, |
@@ -120,7 +128,9 @@ discard block |
||
120 | 128 | */ |
121 | 129 | protected function buildQueryRating( $value ) |
122 | 130 | { |
123 | - if( !is_numeric( $value ) || !in_array( intval( $value ), range( 1, 5 )))return; |
|
131 | + if( !is_numeric( $value ) || !in_array( intval( $value ), range( 1, 5 ))) { |
|
132 | + return; |
|
133 | + } |
|
124 | 134 | return [ |
125 | 135 | 'compare' => '>=', |
126 | 136 | 'key' => 'rating', |
@@ -134,7 +144,9 @@ discard block |
||
134 | 144 | */ |
135 | 145 | protected function buildQueryType( $value ) |
136 | 146 | { |
137 | - if( in_array( $value, ['','all'] ))return; |
|
147 | + if( in_array( $value, ['','all'] )) { |
|
148 | + return; |
|
149 | + } |
|
138 | 150 | return [ |
139 | 151 | 'key' => 'review_type', |
140 | 152 | 'value' => $value, |
@@ -212,7 +212,9 @@ discard block |
||
212 | 212 | $newParents = $parents; |
213 | 213 | foreach( $parents as $parent ) { |
214 | 214 | $parentClass = glsr( Helper::class )->buildClassName( $parent, __NAMESPACE__ ); |
215 | - if( !class_exists( $parentClass ))continue; |
|
215 | + if( !class_exists( $parentClass )) { |
|
216 | + continue; |
|
217 | + } |
|
216 | 218 | $newParents = array_merge( $newParents, $this->getParents( (new $parentClass)->parents )); |
217 | 219 | } |
218 | 220 | return array_values( array_unique( $newParents )); |
@@ -226,7 +228,9 @@ discard block |
||
226 | 228 | $parents = $this->getParents(); |
227 | 229 | foreach( $parents as $parent ) { |
228 | 230 | $parentClass = glsr( Helper::class )->buildClassName( $parent, __NAMESPACE__ ); |
229 | - if( !class_exists( $parentClass ))continue; |
|
231 | + if( !class_exists( $parentClass )) { |
|
232 | + continue; |
|
233 | + } |
|
230 | 234 | $this->allowed = array_values( array_unique( array_merge( (new $parentClass)->allowed, $this->allowed ))); |
231 | 235 | } |
232 | 236 | } |
@@ -33,7 +33,9 @@ discard block |
||
33 | 33 | foreach( glsr( Database::class )->getReviews( $this->args )->results as $review ) { |
34 | 34 | // Only include critic reviews that have been directly produced by your site, not reviews from third- party sites or syndicated reviews. |
35 | 35 | // @see https://developers.google.com/search/docs/data-types/review |
36 | - if( $review->review_type != 'local' )continue; |
|
36 | + if( $review->review_type != 'local' ) { |
|
37 | + continue; |
|
38 | + } |
|
37 | 39 | $reviews[] = $this->buildReview( $review ); |
38 | 40 | } |
39 | 41 | if( !empty( $reviews )) { |
@@ -78,7 +80,9 @@ discard block |
||
78 | 80 | */ |
79 | 81 | public function render() |
80 | 82 | { |
81 | - if( is_null( glsr()->schemas ))return; |
|
83 | + if( is_null( glsr()->schemas )) { |
|
84 | + return; |
|
85 | + } |
|
82 | 86 | printf( '<script type="application/ld+json">%s</script>', json_encode( |
83 | 87 | apply_filters( 'site-reviews/schema/all', glsr()->schemas ), |
84 | 88 | JSON_UNESCAPED_UNICODE | JSON_UNESCAPED_SLASHES |
@@ -129,7 +133,9 @@ discard block |
||
129 | 133 | { |
130 | 134 | foreach( $values as $value ) { |
131 | 135 | $option = $this->getSchemaOptionValue( $value ); |
132 | - if( empty( $option ))continue; |
|
136 | + if( empty( $option )) { |
|
137 | + continue; |
|
138 | + } |
|
133 | 139 | $schema->$value( $option ); |
134 | 140 | } |
135 | 141 | return $schema; |
@@ -242,7 +248,9 @@ discard block |
||
242 | 248 | if( $value != $fallback ) { |
243 | 249 | return $value; |
244 | 250 | } |
245 | - if( !is_single() && !is_page() )return; |
|
251 | + if( !is_single() && !is_page() ) { |
|
252 | + return; |
|
253 | + } |
|
246 | 254 | $method = glsr( Helper::class )->buildMethodName( $option, 'getThing' ); |
247 | 255 | if( method_exists( $this, $method )) { |
248 | 256 | return $this->$method(); |
@@ -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,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(); |
@@ -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 | } |