Cherry-pick: don't save file to recent files list if saveas is canceled

src/DynamoCoreWpf/ViewModels/Core/DynamoViewModel.cs

@@ -2093,10 +2093,10 @@ private void InternalSaveAs(string path, SaveContext saveContext, bool isBackup
         {
             try
             {
-                Model.Logger.Log(String.Format(Properties.Resources.SavingInProgress, path));
-                CurrentSpaceViewModel.Save(path, isBackup, Model.EngineController, saveContext);
+                Model.Logger.Log(string.Format(Properties.Resources.SavingInProgress, path));
+                var hasSaved = CurrentSpaceViewModel.Save(path, isBackup, Model.EngineController, saveContext);
 
-                if (!isBackup)
+                if (!isBackup && hasSaved)
                 {
                     AddToRecentFiles(path);
 
@@ -2168,8 +2168,10 @@ internal void SaveAs(Guid id, string path, bool isBackup = false, SaveContext sa
             try
             {
                 Model.Logger.Log(String.Format(Properties.Resources.SavingInProgress, path));
-                Workspaces.Where(w => w.Model.Guid == id).FirstOrDefault().Save(path, isBackup, Model.EngineController, saveContext);
-                if (!isBackup) AddToRecentFiles(path);
+                var hasSaved = Workspaces.FirstOrDefault(w => w.Model.Guid == id).Save(
+                    path, isBackup, Model.EngineController, saveContext);
+
+                if (!isBackup && hasSaved) AddToRecentFiles(path);
             }
             catch (Exception ex)
             {

src/DynamoCoreWpf/ViewModels/Core/WorkspaceViewModel.cs

@@ -710,7 +710,7 @@ internal JObject GetJsonRepresentation(EngineController engine = null)
         /// <param name="engine"></param>
         /// <param name="saveContext"></param>
         /// <exception cref="ArgumentNullException">Thrown when the file path is null.</exception>
-        internal void Save(string filePath, bool isBackup = false, EngineController engine = null, SaveContext saveContext = SaveContext.None)
+        internal bool Save(string filePath, bool isBackup = false, EngineController engine = null, SaveContext saveContext = SaveContext.None)
         {
             if (String.IsNullOrEmpty(filePath))
             {
@@ -748,7 +748,7 @@ internal void Save(string filePath, bool isBackup = false, EngineController engi
 
                             if (result == MessageBoxResult.Cancel)
                             {
-                                return;
+                                return false;
                             }
                         }
                     }
@@ -791,6 +791,8 @@ internal void Save(string filePath, bool isBackup = false, EngineController engi
                 throw ex;
 #pragma warning restore CA2200 // Rethrow to preserve stack details
             }
+
+            return true;
         }
         /// <summary>
         /// This function appends view block to the model json

test/DynamoCoreWpfTests/DynamoViewTests.cs

@@ -147,9 +147,9 @@ public void ElementBinding_SaveAs()
             Open(saveAsPathInTestDir);
 
             // Assert saved as file doesn't have binding data after open.
-            DynamoUtilities.PathHelper.isValidJson(filePath, out fileContents, out ex);
+            DynamoUtilities.PathHelper.isValidJson(saveAsPath, out fileContents, out ex);
             obj = DSCore.Data.ParseJSON(fileContents) as Dictionary<string, object>;
-            Assert.AreEqual(1, (obj["Bindings"] as IEnumerable<object>).Count());
+            Assert.AreEqual(0, (obj["Bindings"] as IEnumerable<object>).Count());
 
             File.Delete(filePath);
             File.Delete(saveAsPath);