Passed
Push — master ( 554ad1...33081a )
by Paul
04:37
created
plugin/Shortcodes/ButtonGenerator.php 1 patch
Spacing   +24 added lines, -24 removed lines patch added patch discarded remove patch
@@ -47,7 +47,7 @@  discard block
 block discarded – undo
47 47
 			'label' => '['.$tag.']',
48 48
 			'required' => $this->required,
49 49
 			'title' => esc_html__( 'Shortcode', 'site-reviews' ),
50
-		]);
50
+		] );
51 51
 		return $this;
52 52
 	}
53 53
 
@@ -57,12 +57,12 @@  discard block
 block discarded – undo
57 57
 	protected function generateFields( array $fields )
58 58
 	{
59 59
 		$generatedFields = array_map( function( $field ) {
60
-			if( empty( $field ))return;
60
+			if( empty($field) )return;
61 61
 			$field = $this->normalize( $field );
62
-			if( !method_exists( $this, $method = 'normalize'.ucfirst( $field['type'] )))return;
62
+			if( !method_exists( $this, $method = 'normalize'.ucfirst( $field['type'] ) ) )return;
63 63
 			return $this->$method( $field );
64 64
 		}, $fields );
65
-		return array_values( array_filter( $generatedFields ));
65
+		return array_values( array_filter( $generatedFields ) );
66 66
 	}
67 67
 
68 68
 	/**
@@ -71,15 +71,15 @@  discard block
 block discarded – undo
71 71
 	protected function getFields()
72 72
 	{
73 73
 		$fields = $this->generateFields( $this->fields() );
74
-		if( !empty( $this->errors )) {
74
+		if( !empty($this->errors) ) {
75 75
 			$errors = [];
76 76
 			foreach( $this->required as $name => $alert ) {
77
-				if( false !== array_search( $name, array_column( $fields, 'name' )))continue;
77
+				if( false !== array_search( $name, array_column( $fields, 'name' ) ) )continue;
78 78
 				$errors[] = $this->errors[$name];
79 79
 			}
80 80
 			$this->errors = $errors;
81 81
 		}
82
-		return empty( $this->errors )
82
+		return empty($this->errors)
83 83
 			? $fields
84 84
 			: $this->errors;
85 85
 	}
@@ -92,7 +92,7 @@  discard block
 block discarded – undo
92 92
 		return wp_parse_args( $field, [
93 93
 			'items' => [],
94 94
 			'type' => '',
95
-		]);
95
+		] );
96 96
 	}
97 97
 
98 98
 	/**
@@ -110,7 +110,7 @@  discard block
 block discarded – undo
110 110
 			'tooltip' => '',
111 111
 			'type' => '',
112 112
 			'value' => '',
113
-		]);
113
+		] );
114 114
 	}
115 115
 
116 116
 	/**
@@ -118,7 +118,7 @@  discard block
 block discarded – undo
118 118
 	 */
119 119
 	protected function normalizeContainer( array $field )
120 120
 	{
121
-		if( !array_key_exists( 'html', $field ) && !array_key_exists( 'items', $field ))return;
121
+		if( !array_key_exists( 'html', $field ) && !array_key_exists( 'items', $field ) )return;
122 122
 		$field['items'] = $this->generateFields( $field['items'] );
123 123
 		return $field;
124 124
 	}
@@ -128,7 +128,7 @@  discard block
 block discarded – undo
128 128
 	 */
129 129
 	protected function normalizeField( array $field, array $defaults )
130 130
 	{
131
-		if( !$this->validate( $field ))return;
131
+		if( !$this->validate( $field ) )return;
132 132
 		return array_filter( shortcode_atts( $defaults, $field ), function( $value ) {
133 133
 			return $value !== '';
134 134
 		});
@@ -148,9 +148,9 @@  discard block
 block discarded – undo
148 148
 			'tooltip' => '',
149 149
 			'type' => '',
150 150
 			'value' => '',
151
-		]);
152
-		if( !is_array( $listbox ))return;
153
-		if( !array_key_exists( '', $listbox['options'] )) {
151
+		] );
152
+		if( !is_array( $listbox ) )return;
153
+		if( !array_key_exists( '', $listbox['options'] ) ) {
154 154
 			$listbox['options'] = ['' => $listbox['placeholder']] + $listbox['options'];
155 155
 		}
