demus/openhab-addons
1
0
Fork 0
mirror of https://github.com/openhab/openhab-addons.git synced 2025-01-26 15:21:41 +01:00
Code Issues Packages Projects Releases Wiki Activity

[shelly] Fix compiler warnings (#17179)

Browse Source 
* Fix warnings
* Use List interface where possible

Signed-off-by: Jacob Laursen <jacob-github@vindvejr.dk>
Signed-off-by: Ciprian Pascu <contact@ciprianpascu.ro>
This commit is contained in:
Jacob Laursen 2024-08-10 08:15:37 +02:00 committed by Ciprian Pascu
parent ca83e5d312
commit 4609d7156e
19 changed files with 209 additions and 157 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
bundles/org.openhab.binding.shelly/src/main/java/org/openhab/binding/shelly/internal/api/ShellyApiResult.java
View File

@ -92,10 +92,8 @@ public class ShellyApiResult {
httpReason = contentResponse.getReason(); httpReason = contentResponse.getReason();
Request request = contentResponse.getRequest(); Request request = contentResponse.getRequest();
if (request != null) { url = request.getURI().toString();
url = request.getURI().toString(); method = request.getMethod();
method = request.getMethod();
}
} }
} }
} }

55
bundles/org.openhab.binding.shelly/src/main/java/org/openhab/binding/shelly/internal/api/ShellyDeviceProfile.java
View File

