Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Improve the way RequiredParam validates a parameter #79

Merged
merged 1 commit into from Aug 13, 2011

Conversation

Projects
None yet
2 participants
Contributor

rafaelss commented Aug 12, 2011

It changes the regular expression used to check if a parameter value is empty. The current one is not accepting multiline strings that have blank lines inside, like

it have a blank line,\n\n      \n\nbut it's not empty

igrigorik added a commit that referenced this pull request Aug 13, 2011

Merge pull request #79 from rafaelss/fix_required_param_regexp
Improve the way RequiredParam validates a parameter

@igrigorik igrigorik merged commit 1d9cbc9 into postrank-labs:master Aug 13, 2011

Owner

igrigorik commented Aug 13, 2011

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment