Passed
Push — master ( 25e85e...aade3e )
by Paul
03:50
created
plugin/Modules/Html/Partials/SiteReviewsSummary.php 2 patches
Spacing   +16 added lines, -16 removed lines patch added patch discarded remove patch
@@ -34,7 +34,7 @@  discard block
 block discarded – undo
34 34
 	{
35 35
 		$this->args = $args;
36 36
 		$this->reviews = glsr( Database::class )->getReviews( $args )->results;
37
-		if( empty( $this->reviews ) && $this->isHidden( 'if_empty' ))return;
37
+		if( empty($this->reviews) && $this->isHidden( 'if_empty' ) )return;
38 38
 		$this->rating = glsr( Rating::class )->getAverage( $this->reviews );
39 39
 		$this->generateSchema();
40 40
 		return glsr( Template::class )->build( 'templates/reviews-summary', [
@@ -46,7 +46,7 @@  discard block
 block discarded – undo
46 46
 				'stars' => $this->buildStars(),
47 47
 				'text' => $this->buildText(),
48 48
 			],
49
-		]);
49
+		] );
50 50
 	}
51 51
 
52 52
 	/**
@@ -54,16 +54,16 @@  discard block
 block discarded – undo
54 54
 	 */
55 55
 	protected function buildPercentage()
56 56
 	{
57
-		if( $this->isHidden( 'bars' ))return;
57
+		if( $this->isHidden( 'bars' ) )return;
58 58
 		$range = range( Rating::MAX_RATING, 1 );
59
-		$percentages = preg_filter( '/$/', '%', glsr( Rating::class )->getPercentages( $this->reviews ));
60
-		$bars = array_reduce( $range, function( $carry, $level ) use( $percentages ) {
59
+		$percentages = preg_filter( '/$/', '%', glsr( Rating::class )->getPercentages( $this->reviews ) );
60
+		$bars = array_reduce( $range, function( $carry, $level ) use($percentages) {
61 61
 			$label = $this->buildPercentageLabel( $this->args['labels'][$level] );
62 62
 			$background = $this->buildPercentageBackground( $percentages[$level] );
63 63
 			$percent = $this->buildPercentagePercent( $percentages[$level] );
64 64
 			return $carry.glsr( Builder::class )->div( $label.$background.$percent, [
65 65
 				'class' => 'glsr-bar',
66
-			]);
66
+			] );
67 67
 		});
68 68
 		return $this->wrap( 'percentage', $bars );
69 69
 	}
@@ -74,10 +74,10 @@  discard block
 block discarded – undo
74 74
 	 */
75 75
 	protected function buildPercentageBackground( $percent )
76 76
 	{
77
-		$backgroundPercent = glsr( Builder::class )->span([
77
+		$backgroundPercent = glsr( Builder::class )->span( [
78 78
 			'class' => 'glsr-bar-background-percent',
79 79
 			'style' => 'width:'.$percent,
80
-		]);
80
+		] );
81 81
 		return '<span class="glsr-bar-background">'.$backgroundPercent.'</span>';
82 82
 	}
83 83
 
@@ -104,7 +104,7 @@  discard block
 block discarded – undo
104 104
 	 */
105 105
 	protected function buildRating()
106 106
 	{
107
-		if( $this->isHidden( 'rating' ))return;
107
+		if( $this->isHidden( 'rating' ) )return;
108 108
 		return $this->wrap( 'rating', '<span>'.$this->rating.'</span>' );
109 109
 	}
110 110
 
@@ -113,10 +113,10 @@  discard block
 block discarded – undo
113 113
 	 */
114 114
 	protected function buildStars()
115 115
 	{
116
-		if( $this->isHidden( 'stars' ))return;
116
+		if( $this->isHidden( 'stars' ) )return;
117 117
 		$stars = glsr( Partial::class )->build( 'star-rating', [
118 118
 			'rating' => $this->rating,
119
-		]);
119
+		] );
120 120
 		return $this->wrap( 'stars', $stars );
121 121
 	}
122 122
 
@@ -125,9 +125,9 @@  discard block
 block discarded – undo
125 125
 	 */
126 126
 	protected function buildText()
127 127
 	{
128
-		if( $this->isHidden( 'summary' ))return;
128
+		if( $this->isHidden( 'summary' ) )return;
129 129
 		$count = count( $this->reviews );
130
-		if( empty( $this->args['text'] )) {
130
+		if( empty($this->args['text']) ) {
131 131
 			// @todo document this change
132 132
 			 $this->args['text'] = _nx(
133 133
 				'{rating} out of {max} stars (based on {num} review)',
@@ -138,7 +138,7 @@  discard block
 block discarded – undo
138 138
 			);
139 139
 		}
140 140
 		$summary = str_replace(
141
-			['{rating}','{max}', '{num}'],
141
+			['{rating}', '{max}', '{num}'],
142 142
 			[$this->rating, Rating::MAX_RATING, $count],
143 143
 			$this->args['text']
144 144
 		);
@@ -150,7 +150,7 @@  discard block
 block discarded – undo
150 150
 	 */
151 151
 	protected function generateSchema()
152 152
 	{
153
-		if( !wp_validate_boolean( $this->args['schema'] ))return;
153
+		if( !wp_validate_boolean( $this->args['schema'] ) )return;
154 154
 		glsr( Schema::class )->store(
155 155
 			glsr( Schema::class )->buildSummary( $this->args )
156 156
 		);
@@ -183,6 +183,6 @@  discard block
 block discarded – undo
183 183
 	{
184 184
 		return glsr( Builder::class )->div( $value, [
185 185
 			'class' => 'glsr-summary-'.$key,
186
-		]);
186
+		] );
187 187
 	}
188 188
 }
