eigenes Reading mit String befüllen... Syntaxfrage

[MichaelO]

Moin,

irgendwie stelle ich mich mit setReading etwas blöd an, oder verwende den falschen Befehl. Vielleicht kann mir wer weiterhelfen.

Ich habe ein DOIF, das eine Lampe steuert. Wenn das DOIF die Lampe anschaltet, soll im DOIF ein Reading "lastActivationType" auf "auto" gesetzt werden. Schaltet man die Lampe selbst ein oder aus, dann soll das Reading "manuell" enthalten. Das DOIF wertet das Reading aus, um selbst entsprechend anders zu reagieren, je nachdem, ob die Lampe eben manuell oder automatisch geschaltet wurde.

Ich setze den Wert des Reading innerhalb des DOIF mit

Code Auswählen Erweitern

setReading $SELF lastActivationType manuell

Und bekomme im Log stets eine Fehlermeldung. Anführungszeichen um das "manuell" haben nichts geholfen, ich komme hier nicht weiter. Wie kann ich denn das Reading korrekt befüllen?

Danke
Michael

[igami]

Und bekomme im Log stets eine Fehlermeldung.

Hast du die Fehlermeldung denn verstanden?

Hier mal der Link zur commandref

[MichaelO]

Hast du die Fehlermeldung denn verstanden?

Nein, sonst hätte ich das Problem ja selbst lösen oder meine Frage genauer eingrenzen können. Die commandref habe ich auch bereits angeschaut, ich komme jedoch mit der Beschreibung des setReading-Befehls dahingehend nicht zurecht, dass ich nicht erkennen kann, was bei mir falsch sein könnte.

Die Fehlermeldung im Logfile lautet

Code Auswählen Erweitern

2017.07.26 08:48:40 2: di_WZ_DekobeleuchtungAutomatik:        setReading di_WZ_DekobeleuchtungAutomatik lastActivationType auto: Usage: setreadingdesc <devspec> <readingspec> [noCheck] <key>=[<value>|?]
where <devspec> is a single device name, a list separated by comma (,) or a regexp. See the devspec section in the commandref.html for details.
<readingspec> can be a single reading name, a list separated by comma (,) or a regexp.

Da wird mir doch auch nur erklärt, wie denn der Befehl aufgebaut ist. Was genau ich nun falsch gemacht hab, steht da nicht. Oder hab ich was übersehen?

Gruß
Michael

[Otto123]

Hallo Michael,

an dem setreading Befehl selbst ist nichts falsch, ich nehme an der Syntax innerhalb Deines DOIF stimmt nicht. Er macht doch offenbar etwas anderes als setreading
Führe doch den Befehl einfach mal in der Kommandozeile von FHEM aus.
Gruß Otto

[CoolTux]

setReading != setreading

Commandref hätte geholfen

[igami]

setReading != setreading

Commandref hätte geholfen

Aber != ist doch kein Textvergleich, dann kommt nur sowas wie PERL WARNING: Argument "setReading" isn't numeric in numeric
Wenn schon dann "setReading" ne "setreading"
Und FHEM cmds sind nicht case sensitive, kann man mit "DEFINE test dummy" testen, sonst müsste man ja auch immer wissen wie das Modul geschrieben wurde, Dummy, dummy oder DUMMY.

Die Fehlermeldung ist aber nicht wirklich aussagekräftig.
Poste mal bitte ein List von dem device NACHDEM der Fehler auftritt.

[CoolTux]

tatsächlich. man kann den Befehl auch Groß klein schreiben.

[Manul]

Ist jetzt nur geraten, aber wenn ich das

Code Auswählen Erweitern

Usage: setreadingdesc <devspec> <readingspec> [noCheck] <key>=[<value>|?]

hiermit vergleiche

Code Auswählen Erweitern

setReading $SELF lastActivationType manuell

würde ich für denkbar halten, daß da einfach ein "=" fehlt.

Edit: Falsch geraten, wie die commandref zeigt.

[nils_]

Usage: setreadingdesc <devspec> <readingspec> [noCheck] <key>=[<value>|?]
where <devspec> is a single device name, a list separated by comma (,) or a regexp. See the devspec section in the commandref.html for details.
<readingspec> can be a single reading name, a list separated by comma (,) or a regexp.

fehlermeldung kommt denke ich hierher: https://svn.fhem.de/trac/browser/trunk/fhem/FHEM/Unit.pm#L4510

wie das mit DOIF bzw. dem Logeintrag zusammenhängt, kann ich leider nicht beantworten 

[MichaelO]

So, dank Eurer Hinweise hab ich den Befehl ersteinmal in die Kommandozeile getippt, und bekam dieselbe Fehlermeldung - auch, wenn ich statt "auto" dort eine Zahl eingebe. Mit einem "=" war es auch nix.

Aaaaber... es scheint tatsächlich die verdammte Schreibweise gewesen zu sein. Wenn ich setreading statt setReading schreibe, kommt keine Fehlermeldung und das Reading nimmt den gewünschten Wert an. Sowas doofes aber auch... kommt davon, wenn man zu viele unterschiedliche Systeme beackert - ich schreibe beim Programmieren bei Namen von Prozeduren/Variablen immer das erste Wort klein und dann jedes neue, zusammengesetzte groß.

Allerdings ist die Fehlermeldung (wenn es das wirklich war) nicht besonders toll...

Werde das jetzt mal korrigieren und sehen, was die Automatik macht, sobald das DOIF scharf geht.

Gruß
Michael

[Otto123]

Hallo Michael,

kann nicht sein, ich weiß nicht was Du da auf welcher Kommandozeile eingetippt hast.
Ich habe einen dummy Test
Ob ich

Code Auswählen Erweitern

setReading Test lastActivationType auto
oder
setreading Test lastActivationType auto eintippe ist völlig egal. Es funktioniert in beiden Fällen.

Wie er aus setreading -> setreadingdesc macht ist mir noch unklar 

Es kann sein, weil Du es im DOIF aufrufst, oder das es an $SELF liegt. Aber $SELF wird eigentlich korrekt aufgelöst wie die Fehlermeldung zeigt.

Gruß Otto

[MichaelO]

Das ist ja echt komisch, ich habe es gerade nochmal getestet...

Kommandozeile

Code Auswählen Erweitern

setreading di_WZ_DekobeleuchtungAutomatik lastActivationType auto
Keine Fehlermeldung, das Reading wird korrekt gesetzt

Kommandozeile

Code Auswählen Erweitern

setReading di_WZ_DekobeleuchtungAutomatik lastActivationType auto
Fehlermeldung wie gehabt

Code Auswählen Erweitern

Usage: setreadingdesc <devspec> <readingspec> [noCheck] <key>=[<value>|?]
where <devspec> is a single device name, a list separated by comma (,) or a regexp. See the devspec section in the commandref.html for details.
<readingspec> can be a single reading name, a list separated by comma (,) or a regexp.

Das soll wer verstehen

[Otto123]

hast Du eine besondere FHEM Version? Was sagt version?

Gruß Otto

[MichaelO]

Eigentlich mache ich recht regelmäßig Updates...

Latest Revision: 14773

[Otto123]

Sehr komisch.

Ich habe gerade nochmal auch ein DOIF probiert, es ist völlig egal wie ich setreading schreibe, ob groß klein komplett groß - geht alles.

Was sagt denn

Code Auswählen Erweitern

list TYPE=cmdalias

Gruß Otto