wieder Umlautprobleme beim Anlegen von Direktverknüpfungen gesichtet

[Baxxy13]

Describe the issue you are experiencing

Beim Anlegen von Direktverknüpfungen werden wieder "krumme Zeichen" statt korrekter Umlaute in die Beschreibung geschrieben.

Describe the behavior you expected

Korrekte Anzeige der Umlaute

Steps to reproduce the issue

1. DV anlegen, z.B. virt.Taste <--> Schaltakter
2. Text in der Beschreibung der DV betrachten

...

What is the version this bug report is based on?

3.67.10.20230110-82325d

Which base platform are you running?

rpi3 (RaspberryPi3)

Which HomeMatic/homematicIP radio module are you using?

RPI-RF-MOD

Anything in the logs that might be useful for us?

- keine auffälligen Logs

Additional information

Ich kann nicht sagen ab wann das Problemchen wieder Einzug hielt da ich nicht mit jeder Testversion DV's anlege.

Screenshot:
A: alte schon vorhandene DV's
B: mit 3.67.10.20230110-82325d angelegte DV's

[jp112sdl]

Jup, ist bei mir auch so.
Neu angelegte DVs machen Umlautschrott

[jens-maus]

Komisch. Wo kommt denn das jetzt schon wieder her?!? Bitte mal versuchen auf Vorversionen zurückzugehen um rauszubekommen wann das Problem denn wieder Einzug gehalten hat.

[jp112sdl]

Mit der 3.65.11.20221218 ist es noch ok

[jp112sdl]

Was mir nur spontan einfällt, wär die Umstellung von GET auf POST bzgl. ic_ifacecmd.cgi

[jens-maus]

Stimmt, das könnte es sein. Müsste man mal den commit zurücknehmen und schauen ob das das Problem löst. Und dann ne Lösung dafür finden wenn die Parameter an der stelle falsch/anders kodiert ankommen…

[jens-maus]

Kann es nun selbst reproduzieren hier. Wenn jemand jedoch den commit mal testweise zurücknehmen könnte kommen wir dem ganzen aber vllt. etwas schneller auf die schliche.

[jp112sdl]

Wenn ich das hier rückgängig mache, geht es wieder:

RaspberryMatic/buildroot-external/patches/occu/0173-WebUI-SendPOSTRequest.patch

Lines 186 to 187 in 48b6fc8

	- SendRequest('ic_ifacecmd.cgi');
	+ SendPOSTRequest('ic_ifacecmd.cgi');

[jens-maus]

UIuiui. Da haben wir (bzw. @Baxxy13) wohl ne uralt Regression erwischt. Good Job!

Siehe:
b760245#diff-db9c2b95585f64bc2e32e7ecca0791bd1a1543e513a3a382a7ada4d7b89b4457R1309

Damals (2018) hatte ich diese Zeilen eingebaut um Probleme mit dem EDGE Browser auszumerzen (siehe #315) und da ist mir wohl ein Missgeschick passiert weil ich das $ vor der variablen wohl vergessen habe. Komisch das das nicht aufgefallen war. Bitte also einfach mal in der /www/tcl/extern/cgi.tcl an besagter Stelle (Zeile 1309) das $ hinzufügen und einfach nochmal probieren mit aktiviertem SendPOSTRequest(). Sollte nun hoffentlich funktionieren.

Und wäre natürlich gut jetzt vllt. nochmal das EDGE Problem von damals nachzutesten wenn das geht und auch wieder breit auf die Suche zu gehen ob dieser Fix hier jetzt nicht wiederum an anderen Stellen die Umlaute kaputt macht (hoffen wir es mal nicht!). Aber dieses Problem hier sollte damit hoffentlich gelöst sein.

[blackbasket]

Den alten Edge gibt es doch gar nicht mehr @jens-maus

[MichaelN0815]

Bitte also einfach mal in der /www/tcl/extern/cgi.tcl an besagter Stelle (Zeile 1309) das $ hinzufügen

Ich habe diese Änderung durchgeführt und danach eine DV erstellt. Umlaute werden korrekt dargestellt. (mit Firefox & Edge getestet)

[jens-maus]

Na dann war das dieses mal ne recht einfache Sache… puh. Butte aber wie gesagt jetzt mal hingehen un an verschiednsten Stelle testen ob umlaute da auch noch korrekt gehen. Z.b. in Gruppen, anderen DVs, Geräteeinstellungen, etc. etc. Nicht das uns das beim nächsten Release dann wieder auf die Füße fällt…