BeyondCode\Vouchers\Traits\HasVouchers

Complexity

Total Complexity

3

Size/Duplication

Total Lines	33
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	0

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	vouchers()	0	4	1
A	createVouchers()	0	4	1
A	createVoucher()	0	4	1

<?php

namespace BeyondCode\Vouchers\Traits;

use BeyondCode\Vouchers\Models\Voucher;
use BeyondCode\Vouchers\Facades\Vouchers;

trait HasVouchers
{
    /**
     * Set the polymorphic relation.
     *
     * @return mixed
     */
    public function vouchers()
    {
        return $this->morphMany(config('vouchers.model', Voucher::class), 'model');

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

    }

    /**
     * @param int $amount
     * @param array $data
     * @param null $expires_at
     * @return Voucher[]
     */
    public function createVouchers(int $amount, array $data = [], $expires_at = null)
    {
        return Vouchers::create($this, $amount, $data, $expires_at);

The method create() does not exist on BeyondCode\Vouchers\Facades\Vouchers. Did you maybe mean createFreshMockInstance()?

    }

    /**
     * @param array $data
     * @param null $expires_at
     * @return Voucher
     */
    public function createVoucher(array $data = [], $expires_at = null)
    {
        return $this->createVouchers(1, $data, $expires_at)[0];
    }
}