Passed
Push — master ( c272a3...f1c4a0 )
by Paul
07:09
created
plugin/Modules/Html/Builder.php 2 patches
Braces   +9 added lines, -3 removed lines patch added patch discarded remove patch
@@ -80,7 +80,9 @@  discard block
 block discarded – undo
80 80
 		];
81 81
 		if( !isset( $properties[$property] )
82 82
 			|| empty( array_filter( [$value], $properties[$property] ))
83
-		)return;
83
+		) {
84
+			return;
85
+		}
84 86
 		$this->$property = $value;
85 87
 	}
86 88
 
@@ -129,7 +131,9 @@  discard block
 block discarded – undo
129 131
 	 */
130 132
 	protected function buildFieldDescription()
131 133
 	{
132
-		if( empty( $this->args['description'] ))return;
134
+		if( empty( $this->args['description'] )) {
135
+			return;
136
+		}
133 137
 		if( $this->args['is_widget'] ) {
134 138
 			return $this->small( $this->args['description'] );
135 139
 		}
@@ -184,7 +188,9 @@  discard block
 block discarded – undo
184 188
 	 */
185 189
 	protected function buildFormLabel()
186 190
 	{
187
-		if( empty( $this->args['label'] ) || $this->args['type'] == 'hidden' )return;
191
+		if( empty( $this->args['label'] ) || $this->args['type'] == 'hidden' ) {
192
+			return;
193
+		}
188 194
 		return $this->label([
189 195
 			'for' => $this->args['id'],
190 196
 			'text' => $this->args['label'],
Please login to merge, or discard this patch.
Spacing   +26 added lines, -26 removed lines patch added patch discarded remove patch
@@ -55,7 +55,7 @@  discard block
 block discarded – undo
55 55
 	{
56 56
 		$instance = new static;
57 57
 		$instance->setTagFromMethod( $method );
58
-		call_user_func_array( [$instance, 'normalize'], $args += ['',''] );
58
+		call_user_func_array( [$instance, 'normalize'], $args += ['', ''] );
59 59
 		$tags = array_merge( static::TAGS_FORM, static::TAGS_SINGLE, static::TAGS_STRUCTURE, static::TAGS_TEXT );
60 60
 		$generatedTag = in_array( $instance->tag, $tags )
61 61
 			? $instance->buildTag()
@@ -78,8 +78,8 @@  discard block
 block discarded – undo
78 78
 			'render' => 'is_bool',
79 79
 			'tag' => 'is_string',
80 80
 		];
81
-		if( !isset( $properties[$property] )
82
-			|| empty( array_filter( [$value], $properties[$property] ))
81
+		if( !isset($properties[$property])
82
+			|| empty(array_filter( [$value], $properties[$property] ))
83 83
 		)return;
84 84
 		$this->$property = $value;
85 85
 	}
@@ -97,7 +97,7 @@  discard block
 block discarded – undo
97 97
 	 */
98 98
 	public function getOpeningTag()
99 99
 	{
100
-		$attributes = glsr( Attributes::class )->{$this->tag}( $this->args )->toString();
100
+		$attributes = glsr( Attributes::class )->{$this->tag}($this->args)->toString();
101 101
 		return '<'.trim( $this->tag.' '.$attributes ).'>';
102 102
 	}
103 103
 
@@ -107,7 +107,7 @@  discard block
 block discarded – undo
107 107
 	protected function buildCustomField()
108 108
 	{
109 109
 		$className = $this->getCustomFieldClassName();
110
-		if( class_exists( $className )) {
110
+		if( class_exists( $className ) ) {
111 111
 			return (new $className( $this ))->build();
112 112
 		}
113 113
 		glsr_log()->error( 'Field missing: '.$className );
@@ -118,7 +118,7 @@  discard block
 block discarded – undo
118 118
 	 */
119 119
 	protected function buildDefaultTag( $text = '' )
120 120
 	{
121
-		if( empty( $text )) {
121
+		if( empty($text) ) {
122 122
 			$text = $this->args['text'];
123 123
 		}
124 124
 		return $this->getOpeningTag().$text.$this->getClosingTag();
@@ -129,7 +129,7 @@  discard block
 block discarded – undo
129 129
 	 */
130 130
 	protected function buildFieldDescription()
131 131
 	{
132
-		if( empty( $this->args['description'] ))return;
132
+		if( empty($this->args['description']) )return;
133 133
 		if( $this->args['is_widget'] ) {
134 134
 			return $this->small( $this->args['description'] );
135 135
 		}
@@ -141,10 +141,10 @@  discard block
 block discarded – undo
141 141
 	 */
142 142
 	protected function buildFormInput()
143 143
 	{
144
-		if( !in_array( $this->args['type'], ['checkbox', 'radio'] )) {
144
+		if( !in_array( $this->args['type'], ['checkbox', 'radio'] ) ) {
145 145
 			return $this->buildFormLabel().$this->getOpeningTag();
146 146
 		}
147
-		return empty( $this->args['options'] )
147
+		return empty($this->args['options'])
148 148
 			? $this->buildFormInputChoice()
149 149
 			: $this->buildFormInputMultiChoice();
150 150
 	}
@@ -154,13 +154,13 @@  discard block
 block discarded – undo
154 154
 	 */
155 155
 	protected function buildFormInputChoice()
156 156
 	{
157
-		$labelText = !empty( $this->args['text'] )
157
+		$labelText = !empty($this->args['text'])
158 158
 			? $this->args['text']
159 159
 			: $this->args['label'];
160 160
 		return $this->label( $this->getOpeningTag().' '.$labelText, [
161 161
 			'class' => 'glsr-'.$this->args['type'].'-label',
162 162
 			'for' => $this->args['id'],
163
-		]);
163
+		] );
164 164
 	}
165 165
 
166 166
 	/**
@@ -169,7 +169,7 @@  discard block
 block discarded – undo
169 169
 	protected function buildFormInputMultiChoice()
170 170
 	{
171 171
 		if( $this->args['type'] == 'checkbox' ) {
172
-			$this->args['name'].= '[]';
172
+			$this->args['name'] .= '[]';
173 173
 		}
174 174
 		$options = array_reduce( array_keys( $this->args['options'] ), function( $carry, $key ) {
175 175
 			return $carry.$this->li( $this->{$this->args['type']}([
@@ -177,12 +177,12 @@  discard block
 block discarded – undo
177 177
 				'name' => $this->args['name'],
178 178
 				'text' => $this->args['options'][$key],
179 179
 				'value' => $key,
180
-			]));
180
+			]) );
181 181
 		});
182 182
 		return $this->ul( $options, [
183 183
 			'class' => $this->args['class'],
184 184
 			'id' => $this->args['id'],
185
-		]);
185
+		] );
186 186
 	}
187 187
 
188 188
 	/**
@@ -190,11 +190,11 @@  discard block
 block discarded – undo
190 190
 	 */
191 191
 	protected function buildFormLabel()
192 192
 	{
193
-		if( empty( $this->args['label'] ) || $this->args['type'] == 'hidden' )return;
194
-		return $this->label([
193
+		if( empty($this->args['label']) || $this->args['type'] == 'hidden' )return;
194
+		return $this->label( [
195 195
 			'for' => $this->args['id'],
196 196
 			'text' => $this->args['label'],
197
-		]);
197
+		] );
198 198
 	}
199 199
 
200 200
 	/**
@@ -211,11 +211,11 @@  discard block
 block discarded – undo
211 211
 	protected function buildFormSelectOptions()
212 212
 	{
213 213
 		return array_reduce( array_keys( $this->args['options'] ), function( $carry, $key ) {
214
-			return $carry.$this->option([
214
+			return $carry.$this->option( [
215 215
 				'selected' => $this->args['value'] == $key,
216 216
 				'text' => $this->args['options'][$key],
217 217
 				'value' => $key,
218
-			]);
218
+			] );
219 219
 		});
220 220
 	}
221 221
 
@@ -233,10 +233,10 @@  discard block
 block discarded – undo
233 233
 	protected function buildTag()
234 234
 	{
235 235
 		$this->mergeArgsWithRequiredDefaults();
236
-		if( in_array( $this->tag, static::TAGS_SINGLE )) {
236
+		if( in_array( $this->tag, static::TAGS_SINGLE ) ) {
237 237
 			return $this->getOpeningTag();
238 238
 		}
239
-		if( !in_array( $this->tag, static::TAGS_FORM )) {
239
+		if( !in_array( $this->tag, static::TAGS_FORM ) ) {
240 240
 			return $this->buildDefaultTag();
241 241
 		}
242 242
 		return call_user_func( [$this, 'buildForm'.ucfirst( $this->tag )] ).$this->buildFieldDescription();
@@ -256,7 +256,7 @@  discard block
 block discarded – undo
256 256
 	protected function mergeArgsWithRequiredDefaults()
257 257
 	{
258 258
 		$className = $this->getCustomFieldClassName();
259
-		if( class_exists( $className )) {
259
+		if( class_exists( $className ) ) {
260 260
 			$this->args = array_merge(
261 261
 				wp_parse_args( $this->args, $className::defaults() ),
262 262
 				$className::required()
@@ -271,13 +271,13 @@  discard block
 block discarded – undo
271 271
 	 */
272 272
 	protected function normalize( ...$params )
273 273
 	{
274
-		if( is_string( $params[0] ) || is_numeric( $params[0] )) {
274
+		if( is_string( $params[0] ) || is_numeric( $params[0] ) ) {
275 275
 			$this->setNameOrTextAttributeForTag( $params[0] );
276 276
 		}
277
-		if( is_array( $params[0] )) {
277
+		if( is_array( $params[0] ) ) {
278 278
 			$this->args += $params[0];
279 279
 		}
280
-		else if( is_array( $params[1] )) {
280
+		else if( is_array( $params[1] ) ) {
281 281
 			$this->args += $params[1];
282 282
 		}
283 283
 	}
@@ -301,7 +301,7 @@  discard block
 block discarded – undo
301 301
 	protected function setTagFromMethod( $method )
302 302
 	{
303 303
 		$this->tag = strtolower( $method );
304
-		if( in_array( $this->tag, static::INPUT_TYPES )) {
304
+		if( in_array( $this->tag, static::INPUT_TYPES ) ) {
305 305
 			$this->args['type'] = $this->tag;
306 306
 			$this->tag = 'input';
307 307
 		}
Please login to merge, or discard this patch.
plugin/Handlers/EnqueueAdminAssets.php 2 patches
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -44,7 +44,7 @@  discard block
 block discarded – undo
44 44
 			glsr()->version,
45 45
 			true
46 46
 		);
47
-		if( !empty( $this->pointers )) {
47
+		if( !empty($this->pointers) ) {
48 48
 			wp_enqueue_style( 'wp-pointer' );
49 49
 			wp_enqueue_script( 'wp-pointer' );
50 50
 		}
@@ -84,7 +84,7 @@  discard block
 block discarded – undo
84 84
 		$dependencies = apply_filters( 'site-reviews/enqueue/admin/dependencies', [] );
85 85
 		$dependencies = array_merge( $dependencies, [
86 86
 			'jquery', 'jquery-ui-sortable', 'underscore', 'wp-util',
87
-		]);
87
+		] );
88 88
 		return $dependencies;
89 89
 	}
90 90
 
@@ -114,7 +114,7 @@  discard block
 block discarded – undo
114 114
 		$generatedPointers = [];
115 115
 		foreach( $pointers as $pointer ) {
116 116
 			if( $pointer['screen'] != glsr_current_screen()->id )continue;
117
-			if( in_array( $pointer['id'], $dismissedPointers ))continue;
117
+			if( in_array( $pointer['id'], $dismissedPointers ) )continue;
118 118
 			$generatedPointers[] = $this->generatePointer( $pointer );
119 119
 		}
120 120
 		$this->pointers = $generatedPointers;
@@ -126,7 +126,7 @@  discard block
 block discarded – undo
126 126
 	protected function isCurrentScreen()
127 127
 	{
128 128
 		$screen = glsr_current_screen();
129
-		return $screen && ( $screen->post_type == Application::POST_TYPE
129
+		return $screen && ($screen->post_type == Application::POST_TYPE
130 130
 			|| $screen->base == 'post'
131 131
 			|| $screen->id == 'dashboard'
132 132
 			|| $screen->id == 'widgets'
@@ -140,7 +140,7 @@  discard block
 block discarded – undo
140 140
 	{
141 141
 		$variables = [];
142 142
 		foreach( glsr()->mceShortcodes as $tag => $args ) {
143
-			if( empty( $args['required'] ))continue;
143
+			if( empty($args['required']) )continue;
144 144
 			$variables[$tag] = $args['required'];
145 145
 		}
146 146
 		return $variables;
Please login to merge, or discard this patch.
Braces   +12 added lines, -4 removed lines patch added patch discarded remove patch
@@ -36,7 +36,9 @@  discard block
 block discarded – undo
36 36
 			[],
37 37
 			glsr()->version
38 38
 		);
39
-		if( !$this->isCurrentScreen() )return;
39
+		if( !$this->isCurrentScreen() ) {
40
+			return;
41
+		}
40 42
 		wp_enqueue_script(
41 43
 			Application::ID,
42 44
 			glsr()->url( 'assets/scripts/'.Application::ID.'-admin.js' ),
@@ -113,8 +115,12 @@  discard block
 block discarded – undo
113 115
 		$dismissedPointers = explode( ',', (string)$dismissedPointers );
114 116
 		$generatedPointers = [];
115 117
 		foreach( $pointers as $pointer ) {
116
-			if( $pointer['screen'] != glsr_current_screen()->id )continue;
117
-			if( in_array( $pointer['id'], $dismissedPointers ))continue;
118
+			if( $pointer['screen'] != glsr_current_screen()->id ) {
119
+				continue;
120
+			}
121
+			if( in_array( $pointer['id'], $dismissedPointers )) {
122
+				continue;
123
+			}
118 124
 			$generatedPointers[] = $this->generatePointer( $pointer );
119 125
 		}
120 126
 		$this->pointers = $generatedPointers;
@@ -140,7 +146,9 @@  discard block
 block discarded – undo
140 146
 	{
141 147
 		$variables = [];
142 148
 		foreach( glsr()->mceShortcodes as $tag => $args ) {
143
-			if( empty( $args['required'] ))continue;
149
+			if( empty( $args['required'] )) {
150
+				continue;
151
+			}
144 152
 			$variables[$tag] = $args['required'];
145 153
 		}
146 154
 		return $variables;
Please login to merge, or discard this patch.
plugin/Controllers/AjaxController.php 1 patch
Spacing   +16 added lines, -16 removed lines patch added patch discarded remove patch
@@ -24,7 +24,7 @@  discard block
 block discarded – undo
24 24
 	{
25 25
 		wp_send_json( glsr( Html::class )->renderPartial( 'link', [
26 26
 			'post_id' => $request['ID'],
27
-		]));
27
+		] ) );
28 28
 	}
29 29
 
30 30
 	/**
@@ -32,7 +32,7 @@  discard block
 block discarded – undo
32 32
 	 */
33 33
 	public function routerChangeReviewStatus( array $request )
34 34
 	{
35
-		wp_send_json( $this->execute( new ChangeStatus( $request )));
35
+		wp_send_json( $this->execute( new ChangeStatus( $request ) ) );
36 36
 	}
37 37
 
38 38
 	/**
@@ -41,10 +41,10 @@  discard block
 block discarded – undo
41 41
 	public function routerClearLog()
42 42
 	{
43 43
 		glsr( AdminController::class )->routerClearLog();
44
-		wp_send_json([
44
+		wp_send_json( [
45 45
 			'logger' => glsr( Logger::class )->get(),
46 46
 			'notices' => glsr( Notice::class )->get(),
47
-		]);
47
+		] );
48 48
 	}
49 49
 
50 50
 	/**
@@ -54,9 +54,9 @@  discard block
 block discarded – undo
54 54
 	{
55 55
 		$shortcode = $request['shortcode'];
56 56
 		$response = false;
57
-		if( array_key_exists( $shortcode, glsr()->mceShortcodes )) {
57
+		if( array_key_exists( $shortcode, glsr()->mceShortcodes ) ) {
58 58
 			$data = glsr()->mceShortcodes[$shortcode];
59
-			if( !empty( $data['errors'] )) {
59
+			if( !empty($data['errors']) ) {
60 60
 				$data['btn_okay'] = [esc_html__( 'Okay', 'site-reviews' )];
61 61
 			}
62 62
 			$response = [
@@ -76,10 +76,10 @@  discard block
 block discarded – undo
76 76
 	public function routerSearchPosts( array $request )
77 77
 	{
78 78
 		$results = glsr( Database::class )->searchPosts( $request['search'] );
79
-		wp_send_json_success([
79
+		wp_send_json_success( [
80 80
 			'empty' => '<div>'.__( 'Nothing found.', 'site-reviews' ).'</div>',
81 81
 			'items' => $results,
82
-		]);
82
+		] );
83 83
 	}
84 84
 
85 85
 	/**
@@ -87,7 +87,7 @@  discard block
 block discarded – undo
87 87
 	 */
88 88
 	public function routerSearchTranslations( array $request )
89 89
 	{
90
-		if( empty( $request['exclude'] )) {
90
+		if( empty($request['exclude']) ) {
91 91
 			$request['exclude'] = [];
92 92
 		}
93 93
 		$results = glsr( Translator::class )
@@ -95,10 +95,10 @@  discard block
 block discarded – undo
95 95
 			->exclude()
96 96
 			->exclude( $request['exclude'] )
97 97
 			->renderResults();
98
-		wp_send_json_success([
98
+		wp_send_json_success( [
99 99
 			'empty' => '<div>'.__( 'Nothing found.', 'site-reviews' ).'</div>',
100 100
 			'items' => $results,
101
-		]);
101
+		] );
102 102
 	}
103 103
 
104 104
 	/**
@@ -108,11 +108,11 @@  discard block
 block discarded – undo
108 108
 	{
109 109
 		$response = glsr( PublicController::class )->routerCreateReview( $request );
110 110
 		$session = glsr( Session::class );
111
-		wp_send_json([
111
+		wp_send_json( [
112 112
 			'errors' => $session->get( $request['form_id'].'-errors', false, true ),
113 113
 			'message' => $response,
114 114
 			'recaptcha' => $session->get( $request['form_id'].'-recaptcha', false, true ),
115
-		]);
115
+		] );
116 116
 	}
117 117
 
118 118
 	/**
@@ -120,10 +120,10 @@  discard block
 block discarded – undo
120 120
 	 */
121 121
 	public function routerTogglePinned( array $request )
122 122
 	{
123
-		$isPinned = $this->execute( new TogglePinned( $request ));
124
-		wp_send_json([
123
+		$isPinned = $this->execute( new TogglePinned( $request ) );
124
+		wp_send_json( [
125 125
 			'notices' => glsr( Notice::class )->get(),
126 126
 			'pinned' => $isPinned,
127
-		]);
127
+		] );
128 128
 	}
129 129
 }
Please login to merge, or discard this patch.
plugin/Modules/System.php 2 patches
Spacing   +20 added lines, -20 removed lines patch added patch discarded remove patch
@@ -37,7 +37,7 @@  discard block
 block discarded – undo
37 37
 			'inactive-plugin' => 'Inactive Plugins',
38 38
 			'setting' => 'Plugin Settings',
39 39
 		];
40
-		$systemInfo = array_reduce( array_keys( $details ), function( $carry, $key ) use( $details ) {
40
+		$systemInfo = array_reduce( array_keys( $details ), function( $carry, $key ) use($details) {
41 41
 			$methodName = glsr( Helper::class )->buildMethodName( 'get-'.$key.'-details' );
42 42
 			if( method_exists( $this, $methodName ) && $systemDetails = $this->$methodName() ) {
43 43
 				return $carry.$this->implode( $details[$key], $systemDetails );
@@ -54,8 +54,8 @@  discard block
 block discarded – undo
54 54
 	{
55 55
 		$plugins = get_plugins();
56 56
 		$activePlugins = (array)get_option( 'active_plugins', [] );
57
-		$inactive = array_diff_key( $plugins, array_flip( $activePlugins ));
58
-		return $this->normalizePluginList( array_diff_key( $plugins, $inactive ));
57
+		$inactive = array_diff_key( $plugins, array_flip( $activePlugins ) );
58
+		return $this->normalizePluginList( array_diff_key( $plugins, $inactive ) );
59 59
 	}
60 60
 
61 61
 	/**
@@ -79,7 +79,7 @@  discard block
 block discarded – undo
79 79
 	public function getInactivePluginDetails()
80 80
 	{
81 81
 		$activePlugins = (array)get_option( 'active_plugins', [] );
82
-		return $this->normalizePluginList( array_diff_key( get_plugins(), array_flip( $activePlugins )));
82
+		return $this->normalizePluginList( array_diff_key( get_plugins(), array_flip( $activePlugins ) ) );
83 83
 	}
84 84
 
85 85
 	/**
@@ -89,9 +89,9 @@  discard block
 block discarded – undo
89 89
 	{
90 90
 		$plugins = array_merge(
91 91
 			get_mu_plugins(),
92
-			get_plugins( '/../'.basename( WPMU_PLUGIN_DIR ))
92
+			get_plugins( '/../'.basename( WPMU_PLUGIN_DIR ) )
93 93
 		);
94
-		if( empty( $plugins ))return;
94
+		if( empty($plugins) )return;
95 95
 		return $this->normalizePluginList( $plugins );
96 96
 	}
97 97
 
@@ -100,7 +100,7 @@  discard block
 block discarded – undo
100 100
 	 */
101 101
 	public function getMultisitePluginDetails()
102 102
 	{
103
-		if( !is_multisite() || empty( get_site_option( 'active_sitewide_plugins', [] )))return;
103
+		if( !is_multisite() || empty(get_site_option( 'active_sitewide_plugins', [] )) )return;
104 104
 		return $this->normalizePluginList( wp_get_active_network_plugins() );
105 105
 	}
106 106
 
@@ -122,11 +122,11 @@  discard block
 block discarded – undo
122 122
 			'Max Input Vars' => ini_get( 'max_input_vars' ),
123 123
 			'Memory Limit' => ini_get( 'memory_limit' ),
124 124
 			'Post Max Size' => ini_get( 'post_max_size' ),
125
-			'Session Cookie Path' => esc_html( ini_get( 'session.cookie_path' )),
126
-			'Session Name' => esc_html( ini_get( 'session.name' )),
127
-			'Session Save Path' => esc_html( ini_get( 'session.save_path' )),
128
-			'Session Use Cookies' => var_export( wp_validate_boolean( ini_get( 'session.use_cookies' )), true ),
129
-			'Session Use Only Cookies' => var_export( wp_validate_boolean( ini_get( 'session.use_only_cookies' )), true ),
125
+			'Session Cookie Path' => esc_html( ini_get( 'session.cookie_path' ) ),
126
+			'Session Name' => esc_html( ini_get( 'session.name' ) ),
127
+			'Session Save Path' => esc_html( ini_get( 'session.save_path' ) ),
128
+			'Session Use Cookies' => var_export( wp_validate_boolean( ini_get( 'session.use_cookies' ) ), true ),
129
+			'Session Use Only Cookies' => var_export( wp_validate_boolean( ini_get( 'session.use_only_cookies' ) ), true ),
130 130
 			'Upload Max Filesize' => ini_get( 'upload_max_filesize' ),
131 131
 		];
132 132
 	}
@@ -154,13 +154,13 @@  discard block
 block discarded – undo
154 154
 		$settings = glsr( OptionManager::class )->get( 'settings', [] );
155 155
 		$settings = $helper->flattenArray( $settings, true );
156 156
 		foreach( ['submissions.recaptcha.key', 'submissions.recaptcha.secret'] as $key ) {
157
-			if( empty( $settings[$key] ))continue;
157
+			if( empty($settings[$key]) )continue;
158 158
 			$settings[$key] = str_repeat( '*', 10 );
159 159
 		}
160 160
 		$details = [];
161 161
 		foreach( $settings as $key => $value ) {
162
-			if( $helper->startsWith( 'strings', $key ) && $helper->endsWith( 'id', $key ))continue;
163
-			$value = htmlspecialchars( trim( preg_replace('/\s\s+/', '\\n', $value )), ENT_QUOTES, 'UTF-8' );
162
+			if( $helper->startsWith( 'strings', $key ) && $helper->endsWith( 'id', $key ) )continue;
163
+			$value = htmlspecialchars( trim( preg_replace( '/\s\s+/', '\\n', $value ) ), ENT_QUOTES, 'UTF-8' );
164 164
 			$details[$key] = $value;
165 165
 		}
166 166
 		return $details;
@@ -231,10 +231,10 @@  discard block
 block discarded – undo
231 231
 			'WPE_APIKEY' => 'WP Engine',
232 232
 		];
233 233
 		foreach( $checks as $key => $value ) {
234
-			if( !$this->isWebhostCheckValid( $key ))continue;
234
+			if( !$this->isWebhostCheckValid( $key ) )continue;
235 235
 			return $value;
236 236
 		}
237
-		return implode( ',', array_filter( [DB_HOST, filter_input( INPUT_SERVER, 'SERVER_NAME' )] ));
237
+		return implode( ',', array_filter( [DB_HOST, filter_input( INPUT_SERVER, 'SERVER_NAME' )] ) );
238 238
 	}
239 239
 
240 240
 	/**
@@ -255,8 +255,8 @@  discard block
 block discarded – undo
255 255
 	{
256 256
 		$plugins = get_plugins();
257 257
 		$activePlugins = (array)get_option( 'active_plugins', [] );
258
-		$inactive = $this->normalizePluginList( array_diff_key( $plugins, array_flip( $activePlugins )));
259
-		$active = $this->normalizePluginList( array_diff_key( $plugins, $inactive ));
258
+		$inactive = $this->normalizePluginList( array_diff_key( $plugins, array_flip( $activePlugins ) ) );
259
+		$active = $this->normalizePluginList( array_diff_key( $plugins, $inactive ) );
260 260
 		return $active + $inactive;
261 261
 	}
262 262
 
@@ -267,7 +267,7 @@  discard block
 block discarded – undo
267 267
 	protected function implode( $title, array $details )
268 268
 	{
269 269
 		$strings = ['['.$title.']'];
270
-		$padding = max( array_map( 'strlen', array_keys( $details )) );
270
+		$padding = max( array_map( 'strlen', array_keys( $details ) ) );
271 271
 		$padding = max( [$padding, static::PAD] );
272 272
 		foreach( $details as $key => $value ) {
273 273
 			$strings[] = is_string( $key )
Please login to merge, or discard this patch.
Braces   +15 added lines, -5 removed lines patch added patch discarded remove patch
@@ -91,7 +91,9 @@  discard block
 block discarded – undo
91 91
 			get_mu_plugins(),
92 92
 			get_plugins( '/../'.basename( WPMU_PLUGIN_DIR ))
93 93
 		);
94
-		if( empty( $plugins ))return;
94
+		if( empty( $plugins )) {
95
+			return;
96
+		}
95 97
 		return $this->normalizePluginList( $plugins );
96 98
 	}
97 99
 
@@ -100,7 +102,9 @@  discard block
 block discarded – undo
100 102
 	 */
101 103
 	public function getMultisitePluginDetails()
102 104
 	{
103
-		if( !is_multisite() || empty( get_site_option( 'active_sitewide_plugins', [] )))return;
105
+		if( !is_multisite() || empty( get_site_option( 'active_sitewide_plugins', [] ))) {
106
+			return;
107
+		}
104 108
 		return $this->normalizePluginList( wp_get_active_network_plugins() );
105 109
 	}
106 110
 
@@ -154,12 +158,16 @@  discard block
 block discarded – undo
154 158
 		$settings = glsr( OptionManager::class )->get( 'settings', [] );
155 159
 		$settings = $helper->flattenArray( $settings, true );
156 160
 		foreach( ['submissions.recaptcha.key', 'submissions.recaptcha.secret'] as $key ) {
157
-			if( empty( $settings[$key] ))continue;
161
+			if( empty( $settings[$key] )) {
162
+				continue;
163
+			}
158 164
 			$settings[$key] = str_repeat( '*', 10 );
159 165
 		}
160 166
 		$details = [];
161 167
 		foreach( $settings as $key => $value ) {
162
-			if( $helper->startsWith( 'strings', $key ) && $helper->endsWith( 'id', $key ))continue;
168
+			if( $helper->startsWith( 'strings', $key ) && $helper->endsWith( 'id', $key )) {
169
+				continue;
170
+			}
163 171
 			$value = htmlspecialchars( trim( preg_replace('/\s\s+/', '\\n', $value )), ENT_QUOTES, 'UTF-8' );
164 172
 			$details[$key] = $value;
165 173
 		}
@@ -231,7 +239,9 @@  discard block
 block discarded – undo
231 239
 			'WPE_APIKEY' => 'WP Engine',
232 240
 		];
233 241
 		foreach( $checks as $key => $value ) {
234
-			if( !$this->isWebhostCheckValid( $key ))continue;
242
+			if( !$this->isWebhostCheckValid( $key )) {
243
+				continue;
244
+			}
235 245
 			return $value;
236 246
 		}
237 247
 		return implode( ',', array_filter( [DB_HOST, filter_input( INPUT_SERVER, 'SERVER_NAME' )] ));
Please login to merge, or discard this patch.
autoload.php 2 patches
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -2,7 +2,7 @@  discard block
 block discarded – undo
2 2
 
3 3
 defined( 'WPINC' ) || die;
4 4
 
5
-require_once( ABSPATH.WPINC.'/class-phpass.php' );
5
+require_once(ABSPATH.WPINC.'/class-phpass.php');
6 6
 
7 7
 spl_autoload_register( function( $className ) {
8 8
 	$namespaces = [
@@ -16,8 +16,8 @@  discard block
 block discarded – undo
16 16
 	foreach( $namespaces as $prefix => $baseDir ) {
17 17
 		$len = strlen( $prefix );
18 18
 		if( strncmp( $prefix, $className, $len ) !== 0 )continue;
19
-		$file = $baseDir.str_replace( '\\', '/', substr( $className, $len )).'.php';
20
-		if( !file_exists( $file ))continue;
19
+		$file = $baseDir.str_replace( '\\', '/', substr( $className, $len ) ).'.php';
20
+		if( !file_exists( $file ) )continue;
21 21
 		require $file;
22 22
 		break;
23 23
 	}
Please login to merge, or discard this patch.
Braces   +6 added lines, -2 removed lines patch added patch discarded remove patch
@@ -15,9 +15,13 @@
 block discarded – undo
15 15
 	];
16 16
 	foreach( $namespaces as $prefix => $baseDir ) {
17 17
 		$len = strlen( $prefix );
18
-		if( strncmp( $prefix, $className, $len ) !== 0 )continue;
18
+		if( strncmp( $prefix, $className, $len ) !== 0 ) {
19
+			continue;
20
+		}
19 21
 		$file = $baseDir.str_replace( '\\', '/', substr( $className, $len )).'.php';
20
-		if( !file_exists( $file ))continue;
22
+		if( !file_exists( $file )) {
23
+			continue;
24
+		}
21 25
 		require $file;
22 26
 		break;
23 27
 	}
Please login to merge, or discard this patch.
helpers.php 2 patches
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -6,7 +6,7 @@  discard block
 block discarded – undo
6 6
  */
7 7
 function glsr( $alias = null ) {
8 8
 	$app = \GeminiLabs\SiteReviews\Application::load();
9
-	return !empty( $alias )
9
+	return !empty($alias)
10 10
 		? $app->make( $alias )
11 11
 		: $app;
12 12
 }
@@ -15,10 +15,10 @@  discard block
 block discarded – undo
15 15
  * @return \WP_Screen|object
16 16
  */
17 17
 function glsr_current_screen() {
18
-	if( function_exists( 'get_current_screen' )) {
18
+	if( function_exists( 'get_current_screen' ) ) {
19 19
 		$screen = get_current_screen();
20 20
 	}
21
-	return empty( $screen )
21
+	return empty($screen)
22 22
 		? (object)array_fill_keys( ['base', 'id', 'post_type'], null )
23 23
 		: $screen;
24 24
 }
@@ -53,11 +53,11 @@  discard block
 block discarded – undo
53 53
  */
54 54
 function glsr_log() {
55 55
 	$args = func_get_args();
56
-	$context = isset( $args[1] )
56
+	$context = isset($args[1])
57 57
 		? $args[1]
58 58
 		: [];
59 59
 	$logger = glsr( 'Modules\Logger' );
60
-	return !empty( $args )
60
+	return !empty($args)
61 61
 		? $logger->log( 'debug', $args[0], $context )
62 62
 		: $logger;
63 63
 }
Please login to merge, or discard this patch.
Braces   +14 added lines, -7 removed lines patch added patch discarded remove patch
@@ -4,7 +4,8 @@  discard block
 block discarded – undo
4 4
 /**
5 5
  * @return mixed
6 6
  */
7
-function glsr( $alias = null ) {
7
+function glsr( $alias = null )
8
+{
8 9
 	$app = \GeminiLabs\SiteReviews\Application::load();
9 10
 	return !empty( $alias )
10 11
 		? $app->make( $alias )
@@ -14,7 +15,8 @@  discard block
 block discarded – undo
14 15
 /**
15 16
  * @return \WP_Screen|object
16 17
  */
17
-function glsr_current_screen() {
18
+function glsr_current_screen()
19
+{
18 20
 	if( function_exists( 'get_current_screen' )) {
19 21
 		$screen = get_current_screen();
20 22
 	}
@@ -26,7 +28,8 @@  discard block
 block discarded – undo
26 28
 /**
27 29
  * @return \GeminiLabs\SiteReviews\Database
28 30
  */
29
-function glsr_db() {
31
+function glsr_db()
32
+{
30 33
 	return glsr( 'Database' );
31 34
 }
32 35
 
@@ -34,7 +37,8 @@  discard block
 block discarded – undo
34 37
  * @param mixed ...$vars
35 38
  * @return void
36 39
  */
37
-function glsr_debug( ...$vars ) {
40
+function glsr_debug( ...$vars )
41
+{
38 42
 	if( count( $vars ) == 1 ) {
39 43
 		$value = htmlspecialchars( print_r( $vars[0], true ), ENT_QUOTES, 'UTF-8' );
40 44
 		printf( '<div class="glsr-debug"><pre>%s</pre></div>', $value );
@@ -51,7 +55,8 @@  discard block
 block discarded – undo
51 55
 /**
52 56
  * @return \GeminiLabs\SiteReviews\Modules\Logger
53 57
  */
54
-function glsr_log() {
58
+function glsr_log()
59
+{
55 60
 	$args = func_get_args();
56 61
 	$context = isset( $args[1] )
57 62
 		? $args[1]
@@ -67,14 +72,16 @@  discard block
 block discarded – undo
67 72
  * @param mixed $fallback
68 73
  * @return string|array
69 74
  */
70
-function glsr_get_option( $path = '', $fallback = '' ) {
75
+function glsr_get_option( $path = '', $fallback = '' )
76
+{
71 77
 	return glsr( 'Database\OptionManager' )->get( 'settings.'.$path, $fallback );
72 78
 }
73 79
 
74 80
 /**
75 81
  * @return array
76 82
  */
77
-function glsr_get_options() {
83
+function glsr_get_options()
84
+{
78 85
 	return glsr( 'Database\OptionManager' )->get( 'settings' );
79 86
 }
80 87
 
Please login to merge, or discard this patch.
plugin/Router.php 2 patches
Spacing   +10 added lines, -10 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
 		$this->routeRequest( 'admin', $request['action'], $request );
22 22
 	}
@@ -27,7 +27,7 @@  discard block
 block discarded – undo
27 27
 	public function routeAjaxRequest()
28 28
 	{
29 29
 		$request = $this->normalizeAjaxRequest();
30
-		if( !wp_verify_nonce( $request['nonce'], $request['action'] )) {
30
+		if( !wp_verify_nonce( $request['nonce'], $request['action'] ) ) {
31 31
 			glsr_log()->error( 'Nonce check failed for ajax request' )->info( $request );
32 32
 			wp_die( -1, 403 );
33 33
 		}
@@ -41,8 +41,8 @@  discard block
 block discarded – undo
41 41
 	public function routePublicPostRequest()
42 42
 	{
43 43
 		$request = filter_input( INPUT_POST, Application::ID, FILTER_DEFAULT, FILTER_REQUIRE_ARRAY );
44
-		if( !isset( $request['action'] ))return;
45
-		if( !wp_verify_nonce( $request['_wpnonce'], $request['action'] )) {
44
+		if( !isset($request['action']) )return;
45
+		if( !wp_verify_nonce( $request['_wpnonce'], $request['action'] ) ) {
46 46
 			glsr_log()->error( 'Nonce check failed for public request' )->info( $request );
47 47
 			return;
48 48
 		}
@@ -57,9 +57,9 @@  discard block
 block discarded – undo
57 57
 	 */
58 58
 	public function routeRequest( $type, $action, array $request = [] )
59 59
 	{
60
-		$controller = glsr( glsr( Helper::class )->buildClassName( $type.'-controller', 'Controllers' ));
60
+		$controller = glsr( glsr( Helper::class )->buildClassName( $type.'-controller', 'Controllers' ) );
61 61
 		$method = glsr( Helper::class )->buildMethodName( $action, 'router' );
62
-		if( is_callable( [$controller, $method] )) {
62
+		if( is_callable( [$controller, $method] ) ) {
63 63
 			call_user_func( [$controller, $method], $request );
64 64
 			return;
65 65
 		}
@@ -75,7 +75,7 @@  discard block
 block discarded – undo
75 75
 	 */
76 76
 	public function routeWebhookRequest()
77 77
 	{
78
-		$request = filter_input( INPUT_GET, sprintf( '%s-hook', Application::ID ));
78
+		$request = filter_input( INPUT_GET, sprintf( '%s-hook', Application::ID ) );
79 79
 		if( !$request )return;
80 80
 		// @todo manage webhook here
81 81
 	}
@@ -114,14 +114,14 @@  discard block
 block discarded – undo
114 114
 	protected function normalizeAjaxRequest()
115 115
 	{
116 116
 		$request = filter_input( INPUT_POST, 'request', FILTER_DEFAULT, FILTER_REQUIRE_ARRAY );
117
-		if( isset( $request[Application::ID]['action'] )) {
117
+		if( isset($request[Application::ID]['action']) ) {
118 118
 			$request = $request[Application::ID];
119 119
 		}
120
-		if( !isset( $request['action'] )) {
120
+		if( !isset($request['action']) ) {
121 121
 			glsr_log()->error( 'The AJAX request must include an action' )->info( $request );
122 122
 			wp_die();
123 123
 		}
124
-		if( !isset( $request['nonce'] )) {
124
+		if( !isset($request['nonce']) ) {
125 125
 			glsr_log()->error( 'The AJAX request must include a nonce' )->info( $request );
126 126
 			wp_die();
127 127
 		}
Please login to merge, or discard this patch.
Braces   +9 added lines, -3 removed lines patch added patch discarded remove patch
@@ -16,7 +16,9 @@  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'] )) {
20
+			return;
21
+		}
20 22
 		$this->checkNonce( $request['action'] );
21 23
 		$this->routeRequest( 'admin', $request['action'], $request );
22 24
 	}
@@ -41,7 +43,9 @@  discard block
 block discarded – undo
41 43
 	public function routePublicPostRequest()
42 44
 	{
43 45
 		$request = filter_input( INPUT_POST, Application::ID, FILTER_DEFAULT, FILTER_REQUIRE_ARRAY );
44
-		if( !isset( $request['action'] ))return;
46
+		if( !isset( $request['action'] )) {
47
+			return;
48
+		}
45 49
 		if( !wp_verify_nonce( $request['_wpnonce'], $request['action'] )) {
46 50
 			glsr_log()->error( 'Nonce check failed for public request' )->info( $request );
47 51
 			return;
@@ -76,7 +80,9 @@  discard block
 block discarded – undo
76 80
 	public function routeWebhookRequest()
77 81
 	{
78 82
 		$request = filter_input( INPUT_GET, sprintf( '%s-hook', Application::ID ));
79
-		if( !$request )return;
83
+		if( !$request ) {
84
+			return;
85
+		}
80 86
 		// @todo manage webhook here
81 87
 	}
82 88
 
Please login to merge, or discard this patch.
plugin/Commands/CreateReview.php 1 patch
Spacing   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -27,21 +27,21 @@  discard block
 block discarded – undo
27 27
 	public function __construct( $input )
28 28
 	{
29 29
 		$this->request = $input;
30
-		$this->ajax_request = isset( $input['ajax_request'] );
30
+		$this->ajax_request = isset($input['ajax_request']);
31 31
 		$this->assigned_to = $this->getNumeric( 'assign_to' );
32
-		$this->author = sanitize_text_field( $this->get( 'name' ));
33
-		$this->avatar = get_avatar_url( $this->get( 'email' ));
34
-		$this->blacklisted = isset( $input['blacklisted'] );
35
-		$this->category = sanitize_key( $this->get( 'category' ));
36
-		$this->content = sanitize_textarea_field( $this->get( 'content' ));
32
+		$this->author = sanitize_text_field( $this->get( 'name' ) );
33
+		$this->avatar = get_avatar_url( $this->get( 'email' ) );
34
+		$this->blacklisted = isset($input['blacklisted']);
35
+		$this->category = sanitize_key( $this->get( 'category' ) );
36
+		$this->content = sanitize_textarea_field( $this->get( 'content' ) );
37 37
 		$this->custom = $this->getCustom();
38
-		$this->email = sanitize_email( $this->get( 'email' ));
39
-		$this->form_id = sanitize_key( $this->get( 'form_id' ));
38
+		$this->email = sanitize_email( $this->get( 'email' ) );
39
+		$this->form_id = sanitize_key( $this->get( 'form_id' ) );
40 40
 		$this->ip_address = $this->get( 'ip_address' );
41
-		$this->rating = intval( $this->get( 'rating' ));
41
+		$this->rating = intval( $this->get( 'rating' ) );
42 42
 		$this->referrer = $this->get( '_wp_http_referer' );
43
-		$this->terms = isset( $input['terms'] );
44
-		$this->title = sanitize_text_field( $this->get( 'title' ));
43
+		$this->terms = isset($input['terms']);
44
+		$this->title = sanitize_text_field( $this->get( 'title' ) );
45 45
 	}
46 46
 
47 47
 	/**
@@ -50,7 +50,7 @@  discard block
 block discarded – undo
50 50
 	 */
51 51
 	protected function get( $key )
52 52
 	{
53
-		return isset( $this->request[$key] )
53
+		return isset($this->request[$key])
54 54
 			? (string)$this->request[$key]
55 55
 			: '';
56 56
 	}
@@ -67,7 +67,7 @@  discard block
 block discarded – undo
67 67
 		];
68 68
 		$custom = $this->request;
69 69
 		foreach( $unset as $value ) {
70
-			unset( $custom[$value] );
70
+			unset($custom[$value]);
71 71
 		}
72 72
 		return $custom;
73 73
 	}
Please login to merge, or discard this patch.
plugin/Database.php 2 patches
Spacing   +32 added lines, -32 removed lines patch added patch discarded remove patch
@@ -35,7 +35,7 @@  discard block
 block discarded – undo
35 35
 			'post_type' => Application::POST_TYPE,
36 36
 		];
37 37
 		$postId = wp_insert_post( $post, true );
38
-		if( is_wp_error( $postId )) {
38
+		if( is_wp_error( $postId ) ) {
39 39
 			glsr_log()->error( $postId->get_error_message() );
40 40
 			return false;
41 41
 		}
@@ -51,7 +51,7 @@  discard block
 block discarded – undo
51 51
 	 */
52 52
 	public function deleteReview( $metaReviewId )
53 53
 	{
54
-		if( $postId = $this->getReviewPostId( $metaReviewId )) {
54
+		if( $postId = $this->getReviewPostId( $metaReviewId ) ) {
55 55
 			wp_delete_post( $postId, true );
56 56
 		}
57 57
 	}
@@ -64,12 +64,12 @@  discard block
 block discarded – undo
64 64
 	public function getAssignedToPost( $post, $assignedTo = '' )
65 65
 	{
66 66
 		$post = get_post( $post );
67
-		if( !( $post instanceof WP_Post ))return;
68
-		if( empty( $assignedTo )) {
67
+		if( !($post instanceof WP_Post) )return;
68
+		if( empty($assignedTo) ) {
69 69
 			$assignedTo = get_post_meta( $post->ID, 'assigned_to', true );
70 70
 		}
71 71
 		$assignedPost = get_post( $assignedTo );
72
-		if( !empty( $assignedTo )
72
+		if( !empty($assignedTo)
73 73
 			&& $assignedPost instanceof WP_Post
74 74
 			&& $assignedPost->ID != $post->ID ) {
75 75
 			return $assignedPost;
@@ -82,7 +82,7 @@  discard block
 block discarded – undo
82 82
 	 */
83 83
 	public function getReview( $post )
84 84
 	{
85
-		if( !( $post instanceof WP_Post ) || $post->post_type != Application::POST_TYPE )return;
85
+		if( !($post instanceof WP_Post) || $post->post_type != Application::POST_TYPE )return;
86 86
 		$review = $this->getReviewMeta( $post->ID );
87 87
 		$modified = $this->isReviewModified( $post, $review );
88 88
 		$review->content = $post->post_content;
@@ -104,13 +104,13 @@  discard block
 block discarded – undo
104 104
 	{
105 105
 		$metaKey = $this->normalizeMetaKey( $metaKey );
106 106
 		if( !$metaKey ) {
107
-			return (array) wp_count_posts( Application::POST_TYPE );
107
+			return (array)wp_count_posts( Application::POST_TYPE );
108 108
 		}
109 109
 		$counts = glsr( Cache::class )->getReviewCountsFor( $metaKey );
110 110
 		if( !$metaValue ) {
111 111
 			return $counts;
112 112
 		}
113
-		return isset( $counts[$metaValue] )
113
+		return isset($counts[$metaValue])
114 114
 			? $counts[$metaValue]
115 115
 			: 0;
116 116
 	}
@@ -131,9 +131,9 @@  discard block
 block discarded – undo
131 131
 	public function getReviewMeta( $postId )
132 132
 	{
133 133
 		$meta = get_post_type( $postId ) == Application::POST_TYPE
134
-			? array_map( 'array_shift', (array) get_post_meta( $postId ))
134
+			? array_map( 'array_shift', (array)get_post_meta( $postId ) )
135 135
 			: [];
136
-		return (object) $this->normalizeMeta( array_filter( $meta, 'strlen' ));
136
+		return (object)$this->normalizeMeta( array_filter( $meta, 'strlen' ) );
137 137
 	}
138 138
 
139 139
 	/**
@@ -162,7 +162,7 @@  discard block
 block discarded – undo
162 162
 		$paged = glsr( QueryBuilder::class )->getPaged(
163 163
 			wp_validate_boolean( $args['pagination'] )
164 164
 		);
165
-		$reviews = new WP_Query([
165
+		$reviews = new WP_Query( [
166 166
 			'meta_key' => 'pinned',
167 167
 			'meta_query' => $metaQuery,
168 168
 			'offset' => $args['offset'],
@@ -175,8 +175,8 @@  discard block
 block discarded – undo
175 175
 			'post_type' => Application::POST_TYPE,
176 176
 			'posts_per_page' => $args['count'] ? $args['count'] : -1,
177 177
 			'tax_query' => $taxQuery,
178
-		]);
179
-		return (object) [
178
+		] );
179
+		return (object)[
180 180
 			'results' => array_map( [$this, 'getReview'], $reviews->posts ),
181 181
 			'max_num_pages' => $reviews->max_num_pages,
182 182
 		];
@@ -190,7 +190,7 @@  discard block
 block discarded – undo
190 190
 	public function getReviewsMeta( $keys, $status = 'publish' )
191 191
 	{
192 192
 		$keys = array_map( [$this, 'normalizeMetaKey'], (array)$keys );
193
-		if( $status == 'all' || empty( $status )) {
193
+		if( $status == 'all' || empty($status) ) {
194 194
 			$status = get_post_stati( ['exclude_from_search' => false] );
195 195
 		}
196 196
 		return glsr( SqlQueries::class )->getReviewsMeta( $keys, $status );
@@ -206,10 +206,10 @@  discard block
 block discarded – undo
206 206
 			'fields' => 'id=>name',
207 207
 			'hide_empty' => false,
208 208
 			'taxonomy' => Application::TAXONOMY,
209
-		]);
210
-		unset( $args['count'] ); //we don't want a term count
209
+		] );
210
+		unset($args['count']); //we don't want a term count
211 211
 		$terms = get_terms( $args );
212
-		if( is_wp_error( $terms )) {
212
+		if( is_wp_error( $terms ) ) {
213 213
 			glsr_log()->error( $terms->get_error_message() );
214 214
 			return [];
215 215
 		}
@@ -221,7 +221,7 @@  discard block
 block discarded – undo
221 221
 	 */
222 222
 	public function normalizeMeta( array $meta )
223 223
 	{
224
-		if( empty( $meta )) {
224
+		if( empty($meta) ) {
225 225
 			return [];
226 226
 		}
227 227
 		$defaults = wp_parse_args( $meta, [
@@ -229,7 +229,7 @@  discard block
 block discarded – undo
229 229
 			'date' => '',
230 230
 			'review_id' => '',
231 231
 			'review_type' => '',
232
-		]);
232
+		] );
233 233
 		return glsr( CreateReviewDefaults::class )->restrict( $defaults );
234 234
 	}
235 235
 
@@ -240,7 +240,7 @@  discard block
 block discarded – undo
240 240
 	public function normalizeMetaKey( $metaKey )
241 241
 	{
242 242
 		$metaKey = strtolower( $metaKey );
243
-		if( in_array( $metaKey, ['id', 'type'] )) {
243
+		if( in_array( $metaKey, ['id', 'type'] ) ) {
244 244
 			$metaKey = 'review_'.$metaKey;
245 245
 		}
246 246
 		return $metaKey;
@@ -253,10 +253,10 @@  discard block
 block discarded – undo
253 253
 	public function normalizeTerms( $termIds )
254 254
 	{
255 255
 		$terms = [];
256
-		$termIds = array_map( 'trim', explode( ',', $termIds ));
256
+		$termIds = array_map( 'trim', explode( ',', $termIds ) );
257 257
 		foreach( $termIds as $termId ) {
258 258
 			$term = term_exists( $termId, Application::TAXONOMY );
259
-			if( !isset( $term['term_id'] ))continue;
259
+			if( !isset($term['term_id']) )continue;
260 260
 			$terms[] = intval( $term['term_id'] );
261 261
 		}
262 262
 		return $terms;
@@ -269,15 +269,15 @@  discard block
 block discarded – undo
269 269
 	public function revertReview( $postId )
270 270
 	{
271 271
 		$post = get_post( $postId );
272
-		if( !( $post instanceof WP_Post ) || $post->post_type != Application::POST_TYPE )return;
272
+		if( !($post instanceof WP_Post) || $post->post_type != Application::POST_TYPE )return;
273 273
 		delete_post_meta( $post->ID, '_edit_last' );
274
-		$result = wp_update_post([
274
+		$result = wp_update_post( [
275 275
 			'ID' => $post->ID,
276 276
 			'post_content' => get_post_meta( $post->ID, 'content', true ),
277 277
 			'post_date' => get_post_meta( $post->ID, 'date', true ),
278 278
 			'post_title' => get_post_meta( $post->ID, 'title', true ),
279
-		]);
280
-		if( is_wp_error( $result )) {
279
+		] );
280
+		if( is_wp_error( $result ) ) {
281 281
 			glsr_log()->error( $result->get_error_message() );
282 282
 		}
283 283
 	}
@@ -292,7 +292,7 @@  discard block
 block discarded – undo
292 292
 			'post_status' => 'publish',
293 293
 			'post_type' => 'any',
294 294
 		];
295
-		if( is_numeric( $searchTerm )) {
295
+		if( is_numeric( $searchTerm ) ) {
296 296
 			$args['post__in'] = [$searchTerm];
297 297
 		}
298 298
 		else {
@@ -311,9 +311,9 @@  discard block
 block discarded – undo
311 311
 			ob_start();
312 312
 			glsr()->render( 'partials/editor/search-result', [
313 313
 				'ID' => get_the_ID(),
314
-				'permalink' => esc_url( (string) get_permalink() ),
314
+				'permalink' => esc_url( (string)get_permalink() ),
315 315
 				'title' => esc_attr( get_the_title() ),
316
-			]);
316
+			] );
317 317
 			$results .= ob_get_clean();
318 318
 		}
319 319
 		wp_reset_postdata();
@@ -331,9 +331,9 @@  discard block
 block discarded – undo
331 331
 			update_post_meta( $postId, $metaKey, $metaValue );
332 332
 		}
333 333
 		$terms = $this->normalizeTerms( $termIds );
334
-		if( empty( $terms ))return;
334
+		if( empty($terms) )return;
335 335
 		$result = wp_set_object_terms( $postId, $terms, Application::TAXONOMY );
336
-		if( is_wp_error( $result )) {
336
+		if( is_wp_error( $result ) ) {
337 337
 			glsr_log()->error( $result->get_error_message() );
338 338
 		}
339 339
 	}
@@ -345,7 +345,7 @@  discard block
 block discarded – undo
345 345
 	protected function getNewPostStatus( array $review, $isBlacklisted )
346 346
 	{
347 347
 		$requireApprovalOption = glsr( OptionManager::class )->get( 'settings.general.require.approval' );
348
-		return $review['review_type'] == 'local' && ( $requireApprovalOption == 'yes' || $isBlacklisted )
348
+		return $review['review_type'] == 'local' && ($requireApprovalOption == 'yes' || $isBlacklisted)
349 349
 			? 'pending'
350 350
 			: 'publish';
351 351
 	}
Please login to merge, or discard this patch.
Braces   +18 added lines, -6 removed lines patch added patch discarded remove patch
@@ -64,7 +64,9 @@  discard block
 block discarded – undo
64 64
 	public function getAssignedToPost( $post, $assignedTo = '' )
65 65
 	{
66 66
 		$post = get_post( $post );
67
-		if( !( $post instanceof WP_Post ))return;
67
+		if( !( $post instanceof WP_Post )) {
68
+			return;
69
+		}
68 70
 		if( empty( $assignedTo )) {
69 71
 			$assignedTo = get_post_meta( $post->ID, 'assigned_to', true );
70 72
 		}
@@ -82,7 +84,9 @@  discard block
 block discarded – undo
82 84
 	 */
83 85
 	public function getReview( $post )
84 86
 	{
85
-		if( !( $post instanceof WP_Post ) || $post->post_type != Application::POST_TYPE )return;
87
+		if( !( $post instanceof WP_Post ) || $post->post_type != Application::POST_TYPE ) {
88
+			return;
89
+		}
86 90
 		$review = $this->getReviewMeta( $post->ID );
87 91
 		$modified = $this->isReviewModified( $post, $review );
88 92
 		$review->content = $post->post_content;
@@ -256,7 +260,9 @@  discard block
 block discarded – undo
256 260
 		$termIds = array_map( 'trim', explode( ',', $termIds ));
257 261
 		foreach( $termIds as $termId ) {
258 262
 			$term = term_exists( $termId, Application::TAXONOMY );
259
-			if( !isset( $term['term_id'] ))continue;
263
+			if( !isset( $term['term_id'] )) {
264
+				continue;
265
+			}
260 266
 			$terms[] = intval( $term['term_id'] );
261 267
 		}
262 268
 		return $terms;
@@ -269,7 +275,9 @@  discard block
 block discarded – undo
269 275
 	public function revertReview( $postId )
270 276
 	{
271 277
 		$post = get_post( $postId );
272
-		if( !( $post instanceof WP_Post ) || $post->post_type != Application::POST_TYPE )return;
278
+		if( !( $post instanceof WP_Post ) || $post->post_type != Application::POST_TYPE ) {
279
+			return;
280
+		}
273 281
 		delete_post_meta( $post->ID, '_edit_last' );
274 282
 		$result = wp_update_post([
275 283
 			'ID' => $post->ID,
@@ -304,7 +312,9 @@  discard block
 block discarded – undo
304 312
 		add_filter( 'posts_search', [$queryBuilder, 'filterSearchByTitle'], 500, 2 );
305 313
 		$search = new WP_Query( $args );
306 314
 		remove_filter( 'posts_search', [$queryBuilder, 'filterSearchByTitle'], 500 );
307
-		if( !$search->have_posts() )return;
315
+		if( !$search->have_posts() ) {
316
+			return;
317
+		}
308 318
 		$results = '';
309 319
 		while( $search->have_posts() ) {
310 320
 			$search->the_post();
@@ -331,7 +341,9 @@  discard block
 block discarded – undo
331 341
 			update_post_meta( $postId, $metaKey, $metaValue );
332 342
 		}
333 343
 		$terms = $this->normalizeTerms( $termIds );
334
-		if( empty( $terms ))return;
344
+		if( empty( $terms )) {
345
+			return;
346
+		}
335 347
 		$result = wp_set_object_terms( $postId, $terms, Application::TAXONOMY );
336 348
 		if( is_wp_error( $result )) {
337 349
 			glsr_log()->error( $result->get_error_message() );
Please login to merge, or discard this patch.