amarkal / amarkal-ui

Form.php

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

Spacing +24 added lines, -24 removed lines

@@ -59,12 +59,12 @@ 
      * 
      * @param array $components An array of arrays of component arguments
      */
-    public function __construct( array $components = array() )
+    public function __construct(array $components = array())
     {
-        foreach( $components as $args )
+        foreach ($components as $args)
         {
             $name = $args['name'];
-            if(array_key_exists($name, $components))
+            if (array_key_exists($name, $components))
             {
                 throw new \RuntimeException("A component with the name <b>$name</b> has already been created");
             }
@@ -89,18 +89,18 @@ 
      * 
      * @return array The updated values array.
      */
-    public function update( array $new_instance, array $old_instance = array() )
+    public function update(array $new_instance, array $old_instance = array())
     {
-        $this->old_instance   = array_merge($this->get_defaults(),$old_instance);
-        $this->new_instance   = array_merge($this->old_instance,$new_instance);
+        $this->old_instance   = array_merge($this->get_defaults(), $old_instance);
+        $this->new_instance   = array_merge($this->old_instance, $new_instance);
         $this->final_instance = $this->new_instance;
         
-        foreach ( $this->components as $component ) 
+        foreach ($this->components as $component) 
         {
             // Update individual fields, as well as the composite parent field.
-            if ( $component instanceof ValueComponentInterface )
+            if ($component instanceof ValueComponentInterface)
             {
-                $this->update_component( $component );
+                $this->update_component($component);
             }
         }
         
@@ -114,7 +114,7 @@ 
      */
     public function reset()
     {
-        foreach( $this->components as $c )
+        foreach ($this->components as $c)
         {
             $c->value = $c->default;
             $this->final_instance[$c->name] = $c->default;
@@ -139,7 +139,7 @@ 
      * @param string $name
      * @return UI\AbstractComponent
      */
-    public function get_component( $name )
+    public function get_component($name)
     {
         return $this->components[$name];
     }
@@ -161,21 +161,21 @@ 
      * 
      * @param ValueComponentInterface $component The component to validate.
      */
-    private function update_component( ValueComponentInterface $component )
+    private function update_component(ValueComponentInterface $component)
     {
         $component->value = $this->final_instance[$component->name];
         
         // Skip if this field is disabled
-        if( $this->is_disabled($component) )
+        if ($this->is_disabled($component))
         {
             return;
         }
         
         // Apply user-defined filter
-        $this->filter( $component );
+        $this->filter($component);
         
         // Validate value
-        $this->validate( $component );
+        $this->validate($component);
     }
     
     /**
@@ -184,7 +184,7 @@ 
      * @param UI\AbstractComponent $component
      * @return boolean
      */
-    private function is_disabled( $component )
+    private function is_disabled($component)
     {
         return 
             $component instanceof DisableableComponentInterface &&
@@ -199,13 +199,13 @@ 
      * 
      * @param UI\AbstractComponent $component
      */
-    private function filter( $component )
+    private function filter($component)
     {
-        if( $component instanceof FilterableComponentInterface )
+        if ($component instanceof FilterableComponentInterface)
         {
             $filter = $component->filter;
 
-            if( is_callable( $filter ) ) 
+            if (is_callable($filter)) 
             {
                 $component->value = \call_user_func_array($filter, array($this->final_instance[$component->name]));
                 $this->final_instance[$component->name] = $component->value;
@@ -221,22 +221,22 @@ 
      * 
      * @param UI\AbstractComponent $component The component to validate.
      */
-    private function validate( $component )
+    private function validate($component)
     {
-        if( !($component instanceof ValidatableComponentInterface) ) return;
+        if (!($component instanceof ValidatableComponentInterface)) return;
         
         $name     = $component->name;
         $validate = $component->validation;
         
         $component->validity = $component::VALID;
         
-        if(is_callable($validate))
+        if (is_callable($validate))
         {
             $error = '';
             $valid = \call_user_func_array($validate, array($this->final_instance[$name], &$error));
             
             // Invalid input, use old instance or default value
-            if ( true !== $valid ) 
+            if (true !== $valid) 
             {
                 $this->errors[$name]         = $error;
                 $component->value            = $this->old_instance[$name];
@@ -255,7 +255,7 @@ 
     {
         $defaults = array();
         
-        foreach( $this->components as $component )
+        foreach ($this->components as $component)
         {
             $defaults[$component->name] = $component->default;
         }

components/textarea/controller.php

Indentation +3 added lines, -3 removed lines

@@ -8,9 +8,9 @@
 class Component_textarea
 extends AbstractComponent
 implements ValueComponentInterface, 
-           DisableableComponentInterface,
-           FilterableComponentInterface,
-           ValidatableComponentInterface
+            DisableableComponentInterface,
+            FilterableComponentInterface,
+            ValidatableComponentInterface
 {
     public function default_model() 
     {

components/radio/controller.php

Indentation +1 added lines, -1 removed lines

@@ -8,7 +8,7 @@
 class Component_radio
 extends AbstractComponent
 implements ValueComponentInterface, 
-           DisableableComponentInterface
+            DisableableComponentInterface
 {
     public function default_model() 
     {

components/select/controller.php

Indentation +1 added lines, -1 removed lines

@@ -8,7 +8,7 @@
 class Component_select
 extends AbstractComponent
 implements ValueComponentInterface, 
-           DisableableComponentInterface
+            DisableableComponentInterface
 {
     public function default_model() 
     {

components/number/controller.php

Indentation +3 added lines, -3 removed lines

@@ -8,9 +8,9 @@
 class Component_number
 extends AbstractComponent
 implements ValueComponentInterface, 
-           DisableableComponentInterface,
-           FilterableComponentInterface,
-           ValidatableComponentInterface
+            DisableableComponentInterface,
+            FilterableComponentInterface,
+            ValidatableComponentInterface
 {
     public function default_model() 
     {

Spacing +5 added lines, -5 removed lines

@@ -25,8 +25,8 @@ 
             'required'      => false,
             'readonly'      => false,
             'default'       => null,
-            'filter'        => array( $this, 'filter' ),
-            'validation'    => array( $this, 'validation' )
+            'filter'        => array($this, 'filter'),
+            'validation'    => array($this, 'validation')
         );
     }
     
@@ -35,15 +35,15 @@ 
         return floatval($v);
     }
     
-    public function validation($v,&$e)
+    public function validation($v, &$e)
     {
-        if(null !== $this->max && $v > $this->max) 
+        if (null !== $this->max && $v > $this->max) 
         {
             $e = "must be less than {$this->max}";
             return false;
         }
 
-        if(null !== $this->min && $v < $this->min) 
+        if (null !== $this->min && $v < $this->min) 
         {
             $e = "must be greater than {$this->min}";
             return false;

components/checkbox/controller.php

Indentation +1 added lines, -1 removed lines

@@ -8,7 +8,7 @@
 class Component_checkbox
 extends AbstractComponent
 implements ValueComponentInterface, 
-           DisableableComponentInterface
+            DisableableComponentInterface
 {
     public function default_model() 
     {

Template.php

Spacing +11 added lines, -11 removed lines

@@ -25,9 +25,9 @@ 
      * @param array $model
      * @param string $template_path
      */
-    public function __construct( array $model = array(), $template_path = null ) 
+    public function __construct(array $model = array(), $template_path = null) 
     {
-        $this->set_model( $model );
+        $this->set_model($model);
         $this->template = $template_path;
     }
     
@@ -38,9 +38,9 @@ 
      * 
      * @return mixed the argument's value.
      */
-    public function __get( $name ) 
+    public function __get($name) 
     {
-        if( isset( $this->model[$name] ) )
+        if (isset($this->model[$name]))
         {
             return $this->model[$name];
         }
@@ -54,7 +54,7 @@ 
      * 
      * @return mixed the settings' argument value.
      */
-    public function __set( $name, $value )
+    public function __set($name, $value)
     {
         $this->model[$name] = $value;
     }
@@ -74,7 +74,7 @@ 
      * 
      * @return array
      */
-    public function set_model( $model )
+    public function set_model($model)
     {
         $this->model = $model;
     }
@@ -95,22 +95,22 @@ 
      * @return string The rendered template.
      * @throws TemplateNotFoundException Thrown if the template file cannot be found.
      */
-    public function render( $echo = false )
+    public function render($echo = false)
     {
         $rendered_html = '';
         
-        if( file_exists( $this->get_template_path() ) ) 
+        if (file_exists($this->get_template_path())) 
         {
             ob_start();
-            include( $this->get_template_path() );
+            include($this->get_template_path());
             $rendered_html = ob_get_clean();
         } 
         else 
         {
-            throw new \RuntimeException( "Error: cannot render HTML, template file not found at " . $this->get_template_path() );
+            throw new \RuntimeException("Error: cannot render HTML, template file not found at ".$this->get_template_path());
         }
         
-        if( !$echo )
+        if (!$echo)
         {
             return $rendered_html;
         }