Merged revisions 82261 via svnmerge from
authorSteve Murphy <murf@digium.com>
Tue, 11 Sep 2007 20:49:21 +0000 (20:49 +0000)
committerSteve Murphy <murf@digium.com>
Tue, 11 Sep 2007 20:49:21 +0000 (20:49 +0000)
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.
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@82262 65c4cc65-6c06-0410-ace0-fbb531ad65f3

main/cdr.c

index 43b92ae..16fa842 100644 (file)
@@ -977,6 +977,8 @@ static void post_cdr(struct ast_cdr *cdr)
        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))