diff --git a/mercurial/templatefuncs.py b/mercurial/templatefuncs.py --- a/mercurial/templatefuncs.py +++ b/mercurial/templatefuncs.py @@ -34,7 +34,6 @@ evalboolean = templateutil.evalboolean evalinteger = templateutil.evalinteger evalstring = templateutil.evalstring evalstringliteral = templateutil.evalstringliteral -evalastype = templateutil.evalastype # dict of template built-in functions funcs = {} @@ -261,9 +260,10 @@ def ifcontains(context, mapping, args): raise error.ParseError(_("ifcontains expects three or four arguments")) haystack = evalfuncarg(context, mapping, args[1]) + keytype = getattr(haystack, 'keytype', None) try: - needle = evalastype(context, mapping, args[0], - getattr(haystack, 'keytype', None) or bytes) + needle = evalrawexp(context, mapping, args[0]) + needle = templateutil.unwrapastype(needle, keytype or bytes) found = (needle in haystack) except error.ParseError: found = False diff --git a/mercurial/templateutil.py b/mercurial/templateutil.py --- a/mercurial/templateutil.py +++ b/mercurial/templateutil.py @@ -77,7 +77,8 @@ class mappable(object): - "{manifest.rev}" Unlike a hybrid, this does not simulate the behavior of the underling - value. Use unwrapvalue() or unwraphybrid() to obtain the inner object. + value. Use unwrapvalue(), unwrapastype(), or unwraphybrid() to obtain + the inner object. """ def __init__(self, gen, key, value, makemap): @@ -340,18 +341,18 @@ def evalstringliteral(context, mapping, thing = func(context, mapping, data) return stringify(thing) -_evalfuncbytype = { - bytes: evalstring, - int: evalinteger, +_unwrapfuncbytype = { + bytes: stringify, + int: unwrapinteger, } -def evalastype(context, mapping, arg, typ): - """Evaluate given argument and coerce its type""" +def unwrapastype(thing, typ): + """Move the inner value object out of the wrapper and coerce its type""" try: - f = _evalfuncbytype[typ] + f = _unwrapfuncbytype[typ] except KeyError: raise error.ProgrammingError('invalid type specified: %r' % typ) - return f(context, mapping, arg) + return f(thing) def runinteger(context, mapping, data): return int(data)