Add logging of which commands are run

Author: tznind

Showcase/Menu.Designer.cs

@@ -46,7 +46,7 @@ public AddViewOperation(Design design)
     }
 
     /// <inheritdoc/>
-    public override void Redo()
+    protected override void RedoImpl()
     {
         if (this.add == null)
         {
@@ -59,7 +59,7 @@ public override void Redo()
     }
 
     /// <inheritdoc/>
-    public override void Undo()
+    protected override void UndoImpl()
     {
         if (this.add == null)
         {

src/Operations/AddViewOperation.cs

@@ -38,7 +38,7 @@ public CompositeOperation(params Operation[] operations)
     public IReadOnlyCollection<Operation> Operations => new ReadOnlyCollection<Operation>(this.operations);
 
     /// <inheritdoc/>
-    public override void Redo()
+    protected override void RedoImpl()
     {
         foreach (var op in this.operations)
         {
@@ -47,7 +47,7 @@ public override void Redo()
     }
 
     /// <inheritdoc/>
-    public override void Undo()
+    protected override void UndoImpl()
     {
         foreach (var op in this.operations)
         {

src/Operations/CompositeOperation.cs

@@ -57,7 +57,7 @@ public override string ToString()
     /// Throws <see cref="NotSupportedException"/> as you cannot undo a copy.
     /// </summary>
     /// <exception cref="NotSupportedException">Thrown if method run.</exception>
-    public override void Undo()
+    protected override void UndoImpl()
     {
         throw new NotSupportedException();
     }
@@ -66,7 +66,7 @@ public override void Undo()
     /// Throws <see cref="NotSupportedException"/> as you cannot redo a copy.
     /// </summary>
     /// <exception cref="NotSupportedException">Thrown if method run.</exception>
-    public override void Redo()
+    protected override void RedoImpl()
     {
         throw new NotSupportedException();
     }

src/Operations/CopyOperation.cs

@@ -36,13 +36,13 @@ public DeleteColorSchemeOperation(Design design, NamedColorScheme toDelete)
     public NamedColorScheme ToDelete { get; }
 
     /// <inheritdoc/>
-    public override void Redo()
+    protected override void RedoImpl()
     {
         this.Do();
     }
 
     /// <inheritdoc/>
-    public override void Undo()
+    protected override void UndoImpl()
     {
         foreach (var u in this.users)
         {

src/Operations/DeleteColorSchemeOperation.cs

@@ -42,13 +42,13 @@ public DeleteViewOperation(params Design[] delete)
     }
 
     /// <inheritdoc/>
-    public override void Redo()
+    protected override void RedoImpl()
     {
         this.Do();
     }
 
     /// <inheritdoc/>
-    public override void Undo()
+    protected override void UndoImpl()
     {
         for (int i = 0; i < this.delete.Length; i++)
         {

src/Operations/DeleteViewOperation.cs

@@ -140,7 +140,7 @@ public View? DropInto
     /// <summary>
     /// Moves all dragged views back to original positions.
     /// </summary>
-    public override void Undo()
+    protected override void UndoImpl()
     {
         foreach (var mem in this.mementos)
         {
@@ -149,7 +149,7 @@ public override void Undo()
     }
 
     /// <inheritdoc/>
-    public override void Redo()
+    protected override void RedoImpl()
     {
         this.Do();
     }

src/Operations/DragOperation.cs

@@ -41,13 +41,13 @@ public AddOperation(
     }
 
     /// <inheritdoc/>
-    public override void Undo()
+    protected override void UndoImpl()
     {
         this.Remove(this.newItem);
     }
 
     /// <inheritdoc/>
-    public override void Redo()
+    protected override void RedoImpl()
     {
         this.Add(this.newItem);
     }

src/Operations/Generics/AddOperation.cs

@@ -80,13 +80,13 @@ public override string ToString()
     }
 
     /// <inheritdoc/>
-    public override void Redo()
+    protected override void RedoImpl()
     {
         this.Do();
     }
 
     /// <inheritdoc/>
-    public override void Undo()
+    protected override void UndoImpl()
     {
         var list = this.ArrayGetter(this.View).ToList();
 

src/Operations/Generics/MoveOperation.cs

@@ -33,7 +33,7 @@ public RemoveOperation(
     }
 
     /// <inheritdoc/>
-    public override void Undo()
+    protected override void UndoImpl()
     {
         // its not there anyways
         if (this.OperateOn == null)
@@ -48,7 +48,7 @@ public override void Undo()
     }
 
     /// <inheritdoc/>
-    public override void Redo()
+    protected override void RedoImpl()
     {
         this.Do();
     }