Passed
Push — master ( 156e0e...dfcc60 )
by Paul
04:35
created
plugin/Modules/Html/Partials/SiteReviewsSummary.php 2 patches
Spacing   +21 added lines, -21 removed lines patch added patch discarded remove patch
@@ -33,14 +33,14 @@  discard block
 block discarded – undo
33 33
 	public function build( array $args = [] )
34 34
 	{
35 35
 		$this->args = $args;
36
-		$counts = glsr( CountsManager::class )->get([
37
-			'post_ids' => array_filter( array_map( 'trim', explode( ',', $args['assigned_to'] ))),
38
-			'term_ids' => array_filter( array_map( 'trim', explode( ',', $args['category'] ))),
39
-		]);
36
+		$counts = glsr( CountsManager::class )->get( [
37
+			'post_ids' => array_filter( array_map( 'trim', explode( ',', $args['assigned_to'] ) ) ),
38
+			'term_ids' => array_filter( array_map( 'trim', explode( ',', $args['category'] ) ) ),
39
+		] );
40 40
 		$this->ratingCounts = glsr( CountsManager::class )->flatten( $counts, [
41 41
 			'min' => $args['rating'],
42
-		]);
43
-		if( !array_sum( $this->ratingCounts ) && $this->isHidden( 'if_empty' ))return;
42
+		] );
43
+		if( !array_sum( $this->ratingCounts ) && $this->isHidden( 'if_empty' ) )return;
44 44
 		$this->averageRating = glsr( Rating::class )->getAverage( $this->ratingCounts );
45 45
 		$this->generateSchema();
46 46
 		return glsr( Template::class )->build( 'templates/reviews-summary', [
@@ -52,7 +52,7 @@  discard block
 block discarded – undo
52 52
 				'stars' => $this->buildStars(),
53 53
 				'text' => $this->buildText(),
54 54
 			],
55
-		]);
55
+		] );
56 56
 	}
57 57
 
58 58
 	/**
@@ -60,16 +60,16 @@  discard block
 block discarded – undo
60 60
 	 */
61 61
 	protected function buildPercentage()
62 62
 	{
63
-		if( $this->isHidden( 'bars' ))return;
63
+		if( $this->isHidden( 'bars' ) )return;
64 64
 		$range = range( Rating::MAX_RATING, 1 );
65
-		$percentages = preg_filter( '/$/', '%', glsr( Rating::class )->getPercentages( $this->ratingCounts ));
66
-		$bars = array_reduce( $range, function( $carry, $level ) use( $percentages ) {
65
+		$percentages = preg_filter( '/$/', '%', glsr( Rating::class )->getPercentages( $this->ratingCounts ) );
66
+		$bars = array_reduce( $range, function( $carry, $level ) use($percentages) {
67 67
 			$label = $this->buildPercentageLabel( $this->args['labels'][$level] );
68 68
 			$background = $this->buildPercentageBackground( $percentages[$level] );
69 69
 			$percent = $this->buildPercentagePercent( $percentages[$level] );
70 70
 			return $carry.glsr( Builder::class )->div( $label.$background.$percent, [
71 71
 				'class' => 'glsr-bar',
72
-			]);
72
+			] );
73 73
 		});
74 74
 		return $this->wrap( 'percentage', $bars );
75 75
 	}
@@ -80,10 +80,10 @@  discard block
 block discarded – undo
80 80
 	 */
81 81
 	protected function buildPercentageBackground( $percent )
82 82
 	{
83
-		$backgroundPercent = glsr( Builder::class )->span([
83
+		$backgroundPercent = glsr( Builder::class )->span( [
84 84
 			'class' => 'glsr-bar-background-percent',
85 85
 			'style' => 'width:'.$percent,
86
-		]);
86
+		] );
87 87
 		return '<span class="glsr-bar-background">'.$backgroundPercent.'</span>';
88 88
 	}
89 89
 
@@ -110,7 +110,7 @@  discard block
 block discarded – undo
110 110
 	 */
111 111
 	protected function buildRating()
112 112
 	{
113
-		if( $this->isHidden( 'rating' ))return;
113
+		if( $this->isHidden( 'rating' ) )return;
114 114
 		return $this->wrap( 'rating', '<span>'.$this->averageRating.'</span>' );
115 115
 	}
116 116
 
@@ -119,7 +119,7 @@  discard block
 block discarded – undo
119 119
 	 */
120 120
 	protected function buildStars()
121 121
 	{
122
-		if( $this->isHidden( 'stars' ))return;
122
+		if( $this->isHidden( 'stars' ) )return;
123 123
 		$stars = glsr( Partial::class )->build( 'star-rating', ['rating' => $this->averageRating] );
124 124
 		return $this->wrap( 'stars', $stars );
125 125
 	}
@@ -129,9 +129,9 @@  discard block
 block discarded – undo
129 129
 	 */
130 130
 	protected function buildText()
