Passed
Push — master ( af44f0...ee0da0 )
by Paul
04:21
created
plugin/Database/Cache.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -17,11 +17,11 @@
 block discarded – undo
17 17
 			$ipAddresses = array_fill_keys( ['v4', 'v6'], [] );
18 18
 			foreach( array_keys( $ipAddresses ) as $version ) {
19 19
 				$response = wp_remote_get( 'https://www.cloudflare.com/ips-'.$version );
20
-				if( is_wp_error( $response )) {
20
+				if( is_wp_error( $response ) ) {
21 21
 					glsr_log()->error( $response->get_error_message() );
22 22
 					continue;
23 23
 				}
24
-				$ipAddresses[$version] = array_filter( explode( PHP_EOL, wp_remote_retrieve_body( $response )));
24
+				$ipAddresses[$version] = array_filter( explode( PHP_EOL, wp_remote_retrieve_body( $response ) ) );
25 25
 			}
26 26
 			wp_cache_set( Application::ID, $ipAddresses, '_cloudflare_ips' );
27 27
 		}
Please login to merge, or discard this patch.
plugin/Router.php 1 patch
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -16,7 +16,7 @@  discard block
 block discarded – undo
16 16
 	public function routeAdminPostRequest()
17 17
 	{
18 18
 		$request = filter_input( INPUT_POST, Application::ID, FILTER_DEFAULT, FILTER_REQUIRE_ARRAY );
19
-		if( !isset( $request['action'] ))return;
19
+		if( !isset($request['action']) )return;
20 20
 		$this->checkNonce( $request['action'] );
21 21
 		switch( $request['action'] ) {
22 22
 			case 'clear-log':
@@ -42,13 +42,13 @@  discard block
 block discarded – undo
42 42
 	public function routeAjaxRequest()
43 43
 	{
44 44
 		$request = $this->normalizeAjaxRequest();
45
-		if( !wp_verify_nonce( $request['nonce'], $request['action'] )) {
45
+		if( !wp_verify_nonce( $request['nonce'], $request['action'] ) ) {
46 46
 			glsr_log()->error( 'Nonce check failed for ajax request' )->info( $request );
47 47
 			wp_die( -1, 403 );
48 48
 		}
49 49
 		$controller = glsr( AjaxController::class );
50 50
 		$method = glsr( Helper::class )->buildMethodName( $request['action'] );
51
-		if( is_callable( [$controller, $method] )) {
51
+		if( is_callable( [$controller, $method] ) ) {
52 52
 			call_user_func( [$controller, $method], $request );
53 53
 		}
54 54
 		else {
@@ -62,9 +62,9 @@  discard block
 block discarded – undo
62 62
 	 */
63 63
 	public function routePublicPostRequest()
64 64
 	{
65
-		switch( filter_input( INPUT_POST, 'action' )) {
65
+		switch( filter_input( INPUT_POST, 'action' ) ) {
66 66
 			case 'submit-review':
67
-				glsr( PublicController::class )->routerCreateReview( $this->normalize( $_POST ));
67
+				glsr( PublicController::class )->routerCreateReview( $this->normalize( $_POST ) );
68 68
 				break;
69 69
 		}
70 70
 	}
@@ -74,7 +74,7 @@  discard block
 block discarded – undo
74 74
 	 */
75 75
 	public function routeWebhookRequest()
76 76
 	{
77
-		$request = filter_input( INPUT_GET, sprintf( '%s-hook', Application::ID ));
77
+		$request = filter_input( INPUT_GET, sprintf( '%s-hook', Application::ID ) );
78 78
 		if( !$request )return;
79 79
 		// @todo manage webhook here
80 80
 	}
@@ -112,14 +112,14 @@  discard block
 block discarded – undo
112 112
 	protected function normalizeAjaxRequest()
113 113
 	{
114 114
 		$request = filter_input( INPUT_POST, 'request', FILTER_DEFAULT, FILTER_REQUIRE_ARRAY );
115
-		if( isset( $request[Application::ID]['action'] )) {
115
+		if( isset($request[Application::ID]['action']) ) {
116 116
 			$request = $request[Application::ID];
117 117
 		}
118
-		if( !isset( $request['action'] )) {
118
+		if( !isset($request['action']) ) {
119 119
 			glsr_log()->error( 'The AJAX request must include an action' )->info( $request );
120 120
 			wp_die();
121 121
 		}
122
-		if( !isset( $request['nonce'] )) {
122
+		if( !isset($request['nonce']) ) {
123 123
 			glsr_log()->error( 'The AJAX request must include a nonce' )->info( $request );
124 124
 			wp_die();
125 125
 		}
Please login to merge, or discard this patch.
plugin/Handlers/CreateReview.php 1 patch
Spacing   +14 added lines, -14 removed lines patch added patch discarded remove patch
@@ -63,7 +63,7 @@  discard block
 block discarded – undo
63 63
 	protected function createWebhookNotification( Command $command, array $args )
64 64
 	{
65 65
 		$fields = [];
66
-		$fields[] = ['title' => str_repeat( ':star:', (int) $command->rating )];
66
+		$fields[] = ['title' => str_repeat( ':star:', (int)$command->rating )];
67 67
 		if( $command->title ) {
68 68
 			$fields[] = ['title' => $command->title];
69 69
 		}
@@ -76,8 +76,8 @@  discard block
 block discarded – undo
76 76
 		if( $command->author ) {
77 77
 			$fields[] = ['value' => trim( $command->author.$command->email.' - '.$command->ipAddress )];
78 78
 		}
79
-		$fields[] = ['value' => sprintf( '<%s|%s>', $args['notification_link'], __( 'View Review', 'site-reviews' ))];
80
-		return json_encode([
79
+		$fields[] = ['value' => sprintf( '<%s|%s>', $args['notification_link'], __( 'View Review', 'site-reviews' ) )];
80
+		return json_encode( [
81 81
 			'icon_url' => glsr()->url.'assets/img/icon.png',
82 82
 			'username' => glsr()->name,
83 83
 			'attachments' => [[
@@ -86,7 +86,7 @@  discard block
 block discarded – undo
86 86
 				'fallback' => $this->createEmailNotification( $command, $args )->read( 'plaintext' ),
87 87
 				'fields' => $fields,
88 88
 			]],
89
-		]);
89
+		] );
90 90
 	}
91 91
 
92 92
 	/**
@@ -115,12 +115,12 @@  discard block
 block discarded – undo
115 115
 	protected function sendNotification( $post_id, Command $command )
116 116
 	{
117 117
 		$notificationType = glsr( OptionManager::class )->get( 'settings.general.notification' );
118
-		if( !in_array( $notificationType, ['default','custom','webhook'] ))return;
119
-		$assignedToTitle = get_the_title( (int) $command->assignedTo );
118
+		if( !in_array( $notificationType, ['default', 'custom', 'webhook'] ) )return;
119
+		$assignedToTitle = get_the_title( (int)$command->assignedTo );
120 120
 		$notificationSubject = _nx(
121 121
 			'New %s-star review',
122 122
 			'New %s-star review of: %s',
123
-			(int) empty( $assignedToTitle ),
123
+			(int)empty($assignedToTitle),
124 124
 			'The text is different depending on whether or not the review has been assigned to a post.',
125 125
 			'site-reviews'
126 126
 		);
@@ -129,7 +129,7 @@  discard block
 block discarded – undo
129 129
 			sprintf( $notificationSubject, $command->rating, $assignedToTitle )
130 130
 		);
131 131
 		$args = [
132
-			'notification_link' => esc_url( admin_url( sprintf( 'post.php?post=%s&action=edit', $post_id ))),
132
+			'notification_link' => esc_url( admin_url( sprintf( 'post.php?post=%s&action=edit', $post_id ) ) ),
133 133
 			'notification_title' => $notificationTitle,
134 134
 			'notification_type' => $notificationType,
135 135
 		];
@@ -146,14 +146,14 @@  discard block
 block discarded – undo
146 146
 		$args['recipient'] = $args['notification_type'] === 'default'
147 147
 			? get_option( 'admin_email' )
148 148
 			: glsr( OptionManager::class )->get( 'settings.general.notification_email' );
149
-		$result = !empty( $args['recipient'] )
149
+		$result = !empty($args['recipient'])
150 150
 			? $this->createEmailNotification( $command, $args )->send()
151 151
 			: false;
152 152
 		if( $result === null ) {
153
-			glsr_log()->error( __( 'Email notification was not sent: missing email, subject, or message.', 'site-reviews' ));
153
+			glsr_log()->error( __( 'Email notification was not sent: missing email, subject, or message.', 'site-reviews' ) );
154 154
 		}
155 155
 		if( $result === false ) {
156
-			glsr_log()->error( __( 'Email notification was not sent: wp_mail() failed.', 'site-reviews' ));
156
+			glsr_log()->error( __( 'Email notification was not sent: wp_mail() failed.', 'site-reviews' ) );
157 157
 		}
158 158
 	}
159 159
 
@@ -162,7 +162,7 @@  discard block
 block discarded – undo
162 162
 	 */
163 163
 	protected function sendWebhookNotification( Command $command, array $args )
164 164
 	{
165
-		if( !( $endpoint = glsr( OptionManager::class )->get( 'settings.general.webhook_url' )))return;
165
+		if( !($endpoint = glsr( OptionManager::class )->get( 'settings.general.webhook_url' )) )return;
166 166
 		$notification = $this->createWebhookNotification( $command, $args );
167 167
 		$result = wp_remote_post( $endpoint, [
168 168
 			'method' => 'POST',
@@ -173,8 +173,8 @@  discard block
 block discarded – undo
173 173
 			'sslverify' => false,
174 174
 			'headers' => ['Content-Type' => 'application/json'],
175 175
 			'body' => apply_filters( 'site-reviews/webhook/notification', $notification, $command ),
176
-		]);
177
-		if( is_wp_error( $result )) {
176
+		] );
177
+		if( is_wp_error( $result ) ) {
178 178
 			glsr_log()->error( $result->get_error_message() );
179 179
 		}
180 180
 	}
Please login to merge, or discard this patch.
plugin/Handlers/ChangeStatus.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -12,11 +12,11 @@  discard block
 block discarded – undo
12 12
 	 */
13 13
 	public function handle( Command $command )
14 14
 	{
15
-		$postId = wp_update_post([
15
+		$postId = wp_update_post( [
16 16
 			'ID' => $command->id,
17 17
 			'post_status' => $command->status,
18
-		]);
19
-		if( is_wp_error( $postId )) {
18
+		] );
19
+		if( is_wp_error( $postId ) ) {
20 20
 			glsr_log()->error( $postId->get_error_message() );
21 21
 			return [];
22 22
 		}
@@ -37,7 +37,7 @@  discard block
 block discarded – undo
37 37
 			'href' => get_edit_post_link( $postId ),
38 38
 			'class' => 'row-title',
39 39
 			'title' => __( 'Edit', 'site-reviews' ).' &#8220;'.esc_attr( $title ).'&#8221;',
40
-		]);
40
+		] );
41 41
 	}
42 42
 
43 43
 	/**
@@ -47,7 +47,7 @@  discard block
 block discarded – undo
47 47
 	protected function getPostState( $postId )
48 48
 	{
49 49
 		ob_start();
50
-		_post_states( get_post( $postId ));
50
+		_post_states( get_post( $postId ) );
51 51
 		return ob_get_clean();
52 52
 	}
53 53
 }
Please login to merge, or discard this patch.
plugin/Handlers/EnqueueAssets.php 1 patch
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -21,13 +21,13 @@  discard block
 block discarded – undo
21 21
 		$this->dependencies = glsr( Html::class )->getDependencies();
22 22
 		$ajaxNonce = wp_create_nonce( Application::ID.'-ajax-nonce' );
23 23
 		$variables = [
24
-			'action'  => glsr()->prefix . '_action',
25
-			'ajaxurl' => add_query_arg( '_nonce', $ajaxNonce, admin_url( 'admin-ajax.php' )),
24
+			'action'  => glsr()->prefix.'_action',
25
+			'ajaxurl' => add_query_arg( '_nonce', $ajaxNonce, admin_url( 'admin-ajax.php' ) ),
26 26
 			'ajaxnonce' => $ajaxNonce,
27
-			'ajaxpagination' => ['#wpadminbar','.site-navigation-fixed'],
27
+			'ajaxpagination' => ['#wpadminbar', '.site-navigation-fixed'],
28 28
 		];
29 29
 		$this->enqueue();
30
-		wp_localize_script( Application::ID, 'site_reviews', apply_filters( 'site-reviews/enqueue/localize', $variables ));
30
+		wp_localize_script( Application::ID, 'site_reviews', apply_filters( 'site-reviews/enqueue/localize', $variables ) );
31 31
 	}
32 32
 
33 33
 	/**
@@ -35,11 +35,11 @@  discard block
 block discarded – undo
35 35
 	 */
36 36
 	public function enqueue()
37 37
 	{
38
-		$currentTheme = sanitize_title( wp_get_theme()->get( 'Name' ));
38
+		$currentTheme = sanitize_title( wp_get_theme()->get( 'Name' ) );
39 39
 		$stylesheet = file_exists( glsr()->path.'assets/css/'.$currentTheme.'.css' )
40 40
 			? glsr()->url.'assets/css/'.$currentTheme.'.css'
41 41
 			: glsr()->url.'assets/css/'.Application::ID.'.css';
42
-		if( apply_filters( 'site-reviews/assets/css', true )) {
42
+		if( apply_filters( 'site-reviews/assets/css', true ) ) {
43 43
 			wp_enqueue_style(
44 44
 				Application::ID,
45 45
 				$stylesheet,
@@ -47,7 +47,7 @@  discard block
 block discarded – undo
47 47
 				glsr()->version
48 48
 			);
49 49
 		}
50
-		if( apply_filters( 'site-reviews/assets/js', true )) {
50
+		if( apply_filters( 'site-reviews/assets/js', true ) ) {
51 51
 			wp_enqueue_script(
52 52
 				Application::ID,
53 53
 				glsr()->url.'assets/js/'.Application::ID.'.js',
@@ -66,11 +66,11 @@  discard block
 block discarded – undo
66 66
 	 */
67 67
 	public function enqueueRecaptchaScript()
68 68
 	{
69
-		wp_enqueue_script( Application::ID.'/google-recaptcha', add_query_arg([
69
+		wp_enqueue_script( Application::ID.'/google-recaptcha', add_query_arg( [
70 70
 			'hl' => apply_filters( 'site-reviews/recaptcha/language', get_locale() ),
71 71
 			'onload' => 'glsr_render_recaptcha',
72 72
 			'render' => 'explicit',
73
-		], 'https://www.google.com/recaptcha/api.js' ));
73
+		], 'https://www.google.com/recaptcha/api.js' ) );
74 74
 		$inlineScript = file_get_contents( glsr()->path.'js/recaptcha.js' );
75 75
 		wp_add_inline_script( Application::ID.'/google-recaptcha', $inlineScript, 'before' );
76 76
 	}
Please login to merge, or discard this patch.
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/System.php 1 patch
Spacing   +21 added lines, -21 removed lines patch added patch discarded remove patch
@@ -29,7 +29,7 @@  discard block
 block discarded – undo
29 29
 			'inactive-plugin' => 'Inactive Plugins',
30 30
 			'setting' => 'Plugin Settings',
31 31
 		];
32
-		$systemInfo = array_reduce( array_keys( $details ), function( $carry, $key ) use( $details ) {
32
+		$systemInfo = array_reduce( array_keys( $details ), function( $carry, $key ) use($details) {
33 33
 			$methodName = glsr( Helper::class )->buildMethodName( 'get-'.$key.'-details' );
34 34
 			if( method_exists( $this, $methodName ) && $systemDetails = $this->$methodName() ) {
35 35
 				return $carry.$this->implode( $details[$key], $systemDetails );
@@ -46,8 +46,8 @@  discard block
 block discarded – undo
46 46
 	{
47 47
 		$plugins = get_plugins();
48 48
 		$activePlugins = (array)get_option( 'active_plugins', [] );
49
-		$inactive = array_diff_key( $plugins, array_flip( $activePlugins ));
50
-		return $this->normalizePluginList( array_diff_key( $plugins, $inactive ));
49
+		$inactive = array_diff_key( $plugins, array_flip( $activePlugins ) );
50
+		return $this->normalizePluginList( array_diff_key( $plugins, $inactive ) );
51 51
 	}
52 52
 
53 53
 	/**
@@ -71,7 +71,7 @@  discard block
 block discarded – undo
71 71
 	public function getInactivePluginDetails()
72 72
 	{
73 73
 		$activePlugins = (array)get_option( 'active_plugins', [] );
74
-		return $this->normalizePluginList( array_diff_key( get_plugins(), array_flip( $activePlugins )));
74
+		return $this->normalizePluginList( array_diff_key( get_plugins(), array_flip( $activePlugins ) ) );
75 75
 	}
76 76
 
77 77
 	/**
@@ -81,9 +81,9 @@  discard block
 block discarded – undo
81 81
 	{
82 82
 		$plugins = array_merge(
83 83
 			get_mu_plugins(),
84
-			get_plugins( '/../'.basename( WPMU_PLUGIN_DIR ))
84
+			get_plugins( '/../'.basename( WPMU_PLUGIN_DIR ) )
85 85
 		);
86
-		if( empty( $plugins ))return;
86
+		if( empty($plugins) )return;
87 87
 		return $this->normalizePluginList( $plugins );
88 88
 	}
89 89
 
@@ -92,7 +92,7 @@  discard block
 block discarded – undo
92 92
 	 */
93 93
 	public function getMultisitePluginDetails()
94 94
 	{
95
-		if( !is_multisite() || empty( get_site_option( 'active_sitewide_plugins', [] )))return;
95
+		if( !is_multisite() || empty(get_site_option( 'active_sitewide_plugins', [] )) )return;
96 96
 		return $this->normalizePluginList( wp_get_active_network_plugins() );
97 97
 	}
98 98
 
@@ -114,11 +114,11 @@  discard block
 block discarded – undo
114 114
 			'Max Input Vars' => ini_get( 'max_input_vars' ),
115 115
 			'Memory Limit' => ini_get( 'memory_limit' ),
116 116
 			'Post Max Size' => ini_get( 'post_max_size' ),
117
-			'Session Cookie Path' => esc_html( ini_get( 'session.cookie_path' )),
118
-			'Session Name' => esc_html( ini_get( 'session.name' )),
119
-			'Session Save Path' => esc_html( ini_get( 'session.save_path' )),
120
-			'Session Use Cookies' => var_export( wp_validate_boolean( ini_get( 'session.use_cookies' )), true ),
121
-			'Session Use Only Cookies' => var_export( wp_validate_boolean( ini_get( 'session.use_only_cookies' )), true ),
117
+			'Session Cookie Path' => esc_html( ini_get( 'session.cookie_path' ) ),
118
+			'Session Name' => esc_html( ini_get( 'session.name' ) ),
119
+			'Session Save Path' => esc_html( ini_get( 'session.save_path' ) ),
120
+			'Session Use Cookies' => var_export( wp_validate_boolean( ini_get( 'session.use_cookies' ) ), true ),
121
+			'Session Use Only Cookies' => var_export( wp_validate_boolean( ini_get( 'session.use_only_cookies' ) ), true ),
122 122
 			'Upload Max Filesize' => ini_get( 'upload_max_filesize' ),
123 123
 		];
124 124
 	}
@@ -146,13 +146,13 @@  discard block
 block discarded – undo
146 146
 		$settings = glsr( OptionManager::class )->get( 'settings' );
147 147
 		$settings = $helper->flattenArray( $settings );
148 148
 		foreach( ['submissions.recaptcha.key', 'submissions.recaptcha.secret'] as $key ) {
149
-			if( empty( $settings[$key] ))continue;
149
+			if( empty($settings[$key]) )continue;
150 150
 			$settings[$key] = str_repeat( '*', 10 );
151 151
 		}
152 152
 		$details = [];
153 153
 		foreach( $settings as $key => $value ) {
154
-			if( $helper->startsWith( 'translations', $key ) && $helper->endsWith( 'id', $key ))continue;
155
-			$value = htmlspecialchars( trim( preg_replace('/\s\s+/', '\\n', $value )), ENT_QUOTES, 'UTF-8' );
154
+			if( $helper->startsWith( 'translations', $key ) && $helper->endsWith( 'id', $key ) )continue;
155
+			$value = htmlspecialchars( trim( preg_replace( '/\s\s+/', '\\n', $value ) ), ENT_QUOTES, 'UTF-8' );
156 156
 			$details[$key] = $value;
157 157
 		}
158 158
 		return $details;
@@ -232,7 +232,7 @@  discard block
 block discarded – undo
232 232
 				return $value;
233 233
 			}
234 234
 		}
235
-		return implode( ',', array_filter( [DB_HOST, $serverName] ));
235
+		return implode( ',', array_filter( [DB_HOST, $serverName] ) );
236 236
 	}
237 237
 
238 238
 	/**
@@ -253,8 +253,8 @@  discard block
 block discarded – undo
253 253
 	{
254 254
 		$plugins = get_plugins();
255 255
 		$activePlugins = (array)get_option( 'active_plugins', [] );
256
-		$inactive = $this->normalizePluginList( array_diff_key( $plugins, array_flip( $activePlugins )));
257
-		$active = $this->normalizePluginList( array_diff_key( $plugins, $inactive ));
256
+		$inactive = $this->normalizePluginList( array_diff_key( $plugins, array_flip( $activePlugins ) ) );
257
+		$active = $this->normalizePluginList( array_diff_key( $plugins, $inactive ) );
258 258
 		return $active + $inactive;
259 259
 	}
260 260
 
@@ -265,7 +265,7 @@  discard block
 block discarded – undo
265 265
 	protected function implode( $title, array $details )
266 266
 	{
267 267
 		$strings = ['['.$title.']'];
268
-		$padding = max( array_map( 'strlen', array_keys( $details )) );
268
+		$padding = max( array_map( 'strlen', array_keys( $details ) ) );
269 269
 		$padding = max( [$padding, static::PAD] );
270 270
 		foreach( $details as $key => $value ) {
271 271
 			$strings[] = is_string( $key )
@@ -294,8 +294,8 @@  discard block
 block discarded – undo
294 294
 	{
295 295
 		$response = wp_remote_post( static::WP_API_URL );
296 296
 		return !is_wp_error( $response )
297
-			&& !empty( $response['response']['code'] )
298
-			&& in_array( $response['response']['code'], range( 200, 299 ))
297
+			&& !empty($response['response']['code'])
298
+			&& in_array( $response['response']['code'], range( 200, 299 ) )
299 299
 			? 'Works'
300 300
 			: 'Does not work';
301 301
 	}
Please login to merge, or discard this patch.