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

AbstractComponent.php

Spacing +15 added lines, -15 removed lines

@@ -33,7 +33,7 @@ 
      * 
      * @param array $model
      */
-    public function __construct( array $model = array() ) 
+    public function __construct(array $model = array()) 
     {
         parent::__construct($model);
         
@@ -74,24 +74,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']) )
+        if (!isset($model['id']))
         {
             $model['id'] = $model['name'];
         }
         
-        $this->model = array_merge( $this->default_model(), $model );
+        $this->model = array_merge($this->default_model(), $model);
     }
     
     /**
@@ -110,9 +110,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;
         }
@@ -124,14 +124,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++;
@@ -143,10 +143,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');
         }
@@ -169,7 +169,7 @@ 
      * 
      * {@inheritdoc}
      */
-    public function render( $echo = false )
+    public function render($echo = false)
     {
         $this->enqueue_scripts();
         return parent::render($echo);

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

Spacing +2 added lines, -2 removed lines

@@ -22,7 +22,7 @@ 
             'required'      => false,
             'readonly'      => false,
             'default'       => null,
-            'filter'        => array( $this, 'filter' )
+            'filter'        => array($this, 'filter')
         );
     }
     
@@ -47,7 +47,7 @@ 
      */
     public function filter($v)
     {
-        if($v !== 'on') return 'off';
+        if ($v !== 'on') return 'off';
         return 'on';
     }
 }
\ No newline at end of file

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

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