Question them.escrowDays does not work?

[jafix]

console.log('[OFFER] Checking trade hold');

offer.getUserDetails(function(them) { //check for escrow

if (them.escrowDays > 0) {
    offer.decline();
    console.log('[OFFER] Declined - Trade hold');
    tradelog.info('Trade offer #' + offer.id + ' from ' + offer.partner.getSteamID64() + ' declined: trade hold');
    } else {
        //script to check trade contents
    }
});

I have this part in the manager.on('newOffer', function (offer) { part, so them and offer are defined. The error I'm getting is: Cannot property 'escrowDays' of null.

Edited March 11, 2017 by jafix

[jafix]

I don't know how, but recoding it helped.

[Dr. McKay]

The problem was because you were missing a few arguments in your callback function. The names don't really matter, it's the order that matters.

[jafix]

console.log('[OFFER] Checking trade hold');
		
		offer.getUserDetails(function(err, them) { //check for escrow
		if (err) {
			throw err;
		}
		
		if (them.escrowDays != 0) {
			offer.decline();
			console.log('[OFFER] Declined - Trade hold');
			tradelog.info('Trade offer #' + offer.id + ' from ' + offer.partner.getSteamID64() + ' declined: trade hold');
			client.chatMessage(offer.partner.getSteamID64(), "Sorry, we don't accept trade hold");
			} else {
                            //check trade contents
                        }
                    });

This is my script now but my bot accepted an offer resulting in a trade hold, why is that?

[Dr. McKay]

The problem was because you were missing a few arguments in your callback function. The names don't really matter, it's the order that matters.

 

The first argument is me.