amarkal / amarkal-metabox

bootstrap.php

Spacing +2 added lines, -2 removed lines

@@ -6,13 +6,13 @@
  */
 
 // Prevent direct file access
-defined( 'ABSPATH' ) or die( 'No script kiddies please!' );
+defined('ABSPATH') or die('No script kiddies please!');
 
 /**
  * Load module functions. If this amarkal module has not been loaded, 
  * functions.php will not return false.
  */
-if(false !== (require_once 'functions.php'))
+if (false !== (require_once 'functions.php'))
 {
     // Load required classes if not using composer
     

Manager.php

Spacing +24 added lines, -24 removed lines

@@ -29,7 +29,7 @@ 
      */
     public static function get_instance()
     {
-        if( null === static::$instance ) 
+        if (null === static::$instance) 
         {
             static::$instance = new static();
             static::$instance->init();
@@ -44,9 +44,9 @@ 
      * @param array $args
      * @throws \RuntimeException if the given metabox id has already been registered
      */
-    public function add( $id, array $args )
+    public function add($id, array $args)
     {
-        if( !in_array($id, $this->metaboxes) )
+        if (!in_array($id, $this->metaboxes))
         {
             $this->metaboxes[$id] = array_merge($this->default_args(), $args);
         }
@@ -59,11 +59,11 @@ 
      * @param WP_Post $post
      * @param array $args
      */
-    public function render( $post, $args )
+    public function render($post, $args)
     {
         $metabox = $this->metaboxes[$args['id']];
         wp_nonce_field(self::NONCE_ACTION, $args['id'].'_nonce');
-        foreach( $metabox['fields'] as $field )
+        foreach ($metabox['fields'] as $field)
         {
             $field['post_id'] = $post->ID;
             $field_template = new Field($field);
@@ -76,7 +76,7 @@ 
      */
     public function add_meta_boxes()
     {
-        foreach( $this->metaboxes as $id => $args )
+        foreach ($this->metaboxes as $id => $args)
         {
             \add_meta_box(
                 $id,
@@ -94,7 +94,7 @@ 
      * 
      * @param number $post_id
      */
-    public function save_meta_boxes( $post_id )
+    public function save_meta_boxes($post_id)
     {
         /**
          * A note on security:
@@ -112,25 +112,25 @@ 
          * If this is an autosave, our form has not been submitted,
          * so we don't want to do anything.
          */
-        if( defined( 'DOING_AUTOSAVE' ) && DOING_AUTOSAVE ) 
+        if (defined('DOING_AUTOSAVE') && DOING_AUTOSAVE) 
         {
             return $post_id;
         }
  
         // Check the user's permissions.
-        if( array_key_exists('post_type', $_POST) && 'page' == filter_input(INPUT_POST, 'post_type') ) 
+        if (array_key_exists('post_type', $_POST) && 'page' == filter_input(INPUT_POST, 'post_type')) 
         {
-            if( !current_user_can('edit_page', $post_id) ) return $post_id;
+            if (!current_user_can('edit_page', $post_id)) return $post_id;
         } 
         else 
         {
-            if( !current_user_can('edit_post', $post_id) ) return $post_id;
+            if (!current_user_can('edit_post', $post_id)) return $post_id;
         }
 
         // Update the meta fields.
-        foreach( $this->metaboxes as $id => $metabox )
+        foreach ($this->metaboxes as $id => $metabox)
         {
-            $this->save_meta_box( $post_id, $id, $metabox );
+            $this->save_meta_box($post_id, $id, $metabox);
         }
     }
     
@@ -141,12 +141,12 @@ 
      * @param string $id
      * @param array $metabox
      */
-    public function save_meta_box( $post_id, $id, $metabox )
+    public function save_meta_box($post_id, $id, $metabox)
     {
         $nonce_name = $id.'_nonce';
         
         // Check if our nonce is set.
-        if( !array_key_exists($nonce_name, $_POST) ) 
+        if (!array_key_exists($nonce_name, $_POST)) 
         {
             return $post_id;
         }
@@ -154,15 +154,15 @@ 
         $nonce = filter_input(INPUT_POST, $nonce_name);
 
         // Verify that the nonce is valid.
-        if ( !wp_verify_nonce($nonce, self::NONCE_ACTION) ) 
+        if (!wp_verify_nonce($nonce, self::NONCE_ACTION)) 
         {
             return $post_id;
         }
         
-        foreach( $metabox['fields'] as $field )
+        foreach ($metabox['fields'] as $field)
         {
-            $data = filter_input( INPUT_POST, $field['name'] );
-            \update_post_meta( $post_id, $field['name'], $data );
+            $data = filter_input(INPUT_POST, $field['name']);
+            \update_post_meta($post_id, $field['name'], $data);
         }
     }
     
@@ -173,9 +173,9 @@ 
     {
         $cs = get_current_screen();
         
-        foreach( $this->metaboxes as $metabox )
+        foreach ($this->metaboxes as $metabox)
         {
-            if( $metabox['screen'] === $cs->id )
+            if ($metabox['screen'] === $cs->id)
             {
                 echo '<style>';
                 include 'metabox.css';
@@ -190,9 +190,9 @@ 
      */
     private function init()
     {
-        \add_action( 'add_meta_boxes', array( $this, 'add_meta_boxes' ) );
-        \add_action( 'save_post', array( $this, 'save_meta_boxes' ) );
-        \add_action( 'admin_footer', array( $this, 'print_style' ) );
+        \add_action('add_meta_boxes', array($this, 'add_meta_boxes'));
+        \add_action('save_post', array($this, 'save_meta_boxes'));
+        \add_action('admin_footer', array($this, 'print_style'));
     }
     
     /**

Braces +9 added lines, -5 removed lines

@@ -49,8 +49,9 @@ 
         if( !in_array($id, $this->metaboxes) )
         {
             $this->metaboxes[$id] = array_merge($this->default_args(), $args);
+        } else {
+            throw new \RuntimeException("A metabox with id '$id' has already been registered.");
         }
-        else throw new \RuntimeException("A metabox with id '$id' has already been registered.");
     }
     
     /**
@@ -120,11 +121,14 @@ 
         // Check the user's permissions.
         if( array_key_exists('post_type', $_POST) && 'page' == filter_input(INPUT_POST, 'post_type') ) 
         {
-            if( !current_user_can('edit_page', $post_id) ) return $post_id;
-        } 
-        else 
+            if( !current_user_can('edit_page', $post_id) ) {
+                return $post_id;
+            }
+        } else 
         {
-            if( !current_user_can('edit_post', $post_id) ) return $post_id;
+            if( !current_user_can('edit_post', $post_id) ) {
+                return $post_id;
+            }
         }
 
         // Update the meta fields.