frodovdr/node-red
1
0
Fork 0
mirror of https://github.com/node-red/node-red.git synced 2023-10-10 13:36:53 +02:00
Code Issues Releases Wiki Activity

Deprecate synchronous access to jsonata

Browse Source
This commit is contained in:
Nick O'Leary 2023-03-03 11:43:06 +00:00
parent e7c6178391
commit ca53712ee9
No known key found for this signature in database
GPG Key ID: 4F2157149161A6C9
3 changed files with 54 additions and 34 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
packages/node_modules/@node-red/editor-client/src/js/ui/editors/expression.js vendored
View File

@ -294,7 +294,10 @@
} }
try { try {
var result = expr.evaluate(legacyMode?{msg:parsedData}:parsedData); expr.evaluate(legacyMode?{msg:parsedData}:parsedData, (err, result) => {
if (err) {
testResultEditor.setValue(RED._("expressionEditor.errors.eval",{message:err.message}),-1);
} else {
if (usesContext) { if (usesContext) {
testResultEditor.setValue(RED._("expressionEditor.errors.context-unsupported"),-1); testResultEditor.setValue(RED._("expressionEditor.errors.context-unsupported"),-1);
return; return;
@ -319,6 +322,8 @@
formattedResult = RED._("expressionEditor.noMatch"); formattedResult = RED._("expressionEditor.noMatch");
} }
testResultEditor.setValue(formattedResult,-1); testResultEditor.setValue(formattedResult,-1);
}
});
} catch(err) { } catch(err) {
testResultEditor.setValue(RED._("expressionEditor.errors.eval",{message:err.message}),-1); testResultEditor.setValue(RED._("expressionEditor.errors.eval",{message:err.message}),-1);
} }

19
packages/node_modules/@node-red/nodes/core/common/20-inject.js vendored
View File

@ -117,14 +117,21 @@ module.exports = function(RED) {
if (p.v) { if (p.v) {
try { try {
var exp = RED.util.prepareJSONataExpression(p.v, node); var exp = RED.util.prepareJSONataExpression(p.v, node);
var val = RED.util.evaluateJSONataExpression(exp, msg); RED.util.evaluateJSONataExpression(exp, msg, (err, newValue) => {
RED.util.setMessageProperty(msg, property, val, true); if (err) {
} errors.push(err.toString())
catch (err) { } else {
errors.push(err.message); RED.util.setMessageProperty(msg,property,newValue,true);
}
} }
evaluateProperty(doneEvaluating) evaluateProperty(doneEvaluating)
});
} catch (err) {
errors.push(err.message);
evaluateProperty(doneEvaluating)
}
} else {
evaluateProperty(doneEvaluating)
}
} else { } else {
try { try {
RED.util.evaluateNodeProperty(value, valueType, node, msg, (err, newValue) => { RED.util.evaluateNodeProperty(value, valueType, node, msg, (err, newValue) => {

12
packages/node_modules/@node-red/util/lib/util.js vendored
View File

@ -25,7 +25,7 @@ const moment = require("moment-timezone");
const safeJSONStringify = require("json-stringify-safe"); const safeJSONStringify = require("json-stringify-safe");
const util = require("util"); const util = require("util");
const { hasOwnProperty } = Object.prototype; const { hasOwnProperty } = Object.prototype;
const log = require("./log")
/** /**
* Safely returns the object construtor name. * Safely returns the object construtor name.
* @return {String} the name of the object constructor if it exists, empty string otherwise. * @return {String} the name of the object constructor if it exists, empty string otherwise.
@ -672,7 +672,10 @@ function evaluateNodeProperty(value, type, node, msg, callback) {
result = /^true$/i.test(value); result = /^true$/i.test(value);
} else if (type === 'jsonata') { } else if (type === 'jsonata') {
var expr = prepareJSONataExpression(value, node); var expr = prepareJSONataExpression(value, node);
result = evaluateJSONataExpression(expr,msg); result = evaluateJSONataExpression(expr, msg, callback);
if (callback) {
return
}
} else if (type === 'env') { } else if (type === 'env') {
result = evaluateEnvProperty(value, node); result = evaluateEnvProperty(value, node);
} }
@ -767,6 +770,11 @@ function evaluateJSONataExpression(expr,msg,callback) {
}) })
}); });
} }
} else {
log.warn('Deprecated API warning: Calls to RED.util.evaluateJSONataExpression must include a callback. '+
'This will not be optional in Node-RED 4.0. Please identify the node from the following stack '+
'and check for an update on npm. If none is available, please notify the node author.')
log.warn(new Error().stack)
} }
return expr.evaluate(context, bindings, callback); return expr.evaluate(context, bindings, callback);
} }