BillingRegistryService::hasBehaviour() - Code Metrics - Inspection of "HP-2512 Cleaned up BillingRegistryService" - hiqdev/php-billing - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#102)

unknown

created 2025-06-08 07:35 UTC

BillingRegistryService::hasBehaviour() A

↳ Parent: BillingRegistryService::getTariffTypeDefinitionByName()

Complexity

Conditions	1
Paths	1

Size

Total Lines	3
Code Lines	1

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
eloc	1
nc	1
nop	2
dl	0
loc	3
rs	10
c	0
b	0
f	0

<?php declare(strict_types=1);

namespace hiqdev\php\billing\product\Application;

use hiqdev\php\billing\product\AggregateInterface;
use hiqdev\php\billing\product\behavior\BehaviorInterface;
use hiqdev\php\billing\product\behavior\BehaviorNotFoundException;
use hiqdev\php\billing\product\behavior\InvalidBehaviorException;
use hiqdev\php\billing\product\BillingRegistryInterface;
use hiqdev\php\billing\product\Exception\AggregateNotFoundException;
use hiqdev\php\billing\product\Exception\TariffTypeDefinitionNotFoundException;
use hiqdev\php\billing\product\invoice\InvalidRepresentationException;
use hiqdev\php\billing\product\invoice\RepresentationInterface;
use hiqdev\php\billing\product\price\PriceTypeDefinitionInterface;
use hiqdev\php\billing\product\quantity\FractionQuantityData;
use hiqdev\php\billing\product\quantity\QuantityFormatterInterface;
use hiqdev\php\billing\product\quantity\QuantityFormatterNotFoundException;
use hiqdev\php\billing\product\TariffTypeDefinitionInterface;
use hiqdev\php\billing\type\Type;
use hiqdev\php\billing\type\TypeInterface;
filter:
    dependency_paths: ["lib/*"]

final class BillingRegistryService implements BillingRegistryServiceInterface
{
    public function __construct(private readonly BillingRegistryInterface $registry)
    {
    }

    public function getRepresentationsByType(string $representationClass): array
    {
        if (!class_exists($representationClass) && !interface_exists($representationClass)) {
            throw new InvalidRepresentationException("Class '$representationClass' does not exist");
        }

        if (class_exists($representationClass)
            && !is_subclass_of($representationClass, RepresentationInterface::class)
        ) {
            throw new InvalidBehaviorException(
                sprintf(
                    'Representation class "%s" does not implement RepresentationInterface',
                    $representationClass,
                )
            );
        }

        $representations = [];
        foreach ($this->registry->priceTypes() as $priceTypeDefinition) {
            foreach ($priceTypeDefinition->documentRepresentation() as $representation) {
                if ($representation instanceof $representationClass) {
                    $representations[] = $representation;
                }
            }
        }

        return $representations;
    }

    public function getTariffTypeDefinitionByName(string $tariffName): TariffTypeDefinitionInterface
    {
        foreach ($this->registry->getTariffTypeDefinitions() as $tariffTypeDefinition) {
            if ($tariffTypeDefinition->tariffType()->equalsName($tariffName)) {
                return $tariffTypeDefinition;
            }
        }

        throw new TariffTypeDefinitionNotFoundException('Tariff type definition was not found');
    }

    public function createQuantityFormatter(string $type, FractionQuantityData $data): QuantityFormatterInterface {
        $type = $this->convertStringTypeToType($type);

        foreach ($this->registry->priceTypes() as $priceTypeDefinition) {
            if ($priceTypeDefinition->hasType($type)) {
                return $priceTypeDefinition->createQuantityFormatter($data);
            }
        }

        throw new QuantityFormatterNotFoundException('Quantity formatter not found');
    }

    private function convertStringTypeToType(string $type): TypeInterface
    {
        return Type::anyId($type);
    }

    public function getBehavior(string $type, string $behaviorClassWrapper): BehaviorInterface
    {
        if (!class_exists($behaviorClassWrapper)) {
            throw new InvalidBehaviorException(
                sprintf('Behavior class "%s" does not exist', $behaviorClassWrapper)
            );
        }

        if (!is_subclass_of($behaviorClassWrapper, BehaviorInterface::class)) {
            throw new InvalidBehaviorException(
                sprintf('Behavior class "%s" does not implement BehaviorInterface', $behaviorClassWrapper)
            );
        }

        $billingType = $this->convertStringTypeToType($type);

        foreach ($this->registry->priceTypes() as $priceTypeDefinition) {
            if ($priceTypeDefinition->hasType($billingType)) {
                $behavior = $this->findBehaviorInPriceType($priceTypeDefinition, $behaviorClassWrapper);
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
$object = $a->getObject();

                if ($behavior) {
                    return $behavior;

                }
            }
        }

        throw new BehaviorNotFoundException(
            sprintf('Behavior of class "%s" not found for type "%s"', $behaviorClassWrapper, $type),
        );
    }

