Enhance VM creation request validation and UI.
- Introduced `prepareForValidation` method to handle input merging for `behind_traefik` and `devices`. - Updated validation rules to conditionally require `subdomain` based on `behind_traefik`. - Added custom attribute names and error messages for better user feedback. - Improved the create VM form to dynamically show/hide subdomain fields based on the `behind_traefik` checkbox state. - Adjusted the user selection logic to display a message when no customers are available.
This commit is contained in:
@@ -13,14 +13,55 @@ class StoreVmRequest extends FormRequest
|
||||
return $this->user()->can('create', Customer::class);
|
||||
}
|
||||
|
||||
protected function prepareForValidation(): void
|
||||
{
|
||||
$behindTraefik = $this->resolveBehindTraefik();
|
||||
|
||||
$merge = ['behind_traefik' => $behindTraefik];
|
||||
|
||||
if (! $behindTraefik) {
|
||||
$merge['subdomain'] = null;
|
||||
}
|
||||
|
||||
if ($this->has('devices') && is_array($this->input('devices'))) {
|
||||
$devices = collect($this->input('devices'))
|
||||
->filter(fn ($device) => is_array($device) && ! empty($device['type']))
|
||||
->values()
|
||||
->all();
|
||||
|
||||
$merge['devices'] = $devices === [] ? null : $devices;
|
||||
}
|
||||
|
||||
$this->merge($merge);
|
||||
}
|
||||
|
||||
public function wantsTraefik(): bool
|
||||
{
|
||||
return $this->boolean('behind_traefik');
|
||||
}
|
||||
|
||||
private function resolveBehindTraefik(): bool
|
||||
{
|
||||
if (! $this->has('behind_traefik')) {
|
||||
return false;
|
||||
}
|
||||
|
||||
$value = $this->input('behind_traefik');
|
||||
|
||||
if (is_array($value)) {
|
||||
return in_array('1', $value, true) || in_array(1, $value, true) || in_array(true, $value, true);
|
||||
}
|
||||
|
||||
return filter_var($value, FILTER_VALIDATE_BOOLEAN);
|
||||
}
|
||||
|
||||
public function rules(): array
|
||||
{
|
||||
$baseDomain = config('hosting.plesk.base_domain');
|
||||
|
||||
return [
|
||||
'name' => ['required', 'string', 'max:100', 'regex:/^[a-zA-Z0-9][a-zA-Z0-9_-]*$/'],
|
||||
'subdomain' => [
|
||||
Rule::requiredIf(fn () => $this->boolean('behind_traefik')),
|
||||
Rule::excludeIf(fn () => ! $this->wantsTraefik()),
|
||||
Rule::requiredIf(fn () => $this->wantsTraefik()),
|
||||
'nullable',
|
||||
'string',
|
||||
'max:63',
|
||||
@@ -28,7 +69,6 @@ class StoreVmRequest extends FormRequest
|
||||
],
|
||||
'behind_traefik' => ['boolean'],
|
||||
'user_id' => [
|
||||
Rule::requiredIf(fn () => $this->user()->isAdmin()),
|
||||
'nullable',
|
||||
'exists:users,id',
|
||||
],
|
||||
@@ -46,7 +86,7 @@ class StoreVmRequest extends FormRequest
|
||||
|
||||
public function domain(): ?string
|
||||
{
|
||||
if (! $this->boolean('behind_traefik') || ! $this->filled('subdomain')) {
|
||||
if (! $this->wantsTraefik() || ! $this->filled('subdomain')) {
|
||||
return null;
|
||||
}
|
||||
|
||||
@@ -61,4 +101,33 @@ class StoreVmRequest extends FormRequest
|
||||
|
||||
return $this->user()->id;
|
||||
}
|
||||
|
||||
/**
|
||||
* @return array<string, string>
|
||||
*/
|
||||
public function attributes(): array
|
||||
{
|
||||
return [
|
||||
'name' => 'VM-Name',
|
||||
'subdomain' => 'Subdomain',
|
||||
'user_id' => 'Kunde',
|
||||
'cpu' => 'vCPUs',
|
||||
'ram' => 'RAM',
|
||||
'disk' => 'Festplatte',
|
||||
'install_iso' => 'Installations-ISO',
|
||||
'devices.*.type' => 'Gerätetyp',
|
||||
];
|
||||
}
|
||||
|
||||
/**
|
||||
* @return array<string, string>
|
||||
*/
|
||||
public function messages(): array
|
||||
{
|
||||
return [
|
||||
'required' => ':attribute ist erforderlich.',
|
||||
'regex' => ':attribute hat ein ungültiges Format.',
|
||||
'exists' => 'Der gewählte :attribute ist ungültig.',
|
||||
];
|
||||
}
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user