Function subscribeOidAtRuntime broken since v1.4.11

[Scrounger]

@GermanBluefox, @mswiege

Ich schreib jetzt auf deutsch, weil es glaub einfacher ist das Problem zu beschreiben.

Seit der Version 1.4.11 geht die Funktion subscribeOidAtRuntime nicht mehr: https://github.com/ioBroker/ioBroker.vis/blob/f80feae5e5d8fecdf9acc33e344e9feed5c911fe/www/js/vis.js#L3224

Grund ist die Prüfung der Id: https://github.com/ioBroker/ioBroker.vis/blob/f80feae5e5d8fecdf9acc33e344e9feed5c911fe/www/js/vis.js#L276

Es werden keine . _, etc. mehr zugelassen, obwohl diese doch laut Definition in einer Id enthalten sein dürfen?

Kurz zum Hintergrund warum ich diese Funktion subscribeOidAtRuntime geschrieben habe. Wenn man z.B. eine Objekt Id mittels Binding zusammen stellt (Bsp: 0_userdata.0.vis.control{a:local_client; a ? ".Client_" + a : ""}.iconList.primary) kann es passieren, das die Oid nicht subscribed ist, d.h. der Wert kann nicht gelesen werden und es werden auch keine Änderungen beachtet. Grund hierfür ist dass erst zu runtime die Oid korrekt zusammen gesetzt wird. Es gibt weitere Szenarien wo Bindings manchmal nicht subscribed werden und das prüft diese Funktion auch gleich mit. Ich verwende das vorallem um eine Vis multi user fähig zu machen, in Verbindung mit local_* Id's.

Mit der Regex Prüfung der Id (https://github.com/ioBroker/ioBroker.vis/blob/f80feae5e5d8fecdf9acc33e344e9feed5c911fe/www/js/vis.js#L276) geht das aber jetzt nicht mehr.

Bevor ich jetzt einen PR schreibe, wollte ich zunächst verstehen, warum diese RegEx Prüfung eingeführt wurde und warum diese auch eigentlich erlaubte Zeichen einer Id nicht mehr zulässt. Meiner Meinung nach ist das nicht korrekt, es sollten nur Zeichen verboten werden, wie hier dokumentiert, oder? https://forum.iobroker.net/topic/12554/wichtige-%C3%A4nderung-objekt-state-id-bezeichner-d%C3%BCrfen-keine-oder-mehr-enthalten

[Apollon77]

@GermanBluefox I think the error happend when "revertig" the regex ... it needs to be adjusted to be a "blacklist" now and not a whitelist as the "unicode one"

[stale[bot]]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs within the next 7 days. Please check if the issue is still relevant in the most current version of the adapter and tell us. Also check that all relevant details, logs and reproduction steps are included and update them if needed. Thank you for your contributions. Dieses Problem wurde automatisch als veraltet markiert, da es in letzter Zeit keine Aktivitäten gab. Es wird geschlossen, wenn nicht innerhalb der nächsten 7 Tage weitere Aktivitäten stattfinden. Bitte überprüft, ob das Problem auch in der aktuellsten Version des Adapters noch relevant ist, und teilt uns dies mit. Überprüft auch, ob alle relevanten Details, Logs und Reproduktionsschritte enthalten sind bzw. aktualisiert diese. Vielen Dank für Eure Unterstützung.

[Scrounger]

#494 any news on this?