Passed
Push — master ( 25e85e...aade3e )
by Paul
03:50
created
plugin/Controllers/AjaxController.php 1 patch
Spacing   +17 added lines, -17 removed lines patch added patch discarded remove patch
@@ -24,7 +24,7 @@  discard block
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
41 41
 	public function routerClearConsole()
42 42
 	{
43 43
 		glsr( AdminController::class )->routerClearConsole();
44
-		wp_send_json([
44
+		wp_send_json( [
45 45
 			'console' => glsr( Console::class )->get(),
46 46
 			'notices' => glsr( Notice::class )->get(),
47
-		]);
47
+		] );
48 48
 	}
49 49
 
50 50
 	/**
@@ -54,9 +54,9 @@  discard block
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 	/**
@@ -107,11 +107,11 @@  discard block
 block discarded – undo
107 107
 	public function routerSubmitReview( array $request )
108 108
 	{
109 109
 		glsr( PublicController::class )->routerSubmitReview( $request );
110
-		wp_send_json([
110
+		wp_send_json( [
111 111
 			'errors' => glsr( Session::class )->get( $request['form_id'].'errors', false, true ),
112
-			'message' => wpautop( glsr( Session::class )->get( $request['form_id'].'message', '', true )),
112
+			'message' => wpautop( glsr( Session::class )->get( $request['form_id'].'message', '', true ) ),
113 113
 			'recaptcha' => glsr( Session::class )->get( $request['form_id'].'recaptcha', false, true ),
114
-		]);
114
+		] );
115 115
 	}
116 116
 
117 117
 	/**
@@ -119,10 +119,10 @@  discard block
 block discarded – undo
119 119
 	 */
120 120
 	public function routerTogglePinned( array $request )
121 121
 	{
122
-		$isPinned = $this->execute( new TogglePinned( $request ));
123
-		wp_send_json([
122
+		$isPinned = $this->execute( new TogglePinned( $request ) );
123
+		wp_send_json( [
124 124
 			'notices' => glsr( Notice::class )->get(),
125 125
 			'pinned' => $isPinned,
126
-		]);
126
+		] );
127 127
 	}
128 128
 }
Please login to merge, or discard this patch.
plugin/Database/QueryBuilder.php 1 patch
Spacing   +14 added lines, -14 removed lines patch added patch discarded remove patch
@@ -16,11 +16,11 @@  discard block
 block discarded – undo
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 ) )continue;
20 20
 			$methodName = glsr( Helper::class )->buildMethodName( $key, __FUNCTION__ );
21
-			if( !method_exists( $this, $methodName ))continue;
21
+			if( !method_exists( $this, $methodName ) )continue;
22 22
 			$query = call_user_func( [$this, $methodName], $values[$key] );
23
-			if( is_array( $query )) {
23
+			if( is_array( $query ) ) {
24 24
 				$queries[] = $query;
25 25
 			}
26 26
 		}
@@ -35,11 +35,11 @@  discard block
 block discarded – undo
35 35
 	 */
36 36
 	public function buildSqlOr( $values, $sprintfFormat )
