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

Added sanity checks to definition of mrIcons for Screensaver #1290

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

stephan233
Copy link

@stephan233 stephan233 commented Dec 26, 2024

Since some months ago there is a problem regarding the mrIcons[1,2] of the screenSaverEntities. Specifically i see crashes of the javascript-adapter of Iobroker, when values of an OpenDTU instance shall be displayed as mrIcon[1,2]ScreensaverEntity.ScreensaverEntityValue. It seems like the error handling is not good enough yet.
After applying the proposed changes, i see no crashes of the javascript-adapter anymore. Before, as an example, i saw this stack trace:
Ohne Titel 8
This relates to a broken path to one of the relays and a broken promise. Both are caught in this proposed change.

@Armilar Armilar added the ioBroker releated to ioBroker Script label Dec 27, 2024
@tt-tom17
Copy link
Collaborator

Danke für dein PR, ich habe mir das im ganzen noch einmal genauer angesehen.

Das Problem ist eigentlich etwas anderes. In diesem Codeabschnitt soll der Status des Relais ermittelt werden und nur der.
Hier liegt das eigentliche Problem
if (obj.id!.substring(0, 4) == 'mqtt') { da nur auf mqtt geschaut wird, können alle mqtt-Datenpunkte dort durch. Der Pfad zum Relais in 0_userdata wird von der Funktion async function Init_Relays() { sicher gestellt und ist damit immer erreichbar.

Mein Vorschlag wäre die Konstante NSPanelReceiveTopic zu nutzen und anzupassen über substring(), damit wird sicher gestellt, dass das richtige Relais gesetzt wird.

Ich lass es hier als Diskussion offen

@stephan233
Copy link
Author

Ursachenbehebung ist natürlich besser, als Sympthombekämpfung.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ioBroker releated to ioBroker Script
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants