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.

Sign up for GitHub

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

Jump to bottom

Missing RegisterForReflection import when array schema in the spec #6736

Closed
zhfeng opened this issue Nov 1, 2024 · 0 comments
Closed

Missing RegisterForReflection import when array schema in the spec #6736

zhfeng opened this issue Nov 1, 2024 · 0 comments
Assignees
@zhfeng
Labels
bug Something isn't working
Milestone
3.17.0

Comments

@zhfeng
Copy link
Contributor

zhfeng commented Nov 1, 2024

Bug description

When there is any component schemas of the array type are present in the spec like:

      "Pets": {
        "type": "array",
        "maxItems": 100,
        "items": {
          "$ref": "#/components/schemas/Pet"
        }
      },

The generated Pets.java is missing RegisterForReflection import.
And I think it is no need to add such annotion on the array type of Class.
@zhfeng zhfeng added the bug Something isn't working label Nov 1, 2024
@zhfeng zhfeng self-assigned this Nov 1, 2024
zhfeng added a commit to zhfeng/camel-quarkus that referenced this issue Nov 1, 2024
@zhfeng
Fix apache#6736 no need to add RegisterForReflection annotation on ar…
8afddb8 
…ray type Class
@zhfeng zhfeng closed this as completed in 0c2a534 Nov 1, 2024
@github-actions github-actions bot added this to the 3.17.0 milestone Nov 1, 2024
zhfeng added a commit to zhfeng/camel-quarkus that referenced this issue Nov 1, 2024
@zhfeng
Fix apache#6736 no need to add RegisterForReflection annotation on ar…
946f125 
…ray type Class (apache#6737)
zhfeng added a commit that referenced this issue Nov 1, 2024
@zhfeng
Fix #6736 no need to add RegisterForReflection annotation on array ty…
8fb4f13 
…pe Class (#6737) (#6738)
JiriOndrusek pushed a commit to JiriOndrusek/camel-quarkus that referenced this issue Nov 4, 2024
@zhfeng @JiriOndrusek
Fix apache#6736 no need to add RegisterForReflection annotation on ar…
bc5f639 
…ray type Class (apache#6737) (apache#6738)
JiriOndrusek pushed a commit to jboss-fuse/camel-quarkus that referenced this issue Nov 6, 2024
@zhfeng @JiriOndrusek
Fix apache#6736 no need to add RegisterForReflection annotation on ar…
747af66 
…ray type Class (apache#6737) (apache#6738)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@zhfeng zhfeng

Labels
bug Something isn't working
Projects
None yet
Milestone
3.17.0
Development

No branches or pull requests
1 participant
@zhfeng