SearchFilter: Allow strategy on associations #1
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
jpierront
force-pushed
the
jpierront-patch-1
branch
from
c528404
to
e34fbc4
Compare
|
I'm not sure of what I did for the MongoDbOdm part ;-) |
jpierront
force-pushed
the
jpierront-patch-1
branch
10 times, most recently
from
c407e20
to
d6cad23
Compare
Merge 2.5 onto master
* Fix mising @type in output DTO collections * Fix tests * Add a new test * Add tests
* Add possibility to hydrate input before denormalize * Fix @soyuka review * Add test * Remove useless set context * Add deep object to populate to avoid missing data * change name to inputClass to be more explicit * Fix test * Rename PreHydrateInput to DataTransformerInitializer * Add behat test data transformer initializer * fix tests Co-authored-by: Alexandre Vinet <contact@alexandrevinet.fr>
* Add files to test client * Respect method signature in favor of interface * Add missing default * Add request parameters to client. Co-authored-by: Ashura <CoalaJoe@users.noreply.github.com>
Add stateless ApiResource attribute
* Add messenger=persist * Add unit tests * Add functional tests * Add test with Mongo * Add dunglas review * Rename LoopDataPersisterInterface in HandOverDataPersisterInterface * Inject DataPersister in Messenger DataPersister * Update unit tests
Argument would always be ignored as null coalescing operator has a higher precedence than ternary operator
…e-listener-quality fix: simplify PurgeHttpCacheListener
Fix expression language argument for PublishMercureUpdatesListener
* 2.5: fix: simplify PurgeHttpCacheListener Fix expression language argument for PublishMercureUpdatesListener Backport pull request api-platform#3486 (api-platform#3723) Fix mising @type in output DTO collections (api-platform#3699)
fix: update JS assets
…platform#4052) * Fix api-platform#4037 allow POST operations without identifiers * Better * remove line * Update features/main/overridden_operation.feature Co-authored-by: Alan Poulain <contact@alanpoulain.eu> * fix review * Fix mongodb Co-authored-by: Alan Poulain <contact@alanpoulain.eu>
…ier handling (api-platform#4042) * Issue api-platform#4041 bugfix: FilterEagerLoadingExtension foreign identifier handling * test: add a unit test and use a foreign id for DummyCar * chore: add changelog entry Co-authored-by: Alan Poulain <contact@alanpoulain.eu>
* fix: deprecations in Symfony 5.3 * fix more deprecations * fix remaining deprecs * fix phpstan * fix phpstan * fix phpstan
* Fix deprecations * Do not load the class
After disabling itemOperation `GET`
```
itemOperations: [
'get' => [
'controller' => ApiPlatform\Core\Action\NotFoundAction::class,
'read' => false,
'output' => false,
],
]
```
there is not nice way to remove that path from OpenApi docs. Now to do
that you have to write:
```
$pathItem = $openApi->getPaths()->getPath('/resource/{id}');
$openApi->getPaths()->addPath(
'/resource/{id}',
new Model\PathItem(
$pathItem->getRef(),
$pathItem->getSummary(),
$pathItem->getDescription(),
null,
$pathItem->getPut(),
$pathItem->getPost(),
$pathItem->getDelete(),
$pathItem->getOptions(),
$pathItem->getHead(),
$pathItem->getPatch(),
$pathItem->getTrace(),
$pathItem->getServers(),
$pathItem->getParameters(),
)
);
```
After change it will be:
```
$pathItem = $openApi->getPaths()->getPath('/resource/{id}');
$openApi->getPaths()->addPath(
'/resource/{id}',
$pathItem->withGet(null)
);
```
Allow unset PathItem method
Hello, i just add the return type for method `getPaths` Co-authored-by: Abdouni Abdelkarim <abdounikarim@gmail.com>
alanpoulain
force-pushed
the
jpierront-patch-1
branch
from
d6cad23
to
ec51013
Compare
…asier to add new strategies to classes inheriting from SearchFilter class
alanpoulain
force-pushed
the
jpierront-patch-1
branch
from
ec51013
to
70e2893
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Also makes it easier to add new strategies to classes inheriting from SearchFilter class