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

Control layer group validation: @Validated(Save.class), the model cannot be correctly adapted. #243

Closed
zhouping-git opened this issue Apr 22, 2022 · 4 comments · Fixed by #246
Labels
enhancement New feature or request

Comments

@zhouping-git
Copy link

zhouping-git commented Apr 22, 2022

Summary(关于这个pr的描述)

Entity classes cannot generate validation rules based on the control layer's grouping strategy.

Basic example(pr的用例)

image
image
image

Motivation(提出这个pr目的)

Is there a plan to adapt to this requirement?

@zhouping-git zhouping-git added the enhancement New feature or request label Apr 22, 2022
@chenqi146
Copy link
Collaborator

这是是我测试的结果, 目前是没有问题的, 你的smart-doc版本是多少
image
image
image

chenqi146 added a commit to smart-doc-group/smart-doc-example-cn that referenced this issue Apr 23, 2022
@zhouping-git
Copy link
Author

zhouping-git commented Apr 24, 2022

@chenqi146
After testing, it was found that the generated HTML does implement grouped validation, but this is not achieved using the OpenAPI approach.
Version used:
image
In the generated OpenAPI file:
image
image
image
The same schema is used for both create and update operations.

Comparison of HTML and OpenAPI:
image
image
image
image

chenqi146 added a commit to smart-doc-group/smart-doc-example-cn that referenced this issue Apr 24, 2022
@chenqi146 chenqi146 linked a pull request Apr 24, 2022 that will close this issue
@zhouping-git
Copy link
Author

zhouping-git commented Oct 11, 2022 via email

@kezhiyang
Copy link

kezhiyang commented Jun 12, 2023

2.6.8 still has the same issue

@shalousun shalousun changed the title 控制层分组校验:@Validated(Save.class),model中无法正确适配 Control layer group validation: @Validated(Save.class), the model cannot be correctly adapted. Jul 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants