From 7fd789380bd8b55215295e5516552a6f7cb9ba71 Mon Sep 17 00:00:00 2001
From: NaBian <836904362@qq.com>
Date: Sat, 20 Aug 2022 23:51:14 +0800
Subject: [PATCH] enhance: close #1064

---
 .../UserControl/Controls/PinBoxDemoCtl.xaml   |  8 +-
 .../Controls/Input/PinBox.cs                  | 78 ++++++++++++++++++-
 2 files changed, 79 insertions(+), 7 deletions(-)

diff --git a/src/Shared/HandyControlDemo_Shared/UserControl/Controls/PinBoxDemoCtl.xaml b/src/Shared/HandyControlDemo_Shared/UserControl/Controls/PinBoxDemoCtl.xaml
index 728c99a3..7cc2a38e 100644
--- a/src/Shared/HandyControlDemo_Shared/UserControl/Controls/PinBoxDemoCtl.xaml
+++ b/src/Shared/HandyControlDemo_Shared/UserControl/Controls/PinBoxDemoCtl.xaml
@@ -4,9 +4,11 @@
              xmlns:hc="https://handyorg.github.io/handycontrol"
              Background="{DynamicResource RegionBrush}">
     <hc:TransitioningContentControl>
-        <StackPanel Margin="32" VerticalAlignment="Center" hc:PinBox.Completed="PinBox_OnCompleted">
+        <hc:UniformSpacingPanel Spacing="16" Orientation="Vertical" Margin="32" VerticalAlignment="Center" hc:PinBox.Completed="PinBox_OnCompleted">
             <hc:PinBox Length="4" Password="1234"/>
-            <hc:PinBox Length="6" Password="123456" Margin="0,16,0,0" PasswordChar="❤"/>
-        </StackPanel>
+            <hc:PinBox Length="4" Password="1234" Name="PinBoxDemo" IsSafeEnabled="False"/>
+            <TextBox Text="{Binding UnsafePassword,ElementName=PinBoxDemo,UpdateSourceTrigger=PropertyChanged}"/>
+            <hc:PinBox Length="6" Password="123456" PasswordChar="❤"/>
+        </hc:UniformSpacingPanel>
     </hc:TransitioningContentControl>
 </UserControl>
diff --git a/src/Shared/HandyControl_Shared/Controls/Input/PinBox.cs b/src/Shared/HandyControl_Shared/Controls/Input/PinBox.cs
index b22795ff..0cd7810d 100644
--- a/src/Shared/HandyControl_Shared/Controls/Input/PinBox.cs
+++ b/src/Shared/HandyControl_Shared/Controls/Input/PinBox.cs
@@ -88,6 +88,11 @@ public class PinBox : Control
 
         if (e.OriginalSource is System.Windows.Controls.PasswordBox passwordBox)
         {
+            if (!IsSafeEnabled)
+            {
+                SetCurrentValue(UnsafePasswordProperty, Password);
+            }
+
             if (passwordBox.Password.Length > 0)
             {
                 if (++_inputIndex >= Length)
@@ -192,7 +197,9 @@ public class PinBox : Control
     {
         get
         {
-            return string.Join("", _panel.Children.OfType<System.Windows.Controls.PasswordBox>().Select(item => item.Password));
+            return _panel == null
+                ? string.Empty
+                : string.Join(string.Empty, _panel.Children.OfType<System.Windows.Controls.PasswordBox>().Select(item => item.Password));
         }
         set
         {
@@ -212,14 +219,31 @@ public class PinBox : Control
                 return;
             }
 
-            var length = Length;
-            if (string.IsNullOrEmpty(value) || value.Length != length || value.Length != _panel.Children.Count)
+            _isInternalAction = true;
+
+            if (string.IsNullOrEmpty(value))
             {
                 _panel.Children.OfType<System.Windows.Controls.PasswordBox>().Do(item => item.Clear());
             }
             else
             {
-                _panel.Children.OfType<System.Windows.Controls.PasswordBox>().DoWithIndex((item, index) => item.Password = value[index].ToString());
+                _panel.Children
+                    .OfType<System.Windows.Controls.PasswordBox>()
+                    .Take(Math.Min(Length, value.Length))
+                    .DoWithIndex((item, index) => item.Password = value[index].ToString());
+
+                _panel.Children
+                    .OfType<System.Windows.Controls.PasswordBox>()
+                    .Skip(value.Length)
+                    .Take(Length - value.Length)
+                    .Do(item => item.Clear());
+            }
+
+            _isInternalAction = false;
+
+            if (!IsSafeEnabled)
+            {
+                SetCurrentValue(UnsafePasswordProperty, Password);
             }
         }
     }
@@ -321,6 +345,50 @@ public class PinBox : Control
         set => SetValue(CaretBrushProperty, value);
     }
 
+    public static readonly DependencyProperty IsSafeEnabledProperty = PasswordBox.IsSafeEnabledProperty
+        .AddOwner(typeof(PinBox), new FrameworkPropertyMetadata(ValueBoxes.TrueBox, OnIsSafeEnabledChanged));
+
+    private static void OnIsSafeEnabledChanged(DependencyObject d, DependencyPropertyChangedEventArgs e)
+    {
+        var p = (PinBox) d;
+        p.OnIsSafeEnabledChanged((bool)e.NewValue);
+    }
+
+    private void OnIsSafeEnabledChanged(bool newValue)
+    {
+        if (_panel == null)
+        {
+            return;
+        }
+
+        SetCurrentValue(UnsafePasswordProperty, !newValue ? Password : string.Empty);
+    }
+
+    public bool IsSafeEnabled
+    {
+        get => (bool) GetValue(IsSafeEnabledProperty);
+        set => SetValue(IsSafeEnabledProperty, ValueBoxes.BooleanBox(value));
+    }
+
+    public static readonly DependencyProperty UnsafePasswordProperty = PasswordBox.UnsafePasswordProperty
+        .AddOwner(typeof(PinBox), new FrameworkPropertyMetadata(default(string),
+            FrameworkPropertyMetadataOptions.BindsTwoWayByDefault, OnUnsafePasswordChanged));
+
+    private static void OnUnsafePasswordChanged(DependencyObject d, DependencyPropertyChangedEventArgs e)
+    {
+        var p = (PinBox) d;
+        if (!p.IsSafeEnabled)
+        {
+            p.Password = e.NewValue != null ? e.NewValue.ToString() : string.Empty;
+        }
+    }
+
+    public string UnsafePassword
+    {
+        get => (string) GetValue(UnsafePasswordProperty);
+        set => SetValue(UnsafePasswordProperty, value);
+    }
+
     public static readonly RoutedEvent CompletedEvent =
         EventManager.RegisterRoutedEvent("Completed", RoutingStrategy.Bubble,
             typeof(RoutedEventHandler), typeof(PinBox));
@@ -405,6 +473,8 @@ public class PinBox : Control
 
                 _passwordList.Clear();
             }
+
+            OnIsSafeEnabledChanged(IsSafeEnabled);
         }
     }
 }