amarkal / amarkal-ui

functions.php

Braces +3 added lines, -1 removed lines

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

Form.php

Braces +3 added lines, -1 removed lines

@@ -202,7 +202,9 @@
      */
     private function validate( $component )
     {
-        if( !($component instanceof ValidatableComponentInterface) ) return;
+        if( !($component instanceof ValidatableComponentInterface) ) {
+            return;
+        }
         
         $name     = $component->name;
         $validate = $component->validation;

Doc Comments +5 added lines, -5 removed lines

@@ -137,7 +137,7 @@ 
      * Get a component by its name.
      * 
      * @param string $name
-     * @return UI\AbstractComponent
+     * @return AbstractComponent
      */
     public function get_component( $name )
     {
@@ -147,7 +147,7 @@ 
     /**
      * Get all components.
      * 
-     * @return array
+     * @return AbstractComponent[]
      */
     public function get_components()
     {
@@ -181,7 +181,7 @@ 
     /**
      * Check if the given component is disabled.
      * 
-     * @param UI\AbstractComponent $component
+     * @param ValueComponentInterface $component
      * @return boolean
      */
     private function is_disabled( $component )
@@ -197,7 +197,7 @@ 
     /**
      * Filter the component's value using its filter function (if applicable)
      * 
-     * @param UI\AbstractComponent $component
+     * @param ValueComponentInterface $component
      */
     private function filter( $component )
     {
@@ -219,7 +219,7 @@ 
      * If the value is invalid, the old value is used, and an error message is
      * saved into the errors array as component_name => error_message.
      * 
-     * @param UI\AbstractComponent $component The component to validate.
+     * @param ValueComponentInterface $component The component to validate.
      */
     private function validate( $component )
     {

Template.php

Braces +1 added lines, -2 removed lines

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

AbstractComponent.php

Doc Comments +1 added lines, -1 removed lines

@@ -89,7 +89,7 @@
     /**
      * Get the name for this component by parsing the name template.
      * 
-     * @return type
+     * @return string
      */
     public function get_name()
     {

components/composite/controller.php

Doc Comments +1 added lines, -1 removed lines

@@ -123,7 +123,7 @@
     /**
      * 
      * @param type $args
-     * @return type
+     * @return AbstractComponent
      */
     private function create_component( $args )
     {

ComponentFactory.php

Doc Comments +7 added lines, -4 removed lines

@@ -50,6 +50,9 @@ 
         else throw new \RuntimeException("A component of type '$type' has already been registered.");
     }
     
+    /**
+     * @param string $path
+     */
     public static function get_url( $path )
     {
         $url  = str_replace( ABSPATH, '', $path );
@@ -58,8 +61,8 @@ 
     
     /**
      * 
-     * @param type $type
-     * @param type $props
+     * @param string $type
+     * @param type string
      * @throws \RuntimeException
      */
     private static function create_core_component( $type, $props )
@@ -85,8 +88,8 @@ 
     
     /**
      * 
-     * @param type $type
-     * @param type $props
+     * @param string $type
+     * @param type string
      * @throws \RuntimeException
      */
     private static function create_registered_component( $type, $props )

Braces +3 added lines, -3 removed lines

@@ -46,8 +46,9 @@ 
         if( !in_array($type, self::$registered_components) )
         {
             self::$registered_components[$type] = $class_name;
+        } else {
+            throw new \RuntimeException("A component of type '$type' has already been registered.");
         }
-        else throw new \RuntimeException("A component of type '$type' has already been registered.");
     }
     
     public static function get_url( $path )
@@ -73,8 +74,7 @@ 
             if( file_exists( $file_name ) ) 
             {
                 require_once $file_name;
-            }
-            else 
+            } else 
             {
                 throw new \RuntimeException("A component of type '$type' does not exist.");
             }