16050929464

Committed 03 Jul 2025 12:51PM UTC coverage: 22.065% (+0.2%) from 21.821%

Build # 16050929464

Build Type

push

github

Committed by

soyuka

Commit Message

chore: todo improvement

Run Details

11516 of 52192 relevant lines covered (22.06%)

22.08 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

90.18

/src/Metadata/Resource/Factory/ParameterResourceMetadataCollectionFactory.php

<?php

/*
 * This file is part of the API Platform project.
 *
 * (c) Kévin Dunglas <dunglas@gmail.com>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

declare(strict_types=1);

namespace ApiPlatform\Metadata\Resource\Factory;

use ApiPlatform\Metadata\ApiProperty;
use ApiPlatform\Metadata\Exception\RuntimeException;
use ApiPlatform\Metadata\FilterInterface;
use ApiPlatform\Metadata\JsonSchemaFilterInterface;
use ApiPlatform\Metadata\OpenApiParameterFilterInterface;
use ApiPlatform\Metadata\Operation;
use ApiPlatform\Metadata\Parameter;
use ApiPlatform\Metadata\ParameterProviderFilterInterface;
use ApiPlatform\Metadata\Parameters;
use ApiPlatform\Metadata\PropertiesAwareInterface;
use ApiPlatform\Metadata\Property\Factory\PropertyMetadataFactoryInterface;
use ApiPlatform\Metadata\Property\Factory\PropertyNameCollectionFactoryInterface;
use ApiPlatform\Metadata\Resource\ResourceMetadataCollection;
use ApiPlatform\OpenApi\Model\Parameter as OpenApiParameter;
use ApiPlatform\Serializer\Filter\FilterInterface as SerializerFilterInterface;
use ApiPlatform\State\Util\StateOptionsTrait;
use Psr\Container\ContainerInterface;
use Psr\Log\LoggerInterface;
use Symfony\Component\Serializer\NameConverter\NameConverterInterface;
use Symfony\Component\TypeInfo\Type;

/**
 * Prepares Parameters documentation by reading its filter details and declaring an OpenApi parameter.
 *
 * @experimental
 */
final class ParameterResourceMetadataCollectionFactory implements ResourceMetadataCollectionFactoryInterface
{
    use StateOptionsTrait;

    private array $localPropertyCache;

    public function __construct(
        private readonly PropertyNameCollectionFactoryInterface $propertyNameCollectionFactory,
        private readonly PropertyMetadataFactoryInterface $propertyMetadataFactory,
        private readonly ?ResourceMetadataCollectionFactoryInterface $decorated = null,
        private readonly ?ContainerInterface $filterLocator = null,
        private readonly ?NameConverterInterface $nameConverter = null,
        private readonly ?LoggerInterface $logger = null,
    ) {
    }

    public function create(string $resourceClass): ResourceMetadataCollection
    {
        $resourceMetadataCollection = $this->decorated?->create($resourceClass) ?? new ResourceMetadataCollection($resourceClass);

        foreach ($resourceMetadataCollection as $i => $resource) {
            $operations = $resource->getOperations();

            $internalPriority = -1;
            foreach ($operations as $operationName => $operation) {
                $parameters = $this->getDefaultParameters($operation, $resourceClass, $internalPriority);
                if (\count($parameters) > 0) {
                    $operations->add($operationName, $operation->withParameters($parameters));
                }
            }

            $resourceMetadataCollection[$i] = $resource->withOperations($operations->sort());

            if (!$graphQlOperations = $resource->getGraphQlOperations()) {
                continue;
            }

            $internalPriority = -1;
            foreach ($graphQlOperations as $operationName => $operation) {
                $parameters = $this->getDefaultParameters($operation, $resourceClass, $internalPriority);
                if (\count($parameters) > 0) {
                    $graphQlOperations[$operationName] = $operation->withParameters($parameters);
                }
            }

            $resourceMetadataCollection[$i] = $resource->withGraphQlOperations($graphQlOperations);
        }

        return $resourceMetadataCollection;
    }

