frodovdr/node-red
1
0
Fork 0
mirror of https://github.com/node-red/node-red.git synced 2025-12-27 07:31:07 +01:00
Code Issues Packages Projects Releases Wiki Activity

Fix range node to handle input min > max

Browse Source 
and add tests
and add pin hover to show confiog
This commit is contained in:
Dave Conway-Jones
2025-08-29 15:08:23 +01:00
parent 7b51f30d11
commit 4ae8fcbc45
3 changed files with 24 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
packages/node_modules/@node-red/nodes/core/function/16-range.html vendored
View File

@@ -64,6 +64,14 @@
},
inputs: 1,
outputs: 1,
inputLabels: function() { return this.minin + " - " + this.maxin },
outputLabels: function(i) {
var outie = this.minout + " - " + this.maxout
if (this.action === "clamp") { outie = "[" + outie + "]" }
if (this.action === "drop") { outie = "x" + outie + "x" }
if (this.action === "roll") { outie = ">" + outie + ">" }
return outie;
},
icon: "range.svg",
label: function() {
if (this.minout !== "" && this.maxout !== "") { return this.name||this.minout + " - " + this.maxout; }

8
packages/node_modules/@node-red/nodes/core/function/16-range.js vendored
View File

@@ -24,6 +24,14 @@ module.exports = function(RED) {
this.maxin = Number(n.maxin);
this.minout = Number(n.minout);
this.maxout = Number(n.maxout);
if (this.minin > this.maxin) {
let tmp = this.minin;
this.minin = this.maxin;
this.maxin = tmp;
tmp = this.minout;
this.minout = this.maxout;
this.maxout = tmp;
}
if (this.round) {
this.maxout = Math.floor(this.maxout);
this.minout = Math.ceil(this.minout);

8
test/nodes/core/function/16-range_spec.js
View File

@@ -114,6 +114,14 @@ describe('range Node', function() {
genericRangeTest("clamp", 0, 9.5, 0.3, 100.5, true, -13.1, 1, done);
});
it('handles input range where max < min', function(done) {
genericRangeTest("scale", 100, 0, 0, -100, false, 40, -60, done);
});
it('handles input range where max < min', function(done) {
genericRangeTest("scale", 100, 0, -100, 0, false, 10, -10, done);
});
it('drops msg if in drop mode and input outside range', function(done) {
var flow = [{"id":"rangeNode1","type":"range","minin":2,"maxin":8,"minout":20,"maxout":80,"action":"drop","round":true,"name":"rangeNode","wires":[["helperNode1"]]},
{id:"helperNode1", type:"helper", wires:[]}];