[asterisk-bugs] [Asterisk 0012285]: [patch] Permit 'not null' fields to always be set

[noreply at bugs.digium.com]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=12285 
====================================================================== 
Reported By:                Corydon76
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   12285
Category:                   CDR/cdr_pgsql
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     ready for testing
Asterisk Version:           SVN 
SVN Branch (only for SVN checkouts, not tarball releases):  trunk 
SVN Revision (number only!): 110578 
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             03-23-2008 22:47 CDT
Last Modified:              03-24-2008 17:16 CDT
====================================================================== 
Summary:                    [patch] Permit 'not null' fields to always be set
Description: 
Brian Capouch writes, via the -dev list:

I haven't built trunk for a while but tonight I took it up again.

The code builds fine against openWRT, but there seem to be some changes 
to the way that CDR works.  I use Postgres to store CDR records.  I 
can't find mention of the things mentioned below in CHANGES, in any of 
the documentation I could find online, or in the sample configs.

First, it appears that calldate is no longer allowed as a field name in 
the table which will hold the CDR record.  I couldn't get it to work 
unless I set the field name to "start."

There has also been a change in that the fields "accountcode" and 
"userfield" aren't being sent to the backend anymore.  This violates the 
"NOT NULL" constraints on those fields that permeate the documentation 
everywhere online.

I may have missed some critical information along the line, but if not, 
the codebase and existing documentation seem to be out of sync.
====================================================================== 

---------------------------------------------------------------------- 
 Corydon76 - 03-24-08 17:16  
---------------------------------------------------------------------- 
New patch uploaded.  Slightly mo' bettah. 

Issue History 
Date Modified   Username       Field                    Change               
====================================================================== 
03-24-08 17:16  Corydon76      Note Added: 0084467                          
======================================================================