Fix copy/paste bug in EditingService.cs that corrupted DataGrid state when inline editing row items that send PropertyChanged notifications.

Zofware · Zofware · commit 2d310d357b2e · 2022-08-27T11:42:49.000-07:00
diff --git a/Controls/Grid/Grid.UWP/View/Services/Editing/EditingService.cs b/Controls/Grid/Grid.UWP/View/Services/Editing/EditingService.cs
@@ -1,4 +1,4 @@
-﻿using System;
+using System;
 using System.Collections.Generic;
 using System.Linq;
 using System.Text;
@@ -85,7 +85,7 @@ public bool BeginEdit(ItemInfo rowInfo)
                         this.Owner.EditRowLayer.ScheduleFirstEditorForFocus();
                     }
 
-                    this.Owner.Model.CurrentDataProvider.CommitEditOperation(this.EditItem);
+                    this.Owner.Model.CurrentDataProvider.BeginEditOperation(this.EditItem);
 
                     // hide the CurrentItem decoration
                     this.Owner.visualStateService.UpdateCurrentDecoration(-1); 
@@ -189,7 +189,7 @@ public bool CancelEdit(ActionTrigger trigger)
                     this.Owner.EditRowLayer.EditorLayoutSlots.Clear();
                 }
 
-                this.Owner.Model.CurrentDataProvider.CommitEditOperation(this.EditItem);
+                this.Owner.Model.CurrentDataProvider.CancelEditOperation(this.EditItem);
             })
             {
                 Flags = UpdateFlags.AffectsContent

Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`		`-using System;`
	`1`	`+using System;`
`2`	`2`	`using System.Collections.Generic;`
`3`	`3`	`using System.Linq;`
`4`	`4`	`using System.Text;`
`@@ -85,7 +85,7 @@ public bool BeginEdit(ItemInfo rowInfo)`
`85`	`85`	`this.Owner.EditRowLayer.ScheduleFirstEditorForFocus();`
`86`	`86`	`}`
`87`	`87`
`88`		`- this.Owner.Model.CurrentDataProvider.CommitEditOperation(this.EditItem);`
	`88`	`+ this.Owner.Model.CurrentDataProvider.BeginEditOperation(this.EditItem);`
`89`	`89`
`90`	`90`	`// hide the CurrentItem decoration`
`91`	`91`	`this.Owner.visualStateService.UpdateCurrentDecoration(-1);`
`@@ -189,7 +189,7 @@ public bool CancelEdit(ActionTrigger trigger)`
`189`	`189`	`this.Owner.EditRowLayer.EditorLayoutSlots.Clear();`
`190`	`190`	`}`
`191`	`191`
`192`		`- this.Owner.Model.CurrentDataProvider.CommitEditOperation(this.EditItem);`
	`192`	`+ this.Owner.Model.CurrentDataProvider.CancelEditOperation(this.EditItem);`
`193`	`193`	`})`
`194`	`194`	`{`
`195`	`195`	`Flags = UpdateFlags.AffectsContent`