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

Nightly - 3.69.4.20230323 HmIP-WKP keine PIN-Eingabe möglich #2269

Closed
Quarkmax opened this issue Mar 24, 2023 · 12 comments
Closed

Nightly - 3.69.4.20230323 HmIP-WKP keine PIN-Eingabe möglich #2269

Quarkmax opened this issue Mar 24, 2023 · 12 comments
Labels
⚓ upstream issue This is a bug/issue for/in upstream software (OCCU, etc.) 🐛 bug-report Something isn't working 🏷️ HmIPServer This refs the HmIPServer component 🧠 unstable-snapshot This ticket references the use of an unsupported unstable snapshot being used.

Comments

@Quarkmax
Copy link

Quarkmax commented Mar 24, 2023

Describe the issue you are experiencing

Nightly - 3.69.4.20230323 auf PROXMOX
In den Settings für das Keypad HmIP-WKP kann keine PIN, für jeden Benutzer, hinterlegt werden. Nach dem Öffnen der Einstellungen ist statt des leeren PIN-Feldes dort der Eintrag name=NUMERIC_CODE vorgegeben. Mit dem Löschen und Eintragen der PIN mit anschließenden Speichern, wird die PIN nicht übernommen. Es erscheint wieder der Eintrag name=NUMERIC_CODE. Zurück auf die 3.67.10.20230225 ist das Verhalten so, wie es sein soll, die PIN kann gesetzt werden.

Describe the behavior you expected

Die PIN sollte angenommen werden

Steps to reproduce the issue

Zurücksetzen auf 3.67.10.20230225 und erneute Update auf Nightly ergab den gleichen Fehler. Wieder zurück auf 3.67.10.20230225 alles normal.

What is the version this bug report is based on?

3.69.4.20230323

Which base platform are you running?

ova (Open Virtual Infrastructure)

Which HomeMatic/homematicIP radio module are you using?

RPI-RF-MOD

Anything in the logs that might be useful for us?

.

Additional information

No response

@Quarkmax Quarkmax added the 🐛 bug-report Something isn't working label Mar 24, 2023
@jens-maus jens-maus added 🏷️ HmIPServer This refs the HmIPServer component ⚓ upstream issue This is a bug/issue for/in upstream software (OCCU, etc.) 🧠 unstable-snapshot This ticket references the use of an unsupported unstable snapshot being used. labels Mar 29, 2023
@jens-maus jens-maus added this to the next release milestone Mar 29, 2023
@jens-maus
Copy link
Owner

@Quarkmax Nun da das aktuellste nightly snapshot auf OCCU 3.69.6 basiert in dem das Problem beseitigt sein sollte würde ich drum bitten dieser version mal bitte einen Test zu unterziehen und hier zurückzumelden ob das Problem nun in der Tat beseitigt wurde.

@Quarkmax
Copy link
Author

@jens-maus
Ich habe die aktuelle OCCU 3.69.6 installiert. Im sonst leeren PIN-Feld steht jetzt noch ein einzelnes " Anführungszeichen am Anfang. Die PIN kann eingegeben werden und bleibt nach dem Speichern erhalten, ABER - sie funktioniert nicht. Das Keypad quittiert die Eingabe mit einem rotem Blinken und es erfolgt keine Auslösung. Die Kanäle sind "Direktverknüpft". Die vor dem Update hinterlegten PIN's funktionieren ohne Fehler weiter. Neu hinzugefügte PIN's nicht. VM-Backup zurückgespielen und erneutes Update ergab das gleiche Verhalten.

@jens-maus
Copy link
Owner

@Quarkmax Kannst du mal bitte ein Screencast (animated gif) davon hier zeigen damit man sich ein Eindruck davon machen kann auch wenn man selbst kein HmIP-WKP hat?

@Quarkmax
Copy link
Author

Quarkmax commented Mar 30, 2023

