amarkal / amarkal-metabox

composer.php

Spacing +1 added lines, -1 removed lines

@@ -6,7 +6,7 @@
  */
 
 // Prevent direct file access
-defined( 'ABSPATH' ) or die( 'No script kiddies please!' );
+defined('ABSPATH') or die('No script kiddies please!');
 
 // Load module functions
 require_once 'functions.php';

functions.php

Braces +3 added lines, -1 removed lines

@@ -18,7 +18,9 @@
 /**
  * Prevent loading the library more than once
  */
-if( defined( 'AMARKAL_METABOX' ) ) return false;
+if( defined( 'AMARKAL_METABOX' ) ) {
+    return false;
+}
 define( 'AMARKAL_METABOX', true );
 
 if(!function_exists('amarkal_add_meta_box'))

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

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

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.");
     }
     
     /**

Spacing +32 added lines, -32 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,7 +67,7 @@ 
      * @param WP_Post $post
      * @param array $args
      */
-    public function render( $post, $args )
+    public function render($post, $args)
     {
         $metabox = $this->metaboxes[$args['id']];
         
@@ -93,7 +93,7 @@ 
      */
     public function add_meta_boxes()
     {
-        foreach( $this->metaboxes as $id => $args )
+        foreach ($this->metaboxes as $id => $args)
         {
             \add_meta_box(
                 $id,
@@ -111,7 +111,7 @@ 
      * 
      * @param number $post_id
      */
-    public function save_meta_boxes( $post_id )
+    public function save_meta_boxes($post_id)
     {
         /**
          * A note on security:
@@ -129,22 +129,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);
         }
     }
     
@@ -155,14 +155,14 @@ 
      * @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);
         $new_instance = filter_input_array(INPUT_POST);
         
         // 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;
         }
@@ -177,9 +177,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';
@@ -195,13 +195,13 @@ 
      * @param number $post_id
      * @param array $metabox
      */
-    public function print_errors( $post_id, $metabox )
+    public function print_errors($post_id, $metabox)
     {
-        $errors  = \get_transient("amarkal_metabox_errors_$post_id");
+        $errors = \get_transient("amarkal_metabox_errors_$post_id");
         
-        if( $errors )
+        if ($errors)
         {
-            foreach( $errors as $name => $error )
+            foreach ($errors as $name => $error)
             {
                 $component = $metabox['form']->get_component($name);
                 echo "<div class=\"notice notice-error\"><p><strong>{$component->title}</strong> $error</p></div>";
@@ -216,9 +216,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'));
     }
     
     /**
@@ -230,13 +230,13 @@ 
      * @param number $post_id
      * @param array $new_instance
      */
-    private function update_form( $metabox, $post_id, array $new_instance = array() )
+    private function update_form($metabox, $post_id, array $new_instance = array())
     {
         $old_instance   = $this->get_old_instance($metabox, $post_id);
-        $final_instance = $metabox['form']->update( $new_instance, $old_instance );
+        $final_instance = $metabox['form']->update($new_instance, $old_instance);
 
         // Update db if there is new data to be saved
-        if( array() !== $new_instance )
+        if (array() !== $new_instance)
         {
             $this->update_post_meta($final_instance, $post_id);
             
@@ -245,7 +245,7 @@ 
              * a redirect to post.php and then back to our post, which clears
              * the execution thread. See https://www.sitepoint.com/displaying-errors-from-the-save_post-hook-in-wordpress/
              */
-            \set_transient( "amarkal_metabox_errors_$post_id", $metabox['form']->get_errors(), 60 );
+            \set_transient("amarkal_metabox_errors_$post_id", $metabox['form']->get_errors(), 60);
         }
     }
     
@@ -255,11 +255,11 @@ 
      * @param type $final_instance
      * @param type $post_id
      */
-    private function update_post_meta( $final_instance, $post_id )
+    private function update_post_meta($final_instance, $post_id)
     {
-        foreach( $final_instance as $name => $value )
+        foreach ($final_instance as $name => $value)
         {
-            \update_post_meta( $post_id, $name, $value );
+            \update_post_meta($post_id, $name, $value);
         }
     }
     
@@ -270,15 +270,15 @@ 
      * @param number $post_id
      * @return array
      */
-    private function get_old_instance( $metabox, $post_id )
+    private function get_old_instance($metabox, $post_id)
     {
         $old_instance = array();
         
-        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);
             }
         }