Passed
Push — develop ( bd72f8...b35e9b )
by Paul
03:06
created
src/Settings/Settings.php 1 patch
Spacing   +34 added lines, -34 removed lines patch added patch discarded remove patch
@@ -42,14 +42,14 @@  discard block
 block discarded – undo
42 42
 
43 43
 		$this->normalize( $this->app->config->{static::ID} );
44 44
 
45
-		add_action( 'pollux/'.static::ID.'/init',                [$this, 'addSubmitMetaBox'] );
46
-		add_action( 'current_screen',                            [$this, 'register'] );
47
-		add_action( 'admin_menu',                                [$this, 'registerPage'] );
48
-		add_action( 'admin_menu',                                [$this, 'registerSetting'] );
49
-		add_action( 'pollux/'.static::ID.'/init',                [$this, 'resetPage'] );
50
-		add_action( 'admin_print_footer_scripts',                [$this, 'renderFooterScript'] );
51
-		add_filter( 'pollux/'.static::ID.'/instruction',         [$this, 'filterInstruction'], 10, 3 );
52
-		add_filter( 'pollux/'.static::ID.'/before/instructions', [$this, 'filterBeforeInstructions'] );
45
+		add_action( 'pollux/' . static::ID . '/init', [ $this, 'addSubmitMetaBox' ] );
46
+		add_action( 'current_screen', [ $this, 'register' ] );
47
+		add_action( 'admin_menu', [ $this, 'registerPage' ] );
48
+		add_action( 'admin_menu', [ $this, 'registerSetting' ] );
49
+		add_action( 'pollux/' . static::ID . '/init', [ $this, 'resetPage' ] );
50
+		add_action( 'admin_print_footer_scripts', [ $this, 'renderFooterScript' ] );
51
+		add_filter( 'pollux/' . static::ID . '/instruction', [ $this, 'filterInstruction' ], 10, 3 );
52
+		add_filter( 'pollux/' . static::ID . '/before/instructions', [ $this, 'filterBeforeInstructions' ] );
53 53
 	}
54 54
 
55 55
 	public function canProceed()
@@ -67,11 +67,11 @@  discard block
 block discarded – undo
67 67
 		call_user_func_array( 'add_meta_box', $this->filter( 'metabox/submit', [
68 68
 			'submitdiv',
69 69
 			__( 'Save Settings', 'pollux' ),
70
-			[$this, 'renderSubmitMetaBox'],
70
+			[ $this, 'renderSubmitMetaBox' ],
71 71
 			$this->hook,
72 72
 			'side',
73 73
 			'high',
74
-		]));
74
+		] ) );
75 75
 	}
76 76
 
77 77
 	/**
@@ -90,7 +90,7 @@  discard block
 block discarded – undo
90 90
 	 */
91 91
 	public function filterInstruction( $instruction, array $field, array $metabox )
92 92
 	{
93
-		return sprintf( "SiteMeta::%s('%s');", $metabox['slug'], $field['slug'] );
93
+		return sprintf( "SiteMeta::%s('%s');", $metabox[ 'slug' ], $field[ 'slug' ] );
94 94
 	}
95 95
 
96 96
 	/**
@@ -100,10 +100,10 @@  discard block
 block discarded – undo
100 100
 	 */
101 101
 	public function filterSavedSettings( $settings )
102 102
 	{
103
-		if( is_null( $settings )) {
104
-			$settings = [];
103
+		if( is_null( $settings ) ) {
104
+			$settings = [ ];
105 105
 		}
106
-		return $this->filter( 'save', array_merge( $this->getSettings(), $settings ));
106
+		return $this->filter( 'save', array_merge( $this->getSettings(), $settings ) );
107 107
 	}
108 108
 
109 109
 	/**
@@ -123,8 +123,8 @@  discard block
 block discarded – undo
123 123
 	 */
124 124
 	public function register()
125 125
 	{
126
-		if(( new Helper )->getCurrentScreen()->id != $this->hook )return;
127
-		if( $this->app->gatekeeper->hasDependency( self::DEPENDENCY )) {
126
+		if( ( new Helper )->getCurrentScreen()->id != $this->hook )return;
127
+		if( $this->app->gatekeeper->hasDependency( self::DEPENDENCY ) ) {
128 128
 			foreach( parent::register() as $metabox ) {
129 129
 				new RWMetaBox( $metabox, static::ID, $this );
130 130
 			}
@@ -132,7 +132,7 @@  discard block
 block discarded – undo
132 132
 		add_screen_option( 'layout_columns', [
133 133
 			'max' => 2,
134 134
 			'default' => 2,
135
-		]);
135
+		] );
136 136
 		$this->action( 'init' );
137 137
 	}
138 138
 
@@ -147,10 +147,10 @@  discard block
 block discarded – undo
147 147
 			__( 'Site Settings', 'pollux' ),
148 148
 			static::CAPABILITY,
149 149
 			static::id(),
150
-			[$this, 'renderPage'],
150
+			[ $this, 'renderPage' ],
151 151
 			'dashicons-screenoptions',
152 152
 			1313
153
-		]));
153
+		] ) );
154 154
 	}
155 155
 
156 156
 	/**
@@ -159,7 +159,7 @@  discard block
 block discarded – undo
159 159
 	 */
160 160
 	public function registerSetting()
161 161
 	{
162
-		register_setting( static::id(), static::id(), [$this, 'filterSavedSettings'] );
162
+		register_setting( static::id(), static::id(), [ $this, 'filterSavedSettings' ] );
163 163
 	}
164 164
 
165 165
 	/**
@@ -168,12 +168,12 @@  discard block
 block discarded – undo
168 168
 	 */
169 169
 	public function renderFooterScript()
170 170
 	{
171
-		if(( new Helper )->getCurrentScreen()->id != $this->hook )return;
171
+		if( ( new Helper )->getCurrentScreen()->id != $this->hook )return;
172 172
 		$this->app->render( 'settings/script', [
173 173
 			'confirm' => __( 'Are you sure want to do this?', 'pollux' ),
174 174
 			'hook' => $this->hook,
175 175
 			'id' => static::id(),
176
-		]);
176
+		] );
177 177
 	}
178 178
 
