gitgabrio commented on issue #2150: URL: https://github.com/apache/incubator-kie-issues/issues/2150#issuecomment-3421527224
With the suspected invocation `org.kie.kogito.dmn.rest.DMNJSONUtils.ctx(DecisionModel, Map<String, Object>)` the context contains one `targetPrice` value (correctly populated) and another `target price` null, because it is the one declared in dmn but without the FEELPRoperty resolution. But when I replaced that invocation with the "simpler" one `modelInstance.newContext(jsonResolver.resolveAll(getInputs(rsni)));` the code path is ` QualifiedNameNode.evaluate(EvaluationContext)` -> `EvalHelper.getValue(Object, String)` -> `EvalHelper.getDefinedValue(Object, String)` that, again, ignores the FEELProperty, and keep `target price`as null. Following operations returns `null`, but all of that is swallowed up and there is not even an error message, with the decisions being marked as successful. Comparing with our tests that uses POJO, I see that actual POJOs are set as value, while in the above ways those are converted to Map, and then not dealt with as java bean. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
