Bug in 10_CUL_HM.pm

[Guest]

Originally posted by: <email address deleted>

Hallo fhemusers,
hallo Martin,

ich lese schon eine weile mit, nun kann ich das erste mal auch was
beisteuern.

ich habe vor ca. 4 wochen mit fhem + cul + homematic devices angefangen und
bin echt begeistert, was man mit dieser kombination alles machen kann.

zur zeit versuche ich mich an einer selbstgebaselten alarm-anlage auf basis
von mehreren Fensterkontakten HM-SEC-SC.

dabei ist mir ist aufgefallen, das neuerdings mein Fensterkontakt HM-SEC-SC
keine sabotage oder cover open / close events mehr sendet.

Ich habe mir das Modul 10_CUL_HM.pm mal angeschaut (updatefhem vom
2012-10-24).
nach folgendem Patch funktioniert die sabotage meldung wieder:

pi@raspberrypi /usr/share/fhem/FHEM $ diff 10_CUL_HM.pm 10_CUL_HM.pm.orig
857c857
<               if ($model ne "HM-SEC-WDS"){
---
>               if (!$model eq "HM-SEC-WDS"){

ich denke da wurde nur die negation im if falsch geschrieben.

gruss,

jimmibeam

--
To unsubscribe from this group, send email to
fhem-users+unsubscribe@googlegroups.com

[Guest]

Originally posted by: <email address deleted>

Hallo jimmibeam

>
> Ich habe mir das Modul 10_CUL_HM.pm mal angeschaut (updatefhem vom
> 2012-10-24).
> nach folgendem Patch funktioniert die sabotage meldung wieder:
>
> pi@raspberrypi /usr/share/fhem/FHEM $ diff 10_CUL_HM.pm 10_CUL_HM.pm.orig
> 857c857
> <               if ($model ne "HM-SEC-WDS"){
> ---
> >               if (!$model eq "HM-SEC-WDS"){
>
> ich denke da wurde nur die negation im if falsch geschrieben.
>
> da hast du vollkommen recht, der code ist sowieso quatsch: model wird erst
negiert und dann verglichen - da gehoeren klammern hin, oder eben ohne
negation - dass muss aber spaet gewesen sein;-)

Danke
Martin

--
To unsubscribe from this group, send email to
fhem-users+unsubscribe@googlegroups.com