156 156
 		foreach( $listbox['options'] as $value => $text ) {
@@ -167,7 +167,7 @@  discard block
 block discarded – undo
167 167
 	 */
168 168
 	protected function normalizePost( array $field )
169 169
 	{
170
-		if( !is_array( $field['query_args'] )) {
170
+		if( !is_array( $field['query_args'] ) ) {
171 171
 			$field['query_args'] = [];
172 172
 		}
173 173
 		$posts = get_posts( wp_parse_args( $field['query_args'], [
@@ -175,8 +175,8 @@  discard block
 block discarded – undo
175 175
 			'orderby' => 'title',
176 176
 			'post_type' => 'post',
177 177
 			'posts_per_page' => 30,
178
-		]));
179
-		if( !empty( $posts )) {
178
+		] ) );
179
+		if( !empty($posts) ) {
180 180
 			$options = [];
181 181
 			foreach( $posts as $post ) {
182 182
 				$options[$post->ID] = esc_html( $post->post_title );
@@ -206,7 +206,7 @@  discard block
 block discarded – undo
206 206
 			'tooltip' => '',
207 207
 			'type' => '',
208 208
 			'value' => '',
209
-		]);
209
+		] );
210 210
 	}
211 211
 
212 212
 	/**
@@ -214,7 +214,7 @@  discard block
 block discarded – undo
214 214
 	 */
215 215
 	protected function validate( array $field )
