Passed
Push — master ( 9d21ea...9827a3 )
by Paul
02:49
created
helpers.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -5,7 +5,7 @@
 block discarded – undo
5 5
  *
6 6
  * @return mixed
7 7
  */
8
-function pollux_get_post_meta( $metaKey, array $args = [] ) {
8
+function pollux_get_post_meta( $metaKey, array $args = [ ] ) {
9 9
 	return \GeminiLabs\Pollux\Facades\PostMeta::get( $metaKey, $args );
10 10
 }
11 11
 
Please login to merge, or discard this patch.
Braces   +4 added lines, -2 removed lines patch added patch discarded remove patch
@@ -5,7 +5,8 @@  discard block
 block discarded – undo
5 5
  *
6 6
  * @return mixed
7 7
  */
8
-function pollux_get_post_meta( $metaKey, array $args = [] ) {
8
+function pollux_get_post_meta( $metaKey, array $args = [] )
9
+{
9 10
 	return \GeminiLabs\Pollux\Facades\PostMeta::get( $metaKey, $args );
10 11
 }
11 12
 
@@ -14,6 +15,7 @@  discard block
 block discarded – undo
14 15
  *
15 16
  * @return mixed
16 17
  */
17
-function pollux_get_option( $group, $key = false, $fallback = '' ) {
18
+function pollux_get_option( $group, $key = false, $fallback = '' )
19
+{
18 20
 	return \GeminiLabs\Pollux\Facades\SiteMeta::get( $group, $key, $fallback );
19 21
 }
Please login to merge, or discard this patch.
pollux.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -27,8 +27,8 @@
 block discarded – undo
27 27
 
28 28
 $app = Application::getInstance();
29 29
 
30
-register_activation_hook( __FILE__, array( $app, 'onActivation' ));
31
-register_deactivation_hook( __FILE__, array( $app, 'onDeactivation' ));
30
+register_activation_hook( __FILE__, array( $app, 'onActivation' ) );
31
+register_deactivation_hook( __FILE__, array( $app, 'onDeactivation' ) );
32 32
 
33 33
 if( $app->gatekeeper->proceed() ) {
34 34
 	$app->register( new Provider );
Please login to merge, or discard this patch.
autoload.php 1 patch
Braces   +6 added lines, -2 removed lines patch added patch discarded remove patch
@@ -13,9 +13,13 @@
 block discarded – undo
13 13
 	];
14 14
 	foreach( $namespaces as $prefix => $base_dir ) {
15 15
 		$len = strlen( $prefix );
16
-		if( strncmp( $prefix, $class, $len ) !== 0 )continue;
16
+		if( strncmp( $prefix, $class, $len ) !== 0 ) {
17
+			continue;
18
+		}
17 19
 		$file = $base_dir . str_replace( '\\', '/', substr( $class, $len ) ) . '.php';
18
-		if( !file_exists( $file ) )continue;
20
+		if( !file_exists( $file ) ) {
21
+			continue;
22
+		}
19 23
 		require $file;
20 24
 		break;
21 25
 	}
Please login to merge, or discard this patch.
src/AliasLoader.php 2 patches
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -38,9 +38,9 @@  discard block
 block discarded – undo
38 38
 	 *
39 39
 	 * @return AliasLoader
40 40
 	 */
41
-	public static function getInstance( array $aliases = [] )
41
+	public static function getInstance( array $aliases = [ ] )
42 42
 	{
43
-		if( is_null( static::$instance )) {
43
+		if( is_null( static::$instance ) ) {
44 44
 			return static::$instance = new static( $aliases );
45 45
 		}
46 46
 
@@ -60,8 +60,8 @@  discard block
 block discarded – undo
60 60
 	 */
61 61
 	public function load( $alias )
62 62
 	{
63
-		if( isset( $this->aliases[$alias] )) {
64
-			return class_alias( $this->aliases[$alias], $alias );
63
+		if( isset( $this->aliases[ $alias ] ) ) {
64
+			return class_alias( $this->aliases[ $alias ], $alias );
65 65
 		}
66 66
 	}
67 67
 
@@ -73,7 +73,7 @@  discard block
 block discarded – undo
73 73
 	public function register()
74 74
 	{
75 75
 		if( !$this->registered ) {
76
-			spl_autoload_register( [$this, 'load'], true, true );
76
+			spl_autoload_register( [ $this, 'load' ], true, true );
77 77
 			$this->registered = true;
78 78
 		}
79 79
 	}
Please login to merge, or discard this patch.
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -31,7 +31,8 @@
 block discarded – undo
31 31
 	}
32 32
 
33 33
 	private function __clone()
34
-	{}
34
+	{
35
+}
35 36
 
36 37
 	/**
37 38
 	 * Get or create the singleton alias loader instance.
Please login to merge, or discard this patch.
src/Controller.php 2 patches
Braces   +9 added lines, -3 removed lines patch added patch discarded remove patch
@@ -23,7 +23,9 @@  discard block
 block discarded – undo
23 23
 	 */
24 24
 	public function filterWordPressFooter( $text )
25 25
 	{
26
-		if( $this->app->config['remove_wordpress_footer'] )return;
26
+		if( $this->app->config['remove_wordpress_footer'] ) {
27
+			return;
28
+		}
27 29
 		return $text;
28 30
 	}
29 31
 
@@ -64,7 +66,9 @@  discard block
 block discarded – undo
64 66
 	 */
65 67
 	public function removeDashboardWidgets()
66 68
 	{
67
-		if( !$this->app->config['remove_dashboard_widgets'] )return;
69
+		if( !$this->app->config['remove_dashboard_widgets'] ) {
70
+			return;
71
+		}
68 72
 		$widgets = apply_filters( 'pollux/dashoard/widgets', [
69 73
 			'dashboard_primary',
70 74
 			'dashboard_quick_press',
@@ -80,7 +84,9 @@  discard block
 block discarded – undo
80 84
 	 */
81 85
 	public function removeWordPressMenu()
82 86
 	{
83
-		if( !$this->app->config['remove_wordpress_menu'] )return;
87
+		if( !$this->app->config['remove_wordpress_menu'] ) {
88
+			return;
89
+		}
84 90
 		global $wp_admin_bar;
85 91
 		$wp_admin_bar->remove_menu( 'wp-logo' );
86 92
 	}
Please login to merge, or discard this patch.
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -23,7 +23,7 @@  discard block
 block discarded – undo
23 23
 	 */
24 24
 	public function filterWordPressFooter( $text )
25 25
 	{
26
-		if( $this->app->config['remove_wordpress_footer'] )return;
26
+		if( $this->app->config[ 'remove_wordpress_footer' ] )return;
27 27
 		return $text;
28 28
 	}
29 29
 
@@ -33,24 +33,24 @@  discard block
 block discarded – undo
33 33
 	 */
34 34
 	public function registerAssets()
35 35
 	{
36
-		if( $this->app->screen()->id == sprintf( 'toplevel_page_%s', apply_filters( 'pollux/settings/option', Settings::ID ))) {
36
+		if( $this->app->screen()->id == sprintf( 'toplevel_page_%s', apply_filters( 'pollux/settings/option', Settings::ID ) ) ) {
37 37
 			wp_enqueue_script( 'common' );
38 38
 			wp_enqueue_script( 'wp-lists' );
39 39
 			wp_enqueue_script( 'postbox' );
40 40
 		}
41 41
 		wp_enqueue_style( 'pollux/main.css',
42 42
 			$this->app->path( 'assets/main.css' ),
43
-			apply_filters( 'pollux/enqueue/css/deps', [] ),
43
+			apply_filters( 'pollux/enqueue/css/deps', [ ] ),
44 44
 			$this->app->version
45 45
 		);
46 46
 		wp_enqueue_script( 'pollux/main.js',
47 47
 			$this->app->path( 'assets/main.js' ),
48
-			apply_filters( 'pollux/enqueue/js/deps', [] ),
48
+			apply_filters( 'pollux/enqueue/js/deps', [ ] ),
49 49
 			$this->app->version
50 50
 		);
51 51
 		wp_localize_script( 'pollux/main.js',
52 52
 			apply_filters( 'pollux/enqueue/js/localize/name', $this->app->id ),
53
-			apply_filters( 'pollux/enqueue/js/localize/variables', [] )
53
+			apply_filters( 'pollux/enqueue/js/localize/variables', [ ] )
54 54
 		);
55 55
 	}
56 56
 
@@ -60,11 +60,11 @@  discard block
 block discarded – undo
60 60
 	 */
61 61
 	public function removeDashboardWidgets()
62 62
 	{
63
-		if( !$this->app->config['remove_dashboard_widgets'] )return;
63
+		if( !$this->app->config[ 'remove_dashboard_widgets' ] )return;
64 64
 		$widgets = apply_filters( 'pollux/dashoard/widgets', [
65 65
 			'dashboard_primary',
66 66
 			'dashboard_quick_press',
67
-		]);
67
+		] );
68 68
 		foreach( $widgets as $widget ) {
69 69
 			remove_meta_box( $widget, 'dashboard', 'normal' );
70 70
 		}
@@ -76,7 +76,7 @@  discard block
 block discarded – undo
76 76
 	 */
77 77
 	public function removeWordPressMenu()
78 78
 	{
79
-		if( !$this->app->config['remove_wordpress_menu'] )return;
79
+		if( !$this->app->config[ 'remove_wordpress_menu' ] )return;
80 80
 		global $wp_admin_bar;
81 81
 		$wp_admin_bar->remove_menu( 'wp-logo' );
82 82
 	}
Please login to merge, or discard this patch.
src/Container.php 2 patches
Indentation   +14 added lines, -14 removed lines patch added patch discarded remove patch
@@ -16,18 +16,18 @@  discard block
 block discarded – undo
16 16
 	 */
17 17
 	protected static $instance;
18 18
 
19
-    /**
20
-     * The container's bound services.
21
-     *
22
-     * @var array
23
-     */
19
+	/**
20
+	 * The container's bound services.
21
+	 *
22
+	 * @var array
23
+	 */
24 24
 	protected $services = [];
25 25
 
26
-    /**
27
-     * The container's bucket items
28
-     *
29
-     * @var array
30
-     */
26
+	/**
27
+	 * The container's bucket items
28
+	 *
29
+	 * @var array
30
+	 */
31 31
 	protected $bucket = [];
32 32
 
33 33
 	/**
@@ -123,10 +123,10 @@  discard block
 block discarded – undo
123 123
 	}
124 124
 
125 125
    /**
126
-	 * Register a Provider.
127
-	 *
128
-	 * @return void
129
-	 */
126
+    * Register a Provider.
127
+    *
128
+    * @return void
129
+    */
130 130
 	public function register( $provider )
131 131
 	{
132 132
 		$provider->register( $this );
Please login to merge, or discard this patch.
Spacing   +17 added lines, -17 removed lines patch added patch discarded remove patch
@@ -21,14 +21,14 @@  discard block
 block discarded – undo
21 21
      *
22 22
      * @var array
23 23
      */
24
-	protected $services = [];
24
+	protected $services = [ ];
25 25
 
26 26
     /**
27 27
      * The container's bucket items
28 28
      *
29 29
      * @var array
30 30
      */
31
-	protected $bucket = [];
31
+	protected $bucket = [ ];
32 32
 
33 33
 	/**
34 34
 	 * Set the globally available instance of the container.
@@ -37,7 +37,7 @@  discard block
 block discarded – undo
37 37
 	 */
38 38
 	public static function getInstance()
39 39
 	{
40
-		if( is_null( static::$instance )) {
40
+		if( is_null( static::$instance ) ) {
41 41
 			static::$instance = new static;
42 42
 		}
43 43
 
@@ -54,7 +54,7 @@  discard block
 block discarded – undo
54 54
 	 */
55 55
 	public function bind( $alias, $concrete )
56 56
 	{
57
-		$this->services[$alias] = $concrete;
57
+		$this->services[ $alias ] = $concrete;
58 58
 	}
59 59
 
60 60
 	/**
@@ -68,14 +68,14 @@  discard block
 block discarded – undo
68 68
 	 */
69 69
 	public function make( $abstract )
70 70
 	{
71
-		$service = isset( $this->services[$abstract] )
72
-			? $this->services[$abstract]
71
+		$service = isset( $this->services[ $abstract ] )
72
+			? $this->services[ $abstract ]
73 73
 			: $this->addNamespace( $abstract );
74 74
 
75
-		if( is_callable( $service )) {
76
-			return call_user_func_array( $service, [$this] );
75
+		if( is_callable( $service ) ) {
76
+			return call_user_func_array( $service, [ $this ] );
77 77
 		}
78
-		if( is_object( $service )) {
78
+		if( is_object( $service ) ) {
79 79
 			return $service;
80 80
 		}
81 81
 
@@ -92,7 +92,7 @@  discard block
 block discarded – undo
92 92
 	 */
93 93
 	public function singleton( $abstract, $concrete )
94 94
 	{
95
-		$this->bind( $abstract, $this->make( $concrete ));
95
+		$this->bind( $abstract, $this->make( $concrete ) );
96 96
 	}
97 97
 
98 98
 	/**
@@ -104,8 +104,8 @@  discard block
 block discarded – undo
104 104
 	 */
105 105
 	public function __get( $item )
106 106
 	{
107
-		return isset( $this->bucket[$item] )
108
-			? $this->bucket[$item]
107
+		return isset( $this->bucket[ $item ] )
108
+			? $this->bucket[ $item ]
109 109
 			: null;
110 110
 	}
111 111
 
@@ -119,7 +119,7 @@  discard block
 block discarded – undo
119 119
 	 */
120 120
 	public function __set( $item, $value )
121 121
 	{
122
-		$this->bucket[$item] = $value;
122
+		$this->bucket[ $item ] = $value;
123 123
 	}
124 124
 
125 125
    /**
@@ -141,7 +141,7 @@  discard block
 block discarded – undo
141 141
 	 */
142 142
 	protected function addNamespace( $abstract )
143 143
 	{
144
-		if( strpos( $abstract, __NAMESPACE__ ) === false && !class_exists( $abstract )) {
144
+		if( strpos( $abstract, __NAMESPACE__ ) === false && !class_exists( $abstract ) ) {
145 145
 			$abstract = __NAMESPACE__ . "\\$abstract";
146 146
 		}
147 147
 
@@ -183,7 +183,7 @@  discard block
 block discarded – undo
183 183
 			return $this->notInstantiable( $concrete );
184 184
 		}
185 185
 
186
-		if( is_null(( $constructor = $reflector->getConstructor() ))) {
186
+		if( is_null( ( $constructor = $reflector->getConstructor() ) ) ) {
187 187
 			return new $concrete;
188 188
 		}
189 189
 
@@ -218,11 +218,11 @@  discard block
 block discarded – undo
218 218
 	 */
219 219
 	protected function resolveDependencies( array $dependencies )
220 220
 	{
221
-		$results = [];
221
+		$results = [ ];
222 222
 
223 223
 		foreach( $dependencies as $dependency ) {
224 224
 			// If the class is null, the dependency is a string or some other primitive type
225
-			$results[] = !is_null( $class = $dependency->getClass() )
225
+			$results[ ] = !is_null( $class = $dependency->getClass() )
226 226
 				? $this->resolveClass( $dependency )
227 227
 				: null;
228 228
 		}
Please login to merge, or discard this patch.
src/Application.php 2 patches
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -31,7 +31,8 @@
 block discarded – undo
31 31
 			'name' => 'Plugin Name',
32 32
 			'version' => 'Version',
33 33
 		), 'plugin' );
34
-		array_walk( $data, function( $value, $key ) {
34
+		array_walk( $data, function( $value, $key )
35
+		{
35 36
 			$this->$key = $value;
36 37
 		});
37 38
 	}
Please login to merge, or discard this patch.
Spacing   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -23,8 +23,8 @@  discard block
 block discarded – undo
23 23
 
24 24
 	public function __construct()
25 25
 	{
26
-		$this->file = realpath( dirname( dirname( __FILE__ )) . '/pollux.php' );
27
-		$this->gatekeeper = new GateKeeper( plugin_basename( $this->file ));
26
+		$this->file = realpath( dirname( dirname( __FILE__ ) ) . '/pollux.php' );
27
+		$this->gatekeeper = new GateKeeper( plugin_basename( $this->file ) );
28 28
 
29 29
 		$data = get_file_data( $this->file, array(
30 30
 			'id' => 'Text Domain',
@@ -47,10 +47,10 @@  discard block
 block discarded – undo
47 47
 
48 48
 		$controller = $this->make( 'Controller' );
49 49
 
50
-		add_action( 'admin_enqueue_scripts',      array( $controller, 'registerAssets' ));
51
-		add_action( 'admin_init',                 array( $controller, 'removeDashboardWidgets' ));
52
-		add_action( 'wp_before_admin_bar_render', array( $controller, 'removeWordPressMenu' ));
53
-		add_filter( 'admin_footer_text',          array( $controller, 'filterWordPressFooter' ));
50
+		add_action( 'admin_enqueue_scripts', array( $controller, 'registerAssets' ) );
51
+		add_action( 'admin_init', array( $controller, 'removeDashboardWidgets' ) );
52
+		add_action( 'wp_before_admin_bar_render', array( $controller, 'removeWordPressMenu' ) );
53
+		add_filter( 'admin_footer_text', array( $controller, 'filterWordPressFooter' ) );
54 54
 
55 55
 		// Disallow indexing of the site on non-production environments
56 56
 		if( !$this->environment( 'production' ) && !is_admin() ) {
@@ -65,7 +65,7 @@  discard block
 block discarded – undo
65 65
 	{
66 66
 		Facade::clearResolvedInstances();
67 67
 		Facade::setFacadeApplication( $this );
68
-		$this->config = (new Config( $this ))->get();
68
+		$this->config = ( new Config( $this ) )->get();
69 69
 		$this->registerAliases();
70 70
 		$classNames = array(
71 71
 			'MetaBox', 'PostType', 'Settings', 'Taxonomy',
@@ -82,10 +82,10 @@  discard block
 block discarded – undo
82 82
 	 */
83 83
 	public function buildClassName( $name, $path = '' )
84 84
 	{
85
-		$className = array_map( 'ucfirst', array_map( 'strtolower', preg_split( '/[-_]/', $name )));
85
+		$className = array_map( 'ucfirst', array_map( 'strtolower', preg_split( '/[-_]/', $name ) ) );
86 86
 		$className = implode( '', $className );
87 87
 		return !empty( $path )
88
-			? str_replace( '\\\\', '\\', sprintf( '%s\%s', $path, $className ))
88
+			? str_replace( '\\\\', '\\', sprintf( '%s\%s', $path, $className ) )
89 89
 			: $className;
90 90
 	}
91 91
 
@@ -96,7 +96,7 @@  discard block
 block discarded – undo
96 96
 	 */
97 97
 	public function buildMethodName( $name, $prefix = 'get' )
98 98
 	{
99
-		return lcfirst( $this->buildClassName( $prefix . '-' . $name ));
99
+		return lcfirst( $this->buildClassName( $prefix . '-' . $name ) );
100 100
 	}
101 101
 
102 102
 	/**
@@ -106,7 +106,7 @@  discard block
 block discarded – undo
106 106
 	public function environment( $checkFor = '' )
107 107
 	{
108 108
 		$environment = defined( 'WP_ENV' ) ? WP_ENV : 'production';
109
-		if( !empty( $checkFor )) {
109
+		if( !empty( $checkFor ) ) {
110 110
 			return $environment == $checkFor;
111 111
 		}
112 112
 		return $environment;
@@ -120,7 +120,7 @@  discard block
 block discarded – undo
120 120
 		$option = apply_filters( 'pollux/settings/option', Settings::ID );
121 121
 		$settings = get_option( $option );
122 122
 		if( !$settings ) {
123
-			update_option( $option, [] );
123
+			update_option( $option, [ ] );
124 124
 		}
125 125
 	}
126 126
 
@@ -171,6 +171,6 @@  discard block
 block discarded – undo
171 171
 	 */
172 172
 	public function url( $path = '' )
173 173
 	{
174
-		return esc_url( plugin_dir_url( $this->file ) . ltrim( trim( $path ), '/' ));
174
+		return esc_url( plugin_dir_url( $this->file ) . ltrim( trim( $path ), '/' ) );
175 175
 	}
176 176
 }
Please login to merge, or discard this patch.
src/GateKeeper.php 2 patches
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -18,8 +18,8 @@  discard block
 block discarded – undo
18 18
 
19 19
 		$this->plugin = $plugin;
20 20
 
21
-		add_action( 'activated_plugin', array( $this, 'deactivate' ));
22
-		add_action( 'admin_notices',    array( $this, 'deactivate' ));
21
+		add_action( 'activated_plugin', array( $this, 'deactivate' ) );
22
+		add_action( 'admin_notices', array( $this, 'deactivate' ) );
23 23
 	}
24 24
 
25 25
 	/**
@@ -61,7 +61,7 @@  discard block
 block discarded – undo
61 61
 			: sprintf( __( 'Sorry, Pollux requires PHP %s or greater in order to work properly (your server is running PHP %s).', 'pollux' ), self::MIN_PHP_VERSION, PHP_VERSION );
62 62
 
63 63
 		$message2 = $this->checkPhpVersion()
64
-			? sprintf( '<a href="%s">%s</a>', admin_url( 'update-core.php' ), __( 'Update WordPress', 'pollux' ))
64
+			? sprintf( '<a href="%s">%s</a>', admin_url( 'update-core.php' ), __( 'Update WordPress', 'pollux' ) )
65 65
 			: __( 'Please contact your hosting provider or server administrator to upgrade the version of PHP running on your server, or find an alternate plugin.', 'pollux' );
66 66
 
67 67
 		printf( '<div id="message" class="notice notice-error error is-dismissible"><p><strong>%s</strong></p><p>%s %s</p></div>',
@@ -88,7 +88,7 @@  discard block
 block discarded – undo
88 88
 			filter_input( INPUT_GET, 'plugin_status' ),
89 89
 			filter_input( INPUT_GET, 'paged' ),
90 90
 			filter_input( INPUT_GET, 's' )
91
-		)));
91
+		) ) );
92 92
 		exit;
93 93
 	}
94 94
 }
Please login to merge, or discard this patch.
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -14,7 +14,9 @@
 block discarded – undo
14 14
 
15 15
 	public function __construct( $plugin )
16 16
 	{
17
-		if( $this->proceed() )return;
17
+		if( $this->proceed() ) {
18
+			return;
19
+		}
18 20
 
19 21
 		$this->plugin = $plugin;
20 22
 
Please login to merge, or discard this patch.
src/Settings.php 3 patches
Upper-Lower-Casing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -15,7 +15,7 @@
 block discarded – undo
15 15
 	/**
16 16
 	 * @var string
17 17
 	 */
18
-	CONST ID = 'pollux-settings';
18
+	const ID = 'pollux-settings';
19 19
 
20 20
 	/**
21 21
 	 * @var string
Please login to merge, or discard this patch.
Spacing   +37 added lines, -37 removed lines patch added patch discarded remove patch
@@ -39,14 +39,14 @@  discard block
 block discarded – undo
39 39
 
40 40
 		$this->id = apply_filters( 'pollux/settings/option', static::ID );
41 41
 
42
-		add_action( 'admin_menu',                             [$this, 'addPage'] );
43
-		add_action( 'pollux/settings/init',                   [$this, 'addSubmitMetaBox'] );
44
-		add_action( 'current_screen',                         [$this, 'register'] );
45
-		add_action( 'admin_menu',                             [$this, 'registerSetting'] );
46
-		add_action( 'pollux/settings/init',                   [$this, 'reset'] );
47
-		add_action( "admin_footer-toplevel_page_{$this->id}", [$this, 'renderFooterScript'] );
48
-		add_filter( 'pollux/settings/instruction',            [$this, 'filterInstruction'], 10, 3 );
49
-		add_filter( 'wp_redirect',                            [$this, 'filterRedirectOnSave'] );
42
+		add_action( 'admin_menu', [ $this, 'addPage' ] );
43
+		add_action( 'pollux/settings/init', [ $this, 'addSubmitMetaBox' ] );
44
+		add_action( 'current_screen', [ $this, 'register' ] );
45
+		add_action( 'admin_menu', [ $this, 'registerSetting' ] );
46
+		add_action( 'pollux/settings/init', [ $this, 'reset' ] );
47
+		add_action( "admin_footer-toplevel_page_{$this->id}", [ $this, 'renderFooterScript' ] );
48
+		add_filter( 'pollux/settings/instruction', [ $this, 'filterInstruction' ], 10, 3 );
49
+		add_filter( 'wp_redirect', [ $this, 'filterRedirectOnSave' ] );
50 50
 	}
51 51
 
52 52
 	/**
@@ -60,10 +60,10 @@  discard block
 block discarded – undo
60 60
 			__( 'Site Settings', 'pollux' ),
61 61
 			'edit_theme_options',
62 62
 			$this->id,
63
-			[$this, 'renderPage'],
63
+			[ $this, 'renderPage' ],
64 64
 			'dashicons-screenoptions',
65 65
 			1313
66
-		]));
66
+		] ) );
67 67
 	}
68 68
 
69 69
 	/**
@@ -75,11 +75,11 @@  discard block
 block discarded – undo
75 75
 		call_user_func_array( 'add_meta_box', apply_filters( 'pollux/settings/metabox/submit', [
76 76
 			'submitdiv',
77 77
 			__( 'Save Settings', 'pollux' ),
78
-			[ $this, 'renderSubmitMetaBox'],
78
+			[ $this, 'renderSubmitMetaBox' ],
79 79
 			$this->hook,
80 80
 			'side',
81 81
 			'high',
82
-		]));
82
+		] ) );
83 83
 	}
84 84
 
85 85
 	/**
@@ -102,13 +102,13 @@  discard block
 block discarded – undo
102 102
 	public function filterRedirectOnSave( $location )
103 103
 	{
104 104
 		if( strpos( $location, 'settings-updated=true' ) === false
105
-			|| strpos( $location, sprintf( 'page=%s', $this->id )) === false ) {
105
+			|| strpos( $location, sprintf( 'page=%s', $this->id ) ) === false ) {
106 106
 			return $location;
107 107
 		}
108
-		return add_query_arg([
108
+		return add_query_arg( [
109 109
 			'page' => $this->id,
110 110
 			'settings-updated' => 'true',
111
-		], admin_url( 'admin.php' ));
111
+		], admin_url( 'admin.php' ) );
112 112
 	}
113 113
 
114 114
 	/**
@@ -118,8 +118,8 @@  discard block
 block discarded – undo
118 118
 	 */
119 119
 	public function filterSavedSettings( $settings )
120 120
 	{
121
-		if( is_null( $settings )) {
122
-			$settings = [];
121
+		if( is_null( $settings ) ) {
122
+			$settings = [ ];
123 123
 		}
124 124
 		return apply_filters( 'pollux/settings/save', $settings );
125 125
 	}
@@ -128,7 +128,7 @@  discard block
 block discarded – undo
128 128
 	 * @return void
129 129
 	 * @action current_screen
130 130
 	 */
131
-	public function register( $metaboxes = [] )
131
+	public function register( $metaboxes = [ ] )
132 132
 	{
133 133
 		if( $this->app->screen()->id != $this->hook )return;
134 134
 		foreach( parent::register() as $metabox ) {
@@ -137,7 +137,7 @@  discard block
 block discarded – undo
137 137
 		add_screen_option( 'layout_columns', [
138 138
 			'max' => 2,
139 139
 			'default' => 2,
140
-		]);
140
+		] );
141 141
 		do_action( 'pollux/settings/init' );
142 142
 	}
143 143
 
@@ -147,7 +147,7 @@  discard block
 block discarded – undo
147 147
 	 */
148 148
 	public function registerSetting()
149 149
 	{
150
-		register_setting( $this->id, $this->id, [$this, 'filterSavedSettings'] );
150
+		register_setting( $this->id, $this->id, [ $this, 'filterSavedSettings' ] );
151 151
 	}
152 152
 
153 153
 	/**
@@ -160,7 +160,7 @@  discard block
 block discarded – undo
160 160
 			'confirm' => __( 'Are you sure want to do this?', 'pollux' ),
161 161
 			'hook' => $this->hook,
162 162
 			'id' => $this->id,
163
-		]);
163
+		] );
164 164
 	}
165 165
 
166 166
 	/**
@@ -173,7 +173,7 @@  discard block
 block discarded – undo
173 173
 			'columns' => get_current_screen()->get_columns(),
174 174
 			'id' => $this->id,
175 175
 			'title' => __( 'Site Settings', 'pollux' ),
176
-		]);
176
+		] );
177 177
 	}
178 178
 
179 179
 	/**
@@ -189,9 +189,9 @@  discard block
 block discarded – undo
189 189
 		];
190 190
 		$this->render( 'settings/submit', [
191 191
 			'reset' => __( 'Reset Settings', 'pollux' ),
192
-			'reset_url' => esc_url( add_query_arg( $query, admin_url( 'admin.php' ))),
192
+			'reset_url' => esc_url( add_query_arg( $query, admin_url( 'admin.php' ) ) ),
193 193
 			'submit' => get_submit_button( __( 'Save', 'pollux' ), 'primary', 'submit', false ),
194
-		]);
194
+		] );
195 195
 	}
196 196
 
197 197
 	/**
@@ -203,11 +203,11 @@  discard block
 block discarded – undo
203 203
 		if( filter_input( INPUT_GET, 'page' ) !== $this->id
204 204
 			|| filter_input( INPUT_GET, 'action' ) !== 'reset'
205 205
 		)return;
206
-		if( wp_verify_nonce( filter_input( INPUT_GET, '_wpnonce' ), $this->hook )) {
206
+		if( wp_verify_nonce( filter_input( INPUT_GET, '_wpnonce' ), $this->hook ) ) {
207 207
 			update_option( $this->id, $this->getDefaults() );
208 208
 			return add_settings_error( $this->id, 'reset', __( 'Settings reset to defaults.', 'pollux' ), 'updated' );
209 209
 		}
210
-		add_settings_error( $this->id, 'reset', __( 'Failed to reset settings. Please refresh the page and try again.', 'pollux' ));
210
+		add_settings_error( $this->id, 'reset', __( 'Failed to reset settings. Please refresh the page and try again.', 'pollux' ) );
211 211
 	}
212 212
 
213 213
 	/**
@@ -217,7 +217,7 @@  discard block
 block discarded – undo
217 217
 	protected function filterArrayByKey( array $array, $key )
218 218
 	{
219 219
 		return array_filter( $array, function( $value ) use( $key ) {
220
-			return !empty( $value[$key] );
220
+			return !empty( $value[ $key ] );
221 221
 		});
222 222
 	}
223 223
 
@@ -230,11 +230,11 @@  discard block
 block discarded – undo
230 230
 
231 231
 		array_walk( $metaboxes, function( &$metabox ) {
232 232
 			$fields = array_map( function( $field ) {
233
-				$field = wp_parse_args( $field, ['std' => ''] );
234
-				return [$field['slug'] => $field['std']];
235
-			}, $this->filterArrayByKey( $metabox['fields'], 'slug' ));
233
+				$field = wp_parse_args( $field, [ 'std' => '' ] );
234
+				return [ $field[ 'slug' ] => $field[ 'std' ] ];
235
+			}, $this->filterArrayByKey( $metabox[ 'fields' ], 'slug' ) );
236 236
 			$metabox = [
237
-				$metabox['slug'] => call_user_func_array( 'array_merge', $fields ),
237
+				$metabox[ 'slug' ] => call_user_func_array( 'array_merge', $fields ),
238 238
 			];
239 239
 		});
240 240
 		return call_user_func_array( 'array_merge', $metaboxes );
@@ -245,14 +245,14 @@  discard block
 block discarded – undo
245 245
 	 */
246 246
 	protected function normalize()
247 247
 	{
248
-		foreach( $this->app->config['settings'] as $id => $metabox ) {
248
+		foreach( $this->app->config[ 'settings' ] as $id => $metabox ) {
249 249
 			$defaults = [
250
-				'condition' => [],
251
-				'fields' => [],
250
+				'condition' => [ ],
251
+				'fields' => [ ],
252 252
 				'id' => $id,
253 253
 				'slug' => $id,
254 254
 			];
255
-			$this->metaboxes[] = $this->normalizeThis( $metabox, $defaults, $id );
255
+			$this->metaboxes[ ] = $this->normalizeThis( $metabox, $defaults, $id );
256 256
 		}
257 257
 	}
258 258
 
@@ -263,10 +263,10 @@  discard block
 block discarded – undo
263 263
 	 */
264 264
 	protected function normalizeFieldName( $name, array $data, $parentId )
265 265
 	{
266
-		if( !empty( $name )) {
266
+		if( !empty( $name ) ) {
267 267
 			return $name;
268 268
 		}
269
-		$name = str_replace( sprintf( '%s-%s-', $this->id, $parentId ), '', $data['id'] );
269
+		$name = str_replace( sprintf( '%s-%s-', $this->id, $parentId ), '', $data[ 'id' ] );
270 270
 		return sprintf( '%s[%s][%s]', $this->id, $parentId, $name );
271 271
 	}
272 272
 
Please login to merge, or discard this patch.
Braces   +12 added lines, -5 removed lines patch added patch discarded remove patch
@@ -130,7 +130,9 @@  discard block
 block discarded – undo
130 130
 	 */
131 131
 	public function register( $metaboxes = [] )
132 132
 	{
133
-		if( $this->app->screen()->id != $this->hook )return;
133
+		if( $this->app->screen()->id != $this->hook ) {
134
+			return;
135
+		}
134 136
 		foreach( parent::register() as $metabox ) {
135 137
 			new SettingsMetaBox( $metabox );
136 138
 		}
@@ -202,7 +204,9 @@  discard block
 block discarded – undo
202 204
 	{
203 205
 		if( filter_input( INPUT_GET, 'page' ) !== $this->id
204 206
 			|| filter_input( INPUT_GET, 'action' ) !== 'reset'
205
-		)return;
207
+		) {
208
+			return;
209
+		}
206 210
 		if( wp_verify_nonce( filter_input( INPUT_GET, '_wpnonce' ), $this->hook )) {
207 211
 			update_option( $this->id, $this->getDefaults() );
208 212
 			return add_settings_error( $this->id, 'reset', __( 'Settings reset to defaults.', 'pollux' ), 'updated' );
@@ -216,7 +220,8 @@  discard block
 block discarded – undo
216 220
 	 */
217 221
 	protected function filterArrayByKey( array $array, $key )
218 222
 	{
219
-		return array_filter( $array, function( $value ) use( $key ) {
223
+		return array_filter( $array, function( $value ) use( $key )
224
+		{
220 225
 			return !empty( $value[$key] );
221 226
 		});
222 227
 	}
@@ -228,8 +233,10 @@  discard block
 block discarded – undo
228 233
 	{
229 234
 		$metaboxes = $this->filterArrayByKey( $this->metaboxes, 'slug' );
230 235
 
231
-		array_walk( $metaboxes, function( &$metabox ) {
232
-			$fields = array_map( function( $field ) {
236
+		array_walk( $metaboxes, function( &$metabox )
237
+		{
238
+			$fields = array_map( function( $field )
239
+			{
233 240
 				$field = wp_parse_args( $field, ['std' => ''] );
234 241
 				return [$field['slug'] => $field['std']];
235 242
 			}, $this->filterArrayByKey( $metabox['fields'], 'slug' ));
Please login to merge, or discard this patch.