mirror of
https://github.com/golang/go
synced 2024-09-29 12:04:28 -06:00
text/template: check final value for short-circuit and/or
There was a bug in the short-circuit code for and/or added in CL 321490: it ignored the value passed in by an earlier pipeline. For #31103 Change-Id: Ic31f4d7cedfe563ef968cbb712ecfb2413c42eb5 Reviewed-on: https://go-review.googlesource.com/c/go/+/353130 Trust: Ian Lance Taylor <iant@golang.org> Run-TryBot: Ian Lance Taylor <iant@golang.org> Reviewed-by: Bryan C. Mills <bcmills@google.com> TryBot-Result: Go Bot <gobot@golang.org>
This commit is contained in:
parent
e48cf0db4e
commit
82ac9ab83a
@ -721,9 +721,12 @@ func (s *state) evalCall(dot, fun reflect.Value, isBuiltin bool, node parse.Node
|
||||
for _, arg := range args {
|
||||
v = s.evalArg(dot, argType, arg).Interface().(reflect.Value)
|
||||
if truth(v) == (name == "or") {
|
||||
break
|
||||
return v
|
||||
}
|
||||
}
|
||||
if final != missingVal {
|
||||
v = s.validateType(final, argType)
|
||||
}
|
||||
return v
|
||||
}
|
||||
|
||||
|
@ -485,6 +485,10 @@ var execTests = []execTest{
|
||||
{"and short-circuit", "{{and 1 0 (die)}}", "0", nil, true},
|
||||
{"or short-circuit2", "{{or 0 0 (die)}}", "", nil, false},
|
||||
{"and short-circuit2", "{{and 1 1 (die)}}", "", nil, false},
|
||||
{"and pipe-true", "{{1 | and 1}}", "1", nil, true},
|
||||
{"and pipe-false", "{{0 | and 1}}", "0", nil, true},
|
||||
{"or pipe-true", "{{1 | or 0}}", "1", nil, true},
|
||||
{"or pipe-false", "{{0 | or 0}}", "0", nil, true},
|
||||
{"boolean if", "{{if and true 1 `hi`}}TRUE{{else}}FALSE{{end}}", "TRUE", tVal, true},
|
||||
{"boolean if not", "{{if and true 1 `hi` | not}}TRUE{{else}}FALSE{{end}}", "FALSE", nil, true},
|
||||
|
||||
|
Loading…
Reference in New Issue
Block a user