[Input] NotifyFieldChanged is called twice for all FluentInputBase de…

…rived components (#1846)
microsoft · Apr 12, 2024 · b57322f · b57322f
1 parent 1f6ff7d
commit b57322f
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/examples/Demo/Shared/Microsoft.FluentUI.AspNetCore.Components.xml b/examples/Demo/Shared/Microsoft.FluentUI.AspNetCore.Components.xml
@@ -6288,7 +6288,7 @@
             <summary>
             Optionally supplies a template for rendering the pagination summary.
             The following values can be included:
-            {your State parameter name}.CurrentPageIndex (zero-basedd, so +1 for the current page number)
+            {your State parameter name}.CurrentPageIndex (zero-based, so +1 for the current page number)
             {your State parameter name}.LastPageIndex (zero-based, so +1 for the total number of pages)
             </summary>
         </member>

diff --git a/src/Core/Components/Base/FluentInputBaseHandlers.cs b/src/Core/Components/Base/FluentInputBaseHandlers.cs
@@ -27,11 +27,13 @@ public partial class FluentInputBase<TValue>
     /// <returns></returns>
     protected virtual async Task ChangeHandlerAsync(ChangeEventArgs e)
     {
-        var isValid = TryParseValueFromString(e.Value?.ToString(), out var result, out var validationErrorMessage);
+        var _notifyCalled = false;
+        var isValid = TryParseValueFromString(e.Value?.ToString(), out TValue? result, out var validationErrorMessage);
 
         if (isValid)
         {
             await SetCurrentValueAsync(result ?? default);
+            _notifyCalled = true;
         }
         else
         {
@@ -43,7 +45,7 @@ protected virtual async Task ChangeHandlerAsync(ChangeEventArgs e)
                 _parsingValidationMessages.Add(FieldIdentifier, validationErrorMessage ?? "Unknown parsing error");
             }
         }
-        if (FieldBound)
+        if (FieldBound && !_notifyCalled)
         {
             CascadedEditContext?.NotifyFieldChanged(FieldIdentifier);
         }