From f7a07a9e79160401be7bde3223830b10995757a7 Mon Sep 17 00:00:00 2001
From: Bruno Windels <brunow@element.io>
Date: Fri, 20 Nov 2020 11:45:14 +0100
Subject: [PATCH] adjust fileview/tile to pendingevent changes

---
 .../session/room/timeline/tiles/FileTile.js   | 69 +++++++++----------
 .../web/ui/session/room/timeline/FileView.js  | 16 +++--
 .../web/ui/session/room/timeline/common.js    |  2 +-
 3 files changed, 45 insertions(+), 42 deletions(-)

diff --git a/src/domain/session/room/timeline/tiles/FileTile.js b/src/domain/session/room/timeline/tiles/FileTile.js
index c87b090f..f3a31e21 100644
--- a/src/domain/session/room/timeline/tiles/FileTile.js
+++ b/src/domain/session/room/timeline/tiles/FileTile.js
@@ -17,21 +17,17 @@ limitations under the License.
 
 import {MessageTile} from "./MessageTile.js";
 import {formatSize} from "../../../../../utils/formatSize.js";
+import {SendStatus} from "../../../../../matrix/room/sending/PendingEvent.js";
 
 export class FileTile extends MessageTile {
     constructor(options) {
         super(options);
-        this._error = null;
+        this._downloadError = null;
         this._downloading = false;
-        if (this._isUploading) {
-            this.track(this._entry.attachments.url.status.subscribe(() => {
-                this.emitChange("label");
-            }));
-        }
     }
 
     async download() {
-        if (this._downloading || this._isUploading) {
+        if (this._downloading || this.isPending) {
             return;
         }
         const content = this._getContent();
@@ -43,7 +39,7 @@ export class FileTile extends MessageTile {
             blob = await this._mediaRepository.downloadAttachment(content);
             this.platform.saveFileAs(blob, filename);
         } catch (err) {
-            this._error = err;
+            this._downloadError = err;
         } finally {
             blob?.dispose();
             this._downloading = false;
@@ -51,39 +47,40 @@ export class FileTile extends MessageTile {
         this.emitChange("label");
     }
 
-    get size() {
-        if (this._isUploading) {
-            return this._entry.attachments.url.localPreview.size;
-        } else {
-            return this._getContent().info?.size;
-        }
-    }
-
-    get _isUploading() {
-        return this._entry.attachments?.url && !this._entry.attachments.url.isUploaded;
-    }
-
     get label() {
-        if (this._error) {
-            return `Could not decrypt file: ${this._error.message}`;
-        }
-        if (this._entry.attachments?.url?.error) {
-            return `Failed to upload: ${this._entry.attachments.url.error.message}`;
+        if (this._downloadError) {
+            return `Could not download file: ${this._downloadError.message}`;
         }
         const content = this._getContent();
         const filename = content.body;
-        const size = formatSize(this.size);
-        if (this._isUploading) {
-            return this.i18n`Uploading (${this._entry.attachments.url.status.get()}) ${filename} (${size})…`;
-        } else if (this._downloading) {
-            return this.i18n`Downloading ${filename} (${size})…`;
-        } else {
-            return this.i18n`Download ${filename} (${size})`;
-        }
-    }
 
-    get error() {
-        return null;
+        if (this._entry.isPending) {
+            const {pendingEvent} = this._entry;
+            switch (pendingEvent?.status) {
+                case SendStatus.Waiting:
+                    return this.i18n`Waiting to send ${filename}…`;
+                case SendStatus.EncryptingAttachments:
+                case SendStatus.Encrypting:
+                    return this.i18n`Encrypting ${filename}…`;
+                case SendStatus.UploadingAttachments:{
+                    const percent = Math.round((pendingEvent.attachmentsSentBytes / pendingEvent.attachmentsTotalBytes) * 100);
+                    return this.i18n`Uploading ${filename}: ${percent}%`;
+                }
+                case SendStatus.Sending:
+                    return this.i18n`Sending ${filename}…`;
+                case SendStatus.Error:
+                    return this.i18n`Error: could not send ${filename}: ${pendingEvent.error.message}`;
+                default:
+                    return `Unknown send status for ${filename}`;
+            }
+        } else {
+            const size = formatSize(this._getContent().info?.size);
+            if (this._downloading) {
+                return this.i18n`Downloading ${filename} (${size})…`;
+            } else {
+                return this.i18n`Download ${filename} (${size})`;
+            }   
+        }
     }
 
     get shape() {
diff --git a/src/platform/web/ui/session/room/timeline/FileView.js b/src/platform/web/ui/session/room/timeline/FileView.js
index cb99dd0a..62760b3e 100644
--- a/src/platform/web/ui/session/room/timeline/FileView.js
+++ b/src/platform/web/ui/session/room/timeline/FileView.js
@@ -19,11 +19,17 @@ import {renderMessage} from "./common.js";
 
 export class FileView extends TemplateView {
     render(t, vm) {
-        return renderMessage(t, vm, [
-            t.p([
+        if (vm.isPending) {
+            return renderMessage(t, vm, t.p([
+                vm => vm.label,
+                " ",
+                t.button({className: "link", onClick: () => vm.abortSending()}, vm.i18n`Cancel`),
+            ]));
+        } else {
+            return renderMessage(t, vm, t.p([
                 t.button({className: "link", onClick: () => vm.download()}, vm => vm.label),
-                t.time({className: {hidden: !vm.date}}, vm.date + " " + vm.time)
-            ])
-        ]);
+                t.time(vm.date + " " + vm.time)
+            ]));
+        }
     }
 }
diff --git a/src/platform/web/ui/session/room/timeline/common.js b/src/platform/web/ui/session/room/timeline/common.js
index 6f275141..d49e845b 100644
--- a/src/platform/web/ui/session/room/timeline/common.js
+++ b/src/platform/web/ui/session/room/timeline/common.js
@@ -24,7 +24,7 @@ export function renderMessage(t, vm, children) {
         pending: vm.isPending,
         unverified: vm.isUnverified,
         continuation: vm => vm.isContinuation,
-        messageStatus: vm => vm.shape === "message-status" || vm.shape === "missing-attachment",
+        messageStatus: vm => vm.shape === "message-status" || vm.shape === "missing-attachment" || vm.shape === "file",
     };
 
     const profile = t.div({className: "profile"}, [