toString causing issues, please use toString()

[drumfreak]

thisRequest.dps[options.dps.toString] = options.set;

Seems to be missing the () behind toString. This caused issues setting dps values with using this package from homebridge.

For example:

tuya.set({id: 'xxxxxxxxxxxxxxxxxxxx', 'dps': 2, set: true}).then(() => console.log('device was changed'))

Would fail because toString is not a function in prototype.

Proposed change:

thisRequest.dps[options.dps.toString()] = options.set;

or even:

if (options.dps === undefined) {
thisRequest.dps = {1: options.set};
} else {
thisRequest.dps = {};
thisRequest.dps[options.dps] = options.set;
}

tuyapi/index.js

Line 233 in 290d6e9

thisRequest.dps[options.dps.toString] = options.set;

[codetheweb]

Oops, thanks for the heads up. (I should really get around to adding test cases.) Should be fixed now.

[drumfreak]

Thank you @codetheweb