Skip to content

[Jaxrs-cxf] Add method-level cascaded beanvalidation (@Valid)#4921

Merged
wing328 merged 3 commits intoswagger-api:masterfrom
jfiala:cxf_beanval_valid
Mar 23, 2017
Merged

[Jaxrs-cxf] Add method-level cascaded beanvalidation (@Valid)#4921
wing328 merged 3 commits intoswagger-api:masterfrom
jfiala:cxf_beanval_valid

Conversation

@jfiala
Copy link
Copy Markdown
Contributor

@jfiala jfiala commented Mar 4, 2017

PR checklist

  • Read the contribution guildelines.
  • Ran the shell/batch script under ./bin/ to update Petstore sample so that CIs can verify the change. (For instance, only need to run ./bin/{LANG}-petstore.sh and ./bin/security/{LANG}-petstore.sh if updating the {LANG} (e.g. php, ruby, python, etc) code generator or {LANG} client's mustache templates)
  • Filed the PR against the correct branch: master for non-breaking changes and 2.3.0 branch for breaking (non-backward compatible) changes.

Description of the PR

Added @Valid for the request body for CXF #4847

public void addPetTest() {
Pet body = null;
//api.addPet(body);
//api.addPet(body);
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tab instead of 4-space is used. I'll file a PR to fix it.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Filed #5174

@wing328 wing328 added this to the v2.2.3 milestone Mar 23, 2017
@wing328 wing328 merged commit 43aa4a8 into swagger-api:master Mar 23, 2017
@ePaul ePaul mentioned this pull request Mar 26, 2017
3 tasks
spr3nk3ls pushed a commit to spr3nk3ls/swagger-codegen that referenced this pull request Mar 28, 2017
…r-api#4921)

* add ApiResponse/s to operation swagger-api#4718

* add method-level cascaded beanvalidation swagger-api#4847
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants