amarkal / amarkal-metabox

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( isset($_POST['post_type']) && 'page' == $_POST['post_type'] ) 
+        if (isset($_POST['post_type']) && 'page' == $_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,25 +141,25 @@ 
      * @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 ( !isset($_POST[$nonce_name]) ) {
+        if (!isset($_POST[$nonce_name])) {
             return $post_id;
         }
 
         $nonce = $_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);
         }
     }
     
@@ -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'));
     }
     
     /**

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( isset($_POST['post_type']) && 'page' == $_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.

functions.php

Spacing +6 added lines, -6 removed lines

@@ -13,15 +13,15 @@ 
  */
 
 // Prevent direct file access
-defined( 'ABSPATH' ) or die( 'No script kiddies please!' );
+defined('ABSPATH') or die('No script kiddies please!');
 
 /**
  * Prevent loading the library more than once
  */
-if( defined( 'AMARKAL_METABOX' ) ) return false;
-define( 'AMARKAL_METABOX', true );
+if (defined('AMARKAL_METABOX')) return false;
+define('AMARKAL_METABOX', true);
 
-if(!function_exists('amarkal_add_meta_box'))
+if (!function_exists('amarkal_add_meta_box'))
 {
     /**
      * Add a meta box to a given post type
@@ -29,9 +29,9 @@ 
      * @param string $id
      * @param array $args
      */
-    function amarkal_add_meta_box( $id, array $args )
+    function amarkal_add_meta_box($id, array $args)
     {
         $mb = Amarkal\Metabox\Manager::get_instance();
-        $mb->add( $id, $args );
+        $mb->add($id, $args);
     }
 }
\ No newline at end of file

Field.php

Spacing +2 added lines, -2 removed lines

@@ -16,8 +16,8 @@
     
     protected function on_created() 
     {
-        \get_post_meta( $this->post_id, $this->name, true );
-        $this->model['value'] = \get_post_meta( $this->post_id, $this->name, true );
+        \get_post_meta($this->post_id, $this->name, true);
+        $this->model['value'] = \get_post_meta($this->post_id, $this->name, true);
     }
     
     public function get_template_path()