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

Implement a path structure for property constraint violation #11

Open
rodolphocouto opened this issue Jan 15, 2019 · 3 comments
Open

Implement a path structure for property constraint violation #11

rodolphocouto opened this issue Jan 15, 2019 · 3 comments
Milestone
1.x

Comments

@rodolphocouto
Copy link
Member

Actually, the property from ConstraintViolation is a path composed of all nested properties separated by . similar to JSONPath, e.g: employee.dependents[0].name.

This structure is not type-safe and it is not flexible. An important improvement would be to implement a linked list or a recursive structure to store each property and its respective parent and convert to JSONPath only when it is necessary, like JSON payloads.

A good inspiration for this feature is Path interface from Bean Validation API.
@rodolphocouto rodolphocouto added this to the 1.x milestone Jun 3, 2019
@miguelhrocha
Copy link

Hey @rodolphocouto, I was wondering if you need some help with that. Can I take care of this?

@rodolphocouto
Copy link
Member Author

Hi @miguelhrocha, sorry for my delay. I'd appreciate some help here, are you still interesting on that?

@miguelhrocha
Copy link

Hey @rodolphocouto , sure I'll take care of it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Milestone
1.x
Development

No branches or pull requests
2 participants
@miguelhrocha @rodolphocouto