[Home]

Summary:ASTERISK-14140: Abandoned queue calls won't show on CDR
Reporter:Sebastian Gutierrez (sum)Labels:
Date Opened:2009-05-15 12:59:50Date Closed:2009-05-29 14:14:33
Priority:MajorRegression?No
Status:Closed/CompleteComponents:Applications/app_queue
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:
Description:if I make a call to a queue and the call is abandoned, the call doesn't show on CDR, this make the CDR inconsistent to match to queue_log or any other carrier CDR or other platform CDR.
Comments:By: Matthew Nicholson (mnicholson) 2009-05-18 10:03:08

Please test with the patch on issue ASTERISK-12274.
By: Sebastian Gutierrez (sum) 2009-05-18 13:10:24

still not working in these cases:

1) call enter queue there is nobody on the queue to direct the call
2) the call is put on an agent, is ringing when the caller hangs up
By: Matthew Nicholson (mnicholson) 2009-05-18 13:34:27

I'll check it out.
By: Matthew Nicholson (mnicholson) 2009-05-18 16:08:50

I tested those two cases with and without the patch from issue ASTERISK-12274.  Without the patch there was no CDR record for 1 and there were two records for 2.  With the patch there is one CDR record for each.

Are you testing with unanswered=yes?
By: Sebastian Gutierrez (sum) 2009-05-18 16:19:18

I'm testing with unanswered=no, cause the call is answered by the system, in these case the queue, you say I should use unanswered=yes? I tried that some time ago and too many unusefull information was logged at the cdr.
By: Matthew Nicholson (mnicholson) 2009-05-18 16:23:55

If the system answers the call, then unanswered=no is fine.  Also in that case there should be no problems with the CDR.  Test with unanswered=yes and see if you get the records you expect to have.

Also test with SVN 1.4 and with the patch from issue ASTERISK-12274.
By: Sebastian Gutierrez (sum) 2009-05-18 21:35:56

ok, you can close this,
patch worked as expected.

1) if I use answer on the dialplan and not directly the queue I get the cdr record.

2) if I don't use answer and unanswered=yes I get the expected cdr record if the agent did not answer the call, also if there were no agent on that queue

tnkhs!


By: Leif Madsen (lmadsen) 2009-05-21 07:58:42

Just wanted to check with mnicholson to see if we can close this now? Is this waiting on another issue to be closed first?
By: Matthew Nicholson (mnicholson) 2009-05-21 09:31:51

This is waiting on ASTERISK-12274, which will close this as well.
By: Digium Subversion (svnbot) 2009-05-29 13:53:05

Repository: asterisk
Revision: 198068

U   branches/1.4/include/asterisk/cdr.h
U   branches/1.4/main/cdr.c
U   branches/1.4/main/channel.c
U   branches/1.4/res/res_features.c

------------------------------------------------------------------------
r198068 | mnicholson | 2009-05-29 13:53:01 -0500 (Fri, 29 May 2009) | 15 lines

Use AST_CDR_NOANSWER instead of AST_CDR_NULL as the default CDR disposition.

This change also involves the addition of an AST_CDR_FLAG_ORIGINATED flag that is used on originated channels to distinguish: them from dialed channels.

(closes issue ASTERISK-12274)
Reported by: meral
Patches:
     null-cdr2.diff uploaded by mnicholson (license 96)
Tested by: mnicholson, dbrooks

(closes issue ASTERISK-14140)
Reported by: sum
Tested by: sum


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

http://svn.digium.com/view/asterisk?view=rev&revision=198068
By: Digium Subversion (svnbot) 2009-05-29 14:04:26

Repository: asterisk
Revision: 198072

_U  trunk/
U   trunk/include/asterisk/cdr.h
U   trunk/main/cdr.c
U   trunk/main/channel.c

------------------------------------------------------------------------
r198072 | mnicholson | 2009-05-29 14:04:24 -0500 (Fri, 29 May 2009) | 21 lines

Merged revisions 198068 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
 r198068 | mnicholson | 2009-05-29 13:53:01 -0500 (Fri, 29 May 2009) | 15 lines
 
 Use AST_CDR_NOANSWER instead of AST_CDR_NULL as the default CDR disposition.
 
 This change also involves the addition of an AST_CDR_FLAG_ORIGINATED flag that is used on originated channels to distinguish: them from dialed channels.
 
 (closes issue ASTERISK-12274)
 Reported by: meral
 Patches:
       null-cdr2.diff uploaded by mnicholson (license 96)
 Tested by: mnicholson, dbrooks
 
 (closes issue ASTERISK-14140)
 Reported by: sum
 Tested by: sum
