[Home]

Summary:ASTERISK-12431: [patch] Make app_image work like app_sendtext, app_url, etc.
Reporter:Eliel Sardanons (eliel)Labels:
Date Opened:2008-07-22 20:21:12Date Closed:2008-07-28 11:41:50
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Applications/app_image
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) app_image.c.patch
( 1) UPGRADE.patch
Description:SENDIMAGESTATUS is returning 'NOSUPPORT' and 'OK', while other apps are returning:
UNSUPPORTED, FAILURE, SUCCESS.

Also app_image on trasnmission error (or error on filename) is hanginup the channel. Prevent that and set SENDIMAGESTATUS=FAILURE on error.
Comments:By: Tilghman Lesher (tilghman) 2008-07-24 01:11:55

If you're going to change this, then I need corresponding text in the UPGRADE.txt file noting this change.
By: Eliel Sardanons (eliel) 2008-07-24 19:01:08

UPGRADE.patch is the documentation you ask. (Sorry I forgot that).

I think we should move UPGRADE.txt to UPGRADE-1.6.txt and truncate file UPGRADE.txt (will be for 1.8 in the future).
By: Digium Subversion (svnbot) 2008-07-28 11:41:30

Repository: asterisk
Revision: 134088

U   trunk/CHANGES
U   trunk/UPGRADE.txt
U   trunk/apps/app_image.c

------------------------------------------------------------------------
r134088 | tilghman | 2008-07-28 11:41:29 -0500 (Mon, 28 Jul 2008) | 7 lines

Change SendImage() to output a more consistent status variable.
(closes issue ASTERISK-12431)
Reported by: eliel
Patches:
      app_image.c.patch uploaded by eliel (license 64)
      UPGRADE.patch uploaded by eliel (license 64)

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=134088