Prevent creation of duplicate terms on slug change (#384)

* Fetch term by old slug if present

Signed-off-by: Philipp Daun <[email protected]>

* Add a test

---------

Signed-off-by: Philipp Daun <[email protected]>
Co-authored-by: Ryan Mitchell <[email protected]>

src/Taxonomies/Term.php

@@ -74,10 +74,11 @@ public static function makeModelFromContract(Contract $source)
         }
 
         return $class::firstOrNew([
-            'slug'     => $source->slug(),
+            'slug'     => $source->getOriginal('slug', $source->slug()),
             'taxonomy' => $source->taxonomy(),
             'site'     => $source->locale(),
         ])->fill([
+            'slug'       => $source->slug(),
             'uri'        => $source->uri(),
             'data'       => $data,
             'updated_at' => $source->lastModified(),

tests/Terms/TermTest.php

@@ -0,0 +1,32 @@
+<?php
+
+namespace Tests\Terms;
+
+use Illuminate\Foundation\Testing\RefreshDatabase;
+use PHPUnit\Framework\Attributes\Test;
+use Statamic\Eloquent\Taxonomies\Taxonomy;
+use Statamic\Eloquent\Taxonomies\TermModel;
+use Statamic\Facades\Term as TermFacade;
+use Tests\TestCase;
+
+class TermTest extends TestCase
+{
+    use RefreshDatabase;
+
+    #[Test]
+    public function it_doesnt_create_a_new_model_when_slug_is_changed()
+    {
+        Taxonomy::make('test')->title('test')->save();
+
+        $term = tap(TermFacade::make('test-term')->taxonomy('test')->data([]))->save();
+
+        $this->assertCount(1, TermModel::all());
+        $this->assertSame('test-term', TermModel::first()->slug);
+
+        $term->slug('new-slug');
+        $term->save();
+
+        $this->assertCount(1, TermModel::all());
+        $this->assertSame('new-slug', TermModel::first()->slug);
+    }
+}