[gelöst] auf globale Objekt kann nicht zugegriffen werden

[haus-automatisierung]

@ben1983 sagte in auf globale Objekt kann nicht zugegriffen werden:

also kann man auch normal jetzt das messageToAsynch benutzen, oder wie?

Hatte ich doch schon geschrieben, dass die beiden Funktionen praktisch identisch sind. Siehe Code. messageTo hat halt eine etwas andere Signatur und braucht weniger Infos. Siehe Doku.

Ich hatte das hier gefixt: https://github.com/ioBroker/ioBroker.javascript/pull/1343

Einfach die aktuellste Stable version nehmen dann passts.

klein0r created this issue in ioBroker/ioBroker.javascript

closed Blockly improvements, messageToAsync, documentation #1343

[OliverIO]

@ben1983 sagte in auf globale Objekt kann nicht zugegriffen werden:

Schade,dass es nicht geht. Dann bekomme ich halt lauter Fehlermeldungen und weiß nicht, ob es wirklich Fehler sind.

du könntest, zumindest temporär, die zeilen in dem das objekt aufgebaut wird in dein skript kopieren. dann funktioniert auch die vervollständigung.
wenn dann alles getestet ist und es funktioniert, dann entfernst du die zeilen wieder und die zeilen aus global wirken.

das einmalige definieren in global ist vor allem dann gut, wenn du regelmäßig das objekt in seiner definition erweiterst und du dir arbeit sparen willst. dadurch werden fehler reduziert.

[Ben1983]

@haus-automatisierung OK. Cool.

Könntest DU mir freundlicherweise noch sagen, warum ich hier keinen Log der Daten im Callback bekomme?
Ich finde es nicht. Aufruf wird geloggt

sendTo("javascript.0",'toScript', {
        script: 'script.js.x_testskripte._a_Mesagetest', 
        message: 'Testmessage', 
        data: {
            flag: true
        },function(res){
            log(JSON.stringify(res));
        } 
});

onMessage("Testmessage",(messagedata,callback) =>{
    const myData = {input: messagedata,
                    output:{name:"Test"}};
    callback(myData)
    log("Aufruf ");
});

[haus-automatisierung]

@ben1983 sagte in auf globale Objekt kann nicht zugegriffen werden:

Könntest DU mir freundlicherweise noch sagen, warum ich hier keinen Log der Daten im Callback bekomme?

Deine Klammern sind falsch. Die Function ist teil des Msg-Objektes und nicht als separater Parameter definiert. Wenn Du den Code schöner formatieren würdest, siehst Du das auch auf einen Blick

So gehts:

sendTo(
    'javascript.0',
    'toScript',
    {
        script: 'script.js.test.playground', 
        message: 'Testmessage', 
        data: {
            flag: true
        }
    },
    (res) => {
        console.log(JSON.stringify(res));
    }
);

onMessage('Testmessage', (messagedata, callback) => {
    console.log('Aufruf');
    const myData = {
        input: messagedata,
        output: {
            name: 'Test'
        }
    };

    callback(myData);
});

[Ben1983]

@haus-automatisierung OK. Danke

Beim Verwendet von messageToAsynch tritt dieser Fehler auf:

messageToAsync("Testmessage",{name:"Testname"});

	javascript.0 (20591) script.js.x_testskripte._a_Mesagetest: TypeError: Cannot read properties of undefined (reading 'timeout')

[paul53]

@ben1983 sagte: "Besser" das Objekt nur dazu definieren, wo es gebraucht wird?

Ja. Die Gruppe "global" verwendet man für häufig verwendete eigene Funktionen.
Mit globalen Objektdefinitionen müllt man sich den RAM der Javascript-Instanz zu, da sie in jedem nicht globalen Skript kompiliert werden.

[haus-automatisierung]

@ben1983 sagte in auf globale Objekt kann nicht zugegriffen werden:

Beim Verwendet von messageToAsynch tritt dieser Fehler auf:

Ist wohl nen Bug wenn die Options fehlen. So gehts:

const res = await messageToAsync('Testmessage', { flag: true }, { timeout: 5000 });
console.log(JSON.stringify(res));

Machst einen GitHub Issue?

[ticaki]

@ben1983 sagte in auf globale Objekt kann nicht zugegriffen werden:

messageToAsync

ups falsch gelesen

EDIT aber trotzdem relevant und die Beschreibung/Definition sollte geändert werden.

function messageToAsync(target: iobJS.MessageTarget | string, data: any, options?: any): Promise<iobJS.MessageCallback | iobJS.MessageCallbackInfo>;

Der {timeout:30000} (zeit is nur ein Beispiel) sollte im Javascript-Adapter zumindest ein default sein oder die Angabe verpflichtend wie jetzt. Ohne timeout kann das an der Stelle hängen bleiben, wie ich schmerzlich feststellen mußt mit einem gleichzeitigen Defekt des Übertemperaturwächters am Boiler.

[haus-automatisierung]

@ticaki sagte in auf globale Objekt kann nicht zugegriffen werden:

