From 1a5fdc540c5c7612fe1585e52ef13f42369fe845 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=E6=9D=8E=E9=80=9A=E6=B4=B2?= <carter.li@eoitek.com>
Date: Mon, 6 Sep 2021 19:58:41 +0800
Subject: [PATCH] feature<Preference>: query git config after selecting git
 path

---
 src/Models/Preference.cs     |  2 +-
 src/Views/Preference.xaml    |  3 +++
 src/Views/Preference.xaml.cs | 21 +++++++++++++++------
 3 files changed, 19 insertions(+), 7 deletions(-)

diff --git a/src/Models/Preference.cs b/src/Models/Preference.cs
index 04362cc7..3c1c6837 100644
--- a/src/Models/Preference.cs
+++ b/src/Models/Preference.cs
@@ -194,7 +194,7 @@ namespace SourceGit.Models {
         }
 
         /// <summary>
-        ///     检测配置是否
+        ///     检测配置是否正常
         /// </summary>
         [JsonIgnore]
         public bool IsReady {
diff --git a/src/Views/Preference.xaml b/src/Views/Preference.xaml
index 92c24416..6b661f50 100644
--- a/src/Views/Preference.xaml
+++ b/src/Views/Preference.xaml
@@ -265,6 +265,7 @@
                 Margin="0,0,8,0"/>
             <controls:TextEdit
                 Grid.Row="14" Grid.Column="1"
+                x:Name="editGitUser"
                 Height="24"
                 Text="{Binding ElementName=me, Path=User, Mode=TwoWay}"
                 Placeholder="{DynamicResource Text.Preference.Git.User.Placeholder}"/>
@@ -277,6 +278,7 @@
                 Margin="0,0,8,0"/>
             <controls:TextEdit
                 Grid.Row="15" Grid.Column="1"
+                x:Name="editGitEmail"
                 Height="24"
                 Text="{Binding ElementName=me, Path=Email, Mode=TwoWay}"
                 Placeholder="{DynamicResource Text.Preference.Git.Email.Placeholder}"/>
@@ -289,6 +291,7 @@
                 Margin="0,0,8,0"/>
             <ComboBox
                 Grid.Row="16" Grid.Column="1"
+                x:Name="editGitCrlf"
                 Height="24"
                 ItemsSource="{Binding Source={x:Static models:CRLFOption.Supported}}"
                 SelectedValuePath="Value"
diff --git a/src/Views/Preference.xaml.cs b/src/Views/Preference.xaml.cs
index bc5b4d4c..6580e377 100644
--- a/src/Views/Preference.xaml.cs
+++ b/src/Views/Preference.xaml.cs
@@ -24,12 +24,7 @@ namespace SourceGit.Views {
         public bool EnableWindowsTerminal { get; set; } = PathFindOnPath(new StringBuilder("wt.exe"), null);
 
         public Preference() {
-            if (Models.Preference.Instance.IsReady) {
-                User = new Commands.Config().Get("user.name");
-                Email = new Commands.Config().Get("user.email");
-                CRLF = new Commands.Config().Get("core.autocrlf");
-                if (string.IsNullOrEmpty(CRLF)) CRLF = "false";
-            } else {
+            if (!UpdateGitInfoIfReady()) {
                 User = "";
                 Email = "";
                 CRLF = "false";
@@ -42,6 +37,15 @@ namespace SourceGit.Views {
             InitializeComponent();
         }
 
+        private bool UpdateGitInfoIfReady() {
+            if (!Models.Preference.Instance.IsReady) return false;
+            User = new Commands.Config().Get("user.name");
+            Email = new Commands.Config().Get("user.email");
+            CRLF = new Commands.Config().Get("core.autocrlf");
+            if (string.IsNullOrEmpty(CRLF)) CRLF = "false";
+            return true;
+        }
+
         #region EVENTS
         private void LocaleChanged(object sender, SelectionChangedEventArgs e) {
             Models.Locale.Change();
@@ -65,6 +69,11 @@ namespace SourceGit.Views {
             if (dialog.ShowDialog() == true) {
                 Models.Preference.Instance.Git.Path = dialog.FileName;
                 editGitPath?.GetBindingExpression(TextBox.TextProperty).UpdateTarget();
+                if (UpdateGitInfoIfReady()) {
+                    editGitUser?.GetBindingExpression(TextBox.TextProperty).UpdateTarget();
+                    editGitEmail?.GetBindingExpression(TextBox.TextProperty).UpdateTarget();
+                    editGitCrlf?.GetBindingExpression(ComboBox.SelectedValueProperty).UpdateTarget();
+                }
             }
         }