Make orderHashHex a getter instead of a property

1 files changed, 3 insertions, 4 deletions

diff --git a/packages/contracts/util/order.ts b/packages/contracts/util/order.ts
index 57bb2bcbf..12f10fa4e 100644
--- a/packages/contracts/util/order.ts
+++ b/packages/contracts/util/order.ts
@@ -18,7 +18,7 @@ export class Order {
         if (_.isUndefined(v) || _.isUndefined(r) || _.isUndefined(s)) {
             throw new Error('Cannot call isValidSignature on unsigned order');
         }
-        const orderHash = this._getOrderHash();
+        const orderHash = this.getOrderHashHex();
         const msgHash = ethUtil.hashPersonalMessage(ethUtil.toBuffer(orderHash));
         try {
             const pubKey = ethUtil.ecrecover(msgHash, v, ethUtil.toBuffer(r), ethUtil.toBuffer(s));
@@ -29,11 +29,10 @@ export class Order {
         }
     }
     public async signAsync() {
-        const orderHash = this._getOrderHash();
+        const orderHash = this.getOrderHashHex();
         const signature = await this._web3Wrapper.signTransactionAsync(this.params.maker, orderHash);
         const { v, r, s } = ethUtil.fromRpcSig(signature);
         this.params = _.assign(this.params, {
-            orderHashHex: orderHash,
             v,
             r: ethUtil.bufferToHex(r),
             s: ethUtil.bufferToHex(s),
@@ -85,7 +84,7 @@ export class Order {
         };
         return cancel;
     }
-    private _getOrderHash(): string {
+    public getOrderHashHex(): string {
         const orderHash = crypto.solSHA3([
             this.params.exchangeContractAddress,
             this.params.maker,