An die Wetter Modulauthoren API-Dokumentation online

Begonnen von CoolTux, 12 Januar 2019, 17:58:50

Vorheriges Thema - Nächstes Thema

CoolTux

Ich denke nicht das es etwas für die Commandref ist. Soll doch einzig und alleine für den ersten Start mit der demo.cfg sein
Du musst nicht wissen wie es geht! Du musst nur wissen wo es steht, wie es geht.
Support me to buy new test hardware for development: https://www.paypal.com/paypalme/MOldenburg
My FHEM Git: https://git.cooltux.net/FHEM/
Das TuxNet Wiki:
https://www.cooltux.net

nils_

dann warten wir mal auf die erste meldung, wenn einer die def-zeile auf der demo.cfg in seine umgebung kopiert hat  ;D



die erwähnung das es nun einen solchen apikey gibt und was er tut gehört mMn trotzdem in die commandref.
vielleicht will ich mir ja das modul nur mal angucken und sehen was auf der seite gezeigt wird (mit den testdaten). und dafür brauche ich dann nicht gleich nen key woanders beantragen :)
viele Wege in FHEM es gibt!

betateilchen

-----------------------
Formuliere die Aufgabe möglichst einfach und
setze die Lösung richtig um - dann wird es auch funktionieren.
-----------------------
Lesen gefährdet die Unwissenheit!

nils_

viele Wege in FHEM es gibt!

Dr. Boris Neubert

Zitat von: CoolTux am 28 Februar 2019, 09:24:24
Dafür müsste das Weather Modul angepasst werden. In meinen Augen ein mini Patch. Eventuell könntest Du es erst einmal damit testen und wenn es so ausreichend ist reiche ich den Patch bei Boris ein.


diff --git a/59_Weather.pm b/59_Weather.pm
index aa004bc..877e690 100755
--- a/59_Weather.pm
+++ b/59_Weather.pm
@@ -601,6 +601,10 @@ sub Weather_Notify($$) {
     Log3 $hash, 5,
"Weather $name: FHEM initialization or rereadcfg triggered update, delay $delay seconds.";
     Weather_RearmTimer( $hash, gettimeofday() + $delay );
+   
+    ### quick run GetUpdate then Demo
+    Weather_GetUpdate( $hash )
+    if ( lc($hash->{APIKEY}) eq 'demo' );

     return undef;
}
lines 1-15/15 (END)


Ist $hash->{APIKEY} immer vorhanden? Sonst gibt es Warnings und damit Nutzeranfragen.

Vielleicht besser

if( defined($hash->{APIKEY}) &&  lc($hash->{APIKEY}) eq 'demo' );

Wenn es damit klappt: kannst Du es bitte einchecken für mich?
Globaler Moderator, Developer, aktives Mitglied des FHEM e.V. (Marketing, Verwaltung)
Bitte keine unaufgeforderten privaten Nachrichten!

CoolTux

Ohne apikey ist eine Definition des Weather Devices nicht möglich. Ich frage aber gerne ab ob hash->{APIKEY} deklariert ist.
Schecke ich heute Abend oder morgen früh ein.


Grüße
Du musst nicht wissen wie es geht! Du musst nur wissen wo es steht, wie es geht.
Support me to buy new test hardware for development: https://www.paypal.com/paypalme/MOldenburg
My FHEM Git: https://git.cooltux.net/FHEM/
Das TuxNet Wiki:
https://www.cooltux.net

Dr. Boris Neubert

Zitat von: CoolTux am 04 März 2019, 18:02:19
Ohne apikey ist eine Definition des Weather Devices nicht möglich. Ich frage aber gerne ab ob hash->{APIKEY} deklariert ist.

Hmm, sowohl gemäß commandref als auch tatsächlichem Code (Weather_Define) ist apikey optional. Dass ohne keines der derzeit implementierten APIs geht, ist ein anderes Ding.
Globaler Moderator, Developer, aktives Mitglied des FHEM e.V. (Marketing, Verwaltung)
Bitte keine unaufgeforderten privaten Nachrichten!

CoolTux

Danke für die Richtigstellung, ich dachte es gibt eine Abbruch wenn apikey nicht mit übergeben wird.
Dann passt ja die vorherige Abfrage. Bau ich nachher ein.
Du musst nicht wissen wie es geht! Du musst nur wissen wo es steht, wie es geht.
Support me to buy new test hardware for development: https://www.paypal.com/paypalme/MOldenburg
My FHEM Git: https://git.cooltux.net/FHEM/
Das TuxNet Wiki:
https://www.cooltux.net

CoolTux

Du musst nicht wissen wie es geht! Du musst nur wissen wo es steht, wie es geht.
Support me to buy new test hardware for development: https://www.paypal.com/paypalme/MOldenburg
My FHEM Git: https://git.cooltux.net/FHEM/
Das TuxNet Wiki:
https://www.cooltux.net

Christoph Morrison

(Macht doch mal eine Markdown-Version davon ins Git, dann kann man auch pull requests schicken)

CoolTux

Zitat von: Christoph Morrison am 05 März 2019, 15:17:45
(Macht doch mal eine Markdown-Version davon ins Git, dann kann man auch pull requests schicken)

Siehst Du meine Fragezeichen auf dem Kopf  ;D
https://github.com/fhem/Weather
Du musst nicht wissen wie es geht! Du musst nur wissen wo es steht, wie es geht.
Support me to buy new test hardware for development: https://www.paypal.com/paypalme/MOldenburg
My FHEM Git: https://git.cooltux.net/FHEM/
Das TuxNet Wiki:
https://www.cooltux.net

nils_

viele Wege in FHEM es gibt!

Christoph Morrison

Ich mach euch einfach eine und schicke sie euch über einen pull request.


CoolTux

Ist angekommen. Schaue ich mir heute Abend in Ruhe an.

Vielen lieben Dank schon mal.
Du musst nicht wissen wie es geht! Du musst nur wissen wo es steht, wie es geht.
Support me to buy new test hardware for development: https://www.paypal.com/paypalme/MOldenburg
My FHEM Git: https://git.cooltux.net/FHEM/
Das TuxNet Wiki:
https://www.cooltux.net