amarkal / amarkal-ui

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 $component_type = 'number';
     

Spacing +7 added lines, -7 removed lines

@@ -27,8 +27,8 @@ 
             'required'      => false,
             'readonly'      => false,
             'default'       => null,
-            'filter'        => array( $this, 'filter' ),
-            'validation'    => array( $this, 'validation' )
+            'filter'        => array($this, 'filter'),
+            'validation'    => array($this, 'validation')
         );
     }
     
@@ -37,19 +37,19 @@ 
         return floatval($v);
     }
     
-    public function validation($v,&$e)
+    public function validation($v, &$e)
     {
         $max = $this->max;
         $min = $this->min;
         
-        if(null !== $max && $v > $max)
+        if (null !== $max && $v > $max)
         {
-            $e = sprintf( __("Value must be less than %d",'amarkal'), $max);
+            $e = sprintf(__("Value must be less than %d", 'amarkal'), $max);
         }
 
-        if(null !== $min && $v < $min) 
+        if (null !== $min && $v < $min) 
         {
-            $e = sprintf( __("Value must be greater than %d",'amarkal'), $min);
+            $e = sprintf(__("Value must be greater than %d", 'amarkal'), $min);
         }
 
         return $e ? false : true;

components/text/controller.php

Indentation +3 added lines, -3 removed lines

@@ -8,9 +8,9 @@
 class Component_text
 extends AbstractComponent
 implements ValueComponentInterface, 
-           DisableableComponentInterface,
-           FilterableComponentInterface,
-           ValidatableComponentInterface
+            DisableableComponentInterface,
+            FilterableComponentInterface,
+            ValidatableComponentInterface
 {
     public $component_type = 'text';
     

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 $component_type = 'radio';
     

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

Indentation +2 added lines, -2 removed lines

@@ -8,8 +8,8 @@
 class Component_switch
 extends AbstractComponent
 implements ValueComponentInterface, 
-           DisableableComponentInterface,
-           FilterableComponentInterface
+            DisableableComponentInterface,
+            FilterableComponentInterface
 {
     public $component_type = 'switch';
     

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

Spacing +2 added lines, -2 removed lines

@@ -21,7 +21,7 @@ 
             'disabled'      => false,
             'readonly'      => false,
             'default'       => null,
-            'filter'        => array( $this, 'filter' )
+            'filter'        => array($this, 'filter')
         );
     }
     
@@ -46,7 +46,7 @@ 
      */
     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)
     {

Indentation +2 added lines, -2 removed lines

@@ -8,8 +8,8 @@
 class Component_slider
 extends AbstractComponent
 implements ValueComponentInterface, 
-           DisableableComponentInterface,
-           FilterableComponentInterface
+            DisableableComponentInterface,
+            FilterableComponentInterface
 {
     public $component_type = 'slider';
     

Spacing +2 added lines, -2 removed lines

@@ -25,13 +25,13 @@
             'min'           => null,
             'max'           => null,
             'step'          => 1,
-            'filter'        => array( $this, 'filter' )
+            'filter'        => array($this, 'filter')
         );
     }
     
     public function required_arguments()
     {
-        return array('name','min','max');
+        return array('name', 'min', 'max');
     }
     
     public function get_template_path() 

components/button/controller.php

Spacing +1 added lines, -1 removed lines

@@ -29,7 +29,7 @@
     
     public function required_arguments()
     {
-        return array('request_url','label_start');
+        return array('request_url', 'label_start');
     }
     
     public function get_template_path() 

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 )

Spacing +12 added lines, -12 removed lines

@@ -19,7 +19,7 @@ 
      *
      * @param array $components
      */
-    public function __construct( array $components = array() )
+    public function __construct(array $components = array())
     {
         $this->add_components($components);
     }
@@ -30,12 +30,12 @@ 
      * @param string $name
      * @return AbstractComponent
      */
