From 43e25b4acbded4f7ff05de0fd50089a14a95841e Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Manuel=20Baham=C3=B3ndez-Honores?= Date: Wed, 18 Feb 2026 06:00:24 -0300 Subject: [PATCH] Use boolean type for manualAddContacts with ternary conversion Keep setAutoAddContacts/setManualAddContacts passing true/false as originally intended. Convert to 0/1 at the writeByte boundary using a ternary operator. --- src/connection/connection.js | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/src/connection/connection.js b/src/connection/connection.js index 0aca501..ee49fd6 100644 --- a/src/connection/connection.js +++ b/src/connection/connection.js @@ -475,13 +475,13 @@ class Connection extends EventEmitter { } /** - * @param {number} manualAddContacts + * @param {boolean} manualAddContacts * @returns {Promise} */ async sendCommandSetOtherParams(manualAddContacts) { const data = new BufferWriter(); data.writeByte(Constants.CommandCodes.SetOtherParams); - data.writeByte(manualAddContacts); // 0 or 1 + data.writeByte(manualAddContacts ? 1 : 0); await this.sendToRadioFrame(data.toBytes()); } @@ -2569,7 +2569,7 @@ class Connection extends EventEmitter { } /** - * @param {number} manualAddContacts + * @param {boolean} manualAddContacts * @returns {Promise} */ setOtherParams(manualAddContacts) { @@ -2605,12 +2605,12 @@ class Connection extends EventEmitter { /** @returns {Promise} */ async setAutoAddContacts() { - return await this.setOtherParams(0); + return await this.setOtherParams(false); } /** @returns {Promise} */ async setManualAddContacts() { - return await this.setOtherParams(1); + return await this.setOtherParams(true); } /**