Merge pull request #1748 from Hosch250/Issue157

Discover SC projects by project ID rather than name

Author: retailcoder

RetailCoder.VBE/UI/SourceControl/SourceControlViewViewModel.cs

@@ -25,7 +25,7 @@ public enum SourceControlTab
         UnsyncedCommits,
         Settings
     }
-    
+
     public sealed class SourceControlViewViewModel : ViewModelBase, IDisposable
     {
         private readonly VBE _vbe;
@@ -368,11 +368,11 @@ private void AddOrUpdateLocalPathConfig(Repository repo)
                 var existing = _config.Repositories.Single(repository => repository.LocalLocation == repo.LocalLocation);
                 if (string.IsNullOrEmpty(repo.RemoteLocation) && !string.IsNullOrEmpty(existing.RemoteLocation))
                 {
-                    // config already has remote location and correct repository name - nothing to update
+                    // config already has remote location and correct repository id - nothing to update
                     return;
                 }
 
-                existing.Name = repo.Name;
+                existing.Id = repo.Id;
                 existing.RemoteLocation = repo.RemoteLocation;
 
                 _configService.Save(_config);
@@ -421,7 +421,7 @@ private void CloneRepo()
                 Provider = _providerFactory.CreateProvider(_vbe.ActiveVBProject, repo, _wrapperFactory);
                 AddOrUpdateLocalPathConfig(new Repository
                 {
-                    Name = _vbe.ActiveVBProject.Name,
+                    Id = _vbe.ActiveVBProject.HelpFile,
                     LocalLocation = repo.LocalLocation,
                     RemoteLocation = repo.RemoteLocation
                 });
@@ -462,7 +462,7 @@ private void OpenRepoAssignedToProject()
             {
                 OnOpenRepoStarted();
                 Provider = _providerFactory.CreateProvider(_vbe.ActiveVBProject,
-                    _config.Repositories.First(repo => repo.Name == _vbe.ActiveVBProject.Name), _wrapperFactory);
+                    _config.Repositories.First(repo => repo.Id == _vbe.ActiveVBProject.HelpFile), _wrapperFactory);
                 Status = RubberduckUI.Online;
             }
             catch (SourceControlException ex)
@@ -496,7 +496,7 @@ private bool ValidRepoExists()
                 return false;
             }
 
-            var possibleRepos = _config.Repositories.Where(repo => repo.Name == _vbe.ActiveVBProject.Name);
+            var possibleRepos = _config.Repositories.Where(repo => repo.Id == _vbe.ActiveVBProject.HelpFile);
 
             var possibleCount = possibleRepos.Count();
 
@@ -544,7 +544,7 @@ public ICommand CloneRepoCommand
         {
             get { return _cloneRepoCommand; }
         }
-        
+
         private readonly ICommand _showFilePickerCommand;
         public ICommand ShowFilePickerCommand
         {

Rubberduck.SourceControl/IRepository.cs

@@ -8,7 +8,7 @@ namespace Rubberduck.SourceControl
     public interface IRepository
     {
         [DispId(0)]
-        string Name { get; }
+        string Id { get; }
 
         [DispId(1)]
         [Description("FilePath of local repository.")]

Rubberduck.SourceControl/Repository.cs

@@ -8,16 +8,16 @@ namespace Rubberduck.SourceControl
     [ClassInterface(ClassInterfaceType.None)]
     public class Repository : IRepository
     {
-        public string Name { get; set; }
+        public string Id { get; set; }
         public string LocalLocation { get; set; }
         public string RemoteLocation { get;  set; }
 
         //parameterless constructor for serialization
         public Repository() { }
 
-        public Repository(string name, string localDirectory, string remotePathOrUrl)
+        public Repository(string id, string localDirectory, string remotePathOrUrl)
         {
-            Name = name;
+            Id = id;
             LocalLocation = localDirectory;
             RemoteLocation = remotePathOrUrl;
         }