Cviebrock\EloquentSluggable\SluggableScopeHelpers

Complexity

Total Complexity

6

Size/Duplication

Total Lines	67
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes	2
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	getSlugKeyName()	0	10	2
A	getSlugKey()	0	4	1
A	scopeWhereSlug()	0	4	1
A	findBySlug()	0	4	1
A	findBySlugOrFail()	0	4	1

<?php namespace Cviebrock\EloquentSluggable;

/**
 * Class PrimarySlug
 *
 * Helper trait for defining the primary slug of a model
 * and providing useful scopes and query methods.
 *
 * @package Cviebrock\EloquentSluggable
 */
trait SluggableScopeHelpers
{

    /**
     * Primary slug column of this model.
     *
     * @return string
     */
    public function getSlugKeyName()
    {
        if (property_exists($this, 'slugKeyName')) {
            return $this->slugKeyName;

The property slugKeyName does not exist. Did you maybe forget to declare it?

        }

        $keys = array_keys($this->sluggable());

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

        return reset($keys);
    }

    /**
     * Primary slug value of this model.
     *
     * @return string
     */
    public function getSlugKey()
    {
        return $this->getAttribute($this->getSlugKeyName());

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

    }

    /**
     * Query scope for finding a model by its primary slug.
     *
     * @param \Illuminate\Database\Eloquent\Builder $scope
     * @param string $slug
     * @return \Illuminate\Database\Eloquent\Builder
     */
    public function scopeWhereSlug($scope, $slug)
    {
        return $scope->where($this->getSlugKeyName(), $slug);
    }

    /**
     * Find a model by its primary slug.
     *
     * @param string $slug
     * @param array $columns
     * @return \Illuminate\Database\Eloquent\Model|\Illuminate\Database\Eloquent\Collection|static[]|static|null
     */
    public static function findBySlug($slug, array $columns = ['*'])
    {
        return static::whereSlug($slug)->first($columns);

The method whereSlug() does not exist on Cviebrock\EloquentSluggable\SluggableScopeHelpers. Did you maybe mean scopeWhereSlug()?

    }

    /**
     * Find a model by its primary slug or throw an exception.
     *
     * @param string $slug
     * @param array $columns
     * @return \Illuminate\Database\Eloquent\Model|\Illuminate\Database\Eloquent\Collection
     *
     * @throws \Illuminate\Database\Eloquent\ModelNotFoundException
     */
    public static function findBySlugOrFail($slug, array $columns = ['*'])
    {
        return static::whereSlug($slug)->firstOrFail($columns);

The method whereSlug() does not exist on Cviebrock\EloquentSluggable\SluggableScopeHelpers. Did you maybe mean scopeWhereSlug()?

    }
}