[asterisk-commits] rmudgett: branch 11 r401579 - in /branches/11: ./ cdr/cdr_adaptive_odbc.c

[SVN commits to the Asterisk project]

Author: rmudgett
Date: Wed Oct 23 11:46:26 2013
New Revision: 401579

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=401579
Log:
cdr_adaptive_odbc: Also apply a filter when the CDR value is empty.

Extra CDR records are written if a filtered CDR value is empty because the
filter is not checked.

(closes issue ASTERISK-22272)
Reported by: Jordi Llull Chavarria
........

Merged revisions 401577 from http://svn.asterisk.org/svn/asterisk/branches/1.8

Modified:
    branches/11/   (props changed)
    branches/11/cdr/cdr_adaptive_odbc.c

Propchange: branches/11/
------------------------------------------------------------------------------
Binary property 'branch-1.8-merged' - no diff available.

Modified: branches/11/cdr/cdr_adaptive_odbc.c
URL: http://svnview.digium.com/svn/asterisk/branches/11/cdr/cdr_adaptive_odbc.c?view=diff&rev=401579&r1=401578&r2=401579
==============================================================================
--- branches/11/cdr/cdr_adaptive_odbc.c (original)
+++ branches/11/cdr/cdr_adaptive_odbc.c Wed Oct 23 11:46:26 2013
@@ -711,6 +711,14 @@
 					continue;
 				}
 				first = 0;
+			} else if (entry->filtervalue
+				&& ((!entry->negatefiltervalue && entry->filtervalue[0] != '\0')
+					|| (entry->negatefiltervalue && entry->filtervalue[0] == '\0'))) {
+				ast_verb(4, "CDR column '%s' was not set and does not match filter of"
+					" %s'%s'.  Cancelling this CDR.\n",
+					entry->cdrname, entry->negatefiltervalue ? "!" : "",
+					entry->filtervalue);
+				goto early_release;
 			}
 		}