179 179
 	/**
@@ -186,7 +186,7 @@  discard block
 block discarded – undo
186 186
 			'columns' => get_current_screen()->get_columns(),
187 187
 			'heading' => __( 'Site Settings', 'pollux' ),
188 188
 			'id' => static::id(),
189
-		]);
189
+		] );
190 190
 	}
191 191
 
192 192
 	/**
@@ -203,9 +203,9 @@  discard block
 block discarded – undo
203 203
 		];
204 204
 		$this->app->render( 'settings/submit', [
205 205
 			'reset' => __( 'Reset all', 'pollux' ),
206
-			'reset_url' => esc_url( add_query_arg( $query, admin_url( $pagenow ))),
206
+			'reset_url' => esc_url( add_query_arg( $query, admin_url( $pagenow ) ) ),
207 207
 			'submit' => get_submit_button( __( 'Save', 'pollux' ), 'primary', 'submit', false ),
208
-		]);
208
+		] );
209 209
 	}
210 210
 
211 211
 	/**
@@ -217,15 +217,15 @@  discard block
 block discarded – undo
217 217
 		if( filter_input( INPUT_GET, 'page' ) !== static::id()
218 218
 			|| filter_input( INPUT_GET, 'action' ) !== 'reset'
219 219
 		)return;
220
-		if( wp_verify_nonce( filter_input( INPUT_GET, '_wpnonce' ), $this->hook )) {
220
+		if( wp_verify_nonce( filter_input( INPUT_GET, '_wpnonce' ), $this->hook ) ) {
221 221
 			update_option( static::id(), $this->getDefaults() );
222 222
 			add_settings_error( static::id(), 'reset', __( 'Reset successful.', 'pollux' ), 'updated' );
223 223
 		}
224 224
 		else {
225
-			add_settings_error( static::id(), 'failed', __( 'Failed to reset. Please try again.', 'pollux' ));
225
+			add_settings_error( static::id(), 'failed', __( 'Failed to reset. Please try again.', 'pollux' ) );
226 226
 		}
227 227
 		set_transient( 'settings_errors', get_settings_errors(), 30 );
228
-		wp_safe_redirect( add_query_arg( 'settings-updated', 'true',  wp_get_referer() ));
228
+		wp_safe_redirect( add_query_arg( 'settings-updated', 'true', wp_get_referer() ) );
229 229
 		exit;
230 230
 	}
231 231
 
@@ -236,7 +236,7 @@  discard block
 block discarded – undo
236 236
 	protected function filterArrayByKey( array $array, $key )
237 237
 	{
238 238
 		return array_filter( $array, function( $value ) use( $key ) {
239
-			return !empty( $value[$key] );
239
+			return !empty( $value[ $key ] );
240 240
 		});
241 241
 	}
242 242
 
@@ -249,11 +249,11 @@  discard block
 block discarded – undo
249 249
 
250 250
 		array_walk( $metaboxes, function( &$metabox ) {
251 251
 			$fields = array_map( function( $field ) {
252
-				$field = wp_parse_args( $field, ['std' => ''] );
253
-				return [$field['slug'] => $field['std']];
254
-			}, $this->filterArrayByKey( $metabox['fields'], 'slug' ));
252
+				$field = wp_parse_args( $field, [ 'std' => '' ] );
253
+				return [ $field[ 'slug' ] => $field[ 'std' ] ];
254
+			}, $this->filterArrayByKey( $metabox[ 'fields' ], 'slug' ) );
255 255
 			$metabox = [
256
-				$metabox['slug'] => call_user_func_array( 'array_merge', $fields ),
256
+				$metabox[ 'slug' ] => call_user_func_array( 'array_merge', $fields ),
257 257
 			];
258 258
 		});
259 259
 		return call_user_func_array( 'array_merge', $metaboxes );
@@ -271,7 +271,7 @@  discard block
 block discarded – undo
271 271
 	 */
272 272
 	protected function normalizeFieldName( $name, array $data, $parentId )
273 273
 	{
274
-		return sprintf( '%s[%s][%s]', static::id(), $parentId, $data['slug'] );
274
+		return sprintf( '%s[%s][%s]', static::id(), $parentId, $data[ 'slug' ] );
275 275
 	}
276 276
 
277 277
 	/**
Please login to merge, or discard this patch.
src/MetaBox/Condition.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -46,9 +46,9 @@  discard block
 block discarded – undo
46 46
 	protected function normalizeCondition( $conditions )
47 47
 	{
48 48
 		$conditions = ( new Helper )->toArray( $conditions );
49
-		if( count( array_filter( array_keys( $conditions ), 'is_string' )) == 0 ) {
49
+		if( count( array_filter( array_keys( $conditions ), 'is_string' ) ) == 0 ) {
50 50
 			foreach( $conditions as $key ) {
51
-				$conditions[str_replace( '!', '', $key )] = substr( $key, 0, 1 ) == '!' ? 0 : 1;
51
+				$conditions[ str_replace( '!', '', $key ) ] = substr( $key, 0, 1 ) == '!' ? 0 : 1;
52 52
 			}
53 53
 			$conditions = array_filter( $conditions, function( $key ) {
54 54
 				return !is_numeric( $key );
@@ -56,7 +56,7 @@  discard block
 block discarded – undo
56 56
 		}
57 57
 		return array_intersect_key(
58 58
 			$conditions,
59
-			array_flip( $this->filter( 'conditions', static::conditions() ))
59
+			array_flip( $this->filter( 'conditions', static::conditions() ) )
60 60
 		);
61 61
 	}
62 62
 
@@ -102,7 +102,7 @@  discard block
 block discarded – undo
102 102
 	 */
103 103
 	protected function validateIsFrontPage( $value )
104 104
 	{
105
-		return $value == ( $this->getPostId() == get_option( 'page_on_front' ));
105
+		return $value == ( $this->getPostId() == get_option( 'page_on_front' ) );
106 106
 	}
107 107
 
108 108
 	/**
@@ -111,7 +111,7 @@  discard block
 block discarded – undo
111 111
 	 */
112 112
 	protected function validateIsHome( $value )
113 113
 	{
114
-		return $value == ( $this->getPostId() == get_option( 'page_for_posts' ));
114
+		return $value == ( $this->getPostId() == get_option( 'page_for_posts' ) );
115 115
 	}
116 116
 
117 117
 	/**
@@ -120,7 +120,7 @@  discard block
 block discarded – undo
120 120
 	 */
121 121
 	protected function validateIsPageTemplate( $value )
