From 51164063076532b07ec42965e866de9cbb5e51e5 Mon Sep 17 00:00:00 2001
From: Tyler Schicke <tyler_schicke@brown.edu>
Date: Wed, 20 Feb 2019 22:02:56 -0500
Subject: Kind of fixed undo decorator, more work needed to get it to work with
 arrow functions

---
 src/client/util/UndoManager.ts | 57 ++++++++++++++++++++++++++----------------
 1 file changed, 35 insertions(+), 22 deletions(-)

(limited to 'src/client/util/UndoManager.ts')

diff --git a/src/client/util/UndoManager.ts b/src/client/util/UndoManager.ts
index 34910bac3..8e9e11a11 100644
--- a/src/client/util/UndoManager.ts
+++ b/src/client/util/UndoManager.ts
@@ -1,36 +1,49 @@
 import { observable, action } from "mobx";
 import { Opt } from "../../fields/Field";
 
-export function undoBatch(target: any, key: string | symbol, descriptor?: TypedPropertyDescriptor<any>): any {
-    let fn: (...args: any[]) => any;
-    let patchedFn: Opt<(...args: any[]) => any>;
-
-    if (descriptor) {
-        fn = descriptor.value;
-    }
-
-    return {
+function propertyDecorator(target: any, key: string | symbol) {
+    Object.defineProperty(target, key, {
         configurable: true,
         enumerable: false,
-        get() {
-            if (!patchedFn) {
-                patchedFn = (...args: any[]) => {
+        get: function () {
+            return 5;
+        },
+        set: function (value: any) {
+            Object.defineProperty(this, key, {
+                enumerable: false,
+                writable: true,
+                configurable: true,
+                value: function (...args: any[]) {
                     try {
-                        UndoManager.StartBatch()
-                        return fn.call(this, ...args)
+                        UndoManager.StartBatch();
+                        return value.apply(this, args);
                     } finally {
-                        UndoManager.EndBatch()
+                        UndoManager.EndBatch();
                     }
-                };
-            }
-            return patchedFn;
-        },
-        set(newFn: any) {
-            patchedFn = undefined;
-            fn = newFn;
+                }
+            })
+        }
+    })
+}
+export function undoBatch(target: any, key: string | symbol, descriptor: TypedPropertyDescriptor<any>): any {
+    if (!descriptor) {
+        propertyDecorator(target, key);
+        return;
+    }
+    const oldFunction = descriptor.value;
+
+    descriptor.value = function (...args: any[]) {
+        try {
+            UndoManager.StartBatch()
+            return oldFunction.apply(this, args)
+        } finally {
+            UndoManager.EndBatch()
         }
     }
+
+    return descriptor;
 }
+
 export namespace UndoManager {
     export interface UndoEvent {
         undo: () => void;
-- 
cgit v1.2.3-70-g09d2