asterisk users - Oct 2005 - ASTCC -- semantic note of 'callstart' in cdrs?

Looking at the code, it would appear that the 'callstart' column of the
cdrs table should really be
called 'callend':

                $dialstr = "IAX2/$res->{path}/$phone|30|HL(" .
($maxtime * 60 * 1000) .
":60000:30000)";
                $res = $AGI->exec("DIAL $dialstr");
                $answeredtime = $AGI->get_variable("ANSWEREDTIME");
                $dialstatus = $AGI->get_variable("DIALSTATUS");
                $callstart = localtime();
                return $dialstatus;

No?

Eric.

That is true.  It's just one of those things that is easier to leave 
alone to avoid breakage in upgrades.  It would be nice to get fixed 
though....

Darren Wiebe
darren@aleph-com.net

Eric Lyons wrote:
> Looking at the code, it would appear that the 'callstart' column of
> the cdrs table should really be called 'callend':
>
>                $dialstr = "IAX2/$res->{path}/$phone|30|HL(" . 
> ($maxtime * 60 * 1000) . ":60000:30000)";
>                $res = $AGI->exec("DIAL $dialstr");
>                $answeredtime =
$AGI->get_variable("ANSWEREDTIME");
>                $dialstatus = $AGI->get_variable("DIALSTATUS");
>                $callstart = localtime();
>                return $dialstatus;
>
> No?
>
> Eric.
> _______________________________________________
> --Bandwidth and Colocation sponsored by Easynews.com --
>
> Asterisk-Users mailing list
> Asterisk-Users@lists.digium.com
> http://lists.digium.com/mailman/listinfo/asterisk-users
> To UNSUBSCRIBE or update options visit:
>   http://lists.digium.com/mailman/listinfo/asterisk-users