Passed
Push — master ( 156e0e...dfcc60 )
by Paul
04:35
created
plugin/Controllers/SettingsController.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
 	 */
17 17
 	public function callbackRegisterSettings( $input )
18 18
 	{
19
-		if( !is_array( $input )) {
19
+		if( !is_array( $input ) ) {
20 20
 			$input = ['settings' => []];
21 21
 		}
22 22
 		if( key( $input ) == 'settings' ) {
@@ -24,7 +24,7 @@  discard block
 block discarded – undo
24 24
 			$options = $this->sanitizeGeneral( $input, $options );
25 25
 			$options = $this->sanitizeSubmissions( $input, $options );
26 26
 			$options = $this->sanitizeTranslations( $input, $options );
27
-			glsr( Notice::class )->addSuccess( __( 'Settings updated.', 'site-reviews' ));
27
+			glsr( Notice::class )->addSuccess( __( 'Settings updated.', 'site-reviews' ) );
28 28
 			return $options;
29 29
 		}
30 30
 		return $input;
@@ -38,7 +38,7 @@  discard block
 block discarded – undo
38 38
 	{
39 39
 		register_setting( Application::ID.'-settings', OptionManager::databaseKey(), [
40 40
 			'sanitize_callback' => [$this, 'callbackRegisterSettings'],
41
-		]);
41
+		] );
42 42
 	}
43 43
 
44 44
 	/**
@@ -47,7 +47,7 @@  discard block
 block discarded – undo
47 47
 	protected function sanitizeGeneral( array $input, array $options )
48 48
 	{
49 49
 		$inputForm = $input['settings']['general'];
50
-		if( !isset( $inputForm['notifications'] )) {
50
+		if( !isset($inputForm['notifications']) ) {
51 51
 			$options['settings']['general']['notifications'] = [];
52 52
 		}
53 53
 		if( trim( $inputForm['notification_message'] ) == '' ) {
@@ -62,7 +62,7 @@  discard block
 block discarded – undo
62 62
 	protected function sanitizeSubmissions( array $input, array $options )
63 63
 	{
64 64
 		$inputForm = $input['settings']['submissions'];
65
-		if( !isset( $inputForm['required'] )) {
65
+		if( !isset($inputForm['required']) ) {
66 66
 			$options['settings']['submissions']['required'] = [];
67 67
 		}
68 68
 		return $options;
@@ -73,13 +73,13 @@  discard block
 block discarded – undo
73 73
 	 */
74 74
 	protected function sanitizeTranslations( array $input, array $options )
