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
(MODULES-2245) Fixes issue with assumption of strings #122
Conversation
@@ -36,7 +36,9 @@ | |||
){ | |||
sqlserver_validate_instance_name($instance) | |||
validate_re($config_name,'^\w+') | |||
validate_re($value,'^\d+$', "Value for ${config_name}, for instance ${instance}, must be a integer value, you provided ${value}") | |||
if !is_integer($value) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
validate_re always requires a string, so instead we are now using the is_integer method from stdlib and validating this way to avoid casting it to a string just to check if it is an integer
- Puppet < 4 casts numbers to string, now they remain numeric this is to hand that case for next release of Puppet Enterprise
Minor nit with commit - |
Second nit with commit - need an empty line between summary and message. |
fixed commit messages |
please push :) |
We can't even pass lint for our licensing!!
|
We need to add the lint bits in there. Perhaps as another issue . |
(MODULES-2245) Fixes issue with assumption of strings
Darn it, meant to fix up the commit message prior to that merge. |
this is to handle that case for next release of Puppet Enterprise