Passed
Push — master ( 25783a...cd4854 )
by Paul
05:21
created
plugin/Database/QueryBuilder.php 2 patches
Spacing   +16 added lines, -16 removed lines patch added patch discarded remove patch
@@ -16,11 +16,11 @@  discard block
 block discarded – undo
16 16
 	{
17 17
 		$queries = [];
18 18
 		foreach( $keys as $key ) {
19
-			if( !array_key_exists( $key, $values ))continue;
19
+			if( !array_key_exists( $key, $values ) )continue;
20 20
 			$methodName = glsr( Helper::class )->buildMethodName( $key, __FUNCTION__ );
21
-			if( !method_exists( $this, $methodName ))continue;
21
+			if( !method_exists( $this, $methodName ) )continue;
22 22
 			$query = call_user_func( [$this, $methodName], $values[$key] );
23
-			if( is_array( $query )) {
23
+			if( is_array( $query ) ) {
24 24
 				$queries[] = $query;
25 25
 			}
26 26
 		}
@@ -35,10 +35,10 @@  discard block
 block discarded – undo
35 35
 		$string = '';
36 36
 		$values = array_filter( $values );
37 37
 		foreach( $conditions as $key => $value ) {
38
-			if( !isset( $values[$key] ))continue;
38
+			if( !isset($values[$key]) )continue;
39 39
 			$values[$key] = implode( ',', (array)$values[$key] );
40 40
 			$string .= strpos( $value, '%s' ) !== false
41
-				? sprintf( $value, strval( $values[$key] ))
41
+				? sprintf( $value, strval( $values[$key] ) )
42 42
 				: $value;
43 43
 		}
44 44
 		return $string;
@@ -52,11 +52,11 @@  discard block
 block discarded – undo
52 52
 	 */
53 53
 	public function buildSqlOr( $values, $sprintfFormat )
54 54
 	{
55
-		if( !is_array( $values )) {
55
+		if( !is_array( $values ) ) {
56 56
 			$values = explode( ',', $values );
57 57
 		}
58
-		$values = array_filter( array_map( 'trim', (array)$values ));
59
-		$values = array_map( function( $value ) use( $sprintfFormat ) {
58
+		$values = array_filter( array_map( 'trim', (array)$values ) );
59
+		$values = array_map( function( $value ) use($sprintfFormat) {
60 60
 			return sprintf( $sprintfFormat, $value );
61 61
 		}, $values );
62 62
 		return implode( ' OR ', $values );
@@ -71,11 +71,11 @@  discard block
 block discarded – undo
71 71
 	 */
72 72
 	public function filterSearchByTitle( $search, WP_Query $query )
73 73
 	{
74
-		if( empty( $search ) || empty( $query->get( 'search_terms' ))) {
74
+		if( empty($search) || empty($query->get( 'search_terms' )) ) {
75 75
 			return $search;
76 76
 		}
77 77
 		global $wpdb;
78
-		$n = empty( $query->get( 'exact' ))
78
+		$n = empty($query->get( 'exact' ))
79 79
 			? '%'
80 80
 			: '';
81 81
 		$search = [];
@@ -99,7 +99,7 @@  discard block
 block discarded – undo
99 99
 			? Application::PAGED_QUERY_VAR
100 100
 			: 'page';
101 101
 		return $isEnabled
102
-			? max( 1, intval( get_query_var( $pagedQuery )))
102
+			? max( 1, intval( get_query_var( $pagedQuery ) ) )
103 103
 			: 1;
104 104
 	}
105 105
 
@@ -109,11 +109,11 @@  discard block
 block discarded – undo
109 109
 	 */
110 110
 	protected function buildQueryAssignedTo( $value )
111 111
 	{
112
-		if( empty( $value ))return;
112
+		if( empty($value) )return;
113 113
 		return [
114 114
 			'compare' => 'IN',
115 115
 			'key' => 'assigned_to',
116
-			'value' => array_filter( array_map( 'trim', explode( ',', $value )), 'is_numeric' ),
116
+			'value' => array_filter( array_map( 'trim', explode( ',', $value ) ), 'is_numeric' ),
117 117
 		];
118 118
 	}
119 119
 
@@ -123,7 +123,7 @@  discard block
 block discarded – undo
123 123
 	 */
124 124
 	protected function buildQueryCategory( $value )
125 125
 	{
126
-		if( empty( $value ))return;
126
+		if( empty($value) )return;
127 127
 		return [
128 128
 			'field' => 'term_id',
129 129
 			'taxonomy' => Application::TAXONOMY,
@@ -137,7 +137,7 @@  discard block
 block discarded – undo
137 137
 	 */
138 138
 	protected function buildQueryRating( $value )
139 139
 	{
140
-		if( !is_numeric( $value ) || !in_array( intval( $value ), range( 1, 5 )))return;
140
+		if( !is_numeric( $value ) || !in_array( intval( $value ), range( 1, 5 ) ) )return;
141 141
 		return [
142 142
 			'compare' => '>=',
143 143
 			'key' => 'rating',
@@ -151,7 +151,7 @@  discard block
 block discarded – undo
151 151
 	 */
152 152
 	protected function buildQueryType( $value )
153 153
 	{
154
-		if( in_array( $value, ['','all'] ))return;
154
+		if( in_array( $value, ['', 'all'] ) )return;
155 155
 		return [
156 156
 			'key' => 'review_type',
157 157
 			'value' => $value,
Please login to merge, or discard this patch.
Braces   +21 added lines, -7 removed lines patch added patch discarded remove patch
@@ -16,9 +16,13 @@  discard block
 block discarded – undo
16 16
 	{
17 17
 		$queries = [];
18 18
 		foreach( $keys as $key ) {
19
-			if( !array_key_exists( $key, $values ))continue;
19
+			if( !array_key_exists( $key, $values )) {
20
+				continue;
21
+			}
20 22
 			$methodName = glsr( Helper::class )->buildMethodName( $key, __FUNCTION__ );
21
-			if( !method_exists( $this, $methodName ))continue;
23
+			if( !method_exists( $this, $methodName )) {
24
+				continue;
25
+			}
22 26
 			$query = call_user_func( [$this, $methodName], $values[$key] );
23 27
 			if( is_array( $query )) {
24 28
 				$queries[] = $query;
@@ -35,7 +39,9 @@  discard block
 block discarded – undo
35 39
 		$string = '';
36 40
 		$values = array_filter( $values );
37 41
 		foreach( $conditions as $key => $value ) {
38
-			if( !isset( $values[$key] ))continue;
42
+			if( !isset( $values[$key] )) {
43
+				continue;
44
+			}
39 45
 			$values[$key] = implode( ',', (array)$values[$key] );
40 46
 			$string .= strpos( $value, '%s' ) !== false
41 47
 				? sprintf( $value, strval( $values[$key] ))
@@ -109,7 +115,9 @@  discard block
 block discarded – undo
109 115
 	 */
110 116
 	protected function buildQueryAssignedTo( $value )
111 117
 	{
112
-		if( empty( $value ))return;
118
+		if( empty( $value )) {
119
+			return;
120
+		}
113 121
 		return [
114 122
 			'compare' => 'IN',
115 123
 			'key' => 'assigned_to',
@@ -123,7 +131,9 @@  discard block
 block discarded – undo
123 131
 	 */
124 132
 	protected function buildQueryCategory( $value )
125 133
 	{
126
-		if( empty( $value ))return;
134
+		if( empty( $value )) {
135
+			return;
136
+		}
127 137
 		return [
128 138
 			'field' => 'term_id',
129 139
 			'taxonomy' => Application::TAXONOMY,
@@ -137,7 +147,9 @@  discard block
 block discarded – undo
137 147
 	 */
138 148
 	protected function buildQueryRating( $value )
139 149
 	{
140
-		if( !is_numeric( $value ) || !in_array( intval( $value ), range( 1, 5 )))return;
150
+		if( !is_numeric( $value ) || !in_array( intval( $value ), range( 1, 5 ))) {
151
+			return;
152
+		}
141 153
 		return [
142 154
 			'compare' => '>=',
143 155
 			'key' => 'rating',
@@ -151,7 +163,9 @@  discard block
 block discarded – undo
151 163
 	 */
152 164
 	protected function buildQueryType( $value )
153 165
 	{
154
-		if( in_array( $value, ['','all'] ))return;
166
+		if( in_array( $value, ['','all'] )) {
167
+			return;
168
+		}
155 169
 		return [
156 170
 			'key' => 'review_type',
157 171
 			'value' => $value,
Please login to merge, or discard this patch.
plugin/Database/SqlQueries.php 1 patch
Spacing   +20 added lines, -20 removed lines patch added patch discarded remove patch
@@ -23,11 +23,11 @@  discard block
 block discarded – undo
23 23
 	 */
24 24
 	public function deleteAllSessions( $sessionCookiePrefix )
25 25
 	{
26
-		return $this->db->query("
26
+		return $this->db->query( "
27 27
 			DELETE
28 28
 			FROM {$this->db->options}
29 29
 			WHERE option_name LIKE '{$sessionCookiePrefix}_%'
30
-		");
30
+		" );
31 31
 	}
32 32
 
33 33
 	/**
@@ -36,11 +36,11 @@  discard block
 block discarded – undo
36 36
 	 */
37 37
 	public function deleteExpiredSessions( $expiredSessions )
38 38
 	{
39
-		return $this->db->query("
39
+		return $this->db->query( "
40 40
 			DELETE
41 41
 			FROM {$this->db->options}
42 42
 			WHERE option_name IN ('{$expiredSessions}')
43
-		");
43
+		" );
44 44
 	}
45 45
 
46 46
 	/**
@@ -49,14 +49,14 @@  discard block
 block discarded – undo
49 49
 	 */
50 50
 	public function getReviewCounts( $metaKey )
51 51
 	{
52
-		return (array) $this->db->get_results("
52
+		return (array)$this->db->get_results( "
53 53
 			SELECT m.meta_value AS name, COUNT(*) num_posts
54 54
 			FROM {$this->db->posts} AS p
55 55
 			INNER JOIN {$this->db->postmeta} AS m ON p.ID = m.post_id
56 56
 			WHERE p.post_type = '{$this->postType}'
57 57
 			AND m.meta_key = '{$metaKey}'
58 58
 			GROUP BY name
59
-		");
59
+		" );
60 60
 	}
61 61
 
62 62
 	/**
@@ -66,13 +66,13 @@  discard block
 block discarded – undo
66 66
 	 */
67 67
 	public function getExpiredSessions( $sessionCookiePrefix, $limit )
68 68
 	{
69
-		return $this->db->get_results("
69
+		return $this->db->get_results( "
70 70
 			SELECT option_name AS name, option_value AS expiration
71 71
 			FROM {$this->db->options}
72 72
 			WHERE option_name LIKE '{$sessionCookiePrefix}_expires_%'
73 73
 			ORDER BY option_value ASC
74 74
 			LIMIT 0, {$limit}
75
-		");
75
+		" );
76 76
 	}
77 77
 
78 78
 	/**
@@ -81,14 +81,14 @@  discard block
 block discarded – undo
81 81
 	 */
82 82
 	public function getReviewPostId( $metaReviewId )
83 83
 	{
84
-		$postId = $this->db->get_var("
84
+		$postId = $this->db->get_var( "
85 85
 			SELECT p.ID
86 86
 			FROM {$this->db->posts} AS p
87 87
 			INNER JOIN {$this->db->postmeta} AS pm ON p.ID = pm.post_id
88 88
 			WHERE p.post_type = '{$this->postType}'
89 89
 			AND pm.meta_key = 'review_id'
90 90
 			AND pm.meta_value = '{$metaReviewId}'
91
-		");
91
+		" );
92 92
 		return intval( $postId );
93 93
 	}
94 94
 
@@ -98,7 +98,7 @@  discard block
 block discarded – undo
98 98
 	 */
99 99
 	public function getReviewIdsByType( $reviewType )
100 100
 	{
101
-		$query = $this->db->get_col("
101
+		$query = $this->db->get_col( "
102 102
 			SELECT m1.meta_value AS review_id
103 103
 			FROM {$this->db->posts} AS p
104 104
 			INNER JOIN {$this->db->postmeta} AS m1 ON p.ID = m1.post_id
@@ -107,8 +107,8 @@  discard block
 block discarded – undo
107 107
 			AND m1.meta_key = 'review_id'
108 108
 			AND m2.meta_key = 'review_type'
109 109
 			AND m2.meta_value = '{$reviewType}'
110
-		");
111
-		return array_keys( array_flip( $query ));
110
+		" );
111
+		return array_keys( array_flip( $query ) );
112 112
 	}
113 113
 
114 114
 	/**
@@ -123,12 +123,12 @@  discard block
 block discarded – undo
123 123
 			'rating' => "AND m1.meta_value = '%s' ",
124 124
 			'review_type' => "AND m2.meta_value = '%s' ",
125 125
 			'term_ids' => "AND tr.term_taxonomy_id IN (%s) ",
126
-		]);
126
+		] );
127 127
 		$innerjoins = glsr( QueryBuilder::class )->buildSqlLines( $args, [
128 128
 			'assigned_to' => "INNER JOIN {$this->db->postmeta} AS m3 ON p.ID = m3.post_id ",
129 129
 			'term_ids' => "INNER JOIN {$this->db->term_relationships} AS tr ON p.ID = tr.object_id ",
130
-		]);
131
-		return $this->db->get_results("
130
+		] );
131
+		return $this->db->get_results( "
132 132
 			SELECT p.ID, m1.meta_value AS rating, m2.meta_value AS type
133 133
 			FROM {$this->db->posts} AS p
134 134
 			INNER JOIN {$this->db->postmeta} AS m1 ON p.ID = m1.post_id
@@ -153,9 +153,9 @@  discard block
 block discarded – undo
153 153
 	public function getReviewRatingsFromIds( array $postIds, $greaterThanId = 0, $limit = 100 )
154 154
 	{
155 155
 		sort( $postIds );
156
-		$postIds = array_slice( $postIds, intval( array_search( $greaterThanId, $postIds )), $limit );
156
+		$postIds = array_slice( $postIds, intval( array_search( $greaterThanId, $postIds ) ), $limit );
157 157
 		$postIds = implode( ',', $postIds );
158
-		return $this->db->get_results("
158
+		return $this->db->get_results( "
159 159
 			SELECT p.ID, m.meta_value AS rating
160 160
 			FROM {$this->db->posts} AS p
161 161
 			INNER JOIN {$this->db->postmeta} AS m ON p.ID = m.post_id
@@ -179,7 +179,7 @@  discard block
 block discarded – undo
179 179
 		$queryBuilder = glsr( QueryBuilder::class );
180 180
 		$key = $queryBuilder->buildSqlOr( $key, "m.meta_key = '%s'" );
181 181
 		$status = $queryBuilder->buildSqlOr( $status, "p.post_status = '%s'" );
182
-		return $this->db->get_col("
182
+		return $this->db->get_col( "
183 183
 			SELECT DISTINCT m.meta_value
184 184
 			FROM {$this->db->postmeta} m
185 185
 			LEFT JOIN {$this->db->posts} p ON p.ID = m.post_id
@@ -187,6 +187,6 @@  discard block
 block discarded – undo
187 187
 			AND ({$key})
188 188
 			AND ({$status})
189 189
 			ORDER BY m.meta_value
190
-		");
190
+		" );
191 191
 	}
192 192
 }
Please login to merge, or discard this patch.
plugin/Database/CountsManager.php 2 patches
Spacing   +11 added lines, -11 removed lines patch added patch discarded remove patch
@@ -15,10 +15,10 @@  discard block
 block discarded – undo
15 15
 	{
16 16
 		$counts = [];
17 17
 		$greaterThanId = 0;
18
-		while( $reviews = glsr( SqlQueries::class )->getReviewRatings( $args, $greaterThanId, $limit )) {
19
-			$types = array_keys( array_flip( array_column( $reviews, 'type' )));
18
+		while( $reviews = glsr( SqlQueries::class )->getReviewRatings( $args, $greaterThanId, $limit ) ) {
19
+			$types = array_keys( array_flip( array_column( $reviews, 'type' ) ) );
20 20
 			foreach( $types as $type ) {
21
-				if( isset( $counts[$type] ))continue;
21
+				if( isset($counts[$type]) )continue;
22 22
 				$counts[$type] = array_fill_keys( range( 0, Rating::MAX_RATING ), 0 );
23 23
 			}
24 24
 			foreach( $reviews as $review ) {
@@ -36,7 +36,7 @@  discard block
 block discarded – undo
36 36
 	{
37 37
 		$counts = array_fill_keys( range( 0, Rating::MAX_RATING ), 0 );
38 38
 		$greaterThanId = 0;
39
-		while( $reviews = glsr( SqlQueries::class )->getReviewRatingsFromIds( $postIds, $greaterThanId, $limit )) {
39
+		while( $reviews = glsr( SqlQueries::class )->getReviewRatingsFromIds( $postIds, $greaterThanId, $limit ) ) {
40 40
 			foreach( $reviews as $review ) {
41 41
 				$counts[$review->rating]++;
42 42
 			}
@@ -51,7 +51,7 @@  discard block
 block discarded – undo
51 51
 	public function flatten( array $reviewCounts )
52 52
 	{
53 53
 		$counts = [];
54
-		array_walk_recursive( $reviewCounts, function( $num, $index ) use( &$counts ) {
54
+		array_walk_recursive( $reviewCounts, function( $num, $index ) use(&$counts) {
55 55
 			$counts[$index] = isset($counts[$index])
56 56
 				? $num + $counts[$index]
57 57
 				: $num;
@@ -68,16 +68,16 @@  discard block
 block discarded – undo
68 68
 			'max' => Rating::MAX_RATING,
69 69
 			'min' => Rating::MIN_RATING,
70 70
 			'types' => 'local',
71
-		]);
71
+		] );
72 72
 		$counts = array_intersect_key(
73 73
 			glsr( OptionManager::class )->get( 'counts', [] ),
74
-			array_flip( array_intersect( array_keys( glsr()->reviewTypes ), (array)$args['types'] ))
74
+			array_flip( array_intersect( array_keys( glsr()->reviewTypes ), (array)$args['types'] ) )
75 75
 		);
76 76
 		$counts = $this->normalize( $counts );
77
-		array_walk( $counts, function( &$ratings ) use( $args ) {
77
+		array_walk( $counts, function( &$ratings ) use($args) {
78 78
 			$ratings[0] = 0;
79 79
 			foreach( $ratings as $index => &$num ) {
80
-				if( $index >= intval( $args['min'] ) && $index <= intval( $args['max'] ))continue;
80
+				if( $index >= intval( $args['min'] ) && $index <= intval( $args['max'] ) )continue;
81 81
 				$num = 0;
82 82
 			}
83 83
 		});
@@ -89,7 +89,7 @@  discard block
 block discarded – undo
89 89
 	 */
90 90
 	public function getFlattened( array $args = [] )
91 91
 	{
92
-		return $this->flatten( $this->get( $args ));
92
+		return $this->flatten( $this->get( $args ) );
93 93
 	}
94 94
 
95 95
 	/**
@@ -99,7 +99,7 @@  discard block
 block discarded – undo
99 99
 	{
100 100
 		foreach( $reviewCounts as $type => &$counts ) {
101 101
 			foreach( range( 0, Rating::MAX_RATING ) as $rating ) {
102
-				if( isset( $counts[$rating] ))continue;
102
+				if( isset($counts[$rating]) )continue;
103 103
 				$counts[$rating] = 0;
104 104
 			}
105 105
 			ksort( $counts );
Please login to merge, or discard this patch.
Braces   +9 added lines, -3 removed lines patch added patch discarded remove patch
@@ -18,7 +18,9 @@  discard block
 block discarded – undo
18 18
 		while( $reviews = glsr( SqlQueries::class )->getReviewRatings( $args, $greaterThanId, $limit )) {
19 19
 			$types = array_keys( array_flip( array_column( $reviews, 'type' )));
20 20
 			foreach( $types as $type ) {
21
-				if( isset( $counts[$type] ))continue;
21
+				if( isset( $counts[$type] )) {
22
+					continue;
23
+				}
22 24
 				$counts[$type] = array_fill_keys( range( 0, Rating::MAX_RATING ), 0 );
23 25
 			}
24 26
 			foreach( $reviews as $review ) {
@@ -77,7 +79,9 @@  discard block
 block discarded – undo
77 79
 		array_walk( $counts, function( &$ratings ) use( $args ) {
78 80
 			$ratings[0] = 0;
79 81
 			foreach( $ratings as $index => &$num ) {
80
-				if( $index >= intval( $args['min'] ) && $index <= intval( $args['max'] ))continue;
82
+				if( $index >= intval( $args['min'] ) && $index <= intval( $args['max'] )) {
83
+					continue;
84
+				}
81 85
 				$num = 0;
82 86
 			}
83 87
 		});
@@ -99,7 +103,9 @@  discard block
 block discarded – undo
99 103
 	{
100 104
 		foreach( $reviewCounts as $type => &$counts ) {
101 105
 			foreach( range( 0, Rating::MAX_RATING ) as $rating ) {
102
-				if( isset( $counts[$rating] ))continue;
106
+				if( isset( $counts[$rating] )) {
107
+					continue;
108
+				}
103 109
 				$counts[$rating] = 0;
104 110
 			}
105 111
 			ksort( $counts );
Please login to merge, or discard this patch.
plugin/Database.php 2 patches
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -20,10 +20,10 @@  discard block
 block discarded – undo
20 20
 	 */
21 21
 	public function getAssignedToPost( $postId, $assignedTo = '' )
22 22
 	{
23
-		if( empty( $assignedTo )) {
23
+		if( empty($assignedTo) ) {
24 24
 			$assignedTo = get_post_meta( $postId, 'assigned_to', true );
25 25
 		}
26
-		if( empty( $assignedTo ))return;
26
+		if( empty($assignedTo) )return;
27 27
 		$assignedPost = get_post( $assignedTo );
28 28
 		if( $assignedPost instanceof WP_Post && $assignedPost->ID != $postId ) {
29 29
 			return $assignedPost;
@@ -44,7 +44,7 @@  discard block
 block discarded – undo
44 44
 		if( !$metaValue ) {
45 45
 			return $counts;
46 46
 		}
47
-		return isset( $counts[$metaValue] )
47
+		return isset($counts[$metaValue])
48 48
 			? $counts[$metaValue]
49 49
 			: 0;
50 50
 	}
@@ -65,7 +65,7 @@  discard block
 block discarded – undo
65 65
 	 */
66 66
 	public function getReviewsMeta( $key, $status = 'publish' )
67 67
 	{
68
-		if( $status == 'all' || empty( $status )) {
68
+		if( $status == 'all' || empty($status) ) {
69 69
 			$status = get_post_stati( ['exclude_from_search' => false] );
70 70
 		}
71 71
 		return glsr( SqlQueries::class )->getReviewsMeta( $key, $status );
@@ -80,10 +80,10 @@  discard block
 block discarded – undo
80 80
 			'fields' => 'id=>name',
81 81
 			'hide_empty' => false,
82 82
 			'taxonomy' => Application::TAXONOMY,
83
-		]);
84
-		unset( $args['count'] ); // we don't want a term count
83
+		] );
84
+		unset($args['count']); // we don't want a term count
85 85
 		$terms = get_terms( $args );
86
-		if( is_wp_error( $terms )) {
86
+		if( is_wp_error( $terms ) ) {
87 87
 			glsr_log()->error( $terms->get_error_message() );
88 88
 			return [];
89 89
 		}
@@ -100,7 +100,7 @@  discard block
 block discarded – undo
100 100
 			'post_status' => 'publish',
101 101
 			'post_type' => 'any',
102 102
 		];
103
-		if( is_numeric( $searchTerm )) {
103
+		if( is_numeric( $searchTerm ) ) {
104 104
 			$args['post__in'] = [$searchTerm];
105 105
 		}
106 106
 		else {
@@ -119,9 +119,9 @@  discard block
 block discarded – undo
119 119
 			ob_start();
120 120
 			glsr()->render( 'partials/editor/search-result', [
121 121
 				'ID' => get_the_ID(),
122
-				'permalink' => esc_url( (string) get_permalink() ),
122
+				'permalink' => esc_url( (string)get_permalink() ),
123 123
 				'title' => esc_attr( get_the_title() ),
124
-			]);
124
+			] );
125 125
 			$results .= ob_get_clean();
126 126
 		}
127 127
 		wp_reset_postdata();
Please login to merge, or discard this patch.
Braces   +6 added lines, -2 removed lines patch added patch discarded remove patch
@@ -23,7 +23,9 @@  discard block
 block discarded – undo
23 23
 		if( empty( $assignedTo )) {
24 24
 			$assignedTo = get_post_meta( $postId, 'assigned_to', true );
25 25
 		}
26
-		if( empty( $assignedTo ))return;
26
+		if( empty( $assignedTo )) {
27
+			return;
28
+		}
27 29
 		$assignedPost = get_post( $assignedTo );
28 30
 		if( $assignedPost instanceof WP_Post && $assignedPost->ID != $postId ) {
29 31
 			return $assignedPost;
@@ -112,7 +114,9 @@  discard block
 block discarded – undo
112 114
 		add_filter( 'posts_search', [$queryBuilder, 'filterSearchByTitle'], 500, 2 );
113 115
 		$search = new WP_Query( $args );
114 116
 		remove_filter( 'posts_search', [$queryBuilder, 'filterSearchByTitle'], 500 );
115
-		if( !$search->have_posts() )return;
117
+		if( !$search->have_posts() ) {
118
+			return;
119
+		}
116 120
 		$results = '';
117 121
 		while( $search->have_posts() ) {
118 122
 			$search->the_post();
Please login to merge, or discard this patch.
plugin/Controllers/EditorController/Metaboxes.php 2 patches
Spacing   +24 added lines, -24 removed lines patch added patch discarded remove patch
@@ -86,7 +86,7 @@  discard block
 block discarded – undo
86 86
 	 */
87 87
 	public function saveAssignedToMetabox( $postId )
88 88
 	{
89
-		if( !wp_verify_nonce( glsr( Helper::class )->filterInput( '_nonce-assigned-to' ), 'assigned_to' ))return;
89
+		if( !wp_verify_nonce( glsr( Helper::class )->filterInput( '_nonce-assigned-to' ), 'assigned_to' ) )return;
90 90
 		$assignedTo = glsr( Helper::class )->filterInput( 'assigned_to' );
91 91
 		$assignedTo || $assignedTo = '';
92 92
 		if( get_post_meta( $postId, 'assigned_to', true ) != $assignedTo ) {
@@ -101,14 +101,14 @@  discard block
 block discarded – undo
101 101
 	 */
102 102
 	public function saveResponseMetabox( $postId )
103 103
 	{
104
-		if( !wp_verify_nonce( glsr( Helper::class )->filterInput( '_nonce-response' ), 'response' ))return;
104
+		if( !wp_verify_nonce( glsr( Helper::class )->filterInput( '_nonce-response' ), 'response' ) )return;
105 105
 		$response = glsr( Helper::class )->filterInput( 'response' );
106 106
 		$response || $response = '';
107 107
 		update_post_meta( $postId, 'response', trim( wp_kses( $response, [
108 108
 			'a' => ['href' => [], 'title' => []],
109 109
 			'em' => [],
110 110
 			'strong' => [],
111
-		])));
111
+		] ) ) );
112 112
 	}
113 113
 
114 114
 	/**
@@ -117,8 +117,8 @@  discard block
 block discarded – undo
117 117
 	 */
118 118
 	protected function getAssignedToPostId( $postId )
119 119
 	{
120
-		$assignedTo = intval( get_post_meta( $postId, 'assigned_to', true ));
121
-		if(( $post = get_post( $assignedTo )) instanceof WP_Post ) {
120
+		$assignedTo = intval( get_post_meta( $postId, 'assigned_to', true ) );
121
+		if( ($post = get_post( $assignedTo )) instanceof WP_Post ) {
122 122
 			return $post->ID;
123 123
 		}
124 124
 		return false;
@@ -132,12 +132,12 @@  discard block
 block discarded – undo
132 132
 		$meta = wp_parse_args( $meta, [
133 133
 			'rating' => get_post_meta( $review->ID, 'rating', true ),
134 134
 			'review_type' => get_post_meta( $review->ID, 'review_type', true ),
135
-		]);
136
-		return glsr( CountsManager::class )->get([
135
+		] );
136
+		return glsr( CountsManager::class )->get( [
137 137
 			'max' => $meta['rating'],
138 138
 			'min' => 0,
139 139
 			'types' => $meta['review_type'],
140
-		]);
140
+		] );
141 141
 	}
142 142
 
143 143
 	/**
@@ -145,8 +145,8 @@  discard block
 block discarded – undo
145 145
 	 */
146 146
 	protected function setReviewCounts( WP_Post $review, array $counts )
147 147
 	{
148
-		$type = strval( get_post_meta( $review->ID, 'review_type', true ));
149
-		if( !array_key_exists( $type, glsr()->reviewTypes ))return;
148
+		$type = strval( get_post_meta( $review->ID, 'review_type', true ) );
149
+		if( !array_key_exists( $type, glsr()->reviewTypes ) )return;
150 150
 		glsr( OptionManager::class )->set( 'counts.'.$type, $counts );
151 151
 	}
152 152
 
@@ -155,10 +155,10 @@  discard block
 block discarded – undo
155 155
 	 */
156 156
 	protected function increaseReviewCount( WP_Post $review, array $meta = [] )
157 157
 	{
158
-		if( $counts = $this->getReviewCounts( $review, $meta )) {
159
-			$rating = isset( $meta['rating'] )
158
+		if( $counts = $this->getReviewCounts( $review, $meta ) ) {
159
+			$rating = isset($meta['rating'])
160 160
 				? $meta['rating']
161
-				: intval( get_post_meta( $review->ID, 'rating', true ));
161
+				: intval( get_post_meta( $review->ID, 'rating', true ) );
162 162
 			$counts[$rating] -= 1;
163 163
 			$this->setReviewCounts( $review, $counts );
164 164
 		}
@@ -169,8 +169,8 @@  discard block
 block discarded – undo
169 169
 	 */
170 170
 	protected function decreaseReviewCount( WP_Post $review, array $meta = [] )
171 171
 	{
172
-		if( $counts = $this->getReviewCounts( $review, $meta )) {
173
-			$rating = intval( get_post_meta( $review->ID, 'rating', true ));
172
+		if( $counts = $this->getReviewCounts( $review, $meta ) ) {
173
+			$rating = intval( get_post_meta( $review->ID, 'rating', true ) );
174 174
 			$counts[$rating] += 1;
175 175
 			$this->setReviewCounts( $review, $counts );
176 176
 		}
@@ -197,20 +197,20 @@  discard block
 block discarded – undo
197 197
 	 */
198 198
 	protected function updateAssignedToPost( WP_Post $review )
199 199
 	{
200
-		if( !( $postId = $this->getAssignedToPostId( $review->ID )))return;
201
-		$reviewIds = array_filter( (array)get_post_meta( $postId, static::META_REVIEW_ID ));
202
-		if( empty( $reviewIds ))return;
200
+		if( !($postId = $this->getAssignedToPostId( $review->ID )) )return;
201
+		$reviewIds = array_filter( (array)get_post_meta( $postId, static::META_REVIEW_ID ) );
202
+		if( empty($reviewIds) )return;
203 203
 		$this->updateReviewIdOfPost( $postId, $review, $reviewIds );
204
-		$updatedReviewIds = array_filter( (array)get_post_meta( $postId, static::META_REVIEW_ID ));
205
-		if( empty( $updatedReviewIds )) {
204
+		$updatedReviewIds = array_filter( (array)get_post_meta( $postId, static::META_REVIEW_ID ) );
205
+		if( empty($updatedReviewIds) ) {
206 206
 			delete_post_meta( $postId, static::META_RANKING );
207 207
 			delete_post_meta( $postId, static::META_REVIEW_ID );
208 208
 		}
209
-		else if( !glsr( Helper::class )->compareArrays( $reviewIds, $updatedReviewIds )) {
209
+		else if( !glsr( Helper::class )->compareArrays( $reviewIds, $updatedReviewIds ) ) {
210 210
 			$counts = glsr( CountsManager::class )->buildFromIds( $updatedReviewIds );
211 211
 			$counts = glsr( CountsManager::class )->flatten( $counts );
212
-			update_post_meta( $postId, static::META_AVERAGE, $this->recalculatePostAverage( $counts ));
213
-			update_post_meta( $postId, static::META_RANKING, $this->recalculatePostRanking( $counts ));
212
+			update_post_meta( $postId, static::META_AVERAGE, $this->recalculatePostAverage( $counts ) );
213
+			update_post_meta( $postId, static::META_RANKING, $this->recalculatePostRanking( $counts ) );
214 214
 		}
215 215
 	}
216 216
 
@@ -223,7 +223,7 @@  discard block
 block discarded – undo
223 223
 		if( $review->post_status != 'publish' ) {
224 224
 			delete_post_meta( $postId, static::META_REVIEW_ID, $review->ID );
225 225
 		}
226
-		else if( !in_array( $review->ID, $reviewIds )) {
226
+		else if( !in_array( $review->ID, $reviewIds ) ) {
227 227
 			add_post_meta( $postId, static::META_REVIEW_ID, $review->ID );
228 228
 		}
229 229
 	}
Please login to merge, or discard this patch.
Braces   +24 added lines, -8 removed lines patch added patch discarded remove patch
@@ -21,7 +21,9 @@  discard block
 block discarded – undo
21 21
 	 */
22 22
 	public function onBeforeDeleteReview( $postId )
23 23
 	{
24
-		if( get_post_field( 'post_type', $postId ) !== Application::POST_TYPE )return;
24
+		if( get_post_field( 'post_type', $postId ) !== Application::POST_TYPE ) {
25
+			return;
26
+		}
25 27
 		$review = get_post( $postId );
26 28
 		$review->post_status = 'deleted'; // change post_status first!
27 29
 		$this->updateAssignedToPost( $review );
@@ -37,7 +39,9 @@  discard block
 block discarded – undo
37 39
 	public function onBeforeUpdateReview( WP_Post $review, $metaKey, $newValue )
38 40
 	{
39 41
 		$previousValue = get_post_meta( $review->ID, $metaKey, true );
40
-		if( $previousValue == $newValue )return;
42
+		if( $previousValue == $newValue ) {
43
+			return;
44
+		}
41 45
 		$this->decreaseReviewCount( $review );
42 46
 		$this->increaseReviewCount( $review, [$metaKey => $newValue] );
43 47
 	}
@@ -50,7 +54,9 @@  discard block
 block discarded – undo
50 54
 	 */
51 55
 	public function onCreateReview( $postData, $meta, $postId )
52 56
 	{
53
-		if( get_post_field( 'post_type', $postId ) !== Application::POST_TYPE )return;
57
+		if( get_post_field( 'post_type', $postId ) !== Application::POST_TYPE ) {
58
+			return;
59
+		}
54 60
 		$review = get_post( $postId );
55 61
 		$this->updateAssignedToPost( $review );
56 62
 		$this->increaseReviewCount( $review );
@@ -86,7 +92,9 @@  discard block
 block discarded – undo
86 92
 	 */
87 93
 	public function saveAssignedToMetabox( $postId )
88 94
 	{
89
-		if( !wp_verify_nonce( glsr( Helper::class )->filterInput( '_nonce-assigned-to' ), 'assigned_to' ))return;
95
+		if( !wp_verify_nonce( glsr( Helper::class )->filterInput( '_nonce-assigned-to' ), 'assigned_to' )) {
96
+			return;
97
+		}
90 98
 		$assignedTo = glsr( Helper::class )->filterInput( 'assigned_to' );
91 99
 		$assignedTo || $assignedTo = '';
92 100
 		if( get_post_meta( $postId, 'assigned_to', true ) != $assignedTo ) {
@@ -101,7 +109,9 @@  discard block
 block discarded – undo
101 109
 	 */
102 110
 	public function saveResponseMetabox( $postId )
103 111
 	{
104
-		if( !wp_verify_nonce( glsr( Helper::class )->filterInput( '_nonce-response' ), 'response' ))return;
112
+		if( !wp_verify_nonce( glsr( Helper::class )->filterInput( '_nonce-response' ), 'response' )) {
113
+			return;
114
+		}
105 115
 		$response = glsr( Helper::class )->filterInput( 'response' );
106 116
 		$response || $response = '';
107 117
 		update_post_meta( $postId, 'response', trim( wp_kses( $response, [
@@ -146,7 +156,9 @@  discard block
 block discarded – undo
146 156
 	protected function setReviewCounts( WP_Post $review, array $counts )
147 157
 	{
148 158
 		$type = strval( get_post_meta( $review->ID, 'review_type', true ));
149
-		if( !array_key_exists( $type, glsr()->reviewTypes ))return;
159
+		if( !array_key_exists( $type, glsr()->reviewTypes )) {
160
+			return;
161
+		}
150 162
 		glsr( OptionManager::class )->set( 'counts.'.$type, $counts );
151 163
 	}
152 164
 
@@ -197,9 +209,13 @@  discard block
 block discarded – undo
197 209
 	 */
198 210
 	protected function updateAssignedToPost( WP_Post $review )
199 211
 	{
200
-		if( !( $postId = $this->getAssignedToPostId( $review->ID )))return;
212
+		if( !( $postId = $this->getAssignedToPostId( $review->ID ))) {
213
+			return;
214
+		}
201 215
 		$reviewIds = array_filter( (array)get_post_meta( $postId, static::META_REVIEW_ID ));
202
-		if( empty( $reviewIds ))return;
216
+		if( empty( $reviewIds )) {
217
+			return;
218
+		}
203 219
 		$this->updateReviewIdOfPost( $postId, $review, $reviewIds );
204 220
 		$updatedReviewIds = array_filter( (array)get_post_meta( $postId, static::META_REVIEW_ID ));
205 221
 		if( empty( $updatedReviewIds )) {
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
@@ -22,7 +22,7 @@  discard block
 block discarded – undo
22 22
 	 */
23 23
 	public function routerChangeReviewStatus( 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 routerSearchPosts( array $request )
78 78
 	{
79 79
 		$results = glsr( Database::class )->searchPosts( $request['search'] );
80
-		wp_send_json_success([
80
+		wp_send_json_success( [
81 81
 			'empty' => '<div>'.__( 'Nothing found.', 'site-reviews' ).'</div>',
82 82
 			'items' => $results,
83
-		]);
83
+		] );
84 84
 	}
85 85
 
86 86
 	/**
@@ -88,7 +88,7 @@  discard block
 block discarded – undo
88 88
 	 */
89 89
 	public function routerSearchTranslations( array $request )
90 90
 	{
91
-		if( empty( $request['exclude'] )) {
91
+		if( empty($request['exclude']) ) {
92 92
 			$request['exclude'] = [];
93 93
 		}
94 94
 		$results = glsr( Translator::class )
@@ -96,10 +96,10 @@  discard block
 block discarded – undo
96 96
 			->exclude()
97 97
 			->exclude( $request['exclude'] )
98 98
 			->renderResults();
99
-		wp_send_json_success([
99
+		wp_send_json_success( [
100 100
 			'empty' => '<div>'.__( 'Nothing found.', 'site-reviews' ).'</div>',
101 101
 			'items' => $results,
102
-		]);
102
+		] );
103 103
 	}
104 104
 
105 105
 	/**
@@ -113,7 +113,7 @@  discard block
 block discarded – undo
113 113
 			'message' => glsr( Session::class )->get( $request['form_id'].'message', '', true ),
114 114
 			'recaptcha' => glsr( Session::class )->get( $request['form_id'].'recaptcha', false, true ),
115 115
 		];
116
-		if( !wp_validate_boolean( $data['errors'] )) {
116
+		if( !wp_validate_boolean( $data['errors'] ) ) {
117 117
 			glsr( Session::class )->clear();
118 118
 			wp_send_json_success( $data );
119 119
 		}
@@ -125,10 +125,10 @@  discard block
 block discarded – undo
125 125
 	 */
126 126
 	public function routerTogglePinned( array $request )
127 127
 	{
128
-		$isPinned = $this->execute( new TogglePinned( $request ));
129
-		wp_send_json_success([
128
+		$isPinned = $this->execute( new TogglePinned( $request ) );
129
+		wp_send_json_success( [
130 130
 			'notices' => glsr( Notice::class )->get(),
131 131
 			'pinned' => $isPinned,
132
-		]);
132
+		] );
133 133
 	}
134 134
 }
Please login to merge, or discard this patch.
plugin/Handlers/CreateReview.php 2 patches
Spacing   +12 added lines, -12 removed lines patch added patch discarded remove patch
@@ -28,12 +28,12 @@  discard block
 block discarded – undo
28 28
 		$postId = glsr( ReviewManager::class )->create( $command );
29 29
 		if( $postId < 1 ) {
30 30
 			glsr( Session::class )->set( $command->form_id.'errors', [] );
31
-			glsr( Session::class )->set( $command->form_id.'message', __( 'Your review could not be submitted and the error has been logged. Please notify the site admin.', 'site-reviews' ));
31
+			glsr( Session::class )->set( $command->form_id.'message', __( 'Your review could not be submitted and the error has been logged. Please notify the site admin.', 'site-reviews' ) );
32 32
 			return;
33 33
 		}
34 34
 		$this->sendNotification( $postId );
35 35
 		do_action( 'site-reviews/local/review/submitted', $postId, $command );
36
-		glsr( Session::class )->set( $command->form_id.'message', __( 'Your review has been submitted!', 'site-reviews' ));
36
+		glsr( Session::class )->set( $command->form_id.'message', __( 'Your review has been submitted!', 'site-reviews' ) );
37 37
 		if( $command->ajax_request )return;
38 38
 		wp_safe_redirect( $command->referrer );
39 39
 		exit;
@@ -80,8 +80,8 @@  discard block
 block discarded – undo
80 80
 		if( $this->command->author ) {
81 81
 			$fields[] = ['value' => trim( $this->command->author.$this->command->email.' - '.$this->command->ip_address )];
82 82
 		}
83
-		$fields[] = ['value' => sprintf( '<%s|%s>', $args['notification_link'], __( 'View Review', 'site-reviews' ))];
84
-		return json_encode([
83
+		$fields[] = ['value' => sprintf( '<%s|%s>', $args['notification_link'], __( 'View Review', 'site-reviews' ) )];
84
+		return json_encode( [
85 85
 			'icon_url' => glsr()->url( 'assets/img/icon.png' ),
86 86
 			'username' => glsr()->name,
87 87
 			'attachments' => [[
@@ -90,7 +90,7 @@  discard block
 block discarded – undo
90 90
 				'fallback' => $this->createEmailNotification( $args )->read( 'plaintext' ),
91 91
 				'fields' => $fields,
92 92
 			]],
93
-		]);
93
+		] );
94 94
 	}
95 95
 
96 96
 	/**
@@ -100,12 +100,12 @@  discard block
 block discarded – undo
100 100
 	protected function sendNotification( $postId )
101 101
 	{
102 102
 		$notificationType = glsr( OptionManager::class )->get( 'settings.general.notification' );
103
-		if( !in_array( $notificationType, ['default','custom','webhook'] ))return;
103
+		if( !in_array( $notificationType, ['default', 'custom', 'webhook'] ) )return;
104 104
 		$assignedToTitle = get_the_title( (int)$this->command->assigned_to );
105 105
 		$notificationSubject = _nx(
106 106
 			'New %s-star review',
107 107
 			'New %s-star review of: %s',
108
-			intval( empty( $assignedToTitle )),
108
+			intval( empty($assignedToTitle) ),
109 109
 			'The text is different depending on whether or not the review has been assigned to a post.',
110 110
 			'site-reviews'
111 111
 		);
@@ -114,7 +114,7 @@  discard block
 block discarded – undo
114 114
 			sprintf( $notificationSubject, $this->command->rating, $assignedToTitle )
115 115
 		);
116 116
 		$args = [
117
-			'notification_link' => esc_url( admin_url( sprintf( 'post.php?post=%s&action=edit', $postId ))),
117
+			'notification_link' => esc_url( admin_url( sprintf( 'post.php?post=%s&action=edit', $postId ) ) ),
118 118
 			'notification_title' => $notificationTitle,
119 119
 			'notification_type' => $notificationType,
120 120
 		];
@@ -132,7 +132,7 @@  discard block
 block discarded – undo
132 132
 		$args['recipient'] = $args['notification_type'] !== 'default'
133 133
 			? glsr( OptionManager::class )->get( 'settings.general.notification_email' )
134 134
 			: get_option( 'admin_email' );
135
-		if( empty( $args['recipient'] )) {
135
+		if( empty($args['recipient']) ) {
136 136
 			glsr_log()->error( 'Email notification was not sent: missing email, subject, or message.' );
137 137
 		}
138 138
 		else {
@@ -148,7 +148,7 @@  discard block
 block discarded – undo
148 148
 	 */
149 149
 	protected function sendWebhookNotification( array $args )
150 150
 	{
151
-		if( !( $endpoint = glsr( OptionManager::class )->get( 'settings.general.webhook_url' )))return;
151
+		if( !($endpoint = glsr( OptionManager::class )->get( 'settings.general.webhook_url' )) )return;
152 152
 		$notification = $this->createWebhookNotification( $args );
153 153
 		$result = wp_remote_post( $endpoint, [
154 154
 			'blocking' => false,
@@ -159,8 +159,8 @@  discard block
 block discarded – undo
159 159
 			'redirection' => 5,
160 160
 			'sslverify' => false,
161 161
 			'timeout' => 45,
162
-		]);
163
-		if( is_wp_error( $result )) {
162
+		] );
163
+		if( is_wp_error( $result ) ) {
164 164
 			glsr_log()->error( $result->get_error_message() );
165 165
 		}
166 166
 	}
Please login to merge, or discard this patch.
Braces   +9 added lines, -3 removed lines patch added patch discarded remove patch
@@ -34,7 +34,9 @@  discard block
 block discarded – undo
34 34
 		$this->sendNotification( $postId );
35 35
 		do_action( 'site-reviews/local/review/submitted', $postId, $command );
36 36
 		glsr( Session::class )->set( $command->form_id.'message', __( 'Your review has been submitted!', 'site-reviews' ));
37
-		if( $command->ajax_request )return;
37
+		if( $command->ajax_request ) {
38
+			return;
39
+		}
38 40
 		wp_safe_redirect( $command->referrer );
39 41
 		exit;
40 42
 	}
@@ -100,7 +102,9 @@  discard block
 block discarded – undo
100 102
 	protected function sendNotification( $postId )
101 103
 	{
102 104
 		$notificationType = glsr( OptionManager::class )->get( 'settings.general.notification' );
103
-		if( !in_array( $notificationType, ['default','custom','webhook'] ))return;
105
+		if( !in_array( $notificationType, ['default','custom','webhook'] )) {
106
+			return;
107
+		}
104 108
 		$assignedToTitle = get_the_title( (int)$this->command->assigned_to );
105 109
 		$notificationSubject = _nx(
106 110
 			'New %s-star review',
@@ -148,7 +152,9 @@  discard block
 block discarded – undo
148 152
 	 */
149 153
 	protected function sendWebhookNotification( array $args )
150 154
 	{
151
-		if( !( $endpoint = glsr( OptionManager::class )->get( 'settings.general.webhook_url' )))return;
155
+		if( !( $endpoint = glsr( OptionManager::class )->get( 'settings.general.webhook_url' ))) {
156
+			return;
157
+		}
152 158
 		$notification = $this->createWebhookNotification( $args );
153 159
 		$result = wp_remote_post( $endpoint, [
154 160
 			'blocking' => false,
Please login to merge, or discard this patch.
plugin/Modules/Html/Partials/SiteReviewsSummary.php 2 patches
Spacing   +18 added lines, -18 removed lines patch added patch discarded remove patch
@@ -34,9 +34,9 @@  discard block
 block discarded – undo
34 34
 		$this->args = $args;
35 35
 
36 36
 		// get rating counts
37
-		$this->ratingCounts = [0,0,0,0,0,0];
37
+		$this->ratingCounts = [0, 0, 0, 0, 0, 0];
38 38
 
39
-		if( !array_sum( $this->ratingCounts ) && $this->isHidden( 'if_empty' ))return;
39
+		if( !array_sum( $this->ratingCounts ) && $this->isHidden( 'if_empty' ) )return;
40 40
 		$this->averageRating = glsr( Rating::class )->getAverage( $this->ratingCounts );
41 41
 		$this->generateSchema();
42 42
 		return glsr( Template::class )->build( 'templates/reviews-summary', [
@@ -48,7 +48,7 @@  discard block
 block discarded – undo
48 48
 				'stars' => $this->buildStars(),
49 49
 				'text' => $this->buildText(),
50 50
 			],
51
-		]);
51
+		] );
52 52
 	}
53 53
 
54 54
 	/**
@@ -56,16 +56,16 @@  discard block
 block discarded – undo
56 56
 	 */
57 57
 	protected function buildPercentage()
58 58
 	{
59
-		if( $this->isHidden( 'bars' ))return;
59
+		if( $this->isHidden( 'bars' ) )return;
60 60
 		$range = range( Rating::MAX_RATING, 1 );
61
-		$percentages = preg_filter( '/$/', '%', glsr( Rating::class )->getPercentages( $this->ratingCounts ));
62
-		$bars = array_reduce( $range, function( $carry, $level ) use( $percentages ) {
61
+		$percentages = preg_filter( '/$/', '%', glsr( Rating::class )->getPercentages( $this->ratingCounts ) );
62
+		$bars = array_reduce( $range, function( $carry, $level ) use($percentages) {
63 63
 			$label = $this->buildPercentageLabel( $this->args['labels'][$level] );
64 64
 			$background = $this->buildPercentageBackground( $percentages[$level] );
65 65
 			$percent = $this->buildPercentagePercent( $percentages[$level] );
66 66
 			return $carry.glsr( Builder::class )->div( $label.$background.$percent, [
67 67
 				'class' => 'glsr-bar',
68
-			]);
68
+			] );
69 69
 		});
70 70
 		return $this->wrap( 'percentage', $bars );
71 71
 	}
@@ -76,10 +76,10 @@  discard block
 block discarded – undo
76 76
 	 */
77 77
 	protected function buildPercentageBackground( $percent )
78 78
 	{
79
-		$backgroundPercent = glsr( Builder::class )->span([
79
+		$backgroundPercent = glsr( Builder::class )->span( [
80 80
 			'class' => 'glsr-bar-background-percent',
81 81
 			'style' => 'width:'.$percent,
82
-		]);
82
+		] );
83 83
 		return '<span class="glsr-bar-background">'.$backgroundPercent.'</span>';
84 84
 	}
85 85
 
@@ -106,7 +106,7 @@  discard block
 block discarded – undo
106 106
 	 */
107 107
 	protected function buildRating()
108 108
 	{
109
-		if( $this->isHidden( 'rating' ))return;
109
+		if( $this->isHidden( 'rating' ) )return;
110 110
 		return $this->wrap( 'rating', '<span>'.$this->averageRating.'</span>' );
111 111
 	}
112 112
 
@@ -115,10 +115,10 @@  discard block
 block discarded – undo
115 115
 	 */
116 116
 	protected function buildStars()
117 117
 	{
118
-		if( $this->isHidden( 'stars' ))return;
118
+		if( $this->isHidden( 'stars' ) )return;
119 119
 		$stars = glsr( Partial::class )->build( 'star-rating', [
120 120
 			'rating' => $this->averageRating,
121
-		]);
121
+		] );
122 122
 		return $this->wrap( 'stars', $stars );
123 123
 	}
124 124
 
@@ -127,9 +127,9 @@  discard block
 block discarded – undo
127 127
 	 */
128 128
 	protected function buildText()
129 129
 	{
130
-		if( $this->isHidden( 'summary' ))return;
131
-		$count = intval( array_sum( $this->ratingCounts ));
132
-		if( empty( $this->args['text'] )) {
130
+		if( $this->isHidden( 'summary' ) )return;
131
+		$count = intval( array_sum( $this->ratingCounts ) );
132
+		if( empty($this->args['text']) ) {
133 133
 			// @todo document this change
134 134
 			 $this->args['text'] = _nx(
135 135
 				'{rating} out of {max} stars (based on {num} review)',
@@ -140,7 +140,7 @@  discard block
 block discarded – undo
140 140
 			);
141 141
 		}
142 142
 		$summary = str_replace(
143
-			['{rating}','{max}', '{num}'],
143
+			['{rating}', '{max}', '{num}'],
144 144
 			[$this->averageRating, Rating::MAX_RATING, $count],
145 145
 			$this->args['text']
146 146
 		);
@@ -152,7 +152,7 @@  discard block
 block discarded – undo
152 152
 	 */
153 153
 	protected function generateSchema()
154 154
 	{
155
-		if( !wp_validate_boolean( $this->args['schema'] ))return;
155
+		if( !wp_validate_boolean( $this->args['schema'] ) )return;
156 156
 		glsr( Schema::class )->store(
157 157
 			glsr( Schema::class )->buildSummary( $this->args )
158 158
 		);
@@ -185,6 +185,6 @@  discard block
 block discarded – undo
185 185
 	{
186 186
 		return glsr( Builder::class )->div( $value, [
187 187
 			'class' => 'glsr-summary-'.$key,
188
-		]);
188
+		] );
189 189
 	}
190 190
 }
Please login to merge, or discard this patch.
Braces   +18 added lines, -6 removed lines patch added patch discarded remove patch
@@ -36,7 +36,9 @@  discard block
 block discarded – undo
36 36
 		// get rating counts
37 37
 		$this->ratingCounts = [0,0,0,0,0,0];
38 38
 
39
-		if( !array_sum( $this->ratingCounts ) && $this->isHidden( 'if_empty' ))return;
39
+		if( !array_sum( $this->ratingCounts ) && $this->isHidden( 'if_empty' )) {
40
+			return;
41
+		}
40 42
 		$this->averageRating = glsr( Rating::class )->getAverage( $this->ratingCounts );
41 43
 		$this->generateSchema();
42 44
 		return glsr( Template::class )->build( 'templates/reviews-summary', [
@@ -56,7 +58,9 @@  discard block
 block discarded – undo
56 58
 	 */
57 59
 	protected function buildPercentage()
58 60
 	{
59
-		if( $this->isHidden( 'bars' ))return;
61
+		if( $this->isHidden( 'bars' )) {
62
+			return;
63
+		}
60 64
 		$range = range( Rating::MAX_RATING, 1 );
61 65
 		$percentages = preg_filter( '/$/', '%', glsr( Rating::class )->getPercentages( $this->ratingCounts ));
62 66
 		$bars = array_reduce( $range, function( $carry, $level ) use( $percentages ) {
@@ -106,7 +110,9 @@  discard block
 block discarded – undo
106 110
 	 */
107 111
 	protected function buildRating()
108 112
 	{
109
-		if( $this->isHidden( 'rating' ))return;
113
+		if( $this->isHidden( 'rating' )) {
114
+			return;
115
+		}
110 116
 		return $this->wrap( 'rating', '<span>'.$this->averageRating.'</span>' );
111 117
 	}
112 118
 
@@ -115,7 +121,9 @@  discard block
 block discarded – undo
115 121
 	 */
116 122
 	protected function buildStars()
117 123
 	{
118
-		if( $this->isHidden( 'stars' ))return;
124
+		if( $this->isHidden( 'stars' )) {
125
+			return;
126
+		}
119 127
 		$stars = glsr( Partial::class )->build( 'star-rating', [
120 128
 			'rating' => $this->averageRating,
121 129
 		]);
@@ -127,7 +135,9 @@  discard block
 block discarded – undo
127 135
 	 */
128 136
 	protected function buildText()
129 137
 	{
130
-		if( $this->isHidden( 'summary' ))return;
138
+		if( $this->isHidden( 'summary' )) {
139
+			return;
140
+		}
131 141
 		$count = intval( array_sum( $this->ratingCounts ));
132 142
 		if( empty( $this->args['text'] )) {
133 143
 			// @todo document this change
@@ -152,7 +162,9 @@  discard block
 block discarded – undo
152 162
 	 */
153 163
 	protected function generateSchema()
154 164
 	{
155
-		if( !wp_validate_boolean( $this->args['schema'] ))return;
165
+		if( !wp_validate_boolean( $this->args['schema'] )) {
166
+			return;
167
+		}
156 168
 		glsr( Schema::class )->store(
157 169
 			glsr( Schema::class )->buildSummary( $this->args )
158 170
 		);
Please login to merge, or discard this patch.