diff --git a/app/Http/Controllers/AssetModelsController.php b/app/Http/Controllers/AssetModelsController.php index a3afc4f00..5ac958a8a 100755 --- a/app/Http/Controllers/AssetModelsController.php +++ b/app/Http/Controllers/AssetModelsController.php @@ -88,7 +88,7 @@ class AssetModelsController extends Controller $model->requestable = Request::has('requestable'); if ($request->input('fieldset_id') != '') { - $model->fieldset_id = e($request->input('fieldset_id')); + $model->fieldset_id = $request->input('fieldset_id'); } $model = $request->handleImages($model); @@ -101,7 +101,6 @@ class AssetModelsController extends Controller } } - // Redirect to the new model page return redirect()->route('models.index')->with('success', trans('admin/models/message.create.success')); } @@ -166,17 +165,14 @@ class AssetModelsController extends Controller $this->removeCustomFieldsDefaultValues($model); - if ($request->input('fieldset_id') == '') { - $model->fieldset_id = null; - } else { - $model->fieldset_id = $request->input('fieldset_id'); + $model->fieldset_id = $request->input('fieldset_id'); - if ($this->shouldAddDefaultValues($request->input())) { - if (!$this->assignCustomFieldsDefaultValues($model, $request->input('default_values'))){ - return redirect()->back()->withInput()->with('error', trans('admin/custom_fields/message.fieldset_default_value.error')); - } + if ($this->shouldAddDefaultValues($request->input())) { + if (!$this->assignCustomFieldsDefaultValues($model, $request->input('default_values'))){ + return redirect()->back()->withInput()->with('error', trans('admin/custom_fields/message.fieldset_default_value.error')); } } + diff --git a/database/factories/CustomFieldFactory.php b/database/factories/CustomFieldFactory.php index 9407f16b0..21073b1a7 100644 --- a/database/factories/CustomFieldFactory.php +++ b/database/factories/CustomFieldFactory.php @@ -99,7 +99,7 @@ class CustomFieldFactory extends Factory return [ 'name' => 'Test Checkbox', 'help_text' => 'This is a sample checkbox.', - 'field_values' => "One\nTwo\nThree", + 'field_values' => "One\r\nTwo\r\nThree", 'element' => 'checkbox', ]; }); 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 19b2b4dbe..f56c78f58 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 @@ -1,54 +1,80 @@ - {{-- This doesn't seem to fix it, neither does a div? --}} + +
- -
- {{ Form::select('fieldset_id', Helper::customFieldsetList(), old('fieldset_id', $fieldset_id), array('class'=>'select2 js-fieldset-field livewire-select2', 'style'=>'width:100%; min-width:350px', 'aria-label'=>'custom_fieldset', 'data-livewire-component' => $_instance->id)) }} - {!! $errors->first('custom_fieldset', '') !!} -
-
- -
+ +
+ {{ Form::select('fieldset_id', Helper::customFieldsetList(), old('fieldset_id', $fieldset_id), array('class'=>'select2 js-fieldset-field livewire-select2', 'style'=>'width:100%; min-width:350px', 'aria-label'=>'custom_fieldset', 'data-livewire-component' => $_instance->id)) }} + {!! $errors->first('custom_fieldset', '') !!} +
+
+ +
+ @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)
- - -
+ + +
+ @if ($field->format == "DATE") +
+ @elseif ($field->element == "text") - + + + + + @elseif($field->element == "textarea") -
+ + + + + @elseif($field->element == "listbox") - + + + + @elseif($field->element == "radio") + @foreach(explode("\r\n", $field->field_values) as $field_value) - defaultValue($model_id) == $field_value ? 'checked="checked"': '' }} />{{ $field_value }}
+ @endforeach + @elseif($field->element == "checkbox") - @foreach(explode("\r\n", $field->field_values) as $field_value) - defaultValue($model_id))) ? 'checked="checked"': '' }} /> {{ $field_value }}
+ + @foreach(explode("\r\n", $field->field_values) as $field_value) + @endforeach + + @else Unknown field element: {{ $field->element }} @@ -58,9 +84,8 @@
@endforeach +
@endif -
-
@endif