Skip to content

Commit d0ea14f

Browse files
authored
[gpstracker] Fix handling ambigious region names (openhab#17805)
* Fix regionname * Fix SAT * Improve log line Signed-off-by: Leo Siepel <[email protected]>
1 parent 5a27b0e commit d0ea14f

File tree

1 file changed

+9
-3
lines changed

1 file changed

+9
-3
lines changed

bundles/org.openhab.binding.gpstracker/src/main/java/org/openhab/binding/gpstracker/internal/profile/GPSTrackerTriggerSwitchProfile.java

+9-3
Original file line numberDiff line numberDiff line change
@@ -68,10 +68,16 @@ public void onStateUpdateFromItem(State state) {
6868

6969
@Override
7070
public void onTriggerFromHandler(String payload) {
71-
if (payload.startsWith(regionName)) {
72-
OnOffType state = OnOffType.from(payload.endsWith("enter"));
71+
if (!payload.contains("/")) {
72+
logger.debug("Transition trigger with mallformed payload: '{}', missing '\''", payload);
73+
return;
74+
}
75+
String[] splitted = payload.split("/");
76+
if (splitted[0].equals(regionName)) {
77+
OnOffType state = OnOffType.from("enter".equals(splitted[1]));
7378
callback.sendCommand(state);
74-
logger.debug("Transition trigger {} handled for region {} by profile: {}", payload, regionName, state);
79+
logger.debug("Transition trigger with payload: '{}' handled for region {} by profile: {}", payload,
80+
regionName, state);
7581
}
7682
}
7783
}

0 commit comments

Comments
 (0)