amarkal / amarkal-ui

Template.php

Spacing +12 added lines, -12 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;
     }
@@ -65,7 +65,7 @@ 
      * @param string $name The argument's name.
      * @return boolean Whether this arguments exists or not.
      */
-    public function __isset( $name )
+    public function __isset($name)
     {
         return isset($this->model[$name]);
     }
@@ -85,7 +85,7 @@ 
      * 
      * @return array
      */
-    public function set_model( $model )
+    public function set_model($model)
     {
         $this->model = $model;
     }
@@ -106,22 +106,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;
         }

components/composite/controller.php

Spacing +21 added lines, -21 removed lines

@@ -25,16 +25,16 @@ 
      * The __set magic method is overridden here to apply value & name changes to 
      * child components.
      */
-    public function __set( $name, $value )
+    public function __set($name, $value)
     {
         parent::__set($name, $value);
         
-        if( 'value' === $name )
+        if ('value' === $name)
         {
             $this->set_value($value);
         }
 
-        if( 'name' === $name )
+        if ('name' === $name)
         {
             $this->set_name($value);
         }
@@ -46,9 +46,9 @@ 
      * @param array $value
      * @return void
      */
-    public function set_value( array $value )
+    public function set_value(array $value)
     {
-        foreach($value as $n => $v)
+        foreach ($value as $n => $v)
         {
             $component = $this->get_component($n);
             $component->value = $v;
@@ -61,9 +61,9 @@ 
      * @param string $name
      * @return void
      */
-    public function set_name( $name )
+    public function set_name($name)
     {
-        foreach($this->components as $c)
+        foreach ($this->components as $c)
         {
             $c->name_template = str_replace('{{parent_name}}', $this->get_name(), $c->composite_name_template);
         }
@@ -92,7 +92,7 @@ 
      */
     public function required_arguments()
     {
-        return array('name','components','template');
+        return array('name', 'components', 'template');
     }
     
     /**
@@ -126,7 +126,7 @@ 
      */
     public function get_name()
     {
-        if('' !== $this->parent_name)
+        if ('' !== $this->parent_name)
         {
             return "{$this->parent_name}[{$this->name}]";
         }
@@ -135,9 +135,9 @@ 
 
     public function filter($v)
     {
-        foreach($this->components as $component)
+        foreach ($this->components as $component)
         {
-            if($component instanceof FilterableComponentInterface &&
+            if ($component instanceof FilterableComponentInterface &&
                \is_callable($component->filter))
             {
                 $n = $component->name;
@@ -148,15 +148,15 @@ 
         return $v;
     }
 
-    public function validation($v,&$e)
+    public function validation($v, &$e)
     {
-        foreach($this->components as $component)
+        foreach ($this->components as $component)
         {
-            if($component instanceof ValidatableComponentInterface &&
+            if ($component instanceof ValidatableComponentInterface &&
                \is_callable($component->validation))
             {
                 $n = $component->name;
-                if(!$component->validation($v[$n],$e))
+                if (!$component->validation($v[$n], $e))
                 {
                     return false;
                 }
@@ -172,9 +172,9 @@ 
      * @return UI\AbstractComponent
      * @throws \RuntimeException If there's no child component corresponding to the given name
      */
-    public function get_component( $name )
+    public function get_component($name)
     {
-        if(!array_key_exists($name, $this->components))
+        if (!array_key_exists($name, $this->components))
         {
             throw new \RuntimeException("Composite sub-component not found with name $name");
         }
@@ -186,7 +186,7 @@ 
      */
     protected function on_created()
     {
-        foreach( $this->model['components'] as $args )
+        foreach ($this->model['components'] as $args)
         {
             $this->components[$args['name']] = $this->create_component($args);
         }
@@ -197,16 +197,16 @@ 
      * @param type $args
      * @return type
      */
-    private function create_component( $args )
+    private function create_component($args)
     {
         $type = $args['type'];
         
-        if('composite' === $type)
+        if ('composite' === $type)
         {
             $args['parent_name'] = $this->get_name();
         }
         
-        $c = \Amarkal\UI\ComponentFactory::create( $type, $args );
+        $c = \Amarkal\UI\ComponentFactory::create($type, $args);
         
         // Apply the composite name template
         $c->name_template = str_replace('{{parent_name}}', $this->get_name(), $c->composite_name_template);

AbstractComponent.php

Spacing +18 added lines, -18 removed lines

@@ -38,7 +38,7 @@ 
      * 
      * @param array $model
      */
-    public function __construct( array $model = array() ) 
+    public function __construct(array $model = array()) 
     {
         parent::__construct($model);
         
@@ -47,7 +47,7 @@ 
             $this->component_type
         ));
 
-        if($this instanceof DisableableComponentInterface && $this->disabled)
+        if ($this instanceof DisableableComponentInterface && $this->disabled)
         {
             $this->add_html_class('amarkal-ui-disabled');
         }
@@ -84,30 +84,30 @@ 
      * 
      * @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'];
         }
 
         // A name must be specified when a component has a visibility condition
-        if( isset($model['show']) && !isset($model['name']) )
+        if (isset($model['show']) && !isset($model['name']))
         {
             throw new \RuntimeException('Components with a visibility condition (a "show" argument) must have a name');
         }
         
-        $this->model = array_merge( $this->default_model(), $model );
+        $this->model = array_merge($this->default_model(), $model);
     }
     
     /**
@@ -126,9 +126,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;
         }
@@ -140,14 +140,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++;
@@ -159,10 +159,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');
         }
@@ -185,7 +185,7 @@ 
      * 
      * {@inheritdoc}
      */
-    public function render( $echo = false )
+    public function render($echo = false)
     {
         $this->enqueue_scripts();
         
@@ -193,7 +193,7 @@ 
         include dirname(__FILE__).'/AbstractComponent.phtml';
         $html = ob_get_clean();
 
-        if( !$echo )
+        if (!$echo)
         {
             return $html;
         }

components/switch/controller.php

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'),
             'show'          => null
         );
     }
@@ -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

components/toggle/controller.php

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,14 +33,14 @@ 
             'multi'         => false,
             'data'          => array(),
             'default'       => array(),
-            'filter'        => array( $this, 'filter' ),
+            'filter'        => array($this, 'filter'),
             'show'          => null
         );
     }
     
     public function required_arguments()
     {
-        return array('name','data');
+        return array('name', 'data');
     }
     
     public function get_template_path() 
@@ -59,16 +59,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/slider/controller.php

Spacing +2 added lines, -2 removed lines

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

components/number/controller.php

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'),
             'show'          => null
         );
     }
@@ -38,19 +38,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;