[prev in list] [next in list] [prev in thread] [next in thread] 

List:       asterisk-dev
Subject:    Re: [asterisk-dev] [Code Review] High Resolution Call Time for
From:       "Brad Latus" <snuffy22 () gmail ! com>
Date:       2010-03-31 19:03:10
Message-ID: 20100331190310.30011.48067 () hotblack ! digium ! internal
[Download RAW message or body]


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/461/
-----------------------------------------------------------

(Updated 2010-03-31 14:03:10.255346)


Review request for Asterisk Developers.


Changes
-------

Testing completed on all most common modules..


Summary
-------

This allows for the showing of the accurate times for billable time and call \
duration. Adds new flag to CDR function 'f' for the above fields.

This came about by bug #16559 and I thought i'd take a go at making it happen.

We can drop a few of the modules for ones we want to depreciate like sql-lite / odbc \
(in favour of sql-lite3 / adaptive_odbc)


This addresses bug 16559.
    https://issues.asterisk.org/view.php?id=16559


Diffs
-----

  /trunk/CHANGES 253708 
  /trunk/UPGRADE.txt 253708 
  /trunk/addons/cdr_mysql.c 253708 
  /trunk/cdr/cdr_adaptive_odbc.c 253708 
  /trunk/cdr/cdr_odbc.c 253708 
  /trunk/cdr/cdr_pgsql.c 253708 
  /trunk/cdr/cdr_sqlite.c 253708 
  /trunk/cdr/cdr_tds.c 253708 
  /trunk/configs/cdr_custom.conf.sample 253708 
  /trunk/configs/cdr_odbc.conf.sample 253708 
  /trunk/configs/cdr_sqlite3_custom.conf.sample 253708 
  /trunk/configs/cdr_syslog.conf.sample 253708 
  /trunk/configs/cdr_tds.conf.sample 253708 
  /trunk/funcs/func_cdr.c 253708 

Diff: https://reviewboard.asterisk.org/r/461/diff


Testing (updated)
-------

Compile test + Basic tests with cdr_custom.
Now tested with postgres and mysql modules.
Works fine with cdr_odbc and cdr_adaptive_odbc.
I now consider testing complete..


Thanks,

Brad


-- 
_____________________________________________________________________
-- Bandwidth and Colocation Provided by http://www.api-digital.com --

asterisk-dev mailing list
To UNSUBSCRIBE or update options visit:
   http://lists.digium.com/mailman/listinfo/asterisk-dev


[prev in list] [next in list] [prev in thread] [next in thread]