[asterisk-bugs] [Asterisk 0013921]: Configuration reload overrides channel variable setting of ackcall (autologoff, acceptdtmf and enddtmf)

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=13921 
====================================================================== 
Reported By:                davidw
Assigned To:                putnopvut
====================================================================== 
Project:                    Asterisk
Issue ID:                   13921
Category:                   Channels/chan_agent
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     acknowledged
Asterisk Version:           1.6.0 
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             2008-11-18 06:23 CST
Last Modified:              2008-11-25 17:30 CST
====================================================================== 
Summary:                    Configuration reload overrides channel variable
setting of ackcall (autologoff, acceptdtmf and enddtmf)
Description: 
p->ackcall can be set using channel variable, at login time, but is reset
to the global default, by add_agent, if the configuration is reloaded, even
though the agent is still logged in.

======================================================================
Relationships       ID      Summary
----------------------------------------------------------------------
related to          0013098 [patch] p->ackcall not reset when ag...
====================================================================== 

---------------------------------------------------------------------- 
 (0095524) putnopvut (administrator) - 2008-11-25 17:30
 http://bugs.digium.com/view.php?id=13921#c95524 
---------------------------------------------------------------------- 
I came up with a patch that worked in my test setup. If we determine that
one of the settings which can be set with a channel variable has been set,
then we set a flag indicating that the value from the configuration file
should be ignored when reloading.

I have written a patch for 1.6.0 which addresses the ackcall, autologoff,
and wrapuptime variables. You mentioned acceptdtmf and enddtmf, but those
options are not available in 1.6.0. I have only actually tested the ackcall
setting, but the others operate on the same principle. Please test to see
that this works for you as well. 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2008-11-25 17:30 putnopvut      Note Added: 0095524                          
======================================================================