131 131
 	{
132
-		if( $this->isHidden( 'summary' ))return;
133
-		$count = intval( array_sum( $this->ratingCounts ));
134
-		if( empty( $this->args['text'] )) {
132
+		if( $this->isHidden( 'summary' ) )return;
133
+		$count = intval( array_sum( $this->ratingCounts ) );
134
+		if( empty($this->args['text']) ) {
135 135
 			// @todo document this change
136 136
 			 $this->args['text'] = _nx(
137 137
 				'{rating} out of {max} stars (based on {num} review)',
@@ -142,7 +142,7 @@  discard block
 block discarded – undo
142 142
 			);
143 143
 		}
144 144
 		$summary = str_replace(
145
-			['{rating}','{max}', '{num}'],
145
+			['{rating}', '{max}', '{num}'],
146 146
 			[$this->averageRating, Rating::MAX_RATING, $count],
147 147
 			$this->args['text']
148 148
 		);
@@ -154,7 +154,7 @@  discard block
 block discarded – undo
154 154
 	 */
155 155
 	protected function generateSchema()
156 156
 	{
157
-		if( !wp_validate_boolean( $this->args['schema'] ))return;
157
+		if( !wp_validate_boolean( $this->args['schema'] ) )return;
158 158
 		glsr( Schema::class )->store(
159 159
 			glsr( Schema::class )->buildSummary( $this->args )
160 160
 		);
@@ -186,6 +186,6 @@  discard block
 block discarded – undo
186 186
 	{
187 187
 		return glsr( Builder::class )->div( $value, [
188 188
 			'class' => 'glsr-summary-'.$key,
189
-		]);
189
+		] );
190 190
 	}
191 191
 }
Please login to merge, or discard this patch.
Braces   +18 added lines, -6 removed lines patch added patch discarded remove patch
@@ -40,7 +40,9 @@  discard block
 block discarded – undo
40 40
 		$this->ratingCounts = glsr( CountsManager::class )->flatten( $counts, [
41 41
 			'min' => $args['rating'],
42 42
 		]);
43
-		if( !array_sum( $this->ratingCounts ) && $this->isHidden( 'if_empty' ))return;
43
+		if( !array_sum( $this->ratingCounts ) && $this->isHidden( 'if_empty' )) {
44
+			return;
45
+		}
44 46
 		$this->averageRating = glsr( Rating::class )->getAverage( $this->ratingCounts );
45 47
 		$this->generateSchema();
