Remove unnecessary extra sig during sim (#67)

0xFirekeeper · web-flow · commit 544ead85a5f9 · 2024-09-10T06:34:02.000+03:00
diff --git a/Thirdweb/Thirdweb.Wallets/SmartWallet/SmartWallet.cs b/Thirdweb/Thirdweb.Wallets/SmartWallet/SmartWallet.cs
@@ -394,7 +394,10 @@ private async Task<object> SignUserOp(ThirdwebTransactionInput transactionInput,
 
             // Hash, sign and encode the user operation
 
-            partialUserOp.Signature = await this.HashAndSignUserOp(partialUserOp, this._entryPointContract).ConfigureAwait(false);
+            if (!simulation)
+            {
+                partialUserOp.Signature = await this.HashAndSignUserOp(partialUserOp, this._entryPointContract).ConfigureAwait(false);
+            }
 
             return partialUserOp;
         }

Original file line number	Diff line number	Diff line change
`@@ -394,7 +394,10 @@ private async Task<object> SignUserOp(ThirdwebTransactionInput transactionInput,`
`394`	`394`
`395`	`395`	`// Hash, sign and encode the user operation`
`396`	`396`
`397`		`- partialUserOp.Signature = await this.HashAndSignUserOp(partialUserOp, this._entryPointContract).ConfigureAwait(false);`
	`397`	`+ if (!simulation)`
	`398`	`+ {`
	`399`	`+ partialUserOp.Signature = await this.HashAndSignUserOp(partialUserOp, this._entryPointContract).ConfigureAwait(false);`
	`400`	`+ }`
`398`	`401`
`399`	`402`	`return partialUserOp;`
`400`	`403`	`}`