Remove breaking change by introducing $slugDeps and $slugFields with …

…fallback to $slugAttributes
area17 · Nov 20, 2024 · c0a4396 · c0a4396
1 parent 0179d11
commit c0a4396
Show file tree

Hide file tree

Showing 16 changed files with 67 additions and 26 deletions.
diff --git a/docs/content/2_guides/1_page-builder-with-blade/5_configuring-the-page-module.md b/docs/content/2_guides/1_page-builder-with-blade/5_configuring-the-page-module.md
@@ -168,7 +168,7 @@ class Page extends Model
         'description',
     ];
 
-    public $slugAttributes = [
+    protected $slugFields = [
         'title',
     ];
 

diff --git a/...ent/2_guides/building_a_multilingual_site_with_twill_and_laravel_localization/Article.php b/...ent/2_guides/building_a_multilingual_site_with_twill_and_laravel_localization/Article.php
@@ -24,7 +24,7 @@ class Article extends Model implements LocalizedUrlRoutable
         'description',
     ];
 
-    public $slugAttributes = [
+    protected $slugFields = [
         'title',
     ];
 

diff --git a/docs/content/2_guides/prefill-block-editor-from-template/Article.php b/docs/content/2_guides/prefill-block-editor-from-template/Article.php
@@ -57,7 +57,7 @@ class Article extends Model implements Sortable
 
     // #endregion fillable
 
-    public $slugAttributes = [
+    private $slugFields = [
         'title',
     ];
 

diff --git a/examples/basic-page-builder/app/Models/Page.php b/examples/basic-page-builder/app/Models/Page.php
@@ -24,7 +24,7 @@ class Page extends Model
         'description',
     ];
 
-    public $slugAttributes = [
+    protected $slugFields = [
         'title',
     ];
 }
diff --git a/examples/portfolio/app/Models/Partner.php b/examples/portfolio/app/Models/Partner.php
@@ -24,7 +24,7 @@ class Partner extends Model
         'description',
     ];
 
-    public $slugAttributes = [
+    protected $slugFields = [
         'title',
     ];
 

diff --git a/examples/portfolio/app/Models/Project.php b/examples/portfolio/app/Models/Project.php
@@ -27,7 +27,7 @@ class Project extends Model
         'description',
     ];
 
-    public $slugAttributes = [
+    protected $slugFields = [
         'title',
     ];
 

diff --git a/examples/tests-capsules/app/Twill/Capsules/Posts/Models/Post.php b/examples/tests-capsules/app/Twill/Capsules/Posts/Models/Post.php
@@ -26,7 +26,7 @@ class Post extends Model implements Sortable
 
     public $translatedAttributes = ['title', 'description'];
 
-    public $slugAttributes = ['title'];
+    public $slugFields = ['title'];
 
     public $mediasParams = [
         'cover' => [

diff --git a/examples/tests-modules/app/Models/Author.php b/examples/tests-modules/app/Models/Author.php
@@ -56,7 +56,7 @@ class Author extends Model implements Sortable
     public $translatedAttributes = ['name', 'description', 'bio'];
 
     // uncomment and modify this as needed if you use the HasSlug trait
-    public $slugAttributes = ['name'];
+    protected $slugFields = ['name'];
 
     // add checkbox fields names here (published toggle is itself a checkbox)
     public $checkboxes = ['published'];

diff --git a/examples/tests-modules/app/Models/Category.php b/examples/tests-modules/app/Models/Category.php
@@ -25,7 +25,7 @@ class Category extends Model
     public $translatedAttributes = ['title'];
 
     // uncomment and modify this as needed if you use the HasSlug trait
-    public $slugAttributes = ['title'];
+    protected $slugFields = ['title'];
 
     // add checkbox fields names here (published toggle is itself a checkbox)
     public $checkboxes = ['published'];

diff --git a/examples/tests-singleton/app/Models/ContactPage.php b/examples/tests-singleton/app/Models/ContactPage.php
@@ -25,7 +25,7 @@ class ContactPage extends Model
         'description',
     ];
 
-    public $slugAttributes = [
+    protected $slugFields = [
         'title',
     ];
 

diff --git a/examples/tests-subdomain-routing/app/Models/Page.php b/examples/tests-subdomain-routing/app/Models/Page.php
@@ -28,7 +28,7 @@ class Page extends Model implements Sortable
         'description',
     ];
 
-    public $slugAttributes = [
+    public $slugFields = [
         'title',
     ];
 

diff --git a/src/Commands/stubs/model.stub b/src/Commands/stubs/model.stub
@@ -21,8 +21,9 @@ class {{modelClassName}} extends Model {{modelImplements}}
         'description',
     ];
     {{/hasTranslation}}{{hasSlug}}
-    public $slugAttributes = [
+    protected $slugFields = [
         'title',
     ];
+    protected $slugDeps = [];
     {{/hasSlug}}
 }
diff --git a/src/Models/Behaviors/HasSlug.php b/src/Models/Behaviors/HasSlug.php
@@ -183,7 +183,6 @@ public function updateOrNewSlug(array $slugParams): void
                         // The relation was loaded before the old slug even existed, unload it and let it lazy reload as needed
                         $this->unsetRelation('slugs');
                     }
-                    var_dump('updated old', $this->slugs);
                 }
                 if ($isNowActive) {
                     $this->disableLocaleSlugs($oldMatchingSlug->locale, $oldMatchingSlug->getKey());
@@ -257,7 +256,6 @@ public function disableLocaleSlugs(string|array $locale = null, int|array $excep
                     $slug->active = false;
                     $slug->syncOriginalAttribute('active');
                 });
