[asterisk-dev] [svn-commits] alecdavis: trunk r241097 - /trunk/channels/sig_pri.c

[Richard Mudgett]

Alec
There are two places where this would need to be done.  With and without overlap dialing.  The code sections are very similar.

Also see my review of your CDR patch on reviewboard for an additional concern: https://reviewboard.asterisk.org/r/460/

Richard
----- "SVN commits to the Digium repositories" <svn-commits at lists.digium.com> wrote:

> Author: alecdavis
> Date: Mon Jan 18 16:00:24 2010
> New Revision: 241097
> 
> URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=241097
> Log:
> Update CDR variables before pbx starts
> 
> Allows CDR variables added in cdr.c:set_one_cid to become visable
> during the call.
> 
> (closes issue #16638)
> Reported by: alecdavis
> Patches: 
>       cdr_update.diff.txt uploaded by alecdavis (license 585)
> 
> 
> Modified:
>     trunk/channels/sig_pri.c
> 
> Modified: trunk/channels/sig_pri.c
> URL:
> http://svnview.digium.com/svn/asterisk/trunk/channels/sig_pri.c?view=diff&rev=241097&r1=241096&r2=241097
> ==============================================================================
> --- trunk/channels/sig_pri.c (original)
> +++ trunk/channels/sig_pri.c Mon Jan 18 16:00:24 2010
> @@ -2615,6 +2615,8 @@
>  
>  								sig_pri_handle_subcmds(pri, chanpos, e->e, e->ring.channel,
>  									e->ring.subcmds, e->ring.call);
> +
> +								ast_cdr_update(c);
>  							}
>  							if (c && !ast_pbx_start(c)) {
>  								ast_verb(3, "Accepting call from '%s' to '%s' on channel
> %d/%d, span %d\n",
> 
> 
> -- 
> _____________________________________________________________________
> -- Bandwidth and Colocation Provided by http://www.api-digital.com --
> 
> svn-commits mailing list
> To UNSUBSCRIBE or update options visit:
>    http://lists.digium.com/mailman/listinfo/svn-commits