122 122
 	{
123
-		return basename( get_page_template_slug( $this->getPostId() )) == $value;
123
+		return basename( get_page_template_slug( $this->getPostId() ) ) == $value;
124 124
 	}
125 125
 
126 126
 	/**
Please login to merge, or discard this patch.
src/PostType/Archive.php 1 patch
Spacing   +32 added lines, -32 removed lines patch added patch discarded remove patch
@@ -16,7 +16,7 @@  discard block
 block discarded – undo
16 16
 
17 17
 	public static $current;
18 18
 
19
-	public $hooks = [];
19
+	public $hooks = [ ];
20 20
 
21 21
 	/**
22 22
 	 * {@inheritdoc}
@@ -25,12 +25,12 @@  discard block
 block discarded – undo
25 25
 	{
26 26
 		parent::init();
27 27
 
28
-		add_action( 'wp_ajax_pollux/archives/featured/html', [$this, 'getFeaturedImageHtml'] );
29
-		add_action( 'pollux/archives/init',                  [$this, 'registerFeaturedImageMetaBox'] );
30
-		add_action( 'pollux/archives/editor',                [$this, 'renderEditor'], 10, 2 );
31
-		add_action( 'wp_ajax_pollux/archives/featured',      [$this, 'setFeaturedImage'] );
32
-		add_filter( 'pollux/archives/metabox/submit',        [$this, 'filterSubmitMetaBox'] );
33
-		add_filter( 'pollux/archives/show/instructions',     '__return_true' );
28
+		add_action( 'wp_ajax_pollux/archives/featured/html', [ $this, 'getFeaturedImageHtml' ] );
29
+		add_action( 'pollux/archives/init', [ $this, 'registerFeaturedImageMetaBox' ] );
30
+		add_action( 'pollux/archives/editor', [ $this, 'renderEditor' ], 10, 2 );
31
+		add_action( 'wp_ajax_pollux/archives/featured', [ $this, 'setFeaturedImage' ] );
32
+		add_filter( 'pollux/archives/metabox/submit', [ $this, 'filterSubmitMetaBox' ] );
33
+		add_filter( 'pollux/archives/show/instructions', '__return_true' );
34 34
 	}
35 35
 
36 36
 	public function canProceed()
@@ -46,8 +46,8 @@  discard block
 block discarded – undo
46 46
 	public function filterBeforeInstructions()
47 47
 	{
48 48
 		return sprintf( '<pre class="my-sites nav-tab-active misc-pub-section">%s</pre>',
49
-			array_reduce( ['title', 'content', 'featured'], function( $instructions, $id ) {
50
-				return $instructions . $this->filterInstruction( null, ['slug' => $id], ['slug' => $this->getPostType()] ) . PHP_EOL;
49
+			array_reduce( [ 'title', 'content', 'featured' ], function( $instructions, $id ) {
50
+				return $instructions . $this->filterInstruction( null, [ 'slug' => $id ], [ 'slug' => $this->getPostType() ] ) . PHP_EOL;
51 51
 			})
52 52
 		);
53 53
 	}
@@ -59,7 +59,7 @@  discard block
 block discarded – undo
59 59
 	 */
60 60
 	public function filterInstruction( $instruction, array $field, array $metabox )
61 61
 	{
62
-		return sprintf( "ArchiveMeta::%s('%s');", $metabox['slug'], $field['slug'] );
62
+		return sprintf( "ArchiveMeta::%s('%s');", $metabox[ 'slug' ], $field[ 'slug' ] );
63 63
 	}
64 64
 
65 65
 	/**
@@ -68,7 +68,7 @@  discard block
 block discarded – undo
68 68
 	 */
69 69
 	public function filterSubmitMetaBox( array $args )
70 70
 	{
71
-		$args[1] = __( 'Save Archive', 'pollux' );
71
+		$args[ 1 ] = __( 'Save Archive', 'pollux' );
72 72
 		return $args;
73 73
 	}
74 74
 
@@ -79,10 +79,10 @@  discard block
 block discarded – undo
79 79
 	 */
80 80
 	public function getFeaturedImageHtml()
81 81
 	{
82
-		check_ajax_referer( sprintf( '%s-options', static::id() ));
82
+		check_ajax_referer( sprintf( '%s-options', static::id() ) );
83 83
 		static::$current = filter_input( INPUT_POST, 'post_type' );
84 84
 		ob_start();
85
-		$this->renderFeaturedImageMetaBox( intval( filter_input( INPUT_POST, 'thumbnail_id' )));
85
+		$this->renderFeaturedImageMetaBox( intval( filter_input( INPUT_POST, 'thumbnail_id' ) ) );
86 86
 		wp_send_json_success( ob_get_clean() );
87 87
 	}
88 88
 
@@ -104,7 +104,7 @@  discard block
 block discarded – undo
104 104
 	public function register()
