fix: replace CommandRecorder with HistoryManager in curve-related view models

Author: yuto-trd

src/Beutl/ViewModels/Tools/ColorGradingTabViewModel.cs

@@ -1,4 +1,5 @@
 using System.Text.Json.Nodes;
+using Beutl.Editor;
 using Beutl.Engine;
 using Beutl.Graphics.Effects;
 using Beutl.Media;
@@ -130,8 +131,8 @@ public void Visit(IPropertyEditorContext context)
         if (serviceType == typeof(EditViewModel))
             return _editViewModel;
 
-        if (serviceType == typeof(CommandRecorder))
-            return _editViewModel.CommandRecorder;
+        if (serviceType == typeof(HistoryManager))
+            return _editViewModel.HistoryManager;
 
         if (serviceType == typeof(Element))
             return _currentElement;

src/Beutl/ViewModels/Tools/CurvePresenterViewModel.cs

@@ -1,5 +1,6 @@
 using System.Collections.ObjectModel;
 using System.Collections.Specialized;
+using Beutl.Editor;
 using Beutl.Engine;
 using Beutl.Graphics;
 using Beutl.Graphics.Effects;
@@ -10,17 +11,16 @@ public sealed class CurvePresenterViewModel : IDisposable
 {
     private readonly Curves _effect;
     private readonly IProperty<CurveMap> _property;
-    private readonly CommandRecorder _commandRecorder;
-    private CurveMap? _oldValue;
+    private readonly HistoryManager _history;
     private bool _isUpdating;
 
     public CurvePresenterViewModel(string header, Curves effect, IProperty<CurveMap> property,
-        CommandRecorder commandRecorder)
+        HistoryManager history)
     {
         Header = header;
         _effect = effect;
         _property = property;
-        _commandRecorder = commandRecorder;
+        _history = history;
 
         Points = new ObservableCollection<CurveControlPoint>(_property.CurrentValue.Points);
         Points.CollectionChanged += OnCollectionChanged;
@@ -33,23 +33,11 @@ public CurvePresenterViewModel(string header, Curves effect, IProperty<CurveMap>
 
     public void BeginEdit()
     {
-        _oldValue = _property.CurrentValue;
     }
 
     public void EndEdit()
     {
-        if (_oldValue == null) return;
-
-        CurveMap newValue = _property.CurrentValue;
-        CurveMap oldValue = _oldValue;
-        _oldValue = null;
-
-        if (oldValue != newValue)
-        {
-            RecordableCommands.Edit(_property, newValue, oldValue)
-                .WithStoables([_effect])
-                .DoAndRecord(_commandRecorder);
-        }
+        _history.Commit(CommandNames.EditProperty);
     }
 
     // Viewからの変更を反映

src/Beutl/ViewModels/Tools/CurvesTabViewModel.cs

@@ -7,6 +7,7 @@
 using Beutl.ProjectSystem;
 using Beutl.Services.PrimitiveImpls;
 using Avalonia.Media.Imaging;
+using Beutl.Editor;
 using Reactive.Bindings;
 using Reactive.Bindings.Extensions;
 
@@ -276,8 +277,8 @@ public void WriteToJson(JsonObject json)
         if (serviceType == typeof(EditViewModel))
             return _editViewModel;
 
-        if (serviceType == typeof(CommandRecorder))
-            return _editViewModel.CommandRecorder;
+        if (serviceType == typeof(HistoryManager))
+            return _editViewModel.HistoryManager;
 
         if (serviceType == typeof(Element))
             return _currentElement;
@@ -362,7 +363,7 @@ private void SetEditors(Curves? effect)
 
     private CurvePresenterViewModel CreateCurve(IProperty<CurveMap> property)
     {
-        var vm = new CurvePresenterViewModel(property.Name, _currentEffect!, property, _editViewModel.CommandRecorder);
+        var vm = new CurvePresenterViewModel(property.Name, _currentEffect!, property, _editViewModel.HistoryManager);
         _editorContexts.Add(vm);
         return vm;
     }