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)
     {
         return array_filter($this->components, $callable);
     }

AbstractComponent.php

Spacing +16 added lines, -16 removed lines

@@ -38,7 +38,7 @@ 
      * 
      * @param array $model
      */
-    public function __construct( array $model = array() ) 
+    public function __construct(array $model = array()) 
     {
         parent::__construct($model);
         
@@ -79,24 +79,24 @@ 
      * 
      * @return array
      */
-    public function set_model( $model )
+    public function set_model($model)
     {
         // Check that the required arguments are provided.
-        foreach( $this->required_arguments() as $key )
+        foreach ($this->required_arguments() as $key)
         {
-            if ( !isset($model[$key]) )
+            if (!isset($model[$key]))
             {
                 throw new \RuntimeException('The required argument "'.$key.'" was not provided for '.get_called_class());
             }
         }
         
         // Assign the name of the component as the id if no id was specified
-        if( !isset($model['id']) && isset($model['name']) )
+        if (!isset($model['id']) && isset($model['name']))
         {
             $model['id'] = $model['name'];
         }
         
-        $this->model = array_merge( $this->default_model(), $model );
+        $this->model = array_merge($this->default_model(), $model);
     }
     
     /**
@@ -115,9 +115,9 @@ 
      * 
      * @param string $class
      */
-    public function add_html_class( $class )
+    public function add_html_class($class)
     {
-        if( !in_array($class, $this->html_classes) )
+        if (!in_array($class, $this->html_classes))
         {
             $this->html_classes[] = $class;
         }
@@ -129,14 +129,14 @@ 
      * 
      * @param string $class
      */
-    public function remove_html_class( $class )
+    public function remove_html_class($class)
     {
         $i = 0;
-        foreach( $this->html_classes as $c )
+        foreach ($this->html_classes as $c)
         {
-            if( $c === $class )
+            if ($c === $class)
             {
-                array_splice($this->html_classes,$i,1);
+                array_splice($this->html_classes, $i, 1);
                 break;
             }
             $i++;
@@ -148,10 +148,10 @@ 
      * 
      * @param type $validity
      */
-    Public function set_validity( $validity )
+    Public function set_validity($validity)
     {
         $this->validity = $validity;
-        if($validity === $this::INVALID)
+        if ($validity === $this::INVALID)
         {
             $this->add_html_class('amarkal-ui-component-error');
         }
@@ -174,7 +174,7 @@ 
      * 
      * {@inheritdoc}
      */
-    public function render( $echo = false )
+    public function render($echo = false)
     {
         $this->enqueue_scripts();
         
@@ -182,7 +182,7 @@ 
         include dirname(__FILE__).'/AbstractComponent.phtml';
         $html = ob_get_clean();
 
-        if( !$echo )
+        if (!$echo)
         {
             return $html;
         }

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

@@ -24,13 +24,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() 
@@ -49,16 +49,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);
         }