46 48
 		return glsr( Template::class )->build( 'templates/reviews-summary', [
@@ -60,7 +62,9 @@  discard block
 block discarded – undo
60 62
 	 */
61 63
 	protected function buildPercentage()
62 64
 	{
63
-		if( $this->isHidden( 'bars' ))return;
65
+		if( $this->isHidden( 'bars' )) {
66
+			return;
67
+		}
64 68
 		$range = range( Rating::MAX_RATING, 1 );
65 69
 		$percentages = preg_filter( '/$/', '%', glsr( Rating::class )->getPercentages( $this->ratingCounts ));
66 70
 		$bars = array_reduce( $range, function( $carry, $level ) use( $percentages ) {
@@ -110,7 +114,9 @@  discard block
 block discarded – undo
110 114
 	 */
111 115
 	protected function buildRating()
112 116
 	{
113
-		if( $this->isHidden( 'rating' ))return;
117
+		if( $this->isHidden( 'rating' )) {
118
+			return;
119
+		}
114 120
 		return $this->wrap( 'rating', '<span>'.$this->averageRating.'</span>' );
115 121
 	}
116 122
 
@@ -119,7 +125,9 @@  discard block
 block discarded – undo
119 125
 	 */
120 126
 	protected function buildStars()
121 127
 	{
122
-		if( $this->isHidden( 'stars' ))return;
128
+		if( $this->isHidden( 'stars' )) {
129
+			return;
130
+		}
123 131
 		$stars = glsr( Partial::class )->build( 'star-rating', ['rating' => $this->averageRating] );
124 132
 		return $this->wrap( 'stars', $stars );
125 133
 	}
@@ -129,7 +137,9 @@  discard block
 block discarded – undo
129 137
 	 */
130 138
 	protected function buildText()
131 139
 	{
132
-		if( $this->isHidden( 'summary' ))return;
140
+		if( $this->isHidden( 'summary' )) {
141
+			return;
142
+		}
133 143
 		$count = intval( array_sum( $this->ratingCounts ));
134 144
 		if( empty( $this->args['text'] )) {
135 145
 			// @todo document this change
@@ -154,7 +164,9 @@  discard block
 block discarded – undo
154 164
 	 */
155 165
 	protected function generateSchema()
156 166
 	{
157
-		if( !wp_validate_boolean( $this->args['schema'] ))return;
167
+		if( !wp_validate_boolean( $this->args['schema'] )) {
168
+			return;
169
+		}
158 170
 		glsr( Schema::class )->store(
159 171
 			glsr( Schema::class )->buildSummary( $this->args )
160 172
 		);
Please login to merge, or discard this patch.
plugin/Modules/Style.php 2 patches
Braces   +6 added lines, -2 removed lines patch added patch discarded remove patch
@@ -47,7 +47,9 @@  discard block
 block discarded – undo
47 47
 		}
48 48
 		$views = $this->generatePossibleViews( $view );
49 49
 		foreach( $views as $possibleView ) {
50
-			if( !file_exists( glsr()->path( 'views/'.$possibleView.'.php' )))continue;
50
+			if( !file_exists( glsr()->path( 'views/'.$possibleView.'.php' ))) {
51
+				continue;
52
+			}
51 53
 			return $possibleView;
52 54
 		}
53 55
 		return $view;
@@ -84,7 +86,9 @@  discard block
 block discarded – undo
84 86
 	 */
85 87
 	public function modifyField( Builder $instance )
86 88
 	{
87
-		if( !$this->isPublicInstance( $instance ) || empty( array_filter( $this->fields )))return;
89
+		if( !$this->isPublicInstance( $instance ) || empty( array_filter( $this->fields ))) {
90
+			return;
91
+		}
88 92
 		call_user_func_array( [$this, 'customize'], [&$instance] );
89 93
 	}
90 94
 
Please login to merge, or discard this patch.
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -44,12 +44,12 @@  discard block
 block discarded – undo
44 44
 			'templates/form/submit-button',
45 45
 			'templates/reviews-form',
46 46
 		];
47
-		if( !preg_match( '('.implode( '|', $styledViews ).')', $view )) {
47
+		if( !preg_match( '('.implode( '|', $styledViews ).')', $view ) ) {
48 48
 			return $view;
49 49
 		}
50 50
 		$views = $this->generatePossibleViews( $view );
51 51
 		foreach( $views as $possibleView ) {
52
-			if( !file_exists( glsr()->path( 'views/'.$possibleView.'.php' )))continue;
52
+			if( !file_exists( glsr()->path( 'views/'.$possibleView.'.php' ) ) )continue;
53 53
 			return $possibleView;
54 54
 		}
55 55
 		return $view;
@@ -81,7 +81,7 @@  discard block
 block discarded – undo
81 81
 	 */
82 82
 	public function modifyField( Builder $instance )
83 83
 	{
84
-		if( !$this->isPublicInstance( $instance ) || empty( array_filter( $this->fields )))return;
84
+		if( !$this->isPublicInstance( $instance ) || empty(array_filter( $this->fields )) )return;
85 85
 		call_user_func_array( [$this, 'customize'], [&$instance] );
86 86
 	}
87 87
 
@@ -90,9 +90,9 @@  discard block
 block discarded – undo
90 90
 	 */
91 91
 	protected function customize( Builder $instance )
92 92
 	{
93
-		$args = wp_parse_args( $instance->args, array_fill_keys( ['class', 'type'], '' ));
93
+		$args = wp_parse_args( $instance->args, array_fill_keys( ['class', 'type'], '' ) );
94 94
 		$key = $instance->tag.'_'.$args['type'];
95
-		$classes = !isset( $this->fields[$key] )
95
+		$classes = !isset($this->fields[$key])
96 96
 			? $this->fields[$instance->tag]
97 97
 			: $this->fields[$key];
98 98
 		$instance->args['class'] = trim( $args['class'].' '.$classes );
@@ -126,7 +126,7 @@  discard block
 block discarded – undo
126 126
 		$args = wp_parse_args( $instance->args, [
127 127
 			'is_public' => false,
128 128
 			'is_raw' => false,
129
-		]);
129
+		] );
130 130
 		if( is_admin() || !$args['is_public'] || $args['is_raw'] ) {
131 131
 			return false;
132 132
 		}
Please login to merge, or discard this patch.
plugin/Modules/Validator.php 2 patches
Braces   +9 added lines, -3 removed lines patch added patch discarded remove patch
@@ -93,7 +93,9 @@  discard block
 block discarded – undo
93 93
 		foreach( $this->rules as $attribute => $rules ) {
94 94
 			foreach( $rules as $rule ) {
95 95
 				$this->validateAttribute( $attribute, $rule );
96
-				if( $this->shouldStopValidating( $attribute ))break;
96
+				if( $this->shouldStopValidating( $attribute )) {
97
+					break;
98
+				}
97 99
 			}
98 100
 		}
99 101
 		return $this->errors;
@@ -109,7 +111,9 @@  discard block
 block discarded – undo
109 111
 	public function validateAttribute( $attribute, $rule )
110 112
 	{
111 113
 		list( $rule, $parameters ) = $this->parseRule( $rule );
112
-		if( $rule == '' )return;
114
+		if( $rule == '' ) {
115
+			return;
116
+		}
113 117
 		$value = $this->getValue( $attribute );
114 118
 		if( !method_exists( $this, $method = 'validate'.$rule )) {
115 119
 			throw new BadMethodCallException( "Method [$method] does not exist." );
@@ -178,7 +182,9 @@  discard block
 block discarded – undo
178 182
 	 */
179 183
 	protected function getRule( $attribute, $rules )
180 184
 	{
181
-		if( !array_key_exists( $attribute, $this->rules ))return;
185
+		if( !array_key_exists( $attribute, $this->rules )) {
186
+			return;
187
+		}
182 188
 		$rules = (array)$rules;
183 189
 		foreach( $this->rules[$attribute] as $rule ) {
184 190
 			list( $rule, $parameters ) = $this->parseRule( $rule );
Please login to merge, or discard this patch.
Spacing   +14 added lines, -14 removed lines patch added patch discarded remove patch
@@ -74,7 +74,7 @@  discard block
 block discarded – undo
74 74
 		foreach( $this->rules as $attribute => $rules ) {
75 75
 			foreach( $rules as $rule ) {
76 76
 				$this->validateAttribute( $attribute, $rule );
77
-				if( $this->shouldStopValidating( $attribute ))break;
77
+				if( $this->shouldStopValidating( $attribute ) )break;
78 78
 			}
79 79
 		}
80 80
 		return $this->errors;
@@ -89,13 +89,13 @@  discard block
 block discarded – undo
89 89
 	 */
90 90
 	public function validateAttribute( $attribute, $rule )
91 91
 	{
92
-		list( $rule, $parameters ) = $this->parseRule( $rule );
92
+		list($rule, $parameters) = $this->parseRule( $rule );
93 93
 		if( $rule == '' )return;
94 94
 		$value = $this->getValue( $attribute );
95
-		if( !method_exists( $this, $method = 'validate'.$rule )) {
95
+		if( !method_exists( $this, $method = 'validate'.$rule ) ) {
96 96
 			throw new BadMethodCallException( "Method [$method] does not exist." );
97 97
 		}
98
-		if( !$this->$method( $value, $attribute, $parameters )) {
98
+		if( !$this->$method( $value, $attribute, $parameters ) ) {
99 99
 			$this->addFailure( $attribute, $rule, $parameters );
100 100
 		}
101 101
 	}
@@ -144,7 +144,7 @@  discard block
 block discarded – undo
144 144
 	 */
145 145
 	protected function getMessage( $attribute, $rule, array $parameters )
146 146
 	{
147
-		if( in_array( $rule, $this->sizeRules )) {
147
+		if( in_array( $rule, $this->sizeRules ) ) {
148 148
 			return $this->getSizeMessage( $attribute, $rule, $parameters );
149 149
 		}
150 150
 		$lowerRule = glsr( Helper::class )->snakeCase( $rule );
@@ -159,11 +159,11 @@  discard block
 block discarded – undo
159 159
 	 */
160 160
 	protected function getRule( $attribute, $rules )
161 161
 	{
162
-		if( !array_key_exists( $attribute, $this->rules ))return;
162
+		if( !array_key_exists( $attribute, $this->rules ) )return;
163 163
 		$rules = (array)$rules;
164 164
 		foreach( $this->rules[$attribute] as $rule ) {
165
-			list( $rule, $parameters ) = $this->parseRule( $rule );
166
-			if( in_array( $rule, $rules )) {
165
+			list($rule, $parameters) = $this->parseRule( $rule );
166
+			if( in_array( $rule, $rules ) ) {
167 167
 				return [$rule, $parameters];
168 168
 			}
169 169
 		}
@@ -181,7 +181,7 @@  discard block
 block discarded – undo
181 181
 		if( is_numeric( $value ) && $hasNumeric ) {
182 182
 			return $value;
183 183
 		}
184
-		elseif( is_array( $value )) {
184
+		elseif( is_array( $value ) ) {
185 185
 			return count( $value );
186 186
 		}
187 187
 		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
 	/**
@@ -258,7 +258,7 @@  discard block
 block discarded – undo
258 258
 	{
259 259
 		$parameters = [];
260 260
 		if( strpos( $rule, ':' ) !== false ) {
261
-			list( $rule, $parameter ) = explode( ':', $rule, 2 );
261
+			list($rule, $parameter) = explode( ':', $rule, 2 );
262 262
 			$parameters = $this->parseParameters( $rule, $parameter );
263 263
 		}
264 264
 		$rule = glsr( Helper::class )->camelCase( $rule );
@@ -287,7 +287,7 @@  discard block
 block discarded – undo
287 287
 	protected function shouldStopValidating( $attribute )
288 288
 	{
289 289
 		return $this->hasRule( $attribute, $this->implicitRules )
290
-			&& isset( $this->failedRules[$attribute] )
290
+			&& isset($this->failedRules[$attribute])
291 291
 			&& array_intersect( array_keys( $this->failedRules[$attribute] ), $this->implicitRules );
292 292
 	}
293 293
 
@@ -300,7 +300,7 @@  discard block
 block discarded – undo
300 300
 	protected function translator( $key, array $parameters )
301 301
 	{
302 302
 		$strings = glsr( ValidationStrings::class )->defaults();
303
-		if( isset( $strings[$key] )) {
303
+		if( isset($strings[$key]) ) {
304 304
 			return $this->replace( $strings[$key], $parameters );
305 305
 		}
306 306
 	}
Please login to merge, or discard this patch.
plugin/Modules/Validator/ValidationRules.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -28,7 +28,7 @@  discard block
 block discarded – undo
28 28
 		if( strpos( $message, '%s' ) === false ) {
29 29
 			return $message;
30 30
 		}
31
-		return preg_replace_callback( '/(%s)/', function() use( &$parameters ) {
31
+		return preg_replace_callback( '/(%s)/', function() use(&$parameters) {
32 32
 			foreach( $parameters as $key => $value ) {
33 33
 				return array_shift( $parameters );
34 34
 			}
@@ -113,8 +113,8 @@  discard block
 block discarded – undo
113 113
 	public function validateRequired( $value )
114 114
 	{
115 115
 		return is_null( $value )
116
-			|| ( is_string( $value ) && trim( $value ) === '' )
117
-			|| ( is_array( $value ) && count( $value ) < 1 )
116
+			|| (is_string( $value ) && trim( $value ) === '')
117
+			|| (is_array( $value ) && count( $value ) < 1)
118 118
 			? false
119 119
 			: true;
120 120
 	}
Please login to merge, or discard this patch.
plugin/Handlers/EnqueuePublicAssets.php 2 patches
Spacing   +11 added lines, -11 removed lines patch added patch discarded remove patch
@@ -27,7 +27,7 @@  discard block
 block discarded – undo
27 27
 	 */
28 28
 	public function enqueueAssets()
29 29
 	{
30
-		if( apply_filters( 'site-reviews/assets/css', true )) {
30
+		if( apply_filters( 'site-reviews/assets/css', true ) ) {
31 31
 			wp_enqueue_style(
32 32
 				Application::ID,
33 33
 				$this->getStylesheet(),
@@ -35,7 +35,7 @@  discard block
 block discarded – undo
35 35
 				glsr()->version
36 36
 			);
37 37
 		}
38
-		if( apply_filters( 'site-reviews/assets/js', true )) {
38
+		if( apply_filters( 'site-reviews/assets/js', true ) ) {
39 39
 			$dependencies = apply_filters( 'site-reviews/assets/polyfill', true )
40 40
 				? [Application::ID.'/polyfill']
41 41
 				: [];
@@ -55,11 +55,11 @@  discard block
 block discarded – undo
55 55
 	 */
56 56
 	public function enqueuePolyfillService()
57 57
 	{
58
-		if( !apply_filters( 'site-reviews/assets/polyfill', true ))return;
59
-		wp_enqueue_script( Application::ID.'/polyfill', add_query_arg([
58
+		if( !apply_filters( 'site-reviews/assets/polyfill', true ) )return;
59
+		wp_enqueue_script( Application::ID.'/polyfill', add_query_arg( [
60 60
 			'features' => 'CustomEvent,Element.prototype.closest,Element.prototype.dataset,Event',
61 61
 			'flags' => 'gated',
62
-		], 'https://cdn.polyfill.io/v2/polyfill.js' ));
62
+		], 'https://cdn.polyfill.io/v2/polyfill.js' ) );
63 63
 	}
64 64
 
65 65
 	/**
@@ -72,10 +72,10 @@  discard block
 block discarded – undo
72 72
 		// nf-google-recaptcha
73 73
 		if( !glsr( OptionManager::class )->isRecaptchaEnabled() )return;
74 74
 		$language = apply_filters( 'site-reviews/recaptcha/language', get_locale() );
75
-		wp_enqueue_script( Application::ID.'/google-recaptcha', add_query_arg([
75
+		wp_enqueue_script( Application::ID.'/google-recaptcha', add_query_arg( [
76 76
 			'hl' => $language,
77 77
 			'render' => 'explicit',
78
-		], 'https://www.google.com/recaptcha/api.js' ));
78
+		], 'https://www.google.com/recaptcha/api.js' ) );
79 79
 	}
80 80
 
81 81
 	/**
@@ -84,8 +84,8 @@  discard block
 block discarded – undo
84 84
 	public function inlineStyles()
85 85
 	{
86 86
 		$inlineStylesheetPath = glsr()->path( 'assets/styles/inline-styles.css' );
87
-		if( !apply_filters( 'site-reviews/assets/css', true ))return;
88
-		if( !file_exists( $inlineStylesheetPath )) {
87
+		if( !apply_filters( 'site-reviews/assets/css', true ) )return;
88
+		if( !file_exists( $inlineStylesheetPath ) ) {
89 89
 			glsr_log()->error( 'Inline stylesheet is missing: '.$inlineStylesheetPath );
90 90
 			return;
91 91
 		}
@@ -120,7 +120,7 @@  discard block
 block discarded – undo
120 120
 	 */
121 121
 	protected function getFixedSelectorsForPagination()
122 122
 	{
123
-		$selectors = ['#wpadminbar','.site-navigation-fixed'];
123
+		$selectors = ['#wpadminbar', '.site-navigation-fixed'];
124 124
 		return apply_filters( 'site-reviews/localize/pagination/selectors', $selectors );
125 125
 	}
126 126
 
@@ -130,7 +130,7 @@  discard block
 block discarded – undo
130 130
 	protected function getStylesheet()
131 131
 	{
132 132
 		$currentStyle = glsr( Style::class )->style;
133
-		return file_exists( glsr()->path( 'assets/styles/custom/'.$currentStyle.'.css' ))
133
+		return file_exists( glsr()->path( 'assets/styles/custom/'.$currentStyle.'.css' ) )
134 134
 			? glsr()->url( 'assets/styles/custom/'.$currentStyle.'.css' )
135 135
 			: glsr()->url( 'assets/styles/'.Application::ID.'.css' );
136 136
 	}
Please login to merge, or discard this patch.
Braces   +9 added lines, -3 removed lines patch added patch discarded remove patch
@@ -55,7 +55,9 @@  discard block
 block discarded – undo
55 55
 	 */
56 56
 	public function enqueuePolyfillService()
57 57
 	{
58
-		if( !apply_filters( 'site-reviews/assets/polyfill', true ))return;
58
+		if( !apply_filters( 'site-reviews/assets/polyfill', true )) {
59
+			return;
60
+		}
59 61
 		wp_enqueue_script( Application::ID.'/polyfill', add_query_arg([
60 62
 			'features' => 'CustomEvent,Element.prototype.closest,Element.prototype.dataset,Event',
61 63
 			'flags' => 'gated',
@@ -70,7 +72,9 @@  discard block
 block discarded – undo
70 72
 		// wpforms-recaptcha
71 73
 		// google-recaptcha
72 74
 		// nf-google-recaptcha
73
-		if( !glsr( OptionManager::class )->isRecaptchaEnabled() )return;
75
+		if( !glsr( OptionManager::class )->isRecaptchaEnabled() ) {
76
+			return;
77
+		}
74 78
 		$language = apply_filters( 'site-reviews/recaptcha/language', get_locale() );
75 79
 		wp_enqueue_script( Application::ID.'/google-recaptcha', add_query_arg([
76 80
 			'hl' => $language,
@@ -84,7 +88,9 @@  discard block
 block discarded – undo
84 88
 	public function inlineStyles()
85 89
 	{
86 90
 		$inlineStylesheetPath = glsr()->path( 'assets/styles/inline-styles.css' );
87
-		if( !apply_filters( 'site-reviews/assets/css', true ))return;
91
+		if( !apply_filters( 'site-reviews/assets/css', true )) {
92
+			return;
93
+		}
88 94
 		if( !file_exists( $inlineStylesheetPath )) {
89 95
 			glsr_log()->error( 'Inline stylesheet is missing: '.$inlineStylesheetPath );
90 96
 			return;
Please login to merge, or discard this patch.
plugin/Defaults/DefaultsAbstract.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -13,7 +13,7 @@
 block discarded – undo
13 13
 	 */
14 14
 	public function __call( $name, array $args = [] )
15 15
 	{
16
-		if( !method_exists( $this, $name ))return;
16
+		if( !method_exists( $this, $name ) )return;
17 17
 		$defaults = call_user_func_array( [$this, $name], $args );
18 18
 		$className = (new ReflectionClass( $this ))->getShortName();
19 19
 		$className = str_replace( 'Defaults', '', $className );
Please login to merge, or discard this patch.
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -13,7 +13,9 @@
 block discarded – undo
13 13
 	 */
14 14
 	public function __call( $name, array $args = [] )
15 15
 	{
16
-		if( !method_exists( $this, $name ))return;
16
+		if( !method_exists( $this, $name )) {
17
+			return;
18
+		}
17 19
 		$defaults = call_user_func_array( [$this, $name], $args );
18 20
 		$className = (new ReflectionClass( $this ))->getShortName();
19 21
 		$className = str_replace( 'Defaults', '', $className );
Please login to merge, or discard this patch.
plugin/Controllers/AjaxController.php 1 patch
Spacing   +18 added lines, -18 removed lines patch added patch discarded remove patch
@@ -22,7 +22,7 @@  discard block
 block discarded – undo
22 22
 	 */
23 23
 	public function routerChangeStatus( array $request )
24 24
 	{
25
-		wp_send_json_success( $this->execute( new ChangeStatus( $request )));
25
+		wp_send_json_success( $this->execute( new ChangeStatus( $request ) ) );
26 26
 	}
27 27
 
28 28
 	/**
@@ -31,10 +31,10 @@  discard block
 block discarded – undo
31 31
 	public function routerClearConsole()
32 32
 	{
33 33
 		glsr( AdminController::class )->routerClearConsole();
34
-		wp_send_json_success([
34
+		wp_send_json_success( [
35 35
 			'console' => glsr( Console::class )->get(),
36 36
 			'notices' => glsr( Notice::class )->get(),
37
-		]);
37
+		] );
38 38
 	}
39 39
 
40 40
 	/**
@@ -43,9 +43,9 @@  discard block
 block discarded – undo
43 43
 	public function routerCountReviews()
44 44
 	{
45 45
 		glsr( AdminController::class )->routerCountReviews();
46
-		wp_send_json_success([
46
+		wp_send_json_success( [
47 47
 			'notices' => glsr( Notice::class )->get(),
48
-		]);
48
+		] );
49 49
 	}
50 50
 
51 51
 	/**
@@ -55,9 +55,9 @@  discard block
 block discarded – undo
55 55
 	{
56 56
 		$shortcode = $request['shortcode'];
57 57
 		$response = false;
58
-		if( array_key_exists( $shortcode, glsr()->mceShortcodes )) {
58
+		if( array_key_exists( $shortcode, glsr()->mceShortcodes ) ) {
59 59
 			$data = glsr()->mceShortcodes[$shortcode];
60
-			if( !empty( $data['errors'] )) {
60
+			if( !empty($data['errors']) ) {
61 61
 				$data['btn_okay'] = [esc_html__( 'Okay', 'site-reviews' )];
62 62
 			}
63 63
 			$response = [
@@ -77,10 +77,10 @@  discard block
 block discarded – undo
77 77
 	public function routerFetchConsole()
78 78
 	{
79 79
 		glsr( AdminController::class )->routerFetchConsole();
80
-		wp_send_json_success([
80
+		wp_send_json_success( [
81 81
 			'console' => glsr( Console::class )->get(),
82 82
 			'notices' => glsr( Notice::class )->get(),
83
-		]);
83
+		] );
84 84
 	}
85 85
 
86 86
 	/**
@@ -89,10 +89,10 @@  discard block
 block discarded – undo
89 89
 	public function routerSearchPosts( array $request )
90 90
 	{
91 91
 		$results = glsr( Database::class )->searchPosts( $request['search'] );
92
-		wp_send_json_success([
92
+		wp_send_json_success( [
93 93
 			'empty' => '<div>'.__( 'Nothing found.', 'site-reviews' ).'</div>',
94 94
 			'items' => $results,
95
-		]);
95
+		] );
96 96
 	}
97 97
 
98 98
 	/**
@@ -100,7 +100,7 @@  discard block
 block discarded – undo
100 100
 	 */
101 101
 	public function routerSearchTranslations( array $request )
102 102
 	{
103
-		if( empty( $request['exclude'] )) {
103
+		if( empty($request['exclude']) ) {
104 104
 			$request['exclude'] = [];
105 105
 		}
106 106
 		$results = glsr( Translator::class )
@@ -108,10 +108,10 @@  discard block
 block discarded – undo
108 108
 			->exclude()
109 109
 			->exclude( $request['exclude'] )
110 110
 			->renderResults();
111
-		wp_send_json_success([
111
+		wp_send_json_success( [
112 112
 			'empty' => '<div>'.__( 'Nothing found.', 'site-reviews' ).'</div>',
113 113
 			'items' => $results,
114
-		]);
114
+		] );
115 115
 	}
116 116
 
117 117
 	/**
@@ -125,7 +125,7 @@  discard block
 block discarded – undo
125 125
 			'message' => glsr( Session::class )->get( $request['form_id'].'message', '', true ),
126 126
 			'recaptcha' => glsr( Session::class )->get( $request['form_id'].'recaptcha', false, true ),
127 127
 		];
128
-		if( !wp_validate_boolean( $data['errors'] )) {
128
+		if( !wp_validate_boolean( $data['errors'] ) ) {
129 129
 			glsr( Session::class )->clear();
130 130
 			wp_send_json_success( $data );
131 131
 		}
@@ -137,10 +137,10 @@  discard block
 block discarded – undo
137 137
 	 */
138 138
 	public function routerTogglePinned( array $request )
139 139
 	{
140
-		$isPinned = $this->execute( new TogglePinned( $request ));
141
-		wp_send_json_success([
140
+		$isPinned = $this->execute( new TogglePinned( $request ) );
141
+		wp_send_json_success( [
142 142
 			'notices' => glsr( Notice::class )->get(),
143 143
 			'pinned' => $isPinned,
144
-		]);
144
+		] );
145 145
 	}
146 146
 }
Please login to merge, or discard this patch.
plugin/Modules/Validator/ValidateReview.php 2 patches
Spacing   +22 added lines, -22 removed lines patch added patch discarded remove patch
@@ -61,7 +61,7 @@  discard block
 block discarded – undo
61 61
 		$this->validateBlacklist();
62 62
 		$this->validateAkismet();
63 63
 		$this->validateRecaptcha();
64
-		if( !empty( $this->error )) {
64
+		if( !empty($this->error) ) {
65 65
 			$this->setSessionValues( 'message', $this->error );
66 66
 		}
67 67
 		return $this;
@@ -84,12 +84,12 @@  discard block
 block discarded – undo
84 84
 	{
85 85
 		$rules = array_intersect_key(
86 86
 			apply_filters( 'site-reviews/validation/rules', static::VALIDATION_RULES ),
87
-			array_flip( $this->getOption( 'settings.submissions.required', [] ))
87
+			array_flip( $this->getOption( 'settings.submissions.required', [] ) )
88 88
 		);
89
-		$excluded = isset( $request['excluded'] )
89
+		$excluded = isset($request['excluded'])
90 90
 			? (array)json_decode( $request['excluded'] )
91 91
 			: [];
92
-		return array_diff_key( $rules, array_flip( $excluded ));
92
+		return array_diff_key( $rules, array_flip( $excluded ) );
93 93
 	}
94 94
 
95 95
 	/**
@@ -100,7 +100,7 @@  discard block
 block discarded – undo
100 100
 		if( !glsr( OptionManager::class )->isRecaptchaEnabled() ) {
101 101
 			return true;
102 102
 		}
103
-		if( empty( $this->request['recaptcha-token'] )) {
103
+		if( empty($this->request['recaptcha-token']) ) {
104 104
 			return null; // @see $this->validateRecaptcha()
105 105
 		}
106 106
 		return $this->isRecaptchaValid( $this->request['recaptcha-token'] );
@@ -112,17 +112,17 @@  discard block
 block discarded – undo
112 112
 	 */
113 113
 	protected function isRecaptchaValid( $recaptchaToken )
114 114
 	{
115
-		$endpoint = add_query_arg([
115
+		$endpoint = add_query_arg( [
116 116
 			'remoteip' => glsr( Helper::class )->getIpAddress(),
117 117
 			'response' => $recaptchaToken,
118 118
 			'secret' => $this->getOption( 'settings.submissions.recaptcha.secret' ),
119 119
 		], static::RECAPTCHA_ENDPOINT );
120
-		if( is_wp_error( $response = wp_remote_get( $endpoint ))) {
120
+		if( is_wp_error( $response = wp_remote_get( $endpoint ) ) ) {
121 121
 			glsr_log()->error( $response->get_error_message() );
122 122
 			return false;
123 123
 		}
124
-		$response = json_decode( wp_remote_retrieve_body( $response ));
125
-		if( !empty( $response->success )) {
124
+		$response = json_decode( wp_remote_retrieve_body( $response ) );
125
+		if( !empty($response->success) ) {
126 126
 			return boolval( $response->success );
127 127
 		}
128 128
 		foreach( $response->{'error-codes'} as $error ) {
@@ -138,7 +138,7 @@  discard block
 block discarded – undo
138 138
 	{
139 139
 		$rules = $this->getValidationRules( $request );
140 140
 		$errors = glsr( Validator::class )->validate( $request, $rules );
141
-		if( empty( $errors )) {
141
+		if( empty($errors) ) {
142 142
 			return true;
143 143
 		}
144 144
 		$this->setSessionValues( 'errors', $errors );
@@ -155,7 +155,7 @@  discard block
 block discarded – undo
155 155
 	protected function setSessionValues( $type, $value, $loggedMessage = '' )
156 156
 	{
157 157
 		glsr( Session::class )->set( $this->form_id.$type, $value );
158
-		if( !empty( $loggedMessage )) {
158
+		if( !empty($loggedMessage) ) {
159 159
 			glsr_log()->warning( $loggedMessage );
160 160
 			glsr_log()->warning( $this->request );
161 161
 		}
@@ -166,8 +166,8 @@  discard block
 block discarded – undo
166 166
 	 */
167 167
 	protected function validateAkismet()
168 168
 	{
169
-		if( !empty( $this->error ))return;
170
-		if( !glsr( Akismet::class )->isSpam( $this->request ))return;
169
+		if( !empty($this->error) )return;
170
+		if( !glsr( Akismet::class )->isSpam( $this->request ) )return;
171 171
 		$this->setSessionValues( 'errors', [], 'Akismet caught a spam submission:' );
172 172
 		$this->error = __( 'Your review cannot be submitted at this time. Please try again later.', 'site-reviews' );
173 173
 	}
@@ -177,8 +177,8 @@  discard block
 block discarded – undo
177 177
 	 */
178 178
 	protected function validateBlacklist()
179 179
 	{
180
-		if( !empty( $this->error ))return;
181
-		if( !glsr( Blacklist::class )->isBlacklisted( $this->request ))return;
180
+		if( !empty($this->error) )return;
181
+		if( !glsr( Blacklist::class )->isBlacklisted( $this->request ) )return;
182 182
 		$blacklistAction = $this->getOption( 'settings.submissions.blacklist.action' );
183 183
 		if( $blacklistAction == 'reject' ) {
184 184
 			$this->setSessionValues( 'errors', [], 'Blacklisted submission detected:' );
@@ -193,7 +193,7 @@  discard block
 block discarded – undo
193 193
 	 */
194 194
 	protected function validateCustom()
195 195
 	{
196
-		if( !empty( $this->error ))return;
196
+		if( !empty($this->error) )return;
197 197
 		$validated = apply_filters( 'site-reviews/validate/review/submission', true, $this->request );
198 198
 		if( $validated === true )return;
199 199
 		$this->setSessionValues( 'errors', [] );
@@ -208,8 +208,8 @@  discard block
 block discarded – undo
208 208
 	 */
209 209
 	protected function validateHoneyPot()
210 210
 	{
211
-		if( !empty( $this->error ))return;
212
-		if( empty( $this->request['gotcha'] ))return;
211
+		if( !empty($this->error) )return;
212
+		if( empty($this->request['gotcha']) )return;
213 213
 		$this->setSessionValues( 'errors', [], 'The Honeypot caught a bad submission:' );
214 214
 		$this->error = __( 'The review submission failed. Please notify the site administrator.', 'site-reviews' );
215 215
 	}
@@ -219,9 +219,9 @@  discard block
 block discarded – undo
219 219
 	 */
220 220
 	protected function validateRecaptcha()
221 221
 	{
222
-		if( !empty( $this->error ))return;
222
+		if( !empty($this->error) )return;
223 223
 		$isValid = $this->isRecaptchaResponseValid();
224
-		if( is_null( $isValid )) {
224
+		if( is_null( $isValid ) ) {
225 225
 			$this->setSessionValues( 'recaptcha', 'unset' );
226 226
 			$this->recaptchaIsUnset = true;
227 227
 		}
@@ -237,11 +237,11 @@  discard block
 block discarded – undo
237 237
 	 */
238 238
 	protected function validateRequest( array $request )
239 239
 	{
240
-		if( !$this->isRequestValid( $request )) {
240
+		if( !$this->isRequestValid( $request ) ) {
241 241
 			$this->error = __( 'Please fix the submission errors.', 'site-reviews' );
242 242
 			return $request;
243 243
 		}
244
-		if( empty( $request['title'] )) {
244
+		if( empty($request['title']) ) {
245 245
 			$request['title'] = __( 'No Title', 'site-reviews' );
246 246
 		}
247 247
 		return array_merge( glsr( ValidateReviewDefaults::class )->defaults(), $request );
Please login to merge, or discard this patch.
Braces   +27 added lines, -9 removed lines patch added patch discarded remove patch
@@ -166,8 +166,12 @@  discard block
 block discarded – undo
166 166
 	 */
167 167
 	protected function validateAkismet()
168 168
 	{
169
-		if( !empty( $this->error ))return;
170
-		if( !glsr( Akismet::class )->isSpam( $this->request ))return;
169
+		if( !empty( $this->error )) {
170
+			return;
171
+		}
172
+		if( !glsr( Akismet::class )->isSpam( $this->request )) {
173
+			return;
174
+		}
171 175
 		$this->setSessionValues( 'errors', [], 'Akismet caught a spam submission:' );
172 176
 		$this->error = __( 'Your review cannot be submitted at this time. Please try again later.', 'site-reviews' );
173 177
 	}
@@ -177,8 +181,12 @@  discard block
 block discarded – undo
177 181
 	 */
178 182
 	protected function validateBlacklist()
179 183
 	{
180
-		if( !empty( $this->error ))return;
181
-		if( !glsr( Blacklist::class )->isBlacklisted( $this->request ))return;
184
+		if( !empty( $this->error )) {
185
+			return;
186
+		}
187
+		if( !glsr( Blacklist::class )->isBlacklisted( $this->request )) {
188
+			return;
189
+		}
182 190
 		$blacklistAction = $this->getOption( 'settings.submissions.blacklist.action' );
183 191
 		if( $blacklistAction == 'reject' ) {
184 192
 			$this->setSessionValues( 'errors', [], 'Blacklisted submission detected:' );
@@ -193,9 +201,13 @@  discard block
 block discarded – undo
193 201
 	 */
194 202
 	protected function validateCustom()
195 203
 	{
196
-		if( !empty( $this->error ))return;
204
+		if( !empty( $this->error )) {
205
+			return;
206
+		}
197 207
 		$validated = apply_filters( 'site-reviews/validate/review/submission', true, $this->request );
198
-		if( $validated === true )return;
208
+		if( $validated === true ) {
209
+			return;
210
+		}
199 211
 		$this->setSessionValues( 'errors', [] );
200 212
 		$this->setSessionValues( 'values', $this->request );
201 213
 		$this->error = is_string( $validated )
@@ -208,8 +220,12 @@  discard block
 block discarded – undo
208 220
 	 */
209 221
 	protected function validateHoneyPot()
210 222
 	{
211
-		if( !empty( $this->error ))return;
212
-		if( empty( $this->request['gotcha'] ))return;
223
+		if( !empty( $this->error )) {
224
+			return;
225
+		}
226
+		if( empty( $this->request['gotcha'] )) {
227
+			return;
228
+		}
213 229
 		$this->setSessionValues( 'errors', [], 'The Honeypot caught a bad submission:' );
214 230
 		$this->error = __( 'The review submission failed. Please notify the site administrator.', 'site-reviews' );
215 231
 	}
@@ -219,7 +235,9 @@  discard block
 block discarded – undo
219 235
 	 */
220 236
 	protected function validateRecaptcha()
221 237
 	{
222
-		if( !empty( $this->error ))return;
238
+		if( !empty( $this->error )) {
239
+			return;
240
+		}
223 241
 		$isValid = $this->isRecaptchaResponseValid();
224 242
 		if( is_null( $isValid )) {
225 243
 			$this->setSessionValues( 'recaptcha', 'unset' );
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->referer = $this->get( '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.