Hinweis: fheminfo liefert unvollständige CUL_HM Modellliste

[@tango]

Hi,

Nur ein Hinweis. Ein aktueller Aufruf von

Code Auswählen Erweitern

fheminfo

liefert bei mir eine unvolständige CUL_HM Modellliste.

Die folgende Modelle sind bei mir vorhanden wurden aber nicht gelistet:

• HM-SEC-SCO
• HM-LC-SW1-FM
• HM-LC-BL1PBU-FM
• HM-SEC-SD-2

Update Stand aktuell mit CUL_HM und HMinfo Patches aus

https://forum.fhem.de/index.php/topic,123198.msg1177351.html#msg1177351

[frank]

hallo beta-user,

Falls jemand Ideen zu https://forum.fhem.de/index.php/topic,123298.0.html hat: her damit.

es werden nur die devices gelistet, bei denen es sich um "multi-channel-devices" handelt.

zeile 139 ändern zu:

Code Auswählen Erweitern

                (defined($defs{$key}{'chanNo'}) && defined($defs{$key}{device})) ||

jetzt noch virtuelle devices (CCU-FHEM, ACTIONDETECTOR) entfernen:
zb zeile einfügen unter 139:

Code Auswählen Erweitern

                ($defs{$key}{'TYPE'} eq 'CUL_HM' && $model =~ m/^(CCU-FHEM|ACTIONDETECTOR)$/) ||

bei mir ist jetzt alles (nur reale geräte) vorhanden.

[frank]

richtig schön wäre natürlich noch eine sortierung der modelliste je modul. 
zur zeit wird die reihenfolge bei jedem aufruf von fheminfo kräftig "durchgemischt".

edit: ausserdem sehe ich gerade, dass auch alle ignored devices auftauchen.

[Beta-User]

...mal sehen, ob betateilchen doch noch Anpassungen an fheminfo machen will....

Anbei mal ein Vorschlag, der funktionieren könnte. Anmerkungen:
- Ob eine CCU-FHEM vorhanden ist, finde ich - im Unterschied zum zwangsweisen AD - als Info eigentlich auch für die Statistik nicht verkehrt;
- ignored=1 sollten raus sein (ungetestet);
- html-Ausgabe erfolgt sortiert;
- bzgl. defined($defs{$key}{device}) war/bin ich unsicher, ob das die bessere Wahl ist oder die jetzt als Vorschlag eingebaute Alternative, den '01'-Wert nicht auszusortieren.

Ist nur grob angetestet.

[frank]

Ist nur grob angetestet.

sortierung ok.
multi-channel-devices werden doppelt gezählt.
ignored one-channel-devices werden noch gelistet.

edit: alle ignored devices werden gelistet.

[Beta-User]

multi-channel-devices werden doppelt gezählt.

...dann doch deine Variante. Zumindest in meiner Installation scheint es auch keine nicht-CUL_HM-Devices zu geben, die das Internal kennen. Sorry für den Aufwand.

edit: alle ignored devices werden gelistet.

Typo berichtigt, wenn das Attribut "ignore" heißt, sollte man nicht auf "ignored" vergleichen, oder ...?

[frank]

so sieht es gut aus, plus vccu.
also wieder ein modul für die exclude-from-update-list, da der original code ja scheinbar schon in stein gemeisselt ist. 

[Beta-User]

...jetzt wirf' doch die Flinte nicht gleich ins Korn... 

betateilchen war von meiner Annahme ausgegangen, dass das ganze Problem an der "ref"-Abfrage hinge. Die war leider falsch. Nicht gut, aber jetzt wissen wir es ja - dank deiner Analyse ( ) - besser.

Die anderen Punkte sind Vorschläge, kann ja sein, dass schon mal jemand über den einen oder anderen Punkt nachgedacht hatte, aber vielleicht auch nicht? Ich weiß es nicht und werde nach dem Testen (eilt ja nicht) dann nochmal einen Anlauf nehmen und nachfragen, ob man nicht die sowieso schon vorhandene CUL_HM-Sonderlocke dann eben dem aktuellen Stand der Dinge anpassen kann. Ich sehe jedenfalls nicht, dass der implizite Vorwurf gerechtfertigt wäre, dass sich jemand nicht an Absprachen gehalten hätte...

(Mir dämmert dunkel, dass das Thema eigentlich schon uralt ist, bin aber grade zu faul zum suchen).