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

[Tankerkoenig] Fixes #8876 (#8940)

Browse Source 
* [Tankerkoenig] Update StationHandler

Incooperated check for 'null' iot to prevent NullPointerExceptions.
* [Tankerkoenig] Updated WebserviceHandler

Fixed false parsing of OpeningTimes.
* Applied a spotless check   Signed-off-by: Jürgen Baginski <opus42@gmx.de>

Signed-off-by: Jürgen Baginski <opus42@gmx.de>
This commit is contained in:
Jürgen Baginski 2020-11-03 07:48:05 +01:00 committed by Kai Kreuzer
parent 4d3dd96a67
commit be0b3cf5b9
2 changed files with 40 additions and 22 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

52
bundles/org.openhab.binding.tankerkoenig/src/main/java/org/openhab/binding/tankerkoenig/internal/handler/StationHandler.java
View File

@ -128,25 +128,43 @@ public class StationHandler extends BaseThingHandler {
*/
public void updateData(LittleStation station) {
logger.debug("Update Tankerkoenig data '{}'", getThing().getUID());
if (IS_NUMERIC_PATTERN.matcher(station.getDiesel()).matches()) {
DecimalType diesel = new DecimalType(station.getDiesel());
updateState(CHANNEL_DIESEL, diesel);
if (station.isOpen() == true) {
logger.debug("Checked Station is open! '{}'", getThing().getUID());
updateState(CHANNEL_STATION_OPEN, OpenClosedType.OPEN);
if (station.getDiesel() != null) {
if (IS_NUMERIC_PATTERN.matcher(station.getDiesel()).matches()) {
DecimalType diesel = new DecimalType(station.getDiesel());
updateState(CHANNEL_DIESEL, diesel);
} else {
updateState(CHANNEL_DIESEL, UnDefType.UNDEF);
}
} else {
updateState(CHANNEL_DIESEL, UnDefType.UNDEF);
}
if (station.getE10() != null) {
if (IS_NUMERIC_PATTERN.matcher(station.getE10()).matches()) {
DecimalType e10 = new DecimalType(station.getE10());
updateState(CHANNEL_E10, e10);
} else {
updateState(CHANNEL_E10, UnDefType.UNDEF);
}
} else {
updateState(CHANNEL_E10, UnDefType.UNDEF);
}
if (station.getE10() != null) {
if (IS_NUMERIC_PATTERN.matcher(station.getE5()).matches()) {
DecimalType e5 = new DecimalType(station.getE5());
updateState(CHANNEL_E5, e5);
} else {
updateState(CHANNEL_E5, UnDefType.UNDEF);
}
} else {
updateState(CHANNEL_E5, UnDefType.UNDEF);
}
} else {
updateState(CHANNEL_DIESEL, UnDefType.UNDEF);
logger.debug("Checked Station is closed!");
updateState(CHANNEL_STATION_OPEN, OpenClosedType.CLOSED);
}
if (IS_NUMERIC_PATTERN.matcher(station.getE10()).matches()) {
DecimalType e10 = new DecimalType(station.getE10());
updateState(CHANNEL_E10, e10);
} else {
updateState(CHANNEL_E10, UnDefType.UNDEF);
}
if (IS_NUMERIC_PATTERN.matcher(station.getE5()).matches()) {
DecimalType e5 = new DecimalType(station.getE5());
updateState(CHANNEL_E5, e5);
} else {
updateState(CHANNEL_E5, UnDefType.UNDEF);
}
updateState(CHANNEL_STATION_OPEN, (station.isOpen() ? OpenClosedType.OPEN : OpenClosedType.CLOSED));
updateStatus(ThingStatus.ONLINE);
}

10
bundles/org.openhab.binding.tankerkoenig/src/main/java/org/openhab/binding/tankerkoenig/internal/handler/WebserviceHandler.java
View File

@ -293,11 +293,11 @@ public class WebserviceHandler extends BaseBridgeHandler {
}
break;
}
if (foundIt) {
start = open;
ende = close;
break;
}
}
if (foundIt) {
start = open;
ende = close;
break;
}
i = i + 1;
} while (i < o.length);