Passed
Push — master ( 31d720...042452 )
by Paul
04:27
created
plugin/Modules/Html/Form.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -27,13 +27,13 @@
 block discarded – undo
27 27
 	public function getFields( $id )
28 28
 	{
29 29
 		$fields = [];
30
-		$configPath = glsr()->path( 'config/'.$id.'.php' );;
30
+		$configPath = glsr()->path( 'config/'.$id.'.php' ); ;
31 31
 		$values = file_exists( $configPath )
32 32
 			? include $configPath
33 33
 			: [];
34 34
 		$values = apply_filters( 'site-reviews/form/fields', $values );
35 35
 		foreach( $values as $name => $field ) {
36
-			$fields[] = new Field( wp_parse_args( $field, ['name' => $name] ));
36
+			$fields[] = new Field( wp_parse_args( $field, ['name' => $name] ) );
37 37
 		}
38 38
 		return $fields;
39 39
 	}
Please login to merge, or discard this patch.
plugin/Modules/Html/Settings.php 1 patch
Spacing   +14 added lines, -14 removed lines patch added patch discarded remove patch
@@ -23,7 +23,7 @@  discard block
 block discarded – undo
23 23
 			: $this->$method( $id );
24 24
 		return glsr( Template::class )->build( 'pages/settings/'.$id, [
25 25
 			'context' => $context,
26
-		]);
26
+		] );
27 27
 	}
28 28
 
29 29
 	/**
@@ -31,7 +31,7 @@  discard block
 block discarded – undo
31 31
 	 */
32 32
 	protected function getFieldDefault( array $field )
33 33
 	{
34
-		return isset( $field['default'] )
34
+		return isset($field['default'])
35 35
 			? $field['default']
36 36
 			: '';
37 37
 	}
@@ -42,7 +42,7 @@  discard block
 block discarded – undo
42 42
 	protected function getSettingFields( $path )
