'replaceAll() not defined' in slightly aged browsers

[Joerg-rw]

Describe the bug
Beim Anlegen einer neuen Systemvariablen in

>>>  Startseite > Einstellungen > Systemvariablen > "Neu"    

wird ausschliesslich bei Auswahl "Typ=Zeichenkette" keine Aktion auf "OK" ausgefuehrt und damit auch die Systemvariable nicht angelegt.

Steps to reproduce the behavior

1. neue Systemvariable anlegen: Startseite > Einstellungen > Systemvariablen > "Neu"
2. beliebigen Namen und Typ=Zeichenkette auswaehlen
3. "OK" button klicken

Expected behavior
Systemvariable wird angelegt, requester schliesst

Screenshots

System information:

• RaspberryMatic Version: 3.55.10.20210213
• Used Hardware: CCU3
• Used HomeMatic RF-Module: CCU3

Additional context
RPimatic "neuling", CCU3->diese firmware fand vor wenigen tagen statt. Ich kann weder ausschliessen dass bei der migration etwas schief lief, noch dass im verwendeten browser ein bug sein koennte

[jp112sdl]

noch dass im verwendeten browser ein bug sein koennte

Hast du denn nicht zunächst erst einmal einen anderen Browser ausprobiert, um das ausschließen zu können?

replaceAll wird von /www/webui/js/extern/codemirror/addon/search/search.js bereitgestellt.

Warum die Funktion bei dir nicht läuft, ist rätselhaft.
Würde mich auch nicht wundern, wenn da mal wieder irgendein Internet Security / Antivieren Supertool etwas blockiert.

[Joerg-rw]

bisher getestet in (wird aktualisiert):

Firefox

Konqueror

Chromium Version 78.0.3904.108 (openSUSE Build) (64-bit)

selbes ergebnis in allen

[jp112sdl]

Hmm, dann bin ich auch ratlos.

[jens-maus]

Tut mir leid, auch ich kann das nicht reproduzieren. @Joerg-rw Bitte selbst in einer frischen Installation probieren, wenn es dort geht dann liegt ein Fehler in der existenten Installation vor. Wenn nicht gerne das Bugticket wieder aufmachen.

[Joerg-rw]

letzte erfolgreich angelegte Zeichenketten-var, offenbar nicht ueber WebUI sondern remote von user-iPhone bei app-installation angelegt:

pocketControlPushMessage ("Change the value to push. Do not delete")

Verdacht: die an den Browser uebergebene Liste(?) mit zeichenketten-systemvariablen enthaelt ein Element das JS nicht verarbeiten kann.
Wie koennte man diese Liste anzeigen lassen?
Wo auf der CCU wird die Liste mit SysVar gespeichert?

Update1:
Reflash der selben firmware-Version ueber Zentralenwartung "Direkt auf CCU laden und installieren";
deinstallieren der pocketcontrol app auf iPhone und anschliessendes Loeschen von pocketControlPushMessage SysVar in CCU-WebUI sowie anschliessender reboot:
keine Aenderung, Fehler bleibt identisch

Update2:
User konnte von anderem PC aus angeblich eine Zeichenketten-SysVar anlegen. Bis auf weiteres "ticket closed"

[Joerg-rw]

pb += "string sValue = '"+$("txValue").value.replaceAll("'","\\'")+"';";

[jens-maus]

Was für ein browser ist das? Die .replaceAll() funktion sollte es in jedem halbwegs modernen browser geben. Die Fehlermeldung verstehe ich daher nicht ganz bzw. kann sie nicht nachvollziehen (siehe https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/String/replaceAll)

BTW: Bitte Screenshots hier direkt in GitHub hochladen (Drag&Drop). Sowas gehört nicht irgendwo extern hin - hab deinen letzten jetzt hier mal integriert.

[Joerg-rw]

ich bin ratlos

Fehler tritt auf drei Browsern (FF 68.2.0esr (64-bit), Konqueror, Chromium Version 78.0.3904.108 (openSUSE Build) (64-bit)) auf OpenSuse Linux in gleicher Weise auf, auf einem Macbook (FF) jedoch nicht.

Das System wurde vor wenigen Tagen frisch von original-CCU3 nach RasPiMatic migriert und heute nochmal neu auf die gleiche Version ge"update"et
Evtl Ticket Subject anpassen und Ticket wieder oeffnen?

[Joerg-rw]

Was für ein browser ist das? Die .replaceAll() funktion sollte es in jedem halbwegs modernen browser geben. Die Fehlermeldung verstehe ich daher nicht ganz bzw. kann sie nicht nachvollziehen (siehe https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/String/replaceAll)

