fixed some bugs

Author: sbazerque

src/data/collections/causal/CausalArray.ts

@@ -178,7 +178,7 @@ class CausalArray<T> extends BaseCausalCollection<T> implements CausalCollection
     _ordinals : Array<Ordinal>;
 
     constructor(config?: MutableArrayConfig & CausalCollectionConfig) {
-        super(CausalArray.opClasses, config);
+        super(CausalArray.opClasses, {...config, supportsUndo: true});
 
         this.setRandomId();
 
@@ -593,6 +593,8 @@ class CausalArray<T> extends BaseCausalCollection<T> implements CausalCollection
 
     shouldAcceptMutationOp(op: MutationOp, opReferences: Map<Hash, HashedObject>): boolean {
 
+        console.log('VALIDATING....')
+
         if (!super.shouldAcceptMutationOp(op, opReferences)) {
             return false;
         }
@@ -610,16 +612,24 @@ class CausalArray<T> extends BaseCausalCollection<T> implements CausalCollection
                                             this.createDeleteAuthorizerByHash(
                                                     HashedObject.hashElement(op.getInsertOp().element), author);
 
+            console.log('verifying mutation op for ' + this.hash());
+            console.log('auth', auth);
+
             const usedKeys     = new Set<string>();
 
             if (!auth.verify(op, usedKeys)) {
+                console.log('verification failed')
                 return false;
             }
 
+            console.log('verification OK')
+
             if (!Verification.checkKeys(usedKeys, op)) {
                 return false;
             }
 
+            console.log('keys OK: ', usedKeys);
+
         }
 
         return true;

src/data/model/causal/Authorization.ts

@@ -1,3 +1,4 @@
+import { HashedMap } from 'data/model';
 import { HashedSet } from '../immutable/HashedSet';
 import { MutationOp } from '../mutable/MutationOp';
 
@@ -48,12 +49,12 @@ class Attestation {
                   = (a1: Attestator, a2?: Attestator) => 
                             ( async (op: MutationOp) => {
 
-                                    const savedCausalOps = op.getCausalOps().entries();
+                                    const savedCausalOps = new HashedMap(op.getCausalOps().entries());
 
                                     if (await a1(op) && (a2 === undefined || await a2(op)))  {
                                         return true;
                                     } else {
-                                        op.setCausalOps(savedCausalOps)
+                                        op.setCausalOps(savedCausalOps.entries())
                                         return false;
                                     }
                                 }
@@ -77,11 +78,11 @@ class Attestation {
                = (authorizers: Array<Attestator>) =>
                                         async (op: MutationOp) => {
 
-                                            const savedCausalOps = op.getCausalOps().entries();
+                                            const savedCausalOps = new HashedMap(op.getCausalOps().entries());
 
                                             for (const authorize of authorizers) {
                                                 if (!(await authorize(op))) {
-                                                    op.setCausalOps(savedCausalOps);
+                                                    op.setCausalOps(savedCausalOps.entries());
                                                     return false;
                                                 }
                                             }

src/data/model/immutable/HashedObject.ts

@@ -101,8 +101,10 @@ abstract class HashedObject {
         //       would be).
 
         this.author = author;
-        this._signOnSave = true;
-        
+
+        if (author !== undefined) {
+            this._signOnSave = true;
+        }
     }
 
     getAuthor() {