From 0b7fe0fdb9fed7038f811952da142beffdd77395 Mon Sep 17 00:00:00 2001
From: Danny <danny@alcion.ai>
Date: Tue, 30 Aug 2022 14:50:31 -0400
Subject: [PATCH] GC: Small GC Refactor of Restore workflow (#682)

Removal of 'unused' if-clause and category variable being updated.
---
 src/internal/connector/graph_connector.go | 9 +++------
 1 file changed, 3 insertions(+), 6 deletions(-)

diff --git a/src/internal/connector/graph_connector.go b/src/internal/connector/graph_connector.go
index d8a513d79..a6a2046cd 100644
--- a/src/internal/connector/graph_connector.go
+++ b/src/internal/connector/graph_connector.go
@@ -239,11 +239,9 @@ func (gc *GraphConnector) RestoreExchangeDataCollection(
 		category := dc.FullPath()[2]
 		if _, ok := pathCounter[directory]; !ok {
 			pathCounter[directory] = true
-			if policy == control.Copy {
-				folderID, errs = exchange.GetRestoreFolder(&gc.graphService, user, category)
-				if errs != nil {
-					return errs
-				}
+			folderID, errs = exchange.GetRestoreFolder(&gc.graphService, user, category)
+			if errs != nil {
+				return errs
 			}
 		}
 		var exit bool
@@ -264,7 +262,6 @@ func (gc *GraphConnector) RestoreExchangeDataCollection(
 					errs = support.WrapAndAppend(itemData.UUID(), err, errs)
 					continue
 				}
-				category := dc.FullPath()[2]
 				err = exchange.RestoreExchangeObject(ctx, buf.Bytes(), category, policy, &gc.graphService, folderID, user)
 
 				if err != nil {