diff options
author | tilghman <tilghman@f38db490-d61c-443f-a65b-d21fe96a405b> | 2008-07-28 16:49:29 +0000 |
---|---|---|
committer | tilghman <tilghman@f38db490-d61c-443f-a65b-d21fe96a405b> | 2008-07-28 16:49:29 +0000 |
commit | aa5fc8c25632317394c2afcb46a20323900ad793 (patch) | |
tree | d0c3687531dad36fb006d20b1d3e1e496d01d965 /apps/app_image.c | |
parent | 255f52d647373e2a97edfc7556673935c1eda449 (diff) |
Change SendImage() to output a more consistent status variable.
(closes issue #13134)
Reported by: eliel
Patches:
app_image.c.patch uploaded by eliel (license 64)
UPGRADE.patch uploaded by eliel (license 64)
git-svn-id: http://svn.digium.com/svn/asterisk/trunk@134088 f38db490-d61c-443f-a65b-d21fe96a405b
Diffstat (limited to 'apps/app_image.c')
-rw-r--r-- | apps/app_image.c | 24 |
1 files changed, 13 insertions, 11 deletions
diff --git a/apps/app_image.c b/apps/app_image.c index 1e5b0c857..ac44495e3 100644 --- a/apps/app_image.c +++ b/apps/app_image.c @@ -39,16 +39,15 @@ static char *synopsis = "Send an image file"; static char *descrip = " SendImage(filename): Sends an image on a channel.\n" -"If the channel supports image transport but the image send fails, the channel\n" -"will be hung up. Otherwise, the dialplan continues execution. This\n" -"application sets the following channel variable upon completion:\n" -" SENDIMAGESTATUS The status is the result of the attempt, one of:\n" -" OK | NOSUPPORT \n"; +"Result of transmission will be stored in SENDIMAGESTATUS\n" +"channel variable:\n" +" SUCCESS Transmission succeeded\n" +" FAILURE Transmission failed\n" +" UNSUPPORTED Image transmission not supported by channel\n"; static int sendimage_exec(struct ast_channel *chan, void *data) { - int res = 0; if (ast_strlen_zero(data)) { ast_log(LOG_WARNING, "SendImage requires an argument (filename)\n"); @@ -57,14 +56,17 @@ static int sendimage_exec(struct ast_channel *chan, void *data) if (!ast_supports_images(chan)) { /* Does not support transport */ - pbx_builtin_setvar_helper(chan, "SENDIMAGESTATUS", "NOSUPPORT"); + pbx_builtin_setvar_helper(chan, "SENDIMAGESTATUS", "UNSUPPORTED"); return 0; } - if (!(res = ast_send_image(chan, data))) - pbx_builtin_setvar_helper(chan, "SENDIMAGESTATUS", "OK"); - - return res; + if (!ast_send_image(chan, data)) { + pbx_builtin_setvar_helper(chan, "SENDIMAGESTATUS", "SUCCESS"); + } else { + pbx_builtin_setvar_helper(chan, "SENDIMAGESTATUS", "FAILURE"); + } + + return 0; } static int unload_module(void) |