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'))

ComponentFactory.php

Doc Comments +4 added lines, -4 removed lines

@@ -52,8 +52,8 @@ 
     
     /**
      * 
-     * @param type $type
-     * @param type $props
+     * @param string $type
+     * @param type string
      * @throws \RuntimeException
      */
     private static function create_core_component( $type, $props )
@@ -79,8 +79,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.");
     }
     
     /**
@@ -67,8 +68,7 @@ 
             if( file_exists( $file_name ) ) 
             {
                 require_once $file_name;
-            }
-            else 
+            } else 
             {
                 throw new \RuntimeException("A component of type '$type' does not exist.");
             }

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 +4 added lines, -4 removed lines

@@ -129,7 +129,7 @@ 
     /**
      * Get all components.
      * 
-     * @return array
+     * @return AbstractComponent[]
      */
     public function get_component_list()
     {
@@ -163,7 +163,7 @@ 
     /**
      * Check if the given component is disabled.
      * 
-     * @param UI\AbstractComponent $component
+     * @param ValueComponentInterface $component
      * @return boolean
      */
     private function is_disabled( $component )
@@ -179,7 +179,7 @@ 
     /**
      * Filter the component's value using its filter function (if applicable)
      * 
-     * @param UI\AbstractComponent $component
+     * @param ValueComponentInterface $component
      */
     private function filter( $component )
     {
@@ -201,7 +201,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() );
         }

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 )
     {

components/switch/controller.php

Doc Comments +1 added lines, -1 removed lines

@@ -43,7 +43,7 @@
      * handle this.
      *
      * @param [string] $v
-     * @return void
+     * @return string
      */
     public function filter($v)
     {

Braces +3 added lines, -1 removed lines

@@ -47,7 +47,9 @@
      */
     public function filter($v)
     {
-        if($v !== 'on') return 'off';
+        if($v !== 'on') {
+            return 'off';
+        }
         return 'on';
     }
 }
\ No newline at end of file

components/slider/controller.php

Doc Comments +1 added lines, -1 removed lines

@@ -45,7 +45,7 @@
      * will handle this.
      *
      * @param [string] $v
-     * @return void
+     * @return double
      */
     public function filter($v)
     {

ComponentList.php

Doc Comments +1 added lines, -1 removed lines

@@ -159,7 +159,7 @@
     /**
      * Filter the list of components
      *
-     * @param function $callable
+     * @param \Closure $callable
      * @return array
      */
     private function filter( $callable )