jens-maus / RaspberryMatic

:house: A feature-rich but lightweight, buildroot-based Linux operating system alternative for your CloudFree CCU3/ELV-Charly 'homematicIP CCU' IoT smarthome central. Running as a pure virtual appliance (ProxmoxVE, Home Assistant, LXC, Docker/OCI, Kubernetes/K8s, etc.) or on a dedicated embedded device (RaspberryPi, Tinkerboard, IntelNUC, etc.)
https://raspberrymatic.de
Apache License 2.0
1.5k stars 184 forks source link

Geräteeinstellung HmIP-STHO werden nicht angezeigt #2793

Closed eli4711 closed 1 week ago

eli4711 commented 1 week ago

Describe the issue you are experiencing

In der WebUI werden die Einstellungen, in den Geräteeinstellungen nicht mehr angezeigt. STHO

Describe the behavior you expected

Parameter werden in der Geräteinstellung angezeigt

Steps to reproduce the issue

  1. Geräteeinstellungen eines HmIP-STHO aufrufen.
  2. ...

What is the version this bug report is based on?

3.77.5.20240626 (generic-x86_64)

Which base platform are you running?

generic-x86_64 (Generic UEFI capable system, AMD64/x86_64)

Which HomeMatic/homematicIP radio module are you using?

RPI-RF-MOD

Anything in the logs that might be useful for us?

Im Log keine Auffälligkeiten.

Additional information

Das Problem ist Vermutlich wieder, auf Änderungen in der OCCU 3.77.X zurückzuführen. Also wieder ein upstream issue.

Baxxy13 commented 1 week ago

Ach ja, kann ich bestätigen.

andrzejgilas commented 1 week ago

Ich bestätige auch (rpi5) 22 33

jens-maus commented 1 week ago

Irgendetwas in der Browser Konsole oder unter /usr/local/tmp auffindbar um rauszubekommen aus welcher neuen richtung das ggf. kommt?

github-actions[bot] commented 1 week ago

@eli4711, the issue you reported cannot be solved within this project or should be better directly solved in the upstream OCCU project RaspberryMatic is just using. However, for being able to reference and track this upstream issue we will keep this ticket open for the time being so that you can also reference it accordingly.

andrzejgilas commented 1 week ago
Return-Path: <CGI script - do not reply>
From:  <CGI script - do not reply>
To: root
Subject:  CGI problem

CGI environment:
REQUEST_METHOD: POST
SCRIPT_NAME: /config/ic_deviceparameters.cgi
HTTP_USER_AGENT: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:127.0) Gecko/20100101 Firefox/127.0
HTTP_HOST: 192.168.1.138
REMOTE_ADDR: 192.168.1.192
cgi.tcl version: 1.10.0
input:
iface=HmIP-RF&address=000EDBE98B950F&redirect_url=GO_BACK
cookie:
errorInfo:
invalid command name "hmip-stho-a"
    while executing
"hmip-stho-a"
    (procedure "getCondTXThresholdUnit" line 4)
    invoked from within
"getCondTXThresholdUnit $devType $chn"
    (procedure "getCondSwitchTransmitter" line 159)
    invoked from within
"getCondSwitchTransmitter $chn ps psDescr"
    (procedure "set_htmlParams" line 13)
    invoked from within
"set_htmlParams $iface $ch_address ch_ps ch_ps_descr CHANNEL_$ch_descr(INDEX) """
    (procedure "put_channel_parameters" line 129)
    invoked from within
"put_channel_parameters"
    (procedure "put_page" line 133)
    invoked from within
"put_page"
jens-maus commented 1 week ago

@andrzejgilas Danke. Das erklärt das Problem. Mit der 3.77.x wurde diese Änderung in der Datei /www/config/easymodes/etc/uiElements.tcl vorgenommen:

@@ -154,7 +158,10 @@ proc getUnit {param} {

 proc getCondTXThresholdUnit {devType chn} {
    switch [string tolower $devType] {
-        hmip-stho  {
+        hmip-stho
+        hmip-stho-a
+        elv-sh-cth
+        {
           if {$chn == "2"} {return "<B0>C"}
           if {$chn == "3"} {return "%"}
         }

Allerdings fehlt hier hinter dem hmip-stho bzw. hmip-stho-a noch ein - damit die eigentlich gewollte Änderung (die wohl blind umgesetzt wurde) auch wirklich funktioniert. Das ganze müsste dann wie folgt aussehen:

  proc getCondTXThresholdUnit {devType chn} {
     switch [string tolower $devType] {
          hmip-stho -
          hmip-stho-a -
          elv-sh-cth {
            if {$chn == "2"} {return "°C"}
            if {$chn == "3"} {return "%"}
          }
          hmip-slo {return "Lux"}
          hmip-scth230 {
            if {$chn == "2" || $chn == "3"} {return "ppm"}
            if {$chn == "5"} {return "°C"}
            if {$chn == "6"} {return "%"}
          }
        default {return ""}
      }
  }

Kannst ja mal probieren die Datei zu editieren und die fehlenden - hinzuzufügen. Würde mich nicht wundern wenn das das Problem bereits behebt.

eli4711 commented 1 week ago

Jepp, das wars. Läuft wieder. 🤗

jens-maus commented 1 week ago

Jepp, das wars. Läuft wieder. 🤗

Wunderbar. eQ3 ist auch schon informiert und wird den Fix dann hoffentlich in der 3.77.x noch umsetzen.

andrzejgilas commented 1 week ago

Jepp, das wars. Läuft wieder. 🤗

Entschuldigung für die dumme Frage Wie soll ich diese Datei bearbeiten?

44

jens-maus commented 1 week ago

Jepp, das wars. Läuft wieder. 🤗

Entschuldigung für die dumme Frage Wie soll ich diese Datei bearbeiten?

Wenn du das nicht weisst, dann solltest du die in der Tat nicht bearbeiten weil du ansonsten mitunter zuviel kaputt machen kannst. Warte also einfach auf den Fix im offiziellen Release. Aber eins ist klar, mit einer GUI editiert/modifiziert man diese Dateien definitiv nicht :)

andrzejgilas commented 1 week ago

@jens-maus Ich habe es schon geschafft :-) Ich bestätige, dass Ihr Fix funktioniert ;-) 44

MichaelN0815 commented 1 week ago

Ich bin immer wieder entsetzt wie bei eq3 gearbeitet wird.

jp112sdl commented 1 week ago

Ich bin immer wieder entsetzt wie bei eq3 gearbeitet wird.

Naja, einerseits ist hat die 3.77.x noch Beta-Status. Es gab schon immer einige Pre-Releases hier im OCCU, bevor die finale Version kam. Die anderen gefundenen und zwischentlich behobenen WebUI-Bugs in der 3.77 waren auch mehr oder weniger Flüchtigkeitsfehler. Aber es entsteht schon der Eindruck, dass man nur noch versucht, sich irgendwie über Wasser zu halten, bis die HCU endlich draußen ist.

Baxxy13 commented 1 week ago

eQ-3 hat das im OCCU 3.77.6 (3.77.X Beta 5) "gefixt". Hoffen wir mal das nun Ruhe einkehrt. 😉