-    public function get_by_name( $name )
+    public function get_by_name($name)
     {
         $filtered = $this->filter(function($c) use ($name) {
             return $c instanceof ValueComponentInterface && $c->name === $name;
         });
-        if(count($filtered) < 1)
+        if (count($filtered) < 1)
         {
             throw new \RuntimeException("No component with the name <b>$name</b> could be found.");
         }
@@ -48,7 +48,7 @@ 
      * @param string $type
      * @return AbstractComponent[]
      */
-    public function get_by_type( $type )
+    public function get_by_type($type)
     {
         return $this->filter(function($c) use ($type) {
             return $c->component_type === $type;
@@ -107,9 +107,9 @@ 
      * @param AbstractComponent[] $components
      * @return void
      */
-    public function add_components( array $components )
+    public function add_components(array $components)
     {
-        foreach( $components as $component )
+        foreach ($components as $component)
         {
             $this->add_component($component);
         }
@@ -121,9 +121,9 @@ 
      * @param AbstractComponent[] $args
      * @return void
      */
-    public function add_component( array $args )
+    public function add_component(array $args)
     {
-        if(!array_key_exists('type', $args))
+        if (!array_key_exists('type', $args))
         {
             throw new \RuntimeException("Component configuration arrays must have a <b>type</b> argument");
         }
@@ -140,16 +140,16 @@ 
      * @param AbstractComponent $comp
      * @return void
      */
-    private function verify_name_uniqueness( AbstractComponent $comp )
+    private function verify_name_uniqueness(AbstractComponent $comp)
     {
-        if(!($comp instanceof ValueComponentInterface))
+        if (!($comp instanceof ValueComponentInterface))
         {
             return;
         }
 
         foreach ($this->components as $c) 
         {
-            if($c->name === $comp->name)
+            if ($c->name === $comp->name)
             {
                 throw new \RuntimeException("Duplicate component name detected for the name <b>{$c->name}</b>.");
             }
@@ -162,7 +162,7 @@ 
      * @param function $callable
      * @return array
      */
-    private function filter( $callable )
+    private function filter($callable)
     {
         // array_values is needed in order to reindex the array after filtering it
         return array_values(array_filter($this->components, $callable));

components/toggle/controller.php

Indentation +2 added lines, -2 removed lines

@@ -8,8 +8,8 @@
 class Component_toggle
 extends AbstractComponent
 implements ValueComponentInterface,
-           FilterableComponentInterface,
-           DisableableComponentInterface
+            FilterableComponentInterface,
+            DisableableComponentInterface
 {
     public $component_type = 'toggle';
     

Spacing +6 added lines, -6 removed lines

@@ -15,7 +15,7 @@ 
     
     protected function on_created() 
     {
-        if($this->multi && !\is_array($this->default)
+        if ($this->multi && !\is_array($this->default)
         || !$this->multi && \is_array($this->default))
         {
             throw new \RuntimeException("The default value must be an array if multi is set to true, and must be a string otherwise.");
@@ -33,13 +33,13 @@ 
             'multi'         => false,
             'data'          => array(),
             'default'       => array(),
-            'filter'        => array( $this, 'filter' )
+            'filter'        => array($this, 'filter')
         );
     }
     
     public function required_arguments()
     {
-        return array('name','data');
+        return array('name', 'data');
     }
     
     public function get_template_path() 
@@ -58,16 +58,16 @@ 
      */
     public function filter($v)
     {
-        if($this->multi && !\is_array($v)) 
+        if ($this->multi && !\is_array($v)) 
         {
             return \explode(',', $v);
         }
         return $v;
     }
 
-    protected function is_selected( $value )
+    protected function is_selected($value)
     {
-        if($this->multi) 
+        if ($this->multi) 
         {
             return \in_array($value, $this->value);
         }

components/code/controller.php

Indentation +3 added lines, -3 removed lines

@@ -8,9 +8,9 @@
 class Component_code
 extends AbstractComponent
 implements ValueComponentInterface, 
-           DisableableComponentInterface,
-           FilterableComponentInterface,
-           ValidatableComponentInterface
+            DisableableComponentInterface,
+            FilterableComponentInterface,
+            ValidatableComponentInterface
 {
     public $component_type = 'code';
     

Spacing +1 added lines, -1 removed lines

@@ -32,7 +32,7 @@
     
     public function required_arguments()
     {
-        return array('name','language');
+        return array('name', 'language');
     }
     
     public function get_template_path()