Passed
Push — master ( 576dc5...fc7b10 )
by Paul
04:33
created
plugin/Application.php 2 patches
Braces   +6 added lines, -2 removed lines patch added patch discarded remove patch
@@ -153,7 +153,9 @@  discard block
 block discarded – undo
153 153
 	 */
154 154
 	public function scheduleCronJob()
155 155
 	{
156
-		if( wp_next_scheduled( static::CRON_EVENT ))return;
156
+		if( wp_next_scheduled( static::CRON_EVENT )) {
157
+			return;
158
+		}
157 159
 		wp_schedule_event( time(), 'twicedaily', static::CRON_EVENT );
158 160
 	}
159 161
 
@@ -184,7 +186,9 @@  discard block
 block discarded – undo
184 186
 			|| !in_array( plugin_basename( $this->file ), $data['plugins'] )
185 187
 			|| $data['action'] != 'update'
186 188
 			|| $data['type'] != 'plugin'
187
-		)return;
189
+		) {
190
+			return;
191
+		}
188 192
 		$this->upgrade();
189 193
 	}
190 194
 
Please login to merge, or discard this patch.
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -66,7 +66,7 @@  discard block
 block discarded – undo
66 66
 	 */
67 67
 	public function getDefaults()
68 68
 	{
69
-		if( empty( $this->defaults )) {
69
+		if( empty($this->defaults) ) {
70 70
 			$this->defaults = $this->make( DefaultsManager::class )->get();
71 71
 			$this->upgrade();
72 72
 		}
@@ -79,7 +79,7 @@  discard block
 block discarded – undo
79 79
 	public function hasPermission()
80 80
 	{
81 81
 		$isAdmin = $this->isAdmin();
82
-		return !$isAdmin || ( $isAdmin && current_user_can( static::CAPABILITY ));
82
+		return !$isAdmin || ($isAdmin && current_user_can( static::CAPABILITY ));
83 83
 	}
84 84
 
85 85
 	/**
@@ -134,7 +134,7 @@  discard block
 block discarded – undo
134 134
 		$types = apply_filters( 'site-reviews/addon/types', [] );
135 135
 		$this->reviewTypes = wp_parse_args( $types, [
136 136
 			'local' => __( 'Local', 'site-reviews' ),
137
-		]);
137
+		] );
138 138
 	}
139 139
 
140 140
 	/**
@@ -144,15 +144,15 @@  discard block
 block discarded – undo
144 144
 	public function render( $view, array $data = [] )
145 145
 	{
146 146
 		$file = '';
147
-		if( glsr( Helper::class )->startsWith( $view, 'templates/' )) {
147
+		if( glsr( Helper::class )->startsWith( $view, 'templates/' ) ) {
148 148
 			$file = str_replace( 'templates/', 'site-reviews/', $view ).'.php';
149 149
 			$file = get_stylesheet_directory().'/'.$file;
150 150
 		}
151
-		if( !file_exists( $file )) {
151
+		if( !file_exists( $file ) ) {
152 152
 			$file = $this->path( 'views/'.$view.'.php' );
153 153
 		}
154 154
 		$file = apply_filters( 'site-reviews/addon/views/file', $file, $view, $data );
155
-		if( !file_exists( $file )) {
155
+		if( !file_exists( $file ) ) {
156 156
 			glsr_log()->error( 'File not found: '.$file );
157 157
 			return;
158 158
 		}
@@ -166,7 +166,7 @@  discard block
 block discarded – undo
166 166
 	 */
167 167
 	public function scheduleCronJob()
168 168
 	{
169
-		if( wp_next_scheduled( static::CRON_EVENT ))return;
169
+		if( wp_next_scheduled( static::CRON_EVENT ) )return;
170 170
 		wp_schedule_event( time(), 'twicedaily', static::CRON_EVENT );
171 171
 	}
172 172
 
@@ -175,7 +175,7 @@  discard block
 block discarded – undo
175 175
 	 */
176 176
 	public function unscheduleCronJob()
177 177
 	{
178
-		wp_unschedule_event( intval( wp_next_scheduled( static::CRON_EVENT )), static::CRON_EVENT );
178
+		wp_unschedule_event( intval( wp_next_scheduled( static::CRON_EVENT ) ), static::CRON_EVENT );
179 179
 	}
180 180
 
181 181
 	/**
@@ -207,6 +207,6 @@  discard block
 block discarded – undo
207 207
 	 */
208 208
 	public function url( $path = '' )
209 209
 	{
210
-		return esc_url( plugin_dir_url( $this->file ).ltrim( trim( $path ), '/' ));
210
+		return esc_url( plugin_dir_url( $this->file ).ltrim( trim( $path ), '/' ) );
211 211
 	}
212 212
 }
Please login to merge, or discard this patch.
plugin/Shortcodes/Shortcode.php 2 patches
Braces   +6 added lines, -2 removed lines patch added patch discarded remove patch
@@ -76,7 +76,9 @@  discard block
 block discarded – undo
76 76
 		$args = shortcode_atts( $this->getDefaults(), wp_parse_args( $args ));
77 77
 		array_walk( $args, function( &$value, $key ) {
78 78
 			$methodName = glsr( Helper::class )->buildMethodName( $key, 'normalize' );
79
-			if( !method_exists( $this, $methodName ))return;
79
+			if( !method_exists( $this, $methodName )) {
80
+				return;
81
+			}
80 82
 			$value = $this->$methodName( $value );
81 83
 		});
82 84
 		return $this->sanitize( $args );
@@ -141,7 +143,9 @@  discard block
 block discarded – undo
141 143
 		$defaults = array_pad( $defaults, Rating::MAX_RATING, '' );
142 144
 		$labels = array_map( 'trim', explode( ',', $labels ));
143 145
 		foreach( $defaults as $i => $label ) {
144
-			if( empty( $labels[$i] ))continue;
146
+			if( empty( $labels[$i] )) {
147
+				continue;
148
+			}
145 149
 			$defaults[$i] = $labels[$i];
146 150
 		}
147 151
 		return array_combine( range( Rating::MAX_RATING, 1 ), $defaults );
Please login to merge, or discard this patch.
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -22,10 +22,10 @@  discard block
 block discarded – undo
22 22
 			'after_widget' => '</div>',
23 23
 			'before_title' => '<h3 class="glsr-shortcode-title">',
24 24
 			'after_title' => '</h3>',
25
-		]);
25
+		] );
26 26
 		$instance = $this->normalize( $instance );
