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

8531 refactor validators #8534

Merged
merged 10 commits into from
Apr 11, 2022
Merged

8531 refactor validators #8534

merged 10 commits into from
Apr 11, 2022

Commits on Mar 24, 2022

  1. refactor(validation): move email, url and username validation into pa… 

    …ckage IQSS#8531
    @poikilotherm
    poikilotherm committed Mar 24, 2022
    Configuration menu
    Copy the full SHA
    b1d702c View commit details
    Browse the repository at this point in the history

  2. build(deps): add necessary deps for bean validation IQSS#8531

    @poikilotherm
    poikilotherm committed Mar 24, 2022
    Configuration menu
    Copy the full SHA
    b77b7a3 View commit details
    Browse the repository at this point in the history

  3. refactor(validation): reshape EMailValidator and add tests IQSS#8531

    @poikilotherm
    poikilotherm committed Mar 24, 2022
    Configuration menu
    Copy the full SHA
    c1a0364 View commit details
    Browse the repository at this point in the history

  4. refactor(validation): change EMailValidator users sending null context … 

    …IQSS#8531
    @poikilotherm
    poikilotherm committed Mar 24, 2022
    Configuration menu
    Copy the full SHA
    2d82f84 View commit details
    Browse the repository at this point in the history

  5. fix(validation): integrate refactored EMailValidator with DatasetFiel… 

    …dValueValidator IQSS#8503

The context parameter has been removed, need to add the violation
message ourselfs. Also adding a true test for email validation.
    @poikilotherm
    poikilotherm committed Mar 24, 2022
    Configuration menu
    Copy the full SHA
    237d5ba View commit details
    Browse the repository at this point in the history

  6. refactor(validation): simplify ValidateURL IQSS#8531 

    - Simplify the code for URLValidator
- Make it nullsafe
- Make allowed schemes configurable from annotation
- Rewrite tests to JUnit5, more examples and test with real subject
  classes
- Move message string to validation bundle
    @poikilotherm
    poikilotherm committed Mar 24, 2022
    Configuration menu
    Copy the full SHA
    223d5a2 View commit details
    Browse the repository at this point in the history

  7. refactor(validation): cleanup UserNameValidator IQSS#8531 

    - Reuse existing constraints instead of creating our own validator
- Provide programmatic method to use the same from code like OAuth2 etc
- Add much more tests, also for actual bean validation
    @poikilotherm
    poikilotherm committed Mar 24, 2022
    Configuration menu
    Copy the full SHA
    85dcbbf View commit details
    Browse the repository at this point in the history

  8. refactor(validation): cleanup DatasetFieldValueValidator URL validation 

    IQSS#8531
    @poikilotherm
    poikilotherm committed Mar 24, 2022
    Configuration menu
    Copy the full SHA
    9589d3f View commit details
    Browse the repository at this point in the history

Commits on Mar 25, 2022

  1. fix(groups): replace invalid Hibernate CollectionHelper usage with JD… 

    …K set generator IQSS#8531
    @poikilotherm
    poikilotherm committed Mar 25, 2022
    Configuration menu
    Copy the full SHA
    43126e1 View commit details
    Browse the repository at this point in the history

Commits on Apr 10, 2022

  1. Merge branch 'develop' into 8531-refactor-validators

    @poikilotherm
    poikilotherm committed Apr 10, 2022
    Configuration menu
    Copy the full SHA
    f9c326f View commit details
    Browse the repository at this point in the history