A17\Twill\Models\Model

Complexity

Total Complexity

25

Size/Duplication

Total Lines	133
Duplicated Lines	0 %

Test Coverage

Coverage

36.1%

Importance

Changes	1
Bugs	0	Features	0

12 Methods

Rating	Name	Duplication	Size	Complexity
A	scopePublished()	0	3	1
A	isTranslationModel()	0	3	1
A	scopeAccessible()	0	25	6
A	scopeDraft()	0	3	1
A	setPublishStartDateAttribute()	0	3	1
A	getTranslatedAttributes()	0	3	1
A	scopePublishedInListings()	0	7	2
A	tags()	0	8	1
A	bootTaggableTrait()	0	3	1
A	scopeOnlyTrashed()	0	3	1
A	getFillable()	0	26	6
A	scopeVisible()	0	15	3

<?php

namespace A17\Twill\Models;

use A17\Twill\Models\Behaviors\HasPresenter;
use A17\Twill\Services\Capsules\HasCapsules;
use A17\Twill\Models\Behaviors\IsTranslatable;
use A17\Twill\Models\Permission;
use Auth;
use Carbon\Carbon;
use Cartalyst\Tags\TaggableInterface;
use Cartalyst\Tags\TaggableTrait;
use Illuminate\Database\Eloquent\Model as BaseModel;
use Illuminate\Database\Eloquent\Relations\MorphToMany;
use Illuminate\Database\Eloquent\SoftDeletes;
use Illuminate\Support\Str;

abstract class Model extends BaseModel implements TaggableInterface
{
    use HasPresenter, SoftDeletes, TaggableTrait, IsTranslatable, HasCapsules;

The trait A17\Twill\Services\Capsules\HasCapsules requires the property $command which is not provided by A17\Twill\Models\Model.

The trait A17\Twill\Models\Behaviors\IsTranslatable requires the property $translatedAttributes which is not provided by A17\Twill\Models\Model.

The trait Cartalyst\Tags\TaggableTrait requires some properties which are not provided by A17\Twill\Models\Model: $tags, $count

    public $timestamps = true;

    protected function isTranslationModel()
    {
        return Str::endsWith(get_class($this), 'Translation');
    }

    public function scopePublished($query)
    {
        return $query->wherePublished(true);
    }

    public function scopeAccessible($query)
    {
        $model = get_class($query->getModel());
        $moduleName = isPermissionableModule(getModuleNameByModel($model));

        if ($moduleName && !Auth::user()->isSuperAdmin()) {

The method isSuperAdmin() does not exist on Illuminate\Contracts\Auth\Authenticatable. It seems like you code against a sub-type of Illuminate\Contracts\Auth\Authenticatable such as Illuminate\Foundation\Auth\User.

            // Get all permissions the logged in user has regards to the model.
            $allPermissions = Auth::user()->allPermissions();

The method allPermissions() does not exist on Illuminate\Contracts\Auth\Authenticatable. It seems like you code against a sub-type of Illuminate\Contracts\Auth\Authenticatable such as Illuminate\Foundation\Auth\User.

            $allModelPermissions = (clone $allPermissions)->ofModel($model);

            // If the user has any module permissions, or global manage all modules permissions, all items will be return
            if ((clone $allModelPermissions)->module()->whereIn('name', Permission::available(Permission::SCOPE_MODULE))->exists()
                || (clone $allPermissions)->global()->where('name', 'manage-modules')->exists()) {
                return $query;
            }

            // If the module is submodule, skip the scope.
            if (strpos($moduleName, '.')) {

It seems like $moduleName can also be of type true; however, parameter $haystack of strpos() does only seem to accept string, maybe add an additional type check?

                return $query;
            };

            $authorizedItemsIds = $allModelPermissions->moduleItem()->pluck('permissionable_id');
            return $query->whereIn($this->getTable() . '.id', $authorizedItemsIds);
        }
        return $query;
    }

    public function scopePublishedInListings($query)
    {
        if ($this->isFillable('public')) {
            $query->wherePublic(true);
        }

        return $query->published()->visible();
    }

    public function scopeVisible($query)
    {
        if ($this->isFillable('publish_start_date')) {
            $query->where(function ($query) {
                $query->whereNull('publish_start_date')->orWhere('publish_start_date', '<=', Carbon::now());
            });

            if ($this->isFillable('publish_end_date')) {
                $query->where(function ($query) {
                    $query->whereNull('publish_end_date')->orWhere('publish_end_date', '>=', Carbon::now());
                });
            }
        }

        return $query;
    }

    public function setPublishStartDateAttribute($value)
    {
        $this->attributes['publish_start_date'] = $value ?? Carbon::now();
    }

    public function scopeDraft($query)
    {
        return $query->wherePublished(false);
    }

    public function scopeOnlyTrashed($query)
    {
        return $query->whereNotNull('deleted_at');
    }

    public function getFillable()
    {
        // If the fillable attribute is filled, just use it
        $fillable = $this->fillable;

        // If fillable is empty
        // and it's a translation model
        // and the baseModel was defined
        // Use the list of translatable attributes on our base model
        if (
            blank($fillable) &&
            $this->isTranslationModel() &&
            property_exists($this, 'baseModuleModel')
        ) {
            $fillable = (new $this->baseModuleModel)->getTranslatedAttributes();

The property baseModuleModel does not seem to exist on A17\Twill\Models\Model. Are you sure there is no database migration missing?

            if (!collect($fillable)->contains('locale')) {
                $fillable[] = 'locale';
            }

            if (!collect($fillable)->contains('active')) {
                $fillable[] = 'active';
            }
        }

        return $fillable;
    }

    public function getTranslatedAttributes()
    {
        return $this->translatedAttributes ?? [];

The property translatedAttributes does not seem to exist on A17\Twill\Models\Model. Are you sure there is no database migration missing?

    }

    protected static function bootTaggableTrait()
    {
        static::$tagsModel = Tag::class;
    }

    /**
     * {@inheritdoc}
     */
    public function tags(): MorphToMany
    {
        return $this->morphToMany(
            static::$tagsModel,
            'taggable',
            config('twill.tagged_table', 'tagged'),
            'taggable_id',
            'tag_id'
        );
    }
}