D.h. wir verschieben das von AEMessage nach AETask? Bist du sicher?
Ok, dann reverte zunächst deinen (meinen) letztem Commit, dann ist das async erstmal wieder raus.
Dann musst du die Funktion getAttSize aus AEMessage rausholen. Du könntest Sie in EATask stecken, oder einfach als normale Funktion ganz nach unten unterhalb von AEMsgWindowCommands definieren:
async function getAttSize(...) {
...
}Die ist ja unabhängig von irgendwelchen Objekten.
Nach dem Revert, müsste in saveAtt() wieder der simple Aufruf von getAttSize().then( .. irgendwas mit console.log .. ) drin sein. Den ganzen Block, verschiebst du dann dahin wo du ihn haben willst/brauchst. Hast du in AETask Zugriff auf die prefs?
Ziel ist, dass du wieder die Größe der Attachments und auch alle relevanten prefs/variablen im log siehst, die wir zur Entscheidung von sizeToSmall brauchen.
Wenn das wieder klappt, guck ich mir an, welche Funktionen dann alle async werden müssen.
Meld dich, wenn was unklar ist.
Bist du sicher?
Ja, der "neue" Ort ist der korrekte Ort, um zu entscheiden, welche Attachments bearbeitet werden sollen. Es bleibt ja nicht nur beim Speichern, sondern kommt auch zum Folgeprozess des Löschens (je nach Option), welcher mit der aktuellen Lösung zwar funktioniert, aber im Rückfrage-Dialog zu einem undefined führt. Ich habe das im Code alles sehr gut nachvollziehen können. Man könnte zwar das Problem wieder "umschiffen", aber sinnvoll ist das eher nicht. Besser wir halten uns an die korrekte Reihenfolge der Prozesse.
Ich mache das dann später und melde mich. Danke
Ich habe bei meinen Addons auch den Cut gemacht und eine reine TB68+ Versionen erstellt und alles alte rausgeworfen.
ich bin ganz bei dir, so wenig wie möglich zu ändern, um zeitnah auf ATN releasen zu können!
Ich hab die AETask Sache noch nicht verstanden. Z.B. *wie* wird handleAttachment in AETask eigentlich aufgerufen, also das hier:
https://gitlab.com/ThunderbirdMai…_window.js#L470
Ich vermute, irgendwie implizit, weil irgendeiner API das ganz AETask object als callback übergeben wird, aber wo?
Bei AEIndTask finde ich zumindest einen expliziten call:
https://gitlab.com/ThunderbirdMai…_window.js#L763
Oder ist das auch der Aufruf für den AETask.handleAttachment, weil that sowohl AETask als auch AEIndTask sein kann?
Oder ist das auch der Aufruf für den AETask.handleAttachment, weil that sowohl AETask als auch AEIndTask sein kann?
Nein, von dort aus wird es nicht aufgerufen. Das kann man per dump erkennen, da die entsprechende dump-Anweisung dann nicht aufgerufen wird. Es muss also irgendwie anders aufgerufen werden. Das habe ich bisher auch nicht verstanden. Vermutlich sollte man auch deshalb vorsichtig beim Entrümpeln der scheinbar Redundanten Funktionen sein.
Ich habe bei meinen Addons auch den Cut gemacht und eine reine TB68+ Versionen erstellt und alles alte rausgeworfen.
Mit dem Unterschied, dass es nie eine vernünftige Version des AttachmentExtractors für Version 60 gegeben hat.
Was muss ich machen, um deine dumps zu sehen?
Ich bin das WE familiär stark gebunden und werde nix machen können. Wenn du Zeit hast, könntest du mal versuchen herauszubekommen, durch welchen Aufruf am Ende handleAttachment in AETask aufgerufen wird?
Danke!
durch welchen Aufruf am Ende handleAttachment in AETask aufgerufen wird?
Ich vermute, das brauchen wir dringend für async? Ich schaue es mir (zum x-ten mal erneut) an und versuche es zu verstehen.
Ich hatte mir mal einen Crowdin-Account angelegt und bin heute erstaunt, dass man dort monatlich ordentlich Geld blechen muss bzw. müsste. Leider ist BabelZilla inzwischen sowas von veraltet und verschluckt sich beispielsweise an den JSON-Dateien, wenn die im XPI mit drin sind. Nunja, dennoch probiere ich über BabelZilla die Übersetzungen aktualisieren zu lassen, obwohl man das ja auch in GitLab machen könnte.
Danke für deine Mühe! Beim Review muss man die Leute anstupsen, ich mach das mal. Der Doppel-Upload wird evtl. so nicht funktionieren, es kann immer nur eine version im review sein. Die jetzt deaktiviert wurde musst du neu hochladen, wenn die andere durch ist.
Ich muss bei meinen AddOns auch gerade noch nacharbeiten und kann deswegen gerade nicht helfen. Ich melde mich aber, sobald das bei mir durch ist.
Wie genu heißt das AddOn auf ATN? Gibts ne Seite?
VG
John
Vermutlich laufen da also ein paar automatische Tests gar nicht?
Korrekt.
Sonst hast du alles richtig gemacht.
Meinen Infos nach ist 2.0b1 bereits freigeschaltet, oder?
Nein, ist nicht freigeschaltet, sorry.
Nein, ist nicht freigeschaltet, sorry.
genau