    /**
     * @return array{propertyNames: string[], properties: array<string, ApiProperty>}
     */
    private function getProperties(string $resourceClass, ?Parameter $parameter = null): array
    {
        $k = $resourceClass.($parameter?->getProperties() ? ($parameter->getKey() ?? '') : '');
        if (isset($this->localPropertyCache[$k])) {
            return $this->localPropertyCache[$k];
        }

        $propertyNames = [];
        $properties = [];
        foreach ($parameter?->getProperties() ?? $this->propertyNameCollectionFactory->create($resourceClass) as $property) {
            $propertyMetadata = $this->propertyMetadataFactory->create($resourceClass, $property);
            if ($propertyMetadata->isReadable()) {
                $propertyNames[] = $property;
                $properties[$property] = $propertyMetadata;
            }
        }

        $this->localPropertyCache[$k] = ['propertyNames' => $propertyNames, 'properties' => $properties];

        return $this->localPropertyCache[$k];
    }

    private function getDefaultParameters(Operation $operation, string $resourceClass, int &$internalPriority): Parameters
    {
        $propertyNames = $properties = [];
        $parameters = $operation->getParameters() ?? new Parameters();
        foreach ($parameters as $key => $parameter) {
            ['propertyNames' => $propertyNames, 'properties' => $properties] = $this->getProperties($resourceClass, $parameter);
            if (null === $parameter->getProvider() && (($f = $parameter->getFilter()) && $f instanceof ParameterProviderFilterInterface)) {
                $parameters->add($key, $parameter->withProvider($f->getParameterProvider()));
            }

            if (':property' === $key) {
                foreach ($propertyNames as $property) {
                    $converted = $this->nameConverter?->denormalize($property) ?? $property;
                    $propertyParameter = $this->setDefaults($converted, $parameter, $resourceClass, $properties, $operation);
                    $priority = $propertyParameter->getPriority() ?? $internalPriority--;
                    $parameters->add($converted, $propertyParameter->withPriority($priority)->withKey($converted));
                }

                $parameters->remove($key, $parameter::class);
                continue;
            }

            $key = $parameter->getKey() ?? $key;

            if (str_contains($key, ':property') || ((($f = $parameter->getFilter()) && is_a($f, PropertiesAwareInterface::class, true)) || $parameter instanceof PropertiesAwareInterface)) {
                $p = [];
                foreach ($propertyNames as $prop) {
                    $p[$this->nameConverter?->denormalize($prop) ?? $prop] = $prop;
                }

                $parameter = $parameter->withExtraProperties($parameter->getExtraProperties() + ['_properties' => $p]);
            }

            $parameter = $this->setDefaults($key, $parameter, $resourceClass, $properties, $operation);
            // We don't do any type cast yet, a query parameter or an header is always a string or a list of strings
            if (null === $parameter->getNativeType()) {
                // this forces the type to be only a list
                if ('array' === ($parameter->getSchema()['type'] ?? null)) {
                    $parameter = $parameter->withNativeType(Type::list(Type::string()));
                } elseif ('string' === ($parameter->getSchema()['type'] ?? null)) {
                    $parameter = $parameter->withNativeType(Type::string());
                } elseif ('boolean' === ($parameter->getSchema()['type'] ?? null)) {
                    $parameter = $parameter->withNativeType(Type::bool());
                } else {
                    $parameter = $parameter->withNativeType(Type::union(Type::string(), Type::list(Type::string())));
                }
            }

            $priority = $parameter->getPriority() ?? $internalPriority--;
            $parameters->add($key, $parameter->withPriority($priority));
        }

        return $parameters;
    }

    private function addFilterMetadata(Parameter $parameter): Parameter
    {
        if (!($filterId = $parameter->getFilter())) {
            return $parameter;
        }

        if (!\is_object($filterId) && !$this->filterLocator->has($filterId)) {
            return $parameter;
        }

        $filter = \is_object($filterId) ? $filterId : $this->filterLocator->get($filterId);

        if ($filter instanceof ParameterProviderFilterInterface) {
            $parameter = $parameter->withProvider($filter::getParameterProvider());
        }

        if (!$filter) {
            return $parameter;
        }

        if (null === $parameter->getSchema() && $filter instanceof JsonSchemaFilterInterface && $schema = $filter->getSchema($parameter)) {
            $parameter = $parameter->withSchema($schema);
        }

        if (null === $parameter->getOpenApi() && $filter instanceof OpenApiParameterFilterInterface && ($openApiParameter = $filter->getOpenApiParameters($parameter))) {
            $parameter = $parameter->withOpenApi($openApiParameter);
        }

        return $parameter;
    }

