amarkal / amarkal-metabox

Manager.php

Spacing +22 added lines, -22 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,22 +112,22 @@ 
          * 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.
         $post_type = filter_input(INPUT_POST, 'post_type');
-        if( null !== $post_type && !current_user_can('edit_'.$post_type, $post_id) )
+        if (null !== $post_type && !current_user_can('edit_'.$post_type, $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);
         }
     }
     
@@ -138,27 +138,27 @@ 
      * @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';
         $nonce_value = filter_input(INPUT_POST, $nonce_name);
         
         // Check if our nonce is set.
-        if( null === $nonce_value ) 
+        if (null === $nonce_value) 
         {
             return $post_id;
         }
 
         // Verify that the nonce is valid.
-        if ( !wp_verify_nonce($nonce_value, self::NONCE_ACTION) ) 
+        if (!wp_verify_nonce($nonce_value, 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);
         }
     }
     
@@ -169,9 +169,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';
@@ -186,9 +186,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 +2 added lines, -1 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.");
     }
     
     /**