[asterisk-commits] murf: trunk r82262 - in /trunk: ./ main/cdr.c

[SVN commits to the Asterisk project]

Author: murf
Date: Tue Sep 11 15:49:21 2007
New Revision: 82262

URL: http://svn.digium.com/view/asterisk?view=rev&rev=82262
Log:
Merged revisions 82261 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r82261 | murf | 2007-09-11 14:36:15 -0600 (Tue, 11 Sep 2007) | 1 line

this change should fix issue # 10659 -- what I worry about is how many other bug reports it may generate. Hopefully, we can please the/a majority. Hopefully. We shall see. Calls not marked ANSWERED and with only one channel name will not be posted. This should eliminate the double CDR's.
........

Modified:
    trunk/   (props changed)
    trunk/main/cdr.c

Propchange: trunk/
------------------------------------------------------------------------------
Binary property 'branch-1.4-merged' - no diff available.

Modified: trunk/main/cdr.c
URL: http://svn.digium.com/view/asterisk/trunk/main/cdr.c?view=diff&rev=82262&r1=82261&r2=82262
==============================================================================
--- trunk/main/cdr.c (original)
+++ trunk/main/cdr.c Tue Sep 11 15:49:21 2007
@@ -977,6 +977,8 @@
 	struct ast_cdr_beitem *i;
 
 	for ( ; cdr ; cdr = cdr->next) {
+		if (cdr->disposition < AST_CDR_ANSWERED && (ast_strlen_zero(cdr->channel) || ast_strlen_zero(cdr->dstchannel)))
+			continue; /* people don't want to see unanswered single-channel events */
 		chan = S_OR(cdr->channel, "<unknown>");
 		check_post(cdr);
 		if (ast_tvzero(cdr->end))