Passed
Push — master ( 3b699e...87eb8d )
by Paul
04:08
created
plugin/Shortcodes/Shortcode.php 1 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/Modules/Html/Form.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -27,13 +27,13 @@
 block discarded – undo
27 27
 	public function getFields( $id )
28 28
 	{
29 29
 		$fields = [];
30
-		$configPath = glsr()->path( 'config/'.$id.'.php' );;
30
+		$configPath = glsr()->path( 'config/'.$id.'.php' ); ;
31 31
 		$values = file_exists( $configPath )
32 32
 			? include $configPath
33 33
 			: [];
34 34
 		$values = apply_filters( 'site-reviews/form/fields', $values );
35 35
 		foreach( $values as $name => $field ) {
36
-			$fields[] = new Field( wp_parse_args( $field, ['name' => $name] ));
36
+			$fields[] = new Field( wp_parse_args( $field, ['name' => $name] ) );
37 37
 		}
38 38
 		return $fields;
39 39
 	}
Please login to merge, or discard this patch.
plugin/Modules/Html/Fields/Honeypot.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -13,7 +13,7 @@
 block discarded – undo
13 13
 	{
14 14
 		$this->builder->args = wp_parse_args( $this->builder->args, [
15 15
 			'name' => $this->builder->args['text'],
16
-		]);
16
+		] );
17 17
 		$this->builder->tag = 'input';
18 18
 		$this->mergeFieldArgs();
19 19
 		return $this->builder->getOpeningTag();
Please login to merge, or discard this patch.
plugin/Handlers/EnqueueAdminAssets.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -44,7 +44,7 @@  discard block
 block discarded – undo
44 44
 			glsr()->version,
45 45
 			true
46 46
 		);
47
-		if( !empty( $this->pointers )) {
47
+		if( !empty($this->pointers) ) {
48 48
 			wp_enqueue_style( 'wp-pointer' );
49 49
 			wp_enqueue_script( 'wp-pointer' );
50 50
 		}
@@ -84,7 +84,7 @@  discard block
 block discarded – undo
84 84
 		$dependencies = apply_filters( 'site-reviews/enqueue/admin/dependencies', [] );
85 85
 		$dependencies = array_merge( $dependencies, [
86 86
 			'jquery', 'jquery-ui-sortable', 'underscore', 'wp-util',
87
-		]);
87
+		] );
88 88
 		return $dependencies;
89 89
 	}
90 90
 
@@ -114,7 +114,7 @@  discard block
 block discarded – undo
114 114
 		$generatedPointers = [];
115 115
 		foreach( $pointers as $pointer ) {
116 116
 			if( $pointer['screen'] != glsr_current_screen()->id )continue;
117
-			if( in_array( $pointer['id'], $dismissedPointers ))continue;
117
+			if( in_array( $pointer['id'], $dismissedPointers ) )continue;
118 118
 			$generatedPointers[] = $this->generatePointer( $pointer );
119 119
 		}
120 120
 		$this->pointers = $generatedPointers;
@@ -126,7 +126,7 @@  discard block
 block discarded – undo
126 126
 	protected function isCurrentScreen()
