Extra Tag '<' bei Kanalaktion

Fehler in Firmware und WebUI & Workarounds

Moderator: Co-Administratoren

Antworten
jp112sdl
Beiträge: 12072
Registriert: 20.11.2016, 20:01
Hat sich bedankt: 846 Mal
Danksagung erhalten: 2138 Mal
Kontaktdaten:

Extra Tag '<' bei Kanalaktion

Beitrag von jp112sdl » 20.09.2020, 17:08

Weiß nicht, ob man es als "Bug" bezeichnen soll, aber es gibt hier (noch) keine eQ-3-Praktikanten-Typo Kategorie...

Bereits seit CCU FW 3.49 gibt es ein überflüssiges "<"-Zeichen nach einem Kanalaktion-Input-Feld:
Bildschirmfoto 2020-09-20 um 17.03.40.png
Ist mir heute erst aufgefallen, weil ich nur selten Programme editiere.

VG,
Jérôme ☕️

---
Support for my Homebrew-Devices: Download JP-HB-Devices Addon

alchy
Beiträge: 10744
Registriert: 24.02.2011, 01:34
System: CCU
Hat sich bedankt: 65 Mal
Danksagung erhalten: 671 Mal

Re: Extra Tag '<' bei Kanalaktion

Beitrag von alchy » 20.09.2020, 22:35

Sicher?
Bug_Extra Tag bei Kanalaktion.jpg
Alchy

Blacklist................... almost full
Ignoranz ist die Summe aller Maßnahmen die man ergreift, um bestehende Tatsachen nicht sehen zu müssen.

© Sandra Pulsfort (*1974)

Lies bitte die Logik von WebUI Programmen und die Tipps und Tricks für Anfänger.

Wichtig auch CUxD ersetzt System.exec. Die HM Script Doku (Downloadart Skripte) hilft auch weiter.
Zum Testen von Scripten den >> HomeMatic Script Executor << von Anli benutzen.

jp112sdl
Beiträge: 12072
Registriert: 20.11.2016, 20:01
Hat sich bedankt: 846 Mal
Danksagung erhalten: 2138 Mal
Kontaktdaten:

Re: Extra Tag '<' bei Kanalaktion

Beitrag von jp112sdl » 20.09.2020, 22:40

alchy hat geschrieben:
20.09.2020, 22:35
Sicher?
Ja.

Tritt bei Kanalaktionen auf, die nicht vom Typ
- statusDisplayDevice
- statusDisplayEPaperDevice
- ePaperAcousticDisplay
... hab keine Lust weiter aufzuzählen
sind.

Quasi alles was rausfällt um in den else zu kommen
https://github.com/eq-3/occu/blob/maste ... #L630-L632

Code: Alles auswählen

        ...    } else {
              Write('<input id="prgStringOptionValue'#oSD.ID()#'" type="text" class="SelectBox" size="10" value="'#iV#'" onchange="iseSingleDestination.SetValue('#oSD.ID()#',this.value,\'STRING\');" />< ' );
            }

VG,
Jérôme ☕️

---
Support for my Homebrew-Devices: Download JP-HB-Devices Addon

alchy
Beiträge: 10744
Registriert: 24.02.2011, 01:34
System: CCU
Hat sich bedankt: 65 Mal
Danksagung erhalten: 671 Mal

Re: Extra Tag '<' bei Kanalaktion

Beitrag von alchy » 20.09.2020, 22:50

Upps, hatte nur mal kurz in der Klickibunti nachgeschaut und das Zusatztag tauchte bei mir nicht auf wie du siehst. Daher die Nachfrage.
Im Code ist der Fehler ja schön zu finden.

Alchy

Blacklist................... almost full
Ignoranz ist die Summe aller Maßnahmen die man ergreift, um bestehende Tatsachen nicht sehen zu müssen.

© Sandra Pulsfort (*1974)

Lies bitte die Logik von WebUI Programmen und die Tipps und Tricks für Anfänger.

Wichtig auch CUxD ersetzt System.exec. Die HM Script Doku (Downloadart Skripte) hilft auch weiter.
Zum Testen von Scripten den >> HomeMatic Script Executor << von Anli benutzen.

Benutzeravatar
jmaus
Beiträge: 9805
Registriert: 17.02.2015, 14:45
System: Alternative CCU (auf Basis OCCU)
Wohnort: Dresden
Hat sich bedankt: 457 Mal
Danksagung erhalten: 1848 Mal
Kontaktdaten:

Re: Extra Tag '<' bei Kanalaktion

Beitrag von jmaus » 20.09.2020, 23:43

jp112sdl hat geschrieben:
20.09.2020, 22:40
alchy hat geschrieben:
20.09.2020, 22:35
Sicher?
Ja.

Tritt bei Kanalaktionen auf, die nicht vom Typ
- statusDisplayDevice
- statusDisplayEPaperDevice
- ePaperAcousticDisplay
... hab keine Lust weiter aufzuzählen
sind.

Quasi alles was rausfällt um in den else zu kommen
https://github.com/eq-3/occu/blob/maste ... #L630-L632

Code: Alles auswählen

        ...    } else {
              Write('<input id="prgStringOptionValue'#oSD.ID()#'" type="text" class="SelectBox" size="10" value="'#iV#'" onchange="iseSingleDestination.SetValue('#oSD.ID()#',this.value,\'STRING\');" />< ' );
            }
Gutes Finding! PullRequests werden natürlich wie immer gerne dafür entgegen genommen ;)
RaspberryMatic 3.73.9.20240130 @ ProxmoxVE – ~200 Hm-RF/HmIP-RF/HmIPW Geräte + ioBroker + HomeAssistant – GitHub / Sponsors / PayPal / ☕️

