Ronmrcdo\Inventory\Traits\Sluggable

Complexity

Total Complexity

7

Size/Duplication

Total Lines	59
Duplicated Lines	0 %

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	generateUniqueSlug()	0	10	3
A	createUniqueSlug()	0	3	1
A	bootSluggable()	0	8	1
A	isSlugExists()	0	13	2

<?php

namespace Ronmrcdo\Inventory\Traits;

use Illuminate\Support\Str;
use Illuminate\Database\Eloquent\Model;

trait Sluggable
{
	/**
	 * Generate a slug on the model on boot
	 * 
	 * @return void
	 */
	protected static function bootSluggable(): void
	{
		static::creating(function (Model $model) {
            $model->createUniqueSlug();
        });

        static::updating(function (Model $model) {
            $model->createUniqueSlug();
        });
	}

	protected function createUniqueSlug(): void
	{
		$this->slug = $this->generateUniqueSlug();

The property slug does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

	}

	/**
	 * Generate the unique string slug from the $sluggable property
	 * in the model
	 * 
	 * @return string
	 */
	protected function generateUniqueSlug(): string
	{
		$slug = Str::slug($this->attributes[$this->sluggable]);
		$i = 1;

		while($this->isSlugExists($slug) || $slug === '') {
			$slug = $slug.'-'.$i++;
		}
		
		return $slug;
	}

	/**
	 * Assert if the slug exists
	 * 
	 * @return bool
	 */
	protected function isSlugExists(string $slug): bool
	{
		$key = $this->getKey();

It seems like getKey() must be provided by classes using this trait. How about adding it as abstract method to this trait?

		if ($this->incrementing) {
			$key ?? '0';
		}

		$query = static::where('slug', $slug)
				->where($this->getKeyName(), '!=', $key)

It seems like getKeyName() must be provided by classes using this trait. How about adding it as abstract method to this trait?

				->withoutGlobalScopes();

		return $query->exists();
	}
}