Passed
Push — master ( 65c96d...e99773 )
by Paul
08:33 queued 04:19
created
views/partials/editor/metabox-categories.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -16,11 +16,11 @@  discard block
 block discarded – undo
16 16
 	<div id="<?= $tax_name; ?>-all" class="tabs-panel">
17 17
 		<input type="hidden" name="tax_input[<?= $tax_name; ?>][]" value='0' />
18 18
 		<ul id="<?= $tax_name; ?>checklist" data-wp-lists="list:<?= $tax_name; ?>" class="categorychecklist form-no-clear">
19
-			<?php wp_terms_checklist( $post->ID, array( 'taxonomy' => $tax_name, 'popular_cats' => $popular_ids )); ?>
19
+			<?php wp_terms_checklist( $post->ID, array( 'taxonomy' => $tax_name, 'popular_cats' => $popular_ids ) ); ?>
20 20
 		</ul>
21 21
 	</div>
22 22
 
23
-	<?php if( current_user_can( $taxonomy->cap->edit_terms )) : ?>
23
+	<?php if( current_user_can( $taxonomy->cap->edit_terms ) ) : ?>
24 24
 	<div id="<?= $tax_name; ?>-adder" class="wp-hidden-children">
25 25
 		<a id="<?= $tax_name; ?>-add-toggle" href="#<?= $tax_name; ?>-add" class="hide-if-no-js taxonomy-add-new">
26 26
 			<?= sprintf( '+ %s', $taxonomy->labels->add_new_item ); ?>
@@ -29,7 +29,7 @@  discard block
 block discarded – undo
29 29
 			<label class="screen-reader-text" for="new<?= $tax_name; ?>"><?= $taxonomy->labels->add_new_item; ?></label>
30 30
 			<input type="text" name="new<?= $tax_name; ?>" id="new<?= $tax_name; ?>" class="form-required form-input-tip" value="<?= esc_attr( $taxonomy->labels->new_item_name ); ?>" aria-required="true"/>
31 31
 			<input type="button" id="<?= $tax_name; ?>-add-submit" data-wp-lists="add:<?= $tax_name; ?>checklist:<?= $tax_name; ?>-add" class="button category-add-submit" value="<?= esc_attr( $taxonomy->labels->add_new_item ); ?>" />
32
-			<?php wp_nonce_field( 'add-' . $tax_name, '_ajax_nonce-add-' . $tax_name, false ); ?>
32
+			<?php wp_nonce_field( 'add-'.$tax_name, '_ajax_nonce-add-'.$tax_name, false ); ?>
33 33
 			<span id="<?= $tax_name; ?>-ajax-response"></span>
34 34
 		</div>
35 35
 	</div>
Please login to merge, or discard this patch.
plugin/Controllers/ListTableController/Columns.php 2 patches
Braces   +12 added lines, -4 removed lines patch added patch discarded remove patch
@@ -86,7 +86,9 @@  discard block
 block discarded – undo
86 86
 	 */
87 87
 	public function renderFilters( $postType )
