Passed
Push — master ( 999d23...fe5b90 )
by Paul
04:24
created
compatibility.php 1 patch
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -17,6 +17,8 @@
 block discarded – undo
17 17
  * @return void
18 18
  */
19 19
 add_action( 'site-reviews/customize/divi', function( $instance ) {
20
-	if( $instance->tag != 'label' || $instance->args['type'] != 'checkbox' )return;
20
+	if( $instance->tag != 'label' || $instance->args['type'] != 'checkbox' ) {
21
+		return;
22
+	}
21 23
 	$instance->args['text'] = '<i></i>'.$instance->args['text'];
22 24
 });
Please login to merge, or discard this patch.
plugin/Modules/Upgrader.php 1 patch
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -17,7 +17,9 @@
 block discarded – undo
17 17
 		natsort( $routines );
18 18
 		array_walk( $routines, function( $routine ) {
19 19
 			$parts = explode( '__', $routine );
20
-			if( version_compare( glsr()->version, end( $parts ), '<' ))return;
20
+			if( version_compare( glsr()->version, end( $parts ), '<' )) {
21
+				return;
22
+			}
21 23
 			call_user_func( [$this, $routine] );
22 24
 		});
23 25
 		$this->updateVersion();
Please login to merge, or discard this patch.
plugin/Controllers/EditorController/Metaboxes.php 1 patch
Braces   +6 added lines, -2 removed lines patch added patch discarded remove patch
@@ -13,7 +13,9 @@  discard block
 block discarded – undo
13 13
 	 */
14 14
 	public function saveAssignedToMetabox( $postId )
