Merge pull request #319 from kiroboio/txTrace_computed

fix(import): dont verify options when FCT is imported
kiroboio · Dec 17, 2023 · 6e63d95 · 6e63d95
2 parents c1754d6 + 170c80c
commit 6e63d95
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 3 deletions.
diff --git a/src/batchMultiSigCall/batchMultiSigCall.ts b/src/batchMultiSigCall/batchMultiSigCall.ts
@@ -175,6 +175,10 @@ export class BatchMultiSigCall {
   getCallByNodeId = getCallByNodeId;
   getIndexByNodeId = getIndexByNodeId;
 
+  protected _setOptionsWithoutValidation(options: DeepPartial<IFCTOptions>) {
+    return this._options.set(options, false) as RequiredFCTOptions;
+  }
+
   // Static functions
   static utils = utils;
 

diff --git a/src/batchMultiSigCall/classes/Options/index.ts b/src/batchMultiSigCall/classes/Options/index.ts
@@ -41,9 +41,9 @@ export class Options {
   static helpers = helpers;
   private _options: IFCTOptions = initOptions;
 
-  public set<O extends DeepPartial<IFCTOptions>>(options: O): IFCTOptions & O {
+  public set<O extends DeepPartial<IFCTOptions>>(options: O, verify = true): IFCTOptions & O {
     const mergedOptions = deepMerge(this._options, options);
-    // Options.verify(mergedOptions);
+    if (verify) Options.verify(mergedOptions);
     this._options = mergedOptions;
     return this._options as IFCTOptions & O;
   }

diff --git a/src/batchMultiSigCall/methods/FCT.ts b/src/batchMultiSigCall/methods/FCT.ts
@@ -357,7 +357,7 @@ export function impFCT(this: BatchMultiSigCall, fct: IFCT, map?: ReturnType<Batc
     },
   };
 
-  this.setOptions(options);
+  this._setOptionsWithoutValidation(options);
   const { types: typesObject } = typedData;
 
   for (const [index, call] of fct.mcall.entries()) {