Danke! :-) Das war wohl genau auf den Punkt
Ich weiss nicht ob man nicht sowas wie $("txValue").value.replace(/'/g, '\\\''); (oder so aehnlich ;-D, wichtig das "g" ) stattdessen verwenden koennte.

Der Vorteil waere:
in FF z.B. definiert seit version 1 von 2004-11-09 https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/String/replace#browser_compatibility
gegenueber FF v.77 2020-06-02 https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/String/replaceAll#browser_compatibility

ps: >>Wenn nicht gerne das Bugticket wieder aufmachen.
Gerne, aber wie? ;-D

[jens-maus]

Ok, also gehe ich davon aus hattest du eine Firefox Version < 77 im Einsatz? Welche genau?

[Joerg-rw]

Ok, also gehe ich davon aus hattest du eine Firefox Version < 77 im Einsatz? Welche genau?

FF 68.2.0esr (64-bit), Konqueror, Chromium Version 78.0.3904.108 (openSUSE Build) (64-bit)

Danke :-)

[jp112sdl]

Zwar leicht OT: Aber mit meinem älteren Chrome (70.x irgendwas) gehen z.B. die neuen Translate-Funktionen nicht (WebUI Fix 0111). Nach dem Chrome Update ist alles schick.

Ist die Frage, inwieweit man hier (und an anderer Stelle) abwärtskompatibel sein möchte

[Joerg-rw]

Zwar leicht OT: Aber mit meinem älteren Chrome (70.x irgendwas) gehen z.B. die neuen Translate-Funktionen nicht (WebUI Fix 0111). Nach dem Chrome Update ist alles schick.

Ist die Frage, inwieweit man hier (und an anderer Stelle) abwärtskompatibel sein möchte

Evtl zumindest sinnvoller einsatz von try https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/try...catch#conditional_catch-blocks ?
disclaimer: total JS noob here

[jens-maus]

Zwar leicht OT: Aber mit meinem älteren Chrome (70.x irgendwas) gehen z.B. die neuen Translate-Funktionen nicht (WebUI Fix 0111). Nach dem Chrome Update ist alles schick.

Ist die Frage, inwieweit man hier (und an anderer Stelle) abwärtskompatibel sein möchte

Diese Frage müsste @theimo1221 beantworten. Da er die Optimierungen im 0111 WebUI Patch vorgenommen hat, müsste er am besten eine Aussage darüber treffen können inwieweit man die Anpassungen noch kompatibler zu älteren Browserversionen bekommt. Wenn nicht oder das zuviel aufwand bedeuten würden, dann würde ich es allerdings verschmerzbar finden wenn so alte Browser wie Chrome 70.x dann nicht mehr gehen. Der ist ja von ca. 2018 und lässt sich ja auch problemlos updaten. :)

[theimo1221]

Werde ich mir später mal anschauen, an sich bin ich in dem Patch teilweise eher auf Standard JS zurückgegangen, könnte mir spontan nur vorstellen, dass da wegen der ?? Notation gemeckert wird.

[theimo1221]

Werde ich mir später mal anschauen, an sich bin ich in dem Patch teilweise eher auf Standard JS zurückgegangen, könnte mir spontan nur vorstellen, dass da wegen der ?? Notation gemeckert wird.

Jo, ist die ?? Notation. Ich hoffe ich komme dazu das gleich noch zu ändern. Sind in der Firma gerade ordentlich am Gas geben 😄

[jens-maus]

Jo, ist die ?? Notation. Ich hoffe ich komme dazu das gleich noch zu ändern. Sind in der Firma gerade ordentlich am Gas geben 😄

Super. Wäre in der Tat schön wenn du das noch flott reparieren könntest oder einfach kurz nen diff hier postest und dann integriere ich das schnell noch für die nächsten nightly snapshot.

[theimo1221]

Jo, ist die ?? Notation. Ich hoffe ich komme dazu das gleich noch zu ändern. Sind in der Firma gerade ordentlich am Gas geben 😄

Super. Wäre in der Tat schön wenn du das noch flott reparieren könntest oder einfach kurz nen diff hier postest und dann integriere ich das schnell noch für die nächsten nightly snapshot.

Siehe #1151....

Ich hoffe ich komme die Tage noch zu der ProofAndSetValue Thematik habe dafür hier auch schon vorbereiteten Code liegen, werde aber vermutlich erst Samstag dazu kommen.