Merge pull request #1175 from cultuurnet/PPF-491-make-website-require…

…d-on-uitpas

PPF-491 Use dependsOn to make website required when switching to UiTPAS

app/Nova/Resources/Integration.php

@@ -25,6 +25,7 @@
 use Laravel\Nova\Fields\BelongsTo;
 use Laravel\Nova\Fields\DateTime;
 use Laravel\Nova\Fields\Field;
+use Laravel\Nova\Fields\FormData;
 use Laravel\Nova\Fields\HasMany;
 use Laravel\Nova\Fields\ID;
 use Laravel\Nova\Fields\Select;
@@ -135,17 +136,23 @@ public function fields(NovaRequest $request): array
 
             URL::make('Website')
                 ->displayUsing(fn () => $this->website)
-                ->required(fn () => $this->type === IntegrationType::UiTPAS->value)
                 ->showOnIndex(false)
-                ->rules(function () {
-                    $rules = ['url:http,https', 'max:255'];
-
-                    if ($this->type !== IntegrationType::UiTPAS->value) {
-                        $rules[] = 'nullable';
+                ->dependsOn(
+                    ['type'],
+                    function (Text $field, NovaRequest $request, FormData $formData) {
+                        $rules = ['url:http,https', 'max:255'];
+                        $required = true;
+
+                        if ($formData->string('type')->toString() !== IntegrationType::UiTPAS->value) {
+                            $rules[] = 'nullable';
+                            $required = false;
+                        }
+
+                        $field
+                            ->required($required)
+                            ->rules($rules);
                     }
-
-                    return $rules;
-                }),
+                ),
 
             BelongsTo::make('Organization')
                 ->withoutTrashed()