105 105
 	{
106 106
 		$screenId = ( new Helper )->getCurrentScreen()->id;
107
-		if( in_array( $screenId, $this->hooks )) {
107
+		if( in_array( $screenId, $this->hooks ) ) {
108 108
 			$this->hook = $screenId;
109 109
 		}
110 110
 		parent::register();
@@ -116,8 +116,8 @@  discard block
 block discarded – undo
116 116
 	 */
117 117
 	public function registerFeaturedImageMetaBox()
118 118
 	{
119
-		if( !current_user_can( 'upload_files' ))return;
120
-		add_meta_box( 'postimagediv', __( 'Featured Image', 'pollux' ), [$this, 'renderFeaturedImageMetaBox'], null, 'side', 'low' );
119
+		if( !current_user_can( 'upload_files' ) )return;
120
+		add_meta_box( 'postimagediv', __( 'Featured Image', 'pollux' ), [ $this, 'renderFeaturedImageMetaBox' ], null, 'side', 'low' );
121 121
 	}
122 122
 
123 123
 	/**
@@ -127,15 +127,15 @@  discard block
 block discarded – undo
127 127
 	public function registerPage()
128 128
 	{
129 129
 		foreach( $this->getPostTypesWithArchive() as $type => $page ) {
130
-			$labels = get_post_type_labels( get_post_type_object( $type ));
131
-			$this->hooks[$type] = call_user_func_array( 'add_submenu_page', $this->filter( 'page', [
130
+			$labels = get_post_type_labels( get_post_type_object( $type ) );
131
+			$this->hooks[ $type ] = call_user_func_array( 'add_submenu_page', $this->filter( 'page', [
132 132
 				$page,
133 133
 				sprintf( _x( '%s Archive', 'post archive', 'pollux' ), $labels->singular_name ),
134 134
 				sprintf( _x( '%s Archive', 'post archive', 'pollux' ), $labels->singular_name ),
135 135
 				static::CAPABILITY,
136 136
 				sprintf( '%s_archive', $type ),
137
-				[$this, 'renderPage'],
138
-			]));
137
+				[ $this, 'renderPage' ],
138
+			] ) );
139 139
 		}
140 140
 	}
141 141
 
@@ -156,7 +156,7 @@  discard block
 block discarded – undo
156 156
 				'resize' => false,
157 157
 				'wp_autoresize_on' => true,
158 158
 			],
159
-		]);
159
+		] );
160 160
 	}
161 161
 
162 162
 	/**
@@ -165,12 +165,12 @@  discard block
 block discarded – undo
165 165
 	 */
166 166
 	public function renderFeaturedImageMetaBox( $imageId = null )
167 167
 	{
168
-		if( !is_numeric( $imageId )) {
168
+		if( !is_numeric( $imageId ) ) {
169 169
 			$imageId = ArchiveMeta::get( 'featured', -1, $this->getPostType() );
170 170
 		}
171
-		$imageSize = isset( wp_get_additional_image_sizes()['post-thumbnail'] )
171
+		$imageSize = isset( wp_get_additional_image_sizes()[ 'post-thumbnail' ] )
172 172
 			? 'post-thumbnail'
173
-			: [266, 266];
173
+			: [ 266, 266 ];
174 174
 		$thumbnail = get_post( $imageId )
175 175
 			? wp_get_attachment_image( $imageId, $imageSize )
176 176
 			: __( 'Set Featured Image', 'pollux' );
@@ -182,7 +182,7 @@  discard block
 block discarded – undo
182 182
 			'post_type' => $this->getPostType(),
183 183
 			'remove_image' => __( 'Remove featured image', 'pollux' ),
184 184
 			'thumbnail' => $thumbnail,
185
-		]);
185
+		] );
186 186
 	}
187 187
 
188 188
 	/**
@@ -192,8 +192,8 @@  discard block
 block discarded – undo
192 192
 	public function renderPage()
193 193
 	{
194 194
 		$type = $this->getPostType();
195
-		if( empty( $type ))return;
196
-		$labels = get_post_type_labels( get_post_type_object( $type ));
195
+		if( empty( $type ) )return;
196
+		$labels = get_post_type_labels( get_post_type_object( $type ) );
197 197
 		$this->app->render( 'archive/index', [
198 198
 			'columns' => get_current_screen()->get_columns(),
199 199
 			'content' => ArchiveMeta::get( 'content', '', $type ),
@@ -201,7 +201,7 @@  discard block
 block discarded – undo
201 201
 			'id' => static::id(),
202 202
 			'post_type' => $type,
203 203
 			'title' => ArchiveMeta::get( 'title', '', $type ),
204
-		]);
204
+		] );
205 205
 	}
206 206
 
207 207
 	/**
@@ -209,7 +209,7 @@  discard block
 block discarded – undo
209 209
 	 */
210 210
 	protected function getDefaults()
211 211
 	{
212
-		return [];
212
+		return [ ];
213 213
 	}
214 214
 
215 215
 	/**
@@ -218,7 +218,7 @@  discard block
 block discarded – undo
218 218
 	protected function getPostType()
219 219
 	{
220 220
 		$type = array_search( $this->hook, $this->hooks );
221
-		if( !empty( $type ) && is_string( $type )) {
221
+		if( !empty( $type ) && is_string( $type ) ) {
222 222
 			static::$current = $type;
223 223
 		}
224 224
 		return static::$current;
@@ -231,8 +231,8 @@  discard block
 block discarded – undo
231 231
 	{
232 232
 		$types = array_map( function( $value ) {
233 233
 			return sprintf( 'edit.php?post_type=%s', $value );
234
-		}, get_post_types( ['has_archive' => 1] ));
235
-		return array_merge( $types, ['post' => 'edit.php'] );
234
+		}, get_post_types( [ 'has_archive' => 1 ] ) );
235
+		return array_merge( $types, [ 'post' => 'edit.php' ] );
236 236
 	}
237 237
 
238 238
 	/**
Please login to merge, or discard this patch.
src/Notice.php 1 patch
Spacing   +37 added lines, -37 removed lines patch added patch discarded remove patch
@@ -29,10 +29,10 @@  discard block
 block discarded – undo
29 29
 	{
30 30
 		$method = strtolower( $name );
31 31
 		$status = substr( $method, 3 );
32
-		if( 'add' == substr( $method, 0, 3 ) && in_array( $status, ['error', 'info', 'success', 'warning'] )) {
33
-			return call_user_func_array( [$this, 'addNotice'], array_merge( [$status], $args ));
32
+		if( 'add' == substr( $method, 0, 3 ) && in_array( $status, [ 'error', 'info', 'success', 'warning' ] ) ) {
33
+			return call_user_func_array( [ $this, 'addNotice' ], array_merge( [ $status ], $args ) );
34 34
 		}
35
-		throw new BadMethodCallException( sprintf( 'Not a valid method: %s', $name ));
35
+		throw new BadMethodCallException( sprintf( 'Not a valid method: %s', $name ) );
36 36
 	}
37 37
 
38 38
 	public function __get( $property )
@@ -40,7 +40,7 @@  discard block
 block discarded – undo
40 40
 		if( $property == 'all' ) {
41 41
 			return $this->app->notices;
42 42
 		}
43
-		throw new Exception( sprintf( 'Not a valid property: %s', $property ));
43
+		throw new Exception( sprintf( 'Not a valid property: %s', $property ) );
44 44
 	}
45 45
 
46 46
 	/**
@@ -48,28 +48,28 @@  discard block
 block discarded – undo
48 48
 	 */
49 49
 	public function activateButton( array $plugin )