43 43
 	{
44 44
 		$settings = glsr( DefaultsManager::class )->settings();
45
-		return array_filter( $settings, function( $key ) use( $path ) {
45
+		return array_filter( $settings, function( $key ) use($path) {
46 46
 			return glsr( Helper::class )->startsWith( $path, $key );
47 47
 		}, ARRAY_FILTER_USE_KEY );
48 48
 	}
@@ -53,14 +53,14 @@  discard block
 block discarded – undo
53 53
 	 */
54 54
 	protected function getTemplateContext( $id )
55 55
 	{
56
-		$fields = $this->getSettingFields( $this->normalizeSettingPath( $id ));
56
+		$fields = $this->getSettingFields( $this->normalizeSettingPath( $id ) );
57 57
 		$rows = '';
58 58
 		foreach( $fields as $name => $field ) {
59 59
 			$field = wp_parse_args( $field, [
60 60
 				'is_setting' => true,
61 61
 				'name' => $name,
62
-			]);
63
-			$rows.= new Field( $this->normalize( $field ));
62
+			] );
63
+			$rows .= new Field( $this->normalize( $field ) );
64 64
 		}
65 65
 		return [
66 66
 			'rows' => $rows,
@@ -73,7 +73,7 @@  discard block
 block discarded – undo
73 73
 	protected function getTemplateContextForTranslations()
74 74
 	{
75 75
 		$translations = glsr( Translator::class )->renderAll();
76
-		$class = empty( $translations )
76
+		$class = empty($translations)
77 77
 			? 'glsr-hidden'
78 78
 			: '';
79 79
 		return [
@@ -94,7 +94,7 @@  discard block
 block discarded – undo
94 94
 			$path,
95 95
 			glsr( Helper::class )->getPathValue( $path, glsr()->defaults )
96 96
 		);
97
-		if( is_array( $expectedValue )) {
97
+		if( is_array( $expectedValue ) ) {
98 98
 			return !in_array( $optionValue, $expectedValue );
99 99
 		}
100 100
 		return $optionValue != $expectedValue;
@@ -116,13 +116,13 @@  discard block
 block discarded – undo
116 116
 	 */
117 117
 	protected function normalizeDependsOn( array $field )
118 118
 	{
119
-		if( !empty( $field['depends_on'] ) && is_array( $field['depends_on'] )) {
119
+		if( !empty($field['depends_on']) && is_array( $field['depends_on'] ) ) {
120 120
 			$path = key( $field['depends_on'] );
121 121
 			$expectedValue = $field['depends_on'][$path];
122
-			$field['data-depends'] = json_encode([
122
+			$field['data-depends'] = json_encode( [
123 123
 				'name' => glsr( Helper::class )->convertPathToName( $path, OptionManager::databaseKey() ),
124 124
 				'value' => $expectedValue,
125
-			], JSON_HEX_APOS|JSON_HEX_QUOT );
125
+			], JSON_HEX_APOS | JSON_HEX_QUOT );
126 126
 			$field['is_hidden'] = $this->isFieldHidden( $path, $expectedValue );
127 127
 		}
128 128
 		return $field;
@@ -133,9 +133,9 @@  discard block
 block discarded – undo
133 133
 	 */
134 134
 	protected function normalizeLabelAndLegend( array $field )
135 135
 	{
136
-		if( isset( $field['label'] )) {
136
+		if( isset($field['label']) ) {
137 137
 			$field['legend'] = $field['label'];
138
-			unset( $field['label'] );
138
+			unset($field['label']);
139 139
 		}
140 140
 		else {
141 141
 			$field['is_valid'] = false;
@@ -149,7 +149,7 @@  discard block
 block discarded – undo
149 149
 	 */
150 150
 	protected function normalizeValue( array $field )
151 151
 	{
152
-		if( !isset( $field['value'] )) {
152
+		if( !isset($field['value']) ) {
153 153
 			$field['value'] = glsr( OptionManager::class )->get(
154 154
 				$field['name'],
155 155
 				$this->getFieldDefault( $field )
Please login to merge, or discard this patch.
plugin/Modules/Html/Fields/Honeypot.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -13,7 +13,7 @@
 block discarded – undo
13 13
 	{
14 14
 		$this->builder->args = wp_parse_args( $this->builder->args, [
15 15
 			'name' => $this->builder->args['text'],
16
-		]);
16
+		] );
17 17
 		$this->builder->tag = 'input';
18 18
 		$this->mergeFieldArgs();
19 19
 		return $this->builder->getOpeningTag();
Please login to merge, or discard this patch.
plugin/Modules/Html/Builder.php 2 patches
Spacing   +24 added lines, -24 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
 	}
@@ -163,7 +163,7 @@  discard block
 block discarded – undo
163 163
 	protected function buildFormInputMultiChoice()
164 164
 	{
165 165
 		if( $this->args['type'] == 'checkbox' ) {
166
-			$this->args['name'].= '[]';
166
+			$this->args['name'] .= '[]';
167 167
 		}
168 168
 		$options = array_reduce( array_keys( $this->args['options'] ), function( $carry, $key ) {
169 169
 			return $carry.$this->li( $this->{$this->args['type']}([
@@ -171,12 +171,12 @@  discard block
 block discarded – undo
171 171
 				'name' => $this->args['name'],
172 172
 				'text' => $this->args['options'][$key],
173 173
 				'value' => $key,
174
-			]));
174
+			]) );
175 175
 		});
176 176
 		return $this->ul( $options, [
177 177
 			'class' => $this->args['class'],
178 178
 			'id' => $this->args['id'],
179
-		]);
179
+		] );
180 180
 	}
181 181
 
182 182
 	/**
@@ -184,11 +184,11 @@  discard block
 block discarded – undo
184 184
 	 */
185 185
 	protected function buildFormLabel()
186 186
 	{
187
-		if( empty( $this->args['label'] ) || $this->args['type'] == 'hidden' )return;
188
-		return $this->label([
187
+		if( empty($this->args['label']) || $this->args['type'] == 'hidden' )return;
188
+		return $this->label( [
189 189
 			'for' => $this->args['id'],
190 190
 			'text' => $this->args['label'],
191
-		]);
191
+		] );
192 192
 	}
193 193
 
194 194
 	/**
@@ -205,11 +205,11 @@  discard block
 block discarded – undo
205 205
 	protected function buildFormSelectOptions()
206 206
 	{
207 207
 		return array_reduce( array_keys( $this->args['options'] ), function( $carry, $key ) {
208
-			return $carry.$this->option([
208
+			return $carry.$this->option( [
209 209
 				'selected' => $this->args['value'] == $key,
210 210
 				'text' => $this->args['options'][$key],
211 211
 				'value' => $key,
212
-			]);
212
+			] );
213 213
 		});
214 214
 	}
215 215
 
@@ -227,10 +227,10 @@  discard block
 block discarded – undo
227 227
 	protected function buildTag()
228 228
 	{
229 229
 		$this->mergeArgsWithRequiredDefaults();
230
-		if( in_array( $this->tag, static::TAGS_SINGLE )) {
230
+		if( in_array( $this->tag, static::TAGS_SINGLE ) ) {
231 231
 			return $this->getOpeningTag();
232 232
 		}
233
-		if( !in_array( $this->tag, static::TAGS_FORM )) {
233
+		if( !in_array( $this->tag, static::TAGS_FORM ) ) {
234 234
 			return $this->buildDefaultTag();
235 235
 		}
236 236
 		return call_user_func( [$this, 'buildForm'.ucfirst( $this->tag )] ).$this->buildFieldDescription();
@@ -251,7 +251,7 @@  discard block
 block discarded – undo
251 251
 	{
252 252
 		$args = glsr( BuilderDefaults::class )->merge( $this->args );
253 253
 		$className = $this->getCustomFieldClassName();
254
-		if( class_exists( $className )) {
254
+		if( class_exists( $className ) ) {
255 255
 			$args = array_merge(
256 256
 				wp_parse_args( $args, $className::defaults() ),
257 257
 				$className::required()
@@ -266,13 +266,13 @@  discard block
 block discarded – undo
266 266
 	 */
267 267
 	protected function normalize( ...$params )
268 268
 	{
269
-		if( is_string( $params[0] ) || is_numeric( $params[0] )) {
269
+		if( is_string( $params[0] ) || is_numeric( $params[0] ) ) {
270 270
 			$this->setNameOrTextAttributeForTag( $params[0] );
271 271
 		}
272
-		if( is_array( $params[0] )) {
272
+		if( is_array( $params[0] ) ) {
273 273
 			$this->args += $params[0];
274 274
 		}
275
-		else if( is_array( $params[1] )) {
275
+		else if( is_array( $params[1] ) ) {
276 276
 			$this->args += $params[1];
277 277
 		}
278 278
 	}
@@ -296,7 +296,7 @@  discard block
 block discarded – undo
296 296
 	protected function setTagFromMethod( $method )
297 297
 	{
298 298
 		$this->tag = strtolower( $method );
299
-		if( in_array( $this->tag, static::INPUT_TYPES )) {
299
+		if( in_array( $this->tag, static::INPUT_TYPES ) ) {
300 300
 			$this->args['type'] = $this->tag;
301 301
 			$this->tag = 'input';
302 302
 		}
Please login to merge, or discard this patch.
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.
plugin/Modules/Html/Field.php 2 patches
Spacing   +15 added lines, -15 removed lines patch added patch discarded remove patch
@@ -25,7 +25,7 @@  discard block
 block discarded – undo
25 25
 			'is_valid' => true,
26 26
 			'is_widget' => false,
27 27
 			'path' => '',
28
-		]);
28
+		] );
29 29
 		$this->normalize();
30 30
 	}
31 31
 
@@ -71,9 +71,9 @@  discard block
 block discarded – undo
71 71
 		return glsr( Template::class )->build( 'partials/form/field', [
72 72
 			'context' => [
73 73
 				'class' => $this->getFieldClass(),
74
-				'field' => glsr( Builder::class )->{$this->field['type']}( $this->field ),
74
+				'field' => glsr( Builder::class )->{$this->field['type']}($this->field),
75 75
 			],
76
-		]);
76
+		] );
77 77
 	}
78 78
 
79 79
 	/**
@@ -84,10 +84,10 @@  discard block
 block discarded – undo
84 84
 		return glsr( Template::class )->build( 'partials/form/table-row', [
85 85
 			'context' => [
86 86
 				'class' => $this->getFieldClass(),
87
-				'field' => glsr( Builder::class )->{$this->field['type']}( $this->field ),
87
+				'field' => glsr( Builder::class )->{$this->field['type']}($this->field),
88 88
 				'label' => glsr( Builder::class )->label( $this->field['legend'], ['for' => $this->field['id']] ),
89 89
 			],
90
-		]);
90
+		] );
91 91
 	}
92 92
 
93 93
 	/**
@@ -96,16 +96,16 @@  discard block
 block discarded – undo
96 96
 	protected function buildSettingMultiField()
97 97
 	{
98 98
 		$dependsOn = $this->getFieldDependsOn();
99
-		unset( $this->field['data-depends'] );
99
+		unset($this->field['data-depends']);
100 100
 		return glsr( Template::class )->build( 'partials/form/table-row-multiple', [
101 101
 			'context' => [
102 102
 				'class' => $this->getFieldClass(),
103 103
 				'depends_on' => $dependsOn,
104
-				'field' => glsr( Builder::class )->{$this->field['type']}( $this->field ),
104
+				'field' => glsr( Builder::class )->{$this->field['type']}($this->field),
105 105
 				'label' => glsr( Builder::class )->label( $this->field['legend'], ['for' => $this->field['id']] ),
106 106
 				'legend' => $this->field['legend'],
107 107
 			],
108
-		]);
108
+		] );
109 109
 	}
110 110
 
111 111
 	/**
@@ -117,7 +117,7 @@  discard block
 block discarded – undo
117 117
 		if( $this->field['is_hidden'] ) {
118 118
 			$classes[] = 'hidden';
119 119
 		}
120
-		if( !empty( $this->field['required'] )) {
120
+		if( !empty($this->field['required']) ) {
121 121
 			$classes[] = 'glsr-required';
122 122
 		}
123 123
 		return implode( ' ', $classes );
@@ -128,7 +128,7 @@  discard block
 block discarded – undo
128 128
 	 */
129 129
 	protected function getFieldDependsOn()
130 130
 	{
131
-		return !empty( $this->field['data-depends'] )
131
+		return !empty($this->field['data-depends'])
132 132
 			? $this->field['data-depends']
133 133
 			: '';
134 134
 	}
@@ -153,13 +153,13 @@  discard block
 block discarded – undo
153 153
 			'name', 'type',
154 154
 		];
155 155
 		foreach( $requiredValues as $value ) {
156
-			if( isset( $this->field[$value] ))continue;
156
+			if( isset($this->field[$value]) )continue;
157 157
 			$missingValues[] = $value;
158 158
 			$this->field['is_valid'] = false;
159 159
 		}
160
-		if( !empty( $missingValues )) {
160
+		if( !empty($missingValues) ) {
161 161
 			glsr_log()
162
-				->warning( 'Field is missing: '.implode( ', ', $missingValues ))
162
+				->warning( 'Field is missing: '.implode( ', ', $missingValues ) )
163 163
 				->info( $this->field );
164 164
 		}
165 165
 		return $this->field['is_valid'];
@@ -173,7 +173,7 @@  discard block
 block discarded – undo
173 173
 		if( !$this->isFieldValid() )return;
174 174
 		$this->field['path'] = $this->field['name'];
175 175
 		$className = glsr( Helper::class )->buildClassName( $this->field['type'], __NAMESPACE__.'\Fields' );
176
-		if( class_exists( $className )) {
176
+		if( class_exists( $className ) ) {
177 177
 			$this->field = array_merge(
178 178
 				wp_parse_args( $this->field, $className::defaults() ),
179 179
 				$className::required()
@@ -188,7 +188,7 @@  discard block
 block discarded – undo
188 188
 	 */
189 189
 	protected function normalizeFieldId()
190 190
 	{
191
-		if( isset( $this->field['id'] ) || $this->field['is_raw'] )return;
191
+		if( isset($this->field['id']) || $this->field['is_raw'] )return;
192 192
 		$this->field['id'] = glsr( Helper::class )->convertPathToId(
193 193
 			$this->field['path'],
194 194
 			$this->getFieldPrefix()
Please login to merge, or discard this patch.
Braces   +12 added lines, -4 removed lines patch added patch discarded remove patch
@@ -42,7 +42,9 @@  discard block
 block discarded – undo
42 42
 	 */
43 43
 	public function build()
44 44
 	{
45
-		if( !$this->field['is_valid'] )return;
45
+		if( !$this->field['is_valid'] ) {
46
+			return;
47
+		}
46 48
 		if( $this->field['is_raw'] ) {
47 49
 			return glsr( Builder::class )->hidden( $this->field );
48 50
 		}
@@ -153,7 +155,9 @@  discard block
 block discarded – undo
153 155
 			'name', 'type',
154 156
 		];
155 157
 		foreach( $requiredValues as $value ) {
156
-			if( isset( $this->field[$value] ))continue;
158
+			if( isset( $this->field[$value] )) {
159
+				continue;
160
+			}
157 161
 			$missingValues[] = $value;
158 162
 			$this->field['is_valid'] = false;
159 163
 		}
@@ -170,7 +174,9 @@  discard block
 block discarded – undo
170 174
 	 */
171 175
 	protected function normalize()
172 176
 	{
173
-		if( !$this->isFieldValid() )return;
177
+		if( !$this->isFieldValid() ) {
178
+			return;
179
+		}
174 180
 		$this->field['path'] = $this->field['name'];
175 181
 		$className = glsr( Helper::class )->buildClassName( $this->field['type'], __NAMESPACE__.'\Fields' );
176 182
 		if( class_exists( $className )) {
@@ -188,7 +194,9 @@  discard block
 block discarded – undo
188 194
 	 */
189 195
 	protected function normalizeFieldId()
190 196
 	{
191
-		if( isset( $this->field['id'] ) || $this->field['is_raw'] )return;
197
+		if( isset( $this->field['id'] ) || $this->field['is_raw'] ) {
198
+			return;
199
+		}
192 200
 		$this->field['id'] = glsr( Helper::class )->convertPathToId(
193 201
 			$this->field['path'],
194 202
 			$this->getFieldPrefix()
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.