fix undef variable check puppet6#480
Conversation
| authenticationMethod="simple" | ||
| forceBindingLogin="<%= @auth_config['ldap']['force_binding'] %>" | ||
| <%- if @auth_config['ldap']['force_binding_use_root'] != :undef -%> | ||
| <%- if !@auth_config['ldap']['force_binding_use_root'].nil? && @auth_config['ldap']['force_binding_use_root'] != :undef -%> |
There was a problem hiding this comment.
Considering force_binding_use_root defaults to false and not undef, I'm not convinced this conditional is needed at all.
There was a problem hiding this comment.
@fe80 I've taken the liberty of pushing a commit onto your branch (something you can turn off when you open a PR) that removes this conditional and adds a test for this ldap property. Hoping you could follow the pattern and add tests for the other properties you've fixed? Feel free to discard/squash it if you want to add more tests or write them in a different way. :)
|
Thanks for the PR! I think I've come across a similar fix in another module before. So we can confirm everything now works correctly (and won't be broken in future), would you be able to write any tests? |
Since _this_ parameter actually defaults to `false` and not `undef` the template doesn't need the conditional.
alexjfisher
force-pushed
the
puppet6_ldap
branch
from
22f507f to
60dd95f
Compare
|
I've had more test and fix a bug (I've miss a |
|
Obsoleted by #520. |
4 participants
Hello,
The variable check of some template not work with puppet6