iriven / PhpFormGenerator

Elements/Datalist.php

Braces +3 added lines, -1 removed lines

@@ -30,7 +30,9 @@
         $html .= '<datalist id="'. $this->listname.'">';
         foreach( $this->options as $option)
         {
-            if(!$option = trim(strip_tags($option))) continue;
+            if(!$option = trim(strip_tags($option))) {
+              continue;
+            }
             $html .= '<option value="'.$option.'">';
         }
         $html .= '</datalist>';

Elements/Checkbox.php

Braces +20 added lines, -13 removed lines

@@ -37,28 +37,33 @@ 
         $html = '';
         if($this->Types()->has($this->Attributes()->get('type')))
         {
-            if(!$this->Attributes()->has('id'))
-                $this->Attributes()->createElementID($this->Attributes()->get('name',$this->label()->getItemID()));
+            if(!$this->Attributes()->has('id')) {
+                            $this->Attributes()->createElementID($this->Attributes()->get('name',$this->label()->getItemID()));
+            }
             $this->label()->Attribute()->set('fieldtype',$this->Attributes()->get('type'));
             $this->label()->Attribute()->set('for',$this->Attributes()->get('id'));
             if(sizeof($this->options)<2)
             {
-                if($this->Attributes()->get('value'))
-                    $this->Attributes()->set('checked','checked');
+                if($this->Attributes()->get('value')) {
+                                    $this->Attributes()->set('checked','checked');
+                }
                 $this->Attributes()->set('value','1');
                 $this->hiddenInput = new Hidden($this->Label(),['value'=>'0']);
             }
-            if($this->hiddenInput instanceof FormElementInterface)
-                return  parent::RenderHtml().$this->hiddenInput->RenderHtml();
+            if($this->hiddenInput instanceof FormElementInterface) {
+                            return  parent::RenderHtml().$this->hiddenInput->RenderHtml();
+            }
             if($this->Attributes()->has('value'))
             {
-                if(!is_array($this->Attributes()->get('value')))
-                    $this->Attributes()->set('value',[$this->Attributes()->get('value')]);
+                if(!is_array($this->Attributes()->get('value'))) {
+                                    $this->Attributes()->set('value',[$this->Attributes()->get('value')]);
+                }
+            } else {
+                            $this->Attributes()->set('value',[]);
+            }
+            if(substr($this->Attributes()->get('name'), -2) !== '[]') {
+                            $this->Attributes()->set('name',$this->Attributes()->get('name').'[]');
             }
-            else
-                $this->Attributes()->set('value',[]);
-            if(substr($this->Attributes()->get('name'), -2) !== '[]')
-                $this->Attributes()->set('name',$this->Attributes()->get('name').'[]');
             $count = 0;
             $html .= $this->Label()->RenderHtml();
             $this->Attributes()->Ignore(['id', 'value', 'checked', 'required']);
@@ -68,7 +73,9 @@ 
                 $html .= '<input id="'.$this->Attributes()->get('id').'-'.$count.'"';
                 $html .= ' value="'.$index.'"';
                 $html .= $this->Attributes()->RenderHtml();
-                if(in_array($index , $this->Attributes()->get('value'))) $html .= ' checked="checked"';
+                if(in_array($index , $this->Attributes()->get('value'))) {
+                  $html .= ' checked="checked"';
+                }
                 $html .= ' >'.$details.'</label>';
                 ++$count;
             }

Elements/Number.php

Braces +9 added lines, -6 removed lines

@@ -19,14 +19,17 @@
 
         $this->attributes->set('type','number');
 
-        if(!$this->attributes->has('min'))
-            $this->attributes->set('min','0');
+        if(!$this->attributes->has('min')) {
+                    $this->attributes->set('min','0');
+        }
 
-        if(!$this->attributes->has('max'))
-            $this->attributes->set('max','100');
+        if(!$this->attributes->has('max')) {
+                    $this->attributes->set('max','100');
+        }
 
-        if(!$this->attributes->has('step'))
-            $this->attributes->set('step','1');
+        if(!$this->attributes->has('step')) {
+                    $this->attributes->set('step','1');
+        }
     }
 
 }
\ No newline at end of file

Elements/Select.php

Braces +32 added lines, -21 removed lines

@@ -37,40 +37,45 @@ 
         $html = '';
         $multiple=false;
 
-        if (!$this->Attributes()->has('id'))
-            $this->Attributes()->createElementID($this->Attributes()->get('name', $this->label()->getItemID()));
+        if (!$this->Attributes()->has('id')) {
+                    $this->Attributes()->createElementID($this->Attributes()->get('name', $this->label()->getItemID()));
+        }
         $this->label()->Attribute()->set('fieldtype', $this->Attributes()->get('type'));
         $this->label()->Attribute()->set('for', $this->Attributes()->get('id'));
 
 
         if($this->Attributes()->has('value'))
         {
-            if(!is_array($this->Attributes()->get('value')))
-                $this->Attributes()->set('value',[$this->Attributes()->get('value')]);
+            if(!is_array($this->Attributes()->get('value'))) {
+                            $this->Attributes()->set('value',[$this->Attributes()->get('value')]);
+            }
+        } else {
+                    $this->Attributes()->set('value',[]);
         }
