From c85b8c1d70bc2ea14b82255fe80b3d64dcc940c4 Mon Sep 17 00:00:00 2001 From: Korbinian Probst Date: Sat, 26 Dec 2020 22:38:38 +0100 Subject: [PATCH 1/3] Rename BatteryState to SystemState Signed-off-by: Korbinian Probst --- .../org.openhab.binding.senechome/README.md | 12 +++++----- .../internal/SenecHomeBindingConstants.java | 4 ++-- .../senechome/internal/SenecHomeHandler.java | 22 +++++++++---------- .../internal/json/SenecHomeEnergy.java | 4 ++-- .../resources/OH-INF/thing/thing-types.xml | 14 +++++------- 5 files changed, 27 insertions(+), 29 deletions(-) diff --git a/bundles/org.openhab.binding.senechome/README.md b/bundles/org.openhab.binding.senechome/README.md index c76ed30268011..9ac0575af266c 100644 --- a/bundles/org.openhab.binding.senechome/README.md +++ b/bundles/org.openhab.binding.senechome/README.md @@ -38,8 +38,8 @@ The property `limitationTresholdValue` is used as threshold for channel `powerLi | energyProduction | watt | Energy generated by your pv / inverter | | batteryPower | watt | Energy processed by batterie itself, for example while charging | | batteryFuelCharge | percent | Fuel charge of your battery (0 - 100%) | -| batteryState | | Text describing current action of battery (e.g. CHARGE) | -| batteryStateValue | | Value describing current action of battery (e.g. 14) | +| systemState | | Text describing current action of the senec home system (e.g. CHARGE) | +| systemStateValue | | Value describing current action of the senec home system (e.g. 14) | | gridPower | watt | Grid power level, negative for supply, positive values for drawing power | | gridPowerDraw | watt | Absolute power level of power draw, zero while supplying | | gridPowerSupply | watt | Absolute power level of power supply, zero while drawing | @@ -70,8 +70,8 @@ Number SenecHouseConsumption "Current power consumption [%d W]" { channel="senechome:senechome:pvbattery:energyProduction" } Number SenecBatteryPower "Energy processed by battery [%d W]" { channel="senechome:senechome:pvbattery:batteryPower" } Number SenecBatteryFuelCharge "State of Charge [%d %%]" { channel="senechome:senechome:pvbattery:batteryFuelCharge" } -String SenecBatteryState "Current action [%s]" { channel="senechome:senechome:pvbattery:batteryState" } -Number SenecBatteryStateValue "Current action [%d]" { channel="senechome:senechome:pvbattery:batteryStateValue" } +String SenecSystemState "Current system state [%s]" { channel="senechome:senechome:pvbattery:systemState" } +Number SenecSystemStateValue "Current system state [%d]" { channel="senechome:senechome:pvbattery:systemStateValue" } Number SenecGridPower "Grid power level [%d W]" { channel="senechome:senechome:pvbattery:gridPower" } Number SenecGridPowerDraw "Power draw from grid [%d W]" { channel="senechome:senechome:pvbattery:gridPowerDraw" } Number SenecGridPowerSupply "Power supply to grid [%d W]" { channel="senechome:senechome:pvbattery:gridPowerSupply" } @@ -105,8 +105,8 @@ Text label="Power Grid"{ Default item=SenecEnergyProduction Default item=SenecBatteryPower Default item=SenecBatteryFuelCharge - Default item=SenecBatteryState - Default item=SenecBatteryStateValue + Default item=SenecSystemState + Default item=SenecSystemStateValue Default item=SenecGridPower Default item=SenecGridPowerDraw Default item=SenecGridPowerSupply diff --git a/bundles/org.openhab.binding.senechome/src/main/java/org/openhab/binding/senechome/internal/SenecHomeBindingConstants.java b/bundles/org.openhab.binding.senechome/src/main/java/org/openhab/binding/senechome/internal/SenecHomeBindingConstants.java index a06e62e3dee99..17056b36d17e2 100644 --- a/bundles/org.openhab.binding.senechome/src/main/java/org/openhab/binding/senechome/internal/SenecHomeBindingConstants.java +++ b/bundles/org.openhab.binding.senechome/src/main/java/org/openhab/binding/senechome/internal/SenecHomeBindingConstants.java @@ -29,8 +29,8 @@ public class SenecHomeBindingConstants { public static final String CHANNEL_SENEC_POWER_LIMITATION = "powerLimitation"; public static final String CHANNEL_SENEC_POWER_LIMITATION_STATE = "powerLimitationState"; - public static final String CHANNEL_SENEC_BATTERY_STATE = "batteryState"; - public static final String CHANNEL_SENEC_BATTERY_STATE_VALUE = "batteryStateValue"; + public static final String CHANNEL_SENEC_SYSTEM_STATE = "systemState"; + public static final String CHANNEL_SENEC_SYSTEM_STATE_VALUE = "systemStateValue"; public static final String CHANNEL_SENEC_POWER_CONSUMPTION = "houseConsumption"; public static final String CHANNEL_SENEC_ENERGY_PRODUCTION = "energyProduction"; public static final String CHANNEL_SENEC_BATTERY_POWER = "batteryPower"; diff --git a/bundles/org.openhab.binding.senechome/src/main/java/org/openhab/binding/senechome/internal/SenecHomeHandler.java b/bundles/org.openhab.binding.senechome/src/main/java/org/openhab/binding/senechome/internal/SenecHomeHandler.java index ef7134df0e2ca..69120b4aada87 100644 --- a/bundles/org.openhab.binding.senechome/src/main/java/org/openhab/binding/senechome/internal/SenecHomeHandler.java +++ b/bundles/org.openhab.binding.senechome/src/main/java/org/openhab/binding/senechome/internal/SenecHomeHandler.java @@ -149,14 +149,14 @@ private void refresh() { .getChannel(SenecHomeBindingConstants.CHANNEL_SENEC_ENERGY_PRODUCTION); if (channelEnergyProduction != null) { updateState(channelEnergyProduction.getUID(), new QuantityType( - getSenecValue(response.energy.inverterPowerGeneration).setScale(0, RoundingMode.HALF_UP), - Units.WATT)); + getSenecValue(response.energy.inverterPowerGeneration).setScale(0, RoundingMode.HALF_UP), + Units.WATT)); } Channel channelBatteryPower = getThing().getChannel(SenecHomeBindingConstants.CHANNEL_SENEC_BATTERY_POWER); if (channelBatteryPower != null) { updateState(channelBatteryPower.getUID(), new QuantityType( - getSenecValue(response.energy.batteryPower).setScale(2, RoundingMode.HALF_UP), Units.WATT)); + getSenecValue(response.energy.batteryPower).setScale(2, RoundingMode.HALF_UP), Units.WATT)); } Channel channelBatteryFuelCharge = getThing() @@ -230,10 +230,10 @@ private void refresh() { updateState(channelGridFrequency.getUID(), new QuantityType( getSenecValue(response.grid.currentGridFrequency).setScale(2, RoundingMode.HALF_UP), Units.HERTZ)); - Channel channelBatteryStateValue = getThing() - .getChannel(SenecHomeBindingConstants.CHANNEL_SENEC_BATTERY_STATE_VALUE); - updateState(channelBatteryStateValue.getUID(), - new DecimalType(getSenecValue(response.energy.batteryState).intValue())); + Channel channelSystemStateValue = getThing() + .getChannel(SenecHomeBindingConstants.CHANNEL_SENEC_SYSTEM_STATE_VALUE); + updateState(channelSystemStateValue.getUID(), + new DecimalType(getSenecValue(response.energy.systemState).intValue())); Channel channelLiveBatCharge = getThing() .getChannel(SenecHomeBindingConstants.CHANNEL_SENEC_LIVE_BAT_CHARGE); @@ -268,9 +268,9 @@ private void refresh() { updateState(channelBatteryVoltage.getUID(), new QuantityType( getSenecValue(response.energy.batteryVoltage).setScale(2, RoundingMode.HALF_UP), Units.VOLT)); - Channel channelBatteryState = getThing().getChannel(SenecHomeBindingConstants.CHANNEL_SENEC_BATTERY_STATE); - if (channelBatteryState != null) { - updateBatteryState(channelBatteryState, getSenecValue(response.energy.batteryState).intValue()); + Channel channelSystemState = getThing().getChannel(SenecHomeBindingConstants.CHANNEL_SENEC_SYSTEM_STATE); + if (channelSystemState != null) { + updateSystemState(channelSystemState, getSenecValue(response.energy.systemState).intValue()); } updateGridPowerValues(getSenecValue(response.grid.currentGridValue)); @@ -341,7 +341,7 @@ protected void updatePowerLimitationStatus(Channel channel, boolean status, int updateState(channel.getUID(), status ? OnOffType.ON : OnOffType.OFF); } - protected void updateBatteryState(Channel channel, int code) { + protected void updateSystemState(Channel channel, int code) { updateState(channel.getUID(), new StringType(SenecBatteryStatus.descriptionFromCode(code))); } diff --git a/bundles/org.openhab.binding.senechome/src/main/java/org/openhab/binding/senechome/internal/json/SenecHomeEnergy.java b/bundles/org.openhab.binding.senechome/src/main/java/org/openhab/binding/senechome/internal/json/SenecHomeEnergy.java index a1937c7c10ced..fd9994392c4a0 100644 --- a/bundles/org.openhab.binding.senechome/src/main/java/org/openhab/binding/senechome/internal/json/SenecHomeEnergy.java +++ b/bundles/org.openhab.binding.senechome/src/main/java/org/openhab/binding/senechome/internal/json/SenecHomeEnergy.java @@ -30,13 +30,13 @@ public class SenecHomeEnergy implements Serializable { public @SerializedName("GUI_INVERTER_POWER") String inverterPowerGeneration; public @SerializedName("GUI_BAT_DATA_POWER") String batteryPower; public @SerializedName("GUI_BAT_DATA_FUEL_CHARGE") String batteryFuelCharge; - public @SerializedName("STAT_STATE") String batteryState; + public @SerializedName("STAT_STATE") String systemState; public @SerializedName("GUI_BAT_DATA_VOLTAGE") String batteryVoltage; @Override public String toString() { return "SenecHomeEnergy [homePowerConsumption=" + homePowerConsumption + ", inverterPowerGeneration=" + inverterPowerGeneration + ", batteryPower=" + batteryPower + ", batteryFuelCharge=" - + batteryFuelCharge + ", batteryState=" + batteryState + ", batteryVoltage" + batteryVoltage + "]"; + + batteryFuelCharge + ", systemState=" + systemState + ", batteryVoltage" + batteryVoltage + "]"; } } diff --git a/bundles/org.openhab.binding.senechome/src/main/resources/OH-INF/thing/thing-types.xml b/bundles/org.openhab.binding.senechome/src/main/resources/OH-INF/thing/thing-types.xml index e05d8d2301ebf..c26484ab2d759 100644 --- a/bundles/org.openhab.binding.senechome/src/main/resources/OH-INF/thing/thing-types.xml +++ b/bundles/org.openhab.binding.senechome/src/main/resources/OH-INF/thing/thing-types.xml @@ -16,8 +16,8 @@ - - + + @@ -98,17 +98,15 @@ - + String - - Battery + - + Number - - Battery + From a61fdcaace722b5274a0007d34c6290503fb1ec1 Mon Sep 17 00:00:00 2001 From: Korbinian Probst Date: Sat, 26 Dec 2020 22:56:50 +0100 Subject: [PATCH 2/3] Reduce duplicated code Signed-off-by: Korbinian Probst --- .../senechome/internal/SenecHomeHandler.java | 145 +++++++----------- 1 file changed, 53 insertions(+), 92 deletions(-) diff --git a/bundles/org.openhab.binding.senechome/src/main/java/org/openhab/binding/senechome/internal/SenecHomeHandler.java b/bundles/org.openhab.binding.senechome/src/main/java/org/openhab/binding/senechome/internal/SenecHomeHandler.java index 69120b4aada87..4fda0e32a5498 100644 --- a/bundles/org.openhab.binding.senechome/src/main/java/org/openhab/binding/senechome/internal/SenecHomeHandler.java +++ b/bundles/org.openhab.binding.senechome/src/main/java/org/openhab/binding/senechome/internal/SenecHomeHandler.java @@ -118,6 +118,13 @@ private void refresh() { refreshCache.getValue(); } + private void updateChannelState(String channelConstant, org.openhab.core.types.State channelState) { + Channel channel = getThing().getChannel(channelConstant); + if (channel != null) { + updateState(channel.getUID(), channelState); + } + } + public @Nullable Boolean refreshState() { try { SenecHomeResponse response = senecHomeApi.getStatistics(); @@ -136,137 +143,91 @@ private void refresh() { (100 - pvLimitation.intValue()) <= config.limitationTresholdValue, config.limitationDuration); } - Channel channelConsumption = getThing() - .getChannel(SenecHomeBindingConstants.CHANNEL_SENEC_POWER_CONSUMPTION); - if (channelConsumption != null) { - updateState(channelConsumption.getUID(), - new QuantityType( - getSenecValue(response.energy.homePowerConsumption).setScale(2, RoundingMode.HALF_UP), - Units.WATT)); - } + updateChannelState(SenecHomeBindingConstants.CHANNEL_SENEC_POWER_CONSUMPTION, new QuantityType( + getSenecValue(response.energy.homePowerConsumption).setScale(2, RoundingMode.HALF_UP), Units.WATT)); - Channel channelEnergyProduction = getThing() - .getChannel(SenecHomeBindingConstants.CHANNEL_SENEC_ENERGY_PRODUCTION); - if (channelEnergyProduction != null) { - updateState(channelEnergyProduction.getUID(), new QuantityType( + updateChannelState(SenecHomeBindingConstants.CHANNEL_SENEC_ENERGY_PRODUCTION, + new QuantityType( getSenecValue(response.energy.inverterPowerGeneration).setScale(0, RoundingMode.HALF_UP), Units.WATT)); - } - Channel channelBatteryPower = getThing().getChannel(SenecHomeBindingConstants.CHANNEL_SENEC_BATTERY_POWER); - if (channelBatteryPower != null) { - updateState(channelBatteryPower.getUID(), new QuantityType( + updateChannelState(SenecHomeBindingConstants.CHANNEL_SENEC_BATTERY_POWER, new QuantityType( getSenecValue(response.energy.batteryPower).setScale(2, RoundingMode.HALF_UP), Units.WATT)); - } - Channel channelBatteryFuelCharge = getThing() - .getChannel(SenecHomeBindingConstants.CHANNEL_SENEC_BATTERY_FUEL_CHARGE); - if (channelBatteryFuelCharge != null) { - updateState(channelBatteryFuelCharge.getUID(), - new QuantityType( - getSenecValue(response.energy.batteryFuelCharge).setScale(0, RoundingMode.HALF_UP), - Units.PERCENT)); - } + updateChannelState(SenecHomeBindingConstants.CHANNEL_SENEC_BATTERY_FUEL_CHARGE, + new QuantityType( + getSenecValue(response.energy.batteryFuelCharge).setScale(0, RoundingMode.HALF_UP), + Units.PERCENT)); + + updateChannelState(SenecHomeBindingConstants.CHANNEL_SENEC_GRID_CURRENT_PH1, + new QuantityType(getSenecValue(response.grid.currentGridCurrentPerPhase[0]) + .setScale(2, RoundingMode.HALF_UP), Units.AMPERE)); + + updateChannelState(SenecHomeBindingConstants.CHANNEL_SENEC_GRID_CURRENT_PH2, + new QuantityType(getSenecValue(response.grid.currentGridCurrentPerPhase[1]) + .setScale(2, RoundingMode.HALF_UP), Units.AMPERE)); + + updateChannelState(SenecHomeBindingConstants.CHANNEL_SENEC_GRID_CURRENT_PH3, + new QuantityType(getSenecValue(response.grid.currentGridCurrentPerPhase[2]) + .setScale(2, RoundingMode.HALF_UP), Units.AMPERE)); - Channel channelGridCurrentPhase1 = getThing() - .getChannel(SenecHomeBindingConstants.CHANNEL_SENEC_GRID_CURRENT_PH1); - updateState(channelGridCurrentPhase1.getUID(), new QuantityType( - getSenecValue(response.grid.currentGridCurrentPerPhase[0]).setScale(2, RoundingMode.HALF_UP), - Units.AMPERE)); - - Channel channelGridCurrentPhase2 = getThing() - .getChannel(SenecHomeBindingConstants.CHANNEL_SENEC_GRID_CURRENT_PH2); - updateState(channelGridCurrentPhase2.getUID(), new QuantityType( - getSenecValue(response.grid.currentGridCurrentPerPhase[1]).setScale(2, RoundingMode.HALF_UP), - Units.AMPERE)); - - Channel channelGridCurrentPhase3 = getThing() - .getChannel(SenecHomeBindingConstants.CHANNEL_SENEC_GRID_CURRENT_PH3); - updateState(channelGridCurrentPhase3.getUID(), new QuantityType( - getSenecValue(response.grid.currentGridCurrentPerPhase[2]).setScale(2, RoundingMode.HALF_UP), - Units.AMPERE)); - - Channel channelGridPowerPhase1 = getThing() - .getChannel(SenecHomeBindingConstants.CHANNEL_SENEC_GRID_POWER_PH1); - updateState(channelGridPowerPhase1.getUID(), + updateChannelState(SenecHomeBindingConstants.CHANNEL_SENEC_GRID_POWER_PH1, new QuantityType( getSenecValue(response.grid.currentGridPowerPerPhase[0]).setScale(2, RoundingMode.HALF_UP), Units.WATT)); - Channel channelGridPowerPhase2 = getThing() - .getChannel(SenecHomeBindingConstants.CHANNEL_SENEC_GRID_POWER_PH2); - updateState(channelGridPowerPhase2.getUID(), + updateChannelState(SenecHomeBindingConstants.CHANNEL_SENEC_GRID_POWER_PH2, new QuantityType( getSenecValue(response.grid.currentGridPowerPerPhase[1]).setScale(2, RoundingMode.HALF_UP), Units.WATT)); - Channel channelGridPowerPhase3 = getThing() - .getChannel(SenecHomeBindingConstants.CHANNEL_SENEC_GRID_POWER_PH3); - updateState(channelGridPowerPhase3.getUID(), + updateChannelState(SenecHomeBindingConstants.CHANNEL_SENEC_GRID_POWER_PH3, new QuantityType( getSenecValue(response.grid.currentGridPowerPerPhase[2]).setScale(2, RoundingMode.HALF_UP), Units.WATT)); - Channel channelGridVoltagePhase1 = getThing() - .getChannel(SenecHomeBindingConstants.CHANNEL_SENEC_GRID_VOLTAGE_PH1); - updateState(channelGridVoltagePhase1.getUID(), new QuantityType( - getSenecValue(response.grid.currentGridVoltagePerPhase[0]).setScale(2, RoundingMode.HALF_UP), - Units.VOLT)); - - Channel channelGridVoltagePhase2 = getThing() - .getChannel(SenecHomeBindingConstants.CHANNEL_SENEC_GRID_VOLTAGE_PH2); - updateState(channelGridVoltagePhase2.getUID(), new QuantityType( - getSenecValue(response.grid.currentGridVoltagePerPhase[1]).setScale(2, RoundingMode.HALF_UP), - Units.VOLT)); - - Channel channelGridVoltagePhase3 = getThing() - .getChannel(SenecHomeBindingConstants.CHANNEL_SENEC_GRID_VOLTAGE_PH3); - updateState(channelGridVoltagePhase3.getUID(), new QuantityType( - getSenecValue(response.grid.currentGridVoltagePerPhase[2]).setScale(2, RoundingMode.HALF_UP), - Units.VOLT)); - - Channel channelGridFrequency = getThing() - .getChannel(SenecHomeBindingConstants.CHANNEL_SENEC_GRID_FREQUENCY); - updateState(channelGridFrequency.getUID(), new QuantityType( + updateChannelState(SenecHomeBindingConstants.CHANNEL_SENEC_GRID_VOLTAGE_PH1, + new QuantityType(getSenecValue(response.grid.currentGridVoltagePerPhase[0]) + .setScale(2, RoundingMode.HALF_UP), Units.VOLT)); + + updateChannelState(SenecHomeBindingConstants.CHANNEL_SENEC_GRID_VOLTAGE_PH2, + new QuantityType(getSenecValue(response.grid.currentGridVoltagePerPhase[1]) + .setScale(2, RoundingMode.HALF_UP), Units.VOLT)); + + updateChannelState(SenecHomeBindingConstants.CHANNEL_SENEC_GRID_VOLTAGE_PH3, + new QuantityType(getSenecValue(response.grid.currentGridVoltagePerPhase[2]) + .setScale(2, RoundingMode.HALF_UP), Units.VOLT)); + + updateChannelState(SenecHomeBindingConstants.CHANNEL_SENEC_GRID_FREQUENCY, new QuantityType( getSenecValue(response.grid.currentGridFrequency).setScale(2, RoundingMode.HALF_UP), Units.HERTZ)); - Channel channelSystemStateValue = getThing() - .getChannel(SenecHomeBindingConstants.CHANNEL_SENEC_SYSTEM_STATE_VALUE); - updateState(channelSystemStateValue.getUID(), + updateChannelState(SenecHomeBindingConstants.CHANNEL_SENEC_SYSTEM_STATE_VALUE, new DecimalType(getSenecValue(response.energy.systemState).intValue())); - Channel channelLiveBatCharge = getThing() - .getChannel(SenecHomeBindingConstants.CHANNEL_SENEC_LIVE_BAT_CHARGE); - updateState(channelLiveBatCharge.getUID(), + updateChannelState(SenecHomeBindingConstants.CHANNEL_SENEC_LIVE_BAT_CHARGE, new QuantityType( getSenecValue(response.statistics.liveBatCharge).setScale(2, RoundingMode.HALF_UP), Units.WATT_HOUR)); - Channel channelLiveBatDischarge = getThing() - .getChannel(SenecHomeBindingConstants.CHANNEL_SENEC_LIVE_BAT_DISCHARGE); - updateState(channelLiveBatDischarge.getUID(), + updateChannelState(SenecHomeBindingConstants.CHANNEL_SENEC_LIVE_BAT_DISCHARGE, new QuantityType( getSenecValue(response.statistics.liveBatDischarge).setScale(2, RoundingMode.HALF_UP), Units.WATT_HOUR)); - Channel channelLiveGridImport = getThing() - .getChannel(SenecHomeBindingConstants.CHANNEL_SENEC_LIVE_GRID_IMPORT); - updateState(channelLiveGridImport.getUID(), + updateChannelState(SenecHomeBindingConstants.CHANNEL_SENEC_LIVE_GRID_IMPORT, new QuantityType( getSenecValue(response.statistics.liveGridImport).setScale(2, RoundingMode.HALF_UP), Units.WATT_HOUR)); - Channel channelLiveGridExport = getThing() - .getChannel(SenecHomeBindingConstants.CHANNEL_SENEC_LIVE_GRID_EXPORT); - updateState(channelLiveGridExport.getUID(), + updateChannelState(SenecHomeBindingConstants.CHANNEL_SENEC_LIVE_GRID_EXPORT, new QuantityType( getSenecValue(response.statistics.liveGridExport).setScale(2, RoundingMode.HALF_UP), Units.WATT_HOUR)); - Channel channelBatteryVoltage = getThing() - .getChannel(SenecHomeBindingConstants.CHANNEL_SENEC_BATTERY_VOLTAGE); - updateState(channelBatteryVoltage.getUID(), new QuantityType( - getSenecValue(response.energy.batteryVoltage).setScale(2, RoundingMode.HALF_UP), Units.VOLT)); + updateChannelState(SenecHomeBindingConstants.CHANNEL_SENEC_BATTERY_VOLTAGE, + new QuantityType( + getSenecValue(response.energy.batteryVoltage).setScale(2, RoundingMode.HALF_UP), + Units.VOLT)); Channel channelSystemState = getThing().getChannel(SenecHomeBindingConstants.CHANNEL_SENEC_SYSTEM_STATE); if (channelSystemState != null) { From 9511107752fe9ca590ed82568523750fe94339c1 Mon Sep 17 00:00:00 2001 From: Korbinian Probst Date: Tue, 29 Dec 2020 22:55:17 +0100 Subject: [PATCH 3/3] Import State instead of direct use Signed-off-by: Korbinian Probst --- .../openhab/binding/senechome/internal/SenecHomeHandler.java | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/bundles/org.openhab.binding.senechome/src/main/java/org/openhab/binding/senechome/internal/SenecHomeHandler.java b/bundles/org.openhab.binding.senechome/src/main/java/org/openhab/binding/senechome/internal/SenecHomeHandler.java index 4fda0e32a5498..5d582e052d651 100644 --- a/bundles/org.openhab.binding.senechome/src/main/java/org/openhab/binding/senechome/internal/SenecHomeHandler.java +++ b/bundles/org.openhab.binding.senechome/src/main/java/org/openhab/binding/senechome/internal/SenecHomeHandler.java @@ -48,6 +48,7 @@ import org.openhab.core.thing.ThingStatusDetail; import org.openhab.core.thing.binding.BaseThingHandler; import org.openhab.core.types.Command; +import org.openhab.core.types.State; import org.slf4j.Logger; import org.slf4j.LoggerFactory; @@ -118,7 +119,7 @@ private void refresh() { refreshCache.getValue(); } - private void updateChannelState(String channelConstant, org.openhab.core.types.State channelState) { + private void updateChannelState(String channelConstant, State channelState) { Channel channel = getThing().getChannel(channelConstant); if (channel != null) { updateState(channel.getUID(), channelState);