Skip to content
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

Improve @Parameter handling #54

Merged
merged 2 commits into from Sep 4, 2021
Merged

Conversation

neiser
Copy link
Collaborator

@neiser neiser commented Sep 4, 2021

Add an example test to show how to use Spring's ConversionService bean to detect if a given parameter should be considered as having type String.

Also fix #44 by providing a common ParameterMethodConverter which creates an empty Parameter model object if an explicit @Parameter annotation is present.

Of course only if the hidden flag is not true (for whatever reason). This
library is somewhat more conservative what parameters it
takes into account and expects library users to be explicit
what extra paramters should be included (and how exactly).
@sonarcloud
Copy link

sonarcloud bot commented Sep 4, 2021

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

100.0% 100.0% Coverage
0.0% 0.0% Duplication

@neiser neiser merged commit 829d5a1 into master Sep 4, 2021
@neiser neiser deleted the pr-improve-parameter-handling branch September 4, 2021 14:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Include parameter in operation if Swagger @Parameter is present
1 participant