amarkal / amarkal-metabox

Manager.php

Spacing +25 added lines, -25 removed lines

@@ -29,7 +29,7 @@ 
      */
     public static function get_instance()
     {
-        if( null === static::$instance ) 
+        if (null === static::$instance) 
         {
             static::$instance = new static();
         }
@@ -51,9 +51,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);
             $this->metaboxes[$id]['form'] = new \Amarkal\UI\Form($args['fields']);
@@ -67,11 +67,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);
@@ -84,7 +84,7 @@ 
      */
     public function add_meta_boxes()
     {
-        foreach( $this->metaboxes as $id => $args )
+        foreach ($this->metaboxes as $id => $args)
         {
             \add_meta_box(
                 $id,
@@ -102,7 +102,7 @@ 
      * 
      * @param number $post_id
      */
-    public function save_meta_boxes( $post_id )
+    public function save_meta_boxes($post_id)
     {
         /**
          * A note on security:
@@ -120,22 +120,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);
         }
     }
     
@@ -146,20 +146,20 @@ 
      * @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 and verify it
-        if( null === $nonce_value || !wp_verify_nonce($nonce_value, self::NONCE_ACTION) ) 
+        if (null === $nonce_value || !wp_verify_nonce($nonce_value, self::NONCE_ACTION)) 
         {
             return $post_id;
         }
 
-        foreach( $this->get_form_data($metabox, $post_id) as $name => $value )
+        foreach ($this->get_form_data($metabox, $post_id) as $name => $value)
         {
-            \update_post_meta( $post_id, $name, $value );
+            \update_post_meta($post_id, $name, $value);
         }
     }
     
@@ -170,9 +170,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';
@@ -187,9 +187,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'));
     }
     
     /**
@@ -199,20 +199,20 @@ 
      * @param number $post_id
      * @return array
      */
-    private function get_form_data( $metabox, $post_id )
+    private function get_form_data($metabox, $post_id)
     {
         $old_instance = array();
         $new_instance = filter_input_array(INPUT_POST);
         
-        foreach( $metabox['fields'] as $field )
+        foreach ($metabox['fields'] as $field)
         {
-            if( in_array($field['name'], get_post_custom_keys($post_id)) )
+            if (in_array($field['name'], get_post_custom_keys($post_id)))
             {
-                $old_instance[$field['name']] = \get_post_meta( $post_id, $field['name'], true );
+                $old_instance[$field['name']] = \get_post_meta($post_id, $field['name'], true);
             }
         }
         
-        return $metabox['form']->update( $new_instance, $old_instance );
+        return $metabox['form']->update($new_instance, $old_instance);
     }
     
     /**

Braces +2 added lines, -1 removed lines

@@ -57,8 +57,9 @@
         {
             $this->metaboxes[$id] = array_merge($this->default_args(), $args);
             $this->metaboxes[$id]['form'] = new \Amarkal\UI\Form($args['fields']);
+        } 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.");
     }
     
     /**