Test Failed
Push — develop ( 3e3cf0...6ef91c )
by Paul
03:33
created
src/MetaBox/Instruction.php 2 patches
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -22,11 +22,11 @@  discard block
 block discarded – undo
22 22
 	{
23 23
 		$instructions = array_reduce( $this->getInstructionGroups(), function( $html, $metabox ) {
24 24
 			$fields = $this->getInstructionFields( $metabox );
25
-			if( empty( $fields )) {
25
+			if( empty( $fields ) ) {
26 26
 				return $html;
27 27
 			}
28 28
 			return $html . sprintf( '<p><strong>%s</strong></p><pre class="my-sites nav-tab-active misc-pub-section">%s</pre>',
29
-				$metabox['title'],
29
+				$metabox[ 'title' ],
30 30
 				$fields
31 31
 			);
32 32
 		});
@@ -38,10 +38,10 @@  discard block
 block discarded – undo
38 38
 	 */
39 39
 	protected function getInstructionFields( $metabox )
40 40
 	{
41
-		$skipFields = ['custom_html', 'divider', 'heading', 'taxonomy'];
42
-		return array_reduce( $metabox['fields'], function( $html, $field ) use( $metabox, $skipFields ) {
43
-			return $this->validate( $field['condition'] ) && !in_array( $field['type'], $skipFields )
44
-				? $html . $this->filter( 'instruction', "PostMeta::get('{$field['slug']}');", $field, $metabox ) . PHP_EOL
41
+		$skipFields = [ 'custom_html', 'divider', 'heading', 'taxonomy' ];
42
+		return array_reduce( $metabox[ 'fields' ], function( $html, $field ) use( $metabox, $skipFields ) {
43
+			return $this->validate( $field[ 'condition' ] ) && !in_array( $field[ 'type' ], $skipFields )
44
+				? $html . $this->filter( 'instruction', "PostMeta::get('{$field[ 'slug' ]}');", $field, $metabox ) . PHP_EOL
45 45
 				: $html;
46 46
 		});
47 47
 	}
@@ -52,7 +52,7 @@  discard block
 block discarded – undo
52 52
 	protected function getInstructionGroups()
53 53
 	{
54 54
 		return array_filter( $this->metaboxes, function( $metabox ) {
55
-			return $this->validate( $metabox['condition'] )
55
+			return $this->validate( $metabox[ 'condition' ] )
56 56
 				&& $this->hasPostType( $metabox );
57 57
 		});
58 58
 	}
@@ -66,11 +66,11 @@  discard block
 block discarded – undo
66 66
 		return [
67 67
 			'infodiv' => [
68 68
 				'context' => 'side',
69
-				'fields' => [[
69
+				'fields' => [ [
70 70
 					'slug' => '',
71 71
 					'std' => $this->generateInstructions(),
72 72
 					'type' => 'custom_html',
73
-				]],
73
+				] ],
74 74
 				'post_types' => $this->getPostTypes(),
75 75
 				'priority' => 'low',
76 76
 				'title' => __( 'How to use in your theme', 'pollux' ),
@@ -85,7 +85,7 @@  discard block
 block discarded – undo
85 85
 	{
86 86
 		return $this->filter( 'show/instructions', count( array_filter( $this->metaboxes, function( $metabox ) {
87 87
 			return $this->show( false, $metabox );
88
-		})) > 0 );
88
+		}) ) > 0 );
89 89
 	}
90 90
 
91 91
 	/**
Please login to merge, or discard this patch.
Braces   +11 added lines, -5 removed lines patch added patch discarded remove patch
@@ -20,7 +20,8 @@  discard block
 block discarded – undo
20 20
 	 */
21 21
 	protected function generateInstructions()
22 22
 	{
23
-		$instructions = array_reduce( $this->getInstructionGroups(), function( $html, $metabox ) {
23
+		$instructions = array_reduce( $this->getInstructionGroups(), function( $html, $metabox )
24
+		{
24 25
 			$fields = $this->getInstructionFields( $metabox );
25 26
 			if( empty( $fields )) {
26 27
 				return $html;
@@ -39,7 +40,8 @@  discard block
 block discarded – undo
39 40
 	protected function getInstructionFields( $metabox )
40 41
 	{
41 42
 		$skipFields = ['custom_html', 'divider', 'heading', 'taxonomy'];
42
-		return array_reduce( $metabox['fields'], function( $html, $field ) use( $metabox, $skipFields ) {
43
+		return array_reduce( $metabox['fields'], function( $html, $field ) use( $metabox, $skipFields )
44
+		{
43 45
 			return $this->validate( $field['condition'] ) && !in_array( $field['type'], $skipFields )
44 46
 				? $html . $this->filter( 'instruction', "PostMeta::get('{$field['slug']}');", $field, $metabox ) . PHP_EOL
45 47
 				: $html;
@@ -51,7 +53,8 @@  discard block
 block discarded – undo
51 53
 	 */
52 54
 	protected function getInstructionGroups()
53 55
 	{
54
-		return array_filter( $this->metaboxes, function( $metabox ) {
56
+		return array_filter( $this->metaboxes, function( $metabox )
57
+		{
55 58
 			return $this->validate( $metabox['condition'] )
56 59
 				&& $this->hasPostType( $metabox );
57 60
 		});
@@ -62,7 +65,9 @@  discard block
 block discarded – undo
62 65
 	 */
63 66
 	protected function initInstructions()
64 67
 	{
65
-		if( !$this->showInstructions() )return;
68
+		if( !$this->showInstructions() ) {
69
+			return;
70
+		}
66 71
 		return [
67 72
 			'infodiv' => [
68 73
 				'context' => 'side',
@@ -83,7 +88,8 @@  discard block
 block discarded – undo
83 88
 	 */
84 89
 	protected function showInstructions()
85 90
 	{
86
-		return $this->filter( 'show/instructions', count( array_filter( $this->metaboxes, function( $metabox ) {
91
+		return $this->filter( 'show/instructions', count( array_filter( $this->metaboxes, function( $metabox )
92
+		{
87 93
 			return $this->show( false, $metabox );
88 94
 		})) > 0 );
89 95
 	}
Please login to merge, or discard this patch.
src/Application.php 2 patches
Spacing   +19 added lines, -19 removed lines patch added patch discarded remove patch
@@ -18,7 +18,7 @@  discard block
 block discarded – undo
18 18
 	public $gatekeeper;
19 19
 	public $id;
20 20
 	public $name;
21
-	public $notices = [];
21
+	public $notices = [ ];
22 22
 	public $version;
23 23
 
24 24
 	/**
@@ -31,8 +31,8 @@  discard block
 block discarded – undo
31 31
 
32 32
 	public function __construct()
33 33
 	{
34
-		$this->file = realpath( dirname( dirname( __FILE__ )) . '/pollux.php' );
35
-		$this->gatekeeper = new GateKeeper( plugin_basename( $this->file ));
34
+		$this->file = realpath( dirname( dirname( __FILE__ ) ) . '/pollux.php' );
35
+		$this->gatekeeper = new GateKeeper( plugin_basename( $this->file ) );
36 36
 
37 37
 		$data = get_file_data( $this->file, array(
38 38
 			'id' => 'Text Domain',
@@ -57,11 +57,11 @@  discard block
 block discarded – undo
57 57
 		$basename = plugin_basename( $this->file );
58 58
 		$controller = $this->make( 'Controller' );
59 59
 
60
-		add_action( 'admin_enqueue_scripts',           array( $controller, 'registerAssets' ));
61
-		add_action( 'admin_init',                      array( $controller, 'removeDashboardWidgets' ));
62
-		add_action( 'wp_before_admin_bar_render',      array( $controller, 'removeWordPressMenu' ));
63
-		add_filter( "plugin_action_links_{$basename}", array( $controller, 'filterPluginLinks' ));
64
-		add_filter( 'admin_footer_text',               array( $controller, 'filterWordPressFooter' ));
60
+		add_action( 'admin_enqueue_scripts', array( $controller, 'registerAssets' ) );
61
+		add_action( 'admin_init', array( $controller, 'removeDashboardWidgets' ) );
62
+		add_action( 'wp_before_admin_bar_render', array( $controller, 'removeWordPressMenu' ) );
63
+		add_filter( "plugin_action_links_{$basename}", array( $controller, 'filterPluginLinks' ) );
64
+		add_filter( 'admin_footer_text', array( $controller, 'filterWordPressFooter' ) );
65 65
 
66 66
 		// Disallow indexing of the site on non-production environments
67 67
 		if( !$this->environment( 'production' ) && !is_admin() ) {
@@ -76,7 +76,7 @@  discard block
 block discarded – undo
76 76
 	public function environment( $checkFor = '' )
77 77
 	{
78 78
 		$environment = defined( 'WP_ENV' ) ? WP_ENV : 'production';
79
-		if( !empty( $checkFor )) {
79
+		if( !empty( $checkFor ) ) {
80 80
 			return $environment == $checkFor;
81 81
 		}
82 82
 		return $environment;
@@ -95,11 +95,11 @@  discard block
 block discarded – undo
95 95
 			trailingslashit( trailingslashit( $theme->theme_root ) . $theme->template ),
96 96
 			trailingslashit( WP_CONTENT_DIR ),
97 97
 			trailingslashit( ABSPATH ),
98
-			trailingslashit( dirname( ABSPATH )),
99
-			trailingslashit( dirname( dirname( ABSPATH ))),
100
-		]);
98
+			trailingslashit( dirname( ABSPATH ) ),
99
+			trailingslashit( dirname( dirname( ABSPATH ) ) ),
100
+		] );
101 101
 		foreach( (array) $locations as $location ) {
102
-			if( !file_exists( $location . $filename ))continue;
102
+			if( !file_exists( $location . $filename ) )continue;
103 103
 			return $location . $filename;
104 104
 		}
105 105
 		return null;
@@ -132,14 +132,14 @@  discard block
 block discarded – undo
132 132
 	 * @param string $view
133 133
 	 * @return bool
134 134
 	 */
135
-	public function render( $view, array $data = [] )
135
+	public function render( $view, array $data = [ ] )
136 136
 	{
137 137
 		$file = apply_filters( 'pollux/views/file',
138
-			$this->path( sprintf( 'views/%s.php', str_replace( '.php', '', $view ))),
138
+			$this->path( sprintf( 'views/%s.php', str_replace( '.php', '', $view ) ) ),
139 139
 			$view,
140 140
 			$data
141 141
 		);
142
-		if( file_exists( $file )) {
142
+		if( file_exists( $file ) ) {
143 143
 			extract( $data );
144 144
 			return include $file;
145 145
 		}
@@ -152,7 +152,7 @@  discard block
 block discarded – undo
152 152
 	 */
153 153
 	public function url( $path = '' )
154 154
 	{
155
-		return esc_url( plugin_dir_url( $this->file ) . ltrim( trim( $path ), '/' ));
155
+		return esc_url( plugin_dir_url( $this->file ) . ltrim( trim( $path ), '/' ) );
156 156
 	}
157 157
 
158 158
 	/**
@@ -184,7 +184,7 @@  discard block
 block discarded – undo
184 184
 	 */
185 185
 	protected function loadHooks()
186 186
 	{
187
-		if( $file = $this->app->getFile( 'pollux-hooks.php' )) {
187
+		if( $file = $this->app->getFile( 'pollux-hooks.php' ) ) {
188 188
 			include_once $file;
189 189
 		}
190 190
 	}
@@ -198,6 +198,6 @@  discard block
 block discarded – undo
198 198
 			'ArchiveMeta' => 'GeminiLabs\Pollux\Facades\ArchiveMeta',
199 199
 			'PostMeta' => 'GeminiLabs\Pollux\Facades\PostMeta',
200 200
 			'SiteMeta' => 'GeminiLabs\Pollux\Facades\SiteMeta',
201
-		)))->register();
201
+		) ) )->register();
202 202
 	}
203 203
 }
Please login to merge, or discard this patch.
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -99,7 +99,9 @@
 block discarded – undo
99 99
 			trailingslashit( dirname( dirname( ABSPATH ))),
100 100
 		]);
101 101
 		foreach( (array) $locations as $location ) {
102
-			if( !file_exists( $location . $filename ))continue;
102
+			if( !file_exists( $location . $filename )) {
103
+				continue;
104
+			}
103 105
 			return $location . $filename;
104 106
 		}
105 107
 		return null;
Please login to merge, or discard this patch.
src/Config/ConfigManager.php 2 patches
Spacing   +19 added lines, -19 removed lines patch added patch discarded remove patch
@@ -42,13 +42,13 @@  discard block
 block discarded – undo
42 42
 		$yamlFile = $this->getYamlFile();
43 43
 		$yaml = $this->normalizeYamlValues( $this->normalize(
44 44
 			$this->parseYaml( file_get_contents( $yamlFile ), $yamlFile )
45
-		));
46
-		if( !$yaml['disable_config'] ) {
47
-			$config = array_filter( (array) get_option( Config::id(), [] ));
45
+		) );
46
+		if( !$yaml[ 'disable_config' ] ) {
47
+			$config = array_filter( (array) get_option( Config::id(), [ ] ) );
48 48
 		}
49 49
 		return empty( $config )
50
-			? $this->setTimestamp( $yaml, filemtime( $yamlFile ))
51
-			: $this->normalizeYamlValues( $this->normalize( $config ));
50
+			? $this->setTimestamp( $yaml, filemtime( $yamlFile ) )
51
+			: $this->normalizeYamlValues( $this->normalize( $config ) );
52 52
 	}
53 53
 
54 54
 	/**
@@ -58,15 +58,15 @@  discard block
 block discarded – undo
58 58
 	public function compile( $force = false )
59 59
 	{
60 60
 		$configFile = $this->getCompileDestination();
61
-		if( $force || $this->shouldCompile( $configFile )) {
61
+		if( $force || $this->shouldCompile( $configFile ) ) {
62 62
 			$config = $this->normalizeArray( $this->options );
63 63
 			if( $this->parseError ) {
64 64
 				return (object) $config;
65 65
 			}
66 66
 			file_put_contents( $configFile, sprintf( '<?php // DO NOT MODIFY THIS FILE DIRECTLY!%sreturn (object) %s;',
67 67
 				PHP_EOL,
68
-				$this->parseRawStrings( var_export( $this->setTimestamp( $config ), true ))
69
-			));
68
+				$this->parseRawStrings( var_export( $this->setTimestamp( $config ), true ) )
69
+			) );
70 70
 		}
71 71
 		return include $configFile;
72 72
 	}
@@ -77,7 +77,7 @@  discard block
 block discarded – undo
77 77
 	public function convertArrayToYaml( array $array )
78 78
 	{
79 79
 		return !empty( $array )
80
-			? trim( $this->parseRawStrings( $this->dumpYaml( $array )))
80
+			? trim( $this->parseRawStrings( $this->dumpYaml( $array ) ) )
81 81
 			: '';
82 82
 	}
83 83
 
@@ -97,7 +97,7 @@  discard block
 block discarded – undo
97 97
 	 */
98 98
 	public function getYamlFile()
99 99
 	{
100
-		if( $file = $this->app->getFile( 'pollux.yml' )) {
100
+		if( $file = $this->app->getFile( 'pollux.yml' ) ) {
101 101
 			return $file;
102 102
 		}
103 103
 		return $this->app->path( 'defaults.yml' );
@@ -109,10 +109,10 @@  discard block
 block discarded – undo
109 109
 	public function normalizeArray( array $array )
110 110
 	{
111 111
 		array_walk( $array, function( &$value, $key ) {
112
-			if( !is_numeric( $value ) && is_string( $value )) {
112
+			if( !is_numeric( $value ) && is_string( $value ) ) {
113 113
 				$value = $this->parseYaml( $value, $key );
114 114
 				if( $this->parseError == $key ) {
115
-					$value = [];
115
+					$value = [ ];
116 116
 				}
117 117
 			}
118 118
 		});
@@ -137,7 +137,7 @@  discard block
 block discarded – undo
137 137
 	public function setTimestamp( array $config, $timestamp = null )
138 138
 	{
139 139
 		$timestamp || $timestamp = time();
140
-		$config['updated'] = $timestamp;
140
+		$config[ 'updated' ] = $timestamp;
141 141
 		return $config;
142 142
 	}
143 143
 
@@ -162,7 +162,7 @@  discard block
 block discarded – undo
162 162
 		return wp_parse_args(
163 163
 			$config,
164 164
 			$this->parseYaml(
165
-				file_get_contents( $this->app->path( 'defaults.yml' )),
165
+				file_get_contents( $this->app->path( 'defaults.yml' ) ),
166 166
 				$this->app->path( 'defaults.yml' )
167 167
 			)
168 168
 		);
@@ -176,13 +176,13 @@  discard block
 block discarded – undo
176 176
 	protected function parseRawStrings( $configString )
177 177
 	{
178 178
 		$strings = apply_filters( 'pollux/config/raw_strings', static::RAW_STRINGS );
179
-		if( empty( $strings )) {
179
+		if( empty( $strings ) ) {
180 180
 			return $configString;
181 181
 		}
182 182
 		$pattern = '/(\')((' . implode( '|', $strings ) . ')\(?.+\))(\')/';
183 183
 		return stripslashes(
184 184
 			preg_replace_callback( $pattern, function( $matches ) {
185
-				return str_replace( "''", "'", $matches[2] );
185
+				return str_replace( "''", "'", $matches[ 2 ] );
186 186
 			}, $configString )
187 187
 		);
188 188
 	}
@@ -201,13 +201,13 @@  discard block
 block discarded – undo
201 201
 			if( $file ) {
202 202
 				$file = sprintf( '<code>%s</code>', $file );
203 203
 			}
204
-			$this->app->make( 'Notice' )->addError([
204
+			$this->app->make( 'Notice' )->addError( [
205 205
 				sprintf( '<strong>Pollux Error:</strong> Unable to parse config at line %s (near "%s").',
206 206
 					$e->getParsedLine(),
207 207
 					$e->getSnippet()
208 208
 				),
209 209
 				$file
210
-			]);
210
+			] );
211 211
 			return $value;
212 212
 		}
213 213
 	}
@@ -218,7 +218,7 @@  discard block
 block discarded – undo
218 218
 	 */
219 219
 	protected function shouldCompile( $configFile )
220 220
 	{
221
-		if( !file_exists( $configFile )) {
221
+		if( !file_exists( $configFile ) ) {
222 222
 			return true;
223 223
 		}
224 224
 		$config = include $configFile;
Please login to merge, or discard this patch.
Braces   +6 added lines, -3 removed lines patch added patch discarded remove patch
@@ -108,7 +108,8 @@  discard block
 block discarded – undo
108 108
 	 */
109 109
 	public function normalizeArray( array $array )
110 110
 	{
111
-		array_walk( $array, function( &$value, $key ) {
111
+		array_walk( $array, function( &$value, $key )
112
+		{
112 113
 			if( !is_numeric( $value ) && is_string( $value )) {
113 114
 				$value = $this->parseYaml( $value, $key );
114 115
 				if( $this->parseError == $key ) {
@@ -124,7 +125,8 @@  discard block
 block discarded – undo
124 125
 	 */
125 126
 	public function normalizeYamlValues( array $array )
126 127
 	{
127
-		return array_map( function( $value ) {
128
+		return array_map( function( $value )
129
+		{
128 130
 			return is_array( $value )
129 131
 				? $this->convertArrayToYaml( $value )
130 132
 				: $value;
@@ -181,7 +183,8 @@  discard block
 block discarded – undo
181 183
 		}
182 184
 		$pattern = '/(\')((' . implode( '|', $strings ) . ')\(?.+\))(\')/';
183 185
 		return stripslashes(
184
-			preg_replace_callback( $pattern, function( $matches ) {
186
+			preg_replace_callback( $pattern, function( $matches )
187
+			{
185 188
 				return str_replace( "''", "'", $matches[2] );
186 189
 			}, $configString )
187 190
 		);
Please login to merge, or discard this patch.