BeyondCode\LaravelScopeChecks\HasScopeChecks

Complexity

Total Complexity

6

Size/Duplication

Total Lines	29
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	isCheckScopeMethod()	0	4	2
A	getOriginalScopeMethodName()	0	4	1
A	forwardCallTo()	0	16	3

<?php

namespace BeyondCode\LaravelScopeChecks;

use Illuminate\Support\Str;

trait HasScopeChecks
{
    protected function isCheckScopeMethod(string $method): bool
    {
        return Str::startsWith($method, 'is') || Str::startsWith($method, 'has');
    }

    protected function getOriginalScopeMethodName($checkMethodName): string
    {
        return preg_replace('/^(is|has)(.*)$/m', '$2', $checkMethodName);
    }

    protected function forwardCallTo($object, $method, $parameters)
    {
        if ($this->isCheckScopeMethod($method)) {
            $originalScopeMethodName = $this->getOriginalScopeMethodName($method);

            if (method_exists($this, "scope{$originalScopeMethodName}")) {
                $builder = $this->newQuery()->where($this->getKeyName(), $this->getKey());

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

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

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

                $builder = call_user_func_array([$builder, $originalScopeMethodName], $parameters);

                return $builder->exists();
            }
        }

        return parent::forwardCallTo($object, $method, $parameters);
    }
}