jp112sdl
Beiträge: 12072
Registriert: 20.11.2016, 20:01
Hat sich bedankt: 846 Mal
Danksagung erhalten: 2138 Mal
Kontaktdaten:

Re: Extra Tag '<' bei Kanalaktion

Beitrag von jp112sdl » 21.09.2020, 13:19

Der Aufwand lohnt nicht.
Ich vermag da zumindest keine Zeit investieren.

Mir fallen noch 2 weitere Fehler ein, die schon seit Jahren im Verborgenen schlummern, aber (wegen Nicht-Nutzung?) niemandem auffallen.

So müsste die Register-Adresse hier 0xA4 lauten statt 0x24. Obwohl... ich kann mich dunkel erinnern, mit jemandem hier im Forum mal genau über das Problem gesprochen zu haben.

Und auch das hier ...,0x16,0x,0x1D,0xE0F0,0x03"; ist niemals ein gülter Hex-String.

Man könnte ganz ketzerisch meinen, das QM wird komplett vom Forum übernommen ^^ :lol:

VG,
Jérôme ☕️

---
Support for my Homebrew-Devices: Download JP-HB-Devices Addon

Benutzeravatar
jmaus
Beiträge: 9805
Registriert: 17.02.2015, 14:45
System: Alternative CCU (auf Basis OCCU)
Wohnort: Dresden
Hat sich bedankt: 457 Mal
Danksagung erhalten: 1848 Mal
Kontaktdaten:

Re: Extra Tag '<' bei Kanalaktion

Beitrag von jmaus » 21.09.2020, 15:02

jp112sdl hat geschrieben:
21.09.2020, 13:19
Der Aufwand lohnt nicht.
Ich vermag da zumindest keine Zeit investieren.
Schade, denn ich dachte ich kann dir den Vortritt lassen den 80. WebUI Patch ins RaspberryMatic repository zu submitten :D Aber gut, nun hab ich das erledigt:

https://github.com/jens-maus/RaspberryM ... dfd18ae775
jp112sdl hat geschrieben:
21.09.2020, 13:19
Mir fallen noch 2 weitere Fehler ein, die schon seit Jahren im Verborgenen schlummern, aber (wegen Nicht-Nutzung?) niemandem auffallen.

So müsste die Register-Adresse hier 0xA4 lauten statt 0x24. Obwohl... ich kann mich dunkel erinnern, mit jemandem hier im Forum mal genau über das Problem gesprochen zu haben.
Hmm, das ich da zuwenig drinstecke, kannst du erklären wieso das an dieser stelle 0xA4 sein müsste statte 0x24?
jp112sdl hat geschrieben:
21.09.2020, 13:19
Und auch das hier ...,0x16,0x,0x1D,0xE0F0,0x03"; ist niemals ein gülter Hex-String.
Stimmt, das sieht in der Tat kaputt aus. Wenn du aber auch dafür eine Idee hast wie der Hex-String an dieser Stelle aussehen müsste kann ich das gerne auch für eine der nächsten Versionen zum reparieren via eines WebUI Patches vorsehen und das auch darüber dann an eQ3 melden.
jp112sdl hat geschrieben:
21.09.2020, 13:19
Man könnte ganz ketzerisch meinen, das QM wird komplett vom Forum übernommen ^^ :lol:
Stimmt, das könnte man bei der Menge von WebUI Bugfix Patches inzwischen meinen und vielleicht stimmt das auch sogar zum Teil so. Allerdings hilft es auch im Umkehrschluss nicht weiter nur diese Bugs zu benennen, eQ3 dafür zu bashen weil sie es eigentlich besser wissen/machen sollten. Ich bevorzuge da eher die Herangehensweise das als Kooperation zwischen Community+eQ3 zu sehen und das beste draus zu machen und denen auch gerne Bugfixes direkt vorzuschlagen bzw. zukommen zu lassen - denn die Erfahrung hat gezeigt das es nur so auch effektiv / schnell zu Fehlerbehebungen kommt die uns allen ja zugute kommen. Die Alternativ wäre nämlich halt einfach nur Bugtickets bei eQ3 aufzumachen und auf den Tag zu warten an dem eQ3 das repariert. Das mag einigen vielleicht genug sein bzw. der einzig richtige Weg zu sein, ob es der effektivste Weg ist das Problem zu beseitigen wage ich aber zu bezweifeln :D
RaspberryMatic 3.73.9.20240130 @ ProxmoxVE – ~200 Hm-RF/HmIP-RF/HmIPW Geräte + ioBroker + HomeAssistant – GitHub / Sponsors / PayPal / ☕️