-            var_dump('disabled', $this->slugs);
         }
     }
 
@@ -360,13 +358,25 @@ public function getSlugAttribute(): string
         return $this->getSlug();
     }
 
+    public function getSlugDeps(): array
+    {
+        return $this->slugDeps ?? array_slice($this->slugAttributes ?? [], 1);
+    }
+    public function getSlugFields(): array
+    {
+        if (!isset($this->slugFields) && isset($this->slugAttributes)) {
+            trigger_deprecation('area17/twill', '3.5', 'The slugAttributes property has been deprecated instead define slug fields with the slugFields property and additional columns with the slugDeps property');
+        }
+        return $this->slugFields ?? array_slice($this->slugAttributes ?? [], 0, 1);
+    }
+
     public function getSlugParams(?string $locale = null, bool $skipUnchanged = false): ?array
     {
         $translatedAttributes = $this->getTranslatedAttributes();
         $slugParams = [];
-        $attributes = $this->slugAttributes ?? [];
-        $deps = $this->slugDependencies ?? [];
-        if (empty($attributes)) {
+        $deps = $this->getSlugDeps();
+        $fields = $this->getSlugFields();
+        if (empty($fields)) {
             return null;
         }
         foreach (getLocales() as $appLocale) {
@@ -395,9 +405,9 @@ public function getSlugParams(?string $locale = null, bool $skipUnchanged = fals
                     return $this->$attribute;
                 };
 
-                $slugAttributes = array_filter(array_map($getAttributeValue, $attributes));
+                $slugFields = array_filter(array_map($getAttributeValue, $fields));
 
-                if (empty($slugAttributes)) {
+                if (empty($slugFields)) {
                     // Skip empty slugs
                     continue;
                 }
@@ -408,7 +418,7 @@ public function getSlugParams(?string $locale = null, bool $skipUnchanged = fals
                 }
                 $slugParam = [
                         'active' => $translation?->active ?? true,
-                        'slug' => Arr::join($slugAttributes, '-'),
+                        'slug' => Arr::join($slugFields, '-'),
                         'locale' => $appLocale,
                     ] + $slugDependencies;
 

diff --git a/src/Repositories/Behaviors/HandleSlugs.php b/src/Repositories/Behaviors/HandleSlugs.php
@@ -11,7 +11,7 @@ trait HandleSlugs
 {
     public function beforeSaveHandleSlugs(TwillModelContract $object, array $fields): void
     {
-        if (property_exists($this->model, 'slugAttributes')) {
+        if (method_exists($this->model, 'getSlugFields')) {
             $object->twillSlugData = [];
             $submittedLanguages = Collection::make($fields['languages'] ?? []);
 

diff --git a/tests/integration/Anonymous/AnonymousModule.php b/tests/integration/Anonymous/AnonymousModule.php
@@ -94,6 +94,12 @@ class AnonymousModule
     /** @var string[] */
     private array $slugAttributes = [];
 
+    /** @var string[] */
+    private array $slugFields = [];
+
+    /** @var string[] */
+    private array $slugDeps = [];
+
     protected function __construct(public string $namePlural, public Application $app)
     {
         $this->classPrinter = new PsrPrinter();
@@ -203,6 +209,22 @@ public function withSlugAttributes(array $fields): self
         return $this;
     }
 
+
+    public function withSlugFields(array $fields): self
+    {
+        $this->slugFields = $fields;
+
+        return $this;
+    }
+
+    public function withSlugDeps(array $fields): self
+    {
+        $this->slugDeps = $fields;
+
+        return $this;
+    }
+
+
     /**
      * Boots the anonymous module and returns the model class.
      */
@@ -626,6 +648,18 @@ private function getModelClass(string $classNameWithNamespace): PhpNamespace
                 $this->slugAttributes
             );
         }
+        if ($this->slugDeps !== []) {
+            $class->addProperty(
+                'slugDeps',
+                $this->slugDeps
+            );
+        }
+        if ($this->slugFields !== []) {
+            $class->addProperty(
+                'slugFields',
+                $this->slugFields
+            );
+        }
 
         foreach ($this->belongsToMany as $name => $target) {
             $method = $class->addMethod($name);

diff --git a/tests/integration/SlugTest.php b/tests/integration/SlugTest.php
@@ -5,10 +5,6 @@
 use A17\Twill\Models\Behaviors\HasSlug;
 use A17\Twill\Models\Model;
 use A17\Twill\Tests\Integration\Anonymous\AnonymousModule;
-use App\Models\Author;
-use App\Repositories\AuthorRepository;
-use Illuminate\Support\Arr;
-use Illuminate\Support\Facades\DB;
 
 class SlugTest extends TestCase
 {
@@ -36,7 +32,7 @@ public function testMultipleSlugAttributes()
                 'first_name' => [],
                 'last_name' => [],
             ])
-            ->withSlugAttributes([
+            ->withSlugFields([
                 'last_name', 'first_name'
             ])
             ->boot();