Skip to content

Commit 29558f2

Browse files
ragboyjrragboyjr
committed
Expose Pagination in Swagger UI
fixes #2161 Signed-off-by: RJ Garcia <[email protected]>
1 parent 3fdb58c commit 29558f2

File tree

3 files changed

+123
-1
lines changed

3 files changed

+123
-1
lines changed

‎src/Bridge/Symfony/Bundle/Resources/config/swagger.xml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,8 @@
2828
<argument>%api_platform.collection.pagination.page_parameter_name%</argument>
2929
<argument>%api_platform.collection.pagination.client_items_per_page%</argument>
3030
<argument>%api_platform.collection.pagination.items_per_page_parameter_name%</argument>
31+
<argument>%api_platform.collection.pagination.client_enabled%</argument>
32+
<argument>%api_platform.collection.pagination.enabled_parameter_name%</argument>
3133
<tag name="serializer.normalizer" priority="16" />
3234
</service>
3335

‎src/Swagger/Serializer/DocumentationNormalizer.php

Lines changed: 22 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -68,11 +68,13 @@ final class DocumentationNormalizer implements NormalizerInterface, CacheableSup
6868
private $paginationPageParameterName;
6969
private $clientItemsPerPage;
7070
private $itemsPerPageParameterName;
71+
private $paginationClientEnabled;
72+
private $paginationClientEnabledParameterName;
7173

