Return null on failed copy (#2830)

gave92 · web-flow · commit 92ea6b649559 · 2021-01-03T10:32:56.000-05:00
diff --git a/Files/Filesystem/FilesystemOperations/FilesystemOperations.cs b/Files/Filesystem/FilesystemOperations/FilesystemOperations.cs
@@ -209,6 +209,10 @@ await DialogDisplayHelper.ShowDialogAsync(
                         fsResult = fsCopyResult;
                     }
                     errorCode?.Report(fsResult.ErrorCode);
+                    if (!fsResult)
+                    {
+                        return null;
+                    }
                 }
             }
             else if (source.ItemType == FilesystemItemType.File)
@@ -267,6 +271,10 @@ await DialogDisplayHelper.ShowDialogAsync(
                     }
                 }
                 errorCode?.Report(fsResult.ErrorCode);
+                if (!fsResult)
+                {
+                    return null;
+                }
             }
 
             if (Path.GetDirectoryName(destination) == associatedInstance.FilesystemViewModel.WorkingDirectory)

Original file line number	Diff line number	Diff line change
`@@ -209,6 +209,10 @@ await DialogDisplayHelper.ShowDialogAsync(`
`209`	`209`	`fsResult = fsCopyResult;`
`210`	`210`	`}`
`211`	`211`	`errorCode?.Report(fsResult.ErrorCode);`
	`212`	`+ if (!fsResult)`
	`213`	`+ {`
	`214`	`+ return null;`
	`215`	`+ }`
`212`	`216`	`}`
`213`	`217`	`}`
`214`	`218`	`else if (source.ItemType == FilesystemItemType.File)`
`@@ -267,6 +271,10 @@ await DialogDisplayHelper.ShowDialogAsync(`
`267`	`271`	`}`
`268`	`272`	`}`
`269`	`273`	`errorCode?.Report(fsResult.ErrorCode);`
	`274`	`+ if (!fsResult)`
	`275`	`+ {`
	`276`	`+ return null;`
	`277`	`+ }`
`270`	`278`	`}`
`271`	`279`
`272`	`280`	`if (Path.GetDirectoryName(destination) == associatedInstance.FilesystemViewModel.WorkingDirectory)`