37 37
 	{
38
-		if( !is_array( $values )) {
38
+		if( !is_array( $values ) ) {
39 39
 			$values = explode( ',', $values );
40 40
 		}
41
-		$values = array_filter( array_map( 'trim', (array)$values ));
42
-		$values = array_map( function( $value ) use( $sprintfFormat ) {
41
+		$values = array_filter( array_map( 'trim', (array)$values ) );
42
+		$values = array_map( function( $value ) use($sprintfFormat) {
43 43
 			return sprintf( $sprintfFormat, $value );
44 44
 		}, $values );
45 45
 		return implode( ' OR ', $values );
@@ -54,11 +54,11 @@  discard block
 block discarded – undo
54 54
 	 */
55 55
 	public function filterSearchByTitle( $search, WP_Query $query )
56 56
 	{
57
-		if( empty( $search ) || empty( $query->get( 'search_terms' ))) {
57
+		if( empty($search) || empty($query->get( 'search_terms' )) ) {
58 58
 			return $search;
59 59
 		}
60 60
 		global $wpdb;
61
-		$n = empty( $query->get( 'exact' ))
61
+		$n = empty($query->get( 'exact' ))
62 62
 			? '%'
63 63
 			: '';
64 64
 		$search = [];
@@ -82,7 +82,7 @@  discard block
 block discarded – undo
82 82
 			? Application::PAGED_QUERY_VAR
83 83
 			: 'page';
84 84
 		return $isEnabled
85
-			? max( 1, intval( get_query_var( $pagedQuery )))
85
+			? max( 1, intval( get_query_var( $pagedQuery ) ) )
86 86
 			: 1;
87 87
 	}
88 88
 
@@ -92,11 +92,11 @@  discard block
 block discarded – undo
92 92
 	 */
93 93
 	protected function buildQueryAssignedTo( $value )
94 94
 	{
95
-		if( empty( $value ))return;
95
+		if( empty($value) )return;
96 96
 		return [
97 97
 			'compare' => 'IN',
98 98
 			'key' => 'assigned_to',
99
-			'value' => array_filter( array_map( 'trim', explode( ',', $value )), 'is_numeric' ),
99
+			'value' => array_filter( array_map( 'trim', explode( ',', $value ) ), 'is_numeric' ),
100 100
 		];
101 101
 	}
102 102
 
@@ -106,7 +106,7 @@  discard block
 block discarded – undo
106 106
 	 */
107 107
 	protected function buildQueryCategory( $value )
108 108
 	{
109
-		if( empty( $value ))return;
109
+		if( empty($value) )return;
110 110
 		return [
111 111
 			'field' => 'term_id',
112 112
 			'taxonomy' => Application::TAXONOMY,
@@ -120,7 +120,7 @@  discard block
 block discarded – undo
120 120
 	 */
121 121
 	protected function buildQueryRating( $value )
122 122
 	{
123
-		if( !is_numeric( $value ) || !in_array( intval( $value ), range( 1, 5 )))return;
123
+		if( !is_numeric( $value ) || !in_array( intval( $value ), range( 1, 5 ) ) )return;
124 124
 		return [
125 125
 			'compare' => '>=',
126 126
 			'key' => 'rating',
@@ -134,7 +134,7 @@  discard block
 block discarded – undo
134 134
 	 */
135 135
 	protected function buildQueryType( $value )
136 136
 	{
137
-		if( in_array( $value, ['','all'] ))return;
137
+		if( in_array( $value, ['', 'all'] ) )return;
138 138
 		return [
139 139
 			'key' => 'review_type',
140 140
 			'value' => $value,
Please login to merge, or discard this patch.
plugin/Modules/Validator/ValidateReview.php 1 patch
Spacing   +22 added lines, -22 removed lines patch added patch discarded remove patch
@@ -61,7 +61,7 @@  discard block
 block discarded – undo
61 61
 		$this->validateBlacklist();
62 62
 		$this->validateAkismet();
63 63
 		$this->validateRecaptcha();
64
-		if( !empty( $this->error )) {
64
+		if( !empty($this->error) ) {
65 65
 			$this->setSessionValues( 'message', $this->error );
66 66
 		}
67 67
 		return $this;
@@ -86,10 +86,10 @@  discard block
 block discarded – undo
86 86
 			apply_filters( 'site-reviews/validation/rules', static::VALIDATION_RULES ),
87 87
 			$this->getOption( 'settings.submissions.required', [] )
88 88
 		);
89
-		$excluded = isset( $request['excluded'] )
89
+		$excluded = isset($request['excluded'])
90 90
 			? (array)json_decode( $request['excluded'] )
91 91
 			: [];
92
-		return array_diff_key( $rules, array_flip( $excluded ));
92
+		return array_diff_key( $rules, array_flip( $excluded ) );
93 93
 	}
94 94
 
95 95
 	/**
@@ -101,14 +101,14 @@  discard block
 block discarded – undo
101 101
 		if( !$integration ) {
102 102
 			return true;
103 103
 		}
104
-		if( empty( $this->request['recaptcha-token'] )) {
104
+		if( empty($this->request['recaptcha-token']) ) {
105 105
 			return null; // @see $this->validateRecaptcha()
106 106
 		}
107 107
 		if( $integration == 'custom' ) {
108 108
 			return $this->isRecaptchaValid( $this->request['recaptcha-token'] );
109 109
 		}
110 110
 		if( $integration == 'invisible-recaptcha' ) {
111
-			return boolval( apply_filters( 'google_invre_is_valid_request_filter', true ));
111
+			return boolval( apply_filters( 'google_invre_is_valid_request_filter', true ) );
112 112
 		}
113 113
 		return false;
114 114
 	}
@@ -119,17 +119,17 @@  discard block
 block discarded – undo
119 119
 	 */
120 120
 	protected function isRecaptchaValid( $recaptchaToken )
121 121
 	{
122
-		$endpoint = add_query_arg([
122
+		$endpoint = add_query_arg( [
123 123
 			'remoteip' => glsr( Helper::class )->getIpAddress(),
124 124
 			'response' => $recaptchaToken,
125 125
 			'secret' => $this->getOption( 'settings.submissions.recaptcha.secret' ),
126 126
 		], static::RECAPTCHA_ENDPOINT );
127
-		if( is_wp_error( $response = wp_remote_get( $endpoint ))) {
127
+		if( is_wp_error( $response = wp_remote_get( $endpoint ) ) ) {
128 128
 			glsr_log()->error( $response->get_error_message() );
129 129
 			return false;
130 130
 		}
131
-		$response = json_decode( wp_remote_retrieve_body( $response ));
132
-		if( !empty( $response->success )) {
131
+		$response = json_decode( wp_remote_retrieve_body( $response ) );
132
+		if( !empty($response->success) ) {
133 133
 			return boolval( $response->success );
134 134
 		}
135 135
 		$errorCodes = [
@@ -152,7 +152,7 @@  discard block
 block discarded – undo
152 152
 	{
153 153
 		$rules = $this->getValidationRules( $request );
154 154
 		$errors = glsr( Validator::class )->validate( $request, $rules );
155
-		if( empty( $errors )) {
155
+		if( empty($errors) ) {
156 156
 			return true;
157 157
 		}
158 158
 		$this->setSessionValues( 'errors', $errors );
@@ -169,7 +169,7 @@  discard block
 block discarded – undo
169 169
 	protected function setSessionValues( $type, $value, $loggedMessage = '' )
170 170
 	{
171 171
 		glsr( Session::class )->set( $this->form_id.$type, $value );
172
-		if( !empty( $loggedMessage )) {
172
+		if( !empty($loggedMessage) ) {
173 173
 			glsr_log()->warning( $loggedMessage );
174 174
 			glsr_log()->warning( $this->request );
175 175
 		}
@@ -180,8 +180,8 @@  discard block
 block discarded – undo
180 180
 	 */
181 181
 	protected function validateAkismet()
182 182
 	{
183
-		if( !empty( $this->error ))return;
184
-		if( !glsr( Akismet::class )->isSpam( $this->request ))return;
183
+		if( !empty($this->error) )return;
184
+		if( !glsr( Akismet::class )->isSpam( $this->request ) )return;
185 185
 		$this->setSessionValues( 'errors', [], 'Akismet caught a spam submission:' );
186 186
 		$this->error = __( 'Your review cannot be submitted at this time. Please try again later.', 'site-reviews' );
187 187
 	}
@@ -191,8 +191,8 @@  discard block
 block discarded – undo
191 191
 	 */
192 192
 	protected function validateBlacklist()
193 193
 	{
194
-		if( !empty( $this->error ))return;
195
-		if( !glsr( Blacklist::class )->isBlacklisted( $this->request ))return;
194
+		if( !empty($this->error) )return;
195
+		if( !glsr( Blacklist::class )->isBlacklisted( $this->request ) )return;
196 196
 		$blacklistAction = $this->getOption( 'settings.submissions.blacklist.action' );
197 197
 		if( $blacklistAction == 'reject' ) {
198 198
 			$this->setSessionValues( 'errors', [], 'Blacklisted submission detected:' );
@@ -207,7 +207,7 @@  discard block
 block discarded – undo
207 207
 	 */
208 208
 	protected function validateCustom()
209 209
 	{
210
-		if( !empty( $this->error ))return;
210
+		if( !empty($this->error) )return;
211 211
 		$validated = apply_filters( 'site-reviews/validate/review/submission', true, $this->request );
212 212
 		if( $validated === true )return;
213 213
 		$this->setSessionValues( 'errors', [] );
@@ -222,8 +222,8 @@  discard block
 block discarded – undo
222 222
 	 */
223 223
 	protected function validateHoneyPot()
224 224
 	{
225
-		if( !empty( $this->error ))return;
226
-		if( empty( $this->request['gotcha'] ))return;
225
+		if( !empty($this->error) )return;
226
+		if( empty($this->request['gotcha']) )return;
227 227
 		$this->setSessionValues( 'errors', [], 'The Honeypot caught a bad submission:' );
228 228
 		$this->error = __( 'The review submission failed. Please notify the site administrator.', 'site-reviews' );
229 229
 	}
@@ -233,9 +233,9 @@  discard block
 block discarded – undo
233 233
 	 */
234 234
 	protected function validateRecaptcha()
235 235
 	{
236
-		if( !empty( $this->error ))return;
236
+		if( !empty($this->error) )return;
237 237
 		$isValid = $this->isRecaptchaResponseValid();
238
-		if( is_null( $isValid )) {
238
+		if( is_null( $isValid ) ) {
239 239
 			$this->setSessionValues( 'recaptcha', true );
240 240
 			$this->recaptchaIsUnset = true;
241 241
 		}
@@ -251,11 +251,11 @@  discard block
 block discarded – undo
251 251
 	 */
252 252
 	protected function validateRequest( array $request )
253 253
 	{
254
-		if( !$this->isRequestValid( $request )) {
254
+		if( !$this->isRequestValid( $request ) ) {
255 255
 			$this->error = __( 'Please fix the submission errors.', 'site-reviews' );
256 256
 			return $request;
257 257
 		}
258
-		if( empty( $request['title'] )) {
258
+		if( empty($request['title']) ) {
259 259
 			$request['title'] = __( 'No Title', 'site-reviews' );
260 260
 		}
261 261
 		return array_merge( glsr( ValidateReviewDefaults::class )->defaults(), $request );
Please login to merge, or discard this patch.
plugin/Modules/Schema/BaseType.php 1 patch
Spacing   +11 added lines, -11 removed lines patch added patch discarded remove patch
@@ -93,7 +93,7 @@  discard block
 block discarded – undo
93 93
 	 * @param mixed $default
94 94
 	 * @return mixed
95 95
 	 */
96
-	public function getProperty( $property, $default = null)
96
+	public function getProperty( $property, $default = null )
97 97
 	{
98 98
 		return $this->properties[$property] ?? $default;
99 99
 	}
@@ -161,7 +161,7 @@  discard block
 block discarded – undo
161 161
 	 */
162 162
 	public function offsetUnset( $offset )
163 163
 	{
164
-		unset( $this->properties[$offset] );
164
+		unset($this->properties[$offset]);
165 165
 	}
166 166
 
167 167
 	/**
@@ -207,16 +207,16 @@  discard block
 block discarded – undo
207 207
 	 */
208 208
 	protected function getParents( $parents = null )
209 209
 	{
210
-		if( !isset( $parents )) {
210
+		if( !isset($parents) ) {
211 211
 			$parents = $this->parents;
212 212
 		}
213 213
 		$newParents = $parents;
214 214
 		foreach( $parents as $parent ) {
215 215
 			$parentClass = glsr( Helper::class )->buildClassName( $parent, __NAMESPACE__ );
216
-			if( !class_exists( $parentClass ))continue;
217
-			$newParents = array_merge( $newParents, $this->getParents( (new $parentClass)->parents ));
216
+			if( !class_exists( $parentClass ) )continue;
217
+			$newParents = array_merge( $newParents, $this->getParents( (new $parentClass)->parents ) );
218 218
 		}
219
-		return array_values( array_unique( $newParents ));
219
+		return array_values( array_unique( $newParents ) );
220 220
 	}
221 221
 
222 222
 	/**
@@ -227,8 +227,8 @@  discard block
 block discarded – undo
227 227
 		$parents = $this->getParents();
228 228
 		foreach( $parents as $parent ) {
229 229
 			$parentClass = glsr( Helper::class )->buildClassName( $parent, __NAMESPACE__ );
230
-			if( !class_exists( $parentClass ))continue;
231
-			$this->allowed = array_values( array_unique( array_merge( (new $parentClass)->allowed, $this->allowed )));
230
+			if( !class_exists( $parentClass ) )continue;
231
+			$this->allowed = array_values( array_unique( array_merge( (new $parentClass)->allowed, $this->allowed ) ) );
232 232
 		}
233 233
 	}
234 234
 
@@ -238,17 +238,17 @@  discard block
 block discarded – undo
238 238
 	 */
239 239
 	protected function serializeProperty( $property )
240 240
 	{
241
-		if( is_array( $property )) {
241
+		if( is_array( $property ) ) {
242 242
 			return array_map( [$this, 'serializeProperty'], $property );
243 243
 		}
244 244
 		if( $property instanceof Type ) {
245 245
 			$property = $property->toArray();
246
-			unset( $property['@context'] );
246
+			unset($property['@context']);
247 247
 		}
248 248
 		if( $property instanceof DateTimeInterface ) {
249 249
 			$property = $property->format( DateTime::ATOM );
250 250
 		}
251
-		if( is_object( $property )) {
251
+		if( is_object( $property ) ) {
252 252
 			throw new InvalidProperty();
253 253
 		}
254 254
 		return $property;
Please login to merge, or discard this patch.
plugin/Modules/Email.php 1 patch
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -90,8 +90,8 @@  discard block
 block discarded – undo
90 90
 	 */
91 91
 	public function buildPlainTextMessage( PHPMailer $phpmailer )
92 92
 	{
93
-		if( empty( $this->email ))return;
94
-		if( $phpmailer->ContentType === 'text/plain' || !empty( $phpmailer->AltBody ))return;
93
+		if( empty($this->email) )return;
94
+		if( $phpmailer->ContentType === 'text/plain' || !empty($phpmailer->AltBody) )return;
95 95
 		$message = $this->stripHtmlTags( $phpmailer->Body );
96 96
 		$phpmailer->AltBody = apply_filters( 'site-reviews/email/message', $message, 'text', $this );
97 97
 	}
@@ -104,10 +104,10 @@  discard block
 block discarded – undo
104 104
 		$allowed = [
105 105
 			'bcc', 'cc', 'from', 'reply-to',
106 106
 		];
107
-		$headers = array_intersect_key( $this->email, array_flip( $allowed ));
107
+		$headers = array_intersect_key( $this->email, array_flip( $allowed ) );
108 108
 		$headers = array_filter( $headers );
109 109
 		foreach( $headers as $key => $value ) {
110
-			unset( $headers[$key] );
110
+			unset($headers[$key]);
111 111
 			$headers[] = $key.': '.$value;
112 112
 		}
113 113
 		$headers[] = 'Content-Type: text/html';
@@ -119,16 +119,16 @@  discard block
 block discarded – undo
119 119
 	 */
120 120
 	protected function buildHtmlMessage()
121 121
 	{
122
-		$template = trim( glsr( OptionManager::class )->get( 'settings.general.notification_message' ));
123
-		if( !empty( $template )) {
122
+		$template = trim( glsr( OptionManager::class )->get( 'settings.general.notification_message' ) );
123
+		if( !empty($template) ) {
124 124
 			$message = glsr( Template::class )->interpolate( $template, $this->email['template-tags'] );
125 125
 		}
126 126
 		else if( $this->email['template'] ) {
127 127
 			$message = glsr( Template::class )->build( 'templates/'.$this->email['template'], [
128 128
 				'context' => $this->email['template-tags'],
129
-			]);
129
+			] );
130 130
 		}
131
-		if( !isset( $message )) {
131
+		if( !isset($message) ) {
132 132
 			$message = $this->email['message'];
133 133
 		}
134 134
 		$message = $this->email['before'].$message.$this->email['after'];
@@ -139,7 +139,7 @@  discard block
 block discarded – undo
139 139
 		$message = str_replace( ']]>', ']]&gt;', $message );
140 140
 		$message = glsr( Template::class )->build( 'partials/email/index', [
141 141
 			'context' => ['message' => $message],
142
-		]);
142
+		] );
143 143
 		return apply_filters( 'site-reviews/email/message', stripslashes( $message ), 'html', $this );
144 144
 	}
145 145
 
@@ -149,7 +149,7 @@  discard block
 block discarded – undo
149 149
 	protected function normalize( array $email = [] )
150 150
 	{
151 151
 		$email = shortcode_atts( glsr( EmailDefaults::class )->defaults(), $email );
152
-		if( empty( $email['reply-to'] )) {
152
+		if( empty($email['reply-to']) ) {
153 153
 			$email['reply-to'] = $email['from'];
154 154
 		}
155 155
 		$this->email = apply_filters( 'site-reviews/email/compose', $email, $this );
Please login to merge, or discard this patch.
plugin/Modules/Validator/ValidationRules.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -127,8 +127,8 @@
 block discarded – undo
127 127
 	public function validateRequired( $value )
128 128
 	{
129 129
 		return is_null( $value )
130
-			|| ( is_string( $value ) && trim( $value ) === '' )
131
-			|| ( is_array( $value ) && count( $value ) < 1 )
130
+			|| (is_string( $value ) && trim( $value ) === '')
131
+			|| (is_array( $value ) && count( $value ) < 1)
132 132
 			? false
133 133
 			: true;
134 134
 	}
Please login to merge, or discard this patch.
plugin/Modules/Validator.php 1 patch
Spacing   +17 added lines, -17 removed lines patch added patch discarded remove patch
@@ -73,7 +73,7 @@  discard block
 block discarded – undo
73 73
 		foreach( $this->rules as $attribute => $rules ) {
74 74
 			foreach( $rules as $rule ) {
75 75
 				$this->validateAttribute( $attribute, $rule );
76
-				if( $this->shouldStopValidating( $attribute ))break;
76
+				if( $this->shouldStopValidating( $attribute ) )break;
77 77
 			}
78 78
 		}
79 79
 		return $this->errors;
@@ -88,13 +88,13 @@  discard block
 block discarded – undo
88 88
 	 */
89 89
 	public function validateAttribute( $attribute, $rule )
90 90
 	{
91
-		list( $rule, $parameters ) = $this->parseRule( $rule );
91
+		list($rule, $parameters) = $this->parseRule( $rule );
92 92
 		if( $rule == '' )return;
93 93
 		$value = $this->getValue( $attribute );
94
-		if( !method_exists( $this, $method = 'validate'.$rule )) {
94
+		if( !method_exists( $this, $method = 'validate'.$rule ) ) {
95 95
 			throw new BadMethodCallException( "Method [$method] does not exist." );
96 96
 		}
97
-		if( !$this->$method( $value, $attribute, $parameters )) {
97
+		if( !$this->$method( $value, $attribute, $parameters ) ) {
98 98
 			$this->addFailure( $attribute, $rule, $parameters );
99 99
 		}
100 100
 	}
@@ -143,7 +143,7 @@  discard block
 block discarded – undo
143 143
 	 */
144 144
 	protected function getMessage( $attribute, $rule, array $parameters )
145 145
 	{
146
-		if( in_array( $rule, $this->sizeRules )) {
146
+		if( in_array( $rule, $this->sizeRules ) ) {
147 147
 			return $this->getSizeMessage( $attribute, $rule, $parameters );
148 148
 		}
149 149
 		$lowerRule = glsr( Helper::class )->snakeCase( $rule );
@@ -158,11 +158,11 @@  discard block
 block discarded – undo
158 158
 	 */
159 159
 	protected function getRule( $attribute, $rules )
160 160
 	{
161
-		if( !array_key_exists( $attribute, $this->rules ))return;
162
-		$rules = (array) $rules;
161
+		if( !array_key_exists( $attribute, $this->rules ) )return;
162
+		$rules = (array)$rules;
163 163
 		foreach( $this->rules[$attribute] as $rule ) {
164
-			list( $rule, $parameters ) = $this->parseRule( $rule );
165
-			if( in_array( $rule, $rules )) {
164
+			list($rule, $parameters) = $this->parseRule( $rule );
165
+			if( in_array( $rule, $rules ) ) {
166 166
 				return [$rule, $parameters];
167 167
 			}
168 168
 		}
@@ -180,7 +180,7 @@  discard block
 block discarded – undo
180 180
 		if( is_numeric( $value ) && $hasNumeric ) {
181 181
 			return $value;
182 182
 		}
183
-		elseif( is_array( $value )) {
183
+		elseif( is_array( $value ) ) {
184 184
 			return count( $value );
185 185
 		}
186 186
 		return mb_strlen( $value );
@@ -207,7 +207,7 @@  discard block
 block discarded – undo
207 207
 	 */
208 208
 	protected function getValue( $attribute )
209 209
 	{
210
-		if( isset( $this->data[$attribute] )) {
210
+		if( isset($this->data[$attribute]) ) {
211 211
 			return $this->data[$attribute];
212 212
 		}
213 213
 	}
@@ -220,7 +220,7 @@  discard block
 block discarded – undo
220 220
 	 */
221 221
 	protected function hasRule( $attribute, $rules )
222 222
 	{
223
-		return !is_null( $this->getRule( $attribute, $rules ));
223
+		return !is_null( $this->getRule( $attribute, $rules ) );
224 224
 	}
225 225
 
226 226
 	/**
@@ -259,11 +259,11 @@  discard block
 block discarded – undo
259 259
 		$parameters = [];
260 260
 		// example: {rule}:{parameters}
261 261
 		if( strpos( $rule, ':' ) !== false ) {
262
-			list( $rule, $parameter ) = explode( ':', $rule, 2 );
262
+			list($rule, $parameter) = explode( ':', $rule, 2 );
263 263
 			// example: {parameter1,parameter2,...}
264 264
 			$parameters = $this->parseParameters( $rule, $parameter );
265 265
 		}
266
-		$rule = ucwords( str_replace( ['-', '_'], ' ', trim( $rule )));
266
+		$rule = ucwords( str_replace( ['-', '_'], ' ', trim( $rule ) ) );
267 267
 		$rule = str_replace( ' ', '', $rule );
268 268
 		return [$rule, $parameters];
269 269
 	}
@@ -290,7 +290,7 @@  discard block
 block discarded – undo
290 290
 	protected function shouldStopValidating( $attribute )
291 291
 	{
292 292
 		return $this->hasRule( $attribute, $this->implicitRules )
293
-			&& isset( $this->failedRules[$attribute] )
293
+			&& isset($this->failedRules[$attribute])
294 294
 			&& array_intersect( array_keys( $this->failedRules[$attribute] ), $this->implicitRules );
295 295
 	}
296 296
 
@@ -315,11 +315,11 @@  discard block
 block discarded – undo
315 315
 			'regex' => __( 'The format is invalid.', 'site-reviews' ),
316 316
 			'required' => __( 'This field is required.', 'site-reviews' ),
317 317
 		];
318
-		$message = isset( $strings[$key] )
318
+		$message = isset($strings[$key])
319 319
 			? $strings[$key]
320 320
 			: false;
321 321
 		if( !$message )return;
322
-		if( method_exists( $this, $method = 'replace'.$rule )) {
322
+		if( method_exists( $this, $method = 'replace'.$rule ) ) {
323 323
 			$message = $this->$method( $message, $parameters );
324 324
 		}
325 325
 		return $message;
Please login to merge, or discard this patch.
plugin/Helper.php 1 patch
Spacing   +20 added lines, -20 removed lines patch added patch discarded remove patch
@@ -14,11 +14,11 @@  discard block
 block discarded – undo
14 14
 	 */
15 15
 	public function buildClassName( $name, $path = '' )
16 16
 	{
17
-		$className = explode( '_', $this->snakeCase( $name ));
17
+		$className = explode( '_', $this->snakeCase( $name ) );
18 18
 		$className = array_map( 'ucfirst', $className );
19 19
 		$className = implode( '', $className );
20 20
 		$path = ltrim( str_replace( __NAMESPACE__, '', $path ), '\\' );
21
-		return !empty( $path )
21
+		return !empty($path)
22 22
 			? __NAMESPACE__.'\\'.$path.'\\'.$className
23 23
 			: $className;
24 24
 	}
@@ -30,7 +30,7 @@  discard block
 block discarded – undo
30 30
 	 */
31 31
 	public function buildMethodName( $name, $prefix = '' )
32 32
 	{
33
-		return lcfirst( $prefix.$this->buildClassName( $name ));
33
+		return lcfirst( $prefix.$this->buildClassName( $name ) );
34 34
 	}
35 35
 
36 36
 	/**
@@ -39,7 +39,7 @@  discard block
 block discarded – undo
39 39
 	 */
40 40
 	public function buildPropertyName( $name )
41 41
 	{
42
-		return lcfirst( $this->buildClassName( $name ));
42
+		return lcfirst( $this->buildClassName( $name ) );
43 43
 	}
44 44
 
45 45
 	/**
@@ -70,7 +70,7 @@  discard block
 block discarded – undo
70 70
 	 */
71 71
 	public function convertPathToId( $path, $prefix = '' )
72 72
 	{
73
-		return str_replace( ['[', ']'], ['-', ''], $this->convertPathToName( $path, $prefix ));
73
+		return str_replace( ['[', ']'], ['-', ''], $this->convertPathToName( $path, $prefix ) );
74 74
 	}
75 75
 
76 76
 	/**
@@ -81,7 +81,7 @@  discard block
 block discarded – undo
81 81
 	{
82 82
 		$levels = explode( '.', $path );
83 83
 		return array_reduce( $levels, function( $result, $value ) {
84
-			return $result.= '['.$value.']';
84
+			return $result .= '['.$value.']';
85 85
 		}, $prefix );
86 86
 	}
87 87
 
@@ -91,7 +91,7 @@  discard block
 block discarded – undo
91 91
 	 */
92 92
 	public function dashCase( $string )
93 93
 	{
94
-		return str_replace( '_', '-', $this->snakeCase( $string ));
94
+		return str_replace( '_', '-', $this->snakeCase( $string ) );
95 95
 	}
96 96
 
97 97
 	/**
@@ -113,11 +113,11 @@  discard block
 block discarded – undo
113 113
 	 */
114 114
 	public function filterInput( $key, array $request = [] )
115 115
 	{
116
-		if( isset( $request[$key] )) {
116
+		if( isset($request[$key]) ) {
117 117
 			return $request[$key];
118 118
 		}
119 119
 		$variable = filter_input( INPUT_POST, $key );
120
-		if( is_null( $variable ) && isset( $_POST[$key] )) {
120
+		if( is_null( $variable ) && isset($_POST[$key]) ) {
121 121
 			$variable = $_POST[$key];
122 122
 		}
123 123
 		return $variable;
@@ -130,7 +130,7 @@  discard block
 block discarded – undo
130 130
 	public function filterInputArray( $key )
131 131
 	{
132 132
 		$variable = filter_input( INPUT_POST, $key, FILTER_DEFAULT, FILTER_REQUIRE_ARRAY );
133
-		if( empty( $variable ) && !empty( $_POST[$key] ) && is_array( $_POST[$key] )) {
133
+		if( empty($variable) && !empty($_POST[$key]) && is_array( $_POST[$key] ) ) {
134 134
 			$variable = $_POST[$key];
135 135
 		}
136 136
 		return (array)$variable;
@@ -146,13 +146,13 @@  discard block
 block discarded – undo
146 146
 		$result = [];
147 147
 		foreach( $array as $key => $value ) {
148 148
 			$newKey = ltrim( $prefix.'.'.$key, '.' );
149
-			if( $this->isIndexedFlatArray( $value )) {
149
+			if( $this->isIndexedFlatArray( $value ) ) {
150 150
 				if( $flattenValue ) {
151 151
 					$value = '['.implode( ', ', $value ).']';
152 152
 				}
153 153
 			}
154
-			else if( is_array( $value )) {
155
-				$result = array_merge( $result, $this->flattenArray( $value, $flattenValue, $newKey ));
154
+			else if( is_array( $value ) ) {
155
+				$result = array_merge( $result, $this->flattenArray( $value, $flattenValue, $newKey ) );
156 156
 				continue;
157 157
 			}
158 158
 			$result[$newKey] = $value;
@@ -171,7 +171,7 @@  discard block
 block discarded – undo
171 171
 				Whip::IPV4 => $cloudflareIps['v4'],
172 172
 				Whip::IPV6 => $cloudflareIps['v6'],
173 173
 			],
174
-		]))->getValidIpAddress();
174
+		] ))->getValidIpAddress();
175 175
 	}
176 176
 
177 177
 	/**
@@ -184,7 +184,7 @@  discard block
 block discarded – undo
184 184
 	{
185 185
 		$keys = explode( '.', $path );
186 186
 		foreach( $keys as $key ) {
187
-			if( !isset( $values[$key] )) {
187
+			if( !isset($values[$key]) ) {
188 188
 				return $fallback;
189 189
 			}
190 190
 			$values = $values[$key];
@@ -198,7 +198,7 @@  discard block
 block discarded – undo
198 198
 	 */
199 199
 	public function isIndexedArray( $array )
200 200
 	{
201
-		if( !is_array( $array )) {
201
+		if( !is_array( $array ) ) {
202 202
 			return false;
203 203
 		}
204 204
 		$current = 0;
@@ -217,7 +217,7 @@  discard block
 block discarded – undo
217 217
 	 */
218 218
 	public function isIndexedFlatArray( $array )
219 219
 	{
220
-		if( !is_array( $array ) || array_filter( $array, 'is_array' )) {
220
+		if( !is_array( $array ) || array_filter( $array, 'is_array' ) ) {
221 221
 			return false;
222 222
 		}
223 223
 		return $this->isIndexedArray( $array );
@@ -230,7 +230,7 @@  discard block
 block discarded – undo
230 230
 	 */
231 231
 	public function prefixString( $string, $prefix = '' )
232 232
 	{
233
-		return $prefix.str_replace( $prefix, '', trim( $string ));
233
+		return $prefix.str_replace( $prefix, '', trim( $string ) );
234 234
 	}
235 235
 
236 236
 	/**
@@ -276,7 +276,7 @@  discard block
 block discarded – undo
276 276
 	 */
277 277
 	public function snakeCase( $string )
278 278
 	{
279
-		if( !ctype_lower( $string )) {
279
+		if( !ctype_lower( $string ) ) {
280 280
 			$string = preg_replace( '/\s+/u', '', $string );
281 281
 			$string = preg_replace( '/(.)(?=[A-Z])/u', '$1_', $string );
282 282
 			$string = mb_strtolower( $string, 'UTF-8' );
@@ -291,6 +291,6 @@  discard block
 block discarded – undo
291 291
 	 */
292 292
 	public function startsWith( $needle, $haystack )
293 293
 	{
294
-		return substr( $haystack, 0, strlen( $needle )) === $needle;
294
+		return substr( $haystack, 0, strlen( $needle ) ) === $needle;
295 295
 	}
296 296
 }
Please login to merge, or discard this patch.
plugin/Modules/Schema.php 1 patch
Spacing   +29 added lines, -29 removed lines patch added patch discarded remove patch
@@ -36,10 +36,10 @@  discard block
 block discarded – undo
36 36
 			if( $review->review_type != 'local' )continue;
37 37
 			$reviews[] = $this->buildReview( $review );
38 38
 		}
39
-		if( !empty( $reviews )) {
39
+		if( !empty($reviews) ) {
40 40
 			array_walk( $reviews, function( &$review ) {
41
-				unset( $review['@context'] );
42
-				unset( $review['itemReviewed'] );
41
+				unset($review['@context']);
42
+				unset($review['itemReviewed']);
43 43
 			});
44 44
 			$schema['review'] = $reviews;
45 45
 		}
@@ -52,7 +52,7 @@  discard block
 block discarded – undo
52 52
 	 */
53 53
 	public function buildSummary( $args = null )
54 54
 	{
55
-		if( is_array( $args )) {
55
+		if( is_array( $args ) ) {
56 56
 			$this->args = $args;
57 57
 		}
58 58
 		$buildSummary = glsr( Helper::class )->buildMethodName( $this->getSchemaOptionValue( 'type' ), 'buildSummaryFor' );
@@ -60,7 +60,7 @@  discard block
 block discarded – undo
60 60
 		$schema = method_exists( $this, $buildSummary )
61 61
 			? $this->$buildSummary()
62 62
 			: $this->buildSummaryForCustom();
63
-		if( !empty( $count )) {
63
+		if( !empty($count) ) {
64 64
 			$schema->aggregateRating(
65 65
 				$this->getSchemaType( 'AggregateRating' )
66 66
 					->ratingValue( $this->getRatingValue() )
@@ -79,11 +79,11 @@  discard block
 block discarded – undo
79 79
 	 */
80 80
 	public function render()
81 81
 	{
82
-		if( is_null( glsr()->schemas ))return;
82
+		if( is_null( glsr()->schemas ) )return;
83 83
 		printf( '<script type="application/ld+json">%s</script>', json_encode(
84 84
 			apply_filters( 'site-reviews/schema/all', glsr()->schemas ),
85 85
 			JSON_UNESCAPED_UNICODE | JSON_UNESCAPED_SLASHES
86
-		));
86
+		) );
87 87
 	}
88 88
 
89 89
 	/**
@@ -93,7 +93,7 @@  discard block
 block discarded – undo
93 93
 	{
94 94
 		$schemas = glsr()->schemas;
95 95
 		$schemas[] = $schema;
96
-		glsr()->schemas = array_map( 'unserialize', array_unique( array_map( 'serialize', $schemas )));
96
+		glsr()->schemas = array_map( 'unserialize', array_unique( array_map( 'serialize', $schemas ) ) );
97 97
 	}
98 98
 
99 99
 	/**
@@ -103,16 +103,16 @@  discard block
 block discarded – undo
103 103
 	protected function buildReview( $review )
104 104
 	{
105 105
 		$schema = $this->getSchemaType( 'Review' )
106
-			->doIf( !in_array( 'title', $this->args['hide'] ), function( $schema ) use( $review ) {
106
+			->doIf( !in_array( 'title', $this->args['hide'] ), function( $schema ) use($review) {
107 107
 				$schema->name( $review->title );
108 108
 			})
109
-			->doIf( !in_array( 'excerpt', $this->args['hide'] ), function( $schema ) use( $review ) {
109
+			->doIf( !in_array( 'excerpt', $this->args['hide'] ), function( $schema ) use($review) {
110 110
 				$schema->reviewBody( $review->content );
111 111
 			})
112
-			->datePublished(( new DateTime( $review->date )))
113
-			->author( $this->getSchemaType( 'Person' )->name( $review->author ))
114
-			->itemReviewed( $this->getSchemaType()->name( $this->getSchemaOptionValue( 'name' )));
115
-		if( !empty( $review->rating )) {
112
+			->datePublished( (new DateTime( $review->date )) )
113
+			->author( $this->getSchemaType( 'Person' )->name( $review->author ) )
114
+			->itemReviewed( $this->getSchemaType()->name( $this->getSchemaOptionValue( 'name' ) ) );
115
+		if( !empty($review->rating) ) {
116 116
 			$schema->reviewRating(
117 117
 				$this->getSchemaType( 'Rating' )
118 118
 					->ratingValue( $review->rating )
@@ -131,7 +131,7 @@  discard block
 block discarded – undo
131 131
 	{
132 132
 		foreach( $values as $value ) {
133 133
 			$option = $this->getSchemaOptionValue( $value );
134
-			if( empty( $option ))continue;
134
+			if( empty($option) )continue;
135 135
 			$schema->$value( $option );
136 136
 		}
137 137
 		return $schema;
@@ -144,7 +144,7 @@  discard block
 block discarded – undo
144 144
 	{
145 145
 		return $this->buildSchemaValues( $this->getSchemaType(), [
146 146
 			'description', 'image', 'name', 'url',
147
-		]);
147
+		] );
148 148
 	}
149 149
 
150 150
 	/**
@@ -154,7 +154,7 @@  discard block
 block discarded – undo
154 154
 	{
155 155
 		return $this->buildSchemaValues( $this->buildSummaryForCustom(), [
156 156
 			'address', 'priceRange', 'telephone',
157
-		]);
157
+		] );
158 158
 	}
159 159
 
160 160
 	/**
@@ -164,10 +164,10 @@  discard block
 block discarded – undo
164 164
 	{
165 165
 		$offers = $this->buildSchemaValues( $this->getSchemaType( 'AggregateOffer' ), [
166 166
 			'highPrice', 'lowPrice', 'priceCurrency',
167
-		]);
167
+		] );
168 168
 		return $this->buildSummaryForCustom()
169 169
 			->offers( $offers )
170
-			->setProperty( '@id', $this->getSchemaOptionValue( 'url' ));
170
+			->setProperty( '@id', $this->getSchemaOptionValue( 'url' ) );
171 171
 	}
172 172
 
173 173
 	/**
@@ -191,7 +191,7 @@  discard block
 block discarded – undo
191 191
 	 */
192 192
 	protected function getReviews( $force = false )
193 193
 	{
194
-		if( !isset( $this->reviews ) || $force ) {
194
+		if( !isset($this->reviews) || $force ) {
195 195
 			$args = wp_parse_args( ['count' => -1], $this->args );
196 196
 			$this->reviews = glsr( Database::class )->getReviews( $args )->results;
197 197
 		}
@@ -206,14 +206,14 @@  discard block
 block discarded – undo
206 206
 	protected function getSchemaOption( $option, $fallback )
207 207
 	{
208 208
 		$option = strtolower( $option );
209
-		if( $schemaOption = trim( (string)get_post_meta( intval( get_the_ID() ), 'schema_'.$option, true ))) {
209
+		if( $schemaOption = trim( (string)get_post_meta( intval( get_the_ID() ), 'schema_'.$option, true ) ) ) {
210 210
 			return $schemaOption;
211 211
 		}
212 212
 		$setting = glsr( OptionManager::class )->get( 'settings.schema.'.$option );
213
-		if( is_array( $setting )) {
213
+		if( is_array( $setting ) ) {
214 214
 			return $this->getSchemaOptionDefault( $setting, $fallback );
215 215
 		}
216
-		return !empty( $setting )
216
+		return !empty($setting)
217 217
 			? $setting
218 218
 			: $fallback;
219 219
 	}
@@ -227,7 +227,7 @@  discard block
 block discarded – undo
227 227
 		$setting = wp_parse_args( $setting, [
228 228
 			'custom' => '',
229 229
 			'default' => $fallback,
230
-		]);
230
+		] );
231 231
 		return $setting['default'] != 'custom'
232 232
 			? $setting['default']
233 233
 			: $setting['custom'];
@@ -246,7 +246,7 @@  discard block
 block discarded – undo
246 246
 		}
247 247
 		if( !is_single() && !is_page() )return;
248 248
 		$method = glsr( Helper::class )->buildMethodName( $option, 'getThing' );
249
-		if( method_exists( $this, $method )) {
249
+		if( method_exists( $this, $method ) ) {
250 250
 			return $this->$method();
251 251
 		}
252 252
 	}
@@ -257,7 +257,7 @@  discard block
 block discarded – undo
257 257
 	 */
258 258
 	protected function getSchemaType( $type = null )
259 259
 	{
260
-		if( !is_string( $type )) {
260
+		if( !is_string( $type ) ) {
261 261
 			$type = $this->getSchemaOption( 'type', 'LocalBusiness' );
262 262
 		}
263 263
 		$className = glsr( Helper::class )->buildClassName( $type, 'Modules\Schema' );
@@ -272,11 +272,11 @@  discard block
 block discarded – undo
272 272
 	protected function getThingDescription()
273 273
 	{
274 274
 		$post = get_post();
275
-		if( !( $post instanceof WP_Post )) {
275
+		if( !($post instanceof WP_Post) ) {
276 276
 			return '';
277 277
 		}
278
-		$text = strip_shortcodes( wp_strip_all_tags( $post->post_excerpt ));
279
-		return wp_trim_words( $text, apply_filters( 'excerpt_length', 55 ));
278
+		$text = strip_shortcodes( wp_strip_all_tags( $post->post_excerpt ) );
279
+		return wp_trim_words( $text, apply_filters( 'excerpt_length', 55 ) );
280 280
 	}
281 281
 
282 282
 	/**
Please login to merge, or discard this patch.