27 27
 		$partial = glsr( Html::class )->buildPartial( $shortcodePartial, $instance );
28
-		if( !empty( $instance['title'] )) {
28
+		if( !empty($instance['title']) ) {
29 29
 			$instance['title'] = $args['before_title'].$instance['title'].$args['after_title'];
30 30
 		}
31 31
 		return $args['before_widget'].$instance['title'].$partial.$args['after_widget'];
@@ -68,10 +68,10 @@  discard block
 block discarded – undo
68 68
 	 */
69 69
 	public function normalize( $args )
70 70
 	{
71
-		$args = shortcode_atts( $this->getDefaults(), wp_parse_args( $args ));
71
+		$args = shortcode_atts( $this->getDefaults(), wp_parse_args( $args ) );
72 72
 		array_walk( $args, function( &$value, $key ) {
73 73
 			$methodName = glsr( Helper::class )->buildMethodName( $key, 'normalize' );
74
-			if( !method_exists( $this, $methodName ))return;
74
+			if( !method_exists( $this, $methodName ) )return;
75 75
 			$value = $this->$methodName( $value );
76 76
 		});
77 77
 		return $this->sanitize( $args );
@@ -103,13 +103,13 @@  discard block
 block discarded – undo
103 103
 	 */
104 104
 	protected function normalizeHide( $hide )
105 105
 	{
106
-		if( is_string( $hide )) {
106
+		if( is_string( $hide ) ) {
107 107
 			$hide = explode( ',', $hide );
108 108
 		}
109 109
 		$hiddenKeys = defined( 'static::HIDDEN_KEYS' )
110 110
 			? static::HIDDEN_KEYS
111 111
 			: [];
112
-		return array_filter( array_map( 'trim', $hide ), function( $value ) use( $hiddenKeys ) {
112
+		return array_filter( array_map( 'trim', $hide ), function( $value ) use($hiddenKeys) {
113 113
 			return in_array( $value, $hiddenKeys );
114 114
 		});
115 115
 	}
@@ -137,9 +137,9 @@  discard block
 block discarded – undo
137 137
 			__( 'Terrible', 'site-reviews' ),
138 138
 		];
139 139
 		$defaults = array_pad( $defaults, Rating::MAX_RATING, '' );
140
-		$labels = array_map( 'trim', explode( ',', $labels ));
140
+		$labels = array_map( 'trim', explode( ',', $labels ) );
141 141
 		foreach( $defaults as $i => $label ) {
142
-			if( empty( $labels[$i] ))continue;
142
+			if( empty($labels[$i]) )continue;
143 143
 			$defaults[$i] = $labels[$i];
144 144
 		}
145 145
 		return array_combine( range( Rating::MAX_RATING, 1 ), $defaults );
Please login to merge, or discard this patch.
plugin/Commands/RegisterPostType.php 2 patches
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -29,7 +29,9 @@
 block discarded – undo
29 29
 	{
30 30
 		foreach( $args as $key => $value ) {
31 31
 			$property = glsr( Helper::class )->buildPropertyName( $key );
32
-			if( !property_exists( $this, $property ))continue;
32
+			if( !property_exists( $this, $property )) {
33
+				continue;
34
+			}
33 35
 			$this->$property = $value;
34 36
 			unset( $args[$key] );
35 37
 		}
Please login to merge, or discard this patch.
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -30,13 +30,13 @@  discard block
 block discarded – undo
30 30
 	{
31 31
 		foreach( $args as $key => $value ) {
32 32
 			$property = glsr( Helper::class )->buildPropertyName( $key );
33
-			if( !property_exists( $this, $property ))continue;
33
+			if( !property_exists( $this, $property ) )continue;
34 34
 			$this->$property = $value;
35
-			unset( $args[$key] );
35
+			unset($args[$key]);
36 36
 		}
37 37
 		$this->args = wp_parse_args( $args, [
38 38
 			'menu_name' => $this->plural,
39
-		]);
39
+		] );
40 40
 	}
41 41
 
42 42
 	/**
@@ -59,8 +59,8 @@  discard block
 block discarded – undo
59 59
 			'singular_name' => $this->single,
60 60
 			'uploaded_to_this_item' => sprintf( _x( 'Uploaded to this %s', 'Uploaded to this Post', 'site-reviews' ), $this->single ),
61 61
 			'view_item' => sprintf( _x( 'View %s', 'View Post', 'site-reviews' ), $this->single ),
62
-		]);
63
-		unset( $this->args['menu_name'] );
62
+		] );
63
+		unset($this->args['menu_name']);
64 64
 	}
65 65
 
66 66
 	/**
@@ -69,12 +69,12 @@  discard block
 block discarded – undo
69 69
 	protected function normalizeColumns()
70 70
 	{
71 71
 		$this->columns = ['cb' => ''] + $this->columns;
72
-		if( array_key_exists( 'category', $this->columns )) {
72
+		if( array_key_exists( 'category', $this->columns ) ) {
73 73
 			$keys = array_keys( $this->columns );
74 74
 			$keys[array_search( 'category', $keys )] = 'taxonomy-'.Application::TAXONOMY;
75 75
 			$this->columns = array_combine( $keys, $this->columns );
76 76
 		}
77
-		if( array_key_exists( 'pinned', $this->columns )) {
77
+		if( array_key_exists( 'pinned', $this->columns ) ) {
78 78
 			$this->columns['pinned'] = glsr( Builder::class )->span(
79 79
 				'<span>'.$this->columns['pinned'].'</span>',
80 80
 				['class' => 'pinned-icon']
Please login to merge, or discard this patch.
plugin/Modules/Html/Attributes.php 2 patches
Braces   +12 added lines, -4 removed lines patch added patch discarded remove patch
@@ -181,7 +181,9 @@  discard block
 block discarded – undo
181 181
 				$key = $value;
182 182
 				$value = true;
183 183
 			}
184
-			if( !in_array( $key, static::BOOLEAN_ATTRIBUTES ))continue;
184
+			if( !in_array( $key, static::BOOLEAN_ATTRIBUTES )) {
185
+				continue;
186
+			}
185 187
 			$this->attributes[$key] = wp_validate_boolean( $value );
186 188
 		}
187 189
 	}
@@ -196,7 +198,9 @@  discard block
 block discarded – undo
196 198
 				$key = $value;
197 199
 				$value = '';
198 200
 			}
199
-			if( !glsr( Helper::class )->startsWith( 'data-', $key ))continue;
201
+			if( !glsr( Helper::class )->startsWith( 'data-', $key )) {
202
+				continue;
203
+			}
200 204
 			if( is_array( $value )) {
201 205
 				$value = json_encode( $value, JSON_HEX_APOS | JSON_NUMERIC_CHECK | JSON_UNESCAPED_UNICODE | JSON_UNESCAPED_SLASHES );
202 206
 			}
@@ -210,7 +214,9 @@  discard block
 block discarded – undo
210 214
 	protected function normalizeStringAttributes()
211 215
 	{
212 216
 		foreach( $this->attributes as $key => $value ) {
213
-			if( !is_string( $value ))continue;
217
+			if( !is_string( $value )) {
218
+				continue;
219
+			}
214 220
 			$this->attributes[$key] = trim( $value );
215 221
 		}
216 222
 	}
@@ -221,7 +227,9 @@  discard block
 block discarded – undo
221 227
 	 */
222 228
 	protected function normalizeInputType( $method )
223 229
 	{
224
-		if( $method != 'input' )return;
230
+		if( $method != 'input' ) {
231
+			return;
232
+		}
225 233
 		$attributes = wp_parse_args( $this->attributes, ['type' => ''] );
226 234
 		if( !in_array( $attributes['type'], static::INPUT_TYPES )) {
227 235
 			$this->attributes['type'] = 'text';
Please login to merge, or discard this patch.
Spacing   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -117,7 +117,7 @@  discard block
 block discarded – undo
117 117
 	 */
118 118
 	protected function filterAttributes( array $allowedAttributeKeys )
119 119
 	{
120
-		return array_intersect_key( $this->attributes, array_flip( $allowedAttributeKeys ));
120
+		return array_intersect_key( $this->attributes, array_flip( $allowedAttributeKeys ) );
121 121
 	}
122 122
 
123 123
 	/**
@@ -128,7 +128,7 @@  discard block
 block discarded – undo
128 128
 		$globalAttributes = $this->filterAttributes( static::GLOBAL_ATTRIBUTES );
129 129
 		$wildcards = [];
130 130
 		foreach( static::GLOBAL_WILDCARD_ATTRIBUTES as $wildcard ) {
131
-			$newWildcards = array_filter( $this->attributes, function( $key ) use( $wildcard ) {
131
+			$newWildcards = array_filter( $this->attributes, function( $key ) use($wildcard) {
132 132
 				return glsr( Helper::class )->startsWith( $wildcard, $key );
133 133
 			}, ARRAY_FILTER_USE_KEY );
134 134
 			$wildcards = array_merge( $wildcards, $newWildcards );
@@ -142,7 +142,7 @@  discard block
 block discarded – undo
142 142
 	protected function getPermanentAttributes()
143 143
 	{
144 144
 		$permanentAttributes = [];
145
-		if( array_key_exists( 'value', $this->attributes )) {
145
+		if( array_key_exists( 'value', $this->attributes ) ) {
146 146
 			$permanentAttributes['value'] = $this->attributes['value'];
147 147
 		}
148 148
 		return $permanentAttributes;
@@ -194,11 +194,11 @@  discard block
 block discarded – undo
194 194
 	protected function normalizeBooleanAttributes()
195 195
 	{
196 196
 		foreach( $this->attributes as $key => $value ) {
197
-			if( $this->isAttributeKeyNumeric( $key, $value )) {
197
+			if( $this->isAttributeKeyNumeric( $key, $value ) ) {
198 198
 				$key = $value;
199 199
 				$value = true;
200 200
 			}
201
-			if( !in_array( $key, static::BOOLEAN_ATTRIBUTES ))continue;
201
+			if( !in_array( $key, static::BOOLEAN_ATTRIBUTES ) )continue;
202 202
 			$this->attributes[$key] = wp_validate_boolean( $value );
203 203
 		}
204 204
 	}
@@ -209,12 +209,12 @@  discard block
 block discarded – undo
209 209
 	protected function normalizeDataAttributes()
210 210
 	{
211 211
 		foreach( $this->attributes as $key => $value ) {
212
-			if( $this->isAttributeKeyNumeric( $key, $value )) {
212
+			if( $this->isAttributeKeyNumeric( $key, $value ) ) {
213 213
 				$key = $value;
214 214
 				$value = '';
215 215
 			}
216
-			if( !glsr( Helper::class )->startsWith( 'data-', $key ))continue;
217
-			if( is_array( $value )) {
216
+			if( !glsr( Helper::class )->startsWith( 'data-', $key ) )continue;
217
+			if( is_array( $value ) ) {
218 218
 				$value = json_encode( $value, JSON_HEX_APOS | JSON_NUMERIC_CHECK | JSON_UNESCAPED_UNICODE | JSON_UNESCAPED_SLASHES );
219 219
 			}
220 220
 			$this->attributes[$key] = $value;
@@ -227,7 +227,7 @@  discard block
 block discarded – undo
227 227
 	protected function normalizeStringAttributes()
228 228
 	{
229 229
 		foreach( $this->attributes as $key => $value ) {
230
-			if( !is_string( $value ))continue;
230
+			if( !is_string( $value ) )continue;
231 231
 			$this->attributes[$key] = trim( $value );
232 232
 		}
233 233
 	}
@@ -240,7 +240,7 @@  discard block
 block discarded – undo
240 240
 	{
241 241
 		if( $method != 'input' )return;
242 242
 		$attributes = wp_parse_args( $this->attributes, ['type' => ''] );
243
-		if( !in_array( $attributes['type'], static::INPUT_TYPES )) {
243
+		if( !in_array( $attributes['type'], static::INPUT_TYPES ) ) {
244 244
 			$this->attributes['type'] = 'text';
245 245
 		}
246 246
 	}
@@ -254,11 +254,11 @@  discard block
 block discarded – undo
254 254
 		$permanentAttributes = $this->getPermanentAttributes();
255 255
 		foreach( $this->attributes as $key => $value ) {
256 256
 			if( in_array( $key, static::BOOLEAN_ATTRIBUTES ) && !$value ) {
257
-				unset( $attributes[$key] );
257
+				unset($attributes[$key]);
258 258
 			}
259
-			if( glsr( Helper::class )->startsWith( 'data-', $key )) {
259
+			if( glsr( Helper::class )->startsWith( 'data-', $key ) ) {
260 260
 				$permanentAttributes[$key] = $value;
261
-				unset( $attributes[$key] );
261
+				unset($attributes[$key]);
262 262
 			}
263 263
 		}
264 264
 		$this->attributes = array_merge( array_filter( $attributes ), $permanentAttributes );
Please login to merge, or discard this patch.
plugin/Modules/Blacklist.php 2 patches
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -37,7 +37,9 @@
 block discarded – undo
37 37
 		$lines = explode( "\n", $blacklist );
38 38
 		foreach( (array) $lines as $line ) {
39 39
 			$line = trim( $line );
40
-			if( empty( $line ) || 256 < strlen( $line ))continue;
40
+			if( empty( $line ) || 256 < strlen( $line )) {
41
+				continue;
42
+			}
41 43
 			$pattern = sprintf( '#%s#i', preg_quote( $line, '#' ));
42 44
 			if( preg_match( $pattern, $target )) {
43 45
 				return true;
Please login to merge, or discard this patch.
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -12,14 +12,14 @@  discard block
 block discarded – undo
12 12
 	 */
13 13
 	public function isBlacklisted( CreateReview $review )
14 14
 	{
15
-		$target = implode( "\n", array_filter([
15
+		$target = implode( "\n", array_filter( [
16 16
 			$review->author,
17 17
 			$review->content,
18 18
 			$review->email,
19 19
 			$review->ipAddress,
20 20
 			$review->title,
21
-		]));
22
-		return (bool) apply_filters( 'site-reviews/blacklist/is-blacklisted',
21
+		] ) );
22
+		return (bool)apply_filters( 'site-reviews/blacklist/is-blacklisted',
23 23
 			$this->check( $target ),
24 24
 			$review
25 25
 		);
@@ -31,16 +31,16 @@  discard block
 block discarded – undo
31 31
 	 */
32 32
 	protected function check( $target )
33 33
 	{
34
-		$blacklist = trim( glsr( OptionManager::class )->get( 'settings.submissions.blacklist.entries' ));
35
-		if( empty( $blacklist )) {
34
+		$blacklist = trim( glsr( OptionManager::class )->get( 'settings.submissions.blacklist.entries' ) );
35
+		if( empty($blacklist) ) {
36 36
 			return false;
37 37
 		}
38 38
 		$lines = explode( "\n", $blacklist );
39
-		foreach( (array) $lines as $line ) {
39
+		foreach( (array)$lines as $line ) {
40 40
 			$line = trim( $line );
41
-			if( empty( $line ) || 256 < strlen( $line ))continue;
42
-			$pattern = sprintf( '#%s#i', preg_quote( $line, '#' ));
43
-			if( preg_match( $pattern, $target )) {
41
+			if( empty($line) || 256 < strlen( $line ) )continue;
42
+			$pattern = sprintf( '#%s#i', preg_quote( $line, '#' ) );
43
+			if( preg_match( $pattern, $target ) ) {
44 44
 				return true;
45 45
 			}
46 46
 		}
Please login to merge, or discard this patch.
plugin/Modules/Notice.php 2 patches
Braces   +6 added lines, -2 removed lines patch added patch discarded remove patch
@@ -15,7 +15,9 @@  discard block
 block discarded – undo
15 15
 	 */
16 16
 	public function add( $type, $message, array $args = [] )
17 17
 	{
18
-		if( empty( array_filter( [$message, $type] )))return;
18
+		if( empty( array_filter( [$message, $type] ))) {
19
+			return;
20
+		}
19 21
 		$args['message'] = $message;
20 22
 		$args['type'] = $type;
21 23
 		add_settings_error( Application::ID, '', json_encode( $this->normalize( $args )));
@@ -56,7 +58,9 @@  discard block
 block discarded – undo
56 58
 		$notices = array_map( 'unserialize',
57 59
 			array_unique( array_map( 'serialize', get_settings_errors( Application::ID )))
58 60
 		);
59
-		if( empty( $notices ))return;
61
+		if( empty( $notices )) {
62
+			return;
63
+		}
60 64
 		return array_reduce( $notices, function( $carry, $notice ) {
61 65
 			return $carry.$this->buildNotice( json_decode( $notice['message'], true ));
62 66
 		});
Please login to merge, or discard this patch.
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -15,10 +15,10 @@  discard block
 block discarded – undo
15 15
 	 */
16 16
 	public function add( $type, $message, array $args = [] )
17 17
 	{
18
-		if( empty( array_filter( [$message, $type] )))return;
18
+		if( empty(array_filter( [$message, $type] )) )return;
19 19
 		$args['message'] = $message;
20 20
 		$args['type'] = $type;
21
-		add_settings_error( Application::ID, '', json_encode( $this->normalize( $args )));
21
+		add_settings_error( Application::ID, '', json_encode( $this->normalize( $args ) ) );
22 22
 	}
23 23
 
24 24
 	/**
@@ -54,11 +54,11 @@  discard block
 block discarded – undo
54 54
 	public function get()
55 55
 	{
56 56
 		$notices = array_map( 'unserialize',
57
-			array_unique( array_map( 'serialize', get_settings_errors( Application::ID )))
57
+			array_unique( array_map( 'serialize', get_settings_errors( Application::ID ) ) )
58 58
 		);
59
-		if( empty( $notices ))return;
59
+		if( empty($notices) )return;
60 60
 		return array_reduce( $notices, function( $carry, $notice ) {
61
-			return $carry.$this->buildNotice( json_decode( $notice['message'], true ));
61
+			return $carry.$this->buildNotice( json_decode( $notice['message'], true ) );
62 62
 		});
63 63
 	}
64 64
 
@@ -79,7 +79,7 @@  discard block
 block discarded – undo
79 79
 		}
80 80
 		return glsr( Builder::class )->div( $messages, [
81 81
 			'class' => $class,
82
-		]);
82
+		] );
83 83
 	}
84 84
 
85 85
 	/**
@@ -94,13 +94,13 @@  discard block
 block discarded – undo
94 94
 			'type' => '',
95 95
 		];
96 96
 		$args = shortcode_atts( $defaults, $args );
97
-		if( !in_array( $args['type'], ['error', 'warning', 'success'] )) {
97
+		if( !in_array( $args['type'], ['error', 'warning', 'success'] ) ) {
98 98
 			$args['type'] = 'success';
99 99
 		}
100 100
 		$args['messages'] = is_wp_error( $args['message'] )
101 101
 			? (array)$args['message']->get_error_message()
102 102
 			: (array)$args['message'];
103
-		unset( $args['message'] );
103
+		unset($args['message']);
104 104
 		return $args;
105 105
 	}
106 106
 }
Please login to merge, or discard this patch.
plugin/Modules/Session.php 4 patches
Indentation   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -5,8 +5,8 @@
 block discarded – undo
5 5
 use PasswordHash;
6 6
 
7 7
  /**
8
- * This class is derived from WP Session Manager (1.2.0)
9
- */
8
+  * This class is derived from WP Session Manager (1.2.0)
9
+  */
10 10
 class Session
11 11
 {
12 12
 	const SESSION_COOKIE = '_glsr_session';
Please login to merge, or discard this patch.
Upper-Lower-Casing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -153,7 +153,7 @@
 block discarded – undo
153 153
 		$expiredSessions = [];
154 154
 		$sessions = $wpdb->get_results(
155 155
 			"SELECT option_name AS name, option_value AS expiration " .
156
-			"FROM {$wpdb->options} " .
156
+			"from {$wpdb->options} " .
157 157
 			"WHERE option_name LIKE '".static::SESSION_COOKIE."_expires_%' " .
158 158
 			"ORDER BY option_value ASC " .
159 159
 			"LIMIT 0, ".absint( $limit )
Please login to merge, or discard this patch.
Spacing   +16 added lines, -16 removed lines patch added patch discarded remove patch
@@ -33,8 +33,8 @@  discard block
 block discarded – undo
33 33
 
34 34
 	public function __construct()
35 35
 	{
36
-		if( $cookieId = filter_input( INPUT_COOKIE, static::SESSION_COOKIE )) {
37
-			$cookie = explode( '||', stripslashes( $cookieId ));
36
+		if( $cookieId = filter_input( INPUT_COOKIE, static::SESSION_COOKIE ) ) {
37
+			$cookie = explode( '||', stripslashes( $cookieId ) );
38 38
 			$this->sessionId = preg_replace( '/[^A-Za-z0-9_]/', '', $cookie[0] );
39 39
 			$this->expiryTimestamp = absint( $cookie[1] );
40 40
 			$this->expiryTimestampReset = absint( $cookie[2] );
@@ -78,7 +78,7 @@  discard block
 block discarded – undo
78 78
 	public function deleteExpiredSessions( $limit = 1000 )
79 79
 	{
80 80
 		global $wpdb;
81
-		if( $expiredSessions = implode( "','", $this->getExpiredSessions( $limit ))) {
81
+		if( $expiredSessions = implode( "','", $this->getExpiredSessions( $limit ) ) ) {
82 82
 			$wpdb->query(
83 83
 				"DELETE FROM {$wpdb->options} WHERE option_name IN ('{$expiredSessions}')"
84 84
 			);
@@ -94,11 +94,11 @@  discard block
 block discarded – undo
94 94
 	public function get( $key, $fallback = '', $unset = false )
95 95
 	{
96 96
 		$key = sanitize_key( $key );
97
-		$value = isset( $this->sessionData[$key] )
97
+		$value = isset($this->sessionData[$key])
98 98
 			? maybe_unserialize( $this->sessionData[$key] )
99 99
 			: $fallback;
100
-		if( isset( $this->sessionData[$key] ) && $unset ) {
101
-			unset( $this->sessionData[$key] );
100
+		if( isset($this->sessionData[$key]) && $unset ) {
101
+			unset($this->sessionData[$key]);
102 102
 			$this->updateSession();
103 103
 		}
104 104
 		return $value;
@@ -132,7 +132,7 @@  discard block
 block discarded – undo
132 132
 	protected function deleteSession()
133 133
 	{
134 134
 		delete_option( $this->getSessionId() );
135
-		delete_option( $this->getSessionId( 'expires' ));
135
+		delete_option( $this->getSessionId( 'expires' ) );
136 136
 	}
137 137
 
138 138
 	/**
@@ -140,7 +140,7 @@  discard block
 block discarded – undo
140 140
 	 */
141 141
 	protected function generateSessionId()
142 142
 	{
143
-		return md5(( new PasswordHash( 8, false ))->get_random_bytes( 32 ));
143
+		return md5( (new PasswordHash( 8, false ))->get_random_bytes( 32 ) );
144 144
 	}
145 145
 
146 146
 	/**
@@ -152,13 +152,13 @@  discard block
 block discarded – undo
152 152
 		global $wpdb;
153 153
 		$expiredSessions = [];
154 154
 		$sessions = $wpdb->get_results(
155
-			"SELECT option_name AS name, option_value AS expiration " .
156
-			"FROM {$wpdb->options} " .
157
-			"WHERE option_name LIKE '".static::SESSION_COOKIE."_expires_%' " .
158
-			"ORDER BY option_value ASC " .
155
+			"SELECT option_name AS name, option_value AS expiration ".
156
+			"FROM {$wpdb->options} ".
157
+			"WHERE option_name LIKE '".static::SESSION_COOKIE."_expires_%' ".
158
+			"ORDER BY option_value ASC ".
159 159
 			"LIMIT 0, ".absint( $limit )
160 160
 		);
161
-		if( !empty( $sessions )) {
161
+		if( !empty($sessions) ) {
162 162
 			$now = time();
163 163
 			foreach( $sessions as $session ) {
164 164
 				if( $now <= $session->expiration )continue;
@@ -175,7 +175,7 @@  discard block
 block discarded – undo
175 175
 	 */
176 176
 	protected function getSessionId( $separator = '' )
177 177
 	{
178
-		return implode( '_', array_filter( [static::SESSION_COOKIE, $separator, $this->sessionId] ));
178
+		return implode( '_', array_filter( [static::SESSION_COOKIE, $separator, $this->sessionId] ) );
179 179
 	}
180 180
 
181 181
 	/**
@@ -206,7 +206,7 @@  discard block
 block discarded – undo
206 206
 	{
207 207
 		if( headers_sent() )return;
208 208
 		$cookie = $this->sessionId.'||'.$this->expiryTimestamp.'||'.$this->expiryTimestampReset;
209
-		$cookiePath = preg_replace( '|https?://[^/]+|i', '', trailingslashit( (string)get_option( 'home' )));
209
+		$cookiePath = preg_replace( '|https?://[^/]+|i', '', trailingslashit( (string)get_option( 'home' ) ) );
210 210
 		setcookie( static::SESSION_COOKIE, $cookie, $this->expiryTimestamp, $cookiePath );
211 211
 	}
212 212
 
@@ -224,7 +224,7 @@  discard block
 block discarded – undo
224 224
 	 */
225 225
 	protected function updateSession()
226 226
 	{
227
-		if( false === get_option( $this->getSessionId() )) {
227
+		if( false === get_option( $this->getSessionId() ) ) {
228 228
 			$this->createSession();
229 229
 		}
230 230
 		else {
Please login to merge, or discard this patch.
Braces   +6 added lines, -2 removed lines patch added patch discarded remove patch
@@ -161,7 +161,9 @@  discard block
 block discarded – undo
161 161
 		if( !empty( $sessions )) {
162 162
 			$now = time();
163 163
 			foreach( $sessions as $session ) {
164
-				if( $now <= $session->expiration )continue;
164
+				if( $now <= $session->expiration ) {
165
+					continue;
166
+				}
165 167
 				$expiredSessions[] = $session->name;
166 168
 				$expiredSessions[] = str_replace( '_expires_', '_', $session->name );
167 169
 			}
@@ -204,7 +206,9 @@  discard block
 block discarded – undo
204 206
 	 */
205 207
 	protected function setCookie()
206 208
 	{
207
-		if( headers_sent() )return;
209
+		if( headers_sent() ) {
210
+			return;
211
+		}
208 212
 		$cookie = $this->sessionId.'||'.$this->expiryTimestamp.'||'.$this->expiryTimestampReset;
209 213
 		$cookiePath = preg_replace( '|https?://[^/]+|i', '', trailingslashit( (string)get_option( 'home' )));
210 214
 		setcookie( static::SESSION_COOKIE, $cookie, $this->expiryTimestamp, $cookiePath );
Please login to merge, or discard this patch.
plugin/Modules/Schema.php 2 patches
Braces   +6 added lines, -2 removed lines patch added patch discarded remove patch
@@ -108,7 +108,9 @@  discard block
 block discarded – undo
108 108
 	 */
109 109
 	public function render()
110 110
 	{
111
-		if( is_null( glsr()->schemas ))return;
111
+		if( is_null( glsr()->schemas )) {
112
+			return;
113
+		}
112 114
 		printf( '<script type="application/ld+json">%s</script>', json_encode(
113 115
 			apply_filters( 'site-reviews/schema/all', glsr()->schemas ),
114 116
 			JSON_UNESCAPED_UNICODE | JSON_UNESCAPED_SLASHES
@@ -181,7 +183,9 @@  discard block
 block discarded – undo
181 183
 		if( $value != $fallback ) {
182 184
 			return $value;
183 185
 		}
184
-		if( !is_single() && !is_page() )return;
186
+		if( !is_single() && !is_page() ) {
187
+			return;
188
+		}
185 189
 		switch( $option ) {
186 190
 			case 'description':
187 191
 				return get_the_excerpt();
Please login to merge, or discard this patch.
Spacing   +15 added lines, -15 removed lines patch added patch discarded remove patch
@@ -32,10 +32,10 @@  discard block
 block discarded – undo
32 32
 		foreach( glsr( Database::class )->getReviews( $this->args )->reviews as $review ) {
33 33
 			$reviews[] = $this->buildReview( $review );
34 34
 		}
35
-		if( !empty( $reviews )) {
35
+		if( !empty($reviews) ) {
36 36
 			array_walk( $reviews, function( &$review ) {
37
-				unset( $review['@context'] );
38
-				unset( $review['itemReviewed'] );
37
+				unset($review['@context']);
38
+				unset($review['itemReviewed']);
39 39
 			});
40 40
 			$schema['review'] = $reviews;
41 41
 		}
@@ -49,20 +49,20 @@  discard block
 block discarded – undo
49 49
 	public function buildReview( $review )
50 50
 	{
51 51
 		$schema = SchemaOrg::Review()
52
-			->doIf( !in_array( 'title', $this->args['hide'] ), function( ReviewSchema $schema ) use( $review ) {
52
+			->doIf( !in_array( 'title', $this->args['hide'] ), function( ReviewSchema $schema ) use($review) {
53 53
 				$schema->name( $review->title );
54 54
 			})
55
-			->doIf( !in_array( 'excerpt', $this->args['hide'] ), function( ReviewSchema $schema ) use( $review ) {
55
+			->doIf( !in_array( 'excerpt', $this->args['hide'] ), function( ReviewSchema $schema ) use($review) {
56 56
 				$schema->reviewBody( $review->content );
57 57
 			})
58
-			->datePublished(( new DateTime( $review->date ))->format( DateTime::ISO8601 ))
58
+			->datePublished( (new DateTime( $review->date ))->format( DateTime::ISO8601 ) )
59 59
 			->author( SchemaOrg::Person()
60 60
 				->name( $review->author )
61 61
 			)
62 62
 			->itemReviewed( $this->getSchemaType()
63 63
 				->name( $this->getThingName() )
64 64
 			);
65
-		if( !empty( $review->rating )) {
65
+		if( !empty($review->rating) ) {
66 66
 			$schema->reviewRating( SchemaOrg::Rating()
67 67
 				->ratingValue( $review->rating )
68 68
 				->bestRating( Rating::MAX_RATING )
@@ -78,7 +78,7 @@  discard block
 block discarded – undo
78 78
 	 */
79 79
 	public function buildSummary( $args = null )
80 80
 	{
81
-		if( is_array( $args )) {
81
+		if( is_array( $args ) ) {
82 82
 			$this->args = $args;
83 83
 		}
84 84
 		$schema = $this->getSchemaType()
@@ -90,7 +90,7 @@  discard block
 block discarded – undo
90 90
 			->image( $this->getThingImage() )
91 91
 			->url( $this->getThingUrl() );
92 92
 		$count = $this->getReviewCount();
93
-		if( !empty( $count )) {
93
+		if( !empty($count) ) {
94 94
 			$schema->aggregateRating( SchemaOrg::AggregateRating()
95 95
 				->ratingValue( $this->getRatingValue() )
96 96
 				->reviewCount( $count )
@@ -108,11 +108,11 @@  discard block
 block discarded – undo
108 108
 	 */
109 109
 	public function render()
110 110
 	{
111
-		if( is_null( glsr()->schemas ))return;
111
+		if( is_null( glsr()->schemas ) )return;
112 112
 		printf( '<script type="application/ld+json">%s</script>', json_encode(
113 113
 			apply_filters( 'site-reviews/schema/all', glsr()->schemas ),
114 114
 			JSON_UNESCAPED_UNICODE | JSON_UNESCAPED_SLASHES
115
-		));
115
+		) );
116 116
 	}
117 117
 
118 118
 	/**
@@ -120,9 +120,9 @@  discard block
 block discarded – undo
120 120
 	 */
121 121
 	public function store( array $schema )
122 122
 	{
123
-		$schemas = (array) glsr()->schemas;
123
+		$schemas = (array)glsr()->schemas;
124 124
 		$schemas[] = $schema;
125
-		glsr()->schemas = array_map( 'unserialize', array_unique( array_map( 'serialize', $schemas )));
125
+		glsr()->schemas = array_map( 'unserialize', array_unique( array_map( 'serialize', $schemas ) ) );
126 126
 	}
127 127
 
128 128
 	/**
@@ -147,7 +147,7 @@  discard block
 block discarded – undo
147 147
 	 */
148 148
 	protected function getReviews( $force = false )
149 149
 	{
150
-		if( !isset( $this->reviews ) || $force ) {
150
+		if( !isset($this->reviews) || $force ) {
151 151
 			$args = wp_parse_args( ['count' => -1], $this->args );
152 152
 			$this->reviews = glsr( Database::class )->getReviews( $args )->reviews;
153 153
 		}
@@ -161,7 +161,7 @@  discard block
 block discarded – undo
161 161
 	 */
162 162
 	protected function getSchemaOption( $option, $fallback )
163 163
 	{
164
-		if( $schemaOption = trim( (string)get_post_meta( intval( get_the_ID() ), 'schema_'.$option, true ))) {
164
+		if( $schemaOption = trim( (string)get_post_meta( intval( get_the_ID() ), 'schema_'.$option, true ) ) ) {
165 165
 			return $schemaOption;
166 166
 		}
167 167
 		$default = glsr( OptionManager::class )->get( 'settings.reviews.schema.'.$option.'.default', $fallback );
Please login to merge, or discard this patch.
plugin/Modules/Akismet.php 2 patches
Braces   +9 added lines, -3 removed lines patch added patch discarded remove patch
@@ -30,7 +30,9 @@  discard block
 block discarded – undo
30 30
 			// 'is_test' => 1,
31 31
 		];
32 32
 		foreach( $_SERVER as $key => $value ) {
33
-			if( is_array( $value ) || in_array( $key, ['HTTP_COOKIE', 'HTTP_COOKIE2', 'PHP_AUTH_PW'] ))continue;
33
+			if( is_array( $value ) || in_array( $key, ['HTTP_COOKIE', 'HTTP_COOKIE2', 'PHP_AUTH_PW'] )) {
34
+				continue;
35
+			}
34 36
 			$submission[$key] = $value;
35 37
 		}
36 38
 		return $this->check( apply_filters( 'site-reviews/akismet/submission', $submission, $review ));
@@ -56,12 +58,16 @@  discard block
 block discarded – undo
56 58
 	{
57 59
 		$query = [];
58 60
 		foreach( $data as $key => $value ) {
59
-			if( is_array( $value ) || is_object( $value ))continue;
61
+			if( is_array( $value ) || is_object( $value )) {
62
+				continue;
63
+			}
60 64
 			if( $value === false ) {
61 65
 				$value = '0';
62 66
 			}
63 67
 			$value = trim( $value );
64
-			if( !strlen( $value ))continue;
68
+			if( !strlen( $value )) {
69
+				continue;
70
+			}
65 71
 			$query[] = urlencode( $key ).'='.urlencode( $value );
66 72
 		}
67 73
 		return implode( '&', $query );
Please login to merge, or discard this patch.
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -31,10 +31,10 @@  discard block
 block discarded – undo
31 31
 			// 'is_test' => 1,
32 32
 		];
33 33
 		foreach( $_SERVER as $key => $value ) {
34
-			if( is_array( $value ) || in_array( $key, ['HTTP_COOKIE', 'HTTP_COOKIE2', 'PHP_AUTH_PW'] ))continue;
34
+			if( is_array( $value ) || in_array( $key, ['HTTP_COOKIE', 'HTTP_COOKIE2', 'PHP_AUTH_PW'] ) )continue;
35 35
 			$submission[$key] = $value;
36 36
 		}
37
-		return $this->check( apply_filters( 'site-reviews/akismet/submission', $submission, $review ));
37
+		return $this->check( apply_filters( 'site-reviews/akismet/submission', $submission, $review ) );
38 38
 	}
39 39
 
40 40
 	/**
@@ -57,12 +57,12 @@  discard block
 block discarded – undo
57 57
 	{
58 58
 		$query = [];
59 59
 		foreach( $data as $key => $value ) {
60
-			if( is_array( $value ) || is_object( $value ))continue;
60
+			if( is_array( $value ) || is_object( $value ) )continue;
61 61
 			if( $value === false ) {
62 62
 				$value = '0';
63 63
 			}
64 64
 			$value = trim( $value );
65
-			if( !strlen( $value ))continue;
65
+			if( !strlen( $value ) )continue;
66 66
 			$query[] = urlencode( $key ).'='.urlencode( $value );
67 67
 		}
68 68
 		return implode( '&', $query );
@@ -77,7 +77,7 @@  discard block
 block discarded – undo
77 77
 			|| !is_callable( ['Akismet', 'get_api_key'] )
78 78
 			|| !is_callable( ['Akismet', 'http_post'] )
79 79
 			? false
80
-			: (bool) AkismetPlugin::get_api_key();
80
+			: (bool)AkismetPlugin::get_api_key();
81 81
 		return apply_filters( 'site-reviews/akismet/is-active', $check );
82 82
 	}
83 83
 }
Please login to merge, or discard this patch.