[asterisk-commits] twilson: branch 10 r354750 - in /branches/10: ./ funcs/func_cdr.c

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Thu Feb 9 16:03:56 CST 2012


Author: twilson
Date: Thu Feb  9 16:03:51 2012
New Revision: 354750

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=354750
Log:
Note that CDRs are immutable once a bridge is torn down

CDRs cannot be modified after a bridge is torn down, (e.g. after
Dial() returns) even though the CDR() function may be called. Since
modifying the CDR code to change this behavior could very easily
break all kinds of things, this patch just documents this limitation.

(closes issues ASTERISK-16923)
Review: https://reviewboard.asterisk.org/r/1720/
........

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

Modified:
    branches/10/   (props changed)
    branches/10/funcs/func_cdr.c

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

Modified: branches/10/funcs/func_cdr.c
URL: http://svnview.digium.com/svn/asterisk/branches/10/funcs/func_cdr.c?view=diff&rev=354750&r1=354749&r2=354750
==============================================================================
--- branches/10/funcs/func_cdr.c (original)
+++ branches/10/funcs/func_cdr.c Thu Feb  9 16:03:51 2012
@@ -140,7 +140,9 @@
 			with this function, and this variable will be stored on the cdr.</para>
 			<note><para>For setting CDR values, the <literal>l</literal> flag does not apply to
 			setting the <literal>accountcode</literal>, <literal>userfield</literal>, or
-			<literal>amaflags</literal>.</para></note>
+			<literal>amaflags</literal>.</para><para>CDRs can only be modified before the bridge
+			between two channels is torn down. For example, CDRs may not be modified after the
+			<literal>Dial</literal> application has returned.</para></note>
 			<para>Raw values for <literal>disposition</literal>:</para>
 			<enumlist>
 				<enum name="0">




More information about the asterisk-commits mailing list