Sagitarius29\LaravelSubscriptions\Traits\HasFeatures

Complexity

Total Complexity

5

Size/Duplication

Total Lines	30
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	addFeature()	0	4	1
A	addFeatures()	0	4	1
A	features()	0	4	1
A	getFeatureByCode()	0	4	1
A	hasFeatures()	0	4	1

<?php

namespace Sagitarius29\LaravelSubscriptions\Traits;

use Illuminate\Database\Eloquent\Model;
use Illuminate\Database\Eloquent\Relations\HasMany;
use Sagitarius29\LaravelSubscriptions\Exceptions\PlanErrorException;
use Sagitarius29\LaravelSubscriptions\PlanFeature;

trait HasFeatures
{
    /**
     * @param  PlanFeature|Model  $feature
     */
    public function addFeature(PlanFeature $feature)
    {
        $this->features()->save($feature);
    }

    public function addFeatures(array $features)
    {
        $this->features()->saveMany($features);

$features is of type array, but the function expects a object<Illuminate\Database\Eloquent\Relations\iterable>.

    }

    public function features(): HasMany
    {
        return $this->hasMany(config('subscriptions.entities.plan_feature'));

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

    }

    public function getFeatureByCode($featureCode)
    {
        return $this->features()->whereCode($featureCode)->first();
    }

    public function hasFeatures(): bool
    {
        return $this->features()->exists();
    }
}