node-red-dashboard icon indicating copy to clipboard operation
node-red-dashboard copied to clipboard

ui-dropdown doesnt pass correct value for multiple select options - and when msg pass through is enabled.

Open SandeepAgarwal13 opened this issue 1 year ago • 2 comments

Current Behavior

For other dashboard nodes - when the nodes are marked allow msg to pass through they retain the msg.payload values For ui-drop dashboard node - when msg.payload is a string - the node retains the msg.paylod value (the nodes are marked allow msg to pass through )

For ui-drop dashboard node - when msg.payload is a an array and multiple selection is set to true- the node doesnt retains the msg.paylod value

Expected Behavior

For ui-drop dashboard node - when msg.payload is a an array and multiple selection is set to true- the node should retains the msg.paylod value - similar to other nodes which retain msg.payload when "allow msg to act as pass through is selected"

Steps To Reproduce

Create a ui dropdown node with multiple select on and allow msg to pass through. Pass a message with msg.paylod as array (multiple selection). Check the result.

Environment

  • Dashboard version:1.11
  • Node-RED version:3.1.9
  • Node.js version:20.10.0
  • npm version:10.2.3
  • Platform/OS:Windows
  • Browser:Chrome

Have you provided an initial effort estimate for this issue?

I have provided an initial effort estimate

SandeepAgarwal13 avatar Jun 19 '24 21:06 SandeepAgarwal13

Hi @SandeepAgarwal13,

I have tried to reproduce it, but for me both the string and array payload are being passed through:

image

Can you share a simple flow that allows me to reproduce it?

BTW I executed my test on version v1.12.0 which was released today. I have not tried an older version.

bartbutenaers avatar Jun 20 '24 20:06 bartbutenaers

I rechecked this. This seems to be working. Let me spend more time and come back if i made a mistake :-(. Regret to have wasted your time. Let me monitor for few days - else will cancel it.

SandeepAgarwal13 avatar Jun 21 '24 10:06 SandeepAgarwal13