15 15
 	{
16
-		if( !wp_verify_nonce( glsr( Helper::class )->filterInput( '_nonce-assigned-to' ), 'assigned_to' ))return;
16
+		if( !wp_verify_nonce( glsr( Helper::class )->filterInput( '_nonce-assigned-to' ), 'assigned_to' )) {
17
+			return;
18
+		}
17 19
 		$assignedTo = strval( glsr( Helper::class )->filterInput( 'assigned_to' ));
18 20
 		if( get_post_meta( $postId, 'assigned_to', true ) != $assignedTo ) {
19 21
 			$review = glsr( ReviewManager::class )->single( get_post( $postId ));
@@ -28,7 +30,9 @@  discard block
 block discarded – undo
28 30
 	 */
29 31
 	public function saveResponseMetabox( $postId )
30 32
 	{
31
-		if( !wp_verify_nonce( glsr( Helper::class )->filterInput( '_nonce-response' ), 'response' ))return;
33
+		if( !wp_verify_nonce( glsr( Helper::class )->filterInput( '_nonce-response' ), 'response' )) {
34
+			return;
35
+		}
32 36
 		$response = strval( glsr( Helper::class )->filterInput( 'response' ));
33 37
 		update_post_meta( $postId, 'response', trim( wp_kses( $response, [
34 38
 			'a' => ['href' => [], 'title' => []],
Please login to merge, or discard this patch.
plugin/Database/CountsManager.php 1 patch
Braces   +21 added lines, -7 removed lines patch added patch discarded remove patch
@@ -73,7 +73,9 @@  discard block
 block discarded – undo
73 73
 	 */
74 74
 	public function decreasePostCounts( Review $review )
75 75
 	{
76
-		if( empty( $counts = $this->getPostCounts( $review->assigned_to )))return;
76
+		if( empty( $counts = $this->getPostCounts( $review->assigned_to ))) {
77
+			return;
78
+		}
77 79
 		$counts = $this->decreaseRating( $counts, $review->review_type, $review->rating );
78 80
 		$this->setPostCounts( $review->assigned_to, $counts );
79 81
 	}
@@ -84,7 +86,9 @@  discard block
 block discarded – undo
84 86
 	public function decreaseTermCounts( Review $review )
85 87
 	{
86 88
 		foreach( $review->term_ids as $termId ) {
87
-			if( empty( $counts = $this->getTermCounts( $termId )))continue;
89
+			if( empty( $counts = $this->getTermCounts( $termId ))) {
90
+				continue;
91
+			}
88 92
 			$counts = $this->decreaseRating( $counts, $review->review_type, $review->rating );
89 93
 			$this->setTermCounts( $termId, $counts );
90 94
 		}
@@ -106,7 +110,9 @@  discard block
 block discarded – undo
106 110
 			'min' => Rating::MIN_RATING,
107 111
 		]);
108 112
 		foreach( $counts as $index => &$num ) {
109
-			if( $index >= intval( $args['min'] ) && $index <= intval( $args['max'] ))continue;
113
+			if( $index >= intval( $args['min'] ) && $index <= intval( $args['max'] )) {
114
+				continue;
115
+			}
110 116
 			$num = 0;
111 117
 		}
112 118
 		return $counts;
@@ -191,7 +197,9 @@  discard block
 block discarded – undo
191 197
 	 */
192 198
 	public function increasePostCounts( Review $review )
193 199
 	{
194
-		if( !( get_post( $review->assigned_to ) instanceof WP_Post ))return;
200
+		if( !( get_post( $review->assigned_to ) instanceof WP_Post )) {
201
+			return;
202
+		}
195 203
 		$counts = $this->getPostCounts( $review->assigned_to );
196 204
 		$counts = empty( $counts )
197 205
 			? $this->buildPostCounts( $review->assigned_to )
@@ -240,7 +248,9 @@  discard block
 block discarded – undo
240 248
 	 */
241 249
 	public function setTermCounts( $termId, array $reviewCounts )
242 250
 	{
243
-		if( !term_exists( $termId ))return;
251
+		if( !term_exists( $termId )) {
252
+			return;
253
+		}
244 254
 		$ratingCounts = glsr( CountsManager::class )->flatten( $reviewCounts );
245 255
 		update_term_meta( $termId, static::META_COUNT, $reviewCounts );
246 256
 		update_term_meta( $termId, static::META_AVERAGE, glsr( Rating::class )->getAverage( $ratingCounts ));
@@ -258,7 +268,9 @@  discard block
 block discarded – undo
258 268
 		while( $reviews = $this->queryReviews( $args, $lastPostId, $limit )) {
259 269
 			$types = array_keys( array_flip( array_column( $reviews, 'type' )));
260 270
 			foreach( $types as $type ) {
261
-				if( isset( $counts[$type] ))continue;
271
+				if( isset( $counts[$type] )) {
272
+					continue;
273
+				}
262 274
 				$counts[$type] = array_fill_keys( range( 0, Rating::MAX_RATING ), 0 );
263 275
 			}
264 276
 			foreach( $reviews as $review ) {
@@ -307,7 +319,9 @@  discard block
 block discarded – undo
307 319
 	{
308 320
 		foreach( $reviewCounts as &$counts ) {
309 321
 			foreach( range( 0, Rating::MAX_RATING ) as $index ) {
310
-				if( isset( $counts[$index] ))continue;
322
+				if( isset( $counts[$index] )) {
323
+					continue;
324
+				}
311 325
 				$counts[$index] = 0;
312 326
 			}
313 327
 			ksort( $counts );
Please login to merge, or discard this patch.
plugin/Database/ReviewManager.php 1 patch
Braces   +12 added lines, -4 removed lines patch added patch discarded remove patch
@@ -111,7 +111,9 @@  discard block
 block discarded – undo
111 111
 		$termIds = array_filter( array_map( 'trim', explode( ',', $commaSeparatedTermIds )));
112 112
 		foreach( $termIds as $termId ) {
113 113
 			$term = get_term( $termId, Application::TAXONOMY );
114
-			if( !isset( $term->term_id ))continue;
114
+			if( !isset( $term->term_id )) {
115
+				continue;
116
+			}
115 117
 			$terms[] = $term->term_id;
116 118
 		}
117 119
 		return $terms;
@@ -123,7 +125,9 @@  discard block
 block discarded – undo
123 125
 	 */
124 126
 	public function revert( $postId )
125 127
 	{
126
-		if( get_post_field( 'post_type', $postId ) != Application::POST_TYPE )return;
128
+		if( get_post_field( 'post_type', $postId ) != Application::POST_TYPE ) {
129
+			return;
130
+		}
127 131
 		delete_post_meta( $postId, '_edit_last' );
128 132
 		$result = wp_update_post([
129 133
 			'ID' => $postId,
@@ -141,7 +145,9 @@  discard block
 block discarded – undo
141 145
 	 */
142 146
 	public function single( WP_Post $post )
143 147
 	{
144
-		if( $post->post_type != Application::POST_TYPE )return;
148
+		if( $post->post_type != Application::POST_TYPE ) {
149
+			return;
150
+		}
145 151
 		$review = new Review( $post );
146 152
 		return apply_filters( 'site-reviews/get/review', $review, $post );
147 153
 	}
@@ -166,7 +172,9 @@  discard block
 block discarded – undo
166 172
 	protected function setTerms( $postId, $termIds )
167 173
 	{
168 174
 		$terms = $this->normalizeTerms( $termIds );
169
-		if( empty( $terms ))return;
175
+		if( empty( $terms )) {
176
+			return;
177
+		}
170 178
 		$result = wp_set_object_terms( $postId, $terms, Application::TAXONOMY );
171 179
 		if( is_wp_error( $result )) {
172 180
 			glsr_log()->error( $result->get_error_message() );
Please login to merge, or discard this patch.
plugin/Controllers/ReviewController.php 1 patch
Braces   +18 added lines, -6 removed lines patch added patch discarded remove patch
@@ -25,7 +25,9 @@  discard block
 block discarded – undo
25 25
 	{
26 26
 		sort( $termIds );
27 27
 		sort( $oldTermIds );
28
-		if( $termIds === $oldTermIds || !$this->isReviewPostId( $postId ))return;
28
+		if( $termIds === $oldTermIds || !$this->isReviewPostId( $postId )) {
29
+			return;
30
+		}
29 31
 		$review = glsr( ReviewManager::class )->single( get_post( $postId ));
30 32
 		$ignoredTerms = array_intersect( $oldTermIds, $termIds );
31 33
 		if( $review->term_ids = array_diff( $oldTermIds, $ignoredTerms )) {
@@ -45,7 +47,9 @@  discard block
 block discarded – undo
45 47
 	 */
46 48
 	public function onAfterCreate( $postData, $meta, $postId )
47 49
 	{
48
-		if( !$this->isReviewPostId( $postId ))return;
50
+		if( !$this->isReviewPostId( $postId )) {
51
+			return;
52
+		}
49 53
 		$review = glsr( ReviewManager::class )->single( get_post( $postId ));
50 54
 		glsr( CountsManager::class )->increase( $review );
51 55
 	}
@@ -57,7 +61,9 @@  discard block
 block discarded – undo
57 61
 	 */
58 62
 	public function onBeforeDelete( $postId )
59 63
 	{
60
-		if( !$this->isReviewPostId( $postId ))return;
64
+		if( !$this->isReviewPostId( $postId )) {
65
+			return;
66
+		}
61 67
 		$review = glsr( ReviewManager::class )->single( get_post( $postId ));
62 68
 		glsr( CountsManager::class )->decrease( $review );
63 69
 	}
@@ -74,9 +80,13 @@  discard block
 block discarded – undo
74 80
 	{
75 81
 		if( !$this->isReviewPostId( $postId )
76 82
 			|| !in_array( $metaKey, ['assigned_to', 'rating', 'review_type'] )
77
-		)return;
83
+		) {
84
+			return;
85
+		}
78 86
 		$review = glsr( ReviewManager::class )->single( get_post( $postId ));
79
-		if( $review->$metaKey == $metaValue )return;
87
+		if( $review->$metaKey == $metaValue ) {
88
+			return;
89
+		}
80 90
 		$method = glsr( Helper::class )->buildMethodName( $metaKey, 'onBeforeChange' );
81 91
 		call_user_func( [$this, $method], $review, $metaValue );
82 92
 	}
@@ -122,7 +132,9 @@  discard block
 block discarded – undo
122 132
 	 */
123 133
 	public function onChangeStatus( $newStatus, $oldStatus, WP_Post $post )
124 134
 	{
125
-		if( $post->post_type != Application::POST_TYPE || in_array( $oldStatus, ['new', $newStatus] ))return;
135
+		if( $post->post_type != Application::POST_TYPE || in_array( $oldStatus, ['new', $newStatus] )) {
136
+			return;
137
+		}
126 138
 		$review = glsr( ReviewManager::class )->single( get_post( $post->ID ));
127 139
 		if( $post->post_status == 'publish' ) {
128 140
 			glsr( CountsManager::class )->increase( $review );
Please login to merge, or discard this patch.
plugin/Modules/Html/Partials/SiteReviewsSummary.php 1 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 1 patch
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.
plugin/Modules/Validator.php 1 patch
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.