saltstack/salt — pull request #12275 by jeroennijhof (2014-04-24 22:11:51+00:00)
Title: Causing problems for boolean values from config

I just don't know how I didn't see this but really.....
Value could be a boolean i.e. ldap.tls which is false by default, so using "not" in the if statement isn't correct.
And value will always be filled anyway.