jp112sdl
Beiträge: 12072
Registriert: 20.11.2016, 20:01
Hat sich bedankt: 846 Mal
Danksagung erhalten: 2138 Mal
Kontaktdaten:

Re: Extra Tag '<' bei Kanalaktion

Beitrag von jp112sdl » 22.09.2020, 07:56

jmaus hat geschrieben:
21.09.2020, 15:02
Hmm, das ich da zuwenig drinstecke, kannst du erklären wieso das an dieser stelle 0xA4 sein müsste statte 0x24?
Es handelt sich um den Speicher-Index für den Wert des Expertenparameters LONG_ACT_TYPE.

Die Werte für SHORT_xxx werden in 0x24 und 0x25 gespeichert, die LONG_xxx mit einem Offset von 0x80, also 0xA4 und A5

Vgl. Funk-Gongmodul MP3 mit Speicher HM-OU-CM-PCB (rf_cm.xml):
https://github.com/AskSinPP/asksinpp.de ... #L563-L565
oder Funk-Türgong mit Signalleuchte HM-OU-CF-Pl (rc_cf.xml):
https://github.com/AskSinPP/asksinpp.de ... #L539-L545

Der falsche LONG_ACT_TYPE Index 0x24 sorgt nun dafür, dass der vom User gesetzte Expertenparameter-Wert im Gerät in SHORT_ACT_TYPE 0x24 gespeichert wird und diesen überschreibt. Ein Setzen von LONG_ACT_TYPE ist somit nicht möglich.
jmaus hat geschrieben:
21.09.2020, 15:02
Wenn du aber auch dafür eine Idee hast wie der Hex-String an dieser Stelle aussehen müsste
Nein, was an der Stelle der korrekte String wäre, weiß ich nicht.

VG,
Jérôme ☕️

---
Support for my Homebrew-Devices: Download JP-HB-Devices Addon

Benutzeravatar
jmaus
Beiträge: 9805
Registriert: 17.02.2015, 14:45
System: Alternative CCU (auf Basis OCCU)
Wohnort: Dresden
Hat sich bedankt: 457 Mal
Danksagung erhalten: 1848 Mal
Kontaktdaten:

Re: Extra Tag '<' bei Kanalaktion

Beitrag von jmaus » 22.09.2020, 09:03

jp112sdl hat geschrieben:
22.09.2020, 07:56
jmaus hat geschrieben:
21.09.2020, 15:02
Hmm, das ich da zuwenig drinstecke, kannst du erklären wieso das an dieser stelle 0xA4 sein müsste statte 0x24?
Es handelt sich um den Speicher-Index für den Wert des Expertenparameters LONG_ACT_TYPE.

Die Werte für SHORT_xxx werden in 0x24 und 0x25 gespeichert, die LONG_xxx mit einem Offset von 0x80, also 0xA4 und A5

Vgl. Funk-Gongmodul MP3 mit Speicher HM-OU-CM-PCB (rf_cm.xml):
https://github.com/AskSinPP/asksinpp.de ... #L563-L565
oder Funk-Türgong mit Signalleuchte HM-OU-CF-Pl (rc_cf.xml):
https://github.com/AskSinPP/asksinpp.de ... #L539-L545

Der falsche LONG_ACT_TYPE Index 0x24 sorgt nun dafür, dass der vom User gesetzte Expertenparameter-Wert im Gerät in SHORT_ACT_TYPE 0x24 gespeichert wird und diesen überschreibt. Ein Setzen von LONG_ACT_TYPE ist somit nicht möglich.
Alles klar, das ergibt Sinn. Magst du dafür ein seperates Ticket im RaspberryMatic GitHub aufmachen damit ich es darüber fixen und eQ3 sich das dann für zukünftige Versionen besser abschauen kann?
jp112sdl hat geschrieben:
22.09.2020, 07:56
jmaus hat geschrieben:
21.09.2020, 15:02
Wenn du aber auch dafür eine Idee hast wie der Hex-String an dieser Stelle aussehen müsste
Nein, was an der Stelle der korrekte String wäre, weiß ich nicht.
Ok, dann kann ich das nur so an eQ3 melden und schauen was die daraus machen. Ggf. wäre dafür ein separates Ticket im GitHub auch gut man man das eQ3 besser präsentieren kann. Magst du das machen?
RaspberryMatic 3.73.9.20240130 @ ProxmoxVE – ~200 Hm-RF/HmIP-RF/HmIPW Geräte + ioBroker + HomeAssistant – GitHub / Sponsors / PayPal / ☕️

Antworten

Zurück zu „HomeMatic - bekannte Bugs“