Ohne timeout kann das an der Stelle hängen bleiben, wie ich schmerzlich feststellen mußt

Habe ich gerade das erste mal von gehört. Gibts dazu schon einen GitHub Issue? Sollte zumindest in der nächsten Version gefixt sein:

https://github.com/ioBroker/ioBroker.javascript/commit/41e59dfd72f99027a9a303beee8d4d5597826445

0 klein0r committed to ioBroker/ioBroker.javascript

messageToAsync was not working without options

[ticaki]

@haus-automatisierung
Ach mist, nicht das es nicht das gleiche ist. Bei mir ist sendToAsync hängen geblieben. Sorry hab die beiden durcheinandergewürfelt.

[haus-automatisierung]

@ticaki sendToAsync funktioniert bei mir wunderbar

[ticaki]

@haus-automatisierung sagte in auf globale Objekt kann nicht zugegriffen werden:

@ticaki sendToAsync funktioniert bei mir wunderbar

Stoppe den Zieladapter und nutzte dann "sendToAsync" ohne timeout.

[haus-automatisierung]

@ticaki Bei sendToAsync gibts keinen Timeout - eventuell ist das auch dein Problem?

[ticaki]

@haus-automatisierung sagte in auf globale Objekt kann nicht zugegriffen werden:

@ticaki Bei sendToAsync gibts keinen Timeout - eventuell ist das auch dein Problem?

Das ist ja noch schlimmer, ist mir garnicht aufgefallen, dass das nicht durchgereicht wird. Ohne Temperaturwächter setzte ich kein sendToAsync mehr für Warmwassererhitzung ein.

Beide SendtoAsync bleiben hängen, wenn telegram ausgeschaltet ist.

test1()
test2()
async function test1() {
    await sendToAsync("telegram", "send", {user: 'Tim', text: 'Test1'});
    log('test1');
}
async function test2() {
    await sendToAsync("telegram", "send", {user: 'Tim', text: 'Test2'}, { timeout: 1000 });
    log('test2');
}

Das hier in meinem Adapter jedoch nicht:

await this.adapter.sendToAsync(this.options.adapter, 'send', opt, { timeout: 2000 })

https://github.com/ioBroker/ioBroker.javascript/issues/1436

ticaki created this issue in ioBroker/ioBroker.javascript

closed Missing timeout for sendToAsync() #1436

[haus-automatisierung]

Die Funktion hat nur 3 Parameter, alle weiteren werden ignoriert:

https://github.com/ioBroker/ioBroker.javascript/blob/41e59dfd72f99027a9a303beee8d4d5597826445/lib/sandbox.js#L2702

@ticaki sagte in auf globale Objekt kann nicht zugegriffen werden:

Das hier in meinem Adapter jedoch nicht:

Im der Adapter-Klasse gibt es noch einen weiteren, richtig:

https://github.com/ioBroker/ioBroker.js-controller/blob/0b01593055f5ebb61c1a9939a2a962b7b53f7344/packages/adapter/src/lib/adapter/adapter.ts#L7106

[ticaki]

@haus-automatisierung
Um wieder etwas on topic zu werden.

Der timeout sollte(geht halt aktuell nur bei messageToAsync) bei beiden ToAsync gesetzt werden, wenn diese mit await genutzt werden, den wenn der Callback wegen gestopptem Skript/Adapter nicht beantwortet wird, bleibt das Skript an der Stelle stehen.

[haus-automatisierung]

@ticaki Logisch, ist bis dato nur scheinbar niemandem aufgefallen (bzw. wurde nicht gemeldet)

[Ben1983]

@haus-automatisierung irgendwie will es heute nicht...

wieso bekomme ich jetzt keinen Log?

log(JSON.stringify( await messageToAsync("Testmessage",{name:"Testname"},{timeout:5000})));

onMessage("Testmessage",(messagedata,callback) =>{
    const myData = {input: messagedata,
                    output:{name:"Test"}};
    callback(myData)
});

[ticaki]

@haus-automatisierung
Wäre mir wohl so auch nicht klar gewesen aber erste habe ich wegen hängendem sendToAsync im Adapter erfahren dass es da keinen default timeout gibt. Und 2 Tage später fliegen die Leitungen von meinem Boiler und 4 Stunden vorher haben die Logeinträge meines Warmwasserskripts geendet. Da war der Zusammenhang nicht schwer herzustellen. Mist nur das die Überhitzungssicherheitsschaltung zu alt/ kaputt war. Naja, war zum Glück nur ne Riesensauerei.

[ticaki]

@ben1983

Das await müsste bei dir eigentlich als Fehler markiert sein.

test()

async function test() {
    log(JSON.stringify( await messageToAsync("Testmessage",{name:"Testname"},{timeout:10000})));
}
 
onMessage("Testmessage",(messagedata,callback) =>{
    const myData = {input: messagedata,
                    output:{name:"Test"}};
    callback(myData)
});

Ach und mit abgestürzter Javascriptinstanze gibts keine logs