Passed
Push — master ( 75834f...73b77e )
by Paul
04:12
created
plugin/Modules/Html/Field.php 1 patch
Spacing   +18 added lines, -18 removed lines patch added patch discarded remove patch
@@ -26,7 +26,7 @@  discard block
 block discarded – undo
26 26
 			'is_valid' => true,
27 27
 			'is_widget' => false,
28 28
 			'path' => '',
29
-		]);
29
+		] );
30 30
 		$this->normalize();
31 31
 	}
32 32
 
@@ -45,7 +45,7 @@  discard block
 block discarded – undo
45 45
 	{
46 46
 		if( !$this->field['is_valid'] )return;
47 47
 		if( $this->field['is_raw'] ) {
48
-			return glsr( Builder::class )->{$this->field['type']}( $this->field );
48
+			return glsr( Builder::class )->{$this->field['type']}($this->field);
49 49
 		}
50 50
 		if( !$this->field['is_setting'] ) {
51 51
 			return $this->buildField();
@@ -73,9 +73,9 @@  discard block
 block discarded – undo
73 73
 			'context' => [
74 74
 				'class' => $this->getFieldClass(),
75 75
 				'errors' => $this->getFieldErrors(),
76
-				'field' => glsr( Builder::class )->{$this->field['type']}( $this->field ),
76
+				'field' => glsr( Builder::class )->{$this->field['type']}($this->field),
77 77
 			],
78
-		]);
78
+		] );
79 79
 	}
80 80
 
81 81
 	/**
@@ -86,10 +86,10 @@  discard block
 block discarded – undo
86 86
 		return glsr( Template::class )->build( 'partials/form/table-row', [
87 87
 			'context' => [
88 88
 				'class' => $this->getFieldClass(),
89
-				'field' => glsr( Builder::class )->{$this->field['type']}( $this->field ),
89
+				'field' => glsr( Builder::class )->{$this->field['type']}($this->field),
90 90
 				'label' => glsr( Builder::class )->label( $this->field['legend'], ['for' => $this->field['id']] ),
91 91
 			],
92
-		]);
92
+		] );
93 93
 	}
94 94
 
95 95
 	/**
@@ -98,16 +98,16 @@  discard block
 block discarded – undo
98 98
 	protected function buildSettingMultiField()
99 99
 	{
100 100
 		$dependsOn = $this->getFieldDependsOn();
101
-		unset( $this->field['data-depends'] );
101
+		unset($this->field['data-depends']);
102 102
 		return glsr( Template::class )->build( 'partials/form/table-row-multiple', [
103 103
 			'context' => [
104 104
 				'class' => $this->getFieldClass(),
105 105
 				'depends_on' => $dependsOn,
106
-				'field' => glsr( Builder::class )->{$this->field['type']}( $this->field ),
106
+				'field' => glsr( Builder::class )->{$this->field['type']}($this->field),
107 107
 				'label' => glsr( Builder::class )->label( $this->field['legend'], ['for' => $this->field['id']] ),
108 108
 				'legend' => $this->field['legend'],
109 109
 			],
110
-		]);
110
+		] );
111 111
 	}
112 112
 
113 113
 	/**
@@ -116,13 +116,13 @@  discard block
 block discarded – undo
116 116
 	protected function getFieldClass()
117 117
 	{
118 118
 		$classes = [];
119
-		if( !empty( $this->field['errors'] )) {
119
+		if( !empty($this->field['errors']) ) {
120 120
 			$classes[] = 'glsr-has-error';
121 121
 		}
122 122
 		if( $this->field['is_hidden'] ) {
123 123
 			$classes[] = 'hidden';
124 124
 		}
125
-		if( !empty( $this->field['required'] )) {
125
+		if( !empty($this->field['required']) ) {
126 126
 			$classes[] = 'glsr-required';
127 127
 		}
128 128
 		return implode( ' ', $classes );
@@ -133,7 +133,7 @@  discard block
 block discarded – undo
133 133
 	 */
134 134
 	protected function getFieldDependsOn()
135 135
 	{
136
-		return !empty( $this->field['data-depends'] )
136
+		return !empty($this->field['data-depends'])
137 137
 			? $this->field['data-depends']
138 138
 			: '';
139 139
 	}
@@ -143,7 +143,7 @@  discard block
 block discarded – undo
143 143
 	 */
