JsonQueryFunction always defaults to rootScope and fails when alternative Scope is required.

[nickbreentvnz]

When loading all macros, they are all created without specifying a Scope.

jackson-jq/jackson-jq/src/main/java/net/thisptr/jackson/jq/Scope.java

Line 211 in 0cf9166

addFunction(def.name, def.args.size(), new JsonQueryFunction(def.name, def.args, JsonQuery.compile(def.body)));

Which is a call to:

jackson-jq/jackson-jq/src/main/java/net/thisptr/jackson/jq/internal/JsonQueryFunction.java

Lines 28 to 30 in 0cf9166

	public JsonQueryFunction(final String name, final List<String> params, final JsonQuery body) {
	this(name, params, body, Scope.rootScope());
	}

... which uses Scope.rootScope() which is not guaranteed to be the same scope as the query has been compiled with.

My hunch is that Scope.loadMacros just needs to call new JsonQueryFunction with this as the closure scope, though I've yet to prove it or encode it in a unit test.

addFunction(def.name, def.args.size(), new JsonQueryFunction(def.name, def.args, JsonQuery.compile(def.body), this));

[eiiches]

Thanks. I think you are right.

Fixed in 41bda39.

[eiiches]

Fix released in 0.0.9.
https://github.com/eiiches/jackson-jq/releases/tag/jackson-jq-parent-0.0.9