From cc042dabff6cb4bc3fabc3f3264d2e520d7cd46b Mon Sep 17 00:00:00 2001 From: Wouter Born Date: Thu, 24 Sep 2020 17:50:47 +0200 Subject: [PATCH] Remove deprecated handleUpdate from ThingHandler (#8558) The handleUpdate method was deprecated when profiles were introduced (see eclipse-archived/smarthome#4108). Instead the "follow profile" can be used which forwards item updates as commands to handlers. This profile works with any binding instead of only those that implement the handleUpdate method. Related to openhab/openhab-core#1669 Signed-off-by: Wouter Born --- .../internal/handler/Cm11aBridgeHandler.java | 18 ++++----------- .../handler/KNXBridgeBaseThingHandler.java | 6 ----- .../radiora/handler/DimmerHandler.java | 9 -------- .../handler/XiaomiActorGatewayHandler.java | 23 ------------------- .../XiaomiSensorBaseHandlerWithTimer.java | 12 ---------- .../phc/internal/handler/PHCHandler.java | 9 -------- .../live/TelldusLiveBridgeHandler.java | 6 ----- .../profiles/ExecTransformationProfile.java | 1 - .../JavascriptTransformationProfile.java | 1 - .../profiles/JinjaTransformationProfile.java | 1 - .../JSonPathTransformationProfile.java | 1 - .../profiles/MapTransformationProfile.java | 1 - .../profiles/RegexTransformationProfile.java | 1 - .../profiles/ScaleTransformationProfile.java | 1 - .../profiles/XPathTransformationProfile.java | 1 - .../profiles/XSLTTransformationProfile.java | 1 - 16 files changed, 4 insertions(+), 88 deletions(-) diff --git a/bundles/org.openhab.binding.cm11a/src/main/java/org/openhab/binding/cm11a/internal/handler/Cm11aBridgeHandler.java b/bundles/org.openhab.binding.cm11a/src/main/java/org/openhab/binding/cm11a/internal/handler/Cm11aBridgeHandler.java index 9d4f672a48..9d58f96e60 100644 --- a/bundles/org.openhab.binding.cm11a/src/main/java/org/openhab/binding/cm11a/internal/handler/Cm11aBridgeHandler.java +++ b/bundles/org.openhab.binding.cm11a/src/main/java/org/openhab/binding/cm11a/internal/handler/Cm11aBridgeHandler.java @@ -202,20 +202,20 @@ public class Cm11aBridgeHandler extends BaseBridgeHandler implements ReceivedDat // Perform appropriate update based on X10Command received // Handle ON/OFF commands if (cmd == X10ReceivedData.X10COMMAND.ON) { - handleUpdate(channelUid, OnOffType.ON); + updateState(channelUid, OnOffType.ON); handler.setCurrentState(OnOffType.ON); } else if (cmd == X10ReceivedData.X10COMMAND.OFF) { - handleUpdate(channelUid, OnOffType.OFF); + updateState(channelUid, OnOffType.OFF); handler.setCurrentState(OnOffType.OFF); // Handle DIM/Bright commands } else if (cmd == X10ReceivedData.X10COMMAND.DIM) { State newState = handler.addDimsToCurrentState(dims); - handleUpdate(channelUid, newState); + updateState(channelUid, newState); handler.setCurrentState(newState); logger.debug("Current state set to: {}", handler.getCurrentState().toFullString()); } else if (cmd == X10ReceivedData.X10COMMAND.BRIGHT) { State newState = handler.addBrightsToCurrentState(dims); - handleUpdate(channelUid, newState); + updateState(channelUid, newState); handler.setCurrentState(newState); logger.debug("Current state set to: {}", handler.getCurrentState().toFullString()); } else { @@ -223,16 +223,6 @@ public class Cm11aBridgeHandler extends BaseBridgeHandler implements ReceivedDat } } - /** - * The default implementation of this method doesn't do anything. We need it to update the ui as done by the - * updateState function. - * And, update state can not be called directly because it is protected - */ - @Override - public void handleUpdate(ChannelUID channelUID, State newState) { - updateState(channelUID, newState); - } - /** * Get the X10Interface * diff --git a/bundles/org.openhab.binding.knx/src/main/java/org/openhab/binding/knx/internal/handler/KNXBridgeBaseThingHandler.java b/bundles/org.openhab.binding.knx/src/main/java/org/openhab/binding/knx/internal/handler/KNXBridgeBaseThingHandler.java index 305857ed39..465e13bb32 100644 --- a/bundles/org.openhab.binding.knx/src/main/java/org/openhab/binding/knx/internal/handler/KNXBridgeBaseThingHandler.java +++ b/bundles/org.openhab.binding.knx/src/main/java/org/openhab/binding/knx/internal/handler/KNXBridgeBaseThingHandler.java @@ -27,7 +27,6 @@ import org.openhab.core.thing.ThingStatus; import org.openhab.core.thing.ThingStatusDetail; import org.openhab.core.thing.binding.BaseBridgeHandler; import org.openhab.core.types.Command; -import org.openhab.core.types.State; import tuwien.auto.calimero.IndividualAddress; import tuwien.auto.calimero.mgmt.Destination; @@ -51,11 +50,6 @@ public abstract class KNXBridgeBaseThingHandler extends BaseBridgeHandler implem protected abstract KNXClient getClient(); - @Override - public void handleUpdate(ChannelUID channelUID, State newState) { - // Nothing to do here - } - @Override public void handleCommand(ChannelUID channelUID, Command command) { // Nothing to do here diff --git a/bundles/org.openhab.binding.lutron/src/main/java/org/openhab/binding/lutron/internal/radiora/handler/DimmerHandler.java b/bundles/org.openhab.binding.lutron/src/main/java/org/openhab/binding/lutron/internal/radiora/handler/DimmerHandler.java index 99e429b1e0..575c736413 100644 --- a/bundles/org.openhab.binding.lutron/src/main/java/org/openhab/binding/lutron/internal/radiora/handler/DimmerHandler.java +++ b/bundles/org.openhab.binding.lutron/src/main/java/org/openhab/binding/lutron/internal/radiora/handler/DimmerHandler.java @@ -27,7 +27,6 @@ import org.openhab.core.library.types.PercentType; import org.openhab.core.thing.ChannelUID; import org.openhab.core.thing.Thing; import org.openhab.core.types.Command; -import org.openhab.core.types.State; /** * Handler for RadioRA dimmers @@ -76,14 +75,6 @@ public class DimmerHandler extends LutronHandler { } } - @Override - public void handleUpdate(ChannelUID channelUID, State newState) { - if (LutronBindingConstants.CHANNEL_LIGHTLEVEL.equals(channelUID.getId())) { - PercentType percent = (PercentType) newState.as(PercentType.class); - updateInternalState(percent.intValue()); - } - } - @Override public void handleFeedback(RadioRAFeedback feedback) { if (feedback instanceof LocalZoneChangeFeedback) { diff --git a/bundles/org.openhab.binding.mihome/src/main/java/org/openhab/binding/mihome/internal/handler/XiaomiActorGatewayHandler.java b/bundles/org.openhab.binding.mihome/src/main/java/org/openhab/binding/mihome/internal/handler/XiaomiActorGatewayHandler.java index eb9a44d5ee..de1e903248 100644 --- a/bundles/org.openhab.binding.mihome/src/main/java/org/openhab/binding/mihome/internal/handler/XiaomiActorGatewayHandler.java +++ b/bundles/org.openhab.binding.mihome/src/main/java/org/openhab/binding/mihome/internal/handler/XiaomiActorGatewayHandler.java @@ -22,7 +22,6 @@ import org.openhab.core.library.types.PercentType; import org.openhab.core.thing.ChannelUID; import org.openhab.core.thing.Thing; import org.openhab.core.types.Command; -import org.openhab.core.types.State; import org.slf4j.Logger; import org.slf4j.LoggerFactory; @@ -127,28 +126,6 @@ public class XiaomiActorGatewayHandler extends XiaomiActorBaseHandler { logger.debug("Changed volume to {}", lastVolume); } - @Override - public void handleUpdate(ChannelUID channelUID, State newState) { - logger.debug("Update {} for channel {} received", newState, channelUID); - switch (channelUID.getId()) { - case CHANNEL_BRIGHTNESS: - if (newState instanceof PercentType) { - lastBrigthness = ((PercentType) newState).intValue(); - } - break; - case CHANNEL_COLOR: - if (newState instanceof HSBType) { - lastColor = ((HSBType) newState).getRGB(); - } - break; - case CHANNEL_GATEWAY_VOLUME: - if (newState instanceof DecimalType) { - updateLastVolume((DecimalType) newState); - } - break; - } - } - @Override void parseReport(JsonObject data) { parseDefault(data); diff --git a/bundles/org.openhab.binding.mihome/src/main/java/org/openhab/binding/mihome/internal/handler/XiaomiSensorBaseHandlerWithTimer.java b/bundles/org.openhab.binding.mihome/src/main/java/org/openhab/binding/mihome/internal/handler/XiaomiSensorBaseHandlerWithTimer.java index 09087dc17c..9be9f38da8 100644 --- a/bundles/org.openhab.binding.mihome/src/main/java/org/openhab/binding/mihome/internal/handler/XiaomiSensorBaseHandlerWithTimer.java +++ b/bundles/org.openhab.binding.mihome/src/main/java/org/openhab/binding/mihome/internal/handler/XiaomiSensorBaseHandlerWithTimer.java @@ -16,9 +16,7 @@ import java.util.Timer; import java.util.TimerTask; import org.openhab.core.library.types.DecimalType; -import org.openhab.core.thing.ChannelUID; import org.openhab.core.thing.Thing; -import org.openhab.core.types.State; import org.slf4j.Logger; import org.slf4j.LoggerFactory; @@ -87,14 +85,4 @@ public abstract class XiaomiSensorBaseHandlerWithTimer extends XiaomiSensorBaseH timerSetpoint = defaultTimer; } } - - @Override - public void handleUpdate(ChannelUID channelUID, State newState) { - if (setpointChannel.equals(channelUID.getId())) { - if (newState instanceof DecimalType) { - logger.debug("Received update for timer setpoint channel: {}", newState); - timerSetpoint = ((DecimalType) newState).intValue(); - } - } - } } diff --git a/bundles/org.openhab.binding.phc/src/main/java/org/openhab/binding/phc/internal/handler/PHCHandler.java b/bundles/org.openhab.binding.phc/src/main/java/org/openhab/binding/phc/internal/handler/PHCHandler.java index 4f715405a1..b718eb6851 100644 --- a/bundles/org.openhab.binding.phc/src/main/java/org/openhab/binding/phc/internal/handler/PHCHandler.java +++ b/bundles/org.openhab.binding.phc/src/main/java/org/openhab/binding/phc/internal/handler/PHCHandler.java @@ -20,7 +20,6 @@ import java.util.Map; import java.util.Map.Entry; import org.openhab.core.config.core.Configuration; -import org.openhab.core.library.types.DecimalType; import org.openhab.core.library.types.OnOffType; import org.openhab.core.library.types.PercentType; import org.openhab.core.library.types.StopMoveType; @@ -125,14 +124,6 @@ public class PHCHandler extends BaseThingHandler { } } - @Override - public void handleUpdate(ChannelUID channelUID, State newState) { - if (CHANNELS_JRM_TIME.equals(channelUID.getGroupId())) { - times[Integer - .parseInt(channelUID.getIdWithoutGroup())] = (short) (((DecimalType) newState).floatValue() * 10); - } - } - @Override public void handleConfigurationUpdate(Map configurationParameters) { if (isInitialized()) { // prevents change of address diff --git a/bundles/org.openhab.binding.tellstick/src/main/java/org/openhab/binding/tellstick/internal/live/TelldusLiveBridgeHandler.java b/bundles/org.openhab.binding.tellstick/src/main/java/org/openhab/binding/tellstick/internal/live/TelldusLiveBridgeHandler.java index 7e4a96615a..f534fe3193 100644 --- a/bundles/org.openhab.binding.tellstick/src/main/java/org/openhab/binding/tellstick/internal/live/TelldusLiveBridgeHandler.java +++ b/bundles/org.openhab.binding.tellstick/src/main/java/org/openhab/binding/tellstick/internal/live/TelldusLiveBridgeHandler.java @@ -36,7 +36,6 @@ import org.openhab.core.thing.ThingStatusDetail; import org.openhab.core.thing.binding.BaseBridgeHandler; import org.openhab.core.types.Command; import org.openhab.core.types.RefreshType; -import org.openhab.core.types.State; import org.slf4j.Logger; import org.slf4j.LoggerFactory; import org.tellstick.device.TellstickDeviceEvent; @@ -247,11 +246,6 @@ public class TelldusLiveBridgeHandler extends BaseBridgeHandler implements Telld super.handleRemoval(); } - @Override - public void handleUpdate(ChannelUID channelUID, State newState) { - super.handleUpdate(channelUID, newState); - } - @Override public void thingUpdated(Thing thing) { super.thingUpdated(thing); diff --git a/bundles/org.openhab.transform.exec/src/main/java/org/openhab/transform/exec/internal/profiles/ExecTransformationProfile.java b/bundles/org.openhab.transform.exec/src/main/java/org/openhab/transform/exec/internal/profiles/ExecTransformationProfile.java index 6e14d78e1f..91d662f515 100644 --- a/bundles/org.openhab.transform.exec/src/main/java/org/openhab/transform/exec/internal/profiles/ExecTransformationProfile.java +++ b/bundles/org.openhab.transform.exec/src/main/java/org/openhab/transform/exec/internal/profiles/ExecTransformationProfile.java @@ -82,7 +82,6 @@ public class ExecTransformationProfile implements StateProfile { @Override public void onStateUpdateFromItem(State state) { - callback.handleUpdate(state); } @Override diff --git a/bundles/org.openhab.transform.javascript/src/main/java/org/openhab/transform/javascript/internal/profiles/JavascriptTransformationProfile.java b/bundles/org.openhab.transform.javascript/src/main/java/org/openhab/transform/javascript/internal/profiles/JavascriptTransformationProfile.java index 2f2b3a5904..fe58224e6f 100644 --- a/bundles/org.openhab.transform.javascript/src/main/java/org/openhab/transform/javascript/internal/profiles/JavascriptTransformationProfile.java +++ b/bundles/org.openhab.transform.javascript/src/main/java/org/openhab/transform/javascript/internal/profiles/JavascriptTransformationProfile.java @@ -84,7 +84,6 @@ public class JavascriptTransformationProfile implements StateProfile { @Override public void onStateUpdateFromItem(State state) { - callback.handleUpdate(state); } @Override diff --git a/bundles/org.openhab.transform.jinja/src/main/java/org/openhab/transform/jinja/internal/profiles/JinjaTransformationProfile.java b/bundles/org.openhab.transform.jinja/src/main/java/org/openhab/transform/jinja/internal/profiles/JinjaTransformationProfile.java index 112aae1d16..93e7b4624b 100644 --- a/bundles/org.openhab.transform.jinja/src/main/java/org/openhab/transform/jinja/internal/profiles/JinjaTransformationProfile.java +++ b/bundles/org.openhab.transform.jinja/src/main/java/org/openhab/transform/jinja/internal/profiles/JinjaTransformationProfile.java @@ -83,7 +83,6 @@ public class JinjaTransformationProfile implements StateProfile { @Override public void onStateUpdateFromItem(State state) { - callback.handleUpdate(state); } @Override diff --git a/bundles/org.openhab.transform.jsonpath/src/main/java/org/openhab/transform/jsonpath/internal/profiles/JSonPathTransformationProfile.java b/bundles/org.openhab.transform.jsonpath/src/main/java/org/openhab/transform/jsonpath/internal/profiles/JSonPathTransformationProfile.java index b01a3d0b91..1579017e82 100644 --- a/bundles/org.openhab.transform.jsonpath/src/main/java/org/openhab/transform/jsonpath/internal/profiles/JSonPathTransformationProfile.java +++ b/bundles/org.openhab.transform.jsonpath/src/main/java/org/openhab/transform/jsonpath/internal/profiles/JSonPathTransformationProfile.java @@ -84,7 +84,6 @@ public class JSonPathTransformationProfile implements StateProfile { @Override public void onStateUpdateFromItem(State state) { - callback.handleUpdate(state); } @Override diff --git a/bundles/org.openhab.transform.map/src/main/java/org/openhab/transform/map/internal/profiles/MapTransformationProfile.java b/bundles/org.openhab.transform.map/src/main/java/org/openhab/transform/map/internal/profiles/MapTransformationProfile.java index a26eee9b43..2d16366d61 100644 --- a/bundles/org.openhab.transform.map/src/main/java/org/openhab/transform/map/internal/profiles/MapTransformationProfile.java +++ b/bundles/org.openhab.transform.map/src/main/java/org/openhab/transform/map/internal/profiles/MapTransformationProfile.java @@ -83,7 +83,6 @@ public class MapTransformationProfile implements StateProfile { @Override public void onStateUpdateFromItem(State state) { - callback.handleUpdate(state); } @Override diff --git a/bundles/org.openhab.transform.regex/src/main/java/org/openhab/transform/regex/internal/profiles/RegexTransformationProfile.java b/bundles/org.openhab.transform.regex/src/main/java/org/openhab/transform/regex/internal/profiles/RegexTransformationProfile.java index 9b649753b3..423a5ef9aa 100644 --- a/bundles/org.openhab.transform.regex/src/main/java/org/openhab/transform/regex/internal/profiles/RegexTransformationProfile.java +++ b/bundles/org.openhab.transform.regex/src/main/java/org/openhab/transform/regex/internal/profiles/RegexTransformationProfile.java @@ -83,7 +83,6 @@ public class RegexTransformationProfile implements StateProfile { @Override public void onStateUpdateFromItem(State state) { - callback.handleUpdate(state); } @Override diff --git a/bundles/org.openhab.transform.scale/src/main/java/org/openhab/transform/scale/internal/profiles/ScaleTransformationProfile.java b/bundles/org.openhab.transform.scale/src/main/java/org/openhab/transform/scale/internal/profiles/ScaleTransformationProfile.java index 46dcf45485..5b25d2c79c 100644 --- a/bundles/org.openhab.transform.scale/src/main/java/org/openhab/transform/scale/internal/profiles/ScaleTransformationProfile.java +++ b/bundles/org.openhab.transform.scale/src/main/java/org/openhab/transform/scale/internal/profiles/ScaleTransformationProfile.java @@ -83,7 +83,6 @@ public class ScaleTransformationProfile implements StateProfile { @Override public void onStateUpdateFromItem(State state) { - callback.handleUpdate(state); } @Override diff --git a/bundles/org.openhab.transform.xpath/src/main/java/org/openhab/transform/xpath/internal/profiles/XPathTransformationProfile.java b/bundles/org.openhab.transform.xpath/src/main/java/org/openhab/transform/xpath/internal/profiles/XPathTransformationProfile.java index 99cf0b3369..9c4d620970 100644 --- a/bundles/org.openhab.transform.xpath/src/main/java/org/openhab/transform/xpath/internal/profiles/XPathTransformationProfile.java +++ b/bundles/org.openhab.transform.xpath/src/main/java/org/openhab/transform/xpath/internal/profiles/XPathTransformationProfile.java @@ -83,7 +83,6 @@ public class XPathTransformationProfile implements StateProfile { @Override public void onStateUpdateFromItem(State state) { - callback.handleUpdate(state); } @Override diff --git a/bundles/org.openhab.transform.xslt/src/main/java/org/openhab/transform/xslt/internal/profiles/XSLTTransformationProfile.java b/bundles/org.openhab.transform.xslt/src/main/java/org/openhab/transform/xslt/internal/profiles/XSLTTransformationProfile.java index f9dec4ad3c..8e1f001d5e 100644 --- a/bundles/org.openhab.transform.xslt/src/main/java/org/openhab/transform/xslt/internal/profiles/XSLTTransformationProfile.java +++ b/bundles/org.openhab.transform.xslt/src/main/java/org/openhab/transform/xslt/internal/profiles/XSLTTransformationProfile.java @@ -83,7 +83,6 @@ public class XSLTTransformationProfile implements StateProfile { @Override public void onStateUpdateFromItem(State state) { - callback.handleUpdate(state); } @Override -- 2.47.3