openhab / openhab-addons

Add-ons for openHAB
https://www.openhab.org/
Eclipse Public License 2.0
1.86k stars 3.58k forks source link

[miele] reading state at startup and reading state change not working #6311

Closed Alex5719 closed 4 years ago

Alex5719 commented 4 years ago

In snapshot build 2.5.0~S1736-1 the state of the hood light is not read at startup (is still NULL).

Alex5719 commented 4 years ago

I downgraded my openHAB installation to the 2.5.0 M4 build and can confirm that in this older version the miele binding is reading the state of the hood light at startup correctly.

Alex5719 commented 4 years ago

Can someone look into this issue please? It seems that the latest commits on the miele binding have broken the functionality at least for Miele Zigbee devices.

Alex5719 commented 4 years ago

Not only reading the state at startup is affected. When manually changing the hood light directly at the device the state changes are also not read.

Alex5719 commented 4 years ago

The new milestone build 2.5.0 M5 is also affected by the bug. The state of the light of the hood is not read on startup of openhab and also not read when turning the light on and off manually by using the HW button at the hood. Switching the hood light on and off via the openhab switch is working. The Miele binding is not throwing any exceptions. Reading the state at startub and on state change did work in the old milestone build 2.5.0 M4.

kaikreuzer commented 4 years ago

The new milestone build 2.5.0 M5 is also affected by the bug.

Yes, clearly, as it is in the snapshots as you have reported and nobody looked into it / fixed it.

@MartinLepsy Can you confirm the issue? If so, are you able to fix it?

MartinLepsy commented 4 years ago

@kaikreuzer I'll try to have a look on the issue during this week. If I am able to reproduce it i will try to fix it, too.

MartinLepsy commented 4 years ago

Hello @kaikreuzer @Alex5719 - unfortunately I was unable to reproduce this behaviour. I tested it with my development environment and additionally with openhabian (2.5.0~S1759-1) together with a ZigBee hood and additionally a ZigBee washing machine.

The status was always displayed correctly and changed according to my actions. There were no problems with reading the state information at startup.

What hood do you have? Can you provide any more information?

Alex5719 commented 4 years ago

Hello MartinLepsy,

thank you fpr looking into my issue. The hood is indeed a special model: DA 6396 W Puristic ECO. The gateway has the software version 2.4.0 installed and is showing the following device details of the hood:

Technical Type | not defined Electronic Software ID | 0 Fabrication Number | not defined Interface | XKS 3000 Z

Actually the hood is not my only Miele device and the other devices are also not read. I now unregistered all devices from the gateway and only registered the oven. And still the state of my Miele oven is not read by the new binding.

Nevertheless when I switch back to the older version of the binding everything works.

MartinLepsy commented 4 years ago

Hello @Alex5719 did you also try to remove all your items / things in openhab and add them again? If you already tried this and it still does not work then it is hard to diagnose. Maybe you could also change your log level and check for other more detailed messages?

Alex5719 commented 4 years ago

I am using config files for my things and items

MartinLepsy commented 4 years ago

Any chance to get a snippet or the whole config file? So far I always added all things via paper UI so it may be worth analyzing.

Alex5719 commented 4 years ago

I now commented out the miele bridge and miele things in my thing config file. The used config is really standard almost identical to the example in the binding documentation. The only difference is, that the names of the things are chosen to be identical to the names used by the auto discovery function of paper ui. I also removed the miele gateway and things via paper ui. After adding the miele gateway and the miele things using the auto discovery function of paper ui the new binding worked correctly. After again adding the miele bridge and things to the thing config file the binding still works now.

It seems that the new version of the binding is now working for me. I am unsure what other users needs do to migrate from the old version of the miele binding to the new version

kaikreuzer commented 4 years ago

Glad to hear that it seems to be solved. Can the issue be closed then?

Alex5719 commented 4 years ago

The issue can be closed. @MarinLepsy: Thanks for investigation my issue.