144 144
 	protected function getFieldErrors()
145 145
 	{
146
-		if( empty( $this->field['errors'] ) || !is_array( $this->field['errors'] ))return;
146
+		if( empty($this->field['errors']) || !is_array( $this->field['errors'] ) )return;
147 147
 		$errors = array_reduce( $this->field['errors'], function( $carry, $error ) {
148 148
 			return $carry.glsr( Builder::class )->span( $error, ['class' => 'glsr-field-error'] );
149 149
 		});
@@ -170,13 +170,13 @@  discard block
 block discarded – undo
170 170
 			'name', 'type',
171 171
 		];
172 172
 		foreach( $requiredValues as $value ) {
173
-			if( isset( $this->field[$value] ))continue;
173
+			if( isset($this->field[$value]) )continue;
174 174
 			$missingValues[] = $value;
175 175
 			$this->field['is_valid'] = false;
176 176
 		}
177
-		if( !empty( $missingValues )) {
177
+		if( !empty($missingValues) ) {
178 178
 			glsr_log()
179
-				->warning( 'Field is missing: '.implode( ', ', $missingValues ))
179
+				->warning( 'Field is missing: '.implode( ', ', $missingValues ) )
180 180
 				->info( $this->field );
181 181
 		}
182 182
 		return $this->field['is_valid'];
@@ -190,7 +190,7 @@  discard block
 block discarded – undo
190 190
 		if( !$this->isFieldValid() )return;
191 191
 		$this->field['path'] = $this->field['name'];
192 192
 		$className = glsr( Helper::class )->buildClassName( $this->field['type'], __NAMESPACE__.'\Fields' );
193
-		if( class_exists( $className )) {
193
+		if( class_exists( $className ) ) {
194 194
 			$this->field = array_merge(
195 195
 				wp_parse_args( $this->field, $className::defaults() ),
196 196
 				$className::required()
@@ -205,7 +205,7 @@  discard block
 block discarded – undo
205 205
 	 */
206 206
 	protected function normalizeFieldId()
207 207
 	{
208
-		if( isset( $this->field['id'] ) || $this->field['is_raw'] )return;
208
+		if( isset($this->field['id']) || $this->field['is_raw'] )return;
209 209
 		$this->field['id'] = glsr( Helper::class )->convertPathToId(
210 210
 			$this->field['path'],
211 211
 			$this->getFieldPrefix()
Please login to merge, or discard this patch.
plugin/Modules/Validator.php 1 patch
Spacing   +17 added lines, -17 removed lines patch added patch discarded remove patch
@@ -73,7 +73,7 @@  discard block
 block discarded – undo
73 73
 		foreach( $this->rules as $attribute => $rules ) {
74 74
 			foreach( $rules as $rule ) {
75 75
 				$this->validateAttribute( $attribute, $rule );
76
-				if( $this->shouldStopValidating( $attribute ))break;
76
+				if( $this->shouldStopValidating( $attribute ) )break;
77 77
 			}
78 78
 		}
79 79
 		return $this->errors;
@@ -88,13 +88,13 @@  discard block
 block discarded – undo
88 88
 	 */
89 89
 	public function validateAttribute( $attribute, $rule )
90 90
 	{
91
-		list( $rule, $parameters ) = $this->parseRule( $rule );
91
+		list($rule, $parameters) = $this->parseRule( $rule );
92 92
 		if( $rule == '' )return;
93 93
 		$value = $this->getValue( $attribute );
94
-		if( !method_exists( $this, $method = 'validate'.$rule )) {
94
+		if( !method_exists( $this, $method = 'validate'.$rule ) ) {
95 95
 			throw new BadMethodCallException( "Method [$method] does not exist." );
96 96
 		}
97
-		if( !$this->$method( $attribute, $value, $parameters )) {
97
+		if( !$this->$method( $attribute, $value, $parameters ) ) {
98 98
 			$this->addFailure( $attribute, $rule, $parameters );
99 99
 		}
100 100
 	}
@@ -143,7 +143,7 @@  discard block
 block discarded – undo
143 143
 	 */
144 144
 	protected function getMessage( $attribute, $rule, array $parameters )
145 145
 	{
146
-		if( in_array( $rule, $this->sizeRules )) {
146
+		if( in_array( $rule, $this->sizeRules ) ) {
147 147
 			return $this->getSizeMessage( $attribute, $rule, $parameters );
148 148
 		}
149 149
 		$lowerRule = glsr( Helper::class )->snakeCase( $rule );
@@ -158,11 +158,11 @@  discard block
 block discarded – undo
158 158
 	 */
159 159
 	protected function getRule( $attribute, $rules )
160 160
 	{
161
-		if( !array_key_exists( $attribute, $this->rules ))return;
162
-		$rules = (array) $rules;
161
+		if( !array_key_exists( $attribute, $this->rules ) )return;
162
+		$rules = (array)$rules;
163 163
 		foreach( $this->rules[$attribute] as $rule ) {
164
-			list( $rule, $parameters ) = $this->parseRule( $rule );
165
-			if( in_array( $rule, $rules )) {
164
+			list($rule, $parameters) = $this->parseRule( $rule );
165
+			if( in_array( $rule, $rules ) ) {
166 166
 				return [$rule, $parameters];
167 167
 			}
168 168
 		}
@@ -180,7 +180,7 @@  discard block
 block discarded – undo
180 180
 		if( is_numeric( $value ) && $hasNumeric ) {
181 181
 			return $value;
182 182
 		}
183
-		elseif( is_array( $value )) {
183
+		elseif( is_array( $value ) ) {
184 184
 			return count( $value );
185 185
 		}
186 186
 		return mb_strlen( $value );
@@ -207,7 +207,7 @@  discard block
 block discarded – undo
207 207
 	 */
208 208
 	protected function getValue( $attribute )
209 209
 	{
210
-		if( isset( $this->data[$attribute] )) {
210
+		if( isset($this->data[$attribute]) ) {
211 211
 			return $this->data[$attribute];
212 212
 		}
213 213
 	}
@@ -220,7 +220,7 @@  discard block
 block discarded – undo
220 220
 	 */
221 221
 	protected function hasRule( $attribute, $rules )
222 222
 	{
223
-		return !is_null( $this->getRule( $attribute, $rules ));
223
+		return !is_null( $this->getRule( $attribute, $rules ) );
224 224
 	}
225 225
 
226 226
 	/**
@@ -259,11 +259,11 @@  discard block
 block discarded – undo
259 259
 		$parameters = [];
260 260
 		// example: {rule}:{parameters}
261 261
 		if( strpos( $rule, ':' ) !== false ) {
262
-			list( $rule, $parameter ) = explode( ':', $rule, 2 );
262
+			list($rule, $parameter) = explode( ':', $rule, 2 );
263 263
 			// example: {parameter1,parameter2,...}
264 264
 			$parameters = $this->parseParameters( $rule, $parameter );
265 265
 		}
266
-		$rule = ucwords( str_replace( ['-', '_'], ' ', trim( $rule )));
266
+		$rule = ucwords( str_replace( ['-', '_'], ' ', trim( $rule ) ) );
267 267
 		$rule = str_replace( ' ', '', $rule );
268 268
 		return [$rule, $parameters];
269 269
 	}
@@ -290,7 +290,7 @@  discard block
 block discarded – undo
290 290
 	protected function shouldStopValidating( $attribute )
291 291
 	{
292 292
 		return $this->hasRule( $attribute, $this->implicitRules )
293
-			&& isset( $this->failedRules[$attribute] )
293
+			&& isset($this->failedRules[$attribute])
294 294
 			&& array_intersect( array_keys( $this->failedRules[$attribute] ), $this->implicitRules );
295 295
 	}
296 296
 
@@ -315,11 +315,11 @@  discard block
 block discarded – undo
315 315
 			'regex' => __( 'The format is invalid.', 'site-reviews' ),
316 316
 			'required' => __( 'This field is required.', 'site-reviews' ),
317 317
 		];
318
-		$message = isset( $strings[$key] )
318
+		$message = isset($strings[$key])
319 319
 			? $strings[$key]
320 320
 			: false;
321 321
 		if( !$message )return;
322
-		if( method_exists( $this, $method = 'replace'.$rule )) {
322
+		if( method_exists( $this, $method = 'replace'.$rule ) ) {
323 323
 			$message = $this->$method( $message, $parameters );
324 324
 		}
325 325
 		return $message;
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( 'referrer' );
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/Modules/Html/Builder.php 1 patch
Spacing   +28 added lines, -28 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,13 +141,13 @@  discard block
 block discarded – undo
141 141
 	 */
142 142
 	protected function buildFormInput()
143 143
 	{
144
-		if( !in_array( $this->args['type'], ['checkbox', 'radio'] )) {
145
-			if( isset( $this->args['multiple'] )) {
146
-				$this->args['name'].= '[]';
144
+		if( !in_array( $this->args['type'], ['checkbox', 'radio'] ) ) {
145
+			if( isset($this->args['multiple']) ) {
146
+				$this->args['name'] .= '[]';
147 147
 			}
148 148
 			return $this->buildFormLabel().$this->getOpeningTag();
149 149
 		}
150
-		return empty( $this->args['options'] )
150
+		return empty($this->args['options'])
151 151
 			? $this->buildFormInputChoice()
152 152
 			: $this->buildFormInputMultiChoice();
153 153
 	}
@@ -157,13 +157,13 @@  discard block
 block discarded – undo
157 157
 	 */
158 158
 	protected function buildFormInputChoice()
159 159
 	{
160
-		$labelText = !empty( $this->args['text'] )
160
+		$labelText = !empty($this->args['text'])
161 161
 			? $this->args['text']
162 162
 			: $this->args['label'];
163 163
 		return $this->label( $this->getOpeningTag().' '.$labelText, [
164 164
 			'class' => 'glsr-'.$this->args['type'].'-label',
165 165
 			'for' => $this->args['id'],
166
-		]);
166
+		] );
167 167
 	}
168 168
 
169 169
 	/**
@@ -172,7 +172,7 @@  discard block
 block discarded – undo
172 172
 	protected function buildFormInputMultiChoice()
173 173
 	{
174 174
 		if( $this->args['type'] == 'checkbox' ) {
175
-			$this->args['name'].= '[]';
175
+			$this->args['name'] .= '[]';
176 176
 		}
177 177
 		$options = array_reduce( array_keys( $this->args['options'] ), function( $carry, $key ) {
178 178
 			return $carry.$this->li( $this->{$this->args['type']}([
@@ -180,12 +180,12 @@  discard block
 block discarded – undo
180 180
 				'name' => $this->args['name'],
181 181
 				'text' => $this->args['options'][$key],
182 182
 				'value' => $key,
183
-			]));
183
+			]) );
184 184
 		});
185 185
 		return $this->ul( $options, [
186 186
 			'class' => $this->args['class'],
187 187
 			'id' => $this->args['id'],
188
-		]);
188
+		] );
189 189
 	}
190 190
 
191 191
 	/**
@@ -193,11 +193,11 @@  discard block
 block discarded – undo
193 193
 	 */
194 194
 	protected function buildFormLabel()
195 195
 	{
196
-		if( empty( $this->args['label'] ) || $this->args['type'] == 'hidden' )return;
197
-		return $this->label([
196
+		if( empty($this->args['label']) || $this->args['type'] == 'hidden' )return;
197
+		return $this->label( [
198 198
 			'for' => $this->args['id'],
199 199
 			'text' => $this->args['label'],
200
-		]);
200
+		] );
201 201
 	}
202 202
 
203 203
 	/**
@@ -214,11 +214,11 @@  discard block
 block discarded – undo
214 214
 	protected function buildFormSelectOptions()
215 215
 	{
216 216
 		return array_reduce( array_keys( $this->args['options'] ), function( $carry, $key ) {
217
-			return $carry.$this->option([
217
+			return $carry.$this->option( [
218 218
 				'selected' => $this->args['value'] == $key,
219 219
 				'text' => $this->args['options'][$key],
220 220
 				'value' => $key,
221
-			]);
221
+			] );
222 222
 		});
223 223
 	}
224 224
 
@@ -236,10 +236,10 @@  discard block
 block discarded – undo
236 236
 	protected function buildTag()
237 237
 	{
238 238
 		$this->mergeArgsWithRequiredDefaults();
239
-		if( in_array( $this->tag, static::TAGS_SINGLE )) {
239
+		if( in_array( $this->tag, static::TAGS_SINGLE ) ) {
240 240
 			return $this->getOpeningTag();
241 241
 		}
242
-		if( !in_array( $this->tag, static::TAGS_FORM )) {
242
+		if( !in_array( $this->tag, static::TAGS_FORM ) ) {
243 243
 			return $this->buildDefaultTag();
244 244
 		}
245 245
 		return call_user_func( [$this, 'buildForm'.ucfirst( $this->tag )] ).$this->buildFieldDescription();
@@ -259,7 +259,7 @@  discard block
 block discarded – undo
259 259
 	protected function mergeArgsWithRequiredDefaults()
260 260
 	{
261 261
 		$className = $this->getCustomFieldClassName();
262
-		if( class_exists( $className )) {
262
+		if( class_exists( $className ) ) {
263 263
 			$this->args = array_merge(
264 264
 				wp_parse_args( $this->args, $className::defaults() ),
265 265
 				$className::required()
@@ -274,13 +274,13 @@  discard block
 block discarded – undo
274 274
 	 */
275 275
 	protected function normalize( ...$params )
276 276
 	{
277
-		if( is_string( $params[0] ) || is_numeric( $params[0] )) {
277
+		if( is_string( $params[0] ) || is_numeric( $params[0] ) ) {
278 278
 			$this->setNameOrTextAttributeForTag( $params[0] );
279 279
 		}
280
-		if( is_array( $params[0] )) {
280
+		if( is_array( $params[0] ) ) {
281 281
 			$this->args += $params[0];
282 282
 		}
283
-		else if( is_array( $params[1] )) {
283
+		else if( is_array( $params[1] ) ) {
284 284
 			$this->args += $params[1];
285 285
 		}
286 286
 	}
@@ -304,7 +304,7 @@  discard block
 block discarded – undo
304 304
 	protected function setTagFromMethod( $method )
305 305
 	{
306 306
 		$this->tag = strtolower( $method );
307
-		if( in_array( $this->tag, static::INPUT_TYPES )) {
307
+		if( in_array( $this->tag, static::INPUT_TYPES ) ) {
308 308
 			$this->args['type'] = $this->tag;
309 309
 			$this->tag = 'input';
310 310
 		}
Please login to merge, or discard this patch.
plugin/Modules/Html/Partials/Pagination.php 1 patch
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -22,11 +22,11 @@  discard block
 block discarded – undo
22 22
 		$this->args = $this->normalize( $args );
23 23
 		if( $this->args['total'] < 2 )return;
24 24
 		$links = $this->buildLinksForDeprecatedThemes();
25
-		if( empty( $links )) {
25
+		if( empty($links) ) {
26 26
 			$links = $this->buildLinks();
27 27
 		}
28 28
 		$links = apply_filters( 'site-reviews/reviews/navigation_links', $links, $this->args );
29
-		if( empty( $links ))return;
29
+		if( empty($links) )return;
30 30
 		return $this->buildTemplate( $links );
31 31
 	}
32 32
 
@@ -45,7 +45,7 @@  discard block
 block discarded – undo
45 45
 			'total' => $this->args['total'],
46 46
 		];
47 47
 		if( is_front_page() ) {
48
-			unset( $paginateArgs['format'] );
48
+			unset($paginateArgs['format']);
49 49
 		}
50 50
 		return paginate_links( $paginateArgs );
51 51
 	}
@@ -56,17 +56,17 @@  discard block
 block discarded – undo
56 56
 	protected function buildLinksForDeprecatedThemes()
57 57
 	{
58 58
 		$theme = wp_get_theme()->get( 'TextDomain' );
59
-		if( !in_array( $theme, ['twentyten','twentyeleven','twentytwelve','twentythirteen'] ))return;
59
+		if( !in_array( $theme, ['twentyten', 'twentyeleven', 'twentytwelve', 'twentythirteen'] ) )return;
60 60
 		$links = '';
61 61
 		if( $this->args['paged'] > 1 ) {
62
-			$links.= sprintf( '<div class="nav-previous"><a href="%s"><span class="meta-nav">&larr;</span> %s</a></div>',
62
+			$links .= sprintf( '<div class="nav-previous"><a href="%s"><span class="meta-nav">&larr;</span> %s</a></div>',
63 63
 				$this->buildUrlForDeprecatedThemes(-1),
64 64
 				__( 'Previous', 'site-reviews' )
65 65
 			);
66 66
 		}
67 67
 		if( $this->args['paged'] < $this->args['total'] ) {
68
-			$links.= sprintf( '<div class="nav-next"><a href="%s">%s <span class="meta-nav">&rarr;</span></a></div>',
69
-				$this->buildUrlForDeprecatedThemes(1),
68
+			$links .= sprintf( '<div class="nav-next"><a href="%s">%s <span class="meta-nav">&rarr;</span></a></div>',
69
+				$this->buildUrlForDeprecatedThemes( 1 ),
70 70
 				__( 'Next', 'site-reviews' )
71 71
 			);
72 72
 		}
@@ -84,7 +84,7 @@  discard block
 block discarded – undo
84 84
 		$screenReaderTemplate = '<h2 class="screen-reader-text">%2$s</h2>';
85 85
 		$screenReaderText = __( 'Site Reviews navigation', 'site-reviews' );
86 86
 		$innerTemplate = $screenReaderTemplate.'<div class="nav-links">%3$s</div>';
87
-		if( in_array( $theme, ['twentyten', 'twentyeleven', 'twentytwelve'] )) {
87
+		if( in_array( $theme, ['twentyten', 'twentyeleven', 'twentytwelve'] ) ) {
88 88
 			$innerTemplate = '%3$s';
89 89
 		}
90 90
 		else if( $theme == 'twentyfourteen' ) {
@@ -96,7 +96,7 @@  discard block
 block discarded – undo
96 96
 		$template = sprintf( $template, $class, $screenReaderText, $links );
97 97
 		return glsr( Builder::class )->div( $template.'<div class="glsr-loader"></div>', [
98 98
 			'class' => 'glsr-navigation',
99
-		]);
99
+		] );
100 100
 	}
101 101
 
102 102
 	/**
@@ -119,6 +119,6 @@  discard block
 block discarded – undo
119 119
 		return wp_parse_args( $args, [
120 120
 			'paged' => glsr( QueryBuilder::class )->getPaged(),
121 121
 			'total' => 1,
122
-		]);
122
+		] );
123 123
 	}
124 124
 }
Please login to merge, or discard this patch.
views/partials/editor/metabox-categories.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -16,11 +16,11 @@  discard block
 block discarded – undo
16 16
 	<div id="<?= $tax_name; ?>-all" class="tabs-panel">
17 17
 		<input type="hidden" name="tax_input[<?= $tax_name; ?>][]" value='0' />
18 18
 		<ul id="<?= $tax_name; ?>checklist" data-wp-lists="list:<?= $tax_name; ?>" class="categorychecklist form-no-clear">
19
-			<?php wp_terms_checklist( $post->ID, array( 'taxonomy' => $tax_name, 'popular_cats' => $popular_ids )); ?>
19
+			<?php wp_terms_checklist( $post->ID, array( 'taxonomy' => $tax_name, 'popular_cats' => $popular_ids ) ); ?>
20 20
 		</ul>
21 21
 	</div>
22 22
 
23
-	<?php if( current_user_can( $taxonomy->cap->edit_terms )) : ?>
23
+	<?php if( current_user_can( $taxonomy->cap->edit_terms ) ) : ?>
24 24
 	<div id="<?= $tax_name; ?>-adder" class="wp-hidden-children">
25 25
 		<a id="<?= $tax_name; ?>-add-toggle" href="#<?= $tax_name; ?>-add" class="hide-if-no-js taxonomy-add-new">
26 26
 			<?= sprintf( '+ %s', $taxonomy->labels->add_new_item ); ?>
@@ -29,7 +29,7 @@  discard block
 block discarded – undo
29 29
 			<label class="screen-reader-text" for="new<?= $tax_name; ?>"><?= $taxonomy->labels->add_new_item; ?></label>
30 30
 			<input type="text" name="new<?= $tax_name; ?>" id="new<?= $tax_name; ?>" class="form-required form-input-tip" value="<?= esc_attr( $taxonomy->labels->new_item_name ); ?>" aria-required="true"/>
31 31
 			<input type="button" id="<?= $tax_name; ?>-add-submit" data-wp-lists="add:<?= $tax_name; ?>checklist:<?= $tax_name; ?>-add" class="button category-add-submit" value="<?= esc_attr( $taxonomy->labels->add_new_item ); ?>" />
32
-			<?php wp_nonce_field( 'add-' . $tax_name, '_ajax_nonce-add-' . $tax_name, false ); ?>
32
+			<?php wp_nonce_field( 'add-'.$tax_name, '_ajax_nonce-add-'.$tax_name, false ); ?>
33 33
 			<span id="<?= $tax_name; ?>-ajax-response"></span>
34 34
 		</div>
35 35
 	</div>
Please login to merge, or discard this patch.
views/partials/editor/review.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -1,7 +1,7 @@
 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">
Please login to merge, or discard this patch.
plugin/Handlers/CreateReview.php 1 patch
Spacing   +11 added lines, -11 removed lines patch added patch discarded remove patch
@@ -31,7 +31,7 @@  discard block
 block discarded – undo
31 31
 		}
32 32
 		$this->sendNotification( $postId );
33 33
 		do_action( 'site-reviews/local/review/submitted', $postId, $command );
34
-		glsr( Session::class )->set( $command->form_id.'message', __( 'Your review has been submitted!', 'site-reviews' ));
34
+		glsr( Session::class )->set( $command->form_id.'message', __( 'Your review has been submitted!', 'site-reviews' ) );
35 35
 		if( $command->ajax_request ) {
36 36
 			glsr( Session::class )->clear();
37 37
 			return;
@@ -81,8 +81,8 @@  discard block
 block discarded – undo
81 81
 		if( $this->command->author ) {
82 82
 			$fields[] = ['value' => trim( $this->command->author.$this->command->email.' - '.$this->command->ip_address )];
83 83
 		}
84
-		$fields[] = ['value' => sprintf( '<%s|%s>', $args['notification_link'], __( 'View Review', 'site-reviews' ))];
85
-		return json_encode([
84
+		$fields[] = ['value' => sprintf( '<%s|%s>', $args['notification_link'], __( 'View Review', 'site-reviews' ) )];
85
+		return json_encode( [
86 86
 			'icon_url' => glsr()->url( 'assets/img/icon.png' ),
87 87
 			'username' => glsr()->name,
88 88
 			'attachments' => [[
@@ -91,7 +91,7 @@  discard block
 block discarded – undo
91 91
 				'fallback' => $this->createEmailNotification( $args )->read( 'plaintext' ),
92 92
 				'fields' => $fields,
93 93
 			]],
94
-		]);
94
+		] );
95 95
 	}
96 96
 
97 97
 	/**
@@ -101,12 +101,12 @@  discard block
 block discarded – undo
101 101
 	protected function sendNotification( $postId )
102 102
 	{
103 103
 		$notificationType = glsr( OptionManager::class )->get( 'settings.general.notification' );
104
-		if( !in_array( $notificationType, ['default','custom','webhook'] ))return;
104
+		if( !in_array( $notificationType, ['default', 'custom', 'webhook'] ) )return;
105 105
 		$assignedToTitle = get_the_title( (int)$this->command->assigned_to );
106 106
 		$notificationSubject = _nx(
107 107
 			'New %s-star review',
108 108
 			'New %s-star review of: %s',
109
-			intval( empty( $assignedToTitle )),
109
+			intval( empty($assignedToTitle) ),
110 110
 			'The text is different depending on whether or not the review has been assigned to a post.',
111 111
 			'site-reviews'
112 112
 		);
@@ -115,7 +115,7 @@  discard block
 block discarded – undo
115 115
 			sprintf( $notificationSubject, $this->command->rating, $assignedToTitle )
116 116
 		);
117 117
 		$args = [
118
-			'notification_link' => esc_url( admin_url( sprintf( 'post.php?post=%s&action=edit', $postId ))),
118
+			'notification_link' => esc_url( admin_url( sprintf( 'post.php?post=%s&action=edit', $postId ) ) ),
119 119
 			'notification_title' => $notificationTitle,
120 120
 			'notification_type' => $notificationType,
121 121
 		];
@@ -133,7 +133,7 @@  discard block
 block discarded – undo
133 133
 		$args['recipient'] = $args['notification_type'] !== 'default'
134 134
 			? glsr( OptionManager::class )->get( 'settings.general.notification_email' )
135 135
 			: get_option( 'admin_email' );
136
-		if( empty( $args['recipient'] )) {
136
+		if( empty($args['recipient']) ) {
137 137
 			glsr_log()->error( 'Email notification was not sent: missing email, subject, or message.' );
138 138
 		}
139 139
 		else {
@@ -149,7 +149,7 @@  discard block
 block discarded – undo
149 149
 	 */
150 150
 	protected function sendWebhookNotification( array $args )
151 151
 	{
152
-		if( !( $endpoint = glsr( OptionManager::class )->get( 'settings.general.webhook_url' )))return;
152
+		if( !($endpoint = glsr( OptionManager::class )->get( 'settings.general.webhook_url' )) )return;
153 153
 		$notification = $this->createWebhookNotification( $args );
154 154
 		$result = wp_remote_post( $endpoint, [
155 155
 			'blocking' => false,
@@ -160,8 +160,8 @@  discard block
 block discarded – undo
160 160
 			'redirection' => 5,
161 161
 			'sslverify' => false,
162 162
 			'timeout' => 45,
163
-		]);
164
-		if( is_wp_error( $result )) {
163
+		] );
164
+		if( is_wp_error( $result ) ) {
165 165
 			glsr_log()->error( $result->get_error_message() );
166 166
 		}
167 167
 	}
Please login to merge, or discard this patch.
plugin/Handlers/EnqueuePublicAssets.php 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -23,7 +23,7 @@  discard block
 block discarded – undo
23 23
 	 */
24 24
 	public function enqueueAssets()
25 25
 	{
26
-		if( apply_filters( 'site-reviews/assets/css', true )) {
26
+		if( apply_filters( 'site-reviews/assets/css', true ) ) {
27 27
 			wp_enqueue_style(
28 28
 				Application::ID,
29 29
 				$this->getStylesheet(),
@@ -31,7 +31,7 @@  discard block
 block discarded – undo
31 31
 				glsr()->version
32 32
 			);
33 33
 		}
34
-		if( apply_filters( 'site-reviews/assets/js', true )) {
34
+		if( apply_filters( 'site-reviews/assets/js', true ) ) {
35 35
 			$dependencies = apply_filters( 'site-reviews/enqueue/public/dependencies', [] );
36 36
 			wp_enqueue_script(
37 37
 				Application::ID,
@@ -50,12 +50,12 @@  discard block
 block discarded – undo
50 50
 	{
51 51
 		if( glsr( OptionManager::class )->get( 'settings.submissions.recaptcha.integration' ) != 'custom' )return;
52 52
 		$language = apply_filters( 'site-reviews/recaptcha/language', get_locale() );
53
-		wp_enqueue_script( Application::ID.'/google-recaptcha', add_query_arg([
53
+		wp_enqueue_script( Application::ID.'/google-recaptcha', add_query_arg( [
54 54
 			'hl' => $language,
55 55
 			'onload' => 'glsr_render_recaptcha',
56 56
 			'render' => 'explicit',
57
-		], 'https://www.google.com/recaptcha/api.js' ));
58
-		$inlineScript = file_get_contents( glsr()->path( 'assets/scripts/recaptcha.js' ));
57
+		], 'https://www.google.com/recaptcha/api.js' ) );
58
+		$inlineScript = file_get_contents( glsr()->path( 'assets/scripts/recaptcha.js' ) );
59 59
 		wp_add_inline_script( Application::ID.'/google-recaptcha', $inlineScript, 'before' );
60 60
 	}
61 61
 
@@ -66,7 +66,7 @@  discard block
 block discarded – undo
66 66
 	{
67 67
 		$variables = [
68 68
 			'action' => Application::PREFIX.'action',
69
-			'ajaxpagination' => ['#wpadminbar','.site-navigation-fixed'],
69
+			'ajaxpagination' => ['#wpadminbar', '.site-navigation-fixed'],
70 70
 			'ajaxurl' => admin_url( 'admin-ajax.php' ),
71 71
 		];
72 72
 		$variables = apply_filters( 'site-reviews/enqueue/public/localize', $variables );
@@ -78,8 +78,8 @@  discard block
 block discarded – undo
78 78
 	 */
79 79
 	protected function getStylesheet()
80 80
 	{
81
-		$currentTheme = sanitize_title( (string)wp_get_theme()->get( 'Name' ));
82
-		return file_exists( glsr()->path( 'assets/styles/themes/'.$currentTheme.'.css' ))
81
+		$currentTheme = sanitize_title( (string)wp_get_theme()->get( 'Name' ) );
82
+		return file_exists( glsr()->path( 'assets/styles/themes/'.$currentTheme.'.css' ) )
83 83
 			? glsr()->url( 'assets/styles/themes/'.$currentTheme.'.css' )
84 84
 			: glsr()->url( 'assets/styles/'.Application::ID.'.css' );
85 85
 	}
Please login to merge, or discard this patch.