    private function findBehaviorInPriceType(
        PriceTypeDefinitionInterface $priceTypeDefinition,
        string $behaviorClassWrapper
    ): ?BehaviorInterface {
        foreach ($priceTypeDefinition->withBehaviors() as $behavior) {
            if ($behavior instanceof $behaviorClassWrapper) {
                return $behavior;
            }
        }

        return null;
    }

    public function getBehaviors(string $behaviorClassWrapper): \Generator
    {
        foreach ($this->registry->getTariffTypeDefinitions() as $tariffTypeDefinition) {
            foreach ($tariffTypeDefinition->withBehaviors() as $behavior) {
                if ($behavior instanceof $behaviorClassWrapper) {
                    yield $behavior;
                }
            }
        }

        foreach ($this->registry->priceTypes() as $priceTypeDefinition) {
            foreach ($priceTypeDefinition->withBehaviors() as $behavior) {
                if ($behavior instanceof $behaviorClassWrapper) {
                    yield $behavior;
                }
            }
        }
    }

    public function getAggregate(string $type): AggregateInterface
    {
        $type = $this->convertStringTypeToType($type);

        foreach ($this->registry->priceTypes() as $priceTypeDefinition) {
            if ($priceTypeDefinition->hasType($type)) {
                return $priceTypeDefinition->getAggregate();
            }
        }

        throw new AggregateNotFoundException('Aggregate was not found');
    }

    public function findPriceTypeDefinitionsByBehavior(string $behaviorClassWrapper): \Generator
    {
        foreach ($this->registry->priceTypes() as $priceTypeDefinition) {
            if ($priceTypeDefinition->hasBehavior($behaviorClassWrapper)) {
                yield $priceTypeDefinition;
            }
        }
    }
}


1			<?php declare(strict_types=1);
2
3			namespace hiqdev\php\billing\product\Application;
4
5			use hiqdev\php\billing\product\AggregateInterface;
6			use hiqdev\php\billing\product\behavior\BehaviorInterface;
7			use hiqdev\php\billing\product\behavior\BehaviorNotFoundException;
8			use hiqdev\php\billing\product\behavior\InvalidBehaviorException;
9			use hiqdev\php\billing\product\BillingRegistryInterface;
10			use hiqdev\php\billing\product\Exception\AggregateNotFoundException;
11			use hiqdev\php\billing\product\Exception\TariffTypeDefinitionNotFoundException;
12			use hiqdev\php\billing\product\invoice\InvalidRepresentationException;
13			use hiqdev\php\billing\product\invoice\RepresentationInterface;
14			use hiqdev\php\billing\product\price\PriceTypeDefinitionInterface;
15			use hiqdev\php\billing\product\quantity\FractionQuantityData;
16			use hiqdev\php\billing\product\quantity\QuantityFormatterInterface;
17			use hiqdev\php\billing\product\quantity\QuantityFormatterNotFoundException;
18			use hiqdev\php\billing\product\TariffTypeDefinitionInterface;
19			use hiqdev\php\billing\type\Type;
20			use hiqdev\php\billing\type\TypeInterface;
			0 ignored issues – show Bug introduced 2025-05-14 21:54 UTC by Report Bug Copy Issue Report The type `hiqdev\php\billing\type\TypeInterface` was not found. Maybe you did not declare it correctly or list all dependencies? The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. `excluded_paths: ["lib/"]`, you can move it to the dependency path list as follows: filter: dependency_paths: ["lib/"] For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths Loading history...