@ -18,6 +18,7 @@ import static org.openhab.binding.shelly.internal.discovery.ShellyThingCreator.*
import static org.openhab.binding.shelly.internal.util.ShellyUtils.*; import static org.openhab.binding.shelly.internal.util.ShellyUtils.*;
import java.util.HashMap; import java.util.HashMap;
import java.util.List;
import java.util.Map; import java.util.Map;
import java.util.regex.Matcher; import java.util.regex.Matcher;
import java.util.regex.Pattern; import java.util.regex.Pattern;
@ -115,13 +116,11 @@ public class ShellyDeviceProfile {
public ShellyDeviceProfile() { public ShellyDeviceProfile() {
} }
public ShellyDeviceProfile initialize(String thingType, String jsonIn, @Nullable ShellySettingsDevice device) public ShellyDeviceProfile initialize(String thingType, String jsonIn, ShellySettingsDevice device)
throws ShellyApiException { throws ShellyApiException {
Gson gson = new Gson(); Gson gson = new Gson();
initialized = false; initialized = false;
if (device != null) { this.device = device;
this.device = device;
}
initFromThingType(thingType); initFromThingType(thingType);
@ -165,7 +164,8 @@ public class ShellyDeviceProfile {
} }
hasRelays = (numRelays > 0) || isDimmer; hasRelays = (numRelays > 0) || isDimmer;
numRollers = getInteger(device.numRollers); numRollers = getInteger(device.numRollers);
numInputs = settings.inputs != null ? settings.inputs.size() : hasRelays ? isRoller ? 2 : 1 : 0; List<ShellySettingsInput> inputs = settings.inputs;
numInputs = inputs != null ? inputs.size() : hasRelays ? isRoller ? 2 : 1 : 0;
isEMeter = settings.emeters != null; isEMeter = settings.emeters != null;
numMeters = !isEMeter ? getInteger(device.numMeters) : getInteger(device.numEMeters); numMeters = !isEMeter ? getInteger(device.numMeters) : getInteger(device.numEMeters);
@ -269,8 +269,8 @@ public class ShellyDeviceProfile {
} else if (hasRelays) { } else if (hasRelays) {
return numRelays <= 1 ? CHANNEL_GROUP_RELAY_CONTROL : CHANNEL_GROUP_RELAY_CONTROL + idx; return numRelays <= 1 ? CHANNEL_GROUP_RELAY_CONTROL : CHANNEL_GROUP_RELAY_CONTROL + idx;
} else if (isRGBW2) { } else if (isRGBW2) {
return settings.lights == null || settings.lights != null && settings.lights.size() <= 1 List<ShellySettingsRgbwLight> lights = settings.lights;
? CHANNEL_GROUP_LIGHT_CONTROL return lights == null || lights.size() <= 1 ? CHANNEL_GROUP_LIGHT_CONTROL
: CHANNEL_GROUP_LIGHT_CHANNEL + idx; : CHANNEL_GROUP_LIGHT_CHANNEL + idx;
} else if (isLight) { } else if (isLight) {
return CHANNEL_GROUP_LIGHT_CONTROL; return CHANNEL_GROUP_LIGHT_CONTROL;
@ -317,39 +317,42 @@ public class ShellyDeviceProfile {
return ""; return "";
} }
@SuppressWarnings("null")
public boolean inButtonMode(int idx) { public boolean inButtonMode(int idx) {
if (idx < 0) { if (idx < 0) {
logger.debug("{}: Invalid index {} for inButtonMode()", thingName, idx); logger.debug("{}: Invalid index {} for inButtonMode()", thingName, idx);
return false; return false;
} }
String btnType = ""; String btnType = "";
List<ShellySettingsInput> inputs = settings.inputs;
List<ShellySettingsDimmer> dimmers = settings.dimmers;
List<ShellySettingsRelay> relays = settings.relays;
List<ShellySettingsRgbwLight> lights = settings.lights;
if (isButton) { if (isButton) {
return true; return true;
} else if (isIX && settings.inputs != null && idx < settings.inputs.size()) { } else if (isIX && inputs != null && idx < inputs.size()) {
ShellySettingsInput input = settings.inputs.get(idx); ShellySettingsInput input = inputs.get(idx);
btnType = getString(input.btnType); btnType = getString(input.btnType);
} else if (isDimmer) { } else if (isDimmer) {
if (settings.dimmers != null) { if (dimmers != null) {
ShellySettingsDimmer dimmer = settings.dimmers.get(0); ShellySettingsDimmer dimmer = dimmers.get(0);
btnType = getString(dimmer.btnType); btnType = getString(dimmer.btnType);
} }
} else if (settings.relays != null) { } else if (relays != null) {
if (numRelays == 1) { if (numRelays == 1) {
ShellySettingsRelay relay = settings.relays.get(0); ShellySettingsRelay relay = relays.get(0);
if (relay.btnType != null) { if (relay.btnType != null) {
btnType = getString(relay.btnType); btnType = getString(relay.btnType);
} else { } else {
// Shelly 1L has 2 inputs // Shelly 1L has 2 inputs
btnType = idx == 0 ? getString(relay.btnType1) : getString(relay.btnType2); btnType = idx == 0 ? getString(relay.btnType1) : getString(relay.btnType2);
} }
} else if (idx < settings.relays.size()) { } else if (idx < relays.size()) {
// only one input channel // only one input channel
ShellySettingsRelay relay = settings.relays.get(idx); ShellySettingsRelay relay = relays.get(idx);
btnType = getString(relay.btnType); btnType = getString(relay.btnType);
} }
} else if (isRGBW2 && (settings.lights != null) && (idx < settings.lights.size())) { } else if (isRGBW2 && lights != null && idx < lights.size()) {
ShellySettingsRgbwLight light = settings.lights.get(idx); ShellySettingsRgbwLight light = lights.get(idx);
btnType = getString(light.btnType); btnType = getString(light.btnType);
} }
@ -367,13 +370,12 @@ public class ShellyDeviceProfile {
} }
public String[] getValveProfileList(int valveId) { public String[] getValveProfileList(int valveId) {
if (isTRV && settings.thermostats != null) { List<ShellyThermnostat> thermostats = settings.thermostats;
int sz = settings.thermostats.size(); if (isTRV && thermostats != null) {
int sz = thermostats.size();
if (valveId <= sz) { if (valveId <= sz) {
if (settings.thermostats != null) { ShellyThermnostat t = thermostats.get(valveId);
ShellyThermnostat t = settings.thermostats.get(valveId); return t.profileNames;
return t.profileNames;
}
} }
} }
return new String[0]; return new String[0];
@ -381,8 +383,9 @@ public class ShellyDeviceProfile {
public String getValueProfile(int valveId, int profileId) { public String getValueProfile(int valveId, int profileId) {
int id = profileId; int id = profileId;
if (id <= 0 && settings.thermostats != null) { List<ShellyThermnostat> thermostats = settings.thermostats;
id = settings.thermostats.get(0).profile; if (id <= 0 && thermostats != null) {
id = thermostats.get(0).profile;
} }
return "" + id; return "" + id;
} }

12
bundles/org.openhab.binding.shelly/src/main/java/org/openhab/binding/shelly/internal/api/ShellyEventServlet.java
View File

@ -77,9 +77,13 @@ public class ShellyEventServlet extends WebSocketServlet {
@Override @Override
protected void service(@Nullable HttpServletRequest request, @Nullable HttpServletResponse resp) protected void service(@Nullable HttpServletRequest request, @Nullable HttpServletResponse resp)
throws ServletException, IOException, IllegalArgumentException { throws ServletException, IOException, IllegalArgumentException {
if (request == null) {
logger.trace("ShellyEventServlet.service unexpectedly received a null request. Request not processed");
return;
}
String path = getString(request.getRequestURI()).toLowerCase(); String path = getString(request.getRequestURI()).toLowerCase();
if (path.equals(SHELLY2_CALLBACK_URI)) { // Shelly2 WebSocket if (path.equals(SHELLY2_CALLBACK_URI)) { // Shelly2 WebSocket
if (request != null && resp != null) { if (resp != null) {
super.service(request, resp); super.service(request, resp);
} }
return; return;
@ -123,8 +127,10 @@ public class ShellyEventServlet extends WebSocketServlet {
logger.debug("{}: Exception processing callback: path={}; index={}, type={}, parameters={}", deviceName, logger.debug("{}: Exception processing callback: path={}; index={}, type={}, parameters={}", deviceName,
path, index, type, request.getParameterMap().toString()); path, index, type, request.getParameterMap().toString());
} finally { } finally {
resp.setCharacterEncoding(StandardCharsets.UTF_8.toString()); if (resp != null) {
resp.getWriter().write(""); resp.setCharacterEncoding(StandardCharsets.UTF_8.toString());
resp.getWriter().write("");
}
} }
} }

20
bundles/org.openhab.binding.shelly/src/main/java/org/openhab/binding/shelly/internal/api1/Shelly1HttpApi.java
View File

@ -19,6 +19,7 @@ import static org.openhab.binding.shelly.internal.util.ShellyUtils.*;
import java.nio.charset.StandardCharsets; import java.nio.charset.StandardCharsets;
import java.util.Base64; import java.util.Base64;
import java.util.HashMap; import java.util.HashMap;
import java.util.List;
import java.util.Map; import java.util.Map;
import org.eclipse.jdt.annotation.NonNullByDefault; import org.eclipse.jdt.annotation.NonNullByDefault;
@ -33,6 +34,7 @@ import org.openhab.binding.shelly.internal.api1.Shelly1ApiJsonDTO.ShellyRollerSt
import org.openhab.binding.shelly.internal.api1.Shelly1ApiJsonDTO.ShellySendKeyList; import org.openhab.binding.shelly.internal.api1.Shelly1ApiJsonDTO.ShellySendKeyList;
import org.openhab.binding.shelly.internal.api1.Shelly1ApiJsonDTO.ShellySenseKeyCode; import org.openhab.binding.shelly.internal.api1.Shelly1ApiJsonDTO.ShellySenseKeyCode;
import org.openhab.binding.shelly.internal.api1.Shelly1ApiJsonDTO.ShellySettingsDevice; import org.openhab.binding.shelly.internal.api1.Shelly1ApiJsonDTO.ShellySettingsDevice;
import org.openhab.binding.shelly.internal.api1.Shelly1ApiJsonDTO.ShellySettingsDimmer;
import org.openhab.binding.shelly.internal.api1.Shelly1ApiJsonDTO.ShellySettingsLight; import org.openhab.binding.shelly.internal.api1.Shelly1ApiJsonDTO.ShellySettingsLight;
import org.openhab.binding.shelly.internal.api1.Shelly1ApiJsonDTO.ShellySettingsLogin; import org.openhab.binding.shelly.internal.api1.Shelly1ApiJsonDTO.ShellySettingsLogin;
import org.openhab.binding.shelly.internal.api1.Shelly1ApiJsonDTO.ShellySettingsStatus; import org.openhab.binding.shelly.internal.api1.Shelly1ApiJsonDTO.ShellySettingsStatus;
@ -53,8 +55,7 @@ import com.google.gson.JsonSyntaxException;
/** /**
* {@link Shelly1HttpApi} wraps the Shelly REST API and provides various low level function to access the device api * {@link Shelly1HttpApi} wraps the Shelly REST API and provides various low level function to access the device api
* (not * (not cloud api).
* cloud api).
* *
* @author Markus Michels - Initial contribution * @author Markus Michels - Initial contribution
*/ */
@ -279,8 +280,9 @@ public class Shelly1HttpApi extends ShellyHttpClient implements ShellyApiInterfa
@Override @Override
public void setValveMode(int valveId, boolean auto) throws ShellyApiException { public void setValveMode(int valveId, boolean auto) throws ShellyApiException {
String uri = "/settings/thermostat/" + valveId + "?target_t_enabled=" + (auto ? "1" : "0"); String uri = "/settings/thermostat/" + valveId + "?target_t_enabled=" + (auto ? "1" : "0");
if (auto && profile.settings.thermostats != null) { List<ShellyThermnostat> thermostats = profile.settings.thermostats;
uri = uri + "&target_t=" + getDouble(profile.settings.thermostats.get(0).targetTemp.value); if (auto && thermostats != null) {
uri = uri + "&target_t=" + getDouble(thermostats.get(0).targetTemp.value);
} }
httpRequest(uri); // percentage to open the valve httpRequest(uri); // percentage to open the valve
} }
@ -303,8 +305,9 @@ public class Shelly1HttpApi extends ShellyHttpClient implements ShellyApiInterfa
@Override @Override
public void startValveBoost(int valveId, int value) throws ShellyApiException { public void startValveBoost(int valveId, int value) throws ShellyApiException {
if (profile.settings.thermostats != null) { List<ShellyThermnostat> thermostats = profile.settings.thermostats;
ShellyThermnostat t = profile.settings.thermostats.get(0); if (thermostats != null) {
ShellyThermnostat t = thermostats.get(0);
int minutes = value != -1 ? value : getInteger(t.boostMinutes); int minutes = value != -1 ? value : getInteger(t.boostMinutes);
httpRequest("/thermostat/0?boost_minutes=" + minutes); httpRequest("/thermostat/0?boost_minutes=" + minutes);
} }
@ -541,8 +544,9 @@ public class Shelly1HttpApi extends ShellyHttpClient implements ShellyApiInterfa
} }
private void setDimmerEvents() throws ShellyApiException { private void setDimmerEvents() throws ShellyApiException {
if (profile.settings.dimmers != null) { List<ShellySettingsDimmer> dimmers = profile.settings.dimmers;
int sz = profile.settings.dimmers.size(); if (dimmers != null) {
int sz = dimmers.size();
for (int i = 0; i < sz; i++) { for (int i = 0; i < sz; i++) {
setEventUrls(i); setEventUrls(i);
} }

21
bundles/org.openhab.binding.shelly/src/main/java/org/openhab/binding/shelly/internal/api2/Shelly2ApiClient.java
View File

@ -18,6 +18,7 @@ import static org.openhab.binding.shelly.internal.api2.Shelly2ApiJsonDTO.*;
import static org.openhab.binding.shelly.internal.util.ShellyUtils.*; import static org.openhab.binding.shelly.internal.util.ShellyUtils.*;
import java.util.ArrayList; import java.util.ArrayList;
import java.util.List;
import java.util.Map; import java.util.Map;
import java.util.Random; import java.util.Random;
@ -296,9 +297,10 @@ public class Shelly2ApiClient extends ShellyHttpClient {
} }
private int getRelayIdx(ShellyDeviceProfile profile, @Nullable Integer id) { private int getRelayIdx(ShellyDeviceProfile profile, @Nullable Integer id) {
if (id != null && profile.settings.relays != null) { List<ShellySettingsRelay> relays = profile.settings.relays;
if (id != null && relays != null) {
int idx = 0; int idx = 0;
for (ShellySettingsRelay relay : profile.settings.relays) { for (ShellySettingsRelay relay : relays) {
if (relay.isValid && relay.id != null && relay.id.intValue() == id.intValue()) { if (relay.isValid && relay.id != null && relay.id.intValue() == id.intValue()) {
return idx; return idx;
} }
@ -561,12 +563,13 @@ public class Shelly2ApiClient extends ShellyHttpClient {
return; return;
} }
if (profile.settings.dimmers != null) { List<ShellySettingsDimmer> dimmers = profile.settings.dimmers;
ShellySettingsDimmer ds = profile.settings.dimmers.get(0); if (dimmers != null) {
ShellySettingsDimmer ds = dimmers.get(0);
ds.autoOn = dc.light0.autoOnDelay; ds.autoOn = dc.light0.autoOnDelay;
ds.autoOff = dc.light0.autoOffDelay; ds.autoOff = dc.light0.autoOffDelay;
ds.name = dc.light0.name; ds.name = dc.light0.name;
profile.settings.dimmers.set(0, ds); dimmers.set(0, ds);
} }
} }
@ -709,21 +712,22 @@ public class Shelly2ApiClient extends ShellyHttpClient {
} }
} }
protected @Nullable ArrayList<ShellySettingsInput> fillInputSettings(ShellyDeviceProfile profile, protected @Nullable ArrayList<@Nullable ShellySettingsInput> fillInputSettings(ShellyDeviceProfile profile,
Shelly2GetConfigResult dc) { Shelly2GetConfigResult dc) {
if (dc.input0 == null) { if (dc.input0 == null) {
return null; // device has no input return null; // device has no input
} }
ArrayList<ShellySettingsInput> inputs = new ArrayList<>(); ArrayList<@Nullable ShellySettingsInput> inputs = new ArrayList<>();
addInputSettings(inputs, dc.input0); addInputSettings(inputs, dc.input0);
addInputSettings(inputs, dc.input1); addInputSettings(inputs, dc.input1);
addInputSettings(inputs, dc.input2); addInputSettings(inputs, dc.input2);
addInputSettings(inputs, dc.input3); addInputSettings(inputs, dc.input3);
return inputs; return inputs;
} }
private void addInputSettings(ArrayList<ShellySettingsInput> inputs, @Nullable Shelly2DevConfigInput ic) { private void addInputSettings(List<@Nullable ShellySettingsInput> inputs, @Nullable Shelly2DevConfigInput ic) {
if (ic == null) { if (ic == null) {
return; return;
} }
@ -807,6 +811,7 @@ public class Shelly2ApiClient extends ShellyHttpClient {
} }
protected ShellyDeviceProfile getProfile() throws ShellyApiException { protected ShellyDeviceProfile getProfile() throws ShellyApiException {
ShellyThingInterface thing = this.thing;
if (thing != null) { if (thing != null) {
return thing.getProfile(); return thing.getProfile();
} }

45
bundles/org.openhab.binding.shelly/src/main/java/org/openhab/binding/shelly/internal/api2/Shelly2ApiRpc.java
View File

@ -44,9 +44,11 @@ import org.openhab.binding.shelly.internal.api1.Shelly1ApiJsonDTO.ShellySensorSl
import org.openhab.binding.shelly.internal.api1.Shelly1ApiJsonDTO.ShellySettingsDevice; import org.openhab.binding.shelly.internal.api1.Shelly1ApiJsonDTO.ShellySettingsDevice;
import org.openhab.binding.shelly.internal.api1.Shelly1ApiJsonDTO.ShellySettingsDimmer; import org.openhab.binding.shelly.internal.api1.Shelly1ApiJsonDTO.ShellySettingsDimmer;
import org.openhab.binding.shelly.internal.api1.Shelly1ApiJsonDTO.ShellySettingsEMeter; import org.openhab.binding.shelly.internal.api1.Shelly1ApiJsonDTO.ShellySettingsEMeter;
import org.openhab.binding.shelly.internal.api1.Shelly1ApiJsonDTO.ShellySettingsInput;
import org.openhab.binding.shelly.internal.api1.Shelly1ApiJsonDTO.ShellySettingsLogin; import org.openhab.binding.shelly.internal.api1.Shelly1ApiJsonDTO.ShellySettingsLogin;
import org.openhab.binding.shelly.internal.api1.Shelly1ApiJsonDTO.ShellySettingsMeter; import org.openhab.binding.shelly.internal.api1.Shelly1ApiJsonDTO.ShellySettingsMeter;
import org.openhab.binding.shelly.internal.api1.Shelly1ApiJsonDTO.ShellySettingsRelay; import org.openhab.binding.shelly.internal.api1.Shelly1ApiJsonDTO.ShellySettingsRelay;
import org.openhab.binding.shelly.internal.api1.Shelly1ApiJsonDTO.ShellySettingsRoller;
import org.openhab.binding.shelly.internal.api1.Shelly1ApiJsonDTO.ShellySettingsStatus; import org.openhab.binding.shelly.internal.api1.Shelly1ApiJsonDTO.ShellySettingsStatus;
import org.openhab.binding.shelly.internal.api1.Shelly1ApiJsonDTO.ShellySettingsUpdate; import org.openhab.binding.shelly.internal.api1.Shelly1ApiJsonDTO.ShellySettingsUpdate;
import org.openhab.binding.shelly.internal.api1.Shelly1ApiJsonDTO.ShellySettingsWiFiNetwork; import org.openhab.binding.shelly.internal.api1.Shelly1ApiJsonDTO.ShellySettingsWiFiNetwork;
@ -158,7 +160,6 @@ public class Shelly2ApiRpc extends Shelly2ApiClient implements ShellyApiInterfac
} }
} }
@SuppressWarnings("null")
@Override @Override
public ShellyDeviceProfile getDeviceProfile(String thingType, @Nullable ShellySettingsDevice devInfo) public ShellyDeviceProfile getDeviceProfile(String thingType, @Nullable ShellySettingsDevice devInfo)
throws ShellyApiException { throws ShellyApiException {
@ -197,9 +198,15 @@ public class Shelly2ApiRpc extends Shelly2ApiClient implements ShellyApiInterfac
profile.settings.rollers = fillRollerSettings(profile, dc); profile.settings.rollers = fillRollerSettings(profile, dc);
profile.isEMeter = true; profile.isEMeter = true;
profile.numInputs = profile.settings.inputs != null ? profile.settings.inputs.size() : 0; List<ShellySettingsInput> inputs = profile.settings.inputs;
profile.numRelays = profile.settings.relays != null ? profile.settings.relays.size() : 0; profile.numInputs = inputs != null ? inputs.size() : 0;
profile.numRollers = profile.settings.rollers != null ? profile.settings.rollers.size() : 0;
List<ShellySettingsRelay> relays = profile.settings.relays;
profile.numRelays = relays != null ? relays.size() : 0;
List<ShellySettingsRoller> rollers = profile.settings.rollers;
profile.numRollers = rollers != null ? rollers.size() : 0;
profile.hasRelays = profile.numRelays > 0 || profile.numRollers > 0; profile.hasRelays = profile.numRelays > 0 || profile.numRollers > 0;
if (getString(profile.device.mode).isEmpty() && profile.hasRelays) { if (getString(profile.device.mode).isEmpty() && profile.hasRelays) {
profile.device.mode = profile.isRoller ? SHELLY_CLASS_ROLLER : SHELLY_CLASS_RELAY; profile.device.mode = profile.isRoller ? SHELLY_CLASS_ROLLER : SHELLY_CLASS_RELAY;
@ -291,8 +298,9 @@ public class Shelly2ApiRpc extends Shelly2ApiClient implements ShellyApiInterfac
} }
if (profile.isDimmer) { if (profile.isDimmer) {
profile.settings.dimmers = new ArrayList<>(); ArrayList<@Nullable ShellySettingsDimmer> dimmers = new ArrayList<>();
profile.settings.dimmers.add(new ShellySettingsDimmer()); dimmers.add(new ShellySettingsDimmer());
profile.settings.dimmers = dimmers;
profile.status.dimmers = new ArrayList<>(); profile.status.dimmers = new ArrayList<>();
profile.status.dimmers.add(new ShellyShortLightStatus()); profile.status.dimmers.add(new ShellyShortLightStatus());
fillDimmerSettings(profile, dc); fillDimmerSettings(profile, dc);
@ -318,7 +326,7 @@ public class Shelly2ApiRpc extends Shelly2ApiClient implements ShellyApiInterfac
asyncApiRequest(SHELLYRPC_METHOD_GETSTATUS); // request periodic status updates from device asyncApiRequest(SHELLYRPC_METHOD_GETSTATUS); // request periodic status updates from device
try { try {
if (profile.alwaysOn && config.enableBluGateway != null && dc.ble != null) { if (profile.alwaysOn && dc.ble != null) {
logger.debug("{}: BLU Gateway support is {} for this device", thingName, logger.debug("{}: BLU Gateway support is {} for this device", thingName,
config.enableBluGateway ? "enabled" : "disabled"); config.enableBluGateway ? "enabled" : "disabled");
if (config.enableBluGateway) { if (config.enableBluGateway) {
@ -556,6 +564,7 @@ public class Shelly2ApiRpc extends Shelly2ApiClient implements ShellyApiInterfac
@Override @Override
public void onConnect(String deviceIp, boolean connected) { public void onConnect(String deviceIp, boolean connected) {
ShellyThingTable thingTable = this.thingTable;
if (thing == null && thingTable != null) { if (thing == null && thingTable != null) {
thing = thingTable.getThing(deviceIp); thing = thingTable.getThing(deviceIp);
logger.debug("{}: Get thing from thingTable", thingName); logger.debug("{}: Get thing from thingTable", thingName);
@ -751,12 +760,14 @@ public class Shelly2ApiRpc extends Shelly2ApiClient implements ShellyApiInterfac
@Override @Override
public void onError(Throwable cause) { public void onError(Throwable cause) {
logger.debug("{}: WebSocket error: {}", thingName, cause.getMessage()); logger.debug("{}: WebSocket error: {}", thingName, cause.getMessage());
ShellyThingInterface thing = this.thing;
if (thing != null && thing.getProfile().alwaysOn) { if (thing != null && thing.getProfile().alwaysOn) {
thingOffline("WebSocket error"); thingOffline("WebSocket error");
} }
} }
private void thingOffline(String reason) { private void thingOffline(String reason) {
ShellyThingInterface thing = this.thing;
if (thing != null) { // do not reinit of battery powered devices with sleep mode if (thing != null) { // do not reinit of battery powered devices with sleep mode
thing.setThingOfflineAndDisconnect(ThingStatusDetail.COMMUNICATION_ERROR, thing.setThingOfflineAndDisconnect(ThingStatusDetail.COMMUNICATION_ERROR,
"offline.status-error-unexpected-error", reason); "offline.status-error-unexpected-error", reason);
@ -862,8 +873,9 @@ public class Shelly2ApiRpc extends Shelly2ApiClient implements ShellyApiInterfac
public void setRelayTurn(int id, String turnMode) throws ShellyApiException { public void setRelayTurn(int id, String turnMode) throws ShellyApiException {
ShellyDeviceProfile profile = getProfile(); ShellyDeviceProfile profile = getProfile();
int rIdx = id; int rIdx = id;
if (profile.settings.relays != null) { List<ShellySettingsRelay> relays = profile.settings.relays;
Integer rid = profile.settings.relays.get(id).id; if (relays != null) {
Integer rid = relays.get(id).id;
if (rid != null) { if (rid != null) {
rIdx = rid; rIdx = rid;
} }
@ -1190,7 +1202,6 @@ public class Shelly2ApiRpc extends Shelly2ApiClient implements ShellyApiInterfac
rpcSocket.sendMessage(gson.toJson(request)); // submit, result wull be async rpcSocket.sendMessage(gson.toJson(request)); // submit, result wull be async
} }
@SuppressWarnings("null")
public <T> T apiRequest(String method, @Nullable Object params, Class<T> classOfT) throws ShellyApiException { public <T> T apiRequest(String method, @Nullable Object params, Class<T> classOfT) throws ShellyApiException {
String json = ""; String json = "";
Shelly2RpcBaseMessage req = buildRequest(method, params); Shelly2RpcBaseMessage req = buildRequest(method, params);
@ -1202,7 +1213,7 @@ public class Shelly2ApiRpc extends Shelly2ApiClient implements ShellyApiInterfac
String auth = getString(res.authChallenge); String auth = getString(res.authChallenge);
if (res.isHttpAccessUnauthorized() && !auth.isEmpty()) { if (res.isHttpAccessUnauthorized() && !auth.isEmpty()) {
String[] options = auth.split(","); String[] options = auth.split(",");
authInfo = new Shelly2AuthChallenge(); Shelly2AuthChallenge authInfo = this.authInfo = new Shelly2AuthChallenge();
for (String o : options) { for (String o : options) {
String key = substringBefore(o, "=").stripLeading().trim(); String key = substringBefore(o, "=").stripLeading().trim();
String value = substringAfter(o, "=").replace("\"", "").trim(); String value = substringAfter(o, "=").replace("\"", "").trim();
@ -1229,17 +1240,22 @@ public class Shelly2ApiRpc extends Shelly2ApiClient implements ShellyApiInterfac
} }
Shelly2RpcBaseMessage response = gson.fromJson(json, Shelly2RpcBaseMessage.class); Shelly2RpcBaseMessage response = gson.fromJson(json, Shelly2RpcBaseMessage.class);
if (response == null) { if (response == null) {
throw new IllegalArgumentException("Unable to cover API result to obhect"); throw new ShellyApiException("Unable to convert API result to object");
} }
if (response.result != null) { if (response.result != null) {
// return sub element result as requested class type // return sub element result as requested class type
json = gson.toJson(gson.fromJson(json, Shelly2RpcBaseMessage.class).result); json = gson.toJson(response.result);
boolean isString = response.result instanceof String; boolean isString = response.result instanceof String;
return fromJson(gson, isString && "null".equalsIgnoreCase(((String) response.result)) ? "{}" : json, return fromJson(gson, isString && "null".equalsIgnoreCase(((String) response.result)) ? "{}" : json,
classOfT); classOfT);
} else { } else {
// return direct format // return direct format
return gson.fromJson(json, classOfT == String.class ? Shelly2RpcBaseMessage.class : classOfT); @Nullable
T result = gson.fromJson(json, classOfT == String.class ? Shelly2RpcBaseMessage.class : classOfT);
if (result == null) {
throw new ShellyApiException("Unable to convert API result to object");
}
return result;
} }
} }
@ -1284,6 +1300,7 @@ public class Shelly2ApiRpc extends Shelly2ApiClient implements ShellyApiInterfac
} }
private void incProtErrors() { private void incProtErrors() {
ShellyThingInterface thing = this.thing;
if (thing != null) { if (thing != null) {
thing.incProtErrors(); thing.incProtErrors();
} }

53
bundles/org.openhab.binding.shelly/src/main/java/org/openhab/binding/shelly/internal/api2/Shelly2RpcSocket.java
View File

@ -33,7 +33,6 @@ import org.eclipse.jetty.websocket.api.annotations.WebSocket;
import org.eclipse.jetty.websocket.client.ClientUpgradeRequest; import org.eclipse.jetty.websocket.client.ClientUpgradeRequest;
import org.eclipse.jetty.websocket.client.WebSocketClient; import org.eclipse.jetty.websocket.client.WebSocketClient;
import org.openhab.binding.shelly.internal.api.ShellyApiException; import org.openhab.binding.shelly.internal.api.ShellyApiException;
import org.openhab.binding.shelly.internal.api1.Shelly1HttpApi;
import org.openhab.binding.shelly.internal.api2.Shelly2ApiJsonDTO.Shelly2NotifyEvent; import org.openhab.binding.shelly.internal.api2.Shelly2ApiJsonDTO.Shelly2NotifyEvent;
import org.openhab.binding.shelly.internal.api2.Shelly2ApiJsonDTO.Shelly2RpcBaseMessage; import org.openhab.binding.shelly.internal.api2.Shelly2ApiJsonDTO.Shelly2RpcBaseMessage;
import org.openhab.binding.shelly.internal.api2.Shelly2ApiJsonDTO.Shelly2RpcNotifyEvent; import org.openhab.binding.shelly.internal.api2.Shelly2ApiJsonDTO.Shelly2RpcNotifyEvent;
@ -47,10 +46,6 @@ import org.slf4j.LoggerFactory;
import com.google.gson.Gson; import com.google.gson.Gson;
/** /**
* {@link Shelly1HttpApi} wraps the Shelly REST API and provides various low level function to access the device api
* (not
* cloud api).
*
* @author Markus Michels - Initial contribution * @author Markus Michels - Initial contribution
*/ */
@NonNullByDefault @NonNullByDefault
@ -151,11 +146,13 @@ public class Shelly2RpcSocket {
// This is the inbound event web socket // This is the inbound event web socket
deviceIp = session.getRemoteAddress().getAddress().getHostAddress(); deviceIp = session.getRemoteAddress().getAddress().getHostAddress();
} }
Shelly2RpctInterface websocketHandler = this.websocketHandler;
if (websocketHandler == null) { if (websocketHandler == null) {
ShellyThingTable thingTable = this.thingTable;
if (thingTable != null) { if (thingTable != null) {
ShellyThingInterface thing = thingTable.getThing(deviceIp); ShellyThingInterface thing = thingTable.getThing(deviceIp);
Shelly2ApiRpc api = (Shelly2ApiRpc) thing.getApi(); Shelly2ApiRpc api = (Shelly2ApiRpc) thing.getApi();
websocketHandler = api.getRpcHandler(); websocketHandler = this.websocketHandler = api.getRpcHandler();
} }
} }
connectLatch.countDown(); connectLatch.countDown();
@ -182,8 +179,8 @@ public class Shelly2RpcSocket {
* @param str API request message * @param str API request message
* @throws ShellyApiException * @throws ShellyApiException
*/ */
@SuppressWarnings("null")
public void sendMessage(String str) throws ShellyApiException { public void sendMessage(String str) throws ShellyApiException {
Session session = this.session;
if (session != null) { if (session != null) {
try { try {
connectLatch.await(); connectLatch.await();
@ -201,15 +198,15 @@ public class Shelly2RpcSocket {
*/ */
public void disconnect() { public void disconnect() {
try { try {
Session session = this.session;
if (session != null) { if (session != null) {
Session s = session; if (session.isOpen()) {
if (s.isOpen()) { logger.debug("{}: Disconnecting WebSocket ({} -> {})", thingName, session.getLocalAddress(),
logger.debug("{}: Disconnecting WebSocket ({} -> {})", thingName, s.getLocalAddress(), session.getRemoteAddress());
s.getRemoteAddress());
} }
s.disconnect(); session.disconnect();
s.close(StatusCode.NORMAL, "Socket closed"); session.close(StatusCode.NORMAL, "Socket closed");
session = null; this.session = null;
} }
} catch (Exception e) { } catch (Exception e) {
if (e.getCause() instanceof InterruptedException) { if (e.getCause() instanceof InterruptedException) {
@ -230,7 +227,7 @@ public class Shelly2RpcSocket {
/** /**
* Inbound WebSocket message * Inbound WebSocket message
* *
* @param session WebSpcket session * @param session WebSocket session
* @param receivedMessage Textial API message * @param receivedMessage Textial API message
*/ */
@OnWebSocketMessage @OnWebSocketMessage
@ -263,17 +260,18 @@ public class Shelly2RpcSocket {
} else { } else {
for (Shelly2NotifyEvent e : events.params.events) { for (Shelly2NotifyEvent e : events.params.events) {
if (getString(e.event).startsWith(SHELLY2_EVENT_BLUPREFIX)) { if (getString(e.event).startsWith(SHELLY2_EVENT_BLUPREFIX)) {
String address = getString(e.data.addr).replace(":", ""); String address = getString(e.data != null ? e.data.addr : "").replace(":", "");
ShellyThingTable thingTable = this.thingTable;
if (thingTable != null && thingTable.findThing(address) != null) { if (thingTable != null && thingTable.findThing(address) != null) {
if (thingTable != null) { // known device // known device
ShellyThingInterface thing = thingTable.getThing(address); ShellyThingInterface thing = thingTable.getThing(address);
Shelly2ApiRpc api = (Shelly2ApiRpc) thing.getApi(); Shelly2ApiRpc api = (Shelly2ApiRpc) thing.getApi();
handler = api.getRpcHandler(); handler = api.getRpcHandler();
handler.onNotifyEvent( handler.onNotifyEvent(
fromJson(gson, receivedMessage, Shelly2RpcNotifyEvent.class)); fromJson(gson, receivedMessage, Shelly2RpcNotifyEvent.class));
} } else {
} else { // new device // new device
if (e.event.equals(SHELLY2_EVENT_BLUSCAN)) { if (SHELLY2_EVENT_BLUSCAN.equals(e.event)) {
ShellyBluSensorHandler.addBluThing(message.src, e, thingTable); ShellyBluSensorHandler.addBluThing(message.src, e, thingTable);
} else { } else {
logger.debug("{}: NotifyEvent {} for unknown device {}", message.src, logger.debug("{}: NotifyEvent {} for unknown device {}", message.src,
@ -295,12 +293,11 @@ public class Shelly2RpcSocket {
} }
} catch (ShellyApiException | IllegalArgumentException e) { } catch (ShellyApiException | IllegalArgumentException e) {
logger.debug("{}: Unable to process Rpc message ({}): {}", thingName, e.getMessage(), receivedMessage); logger.debug("{}: Unable to process Rpc message ({}): {}", thingName, e.getMessage(), receivedMessage);
} catch (NullPointerException e) {
logger.debug("{}: Unable to process Rpc message: {}", thingName, receivedMessage, e);
} }
} }
public boolean isConnected() { public boolean isConnected() {
Session session = this.session;
return session != null && session.isOpen(); return session != null && session.isOpen();
} }
@ -324,6 +321,7 @@ public class Shelly2RpcSocket {
return; return;
} }
disconnect(); disconnect();
Shelly2RpctInterface websocketHandler = this.websocketHandler;
if (websocketHandler != null) { if (websocketHandler != null) {
websocketHandler.onClose(statusCode, reason); websocketHandler.onClose(statusCode, reason);
} }
@ -340,6 +338,7 @@ public class Shelly2RpcSocket {
// Ignore disconnect: Device establishes the socket, sends NotifyxFullStatus and disconnects // Ignore disconnect: Device establishes the socket, sends NotifyxFullStatus and disconnects
return; return;
} }
Shelly2RpctInterface websocketHandler = this.websocketHandler;
if (websocketHandler != null) { if (websocketHandler != null) {
websocketHandler.onError(cause); websocketHandler.onError(cause);
} }

10
bundles/org.openhab.binding.shelly/src/main/java/org/openhab/binding/shelly/internal/api2/ShellyBluApi.java
View File

@ -18,6 +18,7 @@ import static org.openhab.binding.shelly.internal.api2.Shelly2ApiJsonDTO.*;
import static org.openhab.binding.shelly.internal.util.ShellyUtils.*; import static org.openhab.binding.shelly.internal.util.ShellyUtils.*;
import java.util.ArrayList; import java.util.ArrayList;
import java.util.List;
import java.util.Map; import java.util.Map;
import org.eclipse.jdt.annotation.NonNullByDefault; import org.eclipse.jdt.annotation.NonNullByDefault;
@ -154,11 +155,12 @@ public class ShellyBluApi extends Shelly2ApiRpc {
profile.numInputs = 1; profile.numInputs = 1;
settings.btnType = SHELLY_BTNT_MOMENTARY; settings.btnType = SHELLY_BTNT_MOMENTARY;
if (profile.settings.inputs != null) { List<ShellySettingsInput> inputs = profile.settings.inputs;
profile.settings.inputs.set(0, settings); if (inputs != null) {
inputs.set(0, settings);
} else { } else {
profile.settings.inputs = new ArrayList<>(); inputs = profile.settings.inputs = new ArrayList<>();
profile.settings.inputs.add(settings); inputs.add(settings);
} }
profile.status = deviceStatus; profile.status = deviceStatus;
} }

1
bundles/org.openhab.binding.shelly/src/main/java/org/openhab/binding/shelly/internal/discovery/ShellyBasicDiscoveryService.java
View File

@ -97,6 +97,7 @@ public class ShellyBasicDiscoveryService extends AbstractDiscoveryService {
} }
public void unregisterDeviceDiscoveryService() { public void unregisterDeviceDiscoveryService() {
ServiceRegistration<?> discoveryService = this.discoveryService;
if (discoveryService != null) { if (discoveryService != null) {
discoveryService.unregister(); discoveryService.unregister();
} }

57
bundles/org.openhab.binding.shelly/src/main/java/org/openhab/binding/shelly/internal/handler/ShellyBaseHandler.java
View File

@ -269,6 +269,7 @@ public abstract class ShellyBaseHandler extends BaseThingHandler
public void handleConfigurationUpdate(Map<String, Object> configurationParameters) { public void handleConfigurationUpdate(Map<String, Object> configurationParameters) {
super.handleConfigurationUpdate(configurationParameters); super.handleConfigurationUpdate(configurationParameters);
logger.debug("{}: Thing config updated, re-initialize", thingName); logger.debug("{}: Thing config updated, re-initialize", thingName);
Shelly1CoapHandler coap = this.coap;
if (coap != null) { if (coap != null) {
coap.stop(); coap.stop();
} }
@ -315,6 +316,7 @@ public abstract class ShellyBaseHandler extends BaseThingHandler
// Gen 1 only: Setup CoAP listener to we get the CoAP message, which triggers initialization even the thing // Gen 1 only: Setup CoAP listener to we get the CoAP message, which triggers initialization even the thing
// could not be fully initialized here. In this case the CoAP messages triggers auto-initialization (like the // could not be fully initialized here. In this case the CoAP messages triggers auto-initialization (like the
// Action URL does when enabled) // Action URL does when enabled)
Shelly1CoapHandler coap = this.coap;
if (coap != null && config.eventsCoIoT && !profile.alwaysOn) { if (coap != null && config.eventsCoIoT && !profile.alwaysOn) {
coap.start(thingName, config); coap.start(thingName, config);
} }
@ -453,10 +455,11 @@ public abstract class ShellyBaseHandler extends BaseThingHandler
id = getNumber(command).intValue(); id = getNumber(command).intValue();
} else { } else {
String cmd = command.toString(); String cmd = command.toString();
List<ShellyThermnostat> thermostats = profile.settings.thermostats;
if (isDigit(cmd.charAt(0))) { if (isDigit(cmd.charAt(0))) {
id = Integer.parseInt(cmd); id = Integer.parseInt(cmd);
} else if (profile.settings.thermostats != null) { } else if (thermostats != null) {
ShellyThermnostat t = profile.settings.thermostats.get(0); ShellyThermnostat t = thermostats.get(0);
for (int i = 0; i < t.profileNames.length; i++) { for (int i = 0; i < t.profileNames.length; i++) {
if (t.profileNames[i].equalsIgnoreCase(cmd)) { if (t.profileNames[i].equalsIgnoreCase(cmd)) {
id = i + 1; id = i + 1;
@ -1077,35 +1080,30 @@ public abstract class ShellyBaseHandler extends BaseThingHandler
} }
private void checkVersion(ShellyDeviceProfile prf, ShellySettingsStatus status) { private void checkVersion(ShellyDeviceProfile prf, ShellySettingsStatus status) {
try { if (prf.fwVersion.isEmpty()) {
if (prf.fwVersion.isEmpty()) { // no fw version available (e.g. BLU device)
// no fw version available (e.g. BLU device) return;
return; }
ShellyVersionDTO version = new ShellyVersionDTO();
if (version.checkBeta(getString(prf.fwVersion))) {
logger.info("{}: {}", prf.device.hostname, messages.get("versioncheck.beta", prf.fwVersion, prf.fwDate));
} else {
String minVersion = !gen2 ? SHELLY_API_MIN_FWVERSION : SHELLY2_API_MIN_FWVERSION;
if (version.compare(prf.fwVersion, minVersion) < 0) {
logger.warn("{}: {}", prf.device.hostname,
messages.get("versioncheck.tooold", prf.fwVersion, prf.fwDate, minVersion));
} }
ShellyVersionDTO version = new ShellyVersionDTO(); }
if (version.checkBeta(getString(prf.fwVersion))) { if (!gen2 && bindingConfig.autoCoIoT && ((version.compare(prf.fwVersion, SHELLY_API_MIN_FWCOIOT)) >= 0)
logger.info("{}: {}", prf.device.hostname, || ("production_test".equalsIgnoreCase(prf.fwVersion))) {
messages.get("versioncheck.beta", prf.fwVersion, prf.fwDate)); if (!config.eventsCoIoT) {
} else { logger.info("{}: {}", thingName, messages.get("versioncheck.autocoiot"));
String minVersion = !gen2 ? SHELLY_API_MIN_FWVERSION : SHELLY2_API_MIN_FWVERSION;
if (version.compare(prf.fwVersion, minVersion) < 0) {
logger.warn("{}: {}", prf.device.hostname,
messages.get("versioncheck.tooold", prf.fwVersion, prf.fwDate, minVersion));
}
} }
if (!gen2 && bindingConfig.autoCoIoT && ((version.compare(prf.fwVersion, SHELLY_API_MIN_FWCOIOT)) >= 0) autoCoIoT = true;
|| ("production_test".equalsIgnoreCase(prf.fwVersion))) { }
if (!config.eventsCoIoT) { if (Boolean.TRUE.equals(status.update.hasUpdate) && !version.checkBeta(getString(prf.fwVersion))) {
logger.info("{}: {}", thingName, messages.get("versioncheck.autocoiot")); logger.info("{}: {}", thingName,
} messages.get("versioncheck.update", status.update.oldVersion, status.update.newVersion));
autoCoIoT = true;
}
if (status.update.hasUpdate && !version.checkBeta(getString(prf.fwVersion))) {
logger.info("{}: {}", thingName,
messages.get("versioncheck.update", status.update.oldVersion, status.update.newVersion));
}
} catch (NullPointerException e) { // could be inconsistant format of beta version
logger.debug("{}: {}", thingName, messages.get("versioncheck.failed", prf.fwVersion));
} }
} }
@ -1148,6 +1146,7 @@ public abstract class ShellyBaseHandler extends BaseThingHandler
} }
logger.debug("{}: Starting CoIoT (autoCoIoT={}/{})", thingName, bindingConfig.autoCoIoT, autoCoIoT); logger.debug("{}: Starting CoIoT (autoCoIoT={}/{})", thingName, bindingConfig.autoCoIoT, autoCoIoT);
Shelly1CoapHandler coap = this.coap;
if (coap != null) { if (coap != null) {
coap.start(thingName, config); coap.start(thingName, config);
} }

4
bundles/org.openhab.binding.shelly/src/main/java/org/openhab/binding/shelly/internal/handler/ShellyColorUtils.java
View File

@ -149,9 +149,9 @@ public class ShellyColorUtils {
for (int i = 0; i < rgbw.length; i++) { for (int i = 0; i < rgbw.length; i++) {
values[i] = Integer.parseInt(rgbw[i]); values[i] = Integer.parseInt(rgbw[i]);
} }
} catch (NullPointerException e) { // might be a format problem } catch (NumberFormatException e) { // might be a format problem
throw new IllegalArgumentException( throw new IllegalArgumentException(
"Unable to convert fullColor value: " + rgbwString + ", " + e.getMessage()); "Unable to convert fullColor value: " + rgbwString + ", " + e.getMessage(), e);
} }
if (values[0] != -1) { if (values[0] != -1) {
setRed(values[0]); setRed(values[0]);

35
bundles/org.openhab.binding.shelly/src/main/java/org/openhab/binding/shelly/internal/handler/ShellyComponents.java
View File

@ -16,6 +16,8 @@ import static org.openhab.binding.shelly.internal.ShellyBindingConstants.*;
import static org.openhab.binding.shelly.internal.api1.Shelly1ApiJsonDTO.*; import static org.openhab.binding.shelly.internal.api1.Shelly1ApiJsonDTO.*;
import static org.openhab.binding.shelly.internal.util.ShellyUtils.*; import static org.openhab.binding.shelly.internal.util.ShellyUtils.*;
import java.util.List;
import org.eclipse.jdt.annotation.NonNullByDefault; import org.eclipse.jdt.annotation.NonNullByDefault;
import org.eclipse.jdt.annotation.Nullable; import org.eclipse.jdt.annotation.Nullable;
import org.openhab.binding.shelly.internal.api.ShellyApiException; import org.openhab.binding.shelly.internal.api.ShellyApiException;
@ -100,8 +102,9 @@ public class ShellyComponents {
ShellyDeviceProfile profile = thingHandler.getProfile(); ShellyDeviceProfile profile = thingHandler.getProfile();
ShellySettingsRelay relay = status.relays.get(id); ShellySettingsRelay relay = status.relays.get(id);
ShellySettingsRelay rsettings; ShellySettingsRelay rsettings;
if (profile.settings.relays != null) { List<ShellySettingsRelay> relays = profile.settings.relays;
rsettings = profile.settings.relays.get(id); if (relays != null) {
rsettings = relays.get(id);
} else { } else {
throw new IllegalArgumentException("No relay settings"); throw new IllegalArgumentException("No relay settings");
} }
@ -409,8 +412,9 @@ public class ShellyComponents {
temp.doubleValue(), getString(sdata.tmp.units)); temp.doubleValue(), getString(sdata.tmp.units));
} else if (status.thermostats != null) { } else if (status.thermostats != null) {
// Shelly TRV // Shelly TRV
if (profile.settings.thermostats != null) { List<ShellyThermnostat> thermostats = profile.settings.thermostats;
ShellyThermnostat ps = profile.settings.thermostats.get(0); if (thermostats != null) {
ShellyThermnostat ps = thermostats.get(0);
ShellyThermnostat t = status.thermostats.get(0); ShellyThermnostat t = status.thermostats.get(0);
int bminutes = getInteger(t.boostMinutes) >= 0 ? getInteger(t.boostMinutes) int bminutes = getInteger(t.boostMinutes) >= 0 ? getInteger(t.boostMinutes)
: getInteger(ps.boostMinutes); : getInteger(ps.boostMinutes);
@ -586,9 +590,12 @@ public class ShellyComponents {
.createDimmerChannels(thingHandler.getThing(), profile, dstatus, l)); .createDimmerChannels(thingHandler.getThing(), profile, dstatus, l));
} }
ShellySettingsDimmer ds = profile.settings.dimmers.get(l); List<ShellySettingsDimmer> dimmers = profile.settings.dimmers;
if (ds.name != null) { if (dimmers != null) {
updated |= thingHandler.updateChannel(groupName, CHANNEL_OUTPUT_NAME, getStringType(ds.name)); ShellySettingsDimmer ds = dimmers.get(l);
if (ds.name != null) {
updated |= thingHandler.updateChannel(groupName, CHANNEL_OUTPUT_NAME, getStringType(ds.name));
}
} }
// On a status update we map a dimmer.ison = false to brightness 0 rather than the device's brightness // On a status update we map a dimmer.ison = false to brightness 0 rather than the device's brightness
@ -606,14 +613,12 @@ public class ShellyComponents {
} }
} }
if (profile.settings.dimmers != null) { if (dimmers != null) {
ShellySettingsDimmer dsettings = profile.settings.dimmers.get(l); ShellySettingsDimmer dsettings = dimmers.get(l);
if (dsettings != null) { updated |= thingHandler.updateChannel(groupName, CHANNEL_TIMER_AUTOON,
updated |= thingHandler.updateChannel(groupName, CHANNEL_TIMER_AUTOON, toQuantityType(getDouble(dsettings.autoOn), Units.SECOND));
toQuantityType(getDouble(dsettings.autoOn), Units.SECOND)); updated |= thingHandler.updateChannel(groupName, CHANNEL_TIMER_AUTOOFF,
updated |= thingHandler.updateChannel(groupName, CHANNEL_TIMER_AUTOOFF, toQuantityType(getDouble(dsettings.autoOff), Units.SECOND));
toQuantityType(getDouble(dsettings.autoOff), Units.SECOND));
}
} }
l++; l++;

6
bundles/org.openhab.binding.shelly/src/main/java/org/openhab/binding/shelly/internal/handler/ShellyLightHandler.java
View File

@ -16,6 +16,7 @@ import static org.openhab.binding.shelly.internal.ShellyBindingConstants.*;
import static org.openhab.binding.shelly.internal.api1.Shelly1ApiJsonDTO.*; import static org.openhab.binding.shelly.internal.api1.Shelly1ApiJsonDTO.*;
import static org.openhab.binding.shelly.internal.util.ShellyUtils.*; import static org.openhab.binding.shelly.internal.util.ShellyUtils.*;
import java.util.List;
import java.util.Map; import java.util.Map;
import java.util.TreeMap; import java.util.TreeMap;
@ -335,9 +336,10 @@ public class ShellyLightHandler extends ShellyBaseHandler {
ShellyColorUtils col = getCurrentColors(lightId); ShellyColorUtils col = getCurrentColors(lightId);
col.power = getOnOff(light.ison); col.power = getOnOff(light.ison);
if (profile.settings.lights != null) { List<ShellySettingsRgbwLight> lights = profile.settings.lights;
if (lights != null) {
// Channel control/timer // Channel control/timer
ShellySettingsRgbwLight ls = profile.settings.lights.get(lightId); ShellySettingsRgbwLight ls = lights.get(lightId);
updated |= updateChannel(controlGroup, CHANNEL_TIMER_AUTOON, updated |= updateChannel(controlGroup, CHANNEL_TIMER_AUTOON,
toQuantityType(getDouble(ls.autoOn), Units.SECOND)); toQuantityType(getDouble(ls.autoOn), Units.SECOND));
updated |= updateChannel(controlGroup, CHANNEL_TIMER_AUTOOFF, updated |= updateChannel(controlGroup, CHANNEL_TIMER_AUTOOFF,

8
bundles/org.openhab.binding.shelly/src/main/java/org/openhab/binding/shelly/internal/handler/ShellyThingTable.java
View File

@ -82,7 +82,8 @@ public class ShellyThingTable {
public void startDiscoveryService(BundleContext bundleContext) { public void startDiscoveryService(BundleContext bundleContext) {
if (discoveryService == null) { if (discoveryService == null) {
discoveryService = new ShellyBasicDiscoveryService(bundleContext, this); ShellyBasicDiscoveryService discoveryService = this.discoveryService = new ShellyBasicDiscoveryService(
bundleContext, this);
discoveryService.registerDeviceDiscoveryService(); discoveryService.registerDeviceDiscoveryService();
} }
} }
@ -94,20 +95,23 @@ public class ShellyThingTable {
} }
public void stopDiscoveryService() { public void stopDiscoveryService() {
ShellyBasicDiscoveryService discoveryService = this.discoveryService;
if (discoveryService != null) { if (discoveryService != null) {
discoveryService.unregisterDeviceDiscoveryService(); discoveryService.unregisterDeviceDiscoveryService();
discoveryService = null; this.discoveryService = null;
} }
} }
public void discoveredResult(ThingTypeUID uid, String model, String serviceName, String address, public void discoveredResult(ThingTypeUID uid, String model, String serviceName, String address,
Map<String, Object> properties) { Map<String, Object> properties) {
ShellyBasicDiscoveryService discoveryService = this.discoveryService;
if (discoveryService != null) { if (discoveryService != null) {
discoveryService.discoveredResult(uid, model, serviceName, address, properties); discoveryService.discoveredResult(uid, model, serviceName, address, properties);
} }
} }
public void discoveredResult(DiscoveryResult result) { public void discoveredResult(DiscoveryResult result) {
ShellyBasicDiscoveryService discoveryService = this.discoveryService;
if (discoveryService != null) { if (discoveryService != null) {
discoveryService.discoveredResult(result); discoveryService.discoveredResult(result);
} }

8
bundles/org.openhab.binding.shelly/src/main/java/org/openhab/binding/shelly/internal/manager/ShellyManagerCache.java
View File

@ -16,6 +16,7 @@ import java.util.Date;
import java.util.Map; import java.util.Map;
import java.util.concurrent.ConcurrentHashMap; import java.util.concurrent.ConcurrentHashMap;
import org.eclipse.jdt.annotation.NonNullByDefault;
import org.eclipse.jdt.annotation.Nullable; import org.eclipse.jdt.annotation.Nullable;
/** /**
@ -23,6 +24,7 @@ import org.eclipse.jdt.annotation.Nullable;
* *
* @author Markus Michels - Initial contribution * @author Markus Michels - Initial contribution
*/ */
@NonNullByDefault
public class ShellyManagerCache<K, V> extends ConcurrentHashMap<K, V> { public class ShellyManagerCache<K, V> extends ConcurrentHashMap<K, V> {
private static final long serialVersionUID = 1L; private static final long serialVersionUID = 1L;
@ -56,8 +58,9 @@ public class ShellyManagerCache<K, V> extends ConcurrentHashMap<K, V> {
throw new IllegalArgumentException(); throw new IllegalArgumentException();
} }
for (K key : m.keySet()) { for (K key : m.keySet()) {
@Nullable
V value = m.get(key); V value = m.get(key);
if (value != null) { // don't allow null values if (key != null && value != null) { // don't allow null values
put(key, value); put(key, value);
} }
} }
@ -87,7 +90,8 @@ public class ShellyManagerCache<K, V> extends ConcurrentHashMap<K, V> {
private void cleanMap() { private void cleanMap() {
long currentTime = new Date().getTime(); long currentTime = new Date().getTime();
for (K key : timeMap.keySet()) { for (K key : timeMap.keySet()) {
if (currentTime > (timeMap.get(key) + expiryInMillis)) { Long timeValue = timeMap.get(key);
if (key != null && (timeValue == null || currentTime > (timeValue + expiryInMillis))) {
remove(key); remove(key);
timeMap.remove(key); timeMap.remove(key);
} }

5
bundles/org.openhab.binding.shelly/src/main/java/org/openhab/binding/shelly/internal/manager/ShellyManagerPage.java
View File

@ -591,12 +591,11 @@ public class ShellyManagerPage {
timer.schedule(task, delay * 1000); timer.schedule(task, delay * 1000);
} }
protected @Nullable Map<String, ShellyManagerInterface> getThingHandlers() { protected Map<String, ShellyManagerInterface> getThingHandlers() {
return handlerFactory.getThingHandlers(); return handlerFactory.getThingHandlers();
} }
protected @Nullable ShellyManagerInterface getThingHandler(String uid) { protected @Nullable ShellyManagerInterface getThingHandler(String uid) {
Map<String, ShellyManagerInterface> th = getThingHandlers(); return getThingHandlers().get(uid);
return th != null ? th.get(uid) : null;
} }
} }

15
bundles/org.openhab.binding.shelly/src/main/java/org/openhab/binding/shelly/internal/provider/ShellyChannelDefinitions.java
View File

@ -360,8 +360,9 @@ public class ShellyChannelDefinitions {
Map<String, Channel> add = new LinkedHashMap<>(); Map<String, Channel> add = new LinkedHashMap<>();
String group = profile.getControlGroup(idx); String group = profile.getControlGroup(idx);
if (profile.settings.relays != null) { List<ShellySettingsRelay> relays = profile.settings.relays;
ShellySettingsRelay rs = profile.settings.relays.get(idx); if (relays != null) {
ShellySettingsRelay rs = relays.get(idx);
addChannel(thing, add, rs.ison != null, group, CHANNEL_OUTPUT); addChannel(thing, add, rs.ison != null, group, CHANNEL_OUTPUT);
addChannel(thing, add, rs.name != null, group, CHANNEL_OUTPUT_NAME); addChannel(thing, add, rs.name != null, group, CHANNEL_OUTPUT_NAME);
@ -408,8 +409,9 @@ public class ShellyChannelDefinitions {
// Shelly Dimmer has an additional brightness channel // Shelly Dimmer has an additional brightness channel
addChannel(thing, add, profile.isDimmer, group, CHANNEL_BRIGHTNESS); addChannel(thing, add, profile.isDimmer, group, CHANNEL_BRIGHTNESS);
if (profile.settings.dimmers != null) { List<ShellySettingsDimmer> dimmers = profile.settings.dimmers;
ShellySettingsDimmer ds = profile.settings.dimmers.get(idx); if (dimmers != null) {
ShellySettingsDimmer ds = dimmers.get(idx);
addChannel(thing, add, ds.name != null, group, CHANNEL_OUTPUT_NAME); addChannel(thing, add, ds.name != null, group, CHANNEL_OUTPUT_NAME);
addChannel(thing, add, ds.autoOn != null, group, CHANNEL_TIMER_AUTOON); addChannel(thing, add, ds.autoOn != null, group, CHANNEL_TIMER_AUTOON);
addChannel(thing, add, ds.autoOff != null, group, CHANNEL_TIMER_AUTOOFF); addChannel(thing, add, ds.autoOff != null, group, CHANNEL_TIMER_AUTOOFF);
@ -424,8 +426,9 @@ public class ShellyChannelDefinitions {
Map<String, Channel> add = new LinkedHashMap<>(); Map<String, Channel> add = new LinkedHashMap<>();
String group = profile.getControlGroup(idx); String group = profile.getControlGroup(idx);
if (profile.settings.lights != null) { List<ShellySettingsRgbwLight> lights = profile.settings.lights;
ShellySettingsRgbwLight light = profile.settings.lights.get(idx); if (lights != null) {
ShellySettingsRgbwLight light = lights.get(idx);
String whiteGroup = profile.isRGBW2 ? group : CHANNEL_GROUP_WHITE_CONTROL; String whiteGroup = profile.isRGBW2 ? group : CHANNEL_GROUP_WHITE_CONTROL;
// Create power channel in color mode and brightness channel in white mode // Create power channel in color mode and brightness channel in white mode
addChannel(thing, add, profile.inColor, group, CHANNEL_LIGHT_POWER); addChannel(thing, add, profile.inColor, group, CHANNEL_LIGHT_POWER);

4
bundles/org.openhab.binding.shelly/src/main/java/org/openhab/binding/shelly/internal/provider/ShellyStateDescriptionProvider.java
View File

@ -58,7 +58,6 @@ public class ShellyStateDescriptionProvider extends BaseDynamicStateDescriptionP
this.thingRegistry = thingRegistry; this.thingRegistry = thingRegistry;
} }
@SuppressWarnings("null")
@Override @Override
public @Nullable StateDescription getStateDescription(Channel channel, public @Nullable StateDescription getStateDescription(Channel channel,
@Nullable StateDescription originalStateDescription, @Nullable Locale locale) { @Nullable StateDescription originalStateDescription, @Nullable Locale locale) {
@ -68,6 +67,9 @@ public class ShellyStateDescriptionProvider extends BaseDynamicStateDescriptionP
} }
Thing thing = thingRegistry.get(channel.getUID().getThingUID()); Thing thing = thingRegistry.get(channel.getUID().getThingUID());
if (thing == null) {
return null;
}
ShellyThingInterface handler = (ShellyThingInterface) thing.getHandler(); ShellyThingInterface handler = (ShellyThingInterface) thing.getHandler();
if (handler == null) { if (handler == null) {
return null; return null;

1
bundles/org.openhab.binding.shelly/src/main/resources/OH-INF/i18n/shelly.properties
View File

@ -29,7 +29,6 @@ message.offline.status-error-fwupgrade = Firmware upgrade in progress.
message.offline.status-error-fwcompleted = Firmware upgrade completed, device is restarting. message.offline.status-error-fwcompleted = Firmware upgrade completed, device is restarting.
# General messages # General messages
message.versioncheck.failed = Unable to check firmware version: {0}
message.versioncheck.beta = Device is running a Beta version: {0}/{1} message.versioncheck.beta = Device is running a Beta version: {0}/{1}
message.versioncheck.tooold = WARNING: Firmware might be too old, installed: {0}/{1}, minimal required {2} message.versioncheck.tooold = WARNING: Firmware might be too old, installed: {0}/{1}, minimal required {2}
message.versioncheck.update = INFO: New firmware available: current version: {0}, new version: {1} message.versioncheck.update = INFO: New firmware available: current version: {0}, new version: {1}