Merge pull request #1109 from dhozac/fix-varReplace-repr

Fix only_if statements referencing non-string types

lib/ansible/utils.py

@@ -269,7 +269,8 @@ def varReplace(raw, vars, do_repr=False, depth=0):
         # original)
 
         try:
-            replacement = unicode(_varLookup(m.group(2), vars, depth))
+            replacement = _varLookup(m.group(2), vars, depth)
+            if isinstance(replacement, (str, unicode)):
                 replacement = varReplace(replacement, vars, depth=depth + 1)
         except VarNotFoundException:
             replacement = m.group()
@@ -283,7 +284,7 @@ def varReplace(raw, vars, do_repr=False, depth=0):
                 start -= 1
                 end += 1
         done.append(raw[:start])          # Keep stuff leading up to token
-        done.append(replacement)    # Append replacement value
+        done.append(unicode(replacement)) # Append replacement value
         raw = raw[end:]                   # Continue with remainder of string
 
     return ''.join(done)

test/TestUtils.py

@@ -251,6 +251,16 @@ class TestUtils(unittest.TestCase):
         res = ansible.utils.varReplace(template, vars, do_repr=True)
         assert eval(res)
 
+    def test_varReplace_repr_nonstr(self):
+        vars = {
+            'foo': True,
+            'bar': 1L,
+        }
+
+        template = '${foo} == $bar'
+        res = ansible.utils.varReplace(template, vars, do_repr=True)
+        assert res == 'True == 1L'
+
     def test_template_varReplace_iterated(self):
         template = 'hello $who'
         vars = {