127 127
 	{
128 128
 		$screen = glsr_current_screen();
129
-		return $screen && ( $screen->post_type == Application::POST_TYPE
129
+		return $screen && ($screen->post_type == Application::POST_TYPE
130 130
 			|| $screen->base == 'post'
131 131
 			|| $screen->id == 'dashboard'
132 132
 			|| $screen->id == 'widgets'
@@ -140,7 +140,7 @@  discard block
 block discarded – undo
140 140
 	{
141 141
 		$variables = [];
142 142
 		foreach( glsr()->mceShortcodes as $tag => $args ) {
143
-			if( empty( $args['required'] ))continue;
143
+			if( empty($args['required']) )continue;
144 144
 			$variables[$tag] = $args['required'];
145 145
 		}
146 146
 		return $variables;
Please login to merge, or discard this patch.
autoload.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -2,7 +2,7 @@  discard block
 block discarded – undo
2 2
 
3 3
 defined( 'WPINC' ) || die;
4 4
 
5
-require_once( ABSPATH.WPINC.'/class-phpass.php' );
5
+require_once(ABSPATH.WPINC.'/class-phpass.php');
6 6
 
7 7
 spl_autoload_register( function( $className ) {
8 8
 	$namespaces = [
@@ -16,8 +16,8 @@  discard block
 block discarded – undo
16 16
 	foreach( $namespaces as $prefix => $baseDir ) {
17 17
 		$len = strlen( $prefix );
18 18
 		if( strncmp( $prefix, $className, $len ) !== 0 )continue;
19
-		$file = $baseDir.str_replace( '\\', '/', substr( $className, $len )).'.php';
20
-		if( !file_exists( $file ))continue;
19
+		$file = $baseDir.str_replace( '\\', '/', substr( $className, $len ) ).'.php';
20
+		if( !file_exists( $file ) )continue;
21 21
 		require $file;
22 22
 		break;
23 23
 	}
Please login to merge, or discard this patch.
plugin/Controllers/PublicController.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -50,7 +50,7 @@
 block discarded – undo
50 50
 	public function routerSubmitReview( array $request )
51 51
 	{
52 52
 		$validated = glsr( ValidateReview::class )->validate( $request );
53
-		if( !empty( $validated->error ) || $validated->recaptchaIsUnset )return;
54
-		$this->execute( new CreateReview( $validated->request ));
53
+		if( !empty($validated->error) || $validated->recaptchaIsUnset )return;
54
+		$this->execute( new CreateReview( $validated->request ) );
55 55
 	}
56 56
 }
Please login to merge, or discard this patch.
plugin/Modules/Session.php 1 patch
Spacing   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -34,8 +34,8 @@  discard block
 block discarded – undo
34 34
 
35 35
 	public function __construct()
36 36
 	{
37
-		if( $cookieId = filter_input( INPUT_COOKIE, static::SESSION_COOKIE )) {
38
-			$cookie = explode( '||', stripslashes( $cookieId ));
37
+		if( $cookieId = filter_input( INPUT_COOKIE, static::SESSION_COOKIE ) ) {
38
+			$cookie = explode( '||', stripslashes( $cookieId ) );
39 39
 			$this->sessionId = preg_replace( '/[^A-Za-z0-9_]/', '', $cookie[0] );
40 40
 			$this->expiryTimestamp = absint( $cookie[1] );
41 41
 			$this->expiryTimestampReset = absint( $cookie[2] );
@@ -75,7 +75,7 @@  discard block
 block discarded – undo
75 75
 	 */
76 76
 	public function deleteExpiredSessions( $limit = 1000 )
77 77
 	{
78
-		if( $expiredSessions = implode( "','", $this->getExpiredSessions( $limit ))) {
78
+		if( $expiredSessions = implode( "','", $this->getExpiredSessions( $limit ) ) ) {
79 79
 			glsr( SqlQueries::class )->deleteExpiredSessions( $expiredSessions );
80 80
 		}
81 81
 	}
@@ -89,11 +89,11 @@  discard block
 block discarded – undo
89 89
 	public function get( $key, $fallback = '', $unset = false )
90 90
 	{
91 91
 		$key = sanitize_key( $key );
92
-		$value = isset( $this->sessionData[$key] )
92
+		$value = isset($this->sessionData[$key])
93 93
 			? maybe_unserialize( $this->sessionData[$key] )
94 94
 			: $fallback;
95
-		if( isset( $this->sessionData[$key] ) && $unset ) {
96
-			unset( $this->sessionData[$key] );
95
+		if( isset($this->sessionData[$key]) && $unset ) {
96
+			unset($this->sessionData[$key]);
97 97
 			$this->updateSession();
98 98
 		}
99 99
 		return $value;
@@ -127,7 +127,7 @@  discard block
 block discarded – undo
127 127
 	protected function deleteSession()
128 128
 	{
129 129
 		delete_option( $this->getSessionId() );
130
-		delete_option( $this->getSessionId( 'expires' ));
130
+		delete_option( $this->getSessionId( 'expires' ) );
131 131
 	}
132 132
 
133 133
 	/**
@@ -135,7 +135,7 @@  discard block
 block discarded – undo
135 135
 	 */
136 136
 	protected function generateSessionId()
137 137
 	{
138
-		return md5(( new PasswordHash( 8, false ))->get_random_bytes( 32 ));
138
+		return md5( (new PasswordHash( 8, false ))->get_random_bytes( 32 ) );
139 139
 	}
140 140
 
141 141
 	/**
@@ -145,8 +145,8 @@  discard block
 block discarded – undo
145 145
 	protected function getExpiredSessions( $limit )
146 146
 	{
147 147
 		$expiredSessions = [];
148
-		$sessions = glsr( SqlQueries::class )->getExpiredSessions( static::SESSION_COOKIE, absint( $limit ));
149
-		if( !empty( $sessions )) {
148
+		$sessions = glsr( SqlQueries::class )->getExpiredSessions( static::SESSION_COOKIE, absint( $limit ) );
149
+		if( !empty($sessions) ) {
150 150
 			$now = time();
151 151
 			foreach( $sessions as $session ) {
152 152
 				if( $now <= $session->expiration )continue;
@@ -163,7 +163,7 @@  discard block
 block discarded – undo
163 163
 	 */
164 164
 	protected function getSessionId( $separator = '' )
165 165
 	{
166
-		return implode( '_', array_filter( [static::SESSION_COOKIE, $separator, $this->sessionId] ));
166
+		return implode( '_', array_filter( [static::SESSION_COOKIE, $separator, $this->sessionId] ) );
167 167
 	}
168 168
 
169 169
 	/**
@@ -194,7 +194,7 @@  discard block
 block discarded – undo
194 194
 	{
195 195
 		if( headers_sent() )return;
196 196
 		$cookie = $this->sessionId.'||'.$this->expiryTimestamp.'||'.$this->expiryTimestampReset;
197
-		$cookiePath = preg_replace( '|https?://[^/]+|i', '', trailingslashit( (string)get_option( 'home' )));
197
+		$cookiePath = preg_replace( '|https?://[^/]+|i', '', trailingslashit( (string)get_option( 'home' ) ) );
198 198
 		setcookie( static::SESSION_COOKIE, $cookie, $this->expiryTimestamp, $cookiePath );
199 199
 	}
200 200
 
@@ -212,7 +212,7 @@  discard block
 block discarded – undo
212 212
 	 */
213 213
 	protected function updateSession()
214 214
 	{
215
-		if( false === get_option( $this->getSessionId() )) {
215
+		if( false === get_option( $this->getSessionId() ) ) {
216 216
 			return $this->createSession();
217 217
 		}
218 218
 		update_option( $this->getSessionId(), $this->sessionData, false );
Please login to merge, or discard this patch.
plugin/Modules/Blacklist.php 1 patch
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -11,14 +11,14 @@  discard block
 block discarded – undo
11 11
 	 */
12 12
 	public function isBlacklisted( array $review )
13 13
 	{
14
-		$target = implode( "\n", array_filter([
14
+		$target = implode( "\n", array_filter( [
15 15
 			$review['name'],
16 16
 			$review['content'],
17 17
 			$review['email'],
18 18
 			$review['ip_address'],
19 19
 			$review['title'],
20
-		]));
21
-		return (bool) apply_filters( 'site-reviews/blacklist/is-blacklisted',
20
+		] ) );
21
+		return (bool)apply_filters( 'site-reviews/blacklist/is-blacklisted',
22 22
 			$this->check( $target ),
23 23
 			$review
24 24
 		);
@@ -30,16 +30,16 @@  discard block
 block discarded – undo
30 30
 	 */
31 31
 	protected function check( $target )
32 32
 	{
33
-		$blacklist = trim( glsr( OptionManager::class )->get( 'settings.submissions.blacklist.entries' ));
34
-		if( empty( $blacklist )) {
33
+		$blacklist = trim( glsr( OptionManager::class )->get( 'settings.submissions.blacklist.entries' ) );
34
+		if( empty($blacklist) ) {
35 35
 			return false;
36 36
 		}
37 37
 		$lines = explode( "\n", $blacklist );
38
-		foreach( (array) $lines as $line ) {
38
+		foreach( (array)$lines as $line ) {
39 39
 			$line = trim( $line );
40
-			if( empty( $line ) || 256 < strlen( $line ))continue;
41
-			$pattern = sprintf( '#%s#i', preg_quote( $line, '#' ));
42
-			if( preg_match( $pattern, $target )) {
40
+			if( empty($line) || 256 < strlen( $line ) )continue;
41
+			$pattern = sprintf( '#%s#i', preg_quote( $line, '#' ) );
42
+			if( preg_match( $pattern, $target ) ) {
43 43
 				return true;
44 44
 			}
45 45
 		}
Please login to merge, or discard this patch.
plugin/Commands/CreateReview.php 1 patch
Spacing   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -27,21 +27,21 @@  discard block
 block discarded – undo
27 27
 	public function __construct( $input )
28 28
 	{
29 29
 		$this->request = $input;
30
-		$this->ajax_request = isset( $input['ajax_request'] );
30
+		$this->ajax_request = isset($input['ajax_request']);
31 31
 		$this->assigned_to = $this->getNumeric( 'assign_to' );
32
-		$this->author = sanitize_text_field( $this->get( 'name' ));
33
-		$this->avatar = get_avatar_url( $this->get( 'email' ));
34
-		$this->blacklisted = isset( $input['blacklisted'] );
35
-		$this->category = sanitize_key( $this->get( 'category' ));
36
-		$this->content = sanitize_textarea_field( $this->get( 'content' ));
32
+		$this->author = sanitize_text_field( $this->get( 'name' ) );
33
+		$this->avatar = get_avatar_url( $this->get( 'email' ) );
34
+		$this->blacklisted = isset($input['blacklisted']);
35
+		$this->category = sanitize_key( $this->get( 'category' ) );
36
+		$this->content = sanitize_textarea_field( $this->get( 'content' ) );
37 37
 		$this->custom = $this->getCustom();
38
-		$this->email = sanitize_email( $this->get( 'email' ));
39
-		$this->form_id = sanitize_key( $this->get( 'form_id' ));
38
+		$this->email = sanitize_email( $this->get( 'email' ) );
39
+		$this->form_id = sanitize_key( $this->get( 'form_id' ) );
40 40
 		$this->ip_address = $this->get( 'ip_address' );
41
-		$this->rating = intval( $this->get( 'rating' ));
41
+		$this->rating = intval( $this->get( 'rating' ) );
42 42
 		$this->referrer = $this->get( 'referrer' );
43
-		$this->terms = isset( $input['terms'] );
44
-		$this->title = sanitize_text_field( $this->get( 'title' ));
43
+		$this->terms = isset($input['terms']);
44
+		$this->title = sanitize_text_field( $this->get( 'title' ) );
45 45
 	}
46 46
 
47 47
 	/**
@@ -50,7 +50,7 @@  discard block
 block discarded – undo
50 50
 	 */
51 51
 	protected function get( $key )
52 52
 	{
53
-		return isset( $this->request[$key] )
53
+		return isset($this->request[$key])
54 54
 			? (string)$this->request[$key]
55 55
 			: '';
56 56
 	}
@@ -67,7 +67,7 @@  discard block
 block discarded – undo
67 67
 		];
68 68
 		$custom = $this->request;
69 69
 		foreach( $unset as $value ) {
70
-			unset( $custom[$value] );
70
+			unset($custom[$value]);
71 71
 		}
72 72
 		return $custom;
73 73
 	}
Please login to merge, or discard this patch.