Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Current State passing original message #124

Closed
DKCryptoKing opened this issue May 31, 2019 · 1 comment
Closed

Current State passing original message #124

DKCryptoKing opened this issue May 31, 2019 · 1 comment

Comments

@DKCryptoKing
Copy link

Describe the bug
After updating to version 0.11.0, all my current flows that are using the "Current State" node aren't working anymore. Apparently this node no longer pass the message from the current state entity, but is instead passing the original message.

To Reproduce
Example:

Skærmbillede 2019-05-31 kl  09 21 04

Expected behavior
In the example above, I would have expected the current state node to pass the state for my SONOS entity, so that I can use the switch node to redirect the flow based on the value on my Sonos payer. Instead, what happens is, that the payload of my Xiaomi Push Button is passed through.

Skærmbillede 2019-05-31 kl  09 26 37

Environment (please complete the following information):

  • Node Red Version: 3.1.1
  • Node.js Version: unkown
  • NR Home Assistant Plugin Version: 0.11.0
  • Is Node Red running in Docker: yes

Other (please complete the following information):

  • Have you searched previous issues for duplicates?:

Yes, I have tried my best.

  • Comments:

Thank you very much in advance for looking into this. I see in the release notes that a significant enhancement was introduced to this node, which might very well be the reason behind my issues.

@zachowj
Copy link
Owner

zachowj commented May 31, 2019

You are correct in that the changes to the current-state node is the reason behind your issue. The events: all node payload contains entity_id which will override the current-state node's config. It not that the current-state is sending on the original message but it is getting the current state of the entity id passed in by the events all node and passing that new data along.

Currently the solution is to delete msg.payload.entity_id before the current-state node. A different solution may appear in the next release.

@zachowj zachowj closed this as completed May 31, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants