Dizatech\Transaction\Traits\HasTransaction

Complexity

Total Complexity

5

Size/Duplication

Total Lines	25
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	1

5 Methods

Rating	Name	Duplication	Size	Complexity
A	failedTransactions()	0	3	1
A	pendingTransactions()	0	3	1
A	successfulTransactions()	0	3	1
A	refundedTransactions()	0	3	1
A	transactions()	0	3	1

<?php

namespace Dizatech\Transaction\Traits;

use Dizatech\Transaction\Models\Transaction;

trait HasTransaction
{
    public function transactions()
    {
        return $this->hasMany(Transaction::class, 'order_id');

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

    }

    public function pendingTransactions()
    {
        return $this->hasMany(Transaction::class, 'order_id')->where('status', 'pending');
    }

    public function successfulTransactions()
    {
        return $this->hasMany(Transaction::class, 'order_id')->where('status', 'successful');
    }

    public function failedTransactions()
    {
        return $this->hasMany(Transaction::class, 'order_id')->where('status', 'failed');
    }

    public function refundedTransactions()
    {
        return $this->hasMany(Transaction::class, 'order_id')->where('status', 'refunded');
    }
}