-        else
-            $this->Attributes()->set('value',[]);
 
-        if($this->Attributes()->has('multiple') AND $this->Attributes()->get('multiple')==='multiple')
-            $multiple=true;
+        if($this->Attributes()->has('multiple') AND $this->Attributes()->get('multiple')==='multiple') {
+                    $multiple=true;
+        }
 
         if($multiple)
         {   $this->Attributes()->set('size',$this->Attributes()->get('size',3));
-            if(substr($this->Attributes()->get('name'), -2) !== '[]')
-                $this->Attributes()->set('name',$this->Attributes()->get('name').'[]');
+            if(substr($this->Attributes()->get('name'), -2) !== '[]') {
+                            $this->Attributes()->set('name',$this->Attributes()->get('name').'[]');
+            }
         }
         $html .= $this->Label()->RenderHtml();
         $this->Attributes()->Ignore(['value','selected','optgroup-attributes','option-attributes','placeholder']);
 
         $html .= '<select'.$this->Attributes()->RenderHtml().'>';
-        if(empty($this->Attributes()->get('value')[0]) and $this->Attributes()->has('placeholder'))
-            $html .= '<option value="" disabled selected>'.$this->Attributes()->get('placeholder').'</option>';
+        if(empty($this->Attributes()->get('value')[0]) and $this->Attributes()->has('placeholder')) {
+                    $html .= '<option value="" disabled selected>'.$this->Attributes()->get('placeholder').'</option>';
+        }
 
         foreach($this->options as $index=>$datas):
 
-            if(is_array($datas))
-                $html .= $this->createOptgroup($index,$datas);
-            else
+            if(is_array($datas)) {
+                            $html .= $this->createOptgroup($index,$datas);
+            } else
             {
                 $optionAttr = null;
                 if($this->Attributes()->has('option-attributes'))
@@ -86,7 +91,9 @@ 
                     $html .= ' selected="selected"';
                     $this->selected = true;
                 }
-                if($optionAttr) $html .= ' '.$optionAttr ;
+                if($optionAttr) {
+                  $html .= ' '.$optionAttr ;
+                }
                 $html .= '>';
                 $html .= $datas.'</option>';
             }
@@ -119,13 +126,15 @@ 
             $optionAttr .= $oOption->RenderHtml();
         }
         $output='<optgroup label="'.$label.'"';
-        if($groupAttr) $output .= ' '.$groupAttr ;
+        if($groupAttr) {
+          $output .= ' '.$groupAttr ;
+        }
         $output .= '>';
         foreach($options as $key=>$optLabel)
         {
-            if(is_array($optLabel))
-                $output .= call_user_func_array([$this,__METHOD__],[$optLabel]);
-            else
+            if(is_array($optLabel)) {
+                            $output .= call_user_func_array([$this,__METHOD__],[$optLabel]);
+            } else
             {
                 $output .= '<option value="'.$key.'"';
                 if(!$this->selected and in_array($key,$this->Attributes()->get('value')))
@@ -133,7 +142,9 @@ 
                     $output .= ' selected="selected"';
                     $this->selected = true;
                 }
-                if($optionAttr) $output .= ' '.$optionAttr ;
+                if($optionAttr) {
+                  $output .= ' '.$optionAttr ;
+                }
                 $output .= '>';
                 $output .= $optLabel.'</option>';
             }

Elements/Textarea.php

Braces +3 added lines, -2 removed lines

