Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

HmIPW-DRI32 zeigt nicht alle Felder in der Gerätekonfiguration an #2736

Closed
Alexander060101 opened this issue May 10, 2024 · 13 comments · Fixed by #2822
Closed

HmIPW-DRI32 zeigt nicht alle Felder in der Gerätekonfiguration an #2736

Alexander060101 opened this issue May 10, 2024 · 13 comments · Fixed by #2822
Labels
🐛 bug-report Something isn't working 🏷️ WebUI This refs the WebUI component 🔅 low priority This issue/ticket has low priority (not urgent)

Comments

@Alexander060101
Copy link

Describe the issue you are experiencing

Wenn man die Einstellungen des Gerätes öffnet, sieht man nur diese beiden Einstellungen pro Kanal (bei Kanalverhalten "Kontakt"):
image

Man muss die Einstellung Kanalverhalten erst zu etwas anderem und dann wieder zurück zu "Kontakt" ändern, um alle Einstellungen zu sehen:
image

Describe the behavior you expected

Es sollten sofort alle Einstellungen angezeigt werden

Steps to reproduce the issue

N/A

What is the version this bug report is based on?

3.75.7.20240420

Which base platform are you running?

ova (Open Virtual Infrastructure, AMD64/x86_64)

Which HomeMatic/homematicIP radio module are you using?

HmIP-RFUSB

Anything in the logs that might be useful for us?

N/A

Additional information

No response

@Alexander060101 Alexander060101 added the 🐛 bug-report Something isn't working label May 10, 2024
@Maik2208
Copy link
Contributor

Kann ich bestätigen.
Das Kanalverhalten steht initial auf "Taster", zeigt aber die Feldauswahl von "Schalter" an.
Betrifft den DRI16 ebenfalls.

@jens-maus jens-maus added 🔅 low priority This issue/ticket has low priority (not urgent) 🏷️ WebUI This refs the WebUI component labels May 10, 2024
Copy link
Contributor

github-actions bot commented Aug 9, 2024

There hasn't been any activity on this issue recently. To keep our backlog manageable we have to clean old issues, as many of them have already been resolved with the latest updates.
Please make sure to update to the latest RaspberryMatic version and check if that solves the issue. Let us know if that works for you by adding a comment 👍
This issue has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.

@github-actions github-actions bot added the 🧊 stale Ticket is in a stale state (about to be closed) label Aug 9, 2024
@Alexander060101
Copy link
Author

Problem besteht auch in Version 3.77.6.20240720 noch.

