Committed property changes issue

Author: Jeyasri-Murugan

DataFormMAUI/Behavior/ContactFormBehavior.cs

@@ -13,6 +13,8 @@ public class ContactFormBehavior : Behavior<ContentPage>
         /// </summary>
         private SfDataForm? dataForm;
 
+        private bool isValid;
+
         /// <summary>
         /// Holds the save button instance.
         /// </summary>
@@ -36,7 +38,7 @@ protected override void OnAttachedTo(ContentPage bindable)
                 this.saveButton.Clicked += OnSaveButtonClicked;
             }
         }
-
+     
         /// <summary>
         /// Invokes on data form item validation.
         /// </summary>
@@ -76,13 +78,13 @@ private async void OnDataFormValidateForm(object? sender, DataFormValidateFormEv
                             await App.Current.MainPage.DisplayAlert("", "Please enter the mobile number", "OK");
                         }
                     }
+                    isValid = false;
+
                 }
                 else
                 {
-                    App.Database.UpdateContactAsync((this.dataForm.BindingContext as ContactsViewModel).SelectedItem);
-
-                    await App.Current.MainPage.Navigation.PopAsync();
-                    await App.Current.MainPage.DisplayAlert("", "Contact saved", "OK");
+                    isValid = true;
+               
                 }
             }
         }
@@ -92,9 +94,15 @@ private async void OnDataFormValidateForm(object? sender, DataFormValidateFormEv
         /// </summary>
         /// <param name="sender">The button.</param>
         /// <param name="e">The event arguments.</param>
-        private void OnSaveButtonClicked(object? sender, EventArgs e)
+        private async void OnSaveButtonClicked(object? sender, EventArgs e)
         {
             this.dataForm?.Validate();
+
+            if(isValid)
+            {
+                await App.Current.MainPage.DisplayAlert("", "Contact saved", "OK");
+                await App.Current.MainPage.Navigation.PopAsync();
+            }
         }
 
         /// <inheritdoc/>

DataFormMAUI/Model/ContactFormModel.cs

@@ -39,9 +39,9 @@ public string Mobile
                     {
                         mobile = new string(value.Where(c => Char.IsLetterOrDigit(c)).ToArray());
                     }
-                    RaisePropertyChanged(nameof(Mobile));
                     if (mobile.Length == 10)
                         this.MaskedMobileText = string.Format("({0}) {1}-{2}", mobile.Substring(0, 3), mobile.Substring(3, 3), mobile.Substring(6));
+                    RaisePropertyChanged(nameof(Mobile));
                 }
             }
         }
@@ -71,7 +71,6 @@ public string MaskedMobileText
             set
             {
                 this.maskedMobileText = value;
-                RaisePropertyChanged(nameof(MaskedMobileText));
             }
         }
 

DataFormMAUI/Views/EditPage.xaml.cs

@@ -6,5 +6,16 @@ public EditPage()
         {
             InitializeComponent();
         }
+
+        protected override void OnAppearing()
+        {
+            base.OnAppearing();
+            (this.contactForm.DataObject as ContactFormModel).PropertyChanged += ContactFormBehavior_PropertyChanged;
+        }
+
+        private void ContactFormBehavior_PropertyChanged(object sender, System.ComponentModel.PropertyChangedEventArgs e)
+        {
+            App.Database.UpdateContactAsync(this.contactForm.DataObject as ContactFormModel);
+        }
     }
 }