suyiiyii/yq
1
0
Fork 0
mirror of https://github.com/mikefarah/yq.git synced 2024-11-12 05:38:04 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fixed npe in log #1596

Browse Source
This commit is contained in:
Mike Farah 2023-03-15 13:57:47 +11:00
parent 08a6cb65fe
commit 360a47fddc
2 changed files with 8 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
pkg/yqlib/operator_variables.go
View File

@ -82,10 +82,11 @@ func variableLoopSingleChild(d *dataTreeNavigator, context Context, originalExp
newContext.SetVariable(variableName, variableValue) newContext.SetVariable(variableName, variableValue)
rhs, err := d.GetMatchingNodes(newContext, originalExp.RHS) rhs, err := d.GetMatchingNodes(newContext, originalExp.RHS)
log.Debug("PROCESSING VARIABLE DONE, got back: ", rhs.MatchingNodes.Len())
if err != nil { if err != nil {
return Context{}, err return Context{}, err
} }
log.Debug("PROCESSING VARIABLE DONE, got back: ", rhs.MatchingNodes.Len())
results.PushBackList(rhs.MatchingNodes) results.PushBackList(rhs.MatchingNodes)
} }

6
pkg/yqlib/operator_variables_test.go
View File

@ -13,6 +13,12 @@ var variableOperatorScenarios = []expressionScenario{
"D0, P[], (doc)::{}\n", "D0, P[], (doc)::{}\n",
}, },
}, },
{
skipDoc: true,
document: `{}`,
expression: `.a.b as $foo`,
expectedError: "must use variable with a pipe, e.g. `exp as $x | ...`",
},
{ {
document: "a: [cat]", document: "a: [cat]",
skipDoc: true, skipDoc: true,