@github-actions github-actions bot removed the 🧊 stale Ticket is in a stale state (about to be closed) label Aug 11, 2024
Maik2208 added a commit to Maik2208/RaspberryMatic that referenced this issue Aug 11, 2024
@jens-maus jens-maus linked a pull request Aug 13, 2024 that will close this issue
7 tasks
jens-maus added a commit that referenced this issue Aug 13, 2024
…hannel operation mode display of HmIPW-DRI16 and HmIPW-DRI32 (#2822, #2736, @Maik2208)

Co-authored-by: Jens Maus <mail@jens-maus.de>
@jens-maus jens-maus added this to the next release milestone Aug 13, 2024
@Baxxy13
Copy link
Contributor

Baxxy13 commented Nov 26, 2024

Wurde das damals ausführlich getestet?
Und kann mal jemand mit den HmIPW-DRI16 / DRI32 den Patch 0186 rückgängig machen und schauen wie sich das mit der aktuellen RM 3.79.6.20241122 verhält.

Das Problem ist das diese Änderung (auf aktueller RM 3.79.6.20241122) Auswirkungen auf andere MULTI_MODE_INPUT_TRANSMITTER - Kanäle hat, z.B. beim HmIP-MOD-RC8.
Hier werden nun immer alle verfügbaren Parameter dargestellt obwohl diese, abhängig vom eingestellten Kanalverhalten, teilweise ausgeblendet werden sollten.

RM 3.79.6.20241122 mit 0186-WebUI-Fix-HmIPW-DRI16_DRI32-Channel WebUI patch
MOD-RC8_with_0186_Patch

RM 3.79.6.20241122 ohne 0186-WebUI-Fix-HmIPW-DRI16_DRI32-Channel WebUI patch
MOD-RC8_without_0186_Patch

@jens-maus
Copy link
Owner

Vielleicht braucht es dann in der /www/config/easymodes/etc/hmipChannelConfigDialogs.tcl ne unterscheidung nach "if HmIPW-DRI16 oder DRI32, dann wie jetzt, ansonsten wie früher?"

@Baxxy13
Copy link
Contributor

Baxxy13 commented Nov 26, 2024

Möglich.

eQ-3 nutzt selbst:
append html "var elmOperationMode = jQuery(\"#separate_$CHANNEL\_2\"),"

Und ich habe noch nirgends gelesen das es da Probleme mit den HmIPW-DRI16 / DRI32 gibt.
Das muss aber nichts heißen.
Wie geschrieben, idealerweise wird das auf aktueller RM mal ohne den Patch getestet dann können wir ihn ggf. entfernen.

Mangels IP-Wired bin ich da aber raus. 😉

@Baxxy13
Copy link
Contributor

Baxxy13 commented Nov 26, 2024

Hmm, hab jetzt mal auf dem Hauptsystem geguckt.
Das ist eigentlich "raus" weil noch 3.77.7.

Da sind die Eingänge der DRSIx auch MULTI_MODE_INPUT_TRANSMITTER.
Da ist das nur "mit Patch" korrekt.
Wenn das mit 3.79.7 so bleibt dann...

ne unterscheidung nach "if HmIPW-DRI16 oder DRI32, dann wie jetzt, ansonsten wie früher?"

... eher "if HmIP-MOD-RC8, dann wie füher, ansonsten wie jetzt?"

@jens-maus
Copy link
Owner

eQ-3 nutzt selbst: append html "var elmOperationMode = jQuery(\"#separate_$CHANNEL\_2\"),"

Und ich habe noch nirgends gelesen das es da Probleme mit den HmIPW-DRI16 / DRI32 gibt. Das muss aber nichts heißen. > Mangels IP-Wired bin ich da aber raus. 😉

Ich kann das heute abend mal testen und berichten ob es auch ohne den Patch geht. Aber @Alexander060101 hat das ja hier berichtet und @Maik2208 hat es dann via PR korrigiert. Muss also irgendwie was dran sein.

@Maik2208
Copy link
Contributor

Natürlich konnte ich nicht alle Geräte testen, da ich sie schlichtweg nicht besitze. Aber für den DRI16 und DRI32 konnte ich es testen und da war der Fehler vor der Anpassung da und danach definitiv behoben.

@jens-maus
Copy link
Owner

eQ-3 nutzt selbst: append html "var elmOperationMode = jQuery(\"#separate_$CHANNEL\_2\"),"
Und ich habe noch nirgends gelesen das es da Probleme mit den HmIPW-DRI16 / DRI32 gibt. Das muss aber nichts heißen. > Mangels IP-Wired bin ich da aber raus. 😉

Ich kann das heute abend mal testen und berichten ob es auch ohne den Patch geht.

Also hab das jetzt getestet, und es geht OHNE diesen patch hier nicht mit meinem HmIPW-DRI32. D.h. das Problem ist auch prinzipiell noch in der 3.79.6 vorhanden. Die Frage wäre nur wie schränkt man das jetzt ein? Welche Geräte sind noch vom Typ MULTI_MODE_INPUT_TRANSMITTER die prinzipiell davon betroffen wären? Ich tendiere immer noch dazu diesen patch hier nochmal umzuarbeiten und ne explizite Prüfung nach HmIPW-DRI16/32 einzubauen, denn bisher gab es nur diesbzgl. die Klagen und kein anderer MULTI_MODE_INPUT_TRANSMITTER hatte ähnliche Probleme, oder?

@Baxxy13
Copy link
Contributor

Baxxy13 commented Nov 27, 2024

Also meine Eingänge der DRSIx sind auch MULTI_MODE_INPUT_TRANSMITTER und bei denen funktioniert es auch nur mit Patch.
Laut Device-Doku gibt es aktuell 18 Geräte die MULTI_MODE_INPUT_TRANSMITTER nutzen, unmöglich alle zu testen.

Da eure HmIPW-DRI16/32 und meine HmIP-DRSIx den Patch "brauchen" sollten wir eher auf das HmIP-MOD-RC8 prüfen und aktuell nur dafür "die alte Version" append html "var elmOperationMode = jQuery(\"#separate_$CHANNEL\_2\")," nehmen.

Ich bekomme leider kein if else hin, daher hab ich das für einen Test einfach so gemacht.

if {[string equal $dev_descr(TYPE) "HmIP-MOD-RC8"] == 0} {
    append html "console.log('Typ: $dev_descr(TYPE)');"	
	append html "window.setTimeout(function() {"  
      append html "var elmOperationMode = jQuery(\"#separate_$CHANNEL\_1\"),"
      append html "mode = elmOperationMode.val(),"
      append html "chn = elmOperationMode.prop(\"id\").split(\"_\")\[2\];"
      append html "showHideKeyParams(mode, chn);"
    append html "},100);"
}

if {[string equal $dev_descr(TYPE) "HmIP-MOD-RC8"] == 1} {
    append html "console.log('Typ: $dev_descr(TYPE)');"	
	append html "window.setTimeout(function() {"  
      append html "var elmOperationMode = jQuery(\"#separate_$CHANNEL\_2\"),"
      append html "mode = elmOperationMode.val(),"
      append html "chn = elmOperationMode.prop(\"id\").split(\"_\")\[2\];"
      append html "showHideKeyParams(mode, chn);"
    append html "},100);"
}

Nicht schön, aber zum Testen geeignet.
https://github.com/eq-3/occu/blob/742b3aa8c561025a531d8729617f1db091d905eb/WebUI/www/config/easymodes/etc/hmipChannelConfigDialogs.tcl#L1121-L1126

jens-maus added a commit that referenced this issue Nov 27, 2024
address a regression with HmIP-MOD-RC8 requiring the original operating
mode settings (refs #2736).
@jens-maus
Copy link
Owner

So müsste es jetzt gehen – und siehst auch IMHO schöner aus :)

1ad6198

@Baxxy13
Copy link
Contributor

Baxxy13 commented Nov 27, 2024

Sehr schön, kurz und knackig.
Funktioniert auf dem System mit MOD-RC8 und auch auf dem anderen mit DRSI4 korrekt.

Hoffen wir mal das das MOD-RC8 der einzige "Sonderfall" ist und bleibt... es gibt ja leider noch viele andere mit dem MULTI_MODE_INPUT_TRANSMITTER.
Ich halte die Augen offen. 😉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug-report Something isn't working 🏷️ WebUI This refs the WebUI component 🔅 low priority This issue/ticket has low priority (not urgent)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants