[Home]

Summary:ASTERISK-03147: [patch] Inconsistency in documentation of ANSWEREDTIME variable
Reporter:karlp (karlp)Labels:
Date Opened:2004-12-31 10:15:47.000-0600Date Closed:2008-01-15 15:20:36.000-0600
Priority:TrivialRegression?No
Status:Closed/CompleteComponents:Documentation
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) README.variables.patch
Description:docs/README.variables defines ANSWEREDTIME variable differently than comments in app_dial.c.

app_dial.c clearly sets value to end_time - answer_time so README.variables should be changed to reflect this.

I propose the changes to the README.variables file attached.
Comments:By: Clod Patry (junky) 2004-12-31 10:50:51.000-0600

if im not wrong:
${DIALEDTIME} is like the duration into CDRs.
${ANSWEREDTIME} is like the billsec into CDRs.

By: Trevor Peirce (trev) 2005-01-02 12:28:12.000-0600

Your bug description kind of confused me, but your patch itself looks like it does correct wrong documentation.

By: Mark Spencer (markster) 2005-01-07 00:33:27.000-0600

Fixed in head

By: Russell Bryant (russell) 2005-01-08 15:44:37.000-0600

fixed in 1.0

By: Digium Subversion (svnbot) 2008-01-15 15:20:23.000-0600

Repository: asterisk
Revision: 4703

U   trunk/doc/README.variables

------------------------------------------------------------------------
r4703 | markster | 2008-01-15 15:20:23 -0600 (Tue, 15 Jan 2008) | 2 lines

Fix README for variables (bug ASTERISK-3147)

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

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

By: Digium Subversion (svnbot) 2008-01-15 15:20:36.000-0600

Repository: asterisk
Revision: 4717

U   branches/v1-0/doc/README.variables

------------------------------------------------------------------------
r4717 | russell | 2008-01-15 15:20:35 -0600 (Tue, 15 Jan 2008) | 2 lines

update docs on ANSWEREDTIME and DIALEDTIME (bug ASTERISK-3147)

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

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