amarkal / amarkal-ui

AbstractComponent.php

Doc Comments +1 added lines, -1 removed lines

@@ -89,7 +89,7 @@
     /**
      * Get the name for this component by parsing the name template.
      * 
-     * @return type
+     * @return string
      */
     public function get_name()
     {

components/composite/controller.php

Doc Comments +1 added lines, -1 removed lines

@@ -123,7 +123,7 @@
     /**
      * 
      * @param type $args
-     * @return type
+     * @return AbstractComponent
      */
     private function create_component( $args )
     {

Indentation +3 added lines, -3 removed lines

@@ -8,9 +8,9 @@
 class Component_composite
 extends AbstractComponent
 implements ValueComponentInterface, 
-           DisableableComponentInterface,
-           FilterableComponentInterface,
-           ValidatableComponentInterface
+            DisableableComponentInterface,
+            FilterableComponentInterface,
+            ValidatableComponentInterface
 {
     /**
      * The list of child components.

Spacing +13 added lines, -13 removed lines

@@ -23,11 +23,11 @@ 
      * The __set magic method is overridden here to apply value 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);
         }
@@ -38,9 +38,9 @@ 
      * 
      * @param array $value
      */
-    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;
@@ -69,7 +69,7 @@ 
      */
     public function required_arguments()
     {
-        return array('name','components','template');
+        return array('name', 'components', 'template');
     }
     
     /**
@@ -87,7 +87,7 @@ 
      */
     public function parse_template()
     {
-        return preg_replace_callback('/\{\{([a-zA-Z\d-_]+)\}\}/', function($a){
+        return preg_replace_callback('/\{\{([a-zA-Z\d-_]+)\}\}/', function($a) {
             $component = $this->get_component($a[1]);
             return $component->render();
         }, $this->model['template']);
@@ -102,7 +102,7 @@ 
      */
     public function get_name()
     {
-        if('' !== $this->parent_name)
+        if ('' !== $this->parent_name)
         {
             return "{$this->parent_name}[{$this->name}]";
         }
@@ -114,7 +114,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);
         }
@@ -125,16 +125,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);
@@ -149,9 +149,9 @@ 
      * @return UI\AbstractComponent
      * @throws \RuntimeException If there's no child component corresponding to the given name
      */
-    private function get_component( $name )
+    private 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");
         }

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 $name_template = '{{name}}[]';
     

components/number/controller.php

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,17 +35,17 @@ 
         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 = "must be less than {$max}";
         }
 
-        if(null !== $min && $v < $min) 
+        if (null !== $min && $v < $min) 
         {
             $e = "must be greater than {$min}";
         }