Vis History-adapter bug mit history-provider

[frankjoke]

So, da ich etliche dieser Widgets laufen hab hat mich generft dass sie die Adapterinstanz 'verloren' haben.

Immer wenn man was geändert oder das widget auch nur im editor angeklickt hat hat ist die History-Instant-setting verschwunden.

Jetzt hab ich mal im code nachgeschaut und gesehen dass es im history.html ein Problem in der Funktion unten gibt (das ist meine geänderte Version die jetzt funktioniert).

    if (vis.editMode) {
        vis.binds.history.onHidChanged = function (widgetID, view, newId, fields) {
            var obj = vis.objects[newId];
            if (!obj || !obj.common) return null;
            var changed = [];
            var actualInstance = vis.views[view].widgets[widgetID].data.instance;
            // Check if actual instance still in options
            var firstInstance = '';
            var found = false;
            for (var h in obj.common.custom) {
                if (obj.common.custom[h].enabled) {
                    if (firstInstance==='') firstInstance = h;
                    if (h === actualInstance) {
                        found = true;
                        break;
                    }
                }
            }
            if (!found) {
                changed.push('instance');
                vis.views[view].widgets[widgetID].data.instance = firstInstance;
            }
            return changed.length ? changed : null;
        };
    }

Der Grund war dass der Originalcode in obj.common.history und nicht obj.common.custom nach den historyprovidern sucht.

Noch dazu ist die firstInstance auch nicht gut gelöst gewesen, hab ich die Abfrage auch verändert.

Kann das wer in der nächsten Version einstellen?

MFG

Frank

[Bluefox]

Danke. Habe übernommen.