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
[icalendar] Ensure Bridge is initialized and calendar is available to update child Things #9337
[icalendar] Ensure Bridge is initialized and calendar is available to update child Things #9337
Conversation
…d Things Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wow. This is much cleaner. I have to learn quite much about the framework as i did not know about this possibility.
This also handles the race condition between Thing and Bridge by nearly just using the framework. Nice work!
for (Thing childThing : getThing().getThings()) { | ||
updateChild(childThing.getHandler()); | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
for (Thing childThing : getThing().getThings()) { | |
updateChild(childThing.getHandler()); | |
} | |
getThing().getThings().forEach(childThing -> updateChild(childThing.getHandler()); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I expected this comment ... Why didn't I implemented it in this way directly. 😉
Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de>
… update child Things (openhab#9337) * Ensure Bridge is initialized and calendar is available to update child Things
… update child Things (openhab#9337) * Ensure Bridge is initialized and calendar is available to update child Things Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de>
Related to #9029 and #9230
Signed-off-by: Christoph Weitkamp github@christophweitkamp.de