Function Property Wrapper Cleanup and Improvements

[cornelius-koepp]

No description provided.

[mumpf]

Is ok for me.

[mumpf]

Nach einem Test muss ich feststellen, folgendes funktioniert nicht:
var apduLength = 15; if (typeof online.getMaxApduLength == "function") { apduLength = online.getMaxApduLength(); }
ETS-Log:
2026-04-21 13:31:23,683 [219] INFO Knx.Ets.Hawk.NetworkFacade - MaxCommunicationApduLength(13.0.36)=239 ... 2026-04-21 13:31:23,687 [226] INFO Script - BASE_invokeFunctionPropertyWrapper: APDU = 15
Ich werde die Änderung zurücknehmen.

[traxanos]

warum wurde die schreibweise überhaupt geändert?

[mumpf]

Cornelius meinte wohl, dass eine Exception schlechter ist als ein if.
Das Problem war aber, dass er das nicht vorher getestet hatte. Und bin davon ausgegangen, dass es getestet ist und hatte natürlich nichts gegen die Vermeidung von Exceptions. Und da es ein PR war, hab ich gleich gemerged.

[cornelius-koepp]

Mit dem try-catch würden mögliche andere Fehler verdeckt. Wobei wir hier trotzdem noch ein Fallback vorsehen sollten, damit es niemals daran scheitert.

Die Variable sollte (lokal in der Funktion) mit var deklariert werden.

Habe hier leider versäumt das Draft-Flag zu setzen und den Status hier im Ticket klarer zu dokumentieren. Mein Fehler :-\