@@ -26,8 +26,9 @@
      */
     public function RenderHtml(): string
     {
-        if (!$this->Attributes()->has('id'))
-            $this->Attributes()->createElementID($this->Attributes()->get('name', $this->label()->getItemID()));
+        if (!$this->Attributes()->has('id')) {
+                    $this->Attributes()->createElementID($this->Attributes()->get('name', $this->label()->getItemID()));
+        }
         $this->label()->Attribute()->set('fieldtype', $this->Attributes()->get('type'));
         $this->label()->Attribute()->set('for', $this->Attributes()->get('id'));
 

FormGenerator.php

Braces +18 added lines, -12 removed lines

@@ -267,8 +267,9 @@ 
     {
         if($this->Form instanceof form)
         {
-            if($this->Fieldset instanceof fieldsetGenerator)
-                $this->Form->append($this->Fieldset->Close());
+            if($this->Fieldset instanceof fieldsetGenerator) {
+                            $this->Form->append($this->Fieldset->Close());
+            }
             $this->Fieldset = new fieldsetGenerator($attributes);
             $this->Fieldset->Attributes()->set('name', $this->Form->Attributes()->get('name'));
             $this->Form->append($this->Fieldset->Open());
@@ -316,8 +317,9 @@ 
     {
         if($this->Form instanceof form)
         {
-            if(gettype($html) === 'string')
-            $this->Form->append($html);
+            if(gettype($html) === 'string') {
+                        $this->Form->append($html);
+            }
         }
         return $this;
     }
@@ -617,12 +619,14 @@ 
         $html = null;
         if ($this->Form instanceof form)
         {
-            if($this->Fieldset instanceof fieldsetGenerator)
-                $this->Form->append($this->Fieldset->Close());
+            if($this->Fieldset instanceof fieldsetGenerator) {
+                            $this->Form->append($this->Fieldset->Close());
+            }
             $html = $this->Form->RenderHtml();
             $this->Form = null;
-            if($print)
-                print_r($html);
+            if($print) {
+                            print_r($html);
+            }
         }
         return $html;
     }
@@ -653,8 +657,9 @@ 
                 $Request = $this->Request->query; 
             break;
         }
-        if(!$value = $Request->get($name, $default))
-        $value = $element->Attributes()->get('value', $default);
+        if(!$value = $Request->get($name, $default)) {
+                $value = $element->Attributes()->get('value', $default);
+        }
         return $value;
     }
     /**
@@ -663,8 +668,9 @@ 
      */
     public function Open($attributes = [])
     {
-        if($this->Form instanceof form)
-            $this->Close();
+        if($this->Form instanceof form) {
+                    $this->Close();
+        }
         $this->Form = new form($attributes);
         return $this;
     }

Core/FieldsetGenerator.php

Braces +9 added lines, -6 removed lines

@@ -25,8 +25,9 @@ 
      */
     public function __construct($attributes)
     {
-        if(!$attributes instanceof AttributesBuilder)
-            $attributes  = new AttributesBuilder($attributes);
+        if(!$attributes instanceof AttributesBuilder) {
+                    $attributes  = new AttributesBuilder($attributes);
+        }
         $this->attributes = $attributes;
         if($this->attributes->has('legend'))
         {
@@ -58,8 +59,9 @@ 
         if($this->legend)
         {
             $html .= '<legend';
-            if($this->legendAttributes instanceof AttributesBuilder)
-            $html .= $this->legendAttributes->RenderHtml();
+            if($this->legendAttributes instanceof AttributesBuilder) {
+                        $html .= $this->legendAttributes->RenderHtml();
+            }
             $html .= '>';
             $html .= $this->legend;
             $html .= '</legend>';
@@ -74,8 +76,9 @@ 
     public function Close()
     {
         $html  = '';
-        if($this->isOpened)
-            $html .= '</fieldset>';
+        if($this->isOpened) {
+                    $html .= '</fieldset>';
+        }
         return $html;
     }
 

Core/Libs/LabelGenerator.php

Braces +3 added lines, -2 removed lines

@@ -27,8 +27,9 @@
     public function __construct(string $item, $attributes=[])
     {
         $this->item = $item;
-        if(!$attributes instanceof AttributesBuilder)
-            $attributes  = new AttributesBuilder($attributes);
+        if(!$attributes instanceof AttributesBuilder) {
+                    $attributes  = new AttributesBuilder($attributes);
+        }
         $this->attributes = $attributes;
         $this->attributes->set('type','label');
             return $this;

Core/Libs/AttributesMapper.php

Braces +12 added lines, -4 removed lines

@@ -48,7 +48,9 @@ 
             case 'number':
             case 'time':
             case 'week':
-                if($Type === 'number') $AcceptedAttributes[]='placeholder';
+                if($Type === 'number') {
+                  $AcceptedAttributes[]='placeholder';
+                }
                 $AcceptedAttributes = array_merge($AcceptedAttributes,self::DATE);
                 break;
             case 'color':
@@ -67,7 +69,9 @@ 
                 if($Type === 'radio')
                 {
                     $bannedIndex = array_search('indeterminate', $tmpAttributesList,true);
-                    if(isset($tmpAttributesList[$bannedIndex])) unset($tmpAttributesList[$bannedIndex]);
+                    if(isset($tmpAttributesList[$bannedIndex])) {
+                      unset($tmpAttributesList[$bannedIndex]);
+                    }
                 }
                 $AcceptedAttributes = array_merge($AcceptedAttributes,array_values($tmpAttributesList));
                 break;
@@ -109,7 +113,9 @@ 
                 $AcceptedAttributes = array_merge($AcceptedAttributes,self::HIDDEN);
                 break;
             default:
-                if($Type === 'email') $AcceptedAttributes[]='multiple';
+                if($Type === 'email') {
+                  $AcceptedAttributes[]='multiple';
+                }
                 $AcceptedAttributes = array_merge($AcceptedAttributes,self::TEXT);
                 break;
         endswitch;
@@ -132,7 +138,9 @@ 
                 continue;
             }
         }
-        if($ignore) $Attributes->Ignore($ignore);
+        if($ignore) {
+          $Attributes->Ignore($ignore);
+        }
         return $ignore;
     }
 }
\ No newline at end of file