75 75
 	{
76
-		if( isset( $input['settings']['strings'] )) {
77
-			$options['settings']['strings'] = array_values( array_filter( $input['settings']['strings'] ));
76
+		if( isset($input['settings']['strings']) ) {
77
+			$options['settings']['strings'] = array_values( array_filter( $input['settings']['strings'] ) );
78 78
 			array_walk( $options['settings']['strings'], function( &$string ) {
79
-				if( isset( $string['s2'] )) {
79
+				if( isset($string['s2']) ) {
80 80
 					$string['s2'] = wp_strip_all_tags( $string['s2'] );
81 81
 				}
82
-				if( isset( $string['p2'] )) {
82
+				if( isset($string['p2']) ) {
83 83
 					$string['p2'] = wp_strip_all_tags( $string['p2'] );
84 84
 				}
85 85
 			});
Please login to merge, or discard this patch.
plugin/Modules/Html/Settings.php 1 patch
Spacing   +18 added lines, -18 removed lines patch added patch discarded remove patch
@@ -29,7 +29,7 @@  discard block
 block discarded – undo
29 29
 	 */
30 30
 	protected function getFieldDefault( array $field )
31 31
 	{
32
-		return isset( $field['default'] )
32
+		return isset($field['default'])
33 33
 			? $field['default']
34 34
 			: '';
35 35
 	}
@@ -40,7 +40,7 @@  discard block
 block discarded – undo
40 40
 	protected function getSettingFields( $path )
41 41
 	{
42 42
 		$settings = glsr( DefaultsManager::class )->settings();
43
-		return array_filter( $settings, function( $key ) use( $path ) {
43
+		return array_filter( $settings, function( $key ) use($path) {
44 44
 			return glsr( Helper::class )->startsWith( $path, $key );
45 45
 		}, ARRAY_FILTER_USE_KEY );
46 46
 	}
@@ -55,8 +55,8 @@  discard block
 block discarded – undo
55 55
 			$field = wp_parse_args( $field, [
56 56
 				'is_setting' => true,
57 57
 				'name' => $name,
58
-			]);
59
-			$rows.= new Field( $this->normalize( $field ));
58
+			] );
59
+			$rows .= new Field( $this->normalize( $field ) );
60 60
 		}
61 61
 		return $rows;
62 62
 	}
@@ -67,7 +67,7 @@  discard block
 block discarded – undo
67 67
 	 */
68 68
 	protected function getTemplateData( $id )
69 69
 	{
70
-		$fields = $this->getSettingFields( $this->normalizeSettingPath( $id ));
70
+		$fields = $this->getSettingFields( $this->normalizeSettingPath( $id ) );
71 71
 		return [
72 72
 			'context' => [
73 73
 				'rows' => $this->getSettingRows( $fields ),
@@ -81,12 +81,12 @@  discard block
 block discarded – undo
81 81
 	 */
82 82
 	protected function getTemplateDataForAddons( $id )
83 83
 	{
84
-		$fields = $this->getSettingFields( $this->normalizeSettingPath( $id ));
84
+		$fields = $this->getSettingFields( $this->normalizeSettingPath( $id ) );
85 85
 		$settings = glsr( Helper::class )->convertDotNotationArray( $fields );
86 86
 		$settingKeys = array_keys( $settings['settings']['addons'] );
87 87
 		$results = [];
88 88
 		foreach( $settingKeys as $key ) {
89
-			$addonFields = array_filter( $fields, function( $path ) use( $key ) {
89
+			$addonFields = array_filter( $fields, function( $path ) use($key) {
90 90
 				return glsr( Helper::class )->startsWith( 'settings.addons.'.$key, $path );
91 91
 			}, ARRAY_FILTER_USE_KEY );
92 92
 			$results[$key] = $this->getSettingRows( $addonFields );
@@ -102,7 +102,7 @@  discard block
 block discarded – undo
102 102
 	protected function getTemplateDataForTranslations()
103 103
 	{
104 104
 		$translations = glsr( Translator::class )->renderAll();
105
-		$class = empty( $translations )
105
+		$class = empty($translations)
106 106
 			? 'glsr-hidden'
107 107
 			: '';
108 108
 		return [
@@ -125,9 +125,9 @@  discard block
 block discarded – undo
125 125
 			$path,
126 126
 			glsr( Helper::class )->getPathValue( $path, glsr()->defaults )
127 127
 		);
128
-		if( is_array( $expectedValue )) {
128
+		if( is_array( $expectedValue ) ) {
129 129
 			return is_array( $optionValue )
130
-				? count( array_intersect( $optionValue, $expectedValue )) === 0
130
+				? count( array_intersect( $optionValue, $expectedValue ) ) === 0
131 131
 				: !in_array( $optionValue, $expectedValue );
132 132
 		}
133 133
 		return $optionValue != $expectedValue;
@@ -149,17 +149,17 @@  discard block
 block discarded – undo
149 149
 	 */
150 150
 	protected function normalizeDependsOn( array $field )
151 151
 	{
152
-		if( !empty( $field['depends_on'] ) && is_array( $field['depends_on'] )) {
152
+		if( !empty($field['depends_on']) && is_array( $field['depends_on'] ) ) {
153 153
 			$path = key( $field['depends_on'] );
154 154
 			$expectedValue = $field['depends_on'][$path];
155 155
 			$fieldName = glsr( Helper::class )->convertPathToName( $path, OptionManager::databaseKey() );
156
-			if( is_array( $expectedValue )) {
157
-				$fieldName.= '[]';
156
+			if( is_array( $expectedValue ) ) {
157
+				$fieldName .= '[]';
158 158
 			}
159
-			$field['data-depends'] = json_encode([
159
+			$field['data-depends'] = json_encode( [
160 160
 				'name' => $fieldName,
161 161
 				'value' => $expectedValue,
162
-			], JSON_HEX_APOS|JSON_HEX_QUOT );
162
+			], JSON_HEX_APOS | JSON_HEX_QUOT );
163 163
 			$field['is_hidden'] = $this->isFieldHidden( $path, $expectedValue );
164 164
 		}
165 165
 		return $field;
@@ -170,9 +170,9 @@  discard block
 block discarded – undo
170 170
 	 */
171 171
 	protected function normalizeLabelAndLegend( array $field )
172 172
 	{
173
-		if( !empty( $field['label'] )) {
173
+		if( !empty($field['label']) ) {
174 174
 			$field['legend'] = $field['label'];
175
-			unset( $field['label'] );
175
+			unset($field['label']);
176 176
 		}
177 177
 		else {
178 178
 			$field['is_valid'] = false;
@@ -186,7 +186,7 @@  discard block
 block discarded – undo
186 186
 	 */
187 187
 	protected function normalizeValue( array $field )
188 188
 	{
189
-		if( !isset( $field['value'] )) {
189
+		if( !isset($field['value']) ) {
190 190
 			$field['value'] = glsr( OptionManager::class )->get(
191 191
 				$field['name'],
192 192
 				$this->getFieldDefault( $field )
Please login to merge, or discard this patch.
plugin/Handlers/CreateReview.php 2 patches
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -17,14 +17,14 @@
 block discarded – undo
17 17
 		$review = glsr( ReviewManager::class )->create( $command );
18 18
 		if( !$review ) {
19 19
 			glsr( Session::class )->set( $command->form_id.'errors', [] );
20
-			glsr( Session::class )->set( $command->form_id.'message', __( 'Your review could not be submitted and the error has been logged. Please notify the site admin.', 'site-reviews' ));
20
+			glsr( Session::class )->set( $command->form_id.'message', __( 'Your review could not be submitted and the error has been logged. Please notify the site admin.', 'site-reviews' ) );
21 21
 			return;
22 22
 		}
23
-		glsr( Session::class )->set( $command->form_id.'message', __( 'Your review has been submitted!', 'site-reviews' ));
23
+		glsr( Session::class )->set( $command->form_id.'message', __( 'Your review has been submitted!', 'site-reviews' ) );
24 24
 		glsr( Notification::class )->send( $review );
25 25
 		do_action( 'site-reviews/local/review/submitted', $review );
26 26
 		if( $command->ajax_request )return;
27
-		if( empty( $command->referer )) {
27
+		if( empty($command->referer) ) {
28 28
 			glsr_log()->warning( 'The form referer ($_SERVER[REQUEST_URI]) was empty.' )->info( $command );
29 29
 			$command->referer = home_url();
30 30
 		}
Please login to merge, or discard this patch.
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -23,7 +23,9 @@
 block discarded – undo
23 23
 		glsr( Session::class )->set( $command->form_id.'message', __( 'Your review has been submitted!', 'site-reviews' ));
24 24
 		glsr( Notification::class )->send( $review );
25 25
 		do_action( 'site-reviews/local/review/submitted', $review );
26
-		if( $command->ajax_request )return;
26
+		if( $command->ajax_request ) {
27
+			return;
28
+		}
27 29
 		if( empty( $command->referer )) {
28 30
 			glsr_log()->warning( 'The form referer ($_SERVER[REQUEST_URI]) was empty.' )->info( $command );
29 31
 			$command->referer = home_url();
Please login to merge, or discard this patch.
plugin/Modules/Slack.php 2 patches
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -35,7 +35,9 @@
 block discarded – undo
35 35
 	 */
36 36
 	public function compose( Review $review, array $notification )
37 37
 	{
38
-		if( empty( $this->endpoint ))return;
38
+		if( empty( $this->endpoint )) {
39
+			return;
40
+		}
39 41
 		$args = shortcode_atts( glsr( SlackDefaults::class )->defaults(), $notification );
40 42
 		$this->review = $review;
41 43
 		$notification = [
Please login to merge, or discard this patch.
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -35,7 +35,7 @@  discard block
 block discarded – undo
35 35
 	 */
36 36
 	public function compose( Review $review, array $notification )
37 37
 	{
38
-		if( empty( $this->endpoint ))return;
38
+		if( empty($this->endpoint) )return;
39 39
 		$args = shortcode_atts( glsr( SlackDefaults::class )->defaults(), $notification );
40 40
 		$this->review = $review;
41 41
 		$notification = [
@@ -58,7 +58,7 @@  discard block
 block discarded – undo
58 58
 	 */
59 59
 	public function send()
60 60
 	{
61
-		if( empty( $this->endpoint )) {
61
+		if( empty($this->endpoint) ) {
62 62
 			return new WP_Error( 'slack', 'Slack notification was not sent: missing endpoint' );
63 63
 		}
64 64
 		return wp_remote_post( $this->endpoint, [
@@ -70,7 +70,7 @@  discard block
 block discarded – undo
70 70
 			'redirection' => 5,
71 71
 			'sslverify' => false,
72 72
 			'timeout' => 45,
73
-		]);
73
+		] );
74 74
 	}
75 75
 
76 76
 	/**
@@ -90,11 +90,11 @@  discard block
 block discarded – undo
90 90
 	 */
91 91
 	protected function buildAuthorField()
92 92
 	{
93
-		$email = !empty( $this->review->email )
93
+		$email = !empty($this->review->email)
94 94
 			? '<'.$this->review->email.'>'
95 95
 			: '';
96 96
 		$author = trim( rtrim( $this->review->author ).' '.$email );
97
-		return ['value' => implode( ' - ', array_filter( [$author, $this->review->ip_address] ))];
97
+		return ['value' => implode( ' - ', array_filter( [$author, $this->review->ip_address] ) )];
98 98
 	}
99 99
 
100 100
 	/**
@@ -102,7 +102,7 @@  discard block
 block discarded – undo
102 102
 	 */
103 103
 	protected function buildContentField()
104 104
 	{
105
-		return !empty( $this->review->content )
105
+		return !empty($this->review->content)
106 106
 			? ['value' => $this->review->content]
107 107
 			: [];
108 108
 	}
@@ -127,7 +127,7 @@  discard block
 block discarded – undo
127 127
 	protected function buildStarsField()
128 128
 	{
129 129
 		$solidStars = str_repeat( '★', $this->review->rating );
130
-		$emptyStars = str_repeat( '☆', max( 0, Rating::MAX_RATING - $this->review->rating ));
130
+		$emptyStars = str_repeat( '☆', max( 0, Rating::MAX_RATING - $this->review->rating ) );
131 131
 		$stars = $solidStars.$emptyStars;
132 132
 		$stars = apply_filters( 'site-reviews/slack/stars', $stars, $this->review->rating, Rating::MAX_RATING );
133 133
 		return ['title' => $stars];
@@ -138,7 +138,7 @@  discard block
 block discarded – undo
138 138
 	 */
139 139
 	protected function buildTitleField()
140 140
 	{
141
-		return !empty( $this->review->title )
141
+		return !empty($this->review->title)
142 142
 			? ['title' => $this->review->title]
143 143
 			: [];
144 144
 	}
Please login to merge, or discard this patch.
plugin/Modules/Html/Partials/SiteReviewsForm.php 2 patches
Spacing   +29 added lines, -29 removed lines patch added patch discarded remove patch
@@ -65,7 +65,7 @@  discard block
 block discarded – undo
65 65
 				'response' => $this->buildResponse(),
66 66
 				'submit_button' => $this->buildSubmitButton().$this->buildRecaptcha(),
67 67
 			],
68
-		]);
68
+		] );
69 69
 	}
70 70
 
71 71
 	/**
@@ -77,7 +77,7 @@  discard block
 block discarded – undo
77 77
 			'context' => [
78 78
 				'text' => trim( $this->getLoginText().' '.$this->getRegisterText() ),
79 79
 			],
80
-		]);
80
+		] );
81 81
 	}
82 82
 
83 83
 	/**
@@ -86,12 +86,12 @@  discard block
 block discarded – undo
86 86
 	protected function buildRecaptcha()
87 87
 	{
88 88
 		if( !glsr( OptionManager::class )->isRecaptchaEnabled() )return;
89
-		return glsr( Builder::class )->div([
89
+		return glsr( Builder::class )->div( [
90 90
 			'class' => 'glsr-recaptcha-holder',
91 91
 			'data-badge' => glsr( OptionManager::class )->get( 'settings.submissions.recaptcha.position' ),
92
-			'data-sitekey' => sanitize_text_field( glsr( OptionManager::class )->get( 'settings.submissions.recaptcha.key' )),
92
+			'data-sitekey' => sanitize_text_field( glsr( OptionManager::class )->get( 'settings.submissions.recaptcha.key' ) ),
93 93
 			'data-size' => 'invisible',
94
-		]);
94
+		] );
95 95
 	}
96 96
 
97 97
 	/**
@@ -99,7 +99,7 @@  discard block
 block discarded – undo
99 99
 	 */
100 100
 	protected function buildResponse()
101 101
 	{
102
-		$classes = !empty( $this->errors )
102
+		$classes = !empty($this->errors)
103 103
 			? glsr( StyleValidationDefaults::class )->defaults()['message_error_class']
104 104
 			: '';
105 105
 		return glsr( Template::class )->build( 'templates/form/response', [
@@ -107,8 +107,8 @@  discard block
 block discarded – undo
107 107
 				'class' => $classes,
108 108
 				'message' => wpautop( $this->message ),
109 109
 			],
110
-			'has_errors' => !empty( $this->errors ),
111
-		]);
110
+			'has_errors' => !empty($this->errors),
111
+		] );
112 112
 	}
113 113
 
114 114
 	/**
@@ -120,7 +120,7 @@  discard block
 block discarded – undo
120 120
 			'context' => [
121 121
 				'text' => __( 'Submit your review', 'site-reviews' ),
122 122
 			],
123
-		]);
123
+		] );
124 124
 	}
125 125
 
126 126
 	/**
@@ -139,7 +139,7 @@  discard block
 block discarded – undo
139 139
 		$fields = array_merge(
140 140
 			$this->getHiddenFields(),
141 141
 			[$this->getHoneypotField()],
142
-			$this->normalizeFields( glsr( Form::class )->getFields( 'submission-form' ))
142
+			$this->normalizeFields( glsr( Form::class )->getFields( 'submission-form' ) )
143 143
 		);
144 144
 		return $fields;
145 145
 	}
@@ -149,10 +149,10 @@  discard block
 block discarded – undo
149 149
 	 */
150 150
 	protected function getLoginText()
151 151
 	{
152
-		$loginLink = glsr( Builder::class )->a([
153
-			'href' => wp_login_url( strval( get_permalink() )),
152
+		$loginLink = glsr( Builder::class )->a( [
153
+			'href' => wp_login_url( strval( get_permalink() ) ),
154 154
 			'text' => __( 'logged in', 'site-reviews' ),
155
-		]);
155
+		] );
156 156
 		return sprintf( __( 'You must be %s to submit a review.', 'site-reviews' ), $loginLink );
157 157
 	}
158 158
 
@@ -162,10 +162,10 @@  discard block
 block discarded – undo
162 162
 	protected function getRegisterText()
163 163
 	{
164 164
 		if( !get_option( 'users_can_register' ) || glsr( OptionManager::class )->get( 'settings.general.require.login' ) != 'yes' )return;
165
-		$registerLink = glsr( Builder::class )->a([
165
+		$registerLink = glsr( Builder::class )->a( [
166 166
 			'href' => wp_registration_url(),
167 167
 			'text' => __( 'register', 'site-reviews' ),
168
-		]);
168
+		] );
169 169
 		return sprintf( __( 'You may also %s for an account.', 'site-reviews' ), $registerLink );
170 170
 	}
171 171
 
@@ -177,27 +177,27 @@  discard block
 block discarded – undo
177 177
 		$fields = [[
178 178
 			'name' => 'action',
179 179
 			'value' => 'submit-review',
180
-		],[
180
+		], [
181 181
 			'name' => 'assign_to',
182 182
 			'value' => $this->args['assign_to'],
183
-		],[
183
+		], [
184 184
 			'name' => 'category',
185 185
 			'value' => $this->args['category'],
186
-		],[
186
+		], [
187 187
 			'name' => 'excluded',
188 188
 			'value' => $this->args['excluded'], // @todo should default to "[]"
189
-		],[
189
+		], [
190 190
 			'name' => 'form_id',
191 191
 			'value' => $this->args['id'],
192
-		],[
192
+		], [
193 193
 			'name' => 'nonce',
194 194
 			'value' => wp_create_nonce( 'submit-review' ),
195
-		],[
195
+		], [
196 196
 			'name' => 'referer',
197
-			'value' => wp_unslash( filter_input( INPUT_SERVER, 'REQUEST_URI' )),
197
+			'value' => wp_unslash( filter_input( INPUT_SERVER, 'REQUEST_URI' ) ),
198 198
 		]];
199 199
 		return array_map( function( $field ) {
200
-			return new Field( wp_parse_args( $field, ['type' => 'hidden'] ));
200
+			return new Field( wp_parse_args( $field, ['type' => 'hidden'] ) );
201 201
 		}, $fields );
202 202
 	}
203 203
 
@@ -206,10 +206,10 @@  discard block
 block discarded – undo
206 206
 	 */
207 207
 	protected function getHoneypotField()
208 208
 	{
209
-		return new Field([
209
+		return new Field( [
210 210
 			'name' => 'gotcha',
211 211
 			'type' => 'honeypot',
212
-		]);
212
+		] );
213 213
 	}
214 214
 
215 215
 	/**
@@ -217,7 +217,7 @@  discard block
 block discarded – undo
217 217
 	 */
218 218
 	protected function normalizeFieldErrors( Field &$field )
219 219
 	{
220
-		if( !array_key_exists( $field->field['path'], $this->errors ))return;
220
+		if( !array_key_exists( $field->field['path'], $this->errors ) )return;
221 221
 		$field->field['errors'] = $this->errors[$field->field['path']];
222 222
 	}
223 223
 
@@ -226,7 +226,7 @@  discard block
 block discarded – undo
226 226
 	 */
227 227
 	protected function normalizeFieldRequired( Field &$field )
228 228
 	{
229
-		if( !in_array( $field->field['path'], $this->required ))return;
229
+		if( !in_array( $field->field['path'], $this->required ) )return;
230 230
 		$field->field['required'] = true;
231 231
 	}
232 232
 
@@ -249,8 +249,8 @@  discard block
 block discarded – undo
249 249
 	 */
250 250
 	protected function normalizeFieldValue( Field &$field )
251 251
 	{
252
-		if( !array_key_exists( $field->field['path'], $this->values ))return;
253
-		if( in_array( $field->field['type'], ['radio', 'checkbox'] )) {
252
+		if( !array_key_exists( $field->field['path'], $this->values ) )return;
253
+		if( in_array( $field->field['type'], ['radio', 'checkbox'] ) ) {
254 254
 			$field->field['checked'] = $field->field['value'] == $this->values[$field->field['path']];
255 255
 		}
256 256
 		else {
Please login to merge, or discard this patch.
Braces   +15 added lines, -5 removed lines patch added patch discarded remove patch
@@ -85,7 +85,9 @@  discard block
 block discarded – undo
85 85
 	 */
86 86
 	protected function buildRecaptcha()
87 87
 	{
88
-		if( !glsr( OptionManager::class )->isRecaptchaEnabled() )return;
88
+		if( !glsr( OptionManager::class )->isRecaptchaEnabled() ) {
89
+			return;
90
+		}
89 91
 		return glsr( Builder::class )->div([
90 92
 			'class' => 'glsr-recaptcha-holder',
91 93
 			'data-badge' => glsr( OptionManager::class )->get( 'settings.submissions.recaptcha.position' ),
@@ -161,7 +163,9 @@  discard block
 block discarded – undo
161 163
 	 */
162 164
 	protected function getRegisterText()
163 165
 	{
164
-		if( !get_option( 'users_can_register' ) || glsr( OptionManager::class )->get( 'settings.general.require.login' ) != 'yes' )return;
166
+		if( !get_option( 'users_can_register' ) || glsr( OptionManager::class )->get( 'settings.general.require.login' ) != 'yes' ) {
167
+			return;
168
+		}
165 169
 		$registerLink = glsr( Builder::class )->a([
166 170
 			'href' => wp_registration_url(),
167 171
 			'text' => __( 'register', 'site-reviews' ),
@@ -217,7 +221,9 @@  discard block
 block discarded – undo
217 221
 	 */
218 222
 	protected function normalizeFieldErrors( Field &$field )
219 223
 	{
220
-		if( !array_key_exists( $field->field['path'], $this->errors ))return;
224
+		if( !array_key_exists( $field->field['path'], $this->errors )) {
225
+			return;
226
+		}
221 227
 		$field->field['errors'] = $this->errors[$field->field['path']];
222 228
 	}
223 229
 
@@ -226,7 +232,9 @@  discard block
 block discarded – undo
226 232
 	 */
227 233
 	protected function normalizeFieldRequired( Field &$field )
228 234
 	{
229
-		if( !in_array( $field->field['path'], $this->required ))return;
235
+		if( !in_array( $field->field['path'], $this->required )) {
236
+			return;
237
+		}
230 238
 		$field->field['required'] = true;
231 239
 	}
232 240
 
@@ -249,7 +257,9 @@  discard block
 block discarded – undo
249 257
 	 */
250 258
 	protected function normalizeFieldValue( Field &$field )
251 259
 	{
252
-		if( !array_key_exists( $field->field['path'], $this->values ))return;
260
+		if( !array_key_exists( $field->field['path'], $this->values )) {
261
+			return;
262
+		}
253 263
 		if( in_array( $field->field['type'], ['radio', 'checkbox'] )) {
254 264
 			$field->field['checked'] = $field->field['value'] == $this->values[$field->field['path']];
255 265
 		}
Please login to merge, or discard this patch.
deprecated.php 1 patch
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -3,61 +3,61 @@
 block discarded – undo
3 3
 defined( 'WPINC' ) || die;
4 4
 
5 5
 add_action( 'site-reviews/local/review/submitted', function() {
6
-	if( has_filter( 'site-reviews/local/review/submitted/message' )) {
6
+	if( has_filter( 'site-reviews/local/review/submitted/message' ) ) {
7 7
 		glsr_log()->notice( 'The "site-reviews/local/review/submitted/message" hook has been deprecated.' );
8 8
 	}
9 9
 });
10 10
 
11 11
 add_filter( 'site-reviews/create/review-values', function( $values ) {
12
-	if( has_filter( 'site-reviews/local/review' )) {
12
+	if( has_filter( 'site-reviews/local/review' ) ) {
13 13
 		glsr_log()->notice( 'The "site-reviews/local/review" hook has been deprecated. Please use the "site-reviews/create/review-values" hook instead.' );
14 14
 	}
15 15
 	return $values;
16 16
 });
17 17
 
18 18
 add_filter( 'site-reviews/get/defaults', function( $defaults ) {
19
-	if( has_filter( 'site-reviews/addon/defaults' )) {
19
+	if( has_filter( 'site-reviews/addon/defaults' ) ) {
20 20
 		glsr_log()->notice( 'The "site-reviews/addon/defaults" hook has been deprecated. Please use the "site-reviews/get/defaults" hook instead.' );
21 21
 	}
22 22
 	return $defaults;
23 23
 });
24 24
 
25 25
 add_filter( 'site-reviews/rating/average', function( $average ) {
26
-	if( has_filter( 'site-reviews/average/rating' )) {
26
+	if( has_filter( 'site-reviews/average/rating' ) ) {
27 27
 		glsr_log()->notice( 'The "site-reviews/average/rating" hook has been deprecated. Please use the "site-reviews/rating/average" hook instead.' );
28 28
 	}
29 29
 	return $average;
30 30
 });
31 31
 
32 32
 add_filter( 'site-reviews/rating/ranking', function( $ranking ) {
33
-	if( has_filter( 'site-reviews/bayesian/ranking' )) {
33
+	if( has_filter( 'site-reviews/bayesian/ranking' ) ) {
34 34
 		glsr_log()->notice( 'The "site-reviews/bayesian/ranking" hook has been deprecated. Please use the "site-reviews/rating/ranking" hook instead.' );
35 35
 	}
36 36
 	return $ranking;
37 37
 });
38 38
 
39 39
 add_filter( 'site-reviews/review/build/before', function( $review ) {
40
-	if( has_filter( 'site-reviews/rendered/review/meta/order' )) {
40
+	if( has_filter( 'site-reviews/rendered/review/meta/order' ) ) {
41 41
 		glsr_log()->notice( 'The "site-reviews/rendered/review/meta/order" hook has been deprecated. Please use a custom template instead (refer to the documentation).' );
42 42
 	}
43
-	if( has_filter( 'site-reviews/rendered/review/order' )) {
43
+	if( has_filter( 'site-reviews/rendered/review/order' ) ) {
44 44
 		glsr_log()->notice( 'The "site-reviews/rendered/review/order" hook has been deprecated. Please use a custom template instead (refer to the documentation).' );
45 45
 	}
46 46
 	return $review;
47 47
 });
48 48
 
49 49
 add_filter( 'site-reviews/review/build/after', function( $review ) {
50
-	if( has_filter( 'site-reviews/reviews/review/text' )) {
50
+	if( has_filter( 'site-reviews/reviews/review/text' ) ) {
51 51
 		glsr_log()->notice( 'The "site-reviews/reviews/review/text" hook has been deprecated. Please use the "site-reviews/review/build/after" hook instead.' );
52 52
 	}
53
-	if( has_filter( 'site-reviews/reviews/review/title' )) {
53
+	if( has_filter( 'site-reviews/reviews/review/title' ) ) {
54 54
 		glsr_log()->notice( 'The "site-reviews/reviews/review/title" hook has been deprecated. Please use the "site-reviews/review/build/after" hook instead.' );
55 55
 	}
56 56
 	return $review;
57 57
 });
58 58
 
59 59
 add_filter( 'site-reviews/enqueue/public/localize', function( $variables ) {
60
-	if( has_filter( 'site-reviews/enqueue/localize' )) {
60
+	if( has_filter( 'site-reviews/enqueue/localize' ) ) {
61 61
 		glsr_log()->notice( 'The "site-reviews/enqueue/localize" hook has been deprecated. Please use the "site-reviews/enqueue/public/localize" hook instead.' );
62 62
 	}
63 63
 	return $variables;
Please login to merge, or discard this patch.
plugin/Shortcodes/SiteReviewsButton.php 1 patch
Spacing   +23 added lines, -23 removed lines patch added patch discarded remove patch
@@ -13,15 +13,15 @@  discard block
 block discarded – undo
13 13
 	public function fields()
14 14
 	{
15 15
 		return [[
16
-			'html' => sprintf( '<p class="strong">%s</p>', esc_html__( 'All settings are optional.', 'site-reviews' )),
16
+			'html' => sprintf( '<p class="strong">%s</p>', esc_html__( 'All settings are optional.', 'site-reviews' ) ),
17 17
 			'minWidth' => 320,
18 18
 			'type' => 'container',
19
-		],[
19
+		], [
20 20
 			'label' => esc_html__( 'Title', 'site-reviews' ),
21 21
 			'name' => 'title',
22 22
 			'tooltip' => esc_attr__( 'Enter a custom shortcode heading.', 'site-reviews' ),
23 23
 			'type' => 'textbox',
24
-		],[
24
+		], [
25 25
 			'label' => esc_html__( 'Count', 'site-reviews' ),
26 26
 			'maxLength' => 5,
27 27
 			'name' => 'count',
@@ -29,20 +29,20 @@  discard block
 block discarded – undo
29 29
 			'text' => '10',
30 30
 			'tooltip' => esc_attr__( 'How many reviews would you like to display (default: 10)?', 'site-reviews' ),
31 31
 			'type' => 'textbox',
32
-		],[
32
+		], [
33 33
 			'label' => esc_html__( 'Rating', 'site-reviews' ),
34 34
 			'name' => 'rating',
35 35
 			'options' => [
36
-				'5' => esc_html( sprintf( _n( '%s star', '%s stars', 5, 'site-reviews' ), 5 )),
37
-				'4' => esc_html( sprintf( _n( '%s star', '%s stars', 4, 'site-reviews' ), 4 )),
38
-				'3' => esc_html( sprintf( _n( '%s star', '%s stars', 3, 'site-reviews' ), 3 )),
39
-				'2' => esc_html( sprintf( _n( '%s star', '%s stars', 2, 'site-reviews' ), 2 )),
40
-				'1' => esc_html( sprintf( _n( '%s star', '%s stars', 1, 'site-reviews' ), 1 )),
41
-				'0' => esc_html( __( 'Unrated', 'site-reviews' )),
36
+				'5' => esc_html( sprintf( _n( '%s star', '%s stars', 5, 'site-reviews' ), 5 ) ),
37
+				'4' => esc_html( sprintf( _n( '%s star', '%s stars', 4, 'site-reviews' ), 4 ) ),
38
+				'3' => esc_html( sprintf( _n( '%s star', '%s stars', 3, 'site-reviews' ), 3 ) ),
39
+				'2' => esc_html( sprintf( _n( '%s star', '%s stars', 2, 'site-reviews' ), 2 ) ),
40
+				'1' => esc_html( sprintf( _n( '%s star', '%s stars', 1, 'site-reviews' ), 1 ) ),
41
+				'0' => esc_html( __( 'Unrated', 'site-reviews' ) ),
42 42
 			],
43 43
 			'tooltip' => esc_attr__( 'What is the minimum rating to display (default: 1 star)?', 'site-reviews' ),
44 44
 			'type' => 'listbox',
45
-		],[
45
+		], [
46 46
 			'label' => esc_html__( 'Pagination', 'site-reviews' ),
47 47
 			'name' => 'pagination',
48 48
 			'options' => [
@@ -60,7 +60,7 @@  discard block
 block discarded – undo
60 60
 			'name' => 'assigned_to',
61 61
 			'tooltip' => esc_attr__( 'Limit reviews to those assigned to this post ID (separate multiple IDs with a comma). You can also enter "post_id" to use the ID of the current page.', 'site-reviews' ),
62 62
 			'type' => 'textbox',
63
-		],[
63
+		], [
64 64
 			'label' => esc_html__( 'Schema', 'site-reviews' ),
65 65
 			'name' => 'schema',
66 66
 			'options' => [
@@ -69,49 +69,49 @@  discard block
 block discarded – undo
69 69
 			],
70 70
 			'tooltip' => esc_attr__( 'Rich snippets are disabled by default.', 'site-reviews' ),
71 71
 			'type' => 'listbox',
72
-		],[
72
+		], [
73 73
 			'label' => esc_html__( 'Classes', 'site-reviews' ),
74 74
 			'name' => 'class',
75 75
 			'tooltip' => esc_attr__( 'Add custom CSS classes to the shortcode.', 'site-reviews' ),
76 76
 			'type' => 'textbox',
77
-		],[
77
+		], [
78 78
 			'columns' => 2,
79 79
 			'items' => [[
80 80
 				'name' => 'hide_assigned_to',
81 81
 				'text' => esc_html__( 'Assigned To', 'site-reviews' ),
82 82
 				'tooltip' => esc_attr__( 'Hide the assigned to link?', 'site-reviews' ),
83 83
 				'type' => 'checkbox',
84
-			],[
84
+			], [
85 85
 				'name' => 'hide_author',
86 86
 				'text' => esc_html__( 'Author', 'site-reviews' ),
87 87
 				'tooltip' => esc_attr__( 'Hide the review author?', 'site-reviews' ),
88 88
 				'type' => 'checkbox',
89
-			],[
89
+			], [
90 90
 				'name' => 'hide_avatar',
91 91
 				'text' => esc_html__( 'Avatar', 'site-reviews' ),
92 92
 				'tooltip' => esc_attr__( 'Hide the reviewer avatar if shown?', 'site-reviews' ),
93 93
 				'type' => 'checkbox',
94
-			],[
94
+			], [
95 95
 				'name' => 'hide_content',
96 96
 				'text' => esc_html__( 'Content', 'site-reviews' ),
97 97
 				'tooltip' => esc_attr__( 'Hide the review content?', 'site-reviews' ),
98 98
 				'type' => 'checkbox',
99
-			],[
99
+			], [
100 100
 				'name' => 'hide_date',
101 101
 				'text' => esc_html__( 'Date', 'site-reviews' ),
102 102
 				'tooltip' => esc_attr__( 'Hide the review date?', 'site-reviews' ),
103 103
 				'type' => 'checkbox',
104
-			],[
104
+			], [
105 105
 				'name' => 'hide_rating',
106 106
 				'text' => esc_html__( 'Rating', 'site-reviews' ),
107 107
 				'tooltip' => esc_attr__( 'Hide the review rating?', 'site-reviews' ),
108 108
 				'type' => 'checkbox',
109
-			],[
109
+			], [
110 110
 				'name' => 'hide_response',
111 111
 				'text' => esc_html__( 'Response', 'site-reviews' ),
112 112
 				'tooltip' => esc_attr__( 'Hide the review response?', 'site-reviews' ),
113 113
 				'type' => 'checkbox',
114
-			],[
114
+			], [
115 115
 				'name' => 'hide_title',
116 116
 				'text' => esc_html__( 'Title', 'site-reviews' ),
117 117
 				'tooltip' => esc_attr__( 'Hide the review title?', 'site-reviews' ),
@@ -121,7 +121,7 @@  discard block
 block discarded – undo
121 121
 			'label' => esc_html__( 'Hide', 'site-reviews' ),
122 122
 			'spacing' => 5,
123 123
 			'type' => 'container',
124
-		],[
124
+		], [
125 125
 			'hidden' => true,
126 126
 			'name' => 'id',
127 127
 			'type' => 'textbox',
@@ -134,7 +134,7 @@  discard block
 block discarded – undo
134 134
 	public function getTerms()
135 135
 	{
136 136
 		$terms = glsr( Database::class )->getTerms();
137
-		if( empty( $terms )) {
137
+		if( empty($terms) ) {
138 138
 			return [];
139 139
 		}
140 140
 		return [
Please login to merge, or discard this patch.
plugin/Modules/Notification.php 2 patches
Spacing   +16 added lines, -16 removed lines patch added patch discarded remove patch
@@ -33,7 +33,7 @@  discard block
 block discarded – undo
33 33
 	public function __construct()
34 34
 	{
35 35
 		$types = glsr( OptionManager::class )->get( 'settings.general.notifications', [] );
36
-		$this->email = count( array_intersect( ['admin', 'custom'], $types )) > 0;
36
+		$this->email = count( array_intersect( ['admin', 'custom'], $types ) ) > 0;
37 37
 		$this->slack = in_array( 'slack', $types );
38 38
 		$this->types = $types;
39 39
 	}
@@ -43,7 +43,7 @@  discard block
 block discarded – undo
43 43
 	 */
44 44
 	public function send( Review $review )
45 45
 	{
46
-		if( empty( $this->types ))return;
46
+		if( empty($this->types) )return;
47 47
 		$this->review = $review;
48 48
 		$args = [
49 49
 			'link' => $this->getLink(),
@@ -58,7 +58,7 @@  discard block
 block discarded – undo
58 58
 	 */
59 59
 	protected function buildEmail( array $args )
60 60
 	{
61
-		return glsr( Email::class )->compose([
61
+		return glsr( Email::class )->compose( [
62 62
 			'to' => $this->getEmailAddresses(),
63 63
 			'subject' => $args['title'],
64 64
 			'template' => 'email-notification',
@@ -71,7 +71,7 @@  discard block
 block discarded – undo
71 71
 				'review_rating' => $this->review->rating,
72 72
 				'review_title' => $this->review->title,
73 73
 			],
74
-		]);
74
+		] );
75 75
 	}
76 76
 
77 77
 	/**
@@ -83,7 +83,7 @@  discard block
 block discarded – undo
83 83
 			'button_url' => $args['link'],
84 84
 			'fallback' => $this->buildEmail( $args )->read( 'plaintext' ),
85 85
 			'pretext' => $args['title'],
86
-		]);
86
+		] );
87 87
 	}
88 88
 
89 89
 	/**
@@ -92,23 +92,23 @@  discard block
 block discarded – undo
92 92
 	protected function getEmailAddresses()
93 93
 	{
94 94
 		$emails = [];
95
-		if( in_array( 'admin', $this->types )) {
95
+		if( in_array( 'admin', $this->types ) ) {
96 96
 			$emails[] = get_option( 'admin_email' );
97 97
 		}
98
-		if( in_array( 'author', $this->types )) {
99
-			$assignedPost = get_post( intval( $this->review->assigned_to ));
98
+		if( in_array( 'author', $this->types ) ) {
99
+			$assignedPost = get_post( intval( $this->review->assigned_to ) );
100 100
 			if( $assignedPost instanceof WP_Post ) {
101 101
 				$this->email = true;
102
-				$emails[] = get_the_author_meta( 'user_email', intval( $assignedPost->post_author ));
102
+				$emails[] = get_the_author_meta( 'user_email', intval( $assignedPost->post_author ) );
103 103
 			}
104 104
 		}
105
-		if( in_array( 'custom', $this->types )) {
105
+		if( in_array( 'custom', $this->types ) ) {
106 106
 			$customEmails = glsr( OptionManager::class )->get( 'settings.general.notification_email' );
107 107
 			$customEmails = str_replace( [' ', ',', ';'], ',', $customEmails );
108 108
 			$customEmails = explode( ',', $customEmails );
109 109
 			$emails = array_merge( $emails, $customEmails );
110 110
 		}
111
-		$emails = array_filter( array_keys( array_flip( $emails )));
111
+		$emails = array_filter( array_keys( array_flip( $emails ) ) );
112 112
 		return apply_filters( 'site-reviews/notification/emails', $emails, $this->review );
113 113
 	}
114 114
 
@@ -125,16 +125,16 @@  discard block
 block discarded – undo
125 125
 	 */
126 126
 	protected function getTitle()
127 127
 	{
128
-		$assignedTitle = get_the_title( intval( $this->review->assigned_to ));
128
+		$assignedTitle = get_the_title( intval( $this->review->assigned_to ) );
129 129
 		$title = _nx(
130 130
 			'New %s-star review',
131 131
 			'New %s-star review of: %s',
132
-			intval( empty( $assignedTitle )),
132
+			intval( empty($assignedTitle) ),
133 133
 			'This string differs depending on whether or not the review has been assigned to a post.',
134 134
 			'site-reviews'
135 135
 		);
136 136
 		$title = sprintf( '[%s] %s',
137
-			wp_specialchars_decode( strval( get_option( 'blogname' )), ENT_QUOTES ),
137
+			wp_specialchars_decode( strval( get_option( 'blogname' ) ), ENT_QUOTES ),
138 138
 			sprintf( $title, $this->review->rating, $assignedTitle )
139 139
 		);
140 140
 		return apply_filters( 'site-reviews/notification/title', $title, $this->review );
@@ -147,7 +147,7 @@  discard block
 block discarded – undo
147 147
 	{
148 148
 		$email = $this->buildEmail( $args );
149 149
 		if( !$this->email )return;
150
-		if( empty( $email->to )) {
150
+		if( empty($email->to) ) {
151 151
 			glsr_log()->error( 'Email notification was not sent: missing email address' );
152 152
 			return;
153 153
 		}
@@ -164,7 +164,7 @@  discard block
 block discarded – undo
164 164
 		if( !$this->slack )return;
165 165
 		$notification = $this->buildSlackNotification( $args );
166 166
 		$result = $notification->send();
167
-		if( is_wp_error( $result )) {
167
+		if( is_wp_error( $result ) ) {
168 168
 			$notification->review = null;
169 169
 			glsr_log()->error( $result->get_error_message() )->debug( $notification );
170 170
 		}
Please login to merge, or discard this patch.
Braces   +9 added lines, -3 removed lines patch added patch discarded remove patch
@@ -43,7 +43,9 @@  discard block
 block discarded – undo
43 43
 	 */
44 44
 	public function send( Review $review )
45 45
 	{
46
-		if( empty( $this->types ))return;
46
+		if( empty( $this->types )) {
47
+			return;
48
+		}
47 49
 		$this->review = $review;
48 50
 		$args = [
49 51
 			'link' => $this->getLink(),
@@ -146,7 +148,9 @@  discard block
 block discarded – undo
146 148
 	protected function sendToEmail( array $args )
147 149
 	{
148 150
 		$email = $this->buildEmail( $args );
149
-		if( !$this->email )return;
151
+		if( !$this->email ) {
152
+			return;
153
+		}
150 154
 		if( empty( $email->to )) {
151 155
 			glsr_log()->error( 'Email notification was not sent: missing email address' );
152 156
 			return;
@@ -161,7 +165,9 @@  discard block
 block discarded – undo
161 165
 	 */
162 166
 	protected function sendToSlack( array $args )
163 167
 	{
164
-		if( !$this->slack )return;
168
+		if( !$this->slack ) {
169
+			return;
170
+		}
165 171
 		$notification = $this->buildSlackNotification( $args );
166 172
 		$result = $notification->send();
167 173
 		if( is_wp_error( $result )) {
Please login to merge, or discard this patch.
plugin/Modules/Blacklist.php 2 patches
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -11,13 +11,13 @@  discard block
 block discarded – undo
11 11
 	 */
12 12
 	public function isBlacklisted( array $review )
13 13
 	{
14
-		$target = implode( "\n", array_filter([
14
+		$target = implode( "\n", array_filter( [
15 15
 			$review['name'],
16 16
 			$review['content'],
17 17
 			$review['email'],
18 18
 			$review['ip_address'],
19 19
 			$review['title'],
20
-		]));
20
+		] ) );
21 21
 		return (bool)apply_filters( 'site-reviews/blacklist/is-blacklisted',
22 22
 			$this->check( $target ),
23 23
 			$review
@@ -30,16 +30,16 @@  discard block
 block discarded – undo
30 30
 	 */
31 31
 	protected function check( $target )
32 32
 	{
33
-		$blacklist = trim( glsr( OptionManager::class )->get( 'settings.submissions.blacklist.entries' ));
34
-		if( empty( $blacklist )) {
33
+		$blacklist = trim( glsr( OptionManager::class )->get( 'settings.submissions.blacklist.entries' ) );
34
+		if( empty($blacklist) ) {
35 35
 			return false;
36 36
 		}
37 37
 		$lines = explode( "\n", $blacklist );
38 38
 		foreach( (array)$lines as $line ) {
39 39
 			$line = trim( $line );
40
-			if( empty( $line ) || 256 < strlen( $line ))continue;
41
-			$pattern = sprintf( '#%s#i', preg_quote( $line, '#' ));
42
-			if( preg_match( $pattern, $target )) {
40
+			if( empty($line) || 256 < strlen( $line ) )continue;
41
+			$pattern = sprintf( '#%s#i', preg_quote( $line, '#' ) );
42
+			if( preg_match( $pattern, $target ) ) {
43 43
 				return true;
44 44
 			}
45 45
 		}
Please login to merge, or discard this patch.
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -37,7 +37,9 @@
 block discarded – undo
37 37
 		$lines = explode( "\n", $blacklist );
38 38
 		foreach( (array)$lines as $line ) {
39 39
 			$line = trim( $line );
40
-			if( empty( $line ) || 256 < strlen( $line ))continue;
40
+			if( empty( $line ) || 256 < strlen( $line )) {
41
+				continue;
42
+			}
41 43
 			$pattern = sprintf( '#%s#i', preg_quote( $line, '#' ));
42 44
 			if( preg_match( $pattern, $target )) {
43 45
 				return true;
Please login to merge, or discard this patch.