Sagitarius29\LaravelSubscriptions\Entities\Subscription

Complexity

Total Complexity

8

Size/Duplication

Total Lines	53
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	2

Importance

Changes

0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	scopeCurrent()	0	5	1
A	isPerpetual()	0	4	1
A	getDaysLeft()	0	4	1
A	getElapsedDays()	0	4	1
A	getExpirationDate()	0	4	1
A	getStartDate()	0	4	1
A	subscriber()	0	4	1
A	make()	0	8	1

<?php

namespace Sagitarius29\LaravelSubscriptions\Entities;

use Carbon\Carbon;
use Illuminate\Database\Eloquent\Model;
use Illuminate\Database\Eloquent\Builder;
use Sagitarius29\LaravelSubscriptions\Contracts\PlanContract;
use Sagitarius29\LaravelSubscriptions\Contracts\SubscriptionContact;

class Subscription extends Model implements SubscriptionContact
{
    protected $table = 'subscriptions';

    protected $fillable = [
        'plan_id', 'start_at', 'end_at',
    ];

    public function scopeCurrent(Builder $q, Carbon $date)
    {
        return $q->where('start_at', '<', $date)
            ->where('end_at', '>', $date);
    }

    public function isPerpetual(): bool
    {
        return $this->end_at == null;
    }

    public function getDaysLeft(): int
    {
        // TODO: Implement getDaysLeft() method.
    }

    public function getElapsedDays(): int
    {
        // TODO: Implement getElapsedDays() method.
    }

    public function getExpirationDate(): ?Carbon
    {
        // TODO: Implement getExpirationDate() method.
    }

    public function getStartDate(): ?Carbon
    {
        // TODO: Implement getStartDate() method.
    }

    public function subscriber()
    {
        // TODO: Implement subscriber() method.
    }

    public static function make(PlanContract $plan, Carbon $start_at, Carbon $end_at = null): Model
    {
        return new self([
            'plan_id'   => $plan->id,

Accessing id on the interface Sagitarius29\LaravelSubscriptions\Contracts\PlanContract suggest that you code against a concrete implementation. How about adding an instanceof check?

            'start_at'  => $start_at,
            'end_at'    => $end_at,
        ]);
    }
}