[asterisk-bugs] [Asterisk 0015809]: [patch] callprogress and faxdetect not read when building channel results in fax detection failure

[Asterisk Bug Tracker]

The following issue has been SUBMITTED. 
====================================================================== 
https://issues.asterisk.org/view.php?id=15809 
====================================================================== 
Reported By:                phsultan
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   15809
Category:                   Channels/chan_dahdi
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     new
Asterisk Version:           Older 1.6.1 
Regression:                 No 
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Request Review:              
====================================================================== 
Date Submitted:             2009-09-01 13:15 CDT
Last Modified:              2009-09-01 13:15 CDT
====================================================================== 
Summary:                    [patch] callprogress and faxdetect not read when
building channel results in fax detection failure
Description: 
The faxdetect and callprogress attributes are not read when creating
channels with build_channels with channels/chan_dahdi.c, and therefore, the
following chan_dahdi.conf prevents fax detection from happening :
[channels]
#include "dahdi-channels.conf"
callprogress=yes
faxdetect=both
usecallerid=yes
callwaiting=yes
usecallingpres=yes
callwaitingcallerid=yes
threewaycalling=yes
transfer=yes
canpark=yes
cancallforward=yes
callreturn=yes
echocancel=no

Actually, it looks like the whole [channels] section should be read to
populate the conf variable before calling build_channels().

I'm not sure if it's the right way to solve this issue, and there is a
configuration workaround (see the additional information). The attached
patch does the following :
- make Asterisk read the [channels] section without building channels
first ;
- build a first version of the configuration in the base_conf variable ;
- build a new configuration from base_conf and actually call
build_channels() after reading the channel attribute.

====================================================================== 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2009-09-01 13:15 phsultan       New Issue                                    
2009-09-01 13:15 phsultan       Asterisk Version          => Older 1.6.1     
2009-09-01 13:15 phsultan       Regression                => No              
2009-09-01 13:15 phsultan       SVN Branch (only for SVN checkouts, not tarball
releases) => N/A             
======================================================================