Inkonsistenz in adapter.js

[AlCalzone]

Mir ist da gerade was aufgefallen in adapter.js:

1. Funktion createDevice (Zeile 1643ff.): Das Objekt wird angelegt mit "setObjectNotExists", d.h. nur wenn es nicht existiert.

2. Funktion createChannel (Zeile 1674ff.): Das Objekt wird angelegt mit "setObject", d.h. wird überschrieben wenn es existiert.

3. Funktion createState (Zeile 1643ff.): Das zum State gehörende Objekt wird angelegt mit "setObjectNotExists", d.h. nur wenn es nicht existiert.

Ich gehe davon aus, dass das Verhalten in createChannel nicht absichtlich anders ist.

[AlCalzone]

Nochwas:

Zeile 2186ff, die Reihenfolge der Parameter ist anders als überall sonst:

        that.getDevices = function getDevices(callback, options) {
            if (typeof options === 'function') {
                callback = options;
                options = null;
            }

zum Vergleich:

        that.getChannelsOf = function getChannelsOf(parentDevice, options, callback) {
            if (typeof options === 'function') {
                callback = options;
                options = null;
            }

[Bluefox]

@AlCalzone:

Nochwas:

Zeile 2186ff, die Reihenfolge der Parameter ist anders als überall sonst:

        that.getDevices = function getDevices(callback, options) {
            if (typeof options === 'function') {
                callback = options;
                options = null;
            }

zum Vergleich:

        that.getChannelsOf = function getChannelsOf(parentDevice, options, callback) {
            if (typeof options === 'function') {
                callback = options;
                options = null;
            }

```` `  

Sehr gute Anmerkungen. Ist gemacht:

https://github.com/ioBroker/ioBroker.js … c7f2d2e9c5

[apollon77]

Hey,

die eine Änderung macht sinn, die andere mit der Parameterreihenfolge ist aber inkompatibel!!

Man muss jetzt mindestens im Code checken und ggf drehen!

Ingo

[AlCalzone]

Da hast du Recht, aber aktuell macht die Abfrage auf den optionalen Parameter wenig Sinn, wenn der eh hinten kommt.