21
22			final class BillingRegistryService implements BillingRegistryServiceInterface
23			{
24			public function __construct(private readonly BillingRegistryInterface $registry)
25			{
26			}
27
28			public function getRepresentationsByType(string $representationClass): array
29			{
30			if (!class_exists($representationClass) && !interface_exists($representationClass)) {
31			throw new InvalidRepresentationException("Class '$representationClass' does not exist");
32			}
33
34			if (class_exists($representationClass)
35			&& !is_subclass_of($representationClass, RepresentationInterface::class)
36			) {
37			throw new InvalidBehaviorException(
38			sprintf(
39			'Representation class "%s" does not implement RepresentationInterface',
40			$representationClass,
41			)
42			);
43			}
44
45			$representations = [];
46			foreach ($this->registry->priceTypes() as $priceTypeDefinition) {
47			foreach ($priceTypeDefinition->documentRepresentation() as $representation) {
48			if ($representation instanceof $representationClass) {
49			$representations[] = $representation;
50			}
51			}
52			}
53
54			return $representations;
55			}
56
57			public function getTariffTypeDefinitionByName(string $tariffName): TariffTypeDefinitionInterface
58			{
59			foreach ($this->registry->getTariffTypeDefinitions() as $tariffTypeDefinition) {
60			if ($tariffTypeDefinition->tariffType()->equalsName($tariffName)) {
61			return $tariffTypeDefinition;
62			}
63			}
64
65			throw new TariffTypeDefinitionNotFoundException('Tariff type definition was not found');
66			}
67
68			public function createQuantityFormatter(string $type, FractionQuantityData $data): QuantityFormatterInterface {
69			$type = $this->convertStringTypeToType($type);
70
71			foreach ($this->registry->priceTypes() as $priceTypeDefinition) {
72			if ($priceTypeDefinition->hasType($type)) {
73			return $priceTypeDefinition->createQuantityFormatter($data);
74			}
75			}
76
77			throw new QuantityFormatterNotFoundException('Quantity formatter not found');
78			}
79
80			private function convertStringTypeToType(string $type): TypeInterface
81			{
82			return Type::anyId($type);
83			}
84
85			public function getBehavior(string $type, string $behaviorClassWrapper): BehaviorInterface
86			{
87			if (!class_exists($behaviorClassWrapper)) {
88			throw new InvalidBehaviorException(
89			sprintf('Behavior class "%s" does not exist', $behaviorClassWrapper)
90			);
91			}
92
93			if (!is_subclass_of($behaviorClassWrapper, BehaviorInterface::class)) {
94			throw new InvalidBehaviorException(
95			sprintf('Behavior class "%s" does not implement BehaviorInterface', $behaviorClassWrapper)
96			);
97			}
98
99			$billingType = $this->convertStringTypeToType($type);
100
101			foreach ($this->registry->priceTypes() as $priceTypeDefinition) {
102			if ($priceTypeDefinition->hasType($billingType)) {
103			$behavior = $this->findBehaviorInPriceType($priceTypeDefinition, $behaviorClassWrapper);
			0 ignored issues – show Bug introduced 2025-05-14 21:54 UTC by Report Bug Copy Issue Report Are you sure the assignment to `$behavior` is correct as `$this->findBehaviorInPri... $behaviorClassWrapper)` targeting `hiqdev\php\billing\produ...ndBehaviorInPriceType()` seems to always return null. This check looks for function or method calls that always return null and whose return value is assigned to a variable. class A { function getObject() { return null; } } $a = new A(); $object = $a->getObject(); The method `getObject()` can return nothing but null, so it makes no sense to assign that value to a variable. The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes. Loading history...
104
105			if ($behavior) {
106			return $behavior;
			0 ignored issues – show Bug Best Practice introduced 2025-05-14 21:54 UTC by Report Bug Copy Issue Report The expression `return $behavior` returns the type `void` which is incompatible with the type-hinted return `hiqdev\php\billing\produ...avior\BehaviorInterface`. Loading history...
107			}
108			}
109			}
110
111			throw new BehaviorNotFoundException(
112			sprintf('Behavior of class "%s" not found for type "%s"', $behaviorClassWrapper, $type),
113			);
114			}
115
116			private function findBehaviorInPriceType(
117			PriceTypeDefinitionInterface $priceTypeDefinition,
118			string $behaviorClassWrapper
119			): ?BehaviorInterface {
120			foreach ($priceTypeDefinition->withBehaviors() as $behavior) {
121			if ($behavior instanceof $behaviorClassWrapper) {
122			return $behavior;
123			}
124			}
125
126			return null;
127			}
128
129			public function getBehaviors(string $behaviorClassWrapper): \Generator
130			{
131			foreach ($this->registry->getTariffTypeDefinitions() as $tariffTypeDefinition) {
132			foreach ($tariffTypeDefinition->withBehaviors() as $behavior) {
133			if ($behavior instanceof $behaviorClassWrapper) {
134			yield $behavior;
135			}
136			}
137			}
138
139			foreach ($this->registry->priceTypes() as $priceTypeDefinition) {
140			foreach ($priceTypeDefinition->withBehaviors() as $behavior) {
141			if ($behavior instanceof $behaviorClassWrapper) {
142			yield $behavior;
143			}
144			}
145			}
146			}
147
148			public function getAggregate(string $type): AggregateInterface
149			{
150			$type = $this->convertStringTypeToType($type);
151
152			foreach ($this->registry->priceTypes() as $priceTypeDefinition) {
153			if ($priceTypeDefinition->hasType($type)) {
154			return $priceTypeDefinition->getAggregate();
155			}
156			}
157
158			throw new AggregateNotFoundException('Aggregate was not found');
159			}
160
161			public function findPriceTypeDefinitionsByBehavior(string $behaviorClassWrapper): \Generator
162			{
163			foreach ($this->registry->priceTypes() as $priceTypeDefinition) {
164			if ($priceTypeDefinition->hasBehavior($behaviorClassWrapper)) {
165			yield $priceTypeDefinition;
166			}
167			}
168			}
169			}
170

hiqdev / php-billing

Pull Request — master (#102)

BillingRegistryService::hasBehaviour() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like