Don't reassign the parameter
This commit is contained in:
		@@ -15,11 +15,11 @@ export const orderFactory = {
 | 
			
		||||
        takerTokenAddress: string,
 | 
			
		||||
        exchangeContractAddress: string,
 | 
			
		||||
        feeRecipient: string,
 | 
			
		||||
        expirationUnixTimestampSec?: BigNumber): Promise<SignedOrder> {
 | 
			
		||||
        expirationUnixTimestampSecIfExists?: BigNumber): Promise<SignedOrder> {
 | 
			
		||||
        const defaultExpirationUnixTimestampSec = new BigNumber(2524604400); // Close to infinite
 | 
			
		||||
        expirationUnixTimestampSec = _.isUndefined(expirationUnixTimestampSec) ?
 | 
			
		||||
        const expirationUnixTimestampSec = _.isUndefined(expirationUnixTimestampSecIfExists) ?
 | 
			
		||||
            defaultExpirationUnixTimestampSec :
 | 
			
		||||
            expirationUnixTimestampSec;
 | 
			
		||||
            expirationUnixTimestampSecIfExists;
 | 
			
		||||
        const order = {
 | 
			
		||||
            maker,
 | 
			
		||||
            taker,
 | 
			
		||||
 
 | 
			
		||||
		Reference in New Issue
	
	Block a user