    /**
     * @param array<string, ApiProperty> $properties
     */
    private function setDefaults(string $key, Parameter $parameter, string $resourceClass, array $properties, Operation $operation): Parameter
    {
        if (null === $parameter->getKey()) {
            $parameter = $parameter->withKey($key);
        }

        $filter = $parameter->getFilter();
        if (\is_string($filter) && $this->filterLocator->has($filter)) {
            $filter = $this->filterLocator->get($filter);
        }

        if ($filter instanceof SerializerFilterInterface && null === $parameter->getProvider()) {
            $parameter = $parameter->withProvider('api_platform.serializer.filter_parameter_provider');
        }
        $currentKey = $key;
        if (null === $parameter->getProperty() && isset($properties[$key])) {
            $parameter = $parameter->withProperty($key);
        }

        if (null === $parameter->getProperty() && $this->nameConverter && ($nameConvertedKey = $this->nameConverter->normalize($key)) && isset($properties[$nameConvertedKey])) {
            $parameter = $parameter->withProperty($key)->withExtraProperties(['_query_property' => $nameConvertedKey] + $parameter->getExtraProperties());
            $currentKey = $nameConvertedKey;
        }

        if ($this->nameConverter && $property = $parameter->getProperty()) {
            $parameter = $parameter->withProperty($this->nameConverter->normalize($property));
        }

        if (isset($properties[$currentKey]) && ($eloquentRelation = ($properties[$currentKey]->getExtraProperties()['eloquent_relation'] ?? null)) && isset($eloquentRelation['foreign_key'])) {
            $parameter = $parameter->withExtraProperties(['_query_property' => $eloquentRelation['foreign_key']] + $parameter->getExtraProperties());
        }

        $parameter = $this->addFilterMetadata($parameter);

        if ($filter instanceof FilterInterface) {
            try {
                return $this->getLegacyFilterMetadata($parameter, $operation, $filter);
            } catch (RuntimeException $exception) {
                $this->logger?->alert($exception->getMessage(), ['exception' => $exception]);

                return $parameter;
            }
        }

        return $parameter;
    }

    private function getLegacyFilterMetadata(Parameter $parameter, Operation $operation, FilterInterface $filter): Parameter
    {
        $description = $filter->getDescription($this->getStateOptionsClass($operation, $operation->getClass()));
        $key = $parameter->getKey();
        if (($schema = $description[$key]['schema'] ?? null) && null === $parameter->getSchema()) {
            $parameter = $parameter->withSchema($schema);
        }

        if (null === $parameter->getProperty() && ($property = $description[$key]['property'] ?? null)) {
            $parameter = $parameter->withProperty($property);
        }

        if (null === $parameter->getRequired() && ($required = $description[$key]['required'] ?? null)) {
            $parameter = $parameter->withRequired($required);
        }

        if (null === $parameter->getOpenApi() && ($openApi = $description[$key]['openapi'] ?? null) && $openApi instanceof OpenApiParameter) {
            $parameter = $parameter->withOpenApi($openApi);
        }

        return $parameter;
    }
}

