Skip to content
This repository has been archived by the owner on Feb 20, 2022. It is now read-only.

range check should accept empty value #51

Closed
mkoertgen opened this issue Feb 16, 2015 · 2 comments · Fixed by #54
Closed

range check should accept empty value #51

mkoertgen opened this issue Feb 16, 2015 · 2 comments · Fixed by #54
Labels
bug
Milestone
v0.2

Comments

@mkoertgen
Copy link
Member

empty fields are handled by the RequiredCheck.

cf.: https://github.com/ThomasMentzel/NZazu/blob/master/NZazu.Contracts/Checks/RangeCheck_Should.cs#L52
@mkoertgen mkoertgen self-assigned this Feb 16, 2015
@mkoertgen mkoertgen added this to the v0.2 milestone Feb 16, 2015
@mkoertgen mkoertgen added the bug label Feb 16, 2015
@mkoertgen
Copy link
Member Author

The same holds for StringRegExCheck

@mkoertgen
Copy link
Member Author

And for StringLengthCheck as well.

mkoertgen pushed a commit that referenced this issue Feb 16, 2015
#51 changed checks to accept null or whitespace
e9e91f5
@mkoertgen mkoertgen mentioned this issue Feb 16, 2015
Merged
mkoertgen added a commit that referenced this issue Feb 16, 2015
@mkoertgen
Merge pull request #54 from ThomasMentzel/51_range_check_should_accep…
d80d40e 
…t_empty_value

#51 changed checks to accept null or whitespace
@mkoertgen mkoertgen removed their assignment Mar 12, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
v0.2
Development

Successfully merging a pull request may close this issue.

#51 changed checks to accept null or whitespace awesome-inc/NZazu
1 participant
@mkoertgen