diff --git a/bundles/org.openhab.binding.avmfritz/README.md b/bundles/org.openhab.binding.avmfritz/README.md index 94405eadd..6799e8c20 100644 --- a/bundles/org.openhab.binding.avmfritz/README.md +++ b/bundles/org.openhab.binding.avmfritz/README.md @@ -10,7 +10,25 @@ The binding integrates AVM FRITZ!Boxes with a special focus on the AHA ([AVM Hom FRITZ!Boxes (thing type `fritzbox`) are supported as bridges and they offer channels for call monitoring. To activate the call monitor interface on a FRITZ!Box, you need to dial once `#96*5*` on a connected telephone. +You should hear a short audio signal as confirmation. +This procedure opens TCP/IP port 1012 on your FRITZ!Box. (It can be deactivated again by dialing `#96*4*`.) +You can test if everything is working with the Telnet program from your openHAB server: + +``` +telnet fritz.box 1012 +``` + +If you see an output like this: + +``` +Trying 192.168.178.1... +Connected to fritz.box. +Escape character is '^]'. +``` + +then it successfully connected to the call monitor. +If not, please make sure that the target openHAB system does not block the port on its firewall. Additionally, they serve as a bridge for accessing other AHA devices. For AHA functionality, the router has to run at least on firmware FRITZ!OS 6.00 and it has to support the "Smart Home" service. @@ -61,11 +79,11 @@ Beside four customizable buttons the FRITZ!DECT 440 supports temperature reading The following sensors have been successfully tested using FRITZ!OS 7 for FRITZ!Box 7490 / 7590: -- [SmartHome Tür-/Fensterkontakt (optisch)](https://www.smarthome.de/geraete/eurotronic-smarthome-tuer-fensterkontakt-optisch) - an optical door/window contact -- [SmartHome Tür-/Fensterkontakt (magnetisch)](https://www.smarthome.de/geraete/smarthome-tuer-fensterkontakt-magnetisch-weiss) - a magnetic door/window contact -- [SmartHome Bewegungsmelder](https://www.smarthome.de/geraete/telekom-smarthome-bewegungsmelder-innen) - a motion sensor -- [SmartHome Rauchmelder](https://www.smarthome.de/geraete/smarthome-rauchmelder-weiss) - a smoke detector -- [SmartHome Wandtaster](https://www.smarthome.de/geraete/telekom-smarthome-wandtaster) - a switch with two buttons +- [SmartHome Tür-/Fensterkontakt (optisch)](https://www.smarthome.de/geraete/eurotronic-smarthome-tuer-fensterkontakt-optisch) - an optical door/window contact (thing type `HAN_FUN_CONTACT`) +- [SmartHome Tür-/Fensterkontakt (magnetisch)](https://www.smarthome.de/geraete/smarthome-tuer-fensterkontakt-magnetisch-weiss) - a magnetic door/window contact (thing type `HAN_FUN_CONTACT`) +- [SmartHome Bewegungsmelder](https://www.smarthome.de/geraete/telekom-smarthome-bewegungsmelder-innen) - a motion sensor (thing type `HAN_FUN_CONTACT`) +- [SmartHome Rauchmelder](https://www.smarthome.de/geraete/smarthome-rauchmelder-weiss) - a smoke detector (thing type `HAN_FUN_CONTACT`) +- [SmartHome Wandtaster](https://www.smarthome.de/geraete/telekom-smarthome-wandtaster) - a switch with two buttons (thing type `HAN_FUN_SWITCH`) The use of other Sensors should be possible, if these are compatible with DECT-ULE / HAN-FUN standards. diff --git a/bundles/org.openhab.binding.avmfritz/src/main/java/org/openhab/binding/avmfritz/internal/dto/AVMFritzBaseModel.java b/bundles/org.openhab.binding.avmfritz/src/main/java/org/openhab/binding/avmfritz/internal/dto/AVMFritzBaseModel.java index 0dbe19773..4e8c2be1c 100644 --- a/bundles/org.openhab.binding.avmfritz/src/main/java/org/openhab/binding/avmfritz/internal/dto/AVMFritzBaseModel.java +++ b/bundles/org.openhab.binding.avmfritz/src/main/java/org/openhab/binding/avmfritz/internal/dto/AVMFritzBaseModel.java @@ -206,15 +206,15 @@ public abstract class AVMFritzBaseModel implements BatteryModel { public String toString() { return new StringBuilder().append("[ain=").append(ident).append(",bitmask=").append(bitmask) .append(",isHANFUNDevice=").append(isHANFUNDevice()).append(",isHANFUNButton=").append(isHANFUNButton()) - .append(",isHANFUNAlarmSensor=").append(isHANFUNAlarmSensor()).append(",isButton").append(isButton()) + .append(",isHANFUNAlarmSensor=").append(isHANFUNAlarmSensor()).append(",isButton=").append(isButton()) .append(",isSwitchableOutlet=").append(isSwitchableOutlet()).append(",isTempSensor=") .append(isTempSensor()).append(",isPowermeter=").append(isPowermeter()).append(",isDectRepeater=") .append(isDectRepeater()).append(",isHeatingThermostat=").append(isHeatingThermostat()) .append(",isMicrophone=").append(isMicrophone()).append(",isHANFUNUnit=").append(isHANFUNUnit()) .append(",id=").append(deviceId).append(",manufacturer=").append(deviceManufacturer) .append(",productname=").append(productName).append(",fwversion=").append(firmwareVersion) - .append(",present=").append(present).append(",name=").append(name).append(",battery") - .append(getBattery()).append(",batterylow").append(getBatterylow()).append(getSwitch()) - .append(getPowermeter()).append(getHkr()).toString(); + .append(",present=").append(present).append(",name=").append(name).append(",battery=") + .append(getBattery()).append(",batterylow=").append(getBatterylow()).append(",").append(getSwitch()) + .append(",").append(getPowermeter()).append(",").append(getHkr()).append(",").toString(); } } diff --git a/bundles/org.openhab.binding.avmfritz/src/main/java/org/openhab/binding/avmfritz/internal/dto/DeviceModel.java b/bundles/org.openhab.binding.avmfritz/src/main/java/org/openhab/binding/avmfritz/internal/dto/DeviceModel.java index 3c8ea4774..56c571a9c 100644 --- a/bundles/org.openhab.binding.avmfritz/src/main/java/org/openhab/binding/avmfritz/internal/dto/DeviceModel.java +++ b/bundles/org.openhab.binding.avmfritz/src/main/java/org/openhab/binding/avmfritz/internal/dto/DeviceModel.java @@ -75,8 +75,8 @@ public class DeviceModel extends AVMFritzBaseModel { @Override public String toString() { - return new StringBuilder().append(super.toString()).append(temperature).append(alert).append(getButtons()) - .append(etsiunitinfo).append("]").toString(); + return new StringBuilder().append(super.toString()).append(temperature).append(",").append(alert).append(",") + .append(getButtons()).append(",").append(etsiunitinfo).append("]").toString(); } @XmlAccessorType(XmlAccessType.FIELD)