Fix redirect description persistence

src/Http/Controllers/CP/RedirectController.php

@@ -3,6 +3,7 @@
 namespace Statamic\SeoPro\Http\Controllers\CP;
 
 use Illuminate\Http\Request;
+use Illuminate\Support\Arr;
 use Inertia\Inertia;
 use Statamic\CP\Column;
 use Statamic\CP\PublishForm;
@@ -131,10 +132,11 @@ public function store(Request $request)
 
         $redirect = Facades\Redirect::make()
             ->site($siteHandle)
-            ->source($values['source'])
-            ->destination($values['destination'])
-            ->responseCode($values['response_code'])
-            ->enabled($values['enabled']);
+            ->source(Arr::pull($values, 'source'))
+            ->destination(Arr::pull($values, 'destination'))
+            ->responseCode(Arr::pull($values, 'response_code'))
+            ->enabled(Arr::pull($values, 'enabled'))
+            ->data($values);
 
         $redirect->save();
 
@@ -178,10 +180,11 @@ public function update(Request $request, Redirect $redirect)
         $values = PublishForm::make(Facades\Redirect::blueprint())->submit($request->all());
 
         $redirect
-            ->source($values['source'])
-            ->destination($values['destination'])
-            ->responseCode($values['response_code'])
-            ->enabled($values['enabled']);
+            ->source(Arr::pull($values, 'source'))
+            ->destination(Arr::pull($values, 'destination'))
+            ->responseCode(Arr::pull($values, 'response_code'))
+            ->enabled(Arr::pull($values, 'enabled'))
+            ->data($values);
 
         $redirect->save();
     }

tests/Redirects/StoreRedirectTest.php

@@ -170,4 +170,23 @@ public function source_must_be_unique()
             ])
             ->assertSessionHasErrors('source');
     }
+
+    #[Test]
+    public function can_store_redirect_with_description()
+    {
+        $this
+            ->actingAs(User::make()->makeSuper()->save())
+            ->post(cp_route('seo-pro.redirects.store'), [
+                'source' => '/old-url',
+                'destination' => '/new-url',
+                'response_code' => 301,
+                'enabled' => true,
+                'description' => 'Keep this redirect for the old campaign.',
+            ])
+            ->assertOk();
+
+        $redirect = Facades\Redirect::query()->where('source', '/old-url')->first();
+
+        $this->assertEquals('Keep this redirect for the old campaign.', $redirect->get('description'));
+    }
 }

tests/Redirects/UpdateRedirectTest.php

@@ -214,4 +214,32 @@ public function can_update_redirect_without_changing_source()
         $this->assertEquals('/old-url', $redirect->source());
         $this->assertEquals('/updated-new-url', $redirect->destination());
     }
+
+    #[Test]
+    public function can_update_redirect_with_description()
+    {
+        Facades\Redirect::make()
+            ->id('abc')
+            ->source('/old-url')
+            ->destination('/new-url')
+            ->responseCode(302)
+            ->enabled(true)
+            ->data(['description' => 'Original description.'])
+            ->save();
+
+        $this
+            ->actingAs(User::make()->makeSuper()->save())
+            ->patch(cp_route('seo-pro.redirects.update', 'abc'), [
+                'source' => '/old-url',
+                'destination' => '/new-url',
+                'response_code' => 302,
+                'enabled' => true,
+                'description' => 'Updated description.',
+            ])
+            ->assertOk();
+
+        $redirect = Facades\Redirect::find('abc');
+
+        $this->assertEquals('Updated description.', $redirect->get('description'));
+    }
 }