amarkal / amarkal-ui

bootstrap.php

Spacing +9 added lines, -9 removed lines

@@ -10,13 +10,13 @@ 
  * @link      https://github.com/askupasoftware/amarkal-ui
  * @copyright 2017 Askupa Software
  */
-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_UI' ) ) return;
-define( 'AMARKAL_UI', true );
+if (defined('AMARKAL_UI')) return;
+define('AMARKAL_UI', true);
 
 /**
  * Load required classes
@@ -24,7 +24,7 @@ 
 require_once 'renderer.php';
 require_once 'abstract-component.php';
 
-if(!function_exists('amarkal_ui_render'))
+if (!function_exists('amarkal_ui_render'))
 {
     /**
      * Render a UI component.
@@ -34,14 +34,14 @@ 
      * @param array $props The component's properties
      * @return string The rendered HTML
      */
-    function amarkal_ui_render( $type, array $props = array() )
+    function amarkal_ui_render($type, array $props = array())
     {
         $renderer = Amarkal\UI\Renderer::get_instance();
-        return $renderer->render_component( $type, $props );
+        return $renderer->render_component($type, $props);
     }
 }
 
-if(!function_exists('amarkal_ui_register_component'))
+if (!function_exists('amarkal_ui_register_component'))
 {
     /**
      * Register a custom UI component. The registered component's class should
@@ -51,9 +51,9 @@ 
      * to one of the core component's type, it will override the core component.
      * @param string $class_name The component's class name.
      */
-    function amarkal_ui_register_component( $type, $class_name )
+    function amarkal_ui_register_component($type, $class_name)
     {
         $renderer = Amarkal\UI\Renderer::get_instance();
-        $renderer->register_component( $type, $class_name );
+        $renderer->register_component($type, $class_name);
     }
 }
\ No newline at end of file

Braces +3 added lines, -1 removed lines

@@ -15,7 +15,9 @@
 /**
  * Prevent loading the library more than once
  */
-if( defined( 'AMARKAL_UI' ) ) return;
+if( defined( 'AMARKAL_UI' ) ) {
+    return;
+}
 define( 'AMARKAL_UI', true );
 
 /**

renderer.php

Spacing +13 added lines, -13 removed lines

@@ -18,16 +18,16 @@ 
      */
     public static function get_instance()
     {
-        if( null === static::$instance ) 
+        if (null === static::$instance) 
         {
             static::$instance = new static();
         }
         return static::$instance;
     }
 
-    public function render_component( $type, array $props = array() )
+    public function render_component($type, array $props = array())
     {
-        $component = $this->create_component( $type, $props );
+        $component = $this->create_component($type, $props);
         return $component->render();
     }
 
@@ -42,9 +42,9 @@ 
      * @param string $class_name
      * @throws \RuntimeException
      */
-    public function register_component( $type, $class_name )
+    public function register_component($type, $class_name)
     {
-        if( !in_array($type, $this->registered_components) )
+        if (!in_array($type, $this->registered_components))
         {
             $this->registered_components[$type] = $class_name;
         }
@@ -57,12 +57,12 @@ 
      * @param type $props
      * @return type
      */
-    private function create_component( $type, $props )
+    private function create_component($type, $props)
     {
         try {
-            $component = $this->create_registered_component ($type, $props );
+            $component = $this->create_registered_component($type, $props);
         } catch (\RuntimeException $ex) {
-            $component = $this->create_core_component ($type, $props );
+            $component = $this->create_core_component($type, $props);
         }
         
         return $component;
@@ -74,15 +74,15 @@ 
      * @param type $props
      * @throws \RuntimeException
      */
-    private function create_core_component( $type, $props )
+    private function create_core_component($type, $props)
     {
         $file_name  = __DIR__."/components/$type/controller.php";
         $class_name = 'Amarkal\\UI\\Component_'.$type;
         
         // Load one of the core components
-        if(!class_exists($class_name))
+        if (!class_exists($class_name))
         {
-            if( file_exists( $file_name ) ) 
+            if (file_exists($file_name)) 
             {
                 require_once $file_name;
             }
@@ -101,9 +101,9 @@ 
      * @param type $props
      * @throws \RuntimeException
      */
-    private function create_registered_component( $type, $props )
+    private function create_registered_component($type, $props)
     {
-        if(in_array($type, $this->registered_components))
+        if (in_array($type, $this->registered_components))
         {
             $class_name = $this->registered_components[$type];
             return new $class_name($props);

Braces +3 added lines, -3 removed lines

@@ -47,8 +47,9 @@ 
         if( !in_array($type, $this->registered_components) )
         {
             $this->registered_components[$type] = $class_name;
+        } else {
+            throw new \RuntimeException("A component with a name of '$type' has already been registered.");
         }
-        else throw new \RuntimeException("A component with a name of '$type' has already been registered.");
     }
     
     /**
@@ -85,8 +86,7 @@ 
             if( file_exists( $file_name ) ) 
             {
                 require_once $file_name;
-            }
-            else 
+            } else 
             {
                 throw new \RuntimeException("The component '$type' does not exist.");
             }

abstract-component.php

Spacing +12 added lines, -12 removed lines

@@ -18,18 +18,18 @@ 
      * 
      * @param array $model
      */
-    public function __construct( array $model = array() ) 
+    public function __construct(array $model = array()) 
     {
         // Check that the required parameters are provided.
-        foreach( $this->required_parameters() as $key )
+        foreach ($this->required_parameters() as $key)
         {
-            if ( !isset($model[$key]) )
+            if (!isset($model[$key]))
             {
                 throw new \RuntimeException('The required parameter "'.$key.'" was not provided for '.get_called_class());
             }
         }
         
-        $this->model = array_merge( $this->default_model(), $model );
+        $this->model = array_merge($this->default_model(), $model);
     }
     
     /**
@@ -39,9 +39,9 @@ 
      * 
      * @return mixed the parameter's value.
      */
-    public function __get( $name ) 
+    public function __get($name) 
     {
-        if( isset( $this->model[$name] ) )
+        if (isset($this->model[$name]))
         {
             return $this->model[$name];
         }
@@ -55,7 +55,7 @@ 
      * 
      * @return mixed the settings' parameter value.
      */
-    public function __set( $name, $value )
+    public function __set($name, $value)
     {
         $this->model[$name] = $value;
     }
@@ -67,22 +67,22 @@ 
      * @throws TemplateNotFoundException    Thrown if the template file can 
      *                                        not found.
      */
-    public function render( $echo = false ){
+    public function render($echo = false) {
         
         $rendered_html = '';
         
-        if( file_exists( $this->get_script_path() ) ) 
+        if (file_exists($this->get_script_path())) 
         {
             ob_start();
-            include( $this->get_script_path() );
+            include($this->get_script_path());
             $rendered_html = ob_get_clean();
         } 
         else 
         {
-            throw new \RuntimeException( "Error: cannot render HTML, template file not found at " . $this->get_script_path() );
+            throw new \RuntimeException("Error: cannot render HTML, template file not found at ".$this->get_script_path());
         }
         
-        if( !$echo )
+        if (!$echo)
         {
             return $rendered_html;
         }

Braces +1 added lines, -2 removed lines

@@ -76,8 +76,7 @@
             ob_start();
             include( $this->get_script_path() );
             $rendered_html = ob_get_clean();
-        } 
-        else 
+        } else 
         {
             throw new \RuntimeException( "Error: cannot render HTML, template file not found at " . $this->get_script_path() );
         }