Add support for contributors in GitHubIssueTracker

Author: GeertvanHorrik

src/GitTools.IssueTrackers/Context/Interfaces/IIssueTrackerContext.cs

@@ -6,6 +6,8 @@ public interface IIssueTrackerContext : IDisposable
     {
         string Server { get; set; }
 
+        string DiffUrlFormat { get; set; }
+
         string ProjectId { get; set; }
 
         IAuthenticationContext Authentication { get; }

src/GitTools.IssueTrackers/Context/IssueTrackerContext.cs

@@ -9,6 +9,8 @@ public IssueTrackerContext()
 
         public string Server { get; set; }
 
+        public string DiffUrlFormat { get; set; }
+
         public string ProjectId { get; set; }
 
         public IAuthenticationContext Authentication { get; private set; }

src/GitTools.IssueTrackers/IssueTrackers/GitHub/GitHubIssueTracker.cs

@@ -14,6 +14,8 @@ public class GitHubIssueTracker : IssueTrackerBase
     {
         private static readonly ILog Log = LogProvider.GetCurrentClassLogger();
 
+        private readonly Dictionary<string, User> _userCache = new Dictionary<string, User>();
+
         public GitHubIssueTracker(IIssueTrackerContext issueTrackerContext)
             : base(issueTrackerContext)
         {
@@ -47,23 +49,22 @@ public override async Task<IEnumerable<Issue>> GetIssuesAsync(IssueTrackerFilter
 
             var readOnlyList = forRepository.Where(i => i.ClosedAt > filter.Since);
 
-            //var userCache = new Dictionary<string, User>();
-            //Func<User, string> getUserName = u =>
-            //{
-            //    var login = u.Login;
-            //    if (!userCache.ContainsKey(login))
-            //    {
-            //        userCache.Add(login, string.IsNullOrEmpty(u.Name) ? gitHubClient.User.Get(login).Result : u);
-            //    }
+            Func<User, string> getUserName = u =>
+            {
+                var login = u.Login;
+                if (!_userCache.ContainsKey(login))
+                {
+                    _userCache.Add(login, string.IsNullOrEmpty(u.Name) ? gitHubClient.User.Get(login).Result : u);
+                }
 
-            //    var user = userCache[login];
-            //    if (user != null)
-            //    {
-            //        return user.Name;
-            //    }
+                var user = _userCache[login];
+                if (user != null)
+                {
+                    return user.Name;
+                }
 
-            //    return null;
-            //};
+                return null;
+            };
 
             return readOnlyList.Select(i => new Issue("#" + i.Number.ToString(CultureInfo.InvariantCulture))
             {
@@ -72,10 +73,10 @@ public override async Task<IEnumerable<Issue>> GetIssuesAsync(IssueTrackerFilter
                 Title = i.Title,
                 IssueType = i.PullRequest == null ? IssueType.Issue : IssueType.PullRequest,
                 Labels = i.Labels.Select(l => l.Name).ToArray(),
-                //Contributors = i.PullRequest == null ? new GitReleaseNotes.Contributor[0] : new[]
-                //{
-                //    new GitReleaseNotes.Contributor(getUserName(i.User), i.User.Login, i.User.HtmlUrl)
-                //}
+                Contributors = i.PullRequest == null ? new List<GitTools.IssueTrackers.Contributor>() : new List<GitTools.IssueTrackers.Contributor>
+                {
+                    new GitTools.IssueTrackers.Contributor(getUserName(i.User), i.User.Login, i.User.HtmlUrl)
+                }
             });
         }
 

src/GitTools.IssueTrackers/Models/Issue.cs

@@ -10,6 +10,7 @@ public Issue(string id)
             Id = id;
 
             FixVersions = new List<Version>();
+            Contributors = new List<Contributor>();
         }
 
         public string Id { get; private set; }
@@ -32,6 +33,6 @@ public Issue(string id)
 
         //public Uri HtmlUrl { get; set; }
 
-        //public Contributor[] Contributors { get; set; }
+        public List<Contributor> Contributors { get; set; }
     }
 }