MichaelRubel\Couponables\Models\Traits\DefinesModelRelations

Complexity

Total Complexity

2

Size/Duplication

Total Lines	24
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	3
Bugs	0	Features	1

2 Methods

Rating	Name	Duplication	Size	Complexity
A	redeemer()	0	3	1
A	couponables()	0	7	1

<?php

declare(strict_types=1);

namespace MichaelRubel\Couponables\Models\Traits;

use Illuminate\Database\Eloquent\Model;
use Illuminate\Database\Eloquent\Relations\HasMany;
use Illuminate\Database\Eloquent\Relations\MorphTo;
use MichaelRubel\Couponables\Models\Contracts\CouponPivotContract;

trait DefinesModelRelations
{
    /**
     * Fetch the latest couponables - redeemed coupon records.
     *
     * @return HasMany
     */
    public function couponables(): HasMany
    {
        $pivot = app(CouponPivotContract::class);

        return $this
            ->hasMany($pivot::class)

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

            ->orderBy($pivot->getRedeemedAtColumn(), 'desc');
    }

    /**
     * The only model allowed to redeem the coupon.
     *
     * @return MorphTo
     */
    public function redeemer(): MorphTo
    {
        return $this->morphTo();

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

    }
}