Passed
Push — master ( 36c2a1...ae1d0b )
by Paul
05:48
created
plugin/Modules/Upgrader/Upgrade_3_0_0.php 2 patches
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -74,9 +74,9 @@  discard block
 block discarded – undo
74 74
 	{
75 75
 		$this->newSettings = $this->getNewSettings();
76 76
 		$this->oldSettings = $this->getOldSettings();
77
-		if( empty( $this->oldSettings ))return;
77
+		if( empty($this->oldSettings) )return;
78 78
 		foreach( static::MAPPED_SETTINGS as $old => $new ) {
79
-			if( empty( $this->oldSettings[$old] ))continue;
79
+			if( empty($this->oldSettings[$old]) )continue;
80 80
 			$this->newSettings[$new] = $this->oldSettings[$old];
81 81
 		}
82 82
 		$this->migrateNotificationSettings();
@@ -84,7 +84,7 @@  discard block
 block discarded – undo
84 84
 		$this->migrateRequiredSettings();
85 85
 		$oldSettings = glsr( Helper::class )->convertDotNotationArray( $this->oldSettings );
86 86
 		$newSettings = glsr( Helper::class )->convertDotNotationArray( $this->newSettings );
87
-		if( isset( $oldSettings['settings']['strings'] ) && is_array( $oldSettings['settings']['strings'] )) {
87
+		if( isset($oldSettings['settings']['strings']) && is_array( $oldSettings['settings']['strings'] ) ) {
88 88
 			$newSettings['settings']['strings'] = $oldSettings['settings']['strings'];
89 89
 		}
90 90
 		glsr( OptionManager::class )->set( $newSettings );
@@ -117,8 +117,8 @@  discard block
 block discarded – undo
117 117
 	protected function getOldSettings()
118 118
 	{
119 119
 		$defaults = array_fill_keys( array_keys( static::MAPPED_SETTINGS ), '' );
120
-		$settings = glsr( Helper::class )->flattenArray( get_option( 'geminilabs_site_reviews-v2', [] ));
121
-		if( !empty( $settings )) {
120
+		$settings = glsr( Helper::class )->flattenArray( get_option( 'geminilabs_site_reviews-v2', [] ) );
121
+		if( !empty($settings) ) {
122 122
 			$settings = wp_parse_args( $settings, $defaults );
123 123
 		}
124 124
 		return $settings;
@@ -151,7 +151,7 @@  discard block
 block discarded – undo
151 151
 			'SecretKey' => $this->oldSettings['settings.reviews-form.recaptcha.secret'],
152 152
 			'SiteKey' => $this->oldSettings['settings.reviews-form.recaptcha.key'],
153 153
 		];
154
-		if( in_array( $this->oldSettings['settings.reviews-form.recaptcha.integration'], ['custom', 'invisible-recaptcha'] )) {
154
+		if( in_array( $this->oldSettings['settings.reviews-form.recaptcha.integration'], ['custom', 'invisible-recaptcha'] ) ) {
155 155
 			$this->newSettings['settings.submissions.recaptcha.integration'] = 'all';
156 156
 		}
157 157
 		if( $this->oldSettings['settings.reviews-form.recaptcha.integration'] == 'invisible-recaptcha' ) {
Please login to merge, or discard this patch.
Braces   +9 added lines, -3 removed lines patch added patch discarded remove patch
@@ -74,9 +74,13 @@  discard block
 block discarded – undo
74 74
 	{
75 75
 		$this->newSettings = $this->getNewSettings();
76 76
 		$this->oldSettings = $this->getOldSettings();
77
-		if( empty( $this->oldSettings ))return;
77
+		if( empty( $this->oldSettings )) {
78
+			return;
79
+		}
78 80
 		foreach( static::MAPPED_SETTINGS as $old => $new ) {
79
-			if( empty( $this->oldSettings[$old] ))continue;
81
+			if( empty( $this->oldSettings[$old] )) {
82
+				continue;
83
+			}
80 84
 			$this->newSettings[$new] = $this->oldSettings[$old];
81 85
 		}
82 86
 		$this->migrateNotificationSettings();
@@ -136,7 +140,9 @@  discard block
 block discarded – undo
136 140
 		];
137 141
 		$this->newSettings['settings.general.notifications'] = [];
138 142
 		foreach( $notifications as $old => $new ) {
139
-			if( $this->oldSettings['settings.general.notification'] != $old )continue;
143
+			if( $this->oldSettings['settings.general.notification'] != $old ) {
144
+				continue;
145
+			}
140 146
 			$this->newSettings['settings.general.notifications'][] = $new;
141 147
 		}
142 148
 	}
Please login to merge, or discard this patch.
plugin/Handlers/RegisterTinymcePopups.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -14,14 +14,14 @@
 block discarded – undo
14 14
 	{
15 15
 		foreach( $command->popups as $slug => $label ) {
16 16
 			$buttonClass = glsr( Helper::class )->buildClassName( $slug.'-popup', 'Shortcodes' );
17
-			if( !class_exists( $buttonClass )) {
18
-				glsr_log()->error( sprintf( 'Class missing (%s)', $buttonClass ));
17
+			if( !class_exists( $buttonClass ) ) {
18
+				glsr_log()->error( sprintf( 'Class missing (%s)', $buttonClass ) );
19 19
 				continue;
20 20
 			}
21 21
 			$shortcode = glsr( $buttonClass )->register( $slug, [
22 22
 				'label' => $label,
23 23
 				'title' => $label,
24
-			]);
24
+			] );
25 25
 			glsr()->mceShortcodes[$slug] = $shortcode->properties;
26 26
 		}
27 27
 	}
Please login to merge, or discard this patch.
plugin/Router.php 2 patches
Braces   +15 added lines, -5 removed lines patch added patch discarded remove patch
@@ -17,7 +17,9 @@  discard block
 block discarded – undo
17 17
 	public function routeAdminPostRequest()
18 18
 	{
19 19
 		$request = $this->getRequest();
20
-		if( !$this->isValidPostRequest( $request ))return;
20
+		if( !$this->isValidPostRequest( $request )) {
21
+			return;
22
+		}
21 23
 		check_admin_referer( $request['_action'] );
22 24
 		$this->routeRequest( 'admin', $request['_action'], $request );
23 25
 	}
@@ -39,10 +41,16 @@  discard block
 block discarded – undo
39 41
 	 */
40 42
 	public function routePublicPostRequest()
41 43
 	{
42
-		if( is_admin() )return;
44
+		if( is_admin() ) {
45
+			return;
46
+		}
43 47
 		$request = $this->getRequest();
44
-		if( !$this->isValidPostRequest( $request ))return;
45
-		if( !$this->isValidPublicNonce( $request ))return;
48
+		if( !$this->isValidPostRequest( $request )) {
49
+			return;
50
+		}
51
+		if( !$this->isValidPublicNonce( $request )) {
52
+			return;
53
+		}
46 54
 		$this->routeRequest( 'public', $request['_action'], $request );
47 55
 	}
48 56
 
@@ -51,7 +59,9 @@  discard block
 block discarded – undo
51 59
 	 */
52 60
 	protected function checkAjaxNonce( array $request )
53 61
 	{
54
-		if( !is_user_logged_in() )return;
62
+		if( !is_user_logged_in() ) {
63
+			return;
64
+		}
55 65
 		if( !isset( $request['_nonce'] )) {
56 66
 			$this->sendAjaxError( 'request is missing a nonce', $request );
57 67
 		}
Please login to merge, or discard this patch.
Spacing   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -17,7 +17,7 @@  discard block
 block discarded – undo
17 17
 	public function routeAdminPostRequest()
18 18
 	{
19 19
 		$request = $this->getRequest();
20
-		if( !$this->isValidPostRequest( $request ))return;
20
+		if( !$this->isValidPostRequest( $request ) )return;
21 21
 		check_admin_referer( $request['_action'] );
22 22
 		$this->routeRequest( 'admin', $request['_action'], $request );
23 23
 	}
@@ -41,8 +41,8 @@  discard block
 block discarded – undo
41 41
 	{
42 42
 		if( is_admin() )return;
43 43
 		$request = $this->getRequest();
44
-		if( !$this->isValidPostRequest( $request ))return;
45
-		if( !$this->isValidPublicNonce( $request ))return;
44
+		if( !$this->isValidPostRequest( $request ) )return;
45
+		if( !$this->isValidPublicNonce( $request ) )return;
46 46
 		$this->routeRequest( 'public', $request['_action'], $request );
47 47
 	}
48 48
 
@@ -52,10 +52,10 @@  discard block
 block discarded – undo
52 52
 	protected function checkAjaxNonce( array $request )
53 53
 	{
54 54
 		if( !is_user_logged_in() )return;
55
-		if( !isset( $request['_nonce'] )) {
55
+		if( !isset($request['_nonce']) ) {
56 56
 			$this->sendAjaxError( 'request is missing a nonce', $request );
57 57
 		}
58
-		if( !wp_verify_nonce( $request['_nonce'], $request['_action'] )) {
58
+		if( !wp_verify_nonce( $request['_nonce'], $request['_action'] ) ) {
59 59
 			$this->sendAjaxError( 'request failed the nonce check', $request, 403 );
60 60
 		}
61 61
 	}
@@ -65,10 +65,10 @@  discard block
 block discarded – undo
65 65
 	 */
66 66
 	protected function checkAjaxRequest( array $request )
67 67
 	{
68
-		if( !isset( $request['_action'] )) {
68
+		if( !isset($request['_action']) ) {
69 69
 			$this->sendAjaxError( 'request must include an action', $request );
70 70
 		}
71
-		if( empty( $request['_ajax_request'] )) {
71
+		if( empty($request['_ajax_request']) ) {
72 72
 			$this->sendAjaxError( 'request is invalid', $request );
73 73
 		}
74 74
 	}
@@ -95,7 +95,7 @@  discard block
 block discarded – undo
95 95
 	 */
96 96
 	protected function isValidPostRequest( array $request = [] )
97 97
 	{
98
-		return !empty( $request['_action'] ) && empty( $request['_ajax_request'] );
98
+		return !empty($request['_action']) && empty($request['_ajax_request']);
99 99
 	}
100 100
 
101 101
 	/**
@@ -103,7 +103,7 @@  discard block
 block discarded – undo
103 103
 	 */
104 104
 	protected function isValidPublicNonce( array $request )
105 105
 	{
106
-		if( is_user_logged_in() && !wp_verify_nonce( $request['_nonce'], $request['_action'] )) {
106
+		if( is_user_logged_in() && !wp_verify_nonce( $request['_nonce'], $request['_action'] ) ) {
107 107
 			glsr_log()->error( 'nonce check failed for public request' )->debug( $request );
108 108
 			return false;
109 109
 		}
@@ -118,11 +118,11 @@  discard block
 block discarded – undo
118 118
 	protected function routeRequest( $type, $action, array $request = [] )
119 119
 	{
120 120
 		$actionHook = 'site-reviews/route/'.$type.'/request';
121
-		$controller = glsr( glsr( Helper::class )->buildClassName( $type.'-controller', 'Controllers' ));
121
+		$controller = glsr( glsr( Helper::class )->buildClassName( $type.'-controller', 'Controllers' ) );
122 122
 		$method = glsr( Helper::class )->buildMethodName( $action, 'router' );
123 123
 		$request = apply_filters( 'site-reviews/route/request', $request, $action, $type );
124 124
 		do_action( $actionHook, $action, $request );
125
-		if( is_callable( [$controller, $method] )) {
125
+		if( is_callable( [$controller, $method] ) ) {
126 126
 			call_user_func( [$controller, $method], $request );
127 127
 			return;
128 128
 		}
@@ -140,10 +140,10 @@  discard block
 block discarded – undo
140 140
 	{
141 141
 		glsr_log()->error( $error )->debug( $request );
142 142
 		glsr( Notice::class )->addError( __( 'There was an error (try refreshing the page).', 'site-reviews' ).' <code>'.$error.'</code>' );
143
-		wp_send_json_error([
143
+		wp_send_json_error( [
144 144
 			'message' => __( 'The form could not be submitted. Please notify the site administrator.', 'site-reviews' ),
145 145
 			'notices' => glsr( Notice::class )->get(),
146 146
 			'error' => $error,
147
-		]);
147
+		] );
148 148
 	}
149 149
 }
Please login to merge, or discard this patch.
plugin/Modules/Html/Partials/StarRating.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -12,7 +12,7 @@
 block discarded – undo
12 12
 	 */
13 13
 	public function build( array $args = [] )
14 14
 	{
15
-		require_once( ABSPATH.'wp-admin/includes/template.php' );
15
+		require_once(ABSPATH.'wp-admin/includes/template.php');
16 16
 		ob_start();
17 17
 		wp_star_rating( $args );
18 18
 		$stars = ob_get_clean();
Please login to merge, or discard this patch.
plugin/Modules/Html/Settings.php 1 patch
Spacing   +22 added lines, -22 removed lines patch added patch discarded remove patch
@@ -35,7 +35,7 @@  discard block
 block discarded – undo
35 35
 	 */
36 36
 	protected function getFieldDefault( array $field )
37 37
 	{
38
-		return isset( $field['default'] )
38
+		return isset($field['default'])
39 39
 			? $field['default']
40 40
 			: '';
41 41
 	}
@@ -45,7 +45,7 @@  discard block
 block discarded – undo
45 45
 	 */
46 46
 	protected function getSettingFields( $path )
47 47
 	{
48
-		return array_filter( $this->settings, function( $key ) use( $path ) {
48
+		return array_filter( $this->settings, function( $key ) use($path) {
49 49
 			return glsr( Helper::class )->startsWith( $path, $key );
50 50
 		}, ARRAY_FILTER_USE_KEY );
51 51
 	}
@@ -60,8 +60,8 @@  discard block
 block discarded – undo
60 60
 			$field = wp_parse_args( $field, [
61 61
 				'is_setting' => true,
62 62
 				'name' => $name,
63
-			]);
64
-			$rows.= new Field( $this->normalize( $field ));
63
+			] );
64
+			$rows .= new Field( $this->normalize( $field ) );
65 65
 		}
66 66
 		return $rows;
67 67
 	}
@@ -72,7 +72,7 @@  discard block
 block discarded – undo
72 72
 	 */
73 73
 	protected function getTemplateData( $id )
74 74
 	{
75
-		$fields = $this->getSettingFields( $this->normalizeSettingPath( $id ));
75
+		$fields = $this->getSettingFields( $this->normalizeSettingPath( $id ) );
76 76
 		return [
77 77
 			'context' => [
78 78
 				'rows' => $this->getSettingRows( $fields ),
@@ -86,12 +86,12 @@  discard block
 block discarded – undo
86 86
 	 */
87 87
 	protected function getTemplateDataForAddons( $id )
88 88
 	{
89
-		$fields = $this->getSettingFields( $this->normalizeSettingPath( $id ));
89
+		$fields = $this->getSettingFields( $this->normalizeSettingPath( $id ) );
90 90
 		$settings = glsr( Helper::class )->convertDotNotationArray( $fields );
91 91
 		$settingKeys = array_keys( $settings['settings']['addons'] );
92 92
 		$results = [];
93 93
 		foreach( $settingKeys as $key ) {
94
-			$addonFields = array_filter( $fields, function( $path ) use( $key ) {
94
+			$addonFields = array_filter( $fields, function( $path ) use($key) {
95 95
 				return glsr( Helper::class )->startsWith( 'settings.addons.'.$key, $path );
96 96
 			}, ARRAY_FILTER_USE_KEY );
97 97
 			$results[$key] = $this->getSettingRows( $addonFields );
@@ -108,7 +108,7 @@  discard block
 block discarded – undo
108 108
 	 */
109 109
 	protected function getTemplateDataForLicenses( $id )
110 110
 	{
111
-		$fields = $this->getSettingFields( $this->normalizeSettingPath( $id ));
111
+		$fields = $this->getSettingFields( $this->normalizeSettingPath( $id ) );
112 112
 		ksort( $fields );
113 113
 		return [
114 114
 			'context' => [
@@ -123,7 +123,7 @@  discard block
 block discarded – undo
123 123
 	protected function getTemplateDataForTranslations()
124 124
 	{
125 125
 		$translations = glsr( Translation::class )->renderAll();
126
-		$class = empty( $translations )
126
+		$class = empty($translations)
127 127
 			? 'glsr-hidden'
128 128
 			: '';
129 129
 		return [
@@ -146,9 +146,9 @@  discard block
 block discarded – undo
146 146
 			$path,
147 147
 			glsr( Helper::class )->getPathValue( $path, glsr()->defaults )
148 148
 		);
149
-		if( is_array( $expectedValue )) {
149
+		if( is_array( $expectedValue ) ) {
150 150
 			return is_array( $optionValue )
151
-				? count( array_intersect( $optionValue, $expectedValue )) === 0
151
+				? count( array_intersect( $optionValue, $expectedValue ) ) === 0
152 152
 				: !in_array( $optionValue, $expectedValue );
153 153
 		}
154 154
 		return $optionValue != $expectedValue;
@@ -159,10 +159,10 @@  discard block
 block discarded – undo
159 159
 	 */
160 160
 	protected function isMultiDependency( $path )
161 161
 	{
162
-		if( isset( $this->settings[$path] )) {
162
+		if( isset($this->settings[$path]) ) {
163 163
 			$field = $this->settings[$path];
164
-			return ( $field['type'] == 'checkbox' && !empty( $field['options'] ))
165
-				|| !empty( $field['multiple'] );
164
+			return ($field['type'] == 'checkbox' && !empty($field['options']))
165
+				|| !empty($field['multiple']);
166 166
 		}
167 167
 		return false;
168 168
 	}
@@ -183,17 +183,17 @@  discard block
 block discarded – undo
183 183
 	 */
184 184
 	protected function normalizeDependsOn( array $field )
185 185
 	{
186
-		if( !empty( $field['depends_on'] ) && is_array( $field['depends_on'] )) {
186
+		if( !empty($field['depends_on']) && is_array( $field['depends_on'] ) ) {
187 187
 			$path = key( $field['depends_on'] );
188 188
 			$expectedValue = $field['depends_on'][$path];
189 189
 			$fieldName = glsr( Helper::class )->convertPathToName( $path, OptionManager::databaseKey() );
190
-			if( $this->isMultiDependency( $path )) {
191
-				$fieldName.= '[]';
190
+			if( $this->isMultiDependency( $path ) ) {
191
+				$fieldName .= '[]';
192 192
 			}
193
-			$field['data-depends'] = json_encode([
193
+			$field['data-depends'] = json_encode( [
194 194
 				'name' => $fieldName,
195 195
 				'value' => $expectedValue,
196
-			], JSON_HEX_APOS|JSON_HEX_QUOT );
196
+			], JSON_HEX_APOS | JSON_HEX_QUOT );
197 197
 			$field['is_hidden'] = $this->isFieldHidden( $path, $expectedValue );
198 198
 		}
199 199
 		return $field;
@@ -204,9 +204,9 @@  discard block
 block discarded – undo
204 204
 	 */
205 205
 	protected function normalizeLabelAndLegend( array $field )
206 206
 	{
207
-		if( !empty( $field['label'] )) {
207
+		if( !empty($field['label']) ) {
208 208
 			$field['legend'] = $field['label'];
209
-			unset( $field['label'] );
209
+			unset($field['label']);
210 210
 		}
211 211
 		else {
212 212
 			$field['is_valid'] = false;
@@ -220,7 +220,7 @@  discard block
 block discarded – undo
220 220
 	 */
221 221
 	protected function normalizeValue( array $field )
222 222
 	{
223
-		if( !isset( $field['value'] )) {
223
+		if( !isset($field['value']) ) {
224 224
 			$field['value'] = glsr( OptionManager::class )->get(
225 225
 				$field['name'],
226 226
 				$this->getFieldDefault( $field )
Please login to merge, or discard this patch.
deprecated.php 1 patch
Spacing   +17 added lines, -17 removed lines patch added patch discarded remove patch
@@ -4,7 +4,7 @@  discard block
 block discarded – undo
4 4
 
5 5
 // Database/ReviewManager.php
6 6
 add_action( 'site-reviews/review/created', function( $review ) {
7
-	if( has_action( 'site-reviews/local/review/create' )) {
7
+	if( has_action( 'site-reviews/local/review/create' ) ) {
8 8
 		glsr()->deprecated[] = 'The "site-reviews/local/review/create" hook has been deprecated. Please use the "site-reviews/review/created" hook instead.';
9 9
 		do_action( 'site-reviews/local/review/create', (array)get_post( $review->ID ), (array)$review, $review->ID );
10 10
 	}
@@ -12,18 +12,18 @@  discard block
 block discarded – undo
12 12
 
13 13
 // Handlers/CreateReview.php
14 14
 add_action( 'site-reviews/review/submitted', function( $review ) {
15
-	if( has_action( 'site-reviews/local/review/submitted' )) {
15
+	if( has_action( 'site-reviews/local/review/submitted' ) ) {
16 16
 		glsr()->deprecated[] = 'The "site-reviews/local/review/submitted" hook has been deprecated. Please use the "site-reviews/review/submitted" hook instead.';
17 17
 		do_action( 'site-reviews/local/review/submitted', null, $review );
18 18
 	}
19
-	if( has_filter( 'site-reviews/local/review/submitted/message' )) {
19
+	if( has_filter( 'site-reviews/local/review/submitted/message' ) ) {
20 20
 		glsr()->deprecated[] = 'The "site-reviews/local/review/submitted/message" hook has been deprecated.';
21 21
 	}
22 22
 }, 9 );
23 23
 
24 24
 // Database/ReviewManager.php
25 25
 add_filter( 'site-reviews/create/review-values', function( $values, $command ) {
26
-	if( has_filter( 'site-reviews/local/review' )) {
26
+	if( has_filter( 'site-reviews/local/review' ) ) {
27 27
 		glsr()->deprecated[] = 'The "site-reviews/local/review" hook has been deprecated. Please use the "site-reviews/create/review-values" hook instead.';
28 28
 		return apply_filters( 'site-reviews/local/review', $values, $command );
29 29
 	}
@@ -32,7 +32,7 @@  discard block
 block discarded – undo
32 32
 
33 33
 // Handlers/EnqueuePublicAssets.php
34 34
 add_filter( 'site-reviews/enqueue/public/localize', function( $variables ) {
35
-	if( has_filter( 'site-reviews/enqueue/localize' )) {
35
+	if( has_filter( 'site-reviews/enqueue/localize' ) ) {
36 36
 		glsr()->deprecated[] = 'The "site-reviews/enqueue/localize" hook has been deprecated. Please use the "site-reviews/enqueue/public/localize" hook instead.';
37 37
 		return apply_filters( 'site-reviews/enqueue/localize', $variables );
38 38
 	}
@@ -41,7 +41,7 @@  discard block
 block discarded – undo
41 41
 
42 42
 // Modules/Rating.php
43 43
 add_filter( 'site-reviews/rating/average', function( $average ) {
44
-	if( has_filter( 'site-reviews/average/rating' )) {
44
+	if( has_filter( 'site-reviews/average/rating' ) ) {
45 45
 		glsr()->deprecated[] = 'The "site-reviews/average/rating" hook has been deprecated. Please use the "site-reviews/rating/average" hook instead.';
46 46
 	}
47 47
 	return $average;
@@ -49,7 +49,7 @@  discard block
 block discarded – undo
49 49
 
50 50
 // Modules/Rating.php
51 51
 add_filter( 'site-reviews/rating/ranking', function( $ranking ) {
52
-	if( has_filter( 'site-reviews/bayesian/ranking' )) {
52
+	if( has_filter( 'site-reviews/bayesian/ranking' ) ) {
53 53
 		glsr()->deprecated[] = 'The "site-reviews/bayesian/ranking" hook has been deprecated. Please use the "site-reviews/rating/ranking" hook instead.';
54 54
 	}
55 55
 	return $ranking;
@@ -57,10 +57,10 @@  discard block
 block discarded – undo
57 57
 
58 58
 // Modules/Html/Partials/SiteReviews.php
59 59
 add_filter( 'site-reviews/review/build/after', function( $renderedFields ) {
60
-	if( has_filter( 'site-reviews/reviews/review/text' )) {
60
+	if( has_filter( 'site-reviews/reviews/review/text' ) ) {
61 61
 		glsr()->deprecated[] = 'The "site-reviews/reviews/review/text" hook has been deprecated. Please use the "site-reviews/review/build/after" hook instead.';
62 62
 	}
63
-	if( has_filter( 'site-reviews/reviews/review/title' )) {
63
+	if( has_filter( 'site-reviews/reviews/review/title' ) ) {
64 64
 		glsr()->deprecated[] = 'The "site-reviews/reviews/review/title" hook has been deprecated. Please use the "site-reviews/review/build/after" hook instead.';
65 65
 	}
66 66
 	return $renderedFields;
@@ -68,26 +68,26 @@  discard block
 block discarded – undo
68 68
 
69 69
 // Modules/Html/Partials/SiteReviews.php
70 70
 add_filter( 'site-reviews/review/build/before', function( $review ) {
71
-	if( has_filter( 'site-reviews/rendered/review' )) {
71
+	if( has_filter( 'site-reviews/rendered/review' ) ) {
72 72
 		glsr()->deprecated[] = 'The "site-reviews/rendered/review" hook has been deprecated. Please either use a custom "review.php" template (refer to the documentation), or use the "site-reviews/review/build/after" hook instead.';
73 73
 	}
74
-	if( has_filter( 'site-reviews/rendered/review/meta/order' )) {
74
+	if( has_filter( 'site-reviews/rendered/review/meta/order' ) ) {
75 75
 		glsr()->deprecated[] = 'The "site-reviews/rendered/review/meta/order" hook has been deprecated. Please use a custom "review.php" template instead (refer to the documentation).';
76 76
 	}
77
-	if( has_filter( 'site-reviews/rendered/review/order' )) {
77
+	if( has_filter( 'site-reviews/rendered/review/order' ) ) {
78 78
 		glsr()->deprecated[] = 'The "site-reviews/rendered/review/order" hook has been deprecated. Please use a custom "review.php" template instead (refer to the documentation).';
79 79
 	}
80
-	if( has_filter( 'site-reviews/rendered/review-form/login-register' )) {
80
+	if( has_filter( 'site-reviews/rendered/review-form/login-register' ) ) {
81 81
 		glsr()->deprecated[] = 'The "site-reviews/rendered/review-form/login-register" hook has been deprecated. Please use a custom "login-register.php" template instead (refer to the documentation).';
82 82
 	}
83
-	if( has_filter( 'site-reviews/reviews/navigation_links' )) {
83
+	if( has_filter( 'site-reviews/reviews/navigation_links' ) ) {
84 84
 		glsr()->deprecated[] = 'The "site-reviews/reviews/navigation_links" hook has been deprecated. Please use a custom "pagination.php" template instead (refer to the documentation).';
85 85
 	}
86 86
 	return $review;
87 87
 }, 9 );
88 88
 
89 89
 add_filter( 'site-reviews/validate/custom', function( $result, $request ) {
90
-	if( has_filter( 'site-reviews/validate/review/submission' )) {
90
+	if( has_filter( 'site-reviews/validate/review/submission' ) ) {
91 91
 		glsr_log()->notice( 'The "site-reviews/validate/review/submission" hook has been deprecated. Please use the "site-reviews/validate/custom" hook instead.' );
92 92
 		return apply_filters( 'site-reviews/validate/review/submission', $result, $request );
93 93
 	}
@@ -95,7 +95,7 @@  discard block
 block discarded – undo
95 95
 }, 9, 2 );
96 96
 
97 97
 add_filter( 'site-reviews/views/file', function( $file, $view, $data ) {
98
-	if( has_filter( 'site-reviews/addon/views/file' )) {
98
+	if( has_filter( 'site-reviews/addon/views/file' ) ) {
99 99
 		glsr()->deprecated[] = 'The "site-reviews/addon/views/file" hook has been deprecated. Please use the "site-reviews/views/file" hook instead.';
100 100
 		$file = apply_filters( 'site-reviews/addon/views/file', $file, $view, $data );
101 101
 	}
@@ -103,7 +103,7 @@  discard block
 block discarded – undo
103 103
 }, 9, 3 );
104 104
 
105 105
 add_action( 'wp_footer', function() {
106
-	$notices = array_keys( array_flip( glsr()->deprecated ));
106
+	$notices = array_keys( array_flip( glsr()->deprecated ) );
107 107
 	natsort( $notices );
108 108
 	foreach( $notices as $notice ) {
109 109
 		glsr_log()->notice( $notice );
Please login to merge, or discard this patch.
views/partials/editor/review.php 2 patches
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -1,14 +1,14 @@  discard block
 block discarded – undo
1 1
 <?php defined( 'WPINC' ) || die; ?>
2 2
 
3 3
 <div id="titlediv">
4
-	<input type="text" id="title" value="<?= $post->post_title ? esc_attr( $post->post_title ) : sprintf( '(%s)', __( 'no title', 'site-reviews' )); ?>" readonly>
4
+	<input type="text" id="title" value="<?= $post->post_title ? esc_attr( $post->post_title ) : sprintf( '(%s)', __( 'no title', 'site-reviews' ) ); ?>" readonly>
5 5
 </div>
6 6
 
7 7
 <div id="contentdiv">
8 8
 	<textarea readonly><?= esc_attr( $post->post_content ); ?></textarea>
9 9
 </div>
10 10
 
11
-<?php if( empty( $response ))return; ?>
11
+<?php if( empty($response) )return; ?>
12 12
 
13 13
 <div class="postbox glsr-response-postbox">
14 14
 	<button type="button" class="handlediv" aria-expanded="true">
@@ -19,6 +19,6 @@  discard block
 block discarded – undo
19 19
 		<span><?= __( 'Public Response', 'site-reviews' ); ?></span>
20 20
 	</h2>
21 21
 	<div class="inside">
22
-		<?= wpautop( esc_attr( $response )); ?>
22
+		<?= wpautop( esc_attr( $response ) ); ?>
23 23
 	</div>
24 24
 </div>
Please login to merge, or discard this patch.
Braces   +4 added lines, -1 removed lines patch added patch discarded remove patch
@@ -8,7 +8,10 @@
 block discarded – undo
8 8
 	<textarea readonly><?= esc_attr( $post->post_content ); ?></textarea>
9 9
 </div>
10 10
 
11
-<?php if( empty( $response ))return; ?>
11
+<?php if( empty( $response )) {
12
+	return;
13
+}
14
+?>
12 15
 
13 16
 <div class="postbox glsr-response-postbox">
14 17
 	<button type="button" class="handlediv" aria-expanded="true">
Please login to merge, or discard this patch.
plugin/Modules/Polylang.php 2 patches
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -13,14 +13,14 @@  discard block
 block discarded – undo
13 13
 	public function getPost( $postId )
14 14
 	{
15 15
 		$postId = trim( $postId );
16
-		if( empty( $postId ) || !is_numeric( $postId ))return;
16
+		if( empty($postId) || !is_numeric( $postId ) )return;
17 17
 		if( $this->isEnabled() ) {
18
-			$polylangPostId = pll_get_post( $postId, pll_get_post_language( get_the_ID() ));
18
+			$polylangPostId = pll_get_post( $postId, pll_get_post_language( get_the_ID() ) );
19 19
 		}
20
-		if( !empty( $polylangPostId )) {
20
+		if( !empty($polylangPostId) ) {
21 21
 			$postId = $polylangPostId;
22 22
 		}
23
-		return get_post( intval( $postId ));
23
+		return get_post( intval( $postId ) );
24 24
 	}
25 25
 
26 26
 	/**
@@ -35,7 +35,7 @@  discard block
 block discarded – undo
35 35
 		foreach( $this->cleanIds( $postIds ) as $postId ) {
36 36
 			$newPostIds = array_merge(
37 37
 				$newPostIds,
38
-				array_values( pll_get_post_translations( $postId ))
38
+				array_values( pll_get_post_translations( $postId ) )
39 39
 			);
40 40
 		}
41 41
 		return $this->cleanIds( $newPostIds );
@@ -75,7 +75,7 @@  discard block
 block discarded – undo
75 75
 	 */
76 76
 	protected function cleanIds( array $postIds )
77 77
 	{
78
-		return array_filter( array_unique( $postIds ));
78
+		return array_filter( array_unique( $postIds ) );
79 79
 	}
80 80
 }
81 81
 
Please login to merge, or discard this patch.
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -13,7 +13,9 @@
 block discarded – undo
13 13
 	public function getPost( $postId )
14 14
 	{
15 15
 		$postId = trim( $postId );
16
-		if( empty( $postId ) || !is_numeric( $postId ))return;
16
+		if( empty( $postId ) || !is_numeric( $postId )) {
17
+			return;
18
+		}
17 19
 		if( $this->isEnabled() ) {
18 20
 			$polylangPostId = pll_get_post( $postId, pll_get_post_language( get_the_ID() ));
19 21
 		}
Please login to merge, or discard this patch.
plugin/Controllers/Controller.php 2 patches
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -15,7 +15,7 @@  discard block
 block discarded – undo
15 15
 	 */
16 16
 	public function download( $filename, $content )
17 17
 	{
18
-		if( !current_user_can( glsr()->constant( 'CAPABILITY' )))return;
18
+		if( !current_user_can( glsr()->constant( 'CAPABILITY' ) ) )return;
19 19
 		nocache_headers();
20 20
 		header( 'Content-Type: text/plain' );
21 21
 		header( 'Content-Disposition: attachment; filename="'.$filename.'"' );
@@ -30,8 +30,8 @@  discard block
 block discarded – undo
30 30
 	 */
31 31
 	public function execute( $command )
32 32
 	{
33
-		$handlerClass = str_replace( 'Commands', 'Handlers', get_class( $command ));
34
-		if( !class_exists( $handlerClass )) {
33
+		$handlerClass = str_replace( 'Commands', 'Handlers', get_class( $command ) );
34
+		if( !class_exists( $handlerClass ) ) {
35 35
 			throw new InvalidArgumentException( 'Handler '.$handlerClass.' not found.' );
36 36
 		}
37 37
 		try {
@@ -39,7 +39,7 @@  discard block
 block discarded – undo
39 39
 		}
40 40
 		catch( Exception $e ) {
41 41
 			status_header( 400 );
42
-			glsr( Notice::class )->addError( new WP_Error( 'site_reviews_error', $e->getMessage() ));
42
+			glsr( Notice::class )->addError( new WP_Error( 'site_reviews_error', $e->getMessage() ) );
43 43
 			glsr_log()->error( $e->getMessage() );
44 44
 		}
45 45
 	}
@@ -49,7 +49,7 @@  discard block
 block discarded – undo
49 49
 	 */
50 50
 	protected function getPostId()
51 51
 	{
52
-		return intval( filter_input( INPUT_GET, 'post' ));
52
+		return intval( filter_input( INPUT_GET, 'post' ) );
53 53
 	}
54 54
 
55 55
 	/**
Please login to merge, or discard this patch.
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -15,7 +15,9 @@
 block discarded – undo
15 15
 	 */
16 16
 	public function download( $filename, $content )
17 17
 	{
18
-		if( !current_user_can( glsr()->constant( 'CAPABILITY' )))return;
18
+		if( !current_user_can( glsr()->constant( 'CAPABILITY' ))) {
19
+			return;
20
+		}
19 21
 		nocache_headers();
20 22
 		header( 'Content-Type: text/plain' );
21 23
 		header( 'Content-Disposition: attachment; filename="'.$filename.'"' );
Please login to merge, or discard this patch.