-
-
Notifications
You must be signed in to change notification settings - Fork 838
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(graphql): always allow to query nested resources #5112
fix(graphql): always allow to query nested resources #5112
Conversation
src/GraphQl/Metadata/Factory/GraphQlNestedOperationResourceMetadataFactory.php
Outdated
Show resolved
Hide resolved
@@ -134,6 +133,7 @@ | |||
<argument type="service" id="api_platform.pagination" /> | |||
<argument type="service" id="api_platform.name_converter" on-invalid="ignore" /> | |||
<argument>%api_platform.graphql.nesting_separator%</argument> | |||
<argument type="service" id="api_platform.graphql_metadata.resource.metadata_collection_factory" on-invalid="ignore"></argument> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<argument type="service" id="api_platform.graphql_metadata.resource.metadata_collection_factory" on-invalid="ignore"></argument> | |
<argument type="service" id="api_platform.graphql_metadata.resource.metadata_collection_factory" on-invalid="ignore" /> |
432ff12
to
e38414d
Compare
@@ -133,6 +133,7 @@ | |||
<argument type="service" id="api_platform.pagination" /> | |||
<argument type="service" id="api_platform.name_converter" on-invalid="ignore" /> | |||
<argument>%api_platform.graphql.nesting_separator%</argument> | |||
<argument type="service" id="api_platform.graphql_metadata.resource.metadata_collection_factory" on-invalid="ignore" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure if the on-invalid
is necessary.
e38414d
to
2a75c00
Compare
* fix: update yaml extractor test file coding standard (#5068) * fix(graphql): add clearer error message when TwigBundle is disabled but graphQL clients are enabled (#5064) * fix(metadata): add class key in payload argument resolver (#5067) * fix: add class key in payload argument resolver * add null if everything else goes wrong * fix: upgrade command remove ApiSubresource attribute (#5049) Fixes #5038 * fix(doctrine): use abitrary index instead of value (#5079) * fix: uri template should respect rfc 6570 (#5080) * fix: remove @internal annotation for Operations (#5089) See #5084 * fix(metadata): define a name on a single operation (#5090) fixes #5082 * fix(metadata): deprecate when user decorates in legacy mode (#5091) fixes #5078 * fix(graphql): use right nested operation (#5102) * Revert "fix(graphql): use right nested operation (#5102)" (#5111) This reverts commit 44337dd. * fix(graphql): always allow to query nested resources (#5112) * fix(graphql): always allow to query nested resources * review Co-authored-by: Alan Poulain <contact@alanpoulain.eu> * chore: php-cs-fixer update * fix: only alias if exists for opcache preload Fixes api-platform/api-platform#2284 (#5110) Co-authored-by: Liviu Mirea <liviu.mirea@wecodepixels.com> * chore: php-cs-fixer update (#5118) * chore: php-cs-fixer update * chore: php-cs-fixer update * fix(metadata): upgrade script keep operation name (#5109) origin: #5105 Co-authored-by: WilliamPeralta <william.peralta18@gmail.com> * chore: v2.7.3 changelog * chore: v3.0.3 changelog Co-authored-by: helyakin <CourcierMarvin@gmail.com> Co-authored-by: ArnoudThibaut <thibaut.arnoud@gmail.com> Co-authored-by: davy-beauzil <38990335+davy-beauzil@users.noreply.github.com> Co-authored-by: Baptiste Leduc <baptiste.leduc@gmail.com> Co-authored-by: Xavier Laviron <norival@users.noreply.github.com> Co-authored-by: Alan Poulain <contact@alanpoulain.eu> Co-authored-by: Liviu Cristian Mirea-Ghiban <contact@liviucmg.com> Co-authored-by: Liviu Mirea <liviu.mirea@wecodepixels.com> Co-authored-by: WilliamPeralta <william.peralta18@gmail.com>
if (class_exists($resourceClass)) { | ||
$refl = new \ReflectionClass($resourceClass); | ||
$attribute = $refl->getAttributes(ApiResource::class)[0] ?? null; | ||
$attributeInstance = $attribute?->newInstance(); | ||
if ($filters = $attributeInstance?->getFilters()) { | ||
$apiResource = $apiResource->withFilters($filters); | ||
} | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
issue: This should work with XML/YAML too.
Rework of #5102