50 50
 	{
51
-		$actionUrl = self_admin_url( sprintf( 'options-general.php?page=%s&action=activate&plugin=%s', $this->app->id, $plugin['plugin'] ));
52
-		return $this->button( sprintf( '%s %s', __( 'Activate', 'pollux' ), $plugin['name'] ), [
53
-			'data-name' => $plugin['name'],
54
-			'data-plugin' => $plugin['plugin'],
55
-			'data-slug' => $plugin['slug'],
56
-			'href' => wp_nonce_url( $actionUrl, sprintf( 'activate-plugin_%s', $plugin['plugin'] )),
57
-		]);
51
+		$actionUrl = self_admin_url( sprintf( 'options-general.php?page=%s&action=activate&plugin=%s', $this->app->id, $plugin[ 'plugin' ] ) );
52
+		return $this->button( sprintf( '%s %s', __( 'Activate', 'pollux' ), $plugin[ 'name' ] ), [
53
+			'data-name' => $plugin[ 'name' ],
54
+			'data-plugin' => $plugin[ 'plugin' ],
55
+			'data-slug' => $plugin[ 'slug' ],
56
+			'href' => wp_nonce_url( $actionUrl, sprintf( 'activate-plugin_%s', $plugin[ 'plugin' ] ) ),
57
+		] );
58 58
 	}
59 59
 
60 60
 	/**
61 61
 	 * @param string $title
62 62
 	 * @return string
63 63
 	 */
64
-	public function button( $title, array $atts = [] )
64
+	public function button( $title, array $atts = [ ] )
65 65
 	{
66 66
 		$atts = wp_parse_args( $atts, [
67 67
 			'class' => '',
68 68
 			'href' => '',
69
-		]);
70
-		$atts['class'] = trim( $atts['class'] . ' button button-small' );
69
+		] );
70
+		$atts[ 'class' ] = trim( $atts[ 'class' ] . ' button button-small' );
71 71
 		$attributes = array_reduce( array_keys( $atts ), function( $carry, $key ) use( $atts ) {
72
-			return $carry . sprintf( ' %s="%s"', $key, $atts[$key] );
72
+			return $carry . sprintf( ' %s="%s"', $key, $atts[ $key ] );
73 73
 		});
74 74
 		return sprintf( '<a%s>%s</a>', $attributes, $title );
75 75
 	}
@@ -82,7 +82,7 @@  discard block
 block discarded – undo
82 82
 		if( $unset ) {
83 83
 			$index = array_search( $notice, $this->app->notices );
84 84
 			if( $index !== false ) {
85
-				unset( $this->app->notices[$index] );
85
+				unset( $this->app->notices[ $index ] );
86 86
 			}
87 87
 		}
88 88
 		return $this->buildNotice( $notice );
@@ -93,13 +93,13 @@  discard block
 block discarded – undo
93 93
 	 */
94 94
 	public function installButton( array $plugin )
95 95
 	{
96
-		$actionUrl = self_admin_url( sprintf( 'update.php?action=install-plugin&plugin=%s', $plugin['slug'] ));
97
-		return $this->button( sprintf( '%s %s', __( 'Install', 'pollux' ), $plugin['name'] ), [
98
-			'data-name' => $plugin['name'],
99
-			'data-plugin' => $plugin['plugin'],
100
-			'data-slug' => $plugin['slug'],
101
-			'href' => wp_nonce_url( $actionUrl, sprintf( 'install-plugin_%s', $plugin['slug'] )),
102
-		]);
96
+		$actionUrl = self_admin_url( sprintf( 'update.php?action=install-plugin&plugin=%s', $plugin[ 'slug' ] ) );
97
+		return $this->button( sprintf( '%s %s', __( 'Install', 'pollux' ), $plugin[ 'name' ] ), [
98
+			'data-name' => $plugin[ 'name' ],
99
+			'data-plugin' => $plugin[ 'plugin' ],
100
+			'data-slug' => $plugin[ 'slug' ],
101
+			'href' => wp_nonce_url( $actionUrl, sprintf( 'install-plugin_%s', $plugin[ 'slug' ] ) ),
102
+		] );
103 103
 	}
104 104
 
105 105
 	/**
@@ -107,13 +107,13 @@  discard block
 block discarded – undo
107 107
 	 */
108 108
 	public function updateButton( array $plugin )
109 109
 	{
110
-		$actionUrl = self_admin_url( sprintf( 'update.php?action=upgrade-plugin&plugin=%s', $plugin['plugin'] ));
111
-		return $this->button( sprintf( '%s %s', __( 'Update', 'pollux' ), $plugin['name'] ), [
112
-			'data-name' => $plugin['name'],
113
-			'data-plugin' => $plugin['plugin'],
114
-			'data-slug' => $plugin['slug'],
115
-			'href' => wp_nonce_url( $actionUrl, sprintf( 'upgrade-plugin_%s', $plugin['plugin'] )),
116
-		]);
110
+		$actionUrl = self_admin_url( sprintf( 'update.php?action=upgrade-plugin&plugin=%s', $plugin[ 'plugin' ] ) );
111
+		return $this->button( sprintf( '%s %s', __( 'Update', 'pollux' ), $plugin[ 'name' ] ), [
112
+			'data-name' => $plugin[ 'name' ],
113
+			'data-plugin' => $plugin[ 'plugin' ],
114
+			'data-slug' => $plugin[ 'slug' ],
115
+			'href' => wp_nonce_url( $actionUrl, sprintf( 'upgrade-plugin_%s', $plugin[ 'plugin' ] ) ),
116
+		] );
117 117
 	}
118 118
 
119 119
 	/**
@@ -133,9 +133,9 @@  discard block
 block discarded – undo
133 133
 	 */
134 134
 	protected function addNotice( $type, $messages, $dismissible = true )
