GitClient should return user

Author: StanleyGoldman

src/GitHub.Api/Git/GitClient.cs

@@ -23,7 +23,7 @@ ITask<string> GetConfig(string key, GitConfigSource configSource,
         ITask<string> SetConfig(string key, string value, GitConfigSource configSource,
             IOutputProcessor<string> processor = null);
 
-        ITask<string[]> GetConfigUserAndEmail();
+        ITask<User> GetConfigUserAndEmail();
 
         ITask<List<GitLock>> ListLocks(bool local,
             BaseOutputListProcessor<GitLock> processor = null);
@@ -255,7 +255,7 @@ public ITask<string> SetConfig(string key, string value, GitConfigSource configS
                 .Configure(processManager);
         }
 
-        public ITask<string[]> GetConfigUserAndEmail()
+        public ITask<User> GetConfigUserAndEmail()
         {
             string username = null;
             string email = null;
@@ -273,7 +273,7 @@ public ITask<string[]> GetConfigUserAndEmail()
                 }
             })).Then(success => {
                 Logger.Trace("user.name:{1} user.email:{2}", success, username, email);
-                return new[] { username, email };
+                return new User { Name= username, Email = email };
             });
         }
 

src/GitHub.Api/Git/RepositoryManager.cs

@@ -299,18 +299,9 @@ public ITask UnlockFile(string file, bool force)
         private void LoadGitUser()
         {
             GitClient.GetConfigUserAndEmail()
-                .Then((success, strings) => {
-                    var username = strings[0];
-                    var email = strings[1];
-
-                    var user = new User {
-                        Name = username,
-                        Email = email
-                    };
-
+                .Then((success, user) => {
                     Logger.Trace("OnGitUserLoaded: {0}", user);
                     OnGitUserLoaded?.Invoke(user);
-
                 }).Start();
         }
 

src/UnityExtension/Assets/Editor/GitHub.Unity/UI/InitProjectView.cs

@@ -96,12 +96,9 @@ private void CheckForUser()
             Logger.Trace("Checking for user");
             isBusy = true;
 
-            GitClient.GetConfigUserAndEmail().FinallyInUI((success, ex, strings) => {
-                var username = strings[0];
-                var email = strings[1];
-
+            GitClient.GetConfigUserAndEmail().FinallyInUI((success, ex, user) => {
                 isBusy = false;
-                isUserDataPresent = success && !String.IsNullOrEmpty(username) && !String.IsNullOrEmpty(email);
+                isUserDataPresent = success && !String.IsNullOrEmpty(user.Name) && !String.IsNullOrEmpty(user.Email);
                 hasCompletedInitialCheck = true;
 
                 Logger.Trace("User Present: {0}", isUserDataPresent);

src/UnityExtension/Assets/Editor/GitHub.Unity/UI/UserSettingsView.cs

@@ -115,16 +115,10 @@ private void MaybeUpdateData()
                 {
                     if ((cachedUser == null || String.IsNullOrEmpty(cachedUser.Name)) && GitClient != null)
                     {
-                        GitClient.GetConfigUserAndEmail().FinallyInUI((success, ex, strings) => {
-                            var username = strings[0];
-                            var email = strings[1];
-
-                            if (success && !String.IsNullOrEmpty(username) && !String.IsNullOrEmpty(email))
+                        GitClient.GetConfigUserAndEmail().FinallyInUI((success, ex, user) => {
+                            if (success && !String.IsNullOrEmpty(user.Name) && !String.IsNullOrEmpty(user.Email))
                             {
-                                cachedUser = new User {
-                                    Name = username,
-                                    Email = email
-                                };
+                                cachedUser = user;
 
                                 userDataHasChanged = true;
                                 Redraw();