@jens-maus
Danke für Deine Nachfrage. Es ist gar kein Anführungszeichen, es sind zwei Apostrophe. Sorry.
Und hier der Screencast. Nach dem herauslöschen der PIN sind die beiden Aphostrophe wieder zurück
HmIP-WKP

@jens-maus
Copy link
Owner

@Quarkmax Bei dem Screencast sieht man das doch nur bei leerem PIN Feld die beiden '' im Feld auftauchen, richtig? Sobald man einen validen Code eingibt und die beiden '' vorher weglöscht geht der PIN. Geht er dann auch am Gerät? Und was ist wenn man den PIN code weglöscht? Ist das überhaupt supportet das es da keine PIN geben soll? Und wenn ja, geht es dann trotzdem und nur die Anzeige mit den beiden '' ist ungünstig oder gibt es einen funktionalen Nachteil beim Gerät dann?

@Quarkmax
Copy link
Author

@jens-maus
Ja, die beiden '' sind nur im leeren Feld. Und ja, pro Benutzer einen PIN. Wenn nur vier Benutzer, vergebe ich nur vier PIN. Außerdem müssen/können die Nutzer noch ein Zeitfenster zugeordnet bekommen, also berechtigt sind. Ich habe nicht die maximal mögliche Benutzer/PIN-Anzahl ausgenutzt und es funktioniert. Ich habe jetzt nochmals ein Rollback durch und Update ausgeführt, die Berechtigungen und PIN gesetzt, die Direktverknüpfungen danach angelegt (wie sonst auch für die Tests). Nutzer 3 und 4 haben sofort funktioniert, Nutzer 7 nicht. Dann beim Nutzer 7 nochmal eine andere PIN neu gesetzt und jetzt ist die Funktion für alle angelegten Nutzer gegeben. Dachte erst, es liegt daran, weil zwei inaktive Nutzer dazwischen liegen, aber das kann es ja nicht sein. Lange Rede gar kein Sinn... es funktioniert jetzt. Ich lass die Nightly drauf und beobachte weiter.

Vielen Dank

@jens-maus
Copy link
Owner

[...] Lange Rede gar kein Sinn... es funktioniert jetzt. Ich lass die Nightly drauf und beobachte weiter.

D.h. also, das nur die beiden '' "unschön" sind wenn das Feld leer ist, die Funktion aber gegeben ist das man die PIN leer lassen kann? D.h. man muss nur sicherstellen die beiden '' vorher zu löschen? Richtig?

@Quarkmax
Copy link
Author

Korrekt

@jens-maus
Copy link
Owner

@Quarkmax Kannst du bitte mal probieren in der Datei /www/config/easymodes/etc/uiElements.tcl auf der Zentrale in Zeile 188 die folgende Änderung umzusetzen:

- if {[string equal $value ""] == 1} {set value ''}
+ if {[string equal $value ""] == 1} {set value ""}

um zu schauen das das '' dann ggf. Weg ist wenn das Feld leer gelassen wird?

@Quarkmax
Copy link
Author

@jens-maus
Die '' sind weg! Kann die Funktionalität leider erst morgen testen. Bis dahin...

jens-maus added a commit that referenced this issue Mar 30, 2023
incorrect quoting ending up in empty PINs for a HmIP-WKP to show
superfluous double apostrophes. This refs #2269.
@jens-maus
Copy link
Owner

So, hab nun noch einen eignen WebUI patch hinzugefügt der das '' Problem in der Anzeige beseitigen sollte. Bitte mit dem nächsten kommenden nightly snapshot testen ob nun alles wie erwartet funktioniert und gut aussieht.

@Quarkmax
Copy link
Author

@jens-maus
Sieht jetzt aus wie vorher und damit gut und alles funktioniert wie erwartet.

Danke

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⚓ upstream issue This is a bug/issue for/in upstream software (OCCU, etc.) 🐛 bug-report Something isn't working 🏷️ HmIPServer This refs the HmIPServer component 🧠 unstable-snapshot This ticket references the use of an unsupported unstable snapshot being used.
Projects
None yet
Development

No branches or pull requests

2 participants