frodovdr/node-red
1
0
Fork 0
mirror of https://github.com/node-red/node-red.git synced 2025-03-01 10:36:34 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix non-boolean returned value + try to find validation msg in tooltip

Browse Source
This commit is contained in:
GogoVega 2023-12-17 19:22:44 +01:00
parent ba08cf0417
commit 282d52cf0b
No known key found for this signature in database
GPG Key ID: E1E048B63AC5AC2B
4 changed files with 17 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
packages/node_modules/@node-red/editor-client/src/js/ui/common/popover.js vendored
View File

@ -53,6 +53,7 @@
* Returns the popover object with the following properties/functions:
* properties:
* - element : DOM element - the popover dom element
* - content : string|function - the popover content
* functions:
* - setContent(content) - change the popover content. This only works if the
* popover is not currently displayed. It does not
@ -159,7 +160,7 @@ RED.popover = (function() {
var size = options.size||"default";
var popupOffset = options.offset || 0;
if (!deltaSizes[size]) {
throw new Error("Invalid RED.popover size value:",size);
throw new Error("Invalid RED.popover size value:" + size);
}
var timer = null;
@ -419,6 +420,7 @@ RED.popover = (function() {
}
var res = {
get element() { return div },
get content() { return content },
setContent: function(_content) {
content = _content;
@ -440,7 +442,6 @@ RED.popover = (function() {
}
}
return res;
}
return {
@ -465,6 +466,9 @@ RED.popover = (function() {
content: label,
delay: { show: 750, hide: 50 }
});
popover.getContent = function () {
return content;
}
popover.setContent = function(newContent) {
content = newContent;
}

3
packages/node_modules/@node-red/editor-client/src/js/ui/common/typedInput.js vendored
View File

@ -1274,7 +1274,8 @@
tooltip.delete();
}
}
return !!valid;
// Must return a boolean
return (typeof valid === "string") ? false : valid;
},
show: function() {
this.uiSelect.show();

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

@ -191,6 +191,14 @@ RED.editor = (function() {
const isTypedInput = input.length > 0 && input.next(".red-ui-typedInput-container").length > 0;
if (isTypedInput) {
valid = input.typedInput("validate");
if (valid === false) {
// Try to find validation message in the tooltip
const tooltip = input.data("tooltip");
if (tooltip) {
const content = tooltip.getContent();
valid = label ? label + ": " + content : content;
}
}
}
}
}

2
packages/node_modules/@node-red/editor-client/src/js/ui/utils.js vendored
View File

@ -931,7 +931,7 @@ RED.utils = (function() {
}
} else if (propertyType === 'msg' || propertyType === 'flow' || propertyType === 'global' ) {
// To avoid double label
const valid = RED.utils.validatePropertyExpression(propertyValue, opt ? {} : undefined);
const valid = RED.utils.validatePropertyExpression(propertyValue, opt ? {} : null);
if (valid !== true) {
error = opt ? valid : RED._("validator.errors.invalid-prop");
}