135 135
 	{
136
-		$this->app->notices[] = [
136
+		$this->app->notices[ ] = [
137 137
 			'dismissible' => $dismissible,
138
-			'message' => $this->buildMessage( array_filter( (array) $messages )),
138
+			'message' => $this->buildMessage( array_filter( (array) $messages ) ),
139 139
 			'type' => $type,
140 140
 		];
141 141
 		$this->app->notices = array_unique( $this->app->notices, SORT_REGULAR );
@@ -147,10 +147,10 @@  discard block
 block discarded – undo
147 147
 	protected function buildMessage( array $messages )
148 148
 	{
149 149
 		foreach( $messages as $key => &$message ) {
150
-			if( !is_wp_error( $message ))continue;
150
+			if( !is_wp_error( $message ) )continue;
151 151
 			$message = $message->get_error_message();
152 152
 		}
153
-		return wpautop( implode( PHP_EOL . PHP_EOL, $messages ));
153
+		return wpautop( implode( PHP_EOL . PHP_EOL, $messages ) );
154 154
 	}
155 155
 
156 156
 	/**
@@ -158,10 +158,10 @@  discard block
 block discarded – undo
158 158
 	 */
159 159
 	protected function buildNotice( array $notice )
160 160
 	{
161
-		$class = sprintf( 'notice notice-%s', $notice['type'] );
162
-		if( $notice['dismissible'] ) {
161
+		$class = sprintf( 'notice notice-%s', $notice[ 'type' ] );
162
+		if( $notice[ 'dismissible' ] ) {
163 163
 			$class .= ' is-dismissible';
164 164
 		}
165
-		return sprintf( '<div class="pollux-notice %s">%s</div>', $class, $notice['message'] );
165
+		return sprintf( '<div class="pollux-notice %s">%s</div>', $class, $notice[ 'message' ] );
166 166
 	}
167 167
 }
Please login to merge, or discard this patch.
src/GateKeeper.php 1 patch
Spacing   +54 added lines, -54 removed lines patch added patch discarded remove patch
@@ -17,7 +17,7 @@  discard block
 block discarded – undo
17 17
 	const MIN_PHP_VERSION = '5.6.0';
18 18
 	const MIN_WORDPRESS_VERSION = '4.7';
19 19
 
20
-	public $errors = [];
20
+	public $errors = [ ];
21 21
 
22 22
 	/**
23 23
 	 * @var Application
@@ -39,11 +39,11 @@  discard block
 block discarded – undo
39 39
 		$this->plugin = $plugin;
40 40
 
41 41
 		if( $this->canActivate() ) {
42
-			add_action( 'admin_init', array( $this, 'init' ));
42
+			add_action( 'admin_init', array( $this, 'init' ) );
43 43
 		}
44 44
 		else {
45
-			add_action( 'activated_plugin', array( $this, 'deactivate' ));
46
-			add_action( 'admin_notices',    array( $this, 'deactivate' ));
45
+			add_action( 'activated_plugin', array( $this, 'deactivate' ) );
46
+			add_action( 'admin_notices', array( $this, 'deactivate' ) );
47 47
 		}
48 48
 	}
49 49
 
@@ -52,10 +52,10 @@  discard block
 block discarded – undo
52 52
 		$this->app = pollux_app();
53 53
 		$this->notice = pollux_app()->make( 'Notice' );
54 54
 
55
-		add_action( 'current_screen',                         array( $this, 'activatePlugin' ));
56
-		add_action( 'wp_ajax_pollux/dependency/activate_url', array( $this, 'ajaxActivatePluginLink' ));
57
-		add_action( 'admin_notices',                          array( $this, 'printNotices' ));
58
-		add_action( 'current_screen',                         array( $this, 'setDependencyNotice' ));
55
+		add_action( 'current_screen', array( $this, 'activatePlugin' ) );
56
+		add_action( 'wp_ajax_pollux/dependency/activate_url', array( $this, 'ajaxActivatePluginLink' ) );
57
+		add_action( 'admin_notices', array( $this, 'printNotices' ) );
58
+		add_action( 'current_screen', array( $this, 'setDependencyNotice' ) );
59 59
 	}
60 60
 
61 61
 	/**
@@ -69,7 +69,7 @@  discard block
 block discarded – undo
69 69
 		$plugin = filter_input( INPUT_GET, 'plugin' );
70 70
 		check_admin_referer( 'activate-plugin_' . $plugin );
71 71
 		$result = activate_plugin( $plugin, null, is_network_admin(), true );
72
-		if( is_wp_error( $result )) {
72
+		if( is_wp_error( $result ) ) {
73 73
 			wp_die( $result->get_error_message() );
74 74
 		}
75 75
 		wp_safe_redirect( wp_get_referer() );
@@ -83,19 +83,19 @@  discard block
 block discarded – undo
83 83
 	{
84 84
 		check_ajax_referer( 'updates' );
85 85
 		$plugin = filter_input( INPUT_POST, 'plugin' );
86
-		if( !$this->isPluginDependency( $plugin )) {
86
+		if( !$this->isPluginDependency( $plugin ) ) {
87 87
 			wp_send_json_error();
88 88
 		}
89
-		$activateUrl = add_query_arg([
90
-			'_wpnonce' => wp_create_nonce( sprintf( 'activate-plugin_%s', $plugin )),
89
+		$activateUrl = add_query_arg( [
90
+			'_wpnonce' => wp_create_nonce( sprintf( 'activate-plugin_%s', $plugin ) ),
91 91
 			'action' => 'activate',
92 92
 			'page' => $this->app->id,
93 93
 			'plugin' => $plugin,
94
-		], self_admin_url( 'options-general.php' ));
95
-		wp_send_json_success([
94
+		], self_admin_url( 'options-general.php' ) );
95
+		wp_send_json_success( [
96 96
 			'activate_url' => $activateUrl,
97 97
 			filter_input( INPUT_POST, 'type' ) => $plugin,
98
-		]);
98
+		] );
99 99
 	}
100 100
 
101 101
 	/**
@@ -128,7 +128,7 @@  discard block
 block discarded – undo
128 128
 	 */
129 129
 	public function hasDependency( $plugin )
130 130
 	{
131
-		if( !$this->isPluginDependency( $plugin )) {
131
+		if( !$this->isPluginDependency( $plugin ) ) {
132 132
 			return true;
133 133
 		}
134 134
 		return $this->isPluginInstalled( $plugin ) && $this->isPluginValid( $plugin );
@@ -140,7 +140,7 @@  discard block
 block discarded – undo
140 140
 	public function hasPendingDependencies()
141 141
 	{
142 142
 		foreach( static::DEPENDENCIES as $plugin => $data ) {
143
-			if( !$this->isPluginDependency( $plugin ))continue;
143
+			if( !$this->isPluginDependency( $plugin ) )continue;
144 144
 			$this->isPluginActive( $plugin );
145 145
 			$this->isPluginVersionValid( $plugin );
146 146
 		}
@@ -205,17 +205,17 @@  discard block
 block discarded – undo
205 205
 	 */
206 206
 	public function isPluginVersionValid( $plugin )
207 207
 	{
208
-		if( !$this->isPluginDependency( $plugin )) {
208
+		if( !$this->isPluginDependency( $plugin ) ) {
209 209
 			return true;
210 210
 		}
211
-		if( !$this->isPluginInstalled( $plugin )) {
211
+		if( !$this->isPluginInstalled( $plugin ) ) {
212 212
 			return false;
213 213
 		}
214 214
 		return $this->catchError( $plugin, 'wrong_version', version_compare(
215 215
 			$this->getPluginRequirements( $plugin, 'version' ),
216
-			$this->getAllPlugins()[$plugin]['Version'],
216
+			$this->getAllPlugins()[ $plugin ][ 'Version' ],
217 217
 			'<='
218
-		));
218
+		) );
219 219
 	}
220 220
 
221 221
 	/**
@@ -241,7 +241,7 @@  discard block
 block discarded – undo
241 241
 			__( 'Pollux requires the latest version of the following plugins', 'pollux' ),
242 242
 			$this->getDependencyLinks()
243 243
 		);
244
-		$this->notice->addWarning( [$message, $this->getDependencyActions()] );
244
+		$this->notice->addWarning( [ $message, $this->getDependencyActions() ] );
245 245
 	}
246 246
 
247 247
 	/**
@@ -254,7 +254,7 @@  discard block
 block discarded – undo
254 254
 		$this->printDeactivationNotice( sprintf( '%s %s',
255 255
 			$message1,
256 256
 			$message2
257
-		));
257
+		) );
258 258
 	}
259 259
 
260 260
 	/**
@@ -263,7 +263,7 @@  discard block
 block discarded – undo
263 263
 	protected function addInvalidWPVersionNotice()
264 264
 	{
265 265
 		$message = sprintf( __( 'Pollux requires WordPress %s or greater in order to work properly.', 'pollux' ), self::MIN_WORDPRESS_VERSION );
266
-		if( current_user_can( 'update_core' )) {
266
+		if( current_user_can( 'update_core' ) ) {
267 267
 			$message .= PHP_EOL . PHP_EOL . sprintf( '<a href="%s" class="button button-small">%s</a>',
268 268
 				self_admin_url( 'update-core.php' ),
269 269
 				__( 'Update WordPress', 'pollux' )
@@ -281,12 +281,12 @@  discard block
 block discarded – undo
281 281
 	protected function catchError( $plugin, $error, $isValid )
282 282
 	{
283 283
 		if( !$isValid ) {
284
-			if( !isset( $this->errors[$plugin] )) {
285
-				$this->errors[$plugin] = [];
284
+			if( !isset( $this->errors[ $plugin ] ) ) {
285
+				$this->errors[ $plugin ] = [ ];
286 286
 			}
287
-			$this->errors[$plugin] = array_keys( array_flip(
288
-				array_merge( $this->errors[$plugin], [$error] )
289
-			));
287
+			$this->errors[ $plugin ] = array_keys( array_flip(
288
+				array_merge( $this->errors[ $plugin ], [ $error ] )
289
+			) );
290 290
 		}
291 291
 		return $isValid;
292 292
 	}
@@ -307,14 +307,14 @@  discard block
 block discarded – undo
307 307
 	{
308 308
 		$actions = '';
309 309
 		foreach( $this->errors as $plugin => $errors ) {
310
-			if( in_array( 'not_found', $errors ) && current_user_can( 'install_plugins' )) {
311
-				$actions .= $this->notice->installButton( $this->getPluginRequirements( $plugin ));
310
+			if( in_array( 'not_found', $errors ) && current_user_can( 'install_plugins' ) ) {
311
+				$actions .= $this->notice->installButton( $this->getPluginRequirements( $plugin ) );
312 312
 			}
313
-			else if( in_array( 'wrong_version', $errors ) && current_user_can( 'update_plugins' )) {
314
-				$actions .= $this->notice->updateButton( $this->getPluginInformation( $plugin ));
313
+			else if( in_array( 'wrong_version', $errors ) && current_user_can( 'update_plugins' ) ) {
314
+				$actions .= $this->notice->updateButton( $this->getPluginInformation( $plugin ) );
315 315
 			}
316
-			else if( in_array( 'inactive', $errors ) && current_user_can( 'activate_plugins' )) {
317
-				$actions .= $this->notice->activateButton( $this->getPluginInformation( $plugin ));
316
+			else if( in_array( 'inactive', $errors ) && current_user_can( 'activate_plugins' ) ) {
317
+				$actions .= $this->notice->activateButton( $this->getPluginInformation( $plugin ) );
318 318
 			}
319 319
 		}
320 320
 		return $actions;
@@ -336,10 +336,10 @@  discard block
 block discarded – undo
336 336
 	protected function getMustUsePlugins()
337 337
 	{
338 338
 		$plugins = get_mu_plugins();
339
-		if( in_array( 'Bedrock Autoloader', array_column( $plugins, 'Name' ))) {
339
+		if( in_array( 'Bedrock Autoloader', array_column( $plugins, 'Name' ) ) ) {
340 340
 			$autoloadedPlugins = get_site_option( 'bedrock_autoloader' );
341
-			if( !empty( $autoloadedPlugins['plugins'] )) {
342
-				return array_merge( $plugins, $autoloadedPlugins['plugins'] );
341
+			if( !empty( $autoloadedPlugins[ 'plugins' ] ) ) {
342
+				return array_merge( $plugins, $autoloadedPlugins[ 'plugins' ] );
343 343
 			}
344 344
 		}
345 345
 		return $plugins;
@@ -350,8 +350,8 @@  discard block
 block discarded – undo
350 350
 	 */
351 351
 	protected function getPlugin( $plugin )
352 352
 	{
353
-		if( $this->isPluginInstalled( $plugin )) {
354
-			return $this->getAllPlugins()[$plugin];
353
+		if( $this->isPluginInstalled( $plugin ) ) {
354
+			return $this->getAllPlugins()[ $plugin ];
355 355
 		}
356 356
 		return false;
357 357
 	}
@@ -361,18 +361,18 @@  discard block
 block discarded – undo
361 361
 	 */
362 362
 	protected function getPluginData( $plugin, $data, $key = null )
363 363
 	{
364
-		if( !is_array( $data )) {
365
-			throw new Exception( sprintf( 'Plugin information not found for: %s', $plugin ));
364
+		if( !is_array( $data ) ) {
365
+			throw new Exception( sprintf( 'Plugin information not found for: %s', $plugin ) );
366 366
 		}
367
-		$data['plugin'] = $plugin;
368
-		$data['slug'] = $this->getPluginSlug( $plugin );
367
+		$data[ 'plugin' ] = $plugin;
368
+		$data[ 'slug' ] = $this->getPluginSlug( $plugin );
369 369
 		$data = array_change_key_case( $data );
370
-		if( is_null( $key )) {
370
+		if( is_null( $key ) ) {
371 371
 			return $data;
372 372
 		}
373 373
 		$key = strtolower( $key );
374
-		return isset( $data[$key] )
375
-			? $data[$key]
374
+		return isset( $data[ $key ] )
375
+			? $data[ $key ]
376 376
 			: '';
377 377
 	}
378 378
 
@@ -396,9 +396,9 @@  discard block
 block discarded – undo
396 396
 			$data = $this->getPluginRequirements( $plugin );
397 397
 		}
398 398
 		return sprintf( '<span class="plugin-%s"><a href="%s">%s</a></span>',
399
-			$data['slug'],
400
-			$data['pluginuri'],
401
-			$data['name']
399
+			$data[ 'slug' ],
400
+			$data[ 'pluginuri' ],
401
+			$data[ 'name' ]
402 402
 		);
403 403
 	}
404 404
 
@@ -407,9 +407,9 @@  discard block
 block discarded – undo
407 407
 	 */
408 408
 	protected function getPluginRequirements( $plugin, $key = null )
409 409
 	{
410
-		$keys = ['Name', 'Version', 'PluginURI'];
410
+		$keys = [ 'Name', 'Version', 'PluginURI' ];
411 411
 		$requirements = $this->isPluginDependency( $plugin )
412
-			? array_pad( explode( '|', static::DEPENDENCIES[$plugin] ), 3, '' )
412
+			? array_pad( explode( '|', static::DEPENDENCIES[ $plugin ] ), 3, '' )
413 413
 			: array_fill( 0, 3, '' );
414 414
 		return $this->getPluginData( $plugin, array_combine( $keys, $requirements ), $key );
415 415
 	}
@@ -419,7 +419,7 @@  discard block
 block discarded – undo
419 419
 	 */
420 420
 	protected function getPluginSlug( $plugin )
421 421
 	{
422
-		return substr( $plugin, 0, strrpos( $plugin, '/' ));
422
+		return substr( $plugin, 0, strrpos( $plugin, '/' ) );
423 423
 	}
424 424
 
425 425
 	/**
@@ -442,7 +442,7 @@  discard block
 block discarded – undo
442 442
 			filter_input( INPUT_GET, 'plugin_status' ),
443 443
 			filter_input( INPUT_GET, 'paged' ),
444 444
 			filter_input( INPUT_GET, 's' )
445
-		)));
445
+		) ) );
446 446
 		exit;
447 447
 	}
448 448
 }
Please login to merge, or discard this patch.
src/Config/Config.php 1 patch
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -39,9 +39,9 @@  discard block
 block discarded – undo
39 39
 	{
40 40
 		if( $this->app->config->disable_config )return;
41 41
 
42
-		add_action( 'admin_menu',     [$this, 'registerPage'] );
43
-		add_action( 'admin_menu',     [$this, 'registerSetting'] );
44
-		add_action( 'current_screen', [$this, 'resetPage'] );
42
+		add_action( 'admin_menu', [ $this, 'registerPage' ] );
43
+		add_action( 'admin_menu', [ $this, 'registerSetting' ] );
44
+		add_action( 'current_screen', [ $this, 'resetPage' ] );
45 45
 	}
46 46
 
47 47
 	/**
@@ -65,7 +65,7 @@  discard block
 block discarded – undo
65 65
 			__( 'Pollux', 'pollux' ),
66 66
 			'manage_options',
67 67
 			$this->app->id,
68
-			[$this, 'renderPage']
68
+			[ $this, 'renderPage' ]
69 69
 		);
70 70
 	}
71 71
 
@@ -75,7 +75,7 @@  discard block
 block discarded – undo
75 75
 	 */
76 76
 	public function registerSetting()
77 77
 	{
78
-		register_setting( static::id(), static::id(), [$this, 'filterSavedSettings'] );
78
+		register_setting( static::id(), static::id(), [ $this, 'filterSavedSettings' ] );
79 79
 	}
80 80
 
81 81
 	/**
@@ -94,9 +94,9 @@  discard block
 block discarded – undo
94 94
 			'config' => $this->app->make( ConfigManager::class ),
95 95
 			'heading' => __( 'Pollux Settings', 'pollux' ),
96 96
 			'id' => static::id(),
97
-			'reset_url' => esc_url( add_query_arg( $query, admin_url( 'options-general.php' ))),
97
+			'reset_url' => esc_url( add_query_arg( $query, admin_url( 'options-general.php' ) ) ),
98 98
 			'has_meta_box' => $this->app->gatekeeper->hasDependency( 'meta-box/meta-box.php' ),
99
-		]);
99
+		] );
100 100
 	}
101 101
 
102 102
 	/**
@@ -108,15 +108,15 @@  discard block
 block discarded – undo
108 108
 		if( filter_input( INPUT_GET, 'page' ) !== $this->app->id
109 109
 			|| filter_input( INPUT_GET, 'action' ) !== 'reset'
110 110
 		)return;
111
-		if( wp_verify_nonce( filter_input( INPUT_GET, '_wpnonce' ), static::id() )) {
111
+		if( wp_verify_nonce( filter_input( INPUT_GET, '_wpnonce' ), static::id() ) ) {
112 112
 			delete_option( static::id() );
113 113
 			add_settings_error( static::id(), 'reset', __( 'Reset successful.', 'pollux' ), 'updated' );
114 114
 		}
115 115
 		else {
116
-			add_settings_error( static::id(), 'failed', __( 'Failed to reset. Please try again.', 'pollux' ));
116
+			add_settings_error( static::id(), 'failed', __( 'Failed to reset. Please try again.', 'pollux' ) );
117 117
 		}
118 118
 		set_transient( 'settings_errors', get_settings_errors(), 30 );
119
-		wp_safe_redirect( add_query_arg( 'settings-updated', 'true',  wp_get_referer() ));
119
+		wp_safe_redirect( add_query_arg( 'settings-updated', 'true', wp_get_referer() ) );
120 120
 		exit;
121 121
 	}
122 122
 }
Please login to merge, or discard this patch.