216 216
 	{
217
-		$args = shortcode_atts([
217
+		$args = shortcode_atts( [
218 218
 			'label' => '',
219 219
 			'name' => false,
220 220
 			'required' => false,
@@ -230,13 +230,13 @@  discard block
 block discarded – undo
230 230
 	 */
231 231
 	protected function validateErrors( array $args )
232 232
 	{
233
-		if( !isset( $args['required']['error'] )) {
233
+		if( !isset($args['required']['error']) ) {
234 234
 			return true;
235 235
 		}
236
-		$this->errors[$args['name']] = $this->normalizeContainer([
236
+		$this->errors[$args['name']] = $this->normalizeContainer( [
237 237
 			'html' => $args['required']['error'],
238 238
 			'type' => 'container',
239
-		]);
239
+		] );
240 240
 		return false;
241 241
 	}
242 242
 
@@ -249,10 +249,10 @@  discard block
 block discarded – undo
249 249
 			return true;
250 250
 		}
251 251
 		$alert = esc_html__( 'Some of the shortcode options are required.', 'site-reviews' );
252
-		if( isset( $args['required']['alert'] )) {
252
+		if( isset($args['required']['alert']) ) {
253 253
 			$alert = $args['required']['alert'];
254 254
 		}
255
-		else if( !empty( $args['label'] )) {
255
+		else if( !empty($args['label']) ) {
256 256
 			$alert = sprintf(
257 257
 				esc_html_x( 'The "%s" option is required.', 'the option label', 'site-reviews' ),
258 258
 				str_replace( ':', '', $args['label'] )
Please login to merge, or discard this patch.
plugin/Modules/Validator.php 1 patch
Spacing   +18 added lines, -18 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,14 +88,14 @@  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 94
 		$this->validateRequired( $attribute, $value ) || in_array( $rule, $this->implicitRules );
95
-		if( !method_exists( $this, $method = 'validate'.$rule )) {
95
+		if( !method_exists( $this, $method = 'validate'.$rule ) ) {
96 96
 			throw new BadMethodCallException( "Method [$method] does not exist." );
97 97
 		}
98
-		if( !$this->$method( $attribute, $value, $parameters )) {
98
+		if( !$this->$method( $attribute, $value, $parameters ) ) {
99 99
 			$this->addFailure( $attribute, $rule, $parameters );
100 100
 		}
101 101
 	}
@@ -110,7 +110,7 @@  discard block
 block discarded – undo
110 110
 	{
111 111
 		$message = $this->getMessage( $attribute, $rule, $parameters );
112 112
 		$this->errors[$attribute]['errors'][] = $message;
113
-		if( !isset( $this->errors[$attribute]['value'] )) {
113
+		if( !isset($this->errors[$attribute]['value']) ) {
114 114
 			$this->errors[$attribute]['value'] = $this->getValue( $attribute );
115 115
 		}
116 116
 	}
@@ -147,7 +147,7 @@  discard block
 block discarded – undo
147 147
 	 */
148 148
 	protected function getMessage( $attribute, $rule, array $parameters )
149 149
 	{
150
-		if( in_array( $rule, $this->sizeRules )) {
150
+		if( in_array( $rule, $this->sizeRules ) ) {
151 151
 			return $this->getSizeMessage( $attribute, $rule, $parameters );
152 152
 		}
153 153
 		$lowerRule = glsr( Helper::class )->snakeCase( $rule );
@@ -162,11 +162,11 @@  discard block
 block discarded – undo
162 162
 	 */
163 163
 	protected function getRule( $attribute, $rules )
164 164
 	{
165
-		if( !array_key_exists( $attribute, $this->rules ))return;
166
-		$rules = (array) $rules;
165
+		if( !array_key_exists( $attribute, $this->rules ) )return;
166
+		$rules = (array)$rules;
167 167
 		foreach( $this->rules[$attribute] as $rule ) {
168
-			list( $rule, $parameters ) = $this->parseRule( $rule );
169
-			if( in_array( $rule, $rules )) {
168
+			list($rule, $parameters) = $this->parseRule( $rule );
169
+			if( in_array( $rule, $rules ) ) {
170 170
 				return [$rule, $parameters];
171 171
 			}
172 172
 		}
@@ -184,7 +184,7 @@  discard block
 block discarded – undo
184 184
 		if( is_numeric( $value ) && $hasNumeric ) {
185 185
 			return $value;
186 186
 		}
187
-		elseif( is_array( $value )) {
187
+		elseif( is_array( $value ) ) {
188 188
 			return count( $value );
189 189
 		}
190 190
 		return mb_strlen( $value );
@@ -211,7 +211,7 @@  discard block
 block discarded – undo
211 211
 	 */
212 212
 	protected function getValue( $attribute )
213 213
 	{
214
-		if( isset( $this->data[$attribute] )) {
214
+		if( isset($this->data[$attribute]) ) {
215 215
 			return $this->data[$attribute];
216 216
 		}
217 217
 	}
@@ -224,7 +224,7 @@  discard block
 block discarded – undo
224 224
 	 */
225 225
 	protected function hasRule( $attribute, $rules )
226 226
 	{
227
-		return !is_null( $this->getRule( $attribute, $rules ));
227
+		return !is_null( $this->getRule( $attribute, $rules ) );
228 228
 	}
229 229
 
230 230
 	/**
@@ -264,11 +264,11 @@  discard block
 block discarded – undo
264 264
 		$parameters = [];
265 265
 		// example: {rule}:{parameters}
266 266
 		if( strpos( $rule, ':' ) !== false ) {
267
-			list( $rule, $parameter ) = explode( ':', $rule, 2 );
267
+			list($rule, $parameter) = explode( ':', $rule, 2 );
268 268
 			// example: {parameter1,parameter2,...}
269 269
 			$parameters = $this->parseParameters( $rule, $parameter );
270 270
 		}
271
-		$rule = ucwords( str_replace( ['-', '_'], ' ', trim( $rule )));
271
+		$rule = ucwords( str_replace( ['-', '_'], ' ', trim( $rule ) ) );
272 272
 		$rule = str_replace( ' ', '', $rule );
273 273
 		return [$rule, $parameters];
274 274
 	}
@@ -296,7 +296,7 @@  discard block
 block discarded – undo
296 296
 	protected function shouldStopValidating( $attribute )
297 297
 	{
298 298
 		return $this->hasRule( $attribute, $this->implicitRules )
299
-			&& isset( $this->failedRules[$attribute] )
299
+			&& isset($this->failedRules[$attribute])
300 300
 			&& array_intersect( array_keys( $this->failedRules[$attribute] ), $this->implicitRules );
301 301
 	}
302 302
 
@@ -321,12 +321,12 @@  discard block
 block discarded – undo
321 321
 			'regex' => _x( 'The :attribute format is invalid.', ':attribute is a placeholder and should not be translated.', 'site-reviews' ),
322 322
 			'required' => _x( 'The :attribute field is required.', ':attribute is a placeholder and should not be translated.', 'site-reviews' ),
323 323
 		];
324
-		$message = isset( $strings[$key] )
324
+		$message = isset($strings[$key])
325 325
 			? $strings[$key]
326 326
 			: false;
327 327
 		if( !$message )return;
328 328
 		$message = str_replace( ':attribute', $attribute, $message );
329
-		if( method_exists( $this, $replacer = 'replace'.$rule )) {
329
+		if( method_exists( $this, $replacer = 'replace'.$rule ) ) {
330 330
 			$message = $this->$replacer( $message, $parameters );
331 331
 		}
332 332
 		return $message;
Please login to merge, or discard this patch.
plugin/Modules/Date.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -31,7 +31,7 @@
 block discarded – undo
31 31
 		$diff = time() - strtotime( $date );
32 32
 		foreach( static::$TIME_PERIODS as $i => $timePeriod ) {
33 33
 			if( $diff > $timePeriod[0] )continue;
34
-			$unit = intval( floor( $diff / $timePeriod[1] ));
34
+			$unit = intval( floor( $diff / $timePeriod[1] ) );
35 35
 			$relativeDates = [
36 36
 				_n( '%s second ago', '%s seconds ago', $unit, 'site-reviews' ),
37 37
 				_n( '%s minute ago', '%s minutes ago', $unit, 'site-reviews' ),
Please login to merge, or discard this patch.
plugin/Modules/Session.php 1 patch
Spacing   +16 added lines, -16 removed lines patch added patch discarded remove patch
@@ -33,8 +33,8 @@  discard block
 block discarded – undo
33 33
 
34 34
 	public function __construct()
35 35
 	{
36
-		if( $cookieId = filter_input( INPUT_COOKIE, static::SESSION_COOKIE )) {
37
-			$cookie = explode( '||', stripslashes( $cookieId ));
36
+		if( $cookieId = filter_input( INPUT_COOKIE, static::SESSION_COOKIE ) ) {
37
+			$cookie = explode( '||', stripslashes( $cookieId ) );
38 38
 			$this->sessionId = preg_replace( '/[^A-Za-z0-9_]/', '', $cookie[0] );
39 39
 			$this->expiryTimestamp = absint( $cookie[1] );
40 40
 			$this->expiryTimestampReset = absint( $cookie[2] );
@@ -78,7 +78,7 @@  discard block
 block discarded – undo
78 78
 	public function deleteExpiredSessions( $limit = 1000 )
79 79
 	{
80 80
 		global $wpdb;
81
-		if( $expiredSessions = implode( "','", $this->getExpiredSessions( $limit ))) {
81
+		if( $expiredSessions = implode( "','", $this->getExpiredSessions( $limit ) ) ) {
82 82
 			$wpdb->query(
83 83
 				"DELETE FROM {$wpdb->options} WHERE option_name IN ('{$expiredSessions}')"
84 84
 			);
@@ -94,11 +94,11 @@  discard block
 block discarded – undo
94 94
 	public function get( $key, $fallback = '', $unset = false )
95 95
 	{
96 96
 		$key = sanitize_key( $key );
97
-		$value = isset( $this->sessionData[$key] )
97
+		$value = isset($this->sessionData[$key])
98 98
 			? maybe_unserialize( $this->sessionData[$key] )
99 99
 			: $fallback;
100
-		if( isset( $this->sessionData[$key] ) && $unset ) {
101
-			unset( $this->sessionData[$key] );
100
+		if( isset($this->sessionData[$key]) && $unset ) {
101
+			unset($this->sessionData[$key]);
102 102
 			$this->updateSession();
103 103
 		}
104 104
 		return $value;
@@ -132,7 +132,7 @@  discard block
 block discarded – undo
132 132
 	protected function deleteSession()
133 133
 	{
134 134
 		delete_option( $this->getSessionId() );
135
-		delete_option( $this->getSessionId( 'expires' ));
135
+		delete_option( $this->getSessionId( 'expires' ) );
136 136
 	}
137 137
 
138 138
 	/**
@@ -140,7 +140,7 @@  discard block
 block discarded – undo
140 140
 	 */
141 141
 	protected function generateSessionId()
142 142
 	{
143
-		return md5(( new PasswordHash( 8, false ))->get_random_bytes( 32 ));
143
+		return md5( (new PasswordHash( 8, false ))->get_random_bytes( 32 ) );
144 144
 	}
145 145
 
146 146
 	/**
@@ -152,13 +152,13 @@  discard block
 block discarded – undo
152 152
 		global $wpdb;
153 153
 		$expiredSessions = [];
154 154
 		$sessions = $wpdb->get_results(
155
-			"SELECT option_name AS name, option_value AS expiration " .
156
-			"FROM {$wpdb->options} " .
157
-			"WHERE option_name LIKE '".static::SESSION_COOKIE."_expires_%' " .
158
-			"ORDER BY option_value ASC " .
155
+			"SELECT option_name AS name, option_value AS expiration ".
156
+			"FROM {$wpdb->options} ".
157
+			"WHERE option_name LIKE '".static::SESSION_COOKIE."_expires_%' ".
158
+			"ORDER BY option_value ASC ".
159 159
 			"LIMIT 0, ".absint( $limit )
160 160
 		);
161
-		if( !empty( $sessions )) {
161
+		if( !empty($sessions) ) {
162 162
 			$now = time();
163 163
 			foreach( $sessions as $session ) {
164 164
 				if( $now <= $session->expiration )continue;
@@ -175,7 +175,7 @@  discard block
 block discarded – undo
175 175
 	 */
176 176
 	protected function getSessionId( $separator = '' )
177 177
 	{
178
-		return implode( '_', array_filter( [static::SESSION_COOKIE, $separator, $this->sessionId] ));
178
+		return implode( '_', array_filter( [static::SESSION_COOKIE, $separator, $this->sessionId] ) );
179 179
 	}
180 180
 
181 181
 	/**
@@ -206,7 +206,7 @@  discard block
 block discarded – undo
206 206
 	{
207 207
 		if( headers_sent() )return;
208 208
 		$cookie = $this->sessionId.'||'.$this->expiryTimestamp.'||'.$this->expiryTimestampReset;
209
-		$cookiePath = preg_replace( '|https?://[^/]+|i', '', trailingslashit( (string)get_option( 'home' )));
209
+		$cookiePath = preg_replace( '|https?://[^/]+|i', '', trailingslashit( (string)get_option( 'home' ) ) );
210 210
 		setcookie( static::SESSION_COOKIE, $cookie, $this->expiryTimestamp, $cookiePath );
211 211
 	}
212 212
 
@@ -224,7 +224,7 @@  discard block
 block discarded – undo
224 224
 	 */
225 225
 	protected function updateSession()
226 226
 	{
227
-		if( false === get_option( $this->getSessionId() )) {
227
+		if( false === get_option( $this->getSessionId() ) ) {
228 228
 			$this->createSession();
229 229
 		}
230 230
 		else {
Please login to merge, or discard this patch.
plugin/Modules/Upgrader.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -16,7 +16,7 @@  discard block
 block discarded – undo
16 16
 		natsort( $routines );
17 17
 		array_walk( $routines, function( $routine ) {
18 18
 			$version = str_replace( strtolower( __CLASS__ ).'_', '', $routine );
19
-			if( version_compare( glsr()->version, $version, '>=' ))return;
19
+			if( version_compare( glsr()->version, $version, '>=' ) )return;
20 20
 			call_user_func( [$this, $routine] );
21 21
 		});
22 22
 		$this->updateVersion();
@@ -28,7 +28,7 @@  discard block
 block discarded – undo
28 28
 	public function updateVersion()
29 29
 	{
30 30
 		$currentVersion = glsr( OptionManager::class )->get( 'version' );
31
-		if( version_compare( $currentVersion, glsr()->version, '<' )) {
31
+		if( version_compare( $currentVersion, glsr()->version, '<' ) ) {
32 32
 			glsr( OptionManager::class )->set( 'version', glsr()->version );
33 33
 		}
34 34
 		if( $currentVersion != glsr()->version ) {
Please login to merge, or discard this patch.
plugin/Modules/Editor/Customization.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -57,7 +57,7 @@
 block discarded – undo
57 57
 	 */
58 58
 	protected function isReviewEditable()
59 59
 	{
60
-		$postId = intval( filter_input( INPUT_GET, 'post' ));
60
+		$postId = intval( filter_input( INPUT_GET, 'post' ) );
61 61
 		return $postId > 0
62 62
 			&& get_post_meta( $postId, 'review_type', true ) == 'local'
63 63
 			&& $this->isReviewEditor();
Please login to merge, or discard this patch.
plugin/Controllers/PublicController.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -44,10 +44,10 @@  discard block
 block discarded – undo
44 44
 		$vars[] = Application::PAGED_QUERY_VAR;
45 45
 		// dirty hack to fix a form submission with a field that has "name" as name
46 46
 		if( filter_input( INPUT_POST, 'action' ) == 'submit-review'
47
-			&& !is_null( filter_input( INPUT_POST, 'gotcha' ))) {
47
+			&& !is_null( filter_input( INPUT_POST, 'gotcha' ) ) ) {
48 48
 			$index = array_search( 'name', $vars, true );
49 49
 			if( false !== $index ) {
50
-				unset( $vars[$index] );
50
+				unset($vars[$index]);
51 51
 			}
52 52
 		}
53 53
 		return $vars;
@@ -59,10 +59,10 @@  discard block
 block discarded – undo
59 59
 	public function routerCreateReview( array $request )
60 60
 	{
61 61
 		$validated = glsr( ValidateReview::class )->validate( $request );
62
-		if( !empty( $validated->error )) {
62
+		if( !empty($validated->error) ) {
63 63
 			return $validated->request;
64 64
 		}
65 65
 		if( $validated->recaptchaIsUnset )return;
66
-		return $this->execute( new CreateReview( $validated->request ));
66
+		return $this->execute( new CreateReview( $validated->request ) );
67 67
 	}
68 68
 }
Please login to merge, or discard this patch.
plugin/Modules/Validator/ValidateReview.php 1 patch
Spacing   +23 added lines, -23 removed lines patch added patch discarded remove patch
@@ -55,11 +55,11 @@  discard block
 block discarded – undo
55 55
 	 */
56 56
 	public function validateRequest( array $request )
57 57
 	{
58
-		if( !$this->isRequestValid( $request )) {
58
+		if( !$this->isRequestValid( $request ) ) {
59 59
 			$this->error = __( 'Please fix the submission errors.', 'site-reviews' );
60 60
 			return $request;
61 61
 		}
62
-		if( empty( $request['title'] )) {
62
+		if( empty($request['title']) ) {
63 63
 			$request['title'] = __( 'No Title', 'site-reviews' );
64 64
 		}
65 65
 		return array_merge( glsr( ValidateReviewDefaults::class )->defaults(), $request );
@@ -73,14 +73,14 @@  discard block
 block discarded – undo
73 73
 		$rules = array_intersect_key(
74 74
 			apply_filters( 'site-reviews/validation/rules', static::VALIDATION_RULES ),
75 75
 			array_flip( array_merge(
76
-				['rating','terms'],
76
+				['rating', 'terms'],
77 77
 				glsr( OptionManager::class )->get( 'settings.submissions.required', [] )
78
-			))
78
+			) )
79 79
 		);
80
-		$excluded = isset( $request['excluded'] )
80
+		$excluded = isset($request['excluded'])
81 81
 			? json_decode( $request['excluded'] )
82 82
 			: [];
83
-		return array_diff_key( $rules, array_flip( $excluded ));
83
+		return array_diff_key( $rules, array_flip( $excluded ) );
84 84
 	}
85 85
 
86 86
 	/**
@@ -93,14 +93,14 @@  discard block
 block discarded – undo
93 93
 			return true;
94 94
 		}
95 95
 		$recaptchaResponse = filter_input( INPUT_POST, 'g-recaptcha-response' );
96
-		if( empty( $recaptchaResponse )) {
96
+		if( empty($recaptchaResponse) ) {
97 97
 			return null; //if response is empty we need to return null
98 98
 		}
99 99
 		if( $integration == 'custom' ) {
100 100
 			return $this->isRecaptchaValid( $recaptchaResponse );
101 101
 		}
102 102
 		if( $integration == 'invisible-recaptcha' ) {
103
-			return boolval( apply_filters( 'google_invre_is_valid_request_filter', true ));
103
+			return boolval( apply_filters( 'google_invre_is_valid_request_filter', true ) );
104 104
 		}
105 105
 		return false;
106 106
 	}
@@ -110,17 +110,17 @@  discard block
 block discarded – undo
110 110
 	 */
111 111
 	protected function isRecaptchaValid( $recaptchaResponse )
112 112
 	{
113
-		$endpoint = add_query_arg([
113
+		$endpoint = add_query_arg( [
114 114
 			'remoteip' => glsr( Helper::class )->getIpAddress(),
115 115
 			'response' => $recaptchaResponse,
116 116
 			'secret' => glsr( OptionManager::class )->get( 'settings.submissions.recaptcha.secret' ),
117 117
 		], 'https://www.google.com/recaptcha/api/siteverify' );
118
-		if( is_wp_error( $response = wp_remote_get( $endpoint ))) {
118
+		if( is_wp_error( $response = wp_remote_get( $endpoint ) ) ) {
119 119
 			glsr_log()->error( $response->get_error_message() );
120 120
 			return false;
121 121
 		}
122
-		$response = json_decode( wp_remote_retrieve_body( $response ));
123
-		if( !empty( $response->success )) {
122
+		$response = json_decode( wp_remote_retrieve_body( $response ) );
123
+		if( !empty($response->success) ) {
124 124
 			return boolval( $response->success );
125 125
 		}
126 126
 		$errorCodes = [
@@ -143,7 +143,7 @@  discard block
 block discarded – undo
143 143
 	{
144 144
 		$rules = $this->getValidationRules( $request );
145 145
 		$errors = glsr( Validator::class )->validate( $request, $rules );
146
-		if( empty( $errors )) {
146
+		if( empty($errors) ) {
147 147
 			return true;
148 148
 		}
149 149
 		$this->setSessionValues( 'errors', $errors );
@@ -160,7 +160,7 @@  discard block
 block discarded – undo
160 160
 	protected function setSessionValues( $type, $value, $loggedMessage = '' )
161 161
 	{
162 162
 		glsr( Session::class )->set( $this->form_id.$type, $value );
163
-		if( !empty( $loggedMessage )) {
163
+		if( !empty($loggedMessage) ) {
164 164
 			glsr_log()->warning( $loggedMessage );
165 165
 			glsr_log()->warning( $this->request );
166 166
 		}
@@ -171,8 +171,8 @@  discard block
 block discarded – undo
171 171
 	 */
172 172
 	protected function validateAkismet()
173 173
 	{
174
-		if( !empty( $this->error ))return;
175
-		if( !glsr( Akismet::class )->isSpam( $this->request ))return;
174
+		if( !empty($this->error) )return;
175
+		if( !glsr( Akismet::class )->isSpam( $this->request ) )return;
176 176
 		$this->setSessionValues( 'errors', [], 'Akismet caught a spam submission:' );
177 177
 		$this->error = __( 'Your review cannot be submitted at this time. Please try again later.', 'site-reviews' );
178 178
 	}
@@ -182,8 +182,8 @@  discard block
 block discarded – undo
182 182
 	 */
183 183
 	protected function validateBlacklist()
184 184
 	{
185
-		if( !empty( $this->error ))return;
186
-		if( !glsr( Blacklist::class )->isBlacklisted( $this->request ))return;
185
+		if( !empty($this->error) )return;
186
+		if( !glsr( Blacklist::class )->isBlacklisted( $this->request ) )return;
187 187
 		$blacklistAction = glsr( OptionManager::class )->get( 'settings.submissions.blacklist.action' );
188 188
 		if( $blacklistAction == 'unapprove' ) {
189 189
 			$this->request['blacklisted'] = true;
@@ -199,7 +199,7 @@  discard block
 block discarded – undo
199 199
 	 */
200 200
 	protected function validateCustom()
201 201
 	{
202
-		if( !empty( $this->error ))return;
202
+		if( !empty($this->error) )return;
203 203
 		$validated = apply_filters( 'site-reviews/validate/review/submission', true, $this->request );
204 204
 		if( $validated === true )return;
205 205
 		$this->setSessionValues( 'errors', [] );
@@ -214,8 +214,8 @@  discard block
 block discarded – undo
214 214
 	 */
215 215
 	protected function validateHoneyPot()
216 216
 	{
217
-		if( !empty( $this->error ))return;
218
-		if( empty( $this->request['gotcha'] ))return;
217
+		if( !empty($this->error) )return;
218
+		if( empty($this->request['gotcha']) )return;
219 219
 		$this->setSessionValues( 'errors', [], 'The Honeypot caught a bad submission:' );
220 220
 		$this->error = __( 'The review submission failed. Please notify the site administrator.', 'site-reviews' );
221 221
 	}
@@ -225,9 +225,9 @@  discard block
 block discarded – undo
225 225
 	 */
226 226
 	protected function validateRecaptcha()
227 227
 	{
228
-		if( !empty( $this->error ))return;
228
+		if( !empty($this->error) )return;
229 229
 		$isValid = $this->isRecaptchaResponseValid();
230
-		if( is_null( $isValid )) {
230
+		if( is_null( $isValid ) ) {
231 231
 			$this->setSessionValues( 'recaptcha', true );
232 232
 			$this->recaptchaIsUnset = true;
233 233
 		}
Please login to merge, or discard this patch.
plugin/Modules/Blacklist.php 1 patch
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -12,14 +12,14 @@  discard block
 block discarded – undo
12 12
 	 */
13 13
 	public function isBlacklisted( CreateReview $review )
14 14
 	{
15
-		$target = implode( "\n", array_filter([
15
+		$target = implode( "\n", array_filter( [
16 16
 			$review->author,
17 17
 			$review->content,
18 18
 			$review->email,
19 19
 			$review->ipAddress,
20 20
 			$review->title,
21
-		]));
22
-		return (bool) apply_filters( 'site-reviews/blacklist/is-blacklisted',
21
+		] ) );
22
+		return (bool)apply_filters( 'site-reviews/blacklist/is-blacklisted',
23 23
 			$this->check( $target ),
24 24
 			$review
25 25
 		);
@@ -31,16 +31,16 @@  discard block
 block discarded – undo
31 31
 	 */
32 32
 	protected function check( $target )
33 33
 	{
34
-		$blacklist = trim( glsr( OptionManager::class )->get( 'settings.submissions.blacklist.entries' ));
35
-		if( empty( $blacklist )) {
34
+		$blacklist = trim( glsr( OptionManager::class )->get( 'settings.submissions.blacklist.entries' ) );
35
+		if( empty($blacklist) ) {
36 36
 			return false;
37 37
 		}
38 38
 		$lines = explode( "\n", $blacklist );
39
-		foreach( (array) $lines as $line ) {
39
+		foreach( (array)$lines as $line ) {
40 40
 			$line = trim( $line );
41
-			if( empty( $line ) || 256 < strlen( $line ))continue;
42
-			$pattern = sprintf( '#%s#i', preg_quote( $line, '#' ));
43
-			if( preg_match( $pattern, $target )) {
41
+			if( empty($line) || 256 < strlen( $line ) )continue;
42
+			$pattern = sprintf( '#%s#i', preg_quote( $line, '#' ) );
43
+			if( preg_match( $pattern, $target ) ) {
44 44
 				return true;
45 45
 			}
46 46
 		}
Please login to merge, or discard this patch.