andig / pimatic-fritz

Pimatic plugin for Fritz!Box SmartHome and FritzDect!200 Node
GNU General Public License v2.0
8 stars 6 forks source link

Added legacyMode to support Fritz!OS version smaller than V6.20. This… #8

Closed mwittig closed 9 years ago

mwittig commented 9 years ago

… is to address issue https://github.com/andig/pimatic-fritz/issues/7

andig commented 9 years ago

Wouldn't it be possible to fallback to the old method if xmlm structure is empty or not received automatically? I dislike asking the user to think about his FB FW version?

mwittig commented 9 years ago

The automatic fallback is more difficult to implement and needs further investigation, i.e, which failure states can be regarded as "feature not supported". The advantage of the declarative approach is its simplicity and robustness. If there is no upgrade path for @leader21 & other users to Fritz!OS 6.20 I am happy to investigate the automatic fallback solution.