Merge branch 'fixes/git-client-should-return-user' into fixes/init-proj-view-spam

# Conflicts:
#	src/UnityExtension/Assets/Editor/GitHub.Unity/UI/UserSettingsView.cs

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

@@ -146,13 +146,10 @@ private void UpdateUserDataFromClient()
             Logger.Trace("Update user data from GitClient");
 
             GitClient.GetConfigUserAndEmail()
-                .ThenInUI((success, strings) => {
-                    var username = strings[0];
-                    var email = strings[1];
-
-                    if (success && !String.IsNullOrEmpty(username) && !String.IsNullOrEmpty(email))
+                .ThenInUI((success, user) => {
+                    if (success && !String.IsNullOrEmpty(user.Name) && !String.IsNullOrEmpty(user.Email))
                     {
-                        cachedUser = new User { Name = username, Email = email };
+                        cachedUser = user;
                         newGitName = gitName = cachedUser.Name;
                         newGitEmail = gitEmail = cachedUser.Email;
                         Redraw();