Offensichtlich geht das auch schon (ab circa Version 64) für Thunderbird 68?
Das kann ich dir nicht sagen, ich nutze es noch nicht, da ich derzeit nur für TB68 entwickle. Ich update für TB69 erst nach release von TB68.
Wenn du mit TB68 testen willst, es gibt jetzt release candidates:
Wenn du mit TB68 testen willst, es gibt jetzt release candidates:
Danke, ich verfolge die Newgroup in der das geschrieben wurde und habe dort auch Deine Beiträge gesehen 
Momentan teste ich primär mit dem 68er RC, aber auch mit der 69er-Beta. Das erspart mir hoffentlich, dass ich Dinge "programmiere", die anschließend nicht mehr zu gebrauchen sind. Dabei ist mir auch schon aufgefallen, dass ab dem 69er dann die Toolbar-Menu-Buttons folgendes im XUL benötigen: is="toolbar-menu-button", was aber auch im 68er schon drin stehen darf ohne Probleme zu machen.
Ich versuche mich gerade einzulesen.
Kannst du kurz erklären, was konzeptionell der Unterschied zwischen aeMessenger.saveAttachmentToFolder und aewindow.messenger.saveAttachmentToFile ist, und warum der erstere ein file Objekt zurückgibt und das originale file Objekt des Attachments überschreibt?
aeMessenger.saveAttachmentToFolder ist die Funktion, die am Ende die problematische startupUrl Funktion aufruft, richtig?
Trennen beide Funktionen auch das Attachment ab (also löschen es aus der Mail?) . Wo passiert das?
Danke und viele Grüße
John
der Unterschied zwischen aeMessenger.saveAttachmentToFolder und aewindow.messenger.saveAttachmentToFile
aeMessenger.saveAttachmentToFolder ist die "eigene" Routine des Attachments, die scheinbar aus dem C++ abgeleitet wurde. Dies wird momentan wohl nur noch gebraucht, wenn man bestimmte Attachments aus dem Objekt/Array raus nehmen möchte (aufgrund der Dateigröße), oder wenn man das Datum der zu speichernden Attachments mit dem Empfangsdatum der jeweiligen E-Mail machen will (also letztlich das Datum in den Meta-Daten der resultierenden Datei auf der Festplatte).
aewindow.messenger.saveAttachmentToFile ist der Aufruf von Thunderbirds eigener Detach-Routine. Ich konnte für diese Routine bereits erreichen, dass (optional) nicht mehr bei jeder E-Mail gefragt wird, ob die Dateien tatsächlich gelöscht werden sollen.
aeMessenger.saveAttachmentToFolder ist die Funktion, die am Ende die problematische startupUrl Funktion aufruft, richtig?
Das ist richtig.
Trennen beide Funktionen auch das Attachment ab (also löschen es aus der Mail?) . Wo passiert das?
case states.DETACH:
if (thistask.isDetachEnabled && !this.isNewsMessage() && !this
.isRSSMessage()) {
aewindow.aedump('{function:AEMessage.doDetach}\n', 2);
aewindow.progress_tracker.state = states.DETACH;
thistask.listeningforMessageId = thistask.getMessageHeader()
.messageId;
var acl = aewindow.arraycompact(this.attachments_ct);
if (acl.length > 0) {
if (thistask.detachMode !== 0) {
// AEC experimental detach routines
aewindow.aedump('>>>> thistask.detachMode !== 0 \n', 2);
var deleteAtt = (thistask.detachMode === 1) || !thistask
.isExtractEnabled;
var savedfiles = (deleteAtt) ? null : aewindow
.arraycompact(this.attachments_savedfile);
this.detachTempFile = aeMessenger.detachAttachments(
aewindow.messenger, aewindow.msgWindow, acl,
aewindow.arraycompact(this.attachments_url),
// use .map(encodeURIComponent) to display nonASCII correct
// if there is an confirmation dialog
aewindow.arraycompact(this.attachments_display.map(encodeURIComponent)),
aewindow.arraycompact(this.attachments_uri),
savedfiles);
} else {
// Thunderbirds own detach routines
aewindow.aedump('>>>> thistask.detachMode === 0 \n', 2);
aewindow.messenger.detachAllAttachments(acl.length, acl,
aewindow.arraycompact(this.attachments_url),
// use .map(encodeURIComponent) to display nonASCII correct
// if there is an confirmation dialog
aewindow.arraycompact(this.attachments_display.map(encodeURIComponent)),
aewindow.arraycompact(this.attachments_uri),
false, // false = do not save(?) or ask for destination folder (?)
thistask.detachWithoutConfirmation // if true = delete without warning
);
}
// console.log("setTimeout");
// seems to be working okay
thistask.detachCancellationTimeout = setTimeout(function() {
aewindow.currentMessage.doAfterActions(states
.DELTEMPFILE)
}, 2000);
break;
}
}Die "eigenen" Routinen des Add-ons verstehe ich leider oftmals auch noch nicht richtig. In obigem Code habe ich comments drin, wo ae-eigener Code und wo Thunderbirds Code beginnt bzw. aufgerufen wird.
Also sollte ich eher veruchen über die TB funktionen Attachment Größe und Datum zu erhalten, damit die alte Funktion gelöscht und die Einstellungen vereinfacht werden können. Richtig?
Das wäre bestimmt die beste Lösung, denke ich.
Wenn Dir oder jemand anderem übrigens auf GitLab etwas auffällt, was ich anders machen sollte, dann gebt mir Rückmeldung. Es ist das absolut erste mal, dass ich diese Wochen mit Git / GitLab arbeite. Es ist nicht gerade leicht sich in die Logik der Berechtigungen und der anderen Dinge rein zu denken. Nebenbei verwende ich gleichzeitig auch das erste Mal einen Code-Editor (Visual Studio Code), nachdem ich Jahre lang mit UltraEdit lokal bei mir gearbeitet hatte... Ohne Suchmaschine und Youtube wäre ich wohl verloren 
.
Überall wo du Zugriff auf aewindow.currentTask hast, kannst du via aewindow.currentTask.getMessageHeader() auf die Header Daten der Email zugreifen (https://developer.mozilla.org/en-US/docs/Moz…BHdr#Attributes)
In AEMessage.prototype.saveAtt kannst du direkt als erstes, also hier
https://gitlab.com/ThunderbirdMai…_window.js#L885
mal folgendes einfügen
console.log(aewindow.currentTask.getMessageHeader().subject + " : " + aewindow.currentTask.getMessageHeader().dateInSeconds)
und bekommst für jede Mail den gesuchten Timestamp.
Hilft das?
Um an die Attachment Größe zu kommen, hab ich das hier als Vorlage genommen.: https://searchfox.org/comm-central/s…HdrView.js#1806
Bei meinen Tests hat das funktioniert, aber es gibt noch offene Fragen.
Zunächst habe ich eine weitere Methode von AEMessage definiert:
AEMessage.prototype.getAttSize = async function(_url, isExternalAttachment, isLinkAttachment = false) {
let url = _url;
let size = 0;
let options = { method: "HEAD" };
// NOTE: For internal mailbox, imap, news urls the response body must get
// the content length with getReader().read() but we don't need to do this
// here as libmime streams it already in addAttachmentField(). For imap or
// news urls with credentials (username, userPass), we must remove them
// as Request fails such urls with a MSG_URL_HAS_CREDENTIALS error.
if (url.startsWith("imap://") || url.startsWith("news://")) {
let uri = Services.io.newURI(url);
if (uri.username)
url = url.replace(uri.username + "@", "");
if (uri.userPass)
url = url.replace(uri.userPass + "@", "");
}
let request = new Request(url, options);
await fetch(request)
.then((response) => {
if (!response.ok) {
return null;
}
if (isLinkAttachment) {
if (response.status < 200 || response.status > 304) {
return null;
}
}
return response;
})
.then(async (response) => {
if (isExternalAttachment) {
size = response ? response.headers.get("content-length") : 0;
} else {
let data = await response.body.getReader().read();
size = data.value.length;
}
})
.catch((error) => {
console.warn("getAttSize: error - " + error.message);
});
return size;
};und hier eingefügt:
https://gitlab.com/ThunderbirdMai…_window.js#L883
Aufgerufen hab ich diese Funktion dann zwischen Zeile 897 und 898, also hier:
https://gitlab.com/ThunderbirdMai…_window.js#L898
mit dieser Zeile:
this.getAttSize(attachment.url, attachment.isExternalAttachment).then(function(size) { console.log("Size: " + size); });
Das ist jetzt erstmal nur zum Testen, ob du damit für die verschiedenen Attachment Typen an die richtige Größe kommst.
1. Die Methode die ich als Vorlage benutzt habe, kennt isLinkAttachment, dein AddOn scheint das nicht zu kennen, hab es daher per default auf false gesetzt. Sagt dir isLinkAttachment was?
2. Durch fetch() ist die neue getAttSize Methode async, daher auch der Aufruf mit .then(). Kennst du async/await und Promises? Im Prinzip gabelt sich der Programmablauf durch this.getAttSize() auf, der Aufruf wird in den Hintergrund geschoben und sofort die nächste Zeile (899) abgearbeitet. Irgendwann ist die async Funktion im Hintergrund fertig und führt dann die im then() angegebene Funktion aus. Ähnlich wie ein setTimeout, nur das der Callback nicht nach einer bestimmten Zeit ausgeführt wird, sondern wenn eine andere Funktion fertig ist.
Wenn du damit die richtigen Größen bekommst, und du sicher bist, dass du dann nur noch die Thunderbird eigenen Funktionen und z.B. nicht mehr saveAttachmentToFolder benutzen willst, dann wäre es glaube ich gut, den ganzen Kram dann wirklich zu entfernen (neuer branch im gitlab) und in der reduzierten Version dann echte async Funktionen zu benutzen - oder alles nach dem Aufruf von getAttSize in den .then() zu stopfen, aber das wird so unleserlich...
Erstmal is wichtig, bekommst du den Zeitstempel (mein vorletzter Post) und die Größe und könntest somit den alten Kram rauswerfen?
Wenn du übers Wochenende zunächst erstmal den alten Code rauswirfst, könnte ich am Montag den grundlegenden Umbau hin zu async function machen (den du für fetch() zwingend brauchst). Das ist nicht sehr viel, aber es zu erklären würde länger dauern, als es zu machen.
Wenn du übers Wochenende zunächst erstmal den alten Code rauswirfst, könnte ich am Montag den grundlegenden Umbau hin zu async function machen (den du für fetch() zwingend brauchst). Das ist nicht sehr viel, aber es zu erklären würde länger dauern, als es zu machen.
Dann mache ich Morgen einen neuen Branch auf, in dem ich den alten Code raus werfe und melde mich bis Montag wieder.
GitLab has some strange limitations. I cannot create a merge request for your dev-next-68 branch. Only into your master. Even though I commited my changes into the dev-next-68 branch of my fork.
Could you make your dev-next-68 branch the default branch (if that is possible with GitLab)?
Could you make your dev-next-68 branch the default branch (if that is possible with GitLab)?
Sorry, erst jetzt gesehen. dev-next-68 ist jetzt der default branch.
Also ich komme mit gitlab nicht klar. Ich habe meinen Fork gelöscht, neu geforked, die Änderungen neu gemacht und kann jetzt trotzdem nur auf deinen master mergen. Hier ist jetzt einfach die "neue" Datei am Stück, kannst du die bei dir einfach selbst committen?
Wenn ich merge requests erstellen will, kann ich nur lokale MR erstellen, dein Repo kann ich garnicht auswählen. Manchmal taucht eine vorgegebene Option "merge in ThunderbirdMailDE/attachmentextractor master", wo ich aber nix änder kann. Sehr sehr komisch. Musst du mir irgendwie eine merge berechtigung geben?
Zu deiner Frage: Es geht nicht um Performance. Es geht darum, dass fetch() async ist und wir das jetzt benutzen müssen, um an die size zu kommen. Async ist die Zukunft, weil das multiple threads nutzen kann. Async ist also auf jeden Fall gut (deswegen ist fetch() asyny). Man muss keine Callbacks mehr definieren (so wie jetzt bei setTimeout, z.B.), die geschachtelt echt widerlich werden. Mit async/await kann man async code schreiben wie früher sequentiellen code.
Das Problem, man kann sync code nicht einfach so in async code umwandeln. Die Funktion saveAtt() muss auf jeden Fall async werden, damit wir darin await getSize() machen können. Jetzt muss man in der call sequence nach oben gehen und eigentlich jeden Aufruf in der Hierarchie überprüfen und gucken, wo man den Übergang von sync zu async machen kann. setTimerouts sind super, die erzeugen ja quasi bereits parallelen async code und daher kann man da ganz prima den Übergang machen.
Problematisch ist der Aufruf von saveAtt(0) (ich hab gerade ein Kleinkind aufm Schoß, daher keine Zeilennummern). Wenn ich das so lasse wie jetzt bei mir committet, ändert sich der Execution flow: Anstatt saveAtt(0) auszuführen, wird es als Hintergrund-Task geplant und dann direkt weiter gemacht, ohne auf das Ausführen von saveAtt(0) zu warten. Das muss ich mir noch genauer Angucken. Kannst du einfach mal ausprobieren, ob es funzt?
Musst du mir irgendwie eine merge berechtigung geben?
Das kann sein, ich schaue mal .... sieht eigentlich in den Einstellungen so aus, als könnte jeder (was so aber nicht explizit da steht) einen Merge Request machen. Nur das Approval muss ich als Maintainer bzw. Owner selbst machen. Eigentlich müsste es also gehen. Ich schaue mir später die ganzen Einstellungen noch mal in Ruhe durch.
Deine Datei habe ich zum Testen schon kopiert ich werde diese dann quasi selbst commiten und pushen. Dabei habe ich schon einen Logik-Fehler bei
if (sizeToSmall) {
return;gefunden. Dort muss es wohl korrekt
if (sizeToSmall) {
return aewindow.currentMessage.saveAtt_cleanUp(attachmentindex, true);heißen. Sonst bleibt der ganze Prozess einfach stehen.