From 435f15ec6a6cb696444f2b7f121fbd8415b26798 Mon Sep 17 00:00:00 2001
From: Chaim Lev-Ari <chiptus@users.noreply.github.com>
Date: Thu, 23 Jul 2020 10:44:32 +0300
Subject: [PATCH] refactor(agent): refactor file-uploader to es6 (#4087)

* refactor(host): convert fileUploader to es6

* refactor(agent): rename main file
---
 .../file-uploader/file-uploader-controller.js | 23 ---------------
 .../components/file-uploader/file-uploader.js |  7 -----
 .../{file-uploader.html => fileUploader.html} |  0
 .../file-uploader/fileUploaderController.js   | 29 +++++++++++++++++++
 app/agent/components/file-uploader/index.js   | 10 +++++++
 5 files changed, 39 insertions(+), 30 deletions(-)
 delete mode 100644 app/agent/components/file-uploader/file-uploader-controller.js
 delete mode 100644 app/agent/components/file-uploader/file-uploader.js
 rename app/agent/components/file-uploader/{file-uploader.html => fileUploader.html} (100%)
 create mode 100644 app/agent/components/file-uploader/fileUploaderController.js
 create mode 100644 app/agent/components/file-uploader/index.js

diff --git a/app/agent/components/file-uploader/file-uploader-controller.js b/app/agent/components/file-uploader/file-uploader-controller.js
deleted file mode 100644
index d0c5ad798..000000000
--- a/app/agent/components/file-uploader/file-uploader-controller.js
+++ /dev/null
@@ -1,23 +0,0 @@
-angular.module('portainer.agent').controller('FileUploaderController', [
-  '$q',
-  function FileUploaderController($q) {
-    var ctrl = this;
-
-    ctrl.state = {
-      uploadInProgress: false,
-    };
-
-    ctrl.onFileSelected = onFileSelected;
-
-    function onFileSelected(file) {
-      if (!file) {
-        return;
-      }
-
-      ctrl.state.uploadInProgress = true;
-      $q.when(ctrl.uploadFile(file)).finally(function toggleProgress() {
-        ctrl.state.uploadInProgress = false;
-      });
-    }
-  },
-]);
diff --git a/app/agent/components/file-uploader/file-uploader.js b/app/agent/components/file-uploader/file-uploader.js
deleted file mode 100644
index 6232c7f8d..000000000
--- a/app/agent/components/file-uploader/file-uploader.js
+++ /dev/null
@@ -1,7 +0,0 @@
-angular.module('portainer.agent').component('fileUploader', {
-  templateUrl: './file-uploader.html',
-  controller: 'FileUploaderController',
-  bindings: {
-    uploadFile: '<onFileSelected',
-  },
-});
diff --git a/app/agent/components/file-uploader/file-uploader.html b/app/agent/components/file-uploader/fileUploader.html
similarity index 100%
rename from app/agent/components/file-uploader/file-uploader.html
rename to app/agent/components/file-uploader/fileUploader.html
diff --git a/app/agent/components/file-uploader/fileUploaderController.js b/app/agent/components/file-uploader/fileUploaderController.js
new file mode 100644
index 000000000..fe6f185c2
--- /dev/null
+++ b/app/agent/components/file-uploader/fileUploaderController.js
@@ -0,0 +1,29 @@
+export class FileUploaderController {
+  constructor($async) {
+    Object.assign(this, { $async });
+
+    this.state = {
+      uploadInProgress: false,
+    };
+
+    this.onFileSelected = this.onFileSelected.bind(this);
+    this.onFileSelectedAsync = this.onFileSelectedAsync.bind(this);
+  }
+
+  onFileSelected(file) {
+    return this.$async(this.onFileSelectedAsync, file);
+  }
+
+  async onFileSelectedAsync(file) {
+    if (!file) {
+      return;
+    }
+
+    this.state.uploadInProgress = true;
+    try {
+      await this.uploadFile(file);
+    } finally {
+      this.state.uploadInProgress = false;
+    }
+  }
+}
diff --git a/app/agent/components/file-uploader/index.js b/app/agent/components/file-uploader/index.js
new file mode 100644
index 000000000..6b0b37fc7
--- /dev/null
+++ b/app/agent/components/file-uploader/index.js
@@ -0,0 +1,10 @@
+import angular from 'angular';
+import { FileUploaderController } from './fileUploaderController';
+
+angular.module('portainer.agent').component('fileUploader', {
+  templateUrl: './fileUploader.html',
+  controller: FileUploaderController,
+  bindings: {
+    uploadFile: '<onFileSelected',
+  },
+});