search

PSE-2012 / MMWTV

Objective Quality Assessment Toolkit
3 stars 2 forks source link

Macros speichern #84

Closed sleidig closed 12 years ago

sleidig commented 12 years ago

Die frühere Funktionalität Macros über die PluginList zu speichern ist mit dem fix zu #74 (Doppelklick erstellt Memento) kaputt.

sleidig commented 12 years ago

Insgesamt lässt sich vielleicht dabei auch eine intuitivere Behandlung von Macros in der PluginList umsetzen

rtur commented 12 years ago

Macros lassen sich nun speichern / umbenennen / speichernUnterAnderem namen.

Es scheint ein Problem beim speichern unter anderem Namen zu geben.

Undzwar überprüft die setMemento des Macros ob das rootEntryname == mementoname ist (es sollte nie anders sein, also eine simple consistency überpfürung), aber beim wiederladen dieses mementos taucht unter rootEntry(oder unter memento.name, weiß grad nicht welches das war) das alte (vor saveAs aufruf).

Ich habe die macro methoden dazu durchgeschaut, da sieht alles soweit gut aus. Es passiert wahrscheinlich irgendwo hier:

(reihenfolge ist gleichzeitig aufrufreihenfolge)

VM_PluginLists.onMacroSaveAs VM_PluginLists.mementoSaveAs VM_PluginLists.mementoSave

andere Methoden in den der fehler wahscheinlich nicht zu finden sein wird es aber könnte (d.h. ich hab die durchgeschaut und nichts gefunden, aber die haben Zuggriff auf die entsprechenden Variablen und rufen die auch auf):

VM_PluginLists.onMacroSave Macro.saveSaveAsHelper macro.getMemento Macro.setMemento Macro_PropertyView.saveMacro_Click Macro_PropertyView.saveMacroAs_Click

rtur commented 12 years ago

Das speichern Problem an sich ist nun gelöst. Was noch schiefläuft ist die gui, beim click werden blöderweise alle Elemente unselected.. Ich schaue es mir gleich als nächstes an.