........

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

http://svn.digium.com/view/asterisk?view=rev&revision=198072
By: Digium Subversion (svnbot) 2009-05-29 14:13:05

Repository: asterisk
Revision: 198073

_U  branches/1.6.0/
U   branches/1.6.0/include/asterisk/cdr.h
U   branches/1.6.0/main/cdr.c
U   branches/1.6.0/main/channel.c

------------------------------------------------------------------------
r198073 | mnicholson | 2009-05-29 14:13:03 -0500 (Fri, 29 May 2009) | 28 lines

Merged revisions 198072 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
 r198072 | mnicholson | 2009-05-29 14:04:24 -0500 (Fri, 29 May 2009) | 21 lines
 
 Merged revisions 198068 via svnmerge from
 https://origsvn.digium.com/svn/asterisk/branches/1.4
 
 ........
   r198068 | mnicholson | 2009-05-29 13:53:01 -0500 (Fri, 29 May 2009) | 15 lines
   
   Use AST_CDR_NOANSWER instead of AST_CDR_NULL as the default CDR disposition.
   
   This change also involves the addition of an AST_CDR_FLAG_ORIGINATED flag that is used on originated channels to distinguish: them from dialed channels.
   
   (closes issue ASTERISK-12274)
   Reported by: meral
   Patches:
         null-cdr2.diff uploaded by mnicholson (license 96)
   Tested by: mnicholson, dbrooks
   
   (closes issue ASTERISK-14140)
   Reported by: sum
   Tested by: sum
 ........
................

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

http://svn.digium.com/view/asterisk?view=rev&revision=198073
By: Digium Subversion (svnbot) 2009-05-29 14:13:47

Repository: asterisk
Revision: 198074

_U  branches/1.6.1/
U   branches/1.6.1/include/asterisk/cdr.h
U   branches/1.6.1/main/cdr.c
U   branches/1.6.1/main/channel.c

------------------------------------------------------------------------
r198074 | mnicholson | 2009-05-29 14:13:45 -0500 (Fri, 29 May 2009) | 28 lines

Merged revisions 198072 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
 r198072 | mnicholson | 2009-05-29 14:04:24 -0500 (Fri, 29 May 2009) | 21 lines
 
 Merged revisions 198068 via svnmerge from
 https://origsvn.digium.com/svn/asterisk/branches/1.4
 
 ........
   r198068 | mnicholson | 2009-05-29 13:53:01 -0500 (Fri, 29 May 2009) | 15 lines
   
   Use AST_CDR_NOANSWER instead of AST_CDR_NULL as the default CDR disposition.
   
   This change also involves the addition of an AST_CDR_FLAG_ORIGINATED flag that is used on originated channels to distinguish: them from dialed channels.
   
   (closes issue ASTERISK-12274)
   Reported by: meral
   Patches:
         null-cdr2.diff uploaded by mnicholson (license 96)
   Tested by: mnicholson, dbrooks
   
   (closes issue ASTERISK-14140)
   Reported by: sum
   Tested by: sum
 ........
................

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

http://svn.digium.com/view/asterisk?view=rev&revision=198074
By: Digium Subversion (svnbot) 2009-05-29 14:14:32

Repository: asterisk
Revision: 198075

_U  branches/1.6.2/
U   branches/1.6.2/include/asterisk/cdr.h
U   branches/1.6.2/main/cdr.c
U   branches/1.6.2/main/channel.c

------------------------------------------------------------------------
r198075 | mnicholson | 2009-05-29 14:14:30 -0500 (Fri, 29 May 2009) | 28 lines

Merged revisions 198072 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
 r198072 | mnicholson | 2009-05-29 14:04:24 -0500 (Fri, 29 May 2009) | 21 lines
 
 Merged revisions 198068 via svnmerge from
 https://origsvn.digium.com/svn/asterisk/branches/1.4
 
 ........
   r198068 | mnicholson | 2009-05-29 13:53:01 -0500 (Fri, 29 May 2009) | 15 lines
   
   Use AST_CDR_NOANSWER instead of AST_CDR_NULL as the default CDR disposition.
   
   This change also involves the addition of an AST_CDR_FLAG_ORIGINATED flag that is used on originated channels to distinguish: them from dialed channels.
   
   (closes issue ASTERISK-12274)
   Reported by: meral
   Patches:
         null-cdr2.diff uploaded by mnicholson (license 96)
   Tested by: mnicholson, dbrooks
   
   (closes issue ASTERISK-14140)
   Reported by: sum
   Tested by: sum
 ........
................

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

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