Skip to content
Snippets Groups Projects
Commit 16374ad4 authored by Brett Bryant's avatar Brett Bryant
Browse files

Fix a bug where SENDTEXTSTATUS isn't set properly when it isn't

supported on a channel (yet _another_ useful patch by eliel).

(closes issue #13081)
Reported by: eliel
Patches:
      app_sendtext.c.patch uploaded by eliel (license 64)
Tested by: eliel


git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@132508 65c4cc65-6c06-0410-ace0-fbb531ad65f3
parent 1f744850
No related branches found
No related tags found
No related merge requests found
...@@ -42,7 +42,7 @@ static const char *app = "SendText"; ...@@ -42,7 +42,7 @@ static const char *app = "SendText";
static const char *synopsis = "Send a Text Message"; static const char *synopsis = "Send a Text Message";
static const char *descrip = static const char *descrip =
" SendText(text[,options]): Sends text to current channel (callee).\n" " SendText(text): Sends text to current channel (callee).\n"
"Result of transmission will be stored in the SENDTEXTSTATUS\n" "Result of transmission will be stored in the SENDTEXTSTATUS\n"
"channel variable:\n" "channel variable:\n"
" SUCCESS Transmission succeeded\n" " SUCCESS Transmission succeeded\n"
...@@ -58,24 +58,21 @@ static int sendtext_exec(struct ast_channel *chan, void *data) ...@@ -58,24 +58,21 @@ static int sendtext_exec(struct ast_channel *chan, void *data)
char *parse = NULL; char *parse = NULL;
AST_DECLARE_APP_ARGS(args, AST_DECLARE_APP_ARGS(args,
AST_APP_ARG(text); AST_APP_ARG(text);
AST_APP_ARG(options);
); );
if (ast_strlen_zero(data)) { if (ast_strlen_zero(data)) {
ast_log(LOG_WARNING, "SendText requires an argument (text[,options])\n"); ast_log(LOG_WARNING, "SendText requires an argument (text)\n");
return -1; return -1;
} else } else
parse = ast_strdupa(data); parse = ast_strdupa(data);
AST_STANDARD_APP_ARGS(args, parse); AST_STANDARD_APP_ARGS(args, parse);
if (args.options) {
}
ast_channel_lock(chan); ast_channel_lock(chan);
if (!chan->tech->send_text) { if (!chan->tech->send_text) {
ast_channel_unlock(chan); ast_channel_unlock(chan);
/* Does not support transport */ /* Does not support transport */
pbx_builtin_setvar_helper(chan, "SENDTEXTSTATUS", status);
return 0; return 0;
} }
status = "FAILURE"; status = "FAILURE";
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment