Fix: combination render (#31)

* fix: render form fields with uppercase in type * fix: use field->GetCombinationChildren to (default) render combination fields * fix: lint
NotFoundNL · Jun 19, 2023 · 2aad4c9 · 2aad4c9
1 parent f2415f6
commit 2aad4c9
Show file tree

Hide file tree

Showing 4 changed files with 18 additions and 33 deletions.
diff --git a/resources/views/siteboss/forms/fields/combination.blade.php b/resources/views/siteboss/forms/fields/combination.blade.php
@@ -1,26 +1,10 @@
-<div class="col-sm-12">
-    <div class="row">
-        {% for item in item.fields %}
-            {{ include(item.type|lower ~ '.html', {label: item.label, properties: item.properties, id: item.id }) }}
-        {% endfor %}
-    </div>
-</div>
-
-{% if item.triggerId != '' %}
-<script>
-    $(document).ready(function() {
-        $("[name='{{ item.triggerId }}']" ).change(function (e){
-            thisCombo = $("#{{ item.id }}");
-            if(e.target.value == '{{ item.triggerValue }}') {
-                thisCombo.show();
-            } else {
-                thisCombo.hide();
-                //thisCombo.find('input').val('');
-                //thisCombo.find('textarea').val('');
-                //thisCombo.find("input:checked[type='radio']").prop('checked', false);
-                //thisCombo.find("input:checked[type='checkbox']").prop('checked', false);
-            }
-        });
-    })
-</script>
-{% endif %}
+<div class="row">
+    @foreach ($field->GetChildrenOfCombination($field->form_id, $field->id) as $childfield)
+        <div>
+            <x-dynamic-component
+                :component="$childfield->getBladeComponent()"
+                :field="$childfield"
+            />
+        </div>
+    @endforeach
+</div>
diff --git a/src/Models/Forms/Field.php b/src/Models/Forms/Field.php
@@ -2,6 +2,7 @@
 
 namespace NotFound\Framework\Models\Forms;
 
+use Illuminate\Database\Eloquent\Collection;
 use Illuminate\Database\Eloquent\SoftDeletes;
 use NotFound\Framework\Models\BaseModel;
 
@@ -163,7 +164,7 @@ public function getWithChildren(int $formId, int $combinationId = null)
         return $returnArray;
     }
 
-    public function GetChildrenOfCombination(int $formId, int $combinationId): array
+    public function GetChildrenOfCombination(int $formId, int $combinationId): Collection
     {
         $fieldQuery = $this
             ->where('form_id', $formId);
@@ -174,8 +175,7 @@ public function GetChildrenOfCombination(int $formId, int $combinationId): array
 
         return $fieldQuery
             ->orderBy('order')
-            ->get()
-            ->toArray();
+            ->get();
     }
 
     private function setParentRecursive(&$fields, $parentId, $fieldToSet)

diff --git a/src/Services/Editor/FieldsProperties.php b/src/Services/Editor/FieldsProperties.php
@@ -150,7 +150,7 @@ private function addLayoutFields(array $properties, LayoutForm &$form)
                         //     $checkboxField->setValue(true);
                         // } else {
                         $checkboxField->setValue(false);
-                    // }
+                        // }
                     } else {
                         $checkboxField->setValue($value ?? false);
                     }

diff --git a/src/View/Components/Forms/Fields/AbstractFieldComponent.php b/src/View/Components/Forms/Fields/AbstractFieldComponent.php
@@ -14,11 +14,12 @@ public function __construct(
 
     public function render()
     {
-        if (view()->exists('siteboss.forms.fields.'.$this->field->type)) {
-            return 'siteboss.forms.fields.'.$this->field->type;
+        $typeFile = strtolower($this->field->type);
+        if (view()->exists('siteboss.forms.fields.'.$typeFile)) {
+            return 'siteboss.forms.fields.'.$typeFile;
         }
 
-        return view('siteboss::siteboss.forms.fields.'.$this->field->type);
+        return view('siteboss::siteboss.forms.fields.'.$typeFile);
     }
 
     public function colClasses(): string