database: change reassignment selection to modal dialogue

[Morg42]

Wie im gitter schon geschrieben, komme ich hier nicht weiter. Die Parameter orphanID und newID (im WebUI/JS) sollten an das WebIf (orphan-id, new-id / Python) übergeben werden. Das geht aber (hier?) nicht mehr, und ich habe keinen Plan, wieso.

Falls jemand dazu einen Vorschlag hat, wäre das super...

[bmxp]

Ich rate mal ins Blaue: Du übergibst mit shngPost nur das dict mit orphan_id und new_id. Im Python Code wird unter dem Cherrypy expose aber noch mehr erwartet was nicht none ist, sonst bricht es ab. Hast Du das mal geprüft?

[Morg42]

Zum Einen habe ich - soweit ich das sehen kann - denselben Aufruf wie im "alten" PR, und da schien es zu gehen:

alt: shngPost('', {orphanID: orphanID, newID: value});
neu: shngPost('', {orphan_id: orphanID, new_id: newID});

Ich habe die Parameter zum Testen umbenannt und die Variablen aufgrund der Umstellung auf den Modaldialog.

in webif.py:
alt:

def index(self, reload=None, action=None, item_id=None, item_path=None, time_end=None, day=None, month=None, year=None, time_orig=None, changed_orig=None, orphanID=None, newID=None):
        pagelength = self.plugin.get_parameter_value('webif_pagelength')
        if item_path is not None:
            item = self.plugin.items.return_item(item_path)
        delete_triggered = False
        if orphanID is not None and newID is not None and orphanID != newID:
            self.plugin.reassign_orphaned_id(orphanID, to=newID)

neu:

def index(self, reload=None, action=None, item_id=None, item_path=None, time_end=None, day=None, month=None, year=None, time_orig=None, changed_orig=None, orphan_id=None, new_id=None):
        pagelength = self.plugin.get_parameter_value('webif_pagelength')
        if item_path is not None:
            item = self.plugin.items.return_item(item_path)
        delete_triggered = False

        self.logger.error(f'index called with oid {orphan_id} and nid {new_id}')

        if orphan_id is not None and new_id is not None and orphan_id != new_id:

            self.logger.error(f'calling reassign for {orphan_id} and {new_id}')

            self.plugin.reassign_orphaned_id(orphan_id, to=new_id)

Ich hab die debug-Logs mal dringelassen; die Parameter orphan_id und new_id sind immer None, egal, welche Werte ich mitgebe; und dass, obwohl die Variablen vor dem JS-Aufruf von shngPost Werte haben (zuletzt zum Testen 29 und 32).

Alle anderen Parameter von webif/__init__.py sind optional und leer; ich wüsste nicht, wo es da haken sollte. Zwischendurch hat es ja funktioniert, aber ich weiß nicht, wo da der Wurm ist. Und ich weiß auch nicht, wo ich das debuggen könnte...

[Morg42]

Komisch. Jetzt im Test läuft es auf einmal - nur ein Reload triggert den reassign immer wieder (er "löscht" die POST-Parameter nicht). Das tut zwar nicht weh, weil es nichts zu transferieren gibt, aber schön ist das nicht. :-/ kann man das per XHR auf eine andere Methode statt index leiten? Dann wäre das vom Refresh unabhängig und könnte dann problemlos die aktualisierten Daten anzeigen...

[Morg42]

Okay, jetzt habe ich es mit etwas Rumprobieren nach AJAX/REST umgeschrieben, jetzt passt es soweit.

Was ich noch nicht implementiert habe (weiß nicht, wie... ): ggf. Fehlermeldungen beim Neuzuweisen (v.a. gleiche Zeitstempel) an das WebIf zurückzugeben und anzuzeigen. Falls jemand Ideen hat - gern.

Ansonsten bitte gerne testen und "feedbacken", ich würde es dann Ende der Woche mergen.