QueryBuilderFactory::create() - Code Metrics - Inspection of "Refactor" - zingimmick/laravel-query-builder - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#116)

by Zing

created 2021-11-02 11:17 UTC

QueryBuilderFactory::create() A

↳ Parent: QueryBuilderFactory

Complexity

Conditions	5
Paths	8

Size

Total Lines	19
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	30

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	5
eloc	9
nc	8
nop	2
dl	0
loc	19
ccs	0
cts	10
cp	0
crap	30
rs	9.6111
c	1
b	0
f	0

<?php

declare(strict_types=1);

namespace Zing\QueryBuilder;

use Illuminate\Database\Eloquent\Model;
use Illuminate\Http\Request;

class QueryBuilderFactory
{
    /**
     * @var array<class-string, class-string<\Zing\QueryBuilder\QueryBuilder>>

     */
    protected $builders = [];

    /**
     * QueryBuilderFactory constructor.
     *
     * @param array<class-string, class-string<\Zing\QueryBuilder\QueryBuilder>> $builders

     */
    public function __construct(array $builders)
    {
        $this->builders = $builders;
    }

    /**
     * @param class-string<\Illuminate\Database\Eloquent\Model>|\Illuminate\Database\Eloquent\Builder $baseQuery

     */
    public function create($baseQuery, Request $request): QueryBuilder
    {
        if (is_subclass_of($baseQuery, Model::class)) {
            $baseQuery = forward_static_call([$baseQuery, 'query']);
        }

        $class = get_class($baseQuery);

        if (isset($this->builders[$class])) {
            return $this->resolveBuilder($this->builders[$class], $baseQuery, $request);
        }

        foreach ($this->builders as $expected => $builder) {
            if (is_subclass_of($class, $expected)) {
                return $this->resolveBuilder($builder, $baseQuery, $request);
            }
        }

        return new QueryBuilder($baseQuery, $request);
    }

    /**
     * @param class-string<\Zing\QueryBuilder\QueryBuilder> $builder

     * @param class-string<\Illuminate\Database\Eloquent\Model>|\Illuminate\Database\Eloquent\Builder|bool $baseQuery
     */
    public function resolveBuilder($builder, $baseQuery, Request $request): QueryBuilder
    {
        return new $builder($baseQuery, $request);
    }

    /**
     * @param mixed $builder
     * @param mixed $queryBuilder
     *
     * @return string[]|mixed[]
     */
    public function queryBy($builder, $queryBuilder): array
    {
        $this->builders[$builder] = $queryBuilder;

        return $this->builders;
    }
}


1			<?php
2
3			declare(strict_types=1);
4
5			namespace Zing\QueryBuilder;
6
7			use Illuminate\Database\Eloquent\Model;
8			use Illuminate\Http\Request;
9
10			class QueryBuilderFactory
11			{
12			/**
13			* @var array<class-string, class-string<\Zing\QueryBuilder\QueryBuilder>>
			0 ignored issues – show Documentation Bug introduced 2021-11-02 11:22 UTC by Report Bug Copy Issue Report The doc comment `array<class-string, clas...yBuilder\QueryBuilder>>` at position `2` could not be parsed: Unknown type name 'class-string' at position 2 in array<class-string, class-string<\Zing\QueryBuilder\QueryBuilder>>. Loading history...
14			*/
15			protected $builders = [];
16
17			/**
18			* QueryBuilderFactory constructor.
19			*
20			* @param array<class-string, class-string<\Zing\QueryBuilder\QueryBuilder>> $builders
			0 ignored issues – show Documentation Bug introduced 2021-11-02 11:22 UTC by Report Bug Copy Issue Report The doc comment `array<class-string, clas...yBuilder\QueryBuilder>>` at position `2` could not be parsed: Unknown type name 'class-string' at position 2 in array<class-string, class-string<\Zing\QueryBuilder\QueryBuilder>>. Loading history...
21			*/
22			public function __construct(array $builders)
23			{
24			$this->builders = $builders;
25			}
26
27			/**
28			* @param class-string<\Illuminate\Database\Eloquent\Model>\|\Illuminate\Database\Eloquent\Builder $baseQuery
			0 ignored issues – show Documentation Bug introduced 2021-11-02 11:22 UTC by Report Bug Copy Issue Report The doc comment `class-string<\Illuminate...tabase\Eloquent\Builder` at position `0` could not be parsed: Unknown type name 'class-string' at position 0 in class-string<\Illuminate\Database\Eloquent\Model>\|\Illuminate\Database\Eloquent\Builder. Loading history...
29			*/
30			public function create($baseQuery, Request $request): QueryBuilder
31			{
32			if (is_subclass_of($baseQuery, Model::class)) {
33			$baseQuery = forward_static_call([$baseQuery, 'query']);
34			}
35
36			$class = get_class($baseQuery);
37
38			if (isset($this->builders[$class])) {
39			return $this->resolveBuilder($this->builders[$class], $baseQuery, $request);
40			}
41
42			foreach ($this->builders as $expected => $builder) {
43			if (is_subclass_of($class, $expected)) {
44			return $this->resolveBuilder($builder, $baseQuery, $request);
45			}
46			}
47
48			return new QueryBuilder($baseQuery, $request);
49			}
50
51			/**
52			* @param class-string<\Zing\QueryBuilder\QueryBuilder> $builder
			0 ignored issues – show Documentation Bug introduced 2021-11-02 11:22 UTC by Report Bug Copy Issue Report The doc comment `class-string<\Zing\QueryBuilder\QueryBuilder>` at position `0` could not be parsed: Unknown type name 'class-string' at position 0 in class-string<\Zing\QueryBuilder\QueryBuilder>. Loading history...
53			* @param class-string<\Illuminate\Database\Eloquent\Model>\|\Illuminate\Database\Eloquent\Builder\|bool $baseQuery
54			*/
55			public function resolveBuilder($builder, $baseQuery, Request $request): QueryBuilder
56			{
57			return new $builder($baseQuery, $request);
58			}
59
60			/**
61			* @param mixed $builder
62			* @param mixed $queryBuilder
63			*
64			* @return string[]\|mixed[]
65			*/
66			public function queryBy($builder, $queryBuilder): array
67			{
68			$this->builders[$builder] = $queryBuilder;
69
70			return $this->builders;
71			}
72			}
73

zingimmick / laravel-query-builder

Pull Request — master (#116)

QueryBuilderFactory::create() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like