Merge pull request rrze-mmz#93 from rrze-mmz/92-cannot-update-video-w…

…orkflow-settings

Fix update workflow settings bug

app/Http/Controllers/Backend/VideoWorkflowSettingsController.php

@@ -27,12 +27,12 @@ public function show(): Application|Factory|View
      */
     public function update(UpdateVideoWorkflowSettings $request): RedirectResponse
     {
+
         $setting = Setting::opencast();
 
         $setting->data = $request->validated();
-
         $setting->save();
 
-        return to_route('settings.opencast.show');
+        return to_route('settings.workflow.show');
     }
 }

app/Http/Requests/UpdateVideoWorkflowSettings.php

@@ -3,15 +3,16 @@
 namespace App\Http\Requests;
 
 use Illuminate\Foundation\Http\FormRequest;
+use Illuminate\Validation\Rules\Password;
 
 class UpdateVideoWorkflowSettings extends FormRequest
 {
+    protected $stopOnFirstFailure = true;
+
     /**
      * Determine if the user is authorized to make this request.
-     *
-     * @return bool
      */
-    public function authorize()
+    public function authorize(): bool
     {
         return auth()->user()->can('administrate-superadmin-portal-pages');
     }
@@ -26,15 +27,17 @@ public function rules()
         return [
             'url' => ['required', 'url'],
             'username' => ['required', 'string'],
-            'password' => ['required'],
-            'archive_path' => ['required'],
+            'password' => ['required', Password::min(6)],
             'default_workflow_id' => ['required', 'string'],
             'upload_workflow_id' => ['required', 'string'],
             'theme_id_top_right' => ['required', 'numeric'],
-            'theme_id_top_left' => ['required', 'string'],
-            'theme_id_bottom_left' => ['required', 'string'],
-            'theme_id_bottom_right' => ['required', 'string'],
-            'assistant_group_name' => ['required', 'string'],
+            'theme_id_top_left' => ['required', 'numeric'],
+            'theme_id_bottom_left' => ['required', 'numeric'],
+            'theme_id_bottom_right' => ['required', 'numeric'],
+            'archive_path' => ['required', 'string'],
+            'assistants_group_name' => ['required', 'string'],
+            'opencast_purge_end_date' => ['required', 'date'],
+            'opencast_purge_events_per_minute' => ['required', 'numeric'],
         ];
     }
 }

config/settings.php

@@ -1,5 +1,7 @@
 <?php
 
+use Carbon\Carbon;
+
 return [
     'portal' => [
         'maintenance_mode' => false,
@@ -28,13 +30,15 @@
         'username' => 'admin',
         'password' => 'opencast',
         'archive_path' => 'archive/mh_default_org',
-        'default_workflow' => 'fast',
+        'default_workflow_id' => 'fast',
         'upload_workflow_id' => 'fast',
         'theme_id_top_right' => '500',
         'theme_id_top_left' => '501',
         'theme_id_bottom_left' => '502',
         'theme_id_bottom_right' => '503',
         'assistants_group_name' => 'ROLE_GROUP_TIDES_ASSISTANTS',
+        'opencast_purge_end_date' => Carbon::now(),
+        'opencast_purge_events_per_minute' => '20',
     ],
     'streaming' => [
         'wowza_vod_engine_url' => 'localhost:1935',