Don't append name of repository to clone path

We now specify the path to clone to rather than the parent directory.

Author: jcansdale

src/GitHub.App/Services/RepositoryCloneService.cs

@@ -105,30 +105,26 @@ public async Task<ViewerRepositoriesModel> ReadViewerRepositories(HostAddress ad
         /// <inheritdoc/>
         public async Task CloneRepository(
             string cloneUrl,
-            string repositoryName,
             string repositoryPath,
             object progress = null)
         {
             Guard.ArgumentNotEmptyString(cloneUrl, nameof(cloneUrl));
-            Guard.ArgumentNotEmptyString(repositoryName, nameof(repositoryName));
             Guard.ArgumentNotEmptyString(repositoryPath, nameof(repositoryPath));
 
-            string path = Path.Combine(repositoryPath, repositoryName);
-
             // Switch to a thread pool thread for IO then back to the main thread to call
             // vsGitServices.Clone() as this must be called on the main thread.
             await ThreadingHelper.SwitchToPoolThreadAsync();
-            operatingSystem.Directory.CreateDirectory(path);
+            operatingSystem.Directory.CreateDirectory(repositoryPath);
             await ThreadingHelper.SwitchToMainThreadAsync();
 
             try
             {
-                await vsGitServices.Clone(cloneUrl, path, true, progress);
+                await vsGitServices.Clone(cloneUrl, repositoryPath, true, progress);
                 await usageTracker.IncrementCounter(x => x.NumberOfClones);
             }
             catch (Exception ex)
             {
-                log.Error(ex, "Could not clone {CloneUrl} to {Path}", cloneUrl, path);
+                log.Error(ex, "Could not clone {CloneUrl} to {Path}", cloneUrl, repositoryPath);
                 throw;
             }
         }

src/GitHub.Exports.Reactive/Services/IRepositoryCloneService.cs

@@ -21,7 +21,6 @@ public interface IRepositoryCloneService
         /// Clones the specificed repository into the specified directory.
         /// </summary>
         /// <param name="cloneUrl">The url of the repository to clone.</param>
-        /// <param name="repositoryName">The name of the repository to clone.</param>
         /// <param name="repositoryPath">The directory that will contain the repository directory.</param>
         /// <param name="progress">
         /// An object through which to report progress. This must be of type
@@ -31,7 +30,6 @@ public interface IRepositoryCloneService
         /// <returns></returns>
         Task CloneRepository(
             string cloneUrl,
-            string repositoryName,
             string repositoryPath,
             object progress = null);
 

src/GitHub.StartPage/StartPagePackage.cs

@@ -132,7 +132,8 @@ async Task<CloneDialogResult> ShowCloneDialog(
 
                 if (basePath != null)
                 {
-                    result = new CloneDialogResult(basePath, repository);
+                    var path = Path.Combine(basePath, repository.Name);
+                    result = new CloneDialogResult(path, repository);
                 }
             }
 
@@ -142,7 +143,6 @@ async Task<CloneDialogResult> ShowCloneDialog(
                 {
                     await cloneService.CloneRepository(
                         result.Repository.CloneUrl,
-                        result.Repository.Name,
                         result.Path,
                         progress);
 

src/GitHub.TeamFoundation.14/Connect/GitHubConnectSection.cs

@@ -153,7 +153,6 @@ async Task DoClone()
                     var cloneService = ServiceProvider.GetService<IRepositoryCloneService>();
                     await cloneService.CloneRepository(
                         result.Repository.CloneUrl,
-                        result.Repository.Name,
                         result.Path);
 
                     usageTracker.IncrementCounter(x => x.NumberOfGitHubConnectSectionClones).Forget();