helmut / forms

src/Form.php

Spacing +10 added lines, -10 removed lines

@@ -357,7 +357,7 @@ 
      */
     public function completed($name = null)
     {
-        if($this->submitted($name) && $this->valid()) {
+        if ($this->submitted($name) && $this->valid()) {
             $this->broadcast('completed');
             return true;
         }
@@ -497,7 +497,7 @@ 
         $errors = [];
 
         foreach ($this->fields as $field) {
-            foreach($field->errors() as $error) {
+            foreach ($field->errors() as $error) {
                 $errors[] = $error;
             }
         }
@@ -619,8 +619,8 @@ 
     {
         $config = $this->templateConfig($template);
 
-        if ( isset($config['plugins']) && is_array($config['plugins'])) {
-            foreach($config['plugins'] as $plugin) {
+        if (isset($config['plugins']) && is_array($config['plugins'])) {
+            foreach ($config['plugins'] as $plugin) {
                 $this->addPlugin($plugin);
             }
         }
@@ -636,8 +636,8 @@ 
     {
         $config = $this->templateConfig($template);
 
-        if ( isset($config['plugins']) && is_array($config['plugins'])) {
-            foreach($config['plugins'] as $plugin) {
+        if (isset($config['plugins']) && is_array($config['plugins'])) {
+            foreach ($config['plugins'] as $plugin) {
                 $this->removePlugin($plugin);
             }
         }
@@ -674,7 +674,7 @@ 
             foreach ($field->errors() as $error => $parameters) {
 
                 if ($error == 'userDefined') {
-                    foreach($parameters as $message) {
+                    foreach ($parameters as $message) {
                         $properties['errors'][] = ['error' => $message];
                     }
                 } else {
@@ -683,7 +683,7 @@ 
 
                     $message = $this->translate($error, $field);
 
-                    foreach($parameters as $key => $value) {
+                    foreach ($parameters as $key => $value) {
                         if (is_object($value) && method_exists($value, '__toString')) $value = (string) $value;
                         if (is_array($value)) $value = implode(', ', $value);
                         $message = str_replace('['.$key.']', $value, $message);
@@ -930,7 +930,7 @@ 
      */
     private function broadcast($event, $params = []) 
     {
-        foreach($this->plugins as $plugin) {
+        foreach ($this->plugins as $plugin) {
             $plugin->event($this, $event, $params);
         }
     }
@@ -1078,7 +1078,7 @@ 
     {
         if ( ! method_exists($this, $method))
         {
-            if ( $this->typeExists($method)) {
+            if ($this->typeExists($method)) {
                 return $this->addField($method, array_shift($parameters));
             }
         }

Braces +36 added lines, -12 removed lines

@@ -259,7 +259,9 @@ 
      */
     public function fields($names = null)
     {
-        if (is_null($names)) return $this->fields;
+        if (is_null($names)) {
+            return $this->fields;
+        }
 
         return array_filter($this->fields, function($field) use ($names) {
             return array_key_exists($field->name, $names);
@@ -442,7 +444,9 @@ 
      */
     public function addNamespaceForClass($class)
     {
-        if (is_object($class)) $class = get_class($class);
+        if (is_object($class)) {
+            $class = get_class($class);
+        }
 
         $reflector = new \ReflectionClass($class);
 
@@ -477,7 +481,9 @@ 
         $class = '\\Fields\\'.ucwords($class).'\\'.ucwords($class);
 
         foreach ($this->namespaces as $namespace) {
-            if (class_exists($namespace.$class)) return $namespace.$class;
+            if (class_exists($namespace.$class)) {
+                return $namespace.$class;
+            }
         }
     }
 
@@ -490,9 +496,13 @@ 
      */
     public function errors($name = null)
     {
-        if ($this->submitted()) $this->validate();
+        if ($this->submitted()) {
+            $this->validate();
+        }
 
-        if ( ! is_null($name)) $this->field($name)->errors();
+        if ( ! is_null($name)) {
+            $this->field($name)->errors();
+        }
 
         $errors = [];
 
@@ -684,8 +694,12 @@ 
                     $message = $this->translate($error, $field);
 
                     foreach($parameters as $key => $value) {
-                        if (is_object($value) && method_exists($value, '__toString')) $value = (string) $value;
-                        if (is_array($value)) $value = implode(', ', $value);
+                        if (is_object($value) && method_exists($value, '__toString')) {
+                            $value = (string) $value;
+                        }
+                        if (is_array($value)) {
+                            $value = implode(', ', $value);
+                        }
                         $message = str_replace('['.$key.']', $value, $message);
                     }
                     $properties['errors'][] = ['error' => $message];
@@ -705,7 +719,9 @@ 
      */
     public function renderField($field, $properties = null) 
     {   
-        if (is_null($properties)) $properties = $this->fieldProperties($field);
+        if (is_null($properties)) {
+            $properties = $this->fieldProperties($field);
+        }
 
         return $this->renderTemplate($field->type, $properties, $field);
     }
@@ -719,7 +735,9 @@ 
      */
     public function renderFieldErrors($field, $properties = null) 
     {   
-        if (is_null($properties)) $properties = $this->fieldProperties($field);
+        if (is_null($properties)) {
+            $properties = $this->fieldProperties($field);
+        }
 
         return $this->renderTemplate('errors', $properties, $field);
     }   
@@ -736,9 +754,13 @@ 
 
         $this->setValues();
 
-        if ($this->submitted()) $this->validate();
+        if ($this->submitted()) {
+            $this->validate();
+        }
 
-        if ( ! is_null($template)) $this->setTemplate($template);
+        if ( ! is_null($template)) {
+            $this->setTemplate($template);
+        }
 
         $properties = [];
         $properties['id'] = $this->id;
@@ -992,7 +1014,9 @@ 
      */
     public function pathForClass($class)
     {
-        if (is_object($class)) $class = get_class($class);
+        if (is_object($class)) {
+            $class = get_class($class);
+        }
 
         $reflector = new \ReflectionClass($class);
         $path = dirname($reflector->getFileName()); 

Doc Comments +2 added lines, -2 removed lines

@@ -455,7 +455,7 @@ 
      * Check if a type exists.
      *
      * @param  string  $type
-     * @return string
+     * @return boolean
      */
     public function typeExists($type)
     {
@@ -1070,7 +1070,7 @@ 
      *
      * @param  string  $method
      * @param  array  $parameters
-     * @return mixed
+     * @return Fields\Field|null
      */
     public function __call($method, $parameters)
     {

src/Request.php

Indentation +6 added lines, -6 removed lines

@@ -4,26 +4,26 @@
 
 interface Request {
 	
-	/**
+    /**
      * Fetch all values from the request.
      *
      * @return array
      */
-	public function all();
+    public function all();
 
-	/**
+    /**
      * Fetch a value from the request matching provided key.
      *
      * @var string
      * @return mixed
      */
-	public function get($key);
+    public function get($key);
 
-	/**
+    /**
      * Return array of csrf template properties.
      *
      * @return array
      */
-	public function csrf();	
+    public function csrf();	
 
 }
\ No newline at end of file

src/templates/foundation/config.php

Indentation +3 added lines, -3 removed lines

@@ -1,7 +1,7 @@
 <?php 
 
 return [
-	'plugins' => [
-		'feedback',
-	],
+    'plugins' => [
+        'feedback',
+    ],
 ];
\ No newline at end of file

src/Fields/Dropdown/Dropdown.php

Braces +9 added lines, -3 removed lines

@@ -25,12 +25,16 @@ 
 
     public function setValuesFromDefaults($defaults)
     {
-        if (count($defaults)) $this->value = array_shift($defaults);
+        if (count($defaults)) {
+            $this->value = array_shift($defaults);
+        }
     }
 
     public function setValuesFromModel($model)
     {
-        if (property_exists($model, $this->name)) $this->value = $model->{$this->name};
+        if (property_exists($model, $this->name)) {
+            $this->value = $model->{$this->name};
+        }
     }
 
     public function setValuesFromRequest($request)
@@ -40,7 +44,9 @@ 
 
     public function fillModelWithValues($model)
     {
-        if (property_exists($model, $this->name)) $model->{$this->name} = $this->value;
+        if (property_exists($model, $this->name)) {
+            $model->{$this->name} = $this->value;
+        }
     }   
 
     public function validateRequired()

src/Fields/Text/Text.php

Braces +9 added lines, -3 removed lines

@@ -25,12 +25,16 @@ 
 
     public function setValuesFromDefaults($defaults)
     {
-        if (count($defaults)) $this->value = array_shift($defaults);
+        if (count($defaults)) {
+            $this->value = array_shift($defaults);
+        }
     }
 
     public function setValuesFromModel($model)
     {
-        if (property_exists($model, $this->name)) $this->value = $model->{$this->name};
+        if (property_exists($model, $this->name)) {
+            $this->value = $model->{$this->name};
+        }
     }
 
     public function setValuesFromRequest($request)
@@ -40,7 +44,9 @@ 
 
     public function fillModelWithValues($model)
     {
-        if (property_exists($model, $this->name)) $model->{$this->name} = $this->value;
+        if (property_exists($model, $this->name)) {
+            $model->{$this->name} = $this->value;
+        }
     }   
 
     public function validateRequired()

src/Fields/ParagraphText/ParagraphText.php

Braces +9 added lines, -3 removed lines

@@ -25,12 +25,16 @@ 
 
     public function setValuesFromDefaults($defaults)
     {
-        if (count($defaults)) $this->value = array_shift($defaults);
+        if (count($defaults)) {
+            $this->value = array_shift($defaults);
+        }
     }
 
     public function setValuesFromModel($model)
     {
-        if (property_exists($model, $this->name)) $this->value = $model->{$this->name};
+        if (property_exists($model, $this->name)) {
+            $this->value = $model->{$this->name};
+        }
     }
 
     public function setValuesFromRequest($request)
@@ -40,7 +44,9 @@ 
 
     public function fillModelWithValues($model)
     {
-        if (property_exists($model, $this->name)) $model->{$this->name} = $this->value;
+        if (property_exists($model, $this->name)) {
+            $model->{$this->name} = $this->value;
+        }
     }   
 
     public function validateRequired()

src/Fields/Email/Email.php

Braces +9 added lines, -3 removed lines

@@ -25,12 +25,16 @@ 
 
     public function setValuesFromDefaults($defaults)
     {
-        if (count($defaults)) $this->value = array_shift($defaults);
+        if (count($defaults)) {
+            $this->value = array_shift($defaults);
+        }
     }
 
     public function setValuesFromModel($model)
     {
-        if (property_exists($model, $this->name)) $this->value = $model->{$this->name};
+        if (property_exists($model, $this->name)) {
+            $this->value = $model->{$this->name};
+        }
     }
 
     public function setValuesFromRequest($request)
@@ -40,7 +44,9 @@ 
 
     public function fillModelWithValues($model)
     {
-        if (property_exists($model, $this->name)) $model->{$this->name} = $this->value;
+        if (property_exists($model, $this->name)) {
+            $model->{$this->name} = $this->value;
+        }
     }   
 
     public function validateRequired()

src/Renderer.php

Indentation +98 added lines, -98 removed lines

@@ -4,140 +4,140 @@
 
 class Renderer {
 	
- 	/**
-     * Array of engine definitions using
-     * file extension as key.
-     * 
-     * @var array
-     */
-	protected $engines = [
-		'.mustache.php' => 'Helmut\Forms\Engines\Mustache',
-		'.twig.php' => 'Helmut\Forms\Engines\Twig',
-		'.blade.php' => 'Helmut\Forms\Engines\Blade',
-	];
-
- 	/**
-     * Cache of running engines.
-     *
-     * @var array
-     */
-	protected $running = [];
-
- 	/**
-     * Cache of template files.
-     *
-     * @var array
-     */
-	protected $templates = [];
-
-	/**
+        /**
+         * Array of engine definitions using
+         * file extension as key.
+         * 
+         * @var array
+         */
+    protected $engines = [
+        '.mustache.php' => 'Helmut\Forms\Engines\Mustache',
+        '.twig.php' => 'Helmut\Forms\Engines\Twig',
+        '.blade.php' => 'Helmut\Forms\Engines\Blade',
+    ];
+
+        /**
+         * Cache of running engines.
+         *
+         * @var array
+         */
+    protected $running = [];
+
+        /**
+         * Cache of template files.
+         *
+         * @var array
+         */
+    protected $templates = [];
+
+    /**
      * Add a new engine implementation.
      *
-	 * @param  string  $extension
-	 * @param  string  $class
+     * @param  string  $extension
+     * @param  string  $class
      * @return void
      */
-	public function addEngine($extension, $class)
-	{
-		array_unshift($this->engines, [$extension => $class]);
-	}
+    public function addEngine($extension, $class)
+    {
+        array_unshift($this->engines, [$extension => $class]);
+    }
 
-	/**
+    /**
      * Get an engine implementation.
      *
-	 * @param  string  $key
+     * @param  string  $key
      * @return \Helmut\Forms\Engine
      */
-	public function start($key)
-	{
-		if ( ! isset($this->running[$key])) {
-			$this->running[$key] = new $this->engines[$key];
-		}
+    public function start($key)
+    {
+        if ( ! isset($this->running[$key])) {
+            $this->running[$key] = new $this->engines[$key];
+        }
 
-		return $this->running[$key];
-	}
+        return $this->running[$key];
+    }
 
-	/**
+    /**
      * Get engine file extensions.
      *
      * @return array
      */
-	public function extensions()
-	{
-		return array_keys($this->engines);
-	}
+    public function extensions()
+    {
+        return array_keys($this->engines);
+    }
 
-	/**
+    /**
      * Create a template cache key.
      *
-	 * @param  string  $template
-	 * @param  array  $paths
+     * @param  string  $template
+     * @param  array  $paths
      * @return string
      */	
-	public function key($template, $paths) 
-	{
-		return $template.'-'.md5(serialize($paths));
-	}
+    public function key($template, $paths) 
+    {
+        return $template.'-'.md5(serialize($paths));
+    }
 
-	/**
+    /**
      * Render a template.
      *
-	 * @param  string  $template
-	 * @param  array  $properties
-	 * @param  array  $paths
+     * @param  string  $template
+     * @param  array  $properties
+     * @param  array  $paths
      * @return string
      */
-	public function render($template, $properties, $paths)
-	{
-		if ($this->has($template, $paths)) {
+    public function render($template, $properties, $paths)
+    {
+        if ($this->has($template, $paths)) {
 
-			$template = $this->findTemplate($template, $paths);
+            $template = $this->findTemplate($template, $paths);
 
-			return $this->start($template['engine'])->render($template['path'], $properties);
-		}
-	}
+            return $this->start($template['engine'])->render($template['path'], $properties);
+        }
+    }
 
-	/**
+    /**
      * Check if a template exists.
      *
-	 * @param  string  $template
-	 * @param  array  $paths
+     * @param  string  $template
+     * @param  array  $paths
      * @return boolean
      */	
-	public function has($template, $paths)
-	{
-		return ! is_null($this->findTemplate($template, $paths));
-	}
+    public function has($template, $paths)
+    {
+        return ! is_null($this->findTemplate($template, $paths));
+    }
 
-	/**
+    /**
      * Find template file with engine.
      *
-	 * @param  string  $template
-	 * @param  array  $paths
+     * @param  string  $template
+     * @param  array  $paths
      * @return array
      */	
-	public function findTemplate($template, $paths)
-	{
-		$key = $this->key($template, $paths);
-
-		if ( ! isset($this->templates[$key])) {
-
-			$this->templates[$key] = null;
-
-			$extensions = $this->extensions();
-			foreach ($paths as $path) {
-				foreach ($extensions as $extension) {
-					if (file_exists($path.$template.$extension)) {
-						return $this->templates[$key] = [
-							'engine' => $extension, 
-							'path' => $path.$template.$extension,
-						];
-					}
-				}
-			}
-		}
-
-		return $this->templates[$key];
-	}	
+    public function findTemplate($template, $paths)
+    {
+        $key = $this->key($template, $paths);
+
+        if ( ! isset($this->templates[$key])) {
+
+            $this->templates[$key] = null;
+
+            $extensions = $this->extensions();
+            foreach ($paths as $path) {
+                foreach ($extensions as $extension) {
+                    if (file_exists($path.$template.$extension)) {
+                        return $this->templates[$key] = [
+                            'engine' => $extension, 
+                            'path' => $path.$template.$extension,
+                        ];
+                    }
+                }
+            }
+        }
+
+        return $this->templates[$key];
+    }	
 
 }