7274
/**
7375
* @param ContainerInterface|FilterCollection|null $filterLocator The new filter locator or the deprecated filter collection
7476
*/
75-
public function __construct(ResourceMetadataFactoryInterface $resourceMetadataFactory, PropertyNameCollectionFactoryInterface $propertyNameCollectionFactory, PropertyMetadataFactoryInterface $propertyMetadataFactory, ResourceClassResolverInterface $resourceClassResolver, OperationMethodResolverInterface $operationMethodResolver, OperationPathResolverInterface $operationPathResolver, UrlGeneratorInterface $urlGenerator = null, $filterLocator = null, NameConverterInterface $nameConverter = null, $oauthEnabled = false, $oauthType = '', $oauthFlow = '', $oauthTokenUrl = '', $oauthAuthorizationUrl = '', array $oauthScopes = [], array $apiKeys = [], SubresourceOperationFactoryInterface $subresourceOperationFactory = null, $paginationEnabled = true, $paginationPageParameterName = 'page', $clientItemsPerPage = false, $itemsPerPageParameterName = 'itemsPerPage')
77+
public function __construct(ResourceMetadataFactoryInterface $resourceMetadataFactory, PropertyNameCollectionFactoryInterface $propertyNameCollectionFactory, PropertyMetadataFactoryInterface $propertyMetadataFactory, ResourceClassResolverInterface $resourceClassResolver, OperationMethodResolverInterface $operationMethodResolver, OperationPathResolverInterface $operationPathResolver, UrlGeneratorInterface $urlGenerator = null, $filterLocator = null, NameConverterInterface $nameConverter = null, $oauthEnabled = false, $oauthType = '', $oauthFlow = '', $oauthTokenUrl = '', $oauthAuthorizationUrl = '', array $oauthScopes = [], array $apiKeys = [], SubresourceOperationFactoryInterface $subresourceOperationFactory = null, $paginationEnabled = true, $paginationPageParameterName = 'page', $clientItemsPerPage = false, $itemsPerPageParameterName = 'itemsPerPage', $paginationClientEnabled = false, $paginationClientEnabledParameterName = 'pagination')
7678
{
7779
if ($urlGenerator) {
7880
@trigger_error(sprintf('Passing an instance of %s to %s() is deprecated since version 2.1 and will be removed in 3.0.', UrlGeneratorInterface::class, __METHOD__), E_USER_DEPRECATED);
@@ -100,6 +102,8 @@ public function __construct(ResourceMetadataFactoryInterface $resourceMetadataFa
100102
$this->subresourceOperationFactory = $subresourceOperationFactory;
101103
$this->clientItemsPerPage = $clientItemsPerPage;
102104
$this->itemsPerPageParameterName = $itemsPerPageParameterName;
105+
$this->paginationClientEnabled = $paginationClientEnabled;
106+
$this->paginationClientEnabledParameterName = $paginationClientEnabledParameterName;
103107
}
104108

105109
/**
@@ -275,6 +279,9 @@ private function updateGetOperation(\ArrayObject $pathOperation, array $mimeType
275279
$pathOperation['parameters'][] = $this->getItemsPerPageParameters();
276280
}
277281
}
282+
if ($this->paginationEnabled && $resourceMetadata->getCollectionOperationAttribute($operationName, 'pagination_client_enabled', $this->paginationClientEnabled, true)) {
283+
$pathOperation['parameters'][] = $this->getPaginationClientEnabledParameters();
284+
}
278285

279286
return $pathOperation;
280287
}
@@ -635,6 +642,20 @@ private function getPaginationParameters(): array
635642
];
636643
}
637644

645+
/**
646+
* Returns enable pagination parameter for the "get" collection operation
647+
*/
648+
private function getPaginationClientEnabledParameters(): array
649+
{
650+
return [
651+
'name' => $this->paginationClientEnabledParameterName,
652+
'in' => 'query',
653+
'required' => false,
654+
'type' => 'boolean',
655+
'description' => 'Enable or disable pagination',
656+
];
657+
}
658+
638659
/**
639660
* Returns items per page parameters for the "get" collection operation.
640661
*/

‎tests/Swagger/Serializer/DocumentationNormalizerTest.php

Lines changed: 99 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1971,4 +1971,103 @@ public function testNormalizeWithPropertySwaggerContext()
19711971

19721972
$this->assertEquals($expected, $normalizer->normalize($documentation, DocumentationNormalizer::FORMAT, ['base_url' => '/app_dev.php/']));
19731973
}
1974+
1975+
public function testNormalizeWithPaginationClientEnabled()
1976+
{
1977+
$documentation = new Documentation(new ResourceNameCollection([Dummy::class]), 'Test API', 'This is a test API.', '1.2.3', ['jsonld' => ['application/ld+json']]);
1978+
1979+
$propertyNameCollectionFactoryProphecy = $this->prophesize(PropertyNameCollectionFactoryInterface::class);
1980+
$propertyNameCollectionFactoryProphecy->create(Dummy::class, [])->shouldBeCalled()->willReturn(new PropertyNameCollection(['id', 'name']));
1981+
1982+
$dummyMetadata = new ResourceMetadata('Dummy', 'This is a dummy.', 'http://schema.example.com/Dummy', [], ['get' => ['method' => 'GET', 'pagination_client_enabled' => true]]);
1983+
$resourceMetadataFactoryProphecy = $this->prophesize(ResourceMetadataFactoryInterface::class);
1984+
$resourceMetadataFactoryProphecy->create(Dummy::class)->shouldBeCalled()->willReturn($dummyMetadata);
1985+
1986+
$propertyMetadataFactoryProphecy = $this->prophesize(PropertyMetadataFactoryInterface::class);
1987+
$propertyMetadataFactoryProphecy->create(Dummy::class, 'id')->shouldBeCalled()->willReturn(new PropertyMetadata(new Type(Type::BUILTIN_TYPE_INT), 'This is an id.', true, false));
1988+
$propertyMetadataFactoryProphecy->create(Dummy::class, 'name')->shouldBeCalled()->willReturn(new PropertyMetadata(new Type(Type::BUILTIN_TYPE_STRING), 'This is a name.', true, true, true, true, false, false, null, null, ['swagger_context' => ['type' => 'string', 'enum' => ['one', 'two'], 'example' => 'one']]));
1989+
$resourceClassResolverProphecy = $this->prophesize(ResourceClassResolverInterface::class);
1990+
$resourceClassResolverProphecy->isResourceClass(Dummy::class)->willReturn(true);
1991+
1992+
$operationMethodResolverProphecy = $this->prophesize(OperationMethodResolverInterface::class);
1993+
$operationMethodResolverProphecy->getCollectionOperationMethod(Dummy::class, 'get')->shouldBeCalled()->willReturn('GET');
1994+
1995+
$operationPathResolver = new CustomOperationPathResolver(new OperationPathResolver(new UnderscorePathSegmentNameGenerator()));
1996+
1997+
$normalizer = new DocumentationNormalizer(
1998+
$resourceMetadataFactoryProphecy->reveal(),
1999+
$propertyNameCollectionFactoryProphecy->reveal(),
2000+
$propertyMetadataFactoryProphecy->reveal(),
2001+
$resourceClassResolverProphecy->reveal(),
2002+
$operationMethodResolverProphecy->reveal(),
2003+
$operationPathResolver
2004+
);
2005+
2006+
$expected = [
2007+
'swagger' => '2.0',
2008+
'basePath' => '/app_dev.php/',
2009+
'info' => [
2010+
'title' => 'Test API',
2011+
'description' => 'This is a test API.',
2012+
'version' => '1.2.3',
2013+
],
2014+
'paths' => new \ArrayObject([
2015+
'/dummies' => [
2016+
'get' => new \ArrayObject([
2017+
'tags' => ['Dummy'],
2018+
'operationId' => 'getDummyCollection',
2019+
'produces' => ['application/ld+json'],
2020+
'summary' => 'Retrieves the collection of Dummy resources.',
2021+
'parameters' => [
2022+
[
2023+
'name' => 'page',
2024+
'in' => 'query',
2025+
'required' => false,
2026+
'type' => 'integer',
2027+
'description' => 'The collection page number',
2028+
],
2029+
[
2030+
'name' => 'pagination',
2031+
'in' => 'query',
2032+
'required' => false,
2033+
'type' => 'boolean',
2034+
'description' => 'Enable or disable pagination',
2035+
]
2036+
],
2037+
'responses' => [
2038+
200 => [
2039+
'description' => 'Dummy collection response',
2040+
'schema' => [
2041+
'type' => 'array',
2042+
'items' => ['$ref' => '#/definitions/Dummy'],
2043+
],
2044+
],
2045+
],
2046+
]),
2047+
],
2048+
]),
2049+
'definitions' => new \ArrayObject([
2050+
'Dummy' => new \ArrayObject([
2051+
'type' => 'object',
2052+
'description' => 'This is a dummy.',
2053+
'externalDocs' => ['url' => 'http://schema.example.com/Dummy'],
2054+
'properties' => [
2055+
'id' => new \ArrayObject([
2056+
'type' => 'integer',
2057+
'description' => 'This is an id.',
2058+
'readOnly' => true,
2059+
]),
2060+
'name' => new \ArrayObject([
2061+
'type' => 'string',
2062+
'description' => 'This is a name.',
2063+
'enum' => ['one', 'two'],
2064+
'example' => 'one',
2065+
]),
2066+
],
2067+
]),
2068+
]),
2069+
];
2070+
2071+
$this->assertEquals($expected, $normalizer->normalize($documentation, DocumentationNormalizer::FORMAT, ['base_url' => '/app_dev.php/']));
2072+
}
19742073
}

0 commit comments

Comments
 (0)