88 88
 	{
89
-		if( $postType !== Application::POST_TYPE )return;
89
+		if( $postType !== Application::POST_TYPE ) {
90
+			return;
91
+		}
90 92
 		if( !( $status = filter_input( INPUT_GET, 'post_status' ))) {
91 93
 			$status = 'publish';
92 94
 		}
@@ -103,7 +105,9 @@  discard block
 block discarded – undo
103 105
 	 */
104 106
 	public function renderValues( $column, $postId )
105 107
 	{
106
-		if( glsr_current_screen()->post_type != Application::POST_TYPE )return;
108
+		if( glsr_current_screen()->post_type != Application::POST_TYPE ) {
109
+			return;
110
+		}
107 111
 		$method = glsr( Helper::class )->buildMethodName( $column, 'buildColumn' );
108 112
 		echo method_exists( $this, $method )
109 113
 			? call_user_func( [$this, $method], $postId )
@@ -116,7 +120,9 @@  discard block
 block discarded – undo
116 120
 	 */
117 121
 	protected function renderFilterRatings( $ratings )
118 122
 	{
119
-		if( empty( $ratings ))return;
123
+		if( empty( $ratings )) {
124
+			return;
125
+		}
120 126
 		$ratings = array_flip( array_reverse( $ratings ));
121 127
 		array_walk( $ratings, function( &$value, $key ) {
122 128
 			$label = _n( '%s star', '%s stars', $key, 'site-reviews' );
@@ -139,7 +145,9 @@  discard block
 block discarded – undo
139 145
 	 */
140 146
 	protected function renderFilterTypes( $types )
141 147
 	{
142
-		if( count( glsr()->reviewTypes ) < 2 )return;
148
+		if( count( glsr()->reviewTypes ) < 2 ) {
149
+			return;
150
+		}
143 151
 		echo glsr( Builder::class )->label( __( 'Filter by type', 'site-reviews' ), [
144 152
 			'class' => 'screen-reader-text',
145 153
 			'for' => 'review_type',
Please login to merge, or discard this patch.
Spacing   +15 added lines, -15 removed lines patch added patch discarded remove patch
@@ -22,7 +22,7 @@  discard block
 block discarded – undo
22 22
 		if( $assignedPost instanceof WP_Post && $assignedPost->post_status == 'publish' ) {
23 23
 			$column = glsr( Builder::class )->a( get_the_title( $assignedPost->ID ), [
24 24
 				'href' => (string)get_the_permalink( $assignedPost->ID ),
25
-			]);
25
+			] );
26 26
 		}
27 27
 		return $column;
28 28
 	}
@@ -36,10 +36,10 @@  discard block
 block discarded – undo
36 36
 		$pinned = get_post_meta( $postId, 'pinned', true )
37 37
 			? 'pinned '
38 38
 			: '';
39
-		return glsr( Builder::class )->i([
39
+		return glsr( Builder::class )->i( [
40 40
 			'class' => $pinned.'dashicons dashicons-sticky',
41 41
 			'data-id' => $postId,
42
-		]);
42
+		] );
43 43
 	}
44 44
 
45 45
 	/**
@@ -48,7 +48,7 @@  discard block
 block discarded – undo
48 48
 	 */
49 49
 	public function buildColumnReviewer( $postId )
50 50
 	{
51
-		return strval( get_post_meta( $postId, 'author', true ));
51
+		return strval( get_post_meta( $postId, 'author', true ) );
52 52
 	}
53 53
 
54 54
 	/**
@@ -59,8 +59,8 @@  discard block
 block discarded – undo
59 59
 	public function buildColumnRating( $postId )
60 60
 	{
61 61
 		return glsr( Partial::class )->build( 'star-rating', [
62
-			'rating' => intval( get_post_meta( $postId, 'rating', true )),
63
-		]);
62
+			'rating' => intval( get_post_meta( $postId, 'rating', true ) ),
63
+		] );
64 64
 	}
65 65
 
66 66
 	/**
@@ -82,7 +82,7 @@  discard block
 block discarded – undo
82 82
 	public function renderFilters( $postType )
83 83
 	{
84 84
 		if( $postType !== Application::POST_TYPE )return;
85
-		if( !( $status = filter_input( INPUT_GET, 'post_status' ))) {
85
+		if( !($status = filter_input( INPUT_GET, 'post_status' )) ) {
86 86
 			$status = 'publish';
87 87
 		}
88 88
 		$ratings = glsr( Database::class )->getReviewsMeta( 'rating', $status );
@@ -111,8 +111,8 @@  discard block
 block discarded – undo
111 111
 	 */
112 112
 	protected function renderFilterRatings( $ratings )
113 113
 	{
114
-		if( empty( $ratings ))return;
115
-		$ratings = array_flip( array_reverse( $ratings ));
114
+		if( empty($ratings) )return;
115
+		$ratings = array_flip( array_reverse( $ratings ) );
116 116
 		array_walk( $ratings, function( &$value, $key ) {
117 117
 			$label = _n( '%s star', '%s stars', $key, 'site-reviews' );
118 118
 			$value = sprintf( $label, $key );
@@ -120,12 +120,12 @@  discard block
 block discarded – undo
120 120
 		echo glsr( Builder::class )->label( __( 'Filter by rating', 'site-reviews' ), [
121 121
 			'class' => 'screen-reader-text',
122 122
 			'for' => 'rating',
123
-		]);
124
-		echo glsr( Builder::class )->select([
123
+		] );
124
+		echo glsr( Builder::class )->select( [
125 125
 			'name' => 'rating',
126 126
 			'options' => ['' => __( 'All ratings', 'site-reviews' )] + $ratings,
127 127
 			'value' => filter_input( INPUT_GET, 'rating' ),
128
-		]);
128
+		] );
129 129
 	}
130 130
 
131 131
 	/**
@@ -138,11 +138,11 @@  discard block
 block discarded – undo
138 138
 		echo glsr( Builder::class )->label( __( 'Filter by type', 'site-reviews' ), [
139 139
 			'class' => 'screen-reader-text',
140 140
 			'for' => 'review_type',
141
-		]);
142
-		echo glsr( Builder::class )->select([
141
+		] );
142
+		echo glsr( Builder::class )->select( [
143 143
 			'name' => 'review_type',
144 144
 			'options' => ['' => __( 'All types', 'site-reviews' )] + glsr()->reviewTypes,
145 145
 			'value' => filter_input( INPUT_GET, 'review_type' ),
146
-		]);
146
+		] );
147 147
 	}
148 148
 }
Please login to merge, or discard this patch.
plugin/Controllers/EditorController/Labels.php 2 patches
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -48,7 +48,9 @@
 block discarded – undo
48 48
 	public function filterUpdateMessages( array $messages )
49 49
 	{
50 50
 		$post = get_post();
51
-		if( !( $post instanceof WP_Post ))return;
51
+		if( !( $post instanceof WP_Post )) {
52
+			return;
53
+		}
52 54
 		$strings = $this->getReviewLabels();
53 55
 		$restored = filter_input( INPUT_GET, 'revision' );
54 56
 		if( $revisionTitle = wp_post_revision_title( intval( $restored ), false )) {
Please login to merge, or discard this patch.
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -17,7 +17,7 @@  discard block
 block discarded – undo
17 17
 			'savePending' => __( 'Save as Unapproved', 'site-reviews' ),
18 18
 			'published' => __( 'Approved', 'site-reviews' ),
19 19
 		];
20
-		if( $this->canModifyTranslation() && isset( $wp_scripts->registered['post']->extra['data'] )) {
20
+		if( $this->canModifyTranslation() && isset($wp_scripts->registered['post']->extra['data']) ) {
21 21
 			$l10n = &$wp_scripts->registered['post']->extra['data'];
22 22
 			foreach( $strings as $search => $replace ) {
23 23
 				$l10n = preg_replace( '/("'.$search.'":")([^"]+)/', "$1".$replace, $l10n );
@@ -33,9 +33,9 @@  discard block
 block discarded – undo
33 33
 	 */
34 34
 	public function filterPostStatusLabels( $translation, $text, $domain )
35 35
 	{
36
-		if( $this->canModifyTranslation( $domain )) {
36
+		if( $this->canModifyTranslation( $domain ) ) {
37 37
 			$replacements = $this->getStatusLabels();
38
-			if( array_key_exists( $text, $replacements )) {
38
+			if( array_key_exists( $text, $replacements ) ) {
39 39
 				$translation = $replacements[$text];
40 40
 			}
41 41
 		}
@@ -48,14 +48,14 @@  discard block
 block discarded – undo
48 48
 	public function filterUpdateMessages( array $messages )
49 49
 	{
50 50
 		$post = get_post();
51
-		if( !( $post instanceof WP_Post ))return;
51
+		if( !($post instanceof WP_Post) )return;
52 52
 		$strings = $this->getReviewLabels();
53 53
 		$restored = filter_input( INPUT_GET, 'revision' );
54
-		if( $revisionTitle = wp_post_revision_title( intval( $restored ), false )) {
54
+		if( $revisionTitle = wp_post_revision_title( intval( $restored ), false ) ) {
55 55
 			$restored = sprintf( $strings['restored'], $revisionTitle );
56 56
 		}
57
-		$scheduled_date = date_i18n( 'M j, Y @ H:i', strtotime( $post->post_date ));
58
-		$messages[ Application::POST_TYPE ] = [
57
+		$scheduled_date = date_i18n( 'M j, Y @ H:i', strtotime( $post->post_date ) );
58
+		$messages[Application::POST_TYPE] = [
59 59
 			 1 => $strings['updated'],
60 60
 			 4 => $strings['updated'],
61 61
 			 5 => $restored,
@@ -77,7 +77,7 @@  discard block
 block discarded – undo
77 77
 	 */
78 78
 	protected function canModifyTranslation( $domain = 'default' )
79 79
 	{
80
-		if( $domain != 'default' || empty( glsr_current_screen()->base )) {
80
+		if( $domain != 'default' || empty(glsr_current_screen()->base) ) {
81 81
 			return false;
82 82
 		}
83 83
 		return glsr_current_screen()->post_type == Application::POST_TYPE
@@ -112,7 +112,7 @@  discard block
 block discarded – undo
112 112
 	protected function getStatusLabels()
113 113
 	{
114 114
 		static $labels;
115
-		if( empty( $labels )) {
115
+		if( empty($labels) ) {
116 116
 			$labels = [
117 117
 				'Pending' => __( 'Unapproved', 'site-reviews' ),
118 118
 				'Pending Review' => __( 'Unapproved', 'site-reviews' ),
Please login to merge, or discard this patch.
plugin/Controllers/EditorController/Customization.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -57,7 +57,7 @@
 block discarded – undo
57 57
 	 */
58 58
 	protected function isReviewEditable()
59 59
 	{
60
-		$postId = intval( filter_input( INPUT_GET, 'post' ));
60
+		$postId = intval( filter_input( INPUT_GET, 'post' ) );
61 61
 		return $postId > 0
62 62
 			&& get_post_meta( $postId, 'review_type', true ) == 'local'
63 63
 			&& $this->isReviewEditor();
Please login to merge, or discard this patch.
Braces   +6 added lines, -2 removed lines patch added patch discarded remove patch
@@ -39,7 +39,9 @@  discard block
 block discarded – undo
39 39
 	 */
40 40
 	public function removeAutosave()
41 41
 	{
42
-		if( !$this->isReviewEditor() || $this->isReviewEditable() )return;
42
+		if( !$this->isReviewEditor() || $this->isReviewEditable() ) {
43
+			return;
44
+		}
43 45
 		wp_deregister_script( 'autosave' );
44 46
 	}
45 47
 
@@ -56,7 +58,9 @@  discard block
 block discarded – undo
56 58
 	 */
57 59
 	public function removePostTypeSupport()
58 60
 	{
59
-		if( !$this->isReviewEditor() || $this->isReviewEditable() )return;
61
+		if( !$this->isReviewEditor() || $this->isReviewEditable() ) {
62
+			return;
63
+		}
60 64
 		remove_post_type_support( Application::POST_TYPE, 'title' );
61 65
 		remove_post_type_support( Application::POST_TYPE, 'editor' );
62 66
 	}
Please login to merge, or discard this patch.
plugin/Modules/Email.php 2 patches
Braces   +9 added lines, -3 removed lines patch added patch discarded remove patch
@@ -72,7 +72,9 @@  discard block
 block discarded – undo
72 72
 	 */
73 73
 	public function send()
74 74
 	{
75
-		if( !$this->message || !$this->subject || !$this->to )return;
75
+		if( !$this->message || !$this->subject || !$this->to ) {
76
+			return;
77
+		}
76 78
 		$sent = wp_mail(
77 79
 			$this->to,
78 80
 			$this->subject,
@@ -90,8 +92,12 @@  discard block
 block discarded – undo
90 92
 	 */
91 93
 	public function buildPlainTextMessage( PHPMailer $phpmailer )
92 94
 	{
93
-		if( empty( $this->email ))return;
94
-		if( $phpmailer->ContentType === 'text/plain' || !empty( $phpmailer->AltBody ))return;
95
+		if( empty( $this->email )) {
96
+			return;
97
+		}
98
+		if( $phpmailer->ContentType === 'text/plain' || !empty( $phpmailer->AltBody )) {
99
+			return;
100
+		}
95 101
 		$message = $this->stripHtmlTags( $phpmailer->Body );
96 102
 		$phpmailer->AltBody = apply_filters( 'site-reviews/email/message', $message, 'text', $this );
97 103
 	}
Please login to merge, or discard this patch.
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -90,8 +90,8 @@  discard block
 block discarded – undo
90 90
 	 */
91 91
 	public function buildPlainTextMessage( PHPMailer $phpmailer )
92 92
 	{
93
-		if( empty( $this->email ))return;
94
-		if( $phpmailer->ContentType === 'text/plain' || !empty( $phpmailer->AltBody ))return;
93
+		if( empty($this->email) )return;
94
+		if( $phpmailer->ContentType === 'text/plain' || !empty($phpmailer->AltBody) )return;
95 95
 		$message = $this->stripHtmlTags( $phpmailer->Body );
96 96
 		$phpmailer->AltBody = apply_filters( 'site-reviews/email/message', $message, 'text', $this );
97 97
 	}
@@ -104,10 +104,10 @@  discard block
 block discarded – undo
104 104
 		$allowed = [
105 105
 			'bcc', 'cc', 'from', 'reply-to',
106 106
 		];
107
-		$headers = array_intersect_key( $this->email, array_flip( $allowed ));
107
+		$headers = array_intersect_key( $this->email, array_flip( $allowed ) );
108 108
 		$headers = array_filter( $headers );
109 109
 		foreach( $headers as $key => $value ) {
110
-			unset( $headers[$key] );
110
+			unset($headers[$key]);
111 111
 			$headers[] = $key.': '.$value;
112 112
 		}
113 113
 		$headers[] = 'Content-Type: text/html';
@@ -119,16 +119,16 @@  discard block
 block discarded – undo
119 119
 	 */
120 120
 	protected function buildHtmlMessage()
121 121
 	{
122
-		$template = trim( glsr( OptionManager::class )->get( 'settings.general.notification_message' ));
123
-		if( !empty( $template )) {
122
+		$template = trim( glsr( OptionManager::class )->get( 'settings.general.notification_message' ) );
123
+		if( !empty($template) ) {
124 124
 			$message = glsr( Template::class )->interpolate( $template, $this->email['template-tags'] );
125 125
 		}
126 126
 		else if( $this->email['template'] ) {
127 127
 			$message = glsr( Template::class )->build( 'templates/'.$this->email['template'], [
128 128
 				'context' => $this->email['template-tags'],
129
-			]);
129
+			] );
130 130
 		}
131
-		if( !isset( $message )) {
131
+		if( !isset($message) ) {
132 132
 			$message = $this->email['message'];
133 133
 		}
134 134
 		$message = $this->email['before'].$message.$this->email['after'];
@@ -139,7 +139,7 @@  discard block
 block discarded – undo
139 139
 		$message = str_replace( ']]>', ']]&gt;', $message );
140 140
 		$message = glsr( Template::class )->build( 'partials/email/index', [
141 141
 			'context' => ['message' => $message],
142
-		]);
142
+		] );
143 143
 		return apply_filters( 'site-reviews/email/message', stripslashes( $message ), 'html', $this );
144 144
 	}
145 145
 
@@ -149,7 +149,7 @@  discard block
 block discarded – undo
149 149
 	protected function normalize( array $email = [] )
150 150
 	{
151 151
 		$email = shortcode_atts( glsr( EmailDefaults::class )->defaults(), $email );
152
-		if( empty( $email['reply-to'] )) {
152
+		if( empty($email['reply-to']) ) {
153 153
 			$email['reply-to'] = $email['from'];
154 154
 		}
155 155
 		$this->email = apply_filters( 'site-reviews/email/compose', $email, $this );
Please login to merge, or discard this patch.
plugin/Modules/Schema/BaseType.php 2 patches
Braces   +6 added lines, -2 removed lines patch added patch discarded remove patch
@@ -212,7 +212,9 @@  discard block
 block discarded – undo
212 212
 		$newParents = $parents;
213 213
 		foreach( $parents as $parent ) {
214 214
 			$parentClass = glsr( Helper::class )->buildClassName( $parent, __NAMESPACE__ );
215
-			if( !class_exists( $parentClass ))continue;
215
+			if( !class_exists( $parentClass )) {
216
+				continue;
217
+			}
216 218
 			$newParents = array_merge( $newParents, $this->getParents( (new $parentClass)->parents ));
217 219
 		}
218 220
 		return array_values( array_unique( $newParents ));
@@ -226,7 +228,9 @@  discard block
 block discarded – undo
226 228
 		$parents = $this->getParents();
227 229
 		foreach( $parents as $parent ) {
228 230
 			$parentClass = glsr( Helper::class )->buildClassName( $parent, __NAMESPACE__ );
229
-			if( !class_exists( $parentClass ))continue;
231
+			if( !class_exists( $parentClass )) {
232
+				continue;
233
+			}
230 234
 			$this->allowed = array_values( array_unique( array_merge( (new $parentClass)->allowed, $this->allowed )));
231 235
 		}
232 236
 	}
Please login to merge, or discard this patch.
Spacing   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -39,7 +39,7 @@  discard block
 block discarded – undo
39 39
 	 */
40 40
 	public function __call( $method, array $arguments )
41 41
 	{
42
-		$value = isset( $arguments[0] )
42
+		$value = isset($arguments[0])
43 43
 			? $arguments[0]
44 44
 			: '';
45 45
 		return $this->setProperty( $method, $value );
@@ -96,9 +96,9 @@  discard block
 block discarded – undo
96 96
 	 * @param mixed $default
97 97
 	 * @return mixed
98 98
 	 */
99
-	public function getProperty( $property, $default = null)
99
+	public function getProperty( $property, $default = null )
100 100
 	{
101
-		return isset( $this->properties[$property] )
101
+		return isset($this->properties[$property])
102 102
 			? $this->properties[$property]
103 103
 			: $default;
104 104
 	}
@@ -166,7 +166,7 @@  discard block
 block discarded – undo
166 166
 	 */
167 167
 	public function offsetUnset( $offset )
168 168
 	{
169
-		unset( $this->properties[$offset] );
169
+		unset($this->properties[$offset]);
170 170
 	}
171 171
 
172 172
 	/**
@@ -212,16 +212,16 @@  discard block
 block discarded – undo
212 212
 	 */
213 213
 	protected function getParents( $parents = null )
214 214
 	{
215
-		if( !isset( $parents )) {
215
+		if( !isset($parents) ) {
216 216
 			$parents = $this->parents;
217 217
 		}
218 218
 		$newParents = $parents;
219 219
 		foreach( $parents as $parent ) {
220 220
 			$parentClass = glsr( Helper::class )->buildClassName( $parent, __NAMESPACE__ );
221
-			if( !class_exists( $parentClass ))continue;
222
-			$newParents = array_merge( $newParents, $this->getParents( (new $parentClass)->parents ));
221
+			if( !class_exists( $parentClass ) )continue;
222
+			$newParents = array_merge( $newParents, $this->getParents( (new $parentClass)->parents ) );
223 223
 		}
224
-		return array_values( array_unique( $newParents ));
224
+		return array_values( array_unique( $newParents ) );
225 225
 	}
226 226
 
227 227
 	/**
@@ -232,8 +232,8 @@  discard block
 block discarded – undo
232 232
 		$parents = $this->getParents();
233 233
 		foreach( $parents as $parent ) {
234 234
 			$parentClass = glsr( Helper::class )->buildClassName( $parent, __NAMESPACE__ );
235
-			if( !class_exists( $parentClass ))continue;
236
-			$this->allowed = array_values( array_unique( array_merge( (new $parentClass)->allowed, $this->allowed )));
235
+			if( !class_exists( $parentClass ) )continue;
236
+			$this->allowed = array_values( array_unique( array_merge( (new $parentClass)->allowed, $this->allowed ) ) );
237 237
 		}
238 238
 	}
239 239
 
@@ -243,17 +243,17 @@  discard block
 block discarded – undo
243 243
 	 */
244 244
 	protected function serializeProperty( $property )
245 245
 	{
246
-		if( is_array( $property )) {
246
+		if( is_array( $property ) ) {
247 247
 			return array_map( [$this, 'serializeProperty'], $property );
248 248
 		}
249 249
 		if( $property instanceof Type ) {
250 250
 			$property = $property->toArray();
251
-			unset( $property['@context'] );
251
+			unset($property['@context']);
252 252
 		}
253 253
 		if( $property instanceof DateTimeInterface ) {
254 254
 			$property = $property->format( DateTime::ATOM );
255 255
 		}
256
-		if( is_object( $property )) {
256
+		if( is_object( $property ) ) {
257 257
 			throw new InvalidProperty();
258 258
 		}
259 259
 		return $property;
Please login to merge, or discard this patch.
plugin/Modules/System.php 2 patches
Braces   +15 added lines, -5 removed lines patch added patch discarded remove patch
@@ -91,7 +91,9 @@  discard block
 block discarded – undo
91 91
 			get_mu_plugins(),
92 92
 			get_plugins( '/../'.basename( WPMU_PLUGIN_DIR ))
93 93
 		);
94
-		if( empty( $plugins ))return;
94
+		if( empty( $plugins )) {
95
+			return;
96
+		}
95 97
 		return $this->normalizePluginList( $plugins );
96 98
 	}
97 99
 
@@ -100,7 +102,9 @@  discard block
 block discarded – undo
100 102
 	 */
101 103
 	public function getMultisitePluginDetails()
102 104
 	{
103
-		if( !is_multisite() || empty( get_site_option( 'active_sitewide_plugins', [] )))return;
105
+		if( !is_multisite() || empty( get_site_option( 'active_sitewide_plugins', [] ))) {
106
+			return;
107
+		}
104 108
 		return $this->normalizePluginList( wp_get_active_network_plugins() );
105 109
 	}
106 110
 
@@ -157,7 +161,9 @@  discard block
 block discarded – undo
157 161
 		ksort( $settings );
158 162
 		$details = [];
159 163
 		foreach( $settings as $key => $value ) {
160
-			if( $helper->startsWith( 'strings', $key ) && $helper->endsWith( 'id', $key ))continue;
164
+			if( $helper->startsWith( 'strings', $key ) && $helper->endsWith( 'id', $key )) {
165
+				continue;
166
+			}
161 167
 			$value = htmlspecialchars( trim( preg_replace('/\s\s+/', '\\n', $value )), ENT_QUOTES, 'UTF-8' );
162 168
 			$details[$key] = $value;
163 169
 		}
@@ -230,7 +236,9 @@  discard block
 block discarded – undo
230 236
 			'WPE_APIKEY' => 'WP Engine',
231 237
 		];
232 238
 		foreach( $checks as $key => $value ) {
233
-			if( !$this->isWebhostCheckValid( $key ))continue;
239
+			if( !$this->isWebhostCheckValid( $key )) {
240
+				continue;
241
+			}
234 242
 			return $value;
235 243
 		}
236 244
 		return implode( ',', array_filter( [DB_HOST, filter_input( INPUT_SERVER, 'SERVER_NAME' )] ));
@@ -311,7 +319,9 @@  discard block
 block discarded – undo
311 319
 		];
312 320
 		array_walk( $settings, function( &$value, $setting ) use( $keys ) {
313 321
 			foreach( $keys as $key ) {
314
-				if( !glsr( Helper::class )->startsWith( $key, $setting ) || empty( $value ))continue;
322
+				if( !glsr( Helper::class )->startsWith( $key, $setting ) || empty( $value )) {
323
+					continue;
324
+				}
315 325
 				$value = str_repeat( '•', 13 );
316 326
 				return;
317 327
 			}
Please login to merge, or discard this patch.
Spacing   +23 added lines, -23 removed lines patch added patch discarded remove patch
@@ -40,7 +40,7 @@  discard block
 block discarded – undo
40 40
 			'setting' => 'Plugin Settings',
41 41
 			'reviews' => 'Review Counts',
42 42
 		];
43
-		$systemInfo = array_reduce( array_keys( $details ), function( $carry, $key ) use( $details ) {
43
+		$systemInfo = array_reduce( array_keys( $details ), function( $carry, $key ) use($details) {
44 44
 			$methodName = glsr( Helper::class )->buildMethodName( 'get-'.$key.'-details' );
45 45
 			if( method_exists( $this, $methodName ) && $systemDetails = $this->$methodName() ) {
46 46
 				return $carry.$this->implode( strtoupper( $details[$key] ), $systemDetails );
@@ -57,8 +57,8 @@  discard block
 block discarded – undo
57 57
 	{
58 58
 		$plugins = get_plugins();
59 59
 		$activePlugins = (array)get_option( 'active_plugins', [] );
60
-		$inactive = array_diff_key( $plugins, array_flip( $activePlugins ));
61
-		return $this->normalizePluginList( array_diff_key( $plugins, $inactive ));
60
+		$inactive = array_diff_key( $plugins, array_flip( $activePlugins ) );
61
+		return $this->normalizePluginList( array_diff_key( $plugins, $inactive ) );
62 62
 	}
63 63
 
64 64
 	/**
@@ -92,7 +92,7 @@  discard block
 block discarded – undo
92 92
 	public function getInactivePluginDetails()
93 93
 	{
94 94
 		$activePlugins = (array)get_option( 'active_plugins', [] );
95
-		return $this->normalizePluginList( array_diff_key( get_plugins(), array_flip( $activePlugins )));
95
+		return $this->normalizePluginList( array_diff_key( get_plugins(), array_flip( $activePlugins ) ) );
96 96
 	}
97 97
 
98 98
 	/**
@@ -102,9 +102,9 @@  discard block
 block discarded – undo
102 102
 	{
103 103
 		$plugins = array_merge(
104 104
 			get_mu_plugins(),
105
-			get_plugins( '/../'.basename( WPMU_PLUGIN_DIR ))
105
+			get_plugins( '/../'.basename( WPMU_PLUGIN_DIR ) )
106 106
 		);
107
-		if( empty( $plugins ))return;
107
+		if( empty($plugins) )return;
108 108
 		return $this->normalizePluginList( $plugins );
109 109
 	}
110 110
 
@@ -113,7 +113,7 @@  discard block
 block discarded – undo
113 113
 	 */
114 114
 	public function getMultisitePluginDetails()
115 115
 	{
116
-		if( !is_multisite() || empty( get_site_option( 'active_sitewide_plugins', [] )))return;
116
+		if( !is_multisite() || empty(get_site_option( 'active_sitewide_plugins', [] )) )return;
117 117
 		return $this->normalizePluginList( wp_get_active_network_plugins() );
118 118
 	}
119 119
 
@@ -139,11 +139,11 @@  discard block
 block discarded – undo
139 139
 			'Max Input Vars' => ini_get( 'max_input_vars' ),
140 140
 			'Memory Limit' => ini_get( 'memory_limit' ),
141 141
 			'Post Max Size' => ini_get( 'post_max_size' ),
142
-			'Session Cookie Path' => esc_html( ini_get( 'session.cookie_path' )),
143
-			'Session Name' => esc_html( ini_get( 'session.name' )),
144
-			'Session Save Path' => esc_html( ini_get( 'session.save_path' )),
145
-			'Session Use Cookies' => var_export( wp_validate_boolean( ini_get( 'session.use_cookies' )), true ),
146
-			'Session Use Only Cookies' => var_export( wp_validate_boolean( ini_get( 'session.use_only_cookies' )), true ),
142
+			'Session Cookie Path' => esc_html( ini_get( 'session.cookie_path' ) ),
143
+			'Session Name' => esc_html( ini_get( 'session.name' ) ),
144
+			'Session Save Path' => esc_html( ini_get( 'session.save_path' ) ),
145
+			'Session Use Cookies' => var_export( wp_validate_boolean( ini_get( 'session.use_cookies' ) ), true ),
146
+			'Session Use Only Cookies' => var_export( wp_validate_boolean( ini_get( 'session.use_only_cookies' ) ), true ),
147 147
 			'Upload Max Filesize' => ini_get( 'upload_max_filesize' ),
148 148
 		];
149 149
 	}
@@ -155,8 +155,8 @@  discard block
 block discarded – undo
155 155
 	{
156 156
 		$counts = glsr( CountsManager::class )->getCounts();
157 157
 		$counts = glsr( Helper::class )->flattenArray( $counts );
158
- 		array_walk( $counts, function( &$ratings ) use( $counts ) {
159
-			if( !is_array( $ratings )) {
158
+ 		array_walk( $counts, function( &$ratings ) use($counts) {
159
+			if( !is_array( $ratings ) ) {
160 160
 				glsr_log()
161 161
 					->error( '$ratings is not an array, possibly due to incorrectly imported reviews that are missing their review type.' )
162 162
 					->debug( $ratings )
@@ -195,8 +195,8 @@  discard block
 block discarded – undo
195 195
 		ksort( $settings );
196 196
 		$details = [];
197 197
 		foreach( $settings as $key => $value ) {
198
-			if( $helper->startsWith( 'strings', $key ) && $helper->endsWith( 'id', $key ))continue;
199
-			$value = htmlspecialchars( trim( preg_replace('/\s\s+/', '\\n', $value )), ENT_QUOTES, 'UTF-8' );
198
+			if( $helper->startsWith( 'strings', $key ) && $helper->endsWith( 'id', $key ) )continue;
199
+			$value = htmlspecialchars( trim( preg_replace( '/\s\s+/', '\\n', $value ) ), ENT_QUOTES, 'UTF-8' );
200 200
 			$details[$key] = $value;
201 201
 		}
202 202
 		return $details;
@@ -268,10 +268,10 @@  discard block
 block discarded – undo
268 268
 			'WPE_APIKEY' => 'WP Engine',
269 269
 		];
270 270
 		foreach( $checks as $key => $value ) {
271
-			if( !$this->isWebhostCheckValid( $key ))continue;
271
+			if( !$this->isWebhostCheckValid( $key ) )continue;
272 272
 			return $value;
273 273
 		}
274
-		return implode( ',', array_filter( [DB_HOST, filter_input( INPUT_SERVER, 'SERVER_NAME' )] ));
274
+		return implode( ',', array_filter( [DB_HOST, filter_input( INPUT_SERVER, 'SERVER_NAME' )] ) );
275 275
 	}
276 276
 
277 277
 	/**
@@ -292,8 +292,8 @@  discard block
 block discarded – undo
292 292
 	{
293 293
 		$plugins = get_plugins();
294 294
 		$activePlugins = (array)get_option( 'active_plugins', [] );
295
-		$inactive = $this->normalizePluginList( array_diff_key( $plugins, array_flip( $activePlugins )));
296
-		$active = $this->normalizePluginList( array_diff_key( $plugins, $inactive ));
295
+		$inactive = $this->normalizePluginList( array_diff_key( $plugins, array_flip( $activePlugins ) ) );
296
+		$active = $this->normalizePluginList( array_diff_key( $plugins, $inactive ) );
297 297
 		return $active + $inactive;
298 298
 	}
299 299
 
@@ -304,7 +304,7 @@  discard block
 block discarded – undo
304 304
 	protected function implode( $title, array $details )
305 305
 	{
306 306
 		$strings = ['['.$title.']'];
307
-		$padding = max( array_map( 'strlen', array_keys( $details )) );
307
+		$padding = max( array_map( 'strlen', array_keys( $details ) ) );
308 308
 		$padding = max( [$padding, static::PAD] );
309 309
 		foreach( $details as $key => $value ) {
310 310
 			$strings[] = is_string( $key )
@@ -347,9 +347,9 @@  discard block
 block discarded – undo
347 347
 		$keys = [
348 348
 			'licenses.', 'submissions.recaptcha.key', 'submissions.recaptcha.secret',
349 349
 		];
350
-		array_walk( $settings, function( &$value, $setting ) use( $keys ) {
350
+		array_walk( $settings, function( &$value, $setting ) use($keys) {
351 351
 			foreach( $keys as $key ) {
352
-				if( !glsr( Helper::class )->startsWith( $key, $setting ) || empty( $value ))continue;
352
+				if( !glsr( Helper::class )->startsWith( $key, $setting ) || empty($value) )continue;
353 353
 				$value = str_repeat( '•', 13 );
354 354
 				return;
355 355
 			}
Please login to merge, or discard this patch.
views/partials/addons/addon.php 2 patches
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -10,13 +10,13 @@
 block discarded – undo
10 10
 			<p>{{ description }}</p>
11 11
 		</div>
12 12
 		<div class="glsr-addon-footer">
13
-		<?php if( !is_wp_error( validate_plugin( $plugin ))) : ?>
14
-			<?php if( is_plugin_active( $plugin )) : ?>
13
+		<?php if( !is_wp_error( validate_plugin( $plugin ) ) ) : ?>
14
+			<?php if( is_plugin_active( $plugin ) ) : ?>
15 15
 			<span class="glsr-addon-link button button-secondary" disabled>
16 16
 				<?= __( 'Installed', 'site-reviews' ); ?>
17 17
 			</span>
18 18
 			<?php else: ?>
19
-			<a href="<?= wp_nonce_url( self_admin_url( 'plugins.php?action=activate&plugin='.$plugin ), 'activate-plugin_'.$plugin );?>" class="glsr-addon-link button button-secondary">
19
+			<a href="<?= wp_nonce_url( self_admin_url( 'plugins.php?action=activate&plugin='.$plugin ), 'activate-plugin_'.$plugin ); ?>" class="glsr-addon-link button button-secondary">
20 20
 				<?= __( 'Activate', 'site-reviews' ); ?>
21 21
 			</a>
22 22
 			<?php endif; ?>
Please login to merge, or discard this patch.
Braces   +10 added lines, -4 removed lines patch added patch discarded remove patch
@@ -15,14 +15,20 @@
 block discarded – undo
15 15
 			<span class="glsr-addon-link button button-secondary" disabled>
16 16
 				<?= __( 'Installed', 'site-reviews' ); ?>
17 17
 			</span>
18
-			<?php else: ?>
19
-			<a href="<?= wp_nonce_url( self_admin_url( 'plugins.php?action=activate&plugin='.$plugin ), 'activate-plugin_'.$plugin );?>" class="glsr-addon-link button button-secondary">
18
+			<?php else {
19
+	: ?>
20
+			<a href="<?= wp_nonce_url( self_admin_url( 'plugins.php?action=activate&plugin='.$plugin ), 'activate-plugin_'.$plugin );
21
+}
22
+?>" class="glsr-addon-link button button-secondary">
20 23
 				<?= __( 'Activate', 'site-reviews' ); ?>
21 24
 			</a>
22 25
 			<?php endif; ?>
23
-		<?php else: ?>
26
+		<?php else {
27
+	: ?>
24 28
 			<a href="{{ link }}" class="glsr-addon-link glsr-external button button-secondary">
25
-				<?= __( 'More Info', 'site-reviews' ); ?>
29
+				<?= __( 'More Info', 'site-reviews' );
30
+}
31
+?>
26 32
 			</a>
27 33
 		<?php endif; ?>
28 34
 		</div>
Please login to merge, or discard this patch.
activate.php 2 patches
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -60,7 +60,9 @@
 block discarded – undo
60 60
 	 */
61 61
 	public function deactivate( $plugin )
62 62
 	{
63
-		if( $this->isValid() )return;
63
+		if( $this->isValid() ) {
64
+			return;
65
+		}
64 66
 		$pluginSlug = plugin_basename( $this->file );
65 67
 		if( $plugin == $pluginSlug ) {
66 68
 			$this->redirect(); //exit
Please login to merge, or discard this patch.
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -30,7 +30,7 @@  discard block
 block discarded – undo
30 30
 		$this->versions = wp_parse_args( $versions, array(
31 31
 			'php' => static::MIN_PHP_VERSION,
32 32
 			'wordpress' => static::MIN_WORDPRESS_VERSION,
33
-		));
33
+		) );
34 34
 	}
35 35
 
36 36
 	/**
@@ -41,8 +41,8 @@  discard block
 block discarded – undo
41 41
 		if( $this->isValid() ) {
42 42
 			return true;
43 43
 		}
44
-		add_action( 'activated_plugin', array( $this, 'deactivate' ));
45
-		add_action( 'admin_notices', array( $this, 'deactivate' ));
44
+		add_action( 'activated_plugin', array( $this, 'deactivate' ) );
45
+		add_action( 'admin_notices', array( $this, 'deactivate' ) );
46 46
 		return false;
47 47
 	}
48 48
 
@@ -137,7 +137,7 @@  discard block
 block discarded – undo
137 137
 			filter_input( INPUT_GET, 'plugin_status' ),
138 138
 			filter_input( INPUT_GET, 'paged' ),
139 139
 			filter_input( INPUT_GET, 's' )
140
-		)));
140
+		) ) );
141 141
 		exit;
142 142
 	}
143 143
 }
Please login to merge, or discard this patch.