Please login to merge, or discard this patch.
Braces   +18 added lines, -6 removed lines patch added patch discarded remove patch
@@ -34,7 +34,9 @@  discard block
 block discarded – undo
34 34
 	{
35 35
 		$this->args = $args;
36 36
 		$this->reviews = glsr( Database::class )->getReviews( $args )->results;
37
-		if( empty( $this->reviews ) && $this->isHidden( 'if_empty' ))return;
37
+		if( empty( $this->reviews ) && $this->isHidden( 'if_empty' )) {
38
+			return;
39
+		}
38 40
 		$this->rating = glsr( Rating::class )->getAverage( $this->reviews );
39 41
 		$this->generateSchema();
40 42
 		return glsr( Template::class )->build( 'templates/reviews-summary', [
@@ -54,7 +56,9 @@  discard block
 block discarded – undo
54 56
 	 */
55 57
 	protected function buildPercentage()
56 58
 	{
57
-		if( $this->isHidden( 'bars' ))return;
59
+		if( $this->isHidden( 'bars' )) {
60
+			return;
61
+		}
58 62
 		$range = range( Rating::MAX_RATING, 1 );
59 63
 		$percentages = preg_filter( '/$/', '%', glsr( Rating::class )->getPercentages( $this->reviews ));
60 64
 		$bars = array_reduce( $range, function( $carry, $level ) use( $percentages ) {
@@ -104,7 +108,9 @@  discard block
 block discarded – undo
104 108
 	 */
105 109
 	protected function buildRating()
106 110
 	{
107
-		if( $this->isHidden( 'rating' ))return;
111
+		if( $this->isHidden( 'rating' )) {
112
+			return;
113
+		}
108 114
 		return $this->wrap( 'rating', '<span>'.$this->rating.'</span>' );
109 115
 	}
110 116
 
@@ -113,7 +119,9 @@  discard block
 block discarded – undo
113 119
 	 */
114 120
 	protected function buildStars()
115 121
 	{
116
-		if( $this->isHidden( 'stars' ))return;
122
+		if( $this->isHidden( 'stars' )) {
123
+			return;
124
+		}
117 125
 		$stars = glsr( Partial::class )->build( 'star-rating', [
118 126
 			'rating' => $this->rating,
119 127
 		]);
@@ -125,7 +133,9 @@  discard block
 block discarded – undo
125 133
 	 */
126 134
 	protected function buildText()
127 135
 	{
128
-		if( $this->isHidden( 'summary' ))return;
136
+		if( $this->isHidden( 'summary' )) {
137
+			return;
138
+		}
129 139
 		$count = count( $this->reviews );
130 140
 		if( empty( $this->args['text'] )) {
131 141
 			// @todo document this change
@@ -150,7 +160,9 @@  discard block
 block discarded – undo
150 160
 	 */
151 161
 	protected function generateSchema()
152 162
 	{
153
-		if( !wp_validate_boolean( $this->args['schema'] ))return;
163
+		if( !wp_validate_boolean( $this->args['schema'] )) {
164
+			return;
165
+		}
154 166
 		glsr( Schema::class )->store(
155 167
 			glsr( Schema::class )->buildSummary( $this->args )
156 168
 		);
Please login to merge, or discard this patch.
plugin/Router.php 2 patches
Spacing   +11 added lines, -11 removed lines patch added patch discarded remove patch
@@ -16,7 +16,7 @@  discard block
 block discarded – undo
16 16
 	public function routeAdminPostRequest()
17 17
 	{
18 18
 		$request = $this->getRequest();
19
-		if( !$this->isValidPostRequest( $request ))return;
19
+		if( !$this->isValidPostRequest( $request ) )return;
20 20
 		check_admin_referer( $request['action'] );
21 21
 		$this->routeRequest( 'admin', $request['action'], $request );
22 22
 	}
@@ -42,8 +42,8 @@  discard block
 block discarded – undo
42 42
 	{
43 43
 		if( is_admin() )return;
44 44
 		$request = $this->getRequest();
45
-		if( !$this->isValidPostRequest( $request ))return;
46
-		if( !$this->isValidPublicNonce( $request ))return;
45
+		if( !$this->isValidPostRequest( $request ) )return;
46
+		if( !$this->isValidPublicNonce( $request ) )return;
47 47
 		$this->routeRequest( 'public', $request['action'], $request );
48 48
 	}
49 49
 
@@ -53,11 +53,11 @@  discard block
 block discarded – undo
53 53
 	protected function checkAjaxNonce( array $request )
54 54
 	{
55 55
 		if( !is_user_logged_in() )return;
56
-		if( !isset( $request['nonce'] )) {
56
+		if( !isset($request['nonce']) ) {
57 57
 			glsr_log()->error( 'The AJAX request must include a nonce' )->info( $request );
58 58
 			wp_die();
59 59
 		}
60
-		if( !wp_verify_nonce( $request['nonce'], $request['action'] )) {
60
+		if( !wp_verify_nonce( $request['nonce'], $request['action'] ) ) {
61 61
 			glsr_log()->error( 'Nonce check failed for ajax request' )->info( $request );
62 62
 			wp_die( -1, 403 );
63 63
 		}
@@ -68,11 +68,11 @@  discard block
 block discarded – undo
68 68
 	 */
69 69
 	protected function checkAjaxRequest( array $request )
70 70
 	{
71
-		if( !isset( $request['action'] )) {
71
+		if( !isset($request['action']) ) {
72 72
 			glsr_log()->error( 'The AJAX request must include an action' )->info( $request );
73 73
 			wp_die();
74 74
 		}
75
-		if( empty( $request['ajax_request'] )) {
75
+		if( empty($request['ajax_request']) ) {
76 76
 			glsr_log()->error( 'The AJAX request look invalid' )->info( $request );
77 77
 			wp_die();
78 78
 		}
@@ -94,7 +94,7 @@  discard block
 block discarded – undo
94 94
 	 */
95 95
 	protected function isValidPostRequest( array $request = [] )
96 96
 	{
97
-		return !empty( $request['action'] ) && empty( $request['ajax_request'] );
97
+		return !empty($request['action']) && empty($request['ajax_request']);
98 98
 	}
99 99
 
100 100
 	/**
@@ -102,7 +102,7 @@  discard block
 block discarded – undo
102 102
 	 */
103 103
 	protected function isValidPublicNonce( array $request )
104 104
 	{
105
-		if( is_user_logged_in() && !wp_verify_nonce( $request['nonce'], $request['action'] )) {
105
+		if( is_user_logged_in() && !wp_verify_nonce( $request['nonce'], $request['action'] ) ) {
106 106
 			glsr_log()->error( 'Nonce check failed for public request' )->info( $request );
107 107
 			return false;
108 108
 		}
@@ -131,11 +131,11 @@  discard block
 block discarded – undo
131 131
 	protected function routeRequest( $type, $action, array $request = [] )
132 132
 	{
133 133
 		$actionHook = 'site-reviews/route/'.$type.'/request';
134
-		$controller = glsr( glsr( Helper::class )->buildClassName( $type.'-controller', 'Controllers' ));
134
+		$controller = glsr( glsr( Helper::class )->buildClassName( $type.'-controller', 'Controllers' ) );
135 135
 		$method = glsr( Helper::class )->buildMethodName( $action, 'router' );
136 136
 		$request = apply_filters( 'site-reviews/route/request', $request, $action, $type );
137 137
 		do_action( $actionHook, $action, $request );
138
-		if( is_callable( [$controller, $method] )) {
138
+		if( is_callable( [$controller, $method] ) ) {
139 139
 			call_user_func( [$controller, $method], $request );
140 140
 			return;
141 141
 		}
Please login to merge, or discard this patch.
Braces   +15 added lines, -5 removed lines patch added patch discarded remove patch
@@ -16,7 +16,9 @@  discard block
 block discarded – undo
16 16
 	public function routeAdminPostRequest()
17 17
 	{
18 18
 		$request = $this->getRequest();
19
-		if( !$this->isValidPostRequest( $request ))return;
19
+		if( !$this->isValidPostRequest( $request )) {
20
+			return;
21
+		}
20 22
 		check_admin_referer( $request['action'] );
21 23
 		$this->routeRequest( 'admin', $request['action'], $request );
22 24
 	}
@@ -40,10 +42,16 @@  discard block
 block discarded – undo
40 42
 	 */
41 43
 	public function routePublicPostRequest()
42 44
 	{
43
-		if( is_admin() )return;
45
+		if( is_admin() ) {
46
+			return;
47
+		}
44 48
 		$request = $this->getRequest();
45
-		if( !$this->isValidPostRequest( $request ))return;
46
-		if( !$this->isValidPublicNonce( $request ))return;
49
+		if( !$this->isValidPostRequest( $request )) {
50
+			return;
51
+		}
52
+		if( !$this->isValidPublicNonce( $request )) {
53
+			return;
54
+		}
47 55
 		$this->routeRequest( 'public', $request['action'], $request );
48 56
 	}
49 57
 
@@ -52,7 +60,9 @@  discard block
 block discarded – undo
52 60
 	 */
53 61
 	protected function checkAjaxNonce( array $request )
54 62
 	{
55
-		if( !is_user_logged_in() )return;
63
+		if( !is_user_logged_in() ) {
64
+			return;
65
+		}
56 66
 		if( !isset( $request['nonce'] )) {
57 67
 			glsr_log()->error( 'The AJAX request must include a nonce' )->info( $request );
58 68
 			wp_die();
Please login to merge, or discard this patch.
helpers.php 2 patches
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -6,7 +6,7 @@  discard block
 block discarded – undo
6 6
  */
7 7
 function glsr( $alias = null ) {
8 8
 	$app = \GeminiLabs\SiteReviews\Application::load();
9
-	return !empty( $alias )
9
+	return !empty($alias)
10 10
 		? $app->make( $alias )
11 11
 		: $app;
12 12
 }
@@ -15,10 +15,10 @@  discard block
 block discarded – undo
15 15
  * @return \WP_Screen|object
16 16
  */
17 17
 function glsr_current_screen() {
18
-	if( function_exists( 'get_current_screen' )) {
18
+	if( function_exists( 'get_current_screen' ) ) {
19 19
 		$screen = get_current_screen();
20 20
 	}
21
-	return empty( $screen )
21
+	return empty($screen)
22 22
 		? (object)array_fill_keys( ['base', 'id', 'post_type'], null )
23 23
 		: $screen;
24 24
 }
@@ -53,11 +53,11 @@  discard block
 block discarded – undo
53 53
  */
54 54
 function glsr_log() {
55 55
 	$args = func_get_args();
56
-	$context = isset( $args[1] )
56
+	$context = isset($args[1])
57 57
 		? $args[1]
58 58
 		: [];
59 59
 	$console = glsr( 'Modules\Console' );
60
-	return !empty( $args )
60
+	return !empty($args)
61 61
 		? $console->log( 'debug', $args[0], $context )
62 62
 		: $console;
63 63
 }
@@ -83,7 +83,7 @@  discard block
 block discarded – undo
83 83
  * @return void|object
84 84
  */
85 85
 function glsr_get_review( $post_id ) {
86
-	return glsr( 'Database' )->getReview( get_post( $post_id ));
86
+	return glsr( 'Database' )->getReview( get_post( $post_id ) );
87 87
 }
88 88
 
89 89
 /**
Please login to merge, or discard this patch.
Braces   +18 added lines, -9 removed lines patch added patch discarded remove patch
@@ -4,7 +4,8 @@  discard block
 block discarded – undo
4 4
 /**
5 5
  * @return mixed
6 6
  */
7
-function glsr( $alias = null ) {
7
+function glsr( $alias = null )
8
+{
8 9
 	$app = \GeminiLabs\SiteReviews\Application::load();
9 10
 	return !empty( $alias )
10 11
 		? $app->make( $alias )
@@ -14,7 +15,8 @@  discard block
 block discarded – undo
14 15
 /**
15 16
  * @return \WP_Screen|object
16 17
  */
17
-function glsr_current_screen() {
18
+function glsr_current_screen()
19
+{
18 20
 	if( function_exists( 'get_current_screen' )) {
19 21
 		$screen = get_current_screen();
20 22
 	}
@@ -26,7 +28,8 @@  discard block
 block discarded – undo
26 28
 /**
27 29
  * @return \GeminiLabs\SiteReviews\Database
28 30
  */
29
-function glsr_db() {
31
+function glsr_db()
32
+{
30 33
 	return glsr( 'Database' );
31 34
 }
32 35
 
@@ -34,7 +37,8 @@  discard block
 block discarded – undo
34 37
  * @param mixed ...$vars
35 38
  * @return void
36 39
  */
37
-function glsr_debug( ...$vars ) {
40
+function glsr_debug( ...$vars )
41
+{
38 42
 	if( count( $vars ) == 1 ) {
39 43
 		$value = htmlspecialchars( print_r( $vars[0], true ), ENT_QUOTES, 'UTF-8' );
40 44
 		printf( '<div class="glsr-debug"><pre>%s</pre></div>', $value );
@@ -51,7 +55,8 @@  discard block
 block discarded – undo
51 55
 /**
52 56
  * @return \GeminiLabs\SiteReviews\Modules\Console
53 57
  */
54
-function glsr_log() {
58
+function glsr_log()
59
+{
55 60
 	$args = func_get_args();
56 61
 	$context = isset( $args[1] )
57 62
 		? $args[1]
@@ -67,14 +72,16 @@  discard block
 block discarded – undo
67 72
  * @param mixed $fallback
68 73
  * @return string|array
69 74
  */
70
-function glsr_get_option( $path = '', $fallback = '' ) {
75
+function glsr_get_option( $path = '', $fallback = '' )
76
+{
71 77
 	return glsr( 'Database\OptionManager' )->get( 'settings.'.$path, $fallback );
72 78
 }
73 79
 
74 80
 /**
75 81
  * @return array
76 82
  */
77
-function glsr_get_options() {
83
+function glsr_get_options()
84
+{
78 85
 	return glsr( 'Database\OptionManager' )->get( 'settings' );
79 86
 }
80 87
 
@@ -82,7 +89,8 @@  discard block
 block discarded – undo
82 89
  * @param int $post_id
83 90
  * @return void|object
84 91
  */
85
-function glsr_get_review( $post_id ) {
92
+function glsr_get_review( $post_id )
93
+{
86 94
 	return glsr( 'Database' )->getReview( get_post( $post_id ));
87 95
 }
88 96
 
@@ -90,6 +98,7 @@  discard block
 block discarded – undo
90 98
  * @return array
91 99
  * @todo document change of $reviews->reviews to $reviews->results
92 100
  */
93
-function glsr_get_reviews( array $args = array() ) {
101
+function glsr_get_reviews( array $args = array() )
102
+{
94 103
 	return glsr( 'Database' )->getReviews( $args );
95 104
 }
Please login to merge, or discard this patch.
plugin/Modules/System.php 2 patches
Spacing   +21 added lines, -21 removed lines patch added patch discarded remove patch
@@ -37,7 +37,7 @@  discard block
 block discarded – undo
37 37
 			'inactive-plugin' => 'Inactive Plugins',
38 38
 			'setting' => 'Plugin Settings',
39 39
 		];
40
-		$systemInfo = array_reduce( array_keys( $details ), function( $carry, $key ) use( $details ) {
40
+		$systemInfo = array_reduce( array_keys( $details ), function( $carry, $key ) use($details) {
41 41
 			$methodName = glsr( Helper::class )->buildMethodName( 'get-'.$key.'-details' );
42 42
 			if( method_exists( $this, $methodName ) && $systemDetails = $this->$methodName() ) {
43 43
 				return $carry.$this->implode( $details[$key], $systemDetails );
@@ -54,8 +54,8 @@  discard block
 block discarded – undo
54 54
 	{
55 55
 		$plugins = get_plugins();
56 56
 		$activePlugins = (array)get_option( 'active_plugins', [] );
57
-		$inactive = array_diff_key( $plugins, array_flip( $activePlugins ));
58
-		return $this->normalizePluginList( array_diff_key( $plugins, $inactive ));
57
+		$inactive = array_diff_key( $plugins, array_flip( $activePlugins ) );
58
+		return $this->normalizePluginList( array_diff_key( $plugins, $inactive ) );
59 59
 	}
60 60
 
61 61
 	/**
@@ -79,7 +79,7 @@  discard block
 block discarded – undo
79 79
 	public function getInactivePluginDetails()
80 80
 	{
81 81
 		$activePlugins = (array)get_option( 'active_plugins', [] );
82
-		return $this->normalizePluginList( array_diff_key( get_plugins(), array_flip( $activePlugins )));
82
+		return $this->normalizePluginList( array_diff_key( get_plugins(), array_flip( $activePlugins ) ) );
83 83
 	}
84 84
 
85 85
 	/**
@@ -89,9 +89,9 @@  discard block
 block discarded – undo
89 89
 	{
90 90
 		$plugins = array_merge(
91 91
 			get_mu_plugins(),
92
-			get_plugins( '/../'.basename( WPMU_PLUGIN_DIR ))
92
+			get_plugins( '/../'.basename( WPMU_PLUGIN_DIR ) )
93 93
 		);
94
-		if( empty( $plugins ))return;
94
+		if( empty($plugins) )return;
95 95
 		return $this->normalizePluginList( $plugins );
96 96
 	}
97 97
 
@@ -100,7 +100,7 @@  discard block
 block discarded – undo
100 100
 	 */
101 101
 	public function getMultisitePluginDetails()
102 102
 	{
103
-		if( !is_multisite() || empty( get_site_option( 'active_sitewide_plugins', [] )))return;
103
+		if( !is_multisite() || empty(get_site_option( 'active_sitewide_plugins', [] )) )return;
104 104
 		return $this->normalizePluginList( wp_get_active_network_plugins() );
105 105
 	}
106 106
 
@@ -122,11 +122,11 @@  discard block
 block discarded – undo
122 122
 			'Max Input Vars' => ini_get( 'max_input_vars' ),
123 123
 			'Memory Limit' => ini_get( 'memory_limit' ),
124 124
 			'Post Max Size' => ini_get( 'post_max_size' ),
125
-			'Session Cookie Path' => esc_html( ini_get( 'session.cookie_path' )),
126
-			'Session Name' => esc_html( ini_get( 'session.name' )),
127
-			'Session Save Path' => esc_html( ini_get( 'session.save_path' )),
128
-			'Session Use Cookies' => var_export( wp_validate_boolean( ini_get( 'session.use_cookies' )), true ),
129
-			'Session Use Only Cookies' => var_export( wp_validate_boolean( ini_get( 'session.use_only_cookies' )), true ),
125
+			'Session Cookie Path' => esc_html( ini_get( 'session.cookie_path' ) ),
126
+			'Session Name' => esc_html( ini_get( 'session.name' ) ),
127
+			'Session Save Path' => esc_html( ini_get( 'session.save_path' ) ),
128
+			'Session Use Cookies' => var_export( wp_validate_boolean( ini_get( 'session.use_cookies' ) ), true ),
129
+			'Session Use Only Cookies' => var_export( wp_validate_boolean( ini_get( 'session.use_only_cookies' ) ), true ),
130 130
 			'Upload Max Filesize' => ini_get( 'upload_max_filesize' ),
131 131
 		];
132 132
 	}
@@ -157,8 +157,8 @@  discard block
 block discarded – undo
157 157
 		ksort( $settings );
158 158
 		$details = [];
159 159
 		foreach( $settings as $key => $value ) {
160
-			if( $helper->startsWith( 'strings', $key ) && $helper->endsWith( 'id', $key ))continue;
161
-			$value = htmlspecialchars( trim( preg_replace('/\s\s+/', '\\n', $value )), ENT_QUOTES, 'UTF-8' );
160
+			if( $helper->startsWith( 'strings', $key ) && $helper->endsWith( 'id', $key ) )continue;
161
+			$value = htmlspecialchars( trim( preg_replace( '/\s\s+/', '\\n', $value ) ), ENT_QUOTES, 'UTF-8' );
162 162
 			$details[$key] = $value;
163 163
 		}
164 164
 		return $details;
@@ -230,10 +230,10 @@  discard block
 block discarded – undo
230 230
 			'WPE_APIKEY' => 'WP Engine',
231 231
 		];
232 232
 		foreach( $checks as $key => $value ) {
233
-			if( !$this->isWebhostCheckValid( $key ))continue;
233
+			if( !$this->isWebhostCheckValid( $key ) )continue;
234 234
 			return $value;
235 235
 		}
236
-		return implode( ',', array_filter( [DB_HOST, filter_input( INPUT_SERVER, 'SERVER_NAME' )] ));
236
+		return implode( ',', array_filter( [DB_HOST, filter_input( INPUT_SERVER, 'SERVER_NAME' )] ) );
237 237
 	}
238 238
 
239 239
 	/**
@@ -254,8 +254,8 @@  discard block
 block discarded – undo
254 254
 	{
255 255
 		$plugins = get_plugins();
256 256
 		$activePlugins = (array)get_option( 'active_plugins', [] );
257
-		$inactive = $this->normalizePluginList( array_diff_key( $plugins, array_flip( $activePlugins )));
258
-		$active = $this->normalizePluginList( array_diff_key( $plugins, $inactive ));
257
+		$inactive = $this->normalizePluginList( array_diff_key( $plugins, array_flip( $activePlugins ) ) );
258
+		$active = $this->normalizePluginList( array_diff_key( $plugins, $inactive ) );
259 259
 		return $active + $inactive;
260 260
 	}
261 261
 
@@ -266,7 +266,7 @@  discard block
 block discarded – undo
266 266
 	protected function implode( $title, array $details )
267 267
 	{
268 268
 		$strings = ['['.$title.']'];
269
-		$padding = max( array_map( 'strlen', array_keys( $details )) );
269
+		$padding = max( array_map( 'strlen', array_keys( $details ) ) );
270 270
 		$padding = max( [$padding, static::PAD] );
271 271
 		foreach( $details as $key => $value ) {
272 272
 			$strings[] = is_string( $key )
@@ -309,9 +309,9 @@  discard block
 block discarded – undo
309 309
 		$keys = [
310 310
 			'licenses.', 'submissions.recaptcha.key', 'submissions.recaptcha.secret',
311 311
 		];
312
-		array_walk( $settings, function( &$value, $setting ) use( $keys ) {
312
+		array_walk( $settings, function( &$value, $setting ) use($keys) {
313 313
 			foreach( $keys as $key ) {
314
-				if( !glsr( Helper::class )->startsWith( $key, $setting ) || empty( $value ))continue;
314
+				if( !glsr( Helper::class )->startsWith( $key, $setting ) || empty($value) )continue;
315 315
 				$value = str_repeat( '•', 13 );
316 316
 				return;
317 317
 			}
Please login to merge, or discard this patch.
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.
plugin/Modules/Html/Settings.php 1 patch
Spacing   +15 added lines, -15 removed lines patch added patch discarded remove patch
@@ -29,7 +29,7 @@  discard block
 block discarded – undo
29 29
 	 */
30 30
 	protected function getFieldDefault( array $field )
31 31
 	{
32
-		return isset( $field['default'] )
32
+		return isset($field['default'])
33 33
 			? $field['default']
34 34
 			: '';
35 35
 	}
@@ -40,7 +40,7 @@  discard block
 block discarded – undo
40 40
 	protected function getSettingFields( $path )
41 41
 	{
42 42
 		$settings = glsr( DefaultsManager::class )->settings();
43
-		return array_filter( $settings, function( $key ) use( $path ) {
43
+		return array_filter( $settings, function( $key ) use($path) {
44 44
 			return glsr( Helper::class )->startsWith( $path, $key );
45 45
 		}, ARRAY_FILTER_USE_KEY );
46 46
 	}
@@ -55,8 +55,8 @@  discard block
 block discarded – undo
55 55
 			$field = wp_parse_args( $field, [
56 56
 				'is_setting' => true,
57 57
 				'name' => $name,
58
-			]);
59
-			$rows.= new Field( $this->normalize( $field ));
58
+			] );
59
+			$rows .= new Field( $this->normalize( $field ) );
60 60
 		}
61 61
 		return $rows;
62 62
 	}
@@ -67,7 +67,7 @@  discard block
 block discarded – undo
67 67
 	 */
68 68
 	protected function getTemplateData( $id )
69 69
 	{
70
-		$fields = $this->getSettingFields( $this->normalizeSettingPath( $id ));
70
+		$fields = $this->getSettingFields( $this->normalizeSettingPath( $id ) );
71 71
 		return [
72 72
 			'context' => [
73 73
 				'rows' => $this->getSettingRows( $fields ),
@@ -81,12 +81,12 @@  discard block
 block discarded – undo
81 81
 	 */
82 82
 	protected function getTemplateDataForAddons( $id )
83 83
 	{
84
-		$fields = $this->getSettingFields( $this->normalizeSettingPath( $id ));
84
+		$fields = $this->getSettingFields( $this->normalizeSettingPath( $id ) );
85 85
 		$settings = glsr( Helper::class )->convertDotNotationArray( $fields );
86 86
 		$settingKeys = array_keys( $settings['settings']['addons'] );
87 87
 		$results = [];
88 88
 		foreach( $settingKeys as $key ) {
89
-			$addonFields = array_filter( $fields, function( $path ) use( $key ) {
89
+			$addonFields = array_filter( $fields, function( $path ) use($key) {
90 90
 				return glsr( Helper::class )->startsWith( 'settings.addons.'.$key, $path );
91 91
 			}, ARRAY_FILTER_USE_KEY );
92 92
 			$results[$key] = $this->getSettingRows( $addonFields );
@@ -102,7 +102,7 @@  discard block
 block discarded – undo
102 102
 	protected function getTemplateDataForTranslations()
103 103
 	{
104 104
 		$translations = glsr( Translator::class )->renderAll();
105
-		$class = empty( $translations )
105
+		$class = empty($translations)
106 106
 			? 'glsr-hidden'
107 107
 			: '';
108 108
 		return [
@@ -125,7 +125,7 @@  discard block
 block discarded – undo
125 125
 			$path,
126 126
 			glsr( Helper::class )->getPathValue( $path, glsr()->defaults )
127 127
 		);
128
-		if( is_array( $expectedValue )) {
128
+		if( is_array( $expectedValue ) ) {
129 129
 			return !in_array( $optionValue, $expectedValue );
130 130
 		}
131 131
 		return $optionValue != $expectedValue;
@@ -147,13 +147,13 @@  discard block
 block discarded – undo
147 147
 	 */
148 148
 	protected function normalizeDependsOn( array $field )
149 149
 	{
150
-		if( !empty( $field['depends_on'] ) && is_array( $field['depends_on'] )) {
150
+		if( !empty($field['depends_on']) && is_array( $field['depends_on'] ) ) {
151 151
 			$path = key( $field['depends_on'] );
152 152
 			$expectedValue = $field['depends_on'][$path];
153
-			$field['data-depends'] = json_encode([
153
+			$field['data-depends'] = json_encode( [
154 154
 				'name' => glsr( Helper::class )->convertPathToName( $path, OptionManager::databaseKey() ),
155 155
 				'value' => $expectedValue,
156
-			], JSON_HEX_APOS|JSON_HEX_QUOT );
156
+			], JSON_HEX_APOS | JSON_HEX_QUOT );
157 157
 			$field['is_hidden'] = $this->isFieldHidden( $path, $expectedValue );
158 158
 		}
159 159
 		return $field;
@@ -164,9 +164,9 @@  discard block
 block discarded – undo
164 164
 	 */
165 165
 	protected function normalizeLabelAndLegend( array $field )
166 166
 	{
167
-		if( !empty( $field['label'] )) {
167
+		if( !empty($field['label']) ) {
168 168
 			$field['legend'] = $field['label'];
169
-			unset( $field['label'] );
169
+			unset($field['label']);
170 170
 		}
171 171
 		else {
172 172
 			$field['is_valid'] = false;
@@ -180,7 +180,7 @@  discard block
 block discarded – undo
180 180
 	 */
181 181
 	protected function normalizeValue( array $field )
182 182
 	{
183
-		if( !isset( $field['value'] )) {
183
+		if( !isset($field['value']) ) {
184 184
 			$field['value'] = glsr( OptionManager::class )->get(
185 185
 				$field['name'],
186 186
 				$this->getFieldDefault( $field )
Please login to merge, or discard this patch.
plugin/Modules/Console.php 1 patch
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -108,7 +108,7 @@  discard block
 block discarded – undo
108 108
 	 */
109 109
 	public function get()
110 110
 	{
111
-		return empty( $this->log )
111
+		return empty($this->log)
112 112
 			? __( 'Log is empty', 'site-reviews' )
113 113
 			: $this->log;
114 114
 	}
@@ -120,11 +120,11 @@  discard block
 block discarded – undo
120 120
 	public function humanSize( $valueIfEmpty = null )
121 121
 	{
122 122
 		$bytes = $this->size();
123
-		if( empty( $bytes ) && is_string( $valueIfEmpty )) {
123
+		if( empty($bytes) && is_string( $valueIfEmpty ) ) {
124 124
 			return $valueIfEmpty;
125 125
 		}
126
-		$exponent = floor( log( max( $bytes, 1 ), 1024 ));
127
-		return round( $bytes / pow( 1024, $exponent ), 2 ).' '.['bytes','KB','MB','GB'][$exponent];
126
+		$exponent = floor( log( max( $bytes, 1 ), 1024 ) );
127
+		return round( $bytes / pow( 1024, $exponent ), 2 ).' '.['bytes', 'KB', 'MB', 'GB'][$exponent];
128 128
 	}
129 129
 
130 130
 	/**
@@ -148,9 +148,9 @@  discard block
 block discarded – undo
148 148
 	{
149 149
 		$constants = (new ReflectionClass( __CLASS__ ))->getConstants();
150 150
 		$constants = (array)apply_filters( 'site-reviews/log-levels', $constants );
151
-		if( in_array( $level, $constants, true )) {
151
+		if( in_array( $level, $constants, true ) ) {
152 152
 			$entry = $this->buildLogEntry( $level, $message, $context );
153
-			file_put_contents( $this->file, $entry, FILE_APPEND|LOCK_EX );
153
+			file_put_contents( $this->file, $entry, FILE_APPEND | LOCK_EX );
154 154
 			$this->reset();
155 155
 		}
156 156
 		return $this;
@@ -223,7 +223,7 @@  discard block
 block discarded – undo
223 223
 	 */
224 224
 	protected function interpolate( $message, $context = [] )
225 225
 	{
226
-		if( $this->isObjectOrArray( $message ) || !is_array( $context )) {
226
+		if( $this->isObjectOrArray( $message ) || !is_array( $context ) ) {
227 227
 			return print_r( $message, true );
228 228
 		}
229 229
 		$replace = [];
@@ -251,7 +251,7 @@  discard block
 block discarded – undo
251 251
 		if( $value instanceof DateTime ) {
252 252
 			$value = $value->format( 'Y-m-d H:i:s' );
253 253
 		}
254
-		else if( $this->isObjectOrArray( $value )) {
254
+		else if( $this->isObjectOrArray( $value ) ) {
255 255
 			$value = json_encode( $value );
256 256
 		}
257 257
 		return (string)$value;
@@ -266,7 +266,7 @@  discard block
 block discarded – undo
266 266
 			$this->clear();
267 267
 			file_put_contents(
268 268
 				$this->file,
269
-				$this->buildLogEntry( 'info', __( 'Console was automatically reset (128 KB max size)', 'site-reviews' ))
269
+				$this->buildLogEntry( 'info', __( 'Console was automatically reset (128 KB max size)', 'site-reviews' ) )
270 270
 			);
271 271
 		}
272 272
 	}
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.
views/pages/tools/sync.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -38,7 +38,7 @@
 block discarded – undo
38 38
 <form method="post">
39 39
 	<div class="tablenav bottom">
40 40
 		<div class="actions">
41
-			<?php if( count( $sites ) > 1) : ?>
41
+			<?php if( count( $sites ) > 1 ) : ?>
42 42
 			<label for="bulk-action-selector-bottom" class="screen-reader-text">Select a site to sync</label>
43 43
 			<select name="{{ id }}[site]">
44 44
 				<option>- <?= __( 'Select a Site', 'site-reviews' ); ?> -</option>
Please login to merge, or discard this patch.
Braces   +5 added lines, -2 removed lines patch added patch discarded remove patch
@@ -46,9 +46,12 @@
 block discarded – undo
46 46
 				<option value="<?= $slug; ?>" class="hide-if-no-js"><?= $details['name']; ?></option>
47 47
 				<?php endforeach; ?>
48 48
 			</select>
49
-			<?php else: ?>
49
+			<?php else {
50
+	: ?>
50 51
 				<?php foreach( $sites as $slug => $details ) : ?>
51
-				<input type="hidden" name="{{ id }}[site]" value="<?= $slug; ?>">
52
+				<input type="hidden" name="{{ id }}[site]" value="<?= $slug;
53
+}
54
+?>">
52 55
 				<?php endforeach; ?>
53 56
 			<?php endif; ?>
54 57
 			<input type="hidden" name="{{ id }}[action]" value="sync-reviews">
Please login to merge, or discard this patch.
views/pages/addons/index.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -13,7 +13,7 @@  discard block
 block discarded – undo
13 13
 				'title' => 'Tripadvisor Reviews',
14 14
 			],
15 15
 			'plugin' => 'site-reviews-tripadvisor/site-reviews-tripadvisor.php',
16
-		]);
16
+		] );
17 17
 		$template->render( 'partials/addons/addon', [
18 18
 			'context' => [
19 19
 				'description' => __( 'Sync your Yelp reviews and display them on your site.', 'site-reviews' ),
@@ -22,7 +22,7 @@  discard block
 block discarded – undo
22 22
 				'title' => 'Yelp Reviews',
23 23
 			],
24 24
 			'plugin' => 'site-reviews-yelp/site-reviews-yelp.php',
25
-		]);
25
+		] );
26 26
 	?>
27 27
 	</div>
28 28
 </div>
Please login to merge, or discard this patch.