From 0971abd0ed3d3ebd9f6964661a676ccf2a766cf6 Mon Sep 17 00:00:00 2001
From: ccheng-dev <c.cheng@metaways.de>
Date: Tue, 27 Feb 2024 07:18:41 +0100
Subject: [PATCH] tweak(Felamimail/js): log more attachment preview cache data

---
 tine20/Felamimail/js/MailDetailsPanel.js | 17 ++++++++++++-----
 1 file changed, 12 insertions(+), 5 deletions(-)

diff --git a/tine20/Felamimail/js/MailDetailsPanel.js b/tine20/Felamimail/js/MailDetailsPanel.js
index 978cbda55a9..5ecef296eb3 100644
--- a/tine20/Felamimail/js/MailDetailsPanel.js
+++ b/tine20/Felamimail/js/MailDetailsPanel.js
@@ -510,13 +510,19 @@ Ext.extend(Tine.Felamimail.MailDetailsPanel, Ext.Panel, {
             if (!attachment.promises) attachment.promises = [];
             if (attachment?.isPreviewReady) return;
             const promise = new Promise(async (resolve) => {
-                let validCache = null;
-                await Promise.all(attachment.promises).then((responses) => {
-                    validCache = responses.find((r) => {return r.isPreviewReady});
-                });
+                const start = Date.now();
+                const responses = await Promise.all(attachment.promises);
+                const validCache = responses.find((r) => {return r.isPreviewReady});
+                
                 if (validCache) {
                     attachment.isPreviewReady = true;
-                    return resolve(validCache);
+                    return resolve({
+                        cache: validCache.cache,
+                        createPreviewInstantly: createPreviewInstantly,
+                        isPreviewReady: true,
+                        skipSendRequest: true,
+                        time: Date.now() - start,
+                    });
                 }
                 const attachmentId = [sourceModel, record.id, attachment.partId].join(':');
                 await Tine.Felamimail.getAttachmentCache(attachmentId, createPreviewInstantly)
@@ -525,6 +531,7 @@ Ext.extend(Tine.Felamimail.MailDetailsPanel, Ext.Panel, {
                             cache: new Tine.Tinebase.Model.Tree_Node(cache.attachments[0]),
                             createPreviewInstantly: createPreviewInstantly,
                             isPreviewReady: cache.attachments[0].preview_count !== 0,
+                            time: Date.now() - start,
                         });
                     })
                     .catch((e) => {