1	<?php
2
3	/*
4	* This file is part of the API Platform project.
5	*
6	* (c) Kévin Dunglas <dunglas@gmail.com>
7	*
8	* For the full copyright and license information, please view the LICENSE
9	* file that was distributed with this source code.
10	*/
11
12	declare(strict_types=1);
13
14	namespace ApiPlatform\Metadata\Resource\Factory;
15
16	use ApiPlatform\Metadata\ApiProperty;
17	use ApiPlatform\Metadata\Exception\RuntimeException;
18	use ApiPlatform\Metadata\FilterInterface;
19	use ApiPlatform\Metadata\JsonSchemaFilterInterface;
20	use ApiPlatform\Metadata\OpenApiParameterFilterInterface;
21	use ApiPlatform\Metadata\Operation;
22	use ApiPlatform\Metadata\Parameter;
23	use ApiPlatform\Metadata\ParameterProviderFilterInterface;
24	use ApiPlatform\Metadata\Parameters;
25	use ApiPlatform\Metadata\PropertiesAwareInterface;
26	use ApiPlatform\Metadata\Property\Factory\PropertyMetadataFactoryInterface;
27	use ApiPlatform\Metadata\Property\Factory\PropertyNameCollectionFactoryInterface;
28	use ApiPlatform\Metadata\Resource\ResourceMetadataCollection;
29	use ApiPlatform\OpenApi\Model\Parameter as OpenApiParameter;
30	use ApiPlatform\Serializer\Filter\FilterInterface as SerializerFilterInterface;
31	use ApiPlatform\State\Util\StateOptionsTrait;
32	use Psr\Container\ContainerInterface;
33	use Psr\Log\LoggerInterface;
34	use Symfony\Component\Serializer\NameConverter\NameConverterInterface;
35	use Symfony\Component\TypeInfo\Type;
36
37	/**
38	* Prepares Parameters documentation by reading its filter details and declaring an OpenApi parameter.
39	*
40	* @experimental
41	*/
42	final class ParameterResourceMetadataCollectionFactory implements ResourceMetadataCollectionFactoryInterface
43	{
44	use StateOptionsTrait;
45
46	private array $localPropertyCache;
47
48	public function __construct(
49	private readonly PropertyNameCollectionFactoryInterface $propertyNameCollectionFactory,
50	private readonly PropertyMetadataFactoryInterface $propertyMetadataFactory,
51	private readonly ?ResourceMetadataCollectionFactoryInterface $decorated = null,
52	private readonly ?ContainerInterface $filterLocator = null,
53	private readonly ?NameConverterInterface $nameConverter = null,
54	private readonly ?LoggerInterface $logger = null,
55	) {
56	}	588✔
57
58	public function create(string $resourceClass): ResourceMetadataCollection
59	{
60	$resourceMetadataCollection = $this->decorated?->create($resourceClass) ?? new ResourceMetadataCollection($resourceClass);	102✔
61
62	foreach ($resourceMetadataCollection as $i => $resource) {	102✔
63	$operations = $resource->getOperations();	100✔
64
65	$internalPriority = -1;	100✔
66	foreach ($operations as $operationName => $operation) {	100✔
67	$parameters = $this->getDefaultParameters($operation, $resourceClass, $internalPriority);	100✔
68	if (\count($parameters) > 0) {	100✔
69	$operations->add($operationName, $operation->withParameters($parameters));	28✔
70	}
71	}
72
73	$resourceMetadataCollection[$i] = $resource->withOperations($operations->sort());	100✔
74
75	if (!$graphQlOperations = $resource->getGraphQlOperations()) {	100✔
76	continue;	52✔
77	}
78
79	$internalPriority = -1;	72✔
80	foreach ($graphQlOperations as $operationName => $operation) {	72✔
81	$parameters = $this->getDefaultParameters($operation, $resourceClass, $internalPriority);	72✔
82	if (\count($parameters) > 0) {	72✔
83	$graphQlOperations[$operationName] = $operation->withParameters($parameters);	2✔
84	}
85	}
86
87	$resourceMetadataCollection[$i] = $resource->withGraphQlOperations($graphQlOperations);	72✔
88	}
89
90	return $resourceMetadataCollection;	102✔
91	}
92
93	/**
94	* @return array{propertyNames: string[], properties: array<string, ApiProperty>}
95	*/
96	private function getProperties(string $resourceClass, ?Parameter $parameter = null): array
97	{
98	$k = $resourceClass.($parameter?->getProperties() ? ($parameter->getKey() ?? '') : '');	28✔
99	if (isset($this->localPropertyCache[$k])) {	28✔
100	return $this->localPropertyCache[$k];	24✔
101	}
102
103	$propertyNames = [];	28✔
104	$properties = [];	28✔
105	foreach ($parameter?->getProperties() ?? $this->propertyNameCollectionFactory->create($resourceClass) as $property) {	28✔
106	$propertyMetadata = $this->propertyMetadataFactory->create($resourceClass, $property);	24✔
107	if ($propertyMetadata->isReadable()) {	24✔
108	$propertyNames[] = $property;	24✔
109	$properties[$property] = $propertyMetadata;	24✔
110	}
111	}
112
113	$this->localPropertyCache[$k] = ['propertyNames' => $propertyNames, 'properties' => $properties];	28✔
114
115	return $this->localPropertyCache[$k];	28✔
116	}
117
118	private function getDefaultParameters(Operation $operation, string $resourceClass, int &$internalPriority): Parameters
119	{
120	$propertyNames = $properties = [];	100✔
121	$parameters = $operation->getParameters() ?? new Parameters();	100✔
122	foreach ($parameters as $key => $parameter) {	100✔
123	['propertyNames' => $propertyNames, 'properties' => $properties] = $this->getProperties($resourceClass, $parameter);	28✔
124	if (null === $parameter->getProvider() && (($f = $parameter->getFilter()) && $f instanceof ParameterProviderFilterInterface)) {	28✔
125	$parameters->add($key, $parameter->withProvider($f->getParameterProvider()));	×
126	}
127
128	if (':property' === $key) {	28✔
129	foreach ($propertyNames as $property) {	2✔
130	$converted = $this->nameConverter?->denormalize($property) ?? $property;	2✔
131	$propertyParameter = $this->setDefaults($converted, $parameter, $resourceClass, $properties, $operation);	2✔
132	$priority = $propertyParameter->getPriority() ?? $internalPriority--;	2✔
133	$parameters->add($converted, $propertyParameter->withPriority($priority)->withKey($converted));	2✔
134	}
135
136	$parameters->remove($key, $parameter::class);	2✔
137	continue;	2✔
138	}
139
140	$key = $parameter->getKey() ?? $key;	28✔
141
142	if (str_contains($key, ':property') \|\| ((($f = $parameter->getFilter()) && is_a($f, PropertiesAwareInterface::class, true)) \|\| $parameter instanceof PropertiesAwareInterface)) {	28✔
143	$p = [];	10✔
144	foreach ($propertyNames as $prop) {	10✔
145	$p[$this->nameConverter?->denormalize($prop) ?? $prop] = $prop;	8✔
146	}
147
148	$parameter = $parameter->withExtraProperties($parameter->getExtraProperties() + ['_properties' => $p]);	10✔
149	}
150
151	$parameter = $this->setDefaults($key, $parameter, $resourceClass, $properties, $operation);	28✔
152	// We don't do any type cast yet, a query parameter or an header is always a string or a list of strings
153	if (null === $parameter->getNativeType()) {	28✔
154	// this forces the type to be only a list
155	if ('array' === ($parameter->getSchema()['type'] ?? null)) {	24✔
156	$parameter = $parameter->withNativeType(Type::list(Type::string()));	2✔
157	} elseif ('string' === ($parameter->getSchema()['type'] ?? null)) {	24✔
158	$parameter = $parameter->withNativeType(Type::string());	12✔
159	} elseif ('boolean' === ($parameter->getSchema()['type'] ?? null)) {	16✔
160	$parameter = $parameter->withNativeType(Type::bool());	2✔
161	} else {
162	$parameter = $parameter->withNativeType(Type::union(Type::string(), Type::list(Type::string())));	14✔
163	}
164	}
165
166	$priority = $parameter->getPriority() ?? $internalPriority--;	28✔
167	$parameters->add($key, $parameter->withPriority($priority));	28✔
168	}
169
170	return $parameters;	100✔
171	}
172
173	private function addFilterMetadata(Parameter $parameter): Parameter
174	{
175	if (!($filterId = $parameter->getFilter())) {	28✔
176	return $parameter;	12✔
177	}
178
179	if (!\is_object($filterId) && !$this->filterLocator->has($filterId)) {	20✔
180	return $parameter;	×
181	}
182
183	$filter = \is_object($filterId) ? $filterId : $this->filterLocator->get($filterId);	20✔
184
185	if ($filter instanceof ParameterProviderFilterInterface) {	20✔
186	$parameter = $parameter->withProvider($filter::getParameterProvider());	×
187	}
188
189	if (!$filter) {	20✔
190	return $parameter;	×
191	}
192
193	if (null === $parameter->getSchema() && $filter instanceof JsonSchemaFilterInterface && $schema = $filter->getSchema($parameter)) {	20✔
194	$parameter = $parameter->withSchema($schema);	18✔
195	}
196
197	if (null === $parameter->getOpenApi() && $filter instanceof OpenApiParameterFilterInterface && ($openApiParameter = $filter->getOpenApiParameters($parameter))) {	20✔
198	$parameter = $parameter->withOpenApi($openApiParameter);	10✔
199	}
200
201	return $parameter;	20✔
202	}
203
204	/**
205	* @param array<string, ApiProperty> $properties
206	*/
207	private function setDefaults(string $key, Parameter $parameter, string $resourceClass, array $properties, Operation $operation): Parameter
208	{
209	if (null === $parameter->getKey()) {	28✔
210	$parameter = $parameter->withKey($key);	28✔
211	}
212
213	$filter = $parameter->getFilter();	28✔
214	if (\is_string($filter) && $this->filterLocator->has($filter)) {	28✔
215	$filter = $this->filterLocator->get($filter);	6✔
216	}
217
218	if ($filter instanceof SerializerFilterInterface && null === $parameter->getProvider()) {	28✔
219	$parameter = $parameter->withProvider('api_platform.serializer.filter_parameter_provider');	2✔
220	}
221	$currentKey = $key;	28✔
222	if (null === $parameter->getProperty() && isset($properties[$key])) {	28✔
223	$parameter = $parameter->withProperty($key);	16✔
224	}
225
226	if (null === $parameter->getProperty() && $this->nameConverter && ($nameConvertedKey = $this->nameConverter->normalize($key)) && isset($properties[$nameConvertedKey])) {	28✔
227	$parameter = $parameter->withProperty($key)->withExtraProperties(['_query_property' => $nameConvertedKey] + $parameter->getExtraProperties());	×
228	$currentKey = $nameConvertedKey;	×
229	}
230
231	if ($this->nameConverter && $property = $parameter->getProperty()) {	28✔
232	$parameter = $parameter->withProperty($this->nameConverter->normalize($property));	20✔
233	}
234
235	if (isset($properties[$currentKey]) && ($eloquentRelation = ($properties[$currentKey]->getExtraProperties()['eloquent_relation'] ?? null)) && isset($eloquentRelation['foreign_key'])) {	28✔
236	$parameter = $parameter->withExtraProperties(['_query_property' => $eloquentRelation['foreign_key']] + $parameter->getExtraProperties());	×
237	}
238
239	$parameter = $this->addFilterMetadata($parameter);	28✔
240
241	if ($filter instanceof FilterInterface) {	28✔
242	try {
243	return $this->getLegacyFilterMetadata($parameter, $operation, $filter);	20✔
244	} catch (RuntimeException $exception) {	12✔
245	$this->logger?->alert($exception->getMessage(), ['exception' => $exception]);	12✔
246
247	return $parameter;	12✔
248	}
249	}
250
251	return $parameter;	12✔
252	}
253
254	private function getLegacyFilterMetadata(Parameter $parameter, Operation $operation, FilterInterface $filter): Parameter
255	{
256	$description = $filter->getDescription($this->getStateOptionsClass($operation, $operation->getClass()));	20✔
257	$key = $parameter->getKey();	8✔
258	if (($schema = $description[$key]['schema'] ?? null) && null === $parameter->getSchema()) {	8✔
259	$parameter = $parameter->withSchema($schema);	×
260	}
261
262	if (null === $parameter->getProperty() && ($property = $description[$key]['property'] ?? null)) {	8✔
263	$parameter = $parameter->withProperty($property);	×
264	}
265
266	if (null === $parameter->getRequired() && ($required = $description[$key]['required'] ?? null)) {	8✔
267	$parameter = $parameter->withRequired($required);	×
268	}
269
270	if (null === $parameter->getOpenApi() && ($openApi = $description[$key]['openapi'] ?? null) && $openApi instanceof OpenApiParameter) {	8✔
271	$parameter = $parameter->withOpenApi($openApi);	×
272	}
273
274	return $parameter;	8✔
275	}
276	}

api-platform / core / 16050929464

Source File Press 'n' to go to next uncovered line, 'b' for previous

Source File
Press 'n' to go to next uncovered line, 'b' for previous