helmut / forms

src/Field.php

Doc Comments -1 removed lines

@@ -113,7 +113,6 @@
     /**
      * Set field value using provided default.
      *
-     * @param mixed  $default
      * @return void
      */
     abstract public function setValueFromDefault();

Unused Use Statements +1 added lines, -1 removed lines

@@ -2,8 +2,8 @@
 
 namespace Helmut\Forms;
 
-use Helmut\Forms\Utility\Str;
 use Helmut\Forms\Utility\Reflect;
+use Helmut\Forms\Utility\Str;
 
 abstract class Field {
 

Braces +22 added lines, -10 removed lines

@@ -179,9 +179,11 @@ 
     {
         $values = $this->getValue();
 
-        if (is_null($values)) $values = [];
-
-        else if ( ! is_array($values)) $values = [$this->name => $values];
+        if (is_null($values)) {
+            $values = [];
+        } else if ( ! is_array($values)) {
+            $values = [$this->name => $values];
+        }
 
         return $values;
     }
@@ -196,7 +198,9 @@ 
     {
         $properties = $this->renderWith();
 
-        if (is_null($properties)) $properties = [];
+        if (is_null($properties)) {
+            $properties = [];
+        }
 
         return $properties;
     }
@@ -210,9 +214,11 @@ 
     {
         $buttons = $this->getButtonName();
 
-        if (is_null($buttons)) $buttons = [];
-
-        else if ( ! is_array($buttons)) $buttons = [$buttons];
+        if (is_null($buttons)) {
+            $buttons = [];
+        } else if ( ! is_array($buttons)) {
+            $buttons = [$buttons];
+        }
 
         return $buttons;
     }
@@ -224,7 +230,9 @@ 
      */ 
     public function setFromDefault()
     {
-        if ( ! is_null($this->default)) $this->setValueFromDefault();
+        if ( ! is_null($this->default)) {
+            $this->setValueFromDefault();
+        }
     }
 
     /**
@@ -369,7 +377,9 @@ 
             return false;
         }
 
-        if (isset($this->errors[$method])) unset($this->errors[$method]);
+        if (isset($this->errors[$method])) {
+            unset($this->errors[$method]);
+        }
 
         return true;
     }
@@ -411,7 +421,9 @@ 
      */
     public function error($message)
     {
-        if ( ! isset($this->errors['userDefined'])) $this->errors['userDefined'] = [];
+        if ( ! isset($this->errors['userDefined'])) {
+            $this->errors['userDefined'] = [];
+        }
 
         $this->errors['userDefined'][] = $message;
     }

src/Fields/Checkbox/Checkbox.php

Indentation +4 added lines, -4 removed lines

@@ -25,14 +25,14 @@
 
     public function setChecked()
     {
-         $this->default = true;
-         return $this;
+            $this->default = true;
+            return $this;
     }
 
     public function setUnchecked()
     {
-         $this->default = false;
-         return $this;
+            $this->default = false;
+            return $this;
     }   
 
     public function setValueFromDefault()

src/Fields/Checkboxes/Checkboxes.php

Braces +12 added lines, -4 removed lines

@@ -84,14 +84,18 @@ 
     public function setValueFromDefault()
     {
         foreach (array_keys($this->options) as $key) {
-            if (isset($this->default[$key])) $this->values[$key] = $this->default[$key] ? true : false;
+            if (isset($this->default[$key])) {
+                $this->values[$key] = $this->default[$key] ? true : false;
+            }
         }
     }
 
     public function setValueFromModel($model)
     {
         foreach (array_keys($this->options) as $key) {
-            if (property_exists($model, $this->name.'_'.$key)) $this->values[$key] = $model->{$this->name.'_'.$key} ? true : false;
+            if (property_exists($model, $this->name.'_'.$key)) {
+                $this->values[$key] = $model->{$this->name.'_'.$key} ? true : false;
+            }
         }
 
     }   
@@ -106,7 +110,9 @@ 
     public function fillModelWithValue($model)
     {
         foreach (array_keys($this->options) as $key) {
-            if (property_exists($model, $this->name.'_'.$key)) $model->{$this->name.'_'.$key} = $this->values[$key] ? 1 : 0;
+            if (property_exists($model, $this->name.'_'.$key)) {
+                $model->{$this->name.'_'.$key} = $this->values[$key] ? 1 : 0;
+            }
         }
     }
 
@@ -115,7 +121,9 @@ 
         $checked = false;
 
         foreach (array_keys($this->options) as $key) {
-            if ($this->values[$key]) $checked = true;
+            if ($this->values[$key]) {
+                $checked = true;
+            }
         }
 
         return $checked;

src/Fields/Dropdown/Dropdown.php

Braces +6 added lines, -2 removed lines

@@ -52,7 +52,9 @@ 
 
     public function setValueFromModel($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 setValueFromRequest($request)
@@ -62,7 +64,9 @@ 
 
     public function fillModelWithValue($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 +6 added lines, -2 removed lines

@@ -31,7 +31,9 @@ 
 
     public function setValueFromModel($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 setValueFromRequest($request)
@@ -41,7 +43,9 @@ 
 
     public function fillModelWithValue($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 validate()

src/Fields/Name/Name.php

Braces +21 added lines, -7 removed lines

@@ -38,15 +38,23 @@ 
 
     public function setValueFromDefault()
     {
-        if (isset($this->default['first'])) $this->value_first = $this->default['first'];
-        if (isset($this->default['surname'])) $this->value_surname = $this->default['surname'];
+        if (isset($this->default['first'])) {
+            $this->value_first = $this->default['first'];
+        }
+        if (isset($this->default['surname'])) {
+            $this->value_surname = $this->default['surname'];
+        }
         $this->value = trim($this->value_first.' '.$this->value_surname);
     }
 
     public function setValueFromModel($model)
     {
-        if (property_exists($model, $this->name.'_first')) $this->value_first = $model->{$this->name.'_first'};
-        if (property_exists($model, $this->name.'_surname')) $this->value_surname = $model->{$this->name.'_surname'};
+        if (property_exists($model, $this->name.'_first')) {
+            $this->value_first = $model->{$this->name.'_first'};
+        }
+        if (property_exists($model, $this->name.'_surname')) {
+            $this->value_surname = $model->{$this->name.'_surname'};
+        }
         $this->value = trim($this->value_first.' '.$this->value_surname);
     }
 
@@ -59,9 +67,15 @@ 
 
     public function fillModelWithValue($model)
     {
-        if (property_exists($model, $this->name.'_first')) $model->{$this->name.'_first'} = $this->value_first;
-        if (property_exists($model, $this->name.'_surname')) $model->{$this->name.'_surname'} = $this->value_surname;
-        if (property_exists($model, $this->name)) $model->{$this->name} = $this->value;
+        if (property_exists($model, $this->name.'_first')) {
+            $model->{$this->name.'_first'} = $this->value_first;
+        }
+        if (property_exists($model, $this->name.'_surname')) {
+            $model->{$this->name.'_surname'} = $this->value_surname;
+        }
+        if (property_exists($model, $this->name)) {
+            $model->{$this->name} = $this->value;
+        }
     }
 
     public function validateRequired()

src/Fields/Number/Number.php

Braces +6 added lines, -2 removed lines

@@ -41,7 +41,9 @@ 
 
     public function setValueFromModel($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 setValueFromRequest($request)
@@ -51,7 +53,9 @@ 
 
     public function fillModelWithValue($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 validate()

src/Fields/ParagraphText/ParagraphText.php

Braces +6 added lines, -2 removed lines

@@ -31,7 +31,9 @@ 
 
     public function setValueFromModel($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 setValueFromRequest($request)
@@ -41,7 +43,9 @@ 
 
     public function fillModelWithValue($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/Search/Search.php

Braces +6 added lines, -2 removed lines

@@ -31,7 +31,9 @@ 
 
     public function setValueFromModel($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 setValueFromRequest($request)
@@ -41,7 +43,9 @@ 
 
     public function fillModelWithValue($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()