From 96f283e2c6e83303eb0aa225b507b3bb623f8edf Mon Sep 17 00:00:00 2001
From: Chrystal Maria Griffiths <hello@chrystalgriffiths.com>
Date: Fri, 15 Feb 2019 13:16:21 +0000
Subject: [PATCH] Merge pull request #1477 from
 sharelatex/as-confirm-quick-action-archive-leave

Confirm archive or leave when clicking quick action buttons

GitOrigin-RevId: 95d728f22284bab17887a5ac7bc7461d501db7a2
---
 .../src/main/project-list/project-list.js     | 62 +++++++++++--------
 1 file changed, 37 insertions(+), 25 deletions(-)

diff --git a/services/web/public/src/main/project-list/project-list.js b/services/web/public/src/main/project-list/project-list.js
index fb92c72776..a781afdadc 100644
--- a/services/web/public/src/main/project-list/project-list.js
+++ b/services/web/public/src/main/project-list/project-list.js
@@ -504,16 +504,22 @@ define(['base'], function(App) {
       })
     }
 
-    $scope.openArchiveProjectsModal = function() {
-      const modalInstance = $modal.open({
+    $scope.createArchiveProjectsModal = function(projects) {
+      return $modal.open({
         templateUrl: 'deleteProjectsModalTemplate',
         controller: 'DeleteProjectsModalController',
         resolve: {
           projects() {
-            return $scope.getSelectedProjects()
+            return projects
           }
         }
       })
+    }
+
+    $scope.openArchiveProjectsModal = function() {
+      const modalInstance = $scope.createArchiveProjectsModal(
+        $scope.getSelectedProjects()
+      )
       event_tracking.send(
         'project-list-page-interaction',
         'project action',
@@ -529,29 +535,32 @@ define(['base'], function(App) {
 
     $scope.archiveOrLeaveProjects = function(projects) {
       for (let project of projects) {
-        if (project.accessLevel === 'owner') {
-          project.archived = true
-          queuedHttp({
-            method: 'DELETE',
-            url: `/project/${project.id}`,
-            headers: {
-              'X-CSRF-Token': window.csrfToken
-            }
-          })
-        } else {
-          $scope._removeProjectFromList(project)
-
-          queuedHttp({
-            method: 'POST',
-            url: `/project/${project.id}/leave`,
-            headers: {
-              'X-CSRF-Token': window.csrfToken
-            }
-          })
-        }
+        $scope.archiveOrLeaveProject(project)
       }
+      $scope.updateVisibleProjects()
+    }
 
-      return $scope.updateVisibleProjects()
+    $scope.archiveOrLeaveProject = function(project) {
+      if (project.accessLevel === 'owner') {
+        project.archived = true
+        queuedHttp({
+          method: 'DELETE',
+          url: `/project/${project.id}`,
+          headers: {
+            'X-CSRF-Token': window.csrfToken
+          }
+        })
+      } else {
+        $scope._removeProjectFromList(project)
+
+        queuedHttp({
+          method: 'POST',
+          url: `/project/${project.id}/leave`,
+          headers: {
+            'X-CSRF-Token': window.csrfToken
+          }
+        })
+      }
     }
 
     $scope.openDeleteProjectsModal = function() {
@@ -750,7 +759,10 @@ define(['base'], function(App) {
 
     $scope.archiveOrLeave = function(e) {
       e.stopPropagation()
-      return $scope.archiveOrLeaveProjects([$scope.project])
+      $scope.createArchiveProjectsModal([$scope.project]).result.then(() => {
+        $scope.archiveOrLeaveProject($scope.project)
+        $scope.updateVisibleProjects()
+      })
     }
 
     $scope.restore = function(e) {