Skip to content

Commit e8fe429

Browse files
committed
qmodem_sms: fix: SMS send result check
1 parent c422391 commit e8fe429

File tree

1 file changed

+5
-7
lines changed
  • luci/luci-app-qmodem-next/root/usr/share/rpcd/ucode

1 file changed

+5
-7
lines changed

luci/luci-app-qmodem-next/root/usr/share/rpcd/ucode/qmodem_sms

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -621,13 +621,11 @@ return {
621621
pdu: request.args.pdu
622622
});
623623

624-
warn(`send_sms: Result: ${sprintf('%J', result)}\n`);
625-
626-
// Check return code - 0 means success
627-
let ret_code = result?.code != null ? result.code : -1;
628-
let is_success = (ret_code === 0 && !result?.error);
629-
630-
// Read sent database
624+
warn(`send_sms: Result: ${sprintf('%J', result)}\n`);
625+
626+
// Check status field from tom_modem output
627+
// tom_modem returns {"status":"success"} on success or {"status":"failed","reason":"..."} on failure
628+
let is_success = (result?.status === 'success'); // Read sent database
631629
let db = readSmsDb(config.sms_db_path, config_section);
632630
if (!db.sent)
633631
db.sent = [];

0 commit comments

Comments
 (0)