[asterisk-commits] tilghman: branch 1.6.2 r220292 - in /branches/1.6.2: ./ apps/ main/

[SVN commits to the Asterisk project]

Author: tilghman
Date: Thu Sep 24 14:42:42 2009
New Revision: 220292

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=220292
Log:
Merged revisions 220289 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/trunk

................
  r220289 | tilghman | 2009-09-24 14:41:02 -0500 (Thu, 24 Sep 2009) | 13 lines
  
  Merged revisions 220288 via svnmerge from 
  https://origsvn.digium.com/svn/asterisk/branches/1.4
  
  ........
    r220288 | tilghman | 2009-09-24 14:39:41 -0500 (Thu, 24 Sep 2009) | 6 lines
    
    Implicitly sending a progress signal breaks some applications.
    Call Progress() in your dialplan if you explicitly want progress to be sent.
    (Reverts change 216430, closes issue #15957)
    Reported by: Pavel Troller on the Asterisk-Dev mailing list
    http://lists.digium.com/pipermail/asterisk-dev/2009-September/039897.html
  ........
................

Modified:
    branches/1.6.2/   (props changed)
    branches/1.6.2/apps/app_disa.c
    branches/1.6.2/apps/app_playback.c
    branches/1.6.2/main/pbx.c

Propchange: branches/1.6.2/
------------------------------------------------------------------------------
Binary property 'trunk-merged' - no diff available.

Modified: branches/1.6.2/apps/app_disa.c
URL: http://svnview.digium.com/svn/asterisk/branches/1.6.2/apps/app_disa.c?view=diff&rev=220292&r1=220291&r2=220292
==============================================================================
--- branches/1.6.2/apps/app_disa.c (original)
+++ branches/1.6.2/apps/app_disa.c Thu Sep 24 14:42:42 2009
@@ -187,12 +187,7 @@
 			/* answer */
 			ast_answer(chan);
 		}
-	} else {
-		special_noanswer = 1;
-		if (chan->_state != AST_STATE_UP) {
-			ast_indicate(chan, AST_CONTROL_PROGRESS);
-		}
-	}
+	} else special_noanswer = 1;
 
 	ast_debug(1, "Context: %s\n",args.context);
 

Modified: branches/1.6.2/apps/app_playback.c
URL: http://svnview.digium.com/svn/asterisk/branches/1.6.2/apps/app_playback.c?view=diff&rev=220292&r1=220291&r2=220292
==============================================================================
--- branches/1.6.2/apps/app_playback.c (original)
+++ branches/1.6.2/apps/app_playback.c Thu Sep 24 14:42:42 2009
@@ -452,10 +452,7 @@
 		} else if (!option_noanswer) {
 			/* Otherwise answer unless we're supposed to send this while on-hook */
 			res = ast_answer(chan);
-		} else {
-			ast_indicate(chan, AST_CONTROL_PROGRESS);
-		}
-
+		}
 	}
 	if (!res) {
 		char *back = args.filenames;

Modified: branches/1.6.2/main/pbx.c
URL: http://svnview.digium.com/svn/asterisk/branches/1.6.2/main/pbx.c?view=diff&rev=220292&r1=220291&r2=220292
==============================================================================
--- branches/1.6.2/main/pbx.c (original)
+++ branches/1.6.2/main/pbx.c Thu Sep 24 14:42:42 2009
@@ -8810,8 +8810,6 @@
 		} else if (!ast_test_flag(&flags, BACKGROUND_NOANSWER)) {
 			res = ast_answer(chan);
 		}
-		/* Send progress control frame to start early media */
-		ast_indicate(chan, AST_CONTROL_PROGRESS);
 	}
 
 	if (!res) {