-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
[hue] Channel alert added for groups #7810
Conversation
Fix openhab#7742 Signed-off-by: Laurent Garnier <lg.hc@free.fr>
* This method will cancel previously scheduled alert item state | ||
* restoration. | ||
*/ | ||
private void cancelScheduledFuture() { |
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.
You should call this when disposing this handler as well.
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.
Yes, you're right.
I have done a copy/paste from the hue light handler and this was missing in this handler too.
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.
Fixed
Signed-off-by: Laurent Garnier <lg.hc@free.fr>
Travis tests have failedHey @lolodomo, |
* 2.5.x: (174 commits) [hpprinter] Add additional data points and refactoring (openhab#7805) [neohub] new/legacy API; null annotations; enhancements; bugs; logging (openhab#7767) [meteoalerte] Initial contribution (openhab#7200) [lgwebos] Console command to show the access key (openhab#7801) [hue] Refactored state handling and fix polling after command (openhab#7518) [telegram] add attachment URL (openhab#7816) [siemensrds] readme adjusted to match openhab#7814 (openhab#7819) [lametrictime] correctly parse response (openhab#7818) [Seneye] Bug fix for using Pond or Home sensors. (openhab#7797) [siemensrds] apply UoM quantityType percent for relative humidity (openhab#7814) [alarmdecoder] Add vzone thing for virtual zone control (openhab#7800) [hue] Channel alert added for groups (openhab#7810) [hue] Keep compatibility with hue emulation for groups (openhab#7809) [dscalarm] Bridge/things management refactored (openhab#7748) [avmfritz] Add link to Fensterkontakt (magnetisch) to docs (openhab#7806) [deconz] add light/blind support and additional sensors (openhab#7608) [homekit] add support for min/max values for temperature (openhab#7782) [tesla] Use CXF JAX-RS client builder, if available (openhab#7804) [mqtt.homie] Improve Homie discovery time (openhab#7760) [siemensrds] null annotations; JUnit; UoM; enhancements; bug; refactoring; logging (openhab#7769) ...
* [hue] Channel alert added for groups Fix openhab#7742 * Call cancelSceduledFuture in dispo() Signed-off-by: Laurent Garnier <lg.hc@free.fr>
* [hue] Channel alert added for groups Fix openhab#7742 * Call cancelSceduledFuture in dispo() Signed-off-by: Laurent Garnier <lg.hc@free.fr> Signed-off-by: CSchlipp <christian@schlipp.de>
* [hue] Channel alert added for groups Fix openhab#7742 * Call cancelSceduledFuture in dispo() Signed-off-by: Laurent Garnier <lg.hc@free.fr>
* [hue] Channel alert added for groups Fix openhab#7742 * Call cancelSceduledFuture in dispo() Signed-off-by: Laurent Garnier <lg.hc@free.fr>
* [hue] Channel alert added for groups Fix openhab#7742 * Call cancelSceduledFuture in dispo() Signed-off-by: Laurent Garnier <lg.hc@free.fr>
* [hue] Channel alert added for groups Fix openhab#7742 * Call cancelSceduledFuture in dispo() Signed-off-by: Laurent Garnier <lg.hc@free.fr>
* [hue] Channel alert added for groups Fix openhab#7742 * Call cancelSceduledFuture in dispo() Signed-off-by: Laurent Garnier <lg.hc@free.fr> Signed-off-by: Daan Meijer <daan@studioseptember.nl>
* [hue] Channel alert added for groups Fix openhab#7742 * Call cancelSceduledFuture in dispo() Signed-off-by: Laurent Garnier <lg.hc@free.fr>
Fix #7742
Signed-off-by: Laurent Garnier lg.hc@free.fr