From bbb0d1be17a05e123460cee126360520c15f0054 Mon Sep 17 00:00:00 2001 From: snipe Date: Mon, 15 Nov 2021 21:09:35 -0800 Subject: [PATCH] Possible fix for asset model editing when no custom fieldset is associated Signed-off-by: snipe --- .../CustomFieldSetDefaultValuesForModel.php | 15 ++++++++++++--- ...m-field-set-default-values-for-model.blade.php | 4 +++- 2 files changed, 15 insertions(+), 4 deletions(-) diff --git a/app/Http/Livewire/CustomFieldSetDefaultValuesForModel.php b/app/Http/Livewire/CustomFieldSetDefaultValuesForModel.php index c9478c054..6816edea7 100644 --- a/app/Http/Livewire/CustomFieldSetDefaultValuesForModel.php +++ b/app/Http/Livewire/CustomFieldSetDefaultValuesForModel.php @@ -19,13 +19,22 @@ class CustomFieldSetDefaultValuesForModel extends Component { $this->model = AssetModel::find($this->model_id); // It's possible to do some clever route-model binding here, but let's keep it simple, shall we? $this->fieldset_id = $this->model->fieldset_id; - $this->fields = CustomFieldset::find($this->fieldset_id)->fields; - $this->add_default_values = ( $this->model->defaultValues->count() > 0); + + $this->fields = null; + + if ($fieldset = CustomFieldset::find($this->fieldset_id)) { + $this->fields = CustomFieldset::find($this->fieldset_id)->fields; + } + + $this->add_default_values = ($this->model->defaultValues->count() > 0); } public function updatedFieldsetId() { - $this->fields = CustomFieldset::find($this->fieldset_id)->fields; + if (CustomFieldset::find($this->fieldset_id)) { + $this->fields = CustomFieldset::find($this->fieldset_id)->fields; + } + } public function render() diff --git a/resources/views/livewire/custom-field-set-default-values-for-model.blade.php b/resources/views/livewire/custom-field-set-default-values-for-model.blade.php index b296b7bc5..c1a8373f4 100644 --- a/resources/views/livewire/custom-field-set-default-values-for-model.blade.php +++ b/resources/views/livewire/custom-field-set-default-values-for-model.blade.php @@ -12,9 +12,10 @@ - @if( $this->add_default_values ) {{-- 'if the checkbox is enabled *AND* there are more than 0 fields in the fieldsset' --}} + @if ($this->add_default_values ) {{-- 'if the checkbox is enabled *AND* there are more than 0 fields in the fieldsset' --}}
+ @if ($fields) @foreach ($fields as $field)
@@ -49,6 +50,7 @@
@endforeach + @endif